soldiers with the normal_ext rendertype applied don't fade out correctly after dying

https://i.imgur.com/5MhyF9v.mp4

Forgot to mention this before, but this same transparency issue seems to apply to PBR as well, as I've seen this consistently happen with dynamic sky objects that use PBR materials.

Update on this, looks like the fade for dying soldiers comes from the stock diffuse material colour which is outright unused in the pbr shader and mistakenly overwritten instead of blended in the normal_ext shader.
Both issues will be fixed for the next bug fix release.


[feature]add stale config file

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

/kind feature
What this PR does / why we need it:
add stale bot config file
Which issue(s) this PR fixes:

Fixes #701
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

/assign @Fei-Guo

/lgtm
/approve


Vendored in helm and related packages.

These packages are required for supporting applications with helm chart.

Torpedo dev-test job run - https://jenkins.portworx.co/job/Torpedo/job/tp-dev-test/757/

Ran the dev job once more - https://jenkins.portworx.dev/job/Torpedo/job/tp-dev-test/761/


Update Omniverse import/export

Quick update of NVIDIA's Omniverse USD Composer import and export capabilities. Based on the documentation and the application itself.

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


Kodi: support aarch64

Seems like we need openjre to work on aarch64, which seems feasible: https://hub.docker.com/r/arm64v8/openjdk/

I packaged adoptopenjdk binary for aarch64 builds in #52454.

Nice! That should be enough to get jre on aarch64 going?

also relevant https://github.com/NixOS/nixpkgs/pull/52534/files

2088ae10a312bd53cb6343e391dcf7368732c3af

https://github.com/NixOS/nixpkgs/pull/52667


Enable trusted number SMS verification

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

Currently spaceship fails for accounts that have 2FA with only trusted numbers, but no trusted devices.

#13188
Description

Added the ability to select a trusted phone number and enter in the response.

Tested on a client account we are working with.

Signed CLA

when can  this be merged?

@KrauseFx let me know if there's any blockers or things you'd like me to clean up - we'd like to get this merged as well if possible so we can get back on upstream.

Can this be merged? Thanks

@joshdholtz we added both a phone number and a trusted device to our account, but they are both showing in the trustedDevices list and both working.
Do you have any guidance as to how to set up a phone number so that it's returned in the trustedPhoneNumbers list instead of as a trusted device?

I have an account that has 2 phone numbers and also gets push notifications with the 6 digit code. This is what a request to GET https://idmsa.apple.com/appleauth/auth (fastlane spaceauth via CLI) or PUT https://idmsa.apple.com/appleauth/auth/verify/phone (using the Apple ID login) returns (yes, both return the exact same JSON here. strange):
{
	"trustedPhoneNumbers": [{
		"numberWithDialCode": "+49 ••••• •••••81",
		"pushMode": "sms",
		"obfuscatedNumber": "••••• •••••81",
		"id": 2
	}, {
		"numberWithDialCode": "+49 •••• •••••85",
		"pushMode": "sms",
		"obfuscatedNumber": "•••• •••••85",
		"id": 1
	}],
	"securityCode": {
		"length": 6,
		"tooManyCodesSent": false,
		"tooManyCodesValidated": false,
		"securityCodeLocked": false
	},
	"authenticationType": "hsa2",
	"recoveryUrl"<EMAIL_ADDRESS>	"cantUsePhoneNumberUrl"<EMAIL_ADDRESS>	"recoveryWebUrl"<EMAIL_ADDRESS>	"repairPhoneNumberUrl": "https://gsa.apple.com/appleid/account/manage/repair/verify/phone",
	"repairPhoneNumberWebUrl": "https://appleid.apple.com/widget/account/repair?#!repair",
	"aboutTwoFactorAuthenticationUrl": "https://support.apple.com/kb/HT204921",
	"autoVerified": false,
	"showAutoVerificationUI": false,
	"hsa2Account": true,
	"managedAccount": false,
	"supportsRecovery": true,
	"trustedPhoneNumber": {
		"numberWithDialCode": "+49 ••••• •••••81",
		"pushMode": "sms",
		"obfuscatedNumber": "••••• •••••81",
		"id": 2
	}
}

What do you get @jgadbois?
How do you actually add a trusted device to an account? I couldn't find that.
This is how my "Security" looks like:

So I looked into this independently, great to find a PR already ready.
The problem with your request @joshdholtz is that as a user I can not actually choose if I want the push or the SMS. I always get the push on login, then I can go to a fallback by clicking:

So the only way to trigger the SMS flow would probably be to somehow "escape" from the normal "Please enter the 6 digit code:".
I can't think of an elegant way, but we could add "Enter x as the 6 digit code to start the SMS flow" or something. That then triggers the code that @jgadbois wrote.
Or does the flow work differently for you?
Any better ideas how to implement this?

When this gets merged, we can also get rid of https://github.com/fastlane/fastlane/tree/master/spaceship#bypass-trusted-device-and-use-sms-for-verification and https://github.com/fastlane/docs/blob/master/docs/best-practices/continuous-integration.md#bypass-trusted-device-and-use-sms-for-verification

Hi, @jgadbois when do you think you will fix conflicts and do the requested changes? If you do not have time can we re-open my PR?

I reopened your PR @mariannegru as I saw you already rebased 💕
Let's see if CI likes it and continue from there.

I'm fine with using @mariannegru or rebasing, but I'm not sure how to proceed based on the feedback.  I'm not able to reproduce the situation where there's both 6 digit codes text codes and the 4 digit push/text codes for trusted devices.
I'm happy to clean this up and get it out the door if the team can give me some guidance.

I will spend some time later to re-read the feedback here and also look at @mariannegru 's rebased PR again.

I'm not able to reproduce the situation where there's both 6 digit codes text codes and the 4 digit push/text codes for trusted devices.

What comment are you referring to here?

Can I have a response with trustedDevices, I never get trustedDevices list, and we have both enabled on account (I run the code on a different device)


Refactoring: card component

I extracted a card component, removed bad jquery code and minor fixes..
From my test everything still work.

@fain182 looks good! thank you!


How to configure for Portal ?

Hi,
Does this attachment-viewer support Portal for ArcGIS ?
I have Survey123 app enabled with multiple photos in my Portal.
How do I configure this app for Portal for ArcGIS ?
Thanks,
Mayur

Attachment Viewer should now be supported in Portal for ArcGIS

Closing issue now that Attachment Viewer is supported in Portal for ArcGIS. Feel free to re-open if needed.


Use function.name instead of func_name to support Python3

functon.func_name is not available in Python 3. This PR change to use __name__ instead of func_name

LGTM!


Investigate adding support for Nim ( and reactors )

i left you a note on discord about that. I had success to build wasi app with wasm-sdk-19 + Nim but i'm having trouble to find a wasi web runner that would offer support for reactors in order to set a standard for using https://developer.mozilla.org/en-US/docs/Web/API/window/requestAnimationFrame as it should

References:

https://github.com/WebAssembly/wasi-libc/pull/74
https://github.com/WebAssembly/WASI/issues/13

Thanks Paul! I've responded in discord. This looks very doable. Is there a description somewhere of how to call a WASI binary that uses reactors? I can see that you start by calling _initialize and then you're meant to call a main loop function, but what is that called?
It looks like I might have to read more about the Component model and how it composes collections of imports/exports.

I can see that you start by calling _initialize and then you're meant to call a main loop function, but what is that called?

There is no specific main loop function afterwards, the intention is that the user will then call specific WASM exports as necessary, see https://github.com/WebAssembly/WASI/blob/main/legacy/application-abi.md.
For comparison, see https://github.com/bjorn3/browser_wasi_shim/pull/17 for how another browser WASI shim implements support for WASI reactors.
For @runno/wasi, one can currently do this via
wasi.instance = instance;
wasi.memory = instance.exports.memory;
instance.exports._initialize();

but one could also add a helper function (similar to wasi.start) for this as in https://github.com/bjorn3/browser_wasi_shim/pull/17, but it might make sense to wait until the component model/wasi_snapshot_preview2 is ready.

Thanks @amesgen! This looks pretty simple to do. I've been trying to follow the NodeJS API (https://nodejs.org/api/wasi.html). I'll have a look at implementing it that way.


[Android] Add Requires property support

Adds the Requires property and the methods in the CardRendererRegistration class to be able to register a FeatureRegistration objec.
How verified
An sample was added to the mobile app and the result was as shown below

This PR should probably be targeted against its parent branch, paulcam/featureReg :)

Looks great! Once we can reconcile this against my branch, I think we'll be good!

I have reconciled this branch but I merged master in between so maybe reconciling the base branch with master may remove the extra files

In reply to: 483916178 [](ancestors = 483916178)


Locally deleted file that was added in PR will throw error when trying to open changes

Issue Type: Bug
Repro:

Checkout a PR that adds a file
Delete the file being added
Click on the file to view changes in the PR

Extension version: 2021.2.542
VS Code version: Code - Insiders 1.53.0-insider (dcda88e5a69a52b3a9e141af73b68caae08cb2bf, 2021-01-25T08:41:30.565Z)
OS version: Darwin x64 19.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2200)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 5, 4

Memory (System)
16.00GB (2.00GB free)

Process Argv
-psn_0_1270070 --crash-reporter-id b16ce66d-dd99-4888-ad6f-60ec3335d2e7

Screen Reader
no

VM
0%

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
openlogontheside:30221882
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30208930
wsl2promptcf:30219163
pythonvsdeb440:30224570
unusedpromptcf:30219165
folderexplorer:30219166
openfilemenucf:30219169
pythonvsded773:30223139
vstes627:30244334
pythonvspyt600cf:30251589
core-portspanel:30233467
coreuntitledfile:30249963

Clicking here:

Causes this:

I think expected behavior here is that if you delete the file added in the pull request, you get an error when you try and click on the file saying that it's been deleted.
Does that sound right?

Maybe bring up a readonly diff of the previous state?

We can make sure we take into account whether the local file is deleted when we set the command for the tree item. However, I don't think it's worth listening for file events and updating the tree.
Changes for this are needed here:
https://github.com/microsoft/vscode-pull-request-github/blob/a3040499b41ea55fcab3500bf599b7f0c5e7a98e/src/view/treeNodes/fileChangeNode.ts#L40-L64
If the parentURI file doesn't exist, we should use the command to open a file instead.


Fixed parentheses mismatch and try-except blocks

In reference to issue#26

Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow these steps.

Star this repository. ⭐
SignIn / SignUp in [ClueLess Website]
Go to the profile section from the top right-hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.
That’s it. Follow the leaderboard!
And don’t forget to share your achievement with your friends.


Extract DAO Context into a Package

