Legal microcopy to warn users not to  submit personally idendifable info (PII) in page-feedback forms

This came up with reviewing page feedback with ODI legal counsel.
Steps to complete are:

[ ] Content team writes copy
[ ] Legal team approves copy
[ ] Design team updates Figma
[ ] Engineering team implements
[ ] QA
[ ] Deploy to all sites using the component

Assigning to @yesenia-gtz for labeling. This is high priority.

@zakiya Once we've identified the SME/ contributor, can we also identify where this copy will be housed? I have access to this folder, but want to be sure we're all using it.

Hi @ashleydraper & @yesenia-gtz, I'm the PF owner on the ODI / CalInnovate side, so I can help to clarify/align with @zakiya and @mediajunkie on slack #odi-website-analytics. I will take note of the Design System folder.

On the UI end - I synced already on this with @aaronhans  & the way we discussed for making this happen would be:

add a data attribute on the wrapper of the custom element that can include the text for the finalized microcopy
Connect that data in the UI of the component to display on the front end
update the web component

Keeping the markup as an attribute will allow any legal team to update text to suit their needs.

@chachasikes I'm glad you two were able to discuss this, but I'm lost in what you're describing! Can you explain that in a way that's less technical?

@reganking Sorry it wasn't communicated that you were the  product owner for page feedback. We'll need to coordinate closely as page-feedback is also a design system component.
@ashleydraper I have no preference about where copy should be housed. Liz is our only content designer at the moment.
@chachasikes comment refers to the implementation plan for engineering.

@zakiya How do folks feel about this statement. Personally identifiable information can read a little jargony outside of the state, so I went with terms that resonate with more people.

Do not include any personal or confidential information like your address or phone number.

@ashleydraper Looks good to me. Will run it by legal.

@ashleydraper
The language is ok with legal.
They also suggested we instruct people who use the widget to scrub any PII.
I think we can move forward with updating Figma but we should probably also add some copy to the readme.

@zakiya
That's great.
Can you say more about what they mean by "scrub?" If we're telling folks not to include PII in the first place, what are they scrubbing? Also, just some quick analytics on that word choice (see screenshot).

I don't have the Figma board,  is the readme section be on there too?

Poking my nose in here because I have some of the context from the past, I
think they’re suggesting that we recommend to users of the report that they
remove PII from comments when they see it, which I don’t believe is a
feature that we offer.
Also there are times when the PII is knowingly submitted in hopes of a
response.
Giving the reviewers discretion and making them aware of the risks involved
with PII was the plan of record as I recall.
--
Christian Crumlish (he/they)
Senior Product Manager | CA Office of Data & Innovation
916-272-5214 | @.***
follow us: web https://innovation.ca.gov/ | blog
https://innovation.ca.gov/blog/ | twitter
https://twitter.com/californiaodi

OH, now I see. Okay this is making sense!!
See notes in the screenshot.

Those are both great notes @ashleydraper.
Correct, it's a service we provide right now, and an interested dept/agency would need to work with us.
A feature we would need to add is the ability to flag and remove PII data from the reports that the departments would get.

@zakiya @mediajunkie Thoughts?

To round out the initial ask, seems like this should be assigned to design to add to Figma. Is that Kimberly?

@ashleydraper Let's put a pin in this until I can connect with @reganking and @mediajunkie  Product ownership on this has been unclear and we've had personnel changes and I want to make sure all parties are moving in the same direction. Will have an update for you by end of day.

Works for me, ping me when ready.

Just a quick update on this -- we realized we have some work to do internally about the direction of this product. We have a meeting to level set next week. I'm going to remove the Urgent tag and and move it to in Progress.
I really appreciate the quick turn around @ashleydraper!

Very welcome! Happy to jump back in when the time is right.

@mediajunkie @ashleydraper
We met about this product on Monday. Per that conversation, I think we should make the following changes to the Page Feedback readme:

I can make the update myself if approved.
I think we can work the addition of microcopy into the website we're working on in another repo.

@zakiya I support these edits. Should this issue remain open until we're able to address the microcopy?

I made the edits. Yes let's keep this open until we address microcopy. Going to assign to myself for now. Thanks for your help!


Make the Mirror Queue a queue

For some reason the mirror syncing queue never got converted to the more modern queue format. This PR converts this to a modern queue.
There is also a bug in the from the repo-archive queue PR - the assumption was made that uniqueness could be enforced with by checking equality in a map in channel unique queues - however this only works for primitive types - which was the initial intention but is an imperfect. This is fixed by marshalling the data and placing the martialled data in the unique map instead.
The documentation is also updated to add information about the deprecated configuration values.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@8edda8b). Click here to learn what that means.
The diff coverage is 22.05%.

:exclamation: Current head 6109ee0 differs from pull request most recent head 8d9c3b1. Consider uploading reports for the commit 8d9c3b1 to get more accurate results

@@           Coverage Diff           @@
##             main   #17326   +/-   ##
=======================================
  Coverage        ?   45.16%
=======================================
  Files           ?      784
  Lines           ?    88227
  Branches        ?        0
=======================================
  Hits            ?    39847
  Misses          ?    41902
  Partials        ?     6478

Impacted Files
Coverage Δ

modules/queue/unique_queue_channel.go
2.17% <0.00%> (ø)

modules/setting/mailer.go
52.63% <ø> (ø)

services/mirror/mirror.go
9.45% <5.55%> (ø)

modules/setting/queue.go
67.77% <68.42%> (ø)

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 8edda8b...8d9c3b1. Read the comment docs.


Refactor code

Although the receipt-parser works, it's really not too well structured. Might be better for testability (see #4) to clean up the parsers and make the code a bit more object-oriented overall.
Would be happy for anyone who wants to tackle this. Providing mentorship if needed.

Hello, I want to refactor this code. I have already forked and cloned this repository. Since it is my first proper contribution, I shall ask if I have any questions. I shall be following this guide https://docs.python-guide.org/writing/structure/

👋 Hey @Sando1,
thanks for contributing. Looking forward to what you come up with.
The guide looks reasonable to me. Please go ahead with that.
Cheers,
Matthias 😊

This was done by @Sando1 as part of https://github.com/mre/receipt-parser/pull/14. Thanks for this! 🎉


PHP Conf Issues with multiple installed versions

Hello,
Sadly PHP manages this the hard way when you install PHP on windows, it comes precompiled with the flag option " --with-config-file-path" set to windows directory and the flag option "--with-config-file-scan-dir" set to nothing but rather managed by the Environment Variable "PHP_INI_SCAN_DIR", you can check with:
php --ini

Now the "PHP_INI_SCAN_DIR" env won't help at all and will result in many problems when running multiple PHP versions as we can't specify which version, PHP has a solution to multiple versions with setting paths to .ini file in the registry, sadly that just work for .ini but not additional dir scan.
I attempted to recompile PHP and set the flags like the following:
--with-config-file-path=C:\Users<username>\scoop\persist\php71\cli
--with-config-file-scan-dir=C:\Users<username>\scoop\persist\php71\cli\conf.d
This would be the cleanest way to do it in my opinion and that's how Sury compiles for Debian
https://packages.sury.org/php/

But again the biggest problem with that, that I can't assume the username or pass any variable when compiling, or else i thought of configuring a pipeline to test, compile and build on every newly released version of PHP.
So honestly, I'm left with no clean way to scan multiple dir for multiple versions on windows.
Do you have any thoughts or idea on that, If we can solve it, I will be willing to take the compilation and deployment part.
Similar opened issues:
#1185
#3262
#1401

A solution to this might be adding the private environment variables feature: https://github.com/lukesampson/scoop/pull/2917

Does scoop support multiple insances of the same app in principle? I'm not talking about things like python2 / python3, I'm talking about cases like php, which have a single manifest. My impression was that scoop can only give you the latest version and that's about it.


tokenizing bug

serious bug when writing text.

To replicate add a SEG_TRIGGES char such as comma:

SEG_TRIGGERS = ['་', '།', '\n', ',']

Further bugs:

applying level formatting erases last recorded step gh
cursor jumps to next line when writing on line 3, efd

the cursor jumps to the wrong places when pressing enter several times. This happens when editing in the middle of a paragraph, after the first char, the cursor jumps to the end of the text.

after adding a comma at the end of each line I typed 'b' and the cursor jumps to the end of the paragraph. The second line should be a,bbbb,c,

I just pressed enter several times:


feat: Port/Data.Finsupp.ToDfinsupp

port of data.finsup.to_dfinsupp
includes naming of instances in Data.Finsupp.Defs
Had to unroll the instance defs for sigmaFinsuppLequivDfinsupp (duplicating defs) to address a timeout issue.

bors d+

bors r+


wrong when query greenplum tables

What happens?
I attached a greenplum db as gp,
and I can query data by postgre_query(),
but can't query by gp.table
and then how to write data into greenplum table
To Reproduce
select * from postgres_query('uat','select * from tenant_snowbeer_dm.dim_sku limit 2')--ok
select * from uat.tenant_snowbeer_dm.dim_sku--wrong：ava.sql.SQLException: Catalog Error: Table with name dim_sku does not exist!

