[Bug]: Responsive issue on home-page when width is below 400px

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
In the homepage home page is not responsive like there is some extra width when the width size is below 400px
Expected behavior
It should be responsive below 400px and should not have extra width
Add ScreenShots
Instead of ScreenShots i am adding video where you can easily see the issue
https://github.com/VaibhavArora314/StyleShare/assets/116784047/5d657b76-3ee9-4142-8b7c-bc6c473d8f72
On which device are you experiencing this bug?
Linux
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

This is probably due to "Welcome to Style Share" text causing an x-overflow

I think this has been resolved


ci: Add GitHub action for style checkers

Run isort 5 and black for every pull request or push.
You can see this working in Ana06/capa

The idea is to add jobs to the same action for rule linter and tests and to show a badge in the README with the status of the workflow (the name of the badge is the same as the workflow - CI). See https://github.com/fireeye/capa/pull/151
Closes https://github.com/fireeye/capa/issues/75
Closes https://github.com/fireeye/capa/issues/76

GitHub Actions are working! 🥳 As @williballenthin said I can merge it... there we go! 🚢


2 listView  inside the same entity

Hey guys. Thank you for your hard work.
I was wondering if it was possible to have 2 listViews in the same page. One is linked to the entity "user" for exemple and the other is linked to the entity "author". And each of these lists have their own fields.
Of course each enity will have its own baseApiUrl.

Have a look at how the [dashboard template] (https://github.com/marmelab/ng-admin/blob/master/doc/Dashboard.md#dashboard-template) is working. Its doing more or less what you are requesting

Thank you, that's what I've been looking for. I guess I need to add  on each dashboard to add the create button relative to author and user.

Great. Will close this for now. Reopen it, or open another ticket if needed.


Add children to Wrapper in config

Resolves #93

Thank you! Pre-released as  0.4.6-commit-b59df26.


Not Working Anymore?

I used the tool to make a PNG, and it does work and open. But when uploaded and downloaded through Twitter, it doesn't work anymore. It seems like the image has been compressed to a couple kilobytes. Any ideas? I tried using the original image in the tweet that David posted, but even that didn't work.

Still working: https://twitter.com/David3141593/status/1374073253350760459


Setup Fail2Ban in port 1194

Hi @Nyr
Great script! Just one question:
I want to have Fail2Ban protecting the VPN 1194 port and after researching i have found 2 websites with 2 different configurations. I leave the links for both sites bellow.
http://eon01.com/blog/brute-force-secure-openvpn-with-fail2ban/
https://blogs.fsfe.org/stefan.a/2014/03/30/monitor-block-openvpn-logins-fail2ban/
Can you please help me and tell me the proper way to setup Fail2Ban in order to work with your script to protect VPN 1194 port?
Thanks and keep up the good work!

This is unrelated and not an issue with the script.
OpenVPN community wiki has notes on server hardening. The related section is "Use of --tls-auth"

@auyongcheemeng
Yes i know there is not any issue with the script and i'm sorry to have posted here. But i did not find other place where i can clarify this Fail2Ban question.

@auyongcheemeng is right, what do you want protection from?
Fail2ban is useless, specially considering that tls-auth is set up by default.
Just forget about it.


Fix - status bar remains hidden occationally

https://github.com/dekatotoro/SlideMenuControllerSwift/blob/master/Source/SlideMenuController.swift#L463
case UIGestureRecognizerState.Ended, UIGestureRecognizerState.Cancelled:
            if RightPanState.lastState != .Changed {
                return
            }

When this condition occurs, it just returns without setCloseWindowLevel()
and the status bar remains hidden even though the side bar is closed.
You can reproduce the issue by dragging the sidebar very shortly (and short distance) and release it straight away. (Menu controller - viewWillAppear is called, but RightPanState.lastState != .Changed is true)
Removing the conditions from L465-L464 fixed the issue for me.
I'm not sure if it affects on somewhere else.

It cannot reappear, is this case state begin?

Do you mean you cannot reproduce the issue?

I can reproduce and the correction seams to be fine

@sydneyitguy @gbitaudeau
I am not able to reproduce this issue, but fix is I think OK!
I will merge.

👍


Scheduled posts appear in tag listing

When visiting a tag archive page it shows scheduled posts in the future. I expect these posts not to display.

Good catch. Cheers, @trovster!

I don't think this is the best solution. The other areas of the site make use of "draft" flag as well as the date. This only uses date. It could also generate an empty page if all the tags' posts are "draft".


Favicon with badges - notifications

My suggestion is to exchange early notifications with a new model.
OLD: Favicon + (notification digit) + Title
NEW: Favicon with badges + Title
This way we will save some space.
Suggested library: http://lab.ejci.net/favico.js/

Which file would need to change here?
I can't tell from looking at
https://github.com/go-gitea/gitea/blob/73382d2132f8f242245d83f66c92761e0b477b91/templates/base/head.tmpl#L6
Since we use SVG for the favicon, that could be turned inline (and become dynamic this way).

Should we create a dynamic graph for that?

I don't know Golang enough. Are there libraries to aid with it? Otherwise it will be a bit hairy to write (I could take a stab, but not for the upcoming release).
https://css-tricks.com/svg-favicons-and-all-the-fun-things-we-can-do-with-them/ gives you some ideas on what you could do with a SVG favicon.
I mean, the web being the place it is yield to http://www.p01.org/defender_of_the_favicon/ and friends :laughing:


Not working without jQuery

Hi,
thanks a lot for the plugin, really cool, but I am not able to exploit it without jQuery. I have followed your advice

If you don't use jQuery, you should comment calling the setCorrectPosition() in directive.

and I have commented out the line 62 on the plunker to test it, but it does not show the date-picker.
Then I have tried in our project and the picker is shown, but

once the date has been selected then the date-picker does not close anymore;
the date picker seems not attached to the edited field

I am not using ui-grid-cellnav ui-grid-move-columns

Have I missed something?
Thanks  lot for your attention :wink:

Edit.
I am using the directive with angular-ui-bootstrap 0.14.3 and I have found this plunkr on the page without jquery, but the version is 1.2.4.
Is there any chance to make it work with old versions?
Thanks

@blackat , Yes, you can use it without jQuery.
One thing you need to change (along with commenting the call setCorrectPosition() in directive) is to change visibility from hidden to visible in style.css.
Please, see the following plunker link
Thanks!

@Joiler thanks a lot man! 😉


Make WaveformView work in WebGL

Use uint8 textures instead of float32 for better compatibility
Added old KlustaViewa colormap
Renamed field to default in cluster metadata decorator
Load cluster metadata in kwik
Added backend option in the interface for VisPy

Coverage increased (+0.88%) to 96.01% when pulling d2381b33f6343f0196dc457bd6b0671b4833adbd on rossant:fix-cluster-metadata into d9ae7f6258a77bfb274479cc35414acbc759887d on kwikteam:master.

Coverage increased (+0.91%) to 96.04% when pulling 84a2674147445c71c3dd7f59bec18d5c92d6e735 on rossant:fix-cluster-metadata into d9ae7f6258a77bfb274479cc35414acbc759887d on kwikteam:master.

Coverage increased (+0.91%) to 96.04% when pulling 84a2674147445c71c3dd7f59bec18d5c92d6e735 on rossant:fix-cluster-metadata into d9ae7f6258a77bfb274479cc35414acbc759887d on kwikteam:master.


Prefix app commands with exec for graceful shutdown

Currently, any attempt at shutting down the container will result in a forced stop due to the app being created as its own process in the entrypoint script. Adding exec will run the app as PID 1 and handle graceful shutdowns.
Any forced shutdowns with geth or similar software usually results in corrupted and lost state that will result in syncing from a much greater timeframe on reboot.

thanks!


supabase.auth.getSession is slow

Bug report

[ x ] I confirm this is a bug with Supabase, not with my own application.
[ x ] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
supabase.auth.getSession() can take anywhere from 20ms up to 100ms on a fast processor (M2 Pro) when invoked e.g. during an initial page rendering phase. This feels way too slow for a function that doesn't do any network calls.
For comparison, a custom function I mocked up that reads the cookie, parses accessToken and returns the JWT as an object (json.parse+base64decode+json.parse) takes around 0.3-0.4ms under the same load, and is similarly performant during idle and rendering phase.
Benchmarks (rough):
During rendering phase:

Custom JWT decode: <0.4ms
await supabase.auth.getSession(): ~50ms

Idle phase:

Custom JWT decode: <0.4ms (very stable and predictable)
await supabase.auth.getSession(): 0.8-1.7ms (significantly better, but all over the place even during an idle phase)

Given the above, it's unlikely that it's caused by anything specific to my app, since the custom implementation doesn't slow down at all during rendering phase vs. idle phase.
To Reproduce

Run the below code e.g. when rendering a React component

console.time("getSession");
const { data, error } = await supabase.auth.getSession();
console.timeEnd("getSession");

Refresh the page

Expected behavior
Under 1ms performance for a function that is doing simple local processing..
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: macOS
Browser (if applies) Chrome
Version of supabase-js: latest
Version of Node.js: 20

Additional context
What's worse, it's similarly slow even when there's no cookie present.

Just checked the implementation - could it be due to the locking mechanism? It's a bit unclear, why the function as a whole needs locks? Refresh session, sure, perhaps not the session fetching part.
I imagine it's often used for client-side route protections, etc. Which just adds an unnecessary delay to page renders and delay before data fetching starts.

Just noticed the same thing here!
Meanwhile this is fixed, can you share your JWT implementation pls ?
Thx


SMB2 support for CIFS

Motivation for this change
Kernels 3.7 and above support SMB2 and SMB3 so we might as well enable the support.
With this change, I can now mount SMB2 and 3 in addition to SMB1.
Cc: @wkennington, @cstrahan
The travis error is due to "out of space".
Things done

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

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Applied, but I removed the (versionAtLeast version "3.7") check as we don't have any kernels older than 3.10 (systemd doesn't support anything older than than anyway).


[POC] Proof-of-concept Rocker + Whole Tale support

Not intended to be merged anywhere, just demonstrating a few things:

Ability to configure default buildpack from command line flag
Externalized templates with flag to change (base.tpl, rocker-geospatial.tpl)
Moved R buildpack to package for WholeTale-isms
WholeTale-isms:  start.sh, custom rstudio-server (for breadcrumb customization), healthcheck scripts.

There is some cool stuff in this PR!

Oh my -- I didn't mean to issue this against the upstream!


Merge release/1.0.4 into master if necessary

[ ] https://github.com/couchbase/couchbase-lite-android/commit/9ebcae6b1e350b5bf364af32dfdcfe63117e5237
[ ] https://github.com/couchbase/couchbase-lite-android/commit/c17aeefc33d183c1ef6dc9d36cb0d26b3477c675
[ ] https://github.com/couchbase/couchbase-lite-android/commit/17ea437f8a69a1f60be75708a0db2bc9c5081793


[WIP] CRD Spec: add option to configure HostNetwork and DNSPolicy

this allows the network plugins like Cilium CNI to use etcd-cluster
as kv store when bootstrapping the cluster
fixes #1929

