fix: Forking fails when template has REST datasource with Bearer Token Authentication

Description
When an application is exported with credentials i.e. template application, it does not contain the bearer token authentication credentials. As a result, when user is trying to import this application, the datasource configuration popup appears. This PR fixes this issue.
TL;DR enable export-import application with bearer token authentication
Fixes #19415
Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual
Jest

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=2f7ac615d128fcb5fdb7d64f9a4419f6ca1b5ebe

@sidhantgoel @AnaghHegde can you guys please take a look at this PR little occupied with other tasks.


feat: '/assign' can be used with go-sdk repo

Please follow this checklist before submitting:

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[ ] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
The assign command can now be used in issues on the go-sdk repo from the referenced PR.
Issue reference
Refs: (PR) dapr/go-sdk#460

@mikeee - Suggest changing this sentence below or making it say something more generic, as in "see the list of repositories for each command?
"This reference covers the Dapr bot capabilities from the dapr and components-contrib repositories only."


Create viewer classes for interactive use in iPython notebooks

I find it exceptionally useful and convenient to view images in interactive sessions directly inside the jupyter notebook. So I've written viewer classes that allow to view 2d or 3d arrays in a similar manner like a radiologist would.
BasicViewer
The base class of all viewers. Can show a single image, optionally with mask overlay or label. Also allows to add a title to the image.

please note, this image does not show pneumonia ;)
ListViewer
As subclass of BasicViewer that allows to view multiple images at once. Useful to compare images or to be implemented in other functions. In trainlib I use it in DataLoader.show_batch, to inspect the batch before it is passed to the model.

DicomExplorer
This viewer class provides more useful information about the distribution of intensity values. Could be useful to check if images were normalized correctly.

This PR also adds extensive tests for all viewers. Ideally it is not merged before #9

@kbressem This looks very useful indeed. Thanks for sharing.
@ibro45 I think this should go into the contrib  folder with all the other tools we have. I'll setup a repo for it tomorrow and make it a submodule for this repo

@kbressem Really nice stuff!
@surajpaib Agree; it might even be its own package (what do you think @kbressem?), but contrib would do for the moment

I think it should be it's own package for sure. We can even skip the submodule and have it as an independent package completely. Any medical imaging/ use case specific tools will go there.

We could start using it here and then iteratively refine. When it proofs to be reliable, I can create a stand alone package. I'll move it to contrib

https://github.com/project-lighter/contrib
Want to PR it there?

https://github.com/project-lighter/contrib Want to PR it there?

Yes, will do


C# RPC Error when using SetTimeout

this doesn't work
Alt.SetTimeout(() =>
        {
            Alt.Log("1231");
            var a = rpcEvent.Answer("1");
            Console.WriteLine(a);
        }, 1000);

this works:
 Alt.Log("1231");
 var a = rpcEvent.Answer("1");
 Console.WriteLine(a);

using Alt.OnScriptRPC on client-side

Tried also using .WillAnswer() then use Alt.EmitRPCAnswer() with the rpcEvent answer id that i get from Alt.OnScriptRPC, but I always get Answer not handled on server.
Nuget: 16.2.2

full discussion here: https://discord.com/channels/371265202378899476/1282413248502829099

debug.zip
here is full reproduction code.


isinstance check order of iter method

iter is currently defined as such:
def iter(itr: AnyIterable[T]) -> AsyncIterator[T]:
    if isinstance(itr, AsyncIterator):
        return itr

    if isinstance(itr, AsyncIterable):
        return itr.__aiter__()

This causes an issue for classes defined as such:
class Foo:
    def __aiter__(self):
        return self.__anext__()

    @async_generator
    async def __anext__(self):
       ...

since it won't use the object returned by __aiter__() as both isinstance checks are true.  I believe these two checks should be reversed with a comment why they are in the order they are.

My primary concern with this is that reversing the order will mean that existing iterators will never be used, because iterators are also iterables, and would then always call itr.__aiter__(). You can't have an AsyncIterator without implementing __aiter__ which then means it's also an AsyncIterable:
from typing import AsyncIterator, AsyncIterable

class Aiter:
    def __aiter__(self):
        return self

class Anext:
    async def __anext__(self):
        raise AsyncStopIteration

class Aboth(Aiter, Anext):
    pass

for c in (Aiter, Anext, Aboth):
    a = c()
    itr = isinstance(a, AsyncIterator)
    ita = isinstance(a, AsyncIterable)
    print(a.__class__, itr, ita)

<class '__main__.Aiter'> False True
<class '__main__.Anext'> False False
<class '__main__.Aboth'> True True

More to the point, I believe that your example of Foo.__anext__ is against the specification of PEP 492 (emphasis mine):

An asynchronous iterator object must implement an __anext__ method (or, if defined with CPython C API, tp_as_async.am_anext slot) returning an awaitable.

An asynchronous generator is not itself an awaitable, and therefore doesn't satisfy the API of __anext__.  For your specific example, you should instead be implementing your class like this:
class Foo:
    def __aiter__(self):
        return self._iterator()

    async def _iterator(self):
        yield ...

In this case, the async generator object returned by calling self._iterator() will itself be an AsyncIterator with its own __anext__ method, while allowing your class Foo to be a valid AsyncIterable.


Fix topic deletion

Fixes #64

A new inspection was created.

Fixed w/ 3.0

The inspection completed: No new issues


Fix some typos in README.md

:)

Sharp eyes! I should have had you edit my book. ;)

I've added you as a collaborator. Please continue to use the PR process to get your changes merged.


better handle noninstalled libs error during model loading

What does this PR do?
I found an issue with loading models from extra-packages libraries (e.g. diffusers). TaskManager select and trying to load model even library is not installed in user environment and error message can be confusing for that case. This PR provides small improvement for this case.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Who can review?
@echarlaix, @IlyasMoutawwakil

File "/home/ea/miniforge3/lib/python3.12/site-packages/optimum/exporters/tasks.py", line 2066, in get_model_from_task
config = DiffusionPipeline.load_config(model_name_or_path, **kwargs)
^^^^^^^^^^^^^^^^^
NameError: name 'DiffusionPipeline' is not defined

Yes I agree for the diffusers path the check is_diffusers_available was necessary, but for the other two, isn't it less verbose and more ambiguous like this ? loaded_library = importlib.import_module(library) will try to import the module and fail with the import error, whereas this suggestion will say that the module does not exist, for example when importing transformers, the import might fail due to incompatible tokenizers library or missing third party library, and not because transformers is missing.

Yes I agree for the diffusers path the check is_diffusers_available was necessary, but for the other two, isn't it less verbose and more ambiguous like this ? loaded_library = importlib.import_module(library) will try to import the module and fail with the import error, whereas this suggestion will say that the module does not exist, for example when importing transformers, the import might fail due to incompatible tokenizers library or missing third party library, and not because transformers is missing.

I catch only ModelNotFound exception for this case, so failure due to any other reason still will fail with original error, I do not see any problems and misleading here

