BUG: scipy.optimize._differentialevolution incorrectly converted np.inf values to self.bounds when using scipy.optimized.Bounds

Describe your issue.
Hello scipy team,
thank you for providing such an amazing package. While working with your differential evolution algorithm, I experienced a conversion problem with the optimization variable bounds when working with np.inf values together with scipy.optimize.Bounds. In line 680 of the file scipy.optimized._differentialevolution we have
self.limits = np.array(new_bounds_to_old(bounds.lb,
                                         bounds.ub,
                                         len(bounds.lb)),
                       dtype=float).T

which results in nan-values in self.limits, caused by line 400-401 in scipy.optimize._bounds
lb = [float(x) if x > -np.inf else None for x in lb]
ub = [float(x) if x < np.inf else None for x in ub]

One fix would probably be to substitute the lines above by
lb = [float(x) if x > -np.inf else -np.inf for x in lb]
ub = [float(x) if x < np.inf else np.inf for x in ub]

Thank you very much in advance :)
Reproducing Code Example
import numpy as np
from scipy.optimize import Bounds, differential_evolution

bounds = Bounds(lb=np.array([0, -np.inf]), ub=np.array([np.inf, 0]))

func = lambda x,y: x**2 + y **2

res = differential_evolution(
    func = func,
    bounds = bounds
)