Hi @hongchaodeng
can you please verify the patch. I am guessing the jenkins will trigger only after one of the admin 'verifies' the patch.
I have not added tests yet, as I was not sure if its safe to run e-2-e and unit tests on my laptop and was hoping to leverage ci on this project.
Many thanks in advance.
Thanks
Rajat Jindal

Hi @hasbro17
could you please verify and allow testing this patch
Many Thanks
Rajat Jindal

Hi @hexfusion
can you please mark this PR to be tested via Jenkins? if we find issues during that, I want to make sure I push fixes for it right away.
Thanks
Rajat Jindal

@hongchaodeng  It would be nice if this gets released. 😊


feat(category_theory/const): Constant functor of object from punit

TO CONTRIBUTORS:
Make sure you have:

[x] reviewed and applied the coding style: coding, naming
[x] make sure definitions and lemmas are put in the right files
[x] make sure definitions and lemmas are not redundant

For reviewers: code review check list

Oh, but I think the universe annotation you added is wrong.

You should put it in a the punit that appears before the colon.
It should be an independent universe variable, not 1!

@semorrison Oops, that was silly. Fixed.

Lean will complain that you handed it something of type Sort w, whereas it was looking for something of type Type w.

I would put this in category_theory.punit, not category_theory.const. It happens to use const, but that is an implementation detail.
More generally, I envision each of the modules for "special diagram shapes" (including category_theory.discrete) also having a way to construct functors out of the category and natural transformations between them. (This is part of why I moved pempty and punit to their own files in the limits PR.)

Good suggestion. I moved the definition of of to the punit file.


feat: add codeql workflow with manual build

This PR adds a custom workflow to perform code analysis using a manual build as the autobuild feature fails for this repo.

I tested this workflow in my fork and analysis was performed successfully.


rustc type-check only

Maybe should add a compiler parameter, compiler don't generate any executive file, just type-check only. for raise programmer debug speed.

Already exists as -Zno-trans or cargo check


Refactor update(Flavor|Cohort)Usage to take FlavorResourceQuantitiesFlat

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Preparation for #2596, where we want to add usage to Cohort without needing a workload. See WIP Commit for example of intended usage.
Additionally, gets us closer to #2447 cleanup.
Does this PR introduce a user-facing change?
NONE

/assign @macsko

/assign @mimowo

/hold
To investigate if the failure is related https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kueue/2633/pull-kueue-test-unit-main/1813897188141436928. Feel free to cancel the hold if not related

thanks for the review!
the failure is unrelated. different suite, and noisy field:
        - 			CreationTimestamp: v1.Time{Time: s"2024-07-18 11:29:27 +0000 UTC"},
        + 			CreationTimestamp: v1.Time{Time: s"2024-07-18 11:29:26 +0000 UTC"},

/unhold

/retest


Create E2E Tests for Daily Data Page in Staging

Problem statement
The Daily Data page on the RS website needs End-to-end (E2E) tests created to ensure that it works properly after deployment. Without E2E tests, some features/links may not function/load properly and we wouldn't know until a user tried to use them.
What you need to know

This is based off the investigation from #11307
e2e_Tests_audit.xlsx has notes pointing out specific concerns/targets on some pages
Strategy discussed for checking content load: check to see if every DIV on this page loaded

Acceptance Criteria
As an unauthenticated user, check that:

[x] The page does not load and redirects me to log in

As a receiver, check that:

[x] The page loads
[x] The Daily Data table loads with data
[x] Page content loads without errors (including if the footer is showing, but not its functionality)
[x] All links work and take you to expected page
[x] Components work as expected (eg, accordion expands/collapses, side nav, etc)

As a sender, check that:

[x] The page loads
[x] The Daily Data table loads without data
[x] Page content loads without errors (including if the footer is showing, but not its functionality)
[x] All links work and take you to expected page
[x] Components work as expected (eg, accordion expands/collapses, side nav, etc)s

As a Prime Admin, check that:

[x] The page loads
[x] The Daily Data table loads with data
[x] Page content loads without errors (including if the footer is showing, but not its functionality)
[x] All links work and take you to expected page
[x] Components work as expected (eg, accordion expands/collapses, side nav, etc)

Waiting on E2E Receiver, Sender, Prime Admin accounts to be working properly with deployment


HIVE-24424: Use PreparedStatements in DbNotificationListener getNextNLId

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Thanks @miklosgergely for the review.  Can you please take a look once more? I believe I've addressed your comments.


Allow users to do choropleth maps using date columns

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


[🐞] Custom 404 page does not works

Which component is affected?
Qwik Runtime
Describe the bug
According to the documentation Custom 404 page we must create a file named 404.tsx inside src/routes folder. However,  when we try to build the application pnpm build the process fails the following message:
[vite-plugin-qwik-city] Could not load /Users/keuller/workspace/personal/basic/@qwik-city-plan (imported by src/s_02wmimzeabk.js): More than one route has been found for pathname "/". Please narrow it down to only one of these:
  - /Users/keuller/workspace/personal/basic/src/routes/400.tsx
  - /Users/keuller/workspace/personal/basic/src/routes/index.tsx
error during build:
Error: Could not load /Users/keuller/workspace/personal/basic/@qwik-city-plan (imported by src/s_02wmimzeabk.js): More than one route has been found for pathname "/". Please narrow it down to only one of these:
  - /Users/keuller/workspace/personal/basic/src/routes/400.tsx
  - /Users/keuller/workspace/personal/basic/src/routes/index.tsx

Looks like the builder recognises 400.tsx file as a root file.
Reproduction
N/A
Steps to reproduce
Create a file named 400.tsx inside src/routes folder and try to run the application.
System Info
System:
    OS: macOS 13.4
    CPU: (10) arm64 Apple M1 Pro
    Memory: 16.00 GB
    Shell: /bin/zsh
  Binaries:
    Node: 18.16.0 - /usr/local/bin/node
    npm: 9.5.1 - /usr/local/bin/npm
  npmPackages:
    @builder.io/qwik: ^1.1.4 => 1.1.4
    @builder.io/qwik-city: ^1.1.4 => 1.1.4
    undici: 5.22.1 => 5.22.1
    vite: 4.3.5 => 4.3.5

Additional Information
No response

The issue here seems to be a typo in the filename: it's supposed to be 404.tsx and not 400.tsx

@keuller can we close it?


Improve Platform functions

Fixes LIB-856
Replaces https://github.com/OpenZeppelin/openzeppelin-upgrades/pull/786

Move proposeUpgrade to be in platform module only.
Use usePlatformDeploy as the option name and Hardhat config option name for enabling/disabling Platform deployments.
Add walletId? option for Platform deployments to customize the wallet id to use.
Use new Platform Upgrade API for platform.proposeUpgrade which simplifies input options and can include returning a Safe App url.
Add platform.getDefaultApprovalProcess as a convenience function for users to get the default approval process with its associated address (e.g. multisig or governor address) that they should assign as owner of their proxies.
Improve license type handling: Convert no license or 'UNLICENSED' to 'None' for Etherscan verification. Catch invalid license type from API and wrap with more useful error message.

