skimage image_fetcher is None

🐛 Bug
I'm getting tests failing locally because from skimage.data import image_fetcher returns None and not an ImportError, so this catch does not get activated, see here https://github.com/napari/napari/blob/master/napari/conftest.py#L20-L35
Error are things like
__________________________________________________________________________________ ERROR at setup of test_single_filename __________________________________________________________________________________

    @pytest.fixture
    def single_tiff():
>       return [image_fetcher.fetch('data/multipage.tif')]
E       AttributeError: 'NoneType' object has no attribute 'fetch'

napari/conftest.py:294: AttributeError

I know there was recent stuff with skimage, Pil, tifffile etc, so i tried updating everything, but it still didn't work for me
pip freeze gives me:

(base) czirwc1macos2701:napari nsofroniew$ pip freeze
absl-py==0.9.0
aiobotocore==0.11.1
aiohttp==3.6.2
alabaster==0.7.12
anaconda-client==1.7.2
anaconda-navigator==1.9.7
anaconda-project==0.8.3
appdirs==1.4.3
appnope==0.1.0
appscript==1.0.1
arrow==0.15.5
asciitree==0.3.3
asn1crypto==1.3.0
aspy.yaml==1.3.0
astor==0.8.1
astroid==2.3.3
astropy==4.0
async-generator==1.10
async-timeout==3.0.1
atomicwrites==1.3.0
attrs==19.3.0
av==7.0.1
Babel==2.8.0
backcall==0.1.0
backports.functools-lru-cache==1.6.1
backports.os==0.1.1
backports.shutil-get-terminal-size==1.0.0
backports.tempfile==1.0
backports.weakref==1.0.post1
-e git+https://github.com/hci-unihd/batchlib@ae87a9295ee7e1bea9d6a941ce08b56f5f18d262#egg=batchlib
beautifulsoup4==4.8.2
binaryornot==0.4.4
biopython==1.76
bitarray==1.2.1
bkcharts==0.2
black==19.10b0
bleach==3.1.0
bokeh==2.0.0
boto==2.49.0
boto3==1.10.48
botocore==1.13.14
Bottleneck==1.3.2
cachetools==4.0.0
cachey==0.2.1
certifi==2019.11.28
cffi==1.14.0
cfgv==2.0.1
cftime==1.1.3
chardet==3.0.4
Click==7.0
cloudpickle==1.3.0
clyent==1.2.2
colorama==0.4.3
colored==1.4.2
conda==4.8.2
conda-build==3.18.9
conda-package-handling==1.6.0
conda-verify==3.4.2
contextlib2==0.6.0.post1
cookiecutter==1.7.0
coverage==5.0.2
cryptography==2.8
csbdeep==0.5.1
cycler==0.10.0
Cython==0.29.15
cytoolz==0.10.1
dask==2.15.0
dask-cloudprovider==0.1.1
dask-image==0.2.0
decorator==4.4.2
deeplabcut==<IP_ADDRESS>
defusedxml==0.6.0
Deprecated==1.2.7
distributed==2.12.0
docutils==0.15.2
easydict==1.9
entrypoints==0.3
et-xmlfile==1.0.1
fastcache==1.1.0
fasteners==0.15
filelock==3.0.12
flake8==3.7.9
Flask==1.1.1
freetype-py==2.1.0.post1
fsspec==0.6.2
future==0.18.2
gast==0.2.2
gevent==1.4.0
glob2==0.7
gmpy2==2.0.8
google-auth==1.11.2
google-auth-oauthlib==0.4.1
google-pasta==0.1.8
greenlet==0.4.15
grpcio==1.27.2
h5py==2.10.0
HeapDict==1.0.1
html5lib==1.0.1
hypothesis==5.5.4
identify==1.4.8
idna==2.8
imagecodecs==2020.2.18
imagecodecs-lite==2019.12.3
imageio==2.8.0
imageio-ffmpeg==0.3.0
imagesize==1.2.0
imgaug==0.4.0
importlib-metadata==1.5.0
intel-openmp==2019.0
ipdb==0.13.2
ipykernel==5.1.4
ipython==7.13.0
ipython-genutils==0.2.0
ipywidgets==7.5.1
isort==4.3.21
itsdangerous==1.1.0
jdcal==1.4.1
jedi==0.16.0
Jinja2==2.11.1
jinja2-time==0.2.0
jmespath==0.9.4
joblib==0.14.1
json5==0.9.1
jsonschema==3.2.0
jupyter==1.0.0
jupyter-client==5.3.4
jupyter-console==6.1.0
jupyter-core==4.6.3
jupyterlab==1.2.6
jupyterlab-server==1.0.6
Keras==2.3.1
Keras-Applications==1.0.8
Keras-Preprocessing==1.1.0
keyring==21.1.0
kiwisolver==1.1.0
lazy-object-proxy==1.4.3
libarchive-c==2.8
lief==0.9.0
llvmlite==0.31.0
locket==0.2.0
lxml==4.5.0
magicgui==0.1.3
Markdown==3.2.1
MarkupSafe==1.1.1
matplotlib==3.1.3
mccabe==0.6.1
meshio==4.0.13
mistune==0.8.4
mkl-fft==1.0.15
mkl-random==1.1.0
mkl-service==2.3.0
mock==4.0.1
monotonic==1.5
more-itertools==8.2.0
moviepy==1.0.1
mpmath==1.1.0
mrc==0.1.5
msgpack==0.6.1
msgpack-numpy==<IP_ADDRESS>
multidict==4.7.3
multipledispatch==0.6.0
-e git+https://github.com/sofroniewn/napari@c4cff2daec4ac64d7c2e4204db7e31d3fb9d272e#egg=napari
napari-classification==0.0.0
napari-denoising==0.0.0
napari-detection==0.0.0
napari-dv==0.2.2
napari-io==0.0.0
-e git+https://github.com/sofroniewn/napari-io-test.git@221a4e5c763281a2d3c9e56ebd66b6e12564561c#egg=napari_io_test
napari-plugin-engine==0.1.5
napari-plugins==0.1.0
napari-registration==0.0.0
napari-segmentation==0.0.1
-e git+https://github.com/sofroniewn/napari-svg.git@b6a641b16ccdf11fd994db6315cab544e37de1c1#egg=napari_svg
napari-tracking==0.0.0
naparimovie==0.1
navigator-updater==0.2.1
nbconvert==5.6.1
nbformat==5.0.4
netCDF4==1.5.3
networkx==2.4
nibabel==3.0.0
nilearn==0.6.0
nltk==3.4.5
nodeenv==1.3.3
nose==1.3.7
notebook==6.0.3
numba==0.48.0
numcodecs==0.6.3
numexpr==2.7.1
numpy==1.18.1
numpydoc==0.9.2
oauthlib==3.1.0
olefile==0.46
ome-zarr==0.0.7
omero-napari==0.0.3
omero-py==5.6.2
opencv-contrib-python-headless==<IP_ADDRESS>
openpyxl==3.0.3
opt-einsum==3.1.0
packaging==20.3
pandas==1.0.3
pandocfilters==1.4.2
parso==0.6.1
partd==1.1.0
path==13.1.0
pathlib2==2.3.5
pathspec==0.6.0
patsy==0.5.1
pdfkit==0.6.1
pep8==1.7.1
pexpect==4.8.0
pickleshare==0.7.5
Pillow==7.1.2
-e git+https://github.com/soft-matter/pims@298a8cba5fe4d59d3a08b530379d760ff1088e9b#egg=PIMS
pkginfo==<IP_ADDRESS>
pluggy==0.13.1
ply==3.11
poppy==0.9.0
poyo==0.5.0
pre-commit==2.1.1
ProDy==1.10.11
proglog==0.1.9
prometheus-client==0.7.1
prompt-toolkit==3.0.3
protobuf==3.11.3
psutil==5.7.0
ptyprocess==0.6.0
py==1.8.1
pyasn1==0.4.8
pyasn1-modules==0.2.8
pycodestyle==2.5.0
pycosat==0.6.3
pycparser==2.19
pycrypto==2.6.1
pycurl==<IP_ADDRESS>
pyflakes==2.1.1
PyGithub==1.44.1
Pygments==2.5.2
PyJWT==1.7.1
pylint==2.4.4
pyodbc===4.0.0-unsupported
PyOpenGL==3.1.5
pyOpenSSL==19.1.0
pyparsing==2.4.6
PyPDF2==1.26.0
PyQt5==5.13.2
PyQt5-sip==12.7.0
pyquaternion==0.9.5
pyrsistent==0.15.7
PySide2==5.14.1
PySocks==1.7.1
pytest==5.3.5
pytest-arraydiff==0.3
pytest-astropy==0.8.0
pytest-astropy-header==0.1.2
pytest-cov==2.8.1
pytest-doctestplus==0.5.0
pytest-faulthandler==2.0.1
pytest-openfiles==0.4.0
pytest-ordering==0.6
pytest-qt==3.3.0
pytest-remotedata==0.3.2
pytest-timeout==1.3.4
python-dateutil==2.8.0
pytz==2019.3
PyWavelets==1.1.1
PyYAML==5.3
pyzmq==18.1.1
QtAwesome==0.6.1
qtconsole==4.7.1
QtPy==1.9.0
readme-renderer==24.0
regex==2019.12.9
remotezip==0.9.2
reportlab==3.5.34
requests==2.23.0
requests-oauthlib==1.3.0
requests-toolbelt==0.9.1
rope==0.16.0
rsa==4.0
ruamel-yaml==0.15.87
ruamel.yaml.clib==0.2.0
s3fs==0.4.2
s3transfer==0.2.1
scikit-image==0.17.2
scikit-learn==0.22.1
scipy==1.4.1
-e git+https://github.com/sofroniewn/scorecard@cb4ec6f356c69e21932e9d42f6e32cac025ef42f#egg=scorecard
seaborn==0.10.0
-e git+https://github.com/sofroniewn/segmentify@bd57cfcc94ad2f6dfcb080ae786f410e044659c4#egg=segmentify
Send2Trash==1.5.0
Shapely==1.7.0
shiboken2==5.14.1
simplegeneric==0.8.1
singledispatch==<IP_ADDRESS>
six==1.14.0
sklearn==0.0
slicerator==1.0.0
snowballstemmer==2.0.0
sortedcollections==1.1.2
sortedcontainers==2.1.0
soupsieve==1.9.5
Sphinx==2.4.0
sphinx-rtd-theme==0.4.3
sphinxcontrib-applehelp==1.0.1
sphinxcontrib-devhelp==1.0.1
sphinxcontrib-htmlhelp==1.0.2
sphinxcontrib-jsmath==1.0.1
sphinxcontrib-qthelp==1.0.2
sphinxcontrib-serializinghtml==1.1.3
sphinxcontrib-websupport==1.2.0
spyder==3.3.6
spyder-kernels==0.5.2
SQLAlchemy==1.3.13
stardist==0.5.0
statsmodels==0.11.0
sympy==1.5.1
tables==3.6.1
tabulate==0.8.6
tblib==1.6.0
tensorboard==1.15.0
tensorflow==1.15.0
tensorflow-estimator==1.15.1
tensorpack==0.9.8
termcolor==1.1.0
terminado==0.8.3
testpath==0.4.4
tifffile==2020.5.25
tiffile==2018.10.18
toml==0.10.0
toolz==0.10.0
torch==1.5.0
torchvision==0.6.0
tornado==6.0.4
tqdm==4.42.1
traitlets==4.3.3
twine==3.1.1
typed-ast==1.4.0
typing-extensions==<IP_ADDRESS>
unicodecsv==0.14.1
urllib3==1.25.8
virtualenv==16.7.9
-e git+https://github.com/sofroniewn/vispy@9e0822db0ec31ba720d82952b1867e77f30971f6#egg=vispy
wcwidth==0.1.8
webencodings==0.5.1
Werkzeug==1.0.0
whichcraft==0.6.1
widgetsnbextension==3.5.1
wrapt==1.12.0
wurlitzer==2.0.0
xarray==0.15.0
xhtml2pdf==0.2.4
xlrd==1.2.0
XlsxWriter==1.2.7
xlwings==0.17.1
xlwt==1.3.0
yarl==1.4.2
zarr==2.3.2
zeroc-ice==3.6.5
zict==1.0.0
zipp==2.2.0

