Word converter: line comments in ANTLR are handled badly

Sample input:
reference_type
    : class_type
    | interface_type
    | array_type
    // This is a line comment
    | delegate_type
    | 'dynamic'
    ;

Output in Word:
reference_type
	: class_type
	| interface_type
	| array_type         // This is a line comment
	| delegate_type
	| 'dynamic'
	;

The comment that was intended to be about delegate_type is instead associated with array_type.
Additionally, we can't handle comments before the first colon (as Nigel has discovered with his string interpolation PR).
We should probably check that we don't actually use line comments in any ANTLR bits, as otherwise we're already broken (rather than just "not supporting something that would be nice").

We now use line comments in quite a few places as a result of my merging the unsafe grammar into the main grammar. For example, in <IP_ADDRESS> General, we have:
class_modifier
    : 'new'
...
    | 'static'
    | unsafe_modifier   // unsafe code support
    ;

Sorry, I should be clear - "end of line comments as a suffix" are fine; it's "comment that is a line on its own" that causes a problem.

TL;DR There is quite a bit of usage of such line comments.
There are quite a few uses of such comments:
grep -E '^\s*//[^/]' *.md
produces 185 hits – this number of occurrences will be a little higher after the grammar extraction tool has been run to update grammar.md and more are copied there from other sections.
Of these 164 are currently auto-generated ones in grammar.md where the grammar extraction tool prefixes each extract grammar chunk with a comment, e.g.:
input_element
: Whitespace
| Comment
| token
;
// Source: §7.3.2 Line terminators
New_Line
: New_Line_Character
| '\u000D\u000A'    // carriage return, line feed
;
We get line comments to introduce sections in the grammar (i.e. just as they’re used in above in grammar.md but hand written ones), and in folded comments, e.g.:
fragment Escaped_Identifier
: '@' Basic_Identifier // includes keywords and contextual keywords prefixed by '@',
// see note below
;
from  lexical-structure.md.

On 17/11/2021, at 1:03 am, Jon Skeet @.***> wrote:
Sample input:
reference_type
: class_type
| interface_type
| array_type
// This is a line comment
| delegate_type
| 'dynamic'
;
Output in Word:
reference_type
: class_type
| interface_type
| array_type         // This is a line comment
| delegate_type
| 'dynamic'
;
The comment that was intended to be about delegate_type is instead associated with array_type.
Additionally, we can't handle comments before the first colon (as Nigel has discovered with his string interpolation PR).
We should probably check that we don't actually use line comments in any ANTLR bits, as otherwise we're already broken (rather than just "not supporting something that would be nice").
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/dotnet/csharpstandard/issues/394, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABSYVWYZA36H7SY7GXGKPIDUMJB7TANCNFSM5IECVGCA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thanks for checking that, Nigel. It looks like line comments before a production are okay - it's only within a production that it's a problem.
I'll have a look to see how many we've got within productions - your regex finds various comments in C# code, of course.

Yeah, I thought about writing one which matched the comments and the open/close fences so it was a (little) more obvious which ones were ANTLR and which C# – but I haven’t had my first coffee of the day yet ;-)

On 17/11/2021, at 8:25 am, Jon Skeet @.***> wrote:
your regex finds various comments in C# code, of course

As noted in #434, I think the best we can do before C# 6 is make sure we can find all the places where this happens, either via grep etc or via warnings in the converter, and manually fix them afterwards. I don't think there will be that many.


changing title

the main page title is not correct   please assign me @pa

kindly check already opened ones


appended IDs on anchor hashes

I believe the following is a side-effect of https://github.com/eslint/website/pull/755 (filed as a separate issue as requested).
The site is now somewhat unpredictable to link to programmatically.
Previously, linking to https://eslint.org/docs/rules/<rule-name>#options would link to, and auto scroll to the options section for any given rule.
However now it doesn't work. This is because now anchor hash has a unique ID appended to it.
This means that for a given rule, the options anchor will have a different ID appended, eg:

https://eslint.org/docs/rules/comma-dangle#options-14
https://eslint.org/docs/rules/indent#options-43
https://eslint.org/docs/rules/no-unused-vars#options-116

This has broken all existing links to ESLint docs that people had around the web.
It's not a huge deal, because it still opens the correct page, however it is a weird behaviour.
An important question that arises from this behaviour is this: are these appended IDs guaranteed to be stable forever? I.e. if someone adds a new page to the site (like a new rule doc, or even just a new heading somewhere), will that cause:

all the IDs to change?
some of the IDs to change?
none of the IDs to change?

If the situation was anything but (3), then this would be a pretty big regression, as it would make it impossible for anyone to create a stable link to the ESLint docs.

#782 should remove all unnecessary -1, -2, -3, ...
This doesn't entirely revert to previous header IDs, because in #755 we also changed the algorithm (from  AnchorJS's urlify, to github-slugger + our custom code).
However, I believe the vast majority of header IDs will be exactly the same as before #755.

I don't personally have an issue with having the IDs. It's a little bit weird to see them in the URL, but 🤷♂ (chrome is moving to hiding every but the domain, so that matters less)
The big question for me is the last one I posed:
Are the anchor IDs stable?*
For example; if I add a new rule called aardvark-lint-rule (because it will be alphabetically first) will that cause every single lint rule's #options to get a new ID?
Or will the existing IDs all remain stable, and this new page will get the new ID?
If it's the latter, then that's a problem: anchor permalinks are no longer permanent.

For example; if I add a new rule called aardvark-lint-rule (because it will be alphabetically first) will that cause every single lint rule's #options to get a new ID?
Or will the existing IDs all remain stable, and this new page will get the new ID?

It would be very unstable. I just tried this with the actual version: the new document got id="options-1"  and the old documents had -number suffixes incremented by 1.
We don't actually need -number suffixes, so it's fine to remove them unless there are duplicates on the same page.
I don't think we'll be able to guarantee permanent links (for example, if we add another header with the same name on the same page, the old one could be incremented by 1), but normally they shouldn't change.

Hi all! I can take a look at this to fix this issue. Let me know if that'd be okay. So from what I understand it'd be okay to remove the auto-increment suffixes from the headers.
Does it have to do anything with search on Algolia? Would it be impacted ?

Hi @shobhitchittora, thanks for the help!
We already have PR #782 to remove the suffixes where there are not necessary. If it doesn't look good, we can consider another approach.
The search will be affected (links will point to non-existing IDs) for a short period of time until it updates the index with new IDs.


Add the specific types of Windows OS badges

Currently its just Windows + Windows Server
ie. i mean windows 10, xp, etc that are available with shield.io

added in https://github.com/inttter/md-badges/commit/a962fb1ec62561da891c17d896b6217713cfe6e6


Updated perforce to version 2015.1240625

Just a new version
In the spirit of October #hacktoberfest

Thanks! Merged in 8d92efeb99814bcc95f0acd38f1828b32e990a0d.


Add resize images support in markdown displayed in dashboards

Disclamer: perhaps not the right place to open the issue but finding the good place to report issue and bugs in VSTS is not easy...
You could read from: https://docs.microsoft.com/en-us/azure/devops/project/wiki/markdown-guidance?view=vsts#images

Note
The syntax to support image resizing is only supported in pull requests and the Wiki.

Please add support for resize images support in markdown displayed in dashboards displayed in VSTS/Azure Devops.
There is not a lot of places in widget and displaying images features in markdown is useless if we can't resize it.

Hi Philippe! Thanks for your feedback. We are closing this issue as it does not specifically relate to our docs. However, please feel free to suggest the feature here: https://developercommunity.visualstudio.com/content/idea/post.html?space=21.


Solidify types in a few reducers/Fix category filter

closes #2438, #2440, #2437

get this error after logging in.

@justinbarry @bthaile -- Is this relevant anymore?
constants.ts now has a bunch of conflicts just due to " and ' -- did this code get in through another path?

@justinbarry was making headway on this recently

Moved types stuff to another PR. Closing this.


x/mobile: add option to pass in gobind

% ~/go/bin/gomobile bind -target iOS ./lib

will not work until you set PATH environment variable which points to gobind.
It is something that I don't expect.
I expect that ~/go/bin/gomobile init will fix all issues and everything will be fine.
Could you add option to set path to binary gobind?
% ~/go/bin/gomobile bind -bin ~/go/bin/gobind -target iOS ./lib

/cc @hyangah @hajimehoshi


Dev Environment Setup Documentation

Zoom Video recording
https://youtu.be/C8na_kzCJcs (1st part)
https://youtu.be/_6cgKO0Rod0 (2nd part)

Latest steps (attached Avatar Creation.txt)

Hi Erwin, I've merged your changes and imported the repo at https://avatar-server.readthedocs.io/en/latest/local files.html. Thanks.

Closing ticket with essential changes. Documentation to be enhanced/updated as we go.


DELETE and return 200 OK

I want to return a 200 OK on DELETE method.
I also written a http_codes entry for this, but this won't work.
https://github.com/ruby-grape/grape-swagger/blob/master/lib/grape-swagger/endpoint.rb#L178-L181

That code definitely looks suspicious. Maybe write a spec that reproduces it first?

please see this explanation https://github.com/ruby-grape/grape-swagger/pull/440#issuecomment-222302039

@dopykuh … could it be closed?


Support up to Django 1.10

This shouldn't be too hard. I'm actually currently using it but the tests, for example, aren't supported in Django 1.10 which I'm using at the moment.
Happy for me to take a stab at this?

@kyleobrien91 all yours :)

#30 adds tests that verify that Django 1.10 is supported out of the box with no additional configuration. I think this ticket can be closed off.


step5: add incremental coverage

Codecov Report

Merging #6 (408cbae) into main (e8e2111) will increase coverage by 5.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   39.84%   45.11%   +5.26%
==========================================
  Files           4        4
  Lines         133      133
  Branches       21       21
==========================================
+ Hits           53       60       +7
+ Misses         69       65       -4
+ Partials       11        8       -3

Flag
Coverage Δ

frontend
30.13% <ø> (+9.58%)
:arrow_up:

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

Impacted Files
Coverage Δ