OS:
mac14
PostgreSQL Version:
6.12.1(gp,pg:9.4)
DuckDB Version:
0.9.2
DuckDB Client:
python
Full Name:
wonb168
Affiliation:
Linezone
Have you tried this on the latest main branch?

[X] I agree

Have you tried the steps to reproduce? Do they include all relevant data and configuration? Does the issue you report still appear there?

[X] I agree

Thanks for reporting - could you check if this issue is still present on the latest version of the postgres extension?


Questions about training overfit on human6m dataset

Hi,I used human36m_vol_softmax.yaml to restart training on the human6m dataset (S6, S7, S8). Without using the pre-trained train.pkl, overfitting occurred after 17 epochs. Could you tell me whether the case occur when you train it?

Hi, @sunny0212452!
We observed overfitting only after a week of training (>100 epochs).


Problem with columns being merged in output CSV file

I've been trying to use Tabula to extract the table data from a 53 page PDF. It does a pretty amazing job, but has trouble distinguishing a few of the columns as separate columns. The table has the same format on each page, but the merged columns in the resulting CSV file differ from page to page.
I used the Stream method. When I tried the Lattice method, it returned an empty result (I don't have vertical lines).
Several of the columns have text across the entire column (and these are the columns that appear to be causing the problem), for those, the only indication that there is a new column is in the header line (which is the same on each page).
I'm attaching the PDF below:
Textbooks.pdf
Thanks for the great tool, and for any help you can offer.

Hi @rjp0i:
Thank you for the bug report. Unfortunately, this PDF is pretty pathological. Your diagnosis of the problem is essentially correct: the PDF doesn't have vertical ruling lines to separate columns, and the text in some cells is sometimes so long that it nearly overlaps the next cell. It's this overlap that is confusing the stream algorithm and the lack of vertical ruling lines precludes any possibility of using the lattice algorithm.
Unfortunately, I don't have much of a solution for your PDF. The only possibility I can offer is that you could manually specify the locations of the column boundaries via the command-line tool's --columns option. https://github.com/tabulapdf/tabula-java/


Sway crashes when hotplugging with Assertion layout_output failed

Sway Version: sway version 1.4-82d02ce7 (Mar 28 2020, branch 'master')

Debug Log: https://0x0.st/iB08.txt
Created with WAYLAND_DEBUG=1 sway -d -c /etc/sway/config 2> ~/log.file

Configuration File: default (/etc/sway/config)

Reproducing this requires two monitors. It's worth noting that the second monitor is HDMI and the first monitor is DP

Start sway with only one monitor on (second monitor is plugged in but turned off)
Turn on the second monitor
Sway crashes almost instantly, after first trying to set the second monitor output

Stack Trace, if sway crashes: https://0x0.st/iB0K.txt

Note, we should add checks to all wlr_output_from_resource call sites.


Overflow component

Related PR: https://github.com/patternfly/patternfly-next/pull/2126

closed via https://github.com/patternfly/patternfly-next/pull/2126


Set Travis to use LTS Trusty image

Coverage decreased (-0.07%) to 88.537% when pulling 1417bdae26e11902bfe3752982636071f6d7320a on mprimi:travis-lts-image into 1cd1fba8a0f23dd9d5f578e136157d85c177e1e4 on Netflix:master.


Fix toggle Folder mode control bar button not having a key

Description
Resolves the following warning in console:
Warning: Each child in a list should have a unique "key" prop

Type of change

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

How to test
Check that the warning no longer appears in the console.

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

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

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4726

Relevant Lines:
12570

💛  - Coveralls


Github-deploy stopped working with webpack 2.x

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?

bug

What is the current behavior?

npm run github-deploy

throws an error:
Unknown argument: github-dev

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).

npm run github-deploy

What is the expected behavior?

Expected to deploy the app to github

Please tell us about your environment:

Webpack 2.x

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

https://github.com/webpack/webpack/issues/2254

Any updates on this? This is now an issue beyond env parameters given the recent changes to the webpack configuration syntax (any references to metadata, output, etc. return undefined now).


/middleware/names/auctions/active not listing names with > 10 chars

currently I am testing our java sdk functionalities and query the middleware using the official docker-image (latest) which @noandrea built for me.
I experienced the issue that names with >10 chars are not appearing in the list of active auctions.
I expect all names to appear in this list when the character-length is <=12

and names which have an active auction running also immediately appear when calling /middleware/names.

is this the expected behaviour? I would expect them to appear there when the auction has ended

mhhh probably this isn't only affecting names > 10 characters. now I also got the problem with auction99.chain which has 9 chars.

This was a bug in the auction length code which is now fixed.


Speed-up opening the authentication.authtoken admin by indexing `crea…

…ted_at` column
The reason is the default ORDER BY created_at , which makes a sequence scan on the whole table.
On the local database the difference is significant with and without that index: before: Execution Time: 175.196 ms
after: Execution Time: 0.562 ms

Task linked: QF-5021 Speed-up opening the authentication.authtoken admin by indexing created_at column


DOC: How to define LaTeX macros

Support in JupyterLab: https://github.com/jupyterlab/jupyterlab/pull/5997
Support in the Classic Notebook: https://github.com/jupyter/notebook/pull/4407


feat(frontend): Ask balance with 1 confirmations

Motivation
We want to show the latest balance.
In this PR, we change the requested balance to use only one confirmation.
Changes

Create a new constant BTC_BALANCE_MIN_CONFIRMATIONS.
Use the new constat in the btc wallet scheduler when getting the balance.

Tests
I deplyed the branch to test 2 and confirmed that the balance got updated when the transaction had 1 confirmation.

@peterpeterparker @AntonioVentilii-DFINITY please review


Make multi platform

Ensure this works on our light hotspot hardware but also on all other hardware - Nebra RasPi, Nebra ROCK Pi, RAK, Sensecap, pisces etc

@shawaj won't it be helpful to take new branches of all our code and add/remove functionality. Trying to make the same code base for full and light miner will require significant work on abstractions and those will be short lived. We plan to move 100% to light miners.
I might be totally off tangent here. You might be talking about testing the new software stack on all hardware variants.

Yes exactly @pritamghanghas we need to test this works on all hardware because in the future full hotspots will use this software


add metadata to route

When I can't specify additional values and send them to custom handlers. Then I add the metadata into the constuctor to specify the value of what will be added to the custom handler.
Suppose I want to specify the locale of route
routes.add({ name: 'home', pattern: '/home', page: 'home', metadata: { locale: 'en' } })
The custom handlers receive the metadata
const handler = routes.getRequestHandler(app, ({ req, res, route, query }) => {
  const { metadata: { locale }, page } = route;
  renderWithCache(req, res, page, locale, query);
});

and the simple case if you want to specify some of route for cache just add the { cache: true } in the metadata
reference issue  #127

Coverage increased (+0.08%) to 96.186% when pulling 8fce1a021ad206455ded83c439234709798389fa on iherem:master into 4630d1961dbf65772fc957bb36a38cbfcb072318 on fridays:master.

Naive question perhaps, but why can’t the locale be part of the pattern?

Actually, I used it with my silly case. I don't know how other people do.
for example:
I have the base route /landing if i go straight to url /uk/landing or /ph/landing the content is changing depend on the route country.
but the metadata can be everything you want to send to the server

but the metadata can be everything you want to send to the server

What part is sending stuff to the server? I'm a little confused 🙂

Also, you should update documentation to reflect these changes, and bump the library minor version according to https://semver.org/.


How do I use this if there is no .exe file?

Title

You probably downloaded the Source. Download the release instead.


[WIP] Add more tests

Description
This PR adds some tests.
It addresses following issues:
#39 : Add gather tests

Hello @hannahbaumann! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file openfe_gromacs/tests/protocols/test_gromacs_md.py:

Line 167:1: E122 continuation line missing indentation or outdented


Replace outdated time 0.1 dependancy with latest version of chrono

I also noticed that times are being miscalculated on linux, so I fixed that.
Fixes #2042 and #2043.

Re #2042 I'm thinking I should add a test.
I can use touch to set a file time to specific value (say Rust 1.0 release date) and then check that du returns the correct value. The only downside of this is windows doesn't ship with touch and coreutils doesn't have it ported to windows currently, so I can't use this to test on windows.


sync initial resources in order when starting registry

During kube controller init, Services and Endpoints can race. If the endpoint is processes before its associated service, the endpoint is dropped here:
https://github.com/istio/istio/blob/42978d0808abb710519fc0e0d9225b2b31ffbec3/pilot/pkg/serviceregistry/kube/controller/endpointcontroller.go#L96
This means any time we start istiod, there's a chance existing endpoints won't be discovered.
See:
https://github.com/istio/istio/issues/24117

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
98e4e0018ae5b5a1a712a88a1daa8bff94b124d3
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
98e4e0018ae5b5a1a712a88a1daa8bff94b124d3
link
/test release-notes_istio