note i know i have a lot of stuff in there ;-)

I hit this. I had to pip install pooch to make it work. But then I see pooch is listed in requirements/test.txt so I think the issue is just that the test requirements were not installed?
It's a minor bit of polish, but it would be neat if you got some very definitive error mess like "test requirements are not installed". This is second time I've been confused by a "weird" error as a result of not having the test requirements install. Both my fault but still.

Also I wonder if this code is wrong:
https://github.com/napari/napari/blob/master/napari/conftest.py#L20-L24
It seems like it's trying to say "if fetch_image isn't available" and then it has its own version. But when pooch isn't installed fetch_image exists but it's None.

Yeah, I upgraded both pooch and scikit-image and then everything works fine.
I guess in the none fetch_image case we were trying to create one below https://github.com/napari/napari/blob/master/napari/conftest.py#L26-L35 but I'm not sure why that didn't work. Maybe @jni  or @tlambert03 have ideas, otherwise we can think about closing this

That code was intended to work both with earlier versions of scikit-image in which case there is no skimage.data.image_fetcher and the fallback works fine... or with newer scikit-image, provided that pooch is installed (hence pooch is in our requirements/test.txt).  It's true, that it won't work if you don't have pooch installed... but that would also mean that you haven't run pip install -r requirements/test.txt ... in which case, a lot of our tests would fail right?
while those requirements would be installed when a developer installs following the guidelines in our docs, it's always possible that a developer might have an older un-updated environment.  So since two of you ran into this, if you wanted to here, you could maybe add a try import pooch, and print a "update your tests requirements" message if it fails?

Thanks @tlambert03, makes sense! I think this is one of those things that probably effected us the most as there was this recent transition, but as new developers come they won't run into this. I think it's ok to close, but if someone else wants to put that try/ message in they can.


Support auto track definitions

Auto tracks are generated automatically when the placement algorithm places an item outside the range of the templated tracks. This could be useful in some cases.

Which type of track would you pick, though? The relatively heavy IntrinsicContentTrack? Or a FlexibleTrackSizeTrack(1)? Not sure if we can come up with an intuitive default.
Alternatively, in case of 'missing' tracks, we could not crash/error but instead show overflow bars (on debug builds) and otherwise ignore this content?

Which type of track would you pick, though? The relatively heavy IntrinsicContentTrack? Or a FlexibleTrackSizeTrack(1)? Not sure if we can come up with an intuitive default.

No defaults! We let the user decide.
https://developer.mozilla.org/en-US/docs/Web/CSS/grid-auto-rows

Alternatively, in case of 'missing' tracks, we could not crash/error but instead show overflow bars (on debug builds) and otherwise ignore this content?

Agreed that a crash is a little harsh. Let me think about it.

I think this should actually be quite straightforward to implement. All you'd need to do is resize to fit the item your placement grid instead of throwing an error. I have an implementation of this for my CSS Grid implementation here. As you can see, it's quite short and straightforward. And mine is actually quite a bit more complex than I think you'd need as it needs to support negative indexes (adding negative implicit tracks is a massive pain as it means the origin of grid moves!)
For performance reasons I also have some logic to estimate the required size ahead of time (by scanning through the grid placements and spans of every item and finding the min/max - you'd only need the max). The logic for that is here. This doesn't completely guarantee no resizes as auto-placement can still generate more tracks but it helps in the "explicitly placed out of bounds" case.


[Session] Restore emailAuthFactor and emailConfirmed from last session

Carries over https://github.com/bluesky-social/social-app/pull/3728/files#diff-ba67025ff0027278031f772f88a9ed2f7fa4b12421d0a423ad74b4937aa2ca53R281 from #3728.
My bad that I missed it.
This fixes the false positive "you have not verified your email" popup. The emailConfirmed flag is not required so I forgot to specify it. So when you resume session from storage, the first render will not have the full object.
The full object still arrives a bit later (and gets written to the persistent storage) but then the same problem happens on next refresh. (The reason people don't see the same problem on next refreshes now is due to the auto-snoozing flag for this popup.)
Test Plan
Comment out all the other checks in shouldRequestEmailConfirmation. Verify that the full object comes through when you refresh the page, and the popup does not appear.

Thank you for JJ. Enjoying it fully.
Here is the console log. You should send your style over HTTPS.
P.S. I can't come up with better ways to reach out and say "ty".
All excuse me for off-top.


chore: staging into main

@ff6347 Works fine for me locally. Maybe you want to have a look as well before shipping.


[BUG]: Can't start desktop app on ubuntu, sandboxing issue

How are you running AnythingLLM?
AnythingLLM desktop app
What happened?
After installation on ubuntu using the provided script,i get this error when launching with ./AnythingLLMDesktop/start :
[306733:1129/152127.691509:FATAL:setuid_sandbox_host.cc(158)] The SUID sandbox helper binary was found, but is not configured correctly. Rather than run without sandboxing I'm aborting now. You need to make sure that /home/xxx/AnythingLLMDesktop/anythingllm-desktop/chrome-sandbox is owned by root and has mode 4755.
Trace/breakpoint trap (core dumped)

Are there known steps to reproduce?
No response

I encountered the same issue.  From my experience with running Obsidian as an AppImage file, I made the following change to the start script at line 43:
exec "$BIN" "--no-sandbox"

And it fired up.  I'm not sure that's the right long term solution, but it let me at least get started.

Are you running the AppImage as root? Other than that the file permissions in the error message would resolve this
Related doc I've used before
https://authmane512.medium.com/solve-the-suid-sandbox-helper-binary-was-found-but-is-not-configured-correctly-3-solutions-4f1425a9a76c

I just used the recommended Linux installer script and directions here: https://docs.anythingllm.com/installation-desktop/linux#install-using-the-installer-script.  There was no mention of running it as root.  Perhaps the start script out to make changes to avoid needing to run as root?  This seems like a problem literally everyone using that script will have.


Cannot click or scroll pulseaudio modules other than the first if more than one are present on the bar

Describe the issue
If more than one pulseaudio modules are present on the bar (targeting different sinks) the first one will "eat" all inputs done to the second, including clicks and scroll wheel.
The second module still responds and updates when I change volume and status through other means (e.g. pavucontrol).
Expected behavior:
Both modules should operate independently and handle their own inputs.
Actual behavior:
Scrolling or clicking the second module changes volume and mutes the first module rather than the one being hovered. The first module works normally.
Was it working before?

Did you also experience this bug in an earlier version of polybar?

N/A

If no, what was the last version where this worked correctly?

N/A

To Reproduce
A minimal but complete config with which the problem occurs:
[module/volume-hp]
type = internal/pulseaudio
sink = alsa_output.usb-SABAJ_SABAJ_DA3_v1.2-00.iec958-stereo

use-ui-max = false

format-volume =  <label-volume>
format-volume-underline = ${colors.cBlue}

label-volume = %percentage%%

label-muted = -
format-muted =  <label-muted>
format-muted-underline = ${colors.cRed}

[module/volume-sp]
type = internal/pulseaudio
sink = alsa_output.pci-0000_00_1f.3.analog-stereo

use-ui-max = false

format-volume = <ramp-volume> <label-volume>
format-volume-underline = ${colors.cBlue}

label-volume = %percentage%%
ramp-volume-0 = 
ramp-volume-1 = 
ramp-volume-2 = 

format-muted = <label-muted>
format-muted-underline = ${colors.cRed}
label-muted = 

Polybar Log
Post everything polybar outputs to the terminal when you run it and the issue occurs below
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)
warn: module/i3: Attempting to reconnect socket (reason: Unexpected EOF while reading header)

Screenshots
If applicable, add screenshots to help explain your problem.
Environment:

WM: i3
Distro: Arch Linux
Output of polybar -vvv:

polybar 3.4.3

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags: -D_FORTIFY_SOURCE=2 -march=native -mtune=native -O2 -pipe -fno-plt -Wall -Wextra -Wpedantic -O3 -DNDEBUG
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

Additional context

Duplicate of #1172
See Known-Issues

Ouch, I searched for "pulseaudio" thinking it would be more specific. Sorry!


Tamil character

Does it print tamil characters.

Hey @vimalkumar1811 I am also facing the same issue, Did you solved this?


Public Views Concat and Zip should take Iterables as parameter instead of IterableOnces

With the current signature (taking an IterableOnce parameter), it is easy to build a view that does not support multiple traversal (even though View extends Iterable).
See also the discussion here: https://github.com/scala/collection-strawman/pull/184#discussion_r131584425

Closed in #184.


Follow+Notes: Create client-side feature flag

Subtask of [STORY]: Update 'Status' to 'Follow + Note' feature #2960
Blocked by
N/A
Blocks

#3406
#3429

Definition of Done
A new feature flag, followNotesEnabled, is defined in the client-side library of feature flags.
Implementation details
(See also: our Feature Flags development guide.)

In packages/client/src/helpers/featureFlags/index.js, create a new function called followNotesEnabled() that returns a boolean value based on whether a feature flag named followNotesEnabled strictly evaluates (===) to true.
In packages/client/public/deploy-config.js, register the new followNotesEnabled with a value of true so that this feature is enabled by default in development environments.

Note: Lines 16-17 of this file are currently not formatted according to our ESLint rules. Please reformat so that this file is compliant.

In terraform/staging.tfvars, register the new followNotesEnabled feature flag within the website_feature_flags map with a value of false so that this feature is disable by default in our Staging environment.
In terraform/prod.tfvars,  register the new followNotesEnabled feature flag within the website_feature_flags map with a value of false so that this feature is disable by default in our Production environment.

tried out the feature flag in staging and it is working as expected -thanks @sushilrajeeva !


[reggen] RFC: support for block version registers

Placeholder for register version issues.

Register format proposal doc
Versioning conventions doc (TBD)

Splitting this issue into two RFCs. The first one is the register region format, and that one is required for M2 (see doc).
The second RFC is about the versioning rules, and that needs more time to converge, hence it is deferred to after M3, see: #15856

@msfschaffner i think we can officially postpone this to M3 given the state of things.
What do you think?

Yes that sounds good to me.
CC @alphan @cfrantz

SGTM

SGTM, does this mean I have more time to look at the doc updates?

That is correct, yes.

This RFC has been approved by the TC today.


Cannot access values directly

The Drupal documentation suggests that you should be able to access properties of an Attribute object in a twig template via either dot or bracket notation. But as this object is implemented as a Map, those properties don't exist.
I spent an afternoon messing about with Proxies and Reflect and managed to get something that essentially adds property accessors for the keys available in the Map. If you change the constructor to be like so:
class DrupalAttribute extends Map {
    constructor(it) {
        super(it);
        return new Proxy(this, {
            get(target, prop, receiver) {
                let value = Reflect.get(...arguments);
                return typeof value == 'function' ? value.bind(target) : target.get(prop);
            },
        });
    }
}

That works when running a test script in Node. I can access values in the Attributes object via attributes.id or attributes['id']. Pseudocode is "when someone tries to access a property, call .get to pull it from the map, unless that property is a function, in which case call it as normal".
However much to my disappointment this doesn't seem to work when used within Twig.js (which I suspect is the primary use case for this library?) as it would appear Twig.js does clever things based on type, rather than for example just dumbly executing attributes.id regardless. Specifically: if I wrap a plain object in a Proxy and pass that into a twig template, and use the proxy to log the prop that gets accessed, it is id and everything works. As soon as I wrap a Map object instead, then the property that gets accessed with attributes.id is actually getId. I haven't dug into the twig.js internals to figure that one out yet!
However, the obvious other option I can think of (instead of adding property access to Maps) would be to reimplement this whole class as a plain Object. Am I missing something, or could that be a legit option? I can't think of any of the features of Maps that would be deal-breakers in this case (Keys should always be alphanumeric right? Ordering of attributes shouldn't matter?). Any thoughts?

