[FEATURE] - Invoke-PnPSiteTemplate - Errors should include some contextual info

If a Site Template (PNP XML) fails to deploy, it often simply yields the error from the server.
Describe the solution you'd like
It would be great if the error output showed which Handler, and which element it was processing when an error was thrown. This would save time troubleshooting issues with templates

Do you use Set-PnPTraceLog -On -Level Debug ?
This might give you some of the information you need.

@jimmywim - did you try the above suggestion by @BaronSparky ? It is the standard way to check errors in each handler

Closing this due to lack of response.


Add Terminology Service docs

One more comment - @mattwiller can you add a link from the CodeableConcepts section here to the new index page


added closing bracket

Description

Agriseafood form, step 2 question 2: missing closing bracket

Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist

[x] I have read the CONTRIBUTING doc
[ ] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

SonarQube Analysis: https://sonarqube-vmvfjv-tools.pathfinder.gov.bc.ca/dashboard?id=common-forms-toolkit-pr-119


Table definition query altenative non-functional

Hi there,
in regards to getting table and column metadata information, the provided example query alternative to the stored procedure use-case was not working for me, here is an adjusted version which did work again:
SELECT s.name as schema_name, t.name as table_name, c.* FROM sys.columns AS c
INNER JOIN sys.tables AS t ON t.object_id = c.object_id
INNER JOIN sys.schemas AS s ON s.schema_id = t.schema_id
WHERE t.object_id = object_id('dbo.mytable') AND s.name = 'dbo';

Be aware that the schema name is used again within the object_id function call.
Hope this helps!
Cheers, Alex.

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

ID: ed157539-52dc-0dab-22ca-964143d7095d
Version Independent ID: f8c8107c-8662-95e2-650b-1a1692a44db4
Content: View the Table Definition - SQL Server
Content Source: docs/relational-databases/tables/view-the-table-definition.md
Product: sql
Technology: table-view-index
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

@akisys -- Alexander, thank you for your feedback.  I am updating this article.

@akisys Thanks for catching. The issue is with the object_id function not picking up other schemas. I will simplify the example. Appreciate your time to create this issue. Thanks again for this feedback, your suggestions help improve our documentation.
@WilliamAntonRohm I will handle, will refresh article as well.


Add security scanner config

Description
Changes proposed in this pull request:

Add config for security scanner

Related issue(s)

https://github.com/kyma-project/istio/issues/186

/test pull-istio-lint

/retest


array.compute_chunk_sizes() returns chunks with numpy integers

I am not sure if this is a bug. If you run this code:

import numpy
import dask.array

random_array = dask.array.from_array(numpy.random.randint(1, 10, size=1000000))
random_array = random_array[random_array > 5]
random_array.compute_chunk_sizes()
type(random_array.chunks[0][0])

The generated chunks data type is numpy.int64. I don't know if this is intentional but it breaks dask.array_to_zarr because zarr tries to encode to json  and  it is not able to deal with numpy data types.
There is an issue in zarr with code to reproduce the error:
https://github.com/zarr-developers/zarr-python/issues/569

Dask version: 2.18.1
Python version: 3.8
Operating System: Linux
Install method (conda, pip, source): pip

Thanks for the report. Probably not intentional. We can iterate over the chunk__shape[s] and get the .item() in https://github.com/dask/dask/blob/e7e3b735d4cfee04d44cf7054b122bec1569dd23/dask/array/core.py#L1153 I think.
Any interest in working on this @pziarsolo?

Thanks for the report. Probably not intentional. We can iterate over the chunk__shape[s] and get the .item() in
https://github.com/dask/dask/blob/e7e3b735d4cfee04d44cf7054b122bec1569dd23/dask/array/core.py#L1153
I think.
Any interest in working on this @pziarsolo?

I have been looking at the code and I don't feel able to do it.:(

Thanks for taking a look. @gforsyth would you be able to?

I'll take a look, @TomAugspurger -- thanks for the ping.


Feature request: Add ability to reverse/sort the order of years in DatePicker component.

Is your feature request related to a problem? Please describe.
There is no option to sort the years ascending or descending by choice.
only order is ascending.
this would be very helpful when asking the user to enter dates related to his past details,
right now they need to scroll up.
Describe the solution you'd like
Ability to send a boolean prop like yearsDescend or reverseYears to the component
that would have the ability to control the desired order of years.
Additional context
So instead of the years looking like this:

they will be sorted like this:
2019
2018
2017
...

Upvote +1

https://codesandbox.io/s/momentutils-year-range-13emx

hey @dmtrKovalenko cool, but doesn't that mean that it will always be reversed ?
what if i want to have 2 pickers with different order?

You can have several providers. I will think about moving it to the core via props.
But it will need some time

Agreed, makes sense. +1 for this feature

Makes sense, but i saw at several places that it's a common thing to show dates only by descending, it means to scroll down for past dates and up for future ones.
So do we really need both options?, shouldn't we go for more unified ux?

@XiRoSe who knows...

@dmtrKovalenko thanks for your response,  but idk, i think you add a feature you really need not to have the option, because if you'll need it after you can always still add it..

@dmtrKovalenko so is this something you're interested in supporting?

I don’t think it is a rare case. If somebody needs it — please use this solution

Published better example in documentation for overriding date logic https://material-ui-pickers-5v8lfdtv4.now.sh/guides/date-io-customization#override-date-logic


Remove binding redirects for missing assemblies

/cc @alexgav
FYI only to resolve a build break. This worked for me locally because I didn't do a clean build before pushing before.   This worked for me locally because I didn't do a clean build before pushing before. Sorry 😢
You can view, comment on, or merge this pull request online at:
https://github.com/aspnet/Blazor/pull/181
Commit Summary
Remove binding redirects for missing assemblies
File Changes
M tooling/Microsoft.VisualStudio.BlazorExtension/Properties/AssemblyInfo.cs (14)
Patch Links:
https://github.com/aspnet/Blazor/pull/181.patch
https://github.com/aspnet/Blazor/pull/181.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Fix/Upgrade dashboard example to Bokeh 0.12.1

Even though the NYC taxi example is usable, the hover and legend are still not working.

Bit confused why this was bumped for so long, the old Bokeh based dashboard was replaced with a Panel version which does support more recent versions of Bokeh.

That's odd; #676 says that it closes this issue, but I guess Github's robots didn't parse that. Yes, it should be closed, thanks.


Allow nested attributes as array

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Allow both syntax for nesteds FormModel. Previous attribute.nested = value and as simple array (for example from $_GET/$_POST)
[
    'attribute' => [
        'nested' => 'value',
    ],
]

Looks good. Do we still need to support notation with .?

Good day. I use this trick in my project just two days (for generate standart querystrings in js components). I think it can be saved for BC for first time, but also need changes in HtmlForm helper for generate input names wo dots. Anyway - now i have two cases, on plain html using . from helper, and generated from js as standart arrays

But original HTML Forms send nested attributes with []. So user can use brackets notation instead of dot.

But original HTML Forms send nested attributes with []. So user can use brackets notation instead of dot.

Of course and works with standart arrays more convenient in my opinion. But in current time my most difficult form has max 4th nested levels and input name generated something like Search[addreesses.k_numbers.fiases.ids] etc. It only way to generate it using HtmlForm helper and it's not was big problem before i need generate same querystring using js/vue.

But what's the difference between Search[addreesses.k_numbers.fiases.ids] and Search[addreesses[k_numbers[fiases[ids]]]] if it is generated with a class-helper and is parsed by form?

But what's the difference between Search[addreesses.k_numbers.fiases.ids] and Search[addreesses[k_numbers[fiases[ids]]]] if it is generated with a class-helper and is parsed by form?

Roight format that can be using native php without any parsers must be Search[addresses][k_numbers][fiases][ids]. That PR can be first step to do it

Forms must support the Nested::class Validator syntax, support .dot format.


Links don't work

OpenControl.org links to https://compliance.cloud.gov which has bad links.
https://compliance.cloud.gov/standards/NIST-800-53-AC-2 1.md returns a 404.

Hi @justinjsmith! Thanks for the report, good catch. I'm going to file this as an issue in https://github.com/18F/cg-compliance/issues (and close this one) since this is a problem with that site instead of a problem with the OpenControl site. I work on the cloud.gov team, and we're currently not maintaining/updating the compliance.cloud.gov site, but we hope to get back to it in the future.


SyntaxError: Identifier 'builders' has already been declared (9:3)

Describe the bug
I'm trying to generate code from my ts files with graphql queries, but I get this error:
SyntaxError: Identifier 'builders' has already been declared (9:3)
        at Object._raise (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:776:17)
        at Object.raiseWithData (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:769:17)
        at Object.raise (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:737:17)
        at TypeScriptScopeHandler.checkRedeclarationInScope (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:1392:
12)
        at TypeScriptScopeHandler.declareName (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:1358:12)
        at TypeScriptScopeHandler.declareName (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:5081:11)
        at Object.checkLVal (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:9742:24)
        at Object.checkLVal (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:7298:15)
        at Object.parseVarId (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:12531:10)
        at Object.parseVarId (/usr/src/app/client/node_modules/@graphql-tools/graphql-tag-pluck/node_modules/@babel/parser/lib/index.js:7126:11)

To Reproduce
Steps to reproduce the behavior:
I've tried to reproduce the error, but I got a new one. in codesandbox I get
TypeError: Cannot read property 'buildError' of undefined

https://codesandbox.io/s/zen-platform-dowef?file=/schema.gql

My GraphQL operations:

import { gql } from '@apollo/client';

export const FULL_TASK = gql`
  fragment FullTask on Task {
    id
    title
    isCompleted
    isStarred
    atMyDay
    startDate
    startDateTime
    endDate
    endDateTime
    rrule
    missing
    recurringTaskId
    project {
      id
      title
      icon
      color
      backgroundColor
    }
  }
`;

import { gql } from '@apollo/client';

export const DELETE_TASK_MUTATION = gql`
  mutation deleteTask($taskId: String!) {
    deleteTask(taskId: $taskId) {
      ...FullTask
    }
  }
`;

My codegen.yml config file:

overwrite: true
schema: "./schema.gql"
documents:
  - "./**/*.gql"
  - "./**/*.ts"
generates:
  generated.tsx:
    schema: "http://server:3000/graphql"
    plugins:
      - "typescript"
      - "typescript-operations"
      - "typescript-react-apollo"

