Products Pricing List / Pricing Markup not being replicated to Translated Products Pages

Can you reproduce this issue on default Wordpress theme (eg Storefront)?
No
Can you reproduce this issue when all other plugins are disabled except WooCommerce, Polylang and Hyyan WooCommerce Polylang Integration?
Yes, done.
What product versions and settings are you using when this issue occurs?

PHP: : 7.2.20
WordPress: 5.2.2
WooCommerce: Version 3.6.5
Polylang: Version 2.6.3
Hyyan WooCommerce Polylang Integration: 1.4.3
Browser: Chrome  76.0.3809.100

Steps to Reproduce

Updating products price list.
Example: Default Language: English
More languages: Spanish and Portuguese

What I Expected
Update the Products Price List / Pricing Markup in multilingual, replicates the new pricing to all products in all laguages.
What Happened Instead
It updates only the default language in English(parent). Others products pages in Spanish and Portuguese(child) linked to the same product are not updated automatically.
Workaround:
Once I go to the default(parent) product page, click only on "Update" button, the new price is replicated to others product pages in Spanish and Portuguese. Its a tough work to update it manually one by one.
WordPress Environment

```
`
### WordPress Environment ###
WordPress address (URL): https://shopflixstore.com
Site address (URL): https://shopflixstore.com
WC Version: 3.6.5
Log Directory Writable: ✔
WP Version: 5.2.2
WP Multisite: –
WP Memory Limit: 512 MB
WP Debug Mode: –
WP Cron: ✔
Language: en_US
External object cache: –
Server Environment
Server Info: LiteSpeed
PHP Version: 7.2.20
PHP Post Max Size: 256 MB
PHP Time Limit: 240
PHP Max Input Vars: 5000
cURL Version: 7.62.0
OpenSSL/1.0.2k
SUHOSIN Installed: –
MySQL Version: 5.5.5-10.2.25-MariaDB
Max Upload Size: 256 MB
Default Timezone is UTC: ✔
fsockopen/cURL: ✔
SoapClient: ✔
DOMDocument: ✔
GZip: ✔
Multibyte String: ✔
Remote Post: ✔
Remote Get: ✔
Database
WC Database Version: 3.6.5
WC Database Prefix: wp_
MaxMind GeoIP Database: ✔
Total Database Size: 34.07MB
Database Data Size: 22.32MB
Database Index Size: 11.75MB
wp_woocommerce_sessions: Data: 0.06MB + Index: 0.02MB
wp_woocommerce_api_keys: Data: 0.02MB + Index: 0.03MB
wp_woocommerce_attribute_taxonomies: Data: 0.02MB + Index: 0.02MB
wp_woocommerce_downloadable_product_permissions: Data: 0.02MB + Index: 0.06MB
wp_woocommerce_order_items: Data: 0.02MB + Index: 0.02MB
wp_woocommerce_order_itemmeta: Data: 0.02MB + Index: 0.03MB
wp_woocommerce_tax_rates: Data: 0.02MB + Index: 0.06MB
wp_woocommerce_tax_rate_locations: Data: 0.02MB + Index: 0.03MB
wp_woocommerce_shipping_zones: Data: 0.02MB + Index: 0.00MB
wp_woocommerce_shipping_zone_locations: Data: 0.02MB + Index: 0.03MB
wp_woocommerce_shipping_zone_methods: Data: 0.02MB + Index: 0.00MB
wp_woocommerce_payment_tokens: Data: 0.02MB + Index: 0.02MB
wp_woocommerce_payment_tokenmeta: Data: 0.02MB + Index: 0.03MB
wp_woocommerce_log: Data: 0.02MB + Index: 0.02MB
wp_adsw_activities: Data: 0.27MB + Index: 0.00MB
wp_adsw_ali_meta: Data: 0.31MB + Index: 0.03MB
wp_adsw_search_analytics: Data: 0.02MB + Index: 0.02MB
wp_ads_activities: Data: 0.02MB + Index: 0.00MB
wp_ads_ali_meta: Data: 0.02MB + Index: 0.03MB
wp_ads_attributes: Data: 0.02MB + Index: 0.03MB
wp_ads_orders_item: Data: 0.02MB + Index: 0.03MB
wp_ads_products: Data: 0.02MB + Index: 0.03MB
wp_ads_products_meta: Data: 0.02MB + Index: 0.02MB
wp_ads_search_analytics: Data: 0.02MB + Index: 0.02MB
wp_commentmeta: Data: 0.02MB + Index: 0.03MB
wp_comments: Data: 0.30MB + Index: 0.38MB
wp_duplicator_packages: Data: 0.02MB + Index: 0.02MB
wp_links: Data: 0.02MB + Index: 0.02MB
wp_options: Data: 2.16MB + Index: 0.14MB
wp_payment_discount: Data: 0.02MB + Index: 0.00MB
wp_payment_transaction: Data: 0.02MB + Index: 0.06MB
wp_postmeta: Data: 12.52MB + Index: 7.56MB
wp_posts: Data: 3.52MB + Index: 0.81MB
wp_termmeta: Data: 0.19MB + Index: 0.27MB
wp_terms: Data: 0.22MB + Index: 0.22MB
wp_term_relationships: Data: 1.34MB + Index: 0.41MB
wp_term_taxonomy: Data: 0.31MB + Index: 0.31MB
wp_tinvwl_analytics: Data: 0.02MB + Index: 0.02MB
wp_tinvwl_items: Data: 0.02MB + Index: 0.00MB
wp_tinvwl_lists: Data: 0.02MB + Index: 0.00MB
wp_tm_taskmeta: Data: 0.02MB + Index: 0.03MB
wp_tm_tasks: Data: 0.02MB + Index: 0.02MB
wp_usermeta: Data: 0.02MB + Index: 0.03MB
wp_users: Data: 0.02MB + Index: 0.05MB
wp_wcpdf_invoice_number: Data: 0.02MB + Index: 0.00MB
wp_wc_admin_notes: Data: 0.02MB + Index: 0.00MB
wp_wc_admin_note_actions: Data: 0.02MB + Index: 0.02MB
wp_wc_customer_lookup: Data: 0.02MB + Index: 0.03MB
wp_wc_download_log: Data: 0.02MB + Index: 0.03MB
wp_wc_order_coupon_lookup: Data: 0.02MB + Index: 0.03MB
wp_wc_order_product_lookup: Data: 0.02MB + Index: 0.06MB
wp_wc_order_stats: Data: 0.02MB + Index: 0.05MB
wp_wc_order_tax_lookup: Data: 0.02MB + Index: 0.03MB
wp_wc_product_meta_lookup: Data: 0.22MB + Index: 0.47MB
wp_wc_webhooks: Data: 0.02MB + Index: 0.02MB
wp_yith_wcwl: Data: 0.02MB + Index: 0.02MB
wp_yith_wcwl_lists: Data: 0.02MB + Index: 0.03MB
Post Type Counts
attachment: 1011
custom_css: 1
customize_changeset: 5
elementor_library: 7
jp_img_sitemap: 1
jp_sitemap: 1
jp_sitemap_master: 1
nav_menu_item: 100
page: 60
polylang_mo: 3
post: 3
product: 186
product_variation: 1964
revision: 16
scheduled-action: 698
wpcf7_contact_form: 2
wpforms: 1
Security
Secure connection (HTTPS): ✔
Hide errors from visitors: ✔
Active Plugins (36)
Polylang: by WP SYNTEX – 2.6.3
Livemesh Addons for Elementor: by Livemesh – 2.7.4
AliDropship Woo Plugin: by Vitaly Kukin & Yaroslav Nevskiy & Pavel Shishkin & Denis Zharov – 1.5.8
All In One SEO Pack: by Michael Torbert – 3.2.3
All-in-One WP Migration File Extension: by ServMask
Inc. – 1.5
All-in-One WP Migration: by ServMask – 7.4
WooCommerce MercadoPago: by Mercado Pago – 3.1.0 – Not tested with the active version of WooCommerce
Classic Editor: by WordPress Contributors – 1.5
Polylang Connect for Elementor: by David Decker - DECKERWEB – 1.0.0
Contact Form 7: by Takayuki Miyoshi – 5.1.4
Display PHP Version: by David Gwyer – 1.5
Elementor: by Elementor.com – 2.6.8
Envato Elements: by Envato – 1.1.1
Essential Addons for Elementor: by WPDeveloper – 3.1.5
Site Kit by Google: by Google – 1.0.0-beta.1.0.3
My Custom Functions: by Space X-Chimp – 4.38.1
Ocean Custom Sidebar: by OceanWP – 1.0.4
Ocean Extra: by OceanWP – 1.5.10
Ocean Product Sharing: by OceanWP – 1.0.7 – Not tested with the active version of WooCommerce
Ocean Social Sharing: by OceanWP – 1.0.14
Ocean Stick Anything: by OceanWP – 1.0.2
One Click Demo Import: by ProteusThemes – 2.5.2
PowerPack Lite for Elementor: by IdeaBox Creations – 1.2.5
Premium Addons for Elementor: by Leap13 – 3.6.4
Shortcode in Menus: by Gagan Deep Singh – 3.5
Sticky Header OceanWP: by Oren Hahiashvili – 1.0.5
TI WooCommerce Wishlist Plugin: by TemplateInvaders – 1.13.2
Toolbar Extras: by David Decker - DECKERWEB – 1.4.4
Hyyan WooCommerce Polylang Integration: by Hyyan Abo Fakher – 1.4.3
WooCommerce Refund and Exchange Lite: by MakeWebBetter – 1.0.5
WooCommerce Admin: by WooCommerce – 0.16.0
WooCommerce PayPal Checkout Gateway: by WooCommerce – 1.6.17
WooCommerce Services: by Automattic – 1.21.0
WooCommerce: by Automattic – 3.6.5
WP-Optimize - Clean, Compress, Cache: by David Anderson
Ruhani Rabin
Team Updraft – 3.0.11
WPForms Lite: by WPForms – <IP_ADDRESS>
Inactive Plugins (4)
Cloudflare: by John Wineman
Furkan Yilmaz
Junade Ali (Cloudflare Team) – 3.3.3
Duplicator: by Snap Creek – 1.3.18
Jetpack by WordPress.com: by Automattic – 7.6
WooCommerce Quick Buy: by Varun Sridharan – 1.9 – Not tested with the active version of WooCommerce
Settings
API Enabled: –
Force SSL: –
Currency: BRL (R$)
Currency Position: left
Thousand Separator: .
Decimal Separator: ,
Number of Decimals: 2
Taxonomies: Product Types: external (external)
grouped (grouped)
simple (simple)
variable (variable)
Taxonomies: Product Visibility: exclude-from-catalog (exclude-from-catalog)
exclude-from-search (exclude-from-search)
featured (featured)
outofstock (outofstock)
rated-1 (rated-1)
rated-2 (rated-2)
rated-3 (rated-3)
rated-4 (rated-4)
rated-5 (rated-5)
Connected to WooCommerce.com: ✔
WC Pages
Shop base: #6 - /shop/
Cart: #7 - /cart/
Checkout: #8 - /checkout/
My account: #9 - /my-account/
Terms and conditions: #1573 - /terms-of-service/
Theme
Name: OceanWP
Version: 1.6.10
Author URL: https://oceanwp.org/about-me/
Child Theme: ❌ – If you are modifying WooCommerce on a parent theme that you did not build personally we recommend using a child theme. See: How to create a child theme
WooCommerce Support: ✔
Templates
Overrides: oceanwp/woocommerce/cart/mini-cart.php
oceanwp/woocommerce/content-single-product.php
oceanwp/woocommerce/loop/loop-start.php
oceanwp/woocommerce/single-product/title.php
TI WooCommerce Wishlist Templates
Overrides: –
Action Scheduler
Complete: 697
Oldest: 2019-07-08 23:50:42 -0300
Newest: 2019-08-08 23:21:55 -0300
Pending: 1
Oldest: 2019-08-09 00:21:55 -0300
Newest: 2019-08-09 00:21:55 -0300
Canceled: 0
Oldest: –
Newest: –
In-progress: 0
Oldest: –
Newest: –
Failed: 0
Oldest: –
Newest: –
</details>