unit-tests_istio
98e4e0018ae5b5a1a712a88a1daa8bff94b124d3
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test all

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
98e4e0018ae5b5a1a712a88a1daa8bff94b124d3
link
/test unit-tests_istio

release-notes_istio
86cd717759ab338ea24e4e5fae3a3a5e725ee489
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
86cd717759ab338ea24e4e5fae3a3a5e725ee489
link
/test release-notes_istio

unit-tests_istio
86cd717759ab338ea24e4e5fae3a3a5e725ee489
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test all

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
86cd717759ab338ea24e4e5fae3a3a5e725ee489
link
/test unit-tests_istio

release-notes_istio
ad97cd9dda40f48e0ea6d5e2b6fba42ea2c7b7a6
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
ad97cd9dda40f48e0ea6d5e2b6fba42ea2c7b7a6
link
/test release-notes_istio

integ-ipv6-k8s-tests_istio
ad97cd9dda40f48e0ea6d5e2b6fba42ea2c7b7a6
link
/test integ-ipv6-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

During kube controller init, Services and Endpoints can race. If the endpoint is processes before its associated service, the endpoint is dropped here:

I think this is one problem but not the only one. The other is that we mark ready when all initial config has entered the queue but we should instead mark ready when all initial config has been processed. Your PR addresses that as well though

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-ipv6-k8s-tests_istio
ad97cd9dda40f48e0ea6d5e2b6fba42ea2c7b7a6
link
/test integ-ipv6-k8s-tests_istio

release-notes_istio
7ec0b5918828b432fcc17acee305e7b4356003a2
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
b68a352bcf39a1399b56a7a09c0108cdb6feeae9
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
b68a352bcf39a1399b56a7a09c0108cdb6feeae9
link
/test release-notes_istio

unit-tests_istio
b68a352bcf39a1399b56a7a09c0108cdb6feeae9
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

cc @liminw

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
b68a352bcf39a1399b56a7a09c0108cdb6feeae9
link
/test unit-tests_istio

release-notes_istio
f4ae6740bc57107193739dfdcbd4736dbc56d36d
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
f4ae6740bc57107193739dfdcbd4736dbc56d36d
link
/test release-notes_istio

unit-tests_istio
f4ae6740bc57107193739dfdcbd4736dbc56d36d
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
f4ae6740bc57107193739dfdcbd4736dbc56d36d
link
/test unit-tests_istio

release-notes_istio
96e72933425f9490a0673136971cf585dfd29eac
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
f4ae6740bc57107193739dfdcbd4736dbc56d36d
link
/test unit-tests_istio

release-notes_istio
5700cada1881a85f968d8a61bd8e78462f66c275
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
f4ae6740bc57107193739dfdcbd4736dbc56d36d
link
/test unit-tests_istio

release-notes_istio
57777e5ac0e4b973a5298f06f2ac935d46802b07
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3b58b1ac405ee71b06b617c2cafd13ca356c18da
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

cc @ramaraochavali @hzxuzhonghu can you take a look?

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3b58b1ac405ee71b06b617c2cafd13ca356c18da
link
/test release-notes_istio

integ-multicluster-k8s-tests_istio
3b58b1ac405ee71b06b617c2cafd13ca356c18da
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest
flake in infra

@stevenctl: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
3b58b1ac405ee71b06b617c2cafd13ca356c18da
link
/test integ-multicluster-k8s-tests_istio

release-notes_istio
3b58b1ac405ee71b06b617c2cafd13ca356c18da
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: #26142 failed to apply on top of branch "release-1.7":

In response to a cherrypick label: new issue created for failed cherrypick: #26392


Post release tasks for 4.8.1

Description
This issue attempts to track the steps necessary to complete post-release maintenance tasks for 4.8.1.
The goal is to complete the following tasks, each task must have a PR associated with this issue.
The branches must follow the naming: merge-<ORIGIN_BRANCH>-into-<DESTINATION_BRANCH>. For example, merge-4.8.1-into-4.9.0.
Tasks