Expected behavior
It should generate code without any errors
Environment:
"@apollo/client": "3.3.11",
"@graphql-codegen/cli": "1.21.4",
"@graphql-codegen/introspection": "1.18.1",
"@graphql-codegen/typescript": "1.21.0",
"@graphql-codegen/typescript-operations": "1.17.14",
"@graphql-codegen/typescript-react-apollo": "2.2.1",

Have the same error:
package.json:
    "@graphql-codegen/cli": "^1.21.7",
    "@graphql-codegen/fragment-matcher": "^2.0.1",
    "@graphql-codegen/introspection": "^1.18.2",
    "@graphql-codegen/typescript": "^1.23.0",
    "@graphql-codegen/typescript-document-nodes": "^1.17.16",
    "@graphql-codegen/typescript-graphql-files-modules": "^1.18.1",
    "@graphql-codegen/typescript-operations": "^1.18.4",
    "@graphql-codegen/typescript-react-apollo": "^2.3.1",
    "@graphql-codegen/urql-introspection": "^1.3.1",

codegen.yml:
overwrite: true
schema: 'http://p8.local/escheriql'
documents: './**/*.{tsx,ts}'
generates:
<EMAIL_ADDRESS>    plugins:
      - 'typescript'
      - 'typescript-operations'
      - 'typescript-react-apollo'
      - 'typescript-graphql-files-modules'
      - 'typescript-document-nodes'
      - 'fragment-matcher'
      - 'urql-introspection'
    config:
      noExport: true

query example:
export const QUERY= gql`
  query GetStarBySlug($slug: String!) {
    allArtists(slug: $slug, status: "public") {
      edges {
        node {
          id
        }
      }
    }
  }
`

@eden-lane Just ran into a similar issue with leftover/untracked declaration files conflicting with type gen. Try using the DEBUG mode to see where exactly the error is coming from. https://www.graphql-code-generator.com/docs/getting-started/codegen-config#debug-mode

I think @apollonian is right, there might be a file that's being loaded by graphql-tag-pluck that causes this error. Please try to use narrow glob patterns, and see if the issue still happens.
@ardatan Do you think we can improve the errors on graphql-tag-pluck? (Maybe just print a warning to the log with the invalid file path, and ignore that file?)

(Maybe just print a warning to the log with the invalid file path, and ignore that file?)

💯 This would be wonderful!

Unfortunately I moved to far away from the point where I had this issue (I had stopped generating code from .ts files), so I'm not sure if I am able to reproduce it and try the solution you suggested. As far as I remember I had only two ts files and they are listed in the opening post. Without
  - "./**/*.ts"

in "documents" section there were no issues.

experienced a similar issue after bumping dependencies up to the latest:
documents: 'src/js/**/!(*.d).{ts,tsx}'

found through verbose mode that it was loading a .d.ts types file, so this seems like a globbing issue potentially?
"@graphql-codegen/cli": "1.21.8",
"@graphql-codegen/fragment-matcher": "^3.2.0",
"@graphql-codegen/typescript": "1.23.0",
"@graphql-codegen/typescript-operations": "1.18.4",
"@graphql-codegen/typescript-react-apollo": "2.3.1",

if this is still an issue on v2 or above please file a new issue.


feat: support wasi by tweaking cfg attributes

Adds support for wasi. Previously the crate could be compiled to wasi, but still relied on wasm-bindgen to check for the current node environment so it could not be used outside of a JS runtime.
I am making this PR to try to get swc, which depends on this crate, to run to wasi but this is a blocking issue. I might not be considering some use case but I'm willing to make any requested changes :)

Does it block SWC? If so, I will release a new version tomorrow.

Does it block SWC? If so, I will release a new version tomorrow.

It does, if you could release a new version that'd be awesome!


Cannot install pythonPackages.credstash from nixos-unstable

Issue description
Error: Could not find a version that satisfies the requirement cryptography<2.1,>=1.5 (from credstash==1.14.0)
Steps to reproduce
nix-env -iA nixos.pythonPackages.credstash
Technical details

system: "x86_64-linux"
host os: Linux 4.9.81, NixOS, 18.03pre127921.9b25b9347d9 (Impala)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 1.11.16
channels(root): "nixos-18.03pre130719.a66ce38acea"
channels(lbodor): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Build works on 21.11 and most likely unstable too.


Fixes unsubscribe problem

This pull request fixes the problem when unsubscribing to some topic in wamp.rt .

Btw, are you still working on this project? Thanks

Thanks for the fix.
We use that project every day but there has been no evolutions recently.
Our current needs are covered for now, but that may change as enclosing project is far from finished.


Audit fields in use for Score L

We get some data from multiple sources, such as linguistic isolation from both ACS and EJScreen. We should make sure we're using data directly from ACS in definition L, and we should make sure we're using all the correct fields for the whole definition more generally.

Double checked that we're using ACS fields for demographic data.


[] build failure

Host Environment

OS: [Arch Linux]
Compiler: revision

To Reproduce
Steps to reproduce the behavior:
vcpkg install "opencv" --triplet=arm64-android
Failure logs
-Cut and past the appropriate build messages from the console output.
-Please attach any additional failure logs mentioned in the console output.
Computing installation plan...
The following packages will be built and installed:
  * libwebp[core,nearlossless,simd]:arm64-android -> 1.1.0#4
    opencv[core,dnn,jpeg,png,quirc,tiff,webp]:arm64-android -> 4.5.3
  * opencv4[core,dnn,jpeg,png,quirc,tiff,webp]:arm64-android -> 4.5.3#1
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet arm64-android...
Restored 2 packages from /home/john/.cache/vcpkg/archives in 2.774 s. Use --debug to see more details.
Starting package 1/3: libwebp:arm64-android
Installing package libwebp[core,nearlossless,simd]:arm64-android...
Elapsed time for package libwebp:arm64-android: 6.604 ms
Starting package 2/3: opencv4:arm64-android
Installing package opencv4[core,dnn,jpeg,png,quirc,tiff,webp]:arm64-android...
The following files are already installed in /opt/vcpkg/installed/arm64-android and are in conflict with opencv4:arm64-android

Installed by libwebp:arm64-android
    debug/lib/libcpufeaturesd.a
    lib/libcpufeatures.a

Please ensure you're using the latest portfiles with `./vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: opencv4:arm64-android
  Vcpkg version: 2021-09-10-unknownhash

Additionally, attach any relevant sections from the log files above.

Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

This issue hasn’t been updated in 3 month, if it is still an issue, please reopen this issue.


No longer relying on external translation id.

We used to rely on the external id for sorting the translations correctly, which is not necessary and can cause some issues. With this change we still carry through the externally provided id to the user, but no longer rely on it internally.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Missing:

tests
mypy

I am running into an issue with this fix when using --batch-size > 1 and --max-seq-len:
> head -50 input | python -m sockeye.translate -m model --use-cpu --batch-size 2 --max-input-len 10

[INFO:sockeye.inference] Translator (1 model(s) beam_size=5 beam_prune=off beam_search_stop=all ensemble_mode=None batch_size=2 buckets_source=[10] avoiding=0)
[INFO:__main__] Translating...
[ERROR:__main__] Uncaught exception
Traceback (most recent call last):
  File "/Users/fhieber/miniconda3/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/Users/fhieber/miniconda3/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/Users/fhieber/workspace/nmt/src/sockeye-github/sockeye/translate.py", line 221, in <module>
    main()
  File "/Users/fhieber/workspace/nmt/src/sockeye-github/sockeye/translate.py", line 40, in main
    run_translate(args)
  File "/Users/fhieber/workspace/nmt/src/sockeye-github/sockeye/translate.py", line 111, in run_translate
    input_is_json=args.json_input)
  File "/Users/fhieber/workspace/nmt/src/sockeye-github/sockeye/translate.py", line 188, in read_and_translate
    chunk_time = translate(output_handler, chunk, translator)
  File "/Users/fhieber/workspace/nmt/src/sockeye-github/sockeye/translate.py", line 212, in translate
    trans_outputs = translator.translate(trans_inputs)
  File "/Users/fhieber/workspace/nmt/src/sockeye-github/sockeye/inference.py", line 1174, in translate
    batch_translations = self._translate_nd(*self._get_inference_input(translator_inputs))
  File "/Users/fhieber/workspace/nmt/src/sockeye-github/sockeye/inference.py", line 1221, in _get_inference_input
    source = mx.nd.zeros((len(trans_inputs), bucket_key, self.num_source_factors), ctx=self.context)
  File "/Users/fhieber/miniconda3/lib/python3.6/site-packages/mxnet/ndarray/utils.py", line 67, in zeros
    return _zeros_ndarray(shape, ctx, dtype, **kwargs)
  File "/Users/fhieber/miniconda3/lib/python3.6/site-packages/mxnet/ndarray/ndarray.py", line 3750, in zeros
    return _internal._zeros(shape=shape, ctx=ctx, dtype=dtype, **kwargs)
  File "<string>", line 34, in _zeros
  File "/Users/fhieber/miniconda3/lib/python3.6/site-packages/mxnet/_ctypes/ndarray.py", line 92, in _imperative_invoke
    ctypes.byref(out_stypes)))
  File "/Users/fhieber/miniconda3/lib/python3.6/site-packages/mxnet/base.py", line 255, in check_call
    raise MXNetError(py_str(_LIB.MXGetLastError()))
mxnet.base.MXNetError: Invalid Parameter format for shape expect Shape(tuple) but value='(2, None, 1)', in operator _zeros(name="", dtype="float32", ctx="cpu(0)", shape="(2, None, 1)")

Seems like bucket key is `None` causing `zeros` to fail.
`bucket_key` is assigned with this statement:

bucket_key = data_io.get_bucket(max(len(inp.tokens) for inp in trans_inputs), self.buckets_source)

After some quick debugging it turns out that the first `trans_input` has 11 input tokens (the others 10). With `--max-input-len 10` this yields `None` as `bucket_key`.

This doesn't happen on current master.

I may have found the problem: my toy model does not use source_with_eos yet so it goes into the else block when constructing TranslatorInputs.
[...]
                else:
                    if len(trans_input.tokens) > self.max_input_length:
                        # oversized input
                        logger.debug(
                            "Input %d has length (%d) that exceeds max input length (%d). "
                            "Splitting into chunks of size %d.",
                            trans_input.sentence_id, len(trans_input.tokens),
                            self.buckets_source[-1], self.max_input_length)
                        chunks = [trans_input_chunk
                                  for trans_input_chunk in
                                  trans_input.chunks(self.max_input_length)]
                        input_chunks.extend([IndexedTranslatorInput(trans_input_idx, chunk_idx, chunk_input)
                                             for chunk_idx, chunk_input in enumerate(chunks)])
                    else:
                        # regular input
                        input_chunks.append(IndexedTranslatorInput(trans_input_idx,
                                                                   chunk_idx=0,
                                                                   translator_input=trans_input.with_eos()))