@raz0r7 what do you mean by "Products Price List / Pricing Markup"

Hi Jon,
First of all, thank you for trying to help.
In my site I use AliDropShip to import that products from AliExpress.
Products Price List / Pricing Markup is the section where we setup rules for pricing to be rolled out to the entire site.
What happens is that once the rules are set in this section, it applies only to the default language. Translated pages derived from this are not getting the pricing rules.
I sent the following message to AliDropShip plugin support team:
quote
Hi AliDrop Support team,
I have already reported some issues involving translated products, but I could find a workaround for those, however this time, something must be done at your side, otherwise it will compromise my whole work and website.
The question involves Pricing Markup rules for translated products as well as pricing replications to these products.
Plugins:
AliDropShip
Plugin Translator – Polylang
Integrator - Hyyan WooCommerce Polylang Integration
I noted something odd when I was translating the products and after setting up a rule in Pricing Markup, it was not replicating the prices for these languages. To make it work, I had to go the each translated product and just click on “update” button to make it work. It was not a problem If I had to make it once on the moment that product was being translated, however it got worst.
AlidropShip settings updates on daily basis the prices from AliExpress in USD. It automatically updates all products in the site, but only in English, the default site language.
Now, translated products pages in others languages such as Portuguese and Spanish are not being updated automatically from AliDropShip, and it will impact severely all work Ive done since I bought the Plugin, because the site is multilingual. Can´t have only the default language being updated, AliDropship must replicate pricing updates to them all.
It doesn’t matter If I set a rule to cover translated categories for example, the “main” pricing rules are not being replicated to translated pages.
Please see below some print shots :
Currency settings ( no matter the default currency, USD, BRL,EUR, the issues happens anyway, I have already tested it).
[sent some prints and adm credentials to them ]
And Spanish is the same as above in Portuguese.
Another similar issue that is happenings was reported and it’s attached to this message, the moment a products is translated, the default product page loses the variations settings and then I must set it manually which give me a lot of unnecessary work. Its someway related to the problem reported in this email, involving translated products. I believe its 2 issues with 1 root cause.
Please get your developers to investigate into this issue, and fix the problem, its jeopardizing all my work.
Thank you for your usual assistance.
Best Regards,
Rafael Pereira
unquote
So, this is the msg sent to them and I'm waiting for a solution,  believe now its on their end the real solution but if you can somehow contribute with any knowledge, its a plus.
Thank you Jon.
Rafael

Hi Jon,
I received a feedback from their customer service.
Actually their support for the plugin is terrible.
They will not provide any support for my issue, they just suggested me to use another platform where translations must be done manually. Can´t be believe that. And also supported only from few themes, not the one I use, which is Ocean View.
Can´t believe they will not give me a hand by solving this issue, which is to force the pricing rules to be rolled about to all product pages, no matter the language.
I don't know where to ask help anymore.

Hi @raz0r7 I’m not familiar with the plugin but the normal thing is to identify what additional metadata they are using and use the appropriate filters to control which of these fields are synchronised.
See https://github.com/hyyan/woo-poly-integration/wiki/Product-Meta-and-Custom-Fields

it’s not great but may give you some ideas

I sent this article to them. With a lots of complains, of course.
Its unacceptable this plugin conflicts with Polylang, widely used worldwide.
Let's see what they will answer back. I will keep you posted.
Thank you Jon!
Rafael

Hi Jon, look I what found:
https://wpml.org/forums/topic/how-to-update-price-of-the-product/
Its exactly the problem I'm having. Until now, no response from them after my last message.
Honestly, I don't think they will help after last exchanged messages, so I'm ready to set the site to one single language if indeed they don't provide any help. Unfortunately, because I designed the site to be multilingual. Lets see what happens until the end of this week.

Right yes the thread refers to
“the use of custom tables that hold the information that you want to synchronise that lie outside of the normal Woocommerce solution”
So a custom solution would be needed,
Which also has to be outside of the scope of this plugin.

Thank you Jon for your attention on this matter and for trying to help me.
Best regards,
Rafael

no problem.
for reference, what is the url for this AliDropShip and is it open source?

Hi Jon,
URL - https://alidropship.com/
I'm not sure it's open source, it's a paid plugin, 89 USD, but you can get more info by contacting them.


sudo bash centos2ol.sh -V Could not install base packages. Run 'yum distro-sync' to manually install them.

