Water level unavailable after update to 0.8.10 for Midea Cube dehumidifier

The water sensor is seen as 'unavailable' after upgrading to the latest release 0.8.10 from the previous release 0.8.2.   All the other sensors and functions work fine.
I deleted the integration and re-added it, now the water level sensor is no longer enumerated.  It looks like the integration no longer exposes the water level to HA.  I have automation's triggered on different water the levels so they maybe proactively empty before becoming full and turning the dehumidifier off.
This was working in the previous version.  What changed?
Configuration
HA 2022.7.3
homeassistant-midea-air-appliances-lan 0.8.10
Midea Cube dehumidifier
Update
Also note; this update breaks midea-humidifier-card.
Downgrading to 0.8.2 resolves both the missing water sensors and restores the disabled sensors that prevent midea-humidifier-card working.

Thanks @Turton - indeed there is regression bug in detecting capabilities. I'll publish new version today.

Confirmed 0.8.11 resolves both temperature and midea-humidifier-card issues.  Thank you!


MSVC complex values

Implementation of approach as discussed in https://github.com/GraphBLAS/python-suitesparse-graphblas/issues/64
The core is in a get_extension() method in build.py. It sets up _graphblas as a regular extension rather than a cffi extension. It emits _graphblas.c with emit_c_code() then sets the needed compiler configs. The salient part, converting double _Complex to _Dcomplex and _Fcomplex on MSVC, is done here too. I'm assuming "Windows" == "MSVC".
I moved build.py up a directory to work around an import problem.
suitesparse_graphblas/__init__.py imports a bunch of extensions, such as utils, which don't exist at setup time.
I don't know how to import build.py while skipping __init__.py, so I just moved it up instead.
You'll have to ignore the linter error. It wants a change that will break the imports. The # noqa: E402 would satisfy most linters that this is an ok exception, but apparently not this linter.
Fixes https://github.com/GraphBLAS/python-suitesparse-graphblas/issues/64

As per conversation, this also no longer uses the arm header. Please verify that that's acceptable.

Figured out how to make the linters happy.

This looks pretty good.  I have a couple minor suggestions:

Can we rename build.py to something else such as buildcffi.py or whatever makes sense? I worry about conflicting with build.
Let's add build.py (or what it gets renamed to) to MANIFEST.in so the sdist will be able to compile correctly.

I'm sorry I wasn't able to help you earlier with the linting. Of the current linting rules, I think the need to do
import foo  # noqa: E402 isort:skip

is probably the most annoying. Should we add a note somewhere that shows how to ignore linting rules? I would use E402 as the example since it also shows isort. Where would you have found this note? In linting.yml GitHub Action?
I'm going to try testing this (and the wheels in #66) in python-graphblas CI 🤞

Success!!!!!!! 🎉
See: https://github.com/python-graphblas/python-graphblas/pull/386
Our tests exercise complex dtypes pretty well.
✨👯♀🎊👯✨
@alugowski you're my hero

As per conversation, this also no longer uses the arm header. Please verify that that's acceptable.

Confirmed 👍