When removing .with_eos() from the last line it works. Seems like a copy and paste error.
I will push this update to the branch.

ohh, good catch! was indeed a copy/paste error!


Docs: Storing model instances in components

I'm trying to find out how to fit vue-mc into my app's architecture, and the docs are helpful, but maybe I am missing something obvious:
In the Vue docs data section, it reads:

A rule of thumb is that data should just be data - it is not recommended to observe objects with their own stateful behavior.

Where should I store my model instance otherwise? That's the "vue-ful" way of doing so?
I'd be happy to add the results of this discussion to the vue-mc docs if the maintainers see that it's appropriate.

I think the vue-ful way is to create components granularly and have your own HTTP modules, etc. A vue app done right probably doesn't need a package like vue-mc. In practice, we found that that is harder to achieve than we anticipated, so vue-mc helped to abstract a lot of duplicated patterns. The idea was that vue-mc is just a proxy to "just data", ie. it does what a component would do in its methods, except that the logic is moved to another module that can be isolated from Vue (for the most part), shared, and tested. We didn't want vue-mc to maintain "state" and observers other than those automatically added by Vue. It just encapsulates the data for components. I don't believe that this fundamentally defies "the vue way", but also perhaps not a perfect fit.


Question: Sassdocify + themes

Is it possible to use sassdocify + a custom theme and NOT have those theme files live in my main branch?

I guess if it was in node_modules it would work...

I see two solutions:

Put that custom theme folder in a .gitignore if it's located under your git project.
Work that theme in a completely different folder structure and use npm link.

thanks @pascalduez!
in the npm link option it would still be in same repo or no?

npm link will create an symlink globally, so you can have your module anywhere.

aight thanks again @pascalduez. looks like separate repo is the route to go


OpenGL version and error checking

Not much error checking at the moment, also don't really know what GL version we should be using. Currently set to 3.1.
While this works on Win7, on Ubuntu on the same hardware "invalid operation" is spammed each frame and only the clear colour is rendered.

crafn has added a macro for pre/post checking GL API's in debug builds, issue was actually NanoVG - changed its init to use GL2 instead of GLES2.


Update repo description to "(Karma + Sassy) * (Facebook + Slack + Skype + Hipchat + Telegram) = Kassy"

Repo description (high on the page, seen first) doesn't reflect the current platform support. Should be

(Karma + Sassy) * (Facebook + Slack + Skype + Hipchat + Telegram) = Kassy

We probably just need to remove this entirely (or make it more consise) because Hubot adapters are semi-supported out of the box currently too, plus we will need to add Messenger, and MS Bots (soon anyway).

We probably just need to remove this entirely (or make it more consise) because Hubot adapters are semi-supported out of the box currently too, plus we will need to add Messenger, and MS Bots (soon anyway).

Good point. Perhaps a new tagline entirely then, yes?

Can we make it more user friendly while we are at it, so any non-IT person can understand it. It'd also be great if we can provide some kind of installer mechanism or something to broaden the target audience

Can we make it more user friendly while we are at it, so any non-IT person can understand it.

Though I'm in favour of something "user friendly," I'm not sure non-IT people are the audience?

It'd also be great if we can provide some kind of installer mechanism or something to broaden the target audience

Broaden it to? I'm curious who you think might use something like this beyond fairly technical people?
Phillip.

A lot of my friends are interested in having their own bot but don't want to learn programming. If Kassy can be packaged into an installer and provide an interface to install modules, it'll solve their problems.
Of Course everyone has different expectations from the modules and Kassy itself but everyone wants their own personal-bot

A lot of my friends are interested in having their own bot but don't want to learn programming.

Sure, I get that, and yet I don't think these people are going to be able to figure out Kassy very easily, i.e., where are they going to host it? How are they going to configure it?

If Kassy can be packaged into an installer and provide an interface to install modules, it'll solve their problems.

But maybe those are Kassy's problems?

Of Course everyone has different expectations from the modules and Kassy itself but everyone wants their own personal-bot

What about something like https://flowxo.com/ that is purpose-built for non-technical people?

It'd also be great if we can provide some kind of installer mechanism or something to broaden the target audience

An installer would be nice, its something that could be generated as a build artifact by Travis on a master build. Remember that an installer would also need to install:

A supported version of Node.js
NPM if it wasn't packaged with Node.js
Git for updates and module installs

Unfortunately I don't really have time amoungst the other improvements to Kassy and Life™ to investigate this currently.

A lot of my friends are interested in having their own bot but don't want to learn programming

As in they want to host their own bot and install someone elses modules into it but do not want to create their own? I'm currently trying to patch Kassy to support an easy deploy to heuroku so that all that is needed is a series of copied and pasted commands to host your bot.
@phillipadsmith is right in saying that Kassy is reasonably easy to work out and that some technical knowledge will still probably be required because it needs to be hosted somewhere.

... provide an interface to install modules

Of course it would always be possible to create a module for Kassy (currently a core one, soon not to be) which could manage Kassy itself. That way Kassy itself doesn't change but it is possible to install modules through some kind of GUI (I would suggest an express.js based GUI). This module could come pre-installed as a part of the installer mentioned.
All this being said, I wouldn't have said /kpm install foo was that hard even for the most technophobic people.

because Hubot adapters are semi-supported out of the box currently too

Somehow I missed this. This is crazy-awesome. Can you say more?

@phillipadsmith I will use the discord one as an example. In the current (non PR) version of Kassy you can do this:
cd core/core_integrations
git clone https://github.com/thetimpanist/hubot-discord.git
cd ../../
echo > config.json << EOF
{
    "output": {
        "hubot-discord": {
            "ENV_HUBOT_DISCORD_EMAIL":" <email/username>",
            "ENV_HUBOT_DISCORD_PASSWORD":"<secret password>",
            "ENV_HUBOT_DISCORD_TOKEN":"<oauth token>"
        }
    }
}
EOF

node main.js hubot-discord

This feature is currently not documented but is intended to be a stop-gap measure until a real integration can be created. Not all hubot adapters work, some only work on their second startup (no idea why) and some features of Kassy do not work properly (primarily shutdown and startup). We also support a good percentage of Hubot scripts out of the box too in case you wern't aware (also not documented).
Maybe in the distant future we might do something similar with botkit and some others as well given the required loading infrustructure is in place.

We also support a good percentage of Hubot scripts out of the box too in case you wern't aware (also not documented).

Yes, I read that in an issue recently and my curiosity was piqued.  :-)

@mrkno Not sure what the purpose of this issue is now?


A better error for writing functions with upper-case first letter

This simple program
A : Float
A = 10

gives this error:
I ran into something unexpected when parsing your code!

1| A : Float
     ^
I am looking for one of the following things:

    an equals sign '='
    whitespace

I guess it should say something as:
1| A : Float
     ^
Names of functions/values cannot start with an upper-case letter.

there's more to this. The error @martincerny  gets only happens when there  is only one type constructor and the type signature is given.
Here are some more weird cases

Thank you for the report! I am finishing up an overhaul of the parser that will produce nicer error messages. I made a tweak based on your example, so my development build is now producing the following message:
-- SYNTAX PROBLEM ----------------------------------------------------- temp.elm

I was expecting to see a declaration, but I am getting stuck here:

1| A : Float
   ^
All declarations start with lower-case letters in Elm, so maybe this letter can
be changed to lower-case?

Hint: Here are a couple valid declarations for reference:

add : Int -> Int -> Int
add x y =
  x + y

type User = Anonymous | LoggedIn String

Notice that these declarations start with lower-case letters. Capitalization
makes a difference in Elm!

This should become available once Elm 0.19.1 is released. Thanks again for the report! The tweak in https://github.com/elm/compiler/commit/ef457e759bb50601fc680eb3f38946fa65dfe3a4 is based on your example!


📎 Implement useReadonlyClassProperties - typescript-eslint/prefer-readonly

Description
Implement typescript-eslint/prefer-readonly.
Want to contribute? Lets you know you are interested! We will assign you to the issue to prevent several people to work on the same issue. Don't worry, we can unassign you later if you are no longer interested in the issue! Read our contributing guide and analyzer contributing guide.
I suggested the name useReadonlyClassProperties because it is in line with some of our existing names such as noUnusedPrivateClassMembers.
See the related discussion for some context.
Options
I don't think we should implement the ESLint's option onlyInlineLambdas because it is very rarely used.
I propose adding an option checkPublic that extends the check on all properties (public and protected properties).

What do you think about checkPublicAccessor?

What do you think about checkPublicAccessor?
This affects also protected. Also, Accessor seems related to the new modifier accessor, I could use accessibility instead (we already use it in the rule name useCOnsistentMemberAccessibility).
Thus: checkPublicLikeAccessibility? Looks over long... or checkAllProperties?


Add unit tests for com.sap.psr.vulas.java.JavaId

Hi,
I've analysed your codebase and have noticed some gaps in the coverage of JavaId
I've written the following tests for this functions with the help of Diffblue Cover.
Hopefully these tests should help you detect regressions caused by future code changes. Please let me know if you'd be interested in having more tests written for other classes within the vulnerability-assessment-tool and I would be happy to help.

Having some kind of Description above about what these tests are doing e.g. adding to your existing coverage on this function (if that is the case) would be helpful to us and to them.

The title of this PR is "Add unit tests for JavaIdTest.java #1"
Normally, we use "Add unit tests for " whereas the above is the test class.

@JohnBergqvist commit message resolved.
I'll add the links to Cover during the real PR.

@JohnBergqvist You saw nothing. It has now been truly resolved.

@paulkennethkent Ready to submit once you're happy with this one.


Toast is shown under the modal screen.

When using presentation: 'modal' on a screen, calling Toast.Show() shows the toast under the screen.
Is there a fix for that?

AlertNotificationRoot must surround your entire project

AlertNotificationRoot must surround your entire project

It does, yet I'm still having the issue

This is weird, maybe you have a problem with zindex or a native modal

Hi @daniel-keen ! Did you find a solution to this issue ?


Web Pack requires.context does not work

