Supply digest of images that are selected by policy

You sometimes want to use the digest of an image, rather than the tag; e.g., if you are interested in exactly reproducible builds.
For that reason, it'd be useful to supply the digest of an image selected by a policy object, as well as its tag, in the status. The digest appears to be available via https://godoc.org/github.com/google/go-containerregistry/pkg/v1/remote#Head (but if not, Get in the same place). This has to be done per tags, so while we don't need metadata for sorting/selecting, the policy controller can just fetch it for those images it selects.

I'd like to see this feature added too.
My use case would be that when using a tool such as Bazel to release from mono-repos, it's not really possible to use semver, as it would cause all of the services to be deployed every time a change was made. Instead I want to use timestamps as image tags, and the sha of the image in the deployments. This way I can use the alphabetical rule in the image policy, and set the sha. If the sha hasn't changed, Kubernetes won't deploy the app.

I'd like to see this feature added too.
My use case would be that when using a tool such as Bazel to release from mono-repos, it's not really possible to use semver, as it would cause all of the services to be deployed every time a change was made. Instead I want to use timestamps as image tags, and the sha of the image in the deployments. This way I can use the alphabetical rule in the image policy, and set the sha. If the sha hasn't changed, Kubernetes won't deploy the app.


Plugin does not work with newest sentry version of Sentry On-Premise Docker setup

After installing plugin via adding sentry-auth-gitlab to requirements.txt and rebuilding.
The web, worker, cron and post-process docker containers are constantly restarting.
The docker logs for the web service give this error:
Traceback (most recent call last):
File "/usr/local/bin/sentry", line 8, in <module>
  sys.exit(main())
File "/usr/local/lib/python2.7/site-packages/sentry/runner/__init__.py", line 164, in main
  cli(prog_name=get_prog(), obj={}, max_content_width=100)