This looks pretty good. I have a couple minor suggestions:
* Can we rename `build.py` to something else such as `buildcffi.py` or whatever makes sense? I worry about conflicting with [`build`](https://pypi.org/project/build/).

* Let's add `build.py` (or what it gets renamed to) to `MANIFEST.in` so the sdist will be able to compile correctly.

Done. I used "build_graphblas_cffi" because my Java years drilled over-expressiveness into me :P

I'm sorry I wasn't able to help you earlier with the linting. Of the current linting rules, I think the need to do
import foo  # noqa: E402 isort:skip

is probably the most annoying. Should we add a note somewhere that shows how to ignore linting rules? I would use E402 as the example since it also shows isort. Where would you have found this note? In linting.yml GitHub Action?

That particular line is a special case, I don't think it needs a writeup. It's special because it's doing something weird that in general is not needed. Also it's breaking two rules (imports not in alphabetical order, and statement before import), each discovered by a different linter. I'm also  open to suggestions about how else to do it, this is all I could think of.
In general it's probably a bad idea to have people defeat the linter instead of following the rules. I did my fair share of cursing at this particular linter's demands, but it basically tells you what it wants so it's not an issue. The cursing is only because its standards are different than PyCharm's and forces a few round trips until all the issues are fixed.

Thanks for renaming.
@alugowski do you think this is ready to merge (once CI passes)? LGTM.

Thanks for renaming.
@alugowski do you think this is ready to merge (once CI passes)? LGTM.

Yes, I think so.

This is in! 🚀


Explore assistant not loading for users with User License

Hi
We enabled explore assistant for our production environment. I was loading only for Admin users. We were getting a loading screen for users with an explore (user) license and didn't proceed to the prompt.
On debugging we found that one of the resources was not responding for Explores - /api/internal/core/4.0/sql_queries
To solve this We needed to enable - use_sql_runner permission for user license then it worked for Explores.  Since by default user role does not have - use_sql_runner is this necessary? As it might have cost implications and we don't want to give SQL runner to user license type.
image.png

@arun-subhash this currently is expected behavior. At a minimum a user needs to have a Standard User license to be able to access Explore Assistant. The default "User" role would roll up into the Standard User license but doesn't include all the permissions that a Standard User is entitled too. Our legal terms describe this in a bit more detail. But using SQL Runner is something that is permitted for the Standard User license.


Support Scala 2.11

Problem description
Flink connector works under Scala 2.11 after verified, needs to support Scala 2.11.
Problem location
Add Scala 2.11 build and push in .travis.yml.
Suggestions for an improvement

Closed via #393


Migrating from justinfrench/formtastic to formtastic/formtastic

Hi everyone, just noting some changes I've made lately to ensure that this repo and gem isn't reliant on me as a maintainer.
Ownership and contributors:

I converted the formtastic user (which I had previously registered) into an org
The current owner of the formtastic org is me, @justinfrench, but I'm actively looking for new maintainers and a new owner
I transferred the repo from justinfrench/formtastic to formtastic/formtastic
I invited all previous maintainers (plus @deivid-rodriguez) to the formtastic org (I also removed them as outside contributors)

Linked services:

In #1311 I updated a bunch of URLs and links, mostly in the README, which meant I needed to…
Set up a new formtastic/formtastic RDoc https://rdoc.info/projects/formtastic/formtastic
Set up a new formtastic/formtastic Travis CI https://travis-ci.org/formtastic/formtastic
Set up a new formtastic/formtastic CodeClimate https://codeclimate.com/github/formtastic/formtastic

New 4.0 release:

I published a 4.0.0.rc1 gem today
I cleaned up a bunch of stale issues, branches, milestones, etc — so we're starting pretty fresh
@deivid-rodriguez has been cleaning up master, will help release the final 4.0.0 and address any issues that come up afterwards

Gem publishing:

@mikz and I have permission to publish new gems
I'll be looking to add @deivid-rodriguez as a publisher next

If you notice anything else that is dependent on me, would like to join as a maintainer (and possibly even take over ownership of the project), please open an issue!

Hello - what is the status of this Gem? Is it still being maintained by @justinfrench or @deivid-rodriguez (or anyone else)? There are some outstanding PRs and I am curious if they will be merged or if we need to implement a monkey patch on our end. Thank you.

Got it, thanks very much for the info @mikz !


⚠ Bofrost.be has degraded performance

In 927f699, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 868 ms

Resolved: Bofrost.be performance has improved in 55740a3.


Loading ASE calculator from model trained with basic config template fails

Hello,
I am trying to create an ASE calculator from a trained model like this:
from apax.md.ase_calc import ASECalculator
calc = ASECalculator(Path("test_model-1/"))

Inside test_model-1 is the config file used to train the model, as well as the outputs (in models/apax):
n_epochs: 2000

data:
  directory: models
  experiment: apax
  train_data_path: "./training.traj"
  val_data_path: "validation.traj"

  n_train: 7616
  n_valid: 853
  batch_size: 32
  valid_batch_size: 100

metrics:
  - name: energy
    reductions: [mae]
  - name: forces
    reductions: [mae, mse]

loss:
  - name: energy
  - name: forces
    weight: 4.0

This config file is the one generated by apax template train with the training and validation data paths filled in and number of training and validation structures updated.
When I try to make the calculator (to use for testing the model), I get this error:
----> 1 ASECalculator(Path("test_model-1/"))

File ~/venv3.10-apax/lib/python3.10/site-packages/apax/md/ase_calc.py:117, in ASECalculator.__init__(self, model_dir, dr_threshold, transformations, padding_factor, **kwargs)
    114 self.transformations = transformations
    115 self.n_models = 1 if isinstance(model_dir, (Path, str)) else len(model_dir)
--> 117 self.model_config, self.params = restore_parameters(model_dir)
    118 self.padding_factor = padding_factor
    120 if self.model_config.model.calc_stress:

File ~/venv3.10-apax/lib/python3.10/site-packages/apax/train/checkpoints.py:102, in restore_parameters(model_dir)
     98 """Restores one or more model configs and parameters.
     99 Parameters are stacked for ensembling.
    100 """
    101 if isinstance(model_dir, Path) or isinstance(model_dir, str):
--> 102     config, params = restore_single_parameters(model_dir)
    104 elif isinstance(model_dir, list):
    105     param_list = []

File ~/venv3.10-apax/lib/python3.10/site-packages/apax/train/checkpoints.py:94, in restore_single_parameters(model_dir)
     92 model_config = parse_config(Path(model_dir) / "config.yaml")
     93 ckpt_dir = model_config.data.model_version_path()
---> 94 return model_config, load_params(ckpt_dir)

File ~/venv3.10-apax/lib/python3.10/site-packages/apax/train/checkpoints.py:84, in load_params(model_version_path, best)
     82 except FileNotFoundError:
     83     print(f"No checkpoint found at {model_version_path}")
---> 84 params = jax.tree_map(jnp.asarray, raw_restored["model"]["params"])
     86 return params

TypeError: 'NoneType' object is not subscriptable

I have tried adding in the default model section from the apax template train --full to my config, but I still got the same error.
I also tried setting the directory argument in the ASECalculator to test_model-1/models/apax with the same results.
Am I loading the calculator incorrectly?
Thanks,
-Ali

Hi Ali,
Sorry for the late response, we were all attending a conference these last few days.
The Path you need to specify is the one to ".../directory/experiment".
We don't assume a particular input file name and directory may contain multiple experiments (different model trainings).
Hence, pointing the ASE calculator to the directory with the configuration file is not sufficient.
We could however provide a better error message.
Best
Moritz

No worries, Moritz, I hope the conference went well.
I have tried that directory and get the same error, unfortunately...
From my config file, I should be pointing the ASECalculator class to models/apax, (from the directory and experiments entries in the yaml file in the data heading), but doing so results in the same error mentioned above. This directory has a config.yaml file generated by apax with more details than my initial input file.
Any idea what could be going wrong?

Currently, model loading assumes that you are in the same directory the training was started from.
so if you are in mydir and run a training from there with
directory: models
experiment: exp

then you should be able to to load a model into ASE via
ASECalculator("models/experiment") from a python file located in mydir.
I am not sure if you are attempting to load the model from a different relative path, but I suspect that might be the issue here.
It will be fixed by #212 .
The setup described above is what I use for working with apax standalone.
We usually hand over path handling to our workflow manager, IPSuite, so it's certainly true that training at and loading from arbitrary locations was not well tested until now.

Ah, yes, I was trying to load it from one directory above the training directory. Other codes I use (and want to compare against) don't interface with IPSuite so I wasn't using it, hence this issue.
Thank you for your help!

No problem! thanks for raising this issue.


fix(jaeger): update and use the jaeger bundled CRD

It seems otherwise there's a race condition in which CRD gets installed depending on the helm version.
We don't actually need to use our own CRD for jaeger anymore, so let's instead if the bundled one from jaeger-operator.
Updates jaeger-operator.
Note that as of now cert-manager is a required dependency for jaeger.

Description

Motivation and Context

Regression

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added unit tests to cover my changes.

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge


Inconsistent SoX speed behaviour compared to WavAugment

🐛 Bug
With the following effect chain:
def speed(sampling_rate: int) -> List[List[str]]:
    return [
        # Random speed perturbation factor between 0.9x and 1.1x the original speed
        ['speed', RandomValue(0.9, 1.1)],
        ['rate', sampling_rate],  # Resample back to the original sampling rate (speed changes it)
    ]

being applied to a tensor of 16000 samples (with a sampling rate of 16000) I am receiving an output that has a different number of samples than the input. This behavior is inconsistent with what WavAugment does - the shapes are always identical.
This is tested in Lhotse here for WavAugment and here for torchaudio in PR https://github.com/lhotse-speech/lhotse/pull/124.
To Reproduce
Run Lhotse's test suite with torchaudio 0.7 installed. See e.g. the error in Lhotse's CI here: https://github.com/lhotse-speech/lhotse/pull/124/checks?check_run_id=1386141728
Expected behavior
Equal input and output tensor shapes.
Environment
This is my local MacOS env; the other one is GitHub Actions CI in Lhotse.
PyTorch version: 1.7.0
Is debug build: True
CUDA used to build PyTorch: None
ROCM used to build PyTorch: N/A
OS: Mac OSX 10.15.7 (x86_64)
GCC version: Could not collect
Clang version: 12.0.0 (clang-1<IP_ADDRESS>)
CMake version: version 3.18.4
Python version: 3.7 (64-bit runtime)
Is CUDA available: False
CUDA runtime version: No CUDA
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Versions of relevant libraries:
[pip3] numpy==1.18.1
[pip3] torch==1.7.0
[pip3] torchaudio==0.7.0a0+ac17b64
[conda] blas                      1.0                         mkl
[conda] mkl                       2019.4                      233
[conda] mkl-service               2.3.0            py37hfbe908c_0
[conda] mkl_fft                   1.2.0            py37hc64f4ea_0
[conda] mkl_random                1.1.1            py37h959d312_0
[conda] numpy                     1.18.1           py37h7241aed_0
[conda] numpy-base                1.18.1           py37h3304bdc_1
[conda] pytorch                   1.7.0                   py3.7_0    pytorch
[conda] torchaudio                0.5.1                    pypi_0    pypi
(ignore that last one version, it's some conda+pip quirk... the torchaudio version is 0.7.0 😅)
Additional context

Hi @pzelasko
How off are they each other?
My initial thought is that this is not a bug but the way WavAugment handles speed is different.
Second thought is that based on the libsox being used, it generates a slightly different results. I noticed this when I was writing tests for the sox effects. (sox command installed via OS utility vs the one cmake builds as a part of torchaudio build process)
apply_effects_tensor and apply_effects_file are tested and confirmed that they return an identical result to sox command for speed as the following.
https://github.com/pytorch/audio/blob/0e1d814b76725f50b76d938b474b616a15009b45/test/torchaudio_unittest/assets/sox_effect_test_args.json#L73-L74
https://github.com/pytorch/audio/blob/0e1d814b76725f50b76d938b474b616a15009b45/test/torchaudio_unittest/sox_effect/sox_effect_test.py#L54-L79
https://github.com/pytorch/audio/blob/0e1d814b76725f50b76d938b474b616a15009b45/test/torchaudio_unittest/sox_effect/sox_effect_test.py#L103-L128

Yeah, bug might not be the best description - sorry.
I don't think it's actually the speed behaviour, seems like the difference is in how the rate effect is being applied. Consider the following:
import torch
from torchaudio import sox_effects as se
audio = torch.sin(2 * 3.14 * torch.linspace(0, 1, 16000)).unsqueeze(0)

se.apply_effects_tensor(audio, 16000, [['speed', '0.9']])
Out[10]: (tensor([[ 0.0000,  0.0004,  0.0008,  ..., -0.0040, -0.0036, -0.0032]]), 14400)

se.apply_effects_tensor(audio, 16000, [['speed', '0.9']])[0].shape
Out[11]: torch.Size([1, 16000])

se.apply_effects_tensor(audio, 16000, [['speed', '0.9'], ['rate', '16000']])[0].shape
Out[12]: torch.Size([1, 17778])

se.apply_effects_tensor(audio, 16000, [['speed', '0.9'], ['rate', '-q', '16000']])[0].shape
Out[13]: torch.Size([1, 17778])

That last effect chain in WavAugment would have returned a tensor of shape [1, 16000]. Now that I think of it, it's reasonable that the number of samples would have changed... I'll need to check what are they doing (maybe truncating/padding the output?), if you happen to know please share :)

OK I think I understand it now. When the output length description is specified in their effect chain invocation, they are truncating or zero-padding the signal (I see that here: https://github.com/facebookresearch/WavAugment/blob/master/augment/speech_augment.h#L120). Sorry for the false alarm! 😉

Glad you figured. :)


[devtools] update cache guide

What's changed, or what was fixed?

update the (service worker) cache guide

Fixes: N/A
Target Live Date: 2019-03-27

[x] This has been reviewed and approved by @kaycebasques
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 8 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/tools/chrome-devtools/manage-data/local-storage.md#L20 - Common typo found: 'Service Worker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/tools/chrome-devtools/manage-data/local-storage.md#L71 - Common typo found: 'Service Worker' Should it be 'service worker'? Capitalize Service if it's the first word in the sentence.
src/content/en/tools/chrome-devtools/storage/cache.md#L22 - Line must be at most 100 characters
src/content/en/tools/chrome-devtools/storage/cache.md#L81 - Line must be at most 100 characters
src/content/en/tools/chrome-devtools/storage/cache.md#L91 - Line must be at most 100 characters
src/content/en/tools/chrome-devtools/storage/cache.md#L96 - Line must be at most 100 characters
src/content/en/tools/chrome-devtools/storage/cache.md#L109 - Line must be at most 100 characters
src/content/en/tools/chrome-devtools/storage/cache.md#L119 - Line must be at most 100 characters

:tada: This has been pushed live to https://developers.google.com/web/


Data Saturday Los Angeles

event name - Data Saturday Los Angeles
event date - 06/12/2021
event location, if physical - Virtual
description of your event, including multiple languages, if applicable. - SQL Saturday in Los Angeles
SQLSaturday is a free training event for professionals who use the Microsoft/AWS/GCP data platforms. These community events offer content across data management, cloud and hybrid architecture, analytics, business intelligence, AI, and more.
Sessionize link for CFP - https://t.co/0SFdedqp3u
Eventbrite (or other registration ) link - https://t.co/lYgwzAvRSe
contact information - sqlsatla at datadriventechnologies.org

Can the sessionize and eventbrite be amended to say Data Saturday?

Can the sessionize and eventbrite be amended to say Data Saturday?

@way0utwest , done. Replaced SQL Saturday with Data/SQL Saturday.

@way0utwest , done. Replaced SQL Saturday with Data/SQL Saturday.

please allow the contact organizer name to be custom. for our event, i would like visitors to see SQL Saturday in LA instead of my name.

please allow the contact organizer name to be custom. for our event, i would like visitors to see SQL Saturday in LA instead of my name.

PR merged to fix the organizer name: https://datasaturdays.com/events/datasaturday0007.html

PR merged to fix the organizer name: https://datasaturdays.com/events/datasaturday0007.html

Please add the following:

our event logo - https://datadriventechnologies.files.wordpress.com/2021/02/data.sql_.saturday.jpg?resize=219%2C219
link to volunteer sign up form - https://datadriventechnologies.org/2021/02/22/2021-data-sql-saturday-call-to-volunteers/
link to sponsorship plan - https://datadriventechnologies.org/2021/02/22/2021-data-sql-saturday-sponsorship-plan/

PR submitted #136

Event page has been created


Android app backup dirs feature and other fixes

Please let me know what do you think, feel free to send me feedback, this is my first time sending PR to perkeep project, thanks.

I tried to cleanup the PR and have separate commits for each of the changes so that it is much easier to read and cherry-pick if needed. Also I uploaded android perkeep.apk file in my fork repo for those who wants to try out new changes.
Thanks.
@mpl

@kushtrimjunuzi
Hi, we do not use github for code reviews.
Please see https://perkeep.org/code#contributing

@mpl
I forgot about that, thanks


Fix repository link (hyphen instead of underscore)

Codecov Report

Merging #244 (0ad9825) into dev (1ca0934) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #244   +/-   ##
=======================================
  Coverage   59.62%   59.62%
=======================================
  Files          64       64
  Lines        4679     4679
=======================================
  Hits         2790     2790
  Misses       1889     1889

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 1ca0934...0ad9825. Read the comment docs.


Knife round ready hint at the start of every Round

"Knife round will start when both teams are !ready" appears at the start of every round, more or less at 7 seconds in the freezetime and mid round.

Is it possible that you started two bots at the same time? also, we need your config.json (remove your rcon and mask the ip obviously) and any info you can provide to understand what is going on

It starts the map normally everthing works fine just the message appears every time.

Would you have some minutes to have a closer look at this problem? I'd really appreciate your time! If so, add me on steam: http://steamcommunity.com/id/parisienne
Or, you could try a fix a user found in a similar problem:
Go to the cfg folder and add to the very end of warmup.cfg the following two lines:
log off
log on

and try again.

Just added you on Steam.

Resolved issue: Bot was running on the same LAN as the CSGO Server and serverType was not set to local. The server was once added by the config.json and once "on-the-fly" due to an old logaddress entry on the server. If you run into a similar problem (bot giving out everything double), check your config.json and try to do a "rcon logaddress_delall" and restart the bot.


Update test cases with affected by the account lock handler modifications

This PR updates test cases with affected by the account lock handler modifications done by https://github.com/wso2-extensions/identity-event-handler-account-lock/pull/95.
Note : This should only be merged after https://github.com/wso2-extensions/identity-event-handler-account-lock/pull/95 is merged.

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/1027260405

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/1027260405
Status: success


Deprecate Matomo

https://groups.google.com/g/islandora/c/MtHOxjew-tA
Stop installing Matomo by default in Playbook, while leaving enough infrastructure in place that someone could turn it on, if desired.

This might be as simple as flipping a couple bits of config:

webserver_install_matomo and
webserver_app_configure_matomo

?


PHMSA gas extract step for transmission part k

Overview
Closes #3248.
What problem does this address?
we didn't have the mapping files for Part K!
What did you change?
 - [x] add all the mapping files for part k 2010-present.
- [x] i also removed the `data_label` bc that was not being used anywhere and was probably resulting in a million warnings about a missing column 😬
- [x] add old times - 2009 (no old times part k that was easy)
- [x] add in a notebook extractor into the devtools notebook
- [x] make the extractor warnings quiet/appropriate for this old year split up into new format thing.

Testing
How did you make sure this worked? How can a reviewer verify this?
# To-do list
- [x] Make sure full ETL runs & `make pytest-integration-full` passes locally
- [ ] Update the [release notes](../docs/release_notes.rst): reference the PR and related issues.
- [x] Review the PR yourself and call out any questions or issues you have

Also completely forgot when I first reviewed this, but I actually think data_label is what we use to store "final" vs. "early-release" status and I'm not convinced we should be dropping it.


Android guidelines restructuring

Fixes issue #482 from the Android repository.
Re-organized the Android guidelines following the @bsiegel's effort for iOS. There are a number of TODO items, but I think it's important to get the ball rolling with a PR to get valuable feedback. These guidelines took heavy inspiration from the iOS guidelines where appropriate, so much of the feedback given by @annelo-msft will be reflected here as well.

nit:
On the top level nav bar, there are still pages that will need to be consolidated:

This is what it should look like:

nit:
On the top level nav bar, there are still pages that will need to be consolidated:

This is what it should look like:

I thought I had already made this change, thanks for pointing it out :)