(Note: tests will fail in the meantime until new version of https://www.npmjs.com/package/platform-deploy-client is released that includes some required updates)

Waiting for a new version of https://www.npmjs.com/package/platform-deploy-client to be released that includes some required updates before merging this.


aria2: 1.20.0 -> 1.21.0

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Appears to trigger a build failure for uget:
checking for curl-config... /nix/store/0x6mwlvy3wp0zgph638kjq2v50nd9gxj-curl-7.47.1/bin/curl-config
./configure: 6341: ./configure.lineno: let: not found
./configure: 6342: ./configure.lineno: let: not found
configure: error: Requires libcurl version >= 7.19.1
builder for ‘/nix/store/fqyf2qx4gljl3vx6zldlhv45rim4zy4a-uget-2.0.5.drv’ failed with exit code 1
error: build of ‘/nix/store/fqyf2qx4gljl3vx6zldlhv45rim4zy4a-uget-2.0.5.drv’ failed
The invocation of "nix-build -A uget -A aria /home/travis/.nox/nixpkgs" failed

Perhaps @romildo knows more

I'm having trouble reproducing the travis failure on my 16.03 box. nox-review pr 14523 succeeds locally (and I have nix.useChroot = true set). Is there something I can do to make my build environment more like the one on travis to debug this?

See maintainers/scripts/travis-nox-review-pr.sh. but I'm inclined to trust sandboxed builds more than travis. If uget & aria both compile successfully for you under a sandboxed environments, then this is probably okay.

I can't see any potentially breaking changes in the aria2 changelog either, so this is most likely a case of travis being unhelpful rather than a geniune build failure.

I have no problems building uget-2.0.5 with this commit for aria2-1.21.0.


[botan] Update to 3.1.1

Fixes #33412.
botan
Update botan to 3.1.1.

Remove outdated patch files: arm64-windows.patch, fix-generate-build-path.patch, fix-objectfile-list.patch.

All features are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static

qca
Update botan to 2.3.7.

All features are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static

The usage test passed on x64-windows (header files found):
qca provides CMake targets:

    # this is heuristically generated, and may not be correct
    find_package(Qca CONFIG REQUIRED)
    target_link_libraries(main PRIVATE qca)

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Thanks for the update!


Update demo-simple-rt-task.html

The changes come from step 2 of the following tutorial:
https://github.com/Brain2AI/jsPsychSheet/blob/master/tutorial.md

@BrianOdegaard
modified experiment file is located here: https://github.com/Brain2AI/jsPsychSheet/blob/master/experiment/demo-simple-rt-task-modified.html
In this repo, we do not change demo-rt-task.html so that its easier to compare the modified file with the original one: https://github.com/Brain2AI/jsPsychSheet/blob/master/experiment/demo-simple-rt-task.html

Hi Shashi -
Apologies!  100% mistake on my part - I should have canceled the pull
request.  All good.
Great work!!!
Brian
On Sat, Aug 15, 2020 at 9:09 PM Shashi Kant<EMAIL_ADDRESS>wrote:

Closed #1 https://github.com/Brain2AI/jsPsychSheet/pull/1.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Brain2AI/jsPsychSheet/pull/1#event-3658941727, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACASZUGYXM6ZABFQXMYOYWDSA4WT5ANCNFSM4P7SSIIQ
.

--
Brian Odegaard, PhD
Assistant Professor
Department of Psychology, University of Florida
This email may contain material that is confidential and/or privileged for
the sole use of the intended recipient. Any review, reliance or
distribution by others or forwarding without express permission is strictly
prohibited. If you are not the intended recipient, please contact the
sender and delete all copies.


feat: minimal e2e framework

Ref: https://github.com/celestiaorg/celestia-app/issues/1256
This PR puts together the base pieces for an e2e testing suite. It includes a CLI that can:

Setup the file directories for multiple nodes, including generating a genesis with several accounts
Start the testnet (with different versions and at different start heights)
Stop the network and cleanup the used resources.

Further work aims to support:

compatibility testing
upgrade testing
sync testing
fuzz testing

for non-determinism
invariant checking
as a generalized tool for other teams

Codecov Report

Merging #1586 (a4b32f7) into main (ee84e3d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1586   +/-   ##
=======================================
  Coverage   49.10%   49.10%
=======================================
  Files          84       84
  Lines        4993     4993
=======================================
  Hits         2452     2452
  Misses       2298     2298
  Partials      243      243

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

Are we wanting to eventually enable celestia-node to use this as well? I could see them being able to reuse a lot of the code here.

Yeah Potentially. Matt mentioned having a common repository for e2e testing of various node types in a system. If we are to do this I'd definitely recommend reusing parts of this code.

cc @celestiaorg/testing for visibility

Few questions from my side
why we opted for in-house framework instead of using interchaintest?
state-sync example
chain-upgrade example
why not just re-use their implementation of chain creation using docker-compose and networking?
https://github.com/strangelove-ventures/interchaintest/blob/main/chain/cosmos/cosmos_chain.go
It has the same capabilities as we have rn.
Same with network pruning/setup
https://github.com/strangelove-ventures/interchaintest/blob/main/internal/dockerutil/setup.go
imho, it looks like they have took care of docker networks cleanup which is clumsy
if we want more control over implementation - why not reuse compose-spec like libp2p does?
Seeing the matrix versioning schema that libp2p does - we can def reuse the hard work they pushed.
Pros of above
Both have CI/CD implementation already meaning that we can speed up dev time and contribute back if something we really need

My main concern over this PR - we are signing up on maintaining long-term, while there are tools that can help us achieve the same without maintaining it (interchaintest is constantly being updated, same is for libp2p/test-plan)
cc: @MSevey

Hey,

why we opted for in-house framework instead of using interchaintest?

The simple answer is we have a modified fork of the SDK which is not compatible with interchaintest. There are changes that I've spoken in the past that we can (and I think we should probably make) that means we don't have this compatibility issue and can eliminate the need to maintain the SDK fork.
Secondly, interchaintest is very IBC orientated. This means some of the initial goals of upgrade testing (we will have a different upgrade mechanism to most cosmos chains), compatibility testing and sync testing don't seem to natively fit their framework.
Lastly, for an open source library, I think there documentation and tutorials around using their library are lacking (although the inline commenting is relatively helpful). As well as maintenance we have to also consider onboarding costs to understanding a new (and large) codebase (when something simpler might suffice).

My main concern over this PR - we are signing up on maintaining long-term

I'm sympathetic to this. Ideally we reach our testing goals with little duplication and overlap. When we spoke prior to this it seemed like testground was not suitable. Rene tells me that robusta uses a different more light-weight framework that may be ideal for the outlined purposes. I'm not sure about the other (libp2p) library you mention but I definitely think that what's done here is quite generic: It's simply managing a series of binaries within a network and using the RPC endpoints to make assertions.
What I'm more concerned about is that it seems like these testing goals are perhaps the same across the other teams (node have their "swamp" testing) which seems, at least to me, to heavily overlap with what core/app is doing. If we have a "testing" team then perhaps efforts from individual teams can be consolidated

There are 2 topics that got confused and we clarified them here.
The original idea behind this PR is for teams to do nightly tests using minimal e2e framework. I thought that this is only for core/app and integration ones executed on every PR.
Going forward, we separated into 2 topics:

Nightly Tests.
Integration Tests that are executed per PR.

Nightly tests
@celestiaorg/devops will implement infra features for robusta-nightlies after @cmwaters shares the notion doc with us.
Robusta-nightly is a good point as the telemetry is already there and a fuzzing client can be plugged in easily.
Integration tests
This PR should be used on Integration level per PR run. In addition, we need to migrate it to a standalone repo, so @celestiaorg/celestia-node and @rollkit can use to fulfil their testing strategies
In addition, we will reuse the compose-spec that libp2p/test-plan has to make it CI/CD ready out-of-the-box
Thanks for the input and clarifications 🚀

morge?

I have docker compose installed in my $PATH:

Sorry for missing this @rootulp. docker compose isn't the same binary as docker-compose, it's a command from the docker binary that was recently added. If you haven't got it then I think you need to upgrade your docker version


TransactWriteItemsAsync does not guarantee the order in which the items are recorded

While using TransactWriteItemsAsync to record events from an Event-sourced system as DynamoDB entries, we noticed that in some cases the recorded events appear in the associated DynamoStream out-of-order.
Most of the time, we write a single event per action, but we do have an action that generates 3 events. In this action, we want to write all 3 in a transaction (all written or none written) and want the events to appear in the stream in the same order as we added to the list that is sent to TransactWriteItemsAsync. Each event in our system represents a change in the domain aggregate. The partition key is the aggregate Id, and the sort key is the version of the aggregate caused by each event (first event is version 0, second event, 1, etc.).
So, this single command on the aggregate generates 3 events. We send the 3 events in order in a list to the TransactWriteItemsAsync method. The records get written to the DynamoDB table. In the lambda that consumes the associated DynamoStream, we notice that the events are published to the stream out of order (such as 1,2,0). This affects the lambda that consumes the events and is expected to build a search model from these events.
Is this expected? The docs for BatchWriteItems seems to suggest so (transaction not worried about the order things are written to the DynamoDB table), but it isn't clear if TransactWriteItemsAsync is some type of wrapper around the BatchWriteItems logic or sharing the implementation.
Also, is there some configuration element that activates / deactivates in order writing (maybe related to isolation levels?).?
Expected Behavior
The expectation is that, during the transaction, all the items in the transaction are written to the DynamoDB table in the same order they are supplied to the TransactWriteItemsAsync method.
Current Behavior
Currently, though the operation succeeds, the items written to the DynamoDB table, and published to the DynamoDB stream appear out of the order in which they were supplied.
Your Environment

AWSSDK.Core version used: <IP_ADDRESS>
AWSSDK.DynamoDBv2 : <IP_ADDRESS>

.NET Core Info
.NET Core SDK (reflecting any global.json):
Version:   3.1.101
Commit:    b377529961
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.17763
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.101\
Host (useful for support):
Version: 3.1.1
Commit:  a1388f194c
.NET Core SDKs installed:
3.1.101 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.NETCore.App 3.1.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.1 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Hi @MiguelAlho,
Good afternoon.
Looking at the code in .NET SDK for TransactWriteItemsRequestMarshaller.Marshall(), I do not see the any logic that changes the order of TransactItems. The logic is simply writing the JSON for TransactItems in the order mentioned in the TransactWriteItemsRequest object. Hence, I cannot pin-point if there is a problem with the SDK or the DynamoDb service.
As a workaround, is listing and ordering objects based on SortKey and passing them to Lambda function works for you?
Thanks,
Ashish
Thanks,
Ashish

Hi, Thanks for looking into this.
I suspect it isn't so much a SDK issue as it is either a misinterpretation of the docs (and expectations) on my side or a potential error in the stream publishing. The SDK itself seems to be OK as I can get the items stored in the database.
The problem, though, is that they are not published to the stream in order that I expect them to be. So when I try to pull events from the stream in my lambda function, the batch that I read might not have them in order (though I would expect them to be read / made available in order).
The docs (https://docs.aws.amazon.com/amazondynamodb/latest/developerguide/Streams.html) say "For each item that is modified in a DynamoDB table, the stream records appear in the same sequence as the actual modifications to the item."
What I do know is that in the batch of items saved, the first doc has a much larger payload then any of the others so I would suspect that the time to write / propagate that record could be larger than the other, much smaller docs if parallelism were to be considered, and that could be a cause for out of order on the stream. Is there any prop on the payload in the stream's message that could help me pinpoint that?

It's been a while , but I've finally come around to producing a sample repo that demos the behavior, to to some other order-related issues I'm trying to understand.
https://github.com/MiguelAlho/DynamoDb_TransactWriteItem_OutOfOrder
Hope it is useful. I'm still interested in understanding if the behaviour is expected / by design or if it is an unexpected bit of behavior.


Make progress bar optional

Add show-progress option to make the animation optional.

Hi,
this is a good idea, but I would have implemented it slightly different. What do you think of the following: https://github.com/sebastianha/angular-bootstrap-pwgen/tree/animate-test ?
I use animate="true" to activate progress animation.

Version 0.1.2 adds an "animate" option which does that


Proposal to add R.random

Would you be interested in adding a helper function for a random number generator, R.random?
Math.random is limited in that you can't specify a range (it's [0, 1)), which is why Lodash and Underscore have their own implementation. It would be nice to have something like this in the library.

Something like this maybe?
const random = R.curry((min, max, isFloat) => {
  let range = max - min
  let random = Math.random() * range + min
  return isFloat ? random : Math.floor(random)
})

It might also be useful to add an option to generate a CSPRNG

👎 There are - and rightly so - dedicated libraries for this.

I guess.. but it might be useful to have a purely functional implementation like Haskell's and to make it compatible with ramda

I guess that if you want a random number generator ramda style you'll need to use the IO monad from ramda-fantasy. A function that returns different results for the same input is very much against ramdas philosophy.

A function that returns different results for the same input is very much against ramdas philosophy.

exactly. We actually had a random extension for a while. But it would produce repeatable results based on the initial seed value. I'm inclined to agree with @asaf-romano that this should be the domain of a separate lib.

@buzzdecafe That's along the direction I'm thinking of, but AFAIK there's no library that does this. Maybe it would be worthwhile to create one even if kept separate?

@Risto-Stevcev that sounds good to me (a pseudo ramda number generator?) Here's where the old extension got whacked: https://github.com/ramda/ramda/pull/590/files

It might be worthwhile to just use an existing library and wrap it accordingly, especially something so low level. Not a huge fan of reinventing the wheel.

I would have no problem reinstating that random extension the way we now have separate repos for side projects.  I think it's of reasonable quality, and it might be a good place to add other random-related functions.  Right now it's just a replacement for Math.random and a list shuffle function, but it's easy to imagine others.
We couldn't port it directly in, because when that was written, these were actual extensions, adding functions directly to Ramda.  I'm quite certain we don't want that.  But it would be easy to adjust.  And as tempting as it is, we probably don't want to call it "Ramdom" either.  :smile:.

I raised https://github.com/ramda/ramda-fantasy/pull/92 some time ago to add a State type which can be used to model a pure RNG.
// An example deterministic pseudorandom number generator
// see: https://en.wikipedia.org/wiki/Linear_congruential_generator
const RNG = State.get.chain(seed => {
  const newSeed = (seed *<PHONE_NUMBER> + 12345) & 0x7FFFFFFF;
  const randVal = (newSeed >>> 16) / 0x7FFF;
  return State.put(newSeed).map(_ => randVal);
});

// Produce a random integer from 1 to 6
const rollDie = RNG.map(n => 1 + Math.floor(n * 6));

// Produces a new RNG that returns a Tuple of two random values from the given RNG
const pair = rng =>
  rng.chain(a => rng.chain(b => State.of(Tuple(a, b))));

// eval takes a seed (123 here) to run the RNG.
// 123 could be replaced with Math.random() here.
pair(rollDie).eval(123); // Tuple(2, 5)

@scott-christopher Cool! I'm whipping up a quick xorshift RNG atm

I created a pure random number generator:
https://github.com/Risto-Stevcev/pure-random
I used xorshift because it was very succinct and performant

> const rnd = require('pure-random')
> const IO = require('ramda-fantasy').IO
> const rollDie = IO(_ => rnd.random(rnd.genSeed(), 1, 6))
IO { fn: [Function] }
> rollDie.runIO()
Right(5)
> rollDie.runIO()
Right(1)
> rollDie.chain(R.pipe(R.tap(console.log), IO.of)).runIO()
Right(4)
Right(4)

But rnd.genSeed is not referentially transparent?  That would bother me in the Ramda ecosystem.  I would rather something that required
const myRnd = rnd.genSeed(mySourceOfRandomness());

where mySourceOfRandomness might be a call to Math.random, some mouse-tracking tool, your own private Geiger counter, or whatever, but if genSeed is called twice with the same input, the PNRGs returned should have the same behavior as one another.

The seed you give to the random number generator can be any function that returns an array of 4 Uint32 types, or in JS would be an array of four numbers in the range [0,<PHONE_NUMBER>]. That part is referentially transparent. genSeed just provides a handy way of generating a seed value using Date.now.
In your example, mySourceOfRandomness would just be your own genSeed function that you pass in like this: rnd.random(mySourceOfRandomness(), 1, 6)

Right, it's genSeed I was objecting to.  I hadn't read through the code to realize that you could replace it with your own 4-Uint32-yielding function, but I assumed that you could duplicate it's ability reasonably straightforwardly.  But I would not want genSeed, a non-referentially transparent function in Ramda code.  We've managed so far to be able to say, "Everything is referentially transparent."  I'd hate to change that to "everything but".
Of course this is still not referentially transparent, just repeatable, and perhaps that should be revisited.

I must say too, that I really like the code we borrowed from Johannes Baagøe for seeding the function.  It accepts any parameters at all, turns them all into strings, and uses these strings to generate the seed.

You could leave genSeed out of the implementation. genSeed and genCsSeed are just convenient functions, but I do like  Johannes Baagøe's approach. It would be nice to just have a transparent function that can generally handle Strings, Numbers, and Objects as input and convert it to the required 4-Uint32 seed. I think I'll change the implementation of genSeed to just do that instead, and then you could put in Date.now() or crypto.randomBytes, or mouse gestures, etc.
I should mention though that that code isn't referentially transparent and it doesn't quite accept any parameters at all. If no parameters are supplied, it uses [+new Date], and it can only accept parameters that have a toString method.
I'm not sure what you mean by this:

Of course this is still not referentially transparent, just repeatable, and perhaps that should be revisited.

Actually, I guess all the types have a toString method, so nvm

I'm not sure what you mean by this:

Of course this is still not referentially transparent, just repeatable, and perhaps that should be revisited.

I was thinking of the older extension, but it applies equally well to this version, I suppose.
A expression is referentially transparent if it can be replaced by its value without changing the behavior of a program.  For a function that implies both no side effects and that the same input yields the same output.  But a PNRG that does not accept the current seed breaks both: The whole point is that if a = random(); b = random();, almost certainly, a !== b.  And of course this happens because of the side effect of updating the internal seed.
So, given a particular seed value, you do get a repeatable set of values, but it's not actually referentially transparent.
This could be fixed if the PNRG accepted the current seed value/structure as a parameter.  But that does make it harder to use.

If I'm understanding you correctly, you mean that it would only be truly referentially transparent if you could make the seed function transparent, like something that's analagous to the Haskell's System.Random?
Prelude> import System.Random
Prelude System.Random> let gen1 = mkStdGen 1234
Prelude System.Random> let result1 = fst $ random gen1
Prelude System.Random> result1
-4311284599041119727
Prelude System.Random> let result2 = fst $ random $ mkStdGen result1
Prelude System.Random> result2
4250114187885638776
Prelude System.Random> let result3 = fst $ random $ mkStdGen result2
Prelude System.Random> result3
-1756935596885184753

If that's what you mean, I'm totally on board with that. I don't think it makes it that much harder to use:
> var roll1 = rnd.random(genSeed(1234), 1, 6)
5
> var roll2 = rnd.random(genSeed(roll1), 1, 6)
4
> var roll3 = rnd.random(genSeed(roll2), 1, 6)
1

If you wanted it to be more convenient you could just return it partially applied with Date.now:
> var random = _ => rnd.random(genSeed(Date.now())
[Function]
> var roll1 = random()(1, 6)
2
> var roll2 = random()(1, 6)
6

ok to close this one? I think we can direct people to https://github.com/Risto-Stevcev/pure-random


GraphiQL endpoint not working for me, unpkg.com not returning certain assets

e.g. if you try and fetch<EMAIL_ADDRESS>it returns an empty response, whereas<EMAIL_ADDRESS>works just fine. They're a CDN... so I don't think our usage is outside the norm
https://unpkg.com/

unpkg is a fast, global content delivery network for stuff that is published to npm.

I havent experience this yet …

The script looks fine to me too, however we really should bundle GraphiQL and React to avoid network issues in the future and allow GraphiQL to work offline. I have a branch that should do this which I just opened a PR for here: https://github.com/calebmer/postgraphql/pull/252

Weird, it's not just me it's failing for, but glad to hear it's not everyone. I've had a couple people in the UK confirm that<EMAIL_ADDRESS>just renders a blank page for them. (It's weird how the version without .min in it works fine but the min one fails - perhaps it's some kind of CloudFlare filtering?)

Everyone I've asked in the UK (6 people so far) have had this URL fail; all the US people (or people with US VPNs ;) ) have loaded it successfully. Definitely seems to be some kind of CDN issue. Bad cache key on Cloudflare, maybe?

I have the same issue. Located in Israel.
Workaround?
Uncaught ReferenceError: GraphiQL is not defined(…)

Well - it's kind of a hack - but if you edit
build/postgraphql/http/renderGraphiQL.js:    <script<EMAIL_ADDRESS>and  change it to
build/postgraphql/http/renderGraphiQL.js:    <script<EMAIL_ADDRESS>then graphiql end point works......

I use https://github.com/redound/graphql-ide (or https://github.com/skevy/graphiql-app). They work offline and also let you edit headers (for JWT auth)

Burkhard,
Good one.  Thanks
Danny
On 28 November 2016 at 10:19, Burkhard Reffeling<EMAIL_ADDRESS>wrote:

I use https://github.com/redound/graphql-ide (or https://github.com/skevy/
graphiql-app). They work offline and also let you edit headers (for JWT
auth)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/calebmer/postgraphql/issues/249#issuecomment-263208959,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUYhTNURiV1_9RRBZTO2uwHCu2ckrqHks5rCo6OgaJpZM4K8QRt
.

--
Accelerate product development: www.clearclinica.com
https://mailtrack.io/trace/link/06dee2f3b457ea174572357f7adda06ef00c8be2?url=http%3A%2F%2Fwww.clearclinica.com&signature=481ff6506f24c95f
Security & compliance specialists for biomed:   http://www.s
https://mailtrack.io/trace/link/41cd0c3627b8a283ccc747a5dce64b503e7dbb11?url=http%3A%2F%2Fwww.pathcareblog.com%2F&signature=aea91c3094b752d1
oftware.co.il
https://mailtrack.io/trace/link/8b9ca86f99089d8b6d8f3ea6cf38a8f24962ef57?url=http%3A%2F%2Foftware.co.il%2F&signature=ff0df78ace39e8df
/
Phone: +972 54 447 1114
Sharing with people you care for: http://www.pathcare.co
https://mailtrack.io/trace/link/3dbc48986a66952117940c5b4873d9346981a76e?url=http%3A%2F%2Fwww.pathcare.co%2F&signature=4a5a250b1be1005b
/

Having the same problem. I tried changing to use version 0.8.0 in postgraphql/http/renderGraphiQL.js<EMAIL_ADDRESS>is not corrupt) and running npm build and npm rebuild, but I am unable to get the change out -- it keeps using version 0.7.8 when I run it with postgraphql. Is there something else that needs to be done in order to update it?

