System.Exception was thrown

Any ideas on why I am getting this:

The message is successfully posted to the Webhook, but I get an exception when trying to get the Result back.
Any ideas?

Yep, I've worked with API principles, though, like having a returned status code.
if (!response.IsSuccessStatusCode)
    throw new Exception();

I'll remove this part and it will works as expected.

Could you test with the new version 0.4.2 ? https://www.nuget.org/packages/gitter-api-pcl

Will do, thanks!

I have just upgraded to 0.4.2 and I am no longer getting the exception, however, I am always getting a false returned, when the message did actually post to my webhook url:

any ideas on what I am doing wrong here?

Hum, I will need to test the exact returned response. For now, I am doing that :
return result == "ok";

But, I never used webhook, so maybe it's not the right response, could you try the code ? You can use LinqPad and see the response with your own parameters.

ok, so this is part of the issue:

Will keep doing some digging....

Ok, maybe a good time to start with Integration Tests, at least with a fake room.

That's sounds like a plan to me.
I believe I have found the issue, I will work on implementing a fix, and I will submit a PR.

Nice !

Just in case, have you checked the returned status code ? It could help.

What version of Xamarin do you have installed?  I noticed that when I opened the solution, it made the following change:

Xamarin - 3.11.816.0
Xamarin.Android - <IP_ADDRESS>
Don't know where it comes from.


Getting this error when running test script

hello i don't know if this error has been posted before but i just downloaded tagui and install everything necessary but when i run a test script i get this error.. Used chatgpt to help solve but still not working .. anyone have a clue what i needs to do ...
/usr/local/bin/tagui: line 262: tagui_r/tagui_r.log: No such file or directory
chmod: tagui_r/tagui_r.log: No such file or directory
/usr/local/bin/tagui: line 263: tagui_r/tagui_r_windows.log: No such file or directory
chmod: tagui_r/tagui_r_windows.log: No such file or directory
/usr/local/bin/tagui: line 264: tagui_py/tagui_py.log: No such file or directory
chmod: tagui_py/tagui_py.log: No such file or directory
/usr/local/bin/tagui: line 265: tagui_py/tagui_py_windows.log: No such file or directory
chmod: tagui_py/tagui_py_windows.log: No such file or directory
/usr/local/bin/tagui: line 266: tagui.sikuli/tagui.log: No such file or directory
chmod: tagui.sikuli/tagui.log: No such file or directory
/usr/local/bin/tagui: line 267: tagui.sikuli/tagui_windows.log: No such file or directory
chmod: tagui.sikuli/tagui_windows.log: No such file or directory
Could not open input file: tagui_parse.php
ERROR - automation aborted due to above

Hi @nuprince is this still an issue? This sounds like some fundamental issue. If it is still an issue can you share more on the OS and how you download TagUI and how you run it, what test script?


List of available layers and attributes

Letting the client app to specify the layer (table name) and the attributes to retrieve is nice. But this is not easy for someone that does not have access to the database. It would be nice to have an access point that would list the available layers and attributes.

indeed, nice feature to add, i will look at this soon. It's a sort of introspection so i advocate to make it optionnal (a Postile option like --discover ?)

What would be the need for making it optional? (I can't see the drawbacks of such a function)

Migrated project, to thread is going on at https://gitlab.com/Oslandia/postile/-/issues/3


Add missing UVs to glTF draco loader extension

See https://forum.babylonjs.com/t/multiple-uv-sets-getting-lost-with-draco-compression/21830

Could we add a test for it ?

Could we add a test for it ?

We can, though I'm not sure how useful that will be.


Cheerp support

Website:
https://leaningtech.com/cheerp/
Alternative to emscripten.

@paulo-coutinho – very excited about the outcome from this! Hopefully it will offer something beyond emscripten all by itself.


Add autoconfigure with symfony flex

Add suport for symfony flex
Symfony Flex is used to autoconfigure symfony's packages, and it's quite useful when we are setup the lib in our project.
then I was wondering if we could add support for it :)
Describe the solution you'd like
Add a default config in https://github.com/symfony/recipes-contrib

Hi @eerison, thanks for the suggestion, we've added this to the backlog and will post here as soon as we have updates to share.

Hi @eerison, Just wanted to let you know that we have now added a default config recipe for BugSnag with Symfony. https://github.com/symfony/recipes-contrib/tree/main/bugsnag/bugsnag-symfony/1.13


Resharing with t parties only?

In a scene with {t, n} ECDSA KeyPair, is it possible to reshare if only t+1 parties coordinate in the process?

yes, it is possible


Regression: dots are shown in reversed colours

Bug report
Between v3.18 and v3.19, a regression was introduced that swapped the colours for the dots over the charting line. In this example, the red dot appears on the green line and vice versa.

Codepen
CodePen

This is still not fixed in 3.20.0.

Which browser?
I cannot reproduce.

Since updating the version in the CodePen, it seems fixed. However, it is not in working here for some reason...


[WebPubSub] Fix state object not correct.

Fix #25376
All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

API changes have been detected in Microsoft.Azure.WebPubSub.Common. You can review API changes here
API changes
+         public T GetState<T>(string stateKey);

API changes have been detected in Microsoft.Azure.WebPubSub.Common. You can review API changes here
API changes
+         public bool TryGetState<T>(string stateKey, out T value);

Which part is the most important for releasing a hotfix? The custom deserializer? I am wondering if it can't be a regular update to the GA library.

Split the PR and move the urgent part to here: https://github.com/Azure/azure-sdk-for-net/pull/25398

Split the PR and move the urgent part to here: #25398

The above points to this same PR. Is there another PR with must-fix changes we want in a hotfix?

Sorry about mixed the links, smaller PR is here: https://github.com/Azure/azure-sdk-for-net/pull/25427

I believe this PR can now be closed. @JialinXin?

Separate and fix in a few PRs. Close this one.


don't eagerly close the JobCollectContext/SearchContext if there were no rows

The searchContext will be closed indirectly if the JobCollectContext is
closed.
The JobCollectContext is closed after the Fetch and once we've a separate
FetchContext it can always be closed immediately anyway.

ack

pushed this to master as https://github.com/crate/crate/commit/cef08d5b92818534eac916f59d5166a5b03d1132


build(scanner): Do not hard-code the dependency on scanner plugins

Slightly change the logic for selecting the default scanner: If there is only a single scanner available, use that one, otherwise try to use ScanCode as usual.

Merging despite the unrelated NuGetFunTest failure.


Consider collaboration with @gitattributes org

There's the cool @gitattributes org which would be really cool 😎 to put this repo under to make it very pseudo-official. It undeniably has a better branding/name 😜

Yeah, I'm not opposed if they'll have me :slightly_smiling_face:


How to fill different colour to different parts of a svg graph...

I have a picture(svg fomart), which I want to split into several sections. Is it possible to independently color these sections using Manim?

SVGMobject is an array of vmobjects, so, you can select each one using something like:
svg = SVGMobject("file")
legs = svg[0:2]
arms  = svg[2:4]

SVGMobject is an array of vmobjects, so, you can select each one using something like:
svg = SVGMobject("file")
legs = svg[0:2]
arms  = svg[2:4]

Thanks a lot!...but it's too difficult for me ... is there more details about some example...=-=


Authentication Error for SQL Server (MSSQL) when Connection URL contains special characters

What language are you using?
Python
What version are you using?
python: 3.11.8
connectorx: 0.3.2
What database are you using?
Microsoft SQL Server (MSSQL)
What dataframe are you using?
Pandas and/or Polars
Can you describe your bug?
Authentication works with the SQL Alchemy engine but not ConnectorX with the same connection url
What are the steps to reproduce the behavior?
I have a username with the following special character:
\
I have a password with the following special characters:
!`>]
I am URL encoding both of them with quote_plus.
Database setup if the error only happens on specific data or data type
N/A
Example query / code
from sqlalchemy.engine import URL, create_engine

driver_string = 'FreeTDS'
db_username = "my\\username"
db_password = r'my_special_password_`!]>'
db_username_url = quote_plus(db_username)
db_password_url = quote_plus(db_password)
port = 1433
tds_version = 'auto'

connection_url: str = (
    URL
    .create(
        drivername="mssql+pyodbc",
        username=db_username_url,
        password=db_password_url,
        host=server,
        port=port,
        database=database,
        query={"driver": driver_string,
               "TDS_Version": tds_version,
               "UseNTLMv2": "Yes"}
    )
    .render_as_string(hide_password=False)
)

# This works
my_engine = create_engine(connection_url)
my_engine.connect()

# This doesn't work
cx.read_sql(connection_url, "SELECT 1", return_type='polars')

What is the error?
ERROR tiberius::tds::stream::token] Login failed for user my\username'. code=18456

I can confirm this bug happens for Trino as well.
Do we have this on the roadmap?

Any updates to this issue?

+1 same problem, even when using the options as recommended in the ConnectorX docs instead of +pyodbc. Connection works as usual in pyodbc and pymssql.


⚠ Spryr has degraded performance

In 15e2317, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4298 ms

Resolved: Spryr performance has improved in ec9fea4 after 7 minutes.


Feat: [M3-6713] Plan availability notices for GPU

Description 📝
This PR is a follow up to https://github.com/linode/manager/pull/9253 to add a dynamic version of the GPU Plans support.
It also

fixes a regression with introduced in the previous PR where we would not get the default proper plan selected when cloning or using a backup
fixes the styling (height) of the notice
add the missing UI for the Linode Resize modal

Preview 📷

Before
After

How to test 🧪

⚠ This PR affects ALL linode creation flows (including K8 and node pools). Each flow should be tested for consistency, on both desktop and mobile

The verification steps underneath should be performed on the following paths:
Linode Create:

/linodes/create
/linodes/create?type=One-Click
/create?type=StackScripts
/linodes/create?type=Images
/linodes/create?type=Backups
/create?type=Clone%20Linode
//linodes/[id] click resize

Kubernetes Create

/kubernetes/create

