Cannot upgrade from adoptopenjdk-8-openj9-8u252_b09.openj9_0.20.0-1.x86_64 to adoptopenjdk-8-openj9-8u262_b10.openj9_0.21.0-1.x86_64

Please raise an issue at https://github.com/AdoptOpenJDK/openjdk-installer or https://github.com/AdoptOpenJDK/homebrew-openjdk if it's an issue with our brew casks
Resolving Dependencies
--> Running transaction check
---> Package adoptopenjdk-8-openj9.x86_64 0:8u252_b09.openj9_0.20.0-1 will be updated
---> Package adoptopenjdk-8-openj9.x86_64 0:8u262_b10.openj9_0.21.0-1 will be an update
--> Finished Dependency Resolution

Dependencies Resolved

==============================================================================================================================================================================================================================================================================
 Package                                                               Arch                                                   Version                                                                      Repository                                                    Size
==============================================================================================================================================================================================================================================================================
Updating:
 adoptopenjdk-8-openj9                                                 x86_64                                                 8u262_b10.openj9_0.21.0-1                                                    AdoptOpenJDK                                                 109 M

Transaction Summary
==============================================================================================================================================================================================================================================================================
Upgrade  1 Package

Total size: 109 M
Is this ok [y/d/N]: y
Downloading packages:
Running transaction check
Running transaction test

Transaction check error:
  file /usr/lib/jvm/adoptopenjdk-8-openj9/man/ja from install of adoptopenjdk-8-openj9-8u262_b10.openj9_0.21.0-1.x86_64 conflicts with file from package adoptopenjdk-8-openj9-8u252_b09.openj9_0.20.0-1.x86_64

Error Summary
-------------

A Gradle bug leads to a RPM bug, yay! To quote https://rpm.org/user_doc/faq.html:

How can I replace a directory with a symlink or vice-versa?
You dont. If you are desperate, it can be achieved with a %pretrans scriptlet written with the embedded Lua interpreter, but this is fragile at best. The Fedora project has examples and further guidance you might be helpful.

So let's see if I can get a scriptlet into FPM.

Scriptlets must be written in lua but FPM does not allow that, see https://github.com/jordansissel/fpm/issues/1666. There's no fast way around that.
The best workaround I can think of:
rm -rf /usr/lib/jvm/adoptopenjdk-8-openj9/man/ja
yum upgrade

We're in an awkward place:

People that already have the newest versions (8u262, 11.0.8) have the symlink, people with the old versions have the directory. So even if we go back to the directory, we break the people with the symlink.
Replacing fpm right now is not really an option (lot of work, risk of other breakages is high).
Patching fpm is not really an option (would require fork, testing facilities do not seem to be in good shape).
Pretrans is only possible with a shell script, would break people in Kickstart scenarios.
Creating the symlinks in the post install script does not work because the removal of the old package during an update would remove the symlink.

So, the best option (read: lowest impact) I've come up with: Remove the offending directory/symlink man/ja entirely from the package. This works for everyone, the Japanese man pages are still there (but in a different directory) and aren't included in 14 and newer anyway. We can still bring them back in the future.
There's an open PR that fixes the problem. As soon as it's merged, we'll publish new packages.

The new packages for 8u262 are in. 11.0.8 is currently being uploaded. Might need yum update --refresh to get the -2 package.

I can confirm this has resolved my issue. Thank you for the quick turnaround
`[lcollins@uk-lon01-vps001 ~]$ sudo yum update
Loaded plugins: changelog, elrepo, fastestmirror, product-id, search-disabled-repos, subscription-manager
This system is not registered with an entitlement server. You can use subscription-manager to register.
Loading mirror speeds from cached hostfile

base: mirror.sov.uk.goscomb.net
centos-sclo-rh: mirror.sov.uk.goscomb.net
centos-sclo-sclo: mirror.sov.uk.goscomb.net
elrepo: ftp.nluug.nl
extras: uk.mirror.nsec.pt
remi: mirror.netweaver.uk
remi-php72: mirror.netweaver.uk
remi-safe: mirror.netweaver.uk
rpmforge: mirrors.ircam.fr
updates: mirror.as29550.net
AdoptOpenJDK                                                                                | 1.4 kB  00:00:00
CodeIT                                                                                | 2.9 kB  00:00:00
CodeITmainline                                                                                | 2.9 kB  00:00:00
ansible                                                                                | 2.9 kB  00:00:00
as-repo-centos7                                                                                | 2.9 kB  00:00:00
base                                                                                | 3.6 kB  00:00:00
centos-sclo-rh                                                                                | 3.0 kB  00:00:00
centos-sclo-sclo                                                                                | 3.0 kB  00:00:00
copr:copr.fedorainfracloud.org:jdoss:wireguard                                                                                | 3.3 kB  00:00:00
cr                                                                                | 2.9 kB  00:00:00
datadog                                                                                | 2.5 kB  00:00:00
digitalocean-agent                                                                                | 3.3 kB  00:00:00
docker-ce-stable                                                                                | 3.5 kB  00:00:00
elrepo                                                                                | 2.9 kB  00:00:00
epel                                                                                | 4.7 kB  00:00:00
extras                                                                                | 2.9 kB  00:00:00
jenkins                                                                                | 2.9 kB  00:00:00
lynis                                                                                | 2.5 kB  00:00:00
mysql-connectors-community                                                                                | 2.5 kB  00:00:00
mysql-tools-community                                                                                | 2.5 kB  00:00:00
mysql57-community                                                                                | 2.5 kB  00:00:00
nodesource                                                                                | 2.5 kB  00:00:00
remi                                                                                | 3.0 kB  00:00:00
remi-php72                                                                                | 3.0 kB  00:00:00
remi-safe                                                                                | 3.0 kB  00:00:00
remi-test-debuginfo                                                                                | 3.0 kB  00:00:00
rpmforge                                                                                | 1.9 kB  00:00:00
rundeck-release-bintray                                                                                | 1.3 kB  00:00:00
updates                                                                                | 2.9 kB  00:00:00
yarn                                                                                | 2.9 kB  00:00:00
(1/5): epel/x86_64/updateinfo                                                                                | 1.0 MB  00:00:00
(2/5): remi-safe/primary_db                                                                                | 1.7 MB  00:00:00
(3/5): remi/primary_db                                                                                | 2.7 MB  00:00:00
(4/5): epel/x86_64/primary_db                                                                                | 6.9 MB  00:00:00
(5/5): AdoptOpenJDK/7/x86_64/primary                                                                                |  34 kB  00:00:01
AdoptOpenJDK                                                                                138/138
[elrepo]: 46 kmod packages excluded due to dependency errors
Resolving Dependencies
--> Running transaction check
---> Package ImageMagick.x86_64 0:<IP_ADDRESS>-1.el7.remi will be updated
---> Package ImageMagick.x86_64 0:<IP_ADDRESS>-1.el7.remi will be an update
---> Package ImageMagick-libs.x86_64 0:<IP_ADDRESS>-1.el7.remi will be updated
---> Package ImageMagick-libs.x86_64 0:<IP_ADDRESS>-1.el7.remi will be an update
---> Package adoptopenjdk-8-openj9.x86_64 0:8u252_b09.openj9_0.20.0-1 will be updated
---> Package adoptopenjdk-8-openj9.x86_64 0:8u262_b10.openj9_0.21.0-2 will be an update
--> Finished Dependency Resolution

Dependencies Resolved
======================================================================================================================================================================================================
Package                                             Arch                                 Version                                                    Repository                                  Size
Updating:
ImageMagick                                         x86_64                               <IP_ADDRESS>-1.el7.remi                                       remi                                        90 k
ImageMagick-libs                                    x86_64                               <IP_ADDRESS>-1.el7.remi                                       remi                                       2.4 M
adoptopenjdk-8-openj9                               x86_64                               8u262_b10.openj9_0.21.0-2                                  AdoptOpenJDK                               109 M
Transaction Summary
Upgrade  3 Packages
Total download size: 112 M
Is this ok [y/d/N]: y
Downloading packages:
No Presto metadata available for remi
No Presto metadata available for AdoptOpenJDK
(1/3): ImageMagick-<IP_ADDRESS>-1.el7.remi.x86_64.rpm                                                                                |  90 kB  00:00:00
(2/3): ImageMagick-libs-<IP_ADDRESS>-1.el7.remi.x86_64.rpm                                                                                | 2.4 MB  00:00:00
(3/3): adoptopenjdk-8-openj9-8u262-b10.openj9-0.21.0-2.x86_64.rpm                                                                                | 109 MB  00:00:30
Total                                                                                3.7 MB/s | 112 MB  00:00:30
Running transaction check
Running transaction test
Transaction test succeeded
Running transaction
Updating   : ImageMagick-libs-<IP_ADDRESS>-1.el7.remi.x86_64                                                                                1/6
Updating   : ImageMagick-<IP_ADDRESS>-1.el7.remi.x86_64                                                                                2/6
Updating   : adoptopenjdk-8-openj9-8u262_b10.openj9_0.21.0-2.x86_64                                                                                3/6
Cleanup    : ImageMagick-<IP_ADDRESS>-1.el7.remi.x86_64                                                                                4/6
Cleanup    : ImageMagick-libs-<IP_ADDRESS>-1.el7.remi.x86_64                                                                                5/6
Cleanup    : adoptopenjdk-8-openj9-8u252_b09.openj9_0.20.0-1.x86_64                                                                                6/6
Verifying  : ImageMagick-<IP_ADDRESS>-1.el7.remi.x86_64                                                                                1/6
Verifying  : adoptopenjdk-8-openj9-8u262_b10.openj9_0.21.0-2.x86_64                                                                                2/6
Verifying  : ImageMagick-libs-<IP_ADDRESS>-1.el7.remi.x86_64                                                                                3/6
Verifying  : adoptopenjdk-8-openj9-8u252_b09.openj9_0.20.0-1.x86_64                                                                                4/6
Verifying  : ImageMagick-libs-<IP_ADDRESS>-1.el7.remi.x86_64                                                                                5/6
Verifying  : ImageMagick-<IP_ADDRESS>-1.el7.remi.x86_64                                                                                6/6
Updated:
ImageMagick.x86_64 0:<IP_ADDRESS>-1.el7.remi                  ImageMagick-libs.x86_64 0:<IP_ADDRESS>-1.el7.remi                  adoptopenjdk-8-openj9.x86_64 0:8u262_b10.openj9_0.21.0-2
Complete!
[lcollins@uk-lon01-vps001 ~]$`