same problem, located in UK

Once we get another few rounds of review on https://github.com/calebmer/postgraphql/pull/252, this problem should get fixed :tada:

Should be fixed in 3.0.0 🎉


feat(config): add foundry dir and check for global foundry.toml

Motivation

Introduce ~/.foundry as global config dir
Support ~/.foundry/foundry.toml

Solution
if ~/.foundry/foundry.toml exist, merge in into the Figment

Clean


Hot reload not working in fresh install

Node: v8.4.0
NPM: v5.3.0
Vue-cli v2.8.1
Hello
I'm having an issue where hot reload is not working. I followed the instruction to install
vue init webpack my-project
cd my-project
npm
npm run dev

It then boots up the server and loads the 'Welcome to Your Vue.js App' page, when I go to edit a word for example, it does not reload the page.
Any ideas why this might be happening?

I noticed vue-cli was not the most recent version. Updated to 2.8.2 and it's working now.


Feature/add metrics exporter

Added metrics exporter and associate metrics exporter to cluster
Included  Datadog and Grafana.
Decision

I included Grafana should be in PROD / QA soon, people will got an error if they try to use it before release.
I didn't not include STOP because there is yet no START api, so the way to stop the exporter is just to remove the associations
Association resource could not be updated / imported for now.

It seems some change for Grafana are underway on the backend so please don't merge this PR nw.


Move freewaydata.stations (?) data to staging

Todo:

How to pg_dump only part of a table?
How is the partition setup of the table organized?

Need to bring over:

Last week of January
Last 6 weeks of fridays before that

@joshproehl did this, closing


Kata Editor Page Blanking Out When I Press Command-V

As the title says, the kata editor page is blanking out when I try to paste a code snippet in the description field of the kata editor. The page just turns gray.
The code snippet:
div {
    display: none;
}

a:hover + div {
    display: block;
}

And where I try to place it:
Some description text here

I think the problem is that I am trying to hide a certain part of the kata's description to hide a hint but the browser page thinks that I am changing its HTML. I am doing this with Mac OS Sierra, on Chrome.

I'm not sure, but isn't this because of the selector div? The rule is applied to every
 on the page.
I'd recommend using inline styles for kata descriptions and avoid "type selectors" in general.
Unfortunately, :hover wouldn't work inline. How about using the same color for color and background-color? Haven't tested, but this should hide the hint unless selected.
Some description text here

Some hint below:

hint

If you really need :hover, using class selector (.kata-hint) might work.

@jhoffner,  in the description seems to affect the entire page. I saw this recently on 6 By 6 Skyscrapers's page.

I think this is caused by following

I like how Codewars allows authors to do this because the description of that kata looks nice, but maybe document this behavior or add some warning?

@jhoffner, this is caused by div { display: none; } applied to

.
Pasting div { display: table or flex or inline; } will produce similar observable change, but display: none is more serious issue because it doesn't allow to recover.
div { display: none }
/*                 ^ 'backspace' after typing that last 'e' won't make the editor go back */

