[VC-36032] Configure controller-runtime to log with klog

I assume that the controller-runtime logs were being discarded.
With this change I hope to find some controller-runtime logs among the e2e test logs.
Testing
$ make test-e2e-gke
...
{
  "ts":<PHONE_NUMBER>929.3813,
  "caller": "metrics/metrics.go:97",
  "msg": "cache not synced yet, skipping metrics venaficonnection_*",
  "source": "controller-runtime",
  "v": 0
}
...
{"ts":1732123472568.4834,"caller":"agent/run.go:409","msg":"Data sent successfully","v":0,"logger":"Run.gatherAndOutputData.postData"}

That single message seems to come from venafi-connection-lib, so perhaps it's misleading to label these logs with source: "controller-runtime":

https://github.com/jetstack/venafi-connection-lib/pull/146
https://github.com/jetstack/venafi-connection-lib/blob/13c2342fe0140ff084d2aabfd29ae3d10721691b/internal/metrics/metrics.go#L92-L102

@maelvls Does that log message indicate a bug in the way we're instantiating the venafi-connection-lib client?

cache not synced yet, skipping metrics venaficonnection_*

I remember adding this log line in venafi-connection-lib's metrics.go (internal link). I remember writing this:
// We found a niche problem where `/metrics` would hang forever in case of a
// misconfigured RBAC. This was due to `cache.List` hanging until the cache
// is synced. To prevent that, we skip reporting this subset of the metrics.
if !hasSynced(cc.cache) {
	cc.log.Info("cache not synced yet, skipping metrics venaficonnection_*")
	return
}

I don't know why WaitForCache is taking so long to return. The Helm chart's RBAC is correct, so it shouldn't be this... This needs debugging, but we can do that in the next PR.


Feat: Create Seperate Login & Signup Pages and Add Functionality to Add Custom Images Using Cloudian

Describe the feature

This would allow the website to be a bit more dynamic and allow the users to be differentiated Better during the onboarding process
Currently Every Canteen uses the same image , allowing the user to posting different custom images (In seperate signup pages) would allow for better variation between the two

Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Can you think of an better appraoch , we wont ask canteen owner to upload canteen image on signup page but giving them
upadte canteen info ?? how about this ?

assign me this issue under gssoc24

I would like to work on this.
Can you please assign this to me.

Can you think of an better appraoch , we wont ask canteen owner to upload canteen image on signup page but giving them
upadte canteen info ?? how about this ?

OK I will do it

You can use the approach of creating a  complete different page to handle the adding detail section by the owner . when the owner are registering their hotel with the site they will be asked to upload these pictures which would be stored in cloudinary.
This action should be allowed only when the user is logged in as owner via this it is going to user that first credentials are verified and then registration is done .
Assign this issue to me so that I could work on this issue

@VanshKing30 bro can you assign me this issue as I have raised it?

@dfordp But can you please explain me the appraoch ??
I am not satisfied with your answer

The process I will take -> Creating a user profile section -> adding am update profile along with ability to upload new images -> (Optional) Create a separate on boarding page for canteens allowing all the necessary details

Start working on this


Quiz Exercises: Remove release date for quiz exam exercises

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.

Motivation and Context
When adding a new quiz exercise to an exam, a (server) error is thrown. This is because the client sets the release Date of the quiz exercise, however, no dates should be set for an exam exercise and that's why the server rejects the request.
errorKey: "invalidDatesForExamExercise"
message: "error.invalidDatesForExamExercise"
status: 400
title: "An exam exercise may not have any dates set!"

Description
In the quiz-exercise-detail.component.ts the assignment of a release date for exam quiz exercises was removed, the release date remains undefined.
Steps for Testing

Prerequisites:

1 Editor / Instructor

Log in to Artemis
Navigate to Course Administration
Navigate to the Exam Menu
Create a new Exam
Try to add a Quiz to an Exercise Group and verify, that no error is shown

Review Progress

Code Review

[x] Review 1
[x] Review 2

Manual Tests

[x] Test 1
[x] Test 2

Screenshots

Bug was simultaneously fixed with #5139


Array naming

When i define the array name with undersocre like this:
! array array_name = value value2 value3
rivescipt have a problem with substitute to regex.
Is this a bug? or array name must not include underscore? (I could not find about this in tutorial)

My guess is the _ is causing problems, but _ should be allowed, so I'll mark this as a bug.
Notes to self or anyone who wants to tackle this bug:

It might be this regexp that needs to change. Check if the \b word-boundary matches on _ characters; if so the regexp might need to be expanded out to r'\@([A-Za-z0-9_]+' which I think should match arrays better.
Debug what's happening around here (I usually would just add print statements to see what's in those variables), and the expand_array functions.

@kirsle I check the array regex and it is not causing the problem here. It is more on whether we would like to parse array with underscore first before treating _ as special wildcard.
The example here is included in the unit test
! array colors_bright = white blue

+ what color is my (@colors_bright) *
- Your <star2> is <star1>.


Update documentation

A few very minor corrections to the documentation.

@francoiswnel  Thanks man :+1:

No problem!


The player stopped playing

The player stopped playing, it only shows the first image from the video.
Which player are you experiencing the problem on:

iOS
Android ExoPlayer

Environment info
System:
OS: macOS 11.6
CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Memory: 101.78 MB / 16.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 12.22.0 - /usr/local/bin/node
Yarn: 1.22.11 - /usr/local/bin/yarn
npm: 6.14.11 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
Android SDK:
API Levels: 27, 28, 29, 30, 31
Build Tools: 28.0.2, 28.0.3, 29.0.0, 29.0.1, 29.0.2, 29.0.3, 31.0.0, 32.0.0
System Images: android-29 | Google APIs Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7678000
Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
Languages:
Java: 1.8.0_292 - /usr/bin/javac
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: 0.63.3 => 0.63.3
Library version: I tried 5.1.1 and 5.2
Reproducible sample code
    <Video
      source={{
        uri: 'http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/ElephantsDream.mp4',
      }}
      playInBackground={false}
      style={styles.video}
    />

same

I updated to latest version and problem solved.


build(deps): bump golang.org/x/sys from 0.5.0 to 0.6.0

Fixes #790

Apologies for the delay in replying to this, but it seems that renovate already updated this: https://github.com/cilium/tetragon/blob/47fba01fef79a994885306372b833e237f3157e4/go.mod#L41.
I'll close the PR.
Thanks!


[META] v3.2

Rough overview of features/requirements for v3.2:

HyperLedger Embedded side

[ ] Use appropriate crypto functions to generate keys and transactions.
[ ] Setup appropriate networking stack.

HyperLedgerServer side:

[ ] Setup network and nodes (dockerfile + service on our dev VPS)
[ ] https://github.com/hyperledger/sawtooth-supply-chain

@pcppcp @randomshinichi
Can this be closed?


coreutils not working due to sharrattj/coreutils

Hello and thanks a lot for your work regarding WebAssembly, Wasmer, and wapm.io.
One month ago, someone uploaded the package sharrattj/coreutils to wapm.io which caused echo and ls to not work anymore in webassembly.sh. Previously, your own package syrusakbary/coreutils was used, which worked. Now, the GraphQL API somehow returns the disfunctional package.
I got around this by running wapm install coreutils in webassembly.sh, which installed syrusakbary/coreutils. Do you have an idea on how to fix this globally?
Here's a screenshot:

maybe @john-sharratt can remove it as a first step =) ?


Update Travis-CI

I also simplified the packaging a little bit.

@ocefpaf with the repo name updated to cc-plugin-ugrid want to make sure I didn't break anything

All green! The only failure is coding standards but that is in the allow_failures section: https://travis-ci.org/ioos/cc-checker-ugrid/jobs/287776067


What challenges can/should be added to the workshop?

Hey @tako-black
I really enjoyed going through the challenges so thank you!
I would like to go through again now and add more links to documentation and even add some more challenges about aspects of React that I am finding as a 'beginner' that would be beneficial to learn.
Do you have any thoughts on this? Are there any ideas you have had but maybe no time to implement.
Thought I'd ask first before just diving in and changing your workshop :)
Thanks.

Hi @gyaresu
Thank you for your opening issue!
Of course PR is welcome! If you have some ideas to implement, please make PR!

:+1: Thanks!


Spotify: Don't close Raycast when changing the volume

Description
Previously, whenever the volume up and down commands were executed, the Raycast window would close, necessitating repeated actions. However, this enhancement ensures that the Raycast window stays open after executing the command, greatly improving usability.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Maybe this could keep the current behaviour by default, and add a command config for determining if Raycast should close afterwards?


Use data from https://coronadatascraper.com

I found a new data source which seems very promising: https://coronadatascraper.com
This PR moves us to that data source. I don't think it's ready yet, if only because the data from that source is not complete, but their data model is better and they seem to accumulate more data including:

