Does the Pysplit is compatible with python 3.8.12 on Win-11?

I am trying to install Pysplit via Anaconda prompt using : conda install pysplit
It cannot gather environment resources to download this.
What could be the probable reasons............

PySPLIT must be installed to your Anaconda environments by using pip install

PySPLIT must be installed to your Anaconda environments by using pip install

Didn't worked this at my system.

Ok but what do you get if you type pip install pysplit?  That image shows the expected result of typing conda install pysplit.

Thanks a lot, that worked and successfully installed at my system.


⚠ Samsung has degraded performance

In 35ca786, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2198 ms

Resolved: Samsung performance has improved in 882b851.


New version: YaoSym v0.5.4

Registering package: YaoSym
Repository: https://github.com/QuantumBFS/YaoSym.jl
Created by: @GiggleLiu
Version: v0.5.4
Commit: f8fe14bb83204ebbf134f0a6d6a0b1abcb1f19d0
Reviewed by: @GiggleLiu
Reference: https://github.com/QuantumBFS/YaoSym.jl/commit/f8fe14bb83204ebbf134f0a6d6a0b1abcb1f19d0#commitcomment-40987410

Please close this, the version number is wrong. Sorry


[Bug]: Phone & currency input widget displays default values on different lines on smaller screens

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Phone & currency input widget displays default values on different lines on smaller screens

Steps To Reproduce

Select any country code with 2 or more digits in it's default code
Go to preview mode and reduce the screen size

Observe the default digit appear on different rows
Public Sample App
No response
Environment
Deploy Preview
Severity
Medium (Frustrating UX)
Issue video log
No response
Version
cloud

Should be fixed.


how do I run this? it doens't work with url stuff...

when I run moss_usage.py, it says   "connection reset by peer".
Or if I run the code usage in the markdown, m.send() returns nothing.
Is there something that I have to configure?

I think it was just about registration issue thanks


[Discourse] Use Daskhub in private EKS Cluster with custom CA certificates

https://dask.discourse.group/t/use-daskhub-in-private-eks-cluster-with-custom-ca-certificates/638

Similar to #909


Support _total suffix for counter sample names, according to the spec

I found out that the counter in the example given in this section of the OpenMetrics spec was not being parsed correctly by the parsing functions under prometheus_client.parser. Instead, it only works if the name of the sample matches the name of the metric family (if I get the terminology right).
This PR provides a simple fix for that. I'm aware that there's another parsing function under prometheus_client.openmetrics.parser which does parse that correctly, but that one is also more strict in other ways, and depending on who's emitting the metrics it might not work.
@csmarchbanks

I wasn't aware of what the actual target for prometheus_client.parser and how it differed from the openmetrics one; I was assuming that the former was simply a less strict version of the latter. I had missed that bit of documentation, which helps and I understand your points.
This fragment and its associated commit message led me to think that this parser was also expected to follow OpenMetrics by dealing with the _total suffix specially:
https://github.com/prometheus/client_python/blob/c48e26e2283fee2c2ef628608e2fed6468f6a911/prometheus_client/parser.py#L153-L162
A "generic" parser that would handle both exposition formats (without requiring strict adherence to the OpenMetrics) spec would be helpful to avoid having to know the actual format precisely ahead of time. Also, in the specific case that triggered this, the same exposition that was using the _total suffix following the OpenMetrics was also missing a # EOF at the end.


Could you provide mongo custom query exporter

Hi,
I really need an exporter which gets data by custom queries. Is it possible to implement in in here?

that is outside the scope of this project
something you are looking for could be built easily using prometheus + mongo client libraries


Gallery paging

@cwestMobile @NikKovIos This should fix the paging discussed here: https://github.com/Yummypets/YPImagePicker/issues/135

Collection view naturally stops at image locations thanks to the magic targetContentOffset(forProposedContentOffset: withScrollingVelocity:
Paging is enabled when scrolling backwards (ala instagram)
Adds touch animation when selecting an image (scale + alpha)
Adds an "edit icon" to indicate something happens when tapping.
Set Gallery light gray background to be consistent with filters Screen

@NikKovIos
I wouldn't say it is "necessary", but I feel like it's really overkill to have a xib file for simple layouts, like for instance the collection view cell with the imageView filling.
I'd argue that debugging a xib is quite hard as well, since you never know what checkboxes are the defaults, in code, every line counts and is there for a reason.
Funny enough Apple itself doesn't use interface Builder internally even though they keep telling the developers to use it :p (From a friend that works there)
I'm not against using bare Autolayout in the simplest layouts. Les dependencies is always better of course.
For instance YPSelectionsGalleryCell
sv(
    imageView,
    editIcon,
    editSquare
)

imageView.fillContainer()
editIcon.size(32).left(12).bottom(12)
editSquare.size(16)
editSquare.CenterY == editIcon.CenterY
editSquare.CenterX == editIcon.CenterX

Could be written natively like so :
addSubview(imageView)
addSubview(editIcon)
addSubview(editSquare)

imageView.translatesAutoresizingMaskIntoConstraints = false
editIcon.translatesAutoresizingMaskIntoConstraints = false
editSquare.translatesAutoresizingMaskIntoConstraints = false

addConstraints([
    imageView.topAnchor.constraint(equalTo: topAnchor),
    imageView.rightAnchor.constraint(equalTo: rightAnchor),
    imageView.bottomAnchor.constraint(equalTo: bottomAnchor),
    imageView.leftAnchor.constraint(equalTo: leftAnchor),

    editIcon.widthAnchor.constraint(equalToConstant: 32),
    editIcon.heightAnchor.constraint(equalToConstant: 32),
    editIcon.leftAnchor.constraint(equalTo: leftAnchor, constant: 12),
    editIcon.bottomAnchor.constraint(equalTo: bottomAnchor, constant: -12),

    editSquare.widthAnchor.constraint(equalToConstant: 16),
    editSquare.heightAnchor.constraint(equalToConstant: 16),
    editSquare.centerYAnchor.constraint(equalTo: editIcon.centerYAnchor),
    editSquare.centerXAnchor.constraint(equalTo: editIcon.centerXAnchor)
])

Even for such a simple layout the native way might not be easier to maintain.
The thing is Stevia was is already used throughout the project, and having both native + Stevia shortcuts can also add complexity. By the way, the views that were left as xibs were legacy views originally coming from the Fusuma project.
I understand there is a bit of a learning curve with Stevia, but I honestly think it is worth the effort.
As a heavy user I am biased of course!
PS: In the end it doesn't really matter as long as it works but we need to aim for consistency :)
Hope this makes sense :)


fix fix a bug that reduce memory work on hard limit might be …

…triggered twice
Proposed changes
Issue Number: close #xxx
Problem summary
When the load mem hard limit reached, all load channel should wait on the lock of LoadChannelMgr, util current reduce mem work finished. In current implementation, there's a bug might cause some threads be woke up before reduce mem work finished:

thread A found that soft limit reached, picked a load channel and waiting for reduce memory work finish.
The memory keep increasing
thread B found that hard limit reached (either the load mem hard limit, or process soft limit), it picked a load channel to reduce memory and set the variable _should_wait_flush to true
thread C found that _should_wait_flush is true, waiting on _wait_flush_cond
thread A finished it's reduce memory work, found that _should_wait_flush is true, set it to false, and notify all threads.
thread C is woke up and pick a load channel to do the reduce memory work, and now thread B's work is not finished.

We can see 2 threads doing reduce memory work when hard limit reached, it's quite confusing.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.88 seconds
load time: 451 seconds
storage size:<PHONE_NUMBER>6 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221104094941_clickbench_pr_39986.html


Partnership Program Subcommittee

Purpose
To decide on a partnership program.  Many of these discussion topics will have been answered or discussed in other issues (like this one).  This issue is meant to aggregate them and start a subcommittee to take them forward.
Goals

To align incentives for the StackStorm project as well as partners

Give and take - two-way significant value
What does ST2 provide to partners?  What does it explicitly not provide?
What do partners provide to ST2?  What do they explicitly not provide?

Deadline
Final draft by next ST2 TSC meeting
Governance
Anyone can join and contribute ideas.
Led initially by 4 partners:

Amanda McGuinness (@amanda11)
JP Bourget (@punkrokk)
Ravi Rao (@raviorch)
Mick McGrath (@mickmcgrath13)

Rules

What is okay?

What is in the scope of access to a partner?

What is not okay?

What is not in the scope of access to a partner?

FAQ
Should we create/host an FAQ for a 'soft landing' of governance / conflict of interest?
Disputes
In general, consensus for involved parties should be sufficient.
If a dispute arises, how are they resolved?  This could be:

Raise to TSC for a vote?
or Dedicated arbitration guidance/framework for the subcommittee

Mechanics

use existing issue or a new one?
new issue for subcommittee?
dedicated slack channel?
Where will the draft be created (github issue, Google Drive doc, etc)?

Thanks Mick, for creating the issue. Will post here the recommendation that I sent out to the other thread.

This one:
https://github.com/StackStorm/discussions/issues/72#issuecomment-814239951
?

We should have verbiage to mitigate risk:

I, as a partner, accept to not hold StackStorm accountable [...]

Exactly we need incorporate this type of language. I will work with our legal counsel to get this sorted out if necessary.

For Ammeon representation I'll be delegating to @elewzey who was also on the call. (I lost my mic half way through...)

We should have verbiage to mitigate legal risk from incoming partners in case of possible "bad actors". Maybe something simple along the lines of:

I, as a partner, accept to not hold StackStorm accountable [...]

Here is what LF legal proposed and so it was drafted on the https://stackstorm.com/partners-program/ (beta):

In your [partnership] proposal, please include the following statement:

On behalf of PARTNER NAME ("Participant"), I agree that Participant will comply with the StackStorm Partners Program requirements published at https://stackstorm.com/partners-program/ and Linux Foundation Policies published at https://lfprojects.org/policies/, as updated from time to time, during our participation in the Partners Program.

What would be the language to add to that statement? cc @raviorch

