DNM: Re-enable nfs-ganesha

@jprovaznik it's not building on Ubuntu:
deb http://download.ceph.com/nfs-ganesha/deb-V2.5-stable/luminous/ xenial main
Hit:1 http://security.ubuntu.com/ubuntu xenial-security InRelease
Hit:2 http://archive.ubuntu.com/ubuntu xenial InRelease
Hit:3 http://archive.ubuntu.com/ubuntu xenial-updates InRelease
Hit:4 http://archive.ubuntu.com/ubuntu xenial-backports InRelease
Get:5 http://download.ceph.com/debian-luminous xenial InRelease [8522 B]
Get:6 http://download.ceph.com/debian-luminous xenial/main amd64 Packages [13.4 kB]
Fetched 22.0 kB in 0s (28.5 kB/s)
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package nfs-ganesha
E: Unable to locate package nfs-ganesha-ceph
E: Unable to locate package nfs-ganesha-rgw

@leseb there was a wrong repo name, much better now

@jprovaznik anything else you want to do with this? Or can we go ahead and merge?

@leseb it does what I need, if it looks good to you I think we can merge it


Force modelcards to use utf-8 encoding

hfh v0.10.0 introduced modelcards that was previously hosted on a separate repo (https://github.com/nateraw/modelcards). When moving it into huggingface_hub, the use of utf-8 encoding by default has been lost which cause some issues for windows users as highlighted by @BenjaminBossan in https://github.com/skops-dev/skops/pull/162#issuecomment-1263516507.
This PR forces the use of utf-8 when loading/saving a repocard. Once that is merged, I will cherry-pick the commit in v0.10-release branch and bump the version to v0.10.1 so that this inconsistency stays as little as possible in the wild :)
cc @nateraw @adrinjalali

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


camera API, where can I find it?

In your older commits I found:
camera.pixformat(0)   # 0:JPEG, 1:Grayscale (2bytes/pixel), 2:RGB565
camera.framesize(10)  # 1:96x96, 2:160x120, 3:176x144, 4:240x176, 5:240x240
                      # 6:320x240, 7:400x296, 8:480x320, 9:640x480, 10:800x600
                      # 11:1024x768, 12:1280x720, 13:1280x1024, 14:1600x1200
                      # 15:1920x1080, 16:720x1280, 17:864x1536, 18:2048x1536
camera.quality(5)  # [0,63] lower number means higher quality
camera.contrast(0)  # [-2,2] higher number higher contrast
camera.saturation(0)  # [-2,2] higher number higher saturation. -2 grayscale
camera.brightness(0)  # [-2,2] higher number higher brightness. 2 brightest
camera.speffect(2)  # 0:,no effect 1:negative, 2:black and white, 3:reddish,
                        # 4:greenish, 5:blue, 6:retro
camera.whitebalance(0)  # 0:default, 1:sunny, 2:cloudy, 3:office, 4:home
camera.aelevels(0)  # [-2,2] AE Level: Automatic exposure
camera.aecvalue(0)  # [0,1200] AEC Value: Automatic exposure control
camera.agcgain(0)   # [0,30] AGC Gain: Automatic Gain Control

but where in the ESP_IDF documentation can I find some explanation?
I seem to be getting some over-exposed pictures so maybe aelevel,
aecvalue and/or agcgain need tweaking from the default values.
Thank you,
Dave

Those are camera module specific info not ESP_IDF.
Please search this in google
OV2640 Camera Module Software Application Notes.pdf
You can ESP-IDF camera component source code here, https://github.com/espressif/esp32-camera

OK, thanks.

There is no proper manual concerning the camera component. I did guess works and experiments.
The parameters to the calls can be found here, https://github.com/espressif/esp32-camera/blob/master/sensors/ov2640.c
Sorry, for not being helpful, but this is all I have.

Actually, that does have some information about those values in it (down around line 500).  Yesterday I tried changing the saturation and ended up getting a colour picture!?!
More guess work required.
Thanks again.


chore(deps): update terraform terraform-ibm-modules/landing-zone-vpc/ibm to v7.16.1

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/landing-zone-vpc/ibm (source)
module
patch
7.16.0 -> 7.16.1

Release Notes

terraform-ibm-modules/terraform-ibm-landing-zone-vpc (terraform-ibm-modules/landing-zone-vpc/ibm)
v7.16.1
Compare Source
Bug Fixes