Testing (#3)
States (#5)
Population (#2)

This opens up a lot of possibilities for charts like testing per million, cases per test etc.
I'll work on making the JS codebase easier to switch between the sources, and once they reach parity with the current data source I'll switch.

I adjusted the code to match the functionality prior to the switch.
I also added server-side conversion script that backfills data from the old source to the new one. I think the new code now matches the old, and contains more data that I'll expose in the follow up patches.
Ready to land. LFG!


Denoising

Your project is fantastic, quite a achievement! I saw another project which is similar to yours, they talk about denoising here in this issue:
hoverinc/ray-tracing-renderer#2
I thought you might have experience in denoising techniques and want to add to the discussion. It would be cool to also see denoising ending up in your project.

Thanks for the feedback! Denoising is great to reduce time to convergence, unfortunately it also introduces bias which is currently the exact opposite of what we're up to with this renderer.
The project serves as a code sample and visual reference for the Enterprise PBR shading model. Slow convergence is not that much of an issue but introducing error due to filtering unfortunately is.


Source of images on quay.io/opendatahub

The build and push jobs of multi-arch images of kserve like kserve-agent, router, qpext or kserve-controller have been failing for some time. And since the build jobs are failing, ideally there should not be any image on quay.io for the newer tags like v<IP_ADDRESS>, or v<IP_ADDRESS>. But I see there are amd64 specific tags available but not for ppc64le/s390x/arm64 etc. So, I want to know how these images are built and pushed to quay.io even after GH workflows are failing.

We have openshift-ci (you may have seen it in GitHub checks in PRs), which is also doing builds.
Configuration file is here: https://github.com/openshift/release/blob/master/ci-operator/config/opendatahub-io/kserve/opendatahub-io-kserve-master.yaml.
It only builds amd64 images. I don't know if it is capable of building other architectures.
GitHub actions workflows (which surely can build for other architectures) were broken until you kindly fixed :-)

OK, I'm closing this as looks solved.
Please, re-open if needed.


Add 'downloadable files' to title and body text

We had a couple of cases of people asking for this guidance.
It exists but we didn't mention it in the title

closing for now as it needs a name change and therefore a redirect


Bugfix: Warning message on capture view should take currency's decimal places into account

When you work with a currency with decimal_places != 2, the warning message ("You're about to capture an amount less than the total transaction value.") will be shown when it's not needed. The multiplier is hard coded to 100, but this should be based on the currency's decimal_places imho.
Similar problem occurs in the refund modal (incorrect amount shown), this is a bit more involved; and I've not yet found the time to create a PR. Will try to do this after feedback on this PR.

Looks good, just needs a changelog entry

@glennjacobs Done, thanks!


Add support for streaming responses

Using the a --stream/-s. Flag responses will stream updates when not present they will load and be shown at the end of output.

Closed by https://github.com/Add00/DocBot/commit/d73f33d676a64fb3fbe975a62205131830836098


plot plot by author age

La API está respondiendo con un TypeError:
https://apis.impactu.colav.co/app/affiliation?type=institution&id=65a9a92de5b59075e52c2720&section=research&tab=products&plot=products_age

The pipeline was renovated to filter works without publication date and withour autor birthdate. cf18292c84b5784537ac515664a0d6e665b16e74


sameo/issues/59

This PR closes issue #59.
We basically unit test configURIs() and thus parseCertificateAuthority() in order to check that we build the list of URIs to connect to properly.

Coverage increased (+0.04%) to 54.694% when pulling e7aadc52239e616649e2bec5ec83f713374ded4a on sameo/issues/59 into 670027d46f14f1deb75167ee686ab0b519c9cb35 on master.


Accumulation of backend payloads leading to OOM when the connection stale

Description:
We are seeing the backend payloads being accumulated when the backend connection is being stale.
Affected Product Version:
APIM 3.1.0

since 2.6.0

The issue has been fixed in master branch with the following PRs.

https://github.com/wso2/wso2-synapse/pull/1678
https://github.com/wso2/wso2-synapse/pull/1767


训练问题

我在用你提供的 100sample 进行训练的时候，遇到下面的问题：
环境：
pytorch 版本 1.8.1
执行：  python3 -m torch.distributed.launch --nproc_per_node=4 train.py --world_size=4
(MEMC) xxxxxxxxx@bigo:/data3/xxxxxxxxx/work/demoRIFE/arXiv2020-RIFE$ python3 -m torch.distributed.launch --nproc_per_node=4 train.py --world_size=4

Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.

/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/site-packages/torch/nn/functional.py:3503: UserWarning: The default behavior for interpolate/upsample with float scale_factor changed in 1.6.0 to align with other frameworks/libraries, and now uses scale_factor directly, instead of relying on the computed output size. If you wish to restore the old behavior, please set recompute_scale_factor=True. See the documentation of nn.Upsample for details.
"The default behavior for interpolate/upsample with float scale_factor changed "
/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/site-packages/torch/nn/functional.py:3503: UserWarning: The default behavior for interpolate/upsample with float scale_factor changed in 1.6.0 to align with other frameworks/libraries, and now uses scale_factor directly, instead of relying on the computed output size. If you wish to restore the old behavior, please set recompute_scale_factor=True. See the documentation of nn.Upsample for details.
"The default behavior for interpolate/upsample with float scale_factor changed "
/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/site-packages/torch/nn/functional.py:3503: UserWarning: The default behavior for interpolate/upsample with float scale_factor changed in 1.6.0 to align with other frameworks/libraries, and now uses scale_factor directly, instead of relying on the computed output size. If you wish to restore the old behavior, please set recompute_scale_factor=True. See the documentation of nn.Upsample for details.
"The default behavior for interpolate/upsample with float scale_factor changed "
/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/site-packages/torch/nn/functional.py:3503: UserWarning: The default behavior for interpolate/upsample with float scale_factor changed in 1.6.0 to align with other frameworks/libraries, and now uses scale_factor directly, instead of relying on the computed output size. If you wish to restore the old behavior, please set recompute_scale_factor=True. See the documentation of nn.Upsample for details.
"The default behavior for interpolate/upsample with float scale_factor changed "
training...
training...
training...
Killing subprocess 161896
Killing subprocess 161897
Killing subprocess 161898
Killing subprocess 161899
Traceback (most recent call last):
File "/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/site-packages/torch/distributed/launch.py", line 340, in
main()
File "/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/site-packages/torch/distributed/launch.py", line 326, in main
sigkill_handler(signal.SIGTERM, None)  # not coming back
File "/data3/xxxxxxxxx/anaconda3/envs/lib/python3.7/site-packages/torch/distributed/launch.py", line 301, in sigkill_handler
raise subprocess.CalledProcessError(returncode=last_return_code, cmd=cmd)
subprocess.CalledProcessError: Command '['/data3/xxxxxxxxx/anaconda3/envs/bin/python3', '-u', 'train.py', '--local_rank=3', '--world_size=4']' died with <Signals.SIGSEGV: 11>.

你好，这应该是说供数据的进程挂了
你先试试用单卡能不能训练，怀疑是 dataloader 初始化引起内存泄漏


Video Support

Hello, and thanks for this great library. I plan to use it in my app, yet it seems to be unmaintained anymore, is this the case?
I wanted to know if there is a possibility to have video support on it as well, where we draw on the video and then save/upload it, just like Instagram?
Thanks in advance.

@BaderSerhan curious if you found a solution to this or an alternative library, In the same position

@nnajiabraham Hello. Sadly I have not. I decided to postpone this feature in my app for now. But if you're good in native development you could work it out natively as there are more resources there, and you could bridge your library to react native.

@nnajiabraham Hello. Sadly I have not. I decided to postpone this feature in my app for now. But if you're good in native development you could work it out natively as there are more resources there, and you could bridge your library to react native.

Hello, it has been a long time but dit you find a solution ? I'm thinking that react native is really not a complete framework for apps like instagram or tik tok. I spent so many times to search answers for basic mobile features.


Update and rename README.md to BIDS README.md

I have reported this together with #152 and https://github.com/fieldtrip/website/pull/301 to GitHub.

Thanks @robertoostenveld. We need to label such PRs as "invalid" or "spam", I guess they are mostly due to https://hacktoberfest.digitalocean.com

Spammy pull requests can be given a label that contains the word "invalid" or "spam" to discount them. Maintainers are faced with the majority of spam that occurs during Hacktoberfest, and we dislike spam just as much as you. If you're a maintainer, please label any spammy pull requests submitted to the repositories you maintain as "invalid" or "spam", and close them. Pull requests with this label won't count toward Hacktoberfest.


Fix ChoicePrompt ListStyle.none when set via PromptOptions

Fixes #1370

Parity not needed for C#, unless I misunderstand null-coalescing operators).

Parity PR for Python

Description
ChoicePrompt determines style with:
const choiceStyle: ListStyle = options.style || this.style;

Since ListStyle.none enumerates to 0, which is falsy, ChoicePrompt will fall back to this.style, which is defaulted to ListStyle.auto. This prevents ListStyle.none from being set by passing in PromptOptions.
Specific Changes
- const choiceStyle: ListStyle = options.style || this.style;
+ const choiceStyle: ListStyle = options.style === 0 ? 0 : options.style || this.style;

I don't love the "complex" ternary and am willing to change it to something more like this, if preferred:
const choiceStyle;
if (options.style === 0) {
    choiceStyle = 0;
} else {
    choiceStyle: ListStyle = options.style || this.style;
}

Testing
Added one additional test to ensure that ListStyle.none can be applied via PromptOptions.

Pull Request Test Coverage Report for Build 86562

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.004%) to 85.115%

Totals

Change from base Build 86164:
0.004%

Covered Lines:
4917

Relevant Lines:
5536

💛  - Coveralls

Thanks @mdrichardson. I have added @johnataylor and @stevengum as reviewers and pinged them accordingly.


Move the sleep command to a separate step

As measurements in a single step are run in parallel, sleeping needs to be moved out to a separate step.
/sig scalability
/assign @mborsz
/cc @marseel

/lgtm
/approve


Add the node driver for meta/proxy when saving observability config

Signed-off-by: Jeroen van Erp<EMAIL_ADDRESS>

@hierynomus the solution doesn't seem to be working.... If you do a developer load:
yarn build-pkg observability + yarn serve-pkgs + developer load https://extensions.rancher.io/extensions/extensions-getting-started#test-built-extension-by-doing-a-developer-load
Still get an error while looking at the cluster card in that cluster with the agent installed. Not only the node driver was not active but even if I activated it, it didn't work.
Maybe Richard knows what's missing here.


Unable to remove node with "Waiting on user-node-remove" message

SURE-3520
Rancher Cluster:
Rancher version: 2.5.9
Number of nodes: 3
Node OS version: Rancher OS v1.5.3
Downstream Cluster:
Number of Downstream clusters: 110
Node OS: Rancher OS v1.5.3
RKE/RKE2/K3S version:
Kubernetes version: v1.18.20
CNI: Canal
Longhorn:
Longhorn version:
CPU per node:
Memory per node:
Disk type: HDD/SSD/NVMe
Network bandwidth between the nodes:
Other:
Underlying Infrastructure: AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal
Any 3rd party software installed on the nodes:
User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:

Issue description:
k8 node doesn't get removed from the cluster - UI shows the message: "Waiting on user-node-remove_c-qhshf"
Business impact:
They are updating Operating system in all the downstream cluster nodes. when they remove a node from the vsphere node pool, it was stuck in removing. Also it was not removing from the vCenter as well. They are stuck in removing the nodes and can't upgrade their nodes now.
Troubleshooting steps:

Edit the node in cli -> remove the finalizer -> node gets removed from the UI( but not from the vCenter)

Repro steps:
Not able to reproduce the issue in house.
Workaround:
Is workararound available and implemented? yes
What is the workaround: Remove the finalizer from node yaml.
Actual behavior:
Node stick in removing state
Expected behavior:
Node should be removed from the UI and vCenter.
Files, logs, traces:
finalizers: - clusterscoped.controller.cattle.io/user-node-remove_c-xxxxxxxx (see SURE for actual)
Logs attached to SURE ticket

Resolving as unable to reproduce


Fixed drill down expression for non singular boolean groupings.

Resolves #295

This was merged into release/4.0.0.


Specify optional max durability of Ocarina (durability used per horse summoned) & potential repair item

Whether or not the Ocarina breaks completely or simply becomes non-functional at 0 durability would also be an option.

Default repair item potential: golden carrots!

Repair methods could be extended to shapeless crafting with any number of the repair material (up to 8).


Sync Data mapper code action with latest ballerina api

Purpose
Sync Data mapper code action with latest ballerina api
Remarks

Add unit tests.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Checked the data mapper code action disabling capability within VsCode as well as through the plugin.
Also, manually tested the data mapper code action on VsCode.


Fix parsing escaped quotes in SExpr

Parsing KiCad SExpr files with quotes in strings, which KiCad represents as \", currently fails with "Trouble with nesting of brackets", hitting an assert at ecad/kicad_extra/sexpressions.py:22.
Code after this change may still have problems with backslashes in SExprs, but those are likely much rarer and would require more than a simple change to deal with.

Thanks.
I merged this with a fix for backslash escaping, let me know if you see issues with my fix. I couldn't make it break in my testing.
https://github.com/openscopeproject/InteractiveHtmlBom/commit/dfc6ad672666121aed1f4f8c3e0766a2743bebcf

Thanks. I couldn't break it either, I tried something similar, but I didn't add it to the regex.


[Bug]: NFT Minting Catch Error

Version

[X] Yes

General Question

[X] No

What happened?
If everyone has the same problem as I am (not exact, but similar), please let me know how to resolve this.  This is my first time coding this to try to mint my NFTs that I have created, and so far it's been rather difficult. Half the time I don't know what I'm doing without tutorials that were as far as March of this year when rinkby's was a thing. This is the error I came across trying to mint my NFTs.
PS C:\Users\aarons\Desktop\create-10k-nft-collection-2.0.1> npm run mint

><EMAIL_ADDRESS>mint
> node utils/nftport/mint %npm_config_start% %npm_config_end%

OK to mint ALL? (y/n): n
Exiting...

I followed the tutorial "BEST NFT Collection Minting Site (dApp)" last month, but that was uploaded in March of this year. What really threw me off was that the "create-10k-nft-collection" was updated so it did not have the presale/whitelisted code there as it did on that previous date, nor did it have the Mint Dapp folder to my knowledge. So I ended up manually downloading the minter dapp folder and applied it to the "create-10k-nft-collection" and tried my luck there, but I am still having the same issues. I already created my NFTPort auth key, and my contract address and I already have my 100 images and metadata on IFPS website. The only thing I haven't done is create a netify website for my NFTs because I am currently creating a website based on my nft collections, so I didn't think that I'd needed it if it's optional. Otherwise, I would've made one mistake after another. I would like to give anyone some more info on how I missed a step so you guys can give me some helpful suggestions on how I can successfully mint my NFTs. Please help!
Relevant log output
No response
Protect your information

[X] I Agree

Thanks for the help, but I'll have to start over.


make search box in issue sidebar dropdown list always show when scrolling

as title, replace #31597 after #32460, examples:

The scrolling menu has border, also need this:  Improve issue sidebar UI #32587

Need to fix:  Fix issue sidebar regression #32598


FEAT: Basic Functional on Alpine

Description of Changes:

Build easyaccess image based on alpine:3

Type of Changes:

[ ] Documentation (Add Documentation to Existing Code)
[ ] Bug Fix (Non-Breaking Changes Which Fixes an Issue)
[x] New Feature (Non-Breaking Changes Which Adds Functionality)
[ ] Improvement (Non-Breaking Changes Which Improves the Structures, Performance, etc...)
[ ] Breaking Change (Breaking Changes that Would Cause Existing Functionality to NOT Work as Expected)

To-Do or Other Concerns:

(NONE)

good


syncer do not set hare output for empty layer

Motivation

Closes #3522

Changes

when syncing, remove call to SetHareOutput on empty block ID when layer is empty. it should wait for a certificate instead.
undo retry stream in TestFailedNodes.
emit event Layer_LAYER_STATUS_APPLIED when a block is applied and use that for systest

bors try

try
Build failed:

ci-stage2

@dshulyak can you help me confirming this error?
i checked the nodes, they recovered from the missing hare certificate as expected (layer 15). so i think the failure is caused by grpc stream timeout, as indicated by log below  {namespace="test-ejzd", pod="boot-0-0"}
do you agree?
{"L":"ERROR","T":"2022-09-01T22:10:28.711-0400","N":"1cc8a.app          .grpc         ","M":"finished streaming call with code Unknown","node_id":"1cc8ab7e9b0b3a78e1c62543df148a056e8b76ab1b2c51334ed6a3198c69c448","module":"app","module":"grpc","grpc.start_time":"2022-09-01T21:51:44-04:00","grpc.request.deadline":"2022-09-01T22:49:32-04:00","system":"grpc","span.kind":"server","grpc.service":"spacemesh.v1.MeshService","grpc.method":"LayerStream","peer.address":"<IP_ADDRESS>:44042","error":"send to stream: rpc error: code = Canceled desc = context canceled","grpc.code":"Unknown","grpc.time_ms":1124113.8}

i think the retry logic in TestFailedNode i added is faulty
https://github.com/spacemeshos/go-spacemesh/blob/75ea165d6fb450160a36679861ebb5d00d1a0b9f/systest/tests/nodes_test.go#L179
the stream will subscribe to the latest layer event and would not get the hashes from the older (disagreed) layers again.
what do you think about doing LayerQuery instead?

what do you think about doing LayerQuery instead?

sorry missed that. it is indeed doesn't make sense to retry a stream
it won't be possible to use LayerQuery in watchStreams function, but generally can be used for this test
i think it makes sense to emit event that layer is applied every time it is applied (so that event will be delivered even in case of revert)

Actually sorry i don't understand the workaround and what current failure means.
The only place where layer is confirmed is at https://github.com/spacemeshos/go-spacemesh/blob/develop/tortoise/algorithm.go#L221-L224 . The fact that hare output was applied immediately shouldn't make any difference. Also the rerun is not triggered in this test, so we pretty much can't receive more than one event for the same layer.

bors try

bors try

bors try

bors merge


onSelect is not called when selecting a time (using time select only)

When using the time select only (no date), selecting a time does not fire the onSelect event.
I expect the onSelect to be triggered when selecting a time.
This issue has been noted in 3 other issues which have all been marked as stale and automatically closed.

https://github.com/Hacker0x01/react-datepicker/issues/1590
https://github.com/Hacker0x01/react-datepicker/issues/2156
https://github.com/Hacker0x01/react-datepicker/issues/2829

Commenting to keep this issue active


Add 24-hour format support

Hello.
Being from an european country, I find it extremely confusing to insert/edit timestamps in the current 12-hour (AM/PM).
It would be great if you could make available some sort of configuration so as to be able to enable 24-hour format in "manage time" timestamps.

PS: I love the power-up, keep up the good work!

Hi @ludwig801,
The issue is that Trello's datetime picker that is official doesn't appear to support 24-hour format. I could however create my own and replace it. I'll keep this open and next time I get some time to work on Activity timer I'll look into supporting this. Appreciate the feedback! And glad that you like it 💯

Adding it to next milestone. Will see if I can get to it & include this.

@ludwig801  Sorry for the delay. It's now been released with https://github.com/danniehansen/activity-timer/releases/tag/2.2.0
Decided to change over to 24hr instead of 12hr. Much broader audience for who uses the powerup. Will be adding 12hr support by configuration at a later point. Please let me know what you think of the latest release & how it works.


How to generate multiple address using bitcore wallet service

I am using bitcore wallet service. which gives the features like create wallet, wallet balance,generate proposal...etc, here is the link of
https://github.com/bitpay/bitcore-wallet-client which i have follow.
But if i want to generate multiple address using bitcore wallet service is it possible. because i can not found any details from the entire documents.

didn't API.createAddress work?
https://github.com/bitpay/bitcore-wallet-client#apicreateaddressopts-optsignoremaxgap-cb

Yes, use createAddress.
Take a look of bitcore-wallet-client also.

@matiu and @jeshan   I can not found any request parameter like privateKey or wallet Id, so how we identify or know newly generated address is connected with whom..?

you need a wallet to create the address so isn't that sufficient to know that this is the wallet from which  the address was generated?

@jeshan  If i want to create address using @matiu suggested createAddress, but i can not find any parameter like wallet id or privatekey something which will relate that address.


Change label from Download to Load full graph

"Download" is a bit confusing, one might expect it to start a JSON file download of the graph. Internally, it was called "loadFullGraph" anyway.

@maxkernbach already changed the label of the button in another pr.


How to use Return (i.e. not Enter)

I don't seem to find a way to map the numpad's Return button. Is there a way?

@koekeishiya
Hi, I would like to see that issue, however, issue seems to have gone...
Is there any same list of keycode anywhere?


update scrollTo api to newer

consider react-native version 0.20.0, old scrollTo api will get below warning message.
`scrollTo(y, x, animated)` is deprecated. Use `scrollTo({x: 5, y: 5, animated: true})` instead.

change it to this.refs.scrollView.scrollTo({x: k*size.width, y: 0}); could be solved.

Merged in #24


Support of Discovering Connected Devices ?

Any way to auto discovery or set by hostname ? My router doesn't have mode to keep same ip devices and I can't set static ip on controllers led.

Hey @dudusandi ,
Currently the plugin doesn't support discovering of devices.
But Yes we can discover connected MagicHome devices.
I will work on this implementation in my free time.
Expect this feature to be in upcoming v1.2.0
Maybe we can use MAC Address instead of IP to reference to lights.
Meanwhile if you like the plugin, please star, watch the repository to support the development.
Thanks

Using MAC would solve the problem, thank you very much.

Hey @dudusandi ,
I have updated the plugin. You can use MAC instead of IP. MAC should be in format of XXXXXXXXXXXX.
Due to lack of time, i was not able to test it thoroughly.
Would love if you can test v1.2.0 version and provide your feedback/suggestions.
Thanks

Fixed in v1.2.0

Hey @dudusandi ,
I have updated the plugin. You can use MAC instead of IP. MAC should be in format of XXXXXXXXXXXX.
Due to lack of time, i was not able to test it thoroughly.
Would love if you can test v1.2.0 version and provide your feedback/suggestions.
Thanks

I tested and it works perfectly with MAC, Thanks. Does this search every 30s use a lot of resources or not?

I noticed this problem in the log, but I do not understand what it is

@dudusandi I am glad that new version of the plugin fixed your issue.
The logs you are seeing is because of node-persist plugin. We use this plugin to cache IP:MAC Address mapping in case homebridge restarts or breaks.
I will debug this issue and you can expect a fix in upcoming version.
Meanwhile you can ignore the logs, since they are not a blocker and does not interfere with fundamental features of the plugin.

@dudusandi
I have added a potential fix for Unhandled Rejection logs occuring due to node-persist.
Please try updating the plugin to 1.2.2.
The issue should be fixed, and you should not see any error logs.
Enjoy your Lights
Thanks

@dudusandi
I have added a potential fix for Unhandled Rejection logs occuring due to node-persist.
Please try updating the plugin to 1.2.3.
The issue should be fixed, and you should not see any error logs.
Enjoy your Lights
Thanks

Hey @dudusandi
Looks like you need to install git on your raspberry pi.
Nevermind, looks like node-persist is causing lots of issues.
I will try to refactor the code and remove this dependency.
I will try to cache the IP:MAC Address using native node api instead of node-persist

Hey @dudusandi
Looks like you need to install git on your raspberry pi.
Nevermind, looks like node-persist is causing lots of issues.
I will try to refactor the code and optimize this dependency.

If I can help with anything, just ask.

@dudusandi Thanks for your support.
You can test version 1.2.4.
Hope it resolves your issue.

@dudusandi Thanks for your support.
You can test version 1.2.4.
Hope it resolves your issue.

Testing here, apparently perfect.

Glad to hear that.


Custom ClusterEnvironment breaks even after defining master_address and master_port abstract methods.

🐛 Bug

To Reproduce

In https://github.com/PyTorchLightning/pytorch-lightning/pull/10103, we have renamed master_{port/address} to main_{port/address}. We have followed the deprecation cycle as well, but this breaks the code as main_{port/address} are abstract methods. Not a desired behavior.
TypeError: Can't instantiate abstract class CustomClusterEnvironment with abstract methods main_address, main_port

Expected behavior

If master_{port/address} is defined, show the deprecation warnings but don't raise the error.
Environment

PyTorch Lightning Version (e.g., 1.5.0):
PyTorch Version (e.g., 1.10):
Python version (e.g., 3.9):
OS (e.g., Linux):
CUDA/cuDNN version:
GPU models and configuration:
How you installed PyTorch (conda, pip, source):
If compiling from source, the output of torch.__config__.show():
Any other relevant information:

Additional context

cc @tchaton @awaelchli @ananthsub

Yes, as I noted in the PR:

Yes. User with custom cluster environment plugins won't be able to instantiate their existing plugins due to the requirement of main_port, main_address to be abstract. They will have to rename master_port and master_address methods immediately.

The renaming was not included in any release, so we can still make adjustments if we need to. There is the option to remove the abstract requirement and re-introduce it when the deprecation cycle ends.

@kaushikb11 we are now already removing the old methods and completing the deprecation cycle in #13458. Sorry we couldn't discuss more or do something here, but I think it wasn't really a major issue for any of the few users that did use the custom environments.


[Question] Static vs Uniform

This is kind of a dumb question. What is the difference between static and uniform? For some context, I am using animatediff for vid2vid, and I don't use loop. I just cant figure out the difference.

The difference is in the starting frame of the sliding window at a given step:

with static, the window will always start at the first frame for all the steps.
with uniform, the window will start at a different location at each step. In this case the diffusion is more "spread out" and this will increase the consistency of the animation, but it's subtle in a lot of cases

This video gives a good explanation with examples :)