@annelo-msft I did remove the additional links from the general_sidebar.yml, so I'm not sure why you can see them when generating the HTML. Is there another place I should remove them from?

I was looking at the published site as part of my status check - if you've generated the HTML and they're gone, then we're all good.  Apologies for the false alarm.


Checkout PR doesnt pull changes if the branch already exists

Checkout a PR
go back to master
push commits to that PR from another machine
in the first machine, checkout the PR again

Expectation: New commits are pulled and branch is up to date
Actual: New commits are not pulled and no notification for the same

Ok, I want to upvote the issue I myself logged.
I moved on to review another PR that I had checked out a few days ago, kept wondering why I dont have the changes that I can see in the diff view.
Its because the branch is not synced!


contenteditable on the table makes everything editable

When you set contenteditable="true" on the table it makes everything editable.  At the very least the table headers should be forced to not be contenteditable by default.

I also tested this and setting contenteditable="true" on the table, or an element without an ng-model just changes the DOM but not the backing data model.


"Stress test" for #6

This is based against master instead of the changes-from-apps-infra so that CI runs, see https://github.com/Automattic/AnalyticsSDK-iOS/pull/4#issuecomment-1508227634

The tests have run. This PR doesn't need to stay open.


Use of custom TrackByFunction in MatTree does not preserve expansion state when nodes change

What is the expected behavior?
Use of trackBy should also ensure that the expansion state does not get lost when nodes are updated.
What is the current behavior?
Use of trackBy does not ensure that the expansion state does not get lost when nodes are updated. This is presumably since TreeControl uses an expansionModel which is a SelectionModel where the SelectionModel has no awareness of the TrackByFunction.
What are the steps to reproduce?
Relying upon the TreeControl expansionModel:
https://stackblitz.com/edit/angular-5wpr9o-kjnmsn?file=app%2Ftree-flat-overview-example.ts
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
all
Is there anything else we should know?
Trying to store expansion state in the nodes themselves as a workaround doesn't work (unless you transfer the state):
https://stackblitz.com/edit/angular-5wpr9o-ahptjs?file=app%2Ftree-flat-overview-example.ts
Having a custom expansionModel that is aware of the trackBy state works to preserve state, since the TreeControl's expansion-related methods apparently aren't actually used internally by the tree component:
https://stackblitz.com/edit/angular-5wpr9o-pbmk1g?file=app%2Ftree-flat-overview-example.ts

I spent some time looking into it, but at least with the way things are set up at the moment, there's no nice way to solve it. We can pass the trackBy function to the tree control and have it use the function when tracking which nodes are expanded, but the problem is that the tree control doesn't know the indexes of the items so it can't call the trackBy function correctly. We do have access to the array of all the data nodes from which we can figure out the index, but we'd have to do it very frequently (multiple times per change detection) which would defeat the purpose of a trackBy a bit.

Any updates on this one? I'd love to add a trackBy on our app's tree and improve performance. It's a big tree and trackBy could give some big performance gains.
Using the expansion model mostly worked but fails intermittently after calling this.treeControl.collapseAll() or this.treeControl.expandAll(). (Despite looping through treeControl.dataNodes and adding all node IDs to the expansionModel).

Oh man, I so wish trackBy worked with the expansion state! Without this I'm stuck with paying the performance cost of re-expanding the tree on every change to the dataSource. For even moderately sized trees this can be expensive:
    this.treeControl.expansionModel.select(
      ...this.treeControl.dataNodes.filter((node) => node.expanded)
    );

Has there been any update to this? I see the OP posted a workaround; I will reference that and see if it works me.

Unfortunately this problem means I can't use Mat-Tree. Fortunately, Angular makes it very easy to make your own tree out of a ul and a recursive component.

TLDR; when using trackBy, the expansionKey function must also be set, which is not documented very good.
I tried use the mat-tree together and had very strange effects data changes were fired and trackBy was used (like all nodes collapsed anf were not expandable anymore). After some time banging my head around this, I remembered, that I stumbled into a "Redux Example" some time ago and thought, that should not work, if it is not working for me... the example is nicely hidden in the documentation of the cdkTree and there you can find:
trackBy = (index: number, node: TransformedData) => this.expansionKey(node);
  expansionKey = (node: TransformedData) => node.raw.id;

<cdk-tree
      #tree
      [dataSource]="roots"
      [childrenAccessor]="getChildren"
      [trackBy]="trackBy"
      [expansionKey]="expansionKey">
    ...

Problem solved.
Please add a hint to this in the trackby documentation of the mat-tree here:
https://material.angular.io/components/tree/overview#trackby and for the cdktree here:
https://material.angular.io/cdk/tree/overview#trackby


TX1 Running the Live Camera Detection Demo FPS

Hello!!!
I successful Building from Source on TX1(Before is ubuntu 14.04 + GTX980)
I try the Running the Live Camera Detection Demo
I have some question with the FPS.
./detectnet-camera multiped-500  FPS is about 6.7 FPS
./detectnet-camera ped-100 FPS is about 6.7 FPS
./detectnet-camera facenet-120 is about 12.3 FPS
Does the FPS is correct ? It looks a little slow...
How can I accelerate it ?
Another question is , I put my own model create by https://github.com/NVIDIA/DIGITS/tree/digits-4.0/examples/object-detection
And change "multiped-500 file" to it . It successful to detect car.
But the FPS still 6~7.
How can I accelerate it ?

I use the https://github.com/dusty-nv/jetson-scripts/blob/master/jetson_max_l4t.sh to max performance with tx1.
And it looks a little accelerate 6.7FPS up to 7.0FPS

@FaceOP ,
I tried to get source from https://github.com/ross-abaco/jetson-inference to support USB camera, the FPS I got about 8 when running this command ./detectnet-camera ped-100. I think it should be about 20fps as this graph : https://devblogs.nvidia.com/parallelforall/wp-content/uploads/2016/09/Figure_3-1.png
Anyone have the better result ?

@forever3000 @FaceOP Hi,
If you look carefully at the vertical axis of the image in the previous link, you will actually see that it is "frames per second per watt". The graph is about energy efficiency and not pure performance.

Hi.
I succeeded in installation of OpenFrameworks to TX1.
http://takesan.hatenablog.com/entry/2016/12/19/214455
And detectnet(by jetson-inference) multiped-500 FPS is about 7.8~8 FPS and Camera FPS 60FPS by OF.
http://takesan.hatenablog.com/entry/2017/02/03/201108
I'm sorry, it is all Japanese.
By the way. 'jetson_clocks.sh'  is not effective by this 'jetson-inference'.  Why?

Hi Dustin .
What do you mean?

2017/03/24 7:36、Dustin Franklin<EMAIL_ADDRESS>You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/dusty-nv/jetson-inference/issues/20#event-1013360942, or mute the thread https://github.com/notifications/unsubscribe-auth/AYZGiyZN1ORxE-UZ4u53h7hzcIWwD4OSks5rovPagaJpZM4KktON.


Fix TaskDialogDemo compilation

Removes WindowsDesktop part of the SDK. Reference
Fixes compilation error - net5.0-windows is the correct TargetFramework. Won't compile otherwise.

Might want to check the other samples too.😉

Wow. Fastest approval/merge ever.😁

I happen to open the email :)

Do you think you could update the VB sample as well? https://github.com/dotnet/samples/blob/master/windowsforms/TaskDialogDemo/vb/TaskDialogDemo.vbproj


Automate windows feature tests run with badge creation in appveyor

Because of large number of tests and free/OS license, 60 mins to run with no parallels allowed, need to create separate ci branches for each feature to overcome this.
See comments at end of badge creation

beginnings of this in ci branch.

Note: https://blog.travis-ci.com/2018-10-11-windows-early-release. Maybe wait for this to become stable, and move all to travis


Encapsulate field as read-only or write-only

From a field create a property with getter or setter and optionally change all the references to that field to use the property. There is a refactoring already in Visual Studio for a full encapsulation.
class Foo
{
    private int i;
    void Bar()
    {
        i = 1;
    }
    void Baz()
    {
        var j = i;
    }
}

If changed to read-only:
class Foo
{
    private int i;
    public int I => i;
    void Bar()
    {
        i = 1;
    }
    void Baz()
    {
        var j = I;
    }
}