The only way I found to recover is to remove that rule through Developer Tools.

Yeah div { display: none} is going to make every div element on the page invisible. The markdown isn't somehow scoped or sandboxed. I'm not really sure what the fix here is to prevent people from doing this, all I can think of is to just forbid style tags all together.
For what its worth. div {display: none;} is not likely to ever be something you would want to do within any programming context, unless it is at least scoped (ie: .hide-divs div {display: none;})
Is it enough to just say lets no do this anymore?

I suppose I could try to parse out the style tag and add .markdown  as a prefix to any css line, but that seems like likely to have all kinds of edge cases.

The markdown isn't somehow scoped or sandboxed.

What about JavaScript?

I'm not really sure what the fix here is to prevent people from doing this, all I can think of is to just forbid style tags all together.

I'm against forbidding because it does make the description much easier to read if used properly and many descriptions will break.
I don't know how the kata editor is implemented, but if it's triggering an event whenever user edits, then maybe lightly check the content in the event handler?
I'll try to look into this and let you know if I find anything.

Maybe I should just make the selector div.hint and classify the div tag as hint.

I also think you should still be able to style the description page.

@jhoffner
I've done some experiments and read some of application.js. I don't think it's difficult to modify App.Controls.MarkdownEditor (refresh) and App.Controls.MarkdownDisplay (setMarkdown) to do whatever you decide. I need to read more marked documents, but it has some useful options and its lexer and parser are available to use if that doesn't suffice.

I believe I've found more serious issue which I don't know if I should post here. I have no experience in security and it might not be a risk at all, but it doesn't look safe to me.
Should I email you?

@kazk you can send me the details on the security risk at<EMAIL_ADDRESS>

@jhoffner I actually sent an email to<EMAIL_ADDRESS>2 days ago. I'll try sending it to your address too.


Replace AbstractNode with RenderObject.

Closes #78.
Bump minor version as this would be a breaking change for users of older Flutter SDKs.

@tmpfs thanks for proposing this fix but I went with a non breaking solution

Hi @Kavantix, thanks for fixing it, will upgrade to 0.2.12 🙏


IE & FireFox Issues: Rendering

I try export Tree chart using treant-js to pdf files. Your library is working well on Chrome  but Not working on IE and Firefox.Problem is the line in tree chart is missing.
Chrome:

RESULT: PDF FILE

RESULT ON FIREFOX AND IE:

IE:

FireFox:

Chrome:

[x] You are using the latest version
[x] You are testing using the non-minified version of html2canvas and checked any potential issues

Bug reports:
http://jsfiddle.net/hv5yfe3c/3/
Specifications:

html2canvas version tested with: v1.0.0-alpha.4
Browser & version: IE updated verisons 11.0.49,Chrome Version 63.0.3239.108,Firfox 57.0.2
Operating system: Windows 8.1 Pro

I'm having the same issue with svg.
Chrome:

Firefox:

Fixed when I upgraded to version 1.0.0-alpha.4. Thanks!

I am having a similar problem. I have a card which has 2 svgs. One on top and one on the bottom (as shown).
On Firefox, whenever I generate the Canvas, the BOTTOM svg, is NOT being rendered at all. on Opera / Chrome / Edge, all svgs / images are being rendered correctly. I am using the latest version but with no avail.
Any ideas?
This is the Original (Chrome / Opera / Edge all perfect)

What is being rendered on Firefox

Facing the same issue mentioned by @Nuttiwut-Simsiri .
In chrome its working perfect, In IE 10 it's not rendering the tree node connectors.
using npm package => "html2canvas": "^1.0.0-alpha.12"


[Gazebo] laserscans drift away without movement

From time to time the laserscans drift towards the robot in a weird manner, even if robot is not moving.
only in gazebo!
more information following

this is caused by the robot tilting, inertia / weights / simulated properties must be checked

Wow, good find! I was also looking at the same issue right now.
For reference, here's a video of what's happening:
https://user-images.githubusercontent.com/320188/190986284-d7fb5dad-068e-4367-ac32-48ff3d0c5f42.mp4

I think this fixes it:
git cherry-pick 063d4e75d1414eb70c426b484967347e43faf54e
In noetic, the collision meshes for the wheels were replaced by cylinders. This solves lots of problems like this.

Nice, this helps indeed! Will pull this right away.


The istio application directly accesses the pod IP of the grpc protocol and reports an error. What should I do to solve the NPE?

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities
To ask questions about how to use Istio, please visit https://discuss.istio.io)
Bug description
The istio application directly accesses the pod IP of the grpc protocol and reports an error. What should I do to solve the NPE?
{   "start_time": "2021-05-29T13:37:34.624Z",   "request_id": "-",   "method": "-",   "upstream_host": "-",   "x_forwarded_for": "-",   "bytes_sent": "0",   "upstream_cluster": "-",   "downstream_remote_address": "<IP_ADDRESS>:35640",   "path": "-",   "authority": "-",   "protocol": "HTTP/2",   "upstream_service_time": "-",   "upstream_local_address": "-",   "duration": "0",   "downstream_local_address": "<IP_ADDRESS>:8080",   "upstream_transport_failure_reason": "-",   "response_code": "0",   "response_flags": "DPE",   "requested_server_name": "-",   "bytes_received": "0",   "istio_policy_status": "-",   "trace_id": "-",   "route_name": "-",   "user_agent": "-" }
Istio only knows the protocol of service. How to specify the protocol of direct access to pod IP?
[ ] Docs
[X ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
Istio version - 1.6.8
How was Istio installed?
istioctl
Environment where the bug was observed (cloud vendor, OS, etc)
Additionally, please consider running istioctl bug-report and attach the generated cluster-state tarball to this issue.
Refer cluster state archive for more details.

Do you have strict mtls or something? I am guessing because you send to the pod IP directly we are not applying mTLS (which is expected, Istio works on Services) and then its getting TLS on the server side which is unexpected

Yes, so if so, only grpc port can be released


Added Tailwind CSS

Thank you for fixing the sillies! I was copying tutorial 100% even when creating unneeded files just to make sure it worked seeing that this is my first time working with TailwindCSS.


As a member of a running Game, during each Round, I want to see an image that corresponds to the food item in question.

[x] sopra-fs23-group-34/client#12

closed as of https://github.com/sopra-fs23-group-34/client/commit/a077c103931fcdf5aadc9f2634ebedacda377560


Add content type to resource field

This will add content type name to resource_url field.

Hi @jjkester, can you look on this one?

Hi! Could you rebase your PR, please ? Thank you.

Hi @Linkid,
Done.

QA is not passing. It should be quick to fix ;)

Thank you :).


Some additional feature suggestions

Please add support for proxy per account, also add other earning tasks such as MSN Fitness videos, Microsoft Edge News and MSN Shopping Quiz, this will result in earning almost 200 more points per account daily.

Proxy support is on the list of things id like to implement soon. Proxy support would be up to the user to find and use suitable proxy servers, and ensure those proxy servers are the same geographic location as the account(s) they are used on.
So far I havent had any bans on the 5 accounts I've used for testing, over 33 days. no proxy servers. I'll look into how necessary it is though.
Other methods of getting points is also on my todo list. I dont have an ETA for this though, as my current focus is to make this project maintainable.
Thank you for the suggesttions!


Infer input and output variables automatically

In case the model have just one input and outptut, support options to just pass input parameters without variable names, and without the need to specify output params.

Added in https://github.com/nadavbar/node-cntk/commit/4f51ccca16ad0df437e85c9c4fe2ed5dd442f4e5


[VC] Refactor: Move MultiClusterControllelr related codes to pkg/util

This change move MultiClusterController related framework codes from pkg/syncer to pkg/util so that the framework code can be more easily imported by other sub systems - such as the scheduler.
Most of the changes are code moves. The pkg/syncer/constants/constant.go file is split into two. The framework related constant definitions are moved to pkg/util/constants/constants.go.
Some dependencies still exist. For example, mccontroller still imports syncer/metrics. I will decouple them in next change.

/lgtm
/approve

/lgtm
/approve


VerifyArea test fails on master for FSharpOption.IsNone and IsSome, unclear cause

I'm not sure this is a bug or anything, but on my system (Win7, VS2017 15.4) running the VerifyArea test fails with the following error:

Assembly: FSharp.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a
Expected and actual surface area don't match. To see the delta, run:
windiff C:\Users\Abel\AppData\Local\NCrunch\15476\7\src\fsharp\FSharp.Core.Unittests\SurfaceArea.net40.fs C:\Users\Abel\AppData\Local\NCrunch\15476\7\Debug\net40\bin\CoreUnit_net40_Xml.xml
Unexpectedly missing (expected, not actual):
Microsoft.FSharp.Core.FSharpOption1[T]: Boolean IsNone     Microsoft.FSharp.Core.FSharpOption1[T]: Boolean IsSome
Unexpectedly present (actual, not expected):
Microsoft.FSharp.Core.FSharpOption1[T]: Boolean IsNone [Microsoft.FSharp.Core.FSharpOption1[T]]
Microsoft.FSharp.Core.FSharpOption1[T]: Boolean IsSome [Microsoft.FSharp.Core.FSharpOption1[T]]
at FSharp.Core.Unittests.LibraryTestFx.SurfaceArea.verify(String expected, String platform, String fileName) in D:\Projects\OpenSource\VisualFSharp\src\fsharp\FSharp.Core.Unittests\LibraryTestFx.fs:line 178
at FSharp.Core.Unittests.SurfaceArea.SurfaceAreaTest.VerifyArea() in D:\Projects\OpenSource\VisualFSharp\src\fsharp\FSharp.Core.Unittests\SurfaceArea.net40.fs:line 3534

I'm reporting this because I'm not sure where to look for a cause. I.,e., is it possible that the function signature inference fails because of possible differences in system configuration? I'd assume it shouldn't fail, and I think that master is only merged when there aren't any Unit Test errors (right?), which suggests this may be caused by a difference in system configuration.
As per the DEVGUIDE.md and TESTGUIDE.md, I compile and run the tests in x86 mode using standard NUnit v3 runners in an x86 host process. I tried x64 mode, but this had the same result.
(this was one of the findings after starting investigating test coverage, see #3579)

Could you look into why the test failure isn't failing CI?

@dsyme: I don't understand your question? I assume you mean the CI process run here, and then I assume that the mentioned error is simply not reported, unless of course the test is ignored. The latter seems doubtful, considering this checks the whole surface area.
I can check logs, but no idea where/how/what? Sorry, bit of a learning curve here...

Here's what I tried

Looking at build.cmd, the FSharp.Core unit tests are run in ci_part2 https://github.com/Microsoft/visualfsharp/blob/master/build.cmd#L248

Clicking on this on the front page of the repo:

takes us to https://ci2.dot.net/job/Microsoft_visualfsharp/job/master/job/release_ci_part2_windows_nt/

Clicking on build 612 takes us to https://ci2.dot.net/job/Microsoft_visualfsharp/job/master/job/release_ci_part2_windows_nt/612/

Looking at the plain text console log doesn't show any mention of VerifyArea

Looking at the "Build Artefacts" reveals the output log core unit log in the "TestResults" directory. However the log for the unit tests strangely  doesn't make any mention of VerifyArea or SurfaceArea https://ci2.dot.net/job/Microsoft_visualfsharp/job/master/job/release_ci_part2_windows_nt/612/artifact/tests/TestResults/test-net40-coreunit-output.log doesn't reveal an

@dsyme, tx for showing the steps. I've tried to locate any of the FSharp.Core.UnitTests I am seeing when running it locally and wondering now whether I am running the right things. I tried to find some tests in the different builds, but can't find SurfaceAreaTest either, nor MakeTypeType, TypeForwarding of JaroWinklerTest.
I'm missing a browsable interface that simple shows a report test-by-test, but perhaps Jenkins (I've never worked with that) doesn't provide one. I use TeamCity, where I get a browsable report for each test (and the log as a fallback as well). Anyway, I shouldn't complain about Jenkins, just need to get used to it I guess.