Thanks the video really helped


Fix #893 by passing in duration[:duration] which is a number.

/cc @jekyl/core

I don't even have the time to fix all that mess, it seems https://github.com/cucumber/cucumber-ruby/commit/6c346361ca7bb47bd9c6dd52f492b468d6bc4853 and all the tests are broken because none of the tests regression proof themselves and/or weren't updated :cry:

Thanks, @envygeeks

Issue #893 is now fixed, see https://github.com/cucumber/cucumber-ruby/issues/893#issuecomment-121711362.

Yup, I seen you fixed it so I closed this, thanks!  Is there anyway this can be pushed out today because we have dozens of pull requests failing because of this and I think we'd rather get an update from ya'll than temporarily patch a fix.

Releasing is out of my hands, so I can't say. If @mattwynne or any other of the release managers can find the time ...

I've just shipped v2.0.2 from the master branch.
@brasmusson you just need to create a rubygems account and send me the email address and I'll give you release karma.


Why confetti recreates on top after falling down to bottom?

I created list of BitmapConfetto with 200 different , not the same images. And I need to fall all of the to bottom from the top. But when first  confetti FadeOut at the bottom, then it recreates on the top again.
  int height = container.getHeight();
        int speed = height/4;
        int acceleration = -(height/4)/8;
        final ConfettiSource confettiSource = new ConfettiSource(-container.getWidth()/4, 0,container.getWidth()/2, 0);
        confettiManager = new ConfettiManager(this, myConfettoGenerator, confettiSource, container)
                .setNumInitialCount(2)
                .setTTL(6000)
                .enableFadeOut(Utils.getDefaultAlphaInterpolator())
                .setEmissionDuration(ConfettiManager.INFINITE_DURATION)
                .setInitialRotation(180, 180)
                .setEmissionRate(4f)
                .setVelocityY(speed)
                .setAccelerationY(acceleration)
                .setRotationalVelocity(30, 60)
                .animate();