CentOS 7.
wget https://raw.githubusercontent.com/oracle/centos2ol/main/centos2ol.sh
--2021-01-25 00:06:54--  https://raw.githubusercontent.com/oracle/centos2ol/main/centos2ol.sh
Resolving raw.githubusercontent.com (raw.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to raw.githubusercontent.com (raw.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 20588 (20K) [text/plain]
Saving to: ‘centos2ol.sh’
100%[===================================================================================================================================================================================================>] 20,588      --.-K/s   in 0.002s
2021-01-25 00:06:54 (9.55 MB/s) - ‘centos2ol.sh’ saved [20588/20588]
[root@bitrix /]# sudo bash centos2ol.sh -V

Bitrix.zip
A similar problem was discussed
https://github.com/oracle/centos2ol/issues/51
Could not install base packages.
Run 'yum distro-sync' to manually install them.
For assistance, please open an issue via GitHub: https://github.com/oracle/centos2ol/.

[root@bitrix /]# yum distro-sync
Loaded plugins: etckeeper, fastestmirror, merge-conf
Loading mirror speeds from cached hostfile
epel/x86_64/metalink                                                                                |  33 kB  00:00:00
 * epel: mirror.cspacehostings.com
 * remi: mirror.awanti.com
 * remi-php72: mirror.awanti.com
 * remi-safe: mirror.awanti.com
bitrix                                                                                | 2.9 kB  00:00:00
ol7_UEKR6                                                                                | 2.5 kB  00:00:00
ol7_latest                                                                                | 2.7 kB  00:00:00
http://repo.percona.com/percona/yum/release/%24releasever/RPMS/noarch/repodata/repomd.xml: [Errno 14] HTTP Error 404 - Not Found
Trying other mirror.
To address this issue please refer to the below wiki article

https://wiki.centos.org/yum-errors

If above article doesn't help to resolve this issue please use https://bugs.centos.org/.

 One of the configured repositories failed (Percona Original release/noarch YUM repository),
 and yum doesn't have enough cached data to continue. At this point the only
 safe thing yum can do is fail. There are a few ways to work "fix" this:

     1. Contact the upstream for the repository and get them to fix the problem.

     2. Reconfigure the baseurl/etc. for the repository, to point to a working
        upstream. This is most often useful if you are using a newer
        distribution release than is supported by the repository (and the
        packages for the previous distribution release still work).

     3. Run the command with the repository temporarily disabled
            yum --disablerepo=percona-release-noarch ...

     4. Disable the repository permanently, so yum won't use it by default. Yum
        will then just ignore the repository until you permanently enable it
        again or use --enablerepo for temporary usage:

            yum-config-manager --disable percona-release-noarch
        or
            subscription-manager repos --disable=percona-release-noarch

     5. Configure the failing repository to be skipped, if it is unavailable.
        Note that yum will try to contact the repo. when it runs most commands,
        so will have to try and fail each time (and thus. yum will be be much
        slower). If it is a very temporary problem though, this is often a nice
        compromise:

            yum-config-manager --save --setopt=percona-release-noarch.skip_if_unavailable=true

failure: repodata/repomd.xml from percona-release-noarch: [Errno 256] No more mirrors to try.
http://repo.percona.com/percona/yum/release/$releasever/RPMS/noarch/repodata/repomd.xml: [Errno 14] HTTP Error 404 - Not Found

 cat /etc/yum.conf
[main]
cachedir=/var/cache/yum/$basearch/$releasever
keepcache=0
debuglevel=2
logfile=/var/log/yum.log
exactarch=1
obsoletes=1
gpgcheck=1
plugins=1
installonly_limit=5
bugtracker_url=http://bugs.centos.org/set_project.php?project_id=23&ref=http://bugs.centos.org/bug_report_page.php?category=yum
distroverpkg=centos-release

#  This is the default, if you make this bigger yum won't see if the metadata
# is newer on the remote and so you'll "gain" the bandwidth of not having to
# download the new metadata and "pay" for it by yum not having correct
# information.
#  It is esp. important, to have correct metadata, for distributions like
# Fedora which don't keep old packages around. If you don't like this checking
# interupting your command line usage, it's much better to have something
# manually check the metadata once an hour (yum-updatesd will do this).
# metadata_expire=90m

# PUT YOUR REPOS HERE OR IN separate files named file.repo
# in /etc/yum.repos.d
exclude=ansible1.9,mariadb,mariadb-*,Percona-XtraDB-*

 cat /etc/yum.conf
[main]
cachedir=/var/cache/yum/$basearch/$releasever
keepcache=0
debuglevel=2
logfile=/var/log/yum.log
exactarch=1
obsoletes=1
gpgcheck=1
plugins=1
installonly_limit=5
bugtracker_url=http://bugs.centos.org/set_project.php?project_id=23&ref=http://bugs.centos.org/bug_report_page.php?category=yum
distroverpkg=centos-release

#  This is the default, if you make this bigger yum won't see if the metadata
# is newer on the remote and so you'll "gain" the bandwidth of not having to
# download the new metadata and "pay" for it by yum not having correct
# information.
#  It is esp. important, to have correct metadata, for distributions like
# Fedora which don't keep old packages around. If you don't like this checking
# interupting your command line usage, it's much better to have something
# manually check the metadata once an hour (yum-updatesd will do this).
# metadata_expire=90m

# PUT YOUR REPOS HERE OR IN separate files named file.repo
# in /etc/yum.repos.d
exclude=ansible1.9,mariadb,mariadb-*,Percona-XtraDB-*

Do you have an /etc/yum.conf.rpmnew file? Because you edited the file to add the exclude line, the initial switch can't change this file automatically. We'll have to address that.

Do you have an /etc/yum.conf.rpmnew file? Because you edited the file to add the exclude line, the initial switch can't change this file automatically. We'll have to address that.

Yes:
 ls -lh /etc/yum.conf*
-rw-r--r-- 1 root root 1.1K Jan 25 00:35 /etc/yum.conf
-rw-r--r-- 1 root root  813 Jul 20  2020 /etc/yum.conf.rpmnew

Yes:
 ls -lh /etc/yum.conf*
-rw-r--r-- 1 root root 1.1K Jan 25 00:35 /etc/yum.conf
-rw-r--r-- 1 root root  813 Jul 20  2020 /etc/yum.conf.rpmnew

Do you have an /etc/yum.conf.rpmnew file? Because you edited the file to add the exclude line, the initial switch can't change this file automatically. We'll have to address that.

I edited the file after yum distro-sync

Do you have an /etc/yum.conf.rpmnew file? Because you edited the file to add the exclude line, the initial switch can't change this file automatically. We'll have to address that.

I edited the file after yum distro-sync

Ok, thanks. We need to modify the script to replace the modified /etc/yum.conf with the new version and save the old one for reference.

Ok, thanks. We need to modify the script to replace the modified /etc/yum.conf with the new version and save the old one for reference.

How to do it right?
Manually edit the yum.conf file?
Or will you fix the script?

How to do it right?
Manually edit the yum.conf file?
Or will you fix the script?


docs(helm): Added kubeVersion to documentation.

docs(helm): Added kubeVersion to documentation and examples.

Thank you @eicnix Would you be able to squash this into one commit?

@jascott1 I squashed the commits


feat(lock): add dete-picker-view component

Description
Please provide a brief summary of the changes. If it is related to an issue, please bring the issue number, like fixes #${number}.
Package

[ ] tuya-panel-animation-sdk
[ ] tuya-panel-api
[ ] tuya-panel-electrician-sdk
[ ] tuya-panel-ipc-sdk
[ ] tuya-panel-lamp-sdk
[x] tuya-panel-lock-sdk
[ ] tuya-panel-robot-sdk

Type of change
Please select the relevant option.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] 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

Codecov Report

Merging #51 (0fdc7a6) into main (129b598) will increase coverage by 2.19%.
The diff coverage is 83.49%.

@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
+ Coverage   56.51%   58.70%   +2.19%
==========================================
  Files         129      131       +2
  Lines        4721     5139     +418
  Branches     1101     1202     +101
==========================================
+ Hits         2668     3017     +349
- Misses       2051     2120      +69
  Partials        2        2

Impacted Files
Coverage Δ

...panel-lock-sdk/src/date-picker-view/datePicker.tsx
83.48% <83.48%> (ø)

...tuya-panel-lock-sdk/src/date-picker-view/index.tsx
83.50% <83.50%> (ø)

Continue to review full report at Codecov.

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

#53


SVG export bounding box incorrect.

When exporting an SVG with transperancy, the bounding box doesn't seem to include the outer stroke weight. So if you export a square with a stroke width of 8 px, the exported image will have the outer stroke cut off.
This is also the case in the dev deployment.
The exported svg:  (notice how the borders are visibly cut off)

The project file (renamed to have .txt so github lets me upload it):
Spelling bee cell.graphite.txt
Changing the SVG viewbox, width, height, and remoxing the transformation to restore the boundaries to those present in the project (1000px^2 for me) fixed the issue completely.

Duplicate of #2113.

I'm actually going to make #2113 a duplicate of this, even though this is more recent, because it more clearly describes the problem.

Apologies for the duplicate, I tried looking for SVG errors, not realizing the issue applied to other formats too (as described by #2113) It's really not that big of a deal for me, but I figured you could decide if it's worth fixing.


nginx is compiled and installed using make error

How should I fix it?

Centos 7 Minimal + Nginx-1.8.1
I have used Compilation recipes at https://github.com/SpiderLabs/ModSecurity/wiki/Compilation-recipes

Hi @maxwalk,
please try the v3/dev/parser branch.
Also, nginx/1.8.1 is more than 1 year old, I'd encourage you to use most recent mainline version for testing (1.11.10 at the moment).

Just a quick notice: As of now the v3/dev/parser on ModSecurity was merged into v3/master. Same for the ModSecurity-nginx.


Cleanup controller bootstrap

This PR removes the obsolete TODO in controller bootstrap, ref. https://github.com/cert-manager/trust-manager/pull/187#pullrequestreview-1645832202 and ensures that we only cache full configmaps in source namespace.
/cc @inteon

/retest

Thanks a lot @erikgb, I really appreciate what you do for the project!
/lgtm
/approve


Add a test to check that we can upgrade/downgrade to the commit when we publish a new dev/beta build

The roll script should test upgrading/downgrading to the new dev build.
cc @tvolkert

@Hixie can you provide repro steps, the behavior you're seeing and the expected behavior?

Triage: @pcsosinski Is this still needed?

yes, this is currently a manual test step for a beta release.

@christopherfujino is this something that can be automated?

@christopherfujino is this something that can be automated?

yeah, we have some integration test coverage on this, I'll assign to myself to track.

yep


hash length mistake of ExtrinsicSigner.java

hash is 32 byte length

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Falcon outputs to Falcon_unzip

Dear PacBio team,
I run Falcon and get aassembly genome with 548 contigs and I change some criteria of "run_falcon_asm.sh" then rerun it in 2-asm-falcon folder to get better assembly genome result(614contigs). Then I run falcon_unzip but result look like that falcon_unzip use old data from falcon because they only contain 540 contigs in  all_p_ctg.fa . I also found that in 3-unzip/reads/ctg_list only contain 548 contigs name. Is there a way to fix it?
Thank you.
Best,
Jeff

You have to run unzip from a binary release. (And run falcon from the same binary, so you have consistent versions.)

https://github.com/PacificBiosciences/FALCON_unzip/wiki/Binaries


I can not find the problem here.


Move demo from SSP demo from IBM org.

Issue description / feature objectives
There is a demo which is currently located here: https://github.com/IBM/compliance-trestle-ssp-demo
There is an open question by the IBM team as to whether it should be contributed to the oscal-compass team.
Caveats / Assumptions
Assumption is that there is minimal maintenance / overhead associated with the demo.
Completion Criteria

[ ] Demo is moved
[ ] Minimum CNCF updates (footers; copyright etc) are updated
[ ] Compliance-trestle documentation is updated.

I asked JJ to move the repo. I suspect it will happen pretty quickly, likely Monday morning if not sooner.

@butler54 The repo has been moved. Does this issue cover that alone (then this issue can be closed) or the ramifications to trestle as well?

I can edit the PR now for the documentation.
I think the big issue as highlighted would the required CNCF documentation on the repository and updating the links (e.g. it points back to the github.com/ibm and ibm.github.io URLs across the README.md).

See https://github.com/oscal-compass/compliance-trestle-ssp-demo/pull/13.

degenaro:trestle.develop$ grep -r ssp-demo *
docs/tutorials/Trestle_authoring/ssp_profile_catalog_authoring.md:There is a standalone demonstration of the SSP generation process with trestle in the Trestle SSP Demo that captures the entire process of SSP authoring: from creation of a component definition from CSV file to a final formatted system security plan in Word (.docx) format.

@butler54 @jpower432 @AleJo2995 @vikas-agarwal76 So far, there is no code code change required to the present repo, so no PR is needed. Not sure how to get "official" review in this case.
See above for link to PR for the ssp demo repo.


Fix WIN32_LEAN_AND_MEAN spam

Compiling on windows/mingw throws a lot of warnings "WIN32_LEAN_AND_MEAN"

has this been fixed?

yes


Access to a crate repository

It would be awesome if we could access to a crate repository directly from query.rs

You can use !!!tokio to open the repository, https://query.rs/?q=!!!tokio

I didn't know.
Thanks for this awesome project


discord killed

Starting installation for Discord
Killing Discord Processes
Downloading BandagedBD package
Download seems to have failed, will try once more.
Downloading BandagedBD package

wdym discord killed ? :))
killing discord processes means they closed everything that connects to discord

You have an issue with the installer downloading the installation payload. Follow the manual installation instructions.  https://github.com/rauenzi/BetterDiscordApp#manual-installation


MIOpenDriver - Build Failure on 2.11.0

[100%] Building CXX object driver/CMakeFiles/MIOpenDriver.dir/InputFlags.cpp.o
In file included from /neuralNet/MIOpen-2.11.0/driver/main.cpp:31:0:
/neuralNet/MIOpen-2.11.0/driver/conv_driver.hpp:226:10: error: 'bool ConvDriver<Tgpu, Tref>::UseGPUReference()' cannot be overloaded
     bool UseGPUReference();
          ^~~~~~~~~~~~~~~
/neuralNet/MIOpen-2.11.0/driver/conv_driver.hpp:224:10: error: with 'bool ConvDriver<Tgpu, Tref>::UseGPUReference()'
     bool UseGPUReference();
          ^~~~~~~~~~~~~~~
cc1plus: warning: unrecognized command line option '-Wno-extra-semi-stmt'
cc1plus: warning: unrecognized command line option '-Wno-extra-semi-stmt'
make[2]: *** [driver/CMakeFiles/MIOpenDriver.dir/main.cpp.o] Error 1
driver/CMakeFiles/MIOpenDriver.dir/build.make:62: recipe for target 'driver/CMakeFiles/MIOpenDriver.dir/main.cpp.o' failed
make[1]: *** [driver/CMakeFiles/MIOpenDriver.dir/all] Error 2
CMakeFiles/Makefile2:5791: recipe for target 'driver/CMakeFiles/MIOpenDriver.dir/all' failed
make: *** [all] Error 2
Makefile:162: recipe for target 'all' failed

@kiritigowda We are unable to reproduce this issue. Could you clean the workspace and try again? What is your build environment and configurations? If the issuer persists we can further investigate with full logs available. Thanks!

@junliume thanks for looking at this. I have attached a dockerfile from which you can reproduce this issue. I would really appreciate it if you can tell me what's missing or get to the root of the issue.
to build the docker use the command below.
sudo docker build -f miopenTest.dockerfile -t miopentest .

Issue below
[ 96%] Building CXX object src/CMakeFiles/MIOpen.dir/ocl/clhelper.cpp.o
In file included from /usr/local/include/boost/container/small_vector.hpp:27:0,
                 from /neuralNet/MIOpen-2.11.0/src/include/miopen/op_kernel_args.hpp:8,
                 from /neuralNet/MIOpen-2.11.0/src/include/miopen/oclkernel.hpp:44,
                 from /neuralNet/MIOpen-2.11.0/src/include/miopen/kernel.hpp:43,
                 from /neuralNet/MIOpen-2.11.0/build/kernel_includes.cpp:29:
/usr/local/include/boost/container/vector.hpp: In member function 'void boost::container::vector<T, Allocator, Options>::priv_forward_range_insert_new_allocation(T*, boost::container::vector<T, Allocator, Options>::size_type, T*, boost::container::vector<T, Allocator, Options>::size_type, InsertionProxy)':
/usr/local/include/boost/container/vector.hpp:3031:0: note: -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
          this->m_holder.deallocate(this->m_holder.start(), this->m_holder.capacity());

In file included from /usr/include/c++/7/algorithm:61:0,
                 from /neuralNet/MIOpen-2.11.0/build/kernel.cpp:27:
/usr/include/c++/7/bits/stl_algobase.h: In function '_ForwardIterator2 std::swap_ranges(_ForwardIterator1, _ForwardIterator1, _ForwardIterator2)':
/usr/include/c++/7/bits/stl_algobase.h:178:0: note: -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
       return __first2;