File "/usr/local/lib/python2.7/site-packages/click/core.py", line 722, in __call__
  return self.main(*args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/click/core.py", line 697, in main
  rv = self.invoke(ctx)
File "/usr/local/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
  return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python2.7/site-packages/click/core.py", line 1066, in invoke
  return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/usr/local/lib/python2.7/site-packages/click/core.py", line 895, in invoke
  return ctx.invoke(self.callback, **ctx.params)
File "/usr/local/lib/python2.7/site-packages/click/core.py", line 535, in invoke
  return callback(*args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/click/decorators.py", line 17, in new_func
  return f(get_current_context(), *args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/sentry/runner/decorators.py", line 68, in inner
  return ctx.invoke(f, *args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/click/core.py", line 535, in invoke
  return callback(*args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/click/decorators.py", line 17, in new_func
  return f(get_current_context(), *args, **kwargs)
File "/usr/local/lib/python2.7/site-packages/sentry/runner/decorators.py", line 29, in inner
  configure()
File "/usr/local/lib/python2.7/site-packages/sentry/runner/__init__.py", line 127, in configure
  configure(ctx, py, yaml, skip_service_validation)
File "/usr/local/lib/python2.7/site-packages/sentry/runner/settings.py", line 158, in configure
  skip_service_validation=skip_service_validation,
File "/usr/local/lib/python2.7/site-packages/sentry/runner/initializer.py", line 308, in initialize_app
  django.setup()
File "/usr/local/lib/python2.7/site-packages/django/__init__.py", line 27, in setup
  apps.populate(settings.INSTALLED_APPS)
File "/usr/local/lib/python2.7/site-packages/django/apps/registry.py", line 85, in populate
  app_config = AppConfig.create(entry)
File "/usr/local/lib/python2.7/site-packages/django/apps/config.py", line 94, in create
  module = import_module(entry)
File "/usr/local/lib/python2.7/importlib/__init__.py", line 37, in import_module
  __import__(name)
File "/usr/local/lib/python2.7/site-packages/sentry_auth_gitlab/__init__.py", line 5, in <module>
  from .provider import GitLabOAuth2Provider
File "/usr/local/lib/python2.7/site-packages/sentry_auth_gitlab/provider.py", line 3, in <module>
  from sentry.auth.providers.oauth2 import (
File "/usr/local/lib/python2.7/site-packages/sentry/auth/providers/oauth2.py", line 9, in <module>
  from sentry.auth.provider import Provider
File "/usr/local/lib/python2.7/site-packages/sentry/auth/provider.py", line 6, in <module>
  from .view import ConfigureView
File "/usr/local/lib/python2.7/site-packages/sentry/auth/view.py", line 5, in <module>
  from sentry.plugins.base.view import PluggableViewMixin
File "/usr/local/lib/python2.7/site-packages/sentry/plugins/base/__init__.py", line 3, in <module>
  from .bindings import BindingManager
File "/usr/local/lib/python2.7/site-packages/sentry/plugins/base/bindings.py", line 5, in <module>
  from sentry.plugins import providers
File "/usr/local/lib/python2.7/site-packages/sentry/plugins/providers/__init__.py", line 5, in <module>
  import_submodules(globals(), __name__, __path__)
File "/usr/local/lib/python2.7/site-packages/sentry/utils/imports.py", line 45, in import_submodules
  module = __import__(module_name, globals(), locals(), ["__name__"])
File "/usr/local/lib/python2.7/site-packages/sentry/plugins/providers/base.py", line 7, in <module>
  from social_auth.models import UserSocialAuth
File "/usr/local/lib/python2.7/site-packages/social_auth/models.py", line 12, in <module>
  from .fields import JSONField
File "/usr/local/lib/python2.7/site-packages/social_auth/fields.py", line 10, in <module>
  from sentry.db.models.utils import Creator
File "/usr/local/lib/python2.7/site-packages/sentry/db/models/__init__.py", line 3, in <module>
  from .base import *  # NOQA
File "/usr/local/lib/python2.7/site-packages/sentry/db/models/base.py", line 37, in <module>
  class BaseModel(models.Model):
File "/usr/local/lib/python2.7/site-packages/django/db/models/base.py", line 110, in __new__
  app_config = apps.get_containing_app_config(module)
File "/usr/local/lib/python2.7/site-packages/django/apps/registry.py", line 247, in get_containing_app_config
  self.check_apps_ready()
File "/usr/local/lib/python2.7/site-packages/django/apps/registry.py", line 125, in check_apps_ready
  raise AppRegistryNotReady("Apps aren't loaded yet.")
django.core.exceptions.AppRegistryNotReady: Apps aren't loaded yet.

it seems to me that the plugin isn't compatible anymore with the django version used in sentry?

no activity for 14 days - did you possibly find a solution ?

no activity for 14 days - did you possibly find a solution ?
No, without support from the dev I gave up on it, since it's just a nice to have feature.

nice to have feature
I can't follow that. Authentication coupled to a directory like LDAP in a corporate environment is crucial.
Without it, every user would have to register a loose account on sentry.
I can't believe this is the way everyone is working ^^

it seems to me that the plugin isn't compatible anymore with the django version used in sentry?

That is exactly the case here.
I made it work for our on premise installation in my fork https://github.com/cirmaciu/sentry-auth-gitlab There is big BUT though. I am not python developer. I just changed files so the plugin works. It is not tested at all and plugin has to be installed locally.
I would be happy if anyone could get inspired there and make it to actual pull request here.

I just tested it - basically, the Gitlab<->Sentry connection works again, the Login Page now provides the Button/Link to Gitlab – there you login with your user account, and the callback/redirect back to Sentry also works, BUT
Within Sentry, I am magically logged in as superuser, and no account has been created for me (me speaking as the user which I logged into Gitlab).
I have deleted all users and members. The superuser is the only account left.
Do you encounter the same situation?

are you sure ??? I followed the author's steps but could not achieve it, can you share your operation?


FATAL: Module fuse not found in directory /lib/modules/4.4.0-43-Microsoft

Hi,
I am experiencing a problem trying to mount a webdav share with dav2fs under WSL. Am I overlooking something?

$ sudo mount ~/surfdrive
Please enter the username to authenticate with server
https://surfdrive.surf.nl/files/remote.php/nonshib-webdav/ or hit enter for none.
  Username: ********
Please enter the password to authenticate user ******** with server
https://surfdrive.surf.nl/files/remote.php/nonshib-webdav/ or hit enter for none.
  Password: ********
/sbin/mount.davfs: loading kernel module fuse
modprobe: ERROR: ../libkmod/libkmod.c:586 kmod_search_moddep() could not open moddep file '/lib/modules/4.4.0-43-Microsoft/modules.dep.bin'
modprobe: FATAL: Module fuse not found in directory /lib/modules/4.4.0-43-Microsoft
/sbin/mount.davfs: loading kernel module fuse failed
/sbin/mount.davfs: waiting for /dev/fuse to be created
/sbin/mount.davfs: can't open fuse device
/sbin/mount.davfs: trying coda kernel file system
/sbin/mount.davfs: no free coda device to mount

/sbin/mount.davfs: loading kernel module fuse failed

#17, #2163, User Voice.


Fix anchor link offset with fixed header

Anchor links overlaps behind the fixed header on all viewport except small. I fixed it by matching paddingTop with the header height on different viewport.
Open this URL for example in a new tab/window to see the issue.

This fix work only for h2.
I propose deepest changes for all headers https://github.com/reactjs/reactjs.org/pull/1855

@EPolovniy you're right, however making the rule more generic with all headlines should be relatively easy, I will revise it.
I will have a look at your PR too, not straightforward from first look!

#1944 I created  more clear request for this problem. Review my pull request pls!

#1944

@EPolovniy @lex111
My the new changes in https://github.com/reactjs/reactjs.org/pull/1914/commits/88f04250a1c188b090956adb1063dfeea2a8ebbb is now handling both h2 h3 or even nested h3 inside h2.
You can clone and run locally to test if needed! because I was expecting @reactjs-bot to create a new build for https://github.com/reactjs/reactjs.org/pull/1914/commits/88f04250a1c188b090956adb1063dfeea2a8ebbb so you review my fixes live, but it didn't!

Also I noticed some inconsistency of using h3 directly after h1 instead of h2, examples:

https://reactjs.org/docs/introducing-jsx.html
https://reactjs.org/docs/lists-and-keys.html

You will also notice that the border between sections is not there as in other pages, for example:

https://reactjs.org/docs/lifting-state-up.html
https://reactjs.org/docs/forms.html

However that's out of this PR scope. I will adjust that in a following PR.

because I was expecting @reactjs-bot to create a new build for 88f0425 so you review my fixes live, but it didn't!

In fact, it is. Your new changes are available at the same link!

There is a problem with h4.
https://deploy-preview-1914--reactjs.netlify.com/docs/faq-functions.html#bind-in-constructor-es2015

@lex111 nice! maybe it takes few minutes, because I checked right after I pushed my changes wasn't there.
I was wondering if h4 is being used as an anchor, good catch, will update it now!

@abdelrahmanrifai I've been thinking, maybe there is another solution to the problem. For example, look please this one https://stackoverflow.com/questions/4086107/fixed-page-header-overlaps-in-page-anchors/28824157#28824157

@lex111 looking into applying a solution with :target, looks more simple and consistent!

@abdelrahmanrifai now let's see it in action! :yum:

@lex111 I tested both :target and :target::before, none worked well because:
Target elements h2, h3, and h4 already have different margin and padding applied that is part of the design, and sometimes even we have special conditions like here
To apply this :target trick, it will need much more refactoring efforts, that I'm not sure will work well.
I see the current status of my PR works good enough to fix the problem.

Another drawback related to :target: it only work on click, not when opening the URL in a new window/tab.

@abdelrahmanrifai okay, but what about the alternative?

@lex111 I see MDN, CSS-Tricks and Stack Overflow are using same technique as in my PR.
Actually that technique is already applied for reactjs.org only for mobile, my PR is just adding more media queries and using corresponding header heights + 5px so the headlines don't touch header edge.
I find it so interesting issue, I'd spend more time researching it later. However I still see my PR is good enough for now, what do you think?

This PR change margins between blocks.

@EPolovniy yes it does add few pixels more, however nothing harmful by any means. In contrast it's better for readability check CSS-Tricks for example. It can be addressed and get decreased as it was before if required anyway.

@abdelrahmanrifai note that the margin of the h4 tag has changed:
https://deploy-preview-1914--reactjs.netlify.com/docs/faq-functions.html#how-do-i-pass-an-event-handler-like-onclick-to-a-component
https://reactjs.org/docs/faq-functions.html#how-do-i-pass-an-event-handler-like-onclick-to-a-component

@lex111 Thanks! I just noticed some special conditions flaws too with :first-child or :first-of-type, addressing them all now.

@EPolovniy @lex111 My PR is ready to be reviewed now!
No single UI change should be there now, same distance above all heading. 🙌

@abdelrahmanrifai this PR is ready for merge?

@lex111 Yes it's ready!
I just noticed this so tiny extra pixel in my previous screenshot. that's what this pixel perfection commit all about!
You may need to zoom to see it!

@abdelrahmanrifai :confused:
Before:

After

@abdelrahmanrifai and @EPolovniy, thank you so much both for your help! :hugs:


fix: call str() or prettyPrint() before sending data to MIB-server

:tada: This PR is included in version 1.0.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Microsoft.VisualStudio.Interop

Hello, could you tell me why do you reference the "Microsoft.VisualStudio.Interop.dll"? Thanks! :)

If I remember correctly, that is the interop into visual basic, and not a
DLL for doing interop with other applications from visual basic.
I believe that DLL can get selected when you need to do some things in WPF
and it's interchangeable with several nuget packages that have since
replaced that functionality.
I would say delete the line and see what breaks, or if you want I can look
at what is using it and what replaces that but it'll probably be a few days.
On Sat, Apr 8, 2023, 2:32 AM patrickiel @.***> wrote:

Hello, could you tell me why do you reference the
"Microsoft.VisualStudio.Interop.dll"? Thanks! :)
—
Reply to this email directly, view it on GitHub
https://github.com/PockyBum522/template-inventor-core-addin/issues/1,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPBGD2CBPU76QFEDJDJ6W3XAEBANANCNFSM6AAAAAAWXIYGTA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks for the answer! According to the documentation (https://learn.microsoft.com/en-us/visualstudio/extensibility/visual-studio-sdk?WT.mc_id=visualstudio_extensibility_aka_VSExtensibility&view=vs-2022)

Visual Studio interop assemblies allow managed applications to access the COM interfaces that provide Visual Studio extensibility.

I have not tried it yet, but it looks to me like it is not relevant and can be deleted. I have now deleted this reference, we will see... :)


Support for NetFlow Version 9

Closes #6

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: joschi:white_check_mark: lennartkoopmann:x: bhklimkYou have signed the CLA already but the status is still pending? Let us recheck it.


How to configure MD012

Really sorry for the basic question... For me MD012 looks too crowded, I like a lot of vertical whitespace ;-)
When I look here:
https://github.com/DavidAnson/markdownlint/blob/main/doc/md012.md#md012---multiple-consecutive-blank-lines
I see that I can increase the default maximum, but I can't work out where to do it...
Many thanks,
Dan.

That documentation is here: https://github.com/DavidAnson/markdownlint#optionsconfig
If you are using the library through a CLI, you will probably want to create a .markdownlint.json(c) file instead: https://github.com/DavidAnson/markdownlint-cli2#configuration
A complete example can be found here: https://github.com/DavidAnson/markdownlint/blob/main/schema/.markdownlint.jsonc

Thank you. Where do I put that file? (Sorry!) I'm using vscode.
On Fri, 27 Oct 2023, 4:59 pm David Anson, @.***> wrote:

That documentation is here:
https://github.com/DavidAnson/markdownlint#optionsconfig
If you are using the library through a CLI, you will probably want to
create a .markdownlint.json(c) file instead:
https://github.com/DavidAnson/markdownlint-cli2#configuration
A complete example can be found here:
https://github.com/DavidAnson/markdownlint/blob/main/schema/.markdownlint.jsonc
—
Reply to this email directly, view it on GitHub
https://github.com/DavidAnson/markdownlint/issues/1023#issuecomment-1783155376,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA7NE6XRXH5523MAYENJV3YBPK7JAVCNFSM6AAAAAA6SVHQF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGE2TKMZXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

That's described here: https://github.com/DavidAnson/vscode-markdownlint#markdownlintconfig
Typically the root of the project.


Adding extra search term functionality in JS prior to ajax call.

USER STORY: As a user, I want to be able to access specific information regarding my diet or allergen requirements.
Acceptance Criteria:
GIVEN: I want recipes that follow my diet and other guidelines,
WHEN I click a button that correlates with my diet
THEN: I want to be able to access a list of recipes that follow my diet and other guidelines.

The allergen criteria is complete.  We can search for no eggs and dairy (sorry diet vegan), no tree nuts, and no peanuts.


Proposal: Re-export packages with shorter names (g, h, c)

The problem:
Right now in the docs it is recommended to import library packages with aliases:
package main

import (
	g "github.com/maragudk/gomponents"
	c "github.com/maragudk/gomponents/components"
	. "github.com/maragudk/gomponents/html"
)

func MyPage() g.Node {
	return h.Main(
		// ...
	)
}

The problem comes when the file is saved, if an import is not being used then go fmt deletes it.
This is a problem because to import packages with aliases there is usually no help from the editor, so you must memorize the import, copy and paste it or create snippets to import the packages.
As you can well imagine, this kills productivity and completely takes you out of the flow state.
Proposed solution:
My proposal to solve this problem is to simply re-export the functions of the 3 main packages so that our import looks like this:
package main

import (
	"github.com/maragudk/gomponents/shortexport/g" // For main package
	"github.com/maragudk/gomponents/shortexport/c" // For components package
	"github.com/maragudk/gomponents/shortexport/h" // For html package
)

func MyPage() g.Node {
	return h.Main( // auto imported when file is saved
		// ...
	)
}

This would attack the problem directly by allowing our code editor tools to auto-import the package without having to write aliases or do it manually, which would be of great help to improve our productivity.

Hi @eduardolat, thank you for taking the time to create an issue!
I think this is usually something you can set up in your editor. Either you could have it not delete unused imports, or use an editor that supports importing with aliases. GoLand (from Jetbrains) usually does this quite well, I don't know about other editors. Alternatively, wait with saving the file until you've used the import.
This isn't something I want to add to the library. If you editor doesn't support some of my proposed solutions, maybe you can file an issue with the makers of your editor?
I hope you find a good solution. 😊

Thaks @markuswustenberg for your response.
I think I'll just use the imports without aliases (gomponents, html, components instead of g, h, c).
I'm going to find out about configurations for vscode that solve this in the best possible way, thank you very much for your contributions to the community.

I'm glad you found a workaround that works for you. Feel free to post a solution here if you ever find one!


Add to onboarding reproduction logs

Successfully reproduced results for start-here, experiments-msmarco-passage
Reproduction Environment:
UW Linux server
Java Version: 1.8.0_421
Python version -3.10.12

Please continue along the onboarding path until you reach the end of the exercises, and then we'll take a look all at once.

Having not seen any progress, closing. Please reopen if otherwise.

Good day sir,
My computer currently is not powerful enough. I finish as soon as I get my
hands on a computer with more memory.
On Sat, Aug 31, 2024, 3:39 p.m. Jimmy Lin @.***> wrote:

Having not seen any progress, closing. Please reopen if otherwise.
—
Reply to this email directly, view it on GitHub
https://github.com/castorini/anserini/pull/2573#issuecomment-2323025696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALRI6N6DRZO3IOYUXL5ULWLZUILXBAVCNFSM6AAAAABMV6BKEGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRTGAZDKNRZGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


groups: paste image from clipboard to upload

Problem
Pasting an image in Talk or Groups has no effect.
Proposed Solution
Pasting image binary data from the system clipboard should initiate an S3 upload of the image data, as was the case in Groups 1.

add drop target to chat input


In WAL mode exception is not thrown when deleted the database files.

I tried an experiment :  My application will create the database file if it does not exist on application start.  Once the application is loaded, I deleted all the database files related to WAL mode (.db ,  .db-shm  ,   .db-wal) .   But then on doing insert and read operations, no exception is thrown.  But when I disable WAL mode, exception is thrown and my application can catch it.  I am using REACT with ELECTRON.
So if the database file gets deleted accidintly after the application starts, then all the data in the insert operations is getting lost once I close the application (No feedback can be shown to the user).
Is this expected in WAL mode of sqlite ?  Is there anything I can do as work around except checking for the existance of  the database file each time I do a INSERT, UPDATE, DELETE operations ?  Thanks in advance and thanks for the awesome library.

This is not within the scope of this library. sqlite3 itself does not lock the .shm and .wal files. Deleting a hot journal is one way to get a database corrupted as stated in the official documentation section 1.3. The files are only required for cases where a program exits unexpected (error, system shutdown, power failure). If all transactions / operations are completed and written to the database file, the files are not required as the database should be in a consistent state.
Files don't get deleted accidentally, especially not if they are stored in a proper directory a user does not navigate into normally. I would not go down the rabbit hole of implementing logic, that tries to be smarter than someone who thinks deleting files one does not know about is a good idea. Nor would i become paranoid and check for the existence of the files with every operation.


on each data set I need to add a href link

on each data set I need to add a href link

Please ask questions on StackOverflow: https://stackoverflow.com/questions/tagged/chart.js

@dineshdedhia Any luck on this?
@benmccann It will be helpful if you provide doc on it.


Backport EKS - adding ESK cluster failed in v2.2.2-rc5 #19460

backport for #19460

The bug fix is validated in Rancher v.2.2.2-rc6
Test 1 :

run Rancher container  v.2.2.2-rc6
add an EKS cluster
wait for the cluster to be active
check the cluster's version in both UI and API

Result:
the cluster works well as expected
Test 2:

run Rancher v2.1.8
add an EKS cluster
wait for the cluster to be active
check the cluster's version in both UI and API
upgrade Rancher to v2.2.2-rc6
check the EKS cluster's version in both UI and API
add the second EKS cluster
check the second EKS cluster's version in both UI and API

Result:
both clusters work well as expected, and the information in UI and API can match each other

So far tested only the us-east-1 region.
will test on all US regions and update the comment.


TNO-1717: Add isPress and Aliases to the Contributor

Draft until tno-core PRs merged, then this one will be updated.
In this PR:

added isPress flag and aliases to the contributor
isPress and aliases editable through the contributor form
isPress displayed on contributor list view
added contributors to subscriber lookup
started basic page for PressGallery (start of TNO-1728)

You'll need to redo your DB migration once I merge my PR.  Remember how to properly rollback and remove your snapshot?  If not, ask me.

@Fosol - updated!


E2E: Use zaptest logger for dumping and AWS/Azure destroy

This ensures that the logging in the tests get scoped to the specific test, making the E2E output more legible.

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm
thx!


Edit Homepage to Not Display Images on Mobile

Description
Why
Edit homepage to not display images when webpage is being viewed on mobile. 'Prettier' was run on the source code creating more changes. Only changes to index.js were to add the new HomePageImage class to the image's that should not be displayed on mobile.
Microsoft Reviewers: Open in CodeFlow

@chiaramooney Can we get some before after screen captures? Both desktop/wide screen vs. mobile or narrow screen.


cannot write to custom metadata records in managed package

After I create a managed package version of all this, I can no longer write to the CMDT.
error is
Custom metadata type Unsubscribe_Link__mdt is not available in this organization.
Append this to your URL to install the managed package.
packaging/installPackage.apexp?p0=04taj0000003XInAAM
The Unsubscribe cmdt record is not being updated by the Unsubscribe Setup flow which calls on an invocable method.
The Unsubscribe cmdt object has visibility of public and the record has protected = false.
If i open the record itself, I can modify it, but the flow does not modify it. I tried giving myself access to the all the apex in the managed package in a perm set but that didn't help.

FROM this site https://help.salesforce.com/s/articleView?id=sf.custommetadatatypes_package_install.htm&type=5
Looks like the CMDT cannot be updated in standard editions of Salesforce from a managed package. That means the setup flow will NOT work.
Package Custom Metadata Types and RecordsYou can package custom metadata types and records in unmanaged packages, managed packages, or managed package extensions. Your packages can then be installed in Professional, Developer, Enterprise, Performance, Unlimited, and Database.com Edition organizations.Required Editions
Available in: Salesforce Classic and Lightning Experience
--
Protected custom metadata types in managed packages are available in: Developer Edition and scratch orgsPackage uploads and installs are available in Group, Enterprise, Performance, Unlimited, and Developer Editions
Create, edit, and delete custom metadata type records from installed packages Group and Professional Editions


[channels,rdpdr] fix drive redirection hotplug

Currently, after receiving PAKID_CORE_USER_LOGGEDON, the client will switch to RDPDR_CHANNEL_STATE_USER_LOGGEDON, send Device List Announce, then switch back to RDPDR_CHANNEL_STATE_READY.  As a result, any drives mounted later will not be announced.

PAKID_CORE_USER_LOGGEDON is an optional message that might never be received. sure it is not easier to fix the check in rdpdr_try_send_device_list_announce_request?

Yes, it can be fixed that way.   But the current way of handling PAKID_CORE_USER_LOGGEDON is confusing to me:  we have an RDPDR_CHANNEL_STATE_USER_LOGGEDON state but it is temporary and seems redundant, because we also have a userLoggedOn member in rdpdrPlugin.

@fujames1 it is temporary, yes. (better for debugging if you know the exact state you were in)
the userLoggedOn argument for device list announce is required as the protocol strictly distinguishes between before and after this message was received. (but there is a backward compatibility as well... see  device_announce for some details)

replaced by https://github.com/FreeRDP/FreeRDP/pull/10374
thx for the hint.


flake8 file ignores line length

./openprescribing/.flake8 has ignore = E501 which is against policy.

update @sebbacon ?


Document transaction's details Memo field seialization/deserialization algorithm

Right now when you look at some transaction's details one might noticed that it is not clear how exactly the Memo property value was encoded and how one can decode it back to meaningful form.
{
  "data": {
    "syncStatus": "SYNCED",
    "block": {
      "transactions": {
        "userCommands": [
          {
            "feePayer": {
              "publicKey": "B62qpVZSwJhRKQWzeJQRV4CX6LVdNrBEqan1qzrvE7JHvsT7D3mGSL4"
            },
            "receiver": {
              "publicKey": "B62qovre4KFEZbew2V44v9t7XgfhHV9VaJLZq5F1Hqj3eSH5gvrUQqm"
            },
            "kind": "PAYMENT",
            "memo": "E4YM2vTHhWEg66xpj52JErHUBU4pZ1yageL4TVDDpTTSsv8mK6YaH",
            "amount": "1000000000",
            "failureReason": null,
            "hash": "CkpZ6q13z4ZYLRxL6ezjCwYBRFz1GNrdxkcmTj2F7CcRVHPdZ8pyK"
          }
        ]
      }
    }
  }
}

It will be good to have seialization/deserialization algorithm documented somewhere.

It would be easy to expose these in the ClientSDK. Is that worth doing?

For references, TypeScript decoding:
import base58 from 'base-x'

const BASE58_ALPHABET = '123456789ABCDEFGHJKLMNPQRSTUVWXYZabcdefghijkmnopqrstuvwxyz'
const NULL_VALUE_REGEX = /\0[\s\S]*$/g

export const decodeBase58Message = (message: string) => {
  if (!message) {
    return ''
  }

  const decodedMessage = base58(BASE58_ALPHABET).decode(message)
  const decodedString = new TextDecoder().decode(decodedMessage.slice(2, decodedMessage.length - 4))

  return decodedString.replace(NULL_VALUE_REGEX, '')
}


Set runner to ubuntu-latest, enable KVM

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (aff8223) 87.14% compared to head (02d8c0b) 89.50%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #929      +/-   ##
============================================
+ Coverage     87.14%   89.50%   +2.35%
============================================
  Files           135       86      -49
  Lines         17509    15371    -2138
  Branches        428        0     -428
============================================
- Hits          15259    13758    -1501
+ Misses         1967     1613     -354
+ Partials        283        0     -283

Flag
Coverage Δ

uitests
?

uitests-android
?

unittests
89.50% <ø> (+3.01%)
:arrow_up:

unittests-android
?

unittests-rust
89.50% <ø> (ø)

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

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


[REQ] Colab (or other services) notebook for testing/training

Hi there, the paper (and of course your implementation) is extremely interesting, so it would be great to let anyone test/train it by exploiting an online service like Google Colab/HF Spaces/Paperspace/Kaggle/Jupiter/Deepnote/etc.
Thanks in advance !

Hi, thanks for your interest!
I'm currently working on my master's thesis/code, which tackle the enhancement problem and in a couple weeks I'll make it available. After that I'll be able to manage these other side projects
Best,
Wallace


Project manager/scrum master: Establish process for managing timeline, scope, budget, and tooling

Management system has been established. Interlock with core team: 15 minutes daily call (from monday to friday), biweekly lunch (thurscay) and extraordinary meetings under agreement. Roles and activities have been identified. Next milestone in Februry. Team agreement on personal time to dedicate to Prometeo.

@svaleroc What time would work well for a regular meeting with the IBM team (which will start to include the Service Corps folks)? Tuesdays 4pm your time? Later?

@krook Tuesdays 4pm our time, ok for us. Thanks!


Publish regular Snapshots

As seen in https://repo.eclipse.org/content/repositories/microprofile-snapshots/org/eclipse/microprofile/ Config and Fault Tolerance publish regular Snapshots as recent as July 3 (today)
Could Health also do so?

Health already publishes snapshots.


Circlesclock v0.12

Allow configuration of update interval

Is this still meant to be a draft, or are you happy to merge?

Yes, this is still draft. I plan to add some more changes in the next days. Sorry for putting the draft already here.

Thanks!


Create a feature list for this project

Comment with what are some of the high level features that this project can target.
Acceptance criteria: list of the features for backend and front end

yea so @ralphie9224 said that we should create issues and then just assign them they should get added to the board https://github.com/ralphie9224/kchat/projects/1 automatically

@audip if there's anything "build" related you might find interesting, feel free to make an issue for it and take it on. if not, don't hesitate to ask questions on anything Kotlin/ktor related if you're going to take a stab at something there


Allow empty list streamed response

Goal

Close #706
Fix build issue in branch 0.10.1

 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.Tuyen 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.

CI failed because of spec tests 0.10 which are not part of this issue.
Should be ready for review/merge.

Signed CLA ^^


OreDictionary Support for the PPE and Security Manager.

Added the "workbench" tag for the Crafting Table in the Processing Pattern Encoder, and the "chest" tag for the Security Manager. This allows them to use crafting tables and chests from other mods, including the Crafting Station from Tinkers' Construct and the Chests from Quark. #969
It appears that the Solderer recipes are looking for ItemStacks and do not like the strings for ore dictionary support. I'm not sure how to add the ore dictionary versions to the Crafting Grid, Crafting Upgrade, or Disk Drive.

thanks!


Update docstring for hero.wordcloud

After the discussion on #78
We should add something like:
"To reduce blur in the images, width and height should have the same size, i.e the image should be squared"

Hi @jbesomi @vidyap-xgboost, i'll take this issue.
I also like to add an example in the docstring, but i am not sure how to do this with visualizations. Do you have a suggestion how to do this?

Thank you Sir: assigned!
We cannot really add a visual example in the docstring, but it's cool you asked.
A very interesting solution that soon or later we might want to implement is to create some visual example with sphinx-gallery. Have a look at the documentation of Sphinx-gallery and see if it make sense. For instance, scikit-learn example is using it for the Examples.
We might want to implement that. If you want, I can help you with such task, and apparently also @vidyap-xgboost was interested in contributing :)

Based on the discussion in #78, it seems like width and height does not have to be the same so long as both of them are at least 400. The first example in #78 has the same width and height (200, 200) but it is still blurry.
I think we should rephrase it to:
"To reduce blur in the images, width and height should be at least 400."
What do you think @vidyap-xgboost @jbesomi? :smiley:

Thank you Sir: assigned!
We cannot really add a visual example in the docstring, but it's cool you asked.
A very interesting solution that soon or later we might want to implement is to create some visual example with sphinx-gallery. Have a look at the documentation of Sphinx-gallery and see if it make sense. For instance, scikit-learn example is using it for the Examples.
We might want to implement that. If you want, I can help you with such task, and apparently also @vidyap-xgboost was interested in contributing :)