Seems like starts second wall of  confetti  , and the method doesn't call any more
public Confetto generateConfetto(Random random)

Creating one unique confetti per bitmap is not something the library supports at the moment. The intended usage of the library is that you provide it a set of bitmaps or confetto objects, and the framework will randomly generate confetti for the specified duration / emission rate. You would have to add custom logic to ensure that the confetti manager doesn't re-generate the same confetto.


Static files listed under requests

For some reason when using nginx with version 0.9.1 the list of static files is empty while image, JS and CSS files appear in the top request URL report.

Just a quick update. My system just upgraded to 0.9.2 and the issue is still there.

Sounds like no config file was specified. The config file contains the extensions. You can run it with -p /path/goaccess.conf if for some reason is not under %sysconfdir%.

Ok, that fixed it. For some reason my system lost it's config file and I was using a custom one to run it in a cron. Didn't know that you had to specify the static files there.
Maybe some kind of hint in the output would be nice warning that no statics where found. Or even better, if no statics are defined to have some kind of fallback defaults.
Thank you!

Glad that worked. Thanks for pointing that out, I've added a new issue for this #272.


Address Issue #54

Address issue #54 and will create a random delay between 1 and 5 seconds after each request.  It slows down the bot a bit, but I seem to get less login requests when this delay is introduced.

Per @famewolf, I think this should probably be turned into a parameter/.env solution that is defaulted to off.  The parameters should probably allow the user to set the minimum and maximum seconds to wait.
I can adapt it to be in such a way, but this current solution that is in this pull request does not accommodate this.  I may not be able to get to it for a couple of days.

This is not really needed anymore.


fix: add support for catching uaccess

This was causing a misleading error when debugging an application
The error message could be tweaked. But this is also missing from the documentation on the dev site from the looks.

Thank you!


remove the node_modules from the repo

I'm submitting a ...

[ ] bug report
[ ] feature request
[x] chore

Current behaviour:
The repo contains the node modules making it heavy to clone
Expected behaviour:
The node_modules should be removed and the dependencies in the package.json should be updated if needed.

let me work on this

@rakshans1  thanks for contributing please have a look at some of the other issues or create any new issue and mark a star to the repository if you liked it.


Is it possible to add upload bandwidth control parameter?

Which version of the AzCopy was used?
AzCopy: 10.2.1
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
AzCopy related copy and sync commands had been used.
What problem was encountered?
While trying to upload content from VM (in Azure DataCenter) to Storage Account (in same Data Center). Because of related connection will try to reuse internal connection, AzCopy default 32 concurrent connection multiple related internal connection bandwidth result the network bandwidth reached to around 300 MB/sec. But unfortunately, because of such a sudden and huge bandwidth consumption, related connection had been automatically and enforced interrupted.
How can we reproduce the problem in the simplest way?
The simplest way is create VM and storage account within same data center. And then, try to upload several GB content.
Have you found a mitigation/solution?
Currently, the workaround is set the environment variable "AZCOPY_CONCURRENCY_VALUE" to 4. Even with this settings, sometimes, the upload bandwidth will reach between 190 and 200 MB some time. And the upload speed is quite unstable, can changed between around 40 MB and 190 MB.
So, to control upload speed, it would be better to provide a parameter to control upload speed.
I think similar condition might happened to those company, because AzCopy try to utilize related network bandwidth, it might result IT treat this as wrong usage and break it. Upload speed control will be useful in such condition.

We just added that parameter in version 10.2.1, but it's not on the documentation page yet.
Its called --cap-mbps. Its in mega bits per second. E.g --cap-mbps 800 will mean that AzCopy will not use more than 800 megabits per second, which equals 100 megabytes per second.


Sanic could be MutableMapping ?

The sanic class could be MutableMapping. it's very easy to add any application specific stuffs there.
app["session"] = ClientSession()
app["db"] = MongoDb()

I'm not opposed to it.
People are already doing something similar to this though with attributes:
app.session = ClientSession()

yes, it works with attributes, but it may overwrite default/builtin attributes by mistake.


Use PostgresConfiguration.ianaPortNumber where possible

This is a follow-on from vapor/postgres-kit#201.

Tests on this PR will not pass until vapor/postgres-kit#201 is merged.

These changes are now available in 2.1.1


Allow Builders to leverage Kotlin defaults, or kotlin builder notation when required fields are missing

ex data class in kotlin -
import com.github.pozo.KotlinBuilder

@KotlinBuilder
data class ContactInfo(val phoneNumbers: List<PhoneNumber> = emptyList(),
                       val emails: List<EmailAddress> = emptyList(),
                       val addresses: List<PostalAddress> = emptyList()
)

The create function within the generated builder looks like:
  public ContactInfo create() {     return new ContactInfo(phoneNumbers, emails, addresses);   }
The create here uses an all args constructor. However, if the object builders never set a value for one mandatory field (addresses field, for example) the create() call using an all args constructor will try to set addresses to null, which will throw an NPE type error since this value can't be null in the target kotlin object.
I would like a way for the create() method to:

leverage the default values in Kotlin that an object may have (usually non-nullable types have this), setting a field to the default value if a default exists

This would be amazing and very helpful to generate more accurate builders!!

