Optimization and Clarity Enhancement for Intro to Plugins Documentation

Hey, I just made a Pull Request!
Optimization and Clarity Enhancement for Intro to Plugins Documentation
This pull request introduces a series of enhancements aimed at optimizing the structure of the "Introduction to Plugins" documentation. The initial commit refines the document for clarity, readability, and markdown best practices, setting a solid foundation for further updates. Subsequent commits will address specific requirements and suggestions arising from issue #21945 on the Backstage GitHub repository.
Key Highlights:

Initial Commit: Streamlines language, updates link descriptions for better clarity, and revises the document's layout for improved navigability and user experience. This includes refining alt text for images to enhance accessibility.
Following Commits: Will incorporate feedback and suggestions as detailed in issue #21945, focusing on expanding the documentation to cover new plugin development guidelines, integration tips, and examples.
New Table of Contents: Introducing a "Coming Soon: Table of Contents" for "Backstage Integration Essentials: A Complete Guide to Plugin Development," offering a sneak peek into future documentation structure and topics, including foundational development concepts, advanced integration techniques, deployment best practices, troubleshooting, and real-world case studies.

Objective:
My goal is to ensure that the "Introduction to Plugins" section serves as a comprehensive, easily navigable, and user-friendly resource for both new and existing Backstage contributors. By aligning this documentation more closely with community feedback and the evolving landscape of plugin development, I aim to foster a more vibrant and collaborative ecosystem.
Invitation for Collaboration:
Feedback, suggestions, and contributions are welcomed to continuously improve this documentation. If you have insights or enhancements to propose, please feel free to comment on this PR or contribute directly to the discussion in issue #21945.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@aramissennyeydd, I will incorporate these and my new work this week. Thank you for all the feedback!

@markjacksonfishing Looking forward to it!

@aramissennyeydd can you remove the stale label? I have work to submit but still working through it.

@markjacksonfishing If you comment, it will remove the label if you don't have access to directly remove it,  "shoo stalebot 🌬 " is a favorite of mine.

Still working on this after travels.

I will have the updated committed by eod today.

I am still working through this—a lot of material to pull in and adjust, but it is still in progress and active.

Today I will have the edits addressed.

I will continue debugging the GHA failure tomorrow.

Thank you! I'm gonna rudely mark this as ready for review so that we get some maintainers assigned

@Rugvip i will tinker with the GHA later this morning.

@markjacksonfishing what's the GHA problem you're aiming to fix?

@markjacksonfishing, what's the GHA problem you're aiming to fix?

@Rugvip, mainly this one, but I will be very transparent about the fact that the error could totally be me missing something.

@markjacksonfishing @Rugvip I think something's up with the microsite build step, getting this error in #24069 as well

@markjacksonfishing @Rugvip I think something's up with the microsite build step, getting this error in #24069 as well

@aramissennyeydd Yep, same thing. I attempted to untwine the ball o string here, but that opened a pandora error box, and I reverted it.

@aramissennyeydd, do the items I added to the outline look suitable? If so, I have more to add, but I want to ensure we are aligned.

Sure, will give it a look. Also seems a conflict crept in.

@aramissennyeydd if the table of contents is good, I will get moving (already have items).

@markjacksonfishing Go for it! 🚀

Just wanted to give an update that I am still working through the edits and new content. More to come this week.

This is still ongoing. I have items ready, but I'm doing some final checks

Hi @markjacksonfishing, I saw you push some commits but wasn't sure if you wanted me to review again. Can you let me know, please?

@awanlin no review yet. More coming 🥳

Will have some today to add

Currently working on diagrams

More coming tomorrow evening. I have been traveling.

More coming tomorrow evening as I have been traveling.

@aramissennyeydd @awanlin
I’ve addressed all the comments from the previous review and made the necessary updates. Based on the discussion in the last docs SIG meeting, we agreed to finalize the index.md in this pull request. Once this is merged, we’ll shift our focus to writing the content.
I apologize for this taking longer than expected. Thank you for your patience. Please review the changes when you have a moment. Looking forward to your feedback!
Thank you!

@aramissennyeydd @awanlin Is this ok to merge? I think we agreed to merge this and that will start the further writing process?

@markjacksonfishing, yes, lets get it merged!
@Rugvip or @benjdlambert, can you merge this, please?

Checking back in on this @awanlin

Hi @markjacksonfishing, I can't merge until we get an approval form the Maintainers.

I'm going to hold this until next week's doc sig meeting. I will make sure it gets added to the agenda.

@aramissennyeydd @awanlin still waiting on confirmation of the outline. We had agreed it was good to go but this still hasn't been merged.

@Parsifal-M
I closed this due to inactivity and context switched. I also unassigned the issue from me, so it is available if you want to pick it up and use what was here.


flatDir repositories not inherited by upstream projects

Example project reproducing this issue: https://github.com/eriwen/gradle-repro-1352
Expected Behavior
Given a subproject with a flatDir repository, it should be inherited and used from parent projects.
Current Behavior
It seems as though flatDir repositories are not inherited by parent projects (or some underlying problem).
Workaround
Declare same repositories {} in parent project.
Context
Migrated from GRADLE-1940 with 3 votes
Steps to Reproduce (for bugs)
Run gradle build in example project
Your Environment
This issue affects 1.0 through 3.4

This works as designed, repositories are project scoped. This also doesn't just apply to flat dir repositories, but to any repositories.

This does not justify the current behaviour, I think, at least not completely. If you look at the example project, it's project B that depends on a JAR in its own flatDir repository. Project A does not have any direct dependency on that JAR, it just inherits it from B. So, project A by itself does not need any JAR from that flatDir, however it still does not build unless you add an (apparently useless) flatDir declaration on A too.

It does need that repository, to fetch the transitive dependency. The simplest workaround is defining the same repositories for all projects once, in an allprojects block.


Typed parameters and docstrings.

Hi Developers of BiomationScripterLib©,
I am working on a tool using BiomationScripterLib©, where programmatic introspection occurs on the templates.
What I would like:

For the opentrons templates, typed parameters like how the echo templates have.
Docstrings, specifically for templates, I.e. a brief textual description of the template and each parameter.

I've added some typing to the OTProto templates, does this do what you need? #48


Auto Theme Switch

There should be an option to schedule themes. For example: Use Dark mode from 7:00 PM to 7:00 AM. At 7:00 AM, VS Code should automatically switch back to light theme. There should also be an option to choose our default light and dark modes. For example:
Default Light Mode: Bluloco Light
Default Dark Mode: Solarized Dark
Light Mode Time: 7:00 AM
Dark Mode Time: 7:00 PM
Although this is possible through an extension which I found in the Visual Studio Marketplace, I feel this feature should come built-in.

I think this a good example of functionality that should be provided by an extension, or even the OS. Note that we already have the feature to change themes based on the OS scheme.
All the APIs are there. Extension can innovate, and get as feature rich they want.

