Documentation bugs.

http://s3.thinkaurelius.com/docs/titan/0.9.0-M1/arch-overview.html
http://s3.thinkaurelius.com/docs/titan/0.9.0-M1/getting-started.html (Gremlin2)
http://s3.thinkaurelius.com/docs/titan/0.9.0-M1/gremlin.html (Gremlin2)
...there is alot more Gremlin than just the titanbasics.txt. It would be good for Titan guys to go through each line one by one and make sure its all right -- with appropriate links to TP3 docs.

I've made a first stab at this. If there are other specific doc issues we should open them up individually.


New pull request for Feature apple clone

This is the URL to the original page: http://archive.fo/UW4oR

Screenshot of the HTML Validation:

Screenshot of vscode validation:

Link to the last pull request:
https://github.com/AdesojiCodeMaster/Apple-Clone/pull/6

PROJECT: Building with Backgrounds and Gradients (solo)
Assignment link:
Live link:


[Feature] Button mappings GUI

OS
No response
LinearMouse
No response
Is your feature request related to a problem?
Currently button mappings need to be configured through a configuration file and there is a certain threshold to use.
Describe the solution you'd like
Build a GUI for configuring button mappings, covering common senerios.
Describe alternatives you've considered
No response
Additional context
Closes #244 and GUI implementation will be tracked here.

Closing via #386.


Plugin editor cursor not reflecting actual location

https://jumpshare.com/v/FjqN0I40MF2Chv299pAs
Here the cursor lags behind 1 cursor, it seems to happen every time the cursor reaches the end of the file
https://jmp.sh/6w3HED6H
Here the cursor doesn't look like it's where it should be -- something to do with tabs in the editor, maybe?

Hello, I can't seem to reproduce the issues in the videos, are these always happening for you or are they rare occurrences?

Yeah, it happens all the time for me.
I've recorded this with 0.12.2 along with another issue: https://jumpshare.com/v/t4DAC5OoZi80guYwCAgi

Can you tell me if v0.12.3 helps with the issue at all?

Haven't gotten to use it again for a while, but I can confirm it doesn't happen anymore in v0.13.0

Hi -- seems the issue is back with v0.13.1
https://jumpshare.com/v/K67ZaHK9ASFNZMw7suhE

I'm really confused as to why the bug regressed with v0.13.1, since it didn't contain any changes to the studio plugin.
I will investigate.

Tbh, when I tested it in v0.13.0, it was a fresh install and I only typed a couple lines in the template file then I left it for a bit since I didn't use it again until I saw the v0.13.1 update and installed that. Might be relevant info.


'NoneType' object is not subscriptable

Issue title: 'NoneType' object is not subscriptable
Screenshots:

When is the issue arising: When I go to '/desk'
My Findings: I have found some sites where this issue is getting resolved with bench update/ bench clear-cache/ bench migrate But it's not working for me.
Also, I found that it can be an issue related to database corruption.
System Information:
OS: Ubuntu 18.04
Python: 3.6.9
Bench: 5.0.0
Frappe Branch: bloomstack-v12-staging
ErpNext Branch: bloomstack-v12-staging
Any help is highly appreciated

Solution: The issue is getting resolved by restoring a new database backup.


Use News Module in product widget

i want call _NewsThumbnail.cshtml
in    product widget    How Call it ??
i tested it but it give this error
NullReferenceException: Object reference not set to an instance of an object.

If you are calling the news thumbnail in any module, you have to pass the news item to be displayed in the thumbnail. Just copy the logic inside the news controller then repeat it anywhere you would want to call news item


Feature request: allow locking position of the crafting log's side window

I usually have my crafting log window in the upper right of my screen, which means that this window is hidden off-screen. I'd like to keep my log there, but still have that window in a fixed position, which is currently only possible when it's attached to the right side of the crafting log.
Ideally, it would be nice to be able to attach to it to the right side as well, but the most useful thing for me would be a way to simply lock its position, so that I can put it somewhere and be sure it'll stay right there.

Hi, the option to lock it is already in the plugin, under UI settings.

Ok, I see what you're looking for now. I can add this as an option if the window is unattached from the recipe window for sure.

Much appreciated!

This is now in the plugin.


Spike: Determining intersection point between Augurface and 8knot

Augur has in the works a log in system for users to have pre-selected subsets of repos/org that a user came come back to. We want to capitalize on that functionality. There is further conversations necessary on how to integrate 8knot to the augurface UI

resolved. integration w/ Augur oauth login works


RDK-38406: Startupscreen launch failed with webkitbrowserplugin backend

Reason for change: On startup screen launch the "Internet" precondition is not met for HtmlApp callsign. This precondition is not required for HtmlApp & LightningApp callsigns as it might be possible for launching locally hosted pages with those callsigns
Test Procedure: Verify startupscreen is loaded with webkitbrowserplugin Priority: P1
Risks: None
Signed-off-by: Vivek.A<EMAIL_ADDRESS>

@modeveci need a formal +1 from you. Could you please take a look?

Hi @modeveci, could you review this?


top level folder spaces in names

Top level folders have names in spaces etc "APM - Final Updated" etc. - it's a pain to have to write them from the cmdline - please use underscores/dashes instead of spaces

Great suggestion. I'll plan to do that when I update the course for the final Angular 2 release.

cd TAB
then choose what you want.

I have start using APM - Start.
I am still getting "Loading the app" and not getting anything else. I looked at following link, but no luck. Any other idea ?
http://blogs.msmvps.com/deborahk/angular-2-getting-started-problem-solver/

It would be great to have instructions on how to go further and prepare an actual deployment bundle out from an Angular 2 project like this one.
Awesome course in Pluralsight, thanks!

ankitavyas - Your comment does not belong here on the github issue request for renaming the folders. Please post your comment either as a separate issue here, or better as a comment here:
https://app.pluralsight.com/library/courses/angular-2-getting-started/discussion
Thanks!

hesparza - Thank you so much for the kind words about the course!

I left the original files with the spaces because that is what is shown in the recording for the course. I did remove the spaces in the "updated" versions. Hope this helps.


Fix typos