Interesting question, but first, sorry for the delay, I missed the notification.
This library has been tested using Twing. Twig.js, for all of its pros, isn't a perfect implementation of Twig specification. How it deals with attributes is unknown to me but I suspect it is not implementing the whole specification of Twig, which is quite complex:
https://github.com/twigphp/Twig/blob/f1e0ebc18a6b7e036a1dad8c6daf4fc967d9cc86/src/Extension/CoreExtension.php#L1378
With Twing, the following code output the expected result:
import {TwingEnvironment, TwingLoaderArray} from "twing";

const DrupalAttribute = require('drupal-attribute');

const attribute = new DrupalAttribute();

attribute.setAttribute('foo', 'bar');

const environment = new TwingEnvironment(new TwingLoaderArray({
  index: `{{ attribute.foo }}{{ attribute["foo"] }}`
}));

environment.loadTemplate('index').then((template) => {
  template.render({
    attribute
  }).then((output) => {
    console.warn(output); // barbar
  })
});

This said, using a plain object instead of a Map looks like a totally legit option to me.
I am migrating all my old JS libs to TypeScript so it may be the perfect moment to upgrade drupal-attribute to plain object + TyepScript.

This library has been tested using Twing.

If you only support Twing, can we add that to the README? :)
Earlier this year, I couldn't get Twing to work with Storybook, so I've been stuck using Twig.js for now. I'm seeing lots of Attribute test failures in both Twing and Twig.js, so I'm probably going to have to fork this repo to support both. I'll make sure I pass on the Twing-related fixes back here as PRs.
Thanks for all your work, @ericmorand!


update game over screen

update game over screen to cover canvas
change color to be more pleasing
center game over message

Looks good.


Automate installation of submodules

Ekzo.sass should be installed and attached as submodule after npm install.
Not sure is it possible, but theoretically — yes. Probably we can use post-install script, which will install git submodule if there is no certain submodule yet.
Please, note that current issue related to initial installation only. Updating of submodule should be made with usual git methods, not via npm install. Probably.
Some more information about this: https://github.com/npm/npm/issues/6700

Turns out npm install pulls releases, which already have downloaded submodules, though, they aren't initialized, you need to re-init them manually.

We should consider creating separate kotsu-cli package that will manage kotsu package as dependency and install it in user defined folder correctly


test: fix access to ovsdb in local-test

Export the OVS_DB env var in docker-compose and use it in unit tests to fix local-tests
Fixes: #25
Signed-off-by: Adrian Moreno<EMAIL_ADDRESS>

thanks


Add example for comparison operators infix.

Example for #677
Problem
People are asking from time to time about comparison operators for java.util.Date or org.jode.time.LocalDateTime, etc.
Solution
I think it's useful to have an example in README.md.
Checklist

[x] Unit test all changes
[x] Update README.md if applicable
[x] Add [WIP] to the pull request title if it's work in progress
[x] Squash commits that aren't meaningful changes
[x] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

👍

@gustavoamigo You can approve it by adding comment "Approved" 😉

Approved


Browser support

Hi Ali,
This is almost exactly what I was looking for but I'm curious about which browser are supported and smartphone browsers?
I've tested it on Android v2.3.5 native browser and the ul's are not hidden.
Best regards
Umut Ücler

Hey @umutucler,
I don't have an older Android device to test it on. Can you show me a screenshot of what you see?

Hi,
Here is a screenshot
Med venlig hilsen / Best regards
Umut Ücler

Den 24 May 2015 kl. 06:26 skrev Ali Zahid<EMAIL_ADDRESS>Med venlig hilsen / Best regards
Umut Ücler
I don't have an older Android device to test it on. Can you show me a screenshot of what you see?
—
Reply to this email directly or view it on GitHub.

The image was somehow not added. Here it is.

@umutucler I've finally updated the plugin. Please check if your issue has been resolved now. Also, help me test the plugin for any bugs, if you're still using it.

Hi Ali,
Thanks! I have been ill for a long time and haven't been able to work on the project. Starting again now and will report back with bug findings and suggestions.


Change to exclude list of versions bundling OpenJCEPlus

A list of Java versions to be excluded from bundling OpenJCEPlus with is introduced, in place of the existing boolean value.
Signed-off by: Kostas Tsiounis<EMAIL_ADDRESS>

Depends https://github.com/ibmruntimes/temurin-build/pull/107


LED light is very dim

I'm not sure if this is an issue with ds4drv, but the LED light on my controller is very dim once it's connected. While it's connecting or in pairing mode it flashes at a normal brightness, but when it's connected I need to cup my hand over the light to see it. Is this something anyone else is experiencing? I've tried setting the led color to ff0000 and 0000ff.

Does the color change to red and blue when you set the color?

The color doesn't change - it's always a dim red. I've tried it with hidraw enabled and disabled.

Sounds like it's not reading from your conf file. Make sure there's a ds4drv.conf file in ~/.config directory, if not get it from here https://github.com/chrippa/ds4drv/blob/master/ds4drv.conf
it seems very normal behaviour on connecting the controller to the computer to have a dimmed LED if there is no config file present I've found, except the default color is usually blue, not red.


Improve page styling condition expression

The condition expression for checking current page is prone to error. If user creates a post with the word page in its slug, Jekyll will apply a wrong styling to that post.
A better method is to check for the paginator's current page number. If it's larger than 1, then the user is on page2 or greater. If not, then the user must be on the home page.

That makes sense! Thanks @hieusensei


Syncronizing skeleton

Is your feature request related to a problem? Please describe.
Currently I use this code to sync Leap motion handtracking and models to the clients https://forum.unity.com/threads/synchronizing-an-entire-skeleton.355636/
Describe the solution you'd like
An Mirror variant of this code. Some functions are not available in Mirror

fixed it
https://gist.github.com/minusplusminus/73a2b48065ce3f3c05e79c9e3830a748
added the following to NetworkConnections.cs
public virtual bool Send(short msgType, NetworkWriter writer, int channelId = Channels.DefaultReliable)
        {
            // pack message and send
            byte[] message = Protocol.PackMessage((ushort)msgType, writer.ToArray());
            return SendBytes(message, channelId);
        }

@minusplusminus use this function:

We don't send writers directly anymore, only messages instead. so in your case you just construct a skeletonsyncmessage and send it.
because there is no reason to send random bytes to the client - it always has to be a message.

Yes, i've seen. Networking traffic is allot better managable with this structure.  Like it!

@minusplusminus Could you share your fixed version please?

No, not possible anymore.


Jsonpickle decode() function unsafely form objects that can lead to Remote code execution

JsonPickle 1.4.2 allows remote code execution during deserialization of a malicious payload through the decode() function.
Attack Vectors : The jsonpickle can be exploited by deserialization of malicious jsonpickled payload with default decode() function of its object. The payload can be easily generated by this payload generator:
https://github.com/j0lt-github/python-deserialization-attack-payload-generator
and passed to decode function
like object = jsonpickle.decode(payload)
it will certainly execute command.

According to this project's README:

WARNING: jsonpickle can execute arbitrary Python code. Do not load jsonpickles from untrusted / unauthenticated sources.

This property of pickle is well known and documented:

Warning: The pickle module is not secure. Only unpickle data you trust.
It is possible to construct malicious pickle data which will execute arbitrary code during unpickling

This report was also sent to the project in Tidelift. I've responded there as well indicating that this is a known issue and that there's no known fix for this vulnerability. Recommend to close as wontfix.


Embed check will always work

The endpoint used in https://github.com/ozguralp/gmapsapiscanner/blob/master/maps_api_scanner.py#L28 will always work on valid API keys, even if the key is restricted. Here is why:
If the embed maps are using the basic embed requests, Google will not check restrictions, as the usage of that API is free.

https://www.google.com/maps/embed/v1/place?q=Seattle&key=KEY

But if the advanced map modes are used, then the restrictions apply.

https://www.google.com/maps/embed/v1/search?q=record+stores+in+Seattle&key=KEY

Hey,
Good catch on this one! While using the paid API endpoint on the script could be better for a solid PoC, I still cannot observe the restriction difference on both of the endpoints. If the referer restriction exist, both endpoints returning "This IP, site or mobile application is not authorized to use this API key." error, whether it is paid or not. However not sending referer header bypasses this control on both situations. I mentioned about that vulnerability on my latest blog posting https://medium.com/bugbountywriteup/google-maps-api-not-the-key-bugs-that-i-found-over-the-years-781840fc82aa if you didn't read yet. My script is sending requests without referer headers so if the Embed API is active on a key then it is vulnerable whether referer check exists or not.
So in conclusion:

If referer restriction exist; whether paid api or not, Google servers rejects the request.
If referer header not sent with the HTML code; either way the it could be abused.
It also applies to Staticmap and Streetview API's.

Hope this information helps to clarify your thoughts.

In addition, you can use the leaked API key by Google mentioned on the blog post to test also for this one with both sending referer headers and not to confirm the behavior.

If the referer restriction exist, both endpoints returning "This IP, site or mobile application is not authorized to use this API key."

However not sending referer header bypasses this control on both situations.

If referer restriction exist; whether paid api or not, Google servers rejects the request.

If referer header not sent with the HTML code; either way the it could be abused.

I don't agree. Check the following iframe examples (the key used is the one from Google documentation, as an example):
<!-- Basic mode (free), authorised -->
<iframe src="https://www.google.com/maps/embed/v1/place?q=Seattle&key=AIzaSyD4iE2xVSpkLLOXoyqT-RuPwURN3ddScAI">

<!-- Advanced mode (paid), not authorised -->
<iframe src="https://www.google.com/maps/embed/v1/search?q=record+stores+in+Seattle&key=AIzaSyD4iE2xVSpkLLOXoyqT-RuPwURN3ddScAI">