[100%] Linking CXX shared library ../lib/libMIOpen.so
[100%] Built target MIOpen
Scanning dependencies of target MIOpenDriver
[100%] Building CXX object driver/CMakeFiles/MIOpenDriver.dir/main.cpp.o
[100%] Building CXX object driver/CMakeFiles/MIOpenDriver.dir/InputFlags.cpp.o
In file included from /neuralNet/MIOpen-2.11.0/driver/main.cpp:31:0:
/neuralNet/MIOpen-2.11.0/driver/conv_driver.hpp:226:10: error: 'bool ConvDriver<Tgpu, Tref>::UseGPUReference()' cannot be overloaded
     bool UseGPUReference();
          ^~~~~~~~~~~~~~~
/neuralNet/MIOpen-2.11.0/driver/conv_driver.hpp:224:10: error: with 'bool ConvDriver<Tgpu, Tref>::UseGPUReference()'
     bool UseGPUReference();
          ^~~~~~~~~~~~~~~
cc1plus: warning: unrecognized command line option '-Wno-extra-semi-stmt'
cc1plus: warning: unrecognized command line option '-Wno-extra-semi-stmt'
driver/CMakeFiles/MIOpenDriver.dir/build.make:62: recipe for target 'driver/CMakeFiles/MIOpenDriver.dir/main.cpp.o' failed
make[2]: *** [driver/CMakeFiles/MIOpenDriver.dir/main.cpp.o] Error 1
CMakeFiles/Makefile2:5791: recipe for target 'driver/CMakeFiles/MIOpenDriver.dir/all' failed
make[1]: *** [driver/CMakeFiles/MIOpenDriver.dir/all] Error 2
make: *** [all] Error 2
Makefile:162: recipe for target 'all' failed
The command '/bin/sh -c apt-get -y install sqlite3 libsqlite3-dev libbz2-dev libssl-dev python-dev python3-dev autoconf automake libtool curl make g++ unzip &&         mkdir neuralNet && cd neuralNet && wget https://sourceforge.net/projects/half/files/half/1.12.0/half-1.12.0.zip &&         unzip half-1.12.0.zip -d half-files && sudo cp half-files/include/half.hpp /usr/local/include/ &&         wget https://boostorg.jfrog.io/artifactory/main/release/1.72.0/source/boost_1_72_0.tar.bz2 && tar xjvf boost_1_72_0.tar.bz2 &&         cd boost_1_72_0 && ./bootstrap.sh --prefix=/usr/local --with-python=python3 &&         ./b2 stage -j16 threading=multi link=shared cxxflags="-std=c++11" &&         sudo ./b2 install threading=multi link=shared --with-system --with-filesystem &&         ./b2 stage -j16 threading=multi link=static cxxflags="-std=c++11 -fpic" cflags="-fpic" &&         sudo ./b2 install threading=multi link=static --with-system --with-filesystem && cd ../ &&         git clone https://github.com/RadeonOpenCompute/rocm-cmake.git && cd rocm-cmake && mkdir build && cd build &&         cmake ../ && make -j8 && sudo make install && cd ../../ &&         wget https://github.com/ROCmSoftwarePlatform/MIOpenGEMM/archive/1.1.5.zip && unzip 1.1.5.zip &&         cd MIOpenGEMM-1.1.5 && mkdir build && cd build && cmake ../ && make -j8 && sudo make install && cd ../../ &&         wget https://github.com/ROCmSoftwarePlatform/MIOpen/archive/2.11.0.zip && unzip 2.11.0.zip &&         cd MIOpen-2.11.0 && mkdir build && cd build &&         cmake -DMIOPEN_BACKEND=OpenCL -DMIOPEN_USE_MIOPENGEMM=On ../ && make -j8 && sudo make install && cd ../../ &&         git clone -b v3.12.0 https://github.com/protocolbuffers/protobuf.git && cd protobuf && git submodule update --init --recursive &&         ./autogen.sh && ./configure && make -j8 && make check -j8 && sudo make install && sudo ldconfig && cd' returned a non-zero code: 2

file renamed as txt
miopenTest.txt

@carlushuang Issue is reproducible using user provided docker file, could you take a look?

@kiritigowda This issue has been fixed. Please confirm.

@JehandadKhan thanks for the fix, I will confirm with the build.

2.11.0 is now building. Thanks, @JehandadKhan @junliume for the quick response.
[ 96%] Building CXX object src/CMakeFiles/MIOpen.dir/ocl/clhelper.cpp.o
In file included from /usr/local/include/boost/container/small_vector.hpp:27:0,
                 from /neuralNet/MIOpen-2.11.0/src/include/miopen/op_kernel_args.hpp:8,
                 from /neuralNet/MIOpen-2.11.0/src/include/miopen/oclkernel.hpp:44,
                 from /neuralNet/MIOpen-2.11.0/src/include/miopen/kernel.hpp:43,
                 from /neuralNet/MIOpen-2.11.0/build/kernel_includes.cpp:29:
/usr/local/include/boost/container/vector.hpp: In member function 'void boost::container::vector<T, Allocator, Options>::priv_forward_range_insert_new_allocation(T*, boost::container::vector<T, Allocator, Options>::size_type, T*, boost::container::vector<T, Allocator, Options>::size_type, InsertionProxy)':
/usr/local/include/boost/container/vector.hpp:3031:0: note: -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
          this->m_holder.deallocate(this->m_holder.start(), this->m_holder.capacity());

In file included from /usr/include/c++/7/algorithm:61:0,
                 from /neuralNet/MIOpen-2.11.0/build/kernel.cpp:27:
/usr/include/c++/7/bits/stl_algobase.h: In function '_ForwardIterator2 std::swap_ranges(_ForwardIterator1, _ForwardIterator1, _ForwardIterator2)':
/usr/include/c++/7/bits/stl_algobase.h:178:0: note: -Wmisleading-indentation is disabled from this point onwards, since column-tracking was disabled due to the size of the code/headers
       return __first2;

[100%] Linking CXX shared library ../lib/libMIOpen.so
[100%] Built target MIOpen
Scanning dependencies of target MIOpenDriver
[100%] Building CXX object driver/CMakeFiles/MIOpenDriver.dir/InputFlags.cpp.o
[100%] Building CXX object driver/CMakeFiles/MIOpenDriver.dir/main.cpp.o
[100%] Linking CXX executable ../bin/MIOpenDriver
[100%] Built target MIOpenDriver
[  0%] Built target sqlite_memvfs
[  3%] Built target addkernels
[100%] Built target MIOpen
[100%] Built target MIOpenDriver
Install the project...

@junliume attached is a docker file that uses the deps install from MIOpen, this currently fails. Please take a look at it when you get a chance. Thanks!
Updated DockerFile - miopenTest.txt
sudo cmake -P install_deps.cmake --minimum

Failure
Downloading https://dl.bintray.com/boostorg/release/1.72.0/source/boost_1_72_0.tar.bz2
CMake Error at /usr/local/share/cmake/cmakeget/CMakeGet.cmake:151 (message):
  Download failed: "HTTP response code said error":
  https://dl.bintray.com/boostorg/release/1.72.0/source/boost_1_72_0.tar.bz2;/tmp/cget-19-28-37-zW0IM-1/download/boost_1_72_0.tar.bz2;SHOW_PROGRESS
Call Stack (most recent call first):
  /usr/local/share/cmake/cmakeget/CMakeGet.cmake:293 (cget_download)
  /usr/local/share/cmake/cmakeget/CMakeGet.cmake:394 (cget_fetch)
  /usr/local/share/cmake/cmakeget/CMakeGet.cmake:461 (cmake_get)
  /usr/local/share/cmake/cmakeget/CMakeGet.cmake:383 (cmake_get_from)
  /usr/local/share/cmake/cmakeget/CMakeGet.cmake:461 (cmake_get)
  install_deps.cmake:106 (cmake_get_from)

The command '/bin/sh -c apt-get -y install sqlite3 libsqlite3-dev libbz2-dev libssl-dev python-dev python3-dev autoconf automake libtool curl make g++ unzip &&         mkdir neuralNet && cd neuralNet && wget https://sourceforge.net/projects/half/files/half/1.12.0/half-1.12.0.zip &&         unzip half-1.12.0.zip -d half-files && sudo cp half-files/include/half.hpp /usr/local/include/ &&         wget https://boostorg.jfrog.io/artifactory/main/release/1.72.0/source/boost_1_72_0.tar.bz2 && tar xjvf boost_1_72_0.tar.bz2 &&         cd boost_1_72_0 && ./bootstrap.sh --prefix=/usr/local --with-python=python3 &&         ./b2 stage -j16 threading=multi link=shared cxxflags="-std=c++11" &&         sudo ./b2 install threading=multi link=shared --with-system --with-filesystem &&         ./b2 stage -j16 threading=multi link=static cxxflags="-std=c++11 -fpic" cflags="-fpic" &&         sudo ./b2 install threading=multi link=static --with-system --with-filesystem && cd ../ &&         git clone https://github.com/RadeonOpenCompute/rocm-cmake.git && cd rocm-cmake && mkdir build && cd build &&         cmake ../ && make -j8 && sudo make install && cd ../../ &&         wget https://github.com/ROCmSoftwarePlatform/MIOpenGEMM/archive/1.1.6.zip && unzip 1.1.6.zip &&         cd MIOpenGEMM-1.1.6 && mkdir build && cd build && cmake ../ && make -j8 && sudo make install && cd ../../ &&         wget https://github.com/ROCmSoftwarePlatform/MIOpen/archive/2.11.0.zip && unzip 2.11.0.zip &&         cd MIOpen-2.11.0 && mkdir build && sudo cmake -P install_deps.cmake --minimum && cd build &&         cmake -DMIOPEN_BACKEND=OpenCL -DMIOPEN_USE_MIOPENGEMM=On ../ && make -j8 && sudo make install && cd ../../ &&         git clone -b v3.12.0 https://github.com/protocolbuffers/protobuf.git && cd protobuf && git submodule update --init --recursive &&         ./autogen.sh && ./configure && make -j8 && make check -j8 && sudo make install && sudo ldconfig && cd' returned a non-zero code: 1

https://dl.bintray.com/boostorg/release/1.72.0/source/boost_1_72_0.tar.bz2

Hi @kiritigowda, you may need to update the docker file to:

So instead of downloading from:
https://dl.bintray.com/boostorg/release/1.72.0/source/boost_1_72_0.tar.gz
You should download from:
https://boostorg.jfrog.io/artifactory/main/release/1.72.0/source/boost_1_72_0.tar.gz

P.S. why boost_1_72_0 instead of boost_1_76_0?
Related to:
https://github.com/ROCmSoftwarePlatform/MIOpen/issues/938
https://github.com/ROCmSoftwarePlatform/MIOpen/issues/856#issuecomment-827003984

https://dl.bintray.com/boostorg/release/1.72.0/source/boost_1_72_0.tar.bz2

Hi @kiritigowda, you may need to update the docker file to:

So instead of downloading from:
https://dl.bintray.com/boostorg/release/1.72.0/source/boost_1_72_0.tar.gz
You should download from:
https://boostorg.jfrog.io/artifactory/main/release/1.72.0/source/boost_1_72_0.tar.gz

P.S. why boost_1_72_0 instead of boost_1_76_0?
Related to:
#938
#856 (comment)

I am only running sudo cmake -P install_deps.cmake --minimum, I don't have control over what this file downloads.