Yup I'm interested in implementing it through sphinx-gallery, I want to help :smile:

Based on the discussion in #78, it seems like width and height does not have to be the same so long as both of them are at least 400. The first example in #78 has the same width and height (200, 200) but it is still blurry.
I think we should rephrase it to:
"To reduce blur in the images, width and height should be at least 400."
What do you think @vidyap-xgboost @jbesomi?

I was going to point this out and a little late here, thanks for bringing it up and I agree to your point! But can you also make sure to test for width=500 and height=400 and see if any blur is being introduced? Thank you!

@jbesomi @cedricconol According to the numpy docstring guide as suggested in CONTRIBUTING.md:

The length of docstring lines should be kept to 75 characters to facilitate reading the docstrings in text terminals.

But wordcloud seems to have more than 75 characters in its docstring(and yes, its really difficult to read in VS code), can you please take care of that @cedricconol ? I thought I would do it myself but I do not want to interfere with your PR :smiley: , hence asking you.

Based on the discussion in #78, it seems like width and height does not have to be the same so long as both of them are at least 400. The first example in #78 has the same width and height (200, 200) but it is still blurry.
I think we should rephrase it to:
"To reduce blur in the images, width and height should be at least 400."
What do you think @vidyap-xgboost @jbesomi?

I was going to point this out and a little late here, thanks for bringing it up and I agree to your point! But can you also make sure to test for width=500 and height=400 (you can also test with higher numbers) and see if any blur is being introduced? Follow #96 to see how I tested it on the terminal directly.
Thank you!