Thank you for reporting this issue. I'll look into it.

@natashashams I tried to reproduce the issue what you described without any luck. It seems the issue description is not that clear to me.
Could you please create a very basic project where you can reproduce the described null pointer ? You can use the debug module as a template https://github.com/Pozo/mapstruct-kotlin/tree/master/debug

@Pozo I might be able to help clarify as I'm looking for this feature as well.
If a data class specifies a default value for a field in the constructor, the default is not propagated to the builder and will always be invoked with null.  Passing null is different than passing no value for that field.
For example
data class Foo(
    val bar: String = "bar"
)

import java.lang.String;

public final class FooBuilder {
  private String bar;

  public FooBuilder setBar(String bar) {
    this.bar = bar;
    return this;
  }

  public Foo create() {
    return new Foo(bar);
  }

  public static FooBuilder builder() {
    return new FooBuilder();
  }
}

If no value is passed to the builder, it can result in a NullPointerException or unexpected null values in fields.
val foo = FooBuilder
   .builder()
   .create()
// Created with 'new Foo(null)'

I also came across this problem and tried to fix it. However it seems not that easy to find the fields default values. The only way I found was accessing the java source code as described on stackoverflow article 6373145. However in the java sources the default values are "hidden" in a special constructor which is generated by Kotlin.
It would be nice to access the original Kotlin code, but this seems to be even more cumbersome, see here.


⚠ LPDM Handbook (hosted on Divio) has degraded performance

