Add number of spells cast field

API issue: https://github.com/5e-bits/5e-srd-api/issues/361
I'd like to propose a new optional field for spells similar to damage_at_slot_level: casts_at_slot_level. casts_at_slot_level holds the value of how many times a specific spell is cast per usage.
For instance, Magic Missile can be cast 3 times, increasing by 1 for each slot level. An example field for Magic Missile would contain the following:
"casts_at_slot_level": {
  "1": 3,
  "2": 4,
  "3": 5,
  "4": 6,
  "5": 7,
  "6": 8,
  "7": 9,
  "8": 10,
  "9": 11
}

Another spell that would contain this field would be Eldritch Blast. I'm not sure the extent of spells that would need this field.
I'm not sure if there is a spell that does this, but this may also need to add casts_at_character_level if casts increases by character level and not slot level.

It might just be those two but I think it's a solid approach.

This definitely looks reasonable to me!
Just a note in case it becomes relevant, these types of attributes (damage_at_slot_level, heals_at_level...) that map a level to a value (map[int]->int) are problematic if/when it comes to generating a GraphQL schema from our MongoDB collections. Here's an old issue where it came up in the past: https://github.com/5e-bits/5e-srd-api/issues/60#issuecomment-719352129


Refactoring buttons

Removed the extra mentions to NegButton as per PR review.

Looks like there really isn't anything here, I'm gonna just close it unmerged.


Fractal theme isn't being found

I think this may be as a result of the changes that were made to the static build of Fractal, it may be an issue with using relative paths.
I'm seeing the following error: ✘ No matching route found for /components/detail/theme/govuk/css/fractal-govuk-theme.css.
This happens if I make a change to a theme file.

#140 fixes this.


Fixed build issues related to POM duplicate definitions.

POMs were polluted with duplicate definitions. I fixed it.

Refer to this link for build results (access rights to CI server needed):
/job/nd4j_PR/211/Failed Tests: 7nd4j_PR/org.nd4j:nd4j-aeron: 5org.nd4j.aeron.ipc.response.AeronNDArrayResponseTest.testResponseorg.nd4j.aeron.ipc.AeronNDArraySerdeTest.testToAndFromorg.nd4j.aeron.ipc.AeronNDArraySerdeTest.testToAndFromCompressedorg.nd4j.aeron.ipc.AeronNDArraySerdeTest.timeOldVsNeworg.nd4j.aeron.ipc.NdArrayIpcTest.testIpcnd4j_PR/org.nd4j:nd4j-kryo_2.10: 1org.nd4j.TestNd4jKryoSerialization.testSerializationnd4j_PR/org.nd4j:nd4j-parameter-server-client: 1org.nd4j.parameterserver.client.ParameterServerClientTest.testServer


Feat screen risk

Features:

Way to Search file or class by name
New screen with risked files (top candidates for refactoring)

Codecov Report
Attention: Patch coverage is 20.11173% with 143 lines in your changes are missing coverage. Please review.

Project coverage is 54.47%. Comparing base (7be140e) to head (1a5fda0).

Files
Patch %
Lines

src/Cli/ComponentSearch.go
8.19%
55 Missing and 1 partial :warning:

src/Cli/ComponentFileTable.go
30.35%
33 Missing and 6 partials :warning:

src/Cli/ScreenRisks.go
16.12%
26 Missing :warning:

src/Cli/ComponentClassTable.go
25.00%
18 Missing and 3 partials :warning:

src/Cli/ScreenSummary.go
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   56.56%   54.47%   -2.09%
==========================================
  Files          35       37       +2
  Lines        2461     2614     +153
==========================================
+ Hits         1392     1424      +32
- Misses        902     1014     +112
- Partials      167      176       +9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


This rule (and fixer) checks the "rel" value to make sure that it correctly relates to the tag that the "rel" property is on.

@ljharb Good idea about making this a more generic rule, though I don't know if I want to make it working with React.createElement in this PR.

@ljharb Good idea about making this a more generic rule, though I don't know if I want to make it working with React.createElement in this PR.