Can you please tell me how I can set the theme to  change based on the OS Schemes?
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From: Martin @.>
Sent: 06 July 2021 01:01 PM
To: @.>
Cc: @.>; @.>
Subject: Re: [microsoft/vscode] Auto Theme Switch (#128026)
I think this a good example of functionality that should be provided by an extension, or even the OS. Note that we already have the feature to change themes based on the OS scheme.
All the APIs are there. Extension can innovate, and get as feature rich they want.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode/issues/128026#issuecomment-874529599, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUMH2PFW4PNGYRIN5BUH36TTWKWL7ANCNFSM473Y6AXA.

The setting is "window.autoDetectColorScheme": true. Additionally, you can define your preferred dark and light theme with workbench.preferredDarkColorTheme and workbench.preferredLightColorTheme


feature/warning-built-lineages

Add warning if ranks or root_node is used after build_lineages() was executed since results may differ

Codecov Report

Merging #5 (17c706e) into dev (a780311) will decrease coverage by 0.16%.
The diff coverage is 80.00%.

@@            Coverage Diff             @@
##              dev       #5      +/-   ##
==========================================
- Coverage   97.19%   97.03%   -0.17%
==========================================
  Files          15       15
  Lines        1176     1179       +3
==========================================
+ Hits         1143     1144       +1
- Misses         33       35       +2

Impacted Files
Coverage Δ

multitax/multitax.py
97.00% <80.00%> (-0.81%)
:arrow_down:

tests/multitax/unit/test_functions.py
100.00% <0.00%> (ø)

tests/multitax/integration/test_common.py
100.00% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Adds minLength parameter to options

If minLength is defined, the customSearch() is triggered only when the length of the searchString equals or exceeds the minLength value

Thanks for the PR, but this is not a feature I like to have in the lib unfortunatley.


update dodumentation

this PR:

updates old installation documentation
updates sphinx
removes linked autodocdir

Looks good. Please merge.


Move Formatting Utility to the right folder (winforms => windowsforms)

Move the .NET Core WinForms Formatting Utility sample code to the folder where all other Windows Forms are.

I think the changes are correct, but it worth a review because rpetrusha thought that the task not straightforward like that. (See https://github.com/dotnet/samples/pull/1426)

@Youssef1313 Thank you for pointing that out!
To reiterate the steps pointed out in the linked PR:

[ ] Instead of moving the content, you need to clone the content into the new place and update the urlFragment in the metadata for both the VB and CS readme files. Change the part that is winforms to windowsforms
[ ] Publish the sample (@Thraka will handle)
[ ] File an issue in the dotnet/docs repo to update links to the sample to the new location (@thraka will handle)
[ ] File an issue in the dotnet/dotnet-api-docs repo to update links to the sample to the new location (@thraka will handle)
[ ] Delete the sample from the winforms folder (@augustoproiete or @Youssef1313 can do that)

Thanks @Youssef1313 and @Thraka
I've completed the first step above and updated the pull-request:
✅  Clone the content into the new place and update the urlFragment in the metadata for both the VB and CS readme files


Delhi is missing in Map !!! Please check it

lear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to website
Scroll down to Map view
Then check delhi in map view

OS: iOS
Browser Safari
Version 13.4

i

lear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to website
Scroll down to Map view
Then check delhi in map view

OS: iOS
Browser Safari
Version 13.4

Delhi is there

Closing cause Delhi is there and seems to be working fine :P


Import/Export user data

Could you please add a feature to export/import repo data (comments etc.). There is probably an API call for this, but I what I mean is a button "Export data" such that I can export everything and re-import into another Gogs instance.
Thanks.

https://discuss.gogs.io/t/how-to-backup-restore-and-migrate/991

The link you provide describes how to backup data for admins, not for users!

OK... then what is the point "for users"? I don't think it's going to be implemented.


Update gitignore to correctly exclude multiple env files

When deploying a project to Heroku, we might create mutliple env files (i.e.: .env.production or .env.staging).
It will be a nice and easy change in this line: https://github.com/thinknimble/tn-spa-bootstrapper/blob/main/{{cookiecutter.project_slug}}/.gitignore#L78
.env*
!.env.example

Hi could I do this one?

Hi could I do this one?

Sure! Take it on

I'm having issues pushing the fix. 'remote: Permission to thinknimble/tn-spa-bootstrapper.git denied to JohnPark24'

I'm having issues pushing the fix. 'remote: Permission to thinknimble/tn-spa-bootstrapper.git denied to JohnPark24' fatal: unable to access 'https://github.com/thinknimble/tn-spa-bootstrapper.git/': The requested URL returned error: 403

@JohnPark24 Maybe you can try forking the repo, uploading the changes in your fork and then creating a PR?

That solved it. Thank you very much


[c++] Fix build on MacOS 13.3

Issue and/or context:
The third-party catch2 package has
/Users/johnkerl/git/single-cell-data/TileDB-SOMA/build/libtiledbsoma/_deps/catch2-src/src/catch2/catch_session.cpp:64:25: warning: variable 'reporterIdx' set but not used [-Wunused-but-set-variable]
            std::size_t reporterIdx = 0;

which is not new,  but, newly this errors out in MacOS 13.3 (which yours truly installed just today).
Changes:
Since we don't control this third-party library but we do control how we compile it, apply the correct flags.

Codecov Report
Patch coverage has no change and project coverage change: +26.31 :tada:

Comparison is base (4cce88c) 64.90% compared to head (5e97b38) 91.21%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1206       +/-   ##
===========================================
+ Coverage   64.90%   91.21%   +26.31%
===========================================
  Files          91       30       -61
  Lines        6736     2357     -4379
===========================================
- Hits         4372     2150     -2222
+ Misses       2364      207     -2157

Flag
Coverage Δ

python
91.21% <ø> (ø)

r
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 61 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

A bit late here but thanks for the fix!


Remove exponent parameter from _Pauli{X,Y,Z}

Raising a Pauli to an arbitrary exponent generally does not return another Pauli. Therefore, it doesn't make sense for the constructor of these (private) classes to take the parameter.
Note that functions like __pow__ do the right thing by instantiating an appropriate gate from common_gates.
Also note that this requires custom implementation of algorithms that use with_exponent. I think there is just one implemented in EigenGate. _equal_up_to_global_phase_.

If we remove the exponent from constructor of the Paulis, we'll have to override the _with_exponent virtual method in eigen_gate.py. However, mypy won't allow me to return an XPowGate from _PauliX because of the constraint in original function that return type should be same as self. As a workaround, I had to modify the return type constraint in _with_exponent and _resolve_parameters_ to return an EigenGate instead of TSelf, similar to __pow__.


Include OER

Add the topic Open Education Resources

This might be quite difficult to grasp due to it's meta-meta-character. Or rather, this will be difficult to operationalize since open science resources are open by default and therefore de-facto open educational resources (are they?)? On the other hand this project itself can be seen as OER.
On the other hand there is quite an interesting aspect here: how can the open resources in a project that is conducted according to the principles of open science, be processed/prepared/adapted/embedded in a way that they can be used as teaching materials within education?

For all topic suggestions we have an etherpad collecting material and ideas: https://pad.okfn.org/p/OpenScience101topicsuggestions
Pleas vote for this topic at this comment here with a thumps up reaction (click on the +smiley icon in the top right of the comment)!

[…] since open science resources are open by default and therefore de-facto open educational resources […]

Not sure this is true. AFAIK (and I could be very wrong of course), OER refers to things like textbooks published under open licences.

@RaoOfPhysics hm...initially it was, yes. But I think we progressed from there and be more inclusive - media and documents seem to be quite common nowadays. Don't you think?

Ah, I see what you mean. Apologies, I saw it as "open science" resources, rather than "open science resources", if that makes any sense.

@RaoOfPhysics Haha..don't worry....any feedback is always highly welcome since it always challenges us to think about (or rethink) our ideas!

That chapter could

give an overview of OER in general, linking to the many good resources on the topic that are already out there
describe how open science resources can serve as OER both in the sense of something more like a textbook (e.g. older materials) and of something more like an excursion (e.g. into lab notebooks that are being written to as we watch, which is one of the things that would fall under #1 )
state explicitly that the current project could serve as an OER in both senses (though the latter only while we're still on it)


Change link button margin

We're now only using this at the foot of long-form pages like the 'view-brief' page so the margin-left isn't needed.
Before:

After:

:+1:, do we need a version bump?

Yep, gonna do a PR for the version bump now.


MAP_FAILED

when i try to flash with this error occur
2014-12-13T21:32:29 INFO src/stlink-common.c: Loading device parameters....
2014-12-13T21:32:29 INFO src/stlink-common.c: Device connected is: F4 device, id 0x10016413
2014-12-13T21:32:29 INFO src/stlink-common.c: SRAM size: 0x30000 bytes (192 KiB), Flash: 0x100000 bytes (1024 KiB) in pages of 16384 bytes
mmap() == MAP_FAILED
2014-12-13T21:32:29 ERROR src/stlink-common.c: map_file() == -1
stlink_fwrite_flash() == -1

i use gcc arm cortex m  template
here is the map file
MEMORY
{
  FLASH (rx) : ORIGIN = 0x08000000, LENGTH = 1024K
  RAM (xrw) : ORIGIN = 0x20000000, LENGTH = 192K

  CCMRAM (xrw) : ORIGIN = 0x10000000, LENGTH = 0
  FLASHB1 (rx) : ORIGIN = 0x00000000, LENGTH = 0
  EXTMEMB0 (rx) : ORIGIN = 0x00000000, LENGTH = 0
  EXTMEMB1 (rx) : ORIGIN = 0x00000000, LENGTH = 0
  EXTMEMB2 (rx) : ORIGIN = 0x00000000, LENGTH = 0
  EXTMEMB3 (rx) : ORIGIN = 0x00000000, LENGTH = 0
}

Great, i'm closing this.

I am having the same error.  startup_stm32f4xx.S is present (note capital "S").  I'm stuck here.

Seems to relate to something else then, when taking into account previous feedback. Marking issue as "Resolved".


Add support for auto release with github action

Part of: https://github.com/alibaba/higress/issues/122
When a git tag is created and pushed to github repo, a release workflow can run automatically to release artifacts.

already support


Development / Releases are supported?

Hi,
Are migration of Development links supported in any way?
Is there the ability to migrate also links to PR's/Commits from Jira?

Releases and the fixesVersion/affectsVersion fields have extended support in our PRO offering.
Commit links are supported in the Community edition, read more here: https://github.com/solidify/jira-azuredevops-migrator/blob/master/docs/faq.md#13-how-to-migrate-development-links-commit-pr-branch
PR and Branch links are not supported as of today. We are looking into how we can add support for this. Both of these link types will probably be supported in the PRO edition only. :)

Hey! Thanks,
Just checking cause it's not really clear to me,
If i haven't moved my repository, what would be the way to export the link to the commit?
For example, i have my repo in GitHub and it's staying there, and all i want is to export the link to this commit from the Jira item to the Azure DevOps item.

That is correct, the recommended approach will only work if you have also migrated your Bitbucket repositories to Azure DevOps.
I am not sure how that would work with the GitHub code integration to be honest. The existing solution is designed to work with Bitbucket and has been tested with Bitbucket, so whether it works out of the box or not would depend on whether the API schemas for the commit/branch links are the same for GitHub and Bitbucket links.
Also I do not think that links to existing GitHub repositories which are not scheduled for migration is supported as of today, even if I think that this is technically doable. Both ADO and Jira would need to be integrated with the same GitHub organization at the time of migration of course.
I will create a feature request on this!

Feature request created. I cannot give an ETA for how soon we can start working on the new feature, unfortunately!


Document react provider mountNode prop

The provider from react-fela takes an optional mountNode prop, that is useful if you need to place fela's style declaration and external ones in a specific order.

The mountNode prop is actually deprecated as the new fela-dom render automatically applies style nodes to the head and reflushes style nodes render with fela-dom renderToMarkup or renderToSheetList.
If you pass a mountNode, fela uses the old render behavior which is not optimized like the new one. :P
Maybe we can add a side node for that somewhere.

Damn! My problem is the one I explained, when I did update to<EMAIL_ADDRESS>I did skip the mountNode, which totally broke my styles, fela taking precedence over external stylesheets, while it used to not have priority.
I can add a comment to specify that this behaviour should be avoided as much as possible, but that doesn't solve my problem on the long term. Perhaps I should open another issue for this discussion?

@augustinLF Are you using server-side rendering? because you can then specify the position of the styles in the head which the fela-dom later adapts. Maybe, for pure client-side use we can introduce some kind of "anchor" element, that will be used to insert the styles after/before.

Thanks, using server-side renderToStylesheet did the work! I guess I should close this PR then?
Regarding your suggestion, should I perhaps create an issue for that? Even if I don't need it anymore?

Could be useful for client-side only usage. Feel free to open that issue :) We can close this one here I guess.


Fix blur on low DPI screens

Fixing issues:

https://github.com/liihuu/KLineChart/issues/452
https://github.com/liihuu/KLineChart/issues/258

After debugging I found bars rendering with X values like: 1234.345639857, it give nice smooth zoom but trigger anti aliasing on low DPI screens, it does not affect a lot of zooming and it still feels quite smooth. Aligning also works as before.

The problem a bit deeper. For some sizes of zoom halfGapBar is for example 4.5, then in candle draw:
x: x - halfGapBar + 0.5,

returns nor 134.5 but 135 and 0.5px fix doesn't work anymore

Can confirm this fix does work. To test on retina display in chrome: press F12 cmd+shift+m, add custom dimension like on screenshot, refresh page.

This works even better with wick as line from this https://github.com/liihuu/KLineChart/pull/455

i've released my forked repo with this fix on our service and got confirmations from users. Blur bug is gone

Fixed text blur as well


Missing operator - for struct Time?

[E]  LLVM JIT FAILED:
missing interop function pointer for $::-. expr = ExprOp2 at %privateScripts/audio/audio_boost.das:733:23
        if get_clock() - last_gc > 5.lf
            call_gc()

https://github.com/GaijinEntertainment/daScript/pull/1018


Simplified the TouchableComponent choosing logic

Summary
I just simplified the TouchableComponent choosing logic into one line. I think it's easier to read now :)
Test Plan
--
What's required for testing (prerequisites)?
--
What are the steps to reproduce (after prerequisites)?
--
Compatibility

OS
Implemented

iOS
✅

Android
✅

Checklist

[x] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

i personally think it is easier to read in the non-ternary form :) thanks though


Facilitate Subresource Integrity

To allow SRI reliably, it'd be nice to have access to versioned scripts (and CORS headers) if we're to implement this in a way that helps reduce the risk of XSS on sites that are relying on the CDN.
Steps:
Add a version to the CDN URL:
Currently it's:
<script src="https://cdn.passprotect.io/passprotect.min.js"></script>

A URL such as https://cdn.passprotect.io/1.0.0/passprotect.min.js would then allow us to rely on a specific version (which means that SRI won't break when changes are made)
Add CORS header:
https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS
Set the header on the resource to:
Access-Control-Allow-Origin: *

Add SRI to the docs:
There may be a way of automating this into the docs when changes are made, the command to generate an integrity hash is:
openssl dgst -sha256 -binary file.js | openssl base64 -A

The docs can then show the integrity hash to use like so:
<script src="https://cdn.passprotect.io/1.0.0/passprotect.min.js" integrity="sha256-x4T57/ClC7dEno+EBaTKYKIhHjXDl39zoxv2u8QxQ88=" crossorigin="anonymous"></script>

More info in great blog posts from Scott Helme https://scotthelme.co.uk/subresource-integrity/ https://scotthelme.co.uk/tag/sri/

@MatthewHerbst good call 👍

Will go ahead and implement this =) I'll try to get it done this week.
This has been part of the plan all along, was sorta rushing to get this out ASAP for timing, but excited to get some new stuff in here =)

