Adds some statistical metrics for BlobSidecarPool

Produce metrics like this:
# HELP beacon_blobs_pool_stats Blobs pool statistics
# TYPE beacon_blobs_pool_stats counter
beacon_blobs_pool_stats{type="block",subtype="rpc_duplicate",} 0.0
beacon_blobs_pool_stats{type="block",subtype="gossip_duplicate",} 0.0
beacon_blobs_pool_stats{type="block",subtype="rpc_fetch",} 0.0
beacon_blobs_pool_stats{type="blob_sidecar",subtype="rpc",} 0.0
beacon_blobs_pool_stats{type="block",subtype="gossip",} 470.0
beacon_blobs_pool_stats{type="blob_sidecar",subtype="gossip",} 1411.0
beacon_blobs_pool_stats{type="blob_sidecar",subtype="rpc_duplicate",} 0.0
beacon_blobs_pool_stats{type="blob_sidecar",subtype="gossip_duplicate",} 0.0
beacon_blobs_pool_stats{type="block",subtype="rpc",} 0.0
beacon_blobs_pool_stats{type="blob_sidecar",subtype="rpc_fetch",} 5.0

gossip subtype counts first-seen (non duplicate) blocks\blob_sidecars that has been received via gossip
rpc subtype counts first-seen blocks\blob_sidecars received via RPC (a recent fetch)
gossip_duplicate counts blocks\blob_sidecars coming from gossip that was already present in the pool (duplicate)
rpc_duplicate counts blocks\blob_sidecars coming from rpc that was already present in the pool (duplicate)
rpc_fetch counts number of requested fetches

I may find a better home for the RemoteOrigin enum.

I like it, they make sense to me and would be useful. Would be good if it can be made generic because it applies to both blocks and blobs but since it all goes in the BlobSidecarManager current naming is good.

I like it, they make sense to me and would be useful. Would be good if it can be made generic because it applies to both blocks and blobs but since it all goes in the BlobSidecarManager current naming is good.

My plan is to do a refactor PR to rename BlobSidecarPool to BlockBlobSidecarTrackerPool


Automatic Batchsize should be separately calculated for train & val/test

🚀 Feature
I would like to request auto-batchsize scaling perform separate batchsize operations for train and for val/test.
Motivation
The auto-batchsize scaling feature is neat, but it also only runs training steps, AFAIK.
But during val/test, due to the increase in free memory due to lack of gradients, batchsizes can be much larger.
In addition, there are cases I have had where the val/test input sizes are larger than the training ones (i.e. crop images to fit during train, then use whole image during val/test).
Pitch
Call validate and/or test during the scaling procedure and set the batchsizes directly into the train_dataloader and val/test_dataloader objects directly, or alternatively as a second class parameter such as <module>.batch_size_eval
Alternatives

Additional context

If you enjoy Lightning, check out our other projects! ⚡

Metrics: Machine learning metrics for distributed, scalable PyTorch applications.

Flash: The fastest way to get a Lightning baseline! A collection of tasks for fast prototyping, baselining, finetuning and solving problems with deep learning

Bolts: Pretrained SOTA Deep Learning models, callbacks and more for research and production with PyTorch Lightning and PyTorch

Lightning Transformers: Flexible interface for high performance research using SOTA Transformers leveraging Pytorch Lightning, Transformers, and Hydra.

Dear @AAnoosheh,
Would you be interested in contributing such feature ?
Best,
T.C

Hi, I might be able to.
Given my free time, it might be questionable, but if I cannot find a decent workaround for my personal use case, I'll add the feature properly here.


enhancement: add padding-y prop to the container component

Currently, the gap prop of the Container component is only about padding of x-axis, but there should be another prop for padding of the y-axis
Solution I would like to recommend:
A new prop probably named py which controls the paddingY property of the component

btw, with your consent, can I work on this?

Hey @kr-anurag thanks for the recommendation but, why don't you use css={{ py: 'PADDING' }}?


The UI of the webpage

The layout of page
Making the website more user friendly
Social media handles
Contact us section
Modifying navbar

Please assign this issue to me and I'll surely help in improving the overall user experience.
@akshitagupta15june

Please provide a specific page and section you would like to take up and update your issue accordingly.
To provide other interested contributors opportunity to work on issues/features we would prefer not to assign the entire makeover to one contributor.

I would like to work on home page and about section

Please provide a specific page and section you would like to take up and update your issue accordingly.
To provide other interested contributors opportunity to work on issues/features we would prefer not to assign the entire makeover to one contributor.

I would like to work on home and about section.

please join discord channel for discussion on the UI


3.Observer - DifferentEventAlternatives updated, was not functioning …

…properly before, reference names were inconsistent.
Added reference to MyEventHandler myEventHandler;
Kept comments off.

Thanks for the help!

Thank you so much for letting me contribute to your amazing work Sir! :)))


UI freezes after showing previews in mapping table

If there is no mapping specified, and I'm in mappings table view,  after show mapping preview UI will freeze on both Firefox and Chrome.
steps to reproduce

run atlasmap
click on mapping table
click on Show mappings preview
try to add new mapping

This might freeze also in different steps, but key is to NOT have specified any mapping. If I create mapping before enabling preview, everything works fine.

#563?


Add /etc/fstab in order to remove /cygdrive/ prefix

Describe the pull request

What does your PR fix?
Due to skipping the regular MSYS2 installer, (some) configuration files are missing from the MSYS2 installation generated by vcpkg. This leads to Windows drives being mapped to directories below /cygdrive/ instead of /. Cf. /etc/fstab in a regular MSYS2 installation. Apart from solving a number of strange behaviours, it also leads to shorter paths in general.

Which triplets are supported/not supported? Have you updated the CI baseline?
For Windows hosts.

Does your PR follow the maintainer guide?
-/-

Please sign CLA first.
Thanks.

Please sign CLA first.
Thanks.

Done.

cc @Neumann-A for review this PR.

If it works I am fine with it. ras0219-msft did the msys changes which made cygdrive necessary


errors on tensorboard

Thank you for your repo. When I followed your tutorials, I met a difficulty on tensorboard.
I ran $python submit.py
I got:

Creating dataset directory ./output_gdb13_1K/example/
Creating model subdirectory ./output_gdb13_1K/example/job_0/
Running job as a normal process.
/home/xzhang/miniconda3/envs/graphinvent/bin/python
Running job using HDF datasets located at ./data/gdb13_1K/
Checking that the relevant parameters match those used in preprocessing the dataset.
-- Job parameters match preprocessing parameters.
Traceback (most recent call last):
File "./graphinvent/main.py", line 5, in
import util
File "/home/xzhang/projects/GraphINVENT/graphinvent/util.py", line 20, in
tb_writer = SummaryWriter(log_dir=constants.tensorboard_dir, flush_secs=10)
File "/home/xzhang/.local/lib/python3.8/site-packages/torch/utils/tensorboard/writer.py", line 220, in init
self._get_file_writer()
File "/home/xzhang/.local/lib/python3.8/site-packages/torch/utils/tensorboard/writer.py", line 250, in _get_file_writer
self.file_writer = FileWriter(self.log_dir, self.max_queue,
File "/home/xzhang/.local/lib/python3.8/site-packages/torch/utils/tensorboard/writer.py", line 60, in init
self.event_writer = EventFileWriter(
File "/home/xzhang/miniconda3/envs/graphinvent/lib/python3.8/site-packages/tensorboard/summary/writer/event_file_writer.py", line 73, in init
if not tf.io.gfile.exists(logdir):
File "/home/xzhang/miniconda3/envs/graphinvent/lib/python3.8/site-packages/tensorboard/lazy.py", line 65, in getattr
return getattr(load_once(self), attr_name)
AttributeError: module 'tensorflow' has no attribute 'io'
-- Sleeping 2 seconds.

I reinstalled tensorboard, it still didn't work.

It worked after I installed tensorflow-io
pip install tensorflow-io

But how to use tensorborad? Could you please give me some instructions or show me what you got from the tensorboard? Many thanks

To run a tensorboard session from a local machine, type:
tensorboard --logdir path/to/tensorboard_dir/

So, for instance, if you want to view all the tensorboard logs in projects/GraphINVENT/output_gdb13_1K/tensorboard/, you can run:
tensorboard --logdir projects/GraphINVENT/output_gdb13_1K/tensorboard/

If you have the tensorboard logs on a remote machine, you can view them as per the instructions here: https://github.com/rociomer/cheat-sheets/blob/master/tensorboard.md
The tensorboard logs should look as they do on this page:
https://www.tensorflow.org/tensorboard/get_started
but with the relevant GraphINVENT metrics, e.g., training and validation losses for the generative models, fraction of valid generated molecules, etc.
Does this help? Apologies for the late response.


Tag //kythe/docs/schema as a manual target (it's expensive) [phab:D246]

Created by schroederc at 2015-05-13 19:08:44:

zarko wrote at 2015-05-13 19:49:02:
Can we change the arc presubmit hooks for the indexers to rebuild this target should the indexers change? It's important that we check that all of the examples are still OK after we change that code.

schroederc wrote at 2015-05-13 19:56:08:
The arc presubmit doesn't use Bazel yet.

zarko wrote at 2015-05-13 19:57:36:
Granted, but let's not forget to run the test once we move over.

zarko accepted this revision at 2015-05-13 19:57:36

schroederc closed via commit 9f56ebfe78e732cb279697030a281d03f47d3ee3 at 2015-05-13 20:03:08


08/15/2018 22:09 : Create Pull Request for branch master -> prTesting1_2018081522

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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


[FEAT]: add new multi-agent discussion in paper writing

Description
multi-agent discussion prompt add
Additional Information
No response

supported by @timsanders256 and @keyangds now. Please help PR


Class "Locale" not found

Im trying to add the middlewares to my Laravel 10 app and i get error Class "Locale" not found if i add this

You can add the page view middleware to a specific route group, e.g. web.php like so:

Route::middleware('analytics')->group(function () {
// ...
});
Or add the page view to all middlewares/on an application level like so:
// app/Http/Kernel.php
protected $middleware = [
// ...
\AndreasElia\Analytics\Http\Middleware\Analytics::class,
];

You likely don't have the PHP intl functions enabled in your php.ini file.

i do have it enabled, composer wont let you install the package if you don't enable it.
could it be that i don't use tailwind and i only use bootstrap?

From: Andreas Elia @.>
Sent: March 24, 2024 10:04 PM
To: andreaselia/laravel-analytics @.>
Cc: angeloskalaitzakis @.>; Author @.>
Subject: Re: [andreaselia/laravel-analytics] Class "Locale" not found (Issue #52)
You likely don't have the PHP intl functions enabled in your php.ini file.
—
Reply to this email directly, view it on GitHubhttps://github.com/andreaselia/laravel-analytics/issues/52#issuecomment-2016927664, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXATMYJXIBJFNLF4NGONRRLYZ4WVDAVCNFSM6AAAAABFFVWQYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJWHEZDONRWGQ.
You are receiving this because you authored the thread.Message ID: @.***>

i do have it enabled, composer wont let you install the package if you don't enable it.
could it be that i don't use tailwind and i only use bootstrap?

From: Andreas Elia @.***>
Sent: March 24, 2024 10:04 PM
To: andreaselia/laravel-analytics @.***>
Cc: angeloskalaitzakis @.>; Author @.>
Subject: Re: [andreaselia/laravel-analytics] Class "Locale" not found (Issue #52)
You likely don't have the PHP intl functions enabled in your php.ini file.
—
Reply to this email directly, view it on GitHubhttps://github.com/andreaselia/laravel-analytics/issues/52#issuecomment-2016927664, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXATMYJXIBJFNLF4NGONRRLYZ4WVDAVCNFSM6AAAAABFFVWQYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJWHEZDONRWGQ.
You are receiving this because you authored the thread.Message ID: @.***>

Locale is a PHP intl thing, shouldn't be anything to do with your frontend. Could you take a screenshot of your phpinfo showing the intl extension is enabled? What PHP version are you running?


[FEA] A Python package for hanlding customization.

Is your feature request related to a problem? Please describe.
This feature request is for adding customization like https://github.com/NVIDIA/spark-rapids/issues/5561 . Eventually, we might want to have something similar for mapInArrow and withColumn as well. As a downstream user, my hope is that everything can work out-of-the-box with spark. But I understand that there might be difficulties in upstreaming GPU-specific changes.
Describe the solution you'd like
The feature request is to have a new spark_rapids Python package to interface Python users in the same spirit of dask_cudf and dask_cuda. With this new package, we can add customization points for handling rapids-backed dataframe, either by using free functions like mapInCUDF(dataset, ...) or by using Python reflection to modify existing pyspark functions like mapInPandas to make it return cuDF iterators.
Describe alternatives you've considered

@wbo4958 proposed that we generate IPC handle in spark-rapids and decode the custom cuDF spark IPC message in XGBoost, which is really out of scope for XGBoost and difficult to implement due to memory management in two different projects.
Suggested by cuDF developers, we should use UCX to handle IPC message, which also requires some modification in upstream spark. https://github.com/rapidsai/cudf/pull/11564 is currently under discussion for whether it's necessary to have such code in cuDF.

Would be great if there's a reusable project to host all these customizations for Spark Python.

Similar to https://github.com/NVIDIA/spark-rapids/issues/5561 , I can help work on the feature if it's approved by maintainers here.  Although, we need some additional help from ops for maintaining the Python package.


Includes in Views

Hi,
Great course, really enjoyed it.
I'm having difficulty finding a way to include a view within a view. I have created a News Class which pulls the latest news posts from the DB but I'd like the displayed news posts visible on a few different Views so wanted to include a View within a View.
This may not be the best way of doing it but would normally use an include so feedback would be greatly appreciated.
Thanks,

Hi,
happy to help, but as this question is related to the course, please can you ask the question inside the course on Udemy? That way, other students can benefit from the answers, and even contribute to them.
You can ask a new question in the Q&A section of the course. I'll look out for your question there!
Thanks,
Dave


feat(hasura-auth-js): add support for personal access tokens

Resolves #1853
This is temporarily blocked by https://github.com/nhost/hasura-auth/pull/374

Codecov Report

Merging #1924 (7bfa6c9) into main (69caa34) will decrease coverage by 0.01%.
The diff coverage is 76.71%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #1924      +/-   ##
==========================================
- Coverage   87.12%   87.12%   -0.01%
==========================================
  Files          83       85       +2
  Lines        9052     9263     +211
  Branches      537      555      +18
==========================================
+ Hits         7887     8070     +183
- Misses       1165     1193      +28

Impacted Files
Coverage Δ

packages/hasura-auth-js/src/promises/signInPAT.ts
19.14% <19.14%> (ø)

packages/hasura-auth-js/src/hasura-auth-client.ts
65.17% <85.71%> (+3.82%)
:arrow_up:

...ura-auth-js/src/machines/authentication/machine.ts
97.70% <87.27%> (-0.65%)
:arrow_down:

packages/hasura-auth-js/src/promises/createPAT.ts
98.36% <98.36%> (ø)

...ura-auth-js/src/machines/authentication/context.ts
100.00% <100.00%> (ø)

...sura-auth-js/src/machines/authentication/events.ts
100.00% <100.00%> (ø)

packages/hasura-auth-js/src/promises/index.ts
100.00% <100.00%> (ø)

...ges/hasura-auth-js/src/types/hasura-auth/common.ts
100.00% <100.00%> (ø)

.../hasura-auth-js/src/types/hasura-auth/responses.ts
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes


⚠ GitLab Server has degraded performance

In b3c9690, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 7020 ms

Resolved: GitLab Server performance has improved in 787aaea.


test: Fix vm.install for non-LVM cloud images

Some of our images, like centos-8-stream, are already built from the official
cloud images instead of virt-install with LVM. More images are going to do that
soon [1], so fix vm.install to only do the LVM grow steps if the image actually
uses LVM.
[1] https://github.com/cockpit-project/bots/pull/1518

That testNavigate failure is unrelated. The test log shows that in this run it (rightfully) does run the pvcreate etc. steps, so nothing changed in the image.

That testNavigate failure is unrelated. The test log shows that in this run it (rightfully) does run the pvcreate etc. steps, so nothing changed in the image.

@henrywang: I don't understand that test. It is supposed to filter by release, and that is 1.fc33. So it's quite natural that both httpd and tmux appear. Maybe this was supposed to filter on the version instead? But that already happens in the test case above. So I believe that test has always been racy. Perhaps we have some test package which is not available in fc33? Then the test could use that. Or it just makes sure that both are shown. I am AFK tomorrow (holiday), but I can send a PR on Thursday unless you beat me to it.
In the meantime, I'm landing this PR as it's obviously an unrelated failure, so that the image build PR gets unblocked.

@henrywang: I don't understand that test. It is supposed to filter by release, and that is 1.fc33. So it's quite natural that both httpd and tmux appear. Maybe this was supposed to filter on the version instead? But that already happens in the test case above. So I believe that test has always been racy. Perhaps we have some test package which is not available in fc33? Then the test could use that. Or it just makes sure that both are shown. I am AFK tomorrow (holiday), but I can send a PR on Thursday unless you beat me to it.
In the meantime, I'm landing this PR as it's obviously an unrelated failure, so that the image build PR gets unblocked.

Thanks @martinpitt. I found the root cause. PR #1207 should fix this issue.

Thanks @martinpitt. I found the root cause. PR #1207 should fix this issue.


Permit adjustable 2 side header/footer option

I would like to see side-width also accept a 2 element array.  If a left-side and a right-side are specified, it seems there is an assumed center-side as the sides will conform to a default (1fr, 1fr, 1fr) spacing.  I have a case where I desired a quite wide left side, and a fairly small right side, but I couldn't put in side-width: (5fr, 1fr), and have it work out.

Hi!
Hmmm. That makes sense. Maybe a refactor for the side-width option would be useful. This week I will finish all my academic activities and I'll have time to finish the new version of the package. Then, I will think about this issue in order to find the best solution.
Thanks for the idea 😁!


Tuya Integration Suddenly Started Showing Devices as "Unsupported"

I have almost 30 devices connected to the Tuya Integration and several days ago approximately 80% of them started showing as being "unsupported" in the Device Info and they also lost their Entities. The devices still work fine when operating over the Tuya Cloud or LocalTuya and the problem only seems to exist within the Tuya Integration. I tried reloading, restarting, deleting, reinstalling, etc. and each Device still says "unsupported" and shows no Entities.
I have included a screenshot from one of the times that I reinstalled the integration and you can see several Devices at once and you can see that several of them show as being "unsupported".
I have been using some of these devices for over a year and they have always been connected and supported to the Tuya Integration with no issues until now.
Home Assistant OS
core-2022.5.2
Diagnostics:
config_entry-tuya-d4208934f80f77f0d662c5bd00bbc592.json.txt
Screenshot:

Just to update this I found the following error.
2022-05-15 07:37:54 ERROR (MainThread) [homeassistant.components.light] Error while setting up tuya platform for light Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform     await asyncio.shield(task)   File "/usr/src/homeassistant/homeassistant/components/tuya/light.py", line 382, in async_setup_entry     async_discover_device([*hass_data.device_manager.device_map])   File "/usr/src/homeassistant/homeassistant/components/tuya/light.py", line 375, in async_discover_device     TuyaLightEntity(   File "/usr/src/homeassistant/homeassistant/components/tuya/light.py", line 450, in __init__     h_type=IntegerTypeData(dpcode, **function_data["h"]), KeyError: 'h'


Title and heading are hard coded

Currently the title and heading are hardcoded as 'Sankar's Diary' ;P
Date is fine to show the current date, but should be editable to any date.

@uvsankar
Is this the correct way of working.. i mean to create issues and then adding them as features. Or is there any correct way of doing this?

@nithinsanjey  thanks for the interest in this project. Yeah the should be editable. There are two ways of doing this

Storing it in browser cache itself
Getting all the necessary when initialising the repo for the first time and persisting it in the config.json
Will take it up this weekend.
Yeah this is the correct way of tracking any bugs, improvements , features. When creating an issue please add the corresponding labels.

BTW why should the date be editable ?

@uvsankar The date should be adjustable just in case if the user wishes to write yesterday's entry.


Negative KLL in the notebooks/example.ipynb

The cell [22] plots the NLL and KLL during the training. Why is the KLL negative? Is there any convention that I am not aware of?
https://github.com/noegroup/bgflow/blob/main/notebooks/example.ipynb

Realized this is because the KLL has omitted the H_z part in the calculation, which results in a downshift.


Rewrite block harvesting logic

Remove vanilla methods and create our own to handle drops more precisely.

Implemented in v4.4


[MM-34580] - Truncate image name when text is too long

Summary
Fixes long file names not being truncated and expand/collapse chevron being hidden
Ticket Link
https://mattermost.atlassian.net/browse/MM-34580
Screenshots
Long file name:

Short file name:

/cherry-pick release-5.34

Cherry pick is scheduled.


Names with underscore converting to dash

So first of all, this plugin is amazing! Thank you
Pretty sure I am encountering a bug where if a property has an underscore in the name, it always converts to a dash.
In figma our icons are named with underscores, so for example 'bug_report'
Adding that to the code as [iconName]="'{{property.statusIcon|raw}}'"
but I get back
[iconName]="'bug-report'"
so bug_report converts to bug-report, which isn't the valid name.

Nevermind, it was just delayed in applying the raw filter


[DPE-4860][DPE-4982] get_relation(PeerRelationName) returns None if remove-application has been requested

The main issue here is the storage-detaching after remove-application. That gets more complex if we consider the last unit going away. In this scenario, we cannot consider that relations will be available anymore, such as peer relation databag.
For example, the get_host_ip uses the charm.model.get_binding(peer_relation_name), which returns a valid object even if we have a single unit.
The unit_ip is also called outside of the helper_networking.py, on opensearch_distro. However, all these methods have previous checks, i.e. we never call unit_ip for another unit that its own outside of unit_ips.
Also, we skip acquiring the lock in this case, as the application is going away.
The get_relation behavior
Before remove-application with single unit
Even if we have a single unit, we still have the peer relation databag available. The databag will be used by that single unit across the unit's lifecycle. We can see it with:
root@juju-941823-0:/var/lib/juju/agents/unit-opensearch-0/charm# nano src/charm.py
root@juju-941823-0:/var/lib/juju/agents/unit-opensearch-0/charm# nano lib/charms/opensearch/v0/opensearch_base_charm.py
root@juju-941823-0:/var/lib/juju/agents/unit-opensearch-0/charm# ./dispatch
2024-07-26 14:27:56,864 DEBUG    ops 2.14.1 up and running.
--Return--
> /var/lib/juju/agents/unit-opensearch-0/charm/lib/charms/opensearch/v0/opensearch_base_charm.py(250)__init__()->None
-> import pdb; pdb.set_trace()
(Pdb) p self.model.get_relation(PeerRelationName)
<ops.model.Relation opensearch-peers:0>

After remove-application
After remove-application has been received, we can see that relation object will not be retrieved anymore:
root@juju-6884bf-2:/var/lib/juju/agents/unit-opensearch-1/charm# ./dispatch
2024-08-05 18:38:42,246 DEBUG    ops 2.15.0 up and running.
--Return--
> /var/lib/juju/agents/unit-opensearch-1/charm/lib/charms/opensearch/v0/opensearch_base_charm.py(250)__init_
_()->None
-> import pdb; pdb.set_trace()
(Pdb) p self.model.get_relation(PeerRelationName)
None
(Pdb)

Closes #360
Closes #378
Closes #379

@Mehdi-Bendriss answering the:

I'm not sure that's accurate, the first hook to execute during the teardown flow is storage_detaching - we still have the peer relation data bag then.
(It was proved to successfully work before, no?)

Only backup tests really test remove-application, and that has been taken out because of #243 and our lock not allowing to remove app at once. Once that has been fixed, I rolled back that change and now backup tests hang on remove-application and fail. That is when I started to investigate this particular issue.
Now, according to the ops code, self.model.get_relation calls _get_unique. On git blame, this logic did not change for years now and does return None if no relation is found.
So, either we did not really test the remove-application before my work on backups; or juju changed its behavior.

Cont:
Now, this may be caused by a recent change on juju 3.4.4: https://bugs.launchpad.net/juju/+bug/2024583

Confirmed: it was caused 2 months ago because of that change above and released as part of 3.4.4. Filed a bug on Juju: https://bugs.launchpad.net/juju/+bug/2076599


Skip empty segments when creating uris

https://jira.lib.unc.edu/browse/BXC-3025

Good work, and thanks for adding a test


How to reduce impact to reconciliation time on operator start up?

Not really a bug report, but more of a discussion question
What did you do?
When an operator is first deployed, it will need to reconcile every custom resource (CR) it watches, regardless of whether there are changes.
This can significantly increase the reconciliation time when making an actual change to a CR.
What did you expect to see?
When I deploy an operator and make a change to a CR it watches, it should prioritise that change over automatic reconciliations, or there should be some way to reduce the impact to reconciliation times.
What did you see instead? Under which circumstances?
Reconciliation of the CR that is created is delayed until all other resources have been reconciled.
Possible Solution
I'm wondering if there's a way to use leader election to warm up the operator before switching over to the new deployment.
https://javaoperatorsdk.io/docs/features/#leader-election
We could also write some kind of priority queue but I'm wondering what's the least invasive way to resolve this kind of issue.
Additional context
Related issue, but for timed reconciliation https://github.com/operator-framework/java-operator-sdk/issues/2293

Hi @toshitabata!
thx for raising this, basically the leader election works in a way that if a controller is not a leader it populates the caches of informers / event sources what might have already benefits since it might take long to do so, and the reconciliations happen after that (if becomes a leader).
Unfortunatelly when a controller starts, if I remember correctly the informers see all new events as "Add" events, so we cannot event distinguish based on that - if we would like to have some priority executor or something.
We discussed before the idea to have some priority queue and events with a flag that will indicate that reconciliation should be done on a priority queue. Not sure if this would help though, especially on startup; on the other hand will bring complexity for sure.


Add support for AgentGPT

Hi, love your work (thanks!)
it would be awesome to support AgentGPT, it's a very lean agent app
GH: https://github.com/reworkd/AgentGPT
Demo: https://agentgpt.reworkd.ai
Thank you for your consideration
Ale

there's an issue open on AgentGPT page - linking for tracking
https://github.com/reworkd/AgentGPT/issues/322


parallel_offset 'right' return wrong value

Please note
If you are reporting an installation or module import issue, please note that
this project only accepts reports about problems with packages downloaded from
the Python Package Index. Conda users should take issues to one of the
following trackers:

https://github.com/ContinuumIO/anaconda-issues/issues
https://github.com/conda-forge/shapely-feedstock

Expected behavior and actual behavior.
create a LineString, and parallel_offset to right by 1
l = LineString([(0, 0), (0, 2)])
l = l.parallel_offset(1, 'right')
print(l) # display LINESTRING (1 2, 1 0)
l = l.parallel_offset(1, 'right')
print(l) # display LINESTRING (0 0, 0 2)
expected value: LINESTRING(2 0, 2 2)
(parallel_offset 'left' has no such issue)
Steps to reproduce the problem.
(like shown above)
Operating system
Mac OS X 10.15 Catalina . (also on windows 10)
Shapely version and provenance
1.6b4 installed from PyPI using pip
geos 3.8 (also the same on geos 3.7.3)

This happens because the right offset changes the orientation of the LineString. I don't know if this behavior is intended or not, but it would be good to document it because it confused me as well when I first encountered it.

@LostFan123 agree the change of orientation, though 'left' has no such issue...

It always reverses right side offsets but not left side offsets. There's a mention of it in the change log because of issue #284.
The change log states that the right side reversal was 'noted' but I can't find it in the docs. It should probably be put in.

The change in orientation, although documented on in our docs, was an implementation detail of GEOS. And starting with GEOS >= 3.11, the implementation will now always try to preserve the orientation, instead of changing it.
This has been clarified in the docs in https://github.com/shapely/shapely/pull/1537


ChangeValueEvent Proposal

This is more me typing this up for @Zidane.
ChangeValueEvent will happen any time a value is changed. Values will be mapped to their associated data parameter which was a system added in Minecraft 1.9.
We will need to add a new phase to the cause tracker that occurs at the end of the tick that will basically fire this event for all the data parameters whose values are being written to the client. This kind of system means that "ideally" only one ChangeValueEvent would occur per value per tick.
For example if an entity's health changes many times across the tick then we don't care for any of those values except the last one set during that tick.
Mod support is questionable but not impossible, we just have to review somehow rigging it in to this event (especially when few mods would have data API equivalent values for the data they have added to their own entities).
Any feedback is of course welcome. Any yelling can be directed to @Zidane.

What is the behaviour when a value is modified when called from a ChangeValueEvent?
Would this be put into the list for the next ticks list? Or would a second (third? fourth? fifth?) event be called?

@ZephireNZ I assume that no event would be fired, as listeners that are called after a listener that changes the value will see the new value.

I talked with gabizou at Minecon about this. The problem with only firing one event per value per tick is that it can be changed several times with different Causes. So I proposed having a single ChangeValueEvent subinterface for each Key; that way, the event can be fired multiple times per tick, and only listeners interested in that specific Key would receive events.
I've been busy since Minecon, though, and only got around to it this week. I began writing up some code yesterday, but then hit a roadblock today: offering DataManipulators with multiple Values. Ideally, they would all be fired together. Then I thought of another, very similar idea: using an event for each DataManipulator instead of each Key. It works with multiple values, and if an individual value changes, it can be wrapped in its corresponding DataManipulator and fired. The only requirement is that each Key must be tied to exactly one DataManipulator interface (although individual DataManipulators can still have multiple values).
I'm going to go to bed now; I'll get working on some code tomorrow. In the meantime, let me know your thoughts on this.

I'm almost done with the API side, and I have some ideas for the implementation. However, there's one problem: getting a default Cause when a plugin calls offer() without a Cause (the methods with a Cause aren't even implemented yet). SpongePowered/SpongeAPI#1370 should fix this, but it seems to have gone stale, despite being marked as a "major feature".

@JBYoshi the reason for it to become stale is because we're working on pushing API 6 for a 1.11 release sooner than later. The last time that we accepted major implementation changes it cost us a few months to officially update to 1.10.2 (remember how 1.9.4 was in bleeding for a while but didn't finish?). It is definitely something we will merge in, just not at this very moment.

Okay. Thanks for clarifying!


Depends on not working when creating through parent

I have Calls that belongs to the Client.
If I create via Calls resource then Depends works fine but if I create a Call via Client then the Depends field keeps the field hidden no matter what. I think this has to do with the fact the client field is read-only when you create via Client

Just edited the HTML of the page and removed the disabled attribute and nothing happened, but when I changed the client and changed it back it seemed to works maybe it is an issue with initial validation?


Do we want to provide a spectrum normalization method?

People may want to have a normalized spectrum instead of a flux calibrated one. Do we want to provide that or let the user find another tool to do that?

How difficult it would be to implement this? And what would be the normalization factor? I believe that that could be left for the astronomers or for a future version.

Knowing the answer to your second question the implementation shouldn't be complicated. In other words, the technical part of the implementation doesn't seem to be a problem but it is important to know exactly what we are doing.

I would leave this as a low priority. Maybe for the release in the end of the year. Of course, that depends on how many users are interested in this feature. In my opinion, we should focus in the critical ones.

Normalizing a spectrum is as simple as measuring counts at a given wavelength and dividing the spectrum by that number. Its always a question of to which specific wavelength or wavelength window you want to normalize to, but that should be input by the user. On the other hand, this is so easy and trivial that I wonder if its worth the trouble. This is something users can do on their wavelength calibrated, 1-D spectra, using IRAF or practically anything.
Let's not worry about it now.

so, this is also related to #42

I will close this issue now, we can reopen it later.


Unable to Query for Block-chain Info?

This question might be ridiculous, but how come I can't use the url given in the docs for querying data. I used val rpcURL = "http://cyphernode:8888/getblockchaininfo" and this simple querying function to test the API:
fun query(url:String){
   val response = URL(url).readText()
println(response)
}, but then I got an exception. Exception in thread "main" java.net.UnknownHostException: cyphernode how do I resolve this?

Hi @TBCode523
Try this: echo "GET /getbestblockinfo" | docker run --rm -i --network=cyphernodenet alpine nc proxy:8888 -
The host cyphernode doesn't really exist on your system.  If you want to hit the proxy directly (without going through the auth layer of the gatekeeper), you need to be "inside" the Cyphernode Docker network.  The proxy host only exists in there.
If you want to query Cyphernode from outside the secure Docker network, you need to expose the gatekeeper (answer Yes to that question during setup) and supply the required auth token (JWT).  See https://github.com/SatoshiPortal/cyphernode/tree/master/doc#manually-test-your-installation-through-the-gatekeeper and https://github.com/SatoshiPortal/cyphernode/tree/master/clients for client examples, as well as the Batcher (https://github.com/SatoshiPortal/batcher) which is a Cypherapp in TypeScript used by Bull Bitcoin in production.
Let me know if that answers your question and if you successfully query the API!


feat(provider): add support for linkedin

Will proceed with pull-requests one-by-one, so it will be easier to understand the set of changes

Fixes #3

Thanks for contributing!  Sorry it took me so long to merge it.  Things get added to my crazy long queue.

I updated the code a bit because LinkedIn uses space delimited rather than comma.  It also requires a state be passed.


Errors when used with Electron

It seems there are some issue when trying to use this library in an Electron app. I've attempted to get started with Angular and Vue, with different webpack configurations, but in both cases I end up having issues that start when importing this library. With the Vue project, this is the error that starts appearing as soon as I instantiate a component that imports league-connect:
polyfills.js?a0a3:31 Uncaught TypeError: Cannot read property 'match' of undefined
    at patch (polyfills.js?a0a3:31)
    at patch (graceful-fs.js?0da7:104)
    at Object.eval (graceful-fs.js?0da7:96)
    at eval (graceful-fs.js:356)
    at Object../node_modules/graceful-fs/graceful-fs.js (chunk-vendors.js:1557)
    at __webpack_require__ (app.js:849)
    at fn (app.js:151)
    at eval (index.js?ec2f:5)
    at Object../node_modules/fs-extra/lib/fs/index.js (chunk-vendors.js:1330)
    at __webpack_require__ (app.js:849)

I'm using the library from the render process, so from the actual page that is being shown. The weird thing is that I have been able to get this working when not using any libraries/webpack configuration, just plain old JS. So I suspect this has something to do with webpack.
Thanks in advance! Below I'll provide some additional information that might be useful when debugging.
(dev)dependencies
"dependencies": {
    "axios": "^0.21.0",
    "core-js": "^3.6.5",
    "league-connect": "^4.0.0",
    "vue": "^2.6.11"
  },
  "devDependencies": {
    "@vue/cli-plugin-babel": "~4.5.0",
    "@vue/cli-plugin-eslint": "~4.5.0",
    "@vue/cli-service": "~4.5.0",
    "babel-eslint": "^10.1.0",
    "electron": "^7.0.0",
    "electron-devtools-installer": "^3.1.0",
    "eslint": "^6.7.2",
    "eslint-plugin-vue": "^6.2.2",
    "vue-cli-plugin-electron-builder": "~2.0.0-rc.4",
    "vue-template-compiler": "^2.6.11"
  },

I believe the problem is using league connect from the render process as it requires to access the file system to parse the lockfile as well as being able to execute shell commands to locate the league client. From your stacktrace it seems like an error occurs in fs-extra. The library only uses fs-extra to read the lockfile so I think it's fairly safe to say the issue is it being called from the renderer.
Perhaps you could try using electron's ipc from the renderer to trigger a league-connect call on the main process?

Thanks for the swift response! The confusing thing is that I have a plain js project up and running where doing exactly this actually works. Here is some example code:
head of index.html
<head>
    <script src="https://cdn.jsdelivr.net/npm/axios/dist/axios.min.js"></script>
    <script type="text/javascript" src="index.js"></script>
</head>

index.js
const lc = require('league-connect');
const ClientService = require('./ClientService.js');

// Create the ClientService class.
clientService = new ClientService();

clientService.fetchSummonerInfo()
    .then((res) => {
        document.getElementById('page-title').innerHTML = res.displayName;
    })
    .catch((err) => {

    });

ClientService.js
// Handles requests to the League of Legends client.
class ClientService {
    constructor() {}

    /// Performs a `method` request to the given `endpoint`.
    async fetchFromLeagueClient(endpoint, method) {
        const credentials = await lc.auth()
        const response = await lc.request({
            url: endpoint,
            method: method
        }, credentials);
        const json = await response.json();

        return json;
    }

    /// Fetches information about the current summoner.
    async fetchSummonerInfo() {
        return this.fetchFromLeagueClient('/lol-summoner/v1/current-summoner', 'GET');
    }
}

module.exports = ClientService;

Hmm did you forget to enable nodeIntegration for the BrowserWindow?
https://www.electronjs.org/docs/api/browser-window#new-browserwindowoptions
new BrowserWindow({
  webPreferences: {
    nodeIntegration: true
  }
})

For angular, you should also see https://www.electronjs.org/docs/faq#i-can-not-use-jqueryrequirejsmeteorangularjs-in-electron

I managed to get this working in a Vue project. I have not been able to pinpoint the exact issue, but now I'm once again using your library for the renderer just fine... I still expect it has something to do with webpack configurations, so I've been hopping from electron boilerplate to electron boilerplate, hoping to find something that works.
Thanks for your help in resolving this, it seems my issues weren't caused by your library ;)


Change in sources provoke rebuild twice in continuous mode

Expected Behavior
Changes in task inputs provoke only one rebuild in continuous mode
Current Behavior
Changes in task inputs provoke 2 rebuilds in continuous mode
Context (optional)
No response
Steps to Reproduce
build.gradle.kts
val foo = tasks.register("foo") {
    inputs.file(projectDir.resolve("check.js"))

    outputs.file(buildDir.resolve("check.js"))

    doLast {
        copy {
            from(projectDir.resolve("check.js"))
            into(buildDir)
        }
    }
}

tasks.register("bar") {
    inputs.files(
        foo.map {
            it.outputs.files
        }
    )

    doLast {
        println(buildDir.resolve("check.js").readText())
    }
}

and file check.js inside projectDir

./gradlew bar -t
Make a change in projectDir/check.js
2 rebuilds - first one is valid with modified: ~/untitled/check.js. But the second one is weird with modified: ~/untitled/build/check.js. So Gradle triggering on change inside build directory, despite it is already processed during last build.

Gradle version
8.0
Build scan URL (optional)
No response
Your Environment (optional)
No response

Sorry that you're having trouble with Gradle!
We appreciate the effort that went into filing this issue, but we must ask for more information.
As stated in our issue template, a minimal reproducible example is a must for us to be able to efficiently track down and fix your problem. Our available resources are severely limited, and we must be sure we are looking at the exact problem you are facing.
If we have a reproducer, we may be able also to suggest workarounds or ways to avoid the problem.
The ideal way to provide a reproducer is to leverage our reproducer template. You can also use Gradle Project Replicator to reproduce the structure of your project.
This issue will be closed after 7 days unless you provide more information.

The steps provided aren't enough to reproduce. Please provide more information about your environment or a clearer scenario to reproduce.

There it is.
It is reproducer from "Steps to Reproduce"
To reproduce you need to do

gradle bar -t
change projectDir/check.js
see log

modified: ~/gradle-reproducer/check.js
Change detected, executing build...

> Task :bar
hello9

BUILD SUCCESSFUL in 26ms
2 actionable tasks: 2 executed

Waiting for changes to input files... (ctrl-d to exit)
modified: ~/gradle-reproducer/build/check.js
Change detected, executing build...

> Task :bar
hello9

BUILD SUCCESSFUL in 29ms
2 actionable tasks: 1 executed, 1 up-to-date

There is 2 build rerun, and the second one triggering on input file inside build directory
gradle-reproducer.zip

Basically, it's a race condition.
When the build is finished, we clear LocationsWrittenByCurrentBuild and do not accept updates for it, waiting for the next build to start.
During the build, filesystem changes trigger file events - they end up in the fileEvents queue. A daemon thread waits for an event from that queue. Upon receiving it, it passes the event to FilterChangesToOutputsChangesHandler which checks via LocationsWrittenByCurrentBuild whether the build changed the path.
Cleaning & processing are parallel processes and might happen in any order. Generally, that's not an issue - FS events are quite fast to notice the problem, and I can't even reproduce it with a test. But on a reproducer project LocationsWrittenByCurrentBuild are indeed cleared before FS event is processed. Approximate timeline:
0ms - build/test.js is written
1ms - bar task is executed
3ms - build finished
6ms - FS event is taken from the queue
Because the LocationsWrittenByCurrentBuild is empty after 3rd step, the build is triggered.
I suspect that this might be the problem with threads, not with FS itself: during the take the Thread is parked, and we might hit the minimum parking time. But that would require further investigation and I might be totally wrong.
I think there are two approaches to solving this:

Make sure the fileEvents queue is empty before finishing a build and/or clearing LocationsWrittenByCurrentBuild. I'm not sure how to do it properly.
Involve timestamps. Unfortunately, neither of native API has event timestamp, but we can use the time event was put into a queue. If my guess about threads is right, build finish time can be remembered in LocationsWrittenByCurrentBuild and we can ignore any message put in the queue before that. If my guess is not right, there is a sloppy way to have a cooldown period and ignore any event that happened during that period (but it can be a genuine change by an external tool watching the gradle build).

I’m not sure how these would work.

Do you mean clearing the queue? There are legitimate reasons to re-trigger a build if for example the user has changed an actual input during the build (e.g. they saved a source file in the IDE after the build has already dealt with building that source file).
We could make sure we process all the events before we clear LocationsWrittenByCurrentBuild, but delivery of FS events can always be delayed. So we might reduce the chance of such a problem occurring, but it would be eliminated completely.

This seems like a variation of 1), though we could call the build finished a tiny bit earlier.

I think the fundamental problem is that we can’t really distinguish between legitimate changes made by the user that should re-trigger the build, and changes that happened due to the build execution (but reported late).
Establishing a (configurable?) cool-down period might be the right way to handle this indeed…

Yep, I agree, and I also don't think it's a feasible solution. I meant to process all events before we clear LocationsWrittenByCurrentBuild.
Yes, this wouldn't solve the problem entirely, but we can reduce the probability of handling stale updates. If FS events are generated very slowly themselves, I don't think there is much we can do - I would consider it as OS-related issue.


Write medium article + shaman's post about profiler

Which components does the task require to be changed? (think hard pls)
cairo-profiler
Description
Self-explanatory.

https://github.com/software-mansion/cairo-profiler/issues/72


Error TS6053 Cannot find extended tsconfig

Title: Error TS6053: File 'my-awesome-config' not found when using 'extends' in tsconfig
Description:
Hey there,
I'm encountering an issue while trying to generate .d.ts files. My tsconfig file extends a configuration called "my-awesome-config" as shown below:
// tsconfig.json
{
   "extends": "my-awesome-config",
   // ... other configurations
}

The "my-awesome-config" is a workspace within my npm monorepo, and it's symlinked in the root node_modules directory. However, when running dts-buddy, I'm facing the following error: "error TS6053: File 'my-awesome-config' not found". It's worth noting that the TypeScript compiler and VS Code have no issues with this module.
Any insights or help on resolving this would be greatly appreciated. Thanks!

Hi, please provide a link to a repository with a reproduction.
dts-buddy uses typescripts own functions to parse the config, so i'm a bit surprised you have no issues with other tooling.
https://github.com/Rich-Harris/dts-buddy/blob/985ca33abbae36e7a08e132693149bcadad3be73/src/utils.js#L523

Hi @dominikg ,
Here's the repo link: Your Repository URL
Thanks for the help!

@dominikg  Hi there! Are you thinking about watching it in the near future?


upgrade-deps script

This script could help for #7.
The missing parts are:

automatically create a push request (sadly, seems not so trivial)
run tests (should be done on the mentioned push request)

Maybe we should better consider using the tool, you've sent me on TG?

Yup, dependencies.io looks really good!
Anyway, in order to compare the results, this is what my script says right now:
  "dependencies": {
    "body-parser": "^1.18.2",
    "dotenv": "^5.0.1",
    "express": "^4.16.2",
    "node-telegram-bot-api": "^0.30.0"
  }

@serl can you please show me a copy of your config.yml file?


Purge logic
if (req.method == "PURGE") {
if (client.ip !~ purgers) {
return (synth(403, "Not allowed."));
}
if (req.http.X-Host) {
set req.http.host = req.http.X-Host;
}
return (purge);
}
Ban logic
if (req.method == "BAN") {
if (client.ip !~ purgers) {
return (synth(403, "Not allowed."));
}
if (req.http.Cache-Tags) {
ban("obj.http.Cache-Tags ~ " + req.http.Cache-Tags);
return (synth(200, "Ban added " + req.http.host));
}
if (req.http.X-Url) {
ban("obj.http.X-Url == " + req.http.X-Url);
return (synth(200, "Ban added " + req.http.host));
}
return (synth(403, "Cache-Tags or X-Url header missing."));
}
...
}

https://github.com/mittwald/kube-httpcache/blob/master/README.md#using-built-in-signaller-component


add develop script to view and update test data

try add a dev script to start a dev service base on the test data (models/fixtures/, tests/gitea-repositories-meta and tests/gitea-lfs-meta). then the developer can check and update the test data by ui operation on the test server. I think it will be a more simple way than manually edit. example:
https://github.com/go-gitea/gitea/assets/25342410/85133c57-704c-475e-a07b-cb41176af319
TODOs:

[x] looks need rewrite a dumper to have a better format.

How about to implement MashalYML and UnMashalYML? Or other yml library supported interface?

I do not think it is maintainable to introduce a new "contrib/dev/dev.go" with a lot of magic strings in it.
I have a feeling that many existing test-related code could/should be reused.

And there is a security problem: if by default "git hook" is enabled , and the passwords are well-known ones, attackers could hack into a Gitea developer's computer.

I have a feeling that many existing test-related code could/should be reused.

I think I hasn't change any test data now. just update a little test datas which has bug ...

And there is a security problem: if by default "git hook" is enabled , and the passwords are well-known ones, attackers could hack into a Gitea developer's computer.

I think it's not a problem, developer just use it to edit test data, which should not be running as a public web server.

And there is a security problem: if by default "git hook" is enabled , and the passwords are well-known ones, attackers could hack into a Gitea developer's computer.

oh, sorry I fogot the git hook, I will make it be disabled always.

And there is a security problem: if by default "git hook" is enabled , and the passwords are well-known ones, attackers could hack into a Gitea developer's computer.

luckly looks git hook has been disabled by default, So I thin it's safe. and maybe I can add a logic to update all password to a random value before the dev server start and reset it to the test data when the server stop.


Document the production Redis hosting

Close #3332

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS against 668b28decd5b83c918c58eab331eab27bd764afe


Custom backends support

Allows external crates to write their own Sender implementations and use them with Client.
This involved refactoring a decent amount of the crate, and probably has some backwards-incompatible changes. I didn't change any of the usual APIs though; most code that just does querying should still work.
The primary impetus of this change is getting this crate to work in WebAssembly. In that environment, requests must be made via the XMLHttpRequest API, and this crate must return JS promises. I have written such a backend, which serves as an example; code is here: https://gitlab.com/colonelthirtytwo/elastic-stdweb-rs
Major changes off the top of my head, see commits for more details:

Add TypedSender<TReqInner>, a subtrait of Sender that all backends should implement for all T: ReqInner. This handles deserializing the response from the backend once it is available (ex. using Result.map for SyncSender, Future::map for AsyncSender, and Promise::then for StdwebSender). The response type (an associated type of the trait) has to vary depending on the type being deserialized, hence why this is a separate trait from Sender. Once Generic Associated Types are stabilized, the trait can be merged into Sender.
Merge the different implementations on request types for SyncSender and AsyncSender into one that uses the TypedSender. Rust orphaning rules would forbid implementing Request<MySender> in external crates, so that approach would not scale.
Remove Sender field from SniffedNodes and replace it with an API that was not responsible for fetching the request, since how to fetch the request would change based on backend.
Because of the above, remove the NextParams trait and move its functionality to Sender::next_params.
Create a few traits and implement them on API structs to allow for more introspection to support TypedSender.
Made several sender-related things public so that crates implementing sender can use them.
The SyncSender, AsyncSender, and related code are now behind cargo features, allowing this crate to be built without them (default is to include both, for backwards compatibility). WASM needs this, since the reqwest-based senders won't even build for that target.

Fixes #281

FYI I haven't tested this too hardly yet, but I'm planning on working on a project that makes use of this crate on WASM, so that should exercise all the functionality.

Wow! Thanks for the PR! This is pretty big. So does this allow implementing/creating your own Sender as discussed in #281?
I tried updating to futures 0.3 yesterday and it is a doozy... I wonder if this could help? Would it be crazy if we just remove sync sender now that async/await is stable? I digress and I'll open a different issue to discuss further.

Yes, you can implement your own sender in your own crate. Example for wasm using stdweb: https://gitlab.com/colonelthirtytwo/elastic-stdweb-rs
IMO I'd rather keep syncsender around; tokio and the async stuff is a bit heavy and I would not be surprised if people would not want to include all that.

Any chance you can rebase this onto master so we can run the tests? I assume they might need to be updated too?

Could do. Just wanted a look-over of the code beforehand in case I had to change anything.

One issue I've noticed is that since send is now implemented on RequestBuilder, there is no documentation for send on the request type's pages, which isn't very helpful.

...and now with the new version, cargo keeps pulling in reqwests even with default-features: false set... humm

I am just now starting to look more at this and I have a few initial questions:

Am I to understand that when Generic Associated Types stabilizes we won't need both the Sender & TypedSender<TReqInner> traits? It will just be the Sender trait with an associated type that indicates the type of the sender (WASM vs async etc.)
Is it worth adding your WASM implementation into this crate behind an off by default feature? It doesn't look that big/complicated and might be nice to have...?
Should we make not including sync-sender the default and just make async-sender the default? reqwest is going this route with their next breaking change release (see: https://github.com/seanmonstar/reqwest/commit/7e3c1bc461dd51a9b59ca6b06b82e1699cc1a1ce). Our next release v0.21.0 will have a few breaking changes in it already. Granted this would be a much bigger breaking change but might be worth it? I guess it doesn't pull in any extra dependencies so it might not matter?

I'll add more to my review asap...

Correct. The Sender trait could have a type InProgressResponse<Response>;, which is Result<Response, Error> for SyncSender, Pending<Response> for AsyncSender, and Promise<Response, Error> for StdwebSender. Then typed_send could be moved to the Sender trait and return Self::InProgressResposne<SomeResponseType>.
I'd rather polish the stdweb sender a bit more then include it in a future PR. There are some things I'd like to change with it.
Huh, didn't know that reqwests was turning off the blocking stuff by default now. Doesn't the async stuff require setting up an executor of some sort? Won't that be inconvenient for simple programs? Or did that change with async stabilization?

Nice! That seems very clean.
Makes perfect sense. 👌
The alpha crate is setup like that and I would assume the final release will be too; I haven't seen any Indication otherwise. Yeah, it is kinda annoying to setup (although it should be as easy as a #[tokio::main] or #[runtime::main]—assuming the prerequisite dependencies are all downloaded and in order) and then using async/await. I bet their thinking is that while it is a bit more annoying, it kinda forces the ecosystem into using async I/O which is a benefit in performance and utility in the long run...? But I can't speak for the reqwest devs of course.


Fix docs

Reopening PR to test readthedocs

@Ngoguey42  Docs are finally passing. CI failed due to #14

🤩 Thanks a lot


Fix propagation of user-code across Abstract Closure boundaries

Was accidentally blocking propagation of all effects.

LGTM but I'm not going to bother upstreaming this right away, since there are no other effects in use.


Notifications doesn't removes from DOM

Hi! I've explored your project's demo site and saw, that currently notifications doesn't removes from DOM, when clicking on 'close' button. Is it desired behavior?

It must be a bug introduced with the latest version of material ui. Are you using a local copy of the demo? Can you also tell me your browser and environment (desktop / mobile)

Chrome 55,
react: 15.4.1
material-ui: 0.16.5,
desktop env.
I saw that on your https://puranjayjain.github.io/react-materialui-notifications/

it can't remove Notificaiton Dom when click the close button

lastest version material-ui  did't support key prop.

@trelly thank you for pointing out the reason shall fix this.

this has been fixed in #116


Allow setting Meta.list_serializer_class

The way that DynamicModelSerializer is currently written it is not possible to define a custom list_serializer_class, as that attribute is overridden in the __new__ method. Provided that the user-defined list_serializer_class extends DynamicListSerializer, it should be settable.

:+1: looks good to me, thanks for your contribution :)

Thanks for the quick response!

Has this feature been tested? I'm replaced the default list_serializer_class, first in the Serializer.Meta and then globally, but in both cases it doesn't seem to take effect. I would expect the MyDynamicListSerializer.to_internal_value to be called before the actual serializers to_internal_value, but it doesn't.
My code is pretty straightforward:

class InvoiceItemSerializer(DynamicModelSerializer):
    product = DynamicRelationField(ItemSerializer)

    class Meta:
        model = InvoiceItem
        fields = [
            'id', 'product', 'name', 'description', 'quantity', 'unit_price',
        ]
        read_only_fields = ['id']
        list_serializer_class = MyDynamicListSerializer

class InvoiceSerializer(DynamicModelSerializer):
    items = DynamicRelationField(InvoiceItemSerializer, many=True, deferred=True)

    class Meta:
        model = Invoice
        fields = [
            'id', 'date', 'number', 'customer', 'terms', 'notes', 'total', 'balance', 'items',
        ]

I even tried declaring:
class MyDynamicRelationField(DynamicRelationField):
    class Meta:
        list_serializer_class = MyDynamicListSerializer

and using it instead of DynamicRelationField, but to no avail.
What's going on?


Add v1.3.1 changelog entry

🔧 Changes
Adds the changelog entry for v1.3.1 release
📚 References

🔬 Testing

📝 Checklist

[x] All new/changed/fixed functionality is covered by tests (or N/A)
[x] I have added documentation for all new/changed functionality (or N/A)

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

Comparison is base (9c9892c) 95.18% compared to head (00f1fbb) 95.18%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #320   +/-   ##
=======================================
  Coverage   95.18%   95.18%
=======================================
  Files          46       46
  Lines        8956     8956
=======================================
  Hits         8525     8525
  Misses        329      329
  Partials      102      102

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Closing as we'll release this later

Closing as we'll release this later

hi @ewanharris do you know when this release will be made? TIA 🤝

@caleygoff-invitae there are some other items we want to get into this so I think it'll be a couple weeks before this ultimately gets released


Can we customize the contaier name created by mesos?

By default mesos creates container name staring with mesos-<long number
can we overiride this? can we specify what could be the container name for an app?

Duplicated by https://github.com/mesosphere/marathon/issues/4805

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-4196. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-4196. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


Adding Teams signin status

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/8618

@cazawideh PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


@aspnet\signalR typescript - HttpConnection.ts

I am trying to run the typescript client for signalR and connect to a Hub server running as a console app using .Net framework 4.6.1.
The code below is parsing content to a string and then tries to create an object of type INegotiateResponse. Problem is the case for the ConnectionId property as declared in the interface INegotiateResponse has an upper case 'C', while the one contained in the negotiateResponse object has a lower case 'C'. Therefore the connectionId is set to undefined.
Below is the interface definition from HttpConnection.ts
interface INegotiateResponse {     ConnectionId: string     availableTransports: string[] }
https://github.com/aspnet/SignalR/blob/9482b4976e071207ba3fcc53027346404033afbb/client-ts/signalr/src/HttpConnection.ts#L86
Also createTransport method blows up,  because the  negotiateResponse.availableTransports array is undefined! also i do not see it in the unparsed negotiatePayload.content.
https://github.com/aspnet/SignalR/blob/9482b4976e071207ba3fcc53027346404033afbb/client-ts/signalr/src/HttpConnection.ts#L95
Package.json
"@aspnet/signalr": "^1.0.0-preview1-28189",
"@aspnet/signalr-protocol-msgpack": "^1.0.0-preview1-28189",
To reproduce,

Grab the repo from https://github.com/aspnet/SignalR/
Build everything in VS2017
create a hubconnection to a signalr hub
 let hubConnection = new HubConnection("http://localhost/signalr"); hubConnection.start().then(() => { console.log("connected to hub"); }).catch((error) => { console.log(error); });

FYI, for those trying to use signalR typescript client with a .net signalR server. you can use typed libraries for signalR(@types/signalr) and Jquery(@types/jquery)
Then create your client interfaces for your HubProxy and then you should be able to reference the client/hub methods from angular components.

FYI, for those trying to use signalR typescript client with a .net signalR server. you can use typed libraries for signalR(@types/signalr) and Jquery(@types/jquery)
Then create your client interfaces for your HubProxy and then you should be able to reference the client/hub methods from angular components.

Do you have any samples how this can be done? I am using Angular and wanted to connect SignalR hub running on .net framework 4.6.2.


WIP: use trading calendars

Replace zipline.utils.calendars with trading_calendars.

Coverage decreased (-0.4%) to 85.933% when pulling 01e967ba105507ff3642c03ede242279142ba989 on use-trading-calendars into 7056a9091423d1465968d21498b18de75ba7c6c6 on master.

@ssanderson mind taking a look?

LGTM once the buidl passes.


Fix the capacity queries

Capacity queries are using 'kubelet_volume_stats_used_bytes', which
gives the result in a per node basis (evenwhen the nodes are sharing
a single PVC).
Corrected the query to give only one result per +
combination.
Signed-off-by: Arun Kumar Mohan<EMAIL_ADDRESS>

/retitle Bug 2104492: Fix the capacity queries

/approve
/lgtm


Add support for LBRY Credits

To do:
Dev:

[ ] support for LBRy on Cloudchains back-end infra
[ ] add LBRY to XLite configs

UI:

[x] prepare LBRY asset icon
[x] prepare LBRY info links

QA:

[ ] test XLite LBRY tx's and UI

Info link: https://lbry.com
Explorer: https://explorer.lbry.co
Icon assets:

m

explorer.lbry.com :)


fix docs

Please check if the PR fulfills these requirements

[x] Followed the Contributing guidelines.
[ ] Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Documentation updates

@corradodellorusso - thanks!


Git version in binary

Automate the value of variable GitCommit in the binary

Now the binary should contain the git tag and the git hash


Improve schema command

Fix JSON output of schema command
Improve error handling for unsupported schema formats

The improved error handling is clear to me, we should def. have that, yup!
Not obvious at first what exact behavioral change "fix JSON output" relates to?
There's a linter error and a couple of test failures that you'd need to look into.
Optionally you could submit these two separate issues in separate pull requests,
which'd make it easier for us to start to pull in at least some of the work.

The problem with returning a bytestring is that it outputs b'...' on the command line. This makes it impossible to use it to extract the schema like that:
`$ apistar schema > api-schema.json`

I especially stumbled across this as I wanted to get pretty printing using jq:
$ apistar schema | jq
I forgot to update the tests which should test for unicode strings.

I just fixed the linting errors and tests. You're right that I could split this into two PRs but as both touch the same file with two adjacent lines I wanted to keep it simpler to merge them.

Fab, thanks!


[Depfu] Add support for Gitlab

Fixes #7474

Following the recommendation https://github.com/badges/shields/issues/7474#issuecomment-1009239968 of @chris48s, this PR adds a :vcsType parameter to depfu URLs to select between Github and Gitlab. Badges without a VCS type are redirected to the Github route via a legacy route.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @kris7t!

  Generated by :no_entry_sign: dangerJS against ef98d3d514e2a7750b39e6da80715ff0eb0fa4b0

maybe not worth discussing as part of this PR since the route change is both consistent with what we had before as well as other dependency management/update services, but it seems like this class of badge routes are all missing the NOUN portion from our canonical badge url pattern

@calebcartwright  I think if we want to change it to /depfu/dependencies/(github|gitlab)/:project+ we might as well do it now in this PR. Then we only need to add one redirect from /depfu/:user/:repo to /depfu/dependencies/github/:project+. If we ship it with /depfu/:vcsType(github|gitlab)/:project+ and then change it in another PR we need another redirect to avoid a BC break. Lets not do that if we don't need to. That said, we're doing the same thing on requires.io and libraries.io so I can see a case for just ignoring it.

I just created an example project to test this. It might be a bit trickier, because Depfu seems to require urlencoded subgroup separators: https://depfu.com/gitlab/shields/shields-example-group%252Fsubgroup/example-nodejs (replacing %252F with / doesn't work).
I guess the best approach would be to parse the :project+ argument, and replace all but the last / characters with %252F?

Lets use encodeURIComponent() instead of manually replacing.
For the Gitlab case we can do something like
encodeURIComponent(project.substr(0, project.lastIndexOf('/')))
project.substr(project.lastIndexOf('/'))

We'll also want to make sure project has at least one / in it before doing that, so something like
if (!project.includes('/')) {
  throw InvalidParameter()
}

https://github.com/badges/shields/blob/23c0406bedfc6930735e8f5ea75dfe34faf1f290/core/base-service/errors.js#L157
For the Github case, we can just pass project as-is

I updated this PR according to the review comments:

The endpoint was moved to /depfu/dependencies.
If there is no / in the project, we throw an InvalidParameter error. I left it with a default message, because I couldn't come up with any specific prettyMessage that wasn't way too long.
I added the URI encoding regardless the vcsType to simplify logic (it doesn't do anything for Github, but it won't hurt either).
For testing, I use the example group, subgroup, and repo that I created on my Gitlab account (the test nested repo has to have Depfu enabled, but I couldn't find any existing example repo that was nested in a subgroup).


dart analyze --packages should allow a relative path

The --packages argument on the dart analyze command currently requires the provided path to be an absolute path. It ought to allow the path to be a relative path.

This might do it?
https://dart-review.googlesource.com/c/sdk/+/254901


Modularize etherlime to reduce size of dependencies

Is it possible to modularize the features in etherlime to reduce the size of the dependencies?

Hey, Since 2.0.0 Etherlime CLI is separate from Etherlime-lib. You no longer need to include etherlime as dependency in your project. Etherlime-lib is very lightweight.
https://etherlime.gitbook.io/etherlime/developer-documentation/migration-to-new-version
Is there a specific feature you look to exclude?

@thegostep Did you try Etherlime > 2.0.0 versions ? Do you need something specific ?

I could imagine the compiler being a module and having a different package for vyper vs solidity. Maybe Coverage being a distinct module too.

The zk libs could be separate as well

@thegostep We are going to include the separation of zero knowledge proof libs in our roadmap as well. Thank you.


Create full-stops

https://github.com/govau/content-guide/issues/144

Re-opening as the changes have not actually been merged.


blank map on Android

I am getting a blank map with just the Google logo on Android emulator. I have enabled my API key and it is valid.
ENV INFO:
System:
    OS: Windows 10
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
    Memory: 4.78 GB / 15.80 GB
  Binaries:
    Node: 12.7.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.17.3 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 6.11.3 - C:\Program Files\nodejs\npm.CMD
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.2 => 0.61.2

package.json:
{
  "name": "reactNativeMaps",
  "version": "0.0.1",
  "private": true,
  "scripts": {
    "android": "react-native run-android",
    "ios": "react-native run-ios",
    "start": "react-native start",
    "test": "jest",
    "lint": "eslint ."
  },
  "dependencies": {
    "react": "16.9.0",
    "react-native": "0.61.2",
    "react-native-maps": "^0.26.1"
  },
  "devDependencies": {
    "@babel/core": "^7.6.2",
    "@babel/runtime": "^7.6.2",
    "@react-native-community/eslint-config": "^0.0.5",
    "babel-jest": "^24.9.0",
    "eslint": "^6.5.1",
    "jest": "^24.9.0",
    "metro-react-native-babel-preset": "^0.56.0",
    "react-test-renderer": "16.9.0"
  },
  "jest": {
    "preset": "react-native"
  }
}

Usage:
import React, { Component } from 'react';
import { Dimensions, StyleSheet, View } from 'react-native';

import MapView from 'react-native-maps';

export default class App extends Component {
    render(){
        return(
           <View style={styles.container}>
                <MapView
                    style={styles.map}
                    initialRegion={{
                        latitude: 37.78825,
                        longitude: -122.4324,
                        latitudeDelta: 0.0922,
                        longitudeDelta: 0.0421
                    }}
                />
            </View>
        );
    }
}

const styles = StyleSheet.create({
    container: {
      position: 'absolute',
      top: 0,
      left: 0,
      right: 0,
      bottom: 0,
      justifyContent: 'flex-end',
      alignItems: 'center'
    },
    map: {
      position: 'absolute',
      top: 0,
      left: 0,
      right: 0,
      bottom: 0
    }
  });

Here are the relevant prints from logcat related to Google Maps Android API.... is it the mismatch in client vs package versions causing the issue?
LOGCAT:
10-08 13:04:19.612  3756  3756 I Google Maps Android API: Google Play services client version: 11910000
10-08 13:04:19.628  3756  3756 I Google Maps Android API: Google Play services package version: 16089022
SHOT:

I had a similar issue to yours.
To ensure your API Key is valid, try to install & run the app on a physical Android device. If the issue disappears, then it is the emulator issue (even if it is was working before on the same emulator - like my case)
Potential solutions to the emulator:

Ensure you enable Google Play services https://github.com/react-native-community/react-native-maps/issues/2778#issuecomment-486387049
Cold boot
Normal boot
Wipe emulator's data
Delete and re-create the emulator again

Thanks but the potential emulator solutions did not work. My emulator was missing the Play Store so I created a new device with Play Store installed by default but still no luck.
Going to try installing on a physical device now.

This is an API key issues. The map itself works because the logo is there

This is an API key issues. The map itself works because the logo is there

So what is the issue?
I see traffic metrics for this API key in Google console... what other ways can I validate the key?

@MOwais

I would say specifying the API key with google project

attaching it with your android app package name along adding SSH Key for it! on the console of google
In short making it bind with the single app of yours might help! try that!

@MOwais

I would say specifying the API key with google project
attaching it with your android app package name along adding SSH Key for it! on the console of google
In short making it bind with the single app of yours might help! try that!

Didnt quite get you... how/where do I specify the API key with Google project and attach it with my Android package name?
Can you please leave an example.

I'm facing the same issue here but with a physical device.
I generated my API key following Google Documentation, this repo's doc recommendation.

@MOwais

I would say specifying the API key with google project
attaching it with your android app package name along adding SSH Key for it! on the console of google
In short making it bind with the single app of yours might help! try that!

Didnt quite get you... how/where do I specify the API key with Google project and attach it with my Android package name?
Can you please leave an example.

I think he's refering to Google Cloud Platform key restriction, I did that but with no luck and the blank map persists, I encorage you to do the same hoping for a different outcome.

I just enabled Google Maps SDK Android for my project and for a couple minutes the map was still blank, but now its working..
@MOwais make sure to enable that!

I'm facing the same issue here but with a physical device.
I generated my API key following Google Documentation, this repo's doc recommendation.

@MOwais

I would say specifying the API key with google project
attaching it with your android app package name along adding SSH Key for it! on the console of google
In short making it bind with the single app of yours might help! try that!

Didnt quite get you... how/where do I specify the API key with Google project and attach it with my Android package name?
Can you please leave an example.

I think he's refering to Google Cloud Platform key restriction (as shown here), I did that but with no luck and the blank map persists, I encorage you to do the same hoping for a different outcome.

no restrictions were ever applied. API key has been enabled for almost a week now.
Stumped.

I generated an app using expo (instead of react-native cli) and used the same API key and the map now loads....
Issue is now resolved, although I'm still not sure what the cause was.

Hi @MOwais ,
I had exactly the same issue as you, I was coming from Expo (where it worked) to the bare workflow where the maps suddenly didn't load.
I solved it by:

Adding my keystore.jks file from Expo to the android/app/ directory.
Adding the following to my gradle.properties file in android/:
MYAPP_UPLOAD_STORE_FILE=KEYSTORE_FILE MYAPP_UPLOAD_KEY_ALIAS=KEY_ALIAS_FROM_EXPO MYAPP_UPLOAD_STORE_PASSWORD=KEYSTORE_PASSWORD_FROM_EXPO MYAPP_UPLOAD_KEY_PASSWORD=KEY_PASSWORD_FROM_EXPO
And altering the following in my build.gradle file in android/app/:
signingConfigs {      debug {          storeFile file('debug.keystore')          storePassword 'android'          keyAlias 'androiddebugkey'          keyPassword 'android'      }      release {          if (project.hasProperty('MYAPP_UPLOAD_STORE_FILE')) {              storeFile file(MYAPP_UPLOAD_STORE_FILE)              storePassword MYAPP_UPLOAD_STORE_PASSWORD              keyAlias MYAPP_UPLOAD_KEY_ALIAS              keyPassword MYAPP_UPLOAD_KEY_PASSWORD          }      } } buildTypes {      debug {          signingConfig signingConfigs.release      }      release {          // Caution! In production, you need to generate your own keystore file.          // see https://reactnative.dev/docs/signed-apk-android.          signingConfig signingConfigs.release          minifyEnabled enableProguardInReleaseBuilds          proguardFiles getDefaultProguardFile("proguard-android.txt"), "proguard-rules.pro"      } }
Make sure your SHA-1 Certificate from Expo for your app has been added to your Maps SDK for Android on GCP.

The reason it worked before is because Expo automatically used their generated keystore to sign you app, now you just need to manually put those credentials into the build time for the bare workflow.
After doing these steps your debug app build and your production app build will be signed with your the expo keystore. Therefore both build types are able to use the Maps SDK from Google.

I also face same problem.
IOS is ok, but blank map on android with eas build.
Reason is google map api authorization. (logo exists, but map is brank)
Document was updated on Fri Nov 25 2022.
https://docs.expo.dev/versions/latest/sdk/map-view/

Deploy app with Google Maps
Android

Copy your app's SHA-1 certificate fingerprint
For development builds

If you have already created a development build, your project will be signed using a debug keystore.
After the build is complete, go to your project's dashboard, then, under Configure > click Credentials.
Under Application Identifiers, click your project's package name and under Android Keystore copy the value of SHA-1 Certificate Fingerprint.

Not only uploading Google Play Store's SHA-1 certificate fingerprint on GCP's google map api,
but we need to upload  Expo's SHA-1 certificate fingerprint on GCP (project's dashboard > Configure > click Credentials >  Application Identifiers > click your project's package name and under Android Keystore),
After uploading Expo's SHA-1 certificate fingerprint on GCP, app loads map successfully.


add mention of song artist

https://timbeek.com/royalty-free-music/jingle-bells-8bit/

add to info overlay


fix additional repos, only skip for Micro

What does this PR change?
additional repos should be added no matter the role, but only be skipped if the os is sle micro or opensuse micro

This reverts https://github.com/uyuni-project/sumaform/pull/1625 . Additional repos in SLE micro are being added in combustion, otherwise, if you do it with salt, you need a reboot to have them "activated".

This reverts #1625 . Additional repos in SLE micro are being added in combustion, otherwise, if you do it with salt, you need a reboot to have them "activated".

The problem is that repos are being added twice, with combustion and with salt. Thus, applying the salt states gives an error stating that these repos are already added.
However, I see it was added for BV 5.5 ??


Error with custom UserProvider

Hi,
I tried to use your package with a custom UserProvider
Argument 1 passed to Kbwebs\MultiAuth\Guard::__construct() must implement interface Illuminate\Contracts\Auth\UserProvider, instance of Illuminate\Auth\Guard given

in Guard->__construct(object(Guard), object(Store), 'user') in AuthManager.php line 47

I registered my UserProvider as written in Laravel's doc (I use Laravel 5.1)
Auth::user()->extend('user', function($app) {
    return $this->app->make(\App\Auth\UserProvider::class);
});

Actually I wrote a dirty fix, I registered my provider like that.
Auth::user()->extend('user', function($app) {
    return new \Kbwebs\MultiAuth\Guard($app->make(\App\Auth\UserProvider::class), $app['session.store'], 'user');
});

I'm also experiencing this issue.

Hi guys,
I tried to create a custom EloquentUserProvider about your package and I also had the same problem. I searched for the problem and after some time, I realized that the method callCustomCreator in Kbwebs\MultiAuth\AuthManager class return a Guard type instead of a Provider type.
To resolve this issue, I chained a method getProvider() to return to be a Provider type.
/**
 * Call a custom driver creator.
 * @param  string  $driver
 * @return \Kbwebs\MultiAuth\Guard
 */
protected function callCustomCreator($driver)
{
    $custom = parent::callCustomCreator($driver)->getProvider(); <--------
    if($custom instanceof Guard) {
        return $custom;
    }
    return new Guard($custom, $this->app['session.store'], $this->name);
}

I have not tested to custom DatabaseProvider, that is why I have not sent a PR.
I hope that help.


wierd error  apollo graphql 400 error

Unable to access the GraphQL API.
Is it running and accessible at http://localhost:3000/graphql-alpha from the Storefront UI server.
Unable to access the GraphQL API. Is it running and accessible at http://localhost:3000/graphql-alpha from the Storefront UI server?
{ Error: Network error: Response not successful: Received status code 400
    at new ApolloError (/home/moustafabauommy/Desktop/WORK/reaction-platform/reaction-next-starterkit/node_modules/apollo-client/bundle.umd.js:99:32)
    at /home/moustafabauommy/Desktop/WORK/reaction-platform/reaction-next-starterkit/node_modules/apollo-client/bundle.umd.js:967:45
    at /home/moustafabauommy/Desktop/WORK/reaction-platform/reaction-next-starterkit/node_modules/apollo-client/bundle.umd.js:1329:21
    at Array.forEach (<anonymous>)
    at /home/moustafabauommy/Desktop/WORK/reaction-platform/reaction-next-starterkit/node_modules/apollo-client/bundle.umd.js:1328:22
    at Map.forEach (<anonymous>)
    at QueryManager.broadcastQueries (/home/moustafabauommy/Desktop/WORK/reaction-platform/reaction-next-starterkit/node_modules/apollo-client/bundle.umd.js:1323:26)
    at /home/moustafabauommy/Desktop/WORK/reaction-platform/reaction-next-starterkit/node_modules/apollo-client/bundle.umd.js:917:35
    at process._tickCallback (internal/process/next_tick.js:68:7)
  graphQLErrors: [],
  networkError:
   { ServerError: Response not successful: Received status code 400
       at throwServerError (/home/moustafabauommy/Desktop/WORK/reaction-platform/reaction-next-starterkit/node_modules/apollo-link-http-common/lib/bundle.umd.js:36:21)
       at /home/moustafabauommy/Desktop/WORK/reaction-platform/reaction-next-starterkit/node_modules/apollo-link-http-common/lib/bundle.umd.js:64:17
       at process._tickCallback (internal/process/next_tick.js:68:7)
     name: 'ServerError',
     response:
      Body {
        url: 'http://localhost:3000/graphql-alpha',
        status: 400,
        statusText: 'Bad Request',
        headers: [Headers],
        ok: false,
        body: [Gunzip],
        bodyUsed: true,
        size: 0,
        timeout: 0,
        _raw: [Array],
        _abort: false,
        _bytes: 171 },
     statusCode: 400,
     result: { errors: [Array] } },
  message:
   'Network error: Response not successful: Received status code 400',
  extraInfo: undefined }
[GraphQL error]: Message: Cannot query field "displayTitle" on type "Tag"., Location: [{"line":36,"column":3}], Path: undefined```

@MoustafaBayommy Have you resolved this issue? It seems like your API container (the one running at port 3000) was shut off, this is usually when this error occurs. If you restart the API container, this error should go away.

@kieckhafer  yes it has been solved after i re make the whole reaction  platform

@moustafabayommy has stated this was resolved. Closing.


Rename package paths and group ids

@twebermartins I have just also increase the revision (0.1.0-SNAPSHOT --> 0.2.0-SNAPSHOT) due to the many changed paths. Please provide a quick review before I merge it.


added strikethrough command

++media/icons (x24 black+white from material.io)
https://material.io/icons/#ic_strikethrough_s
++README.md
++commands.js
++package.json
command, menu item, editor item
reordered so that strikethrough follows italic

Thanks for the contribution, @seanmft!


When Adding SSH Public and Private keys via the GIT Repo Config page, SSH key ends up as null.

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I am trying to connect my AWS Code Commit repo to Rancher Fleet GIT Repos. I enter in my public and private keys in OPENSSH format. Once I save and look back at the config it shows SSH Key: Null
This is the Error I see on the Git Repo dashboard:
Git ls-remote  ssh://git-codecommit.us-gov-west-1.amazonaws.com/v1/repos/linc-noss-fleet  refs/heads/main error: exit status 128, detail<EMAIL_ADDRESS>Permission denied  (publickey).  fatal: Could not read from remote repository.  Please make sure you have the correct access rights and the repository exists.
the repository exists as I am copying it from AWS CodeCommit
Expected Behavior
SSH Keys to be generated properly
Steps To Reproduce

Continuous Delivery > Git repo > Add Repository
Use SSH keys generated from AWS Code Commit in OpenSSH format
Save
Look at the config for the repo and notice 'SSH Key: null'

Environment
- Architecture:
- Fleet Version:
- Cluster:
  - Provider:
  - Options:
  - Kubernetes Version:

Logs
No response
Anything else?
No response

I am able to see the secrets created via GitRepo UI page, already created secrets (http/ssh) both are populated. Also, used them in order to deploy applications from private GitHub repository.


Green Background for Allied Races

When I preview (CTRL + LMB) an Item with one of the Allied Races, the background is just green - and i can't change it with CTRL+Scroll.
But when I select one of the Allied Races in the drop down Menu, the Background is normal and changeable.

Oops. The addon is missing the allied race background images. Need to look into adding them in.

Thank you.

Solved the problem for version 2.3.3. Addon should update on Curse soon.

This issue has cropped up again with the newly added Kul Tirans and Zandalari, alongside the inability to preview the new races

I don't have a subscription anymore and altough it's free weekend right now I don't have the races unlocked either. Can't give an estimate as to when I'll get to around adding support for them.

Not sure if it works


Missing mobilenet-ssd.blob

After following steps from here there doesn't seem to be mobilenet-ssd/mobilenet-ssd.blob, so the last step from here fails.
Just a minor issue, but may be annoying for people just starting out.

Thanks @mariusz-r for reporting!  Tagging @jonngai on this.  IIRC he was just mentioning updating this.


fix: add queryAllName to security warning

Hello, this repo is really awesome, I've been using this for few days.
I've been generating the permissions for graphql-shield using the queries and mutations variable inside addCrudResolvers.ts.
While doing so, I noticed duplicate rule being created because queryName specified twice inside the queries.
This diff will replace the duplicate queryName with queryAllName.

@darcien thank you for your contribution and for your feedback!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Command failed with exit code 1: npm install<EMAIL_ADDRESS>

Bug description
I have a database of mysql, and now I want to generate a table in the database,so I wrote schema and then run "npx prisma migrate dev --name init" but I received the following error:

How to reproduce
1.I created a MySQL database named mydatabase1
2.I created a prisma project refered to official documents like this(pn is pnpm,I used alias):

OK,I successfully created a prisma project and created a .env file and then I modified it like this:

3.I wrote a model like this:

4.I run "npx prisma migrate dev --name init"but received the following error:

What did I do wrong?Thanks.
Expected behavior
It seems that it reminds me that I need "npm install npm install @prisma/client@4.5.0",but isn't it downloaded with prisma?And I didn't see it on the official website saying that additional downloads are needed about @prisma/client
Prisma information

Environment & setup

OS: Windows
Database: MySQL
Node.js version: v16.15.1

Prisma Version

Are you using an Npm mirror maybe?
Generally prisma generate (the command that is run in the background of migrate dev will auto install @prisma/client to your package.json. That sometimes fails with other package managers like pnpm, and then things explode. Maybe you can fix this by manually installing @prisma/client with pn i @prisma/client in your project and things will just work after.

Hey @chengpengjava, I was not able to reproduce this with the steps you provided. According to the error<EMAIL_ADDRESS>was not found, although it is published. So maybe it's a caching issue. Could you try it again from the start (as you show in your reproduction steps)? If still unsuccessful, could you try npm cache clean --force?

I went through the process again but same error even if I executed npm cache clean --force.But I succeeded after run pn i @prisma/client because I did use npm mirror through @janpio reminder,thank you all