Existing Cluster Add node pool (you need to test this with both a cluster in a non-eligible region and a cluster in a Premium Plans regions

/kubernetes/clusters/[id]/summary

Verification steps

Confirm this behavior on the GPU tab when no region is selected
Confirm this behavior with a non eligible premium region (ex: Washington DC)
Confirm no message is displayed and form is enabled with a premium plans region (ex: Newark)
Confirm no regression to the existing premium plans tab

Steps and UI may vary slightly depending on the creation flow (ex Backups) but the behavior should be consistent across all.

@mjac0bs @cpathipa @bnussman-akamai feedback addressed, thank you!


don't add a key when stringifying arrays

Resolves #210
Not completely familiar with the process described for testing. I've generated the docker file but am unsure of how to actually run npm test inside the container. Any advice is appreciated so I can add tests for this functionality too ❤

Ideally a test would ensure that the receiving party gets the expected result, e.g.:
http.createServer(function(req, res) {
  // parse request body and maybe return a JSON representation
  res.end(JSON.stringify(parsed))
})

var test_data = { foo: 'bar', list: [...] };
needle.post(server, test_data, function(err, resp, body) {
  should.not.exist(err);
  body.list.should.be.an.Array;
  body.list.length.should.eql(3);
}

Oh, and you don't really need docker to run the test suite. Just npm install and then npm test. You can run a specific one by doing node_modules/.bin/mocha test/my_test.js.

The tests I've added pass (as do the other tests for the stringify method) but as I use MacOS as a dev environment, so just wanted to make sure my changes weren't breaking other functionality. Sorry for any confusion!

Ok, I'll take a look at this when I have a minute. Thanks

Hey @tomas, just checking in to see if there's anything I can do to help get this PR merged? I know managing open source projects can be a bit of a chore so if there's anything I can do please let me know 💖

Hey @davidlumley, I totally forgot about this, sorry. Will test and merge right away.


Feature: get-rgb-brightness

Is there a way to know this actual state of the light ?
Like having a get-rgb-brightness function ?

Unfortunately, afaik, that is not possible.


[Bug]: project initialized with older devbox has error on init

Current Behavior (bug)
I used an older version of devbox to init a project. That same project is throwing an error when I enter it:

Error: Command: /nix/var/nix/profiles/default/bin/nix print-dev-env /Users/alan/src/my-project/.devbox/gen/flake/flake.nix --extra-experimental-features ca-derivations --option experimental-features nix-command flakes --json: exit status 1

I believe this is due to a new version.
I had a very similar error before but it worked itself out. This problem persisted and the only way I could fix it was to rm -rf .devbox && rm .envrc && devbox init. My setup is now working again.

output with DEVBOX_DEBUG=0:
➜  src export DEVBOX_DEBUG=1
➜  src cd my-project
direnv: loading ~/src/my-project/.envrc
direnv: using devbox
direnv: using nix .devbox/gen/shell.nix
config.go:133: findProjectDir: path is
config.go:174: finding devbox.json in dir: /Users/alan/src/my-project
[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/boxcli/midcobra/debug.go:49: Debug mode enabled.
[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/impl/config.go:133: findProjectDir: path is
[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/impl/config.go:174: finding devbox.json in dir: /Users/alan/src/my-project
[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/impl/devbox.go:474: current environment PATH is: /nix/store/xzv575br516sbj3w3lhqcf7yw1r1n6gq-bash-interactive-5.2-p15/bin:/nix/store/r01krwydyya0zkdc68qwp8j2i5lym0aq-nodejs-18.13.0/bin:/nix/store/zfqf5q4xwrs7r7nwl2xqxr46r2k184i8-git-2.39.0/bin:/nix/store/igbnksv215w3nxrxdpkdhwnsna98crdx-git-branchless-0.6.0/bin:/nix/store/bwscr98440mhhlg556fqhp5ll2fxnfic-clang-wrapper-11.1.0/bin:/nix/store/ssfpjhrysqf6ywz5sqi8gpz28zc23kiy-clang-11.1.0/bin:/nix/store/gvl9w6c5q83nd4svz5v5hqhgfzlcpl3y-coreutils-9.1/bin:/nix/store/xh503dqb1yb5w9mjziclp81gi4cqmwf6-cctools-binutils-darwin-wrapper-973.0.1/bin:/nix/store/a17dwfwqj5ry734zfv3k1f5n37s4wxns-cctools-binutils-darwin-973.0.1/bin:/nix/store/gvl9w6c5q83nd4svz5v5hqhgfzlcpl3y-coreutils-9.1/bin:/nix/store/x080znwmkginz9xb1qchm5dd188m59hg-findutils-4.9.0/bin:/nix/store/pnp0lkyjhrhl7vj0fgrpx80zzzmnkqi6-diffutils-3.8/bin:/nix/store/vjnvxrpi26aigy96fvcfwjx5y5j18h7r-gnused-4.9/bin:/nix/store/8j799mid9ai29hg4s4xifx3yn1x1qply-gnugrep-3.7/bin:/nix/store/40sbwcrzl80vrlz9wwdml69b6y224kbs-gawk-5.2.1/bin:/nix/store/wmybxxsdc4h640xgzijmqi1yic3fvs6x-gnutar-1.34/bin:/nix/store/3x74r0m2833bqr7krjf4q8q2wpb5l3v7-gzip-1.12/bin:/nix/store/8wdl82qvvd6hkq361vq8cmi110g3pbvg-bzip2-1.0.8-bin/bin:/nix/store/6lkc7zwfkr21si1g85mq39mmlpf6mn35-gnumake-4.4/bin:/nix/store/n0k8njvgg6yjapkl81rm821s9vx0qrwb-bash-5.2-p15/bin:/nix/store/m5dhzr6yzq199ahyiv3xqpiy4japhr9f-patch-2.7.6/bin:/nix/store/dd4776lwwki6dxrjn2cli3sc9mwmdxp5-xz-5.4.0-bin/bin:/nix/store/4zhg5g546pa5gprszifqz1lx9vg8h8nv-file-5.43/bin:/Applications/Sublime Text.app/Contents/SharedSupport/bin:/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Library/Apple/usr/bin:/Users/alan/.nix-profile/bin:/nix/var/nix/profiles/default/bin
[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/nix/nix.go:136: Running print-dev-env cmd: /nix/var/nix/profiles/default/bin/nix print-dev-env /Users/alan/src/my-project/.devbox/gen/flake/flake.nix --extra-experimental-features ca-derivations --option experimental-features nix-command flakes --json
Error: Command: /nix/var/nix/profiles/default/bin/nix print-dev-env /Users/alan/src/my-project/.devbox/gen/flake/flake.nix --extra-experimental-features ca-derivations --option experimental-features nix-command flakes --json: exit status 1

[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/boxcli/midcobra/debug.go:70: Command stderr: path '/Users/alan/src/my-project/.devbox/gen/flake/flake.nix' does not contain a 'flake.nix', searching up
error: getting status of '/Users/alan/src/my-project/.devbox/gen/flake/flake.nix': No such file or directory

[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/boxcli/midcobra/debug.go:72:
ExecutionID:15adb64df9844cf59aea935fc9f639e1

go.jetpack.io/devbox/internal/nix.PrintDevEnv
	/home/runner/work/devbox/devbox/internal/nix/nix.go:139
go.jetpack.io/devbox/internal/impl.(*Devbox).computeNixEnv
	/home/runner/work/devbox/devbox/internal/impl/devbox.go:483
go.jetpack.io/devbox/internal/impl.(*Devbox).PrintEnv
	/home/runner/work/devbox/devbox/internal/impl/devbox.go:238
go.jetpack.io/devbox/internal/boxcli.runShellCmd
	/home/runner/work/devbox/devbox/internal/boxcli/shell.go:56
go.jetpack.io/devbox/internal/boxcli.ShellCmd.func1
	/home/runner/work/devbox/devbox/internal/boxcli/shell.go:33
github.com/spf13/cobra.(*Command).execute
	/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.6.1/command.go:916
github.com/spf13/cobra.(*Command).ExecuteC
	/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.6.1/command.go:1044
github.com/spf13/cobra.(*Command).Execute
	/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.6.1/command.go:968
go.jetpack.io/devbox/internal/boxcli/midcobra.(*midcobraExecutable).Execute
	/home/runner/work/devbox/devbox/internal/boxcli/midcobra/midcobra.go:67
go.jetpack.io/devbox/internal/boxcli.Execute
	/home/runner/work/devbox/devbox/internal/boxcli/root.go:75
go.jetpack.io/devbox/internal/boxcli.Main
	/home/runner/work/devbox/devbox/internal/boxcli/root.go:84
main.main
	/home/runner/work/devbox/devbox/cmd/devbox/main.go:11
runtime.main
	/opt/hostedtoolcache/go/1.19.7/x64/src/runtime/proc.go:250
runtime.goexit
	/opt/hostedtoolcache/go/1.19.7/x64/src/runtime/asm_arm64.s:1172
[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/impl/config.go:133: findProjectDir: path is
[DEBUG] 2023/03/24 13:01:50 /home/runner/work/devbox/devbox/internal/impl/config.go:174: finding devbox.json in dir: /Users/alan/src/my-project
direnv: export +AR +AS +CC +CONFIG_SHELL +CXX +DEVBOX_SHELL_ENABLED +HOST_PATH +IN_NIX_SHELL +LD +LD_DYLD_PATH +MACOSX_DEPLOYMENT_TARGET +NIX_BINTOOLS +NIX_BINTOOLS_WRAPPER_TARGET_HOST_aarch64_apple_darwin +NIX_BUILD_CORES +NIX_BUILD_TOP +NIX_CC +NIX_CC_WRAPPER_TARGET_HOST_aarch64_apple_darwin +NIX_CFLAGS_COMPILE +NIX_COREFOUNDATION_RPATH +NIX_DONT_SET_RPATH +NIX_DONT_SET_RPATH_FOR_BUILD +NIX_ENFORCE_NO_NATIVE +NIX_HARDENING_ENABLE +NIX_IGNORE_LD_THROUGH_GCC +NIX_INDENT_MAKE +NIX_LDFLAGS +NIX_NO_SELF_RPATH +NIX_STORE +NM +NODE_PATH +PATH_LOCALE +RANLIB +SIZE +SOURCE_DATE_EPOCH +STRINGS +STRIP +TEMP +TEMPDIR +TMP +XDG_DATA_DIRS +__darwinAllowLocalNetworking +__impureHostDeps +__propagatedImpureHostDeps +__propagatedSandboxProfile +__sandboxProfile +__structuredAttrs +buildInputs +buildPhase +builder +cmakeFlags +configureFlags +depsBuildBuild +depsBuildBuildPropagated +depsBuildTarget +depsBuildTargetPropagated +depsHostHost +depsHostHostPropagated +depsTargetTarget +depsTargetTargetPropagated +doCheck +doInstallCheck +mesonFlags +name +nativeBuildInputs +out +outputs +patches +phases +propagatedBuildInputs +propagatedNativeBuildInputs +shell +shellHook +stdenv +strictDeps +system ~PATH ~TMPDIR

Expected Behavior (fix)
devbox upgrades don't break old projects (I can't put too fine a point on this. It is super fundamental to the value-add of devbox.)
Additional context
Please include the output of devbox version -v and
a copy of your devbox.json file.
$ devbox version -v
Version:     0.4.4
Platform:    darwin_arm64
Commit:      469cd559949b0d9f3af284a5056ea5606c0cf9ce
Commit Time: 2023-03-16T16:57:27Z
Go Version:  go1.19.7
devbox.json
{
  "packages": [
    "nodejs",
    "git",
    "git-branchless"
  ],
  "shell": {
    "init_hook": null
  },
  "nixpkgs": {
    "commit": "f80ac848e3d6f0c12c52758c0f25c10c97ca3b62"
  }
}

envrc
# Automatically sets up your devbox environment whenever you cd into this
# directory via our direnv integration:

use_devbox() {
    watch_file devbox.json
    if [ -f .devbox/gen/shell.nix ]; then
        DEVBOX_SHELL_ENABLED_BACKUP=$DEVBOX_SHELL_ENABLED
        use nix .devbox/gen/shell.nix
        eval $(devbox shell --print-env)
        export DEVBOX_SHELL_ENABLED=$DEVBOX_SHELL_ENABLED_BACKUP
    fi
}
use devbox

# check out https://www.jetpack.io/devbox/docs/ide_configuration/direnv/
# for more details

Hey @altano. Thanks for reporting. Agreed that we should not be breaking the project on new versions, and it seems like we're having some challenges with the direnv integration in particular.
I'm trying to pin this bug down -- do you know which version of Devbox you used to generate the project originally?

Agreed that we should not be breaking the project on new versions, and it seems like we're having some challenges with the direnv integration in particular.

Awesome, I'm glad to hear we're on the same wavelength.

I don't know, and because I shuffled machines around recently AND I'm so senile, I have no way of approximating. I think it was this year? heh.
If there's a way to install an old version of devbox and disable auto-updating, let me know and I would be happy to dig in here and try and reproduce the issue.

@altano thanks for reporting, I can repro your issue by manually deleting my flakes.nix. Devbox switched to using flakes in 0.4.3 so an older .devbox directory can cause this issue.
This affects direnv and any other use cases that use devbox shellenv, but without ever running devbox run or devbox shell.
A quick fix to this issue is calling
devbox generate debug which regenerates files in .devbox dir.
I opened a PR for a fix as well.

@altano thanks for reporting the bug, and apologies for the lapse. I wanted to echo John in that we're 100% in agreement around having better stability. We'll be instituting some changes in the coming weeks so that this starts noticeably improving.


I am using Angular 6, ng2-select is not showing any data

I started with a new project and using angular 6, tried to use ng2-select but after binding the data it does not show in the dropdown.
when I look at element in dev tools I see the data bind but it dows not pull on the ui.
here is my simple code..
<ng-select [multiple]="true"
[allowClear]
[items]="items"
placeholder="No item selected">

items: Array = [
{ "id": 1, "text": "Table" },
{ "id": 2, "text": "Chair" },
{ "id": 3, "text": "Light" }
];

I'm also experiencing this issue right now, using the element as follow:
<ng-select [items]="items" [disabled]="isSubmitting" [active]="active" (data)="refreshValue($event)" (selected)="selected($event)" (removed)="removed($event)"  (typed)="typed($event)" placeholder="Search..."></ng-select>
The select element has a charcoal-grey background and white text when not active, which may be an unrelated issue, but typing in the search input doesn't display any items even though I can inspect the component using Augury (Chrome extension for Angular dev) and see that the items are being returned correctly by my service.
Oddly enough, if I hit enter while focused on the ng2-select element it does set the first item as selected and displays that item's text, I just can't see the search results to choose from.

I am not sure how to create it on plunkr as requested by BenjaminRichards

i think the problem is that you are using bootstrap 4.x
this works fine:
https://stackblitz.com/edit/angular-ghvjfm
but if you would change that it used 4x instead of 3.x then it doesn't work anymore
That is also a problem for us because we build fully now in bootstrap 4.

You were saying that it does not work on bootstrap 4.x, correct?

Most likely related to this
`.open {

.dropdown-menu {
display: block;
}

a {
outline: 0;
}
}`

I was able to get the dropdown showing with the following overriding CSS:
.open.dropdown .dropdown-menu { 	display: block; 	transform: none; }

Hello, I'm using the bootstrap 4 as well, I manage to set up select multiple with dynamic items however the match-item don't use the correct style and button:

Any idea how I could override the button style for match-item ? :-)

Most likely related to this
`.open {

.dropdown-menu {
display: block;
}

a {
outline: 0;
}
}`

I still can't show the list. Anyone knows the answer? :( help meeee

any solution for this issue??

on typed event i set display= 'block'
 public typed(value:any):void {     console.log('New search input: ', value);     // let el = document.getElementsByClassName('.ui-select-choices dropdown-menu') as HTMLCollectionOf<HTMLElement>;;     let el:HTMLElement = document.getElementsByClassName("ui-select-choices dropdown-menu")[0] as HTMLElement;     if (typeof el !== 'undefined') {       el.style.display = 'block';     }   }


Tracking our first production recipe run

The first production run of this feedstock (and indeed, our first automatically-triggered production run ever) is now running. Its progress is tracked here:
https://pangeo-forge.org/dashboard/recipe-run/26
Here are some notes on user experience, registration bugs, etc. that I've noticed in starting this run:

Unlike for recipe tests of staged-recipes PRs, @pangeo-forge-bot does not currently post a notification about production builds for merged feedstocks. Perhaps in the future, the bot should open an Issue like this on the associated feedstock, with details about the run such as a link to the recipe run page (as provided above).
On the linked recipe run page, I notice that the version is incorrectly provided as "0.0". I'll fix that to reflect the actual tag from which the run was triggered.
Similarly, I've triggered the run from the tag (and not a commit sha). This tag ended up as the value of the recipe run table's head_sha field. This should be corrected as well.

Unlike for recipe tests of staged-recipes PRs, @pangeo-forge-bot does not currently post a notification about production builds for merged feedstocks.

This might be a good use for the github deployments. Ideally we could just link a user back to a page on pangeo-forge.org that has all of the feedback / info about the attempted run. This would probably require https://github.com/pangeo-forge/pangeo-forge-orchestrator/issues/62.

On the linked recipe run page, I notice that the version is incorrectly provided as "0.0". I'll fix that to reflect the actual tag from which the run was triggered.
Similarly, I've triggered the run from the tag (and not a commit sha). This tag ended up as the value of the recipe run table's head_sha field. This should be corrected as well.

I think we should iterate a bit on whether it really makes sense to tie runs to repo tags. That was just an idea we had. But maybe just using commits would be easier. We already have a "version" in meta.yaml, and it seems confusing to also have a tag version.

This might be a good use for the github deployments.

I like this idea.
And yes, switching to running from the latest commit of a push to the default branch makes sense. Should I just make that change or is there anything further worth considering about it?

Just make the change to commits. It's clearly the right path to me.

The first production run of this feedstock (and indeed, our first automatically-triggered production run ever) is now running.

😱 I had no idea this was the first automatically triggered run!

Just make the change to commits. It's clearly the right path to me.

This might be a good use for the github deployments. Ideally we could just link a user back to a page on pangeo-forge.org that has all of the feedback / info about the attempted run. This would probably require https://github.com/pangeo-forge/pangeo-forge-orchestrator/issues/62.

Both of these points

trigger production runs on pushes to default branch of feedstock repo (not tags)
use deployments to link users back to recipe run page

were implemented in https://github.com/pangeo-forge/registrar/pull/23. We can see a successful end-to-end demo of these features in this test repo, where clicking Deployed here:

brings us to https://pangeo-forge.org/dashboard/recipe-run/27.
This real feedstock (i.e. noaa-coastwatch-geopolar-sst-feedstock) also has a deployments page:

and similarly, clicking Deployed there brings us to https://pangeo-forge.org/dashboard/recipe-run/28. In the case of this real feedstock, the deployments API integration has worked in the sense in that it reflects the correct exit state (failed). In terms of why this production run has failed, the Prefect logs that are piped over to pangeo-forge.org aren't helpful (perhaps there are too many lines to full resolve them?). I've glanced at the logs on the Prefect backend, which tell a more complete story (with tracebacks, etc.) but diagnosing what's gone wrong there will have to wait until Monday.
So in sum, as of the time of writing this, we can now look for to the Environments sidebar header on any feedstock repo

and click through to the Deployments page to find a link to the latest production deployments, which link us to the associated recipe runs on pangeo-forge.org.
There are all sorts of aesthetic and clarity problems to clean up here but I don't think this issue is the right place to track them, so closing this.

One last thought, it looks like conda-forge has the same "commit aesthetics" scenario we ended up with: regardless of who has contributed to the staged-recipes PR, the bot account is the one that is credited with all of the commits for creating the repo; e.g. https://github.com/conda-forge/pangeo-forge-recipes-feedstock/graphs/contributors


CloudWatch StatisticSet: floating point values are not correctly URL encoded

https://github.com/awslabs/aws-sdk-cpp/blob/master/aws-cpp-sdk-monitoring/source/model/StatisticSet.cpp#L90
This produces output that looks like StatisticValues.Sum=5.88181e+07, which causes MalformedInput exception. The + should probably be encoded as %2B.
I'm currently working around it with this patch to remove the scientific notation:
--- PutMetricDataRequest.cpp  2016-02-11 16:12:59.000000000 -0800
+++ PutMetricDataRequest.2.cpp  2016-02-11 16:19:20.000000000 -0800
@@ -12,6 +12,8 @@
 * express or implied. See the License for the specific language governing
 * permissions and limitations under the License.
 */
+#include <iomanip>
+
 #include <aws/monitoring/model/PutMetricDataRequest.h>
 #include <aws/core/utils/StringUtils.h>
 #include <aws/core/utils/memory/stl/AWSStringStream.h>
@@ -29,6 +31,7 @@
 {
   Aws::StringStream ss;
   ss << "Action=PutMetricData&";
+  ss << std::fixed << std::setprecision(std::numeric_limits<long double>::digits10 + 1);
   if(m_namespaceHasBeenSet)
   {
     ss << "Namespace=" << StringUtils::URLEncode(m_namespace.c_str()) << "&";

I can look at adding this to the generator in the morning. Thanks for the report.

I'd try to stick to the scientific notation rather than using fixed precision since the latter can't represent all possible double values unless the number of digits is set to some extreme value. For example the number 1e-300 would need over 300 digits to represent accurately in fixed precision.

So the behavior of all of the other static language sdks is for everything less than 10^-3 and greater than 10^7 to be scientific notation. I will keep this behavior, but make sure it is URL encoded.

ok, give that a try. I think it should be good to go.

Yup it's fine now, thanks!
StatisticValues.Sum=3.3535e%2B33


Proposal: ScrollViewer - Support Offset (begin and end) for scrollbar

Proposal: ScrollViewer should support BeginOffset and EndOffset for scrollbar

Summary
Sometime developers need to overlay a control on top of ScrollViewer. In these situations it is good to reduce the size of the scrollbar by setting some kind of begin and end offset.
Below is an example of a media app displaying the media control overlay with Acrylic Background on a ScrollViewer (ListView)

 Today                                                    Proposal utilising end offset

A Navigation Header with Acrylic background

 Today                                                    Proposal utilising begin offset

Extending ListView into Titlebar

 Today                                                    Proposal utilising begin offset

Rationale

The alternative to this proposal is to override the ControlTemplate which is messy and difficult to be kept in sync with UI changes happening in WinUI.
It is a popular pattern many apps do it. Groove did it with its media controls. Settings app does it with the header

Scope

Capability
Priority

Allow to set the begin and end offset
Must

Expression animations that was taking scroll position as input should not be impacted
Must

Animate when the offset changes (if at that time scrollbar is visible)
Should

Important Notes

   public double BeginOffset {get;set;}
   public double EndOffset {get; set; }

Open Questions

Thanks for the feedback. The existing ScrollViewer.CanContentRenderOutsideBounds can indeed come handy to achieve this behavior. I just played with it a bit and came up with this sample code. I did little testing with it and I am not sure it accounts for all circumstances though. Some values are hardcoded but you get the idea:
XAML fragment:
<ContentControl x:Name="root">
    <Grid>
        <Grid.RowDefinitions>
            <RowDefinition Height="Auto"/>
            <RowDefinition Height="Auto"/>
        </Grid.RowDefinitions>
        <ScrollViewer x:Name="sv" Background="Yellow" Width="300" Height="600"
            CanContentRenderOutsideBounds="True"
            HorizontalScrollBarVisibility="Auto" VerticalScrollBarVisibility="Auto"
            HorizontalScrollMode="Auto" VerticalScrollMode="Auto"
            ZoomMode="Enabled" Grid.Row="1">
            <StackPanel x:Name="sp" Margin="0,-80,0,0">
                <StackPanel Orientation="Horizontal">
                    <Rectangle Width="200" Height="250" Fill="AliceBlue"/>
                    <Rectangle Width="200" Height="250" Fill="Beige"/>
                    <Rectangle Width="200" Height="250" Fill="Aquamarine"/>
                </StackPanel>
                <StackPanel Orientation="Horizontal">
                    <Rectangle Width="200" Height="250" Fill="Bisque"/>
                    <Rectangle Width="200" Height="250" Fill="Aqua"/>
                    <Rectangle Width="200" Height="250" Fill="AntiqueWhite"/>
                </StackPanel>
                <StackPanel Orientation="Horizontal">
                    <Rectangle Width="200" Height="250" Fill="LightBlue"/>
                    <Rectangle Width="200" Height="250" Fill="BlanchedAlmond"/>
                    <Rectangle Width="200" Height="250" Fill="Azure"/>
                </StackPanel>
                <StackPanel Orientation="Horizontal">
                    <Rectangle Width="200" Height="250" Fill="AliceBlue"/>
                    <Rectangle Width="200" Height="250" Fill="Beige"/>
                    <Rectangle Width="200" Height="250" Fill="Aquamarine"/>
                </StackPanel>
                <StackPanel Orientation="Horizontal">
                    <Rectangle Width="200" Height="250" Fill="Bisque"/>
                    <Rectangle Width="200" Height="250" Fill="Aqua"/>
                    <Rectangle Width="200" Height="250" Fill="AntiqueWhite"/>
                </StackPanel>
            </StackPanel>
        </ScrollViewer>
        <Border Background="Brown" Opacity="0.25" Height="80" Width="300"/>
        <Border Height="80" Width="300">
            <TextBlock Text="HEADER" FontSize="30" HorizontalAlignment="Center"
                VerticalAlignment="Center" Opacity="0.6"/>
        </Border>
    </Grid>
</ContentControl>

C# code fragment:
public MainPage()
{
    this.InitializeComponent();

    RectangleGeometry rg = new RectangleGeometry();
    Rect newClipRect = new Rect(0, 0, 300, 680);
    rg.Rect = newClipRect;
    this.root.Clip = rg;

    this.sv.ViewChanged += Sv_ViewChanged;
}

private void Sv_ViewChanged(object sender, ScrollViewerViewChangedEventArgs e)
{
    this.sp.Margin = new Windows.UI.Xaml.Thickness(0,-80 / sv.ZoomFactor,0,0);
}

Does this work for you?   Note that the current ScrollBar control is not Composition/animation friendly. Its height cannot be animated for instance.
It is true that this CanContentRenderOutsideBounds property is currently missing in the new scrolling controls (to be named ScrollView and ScrollPresenter according to https://github.com/microsoft/microsoft-ui-xaml-specs/pull/56). I do think that we should be adding that property to the new controls in the future. We could use this issue #1914 to track that work.
At this point I am not convinced that this is a common enough scenario that it warrants additional dedicated properties like BeginOffset/EndOffset.

Big thank you @RBrid for the sample, it was most helpful. I think it should solve most of my use cases. Can we hope CanContentRenderOutsideBounds  would be available when the new ScrollViewer would come out of preview? I really can't use the new OS ScrollViewer as my min target is 1803.

@harvinders, glad the sample is helpful. I am afraid the first out-of-preview release of the ScrollView control will be without the CanContentRenderOutsideBounds property. We have pretty much locked down its public object model and feature set. Maybe the second release will have that property added.


NIOSSLClientHandler should not have a default have for serverHostname

It's too easy to forget to pass serverHostname so we shouldn't make it have a default value. Users can still pass nil in there but they should be required to pass something
https://github.com/apple/swift-nio-ssl/blob/45fa2eb1afc325ceaf9e2ea077bddce25b11271a/Sources/NIOSSL/NIOSSLClientHandler.swift#L34

@ktoso this could maybe be your prob?

I'll check :)


The package doesn't work on Meteor 1.3.3 or later.

It may be same as #43, I made sure that the package didn't work on Meteor 1.3.3 or later.
This package is an important key for my app.
Are there way to work on latest meteor?
Thank you.

@y-ich Did you find a way to work around this?

I would suggest to switch to: https://github.com/RocketChat/meteor-streamer

@mitar Thanks for the very fast reply. In fact, I found Streamer at the very moment you replied. So thanks again for the confirmation that this will be the way to go :-)


Instagram original quality

Using this command:
instaloader -- -CZVEugIPkVn

I get result image 1080x1080 - however with other tools, I am able to get 1241x1241. Can instaloader get the original quality?

You have to provide a login name and  a password (or enter the password when asked) to get the original size. Up to 1440x1440 for post if the original picture is big enough. Stories can go higher with over 2000px.
Which other tools did you use to get 1241x1241? To my knowledge there are not many working instagram scrapers.
Care to tell us which other tools are able to

https://github.com/89z/mech


ios support

hope you can add support for ios

hope you can add support for ios

I will as soon as I can


Support wrapping classes with self references

This addresses #1, though I think this is not enough as a general solution: it only deals with recursive references, not "real" forward references, e.g.:
@dec
class A:
    b: B
class B:
    ...

A more viable solution might be to not resolve hints at decoration-time, but only store the __annotations__. As long as forward references are resolvable when __init_subclass__ is invoked, that might work ... 🤔

Codecov Report

Merging #3 (f2e8054) into main (7d59da7) will decrease coverage by 1.47%.
The diff coverage is 94.44%.

@@             Coverage Diff             @@
##              main       #3      +/-   ##
===========================================
- Coverage   100.00%   98.52%   -1.48%
===========================================
  Files            3        4       +1
  Lines          103      136      +33
===========================================
+ Hits           103      134      +31
- Misses           0        2       +2

Impacted Files
Coverage Δ

abcattrs/abcattrs.py
97.50% <85.71%> (-2.50%)
:arrow_down:

abcattrs/type_hints.py
94.11% <94.11%> (ø)

tests/test_decorator.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Reformatting code is left-aligning comments.

Highlight and reformat (Code->Reformat Code) any code containing a comment will place the single-line comments against the left-most column (removes indenting) or will left-align the first line of a block comment.  Copy and paste will left align comments in the pasted block as well.
TIR: STB-10533

This also happens for me when I type "/**" then hit enter to auto generate a comment block.  First line is always at column 0.  (IntelliJ 14.1.3, plugin 0.9.4)  Btw, same operation works fine for Java.

The code that does reformatting is IDEA's DefaultCodeFormatter (IIRC).  The PSI for comments is likely not what the formatter is expecting.  I suspect, but I haven't yet confirmed, that the change for #209 is causing the issue.
Incidentally, the unit tests do contain this scenario and are NOT failing.  Therefore, I'm half-expecting some secondary processing step to be re-aligning incorrectly.  Stepping (debugging) through unit test code did not reveal anything unusual.

https://github.com/TiVo/intellij-haxe/blob/ImplementNestedFunctionTypesSupport/src/common/com/intellij/plugins/haxe/ide/formatter/HaxeIndentProcessor.java#L59-59

If I got it right, then I think that it works in a proper way, single line comments in IDEA added from line start, so no problem with that, but if it's added in some other IDE, then this may happen.

@wiggin77 /** indeed works wierd, but /* works just fine

If I got it right, then I think that it works in a proper way, single line comments in IDEA added from line start, so no problem with that, but if it's added in some other IDE, then this may happen.

I don't understand.  Are you saying that single line comments should be aligned on the left-most column (column 1) when the code is reformatted?  If so, then I disagree.
Right now, this:
class Test {
    // a comment
    public function new() {}
}

is reformatted to this:
class Test {
// a comment
    public function new() {}
}

If the comment started in the left-most column (column 1), then it should stay there.  If the comment has any indentation, then it should be indented to match the code.

+1 I had that problem. All my comments were moved to the left-most column and it was a bit annoying.

I checked how it works in Java indeed it works differently
Java:

Notice where it inserts comment (// part)
Always indents code on enter
Haxe:

If I hit enter after method body - indentation is wrong here
Also on comment (//) gets inserted at the beginning of line

Fix is in #372.  Awaiting review.

Fixed with #372 and merged.


Suggestion: Expanded settings for quests.

What does your feature do?
Additional settings for a quest that a. forces user to accept a quest on region entry, and b. enables/disables 'ask to accept'.
How would people use it?
a. forced tutorials, when a player enters the tutorial area, the initial tutorial quest is pushed on them. then the rest can be done with stages. Or dungeons in the world, using entry on dungeon to activate it to control rewards in found dungeons.
and 2. disabling the 'would you like to accept this quest' on a per quest basis would allow users to use WorldGuardExtraFlags to run 'command on entry' and do the same thing as above.

Region entry is already available in the Quests Editor, see option '6' below:

Toggling 'ask to accept' is known as ask-confirmation in config.yml:
https://github.com/PikaMug/Quests/wiki/2-‐-Configuration#configyml

'Set Region' only changes the fact that they 'have to be in the region to accept the quest' it DOESN'T make them take the quest.
And toggling 'ask to accept' is only global, not per quest as I requested.

Looks like I misinterpreted your suggestion - thanks for the clarification.
As you’ve stated, forcing players to take a quest when entering a region should be done with the /questadmin give [player] [quest] command using a plugin like WorldGuardExtraFlags.
Making quest confirmation work on a per-quest basis has been suggested before, but declined with the reasoning that players should have a consistent experience when taking quests.
I’d be glad to reopen this ticket if you’d care to explain why having confirmation per-quest would be preferred over disabling it for all quests.
Sent with GitHawk

I think my issue with the confirmation per-quest actually comes from a (likely non-intended) use of quests.
I want to setup 'danger zones' around the world (IE: dungeons) that on entry activate a quest (IE: Starts spawning of enemies, manages loot, rewards, ect) without the immersion breaking 'yes/no' confirmation (since nothing is 'offering' the quest, it's just there.) But I also want to have quest givers around the world (in cities, towns, and on the roads) that can give quests using the normal quest giving acceptance.
Having the option to have quest confirmation per-quests lets me do both.
I can have zone based forced quests, and npc based optional quests.
(also things like, forced tutorials on server entry, or entry into a new city.
They get put in the tutorial area on first join, and have to go through the quest steps to exit; simple psychology, if they have the option to skip its more likely they are going to end up breaking the rules, and causing trouble.)

That sounds very interesting, but I'm a little confused. You've made it clear that ask-confirmation should be disabled for the tutorial and danger zones, but what is the advantage of then enabling confirmation for city/town/road quests? Were you planning on disabling quitting of those quests?

Advantages of having the ability to turn it on would be quests like the stereotypical fetch quests. Or npc 'I need a iron sword, can you make me one?' 'yes/no'.
An example of a city quest that could be done without ask-confirmation is getting to a new city, and needing to kill X amount of mobs outside of it before your let in. (you get close, guards stop you, auto give quest. then you only get a Luckperms track increase to get inside town if you do the quest.)
and i'm fully okay with players quitting the quests after their given. (ie: in above they decide its not worth it, so just leave and quit the quest.) It's them giving up opportunities, so i'm not worried about that in my case.
--
although now that I think about it, if I could disable quitting on a per-quest basis. you could do some fun stuff with that. Like 'questing' someone with a disease that harms them over time (a delay until stage increase) or having a potion that effects someone with 'lure' (ie: gives them a quest that auto spawns mobs until they complete the time of potion, or a objective of some sort.)
You could even use that per-quest disable quitting to put 'bounties' on players. Where the player loses the quest on death by player, and drops a reward (making them the target).
Hmmm, Is that already in quests? That sounds useful.

I'm sticking by my original stance that requesting confirmation on some quests, rather than all or none, is not conducive to a consistent user experience. Also, the ability to disable per-quest quitting was added under Edit Options -> General some time ago, which partially satisfies this ticket. If avoiding confirmation is still desired, running /questadmin give as mentioned previously should override the config option in question.


integration of reva and ocis

At ownCloud we want to integrate and orchestrate the different reva handlers as micro services. Unfortunately, the business logic inside the handlers is not reusable because they live in the internal namespace.
Another way of integrating reva and micro would be to start reva with a configuration generated by micro, eg. for the host and port names of the relevant services. However, this is currently not possible for multiple reasons:

reva has no way of injecting the micro.Server() into the rgrpc.NewServer(). reva always creates the underlying grpc service.
Same for micro. It also wants to stay in control of the grpc server because it also wants to add tracing etc.
For the same reason micro has no way of fetching host and port information after initializing a service that we could use to create a configuration for reva that would allows us to treat reva as a microservice.

Yet another option is to start reva as a binary, limit it to a single service and tie it to a local/unix socket. We could implement the micro service as an adapter that makes calls to the local socket then. But this is kind of the worst integration, IMO.
An option I want to avoid is forking the business logic an putting it into individual micro services. It would make it easier to get everything started, it would tie us to micro and it would allow reva and ocis to diverge ... which is the opposite of what we set out to do.
A possible solution would be to teach reva how to register handler with a micro.Server().
The core problem we see is that go-micro is trying to solve the same problems reva as a micro services framework tries to solve. In my opinion the business logic in the reva handlers is what matters. Interested parties should be able to integrate them in new ways. In the current state reva is hard to extend. Anyone wanting to integrate with reva has to submit a PR against reva.
If he does not want to do that for whatever reason the only option he has is to fall back to the CS3 api.

Furthermore, I was under the impression that we and every other participating party want to follow the same way of building a single binary. At the conference we started with gokit ... and moved to go-micro over the weeks. But go-micro vs gokit does not make a difference with regard how to reuse reva. While reva is trying to avoid vendor lock in ... it has become a lock in itself.
I think it can and should still be the reference implementation, but to be open and foster collaboration the handlers should be open and reusable as much as possible.

Current line of thinking:

run a reva instance per service
tie it to a unix socket (can be overriden via env var)
implement an adapter the translates the go-micro generated api to the vanilla  protobuf api

I think Reva should be considered as a dependency of OCIS like MySQL or Redis would. You just do not import the MySQL or Redis libraries and run MySQL or Redis server inside your software.
You do not adapt MySQL to meet ownCloud requirements because you use framework X, but adapt OCIS to work against MySQL by using a set of APIS. Idem for Reva with the CS3APIs.
As presented various times Reva can be easily extended by implementing the necessary interfaces as the other projects in the community do.
Anyway, this requires a more high level overview and direction of the OCIS product from the ownCloud company perspective.

Why are handlers moved to internal namespace?
This is limiting reusability a lot, while I see no advantages at all.
@labkode


test(score): augmented unit tests for computing score

Signed-off-by: Frederic BIDON<EMAIL_ADDRESS>

cc: @matthyx @dwertent
Augmented tests ("tok cycle") to prepare forthcoming refactoring actions ("tik cycle").

@dwertent rebased.


Question about behavior manager

I wanted to ask about how the behavior manager worked, is it in development or is their any way to use it(other than name and physical size)? It said in the description something about AI management but I don't know how to access/use it.

Ah, AI management is unimplemented currently; been juggling multiple projects at once and further development on that is on the backburner for the moment.


Editable Model does not re-validates on restore

Consider re-evaluating the validation and dirty states.
Check all possible scenarios

Moved to the logofx-client-mvvm-model repo.


[BUG] from Feature Request #647 - Set/New-PnPTeamsTeam AllowCreatePrivateChannel not working

Hi MS Dev Team,
We have successfully tested #647 (Set/New-PnPTeamsTeam AllowCreatePrivateChannel) on v1.5.53-nightly but now with version v1.6.0 we are getting this exception message:
Requested value 'NotSpecified' was not found.
Regards and Thanks.

Hi, i have the exact same error with a "Set-PnPTeamsTeam -Identity $groupId -Visibility "Private"" command.
It was working well with 1.5.0 version, but not anymore on 1.6.0.


[#31] nullptr guard on AVU fields

testing, will un-draft when we have green on all OS'es

excellent - we'll merge once undrafted

green dots on master on Ub18,16,C7


Duplicate content

The content of this page seems to overlap with the contents of https://docs.microsoft.com/en-us/dotnet/csharp/event-pattern. Seems like these should be unified into a single page.

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

ID: 38e8321f-5516-e451-5429-750d49f0ebed
Version Independent ID: 1fd093e9-f47c-ffdd-91e9-5039c3ccde61
Content: How to: Publish Events that Conform to .NET Framework Guidelines - C# Programming Guide
Content Source: docs/csharp/programming-guide/events/how-to-publish-events-that-conform-to-net-framework-guidelines.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Hi, i would like to work on this. Can you guide me with the task?

Thanks for writing this @Daniel15 Yes, there is too much duplication here.
Based on the potential organization for this, the correct target would be https://docs.microsoft.com/en-us/dotnet/csharp/event-pattern
@tulikavijay You can submit a PR for this, if you'd like.
There are two additional tasks to make sure we do while merging this.
The first is to redirect the article being removed. That's configured in our master redirection file The second is to update all internal links in this repository. It's larger than we'd normal recommend for a first-time contributor, but if you're interested, I'll watch for a PR.


Add load/serialize methods to Colleciton

Ideally the exposed Collection class should also have the ability to be serialized and loaded.

+1 for this. No, actually, +1000 ;-)
We don't need persistence, so my plan is to just use collections and sync them with a db on a server. So being able to load a collection from plain JS objects/JSON string and "inflate" them with a specific prototype would be very important for our usecase.

I postponed this issue to 1.3.0: at the moment Collection has no encapsulated attributes, so - effectively - you can simply do a JSON.stringify to serialize, then do a JSON.parse and iterate the properties of the  parsed object copying them into an instance of collection. An example of this pattern is available here: https://gist.github.com/techfort/519dc03e2bf10251e2df
By 1.3.0 we'll add a utility method to automate this process.

I am going to close this as it really needs its own persistence adapter to work properly, and this might be a separate component.


Deprecated field

Would it be possible to expose the deprecation message for a module as a property?
Ref https://github.com/gulpjs/plugins/issues/225

Hey guys, any thoughts on this? Would love to enhance the gulp plugins project to exclude deprecated modules. Please let me know when you can.


Photoshop Plugin on M1 Mac "is not compatible"

Cant install via creative cloud application manager or other manual ways such as ExtensionManager etc. Also cant remove FROM the creative cloud. I suppose since i cant remove the plugin from my creative cloud, I got the error during manual installation that an application with the same name is already installed. after several reboots and reinstalls of cc and ps, i now get error 641 via manual installation.... seems im stuck..... cant remove from cc and cant install manually..... my ps version is > 23.5

got it running by reinstalling and restarting various times...


Performance problem with zuul

We are using api gateway pattern and making request to our microservices. our microservices are in private subnet hence we are using another service called api-gateway which has zuul in it for routing.when we are load testing our microservices, the time taken for request to hit the microservice and time taken for response to return to api-gateway is more.

api-gateway service deployed in m4.10xlarge machine(40 core machine)
recived data volume is 2.4 MB
zipkin for distributed tracing
Jmeter for load testing with 3000 concurrent requests.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


LDAP Improve suggestions

[ ] Allowed use custom dn for user filter

[ ] Allowed use custom dn for admin filter

Current ldap use userDn as baseDn, it's impossible to filter by group

[ ] Fetch membership from ldap

Use group as user's org
Attribute used to match org name is configurable

[ ] Add connection check and auth check

Currently have to open another browser to verify settings

Releate issues

https://github.com/go-gitea/gitea/issues/2121

Reference

Jenkins LDAP Plugin

Actually userDn is baseDn and you can use custom filter for users and admins, including filter by group. For example:
userDn = OU=company,DC=company,DC=com
userFilter = (&(objectCategory=Person)(memberOf=CN=user-group,OU=company,DC=company,DC=com)(sAMAccountName=%s)(!(UserAccountControl:1.2.840.1135<IP_ADDRESS>3:=2)))
adminFilter = (memberOf=CN=admin-group,OU=company,DC=company,DC=com)

Fetching membership is already described in #2121 so that leaves only auth check that is missing

I use apacheds group instead of ad group, the ldap tree like this

userFilter use userDN as base dn (code here), so, this is impossible, to check is user in group need
base <cn=developer,ou=group,dc=example,dc=com>
filter: (member=%s)

same for admin.

Problem is that filter must return user record not group record as attributes (name, email etc) are fetched from returned query record. In your provided settings it will return group record that is not correct and can not possibly work. I have not used ApacheDS but what I looked up if there is no attribute under user record that links to groups than it is not possible to query users by group at least I don't know how to do it using single ldap query that means that this would require complete LDAP rewrite to support this. Can you give example what attributes are present under ou=Users -> uid=xxx?

Current user filter is another request, just change first argument to UserFilterDN, checkAdmin also send another request, just make first arguments configurable, result is not important for check, just check count is enough like how current ldap filter check do.

Yes, just as I thought there is no way to support that and for this to work most probably new authentication type must be added or need complete rewrite of current one...

+1 to testing the LDAP connection from the admin panel. Was about to create a new issue before I found this one.
Had the user filter wrong (didn't include the placeholder, just a filter that returns a list of users that should be allowed to log in) and had to log out - try the login - log in as local admin again - change settings - redo, which is tedious to do (especially when you don't have another browser available, or at least a private browsing mode that ignores the other active sessions).

I don't know how to do it using single ldap query that means that this would require complete LDAP rewrite to support this.
Gogs knows an separate Query with an separate "Group" Filter. Maybe its possible to copy the actual LDAP Support from Gogs

Is someone aroud who can drop an comment, if thats possible ?

There is plan to implement server global user groups (that could be than used for assigning to teams) and than we could do proper user ldap group implementation

@ptman yes if you enable that

@lafriks my point was that it should maybe be checked in the checkboxes, and maybe something else is already implemented as well

I don't know how to do it using single ldap query that means that this would require complete LDAP rewrite to support this.
Gogs knows an separate Query with an separate "Group" Filter. Maybe its possible to copy the actual LDAP Support from Gogs

@lafriks I found this open issue after searching for Verify group membership to match what is shown on the docs:
https://docs.gitea.io/en-us/authentication/

Verify group membership in LDAP uses the following fields:
Group Search Base (optional)
The LDAP DN used for groups.
Example: ou=group,dc=mydomain,dc=com
Group Name Filter (optional)
An LDAP filter declaring how to find valid groups in the above DN.
Example: (|(cn=gitea_users)(cn=admins))
User Attribute in Group (optional)
Which user LDAP attribute is listed in the group.
Example: uid
Group Attribute for User (optional)
Which group LDAP attribute contains an array above user attribute names.
Example: memberUid

Should an issue be opened against the docs to have that portion removed until support is added to match what Gogs has? Serious question (no snark, etc intended).

Yeah, it should not be there.. :)

Or you could preferably open a PR to implement this for us.

There is plan to implement server global user groups (that could be than used for assigning to teams) and than we could do proper user ldap group implementation

Are there any more specific ideas or plans on the design of these server global user groups? I will use this as my personal go starter project.
There may be multiple ways to add some sort of group support to Gitea, but not each might be suitable. (And once implemented, it's hard to change some decisions due to backwards compatibility)
A few questions:

Will these groups be public visible?
Will there be default groups?
How will the relations between groups and organizations/teams be? (group membership provides org/team membership vs. group membership is necessary for org/team membership)
Group providers? (LDAP: group search base, group filter, group membership filter? Mail: Regex?)
Special admin group vs. extension of existing LDAP admin filter
Option do disallow login if user is not in a group? (additional user filter on groups)

There may be ways to implement this without adding global user groups.

Simplest solution: Just add two additional filters. One for general gitea access, one for admin permissions.
Slightly more complex: Add single group filter for each org/team. Still not generic, but kind of usable in most cases.

Both options above require:

group search base (ou=groups,dc=...)
user attribute in group (cn like (cn=Hermes Conrad,ou=people,dc=planetexpress,dc=com))
Additional filter for general access, admin permissions, org/team membership (%s will be the value of above attribute for the queried user): (&(class=posixGroup)(member=%s)(cn=admingroup))

These kind of filters might be added for other authentication providers.


Add deploy github actions workflows

Signed-off-by: Tyler Auerbeck<EMAIL_ADDRESS>refs #9921

What this PR does / why we need it:
This PR adds the remaining required github actions workflows in order to complete the transition from CircleCI. Specifically the following will now happen:

build-pr will now run when a PR is merged to the main branch
deploy will now happen when:

a tag is pushed to the repository
the workflow is manually run from GitHub Actions with a provided version name (some questions here if this is desired)
on a push/merge to the main branch, a deploy happens with VERSION=canary

Special notes for your reviewer:
@bacongobbler This is about 99% complete. Just wanted to put this out as a draft for you to take a look at while I finish some final testing. One question I had is really around the release process. As I'm not familiar with the release process, I just copied what I saw in the deploy.sh script. So things like do something when a tag is specified, etc. However, after doing some work in other helm repositories (chart-release, chart-testing), I noticed those repo's also had a workflow-dispatch action that allow them to trigger the release manually via GHA. Is this something that would be desired here? I've stubbed out a majority of it in case you would be interested. But otherwise, let me know and I can pull it out.
This will require two secrets to be created inside of the repo in GitHub:

AZURE_STORAGE_CONNECTION_STRING
AZURE_STORAGE_CONTAINER_NAME

To take a look at the output of this, you can look at a successful run of the workflow here:
https://github.com/tylerauerbeck/helm/actions/runs/1099917788
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Should the CircleCI config be removed as part of this? Or should this be done in a follow up after we see these actions working ?

Good question. We probably want to disable CircleCI as soon as this is merged. Otherwise both Github Actions as well as CircleCI will be uploading competing uploads.
We can fix these up in follow-up PRs if need be.

@bacongobbler This should be in good shape. Let me know if there is anything else you would like to see.

I scheduled myself to merge this PR today. Now that I realize it's Black Friday... Probably best to avoid merging code that could affect the release assets... Especially on the busiest sales day of the year. ;)
I'll reschedule reviewing and merging this for Monday.
Curious why CircleCI is failing to report its status here too. No matter - it will soon be gone :)

This has already been completed in another PR so I'm closing this one. Thank you for the work and to try to help us along with this.


Better explain guidance acceleration

Please explain more in detail you guidance acceleration implementation.
Suggestions:

include in the documentation some actual internal generated prompts passed to the LLM engine, to better explain what's going on under the hood
what key/values are being cached?
is the speed up because the inference process is stopped or because KV cache, or both?
more insights about the documented benchmark

Thanks

Hopefully this explains it a little better :).


How to pull more than 2,000 records in a report?

First off, sorry if I'm asking this in the wrong place. I'm very new to programming and python, but this module is great!
One of the reports I'm trying to automate, though, is pulling a 30,000 record report from Salesforce with only one column. I've gotten that column into a list, but it only has 2,000 records which i assume has something to do with the "Report Limits" mentioned in this link - https://help.salesforce.com/articleView?id=rd_reports_limits.htm&type=0.
Is there a way for it to pull all of the records that would be in the report if i actually exported it to Excel?
Thanks!

Has this issue been address ?

Has this issue been address? Did you find any alternative solution?


router, hub, firehose deploys should be zero-downtime

Right now, they're up while the container builds, but I think down while it boots. Not ideal in general, but especially dangerous when the deploys are initiated on my laptop and not in CI, since they pause if I close the lid, lose network, etc. Ugh.
Hopefully this is an easy fix by just tweaking manual_scaling to allow more than one in flight instance, or something similar.
(I don't care about this quite as much for the firehose consumer, since it's pull-only and resumes gracefully, but the router serves push task requests, and hub serves external websocket requests.)

Added router to continuous deploy in f09d8eb0bcddeecd4684bae18ef1e7ab72017064, that's a useful step here.

One hack here might be just to speed up deploy. Maybe I can do that with liveness_check.initial_delay_sec or readiness_check.app_start_timeout_sec, and/or success_threshold on either?

liveness_check.initial_delay_sec seems promising here. From its docs:

The delay, in seconds, after the instance starts during which health check responses are ignored. This setting delays...recreating the instance if the instance is in the process of starting up.

I tried dropping it in atproto_hub.yaml to 30 from its default of 300. I don't have data on deploy times from before the change, but afterward, the deploy (ie post-build) part was ~5:30. Not sure if that's an improvement.
For comparison, router should have basically the same startup, and without this change its post-build deploy is currently ~7m. So maybe it shaved off 90s?

I've added separate deploy steps to our CircleCI config so I can deploy router and atproto-hub there on demand, from the UI.


Fix day label on timetable

All day labels are displayed as Day 1. So, let's fix the others to proper labels.

@Nagumo-7960
Thanks! Assigned 👍


Add support for support arbitrary key:value pairs in ListGenerator

Fix https://github.com/argoproj-labs/applicationset/issues/108

/assign @jgwest

/assign @jgwest

LGTM. Thanks @maruina

LGTM. Thanks @maruina

I don't know if this PR covers everything in https://github.com/argoproj-labs/applicationset/issues/98 but I added a couple of tests. It might be a good start.

I don't know if this PR covers everything in https://github.com/argoproj-labs/applicationset/issues/98 but I added a couple of tests. It might be a good start.

Hello @maruina @jgwest
Nice to see this new feature.
I would like to clarify 2 points please :

the documentation does not seem to reflect this new possibility in the List generator... do you confirm ?
is that a particular reason to keep cluster and url as mandatory fields in the list generator ?

Hi @yogeek,

the documentation does not seem to reflect this new possibility in the List generator... do you confirm ?

Confirmed, I have opened https://github.com/argoproj-labs/applicationset/issues/273 for this.

is that a particular reason to keep cluster and url as mandatory fields in the list generator ?

Nope, see the discussion on https://github.com/argoproj-labs/applicationset/issues/162 re: getting rid of these as mandatory fields.

Hi @jgwest :-)
Great, thank you !


Add translations for SMTX OS driver

Proposed changes

Add translations for a new machine driver - SMTX OS driver.
Docker machine driver repo: https://github.com/smartxworks/docker-machine-driver-smtxos
Rancher UI driver repo: https://github.com/smartxworks/rancher-ui-driver-smtxos
Rancher machine driver data PR: https://github.com/rancher/rancher/pull/20816
Types of changes

New feature
Linked Issues

None
Further comments

None

Bump. Any feedback please?

@fengye87 Sorry for the delayed response. When the mainline PR to add the Driver is merged into master I will get this merged, just ping me directly in this PR when it is. Thanks!

The driver component should include and register its own translations, with something like get(this, 'intl').addTranslations('en-us', { map: { of: { keys: 'Translation for Key' } } } );.  Bundling them in the UI creates a dependency on two things that are independent.


"goto" outside block corrupts execution stack

Add tests in tst/conformance that fail due to this bug
and then fix it.

--- FAIL: Test5Goto (0.02s)
	conformance_test.go:68: (testGotoScopes2 10)=>20 (want 10)
	conformance_test.go:68: (testGotoScopes3 10)=>-1 (want 10)

--- FAIL: Test9For (0.04s)
	conformance_test.go:68: (testForScopes1 10)=>1 (want 10)
	conformance_test.go:68: (testForScopes2 10)=>1 (want 10)

Test (testGotoBackAndScopes 10) kills Emacs.

Fixed as a part of 1aa43965c71abce28cec467c9b3cab0838d38332.


Hard drive errors

This API is unable to get errors/events related to HARD DRIVE/DISK
DVR is sending that event on iVMS but that error/event is not obtained by this API.

'diskfull': 'Disk Full',
'diskerror': 'Disk Error',

These are both supported events.  If you are reporting an event that shows up in the event stream, but is not currently supported, please post a snippet of the XML from the event stream and the type can be added.  If the event does not show up the event stream it's not possible to add support.

I am testing the event stream extensively.
I have added some more ISAPI specifications ( according to my usage )
I propose we can work together and implement more ISAPI specifications to this API, it will be very useful for others.
What do you  propose ?

I gladly accept PR's.  Please feel free to submit any new features and we can work together to get them merged.


Use safe_wraps for wrapping isin()

Fixes #3388

[x] Tests added / passed
[x] Passes flake8 dask

Oops, I accidentally pushed this as a new branch to dask rather than my fork. Interestingly CI passed on my branch but not the pull-request...

Sorry for the delay (was at a conference last week). Looks fine to me. Asked one question above.

Thanks @shoyer. :)


V1.1 user guide

Update finish all commands for v1.1 UserGuide

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

Coverage data is based on head (ddb53c9) compared to base (5f5bcb8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #38   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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.


Feature request: Docker image build (parameterized_docker_build.sh) lacks option for AVX2 optimization

As far as I can understand there is no way to tell the script parameterized_docker_build.sh  to build an AVX2 optimized docker image.
I see a note about Python 2 and 3 but nothing about AVX2:
# TODO(cais): Add support for TF_DOCKER_BUILD_PYTHON_VERSION (PYTHON2/PYTHON3)

@caisq can you comment on this?  Thanks!

BTW in the meantime, I wanted to mention that we're happy to accept pull requests.  :)


chore: Update registry.access.redhat.com/ubi9/s2i-base Docker digest to 2d6f665

This PR contains the following updates:

Package
Type
Update
Change

registry.access.redhat.com/ubi9/s2i-base
final
digest
3217745 -> 2d6f665

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


fix(cloudrun): delete old region tag run_system_package_ubuntu

Description
According to the "deleting sample/region decision tree" decided to delete the region "run_system_package_ubuntu" because it was inner nested to the valid region "cloudrun_system_package_ubuntu"
Fixes b/347348198
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist

[x] I have followed Contributing Guidelines from CONTRIBUTING.MD
[ ] Tests pass:   go test -v ./.. (see Testing)
[x] Code formatted:   gofmt (see Formatting)
[x] Vetting pass:   go vet (see Formatting)
[ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
[ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
[ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
[ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
[x] Please merge this PR for me once it is approved

@code-review-assist review


Workload - Create using Wizard

Updated Mock: https://xd.adobe.com/view/fa7ae40e-66f6-4d03-67d6-2b2f422beae5-8c4b/
Workload Types
Within each type, you have the ability to specify a pod template, which has the same components for all workload types. Either metadata or the pod spec itself. The pod template is located in different locations based on the workload type. In the tabs description, it will refer to a field in the pod template spec as <podTemplate>, which can be nested differently between each workload type. Refer to this table to find out where the pod template spec is nested.
Any other fields will be called out specifically between the types.

Name
Pod Template Spec K8s yaml location

Deployment (Default)
spec.template

Daemon Set
spec.template

Stateful Set
spec.template

Job
spec.template

Cron Job
spec.jobTemplate.spec.template

[ ] Remove Replication Controller, ReplicaSet from the list of types

We will be creating a wizard style process to step through the options.
Wizard Steps

Steps

Select Workload Type

Define Container

Storage

Advanced

In Step 2, we need to add these additional fields:

Workload Type
Field
YAML
Required
Default

Deployment, StatefulSet
Replicas
spec.replicas
X
1

StatefulSet
Service Name
Dropdown
Dropdown of available headless services in the selected namespace. See below for how to find headless services
X

CronJob
Schedule
spec.schedule
X
Example Text: 0 * * * *

Note: all other fields from the different types of workloads should be in Advanced Section

How to find headless services in k8s:
kind: Service
spec:
  type: ClusterIP
  clusterIP: "None"

Required Fields
Step 1:
Workload Type
Step 2:
Name
Image
Namespace

Closing in favor of: https://github.com/rancher/dashboard/issues/964


Mixin Proposal: IndustrialCraft 2 Miner

Target Mod
IndustrialCraft 2
Mixin Functionality
Increases diamond drill speed in the IC2 miner, by changing duration from 50 to 20.
Change Rationale
Gameplay balance tweaks.

Reimplemented


test: enable response checks as required by e2e tests

I really don't know how I made this to work at any point. Anyways, this is fixed now.

You probably need to merge #140 here if you want this to pass, or merge the other PR first...


Confusion while creating static shortcuts in my Godot game

To create a static shortcuts I have added shortcuts.xml in my xml folder and the xml folder is in res folder
Below is the code for shortcuts.xml
<shortcuts xmlns:android="http://schemas.android.com/apk/res/android">
    <shortcut
        android:shortcutId="static"
        android:enabled="true"
        android:icon="@drawable/splash"
        android:shortcutShortLabel="@string/short_label"
        android:shortcutLongLabel="@string/long_label"
        android:shortcutDisabledMessage="@string/disabled_message">
        <intent
            android:action="android.intent.action.VIEW"
            android:targetPackage="com.codingwithnobody.myandroidprojct"
            android:targetClass="com.codingwithnobody.myandroidprojct.GodotApp">
            <extra android:name="shortcut_id" android:value="static" />
        </intent>

    </shortcut>
</shortcuts>

I have few questions

Should I create a separate plugin for static shortcuts and just write my code in the android folder of my Godot game which is generated by installing the android build template from Godot?
For android:targetClass what should be the activity name, currently have written GodotApp but GodotApp is present in com.godot.game so not sure what is the entry point activity?
Is it even possible to create static shortcut for godot at this moment?
If I write my code in the android folder of my Godot game instead of a separate plugin, the moment I update Godot I need to update the android build template as well which in turn override my static shortcut code?

@BraveEvidence Closing this issue as this is more of an Android question rather than a Godot question. Also I don't see how this issue relates to this repo.


UDRs in Figure 3 not included

Figure 3 is not containing the required UDRs on the NVA Spokes to make this work. I know, this is just the vWAN view but maybe confuses readers so we should either add a disclaimer that the UDRs are not shown in the figure or we should add them.

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

ID: 928d7a76-5472-0f2d-c6d9-100099ee5dff
Version Independent ID: 78d77834-58eb-c0b6-197d-73e3af2dc469
Content: Scenario: Route traffic through a Network Virtual Appliance (NVA) - Azure Virtual WAN
Content Source: articles/virtual-wan/scenario-route-through-nva.md
Service: virtual-wan
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@marcelkmfst , Thank you for your feedback. We'll review this and get back to you shortly.

@marcelkmfst, Appreciate your patience. We are working on adding the note to the doc and it would take some time for the edits to get reflect over the document.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.


Breaking Change related to ASP.NET Core on .NET 2.1.5?

I've seen something really strange now that .NET Core 2.1.5 is starting to roll out across Azure.  It looks like a breaking change, but maybe I've done something wrong.
The exact same code base (built and deployed at the same time) is behaving differently in two different Azure Web apps.  Prior to today, it was behaving the same.  One app was restarted, but we haven't actually deployed anything to either of the.
We have an ASP.NET action method that takes in an array of objects, like so:
    [ApiController,
     Authorize,
     Route("widgets")]
    public class WidgetsController : Controller
        [HttpPost("summaries")]
        public async Task<ActionResult<ResultModel[]>> SearchAll(Criteria[] criteria, bool countOnly = false)
        {
              // snip
        }

In one of the deployed instances, posting a form like this works as expected:
[
	{"parameter":"User","comparisonType":"=","value":"*"},
	{"parameter":"Status","comparisonType":"=","value":"New,AttemptedContact"}
]

In the other, posting the exact same payload, the payload is ignored.  The criteria array is empty.
I ran some tests.  If I add [FromBody] to the parameter, then it works as expected again.
I'm having a whale of a time reproducing the issue locally.  Everything worked fine locally until I installed .NET 2.1.5.  Once I did that, I started seeing the same, incorrect behavior that I'm seeing with one of our instances in Azure.  I tried to run my app locally on a previous version of .net by using global.json.  The CLI reported it was running 2.1.300, but the same issue still persisted.  Again, everything worked fine until I installed 2.1.5.
POSTing a single object still works fine, no need for the [FromBody] attribute.  It seems to just be related to posting arrays.
Any suggestions on how I can troubleshoot this further?

I was able to isolate things even further.
This does not bind correctly on ASP.NET Core 2.1.5 (note that the param is an array):
        [HttpPost("test1")]
        public ActionResult Test1(TestModel[] model)
        {
            return Content($"Got: {model.ToJson()}");
        }

This binds fine (note the param is not an array):
        [HttpPost("test1")]
        public ActionResult Test1(TestModel model)
        {
            return Content($"Got: {model.ToJson()}");
        }

Things bind fine in both scenarios running on ASP.NET Core 2.1.2 on .NET Core 2.1.2.

Not sure if it helps, but IEnumerable works fine:
        [HttpPost("test1")]
        public ActionResult Test1(IEnumerable<TestModel> model)
        {
            return Content($"Got: {model.ToJson()}");
        }

However, List and IList do not:
        [HttpPost("test1")]
        public ActionResult Test1(List<TestModel> model)
        {
            return Content($"Got: {model.ToJson()}");
        }

There were changes in this area of the product in 2.1.5. https://github.com/aspnet/Mvc/pull/8272
@dougbu @pranavkm @mkArtakMSFT can you comment?


Refer back to the Elyra Notebook Image

Once the Elyra image is re-enabled, we will need to bring back our text removal.
PR for the removal: https://github.com/opendatahub-io/odh-dashboard/pull/1228
See: https://github.com/opendatahub-io/notebooks/issues/90

We'll also want to add:
"public_api_endpoint":<host>/pipelineRuns/<namespace>

To the elyra config (code)


Inquiry

Sorry, couldn't find your email address, so posting here. @diegoveloper are you and/or your team interested in freelance work? I may have a project you can assist with. If interested, let me know the best place to discuss.

Hello, my mail is the same as my username(diegoveloper@gmail.com). For now we are not doing freelance jobs, but you can find flutter developers on this group: https://www.facebook.com/groups/425920117856409/
Thanks


fix(versions): update Activiti/activiti-cloud-notifications-graphql versions into master

UpdateBot pushed maven dependency: org.activiti.cloud.notifications.graphql:activiti-cloud-notifications-graphql-dependencies to: 7.1.226

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.notifications.graphql:activiti-cloud-notifications-graphql-dependencies 7.1.226


The official release for Windows is configured to work with msvc only

Discussed in https://github.com/clangd/clangd/discussions/2118
Please have a look at https://github.com/mstorsjo/llvm-mingw/issues/454.

Clangd looks for MSVC system headers by default, but you can use --query-driver (discussed at https://clangd.llvm.org/troubleshooting#cant-find-standard-library-headers-map-stdioh-etc and in more detail at https://clangd.llvm.org/guides/system-headers#query-driver) to point it to the system headers of a gcc installation, including mingw.


Lnl create new platform

Add board definition for Lunarlake.
This is first PR from the series to enable new ACE 2.0 platform.
Signed-off-by: Jaroslaw Stelter<EMAIL_ADDRESS>

Nothing wrong popped to me. Looks nice !


Copyright violations: Several pieces of source code were just lifted

Hi there,
From what I can see, you all have just lifted code from multiple crates without any acknowledgement at all. This is legally infringing even if you're copying from permissively licensed crates. Please create a file stating what all third party code has been used.

Hello, yes we are aware of this and we are yet to credit these crates on the readme. We will add it now.

Just for an example:
https://github.com/voltpkg/volt/tree/master/junction is directly lifted from junction
And there are several files scattered across. You should maintain a list of what all third party code is used and then point to their locations and clearly mention that in the readme stating that "This repository contains third party code from ..."
Hopefully you get this set before the authors create reports

The readme has been
updated

@firedupmike Would you like us to create a CREDITS.md?

If you have a small number of things to list, then the README seems fine. If you have too many, maybe state "This project uses third party code from different crates and a list of them can be found here," linking to the credits file. That's how it is doneBest,Mike (aka Will)Sent from my iPhoneOn 6/11/21, 6:21 PM Tejas Ravishankar @.***> wrote:
@firedupmike Would you like us to create a CREDITS.md?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

Cool thanks for bringing this up will do in an hour when I'm free

Looks like GitHub never learned to format email contents :/

@firedupmike we are now maintaining an active list of packages in our CREDITS.md and we're linked that to the README.md. Thanks for raising this issue and I will be closing it now :)

@firedupmike we are now maintaining an active list of packages in our CREDITS.md and we've linked that to the README.md. Thanks for raising this issue and I will be closing it now :)

Your way of maintaining the credits is incorrect. For the sources that have been copied, state which directory from the source tree contains it and its corresponding license.
For example, for the junction folder, add a README.md and link to the original source and license. Your README should state something like:
Volt is licensed under Apache-2 and uses source code from junction in (/path/to/junction/in/your/tree) that is released under the MIT License(link to repo), rustc_lint in (/path) that is released under the Apache license(link to repo), ....
Nowhere are you stating that your codebase itself contains third party code and not via dependencies.

I mean you're literally copying all the repos of an entire project in your source code. Why can't you dynamically link it i.e via a dependency? Also, if you really need some changes, why not PR upstream? This is disgustingly bad practice in open source

We're using junction and the rslint set of libraries because:

Current junction crate is completely broken (we're also removing it as a dependency because we no-longer need junctions)
RSLint is yet to release a new version to crates.io so we're waiting for that, I'm in touch with rslint developer

@XtremeDevX why can't you just use submodules to the repository upstream?

@firedupmike yep didn't think of that - will do soon

Also, we do need to have those rslint libraries physically there - we're customizing error messages etc.

@XtremeDevX And if you really need the changes, why can't you just use git linking in your dependencies? For example:
package = { git="linktorepo" }

Cargo also you to use git dependencies without having to use submodules at all? Read this:  https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-git-repositories

Also, we do need to have those rslint libraries physically there - we're customizing error messages etc.

Huh? Isn't that what submodules are for?

Also, we do need to have those rslint libraries physically there - we're customizing error messages etc.

because this - we're customizing errors on rslint

Also, we do need to have those rslint libraries physically there - we're customizing error messages etc.

because this - we're customizing errors on rslint

And I'm sure you can map_err in many cases since most of the errors use std's result enumeration. And these will anyways be optimized by the compiler when you build your binary

No - rslint is a linter, we're customizing its output messages

@firedupmike we are now maintaining an active list of packages in our CREDITS.md and we've linked that to the README.md. Thanks for raising this issue and I will be closing it now :)

Your way of maintaining the credits is incorrect. For the sources that have been copied, state which directory from the source tree contains it and its corresponding license.
For example, for the junction folder, add a README.md and link to the original source and license. Your README should state something like:
Volt is licensed under Apache-2 and uses source code from junction in (/path/to/junction/in/your/tree) that is released under the MIT License(link to repo), rustc_lint in (/path) that is released under the Apache license(link to repo), ....
Nowhere are you stating that your codebase itself contains third party code and not via dependencies.

I will do this.

Are you using this: https://docs.rs/rslint_core/0.2.2/rslint_core/?

No, that's an outdated version which does not have typescript support

Use upstream?

Via the git linking?

We need to customize the errors which have been output from rslint

that's why we're maintaining physical crates

@firedupmike we are now maintaining an active list of packages in our CREDITS.md and we've linked that to the README.md. Thanks for raising this issue and I will be closing it now :)

Your way of maintaining the credits is incorrect. For the sources that have been copied, state which directory from the source tree contains it and its corresponding license.
For example, for the junction folder, add a README.md and link to the original source and license. Your README should state something like:
Volt is licensed under Apache-2 and uses source code from junction in (/path/to/junction/in/your/tree) that is released under the MIT License(link to repo), rustc_lint in (/path) that is released under the Apache license(link to repo), ....
Nowhere are you stating that your codebase itself contains third party code and not via dependencies.

I will do this.

This is important. Also, try to keep the LICENSE and README for the crates intact. There have been so many open source licensing legalities.

Sure, will do

I literally cloned the crates over here https://github.com/rslint/rslint. So it shouldn't be any issue. The crates don't have a README or a LICENSE attached.

@firedupmike we are now maintaining an active list of packages in our CREDITS.md and we've linked that to the README.md. Thanks for raising this issue and I will be closing it now :)

Your way of maintaining the credits is incorrect. For the sources that have been copied, state which directory from the source tree contains it and its corresponding license.
For example, for the junction folder, add a README.md and link to the original source and license. Your README should state something like:
Volt is licensed under Apache-2 and uses source code from junction in (/path/to/junction/in/your/tree) that is released under the MIT License(link to repo), rustc_lint in (/path) that is released under the Apache license(link to repo), ....
Nowhere are you stating that your codebase itself contains third party code and not via dependencies.

I will do this.

Will still do this tho

That means all of them are licensed under the same license?

Correct - MIT License

I literally cloned the crates over here https://github.com/rslint/rslint. So it shouldn't be any issue. The crates don't have a README or a LICENSE attached.

So you should have cloned the licenses in each file/dir. Why don't you create one single large directory with all the other crates and name it rustlint and add the readme and License there? That simplifies things greatly. Cargo allows you to specify paths at will: "../../wherever"

I'm currently working on a whole project restructure, will do it then

Also, if you don't mind me asking would love a ⭐ (only if you want to of course)

Also it'd be nice to state "We use rslint for linting" directly on your README and link back to rslint and their Open collective page. If you know, Riccardo has spent a huge amount of time and effort on this.

@firedupmike that's what the CREDITS.md is for though. We do not want a cluttered README.


Simplify HyperParameter implementation

The use of HyperParameter as a class and metaclass has led to a fair amount of confusion. I suggest splitting out metaclass functionality into HyperParameterMeta (i.e. __new__).
One consequence is that one can create some concrete hyperparameters directly, like IntHyperParameter([0, 10]). This would also simplify the hyperparameter repr (#121).

Closing this issue as we have redesigned HyperParameters in #129