[TwigComponent] AnonymousComponentRegistry to share anonymous component in your bundles

Q
A

Bug fix?
no

New feature?
yes

Issues
Fix

License
MIT

Let's get crazy! And let the community share anonymous components through their bundles!
This PR introduces a new AnonymousComponentRegistry interface that lets your share the anonymous component inside your bundle

Hey @smnandre -
Really sorry you’re feeling that your efforts, info gathering and POCs are being ignored. I think that’s a fair criticism. Personally, due to time constraints, I have a hard time following all the conversations- especially longer, more technical conversations (and you often do deep research that involves full information). My guess is that this is probably true of others. From experience, I know that PRs (even flawed) tend to illicit more conversation than technical docs. For better or worse, we tend to react more to built things.
So, I hope that helps explain a bit at least. Your efforts around here are HUGE and appreciated. In fact, I think you may be sometimes thinking 3 steps ahead of everyone else. You also, I clearly know, do a lot of the unsung work - support, CI tweaks etc - that help make the project work. I’m not sure what workflow is best for discussing these more complex items, but I’m open to ideas. And, again, from experience, PRs are a powerful tool - even to show how an implementation almost works… but has some flaws. And you can always ping me - but you already know that ;)

Re: this PR.
I also feel that users should control the naming of components - not the bundle itself. I’m also not convinced that anonymous components need to be allowed for 3rd party packages.
What I think makes sense is a way to “import” a component from a package (and name it when you do) or import an entire namespace (allowing the bundle to provide names), but with an optional namespace prefix. FWIW, all of this would match how Blade components handle things.
Cheers!

I’m also not convinced that anonymous components need to be allowed for 3rd party packages.

I'm now thinking the same thing. At least until a time there is a significant performance boost between simple class-based components and anon components.
As it stands, 3rd party bundles have an avenue to auto-register there own class-based components. While not perfect, it works.

I'm thinking that, while bundles can register their own components, we should discourage that (we can't prevent it) and instead offer a nice "import component" option and encourage 3rd party components to be written in a way that makes them NOT automatically registered, but ready to be imported.
Having some YAML config would be the easiest way. But I really hate YAML. So another option, I think, could be a "component provider" system. User creates AppComponentProvider (or even adds an interface to their Kernel and uses that) with a registerComponents(ComponentRegistry $registry) where the can call things like $registry->add('Alert', VendorAlertComponent::class) or an ->addNamespace to register everything from a namespace with default names.

I'd vote for doing things automatically, .... and add a non-configurable bundle prefix.
So vendor/acme/blog-bundle/Twig/Components/SuperWidget.php  would be usable as
<twig:AcmeBlog:SuperWidget />

or (to be decided)
<twig:Acme:Blog:SuperWidget />

This is the more "obvious" / "standard" / "safe" way i think ....
If not, i like your provider idea, as long as they use a "prefix", "custom name", "namespace".
The alias idea is something we can (and will) do in a later move, but all this must be working without, so let's focus on that first.
As the alias must only be a userland thing, not something a Bundle controls.
And we want at first give bundle a way to expose components.

I'd vote for doing things automatically

This'd be my vote as well I think - would be very similar to twig templates in bundles. We would, I believe, still need a way for bundles to let the app know about their components. This could perhaps be the way they also let the app know about their anon components.


Password Reset vs. Unlock in SSPR

We need to make a distinction between the unlock process and password reset allowed in SSPR. Currently, Smart Lockout counters reset only after a full password reset. Account unlock will not work.
Please, update docs to reflect this.

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

ID: 9c7b90ca-a2b1-9cd8-9769-cd48c27f10ea
Version Independent ID: b8c29389-c435-b2f0-4186-1e9c47ab4f3e
Content: Preventing attacks using smart lockout - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-password-smart-lockout.md
Service: active-directory
Sub-service: authentication
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

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

Thanks! I made a pull request to add a note. https://github.com/MicrosoftDocs/azure-docs-pr/pull/103341


Audio/Visual Processing Node

Implements an audio/visual processing (AVP) node.
The current implementation subscribes to streams from the SFU and allows the user to run processing tasks on the stream. I've included a Recorder processor as an example. Eventually I would like to support replacing tracks in the SFU with processed versions. To support things like audio muxing and visual effects.
Please let me know what you think about the overall approach. Any suggestions to improve/generalize things greatly appreciated.

Closing this since we are reworking the approach


Layout Preference

What's the Problem?
No response
Solution/Idea
Hey, i love ur app, its beutifully minimal. I wouild like to suggest a new feature that'd allow u to have a prefence in the "Layout Mode". Every time that i create a new folder, the layout is not the one i like, so it would be an amazing feature having a favorite layout for all the folders.
Thanks!
Alternatives

Priorities

Files Version
<IP_ADDRESS>
Windows Version
Windows 10
Additional comment
No response

Thanks for the suggestion. Merging with #5564.
You can try the following:

turning off "enable individual preferences for folders" option in Files settings
setting your desired layout mode for any directory
turning back on "enable individual preferences for folders"


[imgui] Add Freetype feature

