eMap mapping in GroundTruth

Hello Dr. Bao,
Thank you very much for sharing your very interesting work.
While trying to make the example matlab Code work I noticed that, in the ground truth, materials are mapped from some arbitrary (?) shuffled Indices to the order of materials as shown in the material namens DB.
Is there a specific reason why you did this? Is this baked into the scenes (by actually using eg iron emissivity for sky in some scenes) or just done for ground truth generation?
I ask, because I can not get the matlab Code to output expected materials (I teste for example Asphalt and sky pixels in scene 1) , and am wondering if I need to apply some mapping to the outputs?

Hello Daepilin,
Thanks for your interest in our work.
When the scenes in the database were synthesized, each scene had its own subset of the material library, and each scene was synthesized independently by different artists. They were assigned global material indices in post processing for ground truth generation. That leads to the seemingly random shuffled indices.

Thank you very much for the quick response!

Thank you very much for the quick response!


Run

Enhancement that fixes #70

This enhancement now allows an optional, case-insensitive data-run attribute on the SCRIPT tag, which determines when that script is executed. All page callback values are allowed: beforeInit, init, reinit, beforeAnimation, afterAnimation, back, afterBack, beforeRemove. If data-run is not provided, it defaults to init.

Note that defining functions within inline scripts executed in this method does not work quite as expected. Instead of writing:
function myFunctionHere() {
    //...
}

You must define the function in the style of a variable:
window.myFunctionHere = function() {
    //...
}

Not sure about the use of eval, perhaps use new Function or a data uri?

good


Memory leak in useId

Provide a general summary of the issue here
useId uses global variable idsUpdaterMap for storing id updaters and cleans it only in useLayoutEffect that doesn't run in a server environment thus leading to memory leak in SSR.
Probably cleanup doesn't work on client either because memory leak is reproducing on client (see "steps to reproduce")
https://github.com/adobe/react-spectrum/blob/main/packages/%40react-aria/utils/src/useId.ts#L30-L34C3
let idsUpdaterMap: Map<string, (v: string) => void> = new Map();

let updateValue = useCallback((val) => {
  nextId.current = val;
}, []);

idsUpdaterMap.set(res, updateValue);

useLayoutEffect(() => {
  let r = res;
  return () => {
    idsUpdaterMap.delete(r);
  };
}, [res]);

🤔 Expected Behavior?

useId should not use global object and should cleanup memory on the server side too
Cleanup should work on client

😯 Current Behavior

useId uses global object and doesn't cleanup memory on the server side
Cleanup doesn't work on client

💁 Possible Solution
No response
🔦 Context
This issue leads to memory overflow in a server environment
🖥 Steps to Reproduce
At first I thought the problem is reproducing only on server but tests show that the same problem is appearing on client. Despite the problem much more dangerous on server I will show how to reproduce it on client since it easier to do.
The simplest client example provided here: https://stackblitz.com/edit/stackblitz-starters-127s5h?description=React   TypeScript starter project&file=src/leak.tsx,src/App.tsx&title=React Starter

Take a memory snapshot of the application
Press "Add items" button several times
Press "Remove items" button (in this step we are unmounting all components that use useId. idsUpdaterMap should be cleaned up at this step)
4.0. Optionally force garbage collection to make sure there really is a leak
4.1. Take a memory snapshot and check the objects allocated between Snapshot 1 (step 1) and Snapshot 2 (step 4.1).

There will be alot of small objects allocated in closure by useId. If you will log one of them you will ensure that it is exactly that id updater callback we've set in the idsUpdaterMap in useId

Version
3.29.0
What browsers are you seeing the problem on?
Chrome
What operating system are you using?
MacOS
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

I am not sure this is us, at least on the client. I ran jest with the same steps and exposed our map via export to the test. After adding elements and removing them, the map is empty. Can you try running this with just react's useId?

I am not sure this is us, at least on the client. I ran jest with the same steps and exposed our map via export to the test. After adding elements and removing them, the map is empty. Can you try running this with just react's useId?

I tried this with react's useId and of course there is no any such callback in memory snapshots in this case. In snapshot we can see that this callback created at useId.ts:30. After logging this callback I see the exact signature of this updateValue callback. Maybe somehow these callbacks still available from another part of code and thats why GC can't delete them?
Anyway we can say confidently that there is a leak in the server environment just by seeing at this code. It uses global variable that is never cleared. We can focus specifically on the server side.


docker-compose and passwords from environment

I was experimenting with docker-compose and I saw something a little bit disturbing in my output of phpinfo() function:  LAMPDB_1_ENV_MYSQL_ROOT_PASSWORD	lamppass
my docker-compose.yml:
lampapp:
build: .
ports:
- "8000:80"
links:
- lampdb
- lampmemcache
volumes:
- /var/
lampdb:
image: "mysql:5"
volumes:
- ./mysql:/etc/mysql/conf.d
environment:
- MYSQL_ROOT_PASSWORD=lamppass
lampmemcache:
image: "memcached"
I know that it is set by 'environment:' but my question: Is it O.K. (I think it is not) any application from my 'composed' docker infrastructure to see set passwords from the installation process ?

Opz. This is for docker/compose not for docker/docker. Closing here, reporting at docker/compose

@ctmnz actually, this is a behavior of docker (not docker-compose). You can find a warning about this in the user-guide; https://docs.docker.com/userguide/dockerlinks/#environment-variables

**Warning: ** It is important to understand that all environment variables originating from Docker within a container are made available to any container that links to it. This could have serious security implications if sensitive data is stored in them.

Thanks for the elaboration @thaJeztah


Fix issue with routing performance when using updateOnRouterChange

Fix for #97

Summary of changes:
Intended/example use case:
When using updateOnRouterChange the update would fire multiple times per route change causing performance problems. With this change the update will fire only once after a successful navigation.
Checklist:

[x] npm run build runs without error
[x] ng serve spawns app, Intercom messenger is visible and interactive, and there are no errors in the console

Closes issue: #

Thanks @andy-hook! I’ll take a look to see what’s happening with the test.

Great, thank you @scott-wyatt

Hi @wbhob @scott-wyatt, thanks for looking at this. Is there anything more I can do to get this out in a patch release?


Wrong accessibility?

Using the most recent package in a NET 6 MVC app fails in
       builder.Services.AddAuthentication(OpenIdConnectDefaults.AuthenticationScheme)                   .AddMicrosoftIdentityWebApp(builder.Configuration.GetSection("AzureAd"))                   .AddMappedGroups();
due to not being able to find a matching constructor.
After changing class and method definitions in GroupsMapper and GroupsMapOptainer from internal to public, the application can start.
However the Graph call(s) fails with 'ServiceException: Code: Authorization_RequestDenied Message: Insufficient privileges to complete the operation.' (this may be an issue related to app registration)

Thanks for reaching out.
Indeed, yes. I didn't want to invest time to test the versions starting with .NET 6 against a real life Azure AD, because I haven't had the need using this library myself anymore, and I wans't aware of that somebody is actively using it. So it became more like a showcase of new .NET or C# versions.
That issue is fixed in version 1.26, alongside some dependency updates:
https://github.com/peterwurzinger/AuthOida/pkgs/nuget/AuthOida.Microsoft.Identity.Groups/76618506
If you could try that and confirm it to be working, I would then release it.
The second error you mentioned is most likely related to a app registration missing permissions, yes.

