[Compatibility] Issue with rq==2.0.0 in Python 3.10

Hi,
the current version of DeepCAVE does not support v2.0.0 of the rq package which is installed by default when following the instructions. It results in crashing background workers:
Traceback (most recent call last):
  File "/Users/jannisbecktepe/anaconda3/envs/deep_cave/lib/python3.10/site-packages/deepcave/worker.py", line 21, in <module>
    from rq import Connection, Worker
ImportError: cannot import name 'Connection' from 'rq' (/Users/jannisbecktepe/anaconda3/envs/autonnunet/lib/python3.10/site-packages/rq/__init__.py)

Hi Jannis,
You can simply downgrade rq to 1.10.1.
But yes this needs to be fixed :)

I will close this issue, since we merged a fix into main


Using raw dicts instead of the Document class?

Hello!
I've recently switched over to this library. However, I've noticed it uses a new Document class instead of raw Json. This change is actually kind of cool, but I have a very large codebase and it would take a very long time to change everything over to this new system. Is there a way to use raw dicts like in the original tinydb?
I've tried setting document_class to dict, however this raises an exception when inserting new entries:
AttributeError: 'dict' object has no attribute 'doc_id'
Thanks!

Hi dekrypted, thank you for liking this repo😁.
I have tried my best to maintain compatibility with the original TinyDB, you can insert a raw dict directly and it will be converted to your document class.
The modification you made is why this exception is raised, a document class requires this doc_id attribute, normally you wouldn't want to modify it.

Thank you for the help! For anyone who stumbles on this issue later, here's my resolution:
from asynctinydb import BaseDocument
from typing import Dict, Any

class DictDocument(dict, BaseDocument):
    def __init__(self, value: Dict[str, Any], doc_id: Any):
        super().__init__(value)
        self._doc_id = doc_id

    @property
    def doc_id(self) -> Any:
        return self._doc_id

    @doc_id.setter
    def doc_id(self, value: Any):
        self._doc_id = value

And in any tables:
db.table("<table>", document_class=DictDocument)

Glad you solved your issue. However, here is the definition of the default document class, and I don't see much of a difference. Could you explain the problem you encountered in detail?
class Document(dict, BaseDocument[IDVar]):
    """
    A document stored in the database.

    This class provides a way to access both a document's content and
    its ID using ``doc.doc_id``.
    """

    def __init__(self, value: Mapping, doc_id: IDVar):
        super().__init__(value)
        self.doc_id = doc_id

    @property
    def doc_id(self) -> IDVar:
        return self._doc_id

    @doc_id.setter
    def doc_id(self, value: IDVar):
        self._doc_id = value

    def __repr__(self):
        pp = json.dumps(self, indent=2, ensure_ascii=False, default=str)
        return f"Document(\n  doc_id={self.doc_id} \n  doc={pp})"

I'm actually not sure what the difference really is, but for some reason trying to initialize a class on the default base document as a dict wasn't working. However, after playing around the class I've sent above worked. I think this is just a case of Python being Python 🤷♂


How to use

Hi i have a problem using this variable:
filter_continuous_col:
I give it a date column for example but how does it define the min and max?
Thx a lot!
Fons

Hi @fux002 - the min/max values to query from your database are calculated from the min/max datetime values in your datagrams.
For example if my filter_continuous_column value is my date and the colum contains safeties from 2018-09-01 to 2018-09-12, then the function here will check the database for duplicates in that date range.


Add matcher for Dates to ignore timezone #891

We would need an extension function version too.

@sksamuel It is an extension function on ZonedDateTime. Am I missing something?

Actually, never mind, that is perfectly fine :)
On Tue, 16 Jul 2019 at 16:30, Tiberiu Tofan<EMAIL_ADDRESS>wrote:

@sksamuel https://github.com/sksamuel It is an extension function on
ZonedDateTime. Am I missing something?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kotlintest/kotlintest/pull/896?email_source=notifications&email_token=AAFVSGSLHP4AWH2ULCVESTLP7Y4YTA5CNFSM4IEIEELKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2CG5JQ#issuecomment-511995558,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFVSGSHNRRNWMK5MWJVKB3P7Y4YTANCNFSM4IEIEELA
.


Print logs on top and bar at bottom

How to  print log when processing.
https://github.com/cheggaaa/pb/issues/110
https://github.com/gosuri/uiprogress/issues/42

I need to update the README for this project. It was a learning project and in unsupported going forward. If it were to be re-done, the API would be vastly different.

@sethgrid I'm considering writing a new progress bar / spinner library, would love to hear what you'd do differently.


VIRTS-3035

VIRTS-3035: Pathfinder UI/UX Improvements (Part I)
Affects:

app/parsers/nmap.py
app/pathfinder_gui.py
app/pathfinder_svc.py
scanners/nmap/scanner.py
static/js/pathfinder.js
templates/pathfinder.html
Change: New functionality to

Set report name to file name when loading from disk,
Add a custom name to nmap scans,
Remove reports from memory,
Rename reports post-load or post-scan.

Description
This branch adds new functionality to the current Pathfinder plugin, building off of the Pathfinder API and the new core CALDERA API v2.  Changes include adding a new view tab called "Reports" to perform a few of the missing CRUD operations, including renaming reports in memory and removing reports from memory.  The "Import Scan" functionality now captures the file name of the .xml or .yml report file, and uses this file name as the report name.  Finally, the default nmap scanner was expanded to include a new custom name input field, to de-clutter the report "view" screen after executing a series of reports over a live network.
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

Feel free to pull the branch and let me know what you think!
Areas likely pending update:

currently the new "Reports" tab displays both the console and graph view.   Pending major changes, I am considering adding more descriptive logged status messages for the new PATCH and DELETE requests, and removing the Graph from the view screen.  I found the graph helpful in differentiating between similarly-named reports.
I've elected to hold off on adding dropdown help tooltips until we migrate to the new Alpine.js and Bulma framework, but can work to make this change in the current templating scheme if needed.
There is some duplicity in being able to download reports from either the "View" or the "Reports" pane.  I've elected to keep the feature in both for now, as it makes sense to me as a user to have access to the feature either when investigating the attack graph, or in the CRUD-style menu for downloading and then removing, or renaming and then downloading.

As always, I am more than happy to make changes as needed.  Thank you!

I really like that change, I'll make the edit shortly.

This will close #40


feat: Expand the types supported by tsconfigPath

Summary
Related Issue

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

Thanks for your contribution, can you help update both cn and en doc about the expanded type?

Thanks for your contribution, can you help update both cn and en doc about the expanded type?

Did you mean to update api-docgen.mdx?

yes, just update the type and explain what it means or how to use it.
我理解更新完文档里的类型以后，加一句类似的可以针对每个 entry 分别设置这么个意思就行了

yes, just update the type and explain what it means or how to use it.
我理解更新完文档里的类型以后，加一句类似的可以针对每个 entry 分别设置这么个意思就行了

get


Nit: Should code sample say C++ instead of CPP on MUXC.Direct page ?

Looking at other code sample, C++ code samples are referenced by C++, not CPP. Should the page linked in the document details also use C++ instead of CPP?

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

ID: 353ce7c7-3ad6-f865-37b6-4220d98f33f2
Version Independent ID: cb986179-4d50-14af-14e6-cfe4d67bb592
Content: Microsoft.UI.Xaml.Core.Direct Namespace - Windows UWP applications
Content Source: winui-api-build/xml/ns-Microsoft.UI.Xaml.Core.Direct.xml
Product: uwp
Technology: winui
GitHub Login: @Karl-Bridge-Microsoft
Microsoft Alias: kbridge

Topic revised.


Add a11y to pages and components

Description
Make pages and components more accessible based on the principles of web accessibility as found
https://www.a11yproject.com/
https://www.w3.org/WAI/fundamentals/accessibility-intro/
This would require adding a11y attributes to the components that can be picked up by browsers.
Motivation
No response
Change
No response
Additional Information

[ ] The Rucio API needs to be changed.

@ThePhisch Could you work on this while I am away :) Thanks

Will fix the problems in #450, but needs more prior work on AG grid and stuff


Update Jenkinsfile to use external library

Jenkins has got stuck, presumably because Laura no longer has permissions. I'll fixup this branch later.

Jenkins has got stuck, presumably because Laura no longer has permissions. I'll fixup this branch later.

I think that GitHub was just configured incorrectly. I've changed the webhook from https://ci.dev.publishing.service.gov.uk/github-webhook/, as that differs from other repositories I checked.
Anyway, I manually triggered a scan for this repository to get it to build, and I've added a commit to make Bundler/Rubygems not fail.


Openapi frontend

please add the link to the typescript-axio-generator-documentation to docs/usefullinks:md


Random time-outs with PLC

Hi, I have a long running issue where the AdsSession is randomly is experiencing an time-out with the PLC.
For customers the time-out can mean product loss as the PLC will enter a safe state if the communication times-out.
I'm still not sure on the cause
In the attached log file the issue occurs 2 times starting at timestamp:

09:19:12.040
09:21:41.370

log20241015.log
Are they any common causes for this issue?

A Timeout is exposed on the Client side. What means that the remote device didn't react in the expected time (timeout). This can happen if the target device has a problem (e.g crashed or too high system load), or if the network is unstable.
The communication timeout itself is neither triggering a different state on the PLC nor activates a watchdog. In the most cases the timeout is the symptom not the source.

Hi Ralph,
Thank you for the valuable feedback.
Is there a way to monitor the cycle exceeds and CPU load via ADS?

You could use the Statistics Property of the AdsSession object to get an idea about the number of Errors.
For further analysis I see only the possibilty for Network dumps. Ads Traffic can be dumped with out AdsMonitor Application or with Wireshare - but then there is the Problem to catch the moment where the Problem occurs.
By the way, have you tried to expand the AdsClient.Timeout time? Does this change the situation?

Hi Ralf,
Yes I know about the statistic property.
I think I found the cause of my issues. And they were indeed performance related issues on the PLC.
Apparently some functions I build hit very heavy on the CX9020, which are most likely related to a memory bottleneck on the system or a ARM limitation. As compared to a CX5110, the same function executes between 4 to 9 times faster on the CX5110 compared to the CX9020. While they both have the same TC performance level 30.
But the CX5110 has a x86 CPU and utilizes the TwinCAT RT x86 compared to TwinCAT CE (ARMV7) of the CX9020.
But I face a lineair perfomance hit when accessing a array of pointers on the CX9020. The more entries it iterates over how bigger the hit. And it builds quite fast where having 20 items in the array active it could take up to 0.5 milliseconds or 5% on the 10ms PLC cycle task. And if you have multiple instances the load builds up quite fast.