change acl for_each conditional to fix bug (#725) (60867f4)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 2.4.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix an issue in db connect function

sqlite3.connect() function will throw an exception instead of returning none if running into any errors. So we should try/catch the error instead of checking for none result.

Note: I'm still waiting for my new testbed to be ready to verify the fix.

LGTM pending usual stuff:
(1) test: need a cut-n-paste example of triggering the exception (2) need CI pass when  CI is back

@shaominchen Please rebase your branch with master to get .drone.sec changes.

Not this PR; but I feel all the errors reported to user/admin shouldn't be marked as "internal error". They should be clear as to what error or type of error is.


Cannot resolve keyword 'storage_filename' into field

Describe the bug
RUNTIME Error while trying to add a filter on Storage filename for csv files imported from S3 bucket.
To Reproduce
Steps to reproduce the behavior:

Setup S3 bucket and connect to a project as source storage
Sync the S3 bucket to import CSV files
Go to the Project page
Trying setting a filter using Storage filename filed

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Environment (please complete the following information):
Deployed on Ubuntu.
Additional context
Traceback:
Traceback (most recent call last):
  File "/home/ubuntu/venv/lib/python3.10/site-packages/django/db/models/sql/query.py", line 1955, in add_fields
    join_info = self.setup_joins(name.split(LOOKUP_SEP), opts, alias, allow_many=allow_m2m)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/django/db/models/sql/query.py", line 1648, in setup_joins
    path, final_field, targets, rest = self.names_to_path(
  File "/home/ubuntu/venv/lib/python3.10/site-packages/django/db/models/sql/query.py", line 1562, in names_to_path
    raise FieldError("Cannot resolve keyword '%s' into field. "
django.core.exceptions.FieldError: Cannot resolve keyword 'storage_filename' into field. Choices are: annotations, cancelled_annotations, comment_authors, comment_count, created_at, data, drafts, file_upload, file_upload_id, id, inner_id, io_storages_azureblobimportstoragelink, io_storages_gcsimportstoragelink, io_storages_localfilesimportstoragelink, io_storages_redisimportstoragelink, io_storages_s3importstoragelink, is_labeled, last_comment_updated_at, locks, meta, overlap, predictions, project, project_id, total_annotations, total_predictions, unresolved_comment_count, updated_at, updated_by, updated_by_id

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/ubuntu/venv/lib/python3.10/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/django/utils/decorators.py", line 43, in _wrapper
    return bound_method(*args, **kwargs)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/label_studio/data_manager/api.py", line 227, in get
    queryset = self.get_task_queryset(request, prepare_params)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/label_studio/data_manager/api.py", line 195, in get_task_queryset
    return Task.prepared.only_filtered(prepare_params=prepare_params)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/label_studio/data_manager/managers.py", line 661, in only_filtered
    return queryset.prepared(prepare_params=prepare_params)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/label_studio/data_manager/managers.py", line 472, in prepared
    queryset = apply_filters(queryset, prepare_params.filters, project, request)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/label_studio/data_manager/managers.py", line 350, in apply_filters
    value_type = type(queryset.values_list(field_name, flat=True)[0]).__name__
  File "/home/ubuntu/venv/lib/python3.10/site-packages/django/db/models/query.py", line 867, in values_list
    clone = self._values(*_fields, **expressions)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/django/db/models/query.py", line 835, in _values
    clone.query.set_values(fields)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/django/db/models/sql/query.py", line 2272, in set_values
    self.add_fields(field_names, True)
  File "/home/ubuntu/venv/lib/python3.10/site-packages/django/db/models/sql/query.py", line 1982, in add_fields
    raise FieldError("Cannot resolve keyword %r into field. "
django.core.exceptions.FieldError: Cannot resolve keyword 'storage_filename' into field. Choices are: annotations, cancelled_annotations, comment_authors, comment_count, created_at, data, drafts, file_upload, file_upload_id, id, inner_id, io_storages_azureblobimportstoragelink, io_storages_gcsimportstoragelink, io_storages_localfilesimportstoragelink, io_storages_redisimportstoragelink, io_storages_s3importstoragelink, is_labeled, last_comment_updated_at, locks, meta, overlap, predictions, project, project_id, total_annotations, total_predictions, unresolved_comment_count, updated_at, updated_by, updated_by_id

Hi @hooman-bayer - thanks for your report! I can confirm that I was able to reproduce this issue; unfortunately it seems it's existed for quite a while. We'll work on a fix :)

Thank you @jombooth . It seems to be just a missmatch in naming of the fields. Do you have an ETA for a path? It is essentially an important feature since after syncing a large batch of files from S3 thats the most important field where one can do filtering.
Thanks in advance

Hi @hooman-bayer - this was fixed in our latest release, 1.11.0!


No offer/answer messages are created

Describe the bug
This is in relation to : https://github.com/microsoft/MixedReality-WebRTC/issues/303#issuecomment-621698775
I've created two scenarios, one using the basic node-dss (from a fresh build and project import) and the second from my custom signaling service, which was working before I pulled yesterday.
Currently any call to StartConnection(), CreateOffer() or CreateAnswer() doesn't generate any messages, and the OnSDPOfferReadyToSend/OnSdpAnswerReadyToSend aren't being called.
To Reproduce
Steps to reproduce the behavior:

Pull & build latest Mr-Webrtc master
Open the generated unity project, setup the videoconferencing scene to use the node-dss server that's setup (I added a few debug logs
Hit play
No message is generated

Expected behavior
Previously on step 4 with the basic node-dss example, a message would be generated.
Using a build from the release branch works well with both the node-dss signaling, and my custom websocket signaling.
Environment

Platform: Unity Editor
Unity version: 2018.4

I have not tried, but if I follow exactly your steps then I do not expect anything to be generated.
Messages are only generated when a session starts to be negotiated with StartConnection(). This doesn't happen automatically at Unity startup unless you are in the very specific (and implementation defined behavior) where : your PeerConnection component is initialized first by Unity, AND the Signaler is initialized next, AND the peer connection finished to connect, AND the various tracks are initialized after that and raise a RenegotiationNeeded event, AND you have AutoCreateOfferOnRenegotiationNeeded = true. Any other deviation will fail to automatically generate an offer. And since the order in which Unity starts components is implementation-dependent, there is no expectation that this works. You can force the order of the scripts but I wouldn't rely on that if I were you, this is all too brittle.
Now assuming that you are indeed calling Startconnection(), for example via the "CreateOffer" button in the Unity scene, then I followed those steps:

Start the node-dss server after using set DEBUG=dss* to see requests
git clone --recursive https://github.com/microsoft/MixedReality-WebRTC/
Open the Visual Studio solution, build Debug x64
Open Unity, load the VideoChatDemo scene
Configure the NodeDssSignaler with the correct local/remote peer IDs
Press Play
Start TestAppUWP
Configure the signaler with the correct local/remote peer IDs
Press the "CreateOffer" button in the VideoChatDemo scene

As expected, I do get some offer and answer, and the connection establishes. I can see the 2 video tracks in TestAppUWP.

After investigating further I found my signaler's implemented update method hid the inherited method, perhaps it's possible to implement the base update method as virtual?
Either way, the hidden method handled task dequeues, closing the issue.

@HyperLethalVector I am not sure what base class you are talking about because Signaler already has a virtual Update() method.

Strange, it wasn't on my end but after a revert it was. I'll blame me for
it (don't code at 1am folks!). Either way I was hiding the original method,
which was causing the issue I posted.
Thanks anyways for entertaining my silly mistake.
On Mon, May 4, 2020, 10:17 PM Jerome Humbert<EMAIL_ADDRESS>wrote:

@HyperLethalVector https://github.com/HyperLethalVector I am not sure
what base class you are talking about because Signaler already has a
virtual Update() method
https://github.com/microsoft/MixedReality-WebRTC/blob/050680e4b900f61d1f290b2e98b8fe5b711f3130/libs/Microsoft.MixedReality.WebRTC.Unity/Assets/Microsoft.MixedReality.WebRTC.Unity/Scripts/Signaling/Signaler.cs#L205
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/MixedReality-WebRTC/issues/314#issuecomment-623442513,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACEKFSYLRWMCQSDCVGLTEXTRP22M3ANCNFSM4MWZBX2Q
.


For logged in applicant, update messaging on saving draft

When saved as draft right even if the user is logged in and no email is
sent, currently a message is displayed saying an email is sent. This PR
updates the messaging so it message is not displayed for logged in user.

Fixes #3556

Added the messaging on where to access the draft from.

I think the message is great. What happens if the person have not registered yet and cannot login? Is this a separate issue? Thanks

@fourthletter If you are not logged in when saving a draft you gen an e-mail sent to you with instructions. The message  you see after saving a draft explains this as well. This has been in place for quite some time.
This PR fixes the issue where logged in user got the same message in error.

Clicking on the url/link provided in the email leads to this page for the applicant to either register or login


Implemented LineRange-class and improved overall code-coverage

As requested, these are tests and implementations from some of the exercises during the testing-lecture (summer-semester 2023, University of Applied Sciences Munich):
A LineRange-class was implemented for grouping coverage-information for consecutive lines-of-code.
Various new tests improve overall line-, branch- and mutation-coverage for the project.
Testing done
All tests (old and new) run successfully on my local system (based on current state of the main-branch in the official coverage-model-repository).
### Submitter checklist
- [x] Make sure you are opening from a **topic/feature/bugfix branch** (right side) and not your main branch!
- [x] Ensure that the pull request title represents the desired changelog entry
- [x] Please describe what you did
- [x] (not relevant) Link to relevant issues in GitHub or Jira
- [x] (not relevant) Link to relevant pull requests, esp. upstream and downstream changes
- [x] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Thanks!


Update jose4j to 0.7.3

Updates org.bitbucket.b_c:jose4j from 0.7.2 to 0.7.3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.bitbucket.b_c", artifactId = "jose4j" } ]

labels: library-update, semver-patch

Superseded by #71.

Superseded by #71.


Windows firewall trouble

Hi,
If I make an exception for wacs.exe in the Windows Firewall for the correct port, the authentication fails. If I make the same rule for "all programs", it works fine. As I am not so comfortable leaving the port open while it is unused, how can this be solved? Is it a setting I should change, or something in the software itself? If I test this with e.g. Port Listener (so, opening the same port only for that software), everything works fine.
Kind regards,
mindstormsking

perhaps you are using the shared mode where SYSTEM is doing the http port 80 binding via http.sys?
You can either track down how to allow that in windows firewall, or just do the self-hosted validation mode instead

I used the self-hosted validation method on a custom port, so I don't think that I need to share anything? The router is correctly translating the external port 80 to the internal custom port, and the firewall is correctly set to open up the internal custom port (as checked with Port Listener). But still the request is blocked if only wacs.exe is allowed...
What else could it be?

I would need @WouterTinus  to confirm, but maybe the self hosting option still utilizes http.sys to do the actual binding (meaning you'd need to whitelist that with windows firewall). Doing a cursory search seems to indicate that you can't target a single application:
https://social.technet.microsoft.com/Forums/lync/en-US/b0255585-78d9-4fba-b9b5-03177277f8d4/open-firewall-for-httpsys?forum=w7itpronetworking

I haven't delved into this, but wacs.exe is built as a self-contained executable with the .NET Core runtime embedded inside. When you run it, Microsoft does some magic which involves unpacking a whole bunch of .dlls into a temporary folder. I wouldn't be surprised if the actual process that runs and opens the port is not that very same wacs.exe, but rather a dotnet.exe or something similar.
I also wouldn't be able to tell you how .NET Core implements its HttpListener and whether that's visible to the firewall as coming from a specific process/executable.
In any case you could maybe enable and disable the firewall rule using additional steps in the scheduled task.

Hi,
I also delved a bit, apparently you can add a whitelist for http.sys in the firewall by setting program to SYSTEM. I can confirm this works. I can enable/disable this by the scheduled task, perfect I think.


Typos

@ohadschn : Thanks for your contribution! The author(s) have been notified to review your proposed change.


NTR: wet and dry age related macular degeneration

For new term requests, please provide the following information:
Preferred term label
wet age related macular degeneration
dry age related macular degeneration
Synonyms
wet AMD
dry AMD
wet ARMD
dry ARMD
Textual definition
Wet age related macular degeneration results from the growth of abnormal leaky blood vessels beneath the macula that eventually damage the area of the eye responsible for central vision.
PMID:16178119
Dry age related macular degeneration is characterized by the presence of age-related deposits called drusen and atrophy.
PMID:26319344
Suggested parent term
age related macular degeneration
See: https://github.com/obophenotype/human-phenotype-ontology/issues/2192
These terms are in MEDDRA: http://purl.bioontology.org/ontology/MEDDRA/10075568 and http://purl.bioontology.org/ontology/MEDDRA/10075567

I think these terms are in Mondo, need to add synonyms.
double check


regen deviceupdate with tests

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check for com.azure:azure-iot-deviceupdate
API changes have been detected in com.azure:azure-iot-deviceupdate. You can review API changes here


make docker network for example app

https://www.fpcomplete.com/blog/2017/12/building-haskell-apps-with-docker


Why there is no VGG_4096 in the evaluate_nus.py

In lines 124 and 125 of the evaluate_nus.py, is there missing VGG_4096 of the model input?

Hi @hyqyoung thank you very much for pointing it out 😄


Edit Data Set, fill form with DS data.

Proposed Changes

Edit data set
Get the data of a data set and put this data in the form.
Click on the list of the data sets and go to the edit form.

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests
[x] Build locally
[ ] Code format / lint
[ ] Docs have been reviewed and added

Pull request type
Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A

Fixes #79
What is the new behavior?

When the user clicks on the edit button of the data set in the dataset table, this button calls the onSelectDataSet(selectedDataSet) method, this method redirect to "/dsdetils" passing the single data set, this route is for DataSetCreationComponent.
In the DataSetCreationComponent, checks if the Data Set Object exists, if exists, calls the onSeeDataSet() method, this method fill the fields of the data sets.
The onSeeDataSet() method, this method pass the data to newDataSet and fills the fields of the form. It subscribes to the data from the getDataSet().
The name and the description are set in formGroup1.
In the feature set selection, the radio button is selected if the data set has any feature set related.
The eligibility criteria are filled using newDataSet.eligibility_criteria.
Modified the formGroup1, now have 2 new formControls: name and description, ths is because the material compoment causes errors.
Also is modified the call to the back-end, before for gettin the feature set list, the component calls getFeatureList() method of backEndService, now call getFeaturesetsList(id) method, and get the real feature set list.

Everything is correct and clear


Throwing it all away: remove array melting and map implementation

Before implementing C++ templates for arrays and buffers etc

looks good so far

@jystic saved us from the peril of C++


Use AutoGluon in Kaggle Kernels

A majority of recent Kaggle competitions require to submit via notebooks.
We could use !pip install autogluon, but got the a bunch of errors, even just importing the tabular task. For example
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-17-6f7d1b4fed2f> in <module>
----> 1 from autogluon import TabularPrediction as task

/opt/conda/lib/python3.7/site-packages/autogluon/__init__.py in <module>
     15 logger.setLevel(logging.ERROR)
     16
---> 17 from .utils.try_import import *
     18 try_import_mxnet()
     19

/opt/conda/lib/python3.7/site-packages/autogluon/utils/__init__.py in <module>
     18 from .util_decorator import classproperty
     19 from .custom_process import CustomProcess
---> 20 from .openml_download import *
     21

/opt/conda/lib/python3.7/site-packages/autogluon/utils/openml_download.py in <module>
      1 import os
      2 import numpy as np
----> 3 from sklearn.impute import SimpleImputer
      4 from sklearn.model_selection import train_test_split
      5

/opt/conda/lib/python3.7/site-packages/sklearn/impute/__init__.py in <module>
      2
      3 from ._base import MissingIndicator, SimpleImputer
----> 4 from ._knn import KNNImputer
      5
      6 __all__ = [

/opt/conda/lib/python3.7/site-packages/sklearn/impute/_knn.py in <module>
      7 from ._base import _BaseImputer
      8 from ..utils.validation import FLOAT_DTYPES
----> 9 from ..metrics import pairwise_distances_chunked
     10 from ..metrics.pairwise import _NAN_METRICS
     11 from ..neighbors._base import _get_weights

/opt/conda/lib/python3.7/site-packages/sklearn/metrics/__init__.py in <module>
      5
      6
----> 7 from ._ranking import auc
      8 from ._ranking import average_precision_score
      9 from ._ranking import coverage_error

/opt/conda/lib/python3.7/site-packages/sklearn/metrics/_ranking.py in <module>
     33 from ..utils.sparsefuncs import count_nonzero
     34 from ..exceptions import UndefinedMetricWarning
---> 35 from ..preprocessing import label_binarize
     36 from ..preprocessing._label import _encode
     37

/opt/conda/lib/python3.7/site-packages/sklearn/preprocessing/__init__.py in <module>
      6 from ._function_transformer import FunctionTransformer
      7
----> 8 from ._data import Binarizer
      9 from ._data import KernelCenterer
     10 from ._data import MinMaxScaler

/opt/conda/lib/python3.7/site-packages/sklearn/preprocessing/_data.py in <module>
     34 from ._csr_polynomial_expansion import _csr_polynomial_expansion
     35
---> 36 from ._encoders import OneHotEncoder
     37
     38 BOUNDS_THRESHOLD = 1e-7

/opt/conda/lib/python3.7/site-packages/sklearn/preprocessing/_encoders.py in <module>
      8 from ..base import BaseEstimator, TransformerMixin
      9 from ..utils import check_array
---> 10 from ..utils.fixes import _argmax
     11 from ..utils.validation import check_is_fitted
     12

ImportError: cannot import name '_argmax' from 'sklearn.utils.fixes' (/opt/conda/lib/python3.7/site-packages/sklearn/utils/fixes.py)

Several competitions don't allow use Internet, which means we cannot pip install. Instead, we should submit a PR to add autogluon at https://github.com/Kaggle/docker-python/blob/master/Dockerfile#L129 and

Regarding the notebook error, I don't think this is caused by an issue in AutoGluon. Have you tried restarting the kernel after pip installing to ensure the correct versions are loaded? Additionally, can you do a pip freeze after the pip install + kernel restart to ensure the proper sklearn version is being used?
Regarding adding AutoGluon to the Kaggle Dockerfile, we should at least wait until modularization occurs, and I feel that there will be a large amount of work to get it compatible with the Docker, as we don't necessarily support the latest versions of all the packages we use, and we may have dependency version conflicts with some of the many other packages in this Dockerfile build, which is a bit concerning.

@mli
I was able to use kaggle kernels with multiple older versions of autogluon, c.f. https://www.kaggle.com/jonaswm/covid-ag
so maybe seemingly-incorrect version of dependency can be corrected via: !pip install scikit-learn<=0.22
as a temporary fix?
Longer term it seems like we should consider removing from .openml_download import *.
@tlienart Is it ok to remove that statement to prevent this error from happening when users import autogluon if they have improper sklearn version installed? AFAIK the functions in openml_download are solely used in your unit-test/tutorial, so they don't need to be imported when autogluon is imported?

sounds good

sounds good

@jwmueller So then Thibaut's tutorial would import load_and_split_openml_data how? By using the absolute path?
Is the problem that nothing in utils must depend on sklearn? Seems to make sense. In the beginning, I had this load_and_split_openml_data somewhere next to the bayesopt searcher code.
We could also move it back into the tutorial if you like, since the unit test which calls it is deactivated anyway (it never really worked, but not because of openml).

@mseeger
I agree if the unit-test is not being used it would be nice to move this function to tutorials/ instead.  But I didn't mean that nothing in utils/ can depend on sklearn, this is unavoidable as utils/tabular/ uses sklearn all over.  I suspect the error here is due to incompatible sklearn version present, which is certainly not the fault of the openml_downloads functions.
What I do think though is "optional" imports from sklearn should be limited to narrow scope (ie. those that aren't core autogluon functionality, eg. just for fetching the data in your tutorial).  I'd say nearly half the issues I encounter when trying to use AutoGluon-Tabular with other python packages are due to incompatible sklearn-version dependencies. So it's always pleasant if AutoGluon can mostly work with more versions of sklearn beyond what is officially listed in setup.py (ie. doesn't break upon autogluon-import but maybe only breaks at the moment where a sklearn-version-specific method is called in autogluon)

Can someone suggest a temporary fix to run AutoGluon in Kaggle Kernels ? Tried reverting to older versions as mentioned in the kernel link shared by @jwmueller
!pip install mxnet==1.6.0 autogluon==0.0.6
Still getting :
ImportError: cannot import name 'isStringType' from 'PIL._util' (/opt/conda/lib/python3.7/site-packages/PIL/_util.py)

Same error
ImportError: cannot import name 'isStringType' from 'PIL._util' (/opt/conda/lib/python3.7/site-packages/PIL/_util.py)

@elvinaqa @salil-gtm Can you try to follow the logic of the working Kaggle Kernel linked here?

You'll want to do !pip install autogluon.tabular[all] to get all the models.

Thank you! I got very good information!

@daikikatsuragawa Decided to give a shot at it in minimal LOC: https://www.kaggle.com/innixma/top-3-in-3-lines-of-code-w-autogluon

@Innixma
I seem to have misunderstood. And thank you.
I think it's amazing!
# https://github.com/awslabs/autogluon
# AutoGluon is a SOTA AutoML framework
!pip install autogluon.tabular[all]

https://www.kaggle.com/innixma/top-3-in-3-lines-of-code-w-autogluon

You didn't misunderstand, the reason I don't need to do the other lines (mxnet, pip, setuptools), is because they are already installed on kaggle kernel. It is still necessary outside of Kaggle.
I think it makes sense to keep the general instructions, so I've reverted it in my notebook.
Also, I ran AG for 1 hour in the notebook and got 0.804 without feature engineering :D https://www.kaggle.com/innixma/top-3-in-1-line-of-code-w-autogluon

Closing as it appears there are no issues with installing AutoGluon on Kaggle as of v0.1 and v0.2.


1.0.1 Stopped working on windows

1.0.1 Stopped working on windows

@dospolov this plugin only generates the SSL/TLS certificate on .npm_modules/.vite folder, it doesn't install it on SO/browser.
If you want to install the certificate on the OS (to be used on chrome/edge/chromium/iexplorer):

go to previous folder, open the file in text editor, copy to clipboard from --- BEGIN CERTIFICATE ---- to --- END CERTIFICATE --- (including both lines) and paste in a new file called localhost.cer (you can also export certificate (public key) from browser options)
double click on localhost.cer and click on Install button and follow instructions: Windows will ask you if you want to install the certificate as a new certificate authority, click YES and unsafe message will not be shown again.

@dospolov I'lll send a PR to include a small guide for Windows installation and Firefox (not using OS).

You should get this warning on Chrome, you can then click on "Show advanced", then "Continue toward <IP_ADDRESS>".

Also, this plugin should only be used for local development. If you wish to expose a https version to the web, you should not use this plugin and instead use a proper web server (like NGINX) with a proper SSL certificate (for example, letsencrypt.org). But I imagine "local.simloud.be" actually redirect to localhost?
If you wish to install the certificate authority to your browser to hide this error, you can use this plugin instead: https://github.com/liuweiGL/vite-plugin-mkcert, BUT I advise against doing so.

The error only shows once, then you just have to click on "continue" and the browser won't complain anymore
The solution to copy/paste the certificate is not future-proof: if the node modules folder get deleted or if you have other project, you will have to do this multiple times because it will generate a new certificate each time
The solution to copy/paste the certificate to manually install it to your browser is a real hassle and not the common usecase: I believe people using this plugin just want https to work quickly for testing purpose
It's a bad practice to mess with certificate authorities

So I advise against giving those instructions in the readme.

@hugoattal
thanks for a detailed explanation
I mistakenly used vite-plugin-basic-ssl and vite-plugin-mkcert together. And updating vite-plugin-basic-ssl caused the original problem. Now I'm happy with vite-plugin-mkcert only


Context-aware dynamical decoupling

Summary
Implement context-aware dynamical decoupling (DD) from arXiv:2403.06852.
Details and comments
When selecting decoupling sequences for an idle qubit, context-aware DD takes into account what actions are occuring on neighboring qubits. It ensures that neighboring sequences are mutually orthogonal (to avoid collisions, like in staggered DD) but also whether the qubit is a CX or ECR target/control spectator. This is nicely summarized in a figure of the paper, where the different wire colors denote different decoupling sequences:

This particularly leads to improvement if a lot of qubits are idle at the same time or if they are next to ECR/CX gates. One important category of circuits in this class are the layers in twirled circuits. This pass also works best if the ECR/CX gate times are the same for all connections, which is almost always the case on IBM devices.
Some simple experiments show that this pass performs better than the current standard in Qiskit, PadDynamicaDecoupling with an X-X sequence. Here's some basic benchmarks, see the paper for more:

This code is based off @kdk and @ajavadia 🙂

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

313 of 322 (97.2%)  changed or added relevant lines in 4 files are covered.
149 unchanged lines in 31 files lost coverage.
Overall coverage increased (+0.04%) to 89.775%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/transpiler/passes/scheduling/padding/context_aware_dynamical_decoupling.py
310
319
97.18%

Files with Coverage Reduction
New Missed Lines
%

qiskit/circuit/library/standard_gates/swap.py
1
98.21%

qiskit/primitives/backend_sampler.py
1
98.86%

qiskit/circuit/library/standard_gates/rxx.py
1
97.87%

qiskit/primitives/base/base_sampler.py
1
96.97%

qiskit/circuit/library/standard_gates/rzx.py
1
97.87%

qiskit/circuit/library/standard_gates/ryy.py
2
95.74%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
2
88.39%

qiskit/circuit/library/standard_gates/xx_plus_yy.py
2
96.23%

qiskit/primitives/base/base_estimator.py
2
94.59%

qiskit/circuit/library/standard_gates/xx_minus_yy.py
2
96.23%

Totals

Change from base Build<PHONE_NUMBER>1:
0.04%

Covered Lines:
67251

Relevant Lines:
74911

💛  - Coveralls

Anything holding this up from being merged? Happy to lend a hand if needed to speed up the process @Cryoris @eliarbel

https://github.com/Qiskit/qiskit/pull/12825#pullrequestreview-2404327735 is a very good catch -- the newly added gates that implement the DD sequences didn't have a duration attached, since they weren't present when the scheduling pass was run. I'll add a test to check for this 🙂

After discussion with @ElePT I also updated the default behavior of min_joinable_duration: On realistic devices, the 2q gate lengths vary a little -- yet might not want to insert DD sequences for two concurrent 2q gates, just because they naturally differ slightly. The new default is therefore that we only add DD sequences for idle times beyond 2 * (max_2q_duration - min_2q_duration). This needs some benchmarking on real devices before merging.

#12825 (review) is a very good catch -- the newly added gates that implement the DD sequences didn't have a duration attached, since they weren't present when the scheduling pass was run. I'll add a test to check for this 🙂

Just so you know I'm working on removing per instruction durations in #13506 so we probably don't need to change this now. It would have been important for 1.3, but in case 2.0 we just making extra work for both of us.

I removed any setting and usage of Instruction.duration in 21c4ea0, so hopefully you don't have to update the context-aware DD in #13506 🙂 That means the timeline drawer will fail now, but I assume this will be fixed in your PR 🙂


CalendarMenuView hangs

I've used vertical calendar to show reports. I need to show only selected months. So I've added following delegate method
 (BOOL)calendar:(JTCalendarManager *)calendar canDisplayPageWithDate:(NSDate *)date
Everything works fine.. But when we scroll to the last month from that prev month, it slightly hangs for a fraction of seconds. I've checked with delegate. As it's taking the date of next month which i don't want to show.
I think thats the reason for the hang.
Can you help me on this..

I'm also having a little hang on page change. I've found that if I comment all the code in my
func calendar(calendar: JTCalendarManager!,prepareDayView dayViewV: UIView!)
method the hang disappears. So may be your method is doing to much.
What I've found that I think can be a bug (or at least can be optimised) is that when you go to a new page, it reloads 3 pages (actual, prev and next), but it should load actually only one page.

Hi folks did you get the solution for the above issue?
I am also facing the same.
Let me know, that may help me out.


Dapp 19

AltLayer Airdrop Season One Update

🚀 Exciting Announcement! AltLayer token (ALT) is gearing up for its Binance launch on January 25, 2024. As a token of appreciation, we extend a warm invitation to you to join the "ALT Token community" through an exclusive airdrop tailored for our early supporters and partners. Here's a quick overview:

Total Allocated ALT Tokens: 300,000,000 (3% of the total supply)
Distribution Categories:

Ethereum Mainnet
Binance Mainnet

For a comprehensive breakdown of criteria and further details, refer to the complete announcement Oficial Website.
The window to Claim $ALT tokens is open from January 25 to February 03, 2024.
Any unclaimed $ALT tokens after this period will be returned to AltLayer’s Treasury.
We deeply appreciate your contribution to AltLayer's success! Stay tuned for more exciting developments in Season One.
Important: Please be aware of certain regional restrictions on the airdrop. Review the disclaimer before participating.
0xArsen, @aria1991, @pedrohba1, @ghosharindam, @wkler, @vejin, @corbinpage, @Opdata, @jceb, @BigMurry, @sankumsek, @ehgp, @JustinLee2018, @lushl9301, @githublister, @yhl125, @andy4thehuynh, @0x-r4bbit, @BourneXu, @yellzhi


Why do ASG have separate, different scheduling process with other services?

Is your feature request related to a problem? Please describe.
Hi team, I use and develop instance scheduler but I see that the flow to schedule ASGs is complicated, different than other services, that it uses a separate orchestrator, a lambda to observe change in the dynamodb table stream, and a potential risk of unconvertable schedules into cronjob of ASG scheduling actions.
Describe the feature you'd like
ASG, and other services will be scheduled in the same orchestrator.
Additional context

Thanks for reaching out! Is there a particular requirement or use-case that would benefit from making all services share the same orchestrator?
ASGs require a fundamentally different scheduling flow due to the requirement to use Schedule Scaling Rules as an intermediary step for schedule operation. As such we set up a longer polling interval than normal (1 hour) for ASG scheduling and then added the dynamodb stream lambda as a short-circuit response to changes in the schedule database.
Since this flow is fundamentally different from the busy-polling approach used by EC2 and RDS, we thought it woould be more appropriate to use a separate ochestrator to manage it rather than overloading the existing orchestrator and needlessly complicating the control flow within.

@CrypticCabub  But I saw the main difference in ASG data and its definition.
Separate workflows cause developers very hard to understand the flow and it contain many lambda, hidden rule (1 days affection, 1 hour cron)


Issue #79: Skipping tests in certain package

fixes https://github.com/TestArmada/flank/issues/79

Looks good. Thanks!


[SPARK-11836] [SQL] use existing SQLContext for udf/cast

udf/cast should use existing SQLContext.

Test build #46546 has started for   PR 9915 at commit e89912b.

Test build #46546 has finished for   PR 9915 at commit e89912b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46546/
Test PASSed.

Thanks - merging this.

(Can you close the pull request since I've merged it?)


[bilibili] Fix extraction

Dear sir:
I‘m a college student from China.Can read your youtube-dl  on GitHub,I feel honored.But when I use it,I find biibilibil.py is out of date.So I tried to rewrite some of the code and restored some of its functionality.
My code is not good because I'm just a beginner, and my English is very poor.Hope to get your affirmation.
yours   ZhangXinYang

In #10375, @gdkchan reverses the desktop version player and writes an algorithm. Seems your version is based on the mobile player? It's fine to have multiple algorithms in a single extractor. Would you like to rebase your fix to the current master?

This time I was in accordance with the help of other Collaborator to change the  code, I hope it can pass

This time I was in accordance with the help of other Collaborator to change the code, I hope it can pass

There are still merge conflicts.

where?My Python is not good,please tell me,think you.

Try git fetch origin master and then git merge origin/master.

I have used it, and Is there any problem now？

Could you push the result after git merge?


Version Packages (beta)

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to beta, this PR will be updated.
⚠⚠⚠⚠⚠⚠
beta is currently in pre mode so this branch has prereleases rather than normal releases. If you want to exit prereleases, run changeset pre exit on beta.
⚠⚠⚠⚠⚠⚠
Releases
<EMAIL_ADDRESS>Minor Changes

6579966: existing corner-radius tokens, adding net-new global corner radius tokens, and introducing new alias names that point to these global values.
Design Motivation
More on Spectrum 2 rounding: https://s2.spectrum.corp.adobe.com/page/object-styles/#rounding
In Spectrum 2, rounding also applies to the different t-shirt sized components and varies based on a Major Second logarithmic scale, rounded to whole numbers to avoid using half-pixels. This allows for components to retain a consistent and identifiable shape at all sizes. Because of this there are now component size specific tokens like corner-radius-small-size-medium or corner-radius-medium-size-extra-large to help us keep track of the nuance within t-shirt sizes.


[#1596][FOLLOWUP] fix(netty): Send failed responses only when the channel is writable

What changes were proposed in this pull request?

Send failed responses only when the channel is writable.
Print debug logs when the data is successfully sent, reducing log output.
Reduce the duplicated error log.

Why are the changes needed?
A follow-up PR for: https://github.com/apache/incubator-uniffle/pull/1605.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing UTs.

Why do we need this? Could you add a ut for this?

Why do we need this? Could you add a ut for this?

If channels being closed is the root cause, there is no need to send the failed responses, because the channel is already closed and you are not able to send responses. No need to add UTs. This is just an improvement. The origin codes will only introduce duplicated errors.

LGTM. merged to master and branch 0.9.


HDMI not working

I cannot get HDMI to work under NixOS on my Orange Pi 5 Plus with 32G of RAM.
I am using the latest UEFI firmware here with a config essentially identical to the demo Colmena one provided in this repo (including the nixos-rk3588.nixosModules.boards.orangepi5plus.core module, and on the same nixpkgs version as this repo).
The UEFI firmware menu and GRUB show up on HDMI fine but once the kernel starts I lose signal. I can interact with the kernel fine over Ethernet and console UART (except for issue #37). I don't know why the HDMI doesn't work. /proc/cmdline appears to have the relevant command line options still. I tried setting up a display server and that did not help either.
cc @Naxdy who apparently had this working recently

HDMI works fine using tag<PHONE_NUMBER>

I can confirm that it does work for me using the 6.1.75 kernel and mainline mesa. However, the monitor I'm testing won't recognize the device unless I manually specify the resolution to be of an aspect ratio of 16:9. If I try my monitor's native aspect ratio (32:9), it actually won't recognize it, which I think is weird. However, I was able to observe the exact same behavior with my RPi5, so I think this is more something to do with my monitor than with the device.
Which output device did you test with ?

I have now tried several monitors from various brands, and also a couple HDMI switchers/converters. I can't get a picture after the kernel starts with any device, but all work fine in TianoCore and GRUB.
Can you share your working configuration flake?

My config is part of a huge monorepo with many custom modules, but I'll try to paste in the relevant parts here:
My kernel + graphics setup (haven't updated this repo's ref yet):
let
  modDirVersion = "6.1.75";

  linux-src = pkgs.fetchFromGitHub {
    owner = "Joshua-Riek";
    repo = "linux-rockchip";
    rev = "5c43412639fd134f0ba690de2108eaa7ea349e2a";
    hash = "sha256-aKm/RQTRTzLr8+ACdG6QW1LWn+ZOjQtlvU2KkZmYicg=";
  };

  panthor-base = "aa54fa4e0712616d44f2c2f312ecc35c0827833d";
  panthor-head = "c81ebd8e12b64a42a6efd68cc0ed018b57d14e91";

  panthor-patch = pkgs.fetchurl {
    url = "https://github.com/hbiyik/linux/compare/${panthor-base}...${panthor-head}.patch";
    hash = "sha256-/5SvlGsgHbn1i68+GASOeNZmxoZiIt280L6iUFz3MFU=";
  };

  kernel = (pkgs.linuxManualConfig {
    inherit modDirVersion;

    version = "${modDirVersion}-rockchip-joshua";

    src = linux-src;

    kernelPatches = [{
      name = "panthor.patch";
      patch = panthor-patch;
      extraConfig = { };
    }];

    configfile = ./wearable-pc/kernel_config;

    config = import ./wearable-pc/conf.nix;
  }).overrideAttrs (old: {
    nativeBuildInputs = (old.nativeBuildInputs or [ ]) ++ [
      pkgs.ubootTools
    ];
  });
in
{
  hardware.graphics.package = lib.mkForce pkgs.mesa.drivers; # note this is from `nixos-unstable`, so 24.2.4
  boot.kernelPackages = lib.mkForce (pkgs.linuxPackagesFor kernel);
#...

I'm using systemd-boot:
boot.loader.systemd-boot.enable = true;
boot.loader.efi.canTouchEfiVariables = true;
boot.kernelParams = [ "boot.shell_on_fail" ];

I tried switching to systemd-boot without making any other changes and HDMI comes up perfectly first time. Strange! It also fixes the log spam in #37 .
Not sure why the example config uses GRUB, I prefer systemd-boot anyway and was going to switch over once I figured out HDMI. This might pose a problem for the SD images and stuff, but the demo at least should probably be switched over (and the underlying issue fixed). I would bet there is something with device trees.

I expect the default is grub because the raw-EFI image used here comes from nixos-generators and that is configured to use grub by default.  I am using systemd as well, though I didn't experience any problems when I used grub with the Orange Pi 5 Plus 32Gb.  I considered adding options to the demo for systemd when I rewrote it, but I wrote it before I was sure systemd was going to work.


Fix: confidence values are sometimes floats as strings

fixes #3
Newer versions of tesseract seem to be returning floating point confidence numbers, which pytesseract is not handling correctly yet. Casting to float first will work with current and future versions of pytesseract.

Do you think @ratoaq2 this PR could be merged 🙏

Thanks


Can you please  create any python file for GB or Percentage

Can you please  create any python file for coverting into GB or in Percentage

Yelp/elastalert is no longer maintained. Please use jertel/elastalert. Questions to the discussion below
https://github.com/jertel/elastalert/discussions


Controls points are incorrect for horizontal images.

Hello! Thank you for your library upgrade!
There is some bug I'm trying to fix. When image is not square shape, e.g. horizontal, control points are much lower from the end of image.
http://joxi.ru/823KQn4SaGvKq2
Do you have any idea how to fix that?
Thank you!

Thanks for reporting. I'm not able to follow when the problem happens. Is it when adding control points, or moving them?
Can you post the image that's causing the problem here?


Adding an image from static folder

Hi, I already want to thank you for this theme.
I'm a beginner and I have issues adding a picture in one of my articles.
The image is in the static folder and my article is in the content folder.
As there is no example of ExampleSite I allowed me to open an issue.
My repo is here
Best regards

fix


Image rendering leaves a lot of empty space under long legend

I'm submitting a ...

[x] bug report
[ ] feature request

Current behavior:

Wnen using Graphite render to image api, Graphite leaves empty space below legend - the longer legend is, the bigger empty space. When there is just one metric, the distance between legend and lower border of the image is about 5px. When there is five metrics, it is about 13px. And for twenty three, about 108px. At this point this empty space takes more height than the graph itself in my scenario.
Images: https://i.imgur.com/qcGX338.png https://i.imgur.com/OC8nLxg.png https://i.imgur.com/E8kcExx.png
Expected behavior:

I would expect that empty space will not grow with metrics count.
Steps to reproduce:

I believe there are no some special steps. Just /render?width=460&from=-60minutes&until=-&height=500&hideLegend=false&target=some.avg.metric.*

Sorry, @skaurus , can't reproduce on latest release (1.1.5), also in 1.1.6. I see no difference on distance between text and image edge for 4 or 10 metrics:

Which graphite version do you use?

Issue similar to https://github.com/graphite-project/graphite-web/issues/253, which was fixed in 2013.

Probably in 0.9.12 and newer releases.

I tried to find the version, but couldn't. Where do I look?
#253 looking promising indeed... Did we really not updated for so long...

Lol. /version/ says that we use 0.10.0-alpha 🤦♂
Usually we keep our soft updated, obviously, with Graphite this is not the case.
Thanks!

0.10.0-alpha means old master, before 1.0 release. Can be really old. You can apply only patch above if full upgrade is not desireable.


How to change the center icon?

as above

Current not support :(
You can download the source, and modify FilterMenuDrawable.java  as you want


feat(material-luxon-adapter): Implementation

I've continued work after @crisbeto in #14679
Took his Luxon adapter and tried to add all building stuff like in moment-adapter
Probably have done some work, but I need help with bazel configuration and etc..
Implementing luxon one from #20599

I never gave permission to take the code from #14679.
We haven't decided how new date adapter will be handled within npm packages.

I was planning on reviving the old PR once we've decided how to handle the new adapters.

OK, in any case just trying to help just wanted this feature be in production ASAP

It might be easier to copy over the adapter into your code and use it directly since we can't guarantee when the new adapter package will be available.


Is ulna(r) hypoplasia the same as upper limb mesomelic dysplasia?

Question for community, seeking consensus regarding MONDO:0008620; upper limb mesomelic dysplasia, synonyms: ulna hypoplasia, and Fryns-Hofkens-Fabry syndrome (with potentially redundant related synonyms "Fryns Hofkens Fabry syndrome" and "ulnar hypoplasia").
Is "ulna hypoplasia" truly an exact match? Is "ulnar hypoplasia" a distinct, but related concept?
OMIM 191440 uses "ulnar hypoplasia" as the preferred name and UPPER LIMB MESOMELIC DYSPLASIA as an alternate name, cross-referenced for the MONDO term above.
But would a general finding of "hypoplasia of the ulna" be treated as equivalent to this specific OMIM entry? See HP:0003022
UMLS separates out the finding of "short ulnae" C1862132 and disease of "ulnar hypoplasia" (synonyms: Hypoplasia of the ulna, upper limb mesomelic dysplasia, Fryns Hofkens Fabry syndrome) C1860614, though the HPO term maps to both.

I think @ahamosh may be best able to answer this. Thanks in advance, @ahamosh. :)

I would regard "upper limb mesomelic dysplasia" as a phenotypic feature, that is at least close to this:
https://hpo.jax.org/app/browse/term/HP:0005011
Fryns-Hofkens-Fabry syndrome is defined as isolated upper limb mesomelic dysplasia, which is a disease since it is definiing not one phenotyic feature but the patient ("isolated").
Ulnar hypoplasia is undergrowth of the ulna (one of the two forearm bones). upper limb mesomelia means the middle parts of  the bones of the arm are disproportionately short. So, the two concepts are similar in a way but are not synonymous.


mlocate incorrectly warns about lack of user support

Issue description
nixos-rebuild switch prints trace: warning: mlocate does not support searching as user other than root
The message seems misleading as everything works as expected, even as a regular user.
Steps to reproduce
services.locate = {
  enable = true;
  locate = pkgs.mlocate;
};

Technical details

System: (NixOS: 17.09.git.e1b01a0 (Hummingbird))
Nix version: (nix-env (Nix) 1.11.15)
Nixpkgs version: (17.09.1547.2231575a7d)

This still happens

That's surprising since it works for me without root.
I only experienced issues when the wrong locate binary is used. If I recall correctly findutils has higher priority and causes this.
Can you which locate and paste whatever error you get?

Sorry hit the wrong button by accident.

$ which locate
/run/wrappers/bin/locate
$ file /run/wrappers/bin/locate
/run/wrappers/bin/locate: setgid executable, regular file, no read permission
$ ls -lh /run/wrappers/bin/locate
-r-x--s--x 1 root mlocate 18K May 16 13:00 /run/wrappers/bin/locate

The warning is correct, but the wording is misleading. What it’s trying to warn you is not that locate will only run as root, but rather that updatedb will run as root.
You can prevent the warning from being shown (with no other effect) by explicitly requesting that updatedb run as root with services.locate.localuser = null.

The warning is correct, but the wording is misleading. What it’s trying to warn you is not that locate will only run as root, but rather that updatedb will run as root.

This is warning is still confusing to me. If I'm building an index of files, I'd expect that if I run it as a non privileged user many files will not be reachable and therefore these won't end up in the locate database/index. Given the way locate works, I'd also expect to have one database per system, not one per user.
So from this my expectation is that updatedb can only successfully index everything if run as root.
In other words, I'm still not sure what is the purpose or goal of the message.

Is it just that the updatedb command needs to run as root?

is this bad in any way?
why is this special given the purpose of the tool?

Is it that the fact that it runs as root is in some way a security concern?
Is this to clarify that despite the existence of an mlocate group (and user?) these are not actually used for indexing,

or is the point something else entirely.
Do other locate tools work in such a different way that justifies the existence of this warning for mlocate?

mlocate’s updatedb doesn’t have a --localuser option like findutils’ updatedb does. Therefore, the NixOS module ignores services.locate.localuser when using mlocate (#26191). It prints a warning to indicate that your expectation has been violated.
The mlocate group is there so that the mlocate locate binary can be setgid mlocate rather than setuid root.
Given that mlocate is designed with an updatedb that indexes everything and a locate that only shows you your own files, there doesn’t seem to be much point in running updatedb as another user. If you want that, why not switch back to findutils?

mlocate’s updatedb doesn’t have a --localuser option like findutils’ updatedb does. Therefore, the NixOS module ignores services.locate.localuser when using mlocate (#26191). It prints a warning to indicate that your expectation has been violated.

I see, in this context, the above makes sense.

Given that mlocate is designed with an updatedb that indexes everything and a locate that only shows you your own files, there doesn’t seem to be much point in running mlocate’s updatedb as another user. If you want that, why not switch back to findutils?

I actually want the full system indexing. To me locate has always been a synonym of find / ..., except several times faster.

@andersk Could you include your db28ce3  commit in a pull request? I think that's a good solution and we can close this issue.

@unode It’s been in #79239 this whole time.

My bad, I read only the latest updates and the pull request was higher up.
Did a bit of noise on the pull request. It's a harmless change so I was surprised to see it sitting there for this long.

To avoid updatedb: can not find group `mlocate', looks like both of these are needed:
services.locate.package = pkgs.mlocate;
services.locate.localuser = null;

when using:
environment.systemPackages = with pkgs; [
...
  mlocate
];

with or without:
services.locate.enable = true;

But if not using the environment.systemPackages = with pkgs; [ mlocate ]; (so only using the services.locate.enable = true; alone) then there's no updatedb: can not find group `mlocate' error.
If anyone wants to guess why, here's the source: https://github.com/NixOS/nixpkgs/blob/nixos-23.11/nixos/modules/misc/locate.nix


An error occurred: The checkpoint you are trying to load has model type qwen2_vl but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.

Im getting the following error when using the vLLM template
An error occurred: The checkpoint you are trying to load has model type qwen2_vl but Transformers does not recognize this architecture. This could be because of an issue with the checkpoint, or because your version of Transformers is out of date.
I believe its because the latest version of transformers is required:
pip install git+https://github.com/huggingface/transformers accelerate
Is it possible to install this over the top?

try using the qwenllm/qwenvl:latest container image and a docker cmd similar to this:
python -m vllm.entrypoints.openai.api_server  --served-model-name Qwen2-VL-72B-Instruct-GPTQ-Int4 --model Qwen/Qwen2-VL-72B-Instruct-GPTQ-Int4 --dtype float16 --gpu-memory-utilization 0.8 --tensor-parallel-size 2 --trust-remote-code --max-model-len 8192 --limit-mm-per-prompt image=5,video=1

the above works for me when i create pods (each worker has 2 x A40) . serverless endpoints don't work, sadly


local auth should not redirect by default

Currently one can set successRedirect or failureRedirect but not a null value for either. This is an issue when you are relying on CORS, because browsers do not permit a redirect after a pre-flight options request, which many frameworks depend on.
There should be an option to disable redirects and instead just send back a general success response. I've hacked it together for my project, but should be standardized (and documented) IMO. I would submit a PR, but I'm not exactly sure what the response should be.

@harangue Yes I agree. I assume you are using local authentication without Ajax or sockets in this case?
The redirect was initially introduced for OAuth but I think is also included for local auth when not using ajax.  I could see how that would cause issues with CORS. Do you mind posting a gist or some sample code highlight the issue so that I have a better understanding of how you are sending data for auth?

I'm using the feathers-authentication client. Simply calling
    return this._app.authenticate({
      type: 'local',
      username: username,
      password: password
    });

I get a successful request (with cookies set), but it's a redirect to /auth/success.

XMLHttpRequest cannot load http://localhost:3000/auth/local. The request was redirected to 'http://localhost:3000/auth/success', which is disallowed for cross-origin requests that require preflight.

Problematically, when I change the res.redirect in middleware/index.js to a res.send({}); as an attempted fix, the feathers-authentication client expects the response to contain both the token and user data. Not sure how it's supposed to get that from the redirect either, so it may just be broken entirely.

@harangue Yup got it. Thanks for posting that! It really shouldn't behave that way. I'll probably just remove the redirect altogether from local auth (it's not really needed) or at the very least make it optional.
IMO I made a mistake adding it there. The redirect really is only needed for OAuth (and potentially form posts), if you needed to redirect and are auth'ing over ajax or sockets then you can do that on the client.

Awesome! Should I just switch to websocket authentication in the mean time?

Oh - this is important. Just upgraded to 0.4.0 and feathers-localstorage won't work either because options fails

XMLHttpRequest cannot load http://localhost:3000/storage/user. Response for preflight has invalid HTTP status code 404

Should I open a separate issue?

@harangue Nope this one is fine. I'm going to be working in feathers-auth all day today so I should be able to get a bunch of stuff closed.

:+1:

@ekryski I'll just keep adding things as I run into them here, if that's cool. :)
    app.token = function() {
      return storage().get('token').then(data => data.value);
    };

I don't think it should assume you're authenticated. The natural thing for me to try was using app.token to check if I was logged in, but that of course throws an error if there's no data.value;

@ekryski Pretty sure hook.params.token in client/hooks.js is actually supposed to be hook.params.token.vakue. Was setting Authorization: [object Object] for me until I changed that.

@harangue you are correct. I thought I had committed that fix while working on the RN app.

@ekryski Another issue I encountered - the documentation claims that you can set idField on your authentication({}) call and implies that it will be available to restrict-to-self. This is not the case. I noticed there was a commit to fix this for populateUser, can we have one for restrict-to-self as well? :)

+1 for supporting false for both successRedirect and failureRedirect

With the current version, 0.4.1 Local auth won't give any redirect errors if you specify an Accepts:application/json header.

I have the same issue. I think we only needs to disable redirection for local auth, because if if want to use both local auth and facebook/github/.. we stil need successRedirect and failureRedirect.

I think @ArnaudValensi is right. I started looking at this today. Rather than disable for local auth altogether, I think the best option is to allow people to specify which auth providers they want to use a  redirect for success and failure.
The reason being that if you are doing a form post for local auth then it should redirect to your own route.


chore: pt/de/fr/es app translations

Description
Weekly translation sync
Powered by Globalization WS
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

updating branch ...


Add language identifiers

Added some missing language identifiers.
Fixed some bad indentations.
Fixed some code that had Throw instead of throw and Else instead of else which will cause compile error in C#.

Related to #2192
edit by @mairaw: added the related issue

Resolved some merge conflicts after merging someone else's PR

Fixed other merge conflicts that occurred again.

There were more merge conflicts since then. Can you please check @Youssef1313?

@mairaw, Fixed them.

Thanks for making these updates @Youssef1313
I'm approving this PR.
@mairaw when you are happy with the changes, this is ready to :shipit:

Fixed merge conflicts again.


https://github.com/erickhansx/capstone-ruby/projects/1

The team is comprised of 3 members.
Erick Hans will handle issues labeled A.
Ndorrh Oswald Ebu issues labeled B
Akpuru Solomon Barine issues labeled C
kanban board Link

STATUS: CHANGES REQUIRED ♻♻
Hello team 👋🏽
Good job working on the project so far!
To highlight 👍🏽
✔ You gave details about each team member
✔ You added your project link
✔ You created a board
However, there are some issues that you need to work on to go to the next project but you are almost there!

Required Changes ♻

[ ] Kindly ensure that cards for group tasks are present on your kanban board and assigned to all members of the group.
[ ] As this is a part of the requirements, kindly ensure that each team member's tasks are shared according to this divided requirements
[ ] Some cards on your board for student c do not have an estimated time, please add this.
[ ] Please ensure that the sum of the number of points for each student is equal, that is, each student should have equal time to work on their parts of the project.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.

I think you should give this issue a more descriptive name because this would make your project more professional.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you for your kind reply @Ghiftee I will make all the corrections needed right now!

Approved ✔
Hi Team,
Great job, Your project is complete!  There is nothing else to say other than... it's time to close this issue and start working on the project :shipit:
GOOD LUCK 🤞

(Changes Required) Aspects to improve: ♻

N/A

[Optional] suggestions:

N/A

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
**Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear, and please remember to tag me in your question so I can receive the notification**
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thanks, @Omar-Muhamad for the prompt response!


Updating an ObservableCollection bound to a MenuItemsSource crashes with ArgumentOutOfRangeException

Describe the bug
When a NavigationView uses MenuItemsSource to bind to an ObservableCollection, updates to the ObservableCollection will often cause the app to crash with an System.ArgumentOutOfRangeException: 'This collection cannot work with indices larger than Int32.MaxValue - 1 (0x7FFFFFFF - 1). (Parameter 'index')'.
These issues look similar - I expect the root cause is the same/similar:

https://github.com/microsoft/microsoft-ui-xaml/issues/8684
https://social.msdn.microsoft.com/Forums/en-US/d0d6709e-2da3-450a-8212-f1b07cf9fd51/uwpargumentoutofrangeexception-when-clearing-collection?forum=wpdevelop
https://github.com/microsoft/microsoft-ui-xaml/issues/2818
Here the source is bound to an array instead of an ObservableCollection, but it still crashes (sometimes).
Note that https://github.com/microsoft/microsoft-ui-xaml/pull/3138 claims to fix this bug, but it's not completely fixed. I still see crashes when updating the repro app (attached) to use an array instead of an ObservableCollection.

Fixing this is important to MAUI, given that MAUI shell uses NavigationVIew with MenuItemsSource. Here's the bug we're using to track this issue on the MAUI side: https://github.com/dotnet/maui/issues/16929
Steps to reproduce the bug

Open the repro project, here
NavigationViewUpdateCrash.zip
Build and run it. The app has a NavigationView with a MenuItemsSource set to a simple ObservableCollection, with 7 items.
Select one of the navigation items on the left. The NavigationView_ItemInvoked code adds an item to the ObservableCollection, then removes the first item.

Result:
App crashes with System.ArgumentOutOfRangeException: 'This collection cannot work with indices larger than Int32.MaxValue - 1 (0x7FFFFFFF - 1). (Parameter 'index')'. The full call stack is here: https://gist.github.com/BretJohnson/82f2c831a1a7ed2ddf1d74bb5710c717
Looking at what's happening, the CollectionChanged is triggered by the Add (which works fine) and then again with the Remove (which crashes). The CollectionChanged triggers updating the layout, which then appears to reference some old data in the collection when computing the new layout.
Note that the exception message is also wrong and should really be fixed too - the offending index here (per the debugger) isn't larger than Int32.MaxValue - it's 7, which happens to be one bigger than the current max index of 6. Something in the layout code is referencing an index past the end, which used to be valid but no longer is after the remove.
I tried various workarounds. Doing the remove first then the add doesn't crash, but it's hard to generalize for all the MAUI scenarios that need it (since sometimes the Add and Remove are far removed). Switching from ObservableCollection to just using an array or List and updating MenuItemsSource when it changes didn't work either - that also crashes on updates in WinUI runtime code; the symptoms are a bit different the root cause of that crash looks similar.
Expected behavior
It shouldn't crash.
Screenshots

NuGet package version
None
Windows version
No response
Additional context
No response

Huh, MSFT:33651693 has an internal report of a crash in MeasureOverride in an ItemsRepeater that we were pretty sure was related to a nav view.
MSFT:34550937 mentioned something similar, but I'm not sure that's totally relevant. The notes in our bug (34550937) mentioned something about the backing collection changing while a layout was in progress - I'm not sure that's totally relevant though. We were using a MenuItems and switching to a MenuItemSource fixed it for us^1. The error text though - "The operation attempted to access data outside the valid range" seems closer to what you're seeing here.

https://github.com/dotnet/maui/issues/16324 looks like another variant - another issue with ObservableCollection based bindings acting up during the layout pass. That bug produces a "Child collection must not be modified during measure or arrange." exception, like @zadjii-msft reported happens for their Terminal MenuItems bug https://github.com/microsoft/microsoft-ui-xaml/issues/8770#issuecomment-1695825256

@llongley If this issue is completed, how is the fix being made available?   I see no references in the sidebar to a release containing the solution.

@BretJohnson have you ever been able to fine an open issue that tracks the root cause of this issue in any publicly accessible repo?

https://github.com/microsoft/microsoft-ui-xaml/issues/8684


Fixes the windows executable path for standalone mechanical

Fixes the typos in the documentation for launching standalone mechanical
https://github.com/pyansys/pymechanical/issues/212

@MaxJPRey @RobPasMue are you familiar with documentation build CI failures that look like this? It happened twice in a row:
Get:55 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 cups-bsd amd64 2.4.1op1-1ubuntu4.1 [37.0 kB]
Err:8 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 libgs9-common all 9.55.0~dfsg1-0ubuntu5.1
404  Not Found [IP: <IP_ADDRESS> 80]
Err:12 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 libgs9 amd64 9.55.0~dfsg1-0ubuntu5.1
404  Not Found [IP: <IP_ADDRESS> 80]
Err:19 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 ghostscript amd64 9.55.0~dfsg1-0ubuntu5.1
404  Not Found [IP: <IP_ADDRESS> 80]
Err:20 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 ghostscript-x amd64 9.55.0~dfsg1-0ubuntu5.1
404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/libgs9-common_9.55.0%7edfsg1-0ubuntu5.1_all.deb  404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/libgs9_9.55.0%7edfsg1-0ubuntu5.1_amd64.deb  404  Not Found [IP: <IP_ADDRESS> 80]
Fetched 161 MB in 22s (7357 kB/s)
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/ghostscript_9.55.0%7edfsg1-0ubuntu5.1_amd64.deb  404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/ghostscript-x_9.55.0%7edfsg1-0ubuntu5.1_amd64.deb  404  Not Found [IP: <IP_ADDRESS> 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

@MaxJPRey @RobPasMue are you familiar with documentation build CI failures that look like this? It happened twice in a row:
Get:55 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 cups-bsd amd64 2.4.1op1-1ubuntu4.1 [37.0 kB]
Err:8 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 libgs9-common all 9.55.0~dfsg1-0ubuntu5.1
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:12 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 libgs9 amd64 9.55.0~dfsg1-0ubuntu5.1
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:19 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 ghostscript amd64 9.55.0~dfsg1-0ubuntu5.1
  404  Not Found [IP: <IP_ADDRESS> 80]
Err:20 http://azure.archive.ubuntu.com/ubuntu jammy-updates/main amd64 ghostscript-x amd64 9.55.0~dfsg1-0ubuntu5.1
  404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch [http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/libgs9-common_9.55.0%7edfsg1-0ubuntu5.1_all.deb](http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/libgs9-common_9.55.0~dfsg1-0ubuntu5.1_all.deb)  404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch [http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/libgs9_9.55.0%7edfsg1-0ubuntu5.1_amd64.deb](http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/libgs9_9.55.0~dfsg1-0ubuntu5.1_amd64.deb)  404  Not Found [IP: <IP_ADDRESS> 80]
Fetched 161 MB in 22s (7357 kB/s)
E: Failed to fetch [http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/ghostscript_9.55.0%7edfsg1-0ubuntu5.1_amd64.deb](http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/ghostscript_9.55.0~dfsg1-0ubuntu5.1_amd64.deb)  404  Not Found [IP: <IP_ADDRESS> 80]
E: Failed to fetch [http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/ghostscript-x_9.55.0%7edfsg1-0ubuntu5.1_amd64.deb](http://azure.archive.ubuntu.com/ubuntu/pool/main/g/ghostscript/ghostscript-x_9.55.0~dfsg1-0ubuntu5.1_amd64.deb)  404  Not Found [IP: <IP_ADDRESS> 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?

Hi @koubaa! Yes, we've reported internally this failure. Looks like a transient error - we get these from time to time from the Azure Ubuntu archives. Let's hope it gets solved soon. Many other repos are suffering from it.

Error has been tracked here: https://github.com/actions/runner-images/issues/7452

@RobPasMue it's still failing on multiple reruns. Is it really a transient error?

@RobPasMue it's still failing on multiple reruns. Is it really a transient error?

Not transient as "sometimes works, sometimes doesn't"... it's more like the runners are having issues in their latest update. I can do a quick fix for now, and I'll rvert it as soon as its fixed

@rajesh1359 thank you for the contribution!


Nuget me up!

This package is awesome - some really goo ideas, fancy throwing it up on to nuget?

@Jetski5822 thanks appreciate it!
Regarding the nuget, maybe this might be a new project for Polly.Contrib. We might have several nuget packages to handle different resilience strategies in a generic way, such as Azure SQL Databases, Azure Service Bus, etc. Also, we would need to make them extensible in order to people can inject additionals strategies to the builder (and a lot of enhancements as well, I didn't write this guy thinking about a nuget tho 😅 ), etc what do you think @reisenberger?

@vany0114 I'll dig in and have a look!

Hi @vany0114 . It is great to see everything you are doing around Polly! I dug into the code (and blog post), but quickly, so, please, feel free to say if I have misunderstood anything.
It looks broadly as if this project does three different kinds of thing around Polly:
(1) Specific helper methods around SQL errors (and suggestion we could do similar for eg Azure Service Bus; other services).
(2) New builder syntax for the policies / policywrap
(3) some combining of the sync and async configuration syntax.

If we wanted to take some of this into a/ Polly-Contrib project/s, it might be better (and cleaner for users to understand) to tackle some of these ^ concerns separately:
(1) Specific helper methods around SQL errors (or similar for other downstream components)
These could be great additions to Polly-Contrib. A similar concept to Polly.Extensions.Http, which contains "off-the-shelf" configuration for handling standard http-call errors.  Polly.Extensions.Sql or Polly.Extensions.AzureServiceBus (etc) could make great contrib projects.  They might (like Polly.Extensions.Http) be limited just to the Handle-clause parts of the syntax, unless I have missed some other angle.
(2) New builder syntax for the policies
I like the new builder syntax, but I have different goals / think we have bigger challenges for a syntax overhaul for Polly.  The biggest challenge I see is the proliferation of overloads - for example the large number of Retry policy overload options - the fact that there are overloads for many different possible combinations of parameters.  This design (which I inherited on the project) is preventing Polly growth. I would like to change the Polly policy configuration syntax (major breaking change!) to something like Policy.Handle<>(/* etc */).Retry(Action<RetryConfiguration>) ... in other words, similar to much of the .NET Core configuration syntax.  This Action<RetryConfiguration> pattern is much more open for extension.
I see this ^ as the most pressing syntax problem to solve for Polly.
(3) some combining of the sync and async configuration syntax
Despite looking hard at whether we could merge sync/async policies, the fact that users want and need async delegate hooks for async policies (and if we don't supply this, it leaves users to fall into the pit of failure with those nasty async void problems) ... means probably we will keep the sync/async split in Polly.
(Aside: If an approach such as .Retry(Action<RetryConfiguration>) discussed above was adopted, it might be possible to achieve some commonality of code by having SyncRetryConfiguration : RetryConfiguration and also AsyncRetryConfiguration : RetryConfiguration ....)
(I might have misunderstood what you have done around sync/async tho.)

Please don't be discouraged if you want to nuget this; I am just trying to answer the question which you and @Jetski5822 raised 🙂 , and hopefully provide some useful input!
No obligation also to follow any of these suggestions ...
Thanks.

First things first, @reisenberger thanks for taking the time to dig into the code/blog post I really appreciate it!
Well, the idea behind this was to have a common way to build and manage different resilience strategies thru Polly. I needed to handle Azure SQL databases errors, and most of the times I was using the same combinations of policies (that's why the builder has a method called WithDefaultPolicies). So, the main idea was to build specialize resilience strategies using Polly since every strategy will depend on the component you want to handle, it's not the same, a resilient strategy for Azure SQL databases than for Azure Service Bus.
Regarding the syntax, I just thought in a friendly, flexible, intuitive and easy way to use and build the strategy (like Polly does) but as I said earlier, I wasn't thinking to make a library or something like that, I was just encouraging people to think in that way then they could make their specialized strategies given their use cases, so I just purposed a cool way to do that thru a builder.
Regarding combining of the sync and async configuration syntax, actually, I have two separate implementations for both, sync and async policies that I use thru UseAsyncExecutor or UseSyncExecutor, like this:
var resilientAsyncStrategy = builder
    .UseAsyncExecutor()
    .WithDefaultPolicies()
    .Build();

But it really doesn't matter, as I said, that syntax was only an example of one approach I wanted to show.
Wrapping up, the main idea here is that would be nice having specialized (and common) resilience strategies for a different kind of components, such as Azure SQL databases, Azure Service Bus, etc like you did with Polly.Extensions.Http. I really like that idea because it's just matter of download the nuget and set it up and that's it, you don't have to build that strategy over and over again. Having said that I'd really like to do something like you said earlier:

Polly.Extensions.Sql or Polly.Extensions.AzureServiceBus (etc) could make great contrib projects. They might (like Polly.Extensions.Http) be limited just to the Handle-clause parts of the syntax.

^ that's the main idea behind this.
So, please, let me know your thoughts and if you feel we can make this I'll be more than happy to do it after we release Simmy!
Thanks again.

Thanks @vany0114 . It sounds like we have a similar perspective.  Also, I should be clear: this project is a great example of how people can build their own thing around Polly to suit their needs! And perfect to put all in one project for that.  When I was saying "to tackle these concerns separately", that was only in the context of @Jetski5822 's question "Should we nuget this up?" (and perhaps in Polly-Contrib). For nuget release in Polly-Contrib, I would tackle the aspects separately.
Thanks again for all the awesome things you are doing around Polly! 👍 🙏

So @reisenberger. let me know if you'd like to go further with these ideas on Polly-Contrib 😃

I currently pulled the whole project in locally, and its working great. Really loving it, if we could get this in to polly contrib is some manner, that would be great!


Add docker setups

Intro
This repo will hold docker-compose.yml files to run Sublinks for both demo and local dev purposes (eventually, production/live too). This PR is the first (major) step towards that goal.
CURRENT STATUS
Please note, this repo currently holds configuration files intended for development purposes only. As we get closer to our initial MVP release we will provide sample files for use in a live site.
Demo Site

Run docker-compose -f docker-compose.demo.yml up [-d]

Frontend Dev

Run docker-compose -f docker-compose.frontend.yml up [-d]

NOTE: This needs to be run from a subdirectory of the frontend repo. A separate PR will be opened to replace the current etc/dev directory with a docker directory containing this repo as a submodule

Things look good to me after a first pass. Although I can't say I'm familiar with most things outside the frontend setup. Which did get going and was accessible for me after pointing the app volume to my local frontend root.

Cool. I'm targeting frontend repo first anyway so was hoping it would all be working. I'm intending on introducing a "docker" folder into each repo as a submodule.  That way the instructions would be more or less the same for each repo (eg, cd docker && docker-compose -f docker-compose.<area>.yml up)

Only thing for now is that you definitely want to ignore node_modules/ for the seeder. That's about 600 files that don't need to be committed.

Good catch. I just pushed a commit removing it

Moving to ready for review so I can target integration into frontend repo. I will open new PRs for backend and federation configs

I'm fixing that in the individual repos (to build and push multi-arch images). Already have a PR open in federation


go.dev: Design Changes Harm Readability

(I scanned through issues with go.dev: and didn't see anything like this, forgive me if I missed any.)
What is the URL of the page with the issue?
https://pkg.go.dev/github.com/CloudyKit/jet?tab=doc
What is your user agent?
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.122 Safari/537.36
Screenshot

What did you do?
Visit the page
What did you expect to see?

What did you see instead?
A variety of style changes, mainly font-related, that in my opinion (stated as such) are not improvements, and which make the documentation harder to read.

The existing font stack for godoc gives me Helvetica Neue for most text, and Menlo for the code/monoscape stuff (I am on a Mac).  To me, Roboto and Source Code Pro are not as readable, and are a jarring change.  Work Sans is actually not bad and possibly an improvement for where it's used.
In the Index up top, (shown in screenshot) the fonts are bigger (~4px?) which fits less on the screen, makes all of the text seem higher up the importance hierarchy, and goes counter to its purpose as a summary/definition-overview of the interface of the package.
In the Index, we lost the bullet points next to the methods.  They are "plain", sure, but they made the Index more structured, made it easier to separate the levels when you scan through with your eye.
The content width got significantly larger, from 698px in my browser to 960.  In my opinion this is too wide to be easily readable.  Compare to these GitHub Issues at 639, Google's homepage snippets at 600, Medium at 680, SwiftDoc.org at 847, even the fairly wide GitHub readmes at 882!  I get that these exact numbers are subjective and probably related to my screen size and various other factors, but the point is that it's a lot bigger, and the increased width hurts readability.
In the main body of the docs, the methods are the same size as the types.  In existing godoc, types are 24px and methods are 18px, so when you're scanning through it's easy to visually separate them.  In pkg.go.dev they're both 24px.
Fonts are generally just larger, so between that and the content size increase the page feels like it's shouting at me in comparison to the old one.

Suggestions
Again, these are all my opinions, but since you are asking for feedback, I would suggest:

Please return the font stack to use Helvetica Neue and Menlo where Godoc used it.
Please reduce the content width closer to the existing Godoc; if it needs to increase a little I understand, but even ~800px would be much better than 960.  If you need to keep the overall content width because of other panes in the tab bar, some horizontal padding on the doc pane would work fine.
Please return the bullet points next to methods in the Index.
Please reduce font sizes, perhaps in general, but specifically the Index font size and the sizes of method signatures in the main body of documentation.  A general review to ensure that the typography is communicating the hierarchy of data might not be a bad idea.

In closing, godoc.org is fantastic!  I would suggest we slavishly copy it unless the Go Team is getting a lot of requests to change it (which you might, I don't know).

/cc @fflewddur @spf13

I am having the same problem. I always try to skip the new site and now I am afraid it will replace the documentation site.
Note the comparison below have a zoom on the new website of 80% instead of 100% to at least have some margin on the sides. (With 100%)

I am not a design expert so it is hard for me to know what the possible solutions are. Here are some of the notes I believe makes my experience worse in the new website:

Title was easier to read and the biggest text on the site. Also it have margins around.
In the previous there was one overview. In the new there are two Overview sections and another two overview icons. This also by not being in the overview sections. I don't understand what is this and how all this overviews are related.
Text was easier to read. In my screen the default paragraph ended being render in something like 50-70 characters per line. The new one have larger lines. Maybe 100 characters per line.  Most of the documentation I read is with a more fixed width.
The new design feels more aggressive. I spent hours looking at this docs. The previous one was softer in my eyes.
I default the new website to 80% zoom in the browser. Does reduce my need of scrolling.
Before all the doc was in a single document. So Ctrl+F and search across all package documentation was useful. For example in net/http I was able to find a httptest library subfolder. Right now the documentation feels split, meaning that I might need to click double the times.
The github url is repeated twice now. Both around the name of the package. The previous one had the import line, that was useful some times to copy and paste.
Before things like when was published, the license, version was info collected from github. Right now it is nice to have so much info here. But it does not show neither the stars, forks, or issues. So I need to go to github any way, making all this new info for me useless.
The documentation site feels like the rest of the site. It is hard to distinguish from the rest. Before the documentation was easily identifiable.
This is more about author UX. Before I was able to test documentation site by running my own godoc server and see if everything was ok. Right now I can still use the old one, but once that is remove... If I have some mistake in the doc: do I need to release a new version without even previewing?

Hope it helps.

A preview of the redesign is now available on beta.pkg.go.dev. Give it a try, and feel free to file a new issue with any other suggestions!
Closing this issue for https://github.com/golang/go/issues/41586.


Feature request: Add custom layouts with SVG background

Will be very usefull to add SVGLinearLayout, SVGRelativeLayout etc. with
opportunity to set background as svg file according to SVGImageView.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 14 Aug 2013 at 9:47

Inherited issue, not relevant.


Fix Topic voting

Current behaviour:
votes in a topic cannot be started.
The start button is missing
Wanted behaviour:
The voting should be working

Bug is not appearing anymore


Pip packages install variable

Description of PR
This PR adds the zabbix_install_pip_packages so that we can disable the installation of pip packages when executing the role.
This is especially useful when you have a tight control on the environment you execute the role in.
Type of change
Feature Pull Request
Let me know if you think this is really out of scope.

Hello @dj-wasabi
Not sure if you've seen this PR :)
Does it make sense to you?
If it doesn't I'll keep a fork with that change for me :)
Thanks!

ah, sorry! My bad. Thanks! 👍

ah, sorry! My bad. Thanks!

Thanks @dj-wasabi, will you do a minor release on Ansible Galaxy in the coming days?

Hello @dj-wasabi
sorry for the noise, but will you make a release on Ansible Galaxy soon?
Otherwise, I'll just git clone the master of this repo for my project :)
Thanks,

Ah, will create one today! I totally forgot it again, my bad. I've created/sent some reminders so I won't forget it. :)

@rnsc
I uploaded 1.6.1 to the Galaxy. Sorry for late reply/action, but nevertheless have fun! 😄

@dj-wasabi cool, thanks!


Filepond uploader disappears after deleting image

Hello,
I have an issue I couldn't solve so far.
When I delete an image that is first in a row, the whole Filepond uploader disappears.
If I have a look at source code, everything seems to be hided behind something but cannot figure out what it could be.
What am I doing wrong here?!
https://user-images.githubusercontent.com/39381278/105557165-d4bb4c00-5d0b-11eb-82df-bf55e49541a7.mov
Thanks for your help.
Bruno.

We require the issue template to be filled out on all new issues. The issue template helps us collect all the information we need to address your submission efficiently.
The more information we have, the faster the issue can be resolved.
We will be able to more closely look at your issue once the issue template has been filled out. The issue will be closed until the above is addressed. Please leave a comment after you've updated the issue.

We require the issue template to be filled out on all new issues. The issue template helps us collect all the information we need to address your submission efficiently.
The more information we have, the faster the issue can be resolved.
We will be able to more closely look at your issue once the issue template has been filled out. The issue will be closed until the above is addressed. Please leave a comment after you've updated the issue.