Has this been implemented yet? If so, the documentation needs to be updated.

Not yet -- I'm working on it today, actually. I've been sorta tied down finishing a work deadline and am just getting back to making upgrades to PassProtect now and through the next 2 weeks =)

Hi, any update on this? Would be nice to implement this.


Remove -Wno-unused-variable

LGTM for now
Later, -Wno-unused-variable can be removed once the underlying issue is resolved, or added to the specific files causing this issue.
Originally posted by @kparichay in https://github.com/nnstreamer/nntrainer/pull/1858#pullrequestreview-914989329
After addressing all -Wno-unused-variables w/ g++-11 and ubuntu-bare-meson build, remove that flag.

:octocat: cibot: Thank you for posting issue #1862. The person in charge will reply soon.


[APPM-1096] When Skip Creating Proxying WebApp is enabled, Clicking on the subscribed app in My Subscriptions page redirects to Gateway URL

I have created an app with Skip Creating Proxying WebApp enabled. After subscribing to the application, in "My subscriptions" page, when trying to access an application I am redirected to http://localhost:8280/app1/1/ Following error is displayed   <am:fault> <am:code>0</am:code> <am:type>Status report</am:type> <am:message>Runtime Error</am:message> <am:description>Error occured in the mediation of the class mediator</am:description> </am:fault>   log:   INFO - LogMediator STATUS = Message dispatched to the main sequence. Invalid URL., RESOURCE = /app1/1/ INFO - LogMediator STATUS = Executing default 'fault' sequence, ERROR_CODE = 0, ERROR_MESSAGE = Error occured in the mediation of the class mediator  Reference: https://wso2.org/jira/browse/APPM-1096

Not related to IoT server, hence closing the issue.


Set default configEnvName depending on the process

Currently, both npm run build.dev and npm run build.prod use 'dev' as default value for configEnvName in template_locals.ts.
Although it is not so critical, it would be really nice if user could set default values for different build process in the project.config.ts file. Something like:
ENV_CONFIG_NAME_DEFAULTS: any = {
    dev: 'dev',
    prod: 'prod'
}

So user won't have to set config-env every time s/he runs npm run build.prod.
I can send a PR, if you accept the feature.
Best regards,
Samet

Good point! I'll open a PR with a fix.


Fix simple-counter authentication

Type of this PR

Select at least one type

[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Bug fix
[ ] Dependency, Environment, Build related update
[ ] Documentation
[ ] Other tiny fix

Describe your changes
account id 인증 방식을 fix 했습니다.
Issue ticket number and other helpful resource
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] Leave reference about this pull request, if exist (ex. discord message, google docs, etc)
[ ] If it is a core feature, I have added thorough tests.

Checklist after creating a pull request

[ ] Mention reviews in discord channel directly

unittest, testnet 배포 테스트 완료했습니다.


FancyZones - Span 3 columns

Description of the new feature / enhancement
FancyZones allows spanning 2 columns of zones if you drag near the boundary of 2 zones.  (you can also span 2 rows)
Scenario when this would be used?
I have three columns of zones (3 monitors) and would like to span 3 columns.  Doesn't seem to be a way to do this.
Supporting information
Some people have more than 2 monitors, or 3 zone columns defined for 2 monitors.

Very nice.  There doesn't appear to be any way of discovering this functionality.  The settings panel doesn't mention it.  I would have found it there, but maybe putting a text bubble above the window being dragged with a message like "Hold SHIFT to span two zones.  Hold CTRL to span multiple zones."


No documentation how to install Status Monitor in unattended mode

Hello,
How do you instrument existing application which are installed in ServerCore or windows containers? Status Monitor executable is not MSI package and it's not available to install through proper modern package management solutions.

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

ID: 2c8adb27-7386-3e7a-7c8f-324a5206db0d
Version Independent ID: cc276f0a-5ddd-bcb5-8261-7c4bed92454c
Content: Monitor a live ASP.NET web app with Azure Application Insights | Microsoft Docs
Content Source: articles/application-insights/app-insights-monitor-performance-live-website-now.md
Service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

Thanks for the feedback! We are currently investigating and will update you shortly.

Similar situation, I want to provide unattended install of the Status Monitor - ideally by including it in a nested PowerShell Desired State Configuration that installs and configures the rest of IIS.

@artisticcheese , @mikejonestechno
Was able to get some useful information on from my dev team. "Unattended install" has not been tested on ServerCore so it is not officially supported. The PG appreciates your direct feedback so please share your request for server core support here.
For an unattended install (excluding Server Core), there’s instructions @ https://docs.microsoft.com/en-us/iis/install/web-platform-installer/web-platform-installer-v4-commcontrand-line-webpicmdexe-rtw-release
If there are network limitations use the link above for instructions on how to first download the files to perform an offline install.
Ensure you have appropriate IP addresses/DNS exceptions in place:
https://docs.microsoft.com/en-us/azure/application-insights/app-insights-ip-addresses
After the PowerShell install runs a GUI kicks off and downloads some updated NuGet packages.
From there you should be able to follow the PowerShell steps in the article below:
https://docs.microsoft.com/en-us/azure/application-insights/app-insights-monitor-performance-live-website-now#automation-with-powershell

Here is how to do it unattended.
RUN Invoke-WebRequest "https://go.microsoft.com/fwlink/?linkid=517856" -UseBasicParsing -OutFile appinsights.msi; `
    Start-Process -filepath "appinsights.msi" -ArgumentList "/quiet" -PassThru | Wait-Process; `
    Remove-Item .\* -recurse -force -Verbose