That explains why the AdsClient runs into timeout.
Unfortunately am not very deep in the realtime behaviour on different platforms and I cannot answer why the ARM is slower.
One option could be to contact our support - we have experts there for this topic.
Another idea because you asked for cycle exceeds:
Probably the following product
https://www.beckhoff.com/en-en/products/automation/twincat/texxxx-twincat-3-engineering/te1010.html
https://infosys.beckhoff.com/content/1033/te1010_tc3_realtime_monitor/index.html?id=4418206447557636659
could help you identifying/optimizing the problem. I am not aware about the costs, neither have I personally used it.

Thanks for the tips.
I also thought about the new PLC profiler (TE1210) could help me out. But that's only available on 4026 and I'm still on 4024 for the time being.
Another alteration I made, is I followed the best practice for the CX7000 series. Where I slowed down the IO idle task from 1ms to 10ms and changed the priority order that IO Idle task has the lowest priority.
Only I haven't found anything what the impact is of lowering the IO Idle task time other than it frees up the CPU a bit.


Keyword missing 'intents'

Summary
Needs intents, but when intents added, improper token.
Reproduction Steps
code????
Code
import discord
from discord.ext import commands
import random
import traceback
import asyncio
import json
import os

with open("token.txt", "r") as file:
    token = file.read().strip()

def load_prefix():
    if os.path.exists('PREFIX.txt'):
        with open('PREFIX.txt', 'r') as file:
            return file.read().strip()
    return "!"

PREFIX = load_prefix()

bot = commands.Bot(command_prefix=PREFIX, self_bot=True)

Expected Results
Logged in as user!
Actual Results
Exception has occurred: TypeError
BotBase.init() missing 1 required keyword-only argument: 'intents'
File "C:\Users\esorlexs\Desktop\folders\code\python\selfbots\general\main.py", line 21, in
bot = commands.Bot(command_prefix=PREFIX, self_bot=True)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: BotBase.init() missing 1 required keyword-only argument: 'intents'
System Information
C:\Users\esorlexs>python -m discord -v

Python v3.11.5-final
discord.py v2.4.0-final
aiohttp v3.10.9
system info: Windows 10 10.0.22631

C:\Users\esorlexs>
Checklist

