Trailing / missing in connection API "href" entry in IS-04 controls

Some controller at the Houston interop wasn't adding a slash at the end of the URL we expose. That's the URL's fault, as I read IS-05 sections 2.0 and 3.1.
Easy fix:
commit 747fc8cb86a63641afd9f9be49590e5e7497c322
Author: Bill Trost<EMAIL_ADDRESS>Date:   Thu Feb 21 10:38:32 2019 -0800

    nmos::make_device was omitting the trailing slash required by the IS-05 control entry

diff --git Development/nmos/node_resources.cpp Development/nmos/node_resources.cpp
index aeb1813..9327edf 100644
--- Development/nmos/node_resources.cpp
+++ Development/nmos/node_resources.cpp
@@ -44,7 +44,7 @@ namespace nmos
             auto connection_uri = web::uri_builder()
                 .set_scheme(U("http"))
                 .set_port(nmos::fields::connection_port(settings))
-                .set_path(U("/x-nmos/connection/") + make_api_version(version));
+                .set_path(U("/x-nmos/connection/") + make_api_version(version) + U("/"));
             auto type = U("urn:x-nmos:control:sr-ctrl/") + make_api_version(version);

             for (const auto& host_address : host_addresses)

Hmmm.
https://github.com/AMWA-TV/nmos-device-connection-management/blob/v1. 0/docs/2.0.%20APIs.md#urls-approach-to-trailing-slashes says "the 'primary' path for every resource has the trailing slash omitted."
I think this is a client bug.

Now clarified in the spec (IS-05 v1.0.2, v1.1) that "In order to avoid cases of double slashes or missing slashes, clients are required to exercise caution when appending paths onto this 'href'."


Update README.md

The step to rename sample-env.json to env.json before running dev.sh script is missing
Update the routes

[ ] /health-check to /healthcheck
[ ] /getData to /profile

Please assign it to me


Allow org-admins to create/edit rda_questions in templates

Update the add/edit question pages and admin.js to include the RDA question and to display an optional comment for this question type

closing this out since the RDA work has finished. Please reopen if its still applicable @xsrust


[BUG] Cannot build: Could not resolve all files for :classPath

The bug
Trying to build the current git repo results in the given issue:
https://pastebin.com/srH90ekF (Error-Log)
To Reproduce
Steps to reproduce the behavior:

Clone this project
Open it in IntelliJ
Execute the build-task / ./gradlew.bat

Expected behavior
It successfully builds
Screenshots
Version information:

OS: Windows
Java version: coretto-11 and coretto-17 (both tested, same error)
Minecraft version: 1.19.2
EnderCore version ?
Version ?

Additional context
It seems like a mismatch beween the ModGradle:3.0.2 dependency and the project itself?
I cannot decipher the error message and google didn't really help me. I'd like to build the project and the Conduit-Module, but it seems like there are issues?

configure gradle to use the correct java version


Run as package the default version is empty

I guess the default version should not be empty when run it as package.
https://github.com/cpp-linter/cpp-linter-action/blob/d24e6225d827632732599cb812817ac086781cfd/cpp_linter/run.py#L87
I also would like to change the default version to the latest clang version, at least not v10, maybe v12 or higher.

This default is different from the action.yml because under Windows the version value is incorrect if the value is a digit. In Windows, clang tools' installations do not provide a version alias like clang-tidy-12. It can only be called by

clang-tidy when added to the OS's PATH env var
specifying the path to the executable like path\to\v12\bin\clang-tidy.exe

The default here (when used as a package) ensures that whatever version is installed is the one that is used despite the OS, as long as it was added to the PATH.

I have no objections to changing the default in action.yml to v10.

I see ... OK, I'll change default version to v12.