-Add Freetype feature, which use Freetype instead of stb_truetype to build the font atlases (https://github.com/ocornut/imgui/tree/master/misc/freetype).
-Add imgui_internal.h to public includes, because it is required to use the experimental API and improve debug capabilities. It's also required to build Implot (#11920).
-Add imgui_stdlib files to allow std::string usage with Imgui text functions (https://github.com/ocornut/imgui/issues/2035).
-Fix bindings feature requiring SDL1 dependency instead of SDL2 (Imgui SDL binding is using the latter).

I'd also like to suggest to remove or at least split the bindings feature. Imgui bindings are not really part of the library, and including them all in the same feature require many dependencies to be fetched, which is probably overkill in most cases. Also copying cpp files to the include folder should be avoided.

Thanks for the PR!

I'm not familiar with imgui, are the bindings just example code? Or what is their purpose?
I think having them as a non-default feature is good enough, some ports already have "examples" features that do the same.

These bindings are very well maintained, feature complete and the author recommend to use them by default except if a custom implementation is really needed : https://github.com/ocornut/imgui#integration
I think it's a good thing they are available as features for this port. What I dislike is the inability to pick only the required ones, it's all or nothing. Because of this, you are forced to acquire a lot of dependencies you will probably never need. It's also ugly because it isn't compiled to libs but rely instead on the manual inclusion of cpp files in your project, because some of them depends on platform specific APIs that can't be compiled everywhere.
Splitting these bindings to different features would solve these problems. The only drawback is that we would have a very long list of features :

Allegro5
Dx9
Dx10
Dx11
Dx12
Glfw
Glut
Marmalade
Metal
Opengl2
Opengl3
Osx
Sdl
Vulkan
Win32

I don't know if it's acceptable for ports to have such a long feature list, but if it is, it would be a better solution than the current one.


Fix issue with '_registeredWithBabel' check

Issue: Some addons adds ember-cli-htmlbars-inline-precompile multiple times in its babel options.
Reason: We have a check to handle this case  this._registeredWithBabel in index.js but the problem is that we might not always be dealing with the same this object. Even though the addon could be same the instance may not be same always.
Solution Check if the plugins already have ember-cli-htmlbars-inline-precompile in it before adding.

CI finished succesfully perhaps there is some sort of delay / issue in the status updates?


Consider renaming RPC schemes

Closes #1358
Introduced changes

Renaming some of the names of each model for the RPC.

Hi @Gerson2102 , I temporarily closed the issue as the renaming brings many breaking changes which aren't needed ATM.  We may reopen it in future, thanks.


Emacs from EVM cannot recognize png images

https://travis-ci.org/rexim/pacman.el/jobs/77326855#L1666
Derived from #33

Looks like they are compiled for console only. I don't know what we can do about it.

Ok, here is what I think after a couple of tea cups. They are just unit tests. They do not have to know anything about images. Let's make a convention that we do not cover pacman.el with unit tests. pacman.el should be a place where all the other modules are integrated.


add the option to pin in the notifications monitor

now that you have added to modal window open it is very nice to buy the pinned items if they are still available by just clicking on the search icon. It therefore makes sense to add the pin icon, because sometimes my limit is reached or I want to buy the product later or just to remember that I want to buy such a product

Good point!


add init function

fix: before_app_first_request deprecation
tests: json error message has been changed

@slint yes it is deprecated, but it is still in use. (example: invenio-github, invenio-app-rdm) if we don't maintain it, we move to flask>=2.3.


AppSetting widget in CMV develop version

Hi, @roemhildtg
Have you tried the appSetting widget in the CMV develop version? It seems to me i cannot click any of the checkbox

it has no error, but when look more into it, seems in file AppSetting.js
this function is never called
this.loadAppSettings();
Please see if you know some of the hint to resolve this.
Thanks

Can you post your config for the app settings widget?

Here is the config setting
        shareMap: {
            include: true,
            id: 'shareMap',
            position: 17,
            type: 'titlePane',
            canFloat: false,
            path: 'widgets/AppSettings',
            title: '  Save/Share Current Map',
            options: {
                //required:
                 map: true,
                 layerControlLayerInfos: true,

                 //optional (defaults shown):
                 appSettings: {
                    saveBasemap: {
                        save: true,
                        checkbox: true,
                        value: {},
                        urlLoad: false,
                        label: 'Save Current Basemap'
                    }
                 },
                 parametername: 'cmvSettings',
                 mapRightClickMenu: true,
                 address<EMAIL_ADDRESS>                 subject: 'Share Map',
                 body: 'Check out this map!
 ',
                 emailSettings: ['saveMapExtent', 'saveLayerVisibility', 'saveBasemap'],
                 shareNode: null,
                 shareTemplate: 'Share Map',
                 server: './widgets/AppSettings/php/index.php'
            }
        },

Okay, so your using the php script. Is there an ajax request occurring in your network log? If so, can you post the parameters/data retrieved from the network request?

dont have that info, but i got the following error

when i click the left checkbox, it does do something but the checkbox remains unchecked no matter what.

Can you post the stack trace on that Uncaught TypeError also? That would be helpful in diagnosing the issue

i dont see that error after i reloaded this viewer, (the viewer is Tim's version with WAB widgets plug in).

I'll take a look at that viewer later today and see if I can find out more details. Also just fyi, I updated the demo branch of this repo to use the latest cmv develop branch. https://github.com/roemhildtg/cmv-widgets/tree/demo I didn't see any problems but I didn't test with the php script.

yes, using the develop version, i can plug this share widget in, and it seems ok, but the issue is when i tried to open the map using something like url/?cmvSettings=4, it wont show/hide the layer and basemap like before. Please see if you can re-produce this and figure out what i did wrong. Thanks a lot!

looks there is some issue with the php, it log some records, but it is kind of empty.

Hm, I can't seem to reproduce. I downloaded the wab version that tim had and it doesn't seem to have an issue. I'll try testing some of the other widgets though.
{"mapExtent":{"center":{"type":"point","x":-11488671.598416623,"y":5079434.614815155,"spatialReference":{"wkid":102100,"latestWkid":3857}},"zoom":9},"layerVisibility":{"DamageAssessment":{"visible":false,"visibleLayers":[0]},"timeKansas":{"visible":false,"visibleLayers":[-1]},"Petrolium":{"visible":false},"wellFields":{"visible":false},"my_tiled_layer":{"visible":true,"visibleLayers":[1,2,3,4,6,7,8,9,10,11]}}}

@ERS-Long did you ever find a solution to this?

@roemhildtg I did not get chance to look more into it, if you cannot re-produce it, we can close it for now. I will try to dig into next time. Thanks!


expaneded gitignore

newer versions of automake create a cache folder that should not go into git.  The file 'compile' is generated per system by autogen.sh and thus should not be stored in git.

 I re-wrote your patch and applied it as a commit.


Feels like

For the past few updates I've been noticing that the feels like temperature in Fahrenheit does not match the feels like temperature in the tempest app I don't know whether it's a conversion issue I'm trying to establish a pattern but numbers seem to be so random this has been going on for quite some time but with the excessive heat it's become more and more obvious since I've been utilizing this

I don't really do anything with the Feels Like temperature. I just read the raw data and let Home Assistant do the conversion to Fahrenheit. There can be a delay in the numbers, but I don't assume the temperature is moving that much within a 5-10 minute interval.
So I am puzzled as well.


Work to ensure Travis CI passes

Fix chefspec tests (needs more work)
Correct actions in matchers
Clean up groups in Gemfile for bundler
More rubocop cleanup work

@martinb3  Our CI also started to break due to updating 2.0.1 now I made work around by changing Berksfile entry
cookbook 'firewall', git: 'git@github.com:opscode-cookbooks/firewall.git', ref: '3c4832f3498141287981a8687855531b0d746fc9'

But it helps a lot if version will be bumped up to 2.0.2
Thank you for this fix.

@joelhandwell I've released the latest changes under 2.0.2. Hope that helps!

@martinb3 thanks you!


v0.1.2

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/57528
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.2 -m "<description of version>" ae1c3ccccf60646872a0996d5d3f36a83a075b45
git push origin v0.1.2


miccall-theme add

miccall-theme add

name: MiccallTheme
description: A hexo theme with Gorgeous fullscreen background image
link: https://github.com/miccall/hexo-theme-Mic_Theme
preview: http://miccall.tech
tags:

responsive
card_layout
Home page
Blog
Custom pages
About page
simple

Thanks for the PR!


Add exposedtalent's file

Add exposedtalent's file
created a branch, created a file and made a commit, and opened a pull request

created a branch, created a file and made a commit, and opened a pull request


GitHub Education Benefits for Faculty

Hi @farhansj! Thank you for completing all of the assignments of the Campus Advisor training. However, it looks like you have not yet applied for GitHub Education benefits for faculty. If you are currently employed as faculty:

Apply for GitHub Education benefits for faculty at https://education.github.com/discount_requests/new.
After you are approved for faculty benefits, reapply for the Campus Advisor program. The application link is in the Campus Advisor offer in the GitHub Teacher Toolboox.

In the application, link to this training repository so that we can review the work you've already done.

However, if you are currently a student, you may be interested in the GitHub Campus Experts program instead.

Hi Arelia,
I already applied since 31st March 2020, but still pending for approval until now. Perhaps if you could help to have a look at my application and approve it if you have the authority to do so.

Regards

Hi @farhansj, it looks like you were approved on April 29. Congratulations! You'll receive an email soon to schedule an interview for the Campus Advisors program.

Hi @arelia , I am so sorry that I missed the interview slot. It’s quite challenging to wake up at 4am despite I had locked my alarm clock. Anyway, I am available now for the interview if you still have time.


[BUG] Error in uninstall job

Describe the bug (🐛 if you encounter this issue)
The below error appears in the logs for uninstall job for supportbundle api.
W1222 19:41:49.829202       1 reflector.go:324] github.com/longhorn/longhorn-manager/k8s/pkg/client/informers/externalversions/factory.go:117: failed to list *v1beta2.SupportBundle: supportbundles.longhorn.io is forbidden: User "system:serviceaccount:longhorn-system:longhorn-uninstall-service-account" cannot list resource "supportbundles" in API group "longhorn.io" at the cluster scope

To Reproduce
Steps to reproduce the behavior:

Install Longhorn v1.4.0-rc2
Uninstall Longhorn. using the manifest at https://github.com/longhorn/longhorn/blob/v1.4.0-rc2/uninstall/uninstall.yaml
Error appears in the logs as the supportbundles is missing from the apiGroups's resources

Environment

Longhorn version: v1.4.0-rc2
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl

Verified in longhorn master head images and v1.4.x head images with test steps
Result Pass
In both master-head and v1.4.x head, after uninstall complete, did not see error log related to supportbundles in job longhorn-uninstall


Uncaught SyntaxError: deno/acorn.js' does not provide an export named 'default' while installing snel

I'm getting above error while installing snel on WSL using following command:
deno run --allow-run --allow-read https://deno.land/x/snel/install.ts

sorry for the late response, this is a bundler(see) issue and more specific an esm.sh(see) issue

@borsemayur2  you can test from the dev branch where the problem is mitigated

It's working. Thanks


Add Kasami codes

Fixes #273

Codecov Report
Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (99e5e71) 77.65% compared to head (dd94423) 77.84%.

Files
Patch %
Lines

src/sdr/_sequence/_correlation.py
80.88%
13 Missing :warning:

Additional details and impacted files
@@                Coverage Diff                @@
##           release/0.0.x     #297      +/-   ##
=================================================
+ Coverage          77.65%   77.84%   +0.18%
=================================================
  Files                 56       56
  Lines               3290     3358      +68
=================================================
+ Hits                2555     2614      +59
- Misses               735      744       +9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


SEO Updates: Content

General Content:

Create a "Prediction Markets Explainer" on site post and add to the main landing page.
Review landing page content + add way more content (target 2500 words, currently at 500 but sites with ~2500 are the average for #1 rank on Google)
Get some blog posts out explaining things like prediction markets and for other keywords and get them highly ranked.
Regularly publish blog posts like 2-4 a month, both about informational stuff  and about engineering stuff.

Dev Content:

Get a bi-weekly development blog post written by an engineer. We've had this planned for a while, someone needs to step up and write the initial one. Should have at minimum, two a month. cc: @pgebheim

Ongoing


derive Debug for structs

So that their owners can just use #[derive(Debug)]!
This PR contains some format changes, but I think I'm using rustfmt with default settings.
Thanks :)

Thank you for the pull request


Update Test Distribution Gradle plugin to 2.1

Subject says it all

@bot-gradle test rFM, please

OK, I've already triggered ReadyForMerge build for you.

The CI Status is red but I assume the reason for this was that the master was red when I've created this PR. All the other checks are green. So I guess it's safe to merge?

@bot-gradle test and merge

OK, I've already triggered a build for you.


Dropdown menus with submenus like in top bar ?

Currently the dropdown menus that are native to the top-bar component are nice, support submenus, and behave very accessibly and nicely on small screens.
It would be nice if dropdown menus did the same. Maybe, the submenus + small screen code could make it to the dropdown component, and then the top-bar submenus could just be regular dropdowns then, instead of two different sets of components and feature sets ?

I would include it in the core as it is a very common and useful ui pattern. Any news?


fix: prepare release

@Bockiii on master branch this github workflow for documentation fails. can you help?
see https://github.com/RSS-Bridge/rss-bridge/actions/runs/7759542047/job/21163797094


Migrator for rmrkV1 & rmrkV2 to Assets Hub Polkadot & Kusama

@vikiival can expand on this

https://github.com/kodadot/nft-gallery/issues/7552


iPhone X Crash

When I try with iPhone X, the app crash saying this error :
Terminating app due to uncaught exception 'NSUnknownKeyException', reason: '[<UIStatusBar_Modern 0x7fb9cfd006a0> valueForUndefinedKey:]: this class is not key value coding-compliant for the key foregroundView.'
How should I handle this?

I am still experiencing the same crashes on iPhone X simulator with version 3.3.1. Can you fix it in the latest pods?


Issue creating s3 bucket on Mac

Hi, I'm attempting to install the nRF Asset Tracker into my AWS account. Upon reaching this command, I reach the following error.
npx cdk -a 'node dist/cdk/cloudformation-sourcecode.js' deploy
❌  nrf-asset-tracker-sourcecode failed: Error: nrf-asset-tracker-sourcecode: SSM parameter /cdk-bootstrap/hnb659fds/version not found. Has the environment been bootstrapped? Please run 'cdk bootstrap' (see https://docs.aws.amazon.com/cdk/latest/guide/bootstrapping.html)
I am running on a Mac. I also see the following warning beforehand.
current credentials could not be used to assume 'arn:aws:iam::...:role/...-us-east-2', but are for the right account. Proceeding anyway.
Direnv appears to properly exporting the AWS access tokens and I don't see any clear errors there. Has anyone seen a similar issue? Thank you!

It seems that the bootstrap part now with CDK v2 needs to happy for any CDK usage (previously it was only needed for the second stack, which has a larger template).
Can you run it in this order?
    # One-time operation to support large CloudFormation templates in CDK
    npx cdk bootstrap aws://`aws sts get-caller-identity | jq -r '.Account' | tr -d '\n'`/${AWS_REGION}

    # Create the S3 Bucket for publishing the lambdas
    npx cdk -a 'node dist/cdk/cloudformation-sourcecode.js' deploy

    # Deploy the example (see the note below)
    #   It will prompt:
    #     Do you wish to deploy these changes (y/n)?
    #   twice (the first is for the main stack, and the second is when deploying the web application stack)
    #   Both need to be confirmed with 'y'
    npx cdk deploy '*'

    # This is a fix for a bug with AWS CloudFormation and HTTP APIs
    # See https://github.com/bifravst/aws/issues/455
    node dist/cdk/helper/addFakeRoute.js

Thanks for the quick response! If I do that, I get this after the first command.
Error [ValidationError]: Stack with id nrf-asset-tracker-sourcecode does not exist

Alright, checking it now myself with a blank region.
This works:
npx cdk -a 'node dist/cdk/cloudformation-sourcecode.js' bootstrap aws://`aws sts get-caller-identity | jq -r '.Account' | tr -d '\n'`/${AWS_REGION}
npx cdk -a 'node dist/cdk/cloudformation-sourcecode.js' deploy
npx cdk deploy '*'
node dist/cdk/helper/addFakeRoute.js

Unfortunately this was not catched in the tests, because bootstrap is a per-account-and-region operation therefore the account we use for CI stays bootstrapped between CI runs. Right now it's not even possible to destroy the bootstrap resources.

@rohandkn kindly confirm fix.

Yes, that fixed it. Thank you!

Thanks!


[CONTROLLER/RECORDER] fixes gateway host cache bug

This PR is for:

Server

Fixes gateway host cache bug
Steps to reproduce the bug

learning additional gateway host

Changes to fix the bug

support loading vinterface's subnet id when is 0

Affected branches

main

Checklist

[ ] Added unit test to verify the fix.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Failed to list VLANs on your account. json: cannot unmarshal number into Go struct field Error.code of type string

when I run the ibmcloud sl vlan list command I get the following message
Failed to list VLANs on your account.
json: cannot unmarshal number into Go struct field Error.code of type string
Is this my account error or the cli error

which version you are using?  type 'ibmcloud -v'

ibmcloud version 0.8.0+50d0d38c-2018-07-11T06:52:10+00:00
this is the version that i have currently

<EMAIL_ADDRESS>pls look at this issue.

Hi @ezmoneySniper , this should already be fixed by SL API. I didn't reproduce this issue on my end. can you try again?
if you still have this issue, please post the trace log here by using: ibmcloud config --trace=true
https://github.com/IBM-Cloud/ibm-cloud-developer-tools/issues/53

Hi @ezmoneySniper , Thanks. I found the api endpoint you use was not right.
Normally, SL Cli will use https://api.softlayer.com/mobile/v3.1 as a endpoint to assess sl resource.
but in your log, sl cli use api.ng.bluemix.net, that was not correct.
Did you use ENV_SL_API_ENDPOINT="api.ng.bluemix.net" to set ENV_SL_API_ENDPOINT?
if yes, you can use export ENV_SL_API_ENDPOINT="https://api.softlayer.com/mobile/v3.1" or export ENV_SL_API_ENDPOINT="" to change the api endpoint right.
finally, the sl init output will like this:
Choose how to configure Softlayer authentication:
1. Login with Softlayer user name and password/API key
2. Use IBM Cloud Single-Sign-On
Enter a number> 2
API endpoint URL: [https://api.softlayer.com/mobile/v3.1]>

but this was indeed an issue with sl API, we will work on it to make it more robust.

Thank you @YouHang that solved my problem!


High Alch value on over while in item selection.

Title.

Maybe do like Konduit and while you hold down the "alt" key it shows high alch value of items?

I believe we already have this, look at the Item Prices plugin, however I believe there isn't functionality to hold alt to view prices, I think its either always on/off

Yes, it is already implemented. Just be sure "Hide Tooltips on Inventory Items" is unchecked in the config.

Killer, thank you! Loving the client, finding new features every day :) Got
an ETA on farm timers?
On Apr 25, 2018 12:01 AM, "Andrew Behncke"<EMAIL_ADDRESS>wrote:
Yes, it is already implemented. Just be sure "Hide Tooltips on Inventory
Items" is unchecked in the config.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/runelite/runelite/issues/1842#issuecomment-384154123,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ak32mBHy4ckMo-Cv5WCP9p-5gXTDtE_Pks5tr_UfgaJpZM4TgH7t
.


fix: go library cve

Signed-off-by: Jim Ma<EMAIL_ADDRESS>
Description

Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist:

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

Codecov Report

Merging #666 (ef5d27b) into main (f6b3927) will increase coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #666      +/-   ##
==========================================
+ Coverage   38.56%   38.66%   +0.10%
==========================================
  Files          88       88
  Lines        6976     6976
==========================================
+ Hits         2690     2697       +7
+ Misses       3906     3898       -8
- Partials      380      381       +1

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
38.66% <ø> (+0.10%)
:arrow_up:

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

Impacted Files
Coverage Δ

client/daemon/peer/peertask_base.go
33.67% <0.00%> (ø)

client/daemon/peer/peertask_stream.go
57.92% <0.00%> (+2.26%)
:arrow_up:


User research into exported report of current parks

This ticket is to determine which data we are storing should be exported / how the final export needs to be configured and who will be using this.
This would be used by the Manual Data Consumer user type that we identified in Discovery. They would take the exported info and use it to populate a system that is not able to connect to API.

For MVP, we just need to export a CSV of the full dataset. Consumers can then filter the data they need using Excel if necessary.
Checked with Glenna and Rob about this.


Add One Dark theme to theme-colors.rst

@esethna Would you like to review this or should this be reviewed by UX?


Update SITES.md

add new site  Thinking,Writing And Waiting

I close this. This PR is same with https://github.com/xianmin/hugo-theme-jane/pull/220 .


Updated Skyline Image with correct keystoneAuth URL in downloaded openrc

deployed to DFW to test and verify the keystone url in the openrc, LGTM

Deployment is done in SJC. Looks good as well.


ImportError: cannot import name SubfieldBase

i am trying to install saleor , when i run python manage.py migrate   it is giving this errorfrom django.db.models import SubfieldBase ImportError: cannot import name SubfieldBase
i have Django (1.11.10).. i read in some posts SubfieldBase support for Django (1.11.10) is deprecated. what do we need to do in this case?
i am unable to migrate the db. please help me

How did you solve this? @olahivepriyanka

How did you solve this? @olahivepriyanka


Copy track icon into language track repository

See exercism/exercism.io#2925 for more details.

Thanks!


Proper childId management

Original code was incorrectly nesting spans within each other, instead doing everything within scoped.

@niklaus0823 any chances for quick merging of this + npm publish?


DocSource: default value for required fields

Having a document:
object UserDoc : Document() {
    val groups by keyword()
}

we can describe a document source:
class UserDocSource : DocSource() {
    var groups: List<String>? by UserDoc.groups.required().list()
}

where you can see List<String>? type. We should always check groups field is not null before using it. If we make it required:
class UserDocSource : DocSource() {
    var groups: List<String> by UserDoc.groups.required().list().required()
}

it will throw exception when deserializing if value is missing.
So it is worth to make it possible to set default value for that case:
class UserDocSource : DocSource() {
    var groups: List<String> by UserDoc.groups.required().list().required(default = emptyList())
}

Propose following API:
class UserDocSource : DocSource() {
    var groups: List<String> by UserDoc.groups.required().list().default(::emptyList)
}

default argument for required method is misleading. On the one hand the value is required, but on the other hand it is not as there was specified a default value.
Also default method should take a lambda. In other case we could do:
class UserDocSource : DocSource() {
    var opinion: OpinionDocSource by UserDoc.opinion.default(OpinionDocSource().apply { rating = 0 })
}

// user1 and user2 share single opinion object
val user1 = UserDocSource()
val user2 = UserDocSource()

// user2.opinion is also changed
user1.opinion.rating = 99


Disable Radium console warnings

Hi, is there a way to disable the console.warn that Radium does when checking styles?
Thanks.

+1

+1

+1
Though for those looking for an alternate solution, you can filter warnings out of the console, if that suits your needs...

Hi folks,
Our two calls to console.warn have the "standard" guards of:
if (process.env.NODE_ENV !== 'production') {

just like React and the rest of our ecosystem of libraries have so that these disappear in your webpack frontend prod bundles using DefinePlugin and in node on the backend using NODE_ENV.
Is there an additional use case that we need to support here?


test(reports): adds unit test to the report receiver

Overview
This PR augments unit tests in the report receiver package, to secure forthcoming refactoring actions.

cc: @dwertent  @matthyx

@matthyx the report fixture that I've added is totally random. If someone has a meaningful report out there to be used as a based fixture (I have a method to "grow" the report to trigger report chunking thereafter), I'd be glad to replace the random stuff captured as a big meaningless JSON.

Hi @fredbi
I have replaced the fixture with some real data

Hi @fredbi
I have replaced the fixture with some real data
@Daniel-GrunbergerCA  cool really appreciate it.
could you please sign your commit so the DCO check passes?

Looks to me that I need yet another commit to solve conflicts.

question: can we also use fuzzing? I personally haven't used it
Fuzzying is interesting for a specific function. We could try that feature out in one of the dependency packages such as opa-utils or one of the deployed components that web-socket with each other. Just my opinion of course.

Here we just have a client sending stuff. IMHO, if something needs to be fuzzed, it's the server-side HTTP handler...

AFAIC we can merge this @dwertent

@fredbi should I merge this one too?

@matthyx  yes please do, it's in a final development state.
Again, I'll propose a factorization of the common test utilities in "internal/something"

Merci @matthyx


Implement normal_id_glm_lpdf in OpenCL

Summary
This implements normal_id_glm_lpdf() in OpenCL. A new matrix_cl<T> constructor was added that accepts a single scalar. from_matrix_cl() has two new template parameters that control compile-time row and column count of returned matrix.
There are also some superficial changes to non-OpenCL part of normal_id_glm_lpdf(). Order of some calculations have been changed so as to calculate everything only once and not to duplicate code between CPU and OpenCL parts. This should not affect computation on CPU in any way.
Tests
Existing normal_id_glm_lpdf() tests have been modified to always use GPU if compiling with OpenCL support.
Side Effects
None.
Checklist

[x] Math issue #1319

[x] Copyright holder: Tadej Ciglarič (University of Ljubljana)
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

Thanks! Really excited for this GPU GLMs.
This is the first GPU GLM PR so this one should be reviewed to some extent by more developers. GPU GLMs that will follow will then just have to be done by the principles we agree on here and will be easier to review.
A few notes before going in to make a deeper code review:

This changes the stan/math/prim/mat/prob/normal_id_glm_lpdf.hpp for cases even when STAN_OPENCL is not set. Not saying it should not but the reasons behind it should written down along with a quick explanation of the changes should be put in the top-level comment. Also dont include the /opencl headers if STAN_OPENCL is not set.

The CPU tests for the GLMs should always run. The way its setup now on this PR its either GPU tests if the flag is set or CPU if its not. This would decrease our test coverage here as CPU GLMs on Mac would not get tested ( the GPU tests are done on a Mac machine ). This will duplicate test code but thats better than worse test coverage. We really need to make a OpenCL test system like Bob did with the ad_test solution. But that is another issue/PR.

Could we move the relevant non-kernel GPU code in /opencl/prim/prob or something like that? And just make the regular function call the /opencl one if the conditions are met? I am working on a branch that will reorganize the OpenCL use in prim and rev to keep the OpenCL in /prim and /rev to a minimum. That will help us in the long run with the goal of reducing transfers by replacing /prim and /rev calls with /opencl

Please explain the tunings opts and how you got to the values.

Could we move the relevant non-kernel GPU code in /opencl/prim/prob or something like that?

I was trying to implement this in same way as other prim/rev functions that call OpenCL if the problem is large enough. It just turns out that GLMs are more complex due to high number of  template arguments so there is more extra code if it runs on OpenCL.
I see multiple options how to separate code that handles OpenCL stuff, but I don't really like any of them.
a) Just put kernel call in separate function. This will not do much to shorten OpenCL specific code and will have a huge number of arguments.
b) Pass matrix_cls, need_ ... variables and vector lenghts in. Return operands_and_partials and logp.  Still a huge number of arguments.
c) Separate normal_id_glm_lpdf_opencl() function with same templates and signature as normal_id_glm_lpdf(). Here we have some code duplication between CPU and OpenCL implementations.
d) Put everything that is within #ifdef STAN_OPENCL in separate function. That would probably make the most readable code. However in this case it is hard to define, what exactley the new function does and what it does not do.