[X] I have confirmed I am using unmodified discord.py-self and not the upstream discord.py.
[X] I have searched the open issues for duplicates.
[X] I have shared the entire traceback.
[X] I am using a user token (and it isn't visible in the code).

Additional Information
I added the intents, and it still did not work, it said it was a invalid token.

I have fixxed it by pip uninstalling discord.py, and discord.py-self, and only reinstalling discord.py-self.


[shell] Can't upload to the S3 cloud mount, multiple errors on upload

Describe the bug
After the closing of the bug #4175, I updated my installation to restart the filter.remote.sync command.
When using the command weed filer.remote.sync -dir=/media -filer localhost:8889 the filer seems to start synchronisation, but after a few seconds, a lot of transport error appear, and nothing has been written to the remote storage.
System Setup
The system consists of 2 VM:
1 weed master on the server1
2 weed volume on server1 and server 2
2 filer on server 1 and 2
    The filers use leveldb2
    the filers are replicated
Use of the Cloud drive functionality on /media, to synchronise files to an S3 storage
    The remote storage is a S3 store by OVH

The 2 servers run on Ubuntu 20.04, with the same patch level, updated 1 week ago
weed version: version 30GB 3.42 8821d6b1619b7a55b515a98391193297e77cbe52 linux amd64
The filer is configured from the command line without a file:
weed filer -port 8889 -ip.bind <IP_ADDRESS> -ip X.X.X.X -s3  -master X.X.X.X:9333 -dataCenter=XXXX -rack=XXX -metricsPort=9328
Expected behavior
When using weed filer.remote.sync -dir=/media -filer localhost:8889 the filer is expected to write back the files stored in the filer's dir to the remote storage.
Additional context
I was running the sync process continuously in a tmux term. Had to cut it for two days, when I restarted it, it started to have problem.
I tried to run remote.meta.sync before re-running the command, but this didn't change the outcome.
When using rclone to list the created file on the remote storage, the files don't appear, but the directory to host the file is created. Using rclone to copy the file manually to the destination is successful, and the file can be read back.
This is the output:
I0206 12:01:16.748098 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/f7ba/7a16/CI_1-KBIVC9_20230131091836485.0dcd.jpg
I0206 12:01:16.788175 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/67f7/c306/photoLIL23015716_001_1.0f94.jpg
I0206 12:01:16.788417 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/f59b/3d23/CI_1-KKJ3PU_20230131104737592.c26a.jpg
I0206 12:01:16.802860 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/d5af/a577/CI_1-KKIWCN_20230131143327110.c528.jpg
I0206 12:01:16.836960 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/ebeb/3e20/photoLIL22053406_001_3.f769.jpg
I0206 12:01:16.855845 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/5f82/ec9b/CI_1-KL1ZL2_20230201100140538.52e6.jpg
^CI0206 12:01:16.859267 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/3c76/5140/photoCAM23005511_001_1.39a0.jpg
I0206 12:01:16.859670 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/0ff2/87c3/CI_1-KKIWCN_20230131143344915.5233.jpg
I0206 12:01:16.860099 retry.go:25 retry writeFile: err: upload to ovhMedia/media-prod/ged/b57d/6447/photoBCZ23016369_001_1.543c.jpg: RequestError: send request failed
caused by: Put "$OVH_CLOUD/b57d/6447/photoBCZ23016369_001_1.543c.jpg": net/http: HTTP/1.x transport connection broken: http: ContentLength=20809 with Body length 0
I0206 12:01:16.860335 retry.go:25 retry writeFile: err: upload to ovhMedia/media-prod/ged/fef8/3530/CI_1-KKUA2C_20230201135816116.af45.jpg: RequestError: send request failed
caused by: Put "$OVH_CLOUD/fef8/3530/CI_1-KKUA2C_20230201135816116.af45.jpg": net/http: HTTP/1.x transport connection broken: http: ContentLength=64284 with Body length 0
I0206 12:01:16.862122 retry.go:25 retry writeFile: err: upload to ovhMedia/media-prod/ged/7c5d/a902/photoCAM22074219_001_3.bbf7.jpg: RequestError: send request failed
caused by: Put "$OVH_CLOUD/7c5d/a902/photoCAM22074219_001_3.bbf7.jpg": net/http: HTTP/1.x transport connection broken: http: ContentLength=28119 with Body length 0
I0206 12:01:16.862994 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/94fe/2584/photoDIU21005900_001_1.71e8.jpg

Thanks for this great piece of software.

After a bit more testing, I discovered I had problems with rclone as well. This seems to happen when rclone upload multiple files in batch.
The S3 provider is OVH.
ex:
2023-02-10 00:30:24 ERROR : 00e7/6d09/001.1aea: Failed to copy: s3 upload: 400 Bad Request: <?xml version='1.0' encoding='UTF-8'?>
<Error>XAmzContentSHA256Mismatch<Message>The provided 'x-amz-content-sha256' header does not match what was computed.</Message><RequestId>tx07c6751b87c4472190780-0063e58210</RequestId><ClientComputedContentSha256></ClientComputedContentSha256><S3ComputedContentSha256><html><h1>Unprocessable Entity</h1><p>Unable to process the contained instructions</p></html></S3ComputedContentSha256></Error>

should be related to this https://github.com/seaweedfs/seaweedfs/blob/master/weed/remote_storage/s3/wasabi.go#L58
debugging this is tricky though.

try to turn on this value s3.v4_signature when configuring the remote storage. not so sure.

Sorry, I forgot to add in the debug, I've toggled this flag, but the result is the same.

added a fix https://github.com/seaweedfs/seaweedfs/commit/545d5d0cc39956b5bec0df3ee62e57af4d9c22c6 for a similar issue. Please try it.

Thx for the fix.
After testing it (compiled the weed binary on my machine, then upload, and run the sync from the new binary, with latest version running for filer/volume/master), the error messages have disappeared, but it doesn't seems to upload.
E.g.
# ~/weed filer.remote.sync -dir=/media -filer localhost:8889 2>&1 |grep media-prod/ged/a3f6/cdb0/photoMTU23001360_001_10.9b27.jpg
I0220 06:40:56.771449 filer_remote_sync_dir.go:171 delete ovhMedia/media-prod/ged/a3f6/cdb0/photoMTU23001360_001_10.9b27.jpg
I0220 06:40:56.874020 filer_remote_sync_dir.go:193 create ovhMedia/media-prod/ged/a3f6/cdb0/photoMTU23001360_001_10.9b27.jpg
# rclone lsd  ovh-s3:media-prod/ged/a3f6/cdb0/
# empty result

Tried multiple times, it tries to copy it again.
With rclone lsd, i've found the media-prod/ged/a3f6 directory is created with inside, directory are created today, but not cdb0.

Thanks for the update!


Add the "Framework :: AWS CDK :: 1" classifier

Request to add a new Trove classifier.
The name of the classifier(s) you would like to add:

Framework :: AWS CDK
Framework :: AWS CDK :: 1

Why do you want to add this classifier?

User experience study participants for the AWS CDK mentioned they would like to
be able to identify AWS CDK (Cloud Development Kit) packages in PyPI using a
standard trove classifier, which sounds like a good idea.
Since we are going to release a v2 "soon" (aws/aws-cdk-rfcs#156), I am already
requesting the :: 1 suffix. I'll file a separate request to add a :: 2
suffix once CDK v2.0 is ready for initial release (unless you'd rather I add it
right away).
More info on the project: https://github.com/aws/aws-cdk

Hi, thanks for the PR. Can you provide some links to existing packages that would ideally use this classifier?

There'd already be the 100+ packages that compose the AWS CDK itself (they all have names starting in aws-cdk.):

https://pypi.org/project/aws-cdk.core/
https://pypi.org/project/aws-cdk.aws-lambda/
https://pypi.org/project/aws-cdk.aws-ec2/
...

Then there are also community packages (which we are trying to stimulate), based off a cursory search through PyPI:

https://pypi.org/project/cdk-chalice/
https://pypi.org/project/aws-cdk-dynamodb-seeder/
https://pypi.org/project/aws-cdk-featureflags/
https://pypi.org/project/aws-solutions-konstruk.aws-dynamodb-stream-lambda-elasticsearch-kibana/
https://pypi.org/project/cdk-lambda-powertuner/
https://pypi.org/project/cdk-fargate-express/
...

Thanks! (and thanks for the docs fix as well!)


refactor onoff to separate async notification and standardize terminology

This PR separates generic resource management updates that appear to be nearly stable from #23229.  The changes are:

refactors the onoff service to reduce space by putting static configuration data into a separate structure;
extracts the async notification infrastructure from the on-off service into a self-contained structure that can be used anywhere asynchronous operations may be used;
recasts the on-off service as a manager that is to be used by services (including API changes to make it more consistent with other managers to come);

To proactively address some comments from earlier reviews for which I have not made changes:

Yes, struct async_notify records the result of the operation in a common field rather than in the union wjere it would be available only for spin-wait notifications.  An argument can be made that this is unnecessary, since for signal the result is stored in struct k_poll_signal and for callback it's passed as an argument, thus only spin-wait needs it.  Reasons for retaining the existing approach include:

Consistency: the result of the operation can always be obtained using async_notify_fetch_result() regardless of the notification method used;
Flexibility: An application may multiplex notifications onto one k_poll_signal object (e.g. fire off a dozen and wake up when the first one completes; tricky but not inconceivable), or may not process the operation completion in the callback (in which case the result needs to be persisted somewhere, most likely on the client data that contains the notification result).  If the result is only retained for spinwait notifications the applications have to do extra work.

Yes, the names are long.  They're also unambiguous and unlikely to conflict with future API.  If there is wide consensus that a specific alternative prefix is preferred I'll change them.  (anotify_ instead of async_notify_ has been proposed, but "async" means "not sync(hronous)" so "anotify" means "not notified"?  Meh.)

Could you fix 80liners checkpatch complains?

I think I got all but the ones where there's no obvious place to break a line.

@tbursztyka @anangl if any of you could review that? @carlescufi maybe you know someone that would have interest in reviewing that?

@tbursztyka @anangl if any of you could review that? @carlescufi maybe you know someone that would have interest in reviewing that?

No, unfortunately beyond @tbursztyka and @anangl I can't think of anyone else at this point. Perhaps @wentongwu or @pdunaj?

besides naming issues, lgtm

@carlescufi @tbursztyka naming has been resolved can you look again?

besides naming issues, lgtm

Will merge since the naming has been resolved


fix: parse parent callees when using attributes

As mentioned by @stof here: https://github.com/Behat/Behat/issues/1565#issuecomment-2535430466, AttributeContextReader was incomplete compared to its annotation equivalent.
We add here the missing code to retrieve parent callees, like annotations do.
Issue #1565

@carlos-granados @acoulton when you have some time to check this one 😄

Great, thanks

You're welcome! 😄
And sorry for the confusion with the title of PR. My English is not always good and understandable ahah!


feat: flutter 3.19

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)

NOTE: This PR needs to go in after the other 5 PRs are in as I did the upgrade changes on top of them to make sure everything is working fine

Alright, thats android done. Next up iOS 🤜 🤛

Et voila, we're back.


Added Column Defination Priority for CRDs

Signed-off-by: Vatsal Parekh<EMAIL_ADDRESS>
What this PR does / why we need it:
Added Priority to the CRD Column Definition to show -o wide output
➜  kubevirt git:(wide-output-for-vm) ./cluster-up/kubectl.sh get vm
NAME        AGE   RUNNING   VOLUME
vm-cirros   40s   true
➜  kubevirt git:(wide-output-for-vm) ./cluster-up/kubectl.sh get vm -o wide
NAME        AGE   RUNNING   VOLUME   CREATED
vm-cirros   42s   true               true

➜  kubevirt git:(wide-output-for-vm) ✗ kg vmi
NAME                    AGE   PHASE     IP            NODENAME
vm-cirros               1m    Running   <IP_ADDRESS>   node01
vmi-alpine-efi          50s   Running   <IP_ADDRESS>   node01
vmi-fedora              44s   Running   <IP_ADDRESS>   node01
vmi-flavor-small        39s   Running   <IP_ADDRESS>   node01
➜  kubevirt git:(wide-output-for-vm) ✗ kg vmi -o wide
NAME                    AGE   PHASE     IP            NODENAME   LIVEMIGRATABLE
vm-cirros               1m    Running   <IP_ADDRESS>   node01     False
vmi-alpine-efi          57s   Running   <IP_ADDRESS>   node01     False
vmi-fedora              51s   Running   <IP_ADDRESS>   node01     False
vmi-flavor-small        46s   Running   <IP_ADDRESS>   node01     False

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Added Priority to the CRD Column Definition to show `-o wide` output

Actually nice, I wander what would be other useful fields there for our users.

@petrkotas I'm adding one more here
/hold

@petrkotas good to review now
/hold cancel

@petrkotas done

/retest

@rmohr done

/lgtm
/approve

/retest

/retest

/retest


Add more links

Hello!
Can you help to create an extensive list of awesome accessibility tools?
Feel free to send a PR!

hey is this a valid link? https://github.com/vue-a11y/eslint-plugin-vuejs-accessibility
or should we avoid more specific tools like this?

@vhoyer Of course! anything that helps create an accessible internet


Add descriptive stat scripts, a test and derive main descriptive stats

You pushed two full pickled data. Could you remove one of them?

Let me know when you adress those.

Can we create all the descriptive stats we thought it would be useful to have, and close this PR? Let's make everything functional, so that when we choose another website we can use the same code to get the descriptive stats for that dataset.

are we not going to merge this?


Street to postcode matching for matching places to postcode more effectively

all streets that can be matched are matched now


Docker bookstack installation is not loading properly and causing no css issues

Attempted Debugging

[X] I have read the debugging page

Searched GitHub Issues

[X] I have searched GitHub for the issue.

Describe the Scenario
I followed all the steps from the recommended docker-compose installation of linuxserver docker-bookstack github .
I used the docker-compose file there which is
version: "2"
services:
  bookstack:
    image: lscr.io/linuxserver/bookstack
    container_name: bookstack
    environment:
      - PUID=1000
      - PGID=1000
      - APP_URL=docs.something.com #my own app_url added
      - DB_HOST=bookstack_db
      - DB_USER=bookstack
      - DB_PASS=<yourdbpass> #my own password added
      - DB_DATABASE=bookstackapp
    volumes:
      - /path/to/data:/config #my own path added
    ports:
      - 6875:80
    restart: unless-stopped
    depends_on:
      - bookstack_db
  bookstack_db:
    image: lscr.io/linuxserver/mariadb
    container_name: bookstack_db
    environment:
      - PUID=1000
      - PGID=1000
      - MYSQL_ROOT_PASSWORD=<yourdbpass> #my own password added
      - TZ=Europe/London#my own timezone
      - MYSQL_DATABASE=bookstackapp
      - MYSQL_USER=bookstack
      - MYSQL_PASSWORD=<yourdbpass> #my own password added
    volumes:
      - /path/to/data:/config #my own path added
    restart: unless-stopped

After running docker-compose up -d, everything went well but when I connect to APP_URL (docs.something.com), I get the 'cannot find this page' error. I haven't used any reversed proxy. I'm also not sure where to pin point the errors since I'm not very familiar with php and apache stuffs. It is a completely clean install. I haven't done anything to the containers yet.
I've also tried adding http:// and https:// in front of my app_url environmental variable but they didn't work.
Another weird thing is that the url changed into http://X.X.X.X:6875/docs.something.com/docs.something.com/login when I just used X.X.X.X:6875 instead of app_url. https://X.X.X.X:6875 doesn't work. Only http:// does.
This weird long url works and the page shows but there is no css in the page. I've found some closed issues here similar to that no css loading problem but they are on host installations and seemed to be caused by something entirely different from mine.
Can you help me?
Exact BookStack Version
v22.10.2-ls50
Log Content
No response
PHP Version
8.0.25
Hosting Environment
installed using official installation script(no proxies used for docker containers)

Hi @9ieR,
APP_URL must be the exact base URL used to access your BookStack instance in the browser, so http://X.X.X.X:6875 unless you're using a domain/host name when accessing in browser. Also ensure you are re-upping the compose stack, rather than just restarting, to ensure env changes are used.

Hi @ssddanbrown
Yep, I've done that. When I up the yaml compose file, I made sure that I deleted the existing containers, images, volumes and even networks related to the compose file.
I've also just tried using IP address in place of URL. Still no luck though. The url just changed into
http://X.X.X.X:6875/X.X.X.X:6875/X.X.X.X:6875/login. And css still missing.

I've also just tried using IP address in place of URL. Still no luck though.

Just to confirm, does the current value definitely start with http:// or https://. The only times I've seen the IP/host duplication, as you describe, is specifically when the APP_URL does not start with the protocol.

http://
https:// doesn't work. It causes the page to not load up at all. I'm also wondering why it's working with https but with http. Seems like I can't also call apache's default index.html of my host with https but only with http.

I would not expect https:// to work unless you've done specific setup to support it.
So you currently have APP_URL=http://X.X.X.X:6875 defined for the bookstack container? Where are you setting this? Via the environment of the docker-compose.yaml or via an .env file?

Yep! I changed it in the docker-compose.yaml file. I don't try to change environmental variables after the containers are up and running.
Whenever I want to test something different, I just delete everything about the current containers/images and go back making changes to compose file and up it again.
Currently, the environment variable APP_URL in the docker-compose.yml is APP_URL=http://X.X.X.X:6875

Okay, in that configuration, can you load BookStack (In its broken design state).
View the page source (Right click page > View source).
Find the lines towards the top (About 20 lines down) that look something like this:
<!-- Styles and Fonts -->
<link rel="stylesheet" href="URL_HERE">

Look at the href value in the second line of that part, and compare exactly to the URL you've used for APP_URL. Do these differ?

Oh hey! It worked. But I'm still not satisfied because it's still not working in the way I wanted it to. I also want to know what's the cause of it.
When I changed the APP_URL in the compose file to http://X.X.X.X:6875, it started working normally. But what if I just to use the domain name? Like docs.something.com. How can I make it so?

@9ieR From the perspective of the BookStack, and the linuxserver.io image, you just need to set your APP_URL to be http://docs.something.com, but networking would need to be in place so requests to the domain on port 80 are routed to the container as expected.
It it helps, I created a full end-to-end video for using the linuxserver image and setting it up on a domain with HTTPS:
https://www.youtube.com/watch?v=dbDzPIv8Cf8

@ssddanbrown
Thanks a ton. I'm checking it out.
By the way, am I supposed to close this issue now? This is my first time creating an issue on github. I'll just open a new one if something isn't working. I can right?

Yeah, If there's nothing further to discuss here this can be closed off, I'll go ahead and do this.
If preferred, there's also a help/support channel in our discord server. Can be a little easier to request support there and often more people active there to help out.


figure 8 issue

From @munkm

I noticed also that the figure of Lifecycle Carbon Emissions (Figure 8 in the PDF) says that it is plotting all scenarios, but doesn't include the XNn scenarios in the legend. Either the figure label or the figure should be updated to either remedy the omission or clarify why it isn't included in the figure.

This issue can be closed with a PR that handles this concern.

closed by #50


Merge Dev ke UAT 20 June 2017

Coverage increased (+0.006%) to 65.288% when pulling 6461ef682e112afec2d5a537757609e344b4ba4c on dev into 192d5daa17d8b41543655ed60733b7fa10ba5364 on uat.


Test orphanDependents=true as well

@marun i think it's still better to test orphan case in e2e rather than unit tests because it can collide with garbage collector. how do you think?

@yamt What makes you think it would collide with the garbage collector?

@yamt What makes you think it would collide with the garbage collector?

The use of the same finalizer, FinalizerOrphanDependents.
The following is a copy-and-paste from slack.
yamamoto   [8 days ago]
doesn't our controller race with the gc removing "orphan" finalizer?
yamamoto   [8 days ago]
when our controller runs first, i guess we should not remove the "orphan" finalizer. otherwise the gc might be confused.
yamamoto   [8 days ago]
when the gc runs first, it removes the "orphan" finalizer and our controller misses it.

@yamt I realize you are correct that a race condition exists (see #622).  I think it would be preferable to fix that issue (the race condition).  Would you be interested in tackling it?

@yamt I missed the import of our conversation on slack.  There is no harm in the federation controller removing the orphan finalizer before the gc sees it, because there shouldn't be any dependent resources of a federated resource that the gc needs to remove owner references for.  The reverse is not true.  As per #622, if the gc removes the finalizer before the sync controller sees it, orphaning will not take place.

There is no harm in the federation controller removing the orphan finalizer before the gc sees it, because there shouldn't be any dependent resources of a federated resource that the gc needs to remove owner references for.

i guess nothing prevents another controller unknown to us from creating dependent resources. i agree the opposite case is the bigger concern though.

Closing in favor of #669


doc: Group and Add E2E details under End-to-End Tests section.

Closes #943

/ready-to-test

/ok-to-merge


[Bug] Bottom part of gear table does not translate to English from Swedish

Describe the bug
When I tab into the Gear Tables with the language set to English, the bottom table list (labelled Vanliga tjänster) [that I think is the list of workers available] remains in Swedish. I cannot make this change into English.
To Reproduce
Steps to reproduce the behavior:

Go to '...' Gear Tables
Click on '....' Gears Table -> English
Scroll down to '....' Vanliga tjänster
See error: Remains in Swedish

Expected behavior
I would hope the bottom table would translate into English.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Intel Core i5 with Windows Home 10
Browser: Chrome

Additional context
Add any other context about the problem here.

@Helmaer Thank you, the plan is to add the rest of the gear to the page and provide ways to filter them. Also being able to add stuff to a shopping cart and more. Tracked by #340


Rewrite description for deleting issues

Why:
Closes #18187

What's being changed:

Out of scope
Note that afaict it isn't possible to see who deleted an issue. I tried w/ a personal repository, I tried w/ a free organization, and I tried w/ a team organization using a private repository. None of them told me who deleted the issue (but as long as I could see the repository, they'd at least be willing to tell me the issue was deleted if I used the api).
Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@jsoref Thanks so much for opening a PR! I'll get this triaged for review ⚡


typo fixed "Related Event to Events"

typo fixed "Related Event to Events"

Thanks @vellankikoti ! This is good!


[SPARK-23084][PYTHON]Add unboundedPreceding(), unboundedFollowing() and currentRow() to PySpark

What changes were proposed in this pull request?
Added unboundedPreceding(), unboundedFollowing() and currentRow() to PySpark, also updated the rangeBetween API
How was this patch tested?
did unit test on my local. Please let me know if I need to add unit test in tests.py

Test build #86897 has started for PR 20400 at commit 45545d6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/450/
Test PASSed.

Test build #86897 has finished for PR 20400 at commit 45545d6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86897/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/468/
Test PASSed.

Test build #86919 has started for PR 20400 at commit 25fee39.

Test build #86919 has finished for PR 20400 at commit 25fee39.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86919/
Test PASSed.

Merged build finished. Test PASSed.

Yup, we could also string as a column but I was thinking of matching the
signature with the Scala one for now, just for consistency ..
On 1 Feb 2018 5:24 pm, "Liang-Chi Hsieh"<EMAIL_ADDRESS>wrote:
@viirya commented on this pull request.
In python/pyspark/sql/window.py
https://github.com/apache/spark/pull/20400#discussion_r165284238:

     """

   if start <= Window._PRECEDING_THRESHOLD:

       start = Window.unboundedPreceding

   if end >= Window._FOLLOWING_THRESHOLD:

       end = Window.unboundedFollowing

   if isinstance(start, (int, long)) and isinstance(end, (int, long)):

Is it possibly that we mix int and Column in the parameters?
In python/pyspark/sql/window.py
https://github.com/apache/spark/pull/20400#discussion_r165284328:

                 any value greater than or equal to min(sys.maxsize,<PHONE_NUMBER>854775807).

     """

   if start <= Window._PRECEDING_THRESHOLD:

       start = Window.unboundedPreceding

   if end >= Window._FOLLOWING_THRESHOLD:

       end = Window.unboundedFollowing

   if isinstance(start, (int, long)) and isinstance(end, (int, long)):

ditto.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/spark/pull/20400#pullrequestreview-93195310,
or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGLXhdPpCrSrpXKsNU3qTUy17dXMGLmfks5tQXTDgaJpZM4Rtjrt
.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/584/
Test PASSed.

Test build #87061 has started for PR 20400 at commit f82c7d1.

Test build #87061 has finished for PR 20400 at commit f82c7d1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87061/
Test PASSed.

retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/791/
Test PASSed.

Merged build finished. Test PASSed.

Test build #87307 has started for PR 20400 at commit f82c7d1.

Merged to master.


papermc: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes
Bumped version and refactored to use runCommandLocal and makeBinaryWrapper

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

refactor and version bump should probably be separate commits

done

refactored to use runCommandLocal and makeBinaryWrapper

The refactor doesn't make sense to me. Why did you think a package, not a trivial command, should be written with runCommandLocal instead?

is iestall + makeBinaryWrapper nontrivial?

I'd say look for stdenvNoCC.mkDerivation.
There are of course non-standard ways to make something buildable, for example, passing postFetch and meta into fetchzip. I just don't like to make exceptions for these patterns that are extremely rare in nixpkgs.

runCommandLocal maps too stdenvNoCC.mkDerivation with a few flags so that's just un-abstracting a builder
But I get what you're saying I'll push changes tonight

Result of nixpkgs-review pr 256981 run on aarch64-darwin 1

1 package built:

papermc


slabs: add stats for exact bytes accounting

add quick summary at bottom of stats slabs output which says exactly how many bytes are available for use and how many are used. when bytes_available reaches zero the system must evict.
however the system may evict much sooner than when bytes_availble reaches zero. you must also account for the system moving memory around slab classes.
combine with the output from totalling 'stats items' -> mem_requested to find the slab overhead.
TODO:
This doesn't tell us if we're about to evict, as that calculation is:

if any slab class has zero chunks available
the page allocs are at the memory limit
and slab_global_page_pool is 0
the next allocation will evict.

don't think I'm going to merge this but I'll leave it here as food for thought:

some kind of total allocation counters (include various buffers, hash table, etc)
algorithmic summaries that are commonly used

not doing this right now.


Cleaned Code

aioclient.py

removed unused imports
cleaned code
removed some redundant logic

utils.py

changed logic
cleaned code

Generally cleaned code

I can't merge it, there is several errors, I can keep the ["key"] instead of .get("key") but the way you create session is not good, it will trigger a warning, also the place where your annotation returns Any is wrong, it return Dict. The docstring for class is useless I dont see the point on adding args there is already attribute. Finally maybe I can keep that default appaname func but i dont see the point of using "".join ...
Also there is no point of separating every line by an other. 2 line for class 1 line for func.
If you really want to dod a commit then i guess you can keep that default name func, using version of init and usinjg a f-string instead of join.
You can keep the ["key"] and please fix those space.


Add contentType to options

First, thanks for the great module! It was super easy to plug and play.
I ran into an issue of my image files always downloading instead of being able to be visible in the browser and traced it back to the Content-Type being defaulted to application/octet-stream when it ideally should be set to the file's mime type. I noticed there are already a couple forks of this project that have fixed this issue and don't want to pile on. Would you consider pulling in changes from either CheryCheryBoom's fork or calvium's branch to address the missing options and contentType settings?
For reference:
https://azure.github.io/azure-storage-node/BlobService.html#createWriteStreamToBlockBlob

Thanks for the info. We have merged calvium's branch, and published the updated version to npm as 0.1.3


Failed to stop mongod.service: Unit mongod.service not loaded.

Failed to start mongod.service: Unit mongod.service not found.
Traceback (most recent call last):
File "./src/jarvis/start.py", line 23, in
from jarvis import settings
File "/home/blackhat/Python-ai-assistant-master/src/jarvis/jarvis/init.py", line 46, in
configure_MongoDB(db, settings)
File "/home/blackhat/Python-ai-assistant-master/src/jarvis/jarvis/utils/startup.py", line 97, in configure_MongoDB
from jarvis.skills.registry import CONTROL_SKILLS, ENABLED_BASIC_SKILLS
File "/home/blackhat/Python-ai-assistant-master/src/jarvis/jarvis/skills/registry.py", line 38, in
from jarvis.skills.collection.math import MathSkills
File "/home/blackhat/Python-ai-assistant-master/src/jarvis/jarvis/skills/collection/math.py", line 23, in
from word2number import w2n
File "/home/blackhat/Python-ai-assistant-master/jarvis_virtualenv/lib/python3.7/site-packages/vext/gatekeeper/init.py", line 204, in load_module
raise ImportError("No module named %s" % modulename)
ImportError: No module named word2number
Failed to stop mongod.service: Unit mongod.service not loaded.

Hi @Kaustubh1919,
I add the missing dependency in the requirements.txt. You can pull the the fix and run the setup.sh again or enable the jarvis_virtualenv and install only the missing dependency (pip install word2number==1.1)


Fix/Category and Default Sort - Meetings Page Serialization [PLAT-657]

Purpose
Tweaks for meetings page serialization in SQL
Changes

Add default sort order on download count - highest downloads first.
Do a better job of figuring out what meeting category is.   Now looks in array of tag names on meeting to see if certain tag is present.

For example, meeting with tags ["SPSP2016", "poster", "science"] would be categorized as a poster.
In previous version, was searching through concatenated list of strings "SPSP2016 poster science" for the word "poster".  So a project tagged as "imposter" would show up as a poster. Silly me.

QA Notes

Documentation

Side Effects

Ticket
https://openscience.atlassian.net/browse/PLAT-657

These meetings tweaks are ready @sloria


Existing breakpoints not honored after debugger startup

The debugger sometimes does not halt at breakpoints that were set before the debugger started (reported e.g. in http://discuss.tlapl.us/msg04204.html).
Workaround
Add a new breakpoint or reset an existing breakpoint after startup when the debugger is halted in Init or the first ASSUME.

Fixed with https://github.com/lemmy/tlaplus/commit/7fee0c6e53694416134b0255021c2cd1fdcee19a


Create PoorPigs Problem 458

It was the daily problem on 29th oct
Problem:
There are buckets buckets of liquid, where exactly one of the buckets is poisonous. To figure out which one is poisonous, you feed some number of (poor) pigs the liquid to see whether they will die or not. Unfortunately, you only have minutesToTest minutes to determine which bucket is poisonous.
You can feed the pigs according to these steps:
Choose some live pigs to feed.
For each pig, choose which buckets to feed it. The pig will consume all the chosen buckets simultaneously and will take no time. Each pig can feed from any number of buckets, and each bucket can be fed from by any number of pigs.
Wait for minutesToDie minutes. You may not feed any other pigs during this time.
After minutesToDie minutes have passed, any pigs that have been fed the poisonous bucket will die, and all others will survive.
Repeat this process until you run out of time.
Given buckets, minutesToDie, and minutesToTest, return the minimum number of pigs needed to figure out which bucket is poisonous within the allotted time.
Description
Upon reviewing the problem seems to be related to determining the minimum number of pigs needed to find the poisoned bucket within a certain time frame, considering the constraints on the testing time for each pig. The code employs dynamic programming to solve the problem, using a combination of factorials and a 2D vector to keep track of intermediate results.
My initial thoughts on solving this problem would involve understanding the constraints and requirements thoroughly. This includes analyzing how the testing time for each pig affects the overall strategy, the concept of dynamic programming, and how to leverage it to optimize the solution.
I would also consider the relationship between the number of pigs, the testing time, and the total time available to solve the problem. This understanding is crucial to come up with an efficient algorithm that minimizes the number of pigs required while ensuring the poisoned bucket can be identified within the given time frame.
Additionally, exploring the problem through a few examples to understand its nuances and intricacies would be a crucial step. This approach can help in devising a comprehensive plan for solving the problem effectively and optimizing the code for efficiency and scalability.

HacktoberFest is Ending!!


Remove deprecated functionality from 'aad m365group user list'

We've created these specs because there was a confusion with the current implementation of aad m365group user list around member roles and user types. userType (Members VS Guests) is an AAD related user property, while Owners VS Members is the role the user has in the group.
The change described in this issue removes the deprecated Guest value of the role option.
Options

Option
Description

-r, --role [role]
Filter the results to only users with the given role. Allowed values: Owner or Member.

Remove value (Deprecated) Guest from the list of allowed options.
Default properties
Remove the following property:

userType

Additional Info
Originally posted by @martinlingstuyl in https://github.com/pnp/cli-microsoft365/issues/5557

Let's also specify that we should remove the code that overwrites the userType property.

Since we have v8 branch we may now start working on it!
Let's get this 🚢
and Let's put CLI to the 🚀🌙

Can I work on it?

Can I work on it?

All yours

merged to v9


Support Preact X

I started this before I realized #32 was in progress, but now my branch is a little ahead anyways.
Depends on https://github.com/developit/preact-jsx-chai/pull/69 for tests to pass.

Code looks decent.  Any thoughts on release/maintenance strategies given that most projects in the wild are going to be on Preact < X for a long time to come?

For preact-router they just bumped major, so I guess this would also work here?
https://github.com/preactjs/preact-router/releases/tag/3.0.0

By the way, I just released https://www.npmjs.com/package/preact-localization, which is heavily inspired of preact-i18n but is compatible with Preact X and uses createContext instead of the legacy API. It's less featured because I'm not sure if I can do nested providers with createContext yet, but I'll look into it. That's also the reason why it's not a Pull Request.
I'm not sure how all of this is supposed to work for licensing, do you have any idea @billneff79? A lot of code changed, but some of it is basically the same.

Code looks decent. Any thoughts on release/maintenance strategies given that most projects in the wild are going to be on Preact < X for a long time to come?

@billneff79 I think on NPM we can publish under the next tag for PreactX packages, and wait until the stable release to publish next as latest.

@filoozom re: preact-localization, if the only reason you published it was to support preact X, we'll make sure we find a way to do that in preact-i18n proper.  If you have another reason, then it is fine, but you need to update your license in the product github repo to be compliant with our current BSD-3 license (the copyright for Synacor needs to occur in occur in your repo - read the license file for more details).

@pl12133 Re release strategy, I like using a non-latest tag in npm.   I think that it is going to take a long time for Preact X to be mainstream, so I'm also hesitant to make hte master branch in github be Preact X by default.  What do you think about this as a generic strategy:

Have a preact-x branch in github that contains current preact X compatible code for the repo, and continue to use master branch as < Preact X compatibility
Publish preact-x branch to the preactx tag in npm.  I like that over next to disambiguate what it is for
We update the README to make it clear to users of the package which branch/npm tag to use for which version of preact they are using.  We would publish a patch to the existing tag on master with this readme change.
If/when Preact X gains widespread adoption, we merge the preact-x branch into master on GitHub and publish it to a new major version of the latest tag on npm.

I pushed the changes, along with removing the requirement of preact-jsx-chai so tests would pass, to the preactX branch in this repo.
I have published version 2.0.0-preactx.2 under the preactx tag in npm.  I also updated the readme with info on preact version support
This should be all set now.  Any future changes to support preact X should be done on the preactX branch until it becomes widely adopted.


Fix mkdocs html redirection, setting use_directory_urls

Set use_directory_urls to false, to allow rest api html redirection
@navalev Can you push to your fork so that we'll validate it first there?

@SharonHart  my fork is updated as of 2 hours ago. do u want me to use this branch?

@navalev yes please

@SharonHart done

what should be the difference?

@navalev the redoc swagger doesn't work in all cases

still doesnt seem to work in the fork

@navalev Thanks, will keep investigating. Locally everything works for me


build(submodules): fix ffmpeg branch in gitmodules

Description

This is a minor correction to the .gitmodules file, which was missed in #136.
Screenshot

Issues Fixed or Closed

Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Dependency update (updates to dependencies)
[ ] Documentation update (changes to documentation)
[x] Repository update (changes to repository files, e.g. .github/...)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated the in code docstring/documentation-blocks for new or existing methods/components

Branch Updates
LizardByte requires that branches be up-to-date before merging. This means that after any PR is merged, this branch
must be updated before it can be merged. You must also
Allow edits from maintainers.

[ ] I want maintainers to keep my branch updated

This fix is fine, but maybe we don't want Dependabot spamming us every day with changes from FFmpeg master. What does it do if we specify no branch there?

No branch will be the same as the default branch.
We definitely don't have to address each PR... we could also disable dependabot just for ffmpeg if we wanted to.


Mention Chocolatey in README.md

I've created a chocolatey package for epr. The page for the is https://chocolatey.org/packages/epr.
My source for the package is at https://github.com/cybercatgurrl/chocolatey-pkgs/tree/master/epr.
You can mention something like you can install with Chocolatey "choco install epr" as soon as the package goes public.
Cheers!

Hello there, thankss for making it available in choco, but when I read it in choco package website, looks like it still uses old version of epr. I just made newer binary release: https://github.com/wustho/epr/releases/tag/v2.2.7
Can you update epr in choco by using v2.2.7b (not v2.2.7) which behave like common epub reader which common people accustomed to??

I can update it but just a note for the future, adding a new version in an existing tag makes it harder to process in my automatic updating scripts. I'm working on setting up an appveyor build so that it automatically updates but it would be useful not to do this in the future.

Thanks mate, sorry so much I know nothing about batch scripting or something...

Thankss @cybercatgurrl
But I kinda lost in:

As long as you don't add new versions to already existing tags

Can you please elaborate? What shall I do when releasing new versions so your scrit can automatically pick it up?

2.2.7b is a new version and not an alternate version like md right? Will there be a 2.3.0b or 2.2.8b? If it's a separate build then I can account for that but if it was a one off thing I can't account for that. Like if 2.2.7b is a new version of the program then it would have been picked up if you made a new release.
The script will only pick it up if there is a new release. So for instance if 2.2.8 is released then it will automatically package epr-v2.2.8-win.zip and push it to chocolatey. Or should it be packaging epr-v2.2.8b-win.zip?

Or should it be packaging epr-v2.2.8b-win.zip?

Is it too much bother for you if it should?

Sure I can change the script to reflect that. As long as that continues to be the naming convention for releases.
On 9 May 2019, 7:03 PM +1000, Benawi Adha<EMAIL_ADDRESS>wrote:

Or should it be packaging epr-v2.2.8b-win.zip?
Is it too much bother for you if it should?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Yess thanks so much, I will consistently update it with vX.X.Xb...
Just update the README


[Monitoring] Allow users to configure DNS monitoring

Problem:
Doesn't allow user to configure DNS monitoring
Solution:
Remove false configuration item from mandatory option:

if interested in kube-dns metrics (v1.12)

configuring exporter-kube-dns.enabled item as true when
enabling cluster-level monitoring
guaranteeing the DNS service of kube-system namespace has:

k8s-app=kube-dns label
dnsmasq-metrics port with TCP:10054
skydns-metrics port with TCP:10055

if interested in coredns metrics (v1.14)

configuring exporter-coredns.enabled item as true when enabling
cluster-level monitoring
guaranteeing the DNS service of kube-system namespace has:

k8s-app=kube-dns label
metrics port with TCP:9153

Issue:
https://github.com/rancher/rancher/issues/21066

LGTM


Upgrading to 2.5 from 2.3.3 causes textfields to type in reverse

I am using flutter web and I just upgraded to 2.5 and now when I type in a textfield the letters are reversed. Lets say I am typing ANIMAL it will go A -> NA -> IMA -> etc and the only change was upgrading my flutter version. If I downgrade flutter downgrade back to 2.3.3 it starts working correctly again
final usernameController = TextEditingController();
    TextField(
        controller: usernameController,
        textCapitalization: TextCapitalization.characters,
        decoration:
            InputDecoration(labelText: 'Username', errorText: usernameError),
      ),

This is the result of the upgrade
Flutter 2.5.0 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 4cc385b4b8 (16 hours ago) • 2021-09-07 23:01:49 -0700
Engine • revision f0826da7ef
Tools • Dart 2.14.0

Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.5.0, on macOS 11.5.2 20G95 darwin-x64, locale en-CA)
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 2020.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.2)
[✓] Connected device (1 available)