sudo cmake -P install_deps.cmake --minimum

Let's track progress in #938
Meanwhile would the following workaround work for you?
https://github.com/ROCmSoftwarePlatform/MIOpen/issues/938#issuecomment-844455916

@junliume Thanks. Attached is a docker file to reproduce this issue.
FROM ubuntu:18.04

RUN apt-get update -y
# install base dependencies
RUN apt-get -y install gcc g++ cmake git
# install ROCm for OpenCL dependency
RUN apt-get -y install libnuma-dev wget sudo gnupg2 kmod &&  \
        wget -q -O - https://repo.radeon.com/rocm/rocm.gpg.key | sudo apt-key add - && \
        echo 'deb [arch=amd64] https://repo.radeon.com/rocm/apt/debian/ xenial main' | sudo tee /etc/apt/sources.list.d/rocm.list && \
        sudo apt-get update -y && \
        sudo apt-get -y install rocm-dev
# install miopen
RUN apt-get -y install sqlite3 libsqlite3-dev libbz2-dev libssl-dev python-dev python3-dev autoconf automake libtool curl make g++ unzip && \
        git clone https://github.com/RadeonOpenCompute/rocm-cmake.git && cd rocm-cmake && mkdir build && cd build && \
        cmake ../ && make -j8 && sudo make install && cd ../../ && \
        wget https://github.com/ROCmSoftwarePlatform/MIOpenGEMM/archive/1.1.6.zip && unzip 1.1.6.zip && \
        cd MIOpenGEMM-1.1.6 && mkdir build && cd build && cmake ../ && make -j8 && sudo make install && cd ../../ && \
        wget https://github.com/ROCmSoftwarePlatform/MIOpen/archive/2.11.0.zip && unzip 2.11.0.zip && \
        cd MIOpen-2.11.0 && sudo cmake -P install_deps.cmake --minimum && \
        mkdir build && cd build && \
        cmake -DMIOPEN_BACKEND=OpenCL -DMIOPEN_USE_MIOPENGEMM=On ../ && make -j8 && sudo make install && cd

miopenDeps.txt


Outdated Git Link