If we do a request directly to the endpoints:
$ curl -I https://www.google.com/maps/embed/v1/place\?q\=Seattle\&key\=AIzaSyD4iE2xVSpkLLOXoyqT-RuPwURN3ddScAI
> HTTP/2 200

$ curl -I https://www.google.com/maps/embed/v1/search\?q\=record+stores+in+Seattle\&key\=AIzaSyD4iE2xVSpkLLOXoyqT-RuPwURN3ddScAI
> HTTP/2 403

As mentioned in the docs I've pointed, the maps/embed/v1/place endpoint is free, and doesn't apply the key referer restrictions.
I've also reached Google Cloud support, and they said:

Maps Support, XXX: I came across this issue before as well and I only initially tried using the Basic mode. And now, I just tested and learned that the restrictions are working for Advanced mode.
Maps Support, XXX: I mean, this is a new learning for me as well so thank you for pointing out :)

It seems that you are correct, I conducted the same tests within the ones you sent me however on my first try it was working. Either a cache problem in my web browser (I am %99 sure that it was also working from the Burp Requests/Responses, either I missed something else on that time or something is changed 😄) Anyways thanks for pointing it out, I will add both of the scenarios to the script within both Basic-Advanced (Free-Paid) modes.

Hey again,
Just for you to know, I updated script to scan within both basic and advanced Embed API's. I also clarified this issue on my post and mentioned you & gave your credit at the end of the #Vulnerability 1 for this finding. Thanks again for your help!
Ozgur


Implement proper DAQ Dummy

This issue is for the DAQ dummy.

[ ] Rename it. The RedPitaya should not be in the name
[ ] Implement a rough version that is not fully functional
[ ] Implement a simulation of the DF pipeline, i.e. make a connection between the set tx values and the ref channels
[ ] Implement a simple SPIO simulation. This will likely involve MPISimulation and is a long term goal.

The first two points are rather important since they allow us to do unit testing of all the high-level infrastructure.

Ideally, this would be stateful in a sense that we have a temperature rise (retrievable by dummy temperature sensors) resulting in a drift of amplitude and phase for testing the control loop.


Implement typedoc and sassdoc versioning

We need to think of a way to add a versioning to the current document generation.

@SlavUI could you add this to your planner, we need to spend time on this because will impact both typedoc and sassdoc document generations and the whole Style and Api documentation.


Shell parsing for preview command in Windows is incompatible with fzf#shellescape and cmd.exe

Category

[X] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[ ] Mac OS X
[X] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[ ] fish

Backslashes in the filepath of the preview command are escaped so they don't run in Windows.
The preview script itself (ruby or bash) has no issues with filepaths with backslashes.
https://github.com/junegunn/fzf.vim/pull/418#issuecomment-322641946 is caused by the network drive conflicting with the colon delimiter. I'll open a separate issue/PR for that.

Escaping the preview command with fzf#shellescape doesn't work 😕

Workaround from fzf.vim for escaping filenames correctly if not using a batchfile for the preview command
https://github.com/junegunn/fzf.vim/commit/a4d4986d18074b7867584f837bc09cc37a110fd0#diff-7274b7c6c87526542ef9087f2e997dfaR42
let path = 'C:\path\to\preview_script.sh'
if has('nvim')
  let path = split(system('for %A in ("'.path.'") do echo %~sA'), "\n")[1]
else
     let path = fnamemodify(path, ':8')
