Add new "overview" field in both app.yaml and resource.yaml

What problem are you facing?
Currently there is only a description field as a part of app.yaml and resource.yaml.
How could Crossplane help solve your problem?
It would be useful to have an additional field called overview as a part of app.yaml and resource.yaml that you could use to add in a summary/descriptive text area about the stack or resource respectively. This would be different from the description which is specifically tailored as a delivery system for the README of the stack or resource. Instead this could be used as more of a marketing, human readable intro describing what this stack or resource is.

I don't see overview as being needed as it currently stands because description can simply be a rich markdown of sections and anything needed to describe the application.

Let's sync up @lukeweber @rathpc on what is needed to be added to the Stacks package format spec in v0.3.

I will throw something on our calendars @jbw976

@jbw976 As per our recent meeting, here are the takeaways:
We will be modifying the spec to reflect the following:
Within app.yaml

The description field will be renamed readme.
A new field, overview will be created which should accept plain text only.
A new field overviewShort will be created which should accept plain text only.

Within resource.yaml

The description field will be renamed readme.
A new field, overview will be created which should accept plain text only.
A new field overviewShort will be created which should accept plain text only.

@rathpc group.yaml follows the same conventions and should be updated too.

There may be an advantage to using a README.md in the leaf directories.  We may need to use README-kind.md (or similar) to disambiguate between crd files being described.  This would also incur ambiguity between group and resource README files.
The advantage would be that stack creators do not have to work on markdown within yaml indentations.  They can use featureful markdown editors.

I think that could make sense to me, and I have no problem with a file versus a field as long as it ends up being parsed into an annotation the same way a field in the YAML would have.


MM-41271 : Don't render sidebar channels when collapsed

Summary
Collapsed menu in sidebar are removed from the DOM to improve performance.
Ticket Link
https://mattermost.atlassian.net/browse/MM-41271
Related Pull Requests
None
Screenshots
https://user-images.githubusercontent.com/17708702/152343167-f1ba5c48-022d-4eb3-b73e-c11d8ce7038d.mov
Release Note

Removes collapsed sidebar menu from the dom on sidebar collapse and vise versa on sidebar expand

@M-ZubairAhmed @hmhealey Will this have a negative perceived performance impact when a user expands a channel category?

No, it shouldn't. We expect this to be a decent performance improvement for anyone with a lot of channels in their sidebar who keeps any categories collapsed though.

/e2e-tests

Triggering e2e testing with options:
<nil>

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/148484

/update-branch

triggering e2e again after fixing related e2e tests

/e2e-test

Triggering e2e testing with options:
<nil>

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/148517

/update-branch

/e2e-test

/e2e-test

Triggering e2e testing with options:
<nil>

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/148713

@M-ZubairAhmed
Tested manually - no issues. Verified sidebar collapse and expand is working as expected. Also, new messages in collapsed categories are received as expected, as is the sorting to Unreads from collapsed categories.
E2E report has a failure that seems related to this PR MM-T2003 Can you please take a look? Thanks

@jgilliam17 it seems like that is the old report, i have fixed all the related e2es for this pr, here is the latest report https://automation-dashboard.vercel.app/cycles/1455f4cf-3731-4819-af4d-4e986cfca0a4

@M-ZubairAhmed Thanks for the latest report. Yes, everything looks good, no PR related failures. Good to merge.

/cherry-pick release-6.3

Cherry pick is scheduled.

/cherry-pick release-6.3

Cherry pick is scheduled.


Dead branches

There are some dead branches that could be removed, such as :-
bnb/add-brian
bnb/add-feross
bnb/add-kyle
bnb/add-thlorenz
bnb/add-yosh
revert-74-master
Also, the branches for the PR's I have raised this morning (after they have been merged).

I'm not really concerned with dead branches, but I wouldn't be opposed to them being cleaned up. Perhaps @bnb can raise your permissions to be able to modify the repo.
@bnb, @surlydev has made some good contributions and would make a good additional maintainer, IMO.

Per your recommendation @mblayman, I've added @surlydev as a collaborator.

Thank you for your vote of confidence guys.
On Fri, 5 Jul 2019 at 17:52, Tierney Cyren<EMAIL_ADDRESS>wrote:

Per your recommendation @mblayman https://github.com/mblayman, I've
added @surlydev https://github.com/surlydev as a collaborator.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bnb/awesome-developer-streams/issues/178?email_source=notifications&email_token=AAGXAHYGSQKVC6QJSVSAAR3P5536BA5CNFSM4H4EIYA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZJ6K6Y#issuecomment-508814715,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGXAH3GIDLNXQ6242Y7SZLP5536BANCNFSM4H4EIYAQ
.

Also: feel free to delete any dead branches! I have no attachment to them ❤

Cleaned them up.
FYI the command was ... git push upstream :{old-branch-name}
Yes, the colon was required. Thanks to AdmiringWorm on Twitch.

You may also delete multiple branches at the same time if you need that in the future.
git push upstream :{branch-1} :{branch-2} :{branch-3}

You may not need that now, but just letting you know.

You can also set up your repo to automatically delete branches after merging a PR: https://help.github.com/en/github/administering-a-repository/managing-the-automatic-deletion-of-branches

@ChaelCodes I've turned that on 👍

Yay! I'm glad you liked the suggestion.


Make agent sharding configurable

Agent can now be sharded in the latest PMO https://github.com/giantswarm/prometheus-meta-operator/pull/1292 and the capabilities are built into the observability-bundle 0.5.0


Fix specs when missing plugins

I missed that I broke some specs in a previous review. This fixes the specs and adds in .travis.yml so that doesn't happen again.

LGTM™


Resize unsplash image width query to reduce download for smaller devices

Issue Summary
This issue is referencing the wrongfully closed issue #12245 that still persists. As the Unsplash feature is integrated by default, one would expect that the images are served responsively. The Unsplash API supports such queries, but Ghost returns the images independent of device screen size.
Given that images that are too big are impacting UX as well as SEO, this is a critical feature for users that use the Unsplash feature.
Current workaround is to download the image and upload it on ghost.
Steps to Reproduce

Create article
Use any Unsplash image as feature image
Load page through mobile device or dev tools of your browser
Inspect the served image (should have a width of 2000)

Ghost Version
4.39.
Node.js Version
16.14.0
How did you install Ghost?
Docker Container
Database type
MySQL 8
Browser & OS version
No response
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Fun fact:
While you report the performance improvements for v5, for most users that use the Unsplash feature, implementing this quick change in the query of the image will have a far bigger impact on performance for them! And there is already a PR that implemented this, so this is quite the easy fix.


no-extra-parens jsxIgnore multi-line equivalent?

The airbnb docs have a comment for the no-extra-parens rule, citing jsxIgnore should be ignored there, and react/jsx-wrap-multlines should be used for handling JSX. However that only seems to cover multi-line JSX as the rule name implies.
no-extra-parens can allow for parentheses around multi-line JSX, while also detecting and avoiding it for single line JSX, as per the airbnb guide advice examples. Prettier likewise formats JSX in the same manner with parens for multi-line, and stripping them if used in single line JSX return.
Is this a rule that should be catered for by this plugin? Or should no-extra-parens be used?
Related issue on the airbnb JS style guide repo.

It seems like the gap you're referring to is that the airbnb guide recommends single-line jsx being wrapped in parens, but there's no linter rule in this plugin to cover that.
The intention of the airbnb guide is that the developer should be able to choose whether to use, or omit, parens around single-line jsx. Prettier destroys that choice by always stripping them. Do you have a use case to always include them? (note that the airbnb guide does not mandate this)

It seems like the gap you're referring to is that the airbnb guide recommends single-line jsx being wrapped in parens, but there's no linter rule in this plugin to cover that.

The opposite of that is recommended by airbnb(no parentheses):
// good, when single line
render() {
  const body =
hello
;
  return <MyComponent>{body}</MyComponent>;
}

This is the same as what Prettier does. My issue is that the general JS guide by airbnb provides a eslint config that defers the JSX option of no-extra-parens rule to this react plugin instead, however the plugin cannot reproduce the same behavior and autofix that the no-extra-parens rule is capable of.

or omit, parens around single-line jsx.

This is not an option via this plugin, it either needs to support that or the config for no-extra-parens rule in the main airbnb JS config should not advise this plugin over that rules option for handling JSX.

Do you have a use case to always include them? (note that the airbnb guide does not mandate this)

I exclude them from single line if they're not necessary with JSX, but error for multi-line, both can be auto fixed with eslint. I don't use Prettier personally as it often formats code in an undesirable way that cannot be avoided without noisy disable comments.
It may not be mandated on how to handle parentheses with JSX or not, but the link to the guide references what airbnb considers good style for both cases.

Either this plugin should support the equivalent of:
"no-extra-parens": ["error", "all", { ignoreJSX: "multi-line" }],

Or this comment  should be modified or removed:
ignoreJSX: 'all', // delegate to eslint-plugin-react