If changed to write-only:
class Foo
{
    private int i;
    public int I
    {
        i = value;
    }
    void Bar()
    {
        I = 1;
    }
    void Baz()
    {
        var j = i;
    }
}

If the field is not private and has references outside the class, change those as well.
This is essentially a refactoring, so we are classifying it as a Hidden diagnostic.
Diagnostic id: CC0058
Category: Refactoring

I am moving it to ready.


Updated README.md

📝 Updated Contributors Information in README.md

Added an explanation to the new Contributors.md file.
Provides guidance on steps to take after making the first contribution (adding name and profile link).

Commit https://github.com/maddox05/link-minifier/pull/14/commits/8402def143e4b8f0f73ef5fb871dead1869eda59 should not be included in this PR. I'm unable to undo or delete it. I simply replaced the tag focus with active in that commit and changed some text:

Changed 'Login' to 'Log in' and now is grammatically correct.
Changed 'Logout' to 'Log out' and now is grammatically correct
From 'Mini Link:' to 'Mini Link:' to not use "Title Case" and be consistent throughout the website.


Report builder for Saltcorn - PDF

Hello.
Is there any report builder that we can use with Saltorn, in order to print PDF documents from a Form View?
I used to use Joomla and Fabrik, and I used a library that works with Jaspersoft Studio, and I wonder if we can use somthing like that.
I know there's a module named page 2 pdf, but I don't know exactly how to use it and I wonder if this module is powerfull enough to draw, for example, an Invoice like PDF.
Thanks in advance.

the page-to-pdf action is a bit hard to use. (try putting id in the list of state variables to transfer, that often fixes things). I will improve soon

Thanks Tom.
OK, I've been working a bit with page-to-pdf but, anyway, it looks it works like just printing a page and not really a report.
Do you think is possible to use something like I mentioned before?


Add a "migrate database" API.

Addresses #389
(generated code updates excluded - will run those after #392 is merged)

The new MigrateDatabase method returns a completed LRO object - allowing future implementations to return incompleted LROs for long-running migrations.


e2e: don't remove member used to connect to etcd cluster

Fixes #7204

lgtm defer to @xiang90

LGTM


Settings to match new library from svelte-create

Test of new settings as mentioned in #134

Good catch, merged!


Incorrect latest stamp saved when download is interrupted and resumed

Describe the bug
When --latest-stamps is used and the download is interrupted and then resumed, the saved post timestamp is not that of the newest post in the profile, but of the newest post downloaded in the latest run.
To Reproduce
Steps to reproduce the behavior:

instaloader --latest-stamps some-profile
Interrupt the download with Control-C.
Re-run instaloader and let it finish downloading the profile.
Check latest stamps file, it'll have the timestamp of the newest post of the second run.

Expected behavior
The saved timestamp should be that of the newest post of the whole profile.
Instaloader version
4.9.3, but should happen since 4.9.2.
Additional context
This happens because of the new check for pinned posts. When the NodeIterator is created it is passed a closure that stores the newest date seen and uses this date to determine if a post is the newest (because with pinned posts the first post is not necessarily the newest). But this date is not saved between runs.

I think the solution is to discard that logic of trying to get the absolute newest post, and instead go back to something similar like it was before: store the date of the newest non pinned post. And once this first (non-pinned) post is found, never change the iterator's first_node (whose timestamp is stored between runs) again.
One disadvantage is that if a user had a post on the 10th and that post is saved as the latest timestamp, and on the 11th adds a new post and pins it, the latest timestamp will still be that of the 10th, and subsequent runs will re-download the post of the 11th until a newer non-pinned post is added.
But in order to avoid that, it would be necessary to save the state used by the is_first callback. But since it's just a function, this function could do anything and it's state could be anything.

Never mind the comment above, it's possible to keep the current behavior without having to save the state or suffering from the problem described.


Add option to open text editor setting

that default working with notepad.
I thought that could be change user wanted text editor.
please check that 😋

You mean users should be able to choose which text editor is used for editing .ini files?


Getting the debug console for a specific debug session

I have use case where I would like to send some text to the debug console for a specific debug session when an event occurs. There is vscode.debug.activeDebugConsole which can then call DebugConsole.appendLine() but that reference always points to the console for the active debug session. When I have two debug sessions, sending text to the console for a session isn't as straight forward.
Would it be possible to add a way to find the debug console for a specific debug session? Maybe as a property of a DebugSession?

@ivanhernandez13 this is a fair API feature request which currently is not possible.
The reason why no other extension author asked for this is because they mostly send the debug console output for a specific session from the debug adapter using the DAP protocol output event
Though I see your use case that an extension would like to append to a specific debug console without the debug adapter. Thus I think this request makes sense.
First assigning to @weinand so he considers adding this to the vscode debug api.
I like the approach to have the debugConsole as a field on the DebugSession. Alternative is to add events onDidNewDebugConsole which is not so nice imho since the extension would have to track them.

I am also interested in this feature.
We are currently working on migrating Go extension's debug adapter to Go debugger (dlv)'s native DAP implementation. We start this new DAP implementation from vscode.DebugAdapterDescriptorFactory  and let VSCode connect to it using vscode.DebugAdapterServer. The stdout/stderr of this new DAP+debugger, written in Go, includes both
a) stdout/stderr from the debugged process (the Go debugger places the inferior in the foreground), and
b) logs/panic/crash of this DAP+debugger.
We tried to use vscode.debug.activeDebugConsole and found that it

is not suitable when multiple debug sessions are active.
is not ready for use until the initial protocol handshake is complete (empirically, i saw all messages appended before configuration done are lost) which isn't ideal especially if we need to inspect crashes or initial protocol exchanges.
does not allow us to present stdout and stderr messages differently.

We can try to wrap the our adapter again to capture its stdout/stderr and convert them to DAP output event as a workaround. This new API can help us avoid such an additional layer to adapt our adapter :-)

I think this request makes sense. From the implementation side we could easily surface this.
As for the API there are open questions: should this be part of the DebugSession object? Or should there be a method to fetch the debugConsole for the passed session.

Both work for our use case.
It's more important for use whether this DebugSession's debugConsole will be usable and ready to accept messages by the time our DebugAdapterDescriptorFactory's createDebugAdapterDescriptor is called. (So any logs and failure messages while executing createDebugAdapterDescriptor can be surfaced in the debug console.)

I'm interested in this too. My new debug adapter runs dart test in a mode that outputs test results as JSON, which the debug adapter blindly forwards to the client as custom events. This allows the client to parse them and populate the VS Code test runner). However, this results in no output in the Debug Console (whereas the old DA also wrote output events that would print the status of tests as they completed).
While I can build code into the new debug adapter to parse and understand the test runner JSON format, it feels a bit awkward - the DA is already sending the structured JSON messages on so that the editor can decide how best to present them (this DA will be used by more than just VS Code). It would be nicer if I could write to the debug console from the editor where I'm already parsing the JSON, and then the DA can remain oblivious to the content of the messages it's proxying.
(although, if the DAP was extended with test events that mapped onto the new test runner API, that would make me reconsider, and push this handling into the DA :-))

@weinand this is the issue discussed earlier - my comment above describes what I was trying to do.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!

If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.

I guess "crucial" is somewhat subjective, but this is rather inconvenient for extensions that integrate with the test runner and want to handle providing test results to VS Code as well as providing output to the server in the same place. Right now (assuming the test runner is being run in an API mode), the DA would need to handle results to write some text representation to the Debug Console, and the extension will need to handle them to add them to the VS Code testing tree. It would be far nicer if both of these could be done from the same place (there's a lot of overlap - at least for Dart - I don't know about others).

@weinand Why is this out of scope? vscode-go's support for debugging tests (which I wrote) doesn't work properly because this limitation. The test controller can't know if a test being debugged passed or failed without access to the test process' stdout. Am I supposed to write a new DAP implementation just for debugging tests?


"ValueError" issues concerning tensor shapes

Hi,
Thanks for the implementation !
I created the dataset with my own data accordingly to the Tensorflow TFrecord api.
However, I am experiencing several issues with the training workflow.

First, after the tfrecord is being loaded, the following instructions fails :

train_dataset = train_dataset.map(lambda x, y: (transform_images(x, 416),
                                                transform_targets(
                                                    y, yolo_anchors,
                                                    yolo_anchor_masks, 416)))

with a traceback ValueError: Shape must be rank 3 but is rank 4 for 'Tile' (op: 'Tile') with input shapes: [?,1,2], [4].
This issue occurs line 56 in the transform_targets function from dataset.py. I managed to address it by changing the line from
box_wh = tf.tile(tf.expand_dims(box_wh, -2),  (1, 1, tf.shape(anchors)[0], 1))

to
box_wh = tf.tile(tf.expand_dims(box_wh, -2),  (1, tf.shape(anchors)[0], 1))

(removing the second axis in the second arg). However, since no one reported this issue I am not very confident in this solution.

The second issue occurs in the transform_targets_for_output function, line 21 of dataset.py :
    ValueError: Index out of range using input dim 0; input has only 0 dims for 'strided_slice_3' (op: 'StridedSlice') with input shapes: [], [1], [1], [1] and with computed input tensors: input[3] = <1>.

Finally, I encounter an issue with the loss in the training process when ignoring the transformation mappings in the dataset. Just led this experiment to check if there was any problem before actually training. I get that message :
ValueError: When passing a list as loss, it should have one entry per model outputs. The model has 4 outputs, but you passed loss=[<yolov3_tf2.yolo_models.YoloLoss object at 0x7f27ac0fbfd0>, <yolov3_tf2.yolo_models.YoloLoss object at 0x7f27c00817b8>, <yolov3_tf2.yolo_models.YoloLoss object at 0x7f26fb53a390>]
The model outputs the bbox, scores, classes and valid detections, but there are 3 losses (1 per anchor mask).

Hi, you have to call this line train_dataset = train_dataset.batch(batch_size) to get rid off the above error as it adds new dimension to your input.

Hi, you have to call this line train_dataset = train_dataset.batch(batch_size) to get rid off the above error as it adds new dimension to your input.


is there any way to kill all jobs running in jobby at a point of time?

is there any way to kill all jobs running in jobby at a point of time?

@hellogerard


Updating .gitignore + Pull Request Template Files.

💡 Motivation, context, additional details, changes proposed

This PR is going to update the .gitignore file and also the Pull Request Template.
✏ Checklist

[X] I've checked my code.
[X] I've tested the change manually.
[ ] I've invoked unit tests: ./gradlew testStagingDebug testStagingDebugUnitTest
[ ] I've invoked instrumentation tests:

🔮 Next steps

📸 Screenshots, GIFs

Maybe GitHub has some issues. This build action run on PR and I also ran it manually now, but still, its status is not updating in PR. Maybe I'll need to deselect Required status checks to pass before merging