I tried to find some tests in the different builds, but can't find SurfaceAreaTest either, nor MakeTupleType, TypeForwarding of JaroWinklerTest.

I think the test is being executed on CI. Here's the execution of the unit tests in the CI log
"D:\j\workspace\release_ci_pa---94d249b3\packages\NUnit.Console.3.0.0\tools\nunit3-console.exe" --verbose --framework:V4.0 --result:"D:\j\workspace\release_ci_pa---94d249b3\tests\TestResults\test-net40-coreunit-results.xml;format=nunit3" --output:"D:\j\workspace\release_ci_pa---94d249b3\tests\TestResults\test-net40-coreunit-output.log"  --err:"D:\j\workspace\release_ci_pa---94d249b3\tests\TestResults\test-net40-coreunit-errors.log"  --work:"D:\j\workspace\release_ci_pa---94d249b3\release\net40\bin" "D:\j\workspace\release_ci_pa---94d249b3\release\net40\bin\FSharp.Core.Unittests.dll"
NUnit Console Runner 3.0.5797
Copyright (C) 2015 Charlie Poole

Runtime Environment
   OS Version: Microsoft Windows NT 10.0.14393.0
  CLR Version: 4.0.30319.42000

Test Files
    D:\j\workspace\release_ci_pa---94d249b3\release\net40\bin\FSharp.Core.Unittests.dll

Agent process 6176 starting
Running under version 4.0.30319.42000, Net 4.5
Agent process 6176 exiting

Tests Not Run

1) Explicit : FSharp.Core.Unittests.FSharp_Core.Microsoft_FSharp_Collections.CollectionModulesConsistency.unfold is consistent full

2) Explicit : FSharp.Core.Unittests.FSharp_Core.Microsoft_FSharp_Control.AsyncModule.Async.AwaitWaitHandle does not leak memory

3) Explicit : FSharp.Core.Unittests.FSharp_Core.Microsoft_FSharp_Control.AsyncModule.Async.Choice specification test

Run Settings
    RuntimeFramework: V4.0
    WorkDirectory: D:\j\workspace\release_ci_pa---94d249b3\release\net40\bin
    Verbose: True
    NumberOfTestWorkers: 4

Test Run Summary
    Overall result: Passed
   Tests run: 5336, Passed: 5336, Errors: 0, Failures: 0, Inconclusive: 0
     Not run: 3, Invalid: 0, Ignored: 0, Explicit: 3, Skipped: 0
  Start time: 2017-09-14 08:58:33Z
    End time: 2017-09-14 09:05:46Z
    Duration: 433.217 seconds

Here's an entry from the XML (not the "output log")
<test-suite id="0-6409" duration="431.973069" end-time="2017-09-14 09:05:45Z" start-time="2017-09-14 08:58:33Z" asserts="0" skipped="0" inconclusive="0" failed="0" passed="1" total="1" result="Passed" testcasecount="1" runstate="Runnable" fullname="FSharp.Core.Unittests.SurfaceArea" name="SurfaceArea" type="TestSuite">
<test-suite id="0-6181" duration="0.168959" end-time="2017-09-14 09:05:45Z" start-time="2017-09-14 09:05:45Z" asserts="0" skipped="0" inconclusive="0" failed="0" passed="1" total="1" result="Passed" testcasecount="1" runstate="Runnable" fullname="FSharp.Core.Unittests.SurfaceArea.SurfaceAreaTest" name="SurfaceAreaTest" type="TestFixture" classname="FSharp.Core.Unittests.SurfaceArea.SurfaceAreaTest">
<test-case id="0-6182" duration="0.165406" end-time="2017-09-14 09:05:45Z" start-time="2017-09-14 09:05:45Z" asserts="0" result="Passed" runstate="Runnable" fullname="FSharp.Core.Unittests.SurfaceArea.SurfaceAreaTest.VerifyArea" name="VerifyArea" classname="FSharp.Core.Unittests.SurfaceArea.SurfaceAreaTest" seed="1350782031" methodname="VerifyArea">
<reason>
<message>
<![CDATA[

]]>
</message>
</reason>
</test-case>
</test-suite>

MakeTupleType and TypeForwarding are also in that XML file. I presume the JaroWinkler one is in the FSharp.Compiler.UnitTests.dll log (a separate run of unit tests)

I still don't understand what's causing these differences on your machine though
> Unexpectedly missing (expected, not actual):
>     Microsoft.FSharp.Core.FSharpOption`1[T]: Boolean IsNone
>     Microsoft.FSharp.Core.FSharpOption`1[T]: Boolean IsSome
>
> Unexpectedly present (actual, not expected):
>     Microsoft.FSharp.Core.FSharpOption`1[T]: Boolean IsNone [Microsoft.FSharp.Core.FSharpOption`1[T]]
>     Microsoft.FSharp.Core.FSharpOption`1[T]: Boolean IsSome [Microsoft.FSharp.Core.FSharpOption`1[T]]