Eugene,
We definitely would like to add verbiage like you mentioned below. In
the meantime, you mentioned on the TSC call that you had researched
other open source project partnership charters. Can you please provide
examples of the same so the subcommittee has a starting point instead of
reinventing the wheel. This would really accelerate our progress as
other open source projects may have solved most of the issues we are
seeing here. Please send this out asap as I will use that to put a draft
out before the weekend if that is available.
Ravi
------ Original Message ------
From: "Eugen Cusmaunsa" @.>
To: "StackStorm/discussions" @.>
Cc: "raviorch" @.>; "Mention"
@.>
Sent: 4/6/2021 10:43:41 AM
Subject: Re: [StackStorm/discussions] Partnership Program Subcommittee
(#73)

We should have verbiage to mitigate legal risk from incoming partners
in case of possible "bad actors". Maybe something simple along the
lines of:

I, as a partner, accept to not hold StackStorm accountable [...]

Here is what LF legal proposed and so it was drafted on the
https://stackstorm.com/partners-program/ (beta):

In your [partnership] proposal, please include the following
statement:

On behalf of PARTNER NAME ("Participant"), I agree that Participant
will comply with the StackStorm Partners Program requirements
published at https://stackstorm.com/partners-program/ and Linux
Foundation Policies published at https://lfprojects.org/policies/, as
updated from time to time, during our participation in the Partners
Program.

What would be the language to add to that statement? cc @raviorch
https://github.com/raviorch
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/StackStorm/discussions/issues/73#issuecomment-814308752,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQHYO6U4LQTYYGPZFHXKEDDTHNB43ANCNFSM42PEP2FA.

Mick, JP, Edwin, please let me know what time is the best for us to meet. I have asked Eugene to provide some examples, in the meantime here are some examples I found:
https://www.mongodb.com/partners/partner-program
https://partner.suse.com/English/
SUSE-Application-Process-Final.pdf
https://partners.gitlab.com/English/
There a bunch more, but as it clearly states in the GitLab Channel Partner page, https://about.gitlab.com/handbook/resellers/
It talks about what the community provides to the partner first before getting into what the partner should provide, this is the type of language that we should use.

@raviorch These are good examples. However, you're referring to the projects with the Open Core + Enterprise model. This is what Extreme Networks was before with the EWC. I'd imagine EXTR would absolutely need a complex multi-tier Partners Program with the Legal Agreements, etc, etc as it's a B2B partnership.
This is a very different model compared to how StackStorm 100% Open Source under the neutral non-profit LF governance operates with different values and system behind.
Here is an example for the Debian Open Source Partner program that was a prototype for the https://stackstorm.com/partners/ and https://stackstorm.com/partners-program/ drafts.
https://www.debian.org/partners/
https://www.debian.org/partners/2021/partners
In short, they want to recognize orgs that helped and provide recognition (listing on the page) as "official partner" for those significantly helping the project. StackStorm can go further and provide co-marketing opportunities (examples at https://github.com/StackStorm/discussions/issues/51).
Similar to Debian, do you believe we would be able to focus on simplicity and maintenance-free keeping in mind the Open Source community interests?

Meeting
@raviorch  I can meet for either of the following

Wednesday, April 14 at 3 PM Eastern
Thursday, April 15 at 3 PM Eastern

Benefits
@raviorch  All of the links you posted (mongo, suse, gitlab) clearly display benefits of becoming a partner (to @punkrokk's point), and I think we should outline similar benefits.  @armab's proposal does have a section for it already. It mostly speaks to 2 things:

Redirect community inquires to the partners page
Participate in shared marketing (blogs, etc)

Also, I like the table presentation in the mongo partners page:

GitLab has a similar one:

Possible Tasks

I think for the benefits, we could condense the proposed language into a clearly presented table ('cuz words are hard :P ), and we could probably use the existing proposed language, too.
Badges are nice.  We have a 'Partner' badge already, and we could expand this concept over time.  We should also probably have a section in the partners page that describes our badges program so that it's clear to applying partners what they'll get.

Language
Mongo, gitlab, and debian, do not seem to provide language to the effect of "we'll not hold the project accountable" (if they do, it's not obvious).
Suse might, but you'd have to go through the partner registration to get the terms and conditions pdf (it might be available elsewhere, but I haven't had a chance to look).
The partners code of conduct says a lot (esp. the conflict of interest), but I think it lacks one thing that was discussed during the TSC meeting which is: simple, visible language to say "don't sue ST2, yo".
@armab, the language in the proposed draft also doesn't seem to convey that message (don't sue st2).
I think the key point here is that I, as not a lawyer, don't see anything that explicitly states that ST2 won't be liable.  It's possible it's buried in the verbiage somewhere, but without being well versed in legal-ese, it's not clear, and I suspect that the same will be true for others, so clear, concise language might be helpful?
Possible Tasks

Should we just include a simple message (either to the partners page or to the submission request) to say that the applying partner agrees not to hold ST2 accountable?  Again, something to the effect of I, as a partner, accept to not hold StackStorm accountable [...]

@raviorch
Thursday, April 15 at 3 PM Eastern would work for me

Thanks @mickmcgrath13. I will join the meeting with Ravi.

Invites sent!  Let me know if you did not receive them.

Should we just include a simple message (either to the partners page or to the submission request) to say that the applying partner agrees not to hold ST2 accountable?

Perhaps then we could merge what the LF recommended to us and the proposed "non-accountable" wording:

In your [partnership email] proposal, please include the following statement:

On behalf of PARTNER NAME ("Participant"), I agree that Participant will comply with the StackStorm Partners Program requirements published at https://stackstorm.com/partners-program/ and Linux Foundation Policies published at https://lfprojects.org/policies/, as updated from time to time, and agree to not hold StackStorm liable for any damages or loss during or after our participation in the Partners Program.

@mickmcgrath13 Can you retitle this issue with the date of the meeting and update the issue description? See #74, #75, and #76 for examples.

2021-04-15 - Meeting notes
Discussion about Ravi's proposal
Screenshared the proposal..  Ravi will upload/share
Liability/Indemnification discussion
Main goal is to remove liability from partners to stackstorm.
Just include: "partner agrees StackStorm liable"

on the website?
in the email they send?
in the email that we send back to them?

Mutual indemnification clause will be a case-by-case basis
Common partner contributions
Types of contributions that can be made.
Goal is for partners to be labeled for their core competencies so that the public can evaluate who to come to (properly communicated on the StackStorm page).
Opportunities for badges?

engineering, professional services, business development, marketing
or labels per experience
or bronze, silver, gold, platnum

difficult because of additional mental hops for customers
higher level partners would need higher level contributions from StackStorm

What can stackstorm give back to partners
Really the only way today is to highlight the partners.
Next Steps

@raviorch to upload/share his proposal so we can iterate on the verbiage
Iterate and clean up the doc

clean up "how and when do partnerships end" ?

JP or Ravi to get an indemnification clause to include for the email
Decide on /partners or /partners-program
translate into wordpress
TSC Vote
Should Existing Partners (Bitovi, Orchestral, etc) also send the email?

Partners program draft doc by @raviorch : https://drive.google.com/drive/folders/1x4-9cX-hrvogrSoLc2FE63BH91dUGN9E

..also, I did record the call.  I'll link it here as soon as I can find where it went :P

@raviorch fyi: i updated the doc so that anyone can comment but only some can edit

Here's the recording of the meeting from 2021-04-15:
https://drive.google.com/file/d/179qnP_s0Y3AGTLdUIj_-_twds9fx66mp/view

There have been several comments/suggestions to the doc:
https://docs.google.com/document/d/1EVD3efr_cJ-amHyHfDBtMZXmLTFpb4-I/edit
I think most could be simply resolved (cc @raviorch ).  I'm happy to do it :)
There are some that will require further discussion, though

All changes from the doc have been updated in the draft
Next Steps

Define labels (table headers or table cells?)
Submit formal proposal for TSC voting
Make it live!  (i.e. Page swapping https://stackstorm.com/stackstorm-partners-program-description/?preview=true from https://stackstorm.com/partners-program/)
Determine the criteria for "How and when do partnerships end? > The support, contribution or service is no longer delivered"
Revisit partner levels (i.e. bronze, silver, gold) Once a governing committee is defined/established

Define labels (table headers or table cells?)
We've generally decided that the labels will be based on the table headers.  I've set up some common symbols to be able to be re-used and added as labels per entry in the partners page:
https://stackstorm.com/partners/

Example:

Next steps:
Submit formal proposal for TSC voting

Make it live! (i.e. Page swapping https://stackstorm.com/stackstorm-partners-program-description/?preview=true from https://stackstorm.com/partners-program/)
Determine the criteria for "How and when do partnerships end? > The support, contribution or service is no longer delivered"
Revisit partner levels (i.e. bronze, silver, gold) Once a governing committee is defined/established

Just discussed it in the TSC the screenshot/sketch. This is very helpful to get an idea, thanks a lot @mickmcgrath13 !
A few comments:

Looks like it takes more space on the page extending the entire height. Once we put it together and repeat for each partner, - that might look more overwhelming. More scrolling, fewer partners on a visible page space.

Based on discussions we had, some partners expressed the fact they don't want to be associated with the pre-defined explicit written labeling like that.

If we'll label PackageCloud, CloudSmith, AWS which provide hosting/infrastructure and associate them with the Business Development label, - it's irrelevant and confusing not just for some partners but also users.

Can we make these labels more compact?
Conceptually, it shouldn't take more visibility than the call-to-action More Info button, which is the main URL and landing page target for each partner. Normally Partners would be interested to make the most visible call-to-action URL button and not the label.

How about positioning labels just as icons near the partner's name?

Bitovi label-icon1 label-icon2

We'll add a tooltip for each icon (when the user places his cursor mouse over the icon) hinting if it's a Business Development, Professional Services, Marketing, etc.
This way partners contributing to more categories will just have more icon labels highlighting their level of involvement. It won't overwhelm design much, serve the goal and won't repel potential partners nor confuse users.

See:

@armab I agree the large labels are a bit much.  Icons with tooltips are doable and gives the best of both worlds.
Placing icons beside the partner name is tricky with the thrive architect, so below the name would be easier.  We can do icons below easily with a tooltip:

with tooltip:

Also, I'm not sure I understand this:

Based on discussions we had, some partners expressed the fact they don't want to be associated with the pre-defined explicit written labeling like that.

and this:

If we'll label PackageCloud, CloudSmith, AWS which provide hosting/infrastructure and associate them with the Business Development label, - it's irrelevant and confusing not just for some partners but also users.

are we going with the 4 icons which are currently the headers of the table in the draft page?  if not, what all labels/icons do we need?  Currently, we have:

If this is not what we want for the labels, please recommend a list of labels (and associated icons).  IMO, these 4 'categories' should be sufficient until such time as a partner says "i'd like something different"

also, fyi, to add the labels, do the following:

insert a content box under the partner name text box

remove all margin and padding except for margin-bottom of 20

insert icon into content box

remove all margin and padding
update colors (orange BG, white icon)
set alignment to the left
change "state" to "hover"
add tooltip with the name of the label (position top)
repeat for other icons

:100: That would work! :+1:  @mickmcgrath13
Or even switching from Thrive Architect to raw HTML is OK too to make the icons inline.
After all, we used the tool to get this page initially working and with a stable Partners version we won't need to implement anything new. Just edit old partners and add new ones.
Eventually, it'll be static HTML anyways as we migrate out from the WordPress.

I went down the plain HTML route before, and it didn't seem quite as straight-forward as I had hoped.
I found a way to get the icons beside the partner with Thrive, anyway:

add content box

remove margins and padding

add text box with partner name

align left
display inline
float
margin-right: 7 (remove all other margins and padding)

add icon

align left
display inline
float
margin-right: 7 (remove all other margins and padding)
set size to 18
set padding to 4 all around
update colors (orange BG, white icon)
change "state" to "hover"
add tooltip with the name of the label (position top)
repeat for other icons

It's also helpful to make use of the copy functionality as it copies all the preset settings from the module that is copied.
Here's an example:

Official Vote here: https://github.com/StackStorm/discussions/issues/80

Closing as all the pages are live!


DVC 7695: Add basic mypy and black checks

The code changes in here are all just reformatting. Still working on some of the type errors, but we can get the formatting changes in first and get them fixed in a subsequent PR.

Current dependencies on/for this PR:

main

PR #33

PR #31   👈

This comment was auto-generated by Graphite.


UHF-9113 clear invalid permissions

UHF-9113

What was done

Remove invalid permissions.

Drupal 10 does not allow roles to have permissions that do not exists.
How to install

Make sure your instance is up and running on latest dev branch.

git pull origin dev
make fresh

Update the Helfi Platform config

composer require drupal/helfi_platform_config:dev-UHF-9113-clear-invalid-permissions

Run make drush-updb drush-cr

How to test

[ ] Check that code follows our standards

Other PRs

https://github.com/City-of-Helsinki/drupal-hdbt/pull/800
https://github.com/City-of-Helsinki/drupal-helfi-kasvatus-koulutus/pull/462
https://github.com/City-of-Helsinki/drupal-helfi-platform-config/pull/617

Codecov Report

Merging #620 (2334e26) into main (a382779) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 2334e26 differs from pull request most recent head c9a6a6f. Consider uploading reports for the commit c9a6a6f to get more accurate results

@@            Coverage Diff            @@
##               main     #620   +/-   ##
=========================================
  Coverage     13.29%   13.29%
  Complexity      241      241
=========================================
  Files            30       30
  Lines           910      910
=========================================
  Hits            121      121
  Misses          789      789

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


lestarch: added overridable build cache to fprime-util

Originating Project/Creator

Affected Component

Affected Architectures(s)

Related Issue(s)

Has Unit Tests (y/n)

Builds Without Errors (y/n)

Unit Tests Pass (y/n)

Documentation Included (y/n)

Change Description
Adds in the ability to add a specific build cache using the --build-cache flag.  This way any number of configurations can be maintained by hand.
e.g.
fprime-util generate --build-cache my_fancy_build -DSOME_SETTING=ON -DSOME_OTHER_SETTING=OFF

@vietjtnguyen does this meet your recommendation?  Rather than managing suffixs I figured I'd put in the full explicit override so the user can specify any build cache, even one made by hand.

@Joshua-Anderson would it be possible to have you review this PR?

@LeStarch This would be perfect.

@Joshua-Anderson I made a few more changes.  Thoughts?  This fixes info hash-to-file so that it works with custom cache, and adds an integration test that tests for the custom build setup!


Add IAM role for workers to easily access cloud storage

@jerrychenhf Hi, haifeng. I've finished the code development for this PR. Thanks.

The code looks good to me.
Thank you the feature! It's a huge step forward and it was implemented in a quite short time frame!


Mask access token in migration forms

Mask the field without the downsides of type=password. It might be argued that there should be a "reveal" button, but this is too tricky to implement and I see that as the job of the browser, and I guess some mobile ones will provide such a button. Also, disable autocomplete so these tokens do not end up in form history.
Fixes: https://github.com/go-gitea/gitea/issues/22174
Fixes: https://github.com/go-gitea/gitea/pull/22175
Firefox:

Chrome:

Questions:

Should the webhook auth header field be included? I'm leaning towards yes but the content there is not only a token. Users might have a hard time inputting the exact header without seeing the typed characters
Any other token fields I've missed?

Is this still a draft?

I know some people like to Copy&Paste non-ASCII chars into the password input, to make their password "more secure". Which is pretty valid.
Then for fun, you will see this with this patch -- which is not a general approach.

Actually, let's close this, the firefox workaround is too much of a hack.
Proper solution should be type=password, we just need to figure out how to suppress the browser from offering a "Save Password" prompt. Some other forms like webhook already use type=password and there is no prompt, so it must be possible somehow.


LEGO: Pull request from lego/hb_839_20170529144037140 to master

LEGO: Pull request from lego/hb_839_20170529144037140 to master with localized lcls

Closing due to merge conflict


Fix show/hide completion callback

Otherwise setting the completion callback to e.g. this._showMessageBarAlertComplete() executes the callback straight away, not after the animation finishes
Setting it to this._showMessageBarAlertComplete only partially works - .bind(this) is needed to resolve the references to 'this' inside the callback

Thanks! I'll give this a quick test and merge if everything seems ok

What’s the status of this? Still WIP or ready for review? Thanks for the improvements btw

Oh yeah sorry, yeah it's only commit 4db7e2a that can be merged back in really, the rest of these are custom fixes for our fork (we only use slide from top, and have been a bit pushed for time to work out a general fix for the initial animation lag)

The show/hide animation lag is due to the component starting way off screen (by the full screen width / height) - if the animation starts from there, then most of the animation has been missed by the time it actually arrives on screen.
Initially I tried a solution that automatically recalculated the component size for the onLayout event in View.Animated and started the animation by only this amount off-screen, but this was a bit experimental - ideally the component size needs to be calculated before it's displayed (i.e. before the onLayout event).
In the end we've gone for a general purpose approach of started all slide from top animations from 30% off the top of the screen, as our notifications don't use more than this, but this is for our use only really - for a general approach we need to figure out a way of automatically calculating the correct component size after the title / message is set but before the animation starts.

Actually the Android fix for 5e1e05c can be merged in, but that needs extending in a similar way to the other 3 slide types


Resize in Aspect ratio

Hi,
It is wonderful lib but can this supports resizing with the aspect ratio. like 4:3 or 3:4

Thanks for the additional comments.
Version 0.3.0 has been released. 🎉
isSquare, which expresses a square, has been replaced with AspectRatio.
Added AspectRatio property.
AspectRatio defines the aspect ratio.
AspectRatio is in the form of a CGFloat and means horizontal: vertical.
E.g
If width:height = 1:2
aspectRatio = 1/2.
Have a nice day 👍

Thanks  a lot .
Sanjay Singh Rathor
On Wed, Aug 11, 2021 at 12:15 PM Gwan-ho Kim @.***>
wrote:

Thanks for the additional comments.
Version 0.3.0 has been released. 🎉
isSquare, which expresses a square, has been replaced with AspectRatio.
Added AspectRatio property.
AspectRatio defines the aspect ratio.
AspectRatio is in the form of a CGFloat and means horizontal: vertical.
E.g
If width:height = 1:2
aspectRatio = 1/2.
Have a nice day 👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pikachu987/CropPickerView/issues/8#issuecomment-896546244,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIFF4452P7T6AKFRCH5VD2LT4IMCBANCNFSM5B365Y7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.


Update hiveserver2.py

added llap editor type to session lookup filter by type beeswax, since all hive sessions are cached as beeswax sessions

Thank you @theyaa and congratulation for the first commit!


add message as property for BIMDataLoading

Hello team,
Here is a PR that add 2 properties for the Loading component

message: to replace the default text 'loading...'
subMessage: add an extra space to put content in

I thought to add a third property "color", but there is no need for now, let me know if you want me to add it anyway
Also, the doc for the loaders page looks "old school" because it's not possible to modify properties dynamically (in comparaison to BIMDataInput for exemple). Let me know if you want me to update that
Paul

About the "color" prop, I think it's OK, we can add it later if needed. :)

:tada: This PR is included in version 1.2.0-rc.41 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


JS Screen position out of view frustum errors in Chrome Canary

ensure you are on Windows and have Steam VR installed (tested on Dell Visor MR and Vive headsets)
download Chrome Canary for Windows
ensure these Chrome flags are all marked as Enabled:

WebVR — chrome://flags#enable-webvr
OpenVR — chrome://flags#openvr
Gamepad Extensions — chrome://flags#enable-gamepad-extensions

load https://caseyyee.github.io/unity-webvr-export/Build/
after "Loading..." screen, the scene is black. until I press the "Enter VR" button, there are constant Console errors: Screen position out of view frustum    blob:https://caseyye…c-66cfdf698c02:7636
when I click the "Enter VR" button, the errors stop and the stereo, mirrored canvas appears as expected. when I toggle from mono to stereo, after that, it's fine.

FYI, I got this error in Unity when the camera is very far from the origin.

oh interesting.   do you mean far away from 0,0,0 world origin?  moving the cameraset prefab?  eg 1000, 0, 0 or something like that?  I'll give it a try.  thanks!

I mean super far away from the origin. Not like 1000 but much more.

so, y'all still see this error? what about in PR #132?
I can't reproduce it anymore. I'll try with my other VR setups, one which has a much larger play space (stage).

I'm pretty confident in #132 solving this issue too.


Tar/Untar in a Flow?

Hi,
A common workflow I want to express in globus is to:

select a directory
tar it
move the tar file to a new destination
(essentially what htar does for HPSS)
optional: remove the tar file.

And the opposite:

copy a tar file to a new destination
untar it
optional: remove the tar file.

Is there a flow for this or could this be made an example?
Currently, for really large data sets I am still doing a combination of Globus + screen for untar, and there must be a better way.

Hi!
The cleanest way to solve this today would be with Compute endpoints on both sides (in other words, with direct access to the files on each collection): The flow could then invoke Compute functions on each of the endpoints to tar and untar as needed before/after the transfer.
We're not quite ready to document a general purpose approach to this yet, but I do think this is a very good idea, so I'm adding a story to our internal tracker to create a tutorial for the docs site on how to do this. I can't give an ETA on when that will get tackled, but it will be in the hopper.
If you could use assistance with your particular configuration in the meantime, opening a ticket via<EMAIL_ADDRESS>will help us make sure that we can provide support that is specific to your scenario.

This repo includes a basic example of the flow that Ada mentions; see https://github.com/globus/globus-flows-trigger-examples/tree/main/tar_transfer and its associated trigger script. A simple wrapper function for the "tar" action is in https://github.com/globus/globus-flows-trigger-examples/blob/main/functions/tar_function.py. Let us know if you end up using/extending this in the interim.


Random-route not used on Green app

Hi,
It would be great for us if the deployment of the Green app could use 'random-route' if specified in the manifest. The copy of the plugin we are using now always deploys the Green app with a route of <app_name>-new., which has the potential to cause route conflict.
As an aside, the plugin does correctly carry out a blue-green deploy on an existing random route that is attached to the Blue copy of an app.
Thanks

Is this the behaviour you would expect?

The first time an app is pushed using bgd, a new random route is created
On subsequent 'bgd' calls, the same route is preserved
The 'intermediate' app uses a random route, but with -new on the end (and the same for -old and -failed)
If a host or route is also present in the manifest, the random route isn't honoured

Yes - I think this looks correct. Thanks


AnalysisException: [RequestId=a822a302-12f2-4fb4-b181-a638b39f3171 ErrorClass=INTERNAL_ERROR]

I am using Databricks on AWS using DBR 11.3LTS.
I have a table that is parittioned by a very long URL.  When I try to create a share using the PARTITION command I am getting an Internal Error.
Command:
ALTER SHARE my_share ADD table my_catalog.my_schema.my_table
partition (url = 'https://some-files-from-the-internet.com/machine-readable-files/Public/2023-03-13_usable_company_dba_this_is_a_company_name_here_are_some_files_1_of_2.json.gz');

Error:
AnalysisException                         Traceback (most recent call last)
File <command-3224760991686733>:7
      5     display(df)
      6     return df
----> 7   _sqldf = ____databricks_percent_sql()
      8 finally:
      9   del ____databricks_percent_sql

File /databricks/spark/python/pyspark/instrumentation_utils.py:48, in _wrap_function.<locals>.wrapper(*args, **kwargs)
     46 start = time.perf_counter()
     47 try:
---> 48     res = func(*args, **kwargs)
     49     logger.log_success(
     50         module_name, class_name, function_name, time.perf_counter() - start, signature
     51     )
     52     return res

File /databricks/spark/python/pyspark/sql/session.py:1387, in SparkSession.sql(self, sqlQuery, args, **kwargs)
   1385 try:
   1386     litArgs = {k: _to_java_column(lit(v)) for k, v in (args or {}).items()}
-> 1387     return DataFrame(self._jsparkSession.sql(sqlQuery, litArgs), self)
   1388 finally:
   1389     if len(kwargs) > 0:

File /databricks/spark/python/lib/py4j-<IP_ADDRESS>-src.zip/py4j/java_gateway.py:1321, in JavaMember.__call__(self, *args)
   1315 command = proto.CALL_COMMAND_NAME +\
   1316     self.command_header +\
   1317     args_command +\
   1318     proto.END_COMMAND_PART
   1320 answer = self.gateway_client.send_command(command)
-> 1321 return_value = get_return_value(
   1322     answer, self.gateway_client, self.target_id, self.name)
   1324 for temp_arg in temp_args:
   1325     temp_arg._detach()

File /databricks/spark/python/pyspark/errors/exceptions.py:234, in capture_sql_exception.<locals>.deco(*a, **kw)
    230 converted = convert_exception(e.java_exception)
    231 if not isinstance(converted, UnknownException):
    232     # Hide where the exception came from that shows a non-Pythonic
    233     # JVM exception message.
--> 234     raise converted from None
    235 else:
    236     raise

AnalysisException: [RequestId=a822a302-12f2-4fb4-b181-a638b39f3171 ErrorClass=INTERNAL_ERROR]

From testing around it seems like there may be a character length issue.  If I keep the character length under 117 characters it will work.  Is there a work around without having to modify the partition key?

@gardnmi Thanks for reporting the issue. We'll investigate.
Are you on aws us west 2?

@gardnmi Thanks for reporting the issue. We'll investigate.
Are you on aws us west 2?

Yes


Problem displaying long error messages

When error messages are long the interface looks a bit off, for example see the screenshot below, notice how big the window is and how small the horizontal scroll bar is:

This in turn "hides" the delete button of the raw data:

Good call Yoshiki - very frustrating
On Wed, Jun 17, 2015 at 5:20 PM, Yoshiki Vázquez Baeza <
<EMAIL_ADDRESS>wrote:

When error messages are long the interface looks a bit off, for example
see the screenshot below, notice how big the window is and how small the
horizontal scroll bar is:
[image: screen shot 2015-06-17 at 4 55 31 pm]
https://cloud.githubusercontent.com/assets/375307/8221345/1daadbee-1512-11e5-99d1-2676700db337.png
This in turn "hides" the delete button of the raw data:
[image: screen shot 2015-06-17 at 4 59 18 pm]
https://cloud.githubusercontent.com/assets/375307/8221359/454b9a4e-1512-11e5-9f92-5ed1ae78dff4.png
—
Reply to this email directly or view it on GitHub
https://github.com/biocore/qiita/issues/1271.

--
Gail Ackermann
Knight Lab
UCSD
<EMAIL_ADDRESS><EMAIL_ADDRESS>

We should probably replace the \n by <br> and that will highly improve the situation.

@squirrelo I think this is fixed with one of your latest pull request, if so, can you link it from the PR so it will get automatically closed?

I'll double check. It may need styling fix as well.

Just checked and AFAIK it's fixed. Closing now, please open other issue if you don't agree.


Upgraded knplabs/packagist-api to ~1.5.1

[x] added ext-gd and ext-curl as dependencies in composer.json
[x] upgraded knplabs/packagist-api to ~1.5.1,
[x] now using Guzzle6, rewrote response mocks,
[x] upgraded rest of the dependencies.

Thank you for your contribution, but we have stopped the development of version 2. We are working on version 3 that will be online soon!
https://github.com/PUGX/badge-poser/tree/release/v3.0.0
Thank you


[Installation]: Problem with docker image (ROCm version)

Your current environment
The output of `python collect_env.py`

Collecting environment information...
PyTorch version: 2.3.1+rocm6.0
Is debug build: False
CUDA used to build PyTorch: N/A
ROCM used to build PyTorch: 6.0.32830-d62f6a171
OS: Ubuntu 22.04.4 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.35
Python version: 3.10.12 (main, Mar 22 2024, 16:50:05) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.35
Is CUDA available: False
CUDA runtime version: No CUDA
CUDA_MODULE_LOADING set to: N/A
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Address sizes:                      46 bits physical, 48 bits virtual
Byte Order:                         Little Endian
CPU(s):                             32
On-line CPU(s) list:                0-31
Vendor ID:                          GenuineIntel
Model name:                         Intel(R) Core(TM) i9-14900KS
CPU family:                         6
Model:                              183
Thread(s) per core:                 2
Core(s) per socket:                 16
Socket(s):                          1
Stepping:                           1
BogoMIPS:                           6374.40
Flags:                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology tsc_reliable nonstop_tsc cpuid pni pclmulqdq vmx ssse3 fma cx16 sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch ssbd ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 xsaves avx_vnni umip waitpkg gfni vaes vpclmulqdq rdpid movdiri movdir64b fsrm md_clear serialize flush_l1d arch_capabilities
Virtualization:                     VT-x
Hypervisor vendor:                  Microsoft
Virtualization type:                full
L1d cache:                          768 KiB (16 instances)
L1i cache:                          512 KiB (16 instances)
L2 cache:                           32 MiB (16 instances)
L3 cache:                           36 MiB (1 instance)
Vulnerability Gather data sampling: Not affected
Vulnerability Itlb multihit:        Not affected
Vulnerability L1tf:                 Not affected
Vulnerability Mds:                  Not affected
Vulnerability Meltdown:             Not affected
Vulnerability Mmio stale data:      Not affected
Vulnerability Retbleed:             Mitigation; Enhanced IBRS
Vulnerability Spec rstack overflow: Not affected
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; Enhanced IBRS, IBPB conditional, RSB filling, PBRSB-eIBRS SW sequence
Vulnerability Srbds:                Not affected
Vulnerability Tsx async abort:      Not affected
Versions of relevant libraries:
[pip3] numpy==1.26.3
[pip3] pytorch-triton-rocm==2.3.1
[pip3] torch==2.3.1+rocm6.0
[pip3] torchaudio==2.3.1+rocm6.0
[pip3] torchvision==0.18.1+rocm6.0
[pip3] triton==3.0.0
[pip3] triton-nightly==3.0.0.post20240716052845
[conda] Could not collect
ROCM Version: 6.1.40093-bd86f1708
Neuron SDK Version: N/A
vLLM Version: N/A
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
Could not collect
How you are installing vllm
Using Option 1: Build from source with docker (recommended)
DOCKER_BUILDKIT=1 docker build --build-arg BUILD_FA="0" -f Dockerfile.rocm -t vllm-rocm .
=> ERROR [final 4/8] RUN --mount=type=cache,target=/root/.cache/ccache     --mount=type=cache,target=/root/.cach  0.6s

[final 4/8] RUN --mount=type=cache,target=/root/.cache/ccache     --mount=type=cache,target=/root/.cache/pip     pip install -U -r requirements-rocm.txt     && case "$(ls /opt | grep -Po 'rocm-[0-9].[0-9]')" in         "rocm-6.1")             wget -N https://github.com/ROCm/vllm/raw/fa78403/rocm_patch/libamdhip64.so.6 -P /opt/rocm/lib             && rm -f "$(python3 -c 'import torch; print(torch.path[0])')"/lib/libamdhip64.so* || true;;         *) ;; esac     && python3 setup.py clean --all     && python3 setup.py develop:
0.515 ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements-rocm.txt'

1 warning found (use --debug to expand):

UndefinedVar: Usage of undefined variable '$CPLUS_INCLUDE_PATH' (line 63)
Dockerfile.rocm:138

137 |
138 | >>> RUN --mount=type=cache,target=${CCACHE_DIR}
139 | >>>     --mount=type=cache,target=/root/.cache/pip
140 | >>>     pip install -U -r requirements-rocm.txt
141 | >>>     && case "$(ls /opt | grep -Po 'rocm-[0-9].[0-9]')" in
142 | >>>         "rocm-6.1")
143 | >>>             # Bring in upgrades to HIP graph earlier than ROCm 6.2 for vLLM
144 | >>>             wget -N https://github.com/ROCm/vllm/raw/fa78403/rocm_patch/libamdhip64.so.6 -P /opt/rocm/lib
145 | >>>             # Prevent interference if torch bundles its own HIP runtime
146 | >>>             && rm -f "$(python3 -c 'import torch; print(torch.path[0])')"/lib/libamdhip64.so* || true;;
147 | >>>         *) ;; esac
148 | >>>     && python3 setup.py clean --all
149 | >>>     && python3 setup.py develop

150

ERROR: failed to solve: process "/bin/sh -c pip install -U -r requirements-rocm.txt     && case "$(ls /opt

You should clone vllm repo first. And then, try to build within the repo.
There's COPY command in Dockerfile.rocm. This copies requirements-rocm.txt.


pagination: buggy first/previous/next/last logic

I feel like the logic used to hide/show the various previous/first/next/last links is buggy. Here are some screenshots:

^ I'm on the first page, everything is alright.

^ I'm on the second page, I don't think the "<<" (first page) link is needed (redundant with "<" previous)

^ 4th page, everything is alright

^ 5th page, oops the 6th page link disappeared. And I have both next and last (last might be useless here, and related to #594)

I think, the 6th page disappearing is a bug we need to fix (bug)
As for the "I am on second page", we should keep showing the <<, not really a big deal nor something we should again weight our preference I feel.
Others are good as you said

Some thoughts because obvious always win:

It's not recommended to use only icons most of the time so plain text 'previous', 'first', .. should be default.
it's  not recommended to hide elements from an ui, so visible + disabled should be the default.
the current representation does not inform about the total pages (and when all pages are displayed first/last links are redondant, useless and even confusing when not labelled.
There is a lot of different config for pagination, no silver bullet, it really depends of the usecase.
I'm personally more often attracted by ellipsis solution (always show 3 first pages, and last one)

Ok let's fix the obvious bug then :)

Ok, let's keep this issue focused on the 6th page bug then. We'll have to discuss a bit more to see how we want to tackle the other pagination UI questions

Pull request is here : https://github.com/algolia/instantsearch.js/pull/675


how to add load progress bar and video control UI(overlay)  always user direction

Hi,
we have implemented video streaming app for 360-degree videos and the user can rotate in 360 degrees to watch the video.
Whenever video buffers we show loading circular progress bar and also video controller buttons(seek bar, pause, forward, rewind buttons)  but here the problem is how to show the progress bar and Controller UI(seek bar, forward, rewind , pause buttons) in user watched direction, I mean how to change the loading progress bar and Controller UI position based on user direction.
Whenever volume increased or decreased i can see volume increment or decrement texture showing in user direction, i want to implement the same behavior for my circular loading bar and Controller UI.
Please suggest the approach or any reference sample app.
Thanks

@spakanati123 Hi. Sounds like you need GVRScene.getMainCameraRig().addChildObject which would attach a scene object to the camera rig and should do what you are asking for. You can look at the gvr-switch sample and how the headTracker is added (scene.getMainCameraRig().addChildObject(headTracker)).

@liaxim Thanks for the update, it's working as required.


Accept invalid/self-signed SSL certificates

Hey @TheBinitGhimire,
Fix for: https://twitter.com/WHOISbinit/status/1357018526171553792
The issue was that the module reqwest doesn't allow connecting to a host with invalid/self-signed SSL certificates as it could introduce vulnerabilities (As mentioned in docs). But for use cases like this, we can override it with .danger_accept_invalid_certs().
I believe this solves the issue as a whole. I tested with a random IP I found with invalid certificate and it doesn't reach Err(_).

Also compiles without any warnings too. :+1:
Nice project and Good Luck! :raised_hands::heart:

hope this PR gets accepted :)


How do I get the title of the Paper?

Screenshot:
https://paste.pics/d72fd7940436d35aaff30fef85c99fd1
I am able to retrieve the header text with processHeaderDocument
How can I get the title instead of header text (blue rectangle, see the screenshot img)?
Thank you!

Thank you @kermitt2! The issue has been resolved. I use processFulltextDocument instead. Thank you! ❤


Fixes cgroup play when running 64bit raspbian.

Fixes #88

Just noticed this is a duplicate of #74


sfxr: init at 1.2.1

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Pretty cool program :D Everything seems to work: Creating, playing audio, modifying all kinds of sliders, exporting, importing.

@JohnAZoidberg thanks for your review :+1:
@fgaz thanks for your contribution to nixpkgs! :tada:


feature: support long descriptions for subcommands

I have a subcommand where several of the options take a string, which are interpreted the same way for all of these options. Instead of giving details of possible string values for each options, I'd like to add this to the description of the subcommand.
Currently I can use a multi-line string for the description of the subcommand, and this works well if I do cmd subcmd --help. However, if I just do cmd or cmd --help it shows the full description for that subcommand (instead of just a one-line summary) making the output quite hard to read.
It would be nice if there was a way to specify the "long" and "short" description for the subcommand. The short (one-line) description would only be shown when that subcommand is listed with other subcommands. The long description would be shown when help is given only for that subcommand.

One idea is to keep the API as it exists today, and the "short" description (shown when a bunch of subcommands are listed together) is just the first line of command description.

Would the LongDesc  field in Cmd be suitable for your need ? https://godoc.org/github.com/jawher/mow.cli#Cmd
Given this conf:
app := cli.App("app", "desc")
app.BoolOpt("f", false, "option desc")
app.Run(os.Args)

Calling --help  would normally produce the following output:
Usage: app [OPTIONS]

desc

Options:
  -f=false     option desc

But if you set the app.LongDesc:
app := cli.App("app", "desc")
app.BoolOpt("f", false, "option desc")
app.LongDesc = `
Lorem ipsum dolor sit amet, liber gubergren pro ut, perpetua definiebas complectitur vel ex.
Augue assueverit ad ius, no minimum ullamcorper vim. Sea mazim invidunt te.
Appareat omittantur ex est.
Expetenda efficiantur id duo.
No mei dicam affert dissentias, lorem accommodare pri cu, eu novum facete vulputate est.
`
app.Run(os.Args)

It would instead produce:
Usage: app [OPTIONS]

Lorem ipsum dolor sit amet, liber gubergren pro ut, perpetua definiebas complectitur vel ex.
Augue assueverit ad ius, no minimum ullamcorper vim. Sea mazim invidunt te.
Appareat omittantur ex est.
Expetenda efficiantur id duo.
No mei dicam affert dissentias, lorem accommodare pri cu, eu novum facete vulputate est.

Options:
  -f=false     option desc

Something like LongDesc() for cli.Cmd would be perfect, but it doesn't appear to exist.

Oh, it's field, and it works perfectly. (Sorry, I'm new to golang.) Thanks!

@drewfish No worries, glad it worked out for you 😄


amend(203): Require votes receive 2/3 votes to pass

What
Require that successful proposals receive at least 2/3s of the votes cast.
Why
With quorum defined in #22, having a 50% + 1 voter support on a 50% + 1 eligible vote participation means a group of about 25% of the active players could collude to pass rule-changes through inaction of other players.
Notes

201

+1


No smoothScroller like RecyclerView

I can't find smoothScroller like RecyclerView.
https://developer.android.com/reference/android/support/v7/widget/RecyclerView.SmoothScroller.html

I found a tricky way to do this.
MyUltimateRecyclerView**.mRecyclerView.smoothScrollToPosition(0)**;


Multiple instances of lowfi with MPRIS?

Currently lowfi with MPRIS just exits silently (or prints "bye! :)" when -a flag is passed) when another instance of lowfi is present.
Now we Custom Track Lists I need it more then ever.

This could possibly be an MPRIS problem. Could you try without MPRIS and see if it works with multiple instances?

Yep, confirmed. Issue with mpris implementation.

Well, it sort of makes sense, I mean having two instances of lowfi both with the same MPRIS label seems pretty strange.
I'll test a couple of things, although I'm still a bit confused on why anyone would want to have multiple instances of lowfi running at the same time with MPRIS.

This is how multiple instances of mpv looks like:
mpv.instance8369
mpv.instance8347
mpv.instance8364
mpv # <- first instance
mpv.instance8472
mpv.instance8526
mpv.instance8471

KDE connect appends long (maybe UUID? can't remember) to each instance.
For my use case I would put two or more lowfi's on my second monitor, where first one would play normal lofigirl playlist and second one my custom track list. Then I would switch between the two depending on the mood. I exclusively use lowfi with MPRIS. It is much easier than selecting window in my second monitor especially because I have a script to easily cycle MPRIS players, so it's only take me two key press combinations.

Ohh alright, I'll see if I can integrate that into lowfi.
Is the numbering based on the PID for mpv?

Yeah, mpv uses pid for second and onward instances.
Ideally lowfi instance would include custom tracklist file name if --tracks flag is passed to make it easily identifiable. If it's not too much hassle to implement, of course.

I've got this so far:
lowfi.lofigirl.instance36667
lowfi.micropop.instance37037

Which I think is fine, since checking whether there are other MPRIS players running is much more involved.
Let me know what you think.

Looks great! I can easily shorten instancePID part in my player picker script, like I do with firefox (firefox.instance_1_21 -> Firefox.._21)

Awesome :)
I'm just doing "instanceX" since that appears to be the mostly universal standard.
I'll go ahead and close this, and a new release should be out pretty soon.  Haven't spotted any issue so far.


Adopt pyproject.toml in place of setup.cfg

As of setuptools v61 and PEP 621 it is possible to hold project configuration within pyproject.toml files. Documentation on this approach is available from setuptools.
Prior to making any changes I'd like to get feedback on the idea of adopting this now. Impact of the change in unclear as we don't know how many projects have been derived from the cookiecutter.
Benefits

We are already leveraging this file to control the package builds as well as pytest configuration.
pyproject.toml has native support for centralized version numbers through what it calls dynamic metadata. Could help simplify our version management.
This is the future, so adopting it means getting projects derived from the cookiecutter onto the best practices early.

Considerations

[tool.setuptools] options are still considered beta. They work, but the syntax under this heading may changes in the future.

By using pinned versions of setuptools we can minimize the impact of this but we would still need to track/manage any changes.

Unknown how cruft update will handle this change.
flake8 has not yet moved away from setup.cfg so the options need to be moved somewhere.
It is still early days in support, so adopting it may mean certain incompatibilities/workarounds for a short time (falke8 is one example).

/cc @Collinbrown95 @asolis @ToucheSir

One way to solve the flake8 problem could be to switch to https://github.com/charliermarsh/ruff. It allows for configuration via pyproject.toml and seems generally impressive performance + feature-wise.

Good find! I don't remember which extensions to flake8 we use, if any, but it won't be many. Could potentially replace a couple of the other items we have in pre-commit and centralize their config as well.

It also occurs to me that adopting pyproject.toml can help with the issues of trying to keep conda environments in sync by adopting the tooling proposed in #36.


Order Optimization Use Case: Need example of oc start-build command

Use case doc:
In the Deploy Debezium CDC Postgres connector section of the use case,  the instructions suggest you run the oc start-build command using the output of oc get build command, but that does not work.
I ran
oc get build

NAME                             TYPE     FROM   STATUS     STARTED          DURATION
connect-cluster-demo-connect-1   Source          Complete   55 minutes ago   47s
then I ran:
oc start-build connect-cluster-demo-connect-1 --from-dir vaccine-order-mgr-pg/environment/cdc/connectors --follow

Which failed with:
Uploading directory "connectors" as binary input for the build ...

Uploading finished
Error from server (NotFound): buildconfigs.build.openshift.io "connect-cluster-demo-connect-1" not found

To fix this I had to run this command instead:
oc start-build connect-cluster-demo-connect --from-dir ./connectors --follow

We should make this more clear, perhaps by providing an example of the command to run.

Added an example


SublimeText launched from GUI doesn't find ocamlmerlin

As mentioned in the SublimeREPL repo, "On POSIX system SublimeText launched from GUI does not inherit a proper environment. Often leading to problems with finding interpreters or not using the ones affected by changes in ~/.profile / *rc files".
Though this is a Sublime bug, it has the effect that Merlin often doesn't work for people who open their ml files using the Sublime GUI.... the ocamlmerlin binary often can't be found. The SublimeREPL folks have added a workaround to their plugin. It might be nice to do similar here.
Or, it might be worth documenting that users running into trouble when opening files from the GUI should manually set "ocamlmerlin_path" in their Merlin package user settings.

Hi @rebeccacremona, thanks for reporting this issue.
Indeed, manually setting ocamlmerlin_path fixes this wrong behavior. I currently think that documenting the ocamlmerlin_path variable is easier than starting to update environment variables provided by ST, but will have a look at the solution from SublimeREPL.
Meanwhile, if you want to provide a solution or update the README file, your contribution is welcome. :smile:


Ensure the cache keys include locale, as names can be localized.

This add basic support for spree-i18n.

Thanks @rposborne!


can't install

hello,
i have installed hacs.
but I can't find shutter card.
don need install a repo?

hello,
I have this same problem. I've spend in web 5h for looking answer and none. you found a solution what should do ?


Use Gatsby pathPrefix in theme

Tests pass locally, but it looks like Cypress is not working on CI. Going to merge and publish


Update MD file on new branch 2017-07-04_19-11-58

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Extending object syntax of v-on to support modifiers

What problem does this feature solve?
The object syntax of the v-on directive is vital in achieving conditional binding of event listeners, as shown in the discussions at https://github.com/vuejs/vue/issues/7349.

However, the current object syntax does not allow modifiers.
This feature request suggest that we extend the object syntax in the following way to allow modifiers.

The above example would conditionally install leftClickHandler on "click.left.prevent" and mouseoverHandler on "mouseover.stop".
The embedded object notation is also conceptually consistent with the dot-notation already adopted in both function and inline syntax.
What does the proposed API look like?
The proposed v-on object syntax would like like this, which is an extension of the current syntax.

I think the below syntax would be better (no object embedding):

This is possible today by using https://vuejs.org/v2/guide/render-function.html#Event-amp-Key-Modifiers
<button v-on="{ '~click': () => foo = new Date() }">Trigger only once</button>

Hi! I'm sorry to bring up such an old issue, but with the update to 2.6.11 i found that a thing related to this i used a lot in my project broke down.
I'm attaching v-on:click and v-on:click.native to a <component tag, which can dynamically be a Vue component or a DOM element.
This now emits warnings, flooding logs and affecting development performance.
I tried to attach those listeners dynamically but there's no object syntax for the .native modifier. How can i handle this?

@Zsavajji if you think you found a bug please file a new issue with a  boiled down reproduction

Not sure if it is a bug or expected behavior. Will open a bug report tomorrow :)

Hello.
I tried to do it like this, but it does not work.
// not working
v-on="{ [condition ? 'click.stop' : 'click'] : eventfunc }"
// or
// error
v-on="{ condition ? 'click.stop' : 'click' : eventfunc }"

You need to create a variable.
// in created
this.blabla = condition ? 'click.stop' : 'click';

// in template
v-on="{ blabla : eventfunc }"

Is there any other solution?
Thanks in advance!

This does not seem to work for all event methods. Only those which have the proprietary prefixes.

This is possible today by using vuejs.org/v2/guide/render-function.html#Event-amp-Key-Modifiers
<button v-on="{ '~click': () => foo = new Date() }">Trigger only once</button>

This seems nice, but undocumented. However, the .native modifier has no prefix, and seems unsupported with the object syntax. Is there any workaround for now?

There doesn't seem to be an event amp key modifier for .native, which is what I need.

I need .prevent

This is now documented here:

v-on also supports binding to an object of event/listener pairs without an argument. Note when using the object syntax, it does not support any modifiers.

This means { 'keyup.left': onLeftRelease } doesn't work. The alternative is obviously to handle this the old school way - the native JS equivalents of some modifiers are described here (Event Modifiers section)
On one hand I know v-on object syntax is rarely used, however it's surprising this doesn't exist. As rationale, Vue is doing something similar with multiple v-models no? Seeing the magic behavior of emitting update:propName sort of has the same vibe as passing the magic string keyup.left to v-on. This probably reads funny but hopefully you can see where I'm coming from.

I previously ran into this issue (you can see the PR where I linked here in the feed above about a month ago) and then today I ran into this same problem again. Not content with not having a solution, I kept digging around, and discovered that you can use withModifiers (you can import from @vue/runtime-dom, though it is marked as @private :shrug:). You can see it working on this codesandbox:
https://codesandbox.io/s/nice-brown-j4q94s?file=/src/components/HelloWorld.vue:2364-2565
In fact, I randomly tried to get it working with multiple different listeners (because I wanted to be able to bind both a capture and non-capture event listener) and it totally works.
I have no idea if this is documented anywhere. I'd searched the Vue documentation so carefully before for some solution to this problem. I have also not looked into how this is working in the Vue source code to tell whether this is working by accident or by design (or if there are any significant drawbacks to using this method.
Hope this helps someone else too :grinning:

This problem still at vue3.
v-on supports binding to an object but it does not support any modifiers.

I ended up creating a wrapper component like this:
<template>
  <router-link v-bind="$attrs" v-on="$listeners" @click.native="onClick">
    <slot />
  </router-link>
</template>

For anyone coming here from Google, there seems to be an builtin way to do this in Vue 3 now: https://vuejs.org/api/render-function.html#withmodifiers


Generate View For Elements - "Add all available connections" mystery option

When using the "Generate View For..." feature selecting "Add all available connections" doesn't seem to do anything.
These are the line in the code (GenerateViewCommand.java):
// Don't add connections that are not connected to the main elements if option is set
if(!fAddAllConnections && !(fSelectedElements.contains(elementSource)) && !(fSelectedElements.contains(elementTarget))) {
   continue;
}

There might be a mistake in that one line so it should be:
if(!fAddAllConnections && !(fSelectedElements.contains(elementSource) && fSelectedElements.contains(elementTarget)))

But, to be honest, I can't remember why we would want such an option. Maybe it could be removed.
See https://forum.archimatetool.com/index.php?topic=799

It does do something if you select different elements for the "Generate View" command.
For example, given this:

If you select elements A and D and do a "Generate View" with the option deselected you get:

If you select elements A and D with the option selected you get:

I don't understand because:

I sometime use this option
As far as I can tell, it works

@jbsarrodie Yes, it does. I just couldn't remember...

When in doubt, ask an Archi user you know ;-)


Reassess community taxa after taxon activation

As reported here https://forum.inaturalist.org/t/obs-with-only-subspecies-ids-can-get-stuck-without-a-cid/15992
The CID algorithm for an observation ignores inactive taxa
But if an taxon is activated the CID of its observations are not reassessed
This means that if a curator does the following set of actions which is common in curating taxa: (1) inactivates a taxon and moves it to a new taxon (this wipes the CIDs because moving a taxon reassesses CID and inactive taxa are ignored)
(2) reactivates the taxon
the CID's are not reassessed/restored because this doesn't happen when a taxon is activated
This pull request introduces a callback that reassesses the CIDs of obs associated with a taxon if the taxon is_active status changes

@kueda - if you can give this another look and/or merge thank you

Any chance this was at the root of #2566?


Expansion panels in tabs not working properly

Bug, feature request, or proposal:
Bug
What is the expected behavior?
When I switch between tabs that both have expansion panels, I want the expansion panels to behave like they would in any other situation.
What is the current behavior?
The panels are sort of opened, and their style is messed up :

What are the steps to reproduce?
Stackblitz
Is there anything else we should know?
Unlike #5269 there's no hidden tab. So I think it isn't a duplicate, hence the new issue.

I have exactly the same problem as you do, and I also have looked at #5269 . The solutions there do apply to our case (you wrap the accordion/expansion panel with an *ngIf and only show if the tab is selected.
This did not work for me.
Here's some screenshots that may help:
Expected behavior (without tab switching):

Appearance after switching tabs:

If you're looking for a workaround, you can add this :
 (selectedTabChange)="reloadTabs()"

With a tab condition
<mat-tab label="Tab One">
  <ng-container *ngIf="showTabContent">

And it does this
reloadTabs() {
  this.showTabContent = false;
  setTimeout(() => this.showTabContent = true, 500);
}

Works well for me (once I added a fake loading bar and some animation), but still, it's a workaround, and I expect a fix to come soon

Thanks for the fix @trichetriche .
I also expect there will be a solution for this.

The only issue I'm finding with this workaround is that the screen will scroll up to line up with the expanded content when you switch tabs.

Not sure this is related to the workaround since I faced this issue when I discovered material. My solution was to make the content take the whole space anyway, and change its overflow properties

With the submission of #8921, you should now be able to lazy load the content of the tabs.  With the content of the tabs being lazy loaded, the animations move to the correct state as desired.

But what if I don't want to lazy load the content of the tabs ? Is it the only solution to this problem ?

At this point this is the only viable solution we have found as this is actually an issue of animations failing to trigger.
We hope to be able to determine a better solution in the future, but at this point this is the best solution we have found.

@trichetriche The solution given is not working for me.
My Problem:
When I change the tab to second one, expansion-panel's header is taking large height than necessary and arrow is down as if content is closed but content is visible. Same case with all expansion-panels headers and content in the second tab.

That's the exact same issue I have reported and I gave a workaround around that, that worked for me (and still works as of today).
If you have an issue, then provide a sandbox to show it, like you would open a question. In fact, if you think your problem is different, you should maybe open a question.

I don't really care about your code ... That doesn't show me the issue you're facing, and that doesn't show me if you tried the workaround I gave. And honestly, i didn't even read it, there's like 200 lines of your code that doesn't interest me, I've already got my answer, and that's a ticket for an issue, not for calling help.
So unless you provide a sandbox to test it, I won't be willing to help you, sorry.

I can understand, well right now I can't reveal the real code and show you
the exact issue I am facing via sandbox. You too don't have time and
unfortunately me too. I hoped you would be able recreate the issue with
that code and help me. Anyways thank you for your words.
On 6:56PM, Fri, 20 Apr 2018 trichetriche<EMAIL_ADDRESS>wrote:

I don't really care about your code ... That doesn't show me the issue
you're facing, and that doesn't show me if you tried the workaround I gave.
And honestly, i didn't even read it, there's like 200 lines of your code
that doesn't interest me, I've already got my answer, and that's a ticket
for an issue, not for calling help.
So unless you provide a sandbox to test it, I won't be willing to help
you, sorry.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/angular/material2/issues/9834#issuecomment-383095252,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjS-xuVGSZdzfW7HzEFECYUDvS_Mx3f-ks5tqeIigaJpZM4R-EiC
.

I too faced this issue. A big thanks to @trichetriche for the work around! Made my life easier.

issue is still there even,
https://github.com/angular/material2/issues/5269

Your link is dead and we are aware the issue is still there. As said before, it won't be corrected, but there is 2 workarounds for it.

May be those 2 workarounds doesn't work for everyone. If it is in material
and is public to use then it should be developed in such a way that
everybody can use it without any issue.
Thanks & Regards
Yaseen Nadaf
Full Stack Web Developer
On Tue, Jun 12, 2018 at 1:00 PM, trichetriche<EMAIL_ADDRESS>wrote:

Your link is dead and we are aware the issue is still there. As said
before, it won't be corrected, but there is 2 workarounds for it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/angular/material2/issues/9834#issuecomment-396493661,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjS-xk4JP2GeI8icUBMmlzFIEBnDk7tBks5t724YgaJpZM4R-EiC
.

The purpose of a workaround is to work. If it doesn't, that's not a workaround, it's a bug ...
And in case you're not aware, devs can face limitations. If you cared to read the whole thread you would have seen that it is the case there.
And again, read the thread to answer your second question.

@trichetriche I re-pasted the link. Workarounds take additional time to implement and cost dev time, I rather not be spending time implementing a workaround, therefore it should be fixed. either ways the workaround does work.

@trichetriche  Workaround works great. Thanks! Took no time at all to implement and works as expected

put selectedtabchange in mat tab group
"<mat-tab-group (selectedTabChange)="tabClick($event)">"

create a variable and the function

then put the ng if in accordion elements for each tab


Add typecheck linter for typescript

Solves #20

Linters name: typecheck
Required npm package: ts-type-checker(https://www.npmjs.com/package/ts-type-checker)

It uses my simple wrapper around typescript compiler . npm i -g ts-type-checker would be enough.
typecheck.vim is almost a copy of tslint.vim.

I think this is good. I'll let Travis CI look at this. We might find that it could be a bit slow, but it's certainly better than nothing. If it delays other checks, you can always turn it off. If someone can implement something using HTTP and the test server, I'd also accept that.


fix: Fix time conversion bug when setting session.Options.MaxAge

In sessions.state.CreateState(), the maxAge for the session was being set using:
session.Options.MaxAge = int(20 * time.Minute)
This is problematic because MaxAge is an int that represents seconds [1] ; int(20 * time.Minute) gets translated to 1,200,000,000,000 seconds.
Illustration:
//=== BUGGY CODE
maxAge := int(20 * time.Minute)

// 1,200,000,000,000. This is an integer representing nanoseconds.
fmt.Println(maxAge)

redisExpirationDuration := time.Duration(maxAge) * time.Second

// 267120h 53m 28.87914496s
// This gets converted to 961,635,208,879 ms by redis client code,
// which is approximately 961 million seconds or 30 years
fmt.Println(redisExpirationDuration)

//=== FIXED CODE
maxAge = int((20 * time.Minute).Seconds())

// 1200 (matches expectation)
fmt.Println(maxAge)

redisExpirationDuration = time.Duration(maxAge) * time.Second

// 20m 0s (matches expectation)
fmt.Println(redisExpirationDuration)

Other notes:

MaxAge is to do with the Max-Age attribute in session cookies; however the underlying redisstore code uses MaxAge to set the TTL on the Redis keys as well
See logs from redis (DEV) showing how the redisstore client was setting an expiry via PX<PHONE_NUMBER>79.

[1]<EMAIL_ADDRESS>[2] https://pb.infra.corp.arista.io/q21C9RnB

I don't fully understand the code context so there is some possibility that "fixing" this bug (ie reducing Redis key expiration to actually 20 minutes as opposed to INF) results in some unintended consequences. But it should be fine I think: the oidc_state:XYZ key is stored in Redis when the OIDC browser authorization flow is initialized. So I think the 20 minutes here mean that the user has 20 minutes to go through the browser flow, which is more than reasonable.

FWIW, From scanning the other occurrences of MaxAge in the codebase, it appears that in every other instance that MaxAge is referenced, we are explicitly setting it in the unit of seconds. So I think this is clearly a typo the oidc source and that this is the correct place to do the unit conversion.

Tested this by deploying oidc-authservice in DEV cluster:
> kd -n internal-proxy get deployment oidc-auth -o yaml | grep image
        image: docker.corp.arista.io/oidc-authservice:v1.15-engprod
        imagePullPolicy: IfNotPresent

Then went into redis pod in DEV cluster:
> kd -n internal-proxy exec -it rfr-redis-oidc-auth-2 -- /bin/sh

>> redis-cli -a <PASSWORD>

>>> SCAN 0 count 50 match "oidc_state*"
1) "0"
2) 1) "oidc_state:SOME_TOKEN1"
   2) "oidc_state:SOME_TOKEN2"