RUN Import-Module 'C:\Program Files\Microsoft Application Insights\Status Monitor\PowerShell\Microsoft.Diagnostics.Agent.StatusMonitor.PowerShell.dll'; `
    Start-ApplicationInsightsMonitoring -Name 'default web site' -InstrumentationKey '3c069cb6-fc1a-4bab-974c-0a4245ae7f1b'

@artisticcheese, thanks for sharing this very useful information.
I will proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


CIF-2172 - Improve PDP enrichment example for PWA Studio integration

Description
Update the PDP decoration as follows:

add an image multifield to the content fragment model
move the text under the PDP content
add the images from the content fragment under the text

Add the PLP decoration:

create a category page content fragment with title, text, and a hero banner
create the required components

Generic changes:

add global README with instructions on how to run the extensions locally

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

I think you're also missing some readme updates to explain how the product and category page customizations are applied.


Entities::reserve_entities yields invalid entities that can be used for buffer overflows

Entities::reserve_entities is considered 100% safe as it can be accessed from World::entities, but returns an iterator that yields Entity values that are not valid until World::flush_entities, a safe pub(crate) function. As it currently stands, there is no safe way to use Entities::reserve_entities as it must be paired with an unsafe call to Entities::flush, or Entites::flush_as_invalid.
Likewise, there are zero internal uses of the function due to our current use of flush_and_reserve_invalid_assuming_no_entities in the render world (see #12144).
It may be best to just remove all of these functions to simplify the code, or come up with safe abstractions around this for external use, such as #11602.

I'm in favor of marking unsafe + deprecating and asking users to help us figure out what they're doing.

reserve_entities() is intended to be "just" a more efficient version of calling entities.reserve_entity() and it has the same constraints as that:

You can call it without a mutable World reference, enabling its use in parallel settings
The entities do not actually exist until flush() is called. The entity will be considered to be nonexistent / not show up in queries until flush is called. Getting the entity from world will fail.

Nothing is currently "unsafe" about this behavior and this should this be deprecated. However there is currently a bug. Attempting to insert a component into a "reserved entity" will fail (because the entity does not exist), making it hard to use reserved entities predictably.
Back when inserting components looked like:
world.insert(entity, component)

That insert function's first line was self.flush(). This ensured that reserved entities were converted into actual entities prior to doing the insert.
However once we moved to world.entity_mut(entity).insert(component), we introduced this bug because world.entity_mut(entity) does not call self.flush() first, meaning that when we try to look up the entity, it is still considered invalid and fails as such. WorldEntityMut does have flush() for insert() calls, but we never get there because we can't construct it in the first place.

The obvious solution is to add a flush() to world.entity_mut(). We could even consider removing the flush() from functions on WorldEntityMut (like insert()) because new entities cannot be created when we have a WorldEntityMut. That would actually cut down on spurious flush calls.


swift 1.2

Hello,
i have tried today to run my project on the new xcode 6.3 beta with swift 1.2, but i have seen that your project isn´t compatible with this swift version. My question is when you want to adapt your project to swift 1.2.

as soon as i update cemkit to swift 1.2.
most of the problem fixed when cemkit adopts to 1.2.
i will update (probably in a new 1.2 branch) in a week :+1:

thanks for your fast reply

Hello, I have install last XCode 6.3 & iOS 8.3 on my iMac under Yosemite, I have many issues with your library. Maybe Swift 1.2 SDK has changed and your fix is not adapted...?
Well, I use your amazin library in my project :)
Thank's your response

hi, you can use swift 1.2 branch with xcode 6.3 release version.
https://github.com/cemolcay/MaterialCardView/tree/swift1.2

Very thank's for your reactive response !

Hi,
Still it has build errors, even though i take the latest CEMKit, Can you please help?

Hi,
Did you tried swift 1.2 branch ?
https://github.com/cemolcay/MaterialCardView/tree/swift1.2

On 20 May 2015, at 16:40, Balasubramanian Kasamuthu<EMAIL_ADDRESS>wrote:
Hi,
Still it has build errors, even though i take the latest CEMKit, Can you please help?
—
Reply to this email directly or view it on GitHub https://github.com/cemolcay/MaterialCardView/issues/1#issuecomment-103892521.

Thanks for immediate response!
yes, I tried the swift 1.2, but still have errors
I am using Xcode 6.3.2 and Swift 1.2

where do you get errors ?

Sorry my mistake, It builds now :)
Thanks a lot!

Hi, I am using xcode 6.3.2 and your code triggers many errors.


Full error rework, cleanup, documenting

User can implement mapping logic only for those he will use.

Also includes the ability for Field[P] to compare with both P and Field[P] using the FieldCompare type-class without additional allocations.


Enable wasm optimizer from dfx 0.14.0

I think it would be good to merge this so that we can measure performance improvements beyond wasm-opt and not reimplement optimizations already included in the optimizer.
Note that these benchmarks directly useic-wasm instead of using the optimize: "cycles" feature in dfx in order to preserve the wasm name sections for the flame graphs. For any users reading this, for the general case we recommend using the optimizer through dfx instead as the binary size reductions will be better when dropping the name sections.
For future reference: https://github.com/dfinity/sdk/pull/3090
See also #50 for previous discussions.

Just updated the README.md


Error when trying to upload an aerospike (only?) stdout binary file to azure storage account using CLI

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
asbackup -n trigger -v -z zstd --output-file - | az storage blob upload --auth-mode login --data @- -c containername--account-name accountname --name test_upload_file_name
Describe the bug
Using Aerospike (in this particular case) to export data and upload it to Azure Storage Account using CLI command.
The data is exported and compressed using zstd resulting on a binary file, "posted" on stdout to be picked up by the az storage blob upload command and sent to storage.
Notes:
Works OK without compression.
Works OK when split the command in 2. First Produce the file and second Upload it.
Works NOK when the upload is through pipe.
The command is the **Upload to a through pipe. **
https://docs.microsoft.com/en-us/cli/azure/storage/blob?view=azure-cli-latest#az-storage-blob-upload
The error message is:

To Reproduce
Having Aerospike or any binary file to the stdout. Read it with the az storage blob upload through pipe.
asbackup -n trigger -v -z zstd --output-file - | az storage blob upload --auth-mode login --data @- -c containername--account-name accountname --name test_upload_file_name
Expected behavior
The expected behavior is an upload without issues.
Environment summary
CLI version is 2.36
Additional context
Similar command works fine for AWS.
asbackup -n data -v -z zstd --output-file - | aws s3 cp - s3://BUCKET_NAME/FOLDER_NAME/aerospike_backup.asd
Using the same command with 2 pipes gives the same error.
asbackup -n trigger -v --output-file - | -z zstd | az storage blob upload --auth-mode login --data @- -c containername --account-name accountname-name test_upload_file_name

storage


DS-Sprint-03-PR

Week 3

Part 1
Code review

What went well:

Good job implementing Markup!:

"# Part 1\n",
    "\n",
    "#### Write a lede:\n",
    "\n",
    "**To Degree, or Not to Degree.**

Does having a degree before Lambda affect your chances of getting hired? If so, to what extent? Based on anecdotal evidence, I'm thinking no. But, in the spirit of intellectual rigor, I'm going to analyze anonymous student data to find out. \n"

_What could be better:_

-Work on structuring your idea in the sprint challenge and post your "before & after" scores from the Hemingway App:

Stretch goals

Write more about your idea. Tell us what the story's about. Show us why it's interesting. Continue to follow the inverted pyramid structure.
Improve your readability. Post your "before & after" scores from the Hemingway App.```

Part 2
Code review

What went well:

I enjoyed the links you used! Great sources.:

Sources
North Star Podcast w/ Austen Allred, Lambda CEO discussing the programs and some of their outcomes.
Outcomes, Lambda School's official public placement rate info on their website.

_What could be better:_

- More links could of been added:

#### Stretch goals
- Find more sources.
- Use Markdown to add images from your sources.
- Critically evaluate your sources in writing.

Part 3
Code review

What went well:

Good job explaining how you will follow up :

Follow up with Lambda School to get raw data from them.
Review and request any additional data.

Vast amount of links provided and you clearly described your project. Great job!

What could be better:

- I reccomend trying the stretch goal and posting this on Github pages for a more professional look!
https://pages.github.com/

Additional notes
Code review

If you want to take this further you could:

[ ] Implement Part 3 into a Github Page(https://pages.github.com/)

[ ] Provide more links in part 2


Working Directory

How can I set the working directory?
If not supported, consider supporting something like this:
- name: Build
  uses: jerryjvl/jekyll-build-action@v1
  with:
    working-directory: blog

Hi @chances
You can use my fork ;)
https://github.com/Ealenn/jekyll-build-action


Fix #3202: Format doubles exactly like on the JVM.

We address the differences by implementing the notations ourselves, based on a decimal representation of numbers. We convert the Double values to our Decimal representation based on the native toString() method, which guarantees that its result contains as few digits in the significand as necessary. Once we have a Decimal value, we can round/rescale it as required, using the round-half-up strategy, and finally format it.
We use the opportunity to reorder the notation functions to be in the same order as the conversions they implement.

Started to look at this, just a higher level comment for now. Detailed review is very pending :P

Ah ah! No problem. Take your time.


Add Wagmi

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Please follow the SPL token-list README and ensure your token info adheres to the Uniswap Token Specification

Change ticker, we already submitted this one 6days ago

@cept0r Please follow the SPL token-list README and ensure your token info adheres to the Uniswap Token Specification

!!!


Run an empty cell in the interactive window and UI hangs

If the first cell you run is just this:
# %%

Interactive window will open but never respond.

Validated


Add 'imshow' package

Testing:

https://ci.appveyor.com/project/ingenue/hunter/build/1.0.1464
https://travis-ci.org/ingenue/hunter/builds/215382065

Uploading:

https://travis-ci.org/ingenue/hunter/builds/215394810

Released:

https://github.com/ruslo/hunter/releases/tag/v0.18.27

Wiki:

https://github.com/ruslo/hunter/wiki/pkg.imshow


wrong column getting "clusterip" for gluster storage class

https://github.com/redhat-nfvpe/kube-centos-ansible/blob/2fce185a5a88d90787df1ebfe8dada7112b571ee/roles/glusterfs-kube-config/tasks/main.yml#L89

[centos@kube-master ~]$ kubectl get svc --namespace=gluster | grep 8080 | awk '{print $2}'
ClusterIP
[centos@kube-master ~]$ kubectl get svc --namespace=gluster | grep 8080 | awk '{print $3}'
<IP_ADDRESS>

note to self: using the api is "the right way(TM)"

Also -- take a look at the value for kube_version in group_vars all.
Leif planning on version pinning it -- idea is potentially to change so you can specify "latest" in kube_version variables.

Decided for expediency to use kubectl proxy to query the API for the time being. Later "better" way might be to set up service accounts and give them the correct permissions, but, for now, lab style... This is rather convenient.


docs: Document all environment variable usage in PyFluent

The documentation is intended for developers.

@mkundu1 Looks good. Thank you.
@seanpearsonuk I think we can add similar section in units repo as well with respect to recent angle changes.


Refactor Router alert and add recording rules.

Add some recording rules for Router metrics and use these in the alert.
Simplify the name of the alert.
Add a minimum RPS threshold so it doesn't fire on statistically insignificant input.
Fix a failing alert test.
Improve the messages on test failure and make the tests more self-documenting by adding some assertions on intermediate values.
Clean up formatting.

Still to do in a later PR: actually run the tests pre-merge instead of them being manual 😅

Hm looks like we're missing a kubeconform schema for PrometheusRule. Lemme fix that.

Schema wasn't missing, but was borked somehow. Replaced it with the canonical one from kube-prometheus (crdschemas/prometheusrule.json, generated by running make crdschemas).


execute=True/False default value for PlanToNamedConfiguration()

The prpy readme says that PlanToNamedConfiguration() should execute the trajectory by default, however apparently this feature was changed recently.

The default value of this execute flag is different depending on the use case as shown below:
This is correct as per the recent change:
# The arms do not move
robot.PlanToNamedConfiguration('vertical')

# The arms move
robot.PlanToNamedConfiguration('vertical', execute=True)

The bug:
# The arm moves immediately for this use case
robot.right_arm.PlanToNamedConfiguration('vertical')

I fixed this in #183. Can you test that branch and confirm that it works?


Create release

Can a release be made? I noticed #22 is not released yet -.-
https://github.com/php-http/logger-plugin/compare/1.3.0...master
Would be nice if "request" => Nyholm\Psr7\Request^ { …}, disappeared from our prod logs.
cc @dbu

uh. it seems i tagged on my machine without having pulled after the merge of 22, and thats also why the changelog for 1.3.0 had no date yet... fixed this now, https://github.com/php-http/logger-plugin/releases/tag/1.3.1 includes #22.


using baobab with JSPM

When using with JSPM, you have to point the config.js file to the built file "baobab"<EMAIL_ADDRESS>because the source files use require(*.js) which causes SystemJS to interpret *.js.js and fails to find any files.

Hello @AutoSponge. Why would SystemJS work like this whereas every other module system works differently? A lot of people (I would say a vast majority) use .js in the required path. I personally use this to discriminate what is loaded from the exterior of the module and what is internal and relative (even if, yes, the ./ is pretty self explanatory).
Let me document myself about JSPM and such to see what can be done.

Does that work differently with "baobab"<EMAIL_ADDRESS>

I'm looking here: http://wiki.commonjs.org/wiki/Modules/1.1#Module_Identifiers
~3 = Module identifiers may not have file-name extensions like ".js".
What you're using is ignore by node but more specific implementations, like SystemJS will bomb.
Right now I'm using the compiled version because it doesn't expose deps so I don't have any issues. All of your files use this syntax, so until it's fixed, you can't directly use this lib with JSPM or SystemJS.

I'll probably need to fix my requires then not to include the .js. Do you happen to know the reasons why they choose to drop the .js on ES6 modules? This sounds a little bit crazy to me.