Hi @igloo12, thanks for filing the issue.
Can you please provide a minimal complete reproducible code sample?
Thank you

The issue was from the controller I have to reset the position of the cursor on change now
_usernameController.addListener(() {
      setState(() {
        final upperCase = _usernameController.text.toUpperCase();
        if (_usernameController.text != upperCase) {
          _usernameController.text = upperCase;
         // ------------- NEW CODE---------
          _usernameController.selection = TextSelection.fromPosition(
              TextPosition(offset: upperCase.length));
        }
      });
    });

@igloo12, I don't believe that should be necessary.
Can you share your code sample?
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


feat: add profiling info to benchmarks

Problem
The benchmarks currently return the number of instructions it took to execute each benchmark. While this number is useful to measure performance, it doesn't provide insight into where these instructions are being used and where the performance bottle necks are. Without this information, making informed performance optimizations would require a lot of trial and error.
Solution
The typical solution to this problem is to use some kind of profiler. ic-repl already supports profiling and can output a flamegraph of where instructions are being spent, but it has a few drawbacks that makes it difficult to use:

The names of rust methods are mangled, even when debug = 1 is turned on, making it hard to make sense of the output.
Each benchmark includes logic to first setup, and only after setup would we want to profile, so we'd need a way to programmatically tell the profiler to reset its measurements.
Often we'd like to benchmark blocks of code that aren't functions.
It's difficult to extend, meaning that we can't easily swap the "instruction count" for some other performance measurement, such as "pages written".