Please explain the tunings opts and how you got to the values.

Calculations in GLMs can be separated in two parts. The first is matrix-vector multiplication. The time for that is proportional to number of instances times number of attributes. The weight for this part is normal_id_glm_coeff1. The second part are any operations that are done on result in this multiplication. The time for those is proportional to number of instances. The weight for this part is normal_id_glm_coeff2.
I wrote a program that measures GPU speedup for many problem sizes. It saves the dimensions of cases where the speedup was close to 1. Than I least-squares fitted the coefficients to these measurements. I ran measurements on Core i5 2500 and GTX 1070. Benchmark and the script for fitting (for all GLMs) are here: https://gist.github.com/t4c1/37958d2afcf6ec8d1282be4164f5c507
The graph below shows that data fits this model quite nicely (except for neg_binomial). Lines on graph are threshold defined by coefficients and croses are benchmarked dimensions that had speedup close to 1.

The CPU tests for the GLMs should always run.

I will tackle this after we decide how to split the OpenCL part in a separate function.

I dug in a little bit more and now I understand why the call to OpenCL is not going to be as straightforward. There really isnt a nice solution here...
How about:
a) have an ifndef STAN_OPENCL in the prim/mat/proband aifdef STAN_OPENCL` in /opencl/prim/prob.
b) have them same function in the stan::math::opencl namespace that is then called from the prim/mat/prob if STAN_OPENCL is defined.This does duplicate code but the code is also duplicated the way it is now just in the same function.

Now very little code is duplicated. Maybe a few if sentence header that decide whether certain partial derivatives need to be computed. However even some of these are different in OpenCL and CPU code.
Doing your a) would also duplicate all checks, using, value_of, as_column_vector_or_scalar calls, logp and some other computation. In total around 100 lines of code.
Your b) could duplicate less code, depending on what way we do it. Options I see are written in my earlier post here.

Is this ready for review? I can look it over tmrw

Now it is ready. I talked with @rok-cesnovar and we decided not to split off any part of the implementation in this PR. Maybe at a point when we do opencl/rev.

Cool! I'll give this a readthrough today

@serban-nicusor-toptal this PR is also not able to have clang-format changes pushed to it, @seantalts @syclik @bob-carpenter do either of you have the email username/pw for @stan-buildbot? It should have email inviting it to contribute to the bstatcomp math fork.
I'm gonna try to figure out why this is happening in general, the github permissions are on for owners of the main repo to make commits to the fork.

I don't have the password to @stan-buildbot. It's been changed since I set up that account.
I was able to see all the notifications, though, by logging into the registered email. That doesn't help without a valid login on that account.

Hey Steve, I don't have them either. Sorry

I was able to see all the notifications, though, by logging into the registered email. That doesn't help without a valid login on that account.

If no one has the password could we change it so we know it? I think the only overhead there is adding the PW to the Jenkins instance

I have reversed the encryption methods and decrypted the pass, will send you an email shortly.

@serban-nicusor-toptal much appreciated!!!


CORE-19436: Provision StateManager instance for flow status in the REST worker

Is the rest-worker the only worker type that needs to interact with the new flowStatus state type?.

Yes. It will be responsible for reading and writing to the new type.


fix(render/ids): set ad images for standalone and management clusters

Description

Set AD Images (AD Jobs, AD API) for standalone and management clusters only
For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

Closing so I can re-open to trigger CI.

/merge-when-ready delete-branch

OK, I will merge the pull request when it's ready, leave the commits as is when I merge it, and delete the branch after I've merged it.

I'm sorry but I failed delete the branch after merging the pull request.


[DOC]: Restructure the menu

What's wrong?
The biggest problem with our doc is this thing. The leftmost side is pretty unorganized.

How could it be better?
We have at best to spend 1-4 words for the left heading. We have used many. It seems disorganized. Every similar company follows this guideline. Besides, we need to establish a drawer as well.
Like them.
For this, we have to generalize our headings like other companies. People are used to reading this type of documentation. We should not reinvent the wheel and try to be as generalized as possible, in the case of heading.

Can we add extra detail -- what is the exact menu structure we want, what is missing etc.

what is missing
In the long run, most of the critical things are discussed in the docs. As preparing for launching, we should

what is the exact menu structure we want

Let's divide it into 2 parts. The upper part and the lower part of the menu.
My opinion is that let's take the upper part of the headings from general wisdom. Meaning - Getting Started, About SuperDuperDB, Installation, and other stuff. I will post 2 or 3 proposals that will resemble almost 80% of all documentation sites in the world. When people come into a documentation site for the first time, they look for these keywords in the left lane first. Like: Getting Started, Tutorial, Installation, About Us, Architecture, FAQs, User Guide, API reference, Glossary, Use Cases. Just like we look for the pricing page first if we go into any SaaS site!
Accommodating these keywords as the menu heading should be the first task. And moving the content
In the lower part of the documentation - here we have to be innovative in discussing how we want to present our ideas. The first task is to select our keywords. then organize it. I will propose some.
Then we should discuss.

Closed by #1074


Allow certain keywords to be used as identifiers

For some reason, SQLite allows a subset of the keyword list to be used as identifiers without wrapping double quotes.
Fixes https://github.com/rqlite/sql/issues/29

Thanks! :)


std::bad_alloc called if there's not a complete expression

When the interpreter is started, if just a number is entered we get a std::bad_alloc exception.
For example:
Fantastica! Ver <IP_ADDRESS> (C) 2016
=> 89
terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
Aborted
or:
=> 23 +
terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
Aborted

Fixed with commit 4de30f00a81ea8282f701dc942d7d70dbcf31bb6.  Marking as closed.


onChange is being called multiple times after submitting the form

onChange is being called multiple times after submitting the form
I propose to check diff of values instead of all formik object in
https://github.com/ramitos/formik-observer/blob/master/src/index.js#L19

See #2 if we can do something like that


Fix issue where the cask_opts env variable was not respected when running upgrade, reinstall, uninstall

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Process options using Cask::Cmd and pass the processed arguments to reinstall, uninstall, upgrade.
I'm pretty sure this fixes https://github.com/Homebrew/homebrew-cask/issues/87045. I inspected the verbose/debug output and found that quarantine was not being propagated, although I'm not sure what that means. Is there a robust way to detect whether a cask has been quarantined, so that I might write a test for this?

I added methods in env_config.rb for other potential HOMEBREW_CASK_OPTS flags as well

Thanks!

Thanks for the help @reitermarkus and for the work @whoiswillma!

@reitermarkus I heard through Mike that you were working on improving the frontend for brew cask with CLI::Parser. What's left to do on that front, and what are the ways that I can help?


Will DependencyAgent be updated to use AMA instead of Log Analytics agent

This page provides some guidance on using Azure Monitor Agent (AMA) over the Log Analytics (OMI) agent. Which sounds good to me - given the recent security issues with OMI, and difficulty I've had debugging and configuring it in the past, I'm happy to focus our efforts on an agent that will be maintained + supported into the future; and one that is presumably more secure and efficient.
From testing AMA in place of the OMI agent, the glaring gap is that if one wants to use VM Insights, the Dependency Agent is installed, which installs the Log Analytics agent/extension, which functionally overlaps with the AMA. And, it leaves us with potential security holes like https://www.wiz.io/blog/secret-agent-exposes-azure-customers-to-unauthorized-code-execution .
(Aside: The difficulty of identifying whether installed OMI extensions were vulnerable to these issues or not is what made evaluating migrating to AMA important.)
Can someone from Microsoft clarify whether the Dependency Agent + VM Insights will be migrated to use AMA instead of OMI, and what the timeframe is? If not, I don't see enough incremental value in adopting AMA over OMI.
Would also be good to update this page with that info; I'm sure it's an important bit of info for most companies deciding whether to migrate to AMA.

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

ID: bd3dca2f-629d-36e9-f0f0-8c7e64cc04ab
Version Independent ID: f40ee991-6cb8-1eee-9a35-ebc5833d7c7f
Content: Azure Monitor agent overview - Azure Monitor
Content Source: articles/azure-monitor/agents/azure-monitor-agent-overview.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @bwren
Microsoft Alias: bwren

@johncrim -  Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

Thank you - that's what I wanted to know. I think the new wording is clearer.


fix(connector): get discovery

Description
updated swagger summary and description.
Why
Description and summary was not providing clear picture of usage.
Issue
CPLP-2931
Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines

@Phil91 agree, its a nice improvement to enable an empty body


Store expires_at in archived_jobs

We need the expiry information to determine whether a job should be replayable.

Fixed by 92680e723e20fb73d9e9d9f01a780063f4756210.


adds the option to fine tune hub detr

closes #70

Codecov Report

Merging #71 into master will decrease coverage by 0.62%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
- Coverage   55.23%   54.60%   -0.63%
==========================================
  Files          49       49
  Lines        1586     1564      -22
==========================================
- Hits          876      854      -22
  Misses        710      710

Flag
Coverage Δ

#unittests
54.60% <ø> (-0.63%)
:arrow_down:

Impacted Files
Coverage Δ

mantisshrimp/core/bbox.py
63.26% <0.00%> (-4.01%)
:arrow_down:

mantisshrimp/metrics/metric.py
53.33% <0.00%> (-2.92%)
:arrow_down:

...antisshrimp/metrics/coco_metric/reference_utils.py
19.28% <0.00%> (-2.00%)
:arrow_down:

mantisshrimp/metrics/coco_metric/coco_metric.py
37.83% <0.00%> (-1.64%)
:arrow_down:

mantisshrimp/models/mantis_rcnn/mantis_rcnn.py
37.50% <0.00%> (-1.10%)
:arrow_down:

mantisshrimp/core/mask.py
80.20% <0.00%> (-0.98%)
:arrow_down:

mantisshrimp/core/category.py
85.00% <0.00%> (-0.72%)
:arrow_down:

mantisshrimp/parsers/parser.py
93.75% <0.00%> (-0.19%)
:arrow_down:

mantisshrimp/parsers/splits.py
95.65% <0.00%> (-0.19%)
:arrow_down:

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 0bd3643...d3b78f0. Read the comment docs.


Nearly identical code producing different results.

I've been following your react tutorial and I've spent over an hour trying to figure out why my code doesn't work but the code I copied from your repo does. When I use my code, I am unable to delete items from the todo list; it produces an error saying deleteTodo is undefined in the console. But when I run the code from the repo, it works perfectly fine. I ran it through diffchecker.com and I was unable to find any differences between both pieces of code except for some spacing and semi-colons. Anyone able to help me out I feel like I'm going crazy.
Code below is from my TodoList.jsx file.
import { TodoItem } from "./TodoItem"

// doesn't work
// export function TodoList({ todos, toggleTodo, deleteTodo }) {
//     return (
//

//             {todos.length === 0 && "No Todos"}
//             {todos.map(todo => {
//                 return (
//                     <TodoItem
//                         {...todo}
//                         key={todo.id}
//                         toggleTodo={toggleTodo}
//                         deleteTode={deleteTodo}
//                     />
//                 )
//             })}
//

//     )
// }

// // copied from repo
export function TodoList({ todos, toggleTodo, deleteTodo }) {
    return (

            {todos.length === 0 && "No Todos"}
            {todos.map(todo => {
                return (
                    <TodoItem
                        {...todo}
                        key={todo.id}
                        toggleTodo={toggleTodo}
                        deleteTodo={deleteTodo}
                    />
                );
            })}

    );
}

I'm checking the project and will update you.

Hey, was there ever any update to this? I'm getting a similar error where if there is one Todo in my list i'm able to check and uncheck it just fine. But when mulitple Todos are in the list it sudden doesn't work anymore and I get the "id is undefined" as an error, referencing the id prop from the todo item (todo.id)

{todos.map((todo) => {
try out this, the only thing missing in the code is the brackets of tod in todos.map.
In some, it does show an error if we don't give brackets!


lib/discordgo: add activity to message struct

Add a new MessageActivity type to the Message struct.
This MessageActivity type represents the activity sent with a message,
commonly referred to as an invite to a game session and such.
This is a suggestion on the support server, found here:
https://discord.com/channels/166207328570441728/356486960417734666/1125192216650977360
Signed-off-by: Luca Zeuch<EMAIL_ADDRESS>

any examples on how you'd use it, because it probably starts throwing  at you in most cases.
also upstream dgo has type MessageActivity using constants for Type field

any examples on how you'd use it, because it probably starts throwing
 at you in most cases.

If you mean usage in templates it's simply .Message.Activity --
though I think you meant a use case; such one was presented here:
https://discord.com/channels/166207328570441728/525535451839463424/1125219568453947412
Summarised the OP suggested it to be used to moderate such invites, as
there is no way to prevent them with Discord permissions and most bots
don't read them either.
(Though there could probably be an automod v2 trigger as well -- but
that's not in scope of this PR.)

upstream dgo has type MessageActivity using constants for Type field

I'll take a look at that once I'm free again, thanks for the pointers.
For now I'll convert this to a draft to prevent premature merging.
Luca

if you getMessage random message and .Activity over it, you will get
nil - this is in most cases, so not really that usable if 0.003% of
users are ever going to need it for template code.

I'm not sure if I'm 100% following you here — are you saying just
because it isn't set most of the time it should not be added, even
though it is documented in the Discord API docs?
In a handful of servers I'm in such invites are constantly sent, so I
wouldn't say it's that rare.
Additionally, I wouldn't say just because something is rarely used it
shouldn't be added — it's not like it's a huge amount of code or causing
issues for the bot (unlike .Guild.Members, which we deliberately do
not expose to custom commands — it's still present on the struct though)

Maybe adding it to amV2 as a rule to prevent such activity messages?

That is certainly a good idea, though it still would have to be added
to the message struct nonetheless, how else will we be able to read it
for automod purposes?
I'll look into implementing such a trigger with a decent configuration
interface on the weekend.

Maybe adding it to amV2 as a rule to prevent such activity messages?

While it is a rare usecase, I can come up with a reason why someone might want it to be exposed in the message object.
What if someone wants to maybe ping a specific role when someone sends an invite to a specific game?
What if they want to give people a role that sent an invite to a specific game?
Just random ideas that people might come up with as a usecase for activity to be exposed in message.

Maybe adding it to amV2 as a rule to prevent such activity messages?

While it is a rare usecase, I can come up with reasons why someone might want it to be exposed in the message object. What if someone wants to maybe ping a specific role when someone sends an invite to a specific game? What if they want to give people a role that sent an invite to a specific game?
Just random ideas that people might come up with as a usecase for activity to be exposed in message.

This doesn't tell you game details, just gives you activity type and party id.

This doesn't tell you game details, just gives you activity type and
party id.

party_id contains some marginal info on what the invite is about —
for example, a spotify "listen along" invite will have spotify:123456
as party_id, where 123456 is the ID of the user sending the invite.

I suggest we merge this change to the library and then, whenever we feel
like it, add a trigger to automod v2 (Advanced Automod), seeing as this
change will be necessary either way and I'm not a fan of having this
much logical separation in a single PR.


Update dead links in .github/CONTRIBUTING.md:

Hi guys! I noticed that some links in CONTRIBUTING.md are no longer working:

"this license agreement" link points to 404.
"reduced test case" also points to nowhere
"validate your HTML" link is dead

This is important, because new issue form has got a link to this file in an alert.
Thanks <З (hope you haven't forgotten about docs)

Closed #452


Br log backup checkpoint lag abnormal when there is long duration transaction

Bug Report

What version of TiKV are you using?
│/ # /tikv-server -V
│TiKV
│Release Version:   6.2.0-alpha
│Edition:           Community
│Git Commit Hash:   7824699f4e05feb3eb37b12265ecc1e26511a4d7
│Git Commit Branch: heads/refs/tags/v6.2.0
│UTC Build Time:    2022-08-11 13:49:18
│Rust Version:      rustc 1.62.0-nightly (7c4b47696 2022-04-30)
│Enable Features:   pprof-fp jemalloc mem-profiling portable sse test-engine-kv-rocksdb test-engine-raft-raft-e
│ngine cloud-aws cloud-gcp cloud-azure
│Profile:           dist_release
What operating system and CPU are you using?
K8S, 8C
Steps to reproduce

Start br log backup
Run workload for 10 hours, which consist of long duration transaction over 3 Hours

What did you expect?
Log backup checkpoint should move forward normally.
What did happened?
Checkpoint lag is abnormal.
For one testing where all transaction in different threads begin at almost the same time, checkpoint lag is like below

For another testing , where transaction in different threads begin at different time, it stucks for about 6 hours, then it is normal.

/type bug
/severity Major

/remove-label may-affects-4.0
/remove-label may-affects-5.0
/remove-label may-affects-5.1
/remove-label may-affects-5.2
/remove-label may-affects-5.3
/remove-label may-affects-5.4
/remove-label may-affects-6.0
/remove-label may-affects-6.1

/label affects-6.2


[Bug]: When I try to run normal Bing or Sydney I get an Invalid URL error.

Contact Details
<EMAIL_ADDRESS>What happened?
I installed the program exactly as instructed and inserted a full cookie string into BINGAI_TOKEN, but I still get this error. I've tried restarting the computer and reinstalling the browser and the command line, but unfortunately none of this has helped me. At first I was getting a slightly different error, namely /turing/conversation/create: failed to parse response body. I am using the latest version of this software at the moment (0.4.7).
Steps to Reproduce
Just go to regular Bing or its hacked version (Sydney).
What browsers are you seeing the problem on?
No response
Relevant log output
conversationId: '0952e7ce-51d3-40ac-bbba-f9e4379dc731'
}
bing options {
  conversationId: '0952e7ce-51d3-40ac-bbba-f9e4379dc731',
  context: null,
  systemMessage: null,
  parentMessageId: '00000000-0000-0000-0000-000000000000',
  toneStyle: 'creative',
  onProgress: [Function: wrapper]
}
TypeError [ERR_INVALID_URL]: Invalid URL
    at new NodeError (node:internal/errors:399:5)
    at new URL (node:internal/url:560:13)
    at new ProxyAgent (C:\chatgpt-clone-0.4.7\node_modules\undici\lib\proxy-agent.js:77:25)
    at BingAIClient.createNewConversation (file:///C:/chatgpt-clone-0.4.7/node_modules/@waylaidwanderer/chatgpt-api/src/BingAIClient.js:75:39)
    at BingAIClient.sendMessage (file:///C:/chatgpt-clone-0.4.7/node_modules/@waylaidwanderer/chatgpt-api/src/BingAIClient.js:184:62)
    at askBing (C:\chatgpt-clone-0.4.7\api\app\clients\bingai.js:72:34)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ask (C:\chatgpt-clone-0.4.7\api\server\routes\ask\askBingAI.js:137:20)
    at async C:\chatgpt-clone-0.4.7\api\server\routes\ask\askBingAI.js:75:10 {
  input: 'TRUE',
  code: 'ERR_INVALID_URL'
}

Screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I believe this is not a bug but an IP restriction
https://github.com/waylaidwanderer/node-chatgpt-api/issues/205
So the possible solutions/workarounds are:
set a proxy. (proxy param of BingAIClient or settings.js > bingAiClient > proxy)
make sure your cookies are up to date. Grab them by sending a message to Bing in the browser and copying the cookies from the /turing/conversation/create network request using Dev Tools.

Actually, I was having this issue too.
I updated to our dependency for bing's latest update
Then I needed to log into bing with edge, send a few messages, then get my full cookie string again. It's working now.
Pull the latest update for this project, made the change here: https://github.com/danny-avila/chatgpt-clone/pull/424

Actually, I was having this issue too.
I updated to our dependency for bing's latest update
Then I needed to log into bing with edge, send a few messages, then get my full cookie string again. It's working now.
Pull the latest update for this project, made the change here: https://github.com/danny-avila/chatgpt-clone/pull/424
I believe the headers needed to change

I realise that my question may be rather silly, but how can I pull this update?

No worries! Go into the directory you have the project in and open the terminal from there. Make sure the app is not running. You need git installed to do this but run the following from the directory and it will pull the latest updates.
git checkout main
git pull origin main

No worries! Go into the directory you have the project in and open the terminal from there. Make sure the app is not running. You need git installed to do this but run the following from the directory and it will pull the latest updates.
git checkout main
git pull origin main

You will need to run npm ci again after doing this.
Or you can download the project over again however you did before, and set it up as before.

I installed git in the project directory and ran the "git checkout main" command, but I get this error: fatal: not a git repository (or any of the parent directories): .git

No worries! Go into the directory you have the project in and open the terminal from there. Make sure the app is not running. You need git installed to do this but run the following from the directory and it will pull the latest updates.
git checkout main
git pull origin main

You will need to run npm ci again after doing this.
Or you can download the project over again however you did before, and set it up as before.

Answer me please...

How did you download the project in the first place? you may need to run git init then git remote add origin https://github.com/danny-avila/chatgpt-clone.git
as I said before, if you have issues, just download and setup as you did before

How did you download the project in the first place? you may need to run git init then git remote add origin https://github.com/danny-avila/chatgpt-clone.git
as I said before, if you have issues, just download and setup as you did before

I could of course just reinstall the project, but I have to figure out why my git commands aren't working? Can you please help me to set up this problem? I've tried to install git to both the standard directory (C:/Program Files) and the project directory (C:/cahtgpt-clone), but nothing works in either case. I always opened the command line in the project directory. Now I get this error: Reinitialized existing Git repository in C:/chatgpt-clone-0.4.7/.git/

Now I get this error: Reinitialized existing Git repository in C:/chatgpt-clone-0.4.7/.git/

This is not an error, this is just git telling you it already exists
Here's what you should do:

Open a command prompt.

Navigate to the directory where the project is located. For example, if the project is in a directory named chatgpt-clone on the C: drive, you would type cd C:/chatgpt-clone. Alternatively, open the project directory in your file explorer, right-click empty space, and select the option to open the terminal from there.

Run git init. This initializes a new Git repository in the current directory. Since the repository is already there, Git will tell you that it's reinitializing the repository, but that's okay.

Run git remote add origin https://github.com/danny-avila/chatgpt-clone.git. This sets up a link (called a "remote") between the local repository and the repository on GitHub.

Run git pull origin main. This will fetch the latest changes from the GitHub repository and merge them into the local repository.

If you're still having issues, you should start over in a different folder location with git:
git clone https://github.com/danny-avila/chatgpt-clone.git

@Kolyanpetuh If you aren't too familiar with git and you just downloaded the entire project directly as a zip (instead of cloning it), what you can do is retain a copy of your modified .env file and simply redownload the zip of the latest version and replace what you have.
Then, take the copy of your .env file and put it back where it belongs, and delete the example one.
@danny-avila can correct me if I'm wrong here but without needing to go into the arcane magic of git command line, that shoul do the trick.

@Kolyanpetuh If you aren't too familiar with git and you just downloaded the entire project directly as a zip (instead of cloning it), what you can do is retain a copy of your modified .env file and simply redownload the zip of the latest version and replace what you have.
Then, take the copy of your .env file and put it back where it belongs, and delete the example one.
@danny-avila can correct me if I'm wrong here but without needing to go into the arcane magic of git command line, that shoul do the trick.

 I didn't download the project as a zip, but unzipped it to the root of my system drive. But for some reason I still get this error.

I didn't download the project as a zip, but unzipped it to the root of my system drive

That would be a contradictory statement.
Did you clone it or download a zip?

I didn't download the project as a zip, but unzipped it to the root of my system drive

That would be a contradictory statement.
Did you clone it or download a zip?

To be honest, I don't really understand you. I downloaded the zip first and then unzipped the chatgpt-clone folder from there. I didn't create the project in zip format.

Since the main issue you brought up is resolved, I'm closing this. If you need help setting up the project further, you can open a Discussion under Troubleshooting or visit our discord server. @psarno thanks for supporting!


Page-Navigation Buttons not visible

Hello,
I have created a table which supposed to display some entries which it gets from a server.
The entries are showing just fine, as well the sizePerPage dropdown, but for some reason the page navigation buttons are not showing properly.

I have tried to change alter with the pagination factory options. The current configurations:
const options = {
            pageStartIndex: 1, // first page will be 0, default is 1
            paginationSize: 10,  // the pagination bar size, default is 5
            sizePerPage: rowsPerPages,
            showTotal: true, // display pagination information
            firstPageText: '<<', // the text of first page button
            prePageText: 'Prev', // the text of previous page button
            nextPageText: 'Next', // the text of next page button
            lastPageText: '>>', // the text of last page button
            nextPageTitle: 'Go to next', // the title of next page button
            prePageTitle: 'Go to previous', // the title of previous page button
            firstPageTitle: 'Go to first', // the title of first page button
            lastPageTitle: 'Go to last', // the title of last page button
            hideSizePerPage: false, // hide the size per page dropdown
            hidePageListOnlyOnePage: true, // hide pagination bar when only one page, default is false
        }

I am also attaching the code for the component which includes the table (both jsx and css).
ResultsPage.zip
Thank you for your time.
Alon

Any advices?


v3: Icon size gets overwritten by icon's width/height

Version
v3 latest commit
Description
The size-* class applied by a button gets overwritten by the icon itself.
Left one is in my project, right one is from the docs:

I couldnt reproduce in a fresh project, but my project is pretty clean still with no customizations. Might have something to do with the css order?

What version of @nuxt/icon are you using?

1.10.2

Found the issue:
It depends on the order of adding modules in nuxt config. I have 'nuxt-icon' added to modules, even tho I use nuxt ui, because it is part of a Nuxt layer. With this order the styles are overwritten as shown above:
modules: ['@nuxt/ui', '@nuxt/icon'],

You shouldn't put @nuxt/icon in your modules, @nuxt/ui installs it and configure it for you: https://ui3.nuxt.dev/getting-started/icons/nuxt#usage
https://github.com/nuxt/ui/blob/v3/src/module.ts#L107

Yes I know but it is part of a Nuxt layer, together with its configuration, used in multiple apps (only one of them is using Nuxt UI).

Why is @nuxt/ui in the layer if only used by one app then? 🤔
On the other apps, if @nuxt/ui is part of the layer it will be setup and will install @nuxt/icon anyway.

Sorry for missunderstandement. Nuxt-icon is used by all apps and needs to be configured in the same way for all of them. Nuxt UI on the other hand is only used by one single app which also needs the layer.

I see! Did changing the order of the modules solved the issue then?

Yes but I cant change the order because apparently the Nuxt layer comes first.
If there is no possibility on Nuxt UI side, I have to remove it from the layer or make an additional layer for non-Nuxt UI apps.
I have the thing with nuxt/fonts btw, added by the layer and then added by Nuxt UI again.

If the module comes before and you didn't override the config it shouldn't be a problem because we check if the module is already installed 🤔
https://github.com/nuxt/ui/blob/v3/src/module.ts#L99-L105

It seems you're missing the { cssLayer: 'components' } option, maybe you can try adding it to your layer?

I will try. Do you have a reference to this option? Cant find it.
In the end I guess the issue is only that tailwind css seems to come before icon css.

This is why this option is set, it's not documented no. https://github.com/nuxt/icon/blob/main/src/schema-types.ts#L67

Setting this in nuxt.config.ts did the trick:
icon: {
    cssLayer: 'components',
},


StaticBlob Location should be relative

For full support on #73 location should be relative.

Same for location and displacement of Pendulum


Should we only allow SubFlow state in the states def of ForEach state and Parallel state branches?

What is the question:
Currently we allow any number of states to be defined in ForEach state and Parallel state branches "states" definition.
This can become very verbose for bigger (or smaller) workflow definitions.
I would recommend that we instead only allow a single SubFlow state to be defined.
WDYT?

Fix handled by PR: https://github.com/serverlessworkflow/specification/pull/120

Fixed.


Support for DeBerta

Hi @kamalkraj , thanks for the suggestion. Let me add it to the list of models that were requested. We will keep you posted.

As more and more new models enter the market, we have prepared comprehensive instructions for TRT-LLM developers on adapting to new models of interest. We encourage our community developers to expand the range of supported models, fostering an open ecosystem with rapid iterations.
Please try following these instructions and let us know if you encounter any issues during the adaptation process. We greatly appreciate your dedication.

@laikhtewari


Error: The selected base file does not exist (C1)

Specifically C:\xxxxxxx\xxxxxx\downloads\ahk2exe-master\AutoHotkeySC.bin
When I drag ahk2exe.ahk in, it doesn't convert without displaying this message.
Any help?

I take it that you are trying to compile Ahk2Exe from the source code, rather than just using the released beta 9 Ahk2Exe.exe?
In order to compile Ahk2Exe correctly you need to supply an appropriate Base file from an AutoHotkey download, in order for this to succeed. The beta 9 release was made using AutoHotkeyU32.exe version <IP_ADDRESS> from here. If you wish to use an earlier release of AutoHotkey you will be restricted to using  a *.bin file as the Base file, and an installed version of AutoHotkey will be accessed as well.
Cheers

I’ve updated the Readme.md file to include the Base file requirement.


It is suggested to add some labels, such as "Orchard Core"

Many people can't find this project.

I see the tool and process of creating PO files is already described in the documentation (https://orchardcore.readthedocs.io/en/dev/docs/reference/modules/Localize/)


Device pixel ratio is missed

Mac OS with retina display:

I have a hackintosh, so I cannot test this. Does the triangle example look correct?

@maierfelix how can I test it?

@Reon90 What does the triangle example look like?
You can open a browser console and type in devicePixelRatio. Take this value and use it in the example. I'm not sure though how you would use it then, try playing around with it. I unfortunately never worked with Retina displays

@maierfelix the triangle example is the same.
I think we have to fix it in window creation by glfw.

@Reon90 found this in the GLFW FAQ here:
4.3 - Why is my output in the lower-left corner of the window?
You are passing the window size, which is in screen coordinates, to glViewport, which works with pixels. On macOS with a Retina display, and possibly on other platforms in the future, screen coordinates and pixels do not map 1:1. Use the framebuffer size, which is in pixels, instead of the window size. See the Window handling guide for details.

@Reon90 Latest nvk version 0.3.7 now exposes 3 new properties in VulkanWindow:

frameBufferWidth
frameBufferHeight
devicePixelRatio

Could you try replacing win.width -> win.frameBufferWidth and win.height -> win.frameBufferHeight?

@maierfelix yes it works.

@Reon90 Thanks for the feedback! Considering this closed


php dependency injection lib added

Thanks, I've done this via: https://github.com/ziadoz/awesome-php/pull/440


[Snyk] Security upgrade webpack from 5.75.0 to 5.76.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

736/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 8.3
Sandbox Bypass SNYK-JS-WEBPACK-3358798
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: webpack
    The new version differs by 34 commits.

5d64468 Merge pull request #16792 from webpack/update-version
67af5ec chore(release): 5.76.0
97b1718 Merge pull request #16781 from askoufis/loader-context-target-type
b84efe6 Merge pull request #16759 from ryanwilsonperkin/real-content-hash-regex-perf
c98e9e0 Merge pull request #16493 from piwysocki/patch-1
5f34acf feat: Add `target` to `LoaderContext` type
b7fc4d8 Merge pull request #16703 from ryanwilsonperkin/ryanwilsonperkin/fix-16160
63ea82d Merge branch 'webpack:main' into patch-1
4ba2252 Merge pull request #16446 from akhilgkrishnan/patch-1
1acd635 Merge pull request #16613 from jakebailey/ts-logo
302eb37 Merge pull request #16614 from jakebailey/html5-logo
cfdb1df Improve performance of hashRegExp lookup
4d561a6 Add test for behaviour of filesystem-cached assets with loaders
dfaa3b4 lint: remove trailing comma
dcc3e71 Serialize code generator data to support generated assets
b67626c Merge pull request #16491 from lvivski/main
d957cdf Fix formatting
6011163 Fix formatting
ea5e864 Fix HTML5 logo in README
2112f9b Replace TypeScript logo in README
5513dd6 Merge branch 'webpack:main' into patch-1
4b4ca3b Merge pull request #16500 from Jack-Works/avoid-cross-realm-object
4f39c9f fix: type error
c922ee1 chore: revert breaking change

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

This is a fork of a repo owned by Segment. We don't control dependencies in the upstream repo.


add notes to modelseed compounds with mappings to other resources

Add notes to initial modelseedpy model as derived from #5

seed.reaction is now missing while in the default .sbml is present.
For example, in the initial model:
 <rdf:li rdf:resource="https://identifiers.org/seed.reaction/rxn02201"/>


Check for __linux rather than linux in cpuid code and benchmarks

std=c11 drops the non-underscored macros, potentially breaking build-time cpu identification (in the benchmarks,
only seeding of the random number generator would have been affected).
fixes #2871 as suggested by Flamefire there

It seems that is not the full fix as the correct define is __linux__ according to https://stackoverflow.com/questions/6649936/c-compiling-on-windows-and-linux-ifdef-switch#comment79735932_6649992

gcc, clang and nvcc (formerly PGI pgcc) definitely provide both

Yes, I can confirm that too. But I don't have access to an  Ubuntu Trusty to verify what the others said (see links) so it might be possible, that on certain systems this could be an issue again. From the sources I could find the one with double underscore on both sides seems to be the safest bet.


how to define props that listen to browser url change

How do I write the definition to listen url of parent change. Here is how code it
props: {
        parentUrl: {
            type:       'string',
            required: false,
            default:  () => {
                return window.location.href;
            }
        }
    },

So whenever the location or uri or hash change, fn parentUrl will have current value of url and I can trace the change through the window.xprops.onProps. Is that possible?
related: #130

I would be tempted to set this up like so.
In the parent:
import { onUrlChange } from './lib';

MyComponent({
    onUrlChange: onUrlChange
})

In the child:
window.xprops.onUrlChange(newUrl => {
    ...
});

This way the child window can register a listener and react when the url changes. You would need to define a onUrlChange function which listens for changes and calls a callback though.
function onUrlChange(callback) {
    window.addEventListener('hashchange', () => callback())
}

Sure, the code you posted will work. And yes you can expose a function to the child window to cancel too.

how do I pass the callback from child?

In the parent:
function onUrlChange(callback) {
    const handler = () => callback();
    window.addEventListener('hashchange', handler);
    return {
        cancel: () => window.removeEventListener('hashchange', handler);
    };
}

const MyComponent = create({
   props: {
         onUrlChange: {
               type: 'function',
               required: false,
               value: () => onUrlChange
         }
   }
})

In the child:
const listener = xprops.onUrlChange(url => {
    // ...
});

listener.cancel();

thanks @bluepnume. Much appreciated


Labels editing and validation

RHOAIENG-6994
RHOAIENG-6995
Description

[
https://github.com/user-attachments/assets/3d4622f7-1874-4617-bc4c-e28f83faf65a
](url)
When going to model/version Detail, editing the labels are behaving as inline labels and also now have a validation for a case a new label already exists ( side not the background of the error message is red in paternfly )
How Has This Been Tested?

on the UI
Test Impact

Working on adding test while getting feedback on the implementation
Request review criteria:

Self checklist (all need to be checked):

[X] The developer has manually tested the changes and verified that the changes work
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[ ] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[X] Included any necessary screenshots or gifs if it was a UI change.
[X] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[ ] The developer has tested their solution on a cluster by using the image produced by the PR to main

@yih-wang Haley please take a look as well, and let me know what you think

@mturley @yih-wang I think I got it now. Updated the video.

Thanks for the updates @YuliaKrimerman! My only comment is when handling duplicate label errors, the newly created label should be marked in red to indicate the error. From the video, it looks like the existing label is flagged instead.

And one question to the loading time - from the video the newly added label takes a while to load after saving. Would this delay also occur in the production environment, or is it only in development environment? It’s a bit confusing to me before the label is loaded, so if this is expected in production, adding a loading state might help improve clarity.

Thanks for the updates @YuliaKrimerman! All look good now!
One thing that wasn’t part of the original design proposal but stood out to me as potentially confusing from the video - currently, after saving an edit, the label list keeps its original expanded/collapsed state. This means if the list was collapsed before editing, and the user enters the edit mode then saves any changes, the newly added label isn’t visible because the list stays collapsed. To avoid this potential confusion, it might be clearer to have the label list automatically expand after any edit is saved, regardless of its original collapsed/expanded state.
Anyway it doesn't belong to the original scope of the issue, and I don't know how much work it needs, so not sure whether we should create a new story to improve this experience separately in a future sprint @mturley

@YuliaKrimerman When I reviewed the validation pattern with Anges, she pointed out another inconsistency with the upcoming consistency guidelines - the ✓ should always remain enabled even when there're duplication errors. This provides better accessibility and aligns with the new form validation pattern.
Apologies for the adjustment from the original design, I think it's better to address this comment and change the ✓ behavior to be always enabled.

Hi @yih-wang @kaedward @mturley. Please take a look at the video recording. It has all the requested changes so far : Handling double duplications, proper error display based on amount of duplications and enabling the save button even when error exist. Also added some top padding above the red alert as Mike suggested (see screenshot, not video). Mike if you have a chance you can also do a re-review while I will work on fixing the last test, and brace for any incoming comments if needed.
https://github.com/user-attachments/assets/ce4bfed8-d39b-476c-a408-da627a19d74e

@YuliaKrimerman I'll do a full review asap, but just looking at that screenshot it looks like you lost the red color on the errored labels?

@mturley Haha no, it's just I screenshot with only the original showing, and not the duplicate. In the video the red is there :)

Ohh, I misread sorry!

Thanks @YuliaKrimerman, the latest updates look good!
The only question I have is do we decide whether or not address my earlier comment with this pr? Just make sure it's not overlooked.

One thing that wasn’t part of the original design proposal but stood out to me as potentially confusing from the video - currently, after saving an edit, the label list keeps its original expanded/collapsed state. This means if the list was collapsed before editing, and the user enters the edit mode then saves any changes, the newly added label isn’t visible because the list stays collapsed. To avoid this potential confusion, it might be clearer to have the label list automatically expand after any edit is saved, regardless of its original collapsed/expanded state.
Anyway it doesn't belong to the original scope of the issue, and I don't know how much work it needs, so not sure whether we should create a new story to improve this experience separately in a future sprint

@yih-wang Yeah I didn't address it as part of this issue. I think opening a separate story for this might be a good idea.

@yih-wang for posterity since this work was discussed privately - we did address your comment above in this PR about expanded/collapsed state when saving changes:

currently, after saving an edit, the label list keeps its original expanded/collapsed state. This means if the list was collapsed before editing, and the user enters the edit mode then saves any changes, the newly added label isn’t visible because the list stays collapsed. To avoid this potential confusion, it might be clearer to have the label list automatically expand after any edit is saved, regardless of its original collapsed/expanded state.

When the user saves their edited labels the labels will now become expanded.


Example with ViewQuery

Hi,
could you possibly add an example with ViewQuery? I am trying to implement one but don't know how to structure it. I am mostly lost regarding the result: QueryResult[ViewRow[T], NotUsed.
An example will certainly clarify things for me.
Thanks

Hi,
I guess you can do something like
val usersUnder43: Future[Seq[JsValue]] = bucket.searchView(
  ViewQuery("persons", "by_age", _.stale(Stale.FALSE).includeDocs(true).startKey(0).endKey(42)) ).flatMap(d => Source.fromFuture(d.doc)).asSeq

@joques does it work for you ?

@joques I'll close the issue for the moment but feel free to reopen it if you encounter issues


IPT set the profile field to data-package instead of actual one

Example:
{
  "version": "1.0",
  "profile": "data-package",
  "name": "camtrap-resource",
...
}

@mike-podolskiy90 is this a value that will be set in published Data Packages?

We migrated data packages in rs recently and IPT stopped to set the profile properly. Should be an actual value

👌 And the actual value is going to be a URL to the versioned profile at rs.gbif.org?

Must be this one for UAT:
https://rs.gbif.org/sandbox/data-packages/camtrap-dp/1.0/profile/camtrap-dp-profile.json

and this one for prod:
https://rs.gbif.org/data-packages/camtrap-dp/1.0/profile/camtrap-dp-profile.json

it still has experimental in the profile URL on UAT for some reason.. need to fix that


Magisch: Watch lpenamelwire.com

Magisch requests the watch of the watch_keyword lpenamelwire\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Ryan M in Charcoal HQ

lpenamelwire\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Fix Issue Tracker link in README

Fixes a small typo in the Issue Tracker link in the README. I also dropped the trailing slash since it doesn't seem necessary.

The slash is intentional, please only fix the typo.

Also, since this is a docs fix, make this against 2.0.x.

Done to both.


[buck-yeh-bux] Upgrade to 1.6.5

Bug/quirk fixes & enhancements with minor breakage of includes

[Bug] C_EZArgs::parse() misunderstands negative numbers as flag short names.
[Bug] C_ScannerImpl<>::add() may throw due to losing its track of the last non-null action function pointer.
[Quirk] C_EZArgs::help_full(): Keep "USAGE:" a empty line away from "DESCRIPTION:"
[Dependency] Update according to the reported issue related to vcpkg build.
[Obsolete] Remove ParaUtil.h(bux::C_NumIter<>) due to deprecation of std::iterator<>
[Change] Parameters trigger & parse of all C_EZArgs::add_flag() overloads now passed by rvalue references for better performance.
[Change] Loose the container parameter type requirements of C_PathFmtLogSnap::configPath(uintmax_t fsize_in_bytes,  const auto &fallbackPaths)
[Change] Replace DEF_CONFIGURABLE_LOGGER_FILES with DEF_FALLBACKABLE_LOGGER_FILES(first,...) to initialize the global logger in place.
[Change] Move HRTN() & OXCPT() from LogStream.h to StrUtil.h.
[Change] Rewrite I/O manipulator bux::timestamp utilizing fmt::format() to shorten the boilerplate code.
[Change] Quit assuming std::string::begin() is defined as const char * upon implementing ctor of bux::C_IMemBuf.
[Change] Harden the criteria to make flag values prefixed with '-' (negative numbers) hardly missed (i.e. misunderstood as short-named flags).
[Change] Claim that "Behavior of adding duplicate subcommands or flags of the same name is undefined." i.e it is caller's responsibility.
[Change] No more prepend ./ to the command name of USAGE part of help text.
[Change] Allow bux::scanFile() to provide encoding code of the input stream. The change has a virtue of preventing short strings from guessing the wrong encoding.
[Change] Were C_ScannerImpl<>::add() thrown, the incorporated message will have value of m_CurState.

What does your PR fix?
Nothing

Which triplets are supported/not supported? Have you updated the CI baseline?
Supported

x64-windows
x64-windows-static
x64-windows-static-md
x86-windows
x86-windows-static
x86-windows-static-md

No

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Thanks for the update!


new research spotlight PMID:30914423

Simple procedure for the generation of temperature-sensitive mutations with error-prone PCR

ALso
PMID:30914423
pretty much the same method

I have pushed both papers through together.
text:
Isolation of single amino acid t-s mutants in condensin hinge domain.

I have pushed both papers through together.

Should we have one Spotlight or two?  What was the PMID of the second paper?

ah sorry two.
PMID for second paper PMID:30914423

PMID for second paper PMID:30914423

Is that the PMID for the first image or the second image?

Good point :)
PMID:31042107 is the first

I've added those two Spotlights.


Added guidelines on file/dir names

This is not relevant, since we don't create product docs from the repos.


TrimReadsTrimmerPluginArgumentCollection should be package-protected

After #347, the TrimReadsTrimmerPluginArgumentCollection should be protected, because it should be used only for TrimReads. For other tests using it outside the package, we should implement a naive argument collection or move the tests to the correct package (to test integration).

In addition, TrimAndFilterPipelineUnitTest.testGetPipelineCompatibleWithGATKReadFilterPluginDescriptor should test only the GATKReadFilterPluginDescriptor. And this should be an integration test...


18.2R1.9, 21.2R1 Containers up but not ready (...)

Hi,
First of all big thumbs up for the project!
I'm trying to create a lab on Centos7 with the use of the above images. Both created and up however I'm not sure what's wrong.
When I run ./getpass.sh it will show dots long after the build (...) also getpass will prompt me for the pass but it seems whatever I enter 3 times it will always return the container info with (...)
i.e
[root@tds OpenJNPR-Container-vMX]# ./getpass.sh Password: Password: Password: vMX openjnpr-container-vmx-vmx2-1 (v4:<IP_ADDRESS> v6:) 18.2R1.9 GoigoiF5ushathung4hohWee          ... Password: Password: Password: vMX openjnpr-container-vmx-vmx1-1 (v4:<IP_ADDRESS> v6:) 21.2R1 loi8eihoL8bee6ic6yeey9ah    ...
Containers are up and I can ssh with lab user as seems it can not access default location for rsa:
WARNING: Can't read ssh public key file id_rsa.pub. Creating user 'lab' with same password as root
# docker container ps CONTAINER ID   IMAGE                                   COMMAND        CREATED          STATUS          PORTS                                                                                NAMES 9b06303c3de8   juniper/openjnpr-container-vmx:bionic   "/launch.sh"   40 minutes ago   Up 40 minutes   <IP_ADDRESS>:32790->22/tcp, :::32790->22/tcp, <IP_ADDRESS>:32788->830/tcp, :::32788->830/tcp   openjnpr-container-vmx-vmx2-1 3764e07c2ff3   juniper/openjnpr-container-vmx:bionic   "/launch.sh"   40 minutes ago   Up 40 minutes   <IP_ADDRESS>:32793->22/tcp, :::32793->22/tcp, <IP_ADDRESS>:32792->830/tcp, :::32792->830/tcp   openjnpr-container-vmx-vmx1-1
fpc on both images after some time of testing it's "Unresponsive" :
`lab@openjnpr-container-vmx-vmx2-1> show chassis fpc
Temp  CPU Utilization (%)   CPU Utilization (%)  Memory    Utilization (%)
Slot State            (C)  Total  Interrupt      1min   5min   15min  DRAM (MB) Heap     Buffer
0  Offline         ---Unresponsive---
1  Empty
2  Empty
3  Empty
4  Empty
5  Empty
6  Empty
7  Empty
8  Empty
9  Empty
10  Empty
11  Empty
lab@openjnpr-container-vmx-vmx1-1> show chassis fpc
Temp  CPU Utilization (%)   CPU Utilization (%)  Memory    Utilization (%)
Slot State            (C)  Total  Interrupt      1min   5min   15min  DRAM (MB) Heap     Buffer
0  Present          Testing
1  Empty
2  Empty
3  Empty
4  Empty
5  Empty
6  Empty
7  Empty
8  Empty
9  Empty
10  Empty
11  Empty
`
Attached are both container logs, please note the 21.2 EOF is different to 18.2
Once again I appreciate your work, please let me know what I could improve to get root access and on 21.2 probe devices log error msg. Also I have not included Yaml enviremental vars as this is not clear for me where and if I need to add it.
Was trying to run it on Rocky9 Linux as Centos7 & 8 are dead but had some issues, do you plan to update it for newer distros ?
Best regards
Bartek
Logs:
vmx1-1_v21.2.txt
vmx2-1_v18.2.txt
compose file :
`cat docker-compose.yml
version: "3.9"
services:
vmx1:
image: juniper/openjnpr-container-vmx:bionic
privileged: true
tty: true
stdin_open: true
ports:
- "22"
- "830"
environment:
- ID=vmx1
- LICENSE=license-eval.txt
- IMAGE=junos-vmx-x86-64-21.2R1-S1.3.qcow2
- PUBLICKEY=id_rsa.pub
- CONFIG=vmx1.conf
# - WAITFOR=eth4
- IO_CORE=2
- WORKER_CORE=3
- MASTER_CORE=4
- VMXT_CORE=5
#- HDDIMAGE=/images/p1.qcow2 # if we want it to be persistant
volumes:
# - $PWD/images:/images
- $PWD:/u:ro
- /var/run/docker.sock:/var/run/docker.sock
networks:
mgmt:
net-a:
net-b:
net-c:
vmx2:
image: juniper/openjnpr-container-vmx:bionic
privileged: true
tty: true
stdin_open: true
ports:
- "22"
- "830"
environment:
- ID=vmx2
- LICENSE=license-eval.txt
- IMAGE=junos-vmx-x86-64-18.2R1.9.qcow2
- PUBLICKEY=id_rsa.pub
- CONFIG=vmx2.conf
- IO_CORE=6
- WORKER_CORE=7
- MASTER_CORE=8
- VMXT_CORE=9
volumes:
- $PWD:/u:ro
- /var/run/docker.sock:/var/run/docker.sock
networks:
mgmt:
net-a:
net-b:
net-c:
networks:
mgmt:
net-a:
net-b:
net-c:
`

[vincentbernat] solved this his comments "https://github.com/Juniper/OpenJNPR-Container-vMX/pull/20" This should be addressed


Revised flag help hints

I felt the flag help hints could be improved. I've also edited the English of the README.
For the repo title, I would use:
Reverse proxy which automatically obtains TLS certificates from Let's Encrypt
Hope it helps.

Thanks for your help :)