In 0dd29ac, LPDM Handbook (hosted on Divio) (https://repohubdev.tk) experienced degraded performance:

HTTP code: 500
Response time: 5503 ms

Resolved: LPDM Handbook (hosted on Divio) performance has improved in 6738bfd.


Link to a simple test page

stackblitz

:value-attr="id"
Down arrow IS flipped

Should be default with v-model, anyway doesn't help.
Only in auto or ltr, but after changing to rtl - does not flipped.


Awkward sentence/punctuation....

The sentence "Choose your favorite development environment or command line tools to create your bot in C#, JavaScript, or Typescript. (Java and Python are coming too!)" doesn't look right. Should it be "Choose your favorite development environment or command line tools to create your bot in C#, JavaScript, or Typescript (Java and Python are coming too)."

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

ID: 7ffe447d-f0a8-f4b6-d572-ba2c3bba4aab
Version Independent ID: 6c1d9a6a-4cb3-f404-ce1f-940b443f0e34
Content: Azure Bot Service Introduction - Bot Service
Content Source: articles/bot-service-overview-introduction.md
Service: bot-service
Product: bot-framework
GitHub Login: @Kaiqb
Microsoft Alias: kamrani

Changing to:

Choose your favorite development environment or command line tools to create your bot. SDKs exist for C#, JavaScript, and Typescript. (SDKs for Java and Python are under development.)

This update should go live in a few days.


[zh] update [creating a sample user] link

update  [creating a sample user]  link

Signed recheck.

/retitle [zh] update [creating a sample user] link

Thank you for proposing this fix. However, we will not merge partial fixes as per stated in the guideline. To find out what are the upstream changes to be synced, do this on the main branch:
./scripts/lsync.sh content/zh/docs/tasks/access-application-cluster/web-ui-dashboard.md

/hold

@nitor  Still interested in pushing this forward?


NO-ISSUE: Fix git commit for tag creation on the weekly deploy Jenkins job

This PR fixes the git commit command that we run before creating the weekly snapshot git tag.

PR job #41 was: UNSTABLE
Possible explanation: This should be test failures
Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/41/display/redirect
See console log:

Console Logs
Starting building: KIE » kogito » main » pullrequest » kogito-images.build-image #172Starting building: KIE » kogito » main » pullrequest » kogito-images.build-image #171Starting building: KIE » kogito » main » pullrequest » kogito-images.build-image #169Starting building: KIE » kogito » main » pullrequest » kogito-images.build-image #170Starting building: KIE » kogito » main » pullrequest » kogito-images.build-image #173Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1803 - kogito-base-builder: https://github.com/apache/incubator-kie-kogito-images/pull/1803 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1803 - kogito-jobs-service-ephemeral: https://github.com/apache/incubator-kie-kogito-images/pull/1803 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1803 - kogito-data-index-ephemeral: https://github.com/apache/incubator-kie-kogito-images/pull/1803 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1803 - kogito-jit-runner: https://github.com/apache/incubator-kie-kogito-images/pull/1803 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1803 - kogito-jobs-service-postgresql: https://github.com/apache/incubator-kie-kogito-images/pull/1803 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1803 - kogito-data-index-postgresql: https://github.com/apache/incubator-kie-kogito-images/pull/1803 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1803 - kogito-jobs-service-allinone: https://github.com/apache/incubator-kie-kogito-images/pull/1803 completed: UNSTABLE[Pipeline] unstableWARNING: Tests on kogito-jobs-service-allinone seems to have failed[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // parallel[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - 'https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/41/api/json?depth=0'[Pipeline] readJSON[Pipeline] sh


🏗 Build Analytical Platform Compute EKS

User Story
As an Analytical Platform Product Engineer
I need to host applications that can't live on Cloud Platform
So that we can run the service
Value / Purpose
DPAT's EKS lives in DPAT account, and can't easily be renamed, my proposal it to provision a new Modernisation Platform (analytical-platform-compute), and rebuild our EKS cluster (without any Data Platform components)
Useful Contacts
@jacobwoffenden
User Types
Analytical Platform Product Engineering
Hypothesis
If we rebuild our new EKS cluster in Analytical Platform Compute
Then we have home for Analytical Platform Dashboard, Analytical Platform Tools, and Analytical Platform Airflow
Proposal

Request new Modernisation Platform accounts (analytical-platform-compute)
Retire new Modernisation Platform accounts (data-platform-compute)
Build out EKS cluster

Additional Information
DPAT EKS currently hosts:

Data Platform Assets (web service serving images)
Data Platform's Open Metadata and DataHub instances
GitHub Actions Self Hosted Runner capability

Definition of Done

[x] Request new Modernisation Platform accounts (analytical-platform-compute)
[x] Retire new Modernisation Platform accounts (data-platform-compute)

https://github.com/ministryofjustice/modernisation-platform/issues/7071

[ ] Build out EKS cluster

02/05/24 update:

Build out of cluster in AP Dev continues in https://github.com/ministryofjustice/data-platform/pull/4259

Switched to EKS API for authentication management
Enabled CloudWatch Observability addon

VPCs have been been rearchitected to consume Airflow's VPC range with the intention that our networks will replaces theirs https://github.com/ministryofjustice/modernisation-platform-environments/pull/5927/commits/6577715a65e653c4d01f66d44dce676aef63147d

Cluster creation blocked as MemberInfrastructureAccess cannot create OIDC provider due to org SCP
Error: creating IAM OIDC Provider: operation error IAM: CreateOpenIDConnectProvider, https response error StatusCode: 403, RequestID: 56134a29-6a14-430b-879a-2f4fa59f0e41, api error AccessDenied: User: arn:aws:sts::381491960855:assumed-role/MemberInfrastructureAccess/aws-go-sdk-1715608907993994861 is not authorized to perform: iam:CreateOpenIDConnectProvider on resource: arn:aws:iam::381491960855:oidc-provider/oidc.eks.eu-west-2.amazonaws.com/id/1972AFFBD0701A0D1FD291E34F7D1287 with an explicit deny in a service control policy

13/02/14 update:

In an effort to not build the cluster bit-by-bit, I configured VPC CNI to use IRSA immediately, but theres a dependency on the cluster being "ready" so that the IRSA enabled role can create... 🐔 & 🥚

I switched back to the default of VPC CNI using the node's role, and enabling the VPC CNI policy, but I will perform another experiment tomorrow of the CNI policy being attached to the node and VPC CNI using IRSA immediately (in Analytical Platform Development)

However cluster creation was ultimately blocked by an organisational SCP denying iam: CreateOpenIDConnectProvider

@dms1981 added the permissions to MemberInfrastructureAccess which exposed the SCP
@ewastempel removed the accounts from the OU, and introduced another error because GitHub Action couldn't assume the backend role

15/05/24 update:

Work was carried out with @ministryofjustice/modernisation-platform to unblock EKS deployment

https://github.com/ministryofjustice/modernisation-platform-environments/pull/6071
https://github.com/ministryofjustice/aws-root-account/pull/906
https://github.com/ministryofjustice/modernisation-platform/pull/6998

Cluster has been deployed, but no extra components deployed
Work halted to allow #3881

16/05/24 update:

Work halted still, but agreed with @julialawrence that I'd deploy the following charts via Helm instead of using the native Amazon CloudWatch Observability EKS add-on, because I can't see how we'd pass in arguments to use CMK or IRSA/Pod Identity

https://github.com/aws/eks-charts/tree/master/stable/aws-for-fluent-bit
https://github.com/aws/eks-charts/tree/master/stable/aws-cloudwatch-metrics

20/05/24 update:

aws-cloudwatch-metrics installed but is configured, but set to use hostNetwork: true as it currently doesn't support either IRSA or EKS Pod Identity 😭

[ ] find upstream issue that tracks this, maybe raise with TAM

aws-for-fluent-bit installed, publishing to CMK encrypted CloudWatch log group using IRSA 🎉

21/05/24 update:

Added support for reading from remote AMP workspaces

https://github.com/ministryofjustice/observability-platform/issues/86

Pending working GitHub Actions 🙏 https://github.com/ministryofjustice/modernisation-platform-environments/pull/6184

Added Kyverno

22/05/24 update:

No work carried out
[ ] v3.0.0 of Amazon Managed Prometheus module has been released with support for KMS so we can import into that

https://github.com/terraform-aws-modules/terraform-aws-managed-service-prometheus/releases/tag/v3.0.0

https://github.com/terraform-aws-modules/terraform-aws-managed-service-prometheus/issues/15

https://github.com/terraform-aws-modules/terraform-aws-managed-service-prometheus/pull/16

1.30 compatibility has been added to aws-guardduty-agent

Span out into:

https://github.com/ministryofjustice/analytical-platform/issues/4422
https://github.com/ministryofjustice/analytical-platform/issues/4424
https://github.com/ministryofjustice/analytical-platform/issues/4425


Update bot.py. An easy way to get the ID of group and private chats to restrict access for unauthorized access.

Hello, I want to express my gratitude for this project. I personally use it and greatly appreciate the value it brings. However, I've identified a feature gap that I believe could significantly enhance its utility for myself and others. Specifically, the ability for the bot to operate exclusively within predetermined chats or with certain users, accompanied by an easy method to display user and chat IDs in messages for administrative purposes.
CHANGELOG:
If an attempt is made to interact with the bot outside of permitted groups and private chats, the bot will respond with a message containing the required group or user ID. This provides a simple mechanism for administrators to manage access permissions.
I kindly request the incorporation of these changes into the main codebase to obviate the need for manual updates and to offer a more  user-friendly experience to all users.
Thank you.

Great PR, thank you @wladradchenko, I merged your PR to my fork;
https://github.com/Natgho/chatgpt_telegram_bot/pull/8
If you would like to contribute to my fork, I would be happy to do so. Unfortunately, nobody is interested in this repo.


 Optuna implemented and works though. Some evidences:

Will take nicer plots if time allows, but for now here is evidence of Optuna working.

Better plots. Note that optuna studies are separate from each other as well as trials within the study. Can show that on some of the hyperparameter searches we are able to find hyperparameters that converge to MNIST nicely. Also see the command terminal in the previous comment to observe that the model is keeping the hyperparameters from a previous trial in the study, and sequentially saving these.


added research paper

A new research paper has been added to the list of research papers related to IoT Security.
url: https://www.mdpi.com/1424-8220/19/19/4107

Also, added one case study on IoT Security@CERN - https://doi.org/10.5281/zenodo.1035034

I'll take a look at the two papers later today

@0xricksanchez thanks.


Add password mask

Making new PR for adding of password mask to text boxes.

Fixes #157

@philipbawn Awesome!! Thanks so much! Don't worry about fixing the prop vs field. I have a lot of work to do in that area as it is. I'll probably use something like resharper to do some mass clean up.


Ring of Hungerless not working with Vampirism

Hello, would it be possible to make the Ring of Hungerless work with Vampirism' blood bar? Thank you

Vampirism don't use the vanilla hunger system, so it would required a manual compat solution to work.
Not planned ATM.


Is this project is dead ?

There is no any activity or progress on this project repo from long time.
What is the future of this project ?

Good question! I have the same.

The project is not dead!. But now is in an experimental phase. We are validating if it is an interesting option to include with .NET MAUI. The main PRs and changes are to keep itup to date with the latest versions of .NET MAUI as well as validate possible questions related to accessibility, performance etc.

@jsuarezruiz  I guess this is the most demanded feature in .net maui & to compete with competitor like flutter.
Still why Microsoft is validating it in this area ?

Very interested in this project so I hope it continues. Great work btw!

@jsuarezruiz I guess this is the most demanded feature in .net maui & to compete with competitor like flutter. Still why Microsoft is validating it as interesting option to include with .net maui ?

Decisions are based on many different types of metrics. From your comment here, to the interest in the repository or many interviews with developers and companies. Collecting the data and analyzing it correctly takes time. If you want to share your feedback with more details, I encourage you to do it here or share it with teammates like @davidortinau

i like this project

@jsuarezruiz, you say "The main PRs and changes are to keep itup to date with the latest versions of .NET MAUI". So is the latest nightly build available through the documented nuget feed (as of writing this would be 6.0.101-nightly.253) compatible with .NET MAUI RC1 and all its breaking changes since P14?

@jsuarezruiz, you say "The main PRs and changes are to keep itup to date with the latest versions of .NET MAUI". So is the latest nightly build available through the documented nuget feed (as of writing this would be 6.0.101-nightly.253) compatible with .NET MAUI RC1 and all its breaking changes since P14?

This PR include support for .NET MAUI RC2 https://github.com/dotnet/Microsoft.Maui.Graphics.Controls/pull/106, but need to update CI to generate the new packages.

i like this project!!!

@jsuarezruiz would like to try this out, any kind of ETA on getting this new package released?

@jsuarezruiz would like to try this out, any kind of ETA on getting this new package released?

Yesterday we merged a PR to use .NET MAUI GA versions. The build is running and the nightly working. The latest package have been created today.

@jsuarezruiz Great work, love this project. Long may it continue!

i like this project

Same here. I've been looking for a UI library like this for a long time - no XAML needed.  In the Javascript world, there is QooxDoo UI framework (No HTML/CSS needed) which does the same for a very long time. Unfortunately, the QooxDoo UI is not known to many so it is not very popular but we are very successful with it in all our large software projects.

I like this project

Looking forward to this project very much, hopefully this will find its way into the default MAUI graphics library.
One question though @jsuarezruiz: Currently I can only build from source, the nuget nightly feed is not working for me?!

This is how the real MAUI should be: a real alternative to Flutter.
This project should be absolutely a top priority, together with MAUI Linux host, and MAUI Linux Blazor Hybrid.

I join the crowd in wanting this in MAUI. Go go go!

In my opinion this should be the future direction of MAUI.

In my opinion it is more than necessary. If you look at the numbers of Xamarin.Forms.Visual.Material package: 1.7 million downloads, current version 7.3 thousend downloads.

I really hope this project isn't dead. I hope work continues on this project or that it gets merged into the main .NET MAUI project. It would be nice to know which direction this project may be heading in.

This project and the drawn principles upon which it rests are much needed.
Consider this excellent tabs project https://github.com/roubachof/Sharpnado.Tabs for MAUI. Because of the abstraction principle of MAUI, no desktops (or was even) is supported yet.
The story is the same for other 3rd party controls. It is symptomatic.
With drawn controls, it wouldn't be.

Please continue the work on this project as main project for Maui. It’s currently unbelievable how hard it is, to write controls which look and feel the same on all three platforms (Android, iOS, Windows). And the reason is, that every small information and binding must be projected to a native control, and there is always some unexpected errors in this projections.
Do you want to know which controls worked without an issue and looked and behaved identical on all platforms? The Skia controls we’ve created.
i really wish that all of the app would work this flawless.

Again, bumping and agreeing with everyone above. @jsuarezruiz, you see that users really like this project and what it brings to the MAUI ecosystem.
Would you mind giving us a quick update what the MAUI team' s plan regarding this project or related functionality looks like? We are kind of in the dark here, this project is in the github.com/dotnet namespace, and you are a XF team member, but there is no clear indication why this repository is "kind of separate" from the main dotnet/maui repo?

Again, bumping and agreeing with everyone above. @jsuarezruiz, you see that users really like this project and what it brings to the MAUI ecosystem.
Would you mind giving us a quick update what the MAUI team' s plan regarding this project or related functionality looks like? We are kind of in the dark here, this project is in the github.com/dotnet namespace, and you are a XF team member, but there is no clear indication why this repository is "kind of separate" from the main dotnet/maui repo?

I would love to share more details or information,  but I have nothing new to share. I really think that having drawn pixel perfect controls has potential, I continue to experiment with ideas (templated drawn controls in AlohaKit https://github.com/jsuarezruiz/AlohaKit) or tooling (figma to Maui Graphics https://github.com/jsuarezruiz/figma-to-maui-graphics). In this library I am pending to solve any possible doubt or problem in its viability (a11y, performance, speed to adapt to changes, etc). However, to take any kind of step there are many dependencies and decisions to be made that take time and require many steps. I know that PMs try to do their best to make decisions based on data and in the most adequate way possible. There are meetings where data is taken, surveys, etc. At the moment, all I can share is:

Personally I keep exploring ideas and creating options. If this gets past the experimental phase, this approach can get code or ideas from AlohaKit, etc.
I keep the library up to date for those who want to try.
The library is still in an experimental phase and we continue to capture any feedback. I read and share the feedback from this thread. Thank you very much to all.

@jsuarezruiz thanks for giving us an update, even if there is not much new information. It's always helpful, to see that there is still (at least some) "life" in it. 👍
Sad to hear, that the PMs don't see this as an important topic for MAUI :-/

Thanks @jsuarezruiz for your response and all your work, also glad to see that this is not abandoned. I hope the PMs will listen that this has a higher priority since it affects nearly everyone designing a MAUI UI. In my opinion, it should be given at least more weight than some of the more niche improvements talked about in recent .NET MAUI podcasts.
I really think that entries/text fields as shown on the material.io site are quite standard and should be easy to implement in MAUI. It is one of the basic building blocks of designing a good-looking UI, which is quite difficult to achieve with the standard controls (having to combine Entry, Label and Animations into a custom control).

I love this so much! And 'From Figma to AlohaKit UI' is amazing! 🔥🤩
I hope it gets out of Experimental phase soon. 😊

Why not use Avalonia it does the same thing and works in the browser?

Unfortunately, Avalonia still does a poor job of targeting mobile. But yes, it is superior to MAUI

And Uno Platform too.

Unfortunately, Avalonia still does a poor job of targeting mobile. But yes, it is superior to MAUI

Has nothing to do with Maui vs Avalonia. They do graphics pixel perfect rendering with c# you can render it in MAUI.

And Uno Platform too.

Uno uses native controls Avalonia is a flutter like rendering engine.

Since this has been stalling for a while, anyone following this issue/repo I'd like to point to https://github.com/enisn/UraniumUI or https://github.com/mdc-maui/mdc-maui as alternatives (I did not find them right away, so maybe this comment helps some other people ...)

Reading the latest .NET 8 MAUI preview release notes show me that MAUI is already dead.
It will be only used for mantaining old xamarin.forms projects and blazor hybrid embedding.
Microsoft has no intention to invest in a flutter like alternative. It's really sad.
Time to consider Uno or Avalonia

There is hope:  https://github.com/jonathanpeppers/spice

Reading the latest .NET 8 MAUI preview release notes show me that MAUI is already dead. It will be only used for mantaining old xamarin.forms projects and blazor hybrid embedding.
Microsoft has no intention to invest in a flutter like alternative. It's really sad.
Time to consider Uno or Avalonia or ... Flutter

If a Flutter experience is your aim, then Uno and Avalonia are more aligned to that goal. .NET MAUI is first and foremost native UI, with the option to layer in drawn UI as desired, and as such the work described in the release notes and on our roadmap is aligned with improving that experience.

Thanks @davidortinau for your time responding to my post.
Xamarin.Forms and MAUI native UI are a good goals in theory, but in the past years of the project until now it resulted practically to be a real pain to manage/maintain/performance/customize ... finding the common apis, creating hacks over hacks that lead to frustration, time and poor performances. XF and MAUI teams has done an insane work, and I understand that it's really difficult to develop new features, breaking old, stay updated with the various platforms sdks and UIs. Microsoft don't help you either with WF, WPF, WinUI UWP, WinUI Reunion ...
The flutter/avalonia approach of drawn UI, IMHO is more efficient, more maintainable and future proof, as we can see from flutter success: from nothing to near every platform with 60fps performance in short time and with a very large community.
See this project of drawn controls; Spice cross platform framework that @tcsaddul posted up  https://github.com/jonathanpeppers/spice; AlhoaKit UI, UraniumUI, Photino and SpiderEye for blazor hybrid embedding (even with linux support), demonstrates this needs and this possibility very well: that few developers in a short time shows that this is an efficient and viable solution.
I understand the needs to maintain the old xamarin.forms codebase and to move forward but IMHO, MAUI needs also a flutter approach of drawn UI, otherwise ... it will be the next silverlight.
Microsoft marketing sells MAUI as "Multi platform UI" and probably many users misundertands or they expects some other type of product from MAUI. I respect you and I understand that you are not in a simple position for the many expectations.
So I thank you for pointing clear that MAUI is first and foremost native UI, but we will not be using this "Xamarin++" MAUI for our next projects, even for native ui.

@davidortinau
Definitely what @cyberdarius said.
We need drawn controls as soon as possible in MAUI. Not “as third party thing”.
I work in a large LOB project, which is on the brink of failing cause of xamarin’s/Maui’s never ending rendering bugs. The heck, we started writing our own drawn controls for simple things like image with skia, because it’s less pain to write controls from scratch, than to workaround maui bugs which change with every release or just simply do not get fixed.

@davidortinau You can embed avalonia controls in Maui app they do custom rendering using skia exactly what you want.

Any source as to how? Googled didn't come across a guide for MAUI

@davidortinau https://github.com/AvaloniaUI/AvaloniaMauiHybrid

@cordasfilip Hey, that's great. Thanks for sharing! Probably getting a bit off topic, but the other way round would be awesome too if it were possible. To me the thing that MAUI is missing is a Linux framebuffer backend and/or GTK4 + Wayland, maybe a Maui Graphics backed reference for more options to match Avalonia's platform coverage. Though using Android for embedded is acceptable for my use cases.


Option to stop uv self update from adding . $HOME/.local/share/cargo/env?

Ciao! With rust on my machine already, my shell already sources the cargo/env script. Every time that I upgrade uv to a newer version with uv self update, the line is added to my zshrc and fish conf dir (which I do not use). This is absolutely nothing high priority, but I was wondering if you would consider setting an option to disable this behaviour.

I think you may be looking for --no-modify-path ?

I think you may be looking for --no-modify-path ?

Interesting. I don't see it as an option for uv self update though.

Seconded - I upgraded from uv 0.4.3 -> 0.4.4 just now with uv self update, and it appended ~/.bashrc with:

. "$HOME/.cargo/env"

I would certainly prefer if the self update command by default did not modify any configuration files. My reasoning is that, if updating, the user has presumably already set their paths in the way that they want (in my case I want my .bashrc portable between different machines, so I don't put any paths directly in that file).
Only a small niggle though. I just have to do a git restore on it every time I upgrade, which is mildly inconvenient at worst!

Maybe one could do uv self update --ensurepath or a flag like that and have this behaviour to be opt-in.

Looking into upstream support for this — I don't think we can add it until they do.

Can you try setting INSTALLER_NO_MODIFY_PATH=1 during the update?

I'm on a different machine today (Fedora 39), where it doesn't try to modify ~/.bashrc, but instead creates or modifies ~/.profile

Can you try setting INSTALLER_NO_MODIFY_PATH=1 during the update?

Yes - this seems to do the trick (installed 0.4.4 and tested on uv self upgrade thereafter).

Great — we can document that.

Confirm too, setting INSTALLER_NO_MODIFY_PATH=1 works :)


torchscan: list index out of range

Hi,
The torchscan seems not support CUDA 11.

Hello, thanks for your interest!
Can you share which parameters did you pass to the script to obtain this error? Can you share the debug.log file?
It looks like the problem is due to the input_shape variable.

Hi~ :smile:
I meet the same Error too when using torchscan to estimate model size, especially when I set VIT or CCT as backbone.
I don't know why this happened and how to deal with it, so I just comment out related codes, and other code runs well.
However, if you can fix it up, that would be so great~
Following is some packages version in my running environment :
pytorch                   1.10.0          py3.7_cuda11.3_cudnn8.2.0_0    pytorch
torchscan                 0.1.1                                     0    frgfm

Thank you @zafirshi, we'll try to fix it. Can you share your debug.log file that is generated in the run that gives this error?

Hi~ I know what happened. Actually, this is my problem, not the code problem. I'm really sorry to bother you for my mistakes. :persevere:
As you guessed, it is actually the problem of input image size. If run with ViT or CCT backbone, we need to hard_resize input img to 224 * 224 or 384 *  384 for using the pretrained weight. (CCT needs 384*384 only because we use cct_14_7x2_384. If changing code as demand, CCTs of other sizes can also be used)
I don't know whether the previous questioner reported an error for the same reason as me. @sijieaaa  :wave:
After changing the input_size to 384 *  384, I found that the code runs normally under the CCT configuration.
However, when using ViT as backbone and netvald as aggregation method , an error  will be reported.
AttributeError: 'BaseModelOutputWithPooling' object has no attribute 'norm'

I find that in this line below, variable outputs is not a tensor, it's a instance of class BaseModelOutputWithPooling cause we use third-part package transformers. (In cct backbone, output is a tensor)
https://github.com/gmberton/deep-visual-geo-localization-benchmark/blob/ace2a644422dd109d28a2f9fb77c292e3d1d9e97/model/aggregation.py#L162
I think if we output a tensor, this bug can be fixed. And I find that timm could realize this in a simple manner, and then ViT+netvald model will easily get! :+1:

We just added the possibility to use ViT with NetVLAD, by making sure that also ViT outputs a tensor in all cases  :)


Decimal number in ColumnChart

Hello
I'm using a columnChart and I need to show the value in each column, everything correct. The problem is when I need to show a decimal number in a label of the column, shows an integer and around the number.
How can I show a decimal number?
In the same example, i need configure the range of th Y axis to add 5 to the max value.
How can i configure? i'm using a static function  "Axis axisY = Axis.generateAxisFromRange(0, 120, 10);"

Thanks

Use this way.
ySubColumn.setLabel("${"%.3f".format(value.y)}")


update testing code according to the latest spec

As PR #206 has been merged, the testing code should be updated accordingly.
Signed-off-by: Wang Yan<EMAIL_ADDRESS>

@wy65701436 - could you please push a dummy commit to trigger the CI? (failed due to docker hub rate limit)

@jonjohnsonjr - feel comfortable with this update once we have #239?

It depends on what the point of the test is. The registry indicating that it doesn't support these monolithic uploads will pass this test, which is good in that it's following the spec, but it is a little misleading if I want to know whether or not the registry supports single-request monolithic uploads... does that make sense?
I think I'd rather see verification of the two different expected paths. Right now, the following test does GET request to blob URL from prior request should yield 200, but that cannot fail in a lot of cases because the config blob was uploaded in previous tests. Reusing the config blob for this test means that we don't actually know whether or not the blob was uploaded (even in the 201 case).
It would be nice to verify that the blob does not exist if the registry returns a 202 -- that would satisfy me, I think :) but that requires changing the tests a bit to use a different blob (I think).