@dsyme, I was typing a response yesterday while my computer spontaneously restarted (this has happened a few times since recent installs of VS2017 previews and/or F# nightlies, and seem to happen only when running a lot of tests at once, but I have not yet determined a clear correlation, nothing in the event logs).
My response was essentially that I found the source. NCrunch adds instrumentation data to the methods/functions etc for getting coverage and timing information. For this particular function, it seems to have the effect that the signature of the function changes. I don't know why that is, but I guess that should be send as a report to NCrunch itself (unless of course I find that the cause is in the compiler).
Removing the instrumentation shows that the test runs successfully. I will close this bug report and reopen only if the cause is reproducibly related to the F# toolchain.


BUG: when setting the style to min-height or min-width in the popperStyles property the popper is opened at once without calling the click event

version 16.0.4
when setting the style to min-height or min-width in the popperStyles property the popper is opened at once without calling the click event
check this example please
https://stackblitz.com/edit/stackblitz-starters-4y8lgy?file=package.json,src%2Fmypopper%2Fmypopper.component.ts

Why would you set min-width and min-height that way?
Just use CSS.
I know that applied styles override the CSS.
I can't set a default for every CSS property.
There's no choice right now, due to the Chrome bug.

But do you really have to apply the popperStyles and the popperApplyClass when the popup is closed? Can you not conditionally only apply those when the popup is shown? Would that not solve all the problems we're having now?

But do you really have to apply the popperStyles and the popperApplyClass when the popup is closed? Can you not conditionally only apply those when the popup is shown? Would that not solve all the problems we're having now?

No it wouldn't solve anything and it would risk introducing more regressions. Plus it would degrade the performance if I added conditionals on everything.


lack of "a" before little

[Enter feedback here]
there is no "a" before little, so the statement will be negative expression。

Document Details
https://learn.microsoft.com/en-us/azure/storage/blobs/storage-blobs-latency#latency-metrics-for-block-blobs
Under normal conditions, there is little gap between end-to-end latency and server latency, which is what the image shows for the sample workload.
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 7c34470c-038e-5f44-9aeb-bdb098f9391e
Version Independent ID: 082f7f8e-08dd-b332-6feb-e076ae18b752
Content: Latency in Blob storage - Azure Storage
Content Source: articles/storage/blobs/storage-blobs-latency.md
Service: azure-blob-storage
GitHub Login: @akashdubey-ms
Microsoft Alias: akashdubey

@leoliudan
Thanks for your feedback! We will investigate and update as appropriate.

@leoliudan
Thank you for bringing this to our attention! We've initiated a pull request (PR) to address this issue. Once the author reviews the changes, they will be merged, and the updates should go live shortly. Your diligence is much appreciated!


Impossible to compile the project

Hi,
I'm getting this error during the compilation :
In file included from ../src/binding.cpp:1:
In file included from ../../nan/nan.h:192:
../../nan/nan_maybe_43_inl.h:112:15: error: no member named 'ForceSet' in 'v8::Object'
  return obj->ForceSet(isolate->GetCurrentContext(), key, value, attribs);
         ~~~  ^
In file included from ../src/binding.cpp:1:
../../nan/nan.h:834:18: warning: 'MakeCallback' is deprecated: Use MakeCallback(..., async_context) [-Wdeprecated-declarations]
    return node::MakeCallback(
                 ^
/Users/romain/.node-gyp/10.0.0/include/node/node.h:171:1: note: 'MakeCallback' has been explicitly marked deprecated here
NODE_DEPRECATED("Use MakeCallback(..., async_context)",
^
/Users/romain/.node-gyp/10.0.0/include/node/node.h:88:20: note: expanded from macro 'NODE_DEPRECATED'
    __attribute__((deprecated(message))) declarator
                   ^
In file included from ../src/binding.cpp:1:
../../nan/nan.h:849:18: warning: 'MakeCallback' is deprecated: Use MakeCallback(..., async_context) [-Wdeprecated-declarations]
    return node::MakeCallback(
                 ^
/Users/romain/.node-gyp/10.0.0/include/node/node.h:164:1: note: 'MakeCallback' has been explicitly marked deprecated here
NODE_DEPRECATED("Use MakeCallback(..., async_context)",
^
/Users/romain/.node-gyp/10.0.0/include/node/node.h:88:20: note: expanded from macro 'NODE_DEPRECATED'
    __attribute__((deprecated(message))) declarator
                   ^
In file included from ../src/binding.cpp:1:
../../nan/nan.h:864:18: warning: 'MakeCallback' is deprecated: Use MakeCallback(..., async_context) [-Wdeprecated-declarations]
    return node::MakeCallback(
                 ^
/Users/romain/.node-gyp/10.0.0/include/node/node.h:157:1: note: 'MakeCallback' has been explicitly marked deprecated here
NODE_DEPRECATED("Use MakeCallback(..., async_context)",
^
/Users/romain/.node-gyp/10.0.0/include/node/node.h:88:20: note: expanded from macro 'NODE_DEPRECATED'
    __attribute__((deprecated(message))) declarator
                   ^
In file included from ../src/binding.cpp:1:
../../nan/nan.h:1473:31: warning: 'MakeCallback' is deprecated: Use MakeCallback(..., async_context) [-Wdeprecated-declarations]
    return scope.Escape(node::MakeCallback(
                              ^
/Users/romain/.node-gyp/10.0.0/include/node/node.h:171:1: note: 'MakeCallback' has been explicitly marked deprecated here
NODE_DEPRECATED("Use MakeCallback(..., async_context)",
^
/Users/romain/.node-gyp/10.0.0/include/node/node.h:88:20: note: expanded from macro 'NODE_DEPRECATED'
    __attribute__((deprecated(message))) declarator
                   ^
4 warnings and 1 error generated.
make: *** [Release/obj.target/binding/src/binding.o] Error 1

@Valtyy Did this error started showing when you installed ionic2-super-tabs package? Could you remove it and try to build again?

@eduardoRoth Nevermind, it seems the problem is with the example project, not with the module. The error happens when I try to run npm -i on the example project


Add support for watchtower

Sorry about the excessive merge commits in here. I did them before I knew better.

Have a read of this - you can squash your commits into one sane one ;-)
http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html

So this is an interesting one. I'm currently working on a feature to pin containers to a specific image version so that upgrades are a bit more controlled. Bear in mind that a new image version can often equal a completely new software version with breaking changes - not cool for stability or someone that wants it to "just work" because they're fed up of FreeNAS (ie, me). After running ansible-nas for a year or so multiple containers were updated and subsequently busted by these types of upgrades - transmission-with-openvpn, grafana and Portainer come to mind.
That said, I get that there are people that want the latest and greatest as soon as they are available so I am happy to include Watchtower - I just don't think it should be enabled by default.

Sure, I can see the the desire for stability over all. Definitely an important feature on the "just work" front.
That said, there are options to manage which containers get auto updated  or not:
--label-enable Watch containers where the com.centurylinklabs.watchtower.enable label is set to true.
And no problems about not having it enabled by default. I'll switch that around and try to squash. I might have to delete my remote repo to do it, so not sure what will happen to this PR :D

Closing this due to new, improved, clean history PR #24


Round input from range widget.

In certain browsers the range widget gives values like 7.0000000000000001.

The problem was reproduced in Chrome. It seems to be a browser issue as Chrome is sending odd values to the RDMO backend although different ones were picked in the range selector.  Firefox does not do that.
Here is what the Angular Docs say:
However, step is currently only fully implemented by Firefox. Other browsers have problems when the step value changes dynamically - they do not adjust the element value correctly, but instead may set the stepMismatch error. If that's the case, the AngularJS will set the step error on the input, and set the model to undefined.


Fix Mapper::query() method to pass param types.

When using custom query, it's not possible to use custom types for query parameters. For example:
$mapper->query(
    'SELECT * FROM user WHERE state IN (?)',
    [
        ['pending', 'active']
    ],
    [
        \Doctrine\DBAL\Connection::PARAM_STR_ARRAY
    ]
);

This pull request fixes the problem by passing custom param types to \Doctrine\DBAL\Connection::executeQuery() via Mapper::query().

Off: The custom query above is just an example, right? Because you don't need to use a custom query for that. You can execute it like this: $mapper->where(['state IN' => ['pending', 'active']]);
On: I can't say anything about the usefullness of passing through the custom param type.

Yes, it's just an example for the sake of simplicity. Intended usage of \Spot\Mapper::query() is for custom, complex queries, where you might also have typed parameters. Currently using query() you can't specify them, however it's supported directly by DBAL (see \Doctrine\DBAL\Connection::executeQuery). The method just doesn't proxy them. IMHO this is unnecessary limitation of Spot.

Yes, it's just an example for the sake of simplicity. Intended usage of \Spot\Mapper::query() is for custom, complex queries, where you might also have typed parameters. Currently using query() you can't specify them, however it's supported directly by DBAL (see \Doctrine\DBAL\Connection::executeQuery). The method just doesn't proxy them. IMHO this is unnecessary limitation of Spot.

I agree!

Needs tests, otherwise looks good to me. Valitron bug was already fixed with #217.


RFC: a new design of RisingWave CRD

Signed-off-by: arkbriar<EMAIL_ADDRESS>What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Propose a new design of RisingWave CRD to pursue flexibility and ease of use, including

a new CR RisingWavePodTemplate to declare common pod templates for RisingWave
a re-designed CR RisingWave

Draft a prototype structure of the new CRs

P.S. we will be able to create RisingWave instances in a mixed arch environment when this RFC is implemented.
Checklist

[x] I have written the necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

Looking forward to these changes!


Error on opening repo in a codespace or locally in devcontainer

The error happens on building devcontainer, related to run make  install-cli
Please note, that we did not make any updates to repo, just started with a new docker installation and previously was running fine
#28 8.066   self.initialize_options() #28 8.069 error: cryptography 41.0.1 is installed but cryptography<41,>=0.6 is required by {'msal'} #28 8.099 	 2023-06-02 11:44:19.259Z: #28 ERROR: executor failed running [/bin/bash -o pipefail -c make install-cli && echo -e "\n# Set up tre completion\nsource <(_TRE_COMPLETE=bash_source tre)" >> ~/.bashrc]: exit code: 2
I have also attached the full log:
creation.log
Steps to reproduce:

Clone repo https://github.com/microsoft/AzureTRE-Deployment in
open in visual studio code
Reopen in container - fails
Or:
Open in codespace - the same error.

This happens because msal==1.20.0 requires cryptography<41. msal==1.22.0 would work with cryptography<43, unfortunately the latest version azure-cli-core requires msal==1.20.0.
Fix : enforce cryptography<41 in /cli/setupy.py Line 42:
install_requires=[
        "click==8.1.3",
        "httpx~=0.23.1",
        "cryptography<41",
        "msal==1.20.0",
        "jmespath==1.0.1",
        "tabulate==0.9.0",
        "pygments==2.14.0",
        "PyJWT==2.6.0",
        "azure-cli-core==2.47.0",
        "azure-identity==1.12.0",
        "aiohttp==3.8.4"
    ],

or from the AzureTRE-Deployment repository, add the following line at the end of ,devcontainer/scripts/install-azure-tre-oss.sh:
sed -i 's|"msal==1.20.0"|"cryptography<41",\n"msal==1.20.0"|g' "$oss_home/cli/setup.py"

Thanks @jlabhard-sg . That works

@marrobi Please re-open this and apply the fix to the repo

@SvenAelterman reopened.
If anybody has time to help with the fix would be appreciated.

@marrobi I can create a PR for the fix provided by @jlabhard-sg if that's the right thing to do. (Confirmed it works, but I don't know if there's a better way to do it?)

I have no better suggestions at present. If you can do a PR that would be very helpful and the team can review.


Fill & Submit the AI disclosure form

Fill and submit the AI disclosure form, and save a PDF & add to assignment-3 repository with a link on README file.

Submitted in Github classroom repo


Create CM for configuration of request redirect to tunnel server

Found no  such Comfigmap: kube-system/yurttunnel-server-cfg created when use yurtctl when I try this:
https://github.com/alibaba/openyurt/issues/138#issuecomment-723555668

@vincent-pli thanks for submitting the PR. I just have a question. Can we create the configmap manually?

@charleszheng44
yes, the definition of configmap is existed in config/setup/yurt-tunnel-server.yaml

thanks for explaining


the challenge, although its right, will not submit

Challenge Nest an Anchor Element within a Paragraph has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_5) AppleWebKit/601.2.7 (KHTML, like Gecko) Version/9.0.1 Safari/537.86.2.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
<link href="http://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

cat photos

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

View more  cat photos

The code you have provided does not meet the requirements in the instructions:

Nest your existing a element within a new p element (just after the existing h2 element) so that the surrounding paragraph says "View more cat photos", but where only "cat photos" is a link, and the rest of the text is plain text.

The main issue with your code is you have not done this to the existing anchor tag above the image, but have instead created a second anchor at the bottom.

You should be nesting the existing anchor, not adding a new one. If you add a new one and leave the old one it finds the old one, sees it isn't nested, and fails the test. Please visit the Help Chat if you need more help. Thanks and happy coding!


use installed namespace for CREDS_SECRET_NAMESPACE

Hi
we cannot use cluster credentials for Google Artifact Registry, etc, when we install kubeclarity by helm to our namespace except kubeclarity.
we have two options;

create kubeclarity namespace, and then create cluster credentials to the namespace.

set CREDS_SECRET_NAMESPACE with installed namespace at generating manifests by helm

I think option 2 is better.
If you think, please merge this :)
And if you want, I will add documentation at following section in README.md.
https://github.com/openclarity/kubeclarity#private-registries-support-for-k8s-runtime-scan

@FrimIdan
sorry for late.
I added documentations for CREDS_SECRET_NAMESPACE.


Added comment and changed sum of pointer to Ether and ether len to

packet.unparsed() in packet.AddVLANTag().

Approved


clairctl analyze is finding 0 vulnerabilities

I'm trying to analyze a Docker image from a private registry:
docker  exec -i -t clairctl-clairctl clairctl --log-level debug analyze <my-registry>/ubuntu_trusty:latest
(the postgres, clair, clairctl docker containers orchestration is done via puppet)
and it looks like it cannot find any vulnerabilities (below is the last part from the command output):
2017-09-04 13:21:11.797534 I | clair: analysing layer [sha256:a3ed9] 1/12
2017-09-04 13:21:11.808732 I | clair: analysing layer [sha256:76a4c] 2/12
2017-09-04 13:21:11.814726 I | clair: analysing layer [sha256:d2ff4] 3/12
2017-09-04 13:21:11.820531 I | clair: analysing layer [sha256:f94ad] 4/12
2017-09-04 13:21:11.826078 I | clair: analysing layer [sha256:a3ed9] 5/12
2017-09-04 13:21:11.831556 I | clair: analysing layer [sha256:a3ed9] 6/12
2017-09-04 13:21:11.843649 I | clair: analysing layer [sha256:f9f3f] 7/12
2017-09-04 13:21:11.853565 I | clair: analysing layer [sha256:26aa3] 8/12
2017-09-04 13:21:11.863123 I | clair: analysing layer [sha256:b4cbd] 9/12
2017-09-04 13:21:11.870702 I | clair: analysing layer [sha256:329fe] 10/12
2017-09-04 13:21:11.876297 I | clair: analysing layer [sha256:8e113] 11/12
2017-09-04 13:21:11.881848 I | clair: analysing layer [sha256:a3ed9] 12/12
2017-09-04 13:21:11.881888 D | cmd: Using priority filters:

Image: <my-registry>/<my-registry>/ubuntu_trusty:latest

 Unknown: 0
 Negligible: 0
 Low: 0
 Medium: 0
 High: 0
 Critical: 0
 Defcon1: 0

I checked the postgres database and it seems that the 'vulnerability' table has been populated with CVEs.
Why is this happening?

Is it ubuntu_trusty unsupported or too old? I tried analyzing ubuntu_xenial and there were vulnerabilities found.

Hello @transcedentalia, I think this is an issue / question for https://github.com/coreos/clair.
Clairctl only provides a CLI for Clair. According to the documentation, Trusty should show be analyzed.

@transcedentalia you found the solution?
Even am finding a exact same behavior while analyzing private nexus rep image

Why does registry address seems like //ubuntu_trusty:latest ?


Adding permissions, giving read seems to give write access as well

This is so we can write to the shared directory if we need to.

The permission reference seems to indicate that getting the WRITE_EXTERNAL_STORAGE permission implicitly gives READ_EXTERNAL_STORAGE, not the other way around. Do we depend on the read permission granting write?


keystore.js

keystore.js

Codecov Report

Merging #8 into master will increase coverage by 0.89%.
The diff coverage is 83.13%.

@@            Coverage Diff            @@
##           master      #8      +/-   ##
=========================================
+ Coverage   77.51%   78.4%   +0.89%
=========================================
  Files           5       6       +1
  Lines         169     176       +7
  Branches       28      28
=========================================
+ Hits          131     138       +7
  Misses         38      38

Impacted Files
Coverage Δ

lib/wallet.js
63.93% <100%> (-9.6%)
:arrow_down:

lib/keystore.js
82.92% <82.92%> (ø)

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 4775c59...1c79799. Read the comment docs.


mommy can now change her true nature for you~

This adds the ability to say "cargo mommy i mean daddy" ("daddy" here is an arbitrary value), which makes cargo-mommy copy itself to a new binary called cargo-daddy. As a result of how cargo <subcommand> ... works, this makes cargo daddy ... now resolve.
It also changes cargo-mommy to determine it's "true role" from the name of its binary. So in the above case, cargo daddy would detect its "true role" is "daddy". This makes the default role be "daddy" (was "mommy") and it renames all the env-vars we'd query from CARGO_MOMMYS_* to CARGO_DADDYS_*.
Also as a bonus cargo mommy i mean mommy test just runs cargo mommy test
mommy's happy to change her true nature for you~

fixes #27

Also seems odd that the default pronoun for daddy is her, still? I'm gay, so it doesn't reaaaally matter to me, but just a thought.

...honestly I think I've met more daddies (she/her) than I've met daddies (he/him).

...honestly I think I've met more daddies (she/her) than I've met daddies (he/him).

I wonder how much this is due to one's own social circle; in my queer circles, certainly, but also I am friends with a lot of straight women who I think would beg to differ (see: Pedro Pascal, for example)

Ah, well, I'm certainly not un-familiar with the other kind. :^) I just, hm, how to say,
I'm going to go with "'Sir' is a very choice honorific."
( Also honestly it's been at least one Mass Hatching Event since I've caught up with most people. So social circle definitely does matter, but partly because that's equally likely to throw off my numbers. )

You could already change the default role and pronouns (and still can) the "killer app" of this feature is erasing the last vestiges of "mommy" from the interface/config for people who specifically have issues with that term.


Overflow vulnerability in mul instruction in Goldilocks field

In our RISC-V machine, we use the following constraints to implement multiplication:
    // Multiply two 32-bits unsigned, return the upper and lower unsigned 32-bit
    // halves of the result.
    // X is the lower half (least significant bits)
    // Y is the higher half (most significant bits)
    instr mul Z, W -> X, Y {
        Z * W = X + Y * 2**32,
        X = X_b1 + X_b2 * 0x100 + X_b3 * 0x10000 + X_b4 * 0x1000000,
        Y = Y_b5 + Y_b6 * 0x100 + Y_b7 * 0x10000 + Y_b8 * 0x1000000
    }

where ``X_b1, ..., X_b4, Y_b5, ... Y_b8` are range-constrained to be bytes.
The problem is that the right-hand side of the first equation can overflow in the Goldilocks field (where $p = 2^{64} - 2^{32} + 1$ = 0xffffffff00000001).
For example, to prove the lower half of 1 * 1 = 2, set Y = 0xffffffff and X=0x00000002. With these values, the right-hand side of the first constraint will evaluate to 1:
X + Y * 2**32 = 0xffffffff00000002 = p + 1 = 1
Both X and Y can be decomposed into 4 bytes, so constraints 2 & 3 will very as well.
Note that this is not a problem on the larger BN254 field. On the Goldilocks field, this could be fixed by using the SplitGL machine from the standard library.

We should really fix this.


don't track locked lock for the same reason as RefCell

Thanks for the pull request. Multi-thread is hard and this makes me think about it more and discovered some issues (ex. ba9a12f6aeba18503687ed097e4cdd76f5173778).
Regarding on this PR, I think it's better to just panic immediately if the lock cannot be obtained, instead of blocking (the existing code) or ignoring (the PR). This is because the current code expects all Mutex to be released when the collector is running (or the collector does not run until they are released). If not, then it's a logic error that is ideally surfaced early so it gets noticed.
Right now, the multi-thread collector needs to "stop the world" to be safe and correct. More precisely, it needs to "stop other threads from modifying the object list and object dependency graph" while it is collecting things.

The object list: The collector needs to prevent object creation or deletion, as it wants exclusive access to the linked list and other objects should not mutate it.
The object dependency graph: The collector calls the trace API twice to figure out what objects are alive, and to mark objects as alive. If the trace result suddenly changes that revives some objects that the collector decided to drop, then it'd be bad.

For the object list, the collector takes control of object creation and deletion so it can lock them accordingly.
For the dependency graph, it becomes tricky. Because any T could have some internal mutability that changes what trace visits, and that'd be a nightmare for the collector.
To address the issue, the current solution is to prevent access to T while the collector is running. This is done by the ThreadedCcRef type. ThreadedCcRef is expected to be the only way to access a T stored in ThreadedCc<T> and ThreadedCcRef takes a lock so collector does not run. This is a stronger guarantee than just preventing modifications to the object dependency grpah. However it seems to work well and the Rust borrow checker helps avoiding issues.
Let's go back to Mutex. The only way for a thread to take a mutex lock is to get a MutexGuard object borrowed from a Mutex. The only way to get a Mutex in a ThreadedCc<Mutex> is to get a ThreadedCcRef first. So to lock there needs to be a borrow chain: ThreadedCc -> ThreadedCcRef -> Mutex -> MutexGuard. If ThreadedCcRef goes away, then the borrow checker ensures MutexGuard goes away too. And we already know that ThreadedCcRef and the collector are exclusive. So when the collector is running, there are no ThreadedCcRef and no MutexGuards.
There are some special cases, though. For example, ThreadedCc<Arc<Mutex<T>> allows mutating the T without going through ThreadedCcRef. How do we deal with that? Well, Arc is considered acyclic. Therefore from the collector's point of view, even if T is mutated by other threads, the dependency graph does not change.
Now, if you get this far and the above makes sense for you, could you update the code and comments about why we're doing this?


Scroll View Issue

I am having layout issue with ScrollView
When I use View it was displaying like this.
http://collabshot.com/show/Kis4aJ
http://pastebin.com/Yagt8X38
When I changed it to scrollview
http://collabshot.com/show/f3lEtM
http://pastebin.com/zK0HnjaN
And here are the styles
content: {
flex: 1,
paddingTop: 10,
paddingLeft: 10,
paddingRight: 10,
flexDirection: 'column',
backgroundColor: '#000000',
},
transmit: {
height: 100,
borderWidth: 1,
},
messages: {
flex: 1,
backgroundColor: '#ffff00',
}
Looking for help.
Thanks

Hey @georgi-kovachev . thanks for reporting this! You can try setting the contentContainerStyle of the ScrollView.
BTW, there's an awesome place to ask question like this one: StackOverflow. It's the best system for Q&A. Many people from the community hang out there and will be able to see your question, you can vote on answers and mark question as answered etc. This lets us keep a list of bug reports and feature requests on github and especially Product Pains (again, with voting which is really nice).
If you think StackOverflow works for you please consider posting there instead and closing this issue.
I'm posting this here because github issues haven't been working very well for us and because StackOverflow is so much better. Thanks for reading! :)


feat: custom prisma code block

custom theme based on ruby-dark
custom prisma code blocks
custom decoration (inspired by effect)

https://github.com/user-attachments/assets/9cc3e6cd-2929-4c84-a470-b19c402e943e

"OK" seems like a weird copy confirmation message, how hard would that be to change?

Ha. What would you like? "Copied!" was there before. I expect you'd like that back? Done!
I don't like computers using exclamation points.


missing automatic loading of FHI-aims geometry.in and VASP POSCAR files

pyProbeParticle/basUtils.py - loadGeometry(fname=None,params=None)
https://github.com/Probe-Particle/ProbeParticleModel/blob/a97b04adcfea4228384c4bd946001599c34b4a83/pyProbeParticle/basUtils.py#L440
no flag for something like is_in - meaning is geometry.in file from FHI-aims. It could have names ending on .in or .in.next_step.
there is a procedure for reading the geometry.in files here:
loadGeometryIN
https://github.com/Probe-Particle/ProbeParticleModel/blob/a97b04adcfea4228384c4bd946001599c34b4a83/pyProbeParticle/basUtils.py#L130
If file is named POSCAR or CONTCAR we can use this procedure:
loadPOSCAR
https://github.com/Probe-Particle/ProbeParticleModel/blob/a97b04adcfea4228384c4bd946001599c34b4a83/pyProbeParticle/basUtils.py#L149
We just need to be sure, that the atoms format (xyzs in case of these 2 procedures) is the same as for the other procedures used in loadGeometry.

Maybe loadGeometry should have another argument called format that you could use to by-pass the identification from file extension. That way you can put additional extensions like .next_step to any file format if you manually specify what the actual format is by the argument.

Sorry for repeating myself, but it does seem to me that switching to ASE could solve this also.

I suggest we (actually, I am gonna do it next week, hopefully) try to resolve the issue by a simple patch for now, in a way @ondrejkrejci proposed above: adding a tag is_in to io.py for the *.in and *.in.next_step geometry files and also something like is_vaspcar if the file name begins with POSCAR* or CONTCAR*. I would also add the format argument as suggested by @NikoOinonen in case the identification by file name fails. (Should it be --geometry_format or just --format or something else?) We can switch to ASE later if there is more demand for that.

I think I was originally thinking that it would be something like
def loadGeometry(fname=None, params=None, format=None):
    ...
    if format is None:
        format = inferFromFileExtension(fname)

    if format == 'in':
        geom = loadGeometryIN(fname)
    elif format == 'vaspcar':
        geom = loadPOSCAR(fname)
        ... # And all other formats
    else:
        raise ValueError('Invalid file format')
    ...

That way you get away with a single added argument that can be directly passed from the script, and it's easy to add new file formats later.


adds objc_show method for Objective-C compatibility

Resolves issue #9
Hey Kevin! Thanks so much for contributing this library to open source. It's very well written and easy to use. In the process of including it in a project written primarily in Objective-C, I ended up writing the objc_show method that you had described in issue #9 and it seems to be working just fine so far (though I haven't been making use of the values returned by the show() method).
Cheers!
~Mark

Hello @markedwardmurray !
Thanks for this PR :) 👍

thanks for accepting @kevin-hirsch :grin:

Thanks to you ! 👍