endif
let path = escape(path, '\')
endif

Neovim support for DOS shortname is removed in https://github.com/neovim/neovim/pull/635. This hack is here to stay in fzf.vim 😢 and fnamemodify(path, ':8') and expand('%:p:8') are broken.

@junegunn Do you know why a single double-quote is converted to a double backslash?
I'm testing the bash preview script in fzf.vim on a line with a single quote, https://github.com/junegunn/fzf/blob/master/plugin/fzf.vim#L2, and I get File not found error with the backslashes escaped?

Not quite sure if I follow you. Can you provide the exact steps to reproduce the problem you described?

In Vim/Neovim, run Ag with preview
call fzf#vim#ag('', fzf#vim#with_preview())

Query for a double quote
plugin\fzf.vim:2:1:"
Ag> "

In the preview window, the preview script outputs File not found pluginfzf.vim because the preview script got pluginfzf.vim:2:1\\

I haven't tried it on Windows yet but it works on *nix.
On *nix, starting a command with sh is simple, we can just use ['sh', '-c', command] as the arguments for exec.Command without any preprocessing.
https://github.com/junegunn/fzf/blob/077ae51f0511260ba2d2c793b776582d41fefd6a/src/util/util_unix.go#L20-L23
On Windows, it's a little more complicated. I don't fully remember, but I noticed the same approach didn't work with cmd /c and I had to "shellescape" each token in the command using a third-party library.
https://github.com/junegunn/fzf/blob/077ae51f0511260ba2d2c793b776582d41fefd6a/src/util/util_windows.go#L18-L26
Do you have experience with Go? You can test the code like follows:
package main

import (
	"fmt"
	"os/exec"

	"github.com/mattn/go-shellwords"
)

func main() {
	command := "echo 'foo\\bar:1:2:\"'"
	args, _ := shellwords.Parse(command)
	allArgs := make([]string, len(args)+1)
	allArgs[0] = "/c"
	copy(allArgs[1:], args)
	fmt.Printf("%#v\n", allArgs)
	out, err := exec.Command("cmd", allArgs...).Output()
	fmt.Println(out, err)
}

go get github.com/mattn/go-shellwords
go run shellesacpe.go

and it gives:
[]string{"/c", "echo", "foo\\bar:1:2:\""}
[] exec: "cmd": executable file not found in $PATH

Do you see anything strange about this?

Windows ruby passes it as is so the ruby preview script is necessary after all.

Which echo are we testing on?

I tested it on macOS where cmd doesn't exist, and I knew it was bound to fail. Sorry for the confusion, I just wanted to show how arguments are passed.

I think cygwin bash itself is the issue

Yeah, agreed.

If we port fzf#shellescape in go such that we can shellescape the preview command into a single token, is tokenization still required?

If the double quotes are not escaped somehow such that it is executed as is but previewCmd is wrapped in double quotes when it is run, then
cmd := exec.Command("cmd", "/s", "/c", previewCmd).Output()

should work.
:: internal echo
>cmd /s /c " echo foo\bar" "
foo\:bar"
:: cygwin echo.exe
> cmd /s /c " "echo" foo\bar" "
foo:bar

Imo, Vim and Neovim should use /s instead of sxe and sxq but this is unintuitive because of how cmd.exe dequotes the command.

Have to explicity set syscall.SysProcAttr.CmdLine manually.
https://github.com/golang/go/issues/15566#issuecomment-333274825
https://golang.org/src/syscall/exec_windows.go

Reference: https://play.golang.org/p/aU1PlbNTqM
@junegunn Can you set CmdLine and the run the preview command (no parsing and escaping) as is?
package main

import (
	"fmt"
	"os/exec"
	"syscall"
)

func main() {
	cmd := exec.Command("cmd")
	command := `"echo" foo\:bar"`
	cmd.SysProcAttr = &syscall.SysProcAttr{
	    HideWindow: false,
	    CmdLine: fmt.Sprintf(` /s /c "%s"`, command),
	    CreationFlags: 0,
	}
	fmt.Println(cmd.SysProcAttr.CmdLine)
	out, err := cmd.Output()
	fmt.Printf("%s\n", out)
	fmt.Println(out, err)
}

I still have this issue. Have fzf preview support Window yet?


VSCode WSL remote doesn't work when SymCrypt OpenSSL engine is set as default

When SymCrypt-OpenSSL is set as the default engine for all algorithms in via OpenSSL.cnf, VSCode Remote for WSL fails to connect with the following errors:
[2021-09-16 16:07:30.426] [renderer1] [info] [remote-connection][ExtensionHost][91019…][reconnect] starting reconnecting loop. You can get more information with the trace log level.
[2021-09-16 16:07:30.426] [renderer1] [info] [remote-connection][ExtensionHost][91019…][reconnect] resolving connection...
[2021-09-16 16:07:34.677] [renderer1] [info] [remote-connection][Management   ][04ee5…][reconnect] connecting to <IP removed>:39209...
[2021-09-16 16:07:34.679] [renderer1] [info] [remote-connection][ExtensionHost][91019…][reconnect] connecting to <IP removed>:39209...
[2021-09-16 16:07:34.844] [renderer1] [info] [remote-connection][Management   ][04ee5…][reconnect] received socket close event (wasClean: false, code: 1006, reason: ).
[2021-09-16 16:07:34.845] [renderer1] [error] {"isTrusted":true}
[2021-09-16 16:07:34.851] [renderer1] [error] [remote-connection][Management   ][04ee5…][reconnect][<IP removed>:39209] received error control message when negotiating connection. Error:
[2021-09-16 16:07:34.852] [renderer1] [error] Error: Connection error: Unknown reconnection token
    at W (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:637:154700)
    at vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:637:145469
    at l.fire (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:60:1712)
    at n.fire (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:76:16477)
    at u._receiveMessage (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:76:21798)
    at vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:76:20841
    at l.fire (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:60:1712)
    at b.acceptChunk (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:76:13838)
    at vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:76:13186
    at vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:637:140524
    at l.fire (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:60:1712)
    at FileReader._fileReader.onload (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:637:139084)
[2021-09-16 16:07:34.852] [renderer1] [error] [remote-connection][Management   ][04ee5…][reconnect] A permanent error occurred in the reconnecting loop! Will give up now! Error:
[2021-09-16 16:07:34.852] [renderer1] [error] Error: Connection error: Unknown reconnection token

Compiling with tracing enabled does not yield any errors or warnings in the SCOSSL log:
mitch@MITCHL-Z420:~$ rg ERROR /tmp/scossl.log
mitch@MITCHL-Z420:~$ rg WARN /tmp/scossl.log

openssl.cnf changes:
[ openssl_def ]
engines = engine_section

[ engine_section ]
symcrypt = symcrypt_section

[ symcrypt_section ]
engine_id = symcrypt
dynamic_path = /usr/lib/x86_64-linux-gnu/engines-1.1/libsymcryptengine.so
default_algorithms = ALL
init = 1

Unfortunately I haven't found a good way to debug this yet.

"Unknown reconnection token" could indicate that the VS Code Server was restarted and VS Code tries to reestablish the interrupted connection without being aware of that. Another possibility would be that the connection terminated unexpectedly, but the VS Code Server somehow thinks that it was regularly terminated (or timed out) and it deleted the connection information, not recognizing the 'reconnection token' when VS Code tries to reconnect.
What seems clear is that there was a connection and that was somehow interrupted. At the time of the error VS Code is trying to reestablish that connection, but the server does not recognize the reconnection token anymore. (Reconnection tokens identify a remote session and help resuming sessions after network connections were interrupted.)

I wasn't able to replicate the error - waiting for Mitch to confirm whether it is still an issue for him.

Confirmed that this no longer reproes for me (sorry for the delay)


Update proguard documentation

In the readme, the proguard configuration lists:
-keepclassmembers class ** {
    @de.halfbit.tinybus.Subscribe public *;
    @de.halfbit.tinybus.Produce public *;
}

Should it not be 'com' instead of 'de':
-keepclassmembers class ** {
    @com.halfbit.tinybus.Subscribe public *;
    @com.halfbit.tinybus.Produce public *;
}

Please pull the latest changes. Readme is up-to-date.

Sorry. I misread the first message. Here is update. I changed the package in Version 3 to make it more consistent with Maven artifacts. Readme already describes the changes in v3. I will make it more clear in Readme file in a minute.


Erroneous CC0039 message

In this code:
Private Sub AdjustSample(readBuffer() As Single, ByRef readIdx As Integer, writeBuffer() As Single, ByRef writeIdx As Integer)
	If mIsMuted Then
		For chIdx = 0 To mDestChannels - 1
			writeBuffer(writeIdx) = 0
			readIdx += 1
			writeIdx += 1
		Next

...
there is an erroneous warning "Don't concatenate 'readIdx' in a loop".

I am not being able to repro this. Are you getting this message with the latest version of CodeCracker?
I will close the issue until we find a test case that can be reproduced. Please let us know.

@giggio  I had no problem reproducing this with code from master.  Where is our disconnect?

@adraut I added the test from @mc0re PR (#461) on 252e6dd, and it passed: https://ci.appveyor.com/project/code-cracker/code-cracker/build/<IP_ADDRESS>9
This is the current HEAD on master. Does it match for you?
I need a test case that I can repro.

@giggio Change the parameter to ref parameter.  That's what causes the problem.  The non-ref parameter becomes a local which the diagnostic checks for already and is why your test passes.  It should be like the test in aef112b38afceb6543c28fd2966930e66808353e
public class TypeName
    {
        public void Looper(ref int a = 0)
        {
            while (a < 10)
            {
                a += 1;
            }
        }
    }

@adraut No, it still does not raise CC0039. I added another test on 5f54e6a. You can see the build still passes: https://ci.appveyor.com/project/code-cracker/code-cracker/build/<IP_ADDRESS>5
Also, you can't have a default on a ref parameter, that will raise CS1741. But I tried it just like that, and it also did not raise CC0039.

@giggio Ok, then it is not broken in C#, but it definitely is in VB.  I blindly did the C# after confirming and fixing for VB.

Thanks @giggio - I missed the previous notification. It is about VB code.

Bug confirmed on VB.
I will look into the PR again.


[WB-4543] Add wandb.run.define_metric()

https://wandb.atlassian.net/browse/WB-4543
Description
Add wandb.run.define_metric()
Design
https://paper.dropbox.com/doc/CLI-API-add-run.define_metric--BEm98DroFga614g7tvrdo6IXAg-z9GIYShHF1sa9fTlBfxxz
Testing
unit tests almost 100%
TODO

[x] make sparklines better Not in this PR (combine with settings work)
[ ] handle nested metrics
[x] add tests
[x] PROTOTYPE: get someone to modify the frontend to respect config["_wandb"]["x_axis"] Thanks @KyleGoyette
[ ] Make sure frontend is happy with protobuf encoded data
[x] get feedback on the api
[x] make sure that min,max,last are oneofs They are not
[x] figure out if we can add best, direction
[x] implement proposal changes in design doc
[x] add summary op mean
[x] add hide
[x] implement metric glob
[x] remove MetricItem, (only one metric per record from now on)
[x] implement auto_step logic
[x] get tests working again
[x] add publish recorder fixture (steal from the other PR as this could be first) not needed
[x] add goal? to metric (add summary op best)
[x] add best_step summary op? Not required in this PR, maybe or maybe later
[ ] Figure out how does this work with tensorboard_sync
[x] add docstring
[ ] get coverage up
[x] add telemetry
[x] send new-style metric
[ ] test and check against NaN

Example
import math
import wandb
run = wandb.init()
run.define_metric("global_step", hide=True)
run.define_metric("*", step_metric="global_step", step_sync=True, summary="min,max")
for x in range(5, 100):
    val = 100*math.exp(-(((x-30)/15)**2)) + 60*math.exp(-(((x-70)/20)**2))
    run.log(dict(global_step=x, val=val))
    run.log(dict(me=2))

Output

https://wandb.ai/jeffr/uncategorized/runs/1kwu7w1t

Whoa, this is cool!  I was thinking we may want to also support run.define_metric("val_acc", xaxis="global_step") or run.define_metric("val_acc").set_xaxis("global_step")

I love it! In particular the x-axis feature!
On my side it just needs to be defined globally for now.
But I like Chris idea, to which we could add run.define_metric(['train_loss, val_loss])`.
What happens for people who changed their x-axis in W&B interface?
Would it reset it or does it become the new "default" (when we clear settings)?

I love it! In particular the x-axis feature!
On my side it just needs to be defined globally for now.
But I like Chris idea, to which we could add run.define_metric(['train_loss', 'val_loss']).
I didn't implement a list, but I did add a wildcard.
The reason why i didn't add a list, is it might be necessary to pass a list to support nested metrics.

What happens for people who changed their x-axis in W&B interface?
Would it reset it or does it become the new "default" (when we clear settings)?
The intent is that this would only change the default x-axis for a metric.

Whoa, this is cool! I was thinking we may want to also support run.define_metric("val_acc", xaxis="global_step") or run.define_metric("val_acc").set_xaxis("global_step")

I messed with the usage, it is snow more consistent with our other methods as you recommended (using parameters instead of setters).


Automation results submission for the pizza task

Google checks sent result files online. So to avoid unit test writing we will write a test on Selenium WebDriver that sends results to the submission page
and parse scores

I finished up with SeleniumIDE and firefox. Here is my test case. Need to substitute file paths to appropriate.
type	//input[@type="file"][1]	/Users/greg/Desktop/Hamcrest-1.3.pdf
pause	1000
type	//input[@type="file"][1]	/Users/greg/Desktop/Hamcrest-1.3.pdf
pause	1000
type	//input[@type="file"][1]	/Users/greg/Desktop/Hamcrest-1.3.pdf
pause	1000
type	//input[@type="file"][1]	/Users/greg/Desktop/Hamcrest-1.3.pdf
pause	1000
type	//input[@type="file"][1]	/Users/greg/Desktop/Hamcrest-1.3.pdf
pause	1000
click	xpath=(//button[@type='button'])[25]

Final solution
scripts:

For zipping the source code(bash)
To submit the source code archive and output files via a web form(selenium)

I think a better option is to migrate a selenium script to Java but this way we will encounter a problem with authentification or connecting to an open browser.
See the branch automationResultsSubmission

I'm finishing with with selenium code (but u shoud be logged in from u'r google account in this Chrome browser) when it works i'll comit it.


Update about-azure-devops-services-tfs.md

Test pull request for content developer role
@j-martens

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

#label:"aq-pr-triaged"

Hi @irtizamahmud - This looks like a test PR that can't be merged in. Would you like us to close it? Thanks!
cc @chcomley

Yes please.
Thanks
Irtiza
From: Kristine Toliver @.>
Sent: Thursday, December 9, 2021 11:03 AM
To: MicrosoftDocs/azure-devops-docs @.>
Cc: Irtiza Mahmud @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-devops-docs] Update about-azure-devops-services-tfs.md (PR #11669)
Hi @irtizamahmudhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Firtizamahmud&data=04|01|irtiza.mahmud%40microsoft.com|c9ef44126446482ab66408d9bb35b932|72f988bf86f141af91ab2d7cd011db47|1|0|637746661702787541|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=JK5B4NetbjeueDCsUa28H2d%2B1ZqxZFFjqIIcz8wuERo%3D&reserved=0 - This looks like a test PR that can't be merged in. Would you like us to close it? Thanks!
cc @chcomleyhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fchcomley&data=04|01|irtiza.mahmud%40microsoft.com|c9ef44126446482ab66408d9bb35b932|72f988bf86f141af91ab2d7cd011db47|1|0|637746661702787541|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=PZVryWg%2BR%2BczzYRK4tAwNpKUIJM6uXLm5rCWp7cvNRg%3D&reserved=0

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-devops-docs%2Fpull%2F11669%23issuecomment-990041861&data=04|01|irtiza.mahmud%40microsoft.com|c9ef44126446482ab66408d9bb35b932|72f988bf86f141af91ab2d7cd011db47|1|0|637746661702787541|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=cxE31%2BqtdXMKczD6TjSQyBpEmADPo85m%2FFU9OumKwsw%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPIUK4DJJCP23ZSTQPVFSW3UQDOLNANCNFSM5I7WM6SQ&data=04|01|irtiza.mahmud%40microsoft.com|c9ef44126446482ab66408d9bb35b932|72f988bf86f141af91ab2d7cd011db47|1|0|637746661702787541|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=vV5H9qVEdCE4wd5bzI13Mu12X9ues4yDd6d3m7KG%2BU8%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|irtiza.mahmud%40microsoft.com|c9ef44126446482ab66408d9bb35b932|72f988bf86f141af91ab2d7cd011db47|1|0|637746661702787541|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=3AdiSTzGSYDrWgWkbbvUSbVCq6pchWhnq8QK5frgqHw%3D&reserved=0 or Androidhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|irtiza.mahmud%40microsoft.com|c9ef44126446482ab66408d9bb35b932|72f988bf86f141af91ab2d7cd011db47|1|0|637746661702787541|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=3tI4Gp9XMaUazoyjTozJgeS%2BFnuWdMTAGI1%2FnDjhLfI%3D&reserved=0.


Add clickbench queries to sqllogictest coverage

Which issue does this PR close?
Related to https://github.com/apache/arrow-datafusion/issues/6790
Rationale for this change
I would like test coverage for changes in https://github.com/apache/arrow-datafusion/pull/6827 (especially as it caused a regression that we didn't catch)
I also would like to ensure we don't break clickbench again
What changes are included in this PR?

Add a sample clickbench data file (first 10 rows)
Add clickbench.slt test file with data and queries

Are these changes tested?
They are ONLY tests

Are there any user-facing changes?
No

oh man, of course


Does VideoBaseTexture support transparency of video?

Hi All,
Does VideoBaseTexture support transparency of video?
Thanks.

Have you tried it? :)

It does. The question is, does your browser? :) Chrome is the only browser I know of that supports webm files with an alpha channel right now. Everything else just renders it opaque.

Closing this since it seems answered.

Hi guys, I have created a custom shader & sprite to supports transparency of video in my game. do you have any plan to support something like this?
the video file can download from:
https://jakearchibald.com/scratch/alphavid/compressed.mp4
VideoSprite
https://gist.github.com/games/855e22cb71d5f31b65bb
VideoShader
https://gist.github.com/games/3015a9194d3823654d7e

Is there any progress with transparency video? I have the same problem and so far as I understood I can do it only with webGL


adaptive concurrency: adding response body

Risk Level: low
Testing: updated tests
Docs Changes: n/a
Release Notes: inline
Runtime guard: envoy.reloadable_features.adaptive_concurrency_details
Fixes #13138

Oh excellent - I felt totally silly runtime guarding, but I'd been lazy and done git blame rather than checking extension status, and was trying to be a good role model :-P  I'll back that out.

Merge main to fix CI?
/wait


Review design Region2D and other related modules

In reviewing #45 I had some comments on the current structure of Region2D and related modules.
That PR was merged so that we can move forward on the project, so this issue is a reminder that we should revisit these.
The comments are reproduced below:

review the use case of Region2D to have clarity on whether we should just parse EVR file at init, or change the input to accept both EVR and CSV/JSON that were converted previously.
convert_points:

currently lives in evr_parser.py but I think it should just live in Region2D
it is currently unused. From your comment above it seems that some more work is needed to smooth out the points.

get_points_from_region: this function does not currently work since there is no get_points_from_region under Regions2DPlotter
Region2DPlotter: when should it be initialized?
add docstring for Regions2DMasker.mask (from Region2D.mask

I am under the impression that all these points have been addressed in current/previous PRs and issues, so I will close this issue.


zipkin plugin

What do krs, krf, kaf, kas, khs, kbf, kbs, khf mean?

This is my guess:
k = kong
x = phase
s = start
f = finish
krs = kong request start
krf = kong request finish
kas = kong access start
kaf = kong access finish
khs = kong header filter start
khf = kong header filter finish
kbs = kong body filter start
kbf = kong body filter finish

See: https://github.com/Kong/kong-plugin-zipkin#logs--annotations

I the answer was not right or you need further info, please reopen. I'll close it now.


Enable GRB tab

The work on GRB is split in two parts:

Offline inspection of potential GRB counterparts
Online forwarding of potential GRB counterparts

This PR enables the first point, by adding a new tab in the Science Portal. This tab superimpose alerts measurements with GRB lightcurves gathered by @dturpin-svom over years. The idea is to have a quick visual inspection of the decay of the events in the light of known events.

The user have to specify the trigger time (ideally from a known GRB event) in UTC (YYYY-MM-DD hh:mm:ss) to display the alert measurements as a function of the difference between the alert emission time and this trigger (in seconds). By default (i.e. no trigger entered), we compute the time difference with respect to the first variability of the alert (jdstarthist). Vertical dotted lines show 1 hour, 1 day and 1 week after the trigger.
To do:

[x] enable the GRB tab
[x] interface with Damien lightcurve
[x] allow  to choose the trigger time
[ ] allow to compare with respect to a known GRB event. Ideally we should be able to point to an online resource listing all known events, with their coordinates and trigger times. We should also show the distance to the source.
[ ] Implement Damien other metrics (serendipitous prob, fast transient criterion)

I'm closing this PR. Eventually we will come back to GRB -- but better to restart from scratch.


force_plot matplotlib=True -> values are not rounded

Example:

@dynamik1703
choosen_instance = dataset.loc[[1]].round(2)#The first line of the dataset
shap_values = explainer.shap_values(choosen_instance)
shap.initjs()
shap.force_plot(explainer.expected_value[1], shap_values[1], choosen_instance,matplotlib=True,show=False)

I hope it can help you, but when setting matplotlib=True, the picture cannot show the value of base value. If you can slove it, please reply to me. Thank you.

I "solved" this issue by changing the function format_data(data) in shap/plots/_force_matplotlib.py.
# Format negative features
    neg_features = np.array([[data['features'][x]['effect'],
                              round(data['features'][x]['value'],2) if isinstance(data['features'][x]['value'], float) else data['features'][x]['value'],  --> changed
                              data['featureNames'][x]]
                             for x in data['features'].keys() if data['features'][x]['effect'] < 0])
...
# Format postive features
    pos_features = np.array([[data['features'][x]['effect'],
                              round(data['features'][x]['value'],2) if isinstance(data['features'][x]['value'], float) else data['features'][x]['value'],  --> changed
                              data['featureNames'][x]]
                             for x in data['features'].keys() if data['features'][x]['effect'] >= 0])

I know it's not the most elegant way of doing that, but now I can read the graph.

The idea by @naarkhoo can work in some cases: rounding the features (i.e. the row(s) from the original data that get passed to the shap.plots.force(...) function) did it for me; neither the expected value nor the SHAP values had an influence.


Arc can lead to unknown lifetimes on default methods

Seems related to #11
error[E0310]: the parameter type `AsyncTrait` may not live long enough
  --> jcpc/src/interface.rs:82:13
   |
15 | / #[async_trait]
16 | | pub trait JoyCon: Sync + Send {
   | |_ help: consider adding an explicit lifetime bound...: `AsyncTrait: 'static +`
...
82 |               tokio::spawn(async move {
   |               ^^^^^^^^^^^^ ...so that the type `impl std::future::Future` will meet its required lifetime bounds

error: aborting due to previous error

For more information about this error, try `rustc --explain E0310`.
error: could not compile `jcpc`.

To learn more, run the command again with --verbose.

Sorry, this seems a duplicate of #108


allows to specify iam role inline policies

hi 👋
I've found it quite useful to be able specifying the inline policies in some use cases, rather than ARNs of existing policies

Thank you for adding this feature! I will publish a new release soon.


GCAP Changes Phase 1

Moves certifications to a dedicated table with a many2many relationship to users. Overall the changes to how to change a user's certs doesn't change, but the name will... had to approach this way given the sheer number of certifications and vast differences between facilities.

Phase 2 will follow that rebuilds the activity job and adds an action log for users that is viewable/changeable by staff.


Version bumps in the tests workflow for the 8.15 release

Version bumps in the tests workflow for the 8.15 release

@joemcelroy I think we can merge this now.


Adding auto scroll functionality to side panels.

Fixes #62
Signed-off-by: Suhaib Khan<EMAIL_ADDRESS>

@lukehollis Please review this.

Hey @suheb, thanks for the updates here and your patience. I need to describe the issue better. We need the panels to scroll asynchronously so that when you scroll from poem 1 line 1 being at the top of the window screen to poem 1 line 2 being at the top of the screen, the comments panel should scrolls to poem 1 line 2; the translations panel should scroll to its next translation mapping marker (we need to add these in the demo translation data), and the definitions panel should scroll to the definitions for poem 1 line 2.  Each panel's scrolling should differ based on their content and the line that is currently at the top of the window screen.
Also, we should have one toggle in the header instead of local toggles for panel. I'm closing this PR for now, but let's continue the discussion on https://github.com/cltk/cltk_frontend/issues/62


Turn off highlight color of change

Is there a setting to turn off the highlighting for the commit?  For example the following block of code has been highlighted by gitlens.  I can't seem to figure out the correct setting to disable this.

Which highlighting? GitLens doesn't highlight anything unless you are in the file blame mode.

I am closing this issue because it needs more information and has not had recent activity.  Please re-open this issue when more details can be provided.
Thanks!


Unittests: more commonizing

Here's the latest hunk of changes I'm making to try to set up for pulling common test things over to testuilt/.  Go test and the other checkers show good on this.

Coverage decreased (-2.2%) to 54.74% when pulling 31fab344270665fc017d0f3356f84f0686090587 on tpepper:unittests into ee7f384e0af418a4da3af38a6ac04aa5732b001b on 01org:master.

Coverage decreased (-2.2%) to 54.758% when pulling 31fab344270665fc017d0f3356f84f0686090587 on tpepper:unittests into ee7f384e0af418a4da3af38a6ac04aa5732b001b on 01org:master.

Don't merge yet.  I'll add some testutil/*_test.go so coverage doesn't appear to have gone down.  Everything in testutil/ here has coverage as it's used solely but other _test.go files, but...the tools can't see that.  They see a new package with basically no coverage of it's exported code.

Coverage decreased (-2.2%) to 54.776% when pulling 1d20ca0ab02d77111a09f834cb52a4e591d70faf on tpepper:unittests into ee7f384e0af418a4da3af38a6ac04aa5732b001b on 01org:master.

All SSNTP changes look good to me, thanks.

@tpepper Just a quick comment.  You need to add testutil to the test-cases command that runs the unit tests in .travis.yml.  There are 3 test-cases commands.  The first runs tests in packages that do not need to be root, so I guess it makes sense to add them to this one, e.g.,
   - test-cases -text -coverprofile /tmp/cover.out -short github.com/01org/ciao/ciao-launcher github.com/01org/ciao/ciao-scheduler github.com/01org/ciao/ciao-controller/... github.com/01org/ciao/payloads

Coverage decreased (-2.2%) to 54.767% when pulling d1575d77bdec5eb014b9a6ff88c1417c6422e803 on tpepper:unittests into 7b5bf5188067b18286dbadb5e5b09d5f18ab30a5 on 01org:master.

Coverage decreased (-2.2%) to 54.767% when pulling 06282b3da9a0d33ee0df052e6eee06e1335d83a7 on tpepper:unittests into 7b5bf5188067b18286dbadb5e5b09d5f18ab30a5 on 01org:master.

Coverage decreased (-2.2%) to 54.767% when pulling 8d8ecce62fcd6400691b7febc13fa4b4fb161ab4 on tpepper:unittests into 7b5bf5188067b18286dbadb5e5b09d5f18ab30a5 on 01org:master.

Coverage decreased (-2.2%) to 54.767% when pulling 08f0c9747bcd3b8939b86a9311368b77a8d67afb on tpepper:unittests into 7b5bf5188067b18286dbadb5e5b09d5f18ab30a5 on 01org:master.

Coverage decreased (-0.9%) to 56.413% when pulling ca5daaa103a7c6a75db4607b7f16cfc53df12a24 on tpepper:unittests into cf6b0a064115fb74652b560ddfc97731287bd203 on 01org:master.

It all looks good to me, but this PR needs to be rebased on master.

I'm still adding test coverage for the testutil/ library code.  Will rebase once that's a tad further.

Coverage increased (+0.2%) to 57.36% when pulling 4afed72e22c8b2f1e14f85af8529fd17b76b1b83 on tpepper:unittests into 5a3f50e58acc95fd86bb40b7d05b35489ce0bc49 on 01org:master.

Coverage increased (+0.2%) to 57.332% when pulling f27d91bad4eead191d7bb07ea8b9518c50167fac on tpepper:unittests into 5a3f50e58acc95fd86bb40b7d05b35489ce0bc49 on 01org:master.

Coverage decreased (-0.1%) to 60.66% when pulling bc997ab8ad7e2cec2237eb5bca7e76564cdd7c7b on tpepper:unittests into 7e5d567a613e26c3973f697619c8772d42f3b0fb on 01org:master.

Coverage decreased (-0.1%) to 60.667% when pulling 9d4425701012b0bb4ddd8c95ec107178fc8e6318 on tpepper:unittests into 7e5d567a613e26c3973f697619c8772d42f3b0fb on 01org:master.


Subscription/management group-level and extension/child  resource management

Linked documentation describes how to manage resources on Azure but the information for subscription level(1) resources and management group level(2) are missing.
How portal can be used to list and delete subscription level resources?
Use case 1
I have legacy(4) Microsoft.Security/pricings resources I need to delete and it looks like portal does not give any access to those legacy resources. I have already deployed new plan but legacy plans are triggering false alarms
AFAIK really limited access on portal. in the case of Microsoft.Security/pricings portal logic is failing.
Use case 2
I need to list all resources deployed into subscription/management group level.
AFAIK: not possible at the moment
Use case 3
I need to list all resources on any scope, including sub and extensions resource.
AFAIK: not possible at the moment. Some of the child resources are listed on resource group view with Show hidden types

Related ticket: graph API query (documentation): https://github.com/MicrosoftDocs/azure-docs/issues/83844

The current workaround for this is to enumerate through all known resources(1)(2)(3) with their management API list endpoints. E.g. https://learn.microsoft.com/en-us/rest/api/defenderforcloud/pricings/list?tabs=HTTP. This workaround should be doable on subscription and management group levels because the amount of is kind of low.
Using same method to list child and extension resources on resource groups is not feasible anymore. The amount of management API queries will grow too big.
(1) https://learn.microsoft.com/en-us/azure/azure-resource-manager/templates/deploy-to-subscription?tabs=azure-cli
(2) https://learn.microsoft.com/en-us/azure/azure-resource-manager/templates/deploy-to-management-group?tabs=azure-cli
(3) https://learn.microsoft.com/en-us/azure/azure-resource-manager/management/extension-resource-types
(2) A new 'Containers' plan is available! This plan will replace the existing 'Container registries' and 'Kubernetes' plans. Click here to learn more about the benefits and and additional protection it provides
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: b75537b6-d25d-92f8-179e-3f7ea2e13024
Version Independent ID: e509cf4d-8d4c-79b8-1afe-4faa514ca845
Content: Manage resources - Azure portal - Azure Resource Manager
Content Source: articles/azure-resource-manager/management/manage-resources-portal.md
Service: azure-resource-manager
Sub-service: management
GitHub Login: @mumian
Microsoft Alias: jgao

@jikuja
Thanks for your feedback! We will investigate and update as appropriate.

@jikuja To retrieve all the resources within management group and subscription, you can use below PowerShell as a reference. This code retrieves the hidden resources as well.
try {
    "Logging in to Azure..."
    Connect-AzAccount
}
catch {
    Write-Error -Message $_.Exception
    throw $_.Exception
}

$managementgroups = (Get-AzManagementGroup).Name
foreach ($mg in $managementgroups) {

    $subscriptionList = (Get-AzManagementGroupSubscription -GroupName $mg).Id

    $subscriptions = $subscriptionList.Substring($subscriptionList.Length - 36, 36)

    foreach ($s in $subscriptions) {
        Write-Output ("Setting Current context with " + $s)
        Set-AzContext -Subscription $s
        $ResourceGroups = Get-AzResourceGroup

        foreach ($ResourceGroup in $ResourceGroups) {
            Write-Output ("Showing resources in resource group " + $ResourceGroup.ResourceGroupName)
             Get-AzResource -ResourceGroupName $ResourceGroup.ResourceGroupName | Format-Table

        }

    }

}

For use case 1, I would suggest you to reach out to Microsoft Support team by raising support ticket using these steps.
Also note this channel is mainly for documentation changes, for any technical questions, post your question on Microsoft Q & A  channel. We will close the issue for now. If there are further questions, please revert and we will be glad to assist you.

The script solves only some of the issues

it fetches resources only from resources groups => Management and subscription level resources are missing
API endpoint does not return extension resources

Also note this channel is mainly for documentation changes, for any technical questions, post your question on Microsoft Q & A channel. We will close the issue for now.

Listing know limitations is also part of the documentation IMO. Sadly I have not seen any lists for given limitations or fully working workarounds.


Doesn't do command

Hi, I wanted to make like whitelist thingy, so if new player wants to join, he haves to register on website by filling in whitelist application by buycraft. I made a package which adds player to whitelist like so "whitelist add {name}" but it doesn't work.

You would need to uncheck "require online" but unfortunately that is not yet supported.


dataloader is very slow

train dataloader is very slow
hello, i just move yolov5 to mmdet, but use mmdet's pipeline, but found mmdet loading batch is very very very slow than yolov5 repo!!
for mmdet,  just load and resize image and letterbox for loadandAugTest without any about bboxes
load_piplinev2 = [
    dict(type='loadandAugTest', img_size=640),
    dict(type='DefaultFormatBundle'),
    dict(type='Collect', keys=['img'], meta_keys=()),
]
num_workers = 8
batch_size = 64
tmp_dataset = build_dataset(tmp_data_cfg)
data_loader = build_dataloader(tmp_dataset, batch_size , num_workers, num_gpus=1, dist=False, seed=None)

273/1833 [09:09<55:41,  2.14s/it]

for yolov5 repo, just normal datset and loader without DHcontainer,  i setting batch_size =64, augment=False, rect=False, num_worker=8(without tricks to train faster) , then yolov5 just do the same preprocess(i do not comment out annoatation-process code)
273/1833 [09:09<55:41,  2.14s/it] ---mmdetection (not stable , time range about ( 38:00--1.15.00)
224/1849 [00:53<06:30,  4.17it/s --- yolov5 repo normal dataset (time range about (3:55--6:30) )

dataset is cocodataset

the problem has been solved,  setting opencv num_threads to zeros to avoid processor conflicts
cv2.setNumThreads(0)


impl(common): add internal::RemoveOptions<T...>(Options)

Add a helper function to remove all options in an OptionList.  This is intended as a companion to internal::ExtractOption<T>(Options), for use in services where the Options passed to a client operation are split into two subsets: (1) those options passed explicitly to the connection layer and beyond, and (2) the remainder that are passed implicitly via thread-local state.
For example, in a Client::Foo(..., Options opts), ...
  auto foo1_option = internal::ExtractOption<Foo1Option>(opts);
  auto foo2_option = internal::ExtractOption<Foo2Option>(opts);
  internal::OptionsSpan span(
      internal::RemoveOptions<ExplicitOptions>(std::move(opts)));
  return conn_->Foo(..., {foo1_option, foo2_option});

where ExplicitOptions contains all the option types in category (1).

This change is

For example, in a Client::Foo(..., Options opts), ...
  auto foo1_option = internal::ExtractOption<Foo1Option>(opts);
  auto foo2_option = internal::ExtractOption<Foo2Option>(opts);
  internal::OptionsSpan span(
      internal::RemoveOptions<ExplicitOptions>(std::move(opts)));
  return conn_->Foo(..., {foo1_option, foo2_option});

I think this is fine, assuming opts has not been merged with the *Client options yet.  That is, the code would be:
... Client::Foo(..., Options opts) {
  auto foo1_option = internal::ExtractOption<Foo1Option>(opts);
  auto foo2_option = internal::ExtractOption<Foo2Option>(opts);
  opts = internal::RemoveOptions<ExplicitOptions>(std::move(opts));
  internal::OptionsSpan span(internal::MergeOptions(std::move(opts), options());
  return conn_->Foo(..., {foo1_option, foo2_option});

You probably just forgot about the merge.  No big deal.  Eventually I want to change the prototype from merge options to something like:
ImmutableOptions MergeOptions(Options preferred, ImmutableOptions fallback);

At that time any changes to the result of MergeOptions will not be possible (not without an expensive copy).

I think this is fine, assuming opts has not been merged with the *Client options yet.

I did indeed misplace the merge in the example. It should instead read ...
  opts = internal::MergeOptions(std::move(opts), opts_);
  // Update `opts` via extraction and removal.
  internal::OptionsSpan span(std::move(opts));

so that the "foo options" could come from the client or the operation.
Given that, I'll hold off on this until the dust settles. Perhaps trying to remove the "explicit" options from the span isn't worth it.

But it does mean that I might have to copy an operation's explicit options from (at least) the connection options.

Currently you would be making one copy (in the Merge() call), and avoid the second copy via ExtractOption<>.  Once ImmutableOptions is in place, you would be making one copy (in a get<>() call), since ExtractOption<> would not be applicable anymore [^1].
That is, in the fullness of time, both approaches yield the same number of copies for these explicit options.

Maybe every non-scalar option type should be a shared_ptr<T const> instead?

Possibly. If we only access these via get<>() then that may not optimize anything.  It depends on how the option value is used after accessing it.


Incompatible with newer setuptools

      pystache: using: version '65.5.1' of <module 'setuptools' from '/usr/local/lib/python3.10/site-packages/setuptools/__init__.py'>
      Warning: 'classifiers' should be a list, got type 'tuple'
      error in pystache setup command: use_2to3 is invalid.

This only happens when it's installed as a dependency, not when installed directly. eg, pip install statik fails but pip install pystache succeeds.
Works: Python 3.8, Pip 22.0.4, setuptools 57.5.0
Fails: Python 3.9, Pip 22.0.4, setuptools 58.1.0

Oh, it was fixed in f4db286dea73fd9abc4c069c0122827fd562c444 and statik excessively pins its deps.


Fix java.lang.ClassCastException

java.lang.ClassCastException: class com.fasterxml.jackson.databind.exc.MismatchedInputException cannot be cast to class org.wildfly.channel.UnresolvedMavenArtifactException (com.fasterxml.jackson.databind.exc.MismatchedInputException and org.wildfly.channel.UnresolvedMavenArtifactException are in unnamed module of loader 'app')
	at org.wildfly.prospero.api.exceptions.ArtifactResolutionException.failedArtifacts(ArtifactResolutionException.java:81)
	at org.wildfly.prospero.cli.ExecutionExceptionHandler.handleExecutionException(ExecutionExceptionHandler.java:64)
	at picocli.CommandLine.execute(CommandLine.java:2088)
	at org.wildfly.prospero.cli.CliMain.main(CliMain.java:54)

Fix java.lang.ClassCastException. It's not necessarily an UnresolvedMavenArtifactException e.g.  The exception has been thrown from https://github.com/wildfly-extras/prospero/blob/1.0.0.Beta4/prospero-common/src/main/java/org/wildfly/prospero/api/ProvisioningDefinition.java#L86

thanks @soul2zimate.
I think https://github.com/wildfly-extras/prospero/blob/1.0.0.Beta4/prospero-common/src/main/java/org/wildfly/prospero/api/ProvisioningDefinition.java#L86 should throw MetadataException instead. We should change the ArtifactResolutionException to only accept UnresolvedMavenArtifactException (or a list of ArtifactCoordinates). WDYT?

oh I didn't see there is MetadataException, it does seem it's the correct one to throw in that case.

@soul2zimate could you rebase this PR against future branch please?

rebased to future branch and resolve all the conflicts. It doesn't compile even I build a local installation-manager-api. Then I see future branch waits for Channel and Manifest split.
Also https://github.com/wildfly-extras/prospero/pull/180 is in main, but it's not in future, Do you need a PR to future for that as well ?


cannot load ！

build github.com/sirupsen/logrus/hooks/syslog: cannot load github.com/sirupsen/logrus/hooks/syslog: no Go source files
Please give me some help, thanks~

import (
"fmt"
log "github.com/sirupsen/logrus"
logrus_syslog "github.com/sirupsen/logrus/hooks/syslog"
"github.com/thinkeridea/go-extend/exnet"
"log"
"log/syslog"
)

what is the issue exactly ?

Not enough information to understand the issue


USD Ar 2 MVP workflow

Business Requirements

We need to demonstrate OpenAssetIO reduces the integrating cost
compared to existing multi-tool workflows with direct
integrations that require parallel implementations using native APIs

We need to provide a solution for coherent asset resolution in DCCs
that consume data from a USD stage as well as their own
documents

USD is rising in popularity in the industry and we need to demonstrate
that OpenAssetIO is not in competition with AR2, is compatible with
its workflows to demonstrate the flexibility of OpenAssetIO's API
design.

The work delivered in the sprint can be parked in a functional state
usable by the community so the team can work on other things. A
source-only mechanism for distribution is fine.

It is ok (though undesirable) for the implementation to be non optimal
if it aids completion of sample workflows.

Implementation is of a suitable quality that works as described when
tested by third parties.

User Requirements
USD authors

As a user I can include OpenAssetIO entity references in USDs
resolvable composition arcs, using Python manager plugins.

References should be in the same form as they would be used in
other situations where OpenAssetIO is used to resolve data assuming
USDs native syntax (@<ref>@<rootPrim>). eg: @ftrack://some/entity@
instead of @openassetio://ftrack???some...@

OpenAssetIO resolved locatableContent trait data is assumed to be an absolute file URL.

Non-entity reference strings are handled by the ArDefaultResolver.

The plugin should use standard USD mechanisms for installation.

The plugin should use my default OpenAssetIO configuration.

Manager authors

The resolver plugin follows OpenAssetIO calling conventions including
opt-out and other safe guards.

Software Requirements
Functional

The OpenAssetIO resolver plugin for AR2 does not declare that it uses a context.

An ArDefaultResolver instance is used to provide "default USD
behaviour" for strings that aren't entity refrences.

The plugin obeys OpenAssetIO API contracts for Manager/Host
interactions for required call sequencing.

The plugin obeys OpenAssetIO API contracts for Manager/Host
interactions for opt-in/out behaviour adaption.

The plugin always reports an invalid ArTimestamp for queries
regarding modification time, indicating OpenAssetIO assets are
immutable.

The plugin makes no additional queries to populate GetAssetInfo
calls.

The plugin errors for unsupported asset-centric functionality:

Write workflows (fatal).

The plugins passes through in CreateIdentifier, ignoring anchor path,
if the reference is an entity reference. (will add getRelatedReferences later)

If a non-assetized asset path is anchored to an assetized anchorPath, the anchorPath is resolved to it's locatableContent trait so that standard ArDefaultResolver behaviour is applied to the non-assetized input.

Non-functional

Community members have instructions on how to obtain, build and run
the plugin from source them selves.

The plugin can be used on CY2022 Linux.

The plugin makes optimal use of API methods where possible to avoid
redundant overhead.

Tasks

[x] #11

Closing.
We never got round to the Unmanaged traits, ManagerPolicy implementation, so have moved into another issue #24


Betterbird.Betterbird version 102.6.0-bb26

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Please see PRs #89294 and #90534 where a similar unexplained error occured and @stephengillie added a waiver for the  Validation-Installation-Error

Please see PRs #89294 and #90534 for previous releases of Betterbird where a similar unexplained error occured and @stephengillie added a waiver for the Validation-Installation-Error

Validation is erroring while validating the Japanese-Japan locale install. It seems to be mismatching caps, searching for ja-ja but finding ja-JA among others:
2022-12-18 11:39:26.681 [CLI ] Installer [X64,exe,Unknown,ja-JA] not applicable: Installer locale does not match required locale: ja-JARequired locales: [ja-ja]

Additionally, there is no Betterbird.Betterbird.locale.ja-JA.yaml nor Betterbird.Betterbird.locale.ja-ja.yaml included in this manifest.

@wingetbot waivers Add Validation-Installation-Error

Validation is erroring while validating the Japanese-Japan locale install. It seems to be mismatching caps, searching for ja-ja but finding ja-JA among others:

If this simple change ja-JA -> ja-ja is the solution, it would be great. I will implement this in future updates.

Additionally, there is no Betterbird.Betterbird.locale.ja-JA.yaml nor Betterbird.Betterbird.locale.ja-ja.yaml included in this manifest.

Unfortunately I don't have any idea of the Japanese language. And while I could partitially rely on my own knowledge and mostly on DeepL and Betterbirds project leader for the European languages the Japanese translators haven't provided a Description yet and no one of us can judge DeepLs translation...


Some tweaks

This PR applies the following changes:

Bump Skylib version
Add a .bazelversion file
Update Examples section in README file

@agoessling Can you review it?


Obtaining value of objective function after solving

This is a feature request and not an issue: I was wondering if there is currently an API-based way (as opposed to looping over all selected nodes and edges and summing individual costs) of obtaining the cost of the objective function which is minimized, post the solving?
If not, this would be a useful feature to have.

I believe that’s what ilpy.Solution.get_value() will give you: https://funkelab.github.io/ilpy/api.html#ilpy.Solution.get_value
I’m away from computer at the moment… so i can’t remember exactly how to get there from the motile Solver, but pretty sure it’s publicly accessible.  Let me know if you can’t find it, or if it’s not what you’re looking for

Thanks Talley.
solver.solution.get_value() works nicely!


Update devicelab bot Xcode to 13.4

Some devicelab iPhones have upgraded to iOS 15.5, which is failing when deploying from Xcode 13.0:
    NSLocalizedDescription = "Failed to prepare device for development.";
    NSLocalizedFailure = "This operation can fail if the version of the OS on the device is newer than the version of Xcode that is running.";
    NSLocalizedRecoverySuggestion = "If you are certain that Xcode supports development on this device, try disconnecting and reconnecting the device.";

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8812991800517855921/+/u/Dismiss_dialogs/Run_app_to_dismiss_dialogs/stdout
iOS 15.2 is working fine with Xcode 13.
Create a new cipd package for Xcode 13.4, including simulator runtimes, and update .ci.yaml to 13f17a
https://github.com/flutter/flutter/blob/c27a1a70f3fa17a5e7c4375bc8f522db920da2c5/.ci.yaml#L57

Xcode 13.4 + iOS 15.5 passes:
https://ci.chromium.org/raw/build/logs.chromium.org/flutter/led/magder_google.com/8052d97b77200b250aff974df17dae0493fa876abcf4e3033feda11d66cd21a1/+/build.proto?server=chromium-swarm.appspot.com


Ensure input components have consistent disabled styling

What
Some input components do not look disabled when tested cross-browser, we should try to add styles to input components to makes sure there is an intentional disabled state that works across browsers.
Why
We don't recommend users disabled their inputs but if they do we want them to be as usable and accessible as possible.
Who needs to know about this
Developers, Chris
Further detail
If possible, contrasts should aim for WCAG guidelines to level AA but this is not a requirement.
See this related issue for more context: https://github.com/alphagov/govuk-frontend/issues/1692#issuecomment-574274312
Done when

[ ] File upload has disabled styling
[ ] Select has disabled styling
[ ] Text input has disabled styling
[ ] Textarea has disabled styling

"I wonder if styling the disabled button with cursor: not-allowed would provide more affordance to the user?"
Originally posted by @jesseyuen in https://github.com/alphagov/govuk-design-system-backlog/issues/34#issuecomment-528365796

Another to add: items in autocomplete have disabled styling (for when enhancing a select). It's common practice to have the first item be disabled and something like Select your foo.

I've been looking into this as a random side thing. As we currently rely on user agent styles for disabled states, we have quite a lot of inconsistency in how they appear across different browsers. In some cases, disabled inputs don't appear visually distinct from enabled ones whatsoever.
Based on checking the most recent browsers (and IE 11):

Browser
Text input/textarea
Select
File upload

Chrome 109 (macOS and Windows)
Reduced contrast on text, light grey background added
Reduced contrast on text only
No visual differences

Safari 16.0 (macOS)
Reduced contrast on text only
No visual differences
Reduced contrast on “Choose file” button only

Safari 16.1 (iOS)
Reduced contrast on text and input border
Reduced contrast on arrowhead icon only
No visual differences

Firefox 108 (macOS and Windows)
Reduced contrast on text only
No visual differences
Reduced contrast on “Browse...” button only

Edge 109 (Windows)
Reduced contrast on text, light grey background added
Reduced contrast on text only
No visual differences

Internet Explorer 11 (Windows)
Reduced contrast on text, light grey background added
Reduced contrast on text and arrowhead icon
Reduced contrast on “Browse...” button only

For how we might style these disabled inputs: we currently only provide explicit disabled styles for checkboxes and radio buttons, as we've deviated from using native controls for them. In this case, we've reduced the opacity of the control by 50% and otherwise made no stylistic changes.
I think the same approach can work for these other inputs, though we also need to reset the background and text colours on some of them to account for different user agents applying different default styles. These styles seemed to work consistently across the board in my informal spike:
/* Text input, textarea */
:disabled {
  opacity: .5; /* overrides UA style - Safari iOS */
  color: inherit; /* reset UA styles - all browsers */
  background-color: transparent; /* reset UA styles - Chromium, IE */
}

/* Select */
:disabled {
  opacity: .5;
  color: inherit; /* reset UA styles - Chromium, IE */
  /* We explicitly _don't_ set a `background-color` for Safari iOS, as this changes the default `appearance` for... reasons? */
}

/* File upload */
:disabled {
  opacity: .5;
}

Naturally, this needs more extensive testing for accessibility, high contrast colour modes, etc. so don't take those as gospel. Just getting thoughts down whilst they're still in my head. 🤓

An MVP version of this has been shipped in #3187, however we're still interested in researching and iterating on disabled styles in the future. A new issue has been opened to that effect:

#3323


Common Collections: Dictionaries

This chapter will be extremely detailed, and will need provide an extensive description of how dictionaries work on the lower level, inspired by the current Cairo docs to describe why dicts need to be squashed.
Estimated time: 3 hours

Can I take this one?

Can I take this one?

yes

Hello @rodrigo-pino ! Where are you on this PR ? do you need help :) ?

Hi @fricoben thanks for your offering! I have all the content in place, I'd just haven't found the time to interweave it together in a descent way. I will make a PR shortly (tomorrow or the day after, tops).
I would appreciate very much reviews and suggestions after that!!!


[web] Update view with sortedUpdate

Fix https://github.com/mitmproxy/mitmweb/issues/36

Awesome. Can we add a test for this and not add the compiled assets in the PR?

Thanks for the changes, but why is ever testfile called *Spec now? How about we make a negative RegEx just excluding the tutils?

I think it's better to have a general method to distinguish test files and utilities, we may have __tests__/utils/*.js sometimes later. :)


development philosophy link is broken in contributing guide

Link href is https://pipenv.pypa.io/en/latest/dev/philosophy/ which returns 404
Issue description
also could not find any "philosophy" related files in repository /docs/*
Steps to replicate
Navigate to https://pipenv.pypa.io/en/latest/dev/contributing/
Find Code Contributions > Steps for Submitting Code > link from first step

The page doesn't exist, so let's remove the link from the docs.


Deepracer model names too long for DRfC after spot interruption

Some users report having continued-YYYMMDD-HHMM after spot interruption is too long for their preferred model naming standard and would prefer a simple approach, e.g. -CNN where NN is a number 01 to 99.

Fixed with the code refactoring branch once it's merged - https://github.com/aws-deepracer-community/deepracer-on-the-spot/pull/128/

Which simply add a '1' to the end if there is no number, or increments the number if the model ends with a number


"Error parsing JSON: Wrong resource type" error when creating a virtual network from Sunstone using advanced mode

Description
The error is shown when creating a virtual network from Sunstone using advanced mode. This happens because the JSON parser expects the cluster_id.
To Reproduce
Create a virtual network from Sunstone using advanced mode
Expected behavior
The network should be created.
Details

Affected Component: Sunstone
Version: 5.6.0

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

Probably the parse_json() function should return the complete parsed JSON and search for the required info inside.