no-parens "good, when single line" does not imply that with parens would be bad.

If the airbnb config modified its no-extra-parens config to only ignore multiline jsx, it would be forbidding parens on single-line jsx - which is GOOD and ALLOWED, just not forced by the guide. This is because the guide intentionally leaves it up to the individual developer to decide if single-line JSX should have parens or not - the tooling should not get an opinion on that.

That's fine, but it should have a comment about that, rather than ignoring the rule for JSX with a comment that delegates responsibility to this plugin when this plugin cannot handle the single line behaviour in the same manner, only multi-line.
The comment could instead advise to use the related rule provided by the react plugin when only interested in multi-line linting, and ignoring consistency in single-line JSX handling.
At the very least, it should be clear which rule in the react plugin it's deferring configuration to, and that rule should be clear about not supporting single-line(it roughly is via the rule name, but as a user it's not clear how to handle the single-line case while trusting the advice from the airbnb rule comment to ignore JSX for no-extra-parens.

I'm not really sure what you're looking for here. The airbnb config's choice has nothing to do with this repo. The comments inside the airbnb config are for its maintainers (ie, me) and not for consumers. Separately, even if eslint-plugin-react could handle the single-line form, the airbnb config would not enable that - the desired behavior is that single-line jsx parens are not linted whatsoever, in either direction.
If there's something actionable in this repo, I'm happy to reopen - otherwise I think this can be closed.

The comments inside the airbnb config are for its maintainers (ie, me) and not for consumers.

Is that mentioned anywhere? As a consumer it seems a bit misleading to mention the functionality is handled by the react plugin, when it's not 1:1 parity in functionality.

the airbnb config would not enable that - the desired behavior is that single-line jsx parens are not linted whatsoever, in either direction.

