8muses gives Error: Null when downloading large amount of images

[NSFW]
Ripme version: 1.5.14
Java version (output of java -version): 1.7.0_55
Operating system: Windows 10
Exact URL: anything really large (ex: https://www.8muses.com/comix/album/taboolicious_xxx-comics)
Please include any additional information about how to reproduce the problem: Attempt to download an album with a few hundred (or thousand) images and eventually the program will give the error.

Expected Behavior
Downloads all the albums in an 8muses artists page and puts them in the download directory.  Albums with multiple parts are downloaded to the same folder in the rip.
Actual Behavior
Says 'rip complete' then Error: 'null' if the amount of items being downloaded is too large, then deletes the folder that was created in the download directory.
I would assume this may be an API thing, or something to do with too many requests. It may also be based on the number of albums rather than number of images, but it's difficult to tell.
Ex:  (https://www.8muses.com/comix/album/various-authors/sillygirl) DOES work, and it has nested albums (girly watch pt 1-4 are under separate groups)
But something with a similar layout like (https://www.8muses.com/comix/album/taboolicious_xxx-comics) DOES NOT work.  Note that the latter link has many more images.
Downloads work fine in the 0-400 image range on a few albums, but don't seem to work for the same amount of images in 20-30 albums (some of which are inside other directories).
Note that it is possible to save all the same images from an artist by queuing the albums individually.
There may also be something obvious that I'm missing here, I'm new to this program.
P.S.   If you need me to do anything I'm around to test, otherwise hopefully you guys can figure it out :)

After a bit of debugging it seems that getFullSizeImage() is not returning the full size image url, but rather null. It's probably just another css change and I should have it fixed today

This should be fixed in 1.5.15


Remove auth token

Type: Quesion
There is any way to remove an auth token from .npmrc generated file?
We need invalidate some generated tokens.

Just change the secret field within the database (.sinopia-db.json or .verdaccio-db.json) value for something different and restart the server.

That will invalidate all tokens in all clients.

@juanpicado Is the ability to invalidate a single user in the roadmap?

We have something in progress (https://github.com/verdaccio/verdaccio/issues/541) and in addition with v4 JWT (JSON Web Token) you can set a expired time (https://github.com/verdaccio/verdaccio/pull/896). In v3 the token never expires unless you change the secret key.


Docker Integration

Docker Integration
I have some thoughts about how this should happen. The Docker integration part of this project should take place in two parts. A client side build task, integrated with the existing Gulp workflow we already have, and a server-side (docker host) application that does the heavy lifting.
Client build task
This can actually be pretty simple. Pushing some sort of information to the server, and asking it to start a build. Additionally a web-hook could be made available to automatically build these environments based on branch creation.

[ ] Collect data for build server
[ ] Send data (JSON?) to build server

Server application
This is where things get potentially tricky. I've already worked out a bunch of this, from the 4k-labs demo I gave earlier this year. However, we'll still have to setup this tool to correctly receive the JSON and then build a container based on that information.

[ ] Receive build specific JSON
[ ] Create temporary mysql database, returning credentials/information.
[ ] Run setup tasks based on client-side workflow.

@patrickocoffeyo If you want to get together to discuss this, lets work on something.

Done and merged.


modified assertions to allow for a user supplied callback

Similar to
Mail::assertSent(FooMail::class, function ($mailable) {
    return true;
})

this allows a developer to assert against the $export data. Such as:
Excel::assertDownloaded('my_file.xls', function (\Illuminate\Support\Collection $export){
    return $export->first()->keys()->contains(['foo', 'bar']); // or what ever the developer wants to assert.
})

Interesting! Thanks! I'll play around with it a bit.

@devinfd feel free to check the docs and extend it if you have ideas: https://github.com/Maatwebsite/Laravel-Excel/blob/3.0/docs/export/testing.md

Ah yes! I forgot about the FromCollection interface. It makes the tests much more simple and clear.


Notice the 2KiB950B in the zed lake log output below. It seems this would be more readable if presented as something like 2.95KiB or just truncated to a mere 2KiB , or rounded up to 3KiB.
$ zed -version
Version: v0.29.0-384-g62551977

$ rm -rf $ZED_LAKE_ROOT

$ zed lake init
using environment variable ZED_LAKE_ROOT
lake created: /Users/phil/logs

$ zed lake create -p logs -orderby ts:desc
pool created: logs

$ zed lake load -p logs ~/work/zed-sample-data/zeek-default/snmp.log.gz
1u0Do2tHkIncjpr0WU9XjnSIOLA committed

$ zed lake log -p logs
commit 1u0Do2tHkIncjpr0WU9XjnSIOLA
Author<EMAIL_ADDRESS>Date:   2021-06-15T21:49:43Z

    Add 1u0DnzMevfz3siutYeuEh51kqYo 2KiB950B bytes 65 records
       from time: (00 52 5a 1c 89 d7 3d 2a) to time: (60 35 95 90 57 d5 3d 2a)

Whoever picks picks up this issue, if they're feeling generous, may want to tackle companion Brimcap issue brimdata/brimcap#95, as it's similar in nature.

Verified in Zed commit af73996.
Repeating the original repro steps, now for this case we just see a single value in all in B units.
$ zed -version
Version: v0.29.0-385-gaf73996a

$ rm -rf $ZED_LAKE_ROOT

$ zed lake init
using environment variable ZED_LAKE_ROOT
lake created: /Users/phil/logs

$ zed lake create -p logs -orderby ts:desc
pool created: logs

$ zed lake load -p logs ~/work/zed-sample-data/zeek-default/snmp.log.gz
1u3CiM6kmsCZQS9re9fKhEI4xuE committed

$ zed lake log -p logs
commit 1u3CiM6kmsCZQS9re9fKhEI4xuE
Author<EMAIL_ADDRESS>Date:   2021-06-16T23:10:11Z

    Add 1u3CiKJvyqXbd43ZcCDuFwi626c 2998B bytes 65 records
       from time: (00 52 5a 1c 89 d7 3d 2a) to time: (60 35 95 90 57 d5 3d 2a)

Thanks @mccanne!


settings: Add option to import certificates for linux.

What's this PR do?
Add option to import certificate
Screenshots?
You have tested this PR on:

[ ] Windows
[ ] Linux/Ubuntu
[X] macOS

Argh, i will close this up as well! This has two diffrent PR in one and one part does not work. I will clean it up and we need check it so closing this for now.


Remove item spacing in sibling recordings/blueprints list

What
As the title say ☝🏻
PSA: always set spacing.item_spacing.y to 0 when using ListItem.
Before: unsightly 😄
After:

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested the web demo (if applicable):

Using newly built examples: app.rerun.io
Using examples from latest main build: app.rerun.io
Using full set of examples from nightly build: app.rerun.io

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!

PR Build Summary
Docs preview
Examples preview
Recent benchmark results
Wasm size tracking

Perhaps add a debug_assert to ListItem that checks that spacing.item_spacing.y is zero, if we should always do this?

Good call, lets see how intrusive it gets :)

How can it not be obvious why? O.o

You should at least do the same in other places where entity_db_button_ui is called from, e.g. in crates/re_data_ui/src/entity_db.rs

Good call. I didn't even realise that selecting a data source was a thing.


Handle pinching thumb and handtip mouse click debounce

Handle pinching thumb and handtip mouse click debounce to prevent another click until first click is released

merge

merge


Inconsistencies on CDVMediaStream.m class

Sorry to delete the template, but I think it doesn't apply.
There are some inconsistencies in CDVMediaStream.m file
You can find this lines:
I think this is wrong, label should be some name for the device, but description is sort of stringification of the native object
[dict setObject:device.description forKey:@"label"];
I think this three are wrong, it should be label instead of description
[videoTracks setObject:@"frontcamera" forKey:@"description"];
[videoTracks setObject:@"rearcamera" forKey:@"description"];
[audioTracks setObject:device.deviceType forKey:@"description"];
Also, in last one, deviceType is not available on iOS 9, so it should probably return an empty string. Or even better, return localizedName
I think this is wrong, kind should be "audio" or "video"
[dict setObject:device.deviceType forKey:@"kind"];
I think this two are right
[videoTracks setObject:@"video" forKey:@"kind"];
[audioTracks setObject:@"audio" forKey:@"kind"];
So the inconsistencies are, sometimes it uses @"description" and others @"label" and they use different values in different places for the same key.

Yes, there are some inconsistencies here. We are working on a separate branch issue3 which incorporates some of these changes as well

I've tried to checkout issue3 branch to work over those changes and send the PR with a fix there, but unfortunately it crashes on if([self.camDirection isEqualToString:@"frontcamera"]) with this error:

'NSInvalidArgumentException', reason: '-[NSNull isEqualToString:]: unrecognized selector sent to instance 0x39642310'

I'll send the PR against master, maybe you can then cherry pick to issue3 or rebase

@jcesarmobile yeah, feel free to make changes in the issue3 branch. You should have gotten an email inviting you into the project as a committer.

@macdonst yeah, I received it, but I prefer if somebody review my PRs first
I wrote a comment saying that issue3 branch didn't work and then I deleted it.
The problem is not really issue3 branch, it's that image-capture needs an update as description has been changed to label, I've sent a PR for it
https://github.com/phonegap/phonegap-plugin-image-capture/pull/30

@jcesarmobile yeah, I just wanted to make sure you got the invite and to let you know you can work on the feature branches with us.


New package: Trimble.SketchUp.Pro.2023 version 23.1.340

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] There's a linked issue. Resolve #146641

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

https://download.sketchup.com/SketchUpStudio-2023-1-340-117.exe

https://github.com/microsoft/winget-pkgs/issues/107639


Issues with "select current sentence" feature

I just started using this plugin and found the select current sentence feature to be neat, but the first sentence I tried it with was a bulleted list item, and I found that by default the Regex will select the list bullet. I have managed to fix this by changing the regex to:
[^.!?\-\*\+\s][^.!?]*(?:[.!?](?!['"]?\s|$)[^.!?]*)*[.!?]?['"]?(?=\s|$) (adds \-\*\+ in the first negated set)
I can put this in a PR for constants.ts if you'd like, though perhaps a better option (that I know far too little to work on myself) would be adding 'presets' to the settings page, with human-readable descriptions of a few different methods/expressions.
Note: I'm not the best with Regex but as far as I'm aware this doesn't have any glaring issues with it. Feel free to correct me if this would mess with anything.
Another issue I found is when the sentence you're selecting is at the end of a paragraph. I'm not that great with javascript/typescript but it seems like the issue is in actions.ts where the plugin tries to select the space after a sentence. If the sentence is at the end of a paragraph, though, this will select the next line below. This causes two issues:
If there is no line below what you're selecting, an "Uncaught RangeError" occurs in the console. Removing the +1 from line 170 prevents the error from occurring. I don't think this actually affects anything, but it's still nice to avoid.

Additionally, All characters in the found sentence following the character you're currently selecting will be moved to a new line, along with your editing caret. Removing the +1 from line 170 also prevents this from happening.

I personally don't see the value in selecting the space after the sentence, but it was put there intentionally so I understand you probably have reasons for doing so. The only reason I can think of for doing it would be to delete the sentence in its entirety, which is a good use of the select sentence feature, but could also be moved to its own command. A more user-friendly solution might be to not select the space afterwards, but to remember that it's there until an action is performed. If the delete key is pressed, then delete the space. This means users who were bolding/italicizing/quoting don't have to move their formatting.
Again, I'm not exactly able to do that myself, but I can create a PR for removing the +1 from line 170 in actions.ts if you'd like me to.
P.s. Thanks for all your work on this plugin! It fixes some annoying things with selection/deletion on macOS which can be very inconsistent otherwise, and I imagine it's even nicer to have on Windows/Linux which don't have the paragraph select feature AFAIK

Nice finds, and thanks for the detailed write-up!
I'm happy for you to raise a PR with the suggested fixes, otherwise I can get to it later this month since they're simple changes.
As for selecting the trailing space – the main reason was just to save an extra keystroke when deleting an entire sentence, but I can see how that gets in the way of other operations. Definitely agree with removing the +1 if it fixes those other bugs too 😀


Add zone redundant parameter to az servicebus create command

ServiceBus supports ZoneRedundancy in Premium SKU. There is currently not a mechanism to create ASB with ZoneRedundancy in premium tier using the CLI
https://docs.microsoft.com/en-us/cli/azure/servicebus/namespace?view=azure-cli-latest#az-servicebus-namespace-create
Ideally, this should be implemented similar to how SQL dbs can be created with Zone-Redundancy turned on (by applying --zone-redundant true)
https://docs.microsoft.com/en-us/cli/azure/sql/db?view=azure-cli-latest#az-sql-db-create
If Zone-Redundancy is requested for basic or standard tiers, the command should fail.

For reference: Zone-Redundancy was added to Sql commands via this issue: https://github.com/Azure/azure-cli/pull/5560

@rajrao , Thanks for submitting the bug, right now the ZoneRedundancy is available in 2018-01-01-preview, we are working on making it available in current stable version.
we will update the CLI with --zone-redundant parameter, for now, supporting 2018-01-01-preview API version.
will keep thread posted

@v-Ajnava  any update for this issue?

@v-Ajnava how about the current status of this issue?


Issue with "PAN-OS Policy Optimizer" in @site/docs/reference/packs/policy-optimizer.md

Describe the problem
Can not login the panorama instance using the latest policy optmizer 1.0.8
2022-10-04T11:31:00.028273716-04:00 debug: (PAN-OS Policy Optimizer_instance_117e3fd95-a7a3-49fd-87de-48ac3e9f10f8_PAN-OS Policy Optimizer_test-module)
Traceback (most recent call last):
File "", line 429, in main
File "", line 67, in login
ValueError: invalid literal for int() with base 10: ''
Entry
Executed on 2022-10-04T11:31:00.029010626-04:00
Raw Response section:
invalid literal for int() with base 10: ''.
Trace:Traceback (most recent call last):
File "", line 12025, in main
File "", line 11663, in login
ValueError: invalid literal for int() with base 10: ''
Traceback (most recent call last):
File "", line 429, in main
File "", line 67, in login
ValueError: invalid literal for int() with base 10: ''
Screenshots
Error Attached.
Environment

OS: [e.g. Windows] Linux
Browser: [e.g. chrome, safari, firefox..] Firefox
Browser Version:

Suggested fix

Hi @leocf8585 I close this issue since the issue is not related to the docs.
Please open support ticket.


Create clarity and consistency across different related articles

Create clarity and consistency across different related articles, mainly, the current one and this:
https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-claims-mapping#include-the-employeeid-and-tenantcountry-as-claims-in-tokens

@davidmu1 - Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


2nd time dismiss animation works but panel stay

Panel

Show
Dismiss
Show
Dismiss (Here after dismiss animation, panel stay remain on visible position )

Any help?
Below is my code, took latest from pods (2.0.3)
            if self.currentPanel != nil {
                panelManager?.dismiss()
                self.currentPanel = nil
            } else {
                currentPanel = UIStoryboard.instantiatePanel(identifier: "PanelOptions")
                var panelConfiguration = PanelConfiguration(size: .custom(200))
                panelConfiguration.panelVisibleArea = 30

                panelManager?.show(panel: currentPanel as! UIViewController & Panelable, config: panelConfiguration)
            }

Hi @deshan, I'm going to need more information:

This panel performs dismiss operation but it appears again? or a new panel comes up?
Where the code above is executed? After pressing a button?
Could you check if the panel deinit method is called when the panel is dismissed?

Sorry for the late reply.

Not a new panel, the same panel stay in the same position but dismiss animation shows.
Yes after button press
deinit not get called. Memory grows.

Thank you and again many apologies!

I think found the issue.
This would be needed.
self.panel?.removeFromParentViewController()
self.panel?.view.removeFromSuperview()

Ofc it's been called in
self.panel?.removeContainer()
But the second time it doesn't get called the reason is self.panel is nil
self.panel = nil moving inside the closure may fix the issue?
UIView.animate(withDuration: configuration.dismissAnimationDuration, animations: {
            panelView.frame.origin = CGPoint(x: 0, y: self.containerView!.frame.size.height)
        }) { _ in
            self.panel?.removeContainer()
            completion?()
            self.panel = nil // moving here may fix the issue
        }

@deshan  I have a question you said

Yes I am creating a new panel. but dismiss animation shows, but prev panel stays

how many panels are you presenting? If you create more than one, then you wouldn't have control over it and it will stay on the screen. You should dismiss the first panel before present the new one.


Use GitHub release as download link

Description of your changes
Uses GitHub release as download link
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
make build

Thanks @turkenh


Upgrade Byte Buddy to 1.11.6

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Build Cause: Pull request #1896 opened

Start Time: 2021-07-04T06:08:43.389+0000

Duration: 57 min 18 sec

Commit: 8e8b26b3074b96a7bfd250724c96af65683e9b97

Test stats :test_tube:

Test
Results

Failed
0

Passed
2286

Skipped
19

Total
2305

Trends :test_tube:

:green_heart: Flaky test report
Tests succeeded.

Expand to view the summary

Test stats :test_tube:

Test
Results

Failed
0

Passed
2286

Skipped
19

Total
2305


feat: Add method to remove MediaObjects with specific collectionId

What kind of change does this PR introduce?
This PR adds a clearMediaObjectCollection method to the PeripheralDeviceAPI. This method allows removing documents from the MediaObjects collection, where the given collectionId and the studioId of the device match.

What is the current behavior?
There is no way to empty the MediaObjects collection other than removing each document explicitly.

What is the new behavior (if this is a feature change)?
The clearMediaObjectCollection removes all Media Objects where the given collectionId and the studioId of the requesting device match.

Other information:
This can be used e.g. by the Playout Gateway to manage MediaObjects used for reporting statuses of Viz elements.

Status

[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

We're going to start up a rather large refactoring of the media-objects in the upcoming weeks.
I see no issues with this PR, so I'll merge this in to make sure this functionality is taken into account during that work.


Select and Unselect a person, displays the related transactions

Implementation details :
Click on a panel, create an event that will be catch by the MainWindow, which will update the Transaction list filtered with the person, it goes through Logic to access to the Model.
Unclick, by using "Ctrl + Clic", has also his own event.
Issues #75  #53

All codes look fine and working well. Maybe instead of delete those irrelevant codes we can actually comment it for future reusability.


Add titles and routes for Job Postings and Job Applications

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Issue #TCES-44
Type of change
Please delete options that are not relevant.

[ ] New feature (non-breaking change which adds functionality)
[ ] This change requires a documentation update

Checklist
Please delete options that are not applicable.

[ ] My code follows the style guidelines of this project (required)
[ ] I have performed a self-review of my code (required)
[ ] I have linted and formatted my code (required)
[ ] New and existing unit tests pass locally with my changes (required)
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings

Closing as duplicate of https://github.com/uoftblueprint/tces/pull/120


auth url incorrect

Hey guys, I'm having some issues trying to run my local environment. The host lookup has an extra dot ( . ) betweeen auth and nhost.
'local.auth..nhost.run'
I'm currenly using the nhost_dart: ^1.0.0
Follow my NhostClient and exception log.
return NhostClient(subdomain: Subdomain(subdomain: 'local', region: ''),
[log] Error trying to login: Failed host lookup: 'local.auth..nhost.run'

Hi,
Are you trying to use it localhost? in this case
for local development, pass 'localhost' or 'localhost:1337' to the subdomain

Solved in #119


Config reading issue with Grav 1.6.26 & Ubuntu 20.04 LTS

I setup a new Ubuntu 20.04 LTS VM through Vagrant and got the issue below. I switched it out to Ubuntu 18.04 LTS and Grav ran just fine. For some reason Grav is having trouble reading any of the config files in 20.04. In the image below, $types is empty on line 1612 (20.04) but when setup on 18.04 it gets populated correctly ([html,htm,xml,txt,json,rss,atom]). Am I missing anything or does anyone have a similar with 20.04?

It's not really an issue with Grav then,  sounds like an issue with PHP.  Grav doesn't access anything directly, everything goes through PHP.  What version of PHP is Grav running on your Ubuntu 20.04 compared to your 18.04 ?

On Ubuntu 20.04 I tried Grav with both PHP 7.3.21 & PHP 7.4.9 and had the same issue. However, Grav ran correctly on Ubuntu 18.04 with PHP 7.4.9. This is all on Apache by the way.


Disable the version check issue

Downloaded Ghost v0.7.8 and tried installing using following command:
GHOST_NODE_VERSION_CHECK=false && npm install --production
It should ignore the Node.js version and install it, but I am getting this output:
ERROR: Unsupported version of Node Ghost needs Node version ~0.10.0 || ~0.12.0 || ^4.2.0 you are using version 5.8.0

Hey @jafarovr I'm pretty sure you have to export the env variable first e.g. export GHOST_NODE_VERSION_CHECK=false && npm install --production.
At any rate, if you have any further issues please visit our Slack channel and someone should be able to help you there.

Haha @acburdine you just beat me to it :wink:
This is actually a documentation error, which I have now fixed - you need to run
GHOST_NODE_VERSION_CHECK=false npm install --production
Without the ampersands.

Thanks for that @ErisDS.
Even though I have GHOST_NODE_VERSION_CHECK set up globally, it does give me same output if I just try npm install --production. But it worked with GHOST_NODE_VERSION_CHECK=false npm install --production - This is what I wanted!


better discord java script error

thank you
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=icon
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=link
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
2018-01-25 11:56 GMT+03:00 Inve1951<EMAIL_ADDRESS>
k, reinstall using zere's fork -> https://github.com/rauenzi/
BetterDiscordApp/releases/tag/0.2.82
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Jiiks/BetterDiscordApp/issues/687#issuecomment-360401922,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiHQsHDBPAiwhdFhdcEf5JjTTCYsP7Dsks5tOEG4gaJpZM4RrUJV
.


DTM-10752 Allow onPageBottom to run when library loaded asynchronously.

Coverage remained the same at 98.227% when pulling 2b5651f80c0db148e5e52b2b10116a9359812c68 on async-page-bottom into de3ed6c632ecb649e22f199ce7d29d903e2693df on master.

Coverage decreased (-0.01%) to 98.217% when pulling 09c8fd4211ea4b73c2a54950d2e7df8cb9ce8e9d on async-page-bottom into de3ed6c632ecb649e22f199ce7d29d903e2693df on master.


docs: various improvements

Lead by example I guess.
Closes https://github.com/neovim/nvim-lspconfig/issues/1228 https://github.com/neovim/nvim-lspconfig/issues/951
There's been a lot of discussion about the quality of documentation in recent neovim plugins. My personal philosophy is that good documentation should have three parts:

A front-and-center README.md which serves as a quickstart/showcase/getting help
A more thoroughly documented document (be it docs/readme.md) which is auto-generated (or hand formatted, like here) into a vimdoc (see :help writing-help)
A wiki with community contributions/plugin integrations, etc.

This PR should bring lspconfig in line with that.

I renamed CONFIG.md to servers.md, it's now accessible directly via the vimdoc with gf.
I removed ADVANCED_README.md, that's now the role taken by lspconfig.txt
removed babelfish :cry: in favor of a hand-written readme. There shouldn't be too much documentation overlap as the purpose of readme and lspconfig.txt is now disjoint
Updated the names (and such)

There are probably typos/suboptimalities. I think it's better than it was, so I'm inclined to merge it regardless. Ofc please do give feedback; "suggest changes" (or PRs to my branch) is useful, a little burned out after the documentation sprint.
I also still hate writing vimdoc and think we should support markdown in core :shrug:

The thing I thought might be "controversial" is CONFIG.md -> servers.md, which I think is more descriptive. Also open to server_configurations.md


[Feature Request] Find

Right now, the only way to find a file/folder online is to pull the whole list with drive list -long -no-prompt and apply a filter with grep for instance; this process can be very long, depending on the number of objects online.
I'm wondering whether it could be possible to implement a find-like feature for drive which would return only the results:
drive find <find-like parameters> with features like:

<standard_wildcard_objects>
-name <name>
-path <path>
-regextype <type> -regex <regex>
-maxdepth <depth>

Hello again @jean-christophe-manciot, great to hear from you! Thank you for the issue. My apologies for the late reply, I've been swamped.
We already have this feature --matches, but it matches prefixes(at least promised by Google Drive) but it also matches matches. It is documented throughout the README not as search but as list for example

https://github.com/odeke-em/drive#listing
https://github.com/odeke-em/drive#fetching-and-pruning-missing-index-files
https://github.com/odeke-em/drive#pulling
https://github.com/odeke-em/drive#touching

and here is a sample to list content matching tes
$ drive list --matches tes
/testx/drone
/test-non-existent/hello
/test-non-existent/p1
/test-non-existent/now
/test-checksums/1.txt
/test-push-output/output.log
/test-push-output/h1.go
/test-push-output/h1.o
/test-push-output/hello-world.o
/test-push-output/hello-world.s
/test-push-output/hello-world.go
/test-push-output/h1.s
/test-push-output/h1
/test1/t2
/test-docs/t1
/test-exports/influx
/test-exports/test-pres
/test-exports/few.docs
/test-exports/few
/test-push-mounted/32px-Android_sample.svg.png
/test-overflow/harry-potter-iggy.mp4
/test-overflow/tfx.mp4
/test-overflow/hpx.mp4
/test-overflow/hp.enc
/test-overflow/hp.mp4
/test-rename/ft.txt
/test-rename/6
/test-rename/AB
/test-rename/3
/test-rename/4
/test-rename/flux
/test-rename/2
/test-symlinks/link.txt
/test-symlinks/test.txt
/testing-filename-report/foo
/testing-errors/foo
/Copy of tests.xls
/tests.xls
/testz/pfx
/testz/flux
/testz/全角スペース　含みます
/testz/1
/test.xlsx
/test.numbers
/test.xls
/TestMap
/test/a1
/test/b
/test/a
/test/1.txt
/test/paintball.mp4
/test/Snapchat--5080765389187639557.mp4
/test/Snapchat-6259724940292983060.mp4
/test/2.txt
/test/Emmanuel Odeke interview
/testing/list
/testing/FSM
/testing/today.now
/testing/lang
/testing/gm
/testing/tester.txt
/test_doc.docx
/test_doc.odt
/testing_docs/test_doc.docx
/test_doc
/test.mp4x
/test.txt

OK, except that:

its scope is very limited: only the first characters are matched; no regex/wildcards seem to be accepted
most examples given on https://github.com/odeke-em/drive cannot work as implied (files extensions cannot be matched)

Examples:
Assuming the following:
# drive list "Google Photos" | sort
/Google Photos/002-Modulation-signal-Spectre-ondes-radios-micro-ondes.jpg
...

Only the first characters can be matched:
# drive list --matches Google | sort
/Google Photos/002-Modulation-signal-Spectre-ondes-radios-micro-ondes.jpg
# drive list --matches "Google.*" | sort
/Google Photos/002-Modulation-signal-Spectre-ondes-radios-micro-ondes.jpg
...

Many other searches fail:
# drive list --matches jpg
no matches found!
# drive list --matches .*\.jpg
no matches found!
# drive list --matches ".*\.jpg"
no matches found!
# drive list --matches 002
no matches found!


Pin llvm-openmp version for host, not build

We are getting some strange requirements in the final package; lvm-openmp appears twice, with minimum versions 15 _and_with minimum version 16.
That's no good for assuring compatibility with Mantid which is pinned to version 15. See if setting in host helps?

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


Unable to approve (own) Translation

I just authored a PHP Translation to my newly authored Kata (still in Beta) which was originally written in JavaScript.  However, when I proceeded to approve of my own translation straight after publishing it, no matter how many times I click "I'm sure", nothing happens as if I didn't press any buttons at all:

I publish a Translation to my own Kata, still in Beta:

I click on "Approve" and confirm my decision by clicking on "I'm sure":

Instead of (re-)loading the page and approving the Translation, the box simply disappears as if nothing happened in the first place:

At first, I suspected that this may have been due to recent updates which prevent the original author from approving his/her own translations but seeing as there is a bug in approving Kata ready to get out of Beta, I wonder if these two issues are inexplicably linked to one another?

Nevermind, it finally went through after many tries.


Adjust tailwind content globs

Tailwind content is not going to appear in web_src/css, web_src/fomantic or web_src/svg or the JSON templates, so we don't need to have tailwind scan these directories which will speed up the build.

Actually there will be some tw- usages in go code ..... like gt-object-contain

Yes, we need to include them, but so far there have been none of the used ones.


[External] Documentation Improvements

Issue #, if available: Issues 1108, 1109
Description of changes: Elaborated Azure AD integration instructions, AMI Install steps, and updated documentation links to github.io pages
Checklist:

[x] Have you successfully deployed to an AWS account with your changes?
[x] Is this change also required on the AWS Solution version?

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The PR is related to these issues: https://github.com/awslabs/service-workbench-on-aws/issues/1108 and https://github.com/awslabs/service-workbench-on-aws/issues/1109

Currently awaiting changes requested in PR feedback.

Thanks for the feedback. I've made the changes suggested. Please review and merge.


Updating Title to match the tutorial

The tutorial is for sending emails when adding an item to a SharePoint List. However,  the title says it's when an Item is Created or Modified. This is not correct. I have updated to reflect this. Fixed #6634
Category

[x] Content fix

Related issues

fixes #6634

What's in this Pull Request?
Updating text to better reflect the tutorial which is adding a new item to a list. Not for modifying an item in a list.

Docs Build status updates of commit 8cdfd10:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/business-apps/power-automate/get-started/create-your-first-flow.md
:bulb:Suggestion
View
Details

docs/business-apps/power-automate/get-started/create-your-first-flow.md

Line 64, Column 5: [Suggestion-duplicate-alt-text] Alt text 'Flow designer - send email when an item is added in a list' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 68, Column 5: [Suggestion-duplicate-alt-text] Alt text 'Flow designer - send email when an item is added in a list' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 121, Column 5: [Suggestion-duplicate-alt-text] Alt text 'Flow designer - add owner dynamic content' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 127, Column 5: [Suggestion-duplicate-alt-text] Alt text 'Flow designer - add owner dynamic content' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 8cdfd10:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/business-apps/power-automate/get-started/create-your-first-flow.md
:bulb:Suggestion
View
Details

docs/business-apps/power-automate/get-started/create-your-first-flow.md

Line 64, Column 5: [Suggestion-duplicate-alt-text] Alt text 'Flow designer - send email when an item is added in a list' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 68, Column 5: [Suggestion-duplicate-alt-text] Alt text 'Flow designer - send email when an item is added in a list' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 121, Column 5: [Suggestion-duplicate-alt-text] Alt text 'Flow designer - add owner dynamic content' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.
Line 127, Column 5: [Suggestion-duplicate-alt-text] Alt text 'Flow designer - add owner dynamic content' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning on 1/29/21.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit b5eb56d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/business-apps/power-automate/get-started/create-your-first-flow.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


font.save() fails when trying to overwrite a loaded font

Take a look at this simple script:
from fontTools import ttLib

fontFilePath = 'MyFont.otf'
font = ttLib.TTFont(fontFilePath)

font.save(fontFilePath[:-4] + '-New.otf') # this works
font.save(fontFilePath) # XXX this produces the traceback below

Traceback (most recent call last):
  File "save.py", line 8, in <module>
    font.save(fontFilePath)
  File "/fonttools/Lib/fontTools/ttLib/__init__.py", line 214, in save
    self._writeTable(tag, writer, done)
  File "/fonttools/Lib/fontTools/ttLib/__init__.py", line 626, in _writeTable
    self._writeTable(masterTable, writer, done)
  File "/fonttools/Lib/fontTools/ttLib/__init__.py", line 629, in _writeTable
    tabledata = self.getTableData(tag)
  File "/fonttools/Lib/fontTools/ttLib/__init__.py", line 646, in getTableData
    return self.reader[tag]
  File "/fonttools/Lib/fontTools/ttLib/sfnt.py", line 92, in __getitem__
    data = entry.loadData (self.file)
  File "/fonttools/Lib/fontTools/ttLib/sfnt.py", line 373, in loadData
    assert len(data) == self.length
AssertionError

Am I doing something wrong?

currently it's not possible to overwrite a TTFont, because the reader's file is already open for reading, and can't be also written from the writer at the same time.
You can work around this by first making a temporary copy of the input file and then using the latter when initializing a TTFont object.
This way you can save to the original input file, since now the reader's file is different from the writer's.
There are many ways to do that. Here's one:
import tempfile
import shutil

tf = tempfile.TemporaryFile()
with open(path, 'rb') as f:
    shutil.copyfileobj(f, tf)
tf.seek(0)

font = TTFont(tf)

Alright, thanks for confirming and for providing a workaround.

if there's demand for it, maybe we could do something like this by default upon loading a TTFont? /cc @behdad

This is definitely in the right direction.  Alternatively we can just right the font into memory and wrap a stream around it.  I think I like that even better.
Note that the lazy parameter to TTFont() helps a bit with this, but I never got to implement non-lazy in sfnt.py.

I did that in PR #308.
I read the input file's content into a StringIO stream, which is then passed to the SFNTReader instead of the original file. I also keep a reference to the file's name attribute, in case one wants to know the name of the original file.

Thanks.  The name is indeed used by the bitmaps/ code that you fixed recently.

Right now we do a lot of conversions between file-like object and actual data.  Over time we can change all to keep reference to byte() only.

Another nice thing about this change is that we can now load font from unseekable streams.

Cool!
I'm leaving the input file open in case it's already a file object instead of a path string.

Nice! Thanks guys.

This is great.


Fix windows insiders icons

Fixes https://github.com/microsoft/azuredatastudio/issues/13452

Pull Request Test Coverage Report for Build 392910804

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 41.259%

Totals

Change from base Build 392847099:
0.001%

Covered Lines:
21858

Relevant Lines:
48619

💛  - Coveralls

Test build looks good, still need to update distro for stable build though.


Test correct geocoding for street numbers

OSM Nominatim had issues with street numbers in Austria. Therefor we changed the geocoder to Google Maps. There is a link to Google Maps app in the incident overview now, so we use "Google Maps API with a Google Map" according to Google's license.


Avoid crashing the server process when exiting interactive sessions

When using the interactive socket connection (socat - UNIX-CONNECT:/tmp/edumeet-server.sock), closing the command line session throws a not handled exception that crashes the server process.

Maybe this closes #331

Great catch!
Don't forget to add it to changelog later..

Maybe this close also #640


Error in 6.4.9 with calling JQuery add class.

 Before you open an issue please review the contributing guideline.
Please fill the issue template correctly and:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with isssue

It will save a lot of time for other contributors to check your issue and fix it.

Your issue is closed because it is not filled correctly. Before you open an issue please review the contributing guideline.
Please fill the issue template correctly and:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with isssue

It will save a lot of time for other contributors to check your issue and fix it.

the same issue in 6.4.9, on 6.4.8 - ok

the same issue in 6.4.9, on 6.4.8 - ok


How to obtain the ground truth scene coordinates?

Hi Eric,
I've installed esac extension and set up 7scenes dataset. Then in /esac/environments/7scenes I run
python ../../code/init_gating.py

The error pops up
Traceback (most recent call last):
  File "../../code/init_gating.py", line 36, in <module>
    trainset = RoomDataset("training")
  File "/esac/code/room_dataset.py", line 86, in __init__
    cur_init = os.listdir(scene + '/' + init_dir)
FileNotFoundError: [Errno 2] No such file or directory: '../../datasets/7scenes_chess/training/init/'

It seems that the ground truth scene coordinates are not set up with the dataset.
From the error log I guess it is because the dataset link for that part is down. (Same for the links of the pre-trained models). When I visit that link from a browser, it shows
Internal Server Error

The server encountered an internal error and was unable to complete your request.
Please contact the server administrator if this error reappears multiple times, please include the technical details below in your report.
More details can be found in the server log.

Could you take a look? Thanks!
Best regards,
Qianhao

Hi Qianhao,
it seems the cloud space of the TU Dresden is offline at the moment. It is not just my links, but the whole system. I can do nothing about that at the moment. Let's hope they fix it soon.

Hi @ebrach ,
thanks for the reply! The server has been back online now. I guess it just took a weekend off :)


SqlAzureDacpacDeployment - Get-AgentIPRange fails when password contains "|"

We have an issue with task /tree/master/Tasks/SqlAzureDacpacDeployment
More specific in file DeploySqlAzure.ps1, line 131.
When putting the firewall rules setting to "AutoDetect" it executes
$ipAddressRange = Get-AgentIPRange -serverName $ServerName -sqlUsername $SqlUsername -sqlPassword $SqlPassword
$sqlPassword = firstPart|secondPart
We have a password which contains a "|"
This cmdlet fails, because it interprets the "|" symbol in the password as a pipe, error:
The term 'secondPart' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

Hi @wimmeerschman , this is a known issue. We have applied a fix for this, and the updated task will be available in the next couple of days

Fix is rolling out in phases to all the accounts, it should reach with in couple of weeks in task version 1.1.15.  Please feel free to reopen if the issue is not addressed in that version


Fix the check for institution for UpdateIdentityCommand

depends on https://github.com/SURFnet/Stepup-Middleware-clientbundle/pull/32

Pending CI, :+1:


Upgrade Jersey related libraries

Jersey: 2.39 -> 2.42
Javassist: 3.19.0-GA -> 3.30.2-GA
Swagger: 1.6.9 -> 1.6.14
Swagger-ui: 5.1.0 -> 5.13.0

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

Project coverage is 45.98%. Comparing base (59551e4) to head (3c9e9a8).
Report is 192 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12772       +/-   ##
=============================================
- Coverage     61.75%   45.98%   -15.77%
- Complexity      207      909      +702
=============================================
  Files          2436     1858      -578
  Lines        133233    98119    -35114
  Branches      20636    15814     -4822
=============================================
- Hits          82274    45122    -37152
- Misses        44911    49659     +4748
+ Partials       6048     3338     -2710

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

integration2
?

java-11
?

java-21
45.98% <ø> (-15.64%)
:arrow_down:

skip-bytebuffers-false
45.98% <ø> (-15.77%)
:arrow_down:

skip-bytebuffers-true
?

temurin
45.98% <ø> (-15.77%)
:arrow_down:

unittests
45.98% <ø> (-15.77%)
:arrow_down:

unittests1
45.98% <ø> (-0.91%)
:arrow_down:

unittests2
?

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

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


Moving the Loggable interface file outside of enclosing package causes bug

While refactoring some inter-package code, I tried to move the org.mql.plateforme.stage.aspect.Loggable interface to the org.mql.plateforme.stage.util package, & I get this warning, indicating that the application context initialization does not happen:

WARNING: Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.springframework.context.event.internalEventListenerProcessor': Initialization of bean failed; nested exception is java.lang.IllegalArgumentException: error Type referred to is not an annotation type: Loggable

The Loggable annotation has been dropped off. Closing.


[ISSUE] Album empty, though it should have pictures

After completing the run via npx-Version twice, I find the following confusing thing happens:
There are many album-folders that do not contain any pictures. Neither in "output" nor in "error", the folders are empty. Although those Albums in GooglePhotos contain pictures and I made a full Takeout of all folders via GoogleTakeout...?
Additional Info: The Pics themselves are there in the Takeout-folder, e.g.
P1060024.jpg and P1060024.jpg.json

I re-checked and found that 34(!!) of 287 Album-folders generated in the output-Folder are empty!😱
Although all those pics and according jsons have been in the takeout and are also in output/Photos. What is going wrong here, what can I do?

Hm, hard to debug if I can't reproduce it.
What happens if you run the flat migration on one of the failed albums?

Thanks for the reply. I see now, that one of those folder in question already have only json-files in it, no jpg. Maybe Google did something wrong in creating the takeout?

The pics themselves are there, in Takeout, in other Albums and also they are in the "Photos"-folder of the full migration, but not in the migrated albums...?

I think I found a solution. If I make a new takeout of only the albums that have been empty after migration, then they include the jpgs. So I will migrate and add them like this one by one I guess.


Points.formula

got the other PR in here because I am bad :tophat:
This works:
library(gsplot)
plot(-4:4, -4:4, type = "n")
points(y~x, data=list(x=-3:3,y=-3:3))
# or
plot(-4:4, -4:4)
points(y~x)

fails for this because it can't find limits:
gs <- gsplot() %>%
    points(y~x, data=list(x=-3:3,y=-3:3))
gs

but this works with a warning triggered by the same thing that makes the above fail:
gs <- gsplot() %>%
    points(y~x, data=list(x=-3:3,y=-3:3), ylim=c(-3,3),xlim=c(-3,3))
gs

The thing that fails is in calc_views, set_view_lim(views), because it doesn't know to dive deeper into the list to get x and y. That might be a relatively simple fix.

Now this should work for
gs <- gsplot() %>%
    points(y~x, data=list(x=-3:3,y=-3:3))
gs


Time2Internaldate is not working with int or float time

time.time() or other int or float value throws an exception when it is using as date_time input for Time2Internaldate function.
The error message is:
UnboundLocalError: cannot access local variable 'dt' where it is not associated with a value
if isinstance(date_time, (int, float)): #time.time() returns True
tt = time.localtime(date_time)
and after that
fmt = '"%d-{}-%Y %H:%M:%S %z"'.format(MonthNames[dt.month])
'dt' cannot be used.
Time2Internaldate works correctly by the following way:
dt = datetime.datetime.now(pytz.timezone('Europe/Sofia'))
imaplib2.Time2Internaldate(dt)

Thanks. I haven't taken a deeper look into it, but I think it can be fixed by changing the following line: https://github.com/jazzband/imaplib2/blob/615419b705ba46f8326e5c38cfe48170501c39ef/imaplib2/imaplib2.py#L2388
to dt = datetime.fromtimestamp(time.mktime(time.localtime(date_time))).
But again, I haven't had time to actually test it.


Escaped unicode characters are incorrectly decoded

In strings with espaced Unicode characters (i.e. "\uABCD"), characters in the range from D800 to DFFF always get encoded as FFFD (65533 decimal).
Here's from the F# Interactive:
    > "\uD500".[0] |> uint16 ;;
    val it : uint16 = 54528us
    > "\uD700".[0] |> uint16 ;;
    val it : uint16 = 55040us
    > "\uD800".[0] |> uint16 ;;
    val it : uint16 = 65533us
    > "\uD900".[0] |> uint16 ;;
    val it : uint16 = 65533us
    > "\uE000".[0] |> uint16 ;;
    val it : uint16 = 57344us

Here's an SO question that prompted me to fiddle with it: http://stackoverflow.com/questions/29359408/surrogate-pair-detection-fails

Hi @fsoikin  -
Could you open this bug at http://github.com/Microsoft/visualfsharp?  If it gets fixed there first we'll then integrate it into this edition.
Thanks, Don

@dsyme done: https://github.com/Microsoft/visualfsharp/issues/338

Closing here as the discussion and fix is underway at https://github.com/Microsoft/visualfsharp/issues/338


431 (Request Header Fields Too Large)

Bug description
Captcha shows up and is solvable, but when completed the following status code is shown.
Screenshots / Logs

Last log line on the docker side is "Action requested. Waiting for Captcha to be solved" (and the relevant user/id info)
Configuration:

Browser: Any (Tried Chrome, Firefox, Edge)
Hosting solution [e.g. reverse proxy, HTTPS/HTTP, caching, localhost]: Plain Local HTTP (No reverse proxy)

I imagine something wacky is happening with the cookies. Can you switch to the applications tab in devtools and find the list of cookies for the site? I'd expect around 12 to be there. A screenshot of at least the name of each cookie would help me diagnose what's causing it.

I imagine something wacky is happening with the cookies. Can you switch to the applications tab in devtools and find the list of cookies for the site? I'd expect around 12 to be there. A screenshot of at least the name of each cookie would help me diagnose what's causing it.

The blurred domains are only mine (local) and there are no epic game ones.
This screenshot is taken after solving the captcha.

The blurred domains are only mine (local) and there are no epic game ones.
This screenshot is taken after solving the captcha.

The EPIC_SESSION_PAYMENT_INSTANCE cookie is new, and fairly large. The max header size is 8K and it uses 3K lol. I should probably try filtering the cookies by only what's necessary.
As a temporary solution, you may be able to get one solve in if you clear the cookies.

The EPIC_SESSION_PAYMENT_INSTANCE cookie is new, and fairly large. The max header size is 8K and it uses 3K lol. I should probably try filtering the cookies by only what's necessary.
As a temporary solution, you may be able to get one solve in if you clear the cookies.

This is on a fresh install of chrome with no cookies, it looks like the cookie gets created when solving the captcha, so clearing it doesn't really work sadly.

This is on a fresh install of chrome with no cookies, it looks like the cookie gets created when solving the captcha, so clearing it doesn't really work sadly.

K, I'll work on a fix

K, I'll work on a fix


Postal addresses support added

In the future please add a more descriptive git commit message. It seems this PR adds Address support to both platforms. This is excellent.
[1] http://chris.beams.io/posts/git-commit/

closing as part of https://github.com/rt2zz/react-native-contacts/pull/118


hide Metadata assert insides opendal's own tests features

          OpenDAL has an assertion that to prevent users to access metadata that not been set. For example, visting etag from the metadata returned by list.

I feel the assertion from opendal is too aggressive. Perhaps we could leave those fields as None if they're not set.
We can hide those assert insides opendal's own tests features, so they won't be exposed to user side.
https://github.com/apache/opendal/blob/9d587ea7251ecf6e29e1be3e7bfa118655e8bf79/core/src/types/metadata.rs#L140-L144
Originally posted by @Xuanwo in https://github.com/apache/opendal/issues/4581#issuecomment-2098897841

Hi, I'm willing to work on this issue to look into the core and tests. May I give it a try?

Hi, I'm willing to work on this issue to look into the core and tests. May I give it a try?

Welcome, thanks in advance!

Metadata::content_length returns u64, maybe we should change it to Option<u64> to distinguish between zero and None and in consistent with other public apis?

Metadata::content_length returns u64, maybe we should change it to Option<u64> to distinguish between zero and None and in consistent with other public apis?

content_length is special that we require all services to return it.

Metadata::content_length returns u64, maybe we should change it to Option<u64> to distinguish between zero and None and in consistent with other public apis?

content_length is special that we require all services to return it.

This is true for stat operation, but for metadata returned by list operation its content_length could be None. Should we keep it returning zero for None after removing assertions?


feat: Extended output for deployment

Summary
Extends the output for deployed resources to include the deployment name and config url, if available.
Checklist

[ ] Documentation updated
[x] Unit tests updated
[ ] Integration tests updated
[ ] User Acceptance Tests updated
[ ] Deployer version updated

Deployment Configuration for Testing
https://raw.githubusercontent.com/newrelic/open-install-library/main/test/manual/definitions/infra-agent/debian10-infra.json

We can circle back to the GCP/Azure tests.  These started failing a few weeks back seemingly because of something within the accounts.


Make a pip package

pip package is created.

Find a way to automatically install pycocotools module from https://github.com/cocodataset/cocoapi, so running pp_merge_to_panoptic goes smoothly.

When installing on Windows pip install pycocotools requires MSCV compiler, which makes installation cumbersome as build tools are not by default installed in Windows.
Does it require extra dependencies in a "fresh" linux?
Maybe consider moving it to optional.txt again? (as it is used only on specific scripts) @vincentwen1995 @DdeGeus

Hi Panos, I think it's because pycocotools has some C/C++ dependency that needs to be built.
I only tested it out with gcc on my wsl environment.
Indeed if it is not a pure python package and either gcc/msvc compiler is needed, we should make it optional or note it down in Readme.

From: Panagiotis Meletis @.>
Sent: Monday, February 14, 2022 12:58:01 PM
To: pmeletis/panoptic_parts @.>
Cc: vincentwen1995 @.>; Mention @.>
Subject: Re: [pmeletis/panoptic_parts] Improve pip package (#2)
When installing on Windows pip install pycocotools requires MSCV compiler, which makes installation cumbersome as build tools are not by default installed in Windows.
Does it require extra dependencies in a "fresh" linux?
Maybe consider moving it to optional.txt again? (as it is used only on specific scripts) @vincentwen1995https://github.com/vincentwen1995 @DdeGeushttps://github.com/DdeGeus
—
Reply to this email directly, view it on GitHubhttps://github.com/pmeletis/panoptic_parts/issues/2#issuecomment-1038999207, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHB22GKBKG66KMYIMXZLG6TU3DU4TANCNFSM4M7V4QAQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

ok, it's included now.


use default constructor if more than one found and none have injectors

(Related to #7)
If Zenject finds more than one constructor, and none are explicit, it fails by default.
ZenjectException: Assert Hit! Found null pointer when value was expected. More than one (or zero) constructors found for type 'WWTK.Persistence.DataPersistence' when creating dependencies.  Use one [Inject] attribute to specify which to use.
ModestTree.Assert.Throw (System.String message) (at Assets/UPackages/Zenject-WithSampleGames~5.2.2/Internal/Assert.cs:269)
ModestTree.Assert.IsNotNull (System.Object val, System.String message, System.Object[] parameters) (at Assets/UPackages/Zenject-WithSampleGames~5.2.2/Internal/Assert.cs:200)
Zenject.DiContainer.InstantiateInternal (System.Type concreteType, Boolean autoInject, Zenject.InjectArgs args) (at Assets/UPackages/Zenject-WithSampleGames~5.2.2/Main/DiContainer.cs:815)
Zenject.DiContainer.InstantiateExplicit (System.Type concreteType, Boolean autoInject, Zenject.InjectArgs args) (at Assets/UPackages/Zenject-WithSampleGames~5.2.2/Main/DiContainer.cs:2178)

https://github.com/modesttree/Zenject/blob/0337b58/UnityProject/Assets/Plugins/Zenject/Source/Util/TypeAnalyzer.cs#L280-L287
Yet if there's a default constructor, Zenject could select it.
In our use case, we would like to inject an instance of an existing class with existing constructors with arguments. This will obviously not work. We decided to modify our API to be more injection friendly and add a default constructor without arguments and explicit setters that we can call in our MonoInstaller instance. We would also like that library to stay Zenject free, so adding the [Inject] attribute is not an option.
But Zenject will not default to the new argument-less constructor and instead fail.
Falling back on the default argument-less constructor, the only one usable from Zenject, would allow us to stay backwards compatible.

I'll see if I can address #7, I think it should be doable and that would be a better fix

Although even in that case you have to choose a strategy which might not be obvious necessarily.  So something simple like 'choose the constructor with the least parameters when [Inject] attribute is missing' might be preferable to throwing an exception for now.  Even though that would allow you to accidentally add constructors and get less obvious null ref exceptions or something instead

Rather than trying to choose a constructor would it be possible to add an explicit binding to a particular constructor through the Zenject binding DSL similar to how Guice works? Guice also selects a constructor that is not annotated with Inject if there is only one constructor, but if there are multiple it allows using toConstructor(java.lang.reflect.Constructor<S>) [1].
A quick Google search indicates that C# reflection also supports references to constructors [2].
The advantage of doing it this way is that it's explicit and allows using Zenject with classes from libraries that are out of the control (to change) for the developer writing the application.
This might look something like:
Container.Bind<Foo>()
    .ToConstructor(typeof(Foo).GetConstructor(new Type[] { typeof(string) }));

Container.Bind<IFoo>()
    .To<Foo>()
    .WithConstructor(typeof(Foo).GetConstructor(new Type[] { typeof(string) }));

// or (by abstracting the call to `ToConstructor`)

Container.Bind<Foo>()
    .WithConstructorParameters(typeof(string), typeof(float));

Container.Bind<IFoo>()
    .To<Foo>()
    .WithConstructorParameterss(typeof(string), typeof(float));

[1] https://google.github.io/guice/api-docs/latest/javadoc/com/google/inject/binder/LinkedBindingBuilder.html#toConstructor-java.lang.reflect.Constructor-
[2] https://msdn.microsoft.com/en-us/library/system.type.getconstructor.aspx

Ok I decided to fallback on using the constructor with the least amount of arguments when forced to make a choice.  I think this makes sense since it allows usage with libraries that don't depend on zenject even though it could result in more subtle error behaviour in some cases.
This is committed to develop branch and will be in the next version
@johnlcox I moved your other suggestion to its own issue #417


Remove repository

Is your feature request related to a problem? Please describe.
WIth current version, it's not possible to remove a repository. It would be nice to be able to.
Describe the solution you'd like
2 options:

Only remove if no dashboard use it
Remove all widgets referecing to this repo in dashboard

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
If possible provide visual sample and code impacts
NB: Specify label(s) in right panel to ease requests organization

+1, implementing the deletion of repositories would be great.
Possible scenario: when clicking on the delete button, display a warning window asking confirmation from the user if they really want to delete the repo, as well as all the related widgets (maybe show the list of all the related widgets currently added in dashboards so the user can check directly from there before confirming?)
Besides, we have this "Enable this repository ?" checkbox, which is misleading because it let users think they can enable/disable a repo, while it actually corresponds to the auto reload of widgets at startup
This checkbox should be reworded to be more clear about what it is really doing.
And maybe we can implement a real enable/disable feature, if useful?


Any way to overwrite existing fields on import?

Would be nice to be able to import existing fields, and have that only update the field metadata, and not affect any entries that may be using these fields (unless the fields go away). If i try to import fields that already exist, it tells me the handle name exists and won't import.
I hope this makes sense.

So you'd like to be able to overwrite settings for the field? For instance - a text field has things like placeholder, size, etc. Where it gets tricky is things like element fields which have a limit. What if you change the field limit to 1, but your entries have more than one element (think Assets)? These will be removed next time you edit the entry, which might not be what you want. I'm sure there are other cases too, but its something to consider.

Even with this caveat, I think this is a great idea. ACF Pro for WordPress (hiss) does just this, albeit through its regularly generated JSON files that can be pushed up from local in order to keep fields in sync without having to rely on editing on production first and then distributing on every other server.


rename-apps broke backend for me

After 155 backend doesn't start for me:
nextbackenddocker_1  | [2016-12-12 21:04:02 +0000] [12] [ERROR] Exception in worker process
nextbackenddocker_1  | Traceback (most recent call last):
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/gunicorn/arbiter.py", line 557, in spawn_worker
nextbackenddocker_1  |     worker.init_process()
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/gunicorn/workers/ggevent.py", line 190, in init_process
nextbackenddocker_1  |     super(GeventWorker, self).init_process()
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/gunicorn/workers/base.py", line 126, in init_process
nextbackenddocker_1  |     self.load_wsgi()
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/gunicorn/workers/base.py", line 136, in load_wsgi
nextbackenddocker_1  |     self.wsgi = self.app.wsgi()
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/gunicorn/app/base.py", line 67, in wsgi
nextbackenddocker_1  |     self.callable = self.load()
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/gunicorn/app/wsgiapp.py", line 65, in load
nextbackenddocker_1  |     return self.load_wsgiapp()
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/gunicorn/app/wsgiapp.py", line 52, in load_wsgiapp
nextbackenddocker_1  |     return util.import_app(self.app_uri)
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/gunicorn/util.py", line 357, in import_app
nextbackenddocker_1  |     __import__(module)
nextbackenddocker_1  |   File "/next_backend/next/api/api.py", line 12, in <module>
nextbackenddocker_1  |     app.register_blueprint(home, url_prefix='/home')
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 65, in wrapper_func
nextbackenddocker_1  |     return f(self, *args, **kwargs)
nextbackenddocker_1  |   File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 958, in register_blueprint
nextbackenddocker_1  |     if blueprint.name in self.blueprints:
nextbackenddocker_1  | AttributeError: 'module' object has no attribute 'name'

Found the issue... this should be from next.home.home import home

I knew something would sneak past. Can you make a pull request for this?

https://github.com/nextml/NEXT/pull/158

Fixed.  It was a .pyc issue on my end.


KAFKA-3452: follow-up -- introduce SesssionWindows

TimeWindows represent half-open time intervals while SessionWindows represent closed time intervals

@dguy @guozhangwang @enothereska
This is a follow up to Session Windows. Because TimeWindow represent half-open time interval windows (ie, [start,end)) but sessions are defined a closed time intervals (ie, [start,end]) we should not use TimeWindow to represent a session window. Thus, we introduce SessionWindow with this update.
cf #2337 that revealed this problem
Potential issues: this PR changes abstract class Window that is strictly speaking public API.

Updated. Test for TimeWindow and UnlimitedWindow overlap() will be covered by #2337 after this got merged.

@mjsax - looks like there is a checkstyle failure

@dguy @guozhangwang @enothereska  Updated.

Failing tests are unrelated.

retest this please

LGTM and merged to trunk.


Support for node.js 20+?

Hi there,
Just curious if there are any plans to upgrade this package for use in node.js 20+.  I was using it successfully in an older version of node but came across issues with it once upgrading.
Thanks so much,
S

@sammertime sorry I didn't have time to get back to you sooner. I haven't tried running node.js with the package, what is the issue you're running into?

@clhenrick no worries at all.  I ended up moving forward with a built in leaflet solution.  The issue I was encountering occurred when I upgraded my node (20) and npm packages.  It complained about the color-legend-element package being on an unsupported version of node js I believe.  I was using parcel to develop locally and wasn't able to perform a code build when including the color-legend-element package.  All the best.

@sammertime it's probably because I have "engines" in the package.json set to a specific version of node. I think if I change this to allow that version of node and any greater version than it shouldn't throw this error.
However, it looks like this is only enforced when the engine-strict config flag is set to true, otherwise it will just produce a warning: https://docs.npmjs.com/cli/v10/configuring-npm/package-json#engines
You might try removing that config flag from your .npmrc file if you have it set and see if that changes things.

Reopening for now in case anyone else runs into this issue

@sammertime I just published a new version (1.0.5), if you have a moment let me know if that fixes your issue. Thanks!


Accessing File Tree with Source Codes as a JSON

Is your feature request related to a problem?
I need the full code base including the user changes so that we can evaluate the code and get back to him with a report.
Describe the solution you'd like.
If I get the entire file tree and user code changes as JSON then It would be easier to evaluate.
Describe alternatives you've considered.
Not Applicable.
Additional context
No response

There are couple of options:

Use the "Open In StackBlitz" button on top of the TutorialKit, and use those to "store" user's results. So basically when you click this ⚡ button, you get link to stackblitz.com with the current state of the editor. It includes all lesson and template files, including the modifications that user did.

Create your own custom component that calls TutorialKit's public APIs to get current state of the file system + editor. This is the same data as in option #1, but here you can process the JSON anyway you want. Here is example where we simply print the JSON on the page:

https://stackblitz.com/~/edit/tutorialkit-store-snapshot?file=src/components/SnapshotButton.tsx&file=src/content/tutorial/1-basics/1-introduction/1-welcome/content.mdx
import tutorialStore from 'tutorialkit:store';

// Save current state of the file system
const snapshot = tutorialStore.takeSnapshot();

Does this help?

Thanks. Worked

Solved, I have another question

Can I hide this button ? Without CSS ?
Basically I was looking for full control over the system
I need to add Custom Button Over there, is it possible ?

Can I hide this button ? Without CSS ?

At the moment we don't support hiding the Solve/Reset -button. Feel free to open feature request for this.

Solved

Great, let's close this issue then.


LoRA annotated

Hi there!
It's not an issue, rather a question. Some time ago I wanted to discover how LoRA works under the hood and came across this repo. Though the whole code was not clear for me at first even after reading the LoRA paper one more time.
So I annotated LoRA code in my repo. Perhaps some comments are redundant yet I think it might be helpful for someone who also wants to learn the code.
So the question is: whether this type of changes are welcomed in this repo or not?

I think you did a great job with the comments in the code. Bringing these annotations to lit-llama would be welcomed from my end. @lantiga do you agree?

Thanks guys.
I'll prepare a PR in a day or two and then we will see how it goes 😉:

Ideally we should reach the same level of accessibility throughout the repo.

perhaps it will be a task for me.

Fantastic! 🚀


how to debug "out of memory"

I just saw this exception when trying out the new (20200913) roflcoopter/verison-latest:
[2020-09-13 16:32:41] [lib.motion.cam04        ] [DEBUG   ] - Motion frames: 1, area: 1363.5
Exception in thread Thread-46:
Traceback (most recent call last):
  File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.6/threading.py", line 864, in run
    self._target(*self._args, **self._kwargs)
  File "/src/viseron/lib/camera.py", line 213, in capture_pipe
    pipe = self.pipe(stderr=True, single_frame=True)
  File "/src/viseron/lib/camera.py", line 190, in pipe
    bufsize=10 ** 8,
  File "/usr/lib/python3.6/subprocess.py", line 729, in __init__
    restore_signals, start_new_session)
  File "/usr/lib/python3.6/subprocess.py", line 1295, in _execute_child
    restore_signals, start_new_session, preexec_fn)
OSError: [Errno 12] Cannot allocate memory

I didn't change anything from the earlier version other than to make the default logging info and motion detection level debug.

This is a tough one that i dont think i can help with.
This simply means your machine didnt have enough free memory, it could be Viseron or it could be some other process using up all your resources.
Viseron in itself uses quite a lot of memory, especially for the object detector but also the lookback feature. If you have a lot of cameras with high resolution and long lookback configured you will get even higher memory usage.
In the future i will try to transition to using ffmpeg segments instead of the storing frames in memory for the lookback. I havent quite wrapped my head around how to do this yet tho

Perhaps a data point for the books. Memory measured with top output, sorted by memory (M key):

six 1080p cameras, motion_detect interval: 0.2 (5fps), recorder lookback: 15: Resident memory is 9.8GB
six 1080p cameras, motion_detect interval: 0.5 (2fps), recorder lookback: 15: Resident memory is 7.1GB
six 1080p cameras, motion_detect interval: 1.0 (1fps), recorder lookback: 15: Resident memory is 6.2GB

Okay that is very interesting. Off the top of my head, interval should have nothing to do with the memory usage.
I have to look into this more. Reopening.
I also think i need to prioritize the use of ffmpeg segments instead of storing frames in memory, because memory usage like this could be problematic.

I was quite surprised by the memory usage myself, but I also understand that this is a pretty data-intense problem space. Lots of images hanging around, lots of decoding and resizing and filtering and god knows what else OpenCV has to have memory for so it can do its stuff.
As a comparison, the same cameras being monitored/motion-and-object detected/recorded by SecuritySpy at native camera speed (20fps, I actually reconfigured the cameras for a lower FPS due to this) seems to be about 1.5GB, but the VTDecoderXPC processes each take between 20MB and 200MB. It's definitely not easily comparable.

Closing due to inactivity, feel free to reopen if the issue persists


Plugins implement order during document loading state

Hi I am using CNZZ(an analyse platform) to analyse my website. It use document.write instead of Node.insertBefore(It also invokes document.write in the imported js file, so no matter I use insertBefore or not will experience below case)
Below is what I do to import it:
// plugins/umeng.js
if (process.env.NODE_ENV === 'production') {
  document.write(unescape("%3Cspan id='cnzz_stat_icon_1263089026'%3E%3C/span%3E%3Cscript src='" + cnzz_protocol + "s13.cnzz.com/z_stat.php%3Fid%3D1263089026%26show%3Dpic1' type='text/javascript'%3E%3C/script%3E"))
}

// nuxt.config.js
plugins: [
  { src: '~plugins/umeng.js', ssr: false } // 1.0.0-alpha.4
  { src: '~/plugins/umeng.js', ssr: false } // 1.0.0-rc3
]

When I use 1.0.0-alpha.4 I met this warning
A Parser-blocking, cross site (i.e. different eTLD+1) script, http://s13.cnzz.com/z_stat.php?id=1263089026&show=pic1, is invoked via document.write. The network request for this script MAY be blocked by the browser in this or a future page load due to poor network connectivity. If blocked in this page load, it will be confirmed in a subsequent console message.See https://www.chromestatus.com/feature/5718547946799104 for more details.

It still works though it looks ugly, but when I tried to migrate Nuxt to 1.0.0-rc3 I met this
Failed to execute 'write' on 'Document': It isn't possible to write into a document from an asynchronously-loaded external script unless it is explicitly opened.

It cannot work!
Then I set a breakpoint at the code where I invoke document.write, I typed docment.readyState in the console when the program was paused
output loading when using 1.0.0-alpha.4
output interactive when using 1.0.0-rc3
So I wanna know where can I place the document.write to ensure it implement when the state of document is still loading?
Thanks in advance.

Hi. Scripts are loading as async by vue SSR engine. So as a workaround you may create an static file and include it in head:
static/cnzz.js
  document.write(unescape("%3Cspan id='cnzz_stat_icon_1263089026'%3E%3C/span%3E%3Cscript src='" + cnzz_protocol + "s13.cnzz.com/z_stat.php%3Fid%3D1263089026%26show%3Dpic1' type='text/javascript'%3E%3C/script%3E"))

nuxt.config.js
head: {
      script: [
        { src: '/cnzz.js' }
      ],
 }

That did the trick, you saved my day, thank you @pi0


Export authentication state

Exports the state of the plugin as AuthenticationState so it can be used when not having access to the Vue instance. For example in Vou Router beforeEnter method.
Proposed in #139

@jnt0r I really appreciate your diligence on this. I am going to start working with this and looks promising!


fix: BathExporter should export continuously when batch size is reached

Which problem is this PR solving?
The batch export processor should trigger an export when either of these conditions are met:

It has been more than scheduledDelayMillis since the last export
The queue has maxExportBatchSize items in it

Example from the dotnet implementation https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/BatchExportProcessor.cs#L265
Relevant portion of the specification https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/sdk.md?plain=1#L610

/cc @martinkuba looks like this actually doesn't have the same concurrency issue. He limited it in a slightly different way than i did but it requires a lot less change so I might actually prefer this solution.


[AST] Generalize and rename ClassDecl::getEmittedMembers()

Generalize ClassDecl::getEmittedMembers() to operate on an
IterableDeclContext, so that it can be for other nominal types,
extensions, etc. Rename to getSemanticMembers() to indicate that
these are all of the members that are semantically part of that
context.
Also add getParsedMembers() to retrieve the set of members
that were parsed from the source code, excluding any that were
implicitly synthesized by the implementation. Clients should choose
between these new entry points, because getMembers() itself
changes meaning as the translation progresses.
Clean up the implementation slightly so it only forces type checking
for the conformances within that particular context (using
getLocalConformances()) and doesn't need to list out each of the
protocols it cares about.

@swift-ci please smoke test

@swift-ci please test source compatibility

@swift-ci smoke test

@swift-ci please smoke test platform macOS

@swift-ci please smoke test platform macOS

@swift-ci please smoke test platform macOS

@swift-ci please smoke test platform macOS

macOS failure is unrelated (should be handled by https://github.com/apple/swift-driver/pull/235)


No image found on USB drive.

I've managed to install something to a USB drive I had laying around, but it simply refuses to boot. Gives me an error message saying that no image was found on the USB drive and to press any key to continue, which obviously didn't do anything. This was all compiled on an x86-64 machine with no hiccups. Perhaps its because the USB drive is formatted as GPT in macOS disk utility?

I'll attach another image shortly.

Ok, I'm building this on an x86_64 system now. I'll document every step

If you build it a second time you can use the w argument
ie
./build.sh -wd /dev/sdc

this saves time as it just wipes the drive vs having to re-partition it.

If you build it a second time you can use the w argument ie
./build.sh -wd /dev/sdc

this saves time as it just wipes the drive vs having to re-partition it.

I'll try this. Thanks!

Heres another picture.

Is it booting?

Nope. Still the same issue. Should I reinstall the whole OS at this point?

Here's what I would try on your system where you're building the usb drive
I ran everything as root on an x86_64 machine. I need to test out procedures running sudo
Since I haven't tested it -- I can't guarantee that using sudo will work at the moment.
It should work, but you would need to specify the full path to mkosi if it's located in a users directory.
uninstall mkosi just in case you have an old version or something
[root@black ~]# pip3 uninstall mkosi
Found existing installation: mkosi 14
Uninstalling mkosi-14:
  Would remove:
    /root/.local/bin/mkosi
    /root/.local/lib/python3.11/site-packages/mkosi-14.dist-info/*
    /root/.local/lib/python3.11/site-packages/mkosi/*
    /root/.local/share/man/man1/mkosi.1
Proceed (Y/n)? y

install mkosi from main
python3 -m pip install --user git+https://github.com/systemd/mkosi.git

clone the repo and run the script
git clone https://github.com/leifliddy/asahi-fedora-usb.git
cd asahi-fedora-usb
./build.sh -d /dev/sdc

this is the final part of the log
‣   Setting root password
‣   Generating system users
/root/asahi-fedora-usb/.mkosi.tmp5k51je4w/root/usr/lib/sysusers.d/basic.conf:10: conflict with earlier configuration for user 'root', ignoring line.
‣   Applying presets…
Created symlink /root/asahi-fedora-usb/.mkosi.tmp5k51je4w/root/etc/systemd/system/multi-user.target.wants/remote-cryptsetup.target → /usr/lib/systemd/system/remote-cryptsetup.target.
Created symlink /root/asahi-fedora-usb/.mkosi.tmp5k51je4w/root/etc/systemd/system/dbus-org.freedesktop.home1.service → /usr/lib/systemd/system/systemd-homed.service.
Created symlink /root/asahi-fedora-usb/.mkosi.tmp5k51je4w/root/etc/systemd/system/multi-user.target.wants/systemd-homed.service → /usr/lib/systemd/system/systemd-homed.service.
Created symlink /root/asahi-fedora-usb/.mkosi.tmp5k51je4w/root/etc/systemd/system/systemd-homed.service.wants/systemd-homed-activate.service → /usr/lib/systemd/system/systemd-homed-activate.service.
Created symlink /root/asahi-fedora-usb/.mkosi.tmp5k51je4w/root/etc/systemd/system/sysinit.target.wants/systemd-pstore.service → /usr/lib/systemd/system/systemd-pstore.service.
Created symlink /root/asahi-fedora-usb/.mkosi.tmp5k51je4w/root/etc/systemd/system/basic.target.wants/rpmdb-migrate.service → /usr/lib/systemd/system/rpmdb-migrate.service.
Created symlink /root/asahi-fedora-usb/.mkosi.tmp5k51je4w/root/etc/systemd/system/basic.target.wants/rpmdb-rebuild.service → /usr/lib/systemd/system/rpmdb-rebuild.service.
‣   Recording packages in manifest…
‣   Resetting machine ID
‣   Removing random seed
‣   Relabeling files using targeted policy
‣  Unmounting image…
‣ Saving manifest mkosi.rootfs.manifest
‣ Resulting image size is 1.1G.
### Cleaning up
### Mounting usb partitions and copying files
### Setting uuids for partitions in /etc/fstab
### Running systemd-machine-id-setup
Initializing machine ID from random generator.

### Generating EFI bootloader
boot UUID: a1492762-3fe2-4908-a8b9-118439becd26
root UUID: d747cb2a-aff1-4e47-8a33-c4d9b7475df9
Partition: hd0,gpt2
prefix_grub_dir: /grub2

Generating GRUB image...
EFI Bootloader Created
-rwxr-xr-x. 1 root root 577536 Apr 20 22:19 /boot/efi/EFI/BOOT/BOOTAA64.EFI
### Creating BLS (/boot/loader/entries/) entry
/usr/bin/rm: cannot remove '/boot/dtb-6.2.11-450.asahi.fc38.aarch64': Is a directory

### Generating GRUB config
Generating grub configuration file ...
done
### Adding delay to NetworkManager.service
### Enabling system services
### Disabling systemd-firstboot
### Setting selinux to permissive

### Cleanup
### Unmounting usb partitions
### Done

It should work at this point. If it doesn't maybe something else is screwed up on your mac.

**sudo ./build.sh -wd /dev/sdb
# # wiping usb partitions
./build.sh: line 138: mkosi: command not found**
Thats what I get in return. Why can't just things work? I've reinstalled it, but still nothing.

I said over and over again that I ran the command as the root user and that I can't guarantee that running with sudo will work as the mkosi would need to set statically set.
And then you run it with sudo and run into that exact issue.
Can't you just su to the root user and then those commands like I did?

sudo should work, but you need to change this line in build.sh
from
mkosi
to the location was installed with ie python3 -m pip install --user git+https://github.com/systemd/mkosi.git
ie
/home/leif.liddy/.local/bin/mkosi
so it'd be something like this
before
mkosi_create_rootfs() {
    mkosi clean
    rm -rf .mkosi*
    mkosi
}

after
    mkosi_create_rootfs() {
    /home/leif.liddy/.local/bin/mkosi clean
    rm -rf .mkosi*
    /home/leif.liddy/.local/bin/mkosi
}

I might create a feature that will simplify that....

The USB now boots. Thank you for your help!


Write and Read KZG setup parameters

Draft for #270

Hi @ekrembal is this still worked on?

Since this PR does not appear to be actively worked on. Closing this. Please reopen when this is active.


RuntimeError: Could not get the OCRAgent instance. Please check the OCR package and the OCR_AGENT environment variable.

After struggling with certain bugs, I encounter with bug again. And This time I don't really understand whay kind of bug is.
I am beginner in this field .
Can you guys help resolving the issue.

l have same problem ，please help us


The folder currently open doesn't have a git repository

Type: Bug
I am able to use all git commands on a cloned repo locally in VS Code terminal, but the SCM source control shows "The folder currently open doesn't have a git repository" error. And "Initialize Repository" does nothing.
Steps to reproduce:

git clone a private Github repo using https url.
Authenticate using user id and personal access token.
Successful cloning of the repo locally. Open that folder in VS Code when the prompt appears.
Verified that .git dir exists in the folder. Checked the git status output.
 Everything works from the VS Code terminal.
So, the only issue is that nothing shows up in the SCM source control.

Some outputs for reference:
$ git status
On branch main
Your branch is up to date with 'origin/main'.
$ git --version
git version 2.37.3.windows.1
System info
64-bit operating system, x64-based processor
OS: Windows 10 Enterprise; Version 21H2
VSCode version installed
VSCodeSetup-x64-1.74.1
(Also tried with VSCodeUserSetup-x64-1.71.2, but getting same error)
VS Code version: Code 1.74.1 (1ad8d514439d5077d2b0b7ee64d2ce82a9308e5a, 2022-12-14T10:30:51.966Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: Yes
Extensions (10)

Extension
Author (truncated)
Version

gitlab-workflow
Git
3.57.3

vscode-docker
ms-
1.23.2

isort
ms-
2022.8.0

python
ms-
2022.20.1

vscode-pylance
ms-
2022.12.20

jupyter
ms-
2022.11.1003412109

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.12

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

I had the same issue that the button "Initialize Repository" in source control can not be clicked.

@lszomoru Sure. This is what I see in the git output window after I have opened the folder:

At the same time, if I do "git status" in the terminal, this is what I get:

it can not be initialised

Hi @lszomoru, sorry, but I am still not able to make this work. Do you have any insights? Thanks!

@pratha10, thanks for the additional information. Could you please do the following:

Open terminal and navigate to the folder that you would like to open in VS Code
Execute the following command: code . --log vscode.git:trace
Share the contents of the git output window

@lszomoru thanks for your reply, appreciate it!
See below the git output contents after executing  code . --log vscode.git:trace
2023-01-12 16:16:03.193 [info] Log level: Info
2023-01-12 16:16:03.194 [info] Validating found git in: "C:\Program Files\Git\cmd\git.exe"
2023-01-12 16:16:03.712 [info] Using git "" from "C:\Program Files\Git\cmd\git.exe"
2023-01-12 16:16:38.383 [info] Log level: Info
2023-01-12 16:16:38.383 [info] Validating found git in: "C:\Program Files\Git\cmd\git.exe"
2023-01-12 16:16:38.383 [info] Using git "" from "C:\Program Files\Git\cmd\git.exe"
2023-01-12 16:16:38.383 [info] > git rev-parse --show-toplevel [376ms]
2023-01-12 16:16:38.654 [info] > git rev-parse --git-dir --git-common-dir [291ms]
2023-01-12 16:16:38.654 [info] fatal: not a git repository (or any of the parent directories): .git
2023-01-12 16:16:38.950 [info] > git rev-parse --show-toplevel [272ms]
2023-01-12 16:16:39.138 [info] > git rev-parse --git-dir --git-common-dir [167ms]
2023-01-12 16:16:39.138 [info] fatal: not a git repository (or any of the parent directories): .git
2023-01-12 16:16:39.418 [info] > git rev-parse --show-toplevel [263ms]
2023-01-12 16:16:40.220 [info] > git rev-parse --git-dir --git-common-dir [781ms]
2023-01-12 16:16:40.220 [info] fatal: not a git repository (or any of the parent directories): .git
2023-01-12 16:16:40.516 [info] > git rev-parse --show-toplevel [275ms]
2023-01-12 16:16:40.760 [info] > git rev-parse --git-dir --git-common-dir [223ms]
2023-01-12 16:16:40.760 [info] fatal: not a git repository (or any of the parent directories): .git
2023-01-12 16:16:41.014 [info] > git rev-parse --show-toplevel [229ms]
2023-01-12 16:16:41.225 [info] > git rev-parse --git-dir --git-common-dir [193ms]
2023-01-12 16:16:41.226 [info] fatal: not a git repository (or any of the parent directories): .git
2023-01-12 16:16:41.499 [info] > git rev-parse --show-toplevel [254ms]
2023-01-12 16:16:41.766 [info] > git rev-parse --git-dir --git-common-dir [239ms]
2023-01-12 16:16:41.766 [info] fatal: not a git repository (or any of the parent directories): .git

@pratha10, that output seems weird as I still see "Log level: Info" in the output which means that trace logging was not enabled. Another interesting in the log is that I see that Using git "" from "C:\Program Files\Git\cmd\git.exe". Did you obfuscate the git version? That should have listed the exact version of git that was detected.
Could you try to start VS Code using code --log trace and share the contents of the git output window? Thanks!

@pratha10, thank you very much for the additional information:
2023-01-20 08:20:20.651 [trace] Opening repository: e:\Github\<my-repo-name>
2023-01-20 08:20:20.951 [info] > git rev-parse --show-toplevel [300ms]
2023-01-20 08:20:20.975 [trace] Repository root: \.

Based on the logs, there seems to be a problem as we are determining the root of the repository since you see that in the log we are trying to open a repository at \.. Is E:\ by any chance a mapped drive? I have just fixed a bug related to mapped drives yesterday so I was wondering if you could download the latest Insiders release and see if that resolves the problem. Thanks!

@lszomoru yeah, I did think that that might be an issue. But wasn't sure, because it did work for me on my other PC.
So, yes, it is a mapped drive.

Awesome, thanks! Let me try the the latest Insiders release

@lszomoru I tried the same thing in this new version, but facing the same issue.

2023-01-20 09:46:25.995 [info] > git init [521ms]  "-- same error with or without this (for existing git repo)"
2023-01-20 09:46:26.965 [info] > git rev-parse --show-toplevel [200ms]
2023-01-20 09:46:27.136 [info] > git rev-parse --git-dir --git-common-dir [151ms]
2023-01-20 09:46:27.137 [info] fatal: not a git repository (or any of the parent directories): .git

And in fact, I forgot to mention earlier that I also had tried to git clone my repo and also do git init  in my C: drive, and was getting the same error.

And in fact, I forgot to mention earlier that I had tried to git clone my repo and also do git init  in my C: drive, and was getting the same error; that made me think that mapped drive is not the root cause, it might be something else

@pratha10, are you saying that you have tried running git clone and git init from a terminal and you were still getting errors? If that is the case, then this is indeed not an issue with VS Code but with something else on the machine itself.

@lszomoru thanks for your reply.
git init and git clone don't give any errors in the terminal. For example, git clone works in doing it's job by cloning the Github or Gitlab repo and the folder it creates is also recognized as a git folder in the terminal.
But both git init and git clone give the below errors in git output:
2023-01-20 09:46:25.995 [info] > git init [521ms]  "-- same error with or without this (for existing git repo)" 2023-01-20 09:46:26.965 [info] > git rev-parse --show-toplevel [200ms] 2023-01-20 09:46:27.136 [info] > git rev-parse --git-dir --git-common-dir [151ms] 2023-01-20 09:46:27.137 [info] fatal: not a git repository (or any of the parent directories): .git

@pratha10, apologies for not getting back to you on this until now. Are you able to share the trace with the latest version of VS Code as there are many things that have changed since the issue has been opened originally? Thank you!

Hi @lszomoru, no worries. Thanks for your reply.
I ran code --log trace and am getting the below output (same as before)
2023-04-11 10:54:19.437 [info] Validating found git in: "C:\Program Files\Git\cmd\git.exe"
2023-04-11 10:54:20.557 [info] Using git "" from "C:\Program Files\Git\cmd\git.exe"
2023-04-11 10:55:11.704 [info] Log level: Info
2023-04-11 10:55:11.704 [info] Validating found git in: "C:\Program Files\Git\cmd\git.exe"
2023-04-11 10:55:11.704 [info] Using git "" from "C:\Program Files\Git\cmd\git.exe"
2023-04-11 10:55:11.818 [info] > git rev-parse --show-toplevel [464ms]
2023-04-11 10:55:12.341 [info] > git rev-parse --git-dir --git-common-dir [483ms]
2023-04-11 10:55:12.341 [info] fatal: not a git repository (or any of the parent directories): .git

@pratha10, apologies for not getting back to you on this. Looking at the output it does not look like trace logging has been enabled. Let's try to do it through the UI and see if we are able to enable it and collect additional information to investigate this issue.

Start VS Code, and run the "Developer: Set Log Level..." command from the command palette
In the "Extensions log" group pick the "Git" extension, and click on the button on the end of the "Trace" line

Open the folder/workspace that contains the git repository, and share the contents of the git output window

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


gridsome serve: reloading the same page doesn't work

Description
My browser (Chrome on Mac) just indicates waiting for page if I reload the same page.
Steps to reproduce
gridsome create gridsome-site
gridsome build
gridsome serve
Go to http://localhost:8080 in your browser.
It works fine clicking the about link and start again, but as soon as you click a link to the same page or just reload the page you are on, the browser just waits for the page to load forever.
Expected result
Reloading the same page.
Actual result
Browser waits for page load forever.
Environment

Libs:
- gridsome version: 0.5.5
- @gridsome/cli version: 0.0.8

Browser:
- [X] Chrome (desktop) version 72.0.3626.121
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: 10.15.0
- Platform:  Mac

Others:

The gridsome serve command is very unstable and I don't recommend using it. It serves gridsome from an Express server with server-side rendering, so it doesn't show the site you just built. You can use tools like serve to test your static site.