web/static/js/calculator.js
30.13% <0.00%> (+9.58%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add peers bootstrapper

cc @xichen2020 @martin-mao

Coverage increased (+0.5%) to 80.212% when pulling ba1e6c6e19105c23c86d0381ce980ba3ad9e0df2 on peers-bootstrap into d69946dce627a40b0f1003ebd7fd5ebc5ff22da6 on master.

Coverage increased (+0.5%) to 80.212% when pulling ba1e6c6e19105c23c86d0381ce980ba3ad9e0df2 on peers-bootstrap into d69946dce627a40b0f1003ebd7fd5ebc5ff22da6 on master.

Coverage increased (+1.1%) to 80.834% when pulling 70a120f28d52e2f714239e1216af5ac5a38a17a2 on peers-bootstrap into d69946dce627a40b0f1003ebd7fd5ebc5ff22da6 on master.

Coverage increased (+1.1%) to 80.834% when pulling 70a120f28d52e2f714239e1216af5ac5a38a17a2 on peers-bootstrap into d69946dce627a40b0f1003ebd7fd5ebc5ff22da6 on master.

Coverage increased (+0.5%) to 80.172% when pulling cba75a2a133b5a05476e61da512afcead8ed9b23 on peers-bootstrap into d69946dce627a40b0f1003ebd7fd5ebc5ff22da6 on master.

Coverage increased (+0.5%) to 80.172% when pulling cba75a2a133b5a05476e61da512afcead8ed9b23 on peers-bootstrap into d69946dce627a40b0f1003ebd7fd5ebc5ff22da6 on master.

Coverage increased (+0.4%) to 80.153% when pulling 2f45a0647b2a83ff50a2b6eda0a059827fb1de21 on peers-bootstrap into d69946dce627a40b0f1003ebd7fd5ebc5ff22da6 on master.

Coverage increased (+0.4%) to 80.129% when pulling bd297fac905818470196ef4e6de9748c6ddf8dff on peers-bootstrap into d69946dce627a40b0f1003ebd7fd5ebc5ff22da6 on master.

LGTM

Minor comment on some error logging, but LGTM


Generate quality fieldmosaic from NRMAC results using lowsize images

per suggestion from @Paheding and @SeanHartling we will create a new fieldmosaic product from the NRMAC outputs of fieldmosaic.

generate a geotiff from NRMAC of with single value for all pixels @ low resolution
track these geotiffs in rulechecker just like the left.tifs
stitch together these to create "heat map" of field quality for each scan

@max-zilla Thanks for working on this, let me know if you need any help. I assume the full-field image quality map would be very useful for users.

@max-zilla Thanks, I hope the provided the algorithm flow helped.

for this and other qa/qc products (e.g. fullfield thumbnails) , would it be difficult to overlay the plot boundaries?

Created a PR to include in normal extractor:
https://github.com/terraref/quality-metrics/pull/5
And rulechecker:
https://opensource.ncsa.illinois.edu/bitbucket/users/mburnet2/repos/terraref-rulechecker/pull-requests/6/diff

@max-zilla I have approved the request.

merged.


Language Server: file/checksum fails after uploading a big file (1.4 GB)

Follow up of https://github.com/enso-org/enso/issues/5051
After dropping the Enso AppImage (~1.4 GB), the IDE uploads the file, but then it tries to get the checksum (to make sure it's uploaded correctly), but the file/checksum operation fails with message:
{"jsonrpc":"2.0","id":701,"error":{"code":1,"message":"Service error"}}

I get the following enigmatic error log from Language Server:
 INFO ide_ci::program::command: npm⚠ [error] [2023-05-15T08:31:17.59Z] [org.enso.languageserver.requesthandler.file.ChecksumFileHandler] Failure during [ChecksumFile] operation: ***

Due to this bug, the call to this function will be commented. Once fixed, it should be uncommented again: https://github.com/enso-org/enso/blob/81d4b08c786c1d13c47f663815351b6414d30257/app/gui/src/controller/upload.rs#L145

While we are at it, is there any reason why insist on using SHA-3 256 while MD5 or SHA1 would be perfectly fine, and likely much faster? Internally, for the caches, we have already made that move and that saved us some time.

I don't think anyone insists on using SHA-3 256 here and I also think that changing that to md5sum is a good idea from the perf perspective. @farmaazon what do you think?

I'm not sure. The entire world seems to shift from SHA-1/md5 to SHA-3. Even similar cases like ours, e.g. the digests of Linux installers. On the other hand, I cannot imagine how someone could achieve some malicious goal by exploiting collisions of SHA-1 sums when uploading files to the cloud. I wouldn't rely on this too much, however - attacks can be very inventive.

But, if I understand this correctly, this is just checking if the file we got is the one that user tries to upload, right? If so, what are the scenarios of a theoretical attack here? I can't imagine any of such scenarios.

Yes, and you can say the same about hashes of Linux installers. But still, for some reason, every Linux installer download page uses sha256 for their hashes.
Also, as a rule, not being able to imagine an attack does not mean it does not exist.
In this very case, I would say a bit of paranoia is not harm. So far, computing hashes is a tiny fraction of time spent on uploading them, so I'd say we can afford it.

As would need changing the rust code - putting on hold.
We should do this on the JS implementation.

I suggest starting working on this one, @jdunkerley, as we’re implementing file uploading in the new gui and we’re facing the same problem.

Related to #8662


Update sub navigation documentation

Add component examples
Add component argument definitions

Would be good to show an example of the route that's needed to mack the nav switch state when you navigate around a service. That's missing from the GOV.UK design system at the moment and I haven't been able to get it to work.
If we don't do this now we could follow up quite quickly with this. I think it's more important to have it in and iterate than have it working perfectly.


Draft: Gnome46 extensions23042024

Description of changes
Using the gnome-46-branch I noticed that appindicator-extension is not working/updated because upstream only released 5 days ago. So I tried to do an automated updated following the readme.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Sorry for the noise, rebased/corrected target 🙏🏻


Initial offset for EventProcessorHost is set to 0 and cannot be overridden

Describe the bug
Initial offset for EventProcessorHost is set to 0 and cannot be overridden.
To Reproduce
Steps to reproduce the behavior:

create a new eventhub
create a new eph to connect to that eventhub:

    await EventProcessorHost.createFromConnectionString(
        EventProcessorHost.createHostName($SERVICE_NAME),
        $STORAGE_CONNECTION_STRING,
        $STORAGE_BLOB_NAME,
        $EVENTHUB_CONNECTION_STRING,
        {
            eventHubPath: $EVENTHUB_PATH,
            consumerGroup: $CONSUMER_GROUP,
            initialOffset: EventPosition.fromEnqueuedTime(Date.now()),
            onEphError: info => handleError(info.error, { ...info, error: undefined })
        }
    ).start(handleMessage(await client.getSender()), handleError);

the eph will continuously throw with The supplied offset '0' is invalid. The last offset in the system is '-1'  even if you change initialOffset to any other value in EventProcessorHost.createFromConnectionString or omit it altogether

Expected behavior
The initial offset should be overridable to whatever is passed to the create method. If it's not passed, it should correctly default to -1 (the eventhub default)
Package-name: azure-event-hubs | azure-event-processor-host
Package-version: 1.0.6
node.js version: 9.7.1
OS name and version: OSX 10.14
Additional context
Add any other context about the problem here.

@dpaje  I'm adding a sample code based on what you described above and It works without any errors.
Sample Code:
import {
  EventProcessorHost, OnReceivedError, OnReceivedMessage, EventPosition, EventData, PartitionContext, delay
} from "../lib";
import * as dotenv from "dotenv";
dotenv.config();

const path = "EVENTHUB_NAME";
const storageCS = "STORAGE_CONNECTION_STRING";
const ehCS ="EVENTHUB_CONNECTION_STRING";
// creates a unique storageContainer name for every run
// if you wish to keep the name same between different runs then use the following then that is fine as well.
const storageContainerName = EventProcessorHost.createHostName("test-container");
const ephName = "my-eph";

/**
 * The main function that executes the sample.
 */
async function main(): Promise<void> {
  // Please feel free to use the `./sendBatch.ts` sample to send messages to an EventHub.
  // 1. Start eph.
  const eph = await startEph(ephName);
  // 2. Sleeeping for 90 seconds. This will give time for eph to receive messages.
  await sleep(90);
  // 3. After 90 seconds stop eph.
  await stopEph(eph);
}

// calling the main().
main().catch((err) => {
  console.log("Exiting from main() due to an error: %O.", err);
});

/**
 * Sleeps for the given number of seconds.
 * @param timeInSeconds Time to sleep in seconds.
 */
async function sleep(timeInSeconds: number): Promise<void> {
  console.log(">>>>>> Sleeping for %d seconds..", timeInSeconds);
  await delay(timeInSeconds * 1000);
}

/**
 * Creates an EPH with the given name and starts the EPH.
 * @param ephName The name of the EPH.
 * @returns {Promise<EventProcessorHost>} Promise<EventProcessorHost>
 */
async function startEph(ephName: string): Promise<EventProcessorHost> {
  // This is the enqueue time of my messages.
  const date: Date = new Date('Thu Feb 14 2019 12:14:20 GMT-0800 (Pacific Standard Time)');
  // Create the Event Processo Host
  const eph = EventProcessorHost.createFromConnectionString(
    EventProcessorHost.createHostName(ephName),
    storageCS!,
    storageContainerName,
    ehCS!,
    {
      eventHubPath: path,
      initialOffset: EventPosition.fromEnqueuedTime(date),
      onEphError: (error) => {
        console.log(">>>>>>> [%s] Error: %O", ephName, error);
      }
    }
  );
  // Message handler
  const partionCount: { [x: string]: number } = {};
  const onMessage: OnReceivedMessage = async (context: PartitionContext, data: EventData) => {
    (!partionCount[context.partitionId])
      ? partionCount[context.partitionId] = 1
      : partionCount[context.partitionId]++;
    console.log("##### [%s] %d - Rx message from partition: '%s', offset: '%s'", ephName,
      partionCount[context.partitionId], context.partitionId, data.offset);
    // Checkpointing every 100th event received for a given partition.
    if (partionCount[context.partitionId] % 100 === 0) {
      const num = partionCount[context.partitionId];
      try {
        console.log("***** [%s] Number of partitions: %O", ephName, eph.receivingFromPartitions.length);
        console.log("***** [%s] EPH is currently receiving messages from partitions: %s", ephName,
          eph.receivingFromPartitions.toString());
        await context.checkpoint();
        console.log("$$$$ [%s] Successfully checkpointed message number %d", ephName, num);
      } catch (err) {
        console.log(">>>>> [%s] An error occurred while checkpointing msg number %d: %O", ephName, num, err);
      }
    }
  };
  // Error handler
  const onError: OnReceivedError = (error) => {
    console.log(">>>>> [%s] Received Error: %O", ephName, error);
  };
  console.log(">>>>>> Starting the EPH - %s", ephName);
  await eph.start(onMessage, onError);
  return eph;
}

/**
 * Stops the given EventProcessorHost.
 * @param eph The event processor host.
 * @returns {Promise<void>} Promise<void>
 */
async function stopEph(eph: EventProcessorHost): Promise<void> {
  console.log(">>>>>> Stopping the EPH - '%s'.", eph.hostName);
  await eph.stop();
  console.log(">>>>>> Successfully stopped the EPH - '%s'.", eph.hostName);
}

Package-name:  azure-event-processor-host
Package-version: 1.0.6
node.js version: 10.14.1
OS name and version: Windows 10 Enterprise
Can you please provide more details about the error or stack trace?

@dpaje We havent heard back from you in a while, so we are assuming that you are not seeing this issue anymore. Therefore, we are closing this issue.
If you still see this problem or you have any other feedback on this library, please feel free to open a new issue.


Style reference

Hi Kenny,
Can't find 'Base.TextAppearance.AppCompat.Body2'in styles.xml. android:textAppearance="@style/Base.TextAppearance.AppCompat.Body2"(activity_about.xml)

Thanks for contacting.
Are you sure your AppCompat dependencies are alright?
Base.TextAppearance.AppCompat.Body2 seems to be default String options that aren't set by myself.
I think it all falls on supporting AppCompat.

If it doesn't work eventually, you may remove that line of code, while you work on the features you want to work on. :+1:

Alright.


Offscreen rendering in YUV format

Hi, is there any sample code for off screen rendering in YUV format?
For example, the input format of image is VK_FORMAT_G8_B8R8_2PLANE_420_UNORM then rendering it on output which is VK_FORMAT_G8_B8R8_2PLANE_420_UNORM.

No, there's no such code and I don't plan to create such a sample.

No, there's no such code and I don't plan to create such a sample.


[Bug]: Unable to use Apple Pay in Safari when Copilot is running

Before Reporting

[X] I have checked FAQ, and there is no solution to my issue
[X] I have searched the existing issues, and there is no existing issue for my issue

What happened?
I found out that Copilot interferes with Apple Pay confirmation sheet in Safari. Touch ID input (or password input if Touch ID is disabled) disappears from the view, making it impossible to complete the payment. Quitting Copilot solves the issue immediately.
When I was trying to understand why it happens, I found out that Touch ID input disappears if the Apple Pay sheet is in any way obscured by another window or screen boundary - you can see this happening in the attached video.
Google Drive link to video (even downscaled was to big to fit as attachment on Github): https://drive.google.com/file/d/18a7UAO28YXAwfzZaqztZZq5I2CAkZW64/view?usp=sharing
How to reproduce the bug.
Open a website with Apple Pay functionality (https://applepaydemo.apple.com is great for debugging this), and try to complete the payment while Copilot and Xcode are running.
Please do note that while the issue occurs often, it doesn't occur always. Sometimes Touch ID input is visible despite Copilot and Xcode running in the background. I did not find specific pattern that triggers the issue, but it shouldn't be hard to reproduce though.
Relevant log output
No response
macOS version
14.4.1 (23E224)
Xcode version
15.3 (15E204a)
Copilot for Xcode version
0.32.3

Sadly, Apple Pay is Safari is not supported in my country, I can't even add a card to my Mac.
If anyone wants to help debugging this, the app maintains several windows and all of them can be found in /Core/Sources/SuggestionWidget/WidgetWindowsController.swift.

I have created this projectthat only contains the window part of the app.
For anyone interested in helping debugging this issue and but don't want to build the whole Copilot for Xcode project, you can use that project instead.


Add Hover function to the nav var.

Hyy @harshbhardwaj000, I'm assigning this issue to you.

Can you give star to this repo?

Yes i will😊

Can u reassign this issue to me. I really want to contribute to this project

Can u reassign this issue to me. I really want to contribute to this project

Please show us the sample work that what do you want.

https://codepen.io/harjasae2001/pen/MWNOZgR
https://codepen.io/harjasae2001/pen/poMdqzq
Here are some sample. You can let me know which one should I apply.

Third Number --> https://codepen.io/harjasae2001/pen/MWNOZgR

Okay, Can u pls assign me this issue.

Okay, Can u pls assign me this issue.

Done ;)


[2012-08-20] "Check for upgrades" feature

created by: @sminnee (sminnee)
created at: 2012-08-20
original ticket: http://open.silverstripe.org/ticket/7794

We should add a feature that checks to see if the version of SilverStripe and module installed from the main module repository are at the latest version, and if not, notify the user.
Given the number of different scenarios in which SilverStripe can be installed, we'd need to have different options for how this is run:

Disable entirely
Only run for users of a particular permission
Include a RESTful API for grabbing this data programmatically, which would let someone run a tool that checked all their sites for out-of-date versions in a batch.

This feature could also be used to gain population-wide stats on SilverStripe-installs, version popularity, module popularity, etc.  Although some might fear this to be a "big brother" intervention, it would give the core team real data to assist the community with questions like "should we keep supporting the 2.x line?" ;-)
It would be best to wait until the module improvements planned for 3.1 are in place, but this could be a good feature to get into 3.1.

https://github.com/FriendsOfSilverStripe/silverstripe-maintenance


Solved the problem that easygui-choicebox does not work in python3.10

I have successfully tested with the following example:
import easygui
easygui.choicebox("What's your favourite food?" ,
choices = ['noodles','cookies','beef'])

This issue has been fixed(PR #188).

PR #188  should have fixed this.
From the Collections docs:

Deprecated since version 3.3, will be removed in version 3.10: Moved Collections Abstract Base Classes to the collections.abc module. For backwards compatibility, they continue to be visible in this module through Python 3.9.

So the current logic should work: using the old alias up until v3.9 (inclusive) and the new location thereafter.
When you say "doesn't work" - are you referring to the EasyGui version available from PyPI / installed via PIP? That doesn't include the fix yet - something that I've been meaning to remedy for a while...

This should now work - easygui v0.98.3 available from PyPI has the fix.
I used your test code as an example with Py3.10:

Does that work for you, also? Please let me know if there are any issues and I can re-open this PR.
(Also - sincere apologies that this has taken so long to update!)


Date predicates

I want to add "more than / at least / less than / at most x minutes / hours / days / weeks / months / years ago" as a predicate for the smart folders. They could replace the old predicates like "before last week". Those could be converted on the fly.

At this stage, the following things are missing:

Operators "at least ago" and "at most ago" in the predicate editor template and the translations
Conversion from Predicate to Criteria (planned to use operators less than, more than and set value to e.g. "x days")
Conversion from Criteria to SQL
Conversion of old fixed-distance date criteria to new predicate type when opening the editor, throwing out the old editor templates in turn
Tests of the conversion for the new predicate to criteria and sql and from criteria to predicate

The old fixed-distance date criteria are not yet converted, but they do no harm right now


Reserved Keyword Clash when using DotNet Libraries

1. Describe the bug
If a .Net library is used that exposes functions with a name equal to that of an AL keyword, the compiler assumes that the AL keyword is meant, preventing the use of the function of the .Net library.
2. To Reproduce

Set BC to the OnPremise version to enable .Net libraries
Place MimeKit (https://github.com/jstedfast/MimeKit) in the .netpackages folder
Create simple codeunit trying to use MimeKit
Use a MimeKit function that is named "To".
Try to compile

dotnet
{
    assembly(MimeKit)
    {
        type(MimeKit.MimeMessage; MimeMessage) { }
        type(MimeKit.MailboxAddress; MailboxAddress) { }
    }
}

codeunit 1 Test
{
    procedure Test()
    var
          Message: DotNet MimeMessage;
          MailAddress: DotNet MailboxAddress;
    begin
          MailAddress := MailAddress.MailboxAddress('Bla bla', 'bla@bla.com');
          Message.To.Add(MailAddress);
    end;
}

3. Expected behavior
The compiler correctly identifies the method "To" of the MimeKit.MimeMessage Class and thus the code compiles.
4. Actual behavior
The compiler incorrectly decides to identify the keyword "To" and fails to compile as he does not find a function "Add" that can be invoked.
5. Versions:

AL Language: v8.4.586670
Visual Studio Code: 1.62.3
Business Central: DE Business Central 19.3
List of Visual Studio Code extensions that you have installed: AL Language, AL AZ Dev Tools

Just found out that using
Message."To".Add(MailAddress);
works. It's a bit confusing why one works and not the other, but it works.

Quoting it makes it unambiguous that you want a member or other user-defined symbol, not a reserved keyword, hence why that fixes it I guess.
Was the error you got that the keyword To has no .Add member, or instead an error about the syntax?


Increase the number of chats that can be viewed

Only about 10-12 chats can be viewed without scrolling. It would be ideal to find ways to increase the number that can be viewed. Suggestions may be to:

Reduce font and line spacing
Widen chat area versus elements like chat room names
Full utilize space on margins
Remove 'chat' header (see other issue)

Closing as stale.


Waterlines

Do you think your code can be used to scrape data from this service
http://www.pcn.minambiente.it/arcgis/rest/services
Thanks,
Gijs

It should be able to scrape the individual rest end points of the feature services (those URLs ending in a number). You would need to modify the code to iteratively scape all feature services in a map service and all map services on an ArcGIS server.
Sent using my thumbs
On Mar 16, 2017, at 6:12 AM, gvandendool <notifications<EMAIL_ADDRESS>wrote:
Do you think your code can be used to scrape data from this service
http://www.pcn.minambiente.it/arcgis/rest/services
Thanks,
Gijs
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/crackernutter/EsriRESTScraper/issues/5, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGgO3jCiq2o_UnWJU_eITpAKMAK69i0Fks5rmTUWgaJpZM4MfRx5.


End result does not work

New to extension writing and thought I would give this a go on the new Chromium Edge (79, stable channel)
From walking through this tutorial I was met with an error at the very end and I've not been able to resolve it.
I reverted to the supplied completed version and this also does not work out of the box on fresh install of the extension.
Can we get someone to review this content?
Error:
Unchecked runtime.lastError: Could not establish connection. Receiving end does not exist.
Stack Trace
popup/popup.html:0 (anonymous function)

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

ID: ec889040-6707-74fd-9086-d838c15a9172
Version Independent ID: 73d6355a-b256-c8ab-9889-c51603d3052b
Content: Dynamically Insert NASA Picture Below The Page Body Tag Using Content Scripts - Microsoft Edge Development
Content Source: microsoft-edge/extensions-chromium/getting-started/part2-content-scripts.md
Product: microsoft-edge-chromium
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Doesn't work properly even if on a website. The download version neither works in chrome or edge properly either.
Does display an image placeholder in the active (online) tab that is approx 10x10 but that is it and it errors


[MTSRE-528] Fix serviceMonitor CR in sss.

py-mtcli
Description
The ServiceMonitor generated by py-mtcli has some flaws:

portName is not configurable
Not using the caFile present in the "master" prometheus server (the one living in openshift-monitoring)
Was not using a bearerTokenFile

All ServiceMonitor in managed openshift follow the same structure as the openshift-apiserver one, which is how we should model our configurations as well:
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  annotations:
    include.release.openshift.io/ibm-cloud-managed: "true"
    include.release.openshift.io/self-managed-high-availability: "true"
    include.release.openshift.io/single-node-developer: "true"
  name: openshift-apiserver
  namespace: openshift-apiserver
spec:
  endpoints:
    - bearerTokenFile: /var/run/secrets/kubernetes.io/serviceaccount/token
      interval: 30s
      metricRelabelings:
        - action: drop
          regex: etcd_(debugging|disk|server).*
          sourceLabels:
            - __name__
        - action: drop
          regex: apiserver_admission_controller_admission_latencies_seconds_.*
          sourceLabels:
            - __name__
        - action: drop
          regex: apiserver_admission_step_admission_latencies_seconds_.*
          sourceLabels:
            - __name__
        - action: drop
          regex: apiserver_request_duration_seconds_bucket;(0.15|0.25|0.3|0.35|0.4|0.45|0.6|0.7|0.8|0.9|1.25|1.5|1.75|2.5|3|3.5|4.5|6|7|8|9|15|25|30|50)
          sourceLabels:
            - __name__
            - le
      port: https
      relabelings:
        - action: replace
          replacement: openshift-apiserver
          targetLabel: apiserver
      scheme: https
      tlsConfig:
        caFile: /etc/prometheus/configmaps/serving-certs-ca-bundle/service-ca.crt
        serverName: api.openshift-apiserver.svc
  namespaceSelector:
    matchNames:
      - openshift-apiserver
  selector: {}

@Ajpantuso The whole hypothesis testing framework is fuzzy :sweat_smile: . It runs tests multiple times with different payloads https://github.com/mt-sre/managed-tenants-cli/blob/main/tests/conftest.py#L25-L33

@Ajpantuso The whole hypothesis testing framework is fuzzy 😅 . It runs tests multiple times with different payloads https://github.com/mt-sre/managed-tenants-cli/blob/main/tests/conftest.py#L25-L33

These are my definitions for fuzzy and too fuzzy lol:
fuzzy: I know which assertions are being made, but I don't know what data is being used.
too fuzzy: I don't know which assertions are being made or what data is being used.
When you add randomly executing code blocks you get inconsistent coverage with individual runs. With enough runs the law of large numbers kicks in and you'll get 50/50 with/without a custom port name. But any individual test run could be completely different including the scenario where 0% of the random assertion executes which makes CI less reliable.
But, for this test it's not really important IMO.


Add XR loaders and info

This PR adds in the OpenXR loaders plugin and configures the export templates.
It also adds some instructions and a versions file.
Fixes #11

The android export reports "Invalid package name: The project name does not meet the requirements for the package name format. Please explicitly specify the package name."
This is because we called the project "Godot XR Template". If we set the project name in the project settings (Application/config/name) to "GodotXRTemplate" then the android exports don't complain.

I guess we can do that for now, but this will also become the name in the project inspector. I don't know when this changed but it's annoying. I've been wondering if it isn't better to just change the export settings, but its more hidden and people may not realise where they need to change the export name for their own project.


Content and slug behaviour dependent from currently opened page when npm run dev starts (astro:content)

Problem:
I using this package with astro:content, but can't achieve stable behaviour of my blog when I working on routing to content.
I assumed convention when the first component of the path is language if it is a language different than English.
Example for static pages:
/ - main page in English
/pl - main page in Polish
Example for blog posts
/post/hello-world - blog post in English
/pl/post/widaj-swiecie - blog post in Polish
I have no problem with static pages, like index or about but for content I see that some posts work or do not work depending on the page on which I have opened the browser during the npm run dev command.
Test
I have closed the browser. I starting my blog.
npm run dev

I see http://localhost:3000/ in English, http://localhost:3000/pl in Polish, and http://localhost:3000/es in Spanish as I expected.
My blog post http://localhost:3000/post/hi is displayed correctly.
but when I enter http://localhost:3000/pl/post/witaj-swiecie I see 404.
To see its content I have to use http://localhost:3000/post/witaj-swiecie but then I lose info about the language in the URL and the language of the rest layout is wrongly recognized.
But it is only half of the problem...
Now I typing http://localhost:3000/pl/post/witaj-swiecie  in the browser address again to see 404.
I turned down the server in the Linux console.
Then typed
npm run dev

again and saw my Polish blog post on http://localhost:3000/pl/post/witaj-swiecie
But... it is not a solution because of when I entered now
http://localhost:3000/pl or http://localhost:3000/ I see it works because these are static pages, but when I want to check an English blog post and type
http://localhost:3000/post/hi
I see now 404, to see this post I can't see it even typing
http://localhost:3000/en/post/hi
because I am instantly redirected to http://localhost:3000/post/hi
In the console, I can see warnings
09:11:13 [getStaticPaths] A `getStaticPaths()` route pattern was matched, but no matching static path was found for the requested path `/post/hi`.

Possible dynamic routes being matched: src/pages/post/[...slug].astro.
09:11:13 [serve]    404

I created the branch astro-i18n-aut in my blog so you can reproduce it.
https://github.com/gustawdaniel/gustawdaniel/tree/astro-i18n-aut
Update 1
I found an example https://github.com/jlarmstrongiv/astro-i18n-aut/issues/8 and it works great. I investigating where I made bug.
Update 2
I downloaded your example from mentioned issue and it was working perfectly until I typed
npm update

Then I seed error
[config] Astro found issue(s) with your configuration:
  ! experimental  Invalid experimental key: `redirects`.
Make sure the spelling is correct, and that your Astro version supports this experiment.
See https://docs.astro.build/en/reference/configuration-reference/#experimental-flags for more information..

and when I removed
  experimental: {
    redirects: true,
  },

I was able to reproduce the same error using your code astro-i18n-blog
Update 3
To simplify reproduction I used your example and created 2 branches:
First - your original code with blog, when we do not see this error (it works but has outdated packages)
https://github.com/gustawdaniel/gustawdaniel/tree/astro-i18n-blog-no-update
Second - with a bug that is possible to reproduce after the npm update
https://github.com/gustawdaniel/gustawdaniel/tree/astro-i18n-blog-npm-update
There is open PR with changes in package-lock.json that was reason of problem.
https://github.com/gustawdaniel/gustawdaniel/pull/7/files

Potentially connected issues:
https://github.com/withastro/astro/issues/8226
https://github.com/withastro/astro/issues/8236
https://github.com/withastro/astro/issues/7808

You are right!
One change that is needed is updating the imports in the astro.config.mjs file:
import { i18n, defaultLocaleSitemapFilter } from "astro-i18n-aut/integration";

However, I believe the injectRoutes() behavior has changed and broken astro-i18n-aut in the dev environment from what I’ve tested.

Follow https://github.com/withastro/astro/issues/8241 for updates

Hey @ansmlc there was an issue with handling trailing slashes that I just fixed, please try again with the latest version!
If you have an issue with translations in preview, please open a new issue.  The preview and dev commands depend on different Astro internals, so only dev is broken.  Otherwise, it’s a fixable bug in the library.

Also @ansmlc if you have a repository or example project I could look at, please send it my way

@jlarmstrongiv Please is this still an issue? I'd like to know before I get to work on a blog, I'd like to have this clarified. I'm assuming that it still happens with version 4 of astro, am I right? Also I see that you updated the status of this issue, but I have figured that I'd still ask just to be sure. Thanks!

@jlarmstrongiv Please is this still an issue? I'd like to know before I get to work on a blog, I'd like to have this clarified. I'm assuming that it still happens with version 4 of astro, am I right? Also I see that you updated the status of this issue, but I have figured that I'd still ask just to be sure. Thanks!

 My setup was slightly different, but dev server returns erratic 404's and build/preview is fine.

@dallyh feel free to ask in the Astro discord.  I’ve done everything I can to try and get that issue prioritized.


[WIP] Tree structured inline panels

This PR adds tree structuring into condensed inline panels!  I think this would be mainly helpful for use on menu/taxonomy snippets/settings but I'd also like to try using this for things outside forms, such as configuring collections.

To enable it, just add the condensedinlinepanel.models.Structured model as a base to your inline child model (just like you currently do for Orderable). This will require a migration. This adds an integer field to your model called depth, which is used in conjunction with the sort_order field provided by Orderable to build the tree structure.
TODO:

[ ] Show blue box underneath when dragging a card over another card
[ ] Drag and drop doesn't carry descendants
[ ] Fix bug when form validation errors happen
[ ] Dropping at the end of a branch is a bit fiddly
[x] Add max depth limitation (or make styling work for infinite number of depths)
[ ] Adding forms when other forms have been deleted sometimes adds the new form in the wrong place

Hi @kaedroho! Do you see the condensedinlinepanel.models.Structured model being developed further? While the added dimension of depth seems to work fine for the purposes of this UI, I find myself wondering how I'd use just depth and order to render a multi-level list in a template. Maybe there could be a Manager or QuerySet method to augment each item with additional attributes (like 'parent', 'children', 'next' and 'previous') ?

Yep, we should definitely think of API to help with that, I'll add it to the TODO list. Thanks!


ensmarten snapshot queue

The snapshot queue needs a bit more subtlety. In some cases,
we really do want to do a snapshot right now -- these shouldn't
have to wait for possibly a hundred or more other snapshots
to complete.
In other cases, we don't really care that much whether we do
a snapshot, and just dropping it is probably fine.
To accommodate this, we distinguish between "urgent" and
"normal" snapshots, and between "Immediate" (does an urgent
snapshot, waits for it) and "Enqueue" (might enqueue a snapshot
but also might not if we're already busy). There's a
corresponding "Await" to wait for a snapshot, if one is
pending, but not if one isn't.
We also have a background scan that checks the holder. It will
scan pretty actively when it's finding fragments that need
snapshots (no enqueued snapshot, opN > MaxOpN). It pauses
for a second after every hundred fragments that didn't need
snapshots, and for a minute after each holder scan that didn't
find any. So, if you don't need snapshots, it does basically
nothing, if you do, it'll be moderately aggressive about
submitting tasks -- but it always waits if there's any
requested snapshots in the queues.

i can't reproduce the test failures and it looks like it timed out during go get.


Fixed issues service endpoint includes open issues

Hi,
I couldn't find where https://services.gradle.org/fixed-issues/6.8 is generated, so forgive me if this is not the correct project. I noticed that the endpoint exposes open issues as well and thus is a bit misleading and leads to wrong release notes, for example.
Service response

Release notes

Actual issue

Workaround
I've tested a workaround by filtering open issues in https://github.com/gradle/gradle/blob/master/subprojects/docs/src/docs/release/content/script.js that I could provide via a PR, but obviously it would be better if the fixed issues response doesn't contain unresolved issues in the first place.
Let me know what you think.
Cheers,
Christoph

Thanks.
Part of our process is to make sure no open issues are assigned to milestones when we close them, so we would have caught this then. That issue shouldn't have been assigned to 6.8.


Protobuf incompatible with google-cloud-python?

What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
Environment info
Operating System:
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*):
If installed from binary pip package, provide:

A link to the pip package you installed:
The output from python -c "import tensorflow; print(tensorflow.__version__)".

If installed from source, provide

The commit hash (git rev-parse HEAD)
The output of bazel version

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
What other attempted solutions have you tried?
Logs or other output that would be helpful
(If logs are large, please upload as attachment or provide link).

Could you please fill in the answers to the template and also describe the problem a bit more. From what you've listed ("Protobuf incompatible with google-cloud-python"), I'm unable to figure out what your precise question is. Did you observe some incompatibility? What was the error?

Hi @asimshankar. Sorry, you posted this while I was still working on the question. Perhaps we can loop in the folks from google-cloud-python?

A few things:
(a) You mention "google protobuf", but it seems you're talking about TensorFlow and not protobuf? Or is there something about protobuf that I'm missing?
(b) Does python -c "from google.cloud import logging as gclogging;" succeed if you do not install TensorFlow?
(c) I'm not sure how source installations interact, but can you pip install google-cloud (from https://google-cloud-python.readthedocs.io/en/stable/#) instead of installing google-cloud from sources. Does that work?

TensorFlow pip package depends on protobuf pip package version 3.1.0 according to the installation guide. In the common problems section.

Protobuf's pip package downloaded from PyPI (when running pip install protobuf) is a Python only library, that has Python implementations of proto serialization/deserialization which can be 10x-50x slower than the C++ implementation.

So essentially, the protobuf that comes with PyPI is slow and a c++ based one is much faster.
Yeah, python -c "from google.cloud import logging as gclogging;" succeeds if I don't install Protobuf with either the following command line
$ sudo pip install /tmp/tensorflow_pkg/tensorflow-0.11.0rc0-py2-none-any.whl
# OR
pip install --upgrade https://storage.googleapis.com/tensorflow/linux/cpu/protobuf-3.0.0-cp27-none-linux_x86_64.whl
# OR
pip install dist/protobuf-3.1.0-cp27-cp27mu-linux_x86_64.whl

This issue was referenced and solved here
https://github.com/GoogleCloudPlatform/google-cloud-python/issues/2563


Disable Pre-render

Describe the bug
There is some way to disable the pre-render when using this plugin? I am using an authentication middleware on my documentation, and would like to block some requests.
To Reproduce
Using an authentication middleware won't work because of the following snippet:
export function starlightOpenAPIIntegration(schemas: Schema[]): AstroIntegration {
  const starlightOpenAPI: AstroIntegration = {
    name: 'starlight-openapi',
    hooks: {
      'astro:config:setup': ({ injectRoute, updateConfig }) => {
        injectRoute({
          entrypoint: 'starlight-openapi/route',
          pattern: `[...openAPISlug]`,
          prerender: true,
        })

        updateConfig({
          vite: {
            plugins: [vitePluginStarlightOpenAPISchemas(schemas)],
          },
        })
      },
    },
  }

  return starlightOpenAPI
}

Expected behavior
An option to disable the pre-render
How often does this bug happen?
Every time
System Info
No response
Additional Context
No response

Thanks for the feedback :raised_hands:
This is not something currently supported. This brings up some interesting questions as how this would work as I assume we would not want to parse the schemas for every requests. Some parser utilities are also still used during the rendering process (e.g. to resolve references) and we may not want to bundle them in the final build.
Considering how heavy some schemas can be and how it can increase the build time, if the goal is just to protect the schema pages behind an authentication wall, I also wonder if some other approach could be used, e.g. edge middlewares or something else, so that these pages which don't change very often can still be pre-rendered.
There are definitely more research and investigation to be done here, not something I'll have the bandwidth to do in the near future, but I'll be happy to discuss more if someone wants to take a stab at it.


Fail the tool when analyzers fail to load

The tool currently silently swallows this sort of error.
This change is backward incompatible. Is that OK?

@Smaug123 One thing: Could you add a changelog entry, please.

This was motivated by https://github.com/Smaug123/fsharp-prattparser and specifically https://github.com/Smaug123/fsharp-prattparser/actions/runs/7618965411/job/20751163083 :
info: Running in verbose mode
info: Treat as Hints: []
info: Treat as Info: []
info: Treat as Warning: []
info: Treat as Error: [GRA-IMMUTABLECOLLECTIONEQUALITY-001, GRA-INTERPOLATED-001, GRA-JSONOPTS-001, GRA-LOGARGFUNCFULLAPP-001, GRA-STRING-001, GRA-STRING-002, GRA-STRING-003, GRA-TYPE-ANNOTATE-001, GRA-UNIONCASE-001, GRA-VIRTUALCALL-001]
info: Exclude Files: []
info: Include Files: []
info: Loading analyzers from /home/runner/work/fsharp-prattparser/fsharp-prattparser/.analyzerpackages/g-research.fsharp.analyzers/0.6.0/
error: Trying to load /home/runner/work/fsharp-prattparser/fsharp-prattparser/.analyzerpackages/g-research.fsharp.analyzers/0.6.0/analyzers/dotnet/fs/G-Research.FSharp.Analyzers.dll which was built using SDK version <IP_ADDRESS>. Expect <IP_ADDRESS> instead. Assembly will be skipped.
info: Registered 0 analyzers from 0 dlls
info: No messages found from the analyzer(s)


Packer error: Expected Lua number

Hey there,
since a few days I get an error in lua/todo-comments/highlight.lua, line 278 with Expected Lua Number.
It might be related to the latest API change for nvim_buf_clear_namespace (see https://github.com/neovim/neovim/issues/14090#issuecomment-1004318147).

Just pushed a fix for this. That line in the code actually never did what was intended, which resulted in the error being shown.
Fixed it now :)

Happy to help :)


Show star plot for the selected data point in the right side of Mat Dialog with data details

Attempted to show star plot when click a single data point in the scatter plot, but couldn't draw it perfectly till now. Currently viewable plot is shown in given image.


Feature/divider component

Add a <Divider /> component and examples for the divider. (Note: currently failing the Wercker test as a result of the Babel update)

After updating the one class name and using Grid and Column components, :ship: !


Add preserve inline function for expand style braces

Description
Basically, I want collapse-preserve-inline feature for expand braces style like expand-preserve-inline which is doesn't exist at the moment.
Input
The code looked like this before beautification:
foo() {
    var a = 2;
}

Expected Output
The code should have looked like this after beautification:
foo() { var a = 2; }

Actual Output
The  code actually looked like this after beautification:
foo()
{
    var a = 2;
}

@Kutsan -  expand-preserve inline would look like this:
The code looked like this before beautification:
function foo() { var a = 2; }
function bar() {
    var a = 2;
}

The  code would look like this after beautification:
function foo() { var a = 2; }
function bar()
{
    var a = 2;
}

I'm not sure what you're describing - I need more examples.  Do you want all braces collapsed to inline if possible?  Or do you want expanded braces to be kept but non-expanded to be inlined?

Just leaving a comment as you replied, haha. Yeah, this is exactly what I'm looking for.
I think this is just an expansion of the solution implemented in #382 as to fix import and export newlines you need to use collapse-preserve-inline which limits you to just collapse brace styling. It would be nice to have a preserve-inline-braces option which can be used with any of the braces styling formats.

Okay, I made #1058 . Now that I really look at it, I don't think this fixes your problem. It only preserves functions that already look like your expected output, it does not roll up function from your input to your expected output so I am of no help to you :(


fix(search-service): export datasource name as expected by cli in search service

export datasource name as expected by cli in search service
GH-2136
Description
export datasource name as expected by cli in search service
Fixes #2136
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Intermediate change (work in progress)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] Performed a self-review of my own code
[ ] npm test passes on your machine
[ ] New tests added or existing tests modified to cover all changes
[ ] Code conforms with the style guide
[ ] API Documentation in code was updated
[ ] Any dependent changes have been merged and published in downstream modules

@Surbhi-sharma1 revert the package-lock file changes

@Surbhi-sharma1 revert the package-lock file changes

Done


Add example for dealing with large object with aws s3 component

@davsclaus Thanks for yours help, and create this example to record what we have learnt.

Thanks a lot @rhtevan
If you want, you are welcome to write a blog post about this, we can put on the camel website
https://github.com/apache/camel-website
The blogs are in that repo, and you can find one of the existing blogs, and copy and modify.


Solved the 'Longest substring without repeating character' problem in Python

Made a website using HTML and CSS and added the code of 'Longest Substring Without Repeating Characters in Python'. Solves Issue #89

Have made the changes, kindly verify now

As you specified in the readme.md, I made a website and added the required code to it in python.


Document yield VS yield from

@kelunik updated

BTW: yield from -> what is this from? a PHP keyword? Can't find it in the manual

@prolic yield from is documented here : http://php.net/manual/en/language.generators.syntax.php#control-structures.yield.from

thanks @kelunik @jvasseur I added that link to the docs as well

Thanks!


Teensy 3.6 support

Hi!
Could this work in a teensy 3.6?
Do you see that it is feasible to use the 3.3Volt of the n64 to power the teensy?

It could probably work on Teensy3.6, however due to the reduced RAM, Transferpak emulation would not be supported.
I managed to get it to compile for a Teensy3.6 quite easily, however I dont have one so cannot test.
USB requires 5V. Boosting the B64 3.3V to 5V is outside the scope of this project. I dont want to overload/damage any N64s.

See https://github.com/Ryzee119/usb64/tree/teensy36 branch for Teensy3.6 build.
Completely untested.

This build will very likely need debugging/code changes, so providing a build is not ideal and I will not be able to support this much.
Not sure what your process is but I just did this to build it:
git clone https://github.com/Ryzee119/usb64.git --recursive
cd usb64
git checkout teensy36

Then open the folder in VSCode with the PlatformIO plugin installed

Thanks, I was finally able to compile and program it but it is completely dead. Tried for both ways: USB and hardwire (enable connected to GND)
In windows I do not see any serial port to try to connect via Putty. Any ideas?

Comment stuff out until you get something from the serial port, then add bits back in to identify where it's falling over


v2.1.0

A new view for mobile players! - Mobile now has a completely redesigned sheet popup that is far easier to use
Added ability to delete individual farmhands
Added a new page to manage your account and saves (under Account Settings)
Added an XP bar for farmhand skills
Added a search and location filter to Golden Walnuts
Simplified farmhand creation form
Fixed bug that would occur different user's would update the same co-op save file.
Fixed a crash when users would try and sign in and have an animated profile picture
Fixed an error with empty farmhands
Fixed a bug when editing monster kill stats
Fixed crab pot fish not showing up on the fishing page
Fixed a odd issue with some wiki links not being able to click
Fixed form validation issues
Fixed the Set Dating button showing on undatable villagers

LGTM


Flat roofs on houses in the UK

Flat rooves are rare in the UK on houses, A better default would probably be to assume gabled.

Additionally flat rooves in the UK on houses are unlikely to be covered with plant equipment like the textures used. asphalt rooves are much more common.
https://www.google.com/search?q=flat+roof+uk&rlz=1C1ONGR_en-GBGB987GB987&sxsrf=APwXEddAL1jFB0yAdU5hvCtoHyv_qSoMKg:1683127166109&source=lnms&tbm=isch&sa=X&ved=2ahUKEwj72OOmudn-AhXEUMAKHSUJA8AQ_AUoAXoECAEQAw&biw=1537&bih=797&dpr=1

I'm  fine with the flat default, as it's the most neutral one. F4 is doing the same.

I don't think there should be regional default roof shapes. This would be really confusing.

the only thing which is weird is the texture i feel, if it were to be an actual flat roof i'd be fine with tar paper and the other hinted stuff. but if no info is known then just put a neutral gray or red for tiles.

I agree, maybe should be used on building=industrial only or so.

To me the neutral flat is ok because it makes me understand that that place is missing a roof:shape value, if the value was more "common" (like gabled or hipped) I could assume it does have already the value and I would not improve the map

the only thing which is weird is the texture i feel, if it were to be an actual flat roof i'd be fine with tar paper and the other hinted stuff. but if no info is known then just put a neutral gray or red for tiles.

I like the textured flat roof, they look like ones I see around me.


Update pom.xml

Could you please elaborate on why this is needed?
Thanks.

Not needed. Closing this.


Dataset maturity

Output metadata takes dataset maturity from dataset if it is not specified in the config.
Testing in dev sandbox with geomedian gives expected results.

@omad should this be merged or is it no longer relevant/necessary?


Allow CLI flags thru the cmd

Problem
I need to add --flags and args to my process, but no such provision exists.
Discussion
https://github.com/sevenwestmedia-labs/nx-plugins/blob/980cdc158789880ebfbdedfdbc22007d149ff740/libs/nx-esbuild/src/executors/serve/executor.ts#L89
Can we add ...(options.args || []) to the final bit? Or however we get args thru with nodemon. For instance, my node build via esbuild needs --port=xyz and --hostname=xyz etc. Some i can pass thru the env, but not all

Sounds good, if I get a chance will add it, otherwise happy to accept a PR for this


Fix Snackbar length lint error

Description
Fixed Snackbar length lint error.
Validation

Manually validated snackbar appears when denying the screen capture permission
Passed CI

Submission Checklist

[x] The source has been evaluated for semantic versioning changes and are reflected in versionName under app/build.gradle
[x] The CHANGELOG.md reflects any feature, bug fixes, or known issues made in the source code

@kbagchiGWC No it was never ignored. It just started appearing today. I've seen this happen before with the linter where new lint errors get added into the android gradle plugin without updating the version

@kbagchiGWC No it was never ignored. It just started appearing today. I've seen this happen before with the linter where new lint errors get added into the android gradle plugin without updating the version


Update viewport state after moving the stack

Comment that raised this bug: https://github.com/OHIF/Viewers/pull/1643#pullrequestreview-395252065

Fix bug of clicking consecutive time in measurement item / rtstruct item or seg item to jump to image. If the user clicks the seg to jump after moving the stack, the viewport specific data is set but if the user moves the stack, the state continues the same because just the frame index (which is not tracked changed) so it doesn't cause a rerender in the second click.

Related react-cornerstone-viewport PR: https://github.com/cornerstonejs/react-cornerstone-viewport/pull/78
PR Checklist

[x] Brief description of changes
[ ] Links to any relevant issues
[ ] Required status checks are passing
[ ] User cases if changes impact the user's experience
[ ] @mention a maintainer to request a review

CI is failing
Also, if we're going to sync state for something that changes this frequently, do we need to memoize to reduce re-renders?

CI is failing
Also, if we're going to sync state for something that changes this frequently, do we need to memoize to reduce re-renders?

@dannyrb not sure what to memoize, this is a very specific flow which should not rerender much, maybe I'm missing something?

See above ^
Also, while we're here, can you lock react-cornerstone-viewport to a fixed version?

Done.

CI is failing
Also, if we're going to sync state for something that changes this frequently, do we need to memoize to reduce re-renders?

@dannyrb not sure what to memoize, this is a very specific flow which should not rerender much, maybe I'm missing something?

@igoroctaviano  (cc: @dannyrb as you were mentioned), would you guys be able to resolve these CI issues so we can merge, this issue is important to IDC. Cheers.

CI is failing
Also, if we're going to sync state for something that changes this frequently, do we need to memoize to reduce re-renders?

@dannyrb not sure what to memoize, this is a very specific flow which should not rerender much, maybe I'm missing something?

@igoroctaviano (cc: @dannyrb as you were mentioned), would you guys be able to resolve these CI issues so we can merge, this issue is important to IDC. Cheers.

@JamesAPetts I'll take a look right now. 👍

CI is failing
Also, if we're going to sync state for something that changes this frequently, do we need to memoize to reduce re-renders?

@dannyrb not sure what to memoize, this is a very specific flow which should not rerender much, maybe I'm missing something?

@igoroctaviano (cc: @dannyrb as you were mentioned), would you guys be able to resolve these CI issues so we can merge, this issue is important to IDC. Cheers.

Updated with master, now its passing.


Size of picture change!

Hello
I use this module to flip image or rotate image.
But, my problem is, What the initial size is not the same after.
example, My first image: 7.53Mo
When a rotation of 180°:  14Mo
What?

Probably to do with the quality of the saved image. JPEGs are saved at quality 100 by default but normally it is better to reduce it to 80 or 60.
Jimp.read("image.jpg").then(image => {
    image.rotate(180)
    .write("image-rotate-100.jpg")
    .quality(60)
    .write("image-rotate-60.jpg");
});

PNGs have separate compression settings.

Is this the final answer? It doesnt make sense to me that an unmodified image doubles it size just because the quality is 100%. By reducing it to 80 or 60 must be even lower than the original size.


Add clarification about minitest 5.6.0 errors

Add clarification about minitest 5.6.0 errors

Oops. My mistake, I had loaded 5.5.1 into my grape app accidently


FSI: Implement zero-copy read where possible

https://github.com/intel-ai/omniscidb/pull/94

I'm waiting for a response from omnisci team, to discuss some architectural problems.

we've discussed the patch with Alex, and I've reworked my pr.

@fexolm What is the status?

PR is under review - https://github.com/omnisci/omniscidb/pull/604

PR was merged


Remove BinaryDecoder

BinaryDecoder was used to decode constant values when we stored constants in files.
Since we have moved to embedding constants in MLIR IR, this decoder is not used anymore.
Signed-off-by: Tung D. Le<EMAIL_ADDRESS>

Jenkins Linux s390x Build #3489 [push] Remove BinaryDecoder (#1... started at 23:04

Jenkins Linux ppc64le Build #2604 [push] Remove BinaryDecoder (#1... started at 23:10

Jenkins Linux amd64 Build #3497 [push] Remove BinaryDecoder (#1... started at 22:04

Jenkins Linux s390x Build #3489 [push] Remove BinaryDecoder (#1... failed after 2 min 18 sec

Jenkins Linux ppc64le Build #2604 [push] Remove BinaryDecoder (#1... passed after 58 min

Jenkins Linux amd64 Build #3497 [push] Remove BinaryDecoder (#1... passed after 1 hr 7 min


feat(v2): polyfill automatically depending on browserslist

Motivation
Fix #1784 Close #1808

Set @babel/env useBuiltins to usge. This will automatically use browserlist and import polyfills required

After traversing our codes with babel, babel can automatically figure out required polyfill. So
const set = new Set([1, 2, 3]);
[1, 2, 3].includes(2);

in browsers like ie11 will be replaced with
import "core-js/modules/es.array.includes";
import "core-js/modules/es.array.iterator";
import "core-js/modules/es.object.to-string";
import "core-js/modules/es.set";

const set = new Set([1, 2, 3]);
[1, 2, 3].includes(2);

This does not import whole corejs, its smart polyfilling.
Note:  Only code that are traversed by babel loader is used to determine what polyfills are required.

Modified TerserWebpackPlugin terserOptions like CRA. Otherwise, the minifier will break in ie11

Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
Promise and Object assign is polyfilled in ie11.

Surprisingly this increase main bundle size by ~150kb o.o.

Ie11 compat isnt too needed so im gonna drop this for future ref only. 100kb seems  huge

Safe to review. Bundle size increase is expected. For our case its ~50kb

This is bad, we + migrated sites need to rename so many things. Thankfully still in beta. We should rename it asap.

I tried to rename things up. But found the eslint quite annoying. I can't do
<Link
              className={styles.indexCtasGetStartedButton}
              to={useBaseUrl('docs/introduction')}>
              Get Started
            </Link>

Because React Hook "useBaseUrl" cannot be called inside a callback. React Hooks must be called in a React function component or a custom React Hook function  react-hooks/rules-of-hooks
I wonder if we shall still need to rename ... @yangshun

Renamed and fix eslint issue. Pls re-review

Since withBaseUrl/useBaseUrl is only used to extract the baseUrl from docusaurus.config, would it be better to just import that file from within withBaseUrl so that we don't need it to be a hook and can avoid all the hook rules?

I believe there is a past discussion on this. The reason why importing config directly is bad is related to webpack module concatenation.
If there are 10 pages importing withbaseUrl, when siteconfig change in dev server, all 10 files will need to be hot module replaced. Affecting dev reload performance. This can make our hot reload go as high as few seconds. Each page code splitted bundle size will also increase
Its because the module graph is no longer one flow.
I recommend going with second option for now to throw the error.

PS: I think there's no need to add withBaseUrl because it wont fail the build, only the client side (and people had to open console to actually figure it out)

The error from webpack is quite nice. When we release alpha.28, i'll put the note on breaking changes


Correction for raw affiliation Aeres University of Applied Sciences, Wageningen, The Netherlands

Correction needed for raw affiliation Aeres University of Applied Sciences, Wageningen, The Netherlands
raw_affiliation_name: Aeres University of Applied Sciences, Wageningen, The Netherlands
new_rors: 03jnx2c74
previous_rors: 015d5s513
works_examples: W2515935174
contact: 9c8759cd6cbd7c5ba8cf10407cd48dc7:0440173ef17b4da0 @ aeres.nl

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Bump K3os to v0.20.4 k3s1r0

Sorry, My commit caused some conflicts

https://github.com/rancher/k3os/issues/686

I did a sanity check. VM creation/console access/migration, VLAN network/DHCP works


Missed frames on nvidia 390 with ubuntu 16.04

Good day!
I used your app and found some stuttering in rendering to Vive with GTX 970 and latest SteamVR, OpenVR and OSG from git. hellovr_opengl example works fine without missed frames.
Also I found that at line 715 in hellovr_opengl_main.cpp there is some hack with glFinish();
maybe we need to insert this hack to osgopenvrviewer.

Hi there, thanks for your interest in this project. However, I have no current plans to maintain it as I have moved to Unity for my VR developments.


Exception raised when showing an SVG

Hi all,
Thanks for all the amazing work you're doing.
I'm using the v5.1.0 of the echart panel.
My goal is to display an SVG (a building) and to update it depending on the result of request.
Everything works fine except the first time the dashboard is displayed as the following error is raised:
ECharts Execution Error
Cannot read properties of undefined (reading 'regions')
TypeError: Cannot read properties of undefined (reading 'regions')
    at t.getFilledRegions (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:1552681)
    at e.optionUpdated (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:1553170)
    at e.eval (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:552990)
    at Array.forEach (<anonymous>)
    at wt (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:379245)
    at e.eval (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:552607)
    at t.topologicalTravel (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:545959)
    at e._mergeOption (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:552103)
    at Rd (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:557265)
    at e._resetOption (http://<IP_ADDRESS>:3000/public/plugins/volkovlabs-echarts-panel/module.js:2:551368)

When the dashboard is refreshed, the message disappeared and left a blank at the panel position.
When going back (button with the left arrow in the web browser) and "go ahead" (button with the right arrow in the web browser), the error is not raised and the SVG is displayed.
Here is the definition of the dashboard
option = {
  tooltip: {
  },
  series: [],
  geo: {
    map: 'building',
    type: "map",
    zoom: 2.0,
    regions: [
      {
        name: "dcfc",
        itemStyle: {
          normal: {
            opacity: 0
          }
        },
        label: {
          show: false
        }
      }
    ],
    emphasis: {
      focus: 'self',
      itemStyle: {
        color: null
      },
      label: {
        position: 'bottom',
        distance: 0,
        textBorderColor: '#fff',
        textBorderWidth: 2
      }
    },
    blur: {},
    select: {
      itemStyle: {
        color: '#b50205'
      },
      label: {
        show: false,
        textBorderColor: '#fff',
        textBorderWidth: 2
      }
    }
  }
};

I have been stuck for 2 months and any help will be appreciated,
Thanks in advance,
Paul

@XIIIVI I think you have a race condition.  If you do async, then update it to use sync more or update options when SVG is loaded.
We have an example how to load the SVG file from the data source: https://echarts.volkovlabs.io/d/X1mkMIFVz/geo-map?orgId=1&editPanel=8.
Let me know if it helps.

Hi Mikhail,
Thanks for your feedback.
I load the map like this
fetch('public/img/icons/building.svg')
  .then(function (response) {
    return response.text();
  })
  .then(function (svgText) {
    const parser = new DOMParser();

    svgDOM = parser.parseFromString(svgText, 'image/svg+xml').documentElement;

    echarts.registerMap('building', { svg: svgDOM });
  })
...

To make it synchronous, I changed into
async function loadMap() {
  await fetch('public/img/icons/building.svg')
  .then(function (response) {
    return response.text();
  })
  .then(function (svgText) {
    const parser = new DOMParser();

    svgDOM = parser.parseFromString(svgText, 'image/svg+xml').documentElement;

    echarts.registerMap('building', { svg: svgDOM });
  })
}

try {
  loadMap();
} catch (error) {
  console.error(error);
}

The SVG file is embedded with Grafana.
In your example, I noticed you are using "visualmap".
If I change "visualmap" into "geo", your example got the same error.
Many thanks for your precious help,
Paul

@XIIIVI When do you return ECharts options after the try statement? Then it's not sync.
The options returned before your SVG loaded and map registered.

@XIIIVI Totally forgot that it's explained in the documentation:
https://volkovlabs.io/plugins/volkovlabs-echarts-panel/maps/geojson/#dynamic-maps-loading

Hi Mikhail,
This is the point I missed.
I'm very grateful for this, you saved my life :)
Once again, many thanks for your beautiful work.
All the best,
Paul

@XIIIVI Thank you for confirming.
Let us know if there is anything else.


Refactor rocksdb unwraps to handle errs more tastefully

Info

A pass on the unwraps, the clone and default may need other functions to do this, if there is a place in the codebase that does this as we want, a link will help me ensure we accomplish that with the pattern we want.
Unsure if this PR is actually useful, let me know.

Marked as draft to prevent accidental merges while it's still WIP

Proposing we get this in and worry about default / clone implementations later across the entire codebase.


mesh: remove dependency on verified layer

Motivation

Closes #3744

Changes

mesh can determine which block to apply without knowing the latest verified layer with the following heuristic:

use validity from tortoise updates
if 1 fails, get block validity from database
if there are no valid blocks, apply empty block for the layer
if the block validity is not determined, use the hare output

remove layer hash that captured all valid blocks in the layer. this field is unused. removing it so we don't need to add code to update layer hash when block validity changes.

bors try

bors merge

bors merge

bors merge

bors merge


SSL does not work

Hi!
AppDaemon 3 is no longer available.
How to make version 4 work with SSL?

This add-on provides AppDaemon itself. For instructions on using SSL with AppDaemon, you should check the AppDaemon documentation.

This add-on provides AppDaemon itself. For instructions on using SSL with AppDaemon, you should check the AppDaemon documentation.


Unrelated request - Dimensional Door ID Table fixes

There is an amazing mod called "Dimenionsal Doors" Unfortunatly the way it currently handles Dimensional IDS is poor at best there is a glitch which causes it to re-Assign an ID when a server reloads and since the Dims are not hard set it sometimes overwrites existing ID tables making it so a world corrupts. I was wondering if you could attempt to fix it since your team seems good at ID table management. By the way it is very much open source.
https://github.com/CannibalVox/DimDoors/releases

I will look into this...


Integrate perfio.s3 reader [databricks]

Provides an option spark.rapids.perfio.s3.enabled to improve S3 reads
Adds logging for spark-rapids-private version info properties
Removes duplicate logging of plugin and cudf versiion properties in  checkCudfVersion

Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

build


stream info: expand protocol set

As a follow-up to https://github.com/envoyproxy/envoy/pull/10383, I would like to differentiate between connections handled by HCM vs tcp_proxy. The protocol field in the stream info seems appropriate, but right now it's an enum with HTTP protocol versions. The proposal is to change it to be a bit mask, and and add a few values:

TCP for all TCP including HCM traffic
HTTP as a catch-all for HCM traffic
if there is interest, we can also add gRPC in grpc_stats, mysql, redis, and others.

cc @mandarjog @zuercher for thoughts

This sounds reasonable to me.

if there is interest, we can also add gRPC in grpc_stats, mysql, redis, and others.

My only comment here would be try to figure out a way to keep any extension protocols from creeping into the core, so I would probably try to figure out some way to make this extensible with well known bits and then potential optional bits or a repeated set of strings/codes/whatever.

Regarding the open-ended protocol set, is there a prior art or preference in Envoy?
Option 1: Make it a bit mask and reserve some base number of bits for core, and let the extensions pick the rest (in ad-hoc manner).
Option 2: Create a factory registration pattern that would register a protocol value via a static initializer.

Option 2: Create a factory registration pattern that would register a protocol value via a static initializer.

This would be really nice but more work. Would definitely prefer this. The issue with this is wouldn't necessarily be consistent across deployments. Can we just use a repeated set of string codes or something?


Upload apk to play store with github actions

Android apk and aab are being built by the code in  #5


lazy load background image

First of all, thank you for your wonderful library.
I don't see anything about lazy load for background image in the doc, it would be nice to have this feature.

+1.
would be nice to have.
but not only for internal images, but also for .swiper-slide
Thanks


The SDK section at the top should include links to download them

This was a great page to find, but then I had to figure out how to get the sample code / SDK packages myself.  It would be good to include these in the descriptions of each sdk.
Also, is this all of the SDKs that support daemon auth?

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

ID: 08bcd9e7-d45f-1867-185a-c74636a9f448
Version Independent ID: 20e2b7a4-12b0-8cbf-d36f-63a9b527b7da
Content: Configure daemon apps that call web APIs - Microsoft identity platform
Content Source: articles/active-directory/develop/scenario-daemon-app-configuration.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@hpsin
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@hpsin
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@msewaweru I chatted with Hirsch on this one, there are a couple ways we can go about fixing (one is quick, other is more involved with possible use of INCLUDEs). Passing this over to you for scheduling next sprint. #reassign:mmacy

@msewaweru I chatted with Hirsch on this one, there are a couple ways we can go about fixing (one is quick, other is more involved with possible use of INCLUDEs). Passing this over to you for scheduling next sprint. #reassign:mmacy

ID
Title
Iteration Path
Assigned To
Area Path

1290345
[ISSUE] Add links to download the SDK packages/ code samples (69722)
Technical Content\Backlog
Eunice Waweru
Technical Content\app-dev

Will be fixed by https://github.com/MicrosoftDocs/azure-docs-pr/pull/146577.
#reassign:msewaweru

https://github.com/MicrosoftDocs/azure-docs-pr/pull/146577 has been merged. Proceeding to close this issue.
#please-close


Leaderboard generated queries have additional order true that doesn't work well for druid

SELECT "dimension", sum(cnt) as "measure_0" FROM "datasource" WHERE 1=1 AND "__time" >= '2022' AND "__time" < '2024' AND ("daypart"  IN ('daytime'))  GROUP BY "datasource" ORDER BY true, "measure_0" DESC LIMIT 250

Above generated query has ORDER BY true, "measure_0", the query generated should have ORDER BY "measure_0" only.

The query having ORDER BY true is explained by Druid as "queryType": "groupBy"
The query with no ORDER BY true is explained by Druid as "queryType": "topN"
In Druid, topN and timeseries queries are generally more efficient than groupBy queries. This is because the query engine for topN and timeseries has specific optimizations for single-dimension grouping and sorting use cases.
topN queries are designed for simple, single-dimension grouping and ordering scenarios. They can be faster and more memory-efficient than groupBy queries, but they may return approximate results, especially when working with large datasets or high cardinality dimensions. This trade-off between accuracy and performance is often acceptable for many use cases.
timeseries queries are optimized for time-based aggregations and filtering. They can efficiently handle time-based data and are typically faster than equivalent groupBy queries.
On the other hand, groupBy queries are more versatile and can handle more complex aggregation and ordering scenarios. They return exact results, but they can be slower and more resource-intensive than topN and timeseries queries.


Pin requirements versions

https://nvie.com/posts/pin-your-packages/

https://github.com/yzernik/squeaknode/pull/1310

https://github.com/yzernik/squeaknode/pull/1311

https://github.com/yzernik/squeaknode/pull/1313


Excluding TOC elements does not work with 'toc: true' in the YAML Front Matter

[ ] This is a question about using the theme.
[ ] This is a feature request or enhancement.
[x] I believe this to be a bug with the theme.

[ ] I am using the latest version of the theme.
[ ] I have updated all gems with bundle update.
[ ] I have tested locally with bundle exec jekyll build.

Environment informations

Minimal Mistakes version: 4.10.0
Jekyll version: 3.6.2
GitHub Pages hosted: yes
Operating system: MacOS 10.13.3

Expected behavior

I expect to be able to control what goes in my TOC by using standard Kramdown, i.e. by assigning {:.no_toc} to a header.
Steps to reproduce the behavior

Take a simple MD file:
---
title: "TOC Test"
---

# Header 1

## Header 2

### Header 3
{:.no_toc}

The feature works correctly with the TOC include, i.e. by including {% include toc %} the generated TOC will be
- Header 1
   - Header 2

However the TOC include is going to be deprecated, so I'd prefer to avoid doing it this way.

The feature does not work correctly when enabled via the YAML Font Matter , i.e. by using toc: true the generated TOC will be
- Header 1
   - Header 2
   - Header 3

This doesn't work because toc: true uses JavaScript too build the table of contents instead of Kramdown.
To exclude headers you'll need to use the toc include helper instead.

To exclude headers you'll need to use the toc include helper instead.

Indeed, it's what I'm using right now. But I'm worried by the fact that the toc helper will be deprecated, so the "old" pages that use the toc helper won't work anymore with the new version of the theme.

Even if deprecated you can copy _includes/toc into your repo and still use it.

Ok, thank you!
Btw: thanks for your awesome theme! 😄


enhancement the detect touch screen logic

find some broswers(e.g. firefox 30 in Mac OS 10.9) will has the ontouchstart attr in non-touchable device as a default behaviour, in this case the click function won't work.
I occur this issue with FF 30 in Mac OX 10.9, using the latest sidr source code.

Genius. @artberri can you pull this in?

Preparing a new version with a fix for this https://github.com/artberri/sidr/tree/reborn


Minor improvs to hhnet script

set down interface before enslaving it, to prevent errors if it is already up
limit the attempts in get_ip to 30 to avoid stalling forever

After this passes CI, it looks good to me.


Recreating Modified Generated Files

I may be missing something here, so please correct me.
JavaCC (through standard terminal usage) allows you to modify auto-generated AST classes, and when recompiling it will not automatically recreate them removing all modifications. I cannot get the same behaviour using this plug-in.
Am I missing something?

Without a proper example, it's a little hard for me to tell, but if you are making modifications to the generated classes under build, then of course, it gets cleaned up on a new build. If you want to use your own version of a generated class, simply provide your version under the same java package as where it would be generated. See Issue #13 and Issue #16 for the solution and examples.
Note that you would also benefit from having your modifications version-controlled.
Let me know if that answers your question.

In the first instance, the jjtree file is in src/main/javacc (which is the compileJjtree and compileJavacc tasks' input and output directory) with no other files so naturally they are generated the first time. Once they are generated, however, classes with the exact name already exist for the second build, so shouldn't be re-generated?
build.gradle: https://gist.github.com/ChrisBrenton/42e78537464262ee73a4b3083d0f3a60

I don't understand why you override the inputDirectory and outputDirectory properties. I think the behaviour you are looking for is broken by you having both the input and ouput in the same directory. The default behaviour will do the right thing in using the output of JJTree as the input for JavaCC. If you have Java classes in the JavaCC input directory, it will not regenerate these classes in the output directory.
So, bottom line, I suggest you leave out the input/output configuration, do a one-off generation of your classes, move the ones you need to modify so src/main/javacc, make the modifications and add that to version control. Then, next time you regenerate, these classes won't be generated and your custom AST class will be used.
It's usually bad practice to generate stuff in source folders, hence why the default behaviour does not go this way. You don't need to worry about where the generated classes are, they will still be packaged correctly.

Closing this issue.


Any way to add extractor-args to download command

Hi,
Is there any way to add command arguments extractor-args when passing to yt-dlp download process?
Or is it possible to let user pass extra command arguments when executing the bot?
Thanks for the bot!

Currently there's no way to pass extractor args to yt-dlp, as the used yt-dlp lib (goutubedl) does not support passing that to yt-dlp.


netpbm: update to 10.82.01

Motivation for this change
update to new version of netpbm
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[X] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: netpbm
Partial log (click to expand)
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/pamstretch-gen: interpreter directive changed from "/bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/pnmmargin: interpreter directive changed from "/bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/pnmquant: interpreter directive changed from "/bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/pnmquantall: interpreter directive changed from "/bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/pgmcrater: interpreter directive changed from "/bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/ppmrainbow: interpreter directive changed from "/bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/ppmtomap: interpreter directive changed from " /bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/pamfixtrunc: interpreter directive changed from "/bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
/nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01/bin/.ppmquant-wrapped: interpreter directive changed from "/bin/sh" to "/nix/store/0aa1ngxygf1iqanfbpdb10klc7r343c0-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/928f59z7143b54i7s3c9dhwp6b1fmhs1-netpbm-10.82.01...

Success on aarch64-linux (full log)
Attempted: netpbm
Partial log (click to expand)
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01/bin/ppmfade: interpreter directive changed from "/bin/sh" to "/nix/store/23kq49wszxq7k41dqfk82q5vqwlq5sww-bash-4.4-p19/bin/sh"
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01/bin/pnmflip: interpreter directive changed from "/bin/sh" to "/nix/store/23kq49wszxq7k41dqfk82q5vqwlq5sww-bash-4.4-p19/bin/sh"
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01/bin/pnmtoplainpnm: interpreter directive changed from " /bin/sh" to "/nix/store/23kq49wszxq7k41dqfk82q5vqwlq5sww-bash-4.4-p19/bin/sh"
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01/bin/anytopnm: interpreter directive changed from "/bin/sh" to "/nix/store/23kq49wszxq7k41dqfk82q5vqwlq5sww-bash-4.4-p19/bin/sh"
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01/bin/pcdovtoppm: interpreter directive changed from "/bin/sh" to "/nix/store/23kq49wszxq7k41dqfk82q5vqwlq5sww-bash-4.4-p19/bin/sh"
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01/bin/hpcdtoppm: interpreter directive changed from "/bin/sh" to "/nix/store/23kq49wszxq7k41dqfk82q5vqwlq5sww-bash-4.4-p19/bin/sh"
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01/bin/pbmtox10bm: interpreter directive changed from "/bin/sh" to "/nix/store/23kq49wszxq7k41dqfk82q5vqwlq5sww-bash-4.4-p19/bin/sh"
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01/bin/manweb: interpreter directive changed from "/usr/bin/perl -w" to "/nix/store/ffzbcnc5yc57sqzkznfb5f9dixqvkxkv-perl-5.24.3/bin/perl -w"
checking for references to /build in /nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01...
/nix/store/i5711ixrnipr464mfswzpplw0zlwa958-netpbm-10.82.01


12029 disable export

This PR implements #12029.

Tested in local (not need to check it in staging). All good 👍 .


设置透明黑色没有生效 bg-black-50

UnoCSS version
0.53.5
Describe the bug
bg-black-50 好像没有生效 透明度
Reproduction
https://unocss.dev/interactive/?s=bg-black-50
System Info
No response
Validations

[X] Read the Contributing Guidelines.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

https://unocss.dev/interactive/?s=bg-black:50


Exported PDF is not recognised by Voiceover on iOS

I'm not sure if this is a puppeteer problem, a chromium problem, or an iOS problem.
When I export a basic html page using puppeteer, VoiceOver for MacOS reads the PDF correctly, but VoiceOver on iOS ignores the content or the PDF completely.
Here is the html I'm testing :
<!DOCTYPE html>

This is a test heading

This is some text in a paragraph

This is a test heading level 2

This is some text in a paragraph

Another paragraph

And the code to generate the PDF :
    const browser = await puppeteer.launch({
      args: ['--disable-dev-shm-usage', '--headless',  '--no-sandbox', '--export-tagged-pdf'],
      headless: true
    });

   const html = `${ HTML FROM ABOVE EXAMPLE }`;

   const page = await browser.newPage();
   await page.goto(`data:text/html;charset=UTF-8,${html}`, {
        waitUntil: 'networkidle0'
   });

   const snapshot = await page.accessibility.snapshot();
   console.log(snapshot);

   result = await page.pdf();

   // ...save the resulting buffer to disk

As you can see, I print the accessibility snapshot and all seems correct :
{
  role: 'RootWebArea',
  name: '',
  children: [
    { role: 'heading', name: 'This is a test heading', level: 1 },
    { role: 'StaticText', name: 'This is some text in a paragraph' },
    {
      role: 'heading',
      name: 'This is a test heading level 2',
      level: 2
    },
    { role: 'StaticText', name: 'This is some text in a paragraph' },
    { role: 'StaticText', name: 'Another paragraph' }
  ]
}

Perhaps someone can see a setting or something that I may be missing ?
Using Puppeteer 19.7.0.

Thanks for the report. It looks like a problem with the iOS voice over. But perhaps you could open an issue at crbug.com first? Especially, if it reproduced with PDFs printed using the regular browser window (without Puppeteer). We have no means of fixing this issue inside Puppeteer because we only deliver the PDFs generated by the browser.

Thanks for the report. It looks like a problem with the iOS voice over. But perhaps you could open an issue at crbug.com first? Especially, if it reproduced with PDFs printed using the regular browser window (without Puppeteer). We have no means of fixing this issue inside Puppeteer because we only deliver the PDFs generated by the browser.

Thank you for your reply, especially the suggestion about just printing using the regular browser. I did exactly that with the HTML above, and lo and behold, VoiceOver does not read it on iOS.
I'll elevate the issue to Chromium and/or Apple !