Billink logo

Hello,
Is it possible to add the Billink logo?
I already made the default icon, so hopefully this is correct format. I can add this to a PR if that would be easier.

Yes it is possible and yeah please do create a PR that would be great 😉

Few notes on the svg format, you input is using a few thing we are trying to avoid to make sure the svg is as small as possible:
Your input is with CSS classes that could be used directly as svg fill attributes:
<svg xmlns="http://www.w3.org/2000/svg" width="24" height="16" viewBox="0 0 24 16">
<style type="text/css">
	.st0{fill:#FFFFFF;}
	.st1{fill:#F18D21;}
</style>
<path class="st0" d="M0,2c0-1.1,0.9-2,2-2h20c1.1,0,2,0.9,2,2v12c0,1.1-0.9,2-2,2H2c-1.1,0-2-0.9-2-2V2z"/>
<g>
	<path class="st1" d="M2.1,6.7c0.4-0.3,0.8-0.5,1.3-0.5C4.7,6.1,5.6,7,5.9,8c0.3,1.3-0.6,2.6-1.9,2.8c-0.1,0-0.1,0-0.2,0
		c-0.2,0-0.3,0-0.5,0c-0.1,0-0.1,0-0.2,0c-1-0.2-1.9-1.1-1.9-2.2c0-0.8,0-1.5,0-2.3c0-0.4,0-0.8,0-1.2c0.2,0,0.4,0,0.6,0
		C2,5.2,2.1,5.2,2.2,5.3c0.3,0.2,0.4,0.5,0.3,0.9C2.4,6.3,2.2,6.4,2.1,6.7C2.1,6.7,2.1,6.7,2.1,6.7C2.1,6.7,2.1,6.7,2.1,6.7z
		 M2.3,8.5c0,0.5,0.2,0.8,0.6,1c0.4,0.3,0.9,0.3,1.4,0C4.6,9.3,4.8,9,4.8,8.5C4,8.5,3.2,8.5,2.3,8.5z"/>
	<path class="st1" d="M18.8,10.9c0-1.8,0-3.7,0-5.5c0.4,0,0.7,0,1.1,0c0,1.1,0,2.2,0,3.3c0.5-0.7,1.1-1.4,1.6-2c0.4,0,0.8,0,1.3,0
		c-0.5,0.6-0.9,1.2-1.4,1.8c0.5,0.8,0.9,1.7,1.4,2.5c-0.4,0-0.8,0-1.3,0c0,0,0,0,0-0.1c-0.3-0.5-0.5-0.9-0.8-1.4c0,0,0,0,0-0.1
		c-0.3,0.3-0.5,0.7-0.7,1c0,0.2,0,0.4,0,0.5C19.6,10.9,19.2,10.9,18.8,10.9z"/>
	<path class="st1" d="M14.3,10.9c0-0.3,0-0.5,0-0.8c0-1.1,0-2.2,0-3.4c0,0,0-0.1,0-0.1c0.1,0,0.2,0,0.3,0c0.1,0.1,0.2,0.3,0.3,0.4
		c0.2-0.1,0.4-0.2,0.6-0.3c1.2-0.5,2.4,0.2,2.6,1.5c0,0.1,0,0.2,0,0.3c0,0.8,0,1.5,0,2.3c0,0,0,0.1,0,0.1c-0.4,0-0.7,0-1.1,0
		c0-0.8,0-1.6,0-2.4c0-0.5-0.4-0.8-0.8-0.8c-0.5,0-0.8,0.4-0.8,0.8c0,0.8,0,1.5,0,2.3c0,0,0,0.1,0,0.1C15,10.9,14.6,10.9,14.3,10.9z
		"/>
	<path class="st1" d="M10.5,10.9c0,0,0-0.1,0-0.1c0-1.7,0-3.5,0-5.2c0,0,0-0.1,0-0.1c0.4,0,0.7,0,1.1,0c0,0,0,0.1,0,0.1
		c0,1.8,0,3.5,0,5.3c0,0,0,0.1,0,0.1C11.2,10.9,10.8,10.9,10.5,10.9z"/>
	<path class="st1" d="M8.5,10.9c0,0,0-0.1,0-0.1c0-1.7,0-3.5,0-5.2c0,0,0-0.1,0-0.1c0.4,0,0.7,0,1.1,0c0,0,0,0.1,0,0.1
		c0,1.7,0,3.5,0,5.2c0,0,0,0.1,0,0.1C9.3,10.9,8.9,10.9,8.5,10.9z"/>
	<path class="st1" d="M6.6,10.9c0,0,0-0.1,0-0.1c0-1.2,0-2.4,0-3.6c0,0,0-0.1,0-0.1c0.4,0,0.7,0,1.1,0c0,0,0,0.1,0,0.1
		c0,1.2,0,2.4,0,3.6c0,0,0,0.1,0,0.1C7.4,10.9,7,10.9,6.6,10.9z"/>
	<path class="st1" d="M12.4,10.9c0-0.4,0-0.8,0-1.3c0-0.8,0-1.6,0-2.4c0,0,0-0.1,0-0.1c0.4,0,0.7,0,1.1,0c0,0,0,0.1,0,0.1
		c0,1.2,0,2.4,0,3.6c0,0,0,0.1,0,0.1C13.1,10.9,12.7,10.9,12.4,10.9z"/>
	<path class="st1" d="M7.9,5.9c0,0.4-0.3,0.7-0.7,0.7c-0.4,0-0.7-0.3-0.7-0.7c0-0.4,0.3-0.7,0.7-0.7C7.6,5.2,7.9,5.5,7.9,5.9z"/>
	<path class="st1" d="M13.6,5.9c0,0.4-0.3,0.7-0.7,0.7c-0.4,0-0.7-0.3-0.7-0.7c0-0.4,0.3-0.7,0.7-0.7C13.3,5.2,13.6,5.5,13.6,5.9z"
		/>
</g>
</svg>

You can remove the style tag and replace the classes with the fill and even better add the fill to the group element
<svg xmlns="http://www.w3.org/2000/svg" width="24" height="16" viewBox="0 0 24 16">
	<path fill="#fff" d="M0 2a2 2 0 0 1 2-2h20a2 2 0 0 1 2 2v12a2 2 0 0 1-2 2H2a2 2 0 0 1-2-2V2Z" />
	<g fill="#f18d21">
		... (same content without the class)
	</g>
</svg>

Also with using svgomg we can make you multiple paths in to somethings small:
<svg xmlns="http://www.w3.org/2000/svg" width="24" height="16" viewBox="0 0 24 16">
	<path fill="#fff" d="M0 2a2 2 0 0 1 2-2h20a2 2 0 0 1 2 2v12a2 2 0 0 1-2 2H2a2 2 0 0 1-2-2V2Z" />
	<path fill="#f18d21" d="M2.1 6.7c.4-.3.8-.5 1.3-.5C4.7 6.1 5.6 7 5.9 8c.3 1.3-.6 2.6-1.9 2.8H3.1c-1-.2-1.9-1.1-1.9-2.2V5.1h.6c.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>-.1.1-.3.2-.4.5zm.2 1.8c0 .<IP_ADDRESS> 1 .<IP_ADDRESS> 1.4 0 .3-.2.5-.5.5-1H2.3zM18.8 10.9V5.4h1.1v3.3c.5-.7 1.1-1.4 1.6-2h1.3c-.5.6-.9 1.2-1.4 <IP_ADDRESS>.9 1.7 1.4 2.5h-1.3v-.1c-.3-.5-.5-.9-.8-1.4v-.1c-.3.3-.5.7-.7 1v.5h-1.2zM14.3 10.9V6.6h.3c.<IP_ADDRESS>.3.4l.6-.3c1.2-.5 2.4.2 2.6 1.5V10.9H17V8.5c0-.5-.4-.8-.8-.8-.5 0-.8.4-.8.8v2.4h-1.1zM10.5 10.9V5.5h1.1V11c-.4-.1-.8-.1-1.1-.1zM8.5 10.9V5.5h1.1V10.9H8.5zM6.6 10.9V7.1h1.1V10.9H6.6zM12.4 10.9V7.1h1.1V10.9h-1.1zM7.9 5.9c0 .4-.3.7-.7.7-.4 0-.7-.3-.7-.7 0-.4.3-.7.7-.7.4 0 .<IP_ADDRESS>zM13.6 5.9c0 .4-.3.7-.7.7-.4 0-.7-.3-.7-.7 0-.4.3-.7.7-.7.4 0 .<IP_ADDRESS>z"/>
</svg>

Looking forward to your PR thanks for contributing @ThijmenKort

Hi @GrimLink,
I opened the PR. Please let me know if this is good.


use 'm' for minutes

Is there a way to use m for minutes with the ms shorthand utility? I see m is used for meters currently but its also commonly used for minutes. (even in the ms package)

It might be possible to add in special behavior when it seems like you're converting time (or using ms()) but use the unit m. I'll play around with some ideas and update here if I get something working.

much appreciated!


Fallback for token rates api query on contracts to coin ids.

Describe the bug
Currently we query ethereum token contract addresses for the token rates, but certain tokens do not have conversion rates through the token_price/ethereum?contract_addresses rather they are in a different route simple/price?ids. Example token is LTO.
LTO contract address Etherscan.
Coingecko ethereum token contract call (no conversion).
But
Coingecko LTO simple/price?ids=lto-network
Steps to reproduce (REQUIRED)

Add the LTO token
See no fiat conversion rate.

Expected behavior
Have a fallback for token queries where it might be in a different route.
Additional context (Error Messages, etc.)
This might require storing the values of these specific tokens and matching then on the /coins/list.

We are no longer using CoinGecko so this should be no longer an issue.


[Regression] workspace previous dispatcher breaks if allow_workspace_cycles is set to false.

Steps to reproduce
Be on commit 287e6c4ede30c8e5e85863cd85f157e17e24879c or later
set allow_workspace_cycles = 0
go to workspaces 1 > 2 > 3
workspace previous
Expected
1 2 [3] > workspace previous > 1 [2] 3 > workspace previous > [1] 2 3
Noted
1 2 [3] > workspace previous > 1 2 [3] (and in the logs: [LOG] No previous workspace to change to)
Other behavior changes noted

Since I use workspace cycles as a sort of alt-tab between two workspaces, this commit seems to invert the behavior of allow_workspace_cycles. Previously if it was set to false, and you were on workspace 1 > 2 > 3, then workspace previous, they would cycle between 3 and 2, and if set to true, would go 3 > 2 >1, which makes less sense, so the new logic seems to work as intended.

With allow_workspace_cycles=1
Switching from an empty workspace to an active one only works once. Example:
[1] 2 > workspace 3 > 1 2 [3] > workspace previous > [1] 2 > workspace previous > 1 2 [3] > workspace previous > [LOG] No previous workspace to change to. Unsure of what's the intended behavior.

check with 83f1616a65091fe2e66d054f5ef1343d755a3e0c

Aye, that fixes all issues mentioned here, thanks!


Capsolver puppeter demo pls

Hey, it's conceivable that you create one for [capsolver.com], which offers the quickest and most affordable captcha solution on the market (https://www.capsolver.com/).
They are capable of resolving hcaptcha, recaptcha, turnstile, kasada, akamai, funcaptcha, datadome captcha, geetest, etc.
Additionally, you may review their documentation at https://docs.capsolver.com/.

Hey, it's conceivable that you create one for [capsolver.com], which offers the quickest and most affordable captcha solution on the market (https://www.capsolver.com/). They are capable of resolving hcaptcha, recaptcha, turnstile, kasada, akamai, funcaptcha, datadome captcha, geetest, etc. Additionally, you may review their documentation at https://docs.capsolver.com/.

check https://github.com/openHacking/bypass-captcha/issues/1#issuecomment-1330145187


Add Global Interaction Receiver to Interaction Manager

Add ability to send Tapped and DoubleTapped events to a global Interaction receiver GO in addition to the regular interaction path.
This is useful if you want to trigger something with the Clicker for example (like taking a picture) without going through the focus manager

@ei2kpi,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

BTW unsure if there is a better way to do this an I am just missing it

Ping...

@ei2kpi,
This should be doable currently with how it's setup.  For instance you should be able to use the action in InteractionManager to subscribe to the event globally regardless of the focus object.
Interaction Manager:
public static event Action<GameObject, InteractionEventArgs> OnTapped;
Your global listener:
InteractionManager.OnTapped += TappedDelegate;
Take a look at how the InteractionReceiver is setup.  That should do exactly what you're trying to do.  Please let me know if this doesn't work for you.
thanks,
-pat


Possible to always show the foldout triangle in side bar?

Unlike the default Doxygen theme, Awesome only shows the foldout triangles if you hover over that particular element. Is there a quick way to change that behavior to always show?
Thanks,

Thanks for all the responses! Worked like a charm!

FYI: I have added the new variables --side-nav-arrow-opacity &  --side-nav-arrow-hover-opacity that make the arrow opacity in the sidebar configurable and I adopted the beautiful arrow style by @jpgarza93 in this commit: https://github.com/jothepro/doxygen-awesome-css/commit/20d9586642ff3aba0b7814f26c8f2012fbfc4373
Pls let me know if that helps you to reduce your manual overrides. This will be included in a new release in the next days. :)


Stripe Recipient API Feature

added stripe recipient api endpoints with passing unit tests
@nberardi  please code review
@paulirwin  please code review

For the specific Recipient endpoints, the "card" parameter is used instead of "source", as you mentioned before the Recipient endpoints are being deprecated and have not followed through the rest of the API updates such as the Customer endpoints which now make use of "source" instead of "card".
Sending a Stripe-Version in the header of the requests would be an ideal next step to better support the most current version of the API.
@paulirwin please make that an issue so none of these discussions get lost when we're closing PRs

I'm working through these oldest-first, and this one cannot be merged due to a conflict. Can you get this to where it can be merged by merging this upstream master into your feature/recipient-api-feature branch?

@paulirwin  updated pull request


Clusters show next to accidents when zooming in

I think the problem only occurs after using the date filters on cluster mode.
There are several weird behaviors when doing that..
Any ideas?

@danielhers @aviklai @OmerSchechter @HamutalCohen3 @LironRS :
Did any of you encounter this?
If you did not we can close this one

I did, once. Not recently though. Let's give it a few days to see if anyone
encounters it.
On Sat, Dec 5, 2015 at 10:41 AM Omer Hamerman<EMAIL_ADDRESS>wrote:

@danielhers https://github.com/danielhers @aviklai
https://github.com/aviklai @OmerSchechter
https://github.com/OmerSchechter @HamutalCohen3
https://github.com/HamutalCohen3 @LironRS https://github.com/LironRS :
Did any of you encounter this?
If you did not we can close this one
—
Reply to this email directly or view it on GitHub
https://github.com/hasadna/anyway/issues/488#issuecomment-162161808.

@danielhers I think I've noticed it yesterday when @galraij presented the unstable version.
I think it occurs when we change some of the filters while using cluster mode. Not sure though...
We can leave this closed until anyone encounters it again


GBHFacebookPickerConfig: Possible to insert custom namespaced localiz…

…ed strings

Hello @tof-amsiq ,
Maybe we can create a new struct into GBHFacebookPickerConfig ?
For exemple : public struct TextConfig to put all the text.
Thanks

Yes of course, try to take a look at it now.

Thank you, can you please make a release ?

The develop branch have the select all new feature. I need to test it and improve some layout problem before making a release. You can use the develop branch in your pod if you need absolutely this PR :)

