system.util.runPython

Allows users to run complete Python scripts directly from Ignition.
This would allow users to install any Python library on a local computer and pass data.
from java.lang import ProcessBuilder, String
from java.io import File

# Function to run a Python script using the specified file path and arguments
def runPython(python_script_path, python_executable= r'C:\Python\Python311\python.exe', *args):
    # Construct the command to execute the Python script
    command = [python_executable, python_script_path] + list(args)
    # Create a ProcessBuilder with the command
    pb = ProcessBuilder(command)
    # Start the process
    process = pb.start()
    # Read the output from the process
    output = process.getInputStream().readAllBytes()
    # Return the output
    return output

I will work on this in my free time; however, I wanted to post it for documentation.

This feels like a specialization of #11, but I'm not against it :)


Fitting terminates at around 10%

Description
I am able to run tutorial 0 example. However the training terminates before training is completed at around 10%
How to Reproduce

Go to Tutorial 0
Run fit for AdsGAN plugin

Expected Behavior
I expect the training to run until 10000 as per progress bar
Screenshots

System Information

OS: Mac OS (Apple M3)
OS Version: 14.5
Language Version: Python 3.10
Package Manager Version: Conda
Browser (if applicable): Safari
Browser Version (if applicable): [e.g. 81.0.4044.138]

Hi @ianlokh,
Thanks for your questions. This is expected behaviour and it due to early stopping. If you wish to force the model to train for longer you can pass a value for n_iter_min which means that early stopping will only kick in after n_iter_min training iterations.


Add package.json

This PR adds a package.json file so this module can be published on npm.
Documentation as been updated to reflect it.

Looks good, thanks.


[WIP] Add hub config for geospatial workshop

This PR adds a config file to deploy a hub for the Geospatial Workshop being held in Ghana later this month.
Done:

Deploys a daskhub
Uses Google authentication
list of admins
user limits for dask

Needs:

URL setup in NameCheap

related to: https://github.com/2i2c-org/pilot-hubs/issues/473

I have pushed a commit with folks gmail account for admin access (mostly copy-pasted from configs on the 2i2c cluster). If your email is not there and you'd like it to be, please feel free to add it!

I have tried to set some limits for singleuser and dask gateway by borrowing config from pangeo:

singleuser: https://github.com/pangeo-data/pangeo-cloud-federation/blob/d4f868e4d5c1ea92675facdabff86d49f31c7253/pangeo-deploy/values.yaml#L19-L25
dask gateway: https://github.com/pangeo-data/pangeo-cloud-federation/blob/e0b8afdd5571a8a1e68b4b7a33b789703a97ac7f/deployments/icesat2/config/common.yaml#L149-L154

Would love to know if I'm on the right lines with this in terms of (i) is this correct config? (ii) will it achieve Paige's desires in https://github.com/2i2c-org/pilot-hubs/issues/473#issuecomment-879492933?

In order to set the URL/DNS stuff, I tried deploying the hub with proxy.https.enabled = false using the deployer locally, but ran into the following error
auth0.v3.exceptions.Auth0Error: 403: You reached the limit of entities of this type for this tenant.

https://auth0.com/docs/policies/entity-limit-policy?_gl=1*1ulg0hl*rollup_ga*ODY5Njk2MDcyLjE2MjM4NTg3NDc.*rollup_ga_F1G3E656YZ*MTYyNjM1NjE4Ny4zLjEuMTYyNjM1NjE5OS40OA
This is a bug in creating duplicate auth0 apps

I'm having trouble with the NFS server for deployment.

I created a VM like the nfs-server-01 VM under two-eye-two-see
On that VM, I created /etc/exports and wrote /export/home-01 <IP_ADDRESS>/8(all_squash,anonuid=1000,anongid=1000,no_subtree_check,rw,sync) to it

However, the nfs-share-creator is still in ContainerCreating mode with FailedMount events for the volume "home-base". I'm not sure what to do next 😕

Would love to know if I'm on the right lines with this in terms of (i) is this correct config? (ii) will it achieve Paige's desires in #473 (comment)?

I think you are going in the right direction, but I am not a Dask guy :wink:
@yuvipanda, do you know if the options that are available here: https://github.com/2i2c-org/pilot-hubs/blob/master/hub-templates/daskhub/values.yaml#L183 are somehow "enforced/restricted" by the limits @sgibson91 wants to use in the config file?

I'm having trouble with the NFS server for deployment.

IIRC, @yuvipanda had some issues in the past related to an "insecure" flag

NFS server was again set up manually, and needed the insecure
flag - even though other hubs are setup the same way and didn't
need this. NFS situation needs to be sorted.

Not sure if that is relevant here... but I just remember it 😛  !

I think a lot of this is that I'm totally new to setting up NFS servers! I've just managed to work out that it helps to have the NFS packages installed on the VM! 😝

I have been playing around and the hub is definitely not happy. The hub is not reachable by it's external IP or the URL http://coessing.pangeo.2i2c.cloud

The hub is not reachable by it's external IP or the URL http://coessing.pangeo.2i2c.cloud

I just tried that URL and I was able to, at least, reach it...

Thanks @damianavila! I should be more patient with DNS things 😆 I will come back to this today

Ha, this was 💯 me not being patient enough with DNS stuff! Hub check now passes and I can log in! 🎉

Quick general question, we usually have staging and prod hubs on each cluster we create, ie. https://github.com/2i2c-org/pilot-hubs/blob/master/config/hubs/meom-ige.cluster.yaml
I am only seeing one hub in this config, is that intended?

Yeah. I guess because I'm envisioning both this hub being brought down after the workshop, and the cluster it's on being destroyed once the appropriate constraints have been lifted/amended on the new Pangeo account.

Ok, this is all up-to-date now, I'm gonna merge!

🚀🚀🚀


Terminal doesn't handle multiple OSC 1337 or OSC 7 escape sequences correctly

Type: Bug
From VSCode:

Create either of the following scripts as test.py, and follow the below instructions:

OSC 1337:
import os
import sys

script_dir = os.path.dirname(os.path.realpath(__file__))

if not os.path.exists("foo"):
    print("make a folder called foo first")
    quit()

new_cwd = os.path.join(script_dir, "foo")
print(f"changing cwd OSC 1337 to {new_cwd}", flush=True)
sys.stdout.write(f"\033]1337;CurrentDir={new_cwd}\007")
print("../test.py:37:1: error: use of undeclared identifier 'sdfsfds'")
print("delay...")

# print(f"changing cwd OSC 1337 to {script_dir}", flush=True)
# sys.stdout.write(f"\033]1337;CurrentDir={script_dir}\007")
# print("./test.py:37:1: error: use of undeclared identifier 'sdfsfds'")
# print("delay...")

OSC 7:
import os
import sys

script_dir = os.path.dirname(os.path.realpath(__file__))

if not os.path.exists("foo"):
    print("make a folder called foo first")
    quit()

new_cwd = os.path.join(script_dir, "foo")
print(f"changing cwd OSC 7 to {new_cwd}", flush=True)
sys.stdout.write(f"\033]7;file://localhost{new_cwd}\007")
print("../test.py:37:1: error: use of undeclared identifier 'sdfsfds'")
print("delay...")

# print(f"changing cwd OSC 7 to {script_dir}", flush=True)
# sys.stdout.write(f"\033]7;file://localhost{script_dir}\007")
# print("./test.py:37:1: error: use of undeclared identifier 'sdfsfds'")
# print("delay...")

Run python3 test.py from the inbuilt terminal.

Observe that Ctrl/Cmd+Click on ../test.py opens test.py as expected.

Uncomment the commented lines.

Run python3 test.py.

Observe that Ctrl/Cmd+Click on ../test.py does not open it. Instead, VSCode spins a while and then pops a file picker.

Observe that Ctrl/Cmd+Click on ./test.py opens test.py as expected.

From iTerm2:

Run python3 test.py, with the lines either commented out or not.

Observe that in both cases, Cmd+Click on ../test.py and ./test.py open test.py as expected.

This seems particularly notable as OSC 1337 is an iTerm2 proprietary escape sequence.
VS Code version: Code 1.95.3 (Universal) (f1a4fb101478ce6ec82fe9627c43efbf9e98c813, 2024-11-13T14:50:04.152Z)
OS version: Darwin arm64 24.2.0

I think the problem here is that we tie the cwd to the command being run, not the line range. Changing cwd inside a command like this isn't supported and I'm a little surprised to see it works in iTerm2, but I guess they just went the line route instead of linking it to the full execution. @rzhw have you read somewhere official that this is supposed to work? I think you stumbled upon some largely undefined behavior here.

Ah, I wondered if the similar behavior with OSC 633 ; P ; Cwd was due to OSC 633 being designed for shell integration, and the OSC 1337 and OSC 7 implementations being potentially based off of that and so working off the same assumptions?
I haven't seen anything explicitly saying OSC 7 and OSC 1337 should support multiple subsequent calls, though my naive reading of iTerm2's docs for OSC 1337 and OSC 7:

CurrentDir
OSC 1337 ; CurrentDir=[Ps1] ST

Reports the current directory.
[PS1] is the current directory.
The following synonym is available as a combination of RemoteHost and CurrentDir:
OSC 7 ; [Ps] ST

where [Ps] is a file URL with a hostname and a path, like file://example.com/usr/bin.

And WezTerm's docs for OSC 7:

OSC 7 Escape sequence to set the working directory
OSC 7 means Operating System Command number 7. This is an escape sequence that originated in the macOS Terminal application that is used to advise the terminal of the current working directory.
An application (usually your shell) can be configured to emit this escape sequence when the current directory changes, or just to emit it each time it prints the prompt.
The current working directory can be specified as a URL like this:
printf "\033]7;file://HOSTNAME/CURRENT/DIR\033\\"

When the current working directory has been set via OSC 7, spawning a new tab will use the current working directory of the current tab, so that you don't have to manually change the directory.
If you are on a modern Fedora installation, the defaults for bash and zsh source a vte.sh script that configures the shell to emit this sequence. On other systems you will likely need to configure this for yourself.

Seems to suggest nothing should prevent subsequent calls from working.

Thinking about this a bit more, macOS Terminal also doesn't seem to suggest there's limitations on how often OSC 7 can be sent. Rather, it's just for programs to "notify Terminal of the current working directory ... by sending it commands":

Ah, I wondered if the similar behavior with OSC 633 ; P ; Cwd was due to OSC 633 being designed for shell integration, and the OSC 1337 and OSC 7 implementations being potentially based off of that and so working off the same assumptions?

The similar sequences in 633 were essentially "forked" to that one so that VS Code could have a more complete protocol for shell integration, as opposed to having the mix of 1337, 133, 7, etc. (for better or worse). 633 is mostly intended to only be used in VS Code or in close collaboration with our team for now.

I think I'll close this as out of scope for now as currently we store the cwd against the execute and it would take a decent sized change to support this behavior which doesn't seem well defined. If the links are important I'd suggest to look into OSC 8 hyperlinks to provide consistent links across all terminals that support that feature