@sokra
Hey I am getting this error
Property 'context' does not exist on type 'WebpackRequire'.
when I use require.context('./battle', true, '^(.*.ts)$');
Am I missing any plugins or config in webpack ??

Can you provide a concrete test case?


Is this project still maintained?

There hasn't been any commits in multiple months. Is this the roadmap still correct?

Hi jlocash, yeah we are neglecting gotron right now. I guess we'll keep up the work on gotron somewhere in the first quarter of 2020 cause we need it for a product on our roadmap, But right now we are maintining this only very occasionally. Sorry for that.

Hi. Is there any maintenance for gotron already?
I encounter some gotron-builder issues on production... Thanks


[cmake] NFC: Perform early exit where possible

This reduces the level of nesting and should make the code more
readable.

The diff is quite hard to read, but if this primarily adds
  if(_RULE_TESTONLY AND NOT IREE_BUILD_TESTS)
    return()
  endif()

to add an early exit (NIT: Change PR title from exist to exit) and adjust the indents, I am fine with it :)

Yup. It's just adding the early exit. There is one trick you can use to make code reviews easier: in the File changed panel, click on the gear button and tick "Hide whitespace changes". :)


feat: implement the new landing page - manage wordles (resolves #20)

The related ticket: https://github.com/inclusive-design/inverted-wordles/issues/20

There is a warning that the package.json has no repository field


Add tasks to run main classes in the example

Is this helpful?
I've added while I was playing with the example, maybe it can be helpful for a user or for checking the example during the build.
I don't have strong opinion

Sure, why not. Certainly doesn’t hurt.
But what I really want is for our test suite to also smoke test the example.

I will include this in an other pr


Script uses Chrome and IE to login into Azure

Have tried the automated process. 2 observations.
In step 3 Execute Script, the script does indeed prompt for 2 auths, however, first uses Chrome (works fine) then second time uses IE - which then freezes. On coming out of the execution and re-running the script am stuck in a loop with Chrome to login.

Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to address these items that did not get responses. Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up. Thank you!


Deprecation warning for loaderUtils in dev & production builds

DeprecationWarning: loaderUtils.parseQuery() received a non-string value which can be problematic, see https://github.com/webpack/loader-utils/issues/56
parseQuery() will be replaced with getOptions() in the next major version of loader-utils.

see https://github.com/webpack/loader-utils/issues/56

Hey,
I am looking for beginner issues to take up, Is this issue still available? Let me know in case of any other help.I would love to contribute.

@nisha-kaushik There's currently a PR pending for it.  Usually, if an issue is already assigned to someone, it's already been 'claimed'.  Feel free to take a look at any of the other open issues in the repo, though!

Ok. Thanks :)


Camera Xiaomi presence sensor - chuangmi.camera.046a01

Describe the bug
When I set the: “motion-detection:motion-detection”
It only shows me the sensor at HomeKit, but when there is a motion detected, it doesn't notification at HomeKit,
my question is, how should I set it to receive an alert from the HomeKit as well and not only from the Xiaomi app
It is important for me to point out that the config.json I mentioned is temporary, because I tried almost everything, with true, false, numbers 0,1.. , and also without anything
Expected behavior
when there is a motion detected, receive an notification from the HomeKit
Your config.json
"name": "Home Camera",
"ip": "zzzzzzz",
"token": "xxxxx",
"deviceId": "yyyyyyy",
"model": "chuangmi.camera.046a01",
"pollingInterval": 10,
"deepDebugLog": false,
"customAccessory": false,
"onlyMainService": false,
"buzzerControl": false,
"ledControl": false,
"childLockControl": false,
"modeControl": false,
"propertyMonitor": [
{
"property": "motion-detection:motion-detection",
"name": "Motion Detection",
"value": 1,
"linkedProperty": "motion-detection:motion-detection",
"linkedPropertyValue": true
}
]
},

The property which you are monitoring might not be the actual value of the state but a configuration property, meaning a property which only allows you to turn on or off the Feautrw.

Hey
and you know what is the actual value of the state?
In the manual there is an value “-“

I think that it is an event, but events unfortunately currently does not work


Roll engine 88113925e3a0..2c9e37c34e79 (4 commits)