When do you think the expected date for release is ?

Idk, I don't have a lot of time at this period, currently at the WWDC :)
You can use : pod 'GBHFacebookImagePicker', :git => 'https://github.com/floriangbh/GBHFacebookImagePicker.git', :branch => 'develop' ...


Invoke template live test pipeline as part of common tool sync process

Some common pipelines and scripts are related to live tests

@pakrym while I'm OK with this I'm curious if there was a particular issue we hit that prompted you to request this?

https://github.com/Azure/azure-sdk-tools/pull/1077 PR required invocation of the live tests to verify things are still working. I did it manually but it would be nice if this aspect of common infrastructure was tested along with the release pipeline.

Cool does the live test pipeline for the template project do anything useful in .NET?

Yes, the template is a trimmed down keyvault client with an actual template and test.
https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/template/test-resources.json
https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/template/Azure.Template/tests/MiniSecretClientLiveTest.cs

Pretty sure @chidozieononiwu already enabled this.

I don't think so. https://github.com/Azure/azure-sdk-tools/pull/1233#issuecomment-737533097 doesn't list any test pipelines being invoked.

Awesome! Thank you @chidozieononiwu !


feat: added connection exception logic when trainee current dbc does …

…not match with programme owner dbc TIS21-1072

Just had a quick look at why the Sonar check is failing, seems there are two issues you may want to look at:

there are 6 code smells in the new commited code
test for ExceptionService is needed for the test coverage

Just had a quick look at why the Sonar check is failing, seems there are two issues you may want to look at:

there are 6 code smells in the new commited code
test for ExceptionService is needed for the test coverage

Just had a quick look at why the Sonar check is failing, seems there are two issues you may want to look at:

there are 6 code smells in the new commited code
test for ExceptionService is needed for the test coverage

Thanks @DorisWongNHS I have taken care of those issues

Just had a quick look at why the Sonar check is failing, seems there are two issues you may want to look at:

there are 6 code smells in the new commited code
test for ExceptionService is needed for the test coverage

Thanks @DorisWongNHS I have taken care of those issues


--gaf in genescape build requires a gzipped file

The README shows it as ungzipped.

documentation has been corrected


:bug: copy the custom plugin to m2

API Tests PR: 55
Sorry, install-file ended up failing because it wasn't resolving the plugin deps correctly and was creating artifact without full classpath.

Why this over installing with mvn install-file -s settings.xml using the settings?

@shawn-hurley install goal didnt work. I was seeing errors with classpath when running. I saw that there are additional metadata files with the plugin that weren't present when I do the install.

This is no longer needed as we are using the other plugin now


Add tensorrt test workflow

Description
Added the tensorrt test workflow:
example run:
https://github.com/pytorch/TensorRT/actions/runs/11675576873/job/32510381587?pr=3266
Type of change
Please delete options that are not relevant and/or add your own.

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 not work as expected)
This change requires a documentation update

Checklist:

[ ] My code follows the style guidelines of this project (You can use the linters)
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[ ] New and existing unit tests pass locally with my changes
[ ] I have added the relevant labels to my PR in so that relevant reviewers are notified

Seems like 12.1 got dropped is that us or PyTorch?

@narendasan
yes, cuda 12.1 is deprecated by pytorch, it was merged in just a few hours ago.
https://github.com/pytorch/test-infra/pull/5904
Very wierd is that cuda11.8 is still supported, they only deprecated cuda12.1


Adding different styles to different part of the same heading in Chartjs

Feature Proposal
Please refer to the Question here.
Possible Implementation
No response

@dhana555std have you tried title and subtitle configuration?
DOC:
https://www.chartjs.org/docs/latest/configuration/title.html
https://www.chartjs.org/docs/latest/configuration/subtitle.html

@stockiNail indeed I referred but that is not what I want. I want to apply different styles to different parts of the title.
For example:-
Level: Execution. Path: X->Y->Z.
You can see the above title where the values of the title are Bold whilst the words Level and Path are normal.

Maybe I'm wrong but I don't think this is doable.

I'm not sure we want to re-invent the wheel here, because its quite easy to do it as html outside the chart.
https://codepen.io/kurkle/pen/yLpPmNV

Closing since this isn't going to be on our roadmap. Using an external HTML title is the better way to achieve this


unit test for ServiceConfig

What is the purpose of the change
unit test for ServiceConfig
Brief changelog
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[x] If this contribution is large, please follow the Software Donation Guide.

The UT is failing, please take a look at it.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@50895ff). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1780   +/-   ##
=========================================
  Coverage          ?   39.34%
  Complexity        ?     4215
=========================================
  Files             ?      618
  Lines             ?    29742
  Branches          ?     5257
=========================================
  Hits              ?    11701
  Misses            ?    16178
  Partials          ?     1863

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 50895ff...f88bc0b. Read the comment docs.


Draft: NPM Mirroring

A couple of links to the Garage pages on Nexus/Artifactory. They don't have a huge amount of detail so may not be worth including:

https://www.ibm.com/garage/method/practices/deliver/tool_nexus/
https://www.ibm.com/garage/method/practices/deliver/tool_artifactory/

Add somehting related to rate limits on npm @lholmquist

@lholmquist commercial vs. opensource versions of artifactory/nexus


Integrate EVM-Substrate address converter tool.

Description
Currently, we have two separate address spaces for EVM and Substrate spaces. Unfortunately, it leads to unclarity in transferring funds from one wallet to another space.
It's possible to do this by converting one address into another one.
Example:

You want to transfer funds from Substrate to EVM

You get the receiver address and convert from H160 to SS58 format
You send a transfer to the new converted address and receive funds

You want to transfer funds from EVM to Substrate

You convert address from SS58 to H160 format
You send a transfer to the new address space on the EVM side
Receiver sends extrinsic to get funds (evm.withdraw). The receiver needs to do so because EVM lives as a sandbox and knows nothing about the substrate, but the substrate knows about EVM.

Definition of "Done"
We have a tab with address converter functionality. Integrating the address convertation into our block explorer would be nice, increasing UX. Also, think about incorporating this functionality directly into the transfer, so the user could put a mark that it wants to send funds into EVM and specify the EVM address, and we convert it for him.
Please note that currently, we can integrate into transfer functionality directly only WASM to EVM, but not EVM to WASM, since users are interacting with EVM using Metamask, and other wallets.

[ ] Design address converter functionality
[ ] Design transfer functionality (optional)
[ ] Implement designed functionality
[ ] Test it

Suggested Solution
There is already a converter deployed to the code sandbox. We could review it and integrate something similar to our solution.
https://codesandbox.io/s/evm-substrate-address-converter-3k9q3
Additional Information
Astar article how to convert funds.

@akorchyn Was there a reason why this task didn't go forward since March?
I'm also concerned that the whole repo is a fork of some other one, non official repo, and is a way behind the original polkadot-js explorer.