Error message
Traceback (most recent call last):
  File "secret/scipy_bug.py", line 8, in <module>
    res = differential_evolution(
          ^^^^^^^^^^^^^^^^^^^^^^^
  File "secret/.venv/lib/python3.11/site-packages/scipy/optimize/_differentialevolution.py", line 377, in differential_evolution
    with DifferentialEvolutionSolver(func, bounds, args=args,
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "secret/.venv/lib/python3.11/site-packages/scipy/optimize/_differentialevolution.py", line 689, in __init__
    raise ValueError('bounds should be a sequence containing '
ValueError: bounds should be a sequence containing real valued (min, max) pairs for each value in x

SciPy/NumPy/Python version and system information
1.9.3 1.24.2 sys.version_info(major=3, minor=11, micro=2, releaselevel='final', serial=0)
openblas64__info:
    libraries = ['openblas64_', 'openblas64_']
    library_dirs = ['/usr/local/lib']
    language = c
    define_macros = [('HAVE_CBLAS', None), ('BLAS_SYMBOL_SUFFIX', '64_'), ('HAVE_BLAS_ILP64', None)]
    runtime_library_dirs = ['/usr/local/lib']
blas_ilp64_opt_info:
    libraries = ['openblas64_', 'openblas64_']
    library_dirs = ['/usr/local/lib']
    language = c
    define_macros = [('HAVE_CBLAS', None), ('BLAS_SYMBOL_SUFFIX', '64_'), ('HAVE_BLAS_ILP64', None)]
    runtime_library_dirs = ['/usr/local/lib']
openblas64__lapack_info:
    libraries = ['openblas64_', 'openblas64_']
    library_dirs = ['/usr/local/lib']
    language = c
    define_macros = [('HAVE_CBLAS', None), ('BLAS_SYMBOL_SUFFIX', '64_'), ('HAVE_BLAS_ILP64', None), ('HAVE_LAPACKE', None)]
    runtime_library_dirs = ['/usr/local/lib']
lapack_ilp64_opt_info:
    libraries = ['openblas64_', 'openblas64_']
    library_dirs = ['/usr/local/lib']
    language = c
    define_macros = [('HAVE_CBLAS', None), ('BLAS_SYMBOL_SUFFIX', '64_'), ('HAVE_BLAS_ILP64', None), ('HAVE_LAPACKE', None)]
    runtime_library_dirs = ['/usr/local/lib']
Supported SIMD extensions in this NumPy install:
    baseline = SSE,SSE2,SSE3
    found = SSSE3,SSE41,POPCNT,SSE42,AVX,F16C,FMA3,AVX2
    not found = AVX512F,AVX512CD,AVX512_KNL,AVX512_SKX,AVX512_CLX,AVX512_CNL,AVX512_ICL

Thanks for reporting. That might work, although there are some optimizers that work with None instead of an inf in their bounds. That's probably why the line was written the way it is, and DE only started using the code later. This should be easy to fix in any case, but that's something to keep in mind.

As @mdhaber says None is used by several other minimisers in optimize to signify no bounds. One example is L-BFGS-B.
If the stacktrace your interpreter provided was more expansive the context would be revealed:
    701 if (np.size(self.limits, 0) != 2 or not
    702         np.all(np.isfinite(self.limits))):
--> 703     raise ValueError('bounds should be a sequence containing '
    704                      'real valued (min, max) pairs for each value'
    705                      ' in x')

The way differential_evolution works is by generating random numbers between the lower and upper bounds (remember no gradients here). So we actually want the lower and upper bounds to be finite.
Perhaps adding the word finite to the error message would be a good change: ... containing finite real valued ... .
It's been suggested before that if np.inf is observed in a bounds then replace it by np.finfo(float).max (similar argument for lower limit), but that's not really a good solution. Firstly, it'll cause float arithmetic to overflow in a lot of cases. Secondly, because guesses are generated within those bounds via random number generation, if the objective function is in the least bit noisy there is no chance that a minimum would be found.
I don't think changes to the limits processing makes sense here. Are there other changes you can suggest that would improve usability here @FloWolfStatworx?

I don't think changes to the limits processing makes sense here

Oops. Yeah, clearly DE can't have infinite bounds. I agree that all that is needed is the addition of the word "finite". I'll submit a PR.


Use ForEachActiveObject to iterate system pool object

Problem
What is being fixed?  Examples:

We should stop using inclusive forloop to iterate the system pool object since we have supported CHIP_SYSTEM_CONFIG_POOL_USE_HEAP configuration now. The system pool size is no longer fixed under this configuration.

Change overview
Use ForEachActiveObject to iterate system pool
Testing
How was this tested? (at least one bullet point required)

Confirm TestTCP is not broken with this change.

Do we still have plans to remove the indexed getter and size APIs?  Is this blocked on test code refactors?

Do we still have plans to remove the indexed getter and size APIs? Is this blocked on test code refactors?

That is my plan, have the draft version ready and will post PR after this one get merged

@saurabhst @Damian-Nordic ?

@mspang ?


Fixes tear issue

Btw I wanna be a collaborator...

tested? I think that's work but I need to go to sleep...

Yes tested.

Well done! yeahhhhhh


Variable inspector doesn't launch (no option on right click)

I think I have installed it correctly onto Jupyterlab as it shows up under installed extensions but when I right click my notebook there is no option to open the variable inspector. Has anyone else had this issue? is there some way to enable it that I have missed?

What jupyter labextension list gives you ?

Known labextensions:
app dir: C:\ProgramData\Anaconda3\envs\CondaEnvironment37\share\jupyter\lab
@jupyter-widgets/jupyterlab-manager v1.1.0 enabled  ok
@lckr/jupyterlab_variableinspector v0.4.0 enabled  ok
Uninstalled core extensions:
@jupyter-widgets/jupyterlab-manager
@lckr/jupyterlab_variableinspector
@ryantam626/jupyterlab_code_formatter

What jupyter labextension list gives you ?

This discussion might help you. Try to delete anaconda3\share\jupyter\lab\settings\build_config.json and then rebuild jupyter lab.

Unfortunately there is no settings folder under
anaconda3\share\jupyter\lab\
On Fri, Apr 24, 2020 at 8:28 AM Robileo<EMAIL_ADDRESS>wrote:

This discussion
https://discourse.jupyter.org/t/extension-shows-up-as-both-installed-and-uninstalled-and-doesnt-work/2688
might help you. Try to delete
anaconda3\share\jupyter\lab\settings\build_config.json and then rebuild
jupyter lab.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/lckr/jupyterlab-variableInspector/issues/143#issuecomment-618851764,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGRK4KNVOTXXVUBWUIJJ3QTROE5QVANCNFSM4MNJM33Q
.

I ran into this issue after uninstalling and then reinstalling jupyterlab-variableInspector. Once uninstalling it once, it would never show the option to open the variable inspector.
I was able to fix this by going into my ~/miniconda3/share/jupyter/lab/settings/build_config.json and deleting reference to @lckr/jupyterlab-variableInspector. I did that while JupyterLab was running, and it popped up a notification that said that the Variable Inspector should be added to build. After a rebuild (or manually via $ jupyter lab build it showed back up.

I ran into this issue after uninstalling and then reinstalling jupyterlab-variableInspector. Once uninstalling it once, it would never show the option to open the variable inspector after reinstall.
I was able to fix this by going into my ~/miniconda3/share/jupyter/lab/settings/build_config.json and deleting the reference to @lckr/jupyterlab-variableInspector. I did that while JupyterLab was running, and it popped up a notification that said that the Variable Inspector should be added to build. After a rebuild (or manually via $ jupyter lab build it showed back up.

thank you soooooo much!!  you perfectly solve the problem!

I ran into this issue after uninstalling and then reinstalling jupyterlab-variableInspector. Once uninstalling it once, it would never show the option to open the variable inspector after reinstall.
I was able to fix this by going into my ~/miniconda3/share/jupyter/lab/settings/build_config.json and deleting the reference to @lckr/jupyterlab-variableInspector. I did that while JupyterLab was running, and it popped up a notification that said that the Variable Inspector should be added to build. After a rebuild (or manually via $ jupyter lab build it showed back up.

Unfortunately when I went to /anaconda3/share/jupyter/lab/settings the folder was empty.

@sama2689: When you type jupyter labextension list you'll get something like
JupyterLab v2.1.1
Known labextensions:
   app dir: /Users/$USER/miniconda3/share/jupyter/lab
        @jupyter-widgets/jupyterlab-manager v2.0.0  enabled  OK
        @jupyterlab/debugger v0.2.0  enabled  OK
        @jupyterlab/mathjax3-extension v3.0.0  enabled  OK
        @jupyterlab/toc v3.0.0  enabled  OK
        jupyterlab-execute-time v1.0.0  enabled  OK
        jupyterlab-theme-solarized-dark v1.0.1  enabled  OK

Uninstalled core extensions:
    @lckr/jupyterlab_variableinspector

If it says "Uninstalled core extensions" and the variable inspector is listed in there, then you have a build_config.json file somewhere that contains @lckr/jupyterlab_variableinspector.
Try doing
$ jupyter --paths
config:
    /Users/$USER/.jupyter
    /Users/$USER/miniconda3/etc/jupyter
    /usr/local/etc/jupyter
    /etc/jupyter
data:
    /Users/$USER/Library/Jupyter
    /Users/$USER/miniconda3/share/jupyter
    /usr/local/share/jupyter
    /usr/share/jupyter
runtime:
    /Users/$USER/Library/Jupyter/runtime

and search the data folders for build_config.json.
$ find /Users/$USER/Library/Jupyter -name build_config.json
$ find /Users/$USER/miniconda3/share/jupyter -name build_config.json
$ find /usr/local/share/jupyter -name build_config.json
$ find /usr/share/jupyter -name build_config.json


Make Sized optional in FromHex trait

Better supports fixed types such as [u8; 32] by allowing the compiler to determine the data type's size.

Looks like the compiler doesn't support this feature.... huh. Going to do some local testing.

It's ?Sized not Sized?. Though I'm not sure what you are trying to achieve with this change exactly. Even with : ?Sized it wouldn't compile as the only method returns Self by value which is only allowed when Self is Sized.


Expose WARP VM with Namecheap DNS & TLS

Motivation
Post mrzzy/warp#17, WARP VM is able to expose a web accessible terminal via ttyd.
However its currently locked down by VPC firewall due to security concerns (unencrypted HTTP)

The only way to access it as of the time of  writing is to use an SSH tunnel via remote port forwarding.

Proposal
Allow secure HTTPS web browser only access to WARP VM:

[ ] Create vm.warp.mrzzy.co DNS route via Namecheap with WARP VM's IP address.
[ ] Enable Basic Authentication on TTYD using credentials passed via env vars.

Abandoned the previous approach: Nginx Reverse Proxy on Google Cloud Run:

Network traffic from Cloud Run to VPC VMs requires GCP Serverless VPC.
The minimum scale for GCP Serverless VPC is 2x  e2-micro instances.


Added Classicpress as a service

Wordpress alternative without block editor (Gutenberg), Useful for small sites & lightweight blogs.
PR includes 3 version's with or without database, also logo.svg

Thank you for the PR!


Add info output to items on what determines cost for end users per resource if total is $0 and not in free tier

I was playing around with the aws_db_instance  resource for a few hours trying to get the price past $0 for the default 730 hours a month.  I tried using the usage file since its a usage based resource, hardcoded instance type, etc. and nothing worked.  Finally after awhile I hardcoded the allocated_storage + the instance type together and finally got a cost amount past $0.  It would be useful to somehow let the end user know what determines cost in a resource so they potentially do not run into the same problem I had.  I imagine more than just the aws_db_instance would have this kind of problem.
terminal output
infracost breakdown --path . --usage-file infracost-usage-dev.yml
Detected Terragrunt directory at .
  ✔ Downloading Terraform modules
  ✔ Evaluating Terraform directory
  ✔ Retrieving cloud prices to calculate costs

Project: /terraform

 Name                                                Monthly Qty  Unit                  Monthly Cost

 module.prhlinks.aws_db_instance.selected
 ├─ Database instance (on-demand, Single-AZ, )               730  hours                        $0.00
 └─ Storage (general purpose SSD, gp2)                        20  GB                           $2.30

 module.prhlinks.aws_route53_record.selected
 ├─ Standard queries (first 1B)                 Monthly cost depends on usage: $0.40 per 1M queries
 ├─ Latency based routing queries (first 1B)    Monthly cost depends on usage: $0.60 per 1M queries
 └─ Geo DNS queries (first 1B)                  Monthly cost depends on usage: $0.70 per 1M queries

 OVERALL TOTAL                                                                                $2.30

Non working resource example
resource "aws_db_instance" "selected" {
  allocated_storage                   = local.RDS_ALLOCATED_STORAGE
  identifier                          = local.AWS_BASE_NAME
  engine                              = "mysql"
  engine_version                      = "8.0.31"
  instance_class                      = local.RDS_INSTANCE_CLASS
  parameter_group_name                = "default.mysql8.0"
  username                            = "root"
  password                            = random_password.db_password.result
  db_subnet_group_name                = aws_db_subnet_group.selected.name
  vpc_security_group_ids              = [aws_security_group.selected.id]
  skip_final_snapshot                 = true
  iam_database_authentication_enabled = true
  storage_encrypted                   = true
  snapshot_identifier                 = try(aws_db_snapshot.selected[0].id, null)
}

Working resource example
resource "aws_db_instance" "selected" {
  allocated_storage                   = "100"
  identifier                          = local.AWS_BASE_NAME
  engine                              = "mysql"
  engine_version                      = "8.0.31"
  instance_class                      = "db.t4g.xlarge"
  parameter_group_name                = "default.mysql8.0"
  username                            = "root"
  password                            = random_password.db_password.result
  db_subnet_group_name                = aws_db_subnet_group.selected.name
  vpc_security_group_ids              = [aws_security_group.selected.id]
  skip_final_snapshot                 = true
  iam_database_authentication_enabled = true
  storage_encrypted                   = true
  snapshot_identifier                 = try(aws_db_snapshot.selected[0].id, null)
}

Thanks @Shocktrooper.
Currently Infracost shows a warning log: No products found for aws_db_instance.selected Database instance..., using 0.00 when you run with --log-level=warning
I think we should probably do the following

Show a better warning if we cannot detect the instance type.
Show this warning in the main output even when logging isn't enabled.

In terms of fixing the root cause of this issue, do you have the Terraform code where you're setting the value of local.RDS_INSTANCE_CLASS?


Clipboard cannot delete password automatically

As title.
win+v will show password in the clipboard and it will not be deleted automatically.

There is no easy solution to this, unfortunately. There are some technical reasons why deleting items from the clipboard history is currently not supported. I investigated this a bit more, and I found out that there may be a few options to make it work. I'll track that in #74.

The reason it shows the password there is because you have clipboard history enabled, which will remember old passwords even after they're cleared by pass-winmenu. If you can live without it, the easiest solution is to simply disable this feature (Settings > System > Clipboard > Clipboard history), but if you use it, that's of course not an option.
If you don't want to disable clipboard history, probably the best solution would be to prevent pass-winmenu from placing passwords on your clipboard, and let it type your passwords into login forms instead. You can do this by editing your hotkey configuration and setting copy-to-clipboard to false.

Thanks for quick reply, Since I use this tool in my personal, it doesn't matter actually.

There is the secret option (used by wclip that hides the clipboard content from many clipboard history tools, like Aceme, Ditto, ..


gardener-apiserver: Improve the HPA -> VPAAndHPA transition

How to categorize this PR?

/area auto-scaling
/kind enhancement
What this PR does / why we need it:
See https://github.com/gardener/gardener/blob/d744f46804ebe36ef944b5de2ece32950260d007/pkg/component/gardener/apiserver/deployment.go#L151-L161
Which issue(s) this PR fixes:
Follow-up after #9735
Special notes for your reviewer:
The HPA -> VPAAndHPA transition for gardener-apiserver transition is not perfect as step 2. can also decrease the gardener-apiserver Deployment replicas to 2, set by the GRM's HA webhook because at that time there is no HPA object yet.  Then the new HPA gets created and for HA Garden cluster, it is created with minReplicas=3. There can be also a change in the Pod template in the Deployment - in the minDomains of the TSC which is computed as min(maxReplicas, numberOfZones).
At least, this PR improves from Deployment resources aspect and makes sure that the new gardener-apiserver Pods won't be created with the initial/default Deployment resource requests.
Release note:

gardener-operator: The HVPA -> VPAAndHHPA autoscaling mode transition is now improved for the gardener-apiserver to preserve the Deployment resources.

/approve
@voelzmo could you please check and lgtm? 🙂

/test pull-gardener-e2e-kind-ha-multi-zone


Better Documentation

Is there any better documentation for using this package?

Hi @maxwaiyaki. By better docs, do you mean improved installation instructions or API docs for the internals of the wagalytics codebase? Any more context would be useful.

Since there's not enough information I'm going to close this. I'll reopen if more context is provided. Thanks!


Administratively set global models should not have settable org/is_global fields

In GitLab by @jasonpagetas on Jul 15, 2024, 22:53
When creating a new Device type, or device model you must select an organization before you can submit. Regardless of that choice the option is reverted back to global once submitted and there is no way to unselect this option.
Suggest this changes to include a check box for "is global" which then removes the organisation dropdown
Tasks

Remove form fields organization and is_global from:

[x] manufacturer

[x] device type

[x] device model

[x] software categories

marked the checklist item software categories as completed

added 1m of time spent

marked the checklist item manufacturer as completed

mentioned in commit 7b26fac73d38d9644e20db0b3a85b4def73c9bf1

added 1m of time spent

marked the checklist item device type as completed

marked the checklist item device model as completed

In GitLab by @jasonpagetas on Jul 15, 2024, 22:53
marked this issue as related to #74


Improve UX Further

Short description of what this resolves:
This PR improves the UX of Settings Sync
Changes proposed in this pull request:

If no gists are available, tell the user to skip and upload
Properly download after selecting gist

Fixes: https://github.com/shanalikhan/code-settings-sync/pull/969#issuecomment-513135826
Checklist:

[x] I have read the contribution guidelines.

Did you test the code ?
Now the upload is not working.... was working previously.
Removed the out directory and created package to test and installed.

Reset Settings and Reload Code
Disable the Donot Show button.
Hit upload > The Window is not opening instead shows Invalid Token error.

@shanalikhan I just tested it with a clean portable version and it works exactly as expected.

sorry, my bad. Thanks


Rebuild on top of newterm

I just wrote a PC dependency for launching terminals, with the added functionality of allowing environmental variables to be set. https://github.com/codexns/newterm
It works around all sorts of issues with the current version, including:

Does not invoke scripts so the package can be placed in a .sublime-package file
Uses OS APIs instead of command line scripts, so there is less indirection
Allows control over the width of powershell and cmd.exe processes
Adds use_tabs param for both Terminal.app and iTerm.app on OS X
Handles invoking cmder.exe properly on Windows so that is ends up in the correct folder
Since it uses ctypes to call CreateProcess() on Windows, the various options needed to properly launch powershell are available, thus there is no need for the .bat script anymore and the cmd.exe will not flash momentarily.

The next step is to port this package to use it.
In the process is probably makes sense to break the key binding - perhaps just include a command palette entry and then in the readme include snippets for users to set up their own key bindings.
And finally, we may need to tweak the options in this package and/or add %CWD% support to newterm.

That's cool, I'm willing to help if you need a hand on OS X or Linux.

@wbond On #99, you mentioned that the main task remaining for this is moving users from an older config to the new one. Here are a few questions I have

What are your thoughts on the config? Pretty sure it will be the kwarg parameters from newterm.launch_terminal (i.e. env, terminal, args, width, use_tabs).

Will %CWD% replacement occur in Sublime Terminal still?

What is our plan for the transition? Here are some quick ideas:

Hard switch (e.g. break installations immediately)

Provide user a message
Move immediately to new settings keys
Ignore older keys (e.g. parameters)

Soft switch (e.g. allow old installations to last but consider them deprecated, remove in 3 months)

Provide user a message
Probably write duplicate code that sniffs for parameters and runs appropriately

Alternatively request users to set a custom setting to use newterm (doubt this will get used)

Possible problem: people might ignore it

Create new package

Provide users an installation and upgrade message about Terminal being deprecated in favor of NewTerminal
Let Terminal live forever
Write new package NewTerminal

I hadn't thought too much about exact what the config would look like. I do believe exposing the ability to use tabs on OS X should be a requirement.
I have not currently implemented the ability to pass the working directory as an argument. Instead, my plan is to support all of the major shells for a given OS out-of-the-box, and then if users want something custom, they can deal with the params themselves. The list of supported terminals is currently:

OS X

Terminal.app
iTerm2.app

Windows

Powershell
cmd
ConEmu
cmder

Linux

gnome-terminal
xfce4-terminal
konsole
lx-terminal
mate-terminal
xterm

The way that custom terminal programs will work is that any batch/shell script will be executed within the working directory, and the script can translate that into whatever params the given terminal program needs.
In terms of a launch, I have been considering a hard launch of 2.0 that breaks backwards compatibility in terms of settings and key bindings. However, I'd like to also create a new package named Terminal 1 for users who want the old functionality. It would be the current codebase.

With respect to config, I think exposing the keys 1:1 via JSON which maps directly to kwargs will be the sanest for us to manage. That includes use_tabs so :+1:
With respect to invocation, I am a bit fuzzy on how a custom terminal will work. Here's an example of what I think you are explaining. If that's not right, please correct me =)

Launching the default terminal for an OS

newterm.launch_terminal('/path/to/cwd')

Launching a known terminal, non-default terminal

newterm.launch_terminal('/path/to/cwd', terminal='xterm')

Launching an unknown terminal

newterm.launch_terminal('/path/to/cwd', terminal='/usr/bin/new-term')

Launching a terminal with custom environment variables as arguments

`newterm.launch_terminal('/path/to/cwd', args=['$HELLO'])
This likely won't work because environment variable syntax could change from shell to shell so it's best to avoid it altogether

With respect to launching, I like that plan a lot =) :100:


[question] Impact of "Chromium CDM" and Widevine deprecation

Given the move from Google to deprecate the Widevine CDM for Chrome and Chromium browsers, I was wondering if this will also impact Electron, and how Electron ECS/Castlabs EVS will be impacted by it. What do developers have to do (if any) to ensure their services can continue to run?
https://bitmovin.com/google-deprecating-widevine-drm-cdm/
https://www.xda-developers.com/google-is-deprecating-widevine-for-chromium/

As stated in the articles only old versions of the CDM will be revoked, and well before that time a new CDM version will be available. As long as automatic updates have not been disabled (i.e. the Component Updater for all currently supported versions of ECS) a user is unlikely to even notice this happening.
There are a couple of theoretical edge cases:

If an instance is left open for an extended period of time, a restart may be needed for the update to take effect
If an instance is opened/closed only in quick succession there may not be time to perform the update

Oh, if that's the case, I misunderstood! So they are only deprecating the older Widevine CDM, and the only reason why it's newsworthy is because browsers ship with the CDM and Electron downloads it at the first launch? If so, I think this is answered and can be closed, thanks! 🙏

It is mostly correct too, although the last sentence is certainly misleading. The new CDM started launched in Canary some time back and will progressively be promoted until it reaches all Chrome installations sometime mid-November, including forks. The misleading part is "Widevine being blocked entirely", which should actually only apply the the old CDMs. The new CDM, version 4.10.2557.0 IIRC, will not be revoked.

Thanks! Is there any way to get the current CDM version and know when it's updated? I think there were events on app but I no longer see them in the type definitions, are they still available?

Since ECS moved to using the Component Updater there are no longer any events the same way they existed in the prior implementation. You can however use the components API, specifically the status() function, to retrieve the version of the loaded CDM (or CDMs). The example code in the README has a very simple example of this that just logs the result to the console.

Alright, that's nice :) Does it only check for updates when the application first boots? (e.g. when it calls whenReady?) Or can updates happen at any time?

The whenReady() call will force an immediate installation if no CDM can be found/registered, otherwise it will rely on the Component Updater checking for and installing updates in the background (same as Chrome).

Hello,
Do we have to manually update the Widevine CDM in our electron app? This page suggests that we should use the chrome library and put it in.

No, if you are using ECS the update is automatic (unless you explicitly disable it).
The linked instructions do not apply to ECS, only stock Electron, and since VMP was introduced are they are not really viable for macOS/Windows there either. There is also the fact that bundling the CDM might require you to become a MPEG-LA licensee because of the integrated decoders (as opposed to dynamically installing it form Google servers).

Hey, @khwaaj thanks a lot for shading some light :), however, I'm still a bit confused say we have old ECS version 6 with its CDM v4 do we need to update ECS to be able to update to the latest CDM or once old CDM is revoked, next ECS launch will update it automatically? Thanks in advance

Well, version 6 is certainly not a supported version anymore, that is for sure! 😁
That said, Google still publishes the CDMs to Lorry (this one should land sometime around mid-November), which is the old delivery method used in the wvvmp releases, so the CDM will be installed (unless you disable it) and is likely to work as well. The earliest officially supported version of Chromium for this CDM is 95 though, which would mean ECS 16, so I can't promise there won't be issues.

To be sure, in order to get the latest version of Widevine, should we upgrade to ECS version 16?

You will get the new CDM even with older version, but I'd still recommend upgrading to a currently supported version, which would mean ECS 19. ECS 16 is just the first version to pass the official minimum Chromium version requirement for the new CDM (95), it is no longer maintained/updated.

@khwaaj I was able to update to version electron 21.2.3+wvcus however the after first launch there is no CDM folder, it used to be in win ~\AppData\Roaming .... and now it's empty is it expected or it should be other place?

If you updated from an older version, i.e. from a wvvmp version to a wvcus version, there are breaking changes to the API. Instead of waiting for widevine-ready you need to use the new components API, as described in the README. Not sure if you have already updated your app to support this, but not using the components API to wait for Widevine to become ready would give the problem you are seeing.

Google's, Chrome updates information service Is now returning new CDM version 4.10.2557.0. Hopefully this will be reflected soon in ECS. I am currently running ECS 19.0.15 with auto updates enabled but CDM still updates only up to version 4.10.2449.0

That particular file means that Lorry has been updated, e.g. the delivery mechanism used by the older wvvmp releases. This typically happens after CUS has been updated, but apparently not in this case as I'm still seeing 4.10.2449.0 being delivered too (unless I use the test-request channel). It could be that they are doing a staggered rollout to avoid taxing the CUS servers too hard.

@khwaaj I can see Widevine CDM has now upgraded to 4.10.2557.0 and working as expected.
If I delete the CDM folder to retest the upgrade, why is version 4.10.2391.0 installed first and only then followed by the 4.10.2557.0 update?

Why is version 4.10.2391.0 installed first and only then followed by the 4.10.2557.0 update?

I'm not sure, but I'm seeing the same behavior. This is not something that ECS does, so it is some CUS server-side decision to first deliver the older version. This should change before the older CDMs are revoked, but I'll be sure to ask when I get the chance..

@khwaaj  -  to address a use case of users launching the app with the old CDM version after it has been deprecated. I assume an app relaunch will be required after the new CDM has finished downloading?

Short answer: Yes.
Longer answer: It's complicated. It really comes down to of the CDM utility process has launched or not, if not then a restart would technically not be required. This in turn depends on what features has been used in the opened browser window(s), and down the rabbit hole we go. All in all, I'd assume a restart is required 🙂

In regard to restarting after version 4.10.2557.0 has completed downloading.
In the delivery mechanism used by the older wvvmp release we could use the 'widevine-update-pending' event as a cue on when to restart. As the new mechanism does not fire an event when the CDM has completed downloading, it seems we need to write our own custom poll to watch for any CDM folder changes?
This would especially be required for use cases where:
App is launched with a deprecated CDM.
App is already running while CDM is deprecated.

Why is version 4.10.2391.0 installed first and only then followed by the 4.10.2557.0 update?

I'm not sure, but I'm seeing the same behavior. This is not something that ECS does, so it is some CUS server-side decision to first deliver the older version. This should change before the older CDMs are revoked, but I'll be sure to ask when I get the chance.

I confirmed with the Widevine team that this is a temporary situation, and the rule that is behind it will be removed "soon".

In the delivery mechanism used by the older wvvmp release we could use the 'widevine-update-pending' event as a cue on when to restart. As the new mechanism does not fire an event when the CDM has completed downloading, it seems we need to write our own custom check to watch for any CDM folder changes?

@shmulka, if you think you'll need this, then yes - you'll have to monitor for updates manually since this type of API is not provided after the update to wvcus. Instead of monitoring the file system , you could also monitor using the components.status() API, which will provide the necessary information. IIRC the status field will say updated if an update has been applied.
Typically the edge cases you speak of should be rare since there is often a grace period between a new CDM release and removal of the old one, even Chrome doesn't handle them, but they are certainly possible under some circumstances. In this case the grace period is relatively short as well, which increases chances of this happening somewhat.

@khwaaj,  monitoring the components.status() is a good idea, but it seems there is a another issue now.
While  approx a minute after startup , the status does change from status=new -  version=4.10.2391.0  to  status=updated  - version=4.10.2557.0,  no update occurs.  Only folder visible of version 4.10.2391.0.

@shmulka, indeed, I see the same thing (tested several different ECS versions). There has to be something wrong with the service at the moment, as it appears to update but the updates are never applied regardless of what I do. I'll escalate.

@shmulka, it should be back to working now, but also still serving 4.10.2391.0 initially. There was some kind of issue with the configuration change, so it was rolled back and will be revisited.


Use mv and cp from File::Copy instead of copy and move

Case RE-723:  As it turns out, File::Copy provides copy and cp that behave slightly different.  The copy and move subs use the umask provided by the running processes shell.  The cp and mv try to preserve permissions for the file that is being copied or moved.  As such, we should prefer cp and mv over copy and move from File::Copy.
Changelog: Use mv and cp from File::Copy instead of copy and move
By submitting pull requests to this repo, I agree to the Contributor License Agreement which can be found at: https://github.com/cpanel/elevate/blob/main/docs/cPanel-CLA.pdf

NOTE:  This failure actually shows that elevate succeeded:
https://github.com/cpanel/elevate/actions/runs/11298668305/job/31430465285?pr=524
So I am not going to bother rerunning the CI so that it can get a green check mark (there is a bug in the CI here)

@cPholloway please update this with elevate/main so we can pass the workflow checks.


Missing output from the default-boxfile hook

We discussed this a bit, but I finally pinpointed the reason for the missing logs.  The payload bit: "logtap_host":"x.x.x.x" is required for sending data to logtap in the hook, yet is missing in the payload.
Earlier, it sounded like you were working on additions to the default-boxfile payload, but I could find no related code change.
Seems like adding "logtap_host": config.LogtapHost to the payload here will cause this to be obsolete, as we print the generated boxfile in debug level here

this issue has been fixed in https://github.com/nanobox-io/nanobox-server/commit/5555a083e1aa595ea7e09e894306e9f97b36f139


Don't render null ssh key into cloud-init

What this PR does / why we need it:
Avoids rendering a nil ssh_keys field as some providers cloud-init cant handle that. Also avoids rendering comments into the cloud-config
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #556
Special notes for your reviewer:
none

/retest

/lgtm
/approve


music bot

Description
make a music bot that can play music from any website and any video
Reasoning
it would be a great addition to python discord server because i know many people that like to listen to msuic while coding
Proposed Implementation
i give code
Additional Details

Would you like to implement this yourself?

[x] I'd like to implement this feature myself
[ ] Anyone can implement this feature

Hello @Dragon-Batch,
Thank you for your interest in contributing to one of our open source projects. Unfortunately, we don't want to add the proposed feature to one of our bots or have such a feature available in our community.
One of the reasons is that most music bots violate both copyright laws as well as the terms of service of the websites/service they use as the source for the music they play. Another reason is that we don't want to have such bots in our voice channels: Our voice channels are typically already quite busy with conversations and we don't want to open more voice channels, as moderating voice channels is a challenge.
My suggestion to you is to play music locally on your computer while you're coding.
Kind regards,
Sebastiaan


Python.exe processes being left behind using ADS

Through my normal usage pattern, lots of python.exe processes are getting left behind.  Some of these processes end up holding locks on files, which then tend to break other things (like upgrades to those files).
These python.exe processes are still around after stoppping and restarting ADS many times.
C:\temp>kill python
process python.exe (22448) - '' killed
process python.exe (17164) - '' killed
process python.exe (23784) - '' killed
process python.exe (17636) - '' killed
process python.exe (9828) - '' killed
process python.exe (20128) - '' killed
process python.exe (1836) - '' killed
process python.exe (23864) - '' killed
process python.exe (2036) - '' killed
process python.exe (16404) - '' killed
process python.exe (22960) - '' killed
process python.exe (3692) - '' killed
process python.exe (14404) - '' killed
process python.exe (22216) - '' killed
process python.exe (23968) - '' killed
process python.exe (19880) - '' killed
process python.exe (13816) - '' killed
process python.exe (8252) - '' killed
process python.exe (11588) - '' killed
process python.exe (18856) - '' killed
process python.exe (13644) - '' killed
process python.exe (19480) - '' killed
process python.exe (21096) - '' killed
process python.exe (15808) - '' killed
process python.exe (25832) - '' killed
process python.exe (17456) - '' killed
process python.exe (5960) - '' killed
process python.exe (21592) - '' killed
process python.exe (18988) - '' killed
process python.exe (21248) - '' killed
process python.exe (9896) - '' killed
process python.exe (19624) - '' killed
process python.exe (25144) - '' killed
process python.exe (24928) - '' killed
process python.exe (19124) - '' killed
process python.exe (18088) - '' killed
process python.exe (22136) - '' killed
process python.exe (28504) - '' killed
process python.exe (6756) - '' killed
process python.exe (13008) - '' killed
process python.exe (23292) - '' killed
process python.exe (16752) - '' killed
process python.exe (22816) - '' killed
process python.exe (1364) - '' killed
process python.exe (12008) - '' killed
process python.exe (11072) - '' killed
process python.exe (8172) - '' killed
process python.exe (1416) - '' killed
process python.exe (16196) - '' killed
process python.exe (22352) - '' killed
process python.exe (7392) - '' killed
C:\temp>

This is a dupe of #3477


Frontiers paper (due Feb. 4)

We are planning to re-submit the DORA manuscript to the Frontiers Research Topic on Applications of Statistical Methods and Machine Learning in the Space Sciences. The extended due date for submissions is February 4.

This has been submitted! The Zenodo repository with the datasets we used in the paper is still being finalized, but the link is included in the paper.

@hannah-rae Did you receive the rover data set I sent via LFT on Feb. 7?  I want to be sure I'm not holding things up :)


No more than 20 characters per annotation

Is your feature request related to a problem?
If an annotation is too long, some of its last characters might be inside the buttons that surround it to the right
Desired solution
Have the annotation input changed to not accept more than 20 chars
Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
No response

https://github.com/lars-reimann/api-editor/issues/931 subsumes this issue. We need a more sturdy solution than limiting the number of characters per annotation arbitrarily.


Enable packages to intercept click events on the editor

Currently, we don't have a good story for enabling packages to handle click events in the editor, that I know of. How should we do this?

One idea is to enable packages to register click event handlers for the lines and the gutters. For every click, their handler would be invoked. They would need a way to tell us not to continue processing the event. Do they call stopImmediatePropagation on the event? Return a boolean from their handler?

@maxbrunsfeld mentioned in #6843 something even more general based on commands. Max if you have more thoughts on that maybe you could edit this and add them here?

One thing to add is it would be nice to expose some methods for translating clicks to Atom-specific information. For example, translating the mouse coordinates to a screen line number.

I agree, a function to translate pixel position into a cursor position would be very useful for my atom-touch-events package.
I am still yet to work out how to do that accurately.

@jssln There's a private method, screenPositionForPixelPosition on TextEditor. I don't want it to be public on the model because it shouldn't be called when the editor isn't on screen. But we could add a public method to the view that delegates to it. If you want to open a PR with that, would be an easy merge.

cool, https://github.com/atom/atom/pull/7144

It seems screenPositionForMouseEvent (https://github.com/atom/atom/blob/dac39bdcd5a14cc227d89f1ecc63d4dad8306fe0/src/text-editor-component.coffee#L763) would be more convenient to use. Thoughts on exposing that?

Yeah I use screenpositionformouseevent when handling tap events using atom-touch-events package.
It would be great to be exposed publicly.

I could create a PR to create a public TextEditorView.screenPositionForEventPixelPosition which converts a clientX/Y to a Point row/column.
Would that be an option?

@jssln @russlescai I'm definitely on board for adding a public method like this. Struggling a bit with the name. The existing name (and the concept expressed in the method itself) feels a little weird.
I like where @russlescai is going with the "event pixel position" idea. What about  TextEditorElement::screenPositionForClientPixelPosition. It would take a hash with top and left values and assume the need to normalize relative to the scroll position like screenPositionForMouseEvent does.
Come to think of it, with tiled rendering I wonder how much sense the existing pixelPositionForScreenPosition method makes anymore. We no longer scroll the entire content as a single GPU layer, making absolute pixel positioning relative to the global content quite a bit less useful.


cloud_to_cloud:java:csharp_server:large_unary interop failure

https://grpc-testing.appspot.com/job/gRPC_interop_pull_requests/11791/testReport/junit/(root)/tests/cloud_to_cloud_java_csharp_server_large_unary/
Running test large_unary
Mar 01, 2017 5:40:42 PM io.netty.util.internal.MacAddressUtil defaultMachineId
WARNING: Failed to find a usable hardware address from the network interfaces; using random bytes: e7:2c:69:3f:31:2a:73:70
Exception in thread "main" io.grpc.StatusRuntimeException: INTERNAL: Connection closed with unknown cause
	at io.grpc.stub.ClientCalls.toStatusRuntimeException(ClientCalls.java:227)
	at io.grpc.stub.ClientCalls.getUnchecked(ClientCalls.java:208)
	at io.grpc.stub.ClientCalls.blockingUnaryCall(ClientCalls.java:141)
	at io.grpc.testing.integration.TestServiceGrpc$TestServiceBlockingStub.unaryCall(TestServiceGrpc.java:408)
	at io.grpc.testing.integration.AbstractInteropTest.largeUnary(AbstractInteropTest.java:225)
	at io.grpc.testing.integration.TestServiceClient.runTest(TestServiceClient.java:213)
	at io.grpc.testing.integration.TestServiceClient.run(TestServiceClient.java:197)
	at io.grpc.testing.integration.TestServiceClient.main(TestServiceClient.java:81)
Shutting down

seen again https://grpc-testing.appspot.com/job/gRPC_interop_pull_requests/11728/#showFailuresLink
pr https://github.com/grpc/grpc/pull/9894


Refactor fairness metrics toolbox for classification

Description

main.py as the main entry point
moved test assests to stock-plugins root (stock-plugins/user_defined_files)
refactored the folder and files structure.
Users can now execute the plugin as a module
Plugin will now accept comma separated assets paths (data,model,ground_truth), model classifications, ground truth value

Type of Change

refactor: Code refactoring without changes in functionality

perf: Performance improvements

How to Test
Execute the below commands to run unit tests -
cd aiverify/stock-plugins/aiverify.stock.fairness_metrics_toolbox_for_classification/algorithms/fairness_metrics_toolbox_for_classification/
pytest  .

Checklist
Please check all the boxes that apply to this pull request using "x":

[x]  I have tested the changes locally and verified that they work as expected.
[x]  I have added or updated the necessary documentation (README, API docs, etc.).
[x]  I have added appropriate unit tests or functional tests for the changes made.
[x]  I have followed the project's coding conventions and style guidelines.
[x]  I have rebased my branch onto the latest commit of the main branch.
[x]  I have squashed or reorganized my commits into logical units.
[x]  I have added any necessary dependencies or packages to the project's build configuration.
[x]  I have performed a self-review of my own code.
[x]  I have read, understood and agree to the Developer Certificate of Origin below, which this project utilises.

Screenshots (if applicable)
[If the changes involve visual modifications, include screenshots or GIFs that demonstrate the changes.]
Additional Notes
[Add any additional information or context that might be relevant to reviewers.]

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
   have the right to submit it under the open source license
   indicated in the file; or

(b) The contribution is based upon previous work that, to the best
   of my knowledge, is covered under an appropriate open source
   license and I have the right under that license to submit that
   work with modifications, whether created in whole or in part
   by me, under the same open source license (unless I am
   permitted to submit under a different license), as indicated
   in the file; or

(c) The contribution was provided directly to me by some other
   person who certified (a), (b) or (c) and I have not modified
   it.

(d) I understand and agree that this project and the contribution
   are public and that a record of the contribution (including all
   personal information I submit with it, including my sign-off) is
   maintained indefinitely and may be redistributed consistent with
   this project or the open source license(s) involved.

@imda-peckyoke  : The image_pipeline data set is failing with the below error.So, I am commenting it for now.
(There was an error deserializing the model: XXX/aiverify/stock-plugins/aiverify.stock.fairness-metrics-toolbox-for-classification/algorithms/fairness_metrics_toolbox_for_classification/../../../user_defined_files/pipeline/bc_image_face)

@imda-kelvinkok  : can you help check if the data set is valid ?
ref - https://github.com/aiverify-foundation/aiverify/pull/313/files#diff-55cdd7bb8b61c78e907d7e4cf5a58b741221edfb870a0e933fb66a514034dcc5R93


Tracking of projects that are migrating from GitHub (to GitLab, etc.)

Issue description

GitLab monitoring is here
This is a meta issue thread about projects migrating their upstream from GitHub to GitLab and other places.
How we can structure tracking of this changes?
It is better if someone with Maintainer authority can close this issue and reopen a thread like this, or maintainers form a unified strategy how we handle this and inform the community.

Folowing @volth in https://github.com/NixOS/nixpkgs/issues/41448#issuecomment-394522865:
volth:
16000 of fetchFromGitHub across nixpkgs

We will definitively get a lot of dead links, if not as the result of Microsoft activity then as the result of developers deleting their accounts

For example, if GitHub URL becomes unresponsive, something accordingly watch does the same project exists on GitLab. Maybe after call fetchFromGitHub fails - launch does it on according fetchFromGitLab (gitlab.com) URL, and output a message "Please look, maybe project migrated to GitLab " that can cover huge part of cases of migration.

I think crawling the web for information about packages is hard. It seems like every package does things slightly differently.
nixpkgs-update uses Repology to get information about the URLs for sources and it can even change the URL for a fetchurl src derivation. It should be possible to detect the switch from GitHub to GitLab and change the fetcher. If someone knows an example of a project that has made the move, I'd like to hear about it so I can try testing switcher code on it.

It does not mean bot would switch automatically.
I talked about informing with "a message".
So far as I look not so many big projects moving.
It is mostly personal accounts of developers. But them also have important projects.
So far I found that "Ring project" moved to seft-hosted GitLab (link). But it is so far not in NixPkgs.
You can look Twitter for #movingtogitlab and they have tracking account at @movingtogitlab.

Well, for today this is not needed.


[Update Snap] Silent Shard

Checklist
All items in the list below needs to be satisfied.

[ ] Is the summary of the change documented in this ticket?
[ ] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[ ] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

Update Summary:

Update changelog: Enable force update features for Snap users
We are adding a few new security features in the upcoming releases which will require the users to be moved from the existing security systems to the upgraded ones without anyone left behind in the older versions of the Snap

Source Diff:
https://github.com/silence-laboratories/silent-shard-snap/compare/main...1.2.8
Version to be allowListed: 1.2.8


Fetch relationship. The return

Hi,
Just looking at the test over association. Especially this one :+1:
class Owner < TestResource
  has_many :properties
end

class Property < TestResource
  has_one :owner
end

  def test_load_has_one_nil
    stub_request(:get, "http://example.com/properties/1")
      .to_return(headers: {content_type: "application/vnd.api+json"}, body: {
        data: [
          {
            id: 1,
            attributes: {
              address: "123 Main St."
            },
            relationships: {
              owner: {
                links: {
                  self: 'http://example.com/properties/1/links/owner',
                  related: 'http://example.com/people/1'
                }
              }
            }
          }
        ]
      }.to_json)

    property = Property.find(1).first
    assert_equal(nil, property.owner)
  end

The owner is not include in the first request, however we know that the path of the resource is :
http://example.com/properties/1/links/owner
Could it be possible to make a new request to load it when calling :
property.owner

    def method_missing(method, *args)

      if relationships && relationships.has_attribute?(method)
        if last_result_set.included && last_result_set.included.data[method.to_s.pluralize]
          last_result_set.included.data_for(method, relationships[method])
        else
          // Get ressource at /current_ressource/id/method
        end
      else
        super
      end
    end

What do you think about the idea?

I had implemented the has_one association before the relationships spec was included in the jsonapi spec. You actually don't need the has_one declaration because the resource looks for the relationship definition when calling property.owner if it wasn't included in the response.

Hmm, I don't see my code that follows the link.


Change namespaces

In order to facility allowing users to import and process files at runtime, need to look into changing the namespaces of both the MonoGame.Aseprite and MonoGame.Aseprite.ContentPipeline assemblies.

Currently all importing and processing is done through the MonoGame.Aseprite.ContentPipeline.dll library In order for importing and processing to be performed during runtime, users would need to reference that library in their game project as well as the MonoGame.Aseprite.dll library.

The problem here is that both of these contain .Aseprite in their respective root namespaces, and it could potentially be confusing having the .ContentPipeline namespace as part of a completely different assembly.

Would potentially need to rename the namespaces to resolve this, which would be a breaking change.

Originally posted by @manbeardgames in https://github.com/manbeardgames/monogame-aseprite/issues/7#issuecomment-738489464

Closing this since importing is now possible without the MGCB https://github.com/AristurtleDev/monogame-aseprite/releases/tag/version%2F4.0.0


async-profiler: init at 1.8.4

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 115909 at 0a8370bb run on x86_64-linux 1

1 package still failed to build (pre-existing failures on hydra):

async-profiler

What still needs to be done here?


wrong method used in Documentation Usage.md#manual-authentication

Issue Link :link:
There seems to be a typo in the current version of the documentation. The example code in  Usage.md#manual-authentication won't compile in Xcode. This commit should fix it.

Right you are. Thanks!


Hangs on macOS

hi,
I was trying out your class on macOS and noticed that it is causing an infinite display loop on:
public func display(_ layer: CALayer) {
if you remove the needsDisplay there it will fix it.
needsDisplay = true
I didn't try on iOS.
Thanks!
Ryan
(XCode 15.3, latest etc)

Hi Ryan, thanks for the report and the solution. Indeed, I reproduced what you found. I've pushed basically your change and fixed two tests that run on macOS. Seems to be ok now.
Brad


[🐛 BUG] Setting progress bound value to None raises an error

What would you like to share or ask?
What went wrong? 🤔
Setting the progress bound value to None raises an error and will break the changes done to the UI as an error is raised.
C:\Users\jacta\.conda\envs\develop\lib\site-packages\taipy\gui\gui.py:1477: TaipyGuiWarning: on_action(): Exception raised in 'set_value_to_none()':
Variable 'value' should hold a number: float() argument must be a string or a real number, not 'NoneType'
  _warn(f"on_action(): Exception raised in '{action_function.__name__}()'", e)
C:\Users\jacta\.conda\envs\develop\lib\site-packages\taipy\gui\gui.py:1449: TaipyGuiWarning: on_action(): 'set_value_to_none' is not a valid function.
  _warn(f"on_action(): '{action}' is not a valid function.")

Expected Behavior
It should support the None variable and be in its loading phase.
Steps to Reproduce Issue
from taipy.gui import Gui
import taipy.gui.builder as tgb

progress_value= None

def set_value_to_none(state):
    state.progress_value = None

with tgb.Page() as page:
    tgb.progress("{progress_value}")
    tgb.button("Set value to None!", on_action=set_value_to_none)

Gui(page=page).run(title="Frontend Demo")

Version of Taipy
develop
Acceptance Criteria

[ ] Ensure new code is unit tested, and check code coverage is at least 90%.
[ ] Create related issue in taipy-doc for documentation and Release Notes.

Code of Conduct

[X] I have checked the existing issues.
[ ] I am willing to work on this issue (optional)

I am unable to reproduce this issue.
Steps I followed:

Pulled the latest develop branch.
Installed the taipy library from the dev branch.
Created a main.py file with the provided code.
Ran taipy run main.py
I see no errors on the console. Clicking on the "Set value to None!" button also doesn't raise any errors or warnings.
The progress bar also is in its loading phase.

My bad, I changed the code. You should be able to reproduce it


[Bug] Slice hotkey not functioning properly when using [Click on Selection] mode

Win10 Home   21H1  19043.2130
v<IP_ADDRESS>
Action: Using a slice hotkey when in [Click on Selection] Activation Mode
Result: The hotkey is "stuck" and is continuously being sent to the pie menu upon pressing the pie key.
Kept getting stuck in sub-menus, but it turns out the behavior applies to other slices as well.
Love the project btw :)

I saw the name "picklepissjar" and knew immediately that this issue was coming from a technically literate user and had to be addressed (also this issue happened to be easy).
This should now be fixed in v1.0.25.  Can you confirm that it works on your end?

It's working on my end. 👍


Add logos to new DAOS [M]

Summary
As a new DAO user I would like to see my own logo in my new DAO (not the hypha logo)
AC
Change logo:

[x] on the DAO switcher (left nav)
[x]  on the exploration card (center)
[x]  on the login page
[x] favicon (Browser)
[x] Chagne DAO name to alias on Explore Page (not hypha but Hypha)
[x] Change Copy in Browser Tab too from "Welcome to Hypha DAO" to "Welcome to XX. DAO"

Locations

[ ] https://dao.hypha.earth/samaratrust/login
[ ] https://dao.hypha.earth/originwisdom/login
[ ] https://dao.hypha.earth/btchangedho/login
[ ] https://dao.hypha.earth/momdho/login
[ ] https://dao.hypha.earth/seedscommons/login

@jmgayosso here are the logos with names:
Samara Trust:

Origin Wisdom:

Btchangedho:

momdho:

Seedscommons:
no logo

@jmgayosso I can't really test the favicons because the DEV dos dont have logos. Can you add some?
All of them currently have the old logo for me (I did a hard refresh)

Also the text is not correct:
In the browser Tab it should say "Welcome - Regen Living"
https://dao-dev.hypha.earth/btchangedno/


Output from gradle is hidden causing circle CI to timeout and causing issues in debugging

CircleCI starts a timeout timer when logs stop and normally if you have gradle output, on a hang, logs may stop 10 minutes in.  Instead, the hang appears to start 10 minutes early.  We had to increase timeout to 60 minutes which is really bad in case of real hang in that it waits 60 minutes instead of 10 minutes from when gradle output starts.
Another issue is we ran into issues and added System.out.println("asdfsdf") to gradle build and it is not output to circle CI as it goes to a file.  This is quite annoying as well.
While the summary is quite nice, the gradle output for debugging far outweighs the verboseness (which I understand is also annoying).

Worst case, let developers configure the gradle logging to be on.


Question: Graph2Seq

I'm thinking about implementing graph-to-sequence; I'm considering Graph2Seq (paper and Tensorflow version). The Graph2Seq Encoder is similar to GraphSAGE, but it looks at embeddings/representations within forward and backward neighbor sets.  I've looked through some relevant material here on GraphSAGE, the reddit.py implementation, and NeighborSampler.
I can think of my graphs as directed graphs, for the purpose of this; the nodes in the graphs can be ordered.
I am not exactly sure how to implement this in Pytorch Geometric, but I can see building off of the GraphSAGE layer for CONCAT.  But AGGREGATE seems like it could be a challenge.
I'd appreciate any thoughts you and others may have:

Are there any roadmap plans for graph-to-sequence support in Pytorch Geometric?
Are there better graph-to-sequence alternatives than Graph2Seq?
If I go with Graph2Seq: I'd have to create my own sampler, probably by building up lists of fwd and bwd nodes for each node in the graph.  (That's what the TF version does, see starting here).  My plan was to just keep a static list of fwd/bwd neighbors in each graph, something like

   kwargs={'fwd':fwd_list, 'bwd': bwd_list}  # fwd_list, bwd_list are lists of neighbor lists
   data = Data(x=x, y=y, edge_index=edge_index, **kwargs)

and then use them in a modified version of NeighborSampler (using data.fwd and data.bwd instead of self.adj.sample_adj()), but otherwise try to structure along the lines of reddit.py.  It seems a little strange to me to organize it like reddit.py; it seems better to encapsulate the neighbor sampling within the model, instead of having it be explicit in the training loop(?)

Given the above, are there any performance considerations or concerns?

For ease-of-reference, Graph2Seq framework and encode embedding algorithms are included below, from the paper referenced above.

Thanks for this interesting issue. I think this is a model that can be perfectly implemented with PyTorch Geometric.

As far as I can tell, the proposed model does not make use of sampling. You can implement forward and backward message passing aggregation by making use of the transposed graph connectivity:

def forward(self, x, edge_index):
    row, col = edge_index
    edge_index_T = torch.stack([col, row], dim=0)
    h_forward = self.propagate(edge_index, ...)
    h_backward = self.propagate(edge_index_T, ...)

If you want to make use of sampling, I suggest to create two samplers that take in the original edge_index and its transposed version.
Mean- and max aggregation are straightforward to implement. I would drop LSTM aggregation as it does not work anyway.

OK, thanks for the feedback, I'll give this a try!


Fails to build with OpenSSL 1.1

Trying to build against OpenSSL 1.1 is failing due to fact that the SSL struct in 1.1 is opaque.
ngx_http_auth_ldap_module.c: In function 'ngx_http_auth_ldap_ssl_handshake':
ngx_http_auth_ldap_module.c:1394:79: error: dereferencing pointer to incomplete type 'SSL {aka struct ssl_st}'
         int setcode = SSL_CTX_load_verify_locations(transport->ssl->connection->ctx,
                                                                               ^~

https://github.com/kvspb/nginx-auth-ldap/pull/177


Use relaton-py library to generate BibXML

We currently need two repositories per dataset. The relaton-data one for search indexing (as it provides more data) and the bibxml-data one for serving BibXML.
The latter dataset will no longer be needed once the service integrates the relaton-py library which converts Relaton data to BibXML on the fly. This is also necessary for serving other bibliographic formats like bibtex.
Once this is done we can also remove the unneeded dataset repos.
Originally posted by @ronaldtse in https://github.com/ietf-ribose/bibxml-service/issues/41#issuecomment-1000746885

@ronaldtse this relies on relaton-bib library, which is not ready yet.
@CAMOBAP  doesn't work on bibxml-service  codebase.

@strogonoff the YAML parsing part is already filed here: https://github.com/relaton/relaton-bib-py/issues/6

IETF BibXML service currently generates XML using its own serializer.


[MDv3] Remove extraNew in favor of bools in extra

Otherwise, the journal would not be able to
easily serve up MDs with fleshed-out
ExtraMetadata objects.
Use NewExtraMetadataV3 everywhere.
Use TLF{Writer,Reader}KeyBundleV3 instead of
pointers in more places.
Lots of misc. cleanup.

Appveyor CI is flake, will have fix for the hang out.


Issues with uploading sketch for ha_mqtt_sensor_dht22.ino

This is my first go at using MQTT in HA and trying to upload the sketch to the ESP8266 so please be understanding of my ignorance.
Made the minor adjustments to your ino file for my wifi network and MQTT details. Plus the topic name and the sketch verifies perfectly.
When I hook up the NodeMCU board to the pc to upload I get a rather long error file (below). Any advice would be greatly appreciated.
Arduino: 1.8.4 (Windows 10), Board: "Generic ESP8266 Module, 80 MHz, 40MHz, DIO, 115200, 1M (144K SPIFFS), ck, Disabled, None"
Archiving built core (caching) in: C:\Users\ADMINI~1.ASU\AppData\Local\Temp\arduino_cache_134493\core\core_esp8266_esp8266_generic_CpuFrequency_80,FlashFreq_40,FlashMode_dio,UploadSpeed_115200,FlashSize_1M144,ResetMethod_ck,Debug_Disabled,DebugLevel_None_____7246470ddf479558ce5125d1daf92f35.a
Sketch uses 240089 bytes (27%) of program storage space. Maximum is 876528 bytes.
Global variables use 33296 bytes (40%) of dynamic memory, leaving 48624 bytes for local variables. Maximum is 81920 bytes.
C:\Users\Administrator.ASU-HP7900-DS\AppData\Local\Arduino15\packages\esp8266\tools\esptool\0.4.9/esptool.exe -vv -cd ck -cb 115200 -cp COM3 -ca 0x00000 -cf C:\Users\ADMINI~1.ASU\AppData\Local\Temp\arduino_build_821035/sketch_sep07a.ino.bin
esptool v0.4.9 - (c) 2014 Ch. Klippel<EMAIL_ADDRESS>setting board to ck
setting baudrate from 115200 to 115200
setting port from COM1 to COM3
setting address from 0x00000000 to 0x00000000
espcomm_upload_file
espcomm_upload_mem
setting serial port timeouts to 1000 ms
opening bootloader
resetting board
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
resetting board
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
resetting board
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
trying to connect
flush start
setting serial port timeouts to 1 ms
setting serial port timeouts to 1000 ms
flush complete
espcomm_send_command: sending command header
espcomm_send_command: sending command payload
espcomm_send_command: receiving 2013 bytes of data
read 0, requested 1
error: failed reading byte
warning: espcomm_send_command: cant receive slip payload data
warning: espcomm_sync failed
error: espcomm_open failed
error: espcomm_upload_mem failed
error: espcomm_upload_mem failed
This report would have more information with
"Show verbose output during compilation"
option enabled in File -> Preferences.

Hi @AssuredDust640,
I assume that the compilation went fine, so the problem might come from the upload process.

Do you have something connected to D8? If yes, momentary unplug it.
Do you have chosen the right board? For the NodeMCU board, you have to select the "NodeMCU v1.0" option (Tools>Board).

Sam

Sam, thanks for your reply.
Yes, the compilation went fine, it’s just at the upload stage I get the errors. I’ve played with changing the board between the NodeMCU and generic ESP8266 and both caused issues, but as it is a NodeMCU, I’ll stick with that going forward.
When you mention that is something connected to the DB, should I have the DHT sensor NOT wired up when I upload? Should the DB just be by itself for this process and then re-wire the sensor into place?
Thanks again for your assistance.
Jonathan
From: mertenats<EMAIL_ADDRESS>Sent: Friday, 8 September 2017 03:53
To: mertenats/Open-Home-Automation<EMAIL_ADDRESS>Cc: AssuredDust640<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mertenats/Open-Home-Automation] Issues with uploading sketch for ha_mqtt_sensor_dht22.ino (#17)
Hi @AssuredDust640https://github.com/assureddust640,
I assume that the compilation went fine, so the problem might come from the upload process.

Do you have something connected to D8? If yes, momentary unplug it.
Do you have chosen the right board? For the NodeMCU board, you have to select the "NodeMCU v1.0" option (Tools>Board).

Sam
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mertenats/Open-Home-Automation/issues/17#issuecomment-327870950, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AV8TKRc3mmQQD7cikNPcwtM6O9IAjAFRks5sgCnJgaJpZM4PPp4F.

Hi,
I experienced issues when something was connected to D8 during the upload process (logic level probably set to HIGH).. You should maybe try to unplug/your NodeMCU to the computer, press the flash/reset button before uploading, etc. Do you have another NodeMCU? Have you ever suceeded in uploading something to the board?
Sam

Sadly this is the only NodeMCU I have and I’ve yet to upload anything to it but I’m just about to try unhooking everything, resetting it and trying again. Thanks for your assistance, it’s been very helpful.
Regards,
Jonathan
From: mertenats<EMAIL_ADDRESS>Sent: Friday, 8 September 2017 17:39
To: mertenats/Open-Home-Automation<EMAIL_ADDRESS>Cc: AssuredDust640<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mertenats/Open-Home-Automation] Issues with uploading sketch for ha_mqtt_sensor_dht22.ino (#17)
Hi,
I experienced issues when something was connected to D8 during the upload process (logic level probably set to HIGH).. You should maybe try to unplug/your NodeMCU to the computer, press the flash/reset button before uploading, etc. Do you have another NodeMCU? Have you ever suceeded in uploading something to the board?
Sam
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mertenats/Open-Home-Automation/issues/17#issuecomment-328026232, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AV8TKR6YLxed38-C57GFoEzMoGi_mwACks5sgO8qgaJpZM4PPp4F.

All good, thanks again.
Sketch uploaded and I can see it on my router. Just need to wait and see if the info comes through to HA now but that should be the least of my worries.
Received 2 x Sonoff wifi switches today so on to those next.
Can’t express my thanks enough.
Regards,
Jonathan
From: mertenats<EMAIL_ADDRESS>Sent: Friday, 8 September 2017 17:39
To: mertenats/Open-Home-Automation<EMAIL_ADDRESS>Cc: AssuredDust640<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mertenats/Open-Home-Automation] Issues with uploading sketch for ha_mqtt_sensor_dht22.ino (#17)
Hi,
I experienced issues when something was connected to D8 during the upload process (logic level probably set to HIGH).. You should maybe try to unplug/your NodeMCU to the computer, press the flash/reset button before uploading, etc. Do you have another NodeMCU? Have you ever suceeded in uploading something to the board?
Sam
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mertenats/Open-Home-Automation/issues/17#issuecomment-328026232, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AV8TKR6YLxed38-C57GFoEzMoGi_mwACks5sgO8qgaJpZM4PPp4F.


[NetAppFiles]Remove ServiceLevel from pool patch test

In the next upcoming version of the NetAppFiles API specs version 2020-06-01 the property ServiceLevel is removed from Pool Patch as that is no longer supported.
To be able to pass validation on the spec this PR is needed to remove the reference to the ServiceLevel in Patch operations from the SDK test code as that breaks the Automation SDK Pipeline in the rest-api-specs repo since that property no longer exists in updated SDK.

Hi can we move this along. This is blocking SDK Automation pipeline for upcoming API spec PR.

@allenjzhang  can we have this looked at please

Is there something blocking this from getting looked at?

Could you:

Add a link to the openapi spec review PR here

Check in the metadata TXT file generated when you run generate.ps1(details)

Update release notes in csproj

Thanks

This still targets previous API version 2020-02-01.  All this does is remove a reference to a property removed in the new one.  But the pipeline for the spec runs SDK automation and that fails if the test references the property.
If I understand correctly we need to have completed the API spec via review and have it merged before we can regenerate the .net SDK update the tests accordingly and submit in a another PR.


feat(breaking-change-workflow): add enforcement of breaking change doc

Description

Adds enforcement of breaking change documentation in the workflow
Combines release-preview-notes and enforce-breacking-changes-doc-check into one action called release-lint

This PR will address the issue here
Changes

feat(breaking-change-workflow): add enforcement of breaking change doc

🚀 PR created with fotingo

@gopkri24 I believe that this is now implemented in the lint-release-notes action.  If so, we can probably close this.

This is now part of the lint-release-notes action already, so we can probably close this PR


show_in_notebook not displaying the left "Prediction Probabilities" portion figure

Issues with show_in_notebook for regression

It seems that it does not work properly for wide range variables. I just checked it in the same notebook and it worked for smaller y. Predicted variable from the screenshot above is in the range 0-100_000. @marcotcr that could be solved with scientific notation maybe?

should be fixed by 5c7107e09e85c07a3417686673f2c81756c29280.

Thank you, now I can see the predicted variable. Yet it still overlaps with feature value:
. Can I move it around somehow?


Renamed Swagt_font to Swagat_font

Swagat Font was returning an error as it wasn't able to parse the Swagat folder due to spelling error.

Great thanks 😀👌

Great thanks 😀👌


I2C running at 1/10th the frequency setting

Hi,
I'm using this HAL with an STM32F401RE MCU and have run into an issue with I2C configuration. Tried a few different parameters for i2c1 speed (setting -> observed):

10 kHz -> 8.7 kHz
20 kHz -> 17.4 kHz
40 kHz -> 34.7 kHz
60 kHz -> 6.45 kHz
100 kHz -> 7.5 kHz
200 kHz ->11.5 kHz
400 kHz -> 23.6 khz
1600 kHz -> 94.7 kHz

My delay provider does work correctly, however. delay.delay_ms(100_u16); measures at 99.7 ms.
Any idea why SCL runs at 1/10th (or less) the speed of the setting?  Also, I'm running the MCU at 40 MHz, which could relate to the change in mapping after 40 kHz.

I've created a PR that seems to fix these issues, #34.


test-image: images/image-predefined-groups: stop specifying group in USER directive in Dockerfile

What type of PR is this?
/kind bug

What this PR does / why we need it:
This PR stops specifying a group in USER directive in images/image-predefined-groups/Dockerfile. Specifying group in User directive ignores group membership defined in the image
ref: https://docs.docker.com/engine/reference/builder/#user

When specifying a group for the user, the user will have
only the specified group membership. Any other configured group
memberships will be ignored.

This is not suitable for the test case of SupplementalGroups with pre-defined groups in the container image. In the test case, the default user should to belong to a supplementary group in the container image.
The image is used in #1005.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?

/assign @haircommander

/lgtm


mDNS reply is unreachable

mDNS reply to the peer seem to be advertised on all interfaces (including local host) and peer unreachable on some interfaces.
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Failed to reply to query: OS Error 2101 (0x00000835): Network is unreachable
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Failed to reply to query: OS Error 2101 (0x00000835): Network is unreachable
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Failed to reply to query: OS Error 2101 (0x00000835): Network is unreachable
CHIP:DIS: Replying to DNS-SD service listing request
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Failed to reply to query: OS Error 2101 (0x00000835): Network is unreachable
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Failed to reply to query: OS Error 2101 (0x00000835): Network is unreachable
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Failed to reply to query: OS Error 2101 (0x00000835): Network is unreachable
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Failed to reply to query: OS Error 2101 (0x00000835): Network is unreachable
CHIP:DIS: Directly sending mDns reply to peer on port 5353
CHIP:DIS: Failed to reply to query: OS Error 2101 (0x00000835): Network is unreachable

I'm seeing something similar with the python controller on Linux as well:
chip-device-ctrl > discover -all
[1624055419.310826][122497] CHIP:DIS: mDNS broadcast success
[1624055419.310918][122497] CHIP:DIS: Attempt to mDNS broadcast failed:  OS Error 2101 (0x00000835): Network is unreachable
[1624055419.311116][122497] CHIP:DIS: mDNS broadcast success
[1624055419.311298][122497] CHIP:DIS: mDNS broadcast success
[1624055419.311414][122497] CHIP:DIS: mDNS broadcast success
[1624055419.311465][122497] CHIP:DIS: Attempt to mDNS broadcast failed:  OS Error 2099 (0x00000833): Cannot assign requested address
Waiting for device responses...

Given this is an older issue, does this till reproduce on current builds? Please re-open if so!


⚠ Bofrost.de has degraded performance

In 7706094, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 939 ms

Resolved: Bofrost.de performance has improved in 7ca2ff2 after 6 minutes.


How do debug Bot code accessed via DirectLine API?

We are developing a bot that will primarily be accessed using the DirectLine API. As the bot and the client app are still in development, the development is made difficult by the fact that we cannot Debug both the client app and bot on one box since DirectLine API can't connect back to the local box. What's the story there? Any recommendations on best way to do this?

There are methods to debug Azure web apps using Visual Studio (e.g. attaching a debugger):
https://azure.microsoft.com/en-us/documentation/articles/web-sites-dotnet-troubleshoot-visual-studio/
@dandriscoll do you have more useful suggestions?

@srozga Have you looked into using ngrok? Ngrok is network tunneling software to localhost. You can register a debug version of your bot with the Bot Framework using an ngrok service endpoint, then run your bot and your client locally in debuggers. Messages will go from client to DirectLine and then routed through ngrok to your bot on localhost. www.ngrok.com

This page talks about debugging bots using ngrok and the emulator. Not exactly your scenario, but the ngrok part should still be relevant: https://docs.botframework.com/en-us/tools/bot-framework-emulator/

Closing due to inactivity, please open a new issue or reply to this one asking us to reopen if needed. Thanks!


Cannot import *.excalidraw file on macOS Safari

Works on Chrome though

Hi. If not private, can you upload the file here (you can rename the extension to .txt so GitHub will allow posting it in a comment. If too big, you can upload to https://wormhole.app/).
If private, can you open the console (Opt-Cmd-I) before you import to see what the error says?


Healey school overview page doesn't load

Dug into the logs to try to figure out what's dragging this page down.
Precomputed query is being loaded, doesn't seem to be a precompute issue.
Scout shows that we have a major N + 1 query problem with Event Notes:

Page loads now and doesn't, though it's hard to tell if it's from the N+1 query reduction in #992 or because of the heroku restart from the deploy.


Add Sourcetree extension

Description
View, search and open repositories in Sourcetree.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I have Sourcetree installed, what could go wrong? I guess it's the path of bin. First: would be nice to handle this and show failure toast if something goes wrong

I'll update the toast to the failure state if any errors occur.

Second: how can we improve that? I haven't change anything since I installed the Sourcetree, so it seems like the default folder is wrong in preferences 🤔

If you don't mind, can you share the full path of the bin, if any? I'll change the default bin path in the preference to that.
I installed Sourcetree using homebrew, so the bin file might have also been installed by homebrew.
I'll add a dialog similar to Jetbrains extension when the path is not available.

I installed Sourcetree using homebrew, so the bin file might have also been installed by homebrew.

I installed It from their website a long time ago 😄

If you don't mind, can you share the full path of the bin, if any? I'll change the default bin path in the preference to that.

Error: Command failed: /opt/homebrew/bin/stree /Users/myname/proj/Raycast


⚠ Bofrost.de has degraded performance

In 361af40, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1912 ms

Resolved: Bofrost.de performance has improved in 4744f6a after 1 hour, 30 minutes.


faced a building error

i faced error while building the hpx
can anyone help.
make[2]: *** [examples/resource_partitioner/CMakeFiles/async_customization.dir/build.make:63: examples/resource_partitioner/CMakeFiles/async_customization.dir/async_customization.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:63035: examples/resource_partitioner/CMakeFiles/async_customization.dir/all] Error 2
make: *** [Makefile:141: all] Error 2

HPX Version:latest
Platform (compiler, OS):ubuntu18.04

You have not included the actual error message, but I suspect that you have built the current master branch and there is a known problem with this test. You can ignore it for the time being and try using make -k ... where -k tells make to ignore errors and continue building the rest.


Add recurring command

basic functionality out.
issues:

letting the user add a task with deadline before now (as i will add the day/week/time accordingly upon loading to load the next recurring task)

Will add test cases for recur commands later

Will add some javadocs for public methods later

Codecov Report

Merging #100 (2e1aedf) into master (09f85d2) will decrease coverage by 6.69%.
The diff coverage is 39.21%.

@@             Coverage Diff              @@
##             master     #100      +/-   ##
============================================
- Coverage     74.77%   68.08%   -6.70%
- Complexity      589      599      +10
============================================
  Files            87       91       +4
  Lines          1804     2068     +264
  Branches        222      291      +69
============================================
+ Hits           1349     1408      +59
- Misses          368      518     +150
- Partials         87      142      +55

Impacted Files
Coverage Δ
Complexity Δ

...n/java/seedu/module/logic/commands/AddCommand.java
68.75% <0.00%> (-16.97%)
8.00 <0.00> (ø)

...java/seedu/module/logic/commands/RecurCommand.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...va/seedu/module/logic/parser/ModuleBookParser.java
84.00% <0.00%> (-3.50%)
17.00 <0.00> (ø)

.../seedu/module/logic/parser/RecurCommandParser.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

src/main/java/seedu/module/model/Model.java
100.00% <ø> (ø)
1.00 <0.00> (ø)

src/main/java/seedu/module/model/ModelManager.java
96.15% <0.00%> (-3.85%)
24.00 <0.00> (ø)

...n/java/seedu/module/model/task/UniqueTaskList.java
88.88% <0.00%> (-4.14%)
21.00 <0.00> (ø)

src/main/java/seedu/module/ui/TaskCard.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

.../java/seedu/module/logic/commands/DoneCommand.java
76.47% <28.57%> (-13.19%)
11.00 <0.00> (ø)

src/main/java/seedu/module/model/task/Task.java
58.16% <32.55%> (-22.92%)
42.00 <4.00> (+6.00)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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

Merged upstream repo into current repo for recurring-command:
Failing testcases (that have been commented out):

JsonSerializableModuleBookTest: toModelType_typicalTasksFile_success()

StorageManagerTest: moduleBookReadSave()

JsonModuleBookStorageTest: readAndSaveModuleBook_allInOrder_success()


refactor(List)!: start_corner is now direction

The previous name start_corner did not communicate clearly what the function was doing. The method was renamed to direction and a new enum ListDirection was added.
Fixes #671
Same as #672, I feel like this should be for 0.26.

Rebased on #672

Right I completely forgot about the depreciation warning. I added back start_corner and its tests with the warning.
I'm wondering if this is still a breaking change then?

I'm wondering if this is still a breaking change then?

If we keep the old behavior as deprecated, I don't think so.


feat: improve logging and error handling

Main motivation for this PR is that the nginx log file was very unreadable and I want to write log records to custom file and not only rely on whatever stderr writes too. Intentionally not using PSR or thirdparty log libraries.
Improve error handling and logging. Make log messages prettier. Have only three log levels. Add full stacktrace to php errors. Differentiate log record severity. Make it easier to expand log handlers e.g. writing to log file.
I bumped into some interesting stuff while testing this. Each message passed to error_log() is truncated by php to around 1000 bytes. Also, nginx truncates the total data to around 2000 bytes. This typically happens when there's multiple calls to error_log() in a single execution. A solution to these limits is to simply log to a file instead of using error_log().

Will this change anything about the current access and error lpgs that nginx creates in the docker container?

Yes the data written to error log is changed. An exception will look like this:
[2022-09-03 00:45:59] rssbridge.WARNING Exception in GettrBridge {
    "message": "Exception HttpException: 400 Bad Request at lib/contents.php line 132",
    "code": 400,
    "stacktrace": [
        "#0 index.php:7",
        "#1 lib/RssBridge.php:15",
        "#2 lib/RssBridge.php:58",
        "#3 actions/DisplayAction.php:134",
        "#4 bridges/GettrBridge.php:35",
        "#5 lib/contents.php:132"
    ]
}

A php error will look like this:
[2022-09-03 00:38:51] rssbridge.WARNING Undefined variable: QQQ at bridges/GettrBridge.php line 30 {
    "message": "Exception ErrorException: Undefined variable: QQQ at bridges/GettrBridge.php line 30",
    "code": 0,
    "stacktrace": [
        "#0 index.php:7",
        "#1 lib/RssBridge.php:15",
        "#2 lib/RssBridge.php:58",
        "#3 actions/DisplayAction.php:134",
        "#4 bridges/GettrBridge.php:30"
    ]
}


Support initialising earthranger client with an exisitng token

Alongside username/password auth, allow the client to be created with a jwt

Per discussion with @cisaacstern today, and further to https://github.com/wildlife-dynamics/ecoscope-server/issues/98, a task on this is to look at altering the logic so the client receives just an access token, and if it's expired the init should fail


Creates a<EMAIL_ADDRESS>target for transpiling

When setting the node option to true it'll use babel-preset-env to
target a specific version of Node which means less transpiling and it'll
use the native ES-next features in v8.

We should generally be moving to babel-preset-env (see #10). Do you think you could bring us all the way there?

Yeah sure. I'll open a new PR to get us onto babel-preset-env and then amend this one to take advantage of it.


Managing existing droplets?

Hi,
I have an existing Digital Ocean droplet and I'd like to manage it using Vagrant instead of having to create a new one.
How do I use the plugin to hook into existing droplets on Digital Ocean?
Cheers,
-- Attila

@liquid, did this work for you?

Yup, It did.
Sent from my iPhone

On 12 Jul 2015, at 13:58, XedinUnknown<EMAIL_ADDRESS>wrote:
@liquid, did this work for you?
—
Reply to this email directly or view it on GitHub.


server panicked: internal error: entered unreachable code: disjunction in conjunction

Code that triggered this error (I was mucking around in hyper's code):
#[cfg(any(
    feature = "server",
    all(any(feature = "http1", feature = "http2"), feature = "client"),
))]
pub(crate) use std::marker::Unpin;

Logs:
thread '<unnamed>' panicked at 'internal error: entered unreachable code: disjunction in conjunction', crates/cfg/src/dnf.rs:157:36
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
Panic context:
>
version: 996300f4a 2021-08-23 dev
request: textDocument/codeAction CodeActionParams {
    text_document: TextDocumentIdentifier {
        uri: Url {
            scheme: "file",
            cannot_be_a_base: false,
            username: "",
            password: None,
            host: None,
            port: None,
            path: "/home/jplatte/code/hyper/src/error.rs",
            query: None,
            fragment: None,
        },
    },
    range: Range {
        start: Position {
            line: 278,
            character: 53,
        },
        end: Position {
            line: 278,
            character: 53,
        },
    },
    context: CodeActionContext {
        diagnostics: [],
        only: None,
    },
    work_done_progress_params: WorkDoneProgressParams {
        work_done_token: None,
    },
    partial_result_params: PartialResultParams {
        partial_result_token: None,
    },
}

thread '<unnamed>' panicked at 'internal error: entered unreachable code: disjunction in conjunction', crates/cfg/src/dnf.rs:157:36
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Hmm, that snippet alone is not enough to reproduce the bug for me. And adding that cfg expression to the tests also results in a reasonable DNF expression: #![cfg(any(feature = "server", all(feature = "http1", feature = "client"), all(feature = "http2", feature = "client")))].

Oh, hm. Well I didn't save a snapshot of the repo when RA produced this error message unfortunately, and can't find a simple repro either. Should I close this?

No, it's still useful to track, in case someone else runs into this

Let's through arbitrary at it? This looks very amendable to property-based testing, and I want it to be easy to prop-test stuff in rust-analyzer.
We did, and removed property-based tests for text edits, but that's because there wasn't a nice composable way to do prop-testing.
With the arbitrary crate, I think we can now reasonably arbitrary all the things.

To clarify, I don't think we should outright libFuzzer it, but running arbitratry with just some random seeds might be enough.

Good call, it instantly found a reproducer: #![cfg(all(all(any())))]


Add new contracts to include Poseidon for computing class hash

Update contracts related to contract data (such as class hash) for the newer ones in cairo-lang
Fix errors that were consequences of the new cairo-rs version
Change metadata related to the new class hash computation
Adds newer contracts for calculating class_hash and compiled_class_hash

The PR also renames StructContractClass into DeprecatedCompiledClass to have parity with cairo-lang's naming scheme. From now on, DeprecatedCompiledClass will refer to classes that are not Cairo 1 programs. For DeclareV2 there should be an analogue way of compiling it with the newer contracts.

Codecov Report

:exclamation: No coverage uploaded for pull request base (0.11@cfecb1b). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 4ddb3f0 differs from pull request most recent head e0d5baf. Consider uploading reports for the commit e0d5baf to get more accurate results

@@           Coverage Diff           @@
##             0.11     #242   +/-   ##
=======================================
  Coverage        ?   73.75%
=======================================
  Files           ?       40
  Lines           ?     2690
  Branches        ?        0
=======================================
  Hits            ?     1984
  Misses          ?      706
  Partials        ?        0

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


[Addition]: RAKwireless cluster is trying to fool us

Hotspot b58 Addresses
11PUtLTc79ys4nhDJRXNsJGrEEwfJxrArNDW2WhWE7SY4D2Du2d
112sgeoS5HwwoZPEVs9brkTznngJyi2PuTCLAgiGr7J4uH7QTrBC
118xJDjzrTMsaCo5LPNMxUjvoyT75x17Ztv27DtZLYCaPMUQBSd
112qELZRB3yRPhNzxdGzfkG9E9mQQxA4cMZ4u81ZL5FyuGBgqH7A
Discord Handle
No response
Reason(s)

They are just witnessing themselves

Duplicate of #3833


should not depend on klusterlet addon config in MCE

Signed-off-by: zhujian<EMAIL_ADDRESS>KlusterletAddonConfig belongs to ACM, we should not depend on that in MCE.
And for creating the klusterletaddonconfig after the clusterclaim managed cluster was created, we moved the login to https://github.com/stolostron/klusterlet-addon-controller/pull/105

/assign @ldpliu @skeeey @zhiweiyin318

/approve

/lgtm


SAK-41672 SecurityService covers

@jonespm  @adrianfish ?

@axxter99 I think #6841 also fixes this

Loving the lombok changes @axxter99 !