Hi @vidyap-xgboost, I tested width=500, height=400 and width=1000, height=800 following your code in #96. It seems like the blur is gone :smile:
>>> import texthero as hero
>>> import pandas as pd
>>> df = pd.read_csv('https://github.com/jbesomi/texthero/raw/master/dataset/bbcsport.csv')
>>> import matplotlib
>>> hero.visualization.wordcloud(df['text'],width = 500, height= 400,background_color='White',min_font_size=1)
>>> matplotlib.pyplot.show(block=True)
>>> hero.visualization.wordcloud(df['text'],width = 1000, height= 800,background_color='White',min_font_size=1)
>>> matplotlib.pyplot.show(block=True)

Here are the results:
width=500, height=400

width=1000, height=800

@jbesomi @cedricconol According to the numpy docstring guide as suggested in CONTRIBUTING.md:

The length of docstring lines should be kept to 75 characters to facilitate reading the docstrings in text terminals.

But wordcloud seems to have more than 75 characters in its docstring(and yes, its really difficult to read in VS code), can you please take care of that @cedricconol ? I thought I would do it myself but I do not want to interfere with your PR  , hence asking you.

Thanks for the suggestion @vidyap-xgboost, this is noted, it seems like black does not format the docstrings.

@jbesomi @cedricconol According to the numpy docstring guide as suggested in CONTRIBUTING.md:

The length of docstring lines should be kept to 75 characters to facilitate reading the docstrings in text terminals.

But wordcloud seems to have more than 75 characters in its docstring(and yes, its really difficult to read in VS code), can you please take care of that @cedricconol ? I thought I would do it myself but I do not want to interfere with your PR  , hence asking you.

Thanks for the suggestion @vidyap-xgboost, this is noted, it seems like black does not format the docstrings.

Great work @cedricconol !
Yes, the docstrings are not formatted by black. Any solution here @jbesomi ?

Hey @vidyap-xgboost @cedricconol! You are doing great, thank you!
Apparently black does not format docstring. I guess we will need to do it ourselves (@vidyap-xgboost we might explain this better in the CONTRIBUTING.md, what do you think?). If you want to know more: Black should have an opinion about doc strings (still open issue)

Hey @vidyap-xgboost @cedricconol! You are doing great, thank you!
Apparently black does not format docstring. I guess we will need to do it ourselves (@vidyap-xgboost we might explain this better in the CONTRIBUTING.md, what do you think?). If you want to know more: Black should have an opinion about doc strings (still open issue)