Sync with team to review and figure out what else it might need.
Worth battle testing in core ui app to uncover issues/ideas before we do this.
Can look at getting some of the fetch helper hooks into it at this time:
See Members.tsx in core ui

done


Non digits custom user model primary key password lookup.

If custom user model for example uses UUID as PK, admin password change form cannot be opened.
Another way is dirty hack for custom UserAdmin like
https://gist.github.com/hellysmile/97637aa3b9f69d12692f

Hi. Thanks for the patch. Do you want to open a ticket for this on code.djangoproject.com? A test would also be helpful.

Also, I'd recommend [^/]+ over .+.

buildbot, test this please

I will create tests for master branch with UUIDField as PK for AUTH_USER_MODEL

Tests added, please review.

Please follow https://docs.djangoproject.com/en/dev/internals/contributing/committing-code/#committing-guidelines
Rebase your commits so you will have only one

buildbot, test this please

Is there a ticket for this change?

No, I haven't found...
Is ticket required for so small change?

Yes, a ticket is required for anything except typos. It helps us keep track of pull requests (whether they need improvement, etc.).


Uxie Patch

This is a concept I was hoping to work around. I am not sure if it works but in theory, it should.
https://github.com/TTTrainer/tcgone-engine-contrib/commits?author=TTTrainer

Hi, you need to open the PR against my repo, not yours :) So the target repo must be my repo/master, source is your repo/TTTrainer-patch-1.


Add to hub.helm.sh

I would like to see this chart made available on hub.helm.sh

Done ;-)
https://hub.helm.sh/charts/traefik/traefik


Ditch JSDocs; adopt it's awesome successor, documentationjs

Just started using ternjs, I LOVE IT. For years I tried to avoid writing js because--mainly due to its lack of statically-typed-semantics--the autocomplete/intellisense/what-have-you was a joke. Also because, well, js sucked up until a few years ago. I've always relied heavily on this feature and was very excited to discover this project last night. I'm happily using it in atom right now.
However, I feel there are still somethings that could be improved here; namely, the documentation support. With my current project, I'm only getting signatures and the small subset of JSDoc syntax that ternjs parses currently. I think this is a vital feature! It saves me a bunch of time, I want to improve it. I'd love to see ternjs get to the level of say intellisense or what eclipse has. Providing robust, seamless documentation support would be integral to such an endeavor.
Let's face it, JSDocs are pitiful. The spec leaves is rather skimpy and poorly documented. Use of the de facto standard is not nearly as prevalent as other its counterparts in say, javadocs. Tonight I found [documentationjs/documentation] while trying to figure out how to get tern to pickup documentation from/document a nested map/obj. This is awesome.

A documentation generation system that's beautiful by default, flexible across formats and styles, and powerful enough to support JSDoc's advanced syntax.
ES5 and ES6 support of JavaScript, with support for other transpilers a possibility
Using babel, we have support for a wide range of ES6 & ES7 features, as well as Flow type annotations.

It also can

automatically link imported dependencies to github sources we could then grab that very easily
export your documentation to json, html, markdown

It extends JSDocs into something more modern and has a ton of extra features! I would love to see the JSDoc plugin get revamped to use this package instead. It can still parse tradition JSDocs, so there wont be any messy breaking changes forcing a completely new standard. I think the adoption would be quite easy, and would inevitably lead to both projects continuing to grow.
[documentationjs/documentation]  has a lot of overlap with ternjs and acorn. Letting the user explicitly define the relationships between bits of code could be used to drastically improve the suggestions! I could see resolutions becoming much more efficient/accurate (given the right implementation.) I've thought so many potential benefits from this, I just had to make this post.
I would be happy to contribute towards such an undertaking. I can see there are many #680 other #677 developers #673 who #655 would  #644 probably appreciate this integration as much as I.
Interested to hear what you all think. :beers: cheers

The one majors implication I see--haven't read too much of the ternjs source yet--is that ternjs uses acorn while documentationjs uses babel for it's AST analysis. The two could certainly be fused, and this package has a reaaaaaly strong api too so I'm optimistic. something to chew on

I'd love to see ternjs get to the level of say intellisense or what eclipse has.

You mean ternjs inside Eclipse? If it that please read https://github.com/angelozerr/tern.java/wiki/Getting-Started
If I understand the goal of documentjs, it can generates JSDoc from JavaScript files? If it that, I had started https://github.com/angelozerr/tern-comments-generator and it exists a CodeMirror addon to generate JSDoc when user does "/** + /n" It's a just a POC.
For integrating that inside Eclipse, see https://github.com/angelozerr/tern.java/issues/334

@angelozerr

You mean ternjs inside Eclipse?
No I meant that I would love to see ternjs get to the level of reliability, consistency, and accuracy that you see in say eclipse. (And I was referring to the JavaDoc functionality of it's autocomplete solution.

It doesn't generate JSDocs, It reads them from your annotated source code. That way we could pipe all the detailed data into ternjs so it can be presented to the user during completion suggestion.. But I like what you just linked me, that's
definitely also part of the puzzle.

The way I understand it, documentationjs reads the same type of annotations as JSDoc. Since Tern implements its own annotation parser, I don't think there is much of an issue here. Am I misunderstanding something?

ternjs's parser only supports a subset of JSDoc syntax and is buggy at (@see issues I referenced in my original post). documentationjs is a fleshed-out solution. The JSDoc standard is not going to get any better than it is currently. documentationjs is a rejuvenation of it. I think if it were to become more widely adopted, it would greatly benefit the OpenSauce community. And it's an extension of JSDocs so existing tools (perhaps not a jsdoc linter) should not have any problem reading the data they are equipped to parse.

@rozzzly IMHO I think we should improve https://github.com/ternjs/tern/blob/master/plugin/doc_comment.js to support full JSDoc features that documentationjs supports.
I suggest you that you create an issue per features that you wish to have. Any contribution are welcome!

The problem we're solving isn't the parser, or at least that's the easy part, the problem is the connection between JSDoc-style annotations and Tern's understanding. So no, trying to glue in a library isn't going to help here.


add libev-dev / libev-devel

Having libev headers makes it possible for folks to build the required Python packages for things like flask and other Python apps.

Discussed with @colearendt and decided to use a different image than the session image.


Allow writing tuples of tuples and fix #219

Coverage decreased (-0.41%) to 72.22% when pulling d084f1cbcdd37f166d8ea06adadc44d4e4dc6d61 on sjk/219 into 02552083acb478fb4cb5d92e4994a255e33f7f6b on master.

It seems the three new "missed" lines are actually these, which I expanded from a single line to multiple lines. They are definitely hit.


Make issue completion in SCM input configurable

Examples:
#1234
#1234: Issue Title
Issue Title - #1234

Suggestion from @egamma that could enable this: Do something similar to what window title does: "window.title": "${activeEditorShort}${separator}${rootName}"


Fix threshold when scrolling is limited to one direction

Horizontal threshold will start scrolling even if it is locked to just scroll vertically.
You can't then recognize a swipe right to delete gesture on a vertical only list.
Also, I use scroll starting to cancel my long tap timer.

I updated the description to make more clear.


Clean up for druntime 2.071

Remove everything that is shipped with druntime 2.071.
Sadly, there still seem to be some things (like SO_REUSEPORT) to be missing (meaning this module is not yet obsolete).

Wow removing 1.1K lines. That's a great cleanup!
But doesn't this mean you temporarily loose ldc stable support? Afaict the latest ldc version ships with 2.071, but it's still in beta.

For backwards compatibility it would probably require a __VERSION__ >= 2071 at first, I use 2.070 myself still because my production-level project isn't ready for 2.071 yet due to some regressions in win64 that I haven't addressed yet

It's possible that this would work with 2.070, I'd have to check

$ dub test --compiler=ldc2 runs fine for me, though I believe they may have been added earlier. I'll have a look.

Oh I completely forgot that the unit tests covered all the way down to 2.068. OK Thanks for this!


Webside doc fixes

Website doc changes
Deployed here for testing : https://akartsky.github.io/kubeflow-manifests/main

Broken link: https://github.com/awslabs/kubeflow-manifests/blob/main/deployment/vanilla/kustomization.yaml#L57
Fix: https://github.com/awslabs/kubeflow-manifests/blob/main/deployments/vanilla/kustomization.yaml#L57
In Usage Tracking

Broken link: https://akartsky.github.io/kubeflow-manifests/docs/deployment/cognito/guide/kubeflow-manifests/docs/deployment/add-ons/load-balancer/guide#create-certificates-for-domain
Fix: Unsure
Text: If your cluster is not in N.Virginia(us-east-1), create an ACM certificate in us-east-1 for *.platform.example.com by following the process similar to section 2.0

Broken link: https://github.com/awslabs/kubeflow-manifests/blob/main/docs/deployment/cognito/README-automated.md
Fix: https://github.com/awslabs/kubeflow-manifests/blob/main/docs/deployments/cognito/README-automated.md
Text: Take note of the following values from the previous step or awsconfigs/infra_configs/scripts/config.yaml if you used the automated guide

In https://akartsky.github.io/kubeflow-manifests/main/docs/component-guides/kserve/#create-a-certificate
Broken link in:
Create an ACM certificate for *.platform.example.com and *.staging.platform.example.com in your cluster’s region by following the create certificates for domain steps in the Load Balancer installation guide.

In https://akartsky.github.io/kubeflow-manifests/main/docs/component-guides/profiles/#component-level-implementations
Broken link in:
You can find documentation about the AwsIamForServiceAccount plugin for specific components in the individual component guides. Read on for general configuration instructions.
Fix: https://akartsky.github.io/kubeflow-manifests/main/docs/component-guides/

@akartsky I recently merged #272 in which i followed this format for relative link. See this commit - https://github.com/awslabs/kubeflow-manifests/pull/272/commits/6387387b33e71bfb211e66a191c1253dedf3e3bc
Can you please rebase and check if it is correct?


question: Does quantization affect only weights, or also activations?

With your 8-bit quantized GNMT model, I understand that the weights are quantized to 8-bit and then treated as floating point during inference. Does this mean that the 8-bit weights are interacting with float32 activations? Or are the activations also effectively quantized to 8-bit?

The weights are quantized to Int8 and saved as Int8 variables.
Before inference the Int8 weights are loaded from the checkpoint and dequantized to FP32.
Inference is done in FP32 and there is no interaction at all with quantized values.

Okay, got it - activations are not ever quantized, neither "dynamically" nor "statically" (going by these definitions).
However, are the embedding layer weights included in the weight quantization? If so, then I guess the activations are initially influenced by the quantization, because the activations originate from vectors pulled from the embedding layer. Does that sound right?

Yes, sounds right


DLFW pipeline changes and documentation

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

#2397


Add streams:subscribed search operand

Following #12999 (which completes #8859), we should also add the planned streams:subscribed search operator.  This is a somewhat awkward thing to implement, in large part because we don't have existing search queries with an OR gate between items with a UserMessage row or not, and this will violate our search base-then-add-conditions query archictecture.  I think the operator should do the following:

First, we should try to refactor get_base_query_for_search to use a new join_with_usermessage function to create the need_message and need_usermessage case; we'll reuse this function later.  If this can't be done without regressing, we don't have to, but the this should help us be confident that the below flow works.
Compute search_recipient_ids and full_history_recipient_ids, partitioning the recipient IDs for the user, depending on whether history is available to you, and use them to create subscribed_recipient_cond and full_history_recipient_cond.
Have this operator have ok_to_include_history=True so we get just the message case
Create user_message_query = join_with_usermessage(message_query) (where message_query is whatever is passed in to the by_streams function).  The purpose here is to ensure that any operators processed already apply to both code paths.
return query = or_(user_message_query.where(subscribed_recipient_cond)), message_query.where(full_history_recipient_cond).

Tagged as difficult just because this will take significant care especially in unit tests, but I think @thedeveloperr should be able to make it work given his past work :).