Maybe GitHub has some issues. This build action run on PR and I also ran it manually now, but still, its status is not updating in PR. Maybe I'll need to deselect Required status checks to pass before merging

Maybe GitHub has some issues. This build action run on PR and I also ran it manually now, but still, its status is not updating in PR. Maybe I'll need to deselect Required status checks to pass before merging

Will look into this yml also.


feat: Publish helm chart to GitHub Helm Registry

Is your feature request related to a problem? Please describe.
Most projects publish their helm charts to a registry to make it easy to do package management. This would probably help to spread the project as it can be indexed at sites like https://artifacthub.io/ and would make it easier to install everywhere.
Describe the solution you'd like
Would like to see the helm chart hosted on GitHub as a Helm registry: https://medium.com/@blackhorseya/step-by-step-guide-to-hosting-your-own-helm-chart-registry-on-github-pages-c37809a1d93f
This would enable me to:
tee open-webui-overrides.yaml <<EOT
#override values.yaml
EOT

helm repo add open-webui https://open-webui.github.io/helm-charts
helm repo update open-webui
helm upgrade --install --create-namespace open-webui open-webui/open-webui --version=v1.0.0 -n=open-webui -f open-webui-overrides.yaml

PR welcome!

I want this functionality as well, so I spent some time trying to put a PR together for this in my fork. I found a handful of issues using the approach defined in the Medium article taliesins sent:

You'd need to delete the line in this repo's .gitignore that blocks *.tgz files, because that breaks the Helm Deploy job. I didn't see npm pack actually in use in CI, so it seemed like that might be okay to do.
The Helm release job adds its own tag/ release, which I could see being problematic given that Open WebUI already has a specific tagging and release strategy.
The maintainers would still need to set up the gh-pages branch to support this workflow if we can get the workflow working correctly.

I'm hoping to come back to working on this sometime soon, but can't make any promises. Here's the workflow file I had put together for this: https://github.com/0xThresh/open-webui/blob/main/.github/workflows/helm-release.yml
It seems like it could be less work to split the Helm charts into a separate repo under the open-webui organization, and set the job up against that instead. Feel free to tag me with any thoughts or ideas.

Create a repo here: https://github.com/open-webui/helm-charts

Thanks @tjbck! It looks like I can't fork a Github repo until there's a file committed, could you create a file in the repo for me?
Alternatively, if you'd consider adding me with write permissions to the repo, I could also try to do the Github Pages configuration that's needed without needing to work with a maintainer on that.
Thank you either way!

I got the PR opened on the new repo here: https://github.com/open-webui/helm-charts/pull/2
Once that's merged and we can pull Helm charts from there, we should be good to close this out!

Hey @taliesins, you should be able to pull from https://helm.openwebui.com now!
helm repo add open-webui https://helm.openwebui.com


<EMAIL_ADDRESS>with JFrog CLI 2.55.0?