I installed tokenizers==0.19 and ran the following:
ython 3.10.12 (main, Jul 29 2024, 16:56:48) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import importlib
>>> importlib.import_module("transformers")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/user/.local/lib/python3.10/site-packages/transformers/__init__.py", line 26, in <module>
    from . import dependency_versions_check
  File "/home/user/.local/lib/python3.10/site-packages/transformers/dependency_versions_check.py", line 57, in <module>
    require_version_core(deps[pkg])
  File "/home/user/.local/lib/python3.10/site-packages/transformers/utils/versions.py", line 117, in require_version_core
    return require_version(requirement, hint)
  File "/home/user/.local/lib/python3.10/site-packages/transformers/utils/versions.py", line 111, in require_version
    _compare_versions(op, got_ver, want_ver, requirement, pkg, hint)
  File "/home/user/.local/lib/python3.10/site-packages/transformers/utils/versions.py", line 44, in _compare_versions
    raise ImportError(
ImportError: tokenizers>=0.20,<0.21 is required for a normal functioning of this module, but found tokenizers==0.19.0.
Try: `pip install transformers -U` or `pip install -e '.[dev]'` if you're working with git main

I installed tokenizers==0.19 and ran the following:
ython 3.10.12 (main, Jul 29 2024, 16:56:48) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import importlib
>>> importlib.import_module("transformers")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/user/.local/lib/python3.10/site-packages/transformers/__init__.py", line 26, in <module>
    from . import dependency_versions_check
  File "/home/user/.local/lib/python3.10/site-packages/transformers/dependency_versions_check.py", line 57, in <module>
    require_version_core(deps[pkg])
  File "/home/user/.local/lib/python3.10/site-packages/transformers/utils/versions.py", line 117, in require_version_core
    return require_version(requirement, hint)
  File "/home/user/.local/lib/python3.10/site-packages/transformers/utils/versions.py", line 111, in require_version
    _compare_versions(op, got_ver, want_ver, requirement, pkg, hint)
  File "/home/user/.local/lib/python3.10/site-packages/transformers/utils/versions.py", line 44, in _compare_versions
    raise ImportError(
ImportError: tokenizers>=0.20,<0.21 is required for a normal functioning of this module, but found tokenizers==0.19.0.
Try: `pip install transformers -U` or `pip install -e '.[dev]'` if you're working with git main

It gives better way to handle the issue than simply pip install transformers.

@IlyasMoutawwakil  ImportError != ModuleNotFoundError, so you still get your desired error message, ther is no any problem for your provided case

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

I tried to fix styling from here but didn't work.. 😅


Anonymous warnings

Description
The documentation for the package (version 0.8q, dated 2020/01/31) says the following:

This package can produce a number of informational messages to try and inform the user when something might be going wrong due to package conflicts or something else. As an experimental feature, these can be turn off on an individual basis with the package option warnings-off which takes a comma-separated list of warnings to suppress. A warning will give you its name when printed on the console output; e.g.,

* unicode-math warning: "mathtools-colon"
*
* ... <warning message> ...

This warning could be suppressed by loading the package as follows:

\usepackage[warnings-off={mathtools-colon}]{unicode-math}

The warnings-off option is a welcome addition because of a couple of harmless warnings that come by default when one uses other standard packages such as mathtools. However, there is a discrepancy between the documentation and the behavior of the package. Namely, when compiling my document (with LuaLaTeX, using revision 53609 of the package)…
Package unicode-math Warning: Using \overbracket and \underbracket from
(unicode-math)                `mathtools' package.
(unicode-math)
(unicode-math)                 Use \Uoverbracket and \Uunderbracket for
(unicode-math)                original `unicode-math' definition.

…the name of the warning (mathtools-overbracket in this case) is absent, and I have to go dig into the source code to find it, which is annoying. I think the best solution to the problem of identifying the name of the warning would be to simply make the package conform with its documentation.
I haven’t tried with XeLaTeX, or tried to get it down to a MWE at all, but I can do the latter if so desired.

Thank you @LambdaP for making me discover the option, as well as the name of the two warnings I actually get (mathtools ones like you). :)


Fix: change link to changelog (pun intended)

⚠ IMPORTANT ⚠: This is a public repository. Make sure to not disclose:

[x] personal data beyond what is necessary for interacting with this Pull Request;
[x] business confidential information, such as customer names.

The changelog URL changed as a result of this commit. I don't usually buzz people with PRs on fixing links, but in this case I want to make sure I understood the intention going forward correctly.
Thanks for reviewing and confirming!

Yes, we will keep the changelog split up per major and minor release from now on to simplify it and make it a bit neater in general.

Yes, we will keep the changelog split up per major and minor release from now on to simplify it and make it a bit neater in general.

Love it, thanks!


Realistic "Sandbags"

Make the current sandbags more realistic by making it possible to create sandbags from any ground soil, dirt, grass, sand etc.
Reference: https://www.quora.com/Do-sandbags-play-an-important-role-in-modern-warfare-given-that-they-have-been-in-use-since-WW1-Is-there-an-alternative-that-can-replace-sandbags/answer/Daryl-Tew?srid=uNzPp

Ya that was always the plan, and I removed the current ones we had in the mod. They didn't look very good for the style. If we re-implement them we need to not use funky 3D models no matter how much work went into the model. Instead we should take time to make models that fit Minecraft and make sure to have non-model versions. For example we can use the same model for the cobblestone wall to make sandbags.


Add FooterActionButton to MarkdownEditor

Closes https://github.com/github/primer/issues/2174
I added the ability to add custom footer buttons before the "Markdown is Supported" in the MarkdownEditor. I created MarkdownEditor.FooterButton and MarkdownEditor.Footer components to align with the approach taken for MarkdownEditor.Toolbar and MarkdownEditor.ToolbarButton.
Screenshots
Before

After

Merge checklist

[x] Added/updated tests
[x] Added/updated documentation
[x] Tested in Chrome
[x] Tested in Firefox
[x] Tested in Safari
[x] Tested in Edge

Hi @JelloBagel 👋🏻 Thanks for the contribution! Are you still working on this? If you're ready for a review, please bring this out of "draft" mode and a reviewer will be auto-assigned.

@JelloBagel - the deployment issue is very strange. I haven't seen that before. Can you confirm that it looks as you'd expect when you build locally?

@mperrotti I ran build:docs:preview (was this the right command?) and can confirm that my changes are reflected in the build

Before
After


undefined method visit_[:type, "string"] when trying to use a type in a form validation.

module Types
  include Dry::Types.module
  Percentage = Strict::Decimal.constrained(gt: 0, lt: 1)
  Quantity   = Strict::Int.constrained(gt: 0)
end

class Tier < Dry::Types::Struct
  attribute(:quantity,    Types::Quantity)
  attribute(:percentage,  Types::Percentage)
end

form_schema = Dry::Validation.Form do
  key(:quantity).required(Types::Quantity)
  key(:percentage).required(Types::Percentage)
end

gems/dry-types-0.7.1/lib/dry/types/compiler.rb:15:in `visit': undefined method `visit_[:type, "string"]' for #<Dry::Types::Compiler:0x007f7d16496f98 @registry=Dry::Types> (NoMethodError)

Not sure if this is a valid thing to do, but apparently it shouldn't raise an exception according to @solnic

yeah it's a bug in input processor compiler, it should infer decimal and int types from form.* but for some reason it's crashing.

Hi @joevandyk - I just merged a fix for this over in https://github.com/dry-rb/dry-validation/pull/106. I used your example as the test case so I'm very confident it is resolved :). For the time being you'll need to reference master branch in your Gemfile, the fix will be out in the next release.
I'm going to close this, if you're still having issues open it up again. Thanks.

Works like a charm, thanks guys!


Zig Language Server crashes when supplying print arguments

Environment
Zig: 0.12.0-dev.1127+32bc07767
vscode-zig: 0.5.1
OS: macOS Sonoma 14.1, Apple M1 Silicon
Vscode related settings
{
  "zig.path": "",
  "zig.zls.path": "/Users/ek/Library/Application Support/Code/User/globalStorage/ziglang.vscode-zig/zls_install/zls",
  "zig.initialSetupDone": true,
}

https://github.com/ziglang/vscode-zig/assets/75629102/bde0fc27-ef8f-4495-9608-949a8ff344bf

I believe this issue is equivalent to https://github.com/zigtools/zls/issues/1624 which has been fixed.


Config support for ConvertingPopulator

The ConvertingPopulator is currently not supported in the bundle config.
It should be usable via neusta_converter.converter.<name>.properties.<name>.converter: <name>.

I can implement the functionality if you haven't started working on it yet.

I haven't. But perhaps we should first decide whether we want #41.


Maintenance status

Hi,
Thanks for the work so far on this crate. I'm about to use it (mostly the networking parts) for some work related stuff and I'm making some improvements that I would like to upstream. I can see that there is already a bit of a backlog, i.e. some good PRs that haven't been merged and some improvements on master that haven't been released. Before I split up my changes into PR sized chunks I wanted to hear if there are capacity to review/merge stuff?
I'm happy to help maintaining as well.

Hey,
Thanks for the concern. I do recognize the crate is in bit of a maintance trouble. While I can't speak for other collaborators, I personally haven't had much time to work on OSS projects recently and things have fallen behind. It's an unfortunate situation, but not much I can do about.
If you are interested in becoming a maintainer I am very happy to add you as a collaborator. Steamworks-rs (afaik) is still the de facto crate for accessing steam services for Rust games, and should stay alive.

Hey,
Thanks for the concern. I do recognize the crate is in bit of a maintance trouble. While I can't speak for other collaborators, I personally haven't had much time to work on OSS projects recently and things have fallen behind. It's an unfortunate situation, but not much I can do about.
If you are interested in becoming a maintainer I am very happy to add you as a collaborator. Steamworks-rs (afaik) is still the de facto crate for accessing steam services for Rust games, and should stay alive.

No worries, I'll start doing some PRs then. I'd be happy to help maintain, just send me an invite.

Hey @kristoff3r @Noxime,
Sorry for the ping. Is there any plan for a new version release? There is a lot of cool stuff that's been merged.
I want to release renet_steam, a transport layer for renet that uses steamwork-rs, but I do depend on a few unreleased APIs.

@lucaspoffo I think a release sounds like a good idea. I don't know who (if any) apart form @Noxime that has credentials for crates.io, so I can't do it yet unfortunately.

@lucaspoffo I think a release sounds like a good idea. I don't know who (if any) apart form @Noxime that has credentials for crates.io, so I can't do it yet unfortunately.

I am up for a new release, it's been a while. Are there any outstanding PRs or issues we should resolve before a new crates.io release?

@lucaspoffo I think a release sounds like a good idea. I don't know who (if any) apart form @Noxime that has credentials for crates.io, so I can't do it yet unfortunately.

I am up for a new release, it's been a while. Are there any outstanding PRs or issues we should resolve before a new crates.io release?

@Noxime I don't think so, let's just do a release now and then we can do an SDK update and merge my stuff after.


[Web Portal] fix text broken in create/edit user

#2847
fix text broken in create/edit user

Coverage increased (+0.01%) to 53.324% when pulling 7bbe0d161f7a1e0d9948c075149c9d46841ab66f on lchao/fixTextBroken1 into 8f76949022a811a225a593db2ee162fa5ffe6c8c on master.

Coverage increased (+0.01%) to 53.324% when pulling 7bbe0d161f7a1e0d9948c075149c9d46841ab66f on lchao/fixTextBroken1 into 8f76949022a811a225a593db2ee162fa5ffe6c8c on master.


Error in environments with existing integrated vectorization enabled

This error happens when you re-run prepdocs.sh:
Traceback (most recent call last):
  File "/workspaces/azure-search-openai-demo/./scripts/prepdocs.py", line 483, in <module>
    loop.run_until_complete(main(ingestion_strategy, setup_index=not args.remove and not args.removeall))
  File "/usr/local/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/./scripts/prepdocs.py", line 221, in main
    await strategy.setup()
  File "/workspaces/azure-search-openai-demo/scripts/prepdocslib/integratedvectorizerstrategy.py", line 159, in setup
    await ds_client.create_or_update_data_source_connection(data_source_connection)
  File "/workspaces/azure-search-openai-demo/scripts/.venv/lib/python3.11/site-packages/azure/core/tracing/decorator_async.py", line 77, in wrapper_use_tracer
    return await func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/scripts/.venv/lib/python3.11/site-packages/azure/search/documents/indexes/aio/_search_indexer_client.py", line 367, in create_or_update_data_source_connection
    result = await self._client.data_sources.create_or_update(
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/scripts/.venv/lib/python3.11/site-packages/azure/core/tracing/decorator_async.py", line 77, in wrapper_use_tracer
    return await func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/workspaces/azure-search-openai-demo/scripts/.venv/lib/python3.11/site-packages/azure/search/documents/indexes/_generated/aio/operations/_data_sources_operations.py", line 248, in create_or_update
    map_error(status_code=response.status_code, response=response, error_map=error_map)
  File "/workspaces/azure-search-openai-demo/scripts/.venv/lib/python3.11/site-packages/azure/core/exceptions.py", line 164, in map_error
    raise error
azure.core.exceptions.ResourceExistsError: () There was a conflicting update. No change was made to the resource from this request.

This issue is for a: (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
Create an environment with integrated vectorization enabled
Re-run prepdocs.sh

I'm also getting this error. Any tips for a workaround?

I typically just comment out those last few lines of code. I've asked @srbalakr to take a look, however.

Getting the same issue... @pamelafox which lines exactly do you comment out?

would love an update on this one if possible

Sorry folks, I wasn't available in past months . Will take a look and keep posted.

I'd really like to switch my application to using embedded vectorization, but the only workaround for this issue I found is to delete the index, which is not a good option for live application.
@pamelafox - could you elaborate on your workaround please? I cannot figure out what lines to comment out.


Add support for en-passant moves

This is one of the issues raised in #16. We should add support for en-passant moves.
The function(s) for generating pawn moves - chessengine.moves.get_white_pawn_moves and chessengine.moves.get_black_pawn_moves provide an argument allow_en_passant that will also generate en-passant moves, which is currently set to False and never passed as True. I had added this argument as a temporary way to consider the generation of en-passant move, but this argument might now become redundant as we will implement proper support for en-passant moves.
The way I am thinking of approaching this is as follows -
We will need to add an attribute to the Board class which would keep track of which pawn (if any) is allowed to make an en-passant move, and update that attribute on each move, which will probably need to be done in both Board.move and Board.undo_move. That attribute can then be checked inside chessengine.moves.get_white_pawn_moves and chessengine.moves.get_black_pawn_moves and en-passant moves can be generated, if valid.

Hey, its maku.  Can you assign this one to me?
Thank you!!! :)

Admittedly, this will take some time for me to digest.
Just checked out the code and starting playing with it.
One notable concern that I have with implementing en-passant - Its kind of an "in-between" state.
So something that could be en-passant "now" might not be a valid en-passant if the other opponent makes a move that places the King in check. Additional checks will need to be made on both diagonal and rank.

Its kind of an "in-between" state.

I'm not quite sure what you mean. Could you elaborate?

There needs to be a way to pass in the previous move and check if the  pawn that's currently being moved is applicable to en-passant.  The "in-between" state that I'm referring to is that there needs to be a look ahead and check on that pawn performing the en-passant results in King exposed (making an illegal move).

There needs to be a way to pass in the previous move and check if the pawn that's currently being moved is applicable to en-passant.

Yes, for this, we could probably use the Board.moves list that stores all the moves made in the game so far - see docs.
For the second point you mention, that is an issue with not just pawns, but all pieces. Currently, none of the pieces check if a move they are making results in exposing the king. Fixing that efficiently will require some more thinking and (probably) a more global fix, because checking each time if the king would be in check for each move generated by each piece would be very inefficient and slow down the forward search (at least I can't think of a way to check that way efficiently). However, if you have any ideas on how we could perform that check, please let me know :)

I plan on working on this on Sunday - if someone else wants to take it up prior to, feel free.
Otherwise, I'll commit this to my Sunday code-and-coffee :)

Hey, I'm gonna have to abandon this one.
Unless I can set up a debugger, I don't really have a way of implementing this.

No problem, I'll take it up 👍🏻


Prometheus metrics not incrementing

I noticed that on my Caddy instances, the only Souin Prometheus metric that appears to be working is souin_request_counter.
All of the other ones simply remain at 0.

Hello @jpds what's your configuration please?

Basically:
{
      admin [::]:2019

      # Souin
      order cache before rewrite

      cache {
        default_cache_control "public"

        key {
          hide
        }

        nuts {
          path /var/lib/caddy/souin/
        }

        stale 3h
        ttl 14h
      }

      servers {
        metrics
        log_credentials
      }
}

test.example.com {
      cache

      reverse_proxy [2001:db8::80]:8080
}

I did see that Souin had its own api.prometheus option - but it's that one metric would be working without that.


Git clone and then run examples success,but there is 404 when access curl http://localhost:9000/graphql

➜  example git:(master) ✗ pwd
/Users/davidwu/Documents/git_workspace/graphql-spring-boot/example
➜  example git:(master) ✗ curl http://localhost:9000/actuator/health
{"status":"UP"}%                                                                                ➜  example git:(master) ✗ curl http://localhost:9000/graphql
{"timestamp":"2018-08-21T06:39:07.636+0000","status":404,"error":"Not Found","message":"No message available","path":"/graphql"}%

@wufuwei Just ran the example in example without problems. Can open graphiql and can run queries to /graphql as well. Provide some more information what you're trying to do and the error you see if you're still facing this issue.

Since I wasn't able to reproduce this issue and haven't heard back from you I'm going to assume you were able to fix it and will close this issue. If you're still having problems, you can re-open it and provide some more detail.


Undelivered events

Hi , After 24 hrs if the message is still undelivered,  How can we back up the Event/message and retry for the failed subscribers?  Is it possible to override this behaviour?

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

ID: 92671e7c-03e5-591d-7fe2-7c565220242a
Version Independent ID: 99bdee93-1a4d-d10c-8d90-a58a84bb6187
Content
Content Source
Service: event-grid

@ravi-karur Thank you for the valuable feedback. I forwarded your feedback the author.

@ravi-karur - unfortunately, there is no way to do this right now. We will update this article if that changes.

#please-close


Remove BCH prefix

The prefix bitcoincash: is not needed and the address looks cleaner without.

https://github.com/vultisig/vultisig-ios/pull/961


TypeError in the multi-root dmrgci

Hello, I followed the example/mrpt/41-for_state_average.py to do a nevpt2 calculation for an excited state, but, unfortunately, something went wrong:
from pyscf import gto, scf, mcscf, dmrgscf

mol = gto.M(
    atom = [
        ['O', ( 0., 0.    , 0.   )],
        ['H', ( 0., -0.757, 0.587)],
        ['H', ( 0., 0.757 , 0.587)],],
    basis = '6-31g',
    symmetry = 1)

mf = scf.RHF(mol)
mf.kernel()

mc = dmrgscf.DMRGSCF(mf,4,4)
mc.state_average_([0.5,0.5])
mc.run()

mo = mc.mo_coeff

mc = mcscf.CASCI(mf,4,4)
mc.fcisolver = dmrgscf.DMRGCI(mol)
mc.fcisolver.nroots = 2
mc.run(mo)

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/chenh/anaconda3/lib/python3.6/site-packages/pyscf/lib/misc.py", line 472, in run
    self.kernel(*args)
  File "/home/chenh/anaconda3/lib/python3.6/site-packages/pyscf/mcscf/casci_symm.py", line 63, in kernel
    return casci.CASCI.kernel(self, mo_coeff, ci0, verbose)
  File "/home/chenh/anaconda3/lib/python3.6/site-packages/pyscf/mcscf/casci.py", line 818, in kernel
    cas_natorb=self.natorb, verbose=log)
  File "/home/chenh/anaconda3/lib/python3.6/site-packages/pyscf/mcscf/casci.py", line 877, in canonicalize_
    sort, cas_natorb, casdm1, verbose, with_meta_lowdin)
  File "/home/chenh/anaconda3/lib/python3.6/site-packages/pyscf/mcscf/casci.py", line 396, in canonicalize
    casdm1 = mc.fcisolver.make_rdm1(ci, mc.ncas, mc.nelecas)
  File "/home/chenh/anaconda3/lib/python3.6/site-packages/pyscf/dmrgscf/dmrgci.py", line 318, in make_rdm1
    return DMRGCI.make_rdm12(self, state, norb, nelec, link_index, **kwargs)[0]
  File "/home/chenh/anaconda3/lib/python3.6/site-packages/pyscf/dmrgscf/dmrgci.py", line 332, in make_rdm12
    file2pdm = "spatial_twopdm.%d.%d.txt" %(state, state)
TypeError: %d format: a number is required, not range

I also tested in CASCI and found that it works well.
And if i don't set mc.fcisolver.nroots =2,the calculation can continue. Is the result obtained at this time credible?
from pyscf import gto, scf, mcscf, dmrgscf, mrpt

mol = gto.M(
    atom = [
        ['O', ( 0., 0.    , 0.   )],
        ['H', ( 0., -0.757, 0.587)],
        ['H', ( 0., 0.757 , 0.587)],],
    basis = '6-31g',
    symmetry = 1)

mf = scf.RHF(mol)
mf.kernel()

mc = dmrgscf.DMRGSCF(mf,4,4)
mc.state_average_([0.5,0.5])
mc.run()

mo = mc.mo_coeff

mc = mcscf.CASCI(mf,4,4)
mc.fcisolver = dmrgscf.DMRGCI(mol)
mc.run(mo)

mps_nevpt = mrpt.NEVPT(mc,root=1).compress_approx().kernel()

I cannot reproduce this error.  Can you check the version at
/home/chenh/anaconda3/lib/python3.6/site-packages/pyscf/__init__.py

The second script for NEVPT2 is all right. It's the recommended way to run state-average DMRG-CASSCF then NEVPT2

My version is 1.6.1. I am going to upgrade and test it to see if it still produces errors.

After I upgrade to the latest version 1.6.3, the error is still there.

I found this error is raised by isinstance function of  the following codes:
https://github.com/pyscf/pyscf/blob/3ed5a44ec0cd1bef70bb68576968e09c2614ab37/pyscf/mcscf/casci.py#L83
https://github.com/pyscf/pyscf/blob/3ed5a44ec0cd1bef70bb68576968e09c2614ab37/pyscf/mcscf/casci.py#L401
After I change```
if (isinstance(ci, (list, tuple)) and
not isinstance(mc.fcisolver, addons.StateAverageFCISolver)):
to

    if (isinstance(ci, (list, tuple, **range**)) and
        not isinstance(mc.fcisolver, addons.StateAverageFCISolver)):

from pyscf import gto, scf, mcscf, dmrgscf
mol = gto.M(
atom = [
['O', ( 0., 0.    , 0.   )],
['H', ( 0., -0.757, 0.587)],
['H', ( 0., 0.757 , 0.587)],],
basis = '6-31g',
symmetry = 1)
mf = scf.RHF(mol)
mf.kernel()
mc = dmrgscf.DMRGSCF(mf,4,4)
mc.state_average_([0.5,0.5])
mc.run()
mo = mc.mo_coeff
mc = mcscf.CASCI(mf,4,4)
mc.fcisolver = dmrgscf.DMRGCI(mol)
mc.fcisolver.nroots = 2
mc.run(mo)
WARN: Mulitple states found in CASCI solver. First state is used to compute the natural orbitals in active space.
CASCI state 0  E = -75.9774363937779  E(CI) = -6.07459456016701  S^2 = 0.0000000
CASCI state 1  E = -75.5779925101576  E(CI) = -5.67515067654676  S^2 = 0.0000000
<pyscf.mcscf.casci_symm.SymAdaptedCASCI object at 0x2b4167a0e160>

Thanks for digging into the problem. This bug is fixed in the latest updates.

It's very happy for me to contribute to pyscf.


Give working examples for the checkbox name

What
Give working examples for the checkbox name
Why
Checkboxes in Rails have a bit of a configuration quirk. If you don't add [] to the name, only the last selected value is passed to the controller, rather than an array of all of the selected values.
The name needs to be backed by indicating that the checkbox param is an array in the controller.
For example, if the checkbox is called nationality[], then you need to do the permit as:
params.permit(nationality: [])

Visual Changes
N/A

This is a strange one! Is it related to this gotcha?

It sounds like it. Good find!


appveyor continuously runs into network problems while downloading rustup

See line 10 of this build job
I think this may be caused by powershell somehow because I don't recall ever running into a problem like this one with Travis or pre-powershell AppVeyor configs.

appveyor is now using curl, which is more reliable


how to check strong consistency of database

Hi @anishathalye,
Currently, I am using https://github.com/pingcap/chaos to check strong consistency of TiDB. But  porcupine don't work with too many events so all my tests are out of memory before my chaos run.
Can you give me any solution to check strong consistency of database with chaos?
Besides, I propose that we can run continuously test ( about 200 events logs per test) and use porcupine to check them. If all tests passed, the database would be strong consistency. Is that true?

Regarding the verifier being slow, see #4.
Also, passing tests does not imply that the database is correct -- it just means that a particular run of the database was correct.

Going to close this for now. Feel free to re-open if you have any other questions.


webpack: bump admin-lte dependency

part of closing https://github.com/inveniosoftware/invenio-app-rdm/issues/956
See commit message for more details (+ link to differences in this folder which is relevant: https://github.com/ColorlibHQ/AdminLTE/tree/v3.1.0/dist/css)
webpack: bump admin-lte in semantic-ui for security
release: v1.3.10

Closing as Invenio-admin is being removed.

Closing as Invenio-admin is being removed.

There is an ongoing discussion to not remove entirely invenio-admin as it is still used from other invenio repositories like cds-videos and cds-ils.


Add please .git path to ignore by default

add fix in https://github.com/girishji/scope.vim/pull/24


Error implementing devsecops in jenkins

When we try to push results of trufflehog tool from jenkins to defect dojo using upload results.py (https://github.com/cyclopsbarrack/webapp/blob/master/upload-results.py) we are getting 403 issue.Any Possible solution to fix this?

which version of Truffehog do you use? the situation around v2 and v3 is a mess. reports are incompatibles. It's why we have 2 different parsers.

please provide url, request, response etc. the upload-results.py is a black box to us.

@valentijnscholten made some checks, the script use API v1 and importscan/ endpoint

@cyclopsbarrack there are not enough details here. Could you provide the message of the 403 error and the logs of the wusgi container of DefectDojo? the version of DefectDojo ? the deployment configuration (Docker/godojo) ?

which version of Truffehog do you use? the situation around v2 and v3 is a mess. reports are incompatibles. It's why we have 2 different parsers.

we have used both versions of the truffle log but still facing the same issue we also tried to push zap report still the same issue.

@cyclopsbarrack there are not enough details here. Could you provide the message of the 403 error and the logs of the wusgi container of DefectDojo? the version of DefectDojo ? the deployment configuration (Docker/godojo) ?

Here we are trying to setup a DevSecOps environment (https://owasp.org/www-project-devsecops-maturity-model/#) using this method (https://github.com/devopssecure/webapp) . We have automated everything using Jenkins. This is our project code (https://github.com/cyclopsbarrack/webapp/)  we are trying to push all the reports (truffle hog, owasp dependecy check, sonarqube, Zap, etc ) to defectdojo. Since API v1 has been stopped . we are using API v2. is there any other effective method to push the reports from jenkins to defectdojo?
jenkins.log.1.txt
jenkins.log .txt

Could you please provide the exact request / responses you are seeing? The earlier snippet seems to suggest you are using api v1. And provide info on which version of defect dojo, which install method, etc.

Closed due to inactivity


Run an IO action in the bench without timing it

I want to benchmark a search algorithm implementation. As a result, I'd like to generate random needles for each iteration of the bench. However, I don't want to also measure the PRNG. Is there a way to do this currently?

It's a bit unclear to me, what design you have in mind. How would your benchmark look, if you agree to measure the PRNG overhead?

It would be something like:
bench "Foo" . nfAppIO (\h -> doTheSearch <$> genRandomNeedle <*> pure h) $ haystack
However, this will end up benchmarking both genRandomNeedle and the search for it, whereas I only want to benchmark the search. Furthermore, I only need IO for the random generation, not the search. The idea is that as the search is run many times during the benchmarking process, by randomizing the needle I get a good idea of 'average' performance.

Unfortunately, I don't think there is a reasonable workaround for this particular scenario. Both individual generation of a needle and the search itself are well beyond the granulatiry of a system timer, so we cannot start-stop it.
FWIW I think that genRandomNeedle implemented via http://hackage.haskell.org/package/random-1.2.0/docs/System-Random-Stateful.html#g:11 should be blazingly fast and thus would not affect overall measures in a significant way.
In general I'm inclined to advise against such design: to achieve reliable results with a predictable deviation, it's crucial to benchmark exactly the same data over and over again. Maybe generate several thousands of needles at top level and iterate over them all in a single benchmark?..

Yeah, I think you're right on all counts. However, if I generate all the needles at the top level, then do them all in one bench, I'll get a result for a thousand needles, not the average for one needle. Unless I'm missing something?

Well, you can divide by thousand to approximate the average. I agree that it's not that convenient, but absolute numbers are of limited value anyways.

Fair enough. Thanks for the advice!

A different data point - what if I want to test an external system that needs restarts, preparation or warmup/cache clearing before benchmarking? This is not something one can do outside of the testing loop. This is also something that for example hyperfine does provide.

Unless I'm missing something, this is what env is for.

Does env run for each benchmark or for each timing run?

env runs once, before measurements of a particular routine take place.


why dont make a authentication page ?

Sorry why dont you make authentication ?

For the moment we secure Articulate behind an Nginx reverse proxy with basic auth. But full user management is on our roadmap. #22


debug sporadically breaking unit test

Coverage remained the same at 86.058% when pulling 411744d6fd5687445fd02893b801100ca290b0ee on peterbe:debug-sporadically-breaking-unit-test into a1df4fb4406dc5b77c51c216433f375b280f598e on mozilla:master.


Add Emacs keys

I notice you already use Ctrl-a for Home and Ctrl-e for End, which is expected behavior in Emacs-like editors. Ctrl-p for previous and Ctrl-n for next are in the same family (try them on the shell!), as is Ctrl-g for cancel. This would make it behave as expected for those of us used to these bindings.

Thanks for your PR! Sorry for the waiting time, I have been quite inactive on GH for the last weeks.

v1.5.2 will be released in a few minutes including your patch. :+1:


Create avl_tree implementation in C++

Please follow the Contributing Guidelines & naming conventions in the guidelines accordingly. 🙂
Following naming guidelines is important to maintain consistency.😊
Please NOTE: In case, same implementation of an algorithm is submitted by someone else and it gets accepted, then your contribution won't be merged ( to avoid duplicates )

Hi, I can do this.

Hi ,is the issue still open ?

@ranaakash Don't know, ask @subhahu123 -- these labels are quite confusing, and so far the only way repo owners have chosen to communicate through


C3-Qiskit interface produces results with incompatible state labels

Describe the bug
Qiskit ordering of qubits is different from what is commonly used in Physics textbooks. C3 follows the general Physics textbook style. However when producing output through the C3-Qiskit interface, the expectation is to get results with compatible qubit ordering.
To Reproduce
The following code snippet checks C3 output with Qiskit output. This fails due to qubit indexing mismatch:
    c3_qiskit = C3Provider()
    backend = c3_qiskit.get_backend('c3_qasm_perfect_simulator')
    backend.set_device_config("test/quickstart.hjson")
    qc = get_6_qubit_circuit
    job_sim = execute(qc, backend, shots=1000)
    result_sim = job_sim.result()

    # Test results with qiskit style qubit indexing
    qiskit_simulator = Aer.get_backend("qasm_simulator")
    qiskit_counts = execute(qc, qiskit_simulator, shots=1000).result().get_counts(qc)
    assert result_sim.get_counts(qc) == qiskit_counts

Expected behavior
By default, the c3-qiskit interface should produce qiskit compatible labels with an option to disable this feature if required by the user.
Additional context
Importantly, this change in the representation of multi-qubit states affects the way multi-qubit gates are represented in Qiskit. More details are available here and it might be useful to check if a possible fix goes beyond mere relabelling of qubit state names in the experiment result.

Closed in #59

The fix introduced in #59 misses edge cases. For example 011111 should be flipped to 111110 but that doesn't happen because any leading zeros are automatically removed when converting from binary to hex or int. So 011111 is treated as 11111 and it stays the same. A possible fix would be to delay the conversion to hex until the very last step, before returning the counts in the result data structure. It would then require changes in get_labels() and flip_labels() in c3_backend_utils.py


sway-language: ignore events with empty layout

When using e.g. wtype, which also sends events to emulate keyboard presses, this confuses Waybar and incorrectly hides the module. This fix will ensure compatibility with wtype, simply ignoring events where layout is not set, and only reacting on events which actually change the keyboard layout.

Thanks Maxim!


Dev test: Resistance in a Wire 1.5.0-dev.1

@emily-phet @terracoda Accessible Resistance in a Wire is ready for a dev test. This version includes keyboard navigation and auditory description. This task is mostly to test screen reader behavior, but please verify that the sim works correctly in general.
Before beginning, familiarize yourself with how a screen reader works. Here is a page with information about PhET's supported screen readers and documentation about how to use them:
Screen reader intro
Link to sim: https://www.colorado.edu/physics/phet/dev/html/resistance-in-a-wire/1.5.0-dev.1/phet/resistance-in-a-wire_en_phet.html
The above sim has descriptions that can be read with a screen reader. That content can be "seen" here:
Link to a11y view: https://www.colorado.edu/physics/phet/dev/html/resistance-in-a-wire/1.5.0-dev.1/phet/resistance-in-a-wire_a11y_view.html
While testing, please make sure that all descriptions and alerts accurately describe the state of the simulation. They should also always be readable with a screen reader using common navigation strategies.
PhET supports the following platforms for accessibility so please test these:

[ ] JAWS with latest Windows, latest Firefox
[ ] NVDA with latest Windows, latest Firefox
[ ] VoiceOver with latest macOS, latest Safari

JAWS might complain about Firefox Quantum. If that is the case, please test with https://www.mozilla.org/en-US/firefox/organizations/
Assigning to @phet-steele. @terracoda can you also please review this version? @ariel-phet can you please assign priority to this issue?

@jessegreenberg, sounds awesome in Voice Over and Safari (Mac 10.13.2, Safari 11.02)
It all works great for me.
It is also easy to jump back and forth between PDOM and alerts to compare the state of the equation with the alerts.
The only description that I was  not expecting was this one where there are three explicit "much much larger than"'s. All three sliders are different,  and R is so huge it is off screen. I don't think have a better way to handle this edge case, so I think we have to go with "much much larger" across the equation phrase.
A user would not get a sense of any difference between between rho, L, and A, but they would get a sense that R is huge.

Also, I get the added "a lot" at the appropriate times, I think.

Thanks @terracoda! Sorry I forgot to mention, we usually track progress in qa repository issues but create new bug reports in the sim repo. Ill create an issue in resistance-in-a-wire for the above bug and if you notice anything else please create an issue there.

This is done..finally. Screen readers are hard!

Excellent, thank you @phet-steele and team. Closing issue.


Update JuniDB.md

https://github.com/w3f/Grants-Program/pull/466
Application Checklist

[x] The application template has been copied, renamed ( project_name.md) and updated.
[x] A BTC or Ethereum (DAI/USDT) address for the payment of the milestones is provided inside the application.
[x] I have read and acknowledged the terms and conditions.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted.

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

@andskur You removed the key deliverable of Milestone 1. It looks like your milestone now only includes forking substrate, I don't see how we would pay $7,000 for this.

@mmagician I think the purpose is to update offchain::ipfs to Substrate v3, or at least that's what I read from here. But @andskur I would first clarify that the team behind offchain::ips is ok with that (they may want to do this themselves soon) and I'd argue that your work should be merged into offchain::ipfs as part of the milestone instead of it just living in a separate fork.

@andskur Btw. it looks like you still need to sign the terms and conditions.

And could you please update the total cost and duration too?

Hi all,
@mmagician  It's ok )
@Noc2 Fixed it.
@alxs Made changes.


out of memory exception while running bubbleprof on 10s autocannon

Expected Behavior

After finishing the HTTP benchmarks and pressing CTRL-C on the clinic bubbleprof process it should finish analyzing data and show results in a new browser window.
Current Behavior

After finishing the HTTP benchmarks and pressing CRTL-C the clinic process dies with an out-of-memory exception.
Steps to Reproduce (for bugs)

npx create-hops-app@next --hops-version next --template hops-template-react@next my-app
cd my-ap
./node_modules/.bin/hops build -p
clinic bubbleprof -- node ./node_modules/.bin/hops serve -p
autocannon http://localhost:8080
press CTRL-C on the clinic process after autocannon finishes and observe an out-of-memory exception.

Sample upload

https://upload.clinicjs.org/public/54836c48aa37cb3dd80c003d6c9667e27c1fcd9f3639531c5e87294fe4021f62/12016.clinic-bubbleprof.html

example stack trace output

 /tmp/my-app $ clinic bubbleprof -- node ./node_modules/.bin/hops serve -p
To generate the report press: Ctrl + C
[my-app] started in production mode
Server listening at http://localhost:8080
^CAnalysing data
<--- Last few GCs --->
[12008:0x36fd870]    52854 ms: Mark-sweep 1394.6 (1424.0) -> 1393.9 (1423.0) MB, 959.0 / 0.0 ms  (average mu = 0.054, current mu = 0.019) allocation failure scavenge might not succeed
[12008:0x36fd870]    52862 ms: Scavenge 1394.7 (1423.0) -> 1394.0 (1423.5) MB, 5.7 / 0.0 ms  (average mu = 0.054, current mu = 0.019) allocation failure
[12008:0x36fd870]    52871 ms: Scavenge 1394.9 (1423.5) -> 1394.2 (1424.0) MB, 6.1 / 0.0 ms  (average mu = 0.054, current mu = 0.019) allocation failure
<--- JS stacktrace --->
==== JS stack trace =========================================
0: ExitFrame [pc: 0x326ace1dbe1d]
1: StubFrame [pc: 0x326ace1b4fbf]

Security context: 0x0d3813d9e6e1
2: /* anonymous */ [0x103d0e502ae9] [/home/zaubernerd/.nvm/versions/node/v10.13.0/lib/node_modules/clinic/node_modules/terser/dist/bundle.js:~44] [pc=0x326ace6ebb89](this=0x103d0e502461 ,i=0x05c001e94d21 <AST_Number map = 0x22e3c4822b79>,r=0x16382e83f751 <JSFunction u (sfi = 0x10fb5920a109)>)...
FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
1: 0x8daaa0 node::Abort() [node]
2: 0x8daaec  [node]
3: 0xad73ce v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, bool) [node]
4: 0xad7604 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, bool) [node]
5: 0xec4c32  [node]
6: 0xec4d38 v8::internal::Heap::CheckIneffectiveMarkCompact(unsigned long, double) [node]
7: 0xed0e12 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::GCCallbackFlags) [node]
8: 0xed1744 v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [node]
9: 0xed43b1 v8::internal::Heap::AllocateRawWithRetryOrFail(int, v8::internal::AllocationSpace, v8::internal::AllocationAlignment) [node]
10: 0xe9d834 v8::internal::Factory::NewFillerObject(int, bool, v8::internal::AllocationSpace) [node]
11: 0x113cf9e v8::internal::Runtime_AllocateInNewSpace(int, v8::internal::Object**, v8::internal::Isolate*) [node]
12: 0x326ace1dbe1d
Aborted (core dumped)

Environment

Clinic version: v2.0.0 Doctor v3.5.0

Node version: v10.13.0 npm v6.4.1

Browser name and version: Chrome 70

Operating system and version: Linux 64-bit

This occurs generally because bubbleprof requires a rather large amount of diagnostic data to be generated by Node.js and loaded into the analysis engine. Depending on the amount of async activity being performed by the application, there are possibly two approaches to resolving this:

You can try a shorter benchmark (e.g. 5 seconds instead of 10). This will reduce the amount of profile data necessary for bubbleprof to sift through but it can also limit the fidelity of the profiling, so it is definitely a trade off.

Try using the v8 memory flags (e.g. NODE_OPTIONS="--max-old-space-size=N" clinic bubble ..., etc) to adjust the amount of memory available usable by the Node.js instance.

If either of these do not work, please let us know! :-)


Check error before logging

What this PR does / why we need it:
Check error before logging. My previous PR #5158 was merged before I noticed that it writes to log even if there is no error.
Release note:
None

/assign @stu-gott

/retest

/retest


App crashes after switching to front camera

Hi, I installed the app and when I try to switch to front camera the app crashes.
LG G7, Andorid 9
    Process: com.android.example.cameraxbasic, PID: 729
    java.lang.IllegalArgumentException: CaptureRequest contains unconfigured Input/Output Surface!
        at android.hardware.camera2.CaptureRequest.convertSurfaceToStreamId(CaptureRequest.java:674)
        at android.hardware.camera2.impl.CameraDeviceImpl.submitCaptureRequest(CameraDeviceImpl.java:1093)
        at android.hardware.camera2.impl.CameraDeviceImpl.setRepeatingRequest(CameraDeviceImpl.java:1139)
        at android.hardware.camera2.impl.CameraCaptureSessionImpl.setRepeatingRequest(CameraCaptureSessionImpl.java:298)
        at androidx.camera.camera2.impl.CaptureSession.issueRepeatingCaptureRequests(CaptureSession.java:443)
        at androidx.camera.camera2.impl.CaptureSession$StateCallback.onConfigured(CaptureSession.java:604)
        at androidx.camera.core.CameraCaptureSessionStateCallbacks$ComboSessionStateCallback.onConfigured(CameraCaptureSessionStateCallbacks.java:115)
        at android.hardware.camera2.impl.CallbackProxies$SessionStateCallbackProxy.lambda$onConfigured$0(CallbackProxies.java:53)
        at android.hardware.camera2.impl.-$$Lambda$CallbackProxies$SessionStateCallbackProxy$soW0qC12Osypoky6AfL3P2-TeDw.run(Unknown Source:4)
        at android.os.Handler.handleCallback(Handler.java:873)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loop(Looper.java:193)
        at android.os.HandlerThread.run(HandlerThread.java:65)```

Thanks for the feedback. It sounds like the issue may not be with the sample itself, but potentially with the library. Can you please file a bug in the appropriate component of the Android Issue Tracker and reply here once the you have opened the issue there? It should be under https://issuetracker.google.com/components/618491

Closing due to inactivity

Hi, I have an LG G7 ThinQ too, and this issue occurs time to time while switching to front cam.


Fuzzy generation result problem

Thank you for your outstanding contribution！
On the left is the original video frame, and on the right is the generated result. You can see that it has become much blurry. Is there any solution？

Same problem,the hair cannot be represented well.

 There appears to be some temporal smoothing going on here, and the radnerf is trained on slightly erroneous alignments which may lead to poor convergence (seen this in previous nerf work).

The same problem reasoning results in a fuzzier result than the original video


MUMUP-2578 Source Wiscard widget screenshot from deployed /images .

Applies the staged-for-prod approach for sourcing this screenshot to localhost, using the prod-deployed /images since /images is not reliably deployed to localhost.
Alternative solution considered but not adopted: sourcing the image directly from git.doit's web interface, figuring that localhost running against bleeding edge of that repo modulo git.doit availability would be good tradeoffs.

Coverage remained the same at 15.784% when pulling e58edd740f411c798b39e4c4e34c1ceb027fe771 on apetro:MUMUP-2578_wiscard_screenshot_from_images into 80431bc74be23b1c3d96de7400c694610dec218a on UW-Madison-DoIT:master.

directly hitting git.doit wouldn't be nice for that service.
I would rather just use the local image for localhost instead of hitting my.wisc.edu for the image.

Agree with @vertein


Prep project boards for Migration

Overview
We need to prepare the Project Boards for the forced migration that GitHub is doing in August, so that it is already working well for the team by then.
Action Items
Repeat first 3 steps for each card

[ ]  copy this template to a comment below### Text from project board card, [Add name of board here], [Add name of column here]
```
[PASTE CARD CONTENT HERE]
```
- [ ] Determine where this item goes on the wiki (page and section)
- [ ] copy item to wiki
- [ ] hide this comment when completed.

[ ] copy the content of the card (by clicking the 3 dots and selecting Edit Note, and then CRTL+C) and paste it into your new comment where it says `[PASTE CARD CONTENT HERE]
[ ] delete card from project board
[ ] add project board URL(s) in resources below
[ ] add the project board migration label to this issue
[ ] add issue to project board
[ ] move issue to Prioritized Backlog

Resources/Instructions
Classic Project board URL(s):

Open:

Closed issues
Personas and User Stories
Project Management

Closed:

General
Features
Infrastructure

New Project board URL(s): None
Card Content Spreadsheets*

Project Board Cards: CTI-website-frontend: Personas and User Stories

*Temporarily housed in the All Projects Initiative shared Google drive

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Start Here
[PASTE CARD CONTENT HERE]

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Ice Box
https://github.com/civictechindex/CTI-website-backend/issues/37

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Meeting Review

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Meeting Review

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Meeting Review

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, Meeting Review

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Project Management, on-hold
On hold : Waiting on workflows https://github.com/civictechindex/CTI-website-frontend/issues/62

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Infrastructure, In progress

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Infrastructure, In progress

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.

Text from project board card, Infrastructure, In progress

[ ] Determine where this item goes on the wiki (page and section)
[ ] copy item to wiki
[ ] hide this comment when completed.


Refactor useViewportFeatures

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

37 of 80 (46.25%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.09%) to 69.335%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/react-api/src/hooks/utils.js
0
4
0.0%

packages/react-api/src/hooks/useGeoJsonFeatures.js
15
30
50.0%

packages/react-api/src/hooks/useTilesetFeatures.js
19
43
44.19%

Totals

Change from base Build<PHONE_NUMBER>:
-0.09%

Covered Lines:
935

Relevant Lines:
1259

💛  - Coveralls

@VictorVelarde error management isn't gone, still there. It was just generalised fof having the same one in both hooks.

We will check with @padawannn tomorrow before the merge


fix(github): update issue templates

I noticed that most users report in wrong categories including seasoned @freeCodeCamp/moderators - likely due to the fact the usually one would be in a rush to just report a bug. This update should help a bit.

Might be off-topic to ask this in this PR, but just a question!
I actually had no idea we were supposed to open an issue here when we see a typo or dead link on articles. ^-^'
I always emailed in, and it'd be promptly fixed.
In future, should I be favoring opening an issue instead? (Or alternatively, should emailing the editorial team directly be added as an option on that template?)


Add test module for ACVP known answer tests

While working on upgrading the implementation to match the released standard I have come to realize there is a real lack of test vectors that can be used to black box test the signature and verify functions that were changed since the IPD.
As part of that investigation, I implemented a test script to run the ACVP test vectors. It turns out they all pass on the IPD version. Sso I thought I'd create a pull request to add this test script independently from the new-release pull request, since they all fail there.

Very nice, thank you for this! I just finished updates for FIPS 203 and 204, and am now working on this. Hopefully, similar style updates will make it a bit more straightforward...


Sagemaker Controller supports UpdateWeights API

Is your feature request related to a problem?
The Sagemaker controller doesn't currently support updating weights and capacities (UpdateEndpointWeightsAndCapacities API). Currently, to update these values, users will need to create a new endpoint config, create it, then update the endpoint to use the new config.
Describe the solution you'd like
Ideally, updating the Endpoint would take care of this, new fields may be required to allow for this API to used when the Endpoint spec is changed. Updating the EndpointConfig may not be the best option as multiple Endpoints could be using the same config.
Describe alternatives you've considered
The only available option at this time is to create new EndpointConfigs, and update the endpoint to use that config. This leads to extraneous EndpointConfigs hanging around if not cleaned up, as well as a less than ideal user experience when it comes to A/B testing of models.

/cc @surajkota @aws-controllers-k8s/sagemaker-maintainer

i see, thanks @surajkota--though one use case is for something like a scare instance type (i.e. p4d), where creating new instances may be difficult/impossible, just to update the weights :/
somewhat related, i've created a new issue: https://github.com/aws-controllers-k8s/community/issues/1889

Synced offline to understand the priority. Will keep this issue open incase the workaround for updating desired weights creates operational complexity and if there are other users who are impacted by this.
Thanks

/remove-lifecycle stale

/remove-lifecycle stale


webpack dynamic modules

It's seem that current webpack config don't support require.ensure
Simple component with gl-react
//@flow

import React from 'react'
import ReactDOM from 'react-dom'
import BaseTemplate from '../layout/baseTemplate'

export default class extends React.Component {
    node: any;
    static async getInitialProps({ req }): any {
        const isServer = !!req;
        return {
            isServer: isServer
        }
    }
    componentDidMount(){
        require.ensure([], () => {
            const { Shaders, Node, GLSL } = require("gl-react");
            const { Surface } = require("gl-react-dom");
            const shaders = Shaders.create({
                helloGL: {
                    // This is our first fragment shader in GLSL language (OpenGL Shading Language)
                    // (GLSL code gets compiled and run on the GPU)
                    frag: GLSL`
                        precision highp float;
                        varying vec2 uv;
                        void main() {
                          gl_FragColor = vec4(uv.x, uv.y, 0.5, 1.0);
                        }
                    `
                    // the main() function is called FOR EACH PIXELS
                    // the varying uv is a vec2 where x and y respectively varying from 0.0 to 1.0.
                    // we set in output the pixel color using the vec4(r,g,b,a) format.
                    // see GLSL_ES_Specification_1.0.17
                }
            });
            ReactDOM.render((
                <Surface width={300} height={300}>
                    <Node shader={shaders.helloGL} />
                </Surface>
            ), this.node);
        });
    }
    render(): any {
        return (

 this.node = node}>

        );
    }
}

It's work in dev but when i try to next build and then next start get error that chunk not find
It's name 3.3 and it's located in .next folder 😕
Thanks!
UPD: pull request for fix open

Yes on https://github.com/zeit/next.js/blob/master/server/index.js#L42 we serve only commons.js

I'd love to have a simple solution for dynamic imports, especially with the more definitive ES APIs coming

@rauchg I have idea, it's can look like this :
<Require src="./../components/test">
    <Client>
        {'Hello from Client!'}
        <ProxyComponent isClient={true} ...otherOptions />
    </Client>
    <Server>
        {'Hello from Server!'}
        <ProxyComponent isServer={true} ...otherOptions/>
    </Server>
</Require>

or if we want just make a lazy load component
 <Require src="gl-react"/>

PS: It's still random thoughts

@arunoda, @rauchg  updated to master version of next, still have problem with require.ensure.
In dev mode all works just fine.
In prod mode after next build chunk are created but not served as expected.
For this i still see solution to set a custom chunkFilename, and serve route like
[name].[hash:4] so we get lazy modules support easy and very clean, later on we can think about it more.
Thanks!

@mr47 I think it won't as the current state.
Currently, we haven't looked into what's really going on here of our support for require.ensure.
Since we open the programmatic API with 2.0 and the webpack config, it allows us to play with these features in the userland.
If we need some core support it, let's think about doing it for 2.1 or some other releases.
Keep us updated.

Let's move the discussion to #728, because this will be a priority after 2.0


Istio 1.4.7 to 1.5.1 default profile with egress upgrade failure, 1.4 proxy pods for egress broken

Bug description
After upgrade from 1.4.7 to 1.5.1, existing 1.4 proxy pods that are using egress gateway have connection failures.   503 for http and reset for https.
Same thing with demo profile works.
Expected behavior
Existing 1.4 mesh pods can still connect via the egress gateway.
Steps to reproduce the bug

download 1.4.7
istioctl manifest --set profile=default --set gateways.components.egressGateway.enabled=true
deploy egress sample defined https://istio.io/docs/tasks/traffic-management/egress/egress-gateway and check the traffic and logs as in the example.
download 1.5.1
modify default profile to have egress enabled
istioctl upgrade -f <path to 1.5.1 default profile with egress> --force
check the traffic and logs as in the example.

http fails with 503
https fails with connection reset

sleep proxy logs
[2020-04-09T16:27:33.123Z] "GET /politics HTTP/1.1" 503 UF,URX "-" "-" 0 91 3031 - "-" "curl/7.64.0" "84a66110-e563-42d0-af3b-b3ce4efefb51" "edition.cnn.com" "<IP_ADDRESS>:80" outbound|80|cnn|istio-egressgateway.istio-system.svc.cluster.local - <IP_ADDRESS>:80 <IP_ADDRESS>:32778 - -
[2020-04-09T16:27:37.566Z] "- - -" 0 UF,URX "-" "-" 0 0 1002 - "-" "-" "-" "-" "<IP_ADDRESS>:443" outbound|443|cnn|istio-egressgateway.istio-system.svc.cluster.local - <IP_ADDRESS>:443 <IP_ADDRESS>:44000 - -
[Envoy (Epoch 0)] [2020-04-09 16:27:56.175][19][warning][config] [bazel-out/k8-opt/bin/external/envoy/source/common/config/_virtual_includes/grpc_stream_lib/common/config/grpc_stream.h:91] gRPC config stream closed: 14, upstream connect error or disconnect/reset before headers. reset reason: connection failure
[Envoy (Epoch 0)] [2020-04-09 16:28:12.953][19][warning][config] [bazel-out/k8-opt/bin/external/envoy/source/common/config/_virtual_includes/grpc_stream_lib/common/config/grpc_stream.h:91] gRPC config stream closed: 14, upstream connect error or disconnect/reset before headers. reset reason: connection failure

Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
1.4.7 and 1.5.1
How was Istio installed?
istioctl
Environment where bug was observed (cloud vendor, OS, etc)
IKS

@linsun @rvennam

_logs-pilot-1.5.1.txt
full piot logs

@kramvan1 does your egress gateway change ip before and after upgrade?  The reason I'm asking is if the endpoint has changed and proxy can't connect to istiod (the other https://github.com/istio/istio/issues/22841) then it will attempt to connect to the prior endpoint which can be wrong.
cc @GregHanson

might be the same as https://github.com/istio/istio/issues/22841 if demo works, or similar

@howardjohn yes, PR #22848 seems to fix this.   will test again once it's merged. thx.

Verified with latest 1.5-dev


CI: check out examples dependencies from git instead of crates

@GuillaumeGomez let's have all the tests in these PRs run to completion to be sure everything works as intended.

Fine by me.


Allow users to specify where the logs or results are stored

Provide the options for users to specify where to store the logs or results of a lm-eval job

this is tracked in the trustyai-service-operator repo and the implementation is done. close this one
https://github.com/trustyai-explainability/trustyai-service-operator/pull/347


Question about resolution change in UniMatch V2 for DINOv2 pretraining

I would like to express my sincere gratitude for your tremendous contributions to the UniMatch V2 project. It’s truly inspiring and valuable to the community.
I have a question regarding the resolution change in UniMatch V2. As I understand, the method adapts to the DINOv2 pre-trained model by increasing the image resolution to 518. However, many previous methods, including earlier versions of UniMatch, typically used a resolution of 321.
Given this significant difference in resolution, I wonder whether it is entirely fair to compare the performance of UniMatch V2 directly with previous methods. Would such a comparison be reasonable, or should we account for the resolution difference in some way to ensure a more accurate evaluation?

Thank you for raising this question.
As we stated in the paper, we do not consider our UniMatch V2 as a fair comparison with previous published methods. This is not only because the resolution you mentioned (except Pascal, the resolution used for other datasets is indeed the same as previous methods), but mainly because the DINOv2 encoder we adopted is much stronger than the ResNet encoder adopted by previous works.
However, we appeal that we should replace the ResNet encoder with the much stronger DINOv2 encoder. This will bring much better performance and a broader audience of our semi-supervised segmentation field. To ensure fair comparisons, we have included comprehensive ablation studies with the FixMatch and UniMatch V1 baselines using the same DINOv2 encoder.

Thank you very much for your answer


reconfiguration cae

reconfiguration linear probe cae model and CI.

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


can i know program location in registry windows 10 !

want to add Icon string

Software\Classes\SystemFileAssociations\<filetype>\shell

there is no Key for googleImageSearch
to add an icon
i search for key ex googleSearchImage and add a string "Icon" with value of any icon or exe
(copy path of an .ico )

like this

If you didn't install the program for all users, you should look in the CURRENT_USER hive instead of the LOCAL_MACHINE hive.

Thank you

You're welcome :-)


⚠ Samsung has degraded performance

In 943589e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2293 ms

Resolved: Samsung performance has improved in 4b8bde1.


HIPCC not defined in HIP

The __HIPCC__ predefined macro seems not to be defined during compilation. Here is a test case with the following source:
#include <stdio.h>

int main(int argc, char* argv[]) {
#ifdef __HIPCC__
	printf("__HIPCC__ is defined\n");
#endif
#ifdef __CUDACC__
	printf("__CUDACC__ is defined\n");
#endif
#ifdef __HCC__
	printf("__HCC__ is defined\n");
#endif
	return 0;
}

After compiling with hipcc (/opt/rocm/bin/hipcc -o test-macro -O2 -I/opt/rocm/hip/include test-macro.cpp) and running, it returns just the following output:
__HCC__ is defined

The compiler makes two passes over the source code, once for the cpu target and another for the gpu target, and the discards the source code that is not meant for the target. I believe the __HIPCC__ macro is only define during the gpu pass, so it would not show up on your test.

@ekondis The __HIPCC__ macro is defined by hip_runtime headers and not by hipcc. Modifying the above code to include "hip_runtime.h" fixes the issue.

Including "hip_runtime.h" fixes the issue.