I am currently writing v2. I will drop the .js in it so nobody is left behind.

Done in v2.

Just to add that in my experience with jspm version 0.16.0-beta.3 (which uses SystemJS version 0.17) things work out of the box
jspm install npm:baobab


fix: import cast for required const properties, since it's used in the template

The template for const properties uses cast unconditionally, but the import for cast in ConstProperty.get_imports was conditioned on whether the property was required or not, leading to broken generated code for required const properties.
I'd be happy to add a test for this if desired! A cursory glance suggests that maybe the end_to_end tests would be the right place?
Resolves #1150

Yeah, if you wouldn't mind adding a test so we can prevent this from regressing in the future, that'd be great!

Sure thing, added a test case to both of these files and regenerated baselines:

end_to_end_tests/baseline_openapi_3.0.json
end_to_end_tests/baseline_openapi_3.1.yaml

(If that's not the right place for these tests, let me know what you think would be a better home!)


How to show line numbers

How can I show line numbers in the syntax highlighting?

I'd love to use line numbers as well, any word on this? Let me know if I can help in any way.

I'm working on an eleventy based starter and I was able to get line numbers working with these two steps:

Enable alwaysWrapLineHighlights in the eleventy syntax highlighting plugin options
Add CSS

Step 1
// .eleventy.js
// ...
eleventyConfig.addPlugin(pluginSyntaxHighlight, {
  alwaysWrapLineHighlights: true,
});

Step 2
/* styles.css */
pre {
  counter-reset: lineNumber;
}
code .highlight-line:before {
  -webkit-user-select: none;
  border-right: 1px solid #404040;
  color: #858585;
  content: counter(lineNumber);
  counter-increment: lineNumber;
  display: inline-block;
  font-variant-numeric: tabular-nums;
  margin-right: 1.2em;
  padding-right: 1.2em;
  text-align: right;
  width: 2.4em;
}

Screenshot

Your mileage may vary, remove and styles you don't need. Hope this helps someone! ✌

@joshbuchea, FYI, I'm seeing issues with this approach. I've documented them here: https://github.com/falldowngoboone/falldowngoboone-com/issues/116.

After a bit of testing this, I can assume it's not a stable solution. For example, in PHP code you will have a lot of troubles:

The <?php at the beginning is not wrapped inside highlight-line and you have to "whitelist" it along with code .highlight-line:before, that's fine.
The PHPDocs are... just weird. The whole comment is wrapped inside .token.comment, the opening tag (/**) and closing tag (*/) are not wrapped inside .highlight-line but the other lines are, generating an undesired result:

Adding .token.comment to code .highlight-line:before does not solve: the style is messed up, but above all the closing tag is not even counted:

I saw there's a PR from 2020 to implement line numbers without the messy JS code provided by PrismJS, I hope it can be considered: https://github.com/11ty/eleventy-plugin-syntaxhighlight/pull/24

the solution of @joshbuchea works for the most part but it's not reliable and stable. There are issues with the plugin in rendering span as mentioned by @falldowngoboone.

I'm working on an eleventy based starter and I was able to get line numbers working with these two steps:

Enable alwaysWrapLineHighlights in the eleventy syntax highlighting plugin options
Add CSS

Step 1
// .eleventy.js
// ...
eleventyConfig.addPlugin(pluginSyntaxHighlight, {
  alwaysWrapLineHighlights: true,
});

Step 2
/* styles.css */
pre {
  counter-reset: lineNumber;
}
code .highlight-line:before {
  -webkit-user-select: none;
  border-right: 1px solid #404040;
  color: #858585;
  content: counter(lineNumber);
  counter-increment: lineNumber;
  display: inline-block;
  font-variant-numeric: tabular-nums;
  margin-right: 1.2em;
  padding-right: 1.2em;
  text-align: right;
  width: 2.4em;
}

Screenshot

Your mileage may vary, remove any styles you don't need. Hope this helps someone! ✌

Maybe this will help someone, as I've spend quite a bit of time trying to figure out how to enable showing line numbers only for specific code blocks, not site-wide (so without "step 1" that sets the alwaysWrapLineHighlights to true).
I went through different solutions including enabling markdown-it-attrs (which didn't work, because python {.myclass} wrapped in a code fence - three backticks - doesn't work and myclass is removed by this plugin) and markdown-it-container (where I wanted to wrap the whole
 in a div with a custom class, but that also didn't work, as inside
 I didn't have a separate class for each line of code because I didn't enable the alwaysWrapLineHighlights option).
Turns out this issue can be fixed with a small hack - suffix the language in the code block with "/" (but don't pass any number, unless you want to highlight some code). This enables highlighting lines and highlighting lines in turn enables the line wrapping. And don't forget to add the CSS from step 2 to make this work.
This markdown:

Renders like this:


mouseenter/mouseleave cause unrelated methods to trigger repeatedly

Version
2.6.11
<template>

      <input :id="method2()" type="text" />

      Hover me, and the 'message' prop will get updated.

      But method1() and method2() are also called.

{{ message }}

</template>

Reproduction link
https://codepen.io/rmirabelle/pen/bGpodLx
Steps to reproduce
Open console. Observe output while hovering the target element.
What is expected?
method1 and method2 are called once when loading.  When hovering, the message property is updated.  Only the h1 element, which outputs the message is updated.
What is actually happening?
The message property is updated.  But instead of re-rendering just the h1 that outputs the message, the entire component appears to be re-rendered, thus re-triggering all methods bound to other elements.  This occurs seemingly regardless of how the other element attributes are bound. (e.g. .stop, .prevent, etc.)

Given how simple this example is, I feel like I must be missing something fundamental about Vue reactivity.  Neither of the elements bound to method1 or method2 observes or uses message.  Nor is mouseenter attached to those elements.  So why are the methods being called repeatedly?

Every Vue component, including the root, has a render function (which is compiled from the template code) and that render function will be called every time the component's state (it's prop, data, and computed properties) gets updated. In your case, updating message is updating the component's state and thus component's render function is getting called. We can take a look at the render function compiled from your code and it's obvious why method1 and method2 is called on every update:
function render() {
    with(this) {
        return _c('div', {
            attrs: {
                "id": "app"
            }
        }, [_c('div', [_c('span', {
            key: "foo",
            attrs: {
                "id": method1()
            }
        }), _c('input', {
            key: "bar",
            attrs: {
                "id": method2(),
                "type": "text"
            }
        })]), _c('div', {
            staticStyle: {
                "margin-top": "20px"
            },
            on: {
                "mouseenter": function($event) {
                    message = 'huh???'
                },
                "mouseleave": function($event) {
                    message = ''
                }
            }
        }, [_v(
            "Hover me, and the 'message' prop will get updated."
        ), _c('br'), _v(
            "But EVERY method will also be called.")]), _c('h1', [
            _v(_s(message))
        ])])
    }
}

We can easily inspect template compilation result, i.e. the render function, using a nice website: vue-compiler-online. Thanks to @Magiccwl
Indeed, vue has some diffing algorithm that provides a performant update, trying to prevent unnecessary updates. But this optimization is for DOM operations, in other words, unnecessary DOM operations would be avoided wherever it's possible to do so.
I cannot see the original intention of using methods to produce those two ids, and I would say this could be a code smell. That said, assuming there is a plausible reason to do that, computed properties might help you with the problem since they're cached.

BTW, mouseenter/mouseleave are irrelevant here since they're merely something cause component's state to update. It could be button clicks or even periodic timers.
Comopnent state gets update -> render function gets called -> method1 and method2 get called (they're called within render function. Just normal function calls, no magic at all)

This is expected as @Nandiin amazingly explained! Thank you @Nandiin !

This was helpful thanks.  My example was contrived to minimize clutter.  I incorrectly assumed the method calls would be 'cached' as computed properties are.  This example cleanly demonstrates how untrue that is.


Clarify installation instructions for Laravel 5.4

I haven't yet gotten around to updating my project to Laravel 5.5, and I haven't been following Laravel news, so I wasn't aware of these developments:
https://laravel-news.com/package-auto-discovery
https://medium.com/@taylorotwell/package-auto-discovery-in-laravel-5-5-ea9e3ab20518
This PR essentially reverses 7a8b50b, while differentiating between Laravel 5.5 and 5.4. I think it's worthwhile to include, since AFACT, it's the only tidbit preventing its compatibility with version 2.0^ of this package. Plus, I think it's still relevant to whomever might be using version 1.0^. IIRC, during my installation today, I found the solution by stumbling over the legacy pingpong labs repo. Might as well keep it here until Laravel 5.4 is at least one more version out-of-date.
I also added a teaser re: config options to the readme, both due to today's inquiry (#366) and because I first heard about this project via an (outdated) discussion, where one party was critiquing the package for generating too many folders by default. I think people will want to know that functionality is easily tweakable.

Thanks, but this isn't needed. The instructions for <5.5 are on the 1.0 branch.

Good call, I did not notice that existed. Would you be down to make the "^1.0" in the table into a link to tree/1.0? Could help newbies like me.

Hm I guess that could be a link yes. Though no need for the tree/ part.


overlay images (e.g. for sponsors)

what about overlay images (e.g. for sponsors)?

see https://github.com/drweissbrot/cs-hud/blob/1f9c2e22d381381dee627e10db8267b32abf7a6d/docs/overlay-images.md

hm, ok and there is "Note that animations (for transitions between multiple sponsors, for instance) are possible using all three of these formats." how i can make transitions in png images?

I'd go for WebP instead of PNG, which should be easy enough to create with GIMP for example (see https://www.reddit.com/r/GIMP/comments/h0gg1j/comment/ftlyo10).
If you have to use PNGs, ask your favorite search engine for create animated png :D


Verbose flag not working in heroku pg:backups

I'm trying to figure out what's going on with another command, heroku pg:backups capture -a APPNAME and I'd love to be able to look at the logs. Am I using the flag incorrectly? The documentation says:

You can use the flag --verbose to see logs as your backup progresses.

+1 on this issue. I also tried putting the flag anywhere in between the rest of the command, but nothing worked. I also did not find any examples on the internet about how to achieve this. The worst part is that I am getting an "!    internal error" and there's no way to see what happened...


Update prover-market-page.mdx

Added Ohmar Prover Node

Please resolve merge conflicts.

Conflicts Resolved. Many Thanks!


Correct dependancies

Now that the 0.7.7 leaflet is release, I have the bug that the marker doesn't show because of a file doesn't found in<EMAIL_ADDRESS>I see that the number 0.7.3 is writen in hardcode here: https://github.com/benib/aurelia-leaflet/blob/0.1.1/src/index.js#L4
For this reason, only the leaflet 0.7.3 is compatible with your code, no?
Anyway, thanks for this plugin :)

Thanks for the PR.
You can actually set the path during plugin loading like this:
aurelia.use
  .standardConfiguration()
  .developmentLogging()
  .plugin('benib/aurelia-leaflet', {
    LeafletDefaultImagePath<EMAIL_ADDRESS>  });

This is not documented, but it should totally be. I'll add it in the next days.
I will close the PR as using this configuration, it is possible to use a different version of Leaflet, so no need to pin the minor version of Leaflet. I will change the default path though with a minor update in the next days.
Thanks a lot for the heads up. Would love to hear back if this works for you like that.


option to use standard ssh client to connect to the peer's netbird ssh server

Is your feature request related to a problem? Please describe.
As expected, I can ssh to another peer via sudo netbird ssh <peer>.netbird.cloud
Describe the solution you'd like
But is it possible to connect to the same peer using standard ssh client? e.g. sudo ssh -p 44338 <peer>.netbird.cloud
Can I possibly extract the auto-generated key by netbird and feed it to the ssh command, -i option? If yes, where is it located.
I tried using the SSHKey in /etc/netbird/config.json by pasting it to a mykey file, but getting an error:
sudo ssh -p 44338 -i ./mykey -vv <peer>.netbird.cloud

the key seems too short anyway

debug1: Will attempt key: ./mykey  explicit
debug2: pubkey_prepare: done
debug1: SSH2_MSG_EXT_INFO received
debug1: kex_input_ext_info: server-sig-algs=<ssh-ed25519,sk-ssh-ed25519@openssh.com,sk-ecdsa-sha2-nistp256@openssh.com,ecdsa-sha2-nistp256,ecdsa-sha2-nistp384,ecdsa-sha2-nistp521,rsa-sha2-256,rsa-sha2-512,ssh-rsa,ssh-dss>
debug1: kex_input_ext_info<EMAIL_ADDRESS>(unrecognised)
debug2: service_accept: ssh-userauth
debug1: SSH2_MSG_SERVICE_ACCEPT received
debug1: Authentications that can continue: publickey
debug1: Next authentication method: publickey
debug1: Trying private key: ./mykey
Load key "./mykey": invalid format
debug2: we did not send a packet, disable method
debug1: No more authentication methods to try.
<EMAIL_ADDRESS>Permission denied (publickey).

Additional context
The intention is to use pyinfra's ssh connector without having to install a separate sshd on the peer:
pyinfra @ssh/<peer>.netbird.cloud configure_peer.py
pyinfra can accept ssh keys, etc:
  --ssh-user, --user TEXT         SSH user to connect as.
  --ssh-port, --port INTEGER      SSH port to connect to.
  --ssh-key, --key PATH           SSH Private key filename.
  --ssh-key-password, --key-password TEXT
                                  SSH Private key password.
  --ssh-password, --password TEXT

I would greatly appreciate the addition of this feature. The inability to use the default SSH client significantly limits the ways we can interact with the service. As a result, tools like PuTTY, WinSCP, and others are currently incompatible with the existing implementation.

The SSHKey field is an Ed25519 private key in PKCS#8 PEM format. It needs to be converted into a format openssh understands, here's one way to do it (reference):
sudo cat /etc/netbird/config.json | jq -r .SSHKey | npx --package=sshpk -- sshpk-conv -t ssh -p > ~/.ssh/netbird.pem

Then I added the following to ~/.ssh/config:
Host *.netbird.selfhosted
        User root
        Port 44338
        IdentityFile ~/.ssh/netbird.pem

Now ssh my-server.netbird.selfhosted works!

@deadbeef84 that is very interesting. I never thought to try that! I will give it a try when I get back to work after the Holiday. If it is as straight forward as modifing the format of the keys and copying them over. I will look at how the --ssh option works during its init and see if it's something that can be added.

@deadbeef84 that is very interesting. I never thought to try that! I will give it a try when I get back to work after the Holiday. If it is as straight forward as modifying the format of the keys and copying them over. I will look at how the --ssh option works during its init and see if it's something that can be added.

+1 . Looking forward to this. Thanks!

Do I understand correctly, than enabling "SSH Access" for host in Netbird management panel does not provide option to connect to the host just using
"ssh<EMAIL_ADDRESS>In management panel I set "Allow all (TCP, UDP, ICMP)" for network rules, but I can't connect via SSH and VNC (port 5900) between peers.  (ICMP also doesn't work)
Is it expected?
Sorry if there is not the proper topic, but in my specific case I really need both (SSH and VNC).
Thanks in advance.

Do I understand correctly, than enabling "SSH Access" for host in Netbird management panel does not provide option to connect to the host just using "ssh<EMAIL_ADDRESS>
That is correct, it will start an SSH server on port 44338 that requires a key to authenticate. You can use netbird ssh myhost to connect to it, but using any other ssh client will require setting up these connection parameters manually.


Export agora symbols for better debugging and stack traces

this exports additional ~30k symbols on my machine
[omer@archlinux agora]$ nm ./build/agora-no-export | wc -l
46684
[omer@archlinux agora]$ nm ./build/agora | wc -l
75213

MacOS says no tho

Seems to be linking fine now.


client runs json export intermittently provides bad json report

User Story
original issue: https://github.com/chef/customer-bugs/issues/70
i spoke with support when they received this ticket. the bad report is certainly bad json, but i can't seem to reproduce with 40~ nodes, which makes me wonder if this has anything to do with the large number of nodes being exported?
please see the original issue for more details

https://github.com/chef/automate/pull/2927


Global state gets clobbered if you connect to different DB servers in the same application

Current behavior
If you connect to a Postgres database after connecting to a SQL Server database, the singleton value of SqlGenConfig is in an invalid state for Postgres. In particular, 'updateablePrimaryKey' is now false and inserts to Postgres will fail.
Expected behavior
Postgres inserts should still work even after connecting and disconnecting from SQL Server.
Steps to Reproduce the Problem

Connect to an MS SQL Server database, such that DbDetector.detectDatabaseAndConfigureDbOom is executed.
Connect to a Postgres database and try to insert an object with a DbId primary key column.
Because updatablePrimaryKey is now false, the insert statement won't include the id column and it will fail.

My bad - the docs say that what I'm trying to do isn't supported. Please close this issue with my apologies.

@dhobbs no worries - this is one of the things that we want to change. Actually, it would not be so hard, just to figure how users would choose the database scope, i.e. which database to work with. How one piece of code can work with one database, and the other to work with other.
I had several options in mind:

package differentiation: you simply specify the root package for each database. Every database operation would check the class name and would figure wich database to use.

ThreadLocal storage, probably the implementation of above, where you can turn on manually in the code one database access.

So I am looking for the way that would be simple and clean for developers to specify one or the other target database. Implementation would not be a problem :)
Im going to reopen this as I do plan to work on it. Not in current sprint, but soon.