>>> TTL oidc_state:SOME_TOKEN1
1187 (19 minutes ish, as expected)

In INFRA cluster, the TTL oidc_state:SOME_TOKEN1 results in an outlandishly large number like 955258654 (30 years).
That is, the bugfix works as expected. Nothing seems to be broken in DEV cluster either (did some basic tests).


orchestrator: create a readme + create a mermaid visualization charting the logic of the code

orchestrator: create a readme + create a mermaid visualization charting the logic of the code

The overall orchestrator process is mapped out in mermaid on the main page: https://github.com/open-austin/indigent-defense-stats


net: update documentation, example for error values

Suggested in a review of net: fix inconsistent errors (https://go-review.googlesource.com/#/c/9236/):
In a different CL, could you write some documentation about what kinds of errors people can expect, and perhaps an Example function showing how to work with them? See https://groups.google.com/d/msg/golang-nuts/iNVNBXdZK8Q/irnIrgLD2q4J for the kinds of issues programmers want to understand.

@mikioh Is there more to do here?

I think there's nothing else remains to be done, CLs for docs update in Go 1.9 fixed this issue. Closing.


Modal animation broken.

This issue is related to Issue #203.
The problem is not the delay itself, but rather that the animation that is supposed to happen during that delay is broken. This can be seen on angular-foundation's own plunker: http://plnkr.co/edit/RIsBM1uGbYMwUVTzNtN3?p=preview
The modal does not animate in or out. It shows instantly with no animation, and then closes in two ugly steps with no animation.

As noted in the animation section of the docs, Foundation 5 supplies animations via Javascript, not CSS. To animate the modal, you'll need to supply your own animations via CSS. You can see the animations we use for the demo site here: https://github.com/pineconellc/angular-foundation/blob/master/misc/demo/assets/demo.css#L323-L345

@jbrowning, I see, thanks for the explanation, makes sense. However, should you not consider providing that CSS with this library? Since templates are already provided I feel like providing the basic CSS to make it "work out of the box" would be great too. And shouldn't be difficult since you already have it.


simplify CAS logic a bit

This was bugging me slightly.

Nice, can't see anything wrong with this on my end. I'm gonna assume this is all good and merge it in, thanks for the cleanup :)