Hello!
In my Team we are using JFrogNPM@1 to pack and publish NPM libraries.
Task is failing in version 1.9.4 but works fine before that.
In version 1.9.4, JFrog CLI is 2.54.0.
I have this output when JFROG_CLI_LOG_LEVEL is set to DEBUG.
##[section]Starting: Pack and publish (staging)
==============================================================================
Task         : JFrog npm
Description  : Install, pack and publish npm packages from and to Artifactory while allowing to collect build-info. The collected build-info can be later published to Artifactory by the "JFrog Publish Build Info" task.
Version      : 1.9.4
Author       : JFrog
Help         : [More Information](https://github.com/jfrog/jfrog-azure-devops-extension#JFrog-npm-Task)
==============================================================================
Found tool in cache: jf 2.54.0 x64
Running jfrog-cli from /agent/_work/_tool/jf/2.54.0/x64/jf
JFrog CLI version: 2.54.0
Executing JFrog CLI Command:
/agent/_work/_tool/jf/2.54.0/x64/jf c add "repository - NPM publish_20240416.8_npmpack and publish_deployer_1713290303610" --artifactory-url="https://company.jfrog.io/company" --interactive=false --access-token-stdin
17:58:23 [Debug] JFrog CLI version: 2.54.0
17:58:23 [Debug] OS/Arch: linux/amd64
17:58:23 [Debug] Using access-token provided via Stdin
17:58:23 [Debug] Locking config file to run config AddOrEdit command.
17:58:23 [Debug] Creating lock in: /home/AzDevOps/.jfrog/locks/config
17:58:23 [Debug] Releasing lock: /home/AzDevOps/.jfrog/locks/config/jfrog-cli.conf.lck.47268.1713290303620839471
17:58:23 [Debug] Config AddOrEdit command completed successfully. config file is released.
Executing JFrog CLI Command:
/agent/_work/_tool/jf/2.54.0/x64/jf c use "repository - NPM publish_20240416.8_npmpack and publish_deployer_1713290303610"
17:58:23 [Debug] JFrog CLI version: 2.54.0
17:58:23 [Debug] OS/Arch: linux/amd64
17:58:23 [Debug] Locking config file to run config Use command.
17:58:23 [Debug] Creating lock in: /home/AzDevOps/.jfrog/locks/config
17:58:23 [Info] Using server ID 'repository - NPM publish_20240416.8_npmpack and publish_deployer_1713290303610' (https://company.jfrog.io/company/)
17:58:23 [Debug] Releasing lock: /home/AzDevOps/.jfrog/locks/config/jfrog-cli.conf.lck.47274.1713290303630499127
17:58:23 [Debug] Config Use command completed successfully. config file is released.
Executing JFrog CLI Command:
/agent/_work/_tool/jf/2.54.0/x64/jf npmc --server-id-deploy="repository - NPM publish_20240416.8_npmpack and publish_deployer_1713290303610" --repo-deploy="registry-npm-staging"
17:58:23 [Debug] JFrog CLI version: 2.54.0
17:58:23 [Debug] OS/Arch: linux/amd64
17:58:23 [Info] npm build config successfully created.
Executing JFrog CLI Command:
/agent/_work/_tool/jf/2.54.0/x64/jf npm p
17:58:23 [Debug] JFrog CLI version: 2.54.0
17:58:23 [Debug] OS/Arch: linux/amd64
17:58:23 [Debug] Using npm executable: /agent/_work/_tool/node/20.12.1/x64/bin/npm
17:58:23 [Debug] Running 'npm --version' command.
17:58:23 [Debug] npm '--version' standard output is:
10.5.1
17:58:23 [Debug] Preparing to read the config file /agent/_work/3/repository/.jfrog/projects/npm.yaml
17:58:23 [Debug] Found deployer in the config file /agent/_work/3/repository/.jfrog/projects/npm.yaml
17:58:23 [Info] Running npm Publish
17:58:23 [Debug] Working directory set to: /agent/_work/3/repository
17:58:23 [Debug] Reading Package Json.
17:58:23 [Debug] Usage Report: Sending info...
17:58:23 [Debug] Sending HTTP GET request to: https://company.jfrog.io/company/api/repositories/registry-npm-staging
17:58:23 [Debug] Sending HTTP GET request to: https://company.jfrog.io/company/api/system/version
17:58:23 [Debug] Setting Package Info.
17:58:23 [Debug] Creating npm package.
17:58:23 [Debug] Artifactory response: 200
17:58:23 [Debug] JFrog Artifactory version is: 7.82.2
17:58:23 [Debug] Sending HTTP POST request to: https://company.jfrog.io/company/api/system/usage
npm notice
npm notice 📦<EMAIL_ADDRESS>npm notice === Tarball Contents ===
npm notice 46B   LICENSE
npm notice 2.0kB README.md
npm notice 0B    dist/index.d.ts
npm notice 24B   dist/index.js
npm notice 2.4kB package.json
npm notice === Tarball Details ===
npm notice name:          @product/package-name
npm notice version:       1.1.17-alpha2
npm notice filename:      product-package-name-1.1.17-alpha2.tgz
npm notice package size:  2.1 kB
npm notice unpacked size: 4.6 kB
npm notice shasum:        XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX
npm notice integrity:     sha512-G/YKOovKb4IfW[...]muFXURiNrKV/g==
npm notice total files:   5
npm notice
17:58:24 [Debug] Deploying npm package.
17:58:24 [Debug] Extracting info from npm package: [/agent/_work/3/repository/><EMAIL_ADDRESS>prepare /agent/_work/3/repository/> bash scripts/prepare.sh /agent/_work/3/repository /agent/_work/3/repository/Do not run husky in pipeline /agent/_work/3/repository/product-package-name-1.1.17-alpha2.tgz]
{
  "status": "failure",
  "totals": {
    "success": 0,
    "failure": 0
  }
}
17:58:24 [Error] open /agent/_work/3/repository/><EMAIL_ADDRESS>prepare: no such file or directory
remove /agent/_work/3/repository/><EMAIL_ADDRESS>prepare: no such file or directory
##[error]Error: Command failed: /agent/_work/_tool/jf/2.54.0/x64/jf npm p
Executing JFrog CLI Command:
/agent/_work/_tool/jf/2.54.0/x64/jf c remove "repository - NPM publish_20240416.8_npmpack and publish_deployer_1713290303610" --quiet
17:58:24 [Debug] JFrog CLI version: 2.54.0
17:58:24 [Debug] OS/Arch: linux/amd64
17:58:24 [Debug] Locking config file to run config Delete command.
17:58:24 [Debug] Creating lock in: /home/AzDevOps/.jfrog/locks/config
17:58:24 [Debug] Releasing lock: /home/AzDevOps/.jfrog/locks/config/jfrog-cli.conf.lck.47319.1713290304429550716
17:58:24 [Debug] Config Delete command completed successfully. config file is released.
##[section]Finishing: Pack and publish (staging)

Notes:

Data have been rendered anonymous (company, product, repository, package name)
We use a NPM script prepare to init Husky on local machines but not in pipeline

I didn't post it as a "bug" because I saw a fix about tarball in JFrog CLI 2.55.0 that may fix this problem.
Our temporary solution is to change JFrogNPM@1 to<EMAIL_ADDRESS>My question, when<EMAIL_ADDRESS>with JFrog CLI 2.55.0 would be available?

Hi @VictorRos ,
Sorry for the inconvenience caused by this issue.
The regression is in 2.54.0, and the issue should be completely fixed in 2.56.0.
We plan to release a new extension with the latest CLI in the upcoming days. Note that you can install a custom JFrog CLI version in the meantime using the Tools Installer Task.

Hi @VictorRos ,
Extension version 2.9.5 was just released and sets the default JFrog CLI version to 2.56.1.
Sorry for the delay in releasing it.
We'd appreciate your feedback for it.

Hi @RobiNino
I confirmed it's working again.
We removed the task JFrogToolsInstaller@1 we added temporarily to let JFrogNPM@1 installed version 1.9.5 and its JFrogCLI in version 2.56.1.
I will close the issue.
Thanks for the help for the temporary solution.


more PCells

Add Pcells from papers or other PDKs to the generic PDK

[ ] Mzi modulator
[ ] photodetector
[ ] coherent tx
[ ] coherent rx
[ ] ring modulator
[ ] capacitor
[ ] resistor
[ ] skywater130
[ ] global foundries 180

what layerstack should these be built out of? the generic one here https://gdsfactory.github.io/gdsfactory/notebooks/08_pdk.html#layers ?

Yes, we can use the generic pdk layer stack and then we can re target any fab

Hi Joaquin, I'm currently working with Mehdi in the sky130 project. Is there any documentation of how to generate p cells? Thank you very much.

https://gdsfactory.github.io/gdsfactory/notebooks/03_cells_autoname_and_cache.html


[Feature] Panoptic segmentation metric

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Add 3D panoptic segmentation metric for semanticKITTI and nuScenes.
Modification
Add 3D panoptic segmentation metric. NOTE: To verify the metric, this PR also includes the corresponding utils, but there are not merged into the repo yet.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

Please fix the lint

Please fix the lint

I've fixed them. Please check.

The code LGTM. Please pull the latest code which has fixed the lint about isort. @xizaoqu

@sunjiahao1999 Please review this PR.

Fix lint error.

last round, the overall LGTM

last round, the overall LGTM

I've fixed the problems.


(425) Use working days for task due days

This updates our TaskTransformer to use addWorkingdays instead of the blunt instrument plusDays.
It also (and potentially more controversially) changes the WorkingDayCountService to use a static file. We’re hitting the GOV.UK bank holidays API a lot to make these requests (even though they’re cached in Redis). In a seperate PR, we’ll add a Github action (similar to https://github.com/ministryofjustice/hmpps-approved-premises-ui/pull/1050) to check the file for changes, and open a PR to replace the file if it has.

I'm not sure I understand the value of using a static JSON file for the list of bank holidays as it introduces a continuous manual task for developers to ensure it's up to date (even if it is just approving and merging an automatically generated PR). I don't think we need to worry about accidentally DDoSing the Bank Holidays API, as it's publicly accessible to anyone and (I believe) considered to be the canonical source of truth for bank holidays, so I imagine it's fairly robust. If we're often getting cache misses when looking up this information then it's possibly more beneficial to investigate why that's the case instead.
I think it would be worth getting a second opinion from @tahb on this approach.

@danlivings-dxw Yeah, I'm inclined to agree 👍 I'll remove the static stuff, and we can carry on calling (and caching) the Bank Holidays API

 Repeat the working days count down on other dashboards - CRU & workflow manager dashboards

I agree with @danlivings-dxw's points, I don't think we should remove GOV.UK Bank holidays.
I think we should strive to delegate the responsibility of bank holidays to a service that will manage this long term. If there were no devs on the team to merge pull requests for new bank holidays then the data will stagnate. A small detail but would silently affect turnaround periods and data reported on the back of it for all cas3 bookings.
If we are making too many failed calls to the API in spite of our cache I also suggest that a better approach to this problem could be to review how we cache this bank holiday data. If we can reduce the request rate we'll hopefully reduce the failure rate.


Model save issues with custom optimizer - DPKerasSGDOptimizer

Problem
A keras sequential Model was compiled using DPKerasSGD optimizer and saved. But when the saved model is loaded it throws the below error
"Unknown optimizer: 'DPOptimizerClass'. Please ensure you are using a keras.utils.custom_object_scope and that this object is included in the scope. See https://www.tensorflow.org/guide/keras/save_and_serialize#registering_the_custom_object for details"
Expected Behaviour
Since the model was already compiled using the optimizer it should run perfectly fine when fitted with the data.
Whereas the same works when the optimizer is replaced. Please provide a solution on how to save the model with this custom optimizer.
Standalone code to reproduce the issue
#building the model
model = tf.keras.Sequential([
tf.keras.layers.Dense(128, activation='relu'),
tf.keras.layers.Dense(256, activation='relu'),
tf.keras.layers.Dense(256, activation='relu'),
tf.keras.layers.Dense(1, activation='sigmoid')])
#compiling the model
model.compile(
loss=tf.keras.losses.CategoricalCrossentropy(from_logits=True,reduction=tf.losses.Reduction.NONE),
optimizer=tensorflow_privacy.DPKerasSGDOptimizer(l2_norm_clip=1,noise_multiplier=2,learning_rate=0.01),
metrics=[
tf.keras.metrics.BinaryAccuracy(name='accuracy'),
tf.keras.metrics.Precision(name='precision'),
tf.keras.metrics.Recall(name='recall')
])
#saving the model
model.save('my_model')
#loading the model
keras.models.load_model('mymodel')
#throws the error

Hi, how did you resolve this? I have a similar issue where my optimizer is DPKerasSGDOptimizer and I am training like this:
checkpoint = ModelCheckpoint(
    "best_model.h5", monitor="val_loss", mode="min", verbose=1, save_best_only=True
)

history = model.fit(
    train_ds,
    validation_data=val_ds,
    epochs=EPOCHS,
    callbacks=[early_stopping, checkpoint],
)

best_model = load_model("best_model.h5")

I get the same error "Unknown optimizer: 'DPOptimizerClass'. Please ensure you are using a keras.utils.custom_object_scope and that this object is included in the scope. See https://www.tensorflow.org/guide/keras/save_and_serialize#registering_the_custom_object for details" when loading best_model.h5


Document how the integrator deals with the system dynamics in jaxsim

While dealing with https://github.com/ami-iit/jaxcon/issues/1  I needed to extract the system acceleration and the contact forces from jaxsim.
I had a few issues understanding how the step method worked and extracting those quantities.
In this issue, I want to document what I find out and try to identify how to improve the readability of the code.

The system integration is performed as follows:

We call the step method inside model: https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/api/model.py#L2135
The step method calls an integrator.step method https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/api/model.py#L2239-L2260
The integrator step is a method of the integrator abstract class where the integrator class itself is called in called https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/integrators/common.py#L80-L114
The __call__  method is an abstract  method,  let's see how it is implemented in runge kutta to have an example, it calls the _compute_next_state function
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/integrators/common.py#L228-L240
We have than a call to f which represent the system dynamic which is called in a scan which calls a lax conditioned ( to condition the call to initial condition )
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/integrators/common.py#L358
f icloses an object of system dynamics  on specific args
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/integrators/common.py#L319
self.dynamics is an object of system dynamics
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/integrators/common.py#L177
system dynamics is a protocol
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/integrators/common.py#L38-L41

Then to understand where the specific system dynamics of the robot is defined, we have to go to the initial definition of the integral:

we define the system dynamic as an ode state
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/api/model.py#L264-L269
The ode is wrapped as a system dynamic state
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/api/ode.py#L26-L80
Then the system dynamic function computes the state to be integrated
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/api/ode.py#L374-L452
Then the ode state translates into the input to be integrated
https://github.com/ami-iit/jaxsim/blob/main/src/jaxsim/api/ode_data.py#L541

After this analysis, it was still unclear how to extract the acceleration and forces information. Indeed, the computation of the system acceleration (that needs to be then integrated via the integrator) is done under the hood by the integrator and done for different integrator state $y$.
I think there are two points now:

How can we simplify the  code such that it is easier to  read ? Is it also needed ? I was thinking about having abstract classes instead of protocol and having specific methods with specific generic output, and being more transparent then in  the ode class.
How can we export the joint acceleration and contact wrenches ?  I think we can assume that we want them computed at the final time fo the integration, so that all the information (joint position, velocity etc) are associated to the same time step, one idea could be to re-call the system_velocity_dynamic with the specific input, but we would need to wrap around  other method and it will still not be the simulator info

Do you have any idea/ suggestion ? @ami-iit/darwin

Hi @CarlottaSartore, thank you for this analysis! Indeed, the integrators is quite intricated and can easily lead to confusion. I'll try to answer to each of the issue points.
For what regards:

After this analysis, it was still unclear how to extract the acceleration and forces information.

The acceleration $\dot{\nu} \in \mathbb{R} ^{6 + N}$ can be computed, but not extracted, from either
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/api/model.py#L884-L891
or, if you want to take into consideration the contact dynamics, and the joint limits and frictions, from
https://github.com/ami-iit/jaxsim/blob/d2cf2206ef3962ca69d67c191ebeea86d6cc5d87/src/jaxsim/api/ode.py#L88-L95

How can we simplify the code such that it is easier to read ?

The good thing about having protocols instead of abstract classes is that they help decouple the interface from the implementation, as we don't actually need a class, but a single function the computes the system dynamics. Nevertheless, I'm not against using abstract classes

How can we export the joint acceleration and contact wrenches ? I think we can assume that we want them computed at the final time fo the integration, so that all the information (joint position, velocity etc) are associated to the same time step, one idea could be to re-call the system_velocity_dynamic with the specific input, but we would need to wrap around other method and it will still not be the simulator info

Regarding this, I currently have two ideas:

Modify js.model.step so that is returns additional info. This will allow to access information regarding the system acceleration, but at the same time we would need to propagate some info that are currently inside the code
Separate the dynamics and the integration in two additional methods. This would allow to access intermediate information before manually integrating the dynamics. We could still keep the step function, while adding js.model.forward and js.model.integrate. The first function will compute the complete dynamics of the system returning an object of ODEState with the derivatives of the state variables, while the second one will simply call integrator.step in a completely functional way

One warning that I receive a lot of time when I work with jaxsim, is the following
RuntimeError: The SO(3) integrator received a quaternion at t0 that is not unary.
This is called here
https://github.com/ami-iit/jaxsim/blob/fbb77269a78a4172fb57187616dbcecfb3c2187e/src/jaxsim/integrators/common.py#L501-L511
In the refactoring analysis of the integrator we should understand also why we encounter this error so often , since actually the quaternion should have been already normalized by other functions

We have documented the usage of  integrator in jaxsim, closing issue


Cannot type '@' on Windows+Chrome

It's [Alt Gr] + [Q] for me under Windows 8.1.

With firefox on windows 10, AltGr + q for typing @ does not work as well.
But I was able to fix this for firefox:
( you need to distinguish between "alt" or "ctrl" or "AltGr", that behaves like "alt+ctrl").
function recordModifiers(evt, display) {
var alt = (evt.altKey && !evt.ctrlKey);
var ctrl = (evt.ctrlKey && !evt.altKey);
var modifiers =
(evt.shiftKey ? Squeak.Keyboard_Shift : 0) +
(ctrl ? Squeak.Keyboard_Ctrl : 0) +
(alt || evt.metaKey ? Squeak.Keyboard_Cmd : 0);
display.buttons = (display.buttons & ~Squeak.Keyboard_All) | modifiers;
return modifiers;
}

Nice! I'd write it like this—should work the same, right?
function recordModifiers(evt, display) {
    var shiftPressed = evt.shiftKey,
        ctrlPressed = evt.ctrlKey && !evt.altKey,
        cmdPressed = evt.metaKey || (evt.altKey && !evt.ctrlKey),
        modifiers =
            (shiftPressed ? Squeak.Keyboard_Shift : 0) +
            (ctrlPressed ? Squeak.Keyboard_Ctrl : 0) +
            (cmdPressed ?

Ok, I made a pull request
https://github.com/bertfreudenberg/SqueakJS/pull/47
this adds another change for making this work on chrome for windows too.


Give possibility not to run FreeSurfer for not segmented subjects

Some BIDS datasets can have inconsistencies or FreeSurfer can fail for some subjects. If one run baracus on this kind of dataset, it will try to run FreeSurfer segmentation on unsegmented subjects, but this can be undesirable. I would suggest to add an option to skip this step.

closed by #16


Improvements meta-list

[ ] tests
[ ] better dependency resolution (emit types after the entire pass)
[x] integrate new struct improvements
[ ]  support const array definitions

I think these features have already been implemented and I've updated a new improvements list in #228, so close this.


Please add a copy of manufacturer archive

You guy's might want to add a full copy of the manufacturers archive for this display to this repo before it vanishes
http://pan.jczn1688.com/directlink/1/ESP32 module/2.8inch_ESP32-2432S028R.rar
Includes, among other things LVGL demo, which is why I went looking for it.
I see your comment that you sourced some data from this file, but discarded the rest, this seems wrong to disregard aditional information and examples provided.

I added my reasons for what I discarded in this readme:
https://github.com/witnessmenow/ESP32-Cheap-Yellow-Display/blob/main/OriginalDocumentation/Readme.md
The examples are probably ok to add, but adding 900mb (!?) of code to this repo doesn't seem like a very good idea to me!

well please consider adding in the LVGL examples, its one of the selling points of the display

If adding putting the file in the repo is not an option then the repo owner could create an Internet Archive account and put the file there.
https://help.archive.org/help/uploading-a-basic-guide/
Here are file hashes for 2.8inch_ESP32-2432S028R.rar
SHA1 b164e7e8507ad50aa2bd4f3fd76d57b2a65f81b8
SHA256 21a1757aa3d1a836b0c41c7da653412b8a67928f5454ecc032c579876f674c3e
SHA512 9275ab578e7d962617ed7f1fc8928a95f92bbb56e126bfe961c153e050e2d0caa7c81ab2ad3c0e8c64d6339e8e7d6cb5a481c7bd54faa0dbb8ab47ac20a6b292
The file was downloaded from http://pan.jczn1688.com/directlink/1/ESP32 module/2.8inch_ESP32-2432S028R.rar a few minutes ago


Bump version to 1.2.6

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
Bump version to 1.2.6
Why
Release preparation (1.2.6)
Known limitations
N/A

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr750.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr750.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr750.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr750.previews.kube001.services.stellar-ops.com/


[c++ grpc] Fix includes when bond::Void is used

When a service contains events or parameterless methods, the generated
code uses bond::Void as the request/result type. To serialize
bond::Void, bond_reflection.h needs to be included.
The generated service code now includes bond_reflection.h if any service
contains an event or a parameterless method.
Fixes https://github.com/Microsoft/bond/issues/735

chwarr, is it possible to use gRPC with bond_apply?

It should work with bond_apply.lib; however, you'd have to make sure to #include <bond/core/bond_apply.h> before any of the generated _grpc.h headers. We have an example that does this for the user-generated type.


feat: Update Notebook Magics to match CLI functionality

Key changes:

Make sure all CLI commands have a magic command with the same arguments
Add tests that verify behavior in Jupyter notebooks
Add an example Jupyter notebook that can be run with make jupyter-example in order to see what the output looks like across most commands when a dev is making changes

Note: A follow up PR will be focused on testing and validating Databricks output. More limited in options for that but will see what we can do. It might just be manually running stuff and verify the output works.

make sure to update all the docs


Thanks a lot

this plugin is awesome for me , thanks a lot !!!
🌹🌹🌹

Glad you're liking it! Shoutout to the tldraw team for making such a good drawing app! They're the ones doing the hard work :)


Repo Certificate Authority field for private repos

Public repos pass certificate verification using the system's trusted root certificates. Private repos may require a Certificate Authority that is not in the system's trusted root certificates.
Although some users may be able to add CAs system-wide, it is easiest and cleanest to allow setting CAs on a per-repo basis. This way no change system-wide change is necessary in order to use private repos with custom CAs.
The repo.yaml syntax can be extended with a CA string field so users can provide the CA together with the repository information.
@andreabolognani

If I understand this correctly, we would basically allow setting a CA field in repo.yaml, and then create a custom https client which will only talk to a repo serving a cert form there. Correct?

Yes, the HTTPS client would use the repo.yaml's CA for certificate verification. I'm not sure whether the client should exclusively use the CA or the CA is used in addition to the system-wide trusted root certificates.
This way, a private repo that is available via HTTPS with an internal CA will work.


parser: fill states and intents in appdef for cmds and queries

@mail2nnv IExtensionBuilder doens't implement Intents() and States(), you can fix it in the branch

@mail2nnv IExtensionBuilder doens't implement Intents() and States(), you can fix it in the branch

Can not fix in you branch
Details

> git push michael-say 1670-parser-build-state-and-intens-for-commands-appdef:1670-parser-build-state-and-intens-for-commands-appdef
git: 'credential-manager-core' is not a git command. See 'git --help'.
To https://github.com/michael-say/voedger
 ! [remote rejected] 1670-parser-build-state-and-intens-for-commands-appdef -> 1670-parser-build-state-and-intens-for-commands-appdef (permission denied)
error: failed to push some refs to 'https://github.com/michael-say/voedger'

I made fix at my branch, see issue #1671
Please run git pull upstream main to receive my changes from voedger/voedger main


[docs] Add more detail to MachinePool section of usage docs.

Add more detail to the MachinePool section of usage docs to include information regarding MachinePool platform immutability as well as noting that a MachinePool for workers mirroring the workers defined in the install-config is not required but will result in Hive managing the worker MachineSets.
https://github.com/openshift/hive/pull/1891 should update this once Labels and Taints are merged instead of overridden.
/assign @2uasimojo
/cc @suhanime

:+1: I'll add both of those suggestions.

:heart:
/lgtm


chore: release 1.7.4

:robot: I have created a release beep boop
1.7.4 (2024-07-30)
Miscellaneous

deps: bump github.com/docker/docker from 26.1.3+incompatible to 26.1.4+incompatible in the go_modules group (#151) (e9b0b68)
deps: bump github.com/rudderlabs/rudder-go-kit from 0.34.2 to 0.35.1 (#148) (8de9c7b)
deps: bump google.golang.org/api from 0.188.0 to 0.189.0 (#150) (e9631fb)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/rudderlabs/sqlconnect-go/releases/tag/v1.7.4 :sunflower:


Trying solution for Core issue #132479

All changes:

Connect cooling_enabled binary_sensor directly to self._cooling_enabled
Add cooling_enabled binary_sensor for Adam where missing
Issue #132479 probably results from the fact that self._cooling_enabled is, in __init__.py, linked to the binary_sensor cooling_enabled. When this one is not representing self._cooling_enabled but the xml-key cooling_enabled, errors might occur.

Summary by CodeRabbit
Release Notes

New Features

Introduced a new binary sensor, cooling_enabled, to indicate the cooling functionality for specific devices.

Tests

Enhanced test coverage for cooling functionalities in both the Adam and Anna modules.
Added new test cases to validate the state of cooling features during transitions between heating and cooling modes.

Chores

Updated version number to v1.6.3 in the project configuration.
Revised changelog to document the implementation of cooling-related fixes.

These updates improve the overall functionality and reliability of the cooling features within the system.

Will set to v1.6.3 when approved.


Change the quick start guide missing dependencies

Hi,
Thank you for the PR.
We're changing the quickstart in https://github.com/IBM/the-mesh-for-data/pull/456 and the new version does not require these dependencies (it basically depends just on helm3 and kubectl). I suggest to wait for it to be merged.
However, the dependencies list in the Development docs does not include all of the dependencies you added. Specifically nc and gcc. Would you mind submitting a PR to update it? Also, can you share where gcc was required?

@roee88 : sure, I will raise a PR to updated location
gcc was required for one of the go dependencies
# runtime/cgo
cgo: exec gcc: exec: "gcc": executable file not found in $PATH


fix: set node executable

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove dependencies on yii-core

If we really want this to be useful independently of yii-core we should remove the application from the test suite.
We can't guarantee that it works properly without the core if we still initialize our singletons like Yii::$container and Yii::$app during testing.

That's very true!
But I think we could start with getting the build completely fixed by easier way (with core).
And then remove core.

Done.


fix: #2846 horizontal layout buttons

What

Part of
fixes #2846

@monsieurtanuki

Thank you @abughalib!
Can you share with us the 3 screenshots? The one you attached recently is better but not good: the space between the save button and the right edge is smaller than the space between the card and the right edge. The spaces should be identical.

I'd prefer to not block this pr, having a bit off spacing is better then the whole buttons not aligned

I'd prefer to not block this pr, having a bit off spacing is better then the whole buttons not aligned

@M123-dev I don't block anything: I just give my opinion as required.
And the PR is already approved by someone else, so feel free to approve too or to merge.

Screenshots.

Codecov Report

Merging #2899 (b3b164a) into develop (d9c35d7) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           develop   #2899      +/-   ##
==========================================
- Coverage     6.71%   6.70%   -0.01%
==========================================
  Files          228     228
  Lines        11117   11118       +1
==========================================
  Hits           746     746
- Misses       10371   10372       +1

Impacted Files
Coverage Δ

...h_app/lib/pages/product/edit_ingredients_page.dart
0.00% <ø> (ø)

...h_app/lib/pages/product/nutrition_page_loaded.dart
0.00% <ø> (ø)

...mooth_app/lib/pages/product/simple_input_page.dart
0.00% <0.00%> (ø)

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


Add GOV.UK Mainstream Guide template page

Closes the 1140 issue
Original work by @rivalee - this version adds changes requested in review.

Add GOV.UK Mainstream Guide template to the Prototype Kit
Update the Tutorials and templates page to add a link to the GOV.UK Mainstream Guide template, and a link to the 'Content design: planning, writing and managing content' manual
Update Sass to support: contents list, pagination

Changes to docs/views/includes/head.html
When docs.css is loaded after application.css, the underlined link style overrides the non-underlined link style in the pagination component. Reversing the load order fixes this. This may or may not be the best fix. Ideally, docs.css shouldnt load everything that application.css has already loaded.

paused as we found an issue in the rendering of pagination compared to the govuk component:

small dot to the right
alignment of text
link underline should only be on page title

I think the issue is due to the order of these classes (taken from browser inspector, so top is loaded later than bottom):
.app-prose-scope a, .govuk-link {
    font-family: "GDS Transport", arial, sans-serif;
    -webkit-font-smoothing: antialiased;
    -moz-osx-font-smoothing: grayscale;
    text-decoration: underline;
    text-decoration-thickness: max(1px, 0.0625rem);
    text-underline-offset: 0.1em;
}

.app-pagination__link {
    display: block;
    text-decoration: none;
    padding-bottom: 20px;
}

text-decoration: underline; in _links.scss is overriding text-decoration: none; from _pagination.scss
but I'm not sure why. GOVUK Frontend is loaded first in the code order

the issue is that the docs loads both applications.scss and docs.scss, it is docs.scss that is overriding the link style

@joelanman would you mind adding a bit of context into the message for commit 8e96a496d51b52f0bea5025041affe9c45a83e2d explaining what the changes are? Right now I don't really understand what's being changed, or why.

@lfdebrux on https://github.com/alphagov/govuk-prototype-kit/commit/8e96a496d51b52f0bea5025041affe9c45a83e2d, it was mainly to fix a typo in class names, -- where it should have been -. Probably best to squash all the commits to be honest

@lfdebrux on 8e96a49, it was mainly to fix a typo in class names, -- where it should have been -. Probably best to squash all the commits to be honest

Okay. I think it would be best to keep the change to stylesheet orders in a separate commit, otherwise squashing sounds fine to me. Are you happy to do that?

@lfdebrux sure, sounds good


GPU utilization was quite low

Hello
Thanks for your work on this project. This is an awesome package.
I followed the instructions in the documentation to successfully install PyG, but when running the network under /benchmark/kernel, the GPU utilization was quite low.I run the code on 2080ti , and the GPU utilization is less than 10%， Increasing batch can't improve.

Environment
OS: Ubuntu 16.04
Python version: 3.6
PyTorch version: 1.5.0
CUDA/cuDNN version: 10.1

The benchmark/kernel script runs a hyperparameter grid search with initially quite low number of layers and node dimensionality. Increasing those (+ batch_size) should also increase GPU utilization.

Thank you for your suggestion, this really works. By the way, are there any examples of unpool for EdgePooling?

Thank you for your suggestion, this really works.
But, when I use the EdgePooling layer in my network, the GPU utilization rate is also very low.

I see. EdgePooling is inherently sequential which limits its GPU utilization.


fitness score

After compare the  fitness score  value in the benchmark, can I get the conclusion, the bigger this value, the better the algorithm are?

Hi @IntelDan ,
The smaller fitness score the better (it's the distance between point clouds). In the benchmark, since they all converged, you see almost the same fitness score.

OK, I know.
Thank you so much!


How to use Sequential Chains in LangFlow

How to use Sequential Chains in LangFlow

Chain Sequential COMBINATOR.json
here is a custom component that does that but it needs a bit of imporvement (orders params)

let suppose I have four component chatInput-->Prompts--->AzureOpenAI--->Chatoutput. I have import all component individually in Json file. In vs code i have load like this run_flow_from_json(flow="simple_new.json", input_value=query.question).
how can I perform chaining on it


rustPlatform.buildRustPackage-based packages: cargoBuildFlags is not overridable

Describe the bug
I tried to build i3status-rust with a custom feature and failed. Presumably this is tied to rustPlatform.buildRustPackage being weird.
To Reproduce
Steps to reproduce the behavior:

nix build --impure --expr 'let nixpkgs = import (builtins.getFlake "nixpkgs") {}; in nixpkgs.i3status-rust.overrideAttrs (old: { cargoBuildFlags = ["--features=notmuch"]; buildInputs = old.buildInputs ++ [nixpkgs.notmuch]; })'
Launch i3status-rust with following added to the default config:[[block]]
block = "notmuch"
query = "tag:inbox AND tag:unread"
threshold_warning = 1
threshold_critical = 10
maildir = "~/Maildir"
interval = 30

Receive error: Error in block 'notmuch': Unknown block! signifying that the package was in fact built without the feature.
Alternatively, observe nix log output for the package - around build phase, the cargo command is printed. The build flags injected are absent.

Expected behavior
The build flags from overrideAttrs call are passed to Cargo.
Screenshots
n/a
Additional context
I assume this is present on every package using the function mentioned in the title.
Notify maintainers

cc @globin @backuitist @Ma27 as maintainers of i3status-rust
cc @Mic92 as the one who added cargoBuildFlags to the function and is showing up in git blame
Metadata

system: "x86_64-linux"
host os: Linux 5.9.2, NixOS, 21.03.20201108.90576ad (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.4pre20201102_550e11f
channels(root): ""
channels(vika): ""
nixpkgs: /nix/store/82mrz2yf5a8zs0b8hhgc8f1b1w4sfkgi-source

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
 - i3status-rust
 - rustPlatform.buildRustPackage
 - rustPlatform

# a list of nixos modules affected by the problem
module:

I just checked and it seems as I can use custom cargoBuildFlags on master (cf97b2aeec8d838b49c9ef64224ab47b4c09f1b6). @kisik21 can you check if that's still a problem for you? :)

Yes, this works fine now (and wasn't working when I subscribed).

I guess we can close it then. @kisik21 in case I misunderstood something and the issue still exists, please let us know.

I can confirm that the problem is resolved, as build logs indicate successful application of Cargo build flags when using the example from my original issue report.


[BUG] Farmer doesn't work in Towny.

Farmer V6 is not working properly with the latest version of Towny. You cannot buy a farmer in any plot, and even if a player has not bought a farmer in his plot, the crops are corrupted as if there was a farmer there.
PlotClaim method has been changed, so I think that's why it doesn't work.
Server Version: 1.17.1
Farmer ver: v6

Checked towny and couldn't find issue.


[1.14.4] crash problem.

After joining the world with this mod minecraft crashes.

Please post your minecraft log


Wallpaper not changed on Manjaro cinnamon

Hi,
I installed pywal from the AUR, I tried both the 'python-pywal' and 'python-pywal-git'. They both installed OK and the terminal color do change, but the wallpaper doesn't change for some reason.
The pywal version I got is 0.6.9.
Output from command line:
wal -t -i train_road_smoke_79239_1920x1080.jpg
image: Using image /home/guangzhi/Pictures/wallpapers/train_road_smoke_79239_1920x1080.jpg
colors: Found cached colorscheme.
colors: Set terminal colors.
wallpaper: Set the new wallpaper.
export: Exported all files.
reload: Reloaded environment.

So it saids wallpaper is changed but in fact not.
Another minor suggestion is: in the installation guide you listed a few dependencies, the line xfce, gnome, cinnamon, mate is supported DEs, I assume. Then the following line I assume is for wallpaper setting softwares. I think there is some ambiguity that it could mean any one of feh, nitrogen, bgs, hsetroot, habak, following the DE logic, or all of them. I've installed all of them already.
Please help.

After bit of messing around, I think the code section relevant to wallpaper setting in cinnamon is in 'wallpaper.py':
elif "muffin" in desktop or "cinnamon" in desktop:
    util.disown(["gsettings", "set", "org.cinnamon.desktop.background", "picture-uri", "file://"+urllib.parse.quote(img)])

I tried to run that command manually in terminal:
gsettings set org.cinnamon.desktop.background picture-uri file:///home/guangzhi/Pictures/wallpapers/train_road_smoke_79239_1920x1080.jpg

And gsettings complained that libselinux.so.1 is not found. I installed that from AUR, and now I got this:
GLib-GIO-Message: Using the 'memory' GSettings backend.  Your settings will not be saved or shared with other applications.

Please help

The real dirty hack I use is to comment out the "desktop" and let it fall back directly to "set_wm_wallpaper()" function:
def change(img):
    """Set the wallpaper."""
    if not os.path.isfile(img):
        return

    desktop = get_desktop_env()

    if OS == "Darwin":
        set_mac_wallpaper(img)

    elif OS == "Windows":
        set_win_wallpaper(img)

    #elif desktop:
        #set_desktop_wallpaper(desktop, img)

    else:
        set_wm_wallpaper(img)

This sounds like an issue with your cinnamon install and not with pywal's wallpaper handling.


MKShapesCollection doesn't expose his shapes array

When I was trying to render the my shapes collection as on overlay on a MKMapView I realised that shapes wasn't public.
This: https://github.com/Neku/GEOSwift/commit/ab599efddc365416c91bcd650b877fda6b6d34c2 fixed it for me, but I was wondering if it is a mistake or there is some other way to get them.

No no you're correct, my mistake!
Thanks

fixed on branch Swift-2.0: https://github.com/andreacremaschi/GEOSwift/commit/932317c98ddc48fd18cbade1b1af5982293e81bf


Remove extra newlines from example code

This removes extra trailing \n symbols in the sample code for custom log formatters. Leaving them there results in double newlines in logs.

Current coverage is 22.76% (diff: 100%)

Merging #810 into master will not change coverage

@@             master       #810   diff @@
==========================================
  Files            16         16
  Lines          3884       3884
  Methods         436        436
  Messages          0          0
  Branches        279        279
==========================================
  Hits            884        884
  Misses         2998       2998
  Partials          2          2

Powered by Codecov. Last update 87d92cd...363f260


Chore/bump chrono

A little bit of maintenance while I'm at it : fix #727.
Checklist

[x] An issue to be fixed by this PR is listed above.
[ ] New tests are added to ensure new features are working. End-to-end tests are usually in the ./frb_example/pure_dart example, more specifically, rust/src/api.rs and dart/lib/main.dart.
[x] The code generator is run and the code is formatted (e.g. via just refresh_all).
[ ] If this PR adds/changes features, documentations (in the ./book folder) are updated.
[x] CI is passing.

Remark for author
You will notice that I .expect(...) in codegen on NaiveDateTime conversion : this is the exact same behaviour as chrono under the hood : to convince yourself, please peek at chrono 0.4.20 from_timestamp and chrono 0.4.23 from_timestamp.
There's also some tidy-up from cargo clippy lint of latest Rust 1.66.
Thanks 🙏

Nice!


Draft of prereq database table

US: Developers will require an updated API as specified in https://github.com/YACS-RCOS/yacs-prereq-web/issues/2. This API needs a new table to pull from.

See postgres schema