[24/10/2022] : fix social link icon display

Summary
Issue: https://github.com/Open-Science-Community-Saudi-Arabia/OSCSA_Website/issues/204
In an attempt to improve page loading time, the preload attribute was added to the bootstrap link tag and this broke the social link icon displayed. However, since the bootstrap assets(icons, CSS, etc) are already downloaded into the project, this may not necessarily affect the page loading speed as opposed to using CDN which would be a perfect use for the preload attribute.
This PR is a fix to this issue, I reverted the change made to the bootstrap Icons.
What should a reviewer concentrate their feedback on?

[ ] Everything looks ok?

@BatoolMM  @alswajiab Please review

@BatoolMM  @alswajiab
I noticed the hamburger menu icon for the mobile screen is no more visible on the home page, this is also due to the  changes made to the bootstrap icon.

The revert made in this PR would also fix that

@BatoolMM  the PR to fix the social icon

Ohhh, thank you 🙏


(maint) install released repos even when testing nightlies

Upgrade tests will fail if the only installed repos are the nightly repos.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/205/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/206/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
http://jenkins-enterprise.delivery.puppetlabs.net/job/enterprise_puppetdb_init-multijob_githubpr-stable/207/
Test PASSed.


kubectx must fail when it fails to retrieve namespaces/contexts

when we have
get_contexts() {
  kubectl config get-contexts -o=name | sort -n
}

and we use it like for c in $(get_contexts); do ... it doesn't exit immediately even though set -e is set.
https://stackoverflow.com/questions/44014857/function-invocation-in-for-loop-does-not-fail-even-though-set-e

Isn't the problem from your OP that you don't have pipefail set?
% false | echo hi
hi
% echo $?
0
% set -o pipefail
% false | echo hi
hi
% echo $?
1

The script already has -o pipefail set :)
See the stackoverflow link above. I think it's because we're calling it like for c in $(get_contexts) which isn't failing properly b/c it's in sub-shell.


com_google_fonts_check_name_family_and_style_max_length: Use typographic family name if present

Description
Still draft since I need to query this with Microsoft.
Checklist

[x] update CHANGELOG.md
[ ] wait for the tests to pass
[ ] request a review

Just done some tests and can confirm that nameID 16 is used if it exists.

I'm now fixing the conflict on this one.

Thank you very much! This will unblock a lot of projects for us.


Replace font type for arabic ui

Fixes: #313
PR Description:
Add a new font family for the Arabic view.

Thank you @sandygudie - can I ask why I can see the deployment preview, pls?

Yeah! i am working on it
I was expecting it to show automatically
i will get a preview manually


Introduce new command to allow loading of a single scene

Is your feature request related to a problem? Please describe.
For development purposes / isolation of scenes we should provide a command which allows a user to goto a specific scene and only load that one. It would be helpful for scenes like goerli plaza or in world.
Describe the solution you'd like
Currently we have
/goto-local (retains the current world)
/goto (jumps back to genesis city)

We should introduce something similar:
/goto-local-only (loads local scene and doesn't load any other around it)

@dalkia made a suggestion to just have a scene radius of zero, since that would be the simplest way to achieve this. Then you can jump anywhere with the scene radius being 0.

I re-think it and made only/solo as a last parameter. So we have
/goto 5,7 only
/goto-local 5,7 only
/goto goerli 82,1 only


listenOrDial is not method.

I'm try to using this wrapper. I follow the examples but got this error.
'NngResult<IRepSocket>' does not contain a definition for 'listenOrDial' and no accessible extension method 'listenOrDial' accepting a first argument of type 'NngResult<IRepSocket>' could be found.

Is it i missing something? Or there has a new examples project?

It's only used in the nngcat example to deal with commandline options.

It's only used in the nngcat example to deal with commandline options.

Okay, I see now. I missed it. Thanks :).


Populate the API Spec engagement records field of the Onboarding workitem

This should happen when a user is scheduling an API Review meeting. As part of it, we either get their GH issue or we create one for them.
That data needs to be added into the Onboarding WI under the Data Plane view:

Field name is: API Spec Engagement Records
Value: multiple lines of text. We need to add only if it is not already there

Now, when a new meeting is scheduled, a new line with the Issue link is added to the top of the list


Icons: set baseline min-width of icons

Alaska Airlines Pull Request
Fixes: #93
Summary:
This PR addresses an issue where the baseline width was able to be altered based on surrounding content.
Type of change:

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

:tada: This PR is included in version 4.8.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add files via upload

add STMIK GI MDP Palembang

@NaelXalim I have to close this request without merging, because you requested to add a non-existent domain to this repository: ;; connection timed out; no servers could be reached Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!


Adds oneview_switch module

Actually, the facts module has been mergerd.
https://github.com/HewlettPackard/oneview-ansible/blob/master/library/oneview_switch_facts.py


Fails to build on Ubuntu 22.04 LTS (-Werror)

Hello, on the current Ubuntu LTS (22.04), libssl-dev installs OpenSSL 3.0, which deprecates a few functions. Unfortunately, this causes Azure-DCAP-Client to fail to build from source because of usage of some functions that have been deprecated in 3.0:
local_cache.cpp: In function 'std::string sha256(size_t, const void*)':
local_cache.cpp:244:16: error: 'int SHA256_Init(SHA256_CTX*)' is deprecated: Since OpenSSL 3.0 [-Werror=deprecated-declarations]
  244 |     SHA256_Init(&sha256);
      |     ~~~~~~~~~~~^~~~~~~~~
In file included from local_cache.cpp:13:
/usr/include/openssl/sha.h:73:27: note: declared here
   73 | OSSL_DEPRECATEDIN_3_0 int SHA256_Init(SHA256_CTX *c);
      |                           ^~~~~~~~~~~
local_cache.cpp:245:18: error: 'int SHA256_Update(SHA256_CTX*, const void*, size_t)' is deprecated: Since OpenSSL 3.0 [-Werror=deprecated-declarations]
  245 |     SHA256_Update(&sha256, data, data_size);
      |     ~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from local_cache.cpp:13:
/usr/include/openssl/sha.h:74:27: note: declared here
   74 | OSSL_DEPRECATEDIN_3_0 int SHA256_Update(SHA256_CTX *c,
      |                           ^~~~~~~~~~~~~
local_cache.cpp:246:17: error: 'int SHA256_Final(unsigned char*, SHA256_CTX*)' is deprecated: Since OpenSSL 3.0 [-Werror=deprecated-declarations]
  246 |     SHA256_Final(hash, &sha256);
      |     ~~~~~~~~~~~~^~~~~~~~~~~~~~~
In file included from local_cache.cpp:13:
/usr/include/openssl/sha.h:76:27: note: declared here
   76 | OSSL_DEPRECATEDIN_3_0 int SHA256_Final(unsigned char *md, SHA256_CTX *c);
      |                           ^~~~~~~~~~~~
cc1plus: all warnings being treated as errors
make: *** [Makefile:28: local_cache.o] Error 1

@TimotheeIsnard I'm not from Microsoft Azure, but I just stumbled upon your issue. We had the same problem, and resolved like this:
echo "deb http://security.ubuntu.com/ubuntu focal-security main" | tee /etc/apt/sources.list.d/focal-security.list
apt-get update && apt-get install libssl1.1

So basically there is a libssl1.1 package that you can install, but for this you need the focal-security Ubuntu repo. With this, it worked.

Any update on this? We are looking into implementing this and running into issues


pretrain model weights miss pos_emb

Hi,exnx
I'm sorry to bother you. I want to use the pre-trained model weights you provided to fine-tune for downstream tasks. I found that some model parameters are missing. Is there anything wrong with my approach?
Some refactoring of the model parameter names was done using the code you provided before loading the model weights
Here is a more detailed translation:

pretrained_model_name = 'hyenadna-small-32k-seqlen'
max_lengths = {
'hyenadna-tiny-1k-seqlen': 1024,
'hyenadna-small-32k-seqlen': 32768,
'hyenadna-medium-160k-seqlen': 160000,
'hyenadna-medium-450k-seqlen': 450000,  # T4 up to here
'hyenadna-large-1m-seqlen': 1_000_000,  # only A100 (paid tier)
}
max_length = max_lengths[pretrained_model_name]  # auto selects
%# data settings:
use_padding = True
rc_aug = False  # reverse complement augmentation
add_eos = False  # add end of sentence token
%# we need these for the decoder head, if using
use_head = True
n_classes = 1  # not used for embeddings only
%# you can override with your own backbone config here if you want,
%# otherwise we'll load the HF one in None
backbone_cfg = json.load(open(f'hungging_face_models/hyenadna/{pretrained_model_name}/config.json', 'r'))
model = hyenadna.HyenaDNAModel(**backbone_cfg, use_head=use_head, n_classes=n_classes)
pretrain_w = torch.load(f'hungging_face_models/hyenadna/{pretrained_model_name}/pytorch_model.bin')

RuntimeError                              Traceback (most recent call last)
Cell In[12], line 1
----> 1 model.load_state_dict(pretrain_w)
File /Data/luokai/biotools/anaconda3/lib/python3.9/site-packages/torch/nn/modules/module.py:1667, in Module.load_state_dict(self, state_dict, strict)
1662         error_msgs.insert(
1663             0, 'Missing key(s) in state_dict: {}. '.format(
1664                 ', '.join('"{}"'.format(k) for k in missing_keys)))
1666 if len(error_msgs) > 0:
-> 1667     raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
1668                        self.class.name, "\n\t".join(error_msgs)))
1669 return _IncompatibleKeys(missing_keys, unexpected_keys)
RuntimeError: Error(s) in loading state_dict for HyenaDNAModel:
Missing key(s) in state_dict: "backbone.layers.0.mixer.filter_fn.pos_emb.z", "backbone.layers.0.mixer.filter_fn.pos_emb.t", "backbone.layers.0.mixer.filter_fn.implicit_filter.3.freq", "backbone.layers.0.mixer.filter_fn.implicit_filter.5.freq", "backbone.layers.0.mixer.filter_fn.modulation.deltas", "backbone.layers.1.mixer.filter_fn.pos_emb.z", "backbone.layers.1.mixer.filter_fn.pos_emb.t", "backbone.layers.1.mixer.filter_fn.implicit_filter.3.freq", "backbone.layers.1.mixer.filter_fn.implicit_filter.5.freq", "backbone.layers.1.mixer.filter_fn.modulation.deltas", "backbone.layers.2.mixer.filter_fn.pos_emb.z", "backbone.layers.2.mixer.filter_fn.pos_emb.t", "backbone.layers.2.mixer.filter_fn.implicit_filter.3.freq", "backbone.layers.2.mixer.filter_fn.implicit_filter.5.freq", "backbone.layers.2.mixer.filter_fn.modulation.deltas", "backbone.layers.3.mixer.filter_fn.pos_emb.z", "backbone.layers.3.mixer.filter_fn.pos_emb.t", "backbone.layers.3.mixer.filter_fn.implicit_filter.3.freq", "backbone.layers.3.mixer.filter_fn.implicit_filter.5.freq", "backbone.layers.3.mixer.filter_fn.modulation.deltas", "head.output_transform.weight", "head.output_transform.bias".
Thanks,