Thank you for very fast reaction
Unfortunately I’m not able to install the preview version:
dotnet add package AuthOida.Microsoft.Identity.Groups --version 1.26.0-preview.86
Determining projects to restore...
Writing C:\Users\PRO\AppData\Local\Temp\tmp1301.tmp
info : X.509 certificate chain validation will use the default trust store selected by .NET.
info : X.509 certificate chain validation will use the default trust store selected by .NET.
info : Adding PackageReference for package 'AuthOida.Microsoft.Identity.Groups' into project 'F:\Repos\GDPR\GR1.GDPR.WebSites\GR1.GDPR.WebSites.csproj'.
info : Restoring packages for F:\Repos\GDPR\GR1.GDPR.WebSites\GR1.GDPR.WebSites.csproj...
info :   CACHE https://api.nuget.org/v3-flatcontainer/authoida.microsoft.identity.groups/index.json
error: NU1102: Unable to find package AuthOida.Microsoft.Identity.Groups with version (>= 1.26.0-preview.86)
error:   - Found 10 version(s) in nuget.org [ Nearest version: 1.25.3 ]
error:   - Found 0 version(s) in Microsoft Visual Studio Offline Packages
error:   - Found 0 version(s) in Local packages
error: Package 'AuthOida.Microsoft.Identity.Groups' is incompatible with 'all' frameworks in project '<project>.csproj'.
Med venlig hilsen / Best regards
Peter Rosenkilde Roed
GR-1
From: Peter Wurzinger @.>
Sent: Saturday, 11 March 2023 15.03
To: peterwurzinger/AuthOida @.>
Cc: gr1-pro @.>; Author @.>
Subject: Re: [peterwurzinger/AuthOida] Wrong accessibility? (Issue #6)
Thanks for reaching out.
Indeed, yes. I didn't want to invest time to test the versions starting with .NET 6 against a real life Azure AD, because I haven't had the need using this library myself anymore, and I wans't aware of that somebody is actively using it. So it became more like a showcase of new .NET or C# versions.
That issue is fixed in version 1.26, alongside some dependency updates:
https://github.com/peterwurzinger/AuthOida/pkgs/nuget/AuthOida.Microsoft.Identity.Groups/76618506
If you could try that and confirm it to be working, I would then release it.
The second error you mentioned is most likely related to a app registration missing permissions, yes.
—
Reply to this email directly, view it on GitHub https://github.com/peterwurzinger/AuthOida/issues/6#issuecomment-1464918673 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A6M6T7ACEBKXODA5OD434BLW3SAZDANCNFSM6AAAAAAVWR5HF4 .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/A6M6T7G26P6MLZ2M3BDTZKDW3SAZDA5CNFSM6AAAAAAVWR5HF6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSXKDRJC.gif Message ID: @.*** @.***> >
--
This email has been checked for viruses by Avast antivirus software.
www.avast.com

Hi @gr1-pro,
just wanted to check in if you had the time to test the prerelease packages, and if I can release them into the wild as stable packages?
Cheers

Hello
I answered on 13/3 but it seems that you did not get my reply:
Thank you for very fast reaction
Unfortunately I’m not able to install the preview version:
dotnet add package AuthOida.Microsoft.Identity.Groups --version 1.26.0-preview.86
Determining projects to restore...
Writing C:\Users\PRO\AppData\Local\Temp\tmp1301.tmp
info : X.509 certificate chain validation will use the default trust store selected by .NET.
info : X.509 certificate chain validation will use the default trust store selected by .NET.
info : Adding PackageReference for package 'AuthOida.Microsoft.Identity.Groups' into project 'F:\Repos\GDPR\GR1.GDPR.WebSites\GR1.GDPR.WebSites.csproj'.
info : Restoring packages for F:\Repos\GDPR\GR1.GDPR.WebSites\GR1.GDPR.WebSites.csproj...
info :   CACHE https://api.nuget.org/v3-flatcontainer/authoida.microsoft.identity.groups/index.json
error: NU1102: Unable to find package AuthOida.Microsoft.Identity.Groups with version (>= 1.26.0-preview.86)
error:   - Found 10 version(s) in nuget.org [ Nearest version: 1.25.3 ]
error:   - Found 0 version(s) in Microsoft Visual Studio Offline Packages
error:   - Found 0 version(s) in Local packages
error: Package 'AuthOida.Microsoft.Identity.Groups' is incompatible with 'all' frameworks in project '<project>.csproj'.
Med venlig hilsen / Best regards
Peter Rosenkilde Roed
GR-1
From: Peter Wurzinger @.***
Sent: Saturday, 18 March 2023 16.33
To: peterwurzinger/AuthOida @.>
Cc: gr1-pro @.>; Mention @.***>
Subject: Re: [peterwurzinger/AuthOida] Wrong accessibility? (Issue #6)
Hi @gr1-pro https://github.com/gr1-pro ,
just wanted to check in if you had the time to test the prerelease packages, and if I can release them into the wild as stable packages?
Cheers
—
Reply to this email directly, view it on GitHub https://github.com/peterwurzinger/AuthOida/issues/6#issuecomment-1474879718 , or unsubscribe https://github.com/notifications/unsubscribe-auth/A6M6T7EA3KBV5VZP2DU56NLW4XISTANCNFSM6AAAAAAVWR5HF4 .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/A6M6T7GMWTVA7ZKLLO73KG3W4XISTA5CNFSM6AAAAAAVWR5HF6WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTSX5DQOM.gif Message ID: @.*** @.***> >

According to the error message you're seeing, I would assume, that you haven't added the Prerelease-Feed hosted on Github as NuGet source.
Please see this article for details:
https://docs.github.com/en/packages/working-with-a-github-packages-registry/working-with-the-nuget-registry#authenticating-with-a-personal-access-token

@gr1-pro If it is still relevant to you, I just released 1.26.0, that should cover this fix.
It will likely also be the last version for the 1.x versions of Microsoft.Identity.Web


issue in profile optimization script

Description:

Steps to reproduce:
get a all in one pack.
Start the any profile using below command
sh <PRODUCT_HOME>/bin/wso2server.sh --optimize -Dprofile=

Affected Product Version:

2.5.0
Environment details (with versions):

OS: mac os catalina
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Same issue reported in https://github.com/wso2/product-apim/issues/4148


Links in a slide are not clickable when 'fade: true'

Links in a slide are not clickable when fade is true, even tried touchMove set to false
const settings = {
      slidesToShow: 1,
      speed: 500,
      autoplay: false,
      swipe: false,
      fade: true,
      touchMove: false,
};

Any workaround?

it may happen because fade only affects opacity. Slides may be on top of your frame but invisible.
it affected me while I was using pinch and zoom gestures.
My solution was to add pointer-events: none to .slick-slide
and pointer-events: unset !important to .slick-current

my solution is similar to @outerlook. But .slick-currentis set for only one slide and it might be problematic when you have multiple slides to show
This is how I solved it.
  .slick-slide:not(.slick-active) {
    pointer-events: none;
  }

Having the same issue here. @outerlook thanks for a tip, should work.

Having the same issue here. @outerlook thanks for a tip, should work.

Seems that the @outerlook  solution only make the first slide clickable

for me helped z-index=10 for active and -10 for non active slider


KHR_techniques_webgl missing after updating model from gltf v1.0 to gltf v2.0

I tried to update some gltf v1.0 models to v2.0 using this nice pipeline.
If I understood correctly, this pipeline does so by using KHR_techniques_webgl, which still seems to have draft status according the readme.
The validator as well as cesium complain about the not yet supported gltf extension, see:

Will the issue on the cesium side be solved, once this is merged?
https://github.com/AnalyticalGraphicsInc/cesium/pull/6805
What about the validator?
Have a nice weekend

Yeah Cesium will be able to load those models once https://github.com/AnalyticalGraphicsInc/cesium/pull/6805 is merged. I'm not sure what the plan is for the validator or other engines.

@lilleyse now that #6805 has been merged, do you know whether there's an easy way to preview glTF 2.0 KHR_techniques_webgl in Cesium? I'm testing https://www.virtualgis.io/gltfviewer/ but getting:

RuntimeError: Unsupported glTF Extension: KHR_techniques_webgl

I also get an error in VSCode.

@emackey do you know what would need to be fixed from the VSCode side?
I can't think of other viewers out there that are up to date.

Is there a sample model that shows the problem?  I tried running CesiumAir.gltf through the pipeline and the result seems to work in VSCode's Cesium preview.  (It almost works in the ThreeJS preview, but the textures seem to be missing).
Also, click the "Extensions" button in VSCode, and check that glTF-Tools says version 2.1.17 or higher.  If not you will need to update VSCode and/or the extension.

Actually the screenshot above looks like version 2.1.14.  You'll need to update VSCode and then update the extension.

I'm testing the VSCode extension 2.1.17 with Cesium, and https://www.virtualgis.io/gltfviewer/. The only result I see in VSCode is this:

I'm not sure how to access the script error, though. Let me see if I can share the model.

I can't share the model publicly, but emailed you a copy. Thanks!

Has this been solved already? Thanks

@katSchmid I think so, thanks for the ping


deps: update to<EMAIL_ADDRESS>

Due to changes in Zeebe latest alpha, zeebe-node dependency needs to be updated.

That's because there were breaking changes in alpha 7: https://github.com/camunda-cloud/zeebe/releases/tag/1.0.0-alpha7

Tested with alpha7 (I need this anyhow for my other PR regarding E.T.) and it works:


[DO-NOT-MERGE][DEMO] Add a new incompatible api field for showcasing the changes

Add a new incompatible api field for showcasing the changes

/title: [DO-NOT-MERGE][DEMO] Add a new incompatible api field for showcasing the changes

/test pull-kubevirt-unit-test

/test pull-kubevirt-unit-test

/close


Corrected code samples

Category

[x] Content fix

Related issues

fixes #8377

What's in this Pull Request?
Corrected code samples to include missing quotes.

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

File
Status
Preview URL
Details

docs/sp-add-ins/working-with-folders-and-files-with-rest.md
:white_check_mark:Succeeded
View

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


liveness / readiness mixup

Hello @ohadschn ... We've discussed this and decided that the current definitions are correct. See :point_right: https://github.com/dotnet/AspNetCore.Docs/pull/18266


Add GitHub actions for lint and tests

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


performance issue on Raspberry 3A+ or 4B, "capture overrun occurred" and "playback underrun occurred" printed

As a newcomer to noise cancellation, I've encountered some issues while using your impressive project. Specifically, I've noticed frequent occurrences of "capture overrun" and "playback underrun" messages during processing.
I understand that it has been some time since you completed this project, but I am still hopeful for your kindly help.
Following is my environment setup:

Raspberry Pi 3A+ (and also Raspberry Pi 4B 2GB)
OS: raspbian-2020-02-14, latest distribution since the old distributions were out of maintenace
2 * I2S micrphones, same model and wire connected, installed driver version: 1:2.20.1
CMake: 3.16.3
libasound2 and libasound2-dev are installed, version: 1.1.8-1+rpt2
test app: ffANC

The first issue I met is the minimum period size of device is much larger than the value in constants.h, the log shows:
pi@raspberrypi:~/works/RpiANC/build $ ./ffANC
Capture params: Capture rate: 44100 Period size: 32 Min period size: 32
Playback params: Playback rate: 44100 Period size: 444 Play min period size: 443 Buffer size: 4096
access type = RW_INTERLEAVED
subformat = 'STD' (Standard)
channels = 2
rate = 44100 bps
period time = 10068 us
period size = 444 frames
period size = 3552 bytes
buffer time = 10068 us
buffer size = 4096 frames
periods per buffer = 9 frames
 min period size = 444 frames
 min buffer size = 4096 frames
exact rate = 44101/1 bps
significant bits = 32
is batch = 0
is block transfer = 1
is double = 0
is half duplex = 0
is joint duplex = 0
can overrange = 0
can mmap = 0
can pause = 0
can resume = 0
can sync start = 0
Number of play frames per period ( 444 ) then configuration ( 32 )

It passes after I modified the value to 444 on both capture and playback, but I don't know if this modification is correct.
The second issue is the debug message prints frequently, it seems reporting the performance down:
pi@raspberrypi:~/works/RpiANC/build $ ./ffANC
Capture params: Capture rate: 44100 Period size: 444 Min period size: 32
Playback params: Playback rate: 44101 Period size: 444 Play min period size: 443 Buffer size: 31080
access type = RW_INTERLEAVED
subformat = 'STD' (Standard)
channels = 2
rate = 44101 bps
period time = 10068 us
period size = 444 frames
period size = 3552 bytes
buffer time = 10068 us
buffer size = 31080 frames
periods per buffer = 69 frames
 min period size = 444 frames
 min buffer size = 31080 frames
exact rate = 44101/1 bps
significant bits = 32
is batch = 0
is block transfer = 1
is double = 0
is half duplex = 0
is joint duplex = 0
can overrange = 0
can mmap = 0
can pause = 0
can resume = 0
can sync start = 0
playback underrun occurred
playback underrun occurred
playback underrun occurred
playback underrun occurred
capture overrun occurred
playback underrun occurred
playback underrun occurred
playback underrun occurred
playback underrun occurred
capture overrun occurred
playback underrun occurred
playback underrun occurred
playback underrun occurred
playback underrun occurred
playback underrun occurred
capture overrun occurred
playback underrun occurred

repeating...

playback underrun occurred
Cap time
Average: 8650.68
proc time
Average: 58917.6
play time
Average: 390.66
total time
Average: 63423.4
ffANC: /home/pi/works/RpiANC/Sources/common.cpp:22: void save_vector_to_file(const string&, const std::vector<long int>&): Assertion `file.is_open()' failed.
Aborted

During processing, htop shows two cores are about 60% to 80% usage, another two are in low usage. (seems OpenMP works fine).
And the log shows the processing time is much more than required: 58.9ms vs. 10ms(444 samples in 44.1Khz)
I would greatly appreciate your help in resolving these issues. Thank you!

Hello! Thanks for your interest in that project!
For the first problem, yes, the minimum period size is hardcoded in the driver for the audio devices. But it's possible to modify driver's source coude, lower the minimum value, recompile and use your version.
Could you also check the previous issues, maybe you would find useful information there?
For example:

https://github.com/psykulsk/RpiANC/issues/1
https://github.com/psykulsk/RpiANC/issues/7

Hope that will help a little!


Wrong SKUs listed

In This feature is available for the following SKUs:, you mention non-zone SKUs are supported.
The gateway private IP VNG supports non-zone SKUs, but the connection requires a zoned SKU (step 4).
Thanks

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

ID: 56e6b95b-6755-01d7-6e2d-72dfeb474a1a
Version Independent ID: ef384f98-fe94-688e-12b2-841bb404cdbe
Content: Site-to-Site VPN connections over ExpressRoute private peering - Azure VPN Gateway
Content Source: articles/vpn-gateway/site-to-site-vpn-private-peering.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@woter1832 , thank you for your feedback. We'll review this and get back to you shortly!

@woter1832 , are you getting any error while adding the connection? If yes, could you please share the screenshot of the error or the issue?

@woter1832 , We have assigned this to the content owner/author for review.
@cherylmc , could you please validate the SKUs listed as on Azure Portal, we can see that the connection mentions the below:
Checking this option allows you to use Azure private IPs to establish an IPsec VPN connection. Support for private IPs must be set on the VPN gateway for this option to work. It is only supported on AZ SKUs.

Thanks! We'll take a doc bug on this!

#assign:aybatra

#assign:deeharris-23

Fixed and will publish in the upcoming week. #please-close

#please-close


Idea: Add Support for Pathfinder Tokens: Bestiaries (Foundry VTT)

You can purchase the Bestiary Images separately from Paizo.
https://paizo.com/products/btq02eat?Pathfinder-Tokens-Bestiaries
The naming convention is as:
pf2e-tokens-bestiaries\tokens\aberrant\humanoid\Doppleganger.webp
pf2e-tokens-bestiaries\tokens\elemental\fire\Ember-fox.webp.
Example Names

Idea:
Add an optional handlebar file that can be selected for the bestiary that adds:

token: monster-name.webp
Users would need to purchase the pack from Paizo and copy the content into their vault.
Note: There are 2 variations of images that come with the pack. Rings and Portraits. The naming convention is the same so the user would need to only copy the folder with the images they want to use as copying both folders would cause a conflict.

Here is a copy of the handlebar that will handle the images.
Adds this:
token: "[[{{ more-handlebars-helpers-replace name (more-handlebars-helpers-regexp ' ') '-' }}.webp]]"
Known Issues: There are monsters this wont work for. The Spell casting dragons for example. There are no individual images for these. This will work with the official Bestiary pack. The battlezoo images use underscore (_) instead of dashes (-) between each word in a monster name. This only works with dashes (-). I would recommend renaming the battlezoo images when they are copied into the vault to use dashes (-) so that a common format is maintained across the vault.
monster_handlebar_img.md

This is a cool idea.  I'll look into adding an option to export the tokens if you own them.

Do you think it would make sense for the module detect if an custom token image is owned and to export the referenced .webp image as well into a tokens folder as part of the .zip file, or just add the "token" text reference line to the Markdown file and it is assumed the user will manually copy the images to their Vault?
Based on how each monster references the tokens, it appears that it should be possible for the handlebar to detect which monsters have images supplied by a module (such as the bestiary tokens module) vs. those that are using the default system tokens.   For example, I own the Abomination Vaults and the Beginner Box modules, and when I have those imported into a world, any monsters in the Bestiary that have tokens provided by these modules have a custom texture.src value that starts with "module/..." rather than "system/..."   So, the handlebar can check if the texture.src starts with "modules/" and include the token line, otherwise omit it.

Release v0.6.0 now supports an option (off by default) which will detect non-system-default monster images and will export them.  This will require you have the custom images installed in the world where you are doing the export.
NOTE: This will work with an incomplete set of images that are supplied by any module(s).  For example, I own the Beginner's Box and Abomination Vaults modules and if I enable these modules in my world and do an export with this option on, all of my monsters that have custom artwork provided by these modules will that artwork exported.

One further note - this change is entirely optional (which is why I made it of by default).  If you do not want to have this plugin export the images and want to use the approach listed by @ObsidianTTRPGProject you would instead maintain your own copy of all the images in your vault and use the custom monster handlebar he provided earlier in this thread.


Issue 40012: Query webpart saved charts not rendering in custom view

Rationale
Issue 40012 identified a scenario where our async rendering of webparts would intermittently fail to render properly resulting in a broken user experience.

I've added a new taglib <labkey:loadClientDependencies> that is intended to supplant usages of <labkey:scriptDependency> which was deprecated in 18.1. With this change we'll no longer have to worry about dependencies loading out of order.
Changes

Introduce new <labkey:loadClientDependencies> tag to our tag library. See classes documentation for intended use.
Remove <labkey:scriptDependency/> tag as this leads to erroneous behavior when global callbacks are not supplied. Additionally, the global callback pattern made it difficult to work with in situations where multiple of the same webpart are rendered to a single page.

TODO

I still need to validate the handling of .css dependencies to ensure they load correctly.
Confirm with @labkey-adam non-usage of HtmlString.

There appears to be a superfluous  usage of <labkey:scriptDependency/> in one of our client modules. Triaging.

Thank you for getting rid of ScriptDependenciesTag (and for correcting spelling on loadClientDependencies in the Rationale :)). As for the HtmlString todo, I'm not sure what the question is, but here's a stab at an answer or two. We assume that our tags receive unencoded parameters and the underlying code knows how to encode everything properly; this is the opposite of JSP expressions. I suspect your print(String) is giving you red squiggles. You could use HtmlString internally, but that's not really correct since you're generating JavaScript. (Ideally, we'd have JavaScriptString or something that knows that encoding.) I'll make the code with some other comments. Let me know if you had a different question in mind...

Thank you for getting rid of ScriptDependenciesTag (and for correcting spelling on loadClientDependencies in the Rationale :)). As for the HtmlString todo, I'm not sure what the question is, but here's a stab at an answer or two. We assume that our tags receive unencoded parameters and the underlying code knows how to encode everything properly; this is the opposite of JSP expressions. I suspect your print(String) is giving you red squiggles. You could use HtmlString internally, but that's not really correct since you're generating JavaScript. (Ideally, we'd have JavaScriptString or something that knows that encoding.) I'll make the code with some other comments. Let me know if you had a different question in mind...

Yes, I wasn't sure on our handling of today's unsafe indicators so I appreciate you taking a look. The passing in of a CharSequence resolved that (by obfuscation?). Using PageFlowUtil.jsString() on user input is good for now in lieu of having a JavaScriptString.


Start with empty input.

If the input is empty the select object is null and return a TypeError. So just added a check to ensure that select is not null.

Thanks ! Could you add this to pickATime directive too ?

Done with pleasure.

Thanks, i've merged manually to keep a single commit.

And released as 0.1.5.


Any plan to upgrade tf to 2x?

as the title said.

No, our plan is to migrate to PyTorch

Is the plan in motion? We are using DeepPavlov for a project, and it is currently seriously breaking our dependencies with numpy==1.18 and tf 1.15 - we need to maintain two environments with different versions, which is a disaster waiting to happen...

Hi! Our plan is to migrate from TF 1.x to PyTorch, and abandon TF 1.x entirely. We plan to bring the majority of updates to our models in our upcoming releases, 0.16 (July 2021) and 0.17 (August/September 2021), respectively.
Thanks!
From: Jan Novotný @.>
Sent: Friday, July 9, 2021 4:26 PM
To: deepmipt/DeepPavlov @.>
Cc: Daniel Kornev @.>; Comment @.>
Subject: Re: [deepmipt/DeepPavlov] Any plan to upgrade tf to 2x? (#1457)
Is the plan in motion? We are using DeepPavlov for a project, and it is currently seriously breaking our dependencies with numpy==1.18 and tf 1.15 - we need to maintain two environments with different versions, which is a disaster waiting to happen...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/deepmipt/DeepPavlov/issues/1457#issuecomment-877186405, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABIFFK35OEM4K2WA6LE25WDTW32HRANCNFSM46NGLJFQ.

Hi, @novotnej
1.0.0rc0 pre-release with more free requirements is already available.


xArchive test throws exception for locked files.

Recent versions of xArchive started to throw exceptions for locked files. I assume that this is because more strict checksum checks. As a result, we can't really use it for certain scenarios.
Error message:
PowerShell DSC resource MSFT_xArchive  failed to execute Test-TargetResource functionality with error message: System.InvalidOperationException: An error occurred while opening the file C:\Program Files\Net-SNMP\bin\msvcr100.dll on disk. Please examine the inner exception for details ---> System.Management.Automation.MethodInvocationException: Exception calling ".ctor" with "2" argument(s): "The process cannot access the file 'C:\Program Files\Net-SNMP\bin\msvcr100.dll' because it is being used by another process." ---> System.IO.IOException: The process cannot access the file 'C:\Program Files\Net-SNMP\bin\msvcr100.dll' because it is being used by another process.
I'm really not sure if there is an easy fix for that one. Perhaps changing the way xArchive is reading files on disk would help, or givin an option to soften the requirement (e.g. if file can't be accessed, just ignore it). Else we end up in situation when files important from our point of view are not touched/fixed.

softening the requirement is an issue as DSC is about idempotency..means I can run this script a million times and always get the same state. I think failure is actually what I expect it to do and then error out.
You can add a file resource to test or use script resource to find locked files and release prior to archive but I dont think its the archive resource itself that needs to account for it.  just my 0.5$  :)

Throwing exceptions from test has nothing to do with idempotency. Test/Get should give you as much information as possible and if it's not possible, worse I would expect is $false. If I modify 100 files, none will be updated because file that matches the source is locked - where is idempotency here?


Nominate @zxbyoyoyo as Approver

@zxbyoyoyo has been continuously working on multiple KubeVela's core features for a long time, including

https://github.com/kubevela/kubevela/pull/5812
https://github.com/kubevela/kubevela/pull/5804
https://github.com/kubevela/kubevela/pull/4874
https://github.com/kubevela/kubevela/pull/3468
https://github.com/kubevela/kubevela/pull/3136
https://github.com/kubevela/kubevela/pull/2831
https://github.com/kubevela/kubevela/pull/2758
https://github.com/kubevela/kubevela/pull/2731
https://github.com/kubevela/kubevela/pull/2703

and many others.
He has also been participating in a wide range of discussions around the core capabilities of KubeVela, such as

https://github.com/kubevela/kubevela/issues/5637
https://github.com/kubevela/kubevela/issues/5091
https://github.com/kubevela/kubevela/issues/4244
https://github.com/kubevela/kubevela/issues/5457
https://github.com/kubevela/kubevela/issues/4757
https://github.com/kubevela/kubevela/issues/4702
https://github.com/kubevela/kubevela/issues/3463

As an engineer in the automobile company, he also makes efforts to integrate KubeVela with GitOps and provides CD capabilities for upper layer users. With great appreciation for his contribution, I would like to nominate @zxbyoyoyo to be one of the approvers of the KubeVela community and look forward to more open communications and collaborations.

+1 thanks to @zxbyoyoyo  for so many contributions!

thank you for your recognition, I will continue to participate in kubevela's capacity building

+1 @zxbyoyoyo is a good contributor.

+1

ping @jefree-cat @anoop2811 @dhiguero

+1

LGTM! Thanks for the contributing.

The nomination is accepted. @zxbyoyoyo You can raise a pull request to update the owner file and link it to this issue.


Clarification wished for middleware (best practices)

Hi James,
I haven't played yet with duct, but after reading watching the talk and reading the source, I am utterly seduced. Well done, @weavejester
I have a remaining question regarding best practices:
If a particular middleware makes only sense in a particular endpoint, should the user embed it in the endpoint, or should he pass it along with other middleware directly to the handler? What are the implications of both approaches?
Thanks!
@danielsz

Thanks for the praise.
I'll add a wiki page to clarify this, but in a nutshell the handler middleware is for the entire handler, and middleware only for the endpoint should be applied in the endpoint.
The reason the handler middleware is configurable is that it's often the case that handler middleware differs depending on the environment. For example, in development you might add middleware to display stacktraces, while in production you might have middleware that hides stacktraces and logs them.

Thank you for clarifying this. This is very helpful.

The wiki page is very welcome at the moment I have trouble figuring out how it works exactly compared to the simple functions that are used in a classic handler.

I have trouble figuring out how it works exactly compared to the simple functions that are used in a classic handler.

Do you mean you have problems deciding when to use middleware on endpoints, or that you don't know how to use middleware on endpoints?

I put all my middleware in the handler for now, so my problem is mostly how to describe it in new-handler. Why does it change the way you define it in a vanilla ring app ?
I'm not sure I understand what this does :
  (if (vector? middleware)
    (let [[f & keys] middleware
          arguments  (map #(get component %) keys)]
      #(apply f % arguments))
    middleware))

(defn- compose-middleware [{:keys [middleware] :as component}]
  (->> (reverse middleware)
       (map #(middleware-fn component %))
       (apply comp identity)))```

Does the reverse mean I should write them upside down ?

Before adopting duct my middleware was declared like this :

```(def app
  (-> app-routes
      (wrap-resource "public")
      auth
      (wrap-cors :access-control-allow-origin #".*"
                 :access-control-allow-methods [:get :put :post]
                 :access-control-allow-headers ["Content-Type"])
      wrap-restful-format
      (wrap-defaults middleware)))```

Now I have it like this :

(def my-defaults (-> site-defaults
(assoc-in [:static :resources] "/")
(assoc-in [:security :anti-forgery] false)))
(defn auth [app-routes]
(friend/authenticate app-routes
{:credential-fn (partial creds/bcrypt-credential-fn users)
:workflows [(workflows/interactive-form)]}))
(defn my-wrap-cors [handler]
(wrap-cors
handler
:access-control-allow-origin #".*"
:access-control-allow-methods [:get :put :post]
:access-control-allow-headers ["Content-Type"]))
(new-handler {:middleware [[wrap-resource :dir]
auth
my-wrap-cors
wrap-restful-format
[wrap-defaults :defaults]]
:dir "public"
:defaults my-defaults})```

Ok I ended up writing my handler this way after  following this page of the wiki https://github.com/weavejester/duct/wiki/Components .

(new-handler {:middleware [#(wrap-resource % "public")
                                       auth
                                       #(wrap-cors %
                                         :access-control-allow-origin #".*"
                                         :access-control-allow-methods [:get :put :post]
                                         :access-control-allow-headers ["Content-Type"])
                                       wrap-restful-format
                                       #(wrap-defaults % my-defaults)]})

I am still wondering if the reverse in compose-middleware means that the last listed middleware is applied first.

I don't quite understand what you're having problems with. Maybe some more examples would make it clearer?
Normally you might apply middleware to a handler like so:
(-> handler
    (wrap-foo {:x 1})
    (wrap-bar {:y 2}))

The :middleware key works in a similar fashion:
(new-handler
 {:middleware
  [#(wrap-foo % {:x 1})
   #(wrap-bar % {:y 2})]})

Notice that we have a vector of middleware functions, and these are applied in the order they appear.
However, we can also pull the options out of the functions:
(new-handler
 {:middleware [[wrap-foo :foo]
               [wrap-bar :bar]]
  :foo {:x 1}
  :bar {:y 1}})

This means the same as the two previous examples. The keywords in the vectors are substituted for their corresponding values.
The advantage of this approach is that this allows the configuration to easily affect how the handler middleware behaves. For example, you might want to print stacktraces in development, but hide them in production.
Does that make things clearer? If so I'll update the wiki page with these examples.

I was confused by the use of the reverse function in compose-middleware regarding the order in which the middlewares are applied.
Your exemples could ease comprehension for newcomers it might indeed be a nice addition to the wiki page.

The reverse is just because comp applies functions from last to first, whereas I want the middleware to be applied from first to last.

Thanks for the clarification I did not know about comp behavior

I haven't come across any use-cases where it makes sense for the endpoint middleware to be configurable in the same way. At the very least I suspect it's a rare enough use-case to be handled manually, with some custom endpoint code.

I came across this issue while looking for a way to do exactly this: configure middleware on an endpoint. My use case is a webhook endpoint, which among other things needs to verify that a specific header value (an auth token) is present before processing the hook. That check, which needs to be configured with a valid-token value, seems to be a good use case for middleware. I don't want this check in place on other endpoints, and it doesn't make sense to have the middleware be aware of my application's routing in order to decide on its own when it should or shouldn't apply.
Can you think of a better approach that I haven't considered here?
:heart: for duct so far, though, it's exactly the lightweight set of defaults I was looking for!

The way I'd solve this is to create a bare map component that acts as an additional set of configuration. So something like:
(defn new-system [config]
  (let [config (meta-merge base-config config)]
    (-> (component/system-map
         :app  (handler-component (:app config))
         :http (jetty-server (:http config))
         :foo  (endpoint-component foo-endpoint)
         :auth {:auth-token (:auth-token config)})
        (component/system-using
         {:http [:app]
          :app  [:foo]
          :foo  [:auth]}))))

Then in your endpoint, you get the configuration:
(defn foo-endpoint [{:keys [auth-token]} :auth}]
  (-> (routes ...)
      (wrap-auth {:token auth-token})))

:star: that works very nicely, I hadn't realised that you could use a map as a component! Thanks @weavejester


Request decorator

Pull Request summary:
TLDR;
This PR introduces @request decorator which is used to define endpoint handlers.
Before:
def find_status(self, task_id: str) -> TaskData:
    url = f"/device/action/status/{task_id}"
    return self._get(url).dataobj(TaskData, None)

After:
@request(get, "/device/action/status/{task_id}")
def find_status(self, task_id: str) -> TaskData:
    ...

Benefit?

less code, less errors
documentation automatically generated with pre-commit: ENDPOINTS.md
no unittests needed for endpoints defined with decorator (method body predefined and unittested against all supported parameters, decorator and annotated method are checked during pre-commit)

Description of changes:
Decorator inspects decorated method signature and arguments to specify payload and return type and to build request according to declaration. This decorator automatically builds a request during runtime so no method body is needed.
Various types of return values and payload are supported (Including CustomPayloadType)
When new endpoint is added in code, documentation is automatically generated by pre-commit in ENDPOINTS.md file. All existing APIEndpoints are migrated to use @request decorator in this PR.
@request decorator is implemented in: vmngclient/endpoints/__init__.py
To see all possible use (and misuse) cases check: vmngclient/tests/test_endpoints.py
Additionaly in this PR term "primitives" was replaced to "endpoints" in whole repository as it caused confusion.
Full example of usage:
# 1. Define endpoint using request decorator:
from pydantic import BaseModel, Field
from typing import List
from vmngclient.endpoints import APIEndpoints, request, versions, view, delete
from vmngclient.utils.session_type import ProviderView

class TenantBulkDeleteRequest(BaseModel):
    password: str
    tenant_id_list: List[str] = Field(alias="tenantIdList")

class TenantTaskId(BaseModel):
    id: str

class TenantManagement(APIEndpoints):
    @versions(">=20.4")
    @view({ProviderView})
    @request(delete, "/tenant/bulk/async")
    def delete_tenant_async_bulk(
        self, payload: TenantBulkDeleteRequest
    ) -> TenantTaskId:
        ...

# 2. Login to vManage with provider credentials
from vmngclient.session import create_vManageSession

session = create_vManageSession(
    url="example.com", username="provider", password="password123"
)

# 3. Instantiate endpoints class using session and send request
api = TenantManagement(session)
api.delete_tenant_async_bulk(
    TenantBulkDeleteRequest(password=session.password, tenantIdList=[])
)

Checklist:

[x] Make sure to run pre-commit before commiting changes
[x] Make sure all checks have passed
[x] PR description is clear and comprehensive
[x] Mentioned the issue that this PR solves (if applicable)
[x] Make sure you test the changes

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/305


Task/stable/fact 1600/update testrail and code style on the last of the cli options tests

Updated the last of the command line option tests with TestRail numbers, risk tags, and code style
Moved the --puppet option test into the options directory and updated it
Split the 3 no-cache option tests into separate files
tested on:
arista4-32a, ciscoxr-64a%7Bhypervisor=vmpooler%7D, cisconx-64a%7Bhypervisor=vmpooler%7D, cumulus25-64a, debian8-64a, fedora25-64a, osx1012-64a, redhat7-64a, sles12-64a, solaris11-64a, ubuntu1604-64a, windows10ent-64a, windows2016-64a

CLA signed by all contributors.

👍 I like the use of facter_output. Ship it.


Fix treasury kept and spend when emptied

Fix https://github.com/paritytech/substrate/issues/3879
Superseed https://github.com/paritytech/substrate/pull/3797
Done:

treasury account is created at genesis config.
pot doesn't include existential deposit thus pot can be entirely spend with no issue.

TODO:

[ ] update polkadot: just includes Config in construct_runtime
[x] fix some tests using runtime genesis config

Alternatives:

we could also keep old behavior with treasury being created when it happens that dilution is more than ED. but seem better that we create it once and for all.

Notes:

I still kept resolve_creating to ensure it still works on Kusama if there is no treasury account then it will just get created once on_dilution mint more than ED instead of never be created.
Also in case someone forget to include Config in construct runtime (thus account is not created) then this account can be created later as well.

I would very suspect that treasury does exist already in kusama, but anyway it is fine. and I added both tests.
another solution would be to lazily create the account if it happens that it doesn't exist but whatever is OK

test are failing at least because of some CI issue, let's retry them


added code to expose c_api_pred_contrib in the R package

Works as expected on my machine. I'm not too experienced with C++, but I can verify that the contributions are returning as expected on my toy dataset. I didn't touch too much of the code. Please review!

 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.:x: Zelazny7 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


chore(manifests): Remove configuration objects from manifests

Description of your changes:
Removes configuration objects (ConfigMaps and DB/S3 Default Credential Secrets) from manifests, as it makes managing external services impossible when managed by ODH Operator (operator would overwrite any changes made to these files)
Also provides a new 'init' folder with some Templates that will allow a developer or user to initialize these CMs and Secrets before deploying
Environment tested:

Python Version (use python --version):
Tekton Version (use tkn version):
Kubernetes Version (use kubectl version):
OS (e.g. from /etc/os-release):

Checklist:

[x] The title for your pull request (PR) should follow our title convention. Learn more about the pull request title convention used in this repository.

/lgtm

Closing this issue. Manual merge required.

Manually merged for @rimolive and @gmfrasca  as there are ci issues


Apt-get update goaccess configured multiple times?

Hi,
When doing apt-get update I get this in the log:
W: Target Packages (main/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:2 W: Target Packages (main/binary-all/Packages) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:2 W: Target Translations (main/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:2 W: Target Packages (main/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:3 W: Target Packages (main/binary-all/Packages) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:3 W: Target Translations (main/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:3 W: Target Packages (main/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:2 W: Target Packages (main/binary-all/Packages) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:2 W: Target Translations (main/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:2 W: Target Packages (main/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:3 W: Target Packages (main/binary-all/Packages) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:3 W: Target Translations (main/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list.d/goaccess.list:1 and /etc/apt/sources.list.d/goaccess.list:3
How do I fix this? Mostly it is annoying!

Have you tried any of these?

go to /etc/apt/sources.list.d/
and check the duplicate file/line and delete it.

Any updates on this?


Request: Paired-end function for bedtools coverage

Hi,
is there any change that you are working on a paired-end function for bedtools coverage if -b is a paired-end bam?
Currently, it treates each mate of a pair independeltly and does not consider TLEN for coverage calculation. Would be great!
best,
Alex

@ATpoint: Why did you close this issue? Is the feature there,not needed, or the request a duplicate?

No, but > 1 year old without reaction.

I still think it's a valid request that should not be forgotten. There simply are very pressing issues currently delaying the planned next release afaik, so I wouldn't consider this as something that will never happen.

Hello, would it be possible to reopen this issue?
I'd like to get paired-end "fragment coverage" histograms for specific genomic intervals. Essentially adding the genomecov -pc option to the coverage tool:
bedtools coverage -hist -pc -a intervals.bed -b paired_end_input.bam
Alternatively, a way of specifying a bed file with intervals of interest to genomecov would be useful.
Thanks!


A problem to driver MBI5124 Panel

Hi faptastic ,
When I use the library to driver MBI5124 Panel ,  the drawPixel function dosn't work well.
When I  call the function drawPixel(10,10,myRED), the panel display two lines  on the y position。

How to solve the problem。
Best wishes bo you!

I have tested more to solve the problem.
when I change the input for setBrightness8(20) function, the panel displayed changed, but other number for brightness is not well.


Fixing shift and exclusive mouse mode

Fixes an issue with shift not being called correctly on multiple windows. It now follows where your keyboard is focused. Also resets shift state when the focus is lost (will be gained again if still being held on return to window)
Fix multiple issues with exclusive mode:

If the window were to lose focus and you clicked in the title bar, the ClipCursor no longer was bound to the window, and the cursor was unhidden.
Unable to drag and move the window.
Unable to maximize/minimize the Window
Unable to close the window with the X button.
When mouse button is pushed down (not a click) on frame components, it would disappear.
Mouse could sometimes go out of the bounds of the view area into the close and minimize button resulting in issues with the window.

Removed the raw input requiring the view_hwnd, it just goes off of where your keyboard has focus.
Added a raise exception to show information on possible NULL issue with the _view_hwnd.
Was tested in Windows 10. Any feedback and additional testing appreciated.
Note:
There is still an issue with random hierarchy issues where the mouse can become visible again in exclusive mode, if the window rapidly loses/gains focus, but it seems pretty rare. It seems some libraries (SDL2) just have a schedule that checks every so often if the ClipCursor area was superseded by another window and then re-applies it. Not sure we need to go that intensive on it.

We probably don't need to get go as far as SDL just yet.
How do you feel about merging this in for now?

It's fine merging in as is. This does fix the shift issue, and numerous exclusive mode issues to make it more usable.
It's not a perfect solution but I think it's good enough for most uses.


Move or copy i18n messages without affecting Transifex

This PR closes #722, introducing the magic comment suggested in the issue. By specifying @transifexKey in a comment above an i18n message, a developer can indicate that the message should appear at a different key in Transifex than the default. This prevents a message from requiring retranslation. The comments in bin/util/transifex.js provide more details about how @transifexKey works.
I originally thought of @transifexKey as a way to move a message from one key to another. However, I realized that without much change, @transifexKey could also be used to copy a message from one component to another. @transifexKey can be thought of as a generic way to specify where in Transifex the translations for a particular message can be found — whether those translations are unique to a single component or should be copied to multiple components.
There are benefits to extending @transifexKey to copying, not just moving. Right now, the main way that we share messages between components is to add the messages to en.json5. However, en.json5 is part of the initial bundle, so any messages there are eagerly loaded, even if no components need them right away. Adding a message to en.json5 rather than a component in which it's used also removes the message from its context. For example, I'm about to create a component named EntityFeedEntry for the entity activity feed. The component will need to reuse the title.updateReviewState.approved messages from SubmissionFeedEntry:
https://github.com/getodk/central-frontend/blob/baea8a9c5afc550f6d8ef87a148f623484f141b8/src/components/submission/feed-entry.vue#L273-L289
However, moving those messages from SubmissionFeedEntry to en.json5 would be inconvenient. It would distance the messages from the messages for the other review states and make it harder for SubmissionFeedEntry to dynamically select the message based on the review state. Instead, after this PR, EntityFeedEntry will be able to use @transifexKey to copy the messages' translations without requiring retranslation or moving the messages to en.json5.
There will definitely still be plenty of cases in which adding shared messages to en.json5 is the appropriate choice. However, there will now be a different way for components to share messages for the times when that's useful.
What has been done to verify that this works as intended?
There are a few components that we've wanted to move but haven't because we've wanted to prevent retranslation. This PR moves those components, using @transifexKey to prevent retranslation. When I run restructure.js, the Transifex source file (transifex/strings_en.json) does not change, which is what we want. When I run destructure.js, it also does not change files.
I manually verified the following cases:

[x] Rename a component. Running restructure.js and destructure.js should have no effect.
[x] Move a message from a component to en.json5.
[x] Move a message from en.json5 to a component.
[x] Move a message from one component to another.
[x] Move a group (object) of messages.
[x] Copy (don't move) translations from one component to another. restructure.js should have no effect, but destructure.js should copy the translations to the component specifying @transifexKey.
If a message copies translations from another message, it must:

[x] Have the same English text as the original message (otherwise an error is thrown)
[x] Not have its own comments (other than @transifexKey)

[x] Copy a group of messages.
If multiple messages are moved to the same Transifex key:

[x] They must have the same English text.
[x] Only one can have a comment.

Why is this the best possible solution? Were any other approaches considered?
I think it's important for this process to be lightweight for developers, and I think a comment achieves that. We want to preserve translations and translation history in Transifex, but we also don't to make development slower or more cumbersome. @transifexKey can be specified for a group of messages, so multiple messages can be moved at once without a comment for each message.
The effects of a @transifexKey comment are easy to see in action and verify, because you can immediately run restructure.js and see whether/how transifex/strings_en.json changes.
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
This PR does not change the behavior of the app. The main risk is translations or translation comments breaking somehow. However, I didn't see any issues when I verified the scenarios above.
Does this change require updates to user documentation? If so, please file an issue here and include the link below.
No changes needed.
Before submitting this PR, please make sure you have:

[x] run npm run test and npm run lint and confirmed all checks still pass OR confirm CircleCI build passes
[x] verified that any code or assets from external sources are properly credited in comments or that everything is internally sourced

@ktuite, I've made a number of mostly smaller changes since our interactive review. I think most of them don't need additional review, but I wanted to highlight a few changes:

I renamed @transifexPath to @transifexKey. That was my original idea in #722 and matches the Transifex terminology. The code itself mostly still refers to "paths," which are arrays, not strings.
I moved most of the code related to @transifexKey to its own section in bin/util/transifex.js. I ended up thinking that it'd be nice to keep most of the related logic together. Even though I hadn't added a lot of logic to restructure.js and destructure.js, I think it's ideal for those scripts to be as small as possible. The scripts now call functions rather than implementing @transifexKey themselves.
Before, I validated that if two values (either messages or objects) specify the same @transifexKey, they have the same message text. I now also check that only one of the values specifies a comment for Transifex. That will prevent a conflict between the values' comments, resolving https://github.com/getodk/central-frontend/pull/784#discussion_r1171931815. For example, if component A copies translations from component B, then only component B can specify a comment for Transifex; component A cannot. In other words, it is only component B that is sourcing Transifex; component A has no effect on Transifex. This change is probably the one that would most benefit from review: c244b0b229fc4c8d0968d7903b22d5896c9d8189. I think that review could be done async, but let me know if you'd prefer to review it interactively.

I'm going to go ahead and merge so that I can continue making progress on the entity activity feed, but I can make any changes in a follow-up PR.


How to change identificationAttributeName?

I want to change username to email but do not know how to do it. Doing the following has no effect:
export default JWTAuthenticator.extend({
    identificationAttributeName: 'email',
});

Thank you for reporting, it's a bug.
Should work now, you were doing it correctly.
export default JWTAuthenticator.extend({
    identificationAttributeName: 'email',
    passwordAttributeName: 'pass'
});


chore: Widen ocaml dependency for esy & update deps

This widens the ocaml version in our package.json - It requires an older ocp-indent gets resolved, but that's only used by development tools.

Hmm, esy on Windows seems mad?

Looks to be an issue with js_of_ocaml, so I should probably restrict it.

@ospencer I'm restricting JSOO to v3 until we get the test failures on Windows figured out (ref https://github.com/ocsigen/js_of_ocaml/issues/1237)


Build/debug/etc. commands not resepecting the 'active project' when folder with multiple projects loaded

Here is a sample dir structure:
base/
sample1/
Program.cs
project.json
sample2/
Program.cs
project.json
Select 'sample1' as the running project.  Hit command+shift+p and run the build task.  VS Code will put the task.json in the root directory and then error saying it cannot find the project.json.
If you load sample1 or sample2 independently they work as expected.

This should probably be filed over on https://github.com/OmniSharp/omnisharp-vscode.

Duplicate to this: https://github.com/OmniSharp/omnisharp-roslyn/issues/469
In short, OmniSharp relies on global.json for search path. When global.json doesn't exist, it expect a project.json at current folder.

What about the following scenario:
app\global.json { "projects": ["src", "test"] }
app\src\sampleapp\project.json
app\src\sampleapp...
app\src\secondapp\project.json
app\src\secondapp...
app\test\sampleapp.test\project.json
app\test\sampleapp.test...
In this case, if I open app, I expect that the global.json will be handled correctly at the root.  But what if I open src (which is really the case above)?  If there were a global.json a directory above, does it fall within the context of 469?  This feels like another possibility.

@jeffschwMSFT that's still a vscode issue.  omnisharp-atom supports nested projects, I don't think VSCode does (today anyway).
One big negative is with #469 is that, if someone does something (not very smart) like OmniSharp.exe / that would be an entertaining experiement as they Enumerate all their drives.

Personally, I think we should search for any and all projects beneath the folder and add them to the same Roslyn workspace.


Security Bad Practice - Hardcoded Credentials in bloodhound.config.json

I consider it a very bad practice to hardcode any kind of credentials into config-files.
Maybe you can add a little entrypoint.sh wrapper, that retrieves credentials from environment variables, so that one can set them at runtime. I am no pro-developer, but something like this should do the job:
echo "{\"version\":1,\"bind_addr\":\"<IP_ADDRESS>:8080\",\"metrics_port\":\":2112\",\"root_url\":\"http://<IP_ADDRESS>:8080/\",\"work_dir\":\"/opt/bloodhound/work\",\"log_level\":\"${LOG_LEVEL:-INFO}\",\"log_path\":\"bloodhound.log\",\"features\":{\"enable_auth\":true},\"tls\":{\"cert_file\":\"\",\"key_file\":\"\"},\"database\":{\"connection\":\"user=${DB_USER:-bloodhound} password=${DB_PASSWORD:-bloodhoundcommunityedition} dbname=${DB_NAME:-bloodhound} host=${DB_HOST:-app-db}\"},\"neo4j\":{\"connection\":\"${NEO_CONNECTION_STRING:-neo4j://neo4j:bloodhoundcommunityedition@graph-db:7687/}\"},\"collectors_base_path\":\"/etc/bloodhound/collectors\"}" > /bloodhound.config.json
exec bloodhound -config /bloodhound.config.json

BTW: Thanks for your great work. Hopefully, we can make it even better.

Having creds stored in config files is a correct way to keep/store credentials.

How about instead of just saying "very bad practice" and having a whinge you demonstrate and explain why storing credentials in a config file is bad?

Also your proposed solutions to a problem and doesn't exist has multiple naïve errors.
If any of a environment variables contain JSON escape characters your bash script breaks imminently and would crash.

At minimum, we can provide additional documentation and justification for our practice here even if the risk is indeed low.
The intent of the issue appears genuine and requires an equally genuine response from the BloodHound team, even if we don't take on the author's suggested fixes.

Thanks for the additional information. It will help frame our internal discussion about this issue.


[LbryNet] add configuration magic via start.sh

config options in the .yml file should be able to be set via the docker-compose.yml file.

#61 closes this


Cannot specify build flags to detect correct file for rice embed-go

I have two files:
js_asset_prod.go:
// +build production

package server

import (
	"github.com/GeertJohan/go.rice"
)

var javascriptAssetRiceBox = rice.MustFindBox("./js/prod").HTTPBox() // <--- prod path

and js_asset_dev.go:
// +build !production

package server

import (
	"github.com/GeertJohan/go.rice"
)

var javascriptAssetRiceBox = rice.MustFindBox("./js/dev").HTTPBox() // <-- dev path

And when I run rice -v -i ./server/ embed-go I get output like:
2015/05/06 23:50:17 skipping file "~/server/js-dev.rice-box.go"
2015/05/06 23:50:17 scanning file "~/server/js_asset_dev.go"
2015/05/06 23:50:17 	found box "js/dev"
2015/05/06 23:50:17 scanning file "~/server/server.go"
2015/05/06 23:50:17
2015/05/06 23:50:17 embedding box 'js/dev'
2015/05/06 23:50:17 	to file js-dev.rice-box.go
2015/05/06 23:50:17 	includes dir: ''
2015/05/06 23:50:17
2015/05/06 23:50:17 rice finished successfully

It appears that rice does not recognize js_asset_prod.go because it has an opt-in build flag, and in go.rice/flags.go, it uses build.ImportDir which does not support adding build flags (and the rice CLI does not support adding build flags either).

There could be a new flag in the rice tool, --build-tag, that can be specified multiple times for multiple tags, and use ImportDir as a method from build.Context which we create with the provided --build-tag values. Would you be willing to create this solution and PR?


Helm Chart do not apply tag and variant on injected sidecars

Bug Description
Current helm chart do not apply tag & variant options to injected sidecars. Options are also not available on proxy & proxy_init sections.
global:
  istioNamespace: istio-system
  hub: sample.io/library
  tag: 1.16.2
  variant: "distroless"
  proxy:
    image: proxyv2
    autoInject: enabled
  proxy_init:
    image: proxyv2

The "latest" tag is applied to injected sidecars. However, hub is correctly configured
  istio-proxy:
    Container ID:
    Image:         sample.io/library/proxyv2:latest

Version
kubectl version : v1.23.8

Additional Information
No response

Here the istio configmap applied,
apiVersion: v1
kind: ConfigMap
metadata:
  name: istio
  namespace: istio-system
  labels:
    install.operator.istio.io/owning-resource: unknown
    istio.io/rev: default
    operator.istio.io/component: Pilot
    release: istio
data:
  meshNetworks: |-
    networks: {}
  mesh: |-
    defaultConfig:
      discoveryAddress: istiod.istio-system.svc:15012
      tracing:
        zipkin:
          address: zipkin.istio-system:9411
    enablePrometheusMerge: true
    rootNamespace: istio-system
    trustDomain: cluster.local

We are currently using 1.16.2-distroless version.

I cannot reproduce this with this demo.
can you share how your install steps?

Same way as your demo,
helm install istio-base istio/base -n istio-system
helm install istiod istio/istiod -n istio-system --wait -f ./values.yaml

However, our values.yaml file is customized with a custom image source (private repository, tag, variant, imagePullPolicy...).

the root cause is you render a wrong cm with unknown reason.
please check difference with john's reply https://github.com/istio/istio/issues/43629#issuecomment-1446959807.

I've updated our ConfigMap to match the configuration on john's reply https://github.com/istio/istio/issues/43629#issuecomment-1446959807.
Indeed the TAG is updated, however it is not the custom one configured in Helm Template (ex: v1.16.2) but latest-distroless
  istio-proxy:
    Container ID:
    Image:         sample.io/library/proxyv2:latest-distroless

I've updated our ConfigMap to match the configuration on john's reply #43629 (comment).
Indeed the TAG is updated, however it is not the custom one configured in Helm Template (ex: v1.16.2) but latest-distroless
  istio-proxy:
    Container ID:
    Image:         sample.io/library/proxyv2:latest-distroless

Have you set tag field as well? If set, I think it will be 1.16.2-distroless


Document YAML python.use_system_site_packages option

A user raised (https://github.com/rtfd/readthedocs.org/issues/4362#issuecomment-404766625) that it was hard to find this option
python:
   use_system_site_packages: true

I checked the docs and I didn't find anything about this: https://docs.readthedocs.io/en/latest/yaml-config.html
We need to extend our docs to mention this in the YAML config page.

ha, I thought the user check the option from the web interface. Also, I was documenting this as a new option for the v2


[WFLY-19610-33x] @PostConstruct on Servlet may be called twice

https://issues.redhat.com/browse/WFLY-19610
Back port of fix for WFLY-19610

/retest


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #530 (24db4bc) into main (5a69517) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #530   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

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


cron: Update WIP

Signed-off-by: Ce Gao<EMAIL_ADDRESS>

Hey @gaocegege,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: 42599360-0ffd-11e9-a071-cf0a0b6f72f6

ping @gaocegege

Yeah, it is.


disable pulling of empty .desktop files for Google Docs and Google Sheets

so i installed from master today using go get -u and made the drive init and tried to pull my entire gdrive which worked flawlesly but i noticed that every gdocs .desktop file comes with an empty file with the same name (but without fileneding)
i am curious if this is by design and if so how i can disable that?
running lubuntu 16.04

Hello there @yuri0r, thank you for the question and welcome to drive. My apologies for the late reply, I've been swamped.
We already have this feature that can be disable with -desktop-links=false and it is documented in the README at https://github.com/odeke-em/drive/blob/0b501d3138b75d59aafa5157c76c2aa164aa2e11/README.md#desktop-files
$ drive pull --desktop-links=false mydocs

And yes it is by design so that you can click on those files and directly open up the files in the Google Drive Web UI.
Thanks again, please feel free to reopen if the mentioned remedy doesn't work.

hello @odeke-em I have one question:
Why when pulling a google doc, it creates two files? One is useful, the *.desktop you can click on it to open in the brower. But what about the second empty file? Can drive be configure so that, when pulling a google doc, it ONLY creates a *.desktop file but NOT an empty file? Thank you in advance.


Uppercase VIN and MODE + Problem action Calendar On

The VIN and the 2 load modes must be capitalized in the scripts to launch actions.
However, I cannot activate the calendar after having successfully deactivated it.
With version 2, it could be activated with the function to return the entire calendar.

charge_on never worked but charge_set_schedules did in 2.0.
Is it possible to see what is wrong?
I use these actions a lot when my Zoe starts to charge when I plug it in while the calendar is activated and out of charge schedule
Thank you

After a long time, the schedule switched to SCHEDULE_MODE. I will do more tests to see which action to work.

Final result:

I hadn't done a test in the car.  I just made several and here is the result.
Don't work:
Charge_mode on: KO
Charge_mode off: KO
Hvac off: KO
Work:
Hvac on: OK
Calendar_set: OK
Charge_go: OK
I think the problem with charge_mode is the upper case letters in the script.  if I put SCHEDULE_MODE or ALWAYS_CHARGING in lowercase, the script does not work.  But I think we have to send lowercase to Renault.  can you fix this problem?

Charge-mode issue is a bug on renault-api: https://github.com/hacf-fr/renault-api/pull/87

i haven't really tested hvac off. I clicked on it but it didn't work right away. I may not have waited long enough.
Ok for charge_mode I thought it was because of the caps.

everything is working fine for the moment and I have not noticed any stoppage of operation as on version 2. I am waiting to have more days of hindsight to see if something does not work anymore. Good work ;)


Documentation improvements & updates

Thanks @Apercu !!! I know that master kept changing today so you are rebasing your changes repeatedly * _ * I'll review your change and help on the rebase


Reformatting code (question)

Hi @clinuxrulz,
I wanted to ask if you would be all right with me reformatting the entire crate using the standard Rust code formatting tool rustfmt.
While working inside Sodium I've really been noticing how much harder it is for me to quickly read and understand the code because it's not quite in that canonical format that I'm so used to. We could then add back in the piece of the former CI I took out which was supposed to be checking the code for correct formatting.

Yes you may to so.


upgrade confluent-kafka to 1.9.0

Have you tested this with Snuba/Sentry?

I believe this is just a maintenance release, but just to confirm - did you check the changelog for the Python library as well as the bundled librdkafka 1.9.2? Is there anything of note in there?

I was able to run the snuba testsuite in sentry using this -- the changelog didn't look interesting.  snuba I couldn't get building on M1 -- which I'll probably be able to fix by prebuilding some wheels (which is what this upgrade is blocking somewhat-ironically!)


some more styling for the listing details page

More styling for Listing details page. Pulled out the bookmarks button into its own component

oops, forgot a change

fixed change


[WIP] Fancy deploy progress monitoring

Seems to work quite ok but perhaps a bit too complex for now.
Would fix #2114
The Common.glyph thing could be cherrypicked, windows terminal does not display unicode.

http://www.rubyflow.com/p/zrjhex-tty-spinner-adds-multi-spinner-support

tty-progressbar can now show multiple parallel progress bars, which would simplify this a lot and remove the need for cursor movement trickery from this pr.


use clipboard for windows as nvim recommand

https://github.com/neovim/neovim/wiki/FAQ#how-to-use-the-windows-clipboard-from-wsl

#736


docs: update readme

SPDX-License-Identifier: Apache-2.0
What did we change?
Why are we doing this?
How was it tested?

[ ] Locally
[ ] Development Environment
[ ] Not needed, changes very basic
[x] Husky successfully run
[x] Unit tests passing and Documentation done

@Sandy-at-Tazama good catch! I updated it. Second reference to frms-coe-lib is intentional. That is where the link points to (the protofile which we use as the blueprint for encoding and decoding).


Initial MI250 Guide

Numbered references don't work. Even if I set numFig = True in conf.py, it won't assign numbers to the images/tables. If we add a name so we can reference the image, it won't get a number, if we don't assign a name, it get's a number but can't be refereced from text.

@lawruble13 it doesn't work, no matter what I do. Here, it was likely a typo. conf.py ending with:
for sphinx_var in ROCmDocs.SPHINX_VARS:
    globals()[sphinx_var] = getattr(docs_core, sphinx_var)

numfig = True

And mi250.md containing:
{numref}`mi250-gcd` shows the components ...

```{figure} ../../data/reference/gpu_arch/image.001.png
:scale: 100%
:align: center
:name: mi250-gcd

Structure of a single GCD in the AMD Instinct MI250 accelerator.
```

And I get the following warning during build:
C:\Users\mate\Source\Repos\ROCm\docs\reference\gpu_arch\mi250.md:15: WARNING: Failed to create a cross reference. Any number is not assigned: mi250-gcd

And indeed, the rendered text is not a link, just "mi250-gcd shows the components"... This supposedly should work, as far as MyST Cheat Sheet or the reference docs go. It doesn't matter what order the numref and the figure definition is, I tried reversing the order, but that has no effect.

I spent the better part of Yesterday looking trying to figure this out, looking at docs, guides, the MyST reference, the source code of rocm-docs-core but I can't find where it goes wrong.

Can I request that you try using Markdown figures? You can reference #1980, which we have validated as working.

@lawruble13 I moved all references to numref. The trick to getting the image references working was to convert all figures, and then the Any number is not assigned warning went away.
Note that the only list-table entry gets it's numbering wrong (it starts with Table 2), but at least it's automated. I'd consider that to be a minor issue and eligible for deferral.


Allow multiple lines with ALT+ENTER

Problem
The only way to implement multiple lines in a cell is shift + enter.
If alt + enter is also available, it seems to be more convenient to write down texts.
Solution
Allow multiple lines with ALT+ENTER
Additional details
No response

Hi @Citatap. Thank you for your feature request. I will add this today

After doing a little research, it appears that alt is not usually a key used for inserting new lines on the web. I cannot

Edit: It seems like Google sheets doesn't support shift and uses alt instead which is really interesting.

Added in 8.2.2


Space Travelers Milestone: Implement mission joining and leaving actions

For this Milestone I:
✅ Implemented mission joining - Actions
✅ Switched badges for Missions - Conditional components
✅ Implemented mission leaving - Actions
✅ Displayed joined missions in My profile - Filters

@abiola-arobieke, thank you so much for your feedback! 🤓


Downloaded files are just 94KB and do not contain video content

Hey,
my downloaded files are just 94KB and do not contain video content? I run the command
php start.php -s "learn-vue-2-step-by-step"
it lists all 30 sessions and downloads them to my local folder. But they are too small and do not contain video content.
Any idea?
Cheers, Max
I attached the terminal output.
output.txt

Do you have a valid account & subscription active?

no, but it is a free series I am trying to download.
2017-02-19 18:15 GMT+01:00 Carlos Florêncio<EMAIL_ADDRESS>
Do you have a valid account & subscription active?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/iamfreee/laracasts-downloader/issues/43#issuecomment-280933112,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKmV34TQUCC0gg2otsJr3VjXzuGgdD2ks5reHjEgaJpZM4MFfZQ
.

You are only allowed to download episodes if you a valid subscription i think.


qemu: fix smp boot not enter idle

Summary
fix smp boot not enter idle
Configuring NuttX and compile:
$ ./tools/configure.sh -l qemu-armv8a:nsh_smp
$ make
Running with qemu
$ qemu-system-aarch64 -cpu cortex-a53 -smp 4 -nographic
-machine virt,virtualization=on,gic-version=3
-net none -chardev stdio,id=con,mux=on -serial chardev:con
-mon chardev=con,mode=readline -kernel ./nuttx
Impact
none
Testing
ostest

@hujun260 please update your patch to fix temp ci error

@hujun260

fix smp boot not enter idle

As long as I tested with the ** latest ** upstream (qemu-armv8a:nsh_smp), up_idle() is called on both CPU0 and CPU1.
(gdb) info threads
  Id   Target Id                    Frame
  1    Thread 1.1 (CPU#0 [halted ]) 0x0000000040293ba4 in up_idle () at common/arm64_idle.c:63
  2    Thread 1.2 (CPU#1 [halted ]) 0x0000000040293ba4 in up_idle () at common/arm64_idle.c:63

Please tell us how you found the issue.

@hujun260

fix smp boot not enter idle

As long as I tested with the ** latest ** upstream (qemu-armv8a:nsh_smp), up_idle() is called on both CPU0 and CPU1.
(gdb) info threads
  Id   Target Id                    Frame
  1    Thread 1.1 (CPU#0 [halted ]) 0x0000000040293ba4 in up_idle () at common/arm64_idle.c:63
  2    Thread 1.2 (CPU#1 [halted ]) 0x0000000040293ba4 in up_idle () at common/arm64_idle.c:63

Please tell us how you found the issue.

@hujun260

fix smp boot not enter idle

As long as I tested with the ** latest ** upstream (qemu-armv8a:nsh_smp), up_idle() is called on both CPU0 and CPU1.
(gdb) info threads
  Id   Target Id                    Frame
  1    Thread 1.1 (CPU#0 [halted ]) 0x0000000040293ba4 in up_idle () at common/arm64_idle.c:63
  2    Thread 1.2 (CPU#1 [halted ]) 0x0000000040293ba4 in up_idle () at common/arm64_idle.c:63

Please tell us how you found the issue.

you are right. Our local code repository has such an issue. Let's keep this fix pending for now.

close already merged


M50120 - Fixing Typo

Translator has reported possible source content issue.
Affected Lines: 83, 84 and 85
Description: Missing hyphen between 4-6; 1-2 and 5-10

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

Hi @v-vafune - Could you open this localization pull request in the private repository, and then sign off directly in the new PR? Thanks. We'll close this PR in the public repo.

Hi @ktoliver ! Thanks for pointing that out.
As a reference here's the new PR link: https://github.com/MicrosoftDocs/cloud-adoption-framework-pr/pull/694


Accommodate and parse JSON schema as runtime program metadata

Original issue created by @jyu00 https://github.com/Qiskit/qiskit-ibmq-provider/issues/996
What is the expected enhancement?
Today Qiskit Runtime server takes plain text for parameter/return value metadata for a Qiskit Runtime program. Something that looks like:
  "parameters": [
    {"name": "iterations", "description": "Number of iterations to run. Each iteration generates and runs a random circuit.", "type": "int", "required": true}
  ],

For parameters, return_values, and interim_results, the server plans to switch to accepting JSON schemas instead. So the parameters above would look something like
    "parameters": {
      "type": "object",
      "properties": {
        "iterations": {
          "type": "integer",
          "description": "Number of iterations to run. Each iteration generates and runs a random circuit.",
        }
      },
      "required": [
        "iterations"
      ]
    }
  },

To prepare for this change we'll need to

allow a user to specify JSON schema instead of plain text when uploading a program
parse JSON schema to construct RuntimeProgram objects. JSON schema can be complicated with nested $ref etc, but we can start with some small. This repo can serve as a reference.

13


Book nitpicks that don't matter

Now that I'm re-reading the proofs, I found a few minor things:

Section 1.4.2 - there's an extraneous ")" after the term set.mm.
Section 3.3.2 and 3.3.3 headings - the headings go far past the intended margin (though they're quite readable).

These should be fixed at some point, but I don't think they need to delay publication.
@nmegill - agree?

I agree.

Included in errata.


A question regarding two details mentioned in the paper.

Thank you for publicizing your code! I have a question regarding two details mentioned in the part below:

In the paper, it mentions "the training of a cumulative reward classifier" and "use the first action in the environment".
It seems these are not present in the current demo code for the hopper experiment. Am I understanding correctly?
Thank you!

Hi,
Yes, code for sampling based off the gradient of the value function is not available in the code right now -- we will update the repo by the end of June with the full code.


NPE in TabsAndIndentsVisitor

@pway99 Please fix this in a way that doesn't assume that a particular style property is present. Just have TabsAndIndents do nothing when a property is null rather than falling back on IntelliJ default.


add Ain Shams University - Faculty of Pharmacy domain

@OmarAminn27 I am sorry, the domain pharma.asu.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Unable to share a git-tfs repo between two users

The docs for bootstrap say that I should be able to do this, but it doesn't work.
[user 1] git tfs clone http://blah/blah/blah $/blah
[user 1] cd blah
[user 1] git remote add shared git@someplace:shared/repo.git
[user 1] git push shared master

[user 2] git clone git@someplace:shared/repo.git
[user 2] cd repo
[user 2] git tfs bootstrap

When running the git tfs bootstrap command as user 2, I get the following:
Unable to locate git-tfs remote with id = default

I notice that user 2's clone of the repo does not have a .git\info\refs file, which I assume is where this info is stored.  Any ideas of why that might be the case?

This sounds like a bug. Can you run git tfs bootstrap --debug and paste the output?

After a quick test yesterday, I couldn't reproduce.
So, Bootstrap command is not completely broken ;-)


upgrade to corefx (netstandard1.1)

also >= dotnet 4.0 still supported

Hi, I have no time to maintain this project. Please consider hosting the code/build if you would like support in .net core.

Hi @sjdirect
to port your project abot to corefx you need nrobots as reference also ported. So because of this you maybe should consider to accept this PR

It's never as simple as just accepting a PR. Notice also that the build server failed, meaning it requires work to fix. Then if people have an issues with the new build in either environment, I have to handle that too. I have no time and likely will not be upgrading NRobots or Abot to .NET core anytime soon. That is why I am asking others for help.

Made this announcement today...

The AppVeyour should be reconfigured. This PR looks good.

Hello, thanks for contributing however, I will not accept .net core upgrade pull requests to this repository. Please create another repository like nrobots-dotnet-core and another appveyor build for it that creates the nuget packages. I am unable to support any .net core supporting changes at this time or in any foreseeable future. The only solution is to create seperate repos/projects/builds for the .net core versions of Abot, NRobots, and Commoner and host support them on your own. I may become a contributer in the future.

Ok i created the project and merge schulz3000 and my PR. https://github.com/Kemyke/nrobots-dotnet-core
I also created an appveyor CI build for it and upload a package to nuget.org with the name "NRobotsCore".
Please feel free to review the code.


[BUG] Opensearch 2.x (with security enabled) does not work with istio

What is the bug?
Background:

We use Opensearch in kubernetes environment with istio handling the mtls (pod-to-pod TLS communication).
Istio integration was achieved by disabling SSL for Opensearch on the REST layer by disabling the parameter  plugins.security.ssl.http.enabled.  SSL was only enabled for the transport layer and the port 9300 was excluded from istio envoy redirection.
So a client application pod in the istio mesh could contact opensearch REST APIs (9200) on HTTP  ( for ex. curl http://opensearch:9200 -u admin:admin), where the Opensearch-security would handle the authentication/authorization of the request, while istio would manage the tls for the traffic between these pods.

Issue:
With opensearch 2.x, the config parameter plugins.security.ssl.http.enabled setting cannot be disabled due to change made in securitadmin design (i.e. removal of Transport client authorization).
On disabling this parameter, securityadmin fails with error as explained in https://github.com/opensearch-project/security/issues/1952
As we cannot disable SSL for the REST (9200) layer for Opensearch 2.x, it no longer works with istio mtls.
The istio functionality worked well with Opensearch 1.x and it is now broken with 2.x.
What is the expected behavior?
Istio integration with Opensearch 2.x to work.
What is your host/environment?
Version

OpenSearch version: 2.1.0
OpenSearch-security plugin version: <IP_ADDRESS>

@jimishs Can you look into this issue?

Hey @aggarwalShivani As you mentioned, it is by design that the ability to make remote calls using Transport layer was removed, which leaves REST layer as the only means for communication and so SSL is required to be enabled for HTTP calls. Hope this answers your question.

Hey @aggarwalShivani . Thanks for bringing this to our attention. To add to what @DarshitChanpura already said, for security reasons, the administrative remote calls are required to be authenticated using TLS. The SecurityAdmin tool depends on TLS to authenticate its requests in OpenSearch 1.0.0. In OpenSearch 2.0.0+ this has been changed to a single interface (due to removal of Transport layer for authorization). This leaves us with only the REST layer, which is now being used by the SecurityAdmin tool to operate securely.
We would be interested to hear any ideas on how to securely support this scenario.

[Triage] Closing out this issue as there is no action to take at this time.


Event sourced, try again...

This is a messy cleanup to see if its possible to split the event sourcing part of the aggregate roots into a separate set of classes. This would make it possible to use event sourcing for other parts of EventFlow and not just the aggregate roots.

Will try to do this gradually instead


Improve interactive console

Improve interactive console. Some nice to haves a that aren't currently implemented

[ ] command history
[ ] up arrow or Ctrl+p should cycle through history
[ ] vim bindings for text entry

Recommend doing this by building app on a modified version of cmd2.

I wrote up a relatively detailed question with what I want to do in the cmd2 discussions. I got some good feedback of where to start, if I want to actually use cmd2 to make this work.
What I found in a basic study is that I can use decorators to register commands, without having to keep them in the class that subclasses cmd2.Cmd. A very simple registration decorator:
def make_command(func):
    """Decorator function to add commands to the app."""
    setattr(FirstApp, f"do_{func.__name__}", func)

@make_command
def new_func(self, args):
    """A function that doesn't have to be named with 'do_'"""
    print("A new function is born.")
    print(args)

Adding functionality for pattern detection requires overriding onecmd:
    def onecmd(
        self, statement: Union[cmd2.Statement, str], *, add_to_history: bool = True
    ) -> bool:
        # ...snip...
        if func:
        # ...snip...

        ### ADDED SECTION FOR INPUT PATTERNS
        # This demo uses a check for a single pattern type (float) but could be
        # sent to a more complex command or pattern parser
        elif is_float(statement.command):
            print(f"{float(statement.command) ** 2} {statement}")
            #  stop = func(statement)
            return False
        ### END ADDED SECTION

        else:
        # ...snip...

For functionality with an empty command, we need to override _complete_statement, and replace raise EmptyStatement at the end of the function with statement = self.statement_parser.parse("empty_statement"). We also have to add the following to the App class:
def do_empty_statement(self, args):
    pass # replace with desired behavior.

I'm uncomfortable having to override some obviously important functions of cmd2 to make this work. It means updates to cmd2 could break titr if I'm not careful.

I think the main thing that I find lacking that I want to see in what is currently datum_console is the command history. It's a reflex for me to press up arrow or ctrl+p to get to the last command, and it's disconcerting / feels like not a well thought out app when these inputs result in ^[[A or ^P, respectively.
Looking through the source code of cmd, I can't quite figure out what makes that tick. I think it has something to do with the readline library calls.
My best bet in the near term may be to take datum_console, make it more bullet-proof, write tests for it, and implement this history functionality. It could be a good learning opportunity for how stdin/stdout work, and how input gets captured from the keyboard.
I'm still at a state where it benefits me to learn things more in depth/under the hood than it does to try to use something like cmd/cmd2 out of the box

Noting that the above non-nice behavior seems to only happen in WSL (on this machine). Need to test if it happens in zsh/ubuntu next time I'm there.
Pretty sure it happens on my work computers too.

Well this could be easier than I thought:
>>>import readline
>>> readline.__doc__
'Importing this module enables command line editing using GNU readline.'

And that seems to fix it. I found this by copying the source code for cmd into a new file, and commenting out lines until the functionality broke. Talk about not obvious, but I can't complain.
I think now it's not a big jump to get datum_console to also accept tab completion using some of the functions from the readline library.


Add APIRequest.swift by @NathanFallet

[x] Project URL: https://github.com/GroupeMINASTE/APIRequest.swift
[x] Update contents.json instead of README
[x] One project per pull request
[ ] Screenshot included
[x] Avoid iOS or open-source in description as it is assumed
[x] Use this commit title format if applicable: Add app-name by @github-username
[x] Use approved format for your entry

Sorry this project is more for apps than libraries - closing, thanks!