Dictionary exporting not showing default value

Godot version:
3.1.1
OS/device including version:
Windows 10
Issue description:
When exporting a dictionary on a script the default assigned dictionary is not shown on the Inspector as expected. Instead, an empty dictionary is shown. I attached two screenshots, one with the declaration of the dictionary and the key/value pairs that it contains and another of the inspector with the empty dictionary that it shows.

This may be related to #10018 but it is said to be fixed.
Steps to reproduce:
1.- Export a Dictionary on a script.
2.- Assign it a non-empty dictionary.
3.- Save everything and check the exported dictionary. It is empty instead of having said key/values.
Minimal reproduction project:

Can reproduce on latest stable release (v3.2).

@rapsaGnauJ Can you attach a minimal project that exhibits this behavior? I am not able to reproduce this even on 3.2.

Apparently I can't replicate either on a brand new project. It happens to me on my complex project which is open source.
https://github.com/rapsaGnauJ/Mega-Man-Space-2
To replicate in this project, open the character scene (res://src/characters/character.tscn) and see the exported values. The dictionary in question is the one on the original screenshot and should be one of the lasts. It should say size 9, but it is sized as 0.
I don't want this to be considered spam. It's just that I don't what else can be involved in this issue.

@rapsaGnauJ Thanks for sharing more details; this makes it easier to diagnose the issue.
My original interpretation of the problem was incorrect: it's not just exporting the dictionary that causes the issue. Instead, from what I can understand, it's exporting a dictionary that uses a variable from another script.
In your case, if you were to declare the enum WEAPONS in your character.gd file instead of the global.gd file, the problem goes away. Obviously this isn't a solution, just a diagnosis.

So that is the cause of the problem... Thanks for taking the time to inspection this. I have also verified that using enums in a exported dictionary causes the keys/values to show as ints in the editor's inspector. This is just a suggestion, but showing the enum value name would probably be prefered.

Is this fixed by now?

Any development on this one?


[stable/consul] random resource name causing issues

We are using https://github.com/roboll/helmfile to define our cluster setup through GitOps.
Helmfile runs diff and only does a helm upgrade when there are changes.
Consul has consul-test.yaml template with a random name, causing Helmsfile to always detect changes and try to upgrade.
apiVersion: v1
kind: Pod
metadata:
  name: "{{ .Release.Name }}-ui-test-{{ randAlphaNum 5 | lower }}"
  annotations:
    "helm.sh/hook": test-success

Would it be possible to remove the random part from the resource name, or atleast make the test optional?

Any updates on this? I’m having the same issue.

This is a problem for me as well.. We use helm diff as a part of validation process.. When I do bigger deployments/upgrades in customer facing environments it causes unnecessary noise in our pipelines.

We're experiencing the same problem with the consul chart. It appears to affect a number of other stable charts too:
https://github.com/helm/charts/issues/7979
https://github.com/helm/charts/issues/7708
The original intention behind generating random test pod names was to allow consecutive test runs, but that seemingly no longer applies as helm test has supported a --cleanup flag since https://github.com/helm/helm/pull/1935.
Assuming it is approved, I propose using the same fix as https://github.com/helm/charts/pull/8138 in all affected charts.

/remove-lifecycle stale

/remove-lifecycle stale


Use numpy structured array for iedge table

Peter Ralph suggested we could store the iedge data in a numpy array. We can wrap the whole thing in a class which will preallocate memory in the right way. We could use the numpy recarray class (which is a wrapper for structured arrays) but I have read that it can be slower for accessing attributes, because of the checking it has to do.  I think we can just use structured arrays with a bit of syntactic sugar, and it should be very fast. Here's an outline: I think it might just be a drop-in replacement. I'm hoping that this could make everything much faster:
import collections
import logging
class IedgeTableRow(collections.namedtuple("IedgeTableRow", "child_left, child_right, parent_left, parent_right, child, parent, child_chromosome, parent_chromosome, edge")):
    @property
    def diff(self):
        return self.child_left - self.child_right

class IedgeTable:
    initial_size = 1  # number of rows - change to e.g. 128
    max_resize =  2**18  # maximum number of rows we expand the storage by
    def __init__(self):
        self._datastore = np.empty((self.initial_size), dtype=[(name, np.int64) for name in Row._fields])
        self._data = self._datastore[0: 0]

    def __getitem__(self, index):
        return IedgeTableRow(*self._data[index])

    def _expand_datastore(self):
        print("preallocating new memory ...")  # change to logging.debug
        old = self._datastore
        try:
            self._datastore = np.empty_like(self._datastore, shape=min(2 * len(self._datastore), len(self._datastore) + self.max_resize))
        except MemoryError:
            # should try dumping the table to disk here, I suppose?
            raise
        self._datastore[0:len(old)] = old
        print(f"... allocated space for {len(self._datastore) - len(old)} new rows")  # change to logging.debug
        # Hopefully force garbage collection of old array: but not if the user has a variable
        # accessing the old array (see test case)
        del old

    def add_row(self, child_left, child_right, parent_left, parent_right, *, child, parent, child_chromosome=0, parent_chromosome=0, edge=-1):
        # Check here if size of self._datastore needs increasing. If so, resize to e.g. double the size up to a certain limit
        if len(self._data) == len(self._datastore):
            self._expand_datastore()
        self._datastore[len(self._data)] = (child_left, child_right, parent_left, parent_right, child, parent, child_chromosome, parent_chromosome, edge)
        self._data = self._datastore[0: len(self._data)+1]

    def __len__(self):
        return len(self._data)

    @property
    def _num_preallocated_rows(self):
        return len(self._datastore)

    @property
    def child_left(self):
        return self._data['child_left']
    @property
    def child(self):
        return self._data['child']
    @property
    def parent(self):
        return self._data['parent']
    # Define other properties here

# test it!

t = IedgeTable()
t.add_row(0, 1, 0, 1, child=0, parent=1)
t.add_row(0, 1, 0, 1, child=0, parent=2)
t.add_row(0, 1, 0, 1, child=0, parent=3)
m = t.parent
t.add_row(0, 1, 0, 1, child=0, parent=4)
t.add_row(0, 1, 0, 1, child=0, parent=5)
print(f"{len(t)} rows in table (allocated space for {t._num_preallocated_rows}) rows")
row = t[-1]
print("Last row:", row)
print(row.child)  # attribute accessing
print(row.diff)  # "method" accessing
print("Parents in table:", t.parent)  # array accessing
# but note that `m` is a reference to the "old" array
print(f"kept a reference to {len(m)} rows, indexing into an array of {len(m.base)} rows")

Giving:
preallocating new memory ...
... allocated space for 1 new rows
preallocating new memory ...
... allocated space for 2 new rows
preallocating new memory ...
... allocated space for 4 new rows
5 rows in table (allocated space for 8) rows
Last row: IedgeTableRow(child_left=0, child_right=1, parent_left=0, parent_right=1, child=0, parent=5, child_chromosome=0, parent_chromosome=0, edge=-1)
0
-1
Parents in table: [1 2 3 4 5]
kept a reference to 3 rows, indexing into an array of 4 rows

Note the slight issue at the end that any references to the old arrays will not reflect the original data when a new memory array has been allocated. I guess there's nothing we can do about that, and it's an edge case anyway.

https://github.com/hyanwong/GeneticInheritanceGraphLibrary/tree/numpy-edge-table shows that it can be done. We should probably implement this, but it requires a bit of thought to do it in the simplest way.

Merged in #116


Endpoints returning futures don't work

From #21. Apologies for the formatting.

@taion https://github.com/tavendo/AutobahnCpp/issues/21#issuecomment-71883839:
Actually, it turns out that none of the endpoints that expect futures actually work. The issue is that you actually wait on the future here:
https://github.com/tavendo/AutobahnCpp/blob/b44ee2e0582923fe8e175eff1ec726582575f4e2/autobahn/autobahn_impl.hpp#L771
However, this ends up blocking the running thread. If my endpoint does something like
return session.call(procedure, args).then(/* do some work*/);

then the procedure will never actually return, because the running thread is blocked on the done.wait() call, and cannot process the return value from the called procedure and resolve the future.
You can't wait for the future to be resolved there.

@oberstet https://github.com/tavendo/AutobahnCpp/issues/21#issuecomment-71932930:
Here is an example for endpoint_fvm_t: https://github.com/tavendo/AutobahnCpp/blob/master/examples/register2.cpp#L63

@taion https://github.com/tavendo/AutobahnCpp/issues/21#issuecomment-71933139:
The reason that example works is because the future is ready, so done.wait() returns immediately.
If waiting for the future actually requires handling some input (e.g. you invoke another method and wait for it to finish), then it will not work, because you've blocked the main thread from actually processing the response.

@oberstet https://github.com/tavendo/AutobahnCpp/issues/21#issuecomment-71934192:
I see.
I guess it would be good if we first had an example that demonstrates the issue, and then come up with a solution.
This is the harder problem - extending the solution to the variants should then be straightforward.

The issue with variants is no longer there, but endpoints that return futures still don't work. Some combination of boost::asio::io_service::work and std::shared_ptr will probably be required to resolve this.

This is fixed in my fork of autobahn-cpp. Also, the only reason that the examples even work is due to the fact that the boost tcp resolver actually creates a thread that polls the io service.

@davidchappelle Cool - I've actually switched to CppWAMP, but probably having a working "official" AutobahnCpp implementation is a good thing.

This can now be closed.


cannot start Bedrock server

When I try to start the container I’m getting
Failed to extract download URL '' from https://www.minecraft.net/en-us/download/server/bedrock/
2020/07/17 12:05:27 Failed to get from URLGet https://www.minecraft.net/en-us/download/server/bedrock/: dial tcp [2606:2800:233:1cb7:261b:1f9c:2074:3c]:443: connect: cannot assign requested address

repeatedly, and the server fails to start.
Sometimes it gets through at some point after approx 5-10 minutes, sometimes stays this way.
Any advice appreciated.

It seems like it's some kind of IPv6 issue with your host or docker setup, but I have no idea what to suggest for a solution. FYI, when I ping www.minecraft.net with the bedrock image it resolves to an IPv4 address:
docker run --rm --entrypoint ping itzg/minecraft-bedrock-server -c 1 www.minecraft.net
PING sni1gl.wpc.nucdn.net (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=1 ttl=53 time=12.4 ms

--- sni1gl.wpc.nucdn.net ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 12.406/12.406/12.406/0.000 ms

yes, probably my network, been tinkering with it. tu


Bump CNI plugin to 1.0.1

Signed-off-by: Masashi Honma<EMAIL_ADDRESS>

/release-note-none

Re run LGTM analysis: C/C++ after #207 is merged.

/ok-to-test

/test pull-e2e-ovs-cni

Could anybody re-run LGTM analysis: C/C++ ?

/lgtm
/approve
Ignoring LGTM as it is irrelevant and fights with the rebase.

Ignoring LGTM as it is irrelevant and fights with the rebase.

Ok, thanks.


90 create a pandas table instead of an list of object

There were some test changes

I accidentally forgot to implement the calculation of the frequency.


Chapter 18 (confidence intervals for proportions)

Needs to be drafted.

Done (also chapter 19, which has an inordinate fondness for onesided tests).


JAMES-1975 Move mailets over from james-jdkim

https://issues.apache.org/jira/browse/JAMES-1975
Added the necessary files and changed the pom as needed. Need guidance on whether this was done correctly.

I will trigger a build, and wait over reviews before merging this.

I'm really not sure whether I did this correctly. I just moved over the mailet files and updated the POM. I've run a successful build without tests. When I run a build with tests I get errors unrelated to this, probably something wrong with my local setup.

I'm looking around and found this PR that seems to be waiting for a merge.
@chibenwa, @shrumit: Should we continue the work and merge? It seems to me like we should.

Hello @ieugen ,
As far as I can tell, this PR was not passing unit tests.
The move of the mailet from dkim into James Server code had been contributed here https://github.com/linagora/james-project/pull/1709
We should have been notifying it here.
This pull request likely can be closed.

Something has been merged there so I second the closing of this issue. Thank you for your contribution @shrumit . Please let us know if we missed something.


git-tag-project-version-and-build-number 1.0.1

New Pull Request Checklist
Please mark the points which you did / accept.

[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

Hi @ned1988 ,
thanks for the PR! It looks great to me!


Update MEV-relay-list.md

Added in link to Blocknative's relay dashboard

Thanks for the info. Merging.


[21.11] libarchive: 3.5.2 -> 3.5.3

Description of changes
#132136
#138679
#142981
https://nvd.nist.gov/vuln/detail/CVE-2021-36976
This bump looks like it's really nothing but security fixes.
Also cherry-picked #159900 to ease testing.
Fixed in master with #159465
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Successfully built libarchive.passthru.tests on aarch64-linux.


Add functions to change limitations gradually

Currently the contracts are deployed with limits. Those limits can later be removed, but not changed gradually.
Instead of removing the limits completely, we can add a way to change them.

TokenNetworkRegistry.removeLimits() could be changed to TokenNetworkRegistry.increaseLimits(new_limit), which increases the limit of token networks to the absolute value of new_limit.
TokenNetwork.removeLimits() could be changed to TokenNetwork.increaseLimits(new_channel_participant_deposit_limit, new_token_network_deposit_limit), which would increase those limits.

Both should check, that the values can only increase, not decrease.
@DinaBB

@netcriptus sync with @karlb on Thursday about this.

We won't be implementing this due to not being in scope and it is more a nice to have rather than a must.


Add Prisma LSP

Finishes implementing https://github.com/lspcontainers/dockerfiles/issues/72

This has been completed by #79


UI: async job refreshes to a screen not currently viewd

After deleting a vm and going into the details view of another VM, the screen is refreshed on another view.
ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

main

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

STEPS TO REPRODUCE

Delete a vm (with expunge and disks)
Go into the details view of another VM
When deleting the first VM is done, the screen refreshes into the VM overview list, abandoning the current details view.

EXPECTED RESULTS

The details view is still visible

ACTUAL RESULTS

The overview is shown

@davidjumani can you please check this issue?

This is a code wide issue and might take some time, I'll look into it

Will be easier to fix after https://github.com/apache/cloudstack/pull/4782 is merged

Closing as issue is fixed in #5214


Follow step and unit tests

PR for #233
Added step Follow, with which the bot can follow the commander.
Added unit tests and put it in the tree of steps for each command.

Coverage increased (+0.3%) to 81.494% when pulling 59a8d05000669b3d97f5f80e894dec1f4ecbfcf0 on 233 into ff34535f02118c193cac60b6bc79a72887c6819d on master.

@rultor merge this please

@rultor merge this please

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge this please

@amihaiemil Done! FYI, the full log is here (took me 2min)


Add debian  package . Issue #50

Added the debian package for easy installation of slacktee.

I would like to hear your opinion.
As far as I have researched, storing the package information in the separate repository is the common practice. Since non debian package users are still cloning this repo, keeping this repository small is important.
In order to create a new repository for this package, I came up with two options.

Create a empty repository under Course Hero and ask you to create a PR to add debian directory to it. In this way, Cousre Hero will be a maintainer of this package (Your name still remains as a main contributer).
Ask you to create a repo under your account. We are creating a PR to your repo if we would like to change the package. In this way, you will be a maintainer of this package.
Since you are the author of this package, No.2 is more straight forward to me. But, if you don't want to be a maintainer of this package, I'm more than happy to create a repo under Course Hero.
What do you think about this? Which way do you prefer?

I like the first option , as this repository has most of your work ,also I am still a newbie in open-source so I would be more than happy in updating this Debian package and improving it by sending
PR's to your new repository.
Please share the new repository's link as soon as you create it, so I can send the PR.
Thanks :)

Awesome! I just created an empty repository. It would be great if you can create a PR to this repository!
Also, thank you for removing your personal Slack information from the code.
Thank you so much for your hard work!

No problem :) Made the PR on new repo.
Now closing this PR.

Awesome! I will add the link referring to the package repo to Readme later!


[Bug] element-newline incorrectly reports on siblings

For example, this test fails. It should pass because b and c are skipped:
    {
      code: `
<c></c>
`,
      options: [
        {
          skip: ["b", "c"]
        }
      ]
    }

Actually this seems to be a bug with checking sibling elements in general. Updated the main post.

Hi @RobertAKARobin

This rule has an object option:
"skip": skips newline checking for the specified element's children

This is the intended behavior: the skip option skips the occurrence of child elements of the specified tag.

This additional functionality added in https://github.com/yeonjuan/html-eslint/pull/214


use a new algorithm to do lock one trans

Codecov Report

Merging #471 (a5f0ae1) into main (e704733) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #471      +/-   ##
==========================================
- Coverage   95.98%   95.91%   -0.07%
==========================================
  Files          57       57
  Lines        4206     4212       +6
==========================================
+ Hits         4037     4040       +3
- Misses        122      124       +2
- Partials       47       48       +1

Files
Coverage Δ

dtmsvr/storage/sql/sql.go
97.30% <100.00%> (-1.33%)
:arrow_down:

dtmsvr/trans_status.go
94.14% <100.00%> (+0.05%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Revert "Add log4j2 integration"

@adrianblade, we as reviewers shouldn't merge any pull request, that is a task for @fjlopez.
Plus, I just started to review this.

I'm sorry i didn't know @fjlopez @SAReyes


Updating profile from GitHub should show a notification and redirect to FCC profile

Issue Description
Steps to reproduce:

Make sure you've previously linked GitHub with your Free Code Camp account
Go to https://www.freecodecamp.com/settings
click "update my portfolio from GitHub"
it will do so, then redirect you back to your current challenge, but it won't show any confirmation flash message.

Instead, it should show a success flash message that says "We've updated your profile based on your your GitHub account."
While we're at it, we should change the redirect to send campers to their profile so they can confirm themselves that their biographical information has been updated.
Finally, we should change the button from "update my portfolio from GitHub" to "update my profile from GitHub" since campers often mistakenly expect us to pull in data about their actual GitHub projects - which we do not yet do.
CC @BerkeleyTrue

Hi,
I would be interested in working on this.

Is there any place I can go to get help with my local environment?  All the sudden it seems to have stopped working.
I'm on OS X 10.9.5, using Node 6.5.0, NPM 3.10.3, and Mongo 3.2.9.
Thank you.

@craigstroman did you follow this guide? http://forum.freecodecamp.com/t/how-to-fork-and-maintain-a-local-instance-of-free-code-camp/19116


Remove necessary constraint import in type.bal file

Description:
Regression from this PR https://github.com/ballerina-platform/openapi-tools/pull/1226
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Close with the help https://github.com/ballerina-platform/openapi-tools/pull/1236


Question: Will a 1.19.4 world with GBW be able to update to 1.20?

Hey! I was wondering if I made a world with this mod on 1.19.4, would i be able to update to 1.20 once you update the mod to that version?

It should be able to update without issue. The new update only adds 1 new biome which doesn't affect any biomes added in GBW, and the new tropical islands in GBW 1.2.0 are done separately. If any issues do occur, I'll be sure to iron them out asap!


woodpecker-server: 2.7.0 -> 2.7.1

Description of changes
https://github.com/woodpecker-ci/woodpecker/releases/tag/v2.7.1

Things done

Built on platform(s)

[ ] 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

[ ] Fits CONTRIBUTING.md.

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

backport to latest stable as this version will be important in the near feature and we want to have good adoption...!


Openvino Implementation

Added openvino implementation - an AI inferencing accelerator. Can be used with "--openvino" flag

Added toml file for packaging


[BUG] Azure Graph Client logging too many things at ERROR level

Rancher Server Setup

Rancher version: 2.9.4
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc):

Proxy/Cert Details:

Information about the Cluster

Kubernetes version:
Cluster Type (Local/Downstream):

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:

Describe the bug

The logging is confusingly logging things at ERROR level when they are not errors and are skipped.
https://github.com/rancher/rancher/blob/7839201dc49c3cbd138b0bb99028c5705825d699/pkg/auth/providers/azure/clients/ms_graph_client.go#L241
2024/12/10 14:37:13 [ERROR] [AZUREAD_PROVIDER] Page Iterator received incorrect value of type *models.AdministrativeUnit: &models.AdministrativeUnit{DirectoryObject:models.DirectoryObject{Entity:models.Entity{backingStore:(*store.InMemoryBackingStore)(0xc1dac12840)}}}
2024/12/10 14:37:13 [ERROR] error syncing 'u-texot2c4ev': handler mgmt-auth-userattributes-controller: error refreshing user attribute u-texot2c4ev: NotFound 404: getting user by ID: error status code received from the API, requeuing
2024/12/10 14:37:13 [ERROR] [AZUREAD_PROVIDER] Page Iterator received incorrect value of type *models.AdministrativeUnit: &models.AdministrativeUnit{DirectoryObject:models.DirectoryObject{Entity:models.Entity{backingStore:(*store.InMemoryBackingStore)(0xc1c951d120)}}}
2024/12/10 14:37:13 [ERROR] [AZUREAD_PROVIDER] Page Iterator received incorrect value of type *models.AdministrativeUnit: &models.AdministrativeUnit{DirectoryObject:models.DirectoryObject{Entity:models.Entity{backingStore:(*store.InMemoryBackingStore)(0xc0f8cbdf20)}}}
2024/12/10 14:37:13 [ERROR] [AZUREAD_PROVIDER] Page Iterator received incorrect value of type *models.AdministrativeUnit: &models.AdministrativeUnit{DirectoryObject:models.DirectoryObject{Entity:models.Entity{backingStore:(*store.InMemoryBackingStore)(0xc0faf430e0)}}}
``

**To Reproduce**
<!--Steps to reproduce the behavior-->

Connect Rancher to Azure AD with a directory that contains non-users/groups and we'll get different resources that are logged as errors.

**Result**

**Expected Result**
<!--A clear and concise description of what you expected to happen.-->
These should not be logged at ERROR level (but they should still be available in case we need to debug what resources we get).

**Screenshots**
<!-- If applicable, add screenshots to help explain your problem.-->

**Additional context**
<!--Add any other context about the problem here.-->

This fix should be applied across:

[ ] release/v2.9
[ ] release/v2.10
[ ] release/v2.11


Geoopt update

FYI, the code should be updated to keep in sync with
https://github.com/geoopt/geoopt/pull/77
Write here if any questions
We are in active development stage, but sorry for troubles

Thanks for the heads up! We ended up implementing our own version of what geoopt does but it was heavily inspired by geoopts code so we will make sure to give credit on publication!

What limitations of geoopt led you this way?
On Thu, Aug 15, 2019, 00:37 jdieter31<EMAIL_ADDRESS>wrote:

Thanks for the heads up! We ended up implementing our own version of what
geoopt does but it was heavily inspired by geoopts code so we will make
sure to give credit on publication!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jdieter31/riemannian-nlp/issues/1?email_source=notifications&email_token=ACZJX3VIP3DFYOMAQDBL5Q3QER3IZA5CNFSM4HZETVU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4KF3MI#issuecomment-521428401,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACZJX3W6QJ6IXWB77BRVOYTQER3IZANCNFSM4HZETVUQ
.

It was mainly that geoopt had no in-place operations or great support for working with sparse tensors which my application needed. I ended up making a fork of geoopt but it had grown far enough from the original project that it eventually made more sense to condense the features of it and streamline it as just part of the project codebase.

CC @newkozlukov


Year Range Picker

Describe the solution you'd like
Its an excellent plugin with lot of options and customizations especially month and quarter range pickers. In addition to that,it would be great if we have the year range picker also.
Describe alternatives you've considered
This is the best solution I've found with simple yet with lot of customizable options for range pickers. I've tried airbnb, material ui range pickers, bootstrap range picker, advanced react range picker plugins etc.

Need this feature too!


" shouldn't be auto-completed when typed after @ in vimscript or cmdline

" shouldn't be auto-completed when typed after @ in vimscript or cmdline because @" represents the unnamed register.

Added https://github.com/altermo/ultimate-autopair.nvim/commit/3d061253d4f827a39dff0c2575fc7356e79a7d26

@altermo " is still auto-paired if @ is not the first character in the current line:
https://github.com/user-attachments/assets/7a0bae7a-52d5-4726-951e-3e33b32f8a61

fixed

Thanks for the quick fix!


Using FromSql

I am trying to use FromSql<NaiveDateTime> but I always get a compile error:
^^^ the trait `rusqlite::types::FromSql` is not implemented for `chrono::NaiveDateTime
I added the chrono feature to my dependencies, but I am probably missing some import,
use chrono::prelude::*;
use rusqlite::types::chrono::*; // This fails, chrono is private

What imports do I need so I can do let v: NaiveDateTime = row.get(0)?

Try
use rusqlite::types::FromSql;

It's also possible you were running into a chrono version mismatch (see #241). If you're using chrono 0.3, try updating to rusqlite 0.10.
Please reopen if neither of these work!

Thanks, upgrading to 0.10 worked.

For anyone who encounters this error message, don't forget to add the chrono feature in your rusqlite dependency, like so:
rusqlite = { version = "0.24.1", features = ["bundled", "chrono"] }


Fix #121 - weird menu builder behavior

This inlines the menu-builder from https://github.com/wilzbach/menu-builder.
In general I hope that we can replace the menu (which was only intended as showcase) with integrated controls (like the zooming control). See https://github.com/wilzbach/msa/issues/149 for more details & infos.

@guidorice, @smoretti I will just go ahead and merge this, s.t. you can give it a try with latest, which you can take directly from the S3 bucket to avoid caching from Amazon CloudFront.
http://cdn.bio.sh.s3.eu-central-1.amazonaws.com/msa/latest/msa.min.gz.js

Please let me know if this still didn't fix your problem.

For me the latest version (downloaded 2016-08-05) fixed the problem
Thanks


Impossible to open Files: Store code

Description
I have this window sinc last update:
Files n’est actuellement pas disponible dans votre compte. Assurez-vous que vous êtes connecté au Microsoft Store et réessayez. Voici le code d’erreur, au cas où vous en auriez besoin : 0x803F8001
Steps To Reproduce
try to open it
Requirements

eg. Fix crash that occurs when...
eg. Correct action happens when clicking...

Files Version
2.0.67
Windows Version
11
Log file
debug.log

This is an issue with the Microsoft store. You can try using the classic installer from https://files.community/download

@lyonelf can you verify that you're signed into the Microsoft Store?

I can't sign in because i have no personnal account, only a pro school one.

@lyonelf you're best option would be to use the classic installer that @Josh65-2201 linked. Can you see if that works?

Yes it’s ok like that.
I’ll try to remember this, for next time if hapen again
De : Yair @.>
Envoyé : mardi 25 avril 2023 16:47
À : files-community/Files @.>
Cc : Lionel-BZH @.>; Mention @.>
Objet : Re: [files-community/Files] Impossible to open Files: Store code (Issue #12201)
@lyonelf https://github.com/lyonelf  you're best option would be to use the classic installer that @Josh65-2201 https://github.com/Josh65-2201  linked. Can you see if that works?
—
Reply to this email directly, view it on GitHub https://github.com/files-community/Files/issues/12201#issuecomment-1521922777 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIEEST426KI6D6HOQ2GFKCTXC7PXNANCNFSM6AAAAAAXKVMPDE .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AIEESTZ34WKTJNZDXZ5HNU3XC7PXNA5CNFSM6AAAAAAXKVMPDGWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTS2W2ZNS.gif Message ID: @.*** @.***> >


move build badge

(failed to commit this to #6 😬)
/cc @mit-mit

Maybe flip these around, as I feel people will care more about the pub link.


Non feature tests

Function Checks:
[x ] All tests passing
[x ] Code runs locally
Type of Change:
[ ] New Feature
[ ] Bug fix
Implementation/Fix Details:
adds tests for all services facades and poros
Dummy check: Did this break anything else?
[ ] Yes
[x ] No
If 'yes' checked above, what did it break?
Testing changes:
[ ] No tests have been changed
[ x] Some tests have been changed
If 'yes' checked above, what tests had to be changed and how?
adds tests for all services facades and poros
Final checklist:
[ x] I have reviewed my code
[ ] I have comments in my code for better understanding if needed
[x ] I have tested my code

Looks awesome! Nice sad path testing on the .player_in_party method.
Just gotta delete that application.yml from the remote branch and then its a-okay to merge imo


Global fog is broken on Wolf:ET railgun map

#314 shows the map working correctly.
Now it looks like this:
Before fog:

After fog:

It's binding an incorrect image as the depth map for some reason.

This bug is not present when using bindless textures, so it's only the regular codepath that is broken.


A command to inspect the parsed AST

Something like this (ANTLR):

https://learn.microsoft.com/en-us/dotnet/csharp/roslyn-sdk/get-started/syntax-analysis
Roslyn gives you a side-by-side view, where you can click code and have it highlighted in the tree, and vice versa.

I thought of this as a terminal-friendly tree rendered using Spectre.Console. A GUI for any operation should, IMHO, be moved to a seperate project and binary.


[PDI-14403] Salesforce Error Message

Show real username when testing the Salesforce connection

@pamval Created #2277


ISPN-14940 Manipulating connection pool threads

https://issues.redhat.com/browse/ISPN-14940
I saw another thread dump on CI with a lot of threads, even after the last changes. I am trying something more exaggerated now.
I'll be running on draft first to see how much of the tests will break or leak.

Since I removed the line in the ThreadLeaker, tests outside server/tests also failed with thread leaking. Working a bit more on this.

Instead of manually implementing a wrapper for the HTTP client, I added a Byteman script manipulating the OkHttp connection so we wouldn't have that amount of threads blocking.
I'll be closing this, as I don't think it is worth the effort for all that. If we ever feel the need, we can use this.


Typing is  very slow with Tabnine installed

OS version: Win 10
Editor version: 3
Programming language: python
Tabnine extension version: 3.2.1

Issue Details:
Tabnine becomes very slow when trying to autocomplete. There is a delay of almost 5 seconds between pressing Tab and it autocompleting

Just noticed this and came here. If I stick to sublime then tabnine will be useless to me since I have it disabled now anyway.
My delay isn't 5 seconds but it's enough to annoy me.

Is there any solution of this?


DXSurvey Crashes

Are you requesting a feature, reporting a bug or ask a question?
Bug

What is the current behavior?
Seems like editing a survey is crashing

Thank you for the report, @enricribas ! I've temporary reverted sites to the 0.12.26. We'll fix it ASAP.


Bug: Reader fixes

Remove OpenCV based reader and fully move to decord - closing issue #414
Fix decord and csv reader test cases that weren't testing correctly.
Fix bug with open UDF (https://github.com/opencv/opencv/issues/14095)
Remove timestamp UDF as we now support it using  decord

@tushar-97 If this looks good, we can merge.


fix(google-maps): correct minor typo in the code

Position's comment is wrong

@googlebot I signed it!

Thank you for your review @crisbeto !
It's my first pull request. I have just followed your suggestions but it looks like CI still don't pass my commit. Can you help me?

It's because you only renamed the pull request, but the commit still has its old name. See:

You'll have to rename it yourself. It doesn't have to be in a new PR. Here's some info on how to rename a commit https://docs.github.com/en/github/committing-changes-to-your-project/changing-a-commit-message


The package is not installable

When I run npm install svg-path-bbox I get an npm error:
npm ERR! code ENOENT
npm ERR! syscall chmod
npm ERR! path my-path\node_modules\svg-path-bbox\src\cli.cjs
npm ERR! errno -4058
npm ERR! enoent ENOENT: no such file or directory, chmod 'my-path\node_modules\svg-path-bbox\src\cli.cjs'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent
I am using node v14.7

I have replicated it but after upgrading npm with npm install -g npm the binary has been fixed, hope this helps.

@mondeja Still reproducible on Node v14 (but working fine on v16). The reason is a wrong file name in package.json "bin" section:
  "bin": {
    "svg-path-bbox": "src/cli.cjs"
  },

The file is called src/cli.js, not src/cli.cjs (not the extension). For some reason Node v16 ignores the extension.
Simply renaming the file should fix the issue. For testing, you can use NVM: nvm use 14
@gaitat I was able to find this workaround: in my package.json, add the following into "scripts":
"preinstall": "touch node_modules/svg-path-bbox/src/cli.cjs",

@scriptin Thank you

@gaitat Actually, my workaround doesn't work in some use cases. I've tried it in my CI/CD pipeline, and it still fails. Upgrading to Node v16 (or maybe higher, I haven't tried) is a better option.

Thanks @scriptin, I'll

Fixed on v1.2.3

@mondeja Thank you very much for such a quick response. Awesome library!


add makefile rules for integration tests

Description
Changes proposed in this pull request:

added makefile rules for integration tests

/test all

/test tide

/test tide

/test all


[Feature Request] - Batch spend

If you want to send 2 or more outputs to the same Paynym in a batch spend, you send them to the same address. This issue is to request to add a different address for every output sent to the same Paynym.
Thanks!

Implemented in 84978a3d - Sparrow will now use different addresses when sending batched payments to the same PayNym.


Update to tabula version 1.0.1

I have updated the version of Tabula to 1.0.1 (last stable at the moment).
I have not deleted the old jar file since I did not know if it would cause any incompatibility problem for those who are already using this package. Please delete it if necessary.
Emiliano

Hey @emilianotisatol. Thanks for the pull request. I think it is safe to delete the jar as people would depend on a certain version of this repo. Could you update this pull request?

Done!


Question: Send lua table to function.

Is any way to wrap this lua code to typescript file?
post("exit", {code = 0})
I need to create lua table with custom fields.

Solved. Close issue pls.
TypeScript code is:
post("exit", { code: 0 })


[Master] release/v1.1.1 QA Sign-off

This PR is automatically created by CI to release a new official version of caver-js-ext-kas.
When this PR is approved by QA team, a new version will be released.

caver-js-ext-kas v1.1.1 QA Result
Functional Test Result

KAS-ANCHOR
Total Test Cases 37 = PASS  37 + FAIL  0 + N/A  0
Newly Added Test Cases 0 = PASS  0 + FAIL  0 + N/A  0

KAS-TH
Total Test Cases 138 = PASS  129 + FAIL  0 + N/A  9
Newly Added Test Cases 0 = PASS  0 + FAIL  0 + N/A  0

KAS-KIP17
Total Test Cases 79 = PASS  77 + FAIL  0 + N/A  2
Newly Added Test Cases 0 = PASS  0 + FAIL  0 + N/A  0

KAS-WALLET
Total Test Cases 222 = PASS  222 + FAIL  0 + N/A  0
Newly Added Test Cases 0 = PASS  0 + FAIL  0 + N/A  0

KAS-ETC
Total Test Cases 8 = PASS  8 + FAIL  0 + N/A  0
Newly Added Test Cases 0 = PASS  0 + FAIL  0 + N/A  0

Integration Test Result

KAS-INT
Total Test Cases 258 = PASS  258 + FAIL  0 + N/A  0
Newly Added Test Cases 0 = PASS  0 + FAIL  0 + N/A  0


Cleanup unused variables in zcl_excel_theme

Removing the statement ixml_element->get_attribute( 'xmlns:a' ) is anyway a good thing because getting a namespace attribute this way is not supported by IXML and it is always returning an empty string even if this "attribute" exists.

Obvious code fix -> approving myself.


Debugging notes for required params after optional params error

I spent a bunch of time today debugging an error I was seeing on a Rails 7 app, where we were getting this error and it was pointing here: https://github.com/rails/rails/blob/2459c20afb508c987347f52148210d874a9af4fa/activerecord/lib/active_record/persistence.rb#L405
Eventually, with the help of @jeffcarbs and @paracycle I tracked it down to a def self.update! in an Active Record model. This method had a sig { void } but it seems like for some reason the names matching still triggered this error.
I'm not sure how to fix that or if it's even a bug, but I thought some debugging notes might help the next person to come across it. If it is considered a bug, happy to provide more info to help with replication.

This passes statically but raises at runtime:
# typed: true
require "sorbet-runtime"

module A
  def foo(x = 1, y)
  end
end

class B
  extend T::Sig
  extend A

  sig { params(z: String).void }
  def self.foo(z)
  end
end

B.foo("hi")

As does this
# typed: true
require "sorbet-runtime"

class A
  def self.foo(x = 1, y)
  end
end

class B < A
  extend T::Sig

  sig { params(z: String).void }
  def self.foo(z)
  end
end

B.foo("hi")

Would you consider that a bug?

@jez not sure if you saw ^ but let me know if you'd like me to add that to https://github.com/sorbet/sorbet/tree/master/gems/sorbet-runtime/test somewhere

@ghiculescu I think this is the call that is ultimately causing that error.
In your first example, A#foo ends up being a super method of B.foo (due to extend A) and when Sorbet runtime tries to process the super method, it can't deal with the required params after optional params. Similarly, in the second example, A.foo is the super method of B.foo.
This is arguably a bug, as in, theoretically, it can be better handled by Sorbet runtime, wrt to super methods.

Yes, I think you're right. It sounds like maybe we want to add some more scenarios to this path?


2-4-stable missing latest 3-0-stable niceties

Changelog, cache key compatibility fix, and a bugfix for the installer:
https://github.com/railsdog/spree_avatax_certified/commit/541dfe18ee6b34da0250adcdbbda4043a3df5a95
https://github.com/railsdog/spree_avatax_certified/commit/88bf8f8d88067f9bfe54460144cbdb48a654157a
https://github.com/railsdog/spree_avatax_certified/commit/975349362e8682b29f072a44e6d0f50ca9cbc5fa

cc @acreilly you mentioned needing to retrofit some of these.

I recommend versioning the next update to 2-4-stable as 2.4.1 per the spree-style and semver numbering hybrid system I'm trying out.


component fails to show: cannot load svg 404 (Not Found)

Hi,
cannot get the card working, here the error I get in console:
GET http://<IP_ADDRESS>:8123/local/ha-card-weather-conditions/icons/static/cloudy.svg 404 (Not Found)
Image (async)
(anonymous) @ ha-card-weather-conditions.js:270
setTimeout (async)
(anonymous) @ ha-card-weather-conditions.js:270
Ct @ ha-card-weather-conditions.js:270
(anonymous) @ ha-card-weather-conditions.js:381
ha-card-weather-conditions.js:381  WEATHER-CONDITION-CARD  1.9.6 use HACS path to retrieve icons.
ha-card-weather-conditions.js:270 Icons issue. States: icons_model=climacell - isDay=true - condition: na.
:8123/lovelace-doma/default_view:1 GET http://<IP_ADDRESS>:8123/local/community/ha-card-weather-conditions/icons/animated/undefined.svg 404 (Not Found)

on network tab I see some resources are loaded from correct location:
http://<IP_ADDRESS>:8123/local/community/ha-card-weather-conditions/icons/static/cloudy.svg
also with pollen enabled on card nothig shows with following exception in console:
ha-card-weather-conditions.js:335 Uncaught (in promise) TypeError: Cannot read property 'attributes' of undefined
    at Ht (ha-card-weather-conditions.js:335)
    at ha-card-weather-conditions.js:395
    at HTMLElement._render (ha-card-weather-conditions.js:399)
    at HTMLElement.render (ha-card-weather-conditions.js:390)
    at HTMLElement.update (ha-card-weather-conditions.js:1)
    at HTMLElement.performUpdate (ha-card-weather-conditions.js:1)
    at HTMLElement._enqueueUpdate (ha-card-weather-conditions.js:1)

card configuration is pretty much the one from the full card example
any hints? ty

Post your card config and sensors config. There is likely something wrong in them.

Hi, the calls:
GET http://<IP_ADDRESS>:8123/local/ha-card-weather-conditions/icons/static/cloudy.svg
GET http://<IP_ADDRESS>:8123/local/community/ha-card-weather-conditions/icons/static/cloudy.svg

are used to detect the card location. Only one of them catch the exception, the other contains the right card path.
Try to put here your card configuration to have a support.
Thanks,
Renato

hi, here the card, it should be one copied from tutorial with some custom sensor values
type: 'custom:ha-card-weather-conditions'
animation: true
air_quality:
  co: sensor.cc_co
  epa_aqi: sensor.cc_epa_aqi
  epa_health_concern: sensor.cc_epa_health_concern
  no2: sensor.cc_no2
  o3: sensor.cc_o3
  pm10: sensor.cc_pm10
  pm25: sensor.pm2_5_1
  so2: sensor.cc_so2
weather:
  current:
    current_conditions: sensor.cc_weather_condition
    feels_like: sensor.windchill
    forecast: true
    humidity: sensor.humidity
    precipitation: sensor.rain_rate
    pressure: sensor.absolute_pressure
    sun: sun.sun
    temperature: sensor.outdoor_temperature
    visibility: sensor.cc_visibility
    wind_bearing: sensor.wind_direction
    wind_speed: sensor.wind_speed
  forecast:
    icons:
      day_1: sensor.cc_weather_condition_0d
      day_2: sensor.cc_weather_condition_1d
      day_3: sensor.cc_weather_condition_2d
      day_4: sensor.cc_weather_condition_3d
      day_5: sensor.cc_weather_condition_4d
    precipitation_intensity:
      day_1: sensor.cc_max_precipitation_0d
      day_2: sensor.cc_max_precipitation_1d
      day_3: sensor.cc_max_precipitation_2d
      day_4: sensor.cc_max_precipitation_3d
      day_5: sensor.cc_max_precipitation_4d
    precipitation_probability:
      day_1: sensor.cc_precipitation_probability_0d
      day_2: sensor.cc_precipitation_probability_1d
      day_3: sensor.cc_precipitation_probability_2d
      day_4: sensor.cc_precipitation_probability_3d
      day_5: sensor.cc_precipitation_probability_4d
    temperature_high:
      day_1: sensor.cc_max_temperature_0d
      day_2: sensor.cc_max_temperature_1d
      day_3: sensor.cc_max_temperature_2d
      day_4: sensor.cc_max_temperature_3d
      day_5: sensor.cc_max_temperature_4d
    temperature_low:
      day_1: sensor.cc_min_temperature_0d
      day_2: sensor.cc_min_temperature_1d
      day_3: sensor.cc_min_temperature_2d
      day_4: sensor.cc_min_temperature_3d
      day_5: sensor.cc_min_temperature_4d
  icons_model: climacell

ty

i have the same issue, but no card defined jet, as i can't finde the card inside the integration menu, but can load the icons on the browser... so its there, but not loading.

Same here

Same to me

I have the same issue. Are there any fixes for this?

same here.

i copy the SVG files from Repoitory to the folder www/ha-card-weather-conditions/icons/static/


Depth Scale Consistency

Thanks for sharing this work. No Bugs to report here, but I have a question about depth scale consistency.
Suppose that the model predicts the depth map of frame 0 and frame 1 as depth 0 and depth 1. Are depth 0 and depth 1 consistent with each other in the global map?
In a simple case, imaging that there is only one plane vertical to the principle axis in the scene and the relative motion between frame 0 and frame 1 is [0,0,-1] (forward 1 m). If the max_depth in depth 0 is 25,  the max_depth of depth 1 must be 24, right?
Appreciate your reply.

Yes if all the depth values are estimated correctly. Actually one of the post filtering criteria is to check whether the estimated depths are multiview consistent.
BTW the simple case you mentioned is difficult for mvs. Normally we don't consider moving forward cases.

Thanks for your reply. I agree that moving foward cases are dfficult for MVS currently, I tried Vis-MVSNet on KITTI datasets. The results are acceptable but not as good as those on DTU or Temple and Tanks.


fix questions in focus area tables

we forgot to update the focus area tables when we changed the metric questions. This pull request fixes that.

merging my own pr because there was no comment against it.


Update deps to remove event-stream

See https://github.com/dominictarr/event-stream/issues/116

I'll rebase onto 'add-own-walkthroughs' branch once merged to master

👀


switch to github actions for CI

working on #321

Codecov Report

Merging #323 (60ed282) into master (91441fe) will increase coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #323      +/-   ##
==========================================
+ Coverage   98.44%   98.54%   +0.09%
==========================================
  Files           7        7
  Lines         387      411      +24
  Branches       39       55      +16
==========================================
+ Hits          381      405      +24
  Misses          3        3
  Partials        3        3

Impacted Files
Coverage Δ

bayes_opt/target_space.py
100.00% <ø> (ø)

bayes_opt/util.py
97.63% <100.00%> (+0.29%)
:arrow_up:

bayes_opt/bayesian_optimization.py
99.07% <0.00%> (+0.03%)
:arrow_up:

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 3076d38...60ed282. Read the comment docs.


Investigate actions that make the browser flicker

When I Share Note and click Save, the browser flickers a few times (just something to note, not a fatal flaw)
When I advance to answering questions in a new section w/o saving and click through the modal that prompts me to Save previous answers, I observe the same browser flickering

@stephaniesimms is this still an issue with now that we have redesigned the page?

nope. not that i've noticed in the past few weeks of testing at least. if you don't see flickering either it's probably safe to close @briri

Haven't noticed any flickering either. Closing out


Implementation of account management

This section includes transfer, update, rollback

@yeganemehr
I want to create user Factory for use in feature test but I doesn't access.what should i do?


Sampling annotations at low power -- ideal?

When displaying annotations at low power, they are sampled to prevent cluttering the field. This is alright when thinking about point annotations, but starts to make less sense when considering polygonal annotations. In one project where I use HIstomicsTK to inspect segmentation results on a whole-slide level, "pockets" of absent boundaries appear at low power, like so:

Then when I zoom in, I realize they were just not chosen for display:

This makes is quite difficult to review results on low power to look for places with misclassification, as the viewer is not certain whether this is due to annotation sampling or misclassification. My suggestion is to use static overlays at low power (i.e. images of the annotation field) which gets just resize at different magnification levels, until a critical threshold (potentially controlled by the user) when the actual polygons are rendered and not a static image. I realize this is a complex process, but it will solve some of the key performance issues that slow down the rendering considerably as the number of annotation polygons goes up to the tens of thousands. Currently, the only way I can review results without lag in performance is to post the annotation polygons into multiple documents and to view them one by one. With an overlay (see issue #577 ), that would not be needed, and I can just view everything at low power and only interact dynamically with the elements when I zoom to an area of interest.
There's definitely pros and cons to both the existing and suggested method.

This has got us before when looking at nuclei detection algorithms and I don't think we have an elegant solution. I feel like we should be able to guarantee full rendering of annotations beyond a certain zoom level.

@manthey Just to summarize what we discussed in today's meeting:

Heatmaps: Bad idea because there are multiple styles (red, blue, green, etc), each would require its own heatmap which is too complex.

Static overlays: Good idea but very time consuming and complex to implement. Potantially could make things very fast because the overlay would be rendered on the server side and served to the user in tiles just like the whole slide image itself. The problem is that the extra rendering on the server side would put load on the server and would require the server to use a headless browser compatible with the client. It will need a lot of infrastructure.

Showing centroids: Most practical solution and potentially very scalable too, so this is the option we're going with for the time being.

Closed by: https://github.com/girder/large_image/pull/371


WIP: Rust Proxy

Tracking PR for the Cross Platform work. (#92)
Still To Do:

[x] change installation process to cargo wix for Windows
[x] change over CI
[x] Add kill option and timeout
[x] Add no launch
[x] Validate timeout flag and options
[x] Support --allowDialogs
[x] Fix logging - simpler output and filter for only g-cli messages.
[ ] Ctrl-C handling
[x] Problem with file path expansion to tools dir (has ./ in the middle)
[x] Possible problem with absolute paths still

Note: Linux support has been seperated to #115 and MacOS to #113 .
This PR is just to get to WIndows Running.

Leaving Ctrl-C handling until after an architecture change which will support this better.

This so close!
Just one last issue to getting this merged. There is a race condition between the monitoring thread and the main thread. If the process has exited before we try and stop it ourselves then we get an error because the monitoring thread has already stopped.
Need to either:

Artificially hold the monitoring thread open until we send stop anyway.
Add a signal that it has already shutdown.

I'm probably inclined to hold the thread as it seems simpler and removes other potential race conditions with the signal in reverse.


[Question] am I misinterpreting the use case for .on('stop')?

Hello, @contra and gulp contributors - thanks for the amazing tool.
I am trying to take advantage of the benefits of gulp programmatically, as a part of a larger build process. This issue may be better suited for gulp-sass so I apologize if that is the case, but from my perspective it relates to my understanding of gulp.dest and the Orchestrator events.
Here is an illustration:
// Store the working directory before running anything
var CWD = process.cwd();

var gulp = require('gulp');
require('some/other/location/gulpfile');

// fired when all tasks complete??
gulp.on('stop', function() {
  process.chdir(CWD); // restore cwd to what it was before calling gulp.start
  resolve(); // some promise
});

gulp.start('build');

// gulpfile.js

// Set up the build path relative to the working directory
var buildPath = path.resolve('build');

// Change working directory to the location of the gulpfile, else gulp will not work
process.chdir(path.resolve(__dirname));

gulp.task('build', ['copyCSS']);
gulp.task('copyCSS', copyCSS);

function copyCSS() {
  var files = ['node_modules/normalize.css/normalize.css'];
  gulp.src(files).pipe(gulp.dest(buildPath + '/css'));
}

The issue is, the gulp.on('stop') callback appears to be fired before the files are actually written to disk. All output indicates success, but the physical files were not copied.
A small adjustment works around this observation:
gulp.on('stop', function() {
  setTimeout(function() {
    process.chdir(CWD);
    resolve();
  }, 20); // have to tinker with this number, its not an event loop thing
});

However, as it appears to be a timing issue, it is more likely related to my flawed understanding of the toolset.
Please advise.

You aren't returning your stream, so the task management system doesn't know when the streaming is complete.  However, I don't recommend using the events or gulp.start because neither are gulp-documented functionality and will be completely removed/changed in gulp 4.

oh man. haha thanks. sorry to waste your time.


Update ERC20.sol syntax, batch transfers, permit() / Add ERC20 extensions / Update Onboarding.sol to remove approve() check, use ConsenSys pattern for ether transfer

conform syntax / add batch transfer function

I'll rework with truffle (but wanted to flag some basic nits)

Since it has been open for a long time now and the tests still don't pass I think it's best to close it. @Ro5s I propose you open a new one (maybe directly from this repo) with a PR that is all green then I can review and merge it


What version(s) of Octopus has this been developed against/tested for?

Developed against 3.3.11, and tested against 3.3.11 to 3.3.22.


Galaxy S10 focus issue

I implemented quagga on retroplace.com to scan barcodes of video games.
The implementation works fine but with the Galaxy S10, it seems impossible to focus on any object (using the camera normally ie. not in a web app works fine, though).
I heard there will be an image capture API that will allow to set constraints regarding the focus of mobile phones. Is quagga still active enough to implement this?

You can specify constraints within your call to Quagga.init, such as:
        const constraints = {
            width: {
                min: 640,
            },
            height: {
                min: 480,
            },
            aspectRatio: {
                min: 1,
                max: 2,
            },
            focusMode: 'continuous',
            ...(!this.props.cameraId && { facingMode: 'environment' }),
            ...(this.props.cameraId && { deviceId: this.props.cameraId }),
        };

        Quagga.init({
            inputStream: {
                type: 'LiveStream',
                target: document.querySelector('#scannerViewport'),
                constraints,
            },
            locator: {
                patchSize: 'medium',
                halfSample: true,
            },
            numOfWorkers: window.navigator.hardwareConcurrency || 2,
            decoder: {
                readers: ['upc_reader', 'ean_reader'],
            },
            locate: this.props.camScanLocate,
            area: {
                top: '25%',
                right: '0%',
                left: '0%',
                bottom: '25%',
            },
        }

... which is exactly what I am using in my app.
Looking at the documentation, I'm not seeing "focusMode" as a valid constraint, I'm wondering where I picked that up at.
... well, it seems to be a proposed capability, per https://w3c.github.io/mediacapture-image/index.html#FocusMode
It does seem like it would be a good idea to play with getConstraints() and see what is available before attempting to apply them, but it hasn't seemed to have caused side effects.  I have no idea if Chrome on Android supports this focusMode constraint presently or not.
If you do any specific testing, I'd love to hear of your results.

Did you find any solution?

I had the same problem and the cause in my case was that there are multiple cameras on the Samsung S10 (and other phones), and the default selected one is like a macro camera without autofocus.  In my case I selected a different camera, and that solved my problem.
I initially selected the last camera in the camera list, and that has been satisfactory for my case.
I tried using constraints with "focusMode", but that had no effect.  I do filter on "back" and "front" camera to get a camera that fits the desired function.

Donv is right. I had this issue for quite a while. It is selecting the depth sensor camera by default. Instead of selecting the last camera on the list (The note 10, and s10+ seem to have a different order, so the list solution did not work. Also some of the OnePlus devices do the same.) I changed the scanner resolution from the default 480 to 1281. this make quagga skip the lower res cameras and go straight to the rear regular camera. I am hoping for chrome to release a native for this. They have toyed around with the idea of letting you set default cameras, but haven't yet. I played around with a few different ways of pulling the data hoping to be able to set a default camera, but with device IDs changing on every load i have had no luck. Resolution is probably the best fix, albeit hackish.

deviceIds changing every load sounds quite strange.  My LG G6 has multiple rear cameras, one wide angle, one not, but I provide the user an option to select camera based upon the results of Quagga.CameraAccess.enumerateVideoDevices(), and I store the selected deviceId in localStorage, and restore it at start.  Have had no problems with that method -- although you do make me wonder if adjusting the resolution would be a good idea towards finding the correct one when one hasn't been selected yet!

Eric - Could you please share how you pull the ID and select camera based on that? I think resolution is great for picking it first, but for people who want to switch from front 1 to another

Sure, I'm using a React app with Redux, but basically at application startup, I fire off an action that does
const videoDevices = await Quagga.CameraAccess.enumerateVideoDevices();
videoDevicesReceived(videoDevices);

videoDevicesReceived() stores a list of the deviceIds and device names returned from enumerateVideoDevices.  In the settings for the application, the user can select which camera deviceId to use (associating the device name with the id), and then whenever I call Quagga.init() I have in the config object
constraints: {
    ...defaultConstraints,
   ...(cameraId && { deviceId: cameraId }),
   ...(!cameraId && { facingMode }),
 }

if cameraId is provided, then it adds deviceId: cameraId to the constraints, if it is not provided, it adds facingMode to the constraints.
oh, hey, now that i'm looking at that, i can make it even shorter
    ...(cameraId ? { deviceId: cameraId } : { facingMode })

should work

Donv is right. I had this issue for quite a while. It is selecting the depth sensor camera by default. Instead of selecting the last camera on the list (The note 10, and s10+ seem to have a different order, so the list solution did not work. Also some of the OnePlus devices do the same.) I changed the scanner resolution from the default 480 to 1281. this make quagga skip the lower res cameras and go straight to the rear regular camera. I am hoping for chrome to release a native for this. They have toyed around with the idea of letting you set default cameras, but haven't yet. I played around with a few different ways of pulling the data hoping to be able to set a default camera, but with device IDs changing on every load i have had no luck. Resolution is probably the best fix, albeit hackish.

Can you please share your configuration for having Quagga auto-select based on resolution?


Optional app name and path

Add support for dynamic Backbone application names, sub directories, and .hbs templates.
Why?
The naming conventions are excellent and so are the generators, however, it causes a problem when you need multiple Backbone Apps in a single project. This happens when you have a public design and a CMS design
Generator options added:
# Custom Appname
rails generate backbone:install --appname=CustomApp

# Place code within a sub directory structure
rails generate backbone:install --dir=custom_app

# Create a custom app name
rails generate backbone:scaffold planet -a=CustomApp

# Generate scaffold in sub directory of assets/javascripts & assets/templates
rails generate backbone:scaffold planet -d=custom_app

# Use Handlebars File instead of JST
rails generate backbone:scaffold planet --tpl=hbs

# Use SHT instead of JST as JS Object
rails generate backbone:scaffold planet --tpl_controller=SHT

Tests added & passing
I did not bump versions -- is it possible we can bump version and get this on rubygems.org instead of having a git repo?

Thanks, this looks great. I made some comments inline.

Absolutely. I'll send another commit in a few hours

All set. I did leave the 2 documentations in there though because it is showing that both generators carry this option

@meleyal all set

Released <IP_ADDRESS>: https://rubygems.org/gems/backbone-on-rails


Connection issues for remote port forwarding

Hey,
I use remote port forwarding with tls and auth. It seems that not all connections are successful if hitting port repedately. Here's how to reproduce:
server (with cert.pem and key.pem)
./gost -logtostderr -v=5 -L=socks://admin:password@<IP_ADDRESS>:8080

client
python -m SimpleHTTPServer 8000
./gost -logtostderr -v=5 -L=rtcp://:2222/<IP_ADDRESS>:8000 -F=socks://admin:password@<IP_ADDRESS>:8080

then:
while ~; do curl $SERVER_IP:2222; done

here's the output:
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>Directory listing for /</title>

Directory listing for /

ca-key.pem

ca.csr

ca.pem

csr.json

gost

curl: (7) Failed to connect to <IP_ADDRESS> port 2222: Connection refused
curl: (7) Failed to connect to <IP_ADDRESS> port 2222: Connection refused
curl: (7) Failed to connect to <IP_ADDRESS> port 2222: Connection refused
curl: (7) Failed to connect to <IP_ADDRESS> port 2222: Connection refused
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>Directory listing for /</title>

Directory listing for /

ca-key.pem

ca.csr

ca.pem

csr.json

gost

curl: (7) Failed to connect to <IP_ADDRESS> port 2222: Connection refused
curl: (7) Failed to connect to <IP_ADDRESS> port 2222: Connection refused
curl: (7) Failed to connect to <IP_ADDRESS> port 2222: Connection refused
curl: (7) Failed to connect to <IP_ADDRESS> port 2222: Connection refused

Here are server logs: http://pastebin.com/ecVBtJ7n
Here are client logs: http://pastebin.com/E56gr9FH
Any idea what is wrong? Why is socks5 connection closing and opening? Why gost doesn't keep the connection alive?

Actually the same happens when not using tls or auth

It might be, that some latency between client and server is required or reproducing this bug.

I've managed to fix it by proxying socks5 connection through socks5 connection...
server
./gost -logtostderr -v 5 -L=socks://<IP_ADDRESS>:8080
./gost -logtostderr -v 5 -L=tcp://:2222/localhost:8000 -F=socks://localhost:5000

client
python -m SimpleHTTPServer 8000
./gost -logtostderr -v=5 -L=socks://:5000
./gost -logtostderr -v=5 -L=rtcp://:5000/<IP_ADDRESS>:5000 -F=socks://<IP_ADDRESS>:8080
while ~; do curl <IP_ADDRESS>:2222; done

But this seems like a hacky approach and the latency is terrible... Could you please try to fix this bug, so I can prevent such hacks? Thank you!

There needs some tricks: https://groups.google.com/d/msg/go-gost/ouzBXF0Fqk8/c0Wn9_ykCgAJ

Thanks for the response! That's actually the same solution I've came up with in the comment above (with the difference I forward tcp connection instead of http connection), but it still seems hacky :(
Couldn't gost automatically setup persistent connection? I'd imagine that running following:
gost -L rtcp://:1443/localhost:8080?persistent=true -F socks://<IP_ADDRESS>:1080

would actually automatically do behind the scenes the equivalent of following on client side:
gost -L socks://localhost:RANDOM1
gost -L rtcp://localhost:RANDOM2/localhost:RANDOM1 -F socks://<IP_ADDRESS>:1080

and following on server side:
gost -L tcp://:1443/localhost:8080 -F socks://localhost:RANDOM2

where RANDOM1 and RANDOM2 are temporary random port allocated by gost (ideally it could be implemented in-memory without allocating these two extra ports). Please notice the persistent=true parameter for rtcp connection.
Is it reasonable?

There is a difference between the trick you provided and mine. You expose a SOCKS5 proxy, but it still need to re-bind when a connection is established, in fact it's the same as the original method.
HTTP2 supports multiplexing several streams over a single connection, so when forwarding through HTTP2 proxy, only one connection holds, this is analogous to the persistent param in your proposal.
I admit the better option is to implement persistent connection directly, but it may need to make significant changes to the socks5 protocol, or change to other protocol that supports multiplexing.

It's not entirely the same, as the second socks server on server side:
gost -L tcp://:1443/localhost:8080 -F socks://localhost:RANDOM2

doesn't do reverse port forwarding, but rather forward port forwarding.
It seems that forward port forwarding with socks has no issues with short-lived sessions, as I experience issues only for short-living reverse port forwarding. I hope you get what I mean :)

Yes, I understand what you said. A single persistent connection for reverse port forwarding needs the connection support multiplexing (such as HTTP2), but SOCKS does not have this feature, so the short-lived sessions (HTTP) does not work, you will get many connection refused errors when there are multiple concurrent HTTP requests.

Okay then, I'll leave it up to you what to do with this issue. Feel free to close it.

btw. I was under impression that forward tcp forwarding requires SOCKS5 proxy, but it seems it's only the case for UDP forwarding. Maybe you could consider following alternate version:

Running following:
gost -L rtcp://:1443/localhost:8080?persistent=true -F socks://<IP_ADDRESS>:1080

would automatically do behind the scenes the equivalent of following on client side:
gost -L http2://localhost:RANDOM1
gost -L rtcp://localhost:RANDOM2/localhost:RANDOM1 -F socks://<IP_ADDRESS>:1080

and following on server side:
gost -L tcp://:1443/localhost:8080 -F http2://localhost:RANDOM2

where RANDOM1 and RANDOM2 are temporary random port allocated by gost (ideally it could be implemented in-memory without allocating these two extra ports).

Maybe the SSH protocol can be used to achieve this goal, I will check it later.

The latest version (v2.4-dev) has support the ssh tunnel for tcp port forwarding, it works well for short-live connections.
server:
gost -L ssh://:2222
client:
gost -L rtcp://<IP_ADDRESS>:8080/<IP_ADDRESS>:80 -F ssh://server:2222

I get following issue then trying to go get gost from 2.4 branch ;/
github.com/lucas-clemente/quic-go/h2quic
../../lucas-clemente/quic-go/h2quic/request_writer.go:64: undefined: httplex.PunycodeHostPort

All third-party dependent libraries are in the cmd/gost/vendor directory, you just need to clone the repo:
$ git clone https://github.com/ginuerzh/gost.git
$ cd cmd/gost
$ go build

I also released the binary files of dev version: https://github.com/ginuerzh/gost/releases/tag/v2.4-dev

@ginuerzh I'm testing your implementation :) So far one bug:
./gost -L rtcp://:8001/<IP_ADDRESS>:8000 -F ssh://admin:password@localhost:2222

I0309 15:51:12.750002   96798 ssh.go:146] [ssh-rtcp] listening tcp <nil>:8001
I0309 15:51:12.790475   96798 ssh.go:149] [ssh-rtcp] listen tcp: lookup <nil>: no such host

but following is ok:
./gost -L rtcp://<IP_ADDRESS>:8001/<IP_ADDRESS>:8000 -F ssh://admin:password@localhost:2222

Also, performing the trick we used for socks5 connections seems to be beneficial for ssh reverse tunnel as well, as without it ssh seems to allocate new port upon each connection, like so:
I0309 15:22:34.629725   37230 ssh.go:203] [ssh-rtcp] <IP_ADDRESS>:64148 <-> <IP_ADDRESS>:8001
I0309 15:22:34.670108   37230 ssh.go:205] [ssh-rtcp] <IP_ADDRESS>:64148 >-< <IP_ADDRESS>:8001
I0309 15:22:34.792436   37230 ssh.go:203] [ssh-rtcp] <IP_ADDRESS>:64150 <-> <IP_ADDRESS>:8001
I0309 15:22:34.830017   37230 ssh.go:205] [ssh-rtcp] <IP_ADDRESS>:64150 >-< <IP_ADDRESS>:8001
I0309 15:22:34.949684   37230 ssh.go:203] [ssh-rtcp] <IP_ADDRESS>:64152 <-> <IP_ADDRESS>:8001
I0309 15:22:34.985315   37230 ssh.go:205] [ssh-rtcp] <IP_ADDRESS>:64152 >-< <IP_ADDRESS>:8001
I0309 15:22:35.105746   37230 ssh.go:203] [ssh-rtcp] <IP_ADDRESS>:64154 <-> <IP_ADDRESS>:8001
I0309 15:22:35.142539   37230 ssh.go:205] [ssh-rtcp] <IP_ADDRESS>:64154 >-< <IP_ADDRESS>:8001
I0309 15:22:35.267390   37230 ssh.go:203] [ssh-rtcp] <IP_ADDRESS>:64156 <-> <IP_ADDRESS>:8001
I0309 15:22:35.303185   37230 ssh.go:205] [ssh-rtcp] <IP_ADDRESS>:64156 >-< <IP_ADDRESS>:8001
I0309 15:22:35.425128   37230 ssh.go:203] [ssh-rtcp] <IP_ADDRESS>:64158 <-> <IP_ADDRESS>:8001
I0309 15:22:35.425279   37230 ssh.go:205] [ssh-rtcp] <IP_ADDRESS>:64158 >-< <IP_ADDRESS>:8001


USVString should handle type conversion

It appears some types while completely fine used in FormData are being flagged as incompatible by Flow.
let form = new FormData();
form.append('active', true);
form.append('id', 12);

Both value types result in an error (including just the boolean error for example purposes):
115:     form.append(name, value);
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ call of method `append`. Function cannot be called on any member of intersection type
115:     form.append(name, value);
         ^^^^^^^^^^^^^^^ intersection
  Member 1:
  271:     append(name: string, value: string): void;
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ function type. See lib: /tmp/flow/flowlib_1c0b6bae/bom.js:271
  Error:
  115:     form.append(name, value);
                                      ^^^^^ boolean. This type is incompatible with the expected param type of
  271:     append(name: string, value: string): void;
                                       ^^^^^^ string. See lib: /tmp/flow/flowlib_1c0b6bae/bom.js:271
  Member 2:
  272:     append(name: string, value: Blob, filename?: string): void;
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ function type. See lib: /tmp/flow/flowlib_1c0b6bae/bom.js:272
  Error:
  115:     form.append(name, value);
                                      ^^^^^ boolean. This type is incompatible with the expected param type of
  272:     append(name: string, value: Blob, filename?: string): void;
                                       ^^^^ Blob. See lib: /tmp/flow/flowlib_1c0b6bae/bom.js:272
  Member 3:
  273:     append(name: string, value: File, filename?: string): void;
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ function type. See lib: /tmp/flow/flowlib_1c0b6bae/bom.js:273
  Error:
  115:     form.append(name, value);
                                      ^^^^^ boolean. This type is incompatible with the expected param type of
  273:     append(name: string, value: File, filename?: string): void;
                                       ^^^^ File. See lib: /tmp/flow/flowlib_1c0b6bae/bom.js:273

Having the same problem as @NullDivision


int vs float in data

Make sure that int and float columns are implemented correctly in the simulated data frame.
@boryana-ilieva , here is what I had in mind for the missing data. Please see if this fits. https://pandas.pydata.org/pandas-docs/stable/user_guide/integer_na.html#integer-na
Originally posted by @peisenha in https://github.com/OpenSourceEconomics/soepy/pull/92#issuecomment-590350211

This is done.


Fix Metrics error on Folia server

Metrics uses a runtask which no longer works on Folia.

Disabled metrics completely


Addition of analysis of non-FISH signal in regions defined by FISH signal

Same idea as final step initially developed for DNA DSB project
Keep 1 row per spot, 1 column group per molecule of interest x  measurement (e.g., mean RAD51, standard deviation RAD51)
Use, by default, just the central plane +/- 1 $z$-slice

Checklist:

[x] for regional spots
[ ] for locus-specific spots
[ ] different formats of output (#379)
[ ] proper handling of units of measure
[ ] ensure robustness when a ROI is near an image boundary: https://github.com/gerlichlab/gertils/issues/34
[ ] fine drift correction for signal image(s): #343
[ ] support for multiple timepoints in which signal should be analysed
[ ] simplify the collection of statistics which are computer (in particular, choose just 1-2 ways of dealing with the $z$ dimension, given that fundamentally we're defining the ROI as a 2D entity for the purpose of signal analysis)

For the locus-specific spots, consider whether or not we're also re-measuring pixel values in regional spots (particularly if the same ROI diameter is used for regional and for locus-specific spots), as well as the units of measure! In particular, we need to ensure that the ROI size is specified with units, and that we convert as necessary based on which of the fields of the locus spot records are being accessed in order to define the location of each region. We should probably prefer to continue to work in pixels for the specification of the ROI size/diameter, since we'll use the region center and size in order to define a region (in pixels) from which to extract pixel values and over which to compute statistics.

From a first glance, I think it would be great to have an "average aggregation" of the IF signal, with the modification where we can input the specification of the +/- N step relative to the z center; and not the whole z stack at the given ROI. The idea would be that we then get the mean intensity of the signal over +/- 2-3 slices over the z.
However, depending on how the images look like and the step size (@TLSteinacker you must know this better than me) maybe it's not even necessary and then taking the "exact aggregation" would be enough (since this is what I am using for my analyses with the reparafil pipeline and it gives reliable results).

Thanks @ines-prlesi
Other ideas from discussion:

If a user's specifying a $z$ depth, this could be interpreted either as an absolute value or as a percentage of the total available $z$ depth
If $z$ depth is interpreted absolutely, then care should be taken that units' space is clear/explicit (i.e., image space and units in pixels, or physical space and units as e.g. nanometers)

@vreuter thanks for initiating this discussion
@ines-prlesi what parameters are you currently using? have you tested different ones and how do they change the relative outcome?
I agree that the whole z would not be good, and think that either a selection of z slices (~2-3) or the calculation of the sphere might be preferential.

Thanks @TLSteinacker , what's implemented for the DNA DSB project didn't give the user any flexibility, but rather computes all three of these (central slice, average, and max-projection). The reason I'd like to parameterize this, though, is because there are 5 numbers computed for each method (min, max, median, mean, and standard deviation), so the number of values is already relatively large, and for reasons I think we've discussed together or in pairs, the most used of these methods is the central-plane one.
So the "sphere" is a new idea, but of the previous three I think there's consensus to keep the central plane one but provide flexibility to add a +/- n value, therefore in total having the 2D ROI area x ($2n + 1$) layers of $z$ as the voxel in which to compute the summary statistics.

@ines-prlesi @TLSteinacker how about having either of the following be valid, then...

Option for sphere, with a single parameter value representing either the radius or diameter a sphere in $z$
Option for rectangular prism, with single value representing either the "height" of the box (number of $z$ units), or the half-width?
?

At least one would be required, and then either we could allow one-and-only-one, or allow both and then use both methods, differentiating column names by, e.g., a __box or __sphere suffix.
Here could be the format of a valid specification...
{
    "shape": shape
    dimension: "n px",
}

where $n$ is a positive integer, $shape \in ("sphere", "box")$, and $dimension \in ("sphere", "box")$
This would of course be a particular choice balancing expressiveness/flexibility, user burden, and clarity. WDYT? Do you prefer something else to differently balance these tradeoffs?

Sounds like a good plan to me! Regarding ' with single value representing either the "height" of the box (number of z units), or the half-width?', would there not be one value for the xy size and one for z? Or is xy anyway already specified somewhere else?

would there not be one value for the xy size and one for z? Or is xy anyway already specified somewhere else?

Indeed, sorry, I was operating with the "diameter (xy) is already set" model in my head since that's currently the case, but you're right @TLSteinacker  we'd need separate values for xy.
Another thing just occurred to me...we began by saying +/- a margin from central plane in $z$, and with a diameter in the xy plane. I then started talking about a sphere, but actually we'd have an ellipsoid since no constraint that the $z$ depth match the diameter in $xy$. In fact, I acknowledge it's a bit nonsense to talk this way anyhow since the $xy$ units are pixels in a way that $z$ are not. So how about this...
{
  "x": "<x> px",
  "y": "<y> px",
  "z": <z>,
  "shape": shape
}

where the $x$, $y$, $z$ values are all populated by positive integers, with the "px" added to be clear that it's pixels, not a fixed physical unit. $z$ is left as a scalar to note the difference with the other two. $shape$ must be either "ellipsoid" or "box", and then the values are used accordingly to construct the actual volume of pixels over which summary stats of the pixel values are calculated. This leaves flexibility to define $x$ and $y$ separately in the event (even if rare) that the regions are known/expected to have more rectangular form in $xy$, at the small user burden of adding an extra key-value pair to specify. I'd favor interpreting these (implicitly, without additional specification) as side lengths (not half-widths) in the "box" case, and analogously, axis lengths in the "ellipsoid" case.
WDYT? @ines-prlesi @TLSteinacker

Sorry @vreuter and @TLSteinacker, I'm still new to github so I missed the notifications; however I read the whole thread and think that the final conclusion is good!


20150729 compatibility

This looks ok to me, but it's a bit unfortunate.
Can you update the title to better describe what this is about ("20150729 compatibility" is pretty obscure)?  It would also be nice to have the description explain why these missingProvide comments are here.

I am also interested to know why we need to add these @suppress {missingProvide} annotations.

With the new compiler, every classes tagged with @constructor must by provided with goog.provide.
We can either add a @suppress {missingProvide} annotation or provide the class even if it is not used.

+1 for adding the goog.provide. We'll need them anyway when compiling together with ol3.

ol.SelectEvent and ol.ModifyEvent should probably be renamed to ol.interaction.SelectEvent and ol.interaction.ModifyEvent

Very nice @fredj. LGTM.

Thanks for the review @elemoine.


Journal Entry Page is Not Styled when Opening Just that Page

When opening just the Journal Entry Page of a document, the content does not pick up the dnd5e style. Additionally, the journal-page-header is flex: 1 due to .window-app .window-content > * and there is no padding around journal-page-header nor journal-page-content.

This seems to be a module opening a journal page in a non-standard way, standard method for showing a single journal page to players results in it still opening within a full journal entry application.

I see, you are rendering the journal entry page sheet directly.

Yes, I thought it was a nice way of just showing the specific content I want the user to see rather than within the full journal. As it worked, I thought it was a purposeful, intended method, but just missing the styling. Specifically, I grab the document  from the UUID of the entry using fromUuid, then render the sheet.
Realise now that the lack of padding is most likely because it's not actually fully supported with core Foundry either.

Yes, I thought it was a nice way of just showing the specific content I want the user to see rather than within the full journal. As it worked, I thought it was a purposeful, intended method, but just missing the styling. Specifically, I grab the document from the UUID of the entry using fromUuid, then render the sheet.
Realise now that the lack of padding is most likely because it's not actually fully supported with core Foundry either.

Yeah, probably not the way to go about it. You could always set it up to use a rich tooltip and point them towards rule pages.

Thanks, I'll try that.
I'll also pop an issue over on Foundry VTT side as I think it'd be a nice option and it's almost working as is.


Pip install command doesn't work

When using the pip command located in the README file in Python (I'm using Python 3.7.1), I receive an error saying the following:
ERROR: Could not find a version that satisfies the requirement napari-nibabel (from versions: none)
ERROR: No matching distribution found for napari-nibabel

It looks like it was not uploaded to PyPI. You should be able to install with pip install git+https://github.com/nipy/napari-nibabel.git

That works! Thanks :)


Server never initializes if no workspaceFolders are given

The server never responds to the initialize request from the client, if it contains workspaceFolders: null.
From the LSP specs:
        /**
	 * The workspace folders configured in the client when the server starts.
	 * This property is only available if the client supports workspace folders.
	 * It can be `null` if the client supports workspace folders but none are
	 * configured.
	 *
	 * @since 3.6.0
	 */
	workspaceFolders?: [WorkspaceFolder](https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#workspaceFolder)[] | null;

@jwortmann thanks for reporting! I think you may have hit the problem that #68 addresses (which is about initialization). Please try https://github.com/artempyanykh/marksman/releases/tag/2022-09-02 and let me know if the initialization issue is gone.
When it comes to 'single file mode', some functionality like document symbols and reference links can work in a single file mode but the rest (workspace symbols, renames, cross-links, etc) really needs a "folder".
However, the important bit here is that the 'single file mode' is not a part of the LSP spec. I'm not sure how things are configured in sublime, but I'd recommend setting a folder for you markdown -- this'll give the best experience.
If you do care about the 'single file mode' feel free to open a separate issue specifically about this. I'm generally OK with the idea of having some custom bits to emulate 'single file mode', but since it's not part of the spec it's not very high on my priority list.

Hi @artempyanykh, with 2022-09-02 and single file mode, the server does start but I don't see it provide anything but just return [] and None for everything.

I believe by “single file mode” is meant that workspaceFolders is null in the initialize request, which by the spec is valid.

I can confirm what @jfcherng wrote.

However, the important bit here is that the 'single file mode' is not a part of the LSP spec.

Right, this kind of expression is not explicitly mentioned in the LSP specs, but neither is 'project mode' or 'workspace mode' or whatever. But if you look at the description for the Workspace folders request, there it is written that it

Returns null in the response if only a single file is open in the tool. Returns an empty array if a workspace is open but no folders are configured.

So at least they describe the possibility that only a single file is open in the editor. And there are servers like Microsoft's vscode-css-languageserver or vscode-html-languageserver which seem to operate exclusively in 'single file mode', i.e. they apparently don't care whether workspaceFolders (or rootUri) are provided in the initialize request or not. Probably because CSS and HTML are languages which usually don't have any strict project structure or configuration files associated with the source code. And I would expect the same for Markdown. For example it is clear that relative links to other documents should be interpreted relative to the current document. And I think it should be possible to provide most of the features, e.g. "Document Symbols" even without workspaceFolders.
Anyway, since the latest server version correctly responds to the initialize request, we can close this now. Thanks for the fix!

@jfcherng right now Marksman does need a root/folder and doesn't provide a single-file mode.
But now that I'm thinking a bit more about this, it seems that it'd be quite convenient and a better/less confusing UX overall. I've created a separate issue for this #69


feat: Qna recognizer

closes #3463

I played with this branch and tested a few core functionalities, which worked on my end.
Pasting my testing report here for reference.
You guys are awesome!!!

I played with this branch and tested a few core functionalities, which worked on my end.
Pasting my testing report here for reference.
You guys are awesome!!!

Thanks

There are two scroll bars.

There are two scroll bars.

fixed

Coverage decreased (-1.9%) to 55.123% when pulling cb8863568c0fb23e3278a722a9f514076a77cdb8 on qna-recognizer into 9e70f3229c0f240d30bd9eac0f2825be475902af on main.

When a bot is created from knowledge, the modal persists after you submit the form and it interact-able. The modal needs to go away when the user submits the form.

I tested using this PR. I was unable to get an answer after publishing the KB and tested in Emulator. Here are the steps I followed:

Create from knowledge (QnA Maker)
Enter a URL
Hit the error message

Go back to cmd and run set QNA_SUBSCRIPTION_KEY=MY_QNA_SUBSCRIPTION_KEY, yarn startall
Repeat the steps (1 and 2)
Was able to extract KB in Composer and publish extracted KB
Restart bot and tested in Emulator. Was unable to get answers.

I tested using this PR. I was unable to get an answer after publishing the KB and tested in Emulator. Here are the steps I followed:

Create from knowledge (QnA Maker)
Enter a URL
Hit the error message

Go back to cmd and run set QNA_SUBSCRIPTION_KEY=MY_QNA_SUBSCRIPTION_KEY, yarn startall
Repeat the steps (1 and 2)
Was able to extract KB in Composer and publish extracted KB
Restart bot and tested in Emulator. Was unable to get answers.

Did I miss anything?

@zxyanliu  You should use cmd than powershell in windows.

I tested this branch and went through the weather bot tutorials on August 24, 2020.
For what is covered in the weather bot tutorials, everything worked perfectly on my side except there is a minor issue when shifting b/w Regular Expression recognizer and Default recognizer.

Should this be verified against 4.10.1, rather than the daily?  (Daily builds are built from the main branch by default)

Can we please center these icons?

Should this also be verified against 4.10.1, rather than just the daily? (Daily builds are built from the main branch by default)

Updated to 4.10.1, but I found it didn't include the crosstrainrecognizer fix, we need to include that fix in 4.10.
https://github.com/microsoft/botbuilder-dotnet/commit/06153032d5e55e89ac10b7158d446f3c4abe452c


Dependency check tool doesn't handle post-beta breaking changes

We have the following scenario.
armmonitor depends on<EMAIL_ADDRESS>which added features for fakes/tracing.
We need to release<EMAIL_ADDRESS>for a new feature.  fakes/tracing are not included in this release (will return in v1.8.0-beta.1)
The PR for<EMAIL_ADDRESS>with the new feature, sans fakes/tracing, fails dependency check as armmonitor@latest fails to compile.
Not sure what the rule would be here to make dependency check pass.  Given that this should be a rare corner-case, it might just be simpler to add a flag to bypass dependency checks.

Another permutation.
github.com/Azure/azure-sdk-for-go/sdk/messaging/azeventgrid imports
github.com/Azure/azure-sdk-for-go/sdk/azcore/messaging: module github.com/Azure/azure-sdk-for-go/sdk/azcore@latest found (v1.7.0, replaced by ../../sdk/azcore), but does not contain package github.com/Azure/azure-sdk-for-go/sdk/azcore/messaging

Here, azeventgrid depends on<EMAIL_ADDRESS> If the version fetched by azcore@latest is semantically less that what's currently imported, then it should not perform the replacement.

I think that the rule should be, if a module depends on a version of azcore with a minor version greater than what's to be replaced, that we skip that module.

The above rule has a broken corner-case.  e.g. if<EMAIL_ADDRESS>contains features that aren't going to be released in<EMAIL_ADDRESS>then we still can't replace the beta version with the GA version.


xDS docs: clarify that nonce is per-resource-type in ADS

Commit Message: xDS docs: clarify that nonce is per-resource-type in ADS
Additional Description: Documentation fix.
Risk Level: Low
Testing: N/A
Docs Changes: Included in PR
Release Notes: N/A
Platform Specific Features: N/A
CC @ejona86

/retest


org service v2 and transition for SaaS customers

TBD - we need to discuss this. We need to merge old APIs and new APIs as necessary and enable transition in the SaaS supporting both for a time.
how do we migrate SaaS customers to org service v2? What APIs do we need to add to org service v2? How do we enable it in the template? Who’s using commits and user organizations in our SaaS customer base?
Looking at this document, I suggest we switch phase 4 and 5 around and then implement mod org devcenter show - this is not the right command btw, just we need something like this in both DX and SaaS.
https://docs.google.com/document/d/18mtebL5P2U8RCa8JYRbmkJQrZQGwFjhH8G-6VEg1XMQ/edit
Slack Message
### Tasks
- [x] remove deprecated query from v2
- [x] update template to use v2
- [x] achive temporary v2 template
- [x] add moderne/repos.csv
- [ ] add qbuzz/repos.csv
- [ ] use public/repos.csv + jl/repos.csv in public-mass-ingest
- [ ] use moderne/repos.csv in moderne-mass-ingest
- [ ] generate public/repos.csv inside organizations-multitenant
- [ ] generate moderne/repos.csv inside organizations-multitenant
- [ ] Secure organizations-multitenant
- [ ] change RE to not use `organizations(repositoryInput)` if `Organization` type has `repositories` field
- [ ] remove `organizations(repositoryInput)` on some date.yInput)` once all customers migrated.

using repos.csv inside mass-ingest:
https://github.com/moderneinc/public-mass-ingest/pull/10
https://github.com/moderneinc/moderne-mass-ingest/pull/3

Draft PR: https://github.com/moderneinc/moderne-organizations-multitenant/pull/129

original template is now updated to v2. Temporary template is archived. Documentation updated.

PR https://github.com/moderneinc/moderne-organizations-multitenant/pull/129 is now pending only on removing the public LB.

I think we can park generate public/repos.csv inside organizations-multitenant for now. We will probably do this only a few times a year so automating it is a probably a lower priority.

PR to disable v1 query if v2 is available. Should be merged after  https://github.com/moderneinc/moderne-organizations-multitenant/pull/129
https://github.com/moderneinc/moderne-recipe-execution/pull/925


Examples are not reproduced (lower accuracy)

Just installed NER from Gtihub. Run file training_example.ipynb from https://github.com/deepmipt/ner/blob/master/training_example.ipynb, then the results in example notebook looks bad:
Epoch 4
Eval on valid:
processed 54612 tokens with 5942 phrases; found: 314 phrases; correct: 13.

precision:  4.14%; recall:  0.22%; FB1:  0.42

Eval on train:
processed 217662 tokens with 23499 phrases; found: 1416 phrases; correct: 47.

precision:  3.32%; recall:  0.20%; FB1:  0.38

	LOC: precision:  2.36%; recall:  0.21%; F1:  0.39 635

	MISC: precision:  0.00%; recall:  0.00%; F1:  0.00 0

	ORG: precision:  8.45%; recall:  0.38%; F1:  0.73 284

	PER: precision:  1.61%; recall:  0.12%; F1:  0.23 497

Eval on valid:
processed 54612 tokens with 5942 phrases; found: 316 phrases; correct: 13.

precision:  4.11%; recall:  0.22%; FB1:  0.42

	LOC: precision:  0.69%; recall:  0.05%; F1:  0.10 144

	MISC: precision:  0.00%; recall:  0.00%; F1:  0.00 0

	ORG: precision:  4.23%; recall:  0.22%; F1:  0.42 71

	PER: precision:  8.91%; recall:  0.49%; F1:  0.93 101

Eval on test:
processed 49888 tokens with 5648 phrases; found: 258 phrases; correct: 15.

precision:  5.81%; recall:  0.27%; FB1:  0.51

	LOC: precision:  2.73%; recall:  0.18%; F1:  0.34 110

	MISC: precision:  0.00%; recall:  0.00%; F1:  0.00 0

	ORG: precision:  9.76%; recall:  0.24%; F1:  0.47 41

	PER: precision:  7.48%; recall:  0.49%; F1:  0.93 107

This result is wrong. Please help me!!

What is your tensorflow vrsion? Try tensorflow==1.3.0

Thanks for response. My tensorflow version is 1.7.0. I will try again with version 1.3.0