I agree. Explaining in CONTRIBUTING.md sounds good. Thanks for sharing the link!
I would like to work on further changes in documentation and tutorials after the next release since @henrifroese is already working on the documentation. I wouldn't want to interfere and duplicate the work.


ViolationWarningPacket.java:84: Implement violation...

The puzzle 1-7a7d3b53 from #1 has to be resolved:
https://github.com/shiruka/shiruka/blob/5f0a9e5a9dee93e9f532c73a26a2b3478280c90d/src/main/java/net/shiruka/shiruka/network/packets/ViolationWarningPacket.java#L84-L84
The puzzle was created by Hasan Demirtaş on 25-Jan-21.
Estimate: 15 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 1-7a7d3b53 has disappeared from the source code, that's why I closed this issue.


Attachment points should be within R-Group brackets

Description:
When attachment points are attached to a structure that is a R-Group member,  the attachment points are beyond the R-Group brackets.
Steps to reproduce:

Create a structure.
Make the structure a R-Group member using R-Group Fragment Tool.
Attach attachment points to the structure using Attachment Point Tool.

Actual result:
Attachment points are beyond R-Group brackets.

Expected result:
Attachment points are within R-Group brackets.

Made only for R-Group Fragment, not for S-Group


1080p Premium usability

Hi, I found your YouTube HLS Enabler script while searching for a way to get the 1080p Premium stream without YT Premium, your script supposedly does this, but as of right now there's no easy way to make sure it's working, can you please make it so the script enables the 1080p Premium option in the quality selector and map that to said quality from the HLS manifest?
Thank you.

The premium 1080p option is not included in the quality selector if you are not logged in or if the video is embedded. For that reason the userscript adds an additional 'Hls manifest' toggle. The numbers on it represent the ids of the video and audio streams currently in use, 616 is the id for the premium stream.
If 616 is available for a video the toggle turns on automatically, otherwise it stays off.
There are some configuration options at the top of the userscript, you can change notify616 from false to true to get a toast notification when 616 is available if you want.

Actually, in my case I have the 1080p Premium option in the quality selector, I just can't chose it because I don't have Premium, what I meant with this suggestion is that the userscript could make that option usable for non-Premium users (Obviously also enabling said quality, not just cosmetics)


[bug]: MPS high resolution generation artifacts & nondeterminism

Is there an existing issue for this?

[X] I have searched the existing issues

OS
macOS
GPU
mps
VRAM
64GB
What version did you experience this issue on?
3.0.1post3
What happened?
There are residual bugs in the MPS image generation pipeline that cause visible artifacts at high resolution. The artifacts appear as fairly subtle blotchy colors and smeared details, but the easiest effect to see is nondeterminism at certain resolutions.
The below images show consecutive generations using SDXL, no changes to prompt or settings between them, along with an image difference I generated in an image manipulation program (expanded 5x to make the differences more apparent). Note that the issue is not exclusive to SDXL, but also appears in generations using vanilla SD1.5. SDXL is a more convenient demonstration because its output is more coherent at high resolution, making the artifacts easier to see. Blotchy colors and smeared details start to appear gradually at lower resolutions, but the lowest square resolution that produces nondeterminism is 1536x1536. 1600x1600, for whatever reason, is deterministic, but 1664x1664 and above are not. And at very high resolutions (1856x1856 to 2048x2048), the artifacts become debilitating.
Screenshots
Nondeterminism (look also for blotchy colors, seen most easily as reddish blotches in the green of the mountainsides):

Debilitating artifacts at high res:

Additional context
No response
Contact Details
Adreitz on Discord

I realized I could do a little more experimenting using the Nodes feature, but I got a little frustrated because I was having a hard time reproducing the nondeterminism. It appears like some of the issue with MPS generation is specific to the t2i interface and is not present in Nodes.
SDXL base - duplicated pipeline.json.zip
This is a Nodes graph including just SDXL base (to eliminate variables). The full rendering pipeline is duplicated so that two identical images are generated for each invocation. I first generated two images with the same prompt and seed as in the images above, but at 1664x1664, using the t2i interface and with the refiner disabled. There were visible blotchy colors, smeared details, and nondeterminism. See below:

I then copied the same prompts and seed into the Nodes graph (with slight differences possibly stemming from how Invoke links the main and style prompts in the t2i interface) and generated the below image. It produced two images as desired, but both were exactly the same.

There is a general great improvement in clarity and detail (see the mountainsides near the image center) and reduction in blotchiness (see lake surface) and smeared details (grass in lower left corner).
So, it appears there is something wrong specifically with the t2i interface on MPS.

@Adreitz are you still experiencing this with Invoke 3.6+?

@Millu I didn't test yet against the nodes workflow, but it appears that the t2i interface still exhibits nondeterminism with SDXL on MPS. See below. The pixel difference is multiplied by 5 for visibility.

Great, thanks for checking!


[wip] add ancestry case to the genealogy parent setter

the multiplicity of tree like structures and their differences in behavior makes this overly complicated. we've got a few bugs with multiple parents being set and a lot of code written to address that bug in particular. we're trying to simplify all this by removing relationships.
this genealogy_parent setter needs to eventually handle ancestry cases as well.
when we destroy all relationships, we'll want to just alias this to parent=
@miq-bot assign @kbrock
relies on
https://github.com/ManageIQ/manageiq/pull/20274
for the use_ancestry? method
although at the time of this edit, I've duplicated the six lines I need for this to be passing and will remove them from 20274 if I get the okay to do so

this code is good.
In this context, the relationships array is confusing as to what it even is.
I think this should build on the pr that establishes relationship types and has the call ancestry or other.
then the PR that is trying to get vms working will do this change.

could I please get this reviewed?


[0.4.0] Workflow Task Scheduled event props incorrect

Task queue property value is incorrect, it does not show the task queue name and kind. It seems its actually the workflow results displayed instead.
See v1 below for same event props:

Hmmm… I wonder if this is related to #304. I'll look at the payload and triage this one this morning.

Oh, wait. I don't think we cut a release since #304. I think this is fixed and a duplicate of #287. I can't recreate it in main, which leads me to believe it was fixed. So, I'm going to mark it as a duplicate for now and close this, but if it comes back up in the next release or when you're working off of main, definitely let us know.

seems fixed in 0.5.0


Several fixes/improvements to syncstatic command

This includes:

Fix for django STATIC_URL formatting (see comments in SHA: 06bceac53 )
Rewritten syncstatic command that is blazing fast in the optimistic case
Fix for AUTH_URL setting causing breakage (cherry-picked from rizumu/django-cumulus@a1ab10488)
Fix for CUMULUS_STATIC_CONTAINER not being used consistently (let me know if this is not the right thing to do and I will back it out)

I'm guessing this is not right. The CUMULUS_STATIC_CONTAINER setting and CUMULUS_CONTAINER settings should both exist so that static and default content can point to different containers, correct? If so, I'll update this and resend.

This works for me. To use the syncstatic command you have to specify both CUMULUS_STATIC_CONTAINER and CUMULUS_CONTAINER. Syncstatic now takes about 3 seconds if you have few changes. I'll probably also update it to use the file hash instead of the timestamp at some point in the future.

@poswald I'm looking through all of this today/over the weekend. I'll post questions/thoughts as I have em and hopefully pull this work in before Monday. Thanks for contributing!

@richleland If you could let us know if the intention to have CUMULUS_STATIC_CONTAINER be a separate container, that would be useful. I think it is right the way I have it now but perhaps it needs to pick up a default. Either way, I think it needs to be explicitly mentioned in the docs because I thought it was a typo. Admittedly, I came to this project looking to use the syncstatic command, not the storage part so I was thinking the original container was going to hold my static assets.
I've been using my fork for production deployments for a bit now and it works much better for me. Two things to note:

It sometimes gets all the way through a sync successfully but some files are not in the CDN.  I think that perhaps there is an issue with error handling from rackspace. I need to investigate this more.

Rackspace provides a hash of the file in the initial request I am making. I want to use that instead of the file modification time to determine if we should upload. It will greatly reduct the number of false positives for me since when I deploy, I often do so from a fresh directory so the file times are not a great indication of a match.

I'll be around this weekend. Please let me now if you have questions.

@poswald thanks for the details. A few quick notes:

We do need to have a separate CUMULUS_STATIC_CONTAINER - here's why. If we don't have two separate containers to specify, everything will be thrown into one dir.

I'm working on removing the need for syncstatic. cumulus should really use the built-in collectstatic command. One thing that I'll be adding is a CloudFilesStaticStorage subclass that just uses CUMULUS_STATIC_CONTAINER. This way, user-generated media will go to the default CUMULUS_CONTAINER and static media will go to CUMULUS_STATIC_CONTAINER.

So long story short - I'll be yanking out some of the stuff from your commits, but will be removing syncstatic when everything is working as it should with collectstatic.

I think I get it now. My only recommendation would be to improve the documentation around that because I got it wrong the first time.

I can understand this line of thinking but I would caution against merging that functionality until you make sure all use cases are covered. Right now I have several developers working on the team. When in development mode, most of them do not need collectstatic to upload into the CDN. Only on deploy does this need to happen. Also, I use this project with django-compressor in offline mode which means it runs between collectstatic and syncstatic.

Our workflow is:

Run collectstatic to get everything into one local directory. For most developers, this is enough to get going.
Run compress in offline mode to pre-create compressed files in a /CACHE/ directory. This happens on our development server and for developers working on the front end. Some developers use on-demand mode of the compressor as well.
Run the previous two commands plus syncstatic when deploying to final testing and production servers to get everything into CDN

I can see the desire to make a single command that does everything but I think it makes sense for you to leave this step to be run as a standalone command. Perhaps just adding a --sync switch to the collectstatic command that then automatically runs the sync command.

Great idea and thanks for providing your use case. Really helpful @poswald.


make  -a as default ?

is it possible to make brew CU update all casks (included those with autod update on) by default ?
so that I won't have to type -a each time, and will simply have to enter brew cu -y
(side question , is it dangerous/not advised to update apps that are marked as "auto update" ? )

Persistent config is being implemented in #174

(side question , is it dangerous/not advised to update apps that are marked as "auto update" ? )

Not really, it's up to every person to decide, how they want to manage those casks, either through brew or let them auto-update. You can also combine both of the approaches, etc.

As a new user who just found this hidden gem, I'd say the default behavior is non-intuitive.
This morning, Notion and Slack asked me for a password to auto-update once again, and I searched for a solution.
I wanted a better way to auto-update all the annoying auto-updating apps, and initially I was disappointed that this tool wasn't doing the job for me. I came here to raise or find an existing issue, and landed in this discussion.
Now I'm all set, but maybe there are many other people who don't RTFM and share the same experience.


After deleting Ingress using Kubectl, rancher LB service does not get deleted

Rancher versions:
rancher/server or rancher/rancher: v1.6.18-patch1-rc2
Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
HA
Steps to Reproduce:

There are more than 100 ingress created on the Kubernetes environment.
There will be corresponding 100 loadbalancer services launched on rancher backing the ingress
Delete an Ingress using Kubectl
The corresponding loadbalancer service is not deleted sometimes and gets listed under Rancher UI.
The issue is seen intermittently.

Duplicate of https://github.com/rancher/rancher/issues/18568


feat(lemmy): init

🎉 Theme for Lemmy 🎉
Lemmy is free and open-source software for running self-hosted social news aggregation and discussion forums
💬 Additional Comments 💬
NA
🗒 Checklist 🗒

[x] I have read and followed Catppuccin's submission guidelines.
[x] I have made a new directory underneath /styles/<name-of-website> containing the contents of the /template directory.

[x] I have ensured that the new directory is in lower-kebab-case.
[x] I have followed the template and kept the preprocessor as LESS.

[x] I have made sure to update the
userstyles.yml
file with information about the new userstyle.
[x] I have included the following files:

catppuccin.user.css - all the CSS for the userstyle, based on the
template.
assets/mocha.webp, assets/macchiato.webp, assets/frappe.webp and
assets/latte.webp - individual screenshots of the themed website, in all 4
Catppuccin flavors.
catwalk.webp - image of all individual screenshots stitched together,
generated via catwalk.

I'll take a look at the linter error in a bit, looks like it's failing to parse the metadata (first couple of lines of the userstyle)

I'll take a look at the linter error in a bit, looks like it's failing to parse the metadata (first couple of lines of the userstyle)

I think the issue is the author hasn't added the data to userstyles.yml right?

Actually it looks like Lemmy has the first letter capitalized, so if you could change that in userstyles.yml and catppuccin.user.css that would be great.

Great, can you update the capitalization of Lemmy next? That should be everything you need.

LGTM, but knowing me i might've missed something so i'll let some not merge it right away to let others see what they think, but if nothing changes in the next 48hrs this will be merged.

From the preview images... these bubbles/pills appear to be unthemed:

My eyes are terrible though, correct me if I'm wrong.

The original CSS shows this to be - .text-bg-light { color: #fff !important; background-color: RGBA(var(--bs-light-rgb), var(--bs-bg-opacity, 1)) !important; } and the background-color is using RGB values, if I replace that with hex values (using the variable), it doesn't work. Can you help in how I can resolve this?

You should check out:
https://github.com/catppuccin/userstyles/blob/6ace7f8dd2f3b40bee41c4331a408f2d4635e564/styles/anilist/catppuccin.user.css#L703-L705
https://github.com/catppuccin/userstyles/blob/6ace7f8dd2f3b40bee41c4331a408f2d4635e564/styles/anilist/catppuccin.user.css#L57
what this mixin does is convert the hex values into a raw rgb data, this should be an ideal fix for this situation.

Alright looks good to merge. Thanks for your hard work.


Feat: Websites Development

Issue Description
As we're approaching the completion of the Appcraft project, we are introducing the final feature: Websites. This feature aims to provide users with the ability to create and manage websites using a user-friendly interface, enhancing the overall user experience and extending the functionality of the platform.
Issue Details

Enable users to design and customize websites with unique themes and components.
Integrate a user-friendly page editor for real-time modifications.
Allow users to manage navigation menus, content sections, and more.

Tasks

Design the user interface for the "Websites" feature.
Develop the functionality to create, edit, and manage websites.
Implement real-time editing capabilities for website customization.
Integrate theme styling options for personalized website design.
Allow users to manage navigation menus and content sections.

issue#230 completed


[Request Enum] table 246 "Requisition Line" - "Planning Line Origin"

In out vertical we have an item layout specification planning worksheet and added an option. Please make it extensible.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


python37Packages.scikitlearn: patching build

Motivation for this change
For numpy>=1.17 a test-case broke that required adjustments to
a threshold.
See https://github.com/NixOS/nixpkgs/issues/68494
Related: https://github.com/NixOS/nixpkgs/issues/68361

Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @FRidh due to previous discussion of this

thank you @d-goldin


Word counter reset when input is cleared

Problem: When all text is cleared from the textarea, the word count values should reset to empty values.

I think this is done :)


[util] updates to ensure make_new_dif points at the right hjson

re-use topgen routines to discover ipgen and templated modules
remove hardcoding of IP_USE_IPGEN in both topgen and make_new_dif

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

@timothytrippel let me know if this is headed in the right direction.. or if it is too much.


Implementation of simple InstanceScope

Am aware this is more a question, rather than a bug, but perhaps an indication that an example scope implementation (other that an example of seemingly re-implementing NoScope would be helpful for those new to injector.
Anyway, I am trying to implement a simple custom scope which provides the same instance of a given class to any sub-objects in that class that might require a reference to it.
e,g
class A:
  @inject
  def __init__(self, b: B, c: C):
    self.b = b
    self.c = c

class B:
  @inject
  def __init__(self, c: C):
    self.c =c

class C:
  pass

I essentially need a scope that will ensure that the instance of C that is provided at injection of both A and B is the same one for any given instance of A.
It feels like I should be able to do this by creating a custom scope that is applied on A, but the scope documentation is great for everything except for an idea of how you might go about implementing the scoping itself, I feel like I'm missing something obvious somewhere!
Currently I am handling this by using AssistedBuilder instances in a factory class to provide those C instance, but that feels like the wrong approach...

Yeah, the documentation can be improved in this regard. About your case – can you provide some pseudo-code of what you're trying to achieve?

Sorry, hit enter too early - edited the OP to include an example.

I don't think there's a way to do this at the moment. Possibly the scope system could be improved to handle this, but it'd require some redesign I believe.


[stable/spinnaker] Failed to enable docker registry ( git clone and helm install default values.yaml)

What?
  Failure
Problems in default.provider.dockerRegistry.gcr:
my-release-install-using-hal-8rg78 halyard-install ! ERROR Cannot find provided path: /opt/registry/passwords/gcr (No
my-release-install-using-hal-8rg78 halyard-install   such file or directory).
my-release-install-using-hal-8rg78 halyard-install Problems in halconfig:
my-release-install-using-hal-8rg78 halyard-install - WARNING There is a newer version of Halyard available (1.22.2),
my-release-install-using-hal-8rg78 halyard-install   please update when possible
my-release-install-using-hal-8rg78 halyard-install ? Run 'sudo apt-get update && sudo apt-get install
my-release-install-using-hal-8rg78 halyard-install   spinnaker-halyard -y' to upgrade
my-release-install-using-hal-8rg78 halyard-install - Failed to enable dockerRegistry
my-release-install-using-hal-8rg78 halyard-install
my-release-install-using-hal-8rg78 halyard-install

How to repro?

helm fetch stable/spinnaker && tar -vxf tar -vxf spinnaker-1.14.0.tgz
helm install --name my-release . --timeout 600 --namespace=spinnaker -f values.yaml

Impact

Unable to run the example values.yaml code means we cannot reliably build on it without understanding the root cause of this error.

I've found that this is due to having an old GCR secret despite being disabled within the manifest it is still picked up by halyard..


org.modelmapper.MappingException: ModelMapper mapping errors:

org.modelmapper.MappingException: ModelMapper mapping errors:

Converter org.modelmapper.internal.converter.MapConverter@6d60359c failed to convert java.util.HashMap to java.util.Map.
1 error
at org.modelmapper.internal.Errors.throwMappingExceptionIfErrorsExist(Errors.java:380)
at org.modelmapper.internal.MappingEngineImpl.map(MappingEngineImpl.java:81)
at org.modelmapper.ModelMapper.mapInternal(ModelMapper.java:573)
at org.modelmapper.ModelMapper.map(ModelMapper.java:447)
at com.oms.OrderFullMapper.mapTo(OrderFullMapper.java:150)
at com.oms.ServiceImpl.lambda$getOrder$1(ServiceImpl.java:291)
at java.base/java.util.concurrent.CompletableFuture$UniApply.tryFire(CompletableFuture.java:642)
at java.base/java.util.concurrent.CompletableFuture$Completion.run(CompletableFuture.java:478)
at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: java.lang.ClassCastException: null

We are facing the above errors on production since last couple of days. We had no change in the code.  The issue is intermittent and exists only in some of the containers of a service.
Can anybody tell what can be the probable issue and resolution?

Same here - our production started showing
org.modelmapper.MappingException: ModelMapper mapping errors:
....
Caused by: java.lang.ClassCastException
No code change, no new deployment.

see #606
Feel free if you still have issues.


Gekko crashed when currency has been added to exchange

Note: this is the technical bug tracker, please use other platforms for getting support and starting a (non technical) discussion. See the getting help page for details.
I'm submitting a ...
[x ] bug report
[ ] question about the decisions made in the repository
Action taken (what you did)
Manually added some ETH to Binance when tradebot trading ETH/SNT was running.
Expected result (what you hoped would happen)
Tradebot should run without troubles.
Actual result (unexpected outcome)
Tradebot finished round and started a new one (at this point with more ETH on my exchange account). Bought SNT with ETH, sold SNT and crashed.
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
This is an archived Gekko, it is currently not running anymore.
This is Gekko crashed with the following error: {}

2018-07-30-12-20-tradebot-653781870961659.log

2018-08-01 22:09:41 (INFO):	[ORDER] summary: undefined

Is the problem, will look into this. Might need to add more verbose logging around this. Strange, never seen this before.

Manually added some ETH to Binance when tradebot trading ETH/SNT was running.

Should only effect profit calculations, shouldn't have this impact on the bot itself (adding capital that is).

I was able to reproduce this locally once, but I'm really not sure what causes this. I've now added a ton of debug logging that should point me to the culprit once it happens again..

Added logging around this situation here: https://github.com/askmike/gekko/commit/2bc2945e22e1324c30774ae6235ecb8bd8677c83
Closing this now, if it pops up again please reopen :)