closing in favor of #246
@wy65701436 thanks for drawing attention to this issue


Logout button visible only during session

User Story
As a respondent, I want to see a logout button after I login but not before login, nor after I logout.
Acceptance Criteria
Given that I am on the Form Landing Page, when I have not yet logged in, then I do not need to see the logout button at the top right.
Given that I am on the Dashboard or form pages, when I have logged in, then I want to see the logout button at the top right.

dupe of #256


Like / Reply placement on comments

The placement is the same for all the versions, even though the placement changed.
Heres how it looks for all themes. (note the reply, like placement, and the number color)

Heres how it looked back in 2009 / 2010. The likes and reply button were on the very right of the comment.

In 2012, it looked like this, and the like counter color was green. I dont know if it was like this in 2009 because i couldnt find any comments with likes.

In 2013, the reply and like buttons finally moved down to the bottom, but the counter was still green.

Im gonna take a guess because comments fail to load on Archive after around 2014 and say that in 2017, the counter finally became blue.

I'm well aware of what the old comments looked like, but thanks for the screenshots anyway. Once again, this is something that just hasn't been done yet. (A good rule of thumb is that every CT inaccuracy is something I'm aware of, but haven't gotten to yet) In terms of layout improvements, it's relatively high priority, but code cleanup is top priority right now (which will come with layout improvements elsewhere), so it could be a little while yet before I get to this.

Im gonna take a guess because comments fail to load on Archive after around 2014 and say that in 2017, the counter finally became blue.

I don't think so. I'm almost 100% certain that they've been blue since at least 2016. Maybe even 2015. Maybe even 2014.

Probably 2015, that 2013 one was from 2014, on a snapshot that did work with comments. I was using DJ Earworm's United State Of Pop mashups.

Any way you could send a link to the specific archive? I've never found an archive after mid 2013 that works with comments (until Polymer of course). Also DJ earworm seems to have several of these videos, all of which seem to be pretty popular and have lots of archives, so finding that specific archive is going to be like finding a needle in a haystack.