restrict param autocomplete to only string & number, and respect param type in autocomplete shape

https://staging.coda.io/d/_dL5PQJ5pk51#_tu43U/r462&modal=true
https://golinks.io/bug/20266
autocomplete was using a vague type of string | number. This PR makes it more strict based on the param type.

hold on. I need to piggyback more into this..

There's a separate ticket of fixing return type of functional autocomplete.
It actually takes more work than I expected. Will do that in a separate PR.


add detection for author pages

Just crawled default WP where I learned following pages goes missing namely

Author
Date archives
Tag Archives
RSS Feed + Comment Feed

http://example.com/wp-login.php
text/html
404
Not Found
Non-Indexable

http://example.com/comments/feed/
text/html
404
Not Found
Non-Indexable

http://example.com/category/server/
text/html
404
Not Found
Non-Indexable

http://example.com/author/admin/
text/html
404
Not Found
Non-Indexable

http://example.com/feed/
text/html
404
Not Found
Non-Indexable

http://example.com/tag/cloud/
text/html
404
Not Found
Non-Indexable

http://example.com/2020/06/
text/html
404
Not Found
Non-Indexable

http://example.com/
text/html
200
OK
Indexable

http://example.com/digitalocean/
text/html
200
OK
Indexable

http://example.com/linode/
text/html
200
OK
Indexable

http://example.com/wp-content/themes/twentytwenty/assets/js/index.js?ver=1.2
application/javascript
200
OK
Indexable

http://example.com/wp-content/themes/twentytwenty/print.css?ver=1.2
text/css
200
OK
Indexable

http://example.com/page/2/
text/html
200
OK
Indexable

http://example.com/page/3/
text/html
200
OK
Indexable

http://example.com/wp-includes/css/dist/block-library/style.min.css?ver=5.4.1
text/css
200
OK
Indexable

http://example.com/vultr/
text/html
200
OK
Indexable

http://example.com/wp-content/themes/twentytwenty/style.css?ver=1.2
text/css
200
OK
Indexable

http://example.com/case-study/
text/html
200
OK
Indexable

http://example.com/case-study/cloud/
text/html
200
OK
Indexable

is this for V7 or that work in progress build of V6?

7.0-alpha-007

Hi, I have same problem. I missing all /tag pages.

Authors, Tags and pagination for both of them should be working fine now.


[SPARK-41427][UI][FOLLOWUP] Remove duplicate getMetricValue from ExecutorMetricsSerializer#serialize

What changes were proposed in this pull request?
This pr aims remove duplicate getMetricValue from ExecutorMetricsSerializer#serialize.
Why are the changes needed?
Delete duplicate operations
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass Github Actions

Thanks, merging to master

Thanks @gengliangwang


adding photo to the proposals

related to issue #53

Coverage decreased (-0.12%) to 91.28% when pulling 02b3a92b6d2dd8601e887f37f476a3b8c1cd1b40 on humrochagf:photo_on_proposal into 5bc7b3d3d9115a7b4698ffff29515cf41c566ea5 on luanfonceca:master.

There are any reason for not using that all auth profile picture, when the user is logged by all auth?
Thanks in advance!

I think i don't really get what you are reffering to it tries to get the authors social media first. Then, if don't have any social media, falls back into gravatar photo.

Ohh, sorry i messed up :D
I was confused that method called get_user_photo  and the get_avatar_url . Is everything right with your PR!
I am merging it right now.


ENH: Added FakeEpicsPathSignal for testing

See #1041

Adds a FakeEpicsPathSignal to allow for testing device logic that references an EpicsPathSignal

I'll merge this now- all the tests are passing and everything looks to be in order


[Suggestion] Frame Furniture

Hey there!
I would like to suggest the addition of Frame Furniture, such as chairs, tables, cabinets... and maybe even pillars, although that doesn't really fall under furniture.
The reason for my suggestion is that I just had a look at the mod Builders Crafts & Additions, and liked it, but thought that having to add several new blocks for each material was overkill, especially if you introduce modded blocks to the mix.
I hope you'll consider it :)

Woah! That's cool. I'll keep that in mind, this idea is fantastic. I'm already working on a second mod based on BlockCarpentry, together with Kiro (creator of Mo'Blocks), that adds some block types like ramps, arches and pillars, but only with a decorative purpose and no functionality. Sadly, I have very few time right now, because the next semester has begun last week. I like the idea and will work on it, when I have time (which wont be until christmas, I guess). Thanks for the inspiration!
Regards, Manu

Cool, good luck/have fun with school :)


Tweak mkdocs

Tweaked a couple of things. Spelling, copyright date, and maybe port #

Closing this one, as in the documentation push, we're going to be nuking the default drf-cookiecutter generated mkdocs file, and starting from scratch.


Add FastFlow model training testing inference via Anomalib API

Description

This notebook demonstrates how to train FastFlow model, test and deploy afterwards using the anomalib api.

Fixes #258

Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[ ] My code follows the pre-commit style and check guidelines of this project.
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing tests pass locally with my changes

Since benchmarking.ipynb is renamed, README should be updated to point to the new location. Also, it would be nice to add falsflow notebook to the README as well.


[Sample spectra] Renishaw files

Hi Francois
I wasn't able to contact you by email since it was rejected. Simply sending over issues now
Hi Francois
Thanks for your interest! Recently I got some spare time to improve the parsing abilities. Currently the package (0.1.2) should work without problem for 1D and 2D spectra but image extraction and super-positions are still needed to be added. You may want to check the newer versions since the old ones may be bad about documentation etc.
In case you want to test, I have some sample spectra from an old project that I used for demonstration of renishawWiRE.
https://github.com/alchem0x2A/py-wdf-reader/releases/download/binary/spectra_files.zip
Also, many appreciated if you can provide pull requests / issues / suggestions !
Best
TT

Hi Tran,
many thanks for your reactivity. Your package works fine . I have uploaded a Jupyter example based on your package
https://github.com/Bertin-fap/raman-hyperspectra-examples/blob/master/Check RENISHAW hyperspectra.ipynb
It works fine. Nevetheless I find it a little bit difficlt to decipher your types class. In particular I didn't succed in fiding the x, y, lbd unit, so I lousely fix them to µm and cm-1. Could you help me with this issue
Best
François

Thanks for testing out!
There is still work in progress to set the correct unit output, for most of my samples your assignment of μm and cm^-1 are correct.
The xy-pos information can now be retrieved in WDFReader.origin_list_header, where the first 2 items contain the x and y position units etc.
The spectral units are bit different, which now I read as UnitType(WDFReader.xlist_units). It gives me a RamanShift instance, which supposedly is cm^-1.  In my case I didn't have other types of spectral data than this. Probably you can check with your files as well.

Base on your tip, to retrieve the unit I used this rather akward and unreadeable two lines of code
Unit = {types.UnitType.members[x]:x for x in types.UnitType.members}
unit_x = Unit[data.origin_list_header[0][2]]
Could you suggest a better idea?

unit_x = Unit[data.origin_list_header[0][2]]

I think depending on how you want to output the value of unit? If you just want the normal name then data.origin_list_header[0][2].name gives you "Micron". Otherwise you can always use dict conversion
unit_dict=dict(Micron="μm")
x_unit = data.origin_list_header[0][2]
print("unit of x: ", unit_dict[x_unit.name])

I've made a PR to my package so that starting from version 0.1.5 all the UnitType  etc instances can be converted to a conventional name string by the __str__ method, i.e.:
print("X coordinate unit", str(data.xpos_unit))
print("Y coordinate unit", str(data.ypos_unit))
print("Raman x-data unit", str(data.xlist_unit))

which gives:
X coordinate unit μm
Y coordinate unit μm
Raman x-data unit 1/cm

Nevertheless, I appreciate if you suggest some better naming for these instances. Currently the original naming by Renishaw is bit messy

Thanks a lot for your answer I will use your method.
A last point the warning "Block name WMAP not present in current measurement"  is not understandable for a non expert . It could be suppressed or more explicit.
I think you have to stick with Renishaw notations.

A last point the warning "Block name WMAP not present in current measurement" is not understandable for a non expert . It could be suppressed or more explicit.

Thanks for the input. I made a quick PR to this issue so that only if you create the WDFReader instance with debug=True, will these information output to stderr