Crashes when status isn't an expected value

I've gotten a LD2410C and connected it to my Arduino Micro, this library works great and the debug mode is awesome.
in my case sometimes a package is produced that has a status value of 131 (I don't know why) and translating this to tStatus enum and printing it to serial will crash the whole serial communication and freeze up the microcontroller. I have a PN532 NFC scanner connected on I2C and they share the power line, so perhaps the dips in power cause the faulty package.
In my case I ended up fixing my situation it by not using the statusString() function and just using the getStatus() function, then matching against enum value for moving targets, as that's the only one I'm interested in..
It's possible to build in protection for situations such as these by  updating either the getStatus() function or the statusString() function to ensure access to the enum is legal(fits in size?), but I'm not sure if my case is an outlier or the norm, due to me only having the one setup.

Thank you for discovering this bug.
The status byte, per the documentation, can only take values between 0 and 3. In a rare case like yours, a bit may get flipped resulting in a status outside of this range.
131 = b10000011 -> bit 7 must be 0, but was received as 1.
I fixed this bug in line 209 of MyLD2410.cpp:
sData.status = inBuf[2] & 3;
I will be publishing v1.0.7 later today.

V1.0.7 fixes this bug


WIP: Changes for custom data type

Some changes I've done to prepare GeoRocket for Quantized Mesh implementation.

These are a lot of changes. I know it's still work in progress, but I'd prefer - for the final version - that you create several small PRs. Maybe you can already start with one or two small ones???

And please try to keep the diff small (e.g. by avoiding unnecessary changes such as removing white spaces). This makes reviewing a lot easier. Thanks.

This pull request can be closed. It is way outdated and those changes are not needed.


Implement create database and table operations.

Added Eclipse java style guide XML file.

database API implemenation - first cut

database API implemenation - first cut

Fixed java source and target versions to 1.8

preliminary changes to facilitate testing

database API implemenation - test cases

Fixed pom.xml with junit and mokito dependencies.
Format code with 4 spaces instead of 2.

database API implemenation - test cases

fixes related to tests

change to postgres for testing

fixes related to tests

fixes related to tests

database API implemenation - fix tests

database API implemenation - fix tests

fix edge creation in DB and table creation

minor cleanup

DROP... CASCADE to ensure tables are cleaned up.

Merge issues have surfaced ... this is not a good merge. Will redo it,


Update jackson-databind

The jackson databind <IP_ADDRESS> has the following CVEs:
https://nvd.nist.gov/vuln/detail/CVE-2020-14060
https://nvd.nist.gov/vuln/detail/CVE-2020-14061
https://nvd.nist.gov/vuln/detail/CVE-2020-14062
https://nvd.nist.gov/vuln/detail/CVE-2020-14195
They should be resolved if we update to <IP_ADDRESS>
Reporter: Patrick OFriel / @patrickofriel-wk
Assignee: Gabor Szadovszky / @gszadovszky
Note: This issue was originally created as PARQUET-1895. Please see the migration documentation for further details.

Gabor Szadovszky / @gszadovszky:
2.11.2 is later (in terms of release date) than <IP_ADDRESS>. So, I guess it would be a better choice.


Improve TableSymbol/datashape documentation

I'm working through the docs/tutorial, and am finding the semantics of what I can use in a TableSymbol schema frustrating.
Things that seem strange/aren't obvious:

string works, but str doesn't
float32 works, but float doesn't
int works, but integer doesn't
bool works, but boolean doesn't

IMO, all of the base Python types (float, str, int) should work out of the box.

Indeed, the fact that you've actually run into TableSymbol in the wild is somewhat discouraging, that interface is fairly old.  Can you point me to the docs/tutorial that you're using?
What are you doing that you have to specify a datashape?

hehehehe- I was using your blog post here: http://matthewrocklin.com/blog/work/2014/09/01/Blaze-Expressions/
Based on your blog posts and the docs, this is what I put together: http://nbviewer.ipython.org/github/wrobstory/pdxpython2015/blob/master/PDXPython2015.ipynb#Blaze
I'd be really interested to get feedback on what I should be doing there instead of using TableSymbol (or if there are any cool features I should talk about). I still have time to change it before I talk to a group of devs about Blaze :grin:

There are a few things to discuss around the mapping of python types to datashape types. Datashape types generally follow C conventions (though there may be some inconsistencies there), whereas Python follows its own conventions. float is typically 32 bits in C and in Python it's backed by C's double which is typically 64 bits. int in datashape maps to int32 according to the typical precision of C int.
I'd be +1 on the following:

str -> string
float -> float32

Inside of the Python implementation of datashape are generic type classes integral and boolean both of which are not designed to be used directly (though a case could be made for boolean, but personally i don't see the benefit of that).
In [1]: import datashape

In [2]: datashape.boolean
Out[2]: {boolean}

In [3]: type(datashape.boo
datashape.bool_    datashape.boolean

In [3]: type(datashape.boolean)
Out[3]: datashape.typesets.TypeSet

In [5]: datashape.integral
Out[5]: {integral}

In [6]: datashape.integral.types
Out[6]:
(ctype("int8"),
 ctype("uint8"),
 ctype("int16"),
 ctype("uint16"),
 ctype("int32"),
 ctype("uint32"),
 ctype("int64"),
 ctype("uint64"))

If none of my above suggestions are controversial, then I'm happy to implement them in datashape.

@wrobstory instead of TableSymbol the preferred way to create any kind of Symbol is through the symbol function. TableSymbol simply adds a var dimension to a passed in Record datashape. Here's how you'd do it with your diamonds schema:
In [8]: bz_cols = ('var * {carat: float64, cut: string, color: string, clarity: string, '
           'depth: float64, table: float64, price: int64, x: float64, y: float64, z: float64}')

In [9]: from blaze import symbol

In [10]: diamonds = symbol('diamonds', bz_cols)

In [11]: diamonds.dshape
Out[11]:
dshape("""var * {
  carat: float64,
  cut: string,
  color: string,
  clarity: string,
  depth: float64,
  table: float64,
  price: int64,
  x: float64,
  y: float64,
  z: float64
  }""")

Also, for interactive usage Data is a lot easier to use. Docs here. symbols are nice though when trying to explain how blaze separates expressions from data

@wrobstory one last comment :)
Regarding these lines:
pg_datasource = bz.odo(df_diamonds, "postgresql://postgres:postgres@localhost/pydata::diamonds",
                       dshape=diamonds.dshape)
[x for x in bz.compute(carat_count, pg_datasource).execute()]

You don't have to explicitly call execute on the result of compute, odo can do this for you:
df = odo(compute(carat_count, pg_datasource), pd.DataFrame)

Alright I'm just going to keep commenting :)
When creating the diamonds symbol you don't have to specify a dshape

Given the example I would probably stress the following workflow
>>> diamonds = bz.Data("postgresql://postgres:postgres@localhost/pydata::diamonds")
>>> carat_count = bz.by(diamonds[diamonds.price > 1000].clarity,
...                     carat=diamonds.carat.count())
>>> list(carat_count)
...

or just

>>> carat_count
...

You've been playing with the internal, explicit API which, while interesting to those present, is usually a bit mind-bending to your usual meetup audience.  There was a decent amount of work to make sure that the interface around dealing with a database felt a lot like dealing with a dataframe or numpy array.  Explicit calls to compute and dshape kwargs detract from this.

Thanks to you both- I definitely want to stress how powerful the idea of separating expressions from data sources is. I think it's probably the most interesting/important bit of working with Blaze.
Again, thank you!


RAP110 ex.1 - class ZDMO_GEN_RAP110_SINGLE_2 does not exist

I am doing the exercise below on an SAP BTP ABAP instance (US EAST) which is just created using Booster.
Exercise 1.1: Generate your package Package  --> step-2 Search for ZDMO_GEN_RAP110_SINGLE_2
However, there is only "ZDMO_GEN_RAP110_SINGLE" and not "ZDMO_GEN_RAP110_SINGLE_2".
Can you please advise if I can still proceed?
Thanks and Merry Christmas.
Sunny

Ditto

Dito


Example prop

What changes were proposed in this pull request?

[x] Finshed changes describe
add example properties for byzer.server.mode

How was this patch tested?

[x] Testing done

Are there and DOC need to update?

[ ] Doc is finished

Spark Core Compatibility

test evidence is here


Ad579x doc support

Update documentation for DAC support
Fix command lines hyperlink
Update projects specific rst files and add fixes
Added documentation for ad579x project

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Change Log:
Address comment reviews;
Use uppercase for the tool and tab names
Remove redundant cyclic buffer streaming description
Remove 'full scale' from the data streaming description


Question: There is a way to speed up the archive ?

Do you know if there is a way to speed up the archive task?
It's spending around 31 minutes generating the archive. It would be great if you know some trick/cache to save a few minutes in this task.
Thank you !

Hi @sejas!
Generally we're recommending to turn off the bitcode options, in case you are not looking to send that to Apple and having Apple recompile the app based on that, that can speed up things by a significant amount.

Thanks for your answer ! 🤗
I will try this out.
Have a nice day !


feat: indicate app version in start up icon?

to make it easier for RAs to know they have the latest & greatest version downloaded

now changed to including version number in the welcome screen of task...
will need to remember to update this every time there is a new release

#143


Unsupported Windows Version: 10.0.17686

Can you make a 'riskier' version which supports all Windows 10 builds, but at your own risk. I would love to use this but run the Insider loop.

Insider builds are not supported and won't be. It is a waste of time.

F0x already answered... but to clarify....   Insider builds would be ultra time waster.


Move InkAPI.cc into src/api

This PR moves InkAPI.cc into src/api, so that it's built into the libtsapi.so. A new libtsapicore.a is created to separate out the stuff also used in the core(e.g. Metrics) from the high-level plugin APIs.
Also moved some files around:

Moved Hoststatus.cc from src/traffic_server/ to proxy/ where the header is.
Moved FetchSM.h/cc from src/traffic_server/ to include/api and src/api respectively.
Moved InkContInternal.cc and InkIOCoreAPI.cc from src/traffic_server/ to src/api.
Moved InkAPIInternal.h from proxy/ to include/api.

@cmcfarlen Thanks for review and all the suggestions!


Different order of fetching component chunks when using different ways of setting layout in SSR Mode

Environment

Operating System: Linux
Node Version:     v18.20.3
Nuxt Version:     3.13.2
CLI Version:      3.14.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-fvjwpa?file=layouts%2Fcustom.vue
Describe the bug
So I am using Nuxt3 in SSR mode for few pages for a particular project and I observed an interesting behaviour, when setting layout for pages.

When using definePageMeta to set the layout to 'custom' in  page: indexA, upon checking the Performance panel,when loading the page in Chrome DevTools, the layout chunk is fetched first, followed by the page chunk.

2.When using NuxtLayout to set the layout to 'custom' in page: indexB, the Performance panel in Chrome DevTools shows the page chunk being fetched first, followed by the layout chunk.

What is the reason for this behavior? Is it intentional or a bug?
I’m asking because I am optimizing Core Web Vitals for my project, and it's important that resources—whether JS, CSS, images, etc.—for the page are loaded first. Based on the behavior of NuxtLayout, it seems like it's the correct approach for setting layouts in pages in Nuxt 3.
Additional context
Another question relatively unrelated ,why does vue-router access this path whenever running nuxt in dev mode?
WARN  [Vue Router warn]: No match found for location with path "/"
WARN  [Vue Router warn]: No match found for location with path "/"
WARN  [Vue Router warn]: No match found for location with path "/"
WARN  [Vue Router warn]: No match found for location with path "/"
WARN  [Vue Router warn]: No match found for location with path "/"
WARN  [Vue Router warn]: No match found for location with path "/"
WARN  [Vue Router warn]: No match found for location with path "/" (repeated 4 times)
Logs
No response

Hey @manniL !
Thanks alot for noticing this issue and labelling it under performance, can I get an  update of whether this issue has been solved in a different update or whether the root cause has been discovered yet ?


Add support for _shield related API's

authenticate
clear cached realms
roles

clear cached roles
delete role(s)
put role
get role(s)

user

delete user(s)
put user
get user(s)

This also adds support for any request to be able to specify the es-shield-runas-user
http header though the RunAs property on RequestConfiguration,
The OIS version of requests now directly exposes RequestConfiguration

Few comments, otherwise LGTM :+1:


Keep dims

Oooooh I've been waiting on this one for awhile. Any advice on how to use it, for dummies?

Here is a simple example:
xt::xtensor<double, 4> a = xt::reshape_view(xt::arange<double>(5 * 5 * 5 * 5), {5, 5, 5, 5});

auto res = xt::sum(a, {0, 2}, xt::keep_dims | xt::evaluation_strategy::immediate | initial(5));
auto reso = xt::sum(a, {0, 2}, xt::keep_dims | xt::evaluation_strategy::immediate);

So it has to have an evaluation strategy to be able to specify keep_dims. So, for instance, argmin() won't be able to do it?

There is an default evaluation strategy (lazy), you don't have to specify it:
xt::xtensor<double, 4> a = xt::reshape_view(xt::arange<double>(5 * 5 * 5 * 5), {5, 5, 5, 5});

auto res = xt::sum(a, {0, 2}, xt::keep_dims | initial(5));
auto reso = xt::sum(a, {0, 2}, xt::keep_dims);
bool check = (res == reso); // check is true

Sorry for the confusion, it was a copy paste from a unit test.
However, argmin is not a reducer, thus it does not really make sense to define and use keep_dims for it.

Just FYI, I just ran into this open numpy issue regarding adding keepdim to argmin. So it is something they have thought about! https://github.com/numpy/numpy/issues/8710


'Knock' object has no attribute 'authenticate'

I'm trying to trigger a workflow from the Python Knock client.
From the documentation, I understand it can only be via an authenticated client (enhanced security).
So, now I successfully generated a JWT for my user, I am trying authenticate my client with it. There is not much indication on how to do it in Python though. So I took some inspiration from how it is done in React... Without success though, as no authentication method seems to exist on the Knock class (I also gave a look to the Knock's Python SDK source code).
Is a such mechanism even implemented already in Knock's Python SDK?
For what it's worth, here is my script:
from knockapi import Knock
import jwt

def _get_basic_security_client(knock_api_key: str):
    """Create Knock client (without enhanced security)."""
    return Knock(api_key=knock_api_key)

def get_knock_client(knock_api_key: str, knock_signing_key: str, user_id: str):
    """Create Knock client (without enhanced security)."""
    client = _get_basic_security_client(
        knock_api_key=knock_api_key,
    )
    encoded = jwt.encode(
        {"sub": user_id},
        knock_signing_key,
        algorithm="RS256"
    )
    client.authenticate(user_id, encoded)  # <--- fails here as a such method does not exist
    return client

Note: I also tried modifying my code as follows:
from configuration_variables import (
    USER_ID,
    WORKFLOW_KEY,
    KNOCK_SIGNING_KEY_PEM,
)
import jwt
from knockapi import Knock

if __name__ == '__main__':
    # Create Knock client
    client_api_key = jwt.encode(
        {"sub": USER_ID},
        KNOCK_SIGNING_KEY_PEM,
        algorithm="RS256"
    )
    client = Knock(api_key=client_api_key)
    try:
        # trigger the workflow
        response = client.workflows.trigger(
            key=WORKFLOW_KEY,
            recipients=[
                {
                    "id": USER_ID,
                },
            ],
            data={
                "some_key": "some_data,
            },
        )
    except Exception as e:
        print(e)

Now, by evaluating the Exception e at runtime, I'm obtaining an HTTP_401_UNAUTHORIZED response, with the following content:
b'{"code":"api_key_expired","message":"The API key you supplied has expired","status":401,"type":"authentication_error"}'
Which is not relevant since I've just re-generated copy/pasted the KNOCK_SIGNING_KEY_PEM from the dashboard to my Python variables.
Am I missing something?


Ability to add image(s) to a KB Article

be able to add images to a KB article.

added 1m of time spent

marked this issue as related to #10