https://github.com/flutter/engine/compare/88113925e3a0...2c9e37c34e79
git log 88113925e3a063bd6a249346aaa784e65772a25b..2c9e37c34e79475bbde7c8163eb5e56cdb9662a1 --no-merges --oneline
2c9e37c34 Cause crash in FlutterJNI if invoked on non-main thread (#31263). (flutter/engine#8830)
7ce2666e5 Guard Android logs (flutter/engine#8824)
8584da2c2 Roll src/third_party/skia dc19391eef52..70aab823547a (8 commits) (flutter/engine#8836)
1cdc16322 Roll src/third_party/dart 1577b95c93..dbe80f3397 (43 commits)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (amirha@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Prevent packaging of test files

cc @hroncok

Codecov Report

Merging #1087 (8021583) into master (9792513) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1087   +/-   ##
=======================================
  Coverage   95.84%   95.84%
=======================================
  Files          61       61
  Lines        4020     4020
=======================================
  Hits         3853     3853
  Misses        167      167

Continue to review full report at Codecov.

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

This should fix the problem. To verify, I've included this commit in https://github.com/httpie/httpie/pull/1086

Oh, no. It does not include subpackages :(

Yup I just fixed it now :)

I'd use setuptools.find_packages(include=['httpie*']) to avoid a hardcoded list.

Thanks @BoboTiG & @hroncok! 🎉🥧


Introduce incremental update of sorted nodes based on btree for batch allocation to improve scheduling performance

Currently the scheduling process will sort nodes for every single allocation, this leads to serious overhead and can make a great influence for a large cluster, performance effect can be show in follow graph according to the results of scheduling performance:

I would like to propose an efficient way to improve the scheduling performance, which sorts all nodes just once before doing batch allocation and updates the sorted nodes incrementally via adjusting one node changed in the latest allocation and keeping others unchanged. This approach is applicable when we expect that once batch allocation is momentary so that node resource may not change beyond the scheduling process. In this way, scheduling performance can be enhanced at least 900% as shown in next graph.
Updates:

Add NodeSorter interface and two implements based on slice and btree structures.

SliceBasedNodeSorter is the original approach based on slice.
BtreeBasedNodeSorter is based on btree which provides an in-memory B-Tree implementation for Go and holds the Apache license (more details pls refer to https://github.com/google/btree ).

Update the size of batch allocation from 16 to 256 according to the experiences of scheduling performance test, it can make a great influence on the scheduling performance for SliceBasedNodeSorter which can be shown in next graph which describes performance with different batch size: performance is more better as the batch size grow at the beginning and after 256 there's few changes.

UTs

Update sorter_test.go to test both NodeSorter implements.
Update scheduler_perf_test.go to make the results more accurate via enhancing expected allocation number from 10w to 100w for small-scale cluster with 500 or 1000 nodes.
Update scheduler_smoke_test.go to fit for this PR.

Update node_info.go#GetAvailableResource is to make UT correct since we may test a node whose availableResource is nil in sorters_test.go#TestSortNodesMin/TestSortNodesMax

@yangwwei @wilfred-s  could you pls help to review this PR? Thanks.

LGTM, @wilfred-s do you want to take a look before I merge this?

Close this PR since it's not applicable now, we can strive for greater progress by a more efficient way.


Manager : Managing the connections

Things to cover:

Initialize a ConnectionManager.
Call the function to Generate new connection invitation.
Store the state in the base storage

Reference: https://github.com/hyperledger/aries-cloudagent-python/tree/master/aries_cloudagent/messaging/connections

RFC spec: https://github.com/hyperledger/aries-rfcs/tree/master/features/0023-did-exchange

Not required

Not required


CB-12980: (iOS) Improve instruction and add detail steps for plugins in cordova project.

Embed dynamic binaries for Cordova.framework for xcode 8.3.2
Add step to show how to integrate plugins in xcode which used in cordova project.

Platforms affected
Embed cordova in ios native app documentation only.
What does this PR do?
Improve instruction and add detail steps for plugins in cordova project.
What testing has been done on this change?
N/A
Checklist

[x] Reported an issue in the JIRA database
[x] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
[ ] Added automated test coverage as appropriate for this change.

@sudotamm agreed, the upper section can stay. perhaps change this PR to omit the second section, or create a new PR with just the first section.

Ok, I will close this pull request and create a new one for upper section change.


Unified doc's name & update docs

Fix issue #1063
Unified doc's name:

For .md file, use the uppercase for initial letters, no spacers. For example BuiltinTuners.md, BatchTuner.md
For .rst file, use lowercase for all letters and unferline "_" as a spacer. For example index.rst, training_services.rst, sdk_reference.rst
Treat all abbreviate words as one word. For example, PaiMode.md, SquadEvolutionExamples.md

Thanks for unifying the format of files' names.

@leckie-chn @squirrelsc please help review~


Disable pixelservice on demo server

As discussed, disabling pixels on demo, see discussion in https://openmicroscopy.slack.com/archives/C0K5ME1EW/p1668597336544119
cc @sbesson @jburel

Using a different approach in https://github.com/ome/prod-playbooks/pull/365/commits/5d33112db837595496b5d60900d2aa0db58a7ad1 to the disabling of PixelService, in line with the one taken on training servers, see https://github.com/ome/prod-playbooks/blob/d604a0c9db28c83414bd6fb0c38362b9722cf1bf/omero/training-server/playbook.yml#L502


Make Today button work with 'autoOk' flag

(sorry for the same meme on two requests)
Is your feature request related to a problem? Please describe.
I've implemented the <DatePicker /> component and have the autoOk flag turned on. I also have the "Today" button enabled. At present, the user has to hit today, then click on the already highlighted date to get autoOk to close the calendar modal.
Describe the solution you'd like
It would be nice if, when autoOk is enabled, and the user hits "Today", it would have the same functionality as clicking a date with the autoOk flag enabled (i.e. closing the modal and setting the date without having to hit the confirm button).
Describe alternatives you've considered
No alternative I can think of that doesn't involve serious digging and custom method handlers

What version are you using? Did you try the v4 alpha?
In https://github.com/mui-org/material-ui-pickers/issues/1293, I have proposed to make autoOk the default behavior because it seemed to be more common. What do you think?

@oliviertassinari yes, it looks like a nice idea, I missed that while doing API changes

Thank you for taking the time to respond and act on this!
@dmtrKovalenko @oliviertassinari


[CARBONDATA-1118] Inset Filter Pushdown in 2.1

Inset Filter PushDown.

retest this please

retest this please

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-2.1/49/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2155/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/20/

LGTM


Implement support for Secure Boot V2

https://docs.espressif.com/projects/esp-idf/en/latest/esp32/security/secure-boot-v2.html

Hi what exactly here is needed ?

I'm not entirely sure to be honest, I haven't read through the linked documentation yet. I believe it involves signing the image with some cryptographic key, there may be additional steps.

This is a pretty important production feature - whats the status of this issue is in the backlog? Any planned work to implement this?

At this point we do not really have plans to implement any new features in espflash. While I understand this is important functionality, we unfortunately have not received consistent contributions from the community, and development/maintenance of these applications has been put mainly on my and @SergioGasquez's shoulders. I stopped maintaining this project largely for this reason. We both have other commitments at this time which are of higher priority, and espflash provides the required functionality for us to do our work at this time, so it's hard to justify spending any time here.
I would love to see a PR for this, I would be more than happy to review one. Otherwise, esptool.py is still a valid option, it can consume ELF files.

I guess https://github.com/esp-rs/espflash/issues/715 and https://github.com/esp-rs/espflash/issues/713 are, somewhat, related to this issue


Data import CSV file

Hello if someone can help me.
I had installed CartoDB locally on my Ubuntu 14.04 and the installation was perfect. I can create dataset by CartoDB's GUI and able to visulaize data perfectly without any problem. But when i drag and drop a csv or geojson file externally from my system, it is throwing an error as below
Rails:

And my CartoDb's UI is showing
1 An unknown error has happened
Persisting error? Please send us the following code:
01f18226-a811-4992-84b5-8c2143ca4088
if I run the tests in my repertoire cartodb the results of tests gives me the following results

Unrelated to this project


AstraMC projector

Currently, norm for the AstraMC 2D operator is not working
see here.The reason is because in the master the lines are commented out.
Did we decide, how we want to treat this case?

I'm not sure, but I was thinking we could replace the MC versions
completely by having the user use the ChannelwiseOperator with
AstraProjectorSimple. Then everything should work automatically and we
avoid maintaining any MC code.
On Tue, 12 May 2020 at 23:11, Vaggelis Papoutsellis <
<EMAIL_ADDRESS>wrote:

Assigned #54 https://github.com/vais-ral/CCPi-astra/issues/54 to
@jakobsj https://github.com/jakobsj.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/vais-ral/CCPi-astra/issues/54#event-3329341718, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMDSCDUICRKU43NECMBD4DRRG3QLANCNFSM4M7FT6QA
.

from ccpi.framework import ImageGeometry, AcquisitionGeometry
from ccpi.optimisation.operators import ChannelwiseOperator
from ccpi.astra.operators import AstraProjectorMC, AstraProjectorSimple
from timeit import default_timer as timer
import numpy as np

chan = 80

ig = ImageGeometry(voxel_num_x = 100,
                   voxel_num_y = 100,
                   voxel_size_x = 1.,
                   voxel_size_y = 1.,
                   channels = chan,
                   dimension_labels = ['channel','horizontal_y','horizontal_x'])

ag = AcquisitionGeometry('cone',
                         '2D',
                         np.linspace(0,np.pi,100),
                         pixel_num_h = 100,
                         pixel_size_h = 1,
                         dist_source_center = 10,
                         dist_center_detector = 10,
                         channels = chan,
                         dimension_labels = ['channel', 'angle', 'horizontal'])

Aop1 = AstraProjectorMC(ig, ag, 'cpu')

igtmp = ig.clone()
igtmp.shape = ig.shape[1:]
igtmp.dimension_labels = ['horizontal_y', 'horizontal_x']
igtmp.channels = 1

agtmp = ag.clone()
agtmp.shape = ag.shape[1:]
agtmp.dimension_labels = ['angle', 'horizontal']
agtmp.channels = 1

Aop2 = ChannelwiseOperator(AstraProjectorSimple(igtmp, agtmp, 'cpu'), chan)

x = ig.allocate('random')

t1 = timer()
#a1 = Aop1.norm()
a1 = Aop1.direct(x)
t2 = timer()
print(t2 - t1)

t3 = timer()
#a2 = Aop2.norm()
a2 = Aop2.direct(x)
t4 = timer()
print(t4 - t3)

np.testing.assert_array_almost_equal(a1.as_array(),a2.as_array(), decimal = 4)

It works with Channelwise and have no problem replacing. The only thing is that I do not like doing the tmp trick for geometries. We need somehow to recover geometries without channels

Agree. Can subset do it? I mean for a multichannel data set we can obtain a
specific single channel data set by subset with e.g. channel=1. Will
something like subset be possible just for geometries? In fact, subset must
modify the geometry as part of the dataset it's working on, how does it do
that?
On Wed, 13 May 2020, 11:34 Vaggelis Papoutsellis<EMAIL_ADDRESS>wrote:

from ccpi.framework import ImageGeometry, AcquisitionGeometryfrom ccpi.optimisation.operators import ChannelwiseOperatorfrom ccpi.astra.operators import AstraProjectorMC, AstraProjectorSimplefrom timeit import default_timer as timerimport numpy as np
chan = 80
ig = ImageGeometry(voxel_num_x = 100,
voxel_num_y = 100,
voxel_size_x = 1.,
voxel_size_y = 1.,
channels = chan,
dimension_labels = ['channel','horizontal_y','horizontal_x'])
ag = AcquisitionGeometry('cone',
'2D',
np.linspace(0,np.pi,100),
pixel_num_h = 100,
pixel_size_h = 1,
dist_source_center = 10,
dist_center_detector = 10,
channels = chan,
dimension_labels = ['channel', 'angle', 'horizontal'])
Aop1 = AstraProjectorMC(ig, ag, 'cpu')
igtmp = ig.clone()
igtmp.shape = ig.shape[1:]
igtmp.dimension_labels = ['horizontal_y', 'horizontal_x']
igtmp.channels = 1
agtmp = ag.clone()
agtmp.shape = ag.shape[1:]
agtmp.dimension_labels = ['angle', 'horizontal']
agtmp.channels = 1
Aop2 = ChannelwiseOperator(AstraProjectorSimple(igtmp, agtmp, 'cpu'), chan)
x = ig.allocate('random')
t1 = timer()#a1 = Aop1.norm()
a1 = Aop1.direct(x)
t2 = timer()print(t2 - t1)
t3 = timer()#a2 = Aop2.norm()
a2 = Aop2.direct(x)
t4 = timer()print(t4 - t3)
np.testing.assert_array_almost_equal(a1.as_array(),a2.as_array(), decimal = 4)
It works with Channelwise and have no problem replacing. The only thing is
that I do not like doing the tmp trick for geometries. We need somehow to
recover geometries without channels
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vais-ral/CCPi-astra/issues/54#issuecomment-627869601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMDSCGV77ITMGMGFBK2TLLRRJSTRANCNFSM4M7FT6QA
.

Subset should give you a geometry consistent with the slice you are using.
However  this is going to copy the data to a new container. Maybe we just want a pointer to the start of the data we're interested in?

We don't really need to handle any data. From one geometry with multiple
channels we want to neatly create a new geometry (no data) which is
identical apart from not having any channels. Having the user to create
this single-channel geometry is inconvenient.
If there is no simple way to do this, a workaround could be to keep the
AstraMC projector and as input take the MC geometries. Then in  init do
the generation of the corresponding single-channel geometry, store the
number of channels and use ChannelwiseOperator and AstraProjectorSimple to
set it up. That would be quite neat actually. Then the user needs not worry
about anything other than saying create me an AstraMC projector, without
having to know about ChannelwiseOperator and single-channel geometries.
On Wed, 13 May 2020 at 12:25, Gemma Fardell<EMAIL_ADDRESS>wrote:

Subset should give you a geometry consistent with the slice you are using.
However this is going to copy the data to a new container. Maybe we just
want a pointer to the start of the data we're interested in?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vais-ral/CCPi-astra/issues/54#issuecomment-627894181,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACMDSCEZTLA4YYSAJPYUNA3RRJYRXANCNFSM4M7FT6QA
.

If we are saying the astra projectors won't run across channels then we could move this inside the projector operator, it could take the full geometry in but just set it up with channel=1? I'm not sure if that's logical behaviour though. I guess as long as there was a warning it'd work.
I also think it'd be good to have an option that removes the choice of projector from the user. We could choose what to do ourselves based on the AG. if it's passed channel data it runs channelwise. For example the new AstraProjectorFlexible doesn't care if it's passed a 2D or 3D dataset at the moment... although this is still a wip.
Just as a note, in the new geometry shape and labels will be updated automatically once channels is set equal to 1. So I think it'll look neater then anyway.

ProjectionOperator now handles multichannel data with the ChannelwiseOperator so I am closing this issue.


Support CodeLen

CodeLen has been a part of LSP standard specification for a while now, some LSP such as ccls also implement it, so, can lspsage support the feature?
Describe the solution you'd like
I know another plugin navigator has supported it and this is the example,

Okay. it will be support:)

Hi, sorry for later, because I work on this recently and want to know what's your exactly means about codelens feature. Codelens can also be execute as codeaction and I assume what you want is just a UI like codeaction in lspsaga? Do you know any language server can provide the case that multi codelens one line? Because if it's only one codelens and this feature will slow down efficiency instead of the opposite.
BTW, the ccls's codelens have no effect after execution, it's just same as vim.lsp.codelens.run() and I need to look into it.

Codelens can also be execute as codeaction and I assume what you want is just a UI like codeaction in lspsaga?

yes, I mean the UI.

Do you know any language server can provide the case that multi codelens one line?

Sorry, I didn't notice if any language server provided this feature.


CI: Caching for integration test is broken

for example in this run
in unit test: just compiles the workspace crates and all the deps are used from cache
in integration test: every crate is being compiled

We've migrated the Ci to Nix Flakes + cachix. I think this is fixed now.


Swifttemplate includeFile("../lib/MyFile.swift") not documented: is it supported?

Question:  is it intended that the <%- includeFile("../lib/MyFile.swift") -%> pattern is supported in swifttemplate generation?  I stumbled onto it in the tests, and it appears to work as expected.  Is there any reason we should avoid it?

Submitted PR #892 with doc addition

Submitted PR #892 with doc addition


Function params/args identification

You'll be happy to know this is just a feature request :smile:
It would be really helpful to have this:

Currently, we either get an error because the params are not provided, or nothing at all if no params are required.
screen shot taken from VS Code

Go inside a console.log() and press ctrl + space:

Doesn't show up by default because of https://github.com/atom/autocomplete-plus/issues/329
Being tracked locally here : https://github.com/TypeStrong/atom-typescript/issues/23 (so marking this as duplicate, unless I've misunderstood the issue?

Ah, okay, yes that was what I was looking for. I tried to search for it, but I didn't find that issue.


🚀 Feature: Add Breadcrumbs to Entity Catalog for Improved Navigation

🔖 Feature description
Currently, there is no breadcrumb navigation in the Entity Catalog page, making it difficult for users to navigate across entities. Adding breadcrumbs that reflect entity relationships (e.g., Group > User, System > Domain > Component) would provide better context and improve the user experience, especially when dealing with large entity catalogs.
🎤 Context
In Backstage's Entity Catalog, users often deal with complex hierarchies of entities. Navigating back to parent entities is cumbersome without visual cues like breadcrumbs. This feature would be beneficial for users interacting with related entities (e.g., from systems to components) or large catalogs with multiple layers of relationships.
✌ Possible Implementation
Here are examples of how the breadcrumbs would appear for different entities:

User example:

Component Example

Allow customization of breadcrumb structures depending on the organization's needs.

👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

The PR need review @backstage/catalog-maintainers ^^

The PR is merged, hence closing this issue


SimpleSkin sample model doesn't match tutorial

The SimpleSkin tutorial model doesn't quite match some of the explanations and diagrams in skinning.
Specifically, the sample model places a translation on node index 1, which is actually joint 0, the parent joint.  It places a rotation but not a translation on node 2, joint 1, which is the only child joint.
This means the parent joint is moved to Y + 1, and the child joint is co-located with the parent joint.  The sample model still works, but only because the parent joint never moves, its only job is to hold its vertices in place.
Also, separate from the above, at some point we made the decision to zero out the first two lines of JOINTS_0 (to silence a validation warning about zero weights on non-zero joints), and some of the wording needs to be updated to reflect what happened there.

The model works because for both joints the inverseBindMatrices tanslate the model by -0.5 on the X axis and -1 on the Y axis.
This indicates that the model need to be translated by 1 on the Y axis to be correctly placed. If the parent joints move, the model will follow it without issue.
That said, I think it is more intuitive to place the translation on the joint 1 and change the inverseBindMatrices of the joint 0 to  this:
1.0   0.0   0.0   -0.5
0.0   1.0   0.0    0.0
0.0   0.0   1.0    0.0
0.0   0.0   0.0    1.0

I also take this opportunity to raise another problem on these pages.
The inverseBindMatrices on this page should be
1.0   0.0   0.0   -0.5
0.0   1.0   0.0   -1.0
0.0   0.0   1.0    0.0
0.0   0.0   0.0    1.0

In addition, the images 20b, 20c and 20d should be change to place the origin and the joints at the correct location. For exemple, the first image of 20c should be something like this:

But doing these changes will make the tutorial harder to understand, thus I think it is best to change the model.
The translation on joint 0 should be moved to joint 1, the position of the vertices should be translated by -0.5 on the X axis (the actual position are from (0, 0, 0) to (1, 2, 0), the translated position are from (-0.5, 0, 0) to (0.5, 2, 0))  and the inverseBindMatrices should be changed accordingly.
With these change, the model will truly reflect what we see on the images.

There is a pending update for this model, c.f. https://github.com/KhronosGroup/glTF-Tutorials/pull/41 . This specific model causes a remarkable amount of issues - hope we get this ironed out at some point...

@foxiler I think you're on the right track here.  The original (un-edited) diagram shows how it should ideally work, and I think it was just a mistake that crept into an otherwise carefully hand-crafted sample, where the Y axis translation was applied to the wrong joint, but that joint never moves so it doesn't impact the resulting animation.  Would you be willing to put together a PR?


Docs: Refactor 'Get started' section

Refactor 'Get started' section for usability, reducing redundancy. Highlights:

Tighten up 'Get started' main page to make it more breezy
Move CLI reference to its own page
Remove duplication of Folder Structure
Move prerequisites to Development Environment topic

Fixes https://github.com/grafana/plugin-tools/issues/468

Replaced with https://github.com/grafana/plugin-tools/pull/479


Update docker-compose.yml

Codecov Report

Merging #228 into master will decrease coverage by 0.19%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master     #228     +/-   ##
===========================================
- Coverage     57.73%   57.54%   -0.2%
+ Complexity     6586     6557     -29
===========================================
  Files           460      460
  Lines         25769    25769
  Branches       3391     3391
===========================================
- Hits          14878    14829     -49
- Misses         8319     8368     +49
  Partials       2572     2572

Impacted Files
Coverage Δ
Complexity Δ

...k/devices/model/DeviceEnrollmentBulkDeleteDao.java
34.37% <0%> (-12.5%)
6% <0%> (-3%)

...eviceupdate/model/AbstractFirmwareManifestDao.java
30.76% <0%> (-11.54%)
6% <0%> (-2%)

...ud/sdk/security/model/DeveloperCertificateDao.java
33.33% <0%> (-10%)
8% <0%> (-2%)

...d/cloud/sdk/devices/model/DeviceEnrollmentDao.java
33.33% <0%> (-7.41%)
7% <0%> (-1%)

...ed/cloud/sdk/accounts/model/UserInvitationDao.java
31.03% <0%> (-6.9%)
7% <0%> (-1%)

.../model/AbstractSubtenantTrustedCertificateDao.java
28.12% <0%> (-6.25%)
7% <0%> (-1%)

...k/deviceupdate/model/AbstractFirmwareImageDao.java
33.33% <0%> (-4.17%)
6% <0%> (-1%)

...ud/sdk/security/model/AbstractPreSharedKeyDao.java
28% <0%> (-4%)
5% <0%> (-1%)

.../mbed/cloud/sdk/connect/model/Presubscription.java
51.89% <0%> (-3.8%)
26% <0%> (-2%)

...urity/model/SubtenantTrustedCertificateStatus.java
58.62% <0%> (-3.45%)
10% <0%> (-1%)

... and 25 more

Continue to review full report at Codecov.

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


Update IntExtensions.swift

fix "isPrime()" bug

Checklist

[ ] I checked the Contributing Guidelines before creating this request.
[ ] New extensions are written in Swift 4.
[ ] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[ ] I have added tests for new extensions, and they passed.
[ ] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[ ] All extensions are declared as public.

          1 Warning

:warning:
The source files have been modified. Please consider adding a CHANGELOG entry if necessary.

          1 Message

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

  Generated by :no_entry_sign: Danger

@Asura19 This looks great 💪 Thanks for fixing this 💯

Thanks @Asura19 👍

My pleasure~ @omaralbeik @SD10


Does not work when body has been changed

Bug report
I'm using a library called morphdom to patch document.body with new contents on navigation to different pages. It seesm that after doing this then pulltorefresh stops working or misbehaves. I've tried calling destroy or destroyAll before patching and init afterwards, but it doesn't seem to make any difference.
Is it possible that pulltorefresh does not destroy itself properly thus making it unable to re-initialize?

I don't think any library will persist when you change the DOM, as the events, properties and markup will change. Maybe you can try initializing ptr.js on every DOM change.
Also, please keep the template to report a bug.

Thank you for your reply!
However, this doesn't solve the problem. Reading the things you say then I can assume that destroy and destroyAll functions do not clean up properly and there is still some state somewhere, which will mess things up. This implies that these functions do not work as expected or I am missing something.
There's no reasonable way for me to re-initialize ptr.js again since it is installed into node_modules and packaged as a bigger bundle with every other vendor library.
Are you implying that while initializing ptr.js then some global state will be persisted before calling init?
I tried to understand by looking into the library code, but couldn't see at first glance anything which might be the culprit.
Any help would be appreciated and sorry for not keeping the bug report template.

If you found what you believe it's a bug, feel free to create a PR, explain your case and we'll review it.


Horizontal visulization added in event analytics

Description
Added Horizontal Bar chart as a new visualization type.
Issues Resolved
#885
Check List

[ ] All tests pass, including unit test, integration test and doctest
[ ] New functionality has been documented.
[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

created final PR - https://github.com/opensearch-project/observability/pull/924.
so closing this one.


[stable/prometheus-operator] Namespace override

What this PR does / why we need it:
It adds a namespaceOverride parameter, which allows this chart to be used in cases where templating is used across namespaces. By providing this parameter, the resulting templating will output to the specified namespace.
Special notes for your reviewer:
This change is needed for implementing #18837 and inspired by #18986

My helm lint . here is all OK, my helm version is 2.16.1.

hi @bismarck and @gianrubio, can we please re-open this? what is the next step for this?


Add a warning when fitting models loaded from SavedModels

Add a warning when fitting models loaded from SaveModel having optimizers  with slots.
Ref: https://github.com/tensorflow/tensorflow/pull/42749#issuecomment-683964931

/cc @reedwm  @k-w-w

This is just an exploring proposal. If you have a better entry point let me know.

@bhack @reedwm @k-w-w  This PR is in draft, any update on this? Please. Thanks!

@gbaned I have any private channel so the thread it is what you see here.

Can you give context for this PR? I think adding a warning in fit() for loaded saved models with slot variables is sensible, but I don't see any warnings added in this PR.

@reedwm If you see is a draft as I have asked for comment on 1st September about the entry point I selected in the code to introduce" the condition but as of 2nd December I got any feedback.
I have any other chat channel channel to discuss with devs as Gitter SIG Keras-io/Lobby in the current status is useless for discussion about PRs.

Unfortunately I am not very familiar with the SavedModel code so I'm not sure if this is a good place to do it. Sorry for not responding until now.
@k-w-w, do you think it would be a good idea to add a warning if a SavedModel with slot variables is restored and Model.fit() is used? Currently slot variables are not restored in such a case, which could affect training and is why we should potentially add a warning. If this is a good idea, would the code modified in this PR be a good place to check for the use of slot variables?

@reedwm Thank you. Just to note that I've already asked to @k-w-w on 23th Oct at https://github.com/tensorflow/community/pull/281#issuecomment-715596624

I think directly sending the warning when loading would make sense (see the warning message for the HDF5 format: https://github.com/tensorflow/tensorflow/blob/a6b7e4b94ceb3471a268a57741fe1f32a1472119/tensorflow/python/keras/saving/hdf5_format.py#L211)

The change looks good, although I think directly sending the warning when loading would make sense (see the warning message for the HDF5 format:
https://github.com/tensorflow/tensorflow/blob/a6b7e4b94ceb3471a268a57741fe1f32a1472119/tensorflow/python/keras/saving/hdf5_format.py#L211
)

Do you like the same HDF5 warning message or something else?

Ok Windows is failing for unrelated MLIR issues and Macos CPU is failing probably for other unrelated stuff but has any visible details.
The other checks are passing.

@k-w-w The sanity is temp broken by https://github.com/tensorflow/tensorflow/pull/48294#issuecomment-849923841

@k-w-w The sanity is temp broken by #48294 (comment)

Oh I see, good to know

Can you re-trigger the build? I think that the CI Sanity it is working again.

Do you need anything else here?

@k-w-w If you want you can add a reference somewhere to https://github.com/tensorflow/tensorflow/issues/44670


Expose font-family property for labels / tooltips

Would be great to expose the fontFamily property for the axes and tooltips.

You can style this via CSS but maybe you will have to use the !important priority

Which css class should I try to extend in order to keep the scope inside the library ?

With this style:
.chart *[style] {
  font-size: 11px !important;
  font-family: "Impact", Helvetica, Arial, sans-serif !important;
  -webkit-font-smoothing: antialiased;
}

I have this results:

In order to style the legend as well i need to use something like:
.chart *[font-family] {
  font-family: "Impact", Helvetica, Arial, sans-serif !important;
}

The reason why is that the inlining of the css is sometimes done with style=""  and sometimes with just attribute-name="value".
Worth checking if is possible to have a consistent way to express inline style.

Thanks for your example! Yeah, the legend styles are used as attributes - we need to change this.
We will use CSS styling for the next major version, that's the only reason why I am against the inlining.

Hi @nicolabortignon
how is it going with this issue? do you need help on anything?
Best,
Christoph


Improve article 1602.02769

It may be better to also link the PDF of the ar5iv 1602.02769.
Currently, this document hits a Fatal error, which improves with latexml PR 1858 introduced and becomes a very interesting stress test for advanced equations that combine MathML with SVG.
We may want to study this article's conversion in detail, when we have the focus to maximize the quality of such advanced equations.

We no longer emit a Fatal here, so this article may now be appropriate to use as a stress-test for the SVG+MathML we emit.


update pion/udp

Codecov Report

Merging #406 (60b1d46) into v3 (e5ec65d) will increase coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##               v3     #406      +/-   ##
==========================================
+ Coverage   70.16%   70.35%   +0.19%
==========================================
  Files          68       68
  Lines        5229     5229
==========================================
+ Hits         3669     3679      +10
+ Misses       1179     1170       -9
+ Partials      381      380       -1

Impacted Files
Coverage Δ

net/conn.go
87.50% <0.00%> (-1.57%)
:arrow_down:

dtls/server/server.go
79.25% <0.00%> (+2.22%)
:arrow_up:

net/dtlslistener.go
73.46% <0.00%> (+4.08%)
:arrow_up:

...ent/limitParallelRequests/limitParallelRequests.go
92.75% <0.00%> (+5.79%)
:arrow_up:

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


AI powered search

I am thinking of using GCP's video intelligence api(uses REST API) to train some of the courses video and try to make the search term accurate for the users.
So, whenever a user searches for like "authentication using nodejs" then it will display bunch of courses where one of the course video is teaching how to do that.

@RohankumarM  actually I been working on REST API can u guide regarding this issue

@harkiratsm Yeah what is it?


[Bugfix] GPU memory profiling should be per LLM instance

gpu_memory_utilization was intended to limit the total memory allocation for an instance of an LLM. An update to the memory profiling changed the meaning of this parameter to be a global limit on GPU memory allocation (see this comment).
This PR reverts the change gpu_memory_utilization back to being a per-model-instance limit.
Refer to #10451 for more background and discussion.
FIX #10451

The current changes implement the quickfix I suggested in https://github.com/vllm-project/vllm/issues/10451#issuecomment-2487047316, but, during validation, I noticed another consequence. Even with this PR, the memory allocation will measure all Torch memory allocated by the python instance, not just the memory allocated for the instance of the model being profiled. In other words, it fixes multiple vLLM servers sharing the GPU but not multiple LLMs within a single vLLM instance. So support for that is still a TODO.

multiple LLMs within a single vLLM instance

what is this?


Suggested change to absolute paths + Missing files

Hi! Congrats for the nice paper and thanks for open sourcing the code!  I have one question and one remark about the code:
Question:
I want to run the training example, but I got the following error: No such file or directory: '[..]/ForecastPFN/academic_comparison/metalearned/project/storage/resources/tourism/cache.dill'. However, I cannot find it in the repo. Is it possible to obtain these files somehow?
Remark:
I tried to rerun the inference example, however I faced some problems with some fixed paths, such as the one HERE.
Thanks again!
Best,
Sebastian

thank you for taking a look at our work!
the error here comes from using an auxiliary dataset to validate performance during training. that dataset is large and was part of the Meta-N-BEATS project which we compare to in our paper.
since this isn't necessary for the training pipeline, i have removed it from the repo in src/training/train.py file. it should now run without any problem.


Non-functional version of AWS custom runtime sample barfs on startup

If you use @Configuration instead of an ApplicationContextInitializer the app blows up on startup:
2020-03-24 14:47:14.387  INFO 13868 --- [           main] o.s.c.f.web.source.SupplierExporter      : Starting
2020-03-24 14:47:14.395  INFO 13868 --- [           main] c.f.c.c.BeanFactoryAwareFunctionRegistry : Looking up function 'origin|' with acceptedOutputTypes: []
2020-03-24 14:47:14.403  WARN 13868 --- [           main] c.f.c.c.BeanFactoryAwareFunctionRegistry : !!! Failed to discover function 'origin|' in function catalog. Function available in catalog are: [foobar, origin, functionRouter]
2020-03-24 14:47:14.403  WARN 13868 --- [           main] o.s.c.f.web.source.SupplierExporter      : No such Supplier: origin|
2020-03-24 14:47:14.416  INFO 13868 --- [           main] o.s.c.f.web.source.SupplierExporter      : Stopping
2020-03-24 14:47:14.424  WARN 13868 --- [           main] onfigReactiveWebServerApplicationContext : Exception encountered during context initialization - cancelling refresh attempt: org.springframework.context.ApplicationContextException: Failed to start bean 'sourceForwarder'; nested exception is reactor.core.Exceptions$ErrorCallbackNotImplemented: java.lang.NullPointerException
2020-03-24 14:47:14.454  INFO 13868 --- [           main] ConditionEvaluationReportLoggingListener :

Error starting ApplicationContext. To display the conditions report re-run your application with 'debug' enabled.
2020-03-24 14:47:14.461 ERROR 13868 --- [           main] o.s.boot.SpringApplication               : Application run failed
...
Caused by: reactor.core.Exceptions$ErrorCallbackNotImplemented: java.lang.NullPointerException
Caused by: java.lang.NullPointerException: null
        at org.springframework.cloud.function.web.source.SupplierExporter.stop(SupplierExporter.java:136) ~[spring-cloud-function-web-3.0.4.BUILD-SNAPSHOT.jar:3.0.4.BUILD-SNAPSHOT]
        at org.springframework.cloud.function.web.source.SupplierExporter.lambda$start$1(SupplierExporter.java:120) ~[spring-cloud-function-web-3.0.4.BUILD-SNAPSHOT.jar:3.0.4.BUILD-SNAPSHOT]
        at reactor.core.publisher.FluxPeek$PeekSubscriber.onComplete(FluxPeek.java:243) ~[reactor-core-3.3.2.RELEASE.jar:3.3.2.RELEASE]
...

It looks like maybe the FunctionCatalog is of the wrong type?

I have seen this before and it has to do with the fact that subscription is null. Why? Not sure, but may be it's time to look

The NPE is actually only a side effect of the main problem. It occurs on shutdown if the function exporter failed on startup. It's actually the WARN that is the root of the problem. It would be good to fix both of course.


Update acronym_test.rb

There seem to be 7 in-browser-tests on Exercism, but in the CLI version there is only 1 (the other tests are skipped).
I wonder what is right now? Should there only be one easy test, since this is one of the beginner tracks? The tests seem to be directed to more advanced programmers.

In the README, does it mention to unskip the tests, or in the test file itself?  At some point, early on, when using the CLI this information is shared.  Concept exercises do not have the skips, while the other exercises do.  On the platform they are all "unskipped" for testing automatically.
They are both correct, unskip the tests as you work through it emulating TDD, or disable them all by maybe writing:
  def skip() end

In the README, does it mention to unskip the tests, or in the test file itself? At some point, early on, when using the CLI this information is shared. Concept exercises do not have the skips, while the other exercises do. On the platform they are all "unskipped" for testing automatically.

Hmm strange, I checked the help and readme file again, even the test file but couldn't find any information regarding that. I was confused further since other test files didn't have the skip. But yeah, the TDD emulation part of your explanation makes sense.
I am new to exercism, so I totally believe that there might be something important I haven't read. 😅


Support for Azure Quantum

Summary of the new feature / enhancement
Are there plans to support or add cmdlets for the Azure Quantum service? I can get back a quantum.workspace object, but only the generic properties - none of the specific properties that let me create or edit a quantum.workspace resource.
Proposed technical implementation details (optional)
No response

Is this an issue for Azure/azure-powershell rather than the core PowerShell?

Thanks, I will move it over there.


DndContext reducers all point to the same initial state variable, which is causing id collisions across nested and sibling providers

Hi,  First of all, great job building this. It's an awesome library.
I am facing a problem with a use case where I have a nested structure. Can you please help resolve this?
I have a structure similar to the following:
<DndContext>

        <Draggable />
        <Draggable />
        {/* multiple draggable items */}

          <Droppable>
             <DndContext>
                  <SortableContext>
                        <SortableItem />
                        <SortableItem />
                        {/* multiple sortable items */}
                  </SortableContext>
             </DndContext>
         </Droppable>

        <Droppable>
             <DndContext>
                  <SortableContext>
                        <SortableItem />
                        <SortableItem />
                        {/* multiple sortable items */}
                  </SortableContext>
             </DndContext>
         </Droppable>
         {/* Multiple similar droppable items */}

</DndContext>

Now, I am able to move the draggable items to droppable zones. But the events are not being detected when trying to sort items in the individual droppable zones. I assume it is something related to event bubbling, but not sure. Any help in this regard will be appreciated. Thanks 🙏
Codesandbox Link:
https://codesandbox.io/s/dnd-kit-gm09q?file=/src/App.js
Demo:
https://user-images.githubusercontent.com/17903466/104116991-a325a680-5343-11eb-96e0-469b168f83dd.mov
EDIT:
In the same codesandbox, add an empty section and try to add Page 12 to it before anything else. It isn't getting added. I don't know if I am missing something here.

Hey @pbteja1998, I haven't had time to look in-depth into the root cause of the issue in your case, that is, whether it is an issue in the library or a consumer error, but from what I can tell it appears like there was an id collision with your sortable items. I updated the id of the sortable items to something like sortable-${id} which seems to have resolved the collision. More investigation will be required though.
https://codesandbox.io/s/dnd-kit-forked-hhfkk?file=/src/App.js
As for the issue with pages that are scrolled in the scrollbar, it's an issue that was reported here and for which a fix is in progress: #43

Hey @pbteja1998, I haven't had time to look in-depth into the root cause of the issue in your case, that is, whether it is an issue in the library or a consumer error, but from what I can tell it appears like there was an id collision with your sortable items. I updated the id of the sortable items to something like sortable-${id} which seems to have resolved the collision. More investigation will be required though.
https://codesandbox.io/s/dnd-kit-forked-hhfkk?file=/src/App.js
As for the issue with pages that are scrolled in the scrollbar, it's an issue that was reported here and for which a fix is in progress: #43

hi @clauderic, Thanks a lot for the help. Changing the id to sortable-${id} seems to have fixed it.
Here is the updated code sandbox for reference.
https://codesandbox.io/s/dnd-kit-gm09q?file=/src/App.js
And Changing the collision strategy to closestCenter also fixed the second scroll bug that I asked about.
Again, thanks a lot for the fast response.
I have one last question. Why is the id collision happening? From my understanding, they are in separate lists, aren't they? I was under the assumption that draggable items in the same list should have different ids, but it seems like all the draggable items on the entire page should have a different ID. Can you please clarify this?
Thanks.

hi @clauderic, Thanks a lot for the help. Changing the id to sortable-${id} seems to have fixed it.
Here is the updated code sandbox for reference.
https://codesandbox.io/s/dnd-kit-gm09q?file=/src/App.js
And Changing the collision strategy to closestCenter also fixed the second scroll bug that I asked about.
Again, thanks a lot for the fast response.
I have one last question. Why is the id collision happening? From my understanding, they are in separate lists, aren't they? I was under the assumption that draggable items in the same list should have different ids, but it seems like all the draggable items on the entire page should have a different ID. Can you please clarify this?
Thanks.

Hey @pbteja1998, after some investigation I've found that this was indeed an issue with the library and not a consumer error. Thanks for reporting this issue, it allowed me to uncover a really bad bug https://github.com/clauderic/dnd-kit/pull/51

Hey @pbteja1998, after some investigation I've found that this was indeed an issue with the library and not a consumer error. Thanks for reporting this issue, it allowed me to uncover a really bad bug https://github.com/clauderic/dnd-kit/pull/51

Hey @pbteja1998, after some investigation I've found that this was indeed an issue with the library and not a consumer error. Thanks for reporting this issue, it allowed me to uncover a really bad bug #51

@clauderic Awesome. Thanks for the clarification. Now, I will just wait for #43 to get fixed. Will try to go through the code myself and see if I can fix it myself and will raise a PR if I do so.

Hey @pbteja1998, after some investigation I've found that this was indeed an issue with the library and not a consumer error. Thanks for reporting this issue, it allowed me to uncover a really bad bug #51

@clauderic Awesome. Thanks for the clarification. Now, I will just wait for #43 to get fixed. Will try to go through the code myself and see if I can fix it myself and will raise a PR if I do so.

Hey @pbteja1998, the PR with the fix is already open here #51 if you're curious about the fix, but thanks for offering to look into it

Hey @pbteja1998, the PR with the fix is already open here #51 if you're curious about the fix, but thanks for offering to look into it

Hey @pbteja1998, the PR with the fix is already open here #51 if you're curious about the fix, but thanks for offering to look into it

@clauderic Will this also fix #43 -- fixing RectIntersection algorithm inside a scrollable container?

Hey @pbteja1998, the PR with the fix is already open here #51 if you're curious about the fix, but thanks for offering to look into it

@clauderic Will this also fix #43 -- fixing RectIntersection algorithm inside a scrollable container?

 It will not, no. I have a separate fix in progress for #43

Oh sorry, I misread your comment. It will not, no. I have a separate fix in progress for #43

Oh sorry, I misread your comment. It will not, no. I have a separate fix in progress for #43

Awesome. Thanks.

 It will not, no. I have a separate fix in progress for #43

Awesome. Thanks.

The fix has shipped in 1.0.1, let me know if you run into other issues related to unexpected id collisions

The fix has shipped in 1.0.1, let me know if you run into other issues related to unexpected id collisions

Will update and revert back from sortable-${id} to id and check.

Will update and revert back from sortable-${id} to id and check.


Implement find_full_path within ephys modules

And moved functions to element-data-loader.
All workflow-array-ephys pytests passed.
Rename element-data-loader to element-interface.

Thanks @ttngu207.  Changes implemented.  Ready for review and merge.

@ttngu207 In this pull request, I also tested your changes for the case of a missing fileTimeSecs in the SpikeGLX meta file.

Added

Rename element-data-loader to element-interface


Inputs of the SetTransformer

Hi,
Could you please explain the meanings of the inputs of SetTransformer:

dim_input, num_outputs, dim_output, num_inds=32, dim_hidden=128, num_heads=4, ln=False

Thanks.

I think the names of the variables pretty much explain everything.  Dimension of the inputs, number of outputs (number of seeds for output PMA), dimension of the outputs, number of inducing points, and so on.

For this layer
SetTransformer(dim_input = 95000, num_outputs = 1024, dim_output = 1, num_inds=32, dim_hidden=128, num_heads=8, ln=True)

I get this error:
--> 801     return torch._VF.split(self, split_size, dim)  # type: ignore[attr-defined]
802 else:
803     return torch._VF.split_with_sizes(self, split_size, dim)
IndexError: Dimension out of range (expected to be in range of [-2, 1], but got 2)


Import support

I note in the README the statement about imports being not supported. It looks like upstream they were added a few months ago.
I'm just wodering whether you're already planning on supporting this, or whether it's stalled for now?
You mention the need for careful design and thought on this feature - I quite understand. The main thing to me seems the need to figure out the module import path from relative imports (assuming that you essentially go down the route of one bop file == one go file). I'm keen to discuss your thoughts around this - I may very well be keen to contribute shortly.

I must have missed when this feature was added to the upstream implementation-- Yes we absolutely want to support it, I'll need to review what they implemented before coming up with how it should work over here.

It looks like they also snuck in support for constant declarations in that same push (https://github.com/RainwayApp/bebop/wiki/Writing-Bops%3A-The-Bebop-Schema-Language#const), I'll open an issue for that.

Cool. Very happy to leave you to it if you already know how you'd want to do it. But happy to help if you'd like.

It looks like only relative imports are supported, like you said. My thoughts so far are:

Just combine everything into one file, unfortunately, because relative imports aren't supported in Go or
Accept meaningful comments that would be ignored by another compiler but would be used to determine the module path in Go-- or a predetermined constant name, like go_package. This is similar to how protobuf works in Go e.g. option go_package = "example.com/project/protos/fizz"; (link). The issue with using a constant would be that there's a naming collision possibility-- someone names their const go_package = "go!" and now we can't compile that file, but maybe that's unlikely enough that it doesn't matter.
Some combination of the two, offer either mode, toggled via a compilation flag.

(1) seems initially attractive because it's simple. But, down the road it causes all manner of problems. You've probably split up your protocol into different .bop files because you have some common definitions and then you're reusing them for different audiences. But with (1), if you decide to compile for those different audiences then you'll end up with multiple types-in-go for the same type-in-bop. Provided the opcodes are the same, this might not be a problem, but it would always be odd.
So, I reckon (2) is the right way to go; plus the weight of prior art here (not just protobuf but capnproto does the same too). One bop file == one go file. So then for imports in bop, all you need to do is open the imported file and find the special go_package const and that just tells you how to import that for the current bop file translation. So you just assume that anything imported is already translated (or will be). So then your cli might drop the -o option completely, and maybe drops -i too and you just use all non-flag args to be the files to translate (and maybe have a -r option to recurse through directories).
(3) sounds a little terrifying to convey to the user exactly what the semantics are. I always feel a simple model for the user to understand is better.
Just my 2p :)

oh, one question that I don't think is clear from the upstream:
a.bop defines Foo
b.bop imports a.bop and defines Bar
c.bop imports b.bop.
Does c.bop have access to Foo? I.e. does a bop file export only the types it directly defines, or does it also export everything it imported?

I'll do some kind of compatability testing to see if imports add to what a file itself exports, and work on the magic const implementation. That requires that consts are implemented, which I'm working towards (#11).

Interesting discoveries. Yeah, it does seem shortsighted to me, but, I can also understand the desire to copy the upstream tool too.


GUI

It would be convenient for the user to have a GUI where he can click together his config instead of copying PNG-files and creating/modifying config files.
Ideas:

Multi-Selects for Prefixes und Suffixes
Delays
Radio-Buttons (with the images) for Std/Advanved/Armor Gambles

Sorry what?


text log should remove trailing space

current code is generating every log line with a trailing space, which is not good and should be eliminated
tf := &TextFormatter{DisableColors: true}
byts, _ := tf.Format(&Entry{Message: "msg content"})
fmt.Printf("%q\n", string(byts))

"time=\"0001-01-01T00:00:00Z\" level=panic msg=\"msg content\" \n"

since #100 is reverted, should this be open again?

I'll take a look later this week

@Sirupsen what else changes do you want to see in this set?
https://github.com/crquan/logrus/compare/patch-1

@crquan @Sirupsen Is this still an issue?


wl-kbptr: init at 0.2.1

Description of changes
wl-kbptr make possible to control the mouse pointer with the keyboard on Wayland.

Release notes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

>  nix build .#wl-kbptr
> tree result/
 result
└──  bin
   └──  wl-kbptr

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3966

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1659

@SuperSandro2000 Applied suggestion

@ofborg eval