regarding clang-tools-pip
It might be a good idea to create the binaries using .exe suffix (see https://github.com/cpp-linter/clang-tools-pip/issues/11) and a link to the binary that doesn't use the version in the title.
(venv) PS path > clang-tools -i 12
(venv) PS path > ls venv/Scripts
clang-format-12.exe
clang-format.lnk
clang-tidy-12.exe
clang-tidy.lnk

Though, this might cause conflicts if the unspecified directory (-d) is always sys.executable and multiple versions are installed.

that doesn't use the version in the title.

Do you mean to type clang-format or clang-tidy can execute the clang-format-12.exe or clang-tidy-12.exe?
If multiple versions are installed? which binary should be linked?
If unspecified deriectory(-d) it should be fine because the binary name is different.
PS C:\Users\xshen> clang-tools --install 13
Don't have permission to install clang-format-13 to c:\program files\python38.             Try to run with the appropriate permissions.
PS C:\Users\xshen> clang-tools --install 12
Don't have permission to install clang-format-12 to c:\program files\python38.             Try to run with the appropriate permissions.

I have moved comments to https://github.com/cpp-linter/clang-tools-pip/issues/11, close this one.


Add SuperannuationFund Endpoint

Adding superannuation endpoint model.
More information about this endpoint can be found here:
https://developer.myob.com/api/accountright/v2/payroll/superannuationfund/

@ghiculescu you able to have look over this one for me, please?


Support Profile 111

Newer Xamarin Project Templates are now defaulting to Profile 111, and it would be extremely helpful to have support for both Profile 78 and 111.

I have the same problem!

Pretty please.

Same problem here. Any plans to add support for Profile 111?

Same problem here...

Profile 111 is actually already supported, since Profile 111 targets a subset of Profile 259, which is the selected profile in the PCL ZXing.Net.Mobile.Portable project.
However, since wpa81 is not included in the directory name for the portable DLL:s in the .nuspec file, the attempt to reference the ZXing.Net.Mobile NuGet package from a PCL Profile 111 project will fail. See e.g. this Stackoverflow question and associated answer.
I strongly recommend that the library specification in the .nuspec file for the PCL installation is changed to the standard Profile 259 name, i.e.lib/portable-net45+netcore45+wpa81+wp8/.
This minimal change would automatically enable referencing ZXing.Net.Mobile from a PCL Profile 111 project as well, and it will still be valid for the different Xamarin, UWP and WP8 Silverlight targets.

This is good information.
@anders9ustafsson my only concern is that implying wpa81 support is going to cause some complications / headaches for me.  Do you have any suggestions for making it clear that wpa81 is not actually supported?  Right now I'm thinking of just including the assembly that throws NotImplementedException in all the contract stubs....

@Redth Do you have contract subs in the PCL? Then you won't need an additional stub implementation for wpa81, NuGet will automatically assign the PCL assemblies to a Windows Phone 8.1 non-Silverlight application.
I guess the situation is already the same for Windows 8/8.1 projects since you are including netcore45 in the portable libraries path but there is no designated Windows 8/8.1 class library?

@Redth the easiest way would be a bait and switch. The PCL may be wpa81 friendly, however since you don't want to actually support wpa81, just include a framework reference lib\wpa81 with no dll's. This will allow someone to install the package to a solution containing an 8.1 project without providing references to ZXing.Net to the wpa81 project. I might also suggest updating the ReadMe to note that, so people don't report it as a bug.

@Redth  Could you please update the nuget package with @anders9ustafsson suggestion?

until the bug get fixed, here is a release build for profile 111, android 6.0 (had problems building other version #lazy) windows phone 8.1 (use the stuff in wp8.0 folder) https://wirelessarmy.theproject1.com/images/templates/zxing_Release.zip

@anders9ustafsson could you please review 93404e1 and verify this is what you were suggesting?  I guess I don't need to include UAP10+Xamarin.iOS10+MonoAndroid403 in the PCL target since they already exist in other <file .. /> elements?

@Redth Sorry for the late response, but Yes, your change is exactly like I believe it should be. For maximum compatibility I think it is wise to leave out the UAP/Xamarin part; NuGet handles these associations automatically. Thanks for fixing this!

Thanks for following up :)

@Redth thank you for getting this support. It's great that this is an open source library so I can always clone it and make the tweaks needed, but it's a lot nicer to have my team pulling the official package!

:D


Remove unnecessary imports

whoops forgot to remove these

Hi - any updates on the PR?

Sorry. Been busy this week. Looks good and merged. We should work on getting the coverage and change log to be ready for an npm release


dashboard: Reduce github/provision button padding

When the app-list is long enough to cause a scrollbar on the middle
panel, there's not quite enough width for the github and provision
database buttons and this causes them to appear one above the other
instead of side by side.
This commit has the side-effect of reducing the padding by 0.5rem in
right-hand panel too.
Signed-off-by: Simon Amor<EMAIL_ADDRESS>

Thanks!


Running todo new raises exception

For eg.: if I run todo new "Go to Zeba's Birthday Party, it throws up an error saying:
Usage: todo new [OPTIONS] [SUMMARY]...

Error: Invalid value for "--list" / "-l": Personal. Available lists are:

There are no lists available, do I need to create it manually?

I am also facing same error.

Hello all,
that is indeed something that should be clarified: You need to create task lists yourself. The reason for this is that usually vdirsyncer is supposed to manage task lists at this level.
If you have a configuration like this:
path = ~/.todo/*

this means that you need to create a folder in ~/.todo/, and that will be your new tasklist.
Hope that clears the confusion!

This solves the problem. Thanks :+1:

I'm unsure what to do about this, feel free to open PRs that clarify the docs or improve todoman's UX on this matter. #118 seems like a first step.

I'm closing this in favour of #118. We'll be improving the error message we display if no lists are available (since this is quite counter intuitive/unfriendly).


returnFormat on ButtonGroup not working

Expected Behavior
the returnFormat should be a field on ButtonGroup.
Actual Behavior
Causes an error:
Call to undefined method WordPlate\Acf\Fields\ButtonGroup::returnFormat()
Steps to Reproduce the Problem

Create a ButtonGroup
add ->returnFormat('value')

EG:
use WordPlate\Acf\Fields\ButtonGroup;

ButtonGroup::make('My Button', 'my_button')
    ->instructions("I am a button group")
    ->choices([
        0 => 'Show',
        1 => 'Hide'
    ])
    ->wrapper([
       'width' => '33%',
       'class' => '',
       'id' => '',
    ])
    ->returnFormat('value')

Versions

WordPlate: 8.1.1
PHP: 7.2

Thanks for reporting Ash! I've released an update in version 8.2.0.


NaN can be used as a key in Map

Pull Request Description
Number.nan can be used as a key in Map.
Important Notes
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
All code has been tested:

[ ] Unit tests have been written where possible.
[ ] If GUI codebase was changed, the GUI was tested when built using ./run ide build.

@Akirathan

Number.nan can be used as a key in Map.

Why do we want this? For me NaN is the value that is not equal to itself. That by definition kind of prevents it from being useful as a key in the map.
Even if we wanted this, I'd rather expect some kind of byte-representation-comparison rather than just NaN==NaN assumption.  There are different values (byte-wise) that are considered to be a 'NaN'. Do we want to disregard the differences?


Graph status facet

Create a facet that scans the graph and create statistics about the population of different types of vertices in the graph. Based on the gremlin query
graph.traversal().V().label().groupCount().toList

Because the query is pretty intensive, cache the results.

Done


Add Profile image in docs/images and TEAM.md

Description
Attach Screenshot

Note 2 code reviewer approval needed. Approach in twitter group & discord channel.

@arthurbuhl made the changes.


[UI Core Library] Create a shared tailwindcss config

If it's OK I'll take this and discuss with @pawelklasa since we already need this for pubic pages on .cloud.

One way of doing this could be to define everything in tailwind.config.js and just @apply the rules in our _typography.scss, _colors.scss, etc.
That way, the source of truth will be in a tailwind.config.js, one will be able to use it from class helpers and also in our scss shared files:
Pros:
✅  one source of truth easily shared between .cloud, MC, frontend-shared & possibly other projects in Hazelcast
✅  we could even re-use this for other Hazelcast projects - even those that don't have React dependency, e.g. frontpage etc, which could be awesome for design consistency
✅  possibility to make tree-shake unused classes in all of the projects
Cons:
⚠ We will be dependent on tailwindcss, also on @apply syntax, but I don't think it's a bad thing
Can you think of any? 🤔
WDYT @jozefhruska  @aigoncharov @danad02 @jankoritak @utkukaratas

@jvorcak I have had a chance to play with tailwind recently. I found that "@apply" helper is not flexible enough. For example, you cannot define animation transition time separately, only animation as whole. We would also be stripped of scss math. I mean we would not be able to define some base grid size in tailwind, and multiply it in our scss easily with @apply.
I think we should not import from tailwind, but we should import into tailwind.

That seems like a fair point, we would still get a generated tailwind.config.js that we could still use on other places, right?

Yep. We just need to figure out the best way to import stuff from SCSS into tailwind. Or from JS into both tailwind and SCSS.

@jvorcak @jozefhruska so what are you currently doing in Cloud?
One of the options could be using https://github.com/pmowrer/node-sass-json-importer


Fix/require modules on restart

Note to self, git history must be messed up because their should be only one commit on this branch.

"Merge Hell" I see


How to convert Coins Class to decimal? Only convert to string and then to decimal?

The amount is usually stored in decimal, but with this sdk I can only get the Coins object, the way I'm doing it now is to convert the Coins to string and then convert the string to decimal.

This involves multiple type conversions.
Is there a better way to do it?

Hello, we will upgrade our SDK that users can get coins in numeric types, thank you for detecting this issue!

Hello, we will upgrade our SDK that users can get coins in numeric types, thank you for detecting this issue!

thanks

I do this now:
(decimal)coins.ToBigInt()

I do this now:
(decimal)coins.ToBigInt()

You're ignoring precision.
For example, 1.23, you'll only get 1

We are added support to get decimal value. Check it #21.


copy editing Install and Setup- Advanced configs

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


Deleting Kubernetes Resource And Atlas Cluster

how are you guys?
Context
In my company we are building a platform to give developers autonomy.
One of the principles is that developers don't need to know how to create their service and service dependencies following all best practices enforced by us (by us I mean, my team, SRE)
Dependencies are everything a service needs, for instance: a mongodb, redis, storage (bucket) and so on.
Of course, we are using atlas mongodb solutions for provisioning our mongodb cluster and currently, we are using this project to provide an easy way to create developers mongodb clusters.
Use case
So, one of the features that we (SRE Team) would like to provide, it is a branch context deployment. In other words, a platform feature that could be possible developers deployed their branches in a kubernetes cluster, a code isolation from their principal branch (main or master, for example), it allows testing their branch code in a "production-like".
But, imagine that one of service dependencies is mongodb, when they set up a service, platform will create a mongodb cluster for that service.
It's perfect, because I can do it using this project.
Problem
The problem is, I want to provide a way, when that branch is deleted, every single service dependency needs to be deleted too.
What I figured is that, the operator doesn't delete the cluster, when kubernetes atlas cluster resource is deleted.
Feature
I'd like to suggest two nice features

The first one, of course, it is the possibility to delete an atlas mongodb cluster when kubernetes resource is deleted.
The second one, it is the possibility, maybe, in the CRD or using any annotation,  a feature that impossibility a cluster to be deleted, even the kubernetes atlas cluster resource is deleted, because I don't want the main or master branch has its mongodb cluster deleted :P

What do you think?
Btw, thanks for offering this project :)

Hi @matheusca
Thank you for your detailed feedback!
Let me answer each of the questions:

The removal of the Cluster in Atlas as soon as the K8s resource is removed is already implemented - can you try the latest release and tell us if that works fine
This sounds like a good idea, I'll add it to the list of possible features we support for Trial (~beta) version of the Atlas Operator

Thanks @antonlisovenko :)
1 - Thanks, I didn't know! I am going to update my operator and let you know!
2 - Thanks! If you'd like to, we can discuss how to implement this feature in issue and I could open a PR for you

Hi @matheusca
What we have been thinking so far is two-part:

Add an annotation mongodb.com/atlas-resource-policy="keep" to prevent deleting resources in Atlas when CR's are deleted
Allow creating Atlas project and MongoDBUser CR in a namespace that needs to "connect" to Atlas only and drop secrets with connection strings. So that AtlasCluster CR will not be required if the application only needs to connect to the existing cluster and does not need to manage it.

Feel free to drop a PR if you have an idea of how it could work, so we could discuss it.

related: #213


25% 9/35 build modulesBus error (core dumped)

`/wordpress-hybrid-client/node_modules/cson/node_modules/extract-opts/node_modules/typechecker/source/index.js:17
export function getObjectType (value /* :mixed /) / :string */ {
^^^^^^
0% compilehttp://localhost:8080/webpack-dev-server/
webpack result is served from /
content is served from /home/hillerliao/KuaiPan/nodejs-prj/wordpress-hybrid-client
25% 9/35 build modulesBus error (core dumped)
npm ERR<EMAIL_ADDRESS>devserver: webpack-dev-server --port 8080 --json --progress
npm ERR! Exit status 135
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>devserver script.
npm ERR! This is most likely a problem with the wordpress-hybrid-client package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     webpack-dev-server --port 8080 --json --progress
npm ERR! You can get their info via:
npm ERR!     npm owner ls wordpress-hybrid-client
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 3.19.0-56-generic
npm ERR! command "/usr/bin/node" "/usr/bin/npm" "run" "devserver"
npm ERR! cwd /my/path/wordpress-hybrid-client
npm ERR! node -v v0.10.37
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
`

Your node and npm versions are old. Please update and re install.

hillerliao@hillerliao-ThinkPad-E420:~/KuaiPan/nodejs-prj/wordpress-hybrid-client$ npm run devserver

<EMAIL_ADDRESS>devserver /home/hillerliao/KuaiPan/nodejs-prj/wordpress-hybrid-client
webpack-dev-server --port 8080 --json --progress

0% compilehttp://localhost:8080/webpack-dev-server/
webpack result is served from /
content is served from /home/hillerliao/KuaiPan/nodejs-prj/wordpress-hybrid-client
25% 9/35 build modulesBus error (core dumped)
npm ERR! Linux 3.19.0-56-generic
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "devserver"
npm ERR! node v4.4.1
npm ERR! npm  v2.14.20
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>devserver: webpack-dev-server --port 8080 --json --progress
npm ERR! Exit status 135
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>devserver script 'webpack-dev-server --port 8080 --json --progress'.
npm ERR! This is most likely a problem with the wordpress-hybrid-client package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     webpack-dev-server --port 8080 --json --progress
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs wordpress-hybrid-client
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!
npm ERR!     npm owner ls wordpress-hybrid-client
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/hillerliao/KuaiPan/nodejs-prj/wordpress-hybrid-client/npm-debug.log

On my Mac @shprink
`wordpress-hybrid-client git:(744ad27) ✗ npm run devserver

<EMAIL_ADDRESS>devserver /Users/hillerliao/kuaipan/node-prj/wordpress-hybrid-client
webpack-dev-server --port 8080 --json --progress

0% compileevents.js:141
throw er; // Unhandled 'error' event
^
Error: listen EADDRINUSE <IP_ADDRESS>:8080
at Object.exports._errnoException (util.js:870:11)
at exports._exceptionWithHostPort (util.js:893:20)
at Server._listen2 (net.js:1231:14)
at listen (net.js:1267:10)
at net.js:1376:9
at GetAddrInfoReqWrap.asyncCallback [as callback] (dns.js:63:16)
at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:82:10)
npm ERR! Darwin 15.0.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "devserver"
npm ERR! node v4.4.1
npm ERR! npm  v3.8.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>devserver: webpack-dev-server --port 8080 --json --progress
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>devserver script 'webpack-dev-server --port 8080 --json --progress'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the wordpress-hybrid-client package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     webpack-dev-server --port 8080 --json --progress
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs wordpress-hybrid-client
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls wordpress-hybrid-client
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /Users/hillerliao/kuaipan/node-prj/wordpress-hybrid-client/npm-debug.log`

something changed.
`ERROR in dlopen(/Users/hillerliao/kuaipan/node-prj/wordpress-hybrid-client/node_modules/node-sass/vendor/darwin-x64-46/binding.node, 1): no suitable image found.  Did find:
/Users/hillerliao/kuaipan/node-prj/wordpress-hybrid-client/node_modules/node-sass/vendor/darwin-x64-46/binding.node: truncated mach-o error: segment __LINKEDIT extends to 2493268 which is past end of file 1820763
@ ./lib/scss/bootstrap.scss 4:14-176
ERROR in dlopen(/Users/hillerliao/kuaipan/node-prj/wordpress-hybrid-client/node_modules/node-sass/vendor/darwin-x64-46/binding.node, 1): no suitable image found.  Did find:
/Users/hillerliao/kuaipan/node-prj/wordpress-hybrid-client/node_modules/node-sass/vendor/darwin-x64-46/binding.node: truncated mach-o error: segment __LINKEDIT extends to 2493268 which is past end of file 1820763
@ ./lib/font/font.scss 4:14-171
webpack: bundle is now VALID.`

finally fixed. the GFW made me failding to install node-sass


where is the file saving? if saveAs function is used?

so I got this code here I'm using swift
let dirPath = NSSearchPathForDirectoriesInDomains(NSSearchPathDirectory.DocumentDirectory, NSSearchPathDomainMask.AllDomainsMask, true).first
    let xlsfile = "testtemp.xlsx"
    let fullPath = NSURL(fileURLWithPath: dirPath!).URLByAppendingPathComponent(xlsfile)
    spreadsheet.saveAs(fullPath.absoluteString)

I tested it with
let filemanager = NSFileManager.defaultManager()
    if filemanager.fileExistsAtPath(fullPath.absoluteString){
        print("file is there alright")
    }
    else{
        print("nada nothing")
    }

but it seems like it did not save

Hi @trailburner .
saveAs do not support URL. Use fullPath.path instead.


Introduce ServiceRegistry in controller

This change introduces a ServiceRegistry interface which will eventually
provide access to all service dependencies of the controller. ServiceRegistry
will have a single internal implementation that programmatically configures all
service implementations relevant to the system.
Setup of service implementations thus becomes code and reduces the pain of
associated with service integration (i.e. no more component setup needed).
Additionally this will reduce the complexity of generating services.xml for
controllers in each system.

A big yes! to this.


move LowerCaseString to common for common use

Signed-off-by: wbpcode<EMAIL_ADDRESS>This PR moved Envoy::Http::LowerCaseString to common so that other modules can implement case insensitive string comparison without deps on Http.
This PR is second sub PR of #16049.
Commit Message:
Additional Description:
Risk Level: Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

/retest

It seems clang_tidy simply wants a cleaner constructor here;
Writing fixes to clang-tidy-fixes.yaml ...
clang-tidy check failed, potentially fixed by clang-apply-replacements:
Diagnostics:
- BuildDirectory: /build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy
  DiagnosticMessage:
    FileOffset: 675
    FilePath: include/envoy/common/case_insensitive.h
    Message: use '= default' to define a trivial copy constructor
    Replacements:
    - {FilePath: include/envoy/common/case_insensitive.h, Length: 21, Offset: 723,
      ReplacementText: ''}
    - {FilePath: include/envoy/common/case_insensitive.h, Length: 2, Offset: 744,
      ReplacementText: = default;}
  DiagnosticName: modernize-use-equals-default
  Level: Warning
- BuildDirectory: /build/tmp/_bazel_envoybuild/b570b5ccd0454dc9af9f65ab1833764d/execroot/envoy
  DiagnosticMessage:
    FileOffset: 767
    FilePath: include/envoy/common/case_insensitive.h
    Message: use '= default' to define a trivial copy-assignment operator
    Replacements:
    - {FilePath: include/envoy/common/case_insensitive.h, Length: 50, Offset: 806,
      ReplacementText: = default;}
  DiagnosticName: modernize-use-equals-default
  Level: Warning

/retest

/retest

@lizan I think this PR is ready for review.

/retest

/retest

@lizan Some comments are added.
I personally think it might be better to keep the name Http::LowerCaseString in Http as it is already widely used. Keeping the original name by using would make this PR much cleaner.

Here is a more detailed doc for general tracing. https://docs.google.com/document/d/1gxk5vfHl-Gt3wiNZ-f46Dwmk2NFiEzFJQugmZmpSXk8/edit?usp=sharing
@jmarantz Thank you very much for your continued patience and help. 🌷
Personally, I still think this PR is actually not that closely related to general tracing.
Because all people who want to use the lower case semantics, but do not want to rely on Http can benefit from it. It is not specifically designed for general tracing.

Personally, I still think this PR is actually not that closely related to general tracing.
Because all people who want to use the lower case semantics, but do not want to rely on Http can benefit from it. It is not specifically designed for general tracing.

@wbpcode agreed. This PR seems like a good one to apply if we need such a semantic in the future. There's nothing wrong with moving it, and this PR would be a good start to do that. But I'm not sure if there'd be any more users at this point, so maybe we can put this one on hold till then?
And I still think your C-style function-pointer based validation could (a) be a separate PR from the move and (b) could be a lambda for easier use in some cases.

/wait


Fix the link failure on custom-resource-definition-v1.md

Signed-off-by: sky li (李少凯)-浪潮数据<EMAIL_ADDRESS>There is a link failure problem in the Chines custom-resource-definition-v1.md page, I found and fixed it

Community Recommendations: One language per PR, to facilitate the review of content in different languages. Therefore, it's best to split this problem into two PRs.

https://kubernetes.io/zh-cn/docs/contribute/new-content/
Community Recommendations: One language per PR, to facilitate the review of content in different languages. Therefore, it's best to split this problem into two PRs.


Pr 1555

Description
This PR fixes #
Notes for Reviewers
Resolved issues with PR #1555
Signed commits

[x] Yes, I signed my commits.

@iArchitSharma look good?

@ashutosh-rath02 Can you please add a screenshot of your changes to help it move smoothly?

Before:

After:

@iArchitSharma Sorry for the delay.

Hi @ashutosh-rath02  ,
Thanks for your PR! I encourage you to share this in the Website Meeting tomorrow at 6:30 PM IST. Share your screen, and walk us through it. It's cool to take and share feedback. Check the Doc here to include this PR.
You're welcome at the meeting
Check to community calendar for more details of meetings at Layers.

@ashutosh-rath02
post with authors:

@ashutosh-rath02

still working on it @ashutosh-rath02 ?

still working on it @ashutosh-rath02 ?

Yes, will push commits by tomorrow 😊

@sudhanshutech i am unable to see the recent blog in my branch.

Closing this PR


[ITL-254] Add test for monitor can delete endpoint

What does this PR do?
Starts adding a test for the Monitor can delete enpdoint
Motivation
Make sure we have a test for all endpoints going into the latest generated client (See PR - https://github.com/DataDog/datadog-api-client-java/pull/48/files that includes this endpoint.)
Additional Notes
I noticed an odd behavior where, if the response code isn't 2xx, the data is never serialized into the desired object, but instead returned as an APIException message body. The desired behavior for this endpoint is to get back a 409, and have the nice serialized object showing the list of errors.
ITL-254

Closing, this test was implemented in https://github.com/DataDog/datadog-api-client-java/pull/227


Collection of ideas for the future

On this ticket we want to collect ideas sparked in conversations or implementations... feel free to add yours.
This spec is for now only documenting the 'as-is', however we should not forget 'might-become' ideas too.

💡 -> the profile name available via the standard discovery mechanisms, for example via the Type Indexes (instead of having it in the Solid WebId profile directly).

💡-> have type indices separated per pod. For instance, if I have a Pod for all of my financial data and a separate Pod for my fitness tracking, I wouldn't want the type index for the first pod inside the second pod -- I'd want two separate type indices, based on two separate Solid Profiles
as mentioned on the webId-profile gitter chat.

@timea-solid, just wondering: why are these ideas not in separate issues? Or in discussions, which by default come with an 'idea' category?

bulb -> the profile name available via the standard discovery mechanisms, for example via the Type Indexes (instead of having it in the Solid WebId profile directly). as mentioned on the webId-profile gitter chat

As  I understand it, it is not just the name that Wouter is concerned about, it is everything he labels "social media profile data".   I find it a horrible idea to remove them out of the main profile.  If we end up having to accommodate the second hop on from the WebID Profile to the Solid Profile, this would mean that an app would have to go through three hops to even get a person's name, friends, communities and other data which belongs, in my opinion front and center in the profile.

bulb-> have type indices separated per pod. For instance, if I have a Pod for all of my financial data and a separate Pod for my fitness tracking, I wouldn't want the type index for the first pod inside the second pod -- I'd want two separate type indices, based on two separate Solid Profiles as mentioned on the webId-profile gitter chat. and also here

The idea of multiple profiles per WebID is interesting.  We decided in one of the past meetings that on @timbl's suggestion we stick with one profile per WebID for now and perhaps tackle the more complex issue of multiple profiles per WebID later.  One serious problem with multiple profiles per WebID is - how do they share anything in common e.g. a public profile with an inbox and three sub-profiles with inbox all four accessible by looking for <?WebID> ldp:inbox where WebID is the same for the main profile and the other three.
The idea of multiple type indexes, however is much easier to handle - there is no problem if there are duplicates, one could easily load type indexes from the main profile for public stuff and then additional app-based type indexes too.  If we could store type-registrations in seeAlso files, we could have multiple seeAlsos, each permissioned for a different app.  This would not, AFAIK impact anything else but would solve a major complaint of the interop group that any app can see the type registrations of any other app.

Ok, I have to stop the discussion right here @jeff-zucker and @woutermont.
Since this repo does not 'yet' have a discussion feature I wanted to make sure we do not forget about these ideas.
Also, to answer the previous question: with the clear structure above-> I wanted to avoid ANY discussion on these future ideas because the priority right now is to write the what-is of the Type Indexes.
I personally believe we should (nor do I have time) to digest and get into debates on what there should be in the future, at this point.
That being said: I cannot stop people who do want to debate. These debates can be used later, on the next version of this documentation -> hence -> feel free to open issues and discuss away (label as discussion please).
Of course, if the discussion is on current matters of Type Indexes and not future ideas then we will need to discuss.

Agree, I understood this as a place to jot down thoughts for future exploration, not to put additional tasks on the current draft.

[W]ith the clear structure above-> I wanted to avoid ANY discussion on these future ideas because the priority right now is to write the what-is of the Type Indexes.

That's what I thought, which is why I asked for separate issues/discussions instead of taking over this one 🙂

I personally believe we should (nor do I have time) to digest and get into debates on what there should be in the future, at this point.

I think we should, though, since that's what's going on: people in the ecosystem are thinking and talking about the best way to do data discovery.

These debates can be used later, on the next version of this documentation [...] label as discussion and `future iteration'

Do you characterise this proposal as a documentation of the current use of type indexes? If so, why does it contain normative text? If not, why would well-argumented proposals not be considered for inclusion in the current version?

@jeff-zucker, without in any way devaluing the content of your specific comments, could you maybe hide them, in order to retake them in specific issues/discussions?

I hear both inputs. Thank you.
To try to have a structured discussion with follow-up actions:

I changed this ticket into a discussion on how to work on this repo;
there is now an overview of idea collections here;
feel free to open up a discussion on your ideas as is this and label it with discussionand future idea so the next editors can start from there for next version.

Do you characterise this proposal as a documentation of the current use of type indexes? If so, why does it contain normative text? If not, why would well-argumented proposals not be considered for inclusion in the current version?

@woutermont please open a separate discussion ticket for this additional topic.
I asked Jeff to create separate discussion tickets for his comments.

Do you characterise this proposal as a documentation of the current use of type indexes? If so, why does it contain normative text? If not, why would well-argumented proposals not be considered for inclusion in the current version?

Specification writers can not know in advance whether what they are writing about requires normative text, it is part of their job to find that out .  If someone writes something normative you don't agree with, object.  If someone submits a document which you don't think is a spec by your definition, object.  But you can not demand that the authors of the document tell you in advance what they will be saying.

If not, why would well-argumented proposals not be considered for inclusion in the current version?

Because the community has existed too long without these topics being covered in an authoritative way (whether normatively or not) and getting a spec out the door is a high priority.  You are welcome to submit "well-argumented proposals" in the next draft of the spec.  This doesn't just apply to you, I just proposed something issue #2 and Timea put a "future idea" tag on it.  I'd love it to be in the current draft but recognize that putting it up for future consideration is the best choice at the moment.

@timea-solid, thanks! My first suggestion for easier conversations would be to open up the GitHub Discussions tab, if that is possible.

@jeff-zucker

Specification writers can not know in advance whether what they are writing about requires normative text

Well they should. When you start writing a document, you have a goal. That goal can be of the form "let's describe how things are done" or of the form "let's say how things should ideally be done." I'm not demanding impossible efforts like reciting me the future text of the document. What I am asking is which of the two is the goal of this document.

[T]he community has existed too long without these topics being covered in an authoritative way [...] and getting a spec out the door is a high priority.

So we're cutting corners because all the stuff that works should be confirmed as being correct as soon as possible? There are companies and government agencies eagerly awaiting SAI for almost two years; does that mean that we should ship it without thinking about compatibility with current apps? No, we tell them it will be ready when it is ready, and we make do with parts of it for now. Luckily I can now finally go to them with a new answer: SAI is ready, it stopped bothering with compatibility, since the people to be compatible with don't bother either; although, then I could as well advise them to let Solid go.

Sorry @timea-solid, couldn't help but reacting to @jeff-zucker. I've hidden my comment. @jeff-zucker, if you want to react to it, maybe you better do so in a new issue or in Gitter.

Yes, I likewise hidden everything of mine that was a comment on another suggestion because this issue is for listing issues, not discussing them,


Dictionary Tabs should behave like browser tabs

dictionary tabs should have a dynamic width and should stay on the same row at all times, if possible.

cleaner more organized look. visual hierarchy and makes it easier to focus on the important parts

This should be less of a problem in the next version of JL thanks to https://github.com/rampaa/JL/commit/b5fef5b1db19431fd4f97218a5f534d7364594b1. Also try using shorter names (e.g. "CWD" instead of "Custom Word Dictionary").
I will have a look at this when I have the time but chances are implementing this won't worth the effort.

i agree, hiding dicts without result and renaming should have same effect (have all of them on one line).


Palindrome Flipping Solution : Codechef Starters 60 in java

Problem
Chef has a binary string SS of length NN.
In one operation, Chef can:
Select two indices ii and jj (1 \le i, j \le N, i \ne j)(1≤i,j≤N,i=j) and flip S_iS
i

and S_jS
j

. (i.e. change 00 to 11 and 11 to 00)
For example, if S = 10010S=10010 and chef applys operation on i = 1i=1 and j = 3j=3 then: \underline{1}0\underline{0}10 \rightarrow 00110
1

0
0

10→00110.
Find if it is possible to convert SS to a palindrome by applying the above operation any (possibly zero) number of times.
Note: A string is called a palindrome if it reads the same backwards and forwards, for e.g.<PHONE_NUMBER> and 01100110 are palindromic strings.

@bhumikatewary mam I had added the problem statement and output in the file as per changes requested.
Kindly review my PR under Hacktotoberfest 2022.


Storing in react-context?

This package is super helpful, thank you.
Do you by any chance have any opinions / ideas on storing the csrf token in react context? I'd like to do this for 2 reasons:

Instead of having to pass it around subcomponents, I can pull it out of react context.
I'm using react-query to make my API requests and often they are simply JSON requests. Ideally, I would just use the header defined approach to avoid having to pass it in as a key and polluting my API request body. Having it in the context would aid with that.

Does this present any security concerns?

Happy to hear you're finding edge-csrf useful! Apologies for the late reply. It's definitely possible to use React context to share the CSRF token but it would have some drawbacks, namely that if you fetch the token asynchronously (e.g. inside an useEffect() hook) then your code will still have to handle the case where the token isn't available yet. Alternatively you can create a library function that fetches the CSRF token synchronously and provides it to your code on an as-needed basis. You can cache the token for follow-on requests and further optimize the function by pre-fetching the token on page load or providing it in the initial html depending on your needs. Here are some examples you can use as a guide:
https://github.com/amorey/edge-csrf/tree/next14/examples/next13-approuter-js-submission-static
https://github.com/amorey/edge-csrf/tree/next14/examples/next13-approuter-js-submission-dynamic
I've been thinking about how to add a method to the core library to do this and so far I can't come up with a developer-friendly yet performant way to do it. Maybe "Partial Prerendering" in Next.js 14 might be a good way to do it but I'm not sure yet.


Review Worker Routes

https://developers.cloudflare.com/workers/platform/routing/routes/

done, updated on dashboard to use subdomain


Add github style links to docs sections

Fixes #5106.

The next few days is fine, on firefox there are some buttons in the developer console (Ctrl+Shift+i).

@Anton-4 Okay, that should be it 🙂

Sorry, I've seen that the commits weren't linked to my account and amended them with my correct email. 😅

Can you merge this @rtfeldman?

Rebased and signed commits. 🙂


fix: bump version for v0.9.2 to align '--version' with the tagged installation

Resolves #143

@jufemaiz please fix DCO then we can make it land.

@daixiang0 have done. Please advise.

@jufemaiz thanks for your contribution!

Thanks for a great package!


ngf-file attribute fails when is called in a modal

Hi
i was triying to make the implementation of the ngf-file in my current project, but came across a bug that involves angular-bootstrap ($modal) and of course this repo
I'm calling the select of the file inside a modal
At the first time is called everythig works great but since the modal is closed and try to re-open it the modal makes a compile call wich it will never display again the modal page since the compiler caught as an error the declaration of the attributes ngf-file
i did the test to call the modal several times without that attribute and works as expected but i cant use the plugin as it is in the demo
Additional data:
the template that is loaded in the modal is an outside file

I don't see how this issue is related to this plugin, provide steps to reproduce.


/latest/ doesn't link to the latest version

Example:
https://crate.io/docs/crate/crash/en/latest/ forwards to https://crate.io/docs/crate/crash/en/0.27/ which is not the latest version, 0.28 is. So the note is right, but the redirect from /latest/ is pointing to the wrong version.

On this matter, @hammerhead just submitted https://github.com/crate/infrastructure/pull/2891.

I've just applied the change. I think the problem is fixed. Can you confirm?

nice, looking good! thanks @amotl and @hammerhead 🙌


[FEATURE] Load from CDN

Describe how this feature will help us improve
I like to load as much as possible through CDNs as it decreases my personal bandwidth and increases the chance of a cache hit for my users.
Describe the solution you'd considered
UNPKG (unpkg.com) provides CDN service for all packages on NPM. If the NPM package included minimized js and compiled CSS then we would be able to load it using the UNPKG CDN.
Additional context
Maybe I am missing something that would make this impossible or not useful.

Are you talking about putting the library on a global CDN for example how many libraries are doing?
Example: Google's Hosted Libraries
There is both pros and cons of doing that, the biggest downside is security.

Hi @jminardi, performance improvements via CDN for internal JS are a common myth, see e.g. https://csswizardry.com/2019/05/self-host-your-static-assets/ ("cross domain caching"), in fact, hosting crucial JS externally will most likely degrade performance.
As Storefront UI is very crucial for sites using it (often you don't see a single pixel while it is loading), I'd strongly encourage everybody to self-host it and integrate it in your code base (as a dependency) and workflow/build flow. Then then there is little be concerned about because SFUI is strongly 🚀 performance-driven 😉
AFAIK there are no plans to pre-compile SFUI for now. @filrak correct me if I'm wrong.


improving audio quality? (4-speaker)

Hi, we've been experimenting TTS in two setups:
tacotron2 for mel-spectrogram + vocoder (4-speaker)
tacotron2 for linear-spectrogram (directly, without post-net) + griffin-lim (4-speaker)
sounds better than 2), but it's still not as perfect as human ground-truth speech. (it still has very slight vibrating noise)
(i've uploaded samples below, please check out)
Any ideas on how to improve these further (in terms of post-processing or whatever)? we're pretty sure each component of each setup is trained pretty enough. our data is about 25 hours in total.
Thanks in advance!
samples.zip

It really depends on the training regime you are with Tacotron 2 and the vocoder.

Closing due to inactivity.


Merge Punchblock and Adhearsion

It is likely that at this stage it is worth considering merging Punchblock into Adhearsion core rather than remaining a separate dependency.
Arguments for:

Users never know when to file issues against Adhearsion or Punchblock
Adhearsion and Punchblock's development cycles are linked such that they may as well have one version number. See https://github.com/adhearsion/adhearsion/pull/527, https://github.com/adhearsion/adhearsion/blob/develop/lib/adhearsion/generators/app/templates/Gemfile.erb#L5-L9, etc
Writing Adhearsion apps requires knowledge of Punchblock. See https://github.com/adhearsion/adhearsion/blob/develop/lib/adhearsion/call_controller/output.rb#L21, https://github.com/adhearsion/adhearsion/blob/develop/lib/adhearsion/call_controller/output.rb#L133, https://github.com/adhearsion/adhearsion/blob/5019eba159a0f84dbc5d74064e1b40a4ddabaa5c/lib/adhearsion/call_controller/input.rb#L31-L32, https://github.com/adhearsion/adhearsion/blob/5019eba159a0f84dbc5d74064e1b40a4ddabaa5c/lib/adhearsion/call_controller/record.rb#L56.
Punchblock is not used by anyone else anyway

Arguments against:

Punchblock is used by software other than Adhearsion applications
Adhearsion should remain Rayo-only in core
Punchblock is a cool name and we don't want it to go away

Generally, I think this is a win for the reasons you explained.  I'd like to dig into the arguments against:
Arguments against:

Punchblock is used by software other than Adhearsion applications
Who? What?

Adhearsion should remain Rayo-only in core
I understand the argument for this being keeping the Adhearsion codebase simpler, but why else should this be the case? We can still define a Rayo-like API between the Translator layer (with a noop to Rayo on XMPP, and actual translation when headed to AMI). Does that not satisfy this?

Punchblock is a cool name and we don't want it to go away
This is not a good reason to keep a project alive, though I do agree about the name :)

I've updated those points above in response to your questions.

Closing in favour of https://github.com/adhearsion/adhearsion/pull/561


Updating CreditsState.hx

You know already, so pull it

you could have squashed both changes into 1 pr
if i removed you from the credits people would assume i stole the results screen from your engine (despite you literally giving permission)

you could have squashed both changes into 1 pr
if i removed you from the credits people would assume i stole the results screen from your engine (despite you literally giving permission)

Didn't i told you somewhere I didn't made this results screen? I simply port it from one FNF Engine, not fork

you could have squashed both changes into 1 pr
if i removed you from the credits people would assume i stole the results screen from your engine (despite you literally giving permission)

Didn't i told you somewhere I didn't made this results screen? I simply port it from one FNF Engine, not fork

yeah you still gave me permission to use it though


Bug Report: objID strings are incorrectly converted to number

About me
This bug report is posted on behalf of the BBC
Observed Behavior
When ingesting a rundown with MOS objID's that look like numbers sofie-mos-connection incorrectly converts them to a number type.
E.G:
100.543210 becomes 100.54321
I'm guessing that this is caused by the nativeType: true option for the parser, I'm not sure what impact changing it would have...
https://github.com/nrkno/sofie-mos-connection/blob/a87681fed13d3379c41c78cd487606aabbf48128/packages/helper/src/utils/Utils.ts#L13
Expected Behavior
Data from a MOS device should remain of a valid type.
Version
release51
Severity / Impact
This is a significant blocker for us as our clip objIDs are numeric as above.
A workaround has been implemented so this is less urgent but still an issue.

PR has been made for this fix:
https://github.com/nrkno/sofie-mos-connection/pull/101

This should now be resolved as of version v4.2.0-alpha.1, please reopen this if any issues persists,


What should I do if I deleted my ssh key before.

The exist ssh key in metadata.json has been lost, how can I get the $FLT?

I worked on it for a long time, and the key was deleted from the computer?

Dublicate of #63


When you ask Navigator to display a taxon that is an unresolved homonym, it displays one of them taxa, but needs to ask which one you want to display.

For example, I asked Navigator to display Probscidea. I got a very strange distribution, and realized it was displaying an insect group called Proboscidea, not the mammalian order Proboscidea.
I think in cases like this Navigator needs to ask which group you want displayed like Classic does, displayed below.

I will update the autocomplete operation to provide additional information; the autocomplete code needs to be updated to display it.

I have reactivated this bug and moved it to the highest level of alert. Indeed, we have an autocomplete function that APPEARS to let you select only the mammals, but it is still mixing up the the insects with the mammals when you map Proboscidea.

Here is more data on the current problem from the person who reported it to me. We need to fix and thoroughly test this ASAP.
I had seen that option myself [the current autofil option to slect Proboscidea Mammalia], but when I select it it only shows the insects regardless (while the search bars recognize there is a mammalian proboscidea, it fails to actually recognize that when trying to map taxa; if moving upward (say, from elephantoidea or elephantiformes to proboscidea, it also automatically changes over to the insect group).
This is most obvious on the maps too because it shows lots of stuff predating the cenozoic, whereas probosidean mammals (elephants and such) only show up starting around 60 mya. The workaround I've found is just to target all of the subtaxa contained with proboscidea, but it's a suboptimal solution.

Issue has been fixed, pending merge into master branch and pull to production.


Operation on messages in rx-js-chat

this.messages = this.updates
      // watch the updates and accumulate operations on the messages
      .scan((messages: Message[],
             operation: IMessagesOperation) => {
               return operation(messages);
             },

In above code snippet, what operation exactly is ? - "return operation(messages);". I don't see "operation" defined anywhere except there is an interface.  I am looking for concrete method.
Thanks!

Operations is way to perform changes to all the messages in the stream , at the same time.
In the above snippet operation is an inner function used for concatenation.
.scan((messages: Message[],
 operation: IMessagesOperation) => {
  return operation(messages);              },
Here the incoming message stream is being scanned and then an " operation " is applied to it to add the new messages to the list of messages. The "operation" here being a concat operation, as it is updating the message stream with all the new messages.
You can do this manually yourself, without using the operation function, by adding the concatenation procedure in a later function call ( such as the addMessage() ) to update the message stream directly. However, although this leads to more semantic code, the message stream is no longer composable after this.


is there any plan to support pytorch 2.0+

After modifying code blow, nerfstudio can run with pytroch 2.0.
--- a/nerfstudio/data/utils/nerfstudio_collate.py
+++ b/nerfstudio/data/utils/nerfstudio_collate.py
@@ -23,8 +23,10 @@ from typing import Callable, Dict, Union

 import torch
 import torch.utils.data
-from torch._six import string_classes
-
+try:
+    from torch._six import string_classes
+except:
+    string_classes = (str, bytes)
 from nerfstudio.cameras.cameras import Cameras

But use default fp16 training, it will get error.
ns-train nerfacto --data data/poster --vis wandb

Traceback (most recent call last):
  File "/home/user/.local/bin/ns-train", line 8, in <module>
    sys.exit(entrypoint())
  File "/workspace/3dvision/nerfstudio/scripts/train.py", line 247, in entrypoint
    main(
  File "/workspace/3dvision/nerfstudio/scripts/train.py", line 233, in main
    launch(
  File "/workspace/3dvision/nerfstudio/scripts/train.py", line 172, in launch
    main_func(local_rank=0, world_size=world_size, config=config)
  File "/workspace/3dvision/nerfstudio/scripts/train.py", line 87, in train_loop
    trainer.train()
  File "/workspace/3dvision/nerfstudio/nerfstudio/engine/trainer.py", line 218, in train
    loss, loss_dict, metrics_dict = self.train_iteration(step)
  File "/workspace/3dvision/nerfstudio/nerfstudio/utils/profiler.py", line 43, in wrapper
    ret = func(*args, **kwargs)
  File "/workspace/3dvision/nerfstudio/nerfstudio/engine/trainer.py", line 395, in train_iteration
    self.optimizers.optimizer_scaler_step_all(self.grad_scaler)
  File "/workspace/3dvision/nerfstudio/nerfstudio/engine/optimizers.py", line 130, in optimizer_scaler_step_all
    grad_scaler.step(optimizer)
  File "/home/user/.local/lib/python3.10/site-packages/torch/cuda/amp/grad_scaler.py", line 368, in step
    assert len(optimizer_state["found_inf_per_device"]) > 0, "No inf checks were recorded for this optimizer."
AssertionError: No inf checks were recorded for this optimizer.

When i changged to use fp32 training, the result is not stable.
ns-train nerfacto --data data/poster --vis wandb --mixed-precision False

Pytorch seems cool, but it is unclear how it will interact with other libraries like tinycudann and nerfacc. We will probably let the field settle a little before adding support.

@tancik nerfacc@882d992(ahead of v0.3.5) works fine with pytorch nightly and cuda11.8
❯ python examples/train_ngp_nerf.py --train_split train --scene lego
elapsed_time=1.29s | step=0 | loss=0.07298 | alive_ray_mask=256 | n_rendering_samples=68299 | num_rays=256 |
elapsed_time=90.28s | step=10000 | loss=0.00059 | alive_ray_mask=15833 | n_rendering_samples=261834 | num_rays=47332 |
elapsed_time=186.81s | step=20000 | loss=0.00037 | alive_ray_mask=16590 | n_rendering_samples=261559 | num_rays=50510 |
100%|████████████████████████████████████████████████████████████████████████████████| 200/200 [00:52<00:00,  3.78it/s]
evaluation: psnr_avg=35.57740461349487
training stops

here is my python environment
torch                    2.0.0+cu118                                         c:\users\ksnzh\scoop\persist\miniconda3\envs\pt\lib\site-packages                                       pip
nerfacc                  0.3.5           c:\users\ksnzh\workspace\nerfacc    c:\users\ksnzh\workspace\nerfacc

For tinycudann, I run instant-ngp with cuda12 and it seems okey.

PyTorch 2.0 is now supported


Settings panel toggle does not appear

node version:
npm (or yarn) version:
node 6.9.4
npm 3.10.10

Relevant code or config
SettingsPanel.jsx
What you did:

fresh npm install
make start
create a chat
opened settings panel

What happened:

no toggle

Reproduction repository:

Problem description:

the library react-toggle is installed and in my node modules.

Suggested solution:

solution was re-cloning the repo. works as expected.


[Metal] No rendering on iOS device when MSAA is on

On an iOS device (doesn't repro on Mac or Simulator), enabling MSAA with some rendering steps gives no rendering in the backbuffer.
bgfx calls are:

view 0: clear and touch back buffer
view 1: clear and render to a render target
view 2: render to the back buffer

These steps give no rendering in the backbuffer when MSAA is on.
When MSAA is off or calling setViewClear on view 2 fixes the issue. For technical reason, I can't add a setViewClear as a workaround.
A repro is available here in the 01-cubes sample:
https://github.com/bkaradzic/bgfx/compare/master...CedricGuillemet:iosMSAABug?expand=1
branch:
https://github.com/CedricGuillemet/bgfx/tree/iosMSAABug
cc @attilaz

Hi @attilaz , did you find some time to take a look at this issue?

Hi,
Sorry I somehow missed this issue. I have checked it now. I can reproduce the problem.
Current metal renderer doesn't support switching back to msaa buffer.
On Tiled/Deferred gpus reusing msaa buffers requires copying the large msaa buffers back to system then back to tiled memory again. This is slow. Current implementation resolves color buffer and discards all msaa buffers, so we cannot reuse msaa data.
Correct version would be to always copy all msaa buffers back to system memory.
But to avoid pessimization the common case (without reusing msaa data) we should know if we can skip the copy.
Maybe we can analyze subsequent view's framebuffer usage? @bkaradzic
The problematic storeActions are here.
https://github.com/bkaradzic/bgfx/blob/75fe88fe3afb048ac6f775d9991f67a406e1f4a5/src/renderer_mtl.mm#L3972
https://github.com/bkaradzic/bgfx/blob/75fe88fe3afb048ac6f775d9991f67a406e1f4a5/src/renderer_mtl.mm#L3986
https://github.com/bkaradzic/bgfx/blob/75fe88fe3afb048ac6f775d9991f67a406e1f4a5/src/renderer_mtl.mm#L4001
Forcing stores (storeAndResolve) here solves the problem.
Also checking m_mainFrameBuffer.m_swapChain->m_backBufferColorMsaa for setting store action for ALL framebuffers is a bug?


Added OpenLibrary & Share label to Book Details Screen

Thanks! Not sure whether we should keep the URL in the share context menu yet, but merging in for now.


Stats page broken with "Unable to fetch stats"

Visit https://unpkg.com/#/stats
Page is empty
HTTP error is logged: GET https://unpkg.com/_stats?period=last-month 500 {"error":"Unable to fetch stats"}

Thanks, this was a bad config issue. It's been resolved.


What is the data directory or directories used by steamcache?

I am redirecting the data created by the steamcache docker (see my command line below).
sudo docker run --name steamcache --restart=always -d -v /srv/steamcache/data:/data/cache -v /srv/steamcache/logs:/data/logs -p <IP_ADDRESS>:80:80 steamcache/steamcache:latest
Are the /data/cache and the /data/logs the only directories I have to redirect?
Can I just redirect /data to /srv/steamcache/data (or another directory on the computer running the cache) and get the same result?

Sorry for the late response, Yes you can redirect /data, but be sure to create the 3 sub-directories ( cache, info, logs) as we don't check these currently.

What is stored in the info sub-directory?

The info directory is used when using the watchlog.sh script, to store a cached reference of the depot name, Currently we can only find out what the depot name is on steam, not the game name. We're writing our own API for steamdb to be able to work this information out easily and expose it to the image, at this point there will most likely be a sqlite database with these references in the info directory, this way we won't have to make a request to the API for every line in the logfile, we'll simple request once per depot, and cache the result.
IIRC this part of the watchlog.sh script is actually disabled as it currently uses steamdb site and you could end up with an IP Ban (on steamdb.info), the commented out lines in watchlog.sh can be used at your own risk until our system is in-place.


Should be able to monitor log files like the original Fail2Ban, not just Windows Events

I spent 3 hours trying to figure out why it wasn't loading when it turned out to be that it just can't monitor logs.
For example I'd been trying to use this as one of my rules
{
"logName": "F:/xampp/apache/logs/error.log",
"eventId": 0,
"ipAddressPattern": "\[client (?\d+\.\d+\.\d+\.\d+):\d+\]",
"failurePattern": "AH00124"
}
and it just wouldn't load. The idea was to block an exploit attempt I keep seeing pop up in my Apache error.log
Also
{
"logName": "F:/xampp/apache/logs/modsec_audit.log",
"eventId": 0,
"ipAddressPattern": ""client_ip":"(?\d+\.\d+\.\d+\.\d+)"",
"failurePattern": ""status":"218""
}
This one is to monitor modsecurity's audit log to block certain common exploit attempts I've seen come in.
But it just can't monitor log files like the original can, apparently.

Just wanted to +1 this request. I have a similar challenge with IIS logs. I can have the IIS put its logs into the Event Viewer, but the IP address and the rest of the filtering criteria I'd need to account for are in different EventData fields, so I can't use Fail2Ban4Win for this particular challenge. It'd work if I could trigger off of additional fields in an EventLogSelector, or if I could work with the logfile directly, like @JennaScvl is requesting.

Hi @jmoeller-ua,
Feel free to export and upload an .evtx file for the IIS event(s), and I will try to take a look at them and see if they can be selected.
Thanks.

An example entry looks like this:
date 2024-07-10 time 23:18:54 s-sitename W3SVC1 s-computername <my_host> s-ip <my_ip> cs-method POST cs-uri-stem <my_url> cs-uri-query - s-port 443 cs-username - c-ip <remote_ip> cs-version - cs(User-Agent) Mozilla/5.0+(Windows+NT+10.0;+Win64;+x64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/<IP_ADDRESS>+Safari/537.36+Edg/115.0.1901.203 cs(Cookie) - cs(Referer) - cs-host - sc-status 200 sc-substatus 0 sc-win32-status 0 sc-bytes 15322 cs-bytes 422 time-taken 351
The Event Viewer puts each element neatly and correctly into the EventData hashtable, but since I need to reference elements other than just c-ip to know if it's actually a block-worthy event (cs(User-Agent), cs-method, etc). If I could work with the entire message via regexp, just like a line in a logfile, I could be off and running.
Unhelpfully, the IIS logger puts in all events with the same event ID of 6200. But I guess it makes sense for a web log that it wouldn't be able to discern outcomes.

That wasn't an .evtx file, but I figured out how to generate my own:
iis-access-log.evtx
<Event xmlns="http://schemas.microsoft.com/win/2004/08/events/event">
    <System>
        <Provider Name="Microsoft-Windows-IIS-Logging" Guid="{7e8ad27f-b271-4ea2-a783-a47bde29143b}" />
        <EventID>6200</EventID>
        <Version>0</Version>
        <Level>4</Level>
        <Task>0</Task>
        <Opcode>0</Opcode>
        <Keywords>0x8000000000000000</Keywords>
        <TimeCreated SystemTime="2024-07-10T23:39:12.6648021Z" />
        <EventRecordID>1</EventRecordID>
        <Correlation />
        <Execution ProcessID="10412" ThreadID="10788" />
        <Channel>Microsoft-IIS-Logging/Logs</Channel>
        <Computer>Thor.aldaviva.com</Computer>
        <Security UserID="S-1-5-18" />
    </System>
    <EventData>
        <Data Name="EnabledFieldsFlags">2478079</Data>
        <Data Name="date">2024-07-10</Data>
        <Data Name="time">23:39:09</Data>
        <Data Name="c-ip"><IP_ADDRESS></Data>
        <Data Name="cs-username">-</Data>
        <Data Name="s-sitename">W3SVC1</Data>
        <Data Name="s-computername">Thor</Data>
        <Data Name="s-ip"><IP_ADDRESS></Data>
        <Data Name="cs-method">GET</Data>
        <Data Name="cs-uri-stem">/health-check.txt</Data>
        <Data Name="cs-uri-query">-</Data>
        <Data Name="sc-status">304</Data>
        <Data Name="sc-win32-status">0</Data>
        <Data Name="sc-bytes">0</Data>
        <Data Name="cs-bytes">766</Data>
        <Data Name="time-taken">103</Data>
        <Data Name="s-port">443</Data>
        <Data Name="csUser-Agent">Mozilla/5.0+(Windows+NT+10.0;+Win64;+x64)+AppleWebKit/537.36+(KHTML,+like+Gecko)+Chrome/<IP_ADDRESS>+Safari/537.36</Data>
        <Data Name="csCookie">-</Data>
        <Data Name="csReferer">-</Data>
        <Data Name="cs-version">-</Data>
        <Data Name="cs-host">-</Data>
        <Data Name="sc-substatus">0</Data>
        <Data Name="CustomFields" />
    </EventData>
</Event>

This can be enabled by going to the Logging feature page for a site in IIS Manager, configuring it to use W3C format and log ETW events, and enabling and watching Event Viewer > Application and Service Logs > Microsoft > Windows > IIS-Logging > Logs.

Given this structure, it seems like allowing an additional optional configuration property to be specified in the EventLogSelector to act as a predicate in the XPath query generated by selectorToQuery would allow you to filter based on the user agent string, request method, response status code, and other data element values.

That wasn't an .evtx file,

Indeed, I just wanted to strip out the identifying information. I had set it up just as you had mentioned.

Given this structure, it seems like allowing an additional optional configuration property to be specified in the EventLogSelector to act as a predicate in the XPath query generated by selectorToQuery would allow you to filter based on the user agent string, request method, response status code, and other data element values.

Yes, that's sort of what I was imagining, additional optional configuration items that I could refer to put a specific event in scope of Fail2Ban4Win. Would love to see it if you think it'd be a reasonable feature to implement.

Right, those ETW instructions were for me and anyone else who looks at this issue in the future, because I had to figure it out and was worried I'd forget next time I have to do it (when updating unit tests, adding new dependent features, etc).
I believe adding the additional XPath predicate to the configuration is a reasonable feature to add. I will try to find the time soon to try it out and send you a development snapshot build to see if it fits your use case too before releasing it.

@jmoeller-ua: give this a try and let me know how well it fits your use case
Developer snapshot executable
Fail2Ban4Win.zip
(minimally tested so far)
Configuration
There is now a new optional property of EventLogSelector configuration objects with the key eventPredicate and the value of a string. The value is an XPath 1.0 predicate expression, which must be surrounded by square brackets and is evaluated by ETW against the top-level Event object of the ETW log record element.
Multiple predicates can be constructed with XPath and and or operators, or by concatenating multiple predicates like [predicate1][predicate2] instead of using the and operator. Note that functions like contains() and starts-with() are unfortunately not supported by Windows ETW.
Example
This selector should select only IIS access logs where the response status code is 403.
{
    "logName": "Microsoft-Windows-IIS-Logging/Logs",
    "source": "IIS-Logging",
    "eventId": 6200,
    "ipAddressEventDataName": "c-ip",
    "eventPredicate": "[EventData/Data[@Name='sc-status']=403]"
}

This will result in Fail2Ban4Win using an effective XPath expression of
*[System/EventID=6200][System/Provider/@Name=\"IIS-Logging\"][EventData/Data[@Name='sc-status']=403]

Here is another example that matches requests with both status code 304 and request method GET.
{
    "eventPredicate": "*[EventData[Data[@Name='sc-status']='304'][Data[@Name='cs-method']='GET']]"
}

That would be fantastic. I'd be happy to try it out.

From: Ben Hutchison @.>
Sent: Wednesday, July 10, 2024 5:42:11 PM
To: Aldaviva/Fail2Ban4Win @.>
Cc: Moeller, John D - (jmoeller) @.>; Mention @.>
Subject: [EXT] Re: [Aldaviva/Fail2Ban4Win] Should be able to monitor log files like the original Fail2Ban, not just Windows Events (Issue #33)
External Email

Right, those ETW instructions were for me and anyone else who looks at this issue in the future, because I had to figure it out and was worried I'd forget next time I have to do it (when updating unit tests, adding new dependent features, etc).
I believe adding the additional XPath predicate to the configuration is a reasonable feature to add. I will try to find the time soon to try it out and send you a development snapshot build to see if it fits your use case too before releasing it.
—
Reply to this email directly, view it on GitHubhttps://github.com/Aldaviva/Fail2Ban4Win/issues/33#issuecomment-2221774582, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APM4GNXCFXGYFIEAQ5THIBLZLXIGHAVCNFSM6AAAAABJG57YP6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRRG43TINJYGI.
You are receiving this because you were mentioned.Message ID: @.***>

@jmoeller-ua: give this a try and let me know how well it fits your use case
Developer snapshot executable
Fail2Ban4Win.zip (minimally tested so far)

It works great, this is amazing. I can't believe how fast you did this!

Thanks! Great to hear it's working for you.
This was a pretty straightforward change because it leveraged Windows' existing ETW XPath filtering, which this project was already using. By adding the one new optional, backwards-compatible configuration property, it augmented the existing Event ID and Source filtering that this project had already been set up to use. So it was a small change that did not reinvent the wheel.

I'm splitting the eventPredicate feature that we've been discussing here into a different issue, #34, because it resulted in a different implementation than the topic of this issue, tailing text log files. Any further eventPredicate discussions can happen in #34, while this issue can remain for text log files.


chore(main): release 1.1.2

:robot: I have created a release beep boop
1.1.2 (2023-01-19)
update

添加首页获取推荐与精品数据 (33614dd)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/Knock-Ding-Yanxuan/releases/tag/v1.1.2 :sunflower:


sqlite3.OperationalError: unable to open database file

In the error log for srpbs I get the following (some lines removed):
Traceback (most recent call last):
  File "/home/nclarke/.local/lib/python3.10/site-packages/sqlalchemy/engine/base.py", line 1256, in _execute_context
    self.dialect.do_executemany(
  File "/home/nclarke/.local/lib/python3.10/site-packages/sqlalchemy/engine/default.py", line 605, in do_executemany
    cursor.executemany(statement, parameters)
sqlite3.OperationalError: unable to open database file

The above exception was the direct cause of the following exception:

File "/home/nclarke/.local/lib/python3.10/site-packages/sqlalchemy/engine/default.py", line 605, in do_executemany
    cursor.executemany(statement, parameters)
sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) unable to open database file
[SQL: INSERT INTO files (path, filename, dirname, is_dir, class_) VALUES (?, ?, ?, ?, ?)]
(Background on this error at: http://sqlalche.me/e/13/e3q8)

This is the output of .out file:
/home/nclarke/scratch/srpbs_fmriprep-20.2.7lts_1691842839/data
Namespace(bids_dir=PosixPath('/home/nclarke/scratch/srpbs_fmriprep-20.2.7lts_1691842839/data/fmriprep-20.2.7lts'), output_dir=PosixPath('/lustre04/scratch/nclarke/srpbs_qc'), analysis_level='participant', participant_label=None, session=None, task=None, quality_control_parameters=None, reindex_bids=False, verbose=1)
Quality control parameters: {'mean_fd': 0.55, 'scrubbing_fd': 0.2, 'proportion_kept': 0.5, 'anatomical_dice': 0.97, 'functional_dice': 0.89}

The path is correct.
The background at the link says "OperationalError. Exception raised for errors that are related to the database’s operation and not necessarily under the control of the programmer, e.g. an unexpected disconnect occurs, the data source name is not found, a transaction could not be processed, a memory allocation error occurred during processing, etc.
This error is a DBAPI Error and originates from the database driver (DBAPI), not SQLAlchemy itself.
The OperationalError is the most common (but not the only) error class used by drivers in the context of the database connection being dropped, or not being able to connect to the database. For tips on how to deal with this, see the section Dealing with Disconnects."
I don't think it is a space issue since I have nearly 20,000GB available. I am thinking to increase --mem (currently 8G), what do you think @htwangtw, have you come across this before?

Since it's coming from sqlalchemy, it might be something to do with PyBIDS...
Do you have the trace back to which line in giga_auto_qc is giving you the error?
The first thing I will try is use the --reindex_bids flag

after some googling, I believe this is a pybids related thing and the solution is (potentially) this one:
https://github.com/nipreps/fmriprep/issues/2313#issuecomment-1013680975
AFAIK it's better to

delete the pybids layout sql database that was not built correctly
build the pybids layout first on the full dataset
run the qc with all the default flag.

UKBB didn't have this issue as each subject was treated as an isolated dataset.
If the proposed solution works, do you mind making a PR in the README.md to keep this as note to users?

Ah! Interesting. Here is the relevant part of the traceback ( I think):
  File "/home/nclarke/.local/bin/giga_auto_qc", line 8, in <module>
    sys.exit(main())
  File "/lustre03/project/6003287/nclarke/giga_preprocess2/giga_auto_qc/giga_auto_qc/run.py", line 79, in main
    workflow(args)
  File "/lustre03/project/6003287/nclarke/giga_preprocess2/giga_auto_qc/giga_auto_qc/workflow.py", line 47, in workflow
    fmriprep_bids_layout = BIDSLayout(
  File "/home/nclarke/.local/lib/python3.10/site-packages/bids/layout/layout.py", line 154, in __init__
    indexer(self)

Yes I can try that!

@htwangtw - Progress, but now a new error:
^M0it [00:00, ?it/s]^M0it [00:00, ?it/s]
Traceback (most recent call last):
  File "/home/nclarke/.local/bin/giga_auto_qc", line 8, in <module>
    sys.exit(main())
  File "/lustre03/project/6003287/nclarke/giga_preprocess2/giga_auto_qc/giga_auto_qc/run.py", line 79, in main
    workflow(args)
  File "/lustre03/project/6003287/nclarke/giga_preprocess2/giga_auto_qc/giga_auto_qc/workflow.py", line 69, in workflow
    anatomical_metrics = assessments.calculate_anat_metrics(
  File "/lustre03/project/6003287/nclarke/giga_preprocess2/giga_auto_qc/giga_auto_qc/assessments.py", line 345, in calculate_anat_metrics
    metrics["anatomical_dice"]
  File "/home/nclarke/.local/lib/python3.10/site-packages/pandas/core/frame.py", line 3760, in __getitem__
    indexer = self.columns.get_loc(key)
  File "/home/nclarke/.local/lib/python3.10/site-packages/pandas/core/indexes/range.py", line 349, in get_loc
    raise KeyError(key)
KeyError: 'anatomical_dice'

This is the output:
/home/nclarke/scratch/srpbs_fmriprep-20.2.7lts_1691842839/data
Namespace(bids_dir=PosixPath('/home/nclarke/scratch/srpbs_fmriprep-20.2.7lts_1691842839/data/fmriprep-20.2.7lts'), output_dir=PosixPath('/lustre04/scratch/nclarke/srpbs_qc'), analysis_level='participant', participant_label=None, session=None, task=None, quality_control_parameters=None, reindex_bids=False, verbose=1)
Quality control parameters: {'mean_fd': 0.55, 'scrubbing_fd': 0.2, 'proportion_kept': 0.5, 'anatomical_dice': 0.97, 'functional_dice': 0.89}
Retrieved anatomical reference mask
Use standard template as functional scan reference.
Calculate the anatomical dice score.

I tried adding some print statements to assessments.py to probe the problem but they didn't print which I'm confused about. Any ideas? Thank you!

Yay to the progress!
The error is from the pandas dataframe so it might worth looking into that
I will try to probe at it too

I ran the test on two subjects using this command: pytest -m "not smoke" --doctest-modules -v --pyargs giga_auto_qc. I'm not sure if this is completely correct as this is quite beyond my experience. But this is the output I got, which suggests that it should complete...?
[mii] loading StdEnv/2020 pytest/7.4.0 ...
========================================= test session starts =========================================
platform linux -- Python 3.10.2, pytest-7.4.0, pluggy-1.2.0 -- /cvmfs/soft.computecanada.ca/easybuild/software/2020/avx512/Core/python/3.10.2/bin/python
cachedir: .pytest_cache
rootdir: /lustre03/project/6003287/nclarke/giga_preprocess2/giga_auto_qc
configfile: pyproject.toml
collected 10 items / 3 deselected / 7 selected

giga_auto_qc/tests/test_assessments.py::test_quality_accessments PASSED                         [ 14%]
giga_auto_qc/tests/test_assessments.py::test_dice_coefficient PASSED                            [ 28%]
giga_auto_qc/tests/test_assessments.py::test_check_mask_affine PASSED                           [ 42%]
giga_auto_qc/tests/test_assessments.py::test_get_consistent_masks PASSED                        [ 57%]
giga_auto_qc/tests/test_cli.py::test_help PASSED                                                [ 71%]
giga_auto_qc/tests/test_utils.py::test_get_subject_lists PASSED                                 [ 85%]
giga_auto_qc/tests/test_utils.py::test_parse_scan_information PASSED                            [100%]

============================= 7 passed, 3 deselected in 354.09s (0:05:54) =============================

Great, let me clarify what I was trying to say: you can modify the code in the test file, run it in an interactive session, so we can find out why it is acting weird on your dataset.
Your dataset might have found some edge cases that's not covered currently...
from bids import BIDSLayout
import templateflow
from giga_auto_qc import assessments

bids_dir = "path/to/your/data"
subject_list = ["1"]  # any subject number, remove `sub-`
fmriprep_bids_layout = BIDSLayout(
    root=bids_dir,
    database_path=bids_dir,
    validate=False,
    derivatives=True,
    reset_database=False,  # let's assume you already have a valid bids index
)
template_mask = templateflow.api.get(
    ["MNI152NLin2009cAsym"], desc="brain", suffix="mask", resolution="01"
)
df = assessments.calculate_anat_metrics(
    subject_list,
    fmriprep_bids_layout,
    {"anat": template_mask},
    {"anatomical_dice": 0.97},
)

I think that's what I did, except I didn't change reset_database to False... Is that what I did (see below)? Lol. Shall I re-run with reset_database=False?
def test_calculate_anat_metrics():
    bids_dir = resource_filename(
        "giga_auto_qc",
        "/home/nclarke/scratch/srpbs_fmriprep-20.2.7lts_1691842839/data/fmriprep-20.2.7lts",
    )
    fmriprep_bids_layout = BIDSLayout(
        root=bids_dir,
        database_path=bids_dir,
        validate=False,
        derivatives=True,
        reset_database=True,
    )
    template_mask = templateflow.api.get(
        ["MNI152NLin2009cAsym"], desc="brain", suffix="mask", resolution="01"
    )
    df = assessments.calculate_anat_metrics(
        ["0246", "0603"],
        fmriprep_bids_layout,
        {"anat": template_mask},
        {"anatomical_dice": 0.97},
    )
    print(df)

Hmmm good to know indexing is not an issue. If indexing fails, it will just not run correctly.
My guess is it might be something wrong with one subject?
I genuinely have no idea how that can go wrong.
I can think of an very wasteful hack here:
Loop through all the subjects, run the BIDS-app at participant level, pass the subject id to --participant-label
This way you will get a one-line tsv for each subjects and figure out who is the imposter...

I'll give it a go, thank you!!

Plot twist, I get the same error with ds00030 and the group flag... will see if the hack works with srpbs and then maybe try that too.

Thanks! The most recent one is the one I preprocessed


Added to* methods to readme docs.

Here are the docs to go along with my added methods for string extension.

See pull request #56


Feature/azure

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: ritesh74:x: Soumik patraSoumik patra seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix: Diagnostic module for neovim 0.5.1

If vim.diagnostic yields nil, set diagnostic_highlights as an empty table and
use lsp.diagnostic._get_floating_severity_highlight_name to set
highlight name on nvim 0.5.1.
Refers to issue #14 .

Thanks both 🙇


Avoid using delete operator

The use of the delete operator has performance negative effects for the V8 hidden classes pattern.
In general it's recommended do not use it.
Alternatively, to remove object own enumerable properties, we could create a new object copy without those properties (example using lodash):
_.omit(o, 'prop', 'prop2')

Or even define the property value to null or undefined (which is implicitly ignored when serializing to JSON):
o.prop = undefined

Sounds like a reasonable request. Here's some benchmarks: http://jsperf.com/delete-vs-undefined-vs-null/16
Seems setting to undefined is the most ideal case if its not to problematic to change.

I am not against code optimisations, but I always believed there should be a balance between code readability and clarity and actual performance gains when deciding whether or not an optimisation should be done to code.
In this particular case, we call delete about 5 times (did not count properly) before each API request which usually takes about 200-600ms to complete. I am skeptic on the performance boost gained by ditching delete in favour of setting a property to undefined. In practice, we are probably talking about gains of 1ms/request at most.:)
That being said, lib/apirequest.js sure could use some rewrite! (that's where most delete calls are)

Yep you make a fair point which is why I never second guessed using delete in the first place. That being said, if using it is relatively easy, its always better to do that. Readability of delete options.x vs. options.x = undefined is equal in my opinion.

@Alaneor I agree with you about this if you're thinking about premature optimizations, which in fact could be clearly considered as an anti-pattern, but this case is a bit particular and understood. Let me explain that.
The issue with delete operator, which is specifically applicable (but not limited) to the V8 engine, removes all the benefits of the hidden classes strategy, which is one of the powerful features that makes V8 an efficient JIT compiler. Taking into account node.js/io.js runtime is based in V8, this particular issue should be considered.
From the other hand, the cons of the delete operator is not related of the current code operator, it has other performance issues in the program live-cycle for those objects which remains in the memory stack/head and are not flushed by the garbage collector. The jsPerf test suite just recreates an atomic performance test with no program live-cycle.
Taking into account this package is commonly used in production by thousands of developers and companies, this kind of minor code optimizations is highly appreciated, and just because it cames from Google, and in general the people has a blind faith and expect the best quality from they (this is not a criticism, in fact it's the opposite. People love Google and it's an evidence).
Regarding to the code readability, in this particular case I think doesn't makes any sense, due to the syntax noise in probably the same, and the expressiveness is actually the same. In any case you can always create a helper function to provide more grammatical expressiveness.
And in general I agree about following the YAGNI pragmatic thinking by default.
I've created the issue just after reviewing the code in a couple of modules, and noticing a bit of abuse of this operator. In general, most of the JavaScript hackers rarely used this operator, specially if his code is part of a package which is widely used by the community.

Here are some articles that notice the same performance issue with a better technical explanation and endorsement:
http://www.html5rocks.com/en/tutorials/speed/v8/
http://www.future-processing.pl/blog/javascript-is-slow/
https://github.com/skeeto/resurrect-js#options

@h2non We always welcome pull requests, especially for small optimisations and fixes such as this. This would be a good first bug.

Thanks @ryanseys. Sure! probably I'll send a PR with some improves when I've free time

@h2non @ryanseys @jmdobry I would like to work on this and send a PR.

Before doing any sort of optimizations, we should profile the API and find out where the big perf hits exist.  Blindly making a change like this could help, it could make things slower.  Until we do an analysis and put a measurement framework in place, its impossible to know.
I'm going to close this out for now.

@brandondees @tmornini you guys are great at reminding me about premature optimizations. @JustinBeckwith raises a great point. You don't know what you don't know. 👍

@JustinBeckwith @snuggs If you can't measure it, you shouldn't optimize it.
So, if you really want to optimize your code, measure everything. 😄

You can also delete with destructuring:
const obj1 = { a: 1, b: 2 }
const { a, ...obj2 } = obj1
console.log(obj2) // { b: 2 }

@justinmchase this is inefficient as it creates an additional object (and potentially wakes up the garbage collector).

@snuggs how so?

@ug02fast delete being slow is the platform's fault. That's a THEM problem.
spread-ing is just that. Creating an extra object for the sake of performance. That's an OUR problem (introduced unnecessarily for the sake of performance).
You use delete your code will get faster over time (and patches). Using spread becomes technical debt over time. More right (now) than "right". Spread has a purpose...Unknown object shape. optimization diverges from that purpose. Actually it's the inverse. You know the shape of obj1, just want it to be "faster". I agree with @tmornini re benchmarking. IMHO the only time we should reinvent the platform is for polyfilling.
Hope this helps.

@ug02fast as an addendum come to think of it there are TWO unnecessary objects. obj2 AND a.
The garbage man cometh for obj1 AND a. If you are familiar with how the GC works not only may it be slower but now your code is nondeterministically slow(er at times you least suspect).


add a parser for no nodes

XmlParser.AstParser.Node has no way of catching a NodeError, which would be one way of creating a parser for empty strings. Suppose you have a parser p :: Content a which accepts the empty string. The library should make AstParser.contentNode p succeed on an element with no children, but currently throws NotAvailableNodeError.
Alternatively, AstParser should provide a top-level function of type  a -> Nodes a which returns the given element precisely when XmlParser.NodeConsumerState.fetchNode returns Nothing. For such a function to be useful, the Element and Node monads should have a binary operator for alternatives. (An Alternative instance is impossible due to a lack of reasonable empty value.)

Fix is released in 0.1.1

Thanks for the detailed report! :)


Getting 400 Bad Request when requesting events using api_key

Following: https://secure.meetup.com/meetup_api/key/
I haven't been able to fetch events from the API. I wonder if it might be due to lack of permissions, because I can get the dashboard endpoint successfully only passing the key:
https://api.meetup.com/dashboard?key=KEY -> OK 200
but once I try to get events with the same key
https://api.meetup.com/2/events?key=KEY -> BadRequest 400
Same when I try to fetch events for a particular meetup.

hi @ygt-igor, the /2/events API requires some parameters in order to know which events it should show you. Have a look at the docs for that method and make sure to provide at least one of the fields highlighted in yellow there.


BUG: scipy root treats x0  differently depending on the method

Describe your issue.
I am not sure if my observation is a bug or feature.  Since it is not documented I assume it is a bug.
It seems that scipy root function treats x0 differently depending on the method used for root finding.  More specifically, when krylov method is used, x0 is treated as a scaler number (for the case of 1 equation and 1 unknown). For every method other than krylov it treats x0 as an array of length 1.  This might result in inconsistency in the function behavior.  For example, the sample code below raises a ValueError when used with any method other than krylov.
Reproducing Code Example
import numpy as np
from scipy.optimize import root
from scipy.integrate import solve_ivp

def fun_d(x, d0):
    return(d0 * x)

def ode_fun(t, z, a, b, c, d0):
    x, y = z
    d = fun_d(x, d0)
    u = a*d*x - b*x*y
    v = -c*y + d*x*y
    return([u, v])

def fun_err(d0, a, b, c):
    sol = solve_ivp(ode_fun, t_span=[0, 15], y0=[10, 5],
                    method='RK45', args=(a, b, c, d0), t_eval=[12, ],
                    rtol=1e-11, atol=1e-13)
    obj_sol = solve_ivp(ode_fun, t_span=[0, 15], y0=[10, 5],
                        method='RK45', args=(1.5, 1, 3, 1), t_eval=[12, ],
                        rtol=1e-11, atol=1e-13)
    obj_x = obj_sol.y[0][-1]
    return(sol.y[0][-1] - obj_x)

a = 1.5
b = 1
c = 3

# no error
d_sol = root(fun_err, x0=0.5, args=(a, b, c), method='krylov')
# ValueError
d_sol = root(fun_err, x0=0.5, args=(a, b, c), method='hybr')

Error message
Depending on the method it can raise ValueError: could not broadcast input array from shape (2,1) into shape (2,).

SciPy/NumPy/Python version information
1.8.1 1.23.1 sys.version_info(major=3, minor=9, micro=7, releaselevel='final', serial=0)

I tested your code with all available root finding methods.
Output:
ValueError for method: hybr
ValueError for method: lm

import numpy as np
from scipy.optimize import root
from scipy.integrate import solve_ivp

def fun_d(x, d0):
    return(d0 * x)

def ode_fun(t, z, a, b, c, d0):
    x, y = z
    d = fun_d(x, d0)
    u = a*d*x - b*x*y
    v = -c*y + d*x*y
    return([u, v])

def fun_err(d0, a, b, c):
    sol = solve_ivp(ode_fun, t_span=[0, 15], y0=[10, 5],
                    method='RK45', args=(a, b, c, d0), t_eval=[12, ],
                    rtol=1e-11, atol=1e-13)
    obj_sol = solve_ivp(ode_fun, t_span=[0, 15], y0=[10, 5],
                        method='RK45', args=(1.5, 1, 3, 1), t_eval=[12, ],
                        rtol=1e-11, atol=1e-13)
    obj_x = obj_sol.y[0][-1]
    return sol.y[0][-1] - obj_x

a = 1.5
b = 1
c = 3

for method in ["hybr", "lm", "broyden1", "broyden2", "anderson", "linearmixing", "diagbroyden", "excitingmixing", "krylov", "df-sane"]:
    try:
        root(fun_err, x0=0.5, args=(a, b, c), method=method)
    except ValueError as error:
        print(f"ValueError for method: {method}")

Looks like the Minpack based solvers fail for certain pathological problems, unlike the other solvers, so I considers this a bug.
Regarding your code: with the given parameters, fun_err basically returns 0 in the first iteration. Was that intended/for testing purposes?

To simplify a bit, just to show that the methods are inconsistent when x0 is a scalar:

from scipy import  optimize

def f(x):
    if f.count == 0:
        print(x.shape)
    f.count += 1
    return 0

methods = ["hybr", "lm", "broyden1", "broyden2", "anderson", "linearmixing",
           "diagbroyden", "excitingmixing", "krylov", "df-sane"]

for method in methods:
    print(f"{method} x.shape: ", end="")
    f.count = 0
    optimize.root(f, x0=1, method=method)

hybr x.shape: (1,)
lm x.shape: (1,)
broyden1 x.shape: ()
broyden2 x.shape: ()
anderson x.shape: ()
linearmixing x.shape: ()
diagbroyden x.shape: ()
excitingmixing x.shape: ()
krylov x.shape: ()
df-sane x.shape: ()

I usually prefer to fix things like this, but I think it would be pretty disruptive. Since there are easy workarounds (i.e. use root_scalar for scalar problems, ensure that x is passed as an array) I'm not sure there would be much support.


AttributeError 'Options' object has no attribute 'get_ordered_objects' in Django 1.6

Hi, I'm using version 0.8 of django-authority and I'm following the documentation to create a per-object permission in Django 1.6. When in the admin I select an object from my class and try to 'Edit permissions for selected 'MyObject'", I get the following error:
AttributeError at /admin/activity/myobject/
'Options' object has no attribute 'get_ordered_objects'
Request Method: 	POST
Request URL: 	http://example.com/admin/activity/myobject/
Django Version: 	1.6.2
Exception Type: 	AttributeError
Exception Value:
'Options' object has no attribute 'get_ordered_objects'
Exception Location: 	.virtualenv/lib/python2.7/site-packages/authority/admin.py in edit_permissions, line 84
Python Executable: 	/usr/bin/python
Python Version: 	2.7.6
I found a ticket about the removal of 'get_ordered_objects' [1] and I can't find it in the source code for 1.6, but I don't know enough of what this function is used for, to comment it or replace it.
[1] https://code.djangoproject.com/ticket/19469

@bocribbz can you take a look at this while we are getting django 1.8 support?


Programatic API

There was previously a lintText() method that would allow you to require('standard') and lint a string but it appears this functionality has been removed, was there a reason for this?

lintText() is available on the speed branch. You can see it here: https://github.com/feross/standard/blob/speed/index.js#L30-L39
We're waiting for https://github.com/eslint/eslint/pull/2115 to get merged and released before we can release this branch as the next version of standard.

I'm going to close this, as there's already an issue for adding a programmatic API here: https://github.com/feross/standard/issues/60
:)


permissions-appropriate help

running the help command should give information on only the commands that the runner can use. for non-command plugins, they can be accessed with the special * permission

resolved by https://github.com/TrashMonks/Ereshkigal/commit/d2cb0bfea0da1739d9a77a839e14f4256a802e66


Array flatten

Fixes issue #254

Curious if this happens also with other Array methods...

My bet is yes :-(
The problem is also compounded because opal doesn't have full support yet
either.
On Jul 27, 2015 03:23, "Denny Mueller"<EMAIL_ADDRESS>wrote:

Curious if this happens also with other Array methods...
—
Reply to this email directly or view it on GitHub
https://github.com/voltrb/volt/pull/256#issuecomment-125123095.