feat(lightbox-media-viewer): introduce image viewer

Related Ticket(s)
Refs #3550.
Description
Introduces <dds-lightbox-image-viewer>, the image version of lightbox media viewer content. Also:

Adds <dds-lightbox-media-viewer-body> feature to <dds-lightbox-video-player> and merges them. The original code for <dds-lightbox-media-viewer-body> has become an abstract class for <dds-lightbox-video-player> as well as the new <dds-lightbox-image-viewer>.
Removes hide-caption attribute from <dds-lightbox-video-player>, given it seems for standalone version only.

Changelog
New

<dds-lightbox-image-viewer>, the image version of lightbox media viewer content.

Changed

A change to merge <dds-lightbox-media-viewer-body> feature to <dds-lightbox-video-player>.

Removed

hide-caption attribute from <dds-lightbox-video-player>, given it seems for standalone version only.

@asudoh can you resolve the merge conflicts?

@jeffchew Done!


Support https and how to config?

Describe the bug
By default, Pynecone runs in http mode. No related info found in Docs, how to set it up?
To Reproduce
Steps to reproduce the behavior:

Code/Link to Repo:

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Specifics (please complete the following information):

Python Version:
Reflex Version:
OS:
Browser (Optional):

Additional context
Add any other context about the problem here.

https is supported by default. setting your api_url in rxconfig.py to an https url should work out of the box with no special configuration