ce![image](https://user-images.githubusercontent 仍然r.com/6829706/92305026-95ba7300-ef83-11ea-82e5-a48b2e86367f.jpeg)yicchuxi。yuyxicuolo


How layers are copied to reference application examples

The Problem To Be Solved
Reference application examples require a board (and shield) layer to be complete. We should therefore define how layers are copied to a project.
Suggest A Solution
In PoC project NXP_Sensors_SDK the layers are stored in the directories "Boards" and "Shields".
My proposal is therefore to use the layer type as base directory name, i.e. "Board" and "Shield" and as sub-directory the board name (in case of multiple layers for a board with some additional identifier).  For example, "Board/B-U585I-IOT02A-WiFi/Board.clayer.yml". In the directory "Board/B-U585I-IOT02A-WiFi" the source code and configuration settings of the layer would be stored.
As a consequence, in the PDSC file the layer would be registered as:
<clayer name="B-U585I-IOT02A IoT-WiFi" type="Board" file="Board/B-U585I-IOT02A-WiFi/Board.clayer.yml">

When a user wants to copy the layer to an example project (or a set of example projects), the tool would copy the complete content of the directory "Board/B-U585I-IOT02A-WiFi" to the reference application.
As an example for an USB Device example set the directory may be structured as:
USBD.csolution.yml                                        # the csolution that contains USB Device examples
USBD_HID/HID.cproject.yml                                 # project that demos HID
USBD_MS/MassStorage.cproject.yml                          # project that demos MassStorage
Board/B-U585I-IOT02A-WiFi/Board.clayer.yml                # a board layer that connects it to hardware
Board/MCX1423/Board.clayer.yml                            # another board layer that connects it to hardware

This concept ensures that multiple boards can be configured without using multiple directories for projects.

The additional attributes path and copy-to (see: https://github.com/Open-CMSIS-Pack/Open-CMSIS-Pack-Spec/issues/305) allow to specify the base directory to be copied (path) and the destination directory (copy-to)
cp -r ${CMSIS_PACK_ROOT}/<vendor>/<name>/<version>/<path>  $SolutionDir()$/<copy-to>

I consider this as completed with CMSIS-Toolbox 2.4.0


Fix Html5 Shiv URL.

Fix Html5 Shiv URL HTTP 404.

Fixes #65

fixed here https://github.com/olamy/reflow-maven-skin/pull/11


[objc] Enable "-sim" builds to not force symbols to be undefined

Scenario
I have a static fat lib linked against a Xamarin.iOS managed lib. Run objcgen against it.
Observed
objcgen successfully outputs a .framework linked against the iPhoneOS SDK. Fails to build a .framework against iPhoneSimulator on the final link step when it forces external symbols to be undefined (not done during the iPhoneOS final link step).
Expected
objcgen to successfully output frameworks for both SDK's.
Notes
This seems like nominally an issue with mtouch, as the mtouch arguments passed in by objcgen are nearly identical in both cases. I notice in these commits that whether or not mtouch passes -u symbol for symbols referenced in P/Invoke calls can be determined indirectly of the parameters pass to mtouch.
I'm hoping that there is a perhaps one or more arguments to be passed to mtouch to suppress this for iPhoneSimulator builds as well (at least until #520 is resolved).

I think you're right, this can be solved with adding an argument somewhere.
But first I'd need to fully understand what's happening, and for that I'd need at least a complete build log, and if possible also a test project.

Hello @zgramana, we have not received the requested information. If you are still experiencing this issue please provide all the requested information then click the Reopen Issue button. Thanks!


SPS SW & Crop Name errors

Describe the bug
Vidhya getting Water Budget data for SPS regions, resulted in SW error and Crop Name not matching error.
To Reproduce
If using a custom shapefile please upload ZIP of the same to project google drive folder GISLayers and add link below.
Steps to reproduce the behavior:

Select the feature using select tool
in Water budget tab under agriculture - click calculate
in water availability tab under surface water - click calculate

Expected behavior
Water Budget Calculation - get amount of water required for crop
Water Availability Calculation - get Surface water available and Map of the SW layer
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS Name and Version : Windows 11
QGIS Version:  3.22

Additional context
2023-02-20T12:05:39     WARNING    Traceback (most recent call last):
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\jaltol\gui\jt_waterbudget_widget.py", line 205, in initSWaterAvailability
self.popSWA()
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\jaltol\gui\jt_waterbudget_widget.py", line 258, in popSWA
WaterBodiesS1 = self.S1.calcNoWaterBodies(self.year,self.geometry)
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\jaltol\libs\jaltol_surfacewater.py", line 116, in calcNoWaterBodies
Map.addLayer(self.s1EMwbv,{},'Water Bodies Vector')
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\ee_plugin\Map.py", line 25, in addLayer
add_or_update_ee_layer(eeObject, visParams, name, shown, opacity)
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\ee_plugin\utils.py", line 158, in add_or_update_ee_layer
update_ee_layer_properties(layer, eeObject, visParams, shown, opacity)
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\ee_plugin\utils.py", line 26, in update_ee_layer_properties
layer.dataProvider().set_ee_object(eeObject)
AttributeError: 'QgsRasterDataProvider' object has no attribute 'set_ee_object'
2023-02-20T12:06:23     WARNING    Traceback (most recent call last):
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\jaltol\gui\jt_waterbudget_widget.py", line 358, in initCropWaterRequirement
self.calcCropWaterRequirement()
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\jaltol\gui\jt_waterbudget_widget.py", line 371, in calcCropWaterRequirement
self.crop_details = self.crops_dict.get_crop_details(5) # 5 is no. of crops needed for a season
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\jaltol\libs\jaltol_cropwaterrequirement.py", line 86, in get_crop_details
self.get_kc_list(row['crop']),
File "C:\Users/ATREE/AppData/Roaming/QGIS/QGIS3\profiles\default/python/plugins\jaltol\libs\jaltol_cropwaterrequirement.py", line 146, in get_kc_list
return kc_dict[crop]
KeyError: 'Moong(GreenGram)'

@craigdsouza  replace the crop_details.py with the https://github.com/atreebangalore/jaltol/blob/main/libs/crop_details.py
this should solve the errors due to crop names.

@craigdsouza  the surface water error is due to adding it as a layer using Map.addLayer, error may be due to non availability of the SW in the region, comment out the Map.addLayer to ignore the error.


Speculative Decoding State Reversion after Rejection

Hi, I've been working with the speculative decoding implementation in the provided codebase. I'm curious about the output sequence out when the predictions made by the smaller model (small_net) are rejected by the larger model (net). Upon a rejection, we should revert the state of out to its condition prior to the rejected predictions.  However, it appears that the current implementation of speculative_decoding function does not perform this state reversion.
Say if the original seq_len is 128 and gamma is 5, if all of them are rejected then in this iteration the seq_len should be 128+1, but not 128+5+1.

Second this - it seems to me like this is a major bug in the current implementation. Would be awesome if you could fix this!

@j-geuter sorry i missed this issue
it has been a while, but on a quick glance, i thought this line of code should do it?

Hey! Doesn't seem to me like it - that only determines how many more samples you generate right? But the rejected samples are still added to out. You'd have to make sure only accepted samples are added to out. (In particular, in the batch setting, the lengths of the decoded sequences will diverge, and in each iteration, you'll have to ensure the small model starts decoding each sequence at the correct index.)

@j-geuter oh i see
barely remember my thought process, but think i attempted to take care of this here during the right alignment, but fail to do so for batch size of 1

@j-geuter let me stare at this a bit more tomorrow morning and get it fixed, thank you!

To be fair I didn't really look at the if-clause for batch_size>1, seems like that could be handling it correctly for larger batches. Just noticed for batch_size=1 it doesn't.

Ok thank you!

@j-geuter actually, want to just check to see if the latest change fixes things?

Yes thanks, for batch_size=1 this looks correct now! (I didn't check for batch_size>1 though)

nice! yea, i think batch size > 1 is accounted for here, but feel free to reopen if it breaks when you increase the batch size

@j-geuter working ok?

@lucidrains hi, I test the current version on my project. For batch_size=1 it works fine. For batch_size > 1, there seem to be some minor issues. Specifically, I believe the line here should be:
seq_offset_indices = seq_arange - num_rejected[..., None]

given out is the result of concatenating all gamma tokens with the previou one. After making this modification, the generation results seem to be normal.
I'm not sure if I misunderstood your code, but could you please take another look?

@breaddaerb i don't think that's quite right, as that would push the sequence into negative indices
could not understand my thought process from a year agop, so i completely redid it with my updated skillset..
seems to work fine, could you double check for me? thank you

don't even think batch size > 1 is necessary for research on this topic, more of something industrial labs would care about

@lucidrains I test it on my project and it works fine! Agree that batch_size==1  is probably enough for research purposes lol.

@breaddaerb nice! yeah, had a real "wtf did I write" when reviewing that logic this morning
good luck with your research


Add graphics/screenshots of the buttons in Readme

Add 2 or 3 GIFs to README showcasing the buttons.
Discussed in https://github.com/sButtons/sbuttons/discussions/1631

Originally posted by reev-create June 27, 2022
It would be great and attractive to see some screenshots or previews of the buttons designs in the Readme file

I'll soon add the graphical view in the Readme file

I've assigned you @reev-create

@reev-create are you still working on this?

is this issue still being worked on by assignee ?
If no then I would like to make a contribution.
I am planning to add preview images/gifs next to sample source codes in the Usage section of readme.md

This is a sample how it should look on README.md :

I am currently experimenting with the this on my fork's preview_patch branch where you can see how the sample result might look on your particular device/theme.

Hello @Arsenic-ATG this sounds great! I'll assign you

@shahednasser
After some playing around with different layouts, I shortlisted two of them
I went ahead and applied the changes on 2 different branches of my fork following the different approaches to see the complete effect it would have on the README.md, and I could really use some help choosing the final design amongst them. 😅
Type 1 (using tables )
Completely implemented in preview_patch branch of the fork

 Preview  Source

<button class="animated sbtn atom-btn blue-btn">Animated Button</button>

Type 2 (using details )
Completely implemented in alternative_preview branch of the fork
<button class="animated sbtn atom-btn blue-btn">Animated Button</button>

Preview

Thank you for all your work! I think type 1 is the better approach.

hi, I also think it's a good idea. Is it possible for me to help you?

Is the issue still open?

Is this issue still open?

@DotSlash-A ,looks like either all the maintainers of the repository are busy or the project has been abandoned by the maintainers.
I also opened a PR regarding the same over a month ago (#1643) but never heard from the maintainers since then, and it got closed due to lack of interactions by the bot.
I would recommend against wasting your time and effort trying to submit patches here.
In case you still want the project to grow and ready to take the initiative, then it is much better to maintain a separate fork of the project instead ( though make sure to mention this original repo if you are planning to do so )

Is this issue still open?


Fix load_regridder docstring

load_regridder now works for any regridder type, fixes docstring to reflect this.

Thanks Stephen, this makes sense to me.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.39%. Comparing base (3aa967d) to head (9a9b4ab).
Report is 8 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #404   +/-   ##
=======================================
  Coverage   98.39%   98.39%
=======================================
  Files          37       37
  Lines        4293     4293
=======================================
  Hits         4224     4224
  Misses         69       69

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Upgrade versions of numpy e.a.

Upgrade versions of several packages, including numpy (1.23.5).

@rogervankanten was there a reason for changing from pymesh.n_faces to pymesh.n_faces_strict?

This warning comes from the fpdf2 package that we are using. And it is a known limitation. We probably cannot do much about it.
@rogervankanten Thanks for upgrating the Numpy version.  @aikkala Thanks for checking.  Will merge this pull request.


ClientModel: Clean up after removing types in .Internal.*

API change check
APIView has identified API level changes in this PR and created following API reviews.
System.ClientModel


fix: get correct report period due date

Introduction
The next due report period due date was being computed as the tenth day of the reporting period month which isn't correct.
Resolution
Updates the controller and relevant helper functions to compute the correct report period due date.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main-stbr@1d920ed). Click here to learn what that means.

Additional details and impacted files
@@             Coverage Diff              @@
##             main-stbr    #2349   +/-   ##
============================================
  Coverage             ?   12.18%
============================================
  Files                ?      382
  Lines                ?     8634
  Branches             ?     1716
============================================
  Hits                 ?     1052
  Misses               ?     6228
  Partials             ?     1354

Flag
Coverage Δ

unit-test
12.18% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


SP-2292 - Add Version-Timestamp main build docker tag #minor

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.50%. Comparing base (4a90ca9) to head (7e8d749).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #429   +/-   ##
=======================================
  Coverage   79.50%   79.50%
=======================================
  Files          35       35
  Lines        1059     1059
=======================================
  Hits          842      842
  Misses        156      156
  Partials       61       61

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Skript doesn't work with patient ressources which have the Link attribute.

@NamUKF please specify further and give an example as the issue cannot be understood otherwise


Horovod: adjust base LR used by schedulers to scale with the number of workers

In #2574 it was observed that the learning rate used to initialize learning rate schedulers is in conflict with the way we scale the learning rate with the number of Horovod workers.  Because the learning rate schedulers are initialized before the scaling, the scaling would be overridden.  This PR fixes this so that optimizers and LR schedulers scale up correctly.
Follow-ups to consider would be:

Make scaling work for other backends like DDP.
Add an option to configure the LR scaling in the Trainer.

@williamFalcon @Borda failing tests seem unrelated.  Could you take a look at the PR and see if everything makes sense?
As a follow-up, I want to add a param to Trainer so we can override the LR scale when using Horovod/DDP.

@williamFalcon @Borda failing tests seem unrelated. Could you take a look at the PR and see if everything makes sense?

yes, no connection to this PR

Hello. I was curious if there have been any other PRs that have added the ability for users to specify which learning rate scaling strategy should be used with e.g., DDP/Horovod.


How to render video assets (not embeds)

Thank you for this great project, I learned a lot trying to RichTextPageContent to work! I could not however get a video asset to render in the page - by that I mean a video that is uploaded to Contentful as an Asset, and added as an Entry to a post. It looks like the component assumes that every embedded asset is to be rendered as an image tag, could that be the reason?
https://github.com/whitep4nth3r/nextjs-contentful-blog-starter/blob/3b14f516b9cfd8a7b666d0a02d37626c4a063d53/components/RichTextPageContent/index.js#L166
Did I miss something? The error is that the code is treating the video as an image.

Yes that's probably the reason!
You can look at the contentType of the asset, and render that accordingly.
This might help: https://whitep4nth3r.com/blog/rich-text-field-tips-and-tricks/#rendering-video-assets-not-iframe-embeds-in-rich-text

you have an answer for everything :) 👍  thank you for your prompt reply, I got to a similar solution, but using react-player - struggled a bit with styling but it works. thanks again @whitep4nth3r !

You're welcome, and good luck with the project!


[APM] rename trace_root type to trilean

What does this PR do?
In the backend, we renamed the field that is_trace_root uses from TraceRootFlag to Trilean since it can only have three possible values: TRUE, FALSE, NOT_SET. The purpose of this PR is to maintain consistency in the naming between the agent payload field and the backend payload field

Motivation
The Trilean field name is better for shared code purposes. We might want to create other variables that implement this type

Additional Notes

Possible Drawbacks / Trade-offs
N/A

Describe how to test/QA your changes
This is not a new feature, no new tests need to be created. I have changed is_trace_root field's existing e2e tests to use the new enum name.

/merge


Convert Primefaces to Angulerfaces

Hi,
Please let me know, how to convert following code in AngularFaces.
<p:inputText value="#{cc.attrs[cc.attrs.fieldName]}"/>

And also let me know conversion in composite components.

I'm afraid that's not possible :(. I'd have to investigate a lot to support composite components. But AngularFaces isn't used by many developers, so currently I prefer to invest this time in BootsFaces. When Angular2 get final, the situation may change again.


PATCH-3: Local Image Opimization Service

Localhost Sharp instance to simulate the image optimization service behavior
Force webp format on the optimization service
Separates images imports from .generated/assets-imports.ts imports for better typing
Listens assets folder for changes and re-runs scripts:imports
Added webpack support to GLB, Wasm, Rive and other common creative files
Added comments to make it clear the difference between the View and Controller props

[AUTOMATED] Latest version has been pushed to preview.
Preview deployed at https://preview-0e29665.dev.nextjs-boilerplate.jam3.cloud.jam3.net


Make landing page reactive to screen sizing

Main landing page can be styled to react to screen resizing or mobile native screen size. Follow the tailwindCSS docs for more info:
Two approaches:
(1) https://tailwindcss.com/docs/responsive-design
(2) https://tailwindcss.com/docs/screens#using-custom-screen-names

Closing out this outdated Ticket 1/12/2023


Weighted Variable Sum/Average Query results in inf/nan where sum produces a

On the attached data set the Variable Sum query returns a reasonable value while Weighted Variable Sum/Average return inf/nan. This seems to be a numerical issue. If I scale the the spatial extents by 1e-20 in all directions, I do get "actual floating point" values.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1648
Status: Resolved
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Weighted Variable Sum/Average Query results in inf/nan where sum produces a
Assigned to: Kathleen Biagas
Category: -
Target version: -
Author: Gunther Weber
Start: 11/07/2013
Due date:
% Done: 0%
Estimated time:
Created: 11/07/2013 02:27 pm
Updated: 09/16/2014 11:34 am
Likelihood: 3 - Occasional
Severity: 4 - Crash / Wrong Results
Found in version: 2.7.0
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
On the attached data set the Variable Sum query returns a reasonable value while Weighted Variable Sum/Average return inf/nan. This seems to be a numerical issue. If I scale the the spatial extents by 1e-20 in all directions, I do get "actual floating point" values.
Comments:
If you tell VisIt to use 'double' instead of 'native' precision, then the NAN's disappear. (Options->Preferences). Thanks for the information. This problem actually occurs in context of the Chombo reader. The VTK data set was mainly meant as simple reproducer.I checked the Chombo reader and while it reads data as double, it is passing coordinate arrays as float. Is there any reason for me not to apply the following patch to the 2.7RC?Also, what is our convention about doing these conversions in the reader vs. later in the pipeline? Does VisIt pass to the reader what type of precision is desired? Should I use this information to create float vs. double coordinates?Index: avtChomboFileFormat.C =================================================================== avtChomboFileFormat.C (revision 22388) avtChomboFileFormat.C (working copy)55,7 55,6 #include <vtkCellType.h> #include <vtkDoubleArray.h> #include <vtkFieldData.h>#include <vtkFloatArray.h> #include <vtkInformation.h> #include <vtkIntArray.h> #include <vtkPolyData.h>2435,15 2434,15 vtkRectilinearGrid **rg = vtkRectilinearGrid::New(); rg>SetDimensions(dims); vtkFloatArray **xcoord = vtkFloatArray::New(); vtkFloatArray **ycoord = vtkFloatArray::New(); vtkFloatArray **zcoord = vtkFloatArray::New(); vtkDoubleArray **xcoord = vtkDoubleArray::New(); vtkDoubleArray **ycoord = vtkDoubleArray::New(); vtkDoubleArray **zcoord = vtkDoubleArray::New(); float **ptr = xcoord>GetPointer(0); double **ptr = xcoord>GetPointer(0); if (!allowedToUseGhosts) ptr0 = probLo0 + lowI[patch]**dx[level][0]*aspectRatio0; The preference isn't sent down to the readers, though it might be possible to determine by using avtDatabaseFactory::GetPrecisionType(). I think the reader should work with the data's native precision. VisIt's transform manager will handle converting down to float or up to double as needed by the users' set preference. Thanks. I just committed an update to the Chombo reader that makes the precision of data and coordinates consistent. This resolves the problem for my use case. Are there any other potential problems due to this behavior or can we close the bug report with the Change to the Chombo reader? marking this as resolved, as Gunther's specific issue was addressed when he modified the Chombo reader.There is still the overarching problem of unreported Floating Point Exceptions (underflow/overflow/divbyzero), but there is a ticket for this (#1345).

1655_average_problem.vtk.zip
1655_script.py.zip


Trust the installation of celery[sqs] to handle its dependencies

Currently installing boto3 to solve #14, but I should get rid of it as soon as they fix the dependency listing.

The change was merged into master at Celery already, waiting for a release.

Done in c281091c973f8c49006f0b7f579d1c108e1883c2


Contribute a Kotlin Couchbase Starter Kit

Looking to participate in Hacktoberfest 2024? 👻
Here's a fantastic way to contribute to Couchbase's open-source ecosystem by creating a Kotlin Starter Kit!
Couchbase already has starter kits for frameworks like Spring and Rails, but we're missing one for Kotlin! This is where YOU can make a difference. Help us build a Kotlin Couchbase starter kit to assist other developers, like yourself, in easily integrating Couchbase into their Kotlin applications.
What You’ll Do:

Set up a basic Kotlin app with the framework of your choice that connects to a Couchbase cluster.
Use the Couchbase Kotlin SDK to demonstrate key functionalities.
Include a sample .env file with necessary environment variables for connecting to Couchbase.
Create a README.md with setup instructions.
Configure deployment support for Gitpod and GitHub Codespaces.
Add a test that verifies the connection to Couchbase using GitHub Actions.

Basic Requirements:
Your Kotlin starter kit should include:

[ ] Connection to Couchbase: With both Couchbase Server and Capella support.
[ ] Environment Variables: Use the existing template provided in the samples directory of this repository.
[ ] Deployment Configurations: Provide setup for Gitpod and GitHub Codespaces.
[ ] Testing: Add a simple test workflow to validate the connection to Couchbase.

Resources:
READ THIS FIRST - Instructions for getting started

Couchbase Kotlin SDK Documentation
Sample README for your project
Hacktoberfest Details

How To Get Started:
Follow the contribution instructions, and submit a pull request using the new starter kit pull request template with a link to your Kotlin starter kit.
Good luck on your adventure! ✨

Hi i want to contribute to this

Hi i want to contribute to this

Excellent, welcome @realjoni17! Let us know if you have any questions or need help along the way. Excited to see what you build!
cc: @ldoguin @shivaylamba


Flow does not detect type issues when using Object.defineProperty

I've been using Flow on an API library I'm building, I purely use ES5 techniques to create the OO model.
For e.g: To create a function that sits off the window object I use Object.defineProperty.
I noticed that Flow cannot detect Type issues when you do this:
I've written the same logic below using basic function definition and using Object.defineProperty. Flow should detect that "property length not found in Number" when calling both these functions.
/* @flow */

Object.defineProperty(window, 'testServerLoadA', {
    value: function(noOfHits) {
        var boostLoad = noOfHits.length * 10;  // Flow WONT detect this issue
        console.log(boostLoad);
    },
    writable: true,
    enumerable: true,
    configurable: true
});

window.testServerLoadA(3);

function testServerLoadB(noOfHits) {
    var boostLoad = noOfHits.length * 10; // Flow WILL detect this issue
    console.log(boostLoad);
}

testServerLoadB(3);

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

Looks like it fixed @goodmind
https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBjOA7AzgFzAA8wBeMAbwF8BudAeQCMArAUw3wDoATVqAS2ysACgCc4AB1aj8ATwAURADRgA5PlYEAytIBu0gDJwAhtwCCqlRVRhbYXcZgBXVgC4wUJ9g78c87HD0UAAS-Pi4AJSUNnaxDqJgjHBwBEamZGABQaHhnDCs2ADm+AAWyGAAjAAMNLbAwGAAYvBIAOr0AHIAKmC8Ghxgpfy4YMO4LjGxtlh4cPl5cIXySSn4adwRdLFUSpNgCKJhxoz57viiLruxBU4AttLHp4MXrFd2MwKFTqKPbs8TVE26CInA02j0hhM5nkAGYgahPN58L5sINNPgdKJ9KJ1gAhfyBEJhSLROLGBIrVJQjJZIm5fJFUrlaq1epNFpgVoASQMBl6rH6hCGIzGE1iM1wc1YCyWlLWUKBVHQYIxEJxUPxcKAA


adds entries from Ready Player One Tron Sneakers

Last seen in Ready Player One though originally from Tron and Sneakers. And Phineas and Ferb.

Please add yourself to the authors.txt file!


wizard: prepare a dev env to create the new wizard

Developing a new wizard will require many pull requests and some of them might even be worked on in parallel. To allow to develop all of that without interfering with the legacy wizard this commit introduces a new dev environment alongside an unleash flag when we later want to do the switch.
To use the new dev en use npm run stage-beta:experimental or npm run stage-beta:msw+experimental depending on what you want.
Then unleash flag to toggle in stage and prod is: image-builder.new-wizard.enabled
fixes: HMS-2859

/retest

@jrusz IQE tests are stuck, can we admin merge the PR?

/retest

@jrusz IQE tests are stuck, can we admin merge the PR?

It should be fixed now so unless it's urgent it should be green in < 1h and you can merge regularly :) ... They changed some navigation elements in consoledot again without telling anyone so we had to make some adjustments.


Can't use more than one New Relic account

I have applications the I'd like to add to Grafana dashboard in 2 different new relic accounts. They have different API keys.
The current plugin supports only one datasource that is tied to one account only.
I tried making a copy of the plugin and just renaming in order to create a second data source with a different API key but that didn't work. The list of applications showing up under the second app was from the first new relic account, not the second one.
Any comments on how can I get 2 accounts working?
I tried a prior version (commit a1dc1ab5e4f6cae2eb4264df52602695e3a4494f) which is just a datasource plugin (not an app plugin). I could add multiple data sources with different api keys. However, that version was missing some good features like there was no list of applications, for instance.
Are you looking to bring back this functionality? Or will you be okay with me opening a PR to bring this feature back?
Thank you.

I am no longer using NewRelic my self, so I do not currently have the time to add that functionality back in. Honestly, that was a use case I had not considered.
I do however review PR's and would be happy to merge this functionality in if you get it working.


Add cmorizer scripts for NOAA-ERSST.

Tasks

[x] Create an issue to discuss what you are going to do, if you haven't done so already (and add the link at the bottom)
[x] Give this pull request a descriptive title that can be used as a one line summary in a changelog
[x] Make sure your code is composed of functions of no more than 50 lines and uses meaningful names for variables
[x] Circle/CI tests pass. Status can be seen below your pull request. If the tests are failing, click the link to find out why.
[x] Preferably Codacy code quality checks pass, however a few remaining hard to solve Codacy issues are still acceptable. Status can be seen below your pull request. If there is an error, click the link to find out why. If you suspect Codacy may be wrong, please ask by commenting.
[x] Please use yamllint to check that your YAML files do not contain mistakes

New data reformatting script

[x] Test the CMORized data using recipes/example/recipe_check_obs.yml, to make sure the CMOR checks pass without errors
[x] Add the new dataset to the table in the documentation
[x] Tag @remi-kazeroni in this pull request, so that the new dataset can be added to the OBS data pool at DKRZ and synchronized with CEDA-Jasmin

Closes part of #1802

@axel-lauer, @jvegasbsc, this seems basically ready. Is that again a candidate to put on the waitlist until #1657?

Yes, I will do the conversion when it's merged

Yes, I will do the conversion when it's merged

But does prevent us from merging this now?

Yes, I will do the conversion when it's merged

But does prevent us from merging this now?

Hi, I was checking if ESMValTool had a cmoriser for ERSST and came across that PR. Any reason why this has not been merged?

Hi, I was checking if ESMValTool had a cmoriser for ERSST and came across that PR. Any reason why this has not been merged?

This cmorizer was originally used to format the dataset for an IPCC figure. As the IPCC diagnostic hasn't found yet the way into the public branch, this cmorizer was also not merged.
Now the cmorizer needs to be brought to the new cmorizer design in the tool, the rest should be straight forward.
@rbeucher As the original developer is not active anymore you are very welcome to use this as a base for a new attempt.
We should maybe close this PR and you could open a new one...

I have rebased and will work on this PR directly so that we keep the contribution from @bjoernbroetz

@flicj191 Can you have a look at this and see if it works for you?

@flicj191 Can you have a look at this and see if it works for you?
I'm running into some issues, looks like a few more things to update, I'll come back to it

Thanks @LisaBock , yes I would have used another version of the Core as I only had ESMValTool in develop mode.
I've installed ESMValCore/main into my environment and tested and it still worked for me.
Though I was getting that error previously when trying to use the data so edited the formatter to clean up the time coord. I'll try to replicate again and look into it.

Hi @LisaBock this was still working for me, were the files downloaded recently? They are files per month, maybe that was different previously as the download instructions in the branch previously didn't match what could be downloaded now.

Hi @flicj191 unfortunately it's still not working for me For the ERSSTv5 dataset. I used the downloader of this PR as it is now.
The dtype of the time coordinate varies in the downloaded files between int64 and float64. That's why it could not be concatinated. You could maybe fix them all in _fix_time_coordinate to float64.

Thanks @LisaBock! interesting though the files I downloaded I seemed to just see float64 and then the _unit.date2num would return int64 for me so maybe its the cftime package version. I have cast to float64 as suggested, hopefully that works, still working for me. Cheers!

Hello @ESMValGroup/technical-lead-development-team, let me know if there's anything I need to fix for this PR. Cheers!


soc/intel_adsp: cavs-link.ld: add *(.trace_ctx) sections

The CAVS linker scripts in Zephyr and "plain" (XTOS) SOF seem to share
the same origin and they mostly duplicate each other so they naturally
keep diverging from each other. This commit catches up with the SOF
commit ef43899c580f (thesofproject/sof/pull/2996) which added
the *(.trace_ctx) sections as part of the runtime log filtering
implementation (feature request thesofproject/sof/issues/2172)
Paraphrasing that commit, the goal is to create at link time a C-like
array by regrouping scattered trace contexts so they can be iterated on.
Together with some other changes in SOF, these 3 additional lines are
enough change in Zephyr to fix SOF bug thesofproject/sof/issues/5032
Signed-off-by: Marc Herbert<EMAIL_ADDRESS>

Obviously the write permissions won't change in our setup, but the cacheability will. Stuff in .rodata will be faster in general.

Doh, I remembered these particular structs are mutable. They're just log levels so they don't really need strong coherency but mutable data in .rodata just feels too repulsive, doesn't it? Also, some users may not appreciate log levels being updated after some random delay.

like STRUCT_SECTION_ITERABLE which give you this behavior more or less for free. It's used everywhere. Might be worth up-leveling the SOF layers to use it at some point.

Maybe not yet because they require the use of a STRUCT_SECTION_FOREACH iterator which would require a new and relatively big #ifdef ZEPHYR.
So I would like not to change this PR.
However I'm keeping it as a draft for now because your 700 times bigger #41098 seems close to the finish line and it's obviously more convenient to rebase 3 lines on top of 2000 than the other way around.

Oh sorry, I just assumed they were actual RO data. :)  No, if they're mutable they don't belong there.  And if they're mutable at all, you need to be REALLY careful about coherence, so it's probably best to leave them in .data
But if you do want a r/w cached segment, it exists (but is non-standard) it's called ".cached" in the linker script.

But if you do want a r/w cached segment, it exists (but is non-standard) it's called ".cached" in the linker script.

Thanks, maybe later after some micro benchmarking, for now I would really like thesofproject/sof/issues/5032 to get fixed.

The CAVS linker scripts in Zephyr and "plain" (XTOS) SOF seem to share the same origin and they mostly duplicate each other so they naturally keep diverging from each other. This commit catches up with the SOF commit ef43899c580f

I forgot about the linker script duplication inside Zephyr. @andyross removed a lot of it recently but there is still fair bit so here is: #41282, sorry about the additional PR and review.


feat(changelog): attempt to fix semantic-release

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Managed Reconciler: ExternalObservation type should pass more information to the Create method

What problem are you facing?

When using the managed reconciler pattern, it's sometimes the case that more than just one operation need to be performed in the external resource's Create method, conditioned on the observation made. For instance, an external resource might need 3 distinct sub-components to be provisioned in order, for it to be created. Currently, every time the Create method is called, it uses the CR's fields, to determine what operation it should run, which is not efficient and essentially the observation is done again in the Create method (it is triggered merely based on the fact that ExternalObservation{ ResourceExists: false }).
How could Crossplane help solve your problem?

More information (for instance a map of key-values) could be passed to Create method, making it more context-aware.

I'm not sure if I understood you correctly but in API patterns one-pager, we recommend that the one managed resource should result in one external resource, if that's what you mean. This is because ExternalClient interface is supposed to be the lowest level of operations in Crossplane, i.e. talk to the cloud provider. You can see the details here.
About carrying a map; the assumption of ExternalClient is that all information necessary to provision the resource exists on the spec. Hence, in most of the cases Create just converts spec.forProvider into the cloud provider struct and call the provider's API.
Could you elaborate on what you're trying to achieve?

@muvaf thanks for the comment!
My use case is that in order to provision an external resource A, the creation goes through a series of steps in order. Trying to keep it simple, let's say A roughly represents a KubernetesApplication on a KubernetesCluster. The Create method for A then will look like following:

It needs an instance of KubernetesCluster claim, and should wait for it to be satisfied (either finding an existing cluster, or provisioning a new one).
It then creates the specified KubernetesApplication on the created cluster.

As you can see, the Observe method has more than just a boolean ExternalResource information, and every time that the Create is called, it needs to know what step it had has to start from as there are more than one just a single action depending on what the observation was. One solution is to 'observe' the current step, but that goes against the purpose of Observe method. The other option would be to add some metadata to the status, or spec of the CR, but this metadata is only used for internal wiring and I don't see why it should be exposed.
If the assumption is that managed resource always results in creating only one external resource in a single phase, then the managed reconciler doesn't seem to be appropriate choice for my scenario, which is fine. However I won't be able to use the goodies that are provided by the crossplane-runtime library.

The closest case that I can see in Crossplane is the following:
func (e *external) Create(ctx context.Context, mgd resource.Managed) (managed.ExternalCreation, error) { // nolint:gocyclo
	cr, ok := mgd.(*v1alpha3.RouteTable)
	if !ok {
		return managed.ExternalCreation{}, errors.New(errUnexpectedObject)
	}

	cr.Status.SetConditions(runtimev1alpha1.Creating())

	req := e.client.CreateRouteTableRequest(&awsec2.CreateRouteTableInput{
		VpcId: aws.String(cr.Spec.VPCID),
	})
	result, err := req.Send(ctx)
	if err != nil {
		return managed.ExternalCreation{}, errors.Wrap(err, errCreate)
	}

	cr.UpdateExternalStatus(*result.RouteTable)

	// Create Routes
	if err := e.createRoutes(ctx, cr.Status.RouteTableID, cr.Spec.Routes, cr.Status.Routes); err != nil {
		return managed.ExternalCreation{}, err
	}

	// Create Associations
	if err := e.createAssociations(ctx, cr.Status.RouteTableID, cr.Spec.Associations, cr.Status.Associations); err != nil {
		return managed.ExternalCreation{}, err
	}

	return managed.ExternalCreation{}, nil
}

A RouteTable resource needs multiple resources and even though even though all of them are created regardless in this case, but it could be the case that the creation steps are not idempotent  and only should run conditionally .

If the assumption is that managed resource always results in creating only one external resource in a single phase, then the managed reconciler doesn't seem to be appropriate choice for my scenario, which is fine.

This is the case ideally but I can imagine we do make some compromises as APIs are scattered with more than resources which need a Create call for each. So, it's not the ideal case we're aiming for but a case we have to consider.
That said, ExternalClient functions should be idempotent. What I'd do is to check everything you'd like to be created in Observe and if any one of them doesn't exist, return ResourceExists: false. Then make Create that has multiple calls idempotent; either skip the err if it's AlreadyExists or make GET request for each of them before creation. In case create call results in an identifier each time, you'd have to save that in spec (or in annotations as external name if it's the main resource's identifier), then check its existence.

The closest case that I can see in Crossplane is route table resource on aws provider.
A RouteTable resource needs multiple resources and even though all of them are created regardless of their existence in this case, it could be the case that the creation steps are not idempotent and only should run conditionally.

Sadly, this seems like a buggy implementation where Create never called again even though createRoutes or createAssociations fail because in Observe, only existence of RouteTable is checked.


returned "Insufficiet Permission"

Expected Behavior
Install the google calendar api
Actual Behavior
receive the error "insufficient permission"
Steps to Reproduce the Problem
1.Move the downloaded file to your working directory and ensure it is named credentials.json
2.run pip install --upgrade google-api-python-client oauth2client (successful)
3.run python quickstart.py
4. received the error:
googleapiclient.errors.HttpError: <HttpError 403 when requesting https://www.googleapis.com/calendar/v3/calendars/primary/events?timeMin=2018-08-16T09%3A19%3A45.264518Z&maxResults=10&singleEvents=true&orderBy=startTime&alt=json returned "Insufficient Permission">
Specifications

Python version (python --version) - 3.6
OS (Mac/Linux/Windows) -Windows

Hi @OliveCookie,
I was just able to verify that the quickstart should be working. Maybe you disallowed credentials at some point.

Can you delete credentials.json?
Can you go here and revoke credentials to this Calendar quickstart?

Maybe your domain admin blocked this API? Can you try on a personal Google account?

Any updates on this issue @OliveCookie?

Closing due to inactivity.


Error when loading markets on Rinkeby

Initial markets list load, market is undefined so getting property description  doesn't exist.

error is in add-transactions.js probably this loads before marker information is available and user is auto logged in.
message: "Complete Sets Sold",
          description: `${transaction.meta.amount} sold on Market: ${
            market.description
          }`,
          transactions: [transaction]

could just change market.description to (market || {}).description to handle the race condition.

fixed by https://github.com/AugurProject/augur-ui/pull/2519


Finalize reserved prefix for annotation keys to store notary specific metadata in signature envelope.

We need to finalize notary v2 reserved prefix for annotation keys to store notary specific metadata.
See https://github.com/notaryproject/notaryproject/pull/102
once we have finalized the prefix please update signature-specification.md > Payload section with following blurb.
The prefix `TBD` is reserved for use in Notary v2 and MUST NOT be used outside this specification. In Notary v2 annotations are being used to store signed attributes

Resources: https://github.com/opencontainers/image-spec/blob/main/annotations.md#rules

how about: org.cncf.notary.*
This follows the annotations convention under the image manifest
It's also aligned with org.cncf.oras.artifact.* from PR #82

org.cncf.notary.* sounds good to me. Is there any legal or any other formality required?

how about: org.cncf.notary.*

What relationship does notary have to the Christina Noble Children's Foundation?

I think what @jonjohnsonjr is trying to say is:

The image spec calls out a suggestion to use internet reverse namespaces for annotations. You might want to consider this reverse domain pattern...

Keys MUST be unique within this map, and best practice is to namespace the keys.
Keys SHOULD be named using a reverse domain notation - e.g. com.example.myKey.

We've been using the logical namespace of org.cncf.notary.* but this is confusing when you consider reverse domains, vs. logical ownership. While I don't think we'll have a true collision with the Christina Noble Chidren's Foundation, it's a good thing to cleanup.
How about io.cncf.notary


Clarify/better document (a)synchronicity requirement differences between persist trait and event handling

Tracked at https://github.com/lightningdevkit/rust-lightning/issues/1194


New cluster settings for extra safeguards against clock offset jumps

Setting for HLC to be monotonic across restarts
Setting for HLC to panic on clock jumps

This applies to both 2.0 and 2.1 docs.

The 2.0 branch docs should note that the settings are new in 2.0.1.


Animation dots don't follow Comfy Link option (eg straight)

See https://github.com/chrisgoringe/cg-use-everywhere/issues/135#issuecomment-1994814685

This is a bug in the underlying litegraph code - I've raised an issue (https://github.com/jagenjo/litegraph.js/issues/457) but doubt it'll get fixed.
Maybe I'll work a patch out...

In the latest push there are two modes of animation - dots, pulse (or both or off).
Pulse should work with all link styles, dots only with spline.
So now there is an animation option for everyone :)


test speed

hello ! i want to know how you  test speed on your model (45fps on got10k)?   can u provide the "profile_model.py" like other tracker? thank u!

Hi, I directly took the average FPS printed during inferecne as the running speed of my tracker. The profile_model.py needs the inputs manually prepared, and I do have a messy version. It would be an useful reference, whereas normally it will provide a faster result. Besides, the amount of FLOPs also cannot truly reveal the computation latency of algorithms. The most simple and straightforward way is still running them on the same machine. If you want it, I think you can smoothly modify a script from STARK or OSTrack.


[BUG] Tips and Tricks -> install-profile

Hi,
I have been using an old script of the install-profile, however once I updated the install-profile script, the script broke.

The script initially assumes that the dotbot is located in the meta folder. But once we progress in the script, the git submodule commands are looking for the dotbot dir to update.

As shown in the image, first we cd to the base directory, and the git submodule is looking for the dotbot directory in the base directory which does not exist because it is in the meta/dotbot dir.
fix
1
git -C "${BASE_DIR}" submodule sync --quiet --recursive
git submodule update --init --recursive "${BASE_DIR}"

2
git -C "${META_DIR}/${DOTBOT_DIR}" submodule sync --quiet --recursive
git submodule update --init --recursive "${META_DIR}/${DOTBOT_DIR}"

Nice catch! Anyone is allowed to modify the wiki. Would you like to make your corrections directly?

Thank, I just edited the wiki with fix 1.


Refactor worldview to support scene modules

Switch App to a functional component
Adding timelineSlice for keyframes
Adding hooks for kepler and stage features
kepler util for loading map json
Move new york sample to scene folder

refactor into scene module example

Refactor components to use timelineSlice, support a timeFilter, and use new scene code

pull scene definitions out into their own module
support timeFilter in kepler animation

Pull Request Test Coverage Report for Build 750173847

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 747298568:
0.0%

Covered Lines:
2

Relevant Lines:
2

💛  - Coveralls


feat(storage): improve current process detection for MDBX HSR

We need to try both parent PID and the current PID when checking the process ID that MDBX reported as having a long-running transaction.

do we have instances in our codebase when parent pid != current pid? or is it for external use?

do we have instances in our codebase when parent pid != current pid? or is it for external use?

From man sudo:

When sudo runs a command, it calls fork(2), sets up the execution environment as described above, and calls the execve system call in the child process.

so technically when doing sudo reth node, Reth (child) won't have the same pid as sudo (parent)


review the usage of osmoutils.ApplyFuncIfNoError

review wether osmoutils.ApplyFuncIfNoError(ctx, func(ctx sdk.Context) error needs to be used for taker_fee

this function used to run hooks that in case of failure, won't panic the system and won't be commited


Adding calculator for octagonal pyramid

Under calculator of 3D shapes in the pyramid section I would like to add the calculator for octagonal pyramid. Please assign this to me under GSSoC.

Go ahead
Deadline: 20 April 2021


Handle GCS object not found error

This makes the Google Cloud Storage store consistent with the S3 store implementation by making GetInfo return ErrNotFound when the object is not found.
This also makes the HEAD request return 404 instead of 500 when attempting to get the upload status of a non-existent file stored on GCS.

That's looking good, thank you very much for this contribution :+1:


OData: add a service+metadata document at the project root

right now we have one OData service per form.
so for instance: /v1/projects/1/forms/birds.svc is a service document (a directory, essentially) which offers up at least a Submissions table (/v1/projects/1/forms/birds.svc/Submissions), plus whatever repeats are a part of the birds form.
it would be nice if we could allow /v1/projects/1.svc which lists all the tables for all the forms in the project as a whole. it would use the just form document urls (eg the /Submissions url listed above) for all the actual data.
xmlformid is guaranteed unique, so this should be safe to do.
the main headache is merging together all the entities for all the forms for the CSDL. it's not difficult, but it may be tedious.

Closing after discussion with @lognaturel. We can always reopen if we end up wanting to prioritize this change.


add japanese *.po

These translation files are based on https://github.com/nemtech/nem2-docs/commit/bc586d0d699ceff19b311c0b2f3ede2b5fc91d0d

Thanks! We'll review it and merge the PR.


[NO QA] Add Daily label to issue templates

Details

Discussion: https://expensify.slack.com/archives/C01GTK53T8Q/p1627294620259300
Fixed Issues

N/A
Tests
N/A
QA Steps
N/A
Tested On
N/A (Tested on personal repo)

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging         in version: 1.0.80-3🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production         in version: 1.0.81-4🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Simplify SQL Builder and Model types generation

Is your feature request related to a problem? Please describe.
According to the document, the generator need a running database to generate the model& builder, in the daily development it’s not so convenient.
**Describe the solution you'd
May be we can use a sql parser to parse the schema file to generate the stuffs, for those stuffs the schema sql file is enough.

This has been proposed before. Check this discussions: https://github.com/go-jet/jet/discussions/150, https://github.com/go-jet/jet/discussions/165


Hidden layer dimension

Can it be understood that the hidden layer dimension is increased? If other spatio-temporal diagram methods are used and the dimension is set to 4D, the effect may be significantly improved.

Simply increasing the hidden dimension may not significantly improve the performance, and even may decrease it in some datasets due to overfitting.


Update to use newest lookout-statsd gem

lookout-statsd had a major API bump so need to manually adapt to it.

LGTM


Add Discogs Sensor platform

Description:
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#4138
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: discogs
     token: xxxxxxxx

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Not really sure how to fix linting errors from Travis:
./homeassistant/components/sensor/discogs.py:1:1: D100 Missing docstring in public module

C:  1, 0: Missing module docstring (missing-docstring)
ERROR: InvocationError: '/home/travis/build/home-assistant/home-assistant/.tox/pylint/bin/pylint homeassistant'

Other components didn't seem to do this when importing asyncio

Not really sure how to fix linting errors from Travis:

The file header is missing which is a docstring. E.g.,
"""
Support for stored record collections on Discogs.

For more details about this platform, please refer to the documentation at
https://home-assistant.io/components/sensor.discogs/
"""


Layout issue on iPhone 5 SE

Cannot access the 3 or the backspace....

https://github.com/automat-berlin/afone#known-issues

@shadzik oh I see but because this is on the app store, I thought that maybe that could be useful to know... I'll see if I find time to do a PR.

@lolobosse a PR will be very welcome and appreciated :)


Set CMake's CMP0079 policy

Explicitly opt into the new behavior.
For more information on the CMP0079 policy, see CMake's documentation
Fixes #88
Signed-off-by: Michael Carroll<EMAIL_ADDRESS>

CC: @traversaro.  Do you see any reason not to opt into the new behavior?  I have been using it for a bit with no issue on my focal machine.

CC: @traversaro.  Do you see any reason not to opt into the new behavior?  I have been using it for a bit with no issue on my focal machine.

CC: @traversaro. Do you see any reason not to opt into the new behavior? I have been using it for a bit with no issue on my focal machine.

Yes, I think it should be harmless to switch to the new policy settings.

CC: @traversaro. Do you see any reason not to opt into the new behavior? I have been using it for a bit with no issue on my focal machine.

Yes, I think it should be harmless to switch to the new policy settings.


ISV sale options

I've added options to complete a sale if you have an ISV account.
They work ok as is, but since we just started integration I will continue to monitor and update if any bugs are found.
More info about the ISV partner program:
https://developer.vivawallet.com/isv-partner-program/
ISV options are shown here:
https://developer.vivawallet.com/apis-for-point-of-sale/card-terminal-apps/android-app/sale/

Hi, thanks. I'm not an ISV partner, so I didn't implemented it. This should be fine. Please test it and let me know if it works ok, so I can merge it to main.
Anyway, this plugin works for a 1,5 years on a various phones without a problem so far. Probably should remove the beta and update descriptions.
Thanks again.
Denis

I had 0 issues with the plugin, it worked out of the box with our account.
Excellent work 👍
We'll do much more tests with the ISV options next month and report back for the merge.

Hi, did you have a chance to test the plugin with ISV options added so I can merge your changes?

Cool, great. Let me know then...
btw where was the expo?

https://horecaexpo.gr/en/

Superb
And what is your product? I'm just curious ;) You can mail me to<EMAIL_ADDRESS>

I made a new function, isvSale, separate from the old sale to avoid any confusion with the nullable fields.
Tested them both with my ISV connected merchant account and both work ok, isvSale passes the isvAmount to the ISV account.
All ISV charges were processed correctly during my tests, so I think we are safe to merge.
For anyone interested, the following example request:
      final response = await pos.isvSale(
        amount: 10.0,
        tipAmount: 0.2,
        isvAmount: 0.1,
        clientTransactionId: 'CLIENT_TRANS_ID',
        isvClientId: 'ISV_CLIENT_ID',
        isvClientSecret: 'ISV_CLIENT_SECRET',
        isvMerchantId: 'ISV_MERCHANT_ID',
        isvClientTransactionId: 'ISV_CLIENT_TRANS_ID',
      );

will show up in your ISV account logs with 0.10euro on the ISV Amount column.
Notes

isvCurrencyCode is required and set by default as Euro (978), please change it if you are not using Euro (See ISO4217).
isvSourceCode is set as 'Default', if needed, please setup yours from: Sales > Physical Payments > Stores

New isvSale function required an ISV multi merchant account, you cannot use it with your normal account.

Great job. New function is really better option because of all that ISV params.
I will make a quick check / test later and then merge your changes.

Everything is good, merged your changes. Also, the new package is published on pub.dev
Thanks for your contribution.
Cheers.

Thank you for providing this excellent package. However, could you please confirm if iOS support will be available in future updates?

Hi, unfortunately there is no planned support for IOS. I'am Android developer only. If anyone is willing to contribute to IOS support, I will be happy to publish the updated package.


expose KHR_debug functionality on native

Required for https://github.com/gfx-rs/gfx/issues/2826
I've already written a native implementation for the KHR_debug functions, but it adding it to Context would break the unified API since WebGL has no equivalent (I see WEBGL_debug has been proposed but not implemented). How should we expose these functions?
For reference, the functions in question:
glDebugMessageControl
glDebugMessageInsert
glDebugMessageCallback
glGetDebugMessageLog
glGetPointerv
glPushDebugGroup
glPopDebugGroup
glObjectLabel
glGetObjectLabel
glObjectPtrLabel
glGetObjectPtrLabel

We can expose them here and just panic for WebGL. In gfx we can use cfg to avoid calling these for WebGL or add a supports_debug_features feature flag like we already do for some other features.
In the future, in glow we can probably expose some functions for feature checking (to move this functionality over from gfx). So then we could expose something like supports_debug_features() which would be hardcoded to false for WebGL.


Google assistant problem

I can correctly play specific songs with Google assistant, but onSetMediaItems will also be called even if the search result is empty (but Google assistant still shows title which is not in my app, it's weird), I have to return MediaItemsWithStartPosition.

What should I do if there is no search result?
Google assistant doesn't call onSearch and onGetSearchResult, is this normal? It directly calls onSetMediaItems. Can you make MediaItemsWithStartPosition in onSetMediaItems nullable to interrupt the action (including the following play action)?

I shouldn't determine the number of songs in Player's handleSetMediaItems, because it should also happens when playlist gets cleared. Even if I early return in this case, then the play will be called, and I can't control it.

Google assistant doesn't call onSearch and onGetSearchResult, is this normal? It directly calls onSetMediaItems.

This part is working as expected. onSearch/onGetSearchResult is used to browse the library without any immediate intent to play. Note that these values also don't have a MediaItem input. onAddMediaItems is used for new items that are supposed to be added to the player and it has a MediaItem input to define what should be played. (MediaItem.RequestMetadata.searchQuery will contain the search query, and you can also identify which controller is requesting this playback by checking the ControllerInfo parameter).

Can you make MediaItemsWithStartPosition in onSetMediaItems nullable to interrupt the action (including the following play action)?

You can return a failed or cancelled future from onSetMediaItems to interrupt the action and prevent any further call to play. This should also address your follow-up comment. If you do everything inline, you could for example use Futures.immediateFailedFuture(new UnsupportedOperationException()), which is also what happens by default if you don't even implement the method.

the search result is empty

This may be clear to you already, but Assistant will send an empty search request if the user says something like "Hey Google, play music in [your app]". The expectation is that you either play something suitable (for example, resume previous playback) or let the request fail.

Ok, thanks for the detailed explanation.


Auto cleanup as data grows

Is there an easy way to clean up docs in a groundDB to prevent it from growing? Should we simply listen to  isDone event and call keep when it goes over a number of docs?

you could in theory add some offline metrics like size/usage/age of documents, then have a sort function using weights to figure out the most important documents. I've done this in the past and wanted this in gdb at some point. It's something to think about.

Thanks @raix
I had an idea to make sure we keep data well sync-ed with server. The problem is with all the subscriptions, it's tricky running a keep as you may lose data that is still on server but not subscribed to yet.
We could have a server method to which we send all the ids currently in dgb and it returns the ones that are valid, and we would run on the client a keepIds (a clone of keep but that depends on id's).
Thoughts? If you are willing to accept a PR, I could work on it. It's a simple client server called sync() that would take care of everything.


add lint/accessibleEmoji for jsx-a11y

This pr adds a rule that checks an emoji is wrapped in a span that has role='img' and an aria-label/aria-labelledby
Rule
from the list in here
I could use some advice on what I should implement for the emoji check. There's different online articles about the check itself, but if you have suggestions, happy to update.

I've been keeping an eye on this repo, as I'm keen to give it a go.
I reported this rule last year as leading to a degraded experience for users of accessibility technologies in the eslint-plugin-jsx-a11y.  The issues still open there https://github.com/evcohen/eslint-plugin-jsx-a11y/issues/627, not sure how active that project is.
The report is as follows.
From some testing on os x with VoiceOver, it seems the guidance provided in the rule jsx-a11y/accessible-emoji leads to a degraded experience with the label not being announced to the user.
I've setup a codepen which demonstrates this at the following link.
https://codepen.io/anon/pen/ymoBgb
The rule seemed to have been created based on the advice in a blog post from a few years ago now, so perhaps browsers and assistive technologies have advanced since then.
#136

I'm not an expert in a11y by any means so I'm hesitant to add in rules that are potentially sketchy. If the advice is indeed outdated then I don't think we'd want this rule.

@VictorHom What do you think?

Good spot @luke-john !
I gave a spin to the codepen using the VoiceOver of the macOS and you're right!
The labels that wrap the emoji with a span creates an awful user experience: the phrase is cut off and you need to you use keyboard to navigate the whole paragraph.
On the other hand, the ones without the accessibility suggestion are spot one and the VoiceOver is able to pronounce the label associated to the emoji together with the phrase.
I'd say we can remove the suggestion.
P.S.: as far as I know, accessibility tests are done most of the time using the VoiceOver of macOS.

Sweet! I'm going to close this out then.


Bump

@mhearne-usgs thanks to https://github.com/conda-forge/obspy-feedstock/pull/38 we can update this feedstock.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.


Make cost analysis better to use

Sources:

https://github.com/pa-bru/graphql-cost-analysis

@michaelstaib Closing, since from what I can tell this has been implemented and documented in v12.


Use non-reentrant lock for GrowableArrayBlockingQueue tail

Motivation
GrowableArrayBlockingQueue, as the name implies, doesn't have a max size and will never block the producers. Because of that, we don't need a reentrant lock to guard the tail of the queue, instead, we can use the lighter-weight StampedLock.

[x] doc-not-needed

The data array is mutated and read under 2 separate locks which doesn't seem right.

@lhotari All guarded by headLock?

The data array is mutated and read under 2 separate locks which doesn't seem right.

@lhotari All guarded by headLock?

@Jason918 it's not all guarded by headLock. for example, put isn't guarded by headLock.
There's a thread safety issue in GrowableArrayBlockingQueue since the data array is mutated and read under 2 separate locks.

The data array is mutated and read under 2 separate locks which doesn't seem right.

@lhotari All guarded by headLock?

@Jason918 it's not the case that everything is guarded by headLock. for example, put isn't guarded by headLock. There's a thread safety issue in GrowableArrayBlockingQueue since the data array is mutated and read under 2 separate locks.

@lhotari The data array object mutation only happens in the code path put() -> expandArray(), where put() acquires the tailLock, and expandArray() acquires the headLock. So the data array object should be thread safe if you acquires either tailLock or headLock before accessing the object.

@lhotari The data array object mutation only happens in the code path put() -> expandArray(), where put() acquires the tailLock, and expandArray() acquires the headLock. So the data array object should be thread safe if you acquires either tailLock or headLock before accessing the object.

data array elements are mutated and read with 2 different locks in the methods in GrowableArrayBlockingQueue class. For example, poll is guarded by headLock and put is guarded by tailLock. I don't see how that could be thread safe. There's a related StackOverflow answer https://stackoverflow.com/a/8978397 which states:

"This means that you can safely write to two different indexes concurrently. However you need to synchronize a write/read to the same index if you want to make sure the consumer thread sees the last value written by the producer thread."

@lhotari OK, I get your point. It's the array elements, not the array object. This concern make sense. My guess on this is that there are happen-before relations here:

put() ->
data[tailIndex.value] = e;//write Ops -->
SIZE_UPDATER.getAndIncrement(...) in put() -->
SIZE_UPDATER.get() in poll() -->
T item = data[headIndex.value];//Read Ops.
So the array element written by put is always visible to poll.

Not 100% sure about this :)

@lhotari OK, I get your point. It's the array elements, not the array object. This concern make sense. My guess on this is that there are happen-before relations here:

put() ->
data[tailIndex.value] = e;//write Ops -->
SIZE_UPDATER.getAndIncrement(...) in put() -->
SIZE_UPDATER.get() in poll() -->
T item = data[headIndex.value];//Read Ops.
So the array element written by put is always visible to poll.

Not 100% sure about this :)

Good analysis @Jason918 . Yes it seems that SIZE_UPDATER saves it in this case, however there's a risk that there's an access path which isn't thread safe.

@lhotari OK, I get your point. It's the array elements, not the array object. This concern make sense. My guess on this is that there are happen-before relations here:

put() ->
data[tailIndex.value] = e;//write Ops -->
SIZE_UPDATER.getAndIncrement(...) in put() -->
SIZE_UPDATER.get() in poll() -->
T item = data[headIndex.value];//Read Ops.
So the array element written by put is always visible to poll.

Not 100% sure about this :)

Good analysis @Jason918 . Yes it seems that SIZE_UPDATER saves it in this case, however there's a risk that there's an access path which isn't thread safe.

Sure, we can try to construct a unit test to verify this issue.


Moddable SDK moving to ESP-IDF v5.1.2

The January release of the Moddable SDK will recommend ESP-IDF v5.1.2 (previously v5.1.1). That will require updating these two lines in xs-dev:
https://github.com/HipsterBrown/xs-dev/blob/8a07669682572b32a7ac09605ac8fb55e0947449/src/toolbox/setup/esp32.ts#L17
https://github.com/HipsterBrown/xs-dev/blob/8a07669682572b32a7ac09605ac8fb55e0947449/src/toolbox/update/esp32.ts#L13
cc @mkellner

Thanks, @HipsterBrown!

Thanks for reporting the update! We'll make this more seamless in the future 😄


Custom shader breaks Modulate

Godot version
v4.0.alpha6.official [e4f0fc50f], v4.0.alpha.custom_build [b0ba818b8]
System information
Windows 11, Geforce 1660, Vulkan Clustered
Issue description
Custom shader breaks Modulate
https://user-images.githubusercontent.com/10379487/162537075-b0d8a7d3-d602-443e-a0e5-7193a666467c.mp4
v3.5.beta.custom_build [fa9b77dbe] doesn't have this problem.
Steps to reproduce
Change the modulation of the sprites and compare their behavior.
Minimal reproduction project
CustomShaderModulateTest_G4.zip

Duplicate of https://github.com/godotengine/godot/issues/49781.


[RFE] Project Categories

Request for Enhancement
What
Categories of projects so that the dashboard can show just the projects of the desired context.
Details
I'm looking for a way to add my own Projects tabs to the dashboard so I can organize the projects by context/category (work, home, garden, etc).
What might be the best way to go about this?  I was thinking of making copies of $:/plugins/nico/projectify/ui/dashboard/Projects that with category names and figuring out how to edit $:/plugins/nico/projectify/ui/dashboard/InboxAndProjects to have a drop-down selector for the projects group I want to work on. As I dig in I want to be sure this is the best way and you don't already have something else on the roadmap that I missed.
I'm happy to hack at this a bit to create a pr but would be interested in any pointers or direction you might be able to offer.
Thank you :)

I was also creating individual inboxes for different projects, but it's not right to add new files to the plugin for each new Project category or inbox for said cat.  Think it would be better to break project/inbox sets out to be generated much like individual projects are tiddlers. Not sure if that makes sense

Optimally I would be able to drag and drop from the inbox into projects and then later in each project, drag and drop from their own inbox into their sub projects

@Tejeev thanks for the post. Would you mind moving your ideas to the discussion tabs here https://github.com/NicolasPetton/Projectify/discussions? I think it would help keep the GitHub project clean.

I've not the permissions to do so but I'm happy to move this there.  Sorry I'm used to using issues for RFEs even when they are larger and might need some back and forth.

I think this could be better thought out in the request anyway, so I'll close the issue and manually rewrite something for discussion.  Thank you @NicolasPetton


⚠ Bofrost.fr has degraded performance

In d0093db, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1751 ms

Resolved: Bofrost.fr performance has improved in 6486f77 after 25 minutes.