@ljharb I have changed the name of the rule and added the option as you said. While reworking the internals a bit (though I haven't tested the code with non-rel items)

@ljharb I have changed the name of the rule and added the option as you said. While reworking the internals a bit (though I haven't tested the code with non-rel items)

I know that there are some failing tests for some older versions of eslint. Will look at them soon.

I know that there are some failing tests for some older versions of eslint. Will look at them soon.

No rush, please mark as ready for review once tests are passing.

No rush, please mark as ready for review once tests are passing.

@ljharb I have added checking React.createElement.

Codecov Report

Merging #2863 (ce77747) into master (9f1d618) will decrease coverage by 0.07%.
The diff coverage is 93.10%.

@@            Coverage Diff             @@
##           master    #2863      +/-   ##
==========================================
- Coverage   97.22%   97.14%   -0.08%
==========================================
  Files         110      111       +1
  Lines        7312     7428     +116
  Branches     2669     2701      +32
==========================================
+ Hits         7109     7216     +107
- Misses        203      212       +9

Impacted Files
Coverage Δ

index.js
100.00% <ø> (ø)

lib/rules/no-invalid-html-attribute.js
93.10% <93.10%> (ø)

lib/util/Components.js
95.16% <0.00%> (-0.21%)
:arrow_down:

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 9f1d618...ce77747. Read the comment docs.

Maybe https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/autocomplete but I think that could easily be done in a separate PR.

Rebased again; I tweaked the test. <Foo rel="" /> must always be valid - this rule should only be checking HTML elements, never custom elements.
A few tests are still failing. @Nokel81?

Well since you just changed the logic or tests (cannot tell). I was whitelisting elements whereas I think you want me to blacklist html elements that aren't on the list.

All I did is move the two "Foo" tests from invalid to valid.

I'm fine with an inclusion-list-based approach, but it should ignore custom elements entirely.

@ljharb Sorry it took so long, this PR has been updated to ignore non-HTML tags.

@Nokel81 i enabled all the test cases and used the messageId pattern and rebased.


pip install brunel error

Hello -
I tried to pip install brunel but I get this error:
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-oxnvqc7e/brunel/
Please take a look into this issue and help out.
Thanks!
Albert.

I'm having trouble installing brunel with pip too. It just started today, after upgrading to pip 10.0.0.
$ pip install brunel
Collecting brunel
  Downloading https://files.pythonhosted.org/packages/5d/94/20ad0cb14c6ee0c873aa377b4edffc72b93c98221771e898eafb128433f3/brunel-2.3.tar.gz (2.1MB)
    .....
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/7l/psm5gvkj4c3bl4h83_kzz88r0000gn/T/pip-install-sybwjd/brunel/setup.py", line 30, in <module>
        pip.main(['install', 'jupyter-pip']); cmdclass = importlib.import_module('jupyterpip').cmdclass
    AttributeError: 'module' object has no attribute 'main'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/7l/psm5gvkj4c3bl4h83_kzz88r0000gn/T/pip-install-sybwjd/brunel/

https://github.com/Brunel-Visualization/Brunel/blob/937a26cae654607dfbb0b569ec19ded521a7649e/python/setup.py#L30
It looks like the 10.0.0 version of pip has removed the pip.main() function and buried it at pip._internal.main

@opalbert If your problem is the same as mine, you can temporarily work around the issue by explicitly installing jupyterpip manually before installing brunel
pip install jupyter-pip

@danfike Thank you for your suggestion, the installation almost worked except that I get this new error which seems to me I need to install Java/JDK:
"RuntimeError: No Java/JDK could be found. I looked in the following directories:
/usr/lib/jvm/jre-1.7.0
/usr/lib/jvm/jre-1.6.0
/usr/lib/jvm/jre-openjdk
/usr/lib/jvm/java-1.6.0-openjdk-<IP_ADDRESS>.x86_64
/usr/lib/jvm/jre
/usr/lib/jvm/jre-1.6.0-openjdk.x86_64
/usr/lib/jvm/jre-1.7.0-openjdk.x86_64
/usr/lib/jvm/java-1.7.0-openjdk-<IP_ADDRESS>.x86_64
/usr/lib/jvm/java-1.7.0-openjdk-<IP_ADDRESS>.x86_64

Please check that you have it installed.

If you have and the destination is not in the above list, please find out where your java's home is, set your JAVA_HOME environment variable to that path and retry the installation.
If this still fails please open a ticket or create a pull request with a fix on github:
https://github.com/tcalmant/jpype/

----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-tydkch7j/JPype1-py3/"

After installing pip install jupyter-pip on a new machine, pip install brunel works.
Thanks all.
This may be closed.


Redesigning budgeter workflow

Adding a progress bar to indicate progress through initial series of modals.
Considering alternatives to using modals.
Soliciting and incorporating feedback from users.

Adding buttons to return to each of the initial modals for bulk metadata input.

[ ] Using radio buttons for the type selection modal


Upgrade to node 16

What this PR does / Why we need it
Actions Ecosystem is requiring node 16 now, without updating their ecosystems actions.
this bumps node to 16 on the ecosystem action
Which issue(s) this PR fixes
Fixes #372

#372

Any chance of this being merged?


How do I pass an array of objects to stepsArray and manipulate it?

I had an requirement of sendingan rray of custom objects and show the labels and selection based on it. Anyway I can do it? when I send the objects everything is coming undefined.

Please provide a JSFiddle showing what yo uare trying to do. Also, stepsArray has examples on the demo site.

Hi,
To reiterate,
I am passing an array of objects to the stepsArray where the value is time. This works fine and I get the date as modelvalue. what I would like to get is the entire object itself on the onchange event.
thanks,
George

This is not possible, the onChange only receives the value item of the object.


Supply form to state/store using @caldera-labs/state #3

I think this is actually out of scope because this module should manage the state of the form it is being used to edit.


Unpin PyMC and PyMC-Experimental

Closes #332

:books: Documentation preview :books:: https://pymc-marketing--338.org.readthedocs.build/en/338/

See #339

This was mostly for the PyMC pin. I'll change after the tests pass just to see if anything else is broken

Ci is passing here and not in #339. Shall we merge and rebase #339?


Update built_value to >= 4 <= 6 ?

When trying to upgrade flutter to the latest alpha release I was not able to find a set of dependencies that worked out for me and one of the constraint's was the pinning of built_value to 4.x
Would it be possible to upgrade the dependency?
The breaking changes apparently are

Prohibit use of extends in built_value classes. Classes should inherit API using implements and API+implementation using extends Object with.
Prohibit use of show or as when importing 'package:built_value/built_value.dart'. The generated code needs access to all symbols in the package with no import prefix.
Prohibit use of the mutable collection types List, Set, Map, ListMultimap and SetMultimap. Suggest built_collection equivalents instead.

Not sure if this causes any issues with flutter_built_redux.
This would be great.
Also: thank you very much for the great project

merged charafau's update, getting it pushed to pub right now

Thanks for the heads up! Glad you like it


Inconsistent methods to fetch uri from config

Related to Addon: Navi Data
We have multiple ways within the addons to fetch the uri's from the config.
The metadata adapter looks at the first object in the array
https://github.com/yahoo/navi/blob/master/packages/data/addon/adapters/bard-metadata.js#L11
The facts adapter looks for a specific name if available to find the uri https://github.com/yahoo/navi/blob/master/packages/data/addon/adapters/bard-facts.js#L184
We should standardize the way the uri is fetched from the config.

This is a continuing part of multi data source refactor, it will be covered in the next PR for multi data source support as we are adding support for metadata


Cosmos DB: Rewrite RowConverter

This PR rewrites the CosmosRowConverter completely, adding support for:

InternalRow -> ObjectNode
Row -> ObjectNode
ObjectNode -> Row
ObjectNode -> InternalRow

And adding new unit tests (Spec) to cover the behavior on the converter.
Closes #18151
Closes #18150

awesome work. @ealsur
I see that you already added many new unit tests. that's great thanks.
Please make sure we add test coverage for every single type (nested, array, null, etc)

Thanks @moderakh ! Current test suite covers basic types (strings, numbers), date, time, map, struct, null, array, nested in both directions (Row to Json and Json to Row).

awesome work. @ealsur
I see that you already added many new unit tests. that's great thanks.
Please make sure we add test coverage for every single type (nested, array, null, etc)

Thanks @moderakh ! Current test suite covers basic types (strings, numbers), date, time, map, struct, null, array, nested in both directions (Row to Json and Json to Row).


Warn if/when deallocated

I just tracked an error when the Bridge instance was deallocated since I had not assigned it to a instance variable in my ViewController. I guess other might also fall into this trap (should it not be retained when setting itself as the delegate? Oh well) and so at least we can log the dealloc.

Fixed in 47cb1aee4a5ed9dc


Add analytics-panel to public dashboards

Add analytics panel to all public dashboards so we can get some usage
data to help us guide development planning.
Closes: https://github.com/giantswarm/roadmap/issues/379

Checklist

[ ] Update changelog in CHANGELOG.md in an end-user friendly language.

This depends on https://github.com/giantswarm/g8s-grafana/pull/326 so should only be merged after that one is (because without the plugin in grafana, dashboards which use the panel that it adds would show errors).

Found out that my ad blocker (uBlock origin) is blocking the request to the backend.

Interesting, did you ever find out why? Is it something to do with the word analytics in the URL? Kinda not ideal 😉


cargo fmt + GH workflow

If cargo fmt-ed code is desirable here, the first commit runs it initially, and the second provides a GH action to check cargo fmt, cargo doc and cargo clippy, which was extracted and modified from https://github.com/jonhoo/faktory-rs
Example failure and pass of the actions here

cargo clippy outputs some warnings for me but they don't show in ran action?
warning: direct implementation of `ToString`
   --> src/main.rs:94:1
    |
94  | / impl ToString for NodeAddress {
95  | |     fn to_string(&self) -> String {
96  | |         match &self.host {
97  | |             Host::Ipv4(ip) => format!("{}:{}", ip, self.port),
...   |
101 | |     }
102 | | }
    | |_^
    |
    = help: prefer implementing `Display` instead
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#to_string_trait_impl
    = note: `#[warn(clippy::to_string_trait_impl)]` on by default

warning: writing `&String` instead of `&str` involves a new object where a slice will do
   --> src/main.rs:958:16
    |
958 |     seed_name: &String,
    |                ^^^^^^^ help: change this to: `&str`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg
    = note: `#[warn(clippy::ptr_arg)]` on by default

warning: single-character string constant used as pattern
    --> src/main.rs:1044:50
     |
1044 |                     if !filter_label.starts_with("x") || filter_label.starts_with("x0") {
     |                                                  ^^^ help: consider using a `char`: `'x'`
     |
     = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_char_pattern
     = note: `#[warn(clippy::single_char_pattern)]` on by default

Yes, this is somewhat on purpose. I thought it could be a bit annoying to have all clippy lints fail CI, but this is possible to enable by using -Dwarnings flag with clippy.
In addition, this will only check diff for failures (which can also be configured), but I think that's more sensible?
Finally, there was also a bug -- reviewdog was trying to use "github checks" APi (whatever that is exactly), which requires a GitHub App or something to run. It's apparently "better", but sounds more annoying to me.
So currently I have it commenting >= warnings in my fork: https://github.com/willcl-ark/dnsseedrs/pull/1 but it still "passes" CI. I think this is a good combination, but can have it fail on warnings if perferred?

I think having it fail on warnings is fine. I don't know enough rust to be opinionated, and if there are linters that are, then that's all the better.

Ok done. Also, that reviewdog was a pain, so reverted to regular CI failures (i.e. check the logs) and simpler CI workflow file.
Example of failure without the included clippy commit here: https://github.com/willcl-ark/dnsseedrs/actions/runs/8411930712?pr=1

fwiw this doesn't check that the code actually builds
you'd need a cargo build step.

fwiw this doesn't check that the code actually builds
you'd need a cargo build

This was intentional -- it's just supposed to format/lint/doc. I usually see the build implicit to a cargo test job in CI, but as there are no tests here, perhaps it would be wise to add a build step...

I usually see the build implicit to a cargo test job in CI,

I'll need to find another free week to learn to write tests in rust.


Remove CredentialAttribute Type from left overs

Cleaning out commands that still had the CredentialAttribute() type added to credential parameters.
This does not allow double-hop situations, particularly when you have remote sessions in use.
It should not break anything but we will see 🤞

Why does that stinking CodeCov keep showing up? // @niphlod that is the only thing that caused this to fail.

@wsmelton : from "my" management panel on codecov no hooks are defined. I'll send a mail to their wonderful support to see what's going on


Refactored the JSOM.Provisioning App and also added success notifications

@vman, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Thank!


Stream directly out of momo

How do I live video stream, directly on players like VLC or any other player, without the support of a webpage? Is there any way to get only the video?

Discord での相談がないので閉じます。


Explore fifo for chroots to control what information the application gets

Is your feature request related to a problem? Please describe.
Xochitl skips SystemD by directly interacting with /sys/power/state.
Describe the solution you'd like
Some sort of ability to control how it's handled by using a FIFO or something similar to handle events. Could likely be expanded to more places than just /sys/power/state.
https://man7.org/linux/man-pages/man7/fifo.7.html

The main problem I seem to be running into is that all systemd communication (perhaps all dbus?) freezes up when a chroot is active.

And fixed. Never bind mount /sys. Instead do a sysfs mount.

Looks like controlling /sys/power/state works. Can't use this for evdev devices though.


Issue trying to connect Remix with Besu making use of RPC-HTTP connection

Description
I am trying to connect a Besu node with a Remix smart contract, making use of the Web3 Provider of Remix. I am following an example  provided in the Hyperledger Besu official website, but I am not able to make the connection.
I am making use of an AWS EC2 ubuntu instance and Docker. The docker compose file can be found attached, together with an screenshot proving that I am actually allowing AWS to listen on port 8545 and 8546, which correspond to RPC-HTTP and WebSocket connections.
When trying to connect to the port 8545 (I attach an screenshot of how I am doing so on Remix), I get an error message which I also attach below. I am checking the logs of the Besu container, but I don't get to see any actual logs corresponding to this connection. I am also attaching a screenshot of the starting logs of Besu, in which it can be seen that Besu is actually listen for RPC-HTTP connections on the port 8545.
Steps to Reproduce (Bug)

I am first running the docker-compose.yaml file on the VM machine in VSCode, making use of a Remote SSH connection to AWS.
Afterwards I go to remix, and I try to connect with the Besu node

Expected behavior: [What you expect to happen]
The expected behavior is that Remix would connect to the Besu node on port 8545.
Actual behavior: [What actually happens]
The actual behavior is that Remix is not able to connect to the Besu node, and I get no logs on the Besu docker container.
Frequency: [What percentage of the time does it occur?]
It happens all the time. I haven't been able to make the connection even once.
Versions (Add all that apply)

Software version:  Version 3
OS Name & Version:  Windows 10
Virtual Machine software & version: Ubuntu 18.04
Docker Version: latest
Cloud VM, type, size: AWS EC2 t2.medium instance.

I see you have added http://remix.ethereum.org to the cors but the recommendation is to add https://remix.ethereum.org - have you tried that?

@a905284 - did @macfarla rec fix your issue?
Closing for now, reopen if it's still an issue.

FWIW, I recently was using besu with remix and figured I would add some color to this issue.  I was able to connect to a local besu instance using the "Web3 provider" URL, but when it connects, remix does not have any accounts to use.  For this option remix is expecting a geth node which has a local account.  Besu does not have a concept of local accounts and it expects all transactions sent through it to be pre-signed.
The best way I found to use besu with remix is to use the "Injected Web3 Provider" option, with metamask.  I configured metamask to point to my local besu instance and it worked seamlessly (and allows you to use any private key you have configured with metamask)


Change GID in addition to UID for mounted volume

Thank you for creating this awesome tool!
Currently, ${BUILD_MOUNT_PATH} is created with group ID 0. I ran into an issue using this with a containerized build chain for embedded linux (crops/poky for the yocto project) and had to change the group ID to fix it.

LGTM, thanks for fixing this.


Filter checkboxes

Hello, I search to make a filter in a relation n,n (many to many) with checkboxes in order to have an OR filter
As it mentions here, it is possible since 0.6.2
https://blog.saltcorn.com/view/ShowPost/saltcorn-062---dynamic-menus-s3-storage-filter-checkboxes
But I can't find it neither in the Components nor in the wiki
https://wiki.saltcorn.com/view/ShowPage/flexible-filtering

Have you tried filter-button-group? It might help in simple cases...

Hello @Ingenu1ty, thanks for your suggestion, indeed it helped me for another case but not for many to many
Some details for my particular case :

Table structure

How it work with DropDownFilter

And here how it (not) work with FilterButtonGroup

As we can see in FilterButtonGroup the tagged.jeu->tag is not showing up and I want to filter on this field
I also try to create a FilterButtonGroup with tags table and tagged table but the filter don't work on my vue
As I want a OR filter the DropDownFilter can't be usefull and it's a pity that a can't choose the right field in FilterButtonGroup
I hope this can be fixed or maybe I just didn't understand how it works


Adding an option in Generate Code to include defaults arguments as imported from special.json

^

I'm actually planning on fully intergrating the special.json eventually but I've been putting it off for a while.

Some challenges with implementing this:

Not everyone's special.json is the same.

Can an unknown type determined to be an ENUM or a STRUCT?

Some default values consist of expressions:

Constants.
Enum values.
Logical/arithmetic expressions on constants.

For C++ the types and constsants have to be declared in the correct order.
Should this work with "use native types"?
Enums only assume the vanilla game, how should extra content be modeled for enums (e.g. model names)?

For TypeScript: ENUM_NAME | number
For Rust: num_enum's catch_all
For C++: ?
For C# (If relevant): ?

Actions to be completed before this can be implemented:

[ ] Extend https://github.com/DottieDot/gta-5-native-documentation-generator:

Remove unused declarations (but what about constants that should be used with natives?).
Sort for declaration order.
Model expressions.

Yes, speaking of the special.json generator, it worked well for most of the .sch files however when it hits an ifdef such as #IF IS_DEBUG_BUILD it breaks and skips to the next .sch

I'm not familiar with TS or Rust, but with the json schema implemented it should be relatively easy to control the input, and as far as extra content goes, I've found it easier to include everything regardless of game version.

I'm not familiar with TS or Rust, but with the json schema implemented it should be relatively easy to control the input, and as far as extra content goes, I've found it easier to include everything regardless of game version.

The problem is that this will break some enums and structs. For example, some enums will have something like this:
ENUM FOO
  // Fields...
#IF FLAG
  SOMETHING,
#ENDIF
#IF NOT FLAG
  UNUSED,
#ENDIF
  // More fields...
ENDENUM

Which is why this problem exists:

Yes, speaking of the special.json generator, it worked well for most of the .sch files however when it hits an ifdef such as #IF IS_DEBUG_BUILD it breaks and skips to the next .sch

Because making a proper for system for this is a good amount of effort.

and ignoring them would be a bandaid, I see, well considering that generating the json is on a user-by-user basis, would it be an idea to add an option to the generator for users to define which #IFs they want defined and the site handles it as it is?

would it be an idea to add an option to the generator for users to choose which #IFs they want defined and the site handles it as it is?

That's the thing that would be a good amount of effort, and since most people will only do this once it doesn't seem worth it to me. However, I'm considering making some other changes at which point I could possibly add it.


Annotated with optional meta

Hello I'm bumping my head trying to develop a custom Annotated class...
Annotated allows to add some metadata to type hint that can be checked at runtime:
from typing import Annotated

some: Annotated[int, 'must be even']

so metadata is always required - but I want to develop a similar type that initialises metadata with some default value:
some: MyAnnotated[int] # <- this must now be equal to Annotated[int, '<default-meta>']

I'm able to make it work with this code:
from typing import Generic, TypeVar, Annotated, Any

T = TypeVar('T')

class MyAnnotated(Generic[T]):
    @classmethod
    def __class_getitem__(cls, param: Any) -> T:
        if isinstance(param, tuple):
            return Annotated[param[0], param[1]]  # type: ignore
        return Annotated[param, '<default-meta>']  # type: ignore

assert MyAnnotated[int, 'my-meta'] == Annotated[int, 'my-meta']
assert MyAnnotated[int] == Annotated[int, '<default-meta>']

this works as expected - but VScode (pylance) does not understand it and cannot add autocompletion:

while works fine with default Annotated class:

is there a way to fool  pylance somehow ?
The high level idea is to make some type container that allows to set default:
T = TypeVar("T")
Vec: TypeAlias = Annotated[T, MaxLen(10)]

v1: Vec[int]              # <-- will have meta max-len = 10
v2: Vec[int, MaxLen(20))  # <-- will have meta max-len = 20 - but this example fails with validation error

Thanks for the issue.
I believe what you're asking for isn't possible. Pylance doesn't know that MyAnnotated is the same as Annotation because Annotation is treated specially. Internally Pylance (actually Pyright in this case) just treats 'Annotated' as the same thing as the first type argument and ignores everything else. It doesn't do any analysis of other classes to see if they're behaving like Annotated which is what I believe would be necessary here to get the completions to work.
I think your only resolution (if you want to do this in Pylance) is to :

Use Annotated everywhere and just provide the default arguments.
Use your custom MyAnnotated but then lose completions and type information from the point of view of Pylance.

You might try asking on the typing forums to see if there's perhaps another alternative to your MyAnnotated idea.
https://discuss.python.org/tag/typing
https://github.com/python/typing/discussions

Yes, what Rich said is correct. typing.Annotated is a special form that requires special handling inside of a type checker. You can't create your own version of Annotated with similar properties without extending the type system. I recommend building on Annotated rather than trying to replace it with something custom. For example, you could use some sentinel value to represent "default value".
some: Annotated[int, META_DEFAULT]


Publish CEPs on a website for easier linking (e.g. https://ceps.conda.org)

Checklist

[X] I added a descriptive title
[X] I searched open requests and couldn't find a duplicate

What is the idea?
It would be beneficial to linking to CEPs if it would be hosted under a domain like ceps.conda.org or similar. See https://peps.python.org for an example.
Why is this needed?
No response
What should happen?
No response
Additional Context
No response

See https://github.com/conda/conda-dot-org/pull/220 and https://deploy-preview-220--conda-dot-org.netlify.app/community/ceps


Replace all usage of wget with curl

See https://crbug.com/781538
wget is going away, so we should switch to using curl.  Fortunately, the crouton meta-installer uses curl, so this will be transparent to users.
I think there are only a couple uses of wget that happen outside of the chroot, so those definitely need to change, but we should probably change the uses inside the chroot as well for consistency.

the crouton installer fails hard due to 3514


Add UI to let system request particular examples

includes:

adding events, external functions to 'examples.js'
augmenting example objects with a start rule
modifying 'splitters.js' to allow splitting of smaller elements than body
moving css rules from index.html to their own files

Mostly looks good, just a few comments. Though when I tried it out, I saw a bunch of errors in the console:

Looks like maybe example.text is undefined on line 42 in exampleWorkerManager.js?

I'll fix that error in the next commit before the pull

merged manually in commit 7c0328d


Added example of from_tensors and from_tensor_slices

Added very important examples which differentiates from_tensors and from_tensor_slices in Dataset. The example describes the property of from_tensor_slices to merge the input tensor. In case of from_tensors, it doesn't merge input tensor.

@ashutosh1919 Thank you for your interest in improving the docs!
I'm confused about what you mean by "merge the input tensor". Both methods convert their input argument to a Tensor. The only difference is whether they produce the whole tensor, or slices of the tensor. In particular, dataset.from_tensor_slices(x) is functionally equivalent to dataset.from_tensors(x).unbatch(). Does that clarify how things work?

@ashutosh1919 Thank you for your interest in improving the docs!
I'm confused about what you mean by "merge the input tensor". Both methods convert their input argument to a Tensor. The only difference is whether they produce the whole tensor, or slices of the tensor. In particular, dataset.from_tensor_slices(x) is functionally equivalent to dataset.from_tensors(x).unbatch(). Does that clarify how things work?

@aaudiber , Thanks for the comment. Yes I understand that. Sorry for the language, I understand what it means but failed to describe it properly.

@ashutosh1919 Can you share what behavior you originally expected, and how that was different from the actual behavior? Then we can figure out how to modify the doc to address the misunderstanding better

@ashutosh1919 Can you share what behavior you originally expected, and how that was different from the actual behavior? Then we can figure out how to modify the doc to address the misunderstanding better

Sure. Please take a look at this answer's second part. I observed this behavior that from_tensor_slices slices the tensor along it's first dimension. But  from_tensors doesn't do that. Since, there was no example given in doc to explain that behavior difference in both, I have raised this PR.

@aaudiber , Thank you so much for clearing my doubts on this. I have fixed the comments and also changed the example as per your suggestions. I think now it makes more sense. Please review this one. Thanks again.

@aaudiber , Changed examples and comments so that readers can differentiate both methods by looking at the results we get in two examples.

@aaudiber , Removed section for from_tensor_slices and keeping other things as it is.

@ashutosh1919 thank you, it is failing doctest can you please check here for logs.
Please run the doctest locally as mentioned here in the contributor guidelines.

@gbaned , @mihaimaruseac and @aaudiber , Sorry for that. You need to approve it again. There was slight problem in the doc which was failing tests.


adding TV Shows to Restaurants results in broken activeRecipe

Trying to load [demo] TV shows fails and outputs the following in the console
synthetic-storage.js:132 Error parsing manifest at synthetic://arc/handles/firebase://arcs-storage.firebaseio.com/AIzaSyBme42moeI-2k8WgXh-6YK_wYyjEXo4Oz8/0_6_0/chenbrian--LVyQm3hfYSNO7wjCA88:
Post-parse processing error caused by 'undefined' line 401.
Hosted particle 'TVMazeShowTile' does not match interface 'hostedParticle'
      hostedParticle = TVMazeShowTile

Repro steps:

Visit http://localhost:8786/shells/web-shell/index.html as a newly created user.
Enter * in the search bar.
Try clicking on [demo] TV show from the search list.

Thanks @thebrianchen, I got the repro now. It looks like something weird in how Restaurants and TV recipes fit together.

Here is one of the problematic serializations:
restaurant_tv.txt
There are a lot of duplicate definitions there, maybe due to #2121? Also, figuring out what is what is complicated by the generic naming, which I'm noting just as backup motivation for #2607 and #2578.

TVMazeShowTile which should contain a type variable is serialized with 2 different concrete types:
interface HostedTileInterface
  in TVMazeShow {Text showid, Text name, Text description, Text image, Text network, Text day, Text time, Text suggestion, Text extra, Boolean favorite, Boolean delete} *
  consume tile

interface HostedTileInterface
  in Restaurant {Text id, Text identifier, Text reference, Text name, Text description, Text icon, Text photos, Text photo, Number rating, Text address} *
  consume tile

Investigating.

@Cypher1 Will work on it next, we decided it's P1.

Reassigning for Piotr to check as Maria may have already fixed this.

Type variables from interface types get entangled with type variables from recipes. As recipes get type-checked, we assign resolutions to type variables. When we then serialize interface types, we print out type resolutions instead of abstract type variables. This is a bug, we should disregard type variable resolution when serializing, which BTW we do already for serializing particle specs.
I'm working on a fix now.


[Bugfix] Pass PYTHONPATH from setup.py to CMake

In clean build environments such as cibuildwheel, build dependencies are installed in a virtual environment located at /tmp/pip-build-env-xxx. This path is in the PYTHONPATH of the setup.py file, but it is not passed to CMake and subsequent Python calls. This PR fixes this behavior by passing the PYTHONPATH from setup.py to CMake, and finally to the machete/generate.py script (which uses jinja2 from build dependencies).

/ready


reduced genome, Bowtie command clarification.

Hi, could you please explain why we need a reduced reference genome?
Is it not possible to map the 4C to the whole reference genome and then proceed?
Also, at your bowtie2 command you trim the first 26 bases, I expect that those are the adapter and RE that needs to be removed. So depending on the experiment those could vary.

Since expect the ligated interactions to always be adjacent to a RE site we map the reads to a reduced genome and gets rid of any spurious ligation events. The bowtie2 command is specific for the example and yes it should be changed according to your samples.

thank you for your answer, so you clean up your mapped reads in a more efficient way (mapping tools) than using R.
But I got a follow up. I've noticed that using >50 nt length for mapping is inefficient and sometimes when usable reads shorted inadequate. I am using instead bowie v (1.2.0) or bwa aln (>0.6). Overall mappability is greater (especially when read lenght is < 30 nt). So, maybe it would be favorable to add those options?

Thanks for the suggestion - I will look into these alternative methods for mapping.


[question] i dont find complete learning video

your question
ask your question here. be as specific as possible.
context
add any relevant context that might help us answer your question.

ask your question here. be as specific as possible.


googleapi error 400 after passing list with -q (query)

To get the folder by name, I've always used:
gdrive list -q "query"
example:
gdrive list -q "title = \"11_BLOG\""
I just compiled gdrive on my fresh El Capitain, and I'm getting:
wk:repos/me/scripts> ~/Downloads/gdrive list -q "title = \"11_BLOG\""
Failed to list files: googleapi: Error 400: Invalid Value, invalid

I tried to get pre-built binary, but the result is the same. Did something change?

Yeah, the api changed, the title field is now called name. Overview can be found here: https://developers.google.com/drive/search-parameters


re-write trim adapters

use functions
make it faster
buffered writes
remove single mismatch constraint

Here's a cleaned up version that I wrote a while ago:
https://github.com/nboley/regulatory_network_tools/blob/master/scripts/trim_ATACseq_reads.py

Nathan - did you have a way of evaluating the improvement of your trimmer?
We are considering replacing this trimmer with a published equivalent (
https://github.com/jhhung/PEAT)
On Sun, May 15, 2016 at 3:48 PM, Nathan Boley<EMAIL_ADDRESS>wrote:

use functions
buffered writes
remove single mismatch constraint

Here's a cleaned up version that I wrote a while ago:
https://github.com/nboley/regulatory_network_tools/blob/master/scripts/trim_ATACseq_reads.py
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/kundajelab/bds_atac/issues/9

The only advantages are that it is much faster, allowed for more than 1 mismatch, and the code is cleaner. If PEAT works then I would say go with that.


[BUG] The value for one of the HTTP headers is not in the correct format. x-ms-version: 2021-04-10

Library name and version
Azure.Storage.Blobs 12.11.0
Describe the bug
Upgrading dependency from 12.10.0 to 12.11.0 to get the new implicit dependency on System.Text.Encodings.Web 4.7.2, causes failures in tests.
Expected behavior
The requests to be successful. Especially given that 2021-04-10 is the recommended current version https://docs.microsoft.com/en-us/rest/api/storageservices/previous-azure-storage-service-versions.
Actual behavior
  Error Message:
   Azure.RequestFailedException : The value for one of the HTTP headers is not in the correct format.
RequestId:19911bc0-5a01-4bef-8acc-1195c629319d
Time:2022-03-22T00:52:32.621Z
Status: 400 (The value for one of the HTTP headers is not in the correct format.)
ErrorCode: InvalidHeaderValue

Additional Information:
HeaderName: x-ms-version
HeaderValue: 2021-04-10

Reproduction Steps
Code causing the failure:
internal BlobContainerClient CreateBlobClient() => new("UseDevelopmentStorage=true", this.containerName);

Environment
No response

I had the same issue. Had to revert to 12.10.0

@JoeRobich I am not able to reproduce this issue and it BlobClient is working fine while returning 2021-04-10

Could you please provide more details and code snippet if possible to reproduce.
Thanks
Saurabh

@rossdargan could you provide the RequestId like the original person did in their post?

Thanks for investigating. Looks like they have a bug open for this problem (https://github.com/Azure/Azurite/issues/1404) and a possible fix in the works (https://github.com/Azure/Azurite/pull/1393).


nixos/healthchecks: add settingsFile option

Single unified env file read by systemd
Description of changes
Current settings option requires multiple files if you want to set multiple environment secrets, as well as being read by the service program, meaning that things like agenix won't work with the default service user.
Added a settingsFile option that allows a single secret file, and doesn't break any current behavior for existing settings options, and plays well with agenix. Env variables can be set in both settings and settingsFile.
Tested both the webservice, and the healtchecks-manage script created by the module.

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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

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

It should be noted that pr author originally created https://github.com/NixOS/nixpkgs/pull/314173, but wanted to rename the branch as stated in his last comment there

Previously the module would have you set an env variable that would point to a file for each secret you wanted to set, and the program would read the secret file rather than systemd.
Only one file is required. The settingsFile option I can handle all secret requirements just fine, as it is using the systemd environmentFile directly.
But as to not to make any breaking changes I left the previous env variable options as is.


Tooltip about poverty

Add a tooltip explaining the geography and maybe methods behind the "poverty" estimate (ACS 2015 at PUMA level)

Solved with link to Census poverty methodology in description.


WFLY-18341 Restore incorrectly updated copyright dates in Jipijapa

This reverts incorrect changes from
096a516e745663a99fce0062ff4bb93a4ca1066f: WFLY-18197 Upgrade to Hibernate Search 6.2.0.CR1
https://issues.redhat.com/browse/WFLY-18341
See also https://github.com/jbossas/jboss-eap8/pull/143#issuecomment-1674803759

@yrodiere I've started a discussion thread here: https://wildfly.zulipchat.com/#narrow/stream/174184-wildfly-developers/topic/GH.20wildfly-bot
The current rule intended for PR descriptions seems to be getting applied to commit messages.

Thanks @yrodiere


ResponseOf.java:44-46: Servlet request and response...

The puzzle 682-06e5396c from #682 has to be resolved:
https://github.com/yegor256/takes/blob/87f8ee3a80cb27136838f640677b1890f5adda79/src/main/java/org/takes/servlet/ResponseOf.java#L44-L46
The puzzle was created by g4s8 on 09-Apr-18.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@paulodamaso/z please, pay attention to this issue

Job #866 is now in scope, role is DEV

The job #866 assigned to @ypshenychka/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@0crat refuse

@0crat refuse (here)

@ypshenychka The user @ypshenychka/z resigned from #866, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @ypshenychka/z

The job #866 assigned to @olenagerasimova/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@olenagerasimova/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat refuse (here)

@olenagerasimova The user @olenagerasimova/z resigned from #866, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @olenagerasimova/z

The job #866 assigned to @robsonataide/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be a monetary reward for this job

@robsonataide/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

The user @robsonataide/z resigned from #866, please stop working. Reason for job resignation: It is older than 10 days, see §8

Resigned on delay, see §8: -30 point(s) just awarded to @robsonataide/z

@0crat assign @fabriciofx

@0crat assign @fabriciofx (here)

@paulodamaso The job #866 assigned to @fabriciofx/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @paulodamaso/z

The user @fabriciofx/z resigned from #866, please stop working. Reason for job resignation: It is older than 10 days, see §8

Resigned on delay, see §8: -30 point(s) just awarded to @fabriciofx/z

@0crat assign @fabriciofx

@0crat assign @fabriciofx (here)

@paulodamaso The job #866 assigned to @fabriciofx/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @paulodamaso/z


Implement per-operation options for storage standalone functions

The storage library has a few standalone functions that refactor common workflows making multiple calls to the storage::Client class. For these classes we need a different implementation of the per-operation options.  Consider, for example, ComposeMany().  Basically it does something like this:
template <typename Options>
StatusOr<ObjectMetadata> ComposeMany(
      storage::Client client, std::vector<std::string> object_names, Options&&... options) {
  auto c1 = client.Compose(/* some of the objects from `object_names */, std::forward<Options>(options));
  auto c2 = client.Compose(/* some of the objects from `object_names */, std::forward<Options>(options));
....
  auto cN = client.Compose(/* some of the objects from `object_names */, std::forward<Options>(options));

  return client.Compose(c1 .... cN, std::forward<Options>(options));
}

The problem is that any google::cloud::Options hidden in the parameter pack may get moved-copied in the first call creating c1.  The remaining calls get the wrong overrides.  We need to do something like:
template <typename Options>
StatusOr<ObjectMetadata> ComposeMany(
      storage::Client client, std::vector<std::string> object_names, Options&&... options) {
  auto const grouped = storage::internal::GroupOptions(std::forward<Options>(options)...);
  auto c1 = client.Compose(/* some of the objects from `object_names */, std::forward<Options>(options)..., grouped);
  auto c2 = client.Compose(/* some of the objects from `object_names */, std::forward<Options>(options)..., grouped);
....
  auto cN = client.Compose(/* some of the objects from `object_names */, std::forward<Options>(options)..., grouped);

  return client.Compose(c1 .... cN, std::forward<Options>(options)..., grouped);
}

I think the relevant functions are:

DeleteByPrefix()
ComposeMany()
ParallelUploadFile()

These functions have many helpers in the internal namespace, but I think we can ignore those?

Evidently we don't have time to work on this. To the backlog.


All stages ux updates pat

I believe Stages 1 & 2 and their intros should be more or less fully functional on a clean single pass through (i.e., as long as user doesn't log out and log back in or do anything crazy), except:

See mixed up Stage 2 marker issue #36. @Carifio24, can you please help fix this before we deploy it this afternoon?
Stage 2 doesn't automatically link to Stage 3, but I can fix that in the morning.
There was a merge conflict from PR #30. I wasn't sure which should take precedence, so I accepted both and nothing has broken yet. Please fix if I've done that incorrectly.

Closing this PR because it only has 3 commits. See instead PR #39. (I renamed this branch part way in, and it seems like it only sent the commits from prior to the name change).


Not able to run Mocha tests with node-schedule npm module

I've setup a suite of tests in mocha using Sinon and Chai.
What I'm aiming for is that the tests will run daily and then email the results to my team.
So I installed below npm modules,
--> node-schedule: To run tests daily
--> mocha-runner-reporter: To send the test report as email.
The tests are all running and sending email as expected only first time. From second time on wards it's neither running the tests nor sending email. I have attached the sample project as well.
i'd appreciate any suggestions.
Samplecode.zip

I'm not familiar with mocha-runner-reporter, but if it uses Mocha's programmatic API and isn't doing anything to work around Mocha's known issues with running the programmatic API multiple times then you might have to apply some of those workarounds yourself if possible (e.g. see if it works if you clear the require cache before each run). Let me know if that works for you!

@ScottFreeCode Thank you for the reply. I have tried all workarounds mentioned in above comment. This work around solved my problem.

Awesome! Glad that worked.
I'm going to go ahead and close this since we have those other issues for the same problem already, but feel free to participate there if you have any new ideas, suggestions or questions!

@manikumar446 Could you post your updated or modified sample code if possible. Still facing the issue.


fix loading WorkGraph with PythonJob

When loading a WorkGraph, it loads the orignal Python data from the AiiDA Data node for the args and kwargs of the function.

Codecov Report
Attention: Patch coverage is 80.00000% with 5 lines in your changes missing coverage. Please review.

Project coverage is 75.19%. Comparing base (e0c448f) to head (706782a).

Files
Patch %
Lines

aiida_workgraph/utils/__init__.py
75.00%
4 Missing :warning:

tests/test_python.py
88.88%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #172      +/-   ##
==========================================
- Coverage   75.25%   75.19%   -0.07%
==========================================
  Files          69       69
  Lines        4591     4616      +25
==========================================
+ Hits         3455     3471      +16
- Misses       1136     1145       +9

Flag
Coverage Δ

python-3.11
75.19% <80.00%> (+0.02%)
:arrow_up:

python-3.9
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


disable download when result is 0 alerts

I don't think this is an issue anymore as we've got cartodb integration and subscriptions. Reopen in case I'm wrong, though!


[NEW][WIP] Add Event login Components

This PR adds a component for sign-up sign-in with the event.
Closes: RocketChat/RC4Conferences#2
Thank you!

Closing.   Will never be in this repository.


What does the "majorVersion" refer to?

Hello,
What does the "majorVersion" value in the extension bundle refer to?
Does this value indicate any change to the binding itself?
Example:

If I have an Azure Function that uses a single input binding of "queue", is there any change to the performance of the queue binding whether I'm using extension bundle version [2.*, 3.0.0) or [3.*, 4.0.0)?

@ericthomas1 -

Extension release 2.x has major version 4 of Storage package, which means 4.x Nuget package for Microsoft.Azure.WebJobs.Extensions.Storage got downloaded - https://github.com/Azure/azure-functions-extension-bundles/blob/ee664bf4f0572bab10d2686730d3b3ff697249cf/src/Microsoft.Azure.Functions.ExtensionBundle/extensions.json#L4

Extension release 3.x and release 4.x has major version 5 of storage packages, which means latest 5.x (GA version) will be pulled for Microsoft.Azure.Functions.Worker.Extensions.Storage.Queues at the time of extension bundle download.
https://github.com/Azure/azure-functions-extension-bundles/blob/55829628c31333630e2a82fb3399022107021eba/src/Microsoft.Azure.Functions.ExtensionBundle/extensions.json#L23
https://github.com/Azure/azure-functions-extension-bundles/blob/16124263a8a6a6d9f73819a8da5da3fefb71e283/src/Microsoft.Azure.Functions.ExtensionBundle/extensions.json#L15

For details, you can refer the document specific to Azure Queue storage trigger and bindings for Azure Functions overview which details difference between 5.x and previous versions of storage packages and changes that have been introduced.

@BigMorty , @vrdmr , Another example of how documentation confuses the users between extension versions and bundles version.
Another such example - #391


Code for video visualization

Thanks for your instruction for the demo. But the demo can only generate an SMPLX model for each frame, can you publish the code which generates the video visualization of your method on your project page?

Thanks for your instruction for the demo. But the demo can only generate an SMPLX model for each frame, can you publish the code which generates the video visualization of your method on your project page?

Hi azuki-miho, have you figured out how to show this with video?


Simulate a "Garage Door" or "Gate" HK module with multiple ZWAVE devices

Description
It would be great to create a way to emulate a HK Garage Door Opener with in Homebridge using :

a module to Open / Close (Single push button, or dual buttons)
sensors to define the door states (Open / Closed)

I didn't find the way to do it with the way the plugin works.
Thank you for your help.
Logs
No response
Plugin config
No response
Plugin version
2.1.3
Fibaro HC / Yubii Home type and firmware
Fibaro HC2

If I understand corectly you want to combine door sensor and e.g. smart implant (or other device that closes for while two cables to give a signal for the gate) into one device that will create garage door in homekit - if door sensor is open then garage door is open (simulate that garage open).

Workaround:
You can create a virtual device, such as a garage door, in some homebridge plugin. You will create automation - when the door sensor is open open virtual gate. And the second automation that when the sensor is closed, the gate is closed. The third automation when the virtual gateway is controlled run this smart implant or orher device that control gate.

I will try that, but don't you think the automation from the sensor to open/close Virtual Gate will activate the device to control the gate ?

I will try that, but don't you think the automation from the sensor to open/close Virtual Gate will activate the device to control the gate ?

Ah.. Yes, sorry, my bad.

I found that, but I unfortunately don't have a HC3. But I think it would answer my need...
https://github.com/Strik777/homebridge-fibaro-hc3-garagedoor

That's it, and if I understand well, the sensor module will trigger the state Open / Close of the garage door (which is different from switching the open/close command done by the "switch" module ).

That's it, and if I understand well, the sensor module will trigger the state Open / Close of the garage door (which is different from switching the open/close command done by the "switch" module ).

In order for this solution to work properly, the created virtual garage door must:

take the state from the ocupancy sensor, but not fire this virtual garage door when state changes.
on every change (open/close) run the controller that is connected to the gate (smart implant, relay, or something similar). This controller wire the cables for a moment and after, for example, 1 second it unwire.

That's it, and if I understand well, the sensor module will trigger the state Open / Close of the garage door (which is different from switching the open/close command done by the "switch" module ).

In order for this solution to work properly, the created virtual garage door must:

take the state from the ocupancy sensor, but not fire this virtual garage door when state changes.
on every change (open/close) run the controller that is connected to the gate (smart implant, relay, or something similar). This controller wire the cables for a moment and after, for example, 1 second it unwire.

That's it, and if I understand well, the sensor module will trigger the state Open / Close of the garage door (which is different from switching the open/close command done by the "switch" module ).

In order for this solution to work properly, the created virtual garage door must:

take the state from the ocupancy sensor, but not fire this virtual garage door when state changes.
on every change (open/close) run the controller that is connected to the gate (smart implant, relay, or something similar). This controller wire the cables for a moment and after, for example, 1 second it unwire.

Correct.
I imagine something litke that :

when you push the "OPEN GATE" button in Homekit it triggers the "switch" to open the door/gate,
the gate will start to open, causing the "CLOSE sensor" to change state, and you should see "OPENING" state in Home app.
until the "OPEN sensor" is triggered --> the state changes to "OPEN" in Home app.
when you push the "CLOSE GATE" button it triggers the "switch" module to close the gate,
the gate start to close, causing the "OPEN sensor" to change its state, and you see "CLOSING" state in Home app.
until the "CLOSE sensor" is triggered, when the state changes to "CLOSED" in Home app.

The difficulty is usually gates or garage door can be opened/closed by a single push button style switch. In this case, the "OPENING" and "CLOSING" state can be emulated with configured delayed.

Hmm... Everything is correct. Only this plan has one fundamental disadvantage - both: the door window sensor and the smart implant / relay must be from Fibaro, or at least zwave connected to the Fibaro hub (home center, yubi). Why does I write about it? I currently don't have any door / window sensor, and if I had to buy it, I would choose one with Matter over Thread and connect it directly to HomeKit. In addition, it is 2x cheaper.


fisher ls-remote hides installed packages

Actual:
$ fisher ls-remote
foo bar baz

$ fisher install foo

$ fisher ls-remote
bar baz

Expected:
$ fisher ls-remote
foo bar baz

$ fisher install foo

$ fisher ls-remote
foo bar baz

There's currently no way to show all available packages, including those already installed. It may be reasonable to hide this information from the user, but seems less reasonable to be completely inaccessible. Perhaps different behaviour when STDOUT is a TTY, and when it is not?
There's reasonable cases to show or hide this, so it might make sense to have a pair of flags to control this behaviour?
fisher ls-remote --show-installed would force installed packaged to be displayed.
fisher ls-remote --hide-installed would force them to be hidden.

There's currently no way to show all available packages, including those already installed.

After #249 lands,
begin
fisher ls
fisher ls-remote
end | cat

will do it.

@fisherman/all Thoughts?
Ping @jethrokuan @herrbischoff @orangecms @lordlycastle @Markcial @MadcapJake

Personally i would like to use the full color support to display what's going on on certain packages. So the installed packages would have orange color, the packages not yet installed would be green, the installed ones, blue. If the color support is not enabled then symbols would be appended/prepended to the packages.
Anyways it is true that ls-remote should not hide already installed packages if they are remotely available, let's take as example another package managers as yum, apt o pacman, they list available packages even the installed ones, but mark them with a letter or symbol so user notices that that packages are already installed.

My logic is clearly expressed in the README.
See what's installed.
fisher ls

See what you can install.
fisher ls-remote

following that kind of logic we might end up with a new action for listing the packages that can be updated.
I guess that listing packages that are available remotely includes the ones already installed, but maybe not as the ones that are not installed.

That is the logic as it is written and described in the README since the feature was implemented and I don't see what's wrong with it.
And no, there will be no more new actions, there could be an internal __fisher_* like function, but I'd rather land #249 and then open the door to https://github.com/fisherman/fisherman/issues/250#issuecomment-216631163.
I am not very fond of --show-installed or --hide-installed either.

@Markcial, @colstrom: what use do you two see in showing all packages (included installed)? "Other package managers do it" doesn't really explain what it's useful for.

@Markcial: Personally i would like to use the full color support to display

fisher ls-remote --format=string

You can embed colors in that string and get the output exactly the way you want it, that's why the format string exists. See: https://github.com/fisherman/fisherman/releases/tag/2.2.0

I'd also expect ls-remote to show everything and not just what's not installed. Otherwise it is misleading and I will not see if e.g. a package is removed. That is what I would understand now from the result. If I have something installed and ls-remote doesn't show it, it means that it cannot be installed, so it doesn't exist anymore. And yes, other package/pkugin managers wouldn't do this. They would rather show more verbose information and tell me in the list if a package is already installed, can be updated, etc.. Debian's dpkg -l for example even tells me if a package has once been installed, if it is installed now, and if I can just install it.

I will implement #249 which is somewhat related to this ASAP. But since I don't have a solid use case for #250, this is unlikely to be implemented any time soon. I have added the PR WELCOME, so if someone wants to take a stab at it, they are more than welcome to do so. Also make sure to update the documentation and propose the CLI changes in advance.

@MadcapJake "Other package managers do it" isn't a reason in itself, but it is a strong indicator of what a user is likely to expect. I found this behaviour to be very surprising, but I'm hardly a representative sample. ;p

I can filter data I don't want.
I cannot conjure data isn't there.
This omission is deliberate, and silent.
There is no mechanism to override this.

@bucaran I understand that the README says it works a certain way, though that feels a bit like circular reasoning: It's documented because that's what it does. It does that because that's what the docs say.
The way this is current implemented appears to have more limitations than benefits.

It would be nice to be able to format remote packages based on their installed status.
It would be nice to query ls-remote to see if a package name is currently in use.
With a full list of packages, I could conveniently bulk-clone or update all the current packages, for offline use (I commute via train, and cell service along the route is spotty at best).
The list could be used to present the packages in different ways (sorted by GitHub stars, for instance).

The point is that we can do all sorts of things with information we have, but miss out on that when the data is deliberately discarded. In my view, this is a case where fisherman assumes too much about how it will be used, resulting in default behaviour that is both counterintuitive, and counterproductive.

So, what is your use case again?

#257

From #257:

[x] Do not hide installed plugins in fisher ls-remote

I read some part of your long discussion on slack, but I missed the moment when @colstrom convinced @bucaran to make this decision. What's the reason? (I'm just curious)

Well, in a way, you convinced me with https://github.com/fisherman/fisherman/issues/256
If we are going to use the index as the source of completions, there is no longer a need to hide installed plugins in ls-remote, since that will be the default behavior when auto completing the pager.


[gdk-pixbuf] Add ipv6 source code source

Add ipv6 source code source.

More reliable servers all the things! Thanks!


[Update Request]: Splitting Discord.Discord into two packages for each architecture

What type of update are you requesting?
A new locale, architecture, or installer type of an existing version
Current Package Identifier
Discord.Discord
Package Version
1.0.9030
Please describe the changes you would like to see
Would it be possible to split Discord into two packages: Discord.Discord.x86 & Discord.Discord.x64
The current system keeps showing me updates for x86 Discord when i have x64 Discord installed.

@pckgrbot - Looks like your automation isn't pulling in the x64 URLs

@PckgrBot - Looks like your automation isn't pulling in the x64 URLs

I think I saw somewhere that they have different behavior and they need to be split up.

@PckgrBot - Looks like your automation isn't pulling in the x64 URLs

https://github.com/microsoft/winget-pkgs/issues/137157

Because Discord uses the built-in auto-updater and upgrading through WinGet causes issues with Discord, this set of PRs should fix the issue - https://github.com/microsoft/winget-pkgs/pulls?q=is%3Apr+"Discord"+"Deny"

Discord.Discord - does self-updates and does not update the version number in the registry when doing so, applying an update via winget will break/override the user preferences and forcefully enable auto-start on system boot #90642

This has been added to Review.csv.

Discord.Discord - does self-updates and does not update the version number in the registry when doing so, applying an update via winget will break/override the user preferences and forcefully enable auto-start on system boot #90642

This has been added to Review.csv.

Can you describe the check?
To be clear, new versions of Discord.Discord can be added to the repo, especially if a user wants to do a fresh install or winget install --force, it's jut that the UpgradeBehavior: deny should be set

Proposed separation of package.
New manifests attached in zip file.
Discord.zip

The current system keeps showing me updates for x86 Discord when i have x64 Discord installed.

If this is right then when x64 is installed, it should update to x86 newer version automatically.
I tested this by installing x64 from manifest and it did not upgrade.
Previously it only showed winget upgrade because both architecture were using same ID & x86 had higher version number.

If you are right then when x64 is installed, it should update to x86 newer version automatically. I tested this by installing x64 from manifest and it did not upgrade. Previously it only showed winget upgrade because both architecture were using same ID & x86 had higher version number.

Thats not quite what I meant. Discord itself is not going to detect updates incorrectly - internally it knows which architecture it is running.
The issue is that winget will have no way to know which version a user has installed. If you look at the information Discord writes to the registry, the DisplayName, Publisher, and ProductCode are exactly the same between the x86 and x64 versions. This is why they can be in the same manifest without requiring AppsAndFeaturesEntries. If the two manifests are split, winget may not be able to determine which package is actually installed since both Discord.Discord.x86 and Discord.Discord.x64 will be a strong match to the data in the registry.

If you are right then when x64 is installed, it should update to x86 newer version automatically. I tested this by installing x64 from manifest and it did not upgrade. Previously it only showed winget upgrade because both architecture were using same ID & x86 had higher version number.

Thats not quite what I meant. Discord itself is not going to detect updates incorrectly - internally it knows which architecture it is running.
The issue is that winget will have no way to know which version a user has installed. If you look at the information Discord writes to the registry, the DisplayName, Publisher, and ProductCode are exactly the same between the x86 and x64 versions. This is why they can be in the same manifest without requiring AppsAndFeaturesEntries. If the two manifests are split, winget may not be able to determine which package is actually installed since both Discord.Discord.x86 and Discord.Discord.x64 will be a strong match to the data in the registry.

Ok. But this way atleast user has a choice which architecture to install. Instead of just installing latest x86 from winget (because x86 has higher version number which was selected by default).

Thats what --architecture x64 is for

Requesting manifest for HostIdentifier { name: APP, release_channel: STABLE, platform: WIN, arch: X64 }
Is there any way to check how the HostIdentifier is defined in registry. Maybe winget can use that to identify installation.

Requesting manifest for HostIdentifier { name: APP, release_channel: STABLE, platform: WIN, arch: X64 }
Is there any way to check how the HostIdentifier is defined in registry. Maybe winget can use that to identify installation.

Winget can't implement special behavior for every app, and it would need to be built into the CLI to perform that type of check. If Discord changed their installer to write the architecture as part of the name or product code, then it would be easy, but they don’t

Alternatively remove x86 version entirely. Discord is gamer focused & all modern gaming systems are x64 only.
Windows 11 doesn't even ship x86.

Due to the difference in version numbering, so long as each version manifest is configured correctly (i.e. specifying either x86 or x64, but not both), the app should be upgraded as expected.

The 32-bit one is no longer supported: https://support.discord.com/hc/en-us/articles/17997797368471--Known-Issue-Support-for-32-bit-Windows-Operating-Systems

The 32-bit one is no longer supported: https://support.discord.com/hc/en-us/articles/17997797368471--Known-Issue-Support-for-32-bit-Windows-Operating-Systems

So then only the 1.0.91xx versions need to/will be added going forward in winget?
Discord seems to split x86 -> 1.0.90xx and x64 -> 1.0.91xx.

The 32-bit one is no longer supported: support.discord.com/hc/en-us/articles/17997797368471--Known-Issue-Support-for-32-bit-Windows-Operating-Systems

So then only the 1.0.91xx versions need to/will be added going forward in winget?
Discord seems to split x86 -> 1.0.90xx and x64 -> 1.0.91xx.

https://dl.discordapp.net/distro/app/stable/win/x64/1.0.9040/DiscordSetup.exe

The 32-bit one is no longer supported: support.discord.com/hc/en-us/articles/17997797368471--Known-Issue-Support-for-32-bit-Windows-Operating-Systems

So then only the 1.0.91xx versions need to/will be added going forward in winget?
Discord seems to split x86 -> 1.0.90xx and x64 -> 1.0.91xx.

https://dl.discordapp.net/distro/app/stable/win/x64/1.0.9040/DiscordSetup.exe

Yeah, for older versions. It seems around the time of the 32-bit deprecation they changed to using 1.0.91xx for new 64-bit builds. Leaving x86 on 90xx.

The 32-bit one is no longer supported: support.discord.com/hc/en-us/articles/17997797368471--Known-Issue-Support-for-32-bit-Windows-Operating-Systems

So then only the 1.0.91xx versions need to/will be added going forward in winget?
Discord seems to split x86 -> 1.0.90xx and x64 -> 1.0.91xx.

https://dl.discordapp.net/distro/app/stable/win/x64/1.0.9040/DiscordSetup.exe

Yeah, for older versions. It seems around the time of the 32-bit deprecation they changed to using 1.0.91xx for new 64-bit builds. Leaving x86 on 90xx.

The x0xx and x1xx are just version indicators used in Discord's numbering convention.  E.g. 1.0.9050 and 1.0.9150 are the same version, just compiled for x86 and x64 respectively.
If it's helpful, you could breakdown Discord's numbering system as version-platform-revision. E.g. 1.0.9150 coincides with version 1.0.9 - x64 - revision 50.

Weird. Even if the 32-bit one has been EOL'd, the publisher is still publishing new releases for it in the last two weeks.


install with NPM

Please create a NPM package so it can be install using
npm install angular-bootstrap-multiselect --save

See  npm-create or create npm package.

Not an official source, but here you go:
https://www.npmjs.com/package/angular-bootstrap-multiselect

Thanks Amit :-)
On Tue, Jun 14, 2016 at 12:28 PM Amit Moryossef<EMAIL_ADDRESS>wrote:

Not an official source, but here you go:
https://www.npmjs.com/package/angular-bootstrap-multiselect
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bentorfs/angular-bootstrap-multiselect/issues/9#issuecomment-225842009,
or mute the thread
https://github.com/notifications/unsubscribe/AGFQ_MJW4qRjCtlYfeIivNuVsgook7vIks5qLoJigaJpZM4Gj43R
.

No problem
You can close if you want

Hi @AmitMY I tried to update the npm package to the latest version to fix #30 but I couldn't update it as you're the npm package owner. Could you please update it to the latest version?

Done.
@bentorfs Can I add you to the list of "collaborators" on npm, such that you will be able to publish? (what is your npm username?)
Such a simple process:

git clone https://github.com/bentorfs/angular-bootstrap-multiselect.git
cd angular-bootstrap-multiselect/
npm publish

My username is 'bentorfs' (In fact I only made it just for this purpose).
I suggest that you add Stanislava as collaborator as well. He's taking care
of business here lately :-)
Cheers,
Ben
On Thu, Dec 22, 2016 at 6:13 PM Amit Moryossef<EMAIL_ADDRESS>wrote:

Done.
@bentorfs https://github.com/bentorfs Can I add you to the list of
"collaborators" on npm, such that you will be able to publish? (what is
your npm username?)
Such a simple process:

git clone
https://github.com/bentorfs/angular-bootstrap-multiselect.git
cd angular-bootstrap-multiselect/
npm publish

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bentorfs/angular-bootstrap-multiselect/issues/9#issuecomment-268845696,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGFQ_PliBpn-7-No9gbFSX2Y5J_WgZe8ks5rKq-sgaJpZM4Gj43R
.

You were both added as collaborators
Great to hear that this repo is not being neglected.


Support for qiskit 1.0

Adds the bare minimum changes needed to support running qiskit 1.0.0
Changes in this PR:

Updates requirement to qiskit >= 1.0.0 to avoid https://docs.quantum.ibm.com/api/migration-guides/qiskit-1.0-installation#:~:text=ImportError%3A Qiskit is,during environment resolution.
Gate method updates per https://docs.quantum.ibm.com/api/migration-guides/qiskit-1.0-features#quantumcircuit-gates
Removed QuantumCircuit.qasm per https://docs.quantum.ibm.com/api/migration-guides/qiskit-1.0-features#quantumcircuitqasm
Removed qiskit.extensions per https://docs.quantum.ibm.com/api/migration-guides/qiskit-1.0-features#qiskitextensions

is the 1.0.0 pin necessary? it seems like all of the changes in this pr would be compatible with qiskit>=0.45.0, and the linked upgrading issues shouldn't matter if e.g. someone has 0.45.1 and wants to stay on that version
(we definitely do have to drop 0.44.* though because the qiskit.extensions gates weren't moved until 0.45.0)

I was thinking about dropping the qiskit-ibm-provider>=0.7.2 req actually, and it seems like we may need to because of the failing qiskit-superstaq coverage check.  If we do that, then I am imagining we go back to using qiskit.qpy.dump(circuits, buf) but in light of https://github.com/Infleqtion/client-superstaq/blob/3ab6c9c4afa1487dff50c6a8179ccf6f09d78c10/qiskit-superstaq/qiskit_superstaq/serialization.py#L114-L117 we may want to use something like qiskit.qpy.dump(circuits, buf, version=qiskit.qpy.common.QPY_COMPATIBILITY_VERSION). But version was only added in 1.0.0 so it may cause issues for users with qiskit 0.45.1. If not, I think you're right that we should support qiskit>=0.45.0.


Request: Migration Guide from Webpack v1 => v2

I've been using the boilerplate to build out an application, and recently thought I should migrate from the old, Webpack1 based boilerplate to Webpack2. I'm pretty deep into the application, so I can't very well ditch the current code.
I've sunk a lot of time into migrating my app onto webpack 2, but run into persistent issues. Mostly, I've been attempting to incorporate the code on the current master branch of this repository. Was hoping the authors could throw together a quick guide on moving an existing application over to v2, with some provided diffs, and explanations. Much appreciated.

Did you change the webpack config that came with the template very much?
Even if you did, the easiest way would probably be to simply create a new project, copy the build files over, and update

webpack
webpack-dev-middleware
webpack-hot-middlewar
extract-text-webpack-plugin

to their latest versions.

Will do! Appreciate it very much.


Delete expired items before setting

Check if an item exists and it has expired before setting so that
onEvicted is actually called.
Fixes #48.

@patrickmn for your review 🙏

No. Always.
We overwrite an existing item. It should be comparison tha the item that we
overwrite is a different one. So, this comparison is deliberate. Usage of
expired produces wrong behavior.
On Thu, 15 Oct 2020 at 01:58, Jaime Martinez<EMAIL_ADDRESS>wrote:

@jaimem88 commented on this pull request.
In cache.go
https://github.com/patrickmn/go-cache/pull/132#discussion_r505085507:

@@ -57,6 +57,10 @@ func (c *cache) Set(k string, x interface{}, d time.Duration) {
if d > 0 {
e = time.Now().Add(d).UnixNano()
}

// delete before setting and call onEvicted
c.deleteIfExpired(k)

updated :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/patrickmn/go-cache/pull/132#discussion_r505085507,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AASOSQK3G7GDYXNVFTMGGCLSKY3KJANCNFSM4SNYEWEA
.

@patrickmn Would you mind looking at this?

Note this is a incompatible change, and some people may be relying on the current behaviour.
Also this library doesn't seem maintained at the moment, so little chance it will get merged. It's better to just maintain your own fork if you need this behaviour.

Trying to find a cache with TTL that has been rigorously tested and this seems to be a popular choice.
However, looks like repo is not maintained and hasn't been released since 2017.
@koalalorenzo Where you able to find any other forks reliable?

Found this fork: https://github.com/arp242/zcache


Is this expected Behaviour in /v0/send-bitclout?

I was running bit clout on my local machine in TESTNET mode, and i am trying out different API's from Backend API List through postman. Everything is working great when i call

api/v0/send-bitclout

I get an response, but immediately when i try to call next API

api/v0/submit-transaction

with transactionhex as input
i am getting

{
"error": "SubmitTransaction: Problem processing transaction: VerifyAndBroadcastTransaction: Problem validating txn: ValidateTransaction: Problem validating transaction: : ConnectTransaction: : _connectBasicTransfer: Problem verifying txn signature: : RuleErrorInvalidTransactionSignature"
}

when try to log few transaction data on my console i noticed below output is getting printed:

< TxHash: f285f5199b27eac5f40e9b2d194628c3d3d2612b667e4bd5a496e7e264c95c2e, TxnType: BC1YLgJxJq5BHAMhmcWjAxKjphgXE7fogodt1RDk9jhGWTLPz1j3qHP, PubKey: BASIC_TRANSFER >

here you can see

PubKey -> is having transaction type value and TxnType is having public key

it just swapped the data which is strange, i don't understand how this is happening? can someone shed some light here please?
Above log is printed when calling "/send-bitclout" API, inside "SendBitClout" method call -> above  line #987.
File: routes/transaction.go#L987

Did you sign the transaction hex before sending it to submit-transaction, or just lift it directly off the send-bitclout response?

Did you sign the transaction hex before sending it to submit-transaction, or just lift it directly off the send-bitclout response?

That's the exact issue.. Thanks for pointing out.. :)
Just figured out by running frontend on my local machine. Everything works smooth..
but i still don't get why the keys are swapped?

My guess is that's a typo in the logging code somewhere, it definitely shouldn't be like that.

I am not doing anything extra here! i am just printing txnn..
glog.Infof("========raw data Just before Sanity check=========: %v", txnn)

The transaction String() method has these parameters swapped. It's an easy fix in the core repo if you want to try it :)


Entities 'lclick' not always working

We've set up a pretty basic globe with some entities as pins on it.  As a test to make sure I got it working correctly I set up a JS array to loop through and add 4 pins to the globe.
const osm = new og.layer.XYZ("OpenStreetMap", {
  attribution: 'Data @ OpenStreetMap contributors, ODbL',
  isBaseLayer: true,
  url: "//{s}.tile.openstreetmap.org/{z}/{x}/{y}.png",
  visibility: true,
});

let globe = new og.Globe({
  "target": "og-container",
  "name": "Earth",
  "terrain": new og.terrain.GlobusTerrain(),
  "layers": [osm],
  "autoActivated": true
});

const items = [
  {id: 1, lat: 42.18380, lon: 3.7492, data: 'string1'},
  {id: 2, lat: 38.8950368, lon: -77.036542, data: 'string2'}
];

items.forEach((item, index) => {
  const entity = new og.Entity({
    name: item.id,
    lonlat: [item.lon, item.lat],
    billboard: {
      src: '/images/marker.png',
      size: [30, 35],
      offset: [0, 16]
    }
  });

  const pointLayer = new og.layer.Vector(`pointLayer[index]`, {
    entities: [entity],
    async: false,
    nodeCapacity: 10000,
    clampToGround: true
  });
  pointLayer.events.on('lclick', () => {
    itemData = item.data;
    console.log(itemData);
  });
  pointLayer.addTo(globe.planet);
});

That's the code I'm currently using to add the 4 entities to the globe with a marker so they're clickable.  Only problem is that when we turn the globe far enough from where it initially loads the pins are no longer clickable until you move back to somewhere near the original starting point or to very specific spots on the globe.  Is it an issue with how I'm trying to set up the 'lclick' event or with how the layers load as you move to different spots on the globe?

Just wanted to throw in a quick update, but the problem ended up being fixed by adding myPopup.setCartesian3v(e.pickingObject.getCartesian());.
I'm not too sure why I ended up excluding it from the original popup creating even though it's part of the example on the site.


Patch for compatibility with RelatedManager (reverse foreign-key)

It wasn't working with nested reverse foreign-keys, need to add .all():
def to_value(self, instance):
    fields = self._compiled_fields
    if self.many:
        serialize = self._serialize
        return [serialize(o, fields) for o in instance.all()]
    return self._serialize(instance, fields)

This enables to do something like:
class ParentSerpy(serpy.Serializer):
    children = ChildSerpy(attr='child_set', many=True)

You can do something similar to what is described in #17
children = ChildSerializer(many=True, attr='child_set.all', call=True)

Does that work for you?

Yes I tried that and it works, thanks. I did not put call=True before, trying attr='child_set.all()' did not work. Thanks for the help, this is even better.


Storyshots addon refactoring

Issue: including changed needed for #2517 (probably for #3152 too)
This PR is rebased on "storyshots-separation" branch because I didn't want to wait for the #3742 to be merged.
What I did

Created and expoded a class called Stories2SnapsConverter in order to map story-files to snapshot-files and vise versa.
Refactored a structure of the addon to make it more maintainable/readable.

LMK when it's ready for review

I think it's ready. Will work on other stuff in smaller PRs.

@Hypnosphi, can I merge this?


fix(material-experimental/mdc-tooltip): fix text alignment of multili…

…ne tooltips

Add the 'mdc-tooltip--multiline' class when a tooltip overflows
to make the text align left (or right in rtl) instead of center

Should we have a test for the new code?

Yes, it definitely should. I'll add one today 👍🏽


Simplify installation by adding noble and serialport to project dependencies

Related to https://github.com/orbotix/sphero.js/pull/42.
Could the project require both dependencies and use config options to determine which device is being used?
This would probably simplify the installation process and make the examples clearer?
e.g
device = sphero('########', { device: 'ollie' });

device = sphero('########', { device: 'bb8' });

device = sphero('########'); // falls back to original setup

Thanks for the comment @zaccolley we cannot do this, due to not all machines (especially older Windows) having the BLE stack needed to install Noble. We want to allow those users to continue to use Sphero.js in their use case. Hence the current more "dynamic" dependency.

OK gotcha


[FEATURE] Hide the right y axis when all series of this axis are hidden

When I hide the all series of y axis on right, the labels still continue, losing more space for viewing, especially on mobile

I removed it and use margin with negative value to USE all space

Couldn't this be extended to apply to all axes when the series is hidden?

display: 'auto'  for axis has been implemented already.
https://www.chartjs.org/docs/latest/axes/


Debug for EventwBayesClass because peaks overlapping

Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
Debug EventwBayesClass. In very rare cases the peaks' "time" can be the same, so simply use the matching between time https://github.com/XENONnT/straxen/blob/c61848018f2fd49504e6d3b50de1343ecfb056b0/straxen/plugins/events/event_w_bayes_class.py#L29-L30 can cause error like:
CRITICAL:ThreadedMailboxProcessor:Target Mailbox (event_w_bayes_class_paired) killed, exception <class 'strax.mailbox.MailboxKilled'>, message (<class 'ValueError'>, ValueError('NumPy boolean array indexing assignment cannot assign 48003 input values to the 47986 output values where the mask is true'), <traceback object at 0x7f62537a4e00>)

Can you briefly describe how it works?
Mimicking what EventAmbience does, split the peaks for each event and assign the fields based on indices: https://github.com/XENONnT/straxen/blob/c61848018f2fd49504e6d3b50de1343ecfb056b0/straxen/plugins/events/event_ambience.py#L42-L57.
The results are not changed so I will not update the __version__.
Can you give a minimal working example (or illustrate with a figure)?
Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

However, this plugin is not widely used.

I am wondering whether we should not remove this plugin. Is it still being maintained? I do not think so.

I am wondering whether we should not remove this plugin. Is it still being maintained? I do not think so.

For the reproducibility of old results, we still need it. I think we can keep it for a while.

Old results use old container no? Otherwise it is not reproducible anyhow?

Old results use old container no? Otherwise it is not reproducible anyhow?

Yes. You are right. I mean the reproducibility of the codes in other repo which still uses this plugin.

coverage: 90.818% (+0.01%) from 90.806%
when pulling 66a380d422f6e2b212d5e556a7fe49363e361ade on debug_event_w_bayes_class
into c61848018f2fd49504e6d3b50de1343ecfb056b0 on master.


Verify BRDF for image-based lighting

Cesium uses the squared roughness to select the LOD of the specular environment map, which appears to be incorrect. The glTF reference implementation uses "perceptual roughness" for environment map LOD selection.
Here are some rough comparisons in Cesium, compared to the reference implementation. Note: for these comparisons, all are using the "Helipad Golden Hour" environment map for the specular component. But Cesium is not using the corresponding coefficients for the diffuse component.
Clearcoat Car Paint model, using squared roughness (current behavior):

Clearcoat Car Paint model, using perceptual roughness (test):

Clearcoat Car Paint model, reference implementation:

At first glance, the squared roughness result in Cesium appears to be noisier. Also the environment map appears to be mirror-flipped relative to the reference?
Barn Lamp model, using squared roughness (this affects both LOD selection and anisotropy calculations):

Barn Lamp model, using perceptual roughness:

Barn Lamp model, reference implementation:

The squared roughness result does not show as much smearing along the brush strokes in the metal, indicating that perceptual roughness should be used for the anisotropy calculations.
We will need to verify the correctness before switching. Cesium's approach has some fundamental differences with the glTF reference implementation which can make comparisons challenging:

For the specular component: Cesium renders environment maps to an octahedral projection for sampling, with a custom setup for generating and sampling mipmaps. The reference implementation samples the cubemap directly, using the native WebGL2 function textureLod for mip selection.
For the diffuse component: Cesium samples pre-generated spherical harmonic coefficients. The reference implementation samples a pre-generated diffuse environment map along with a BRDF look-up table.

Material roughness is used in three places for IBL:

Generating the bent normal for anisotropic materials, to point to the appropriate location in the environment map.
Reading the BRDF from a look-up texture.
Selecting the appropriate LOD from the environment map.

We will need to verify which roughness to use in each place. See BrdfLutGeneratorFS.glslto for the generation of our BRDF look-up.
Assuming we store both perceptual roughness and squared ("alpha") roughness on the czm_modelMaterial struct, there are 2 things to keep in mind:

The current roughness property should really be named alphaRoughness or squaredRoughness for clarity. But would this be a breaking change? The czm_modelMaterial struct is indirectly part of the public API, via CustomShaders.
We may need to store two values for clearcoat roughness as well. The clearcoat is never anisotropic, but we do need to be able to compute both direct and image-based lighting.

A bigger issue for our IBL is the way we handle the diffuse component. We currently expect the user to compute the spherical harmonic from the environment map and set the coefficients manually. Then we sample it and add it directly to the specular component, without any BRDF or Fresnel terms.

A bigger issue for our IBL is the way we handle the diffuse component.

The BRDF and Fresnel terms for the diffuse component in the glTF Sample Viewer are actually part of a multi-scattering approximation. The terms are explained toward the end of this very helpful article.
These multi-scattering terms fix energy conservation problems in two main areas:

Brighten the appearance of rough metals. Without the terms, rough metals absorb energy.
Reduce the excessive Fresnel reflection at glancing incidence on dielectrics.

Despite the name, the approximate "multi-scattering" terms are actually cheap to compute, so we should incorporate them into our code in ImageBasedLightingStageFS.glsl.


Remove notes about ITuple changes(#7633) from v6.2.0-preview.1 updates.

PR Summary
Related #8070, #8077.
The notes of #7633 was already included in PowerShell Core 6.1.0 stable.
Note: We also must update Release notes of v6.2.0-preview.1.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[ ] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[ ] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

The update is useful for next release.

Reopen PR to restart CIs.

@SteveL-MSFT @TravisEz13 Can we fix the change log issue?

If the changelog is wrong, we should fix it.  We can ignore the WIP bot since it's clear it doesn't apply to this PR.

@TravisEz13 Please merge the PR.


WIP: Documentation pulled from the discussion in #642

Preliminary documentation based on the comments and discussion in issue #642.

@surli anything you feel that I have missed?

Pull Request Test Coverage Report for Build 1935

0 of 0   changed or added relevant lines in 0 files are covered.
220 unchanged lines in 22 files lost coverage.
Overall coverage increased (+0.3%) to 54.782%

Files with Coverage Reduction
New Missed Lines
%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/step/BuildProject.java
1
82.35%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/step/WritePropertyFile.java
1
85.19%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/step/TestProject.java
1
78.57%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/inspectors/metrics4bears/projectMetrics/ProjectMetrics.java
1
92.31%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/maven/RunnableMavenInvoker.java
1
73.08%

repairnator/repairnator-core/src/main/java/fr/inria/spirals/repairnator/states/PipelineState.java
1
0.0%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/step/CloneRepository.java
3
78.72%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/inspectors/metrics4bears/reproductionBuggyBuild/ProcessDurations.java
3
89.29%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/step/paths/ComputeSourceDir.java
3
92.5%

repairnator/repairnator-pipeline/src/main/java/fr/inria/spirals/repairnator/process/inspectors/metrics4bears/reproductionBuggyBuild/ReproductionBuggyBuild.java
5
70.59%

Totals

Change from base Build 1890:
0.3%

Covered Lines:
2881

Relevant Lines:
5259

💛  - Coveralls

@btellstrom I merge it we can always improve it later :) Thanks!


Fix equation in documentation of QuantileDeltaMapping

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[ ] Tests for the changes have been added (for bug fixes / features)

[ ] (If applicable) Documentation has been added / updated (for bug fixes / features)

[ ] HISTORY.rst has been updated (with summary of main changes)

[ ] Link to issue (:issue:number) and pull request (:pull:number) has been added

[ ] bumpversion patch has been called on this branch
[ ] The relevant author information has been added to .zenodo.json

What kind of change does this PR introduce?

Fixed equation in documentation of Fix equation in documentation of QuantileDeltaMapping

Does this PR introduce a breaking change?
No, there are no changes to the code.
Other information:
I think the equation in the documentation of QuantileDeltaMapping has the hist and ref  subscripts the wrong way around. This can be traced back to equations 3, 4, 5, 6 of the Cannon 2015 paper (linked in the documentation) with the following mapping from the notation in the paper to xclim:

_m,p maps to _sim
_m,h maps to _ref
_o,h maps to _hist
x_m,p(t) maps to sim
tau_m,p maps to F_sim(sim) (equation 3)

Coverage remained the same at 91.671% when pulling 09bce2646749e59338155f671352a151b066801a on maximerischard:master into 5cb07daf946349158e7e6770500342d993b10b12 on Ouranosinc:master.

Ah of course! I read that passage of the documentation several times but still got it swapped in my head. Thank you for pointing out my error.