@zulipbot claim

@zulipbot claim

Still working on it.

@zulipbot claim

@zulipbot claim
Will resume my work on it.

Still working on it.

@timabbott  Can i work on it.

@timabbott  Can i work on it.

@tushar912 hi feel free to pick it up. I was intially responsible for this issue but due to some personal issues I can't devote time to it. Please feel free to ping me on chat.zulip.org for any questions. Happy to help 😃

@tushar912 hi feel free to pick it up. I was intially responsible for this issue but due to some personal issues I can't devote time to it. Please feel free to ping me on chat.zulip.org for any questions. Happy to help 😃

@thedeveloperr Thanks . I will start work on it.

@thedeveloperr Thanks . I will start work on it.

@zulipbot claim

@zulipbot still working


Log output has wonky background/foreground colors in Rider

Currently the background and foreground colors are explicitly set in ConsoleLogMessage, which makes the log output on light terminals difficult to read in JetBrains Rider:

(JetBrains Rider, Visual Studio Light theme)
Interestingly the issue remains when using a dark theme, since Rider apparently inverts the colors anyway:

(JetBrains Rider, Rider Melon Dark theme)
It looks like Konsole handles it much better, though it's not perfect:

I've worked around it by adjusting the console colors to match Statiq, though it's less than ideal. I see a few potential fixes:

Don't explicitly setting the background color to Black in ConsoleLogMessage - though this could also cause issues if someone has their terminal background lightly colored
Allow the background/foreground colors to be configurable
Dynamically adjust colors based on terminal settings (is that even possible?)
Ignore it, since it could be argued that this is more of a Rider bug than a Statiq bug.

To the extent that this can be improved in Statiq, I'm all for doing so - especially since I'm a heavy Rider user myself. What's interesting is that I use light themes and haven't noticed this behavior:

It's been a while since I've been in this code, but I think what might be going on is that the "default" console background is black and most console emulators treat a black background as transparent, which is why it works in many cases including mine. Your case is interesting though because that's obviously not happening, and the black background is treated explicitly. Any ideas why the black background is showing up in your console? Is it a standard Windows terminal, PowerShell, or something else?
I'm wondering if just not setting a background will do the trick. I seem to remember that there was a reason I had to, but maybe that's been long fixed/overcome and there's no longer a need to. Running some quick tests right now it didn't seem to have a detrimental impact removing the background color entirely. I'll play around with it a little more and will just go ahead and ship it without if I don't hit any snags - if there's an issue, users will file issues and I can revert.

That's fixed it, thanks. I'm running Linux (Fedora 37 KDE), which is probably where the discrepancy is from. I thought that since it's built in, terminal behavior would be consistent across platforms, but I guess that's not the case.


Improvements to UX and UI

[x] [Dashboard] the margin to the top header is quite large for no apparent reason
[x] [Balances] additional padding. This is provided by default, and there is no reason to have more; right now it breaks the constistent padding across all pages
[x] [Dashboard] Background color. Right now it's white, which draws a lot of attention, even though there's nothing else on the page. Could perhaps consider using the same background color, and a border? (just a suggestion). This way, you retain the theme throughout the application, which is relatively minimalistic (emphasis on relatively).
[x] [Dashboard] Colors on the tabs. Perhaps we could reuse the accent color? Right now it's kind of a dull gray, which is a color that - to my knowledge - is not used anywhere else on the website. I think something alone the lines of bootstraps tabs would suit the style better.
[x] [Balances] The month picker on this page is very unintuitive. It should not be necessary to scroll through 36 pages if you want to see the month balance three years ago. Something along the lines of the proposed month picker in HTML would be nice. I'm sure there are good libraries for this.
[x] [Balances] The "View" picker on this page goes against the current input design, which is flat. I'm not trying to impose the styling from /transactions on the components here, but I do think we should agree on one style. That being said, there are already defined inputs for text/number/date ++, so I think going for the flat style would be easier. Furthermore, as of now, the select is too small for the content, so on my screen at least, the top of the text is being cut (the text being "Calendar" or "List").
[x] [Dashboard] The shadows on the tab pills and the tab content don't make sense. The pills have a light coming from bottom left, whereas the content has light coming from the upper left.
[x] [Dashboard] There's a gap between an unselected nav pill and the tab content.
[x] [Dashboard] The font-weight on the nav pills is too high, so it's quite visually dominant, even though I'd argue that it doesn't have to be.

Please add more potential visual improvements if you come up with any.

Fixed by #160.


Show puzzle replay wins differently from wins in puzzle stats

Maybe use rounded corners, too
see

Implemented in v2.3.155


Modern mode should alter css generation to avoid generating of unneeded vendor prefixes

What problem does this feature solve?
Since modern mode was placed on the root level of the config (in Nuxt-edge) and creates a js bundle version that can be used only by the modern browsers it would be logical to also generate CSS that is free from all unneeded stuff such as vendor prefixes for old browser versions.
What does the proposed changes look like?
No additional external settings should be necessary. Same as current version.
export default {
modern: true
}

This feature request is available on Nuxt community (#c8113)

Likely related: https://github.com/webpack-contrib/mini-css-extract-plugin/issues/202

RFC: https://github.com/nuxt/rfcs/issues/7

Let's continue there


Hierarchical pathfinding question

I have seen your implementation of the hierarchical pathfinding and mine is slightly different. I just have a tilemap and each tile has a node with connections. Now I want to divide my map in "sectors" and build nodes on the edges where there is a entrance.

Is there some implementation already available that does this for me? Since this would be pretty much the same for each basic tile map I would expect something like this is already in place. Something like this:

 (forgot a view connections).
If so, I would like to know :)
If not:

So i lookup all the borders and add level 2 nodes where there is an entrance. I (try) to connect all those within a "sector" and the bordering neighbors.

Is this sufficient, does the pathfinder find paths in between the level 2 nodes once it finds a level 2 path? Or do I need to store the actual shortest path in the connections? I can imagine it is the former since looking up A -> B -> C -> ... -> Z is much faster then looking up A -> Z where the open list will grow out of control.

I see you store all the nodes in a single array and use an offset for higher levels. However I want to make a dynamic map. And once a node changes it will update it surrounding nodes and if level 2 changes as well it does the same. But that means I need something more flexible then a ArrayList or (like I do on the 1st level) create all possible nodes and alter there connections.
I would appreciate some pointers in the right direction.
Thanks in advance,

What you're trying to do is known as "graph partitioning" in pathfinding literature.
There are many techniques, each with pros and cons, but none of them are currently supported.


Undefined is not function when click on notification

Hi, when I click on notification I get Undefined is not a function. I just want to open a specific screen and pass parameter I am using react navigation
Code:-
onOpened(openResult) {
    console.log('Message: ', openResult.notification.payload.body);
    console.log('Data: ', openResult.notification.payload.additionalData, openResult.notification.payload.additionalData.catalog);
    console.log('isActive: ', openResult.notification.isAppInFocus);
    console.log('openResult: ', openResult);

    //if (typeof openResult.notification.payload.additionalData.catalog !== 'undefined'){
      this.props.navigation.navigate('CatalogInfo', {
        query: openResult.notification.payload.additionalData.catalog,
      });
    //}
  }

@anehkumar The message tells you what is wrong, for whatever reason, the this.props.navigation.navigate object is undefined. This would not be an issue with our SDK. I would recommend looking into how you are using the react navigation SDK to make sure you're setting it up properly:
https://github.com/react-navigation/react-navigation/issues/832


Weird Tmux colors

When using Vim from tmux I get some janky colors:

Any idea why this is?

@jdsimcoe Did you follow the Tmux instructions from the website instructions? My guess would be you are either on an old version of tmux or you aren't using the needed .tmux.conf pieces from the linked example.

👍


[WIP] Inclusive range support

I see your point on situation with RangeInclusive support #4. Whatever the destiny of the language feature may be I wanted to experiment with its support in twiddle. So here is the result of my exercise - I thought I'd share it to get your opinion on general approach:
I introduced a new intermediate structure BitRange and used Into<BitRange> bound on  argument for all the relevant Twiddle functions.
So adding new kind of range representation R becomes a matter of impl From<R> for BitRange.
What do you think ?

Ah, sorry for not responding, for some reason I must've missed a notification for this!
It's an interesting approach, though I think I'd rather wait until there's something solid in place, and at that point I would probably just switch to using only inclusive ranges, so I don't think this is something I'd merge, sorry.


Remove datasets with multiple versions

Fixes #5
I added the earlier versions to ignore, so only the actual latest version is kept.
The following all have newer versions:

CMIP6/CMIP/CCCma/CanESM5/historical/r1i1p1f1/Amon/ta/gn/v20190306
CMIP6/CMIP/CCCma/CanESM5/historical/r1i1p1f1/day/ta/gn/v20190306
CMIP6/CMIP/NCC/NorCPM1/historical/r1i1p1f1/Amon/ta/gn/v20190914

If it is a bug in the esgm downloader, then it would be better to implement the fix there, which is a bit out of scope. I suggest merging this and keep this in mind for future development in a new issue.

If it is a bug in the esgf downloader, then it would be better to implement the fix there

Did you look into this? Is it a bug in esgf-pyclient or the ESGF API or do we somehow not understand how to use it correctly?

Should we fix the download script to fix this instead?

Yes, that would be the easiest solution

Hi @bouweandela , I added a function to select only the most recent version of each dataset. Could you have a look?


fix: enable serde feature "derive" for non-dev

This fixes at least the following case:

Clone agent-rs into ~/agent-rs
Create a new project ~/ic-types-test
Add a dependency on ../agent-rs/ic-types
~/ic-types-test fails to build
Apply the change from this PR in ~/agent-rs
~/ic-types-test builds

Fixes https://github.com/dfinity/agent-rs/issues/208

Thanks for the patch!


Regression: JSON reporters are broken after Karma 2.0.1

Starting from karma 2.0.1, the JSON reporters available:

https://www.npmjs.com/package/karma-json-reporter
https://www.npmjs.com/package/karma-structured-json-reporter
https://www.npmjs.com/package/karma-json-result-reporter

are all failing as Karma now returns an object with a circular reference and all the reporters use JSON.stringify to generate their reports.
Expected behaviour
The JSON reporters should continue to work as they were previously.
Actual behaviour
Running a simple test with karma-json-reporter produces:
Chrome 67.0.3396 (Mac OS X 10.13.5): Executed 43 of 43 SUCCESS (9.014 secs / 8.844 secs)
10 07 2018 11:30:43.905:ERROR [karma]: TypeError: Converting circular structure to JSON
    at JSON.stringify (<anonymous>)
    at /Users/cedric/Code/ninjasquad/ponyracer/node_modules/karma-json-reporter/index.js:35:24
    at /Users/cedric/Code/ninjasquad/ponyracer/node_modules/karma/lib/helper.js:145:7
    at /Users/cedric/Code/ninjasquad/ponyracer/node_modules/graceful-fs/polyfills.js:287:18
    at FSReqWrap.oncomplete (fs.js:153:5)

Environment Details

Karma version (output of karma --version): 2.0.2 (same issue with 2.0.3 and 2.0.4)
Relevant part of your karma.config.js file

module.exports = function (config) {
  config.set({
    basePath: '',
    browserNoActivityTimeout: 30000,
    frameworks: ['jasmine', '@angular-devkit/build-angular'],
    plugins: [
      require('karma-jasmine'),
      require('karma-chrome-launcher'),
      require('karma-jasmine-html-reporter'),
      require('karma-coverage-istanbul-reporter'),
      require('karma-firefox-launcher'),
      require('karma-json-reporter'),
      require('@angular-devkit/build-angular/plugins/karma')
    ],
    client: {
      clearContext: false // leave Jasmine Spec Runner output visible in browser
    },
    coverageIstanbulReporter: {
      dir: require('path').join(__dirname, '../coverage'),
      reports: [ 'html', 'json-summary'],
      fixWebpackSourcePaths: true
    },
    jsonReporter: {
      stdout: false,
      outputFile: '../results/karma-results.json'
    },
    reporters: ['progress', 'json', 'kjhtml'],
    port: 9876,
    colors: true,
    logLevel: config.LOG_INFO,
    autoWatch: true,
    browsers: ['Chrome'],
    singleRun: false
  });
};

Steps to reproduce the behaviour

Add karma-json-reporter to karma.config.js
Run the tests

Other info
After a brief analysis, it looks like the migration of the Browser class to ES2015 introduced the regression (see https://github.com/karma-runner/karma/commit/bb012e2586cc0580beea48bc516c10a9be8ca79e) as it added a bunch of fields on the Browser object, some of them self referencing.

I submitted #3076 as a potential fix for this issue, by adding a toJSON() method to the problematic Browser class.


For summarizing, fetch snapshot fails when the snapshot is too large

https://portal.microsofticm.com/imp/v3/incidents/details/286391295/home
There is a behavior captured in the above incident which seems to indicate that what is specified here does not hold and the size limit is actually taken into consideration further down in the code when this.hostPolicy.summarizerClient is enabled.
We need to confirm that the limit does not apply to the summarizer (preferably adding some tests for the specific scenario). And if it does not apply, we need to follow how the summarizerClient flag is managed.
Update
Snapshot limits should be disabled when requests are coming from a summarizer. See: https://github.com/microsoft/FluidFramework/blob/51d2d7b5541e2ad344d00509ff127f58fd792ea9/packages/drivers/odsp-driver/src/odspDocumentStorageManager.ts#L585
The property is set by the  OdspDocumentService which it turn reads it from the odspResolvedUrl.summarizer property:
https://github.com/microsoft/FluidFramework/blob/51d2d7b5541e2ad344d00509ff127f58fd792ea9/packages/drivers/odsp-driver/src/odspDocumentService.ts#L149
However, the resolved url is not set properly.
The summarizer does try to set it https://github.com/microsoft/FluidFramework/blob/51d2d7b5541e2ad344d00509ff127f58fd792ea9/packages/runtime/container-runtime/src/summarizer.ts#L126 but then the relative loader will just reuse the container's url: https://github.com/microsoft/FluidFramework/blob/51d2d7b5541e2ad344d00509ff127f58fd792ea9/packages/loader/container-loader/src/loader.ts#L77
This needs to be consistent and fixed.
As an implementation hint, it would probably make more sense to rely on the client type (summarizer vs interactive) and have the relative loader propagate the client type (for all requests). Then, the OdspDocumentService can adjust policy accordingly to the client type.

@vladsud A good question was raised today during our sync-up about this snapshot size behavior difference between summarizers and non-summarizers and I think you'd be able to shed some light on this.
Assuming we are able to send a large snapshot and this bug is fixed. Only the client who summarizes is able to download it, right? Is this a resilience mechanism to hope for the garbage collector to purge the snapshot which would afterwards (hopefully) allow all other clients to download it? In other words, why are we allowing one client to download the snapshot when nobody else can?

Teams client wanted it to better control bandwidth consumption. Their take - if files are large, they want to render link in chat and let users decide if they want to follow link or not (and thus pay the cost or not).
And yes, other clients may open it just fine.
Summarizer has to be excluded, such that if file grew 1 byte over the limit in this window (but was opened in Teams client before limit was increased), we are not leaving file unsummarized.

@chensixx , any chance you can take a look next week? Sorry to dump on you the last week of February, but that issue is rather important, and pretty well understood. It was raised by Teams team through ICM, and somehow priority of this item (or rather the fact that it's well understood) escaped me.
Please take a look. If you have any questions, I, Andrei or Jatin can help with it.
The core of the issue is that we lose property tracking that it's a summarizer (as it's passed from loader headers to resolved URI to driver), and as result driver does not reset mds property that blocks large summaries.

question: when @andre4i said we should let relative loader propagate client type, doesn't request already have the clientType in the headers? am I reading this correctly?

currently manually testing loader is passing the correct type to odsp driver, and for summarizer client, the mds limit and timeout are undefined.

manual test completed, hostpolicy is set correctly


OCC-36: Locally maintained tax rates

OCC-36
Fixes #158

Disregard failing post-pull-request-checks-automation workflow, I've opened an issue for that:
https://github.com/Lombiq/GitHub-Actions/issues/182


ORCNN required?

Attempting to use this w/ detectron2. Training metrics look good, but completely fails in evaluation. Currently debugging, but would there be any reason why this couldn't work with a regular RPN and ROI heads versus oriented?

@TomStarshak please ask the author of oriented-rcnn.

I feel like this is a question about your backbone? Did you ever try it as a backbone with a regular faster-rcnn/mask-rcnn?

However, in my own opinion, the backbone is only a feature extractor, where the features are enhanced by attention. It usually does not influence the head of downstream tasks.
I notice that you say it fails in the evaluation? Is it possible that the parameters after training are not loaded？

I agree that it's just a feature extractor. I thought at worse there would be degraded performance.
I don't think that's possible as my training works with other backbones. Given you also don't think there's a reason why it couldn't work with vanilla rcnn I suspect I have made some error porting to detectron2.. Thanks for the discussion.


Replace Vagrant test setup with Molecule

Instead of maintaining a custom Vagrant test setup, use the Molecule tool. Molecule helps keep the testing process uniform by providing support for Ansible, Vagrant, and Testinfra all in one tool.
See also: http://molecule.readthedocs.org/

The README still has a link to the now deleted examples directory.

molecule test worked for me
$ molecule test
... provisioning ...

PLAY RECAP *********************************************************************
ansible-java               : ok=8    changed=3    unreachable=0    failed=0

Idempotence test in progress (can take a few minutes)...
Idempotence test passed.

Executing testinfra tests found in tests/.
============================= test session starts ==============================
platform darwin -- Python 2.7.9, pytest-2.9.1, py-1.4.31, pluggy-0.3.1
rootdir: /Users/jwalgran/Projects/ansible-java/tests, inifile:
plugins: testinfra-1.1.1
collected 1 items

tests/test_java.py .

=========================== 1 passed in 0.30 seconds ===========================

No serverspec tests found in spec/.

==> ansible-java: Forcing shutdown of VM...
==> ansible-java: Destroying VM and associated drives...

:+1:

Replaced by https://github.com/azavea/ansible-java/pull/18.


Capstone chogum

Capstone-Project-CHOGM-2022
|This is a Capstone CHOGM meeting website
the 20th June 2022 - 25th June 2022, the Commonwealth Heads Of Government Meeting (CHOGM) took place in Rwanda, In this project, I have created a website that can be used in CHOGM meetings to inform and gives information about the meeting.
I have created this project with JavaScript, HTML, and CSS

Project Description video

Thank you Mr. @keneogu for reviewing my project,
Thank you for your motivating feedback too I really appreciate
I think it will be better for me to make some changes if I get the clear explanations on  " The about page background is not the same as the front page. it has a dark background. please fix this."  as suggested

If you look at the about page design on the given design you will notice it doesn't have the image background like the home page. you can check it on the mobile version, its clearer.

Thank you @keneogu, I have now understood what to do.


Include body and headers of requests on the logs

Is your feature request related to a problem? Please describe.
I need to see the body of incoming POST request on logs. By running:
docker run -d --mount type=bind,source=/home/username/your-data-file.json,target=/data,readonly -p 3000:3000 mockoon/cli:latest -d data -i 0 -p 3000

I see the logs like:
mockoon_1  | {"level":"info","message":"POST / | 200","timestamp":"2022-01-19T17:35:32.614Z"}
mockoon_1  | {"level":"info","message":"POST /users | 200","timestamp":"2022-01-19T17:35:48.956Z"}

It will be useful to see the body and headers of the request.
Describe the solution you'd like
Allow to specify by a parameter to increase the logs details of incoming http requests.

This is now possible with the --log-transaction flag. You will get the same information you see in the desktop app.
Be careful, logs will be quite big with this option.

@255kb thank you for the answer and doc!


[MM-17094] Spacebar now activates add button when focused on multiselect

Summary
When doing accessibility work with JAWS, we noticed that in tutor mode (on by default) it would inform the user to press spacebar to activate the Add button in the multiselect control. This PR adds the actual functionality for the spacebar, as before it wasn't working.
Ticket Link
https://mattermost.atlassian.net/browse/MM-17094

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3344.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

@esethna What browser/OS did you test with? I'm not able to reproduce on Chrome on Mac/Windows. Spacebar works fine for me.

Test server destroyed

@devinbinnie Please help cherry-pick to v5.15.


OCTT - TC_E_22_CS - TC_E_24_CS - TC_E_25_CS - Testcases Fail

OCPP Version
OCPP2.0.1
Describe the bug
Everest version: Everest 2024.5.0
Ocpplib version: 0.12.0
When we run this test case on the OCTT tool, it continues to the end of the test cycle but it fails with reason:
"Message field triggerReason should be ChargingStateChanged."
While the actual triggerReason we get back from the charger to the OCTT tool is "triggerReason": "StopAuthorized".
OCTT Logs:
pdspe_base_TC_E_22_CS2024-06-27T11_21_03.889802.log
pdspe_base_TC_E_24_CS2024-06-27T11_49_12.738301.log
pdspe_base_TC_E_25_CS2024-06-17T12_06_53.615617.log
To Reproduce
Execute the testcase in OCTT tool:
OCPP Version : OCPP 2.0.1
Build Number : Release_2024-04_v2
Anything else?
No response

Hi @msozen , thanks for reporting this! I could reproduce the issue and we are working on a fix. Looks like the issue was introduced with https://github.com/EVerest/libocpp/pull/594


changed "used" to "created"

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

@rjong999 : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@curtand - Please review the proposed changes.  Thanks

#sign-off

Hi @GitHubber17 ,
This seems ready. :)