The link to "IntuneWinAppUtil.exe command-line tool." in Appendix B (https://github.com/Microsoft/Intune-Win32-App-Packaging-Tool) is out dated the repository is now at https://github.com/Microsoft/Microsoft-Win32-Content-Prep-Tool

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bcc63299-cc1c-e031-deb7-915914b8f94e
Version Independent ID: 356eaba5-16b0-952f-fc19-c8d5be5feb5f
Content: Demonstrate Autopilot deployment - Windows Deployment
Content Source: windows/deployment/windows-autopilot/demonstrate-deployment-on-vm.md
Product: w10
Technology: windows
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Thank you for pointing out this change. I will post a PR to have that link updated.


added 1598.cpp: issue number 1417

Fixes #{Insert your issue number here}

My code sample is in :

[ ] CPP
[ ] Java

Screenshot of working code sample  :

Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My code sample is formatted and works correctly

add issue number and needs formatting

I have made the changes

please see other merged prs for getting reference on how to update pr comment

Please can you check once again?

Kindly check my PR and please inform if anything else needs to be updated?


Use const-TLS for the WorkerThread pointer (MSRV 1.59)

This enables more efficient code -- stabilized in rust-lang/rust#91355.

bors r+


Add a lovely radio input component

💬 Description
Just like the previous checkbox pr, but in radio button form, oooh.
📹 GIF (optional)

✅ Checklist

[ ] Tests written
[ ] Browser tested
[ ] Added to documentation
[ ] Added to storybook

 Add custom radio / checkbox UI to gather-ui


Import refactoring

Hi there!
Following on from thoughts in https://github.com/pachterlab/ffq/issues/22 we have been playing with the idea of wrapping ffq in a very simple wrapper to offer it as a REST API web service. If you're ok with the idea, we were thinking that we would be happy to host this at @SeqeraLabs as an open service for anyone to use.
I whipped up a quick proof of concept using FastAPI in this repo: https://github.com/seqeralabs/ffq-api - it's running at https://ffq.seqera.io with docs here.
Thanks to the way that ffq is written, it was fantastically easy to do this (it's basically just these lines). However, I did find that I needed to make a few minor upstream changes to ffq to get everything to play nicely:

Moving the argparse CLI code into a separate function, so that the main call can be done without this
Removing the sys.exit(1) calls which make sense for a command-line tool, but which kill an API service.

 It should be functionally identical, but means that other tools making use of ffq as a library can capture and handle those exceptions however they need.
In addition to the API, these changes should also help people to use ffq in other situations, such as Jupyter Notebooks and inclusion into other Python tools.
On opening this PR I saw  that tests are failing - I think it's due to the NCBI issue you mention in the readme. If you'd like me to rebase / rewrite these changes against the devel branch instead then that's no problem. Equally if you'd prefer these changes done in a different way or not done at all then that's also fine 😀 Just thought it would be good to get the ball rolling..
Cheers, and many thanks again for a very useful tool 👏🏻
Phil

Hi Phil,
This is awesome- thanks for taking the time to make these changes. I think the idea of an open REST API service that uses ffq to fetch metadata would great to have and the changes you've made look to be useful beyond that as well. I think the replacing the sys.exit(1) calls with custom exceptions makes a lot of sense and that separating out the argparse setup from the main ffq call is reasonable. A few comments:

Can you rewrite your changes against devel? We are working to address bugs with NCBI's entrez in that branch (e.g. https://github.com/pachterlab/ffq/issues/23)
Can you run make check and make test to check for code quality and to verify tests? (We understand if not all pass due to the limitations described in the README.)

We can then merge these changes and put out a release (and publish to pypi.)
Thanks again!

Fantastic! Yes, happy to make those changes - I must admit I was kicking myself when I opened the PR and realised that there was a devel branch, should have checked that before touching anything.. 🤦🏻♂
I had issues running the tests locally as nose doesn't work with Python 3.10. I'll set up a python 3.8 env to test though and create an issue about it.
Phil

Ok, there is a merge conflict between devel and master which made switching target branch a little fiddly. As the changes are minor I have created a new PR instead: #31


fixing box size of message and adding visible border

Worked on the Issue: BUG: fixing the size of message box and adding visible border to it #101

@Its-Aman-Yadav

Changes Approved, Thank you so much @divyalaldinani


Range Expressions Are Not Properly Compiled

When the range is empty, execution of the corresponding JS code causes a "The input must be non-negative." exception.
Example: [ for i in 0 .. -1 -> 42 ]
Notes:
This bug only occurs in Client-Server applications, not in HTML applications.
WebSharper version: <IP_ADDRESS>

Duplicate of #412; this issue is fixed in the source and will be available in the next release.


Combination of parens and info icon looks off

Testing #153349
IMHO the parens look OK for something like:

But they don't look so good here, before the info icon:

Verification steps:

Search for the word wrap setting in the Settings editor.
There should be no info icon there, now.


tesseract ocr bounding box

hello
how can i get tesseract ocr bounding box?
regards

unfortunately thats not available.


WIP/Draft: Make sure plugin can be built without SimpleITK

Not 100% sure if we want to merge this, but here's a suggestion for fixing #5
Please let me know if I've misunderstood something, of if you can think of any better solutions.
The problem:

When I do a fresh clone, I can't built the project because SimpleITKCSharpNative.dll (or libSimpleITKCSharpNative.so) is missing (the GodotVolumetricRendering.csproj tries to copy it). I need to build the project before I can download SimpleITK.

Changes:

I've removed SimpleITKCSharpManaged.dll from the project, and changed the project file (.csproj) a bit so that GVR_USE_SIMPLEITK will be conditionally defined (and SimpleITK-related code will be built) only after downloading SimpleITK.
I also created a separate .csproj for the addon, that we include from the outer project. It would be ideal if users of the addon wouldn't have to modify their .csproj file at all to use this addon, but they already need to do that because of the DLLs we depend on - now it's just a matter of adding a <ProjectReference Include="addons/volumetric_importer\VolumetricImporter.csproj" />

The downside of this is that you'll need to manually re-build the project again after downloading SimpleITK (wonder if we can trigger that from code?).

Again, feel free to let me know if I've missed something.. I'm still new to Godot 😅

Sorry, I realised this is a bit over-kill. I can just add a "Condition" to the "<Content Include ..>" in the .csproj, and then it will work fine.
I'll make a new PR later :)
I'll keep this branch alive though. I think maybe it will make things a bit easier if we add a separate .csproj for the addon, so users can just include that, instead of copying all those DLL-references into their own .csproj - or what do you think @riccardolops  ?

I think using GVR_USE_SIMPLEITK condition is a good call so as to keep SITK libraries out of the project, I was looking for a solution to trigger project rebuild just like you did in Unity, unfortunately Godot mono doesn't support this feature yet and a while ago I also opened this issue at godotengine/godot-proposal: https://github.com/godotengine/godot-proposals/issues/8652, there is a clever and hacky solution to this that can be used to trigger compilation from code: https://github.com/lewiji/RebuildCsOnFocus


Unable to add more thank 6 custom props

I Have been trying to add custom weapons to the ammunation using it's ytyp file to add custom props and it seems like i can only add 6 custom props then it will not let me add any more props.

Not an issue related to Sollumz, ask pertinent questions through the appropriate information channels.


CI: vscode extension: first cd then install

quickfix for a CI issue

Codecov Report
Patch and project coverage have no change.

Comparison is base (75f2406) 93.08% compared to head (e8c5976) 93.08%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1824   +/-   ##
=======================================
  Coverage   93.08%   93.08%
=======================================
  Files         353      353
  Lines       25656    25656
=======================================
  Hits        23881    23881
  Misses       1775     1775

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Refactor table retrieval getDoctrineSchemaManager removed in laravel 11

I updated a file to support Laravel 11 by refactoring the table retrieval, as the getDoctrineSchemaManager function was removed in Laravel 11.
I welcome any questions or feedback regarding this change request. This is my first contribution to a GitHub repository, so I'm open to suggestions and guidance.

Thanks for your support @paulpwo


Qa

Coverage decreased (-0.7%) to 85.734% when pulling e3b5ab3a5330a18e4b716bd78515843ffc0581d5 on qa into 6d985a9b0d05026d5725faae66cdad152bd6ac8d on master.


Update documentation preparation

Description:

Closes #83
Closes #196
Update the way the API reference is updated in the documentation.
Update the all_strategies.md file to actually include all strategies.
Remove the now unused invoke tasks related to update the API reference and landing page in the documentation.
Add a line in the README referencing the documentation page including all strategies.
Update the link used for the above reference to be relative when generating the documentation landing page.
Type of change:

[ ] Bug fix.
[ ] New feature.
[ ] Documentation update.

Checklist for the reviewer:

This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand, including clearly named variables?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?

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

Comparison is base (a7bcd3a) 88.67% compared to head (b7a88e4) 88.67%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #209   +/-   ##
=======================================
  Coverage   88.67%   88.67%
=======================================
  Files          15       15
  Lines         459      459
=======================================
  Hits          407      407
  Misses         52       52

Flag
Coverage Δ

linux
88.67% <ø> (ø)

windows
?

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.


chore(pageserver): remove unnecessary logging in tenant task loops

Problem

Close #3830

Summary of changes

Add the #[cfg(feature = "testing")] attribute to unnecessary loggings in pageserver/src/tenant/tasks.rs

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Thanks for working on this! I'll try to get a CI run done for this :)

reset the other PR to 5eebaf9

reset the other PR to 5eebaf9

Um, sorry, what do you mean by reset the other PR?

reset the other PR to 5eebaf9

Um, sorry, what do you mean by reset the other PR? @koivunej Do you mean the other PRs I created? PR #5165 & PR #5196

No, I meant the https://github.com/neondatabase/neon/pull/5193 which I reset to that commit. Just a reminder for myself I don't do it twice :)
I think this is good now, thanks.
Confirmed no more logging, old vs. this pr.


Submission

We are a group of three devs in this capstone @blase147 @bill7pearl @soulemanou-software
Kanban screenshot:

Link to the project https://github.com/users/blase147/projects/9/views/1?layout=board

Hi Team,
Great work on making the changes 👏
You've done well implementing some of the requested changes, but there are still some which aren't been addressed yet.
Highlights

The distribution of tasks are equitable enough 👍
You seem to cover the core elements of the project in general 👍

Optional suggestions

You ough to have an "In Review" comlumn as well
Consider adding the issues to the "Todo" column
I think maybe you should specify what is front-end vs back-end, if a task includes both elements, consider stating clearly in description what is back-end vs front-end
[OPTIONAL] Consider adding labels to your issues, it makes it easier to filter by different types and gives another added level of differentiation among tasks (front-end vs back-end etc.)
[OPTIONAL] Ensure that you are adding realistic weight to your tasks.
while the weights are generally equitable distributed tamong your team, there is one team member that is cutting it close. Remember that you must ensure that the total weight of for project for any individual team member is not significantly different in weight from another, ie, no team member should have a difference in weight of more than or equal to fine (or more than 4) when compared with any other team member.

Take your time in planning, if you plan well, it makes everything else easier.
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the Issue thread if something is not 100% clear.
Please, do not open a new Issue for re-reviews. You should use the same Issue submitted for the first review, either valid or invalid unless it is requested otherwise.

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

Hi team,
Great work on making the changes requested by a previous reviewer 👏
You've done well implementing some of the requested changes, but there are still some which aren't addressed yet.

Required Changes ♻

[ ] At the time of this review, these are the pointings:

@blase147 17.5
@soulemanou-software 18.5
@bill7pearl 13
As the previous reviewer stated, no team member can have more than 4 points difference.  However, @bill7pearl only has 13 points while the others have more than 4 in difference.  (17.5 - 13 = 4.5) (18.5 - 13 = 5.5).
Kindly, correct the pointings so they are more equal (no more than 4 in difference than any team member).
Optional suggestions

[ ] Kindly consider adding descriptions to your tasks.

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

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

Done

Hi team,
Wow, you did it 🎉

Thank you for implementing the changes requested by the previous reviewer 💪 🥇 ㊗
Well done!
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To highlight

Pointings are now more equal✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

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


Add x-overflow to code blocks for long lines

I think the previous version had overflow-scrolling for iOS – is this important here as well, given that this fixes an issue with mobile rendering?

You're right, adding that back too - for context, it allows for "kinetic" touch scrolling on Webkit browsers, rather than the "stuck in the mud" style touch scrolling :)

@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


Worker group ignores connection option

Started to use the connection option and realize that allways localhost was the target instead of another  host.
The tests are runnig but unfortunately I did not managed to create a test for it.
I hope the fix will be merge otherwise give me a short feedback.
Thanks.

Closing due to inactivity.


[documentation] added commonjs/browserify example (#34)

As discussed in #39

Could you squash down these commits? LGTM

@rachel-carvalho
http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html

Done! Sorry for the delay, I got caught up with some projects.

No prob!
@lukasolson this LGTM
We should tag and publish this to npm with the updated docs as it's relevant to browserify users.
Also is this project published to bower?

@michaelBenin Yes, the project is published in bower.

@lukasolson I added you as an owner on npm:
npm owner add lukasolson backbone-super


[WIP] Admission Webhooks (PROJQUAY-1622)

webhook: add k8s admission webhooks (PROJQUAY-1622)
Add a mutating and validating admission webhook to
fulfill the tasks of populating default values on
a QuayRegistry, and validating it and the referenced
'spec.configBundleSecret'.
Signed-off-by: Alec Merdler<EMAIL_ADDRESS>

Closing this thingy, if we need this we will implement in a new PR as this is already pretty much incompatible with what we have.


Initial draft of training material

This merges the training material into the master branch of the starter kit.

I couldn't have written this better myself ;)


Aggregate or Remove Anonymous Givers from Top List(s) to avoid apparent duplication

See screenshot

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1454776-aggregate-or-remove-anonymous-givers-from-top-list-s-to-avoid-apparent-duplication?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

Closing until top lists ever come back.


Is it possible to record a video of several activities?

Could you provide more information on the request here?

Woah, sorry, I think I've accidentally hit "Submit" without filling out all the details for this ticket.
I think I figured out the solution anyway, sorry for pinging you! Thanks for a great library and have a great day :)


Fancy Zones keeps defaulting to a layout I don't use (and can't even delete)

Microsoft PowerToys version
0.69.1
Installation method
WinGet
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce

Create new custom fancy zones
Select the newly created layout as active layout for the screen
Restart computer

✔ Expected Behavior
That the layout I selected before restarting the computer is still the active one
❌ Actual Behavior
The "Priority Grid" one (which I never used) becomes the active layout
Other Software
No response

I have this same issue. It also seems to happen any time there is a power toys update.

Found out why. You have to open the editor, click on the Edit button on the layout you want to use as default and then select Default layout for [horizontal|vertical] monitor orientation.


Clarify relationship of getSyncTime() and timeOffset

I'm trying to understand how the return value of getSyncTime() and the timeOffset returned by reportFunction are related. I would expect that timeOffset + localTime would equal syncTime. I've checked this, however, and sometimes it does, sometimes it doesn't. Having looked at the source code,  I see that they are calculated in quite different ways and getSyncTime() includes stabilisation. Does this make the return of getSyncTime() more accurate? I ask because, for my use case, it would be better for the timeOffset to be set when the status changes so that I have control over when to update the offset to localTime.

Since version 2.0, getSyncTime() does not apply directly a new timeOffset. It is applied gradually, to ensure stabilisation, and monotonicity. Indeed, it is less accurate.
If you want the previous behaviour, you should set options.estimationMonotonicity to false in the constructor.
But I wonder what is your use case. Do you really need timeOffset? Why not using getSyncTime() and getLocalTime(), for proper conversion?

Thanks. The core of what I'm trying to do is to have visual metronomes that are synchronised across multiple devices. The reason I thought it would be better to use timeOffset is so that I can update this value at the start of each beat and then calculate the difference between this and the timeOffset value for the previous beat. This would allow me to spread any necessary adjustment over the duration of a beat. I imagined that if, instead, I was calling getSyncTime() on each animation frame then a change in the sync time could cause a visible jump. Perhaps I have misunderstood, though, and the way in which getSyncTime() updates would prevent this being an issue?

Yes, getSyncTime() should not jump any more, so using it should solve your issue. The stabilisation spreads the adjustment of timeOffset over time. (Except when the synchronisation process restarts, eg. after a long network failure.)

Great. That sounds like just what I need then.


Support BinaryData.ContentType

Once we upgrade azure-sdk-for-net to reference a version of BinaryData that contains the new ContentType property, we will need to update code gen to appropriately set the ContentType on deserialization based on the encoding context.

@JoshLove-msft, could you share the expected deserialization code would look like that leverage ContentType property?


closed connection, reason getaddrinfo ENOTFOUND web.whatsapp.com

type ping web.whatsapp.com on Command Line
If Not Output Maybe Your Port Is Blocked


Add ability to record or capture screen while mirroring

Hi, first off I'm a big user/fan of this tool! I use it pretty much every day while developing for Android.
My typical usage is to mirror the screen of a connected device - which is extra helpful when screen sharing.
I'll often want to capture a screenshot or video as well. It's no problem today to just run a separate command to do this. But, I was thinking it'd be a great to have a keyboard shortcut/menubar command which could be used while mirroring the device to do a screenshot or start a video capture - without having to open a second window.
thanks
joe

Yes, it might be useful, but it's not so trivial.
Ref: https://github.com/Genymobile/scrcpy/issues/1186#issuecomment-592386916

Recording may only start on new keyframe (it may not just be "passive", record where you are). To start recording in the middle of the stream, the client must request to the device to generate a new keyframe (typically, just restart the encoding like on device rotation), then ignore all frames received before that keyframe (+ the config packet).


Images are not responsive when the text size is decreased.

When the text size is decreased the home page images remains in the same size.
Their size does not decrease.
Screenshot

I think the images should be responsive as soon as the text size is decreased the images size should also decrease.

@aashimgarg Can you provide more details about this issue? To be precise, which images are you referring to here. It would be really helpful if you could provide screenshots of the same.

i am working on this issue!

I'm closing this as we are currently beginning a redesign process which will address this issue along with other improvements. Thanks for reporting it!


fix: Ensure neccesary fields are 64-bit aligned

Change Description
Moves the Client.ConnectionsCounter  field to the start of the struct since these values are guaranteed to be 64-bit aligned. Also adds a regression test to ensure that it doesn't get moved out of alignment in the future.
Checklist

[x] Make sure to open an issue as a
bug/issue
before writing your code!  That way we can discuss the change, evaluate
designs, and agree on the general idea.
[x] Ensure the tests and linter pass
[ ] Appropriate documentation is updated (if necessary)

Relevant issues:

Fixes #543

@Carrotman42 PTAL if you have time

I have a small criticism that usually one should put the more interesting (and more required) fields at the top of some struct, and have optional/uninteresting fields more toward the bottom. So putting this field up at the top of the struct seems wrong to me, especially since it seems like this field should never be set by users directly anyway. (see later when I suggest that we unexport it.)
If we are going to have a unit test anyway to verify that it's aligned appropriately, to me it seems better to leave the field be and just add padding until the test passes. I admit that's not ideal from a maintenance perspective, but if you're only ever tacking fields on to the end of the struct it won't require additional changes. Plus it's very easy to fix, just tedious.
Another option that I just thought of: the restriction applies only to 64 bit integers, so we can just change the type of this field to uint32. There's no way this proxy (or the Cloud SQL backend, or any database for that matter) is going to support more than 2^32 connections at the same time in any near future, that sounds just too high to be realistic especially given that each connection requires at least 4-6K of memory to maintain on this client Proxy side.
(and along with a type change, I think we should unexport that field by renaming it to connectionsCounter: it seems like a mistake to have exported it because we don't want outside callers to modify this field at all. If someone needs to read the current value, we can expose a getter for it.)
wydt?

@Carrotman42 So I actually did consider switching it to uint32 instead for the same reasons you mentioned, but ended up dismissing that route because I wanted to avoid a breaking change. FWIW, I agree with you that it probably shouldn't have been exported, but at this point it's been like that for 3 years so unless I'm missing something it's pretty much stuck as is.
RE: Padding vs top of struct, padding the struct feels like a pretty hacky/unintuitive approach. If the only concern is signaling importance, it seems better to leave it at the top of the struct.

I still disagree with the field placement but don't feel strongly enough :)
Lgtm, thanks for handling fixing this!


Can this be used for regular Qt applications built in C++?

If so, how can this be achieved?  I tried using the template file as a qss, but it didn't seem to work.

Hi @GavinNL,
Are you using the material.css.template as it or after rendering?

I'm not quite familiar with how to use the .templates. I've only ever used the .qss file.  I renamed the .template to .qss and tried to import it using the standard method
	//create the application and the main window
	QApplication app(argc, argv);
	MainWindow w;

	//open qss file
	QFile file("material.qss");
	file.open(QFile::ReadOnly);

	QString styleSheet { file.readAll() };

I assume you need a generate a .qss file somehow before it can be used within Qt as a stylesheet?

Try with this file dark_teal.txt (as .qss), if it works, the colors and widgets must change, but with missing icons.

Okay that worked! Thanks
I found the build_stylesheet() function, I managed to get it to build the qss sheet.
#!/usr/bin/python3
import sys
from PySide2 import QtWidgets

from qt_material import build_stylesheet

# create the application and the main window
app = QtWidgets.QApplication(sys.argv)

print( build_stylesheet(theme='light_blue.xml') )

I believe you can embed base64 images right into the stylesheet (https://forum.qt.io/topic/89128/qt-stylesheet-image-can-i-add-svg-code-in-image-instead-of-specifying-url/2 )
I think a really useful feature would be a command line function to generate the qss files with embedded images. That way people can use your themes outside of pyside.
python3 -m qt_material.buildqss "light_blue.xml" > output.qss

I was trying with data:image/svg+xml instead of base64 because I have SVG images, but I can not view them.
The template assumes that there is an icon:/ path, I'm adding this path with QDir.addSearchPath('icon', 'theme')
Can you please try this in C++, with this theme.zip generated theme folder as SearchPath.
Now you could see the theme and icons, or not.

I added this line to my C++ code, and placed the "theme" folder in the same directory as my executable.
I don't think the icons showed up.
QDir::addSearchPath("icon", "./theme");

In the qss file, I noticed
QSpinBox::down-button:disabled {
  image: url(:/icon/disabled/downarrow.svg);
}

So i tired setting the folder structure as:
theme
+ -- icon
      +--disabled
      +--primary

But that didn't work either

I think I managed to get it to work...but I had to do the following
I had to replace all instances of url(:/icon/XXXXXX) with url(icon/XXXXX)
Then i had to have my folder structure looking like this:
executable
icon
+--primary
+--disabled

Then it was able to find the files properly

Okay more success.
In my C++ code I added this:
QDir::addSearchPath("icon", QDir::currentPath() + "/theme");

Then I had to modify all the urls to look something like this:
url(icon:/disabled/downarrow.svg")

Then it started working and I could see the various icons.

At this point, I can add a method to generate the qss and the icons folder for a specific theme and let the users add it to their code.

That would be awesome!

Ok
In the last commit, you can do:
from qt_material import export_theme

extra = {

    # Button colors
    'danger': '#dc3545',
    'warning': '#ffc107',
    'success': '#17a2b8',

    # Font
    'font_family': 'monoespace',
    'font_size': '14px',
    'line_height': '14px',
}

export_theme(theme='dark_teal.xml', qss='dark_teal.qss', output='theme', prefix='icon:/',
             invert_secondary=False, extra=extra,)

or
from qt_material import export_theme

export_theme(theme='dark_teal.xml', qss='dark_teal.qss', output='theme', prefix='icon:/')

With the same documented arguments for apply_stylesheet and new ones:

qss: The .qss filename
output: The folder path (absolute or relative) for the SVG resources
prefix: The prefix used on the QSS file ('icon:/' by default).

So, try it and tell me how works for you, or if there are needed more options.

Looks like its working! Thanks for the quick turn around on this!

@akshaybabloo , did you properly load the qss file from the correct location?
Edit the .qss file and make sure all the paths to the icons look correct, they should look something like this:
  image: url(icon:/disabled/downarrow.svg);

Make sure that the qss file and the theme folder that was generated are in the same folder as your binary file. You can change this, but then you'll have to update the paths in the sample code I gave you.

I get segfaults when trying to produce QSS.
>>> from qt_material import export_theme
>>>
>>> extra = {
...
...     # Button colors
...     'danger': '#dc3545',
...     'warning': '#ffc107',
...     'success': '#17a2b8',
...
...     # Font
...     'font_family': 'monoespace',
...     'font_size': '14px',
...     'line_height': '14px',
... }
>>> export_theme(theme='dark_teal.xml', qss='dark_teal.qss', output='theme', prefix='icon:/',
...              invert_secondary=False, extra=extra,)
Segmentation fault

Needs Python 3.7+ - although setup.py has python_requires set correctly, I was using pip 8.1.1 which doesn't recognize this constraint. Would be good to mention in README.md
On Python for Windows with PyQt5 it crashes due to https://bugreports.qt.io/browse/PYSIDE-156 - Fix is to create a PyQt5.QApplication first

import sys
import PyQt5
from qt_material import export_theme

extra = {
}

app = PyQt5.QtWidgets.QApplication(sys.argv)

export_theme(theme='dark_amber.xml', qss='dark_amber.qss', output='dark_amber_icons', prefix='icon:/', invert_secondary=False, extra=extra,)


Tolerate older usage with mandatory routename and optional path

With the following review:
https://github.com/bbangert/routes/commit/0a417004be7e2d950bdcd629ccf24cf9f56ef817
Routes 2.3 has changed the function signature to mandate
path in addition to routename for the connect method.
In this patch, we try to get back to path being optional by
checking the length of the args list. No other change in
logic.
This hopefully fixes the problem reported here:
https://github.com/bbangert/routes/issues/64

@bbangert We are in the release mode for OpenStack and saw Routes===2.3 cause failures in a couple of our projects (https://bugs.launchpad.net/trove/+bug/1563028, https://bugs.launchpad.net/neutron/+bug/1563157) Can you please consider a 2.3.1 with the fix in this PR if this is correct and appropriate.
Thanks,
Dims

@dims
JFYI @bbangert usually asks for an entry in CHANGELOG

As @webknjaz notes, this needs a CHANGELOG entry. The other issue is that since no test verifies/requires this behavior, we're at the same risk as it breaking again in the future. A test should be included to ensure breakage doesn't pop backup again.

@haypo @webknjaz @bbangert - i've added a test, made the function signature better, updated the changelog. will wait for CI tests to confirm now.

Looks good to me, thanks for the PR @dims and the review @webknjaz!

LGTM

@haypo i am trying to do the absolute minimum changes without changing any logic :)

oh, I'm late with additional comments..
@bbangert if you don't mind I'll PR some refactoring once later :)

@bbangert so we can expect a 2.3.1 soon? :) Thanks

@dims
https://github.com/bbangert/routes/issues/64#issuecomment-203267909

A couple of us over at Rackspace ran into this issue the other day, and solved it by pinning back to 2.2. A 2.3.1 release would be nice so we can stay up to date. Thanks!

@ClifHouck already out! :)

yay! thanks @bbangert


Preview custom themes during development

I want to know if there's a way to preview a custom theme during development, if not then it would be great to provide a way to achieve that.

Hello! Unfortunately, this is not possible because $\LaTeX$ itself doesn't have this feature. You need to run RenderCV again whenever you want to see the updated version.
Additionally, I don't suggest developing a new theme with RenderCV. You could develop the new theme in a $\LaTeX$ editor like Overleaf, and when you're happy with the result, you can move the code to RenderCV. For example, the classic theme is already in Overleaf.


handle groups

created function handle_shapes which handles the shapes returned from a slide.
created function handle_text_frame and handle_table to handle the respective shapes (no other code changes)
handle_shapes sends the shape to the appropriate handler, or back to handle_shapes if it's a group of shapes.

Hi, Thanks for the contribution. Please fix the error in the pipeline. Seems like you have not defined the variable GROUP used to check if a shape is a group or not.

I’ve replaced GROUP with 6.
The GROUP enum is from the PPTX library, not sure how to access that correctly.

@idling-mind could you rerun the pipelines when you get a chance?

@idling-mind thoughts about including this into the library?

hi, Looks good to me!


Implement hermitian conjugate utilities

x.T.conj() -> x.H
x.mT.conj() -> x.mH
Similar to pytorch behavior, very useful for dealing with complex arrays.

Hi - typically for array attributes and properties we've followed NumPy and/or the Python array API standard in order to minimize the cognitive overhead for people switching between standard Python array syntax. NumPy does not have .H and .mH, and this idea has been proposed and rejected on several occasions (see e.g. https://github.com/numpy/numpy/issues/8882 and links within). In the Array API standard, the idea is being considered (https://github.com/data-apis/array-api/issues/555), but is not yet part of the standard, though it appears to be on the draft roadmap for a future revision: https://github.com/data-apis/array-api/issues/643.
That's not to say that we shouldn't add this, but it means there is a higher bar before we consider adding such syntax. Some of the concerns in the above discussions don't really apply to JAX, particularly in JIT-compiled mode where the hermitian transpose operation can be optimized away. If the array API standard adds this, we should ensure that JAX's definition is compliant, otherwise we'll be facing a rather painful deprecation process for adjusting the syntax if the array API standard does add these attributes.
With all that in mind, I think we should hold off on this for now – what do you think?

Thanks for your detailed answer. My point of view on the matter is that this feature is:

very simple to add and maintain,
very practical for users who work on complex arrays regularly (I personally use x.conj().T tens of times every day),
already implemented in pytorch (although they are much less compliant with the numpy API of course).

That being said, your arguments are also very sound. I'll let you guys decide, but I'm very much in favor of the feature :)