I have no Nginx or something similar to setup cert or key, just only Reflex/Pynecone. Where should I put the cert and key files? I want both of frontend and backend are https enabled.

@evandeng2009 while i believe uvicorn and probably next itself support TLS certs, etc natively, reflex doesn't currently expose it.
If you're looking for something simple to terminate TLS, then I'd recommend taking a look at caddy with automatic HTTPS.
You can get started with a Caddyfile in your app directory like so:
localhost
reverse_proxy localhost:3000
reverse_proxy /event/ localhost:8000

(change localhost on the first line to whatever domain you own, and it will provision TLS via letsencrypt automatically for you)
Execute caddy run, then fire up https://localhost to see your reflex site.

Thank you @masenf , Caddy is a good tool for auto https. With reverse_proxy /event/ localhost:8000 and api_url=https://domain_only_without_specified_backend_port,  users will see 80/http and 443/https only and this is what I want.
Caddy is more easy to use than other proxies like Ghostunnel as Caddy itself help get and renew the cert and key.


Optimize container image build

Changes

poetry with dev and product dependencies classified - WIP
create wheel package for ambuda
use python slim image as base image and adopt docker multistage build approach
remove make and other build tools, as much as possible, from runtime

Next

Build prod a lighter image
Prune unused files and build-time caches
Arrive at a small essential and sustainable base image