It has some implementation here -> https://github.com/GoldenGateGGX/blockchain-explorer/pull/12. But I'm not sure if it implements all the requirements,  I didn't really track this repo. I dunno, if it's deployed, though.
We had used a fork because when we had a repo, polkadot js hadn't supported weight system V2. It was merged, and we could potentially switch to the original repo we didn't have a capacity to review incoming changes (and fix merge conflicts)

@akorchyn Thanks for explaining. Let's keep this issue open and see in #12 addresses all of these issues or not.
I'm going to create and issue to estimate the efforts and possibility to rebase from the original fork. Maybe it's even too late, but at least we can get some resources estimation.

It seems there is a bit of a design issue. It shows like an empty result with able to click on it.


Accessing @conn in Commander?

I'm wondering which is the canonical way to be able to access @conn in a Commander handler. For example, I've tried to render peekable @conn adding <%= if true do %><% conn = @conn %><% end %> in the template, like in the following example:
# lib/commander/page_commander.ex

...

  defhandler peek_test(socket, sender) do
     socket
     |> peek("index.html", :conn)
     |> do_something_with_conn()
  end

# lib/commander/page_controller.ex

...

  def index(conn, _) do
    render(@conn, "index.html")
  end

# lib/templates/page/index.html.drab

    <%= if true do %>
    <% conn = @conn %>
    <% end %>

    <button type="button" drab="click: peek_test">Peek Test</button>

It works, but is it legitimate, or is there a better way?

You can always put it in the session and access the session in the commander, see https://tg.pl/drab#store
But the real question is, what do you need conn for? You should not need it after the page is rendered.