Oh no, I was talking about the 2014 archive that had comments. Unless you were mistaken, and there was no 2014 archive with working comments (which wouldn't surprise me).

Yeah i was mistaken, i was thinking of the 2013 one, but it has actually been blue since late 2013!

This image is from an article in November 2013 talking about the petition to remove the Google+ requirement to comment. Link to article: https://www.latimes.com/business/technology/la-fi-tn-youtube-comments-46000-petition-20131108-story.html


Tranq shells need either more crafting requirements or chloral threshhold needs to be increased

They're basically new instant-win buttons and as of late I'm seeing certain secoffs trying to get them at the very start of the round solely for dopamine chasing and validhunting, makes for very fucking boring gameplay where all sec does is fire one round and end any conflict that crops up.
it's either that or i start smiting secoffs who abuse it your call really

Yea they are in a weird place balance-wise.    I'm not sure if the 'sleep' is any more unique than a normal 'stun' at that point to justify their existence.  Maybe we just need resistance to instant injectors. So wearing any kind of armor would reduce the number of chems injected, from just a little with the vests to blocking lots of the chems in a jugger suit. This would theoretically balance out the hypo spray too to stop the cmos from near insta killing nukies with lex hypo

I'm starting to lean towards the fact that instant injectors just need to not be instant anymore period, hypo included. Leave the actions as is but have the chems build up in the chemstream gradually rather than immediate transfer of contents and having those contents immediately show up in chemstream.

Yea that would also be a solution, and would also help reduce the hypo issues too

I'm absolutely for some kind of delayed-release mechanism.  Being able to add reagents slowly instead of all at once allows a lot more flexibility with balance.

Its been rebalanced at least twice, and I know the tranq reflection issues and fixtures issues were resolved.


az account list-locations does not provide "ExtendedLocation" option per swagger spec

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

Related command
 az account list-locations
Describe the bug

Per Swagger spec, "list locations" should provide "includeExtendedLocations" parameter to list EdgeZones as part of the response.
Ref:
GET https://management.azure.com/subscriptions//locations?api-version=2021-04-01&includeExtendedLocations=true
Authorization: Bearer
To Reproduce

Expected behavior

az account list-locations should provide a parameter "includeExtendedLocatios" and should list the EdgeZones as part of the response.
Environment summary

Cloud shell.
PS /home/vishnu> az -v                                                                                azure-cli                         2.44.1
core                              2.44.1
telemetry                          1.0.8
Extensions:
ai-examples                        0.2.5
ml                                2.12.1
ssh                                1.1.3
Dependencies:
msal                              1.20.0
azure-mgmt-resource             21.1.0b1
Python location '/usr/bin/python3.9'
Extensions directory '/home/vishnu/.azure/cliextensions'
Extensions system directory '/usr/lib/python3.9/site-packages/azure-cli-extensions'
Python (Linux) 3.9.14 (main, Dec 15 2022, 01:12:13)
[GCC 11.2.0]
Legal docs and information: aka.ms/AzureCliLegal
Additional context

@jiasli for awareness

Per Swagger spec, "list locations" should provide "includeExtendedLocations" parameter to list EdgeZones as part of the response.

Could you please provide the source of this statement?
I searched for includeExtendedLocatios in the swagger repo but didn't see any result:
https://github.com/Azure/azure-rest-api-specs/search?q=includeExtendedLocatios

The swagger for 2021-04-01 API is https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Resources/stable/2021-04-01/resources.json
Public documentation of Subscriptions - List Locations REST API also has no reference regarding includeExtendedLocatios.
https://learn.microsoft.com/en-us/rest/api/resources/subscriptions/list-locations

But I do see Azure PowerShell has a similar argument -ExtendedLocation from Get-AzLocation: https://learn.microsoft.com/en-us/powershell/module/az.resources/get-azlocation?view=azps-9.3.0#-extendedlocation

Ops, apparently, I was looking for the wrong "droid" includeExtendedLocatios. It should be includeExtendedLocations. (I was just copy-and-pasting without checking).
The only place I see includeExtendedLocations is in 2021-01-01:
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/resources/resource-manager/Microsoft.Resources/stable/2021-01-01/subscriptions.json#L53
But the API version you provided is 2021-04-01. Could you please confirm which one is correct?
BTW, the document https://learn.microsoft.com/en-us/rest/api/resources/subscriptions/list-locations is still for 2020-01-01, which is certainly lagging behind the swagger. I have created an issue to track it:

https://github.com/Azure/azure-rest-api-specs/issues/22399

Hi @jiasli , is there any update on this issue? Any plans on when this is getting fixed?

Verified this issue is fixed with az version 2.56.0.
az account list-locations --include-extended-locations true

az account list-locations was moved from profile command module to resource command module and it supported --include-extended-locations at the same time: https://github.com/Azure/azure-cli/pull/27400.


Support customizable shortcuts

The existing shortcuts are good defaults but some might want something different.
Work required:

converting existing shortcuts to UserSetting entries stored in DB
settings page where people can add bookmarks -- maybe just storing the JS values similar to https://www.toptal.com/developers/keycode
figure out how to handle shift, control, etc
endpoint to read the javascript keycodes (or whatever) so that the mappings can be imported into lute.js
change the shortcuts template to read in the current shortcuts?  Or simply redirect to the settings?
acceptance test showing the change of the existing settings

Copied from #59 :
Current hotkeys are good, but some may collide with existing hotkeys (eg golden dict uses "c", but so does Lute).  Perhaps allow customizing.
Involves:

adding usersettings for the keys
form edits for the user settings
set the defaults for the existing keys
somehow handle the modifiers (shift, ctl, etc)
change the javascript switch-case or whatever to use the user settings -- likely render the user settings into the global namespace for js, and refer to that in the js
tests
keys must be set

This is helpful: https://www.toptal.com/developers/keycode
ChatGPT hackery:
data model in db
class KeyBinding(db.Model):
    id = db.Column(db.Integer, primary_key=True)
    user_id = db.Column(db.Integer, db.ForeignKey('user.id'))
    key_combination = db.Column(db.String(50), nullable=False)
    function_name = db.Column(db.String(100), nullable=False)

front end listener
let userKeyBindings = {};  // Will hold keybindings fetched from the server

document.addEventListener('keydown', function(event) {
    const key = `${event.ctrlKey ? 'Ctrl+' : ''}${event.altKey ? 'Alt+' : ''}${event.shiftKey ? 'Shift+' : ''}${event.metaKey ? 'Meta+' : ''}${event.key}`;
    if (userKeyBindings[key]) {
        const functionName = userKeyBindings[key];
        if (typeof window[functionName] === 'function') {
            window[functionName]();  // Call the predefined JavaScript function
        }
    }
});

// Example predefined function
function myCustomFunction() {
    console.log("Custom function called!");
}

get bindings
fetch('/api/keybindings')
    .then(response => response.json())
    .then(data => {
        userKeyBindings = data;  // Keybinding object: { "Ctrl+a": "myCustomFunction", ... }
    });

api route
@app.route('/api/keybindings')
def get_keybindings():
    user_id = current_user.id  # Assuming you're using Flask-Login
    keybindings = KeyBinding.query.filter_by(user_id=user_id).all()
    bindings_dict = {kb.key_combination: kb.function_name for kb in keybindings}
    return jsonify(bindings_dict)

Would also need a form etc for user to defined the bindings

Some javascript:
function getKeyString(event) {
    const keys = [];

    // Check for modifier keys
    if (event.ctrlKey) keys.push('ctrl');
    if (event.shiftKey) keys.push('shift');
    if (event.altKey) keys.push('alt');
    if (event.metaKey) keys.push('meta');

    // Map special keys to their names
    const keyMap = {
        'Enter': 'return',
        'Escape': 'esc',
        'ArrowLeft': 'left-arrow',
        'ArrowRight': 'right-arrow',
        'ArrowUp': 'up-arrow',
        'ArrowDown': 'down-arrow',
        ' ': 'space'
    };

    const key = keyMap[event.key] || event.key.toLowerCase();

    // If it's a normal key (not a modifier), add it to the keys array
    if (!['shift', 'ctrl', 'alt', 'meta'].includes(key)) {
        keys.push(key);
    }

    return keys.join('+');
}

// Usage: Attach this function to the 'keydown' event
document.addEventListener('keydown', function(event) {
    const keyString = getKeyString(event);
    console.log(keyString);  // For debugging, logs the key string
});

Started in branch  wip_issue_141_customizable_shortcuts

Done in develop, releasing as beta 3.5.7b2 today.

Launched in 3.5.7.


Rework polynomial hierarchy

We now have two polynomial implementation:

nttec::Polynomial which represents a polynomial with sparse coefficient (and is based on std::map
nttec::Vec::Poly which represents a "full" polynomial (with all the coefficients) and is based on a Vec.

The problems are:

nttec::Vec::Poly is a polynomial, not a vector (thus it doesn't belong to the nttec::vec namespace)
both classes represent the same thing (a polynomial) and have the same interface (but it's not enforced).

A more logical representation would be:

an abstract base class Polynomial that defines the interface of a polynomial (eval, derive, …)
a child class SparsePolynomial (that would correspond to the current nttec::Polynomial)
a child class FullPolynomial (that would be your new nttec::vec::Poly).

Also, inheritance represents an IS-A relations, and you can't really say that a polynomial IS-A vector (nor IS-A map) so it would make more sense to use composition here (having a member Vec the FullPolynomial class, rather than inheriting from it).

No longer relevant as we have removed nttec::Polynomial.


Use EncodeUri() if part of the URL path or EncodeUriComponent() if used as a URL parameter

https://github.com/bhensley/gtm-ninjatrack-integration-unofficial/blob/db9b0382e39850ed6082202328c2a348db89a249/template.tpl#L54C83-L54C95
When reviewing your template, we've noticed that it is using user-defined parameters to build a URL. It's best practice to use encodeUri() if part of the URL path or encodeUriComponent() if used as a URL parameter.

Thank you for the feedback, @semisight! I've gone ahead and updated the template to follow best practice here.


Fix style={styles.xxx} automatic completion failed with nested css file

Codecov Report

Merging #1004 (a33278f) into feat-update-style-heliper (18da9ce) will not change coverage.
The diff coverage is n/a.

@@                    Coverage Diff                     @@
##           feat-update-style-heliper    #1004   +/-   ##
==========================================================
  Coverage                      76.14%   76.14%
==========================================================
  Files                             29       29
  Lines                           1006     1006
  Branches                         206      206
==========================================================
  Hits                             766      766
  Misses                           237      237
  Partials                           3        3

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 18da9ce...a33278f. Read the comment docs.


CVE-2024-24790

The latest temporalio/admin-tools, temporalio/server and temporalio/ui images are using a version of Go that is impacted by CVE-2024-24790.
Updating Go to 1.21.11 or newer would fix the issue.

Thanks for reporting this.
The latest temporalio/admin-tools, temporalio/server, and temporalio/ui images are using a version of Go that is impacted by CVE-2024-24790.
Updating Go to 1.21.11 or newer would fix the issue.
Here is a way to work around the issue:

You can use older images of these tools.

Let us know if you have any other questions.


Change license to BSD 3 Clause and add DCO file

As per the Oct. 20, 2023 OFMF Working Group meeting, the BSD 3 Clause license and Developer Certificate of Origin certification were deemed most appropriate for this repository.  This updates the repository before we start adding new code.
Signed-off-by: Doug Ledford<EMAIL_ADDRESS>

Commits are now signed, but I also disabled the requirement that they be so.


feat: add string/base/last

Resolves #1696
Description

What is the purpose of this pull request?

This pull request adds the package @stdlib/string/base/last
This pull request:

resolves #1696

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

tests pass - this pr is ready for review @kgryte @Planeshifter