Display the current directory in the file prompts

Continuing from #300, adding ./ (and possibly ../ for consistency, but it's nowhere as important) to the file prompts will avoid confusing the users of other completion systems that expect the Tab Enter combo to act on the completed directory, not the first file in this directory.
It's especially important for cases like using dired-do-copy with dired-dwim-target enabled as right now Selectrum "suggests" overwriting the first file in the target directory.

It's especially important for cases like using dired-do-copy with dired-dwim-target enabled as right now Selectrum "suggests" overwriting the first file in the target directory.

For me it suggests the other directory, how can I reproduce this?

It's especially important for cases like using dired-do-copy with dired-dwim-target enabled as right now Selectrum "suggests" overwriting the first file in the target directory.

For me it suggests the other directory, how can I reproduce this?

confusing the users of other completion systems that expect the Tab Enter combo to act on the completed directory

While I understand it is confusing at first you just need to press Enter to act on the directory, once you get used to it is nice but as I mentioned in #300 we can add an option to enable the other behaviour.

confusing the users of other completion systems that expect the Tab Enter combo to act on the completed directory

While I understand it is confusing at first you just need to press Enter to act on the directory, once you get used to it is nice but as I mentioned in #300 we can add an option to enable the other behaviour.

Initial state with my cursor on foo_file:

I press R M-n (which acts as an on-demand dired-dwim-target, my bad if the behavior actually differs!):

I press Enter to demonstrate that the existing file is indeed pre-selected:

Initial state with my cursor on foo_file:

I press R M-n (which acts as an on-demand dired-dwim-target, my bad if the behavior actually differs!):

I press Enter to demonstrate that the existing file is indeed pre-selected:

Yes, I do have selectrum-mode enabled.

Yes, I do have selectrum-mode enabled.

Okay, thanks I have dired-dwim-target set to t in which case the prompt is selected, I have to check why this isn't the case when using M-n.

Okay, thanks I have dired-dwim-target set to t in which case the prompt is selected, I have to check why this isn't the case when using M-n.

Okay the case of not selecting the prompt is fixed, thank you for making me aware of minibuffer-default which we ignore before. What remains is adding an option for including ./ for people which would prefer that.

Okay the case of not selecting the prompt is fixed, thank you for making me aware of minibuffer-default which we ignore before. What remains is adding an option for including ./ for people which would prefer that.

Awesome, thank you!

Do you have other use cases where ./ would be preferable? After reflection I would prefer not adding an option for it, I think this would be better placed in the potential fido/ido-emulation-mode mode as mentioned in #218 (the setup lives also on the wiki now).

Do you have other use cases where ./ would be preferable? After reflection I would prefer not adding an option for it, I think this would be better placed in the potential fido/ido-emulation-mode mode as mentioned in #218 (the setup lives also on the wiki now).

No, ever since your last fix I was pretty comfortable in that regard. We can close this issue if you consider this option unintended.

No, ever since your last fix I was pretty comfortable in that regard. We can close this issue if you consider this option unintended.

Great, thanks :)

Great, thanks :)

@clemera I was hoping you'd put that option in. I have a hacked version of selectrum--completing-read-file-name without the (not (member i '("./" "../"))), and my life is infinitely better for it.  Any way I could persuade you? (Or submit a pull request?)

@clemera I was hoping you'd put that option in. I have a hacked version of selectrum--completing-read-file-name without the (not (member i '("./" "../"))), and my life is infinitely better for it.  Any way I could persuade you? (Or submit a pull request?)

If you still have the predicate in there you probably want to update because it caused slow down problems with tramp and slow file systems. Do you also want the ../ candidate? Could you elaborate why you want them, maybe I could convince you that the "selectrum way" makes them unnecessary.

If you still have the predicate in there you probably want to update because it caused slow down problems with tramp and slow file systems. Do you also want the ../ candidate? Could you elaborate why you want them, maybe I could convince you that the "selectrum way" makes them unnecessary.

@clemera When I select a directory, I use tab to descend down the directory hierarchy until I get to the directory I want. When I press RET, with the Selectrum way, I don't get the directory I've just completed but the first subdirectory of that in the list.
This is especially pernicious when I'm selecting a directory for saving a file, say - the result is usually wrong, and if I don't notice the problem then, I later puzzle over where that file went.
I guess I should do a RET instead of the last Tab, but I haven't been able to retrain my brain to do that - I certainly tried for a while. Leaving ./ in the list completely solves my problem.
BTW: Many thanks for the work on Selectrum - it's a genuine pleasure to use!

@clemera When I select a directory, I use tab to descend down the directory hierarchy until I get to the directory I want. When I press RET, with the Selectrum way, I don't get the directory I've just completed but the first subdirectory of that in the list.
This is especially pernicious when I'm selecting a directory for saving a file, say - the result is usually wrong, and if I don't notice the problem then, I later puzzle over where that file went.
I guess I should do a RET instead of the last Tab, but I haven't been able to retrain my brain to do that - I certainly tried for a while. Leaving ./ in the list completely solves my problem.
BTW: Many thanks for the work on Selectrum - it's a genuine pleasure to use!