To address the issues above, this commit introduces a "poor man profiler". This profiler is manual, in the sense that the developer adds to the code hints for what they care about profiling. In this PR, I added a few of these hints, and the benchmarks now return an output that looks like this:
dfx canister call benchmarks btreemap_insert_blob_256_1024
[Canister rwlgt-iiaaa-aaaaa-aaaaa-cai] {
    "btreemap_insert": "3_511_533_555 (97%)",
    "load_keys": "1_145_464_345 (31%)",
    "load_values": "876_374_691 (24%)",
    "node_load": "2_808_121_190 (77%)",
    "node_save": "182_500_379 (5%)",
}

This approach is simple and effective, but it does have drawbacks.

It makes the instructions count inaccurate. The profiling logic itself consumes cycles, making the instructions count inaccurate. I think we can limit this inaccuracy by making the profiler crate internally account for its own overhead and deducting those from its measurements.
It doesn't currently support depth, meaning that the profiler doesn't know that, in this specific example, it's unaware that load_keys is part node_load, but that's easy to introduce.
Syntax isn't the cleanest, but it can be improved with something like a macro.

Looks like a reasonable approach to me.
Reading the list of drawbacks you mention for the profiler included with ic-repl, I'm wondering whether it would make sense to try to fix those drawbacks instead of introducing our own custom profiler. It's always a bit easier to say "the current tooling isn't working well for us, let's create our own version" but I'm worried that we're reinventing our own approach when there might be a path forward that allows us to reuse existing machinery as much as possible.
That said, I'd also assume you've done some research already on the potential of reusing or what it would take to extend the current profiling tooling to cover our needs and maybe it's not worth it. Mostly I want to make sure we're making a conscious decision. In any case, I'd give this feedback for ic-repl to Yan and see if we can make improvements long term. The problems you noticed might be issues that other developers have hit or will hit at some point in the future.