Test plan

Pull docker image

$ docker pull ghcr.io/kvchitrapu/ambuda:ambuda-ci
$ $ docker images
REPOSITORY                  TAG                  IMAGE ID       CREATED          SIZE
ghcr.io/kvchitrapu/ambuda   ambuda-ci            ec5cc9de03a9   25 minutes ago   858MB

Update docker-compose.yml

~/src/ambuda-kvic$ git diff docker-compose.yml
diff --git a/docker-compose.yml b/docker-compose.yml
index 31f22a9..bcc90e6 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -2,7 +2,7 @@ version: '3.4'

 services:
   app:
-    build: .
+    image: ghcr.io/kvchitrapu/ambuda:ambuda-ci
     tty: true
     volumes:
       - .:/app
@@ -20,7 +20,7 @@ services:
       - AMBUDA_BOT_PASSWORD=insecure bot password

   celery:
-    build: .
+    image: ghcr.io/kvchitrapu/ambuda:ambuda-ci
     command: /venv/bin/celery -A ambuda.tasks worker --loglevel=INFO
     volumes:

Run

On Linux, run  docker-compose up
On Mac, run docker compose up

Result

app_1     | [1]
app_1     | [1] Rebuilding...
app_1     | [0]  * [OK] Ambuda database check has passed.
app_1     | [0] [2022-10-10 12:31:56,102] INFO in _internal:  * Running on all addresses (<IP_ADDRESS>)
app_1     | [0]    WARNING: This is a development server. Do not use it in a production deployment.
app_1     | [0]  * Running on http://<IP_ADDRESS>:5000
app_1     | [0]  * Running on http://<IP_ADDRESS>:5000 (Press CTRL+C to quit)
app_1     | [0] [2022-10-10 12:31:56,117] INFO in _internal:  * Restarting with watchdog (inotify)