Found via typos --format brief
Discord username (optional) please include so we can attribute you with our Contributor role (like so elvis#4747)
Description of changes (updated or new workflows)

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:

@alokedesai How about adding typos to CI pipeline?

@kianmeng I'd be open to that, feel free to send a PR! The one thing I'd want to be conscious of is avoiding false positives, so it may make the most sense to not that step be blocking


IReader - Add method for reading key

IReader - Add method for reading key
int ReadKey()

Or for RSSFeedify project, it would be better to return ConsoleKeyInfo


Tweak NurseConnect Jembi post

Make sanc and persal strings for now

@gsvr review please

:+1: when travis is happy


Fix broken link to Knife setup docs

This page's link to the knife setup docs is currently broken. I took a random guess at where it should be linking to so feel free to correct me if this is incorrect.
:nut_and_bolt: Description: What code changed, and why?

:chains: Related Resources
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
:white_check_mark: Checklist

[ ] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[ ] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable

Hi @ChefAustin! Thank you for noticing this link issue and submitting a PR to fix this. Could you please add a DCO to your commit? We can't merge until this PR has a DCO. See: https://blog.chef.io/2016/09/19/introducing-developer-certificate-of-origin/
Let me know if I can help at all!

Hi @ChefAustin! Thank you for noticing this link issue and submitting a PR to fix this. Could you please add a DCO to your commit? We can't merge until this PR has a DCO. See: https://blog.chef.io/2016/09/19/introducing-developer-certificate-of-origin/
Let me know if I can help at all!

Fixed (I think), @mjingle.

Hi @ChefAustin, this PR is currently blocked in the pipeline as this PR needs to be rebased. The pipeline fix was merged yesterday, so you should be golden after this rebase. Thank you for your patience while we had sorted this out.


packaging + module request: FreshRSS

Project description
FreshRSS is a self-hostable RSS aggregator web application. It is written in PHP.
Metadata

homepage URL: https://www.freshrss.org/
source URL: https://github.com/FreshRSS/FreshRSS
license: AGPL3
platforms: unix, linux, darwin (everything with PHP)

Motivation: I am currently using  TinyTinyRSS (tt-rss) for aggregating my feeds, but I'd like to move away from that for 2 reasons and FreshRSS is a viable, actively developed alternative:

tt-rss doesn't do proper releases, you just have to pull it from the master branch when installing
The tt-rss community (especially the maintainer) find the concept of gas chambers and the Holocaust funny. I personally do not feel comfortable using a software developed with an edgy-antisemitic mindset.

I might get around to package FreshRSS myself at some point, but am currently busy with other packages.

I am interested in this for the same reasons! Additionally, FreshRSS has the benefit over tt-rss of an excellent mobile layout.

Closed via #190172


Give IntoFieldError::into_field_error access to Context

In the app I'm working on we want to track all errors in Rollbar. The way we're currently doing that is by having a custom error type (AppError) and implementing IntoFieldError for that type. In into_field_error we then make an API call to Rollbar and return a FieldError.
This works well however we would like for the data we send to Rollbar to contain meta data such as the current user's id and the operation name(s). This is a bit problematic because IntoFieldError::into_field_error doesn't have anyway of getting to the Context, which is where we store these things.
We solve that by having a type that wraps a normal error and has the data from the Context we need. Something like:
pub struct ReportableError {
    error: AppError,
    current_user_id: Option<i32>,
    operation_names: Vec<String>,
}

We then have some setup for creating those errors from a Result<_, AppError> and the Executor. Something like:
let result: Result<_, ReportableError> = some_result_value
    .map_err_to_reportable(executor);

This does work but I'm thinking we could simplify things quite a bit if IntoFieldError::into_field_error got the Context or Executor as an argument. That way I imagine we wouldn't need a wrapper type and the conversion should happen automatically.
What do you think?
I'm guessing this would be a breaking change.

I wouldn't mind giving this a shot if you think its a good idea 😊

We definitely want a way to do this, but this is closely correlated with things like general logging and tracing.
My preferred solution to this is moving towards a stateful Executor, which makes adding things like error/reporting/tracing hooks easier.

Fair enough. I'll close the PR.


Nodejs v5.7.0 is out

https://nodejs.org/en/blog/release/v5.7.0/

Thanks for notification! I appreciate it.
https://github.com/kazuhisya/nodejs-rpm/commit/ab8741bdeba3a18977ba3e9afdfaf5ead96f3866


feat: Implement cross_validate functionality

Introduces a new item type, CrossValidateItem, as well as a top-level function skore.cross_validate, which detects the type of ML task based on the input estimator and target, adds some scorers, and runs scikit-learn's cross_validate with those extra arguments.
The results are stored in the Project given as input, always under key cross_validation, along with an interactive summary plot.
The output of skore.cross_validate is as close as possible to scikit-learn's, except for some edge cases. Also, if the user passes scorer="my_metric", the output dict will contain key test_my_metric as well as test_score, whereas in the scikit-learn version the dict only has test_score.
TL;DR:

Make Altair a mandatory dependency
Add CrossValidateItem
Add cross_validate function
Add plot_cross_validation function
Refactor tests (add fixture in_memory_project and use that instead of project for clarity -- it would have been great to do from conftest import in_memory_project as project but pytest doesn't work that way)

How to test:

I'm in a notebook/VSCode, I have a sklearn estimator, I run skore.cross_validate on it just like I would with sklearn.model_selection.cross_validate, except I also give the argument project=project to let Skore know where to store things (if the argument is not passed we'll do cross-validation but nothing will be saved)
A plot appears in the UI and in my notebook
I forgot a metric; I change the cell and re-run
A new plot is outputted and the one on the UI updates

Addresses the first part of #383

To-do:

[x] Add Marie's tests as automated tests
[ ] Potentially use pytest-cases for cross-validation tests
[x] Deal with multi-class classification (use ["roc_auc_over_weighted", "neg_log_loss", "recall_weighted", "precision_weighted"])
[x] Deal with the case where X and y are not numpy arrays

At least set up the infrastructure to easily support pandas, numpy and lists; if only numpy works for now, that's okay
Look into struct.pack

[x] Remove metrics depending on e.g. if estimator has predict_proba

~Note for example SVC has the method defined, but it raises if the model isn't initialized with probability=True, so a try-except will be more accurate than hasattr~

[x] Deal with the case where the estimator params are not serializable

Use repr for the time being

[x] Make it so when the user passes metric="my_metric", the output contains test_my_metric as well as test_score
[x] Ensure that the stored cv_results are serializable

[x] Serialize the numpy arrays
[x] Deal with the case where the user passes return_estimator=True (we can't pass that to CrossValidateItem because it's not serializable)

Asked Marie

[ ] Confirm that the get() output is appropriate

[x] Fix plot not updating in the UI
[x] Remove default metrics for clustering
[x] Stop storing the plot directly in CrossValidateItem, instead store JSON

Rename the attribute to plot_serialized, add a property plot

[ ] ~Investigate Marie's issue of the empty plot~

hi!
When I did make install-skore to test this PR, some files were created in the frontend folder. Git wants to follow these files. If they do not exist already in the folder, I supposed it should be in gitignore?

At first I tried this (cf screenshot). I understood after that I had to do what is commented.

Is there a technical constraint to prevent us from having the cross_validate function at the root of the lib? would it be bad practice?

It's super cool to see this live :star_struck:
I tested:

creating an error in scikit-learn --> the error message is passed correctly!
a regression problem (with diabetes dataset & lasso)
a multi-class classification problem (with iris & random forest)
a binary classification problem
a classification problem

Questions and remarks:

Can we drop score_time? I don't find this metric very interesting, and I have the feeling that it overloads the options
What is the test_score?
The plot cross_validate in skore ui doesn't update
in the classif multiclass. I hadn't think of this usecase before actually. The default values for binary and multi-class shouldn't be the same.
a. the default I chose aren't correct, it creates an error. let's create an additional elif, and instead of recall and precision, use recall_weighted and precision_weighted.
b. Because the default aren't correct, there was an error in the process. Yet, some part of cross_validate in scikit learn side ran to the end, and we have some results. Is it normal not to have at least these results displayed? (cf screen shot below).
Again, for classification, the default are not correct on my side. The silhouette score exists in scikit-learn, but you have to pass it to cross_validate through a callable, and not a string. I'm surprised, so I would tend to say that we shouldn't use it as default. @sylvaincom what do you think about this? (I asked also in OS chan in slack).

I tested:

creating an error in scikit-learn --> the error message is passed correctly!
a regression problem (with diabetes dataset & lasso)
a multi-class classification problem (with iris & random forest)
a binary classification problem
a classification problem

That's really great, thanks! It would be really useful to add these use-cases in our tests directly, let's sync to do this.

Can we drop score_time?

It depends if we want to uphold the requirement that our cross_validate returns the same as scikit-learn's. IMO it makes sense to follow the principle of least surprise, so keep it.

What is the test_score?

If you don't pass any scoring parameter (or if you pass a single metric, like scoring="r2"), this metric is the one shown in test_score. If you don't pass scoring, in many cases scikit-learn can make a guess at what metric to use.

The plot cross_validate in skore ui doesn't update

Will investigate.

Because the default aren't correct, there was an error in the process (cf screen shot below).

This is not normal; let's sync to reproduce this.

Let's remove all default scores for clustering

Got it!

It looks like neg_brier_score requires the estimator to have a predict_proba method. I guess we can check the estimator beforehand and remove this scorer if don't have that method.

@MarieS-WiMLDS The scorer recall_weighted does not exist; did you mean recall_score(average="weighted")?

@MarieS-WiMLDS What should be the output of project.get("cross_validate")? For now, it outputs something like what scikit-learn outputs ({"test_score": ..., ...}), but this doesn't work when the user e.g. sets return_estimator=True, because skore is not meant to store anything and everything. An option could be to return the altair plot, another could be something like the scikit-learn outputs but without the non-serializable things...

For multi-class classification, the following metrics should be used to be compliant:

roc_auc_ovr_weighted
recall_weighted
precision_weighted
neg_log_loss

hellow!
It looks like it's going to be published pretty soon :D!
I spotted a few elements:

the output produced by regression is not consistent with the output done by scikit-learn (cf code below)
I don't know if it's possible, but if: would it be possible to reset zoom when we change the metric? Otherwise since they are not the same magnitude sometimes, we completely loose the track of the new metric we want to display.
The cross_val plot in the UI doesn't update if the view is open. If I open another one and go back to the first, it works, but it doesn't update automatically like other variables.

from sklearn.datasets import load_diabetes
from sklearn import linear_model
from skore import cross_validate
from sklearn.model_selection import cross_validate as cross_validate_sk
diabetes = load_diabetes()
X = diabetes.data[:150]
y = diabetes.target[:150]
lasso = linear_model.Lasso()
cross_validate_sk(lasso, X, y, cv = 7, scoring = "r2")
cross_validate(lasso, X, y, cv = 7, project = project, scoring = "r2")

We need documentation about the feature, there are many interesting reasons why it has been designed like this, and we should reflect that.
@MarieS-WiMLDS just checking we are in sync as @sylvaincom is working on the larger documentation.

Hi, here are some important fix:

[ ] For linear regression, it is redundant to say test_r2 and test_score.

For the fit time:

[ ] We must specify the units (seconds for example) of the fit time
[ ] and make the fit time more readable: 0.0025 seconds is not readable, use this for example:

import datetime
print(str(datetime.timedelta(seconds=1_000)))

[ ] We must distinguish scores (the higher, the better) and errors (the higher, the worse), for example this is an error but the title is still "Cross-validation scores" and the y-label is "Score": this is misleading

Maybe we can say something like "Cross-validation results" and the y-label is "Results", so that it can be appropriate for scores, errors, and times

[ ] Why don't we include the test time and the train scores?

Agreed, datetime.timedelta displays nicely for large numbers of seconds but not decimal seconds
-> Maybe writing the following format: 1h23min or 23 ms is nice

@sylvaincom @augustebaum
It feels like we can harvest some comments and put them in a fresh issue. Please cutoff the feature scope, so dev stops here, unless the last minute change must absolutely be included in Tuesday's release..

Follow-up issue: https://github.com/probabl-ai/skore/issues/544

👏


Hi, Is it possible to add the ARN of the Lambda Layer to my AWS Lambda Container Image

For more context, I'm AWS lambda Container Image to handle my lambda function

Hey @iamcristos !
This sounds like an interesting use case. I went online and found this blog post in the "Copy the contents of a Lambda layer into a container image" section which discusses steps that I think will be helpful to you.
For example, if you are using OTel Python Container images, you would want the OTel Python Lambda Layer ARN. Then, you could do
curl $(aws lambda get-layer-version-by-arn --arn arn:aws:lambda:us-west-2:901920570463:layer:aws-otel-python38-ver-1-7-1:1 --query 'Content.Location' --output text) --output layer.zip

to get the contents of the Lambda Layer and add them to your Lambda function.
Another solution is to create the Lambda function with the Docker Image, and then use the aws cli afterwards to add the Lambda Layer. This is also explained in the blog post.
aws lambda create-function --region us-east-1 --function-name my-function \
    --role arn:aws:iam::123456789012:role/lambda-role \
    --layers \
        "arn:aws:lambda:us-west-2:901920570463:layer:aws-otel-python38-ver-1-7-1:1" \

Otherwise, if you were looking for a setting that added it at once in the Docker Image it seems that the Docker Images don't support that. If that is a feature you want to request I recommend looking at the aws-lambda-base-images GitHub Repository where you can voice your use case 🙂
Hope that helps!


display issues with iPad

this QR code used to be full of screen

@zhangzhongnan928 The distorted ERC721 image issue has been fixed in a recent TestFlight version.
Do you still want to make the QR code occupy the same width? Might be harder to scan because the user doing the scanning has to hold the scanner further away, especially for 12.9"

no, all good now


review overbracket, underbracket (and maybe others)

Much like #23, our definition isn't quite right (lacking the "stack" boolean for mathjax's UnderOver method).
However, it turns out mathjax now has a core macro providing these - from the mathtools extension. In addition, mathtool's definition is quite a bit more involved:
https://github.com/mathjax/MathJax-src/blob/f939f2ff2c4678785e9bf3d3b17e8c6151f0ce9c/ts/input/tex/mathtools/MathtoolsMethods.ts#L289-L312
We could consider if and how to revise it here.

(lacking the "stack" boolean for mathjax's UnderOver method).

To clarify, I wrote this because IIUC
https://github.com/mathjax/MathJax-src/blob/f939f2ff2c4678785e9bf3d3b17e8c6151f0ce9c/ts/input/tex/mathtools/MathtoolsMethods.ts#L310
does what "1"
https://github.com/mathjax/MathJax-src/blob/8565f9da973238e4c9571a86a4bcb281b1d98d9b/ts/input/tex/base/BaseMappings.ts#L514
Then again, I'm not sure if unicode-math has these working this way cc @davidmjones

Oh, and certainly the constructions here don't work well with the default mathjax v3 fonts. Not sure about v4.


Store route on trie node

This reverts #66 in favor of storing the route on the trie node. Using a proxy callback worked for mitigating the immediate problem with reusing route callbacks but it also makes it difficult to register anything other than functions as route callbacks.
I.e. if I was to register a class as a route callback I'd probably want that class returned from .match and not a wrapper function. This is relevant if you'd want to register top level components as routes in choo.
This does however break the undocumented functionality of accessing the route on this in the callback. I dunno what that means for semver 🤔

This does however break the undocumented functionality of accessing the route on this in the callback. I dunno what that means for semver thinking.

Ghmmm, yeah good one. We did make for an explicit patch to add this at some point - but I don't believe I ended up using it anywhere in Choo source.
I'm leaning towards ignoring it, and we'll hear it if it breaks. It's a quite small feature, and like you're saying: undocumented.

We can conservatively do a major for this and use it in Choo 7 @tornqvist


[vcpkg] Error when building a port with certain feature

Describe the bug
When a certain port has multiple features and there is a dependency relationship between feature and feature, the feature that installs the port's dependency and other features will generate an error:

Package *** does not have a *** feature.

Example: protobuf-c has feature tools and test, and feature test depend on tools.
Environment

OS: [e.g. Windows/Linux etc...] All triplets
Compiler: revision

To Reproduce
Steps to reproduce the behavior:

./vcpkg install protobuf-c[test]
See error
Repro code when

Expected behavior
Port should be built succesfully as expected.
Failure logs
N/A
Additional context
N/A

https://github.com/microsoft/vcpkg/blob/e792d1c171a18d32401db0f95251515cc15056f5/toolsrc/src/vcpkg/dependencies.cpp#L683
It looks like I found the wrong port and feature matching value here.

try changing:
protobuf[tools]
to
protobuf[core,tools]

@Neumann-A Not working.

Well... I noticed the feature test dependency is wrong now...


Windows device guidelines missing

In the beginning of the doc we have 3 examples of devices - Windows, Android, or iOS.
Though from that point forward we're missing guidelines for the windows device.
Please let us know where we can find them in the docs.

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

ID: de2a4498-fdfc-638a-5afd-ef2905f8f4a3
Version Independent ID: 077aef5e-b022-434d-1ad5-64f756df04d6
Content: Certificate-based authentication - Azure Active Directory
Content Source: articles/active-directory/authentication/active-directory-certificate-based-authentication-get-started.md
Service: active-directory
Sub-service: authentication
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@bpcastilho
Thanks for your feedback! We will investigate and update as appropriate.

Thanks for the feedback, @bpcastilho. Is there something specific you're look for for Windows? The first section of testing the certificate is for generic browser-based apps on a device, like a Windows device - https://docs.microsoft.com/en-us/azure/active-directory/authentication/active-directory-certificate-based-authentication-get-started#testing-your-certificate. The iOS and Android test sections are again just to validate you can connect as expected using certificate-based auth.
There's no specifics on installing the certificate on any device - Windows, iOS, or Android. The expectation is that you'd use whatever mechanism currently in place to distribute those. This article is more focused on how to generate and configure certificate for use with Azure AD.
#in-progress

If there's additional feedback, @bpcastilho, we can review this. Otherwise, for now #please-close


Rayquaza's mega not working

When in a battle you click the mega button and choose a move, when rayquaza attacks, it doesn't transform in his mega form.
Necrozma infused with lunala/solgaleo with ultranecrozmanite-z, when in a battle you click the mega button and choose a move it doesn't work
Ign kryla
Server titan

These are both known issues with the Pixelmon Mod itself, there's nothing we can do for them serverside, unfortunately. Closing.


Fixes race condition when creating and deleting containers

Does not add containers to the ContainerService.containers list until they are done "initializing"
Marks them as Destroyed immediately after Stop was called
Handles filtering GetContainers in the race condition that the container was deleted whilst filtering the list

[#93802546]
Signed-off-by: Dave_Goddard<EMAIL_ADDRESS>

it's been a while with no activity, besides the tracker story referenced in the pr has been closed.


Needed Search filter for both student-portal-all-classes/:email and student-portal-my-classes.email

We need who implement a search filter for the all programs and my classes page.

https://github.com/Pmacdon15/final_project_web/pull/45
https://github.com/Pmacdon15/final_project_web/pull/42


The docker-compose file not meet the network topology.

The docker compose define port 5000 for the Admin Api but the admin api is configure to use 5001 port, and also put the 80 port to the STS site, it should use 5001 for Api and 5000 for STS

This was rewritten - please check master branch. Thank you.


metasploit: 6.0.46 -> 6.0.47

Motivation for this change
Update to latest upstream release 6.0.47
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 125768 run on x86_64-linux 1

1 package built:

msf (metasploit)

Result of nixpkgs-review pr 125768 at ca96bed8 run on aarch64-linux 1

1 package built successfully:

metasploit

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/security/metasploit/default.nix:24:3:
   |
24 |   installPhase = ''
   |   ^


InventoryCleaner

Mod doesn't seem to work, when I do the commands (/ic additem <item> or /ic delitem <item>) it doesn't change anything, I also can't find the inventorycleaner.properties file in the config folder.

could you tell me the version

InventoryCleaner 1.0.4 for Minecraft 1.20.2 with fabric

So the problem is that I didn't add the commands in that version you need to use a newer one

alright, I'll use a newer version, thanks you!


retry S3 request for storage.get and storage.exists

Typically AWS SDKs have retry logic in place on certain errors, it seems like rusoto does not:
https://github.com/rusoto/rusoto/issues/234
This PR retries some requests and should prevent https://sentry.io/organizations/rust-lang/issues/2790801747/ .
The official AWS S3 SDK does do better retries, but still counts as developer preview ( but it works ).

r? @Nemo157 any opinion @jyn514 ?

Makes sense to me. I'm not sure if there are other error variants we should retry though, the RusotoError docs are very lacking in saying what they actually mean.

This handles at least the ones I'm seeing in sentry, IMO that's enough to start with.
We always can add more.


Double prompt

The problem
My terminal is rendering 2 prompts when initially loading the terminal.

with SPACESHIP_PROMPT_ADD_NEWLINE=false

with updated lib/prompts -> added just echo before outputting the prompt

Relevant Zsh configuration
First I thought it has something to do with `SPACESHIP_PROMPT_ADD_NEWLINE`, but when I set `SPACESHIP_PROMPT_ADD_NEWLINE=true` (or leave the default) I get an empty line and prompt on the second lint (correct). But when I change it to `false`, then I get 2 prompts on each line. I have tinkered with the below code and I have no idea what's happening there, because when I put extra `echo -n "TEST"` before the output, then it doesn't render twice, but when I don't log anything witch echo, it renders the prompt twice??

// lib/prompts.zsh
spaceship::prompt() {
  _spaceship_prompt_opened="$SPACESHIP_PROMPT_FIRST_PREFIX_SHOW"

  # Compose prompt from the order
  local prompt="$(spaceship::core::compose_order $SPACESHIP_PROMPT_ORDER)"

  # Allow iTerm integration to work
  if [[ "${ITERM_SHELL_INTEGRATION_INSTALLED:-}" == "Yes" ]]; then
    prompt="%{$(iterm2_prompt_mark)%}${prompt}"
  fi

  # Should it add a new line before the prompt?
  if [[ $SPACESHIP_PROMPT_ADD_NEWLINE == true ]]; then
    prompt="\n${prompt}"
  fi

  # Print the prompt
  echo -n "$prompt"
}

renders on first load

```bash
~$
~$

// lib/prompts.zsh
spaceship::prompt() {
  _spaceship_prompt_opened="$SPACESHIP_PROMPT_FIRST_PREFIX_SHOW"

  # Compose prompt from the order
  local prompt="$(spaceship::core::compose_order $SPACESHIP_PROMPT_ORDER)"

  # Allow iTerm integration to work
  if [[ "${ITERM_SHELL_INTEGRATION_INSTALLED:-}" == "Yes" ]]; then
    prompt="%{$(iterm2_prompt_mark)%}${prompt}"
  fi

  # Should it add a new line before the prompt?
  if [[ $SPACESHIP_PROMPT_ADD_NEWLINE == true ]]; then
    prompt="\n${prompt}"
  fi

  # Print the prompt
  echo "TEST" # Doesnt work when I put -n , then it renders even the TEST twice??
  echo -n "$prompt"
}

renders on first load
TEST
~$

### What is version of Spaceship are you using?

v4.5.0

### What is version of Zsh are you using?

zsh 5.8

### What operation system are you running?

macOs

### Do you use any Zsh framework or plugin manager?

_No response_

### Which terminal emulator do you use?

iTerm

### Additional information

_No response_

Here is what the $prompt prints
%{%B%F{white}%}%0~%{%b%f%}%{%B%} %{%b%}%{%B%F{green}%}$%{%b%f%}%{%B%} %{%b%}

Hey! Thanks for reaching out. This seems like a duplicate of #1177


How to call Python automation code from a UI button?

I saw your code and it was very helpful. Thanks a lot for sharing it.
Can you please guide me on how can I call this python automation code from a UI button in Unreal Engine? I want to create a plugin which will show a UI button and I want to call my python automation script when this button is pressed.
Thanks in advance.

Hey, I'm trying to do something similar, but with text. Can you let me know if you were able to get this done, how you did it?

It's not easy. I had to use C++ to create a UI button and then from there it called the python automation script.

@avidews If you still have the script could you upload it here? I'm not trying to use an automation, I'm just trying to get text from a UEditableTextBox and if your script has to do with interfacing with UI elements I think it could help point me in the right direction.

Really sorry for late reply. I am also finding a solution, still.


Empty config file / no spaces

Config files are empty when written. The restrict.properties file does not contain anything. I am able to directly add config options but if there are any spaces around the equals sign it wont work
allow-guests = true # doesnt work
allow-guests=true # does work

Should now be fixed in 2287c05252b70662aee945773dfe8c3977578ec6

@DeathCradle I will test this out tonight. Thanks bud.

Tested this out, looks like all is well.

Thanks


Allow linting to run on both Windows/Mac

Fix lint script in package.json so it works on both Windows and Mac.

@kswenson agreed.  I extended it to the other lint scripts.


Failure to add asset

What version of Wallet are you using?
4
What is happening?
Asset 0x1bdeed96ee1e5eca0bd1d7eeeb51d03b0202c1faf764fec1b276ba27d5d61d89 decimals is not valid
steps to reproduce
Trying to add asset on swayswap, tried to follow instructions from bot on discord server
quoted. To add an asset to your wallet, you can use the addAsset function with the asset's details like this: const asset = { assetId: BaseAssetId }; const isAdded = await fuel.addAsset(asset); [1]. If you're using a user interface, you can click the "Add Asset" button and provide the asset ID, name, symbol, and image URL to add a new custom asset [2].
How it should behave?
Asset adding to portfolio
What os are you seeing the problem on?
Windows Desktop
What browser?
Chrome
What is your browser version?
Version 120.0.6099.200 (Official Build) (64-bit)

Thanks for opening an issue, SwaySwap is no longer maintained, this is why you experienced the issues. We have updated the project readme to better inform other users.


Huge changes to IElasticClient in NEST 7.0 beta 1, is that on purpose?

NEST/Elasticsearch.Net version:
7.0 beta 1
Elasticsearch version:
7.1.1
Description of the problem including expected versus actual behavior:
Bunches of Methods were removed from IElasticClient, e.g. Alias, IndexExists
Steps to reproduce:

Install NEST 7.0 alpha 2 Nuget package, those methods were there.
Install NEST 7.0 beta 1 Nuget package, those methods were gone.

Provide ConnectionSettings (if relevant):
n/a
Provide DebugInformation (if relevant):
n/a

@lummar I answered your question on https://github.com/elastic/elasticsearch-net/issues/3563#issuecomment-501564124

IndexExists -> Indices.Exists
Alias -> Indices.BulkAlias

Client.Refresh(Nest.Indices.All); -> Client.Indices.Refresh(Nest.Indices.All);

@russcam thank you very much


Add GitHub Actions

@RamLavi it takes 20 minutes due to emulation mode, but tests then finish. BTW this PR also fixes an issue in the test script.


Add Equipment Inspector plugin

This plugin adds a "Inspect Equipment" option to the right-click player menu which opens a side-panel displaying the equipment that player currently has equipped.

I often find myself seeing some gear or some nice fashionscape and wondering what that player was wearing but I'm not able to find out because the player is afk or has public chat off. So I decided to create this plugin to address that issue.
I posted this in the Discord and received feedback that it could be potentially OP for PKing, particularly as it's possible to see item charges, durability for barrows armour etc. This could maybe be addressed by hiding charge/durability info or adding code to disable it in the wilderness. I'd be interested to hear other thoughts on this, as I myself am not a PKer so I'm not fully aware of how the plugin would be advantageous in those scenarios.
The plugin seems to work most of the time, but quite often it seems that players which are loaded and on screen are missing from the player cache and therefore the plugin is unable to get their equipment. Would be grateful if anyone could offer any guidance on this, not sure why this is happening. Fixed in f63143e.
The plugin is disabled by default to avoid cluttering the sidebar for players not using this plugin.

Small nitpick but I would change "Inspect Equipment" to "Inspect", to match the length of other right click ops on players

I dont think this plugin should be implemented, seems quite op for e.g scouts when pking.

Yeah I abandoned this based on the consensus that it’ll never be added.
On 2 Sep 2018, at 7:37 pm, Tomas Slusny <notifications<EMAIL_ADDRESS>wrote:
I dont think this plugin should be implemented, seems quite op for e.g scouts when pking.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frunelite%2Frunelite%2Fpull%2F3733%23issuecomment-417950660&data=02|01||7545d1409343431a64fe08d611032d85|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636715102668980980&sdata=2NZiYcoik5fkQf2lwnK3xhWvay4tFh8ewrUag%2FXol5Y%3D&reserved=0, or mute the threadhttps://eur04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACBTFrWA16l12uLQORJnUjCAjcSGCuMOks5uXCV4gaJpZM4Uh27z&data=02|01||7545d1409343431a64fe08d611032d85|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636715102668980980&sdata=EDoKeqe%2BvTyHPCCYQAdMCjoKfbuqiwVbOA5jmj4GE6k%3D&reserved=0.

Is it really that op though? It doesn't give you any info you cant get just by looking at the person. (As long as items with charges are all displayed as the fully charged version)

Well sometimes you can't see stuff like dboots under void etc, happens with plethora of items

IMO this is fine and the pros outweigh the cons, especially since it doesn't count risk and can't show ring and arrow slot. Most of the time people have switches in the wilderness, anyway.

Hmm maybe, its hard to evaluate. Either way, if we would decide to implement this imo it should be part of the hiscore plugin with just icons and names as tooltips probably.

I would like to see the official equipment interface (or similar in-game interface) be used for this, to be honest. Sidebar feels like the wrong place for this.

Ah right, yeah that would be even better

What if you could disable the feature in the wilderness and pvp worlds?
this would prevent using it to see boots under void, quickly easily determine someone's risk and scouting.
Checking if player is logged in to a pvp world should be doable, but no idea if you can check if other player is in the wilderness while localplayer is not to prevent it being abused from the other side of the ditch.

Now that the plugin hub has been created, this is a plugin which would be well-suited for inclusion there.
That aside, is the HEAD KitType change still valid?

added to pluginhub https://github.com/runelite/plugin-hub/pull/105


Detail Instruction about installation Cloud-Portal

Hello, do you have a document about details instructions for installing Cloud-Portal? I would really appreciate if you can provide one.
Thank You

Please have a look at the prerequisites section of the README.md file:
https://github.com/chrisipa/cloud-portal#prerequisites
Installation is basically pretty easy. You can either use the Ansible installer or start from scratch:
https://github.com/chrisipa/cloud-portal/blob/master/docs/installation/README.md
In case you are starting from scratch you have to install docker & docker-compose on your Linux machine and follow these instructions:
https://github.com/chrisipa/cloud-portal/blob/master/modules/cloud-portal-server/README.md#Usage
Are you still missing some details in the documentation? Please let me know.

Closed due to inactivity ...

Sorry for the late reply.. Going to give another shot again.. I will let you know how it goes.. thanks again


Provide a function for fetching schema from Schema Registry using group name, schema name and schema version.

Currently, I am fetching the schema from the Schema Registry using the following endpoint
/$schemaGroups/{groupName}/schemas/{schemaName}/versions/{version}
There is no function provided in the azure.schemaregistry.SchemaRegistryClient class for pulling the schema from Event Hub Schema Registry using group name, schema name, and version.
Can we have a function for the same?
Also, let me know if you are open to a PR?

Hi @harshdeep78singh, thank you for opening an issue! I'll tag the appropriate folks so we can answer your question about the potential availability of this API.
As for opening a pull request, we're always happy to accept PRs 🙂 For something like adding a client method there will need to be some design discussion, but opening a PR could kickstart that conversation! I'd recommend waiting just a bit before doing any work so that someone who works with the service can provide some input in this case.

Hi @harshdeep78singh - Thanks for bringing this to our attention! I have discussed this with the team and we will be adding this functionality asap. I saw that you created a PR and took a look.
Unfortunately, we will not be able to merge this, because the changes you made were to our "generated" code. This code is automatically generated from our internal "swagger" file that describes operations that we allow. So, ultimately, the changes that you made will actually be added upon re-generation after we update our swagger file to match. Really appreciate your efforts here, however.
We are currently making changes to the swagger file and will be having API design discussions after that. I'll keep you updated as we continue to make progress. Thanks again!

Hi @harshdeep78singh - Appreciate your patience here! I just merged the feature. (sample [here]), and it'll be out in our next release. I'll update this issue as soon as it's out.
In the meantime, if you're interested, you can manually build and install the wheel:

azure-sdk-for-python/sdk/schemaregistry/azure-schemaregistry> python setup.py bdist_wheel -d [dest_folder]
[dest_folder]> pip install azure_schemaregistry-1.2.0-py3-none-any.whl

Thanks!

Thank you for the update

@swathipil thank you for the update

Hey @harshdeep78singh! After a review from our architects, the API to get schema by version has been updated. Rather than a separate get_schema_by_version method, the get_schema method now takes group_name, name, and version as keyword arguments: get_schema(group_name=group_name, name=name, version=version). The updated sample is [here)].

azure-schemaregistry 1.2.0 has been released: https://pypi.org/project/azure-schemaregistry/1.2.0


9054 DEV Add migration query to deprecate old ssh-key integration in master_integrations.sql (set isDeprecated to true for ssh-key).

https://github.com/Shippable/pm/issues/9054

Verified on rc.


Category listing with images (feature request)

Request for the categories module to be extended with the option to set a grid layout similar to that for products with images above titles, as often used on ecommerce sites.
Also to be able to create a hierarchy of product categories and sub-categories where the categories module would display just the sub-categories of the selected category.

I've found that the Content Categories module in the shopmag template shows a grid layout with images. Should that module be adopted into userfiles/modules/ ?
It would be nice to extend the Content Categories module with the same configuration options as found in the Categories module, or merge the two by having a 'Grid' template layout in the Categories module.

Looks like this is now included in userfiles/modules/categories/categories_images however I notice that the admin popup does not set the selected parent category. In userfiles\modules\categories\category_images\admin.php the $selected_category parameter is populated with the saved parent category id but is not used to set the selcted option.
I notice for the categories module in userfiles\modules\categories\admin_live_edit.php that there is an extra line $pt_opts['active_code_tag'] = '   selected="selected"  '; but this line made no difference when added to the category_images module.

Adding the following lines in userfiles\modules\categories\category_images\admin.php after $pt_opts['include_first'] = 1; seems to correctly set the selected parent category in the admin popup:
$pt_opts['active_ids'] = $selected_category;
$pt_opts['active_code_tag'] = '   selected="selected"  ';


function create no longer works with latest knative/build

Something has changed to the build part of the kservice
Error: Internal error occurred: admission webhook "webhook.serving.knative.dev" denied the request: mutation failed: cannot decode incoming new object: Object 'Kind' is missing in '{"serviceAccountName":"riff-build","source":{"git":{"revision":"master","url":"https://github.com/trisberg/upper.git"}},"template":{"arguments":[{"name":"IMAGE","value":"gcr.io/cf-spring-funkytown/upper-s1p"}],"name":"riff-cnb"}}'

Created issue in knative/serving - https://github.com/knative/serving/issues/2096

I just tried again today, seeing that knative/serving#2096 is closed. It seems to work now.

Actually, I may have closed this a bit hastily... @trisberg: does riff@master currently reference the latest knative manifests?


Support Safari

I would love to see this as well. I'm not 100% sure, but I think you'd need to create an app in the app store. I would be more than willing to pay a few dollars for it to help cover the developer fee and support development. A similar open source extension, Dark Reader, sells for $1.99
https://itunes.apple.com/us/app/dark-reader-for-safari/id1438243180?mt=12

The problem is not with the fee. I've paid for Chrome as well.
Actual problem is that I don't use neither Safari nor Mac and haven't ever used.
So I basically cannot develop and test it.
PS: Dark Reader is actually $4.99 (probably it depends on a region)

The problem is not with the fee. I've paid for Chrome as well.
Actual problem is that I don't use neither Safari nor Mac and haven't ever used.
So I basically cannot develop and test it.
PS: Dark Reader is actually $4.99 (probably it depends on a region)

Darker reader can't work as well as midnight.


Form style folder structure

Is your feature request related to a problem? Please describe.

The problem is very minor but I noticed that we have a directory for form styles which includes styles for inputs and selects. However the styles for radios, checkboxes and buttons are located under the elements directory.
Describe the solution you'd like

Move the radio, checkbox and button styles into the form directory as these are primarily form/user input elements.
Describe alternatives you've considered

Button could potentially go either way, stay in elements or move to forms but I feel it would be better in forms.

How about this? 🤔
I referred to the document of bulma. 🔍👀
|- elements
|    |- buttons.scss
|- form
     |- inputs.scss
     |- radios.scss
     |- checkboxes.scss
     |- selects.scss

Yes that's exactly what I had in mind. I just wasn't sure about button.scss It's kind of a form element in that it's a user input but it's also used more generally than in forms. I can see it in either folder so it's up to you.


adding tools for interacting with hub

Adding tools for downloading a repo through http api and for creating/deleting repos through api. Script is available at the bottom thanks to @Chiiizzzy . Basic testing done for now but more testing will need to be done once hub directory format is decided.

/approve
/lgtm


Limit abilities of backend wrappers

The gist:

SecondOrder should only be able to call second-order operators (already the case)
AutoSparse should only be able to call jacobian and hessian (breaking)

Advantages:

Reduces fallback boilerplate like the hideous https://github.com/gdalle/DifferentiationInterface.jl/blob/main/DifferentiationInterface/src/sparse/fallbacks.jl
Reduces the risk of error

Drawbacks:

Users will often have to specify several backends to get things to work. For optimization or nonlinear solve:

The user gives a forward backend and a reverse backend
The package dev takes care of mixing them in the appropriate way for gradients, Jacobianss and Hessians

AutoSparse should only be able to call jacobian and hessian (breaking)

I think this is the right call. Any other operators besides the Jacobian and Hessian are non-sparse and it would be misleading to call them using AutoSparse.


[toolbar] template after-end slot has wrong part value

Describe the bug:
Toolbar template after-end slot has part="before-section-start" and should be part="after-section-end"
Expected behavior:
This should be: part="after-section-end" according to documentation
Please complete the following information:

Forge version: 3.3.1
I have searched existing issues before creating this report? Y
Browser: All
Platform: All
OS: All

Fixed in #739


New sheet tab column's autofit is not correctly setup

This probably happens because the values inside the sheet have not been set up before the column autofit is called.

This is now fixed with pull request #6


TypeVar locking in types inherited from generics

Currently it's impossible to inherit from generic types and lock/fix their type variables
According to python docs it should be possible to do so:
T = TypeVar('T')

class MyDict(Mapping[str, T]):
    ...

I created a repository with a reproducable example:
https://github.com/ThirVondukr/strawberry-generics-issue
from typing import Generic, TypeVar

import strawberry

Node = TypeVar("Node")
Cursor = TypeVar("Cursor")

@strawberry.interface
class Edge(Generic[Node, Cursor]):
    node: Node
    cursor: Cursor

@strawberry.type
class SpecificEdge(Edge[str, str]):
    pass

@strawberry.type
class Root:
    edge: Edge[int, int]
    specific_edge: SpecificEdge

schema = strawberry.Schema(query=Root)

Expected behavior: specificEdge on schema having {node: String!, cursor: String!} type,
currently error occurs.

Similar test: https://github.com/strawberry-graphql/strawberry/blob/86eba966b112c43e6e05e1cc3c030741648b75bc/tests/schema/test_generics.py#L708-L745
However, no field on Node uses the generic. Note that the _resolve is not marked with @strawberry.field. Perhaps this test has a bug

Here's a more minimal reproduction of the issue:
T = TypeVar("T")

@strawberry.type
class MyType(Generic[T]):
    field: T

strawberry.Schema(MyType[int])

Trying to find a fix now...

After investigation, we're going to defer solving this issue until we've implemented StrawberryObject. At that point we can make StrawberryObject.__class_getitem__ directly return a new concrete StrawberryObject generic copy.


fix python3 compatibility (sorted)

This one wasn't necessary (but welcome). I'm not sure why it kept showing up. I think there was a problem with a stale JENKINS_ tag.

Thanks Eric, I will merge it anyway, then.


Slow data readings

Hi, first of all: thanks for the great work on this. I have an issue with the library though. While the readings are significantly better than with the other HX711 library, I have a la in data display. It takes about 2 seconds to get to a weight reading and once the weight is removed it takes another 2 seconds to get back to zero. I have this challenge both with the example code, as well as with my own adaption of it.
Any guidance how to speed this up?
One more question: In the Readme it is stated:

Hardware and ADC noise: Wires between HX711 and load cell should be twisted and kept as short as possible.

Does this mean the dout and sck should be twisted and the gnd and vcc (in twisted pairs)?
Thanks!

Hi!
There is no benefit to twist the dout and sck wires unless they are very long (these are digital signal wires).
The settling time with default settings and the HX711 in LOW RATE is 1.8 seconds. This is a result of the HX711 sample rate (HIGH or LOW) and the settings you have in the config.h file.  See description in the config.h file:
Allowed values for "SAMPLES" is 1, 2, 4, 8, 16, 32, 64 or 128.
Higher value = improved filtering/smoothing of returned value, but longer setteling time and increased memory usage
Lower value = visa versa
The settling time can be calculated as follows:
Settling time = SAMPLES + IGN_HIGH_SAMPLE + IGN_LOW_SAMPLE / SPS
Example on calculating settling time using the values SAMPLES = 16, IGN_HIGH_SAMPLE = 1, IGN_LOW_SAMPLE = 1, and HX711 sample rate set to 10SPS:
(16+1+1)/10 = 1.8 seconds settling time.
Note that you can also overide (reducing) the number of samples in use at any time with the function: setSamplesInUse(samples).

Thank you so much for the response! What would be your recommendation with regards to the config.h in order to optimize for Accuracy & Speed?
Which wires would you twist between the load cell and HX711?

There is no general sweet spot really, it depends on your requirements and it will always be a compromice between speed vs accuracy.

Ok, thank you very much!


Make search sheet actually searches cards

Ticket: CS-5458
https://github.com/cardstack/boxel/assets/12637010/7d210a20-78f5-4b65-b207-00797df52466

I don’t know if this is part of the design but I think this would benefit from having an animation or some kind of progress indicator when search results haven’t returned yet 🤓

Good idea! I'll add it.

I don’t know if this is part of the design but I think this would benefit from having an animation or some kind of progress indicator when search results haven’t returned yet 🤓

Good idea! I'll add it.

I decided to update the label and make it a loading indicator.
https://github.com/cardstack/boxel/assets/12637010/a28db13b-7b57-453b-8144-616c3c9c1b82


BuyerContactEMail::id

While refactoring code i noticed following lines
    struct BuyerContactTelephone final {
        static constexpr std::u8string_view id{u8"BT-57"};
....
    };

    struct BuyerContactEMail final {
        static constexpr std::u8string_view id{u8"BT-57"};
...
    };

I suppose BuyerContactEMail should be with BT-58 id

Thanks for finding this. Would you maybe fix this in your refactoring branch?

Thanks for finding this. Would you maybe fix this in your refactoring branch?

Hi, yes, I will fix it there
Just wanted to make sure that this is not expected


lab 2 knit not saving

@JasonSills i'm not sure why my lab isnt uploading to knit as well. can you help me. i keep getting this,
processing file: Lab-02-Benally.Rmd
|.......................                            |  44% [unnamed-chunk-15]
Quitting from lines 287-290 [unnamed-chunk-15] (Lab-02-Benally.Rmd)
Error:
! object 'total' not found
Execution halted

@bbenall4
It looks like you have an issue with with code in lines 287-290 that is returning the error. The error is indicating an object called 'total" that was not defined. Can you provide the code chunk with the code spanning lines 287-290?

now it's saying this,
processing file: Lab-02-Benally.Rmd
output file: Lab-02-Benally.knit.md
"C:/Program Files/RStudio/resources/app/bin/quarto/bin/tools/pandoc" +RTS -K512m -RTS Lab-02-Benally.knit.md --to html4 --from markdown+autolink_bare_uris+tex_math_single_backslash --output Lab-02-Benally.html --lua-filter "C:\Users\BTB#12\AppData\Local\R\win-library\4.3\rmarkdown\rmarkdown\lua\pagebreak.lua" --lua-filter "C:\Users\BTB#12\AppData\Local\R\win-library\4.3\rmarkdown\rmarkdown\lua\latex-div.lua" --embed-resources --standalone --variable bs3=TRUE --section-divs --table-of-contents --toc-depth 3 --variable toc_float=1 --variable toc_selectors=h1,h2,h3 --variable toc_collapsed=1 --variable toc_smooth_scroll=1 --variable toc_print=1 --template "C:\Users\BTB#12\AppData\Local\R\win-library\4.3\rmarkdown\rmd\h\default.html" --highlight-style haddock --variable theme=cerulean --mathjax --variable "mathjax-url=https://mathjax.rstudio.com/latest/MathJax.js?config=TeX-AMS-MML_HTMLorMML" --include-in-header "C:\Users\BTB#12\AppData\Local\Temp\RtmpMx6wrD\rmarkdown-str25e07181777b.html"
Could not find data file templates\default.html
Error: pandoc document conversion failed with error 97
Execution halted

@bbenall4
Is R and R Studio saved on your network drive? If so, you will want to uninstall and reinstall on your c drive.
Here is more information:
https://community.rstudio.com/t/error-message-when-i-try-to-knit-the-r-markdown-code/137455/6
https://github.com/yihui/knitr/issues/855


CSS definition for multiple animations that is compatible with postcss

For fixing errors in builds with postcss, example:
  701 |   -webkit-animation:
  702 |         circular-indeterminate-bar-dash___3DSlU 1.5s ease-in-out infinite,
> 703 |         colors___Z_PDt 6s :local(ease-in-out) :local(infinite);
      |         ^
  704 |           animation:
  705 |         circular-indeterminate-bar-dash___3DSlU 1.5s ease-in-out infinite,

    at Input.error (/app/node_modules/postcss/lib/input.es6:101:16)
    at Parser.error [as checkMissedSemicolon] (/app/node_modules/postcss/lib/parser.es6:513:22)
    at Parser.checkMissedSemicolon [as decl] (/app/node_modules/postcss/lib/parser.es6:240:46)
    at Parser.decl [as other] (/app/node_modules/postcss/lib/parser.es6:108:18)
    at Parser.other (/app/node_modules/postcss/lib/parser.es6:56:16)
    at parse (/app/node_modules/postcss/lib/parse.es6:8:12)
    at new parser (/app/node_modules/postcss/lib/lazy-result.es6:41:16)
    at Processor.<anonymous> (/app/node_modules/postcss/lib/processor.es6:109:12)
    at Processor.process (/app/node_modules/postcss/lib/processor.js:119:23)
    at Function.process (/app/node_modules/postcss/lib/postcss.es6:126:43)
    at OptimizeCssAssetsWebpackPlugin.processCss (/app/node_modules/optimize-css-assets-webpack-plugin/src/index.js:65:21)
    at Object.processor (/app/node_modules/optimize-css-assets-webpack-plugin/src/index.js:11:18)
    at each (/app/node_modules/last-call-webpack-plugin/src/index.js:150:10)
    at arrayEach (/app/node_modules/lodash/_arrayEach.js:15:9)
    at forEach (/app/node_modules/lodash/forEach.js:38:10)
    at OptimizeCssAssetsWebpackPlugin.process (/app/node_modules/last-call-webpack-plugin/src/index.js:147:5)
    at compilation.hooks.optimizeChunkAssets.tapPromise.chunks (/app/node_modules/last-call-webpack-plugin/src/index.js:178:28)
    at _err0 (eval at create (/app/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:20:1)
    at taskRunner.run (/app/node_modules/uglifyjs-webpack-plugin/dist/index.js:315:9)
    at step (/app/node_modules/uglifyjs-webpack-plugin/dist/TaskRunner.js:76:9)
    at done (/app/node_modules/uglifyjs-webpack-plugin/dist/TaskRunner.js:84:30)
    at tryCatcher (/app/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/app/node_modules/bluebird/js/release/promise.js:512:31)
    at Promise._settlePromise (/app/node_modules/bluebird/js/release/promise.js:569:18)
    at Promise._settlePromise0 (/app/node_modules/bluebird/js/release/promise.js:614:10)
    at Promise._settlePromises (/app/node_modules/bluebird/js/release/promise.js:693:18)
    at Promise._fulfill (/app/node_modules/bluebird/js/release/promise.js:638:18)
    at Promise._resolveCallback (/app/node_modules/bluebird/js/release/promise.js:432:57)
    at Promise._settlePromiseFromHandler (/app/node_modules/bluebird/js/release/promise.js:524:17)
    at Promise._settlePromise (/app/node_modules/bluebird/js/release/promise.js:569:18)
    at Promise._settlePromise0 (/app/node_modules/bluebird/js/release/promise.js:614:10)
    at Promise._settlePromises (/app/node_modules/bluebird/js/release/promise.js:693:18)
    at Promise._fulfill (/app/node_modules/bluebird/js/release/promise.js:638:18)
    at Promise._resolveCallback (/app/node_modules/bluebird/js/release/promise.js:432:57)
    at Promise._settlePromiseFromHandler (/app/node_modules/bluebird/js/release/promise.js:524:17)
    at Promise._settlePromise (/app/node_modules/bluebird/js/release/promise.js:569:18)

Latest version of css-loader is no longer incompatible with CSS definitions in react toolbox (Error in description was detected at "css-loader": "^2.0.0", does not occur with "css-loader": "^2.1.0"), closing this now


Media updates in Firefox 81

Acceptance criteria
The following bugs are dev-doc-complete, BCD is updated, an entry to Firefox release notes is added, experimental features page is updated, the content has been reviewed as needed.

[ ] Enable MediaControl in release — looking at our internal explainer, it looks like this is just about making media aplyable using computer hardware keys, etc., and we only support <audio> and <video> right now (not other media such as Web Audio API). I suspect we probably want to include a rel note, and mention it on the audio/video pages, and mention it in the MediaSession API documentation, how it is related, etc? This looks related, but we don't seem to mention this at all. Bit of investigation needed, write an example, etc. Ask the engineer if you are confused.

Estimated total: 1-2 days.

Happy to have you assign this to me @chrisdavidmills 👍

@Rumyra done! BTW, I invited you to this repo with write access, so you assign yourself in future.

Looking at this again, I don't think there is anything to do here. See https://bugzilla.mozilla.org/show_bug.cgi?id=1648100#c10
I added a quick note about it to the top of https://wiki.developer.mozilla.org/en-US/docs/Web/API/Media_Session_API.


Teams - crashes after about 20 minutes every time. Error caa70004

Hi
I'm not able to use Teams on my pc. After 20 min I'm getting error code - CAA70004. After that i have to restart but the same issue stays. Even clearing the cache does not help. The only thing I can do is to reinstall it but after 20 min it's the same error again. :-S
Who can help to solve this?
Greetings

@fuserke1981 , Could you please use Report a Problem feature in help section of Microsoft Teams? So we can see the catch the error logs.

@Trinetra-MSFT Well when i look in the webapp i can only see:

And in the desktop app i cannot do anything
greetings

@fuserke1981 ,
Could you please follow below steps to get the error logs?
Please follow below steps to provide error logs when you sign in

Fully exit the Microsoft Teams desktop client. To do this, either right click Teams from the Icon Tray and select ‘Quit’, or run Task Manager and fully kill the process.

Once in the directory, delete all files in this folder.
Restart Teams and try to reproduce this issue.
Right click Teams icon in system tray and click Get logs. Save the log file to your desktop and send to us

Please re-open with requested information if you are still facing the issue.

Hi there,
I am too facing the same issue, followed the steps mentioned above, but it didn't work


Adapt empty size regarding distance to parent/children

Describe the bug
If you try to import the RobotExpressive.glb (https://github.com/mrdoob/three.js/blob/dev/examples/models/gltf/RobotExpressive/RobotExpressive.glb)
from the three.js examples some weird transformation are shown
To Reproduce
Steps to reproduce the behavior:

Download https://github.com/mrdoob/three.js/blob/dev/examples/models/gltf/RobotExpressive/RobotExpressive.glb

Import into blender:

See error:

Version

OS: Win10
Blender Version 3.0.0

If you're talking about the Empty lines, that's not a weird transformation, it's just that the display size on empties is always the default 1.0. AFAIK this is purely visual, but it would be easy enough to guess a size the same way bones are done, by looking at the distance to its parent/child nodes.
Model from #349 is another test case.

Setting as enhancement, and I updated the description

If you're talking about the Empty lines, that's not a weird transformation, it's just that the display size on empties is always the default 1.0. AFAIK this is purely visual, but it would be easy enough to guess a size the same way bones are done, by looking at the distance to its parent/child nodes.
Model from #349 is another test case.

And how do I do that?

If you mean, how would you implement that, you could do a patch like this
diff --git a/addons/io_scene_gltf2/blender/imp/gltf2_blender_node.py b/addons/io_scene_gltf2/blender/imp/gltf2_blender_node.py
index 550bd5a0..d3a41935 100644
--- a/addons/io_scene_gltf2/blender/imp/gltf2_blender_node.py
+++ b/addons/io_scene_gltf2/blender/imp/gltf2_blender_node.py
@@ -77,6 +77,7 @@ class BlenderNode():
         else:
             name = vnode.name or vnode.default_name
             obj = bpy.data.objects.new(name, None)
+            obj.empty_display_size = BlenderNode.calc_empty_display_size(gltf, vnode_id)

         vnode.blender_object = obj

@@ -111,6 +112,17 @@ class BlenderNode():

         return obj

+    @staticmethod
+    def calc_empty_display_size(gltf, vnode_id):
+        # Use min distance to parent/children to guess size
+        sizes = []
+        vids = [vnode_id] + gltf.vnodes[vnode_id].children
+        for vid in vids:
+            vnode = gltf.vnodes[vid]
+            dist = vnode.trs()[0].length
+            sizes.append(dist * 0.4)
+        return max(min(sizes, default=1), 0.001)
+
     @staticmethod
     def create_bones(gltf, arma_id):
         arma = gltf.vnodes[arma_id]

Of course there are multiple formulas you could use to guess a size from the distances. Also it doesn't take scale into account.

Thanks @scurest for the diff. I just create a PR with your proposal


Scrape/archive supplemental FERC forms

There are several smaller FERC forms that supplement their primary forms (1,2,6,60,714). These supplemental forms include 1F, 3Q Electric, 2A, 3Q Gas, and 6Q. We should be archiving these supplemental forms even if we are not actively using them at this point in time.

Issue moved to catalyst-cooperative/pudl-archiver #6 via Zenhub


Change avatarPhotoLink return for regular avatars

Reopened in a another PR, because “The head ref may contain hidden characters: ‘\u0441hange-avatarPhotoLink-return-for-regular-avatars’”


"I have a question" link 404s

The "I have a question" link in the footer points to:
https://siahub.readme.io/discuss
But that's currently showing a 404 error.

oops, yes closed questions section.
Since all questions were about exchanges and "lost" transactions, not about siahub 😬


Failing docs because of new pygments release

The docs build are seemingly failing since the recent pygments release (May 3).

Issue looks fixed.
Maybe via: https://github.com/kivy/kivy/pull/7623 ?


Cannot change language

Hello
I spent a few hours trying to change the language to English without any luck, this is for a docker container.
I have a .env file in the root directory and in the /docker directory just to be sure. Both look like this:
APP_NAME="GPS Tracker"
APP_ENV=production
APP_KEY=
APP_DEBUG=false

# Configure this if you are using a web server as proxy
APP_URL=http://localhost:8092

# Configure the default language (es/en)
APP_LOCALE=en
...

I even made the following changes to /config/app.php:
'locale' => 'en',
'fallback_locale' => 'en',
...

I followed the README  to the letter, but the container works in Spanish every single time. It won't change to English.
Please help, how am I supposed to change the language? Thank you
EDIT:
By complete accident I found http://<IP_ADDRESS>:8092/profile and I can change the language from there. Can I have the container boot into English by default?

Ok, spainsh is default system language, I need to update this setting to match with the default APP_LOCALE.
https://github.com/eusonlito/GPS-Tracker/blob/master/app/Domains/Language/Seeder/data/language.json

Ok, spainsh is default system language, I need to update this setting to match with the default APP_LOCALE.
https://github.com/eusonlito/GPS-Tracker/blob/master/app/Domains/Language/Seeder/data/language.json

Thank you.

For now, you can change the default language from database:
UPDATE `language` SET `default` = FALSE;
UPDATE `language` SET `default` = TRUE WHERE `code` = "en";

Solved from commit https://github.com/eusonlito/GPS-Tracker/commit/1f9fa0278906b5cec676e7a1ccd54f2695161d02
Now the default language is checked from .env instead the default column.


refactor to become more inline with standard go practice

See: same issue on arcadia-lib
i18n should be able to be shared by multiple apps in the repo, therefore it should not be located under the app directory, should be at same level. i18n will only be dependent on internal l10n and command can be dependent on i18n, now that the circular dependency has been fixed (by moving i18n out of main, which is the wrong place).
main must contain as little code as possible and can never contain anything that other code depends on.

This actually implemented the bootstrap mechanism to get rid of global state


Convert _ functions and operations to use new access modifiers, remove 6-month deprecated callables.

This PR converts many operations and functions in the standard library that were marked with underscores (_) to use the new internal access modifier keyword added in 0.11. Some _ callables cannot yet be converted, due to accidental dependencies taken across assembly boundaries.
This PR also removes @Deprecated callables that have been deprecated more than six months, keeping to the Q# API design principles.

Retargeting to feature branch to test cross-repo changes.


Too many instances of PrismaClient

Bug description
I’m getting the classic FATAL: remaining connection slots are reserved for non-replication superuser connections error for my Postgres DB because there are too many PrismaClient instances going on.
Problem is, I’m using REST API routes (using NextJS) and not GraphQL, so each time there’s an incoming request to a route (and there are many), I’m instantiating a PrismaClient, which results in the aforementioned error.
Is there a way to avoid this?
Environment & setup

OS: macOS
Database: PostgreSQL
Node.js version: 12.6.0
Prisma version:

2.11.0

The solution that I've seen is create a single instance during app initialization that is used as a dependency for modules to consume. So if you're using NestJS, I would probably just create a module that's initializes prisma client, and then have that being re-used by other services.

@bbbryan14 how, specifically, would it be done with NextJS (not NestJS) REST APIs? As far as I know, each API is called individually at various times. Even if I separate it into modules, won’t a new Prisma Client be instantiated each time an API is called?
module
// Prisma
import { PrismaClient } from '@prisma/client'
const prisma = new PrismaClient()

export default prisma

API 1
import prisma from 'lib/utils/prisma'
...

API 2
import prisma from 'lib/utils/prisma'
...

@heymartinadams try this: https://github.com/prisma/prisma-client-js/issues/228#issuecomment-618433162

Will give this a try, @pantharshit00!

@heymartinadams sorry to bother u but how did u actually made it work? If I create the module in /public/lib... and import and use it inside getServerSideProps I get:
error - ./node_modules/@prisma/client/runtime/index.js:26261:39
Module not found: Can't resolve 'async_hooks'

🙈

related to https://github.com/prisma/prisma/issues/4393 maybe?

Works like a charm, @vladandrei0.
import { PrismaClient } from '@prisma/client'

// See here: https://github.com/prisma/prisma-client-js/issues/228#issuecomment-618433162
let prisma

if (process.env.ENV === 'production') {
	prisma = new PrismaClient()
}
// `stg` or `dev`
else {
	if (!global.prisma) {
		global.prisma = new PrismaClient()
	}

	prisma = global.prisma
}

export default prisma

yes, all in order now. thanks. I don't even know what I did wrong the first time...

Hey @heymartinadams,
Just to verify that I understand the problem correctly – is this only happening when using the next dev development server?

Correct, @2color. I only use next, not sure if the same as next dev. ☺


Andrew7234/runtime tx denoms

Adds denomination fields for runtime transactions as fee_symbol and amount_symbol.
Also updates is_likely_native_token_xfer flag to be more accurate, resolving https://app.clickup.com/t/8693vwwpz

Huh we still have damask emerald disabled in our e2e_regression tests. I tried again and still had errors trying to query historical balances in the runtime finalizeFastSync. The comment says we probably need to redeploy the archive node now that 5540 is merged, which seems to have happened. The archive node is on labs infra and is running this image tag, which was built 5mo ago and should have worked. Will file a follow-up ticket to investigate further


Use Grafana Agent Logo as icon and in the docu

https://raw.githubusercontent.com/grafana/agent/master/docs/assets/logo_and_name.png

Had to trace it myself since there is no vector version online and the gradient makes it impossible to autotrace, but here you go. 👍🏽


Update custom ref docs with text from latest TypeSpec

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

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


documentation for run designs

https://github.com/The-OpenROAD-Project/OpenLane/blob/5771836bf47dc77996b35471f2bf40a6acdb5983/docs/source/usage/exploration_script.md?plain=1#L73
the command line parameters are unclear
there are a few examples, I guess that --matrix and a json file replaces the previous config.txt.
there is no matrix.json file example

https://github.com/efabless/openlane-ci-designs/blob/d9ba6ea51cdd50589ad64842f85c5aa238cf003a/wbqspiflash/matrix.json

for your reference, here are the options I can't work out what they do from the help message:
--configuration_parameters
--append_configurations

If you can explain what they do I will make  PR on the docs

@mattvenn Looking a the source code, run_designs.py generates a csv report file that contains a set of variables. --configuration_parameters points to a file that contains a comma separated list of variable names. Depending on --append_configurations, these variables either are added to or replace the set of variables in the csv report file. Note that this hasn't been tested in a while and might be broken. I will test them as soon as I get a chance.

ok. I think the addition of the matrix file and a link to the online docs would be helpful


no option to check if plainOverlay is showing

It would be nice to have an option to see if plainOverlay is showing, without using events.

isShown property of internal object may help you:
$element.data('plainOverlay').isShown // `true` if it's showing, otherwise `false`

perfect! many thanks

😃
So, this issue was solved? It's not yet closed.

Yes  :smile:


Dependency conflict

When running with the default options (i.e., as provided), the Install dependencies code box fails with the following error:
ERROR: Cannot install jax[cuda11-cudnn805]==0.3.10, jax[cuda11-cudnn805]==0.3.11, jax[cuda11-cudnn805]==0.3.12, jax[cuda11-cudnn805]==0.3.13, jax[cuda11-cudnn805]==0.3.14, jax[cuda11-cudnn805]==0.3.15, jax[cuda11-cudnn805]==0.3.16, jax[cuda11-cudnn805]==0.3.17, jax[cuda11-cudnn805]==0.3.18, jax[cuda11-cudnn805]==0.3.19, jax[cuda11-cudnn805]==0.3.20, jax[cuda11-cudnn805]==0.3.21, jax[cuda11-cudnn805]==0.3.22, jax[cuda11-cudnn805]==0.3.23, jax[cuda11-cudnn805]==0.3.24, jax[cuda11-cudnn805]==0.3.25, jax[cuda11-cudnn805]==0.3.8 and jax[cuda11-cudnn805]==0.3.9 because these package versions have conflicting dependencies.
ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

fixed in master


TypeError: Cannot read property 'default' of undefined

I get window.Library undefined when accessing through cdn.
This is the built library: https://www.npmjs.com/package/regex-colorize

Hello @geongeorge,
The library seems to work for me. Here's the sandbox:
https://codesandbox.io/s/festive-wright-m3pen
Feel free to reach out if you still have issues getting it to run.

Thanks for the library <3
used your basic version for canvas-txt before
you can add this into your "made with" section:
https://github.com/geongeorge/regex-colorizer

Will do - Thanks for sharing and keep up the good work !


Update browser plugin

Platforms affected
Browser and Electron
Motivation and Context

Using the camera plugin with the Browser platform you can not specify the location, where the camera opens. Once you call navigator.camera.getPicture, for example, from your index.js with sourceType: Camera.PictureSourceType.CAMERA option, it creates a container and opens the Camera in it.
The problem is, the container with the video is always appended to the document.body and the user has no control of it, except the width and height. The only way to customize its style is by selecting cordova-camera-capture class and applying your styles. Besides, this makes close impossible to open the camera inside a modal, unless you're willing to do crazy DOM manipulation after you call  navigator.camera.getPicture method. Same applies for the capture button.
Moreover, I'm missing the Cancel/Close button because once you open the camera, now there's no way to close it unless you take a picture.
Similarly when you call navigator.camera.getPicture, for example, from your index.js with sourceType: Camera.PictureSourceType.PHOTOLIBRARY  or sourceType: Camera.PictureSourceType. SAVEDPHOTOALBUM  option, it creates the input field.This field is also appended to the document.body and the user has no control of it.
I've felt, the need to allow the user to choose where he wants to open the camera, which capture/ cancel button or which input field he wants to use. For example, the user could create a container element and provide us with the ID of the element, where he wants to open the camera. This would give the user more control of the camera plugin and let easier modify its container. This would allow the user to easily open the Camera in a modal or where ever he wants.
Description

Allow users to set 4 additional options:
- customCameraContainer (id of custom camera's container element)
- customCaptureButton (id of custom camera's capture button)
- customCancelButton (id of custom camera's cancel button)
- customSourceInput (id of the custom source input element, when using different mode than the default sourceType)
The default values of these options are null. If the user does not provide the ID of a custom element, the *default element will be used.
Besides, these changes, I've refactored the code to use const and let instead of var. Also, I've broken down big function to functions smaller.
Testing

npm t
I've already tested the code manually.
Also tested the following cases manually.

Calling navigator.camera.getPicture with no options:

Results:
Once function is called, it create the following container. Streams a video form the Camera.

If Capture is clicked, it returns the image data as base64, stops video stream and deletes the
 element.
If Cancelis clicked, it stops video stream and deletes the
 element.

<div class="cordova-camera-capture" style="position: relative; z-index:<PHONE_NUMBER>;">
    <video width="320" height="240" src="blob:file:///90647c34-f521-4c8d-9df4-1224f60c981e"></video>
    <button>Capture</button>
    <button>Cancel</button>

Calling navigator.camera.getPicture with options:

    sourceType: Camera.PictureSourceType.PHOTOLIBRARY

Results:
Once function is called, it create the following input field.

If Input is clicked, it open default file manager and allows to upload 1 image. Return the image data as a callback.

<input class="cordova-camera-select" type="file" name="files[]" style="position: relative; z-index:<PHONE_NUMBER>;">

Calling navigator.camera.getPicture with options:

    sourceType: Camera.PictureSourceType.PHOTOLIBRARY,
    customCancelButton: 'cordova-source-input'

Results:
Once function is called, it create the following input field.

If Input is clicked, it open default file manager and allows to upload 1 image. Return the image data as a callback.

<input class="cordova-source-input" type="file" name="files[]">

Calling navigator.camera.getPicture with options:

    customCameraContainer: 'cordova-camera-preview',

Results:
Once function is called, it create the following container. Streams a video form the Camera.

If Capture is clicked, it returns the image data as base64, stops video stream and deletes everything inside
 element.
If Cancelis clicked, it stops video stream and deletes the
 everything inside  element.

    <video width="320" height="240" src="blob:file:///90647c34-f521-4c8d-9df4-1224f60c981e"></video>
    <button>Capture</button>
    <button>Cancel</button>

Calling navigator.camera.getPicture with options:

    customCameraContainer: 'cordova-camera-preview',
    customCaptureButton: 'cordova-camera-capture',
    customCancelButton: 'cordova-camera-cancel'

Results:
Once function is called, it create the following container. Streams a video form the Camera.

If Capture is clicked, it returns the image data as base64, stops video stream and deletes the <video> element.
If Cancelis clicked, it stops video stream and deletes the <video> element.

    <video width="320" height="240" src="blob:file:///90647c34-f521-4c8d-9df4-1224f60c981e"></video>
    <button id="cordova-camera-capture">Capture</button>
    <button  id="cordova-camera-cancel">Cancel</button>

Checklist

[x] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[ ] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[ ] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[x] I've updated the documentation if necessary

Any idea when this feature will be included? This is exactly the thing I've been needing in my app.

@baibhavprdn - No. As you can see, there are still unresolved discussions that need to be worked on first. Generally: No ETA on features, bugs and new releases.


Add more Cypress tests

@elsylambert will determine these.

@birchamp @BincyJ
May I please create new issue:  "Create cypress test for autosave (refactor branch)"
after this morning discussion

@ancientTexts-net yes please go ahead

@ancientTexts-net #1194 written up for Auto save


Revert "[HTML5] Fetch API now passes credentials."

Reverts godotengine/godot#57934

Fixes #58615

Cherry-picked for 3.5.


docs: Fix typo in README

Summary
endpoints[].ui.badge.reponse-time -> endpoints[].ui.badge.response-time
Checklist

[ ] Tested and/or added tests to validate that the changes work as intended, if applicable.
[x] Updated documentation in README.md, if applicable.

Thank you for the contribution, and merry Christmas :)


add rotate functionality

This adds rotate to the available features.  I could use some advice before this gets merged, because it causes an error to be logged even though it appears to work in the storybook.
Error: <rect> attribute height: A negative value is not valid. ("-800") in react-dom
I don't understand why it's doing that.  Maybe it would be better to do the rotation using the svg rotate() transformation instead of matrix()?

This PR breaks a lot of viewer features, like the ability to pan the image, the miniature,.... Probably there's some extra work to do.
In general, I think that the rotation is not so easy to develop but I can accept a fully working PR.

Thanks anyway ;)


Ad issue57

Changes

Added documentation to uninstall the component

/kind documentation

Fixes #57

/ok-to-test

/check-cla

I am guessing @bvennam needs to also give you a LGTM and APPROVE to merge this?


WIP systemd: 249 -> 250

Motivation for this change
Continuation of #150491
Opening PR so I can start large build job before getting on a plane.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

This needs to target staging, and needs to be bumped to systemd/systemd-stable v250.3.
Furthermore, we should at least skim over the changes from v250..v250.3  and verify that they don't break any NixOS things.

Yea, traveling right now. I'll mark as draft until it's actually ready

I'll do a proper rebase and merge conflict analysis once I get home.

@ofborg eval

@jonringer commit systemd: 249.5 -> 250.3 needed rename to systemd: 249.7 -> 250.3

rebased on latest master

I saw your PR too late and opened up https://github.com/NixOS/nixpkgs/pull/156438

I am closing this just now to avoid pings on rebase...

systemctl --version
systemd 250 (250.3)
+PAM +AUDIT -SELINUX +APPARMOR +IMA +SMACK +SECCOMP +GCRYPT -GNUTLS +OPENSSL +ACL +BLKID +CURL +ELFUTILS +FIDO2 +IDN2 -IDN +IPTC +KMOD +LIBCRYPTSETUP +LIBFDISK +PCRE2 -PWQUALITY -P11KIT -QRENCODE +BZIP2 +LZ4 +XZ +ZLIB +ZSTD -BPF_FRAMEWORK -XKBCOMMON +UTMP -SYSVINIT default-hierarchy=unified

works on my laptop

I'll try this on my laptop as well.
I think we could probably merge into staging after next staging-next cycle begins

I'm running a hydra job on this: https://hydra.jonringer.us/jobset/nixpkgs/systemd250

I think we could probably merge into staging after next staging-next cycle begins

@jonringer there have been some more staging cycles (https://github.com/NixOS/nixpkgs/pull/158080, https://github.com/NixOS/nixpkgs/pull/158080).
This also needs a rebase, as there have been some conflicts. Is there anything else preventing this from being merged? (I see the PR title still contains a "WIP" too).

I am using this for some time on my laptop now without noticeable impact so far.

@flokli I was going to use this staging cycle for glibc https://github.com/NixOS/nixpkgs/pull/133431#issuecomment-1053697179. Once it begins, I think we can merge this.

I can still rebase though

I rebased this onto latest staging, fixed the conflicts, and dropped -Defi-ld=gold as suggested in https://github.com/NixOS/nixpkgs/pull/156096#discussion_r820131894.
Considering this has been open for quite some time, and tested by some people already, I'll run the nixosTests.systemd{-boot} tests once more, and then we should merge this.

nixosTests.systemd-boot succeeded.
nixosTests.systemd couldn't run, due to /nix/store/i3axx4kkdbfl01r0dfsps9qnj8biryxh-python3.9-scikit-build-0.12.0.drv being broken right now. However, this has been sufficiently tested previously.
Let's get this into staging.

This merge broke nixosTests.containers-ip.  I checked the commit right before this merge and the merge commit (after some bisection steps, as I started with a range of many hundred commits).
Maybe also some other small-channel blockers from this list regressed together with it: https://hydra.nixos.org/eval/1748773#tabs-still-fail  (you can see logs there or reproduce cheaply on that particular commit)

Thanks for the heads-up, I'll take a look.

I built this locally. The container startup fails while trying to bind-mount /nix/var/nix/daemon-socket into the container:
machine # [    5.364468] systemd[1]: Starting Container 'webserver'...
machine # [    5.448067] container webserver[879]: Spawning container webserver on /var/lib/containers/webserver.
machine # [    5.448712] container webserver[879]: Press ^] three times within 1s to kill container.
machine # [    5.450500] container webserver[885]: Failed to stat /nix/var/nix/daemon-socket: No such file or directory
machine # [    5.454873] systemd[1]<EMAIL_ADDRESS>Main process exited, code=exited, status=1/FAILURE
machine # [    5.455510] systemd[1]<EMAIL_ADDRESS>Failed with result 'exit-code'.
machine # [    5.456098] systemd[1]: Failed to start Container 'webserver'.
machine # [    5.763319] systemd[1]<EMAIL_ADDRESS>Scheduled restart job, restart counter is at 2.
machine # [    5.769093] systemd[1]: Stopped Container 'webserver'.
machine # [    5.770123] systemd[1]: Starting Container 'webserver'...

I ran the interactive test driver, and looked at the VM. Indeed /nix/var/nix/daemon-socket doesn't exist, because nix-daemon.socket failed to activate:
[  237.187747] systemd[1]: Nix Daemon Socket was skipped because of a failed condition check (ConditionPathIsReadWrite=/nix/var/nix/daemon-socket).

Okay, I figured this out - this is actually a bug in the upstream nix-daemon.socket unit.
I have a workaround in nixpkgs, and will send a fix to the Nix repo.

@vcunat the workaround is in https://github.com/NixOS/nixpkgs/pull/164644, and the proper fix in Nix is in https://github.com/NixOS/nix/pull/6282.


Ugoo-Eze

Hey @Ugoo-Eze closing this PR now due to long in activity. Hope this is ok for you. Please feel free to reopen the PR or make a new one.
__
  Summary: :hammer_and_wrench: This code needs a fix