can you describe more? eg, which codebase are you using, colab, standalone_hyena, or main Pytorch Lightning?
At first glance, it may be similar to the issue described here.

Also, in your title, you mention pos_emb, but don't describe the issue in the body.  For HyenaDNA, we do not use positional embeddings for the tokens.  It uses a convolution, which has a sense of position already and so no need for pos_emb.
We do use positional embeddings differently in the filter function to create the implicit convolution kernel.  Is that what you were referring to?  Those are not learned and so do not have parameters.

Thanks for the reply @exnx, I use "standalone_hyena"。 It's should be positional embeddings of  implicit convolution kernel which you mentioned. Thanks again,Great work!  I'll review the model structure or try the Pytorch Lightning version training code you provided

Sure thing.  I recommend using the colab example to work through the colab.  Once that works then try the other training code.

Did you figure it out?

Did you figure it out?

Sorry for the delayed response to your information. The issue has been resolved, It's due to not inject   "mixer.layer" during the loading process, which resulted in the missing of these weights.
Currently, the performance indicators of Hyenadna model are comparable to those of DNAbert in my task; however, given the short sequence length of this particular task, it may not fully reflect its advantages. Thank you for your exceptional work and assistance.

need inject "mixer.layer"


Vue JSX does not support spread.

When using spread vue's JSX transform adds an import which can not be handled at the moment. You can replicate that by trying:
const App = {
  data: () => ({
  	sel: 2
  }),
  render() {
    return (
      <input type="checkbox" {...{}} />
    );
  }
}

new Vue({
  el: '#app',
  render: h => h(App)
})

I just added a view compiled so that you could know what happened behind the scene:

I guess we can simply:
import merge from '!raw-loader!babel-helper-vue-jsx-merge-props'

transformedCode.replace(/import _mergeJSXProps from "babel-helper-vue-jsx-merge-props";/, merge)

I was thinking of:
import('babel-helper-vue-jsx-merge-props').then(vueMergeJSXProps => window.vueMergeJSXProps = vueMergeJSXProps)

/*...*/