feat: add Enkrypt

Add Enkrypt to list of wallets

Thanks for the PR!
We've just re-worked RainbowKit internals in v0.7 so that wallets can be imported individually, ensuring our bundle size doesn't keep increasing as we continue to add more wallets.
Thanks for giving us access to your fork — I've pushed some updates to your PR migrating it to the new API and making some minor changes. Once I've prepared a few more wallet PRs to batch together, I'll get this one merged.

I just tried this integration and noticed the following:

On mobile, I'm presented with the Enkrypt icon but nothing happens when I tap on it.
The mobile download links are for MEW wallet, not Enkrypt, even though you also have a separate PR adding MEW wallet. It seems like Enkrypt is only available as a browser extension, is that right? If so, should this wallet option be hidden on mobile?

I just tried this integration and noticed the following:

On mobile, I'm presented with the Enkrypt icon but nothing happens when I tap on it.
The mobile download links are for MEW wallet, not Enkrypt, even though you also have a separate PR adding MEW wallet. It seems like Enkrypt is only available as a browser extension, is that right? If so, should this wallet option be hidden on mobile?

Hello,
I removed the MEW wallet links to iOS and Playstore, and I added a link to our Enkrypt site on the mobile view.

I just tried this integration and noticed the following:

On mobile, I'm presented with the Enkrypt icon but nothing happens when I tap on it.
The mobile download links are for MEW wallet, not Enkrypt, even though you also have a separate PR adding MEW wallet. It seems like Enkrypt is only available as a browser extension, is that right? If so, should this wallet option be hidden on mobile?

Hello,
I removed the MEW wallet links to iOS and Playstore, and I added URI for mobile.