I have added DbOom - it is now the central class for running all DbOom things.
You can use it if you have multiple DBs.


remove static restriction for str ref

Opening another PR based on the discussion in: https://github.com/igiagkiozis/plotly/pull/107

@mfreeborn, @aleb , the patch is incorrect because it is missing lifetimes. I have a fix for the above. The function signature should be
pub fn to_inline_html<'a, T: Into<Option<&'a str>>>(&self, plot_div_id: T) -> String {

I build locally and the examples work fine with the above.
Should I fork and make a different PR or is it better we fix it in this already existing PR ?

Hang on a sec, I've just read this properly and I believe what we are looking for is:
pub fn to_inline_html<T: Option<AsRef<str>>(&self, plot_div_id: T) -> String {
    let plot_div_id = plot_div_id.into();
    let plot_div_id = match plot_div_id {
        Some(id) => id.as_ref(),
        None => Alphanumeric.sample_string(&mut thread_rng(), 20),
    };
    self.render_inline(plot_div_id)
}

EDIT: updated what I think should work as an implementation.

@mfreeborn , I tried your suggestion , but it does not work as is. I will try to see if I can get AsRef to work for this case.

This implementation would work
    pub fn to_inline_html<T: AsRef<str>>(&self, plot_div_id: Option<T>) -> String {
        let default = Alphanumeric.sample_string(&mut thread_rng(), 20);
        let plot_div_id = match &plot_div_id {
            Some(id) => id.as_ref(),
            None => default.as_str(),
        }

however the above will not allow one to write
plot.to_inline_html(None)

but the type must be speficied, which I think is ugly
plot.to_inline_html::<String>(None)

Do you @mfreeborn have any suggestion?

Closed by #115 - thanks for your input :)

@mfreeborn , thanks!


An error occurred  while attempting to reload a python project.

Describe the bug
this issue is not easy to reproduce, it doesn't happen very often.
Steps to Reproduce
1.Create a python application project named PythonApplication1
2.Right-click in PythonApplication1 and select unload this project.
Expected behavior
no any errors occured
Actual behavior
An error occurred in 'Solution 'PythonApplication1' (0 of 1 project)' while attempting to open 'PythonApplication1'

Build of Visual Studio: 16.8.0 Preview 5.0 [30615.2102.d16.8]
Build of Python Package: 16.8.20241.2

0x80004002 is E_NOINTERFACE


Product level metrics graphs not showing expected data

Bug description
Looking at a product-level metrics, the graphs do not match at all the expected values to be found (at least not for me).
However, when on the "Metrics" tab at the product level, if we consider for each Open, Closed, Accepted and New findings, clicking on the link of each metrics overview (showing 0 typically most of the time) leads to a grid view where the expected results seem to all show.
Example:
No idea about the 1 here, maybe some manual mods somewhere on a finding triggering?

Yet, going to the link for Open findings:

I would expect the graph to reflect the grid.
Steps to reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Deployment method (select with an X)

[ ] Kubernetes
[x] Docker
[ ] setup.bash / legacy-setup.bash

Environment information

dev

After some investigation, the metrics are defined as follows:
Open Finding: false_p=False, verified=True, duplicate=False, out_of_scope=False, active=True
Closed Finding: false_p=False, verified=True, duplicate=False, out_of_scope=False
Accepted Finding: Any finding in the product that is also in a risk acceptance.
There could be many solutions here. Some kind of label needs to be present somewhere indicating what each graph entails. Something like help text or a similar method.

There was a change a couple months where Open findings didn't need to be verified, only active. So the metrics graphs were not updated.
Wild thought: Would there be a way or would it make sense to elevate these definitions at the global level, and they would then trickle down to all parts of the systems for better consistency throughout?

That would be the best solution, but yes, it would be quite the undertaking. I can only assume there are other places that were not updated to abide by the new definition of active.


Recruit Data Platform Users - HMPPS

Initial engagement email sent to Howard Smith. Awaiting response.

Meeting scheduled for the 19th of June.

We talked to Howard. He suggested we contact Grant Chequer about the ARNS dataset as a data product candidate.

Edward and Greg are engaging with the HMPPS to see which use cases are ready for the team to explore.
Edward has given an indicative time frame of 2 months before we can be confident of the priorities to focus on.
This was fed back to the team at Retro - 22/06/23.


update(query): add CWE infos to terraform queries

Reason for Proposed Changes

terraform queries don't provide cwe infos;

Proposed Changes

add CWE infos to all terraform queries;
fix query typos;
update e2e with cwe field;

I submit this contribution under the Apache-2.0 license.

Any update on when this PR and the many others regarding the CWE infos will be merged and released?

Hi @haerter-tss,
Thank you for your patience and for bringing this up.
We are planning to merge all PRs related to the CWE additions as soon as possible. Currently, some vulnerabilities on master are preventing us from merging these PRs.
Once we resolve those issues, we’ll be able to move forward with merging all related PRs.
Thanks again for your concern and we will keep you updated!

Hey @haerter-tss,
CWE ID additions will be available in all queries very soon.
Thanks for the patience!
Best Regards.


"Updated" Slider Should Work for Less than 1,000 cases

Problem: Currently the "Updated" slider has a resolution of 1,000. This means that weird things happen when an incident has fewer than 1,000 cases. The current workaround is to omit the slider when the case load is less than 1,000.
Instead, make the resolution dynamic: Set it to the number of currently existing cases if the number is less than 1,000, and 1,000 if the total number of cases is greater than or equal to 1,000.

Looks good.


Allow the usage of a postfix for the add_git_tag action

Checklist

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

Motivation and Context
For a project of mine we tag builds with v10.5-RC1 where RC number gets appended with the Jenkins build number. Therefore I would like to be able to append a postfix to the build number.
Description
I added an :postfix option to the add_git_tag action.

I signed it!

No one else in need of such a feature?

Hi @dunyakirkali ! Thank you for submitting this PR, the code looks ok to me and the feature does not break any existing code.

Yes, looks like a useful feature. Feel free to squash and merge @minuscorp.


⚠ File access has degraded performance

In 14e3e7c, File access (https://fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 6690 ms

Resolved: File access performance has improved in ec96f50 after 8 minutes.


Add support for multi-transaction update

https://github.com/ArweaveTeam/arweave/issues/105 introduced support for multiple transactions given posted transactions reference the last_tx pulled from /tx_anchor. This package needs to pull the tx from the anchor and use that when posting transactions to full support multiple transactions per block.

Hey @JustMaier, good spot!
We're just trialing this feature a bit more before baking it into the JS library. We'll be updating this soon though! :rocket:

This is implemented in recent versions of Js SDK / arweave-deploy and most community SDKs and so closing this.


isFailure includes isCancelled

Changes

isFailure includes isCancelled, but some functions are calling these isFailure and isCancelled separately.
https://github.com/tektoncd/pipeline/blob/43b733028e441be2f88362c1369fed2d3dddadd7/pkg/reconciler/pipelinerun/resources/pipelinerunresolution.go#L112-L115
This change removes the separate calls to isCancelled where isFailure already exists.
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[n/a] Docs included if any changes are user facing
[n/a] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Release notes block below has been filled in (if there are no user facing changes, use release note "NONE")

Release Notes

NONE

/kind misc


How to disable newlines (Act as input) in react-quill 2.0.0

Hi there,
I've googled and googled but really can't see a way to disable newlines/enters for ReactQuill. I see plenty of posts for regular Quill but I'm not too sure how different the APIs/functionalities are.
Unfortunately, when I use my component like below.
<ReactQuill
  onKeyUp={(e) => handleKeyDown(e)}
  formats={["header"]}
  modules={{
    toolbar: false,
  }}
  ref={props.quill}
/>

const handleKeyDown = (e) => {
  if (e.keyCode === 13) {
    e.preventDefault()
  }
}

I call handleKeyDown(e) which gets triggered but when calling e.preventDefault() it does not work for the enter key (It's called and definitely reached!). However, it will work for any other key and if I only call e.preventDefault() it will stop every key except the enter button.
Any help as to why this doesn't work would be greatly appreciated.

up

up


V3

Coverage increased (+0.2%) to 98.462% when pulling 0b1c38e88b51dd45c55696361cb4a90aab862f23 on v3 into 7917babc7977983a149ebcce6b6f1ae91aa228df on master.


Do we have duplicated data?

I guess Xarray is being clever and dropping the duplicates so we haven't noticed this yet?

No longer seems to be the case?


Wrong Assertion in Unit Test

:bug: The Bug
I recently submitted a pull request which does not match the conventions of other tests in that file.
:dart: The Goal
Change the code I added to match the other tests in tests/integration/routes/profile/patch-profile-test.js.
:clipboard: Step by Step

[x] Claim this issue (@kazekagegaara)
[ ] Setup repository locally and make sure that all tests pass
[ ] Updatetests/integration/routes/profile/patch-profile-test.js with correct assertion. Spoiler alert: If you get stuck, you might want to look here
[ ] Run test again wiht npm test to make sure that all test are still passing.
[ ] Commit with message test: Wrong assertion in tests/integration/routes/profile/patch-profile-test.js
[ ] Start a Pull Request. Set description to Closes  #122.
If this is your first, welcome! :tada: :smile: Here is a great tutorial on how to send a Pull Request)
[ ] Done :+1: Ask in comments for a review :)

So the change to make is to change t.deepEqual on line 111 to t.is? Or is there anything else too?

I can look on it, if it's free

So the change to make is to change t.deepEqual on line 111 to t.is

Yes :) @kazekagegaara wanna work on it? Otherwise @piton4eg would take it. It’s first-come-first-served :)
I’ll also prepare more issues this weekend, so watch this place :)

