Facing Issue related to completing challenges. Getting something went wrong error.

Issue Description

Not able to go to next challenge or visit challenges in map.
Map is totally unresponsive. Cannot visit any links.   Cannot complete and go to next challengesas well by pressing Ctrl-Enter or Clicking Modal.
I get Something went wrong error everytime. Attached screenshots for better understanding.
Browser Information

Browser Name, Version: Firefox - 58.0b7 (64-bit), Chrome - 63.0.3239.132 (Official Build) (64-bit)
Operating System: Ubuntu 17.10, 64bit
Mobile, Desktop, or Tablet: Desktop

Screenshot

The issue still persists! So it should be an open issue

@LiteSoul Both issues are open. No-one has closed either issue

Hi All
Agreed still seems to be an issue.
Get the same error. Base in South Africa.
Interestingly on a 4Mbps Fiber connection i get the issue. On a 10Mbps +- 3G connection i do not get the message.  Perhaps some sort of time out setting some where not allow lower bandwidth connections to download all. Just a guess.

Hi @BerkeleyTrue @Bouncey
From what I understand the only way to cleanly get around this is to have a pre-loading animation while the bundle is available.
The issue, is that actions done while the bundle is in flight, break due to the RX pipeline. I am not an expert, so do not have a pin point cause. And this subsequently results in the exception being thrown. Post which everything is blocked.
What I mean is that that if we let the intial download / render go through without any interuption, our app performs much better.
Now, I am not saying we should ignore the underlying errors in anyway. They should be fixed.
But, I think one way to go would be to not perform actions while the app is not ready.
Berkeley's previous fix, on reducing and lazy loading seems to have improved this performance quite good, but, I am afraid even with CDN, a majority of user's behind slow networks (average 3G like speeds) or VPNs are still needing quite a bit before the bundle is available.
I would love some thoughts on the approach here?

@raisedadead I'm going to add source maps to beta so we can pinpoint where the error is happening. Hopefully we can find it without having to resort to loading icons, which would remove all the work we are doing make ssr happen.

Testing locally it appears the first initial error (the xhr timeout) is the map service.

This is most likely leading to the other errors but I'm not able to replicate. Adding source maps should help illuminate the problem. The map is definitely a problem, since we've added so many challenges it has ballooned up to several hundred challenges in dozens of blocks. We'll need to optimize this before beta launch.
A possible solution is to make this a static file but this has the draw back of needed to build for every updated.
I know this is something that could be solved with graphql, but I'd rather not add more complexity (moving the goal post) to an already hugely delayed project.
I'm open to other suggestions as well.
cc @Bouncey

How about splitting the map service into two separate services.

map service: provides data for rendering the map UI only
challenge service: provides data relevant to the current block/superblock only. Provides other blocks/superblocks on request.  I.e the user finishes a block and moves on to the next block. Logic for this could be when executing a challenge, check to see if the current challenge is the last in the current block. If so, request the next block in anticipation

That could be done, the map service would just deliver the map tree. We then use the rest api (which has limit/skip) to get the rest of the information. This would add some complexity but it wouldn't be so bad.

Would you even need to skip/limit? We could just request by block name. If my current challenge is in the ES6 block, I don't need the redux block, or any of the others. Only the ES6 block.
If I click on a challenge outside of my block and the data is not in the store, I could have a loader rendered while the challenge service brings me the data for that block.
We could even take it as far as caching the blocks in local storage, but that would be down the line

We could just pull a block and all it's challenges, but keep in mind that some blocks have dozens of challenges.

That's what I was trying to say. One block's worth of challenges in a single payload is a lot lighter than all of the challenges from every block. If the user doesn't need it right away, we shouldn't send it.

Agreed

I am no expert and know very little. Just started the journey to learn all this.
But just another guess, is there no where one can change the timeout limit? Perhaps make it higher to allow lower bandwidth connections time to download.

@kdesCode Whilst that would help in alot of cases, the simple fact is we are sending around 6MB of challenge data to everyone, even when they don't need it. The fix suggested is to send selected chunks of challenges that are relevent to the user.
This will reduce the amount to be downloaded, which will have the side effect of reducing the time required for the request to complete.

The direction of this issue's discussion is apparent: Apollo client with GraphQL/Express server. This is just something I wanted to propose, but a migration would definitely delay the release greatly.

Well, anything is better that the current state which is virtually unusable.
Me learning GraphQL is just months away, sorry I can't contribute right now... but maybe Bouncey suggestion is simpler

@BerkeleyTrue

Sweeeet!

Amazing! If you need any help with Apollo I'm down

@chethanbhat , @dhuber666 , @kdesCode do you still experience the issue? It may have been fixed. I have it sometimes, but not all the time.

Cross-posting so that you can still keep working on the challenges until we get this fixed

What you could do to keep the challenges coming is navigate directly to the challenges themselves. Hovering over the challenge in the map will show you the address of the next challenge.

I know this is not ideal, I should have a PR ready today.


fix: stop spying on a method, when it's restored

Fixes #3127

So what are vitest consumers meant to do when they want to restore original implementation and continue spying ?

So what are vitest consumers meant to do when they want to restore original implementation and continue spying ?
mockClear doesn't affect implementation and mockReset restores an empty implementation (not the original)

Please, don’t comment on old and unrelated pull requests. If you have a question, formulate it and ask in discussions page or in discord.


coqPackages_8_13.VST: fix by using compatible version of ITree

Description of changes
Follow-up of https://github.com/NixOS/nixpkgs/pull/220042#issuecomment-1488474874 (cc @vbgl)
I am currently testing that VST builds correctly like this, but I can already confirm that it evaluates with the expected (compatible) version of the ITree dependency when evaluating coqPackages_8_13 and with the default version otherwise.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

File "./progs64/nest3.v", line 2, characters 0-85:
Error: Cannot load Flocq.IEEE754.SpecFloatCompat: no physical path bound to
Flocq.IEEE754

It's strange because Flocq 3.4.3 is provided (as part of the propagated build inputs of CompCert 3.10) and it does provide this module.

I’ve just tried the following patch instead, and it seems to work:
--- a/pkgs/top-level/coq-packages.nix
+++ b/pkgs/top-level/coq-packages.nix
@@ -112,10 +112,12 @@ let
       trakt = callPackage ../development/coq-modules/trakt {};
       Velisarios = callPackage ../development/coq-modules/Velisarios {};
       Verdi = callPackage ../development/coq-modules/Verdi {};
-      VST = callPackage ../development/coq-modules/VST (lib.optionalAttrs
+      VST = callPackage ../development/coq-modules/VST ((lib.optionalAttrs
         (lib.versionAtLeast self.coq.version "8.14") {
           compcert = self.compcert.override { version = "3.11"; };
-        });
+        }) // (lib.optionalAttrs (lib.versions.isEq self.coq.coq-version "8.13") {
+          ITree = self.ITree.override { version = "4.0.0"; };
+       }));
       zorns-lemma = callPackage ../development/coq-modules/zorns-lemma {};
       filterPackages = doesFilter: if doesFilter then filterCoqPackages self else self;
     };

Thanks! Never mind why my patch was not working, yours looks like a more principled and simpler fix!

The previous error was indeed rather mysterious… Thanks again!


Pagination only works when there's already a hash in the URL

I'm having an issue with the pagination where it'll only work when there's already a hash in the URL. I.e., when I go to the page once it doesn't work but if I reload with a section data name, it'll then work. I copied your example, $(".pagination a").on("click",$.scrollify.move); which again, does work if there's already a section hash in the URL. Any thoughts?
Also, unfortunately, I'm working on my site locally so I can't provide an example.

Sounds like you're seeing the browsers native anchoring behaviour, rather than Scrollify scrolling the page.
Are there any messages coming up in the console?

No, I'm not receiving any messages of any kind actually. Here is my complete script, maybe I'm looking at something obvious:
`    // Initialize Scrollify
$.scrollify({
section : ".vc_section",
sectionName : "section-name",
interstitialSection : "",
easing: "easeOutExpo",
scrollSpeed: 1500,
offset : "",
scrollbars: false,
standardScrollElements: "",
setHeights: true,
overflowScroll: true,
updateHash: true,
touchScroll: true,
before:function(i,panels) {
var ref = panels[i].attr("data-section-name");
        $(".pagination .active").removeClass("active");

        $(".pagination").find("a[href=\"#" + ref.replace(/\s+/g, '-') + "\"]").addClass("active");
    },
    after:function() {},
    afterResize:function() {},
    afterRender:function() {
        var pagination = "
";
        var activeClass = "";
        $(".vc_section").each(function(i) {
            activeClass = "";
            if(i===0) {
                activeClass = "active";
            }
            pagination += "
" + $(this).attr("data-section-name");
            pagination += "" + $(this).attr("data-section-description") + "";
            pagination += "
";
        });

        pagination += "
";

        $("#pagination").append(pagination);
    }
});

// Move to section on pagination click
$(".pagination a").on("click",$.scrollify.move);`

That all looks fine. Do you have any IDs set on your .vc_section elements?

Yea they are. All good, if you don't see anything noticeable in the initialization I'll assume there's a conflict somewhere else. Thank you much!

If their IDs match the section-name then that will be the reason for your issue.

Which ID's? The sections aren't using ID's, only the data-section-name. If I add the same ID's to the section, it'll jump normally but won't use the Scrollify attributes.

Exactly, that's what I was trying to ascertain. Your previous messages suggested you were using IDs.


Automated cherry pick of #3690: remove useless aws instance type config

Cherry pick of #3690 on release/2.12.
#3690: remove useless aws instance type config

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@02a8de8). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.12   #3691   +/-   ##
==============================================
  Coverage                ?   6.94%
==============================================
  Files                   ?     500
  Lines                   ?   84023
  Branches                ?       0
==============================================
  Hits                    ?    5838
  Misses                  ?   77691
  Partials                ?     494

Flag
Coverage Δ

#aFlag
6.94% <0%> (?)

Impacted Files
Coverage Δ

pkg/multicloud/aws/region.go
0% <ø> (ø)

pkg/compute/models/skus.go
1.29% <0%> (ø)

pkg/multicloud/aws/host.go
0% <0%> (ø)

pkg/compute/guestdrivers/managedvirtual.go
0% <0%> (ø)

pkg/multicloud/aws/instance.go
0% <0%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 02a8de8...ba2da67. Read the comment docs.

/approve
/lgtm


Bug: Scheduling post obscure error

Issue Summary
When attempting to schedule a post less than 2 minutes into the future, an error notification is provided with no details as to what went wrong:

Error is shown in console:

Steps to Reproduce

Load a draft post in Ghost-Admin
Open PSM and select time 1 minute in future.
Save Draft
Schedule Post

Technical details:

Ghost Version: https://github.com/TryGhost/Ghost/commit/71282a241e76608df5d99112df005ebbc7a3204b
Node Version: 4.4.7

I'm not able to reproduce this.
When I select a draft post and set the publish time to be 1 minute in future, click on Save Draft, then my publish time get reseted to what it was before. I don't have the Schedule Post option.
I was able to reproduce the following situation:

Load draft post
Open PSM and select 3 minutes in future
Save Draft
Wait 2 minutes
Schedule Post

Usually, this should (and did, when we merged it in first place) output the servers' error message in the red notification like this:

I'm on it now and try to find out where it goes wrong...

I have the same Issue.


DoubleTapped event is not fired correctly when the event source changes

Describe the bug
When having a DoubleTapped event on a control, and the control changes during double pressing, the double tapped event is not then correctly emitted for the new control when continuously clicking.
Scenario is a file explorer-like container with items in it, double clicking opens a folder. Opening the folder would present another folder control on the same place, and when user wants to double click that item quickly, it wont work since the gesture is still tied to the previous control.
To Reproduce
Steps to reproduce the behavior:

Make a List/GridView of items
The items have DoubleTapped event handler, which replaces the content of the List/GridView with another items
Double clicking the first item changes the content, don't stop clicking with input device nor move the pointer.
Another items is presented under the pointer, however the List/GridView content is not changing due to DoubleTapped event not being produced.
Moving the pointer, or pausing clicking for a second and then proceeding again correctly changes the content again (DoubleTapped event is produced)

Expected behavior
Double clicking a control would trigger DoubleTapped event, and proceeding with clicking while the underlying control changes should still trigger another double-clicked event when clicking at least twice on the new control.
Desktop (please complete the following information):

OS: Windows
Version [e.g. 0.10.13]

It seems problem is in this method https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Base/Input/Gestures.cs#L92-L114
Gesture is initiated only when there is <= 1 click count. And if user continuously clicks for 3th or 5th time it doesn't reset the gesture.


Toggle command mode

As a user, I can toggle input commands without repeating the command word.
Examples:
Go into add mode, then no need for user to input add command in front until user exits add mode.

As an advanced user, I can toggle between input commands without repeating the command word, to increase the efficiency of operations.

User story moved to Project user stories board here.


No module named create-flask-app

Hi there,
I would love to use this, but when I run python -m create-flask-app my-app , I get the following error:  C:\Users\user\anaconda3\python.exe: No module named create-flask-app
Python 3.8.3 (default, Jul  2 2020, 17:30:36) [MSC v.1916 64 bit (AMD64)] :: Anaconda, Inc. on win32
setuptools 49.2.0.post20200714  (Python 3.8)

Hello,
I am aware of this problem. This problem is caused by improper project structure and incorrectly set up setup.py config file.
I was thinking of solving this issue for good next week, unless you want to have a go at it.

The fix is merged into master(c78cc0b4accfd692583857fab58637f7eeac32ff), as well as thoroughly tested and published to pypi.
Closing for now, feel free to ask anything if something is left unclear.

The fix is merged into master(c78cc0b4accfd692583857fab58637f7eeac32ff), as well as thoroughly tested and published to pypi.
Closing for now, feel free to ask anything if something is left unclear.

Great it is working now, thanks!

Great it is working now, thanks!


Correct vector's size begin checked

This caused a null pointer being used when called with p_vtx_attrs = nullptr and p_res_bindings != nullptr

 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.


chore(weave): LiteLLM Support + Stream-by-flag

We need to support a common vendor pattern: streaming is a flag, not a different symbol. This PR adds support for this case as well as support & tests for LiteLLM

Thanks for integrating litellm here @tssweeney curious - do you also use the proxy internally?

Thanks for integrating litellm here @tssweeney curious - do you also use the proxy internally?

Hey @krrishdholakia! We do not use the LiteLLM proxy internally, but would be interested in learning more about the capabilities. I'm still working on this PR, but soon i'll have some screenshots to show the results of the integration

Hey @tssweeney here's the quick start - https://docs.litellm.ai/docs/proxy/quick_start


file_get_contents error with Portuguese and Deutch

Hello, i'm having an error when i try to process a text in Portuguese and Deutch.
ErrorException
file_get_contents(http://localhost:8011/v2/check): Failed to open stream: HTTP request failed! HTTP/1.1 500 Internal Server Error

That's my code, and it's working with Spanish, English, etc.
if ( $from == 'pt' ) {
     $from = 'pt-PT';
}
if ( $from == 'de' ) {
     $from = 'de-DE';
 }
// LanguageTools expects language formatted with a dash `en-US`
$misspellings = $spellchecker->check($text, [$from], ['from_example']);

I've tried with pt-PT and de-DE with a the silviof/docker-languagetool docker image and it returns a proper misspelings iterator of


Revert "Force CI to test against ubuntu-20.04"

Reverts GameBridgeAI/ts_serialize#122

ouefff messy, but all good, I think yuo need to update the settings again, damn. I'll look into making that better in the future


vscode use too much memory ，it can reach 14

4G that is 95% of my computer .
vscode 1.42.1
os win7

@wflixu please review https://github.com/microsoft/vscode/wiki/Performance-Issues and consider submitting more information.

Thanks so much，I disabled all the extendsions, then the problem was reversed. So is caused by some extendsion I installed.

You can try to enable one extension at a time and see when the problem shows up again.


xSQLServerRole: Fix for Issue #790

Pull Request (PR) description
Explicit Casting to System.String[] for $membersInRole
This Pull Request (PR) fixes the following issues:
Fixes #790
Task list:

[x] Change details added to Unreleased section of CHANGELOG.md?
[x] Added/updated documentation, comment-based help and descriptions in .schema.mof files where appropriate?
[x] Examples appropriately updated?
[x] New/changed code adheres to Style Guidelines?
[x] Unit and (optional) Integration tests created/updated where possible?

This change is

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #791 into dev will not change coverage.
The diff coverage is 100%.

@@         Coverage Diff         @@
##            dev   #791   +/-   ##
===================================
  Coverage    97%    97%
===================================
  Files        30     30
  Lines      3255   3255
===================================
  Hits       3170   3170
  Misses       85     85

First, thanks for sending this fix in! 😄
Could you please

Add this change to the Unreleased section of the CHANGELOG.md.
Add a regression test (that fails for the old code, and passes for the new code) in MSFT_xSQLServerRole.Tests.ps1.

If you need help with anything, let me know! 😄

I updated the PR description to use the template. You removed that by mistake when entering you PR description, when you sent in the PR.  It's easier for me to have the PR's use the template. But no worries. It was easy fixed. :)

Added entry to changelog.md and "Should return the members as string array"

Just minor review comments! Great work! :smile:

Reviewed 1 of 1 files at r3, 2 of 2 files at r4.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

CHANGELOG.md, line 17 at r4 (raw file):

Fixed Error due to Return Variable Type

Please change to something more descriptive. Like "When running Get-DscConfiguration it longer throws an error saying that the property Members is not an array (Issue #790)."