[alpaka] Fix CachingAllocator for pinned host memory in presence of multiple devices

I noticed that alpaka (and alpakatest)  was crashing with
Processing 1000 events, of which 2 concurrently, with 2 threads.
terminate called after throwing an instance of 'std::runtime_error'
  what():  .../pixeltrack-standalone/external/alpaka/include/alpaka/event/EventUniformCudaHipRt.hpp(157) 'cudaEventRecord( event.m_spEventImpl->m_UniformCudaHipEvent, queue.m_spQueueImpl->m_UniformCudaHipQueue)' returned error  : 'cudaErrorInvalidResourceHandle': 'invalid resource handle'!

when run on 2 GPUs. I managed to trace the problem into the pinned host memory allocations in CachingAllocator, when a pinned host memory block was previously used on device A, and is then used on device B.
The current code does
// Take previous device from *the argument block*
auto const& previousDevice = block.device();
// Take the queue of the argument block
auto queue = std::move(*(block.queue));
// Copy the information of the block to be reused
block = iBlock->second;
// Assign the queue originally from the argument block
block.queue = std::move(queue);

Since the block.device() returns the device of its Queue, this means that both block.device() and previousDevice point to the same device, and the subsequent code to create a new Event never runs.
A minimal fix would be
-auto const& previousDevice = block.device();
+auto const& previousDevice = iBlock->second.device();

but given that the point is to create a new Event if the device of the Queue accessing the block differs from the device of the Event that the block had, I think testing explicitly that condition makes the intention of the code more clear.

Thanks @makortel , the change looks good - thanks for catching this and preparing a fix.


Correct error in checking for primitive data type

Describe the PR
This PR corrects the error in checking for primitive data type.
Relation issue
Bug introduces in https://github.com/swaggo/swag/commit/d74ed70e44450ffd5ca0016f0e78e2c569abbc45.
Additional context
I am seeing this error in the latest build:
$ swag init -g cmd/cnc.go
2019/09/11 11:35:02 Generate swagger docs....
2019/09/11 11:35:02 Generate general API Info, search dir:./
2019/09/11 11:35:02 ParseComment error in file cmd/cnc.go :can not find schema type: "cmd.int"

Codecov Report

Merging #516 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #516   +/-   ##
=======================================
  Coverage   86.78%   86.78%
=======================================
  Files           7        7
  Lines        1536     1536
=======================================
  Hits         1333     1333
  Misses        119      119
  Partials       84       84

Impacted Files
Coverage Δ

operation.go
92.6% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

My swag is failing when there is a string comment in @Success. e.g.
@Success 200 {string} OK

However, the latest release (v1.6.3, July 2019) doesn't support *ast.SelectorExpr. I am having some difficulties building a working doc now.


[FEATURE] Custom Icons for Moon Phase

Hi, honestly amazed with the module, probably my favorite one out of all of them. If possible, I think that the functionality to add a custom icon for moon phases that appear on the calendar would be useful. This would allow for irregular events such as blood moons or solar eclipses.

I do like this idea! It's quite minor, but some really cool things could be done by using the moons, and perhaps this could be extended to custom 'holidays'.

I can definitely look into allowing custom icons to be used for moon phases. Thanks for the suggestion!


Attrition is broken

...due to the CORS service was taken down: https://github.com/Rob--W/cors-anywhere

So we have to modify the backend to fetch the data and store them in our database.

A user-side workaround was added in df858dd


nixos/sshd: only include AuthorizedKeysCommand and AuthorizedKeysComm…

…andUser options if explicitly set

Motivation for this change
Fix for https://github.com/NixOS/nixpkgs/pull/82413
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

I am thinking of removing these options: now that we have openssh.settings, we dont need nixpkgs support for it: the user can set services.openssh.settings.AuthorizedKeyCommand and the validation of the config will raise an alarm if the user forgot to set AuthorizedKeysCommandUser. What do you think ?

Yeah definitely. That sounds good. Thanks for all your work on the ssh module @teto.

cool. Done in https://github.com/NixOS/nixpkgs/pull/215397


remove virtual ip labels