I just tried this integration and noticed the following:

On mobile, I'm presented with the Enkrypt icon but nothing happens when I tap on it.
The mobile download links are for MEW wallet, not Enkrypt, even though you also have a separate PR adding MEW wallet. It seems like Enkrypt is only available as a browser extension, is that right? If so, should this wallet option be hidden on mobile?

Hello, I removed the MEW wallet links to iOS and Playstore and added URI for mobile.

@markdalgleish Hello, pinging you to see if I can get another review on this along with the MEW wallet pull request. Thank you.


Feat/integration-OpenAI-autolog

Description
openai<=0.28
添加了与OpenAI 0.28及以前版本的autologging集成，需要安装openai==0.28
测试代码为：
import os
import sys
import openai
from swanlab.integration.openai import autolog

autolog(init=dict(experiment_name="openai_logging123"))

chat_request_kwargs = dict(
    model="gpt-3.5-turbo",
    messages=[
        {"role": "system", "content": "You are a helpful assistant."},
        {"role": "user", "content": "Who won the world series in 2020?"},
        {"role": "assistant", "content": "The Los Angeles Dodgers"},
        {"role": "user", "content": "Where was it played?"},
    ],
)
response = openai.ChatCompletion.create(**chat_request_kwargs)

效果：

openai>=1.0
增加了OpenAI API的多版本集成支持，增加了Autologging对于Client类型方法改写的支持

from swanlab.integration.openai import autolog

autolog(init=dict(experiment_name="openai_autologging"))
client = autolog.client

# chat_completion

response = client.chat.completions.create(
    model="gpt-3.5-turbo-0125",
    messages=[
        {"role": "system", "content": "You are a helpful assistant designed to output JSON."},
        {"role": "user", "content": "Who won the world series in 2015?"},
    ],
)

response2 = client.chat.completions.create(
    model="gpt-3.5-turbo-0125",
    messages=[
        {"role": "system", "content": "You are a helpful assistant"},
        {"role": "user", "content": "Please write short story about ice cream?"},
    ],
)

# text_completion

response3 = client.completions.create(model="gpt-3.5-turbo-instruct", prompt="Write a song for jesus.")

Closes: #494, #500

解决差不多了


Incompatibility with lcobucci/jwt:^4.0

The https://github.com/lcobucci/jwt/blob/4.1.x/src/Signer/Ecdsa.php class now has a SignatureConverter constructor parameter which is not sent for EC algorithms when initiating the signer instance in getSignerForAlgorithm
\Lexik\Bundle\JWTAuthenticationBundle\Services\JWSProvider\LcobucciJWSProvider
...
private function getSignerForAlgorithm($signatureAlgorithm)
    {
        $signerMap = [
            'HS256' => Signer\Hmac\Sha256::class,
            'HS384' => Signer\Hmac\Sha384::class,
            'HS512' => Signer\Hmac\Sha512::class,
            'RS256' => Signer\Rsa\Sha256::class,
            'RS384' => Signer\Rsa\Sha384::class,
            'RS512' => Signer\Rsa\Sha512::class,
            'EC256' => Signer\Ecdsa\Sha256::class,
            'EC384' => Signer\Ecdsa\Sha384::class,
            'EC512' => Signer\Ecdsa\Sha512::class,
        ];

        if (!isset($signerMap[$signatureAlgorithm])) {
            throw new \InvalidArgumentException(
                sprintf('The algorithm "%s" is not supported by %s', $signatureAlgorithm, __CLASS__)
            );
        }

        $signerClass = $signerMap[$signatureAlgorithm];

        return new $signerClass();
    }

This will cause error:
In Ecdsa.php line 15:

Too few arguments to function Lcobucci\JWT\Signer\Ecdsa::__construct(), 0 passed in /vendor/lexik/jwt-authentication-bundle/Services/JWSProvider/LcobucciJWSProvider.php on line 217 and exactly 1 expected

Possible solution:
return new $signerClass(new Signer\Ecdsa\MultibyteStringConverter()); for signers that are instance of \Lcobucci\JWT\Signer\Ecdsa

Thank you for the report, I'm going to release a fix for this in the next couple of days.

Thank you for the report, I'm going to release a fix for this in the next couple of days.

Fixed in v2.10.6

Fixed in v2.10.6


Inconsistency in Configuration Schema for CosmosDb Component compared to other azure components

Configuration Schema for Blob follows the convention Aspire : Azure: Storage : Blobs, which is consistent with other Azure components like Key Vault (Aspire:Azure:Security:KeyVault) and Service Bus (Aspire:Azure:Messaging:ServiceBus).
Schema for Cosmos DB (and SQL) seems to follow a different pattern: Aspire : Microsoft : Azure : Cosmos.

The configuration section follows the package name of the component. The package name of the component follows the underlying client library name. Since the CosmosDB library name is https://www.nuget.org/packages/Microsoft.Azure.Cosmos, the Aspire component is named Aspire.Microsoft.Azure.Cosmos. Then the config section follows that name.

Right - Azure Cosmos DB doesn't have a Track 2 Azure SDK that follows the typical conventions, so a bunch of our naming predates the consistency of those packages.  I think this is by design.

@Pilchie - do you imagine a "Track 2 Azure SDK" for Cosmos DB that follows the naming?
If so, this is why I want the Aspire components to follow the underlying client library naming. That way we don't take Microsoft.Azure.Cosmos and call the Aspire component Aspire.Azure.Cosmos and block us from ever making an Aspire component for a Azure.Cosmos library if it ever came.

Yes, a track 2 SDK is on our roadmap, though still a ways out.  I agree that we should artificially break the pattern and use Aspire.Azure.Cosmos now (though I expect our Track 2 SDK would actually be Azure.Data.Cosmos, not just Azure.Cosmos.

Thanks, @Pilchie.
Closing as this is by design.


Ability to record category options according to the combination of categories of the data set not fully OK

When accessing with the App to a data set for which there is a combination of categories of the data set, it is still not possible to select the category option of the combination of categories of the data set.

Which version of DHIS 2 instance are you running? Data set categories are supported for 2.24+.

We are using 2.23 but field is available in data set (and we are actually using it):

id="categoryComboId"


F12 preview window gets in way of actual Go To Definition

Issue Type: Bug
When I hit F12, I want to go to the definition of what the cursor is on.  However, VSCode always pops open some preview window which is not useful and just an unnecessary road block in development.  I'm also not seeing a way to turn it off; if there is one, it's not obvious (I would argue it should be off by default).
VS Code version: Code 1.66.0 (e18005f0f1b33c29e81d732535d8c0e47cafb0b5, 2022-03-30T05:50:14.623Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Xeon(R) W-2255 CPU @ 3.70GHz (20 x 3696)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
63.69GB (41.54GB free)

Process Argv
--crash-reporter-id f53a4338-faa4-4c22-8800-ef494cfc4959

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

prettier-vscode
esb
9.5.0

vs-keybindings
ms-
0.2.1

vscode-typescript-tslint-plugin
ms-
1.3.4

debugger-for-chrome
msj
4.13.0

vscode-react-native
msj
1.9.2

vscode-jest
Ort
4.2.1

save-and-run
wk-
0.0.22

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
pynewfile477:30463512

/needsMoreInfo
You should get peek preview instead of navigation when there are multiple locations where you can be navigated. For single locations you should be navigated right away.
Is it a behavior that you are observing?
Also you might want to tune "family" of editor.gotoLocation settings that control this behavior.


IOS support Please

Is better then use 2 plugins...

Opened https://github.com/homerours/cordova-music-controls-plugin/pull/41 for review

Thank you very much for the contribution @0505gonzalez !
I just merged it. Sadly I'm not able to try it... I can give you push access if you want.

@homerours No problem. Re: push access, sure thing. I can also help manage issues if you want.

I just sent an invitation. Yes of course, you are very welcome ;)

@homerours Could you please release the updated plugin so that it can be installed via the cordova cli?

Ionic IOS error - Object literal may only specify known properties, and 'album' does not exist in type 'MusicControlsOptions'.

@derasmus are you using ionic Typescript?

@0505gonzalez yes, i am

@0505gonzalez How can i fix the issue, running Ionic 2 & all the IOS propperies are not working?

@derasmus MusicControlsOption is a Typescript class defined in the ionic native module. To fix the issue, we would need to update the fields in the ionic native repo and create a PR. I was planning on doing that but have not had the time. https://github.com/driftyco/ionic-native

https://github.com/driftyco/ionic-native/blob/master/src/plugins/music-controls.ts#L4

@0505gonzalez thanks, will keep an eye out for the update

@0505gonzalez I am very sorry for the delay, I do it right now!

@homerours @0505gonzalez thank you, when can i update my project with the new fix?

@0505gonzalez - i'm still getting the following error:
Object literal may only specify known properties, and 'album' does not exist in type 'MusicControlsOptions'.


AVG flags install as "generic virus"

AVG flags the installation as a "generic virus". VirusTotal does not pick up anything so it is probably a false alarm but I would like to know why the install appears to be a virus.

 Sorry. If you wish to check the files use some paid antivirus trial. Like Kaspersky.


allow for custom dlls to be in a different folder from installation

Hi,
Why would you like to allow for custom dlls to be in a different folder from installation?
Regards,

i have some custom workflows (using your nuget package) specific to a project. when i deploy i dont want to have to copy the updates to the installation directory. i would like it to stay inside the deployed project

Hi,
I added this feature in the version 3.7.
Here is how it works:
You can put your custom dlls in the folder C:\Wexflow\Tasks. The path of the folder C:\Wexflow\Tasks can be configured through tasksFolder setting in the configuration file C:\Wexflow\Wexflow.xml.
Kind regards,

Can I close this issue?

yes thanks. all working


Consistent logging

In Stargate many modules import logback-core, which is probably not ideal. It make log operation obscure in runtime since logging is not controlled in one place.
Most modules should probably import only slf4j. Then, we could have a separate OSGi bundle that would contain an isolated set of jars implementing slf4 and control logging in runtime.
Downstream project will have the option of providing their own log implementation bundles or simply overwrite the default logback.xml config.

Refactoring for Stargate V2 will require thinking through logging aspects (also due to move to Quarkus).
Closing as being tied to SGv1 design; will create new issue(s) as necessary.


Update template path

@GrantMeStrength : PR passes checklist criteria. Approve and merge as appropriate. Thanks!

Thanks for the correction!


solve_order()'s after argument, when list is used, ends up in before_l list

Hi,
I think there is an error in solve_order() when after argument is passed as a list, it is filling before_l instead of after_l
https://github.com/fvutils/pyvsc/blob/7accbb32aa4bc165859edfeae4f4740e0fbe69d9/src/vsc/constraints.py#L415

I didn't realize you could pass lists to solve_order()! Easy fix, but might be good to add an example to the docs and a testcase with it.
Example:
@vsc.randobj
class my_s(object):

    def __init__(self):
        self.a = vsc.rand_bit_t(1)
        self.b = vsc.rand_bit_t(16)

    @vsc.constraint
    def ab_c(self):
        vsc.solve_order(self.a, [self.b])
        with vsc.implies(self.a == 0): self.b < 5

obj = my_s()
obj.randomize(debug=True)

Debug snippet:
    constraint ab_c {
        solve a,b before
        (a == 0) -> {
            (b < 5);
        }
    }