transformedCode.replace(/import ([_$a-zA-Z\xA0-\uFFFF][_$a-zA-Z0-9\xA0-\uFFFF]*) from ['"]babel-helper-vue-jsx-merge-props['"];?/, 'const $1 = window.vueMergeJSXProps)

I guess " and ; are not likely to disappear but babel's naming is unpredictable with their versions and should be universal.


Better error messages when locked id password was wrong.

@dpc Are you certain the password should be requested in a loop? As a counter example, provided an incorrect password, Git does not re-request the password. It just fails. Provided no work is lost (ensure password requested before work done), it seems cleaner just to end the program.

The way I think about it: What's the point of failing? If the user wanted to perform certain operation, and entered a wrong password, most probably it was just a typo etc. and they will want to try again. I assume most users know how to press Ctrl+C or Ctrl+D to terminate the process.

You've convinced me, that makes sense.
Should the passphrase closure function be passed all the way down the stack to local.to_unlocked(...)? I think that looping for the correct password would be simplest at that point.

@rffrancon I guess. I mean - it can easily be changed later, so no need to worry about it too much.
Please create some
pub type PassphraseFn =  alias, like in rdedup, propagate it where it makes sense, and that's the bulk of it. Make sure to rebase, since I did some refactors recently.

@rffrancon yep.


fixed migration to swift3

Thanks! I'm going to bring all plugins up to date and release them alongside Nuke 4.0-rc on Sunday.


Include cognito client ID in Lambda env when a lambda resource is allowed in defineAuth

Environment information
System:
  OS: Linux 6.8 Ubuntu 24.04.1 LTS 24.04.1 LTS (Noble Numbat)
  CPU: (16) x64 AMD Ryzen 7 7700X 8-Core Processor
  Memory: 17.55 GB / 30.46 GB
  Shell: /bin/bash
Binaries:
  Node: 20.18.1 - ~/.nvm/versions/node/v20.18.1/bin/node
  Yarn: undefined - undefined
  npm: 10.8.2 - ~/.nvm/versions/node/v20.18.1/bin/npm
  pnpm: 9.5.0 - ~/.local/share/pnpm/pnpm
NPM Packages:
  @aws-amplify/auth-construct: 1.3.1
  @aws-amplify/backend: 1.3.0
  @aws-amplify/backend-auth: 1.2.0
  @aws-amplify/backend-cli: 1.2.8
  @aws-amplify/backend-data: 1.1.4
  @aws-amplify/backend-deployer: 1.1.4
  @aws-amplify/backend-function: 1.5.0
  @aws-amplify/backend-output-schemas: 1.2.0
  @aws-amplify/backend-output-storage: 1.1.2
  @aws-amplify/backend-secret: 1.1.3
  @aws-amplify/backend-storage: 1.2.0
  @aws-amplify/cli-core: 1.1.3
  @aws-amplify/client-config: 1.3.2
  @aws-amplify/deployed-backend-client: 1.4.1
  @aws-amplify/form-generator: 1.0.3
  @aws-amplify/model-generator: 1.0.8
  @aws-amplify/platform-core: 1.1.0
  @aws-amplify/plugin-types: 1.3.0
  @aws-amplify/sandbox: 1.2.2
  @aws-amplify/schema-generator: 1.2.4
  aws-amplify: 6.6.2
  aws-cdk: 2.160.0
  aws-cdk-lib: 2.160.0
  typescript: 5.6.2
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

Describe the feature
When using the access field of defineAuth to provide bestow userPool permissions to a lambda, only the
userPool ID (env.AMPLIFY_AUTH_USERPOOL_ID) is added to the environment variables by default. I propose that the userPool client ID should be included as well, for example in env.AMPLIFY_AUTH_USERPOOL_CLIENT_ID
Use case
My use case for this feature would be to more easily use the CognitoJwtVerifier package within a custom authorizer lambda. My team uses this custom authorizer to resolve fine-grained permissions defined in a userInfo table that is associated with our cognito users.
The JWT verification step ensures that the token is valid and to obtain the claims from the access key. One such claim is the user sub which is used to obtain the full userInfo record. From there the permissions are resolved based on the graphQL and the permissions object.
The CognitoJwtVerifier package requires the cognito client ID. Currently this can be done within the backend configuration by adding an environment variable after defineBackend is called (or by a bootstrapping step involving Amplify secrets, as was required to avoid circular dependencies within the cfn stacks, though this is no longer necessary). By adding env.AMPLIFY_AUTH_USERPOOL_ID to the lambda env by default, the need for workarounds goes away entirely

Hey @dhaub-exelixis, thank you for opening this. Marking as feature request to add AMPLIFY_AUTH_USERPOOL_CLIENT_ID to function env variables.
Additionally, as discussed do let us know if the using the resourceGroupName on the function does mitigate the circular dependency issue when adding the client-id as env variable.


Updated packages and adding FluxLifeCycleStep to router

I think this fixes the issues people have on #11.
All the tests still ran fine, but you better double check I guess.

Hello,
I have still the same error if I use your version.

September release?

yes.
Is it only a problem of not using the right aurelia-router version?

Yes I think so


Access Control

Access to create static files
Access to create DIDs
Separate access controls for update methods

Access control should support OAuth tokens

Access control is needed for did-bloc


Fixing a crash in bertha and network issue

Hi @rachitb777, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Allow trailing commas at register macros

Rationale: Multi-line formatting, like:
ral::write_reg!(ral::flexio, flexio, TIMCTL[0],
    TRGSEL: 0,
    PINCFG: 0b11,
    PINPOL: 1,
);

which currently has to be written like this:
ral::write_reg!(ral::flexio, flexio, TIMCTL[0],
    TRGSEL: 0,
    PINCFG: 0b11,
    PINPOL: 1
);

Which makes it annoying to edit; every time a new entry gets added to the end, the previous line also has to be changed and git blame shows the wrong owner. Also it's just annoying in general, as all Rust built-in macros are capable of this, like println!:
println!("{}, {}!",
    "Hello",
    "World",
);

I've published this as 0.1.3.


Failure to find com.sap.db.jdbc:ngdbc:jar:2.0.13

Step 4 - Build the application "Note: When running the application in internet..."
executing mvn install:install-file -Dfile= -DgroupId=com.sap.db -DartifactId=ngdbc -Dversion=2.0.13 -
Dpackaging=jar
Installing the driver this way will result in an error when the application is compiled afterwards:
Failure to find com.sap.db.jdbc:ngdbc:jar:2.0.13
The application code includes com.sap.db.jdbc !
executing mvn install:install-file -Dfile= -DgroupId=com.sap.db.jdbc -DartifactId=ngdbc -Dversion=2.0.13 -
Dpackaging=jar
works.

Noted and updated. Thanks for your input.


Error trying to generate documentation

`$ typedoc --out ./docs ./src
Using TypeScript 2.7.2 from /home/user/.nvm/versions/node/v9.8.0/lib/node_modules/typedoc/node_modules/typescript/lib
Error: /home/user/git/frontend/node_modules/@types/sinon/index.d.ts(488)
';' expected.
`

same issue

same

Seems the default behavior of ignoring these files is broken (seems it is meant to be opt in given --includeDeclarations exists).

+1

+1

+1 for me still, sinon

Thanks for the repro to look at.
I haven't cloned to check it out, but I did notice that you are using ^0.11.1, if you update to ^0.14.2 you may find that the types work.

oh shooot sorry thanks

Closing this since as far as I can tell the error was caused by a TypeScript version incompatibility. As TypeDoc (master at least) is now on 3.4, this should not longer be an issue.
In July 2018 TypeDoc was on TypeScript version 2.7.2. The Sinon types at that time required conditional types, which were introduced in 2.8 git blame


Cumulative examples

Here I am trying to add docstring example to the cumulative functions. I did this by basically copying the method used for the reduction functions. Not sure at all if I did this correctly so i'm marking it a draft

[x] Tests added
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst
[x] New functions/methods are listed in api.rst

Very nice, this is something that's been on the TODO list! :)
I believe we wanted to rename generate_reductions.py to generate_aggregations.py so cumsum et al could be included and generated there as well. Is there a lot of work for you if try to merge these into that one?

@Illviljan That is definitely something I could do. Are there any other methods I should be including in this?

Right now, I think cumsum and cumprod is enough. numpy-groupies has a few more examples that I suppose we could support in the future.

Great I'll start working on that. Shouldn't take too long

Thanks @dcherian,
I'll try to work that in.
Is there a particular reason why there is no cumprod for GroupBy objects?

Is there a particular reason why there is no cumprod for GroupBy objects?

Nope. Just wasn't added in :)

def cumsum(..., dim):
	return xr.apply_ufunc(
	    np.cumsum if skipna else np.nancumsum,
		obj,
	    input_core_dims=[dim],
	    output_core_dims=[dim],
        kwargs={"axis": -1},
    )
    # now transpose dimensions back to input order

I'm running into an issue with variables without the core dimensions. Would it be better to do a work around in cumsum or in apply_unfunc like you mentioned in  #6391

I'm running into an issue with variables without the core dimensions. Would it be better to do a work around in cumsum or in apply_unfunc like you mentioned in  #6391

While this is definitely worth an improvement, it is quite out of scope for this PR.
Can you define Datasets such that this problem does not occur?

I've merged the cumulative and reduction files into generate_aggregations.py and _aggregations.py. This uses the original version of reductions with an additional statement on the dataset methods that adds the original coordinates back in.
Using apply_ufunc and np.cumsum/cumprod has some issues as it only finds the cumulative across one axis which makes iterating through each dimension necessary. This makes it slower than the original functions and also causes some problems with the groupby method.
Happy for any input on how the method using apply_ufunc might be usable or on any ways to change the current method.
I'm getting a few issues I don't quite understand:

When running pytest on my local repository I get no errors but it's failing the checks here with a NotImplementedError
Black is having an issue with some of the strings in generate_aggregations. It's saying it cannot parse what should be valid code.

Thanks!

I don't think you have flox installed, if it's not installed the code will take the old path.
Do conda install flox and I think you'll get the NotImplementedError. Then you maybe have to change the default settings in cumsum so flox is not used.

@dcherian Do you think it would be better to finish this PR as the creation of _aggregations.py to give the cum methods better documentation? Then start a new one to fix #6528?

o you think it would be better to finish this PR as the creation of _aggregations.py to give the cum methods better documentation? Then start a new one to fix https://github.com/pydata/xarray/issues/6528?

Sure that would be a good intermediate step. Let us know if you need help.

@keewis do you understand this blackdoc error?
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...............................................................Passed
debug statements (python)................................................Passed
mixed line ending........................................................Passed
autoflake................................................................Passed
isort....................................................................Passed
pyupgrade................................................................Passed
black....................................................................Passed
black-jupyter............................................................Passed
blackdoc.................................................................Failed
- hook id: blackdoc
- exit code: 123

error: cannot format /code/xarray/util/generate_aggregations.py: Cannot parse: 201:27: EOF in multi-line string
Oh no! 💥 💔 💥
215 files left unchanged, 1 file fails to reformat.

@dcherian cumsum for resample fails for some reason do you have any ideas?
import numpy as np
import pandas as pd
import xarray as xr
da = xr.DataArray(
    np.array([1, 2, 3, 1, 2, np.nan]),
    dims="time",
    coords=dict(
        time=("time", pd.date_range("01-01-2001", freq="M", periods=6)),
        labels=("time", np.array(["a", "b", "c", "c", "b", "a"])),
    ),
)
ds = xr.Dataset(dict(da=da))
a = ds.resample(time="3M")
a.cumsum()

Traceback (most recent call last):

  File "C:\Users\J.W\anaconda3\envs\xarray-tests\lib\site-packages\spyder_kernels\py3compat.py", line 356, in compat_exec
    exec(code, globals, locals)

  File "c:\users\j.w\documents\github\xarray\xarray\util\untitled2.py", line 20, in <module>
    a.cumsum()

  File "C:\Users\J.W\Documents\GitHub\xarray\xarray\core\_aggregations.py", line 4921, in cumsum
    return self.reduce(

  File "C:\Users\J.W\Documents\GitHub\xarray\xarray\core\resample.py", line 395, in reduce
    return super().reduce(

  File "C:\Users\J.W\Documents\GitHub\xarray\xarray\core\groupby.py", line 1357, in reduce
    return self.map(reduce_dataset)

  File "C:\Users\J.W\Documents\GitHub\xarray\xarray\core\resample.py", line 342, in map
    return combined.rename({self._resample_dim: self._dim})

  File "C:\Users\J.W\Documents\GitHub\xarray\xarray\core\dataset.py", line 3646, in rename
    return self._rename(name_dict=name_dict, **names)

  File "C:\Users\J.W\Documents\GitHub\xarray\xarray\core\dataset.py", line 3587, in _rename
    raise ValueError(

ValueError: cannot rename '__resample_dim__' because it is not a variable or dimension in this dataset

do you understand this blackdoc error?

so the reason is the position of the triple quotes: with
def f():
    """
    >>> 1 + 2"""
    pass

the extracted line will become: 1 + 2""", which when handed to black results in that error.
Technically, that's a bug (the triple quotes mark the end of the docstring), but one that is a bit tricky to fix: blackdoc is implemented using a line parser, which does not work too well if the transition happen somewhere within the line.
My guess is that it would have to start counting quotes which I've tried to avoid up until now since there's a lot of details to get right (see also keewis/blackdoc#145)

@keewis What does the numbers mean in Cannot parse: 201:27? Because I can't find any functions with docstrings in this file, it's just a bunch of multiline strings that are defined to variables and in my mind shouldn't trigger blackdoc.

well, that's just it: there's no way to discriminate between docstrings and multi-line triple-quoted strings without parsing the python code (which is definitely out of scope), so blackdoc doesn't even attempt to. Instead, any line that begins with >>>  will be assumed to be doctest lines (because ). So with that, the numbers mean: line 201, character 27.
When created that file we decided to skip the generate_* file, but because the file was renamed that rule does not work anymore. Could you update the pre-commit configuration?

@patrick-naylor, feel free to try out a better default example if you want.

Thanks @Illviljan, @dcherian, and @keewis so much for the help.


Client Allocation Request for: Himelight

Client Allocation Questions
Core Information

Name: Himelight
Website / Social Media: https://www.himelight.com/
Region: Greater China Region
DataCap Requested: 50TiB
Addresses to be Notarized: f1e6gxhnb7dwcmcdauz6hmxsxf64nmvtne746nxsa
Notary Requested: 1475Notary

@Selina665 Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Thanks for the request! Please answer the following questions giving as much detail as possible.
Please introduce yourself and the organization you represent, what is its name, nature, country of incorporation and date of establishment?
How many employees does the company currently have?
In which region do you plan to use DataCap?
How did you hear about the FIL+ project?
What kind of data do you plan to store? Can you share any kind of sample data?
Please provide the business license of your company.
Are you willing to store data outside of Greater China?


Add Molecule tests, change handlers

Hi @evrardjp ,
thank you for creating this role. It's really helpful.
My initial purpose was propose the PR for replace the synchronize module here with copy - to make this role do not require passwordless sudo.
But beforehand I make the current PR with proposal to update your role by Molecule tests. AFAIK currently you have the automatic syntax check - and I want to propose use Molecule for additional tests. It may check the syntax, run the role simultaneously in several OS, verify idempotency - and finally ensure that each prepared node able to ping other nodes over VPN.
I would be appreciated if you'll find a time to review this idea and related changes proposed to variables, tasks and handlers. Hope that next explanations will be helpful.
Updated .travis.yml will run molecule command two times and verify:

default scenario for ring topology
the star topology as well

Both scenarios described in corresponding subdirectories
We may simply rename default to the ring - however the currently proposed naming convention allow anybody just execute well-known command molecule test in repository root - and this will be enough to run basic test, without necessity additionally check documentation.
star scenario re-use the 2/3 playbooks, so the duplicated code defined as softlinks from molecule/star to molecule/default
Difference in:

default/molecule.yml
star/molecule.yml

Files are similar except the variables scenario.name and groups
Current config will spin-up 5 Docker containers, apply the role and execute simple check defined in verify.yml
I'm not going to recommend run the Tinc VPN on different OSs with different package versions in production - but this might be ok for testing purposes I think.
Docker give some additional specific:

code privileged: true with command: /sbin/init in molecule.yml enable Systemd on CentOS container. Although Ubuntu here doesn't use Systemd (if necessary, this could be changed by using some other specially prepared Ubuntu images, I think)
Docker images lack some standard software, so converge.yml take care about installing necessary dependencies
https://github.com/ansible-community/molecule/issues/959 say that Docker doesn't let us modify /etc/hosts in a container. To workaround this I added molecule-notest tag to this task and instead /etc/hosts will be modified during container creation - as configured by etc_hosts directives in molecule.yml

After failing first tests I found that on CentOS containers Tinc additionally required manual execution of systemctl restart tinc. Maybe I missed something in corresponding conversation but my assumption was that currently these handlers may doesn't work as expected. I think, that on systems with Systemd this code may trigger only systemd daemon-reload and Ansible may not restart the service afterwise. Not sure, maybe it's depends on Ansible version through.
Instead I want propose updated handlers which will be triggered by this code. I  have to agree that two-rows call is not ideal and would be great if somebody will propose something more laconic.
As I added enabled: yes to service description in handlers - this PR propose to remove last two tasks from tinc_configure.yml
I tried to imagine scenarios when we the settings in handlers will not be enough - and I can't propose anything. Please correct if I'm wrong.
Updated defaults/main.yml:

remove the variable service_name (I found it only in handlers and replaced with existing tinc_service_name)
define defaults for tinc_netname and tinc_control_plane_bind_ip - they required during Molecule execution.

This PR remove the tests folder used by original .travis.yml
This PR update meta/main.yml - after passing the tests I've extended the list of supported Ubuntu/CentOS systems in this file.
Thanks again for your time.
Looking forward for your feedback.

OMG this is awesome! :)
I didn't dig into your proposal yet as I am busy with other work today/tomorrow, but will evaluate it as soon as possible, latest Friday.

Thanks for confirmation about taking this request into consideration.
Please take your time - this issue is not a blocker for me.
Glad to see that it might be helpful.

Hi. Thank you for feedback @evrardjp
I made mistake by summarizing all these changes into one commit. Now I'm not sure if it possible break them back into several parts.
If you don't mind, I would propose close this PR as rejected. Instead I'll propose the other one, which will add only Molecule tests with several vars in defaults/main.
What do you think?

That sound good, let's do it step by step. The minimum to include molecule would be the first step, second would be the new features. That's awesome @niluid , thanks for the work.

Is there any way I can help? Should I build the molecule support with you?

Thank you @evrardjp
I glad to see tests added in https://github.com/evrardjp/ansible-tinc/pull/52
Now we need have these tests return correct 'green' status. As was agreed, this may require some improvements in handlers.
I going to re-check current behavior in Docker and VMs with Ansible 2.9 and 2.10. I'll submit my findings during a week. This probably will be formulated as issue with the list of proposed solutions - to discuss the best approach and then propose PR.
See you later!

Thanks! I have a PoC locally, workarounding the issue. I can probably issue a PR with workaround AND ideal state. Would you be okay to review it ?

Sure! I'll review it and response on first opportunity, during a day.
Thank you.


Selected numbers, increase font size

The displayed calendar dates (numbers) and currently focused hours or minutes should have a slightly larger font size. Just slightly as they appear a little too small and should be differentiated from the remaining text. Happy to discuss.

Can you post a screenshot on how you'd like it to be, compared to how it is now?
I'm happy with the current font sizes, but it might all render slightly different on different devices.

Parameters, it's all about parameters. We could introduce a font size (and color) parameter for the calendar. And whilst we're adding parameters a string for the "Done" button text!
But let me have a think about exact changes. I know it was my initial thought upon seeing the picker. If I still think it requires a change after a couple of days I'll let you know and will provide more information.

I'm not sure I agree with the premise that everything should be a parameter. Especially not if it's an open source project. If your change is too niche, you should fork rather than make it a parameter.
That said: the things you are looking for are already parameters. Take a look at /res/values/colors.xml and /res/values/dimens.xml
You can overwrite any of those values in your project style file.

Given that I'd be looking to override the time_picker_selector?
I am aware of the color parameters, they simply aren't extensive enough.

Rather than changing the colour, you might want to change the opacity of the selector (like the original google picker).
This is currently a hardcoded value.
I'll see if I can add both of those as an xml parameter next time I revisit this.
I also think I've found a way to make the text white when you exactly select the value. This should make things work for hours and most of the common minutes use cases. It requires a bit of work though, and I'm short on time at the moment, but I'll get around to it eventually.

Thanks, I look forward to any future updates.
If I have time (limited also at the moment) I'll fork the project.


[UI] Allow user to delete all K8s cluster

Current Behavior

When user tries to remove last remaining (or only available K8s cluster), user gets the below message and user is not able to delete the cluster.

Desired Behavior

Allow user to delete all connected K8s cluster.
Screenshots/Logs

Environment

Browser: Chrome Safari Firefox
Host OS: Mac Linux Windows
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Docker Kubernetes

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hi @MUzairS15 may I take on this :)

@legendarykamal Any updates on this??

Sorry  @MUzairS15 , My semester exams are going on but I fill try to finish this as soon as possible .

can I get this issue? if @legendarykamal don't have any problem


add IsCountable validator, to enforce type and count elements.

hi Team,
i created something like this in a private project (to enforce a countable has at least one element) but thought a more configurable version might be useful to others. what do you think?

thank you for the feedback. i'll get this fixed up tonight.

thanks everyone for the review. i think i took care of everything asked. @weierophinney i chose to implement the mutex logic in setOptions because it is invoked from the constructor. my test case covers behavior of the constructor and setOptions.

@abacaphiliac I've added some commits that do the following:

Make having count and one or both of min or max mutually exclusive; setting a count when either min or max are defined, or setting min or max when count is defined will raise an exception now. This prevents problems issues when you merge configuration; if you expect one set of behavior, you should not have a different behavior!
Adds documentation.

I'm expecting the only test failure to be due to the hostname TLD list being out-of-date (again! those used to update more on an order of weeks or months, and now it's practically daily!); if that's the case, I can merge this.

Thanks for this great feature, @abracaphiliac! I have merged it to develop for an upcoming 2.10.0 release (likely today or tomorrow).

my pleasure. thank you for the assist : )


Added ARC4 CPU support

Sleigh implementation of ARC4 CPU as described in "ARCtangent™-A4 Programmer’s Reference". Successfully used to RE Kodak digital picture frame firmware.

Are you aware of https://github.com/NationalSecurityAgency/ghidra/pull/3006 ?

@mumbel This looks like a re-implementation of #3233. #3006 is a different ISA.

@cyrozap Thank you for the reference! I was not aware of this work. I believe our implementation is a bit more complete. Specifically, we implement most of the features mentioned as unfinished in #3233, e.g., delay slot handling, loop instruction, and short immediate operands. We have also tested our implementation on real-world firmware.
Do you have advice on how we should proceed?

Looking more closely at the commit history it looks like most of the unfinished features have actually been finished.

Nice! Your implementation is way cleaner than ours:)
The only nit is that it doesn't look like it handles the zero-overhead loop setup, that uses sr to directly write loop start and end address into the loop registers. It's not actually possible to implement this properly because it's effectively a computed jump but should be flagged in some way.


Br config updates

adding dotenv
adding docs to readme
adding gitignore for dotenv
commenting out zk libraries

should we keep this on a separate branch?

fixing deploy script (commit) - see comments for follow up

when deploying locally I now get:
> npx hardhat deploy --network localhost --show-stack-traces

Deploying all contracts...
GeoWebCoordinate deployed to: 0xeC1BB74f5799811c0c1Bff94Ef76Fb40abccbE4a
GeoWebCoordinatePath deployed to: 0xF6a8aD553b265405526030c2102fda2bDcdDC177
GeoWebParcel deployed to: 0x09120eAED8e4cD86D85a616680151DAA653880F2
Accountant deployed to: 0x3E661784267F128e5f706De17Fac1Fc1c9d56f30
ERC721License deployed to: 0x6732128F9cc0c4344b2d4DC6285BCd516b7E59E6
ETHPurchaser deployed to: 0x15Ff10fCc8A1a50bFbE07847A22664801eA79E0f
ETHExpirationCollector deployed to: 0xAe9Ed85dE2670e3112590a2BB17b7283ddF44d9c
SimpleETHClaimer deployed to: 0xD1760AA0FCD9e64bA4ea43399Ad789CFd63C7809
Contracts deployed.

Setting default configuration...
Successfully set Accountant fee.
Successfully set Accountant validator.
Successfully set ETHExpirationCollector minContributionRate.
Successfully set ETHExpirationCollector minExpiration.
Successfully set ETHExpirationCollector maxExpiration.
Successfully set ETHExpirationCollector receiver.
Successfully set ETHPurchaser dutchAuctionLengthInSeconds.
Successfully set SimpleETHClaimer claimer.
Default configuration set.

Setting roles...
Successfully granted Accountant.MODIFY_CONTRIBUTION_ROLE to ETHExpirationCollector
Default roles set.

one more thing to note here - im not sure if this test was failing

@codynhat so the changes around the deploy script are just made right now so that I can get the local environment setup. What I am seeing when I look through these scripts is that the address for the different contracts is duplicated, ie there is a requirement on the accountant deploy for both params:
{ contract: accountant, contractAddress: accountant }... and so on for each contract

contractAddress is always equal to contract throughout the scripts - I could clean this up in the tasks if you want by removing the duplication but I wanted to check first if there was a reason im experiencing errors on my side that you're not experiencing on yours.
example error on account deploy without duplicated address
Deploying all contracts...
...
Contracts deployed.

Setting default configuration...
An unexpected error occurred:

TypeError: accountant.setPerSecondFee is not a function
    at SimpleTaskDefinition.action (/Users/brianrossetti/Projects/current/web3/projects/geoweb-top/core-contracts/tasks/Accountant.ts:54:36)
    at Environment._runTaskDefinition (/Users/brianrossetti/Projects/current/web3/projects/geoweb-top/core-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:217:35)
    at Environment.run (/Users/brianrossetti/Projects/current/web3/projects/geoweb-top/core-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:129:25)
    at SimpleTaskDefinition.action (/Users/brianrossetti/Projects/current/web3/projects/geoweb-top/core-contracts/hardhat.config.ts:45:13)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)
    at runNextTicks (internal/process/task_queues.js:64:3)
    at listOnTimeout (internal/timers.js:526:9)
    at processTimers (internal/timers.js:500:7)
    at async Environment._runTaskDefinition (/Users/brianrossetti/Projects/current/web3/projects/geoweb-top/core-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:217:14)
    at async Environment.run (/Users/brianrossetti/Projects/current/web3/projects/geoweb-top/core-contracts/node_modules/hardhat/src/internal/core/runtime-environment.ts:129:14)

The reason for this is due to the zkSync hardhat deploy plugin. I couldn't figure out a good way to get the address of an already deployed contract in zkSync. So when we deploy in zkSync, we need to pass the actual instance of the contract and not just the address.
Only one of each of these fields is needed (contract or contractAddress). But contract should be the actual contract instance where contractAddress is the address as a string.
Is  accountant a string in that example? Maybe we should start using typescript, or find a better way to deploy to zkSync.

@codynhat yeah contract is set as an address when I am running locally - is there a command you're running where accountant is the instance of the contract?

It is an instance of the contract when using the zkSync deploy tasks. See the difference in these tasks: https://github.com/Geo-Web-Project/core-contracts/blob/main/tasks/Accountant.ts#L11-L28

@codynhat updated the deploy.ts to only pass the contract addresses and removed the contract fields


[Task] Some widgets dimensions are not a factor of 10px

Issue -
Designs are created by taking the fact that the widget height will always be a factor of 10.
This is not respected for some widgets like text widget & button widget.
TODO:

Determine source of truth for the size - bounding box or content.

A poc is implemented here. It is working now.
But, I still didn't get why there's a product requirement to increment dimensions in a factor of 10px.


Entity not found

So first this: https://github.com/OpenMods/OpenPeripheral/issues/183
The only thing changed are a few method names.
Its still an issue (after more then 1 year wow) and i cant understand why "getPlayerByName" or "getPlayerByUUID" is putting out such an error and let the program crash instead of returning nil?
I have CC 1.7.4, OpenModsLib 0.8, OpenPeripheral Core 1.2, OpenPeripheral Addons 0.4, and Forge <IP_ADDRESS>8 installed for Minecraft 1.7.10 and i am playing the FTB Infinity Evolved Modpack.

This issue was fixed few days after reporting, I just forgot to close it.
Of course, such crash still may happen, if player moves out of the range between getPlayers and getPlayerByName, but this is unavoidable.
About error: well, it would be nice to do it in Lua way (return (nil, 'error')), but it's legacy thing. Since it worked this way since beginning, some programs may depend on it. So only thing left is pcall.

Ohh okay i see. Im not that huge lua expert and did not know pcall. Thank you for mention pcall that will help me very much (rftools had something similar with dialing device). So i think this can be closed (dont forget it :D)


feat: add provider ID to session

Adds the provider ID to a session, encoding the provider which was used to initially issue the session.
This allows developers to:

Revoke sessions issued by a potentially insecure provider.
Allow us to track the provider in audit logs.
In the coming Session Control API, it will allow developers to show their users how that session was initially established.

@kangmingtay It helps us identify which session was issued by which provider. This allows developers to:

Revoke sessions issued by a potentially insecure provider.
Allow us to track the provider in audit logs.
In the coming Session Control API, it will allow developers to show their users how that session was initially established.

@hf the auth.identities table also stores information about a provider. Given that a session is created from one of the user's identity, we can probably associate the identity id to the session.
I would prefer to have a single source of truth for the provider rather than storing this information in the auth.identities table and auth.sessions table


fix(connlib): ignore IPv6 multicast to all router packets

To start with, I only added the single one we keep seeing in the logs. We can extend this later with more IPs.
Resolves: #4024.

Would it be worth adding the IPv4 equivalent too? See discussion at #3015

Would it be worth adding the IPv4 equivalent too? See discussion at #3015

Yeah I can add that.


Camera up side down

Im new to it im currently gathering all the thing I need to build my room and I was wondering if it matter if the ps camera  are up side down screw to my ceiling ?

its work when camera up side down do calibration and this work


Warning about invalid sub_api version in calamari/7seg node type

Issue Description
Using a FBP containing
display(calamari/7seg:clear_pin=473,clock_pin=340,data_pin=474,latch_pin=475)

either with sol-fbp-runner or sol-fbp-generator, will yield in the end the following error
WRN:sol-flow-calamari ./src/modules/flow/calamari/calamari.c:221 calamari_7seg_child_opts_set() child_opts(0xd33100)->sub_api(0) != SOL_FLOW_NODE_TYPE_GPIO_WRITER_OPTIONS_API_VERSION(1)
Board information
Host OS and architecture: Fedora 23
Target OS and platform: MinnowBoard
Soletta version: beta19

@cmarcelo I believe it's fixed now.
If not please reopen this ticket
Thanks


sniper mode

when i try sniper mode, apeear this error, sorry for BAD quali

Sniper mode is work in progress. have been busy with other things in life. will try to work on it soon.


add syntax highlight in both edit and preview mode

Hi, thank you for this useful plugin!
I have enhanced this plugin with some improvements:

Thanks for the interest and contribution! :) I didn't have plans to extend this when I started the project, but I think it'd be a nice addition to have something like Prism & CodeMirror (I've personally been exploring Shiki for my Anki setup).
I'm unfamiliar with the Plugin APIs you introduced so I'll try to go over this during the weekend and see if it's all good.

No problem at all, just take a look at it in your free time. I added these features for my own usage needs and thought maybe I could contribute something to the source.
The APIs I introduced are Prism and CodeMirror mode. Prism is pretty simple, with only one highlight function included in obsidian.
For codemirror mode, you can refer to:

cm5 doc in this page
mlir-syntax-highlight plugin code
obsidian-gdscript code

This implementation is just utilizing builtin html/css/js modes.


Add missing questionToken in emitMethod/PropertyDeclaration

Fixes #15971

@ikatyang,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@ikatyang, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot


mrmr_classif doesn't return any results

Hi everyone,
I've been using the mrmr_classif method from the mRMR library for a while without any issues. However, recently it started returning no results and doesn't give any error messages. I've tried running it on Google Colab to see if the issue was specific to my local environment, but I encountered the same problem there.
Has anyone else experienced this or have any suggestions on how to resolve it?

@YsabelleSousa I faced the same problem. However, changing the type of the input dataframe using polars , as in your case X=polars.DataFrame(X), worked for me.


ERROR, I use pytroch to train but I got this

ccumulating evaluation results...
DONE (t=10.48s).
IoU metric: bbox
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.000
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.000
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.000
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.000
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.000
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.000
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.000
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.000
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.000
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.000
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.000
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.000
best_stat:  {'epoch': 0, 'coco_eval_bbox': 3.9432714116975085e-10}

It is normal when I train rtdetr-r18 on coco. I think you should give more information about what you have done , not just show some final result.
IoU metric: bbox
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.151
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.232
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.160
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.095
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.182
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.199
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.220
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.400
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.490
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.245
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.516
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.714
best_stat:  {'epoch': 0, 'coco_eval_bbox': 0.15060986268807167}

It is normal when I train rtdetr-r18 on coco. I think you should give more information about what you have done , not just show some final result.
IoU metric: bbox
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.151
 Average Precision  (AP) @[ IoU=0.50      | area=   all | maxDets=100 ] = 0.232
 Average Precision  (AP) @[ IoU=0.75      | area=   all | maxDets=100 ] = 0.160
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.095
 Average Precision  (AP) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.182
 Average Precision  (AP) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.199
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=  1 ] = 0.220
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets= 10 ] = 0.400
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=   all | maxDets=100 ] = 0.490
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= small | maxDets=100 ] = 0.245
 Average Recall     (AR) @[ IoU=0.50:0.95 | area=medium | maxDets=100 ] = 0.516
 Average Recall     (AR) @[ IoU=0.50:0.95 | area= large | maxDets=100 ] = 0.714
best_stat:  {'epoch': 0, 'coco_eval_bbox': 0.15060986268807167}

I changed nothing but logger.py, I will check again, thanks.

I use single gpu to train:
python tools/train.py -c configs/rtdetr/rtdetr_r50vd_6x_coco.yml

output this:
Not init distributed mode.
Start training
Load PResNet50 state_dict
Initial lr:  [1e-05, 0.0001, 0.0001, 0.0001]
loading annotations into memory...
Done (t=0.00s)
creating index...
index created!
loading annotations into memory...
Done (t=0.49s)
creating index...
index created!
number of params: 42862860
Traceback (most recent call last):
  File "tools/train.py", line 38, in <module>
    main(args)
  File "tools/train.py", line 24, in main
    solver.fit()
  File "/home/aebuser/RT-DETR-main/rtdetr_pytorch/tools/../src/solver/det_solver.py", line 37, in fit
    train_stats = train_one_epoch(
  File "/home/aebuser/RT-DETR-main/rtdetr_pytorch/tools/../src/solver/det_engine.py", line 35, in train_one_epoch
    for samples, targets in metric_logger.log_every(data_loader, print_freq, header):
  File "/home/aebuser/RT-DETR-main/rtdetr_pytorch/tools/../src/misc/logger.py", line 238, in log_every
    header, total_time_str, total_time / len(iterable)))
ZeroDivisionError: float division by zero

datasets is here:
(rt-petr) aebuser@Precision-5820:~/RT-DETR-main/rtdetr_pytorch$ ls dataset/coco
annotations  images  labels  LICENSE  README.txt  test-dev2017.txt  train2017  train2017.txt  val2017  val2017.txt

train rtdetr-r50 on one gpu

python tools/train.py -c configs/rtdetr/rtdetr_r50vd_6x_coco.yml

Not init distributed mode.
Start training
Load PResNet50 state_dict
Initial lr:  [1e-05, 0.0001, 0.0001, 0.0001]
loading annotations into memory...
Done (t=18.14s)
creating index...
index created!
loading annotations into memory...
Done (t=0.57s)
creating index...
index created!
number of params: 42862860
Epoch: [0]  [    0/29571]  eta: 1 day, 3:30:36  lr: 0.000010  loss: 52.9165 (52.9165)  loss_vfl: 0.3373 (0.3373)  loss_bbox: 2.4941 (2.4941)  loss_giou: 1.6976 (1.6976)  loss_vfl_aux_0: 0.3209 (0.3209)  loss_bbox_aux_0: 2.4534 (2.4534)  loss_giou_aux_0: 1.7099 (1.7099)  loss_vfl_aux_1: 0.2884 (0.2884)  loss_bbox_aux_1: 2.4570 (2.4570)  loss_giou_aux_1: 1.7213 (1.7213)  loss_vfl_aux_2: 0.2807 (0.2807)  loss_bbox_aux_2: 2.5275 (2.5275)  loss_giou_aux_2: 1.7080 (1.7080)  loss_vfl_aux_3: 0.2857 (0.2857)  loss_bbox_aux_3: 2.4716 (2.4716)  loss_giou_aux_3: 1.7132 (1.7132)  loss_vfl_aux_4: 0.3352 (0.3352)  loss_bbox_aux_4: 2.4743 (2.4743)  loss_giou_aux_4: 1.6857 (1.6857)  loss_vfl_aux_5: 0.3139 (0.3139)  loss_bbox_aux_5: 2.5399 (2.5399)  loss_giou_aux_5: 1.7607 (1.7607)  loss_vfl_dn_0: 1.0065 (1.0065)  loss_bbox_dn_0: 1.3145 (1.3145)  loss_giou_dn_0: 1.2791 (1.2791)  loss_vfl_dn_1: 0.9514 (0.9514)  loss_bbox_dn_1: 1.3145 (1.3145)  loss_giou_dn_1: 1.2791 (1.2791)  loss_vfl_dn_2: 0.9279 (0.9279)  loss_bbox_dn_2: 1.3145 (1.3145)  loss_giou_dn_2: 1.2791 (1.2791)  loss_vfl_dn_3: 0.9253 (0.9253)  loss_bbox_dn_3: 1.3145 (1.3145)  loss_giou_dn_3: 1.2791 (1.2791)  loss_vfl_dn_4: 0.9912 (0.9912)  loss_bbox_dn_4: 1.3145 (1.3145)  loss_giou_dn_4: 1.2791 (1.2791)  loss_vfl_dn_5: 0.9766 (0.9766)  loss_bbox_dn_5: 1.3145 (1.3145)  loss_giou_dn_5: 1.2791 (1.2791)  time: 3.3491  data: 0.7294  max mem: 2441

It works, thanks!


Create mockup of a usable and aesthetically pleasing chatbot user interface

The customer has a strong desire for a better overall layout of the application. Mockups are the most effective way to demonstrate our ideas. Please include mockups as an attachment to this issue and the team can provide feedback, after which the SRS can be updated with new mockups.

Updated main menu with nav item to open chat bot window

Actual chat window

looks about right to me


IndoorLevel are not visible the second time i open the page

My default page is a big Map created with nativescript-google-maps-sdk . When I first open my app I can see the IndoorLevel and IndoorBuilding but if I change my page and come back to the default I can't see anymore the IndoorLevel, however, it's still functional (zoom, translation, click on infoWindow, ..).
My map is used to visit IndoorLevel, so it's a big problem for me.
Any idea?

Example of code if it can help:
This app is used to show a list of artworks on specific level of building since GoogleMaps only use latitude and longitude and not the IndoorLevel
map.component.html
<ActionBar class="action-bar">
    <!--
    Use the NavigationButton as a side-drawer button in Android
    because ActionItems are shown on the right side of the ActionBar
    -->
    <NavigationButton ios:visibility="collapsed" icon="res://menu" (tap)="onDrawerButtonTap()"></NavigationButton>
    <!--
    Use the ActionItem for IOS with position set to left. Using the
    NavigationButton as a side-drawer button in iOS is not possible,
    because its function is to always navigate back in the application.
    -->
    <ActionItem icon="res://navigation/menu" android:visibility="collapsed" (tap)="onDrawerButtonTap()"
        ios.position="left">
    </ActionItem>
    <Label class="action-bar-title" text="Map"></Label>
</ActionBar>

<GridLayout>
    <MapView #mapView [latitude]="latitude" [longitude]="longitude"
             [zoom]="zoom" [minZoom]="minZoom" [maxZoom]="maxZoom" [bearing]="bearing"
             [tilt]="tilt" i-padding="50,50,50,50" [padding]="padding" (mapReady)="onMapReady($event)"
             (indoorBuildingFocused)="onIndoorBuildingFocused($event)"
             (indoorLevelActivated)="onIndoorLevelActivated($event)">
    </MapView>
</GridLayout>

map.component.ts
import { Component } from "@angular/core";
import { registerElement } from "nativescript-angular/element-registry";
import { MapView, Marker, Position, Style } from "nativescript-google-maps-sdk";
import { RadSideDrawer } from "nativescript-ui-sidedrawer";
import * as app from "tns-core-modules/application";
import { Artwork } from "~/app/models/artwork";
import { ArtworksService } from "~/app/services/artworks.service";

declare const com: any;

// Important - must register MapView plugin in order to use in Angular templates
registerElement("MapView", () => MapView);

@Component({
    moduleId: module.id,
    selector: "map",
    templateUrl: "./map.component.html",
    styleUrls: ["./map.component.scss"]
})
export class MapComponent {

    latitude =  45.497748;
    longitude = -73.571746;
    zoom = 13;
    minZoom = 11;
    maxZoom = 22;
    bearing = 0;
    tilt = 0;
    padding = [20, 20, 20, 20];
    mapView: MapView;

    style = `
        [
          {
            "elementType": "geometry",
            "stylers": [
              {
                "color": "#f5f5f5"
              }
            ]
          },
          {
            "elementType": "labels.icon",
            "stylers": [
              {
                "visibility": "off"
              }
            ]
          },
          {
            "elementType": "labels.text.fill",
            "stylers": [
              {
                "color": "#616161"
              }
            ]
          },
          {
            "elementType": "labels.text.stroke",
            "stylers": [
              {
                "color": "#f5f5f5"
              }
            ]
          },
          {
            "featureType": "administrative",
            "elementType": "geometry",
            "stylers": [
              {
                "visibility": "off"
              }
            ]
          },
          {
            "featureType": "administrative.land_parcel",
            "elementType": "labels.text.fill",
            "stylers": [
              {
                "color": "#bdbdbd"
              }
            ]
          },
          {
            "featureType": "poi",
            "stylers": [
              {
                "visibility": "off"
              }
            ]
          },
          {
            "featureType": "poi",
            "elementType": "geometry",
            "stylers": [
              {
                "color": "#eeeeee"
              }
            ]
          },
          {
            "featureType": "poi",
            "elementType": "labels.text.fill",
            "stylers": [
              {
                "color": "#757575"
              }
            ]
          },
          {
            "featureType": "poi.park",
            "elementType": "geometry",
            "stylers": [
              {
                "color": "#e5e5e5"
              }
            ]
          },
          {
            "featureType": "poi.park",
            "elementType": "labels.text.fill",
            "stylers": [
              {
                "color": "#9e9e9e"
              }
            ]
          },
          {
            "featureType": "road",
            "elementType": "geometry",
            "stylers": [
              {
                "color": "#ffffff"
              }
            ]
          },
          {
            "featureType": "road",
            "elementType": "labels.icon",
            "stylers": [
              {
                "visibility": "off"
              }
            ]
          },
          {
            "featureType": "road.arterial",
            "elementType": "labels.text.fill",
            "stylers": [
              {
                "color": "#757575"
              }
            ]
          },
          {
            "featureType": "road.highway",
            "elementType": "geometry",
            "stylers": [
              {
                "color": "#dadada"
              }
            ]
          },
          {
            "featureType": "road.highway",
            "elementType": "labels.text.fill",
            "stylers": [
              {
                "color": "#616161"
              }
            ]
          },
          {
            "featureType": "road.local",
            "elementType": "labels.text.fill",
            "stylers": [
              {
                "color": "#9e9e9e"
              }
            ]
          },
          {
            "featureType": "transit",
            "stylers": [
              {
                "visibility": "off"
              }
            ]
          },
          {
            "featureType": "transit.line",
            "elementType": "geometry",
            "stylers": [
              {
                "color": "#e5e5e5"
              }
            ]
          },
          {
            "featureType": "transit.station",
            "elementType": "geometry",
            "stylers": [
              {
                "color": "#eeeeee"
              }
            ]
          },
          {
            "featureType": "water",
            "elementType": "geometry",
            "stylers": [
              {
                "color": "#c9c9c9"
              }
            ]
          },
          {
            "featureType": "water",
            "elementType": "labels.text.fill",
            "stylers": [
              {
                "color": "#9e9e9e"
              }
            ]
          }
        ]
    `;

    mapIsReady: boolean = false;
    currentLevelName: string;
    artworks: Array<Artwork>;
    displayedArtworks: Array<Artwork>;

    constructor(private artworksService: ArtworksService) {
        this.getArtworks();
    }

    onDrawerButtonTap(): void {
        const sideDrawer = <RadSideDrawer>app.getRootView();
        sideDrawer.showDrawer();
    }

    onMapReady(event) {
        this.mapView = event.object;
        this.mapView.settings.indoorLevelPickerEnabled = true;
        this.mapView.setStyle(<Style>JSON.parse(this.style));

        this.mapIsReady = true;
        if (this.artworks) {
            this.initMarkerOnMap();
        }
    }

    initMarkerOnMap() {
        this.mapView.removeAllMarkers();

        for (const artwork of this.displayedArtworks) {
            const marker = new Marker();
            marker.position = Position.positionFromLatLng(artwork.latitude, artwork.longitude);
            marker.title = artwork.name;
            marker.snippet = artwork.place.name;
            marker.userData = {index: 1};
            this.mapView.addMarker(marker);
        }
        console.log("Marker refreshed");
    }

    onIndoorBuildingFocused(args) {
        console.log("Building changed -> " + args.indoorBuilding);
        if (args.indoorBuilding) {
            this.currentLevelName = args.indoorBuilding.levels[args.indoorBuilding.defaultLevelIndex].name;
        } else {
            this.currentLevelName = null;
        }
        this.refreshDisplayedArtworks();
    }

    onIndoorLevelActivated(args) {
        console.log("Level changed -> " + args.activateLevel);
        if (args.activateLevel) {
            this.currentLevelName = args.activateLevel.name;
        } else {
            this.currentLevelName = null;
        }
        this.refreshDisplayedArtworks();
    }

    refreshDisplayedArtworks() {
        this.displayedArtworks = [];
        console.log("New displayed artwork -> " + this.currentLevelName);
        for (const artwork of this.artworks) {
            if (this.currentLevelName && this.currentLevelName && artwork.level === this.currentLevelName) {
                this.displayedArtworks.push(artwork);
            }
        }
        this.initMarkerOnMap();
    }

    getArtworks() {
        this.artworksService.list().subscribe(
            (artworks) => {
                this.artworks =  artworks.results.map(
                    (item) => new Artwork(item)
                );
                this.displayedArtworks = this.artworks;
                if (this.mapIsReady) {
                    this.refreshDisplayedArtworks();
                }
            }
        );
    }
}

I saw something similar last year and concluded that mapReady was not firing when returning to the map page.  I handled this by adding logic to onLoaded.

@dlcole I tried putting something into onMapReady to see if it was firing the second time i load the page and it seems that this block was working great. But since i have really no more idea at this point i will investigate your onLoaded to check if i can fix it with it

In my instance, I really only wanted the mapReady event to fire once, on initial load/ready, and never again... to do this, I removed the event listener via mapView.off('mapReady') and the end of my mapReady event handler. Not sure if this info is of any use to you, but it's here nonetheless!


:bug: Fixed part of issue #18 (items 2 and 3)

This will make latex comments be reflowed properly. This fix problems pointed by @hrothgar in items 2 and 3. Item 1 will need more effort though.
I specifically tested the same examples @hrothgar provided and they worked just fine. I'm using atom version 0.141.0.

Could you add a spec for this new comment delimiter? Thanks

Hi @thmosqueiro, thanks for sending this PR!
The Atom team is currently migrating many core Atom packages back into the main Atom repository.  autoflow is now being moved as a part of this migration!  Since this PR has conflicts with master we can't currently merge it so I'm closing it for now with the hope that you'll be willing to reopen it on atom/atom at some point in the future.
Thanks again for contributing to Atom! :sparkles: :heart: :atom:


BUG: fix elusive MPI runtime error in tests

closes #406
Finally figured it out. Our MPIBackend unit tests use a reduced network model to speed things up. This revealed a rare edge case that only shows up on machines with a large number of logical cores, where more MPI threads were created than there were neurons contributing to the dipole. When NetworkBuilder().aggregate_data() then tried to add up and reduce its NEURON data instances across both intra-rank and inter-rank neurons, it hung due to to a mismatch between h.Vector() sizes.

@cjayb All the tests on your M1 Mac should pass after this PR. Feel free to give it a try if you feel inspired!

@cjayb All the tests on your M1 Mac should pass after this PR. Feel free to give it a try if you feel inspired!

I did, and they do! Amazing sleuthing @rythorpe —tip of the old hat to you, Sir!

Codecov Report

Merging #545 (1995230) into master (52ee292) will decrease coverage by 0.01%.
The diff coverage is 92.85%.

@@            Coverage Diff             @@
##           master     #545      +/-   ##
==========================================
- Coverage   90.24%   90.23%   -0.02%
==========================================
  Files          20       20
  Lines        3988     3992       +4
==========================================
+ Hits         3599     3602       +3
- Misses        389      390       +1

Impacted Files
Coverage Δ

hnn_core/network_builder.py
93.83% <90.90%> (-0.27%)
:arrow_down:

hnn_core/network.py
92.16% <100.00%> (ø)

hnn_core/parallel_backends.py
82.36% <100.00%> (+0.04%)
:arrow_up:

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

Very cool! What was the reason that the test was failing intermittently? Is their some non-deterministic part in the rank assignment?

What was the reason that the test was failing intermittently?

I'd call this a consistent issue for those of us with more >6 logical cores. The oversubscription test creates 1.5x threads—for my 8 cores this is 12. There are 3 x 3 == 9 grid points in the test network, each with 2 pyramidal cells that contribute to the aggregate dipole moment. The gid assignment starts by looping over cells, which 'run out' at 18—that leaves 24-18==6 threads without a cell that contributes, which chokes the post-simulation reduction operations.
Sound right @rythorpe ?
The CI tests run on less than 6 cores, hence no problem.

Yes, that's exactly it!

looks good! I'm trying to enable auto-merge ... will see if I can make it work on this PR

Thanks @rythorpe !!! 🥳 This certainly deserves a celebration


Bundler executable not available

I'm having a problem with running bundler. The deploy completes deploy:symlink:linked_dirs and then fails on bundler:install:
 01 RBENV_ROOT=$HOME/.rbenv RBENV_VERSION=2.4.2 $HOME/.rbenv/bin/rbenv exec bundle install --path /home/ubuntu/rails/<APPLICATION>/current/bin/bundle --without development test --deployment --quiet --binstubs
 01 rbenv: bundle: command not found

I've tried installing it manually with rbenv exec gem install bundler and rbenv rehash, but the executable bundle does not appear.
capistrano 3.10.0
capistrano-rbenv 2.1.3
capistrano-bundler 1.3.0
capistrano-rails 1.3.0
rails 4.2.10
rbenv 1.1.1

If you run RBENV_ROOT=$HOME/.rbenv RBENV_VERSION=2.4.2 $HOME/.rbenv/bin/rbenv exec gem list | grep bundler, what's the output?

The gem is there, but not the executable.

Shot in the dark, but does it work if you run RBENV_ROOT=$HOME/.rbenv RBENV_VERSION=2.4.2 $HOME/.rbenv/bin/rbenv rehash ?

I'ma also having this issue. Bundler is there.

@maynardewm: Same exact output? Could you paste the error message?

It's slightly different:
INFO [02658949] Running bundle exec puma -C /home/ubuntu/app_name/shared/puma.rb --daemon as<EMAIL_ADDRESS>
 DEBUG [02658949] Command: cd /home/ubuntu/medcordance/current && ( export RBENV_ROOT="$HOME/.rbenv" RBENV_VERSION="2.5.0" RACK_ENV="staging" ; bundle exec puma -C /home/ubuntu/medcordance/shared/puma.rb --daemon )

 DEBUG [02658949] 	bash: bundle: command not found

Everything works up until this command. Which is weird because that means it's running bundle install....?

I think you have a different issue, and should probably open your own ticket.
Offhand, I'd guess that you are running something like execute 'puma -C .....' instead of execute :puma, '-C ......'. If that's the case, try the latter syntax. If that is not the case, open a new ticket, or better yet, open a Stack Overflow question, since this is more of a support issue than a bug.

I had the same problem. What worked for me was adding puma and pumactl to rbenv_map_bins. Hope this helps.

I had the same problem. What worked for me was adding puma and pumactl to rbenv_map_bins. Hope this helps.

@ngetahun it worked perfectly for my case too. Thank you very much <3


Create trie.py

PR Description
Adds trie data structure
Related Issue
Closes: #62
Changes Made
List the changes you made in this pull request:

Adds trie data structure.

Author
Raghav Sharma (@rghvsh )

Hey @Tinny-Robot made some change. Is

Hey

Hey! @Tinny-Robot checked the other files and made the file analogous , changed its file structure. Is this better?

fixed indentation


make fork + exec non blocking on unix

Dear all,
Following the bug #414, here is a patch which makes the fork + exec operation non blocking on Unix.
The patch is  probably not ready to land, but I believe it's in pretty good shape.
This patch require a few changes in cpython (subprocess.Popen), which are included in this patch for discussion (in tmp_subprocess, as a child class of subprocess.Popen). Those changes have been tested with cpython unittests on my laptop. The whole patch has been tested with Python 3.5.2 on Linux, so the CI may report undetected errors for other configurations.
The problem:
subprocess.Popen() will fork and synchronously read on a pipe until it is closed as a way to wait and check the result of the exec() call in the new child process. All of this is done in the constructor of Popen.
This is an issue when the preexec_fn (user function called before exec) or exec() is slow.
This PR does the following:

breaks the private Popen._execute_child() method into several methods so we can isolate the blocking reads (on the fd errpipe_read) in a (temporary) child class tmp_subprocess._Popen.

introduces _NonBlockingPopen, a version of _Popen which uses the loop and a future to signal the caller when the exec() is done (or failed). It means that an error will usually be reported by the future instead of raised when the _NonBlockingPopen object is created. It also means that to follow the (implicit) contract of Popen, _NonBlockingPopen is responsible of cleaning resources it opened if an error occurs before exec(). Note that _NonBlockingPopen also uses sockets instead of pipes (following what was done in  _UnixSubprocessTransport._start()).

since it is now required to wait on a future (but only on unix so far), _BaseSubprocessTransport._start() can be a coroutine. The implementation of the transport has been modified to support this change: the fork is now done in a task scheduled by the constructor. This doesn't change much for the caller (_make_subprocess_transport()) which awaited a future reporting errors during the transport initialization or is done once the subprocess transport is in a stable condition (_connect_pipes() finished). Special care has been taken to possible races conditions due to cancellations (usually when loop.subprocess_exec/shell() are canceled).

loop._make_subprocess_transport() doesn't change much. The biggest change is that the child_watcher is handled by the transport itself. This is required because we don't want this method to handle the process termination if something failed before the subprocess exec-ed, while now _NonBlockingPopen still have to handle this asynchronously.

Two tests have been added: one handling a new possible race condition (cancellation before exec), and the other one ensuring that _NonBlockingPopen will eventually raise an exception if something wrong happens in preexec_fn (this was previously the responsibility of Popen).

One test have been modified (test_proc_exited): it assumed that the transport was ready before the waiter was done. I believe this assumption is incorrect, and doesn't hold true now that the _BaseSubprocessTransport._proc attribute is set asynchronously (instead of in the constructor).

This is my biggest PR for asyncio so far, and my first attempt at patching cpython, I'm eager to receive feedback and comments about the patch and about how it can successfully land in the codebase (for instance, should I get in touch with the owner of cpython's subprocess?).
Thanks for your time,
Martin

@gpshead This seems your territory -- do you have time to look into this? The ideas seem solid.

Thank you @Martiusweb for this patch. There are a lot of changes here and there are many concerns. For example the asyncio library should work with Python 3.3 and up (at least the version from this repo -- the one in the CPython repo only needs to work with the version it's part of, but we try to keep the source differences minimal). My other overarching concern is that we're already in feature freeze for Python 3.6 (3.6b1 went out a few weeks ago) and while this is technically not a new feature it certainly is a complex thing to land close to a release. You should probably also create an issue in bugs.python.org with at least a patch for subprocess.py, and a reference here.

+1 to what @gvanrossum said.  I can take a look at the patch in a couple of weeks (assuming you fix the CI).  We can probably aim for 3.6.1 to have this.

Thank you for your feedback.
I pushed a commit which adds compatibility with Python 3.3. It adds a tmp_subprocess33 module which is the Popen patch backported for python 3.3. The modules tmp_subprocess and tmp_subprocess33 can probably be merged, but I guess that it can wait until we know how things will evolve on the cpython side.
The CI is not yet fixed on windows (I don't have a windows setup to work on yet). I expect (...hope) that the bug which make the tests hang will be easy to fix as there is not much change in the windows code path.
I also opened an issue on bugs.python.org about the changes in Popen: http://bugs.python.org/issue28287
I'm all in favor for not rushing the patch: I prefer if I can make it in python 3.6.1 or later instead of seeing it reverted because something was missing.

Hello, and thanks for your contribution!
Unfortunately we couldn't find an account corresponding to your GitHub username at bugs.python.org (b.p.o). If you don't already have an account at b.p.o, please create one and make sure to add your GitHub username. If you do already have an account at b.p.o then please go there and under "Your Details" add your GitHub username.
And in case you haven't already, please make sure to sign the PSF contributor agreement (CLA); we can't legally look at your contribution until you have signed the CLA.
Once you have done everything that's needed, please reply here and someone will verify everything is in order.

Hi,
I updated the PR: if my patch on Popen (submitted on b.p.o) is merged, asyncio will detect that it's able to make a NonBlockingPopen and use it, else, it will use Popen and block, but work as it used to. When it's done, I will remove the module tmp_subprocess (which monkey patches Popen) so the PR will be ready fro review.
@the-knights-who-say-ni  My account on b.p.o is martius, I add my github username there. I signed the CLA.

Can you link to the specific issue you opened for your CPython patch?

Yes, this is this one: http://bugs.python.org/issue28287

Oh, so that's scheduled for Python 3.7.


docs: Clarify that RegexMatcher uses a full regex match

Signed-off-by: Greg Greenway<EMAIL_ADDRESS>
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/lgtm api


Union types are not being narrowed down correctly in 4.3

Bug Report
Potentially related to #44382.

🔎 Search Terms

union types
narrowing
4.3

🕗 Version & Regression Information

This changed between versions 4.2 and 4.3

⏯ Playground Link

Playground link with relevant code
💻 Code

interface X {
  a?: { aProp: string };
  b?: { bProp: string };
}
type AorB = { a: object; b: undefined } | { a: undefined; b: object };

declare const q: X & AorB;

if (q.a !== undefined) {
  q.a.aProp;
} else {
  // q.b is incorrectly inferred as potentially undefined:
  q.b.bProp;
}

🙁 Actual behavior

q.b is inferred as potentially undefined in the else block. Given that q conforms with AorB, and q.a is known to be undefined in the else block, this is incorrect.
🙂 Expected behavior

q.b should be inferred as present. In particular, it should be { bProp: string }.

@RyanCavanaugh is it planned to fix this at some point in the near future? I see you added it to the backlog, but that list contains 2.6K issues and some of them are years old.
If it turns out that this bug will be ignored, then I'd prefer to start looking for workarounds now, and beginning to rewrite a number of TypeScript projects that rely on the old behaviour.

@KnorpelSenf barring other people encountering this or other prioritization data appearing, I don't expect it'd be fixed soon unless someone from outside the team wrote a PR (not unlikely but obviously I cannot predict when that might happen)

Thanks for the quick response, then I'll start looking into workarounds

Potential fix in #44771
Regression in this commit

Potentially related to #44382.

I've found that the regression is not related to this issue.

As a workaround it is possible to add a superfluous check for undefined. (Alternatively, a type cast could be performed.) This is obviously not a satisfying solution.
Naturally, I would much rather have see #44771 merged.

I’ve got another way to reproduce this. Has been fixed by the same fix in #44771, I’m leaving this in case it helps anyone with a similar case find this issue.
Union of an intersection type is not narrowed down after type guards: Playground
type T = { x: string } & { y: string };

const good: T | "none" = "none";
// good is narrowed to "none" in the conditional
if (good === "none") good.toUpperCase();

const bad = "none" as T | "none";
// bad is not narrowed in the conditional, “toUpperCase does not exist on type T”
if (bad === "none") bad.toUpperCase();

// typeof good === typeof bad === (T | "none")

const noIntersection = "none" as { x: string; y: string } | "none";
// noIntersection is narrowed to "none" in the conditional
if (noIntersection === "none") noIntersection.toUpperCase();

Worked correctly in 4.2. Works incorrectly on 4.3. Works incorrectly on 4.4.0-dev.20210712. Works correctly on 4.4.0-dev.20210714.


About displaying distorted objects

Hello good afternoon
About displaying distorted objects
I'm trying to simulate the example "water"
https://webvr.donmccurdy.com/water/
But in the use of the library "aframe-extras.min.js" which is pointed out in its reposition of the github see, which is displaying the distorted objects
https://codepen.io/mozgbrasil/pen/yRbMvM

Hi, I'm not sure what you're asking sorry. The Codepen example shows the ocean correctly for me.
If you're trying to raise the camera above the ocean and let the user look around, recent versions of A-Frame and this library work differently. See https://github.com/donmccurdy/aframe-extras/tree/master/src/controls, or:
<a-entity id="rig" movement-controls>
  <a-entity camera
            position="0 1.6 0"
            look-controls="pointerLockEnabled: true"></a-entity>
</a-entity>

you need set a entity around the camera and set the position
this should working (in your example you are under the water)
https://codepen.io/anon/pen/yRbpdo

Thank you my friend, it worked as expected.