I'll take this one. Sorry @piton4eg !

Ok, I get an error while doing npm install. Here's the error :
npm ERR! 404 Not Found
npm ERR! 404
npm ERR! 404 'gar/hapi-json-api' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it
npm ERR! 404 It was specified as a dependency of 'hoodie-server-account'
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, or http url, or git url.

npm ERR! System Darwin 14.5.0
npm ERR! command "node" "/usr/local/bin/npm" "install"
npm ERR! cwd /Users/ASUuser/Desktop/hoodie/hoodie-server-account
npm ERR! node -v v0.10.33
npm ERR! npm -v 1.4.28
npm ERR! code E404
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /Users/ASUuser/Desktop/hoodie/hoodie-server-account/npm-debug.log
npm ERR! not ok code 0

I tried npm install @gar/hapi-json-api but I get the same error.
Also, you might want to add npm-debug.log in .gitignore.

hm the package is there: https://www.npmjs.com/package/@gar/hapi-json-api. Do you use any kind of proxy like https://github.com/nolanlawson/local-npm?
can you try npm_config_loglevel=verbose npm install @gar/hapi-json-api and paste your output here?

I'm not using any proxy(none that I'm aware of). But, I can go back home and try this again on my personal computer.
The requested output :
npm info it worked if it ends with ok
npm verb cli [ 'node', '/usr/local/bin/npm', 'install', '@gar/hapi-json-api' ]
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm WARN package.json<EMAIL_ADDRESS>No description
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm WARN package.json<EMAIL_ADDRESS>No README data
npm WARN package.json<EMAIL_ADDRESS>No repository field.
npm verb readDependencies using package.json deps
npm verb cache add [ '<EMAIL_ADDRESS>null ]
npm verb cache add name=undefined spec="<EMAIL_ADDRESS>args=["@gar/hapi-json-api@^2.0.1",null]
npm verb parsed url { protocol: null,
npm verb parsed url   slashes: null,
npm verb parsed url   auth: null,
npm verb parsed url   host: null,
npm verb parsed url   port: null,
npm verb parsed url   hostname: null,
npm verb parsed url   hash: null,
npm verb parsed url   search: null,
npm verb parsed url   query: null,
npm verb parsed url   pathname: '<EMAIL_ADDRESS>npm verb parsed url   path: '<EMAIL_ADDRESS>npm verb parsed url   href: '<EMAIL_ADDRESS>}
npm verb cache add name=""<EMAIL_ADDRESS><EMAIL_ADDRESS>npm verb parsed url { protocol: null,
npm verb parsed url   slashes: null,
npm verb parsed url   auth: null,
npm verb parsed url   host: null,
npm verb parsed url   port: null,
npm verb parsed url   hostname: null,
npm verb parsed url   hash: null,
npm verb parsed url   search: null,
npm verb parsed url   query: null,
npm verb parsed url   pathname<EMAIL_ADDRESS>npm verb parsed url   path<EMAIL_ADDRESS>npm verb parsed url   href<EMAIL_ADDRESS>}
npm verb cache add name="gar/hapi-json-api" spec="^2.0.1" args=["gar/hapi-json-api","^2.0.1"]
npm verb parsed url { protocol: null,
npm verb parsed url   slashes: null,
npm verb parsed url   auth: null,
npm verb parsed url   host: null,
npm verb parsed url   port: null,
npm verb parsed url   hostname: null,
npm verb parsed url   hash: null,
npm verb parsed url   search: null,
npm verb parsed url   query: null,
npm verb parsed url   pathname: '^2.0.1',
npm verb parsed url   path: '^2.0.1',
npm verb parsed url   href: '^2.0.1' }
npm verb addNamed [ 'gar/hapi-json-api', '^2.0.1' ]
npm verb addNamed [ null, '>=2.0.1-0 <3.0.0-0' ]
npm verb lock<EMAIL_ADDRESS>/Users/AU/.npm/e6511909-gar-hapi-json-api-2-0-1.lock
npm verb request where is /gar/hapi-json-api
npm verb request registry https://registry.npmjs.org/
npm verb request id e03781976e58794b
npm verb url raw /gar/hapi-json-api
npm verb url resolving [ 'https://registry.npmjs.org/', './gar/hapi-json-api' ]
npm verb url resolved https://registry.npmjs.org/gar/hapi-json-api
npm verb request where is https://registry.npmjs.org/gar/hapi-json-api
npm info trying registry request attempt 1 at 14:24:17
npm http GET https://registry.npmjs.org/gar/hapi-json-api
npm http 404 https://registry.npmjs.org/gar/hapi-json-api
npm ERR! 404 Not Found
npm ERR! 404
npm ERR! 404 'gar/hapi-json-api' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it
npm ERR! 404 It was specified as a dependency of 'hoodie-server-account'
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, or http url, or git url.

npm ERR! System Darwin 14.5.0
npm ERR! command "node" "/usr/local/bin/npm" "install" "@gar/hapi-json-api"
npm ERR! cwd /pathto/hoodie/hoodie-server-account
npm ERR! node -v v0.10.33
npm ERR! npm -v 1.4.28
npm ERR! code E404
npm verb exit [ 1, true ]
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /pathto/hoodie/hoodie-server-account/npm-debug.log
npm ERR! not ok code 0

ahh I know what the problem is, your node / npm versions are very outdated. Best is to install node from scratch form https://nodejs.org/en/, both LTS and stable will work. It will update npm, too. Could you please try that?

Ahh. I will have to use my personal computer then, I don't have admin access on this one. Sorry about the delay, expect a PR within 4-5 hours.

you should not need admin access to install it, I think. But also no worries about the delay, there is no rush, thanks for hanging in there :)

fixed via https://github.com/hoodiehq/hoodie-server-account/pull/123
Welcome at Hoodie, Manit! :wave: :dog: Are you on twitter? We like giving shout outs to new contributors :)
Let me know if you’d like to keep contribution, I’m happy to help find something.