I've reincarnated this draft in #116.


Error on attaching category by slug

In the documentation it says
// Single category slug
 $post->attachCategories('test-category');

However this generates an error Invalid argument supplied for foreach()
doing $post->attachCategories(['test-category']); works.

So think I have found the problem in function prepareCategoryIds($categories)
    if (is_string($categories) || (is_array($categories) && is_string(array_first($categories)))) {
        $categories = app('rinvex.categories.category')->whereIn('slug', $categories)->get()->pluck('id');
    }

The problem is that if $categories is a string it is passed unchanged the the whereIn clause, which expects an array.

I can try and do a PR if that's ok.

I also had a problem inserting categories using an array of IDs... whereas using array of slugs works fine... :(

@zcwilt Thank you for this catch and sorry for the -long- delay noticing this issue.
Fixed in recent commit https://github.com/rinvex/laravel-categories/commit/c25d97d29622dd6ddb455ed39cfbbe6b764dfd97

@bzx thank you for your patience, this has been fixed in recent commit too https://github.com/rinvex/laravel-categories/commit/8ca17e72ed7849a9d38a5aecda954c3a81cc2938


v0.5.3

Codecov Report

Merging #13 (dc9a2c3) into master (4a34e35) will decrease coverage by 0.29%.
Report is 1 commits behind head on master.
The diff coverage is 64.06%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
- Coverage   68.48%   68.20%   -0.29%
==========================================
  Files          27       27
  Lines        1006     1016      +10
==========================================
+ Hits          689      693       +4
- Misses        269      273       +4
- Partials       48       50       +2

Flag
Coverage Δ

unittest
68.20% <64.06%> (-0.29%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

config/option.go
87.32% <ø> (+1.99%)
:arrow_up:

config/reader.go
79.62% <72.22%> (-1.56%)
:arrow_down:

config/value.go
47.58% <73.33%> (-1.59%)
:arrow_down:

config/config.go
52.17% <37.50%> (+0.48%)
:arrow_up:

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


Improve marathon spec tests

After https://github.com/Capgemini/Apollo/commit/4f44df0b042346540910758273934c2b4c02a2ba got in, we could improve the spec tests around marathon to include -

[ ] Test existence of file marathon-wait-for-listen.sh (with correct mode etc...)
[ ] Run the above file and test we get a 'pong'
[ ] Test /etc/init/marathon.override does NOT exist
[ ] Test marathon consul service definition file exists
[ ] Test /etc/init/marathon.conf exists with correct content/permissions

this is covered by https://github.com/Capgemini/Apollo/pull/338/files

also https://github.com/Capgemini/Apollo/pull/344/files


Add manual dispatch trigger to publish platform

only public platform ci was changed in PR, one test failed randomly, not connected with changes


Github webhook build failed

I have phpci configured for github private repository and it is working fine if i run build manually or with default webhook confuration. But i'm trying to configure webhook for pull request event. I have added github token into phpci config.yml. But getting strange error:
Submodule 'asset/layouts/widget' (git@github.com:user/widget.git) registered for path 'asset/layouts/widget'
Submodule path 'asset/layouts/widget': checked out '967b036b561e5e785ddad1a9cff05cc142c25ba4'
Cloning into '/home/phpci/www/phpci/PHPCI/build/121_616e1'...
Cloning into 'asset/layouts/widget'...

Switched to a new branch 'phpci/121'
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Failed to clone remote git repository.
Exception: Could not create a working copy.
Removing Build.

But manually runned builds are ok:
Submodule 'asset/layouts/widget' (git@github.com:user/widget.git) registered for path 'asset/layouts/widget'
Submodule path 'asset/layouts/widget': checked out '967b036b561e5e785ddad1a9cff05cc142c25ba4'
Cloning into '/home/phpci/www/phpci/PHPCI/build/118_72e34'...
Cloning into 'asset/layouts/widget'...

6cab5ba91a1f329a64e5bf83670dcb9bb5ec2836
Working copy created: /
RUNNING PLUGIN: composer
Using --prefer-dist flag
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file

Generating autoload files

PLUGIN: SUCCESS
RUNNING PLUGIN: mysql
PLUGIN: SUCCESS
RUNNING PLUGIN: shell
PLUGIN: SUCCESS
RUNNING PLUGIN: codeception
PLUGIN: SUCCESS
RUNNING PLUGIN: mysql
PLUGIN: SUCCESS
BUILD SUCCESS
Removing Build.

Any ideas how to fix this? Does phpci generates any logs or has verbose mode for runner?

So, i found this pretty bug.
file PHPCI/Model/Build/GithubBuild.php
/**
     * Handle any post-clone tasks, like applying a pull request patch on top of the branch.
     * @param Builder $builder
     * @param $cloneTo
     * @return bool
     */
    protected function postCloneSetup(Builder $builder, $cloneTo)
    {
        $buildType = $this->getExtra('build_type');
        $success = true;
        try {
            if (!empty($buildType) && $buildType == 'pull_request') {
                $remoteUrl = $this->getExtra('remote_url');
                $remoteBranch = $this->getExtra('remote_branch');
                $cmd = 'cd "%s" && git checkout -b phpci/' . $this->getId() . ' %s && git pull -q --no-edit %s %s';
                $success = $builder->executeCommand($cmd, $cloneTo, $this->getBranch(), $remoteUrl, $remoteBranch);
            }
        } catch (\Exception $ex) {
            $success = false;
        }
        if ($success) {
            $success = parent::postCloneSetup($builder, $cloneTo);
        }
        return $success;
    }

But git pull -q --no-edit %s %s' require ssh authorization for private repos!
As hotfix i made
/**
     * Handle any post-clone tasks, like applying a pull request patch on top of the branch.
     * @param Builder $builder
     * @param $cloneTo
     * @return bool
     */
    protected function postCloneSetup(Builder $builder, $cloneTo)
    {
        $buildType = $this->getExtra('build_type');

        $success = true;

        try {
            if (!empty($buildType) && $buildType == 'pull_request') {

                $keyFile = $this->writeSshKey($cloneTo);

                if (!IS_WIN) {
                    $gitSshWrapper = $this->writeSshWrapper($cloneTo, $keyFile);
                }

                $remoteUrl = $this->getExtra('remote_url');
                $remoteBranch = $this->getExtra('remote_branch');

                $cmd = 'cd "%s" && git checkout -b phpci/' . $this->getId() . ' %s && git pull -q --no-edit %s %s';

                if (!IS_WIN) {
                    $cmd = 'export GIT_SSH="'.$gitSshWrapper.'" && ' . $cmd;
                }

                $success = $builder->executeCommand($cmd, $cloneTo, $this->getBranch(), $remoteUrl, $remoteBranch);
            }
        } catch (\Exception $ex) {
            $success = false;
        }

        if ($success) {
            $success = parent::postCloneSetup($builder, $cloneTo);
        }

        return $success;
    }

but this perform some useless duplicating and idealy $gitSshWrapper needs to be passed as argument or stored in class params. I can make a pr if needs.

@hotrush   +1        Experienced the same issue with pull request hook build and this patch fixed. I think your patch can be simplified to:
**
     * Handle any post-clone tasks, like applying a pull request patch on top of the branch.
     * @param Builder $builder
     * @param $cloneTo
     * @return bool
     */
    protected function postCloneSetup(Builder $builder, $cloneTo)
    {
        $buildType = $this->getExtra('build_type');

        $success = true;

        try {
            if (!empty($buildType) && $buildType == 'pull_request') {
                $remoteUrl = $this->getExtra('remote_url');
                $remoteBranch = $this->getExtra('remote_branch');

                $cmd = 'cd "%s" && git checkout -b phpci/' . $this->getId() . ' %s && git pull -q --no-edit %s %s';

                if (!IS_WIN) {
                    $keyFile = $this->writeSshKey($cloneTo);
                    $gitSshWrapper = $this->writeSshWrapper($cloneTo, $keyFile);
                    $cmd = 'export GIT_SSH="'.$gitSshWrapper.'" && ' . $cmd;
                }

                $success = $builder->executeCommand($cmd, $cloneTo, $this->getBranch(), $remoteUrl, $remoteBranch);
            }
        } catch (\Exception $ex) {
            $success = false;
        }

        if ($success) {
            $success = parent::postCloneSetup($builder, $cloneTo);
        }

        return $success;
    }

@bdanchilla yeap, thanks!

@hotrush @bdanchilla - Sorry it has taken me ages to get to this. Would either of you mind submitting a PR for this fix? It would be super helpful.
Thanks! 👍

@dancryer Hi, i will take a look bit closer on holidays and prepare a PR.


MasterDetailPage Xamarin Forms iOS Swipe Issue

Hi there,
just implemented MapsUi, works like a charm on Android. But on iOS swiping stucks.
Found out that the issue is MasterDetailPage. If I set the Property " IsGestureEnabled" from MasterDetailPage to false, swiping works as expected.
Have anybody made the same experience with that?
Project is .net Standard ( Xamarin Forms) with the newest nugets.
Thank you in Advance.

Thanks for your report. I am currently trying to avoid Forms but hope to get to it in the coming weeks. I will investigate this then. I assume the gesture captures the event we use for regular panning. I think we should take some time to consider how we should work with events. My idea is to keep it 'open', in a way to users can use the existing events and call Mapsui features from that instead of adding a Mapsui layer of events and gestures to hide the existing events. Not sure if this is possible.
If I understand you correctly you do have a workaround, so this is not an urgent matter for you?

Hi XamHans, you need to set the IsGestureEnabled to false as you mentioned, otherwsie the MasterDetail view will override the swipe guesture, this is a common design issue with Xamarin.Forms MD with a detail view that uses touch and nothing to do with Mapsui.

Hi Mark,
nice to hear from you.
I want to ask, if you have some spare time to look into the Gorms implementation of Mapsui. You could find it (here)[https://github.com/charlenni/Mapsui/tree/Forms?files=1]. It uses only native Xamarin.Forms.

@charlenni - Anythign you'd like me to have a look at specifically?

@radderz: Any comment is appreciated 😉
I tried to make a native Xamarin.Forms component for Mapsui not a wrapper. There is no one, I could ask, if it is correct and works. Paul want to look into it later.
Btw. You said, that you have code to move the map with a gesture (flinger?). I added a velocity tracker to the shared code of Mapsui as first part. Perhaps you can help, how to do the next steps. And you told, that you have animations for moving the map. No problem with Xamarin.Forms part, but would be wonderful to have it for all platforms.

@charlenni - @pauldendulk - Is this something we would bring into the main repo? If so I can have do some validation and start doing some changes where applicable.

Do you mean the Xamarin.Forms MapControl or the other things? I think both are useful parts for Mapsui.

I looked into the MapControl of the Xamarin.Forms implementation: it isn‘t only a Xamarin.Forms, but could be used as a native Skia control. So it could be used on all platforms that SkiaSharp supports. Around this control is a MapView, that derives from Xamarin.Forms View.


[Edge] CSS from multiple layouts is being included on individual pages

Version
<EMAIL_ADDRESS>and v1.4.2 (see below).
Reproduction link
https://github.com/markplewis/nuxt-layout-css-issue
Steps to reproduce

Clone the following example repo.
Run npm install then npm run dev.
Navigate between the pages and inspect the CSS that's applied to the  element via your browser's dev tools.

https://github.com/markplewis/nuxt-layout-css-issue
What is expected ?
When a page specifies a layout, like this:
pages/index.vue:

Only the CSS from the corresponding layout file should be included in the page:
layouts/home.vue:

What is actually happening?
The CSS from every layout file is being included on every page, regardless of which layout is specified in the page's .vue file.

Additional comments?
My example repo is using Nuxt Edge.

This bug report is available on Nuxt community (#c7723)

Can confirm this bug.
Workaround:
 build: {
    splitChunks: {
      layouts: true
    }
}

This is also a problem in Nuxt v1 (see the v1 branch of my demo repo). Styles from multiple layouts are still being pulled into the page, but at least I'm not seeing styles from the default layout:

I have same issue.

@hareku I was able to solve my issue using
build: {
splitChunks: {
layouts: true
}
}

@Legym
Thanks!!!

I upgraded my demo project to Nuxt 2.1.0 and I'm still experiencing this problem. As @manniL stated above, adding splitChunks: { layouts: true } solves the problem for first render (i.e. loading the page directly or hard refreshing your browser), but the problem persists when navigating back and forth between pages after that.
My actual, real-world project only uses 2 layouts and it doesn't seem to be affected by this issue. I've also added splitChunks: { layouts: true } to that project. The problem persists in my demo project, however, which uses 3 layouts. So, I wonder if the problem only crops up when there are more than 2 layouts?

@markplewis I had a different folder setup than you. We created a theme system inside our nuxt build and each theme was locked into their own URL. We didn't go from one theme to another directly and so we didn't run into your issue.

splitChunks: { layouts: true }  did solve our issue because originally our styles in Layout would bleed into other themes. I.e Styles from Desktop were leaking into Mobile.
However I would say the issue you are having is definitely a bug. The only suggestion I have is to do something like this
Layout 1
<template>

    <nuxt/>

</template>

Layout 2
<template>

    <nuxt/>

</template>

The same issue happening for me also while loading loading layout dynamically

Still experiencing this problem too,  any solutions? ,  layouts true doesn't fix it.

We are currently working on a solution that could work without any changes, for both critical CSS and extractCSS modes. Stay tuned :)

Thanks,  my temporary solution was to import scss or styl file inside

I want to use splitChunks with extractCSS option.

Any solution yet, I'm experiencing same here. Please I'm seeing a lot of temporaty fix but no detailed explanation of them. I need your help
Thanks

Issues that are labeled as 🕐Pending will not be automatically marked as stale.

Oops Stale Bot - you had one job! 😂
Really excited for the Nuxt team to fix this one. Go team go! 😄
(please don't stale/expire this issue...)

I find a way to make it work on production, (no way yet for development).
in nuxt.config.js
import crypto from 'crypto';

const BUILDID = crypto.randomBytes(10).toString('hex');

module.exports = {
   // ...
  build: {
    extractCSS: true,
    // ...
    filenames: {
      css: `${BUILDID}.[name].css`,
    },
    // ...
  },
  env: {
    BUILDID,
  },
};

in layouts/*.vue
<template>
  <nuxt />
</template>

As the dev server use in-memory compilation, i wasn't able to find the correct path for integrate compiled css file into current page in development. If you want to try, make sure you freeze the extractCSS property, ex: ...Object.freeze({ extractCSS: true }),;

So, after some investigation, I succeeded make it work in dev :
in nuxt.config.js
import crypto from 'crypto';

const BUILDID = crypto.randomBytes(10).toString('hex');

module.exports = {
  // ...
  modules: [
    // ...
    '~/modules/ForceExtractCSS',
  ],
  // ...
  build: {
    extractCSS: true,
    // ...
    filenames: {
      css: `${BUILDID}.[name].css`,
    },
    // ...
  },
  env: {
    BUILDID,
  },
};

in modules/ForceExtractCSS.js
export default function ForceExtractCSSModule() {
  this.nuxt.hook('ready', () => {
    this.options.build.extractCSS = true;
  });
}

and in layouts/*.vue
<template>
  <nuxt />
</template>

Canvas : HMR isn't working, head() isn't recalculated after HRM call

FYI: I just updated my demo project to Nuxt 2.4.3.

@markplewis Are you still experiencing this issue in the latest version?
I am currently in the middle of build using Nuxt 2.4.0 and undesired multiple Layout/CSS cross-compilation is still an issue.

@nick-lawrence I just upgraded my demo project to Nuxt 2.5 and I'm still experiencing this issue.

So, I wanted my login page to have a coloured background and the default layouts to have a white background.
This silly workaround does the trick for me.
  // login.vue
  beforeCreate() {
    document
      .getElementsByTagName('html')[0]
      .setAttribute('style', 'background-color: #4fc08d')
  }

  // default.vue
  beforeCreate() {
    document.getElementsByTagName('html')[0].removeAttribute('style')
  }

+1 (Nuxt v 2.4.0)
In my case not only the layout "leaks" CSS, "pages/index.vue" too.
I'm adding an extra layout and extra pages (using this layout) through a Nuxt module, as a some sort of sub-application:
this.addLayout(path.resolve(__dirname, 'custom/layout.vue'), 'custom') this.extendRoutes((routes, resolve) => {   routes.push({     path: '/subapp',     name: 'subapp',     component: resolve(__dirname, 'custom/subapp.vue')   }) })

my workaround for now is
nuxt.config.js
build: {
    splitChunks: {
      layouts: true
    }
}

and then reload page beafore each layout change, mixin in all layouts:
    beforeCreate() {
        if (!process.server) {
            if (window.__NUXT_LAYOUT__ && window.__NUXT_LAYOUT__ !== this) {
                window.location.reload();
            }
            window.__NUXT_LAYOUT__ = this;
        }
    },

Hello!
Is there any solution yet? Still experiencing the issue in Nuxt 2.9.2

We have the same problem. It seems to work for us with scoping every layout

+1
Nuxt 2.10.1

I am having 2 languages, with RTL and LTR...
So i wanted to build 2 layouts to load RTL.CSS or LTR.CSS
And it uploads both of them :( so the website brokes...
Waiting for a solution.

Without this feature layouts are unusable for our projects.... Can we help you guys somehow?

for now i m using this way.
give each layout a middleware.
each layout middleware should have code like this :
export default function(ctx) {
  const { redirect, app, from } = ctx;

  const link = [
    { rel: "icon", href: "admin-layout/img/icon.ico", type: "image/x-icon" },
    {
      rel: "stylesheet",
      href: "/admin-assets/css/bootstrap.min.css"
    },
    {
      rel: "stylesheet",
      href: "/admin-assets/css/atlantis.min.css"
    },
    {
      rel: "stylesheet",
      href: "/admin-assets/css/fonts.min.css"
    },
  ]

  const script = [
    { src: "/admin-assets/js/core/jquery.3.2.1.min.js" , body: true },
    { src: "/admin-assets/js/core/popper.min.js", body: true },
    { src: "/admin-assets/js/core/bootstrap.min.js", body: true },
    { src: "/admin-assets/js/plugin/jquery-ui-1.12.1.custom/jquery-ui.min.js", body: true },
    { src: "/admin-assets/js/plugin/jquery-ui-touch-punch/jquery.ui.touch-punch.min.js", body: true }
  ]

  app.head.link = link;
  app.head.script = script;
}

for navigation between layout you should use window.location.href

Any progress on this? I was sure I was doing something wrong. It's pretty much useless to use more than one default layout if CSS is getting merged...

Without this feature layouts are unusable for our projects.... Can we help you guys somehow?

exactly same here, we stuck totally with this, I already blaming myself for switching to nuxt.
Overall layout styles should be scoped for everything what is included in that particular layout, otherwise they are just unusable.

is this going to be fixed anytime soon please?

We are currently working on a solution that could work without any changes, for both critical CSS and extractCSS modes. Stay tuned :)

Dear @Atinux, @alexchopin, @manniL, would you mind to give some feedback on this ticket. I'm sorry I'm and others bothering you here, but as you could probably see, this is a very important feature for us. For someone like me, it's a show-stopper.
Please note, that I'm just asking about some feedback, not the implementation itself. For a project with 24k stars a year w/o a feedback doesn't make it trustworthy.
Have you started working on it? Could you please share some details of the implementation so we could potentially join? Or if you haven't, we would know that we can start our own research and potentially propose some PR?
Again, thank you guys for all you work you're doing here! This is worthless and today it is the best SSR framework for Vue. Sorry for my impatience and curiosity.

I would love to help too, if there's anything people can do please tell us.

This is still broken in 2.11.0

I think this is related with Issue with template styles not unmounting on programmatic navigation #2561 and the problem is in vue-loader when does not remove unused css in different layout/page (when you have build.splitChunks.layouts === true ).

Another workaround could be to assign a class to the -element and make sure, that all layout-related styles are part of that class-scope, e. g.
body.layout1 .menu {  }

Hi there,
Sorry for the late answer and happy new year at the same time!
Actually one possibility is to use @gambolputty suggestion with bodyAttrs:

And to prefix your style with .dark-theme .my-class { ... }.
Please find a live example here: https://codesandbox.io/s/nuxt-layouts-theming-ru954
We will beta test Vue 3 to test better layout scoping, will keep this thread updated.

I am using nuxt v.2.10 for a project and unfortunately it does not work with style files included in different layouts.
Layout A:
<template />

Layout B:
<template />

I tried it with the options in nuxt.config  splitChunks.layout and extractCss but both configurations do not work. If the user switches the layout from Layout A to Layout B
the style of the former layout is loaded. Our workaround at the moment is a hard page reload.

Still broken ^2.11.0

@pi0 any idea on this if it's on the roadmap to fix or anything?

Still broken (^2.0.0)

I hope the guys who were working on it did not died of Covid  🤞

any news on this ? 😢

After multiple investigations, we won’t be able to fix this in 2.X, better is to use a normal link (not Nuxt-link) if you really want to have a completely different CSS for each layout when switching.
The issues comes with vue-loader that include the

Next.js also has this problem. Don't tangle. Caused by webpack hot update and history api.   one project try to use one reset css. one common css.  Or don't use , that should be ok. Or try listen to router change  to add or remove root className

@icai mean Nuxt.js not Next.js

@NotMoni And @icai means that Next.js (react) also has the same problem. It's not exclusive to Nuxt

Oh sorry. Misunderstood that

Have multiple layouts, and each layout has it css - issue css gets overlapped all layouts.
Example
Layout A
Layout B
Layout A css get added to layout B css to
How to avoid this ?

This issue is now resolved in 2.12.2 !
Patch note: https://fr.nuxtjs.org/guide/release-notes/#v2.12.2
"#7139 Sanitize layouts after resolve for splitChunks.layout"
Now you can use this in your nuxt.config.js:
build: {     splitChunks: {       layouts: true     } }

Well actually it won’t remove the cas injected by Layout A when moving to Layout B 😬
We are looking to add this feature in Nuxt 3 / Vue 3, it is something I would like to have inside Nuxt :)

This example by @Atinux and @gambolputty below will work fine. But it's important to define the bodyAttrs class in all your layout files.

If you in some layouts don't want to define a classname you can add an empty classname in those layouts.

Really big issue for me too, have tried all the things mentioned in this thread. The bodyAttrs workaround is too cumbersome and impractical for me since I have many CSS properties.

Prefixing styles with a class added through bodyAttrs doesn't help me since I need to @import a generic file 🤷♂
Too bad … my solution will be to add it in every page where necessary.

any updates?

I think all of this could be saved by vue-meta if we were only allowed to include assets within head(). I'm considering rewriting in PHP because this is a bigger issue than I ever expected. Shame I have to have this issue near the end of my project. My frustrations with Nuxt goes further than this issue and my only hope is that some day I can come back to it, but this is the third major strike out for me.

Bug still going strong

I have fixed this by disabling extractCSS.
I also noticed a performance boost in lighthouse.
  // nuxt.config.js
  build: {
    extractCSS: false, // true causes duplicate css
    splitChunks: {
      layouts: true
    },
  },

So do @nuxt plan to solve this problem?

still same

3 years age, bug still here

👍 on here

i have the same issue with nuxt keeping the css from the previous pages after navigation in nuxt 3.6.3

@matthew-dean I suspect you're talking about the browser behaviour that respects the first definition of a global style. (Also, once loaded, CSS won't be unloaded from the browser.)
It's not a Nuxt bug, or indeed a bug at all.
Rather, you should probably use scoped styles or CSS modules to avoid this kind of specificity issue.

@matthew-dean Would you open a new issue with a reproduction, so we can discuss it better? This issue is about layout CSS and is not related to Nuxt 3 - it was opened 5 years ago.

I wonder if this wouldn't be the same solution as #22871 - treating it not as a bug but possible expecting behavior which can be resolved by using the deep selector (Nuxt 2/Vue2) 🤔


[MacOS] Editor crashes on launch, mscorlib.dll missing

Godot version:
3.0
OS/device including version:
MacOS Sierra / 10.12.6
Issue description:
Editor crashes on launch.
I've installed Mono 5.8 but when I launch the Godot executable, it keeps looking for mscorlib.dll in the 5.4.1 folder (which doesn't exist)
Mono: Initializing module...
The assembly mscorlib.dll was not found or could not be loaded.
It should have been installed in the `/Library/Frameworks/Mono.framework/Versions/5.4.1/lib/mono/4.5/mscorlib.dll' directory.

When I run the --version flag on the command line, I get the correct mono version (which has mscorlib.dll) - but Godot seems unable to find it.
➜  Versions mono --version
Mono JIT compiler version <IP_ADDRESS> (2017-10/9aa78573ee2 Wed Jan 10 04:30:03 EST 2018)
Copyright (C) 2002-2014 Novell, Inc, Xamarin Inc and Contributors. www.mono-project.com
	TLS:           normal
	SIGSEGV:       altstack
	Notification:  kqueue
	Architecture:  amd64
	Disabled:      none
	Misc:          softdebug
	LLVM:          yes(3.6.0svn-mono-master/8b1520c8aae)
	GC:            sgen (concurrent by default)

#16304

This is know issue - #13355, you can force mono path by setting MONO_PATH environment variable to /Library/Frameworks/Mono.framework/Versions/Current/lib/mono/4.5/, or rebuild Godot from source with newer Mono version.

Ok, the Mono issue seems to be fixed this way, I can see the project manager, but selecting a project I get this now:
Stacktrace:

Native stacktrace:

Mono: Runtime cleanup...

[Process completed]

Probably unrelated so I'm closing this for now.


catch uncaught error which triggered when youtube id not found

Anyone available to do a review on this one?  Thanks.

Thanks @jcop007 👍


validate when field not empty

i expect to validate a field that should be empty or not empty in email format.
how can i achieve that(with buildin functions,dont want to custom validators or use must lamba^_^)?
thanks

i know that ,sorry to bothered you