Ok.. no problem I can just use the non-react plugin rule as mentioned without deferring to the plugin to get the consistent single line style behavior.
Probably should remove the "good" example that was linked earlier from JSX no-parens example, as that seems rather misleading too (as in if the other style isn't "bad", it's unclear how that is "good"), although from what I understand, there is some line where something is just advised as good style, but no enforcement/support from config, even when that is possible...? (I think that's not uncommon for consumers)
Cheers.

It doesn’t say that - it says “delegated”.


STREAMLINE-727 bootstrap-stroage script should allow create only and …

…keep deletion as an option

bootstrap-storage.sh receives one optional argument

default value is "create"
"create": create tables
"drop": drop tables
"drop-create": drop then create tables
"check-connection": only check connection

create new class StreamlineTableInitializer to keep SQLScriptRunner usable for general case

new class will handle only the case what bootstrap-storage provides
refactor SQLScriptRunner to let new class uses features

Regarding concurrent execution, since we're using IF NOT EXISTS for creating table and IF EXISTS for dropping table so it should be safe.

@satishd
I might be misunderstanding your comment. Where do you think mysql specific logic should be placed?
Please let me know that you're thinking about moving out mysql lib download to the bootstrap script.

@HeartSaVioR Respective storage type scripts like mysql and postgres can have their own scripts (bootstrap-storage-<mysql/postgres>.sh) which can share common functionality from other script like bootstrap-storage-common.sh.

@satishd @HeartSaVioR we are waiting on this to unblock ambari changes and QE. I added my comments to make this generic. Let me know if you've any questions regarding this.

@satishd @harshach
I feel we need to arrange this. Let's look at the history of change.

For the first time, this was implemented to two classes: config reader / sql runner. (IOT-441, #265)
Config reader reads config and figures out connection information, and sql runner runs the scripts for given dbtype.

After that we changed config reader to an utility class (hence not called by script), and utilize config reader class into sql runner. (STREAMLINE-612, #419)

We needed downloading MySQL driver because of license issue, and added the feature: download MySQL driver to sql runner. (STREAMLINE-643, #449)
Before this change it was generic.

If we want to make the thing generic, let's just exclude mysql driver download to the outside of sql runner, and maybe, even outside of bootstrap-storage. Sql runner was generic before handling mysql driver download.
(Users might need to execute separate script for MySQL for the first time. If we think it hurts UX and don't want to do that, bootstrap-storage or sql runner should have it as exceptional case and can't be generic.)
What do you think?

@HeartSaVioR mysql driver will be downloaded only when we configure the db.type to be "mysql". why we need to change that?

@harshach
What @satishd commented about moving out specific implementation is all related to mysql download. (Please look through where @satishd left comments.)
So my understanding of review comments are allowing non-generic vs generic and move out mysql driver download. It doesn't represent that I want to move out the logic, just thinking about the way to make it db-type generic. Either (keep as it is or change it) is fine for me.
@satishd Please follow up the thread and let us know your thought.

@HeartSaVioR @harshach I am fine with the change as if it is one off the case.

+1


3D bone attachment position incorrect when importing from Blender

Godot version
4.0.beta.7.mono
System information
Endeavour OS Zen Kernel 6.1 Ryzen 5900x Nvidia GTX 1060 6GB (proprietary drivers)
Issue description
When importing my character rig from Blender into Godot 4 as a GLTF the weapons which are parented to bones from the rig are placed improperly breaking the animations. This issue doesn't occur in the latest Godot 3.5 so it's likely a regression issue.
Steps to reproduce
Make an armature in Blender, add custom bones and parent meshes to said bones. When importing into Godot 4 the positions will be incorrect.
Minimal reproduction project
GithubBrokenAnimationsProject.zip

Confirmed, it seems to be caused by tracks being created for the objects assigned as children of the bone attachments being put in the wrong place. Will investigate further.

Okay, so this is actually quite a tricky bug. It would appear that the way we're handling the offset of nodes which get created with bone attachments is COMPLETELY messed up, and in fact, it's messed up in 3.x too. The only reason it works in 3.x is because for whatever reason, we don't create tracks for meshes which don't directly have explicit keyframes, whereas we create them in _optimize_track_usage function. We can probably hack our way around this to restore 3.x-like behaviour, but we probably want to completely overhaul and re-evaluate how this aspect of the import pipeline works.

I can tell you why it's doing that now and while this doesn't fix the larger issue, it will at least be a temporary workaround. Go into the advanced import settings for your glb file, click on the AnimationPlayer node and set the import track settings to this.

This does not fix the broader issue, but will at least make it behave like it did in 3.x. I consider 3.x's behaviour to be broken too, this is specific issue is a symptom of a much larger problem.

It's also worth noting that this model's import seems to be broken in Blender's GLTF importer too. I'm not sure what the correct behaviour is actually meant to be in relation to the spec.

@ZachAR3 I have determined the issue is because you are exporting from an old version of Blender with known bugs.
Please update to Blender 3.4 or later and see if that solves your issue. I'm going to close this issue, but please reopen if you continue to have trouble after exporting from 3.4 or later.
Here you can see glb exports from Blender 3.2 and from Blender 3.4:
PlayerFPS_exports.zip
Blender 3.2 shows the issue:

but Blender 3.4 does not show any issue:

Thanks for reporting the issue though. This file certainly made for a good testcase. Please continue giving the whole import pipeline a good test before 4.0 is shipped so we can catch more issues like this.

The only mystery is why "This issue doesn't occur in the latest Godot 3.5"
That slightly baffles me, given that the problem even showed up in Windows 3D Object viewer as well as reimport into Blender itself.
My only guess is perhaps Godot 3.x has another bug which somehow counteracts the bug in Blender's export, (sometimes two bug can cancel each other out) but in any case it was not conforming to the glTF spec, so it's not behavior we would be able to keep in Godot 4 and later.

The only mystery is why "This issue doesn't occur in the latest Godot 3.5"
That slightly baffles me, given that the problem even showed up in Windows 3D Object viewer as well as reimport into Blender itself. I tend to take https://gltf-viewer.donmccurdy.com/ as a source of truth and it also shows the problem with the 3.2 export but not the 3.4 export.
My only guess is perhaps Godot 3.x has another bug which somehow counteracts the bug in Blender's export, (sometimes two bug can cancel each other out) but in any case it was not conforming to the glTF spec, so it's not behavior we would be able to keep in Godot 4 and later.

Not sure, Thanks for all the help! The problem ended up being fixed by going back into blender and rekeyframing the first frame of each action, and somethings even broke in blender and I had to redo them so it was definitely partially due to a broken project.


Try to reconnect to channel if connection is lost

cchc-crawler  | time="2021-08-07T00:59:21Z" level=error msg="Error putting item in queue for metadata processing" error="Failed to publish item metadata message: Exception (504) Reason: \"channel/connection is not open\"" item_id="http://www.loc.gov/item/sn83030272/1865-11-01/ed-1/"

Also restart the message broker if it goes down


Make main thread actions sequential

As listed in issue: #16, the pomodoro main thread can handle more operations sequentially without the overhead of creating tokio threads.
It should be noted that the CREATE option has  been refactored and moved to a separate method, but it would still be quite necessary a refactor for the whole system (isolating different logics, like for example the DataBase logic has to be completely decoupled from main.rs, the parsing arguments part should be delegated to argument.rs, and so much more) but this has to be thinked and talked in another issue.
In addition, I can say that theoretically this pull request does not break any of the current functionalities of the code (since I have tested the application and it seems to work the same as before), although, as at the moment there are no unit tests, I cannot ensure that this is true for the 100% of cases.

[Request]
It seems already merged commits exist, ba6a416 and
080a0b7. Could you make commit logs clear?

[Request]
Could you run this command right under the directory and resolve warning? About Hint warning, you can ignore it
cargo clippy

The changes has been made to the branch. I had to do a "git rebase" for deleting the documentation commit and that's why in the pull request now appears:
Pablito2020 force-pushed the main-sequential branch from e9c2030 to af68323 1 minute ago

[Question]
I don't know whether you can merge yourself or not, if you can, please do it!

[Question] I don't know whether you can merge yourself or not, if you can, please do it!
I can't merge it to the repository because I don't have write access to it.

Aha! Sorry, I didn't know that. Thank you for the check!


Added CI so I can get public docker images

I really needed those public docker images, so I've copied all the scripst over from the project referenced by @ibuildthecloud in #26.
I've got no experience with dapper/drone so hopefully it just works. Commits from maintainers are allowed.

No worries, it's the least I can do to give something back!

Little bump for this PR to not go stale :)

Bump!

I'll take a look at this on Monday. The Drone runners are managed by a different team, so I will probably need their assistance to get this turned up.

@toonsevrin this is done, but master turned out to need another fix for connection pooling support in order to build properly. Also note that since this is configured as a static/nocgo build, sqlite support is not available so the kine docker images will require an external database.

Oh, that sounds like something undesirable (not supporting sqlite), anyways, thanks for the merge, hopefully the builds will pass soon ;)

Seems like images are not being published to docker hub, could this be added @brandond.

I'm also hoping to get the official images pushed at some point.

@rothgar they should be out there now. We just haven't tagged many releases lately.

I was hoping one could be manually built from an existing tag. There's nothing at https://hub.docker.com/r/rancher/kine and dockerhub is a bit of a mess to search because "kine" gets almost 800 other containers


[ServiceBus] List Sessions Support

NOTE: This currently is classified as a post-GA feature.
In track1, we have the list sessions on the Receiver which list session ids on a sessionful.
But there're issues that it may only list sessions that the state of which have been set, and it only returns session id, no other information.
track2 should investigate the issue and to see if it's possible to return other useful informations.

closing this issue now as currently there's no feature request on this one, we could always re-open the issue when needed.


Problem starting the app after the first usage

I installed it from the app store and it worked perfectly the first time I tried it.
But after completing the checklist the first time, it has failed to start, stuck with a blue box cycling through "Preparing WatsonML"
I can swipe to the other screens, for the checklist, camera, player, although the camera screen does not show an image and the "Preparing WatsonML" message persists.
I tried uninstalling and reinstalling the app as well.
Using iPhone 6s - iOS 11.3.1
I do have other apps installed using CoreML working OK.

workaround from David: if you go to the player screen and tap the leaderboard button, itll try to load the leaderboard with a loading indicator instead. going away from this screen dismisses it
That works for me.

We believe we've fixed this issue, and the app is pending an update for v1.1 in the App Store. As soon as it's available, we'll test it and re-visit this.

Fixed by #81


Support interacting with "video" html5 element via mouse click

What is your Test Scenario?
Test a video playing in the <video> element.
What are you suggesting?
Video playing should start with the mouse click. At present, nothing happens when TestCafe clicks the video on the page.
What alternatives have you considered?
document.querySelector('video').play();
Additional context
SC ticket: https://supportcenter.devexpress.com/internal/ticket/details/T1028548

TestCafe normally works with the video elements in the NativeAutomation mode.
import { Selector } from 'testcafe';

fixture `Fixture`
    .page('https://developer.mozilla.org/en-US/docs/Web/HTML/Element/video');

test('test', async t => {
	await t
	    .click(Selector('main .interactive.is-tabbed-standard-height'))
		.switchToIframe('iframe.interactive')
	    .switchToIframe('#output-iframe')
		.click('video', { offsetX: 20, offsetY: 80 })
		.wait(3000);
});

So, it's necessary to add the support of the NativeAutomation mode for TestCafe Studio.


Flask setup.py should use "Werkzeug ~= 2.2.2" instead of "Werkzeug >= 2.2.2"

The recent release of Werkzeug 3.x has affected the logic of Flask setup.py causing it to install an incompatible version of Werkzeug. A change to setup.py from >= to ~= would  remedy the situation until the community can verify Flask 3.x is at a known stable release point.
Actual:
pip install Flask==2.2.2 does not create a working install of Flask
Expected:
pip install Flask==2.2.2 should create a working install of Flask
Environment:

Python version: 3.11
Flask version: 2.2.2

You need to pin your transitive dependencies, and update them together as well. Please read one of the many issues related to this. See e.g. this comment for details.
Also, in a new project, you should use recent versions instead of older/unsupported ones.

Health care software cannot use the latest versions without going through FDA mandated verification and validation.

But you can use outdated and thus unsupported versions?! Anyway, in that case you should probably either:

pin the proper (older) version of flask, werkzeug, jinja, etc
use packages provided by your (enterprise-grade, I assume, ie RHEL) distribution instead of installing from pip

We cannot pin your dependencies because we cannot know when to move them.

That makes no sense...
At point X you install Flask (which pulls in werkzeug and jinja). So at that point you have a working set of dependencies. You pin those (ideally using a tool like pip-compile).
And you just said that newer versions need to be "verified and validated" (whatever that means in your context). So that's how you know when to update: As soon as you're allowed to use the newer versions...

We make new images constantly via CI. We do not install Flask once.
We can lock Werkzeug now that we know your stance, but this creates the opposite problem when we need to move Flask up of having to figure out which other set of libraries needs to be moved. To do that, we'd have to unpin them in our pyproject.toml and re-pin them just-in-case.

When writing an application, you must use a tool like pip-tools to pin your application's full dependency tree. This gives you reproducible deployments, allowing you to control when you get updates. Be sure to run your tests with deprecation warnings treated as errors so that you get notified of those types of changes early.
Please review any of the following for more information:

https://hynek.me/articles/semver-will-not-save-you/
https://www.youtube.com/watch?v=WSVFw-3ssXM&t
https://snarky.ca/why-i-dont-like-semver/
https://caremad.io/posts/2016/02/versioning-software/
https://bernat.tech/posts/version-numbers/
https://iscinumpy.dev/post/bound-version-constraints/


[Demo] Enhance Twitter Data Saver demo

I believe our Twitter Data Saver demo also included support for embedded videos in Tweets: https://github.com/GoogleChromeLabs/adaptive-loading/blob/master/react-twitter-save-data-loading(hook)/src/components/Tweet/Video.js
If we have time, I would like for us to enhance the current Hooks demo so that the 4th sample tweet is https://twitter.com/mrdoob/status/1131817655134896128. This would allow us to demonstrate what a LQIP/blurred image for a video entry would look like (just an image) when data saver is on and we can then load the video on tap the way Twitter's current experience works.
The video for this tweet is attached 📦
main-video2.mp4.zip
cc @anton-karlovskiy

@addyosmani
I've updated the client hint version accordingly since I thought it would take 20 minutes or so.
https://adaptive-loading.web.app/react-twitter-save-data-loading(client-hint)

@addyosmani
FYI: I think the codebase is a bit messy as I just adopted react-tweet project and did not refactor it because I thought it would take quite some time to refactor adequately.

No worries. Given this is a demo, I think that is okay. Could we update the tweet text, avatar and username for that entry to match the tweet I linked to as well? Would it be hard to also add this change to the hooks demo?

No worries. Given this is a demo, I think that is okay. Could we update the tweet text, avatar and username for that entry to match the tweet I linked to as well? Would it be hard to also add this change to the hooks demo?

I think it's not so difficult. After adding, we might have to adjust the view on desktop and mobile.

@addyosmani
The look and feel of the video tweet is updated with copy of the original appearance.
Please review this.

@addyosmani
Looks like this project does not quite fit for mobile responsive view, although I tweaked a bit in the past.
This is also mentioned in TODO: of the orignal project.
I've checked that it's okay on Sumsung Galaxy 9.

This looks perfect, Anton! Thank you!


Can't cimport mesh: pxd file broken

In https://github.com/raysect/source/blob/b7304382469e649ba0dd69c7cc39f0eb9e03eeb5/raysect/primitive/mesh/init.pxd, from raysect.primitive.mesh cimport Mesh should be from raysect.primitive.mesh.mesh cimport Mesh.

Fixed with PR #300.


docs(readme): Englishing wording changes

Made a few more tweaks to the English in the README.md.

Thanks for your PR. I made some small tweaks.


🐞 System terminal broken by CodeEdit

Description
When CodeEdit is running for a while and I try starting Terminal.app, I see this:

According to this stack overflow question, closing the problem application fixes it. I close CodeEdit, open Terminal.app and it works.
Additional Context
After trying to build CodeEdit, I get this...

Upon clicking details...
Details

Pseudo Terminal Setup Error
Domain: IDEPseudoTerminalDomain
Code: 7
Failure Reason: ErrorCode: 7 Errno: 6
User Info: {
    DVTErrorCreationDateKey = "2023-09-08 01:38:56 +0000";
    IDERunOperationFailingWorker = DBGLLDBLauncher;
}
--
The operation couldn’t be completed. Device not configured
Domain: NSPOSIXErrorDomain
Code: 6
Failure Reason: Device not configured
--

Analytics Event: com.apple.dt.IDERunOperationWorkerFinished : {
    "device_model" = "MacBookPro18,2";
    "device_osBuild" = "13.4.1 (22F82)";
    "device_platform" = "com.apple.platform.macosx";
    "launchSession_schemeCommand" = Run;
    "launchSession_state" = 1;
    "launchSession_targetArch" = arm64;
    "operation_duration_ms" = 66;
    "operation_errorCode" = 7;
    "operation_errorDomain" = IDEPseudoTerminalDomain;
    "operation_errorWorker" = DBGLLDBLauncher;
    "operation_name" = IDERunOperationWorkerGroup;
    "param_consoleMode" = 0;
    "param_debugger_attachToExtensions" = 0;
    "param_debugger_attachToXPC" = 1;
    "param_debugger_type" = 3;
    "param_destination_isProxy" = 0;
    "param_destination_platform" = "com.apple.platform.macosx";
    "param_diag_MainThreadChecker_stopOnIssue" = 0;
    "param_diag_MallocStackLogging_enableDuringAttach" = 0;
    "param_diag_MallocStackLogging_enableForXPC" = 1;
    "param_diag_allowLocationSimulation" = 1;
    "param_diag_checker_tpc_enable" = 1;
    "param_diag_gpu_frameCapture_enable" = 0;
    "param_diag_gpu_shaderValidation_enable" = 0;
    "param_diag_gpu_validation_enable" = 0;
    "param_diag_memoryGraphOnResourceException" = 0;
    "param_diag_queueDebugging_enable" = 1;
    "param_diag_runtimeProfile_generate" = 0;
    "param_diag_sanitizer_asan_enable" = 0;
    "param_diag_sanitizer_tsan_enable" = 0;
    "param_diag_sanitizer_tsan_stopOnIssue" = 0;
    "param_diag_sanitizer_ubsan_stopOnIssue" = 0;
    "param_diag_showNonLocalizedStrings" = 0;
    "param_diag_viewDebugging_enabled" = 1;
    "param_diag_viewDebugging_insertDylibOnLaunch" = 1;
    "param_install_style" = 0;
    "param_launcher_UID" = 2;
    "param_launcher_allowDeviceSensorReplayData" = 0;
    "param_launcher_kind" = 0;
    "param_launcher_style" = 99;
    "param_launcher_substyle" = 8192;
    "param_runnable_appExtensionHostRunMode" = 0;
    "param_runnable_productType" = "com.apple.product-type.application";
    "param_testing_launchedForTesting" = 0;
    "param_testing_suppressSimulatorApp" = 0;
    "param_testing_usingCLI" = 0;
    "sdk_canonicalName" = "macosx13.3";
    "sdk_osVersion" = "13.3";
    "sdk_variant" = macos;
}
--

System Information

macOS Version 13.4.1 (Build 22F82)
Xcode 14.3.1 (21815) (Build 14E300c)
Timestamp: 2023-09-07T20:38:56-05:00

I closed Terminal, restarted Xcode and it built just fine.
When I first built CodeEdit, I opened the terminal utility and the terminal in the drawer looks fine. I opened Terminal.app again and I get the same error:
[forkpty: Device not configured]
[Could not create a new process and open a pseudo-tty.]

I open subsequent tabs in my terminal and the terminal is blank.

 1. Did you used the terminal in CodeEdit?
 2. Did you sent a process to the background?
 (multiple ways, e.g. putting & on the end of the command example: brew upgrade &),
 2.a if you did that did you use disown to separate it from the CLI process?
 (why those questions, it could be that there was still a process running somewhwere)

I did not use the terminal in CE, just opened it

This hasn't happened to me in some time. I have updated to Sonoma since. Not sure if that fixed it or not but I am closing this for now unless someone else experiences this.


Error handling response: TypeError: Cannot read properties of undefined (reading 'join')

Sorry for not noticing this issue until now. I had fixed it in 2023/9/19. Please replace popup.js with the newest version. Thank you.
If you still encounter other related issues, feel free to respond me.
Sorry again for not replying for such a long time. (╥﹏╥)


Add creation time and date - 54

[Auto generated]
Number: [#54]

hive-claim-manager(etbYW)[PR 54]: Closing issue for PR: Add creation time and date.
PR was closed.


Upgrade to flask_jwt_extended>3

Update usage of flask_jwt_extended decorator to be compatible with flask_jwt_extended>3, which is a requirement for using Flask>=3.
See https://github.com/sorgerlab/indra/issues/1448.

All tests but one in the test_XDD passed. The failing test seem to be a 500 error when I checked locally, so it's difficult to determine if it's our fault or the server's.

This code and the updated code in ui_util and indra have been tested and is running on emmaa.indra.bio now.


Handle bad words

Description (what's changed?)
Enter "dick" as the prompt. You'll get an error message.

Task linked: CU-86ayt6gf9 Add Inappropriate warning message


RFC: Hash and Eq requirement for values

Maybe a duplicate of https://github.com/jonhoo/rust-evmap/issues/45 but I figured I'd ask anyway.
I'm trying to use evmap as the base store for an MQTT broker. For simple subscription matches I'm keying off the topic, and my initial attempt uses a struct containing a token+futures::channel::mpsc::Sender as the value item in order to route messages. The evmap crate seems to be a good fit for this application: the most common case is using the collection to look up subscribers for a topic, and then pushing publish messages into their queues. I want this to be lock-free and fast. Occasionally I have to modify the subscriptions list when subscribers come and go, and I don't want to have to block all publishers to do so. And having the value be a collection is actually useful, because each topic can have multiple routing destinations. Storing a list of these items is something I'd have to do anyway. But using hashbag internally creates some challenges:

futures::channel::mpsc::Sender doesn't impl Eq or Hash, and in fact most senders don't even implement PartialEq let alone Hash (futures_channel, crossbeam, flume). I used the derivative crate to try out an implementation where senders to the same receiver were equivalent, but...
A lot of sender implementations require &mut to send. For reasons that are obvious in retrospect (extensive use of interior aliasing) evmap isn't set up to allow interior mutability of values. I can work with unbounded channels in a prototype because I don't need &mut, but this isn't a great approach in a production application-- I'm trying to bound memory usage and balance performance for clients that can publish lots of messages in a short time, and using an unbounded channel could cause resource starvation or long tails elsewhere.

I can actually work around the second issue with flume, because the sender doesn't need to be mutable. But flume doesn't have a way to determine whether two senders point at the same receiver, and I'm not sure how to sanely implement Hash for it either.
So I guess I'm trying to figure out whether it makes any sense at all to attempt a feature patch/PR that removes the Eq and Hash requirement by allowing a different container, perhaps a normal linear sequence like Vec for Long. Are there other reasons for using hashbag that I'm not seeing?

Sorry for the super late reply!
So, this is a tricky one. The hashbag is there very specifically to support efficient removes from value sets, and there it's pretty much impossible to avoid. I think that the workaround here would be to have a complete copy of the implementation that was single-valued (as you observed, this is #45), but that's also a sub-optimal solution.
Now, in your particular case, since you don't need to remove individual values from a value set, you can actually work around this by sticking a newtype around your tuple and just implement Hash + Eq for it with a trivial implementation that always returns true for equality and always hashes to the same value. It feels dirty, but it should work as long as you only ever use that newtype for your thing when it's placed in an evmap value, and as long as you never have more than one value in any given value set. If you do need to have multiple values per value set, but just don't need support for individual removes, you'd have to make Eq return false instead, but that violates the spec of Eq (it's no longer reflexive) and is more likely to cause unexpected issues.
The broader solution here, I think, is to find a way to implement #45 without copying the whole implementation. I'll put a comment over there with some thoughts.

Thanks! Sorry, I didn't realize you were in the middle of your dissertation defense. How did it go?

Hehe, I have now graduates :tada: :)

I'm going to close this following the merge of #73 :)


Readme.md Update

No need to review this now; it's just a work in progress.

The way the individual methods are broken up feels really cluttered, with it being something like:
###Methods
###Parameters
###Methods
###Parameters
It's not clear as a user what method I'm actually looking at just by skimming.

More specifically, this is what I'm referring to:

Instead of having textual headers, why not just use the method names? The textual headers could be a short description below.


Can't find Joplin via PowerToys Run

Microsoft PowerToys version
v0.79.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
PowerToys Run
Steps to reproduce
Complete the Joplin installation > use the shortcut key to invoke PowerToys Run > enter Joplin。
✔ Expected Behavior
See the Joplin app
❌ Actual Behavior
Don't see the Joplin app
Other Software
I saw the Joplin application under Applications in Settings, so I went to PowerToys instead of Joplin.
Joplin 2.14.20 (prod, win32)
https://github.com/laurent22/joplin/
revised edition: cfd98e3
Backup: 1.4.0
CodeMirror Options: 1.0.1
Windows 10 Pro 22H2  OS components: 19045.4046

You may need a shortcut at desktop or somewhere at Start Menu I think.


Spaces break line in $ENV_VARIABLE

Hey, there.
I stucked with same problem as here
Example:
SoleSensei@SoleSurface:~$ $BROWSER
-bash: /mnt/c/Program: No such file or directory
SoleSensei@SoleSurface:~$ export BROWSER='"/mnt/c/Program\ Files/Firefox/firefox.exe"'
SoleSensei@SoleSurface:~$ $BROWSER
-bash: "/mnt/c/Program\: No such file or directory
SoleSensei@SoleSurface:~$ export BROWSER="/mnt/c/Program\ Files/Firefox/firefox.exe"
SoleSensei@SoleSurface:~$ $BROWSER
-bash: /mnt/c/Program\: No such file or directory
SoleSensei@SoleSurface:~$ export BROWSER=/mnt/c/Program\ Files/Firefox/firefox.exe
SoleSensei@SoleSurface:~$ $BROWSER
-bash: /mnt/c/Program: No such file or directory
SoleSensei@SoleSurface:~$ export BROWSER='/mnt/c/Program Files/Firefox/firefox.exe'
SoleSensei@SoleSurface:~$ $BROWSER
-bash: /mnt/c/Program: No such file or directory
SoleSensei@SoleSurface:~$ xdg-open https://github.com
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: /mnt/c/Program: not found
/usr/bin/xdg-open: 851: /usr/bin/xdg-open: /mnt/c/Program: not found
xdg-open: no method available for opening 'https://github.com'

Path just breaks on first space. Any ideas?
_Originally posted by @SoleSensei in https://github.com/Microsoft/WSL/issues/1766#issuecomment-432814055

You'll need to quote your environment variable usage:
"$BROWSER"
https://superuser.com/questions/616351/spaces-in-linux-environmental-variables

@benhillis, okay, but what about xdg-open and other applications using unquoted variable environments?

@SoleSensei - Their scripts should be updated because they are incorrect.  Spaces are not common in Linux paths, but they are not illegal either.

okay, thanks, so i'm closing issue. And trying not to use Windows paths with spaces in WLS.

I have used :
export BROWSER='eval "/mnt/c/Program Files (x86)/Google/Chrome/Application/chrome.exe"'
It worked for me.


Footer

Naming Rules
Name your PR like this: Footer component
Description
footer component is created and another button component is created for donation. That button component is inside footer. the footer component is added in the about.js file.

View rendered .github/CODE_OF_CONDUCT.md
View rendered .github/CONTRIBUTING.md
View rendered .github/ISSUE_TEMPLATE/bug-report.md
View rendered .github/ISSUE_TEMPLATE/feature-request.md
View rendered .github/ISSUE_TEMPLATE/user-story.md
View rendered .github/PULL_REQUEST_TEMPLATE.md
View rendered README.md
View rendered bin/README.md

I think this was intended for your project, not the upstream starter kit


Using a variable's value as translate ID in a filter ?

I have a variable which content I'd like to use as translateId:
.. locale.js: {
TRUE: 'true'
... }
in JavaScript:
data.value = 'true'
in my template:
{{data.value | translate}}
The problem is that data.value is not evaluated so the end result is: data.value.
I tried to chain filters like this, but it didn't help:
{{data.value | uppercase |translate}}
Is it possible to make it work or do I have to rely on JavaScript ? (adding a function that would do that for me in the scope isn't a problem).

Ehm.. if your translations is lookking like { TRUE: 'true' } you should use an expression like data.value = 'TRUE' together with {{data.value | translate}}.
OK?


Open sftp file using command line

I wonder if there's any way to open a remote source file using the command line, like:
"code sftp:/foo/bar.cpp"

Try this https://marketplace.visualstudio.com/items?itemName=liximomo.remotefs


make flake8 happy

just annoying flake8 stuff to get the green check :)

irgnoring the changelog error, because not relevant for this commit.


[AutoPR-confidentialledger]autogenerated

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

/azp run prepare-pipelines


Error handling - Bug fixes

Features of this PR

Fixed bugs in handling cases when invalid start and end date-times are given for an Event either while adding or editing.

Previously, an Exception was thrown and not being caught.

invalid start and end date-times

Could you specify what can be considered as invalid here? Perhaps adding an example might help others to understand which cases are now covered by this PR

The changes have been made in a new branch. PR will be given from that branch.


fix: update getImageApi signature

sry for so long merge, trying to update everything now
will update manually because of source changes in lib/


/en-US/docs/Web/JavaScript/Guide/Using_promises

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

There is typo on https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Using_promises#Error_propagation. The second paragraph states "Basically, if there's an exception, promise chain will looking down the chain for catch handlers or onRejected. This is very much modeled after how synchronous code works:"
Looking should be changed to look.
Thanks for creating and maintaining these docs. They are extremely helpful!

Ah yes, that sentence was a bit scrappy; I've sorted it out.
Thanks for reporting, and I'm really glad to hear you are finding the docs helpful!


Error SystemStackError when ruby-lsp tries to index .crt file

Description
Reproduction steps

Initialize new project with bundle init
Add gem 'aws-sdk-core', '3.132.0' into Gemfile
Install gems with bundle install --path=.bundle
Create file .index.yml with:

included_patterns:
  - ".bundle/**/*"`

Run ruby-lsp-doctor
Observe stack level too deep (SystemStackError) error

Here are logs from this run:
Globbing for indexable files
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/config
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/bin
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/bin/jmespath.rb
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/build_info
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/cache
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/cache/aws-eventstream-1.3.0.gem
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/cache/aws-partitions-1.934.0.gem
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/cache/aws-sdk-core-3.132.0.gem
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/cache/aws-sigv4-1.8.0.gem
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/cache/jmespath-1.6.2.gem
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/doc
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/extensions
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/CHANGELOG.md
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/LICENSE.txt
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/VERSION
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib/aws-eventstream
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib/aws-eventstream/decoder.rb
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib/aws-eventstream/encoder.rb
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib/aws-eventstream/errors.rb
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib/aws-eventstream/header_value.rb
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib/aws-eventstream/message.rb
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib/aws-eventstream/types.rb
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-eventstream-1.3.0/lib/aws-eventstream.rb
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-partitions-1.934.0
indexing: /home/egor/Documents/work/aiir/tmp/.bundle/ruby/3.2.0/gems/aws-partitions-1.934.0/CHANGELOG.md
/home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/dispatcher.rb:216:in `each': stack level too deep (SystemStackError)
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/dispatcher.rb:216:in `visit_call_node'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/node.rb:2513:in `accept'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/visitor.rb:31:in `block in visit_child_nodes'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/visitor.rb:31:in `each'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/visitor.rb:31:in `visit_child_nodes'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/dispatcher.rb:105:in `visit_arguments_node'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/node.rb:660:in `accept'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/prism-0.29.0/lib/prism/visitor.rb:31:in `block in visit_child_nodes'
         ... 10385 levels...
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/3.2.0/bundler/friendly_errors.rb:117:in `with_friendly_errors'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/lib/ruby/gems/3.2.0/gems/bundler-2.4.1/libexec/bundle:33:in `<top (required)>'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/bin/bundle:25:in `load'
        from /home/egor/.local/share/mise/installs/ruby/3.2.0/bin/bundle:25:in `<main>'

Thank you for the issue report. There are certain occasions where you may want to index files with different extensions. For example, we may want to index RBI/RBS files to enhance the index information with type information (currently doesn't work, but it might in the future).
I don't believe the index should enforce that all included files must have a Ruby extension. I think the fix here is to restrict your glob pattern to Ruby files.
included_patterns:
  - ".bundle/**/*.rb"

Also, please note that we're discussing the existence of this .index.yml file as it kind of breaks the approach taken by most language servers, which rely on editor configuration files (like .vscode/settings.json for VS Code) instead of using custom files. That is, this file may be going away some time soon.

@vinistock Thank you for quick response!
By ruby files I did not only meant all .rb files, but just all files that will be useful for completeion inside ruby files, so that also includes RBI/RBS.
I don't understand how files with cerificates, code of conduct, etc. will be helpful for completion.
But maybe you are right, it is better for the editor to provide this type of information (on what to index)
Sometimes I add more than one project folder to editor workspace.
Looking forward to it!🙌


Update experiments-msmarco-passage.md

Everything worked.
System Information:
Operating System: Ubuntu 22.04.3 LTS on WSL
Python Version: Python 3.10.2
Java Version:
openjdk <IP_ADDRESS> 2023-08-24
OpenJDK Runtime Environment (build <IP_ADDRESS>+1-post-Ubuntu-0ubuntu122.04) OpenJDK 64-Bit Server VM (build <IP_ADDRESS>+1-post-Ubuntu-0ubuntu122.04, mixed mode, sharing) Maven Version: Apache Maven 3.6.3 gcc version: gcc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Environment Configuration:
Number of CPUs: 20
Amount of Memory: 9GB
Disk Space: 512GB

Please continue along the onboarding path.

Having not seen further progress, closing.

Having not seen further progress, closing.

I merged all reproductions to the other pull request as instructed. Sorry for the inconvenience.


Store all user events related to a skill id, whether applied or not

Currently only successfully applied skill events are stored for a user. Save all events to support deeper metrics on continued skill usage after achievement.

user_events do not seem to be recorded when the skill being reported is not outside of the minimum time window between occurrences to receive points.
Steps to repeat:

Visit Iron Man Skill: https://ip-10-113-80-245.evoforge.org/projects/movies/subjects/action/skills/IronMan/addSkillEvent

Perform a baseline count of user_events: select count(*) from user_events:

Add an event for user1 for today, and then refresh the count:

Add an event for user1 for the same day, and then refresh the count:

Note: if the skill has already been achieved (vs. not being outside the minimum time window), then the user_event does seem to added as expected.

spoke with @mosefer - the test above was invalid.  There is a count column that is being incremented even if the reported event is not outside of the minimum time window:


adb fails to collect certificates, even if make keystore has already been run

I'm using the Terminator terminal on Arch Linux. The Java development kit is installed, Android Studio is installed and I followed the setup instructions up until this command below.
Command:
make push run
Output:
echo "Installing" org.yourorg.cnfgtest
Installing org.yourorg.cnfgtest
adb install makecapk.apk
Performing Streamed Install
adb: failed to install makecapk.apk: Failure [INSTALL_PARSE_FAILED_NO_CERTIFICATES: Failed to collect certificates from /data/app/vmdl807465674.tmp/base.apk: Attempt to get length of null array]
make: *** [Makefile:143: push] Error 1

Please try re-running with make clean run and posting entire command output.

```make clean run`` solves the issue. Apparently there was no keystore before.


尝试master分支。

          尝试master分支。

Originally posted by @al-one in https://github.com/al-one/hass-xiaomi-miot/issues/1594#issuecomment-2017122217

      尝试master分支。


Error when change path for admin

I want to change the default prefix admin route from www.mysite.com/admin to www.mysite.com.
In routes/web.php I added:
Voyager::routes();
And in config/voyager.php I change :
'redirect' => '/admin',
to:
'redirect' => '/',
But give me the following error:

Laravel Version: 5.7.6
Voyager Version: 1.1
Voyager Frontend Version: 0.8.3
PHP Version: 7.1.7

@snggh this is overriding the default login route /login. You can update the Voyager Frontend views with the appropriate Voyager routes, or override the Voyager routes from the defaults. See the Voyager docs.


Block & Twitter don't store color in input field

The Block & Twitter components both have an input field which does not display the current hex code of the color prop.
Is there a workaround for this?

I noticed this, too.  Would be great to have a fix for this.

This has been fixed in<EMAIL_ADDRESS>!

Thanks @casesandberg

Thank you!


Release Google.Cloud.Notebooks.V1Beta1 version 2.0.0-beta04

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Created release for Google.Cloud.Notebooks.V1Beta1-2.0.0-beta04


feat: atomic-product-listing

COMSDZ-573
In this PR:
The atomic-product-listing component which is the equivalent to the atomic-result-list for the ProductListingEngine

Closing those PR as there was a focus shift regarding those component in the commerce unit.
Re-open if necessary.


Check a Route Exists fails if host is omitted

Issue
In Check a Route Exists, host is documented as only required if the domain supplied is shared. However, if the host is omitted (as in the example) the route exists request fails (404) indicating the route does not exist when, in fact, it does.
See also issue #648.
Steps to Reproduce
A private domain is created, and a route is created with that domain, but with no host supplied. A subsequent Check a Route Exists call is made (without a host "parameter") and returns 404.
First, create a private domain:
POST /v2/domains
{"name":"test.domain.8djpf"
,"owning_organization_guid":"6f314197-4cee-4ac2-a37d-f0be66d942c1"
,"wildcard":true
}

getting the result (actual example):
HTTP/1.1 201 Created
{
  "metadata": {
    "guid": "5c711cf0-39f7-4e2e-ab4a-ef3efcc941e5",
    "url": "/v2/domains/5c711cf0-39f7-4e2e-ab4a-ef3efcc941e5",
    "created_at": "2016-07-20T10:40:23Z",
    "updated_at": null
  },
  "entity": {
    "name": "test.domain.8djpf",
    "owning_organization_guid": "6f314197-4cee-4ac2-a37d-f0be66d942c1",
    "shared_organizations": [],
    "owning_organization_url": "/v2/organizations/6f314197-4cee-4ac2-a37d-f0be66d942c1",
    "spaces_url": "/v2/domains/5c711cf0-39f7-4e2e-ab4a-ef3efcc941e5/spaces"
  }
}

Second, create a route:
POST /v2/routes
{"domain_guid":"5c711cf0-39f7-4e2e-ab4a-ef3efcc941e5"
,"space_guid":"633786f6-a363-4023-87be-c8cce8dbfb65"
}

getting the result (actual example):
HTTP/1.1 201 Created
{
  "metadata": {
    "guid": "38edf90e-61a9-48fc-aabf-13b45255ed13",
    "url": "/v2/routes/38edf90e-61a9-48fc-aabf-13b45255ed13",
    "created_at": "2016-07-20T10:40:23Z",
    "updated_at": null
  },
  "entity": {
    "host": "",
    "path": "",
    "domain_guid": "5c711cf0-39f7-4e2e-ab4a-ef3efcc941e5",
    "space_guid": "633786f6-a363-4023-87be-c8cce8dbfb65",
    "service_instance_guid": null,
    "port": null,
    "domain_url": "/v2/domains/5c711cf0-39f7-4e2e-ab4a-ef3efcc941e5",
    "space_url": "/v2/spaces/633786f6-a363-4023-87be-c8cce8dbfb65",
    "apps_url": "/v2/routes/38edf90e-61a9-48fc-aabf-13b45255ed13/apps",
    "route_mappings_url": "/v2/routes/38edf90e-61a9-48fc-aabf-13b45255ed13/route_mappings"
  }
}

Third, check the route exists:
GET /v2/routes/reserved/domain/5c711cf0-39f7-4e2e-ab4a-ef3efcc941e5

Expected result
HTTP/1.1 204 No Content
Current result
HTTP/1.1 404 Not Found
Possible Fix
Unknown; but if it is working correctly, please update the doc to explain the criteria whereby the host may be omitted from the request.

Also can't reproduce, am I missing something? Can you do cf api to get the version of the API you are using?
○ → cf curl /v2/domains -X POST -d '{"name":"test.domain.8djpf", "owning_organization_guid":"'$(cf org nick --guid)'","wildcard":true}'
{
   "metadata": {
      "guid": "90f9a63f-0171-4cf3-b295-e435fdab8093",
      "url": "/v2/private_domains/90f9a63f-0171-4cf3-b295-e435fdab8093",
      "created_at": "2016-07-20T18:41:19Z",
      "updated_at": null
   },
   "entity": {
      "name": "test.domain.8djpf",
      "owning_organization_guid": "7e21c7e2-f60c-49ab-ad18-6d9e8a26add3",
      "owning_organization_url": "/v2/organizations/7e21c7e2-f60c-49ab-ad18-6d9e8a26add3",
      "spaces_url": "/v2/domains/90f9a63f-0171-4cf3-b295-e435fdab8093/spaces"
   }
}

○ → cf curl /v2/routes -X POST -d '{"domain_guid":"90f9a63f-0171-4cf3-b295-e435fdab8093", "space_guid":"'$(cf space nick --guid)'"}'
{
   "metadata": {
      "guid": "97d12c79-6607-43cb-af74-2fd7ce8f189e",
      "url": "/v2/routes/97d12c79-6607-43cb-af74-2fd7ce8f189e",
      "created_at": "2016-07-20T18:42:27Z",
      "updated_at": null
   },
   "entity": {
      "host": "",
      "path": "",
      "domain_guid": "90f9a63f-0171-4cf3-b295-e435fdab8093",
      "space_guid": "cfdcbe8e-788a-440b-b191-9f1054660ac0",
      "service_instance_guid": null,
      "port": null,
      "domain_url": "/v2/private_domains/90f9a63f-0171-4cf3-b295-e435fdab8093",
      "space_url": "/v2/spaces/cfdcbe8e-788a-440b-b191-9f1054660ac0",
      "apps_url": "/v2/routes/97d12c79-6607-43cb-af74-2fd7ce8f189e/apps",
      "route_mappings_url": "/v2/routes/97d12c79-6607-43cb-af74-2fd7ce8f189e/route_mappings"
   }
}

○ → cf routes
Getting routes for org nick / space nick as admin ...

space   host           domain                port   path   type   apps              service
nick    nickdora       xena-app.cf-app.com                        v3-nickdora-web
nick    lattice8080    xena-app.cf-app.com
nick    lattice8081    xena-app.cf-app.com
nick    lattice8082    xena-app.cf-app.com
nick    doradora       xena-app.cf-app.com                        doradora
nick    async-broker   xena-app.cf-app.com                        async-broker
nick    nicknode       xena-app.cf-app.com                        nicknode
nick    nicktest       xena-app.cf-app.com                        nicknode
nick                   test.domain.8djpf

ruby 2.3.1p112 canvasback in ~/workspace/cf-release/src/capi-release/src/cloud_controller_ng
○ → CF_TRACE=true cf curl /v2/routes/reserved/domain/90f9a63f-0171-4cf3-b295-e435fdab8093

REQUEST: [2016-07-20T11:43:06-07:00]
GET /v2/routes/reserved/domain/90f9a63f-0171-4cf3-b295-e435fdab8093 HTTP/1.1
Host: api.xena.cf-app.com
Accept: application/json
Authorization: [PRIVATE DATA HIDDEN]
Content-Type: application/json
User-Agent: go-cli 6.19.0+b29b4e0 / darwin

RESPONSE: [2016-07-20T11:43:06-07:00]
HTTP/1.1 204 No Content
Connection: keep-alive
Date: Wed, 20 Jul 2016 18:43:06 GMT
Server: nginx
X-Content-Type-Options: nosniff
X-Vcap-Request-Id: 4cadf806-6f74-45b8-6e63-63da71e78a90
X-Vcap-Request-Id: 4cadf806-6f74-45b8-6e63-63da71e78a90::2b619206-0a70-4432-96ee-903133035cb8
Content-Length: 0

I'm using PcfDev 17 which is at 235/2.54.0.  You are probably at 239/2.58.0, which could explain my problem.

Looks like there was a change made by the routing team between CF-235 and CF-236.
The endpoint that includes the host in the path in CF-235 was changed to Check a HTTP Route exists in CF-236 when they added support for Check if a (TCP) Route exists.
The endpoint you are using without the host in the path didn't exist until CF-236.


Fixes Image and converter for incomplete src and srcset

fixes https://github.com/frontity/saturn-theme/issues/81

@luisherranz ready for review.

@orballo D:!

:tada: This PR is included in version 0.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add getModName function

Sometimes the name of mod needs to be displayed, such as on the tip of the button.

Hey @wowlusitong thanks for submitting this. I'm not sure exactly how I feel about it though, because it gets into the "UI" realm where it has to decide what string to return.
I'd be okay with a change to make it export an isMac() helper, so that people can choose their own strings, but still have access to the internal environment state that the library uses instead.

Thanks


AV-186753: Added pagination support in vRO plugin.

Updated get vRO plugin method to return next parameter, in order to be able to fetch the list with more than 25 item.

Tested and Verified that pagination should be operational like shown in SWAGGER. next parameter is shown in the response of the get method. Screenshot attachment for the same.


test: add tests for field-editor-date timezone logic

Cover with cypress tests the case described here: https://github.com/contentful/field-editors/pull/1239

Does this also cover cases where the viewer's time zone is different from the one selected?

yes, that's why I added two tests with different time zones

@bgutsol Isn't it the field value being set?

@bgutsol Isn't it the field value being set?

what do you mean?

@bgutsol I mean you're setting different values for the field, but in both test cases the system time is the same. Looks to me like we should cover some cases where we use https://docs.cypress.io/api/commands/clock#Change-current-system-time

@bgutsol I mean you're setting different values for the field, but in both test cases the system time is the same. Looks to me like it would be beneficial for us to add some tests where we use https://docs.cypress.io/api/commands/clock#Change-current-system-time

@denkristoffer tbh it doesn't matter what system time is on the machine since it doesn't take system time into account when converting a date. The only system parameter that affects the conversion is the timezone. That's why I'm running two tests with different UTC offsets in the initialValue to cover most cases. The other way can be to read the current local system timezone and based on it prepare a UTC offset for the initial value, but it seems artificial

@denkristoffer tbh it doesn't matter what system time is on the machine since it doesn't take system time into account when converting a date. The only system parameter that affects the conversion is the timezone. That's why I'm running two tests with different UTC offsets in the initialValue to cover most cases. The other way can be to read the current local system timezone and based on it prepare a UTC offset for the initial value, but it seems artificial to me. What are your thoughts?

@bgutsol But the problem we had before was exactly that, that the date displayed took local timezone into consideration, right? So I think it makes sense to cover that situation here with a test that makes sure that it doesn't change the displayed data no matter what time zone you're in.
Although I don't think we have to test it with an integration test.

@denkristoffer tbh it doesn't matter what system time is on the machine since it doesn't take system time into account when converting a date. The only system parameter that affects the conversion is the timezone. That's why I'm running two tests with different UTC offsets in the initialValue to cover most cases. The other way can be to read the current local system timezone and based on it prepare a UTC offset for the initial value, but it seems artificial to me. What are your thoughts?

@bgutsol But the problem we had before was exactly that, that the date displayed took local timezone into consideration, right? So I think it makes sense to cover that situation here with a test that makes sure that it doesn't change the displayed data no matter what time zone you're in.
Although I don't think we have to test it with an integration test.

Added some unit tests, @denkristoffer pls take a look


Add logging of headers of DICOMs received by orthanc-raw

Optionally logs selected tags from DICOM instances received by orthanc-raw to a CSV file.

I have added a fresh commit here for modifying the test.csv, but it's not appearing


Drag slider with plot updating performance test

Drag Slider with Plot Updating Performance Test
I've created a basic application with a slider and a Plotly plot. I want to explore my data/ model interactively by dragging the slider fast back and forth.
As you can see below only Panel can deliver the performance needed.
You can find the source code here.
Test Results

Framework
Supports drag events
Snappy experience
Comment

Dash
yes
no
Dash supports dragging the slider but quickly gives up

Panel
yes
yes
No problem

Streamlit
no
no
Seems not supported.

https://user-images.githubusercontent.com/42288570/149648993-5a850b36-cbcc-4b1f-9908-3778557759e0.mp4
Discussion

Dash: As you can see Dash quickly gives up. I believe that the combination of Flask and http will never be able to provide really snappy updates. I've been told that an additional issue is that dash sends a lot of data back and forth on each update.
Panel: I've always been impressed by the performance of Bokeh. Panel builds on Bokeh and is capable of achieving similar performance. My hypothesis is that Panel would be even more snappy with Bokeh/ Holoviews plots.
Streamlit: I could not find a way to react to slider dragging. I believe the run script top to bottom architecture of Streamlit will never be able to provide really snappy updates. See this discussion Make st.slider and st.color_picker only rerun script on mouse up and Improve performance of Streamlit to update plots.

For example Dask Dashboard is developed in Bokeh and is very, very performant. I don't believe its possible to build a performant application like that in Streamlit or Dash. It would be in Panel though.
I would expect interactive applications with streaming to perform much better in Panel than Dash. Streamlit does not really support interactive applications with streaming.
Another noticeable difference is just how much code it takes to make a basic Dash application.
Source
https://user-images.githubusercontent.com/42288570/149649005-24f9f4d2-b84f-45d2-888a-c2b4b618c66b.mp4
https://user-images.githubusercontent.com/42288570/149649006-44307849-273b-4b35-b3f1-bf657f8b5a36.mp4
https://user-images.githubusercontent.com/42288570/149649007-eca75cf9-fdf6-4304-85ef-6eea26ccd3c0.mp4
Promo Gif

Promo Video
https://user-images.githubusercontent.com/42288570/149649087-f1f94166-f776-4b21-bbe9-fa9425d7dae0.mp4
Commands
docker run --rm -it -p 80:80 dataappperformance:latest -c 'invoke slider-plot.panel'
docker run --rm -it -p 80:80 dataappperformance:latest -c 'invoke slider-plot.streamlit'
docker run --rm -it -p 80:80 dataappperformance:latest -c 'invoke slider-plot.dash'

Just wanted to say thank you for putting together this comparison!


use pipdeptree for generating requirements file

🛰 Feature
Description

Currently, the versions.txt contains package names, not pip names. It would be nice if we could generate a full requirements.txt.
This appears to be very hard using pip or python built in features alone. I stumbled upon the package pipdeptree, which looks like it could be useful for that task.
Solution

Investigate if pipdeptree helps, if so modify dumptools.packagefinder such that it generates pip names.
Alternatives

Keep it the way it is.

I think it's worth taking a look at pipdeptree to see if it helps. It seems to have a decent user base according to pypi stats and looks to have stable versions.

actually pipdeptree isn't needed: #425


DevAudit does not find artifact/package when version range used in query.

Consider this dependency in compose.json:
"guzzlehttp/guzzle": "~6.1",

When used with the artifact search we receive any artifacts that can match this range:
GET http://localhost:8080/v1.1/search/artifact/composer/guzzlehttp/guzzle/~6.1

[
  {
    "id":<PHONE_NUMBER>,
    "groupid": "guzzlehttp",
    "name": "guzzle",
    "version": "<IP_ADDRESS>",
  },
  {
    "id":<PHONE_NUMBER>,
    "groupid": "guzzlehttp",
    "name": "guzzle",
    "version": "<IP_ADDRESS>",
  }
]

Unfortunately, it seems that DevAudit is looking through these artifacts and deciding that none of them match the requested version, so it reports that no artifact is found, when in reality there are two known matches. Try changing the dependency in the composer.json file:
"guzzlehttp/guzzle": "<IP_ADDRESS>",

This will now return a result in DevAudit.
I think that one of the following are a reasonable way of handling the situation:

Use the oldest artifact returned as the match (using its version). This is reasonable because unless the user updates they could be using ANY of the versions in the range, and it is best to assume the worst.
Use the newest artifact as the match. I don't like this as much, but it is a valid way of looking at things.
Ignore the versions returned -- use all of the data but calculate a reasonable version from the requested range. Specifically calculate the lowest reasonable range that matches the request. For example, if the range is "~6.1" then 6.1 would be the version to use.

I THINK THE THIRD OPTION IS THE BEST as it will keep working even if the server returns a nonsense version number.

This should be fixed by 20778af as it uses the most recent version of the Versatile library which properly handles versions like ~6.1. <IP_ADDRESS> matches version 6.1~ according to Composer versioning rules so no change to the artifact handling is needed.

It looks better, but there still seems to be some sort of issue. DevAudit reports:
[3/4] guzzle 1 known vulnerability, 0 affecting installed package version(s).

The v2.0 API returns a different vulnerability from the v1.1 API. This discrepancy needs to be resolved.

The original issue is definitely resolved, while the last few updates are completely confused and nonsensical.


Easee equalizer import energy

The problem
The Easee equalizer import energy shows the value with decimal in the wrong place.
Example:
shows 521961,0 kwh, but the correct value is 52196,1 kwh
Version of Easee integration having the issue?
v0.9.45
Version of Home Assistant Core having the issue?
2022.9.7
Anything in the logs that might be useful for us?
No response
Additional information
No response

Since this is most likely a bug in the equalizer and not in the HA component (you probably see the same incorrect value in the easee app, right?) you should contact Easee about this.

Found it on the Easee app and that is tight, it's the wrong value there too.
I have sent a bug report to Easee now.

Any update on this? My dad's meter seems to be very buggy and reporting all over the place...

Please report your problem to Easee, we are not able to do anything about this at our end.

Contact Easee support, they made a firmvare update for me and now everything works fine :)

Are you saying that is behaves randomly such that the decimal point moves back and forth?
Does the Easee app show the same values as the HA integration?
The only conversion we do is to change between kWh and Wh depending on the settings you selected in the integration. A division or multiplication by 10 should not be possible.
If you enable debug logging, you can see what the values that are transferred from the easee cloud are.
Add this to your configuration.yaml:
logger:
  default: info
  logs:
    pyeasee: debug
    custom_components.easee: debug

Then you should see the corresponding data in the log file by searching for lines that look like this:
Callback xxxxx 45 state_cumulativeActivePowerImport 73262.7265625
Callback xxxxx 46 state_cumulativeActivePowerExport 11993.2900390625

This is the raw data that is sent by the cloud. If it is wrong already here, there is not much we can do...

It seems to be randomly, however report on the hour are almost every time correct.
Thank you for the logger parameters, I have been looking for a way to log readings, and will implement this a.s.a.p.

Thank you for the tip on how to debug log the integration!
As @onelius earlier wrote I could today caught Easee cloud having the same wrong decimal positions as I experience it in the integration.
It was actually for all cumulative values and not only import and export as I first thought.
Easee support claims that this can be caused by drop out of connection between the Equalizer and easee.cloud, but I have a hard time believing that a real time reading could move the value by 10 times up and down depending on a connection dropout.
I have reported it again, awaiting response coming week.

Update: If some one else ends up here for help with the same issue.
Easee support has finally recognized that there is a problem with the Equalizer integration to "Some Swedish Aidon meters".
They wrote: "Some Swedish Aidon meters output the energy value at 10x of what they should. The Equalizer code has some compensation for it. But it appears to not be perfect. I have escalated the issue, and we will look into it."
I am waiting for their correction.

Update: Two month later Easee still hasn't been able to correct the erroneous meter reading by Easee Equalizer from Swedish Aidon meters.

@fondberg
First: Is there a way to correct historic data for the integration and that way being able to look back on correct data?
I have DB access and have been able to identified wrong stored reported data but it will not affect my energy-usage-graph.
Is that stored in a different table?
Secondly,  is it possible to change the code locally in my HA, to divide the measurement by 10 if the new reported value is greater that 10 times the previous reported value?


Create template for PRs in .github directory #7

📝 Create template for PRs in .github directory #7
📖 Description
This pull request introduces a standardized template for all future pull requests in the repository. The template aims to improve the clarity and consistency of PR submissions, making it easier for reviewers to understand the changes proposed and the reasoning behind them.
❓ What does this PR do?

Creates a pull request template in markdown format.
Guides contributors on the necessary information to include when submitting a PR.

🚀 Why is this change necessary?

Ensures that all PRs contain essential details, reducing back-and-forth communication.
Helps maintain quality and consistency across submissions, making the review process smoother.

✅ Checklist
Please ensure your PR meets the following criteria before submission:

[x] The code follows the project's coding standards and style guidelines.
[x] I have tested the changes locally and they work as expected.
[x] I have added/updated the necessary documentation (if applicable).
[x] I have updated any relevant tests and they are passing.
[x] This PR does not include any commented-out code or debug statements.

🔍 Changes Made

Created a new directory .github/PULL_REQUEST_TEMPLATE.
Added a markdown file template.md for the PR template.
Included sections for description, checklist, changes made, related issues, screenshots, and additional notes.

🔗 Related Issues

issue 7

@danielcdz

@MariangelaNM Thank you! I left some comments

@MariangelaNM can you left the template with the sections I added to the description of this PR?

📝 Create template for PRs in .github directory #7
📖 Description
This pull request introduces a standardized template for all future pull requests in the repository. The template aims to improve the clarity and consistency of PR submissions, making it easier for reviewers to understand the changes proposed and the reasoning behind them.
📸 Screenshots (if applicable)

📝 Additional Notes

Add any relevant note that can enrich the description of the PR.

@danielcdz  See how the implementation would be


Tidy base + Docs base

Changes

code style brought up to pep8 (mostly)
clear out some junk (.ipynb-checkpoint files, random file headers, setup.py changes)
add assets for website
add a _version.py file -- single central place to track version (not fully implemented yet, will have to do more here when we want to deploy to pypi etc)
probably other minor housekeeping I'm forgetting
I'd also like to change the default branch to develop if that's cool.

I did not change any functionality. The goal here is to just clean things up to continue moving forward from.
Docs

sets up framework for docs.
the docs are not complete, but this is a start

Rename repository
I would like to rename the repository. Not sure if immediately following this PR is a good time to make that change (with a combination of another PR and some triggers on GitHub, Travis), or if we want to wait until we drop the BMI support and house it elsewhere.
Related
#14 #2 #3 #4 #5 #6 #7

I messed up trying to rebase this. Closing, will PR a new one.


CVAT.org dont work

I cant to use the cvat.org. I only see the white screen. What wrong?

Experiencing the same problem since the last 12 hours. a white screen

@watermellon2018 Fixed for now, sorry for the inconvenience.


Need to have trace logs to troubleshoot OIDC federation flow

Is your suggestion related to an experience ? Please describe.
During OIDC federation flow+JIT provisioning, the locally created user could be missing claim values coming from the federated IdP which we were expecting. In this scenario, it is hard to locate whether the actual response from federated IdP missed them since claim values are obtained via backchannel token calls by JIT provisioning. The only way is to enable debug logs and monitor them.
Describe the improvement
There should be trace logs related to this flow. Especially, to have the trace logs in a separate log file.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!