[ ] Merge branch 4.8.1 into branch 4.9.0.
[ ] Merge branch 4.9.0 into branch master.
[ ] Publish GitHub release as latest.
[ ] Delete all stage pre-releases for 4.8.1 (https://github.com/wazuh/wazuh-indexer/releases).
[ ] Delete all stage tags for v4.8.1. BE CAREFUL NOT TO DELETE DEFINITIVE TAG. Example: v4.8.1-rc1, 4.8.1-alpha1, etc.

There's no branch 4.8.0 in this repository.


Update runtime dependency "tslib"

This release changes __exportStar and __importStar to use __createBinding which is incompatible with versions of TypeScript older than 3.9 (i.e. 3.8 and below).
https://github.com/microsoft/tslib/releases/tag/2.0.0

Should tslib be updated now that we have upgraded to TS 3.9?

@bterlson, @xirzec: Can you please review this runtime dependency update?  All CI pipelines are green.  I will run storage-blob live tests as an additional check.

/azp run js - storage-blob - tests

/azp run js - storage-blob - tests

Do not merge until all June package releases are complete


CarouselView weighted method not defined - Uncontained Model Unavailable in Flutter 3.24

Steps to reproduce
Please tell us exactly how to reproduce the problem you are running into.

Create a new Flutter project using the command flutter create bug.
Replace the content of the main.dart file with the following code:

import 'package:flutter/material.dart';

class MyCarousel extends StatefulWidget {
  const MyCarousel({super.key});

  @override
  State<MyCarousel> createState() => _MyCarouselState();
}

class _MyCarouselState extends State<MyCarousel> {
  final images = List.generate(
    5,
    (index) => Hero(
      tag: 'image-$index',
      child: Image.asset('assets/image/image-$index.jpg'),
    ),
  );

  @override
  Widget build(BuildContext context) {
    return ConstrainedBox(
      constraints: BoxConstraints(
        maxWidth: MediaQuery.of(context).size.width - 20,
        maxHeight: 250,
      ),
      child: CarouselView.weighted(
        flexWeights: [8, 4, 2],
        itemSnapping: true,
        children: images,
      ),
    );
  }
}

Run the application on an emulator or physical device.

Expected results
The CarouselView widget should support the weighted method to adjust the weights of items within the carousel, allowing for flexible control over the display of items.
Actual results
When attempting to compile the code, the following error occurs:
The method 'weighted' isn't defined for the type 'CarouselView'.
Try correcting the name to the name of an existing method, or defining a method named 'weighted'.

The CarouselView does not recognize the weighted method, making it impossible to use the Uncontained model.
Code sample
Code sample
import 'package:flutter/material.dart';

class MyCarousel extends StatefulWidget {
  const MyCarousel({super.key});

  @override
  State<MyCarousel> createState() => _MyCarouselState();
}

class _MyCarouselState extends State<MyCarousel> {
  final images = List.generate(
    5,
    (index) => Hero(
      tag: 'image-$index',
      child: Image.asset('assets/image/image-$index.jpg'),
    ),
  );

  @override
  Widget build(BuildContext context) {
    return ConstrainedBox(
      constraints: BoxConstraints(
        maxWidth: MediaQuery.of(context).size.width - 20,
        maxHeight: 250,
      ),
      child: CarouselView.weighted(
        flexWeights: [8, 4, 2],
        itemSnapping: true,
        children: images,
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
C:\Users\####>flutter doctor -v
[√] Flutter (Channel stable, 3.24.0, on Microsoft Windows [versÆo 10.0.22631.3958], locale pt-PT)
    • Flutter version 3.24.0 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80c2e84975 (8 days ago), 2024-07-30 23:06:49 +0700
    • Engine revision b8800d88be
    • Dart version 3.5.0
    • DevTools version 2.37.2

CarouselView.weighted

This is currently available on Master channel only. See https://main-api.flutter.dev/flutter/material/CarouselView-class.html
Closing for now.

CarouselView.weighted

This is currently available on Master channel only. See https://main-api.flutter.dev/flutter/material/CarouselView-class.html
Closing for now.

I don't understand why it works like that. CarouselView was presented like a part of last flutter sdk update (3.24.0). Also CarouselView.weighted method available in documentation here https://main-api.flutter.dev/flutter/material/CarouselView-class.html... for what reason? to be unavailable in Stable channel?


SynchronousMethodHandler no longer wraps Decoder#decode unchecked exc…

…eptions in DecodeException
Removing exception wrapping adds flexibility to the interplay of Decoder and ErrorDecoder:
A Decoder can now delegate to an ErrorDecoder and the original ErrorDecoder exception gets
thrown rather than a Feign-specific DecodeException. An example use-case is a Jackson/Gson
implementation of special 404-handling of Optional methods: when the Feign client has
decode404() enabled, then the Decoder is in charge of dispatching 404 to Optional#absent
where applicable, or to a delegate ErrorDecoder otherwise; consistent exception handling
requires that the exception produced by the ErrorDecoder does not wrapped.

NetflixOSS » feign » feign-pull-requests #177 SUCCESS
This pull request looks good

@adriancole

Hmm I didn't realize that the decode method you removed here already attempted to not double-wrap FeignException.
This solution will break folks. I originally understood that you tried to throw a FeignException, but that it became wrapped in a DecodeException.
That seems not the case. It seems that you want to throw arbitrary exceptions when decode404 is flagged. I could buy that if you kept it contained to the branch where that flag is enabled.
Can you revert deleting the decode method and only call decoder.decode directly when the decode404 flag is set? Also, add a test to make sure it works!
Thx

OK, take another look please, @adriancole

NetflixOSS » feign » feign-pull-requests #178 SUCCESS
This pull request looks good

Not a fan of this change, but I see what you're trying to do and think it is relatively harmless.
I'd like the documentation around this feature moved from Feign into a subpoint on Decoder, as it only applies to those making custom decoders, and only under the circumstance of splitting hairs on 404.

NetflixOSS » feign » feign-pull-requests #179 SUCCESS
This pull request looks good

:clap:
Thanks, Robert

8.12.1 is on the way out now

Cheers Adrian. Responsive as always :)


pipes disappering when trying to pick them up

when we try to pick up pipes or pressurizer with anything pickaxe or wench it just disappears

I am having the exact same issue on my server, is there anyway I can fix it myself?


Add util.py ebml2xml command-line options

The ebml2xml() function has options for  omitting EBML headers, as well as diagnostic attributes (element IDs, sizes, offsets, and data types). These should be exposed as options when using util.py as a command-line utility.

Pretty sure this was done with the --min option, if not we can open a new ticket for what remains to be changed


Add LayerEnv::chainable_insert

Adds LayerEnv::chainable_insert that allows for easy inline creation of LayerEnv values without a builder. #144 introduced a LayerEnvBuilder which will be no longer necessary with this change, keeping documentation about layer environment variable in a central type.

Oh I suppose they are all builders, in which case that is expected.

Yet, your concern is valid. We have some builders that are not really advertising that they are builders. Nothing for this PR, but consistency is something we want.


New Component: grid

MLOps project card:

page["project_2"] = ui.form_card(
    box="projects",
    items=[
         # Missing functionality - Grid layout
        ui.inline(items=[
            ui.text("Experiment Count"),
            ui.text("Registered Model Versions"),
        ]),
        ui.inline(items=[
            ui.text_xl("21"),
            ui.text_xl("12"),
        ]),
        ui.inline(items=[
            ui.text("Deployed Model Version"),
            ui.text("Total Deployments"),
        ]),
        ui.inline(items=[
            ui.text_xl("32"),
            ui.text_xl("43"),
        ]),
    ]
)

There are times when you would want to create rows and columns within a form or other card, calling for a new ui.grid component. I am open to suggestions for how the user adds items. A list of lists might be good?
        ui.grid(
            items=[
                [ui.text("Experiment Count"), ui.text("Registered Model Versions")],
                [ui.text_xl("21"), ui.text_xl("32")],
                [ui.text("Experiment Count"), ui.text("Registered Model Versions")],
                [ui.text("Experiment Count"), ui.text("Registered Model Versions")],
            ]
        )

https://xd.adobe.com/view/eebb600c-026b-4ca1-bb6b-0a439235452b-bbe8/specs/

@mtanco You need to control the width (Experiment count, 21, Deployed Model Version, 32 - all have to have the same width) which is introduced in #795. That way, the alignment depicted in the image can be achieved. This is solution is not scalable (as for code maintainability) though.
@lo5 do you feel like we need to introduce the grid or is the width solution good enough?

Thanks @mturoci ! It looks like #795 is on all form elements so I am happy to close this. To be clear, just width='50%' could be used for each text element?

To be clear, just width='50%' could be used for each text element?

Yes, both px and % values should work.


feat: add footer component

Codecov Report

Merging #14 into master will decrease coverage by 1.33%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
- Coverage   73.68%   72.34%   -1.34%
==========================================
  Files          49       51       +2
  Lines         475      481       +6
  Branches       37       37
==========================================
- Hits          350      348       -2
- Misses        110      117       +7
- Partials       15       16       +1

Impacted Files
Coverage Δ

src/environments/environment.ts
100% <ø> (ø)

src/app/core/footer/footer.component.ts
100% <100%> (ø)

src/app/auth/auth.service.ts
42.55% <0%> (-12.77%)
:arrow_down:

src/app/auth/auth-error/auth-error.component.ts
23.07% <0%> (-4.2%)
:arrow_down:

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 b134364...1b99c44. Read the comment docs.


[Addon] create a clickhouse addon for data house

Is your feature request related to a problem? Please describe.
Clickhouse is a  popular data house, we can build an adon for it based on the operator: https://github.com/Altinity/clickhouse-operator .
Describe the solution you'd like
The addon should contain:

[x] installation of the crd and operator https://github.com/Altinity/clickhouse-operator
[ ] Useful and reliable ComponentDefinition

[ ] Health check of clickhouse cluster
[x] Expose an endpoint for it
[ ] Persisten Volume should be configurable, support external cloud storage as alternative.
[ ] Support scale up (scale down can be optional)

[ ] Documentation

[x] use clickhouse dashboard for demo which means the addon installation is ready
[x] ready for clickhouse-client to connect and run
[ ] Other usage integration such as grafana/promethues connecting for o11y.

Describe alternatives you've considered

Additional context

There's already a clickhouse addon but some of the work still need to be done such as scale up and persistent volume configuration.
After that, we can make it a formal addon.


message: "invalid value in the type field" error

When using Workload Identity, field has an invalid value.  error
code
$options = [
            'projectId' => 'test',
            'keyFilePath' => config('config.workload_identity_path'),
        ];
$pubSub = new PubSubClient($options);

// Get an instance of a previously created topic.
$topic = $pubSub->topic('test-topic');

// Publish a message to the topic.
$topic->publish((new MessageBuilder)->setData('test publish')->build());ï

output
/vendor/google/cloud-core/src/RequestWrapper.php(207): Google\Cloud\Core\RequestWrapper->convertToGoogleException(Object(InvalidArgumentException)) #1
/vendor/google/cloud-core/src/RestTrait.php(99): Google\Cloud\Core\RequestWrapper->send(Object(GuzzleHttp\Psr7\Request), Array) #2
/vendor/google/cloud-pubsub/src/Connection/Rest.php(113): Google\Cloud\PubSub\Connection\Rest->send('topics', 'publish', Array) #3
/vendor/google/cloud-pubsub/src/Topic.php(495): Google\Cloud\PubSub\Connection\Rest->publishMessage(Array) #4
/vendor/google/cloud-pubsub/src/Topic.php(448): Google\Cloud\PubSub\Topic->publishBatch(Array, Array) #5
/app/Http/Controllers/Admin/AnalyticsReportsController.php(211): Google\Cloud\PubSub\Topic->publish(Object(Google\Cloud\PubSub\Message)) #6 [internal function]:

@ma684351 from the provided info, the exact issue isn't very clear.
From what I can tell, RequestWrapper->send gives Google Exception if either the auth params were incorrect (which could be due to config('config.workload_identity_path') not returning a valid path) OR the request was actually rejected by the server. Hope it helps in your debugging.

@vishwarajanand
I think not considered  type: external_account this code
https://github.com/googleapis/google-auth-library-php/blob/main/src/CredentialsLoader.php#L126-L150

@ma684351 If you are using workflow identity federation, then rather than external_account I think it should go under impersonated_service_account, read more about it here.
Is there a reason you are piching for external_account ?

@vishwarajanand
I'm looking to call pub/sub on EC2 without using service_account.
more secure method than keeping credentials on a EC2 and I'd love to use it for my PHP projects.

This would need some changes in our upstream libraries to support.
I would open an issue and close this.


Write error occurs when invalid array declaration

Describe the bug?
Arrays can be declared with length specifications
The compilation succeeds, but a Write error is received
Reproducible code
[Int; 1]

[1; 1]

Expected result
Above is success, below is error
Actual result
Write error: 既存の接続はリモート ホストに強制的に切断されました。 (os error 10054)
error: process didn't exit successfully: target\debug\erg.exe (exit code: 1)
Additional context
No response
Erg version
Erg 0.6.2-nightly.1
Python version
Python3.11.x
OS
Windows 10

[x; n] is a shorthand for [x, x, x, ...].
i.e. [Int; 2] == [Int, Int], [1; 2] == [1, 1]
[1; 2] does not work properly, I fixed.


proposal: encoding/json: allow skipping field by returning an error in MarshalJSON() ([]byte, error)

With the introduction of generics I have started using the following pattern to properly deal with PATCH requests by clients.
type Field[T any] struct {
	value T
	IsSet bool `json:"-"` // Set means the value has been set
}

type MyStruct struct {
	value Field[string] `json:"value"`
}

This allows me to differentiate between zero values and non-set json fields. This field has a unique UnMarshaler and Marshaler
// UnmarshalJSON is a custom function to unmarshal JSON into a Field
func (i *Field[T]) UnmarshalJSON(data []byte) error {
	// If this method was called, the value was set.
	i.IsSet = true

	var temp T
	if err := json.Unmarshal(data, &temp); err != nil {
		return err
	}
	i.value = temp
	return nil
}

// MarshalJSON is a custom function to marshal a field into JSON
func (i Field[T]) MarshalJSON() ([]byte, error) {
	return json.Marshal(i.value)
}

The issue is that if I choose to UnMarshal the MyStruct when the value field has IsSet=false I would end up with the following JSON: {"value":""} rather than {}.
I propose the ability to return a specific error json.ErrSkipField from the MarshalJSON that would cause the field to be skipped. This simplifies handling in these scenarios, and allow the logic to hide/show the struct live together with the implementation.
Writing a custom MarshalJSON function for every struct that happens to use the Field will quickly get tiring and introduce risk of errors.

You can instead use a pointer to a struct with omitempty, then it will not marshal it into the JSON.
type MyStruct struct {
	value *Field[string] `json:"value,omitempty"`
}

I recently stumbled across the same issue, and also thought that having a error to skip a field would be something nice to have.
For now you can use a pointer to a type with omitempty to fix that, like:
type MyStruct struct {
	value *string `json:",omitempty"`
}

encoding/Json docs:

The "omitempty" option specifies that the field should be omitted from the encoding if the field has an empty value, defined as false, 0, a nil pointer, a nil interface value, and any empty array, slice, map, or string.

I think that it would be nice to have something like json.ErrSkipField, but I feel like it should be somehow integrated with the omitempty, so that the  json.ErrSkipField only makes and effect when the omitempty is specified. (Probably then it should be named differently like: json.ErrEmptyField)

FYI, changes to the JSON package should probably be raised at https://github.com/go-json-experiment/json since AFAICT, that repo is being experimentally primed to replace the existing JSON package.

Duplicate of #50480

I recently stumbled across the same issue, and also thought that having a error to skip a field would be something nice to have.
For now you can use a pointer to a type with omitempty to fix that, like:
type MyStruct struct {
	value *string `json:",omitempty"`
}

encoding/Json docs:

The "omitempty" option specifies that the field should be omitted from the encoding if the field has an empty value, defined as false, 0, a nil pointer, a nil interface value, and any empty array, slice, map, or string.

I think that it would be nice to have something like json.ErrSkipField, but I feel like it should be somehow integrated with the omitempty, so that the json.ErrSkipField only makes an effect when the omitempty is specified. (Probably then it should be named differently like: json.ErrEmptyField)

Thanks for the tip, it's just a shame that a simple thing like this has to be achieved by introducing more risks and headache for the developer by introducing pointers. But it's the solution we have to go with for the time being at least.

FYI, changes to the JSON package should probably be raised at https://github.com/go-json-experiment/json since AFAICT, that repo is being experimentally primed to replace the existing JSON package.

Thank you, I weren't aware of that 👍

Duplicate of #50480

That was itself marked as a duplicate as mentioned by rsc:
This proposal is a duplicate of a previously discussed proposal, as noted above,
and there is no significant new information to justify reopening the discussion.
The issue has therefore been [declined as a duplicate](https://golang.org/s/proposal-status#declined-as-duplicate).
— rsc for the proposal review group

But I couldn't find any link to the duplicate he was referring to or reasoning for closing, would you mind sharing if you have that available?

This proposal is a duplicate of a previously discussed proposal, as noted above,
and there is no significant new information to justify reopening the discussion.
The issue has therefore been declined as a duplicate.
— rsc for the proposal review group


Clarify how to access Atrament class

It should be clarified that Atrament will never be exposed automatically. From the docs, I expected it to be in global scope, when including atrament.min.js.
Do not get me wrong though, I think it is good that there is only one global. This is just about clear documentation.

This is actually a bug, Atrament should be exposed when you include the lib in a script tag. Thanks for flagging this up!

You're welcome

Hi @raphinesse - just FYI, in Atrament 1.0.0 the Atrament class is the default export, I got rid of the factory function.


Can't train at all in Colab, said there are tensors on two devices.

I can't seem to run DreamArtist in a fresh install because of this error, and I can't seem to find out where did it went wrong. Got any ideas on how to fix this? Thanks in advance.
Traceback (most recent call last):
  File "/content/stable-diffusion-webui/modules/call_queue.py", line 45, in f
    res = list(func(*args, **kwargs))
  File "/content/stable-diffusion-webui/modules/call_queue.py", line 28, in f
    res = func(*args, **kwargs)
  File "/content/stable-diffusion-webui/extensions/DreamArtist-sd-webui-extension/scripts/dream_artist/ui.py", line 30, in train_embedding
    embedding, filename = dream_artist.cptuning.train_embedding(*args)
  File "/content/stable-diffusion-webui/extensions/DreamArtist-sd-webui-extension/scripts/dream_artist/cptuning.py", line 542, in train_embedding
    loss.backward()
  File "/usr/local/envs/automatic/lib/python3.10/site-packages/torch/_tensor.py", line 396, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
  File "/usr/local/envs/automatic/lib/python3.10/site-packages/torch/autograd/__init__.py", line 173, in backward
    Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument weight in method wrapper__convolution_backward)

More details with the problem: Somehow this error is thrown out whenever I enabled the "Train with reconstruction" option.


SPRING_CLOUD_APPLICATION_GROUP not set

A quick github search for SPRING_CLOUD_APPLICATION_GROUP yields nothing in this repo, which may be a problem.
This is the SO thread that triggered this search: http://stackoverflow.com/questions/42225015/programmatically-getting-the-stream-name

Is this supposed to be in https://github.com/spring-cloud/spring-cloud-deployer-cloudfoundry ?

no, CF and k8s have it (haven't checked YARN). Mesos seems to be missing it

This was part of the 1.1.x versions but Mesos lags behind and is currently released at 1.0.4 and master is at 1.1.0.BUILD-SNAPSHOT, no milestone releases and no work to keep up with the 1.1.x versions of the deployer yet.


How to run this from within ArcGIS Pro?

does this require API key or login to to OpenAi (have both)

Api key
Il giorno lun 1 mag 2023 alle 15:33 thomsonr @.***> ha
scritto:

does this require API key or login to to OpenAi (have both)
—
Reply to this email directly, view it on GitHub
https://github.com/nicogis/PAMChatGPT/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJMVXSU6SNTVKCSYGN6T5LXD63UHANCNFSM6AAAAAAXRY2ZKQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

You should set your API Key in ArcGIS Pro settings named: ChatGPT AddIn Settings

You should set your API Key in ArcGIS Pro settings named: ChatGPT AddIn Settings

Thanks, how do I start/run PAMChatgpt?

Available release 1.0


Pointing K8 cluster to vSphere Storage Cluster

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind feature

What happened:
IHAC who is starting up with Kubernetes running on vSphere. They have a concerns as while configuring K8 cluster user need to specify the datastore manually for every deployment. As the environment grows if later at any point they need to re-point it to a different datastore then how this will happen? Also is there any possibility of pointing K8 cluster to vSphere Storage Cluster instead of specifying datastore which can make the process simpler.
What you expected to happen:
I need help on this query how to address it
How to reproduce it (as minimally and precisely as possible):
This is default behavior of K8 cluster
Anything else we need to know?:
NA
Environment:

Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Hi Everyone,
Has anyone tested this scenario in their environment? Looking out for help from community

@sidharthswami This is a known issue also tracked by #471 and #203. This feature may not be implememted in the near future due to our current tight schedule. @tusharnt


Ensure we have the right modules loaded for weave-npc

Fix for #2726
At least when weave-npc is executed via the weave-kube daemonset.  It would be nicer to separate those two concerns, but we already need to know that weave-npc is in use when creating the bridge.

I have tested this by hand, by modifying the Docker daemon parameters to include --iptables=false and starting up weave-kube.


More explanation on the tip please

I'm trying to have 2 different forms, one for editing, one for creating a new form. I dont' know what this means in the "tip" to set and use "a variable" .... 'Set different values for a variable in the OnNew, OnView, and OnEdit formulas. You can use this variable in the OnSave formula to determine which form is being used." Do i need to create a variable first to point to my different forms?

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

ID: ed2000b4-4917-deb6-148f-98c97691f4fb
Version Independent ID: 241ca44d-5ed2-335a-ae0e-78c3d90b27e2
Content: Understand SharePoint forms integration - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/sharepoint-form-integration.md
Service: powerapps
GitHub Login: @NickWaggoner
Microsoft Alias: niwaggon

Thank you for reaching out. You can create variables with the formulas. However, for additional information regarding any specific scenarios, please reach out to support teams with a support ticket. You can also reach out to the Power Apps community forum for general discussions.


Improve Navbar Links Design to Make it Responsive

Want to work on navbar links to make it responsive and improve UI .
Want to contribute this project under GSSoC'23

I am a GSSoC'23 contributor. I want to improve Navbar Links Design to Make it more Responsive. I want to work on this issue.

From Blog Page we can't go to the Home Page(Main Page). I will fix this.

This navbar is not looking good.
Please assign me this issue

Please assign me this issue to me

Want to work on navbar links to make it responsive and improve UI .
Want to contribute this project under GSSoC'23

Yes go ahead!! @Geeta259

Please assign me this issue , I want to work on this issue under the GSSoC'23.

Want to work on navbar links to make it responsive and improve UI .
Want to contribute this project under GSSoC'23

Yes go ahead!! @Geeta259

Yes ! I am working on it . Please add GSSOC'23 label too.

Open a PR, we will review it and add the label soon.
Thank you!

I made a PR  ! check it @akshathar-navada

Yes sure!


License template update

Based on our discussion @vsoch
The license file may need an update, I have reached out to<EMAIL_ADDRESS>for response, the MIT license from timemory https://raw.githubusercontent.com/NERSC/timemory/develop/LICENSE resembles MIT content but differs in the title.
Copyright (c) 2019, The Regents of the University of California,
through Lawrence Berkeley National Laboratory (subject to receipt of any
required approvals from the U.S. Dept. of Energy).  All rights reserved.

The rest of the text matches what is provided by MIT license see https://opensource.org/licenses/MIT

You can usually put multiple headers. Take a look at the older Singularity licenses when Greg was still at LBNL.

You can usually put multiple headers. Take a look at the older Singularity licenses when Greg was still at LBNL.


Cannot add menu button utility classes for default, primary, secondary, and highlight theme colours

Added required utility classes


Fhir R4

Support for FHIR R4 plus other bug fixes that came up along the way.
Test this against @markkramerus's shr_spec branch: dev-R4map-mk
He now has configs for all supported FHIR versions:

ig-all-dstu2-config.json
ig-all-stu3-config.json
ig-all-r4-config.json

Current a DRAFT pull request because I need to bump the version to non-beta once you approve and before I merge.

Branch setup:
shr-adl-bmm-export       [master]
shr-cli                  [fhir-r4]
shr-es6-export           [fhir-r4]
shr-expand               [master]
shr-fhir-export          [fhir-r4]
shr-grammar              [master]
shr-json-export          [master]
shr-json-javadoc         [master]
shr-json-schema-export   [master]
shr-models               [master]
shr-test-helpers         [master]
shr-text-import          [master]
shr_spec                 [dev-R4map-mk]

Running on DSTU2 config:
Elapsed time: 101.530s
1314 errors (shr-es6-export)
326 warnings (shr-text-input, shr-json-export, shr-fhir-export, shr-json-schema-export)

Running on STU3 config:
Elapsed time: 138.608s
1541 errors (shr-es6-export)
192 warnings (shr-text-input, shr-json-export, shr-fhir-export, shr-json-schema-export)

Running on R4 config:
Elapsed time: 127.799s
1370 errors (shr-es6-export)
170 warnings (shr-text-input, shr-json-export, shr-fhir-export, shr-json-schema-export)

Assuming that there are still known errors in ES6 export, the rest works without error.
Tests pass, with some marked as pending, but no failures.

Thanks @kjmahalingam.  Just to be sure we didn't drastically increase the errors, I also ran the DSTU2 and STU3 configs against the latest stable release 5.17.4.  The number of ES6 errors remained the same in both cases and the number of warnings actually fell (likely due to some fixed bugs).
Running 5.17.4 stable release on DSTU2 config:
Elapsed time: 96.175s
1314 errors (shr-es6-export)
297 warnings (shr-text-input, shr-json-export, shr-fhir-export, shr-json-schema-export)

Running 5.17.4 stable release on STU3 config:
Elapsed time: 110.191s
1541 errors (shr-es6-export)
175 warnings (shr-text-input, shr-json-export, shr-fhir-export, shr-json-schema-export)

So...  knowing that you also manually reviewed the code, I think we're good.

Perfect, then I'm on board with that!

@kjmahalingam -- looking for your feedback here.  There are a lot of commits.  Do you think the commit log is useful or should I squish them?  I think we have 3 possible approaches:

leave as-is
squish into one big commit
rebase into 6 commits (one commit per beta release containing all the changes for that beta)

@cmoesel I'm partial to the approach with rebasing into 6 commits, since I agree that it's a bit large in terms of number of commits, but having some granularity in the commit logs would be useful.


Weather module, openweathermap, can't get more than 5 days forecast

I found a bug in MagicMirror
Platform: raspi 2 b
Node Version: 18.19.0
MagicMirror² Version: 2.26.0
Description: Weather module using openweathermap does not return more than 5 days forecast. changed both maxnumberofdays and max entries and no luck. changed maxnumberofdays to 3 as a test that the variable is getting passed and it works just seems to not get anything over 5.
Steps to Reproduce: configure weather module to use openweathermap, forecast, maxnumberofdays > 5
Expected Results:  more than 5 day forecast output
Actual Results: max of 5 days forecast
Configuration:
{
module: "weather",
position: "top_right",
header: "Weather Forecast",
config: {
weatherProvider: "openweathermap",
type: "forecast",
maxNumberOfDays: 7,
fade: false,
location: "Wylie",
locationID: "4743275", //ID from http://bulk.openweathermap.org/sample/city.list.json.gz; unzip the gz file and find your city
apiKey: "",
},
},
Additional Notes: other modules/configs working as expected

Unfortunately the free API from openweathermap only allows a 5 day forecast. So nothing we can do codewise here.
Of course this would be good to add to the Documentation (if it is not in there). Maybe this is something you want to help us with?

Unfortunately the free API from openweathermap only allows a 5 day forecast (see https://openweathermap.org/api). So nothing we can do codewise here. Of course this would be good to add to the Documentation (if it is not in there). Maybe this is something you want to help us with?

Ok i was afraid that was the answer but makes sense. If i can figure out how to update the docs i will try and do so.
@rejas Curious if you know, i was looking at the api page and it says you get 1000 free calls a day. with the default configs is it making anywhere close to that? meaning if i do subscribe for a paid account i would ideally never be charged at the default rate calls are being made right? Also, i saw mention of moving to 3.0 on openweather docs so not sure if anyone looking at that at all?

@rejas https://github.com/MagicMirrorOrg/MagicMirror-Documentation/pull/228
never done pull request so hopefully everything is set up correctly

thanks, had a comment, so please rework


Fix typo

Cheers!

Thanks for the fix! Things have changed decent bit in this repo with 0.17 and the tutorial moving to http://guide.elm-lang.org/architecture/index.html so this issue was "fixed" an alternate way.


⚠ DesiTorrents has degraded performance

In 3168b22, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 2941 ms

Resolved: DesiTorrents performance has improved in b1da7a4 after 19 minutes.


Argo workflow UI compilation error

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issues exists when I tested with :latest
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?
Cloned the latest argo-workflow repo
Followed every steps mentioned here f
Opened the code is VSCode and ran open in dev container
make start UI=true
I was supposed to see a properly working argo ui but found compilation error

Version
v3.4.8
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
git clone git@github.com:argoproj/argo-workflows.git
set $GOPATH ~/home/go/src/github.com/argoproj/argo-workflows
Use dev container plugin on vscode to open the repo
edit the hosts file and add the following entries
<IP_ADDRESS> dex
<IP_ADDRESS> minio
<IP_ADDRESS> postgres
<IP_ADDRESS> mysql
<IP_ADDRESS> azurite
Run make start and then stop and run make start UI=true
Access the frontend on localhost:8080

Logs from the workflow controller
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container
kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

Findings:
Error: Your lockfile needs to be updated, but yarn was run with --frozen-lockfile so I deleted the yarn.lock file on the local UI folder and ran the make start command to see it with compilation error

@Joibel I think you use Dev Container, right? Any comments here?

I can reproduce this inside a dev container by deleting ui/yarn.lock, but the UI builds OK if you don't do that.
I am unable to build the UI outside of a dev container at the moment, so can't compare to see if it does work correctly outside, and I'm not enough of a UI person to diagnose further.

As far as I can tell just deleting yarn.lock and hoping for the best isn't normally successful, and actual curated updating is the way to go. I'm not sure this is a bug as such.

Agreed with the above; deleting yarn.lock means changing build versions and is not a normal procedure.
The specific build error mentioned here is from upgrading swagger-ui-react, which I investigated in #11640 and is no longer relevant.
So this is not a bug per both of the above.

Error: Your lockfile needs to be updated, but yarn was run with --frozen-lockfile

If you want to intentionally update a dependency, you can temporarily remove the --frozen-lockfile setting from the .yarnrc. I have done this quite frequently recently as I've been updating the UI build (e.g. #11628) and deps


ci(scorecards): add analysis workflow

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
344

Relevant Lines:
1095

💛  - Coveralls


ModuleNotFoundError: No module named 'fastapi'

Why? Why did you have the same issue 2 weeks ago and it is still an issue? :(
How to fix it?

A1111 ; https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast_stable_diffusion_AUTOMATIC1111.ipynb
Dreambooth : https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast-DreamBooth.ipynb
use these links regularly to get the updated notebooks

A1111 ; https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast_stable_diffusion_AUTOMATIC1111.ipynb
Dreambooth : https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast-DreamBooth.ipynb
use these links regularly to get the updated notebooks

Done that. Still broken.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored. ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored. ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored. ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored. ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored. ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored. Traceback (most recent call last):   File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 7, in <module>     from fastapi import FastAPI ModuleNotFoundError: No module named 'fastapi'

A1111 ; https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast_stable_diffusion_AUTOMATIC1111.ipynb
Dreambooth : https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast-DreamBooth.ipynb
use these links regularly to get the updated notebooks

Done that. Still broken.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
Traceback (most recent call last):
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 7, in <module>
    from fastapi import FastAPI
ModuleNotFoundError: No module named 'fastapi'

Fix immediately!

Man! You are using this project for free! The author of this project is constantly improving his product and fixing all the bugs. What right do you have to write "Fix immediately!" ? If something doesn't suits you, write your own version of the colab and do it as fast as you can!

A1111 ; https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast_stable_diffusion_AUTOMATIC1111.ipynb
Dreambooth : https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast-DreamBooth.ipynb
use these links regularly to get the updated notebooks

Done that. Still broken.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
Traceback (most recent call last):
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 7, in <module>
    from fastapi import FastAPI
ModuleNotFoundError: No module named 'fastapi'

Fix immediately!

Man! You are using this project for free! The author of this project is constantly improving his product and fixing all the bugs. What right do you have to write "Fix immediately!" ? If something doesn't suits you, write your own version of the colab and do it as fast as you can!

If I knew how to code my own version of the colab, I would. But I don't.
I don't even know how to fix this issue.

A1111 ; https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast_stable_diffusion_AUTOMATIC1111.ipynb
Dreambooth : https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast-DreamBooth.ipynb
use these links regularly to get the updated notebooks

Done that. Still broken.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
Traceback (most recent call last):
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 7, in <module>
    from fastapi import FastAPI
ModuleNotFoundError: No module named 'fastapi'

Fix immediately!

Is it really that hard to delete all the files from your google drive and reinstall everything?

A1111 ; https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast_stable_diffusion_AUTOMATIC1111.ipynb
Dreambooth : https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast-DreamBooth.ipynb
use these links regularly to get the updated notebooks

Done that. Still broken.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
Traceback (most recent call last):
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 7, in <module>
    from fastapi import FastAPI
ModuleNotFoundError: No module named 'fastapi'

Fix immediately!

Man! You are using this project for free! The author of this project is constantly improving his product and fixing all the bugs. What right do you have to write "Fix immediately!" ? If something doesn't suits you, write your own version of the colab and do it as fast as you can!

If I knew how to code my own version of the colab, I would. But I don't.
I don't even know how to fix this issue.

Everything works, you just need to completely delete all files from Google Drive and reinstall from the latest version of notepad

A1111 ; https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast_stable_diffusion_AUTOMATIC1111.ipynb
Dreambooth : https://colab.research.google.com/github/TheLastBen/fast-stable-diffusion/blob/main/fast-DreamBooth.ipynb
use these links regularly to get the updated notebooks

Done that. Still broken.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
ERROR: ld.so: object 'libtcmalloc.so' from LD_PRELOAD cannot be preloaded (cannot open shared object file): ignored.
Traceback (most recent call last):
  File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py", line 7, in <module>
    from fastapi import FastAPI
ModuleNotFoundError: No module named 'fastapi'

Fix immediately!

Is it really that hard to delete all the files from your google drive and reinstall everything?

It's not hard, it's tedious. I have to move all my models, LoRA, VAE, hypernetworks, textual embeddings, etc., reinstall ControlNet, reinstall all my Extensions

you're not using the latest colab from the link you're likely using an old saved version of the notebook


Composer requirements

The Problem/Issue/Bug
During the workspace spinning up I get this error
Your requirements could not be resolved to an installable set of packages.
  Problem 1
    - Root composer.json requires drupal/devel ^4.1 -> satisfiable by drupal/devel[4.1.0, ..., 4.x-dev].
    - drupal/devel[4.1.0, ..., 4.x-dev] require drupal/core ^8.8 || ^9 -> found drupal/core[8.8.0-alpha1, ..., 8.9.x-dev, 9.0.0-alpha1, ..., 9.5.x-dev] but the package is fixed to 10.0.0-alpha6 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.
composer [require drush/drush:^11.0 drupal/coder:^8.3 drupal/devel:^4.1 drupal/admin_toolbar:^3.1] failed, composer command failed: exit status 2. stderr=

How this PR Solves The Problem
Install the 5.0.0-beta1 version of devel module.
Manual Testing Instructions
@shaal I need help to understand how can I test it because I am newbie on DrupalPod.
Related Issue Link(s)
#97
Release/Deployment notes
Closes #97

Aside from devel 5 not being in beta anymore I think the workaround should be removed to be able to install devel in Drupal 10.1.x and just use devel 5 for Drupal 9 and 10.
Can you you make those changes so I can close my other PR related to this?
Also a better PR title that includes the word devel can help.
Thanks for bringing it up.

@rodrigoaguilera title changed. I'm going to update this PR as soon as I have spare time. Thank you!

:thinking: Now I'm wondering if we should maybe just let composer figure out the version instead of restricting so we don't have the same problem when devel 6 is released. What do you think?


Document config vars in component Gruntfiles

I'm talking about this: https://github.com/cfpb/cf-buttons/blob/gh-pages/Gruntfile.js#L31-L64

We've switched to Gulp some time ago. This issue no longer applies.


Run yarn lint and commit

Codecov Report

Merging #289 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #289   +/-   ##
=======================================
  Coverage   90.37%   90.37%
=======================================
  Files          21       21
  Lines         966      966
  Branches      260      260
=======================================
  Hits          873      873
  Misses         75       75
  Partials       18       18

Impacted Files
Coverage Δ

...ages/emotion/test/extract/extract.test.emotion.css
100% <ø> (ø)
:arrow_up:

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 9c65efd...913a9b6. Read the comment docs.


feat: added custom condition to the aria-label user button

Added a condition to set the aria-label to differentiate the "regular users" from the admin users.
The condition is the same one used inside the 'PersonalTools' component, which opens after the user button is clicked.
Issue with discussion:
https://github.com/plone/volto/issues/5119

@stevepiercy Any idea what's up with the failing ReadTheDocs CI step?

@sneridagh I'd ignore it as it is for the deleted RTD project, and was probably due to our fiddling around with it, based on the timing.

@Wagner3UB Can you change the label to 24 status: ready if there's nothing left to do in this PR?

Should we remove the messages for Personal Tools, since this PR seems to make it unnecessary?

@Wagner3UB I agree with Steve if we no longer use Personal Tools I would remove it.
I would also like to get translations for these 2 new strings at least for the languages that had Personal tools translated.
I don't think it's fine for us to replace their translated strings with untranslated english strings.
We can always also CC other translators to double-check if what Google Translate or other services suggested we use is accurate.


Warn when types exposed to external code (via exported functions) aren't exported

This is a feature request. It would be nice to get warnings when types used in the signature of exported functions are themselves not exported.
In the following code, can we get a warning saying that ExampleType and SomeOtherType must also be exported?
type ExampleType = { n: number }
type SomeOtherType = { r: string }

export function Hello(arg: ExampleType) : SomeOtherType {
  //....
}

With .flow files in v0.19, integrating flow into npm has become very straight forward. By just pushing "somelib.js" (transpiled) and "somelib.js.flow" (original) to npm, consumers of somelib can start using flow. As a result, it is no longer necessary to write separate def files and the above use case might become somewhat common.
I hope it makes sense.

There is no necessity  to export these types. Flow uses structural typing, so one can implement a type without importing it.


What is the current behavior?
In Bazel by default, executable binaries have a built-in args attribute, so bazel run will automatically use those arguments. However, the args are dropped if the binary is invoked directly via bazel-bin/... or in the runfiles of another binary. Quoting from https://bazel.build/reference/be/common-definitions#common-attributes-binaries:

args - List of strings; optional; subject to $(location) and "Make variable" substitution, and Bourne shell tokenization; nonconfigurable

Command line arguments that Bazel will pass to the target when it is executed either by the run command or as a test. These arguments are passed before the ones that are specified on the bazel run or bazel test command line.

NOTE: The arguments are not passed when you run the target outside of Bazel (for example, by manually executing the binary in bazel-bin/).

For some rules (such as js_run_devserver()) executing a binary from runfiles is required and args is effectively useless, since you can't guarantee it will be present. It would be great to have an easy way to ensure hard-coded arguments in the BUILD file are always present, regardless of how the binary is executed.
Describe the feature
I propose a new baked_binary() rule which takes a baked_args attribute and generates a wrapper binary which hard-codes those arguments, with any subsequent arguments appended after it. This means arguments are "baked" into the binary and will be retained regardless of how the binary is executed. This would look like:
load("@aspect_bazel_lib//lib:baked_binary.bzl", "baked_binary")

sh_binary(
    name = "server",
    srcs = ["server.sh"],
    data = [":site"],
    # Can't rely on `args`, because it won't always be there.
    # args = ["path/to/site/"],
)

baked_binary(
    name = "baked_server",
    binary = ":server",
    # Bake the args so they are always present.
    baked_args = ["path/to/site"],
)

# When this runs `./baked_server --port 1234`, `path/to/site` is already included
# and does not need to be repeated.
sh_test(
    name = "test",
    srcs = ["test.sh"],
    data = [":baked_server"],
)

I made my own implementation of this in https://github.com/dgp1130/rules_prerender/commit/c0065dc2598be72120abb9ecc9bea8af20aed1ff#diff-2f9d8f02c056b3df3e9ff8b6fb76b84689e08c39773d3661dcfcd482cf13a267, though it isn't great since it requires Bash, doesn't support Windows, and needed some hacks to be compatible with js_run_devserver() (chdir messes up Bash runfiles).
Fund our work

[ ] Sponsor our open source work by donating a feature bounty

Another examples of prior art is the command rule that is used with multirun: https://github.com/atlassian/bazel-tools/blob/master/multirun/README.md
@alexeagle You had some thoughts on this one if I recall correctly

Yeah I had a general wrapped_binary concept that could do a bunch of little utility things, based on my "genrule is bestrule" principle.
https://github.com/aspect-build/bazel-lib/pull/251
I haven't had time to come back to it though.


[trees] figure regression

this figure in cart-intro.pdf seems wrong: https://docs.google.com/presentation/d/1Pns5UJyc-Aqyk9d-CLekA46BC7UJ_e0uSdErat47opM/edit#slide=id.p
In interval 1 and 3 this can not be the mean. Where is the code that produced these estimates?

cart_dis_2.pdf same

Please compute the correct values and change the slides.

Please compute the correct values and change the slides.

more generally all images in this chapter look awful


Pagination for the wallet transactions in the UI

looks like the api already supports pagination - you can request transactions from a set height to a set height. Users might prefer to see a set number though, which would require some backend refactoring.

This is not pagination as it returns an unknown number of transactions. To
solve the UI issue definitely you to be able to limit what is returned.

Per
On Nov 13, 2015 7:20 AM, "David Vorick"<EMAIL_ADDRESS>wrote:

looks like the api already supports pagination - you can request
transactions from a set height to a set height. Users might prefer to see a
set number though, which would require some backend refactoring.
—
Reply to this email directly or view it on GitHub
https://github.com/NebulousLabs/Sia-UI/issues/130.

what is a sensible limit? 1000?
Even now, the api request itself is not that slow, maybe 1-2 seconds. It's the javascript that is slow. Users might only want to see 25 at a time, but the api should probably be serving a much larger number.

The limit should ideally be configurable. For instance:
/wallet/transactions/limit=10&start=10
will return the last 10 transactions (1 through 10; where 1 is the most recent transaction).
/wallet/transactions/limit=10&start=20
returns the transactions 11 through 20
If you hardcode a limit, how then will I be able to return transactions before that limit?

doesn't it make sense to go the other way?
limit=10&start=0 ==> return transactions 0-10?

I see your point, but transactions are returned in chronological order. So, in your case start actually implies end, which I think is more confusing.
In a different # issue, I also suggested this (inspired by other APIs):
Return the last 100 transactions:
GET /wallet/transactions?size=100&from=100
and the next 100 transactions:
GET /wallet/transactions?size=100&from=200
or, alternatively:
Return the last 100 transactions:
/wallet/transactions?start=100&end=0
and the next 100 transactions:
/wallet/transactions?start=200&end=100

Solved by #140


Spike: Create an order with CPK

Made a script for buying tokens A with tokens B; worked fine. After approving the CPK to use the token (just the token used for buying, B), I sent these transactions in a batch:

transfer tokens B from the user to the CPK
approve the batch exchange to move tokens B,
deposit tokens B in the batch exchange
place order buying tokens A with B, valid until batchId N
request future withdraw of A at batchId N + 1
request future withdraw of B at batchId N + 1

One open problem is that, after the withdraws are done, the tokens will be kept in the CPK. An easy solution here would be to add an infinite approval from the CPK to the user for both tokens. Then anyone can call a transferFrom and move those tokens. Of course, for making this fully "automated", this should be done by gelato somehow.


fixed add magnet bottom sheet hide issue

Fixes #157
Describe the changes you have made in this PR -

Added bottom view insets padding to the add magnet bottom sheet which allows the bottom sheet to move along with the keyboard

Screenshots of the changes (If any) -

@amitamrutiya2210  please resolve the conflict in this PR

I have resolved my conflict, so you can proceed with the review.


Yolo v4

@hpanwar08 hi there,

have you tested yolo v4?
do you think yolov4 is suitable for detecting text?
are you available for freelancing?

Hi @seekingdeep

I haven't tried yolov4 but yolov3. Detectron2 is better on smaller object compared to yolov3. Although yolov3 is much faster
No idea for yolov4, but it would depend on the type of text you want to detect. You can have a look at http://arxiv.org/abs/2006.04305
I am a little tied up as of now

Hi @seekingdeep

I haven't tried yolov4 but yolov3. Detectron2 is better on smaller object compared to yolov3. Although yolov3 is much faster
No idea for yolov4, but it would depend on the type of text you want to detect. You can have a look at http://arxiv.org/abs/2006.04305
I am a little tied up as of now

thanks

thanks


HIPCC support ccache to accelerate compiling

NVCC supports ccache and using
CUDA_NVCC_EXECUTABLE="/usr/local/bin/ccache /usr/local/cuda/bin/nvcc"
Does HIPCC supports using ccache as well, such as setting
set(HIP_HIPCC_EXECUTABLE "/usr/local/bin/ccache ${HIP_HIPCC_EXECUTABLE}")
While seems cmake error thrown after the above change to HIP_HIPCC_EXECUTABLE.

@qili93 Sorry for the lack of response.  At the moment, this is not supported.  Thanks!


Make sure a Library includes the dataRequirements from its dependencies

@ewoutkramer , @richfirely , this PR is 2 months old, shall I approve it?

please, it has been in use by Rich and it did what it is supposed to do.


super() builder inconsistent when attributes are defined inside the init

Hi,
I’m struggling a little bit with this code:
import justpy as jp

class UpperMenu(jp.QDiv):
    def __init__(self,  **kwargs):
        #self.classes = ['grid','grid-cols-7','gap-4'] # this doesn't work
        super().__init__(**kwargs)
       #self.classes = ['grid','grid-cols-7','gap-4'] # this works
        root = self
        country_selector = jp.QSelect(outlined=True, classes='col-span-1', label="Outlined", a = root)
        country_selector2 = jp.QSelect(outlined=True, classes='col-span-2', label="Outlined1", a = root)

def page():
    wp = jp.QuasarPage(tailwind = True)
    menu = UpperMenu(a = wp,
                   #classes = ['grid','grid-cols-7','gap-4' ] #This works
                    )
    return wp

app = jp.app
jp.justpy(page, start_server=False)

somehow the classes argument works when is declared outside the class definition, but when declaring self.classes = [something] it does not render as it is supposed to. I had a similar problem with class MyPage(jp.QuasarPage) and defining self.tailwind =True. I'm following this example.
BTW calling  super().__init__(classes = ['grid','grid-cols-7','gap-4'], **kwargs) also works

Please try moving the classes assignments after the super().init statement in init and let me know if it works.

Please try moving the classes assignments after the super().init statement in init and let me know if it works.

yes! that works too
class UpperMenu(jp.QDiv):
    def __init__(self,  **kwargs):
        super().__init__(**kwargs)
        self.classes  = ['grid','grid-cols-7','gap-4','m-8']

what would be the clearest (and consistent) way to implement this?

yes! that works too
class UpperMenu(jp.QDiv):
    def __init__(self,  **kwargs):
        super().__init__(**kwargs)
        self.classes  = ['grid','grid-cols-7','gap-4','m-8']

what would be the clearest (and consistent) way to implement this?

Just as you did above.

Just as you did above.

Many thanks @elimintz !! I would use that then

Many thanks @elimintz !! I would use that then


Fix main

Main is failing because of missing .trimIndentation().

Edit: I thoght it was just missing trimIndentation() but somehow we introduced a regression with #5212. The second commit fixes that regression.


⚠ Bofrost.de has degraded performance

In 44436d4, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1432 ms

Resolved: Bofrost.de performance has improved in c7d8ae5.


Mathjax rendering issues on github CI

I'm having strange rendering issues with inline equations when building my docs with github ci. Building on github ci gives me results like this:

while rendering on my local pc (M1 mac) gives this:

This was fixed by #169, but maybe you still have to manually update this CSS file


fix: return command nail

Description
pnpm package names cause breakage because of the quotes.
pnpm run "drizzle-kit generate:pg --config=drizzle.config.js"
"drizzle-kit dont found global
now fixed
pnpm run drizzle-kit generate:pg --config=drizzle.config.js

Linked Issues
Additional context

The quotes are added intentionally 3 lines above - Just removing them doesn't justify the use cases, you need to provide more information and find a solutions that doesn't break the previous behaviour