Thanks, that is great to hear! Getting rid of habits can be hard ;) You could use a custom refine function to fix that:
(defun selectrum-refine-candidates-using-completions-styles-with-dots (input candidates)
  "Use INPUT to filter and highlight CANDIDATES.
Uses `completion-styles'."
  (let ((cands (selectrum-refine-candidates-using-completions-styles input candidates)))
    (if (and (eq (selectrum--get-meta 'category) 'file)
             (string-empty-p input))
        (cons "./" cands)
      cands)))

If you are using presicent you could replace selectrum-refine-candidates-using-completions-styles with prescient-filter. Would that work for you?

Thanks, that is great to hear! Getting rid of habits can be hard ;) You could use a custom refine function to fix that:
(defun selectrum-refine-candidates-using-completions-styles-with-dots (input candidates)
  "Use INPUT to filter and highlight CANDIDATES.
Uses `completion-styles'."
  (let ((cands (selectrum-refine-candidates-using-completions-styles input candidates)))
    (if (and (eq (selectrum--get-meta 'category) 'file)
             (string-empty-p input))
        (cons "./" cands)
      cands)))

If you are using presicent you could replace selectrum-refine-candidates-using-completions-styles with prescient-filter. Would that work for you?

Perfect, many thanks! (prescient-filter did not work, but selectrum-refine-candidates-using-completions-styles-with-dots did.)

Perfect, many thanks! (prescient-filter did not work, but selectrum-refine-candidates-using-completions-styles-with-dots did.)

For using prescient-filter you would have to install the prescient package, or you can also use orderless which uses completion-styles, see the README for details.

For using prescient-filter you would have to install the prescient package, or you can also use orderless which uses completion-styles, see the README for details.

@clemera Could you clarify what you mean by "replace withprescient-filter" - isn't that exactly what selectrum-prescient` does?

@clemera Could you clarify what you mean by "replace withprescient-filter" - isn't that exactly what selectrum-prescient` does?

@clemera Aah, never mind - I figured it out. Sorry for the hassle and again thanks for your help!

@clemera Aah, never mind - I figured it out. Sorry for the hassle and again thanks for your help!

@mikesperber what did you end up using? I too am overwriting files in dired on accident due to differences between what I was used to with ivy.

What @clemera said:
(defun selectrum-refine-candidates-using-completions-styles-with-dots (input candidates)
  "Use INPUT to filter and highlight CANDIDATES.
Uses `completion-styles'."
  (let ((cands (prescient-filter input candidates)))
    (if (and (eq (selectrum--get-meta 'category) 'file)
             (string-empty-p input))
        (cons "./" cands)
      cands)))

(setq selectrum-refine-candidates-function #'selectrum-refine-candidates-using-completions-styles-with-dots)

Ah! Thank you. That works perfectly.


feat/update-playwright-version-to-1.44: updated playwright version to 1.44

Description:
updated playwright version to 1.44
Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[ ] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

Hey there and thank you for opening this pull request! 👋🏼
We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.
Details:
No release type found in pull request title "feat/update-playwright-version-to-1.44: updated playwright version to 1.44". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - fix
 - hotfix
 - feat
 - feature
 - chore

/deploy-review


fs.existsSync is not a function

Getting this error on the deprecated function when I add your library to my project.

yeah i got the same thing as @dpgailey


Added Fork to list of git GUIs for Mac

Hello,
I'm developing a new free Git client for Mac called Fork. I'd like to add it to the list of git clients.
I made the change and tested it on my local machine, everything looks fine. Please let me know if you have any question.
Thank you,
Dan

Thanks, looks good (and good luck on your new client).


From >= v0.33.1 pyocd zephyr  plugin not working with zephyr2.5

pyocd zephyr  plugin not working with zephyr v2.5 and older. Investigation off problem show, that pyocd change debug information variables names.
It was not good idea.  New zephyr threads debug information support old names as alias. So not was necessary to change threads debug names.
Edit: pyocd change names from v0.33.1
#Required Zephyr symbols.
ZEPHYR_SYMBOLS = [
"_kernel",
"_kernel_thread_info_offsets",
"_kernel_thread_info_size_t_size",
]
Old names < v0.33.1:
# Required Zephyr symbols.
ZEPHYR_SYMBOLS = [
"_kernel",
"_kernel_openocd_offsets",
"_kernel_openocd_size_t_size",
]

If I try downgrade into  version v0.30, then pyocd not starting, because packages problems.


Review if Thumbprints are failing because of 'casesensitivity'

Customer reported to use a workaround:
http://stackoverflow.com/questions/25593435/jwtsecuritytokenhandler-4-0-0-breaking-changes/25625447?noredirect=1#comment49933145_25625447
Check and see if we need to adjust for this.
Closing this requires tests to be written.

@brentschmaltz I don't think this is an issue in 5.x. It seems to be related to ResolveIssuerSigningKey method which is not currently implemented in 5.x. We can drop this off of beta7 list I guess?

This is not an issue in 4.x too. The customer should use correct SecurityKey or SecurityToken when setting TokenValidationParameters. In the reported issue, customer should use X509SecurityKey instead of RsaSecurityToken.


Add support for js promises, and wrap them in a dart future

see https://github.com/dart-lang/polymer_elements/blob/master/lib/src/platinum-push-messaging/platinum-push-messaging.html#L382 as an example

@jakemac53  for me above link is broken -  isn't this: https://github.com/dart-lang/polymer_elements/blob/400b1d7571a95b6799404a883408e9d7fc4bd0d3/test/common.dart#L127  what you meant ?

yep thanks, updated comment


use @soroban-react library

Hello @altugbakan  , how are you?
I see that you are using a lot of functions that are included in the @soroban-react library.
Would you like to move into use the library instead? If you say yes I can make the transition and offer a PR!

Hey!
Thank you for the library functions. I used just parts of the library for custom user flow, as I needed more precise control like non-automatic login prompt.
I’m not sure if I will be improving upon this project, so you don’t need to spend time on this.
Thanks!


[REQ] Can you make the old roblox ui app

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
i mean i 2022 roblox has removed the old roblox ui app and make this laggy ui app, can you make a mod or costumaztion to bring back to the old one

icl i agree with this but the thing is that the old roblox used to be website based meanin there is no actual app launcher. you will normally load into your game from wbesite. but by all means i think they can make the app to its original state

i think this is what it used to look like

can u also resend the image u sent? it not showing up correctly

Not possible


Allow the package to be installed solely by pip via PyPI

The yamm package is ready to be installed with pip but still needs to uploaded to PyPI. Once yamm is added to PyPI, users will be able install yamm solely by pip.

You should check the install_requires in setup.py.  You are only listing osmnx and pygeos, but the environment.yaml also lists folium and requests and requirements.

I think I'm blocked by this so let me know if it needs another set of eyes.

FYI yamm is already on pypi: https://pypi.org/project/yamm/
Perhaps this is why #50 was created?

Closing this issue now that the package has been renamed.


Fix build error - error: function "torchao::marlin_24_gemm" has already been defined (previous definition at line 83)

For CUDA_ARCH < 800
https://github.com/pytorch/ao/blob/3ac2ab8c9ffb0a624b48a51c105b0a780cafe483/torchao/csrc/cuda/sparse_marlin/marlin_kernel_nm.cu#L55-L57
we get multiple definition of marlin_24_gemm due to incorrect placement on endif.
https://github.com/pytorch/ao/blob/3ac2ab8c9ffb0a624b48a51c105b0a780cafe483/torchao/csrc/cuda/sparse_marlin/marlin_kernel_nm.cu#L83-L92
https://github.com/pytorch/ao/blob/3ac2ab8c9ffb0a624b48a51c105b0a780cafe483/torchao/csrc/cuda/sparse_marlin/marlin_kernel_nm.cu#L1012-L1019
Fix - Move the endif to the appropriate location.

mind just pasting the error you get before this change as well? otherwise lgtm

Sure, I have updated the PR description. Thanks for the quick review :)


CupertinoAlertDialog does not blur UIKit based views

Steps to reproduce

Use provided sample code
Run app
Tap on one button

Expected results
I would expect the entire background of the Dialog to be blurred like the upper button is.
Actual results
The lower (UIKit based, https://github.com/flutter-stripe/flutter_stripe/blob/main/packages/stripe_ios/ios/Classes/ApplePayButtonView.swift) button is not blurred.
Code sample
Code sample
import 'package:flutter/cupertino.dart';
import 'package:flutter_stripe/flutter_stripe.dart';

void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  Stripe.publishableKey = 'xxx';
  Stripe.merchantIdentifier = 'xxx';
  await Stripe.instance.applySettings();
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return const CupertinoApp(
      title: 'Test',
      home: HomePage(),
    );
  }
}

class HomePage extends StatelessWidget {
  const HomePage({
    super.key,
  });

  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      child: Center(
        child: Column(
          mainAxisSize: MainAxisSize.min,
          crossAxisAlignment: CrossAxisAlignment.stretch,
          children: [
            CupertinoButton.filled(
              child: const Text("Touch me…"),
              onPressed: () {
                _showDialog(context);
              },
            ),
            const SizedBox(
              height: 16.0,
            ),
            PlatformPayButton(
              onPressed: () {
                _showDialog(context);
              },
            )
          ],
        ),
      ),
    );
  }

  void _showDialog(BuildContext context) {
    showCupertinoDialog(
      builder: (context) => CupertinoAlertDialog(
        title: const Text('Title'),
        content: const Text('Content'),
        actions: [
          CupertinoButton(
            onPressed: () {
              Navigator.of(context).pop();
            },
            child: const Text('Close'),
          )
        ],
      ),
      context: context,
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration

Logs
No response
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.24.0, on macOS 15.1 24B83 darwin-arm64, locale en-US)
    • Flutter version 3.24.0 on channel stable at /Users/dirkmika/projects/SDKs/Flutter/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80c2e84975 (3 months ago), 2024-07-30 23:06:49 +0700
    • Engine revision b8800d88be
    • Dart version 3.5.0
    • DevTools version 2.37.2

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/dirkmika/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16A242d
    • CocoaPods version 1.15.2

[✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome)
    ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.95.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.98.0

[✓] Connected device (4 available)
    • sdk gphone64 arm64 (mobile)     • emulator-5554                        • android-arm64 • Android 14 (API 34) (emulator)
    • iPhone 15 Pro (mobile)          • ED1976BA-F0CC-4A90-BE74-25A9E49D110D • ios           • com.apple.CoreSimulator.SimRuntime.iOS-17-2 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64  • macOS 15.1 24B83 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin        • macOS 15.1 24B83 darwin-arm64
    ! Error: Browsing on the local area network for Mein 13 Pro. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Thanks for the report!
This is a known issue being tracked at https://github.com/flutter/flutter/issues/138443. Closing this issue as a duplicate, so please follow up on it for further updates.


Adds the ability to add Mastodon feeds to iOS

This adds the ability to add Mastodon feeds to iOS (related to #3748)
If we are happy with this implementation, I'll work on the macOS equivalent.

Why is this UI so very different than the other 4 options we have to add items?

Visually, it's more inline with Add Account UI.
It's doing more than standard Add Feed UI, given the federated nature of Mastodon.
We need username and instance. To get that we could just ask for the full URL from the user and append .rss and see if it's correct. But I felt splitting out username and instance was a better approach.
For following tags, the UI doesn't give an option for the instance on which to follow the tags (which I actually think I need to correct).

It does look like an Add Account UI, but nothing like any thing else under the plus button. None of the other items there have logos or banners or explanation blocks at the bottom.
The Add Reddit Feed and Add Twitter Feed do just as much as the Add Mastodon Feed, if not more. They are done in completely different ways though.
Shouldn't we try to make sure the NetNewsWire UI is consistent for both usability reasons and aesthetic ones?

The other option I can think of would be to utilise the Mastodon Search API.
If we used Mastodon.social the lookup would be limited to Mastodon.social and its federation.

I don't have a problem with username/tag and instance being separate fields.
The username vs. tag should be a table that pushes the next screen for the user like the Twitter and Reddit add options. We should also use the same mechanism to select the Account/Folder as all the other options currently do.
I understand if you think the Mastodon option is done better. It doesn't really matter though unless we commit to changing all the other Add options as well. Consistency in app design is very important.  We want it to look like it was made by a cohesive team, right?

I get what you mean now that I've been noodling in these views for a few days. I'll come back to this once all the Settings stuff is squared away.

Will rework this!


Content-Disposition filename saving

I hope you don't mind this feature request. When I had requested it on BobbyWibowo's repo, sadly it got put on hold.
https://github.com/BobbyWibowo/lolisafe/issues/192
I've been trying to find a way to get files to save as their original filenames after uploading and doing it directly with nginx modules might be a bit too advanced for its purpose. Could it be possible with the serveFilesWithNode option enabled? It would have to look at the filename on disk it's loading and match it to the originalname pair on the database.
The idea is basically like this on Go https://github.com/uhthomas/kipp/blob/ef9de480f2e63b2f1358f91a20edf3b37a6736a6/server.go#L139-L141
Adding a Content-Disposition response header with the filename field matching the original name so when the client saves the file, the url https://msafe.tld/2ibn.png would save as the original name that is in the DB test-image-revb.png.
I found something similar to what the serveFilesWithNode option outputs, with Content-Disposition. However it is without the table matching
https://github.com/dhknudsen/downloadheaders/blob/master/download.js#L14
A few others I found as well
https://github.com/jshttp/content-disposition
https://github.com/SeregPie/saveFile

I don't mind the feature request, and in fact I'm happy that you've taken interest in my fork.
I'm not currently working on this fork actively, if I were to I would just be too tempted to rewrite the whole thing, base lolisafe has a lot of issues, and so does my fork, and attempting to work-around them is frankly too much hassle.
Regarding Content-Disposition, I'm not really familiar with the header in general, and honestly I'm not sure I understand your feature request,
Do you want the files to be saved on disk with their original upload name, but retain a random-generated URL for accessing via CDN? Am I understanding that correctly?
Sorry, it's been a while since I've messed with code for this, so sorry for any mistakes, however:
It seems that the original filenames are already saved in the db

I believe that simply changing https://github.com/weebsquad/msafe/blob/master/controllers/uploadController.js#L34
const name = file.originalname
and redoing to check for matches in the db instead of existing files for the cdn would likely be what you're looking for https://github.com/weebsquad/msafe/blob/master/lolisafe.js#L155
If you happen to work on it and end up getting it to function, feel free to open a PR or clarify me exactly what you need and I'll try to help, however, at this time I'm realistically only maintaining my own hosted instance of this fork, and resolving issues that arise from it at a very low priority, sorry.


Intelligently Suggest Submissions to be Reviewed

As of now, the app presents all submissions in a (unordered?) list and Reviewers are free to score as many as they'd like.
In the future, limits will be introduced that allow Sponsors to determine how many Reviews they would like to allow. This combined with a Reviewer Reward Pool will give Sponsors complete control over budget.
We can imagine a case where a Challenge receives 10 Submissions and has a limit of 50 Reviews.
It is possible that each Submission receives 5 Reviews, but it may be more beneficial for the best Submissions to receive more Reviews than the worst. This enables more granularity for top scoring submissions and lessens time wasted on poor submissions.
For example, an optimal review set may look like this rather than equally distributed.

Submission
Reviews
Score

1
9
95

2
12
97

3
8
78

4
8
68

5
2
50

6
4
44

7
2
25

8
2
25

9
2
25

10
1
0

Are there ways in the app that we can de-prioritize additional scoring of the low scored submissions and prioritize scoring of the top submissions?
As a sponsor, I want to optimize the use of reviewer time, so that I have high trust in the best submissions.

@drakedanner  this is an important feature that allows bounty operations to control budget. Thanks for detailed documentation of the problem as well as potential solution.
Do you have any thoughts on how do we make sure every submission gets minimum number of reviews required? In the hypothetical example above, submission 10 got only 1 review. This is not good for analysts if the sole reviewer fat fingers a submission as spam / bad.
Please see if we can allow setting min reviews a submission should get. Ex: every submission should get at least 2 or 3 reviews.

Summarizing the two posts above I think two thresholds are essential:

A minimum review count per submission ( > 1, more like 2-3)
A maximum number of reviews for the entire challenge

However I don't think that weighing reviews by score is the best approach here. If a submission got 6 Stellar reviews than there really is no need for more.
The more important thing would be to review submissions that have a higher spread across reviewers.
(e.g. https://metricsdao.xyz/app/market/0x280b3b748Ccc42D5062CE59111fAd08594f51d9F/submission/289?sortBy=blockTimestamp or https://metricsdao.xyz/app/market/0x280b3b748Ccc42D5062CE59111fAd08594f51d9F/submission/297?sortBy=blockTimestamp)
If there are none of these uncertain grades remaining then sure, reviews should go to the best submissions, but while there are these should have priority imo.


Feature/RST-1127 - Update error handling and strings

Updates errors, removing (optional) from error fields and providing better feedback to the user. Heavily based on @AlexaVB's changes in RST-1127 but I overhauled error messages across the board.

comments made on https://tools.hmcts.net/jira/browse/RST-1127


Maintenance Request Oxmaint (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

@microsoft-github-policy-service agree [company="Oxmaint Inc."]

@microsoft-github-policy-service agree
@microsoft-github-policy-service agree company="Oxmaint Inc."

Hello @JRS-Connector,
Kindly agree the CLA and follow below documents and submit your connector.
Settings.json and icon.png files are not required to submit.
https://learn.microsoft.com/en-us/connectors/custom-connectors/certification-submission-ip https://github.com/microsoft/PowerPlatformConnectors/tree/dev/templates/Independent Publisher

Hi @JRS-Connector,
Kindly address above comments and re submit this PR.

Hello,
The changes have been addressed in the same PR. Do I have to create a new PR?

Hello @JRS-Connector,
Thank you for your reply, I can still see the settings.json and icon.png files in the artifacts, please see the screenshot below.

Sorry about that! I have made the changes now. Could you please have a look?

x-ms-connector-metadata has been updated.
And please let me know about the issue with iconColor

Apologies for missing the link, please refer the link below and update the iconBrandColor.
https://github.com/microsoft/PowerPlatformConnectors/blob/dev/templates/Independent Publisher/apiProperties.json

@vmanoharas The changes have been done.

I have updated the IconBrandColor as per your suggestion. Could you please review it.

[[certify-connector]]

Could you please let me Know what is the further process so that I do that.

Hello @JRS-Connector,
You don't have to take any action further, we will process your connector and deploy to all the regions in next 3 to 4 weeks.

Can you please provide update over this connector as in the mail I am unable to open it and get over the mail data. If there is anything in the email that would need action from my side you can drop the mail at<EMAIL_ADDRESS>


Base stats calculated different from game

Describe the bug
Feral Druid (Cat): According to the simulator I should have 1875 Agility. In-game I only have 1839. When I reduce the amount in the simulator manually by 36 points, it reduces the Agility to 1836 instead.
Also: I have disabled every visible buff to see the absolute base stats of my character and for some reason the Agility tool tip adds some 55 Agility to the whole and I am using the recommended gems from the simulation.
To Reproduce
Import my character (in-game screenshots and sim screenshots added additionally)
Expected behavior
In-game major stats with a maybe +/- 1 point difference
Sim Links and Screenshots
https://wowsims.github.io/wotlk/feral_druid/#eJztVE9IFFEYf+/NuuggOG0Go6i95hCrpMzO4KGDzOhFOwiWtyBo2pnVod2ZZXY20ENMwQoq/SEqdCOKtKtBkeipSCUFJVlIMYhMKhIPbtDCXqLezG63MCvr1I+Z977v932/73s8Hh8N2kAAVF/0X4T+rqjSp1mlfgZWr0G6ghm6RLGbtZx/cgY9nkZ0GZMbpDjKNcuZ2SsUu1773Xs9TLE3ajjqBfEqmDkSy7u6aVQgbg9SbKqO8xOPcHSAeT9AsddruLKMR0x6SY8uUx7xwPsIkckjdpuUId7KV7fLm23E3iRdMt5hNtzDFMzNgaLpY7Y+uOtw1i2QJ12WCgXcioAERoYoDtxDOUQ/ReA//ggB/lcV/eAhbKKDDGRLMAzCFgAcWS7pqLoKK0YgdQoAudWBdffh0uJiyxSE1YgJDsH6xmZeFHghJAqiIPAhgW8WycKLvBjim0OhRkKQDBLYgGzp56VyZvwdYueeIJx+iYLPM4ifyaCP0JeFKA3BMwTm0TxcJv8rCNIIrBXfwWGZW/aVeaZzTt5XICvlqvSoiwXpaIFZlg5FPKxIwl1fPHcH9e/v1g3VUnrMhIKDHZpl6uF63M2WNvzlJxYclPeijLMw+5M63TvF8flPqda9OMeP4F264+TkUVgg1qVir7x0GjoQXoO3YODAMSMcTaoa7uxL2HoYtyUjEc1uuAA7TVWLYrtXw5YSi+tGD44pPSRBVciuYVs5qxlY1c6QfKzE41FdU7GatNzErl4loWERmxFPrxlhM2nYmnUE6wZWVFW3ddPAtulFDdOKKdGiJoRjWrhXMfRwoumEN2hED5R8MLV1fKqO2uk2fwNZqXKi/W1/pU9eHYu30KtQ7iDeRPsXadx1x4DstU3tdd9/h+KgcXh5t4qTxdGUlXar+AbYEuvH

Desktop (please complete the following information):

OS: Win 11 Pro
Browser: Firefox
Version: 120.0.1 (64-bit)

In-game I only have 1839. When I reduce the amount in the simulator manually by 36 points, it reduces the Agility to 1836 instead.

That's expected, the reduction is before % modifiers from buffs and talents, in your case improved mark of the wild and survival of the fittest: 36 * 1.06 * 1.02 ~= 39.

Also: I have disabled every visible buff to see the absolute base stats of my character and for some reason the Agility tool tip adds some 55 Agility to the whole and I am using the recommended gems from the simulation.

That's MotW, you can't disable it since it's not a raid buff but a self buff.

as said, compare with motw on, is its always 'enabled'


ClassCastException / crash when running --make-species-subset

Hi.
We're having trouble with processing the latest go-plus.owl with owltools.
With this command line:
(cd /tmp/; wget http://purl.obolibrary.org/obo/go/extensions/go-plus.owl) &&
(cd go-svn/ontology/extensions; owltools /tmp/go-plus.owl --use-catalog --reasoner elk --make-species-subset -t NCBITaxon:4896 -o -f obo /tmp/go-plus-pombe-only.obo)
we get this exception:
2018-08-06 21:53:38,189 WARN  (IndexedObjectUnionOf:92) [reasoner.indexing.IndexedObjectUnionOf]ELK does not support positive occurrences of ObjectUnionOf. Reasoning might be incomplete!
2018-08-06 21:53:38,190 WARN  (ChangeIndexingProcessor:66) [reasoner.indexing.axiomIgnored]ELK does not support ObjectMinCardinality. Axiom ignored:
EquivalentClasses(<http://purl.obolibrary.org/obo/UBERON_0000037> ObjectIntersectionOf(<http://purl.obolibrary.org/obo/UBERON_0001305> ObjectSomeValuesFrom(<http://purl.obolibrary.org/obo/BFO_0000051> <http://purl.obolibrary.org/obo/UBERON_0000039>) ObjectMinCardinality(2 <http://purl.obolibrary.org/obo/RO_0002180>)))
2018-08-06 21:53:38,192 WARN  (ChangeIndexingProcessor:66) [reasoner.indexing.axiomIgnored]ELK does not support InverseObjectProperties. Axiom ignored:
InverseObjectProperties(<http://purl.obolibrary.org/obo/RO_0002005> <http://purl.obolibrary.org/obo/RO_0002134>)
Exception in thread "main" java.lang.ClassCastException: uk.ac.manchester.cs.owl.owlapi.OWLLiteralImplString cannot be cast to org.semanticweb.owlapi.model.IRI
        at org.obolibrary.macro.MacroExpansionVisitor$MacroExpansions.expand(MacroExpansionVisitor.java:162)
        at org.obolibrary.macro.MacroExpansionVisitor$MacroExpansions.<init>(MacroExpansionVisitor.java:122)
        at org.obolibrary.macro.MacroExpansionVisitor.expandAll(MacroExpansionVisitor.java:95)
        at owltools.mooncat.SpeciesSubsetterUtil.<init>(SpeciesSubsetterUtil.java:139)
        at owltools.cli.CommandRunner.runSingleIteration(CommandRunner.java:768)
        at owltools.cli.CommandRunnerBase.run(CommandRunnerBase.java:76)
        at owltools.cli.CommandRunnerBase.run(CommandRunnerBase.java:68)
        at owltools.cli.CommandLineInterface.main(CommandLineInterface.java:12)

It last worked successfully on August 2nd.
Thanks in advance.
CC: @ValWood

Hi was this fixed?
Val

@kimrutherford - does this work
@balhoff - what's the best way of preventing these typos in the ontology in future? A custom sparql query?

We should probably just have a SPARQL check that never_in_taxon only has IRI values. This has happened repeatedly.

@kimrutherford - does this work

It works for me now.

When I run the above command I get an error catalog-v001.xml (No such file or directory) and I was wondering where to get the catalog file specific gene ontology?

@wkpalan you don't need the catalog file; just delete that part from the command. You can just provide the ontology file directly. The catalog provides a customized map from ontology IRIs to ontology files.

Thank you. I could get the command working, but wasn't sure whether not having a catalog file would change the results.


Adding reviewers.py to help tag reviewers in commit message

This is a simple python script which looks at the commit log and lets you easily find names + emails for the "Reviewers" line.
You can search by first name or email. Here's a sample session:
$ ./reviewers.py
Utility to help generate 'Reviewers' string for Pull Requests. Use Ctrl+D or Ctrl+C to exit

Name or email (case insensitive): Dav <-- Just need a few letters

Possible matches (in order of most recent):
[1] David Jacot<EMAIL_ADDRESS>(141)  <-- Shows a count of occurrences in the commit log
[2] David Arthur<EMAIL_ADDRESS>(102)
[3] David Arthur<EMAIL_ADDRESS>(5)
[4] David Jacot<EMAIL_ADDRESS>(8)

Make a selection: 2
Reviewers so far: [('David Arthur'<EMAIL_ADDRESS>102)]

Name or email (case insensitive): chia

Possible matches (in order of most recent):
[1] Chia-Ping Tsai<EMAIL_ADDRESS>(358)
[2] Chia-Ping Tsai<EMAIL_ADDRESS>(3)

Make a selection: 1
Reviewers so far: [('David Arthur'<EMAIL_ADDRESS>102), ('Chia-Ping Tsai'<EMAIL_ADDRESS>358)]

Name or email (case insensitive): ism

Possible matches (in order of most recent):
[1] Ismael Juma<EMAIL_ADDRESS>(1514)
[2] Ismael Juma<EMAIL_ADDRESS>(3)
[3] Ismael Juma<EMAIL_ADDRESS>(4)
[4] Ismael Juma<EMAIL_ADDRESS>(19)
[5] Ismael Juma<EMAIL_ADDRESS>(7)

Make a selection: 1
Reviewers so far: [('David Arthur'<EMAIL_ADDRESS>102), ('Chia-Ping Tsai'<EMAIL_ADDRESS>358), ('Ismael Juma'<EMAIL_ADDRESS>1514)]

Name or email (case insensitive): ^C

Reviewers: David Arthur<EMAIL_ADDRESS>Chia-Ping Tsai<EMAIL_ADDRESS>Ismael Juma<EMAIL_ADDRESS>
ctrl+d or ctrl+c will exit the program and print out the "Reviewers" line based on the selection.

Thanks, this is helpful. Any chance we can add a map that deduplicates emails from committers? In this PR, we could add the map and a couple of cases and then we can iterate as needed.

@ijuma I've been using this script for a while now with good results. Can we merge this and defer improvements to future PRs?

@mumrah Can we please check builds before merging? The build caught the license issue.


Actions: How to run jobs on issue_comment only if PR

Why:
The issue_comment event occurs for both, regular issues and pull requests. This should be pointed out.
What's being changed:
Add remark and also provide an example how to tell issues and PRs apart by testing for the presence of  github.event.issue.pull_request.
Check off the following:

[ ] ~All of the tests are passing.~
[ ] ~I have reviewed my changes in staging.~
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

Thanks for opening a PR @Simran-B! I'll get this triaged for review ⚡

This is not stale! We're working our way through reviews 💛

Feel free to commit the suggested changes. I can also work on it tomorrow however.

Alright, please take another look if this is good to go now.

@rachmari not really related to the change, but would using the automerge and autoupdate labels on these PRs help so you don't have to chase the up-to-date branch check?

@nschonni

@rachmari not really related to the change, but would using the automerge and autoupdate labels on these PRs help so you don't have to chase the up-to-date branch check?

We don't have that capability yet, but we're working on it!

Fair enough, I thought they were working already because they're used for the repo-sync job

@nschonni soon hopefully🤞! Chasing down CI builds is definitely something for a 🤖 to do. 😂


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_697bdb28-9ca6-47c8-b18c-6468d76f5761/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_697bdb28-9ca6-47c8-b18c-6468d76f5761.git

Docs Build status updates of commit cb1ac6c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-source' (branch: Release_Archive_main_2022-05-30-04-20-56) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix hiredis.h include path in example.c

Without this change, examples/example.c fails to compile.
[kp@trantor examples (master)]$ gcc -lhiredis example.c
example.c:5:21: fatal error: hiredis.h: No such file or directory
 #include <hiredis.h>
                     ^
compilation terminated.

Signed-off-by: Krishnan Parthasarathi<EMAIL_ADDRESS>

Except that breaks our own Makefile:
r Makefile:
% make hiredis-example
cc -o examples/hiredis-example -O3 -fPIC  -Wall -W -Wstrict-prototypes -Wwrite-strings -g -ggdb    -I. examples/example.c libhiredis.a
examples/example.c:5:29: fatal error: hiredis/hiredis.h: No such file or directory
 #include <hiredis/hiredis.h>

                             ^
compilation terminated.
Makefile:117: recipe for target 'hiredis-example' failed
make: *** [hiredis-example] Error 1

For an external program using hiredis as a library hiredis/hiredis.h is the right thing (we install files into .../include/hiredis/).
We also use include <hiredis.h> in all other examples. I'd say we just stay with that for now and include documentation about how to include in the Readme.

OK. That makes sense. I shall close this pull request.


possible fork?

Does this repo has any relation to https://github.com/RomaniukVadim/backup-my-github ?

@nuxdie seems like he's just stole it, take a look into first commit - https://github.com/RomaniukVadim/backup-my-github/commit/8e93d4e726eedb924caa271d87c6e71cd03f368c
there is my copyright
but, since I've licensed it under Do What The F*ck You Want... it's not a big deal, imho

Cool! Thanks for clarifying that!


Allow for restricting max-height on code blocks

Fixes #1969. Example usage:

        some beautiful code

I'd hoped to be able to do this with a single div (say, code-max-h max-h-25), but the need to keep the copy buttons absolutely positioned required a containing div, alas.

@justinvp Yes, exactly right!


fix version binding for pytest-testslide, tweak makefile

Fix setup for pytest-testslide

[X] Added tests, if you've added code that should be tested
[X] Updated the documentation, if you've changed APIs
[X] Ensured the test suite passes
[X] Made sure your code lints
[X] Completed the Contributor License Agreement ("CLA")

Pull Request Test Coverage Report for Build 469364040

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.089%

Totals

Change from base Build 469316913:
0.0%

Covered Lines:
2531

Relevant Lines:
2690

💛  - Coveralls

Pull Request Test Coverage Report for Build 469364040

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.089%

Totals

Change from base Build 469316913:
0.0%

Covered Lines:
2531

Relevant Lines:
2690

💛  - Coveralls


(reddit.com) Videos linked/embedded in posts can't be downloaded

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I'm running yt-dlp version 2023.10.13 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Sweden
Example URLs
https://reddit.com/link/178yasm/video/p5rowtg41iub1/player
Provide a description that is worded well enough to be understood
Reddit posts can have videos embedded in them now and in old reddit they are just rendered as links like the one above, example of a post with videos linked: https://old.reddit.com/r/uBlockOrigin/comments/178yasm/youtube_antiadblock_and_ads_october_16_2023/
These links can be clicked and played in a fullscreen player but yt-dlp can't download them sadly.
I wasn't sure what type of issue this should be so I hope this is correct..
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
.\yt-dlp.exe -vU https://reddit.com/link/178yasm/video/p5rowtg41iub1/player
[debug] Command-line config: ['-vU', 'https://reddit.com/link/178yasm/video/p5rowtg41iub1/player']
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[4e38e2ae9] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.19045-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 5.1.2-essentials_build-www.gyan.dev (setts), ffprobe 5.1.2-essentials_build-www.gyan.dev, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.07.22, mutagen-1.47.0, requests-2.31.0, sqlite3-3.35.5, urllib3-2.0.6, websockets-11.0.3
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1890 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 9166198bb7bbe419d352dc0dbc218c05fc5c7027729d0a727339171f3ecf2514
yt-dlp is up to date<EMAIL_ADDRESS>[generic] Extracting URL: https://reddit.com/link/178yasm/video/p5rowtg41iub1/player
[generic] player: Downloading webpage
[redirect] Following redirect to https://www.reddit.com/link/178yasm/video/p5rowtg41iub1/player
[generic] Extracting URL: https://www.reddit.com/link/178yasm/video/p5rowtg41iub1/player
[generic] player: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] player: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://www.reddit.com/link/178yasm/video/p5rowtg41iub1/player
Traceback (most recent call last):
  File "yt_dlp\YoutubeDL.py", line 1567, in wrapper
  File "yt_dlp\YoutubeDL.py", line 1702, in __extract_info
  File "yt_dlp\extractor\common.py", line 715, in extract
  File "yt_dlp\extractor\generic.py", line 2575, in _real_extract
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.reddit.com/link/178yasm/video/p5rowtg41iub1/player

If you want to just download them, you can use the link of the post.

If these player URLs are made available in the site UI, eg by a Share function, they should be supported.
This approach worked, while also fetching available metadata not provided directly in the player page itself:

extract the <div ....> element with id video-{thread_id}, where thread_id is 178yasm in the example
extract default data from the URL in its data-link-url attribute
replace data['media_metadata'] with a list containing a single media item comprising the dashUrl and hlsUrl from the data-mpd-url and data-hsl-url attributes of the #video-{thread_id} element
complete the extraction from data, more-or-less as before.


register arabic woff does not work

I have a project, where I would like to use latin and arabic fonts together in the same pdf-document:
I can register the latin font like
NotoSans-Regular-webfont.woff
and use it for screen and pdfkit
but with the arabic font I have to register a separate ttf for pdfkit like
NotoKufiArabic-Regular.ttf
the woff I use for screen
notokufiarabic-regular-webfont.woff
produces only rectangles in the pdf.
same if I use the
notokufiarabic-regular-webfont.ttf

Can you please provide an example demonstrating the issue?

I tried to make an example and also tried a different font – and it worked.
So must be a problem with the first font. Sorry.


Issue #36283 - Cleanup your project with Modules accepting both './routes' and './routes.js'

[ X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] None of my changes are plagiarized from another source without proper attribution.
[X] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[X] My changes do not use shortened URLs or affiliate links.

Closes #36283

Hey @wilsonsilva80
It seems that similar changes are already in PR #36293 which we expect to merge for the issue referenced.  Also, your regex would allow invalid syntax because it would allow the following (mismatched quotes):
require("./routes.js');

OR
equire('./routes.js");

If you feel you have more to add, please feel free to open up a new PR.
Thanks again! 😊

If you have any questions, feel free to reach out through Gitter or by commenting below. 💬


Support list values in tags in Span and SpanCustomizer

Resolves #785.
I've named the functions tagOfStrings, tagOfDoubles, etc., because tag would lead to clashes due to type erasure.

Congrats @Jasz on your first contribution :)

Thanks a lot @marcingrzejszczak!


Diskpool creation fails with "Unable to find io-engine node k8w1" error

Describe the bug
Can't create diskpools with "Unable to find io-engine node k8w1" error.
kubectl describe -n mayastor diskpool mayastore-pool-k8w1
Name:         mayastore-pool-k8w1
Namespace:    mayastor
Labels:       <none>
Annotations:  <none>
API Version:  openebs.io/v1beta1
Kind:         DiskPool
Metadata:
  Creation Timestamp:  2023-10-18T20:05:20Z
  Finalizers:
    openebs.io/diskpool-protection
  Generation:        1
  Resource Version:  54207260
  UID:               b5f74355-bafe-4eae-a680-dea5404e00bb
Spec:
  Disks:
    /dev/nvme0n1
  Node:  k8w1
Status:
  Available:  0
  Capacity:   0
  cr_state:   Creating
  Used:       0
Events:
  Type      Reason   Age   From          Message
  ----      ------   ----  ----          -------
  Critical  Failure  9s    dsp-operator  Unable to find io-engine node k8w1

To Reproduce

Install Mayastor with helm
Create a diskpool

apiVersion: "openebs.io/v1beta1"
kind: DiskPool
metadata:
  name: mayastore-pool-k8w1
  namespace: mayastor
spec:
  node: k8w1
  disks: ["/dev/nvme0n1"]

Expected behavior
Diskpool properly created
** OS info (please complete the following information):**

Distro: Ubuntu 22.04.3
Kernel version 5.15.0-86-generic
MayaStor revision or container image 2.4

Hi,
Solved installing nvme module on nodes. Some hints in the docs could be useful (see https://github.com/openebs/mayastor/issues/1530)
Thank you very much


PR for OpenJDK-1.8.144

Hi guys,
I noticed that the JDK version number could be updated on the conda-forge channel.
I'll correct the sha256sum from information provided by travis builds.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hey, so I noticed that the builds are failing due to https://github.com/conda/conda-build not being there.
Could you help me by correcting the recipe for the correct  build procedure ?

You cannot switch the versioning scheme like this, also it wasn't wrong before.
@conda-forge/core, I think we should use https://bintray.com/jetbrains/intellij-jdk/ instead. Seems it brings many benefits. In order of importance for me:

Windows 32-bit support.
Well tested with the intellij suite (I want to add this stuff).
Better font rendering.

@mingwandroid
Would it make more sense to turn openjdk into a "virtual" package and create azul-jdk and intellij-jdk?
What are your plans for openjdk in Anaconda/Continuum?

I don't know what our plans are exactly. What conda-forge does is a very important consideration for those.

Guys, it'd be great if we can find an a way to easily update the openJDK to a newer build.
Also with Java-9 coming up, availability of various versions make sense.

Can someone give me a good reason for wanting more that 1 different binary repackaged openjdk build? Personally I'm not aware of any and it seems like make-work to me and I see no persuasive reason ATM not to come to a consensus on which one to use.
AFAICT, that Java-9 is coming up has no bearing on whether we repackage Azul and Intellij's builds (or both).

@mingwandroid switching to intellij-jdk sounds fine to me. If it is better, I see no reason not to do it. @chapmanb do you have any objections regarding GATK or other java based tools in bioconda?

@mingwandroid , I personally am pointing towards the general availability of java9 through conda. I'd leave the decision for provision of different jdks on the main maintainers here.
But in any case, the appveyor builds needs to be worked out.

Updating to the latest point version would be great, thanks for working on this. I'm agnostic about whether we use the Azul or IntelliJ builds; if it provides improvement it's fine to switch over. I don't think it's worth maintaining both -- having one openjdk build is good with me.

@abhi18av,
Can you try to rebase against master and try again. I believe the windows issue is solved now.

@sodre  sure.
But I've kinda messed up with the syncing part on github so started with a fresh fork and a fresh PR here
https://github.com/conda-forge/openjdk-feedstock/pull/18#issuecomment-329780172

Guys, all checks have passed on this PR here https://github.com/conda-forge/openjdk-feedstock/pull/18

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.


cmd/compile: print better error message when assigning to keywords

I saw a new programmer struggle with the following:
func main() {
	map := "f"
	fmt.Println(map)
}

which is an invalid program since map is a keyword.
However, the error being printed is currently:
prog.go:6: syntax error: unexpected :=, expecting [
prog.go:7: syntax error: unexpected ), expecting [
prog.go:8: syntax error: unexpected }, expecting ]

It should really print a more helpful error message:
prog.go:6: identifier cannot be a keyword: map

/cc @griesemer

Doing this well requires some care and is not trivial - it may not be worth the effort. For instance, if the code would have been
func main() {
   for := "f"; g(); {}
   // ...
}

it's not even clear what the error should be: When the compiler sees the for it assumes a for loop, and the error might be that there's a missing identifier after the for. Or it might be that for was taken as an identifier.
Given that Go has only 25 identifiers, it's not unreasonable to assume that Go programmers actually know them all.
Leaving open for now, in case somebody wants to play with this.


grid Background Update and new Enne-san Boards

Error:
../_maps/Ennesan39B/Ennesan39B.yaml - 1 issue(s):
YAML files shall no longer contain the line '---' or '...'
../_maps/Ennesan39A/Ennesan39A.yaml - 1 issue(s):
YAML files shall no longer contain the line '---' or '...'


Unexpectedly found nil while unwrapping an Optional value Lost connection to device.

Once we load video from a local file, then app crashes every-time. We used the most recent version 2.0.4, and when we tried the preview versions, the same problem occurred.
Local asset video file are working fine. But if we download and save video on document directory(getApplicationDocumentsDirectory()) then we load document directory path app crash.
here is log
flutter_to_airplay/FlutterAVPlayer.swift:34: Fatal error: Unexpectedly found nil while unwrapping an Optional value
Lost connection to device.
Error: Unable to terminate com.xxx on EE5BC5C6-4C03-4748-9B4C-3679047D98D6:
ProcessException: Process exited abnormally with exit code -2:
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=3):
Simulator device failed to terminate com.xxx.
found nothing to terminate
Underlying error (domain=NSPOSIXErrorDomain, code=3):
The request to terminate "com.xxx" failed. found nothing to terminate
found nothing to terminate
Command: /usr/bin/arch -arm64e xcrun simctl terminate EE5BC5C6-4C03-4748-9B4C-3679047D98D6
the Dart compiler exited unexpectedly.

issue has been fixed Accept pull request https://github.com/MrJai/flutter_to_airplay/pull/30


Definition of Maps

Maps are used in two places within the model.  We need to specify exactly what these are. This could be achieved by using a complex data type (as per Range, Quantity)

ScheduledDecisionInstance
SyntaxTemplateDictionary

Diagrams as shown at daily scrum 12th March
2024 03 12 Maps.pptx

@dih-cdisc @BSnoeijerCD : Craig and I are considering whether the 'target' relationship name should be something like 'conditionalTarget' instead. Target could be used/misused multiple ways so adding an adjective to it makes it more concise. It also fits in with the names of some of the new class elements. Thoughts?

@dih-cdisc @BSnoeijerCD : Are the ParameterMap.tag ParameterMap.reference attributes always related to HTML? Asked another way, are the semantics for those two attributes specifically for 'HTML tag' and 'HTML tag reference'?

@BSnoeijerCD @dih-cdisc : See proposed CT changes to support this ticket:
DDF 261.xlsx

@dih-cdisc @BSnoeijerCD : Are the ParameterMap.tag ParameterMap.reference attributes always related to HTML? Asked another way, are the semantics for those two attributes specifically for 'HTML tag' and 'HTML tag reference'?

tag is just a string name/identifier
reference is XHTML formatted

@dih-cdisc @BSnoeijerCD : Craig and I are considering whether the 'target' relationship name should be something like 'conditionalTarget' instead. Target could be used/misused multiple ways so adding an adjective to it makes it more concise. It also fits in with the names of some of the new class elements. Thoughts?

The two attributes are bound within the class of ConditionAssignment so they have context. They are a "name/value" pair, a key and value. This condition, go here ... so [condition, target] or [condition, conditionTarget] or [condition, destination] or [condition, assignment] .. don't make it too complicated

@EMuhlbradt @czwickl @dih-cdisc
Summarizing the above and for maintaining the consistency with the rest of the model, it is better to keep it to condition and target.
They are both within the ConditionAssignment context. So that means they are the condition of the ConditionAssignment and the target of the ConditionAssignment.
If target is too vague then conditionTarget is the most clear term as it directly indicates what the target is when the condition is met. ConditionalTarget is less clear.


Update to rollup 3

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[x] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

The PR fulfills these requirements:

[x] It's submitted to the main branch for v2.x (or to a previous version branch)
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

@yyx990803 could you please review my code? I have updated rollup to v3 in this PR but it has not yet been reviewed. Thank you for your help!


unexpected 'function' (T_FUNCTION)

Sorry, i got this issue when trying to use Pestle
PHP Parse error:  syntax error, unexpected 'function' (T_FUNCTION), expecting identifier (T_STRING) or \\ (T_NS_SEPARATOR) in phar:///var/www/magento2s/pestle.phar/modules/pulsestorm/pestle/runner/module.php on line 8

@anybao Pestle requires PHP 5.6 or greater.  If you're running with PHP 5.6, please reopen and let us know so we can investigate further!
Thanks for giving pestle a try.


[UNR-3128] Fix startup actors receiving empty lists

Description
Scenario where this would break:

A client checks out a bNetLoadOnClient startup actor which has a replicated TArray, with let's say [0,1,2]
The client walks away so the actor is no longer checked out
The startup actor is updated by the server, so the array is now empty []
The client comes back to the startup actor and checks it out again, the empty array would not be applied

Release note
Fixed a bug where bNetLoadOnClient actors would not be updated if a list (such as a TArray) was cleared while the actor was not in view.
Tests
Not-so-soon to come in EngineNetTest (needs some frameworky stuff for bNetLoadOnClient actors, also I think I accidentally deleted my local test :( ).
Documentation
Release note
Primary reviewers
@m-samiec

Give me a release note or give me death

Updated this once again, to move the code into ApplyComponentData, but it has a small problem, namely we expect the replicator to exist.


better UX for matrix tooltips

Right now, the tooltips require JS before they render anything. The takes several (more than 5) seconds before the JS executes on my desktop with a good connection, so if you click before that, nothing shows up. Could this progressively enhance better? For example, could we have the images contain alt tags, which are replaced by the data- attributes for tooltips after the JS loads?

Hey @moschel ,
That's actually a really great idea. I do not have any issues on my end, if I refresh and click, I get the the tooltip immediately on the live site.  I think this would require a JS dev to implement your example.  Do we have any one available to work on that? I'll try to think of some other options too.


Warning about deprecated module manifest member should be suppressed.

When a module manifest contains 'ModuleToProcess' the warning
WARNING: The module manifest member 'ModuleToProcess' has been deprecated. Use the 'RootModule' member instead.

shows up.
If the module is targeting PowerShell version 2 then changing 'ModuleToProcess' to 'RootModule' will break the module.
This warning should be suppressed when the module is targeting PowerShell version 2.

Hi Francis.
Thank you for identifying the issue. We will fix it soon.
-Raghu


Add support for Turborepo v2

Summary
Right now this Github Action only supports Turborepo v1. We also want it to support V2.

🗒 Details

Turbo v1 should still be supported (either by auto-detecting the Turborepo version used or by requiring Github Action users to migrate to a new v2 of this Action)
Usage of the Github action should be as similar as previously as possible to reduce migration pain

I forgot to mention this issue in #219, but thanks to @philtremblay we were able to finalize this PR and now have Turborepo v2 support 🚀 Simply upgrade to the v2 version of this action!


docs: fix english grammar

Hello, @ykzts. I found a grammar error in your README, so I thought I would fix it for you.

I guess I'll close this PR...

@ykzts I re-opened this PR.

@ykzts, are you ready to merge this PR? It has been about a week since I opened it.

It's almost been 10 days since I opened this PR, and a change this simple should be able to be quick to merge. I am permanently closing this PR.


export on IE hangs after 3 to 4 downloads.

Hi,
I am using IE11 and after 3 to 4 downloads, browser hangs.
regds,
DD

Are these very large SVGs?
Are there any errors or other messages in the developer console?

even if i have 50 to 60 nodes, it hangs after 4 to 5 downaloads.. As such there are no error/any messages. But in the task manager, i could see the growth in memory consumption.

Does the number of downloads before hanging change at all as you reduce the complexity of the SVG?
Do you see the same behavior in any other browsers?
Does your SVG contain any images that have to be inlined?
Do you see the same behavior if you exclude all CSS?
A bug in the image or CSS inlining node is the only thing in the script I can think of that would consume a noticeable amount of memory.

yes.. if I have less number of nodes, browser hangs after 7 to 8 downloads. and sometimes if we reload the browser , that doesnt seems to help as well..
we do have images for nodes and links. I have not seen this behaviour in firefox. it works fine there. Only IE has this issue.

This is not related to the topic, but could you maybe post your code? Because I am not able to get this running at all in IE11. I know the thing with the canvg, but I am doing something wrong.

Hey..sry!! I left tht company around an year ago..with canvg too, u HV to
make some more changes..I am not able to recall those changes.. Really sry
for not able to help u on this..
Cheers,
D
On Tue, Jul 3, 2018, 4:06 PM Florian Leitgeb<EMAIL_ADDRESS>wrote:

This is not related to the topic, but could you maybe post your code?
Because I am not able to get this running at all in IE11. I know the thing
with the canvg, but I am doing something wrong.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/exupero/saveSvgAsPng/issues/131#issuecomment-402101401,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWP6VmkHRiVfjiuElouuQg6PEIe8oS9hks5uC0lGgaJpZM4LxDYM
.


No such container: app

Hi, I am getting error when try to deploy meteor app using DigitalOeacn cloud hosting.
I am following mupx command like this.
Mupx setup

Installing Docker
Installing Docker: SUCCESS
Setting up Environment
Setting up Environment: SUCCESS
Copying MongoDB configuration
Copying MongoDB configuration: SUCCESS
Installing MongoDB
Installing MongoDB: SUCCESS
=================================================

root@autoleads:~/meteorapp# mupx deploy
Meteor Up: Production Quality Meteor Deployments
Configuration file : mup.json
Settings file      : settings.json
“ Checkout Kadira!
It's the best way to monitor performance of your app.
Visit: https://kadira.io/mup ”
Meteor app path    : /root/meteorapp
Using buildOptions : {}
=> Build Error. Check the logs printed above.
===============================================
When check error  through  :  mupx logs  it show below.
Error: No such container: app
root@autoleads:~/meteorapp#
my mupx.json file.
{
// Server authentication info
"servers": [
{
"host": "####",
"username": "root",
"password": "####",
// or pem file (ssh based authentication)
// WARNING: Keys protected by a passphrase are not supported
//"pem": "~/.ssh/id_rsa"
// Also, for non-standard ssh port use this
//"sshOptions": { "port" : 49154 },
// server specific environment variables
"env": {}
}
],
// Install MongoDB on the server. Does not destroy the local MongoDB on future setups
"setupMongo": true,
// Application name (no spaces).
"appName": "app",
"app": ".",     // Because mupx.json file in meteor project directory
// Configure environment
// ROOT_URL must be set to your correct domain (https or http)
"env": {
"PORT": 80,
"ROOT_URL": "http://#.#.#.#"
},
// Meteor Up checks if the app comes online just after the deployment.
// Before mup checks that, it will wait for the number of seconds configured below.
"deployCheckWaitTime": 30,
// show a progress bar while uploading.
// Make it false when you deploy using a CI box.
"enableUploadProgressBar": true
}

+1

+1


Fix swagger UI not including the header

Related issue https://github.com/swaggo/swag/issues/314
We need to set // @Security ApiKeyAuth  for every single individual endpoint


[FEATURE] Construct Binary Search Tree from Preorder Traversal

Detailed description
Construct Binary Search Tree from Preorder Traversal.
Problem Statement:
Given an array of integers preorder, which represents the preorder traversal of a BST (i.e., binary search tree), construct the tree and return its root.
Test Case:
Input: preorder = [8,5,1,7,10,12]
Output: [8,5,10,1,7,null,12]
Context
This is a very famous and standard problem asked in most of the interviews. This will enhance the logical thinking of the programmer.
Possible implementation
A binary search tree is a binary tree where for every node, any descendant of Node.left has a value strictly less than Node.val, and any descendant of Node.right has a value strictly greater than Node.val.
A preorder traversal of a binary tree displays the value of the node first, then traverses Node.left, then traverses Node.right.
Additional information
No response

I would also like to contribute a solution to this problem.

Hello Maintainer,
I want to solve this problem ,please assign this issue to me.

Hello, Can you please assign this issue to me?
Thanks in advance :)

Hi sir/mam can you please assign this issue to me ,
i think i can do it very well

Hello, I want to work on this issue and I have a good knowledge of DSA. Can you please assign it to me?

Hello sir!
I am already done with this issue please assign me so I can handover you the code

hello sir can you assign this issue to me i want to solve this issue.

I would like to contribute to the algorithm. Please assign this task to me.


Cleanup imaging project and register image file factory in IoC container.

Sorry, I haven't forgotten about this. I'll try and take a look at this very soon.


Fix S3Bucket.copy_object target path resolution

Closes #384
Uses other bucket's _resolve_path method when copying to another S3Bucket object. If to_bucket is a string, doesn't resolve. Copies to self when to_bucket is None.
Example
from prefect_aws import S3Bucket

source = S3Bucket(bucket_name="source-bucket")
target = S3Bucket(bucket_name="target-bucket", bucket_folder="subpath/")
source.copy_object("abc.txt", "copies/abc.txt", target)
# subpath/copies/abc.txt

Screenshots

Checklist

[x] References any related issue by including "Closes #" or "Closes ".

If no issue exists and your change is not a small fix, please create an issue first.

[x] Includes tests or only affects documentation.
[x] Passes pre-commit checks.

Run pre-commit install && pre-commit run --all locally for formatting and linting.

[ ] Includes screenshots of documentation updates.

Run mkdocs serve view documentation locally.

Unclear what's up with the error message. Did something break in main?

AttributeError: 'CallSpec2' object has no attribute 'funcargs'

hey @dominictarro - that's because we hadn't yet pinned pytest. that change is merged now and we should be back to normal here. thanks for catching!

I just noticed that the same issue is in the move methods, so don't merge yet. Going to revisit this when I have time.

This is ready for review.


Can't create pods in project with limits

Rancher versions: 2.0 master 7/30
Steps to Reproduce:

Create a project with a pod limit that 8
Go to project and try to create a workload with 1 pod

Results:  Fails.
Pods "wrkld1-fc5d74c46-5r65z" is forbidden: failed quota: default-gfgvj: must specify limits.cpu,limits.memory,requests.cpu,requests.memory; Deployment does not have minimum availability.

Tested with master,
Create a project with resource quota for CPU and Pods.
Create a workload with just pods
Error pops up in UI saying "CPU limit" is required

Can we add more information to the error message saying "CPU limit is required as resource quota is enforced on the project"

Change to the error message is not a UI issue

Nope, i'm wrong. this vlidation is in the UI

Version - 2.1 master 9/17
Verified fixed


PR to add new metric results

Dear Author,
With @GuillaumeStaermanML we would like to add new metrics results specifically:
BaryScore https://arxiv.org/abs/2108.12463
And DepthScore https://arxiv.org/abs/2103.12711
And InfoLM (currently unpublished).
Is it something you could be interested in ?
Kindest regards,

Hi Pierre!
Always happy to take a look at pull requests if you want to add those metrics!
Best,
Alex


custom configuration file name

support custom config name

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


:fire: Chore: Remove unused submodules

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

#1167  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


#35 - Add matrix sender

Add matrix sender
Add matrix sender to CLI
Update README

Hello @jcklie,
Thanks for contributing!
Let me know when you think it's ready for review. The current version looks good to me.
Victor

I just checked the README and doc strings, from my side it is ready for review.


TEAM - add new member: Andrea Castiglione

Andrea Castiglione
For over a decade, Andrea has been working at the crossroad of technology, entrepreneurship and human relationships. Coming from a deeply analytical background, he has started multiple businesses and began investing in what he defines “digital emerging markets”, which in all essence, are companies designed to be digital and borderless; not bound by geography.
As a venture builder, Andrea believes it’s time to reinvent the very architecture of the internet and he’s ready to support Threefold in distributing its technology and values through his experience and a global network of partners.
https://www.linkedin.com/in/castiglioneand/

done


Add minutes for August 25 Meeting

HackMD can't push to main branch anymore due to branch protection rules so merging this PR from hackmd branch to main branch to get the latest minutes.


DeepARE loss becoming negative

Dear Sir/Madame,
I'm new to gluonts and probabilistic forecasting.
I'm training a DeepARE model on a daily time-series with the following parameters:
Number of Epochs=5
Learning Rate=0.01
Number of RNN layers=2
Number of RNN cells for each layer=40
Number of RNN cells for each layer=lstm
For the rest, I leave all the default values.
When I do the training, I obtain the loss decreasing passing from positive to negative values.
How is it possible? This is how it looks like:
Start model training
Epoch[0] Learning rate is 0.01
Number of parameters in DeepARTrainingNetwork: 26044
Epoch[0] Elapsed time 30.988 seconds
Epoch[0] Evaluation metric 'epoch_loss'=0.188573
Epoch[1] Learning rate is 0.01
Epoch[1] Elapsed time 30.306 seconds
Epoch[1] Evaluation metric 'epoch_loss'=-0.131554
Epoch[2] Learning rate is 0.01
Epoch[2] Elapsed time 30.330 seconds
Epoch[2] Evaluation metric 'epoch_loss'=-0.273915
Epoch[3] Learning rate is 0.01
Epoch[3] Elapsed time 30.184 seconds
Epoch[3] Evaluation metric 'epoch_loss'=-0.337020
Epoch[4] Learning rate is 0.01
Epoch[4] Elapsed time 30.069 seconds
Epoch[4] Evaluation metric 'epoch_loss'=-0.398353
Loading parameters from best epoch (4)
Final loss: -0.39835323095321656 (occurred at epoch 4)
End model training
Sorry for the maybe stupid question, but I don't understand why the loss is becoming negative. What is the loss function that the algorithm is minimizing during training?
Please let me know.
Best,
Sergio

The loss being minimized is the negative likelihood of the model distribution (student-t by default) it can be negative in some cases, nothing wrong with that.

cheers!


any plan of using it on yolox？

We have a plan to apply it to YOLOX after another project. But it may take some time.

We have a plan to apply it to YOLOX after another project. But it may take some time.

looking forward to it


File: Add logo original source.

Add original file so anybody can edit it if it's needed.

Should svg be counted as an original source? As software like inkscape can edit it

Added svg instead.


Could  you share the I3D features of the ActivityNet 1.3 dataset?

https://github.com/ispc-lab/ACM-Net

thank you so much.


BUILD-1076: Restore stable-v1 Channel

Restoring the stable-v1 channel, which we created as part of the Builds for OpenShift 1.1 release. We cannot remove operator channels without risk of breaking customers who installed Builds for OpenShift on their clusters.
Was removed in #12.

Closing - will replace with #13


Undecompose API according to the number of parameters

With #295, it looks like we don't need this decomposition anymore.
Let me know if you had specific cases where it was needed, we should add expect-error tests to make sure we can rely on typescript to infer things correctly, at least in regular use cases

SHIPIT!
pretty sure @skiplabsdaniel will be ok with it.


supercluster types error

I'm using react-mapbox-gl  v4.4.0  and @types/supercluster v5.0.1.
On building / running, I get this:
ERROR in [at-loader] ./node_modules/react-mapbox-gl/lib/cluster.d.ts:4:10      TS2305: Module '"../../@types/supercluster"' has no exported member 'Supercluster'.
Can see something similar came up before, but I can't find any other mention of this issue.  Am I doing something wrong?

This project still use supercluster 3.x

Ah, silly me - problem solved by installing<EMAIL_ADDRESS>


GH-701 Fix type cast honoring global script classes

Fixes #701

Applied to main (=2.1) and branch 2.0.


Update silencer-lib, silencer-lib_2.13.7, ... to 1.7.15

About this PR
📦 Updates

com.github.ghik:silencer-lib
com.github.ghik:silencer-lib_2.13.7
com.github.ghik:silencer-plugin

from 1.7.9 to 1.7.15
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.ghik" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.github.ghik" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #261.


Simplify multi-processing error handling

This ensures that any spawned grandchildren that are terminated or exit with a non-0 code are correctly handled.

Somewhat related to #47

Most of yesterday was spent trying to figure out a very flaky test 1/10 tests. I suspect it was an actual bug, but one that only occurred if you killed multiple children at nearly the same time.
Strangely, using a threading.Lock did not fix it, but using a boolean switch to ensure that a shutdown only gets called once worked. I hope... tests are currently running.

Well, that test still fails. Which is weirdly stochastic since locally they succeed. Perhaps the number of workers involved... I best revert the last odd commit.

Ofc, adding debug print to check what happens in the actual CI testing makes things pass. Let's see if reverting them has this pass the tests...

@pkcakeout , tagging you since you have the most knowledgable person about process level management than I know =)

Seems there was a test that relied on the explicit ordering of the results matching that of the predictions. This is now fixed.

Explanation...:
def listen_to_children_errors():
    def handler(*_, **__):
        print("A child failed, terminating all other children")
        _terminate_child_processes()

    signal.signal(signal.SIGCHLD, handler)

This does only check if a child-process has terminated in some form. Not if it is successful or not. Try executing this little sample program (requires Linux!):
import signal
import os
import sys

def listen_to_children_errors():
    def handler(*_, **__):
        print("XXX")

    signal.signal(signal.SIGCHLD, handler)

delay = 1

def create_child():
    global delay

    delay += 1
    import os
    pid = os.fork()
    if pid == 0:
        print("Child process")
        import time
        time.sleep(delay)
        print("Child process done")
        sys.exit(0)
    else:
        return pid

if __name__ == '__main__':
    listen_to_children_errors()
    children = [create_child() for _ in range(5)]
    print("Waiting for children to finish")
    for _ in children:
        os.waitpid(-1, 0)
    print("All children finished")

All child processes terminate successfully, but the handler is called. I do not think this is what you want: Terminating all child processes in case anything finishes what it is doing successfully. This is the expected way things should work.
I am not quite sure how you want to handle this case... why is it necessary in the first place?

Explanation...:
def listen_to_children_errors():
    def handler(*_, **__):
        print("A child failed, terminating all other children")
        _terminate_child_processes()

    signal.signal(signal.SIGCHLD, handler)

This does only check if a child-process has terminated in some form. Not if it is successful or not. Try executing this little sample program (requires Linux!):
import signal
import os
import sys

def listen_to_children_errors():
    def handler(*_, **__):
        print("XXX")

    signal.signal(signal.SIGCHLD, handler)

delay = 1

def create_child():
    global delay

    delay += 1
    import os
    pid = os.fork()
    if pid == 0:
        print("Child process")
        import time
        time.sleep(delay)
        print("Child process done")
        sys.exit(0)
    else:
        return pid

if __name__ == '__main__':
    listen_to_children_errors()
    children = [create_child() for _ in range(5)]
    print("Waiting for children to finish")
    for _ in children:
        os.waitpid(-1, 0)
    print("All children finished")

All child processes terminate successfully, but the handler is called. I do not think this is what you want: Terminating all child processes in case anything finishes what it is doing successfully. This is the expected way things should work.
I am not quite sure how you want to handle this case... why is it necessary in the first place?

It's a good point. It is a bit of a weird fix because I initially thought a random child dying was not being picked (by the issue linked). However, I have since changed my mind and I think it works just fine. I left it in because I thought it could not hurt, however, your keen eye has made me change my mind.
I've removed it!


Update ms.date

... to reflect actual update history

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/checklist/resiliency-per-service.md
:white_check_mark:Succeeded
View

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


Align Tip Records view with design and polishing

polish a little the img fallback
native browser lazy loading of images
basically add loading="lazy" to the tip images which most modern browsers support (though it is still "experimental")
added hover states of tip blocks #219
Other minor visual issues
clean and format some of the css (TipRecord.vue)

Can you also resolve the conflicts?


Added new information in Eligibility section.

Hi @Rishabh04-02. I added the information that I presented to you a couple of days ago. Thank you so much for the opportunity! :)

@CBID2 thanks for the contribution.

@CBID2 thanks for the contribution.

You're welcome @Rishabh04-02 . Feel free to follow me if you'd like. Google Summer of Code sounds like an interesting program, and I'd love to participate in it.


Fix Vector4, Vector4i, and Projection missing from ClassDB tests

Without this PR, binding Vector4, Vector4i, and Projection default arguments will fail the tests.
I fixed it by adding these, and by removing the default: to ensure this does not happen again.

Thanks!


⚠ Fortnite has degraded performance

In fc8f0d8, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1638 ms

Resolved: Fortnite performance has improved in f01ce4f.


Solution for #138

This is a draft solution of #138.
I have several questions:

During editing of the JavaScript code, I saw that you were using ES5, is it because of compatibility with older browsers (IE)? Can I use ES6 in code (like classes, lambdas, etc.)? As far as I know browser support should be almost full, Compatibility table.

I've been editing values.js module in order to add support for alpha channel. Should I keep it like that or should I create another class, that will work with vector4, instead of vector3 colors? I also thought to rewrite this class using ES6 class syntax.

For the colors that are provided as array, but do not have alpha channel I'll need to append it as 1.0, so far I did on the Python side, but I think it can be done on JavaScript side as well. As far as I understand
serializing/deserialing doesn't affect alpha channel in any way, since it creates 1-D array before transferring data  to the front-end side.

I have deleted some unreachable code and commented code, example1 and example2. Is it safe to delete? Did you plan to use it somehow?

Example of alpha blending

White brain mesh with alpha = 0  with blue regions (alpha = 0.3) on to of the grey brain mesh.
Custom blending parameters used for the white mesh material:
mat.blendDst = BlendFactors.OneMinusSrcAlphaFactor
mat.blendEquation = Equations.AddEquation
mat.blendSrc = BlendFactors.SrcAlphaFactor

mat.alphaTest = 0.1
mat.transparent = True
mat.side = Side.DoubleSide

There is some white glowing on the edges of blue regions and throughout the mesh, I don't know what can be the cause of those, do you have any idea? I want to obtain something as with color blending.
Color Blending Example

That doesn't look like glow to me it looks like the lighter curvature color showing up? Try setting the curvature color to a uniform gray instead of binary light/dark and see if the"glow" goes away.
Also I think the curvature is backward, gyri should be light and sulci dark.
But it looks like it's working!
As for es5 vs 6 with GL (I assume that's what we're talking about?) IIUC it's best to stick with the lowest version reasonably possible in order to maintain highest device and application compatibility. How hard would it be to stick with 5?

Also can you add to the todo list (after alpha blending) to get surface normals working? It will fix / smooth out the appearance of the individual triangles

Oh I was looking at the color blending example, I see what you mean about the glow in the alpha example now. No immediate ideas but I'll think about it.

@OlehKSS for me it looks fine:

See https://github.com/OlehKSS/mne-gsoc2018-3d/pull/2. It looks like the same changes you list above, not sure why it's not working for you?

@larsoner Sticking to es5 will not require anything additionally, but with es6 version of JavaScript code will look nicer in general and more handy features will be available.

@maartenbreddels I have updated pull request and removed most of the styling changes.
Could you explain more on colors as arrays on strings? Because I didn't see that part in documentation, so I don't fully understand what these array will look like and which purpose they are going to serve? As I explained before, I will need to add an alpha channel to the colors specified without one, for now I've done it on Python side. However, I think it might be easier to handle all the cases on JS side, after deserialization, I am not sure where to put this code.
Concerning JavaScript version, as far as I understand you are using Webpack as your module manager and builder, I suppose it can be reconfigured to change ES6 code into ES5, I am not sure how easy it will work with TypeScript. Nevertheless, I suppose it should be done in a separate pull request, alongside code style changes that I've done before.

Also a note that travis isn't happy:
https://travis-ci.org/maartenbreddels/ipyvolume/jobs/387141398#L973
Making travis happy could be a way to determine how trivial it'd be to support newer versions of ES

@maartenbreddels
I have a problem with Travis, with is caused by the fact that on Python side we have pythreejs version 0.85 and on JavaScript side we have threejs version 0.91. Have been facing this kind of problems before?

If you rebase against master, you should get a green light again!

woot! nice job @OlehKSS and thanks for helping out @maartenbreddels !

Could you maybe share some notebooks and data to reproduce the plots above? Maybe it's a good idea to put them in the docs, and similar to the other datasets/examples (e.g. see ipyvolume/examples.py and datasets.py) provide an easy way to download them.

we'll get there eventually @maartenbreddels :-)

It looks so cool, I want it now :)
(from mobile phone)
On Mon, 11 Jun 2018, 22:09 Chris Holdgraf<EMAIL_ADDRESS>wrote:

we'll get there eventually @maartenbreddels
https://github.com/maartenbreddels :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/maartenbreddels/ipyvolume/pull/139#issuecomment-396370182,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABryPWUiwuNIC_zcLQMQpMGU2oNNnMIPks5t7s5cgaJpZM4UUJUr
.


Update desktop guide

As suggested in #548

Name the columns Windows, Mac and Linux (rather than having "OS X", which is less obvious)
Re-order rows so we have all the Yes, Yes, Yes at the top, then Yes, X, X, then [BLANK], YES, X. so it's easier to see what's in Windows (most popular),
Re-order the settings in the section below so it matches the table.
Fix a typo in table, removing "Start App on Login" from Mac

+1


UI changes

Is there an easy way to change UI? look and feel while integrating with an existing project?
Thanks.

This
https://thinktecture.github.io/Thinktecture.IdentityServer.v3.Documentation/docs/advanced/customizingViews.html
or this:
https://github.com/thinktecture/Thinktecture.IdentityServer.v3/issues/306#issuecomment-60138140

Feel free to update the docs with a PR with the second approach. :)


Not all available apps are documented

Forward Spam, Return Path, etc. are not documented in the Web API Filter settings https://sendgrid.com/docs/API_Reference/Web_API/filter_settings.html

Return Path is not an app that is available but forward spam is.
Forward Spam is documented here: https://sendgrid.com/docs/User_Guide/Apps/forward_spam.html  It is confusing that we have it documented in two places, however I believe that with some work that I am currently doing, this confusion should be alleviated.
Thank you!


use of deleted function

Hi, I got this error when making qtum by  make -j2:
CXX      cpp-ethereum/libethereum/libbitcoin_common_a-ExtVM.o
In file included from /usr/include/boost/optional.hpp:15:0,
from /usr/include/boost/format/internals.hpp:20,
from /usr/include/boost/format.hpp:38,
from /usr/include/boost/math/policies/error_handling.hpp:31,
from /usr/include/boost/multiprecision/detail/default_ops.hpp:9,
from /usr/include/boost/multiprecision/detail/generic_interconvert.hpp:9,
from /usr/include/boost/multiprecision/number.hpp:22,
from /usr/include/boost/multiprecision/cpp_int.hpp:12,
from ./cpp-ethereum/libdevcore/Common.h:51,
from ./cpp-ethereum/libethcore/Common.h:28,
from cpp-ethereum/libethereum/ExtVM.h:26,
from cpp-ethereum/libethereum/ExtVM.cpp:22:
/usr/include/boost/optional/optional.hpp: In instantiation of ‘void boost::optional_detail::optional_base::construct(boost::optional_detail::optional_base::argument_type) [with T = dev::eth::owning_bytes_ref; boost::optional_detail::optional_base::argument_type = const dev::eth::owning_bytes_ref&]’:
/usr/include/boost/optional/optional.hpp:230:20:   required from ‘boost::optional_detail::optional_base::optional_base(boost::optional_detail::optional_base::argument_type) [with T = dev::eth::owning_bytes_ref; boost::optional_detail::optional_base::argument_type = const dev::eth::owning_bytes_ref&]’
/usr/include/boost/optional/optional.hpp:526:46:   required from ‘boost::optional::optional(boost::optional::argument_type) [with T = dev::eth::owning_bytes_ref; boost::optional::argument_type = const dev::eth::owning_bytes_ref&]’
cpp-ethereum/libethereum/ExtVM.cpp:108:22:   required from here
/usr/include/boost/optional/optional.hpp:346:8: error: use of deleted function ‘dev::eth::owning_bytes_ref::owning_bytes_ref(const dev::eth::owning_bytes_ref&)’
new (m_storage.address()) internal_type(val) ;
^
In file included from cpp-ethereum/libethereum/ExtVM.h:27:0,
from cpp-ethereum/libethereum/ExtVM.cpp:22:
./cpp-ethereum/libevm/ExtVMFace.h:72:2: error: declared here
owning_bytes_ref(owning_bytes_ref const&) = delete;

fixed by upgrading boost to 1.65

Why don't you add the following sentence to 'Build on Ubuntu' on ReadMe.md?
Note: The version of Boost must be 1.65 or later.


ViewSTL app

This is a first version of an STL file viewer. The code makes use of a few inlined C routines, therefore both a version with a base64 encoded binary blob (viewstl.min.js) and the full source (viewstl.app.js) are provided. I am too much of a JS/HTML amateur to figure out how to make a nice interface for uploading STL files, I am using the web IDE upload file functionality instead.

This is awesome - thanks!

... I wonder how long before someone does a raycasted game with inline C :)


Losslessly compressed images to speed up site

Could you also take care of the images on the about, connect, how and team pages? Those are the ones that are especially slow in loading right now. And is there a reason for (only?) compressing two of the team pictures?

This should be good to go. Something was wacky last time I ran the page insights because those were the only ones that had come up. Not this time! :)

@tinakumar This is not really the wanted result: at least some of the pictures have changed in size (width/height). You can see the result of that in this image for example:

Then other files have been reduced in quality to the extent that they are not viable on the page. For example:

The idea is to losslessly compress the images, meaning they neither change in their dimensions, nor in the quality. Looking at the original site, using Dev Tool's Page Speed analyzer, it doesn't offer to losslessy compress many of the pictures you have compressed. Not sure why they have been showing up for you in Page Speed...
Regarding the big header images, maybe you'll need to use an external tool to compress those without changing their dimensions. While speed is important, it is also important to keep the dimensions and the quality of the images as is.

Hey Tina, is this still something you would like to look at at some point, or shall I re-assign the task? Thanks! :)


Regular expression comparison ignores ES2015 "unicode" flag

e.g., /\u{1D306}/u.flags === "u".
Using the new flags property in equiv:callbacks["regex"] should future-proof us against all future additions.

and it seems like a easy one to fix.
@mixed, would you like to help fixing this?

@leobalter
OK. I have fixed it. :smirk:


CLN: Have Sound.__init__ validate that data.dtype is float64, and warn if abs(min/max) >> 1.0

Related to #126, see discussion there

Not doing this, see #131


DTL2020 new commits

I seen DTL2020 have new commits, can you add this??

There is branch master-1 where those changes are applied ( https://github.com/Asd-g/AviSynth-JincResize/tree/master-1 ). It's still work in progress.

Thanks


Fixup jupyterlab1 theme support

Ping @jasongrout

@jasongrout I am thinking of tagging a new 0.11.x version - also maybe supporting JupyterLab 2.0.


SQL Error on Minecraft 1.8 server running PEX 1.23.2/2x(?)

Description
[Insert description of issue here]
Detailed Information
Is available here

Unfortunately PermissionsEx is currently unmaintained and no further updates are planned. Because of this, all pull requests are being closed. If you are looking for an actively maintained permissions plugin, take a look at LuckPerms. It is compatible with major server platforms and can import data from PEX.