Note
Not sure why GH picks up my commits all the way from August. Looking to move

Just took a first look -- looks fine, but I also haven't used Docker extensively, so when I see mention of using four separate images (ambuda-base, ambuda-build-deps, ambuda-prod, ambuda-dev) I get nervous about what this means for overall system complexity.
I'll patch this PR and give it a try for dev on my machine tomorrow. If it's tolerable, we should land then iterate from there.
@epicfaace and @shreevatsa I think you both have more Docker experience than me -- how do you feel about these changes?

Just took a first look -- looks fine, but I also haven't used Docker extensively, so when I see mention of using four separate images (ambuda-base, ambuda-build-deps, ambuda-prod, ambuda-dev) I get nervous about what this means for overall system complexity.
I'll patch this PR and give it a try for dev on my machine tomorrow. If it's tolerable, we should land then iterate from there.
@epicfaace and @shreevatsa I think you both have more Docker experience than me -- how do you feel about these changes?

Package management and the build seem complex as they pull hundreds of dependencies and involve several steps. The general idea here is to clearly mark the intermediate images and the final image to arrive at a rapid building slim production image.
Input

Packages (poetry categorized into app dependencies and dev tools;)
Code
Build steps (Dockerfile has the shortest and fastest path for your dev or prod purposes)

Process
2. Use pre-built base images like 3.10.8-slim-buster with our own packages
3. Use pre-built dependencies from ambuda-build-deps.
4. Above two images are just internal states in the build system. Those images are occasionally built to upgrade packages like Python, selenium, etc.
5. Docker multistaging approach adds only essential files. Keeps the container image thin.
6. Build flag picks the course of the build system to generate dev or prod images.
Output
7. ambuda-prod - contains minimal packages necessary to run production code on DO
8. ambuda-dev - contains all tools to develop and test code on laptop
A developer only interacts with ambuda-dev image.