@jfrazelle changes like this require a new index... (which are hard to diff, but are expected by cosmos).  so this change in mysql/0 should result in a `mysql/`` with change.   don't forget to rerun the index scripts.

hmm i did but maybe on an old checkout

updated :)

wow no sleep is killing me, now its good

@jfrazelle perfect!

this service fails to install

ok we need to bump the memory for some reason on mine it doesnt need that much, will update thanks so much

ok i have bumped to 512

@jfrazelle I had the same failure... I bumped mem to 1024 and cpu to 1 and got it running.  I would add that the service doesn't recover well.  multiple attempts with failures resulted /tmp/mysql being established... then when a "good" config worked it failed because the dir existed.
Let me know what you want to do about the resource needs.  It isn't clear to me what the real needs of the service are.

Hmmm ya it's not exactly clear to me either what the resource needs I upped the values though :/

LGTM  merging!

thanks!
On Wed, Apr 20, 2016 at 1:13 PM, Ken Sipe<EMAIL_ADDRESS>wrote:

LGTM merging!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/mesosphere/universe/pull/423#issuecomment-212584579


infa(fetch) add hide option to the fetch script, hide deprecated package

Fixes #2573

We don't have access to add Netlify bot to publish a comment saying that the preview is ready, right?

We don't have access to add a Netlify bot that publishes a comment saying that the preview is ready, right?

We don't, it just needs to be enabled, I think @EugeneHlushko can take care of that. :)

Just added, lets see how it plays out @montogeek @anshumanv


Credentials Provider bug with powertools-idempotency and Lambda SnapStart

Hi! While using version 1.15.0 of powertools-idempotency and java11 Lambda runtime, I started using AWS SnapStart and my function broke (previously working fine). I suspect it is an issue with the credentials provider chain not using the container provider like the default chain is supposed to (which I understand is required to work with SnapStart). As a result, it currently seems impossible to use this library with SnapStart, as I cannot seem to override the credentials provider.
The error message below seems to clearly indicate this. NOTE: the exception looks like it went to System.out - in other words, it isn't a Log4J log message.
EDIT: I looked in the persistence code and found this, which seems to confirm my suspicions:
DynamoDbClientBuilder ddbBuilder = DynamoDbClient.builder()
                        .credentialsProvider(EnvironmentVariableCredentialsProvider.create())
                        .httpClient(UrlConnectionHttpClient.builder().build())
                        .region(Region.of(System.getenv(AWS_REGION_ENV)));
                this.dynamoDbClient = ddbBuilder.build();

in: DynamoDBPersistenceStore.java
Expected Behavior
The idempotency library works with AWS SnapStart as normal.
Current Behavior
The runtime cannot find credentials and therefore fails to write records to DynamoDB.
Possible Solution
Fix code to use default credentials provider chain that includes the container provider?
Steps to Reproduce (for bugs)
I suspect any SnapStart enabled Lambda will have this issue, but if not that would be great to know.
Environment

Powertools version used: 1.15.0
Packaging format (Layers, Maven/Gradle): Maven
AWS Lambda function runtime: Java11
Debugging logs

How to enable debug mode**

Failed to save in progress record to idempotency store. If you believe this is a powertools bug, please open an issue.: software.amazon.lambda.powertools.idempotency.exceptions.IdempotencyPersistenceLayerException
software.amazon.lambda.powertools.idempotency.exceptions.IdempotencyPersistenceLayerException: Failed to save in progress record to idempotency store. If you believe this is a powertools bug, please open an issue.
	at software.amazon.lambda.powertools.idempotency.internal.IdempotencyHandler.processIdempotency(IdempotencyHandler.java:91)
	at software.amazon.lambda.powertools.idempotency.internal.IdempotencyHandler.handle(IdempotencyHandler.java:66)
	at software.amazon.lambda.powertools.idempotency.internal.IdempotentAspect.around(IdempotentAspect.java:74)
	at com.mindcareone.auth.handler.PostConfirmationHandler.handleRequest(PostConfirmationHandler.java:81)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.base/java.lang.reflect.Method.invoke(Unknown Source)
Caused by: software.amazon.awssdk.core.exception.SdkClientException: Unable to load credentials from system settings. Access key must be specified either via environment variable (AWS_ACCESS_KEY_ID) or system property (aws.accessKeyId).
	at software.amazon.awssdk.core.exception.SdkClientException$BuilderImpl.build(SdkClientException.java:111)
	at software.amazon.awssdk.auth.credentials.internal.SystemSettingsCredentialsProvider.resolveCredentials(SystemSettingsCredentialsProvider.java:58)
	at software.amazon.awssdk.core.internal.util.MetricUtils.measureDuration(MetricUtils.java:50)
	at software.amazon.awssdk.awscore.internal.authcontext.AwsCredentialsAuthorizationStrategy.resolveCredentials(AwsCredentialsAuthorizationStrategy.java:100)
	at software.amazon.awssdk.awscore.internal.authcontext.AwsCredentialsAuthorizationStrategy.addCredentialsToExecutionAttributes(AwsCredentialsAuthorizationStrategy.java:77)
	at software.amazon.awssdk.awscore.internal.AwsExecutionContextBuilder.invokeInterceptorsAndCreateExecutionContext(AwsExecutionContextBuilder.java:120)
	at software.amazon.awssdk.awscore.client.handler.AwsSyncClientHandler.invokeInterceptorsAndCreateExecutionContext(AwsSyncClientHandler.java:69)
	at software.amazon.awssdk.core.internal.handler.BaseSyncClientHandler.lambda$execute$1(BaseSyncClientHandler.java:78)
	at software.amazon.awssdk.core.internal.handler.BaseSyncClientHandler.measureApiCallSuccess(BaseSyncClientHandler.java:179)
	at software.amazon.awssdk.core.internal.handler.BaseSyncClientHandler.execute(BaseSyncClientHandler.java:76)
	at software.amazon.awssdk.core.client.handler.SdkSyncClientHandler.execute(SdkSyncClientHandler.java:45)
	at software.amazon.awssdk.awscore.client.handler.AwsSyncClientHandler.execute(AwsSyncClientHandler.java:56)
	at software.amazon.awssdk.services.dynamodb.DefaultDynamoDbClient.putItem(DefaultDynamoDbClient.java:4243)
	at software.amazon.lambda.powertools.idempotency.persistence.DynamoDBPersistenceStore.putRecord(DynamoDBPersistenceStore.java:148)
	at software.amazon.lambda.powertools.idempotency.persistence.BasePersistenceStore.saveInProgress(BasePersistenceStore.java:162)
	at software.amazon.lambda.powertools.idempotency.internal.IdempotencyHandler.processIdempotency(IdempotencyHandler.java:84)
	... 7 more

Thanks for reporting this. Before the fix is released you can create your own DynamoDbClient and pass it in to the withDynamoDbClient method of DynamoDBPersistenceStore.
For example:
Idempotency.config().withConfig(
                IdempotencyConfig.builder()
                        .withEventKeyJMESPath("powertools_json(body).address")
                        .build())
        .withPersistenceStore(
                DynamoDBPersistenceStore.builder()
                        .withDynamoDbClient(client)
                        .withTableName(System.getenv("IDEMPOTENCY_TABLE"))
                        .build()
        ).configure();

@mhermus, it would be good to chat about the fix if possible msailes at amazon dot co dot uk

fixed in #1161

Released in 1.16.0


[Source Request] Multi sources

Anime related sites:

Name: Okanime
Link: https://okanime.tv
Language: Arabic / العربية

Name: Shahiid Anime
Link: https://shahiid-anime.net/
Language: Arabic / العربية

Name: شبكة كونان العربية
Link: https://detectiveconanar.com/
Language: Arabic / العربية
Note: It has everything related to Case Closed (Ova, Movies, etc)

Name: Animeiat أنميات
Link: https://www.animeiat.tv/
Language: Arabic / العربية

Name: Anime Titans
Link: https://animetitans.com
Language: Arabic / العربية
Note: Good alternative of Animeiat

Name: Anime Silver
Link: https://www.animesilver.com
Language: Arabic / العربية
Note: Decent alternative of Animeiat

Manga related sites:

Name: 3asq العاشق
Link: https://3asq.org/
Language: Arabic / العربية

Closing as multiple source requests were made. Please make a separate issue for each extension.


slk_appropriateKeyboardHeight not returning correct value

Device iOS version: 8.0.2
Base SDK: 7.0
Problem:
In iPad landscape, when showing the keyboard height is so big that it is impossible to see the conversation.
Why:
The slk_appropriateKeyboardHeight  is returning a value related with the device width instead of its height. The SLK_IS_IOS8_AND_HIGHER macro is only taking in account the device version, so it is entering the else statement. In this case should take in account the Base SDK version to invert width and height using SLKRectInvert(endFrame).
Code related:
- (CGFloat)slk_appropriateKeyboardHeight:(NSNotification *)notification
{

   ...

    if(!SLK_IS_IOS8_AND_HIGHER &&
       (endFrame.size.width == bounds.size.height || endFrame.size.height == bounds.size.width)) {
        endFrameConverted = SLKRectInvert(endFrame);
    }
    else {
        endFrameConverted = endFrame;
    }

  ...

}

Another problem I had is that when dismissing the keyboard by dragging it, at some point the endFrameConverted.origin.y is negative, and it causes the screen to turn blank for a moment.
Easy fix:
- (CGFloat)slk_appropriateKeyboardHeight:(NSNotification *)notification
{

   ...

if (keyboardHeight < 0 || endFrameConverted.origin.y < 0) {
        keyboardHeight = 0.0;
    }

    return keyboardHeight;
}

Of course, submitting it in a moment. :+1:


Intersept network requests to the admin api and add auth automatically.

Is your feature request related to a problem? Please describe.
If a sidekick plugin wants to make requests to the admin api, it cannot easily obtain the auth token.
Describe the solution you'd like
Network requests to the admin api just work
Suggestion:
use the https://developer.chrome.com/docs/extensions/reference/declarativeNetRequest/ api to intercept the requests to the admin api and inject the auth header automatically.

💯
This is a relatively large issue for our Franklin migration.

I think https://developer.chrome.com/docs/extensions/reference/declarativeNetRequest/ is a manifest v2 API.

I think https://developer.chrome.com/docs/extensions/reference/declarativeNetRequest/ is a manifest v2 API.

ah... I think, I mixed this up with https://developer.chrome.com/docs/extensions/reference/webRequest/

Wait, you were actually right the first time 😁  - webRequest is v2, declarativeNetRequest is the equivalent in v3 👍

Correction: webRequest is still available in v3, will use that.

Re-correction: webRequest won't work because request blocking has been removed in v3. So declarativeNetRequest is the way to go after all.


Revert using of CL_HAS_ANON_STRUCT in cl.h

This PR addresses #121, by reverting changes on cl.h of PR #116

 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.Yilong Guo seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I also noticed that the compile options for MSVC and gcc/clang are a little different in test builds:
    # ICD on Windows uses system headers, which aren't strictly ANSI conformant
    # and trigger warnings
    if(${NAME} STREQUAL cl_icd_h)
      set(MSVC_COMPILE_DEFS /W4 /WX)
    else()
      set(MSVC_COMPILE_DEFS /W4 /WX /Za)
    endif()
    target_compile_options(${TEST_EXE}
      PUBLIC
        $<$<NOT:$<CXX_COMPILER_ID:MSVC>>:-Wall -Wextra -Werror -pedantic -Wno-format>
        $<$<CXX_COMPILER_ID:MSVC>:${MSVC_COMPILE_DEFS}>
    )

gcc uses -std=gnu90 before v5.1 and -std=gnu11 starting from v5.1 (released in 2015). gnuXX means cXX standard with GNU extensions. And C11 starts to support anon unions.
Setting /Za will force MSVC conform with the ancient C89 standard, which doesn't allow anon unions. That's why the test build was failing.
I guess the anon union used in cl_image_desc is for OpenCL 1.2 compability, which is probably inevitable. So why not enable MSVC's extensions to offer same capability as GCC for anon unions?

Abandoned. See PR #125.


Add option to rename wallet straight from a dotted menu in the top right

https://zpl.io/VkPqp8g

@colourfreak  Not a biggie now, but there's a bunch of unrelated functions tucked under the … button. Copy Address, Show My Wallet Address and Rename This Wallet are related to the wallet/address, maybe we can look into moving them somewhere in the future? Maybe when the title "tomek.eth" is tapped or when the blockie/ENS avatar is tapped?

That was fast. I am impressed how things just speeded up :)


The wrong Chinese documentation

I checked the Chinese documents of minio-go and found that the contents of the Chinese documents have been out of date for a long time but have not been corrected in time. I hope to update the contents of the Chinese documents in time for future maintenance

Those are stale and not maintained anymore. Feel free to send a PR currently it is not a priority.


Unable to execute command line tool to dump script to single output file

Issue
When attempting to use Marten.CommandLine to dump a single database script out to the file system I get the following exception:

ERROR: System.UnauthorizedAccessException: Access to the path 'c:\temp\foo\db.sql' is denied.
at System.IO.FileStream.ValidateFileHandle(SafeFileHandle fileHandle)
at System.IO.FileStream.CreateFileOpenHandle(FileMode mode, FileShare share, FileOptions options)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options)
at System.IO.StreamWriter.ValidateArgsAndOpenPath(String path, Boolean append, Encoding encoding, Int32 bufferSize)
at System.IO.StreamWriter..ctor(String path)
at System.IO.File.WriteAllText(String path, String contents)
at Baseline.FileSystem.WriteStringToFile(String filename, String text)
at Marten.Storage.TenantSchema.WriteDDL(String filename, Boolean transactionalScript)
at Marten.CommandLine.Commands.Dump.DumpCommand.execute(IDocumentStore store, DumpInput input)
at Marten.CommandLine.MartenCommand1.Execute(T input)    at Oakton.OaktonCommand1.Oakton.IOaktonCommand.Execute(Object input)
at Oakton.CommandRun.Execute()
at Oakton.CommandExecutor.execute(CommandRun run)

Assume the (Windows) directory "c:\temp" already exists, then provide the following command line parameters:
dump c:\temp\database.sql
Instead of trying to create a "database.sql" file within "c:\temp" the code seems to create a directory called "database.sql" within "c:\temp".  It then attempts to write the file content to this directory (thinking it's a file) which then throws the above exception.
Steps to reproduce

Create a project that uses Marten.CommandLine
Configure the options as per https://martendb.io/documentation/cli/
Execute the command to dump a single script out to the file system:
my-project.exe dump c:\temp\database.sql

Investigation
I've had a look through the code and I think the issue is linked to this call within DumpCommand.cs
new FileSystem().CleanDirectory(input.FileName);
https://github.com/JasperFx/marten/blob/fec62b50a0d30bc6b8860d76e9ed3f502d9d3052/src/Marten.CommandLine/Commands/Dump/DumpCommand.cs#L29
The CleanDirectory method within Baseline FileSystem seems to always treat the input string as a directory, deleting it if it exists, then creating the directory - thus we end up with a directory on the file system with the same name as our desired output file.

@lowds I will have a look at this issue and revert.

@lowds confirming this to be a bug, will look at providing a fix in the next release. In the interim, either pass just file name like database.sql or folder c:\temp. Failure happens only when a full path is passed containing both folder and file name fails.

@mysticmind I'm into things today, I can deal with this real quick


snaps for Linux

hey there,
I wonder if you have considered packaging the CLI app in a snap and distributing it via https://snapcraft.io/?
would be really cool to have a homebrew-ish way of installing it on Linux :)

Hi @antonpaisov thank you for reaching out. We will discuss this and come back to you! Definitely considering it, though there a few things that we might do first (regarding the client).

Hi @antonpaisov. Providing snaps is a good idea but currently not on our roadmap. I will close this issue for the time being and reopen it when things change on our end.

@tobstarr, @sacry-dyn sure! I'll try to submit a PR with a yml for snapcraft when I have time, so you'll be the owners of that snap (I believe configuration there is quite simple)


Customizing property with private/protected setter

Consider we have a class to create a fixture like this:
class MyClass {
    var name: String = "initial value"
        protected set
}

when we do:
kotlinFixture {
    property(MyClass::name) { "test" }
}

will throw an exception like this:
java.lang.UnsupportedOperationException: Unable to handle MyClass
Unable to resolve class MyClass
    Unable to populate class MyClass
    Unable to instantiate class MyClass using factory methods

    at com.appmattus.kotlinfixture.Fixture.create(KotlinFixture.kt:85)

because the proeprty name doesn't provide public setter.
How do I customize that field?

This is somewhat linked to issue #90
There are effectively 2 issues happening here.
Firstly, there's some validation that occurs that won't let you define that property override. It does this because MyClass::name is not a mutable property and so it checks to see if it appears in the constructor, as that would then be the only way for that override to work. Of course the property doesn't exist in the constructor so it fails because of the validation  exception.
Secondly, if I remove the validation, the object will be created but the value will not be overridden as currently the overrides only work on public setters or constructor parameters.
Thinking about the other open issue (#90) which relates to latent vars I realised that there's no way to easily ignore property setting; and likewise customise properties that would otherwise not be set which is what you want here. This is something I'm currently looking at.


Can't access plugin settings

I'm moving Indigo to a different computer, so have cloned the OS Drive to a portable drive and have cloned the copy to the internal drive of the target iMac.  OS for Indigo is 10.15.6, and Indigo V7.5
I know enough about networking so have set up the target iMac to get the same IP address from the router as the previous Mini running Indigo. I'm using Geofency , on port 6192, and that port is forward on the router to the same old IP address used on the mini, now being used in the iMac running Indigo.  I've restarted everything, and access from the iOS Touch app as well as indigo web pages work just fine. Connections to those ports work as expected.
In reading up on this, it seems I should be able to have a "settings" menu in the Beacon plugin to change the port number, but don't. I've tried re-installing the plugin, but that hasn't worked either.
From my phone, when trying to send  Enter/Exit data, all I get is a error message:  "Failed. Cannot connect to the server. "
While I guess its possible my ISP has started blocking this port, the chances of that happening at the same time as moving my Indigo server seems remote at best.
I've also tried creating a new "fence", but even that gives the same error message.
I'm guessing since I can't access the "settings" in the plugin menu, that's where my problem is.

Run the Indigo installer again to make sure all the permissions are set correctly.

Hey Joe
Thank you so much for your speedy reply.
I was having some other issues, so have since done what you’ve suggested… re-installed Indigo… and after that install, I re-installed the plugin for good measure as well.
Still only have Disable, Reload, Show… About… options in the plugin.
I manually deleted the plugin, re-started the computer, re-installed the plugin, but still not working.
As far as permissions go, which ones (file/folder) might be incorrect?
FYI I’m on 1.5.3 for the plugin.
I don’t think its a networking issue… the firewall is OFF in the iMac… but just can’t figure out what be causing this to not work……. I even tried creating a new database just to see… but still not working...
Jeffrey

On Apr 6, 2021, at 4:14 PM, Joe Keenan @.***> wrote:
Run the Indigo installer again to make sure all the permissions are set correctly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/IndigoDomotics/Beacon/issues/2#issuecomment-814444187, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMEY7QZCQU4TJU34HFI7A3THN2TBANCNFSM42PO6ERQ.

Check inside the plugin package and make sure the PluginConfig.xml file is there.  And has the right permissions.
jkeenan@Kraken Downloads % ls -lR Beacon.indigoPlugin
total 0
drwxr-xr-x@ 5 jkeenan  staff  160 Dec  8  2017 Contents

Beacon.indigoPlugin/Contents:
total 8
-rwxr-xr-x@ 1 jkeenan  staff  710 Dec  8  2017 Info.plist
drwxr-xr-x@ 3 jkeenan  staff   96 Dec  8  2017 Resources
drwxr-xr-x@ 6 jkeenan  staff  192 Dec  8  2017 Server plugin

Beacon.indigoPlugin/Contents/Resources:
total 40
-rw-r--r--@ 1 jkeenan  staff  17556 Dec  8  2017 icon.png

Beacon.indigoPlugin/Contents/Server plugin:
total 72
-rwxr-xr-x@ 1 jkeenan  staff   1538 Dec  8  2017 Devices.xml
-rwxr-xr-x@ 1 jkeenan  staff   1801 Dec  8  2017 Events.xml
-rwxr-xr-x@ 1 jkeenan  staff   4925 Dec  8  2017 PluginConfig.xml
-rwxr-xr-x@ 1 jkeenan  staff  16949 Dec  8  2017 plugin.py
jkeenan@Kraken Downloads %

Hey Joe
First of all, I’m back in business!
The short story is while doing all this migrating, and running into problems(Beacon was the one that stood out, there may have been others….), I decided to give Big Sur a try since it was already loaded on the target iMac. And since I had the Indigo 7.5 client loaded, it was simple enough to go ahead and manually move all the plugins, and their settings to Big Sur, and start the Indigo Server from there.
As of now, all is well,
I did go back and check the permissions from the old Catalina/Indigo server, and I’m attaching a pic as the all look correct… at least they all look like my working copy on Big Sur. Still not sure what was wrong… but there could have been some OS migration issues, so I’m going back to take a look at those as well. If I find anything, I’ll keep you posted.. I honestly thought this was going to be a 20 minute task, but its taken all afternoon. But, there are worse ways to spend my time for sure.
Again, thank you so much for developing this plug in… I use Geofency day in and day out… and I also appreciate your taking time to help with this particular issue.
Ill let you know if I find anything else.
Cheers
Jeffrey Talbot
Here’s the link for the pic:
http://jeffreytalbot.com/indigo/Screen Shot 2021-04-06 at 6.14.44 PM.png http://jeffreytalbot.com/indigo/Screen Shot 2021-04-06 at 6.14.44 PM.png

On Apr 6, 2021, at 6:27 PM, Jeffrey Talbot @.***> wrote:
Hey Joe
First of all, I’m back in business!
The short story is while doing all this migrating, and running into problems(Beacon was the one that stood out, there may have been others….), I decided to give Big Sur a try since it was already loaded on the target iMac. And since I had the Indigo 7.5 client loaded, it was simple enough to go ahead and manually move all the plugins, and their settings to Big Sur, and start the Indigo Server from there.
As of now, all is well,
I did go back and check the permissions from the old Catalina/Indigo server, and I’m attaching a pic as the all look correct… at least they all look like my working copy on Big Sur. Still not sure what was wrong… but there could have been some OS migration issues, so I’m going back to take a look at those as well. If I find anything, I’ll keep you posted.. I honestly thought this was going to be a 20 minute task, but its taken all afternoon. But, there are worse ways to spend my time for sure.
Again, thank you so much for developing this plug in… I use Geofency day in and day out… and I also appreciate your taking time to help with this particular issue.
Ill let you know if I find anything else.
Cheers
Jeffrey Talbot
<Screen Shot 2021-04-06 at 6.14.44 PM.png>

On Apr 6, 2021, at 4:58 PM, Joe Keenan @.*** @.***>> wrote:
Check inside the plugin package and make sure the PluginConfig.xml file is there. And has the right permissions.
jkeenan@Kraken Downloads % ls -lR Beacon.indigoPlugin
total 0
drwxr-xr-x@ 5 jkeenan  staff  160 Dec  8  2017 Contents
Beacon.indigoPlugin/Contents:
total 8
-rwxr-xr-x@ 1 jkeenan  staff  710 Dec  8  2017 Info.plist
drwxr-xr-x@ 3 jkeenan  staff   96 Dec  8  2017 Resources
drwxr-xr-x@ 6 jkeenan  staff  192 Dec  8  2017 Server plugin
Beacon.indigoPlugin/Contents/Resources:
total 40
-rw-r--r--@ 1 jkeenan  staff  17556 Dec  8  2017 icon.png
Beacon.indigoPlugin/Contents/Server plugin:
total 72
-rwxr-xr-x@ 1 jkeenan  staff   1538 Dec  8  2017 Devices.xml
-rwxr-xr-x@ 1 jkeenan  staff   1801 Dec  8  2017 Events.xml
-rwxr-xr-x@ 1 jkeenan  staff   4925 Dec  8  2017 PluginConfig.xml
-rwxr-xr-x@ 1 jkeenan  staff  16949 Dec  8  2017 plugin.py
jkeenan@Kraken Downloads %
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/IndigoDomotics/Beacon/issues/2#issuecomment-814464360, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMEY7T3T5NDLCGAC7JOVWLTHN7Y7ANCNFSM42PO6ERQ.

I didn't develop the plugin.  The original author abandoned it, and it's now in the IndigoDomo open source repository.  Which I help manage.  But I've never even looked at this plugin.  ;)


make sure content-type is signed

For a request like:
aws s3 cp --endpoint-url https://cloudflare-b2-proxy.foo.workers.dev hello.txt s3://foo/hello.txt

the request Authorization header's SignedHeaders also includes the content-type header.
aws --version
aws-cli/2.9.23 Python/3.11.1 Darwin/20.6.0 source/x86_64 prompt/off

2.9.23 was released on Feb 10, 2023 so I suspect this is newer behavior.
https://github.com/mhart/aws4fetch/blob/master/src/main.js#L25 appears to ignore that header.
Before:
In the request Authorization header:
SignedHeaders=content-md5;content-type;host;x-amz-content-sha256;x-amz-date
In the signedRequest Authorization header:
SignedHeaders=content-md5;host;x-amz-content-sha256;x-amz-date
After:
In the request Authorization header:
SignedHeaders=content-md5;content-type;host;x-amz-content-sha256;x-amz-date
In the signedRequest Authorization header:
SignedHeaders=content-md5;content-type;host;x-amz-content-sha256;x-amz-date

May also resolve https://github.com/backblaze-b2-samples/cloudflare-b2-proxy/issues/7

Good catch. Omitting allHeaders means that aws4fetch filters the headers against its list of 'unsignable headers'. We never want this when verifying an incoming signature - we always want to include all the signed headers from the incoming request.
Many thanks!


Build error on windows

Make it compatible with windows or convert error to warning, please.

The idea was that the user would only import it on linux using version(linux). I am open to making it use pragma(msg, "...") instead of the assert but is there a reason not to version it from user code?
And in regards to implementing this on Windows, I don't have a machine to test or develop. A pull request to implement it would be great though.

I simply add dependency to dub.sdl:
dependency "backtrace-d" version="*"

And catch this error:
Error: static assert  "backtrace only works in a Linux environment, the advanced backtrace won't be attached."

Where i can insert version(linux)?

dub automatically compiles all dependencies. Although there is a way to specify different dependencies depending on architecture, I've simply replaced the static assert with pragma(msg, ...) because it's easier for users.
The symbols provided by the library are also versioned out on non-linux systems. So users must version their usage.
If there is still difficulty in using the library, report it please.
The changes are in version v0.1.1 on dub.

Thank you :-) How about autoinstall when can?

Uhmm, I am not sure it is the best thing to do, especially that you can use this library to get the stack trace without installing the printer.
I will close this issue because it is about the Windows thing. If you want to discuss the auto-install feature, please open another issue.


Rewrite optional-field.deno test to Rust.

The rewrite omits the last part of the test:
## Correctly populate from optional fields
$CHISEL apply --version=v2
# CHECK: Applied:
$CHISEL populate --version=v2 --from=dev
$CURL $CHISELD_HOST/v2/sum
# CHECK: HTTP/1.1 200 OK
# CHECK: the sum is: 1100;

Which is testing population of optional fields. This is because from what I've seen in the populate.deno file, this seems to be covered by populating optional ceo field. @dekimir as you are the author of the test, are you ok with this, or should I add an explicit population test as well?

Yeah I had it all in a macro, but it was weird and didn't work for array and Entity... This looked like the best compromise between length and readability.

Oh yeah, macro sounds nasty; would not a plain function work? On a cursory look, the only difference between number and string seems to be in the type name given to write_files() and the JSON values given to requests and expected in responses.


Add missing properties to balanceBuilder

Summary
The balanceBuilder is missing fiatBalance and fiatConversion properties/mock values. This adds them with a default value of null.
Changes

Add fiatBalance and fiatConversion to balanceBuilder.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7161

Relevant Lines:
7425

💛  - Coveralls


Add reading list for event based sensing

out of interest, what is the difference to this? https://github.com/uzh-rpg/event-based_vision_resources

You are completely right, I completely forgot about it :rofl: We can close this issue


Downtime during Flux upgrades

Hello,
Whenever our flux instance upgrades it uninstalls all the application code, and reinstalls it using the latest version of the CRDs. The terraform we are using to do the install uses the flux_install and flux_sync data sources. These uninstall and reinstall all our application code during upgrades, which causes an extended outage. This appears to be because all the git repositories are removed and then created again, causing Flux to trigger reconciliation on all installed code.
If we switch to flux_bootstrap_git has this been addressed, or will we still see the same behaviour?

Using flux_bootstrap_git the upgrade is seamless, no reconciled apps are touched.

@stefanprodan I think we can close this.


MINOR - Clean ingestion dependencies

Describe your changes:
We got some feedback that the ingestion package was too big and users were having issues installing it on AWS Lambda.
The goal of this PR is to:

Remove packages I can't seem to find being used anywhere
We had some need from sample data (avro, grpc) that might not be needed on the base requirements. Moved them to sample-data to ease our tests. They are in their own plugin for other required connectors
The run_local_docker did not properly print out the Sample Data DAG logs. Updated the CI to make sure we have all the necessary information

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@pmbrull how big is the package after these optimizations? looks like lambda limits is 250MB from what I was reading

@pmbrull how big is the package after these optimizations? looks like lambda limits is 250MB from what I was reading

make install on this branch + du -sh venv-size/lib/python3.10/site-packages gives us 170M
make install on 1.3.2 + du -sh venv-size-132/lib/python3.10/site-packages gives us 238M


Fix to denominator of Cook-Torrance brdf

I believe that there are parentheses missing in the denominator for the Cook-Torrance BRDF.  This would add a possible divide-by-zero error, so I padded the dot products a little. Does this seem correct to you?

Hi @moneimne, something doesn't look right here.  Check out the look of "MetalRoughSpheres" before and after this change, all the spheres look the same for me when this change is applied.

Idea for applying PBR to geospatial: Alternating metallic/roughness properties at runtime to achieve certain material effects. For example, one can add highlights or rim lights onto a model to represent selection by manipulating metallic and roughness.

According to equation 2 in http://blog.selfshadow.com/publications/s2013-shading-course/karis/s2013_pbs_epic_notes_v2.pdf , the fix should be correct. If the rendered output is not correct, then

either something else is wrong, which was "compensated" by the current (otherwise "wrong") implementation
or the current implementation (without the fix) does not correspond to what the PBR implementation guides are usually writing

(The latter is unlikely , but not impossible. I used the same formula as in the commit during my PBR experiments, e.g. https://github.com/javagl/gltfTestModels/blob/master/SimpleSpheres/glTF/pbr.frag#L168 - including the arbitrary padding. And as mentioned as a comment there: The Epic notes refer to the original Cook paper for equation 2. But (although I admittedly haven't read the Cook paper in its entirety) I tried to "match" the formula from the Epic paper to any formular written in the Cook paper, and if there is a relationship, it's not obvious - at least, not for me... )

Thanks for the input @javagl.  I agree that the code change looks like it should bring the shader more in line with published formulas.  But when I run with this change, this is what I see:

In master, without this change, I get an image that's much closer to what I would expect:

@snagy Any thoughts on this?

One more data point:  With this change, when I hover the mouse over the word "metallic" or "roughness", all of the spheres turn black.  The expected behavior is that the spheres along the top should turn white when hovering "metallic", and the spheres down the right side (in the image above) should turn white when hovering "roughness."

when I hover the mouse over the word "metallic" or "roughness", all of the spheres turn black

I think I found the reason. I created a new issue for this: #11
As for the Cook-Torrance BRDF, Naty Hoffman describes the terms in the denominator as "correction factors" towards the other terms. There is a SIGGRAPH presentation by him that I found very intuitive if anyone feels like watching.

I could only hazard a guess. (This could be a tad easier if one could try out how the rendering "behaves" when rotating the camera and changing the other settings).
But considering the influencing factors of the BRDF, I'm a bit confused about the u_Camera: This seems to be initialized to 0,0,-4 and then computed in a very creative way from the view matrix at https://github.com/KhronosGroup/glTF-WebGL-PBR/blob/82fc52fdc8f1a57a018ad81330319007f7d0284f/scene.js#L131
From what I've read so far, I think that all these computations take place in view space, and the camera position in view coordinates should be fixed at (0,0,1). (At least, that's what I assumed at https://github.com/javagl/gltfTestModels/blob/master/SimpleSpheres/glTF/pbr.frag#L149 , but that may be wrong, of course)
Again: I'm not sure if this could be a reason for the effect that you're observing now, but might be related, and is something to have a look at in any case...

Hi folks!  Sorry for the slow response here.
The initial issue (missing parenthesis) was entirely a bug; the entire 4NdotLNdotV term should be the denominator of the equation.  I'm seeing the same problem as the 'bright hemispheres' above and I'm tracking that down now.
With regards to the space that all the math is happening in, I believe it is all consistently happening in world space.  u_Camera, v_Position and the transformed normal are all in world space.
I'll look at this a bit more and update with a working fix for the denominator problem.

I looked a little bit into the issue as well, and it seems that by padding the dot products to avoid dividing by zero, I end up blasting the BRDF value to a really large one when NdotL is almost zero. I would probably approach the math a little differently than written.
Since we are evaluating the light transport equation, we want to determine the outgoing light at any given point on the mesh.

Le refers to the emitted light at that point, but that is handled later by the emissive texture (if any).
Since we are only dealing with delta lights when computing the BRDF, I think we can ignore the integral since there is only one valid incident light direction for any given point.
Li is the incident light along the one valid direction, which I believe is equal to the light's color * NdotL.
Most of the work we are doing is computing the BRDF, fr, which is equal to the diffuse BRDF + the specular BRDF.
Then, I think what we are missing is the multiplication of the cos(theta) term, which should be NdotL. This would act as a "visibility check" for geometry that isn't in view of the light source, causing the bright areas to be black (or close to it).
Overall, this would mean that you could compute the color as:
color = (lightColor * NdotL) * (lambertianDiffuse + F*G*D/(4*NdotL*NdotV)) * NdotL;
Does this make sense? When I follow this logic, I become confused where the M_PI term and 1.0 - F terms come from in specContrib and diffuseContrib, respectively.

That makes sense, @moneimne .  I had come to the same conclusion and written my spec contribution without the last * NdotL (to avoid double applying the light visibility term) but I'm still getting some flickering pixels around the edges when NdotV is going to 0 but NdotL is still relatively large.
I'm currently going through the other math to look at what I should expect to happen as NdotV goes to 0.

With regards to the space that all the math is happening in, I believe it is all consistently happening in world space. u_Camera, v_Position and the transformed normal are all in world space.

Now, this is becoming quite a bit more confusing for me. In https://github.com/KhronosGroup/glTF-WebGL-PBR/blob/82fc52fdc8f1a57a018ad81330319007f7d0284f/scene.js#L145 it looks like the normal matrix in this implementation is simply the model matrix (!). I thought that the convention was normalMatrix = transpose(inverse(modelViewMatrix)), and the computations take place in eye space.  But ... well, one can probably (technically and mathematically) do the computations in arbitrary spaces (as long as they are consistent).

The visual result of this branch is looking much better to me now.  The only difference I can spot with the naked eye is just the size of the specular highlights has been slightly reduced:

@javagl maybe file a new issue for the normal transform matrix questions?
If there are no further objections, I will merge this.

@emackey I think the highlight change is expected with the fix, and I'm sure the fix is more correct than it was previously.
@javagl the math should work the same in either space; I preferred to do it in world space for the reference implementation because I find it easier to break down into steps and understand, which I considered to be one of the goals of this implementation.  I should have renamed the 'normal matrix' to properly match the new usage as a model -> world matrix.

@snagy Thanks.  In fact I think I had previously complained to a coworker that the highlights "looked too large" but I didn't have any math to back up my claim.  Glad they're fixed!


Website content translation to Russian

Website content translation to Russian

https://docs.google.com/document/d/1hxN_6TQY3m53oULviD3YY1sduAGYITOPIg21TO7yxdo/edit


Also show kernel modules like [uprobes].

However, those modules can not be loaded.

What is the purpose of showing these modules? Can you add a "why" to the PR description? Is there a buganizer bug that you could link?

As discussed, this should not have the UI changes, right? Are you going to close this one or change it as we agreed earlier?


Update hardware-requirements.md

Added a few edits to make it more clear for the end users.

@robinharwood, @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team