DSCResources/MSFT_xSQLServerRole/MSFT_xSQLServerRole.psm1, line 70 at r4 (raw file):

            try
            {
                [System.String[]]$membersInRole = $sqlServerRoleObject.EnumMemberNames()

Please add a space between the type and the variable name
[System.String[]] $membersInRole = ...

Tests/Unit/MSFT_xSQLServerRole.Tests.ps1, line 215 at r4 (raw file):

                }

                It 'Should return the members as string array' {

Could you please add a comment saying that this is a regression test?
# Regression test for issue #790

Please do this for each of these new tests

Comments from Reviewable

Did the changes you mentioned in the review

i'm new to "code-review/reviewable", how can I acknowledge the changes you mentioned in the discussion ?

Could you please go into Reviewable and write 'Done' (or click the Done-button) on each review comment you solved (or comment if it needs to be discussed). After you replied on all review comments, then you press the big publish-button at the top of the Reviewable page and all your comments will be sent back here.
When you do this I can acknowledge the comments which will resolve the reviews comments.
You find the link to Reviewable in the big purple button in the PR description.

CHANGELOG.md, line 17 at r4 (raw file):
Previously, johlju (Johan Ljunggren) wrote…

Fixed Error due to Return Variable Type

Please change to something more descriptive. Like "When running Get-DscConfiguration it longer throws an error saying that the property Members is not an array (Issue #790)."

Done.

Comments from Reviewable

DSCResources/MSFT_xSQLServerRole/MSFT_xSQLServerRole.psm1, line 70 at r4 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
Please add a space between the type and the variable name
[System.String[]] $membersInRole = ...

Done.

Comments from Reviewable

Tests/Unit/MSFT_xSQLServerRole.Tests.ps1, line 215 at r4 (raw file):
Previously, johlju (Johan Ljunggren) wrote…
Could you please add a comment saying that this is a regression test?
# Regression test for issue #790

Please do this for each of these new tests

Done, Added comment in front of Regression Test

Comments from Reviewable

Reviewed 1 of 1 files at r3, 2 of 2 files at r4, 3 of 3 files at r5.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

Comments from Reviewable

Reviewed 3 of 3 files at r5.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable

@seizste Awesome work! Thanks for this! 😄

@johlju Thanks for your help and guidance


Refactor create config file

Signed-off-by: Vijayan Balasubramanian<EMAIL_ADDRESS>Description
chmod doesn't work for windows operating system. Hence, in windows
don't create default config file and check for permission. This will
let users responsible for creating their own config file and set
appropriate permission.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has documentation added

[ ] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #36 (5a21dae) into main (d966330) will increase coverage by 0.07%.
The diff coverage is 76.00%.

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
+ Coverage   67.19%   67.26%   +0.07%
==========================================
  Files          30       31       +1
  Lines        1844     1848       +4
==========================================
+ Hits         1239     1243       +4
  Misses        499      499
  Partials      106      106

Flag
Coverage Δ

opensearch-cli
67.26% <76.00%> (+0.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

commands/root.go
81.81% <ø> (+4.67%)
:arrow_up:

commands/file_operation_unix.go
73.91% <73.91%> (ø)

commands/profile.go
51.66% <100.00%> (-1.06%)
:arrow_down:

Tested in windows


Little fixes to stop buildifier warnings and fix CI.

Spelling fixes. BUILD file repair.

Do not merge until after https://github.com/bazelbuild/rules_pkg/pull/784 is merged


Error invoking Swift hello world example

Following this guide and the OpenWhisk installation guide, I set up a Swift project and got signed into IBM's system.
At that point, serverless deploy ran successfully, but sls invoke -f hello returns this:
bensyverson@laptop:~/Local/helloswift$ sls invoke -f hello

  Serverless Error ---------------------------------------

  Failed to invoke function service (hello) due to application error:

     error: terminated(1): /usr/bin/swift-build-tool -f /swiftAction/.build/release.yaml main output:

Compile Swift Module 'RestKit' (8 sources)
Compile Swift Module 'VisualRecognitionV3' (19 sources)
Compile Swift Module 'ToneAnalyzerV3' (14 sources)
Compile Swift Module 'PersonalityInsightsV3' (11 sources)
Compile Swift Module 'NaturalLanguageUnderstandingV1' (45 sources)
Compile Swift Module 'NaturalLanguageClassifierV1' (10 sources)
Compile Swift Module 'LanguageTranslatorV3' (12 sources)
<unknown>:0: error: unable to execute command: Killed
<unknown>:0: error: compile command failed due to signal 9 (use -v to see invocation)

1

     Check logs for activation: 5cb6...

  Get Support --------------------------------------------
     Docs:          docs.serverless.com
     Bugs:          github.com/serverless/serverless/issues
     Issues:        forum.serverless.com

  Your Environment Information -----------------------------
     OS:                     darwin
     Node Version:           11.7.0
     Serverless Version:     1.36.3

So obviously the first problem is the fact that the example doesn't run. But if you look at the Swift modules in the output, it's kind of alarming. PersonalityInsightsV3, VisualRecognitionV3... why are these being compiled with my function? Are they being applied to my input or output?

@bensyverson Hmm that doesn't look good!
Let me try to look at this during the week.

Beautiful, thank you!


Adding build dependency on TH2 SDK

Hi @rachitb777, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Feature request: Custom error messages and fallback image

Hey there, yesterday (by mistake) I upgraded to v2 and am pretty impressed of the new UI and features. Awesome work!
What I already missed in v0 is the possibility to individualize error messages. A custom error text, at best with automatic localization, would be an excellent start. As an extension it would be great if instead of the black frame the last snapshot or a fallback image could be shown.

Hey @philipptrenz,
Thanks for the words. We are doing our best :-)
Can you tell us which messages you are missing? And describe that in more detail?
The problem with the stability is known and not so easy. I can say more about that as soon as we have found the best and cleanest solution.


information mismatch with App Service document

[Enter feedback here]
This document indicate below. We can understand "User needs to deploy DNS proxy for name resolution from App Service using virtual network integration".
Scenario : Name resolution from an Azure App Service (Web App, Function, or Bot) using virtual network integration to role instances or VMs in the same virtual network.
Solution : Customer-managed DNS servers forwarding queries between virtual networks for resolution by Azure (DNS proxy). See Name resolution using your own DNS server.
However, the document from App Service indicate below.
https://docs.microsoft.com/en-us/azure/app-service/web-sites-integrate-with-vnet#azure-dns-private-zones
These settings send all of your outbound calls from your app into your VNet and enable your app to access an Azure DNS private zone. With these settings, your app can use Azure DNS by querying the DNS private zone at the worker level.
Which is correct information? If it's correct information from App Service, I'd like you to revise DNS document.
Thanks,

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

ID: a7b1b215-0434-145b-cd11-95d0f32bc015
Version Independent ID: d328a868-a151-1985-2b62-f3884f7ad76c
Content: Name resolution for resources in Azure virtual networks
Content Source: articles/virtual-network/virtual-networks-name-resolution-for-vms-and-role-instances.md
Service: virtual-network
GitHub Login: @rohinkoul
Microsoft Alias: rohink

thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

@GitaraniSharma-MSFT Please change the service label to dns/svc . Thanks!

@GitaraniSharma-MSFT Please remove label virtual-network/svc. Thanks!

@mbender-ms , label virtual-network/svc removed.


[Technical] Implementation of general debounce function

Technical

Create a service function to debounce functions
Located under services

Problem description

Debouncing is implmenent in typescript via timeout. To not flood the code with various timeout function a abstract debounce function is needed to outsource some boilerplate code.
Possible approch

https://www.freecodecamp.org/news/debouncing-explained/
https://stackoverflow.com/questions/23123138/how-to-perform-debounce
https://blog.logrocket.com/create-custom-debounce-hook-react/
Look at ProjectDescriptionStepper.hooks, ProjectDescriptionCard.hooks and Searchbar for more information of timeout usage

Acceptance criteria

[ ] Debounce function is successfully implemented
[ ] timout functions were replaced with debounce function

Definition of Done

[ ] All acceptance criteria are met
[ ] The implementation of the User Story has been successfully accepted by the Product Owner
[ ] The implementation is in GitHub
[ ] A code review has been performed (or created in pairing program)
[ ] Code conventions are followed
[ ] Unit test coverage must be greater than 90%
[ ] All tests must have passed
[ ] Technical documentation created (Feature documented in GitHub Wiki)
[ ] UML class diagrams and component diagrams created and up-to-date
[ ] Documentation in realization created (source code)
[ ] Changelog created (Clear description in merge request)
[ ] There are no known bugs
[ ] Rendering behaviour is optimized

Hat Martina

Hat Martyna immernoch.


SERVICE WIKA SWH<PHONE_NUMBER>9

SERVICE WIKA SWH<PHONE_NUMBER>9.

Hi @andrewijaya, I'm your friendly neighborhood Microsoft Open Technologies, Inc. Pull Request Bot (You can call me MSOTBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.msopentech.com.

TTYL, MSOTBOT;


Training loss

Excuse me. I use "python3 train.py configs/car_auto_T3_train_train_config configs/car_auto_T3_train_config" for training on KITTI dataset. But the loss is showed as follow:
cls:nan, loc:nan, reg:nan, loss: nan
Class_0: recall=0.957521, prec=0.969269, mAP=0.000000, loc=0.000000
x=0.0000 y=0.0000 z=0.0000 l=0.0000 h=0.0000 w=0.0000 y=0.0000
Class_1: recall=0.145631, prec=0.034169, mAP=0.000000, loc=1.847784
x=1.7500 y=2.4132 z=2.5065 l=0.2367 h=0.1836 w=0.1582 y=5.6863
Class_2: recall=0.003945, prec=0.019802, mAP=0.000000, loc=3.094378
x=0.8403 y=2.0149 z=12.8085 l=0.0637 h=0.0760 w=0.2339 y=5.6234
Class_3: recall=0.002632, prec=0.004808, mAP=0.000000, loc=0.000000
x=0.0000 y=0.0000 z=0.0000 l=0.0000 h=0.0000 w=0.0000 y=0.0000
I dont know why. If you have faced the same error, please point out how to fix that. Thank you very much!

Hi @wb-finalking Could you help to provide more info?

Did you change config or train_config? e.g. num of GPU and batch size
On which epoch does the Nan occur? If it's the first epoch, can you try to clear the checkpoint and start again?

Hi @WeijingShi  I change the batchsize to 1 and gpu num to 1. When I decrease the learning rate, cls loss and loc loss became normal, but reg loss still be NAN.

@wb-finalking batch size 1 is pretty small. If you further reduce the learning rate, do you still get the NaN?

I solve this problem by increase batch size to 2. Thanks for your patient answer.

Good know that it works out!


Assertions not triggering with -O and test suite failure

Hi,
version <IP_ADDRESS>.
When you test with  ./Setup build test:test-evp-base64 --ghc-option=-O0 or in ghci, then a test failure occurs (I've inserted some more stack traces and prints for your convenience):
in assertFunction: print (n, x, y, r)
("decodeBase64BS","YWJjZGVmZ2hpams=\n","abcdefghijk","decodeBlock "YWJjZGVmZ2hpams=\n"
*** Exception: Assertion failed
CallStack (from HasCallStack):
  assert, called at ./OpenSSL/EVP/Base64.hs:111:45 in main:OpenSSL.EVP.Base64
  decodeBlock, called at ./OpenSSL/EVP/Base64.hs:130:18 in main:OpenSSL.EVP.Base64
  decodeBase64BS, called at Test/OpenSSL/EVP/Base64.hs:51:37 in main:Main

This assertion triggers: https://github.com/vshabanov/HsOpenSSL/blob/a9efae0b598b9499443721a1989055ca017fd01f/OpenSSL/EVP/Base64.hs#L109-L111
This is because in the test https://github.com/vshabanov/HsOpenSSL/blob/master/Test/OpenSSL/EVP/Base64.hs#L49-L58
the last test case, "YWJjZGVmZ2hpams=\n" has 17 chars, and 17 mod 4 is not 0.
This assertion failure usually goes completely unnoticed, because assertions are compiled away when -O is used (which is the default).
Are these assertions relevant for security or correctness?
If yes, then they should not be assertions, because assertions should not be used for control flow and input validation. Alternatively, -fno-ignore-asserts can be used.

Bad, bad code. The more I look at this code the more I want to deprecate it in favor of https://hackage.haskell.org/package/base64-bytestring
decodeBase64 "YWJjZGVmZ2hpams=\n" leads to segmentation fault (and decodeBase64 "YWJj\n" too)
decodeBase64BS fails with exception on 1,2,3 character inputs or any other invalid input (due to _DecodeBlock returning negative value)
decodeBase64LBS has segmentation fault on any input not divisible by 4 (due to infinite loop which must be prevented by assert).
Counting '=' is bad as well:
https://github.com/vshabanov/HsOpenSSL/blob/a9efae0b598b9499443721a1989055ca017fd01f/OpenSSL/EVP/Base64.hs#L119
OpenSSL seems to just decode 4-byte blocks (I can't understand what it does with newlines but it seems to stop on them). It can decode "AB==CD==EF==" and removing 6 bytes from output makes no sense.
Current plan:

these asserts must be converted to error
count '=' should be replaced to checking whether input ends on "=" or "=="
negative results of _DecodeBlockmust be handled with error
note about decoding of "AB==CD==" should be added
module should be deprecated in favor of base64-bytestring

@vshabanov That plan sounds good to me!


Do not install dev dependencies by default

Fixes https://github.com/alphagov/govuk-prototype-kit/issues/1128.
Users of the prototype kit do not need to install dev dependencies such as Jest or StandardJS [1]. Currently, because we distribute the kit as a complete Node project rather than as a package, they are installing them by default.
This commit restructures a few things to make sure that dev dependencies are not installed unless explicitly requested. Unfortunately, the way we need to to do this is a little convoluted.
The npm CLI does allow specifying that devDependencies should not be installed by adding a config variable to the .nvmrc file [2]; however, if the directive omit=dev is given, which is what we want, the NODE_ENV environment variable will also always be set to production, which is definitely not what we want, as we use NODE_ENV to determine whether a user is running the kit locally or on Heroku/Glitch.
We could make sure to set NODE_ENV as appropriate for each script, but that seems a bit brittle, and wouldn't account for any non-standard setups. And we'd also have to more workaround code for Glitch.
So instead we move our devDependencies to be optionalDependencies, say omit=optional, and tell developers to make sure to ask for optional dependencies when installing (either with a command line flag or the environment variable npm_config_include=optional). This is an abuse of the optionalDependencies feature, but feels less likely to break things for users.
When/if the kit is made into a package, these changes can be reverted.

Something that just occurred to me, we'll have to be careful in future reviews to make sure we only add to optionalDependencies, not devDependencies.


Add ci (github actions) config

Resolves #8.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@aa872b1). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #19   +/-   ##
=========================================
  Coverage          ?   86.20%
=========================================
  Files             ?       18
  Lines             ?      174
  Branches          ?       50
=========================================
  Hits              ?      150
  Misses            ?       24
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa872b1...6dea65d. Read the comment docs.


Edit history for Issue and PR comments

Feature Description
It would be very helpful if issue and pr comments had an edit history, like github has.
Every comment that has been edited should have a message informing the user about when and by whom the comment was last edited. Ideally, it should also have a way to view older versions and/or diffs.
Currently, there is no such indication, so people could, for example, make a popular comment and after getting approval from others change the content of the original comment to create a false impression for anyone coming late to the discussion.
This is made worse by the fact that by default, every user with write permissions can change every other user's comments without any indication that this has happened - see #17585.
Screenshots

We already have it (in 1.16, I submitted the PR)

https://github.com/go-gitea/gitea/pull/16909

Try it on try.gitea.io


Update Theme: Cleaned url bar

Fixed the shield & star alignment issue.
Added an option to disable transparency.

Oh, perhaps the branch isn't up to date with main, I'll fix that

Okay i think I will just submit a new PR


Upgrade Jetty from 9.4.18.v20190429 to 9.4.31.v20200723

Task Description
We need to upgrade Jetty from version 9.4.18.v20190429 to 9.4.31.v20200723.
Please, note that you might need to override some of the spring-boot Jetty-related dependencies.
Tasks
The following tasks will need to be carried out:

[ ] Update the version in the strongbox-parent project.
[ ] Apply any required code changes, or dependency excludes.
[ ] Test and confirm that there are no issues with the core projects (strongbox, strongbox-web-integration-tests, etc) after upgrading.

Useful Commands
To check the dependency tree:
mvn dependency:tree

Usually, it's more convenient to output the results to a file, so that it's searchable:
mvn dependency:tree > dep.tree

Help

Check our article on how to upgrade dependencies and plugins.
Our chat
Points of contact:

@carlspring
@sbespalov
@steve-todorov

GHC

@MadisonReese ,
Feel free to proceed and good luck! :)

I would like to work on this, if anyone has not taken up this task.

@madisonreese: Are you still looking into this?

Hey, if no one has touched this yet could I try it?

Hey @vivekmthr ! Go for it!

Hi @carlspring! Can I work on this?

Hi, Pablo!
Yes, please! And... yeah it needs to be upgraded to the latest one.


Please help. Same problem. Need assistance

Hi, I am working on almost same thing as yours but the problem is that I have this data of images with text that I have to search. My problem is that how can I do it when I have more than 20M images? Please help. I have raised a question too.
Manage Memory and Speed for Image Search Engine

Use smaller sized vectors so you can load more images in memory.
You can also use fast knn algorithms like ball tree ( available in sklearn) in order to find the closest match.


Revert "Bump GtkSharp from <IP_ADDRESS> to <IP_ADDRESS> (#230)"

This reverts commit 31e3a842295be2362518afa765bc4cc35d31ba50.
Fixes https://bugs.launchpad.net/pinta/+bug/1993035
Note: gtksharp does not have a <IP_ADDRESS> release on their github releases page, the latest is still <IP_ADDRESS>

Thanks for looking into this! Yeah, it's not very clear which GtkSharp commit the latest Nuget package corresponds to. I assumed it was only 4 commits beyond the .34 release
I think what actually happened is that the .38 package includes the changes from https://github.com/GtkSharp/GtkSharp/pull/316, which fixes broken bindings for Gdk.Pixbuf.SaveToStreamv
This was a fix we needed to pull in for the next Pinta release (https://bugs.launchpad.net/pinta/+bug/1959642) but it requires a couple code changes on Pinta's side

That makes sense - could the version bump and related work go in an integration branch?
flavour text: I just found the project today - and I've gotten some great use out of it already, excellent work here.  I had an issue with the snap so I turned to the source, and assumed master would be in a working state.  If that is not the expectation, a heads up in the readme would be good touch

The fix to get Pinta working with the latest GtkSharp is pretty easy, so I've just committed that in e973699
The master branch is intended to be usable but can occasionally have recently introduced bugs like this, so thanks for reporting this!


Look into some failed Fortran tests in humanEval-Fortran

https://github.com/HPC-FAIR/humanEval-Fortran

https://github.com/HPC-FAIR/humanEval-Fortran/blob/main/README.md

Three steps

compile
link
execution

made some progress. A few remaining tests still have problems.


RHEL7 build issue: installing docker_py=1.10.6 fails

Seems like this is result of some work-around from 5fe4cbbea27b2299e61e889f78414a9ec65ee70a commit.   But explicitly installing v1.10.6 fails for me with:
$ easy_install -q --user docker_py==1.10.6 docker-squash==1.0.5
error: Setup script exited with error in docker-py setup command: Invalid environment marker: python_version < "3.5"

OTOH, easy_install -q --user docker-squash==1.0.5 works just fine. Do we still need explicit version of docker_py?

I don't see reason for this workaround - it is quite old so I guess a problem could be fixed now.


feat: add noIrregularWhitespace rule

Summary
Implementation of the no-irregular-whitespace rule from ESLint in Biome.
I also had to do a change in the biome_diagnostics crate to validate the range of irregular whitespaces properly and show them correctly in the output.
Closes #53
Valid
const foo = '\\\u2028';
const foo = '\\\u2029';
/* \u{b}    */ const foo = '';
/* \u{c}    */ const foo = '';
/* \u{20}   */ const foo = ' ';
/* \u{85}   */ const foo = '';

Invalid
/* \u{b}    */ constfoo='thing';
/* \u{c}    */ constfoo='thing';
/* \u{85}   */ constfoo='thing';
/* \u{feff} */ constfoo='thing';
/* \u{a0}   */ const foo = 'thing';
/* \u{1680} */ const foo = 'thing';

Test Plan
Added snapshots, for valid and invalid cases according to the original rule tests.

@michellocana Thank you, could you resolve the conflicts please?

@michellocana Thank you, could you resolve the conflicts please?

done!

Thank you @michellocana for this rule. If you have the chance, can you send a PR to add a changelog entry?

Thank you @michellocana for this rule. If you have the chance, can you send a PR to add a changelog entry?

sure, here it is @ematipico :) #3418


:seedling: Bootmanager for UKI

Options to evaluate, considering static measured boot support in the picture (each one of those have pro/cons):
UKI = systemd-stub + kernel +initrd

Native EFI + UKI
systemd-boot + UKI
GRUB2 + UKI
... (?)

Motivations

is it easy measurable?

systemd-boot + UKI looks like

Some more context:
UKI can bee booted directly from the UEFI interface, after all its an EFI binary. But as soon as you do that, there is no version management, nor boot assesmentet, nor visibility on what you are booting unless you want to enter into the UEFI and select your EFI binary from there directly.
Having a bootloader in front brings a host of improvements to this process. Easy way of seeing your choices, boot assesment in some cases (systemd-boot), automatic fallback Active->Passive-Recovery (which is also possible with UEFI directly but its not as clear, as EFI logging is not that good)
Another point to check, if we use a bootloader to boot the UKI, do we need to bundle systemd-stub in the UKI image?? I guess so, but maybe we dont and its one piece less.

We have evaluated the solutions, with having in mind the following criteria :

Lifecycle ( Upgrade/reset/install actions)
Boot measurement
Boot assessment
Maintainability (code base, etc.)

Taking into consideration, that we have to still develop the UKI upgrade/reset/install mode into the lifecycle management portion of our stack.

Having a boot manager in front  (systemd-boot/GRUB2)
NOT Having a boot manager in front    (native EFI)

Lifecycle ( Upgrade/reset/install actions)
Slightly more complex as it has to consider the boot manager efi file. Need to update efi file to avoid invalidating boot measurement hashes. (BAD)
Simple, as it has to manage only one file. Need to manage EFI entries of the shell manually from the running OS. (MEH)

Boot measurement
"Slightly" more complex as it has to take into account of an additional file. (MEH)
Simple, as it comes for "free" with UKI, and it's just one file. (GOOD)

Boot assessment
Assessment capabilities remain untouched. Can port the current GRUB logic to systemd-boot or simplify the logics with systemd-boot. (GOOD)
Lose current boot assessment strategies tied with GRUB and kernel panics. Can assess boot during early init stages to check if booted or not and react accordingly. (MEH)

Maintainability (code base, etc.)
Changes required on lifecycle management and additional steps for boot measurement. (BAD, also for security)
Very simple, as it is one file. Everything related to creating, managing, and debugging becomes extremely simple. (GOOD)

We decided that we are going to try with Native EFI, also because, whatever we build around it, it's easy to extend to a boot manager later if we go back at this.


Validation: Set status on IngressRoute if TLS secret invalid

If an IngressRoute specifies a secret used for TLS certificates and that secret does not exist, the status field on the IR should be updated with an appropriate message.
Current implementation ignores the secret if issue: https://github.com/heptio/contour/blob/master/internal/dag/dag.go#L375
// ref https://github.com/heptio/contour/issues/458

I think we should push to 0.7.0 as non-essential to cleanup IngressRoute work.  I believe most IngressControllers don't take an action or update status if there's an issue with Secrets, so we're not any worse than the standard.

I'm seeing strange behaviour, I'm using ingress not ingressRoute, using cert-manager to get TLS secrets.
I have about 20 ingress manifests, and I had 4 TLS secrets that failed (acme challenge provider misconfiguration), contour seems to not provide any config to envoy. (cert list was empty, 8080/8443 listeners were not opened, none of the ingress worked).
$ k version --short
Client Version: v1.11.6
Server Version: v1.11.6

gcr.io/heptio-images/contour:v0.8.1
docker.io/envoyproxy/envoy-alpine:v1.7.0

contour logs are not showing anything...
time="2019-02-08T12:28:40Z" level=info msg=stream_wait connection=120 context=grpc error_detail=nil resource_names="[ingress_http]" response_nonce= type_url=type.googleapis.com/envoy.api.v2.RouteConfiguration version_info=
time="2019-02-08T12:28:40Z" level=info msg=response connection=120 context=grpc count=1 error_detail=nil resource_names="[ingress_http]" response_nonce= type_url=type.googleapis.com/envoy.api.v2.RouteConfiguration version_info=
time="2019-02-08T12:28:40Z" level=info msg=stream_wait connection=120 context=grpc error_detail=nil resource_names="[ingress_http]" response_nonce= type_url=type.googleapis.com/envoy.api.v2.RouteConfiguration version_info=

is this expected behaviour? is this linked to this tracking issue on handling invalid TLS secrets for ingress resources?


Upload dev builds please

I think we would benefit from having debug engine and templates uploaded somewhere whenever you push to master (or at least daily if you push more frequently on a day). Don't think there is a reason to upload release versions except for official releases though.
Building takes quite a while and is fiddly for some platforms, but there are a lot of us needing to use them...

hmm jenkins should be in position to do this, but should sort of make sure
there is a proper build-rotation system up.. Will ask Ariel to help with
this when he is back from Germany.
On Sun, Feb 22, 2015 at 3:53 PM, Bil Bas<EMAIL_ADDRESS>wrote:

I think we would benefit from having debug engine and templates uploaded
somewhere whenever you push to master (or at least daily if you push more
frequently on a day). Don't think there is a reason to upload release
versions except for official releases though.
Building takes quite a while and is fiddly for some platforms, but there
are a lot of us needing to use them...
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/1421.

A approve of having dev. builds made available now and then (like for Win64 which is the platform I'm on).
I don't think it has to be done every single day though, a good way to do it would to be to have it done whenever a major new piece of functionality or one of the release targets is committed (and with things like 2D lighting making it in recently, we could say it's due).

Things aren't pushed/merged to master every day. I was just saying that doing it more than every day would be crazy...
But yes, having dev builds now and then would be fine (rather than 2 months since the last upload, which was the formal 1.0 release!).

some type of auto generated change log would be great as well.

Automatically built nightlies via an Jenkins including a changelog would be great! The changelog could be generated with the commit messages from github, an Jenkins has access to them.

There are now (somewhat) nightly builds on the official website, so this issue is more or less fixed. We might want to look a bit more into it when implementing proper CI #1553, so reuse CI builds as nightly builds).


Add ExpirementalSpreadProperty

Current coverage is 99.75%

Merging #24 into master will not affect coverage as of 432eed2

@@            master     #24   diff @@
======================================
  Files           79      80     +1
  Stmts          806     811     +5
  Branches       102     102
  Methods          0       0
======================================
+ Hit            804     809     +5
  Partial          0       0
  Missed           2       2

Review entire Coverage Diff as of 432eed2

Powered by Codecov. Updated on successful CI builds.

@dmitriiabramov hey do you know what is the difference between ExpirementalSpreadProperty and SpreadElement? We don't have tests hitting SpreadElement want to add some but can't find the difference


WIP: clean-up the dynamic lighting and shadowing selection code

This is low priority, but it may prove itself to be useful as a basis for various other tasks.
First I deeply rewrote the dynamic lighting and shadowing selection code, to make it more readable and less confusing.
The r_dynamicLight cvar to enable dynamic lighting or not is now a boolean and is totally independant of the rendering technique. The related r_staticLight behaves the same (though, I don't really know what it is about).
The dynamic light renderer is now selected with a dedicated cvar named r_dynamicLightRenderer, The value of 0 is the legacy forward one, 1 is the current tiled one.
As an example of how this clean-up may be useful, a 2 line commit makes the forward renderer working again to render dynamic lights.
I hope this work may prove to be as useful to help making dynamic shadows working again too.
Now that the forward dynamic light renderer is working again, and once dynamic shadowing will be working again, at least on the forward renderer, we may start NUKING things that don't break those features when we remove them.
I noticed that the dynamic lights are too bright when using the legacy forward renderer, which is a bug we had with the tiled renderer.
Even if the forward renderer is slower, it may still be useful. In the future I would like to introduce the ability to make the difference with gameplay-focused dynamic lights (like a lamp a player can carry with him) and purely optional and cosmetic ones.
Since the tiled renderer doesn't run on the complete hardware range supported by the engine, if we had a way to only render the only one or two dynamic lights a game may require for its gameplay (like a flashlight), having an alternate renderer supported by lower end hardware may prove useful, and its lower performance may not be a problem if unlike the tiled renderer it is not meant to render thousands of lights but only a couple of gameplay-related ones.

As an aside, forward and tiled are 2 orthogonal things. Forward refers to a renderer where, after an optional depth pre-pass, it draws all the meshes with lighting etc. A deferred renderer outputs a bunch of screen-sized buffers first (diffuse, normal etc), then in another pass it renders a full-screen quad and combines all the buffers output earlier with lighting etc.
"Normal" (non-tiled, non-clustered) just iterates over every light for every fragment and adds up thei rcontributions. Tiled divides the view into tiles and assigns lights to them, then in the fragment shader only computes the lights that are in the same tile as the fragment. Clustered divides the view into clusters (i. e. along all 3 axes) and assigns lights to them, then computes only those lights that are in the same cluster as the fragment.
Both forward and deferred renderers can be "normal", tiled or clustered.

I noticed that the dynamic lights are too bright when using the legacy forward renderer, which is a bug we had with the tiled renderer.

This might be a problem with falloff since normally the lights attenuation is inversely proportional to the distance to light source squared, but gamma correction on the monitor effectively makes it squared instead. There might be an error somewhere there.

This is low priority, but I appreciate the fact it makes possible to better evaluate our lighting code and some of our fallbacks, and the shadowing code.
The code is also better at allocating or not allocating the related stuff used by the various features (so there can be some memory improvements).

Were you able to test the "static light" thing? Here it can be seen that only a handful of maps us the light entity. Curiously, old versions of Forlorn and Yocto used it, but it was removed years ago.

Were you able to test the "static light" thing? Here it can be seen that only a handful of maps us the light entity. Curiously, old versions of Forlorn and Yocto used it, but it was removed years ago.

I only tested the cvars, no specific assets. But anyway fixing features is not the purpose of this PR, all I want is to have the selection of them being meaningful, and the code more explicit.

LGTM

I removed the extra commit that was claiming to fix dynamic lights with legacy renderer, it was likely wrong.
I discovered that enabling the legacy dynamic light renderer can't get it working with vid_restart, one has to load the next map. I don't know what causes that but it was already that way before this PR and this PR doesn't change anything on that (since it's currently only a rewrite to make the code easier to read).
So, both legacy and tiled dynamic renderers are already working, we just cannot switch from tiled to legacy with vid_restart yet because of a bug that is outside the topic of this PR.


Dependency check Gradle plugin

Description
We would like to create a dependency-check Gradle plugin for maintaining the dependencies of this project.
Like suggested in the issue https://github.com/kotools/types/issues/217#issuecomment-1725746437, this plugin should provide the following tasks:

dependencyCheck checking that no new dependency was introduced by comparing the result of the dependencies task with a ${project.name}.dependencies file, where ${project.name} is the name of the Gradle project executing this task
dependencyDump writing the result of the dependencies task in a ${project.name}.dependencies file, where ${project.name} is the name of the Gradle project executing this task.

The dependencyCheck task should be included in the integration workflow.

Dependencies
This issue is blocked by the following ones:

[x] #217

Checklist

[ ] Implement.
[ ] Test.
[ ] Refactor.
[ ] Check the generated documentation.
[ ] Check and update the public API binaries.
[ ] Update the Unreleased section in changelog.

This idea is unrelated to Kotools Types.


AttributeError no _plugin with custom Reader

Describe the Bug
When trying to print/display a BioImage when passing a custom reader, there is no _plugin attribute set:
AttributeError: 'BioImage' object has no attribute '_plugin'
This is because self._plugin is not set in the else clause here: https://github.com/bioio-devs/bioio/blob/main/bioio/bio_image.py#L267-L279
Happy to fix it if you want!
Expected Behavior
Should be able to print in both cases!
Reproduction
from bioio import BioImage
import bioio_tifffile

img = BioImage("my_file.tiff")
print(img) # All good

img = BioImage("my_file.tiff", reader=bioio_tifffile.Reader)
print(img) # AttributeError

Environment

OS Version: macOS 14.6
bioio Version: 1.0.3

I think we would happily submit a pull request for this one! Thanks for finding the issue!

Apologies for the delay, but looks like this was fixed in #67, so closing!


Documentation on updating your SDK ahead of a VS update

Help us make content visible
SDK, VS, runtime, global.json
Describe the new article
During internal discussions, some questions came up around SDK/runtime releases and customers who either cannot and haven't updated their Visual Studio versions. In particular, we weren't sure if we could release a VS release on the same day as a .net security release and wanted a playbook we could point to if we had to do this release.
Is there a good location for this note/playbook?
The recommendation for the customer would be to install the newer stand-alone SDK from the same feature band as their current VS. Then the customer would have to ensure their global.json doesn't pin them to an exact version.
Alternatively, customers could use RuntimeFrameworkVersion to override the implicit version but this requires updating the project (and would need to be removed later) and doesn't support multi-targeting so it's not a recommended solution.

CC @rbhanda

I'd suggest the new article live in this part of the TOC: https://learn.microsoft.com/en-us/dotnet/core/install/remove-runtime-sdk-versions.


BUG: M2XClient.SearchDevices() does not search correctly under specific conditions

NOTE: THIS APPEARS TO BE AN API BUG, not A LIBRARY BUG
When executing M2XClient.SearchDevices() and using the name parameter, the API appears to split the string supplied on spaces and test for each part of that array rather than search for the entire string provided.

@azarc3 can you provide an example of this behaviour? I think this is actually part of how the API was designed but I'd like to understand the scenario to give a more informed answer :)


Add a AS/IJ plugin doc

@martinbonnin I think you meant to tag @trevorblades 😄

Added a few videos (as animated webp) that I recycled from the announcement blog post :) I'll merge for now, and we can revisit it later if we can add a 'tour' video!


Don't count required keyword args when specifying CountKeywordArgs: fals...

...e for ParameterLists.
Hey! First time contributing here. I thought it was awesome that the parameter list cop had an option for ignoring keyword params. I modified it to also ignore keyword params that don't have default values. My guess is the intention behind this cop was to warn coders of huge lists of positional params (hence the CountKeywordArgs option). So this feels like more of a bug fix instead of a change in intent.
What do you guys think?

I don't know why this change decreased code coverage. Is it safe to ignore the coverage error?

I agree that it's a bug fix. The code looks good to me. :+1:

:+1: Thanks!


Feature/core #27

EfficientNet

Codecov Report

Merging #100 into develop will increase coverage by 5.39%.
The diff coverage is 97.46%.

@@             Coverage Diff             @@
##           develop     #100      +/-   ##
===========================================
+ Coverage    92.85%   98.25%   +5.39%
===========================================
  Files           22       36      +14
  Lines          672     1144     +472
===========================================
+ Hits           624     1124     +500
+ Misses          48       20      -28

Impacted Files
Coverage Δ

glasses/nn/blocks/__init__.py
100.00% <ø> (ø)

glasses/utils/ModuleTransfer.py
100.00% <ø> (ø)

setup.py
0.00% <0.00%> (ø)

test/test_ResNet.py
100.00% <ø> (ø)

...lasses/nn/models/classification/resnet/__init__.py
94.44% <80.00%> (-0.25%)
:arrow_down:

test/test_EfficienetNet.py
92.59% <92.59%> (ø)

.../nn/models/classification/efficientnet/__init__.py
97.84% <97.84%> (ø)

...ses/nn/models/classification/mobilenet/__init__.py
98.82% <98.82%> (ø)

glasses/nn/blocks/residuals.py
100.00% <100.00%> (ø)

glasses/nn/models/__init__.py
100.00% <100.00%> (ø)

... and 39 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 62109e7...f45b87f. Read the comment docs.


Custom server/index.js doesnt work in run watch

Summary
Custom server/index.js doesnt work in run watch
Steps To Reproduce:
Add a app.get("/test", (req,res) => { res.send("hello"); }); to src/server/index.js
Expected result
surfing to localhost:3001/test should yield "hello"
Actual result
This works once I pushed to prod (in my case heroku) but it doesnt work in local context using npm run watch
Additional information

watch mode runs only webpack-dev-server for LWC development. If you want to spawn up Express besides it during development you’ve to start it separately, based on your needs, with a script.

Just for reference for those you read this issue later, this is a simple example express-dev.js file that I use in these kind of projects:
const path = require('path');
const express = require('express');

const customServer = require(path.resolve('./lib/server/')).default;

const app = express();

customServer(app);

app.listen(3002, () => {
    // eslint-disable-next-line no-console
    console.log('Yay, local server started');
});

And starting (with a TypeScript based Express project) like this:
"watch": "lwc-services watch & yarn watch:server",
"watch:server": "tsc -b ./src/server && node scripts/express-dev.js"

@muenzpraeger
What is the purpose of src/server/index.js in this scenario then?
Thanks,
Jordan

That's the entrypoint file for your custom Express (backend) logic. I'll add an example to the scaffolding project, good call!
export default function(app) {
    // Custom route
    app.get('/api/1/something', (req, res) => {
        // do stuff
    });

}


Request for test mode flag added to scenario page to skip selecting investigators and so on

I don't know if this enhancement has been asked for and sorry for being a nuisance.
Is it possible to have a checkbox added to the Scenario page for MoM called Test Mode?
If it's ticked, the page describing the scenario is skipped and the first two investigators are automatically selected.
This would make testing quicker as three continue buttons would no longer have to be clicked before the first event is triggered.

https://github.com/NPBruce/valkyrie/issues/290

Thanks for the link and informing me something similar is already planned.
I think my enhancement might be easier to code than a button as it just consists of adding a tickbox (checkbox) to the UI page and then checking if the tickbox is ticked on the initial three pages.

https://github.com/NPBruce/valkyrie/issues/488

This would be done in the editor as per #290 as I don't want to clutter things for players.  Perhaps it will just pick random investigators unless you have specified as per #488.  Closing as duplicate.


Update podspecs after connectrpc organization move

Nice catch, didn't realize this stuff was in these files too.


Slow speed when inferencing

How to be faster by modify some params?

You can strip away the poisson blending operation, which is time consuming.

OK. Thanks! I'll have a try.

By the way, I think FGT has a better effect than ProPainter in some videos I tested. For example: A frame in which a person fill very large area, exactly in the starting.


I don't need null, I need undefined!

This is probably better suited as a feature request: I would like to be able to alternate between null and undefined, are there plans to extend this package to support undefined as well? Thanks in advance!

I am not quite sure what to think of that, I am afraid. On one hand nulll is an opinionated framework for a reason and we should not dismiss the intrinsic value of this. On the other hand though, an exaggerated customisability has always been a fantastic idea and I do believe nulll could do with a vast array of configuration options via dependency injection.
It's a tough call.

@justusromijn  Until they respond to your completely reasonable request, the obvious answer is to split the library and publish a new one.

Thanks, I guess I will need to include them both in my project for the time being. What I really need is one huge package that fixes all my problems that I have now, ever will, or never will have. I will continue my search for this silver bullet.


Search not working in AWS Lambda

Hello
I am building a small service where we need to get the user details from LDAP. I was able to search the users successfully when I run the code in my desktop however once the code is deployed in AWS lambda, search is not working, from the logs I can see that the ldap connection is good and there are no errors. its just that search is not returning the user info. Please advise.
import { Client, SearchOptions, createClient } from 'ldapjs';

import { Injectable } from '@nestjs/common';
import { Config} from<EMAIL_ADDRESS>import { LogService} from<EMAIL_ADDRESS>import { SecretManagerService} from<EMAIL_ADDRESS>
@Injectable()
export class DSLdap {
	private client: Client = null;

	constructor(private readonly logService: LogService, private readonly config: Config, private readonly secretService: SecretManagerService) {}

	async config(): Promise<void> {
		let username: string;
		let password: string;

		const secretString: string = await this.secretService.getSecretValue(this.dsConfig.ldap.secretKeyName);
		if (secretString) {
			this.logService.info('LDAP secret retrieved successfully.');
			({ username, password } = JSON.parse(secretString));
		}
		this.logService.info('Connecting to LDAP server...');
		this.client = createClient({
			url: this.config.ldap.url,
			bindCredentials: '',
			tlsOptions: {
				rejectUnauthorized: false
			},
			log: this.logService
		});
		this.logService.info('Connected to LDAP server...');

		await new Promise((resolve, reject) => {
			this.logService.info('Binding to LDAP server...');
			this.client.bind(username, password, (err) => {
				if (err) {
					reject(err);
				}
				this.logService.info('Bound to LDAP server...');
				return resolve(true);
			});

			this.client.on('error', (error) => {
				this.logService.error(error);
				return reject(error);
			});
		});
	}

	async searchByEmail(emailAddress: string): Promise<any> {
		const opts: SearchOptions = {
			filter: `(userPrincipalName=${emailAddress})`,
			scope: 'sub'
		};
		const entries: any = [];
		const referrals: any = [];
		return new Promise((resolve, reject) => {
			this.client.on('error', (error) => {
				this.logService.error(error);
				return reject(error);
			});

			this.client.search(this.config.ldap.baseDn, opts, (err, resp) => {
				if (err) {
					reject(err);
				}
				resp.on('searchEntry', (entry) => {
					this.logService.info('Entry ' + entry);
					entries.push(entry.object);
				});
				resp.on('searchReference', (referral) => {
					this.logService.info('Referral ' + referral);
					referrals.push(referral.uris);
				});

				resp.on('error', (error) => {
					this.logService.error(error);
					return reject(error);
				});

				resp.on('end', (result) => {
					this.logService.info('Result ' + result);
					return resolve({
						entries: entries,
						referrals: referrals
					});
				});
			});
		});
	}
}

sorry wrong button...

@UziTech
Thank you for your input. I did not realize that client itself is a subscriber. I added the error event and also updated the code above but still I do not see anything in the log. The same code is working fine in my desktop, however my desktop is in the same network/domain as of our ldap server and I am not sure if it has anything to do with it. I can see that the service was able to connect and perform the search however no info is returned. This is the output I got from end event.
Result {"messageID":3,"protocolOp":"LDAPResult","status":0,"matchedDN":"","errorMessage":"","referrals":[],"controls":[]}
Logs from Lambda startup.

Endpoint log:

Thanks for getting back to me. We are using Typescript for our server side and the code given above is the one we use.

This project does not provide support for transpiled code. I also am not willing to try and interpret TypeScript. If you are unable or unwilling to provide a minimal reproduction of the error in JavaScript then I cannot help you.

It seems like everything is working if end is called. The only thing I can think is the server isn't returning any results for the query for some reason. Unfortunately we are not able to help with that.

I did notice one but in your code:
  this.client.search(this.config.ldap.baseDn, opts, (err, resp) => {
    if (err) {
-     reject(err);
+.    return reject(err);
    }

@UziTech  @jsumners
Thanks again. I will try to convert this code to JavaScript later and can share it with you. I will also check with our LDAP admin if they see anything on their side.

@saachinsiva
Were you able to solve this issue?

@DanielSamsonraj
Not yet but I believe the LDAP server is throwing certificate error but has been ignored silently by this library. I may need to add the certificate to the truststore to see if it works. I have been working on other high priority item not but will keep you posted if I make any progress.

@DanielSamsonraj
I tried adding the certificate to the nodejs env and it works perfectly fine in local but still no luck in lambda. No error nothing...just get empty response.

Hello Everyone,
I was able to make it work. The problem is with webpack. When we bundle the application, webpack config uses managle and also rename the functions and classes name to smaller character in order to save space which affects the event emitter. In my case the function name SearchEntry is replaced with character 'y' due to mangle and this causes issue when it reaches this line of code " msg.constructor.name" in manageCallback shown below where it expects the constructor name as SearchEntry which will then be changed to searchEntry and send that as an event.  However in my case the constructor name is coming as 'y' and the event is emitted as 'y' which is why it is not getting any results. In order to keep the function name and class name I used below config for my webconfig.
From client.js
 function messageCallback (msg) {
    if (timer) { clearTimeout(timer) }

    log.trace({ msg: msg ? msg.json : null }, 'response received')

    if (expect === 'abandon') { return sendResult('end', null) }

    if (msg instanceof SearchEntry || msg instanceof SearchReference) {
      let event = msg.constructor.name
      event = event[0].toLowerCase() + event.slice(1)
      return sendResult(event, msg)
    } else {
      tracker.remove(message.messageID)
      // Potentially mark client as idle
      self._updateIdle()

      if (msg instanceof LDAPResult) {
        if (expect.indexOf(msg.status) === -1) {
          return sendResult('error', errors.getError(msg))
        }
        return sendResult('end', msg)
      } else if (msg instanceof Error) {
        return sendResult('error', msg)
      } else {
        return sendResult('error', new errors.ProtocolError(msg.type))
      }
    }
  }

Webpack fonfig:
minimizer: [
new TerserPlugin({
terserOptions: {
mangle: true,
keep_classnames: true,
keep_fnames: true,
sourceMap: true
},
parallel: true
})
]

Closing this as it sounds like you have found your issue and solution.
Reminder: we do not support transpiled versions of the library.

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Update FrontDoor FAQ

Added cipher list for AFD when configured with TLS 1.2

@amitsriva : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@sharad4u @amitsriva Please resolve the merge conflict so that the PR can be merged. Thanks!

The issue that's addressed in the PR appears to be addressed in master. We'll close the PR. If there are outstanding issues or questions, the PR can be reopened.


Min and max properties of ion-datetime are not working properly

Bug Report
Ionic version:
[x] 4.0.0-rc.1
Current behavior:
The selected date on the datetime picker does not update while choosing a date, this has a few issues with the min and max properties:
min property
If I set the the min property to today's date, the datetime picker does not show the past days from the current month as expected. But when scrolling to the next month, the previous days are still not available in the picker. To select a day that is lower than the current day, you need to first select the next month,  tap "Done" and then open the picker again and select the desired day.
E.g.:
<ion-datetime
    displayFormat="DD/MMM"
    min="2019-01-11">
</ion-datetime>

If I want to select 2nd of February, I need first to select February, then re-open the picker and select the day.
max property
A similar issue happens with the max property. If you set the max property to any month in the future, you can first scroll to a day that is after the max day, and then scroll to the month from the max property. This will allow you to select a date that is after the max property. When using (ionChange) to view these values, you can check that the emitted value is actually after the max property.
Expected behavior:
When using the datetime picker, I expect to be able to select any value between the min and max properties, without having to close and open the picker again to update the available options.
Steps to reproduce:
For min bug:
Use the code below and scroll down to Feb. Days between 1 and 11 are to available unless you click done and open again.
<ion-datetime
    displayFormat="DD/MMM"
    min="2019-01-11">
</ion-datetime>

For max bug:
Use the code below and scroll down to day 30. Scroll down to May and you will be able to select a date after 2019-05-11.
<ion-datetime
    displayFormat="DD/MMM"
    max="2019-05-11">
</ion-datetime>

Related code:
Please, update the plunker links to provide a Ionic 4 same app 👍
Other information:

Ionic info:
Ionic:

   ionic (Ionic CLI)             : 4.5.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.0-rc.1
   @angular-devkit/build-angular : 0.10.7
   @angular-devkit/schematics    : 7.1.4
   @angular/cli                  : 7.0.7
   @ionic/angular-toolkit        : not installed

Cordova:

   cordova (Cordova CLI) : 7.1.0
   Cordova Platforms     : android 6.4.0
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 2.3.1, (and 5 other plugins)

System:

   Android SDK Tools : 26.1.1 (/Users/~username~/Library/Android/sdk/)
   ios-deploy        : 1.9.2
   NodeJS            : v8.12.0 (/usr/local/bin/node)
   npm               : 6.4.1
   OS                : macOS Mojave
   Xcode             : Xcode 9.2 Build version 9C40b

Disclaimer: English is not my native language so the issue description might be a bit confusing. Please let me know if you need a clearer example in order to reproduce this issue.

We are having the exact same issue here. A quick fix for this would be awesome. (related to this Ionic 3 bug - 12319)

Hi there,
Thanks for opening an issue with us. I am going to close this for now, but we will be re-evaluating the ion-datetime functionality and bugs soon during which we will take a closer look at this issue.
Thanks for using Ionic!


brcmfmac43430-sdio.txt error 404

Hi, thanks for this library.
I can't create an image due to this resource being not found:
https://github.com/RPi-Distro/firmware-nonfree/raw/master/brcm80211/brcm/brcmfmac43430-sdio.txt

I don't know what it does so I don't know how to fix this. Can you explain what is this for and why is it hardcoded in the script?

If this doesn't have to be up to date, perhaps we can use the URL of the specific commit instead of pointing to master

Fixed by changing the URL to
https://github.com/RPi-Distro/firmware-nonfree/raw/8d4f3825be09e556c01543119ed50515bd7dd661/brcm/brcmfmac43430-sdio.bin in rootfs.mak

shoot thanks I'll update this

thanks for the report, fixed


tonic-reflection: Refactor descriptor parser to be separate

As part of reintroducing the v1alpha reflection support, this extracts the parser into a separate module.

Motivation
If we were to naively reintroduce the v1alpha support, there would be a lot of code duplication.

Solution
The parser is extracted into a separate module that can be shared between both protocols.

This should be superseded by #1888. @ttkjesper please close if you find #1888 satisfactory.


Remove "private" item names from XSD/IntelliSense

Fixes #2125 by removing the section of the XSD that was commented as
"only to avoid schema validation warnings". That's obsolete because VS
no longer checks schema for MSBuild. And we don't need to expose these
names in autocomplete.

@dotnet-bot test this please
Retargeted to master.

@dotnet-bot test Windows_NT Build for CoreCLR please

@dotnet-bot test Windows_NT Build for CoreCLR please
@dotnet-bot test OSX Build for CoreCLR please


Appbar cannot be imported directly as it shows error in theme

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example:
Steps:

Import AppBar from @mui/material
use in code

Current behavior 😯
When imported, the Appbar shows a console error

Expected behavior 🤔
Appbar should be displayed
Context 🔦
Trying to make a Navigation bar, but couldn't get Appbar worked
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://mui.com/r/issue-template), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


solving system of Funs

Is something like this possible?

d = Interval(0.,50.)
D = diff(d)
t = Fun(identity,d)

F = D^2 +.5D + 1
BC = [ldirichlet(d), lneumann(d)]
BC0 = [1.0, 0.0]
x = [BC, F]\[BC0, 0.0] #evolution of undriven equation x'' + .5x' + x = 0, works beautifully

A= [  ldirichlet(d)   0;
        lneumann(d)     0;
        rdirichlet(d)   0;
        F               -1; ]

b = [1.0; 0.0; 2.0; 0.0]

xu = A\b

xu should be two Funs corresponding to x'' + .5x' + x = u with the desired boundary conditions.
Does that seem reasonable?

I don't think your original equation is well-posed: given any solution pair (x,u) and function p that satisfies the boundary conditions (e.g., p(x)=x^2*(x-50), then (x+p,u+p''+.5p'+p) is also a solution to the original equation.

yes, you are correct, it's not well-posed. It's still a linear problem and you mentioned that approxfun would regularize by finding the smoothest (probably the smallest L2 norm on the coefficients?) function.
I hope to still return to this one day!

That's only when the operator is banded.  If we interlace this operator, it would be "skew-banded" in the sense that it's banded around the entries that grow 2x for columns and 1x for rows
Sent from my iPad

On 22 Apr 2015, at 6:58 am, Gustavo Goretkin<EMAIL_ADDRESS>wrote:
yes, you are correct, it's not well-posed. It's still a linear problem and you mentioned that approxfun would regularize by finding the smoothest (probably the smallest L2 norm on the coefficients?) function.
I hope to still return to this one day!
—
Reply to this email directly or view it on GitHub.

Ah, I can see why the interlaced operator is skew-banded. But I don't understand what that means for the type of the solution it returns. Shouldn't it be possible to produce a minimum-norm solution either way?
(In reality, for this kind of control problem, you'd have some additive cost on u to make the problem well-posed, but there's probably an additive cost that corresponds to the minimum-norm Chebyshev coefficients.)

It's because there is only currently adaptiveqr for "BandedOperator".  Adding a SkewBandedOperator may be worthwhile.
Sent from my iPad

On 22 Apr 2015, at 7:14 am, Gustavo Goretkin<EMAIL_ADDRESS>wrote:
Ah, I can see why the interlaced operator is skew-banded. But I don't understand what that means for the type of the solution it returns. Shouldn't it be possible to produce a minimum-norm solution either way?
(In reality, for this kind of control problem, you'd have some additive cost on u to make the problem well-posed, but there's probably an additive cost that corresponds to the minimum-norm Chebyshev coefficients.)
—
Reply to this email directly or view it on GitHub.

This is now implemented in development branch!  (Ignore what I said about SkewBandedOperator, it recognizes that ConstantSpace should just append columns).
Though you need to put the constant terms in the first column:
d = Interval(0.,50.)
D = Derivative(d)
t = Fun(identity,d)
F = D^2 +.5D + 1
BC = [ldirichlet(d), lneumann(d)]
BC0 = [1.0, 0.0]
x = [BC, F][BC0, 0.0] #evolution of undriven equation x'' + .5x' + x = 0, works beautifully
A= [0  ldirichlet(d);
0    lneumann(d);
0    rdirichlet(d);
-1    F; ]
u,x=A[1.,0.,2.,0.]
norm(F*x-u)  # is ~zero

Also ignore the comment about it not being well-posedness: it is when u is assumed to be constant.


x/vuln: Unclear how to request standard-library vulnerabilities be added to DB

What did you do?
The release announcement for Go 1.19.1 mentions two vulnerabilities, one in net/http and the other in net/url. They're obviously public, but as far as I can tell, those vulnerabilities are not present in golang/vulndb. They're not specifically excluded, either.
$ git remote -v
origin  https://github.com/golang/vulndb.git (fetch)
origin  https://github.com/golang/vulndb.git (push)
$ git rev-parse HEAD
3859e52b1385c13b76efe662bcf02c80beda4321
$ git grep -i 'CVE-2022-27664|CVE-2022-32190'
$

They are vulnerabilities, though, aren't they? Or is there some nuance to the definition that I've missed?
The form for submitting additional vulnerabilities says it's for public vulnerabilities that are "not maintained by the Go Team." What's the process for adding vulnerabilities to the database that are maintained by the Go Team?
What did you expect to see?
I expected to see the known vulnerabilities accounted for in the database already, or else I expected to see some instructions about how to get Go-Team-managed vulnerabilities into vulndb.
What did you see instead?
I only see instructions about other vulnerabilities
Among the open issues for vulndb, I see ones for vulnerabilities in the standard library (including one for CVE-2022-27664). Does that mean someone misused the submission form, or what?

cc @golang/vulndb

Hi Rob, thanks for the report! These two vulnerabilities should definitely be in the database. We will add them today.
For future reference, there is now a lightweight way to tell us about a missing CVE or GHSA: https://go.dev/s/vulndb-missing-cve

That new form looks just right. Thanks for adding it!


Redirect chain initiated from outside program or webmail but not in top 5k domains

Alert should probably exclude URLs where redirect chain was initiated by typing directly into omnibox.


feat(Menu): Add multilevel menu (CSS)

Changes

Added new iui-multilevel mxin that adds styling for 'headers' on menus.
Added styling for iui-multilevel-selected - for the name of a selection on the end of a menu item.
Added styling for iui-flat - to give a 'flat' look to the menu for placement in a side nav, info panel, etc.

Testing

Added examples to html test page for Menu. Examples include different types of usages:

disabled
long list (scroll)
icons
subtext
long text (overflow)

both regular text and subtext
and with selected item on the menu item

Have not tested for:

skeletons - is this necessary?
'fancy' example - will do later after review
in context - will do later after review

Made test images for multilevel menu.
Docs
Added changeset.
Will update website later after review

I can't figure out why the skeleton and demo header test images are changing. I undid all of my changes and they still fail.

These things can affect the CSS and HTML, so I would suggest handling the React now, and testing in playgrounds. It can be in the same PR or a separate PR that can be opened in parallel and reviewed together with this one. Until then, this PR could be moved back to draft. If you have any specific questions about the HTML/CSS, you could leave comments and @ team members or set up a meeting.

This is a good suggestion; thanks, I'll do that. And thanks for the list of things to think about.


only wrap tileBounds if noWrap is false. #4646

#4646 Thanks for discovering the problem. I really needed this to work. This is what I did.

Your fix looks sane. However:

There is a spurious call to unproject if noWrap is set to false
Tile URLs are critical enough to have a unit test. There should be a test to check that the scenario described in #4646 results in just one tile loaded.

Even if this is not a complete bug fix, it's a very good starting point for one!!

Superseded by #4908


has_contract() : Determine whether a account deploied contract

Change Description
bool has_contract() : Determine whether to deploy the contract

Consensus Changes

API Changes

Documentation Additions

In wasm_interface, should (has_contract, int(int64_t) ) be (has_contract, int(int8_t) ) because it is returning a boolean?

I had same feeling when I saw it, but all previous code for booleans is using int64_t, not sure if there is a real reason.

The comment is really inaccurate。
bool has_contract(account_name name)  //  account_name is int64_t

In wasm_interface, should (has_contract, int(int64_t) ) be (has_contract, int(int8_t) ) because it is returning a boolean?

I had same feeling when I saw it, but all previous code for booleans is using int64_t, not sure if there is a real reason.

The comment is really inaccurate。
bool has_contract(account_name name) // account_name is int64_t

In wasm_interface, should (has_contract, int(int64_t) ) be (has_contract, int(int8_t) ) because it is returning a boolean?

I had same feeling when I saw it, but all previous code for booleans is using int64_t, not sure if there is a real reason.

The comment is really incorrect, sorry.
bool has_contract(account_name name) // account_name is int64_t


defaultWorkerFactory.ts: deprecated require.toUrl() use

The context is: https://github.com/microsoft/vscode/pull/107437 and the deprecation of require.toUrl
I noticed 2 uses of require.toUrl that ideally we can replace with FileAccess.asFileUri and FileAccess.asBrowserUri:
https://github.com/microsoft/vscode/blob/08c92c9161a57cfe6f8e7badabbac6f96516737a/src/vs/base/worker/defaultWorkerFactory.ts#L22-L22
https://github.com/microsoft/vscode/blob/08c92c9161a57cfe6f8e7badabbac6f96516737a/src/vs/base/worker/defaultWorkerFactory.ts#L39-L39
I would have done it but wasn't exactly sure, my guess would be to use the FileAccess.asBrowserUri variant here since we are in browser context.

I'm a bit concerned about changing this, as this piece of code runs in the standalone editor too. So before this piece of code would go directly to the loader (which could be require.js and not our loader) and get back a string.
With the proposed change to use FileAccess.asBrowserUri, this will now go through our URI.parse and I believe URI.parse(str).toString() is not guaranteed to be equal to str.

@alexdima understood and that is also the reason why I wanted to ask you about it since I was not so sure.
When I test from the vscode-file branch it seems to me the worker is coming up properly, at least on macOS:

So it may well be that things continue to work, either with file or vscode-file protocol because we load the worker in a browser context always.
I am fine leaving this use as is and maybe add a comment. I just wanted to bring this up to ensure this continues to work, even when we switch to the new base URL with vscode-file://vscode-app/... structure.

I just added a comment explaining the use.


Problem in multi-GPU evaluation: dataloader returns the first few items at the end of the epoch

Hi there,
I am doing evaluation on a dataset with 5097 samples. I ran the codes on a 2-GPU machine and set the batch_size_per_device to 8.
My code is something like:
dataloader = DataLoader(
    dataset,
    collate_fn=data_collator,
    batch_size=args.per_device_eval_batch_size,
)
model, dataloader = accelerator.prepare(model, dataloader)
...
output_preds = []
for step, batch in enumerate(dataloader):
        with torch.no_grad():
            generated_tokens = accelerator.unwrap_model(model).generate(
                batch["input_ids"],
                attention_mask=batch["attention_mask"],
                **gen_kwargs,
            )
generated_tokens = accelerator.pad_across_processes(
    generated_tokens, dim=1, pad_index=tokenizer.pad_token_id
)
generated_tokens = accelerator.gather(generated_tokens).cpu().numpy()
output_preds += generated_tokens

However, I found that the length of output_preds is always 5104.
I printed the indices of the samples in each batch and found that at the end of the epoch, the indices on 2 GPUs are:
tensor([5088, 5089, 5090, 5091, 5092, 5093, 5094, 5095], device='cuda:0')
tensor([5096,    0,    1,    2,    3,    4,    5,    6], device='cuda:1')

It turns out that the dataloader on the second GPU returned the first few items in the dataset. I wonder how can I fix it?

There is no fixing it. This is how Accelerate ensures you have batches on the same size on all GPUs, to avoid errors.
You need to truncate your last batch of samples for the evaluation loop.

Thanks for letting me know!


Image processing rework

I have returned, with quite a lot of changes:

Introduce a general purpose img_info structure, that contains a image's header (making use of a union).
Split the add_<format>_data functions up:

a parse_<format>_header which writes relevant info into a struct img_info
the pdf_add_<format>_data which relies on the passed img_info and doesn't parse the img header itself.

Introduce parse_image_header (in the .h file), to be able to obtain image information before adding the image, this can be used to address #95.
renamed header_to_image to determine_image_format

Disclaimer: I started writing this a few months ago and only picked it up now.
I probably have introduced a bug here or there, especially the memory management in pdf_add_png_data should be closer inspected.
I ran tests.sh and the two generated PDFs looked the exact same to me.
Diffing the two pdfs showed that only the /CreationDate in line 10 and the /ID in line 22916 have changed.
There are also a handful of todo comments that I intend to address before merging, but I decided to create the PR already so you can take a look at the changes.

Thanks for sending this through. I've been busy the last few days, but I'll try and get into this soon and let you know about the pr.

Sorry, I'd completely forgotten about this PR. Were you still working on this, or waiting for some feedback?

Sorry, I'd completely forgotten about this PR. Were you still working on this, or waiting for some feedback?

No worries, so did I.
I was waiting for feedback, especially around the memory stuff.

I had another quick look. I didn't see any real memory issues, but I would like to make a couple of minor changes. One is using the pdf_ prefix to try and minimise namespace clashes. The other is using an anonymous union to avoid needing format_specific_img_info. I've made the changes, and pushed them to a branch here:  https://github.com/AndreRenaud/PDFGen/tree/image_processing_rework
The code reshuffle had broken BMP/PPM loading, but I've fixed it in that branch, along with Doxygen comments - Doxygen doesn't seem to like anonymous unions though, so I sort of had to force a hack in for that.
I'm mostly relying on Valgrind & fuzz to catch memory errors, and they're not triggering on these builds, so I assume it's ok.

Thank you for the review. If your branch is working as intended I think we can merge yours and close this PR.
Or merge this one, then add your changes afterwards, whichever you prefer.

Ok - I've create it as a new PR - are you able to check out the code from https://github.com/AndreRenaud/PDFGen/pull/105 and confirm that it works ok for you? If so, I'll close this PR and merge 105.

That's fine by me, thank you :)


Suit Storage Visual Change

Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:Alekshhh

tweak: Changed suit storages visually

sorry but it doesn't look attractive...

this isn't a dating app

mockup for flat-topped suit storage. just something to ponder.

Flat Top mockup looks really pretty to me

i love the flat-topped so much more

add it nerd

flad toppe good can you do that

done


Space Travelers': Display joined missions

What is added

Add joined missioned to the profile section.
Give style to the joined missions.

Hi @mahdinoori2000 😄 ,
You did a great job displaying missions in the profile.
Congrats 🎉
It's time to merge it now 💯


Error: Custom { kind: Other, error: StringError("no tarball found inside binary") }

Hi, I'm getting an error when I try to execute the .exe in an Azure DevOps pipeline.
After pulling the repo of my wiki, I do this in Powershell:

When I run tree /F D:\a\1\s in Powershell, I get this:

Now I know that the exe is downloaded to the root of the wiki.
But when I do this:

I get an error:

Strangely enough, the script has succeeded one time:

I was with the same code in the pipeline. I wasn't able to get it working again.

I have fixed this error by downloading the exe from the releases:
$url = "https://github.com/MaxMelcher/AzureDevOps.WikiPDFExport/releases/download/v1.4.0/azuredevops-export-wiki.exe"
and hardcoding these paths:
D:\a\1\s\azuredevops-export-wiki.exe --path "D:\a\1\s" --output D:\a\1\s\export.pdf