It was more a 'philosophical' question than practical, to better understand how Drab to expose assigns. But there was also a practical reason: having access to @conn can be useful when you have to call from the Commander some library functions that expect the current @conn as a parameter (of course only if they doesn't attempt to modify it)

Ahh, this is a bug. @conn should not be available in this case. Fixing it.

@conn case must be reconsidered.
I am going to use @OvermindDL1's idea from #51 to use only the specific fields of it (configurable).

The reason for that is to reduce the payload size? In this case maybe can be useful allow to get all the structure if wanted/needed, leaving the performances responsibility of this choice on the developer

This is what we want to do, see #51

Fixed in v0.7.7


pass all the parameter as configuration elements

Do you mean as members of the Configuration struct ?
Using a HashMap aims to avoid having to know about client specific parameters, each client implementing its own analyser might pass different parameters that are not relevant to other implementation
I used the Configuration struct as a convenience solution as it is already provided through the pipeline to the analyser

Oh, yes, I see : server parmaters versus Custom (payload layer) configuration. Yes, why not. To discuss, clarify and write alittle explanation intthe README at the end : I keep the issue opened, I add the documentation tag.

Sure this is open for discussion, I did it kind of the quick and easy way to be able to implement the setting override from a file in the jam detector
As it could be useful short term but not mandatory I did not feel the need to mess up the whole Configuration struct and process

Merged in 2.0.0
Additional parameters that could be needed can easily be added via Configuration modification, ini file loading is ready
Closing issue


feat(WEB-8993): Add Accordion hook for post, tag, and author forms

Changes

Add Accordion hook
Apply to existing accordion component in form_component

Pull Request Test Coverage Report for Build 987b49a82af6dfe34bce92435b5d95e5e45f7548

24 of 24 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 69.857%

Totals

Change from base Build fc84beeb6572c0056061fe26cfc7ed29176d6b2b:
0.1%

Covered Lines:
1219

Relevant Lines:
1745

💛  - Coveralls


unknown keyword: :uses_transaction

I like to switch from mysql2 to trylogy but if i add the "activerecord-trilogy-adapter" gem in my Gemfile and change the adaptor in my "config/database.yml" to trilogy i get the next error:
unknown keyword: :uses_transaction
      def call(env)
        env[Const::PUMA_CONFIG] = @config
        @app.call(env)
      end
    end

Using Rails <IP_ADDRESS>
Puma version: 6.4.3 (ruby 3.2.3-p157)

First lines full Trace:
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract_adapter.rb:998:in with_raw_connection'](http://crm.lvh.me:3000/crm#) [activerecord-trilogy-adapter (2.2.0) lib/active_record/connection_adapters/trilogy_adapter.rb:210:in get_full_version'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract_mysql_adapter.rb:87:in get_database_version'](http://crm.lvh.me:3000/crm#) [activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/schema_cache.rb:374:in database_version'
....
Any help I would greatly appreciate

If you're using Rails 7.1+, you don't need to use this gem and it should be removed from your gemfile; Rails 7.1+ itself includes a version of this adapter.
Duplicate of #68.

Hi bensheldon, thanks for you feedback. Indeed its mentioned that you don't need to install the gem "activerecord-trilogy-adapter" when using rails v7.1+. But when booting rails server i get the message:
rails s
=> Booting Puma
=> Rails <IP_ADDRESS> application starting in development
=> Run `bin/rails server --help` for more startup options
Exiting
/usr/share/rubygems-integration/all/gems/bundler-2.4.20/lib/bundler/rubygems_integration.rb:280:in `block (2 levels) in replace_gem': Error loading the 'trilogy' Active Record adapter. Missing a gem it depends on? trilogy is not part of the bundle. Add it to your Gemfile. (LoadError)


If get fails, we exceeded API rate of amazon

#1969 to master

OSError: [Errno 2] No such file or directory: 'jenkins/test/validators/'

Broken tests, should be fine.


Update index.html

Add relative path.

Description

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Please explain what is the issue you're fixing with this PR


Upgrade version.mockito 3.7.0 -> 3.7.7

This PR contains the following updates:

Package
Update
Change

org.mockito:mockito-inline
patch
3.7.0 -> 3.7.7

org.mockito:mockito-core
patch
3.7.0 -> 3.7.7

Release Notes

mockito/mockito
v3.7.7
Compare Source
Changelog generated by Shipkit Changelog Gradle Plugin
3.7.7

2021-01-16 - 1 commit(s) by Stefan Bohn
Fix reversed order of verify parameters (#2179)
Feature request: Unify order of parameters (#2173)

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Mockito 3.7.0 -> 3.7.7

See:
- https://github.com/mockito/mockito/blob/release/3.x/doc/release-notes/official.md
- https://github.com/mockito/mockito/releases/tag/v3.7.1
- https://github.com/mockito/mockito/releases/tag/v3.7.2
- https://github.com/mockito/mockito/releases/tag/v3.7.3
- https://github.com/mockito/mockito/releases/tag/v3.7.4
- https://github.com/mockito/mockito/releases/tag/v3.7.5
- https://github.com/mockito/mockito/releases/tag/v3.7.6
- https://github.com/mockito/mockito/releases/tag/v3.7.7
- https://github.com/mockito/mockito/compare/v3.7.0...v3.7.7

Suggested commit message:
Upgrade Mockito 3.7.0 -> 3.7.7

See:
- https://github.com/mockito/mockito/blob/release/3.x/doc/release-notes/official.md
- https://github.com/mockito/mockito/releases/tag/v3.7.1
- https://github.com/mockito/mockito/releases/tag/v3.7.2
- https://github.com/mockito/mockito/releases/tag/v3.7.3
- https://github.com/mockito/mockito/releases/tag/v3.7.4
- https://github.com/mockito/mockito/releases/tag/v3.7.5
- https://github.com/mockito/mockito/releases/tag/v3.7.6
- https://github.com/mockito/mockito/releases/tag/v3.7.7
- https://github.com/mockito/mockito/compare/v3.7.0...v3.7.7


Commented out player entity check causes crash if blocks are placed by non-player entity

https://github.com/Fabricators-of-Create/Create/blob/d9851d3035dfd9b1a182be1ca52385179324c5ab/src/main/java/com/simibubi/create/content/curiosities/symmetry/SymmetryHandler.java#L49
This commented out check causes a crash if a modded entity places blocks see: https://github.com/skylorbeck/MageHand/issues/8

This is the Forge version for reference https://github.com/Creators-of-Create/Create/blob/mc1.18/dev/src/main/java/com/simibubi/create/content/curiosities/symmetry/SymmetryHandler.java#L52

Heads up, this bug is back and the code is commented out again. https://github.com/Fabricators-of-Create/Create/blob/8e42841e2577ee4c5535ff1563e04dd7ef9dbf4b/src/main/java/com/simibubi/create/content/curiosities/symmetry/SymmetryHandler.java#L49

How…

Also this code is commented out in every branch I could find.

looking at this again, this is not an issue. The comment could be removed, but my fix was a null check a few lines down, which is still present. If something is crashing, it's a separate issue.

Okay well I guess having support for nonplayer entities placing blocks isn't important then. I'll be sure to pass that onto all my users who crash because of it.

This event only listens for players placing blocks. If non-players placing is crashing, it's a separate issue and I'd still like to see the log and maybe fix it.

This event only listens for players placing blocks. If non-players placing is crashing, it's a separate issue and I'd still like to see the log and maybe fix it.

You should read the whole thread. You guys already fixed this bug once and I've linked a user's crash log in the opening post. You aren't listening for players, you're listening for onblockplaced and you explicitly commented out the filter for players. I don't really care if it's fixed or not but at least understand what you're saying.

This check will fail when a nonplayer entity places a block. https://github.com/Fabricators-of-Create/Create/blob/8e42841e2577ee4c5535ff1563e04dd7ef9dbf4b/src/main/java/com/simibubi/create/content/curiosities/symmetry/SymmetryHandler.java#L56
You are in OnBlockPlaced which is called when blocks are placed. You cannot guarantee players will be the only code that reaches here. I don't even know why this is an issue.


Load custom attribute columns from miq expression when report is generated

Purpose or Intent
Custom virtual dynamic field for reporting have been introduced here https://github.com/ManageIQ/manageiq/pull/9451
When we are using custom virtual dynamic fields in filters (MiqExpression) and these field are not listed in MiqReport#cols (selected fields for report) then it was causing error message when such report is generated for first time.
This fix is loading virtual custom fields from MiqExpression.
Links

https://github.com/ManageIQ/manageiq/pull/9451

cc @gtanzillo
fyi @imtayadeway
@miq-bot add_label reporting, bug

@miq-bot assign @gtanzillo

/cc @kbrock I imagine you might have a stake in this

@lpichler what is a custom virtual column?
It is a column that is not a real column but rather a user defined custom column?
I do worry about calling these virtual_columns since that phrase means something specific in our code base. `
Can we call this custom_attributes or custom_attribute_columns?

Looks like nil columns are coming back in the tests.

@kbrock yes I took custom_attribute_columns

@lpichler what is a custom virtual column?

it is virtual column which is added dynamically and it is returning requested custom attribute from
table CustomAttribute
it was added here https://github.com/ManageIQ/manageiq/pull/9451/commits/3b2b7ee73b5ad9fb49848e4463e80a2695069f4e
thanks

@lpichler this looks promising :thumbsup: I have some questions about your algorithm above - they are kind of naive questions though so I'm mostly asking "can you assume that this is the case here", as I don't know. I'm thinking that if the answer is yes then for now it might be better to inline this logic as it may not be possible to generalize it at this point (i.e. not all expressions have fields, etc..). If no then there are a few more edge cases to cover but it's looking good. @gtanzillo please do keep me honest here!

@imtayadeway thank you for your feedback! I did changes. I created special method for getting custom_attributes from MiqExpression (https://github.com/ManageIQ/manageiq/pull/9807/commits/e63c6bd548dae4e53e094fb8a0485b76078e77fb)
Custom Attributes columns can be only in 'field' type of filtering(https://github.com/ManageIQ/manageiq/pull/9807/commits/e63c6bd548dae4e53e094fb8a0485b76078e77fb#diff-ff5e4f0f23adaff0ec06a295cd2376f8R1573) in MiqExpression.
Also I change regex in Field class
https://github.com/ManageIQ/manageiq/pull/9807/files#diff-b493717a1e4beaf06a12b8cf90b463eaR5
because custom attributes can contain capital letters in the middle.
Tested on for example
https://gist.github.com/lpichler/f0f335c17faa25a305db5e5e15d91467

@lpichler LGTM, barring some mostly minor comments

LGTM

@miq-bot remove_label wip


setup.py: add data_files to fix #29

This pull request fixes my problem (#29) but that's only one problem preventing the Travis CI build from passing. I think the other (at least one) problem is unrelated and harmless. See:
https://travis-ci.org/jminardi/mecode/jobs/60706173
Although it causes the test to fail (under python 3.3.5), the difference between these lines is insignificant:
G1 E0.000000 Z10.000000
G1 Z10.000000 E0.000000

Thanks for the fix


Milestone: Release version I of web application

Changes in this pull request

In this pull request I

Added a homepage that shows a list of categories that can be filtered out by some parameter.
Displayed numeric values from an API along with the category values
Added an onClick event when a user clicks on a given category
Added a details page when a user clicks on a particular country
Added unit tests for the application components
Added a go back to home page button when a user clicks on the < icon on the navigation menu

Demo video link

Please watch the video for more information about the project

Demo video link >>>>

Hi @omarsalem7
Thank you for the prompt response


feat: smaller bundle size using crypto-lite in browser

The promised PR. I tested it by just temporarily changing './hmac' to './hmac-browser' in lib/index.js, followed by the default 'npm test'.

Bundle size (min + gzip):
before: 95.5 kB
after: 10.7 kB

That’s great, thanks @marten-de-vries! As we make this compatible with browser, can we run tests in browsers, too? I'll make a follow up issue for that


Only import Azure opencensus when needed

This saves around 0.7 s when doing a fresh import without activating logging

bors merge


Pull Model Command

Changes

Adds the /pull-model command to be used to pull models into the ollama local library.

This does not mean the model can be selected yet. The command for "switching" preferred models needs to me implemented first.

@kevinthedang you sire have failed a test!

Test failure found in pullModel.ts.
import { ollama } from "../client.js";

Will have to figure out passing this in a async way to allow pass through.

Will reference #124 as the Pull command is complete.


feat: Store keys upon put

Description
This PR should unblock liftedinit/roadmap#17 by populating a list of keys that can be fetched as the user writes them.

Related Issue

Fixes https://github.com/liftedinit/roadmap/issues/17 (But doesn't really)
Testing

All tests pass.
Breaking Changes (if applicable)

None.
Screenshots (if applicable)

Checklist:

[x] I have read and followed the CONTRIBUTING guidelines for this project.
[x] I have added or updated tests and they pass.
[x] I have added or updated documentation and it is accurate.
[x] I have noted any breaking changes in this module or downstream modules.

Hmm... CI failed. Investigating.

Working now.

Oh, this is already merged. Okay.
I think you'll have to copy the changes from https://github.com/stanleyjones/gwen/pull/1 into your own branch then.


Update telegram-bot.js

call internalGetUpdates in callback
check before reassign _updatesOffset

Refer to 0.3.0 as an improvement of proposed fix.


[BUG] EventHubs prefetch is over-prefetching

My setup has the following config:

prefetch: 1000
partitions: 32
batch size 10

If I run EHProcessor with:

these parameters
relatively long processing callback
tons of messages in the hub
in a single process

I'd see enormous memory usage.
Memory dump shows the following:

there are 32 reactor.util.concurrent.SpscArrayQueues (reactor queue created for each partition)
each queue has capacity of 1024 (prefetch size)
each element of the queue is an Array list of 10 (batch size) elements containing PartitionEvent
each event takes ~1.3 kB on the heap

So, the actual prefetch number goes up to prefetch * partitions * batch_size and in my case ends up being 320k events (!) or ~400 MB.
I think the problem is on line 212 below (.publishOn(scheduler, false, prefetch)) which sets prefetch on fluxes representing batched and not messages
https://github.com/Azure/azure-sdk-for-java/blob/c2f6e5ca010a561a20b7a2ccbfe98b94545ae3b1/sdk/eventhubs/azure-messaging-eventhubs/src/main/java/com/azure/messaging/eventhubs/PartitionPumpManager.java#L204-L216
But also, we need to:

document that prefetch size is per partition
check against track1 SDK, what's done there

changing publishOn(scheduler, false, prefetch) to publishOn(scheduler, false, prefetch / batchSize) predictably results in x10 decrease in number of items in the queue and overall memory usage

Another over-prefetching problem:
https://github.com/Azure/azure-sdk-for-java/blob/c2f6e5ca010a561a20b7a2ccbfe98b94545ae3b1/sdk/eventhubs/azure-messaging-eventhubs/src/main/java/com/azure/messaging/eventhubs/PartitionPumpManager.java#L210-L211
.concatMap(Flux::collectList)  is prefetching 32 lists by default.
As a result, I get
32 partitions x 32 default  prefetch x 128 events x event size in memory
should be changed to .concatMap(Flux::collectList, 1)

Related: https://github.com/Azure/azure-sdk-for-java/issues/33341


verify that RandomData_Benchmark_Small.flow example works

for the refactor-routines branch

found issue getModel pca | Uncaught TypeError: Cannot read property 'flow' of undefined https://github.com/micahstubbs/drift/issues/60

these all work now 🎉


API Enhancement

🎯 Title and Issue Number

Closes #<#37>

🗒 Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x]  I have gone through rules of when PR will get merged.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have followed proper file structure.
[x] I have gone through CONTRIBUTING GUIDELINES.md.

Thank you for contributing!❤
Don't forget to show some love by ⭐ the repository!

Hey @Vibgitcode27, Thank you for creating PR! We will review it super soon. 😇❣
As we continue to evolve and innovate, we encourage you to share your ideas, feedback, and code to help us push the boundaries of what's possible in the world of APIs🚀.
We hope you have followed CONTRIBUTING GUIDELINES.
Thank you for being an essential part of our community. Together, we'll keep building amazing things!
Don’t forget to ⭐ our repository!
Happy Coding!✨
Will See you soon❣

@Vibgitcode27 We're thrilled to inform you that your pull request has been successfully merged into the APIverse🚀🎁
Your hard work, dedication, and expertise have made a significant impact on our project, and we couldn't be more grateful for your contributions.😇
Your code will now be a part of our growing ecosystem, helping developers worldwide to create exceptional APIs.❣🔥
Happy Coding! ✨
Will See you soon❣


Migration tool erroneously null-checks variable whose type is a type parameter

Attempting to migrate the following code:
T foo<T>(T Function() fun) {
  T x = null;
  x = fun();
  return x;
}

int bar(int Function() fun) {
  int x = null;
  x = fun();
  return x;
}

Produces the result:
T  foo<T>(T  Function()  fun) {
  T? x = null;
  x = fun();
  return x!;
}

int  bar(int  Function()  fun) {
  int? x = null;
  x = fun();
  return x;
}

The null check added in foo is bad for two reasons: (1) it's unnecessary (just returning x would be fine), and (2) it changes the semantics of the code being migrated (consider calling foo<dynamic>(() => null)).

As of 1c7fe7196dd7a542f7f91cd6ea2ac81e1a13146c, the null safety migration tool has been removed from active development and retired. No further work on the tool is planned.
If you still need help, or you believe this issue has been closed in error, please feel free to reopen.


Remoting/3.5

Changes summary:

[x] JENKINS-39922 - Update the Jenkins agent executable to the latest version

Remoting 3.x is still compatible with old Jenkins masters working on Java 7 or above (1.610 +)
This version introduces the new encrypted (JNLP4 protocol](https://github.com/jenkinsci/remoting/blob/master/docs/protocols.md#jnlp4-connect)
Picked many bugfixes
Full Changelog

[x] Update the image metadata a bit

Compatibility notes
Java 6 support is dropped in Remoting 3.x. If somebody uses Docker agents with Java 6 masters, we could create another image with Remoting 2.x stable.
@reviewbybees

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@ndeloof alpine has not been updated

so my preference for build args and a CI service to update docker image :P


[Support]: How to rerun a Task?

Support Request
Hi,
I am creating a tool that manages multiple processes. These processes move back-and-forth between 3 states: inactive, building and active. I would like to use listr2 to provide a nice overview of the processes and their changing states.
I would like it to look something like this:
⠧ Process 1 building
✔ Process 2 active
✖ Process 3 inactive

Later when Process 1 becomes active, I would like the output to change to:
✔ Process 1 active
✔ Process 2 active
✖ Process 3 inactive

And yet later when Process 3 is building, I would like the output to change to:
✔ Process 1 active
✔ Process 2 active
⠧ Process 3 building

So what I am trying to do is rerun tasks as the states of the processes change. Is such a thing possible with listr2? Or is it not meant to be used in such a way?
Things I have tried

tasks.runTask(tasks.tasks[0])
tasks.tasks[0].run();

^ Both of these did not work as expected.
Any help would be greatly appreciated! :D
Preflight

[X] Read/Search existing issues and the provided documentation.

Hey @marnixhoh,
Very interesting use case indeed. I do not think that there is currently a mechanism to rerun a given task since the tasks are marked as finalized after either they succeed, fail, or are skipped.
This can be achieved in a different manner though by adding tasks to the current running list by utilizing the Manager, but I have to add a new function for this to exactly work.
As I understand it initially you want to run all three tasks at the same time concurrently and depending on the outcome of a certain task let's say Process 3 you want to go ahead and re-run Process 1 again. If you can elaborate on that a little bit or whether my assumption is correct, I can try my best to help.
This can currently be achieved in some form like this where you can add dynamic tasks from a certain task. So you have to define your tasks outside of Listr as constants and run them in parallel first.
A mock code that is not valid and I have not tested:
const ctx = { task1Finished: new Promise() }
const manager = new Manager({ ctx })

const task1 =  {
    title: "Process 1 inactive",
    task: (ctx, task) => {
      task.title = "Process 1 building"
      task.title = "Process 1 active"

      ctx.task1Finished.resolve()
      // you have to also handle the failing case since if task 3 waits for the promise then it would be stuck
    }
}

manager.add([
  task1,
  {
    title: "Process 2 inactive",
    task: () => {
      task.title = "Process 2 building"
      task.title = "Process 2 active"
    }
  },
  {
    title: "Process 3 inactive",
    task: async (ctx) => {
      // if you need task 1 for this as i have understood
      await ctx.task1Finished

      task.title = "Process 3 building"
      task.title = "Process 3 active"

      manager.add([
          task1
      ])
    }
])

// i can add a exhaust method to `Manager` to do this while loop, because i will also need it soon
while (manager.tasks.length > 0) {
  await manager.runAll()
}

Visually I do not think it will be the same but yeah, I have intentionally disabled touching the task after it is finished due to some resource optimizations so the thing you might be asking for is in my eyes (if i understood correctly) possible.

Thank you so much for your incredibly quick reply! 😄
To elaborate a little bit on my use case:
I am working on a server, which manages processes. Ideally, the server would be able to send events to Listr. For example, when the server finishes building process 1, it would then send an event to Listr informing it that process 1 should now be marked as "completed". In other words, it would be great if the code could be running "outside" of Listr and push events to Listr as a way to visually inform the user.
The code you provided adds additional tasks right? As in it does not replace existing ones? If I'm correct here, then it would definitely be better than my current setup (which is just console.logging every status change 😆 ), but ideally I would like to "update" each task instead.
Again, thank you so much for your help here. It really does mean a lot. And even if Listr might not be the right tool to achieve this, I do want to let you know that it's a really cool and amazing tool to use :) Thanks a lot for maintaining it!

No worries, thanks for the kind words.
Yes, the example was mostly about adding additional tasks while running. I mean it sounds different than I have understood it but, if you can manage to do this kind of state management in plain JavaScript then you might use Listr as the display tool.
RxJS kind of magic or promise-wrapping an event-emitter can also be good here since tasks also accept observables and you can always push data to an observable from anywhere. But I do not think this library alone would be the most perfect tool to handle this job. But showing ✖ for a not started task will not be possible in this case. Sorry just throwing in some ideas since I have failed to help in this case.
The key point here would be to not finish/finalize the task until it will not update the displaying status anymore.
You might also achieve this with a custom implementation without this tool with something like log-update or stdout-update from npm.


Could not find keycloak import file

I see a message in the log file:
2020-05-20 08:34:43.581  INFO 4056 --- [           main] c.g.t.k.e.EmbeddedKeycloakApplication    : Could not find keycloak import file ServletContext resource [/keycloak-realm-config.json]
The realm file is declared in application.yml of embedded-keycloak-server-plain module.  But there is no such a file in sources.

This is optional and only serves for demonstration purposes. You can specify a file via the keycloak.custom.migration.importLocation property to import an existing realm.
I could add it to the -custom example module though.

@thomasdarimont is there any example of how specifying realms directory to be imported?
I add the realm directory under the resources and specifying the properties as follows:
    migration:
      importProvider: dir
      importLocation: realms

but yet the directory is not found as it tries to find it under this base path
/tmp/undertow-docbase.3416615456268389963.8080/realms

I have put keycloak-realm-config.json in META-INF but i still get error Could not find keycloak import file ServletContext resource [/META-INF//keycloak-realm-config.json]

I have put keycloak-realm-config.json in META-INF but i still get error Could not find keycloak import file ServletContext resource [/META-INF//keycloak-realm-config.json]

i've specified it as classpath resource and put t in src/main/resources
keycloak.custom.migration.import-location=classpath:/keycloak-realm-config.json

and check if the resources block in the maven pom build section (if present) includes json files

Putting a realm JSON file in the classpath doesn't even work for me:
java.io.FileNotFoundException: class path resource [keycloak-realm-config.json] cannot be resolved to absolute file path because it does not reside in the file system: jar:file:/C:/Users/user/restmock/target/restmock-undertow-0.0.1-SNAPSHOT.jar!/BOOT-INF/classes!/keycloak-realm-config.json
        at org.springframework.util.ResourceUtils.getFile(ResourceUtils.java:217) ~[spring-core-5.3.9.jar!/:5.3.9]
        at org.springframework.core.io.AbstractFileResolvingResource.getFile(AbstractFileResolvingResource.java:154) ~[spring-core-5.3.9.jar!/:5.3.9]
        at com.github.thomasdarimont.keycloak.embedded.EmbeddedKeycloakApplication.tryImportRealm(EmbeddedKeycloakApplication.java:102) ~[embedded-keycloak-server-spring-boot-support-5.0.2.jar!/:5.0.2]

I think this is caused due to the following code, which treats the resource as if it was a file on the filesystem:
https://github.com/thomasdarimont/embedded-spring-boot-keycloak-server/blob/fc9c610f27b784dc76300a622f0b2840264ec5f3/embedded-keycloak-server-spring-boot-support/src/main/java/com/github/thomasdarimont/keycloak/embedded/EmbeddedKeycloakApplication.java#L100-L114
It'd be nice to let Keycloak read the file from an input stream and not depend on concrete details on how it is stored.

In my case setting import-location to file:keycloak-realm-config.json worked. The JSON file was read from the current working directory (i.e. where I ran java -jar). It looks like Spring Boot interprets resources that do not start with that prefix as web resources. I'm using the latest version of this project at the time of writing, 5.0.2.
Interestingly, when I launched my application during Spring integration tests, the classpath resource approach worked. I think that's because in that case the classpath is composed of actual files in the filesystem, and are not yet in an archive.
It also looks like the Keycloak implementations of the ImportProvider interface don't really support classpath resources. For example, the single file import provider requires a File for construction. It also doesn't look like there is a supported and documented way to register your own provider with Keycloak.


Fix AttributeError: 'TzktDatasource' object has no attribute 'block'

Hey Dipdup team, I was upgrading to v3 and kept getting AttributeError: 'TzktDatasource' object has no attribute 'block' when the bigmap indexes switched to realtime.
After this small fix everything worked fine.

Thank you! ✨
This kind of stupid typos should never pass CI, but unfortunately Pydantic isn't compatible with static analyzers yet. Waiting for a new version with this pull request.


Add support for S3 in spark-hdfs

Fixes https://github.com/quasar-analytics/quasar/issues/2264

Needs a semver label

@drostron now, question, should I consider this a new feature? I guess so...

@rabbitonweb I'm amenable to version: feature

@drostron done


2.0 in template

:v:
/cc @zendesk/vegemite
Description
updated template for iframe only
References

JIRA:

Risks

+1

:+1:

PHIL HAPPY NOW?


SD Card Corrupted. Reinstalled Retronas how to add Existing Drive back in?

My SD Card became corrupted so it would no longer boot to Retronas. I have re-installed it but I do not want to lose the original Hard Drive I had setup.
How do I "Add" that Hard Drive back in so Retronas will use it WITHOUT it creating a new Drive and erasing the contents?

Run the RetroNAS installer, then install Cockpit. You can then use Cockpit to mount the drive without the partitioning/formatting steps (you should see the partition and file system still on the disk).
Once mounted, set the mount path in RetroNAS and rerun your service installers. These will just install and configure the services. No data will be overwritten.

Perfect thanks dan. The issue I had in the end was because of the USB to SATA Adapter which I have seen before. It reports back that the drive has no partitions on it :) I need to throw that Adapter away as its caught me out once before.
Changed the USB Adapter for another one and now I can see the original EXT4 partition and could mount it as /storage again and everything is happy (MiSTer CIFS, PS2SMB etc).

I think this should feed some basic troubleshooting/recovery doco in the wiki before it is closed, there is a possibility that more people my run into this situation


multiprocessing pool issue on windows stderr (hashtag731)

hi, I got an issue that message from different processes got printed in the same line when I was using in my project.
here's the simplified structure of my project helping to reproduce the issue.
PS. I ran it in Python 3.8.0
-- app.py
-- logissue
|--init.py
|--engine.py
|--strategy.py
init.py
from loguru import logger
import sys

logger.remove()
logger.add(
    sys.stderr,
    colorize=True,
    format="<green>{time:YYYY-MM-DD HH:mm:ss}</green> {process} [{level}] {name} - <level>{message}</level>",
    level='DEBUG', enqueue=True)

engine.py
from loguru import logger
from logissue.strategy import Strategy
from multiprocessing import Pool
import time

class Engine(object):

    def __init__(self) -> None:
        self.strategies = []

    def add_strategy(self, strategy: Strategy):
        self.strategies.append(strategy)

    def execute(self, process_count=None):
        start_time = time.perf_counter()
        args_list = []
        for i in range(1, 10):
            for strategy in self.strategies:
                args_list.append((strategy, i))

        p = Pool(process_count)
        p.map(self._execute_in_process, args_list)
        p.close()
        p.join()
        end_time = time.perf_counter()
        logger.info('time elapsed: {elapsed} seconds', elapsed=end_time - start_time)

    def _execute_in_process(self, args):
        strategy, idx = args

        result = strategy.do_sth_slowly(idx)
        logger.info('result: {r}', r=result)

strategy.py
import time
from loguru import logger

class Strategy(object):

    def do_sth_slowly(self, i):
        time.sleep(0.2)
        logger.info("i {i}", i=i)
        return time.time()

app.py
from logissue.engine import Engine
from logissue.strategy import Strategy

engine = Engine()
strategy1 = Strategy()
strategy2 = Strategy()
engine.add_strategy(strategy1)
engine.add_strategy(strategy2)

if __name__ == '__main__':
    engine.execute(process_count=None)

just run app.py in its folder we will get something like this in the console
python app.py

2022-10-23 16:55:222022-10-23 16:55:22 9752 [INFO] logissue.strategy -  16640 [INFO] logissue.strategy - i 1i 1

2022-10-23 16:55:22 9752 [INFO] logissue.engine - 2022-10-23 16:55:22result:<PHONE_NUMBER>.3220634 16640 [INFO] logissue.engine -
result:<PHONE_NUMBER>.3220634
2022-10-23 16:55:222022-10-23 16:55:222022-10-23 16:55:222022-10-23 16:55:22 19428 [INFO] logissue.strategy -  21352 [INFO] logissue.strategy -  14252 [INFO] logissue.strategy -  8700 [INFO] logissue.strategy - 2022-10-23 16:55:22i 2i 2i 4i 3 4768 [INFO] logissue.strategy -

i 32022-10-23 16:55:222022-10-23 16:55:222022-10-23 16:55:22
 19428 [INFO] logissue.engine - 2022-10-23 16:55:22 21352 [INFO] logissue.engine -  14252 [INFO] logissue.engine - result:<PHONE_NUMBER>.33905822022-10-23
16:55:22 8700 [INFO] logissue.engine - result:<PHONE_NUMBER>.3390582result:<PHONE_NUMBER>.3390582
 4768 [INFO] logissue.engine - result:<PHONE_NUMBER>.3390582result:<PHONE_NUMBER>.340062

2022-10-23 16:55:22 21276 [INFO] logissue.strategy - i 4
2022-10-23 16:55:22 21276 [INFO] logissue.engine - result:<PHONE_NUMBER>.354059
2022-10-23 16:55:222022-10-23 16:55:22 9752 [INFO] logissue.strategy -  16640 [INFO] logissue.strategy - i 5i 5

2022-10-23 16:55:22 9752 [INFO] logissue.engine - 2022-10-23 16:55:22result:<PHONE_NUMBER>.526792 16640 [INFO] logissue.engine -
result:<PHONE_NUMBER>.526792
2022-10-23 16:55:22 21352 [INFO] logissue.strategy - i 6
2022-10-23 16:55:22 21352 [INFO] logissue.engine - result:<PHONE_NUMBER>.5427985
2022-10-23 16:55:222022-10-23 16:55:22 8700 [INFO] logissue.strategy - 2022-10-23 16:55:22 14252 [INFO] logissue.strategy - 2022-10-23 16:55:22i 7 4768
[INFO] logissue.strategy - i 6 19428 [INFO] logissue.strategy -
i 8
i 7
2022-10-23 16:55:22
2022-10-23 16:55:222022-10-23 16:55:22 8700 [INFO] logissue.engine -  14252 [INFO] logissue.engine - 2022-10-23 16:55:22 4768 [INFO] logissue.engine - result:<PHONE_NUMBER>.5577984result:<PHONE_NUMBER>.5577984 19428 [INFO] logissue.engine - result:<PHONE_NUMBER>.558798

result:<PHONE_NUMBER>.558798

2022-10-23 16:55:22 21276 [INFO] logissue.strategy - i 8
2022-10-23 16:55:22 21276 [INFO] logissue.engine - result:<PHONE_NUMBER>.5747962
2022-10-23 16:55:22 9752 [INFO] logissue.strategy - 2022-10-23 16:55:22i 9 16640 [INFO] logissue.strategy -
2022-10-23 16:55:22i 9 9752 [INFO] logissue.engine -
result:<PHONE_NUMBER>.7432042022-10-23 16:55:22
 16640 [INFO] logissue.engine - result:<PHONE_NUMBER>.743204
2022-10-23 16:55:22 13728 [INFO] logissue.engine - time elapsed: 0.9871249000000001 seconds

Hi.
This might be due to the way Windows spawns child process. See this section of the documentation: [Compatibility with multiprocessing using enqueue argument](
In short, you'll probably need to pass the logger around to the classes and functions that need it instead of simply importing and using it.

thanks, issue solved


Adding Scenario Outline section to Gherkin page and updating tag on SUMMARY.md

Scenario Outline section is taken from https://github.com/cucumber/cucumber/wiki/Scenario-Outlines and pages 74-75 of The Cucumber Book.

Thank you @siwood! I'd like to see more content about Gherkin being merged before the Scenario Outline gets in (I'm thinking about #34).
Let's keep this one open in the meanwhile.

@jbpros #34 seems to be stuck in a bit of a tangled mess. Why don't we merge this one - I think @siwood's copy about scenario outlines is good quality - and then fill out the gaps after that?
I know you don't like scenario outlines @jbpros but don't let your personal biases get in the way! ;)

This is good quality indeed, let's merge it and add gherkin basics.

@jbpros and I reviewed this together today. What we like:

it's original and well-written content
the commits are granular, and clearly commented

Thanks @siwood - we're going to give you commit access, so you can make your contributions directly into this repo in future, as well as help reviewing and merging other people's contributions.

@mattwynne @jbpros Thank you! I am still figuring out git so I may ask some questions along the way.


License?

Hi.  I'd like to use this plugin, but would need to know what the license is.  Have you made a decision yet?

Hi. I've updated the license. https://github.com/danveloper/grails-rest-renderers#license


DEX-679: tests to confirm time/timeSpecificity in various /users/xxx api calls

I don't think this PR needs to be draft.  The jira ticket is already marked as "done" and this PR just adds extra asserts to make sure /api/v1/users/<user-guid>/asset_group_sightings and /api/v1/users/<user-guid>/sightings contain the time fields.


Update GitHub workflows

Before you start

[x] Have you checked if a similar PR has already been requested?
[x] Have you built and ran the app?

Related/Fixed issues
Address related/fixed issues

Related: None
Fixed: None

Description
Details of changes are listed here
As the title
Motivation and Context
Items resolved/related issues by this PR.
As the title
Assets (if appropriate):
A list of assets (screenshots, mockups) relevant to this pull request.
None

maybe beed to delete both.


Datepicker appears half in the screen and half out of the screen

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Datepicker should calculate the height of the screen and decide to appear on the bottom or to the top of the field
What is the current behavior?
It always appear at the bottom of the field and if the field is at the edge of the window the datepicker goes out of the screen
What are the steps to reproduce?
scroll to a field that is almost at the edge of the bottom of the window and open the datepicker

Looked into it a bit and there are a couple of issues:

We haven't set up the fallback positions for the datepicker.
Even if we had set up the fallback position, the ConnectedPositionStrategy measures the element too early and won't determine the proper positioning.

dupe of https://github.com/angular/material2/issues/4406

Closing as a dupe of #4406.


migrate to python 3

Are there any parts that are not currently python 3 compatible? I know we are fully python2 compatible, and that the core interface is python 3 compatible, but are there known issues with any other parts?

I just don't know about the hardware libraries.
@ksunden if you agree, I move that we close this issue and call the package python 3 ready
The hardware libraries can be addressed on a case-by-case basis.

I agree, new issues should be created if there are any python2 specific code in particular portions


How often / when do we meet?

Currently we have a weekly planned for 2:00 pm EDT. We could do alternating times to be more inclusive of those in different time zones. We could do a doodle, but there are way too many folks to really figure this out, so I'll make a proposal.
Alternating between 2:00pm EDT and 9:00am EDT
This goes early enough that those in EU will likely be able to take the meeting one of the meeting within business hours and not making those on the west coast get up far too early (although 6am is kind of gross).

+1 from me.

What motivates the 9am time?  Currently that's 2pm in Ireland.  Going to 10am (or perhaps even 11am) would make things easier for west coast US.

TSC current does 8am, 12pm, 5pm EDT
9am seemed to offer a good afternoon time for EU and a time for China and Japan in the evening (9pm, 10pm), midnight for Australia. While 8 am would serve Pacific region a bit better it seemed dar to early for West coast.
For those in EMEA and APAC do you have a preference? If no one speaks up perhaps we simply bump to 10am EDT and then iterate?

The 2pm Eastern time is rather early in the morning for folks who may want to join from parts of Asia or Australia. Perhaps the same schedule as currently used for the TSC would work?

@jasnell the 2pm EDT time is preferred for the moment as one of the times as a large number of board members from both the JSF and the Node.js foundation have it marked off on their calendars already.
@rubys I am waiting for us to reach some sort of consensus about an earlier time or get the go ahead from @dshaw or someone else from the individual membership group that we can replace their time slot... neither has happened yet

Let's call the meeting for next week at 9am, if we have issues with people joining we can discuss and iterate. Expect a meeting issue to be opened v soon

FWIW, 9am ET/6am PT on Mondays will generally not work for me do to a standing conflict.
Perhaps rather than selecting a time like this, we follow a similar kind of process that was used to select the TSC call times... that is, provide a google speradsheet where folks can indicate their generalized availability through the typical week and we iterate from there... and in the meantime, keep the 2pm ET call next week as was said on the previous call.

@jasnell I've been informed by @dshaw that he can move the other meeting, so unless anyone objects lets move the meeting back to 2pm. Sorry about the back and forth, we have time on the agenda to discuss the timing lets lock down something during this meeting.

So we are aiming for 2pm ET on Monday, November 5. I want to make sure get this shared on social.

We also meet to add the meeting on the node.js calendar as I don't thin kit is there yet.

Meeting times have been added to the calendar. We are indeed aiming for 2pm ET on Monday

We had consensus in the last meeting to stick with the 9am and 2pm ET times for now.
It was brought up that a member of the LF team who is going to support us will be unable to joint for the 2pm ET time, are there any objection to moving that time to 1pm ET? Alternative times to suggest?

9am ET is perfect for me, while 2pm ET is unfeasible.
Unfortunately today I have not been able to join because of the Node.js security release work.

Agreement in today's meeting to keep the current alternating times.


Support ASON

Hey!
I couldn't run the aspect tests until I ran prettier and updated the as-pect version. This pull request is a lot bigger than I intended it to be. Sorry for the trouble.

For some reason, the tests don't pass locally, despite empty.wat passing diff? I could probably use some assistance.
I also recommend keeping up to date with the latest compiler.

Hey @ColinEberhardt I was thinking that we could have a little pow-wow about this software and see where it takes us. I would love to help update this software to the latest version of ASON and as-pect.


Drop country middleware

This PR drops the country middleware (saleor.core.middleware.country), which was using GeoIP to determine the user's country. In the new approach, user location is provided as an argument in queries or mutations that need it for tax or stock calculations.
Todo:

[x] update test_variant_availability.py tests to use new address parameter
[ ] add tests for queries that use the new param
[ ] deprecate CheckoutPaymentCreate.billingAddress input on 2.11?

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-1064 Add "country" input field in storefront's product queries (drop GeoIP)

Task linked: SALEOR-1064 Add "country" input field in storefront's product queries (drop GeoIP)

PR is currently blocked by storefront/SDK which currently uses the billingAddress field that this PR removes from API.

PR is currently blocked by storefront/SDK which currently uses the billingAddress field that this PR removes from API.


Remove the Scale Schedule Algorithm

The algorithm form of scale schedule has been deprecated. It is available as an argument to the trainer.
Closes #434.

This also removes the README about scale schedule, which I think it an integral part. Perhaps we leave it deprecated, but existing in the codebase, for now until we have a better path.

This also removes the README about scale schedule, which I think it an integral part. Perhaps we leave it deprecated, but existing in the codebase, for now until we have a better path.

The README for the scale schedule is a stub. It currently read:

🗜 Scale Schedule
This method is deprecated. It has been replaced by the scale_schedule_ratio param supported directly by the Composer Trainer. For backwards compatibility, the Composer Trainer detects when this algorithm has been initialized and pulls the ratio param accordingly.

We seemed to have lost the original method card for scale schedule (https://docs.mosaicml.com/en/v0.3.1/method_cards/scale_schedule.html). Would it make sense to add this documentation back under the "Using The Trainer", since this flag is now under the trainer args?

We have docs for the scale schedule ratio here: https://docs.mosaicml.com/en/v0.5.0/trainer/schedulers.html#scale-schedule-ratio

Hmm defer to @jfrankle on visibility concerns of scale schedule here, but would it be possible to keep the Scale Schedule stub in the README, and point it to the appropriate documentation?
It's one of our core methdology, so want to make sure it's not buried somewhere.

I'd like to keep the stub so that, when people go looking for it, they get redirected to the right place. It should be as visible as possible. Ideally, the stub will produce an error if used so that people can correct their behavior.

Finally got around to revising the docs. I took the method card we had in the 0.3.1 release and modified it to reflect the new implementation. Could you please re-review? Thanks!


Running on languages other than C#?

Hi,
Thanks for releasing this amazing library!
I am trying to build this library on top of java codebase, i wonder if there is any high level guideline to run the code on languages other than C#.
For example, do I need to redo everything in ExpressionDataExtractor and SourceGraphExtractionUtils for java to allow the python models in Models/ to take java code as the legitimate input?
Or I can generate my own java AST (and probably my own meta-date?), and use this as the input to the python models in Models/?
Thanks!

Heya,
Thanks for reaching out. Currently, there is no support for any language than C#. As you observe, you roughly need to replace the ExpressionDataExtractor and the SourceGraphExtractionUtils to generate similar data. There are a number of partial solutions for this for other languages (e.g., https://github.com/acr31/features-javac implements graph extraction for Java) that you could possible re-use.
Marc


Resolve layout partials without fully compiling

I'm using express handlebars mainly on server architecture, but would also like to use its partial & helper registration in serverless architecture. Unfortunately, the resources available in AWS Lambda aren't enough to scan all dependency directories to find partials, and then compile and render the template.
The layout used by my express apps requires many partials from dependencies. What I would really like to do is to pre-resolve these partials, and generate a new template which can be used in Lambda, so that Lambda doesn't have to resolve the partials for the layout, and can concern itself just with the partials needed for the body of the page.
i.e
layout.html
<html>
{{>my-dependency/partial}}
<body>
{{body}}
</body>
</html>

my-dependency/partial.html
<head>
 <link ...>
</head>

desired output
<html>
<head>
 <link ...>
</head>
<body>
{{body}}
</body>
</html>

Can you advise if there's some way I could hack in to the template resolution & compilation to achieve this?

Would precompiled: true be the answer?


SNOW-1635365: Update snowpark pandas imports in method documentation

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

Fixes SNOW-1635365 (resolves #2139)

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.

Please describe how your code solves the related issue.

A few methods in documentation had an outdated import method for Snowpark pandas (import snowflake.snowpark.modin.pandas as pd). This PR updates docstrings to use the correct method (import modin.pandas as pd; import snowflake.snowpark.modin.plugin).

FYI @sfc-gh-azhan @sfc-gh-nkrishna I had to make an additional change to the default argument of the typ flag of pd.read_json to get doctests to pass. CI was failing on src/snowflake/snowpark/modin/plugin/extensions/pd_overrides.py::snowpark.modin.plugin.extensions.pd_overrides.read_json, but the failure occurred only if the full doctest suite was run (not sure if there's a smaller subset that can trigger it). I suspect there might be some hidden conflicts between the read_* methods we define in snowflake/snowpark/modin/pandas/io.py and those we define in snowflake/snowpark/modin/plugin/extensions/pd_overrides.py; I'll investigate further in a separate PR if I have time before GA.


Configure a logging handler to send error emails to admins

Will reconsider this at a later point if site scales to make this necessary.


[All] Materialv2 breaks app theme

Current behavior
When migrating from material v1 to v2 the theme of the app is broken
Dark :

Light :

It happens in multiple apps using uno and it is reproductible in the uno app template who was migrated to v2 recently
https://github.com/nventive/UnoApplicationTemplate/commit/a162cab04ec1429d067c013fba31a7e11f9d36f1
Expected behavior
The theme should switch from dark to light like with materialv1
How to reproduce it (as minimally and precisely as possible)

Install the uno app template
Pull the latest version of the main branch
Deploy it on any platform
Go to any page (create account page is the most noticable)
switch from light to dark and see that the theme is broken

Workaround
None found
Works on UWP/WinUI
No
Environment
Uno.WinUI / Uno.WinUI.WebAssembly / Uno.WinUI.Skia, Uno.UI.RemoteControl / Uno.WinUI.RemoteControl
NuGet package version(s)
4.7.44
Affected platforms
Android, iOS, Windows
IDE
Visual Studio 2019
IDE version
17.6.2
Relevant plugins
No response
Anything else we need to know?
No response

Anything change if you change this line to be x:Key="Default" ?

@Lee31416 I found the main issue with the code you are using. Starting with Materialv2, themeResources need to be called ThemeResources in the style. I made this branch where I fixed the problem on the login page of the applicationtemplate, https://github.com/nventive/UnoApplicationTemplate/tree/dev/maso/workswiththemes to give an idea of what you will need to do.

@Lee31416 let me know if this can be closed with the changes that @Marc-Antoine-Soucy suggested

@kazo0 I think this can be closed because the change marc suggested works


[mailbox] add workaround for pagination when using keyword_to_search

Code changes

[x] offset and limit query params added to fetchSearchResultThreads
[x] num_page_visible prop  added to PaginationNav.svelte

Screenshot
Notice that requests are fetched only when new threads are requested. The example below has a items_per_page of 3.
https://user-images.githubusercontent.com/34139730/148395656-ac5d0880-f52e-4d43-82b4-96c97749fdd4.mov
Readiness checklist

[x] Added changes to component CHANGELOG.md

License
I confirm that this contribution is made under the terms of the MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Hey @krisgardiner what about the first/last buttons? Shouldn't they be hidden if keyword search is enabled?

@AaronDDM you can technically still use them once the the messages are cached.


read path WIP

cmd:
./util/dvsim/dvsim.py ./hw/ip/i2c/dv/i2c_sim_cfg.hjson -i target_smoke -r 1 -v m -s 1 -w fsdb
Test sends 1 read transaction as follows: (Rcv comes from dut)
S(start) - Address - Rcv[Ack] - Rcv[rdata] - Ack - Rcv[rdata] - Nack - P(stop)
0x32                                 0xA4                   0x92
After sending address and receiving Ack from dut, receive read data doesn't work.

hw/ip/i2c/rtl/i2c_fsm.sv is not the same as HEAD. It is locally updated version after write trial.


Configurable Script Timeout

Is your feature request related to a problem? Please describe.
In the big Software Factory package, the Atlassian suite and Jenkins are installed as separate components after Big Bang deploys, but they have to wait until Istio is running or they won't get sidecar injections. I have this in those components, so it will wait for Istio to be running before trying to deploy:
    scripts:
      retry: true
      before:
        # Check to see if istiod is running before trying to deploy. If Istio is running then so is Gatekeeper. This is a poor man's version of "DependsOn" since we aren't doing real gitops yet
        - |
          test $(/usr/sbin/kubectl get pods -n istio-system -l app=istiod --field-selector=status.phase=Running -o json | ./jq -r '.items | length') -gt 0

The problem is that it can sometimes take a few minutes for Istio to be running and Zarf times it out at 2 minutes 60 retries with a 2 second wait between tries which roughly equates to 2 minutes if you don't count actual processing time.
Describe the solution you'd like
I'd like the ability to configure the timeout, instead of it being hardcoded to 2 minutes.
Describe alternatives you've considered
I'm not really aware of any that would let me do this all in one package, automatically, without doing something super hacky with initcontainers or something.

@RothAndrew this is useful feature, but I think you said you aren't blocked anymore on this so less urgent for you, is that correct?

yes, correct. I think it is still a worthwhile addition but not a priority.

copy and greed, thanks.


Is it possible to hide the indicator?

I have a long ItemList, and I am using a "cards:CarouselView"
<cards:CarouselView
Items="{Binding Items}"
SlideShowDuration="0"
CurrentIndex="{Binding CurrentIndex}">
And I would like to know if it is possible to hide the indicador, or show a simple page indicator: "showing 3 of 2500 records" style
thanks

Yes, sure, just remove Indicators control from your xaml :)
Find "IndicatorsControl" and delete it.
Hope it's clear )


OpenSSL::Digest::Digest -> OpenSSL::Digest

No more deprecation warnings.

@dpickett - Have you had any chance to look at this?

thank you for the ping! this fell off my radar. Merging.

Thanks Dan, you rock!


import a pre-populated SQLite database from application for iOS an Android

Is there any way to import a prepopulated database for iOS and Android like we have in this plugin
https://github.com/andpor/react-native-sqlite-storage#importing-a-pre-populated-database

Duplicate of https://github.com/craftzdog/react-native-sqlite-2/issues/54


Fixed typo (Aure)

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

Correction confirmed. Thanks.


Resolved Issue#488

Description
The fix for Issue#488. Updated the RVTEST_ISA string for fnmadd.d_b15, fmadd.d_b15, fnmsub.d_b15, fmsub.d_b15 tests to stop running it on RV32F/RV64F. These tests will only run for RV32FD/RV64FD now.
Related Issues
Issue#488
Ratified/Unratified Extensions

[x] Ratified
[ ] Unratified

PR #489 was already added to resolve the issue, this one is the duplicate version of it. Closing this one.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4351

Questions ?
Please refer to the Backport tool documentation


Need better error when debugger save in block context

When you are in a block context in the debugger, and edit the method and want to save, it doesn't work.  This annoys me a lot in GBS but I accept this.  However, the walkback in Oscar (3.0.26) is not helpful.
a ArgumentTypeError occurred (error 2094), reason:rtErrBadArgKind, expected a CharacterCollection
ArgumentTypeError (AbstractException) >> _signalWith: @6 line 25
ArgumentTypeError (AbstractException) >> signal @2 line 47
UndefinedObject (Object) >> _error:args: @15 line 11
UndefinedObject (Object) >> _errorExpectedClass: @2 line 5
UndefinedObject (Object) >> _validateClass: @4 line 6
Symbol class >> _existingWithAll: @2 line 10
SymbolList >> resolveSymbol: @4 line 9
SymbolList >> objectNamed: @2 line 8
RwGsImage class >> objectNamed: @3 line 6
RwGsPlatform >> globalNamed: @3 line 5
Rowan class >> globalNamed: @3 line 5
RowanClassService >> classFromName @2 line 3
RowanClassService >> packageIsDirty @2 line 4
RowanClassService >> arePackageAndProjectClean @2 line 3
RowanClassService >> servicePerform:withArguments: @2 line 3
[] in  ExecBlock1 (JadeServer) >> updateFromSton: @10 line 12
Array (Collection) >> do: @5 line 10
JadeServer64bit32 (JadeServer) >> updateFromSton: @7 line 6
GsNMethod class >> _gsReturnToC @1 line 1

Added a message in the debugger warning that the user can't save in an exec block.
Fixed in 3.0.28.


Inverse colors

Hi,
When trying samples my color are inversed (red is cyan, black is white...)
In my app, I have to inverse all color bits to get the right colors :)

Hi,
Please modify the SPFD5408_Adafruit_TFTLCD line 307
before:
// *** SPFD5408 change -- Begin
// Not tested yet
//writeRegister8(ILI9341_INVERTOFF, 0);
//delay(500);

after:
// *** SPFD5408 change -- Begin
// Not tested yet
writeRegister8(ILI9341_INVERTOFF, 0);
//delay(500);

Im getting 2 half screens. Idk how to resolve it.

writeRegister8(ILI9341_INVERTON, 0);
this is commented
//writeRegister8(ILI9341_INVERTOFF, 0);
This fixed the color issue on my LCD

Hello manniakk.
Your TFT controller is NOT SPFD5408.
It's ILI9342.
https://forum.arduino.cc/index.php?topic=473885.0

In my case, pure colors and secondary colors kinda work fine, but the other colors and black always are weird. Like, black looks like purple.

Hello manniakk.
Try this.
https://github.com/prenticedavid/MCUFRIEND_kbv/tree/master/examples/LCD_ID_readreg
You can see your tft driver.


Implement target discovery in own k8s namespace

This commit adds support for discovering targets from the same
Kubernetes namespace as the Prometheus pod itself. Own-namespace
discovery can be indicated by using "." as the namespace.
Fixes #9782
Signed-off-by: fpetkovski<EMAIL_ADDRESS>

Apologies for not commenting on the original issue. I don't think I'm a fan of the magic value, though I agree that it probably shouldn't be mutually exclusive to the name-list. How about we still have the explicit field of includeOwnNamespace or something like that, and essentially concatenate that with the list specified in names.
cc @roidelapluie

Okay, we can have own_namespace: bool: Add current namespace to the namespaces to watch. If namespaces is not set, only the current namespace will be monitored.
I would not call it "include" because the default namespaces include it already (because it does not filter on namespace)

Thanks both for the feedback. I updated the PR with an explicit own_namespace variable instead of relying on a magic value. PTAL again :)

lgtm, giving @roidelapluie last call

Thanks!


Confusion on Alerting for Private DNS.

Regarding the Note at the beginning of this page, these metrics are available for Private DNS via the Metrics Explorer. There are lulls though and is that why we shouldn't use them for alerting? The rest of the document is good but should we assume that the alerts should only be created for Azure Public DNS? Are there alerts that should be created for Azure Private DNS?

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

ID: ec19e272-10a0-5c69-fab7-8b751e1c7626
Version Independent ID: 378b3a12-9e9d-21b0-0a81-346d7c33f5c0
Content: Metrics and alerts - Azure DNS
Content Source: articles/dns/dns-alerts-metrics.md
Service: dns
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

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

@bzabber
Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Hi - the comment doesn't say you shouldn't use them.  It says they aren't available for private DNS zones.  I'll check and see if something has changed. #please-close


Add data schema to stage nodes

StageNode currently doesn't have RowType or DataSchema which is part of the logical plan node result.
Sending it over the wire by encoding the columnNames and columnDataTypes.

Codecov Report

Merging #8985 (0a28da4) into master (09fcdd5) will decrease coverage by 42.96%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #8985       +/-   ##
=============================================
- Coverage     69.69%   26.73%   -42.97%
+ Complexity     4630        1     -4629
=============================================
  Files          1813     1805        -8
  Lines         94521    94538       +17
  Branches      14116    14150       +34
=============================================
- Hits          65879    25273    -40606
- Misses        24033    66882    +42849
+ Partials       4609     2383     -2226

Flag
Coverage Δ

integration1
26.73% <0.00%> (+0.08%)
:arrow_up:

integration2
?

unittests1
?

unittests2
?

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

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


Missing module after updating worker

Hi, I just downloaded the binaries for the latest version of nexrender worker and server and I am struggling to get it to run.
IDK how to explain it a better way ^^', I just use the same command as before.

Which version of nexrender are you using? latest release (1.42.2)
What kind of the setup do you have? server+worker
Are you using pre-built binaries or globally installed npm packages? Binaries
Which kind (and version) of operating system you are running it in? Windows 11
after effects version 23.2

Hi ! I have exactly same error as you since the push of http cache

OK, the 1.41.1 seems to be working well. And the 1.42.0 is causing the bug.

Ah, that would make sense. I will try to focus on updating the embedded binary version of node in next updates

Published v1.42.3, try testing it out: https://github.com/inlife/nexrender/releases/tag/v1.42.3

Okay I'll try, thanks

It seems to work properly, I'm closing the Issue. thank you.


Introduce Effects & Side Effects

The 'read' acton (and accompanying saga) is an example of a generic (in the sense that it can be triggered for every element) action.
Provide a mechanism to declare updates for such generic actions.
Initially, this registration can be done indiscriminately (for all elements); later we can add actions applying to just some elements.

Effects are implemented with #61


xcodebuild fails with "/node_modules/@sentry/cli/sentry-cli: cannot execute binary file"

OS:

[ ]  Windows
[X]  MacOS
[ ]  Linux

Platform:

[X]  iOS
[ ]  Android

Output of node -v && npm -v && npm ls --prod --depth=0
bash-3.2$ node -v && npm -v && npm ls --prod --depth=0
v9.10.0
5.6.0
/Users/distiller
└── (empty)

Config:
<EMAIL_ADDRESS>{
  logLevel: SentryLog.Error
}).install()

I have following issue:
xcodebuild fails during Bundle React Native code and images build phase. The build phase has the following content,
export SENTRY_PROPERTIES=sentry.properties
export NODE_BINARY=node
../node_modules/@sentry/cli/bin/sentry-cli react-native xcode \
    ../node_modules/react-native/scripts/react-native-xcode.sh

The error I get is the following,
[19:25:35]: ▸     export arch=arm64
[19:25:35]: ▸     export variant=normal
[19:25:35]: ▸     /bin/sh -c /Users/distiller/project/fastlane/generated/Build/Intermediates.noindex/ArchiveIntermediates/PoolOfficeManager/IntermediateBuildFilesPath/PoolOfficeManager.build/Release-iphoneos/PoolOfficeManager.build/Script-00DD1BFF1BD5951E006B06BC.sh
[19:25:35]: ▸ /Users/distiller/project/node_modules/@sentry/cli/sentry-cli: /Users/distiller/project/node_modules/@sentry/cli/sentry-cli: cannot execute binary file
[19:25:35]: ▸ Command /bin/sh failed with exit code 126

I am using,

Fastlane 2.78.0
CircleCI 2.0
XCode 9.3
React Native 0.53.0
React Native Sentry 0.36.0

Important Notes

I am running this on CircleCI.
If I comment out the Sentry part of the Bundle React Native code and images phase the build succeeds.
The build succeeds when running locally but fails when run in CircleCI.

Steps to reproduce:
(Kind of hard to reproduce with CircleCI being involved however this is the xcodebuild command that fails)

set -o pipefail && xcodebuild -scheme  -project ios/.xcodeproj -configuration Release -destination 'generic/platform=iOS' -archivePath fastlane/generated/POM\ 2018-05-01\ 19.22.13.xcarchive -derivedDataPath 'fastlane/generated' clean archive | tee /Users/distiller/project/fastlane/generated/.log | xcpretty

Actual result:

Build fails saying /Users/distiller/project/node_modules/@sentry/cli/sentry-cli: cannot execute binary file

Expected result:

Build Succeeds

Please let me know if you need any additional details (React Native + Fastlane + CircleCI + MacOS + Xcode is a lot of moving parts), been trying to fix this for about 7 hours now and any help would be greatly appreciated!

Thanks for the detailed bug report 👍
This issue is basically about sentry-cli.
Still, this is a tough one, we've seen this error if you try to run sentry-cli on Alpine Linux but not on Mac.
What you could try is, install sentry-cli globally with homebrew or our script see: https://github.com/getsentry/sentry-cli
You have to change the path of sentry-cli in your Run script phase to call the global one instead of the one from node_modules.

Thank you for the quick reply. I had actually tried installing the CLI via brew earlier but forgot to update the build phase to use the brew installed version of it.
After updating the build phase to use the system installed CLI as opposed to the node_modules installed CLI it works! 🎉
Feel free to close this issue or if you want more info to try to address the wonkiness let me know.
Thanks for the help!

@amast09 Great.
I will close this issue, debugging on a foreign CI isn't really feasible I would say 😅
But thanks nevertheless 😄

For me this probelm was caused by having spaces in my project directory. My project was "Users/me/Desktop/Some Folder/my-project" - I just took the space away from "Some Folder"


Update README.md

Pull Request
📖 Description
Test PR from fork.

Closing experimental PR from fork.


Initial conditions based on other variable

Allow setting initial conditions that depend on another variable (requires evaluating the initial conditions for that variable)

This is much harder and messier to fix than I thought, and can always easily be avoided by redefining the model, as long as a clear error is raised (as is already the case).


Resolve path to module

I changed the way how mocked modules are located. Now it works just like require, i.e. you registerMock with path relative to the current directory. This way you only need to register the mock once, no matter how many times with what different relative path it is required.
I found this useful when writing tests for my application.

Works like a charm! Thanks!
:+1:

I hope this gets merged soon

This will behave unexpectedly if you require('mockery') from multiple files.
// testA.js
require('mockery');

// other/dir/testB.js
require('mockery');

Mockery's module.parent will point to testA.js, so all paths will be resolved relative to testA instead of other/dir/testB
Mockery could remove itself from the require cache, meaning that every time it's required, a new instance is created.  That way, module.parent would always refer to the module that require()d it.


Windows Installer - Adding Icon on Desktop for the Admin Page

Windows Installer - Will add Icon on Desktop for accessing the Admin Page - http://localhost:8001 in the default configuration.

If possible, create also a shortcut (without folder) in the start menu.
For "All users".


📚[Docs]: Improve guidelines in CONTRIBUTING.md

{minor change in the GitHub command to set the upstream repository, mentioned in the Contributing.md file}

LGTM 🚀

@AyushShri Please review this PR, so we can merge it if everything is fine.


[Logger] Add ramoops_logger as a replacement for sqlite logging

a previous commit removed logger_backend_sqlite. This was a valuable tool for getting the reason for a start failure.
Adding ramoops_logger will be able to accomplish the same thing with pretty much no overhead

This looked like it would be easier, but it looks like farmbot_system_rpi3 is missing something preventing this from working


golang atomic.Value can't load the latest value?

What version of Go are you using (go version)?

$ go version
go version go1.16.3 darwin/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"

What did you do?

I read the resource code in golang atomic.Value, doubt the Load method cant get the latest value.
func (v *Value) Load() (x interface{}) {
    vp := (*ifaceWords)(unsafe.Pointer(v))
    typ := LoadPointer(&vp.typ)
    if typ == nil || uintptr(typ) == ^uintptr(0) {
        // First store not yet completed.
        return nil
    }
    data := LoadPointer(&vp.data)
    xp := (*ifaceWords)(unsafe.Pointer(&x))
    xp.typ = typ
    xp.data = data
    return
}

if another goroutine exec Store after data := LoadPointer(&vp.data), the Load returned value wont the latest
What did you expect to see?
Load can get the latest store value
What did you see instead?
Load may not get the latest value

I don't see why that is a problem. The implementation is free to treat the Load as happening before the Store.


Compile error in generated REST controller due to filterCustomHeaders = true

Probable cause is the class hierarchy of BeanParams.
In the UML model, we have 2 POJOs:
@BeanParam( supportsCustomHeaders=false )
OSDMWriteContext extends OSDMContext

@BeanParam( supportsCustomHeaders=true )
OSDMContext

The code-generation for the RESTOperation (right-hand side in screenshot) uses hasParamCustomHeaders which checks the parent class.
The code-generation for the RESTResource (left-hand side) uses filterCustomHeaders which doesn't check the parent class.
Work-around: Setting OSDMWriteContext.supportsCustomHeaders = true fixes the code-gen error,
but creates an ugly subclass that has it own customHeaders field, "hiding" the same field in the parent.


Doesn't work after upgrading to Windows 10 1909 (build 18363.720)

The windows terminal was installed from  Microsoft Store. It does not work after upgrading to win 10 1909(build 18363.720). I try to reinstall Terminal  with release version 'Preview v0.10.761.0', doesn't work neither. The error in the picture said "The specified user does not have a valid configuration file"

Are you using "Run as user"?

Are you using "Run as user"?
I used to double click. I tried running with administrator, but got the same error.  Is it possible because I am a domain user.

I restart windows, and it's working now. Not only terminal got this problem, many windows apps has the same problem, but disappeared after restart.
Thanks!