@gr2m definitely. I would love to contribute more and learn. My twitter handle is @ manitsinghkalsi.

Cool feel free to look around if you see something interesting :) I’ll also look myself as soon as I find time and get back to you


Installing API request transformer prevents bot stop

Problem
Installing an (even empty) API request transformer makes bot stuck on bot.stop() until the next update arrives.
Reproduction
import { Bot } from "grammy"

function log(text: string) {
  console.log(`${new Date().toISOString()}: ${text}`)
}

const bot = new Bot(process.env.BOT_TOKEN)

// This makes the bot stuck on stop()
bot.api.config.use((prev, method, payload) => prev(method, payload))

bot.command("start", async (ctx) => {
  log("/start handler")
  await ctx.reply(`Welcome here.`)
})

setTimeout(async () => {
  log("calling bot.stop()")
  await bot.stop().then(() => log("bot.stop() completed"))
}, 5000) // <--- 3000 or smaller doesn't cause the bug.

log("calling bot.start()")
bot.start().then(() => log("bot.start() completed"))

Result
2022-02-08T05:26:56.329Z: calling bot.start()
2022-02-08T05:27:01.330Z: calling bot.stop()
2022-02-08T05:27:01.614Z: bot.stop() completed
2022-02-08T05:27:13.227Z: /start handler <------ this is when I sent /start to the bot
2022-02-08T05:27:13.613Z: bot.start() completed

This repeats consistently.
Removing the request transformer fixes the problem:
2022-02-08T05:32:05.909Z: calling bot.start()
2022-02-08T05:32:10.909Z: calling bot.stop()
2022-02-08T05:32:10.915Z: bot.start() completed
2022-02-08T05:32:11.297Z: bot.stop() completed

Environment
<EMAIL_ADDRESS>node 16.13.0
I know this all sounds odd, but I was getting the problem consistently in production, then started to reduce code until I arrived at this minimum reproduction repo.

This is not an empty API transformer. It's an API transformer which discards the AbortSignal instance that is being used to cancel requests. Hence, stopping the bot won't be able to cancel the last call to getUpdates.
This is a documentation problem.
Note also that we are planning on completely restructuring the API transformer parts of grammY for a distant 2.0 release. This will make it impossible to forget passing along the signal.


Added physics_util_2d.gd

Added a utility class to easily get the corresponding int generated by combining any 2d physics layers.
Also has a method which which will get you the bit which is used by a given layer name.

First of all, thank you for opening this pull request! The functionality seems to be quite useful. If you don't mind, I have a few questions/suggestions. I'd be happy to hear your thoughts on them.

Since the "layers" are all really one system basically, and are just arbitrarily divided into render/physics and 2d/3d, would you be interested in refactoring it to support all 4 combinations of layer types? I think that would re-use much of the implementation, but allow it to be used for a far wider variety of contexts. It's really just about using layer_names/ and then appending to the string 2d or 3d and _render or _physics, depending on which layer type is needed (which can easily be converted into constants on the script, e.g. const RENDER := 0; const PHYSICS := 1, etc and then passed in as parameters to get_* methods).
Since this is a class that specifically deals with layers, would you be okay with changing the name to PhysicsLayers (or, just Layers if you like idea 1)? I think that would communicate its scope and purpose much better to someone who stumbles upon it.
I don't see any reason why this should need to be an autoloaded node. Would you have any issues refactoring it to just be a script class? If so let me know, but if you are likewise interested in removing the autoload configuration dependency, then...

Make the class extend Reference instead of Node2D.
Change _PHYSICS_LAYERS_BIT from a var to a const (you will still be able to edit it).
Change the _ready() method to a setup() method.
Call the setup() method during any of the get_* methods if _PHYSICS_LAYERS_BIT is empty, before executing any other logic. This will ensure that the data is always initialized before it is actually accessed.
Add class_name PhysicsUtil2D, class_name PhysicsLayers, or class_name Layers to the top of the script under the extends statement (depending on your thoughts) of the earlier ideas.
Relocate the script to the addons/godot-next/globals section of the repository.

Let me know what you think.

Valid points.
For the Render Layers, I am honestly not familiar how they are used in godot. I did not find anywhere a way to specify a render layer for any of my nodes, unless this referes to the Z-index (I have only worked in 2D), which seems unlikely, since there's a only 20 layers for rendering and you can have a z-index higher than that, so I'm not really sure how this could be useful in any way for render layers.
As for the autoload part, the script doesn't really need to be one, it's just more practical from my experience. I have it autoloaded as "physics", so anywhere I can just do
physics.get_collision_layer(["enemies"], ["walls"])
Which is really only a preference. I guess I have no preference in this regard, and can see no benefit from either approach.
Renaming to PhysicsLayers and adding 3D sounds how it should have been done from the beginning, I just skip 3d stuff in my brain at this moment :P
Regarding the constants and having a parameter name, I guess i would just preffer having specific methods to go over 2d stuff and 3d stuff, as in "get_collision_mask_2d" rather than having a parameter to make a difference, what do you think about this?

I think that sounds good. Could make some private methods for handling the logic and then public methods that just call into it.
I'm not sure if I would use the render layers either, but given that it would be minimal effort to include, I think to myself, "might as well."
Using it as a script class with a const Dictionary would give it a similar workflow as using an autoload, but it would work even with tool scripts and would still be globally accessible.
You okay doing the implementation changes or okay with me doing them (either way)?


feat: Add some extra features

Add set loglevel to cmd
Add commit_write (it calls write_reg and commit)
Cleanup regtest outputs
Allow driver to be passed into MmIf class
Bump version to 1.1.0

Codecov Report

Merging #11 (3f421fb) into master (cfe2399) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   99.27%   99.30%   +0.03%
==========================================
  Files           3        3
  Lines         550      574      +24
  Branches       77       78       +1
==========================================
+ Hits          546      570      +24
  Misses          3        3
  Partials        1        1

Flag
Coverage Δ

99.30% <100.00%> (+0.03%)
:arrow_up:

3.8
99.30% <100.00%> (+0.03%)
:arrow_up:

3.9
?

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

Impacted Files
Coverage Δ

mm_pal/mm_cmd.py
100.00% <100.00%> (ø)

mm_pal/mm_if.py
100.00% <100.00%> (ø)

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 cfe2399...3f421fb. Read the comment docs.


Add test container for UT of MySQL passwordless feature

As title.

API change check
API changes are not detected in this pull request.

What is the state of the PR? I can for sure help with the Testcontainers integration side, but currently there is a lot of commented-out code, so I am not sure if it makes sense to review already?

What is the state of the PR? I can for sure help with the Testcontainers integration side, but currently there is a lot of commented-out code, so I am not sure if it makes sense to review already?

Sorry for the late reply @kiview, but we think this PR is blocked by the discussion in https://github.com/Azure/azure-sdk-for-java/issues/31942.


[Dropdown] - Search selection shows typed value under option

Steps to Reproduce

Click on Search Selection example of Dropdown
Start typing first letters (ex. Canada_
Click on "Canada" option

Expected
Should show option and do not show typed text
Result
Shows typed text under option

It looks like the version of SUI in the docs somehow is out of date its using 2.0.1, this bug was fixed in 2.0.2.
My apologies

I've redeployed now this should be fixed.


Handle indented heredocs in 5.26+

https://www.effectiveperlprogramming.com/2016/12/strip-leading-spaces-from-here-docs-with-v5-26/

This has been implemented in #255


Parse decimals and ints with InvariantCulture since JSON/YAML doesn't vary by culture.

Parse decimals and ints with InvariantCulture since JSON/YAML doesn't vary by culture.
For example, the parser should always treat 100.000 as one hundred, not one hundred thousand, even if the operating system on the machine is set to use French.
Fix #323

Note that bool.Parse is already always cultureInvariant (that culture parameter doesn't exist). It only recognizes "true" and "false" regardless of culture.


Compile instrument client on windows

Hi
can u please guide me how i can compile it on windows ?
i m able to successfully compile on mac . but struggling with windows
Thanks

@troybowman any help sir ?


Missing LICENSE

Cool project! But it's not clear under which license you are making this code available... am I allowed to use it? Under what conditions?
Thou shall ALWAYS choose a license... 😈

Oh, https://github.com/poser55/linked-db-rows/blob/3c0bfee51cddc7cd2db0bb80687125f5bca832ac/README.md#license
ergo #5

Just added a license:
https://github.com/poser55/linked-db-rows/blob/master/LICENSE


Put uchiwa behind Apache

I am proposing this as alternative to #62.
This change configures both uchiwa and kibana to run behind an Apache frontend, and enforces SSL access to the web UI.  Both applications can be run on the same host (or on different hosts).  In addition to changes to the uchiwa, kibana, and httpd roles, this introduces a new "opstoolsvhost" role that is responsible for a top-level Apache configuration file.
This does not use the generalized proxy support proposed in #62 because the applications each required unique proxy configurations.

Incidentally, I've tested this with all services on a single host:
[logging_hosts]
server0

[am_hosts]
server0

And with logging and monitoring services separate:
[logging_hosts]
server0

[am_hosts]
server1

Note that there are still some outstanding issues with the default ip addresses we use for binding and connecting, but I would like to make that part of a separate PR.

The role/httpd/proxy is no longer used , I think it should be deleted.

+1, It works for me