LGTM!


Validation loss again

Based on PR #914 "Add validation loss" by @rockerBOO works, this proposal aims to integrate the feature into the latest version with minimal modifications to the existing functionality.
This update revises the algorithm for calculating validation loss, utilizing the sum and average of fixed timesteps at [10, 350, 500, 650, 990], which requires more computational time, but should give more accurate results for diffusion models.

Additionally, two arguments have been introduced to facilitate the use with larger datasets:
--validation_every_n_step specifies the number of train steps after which to calculate the validation loss. By default, the validation loss is calculated once per train epoch.
--max_validation_steps determines the number of max validation steps used in each validation loss calculation. By default and maximum, it utilizes the entire validation dataset.
Currently, this feature is supported in train_network.py, with the possibility of extending support as needed.
Recommended for small datasets:
--validation_split=0.1 --validation_seed=1234 --validation_every_n_step=50 --max_validation_steps=5
This will approximately increase the training time by 10% and yield reliable validation results.
For larger datasets, it is advisable to increase the max_validation_steps for more representative outcomes.

Hey! I'd like to really thank you for this feature, it's absolutely incredible. However, I have a question that came up regarding the Text Encoder.
I understood the principle that, depending on the training, at some point it starts to overfit and the validation loss begins to increase. But what causes more problems in inference is the overfit in the Text Encoder, so much so that when I use my LoRAs, I reduce the weight during inference.
That said, I believe that the loss of the validation steps are also affected because the TE is frying/fried, while the Unet is not.
Is there a possibility to reduce the weight of the LoRA/Unet/Text-Encoder in these situations/steps/samples?

That said, I believe that the loss of the validation steps are also affected because the TE is frying/fried, while the Unet is not.

Part of the issue is the TE is presented as cross attention to the UNet, so used in concert. Quantifying the TE would be a little more complex than just seeing the validation loss. Would need to have some comparison of the image CLIP value and the caption CLIP (TE) value and compare the similarity, possibly. I'm not sure how well this would ultimately line up or if it would be valuable to achieve though.

Adjusting is somewhat difficult for me because I'm unsure how to properly adjust the magnitude of TE and UNet for the different LoRA formats. Perhaps someone knowledgeable could provide a method.

Not about validation but in relation to the TE overfit - the DB training method has the "Stop text encoder training" option which isn't available to TI and Lora. If it is technically possible to add that option to Lora training, may it be able to help with the TE overfit, perhaps? You could at least experiment by stopping TE training at different points.

It seems someone is working on this (PR #1011), but I'm not quite clear on the current progress.

I got it, I even realized some things in the trainings that indicate that an eventual adjustment in the LoRA weight wouldn't help with my 'problem'.
But now I'm going to bother you again, now with something more useful and related to the PR.
I've come across a problem with the validation system automatically creating the dataset in question, because I can't use data augmentation along with the current validation system, as the same image, but with some subtle differences, could end up going to training and another to validation.
What would be the feasibility of, just like it's possible to define a 'reg' directory for regularization images, creating a 'val' for validation? That way, I would manually create the validation.

I got it, I even realized some things in the trainings that indicate that an eventual adjustment in the LoRA weight wouldn't help with my 'problem'.
But now I'm going to bother you again, now with something more useful and related to the PR.
I've come across a problem with the validation system automatically creating the dataset in question, because I can't use data augmentation along with the current validation system, as the same image, but with some subtle differences, could end up going to training and another to validation.
What would be the feasibility of, just like it's possible to define a 'reg' directory for regularization images, creating a 'val' for validation? That way, I would manually create the validation.

That's a good question. Previously, I hadn't considered the issue of data augmentation.
Perhaps you could place the data that will be augmented in the reg dataset, since the current val dataset will ignore the reg dataset.
Maybe I can create a specific exclusion condition for the val dataset dir name, to prevent it from entering the train dataset.
Otherwise, exclude the same image paths.

I think I'm gonna adopt a workaround with your suggestion of using the reg folder, but I'm gonna put all the training in the reg folder, and in the train folder I'll put the images I want to use for validation, then I'll set --validation_split=1. 🤔

I got it, I even realized some things in the trainings that indicate that an eventual adjustment in the LoRA weight wouldn't help with my 'problem'.
But now I'm going to bother you again, now with something more useful and related to the PR.
I've come across a problem with the validation system automatically creating the dataset in question, because I can't use data augmentation along with the current validation system, as the same image, but with some subtle differences, could end up going to training and another to validation.
What would be the feasibility of, just like it's possible to define a 'reg' directory for regularization images, creating a 'val' for validation? That way, I would manually create the validation.

That's a good question. Previously, I hadn't considered the issue of data augmentation.
Perhaps you could place the data that will be augmented in the reg dataset, since the current val dataset will ignore the reg dataset.
Maybe I can create a specific exclusion condition for the val dataset dir name, to prevent it from entering the train dataset.
Otherwise, exclude the same image paths.

I think I'm gonna adopt a workaround with your suggestion of using the reg folder, but I'm gonna put all the training in the reg folder, and in the train folder I'll put the images I want to use for validation, then I'll set --validation_split=1. 🤔

After re-reading the code, I realized that the same image paths being divided into training or validation shouldn't happen because the distribution of image paths is done before multiplying by num_repeats. However, I found that for the val_dataset, setting num_repeats=1 and canceling caption_dropout can make the val_loss calculation more consistent.
Additionally, one issue observed is that the val loss behaves abnormally after resuming training. I suspect this is due to the distribution of the val_dataset changing after resuming. (It's also possible that the resume function itself has issues. Who knows?)

Additionally, one issue observed is that the val loss behaves abnormally after resuming training. I suspect this is due to the distribution of the val_dataset changing after resuming. (It's also possible that the resume function itself has issues. Who knows?)

Would make sure you're using the same seed. Validation also has it's own seed too which should keep it consistent (otherwise uses the seed for the run)