Fix boltdb

Fix a couple of issues identified by a static analysis tool.

Refer to this link for build results (access rights to CI server needed):
https://edge-jenkins.isgtesting.com/job/test_team-build-maestro/152/

Refer to this link for build results (access rights to CI server needed):
https://edge-jenkins.isgtesting.com/job/test_team-build-maestro/152/


Dependabot security alerts cannot warn about vulnerable sub-dependencies, as repo does not have a lockfile

Cause
This repo does not have a lockfile, as we think having one would cause issues for users [1, 2]. This means that npm sub-dependencies are not tracked.
Consequence
Dependabot will not create security alerts about sub-dependencies, so we may be causing users to install vulnerable code because our dependencies require sub-dependencies with known vulnerabilities. This happened recently with node-sass and tar [3]; we did not know we needed to update node-sass in this repo until there was a post on the GitHub blog about the vulnerability in tar.
Impact of debt
Low
Effort to pay down
Medium.
It would be fairly easy to add a lockfile. However if we did so that might cause issues for our users. If we added a lockfile but also added it to the gitattributes file so it is not included in releases then users would be asked by npm to commit a lockfile.
Overall rating
Low

If we added a lockfile but also added it to the gitattributes file so it is not included in releases then users would be asked by npm to commit a lockfile.

@lfdebrux I tried googling and didnt have much luck - why would npm ask for this, if package-lock.json is not present, since it doesnt ask now. Is it an attribute in package.json?

@joelanman We currently tell npm not to generate a package lock in .npmrc.
I think in this scenario, we'd remove that instruction, and instead add package-lock.json as an export-ignore in the .gitattributes file.
So npm would expect a package-lock to be installed on user's machines.

Could we potentially hook into the release process and add the line to .npmrc for zip releases?

I'm going to close this as we have now added a lockfile.
There's more work to be done deciding how to manage the end user experience with this change, we've added a new issue here: https://github.com/alphagov/govuk-prototype-kit/issues/1145


Bump version to 5.0.1

Description
Purpose

uh. Can you remove the package-lock please? With this the ortc shim would be locked even for patch versions


Metal cakes and other container weirdness

Example: cakes, made with milk buckets, think they have metal in them, because milk buckets are made of metal.
In other words, items in recipes that ordinarily return their containers don't account for their containers being returned, so the outputs contain things they shouldn't.
Orion says he'll sort this out in Aequivaleo but I'm putting it on the tracker so people don't report it.

Believe this is fixed now!


fix(helm): Non-zero exit code on failed chart repository update

Returns non-zero exit code when one of chart repositories failed to update. I think it might be useful for CI processes.
Before:
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Successfully got an update from the "stable" chart repository
Error: Get https://charts.gke/app/index.yaml: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
...Unable to get an update from the "app" chart repository (https://charts.gke/app/):
	plugin "bin/helmpush" exited with error
Update Complete. ⎈ Happy Helming!⎈

After:
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Successfully got an update from the "stable" chart repository
Error: Get https://charts.gke/app/index.yaml: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
...Unable to get an update from the "app" chart repository (https://charts.gke/app/):
	plugin "bin/helmpush" exited with error
Error: Update Failed. Check log for details

Is it looks fine? Please comment.

cncf-cla ok

this might be a breaking change so we might not be able to introduce this in Helm 2 unless it was behind a feature flag.

Seems it's nice approach for this time, I think. I'll return to this tomorrow and rename PR.
Thanks @bacongobbler

This is something we should quickly discuss on the next dev call. I've added it to the agenda.

Sure!

hey @leominov, I'm currently looking at the 2.11 list and considering this as a candidate to be merged. Would you like to have this in for the 2.11 release, and if so would you be able to take the time to finish this off? Thanks!

due to inactivity I'm going to bump this off the 2.11 milestone.

@bacongobbler should we add the test in a separate PR if @leominov doesn't have the time to do it? Then we could merge them together. Just thinking out loud.

@mattfarina yeah I think that's a good idea

Is this intentionally unavailable in v3? Is there another more global strategy to accomplish this same behavior? It seems like even the latest v2.16.1 has this code so it wasn't reverted in v2?

At some point in mid-2018, tiller was removed from the helm 3 codebase. With the amount of code removed, we could no longer cherry-pick branches into Helm 3, so we were porting them over manually. Looks like this one was missed during the transition.
I’ll mark this as a PR we should look into porting to Helm 3. If you would like to contribute the port, please feel free.


fix: issue with pericopies being cutted

Outdated. All commits  are in the
https://github.com/etenlab/crowd.rocks/pull/562


empty email is not valid

new EmailValidator().isValid('') returns false.
IMO it should return true. Most of the times an empty email is ok. If it's a required field it should fail under another validation.

This library deals with email address validation, not with form validation. The empty string is not a valid email address:

It doesn't conform to Addr-Spec in RFC5322
It is not considered valid by the Java Commons-Validator that this library is a port of
Gmail won't let you send to it.


[zshmarks] no longer need showmarks

From jocelynmallon/zshmarks#17.
showmarks<RET> can be replaced by jump<TAB>, i.e. autocompleting all available bookmarks.

Reverted; somehow the old autocompletion script (_jump_showmarks_deletemark) is still invoked.

No longer worth the hassle.


App doesn't receive token after being refreshed

What I'm trying to achieve
Refresh the app inside the iframe by using window.location.reload()
Describe a proposed solution
Dashboard / App Bridge should detect the app reload and re-initialize
Other solutions I've tried and won't work
Reloading the app manually by user - it works, but it doesn't really solve the root cause of the issue 😅
Screenshots or mockups
https://user-images.githubusercontent.com/4144459/229093777-b4409515-35ae-4273-aed2-fe86095f93ae.mp4

Please try this
appBridge.sendNotifyReadyAction()

this should trigger dashboard to retry, let me know :)


Change/clean validation error handling.

Errors now look as such:
[
  post: [
    must_be_longer_than: 2,
    just_another_test: nil
  ],
  number: [
    is_42: false
  ]
]

Adding errors to a field will concatenate them to the existing error key or create a new key for the field.
ALL errors must have the type {atom, term}. This means that single atom errors must have nil as the second index.

Whoops! Forgot to change cast/4 and merge/2.

Should errors be merged, e.g.,
changeset = change(%Post{}, %{title: ""})
  |> add_error(changeset, :title, {must_be_longer_than: 2})
  |> add_error(changeset, :title, {must_be_longer_than: 3})

Should it result in:
[title: [must_be_longer_than: 3]] or
[title: [must_be_longer_than: 2, must_be_longer_than: 3]]?

That's somewhat how they were used to work but I would prefer to keep the errors in the order they are defined which is how it works today. If we are going to move to this direction, then we should make them maps, but maps are somewhat awkward to match on for the empty case.
Let me pull @drewolson and @jeregrine to see if they have any preference.
Thanks for the pull request!

I'm fine with the current error structure. It's easy enough to write a helper that joins errors together by field name for display purposes -- this could even be provided by phoenix.
I find the single atom error case with a value of nil pretty bizarre, seems like an indication that this solution has some gaps.

@drewolson if you have the field name, you can easily group them with Keyword.get_values/2. General grouping can done easily with Enum.group_by/2. :) Closing this then!


Contract interaction broken since Hangzhou

When interacting with a function on a contract and signing the tx with Temple wallet (in firefox) i recently always get a
"Failed to send operations. cannot use 'in' operator to search for "prim" in "{"int":"0"}" in Temple wallet for all interactions across dfferent contracts
for example:
https://better-call.dev/hangzhou2net/KT18pA1mwpefkE2obC6d2jcQbQH9MaRQCRTZ/interact?entrypoint=default
this is the tx data from the temple wallet screen before sending the (failed) tx.
[
  {
    "destination": "KT18pA1mwpefkE2obC6d2jcQbQH9MaRQCRTZ",
    "kind": "transaction",
    "amount": 1,
    "parameters": {
      "entrypoint": "default",
      "value": "{\"int\":\"0\"}"
    }
  }
]

Fixed

Thank you. When will the fix be deployed? I'm giving a workshop tomorrow and wanted to show contract interaction to the audience.

It's already deployed, might require a force page refresh


Enhance SortMergeJoin to support Join filters(non-equal Join conditions)

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
(This section helps Arrow developers understand the context and why for this feature, in addition to  the what)
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi @alamb,
Just wanted actualize this issue and enrich with some information.
Currently Join filters are disabled for SortMergeJoin with TODO and it produces error in such case.
So if I will disable prefer_hash_join config, it will lead to errors for such queries.

Thanks for the report @DDtKey  -- it would likely help to write some tests (maybe sqllogictest style) that showed the errors (and leave a comment pointing back at this ticket).

AFAIK closed by https://github.com/apache/datafusion/pull/9080


'AllParametersOptional = true' doesn't work at design-time on Linux, .netcore2.2

Hi,
The following code (taken from examples on main page)
use conn = new Npgsql.NpgsqlConnection(DbConnStr) in conn.Open()
use cmd = new FSharp.Data.Npgsql.NpgsqlCommand<"
                            SELECT coalesce(@x, 'Empty') AS x"
						, Connection = DbConnStr
						, AllParametersOptional = true
						, SingleRow = true>(DbConnStr)
cmd.Execute( Some "test" )

compiles successfully, but gives me the following error in IDE:
internal error: destNullableTy: not a Nullable type (Failure) F# Compiler (73)
This error renders all code in red and overwrites any other errors that language server may have generated.
If AllParametersOptional is set to false - everything works in IDE.
The behaviour is the same if i'm using .CreateCommand<...> type.
I also tried to use both net and netcore for FsAutoComplete - results are the same.
I'm using VS Code with Ionide on CentOS 7.
mono --version
Mono JIT compiler version <IP_ADDRESS> (tarball Tue Feb  4 19:25:27 UTC 2020)

dotnet --version
2.2.402

FSharp.Data.Npgsql is 0.1.50-beta
Target platform of project is netcoreapp2.2

FSAC recommends having SDK 3.0+. Have you tried it?

Ah, here you go https://github.com/dotnet/fsharp/issues/8646. The TP is not the problem.

Thanks, downgrading Ionide worked for me!

As a note: https://github.com/dotnet/fsharp/pull/8655 should fix it once approved by Don.


docs(setup): add instruction for build toolchain

Description
Adds some documentation as well as CodeSandbox examples to explain how to use build toolchain.
Changelog
New

Some documentation as well as CodeSandbox examples to explain how to use build toolchain.

Thank you for spotting those typos @kennylam! Updated.


Material Design Color Palette Shift

Cleaning up and prettifying the main view
Moved most colors into variables files for ease of use
General overall cleaner sass
Removal of all gradients (so 2016)
Overall flatter design

When I reset my branch to before I made changes I FUBAR'd my master branch, closing this PR and opening a new one with the cleaned up formatting.


Rclone

additional fields needed
           dropbox_loki: true
+          mount_rclone_output:
+              target: /mnt/integrity_store/starling/shared/
+              src: /mnt/integrity_store/store/rclone/sync/shared-input/
+          rclone_remote_path: /integrity-stg-starling-org

Need to deal with the mounting
should be like
/mnt/integrity_store/starling/shared /mnt/integrity_store/store/rclone/sync/shared-output none defaults,bind 0 0


Bump react-resizable-panels version.

As react-resizable-panels is still using 0.0.x, I think the upload does not happen automagically.
https://github.com/npm/node-semver#caret-ranges-123-025-004

Ugh! forgot about lockfiles, I guess I'll need to checkout the branch and push, I made the previous edit in the Github UI 😛

Alright, lockfiles updated, commit message fixed.
(Note: the commit msg linting hook does not work with commit --amend 🤷 , I guess the file is not called COMMIT_MSG in that case... )

What version of pnpm are you using? We switched the lock file format to 6.0 and your PR appears to use 5.3. Could you try using<EMAIL_ADDRESS>and above, please?

Oof, sorry, I guess I haven't used pnpm in a while 😬  - updated to 8.6.5 👍


Question: How did you implement this in your materialize sidnav?

I cant seem to get the scrollbar to work using your materialize lib.  Just wondering if there was a trick to it or something to look for?
Great projects, btw!!

@theiviaxx thx!
I used the MalihuScrollbarService to initialize the scrollbar by targeting the sidenav.
You can see the code here: https://github.com/sherweb/ng2-materialize/blob/master/demo-app/src/app/app.component.ts#L47-L49


Configs won't save until server shutdown

All configs associated with a YAML-file won't save new data until the server shuts down or the plugin gets disabled.
While this does not directly affect performance and doesn't cause any bugs, it would be helpful for manual edits of the user config, etc.

Redundant since v2 is now the development version.


Render only one region

Hi,
I am quite new to Meteor / Iron Router.
I have an app with a layout with {{> yield}} and {{> yield "subcontent"}}.
One address "/main/:_pageNumber" sends a data context and template to {{> yield}}.
I would like to send another data context to {{> yield subcontent}} without re-rendering {{> yield}} via the address "/main/:_pageNumber/edit/:_id".
I manage to send both data context but both regions re-render. I would like to render only one and leave the other one.
Is there a way to do it ?
Thanks for the help !

hey @16292 without seeing the actual code, it's tough to say why the re-rendering is happening


Embed fix in earth_photos

Relevant Issues
Fixes a small visual bug on mobile in the .earth_photos command
Description
Made a change suggested by ToxicKidz on discord shown here
Reasoning
The bug was annoying
Screenshots
Before fix:

After fix:

Did you:

[x] Join the Python Discord Community?
[didnt change] If dependencies have been added or updated, run pipenv lock?
[x] Lint your code (pipenv run lint)?
[x] Set the PR to allow edits from contributors?

@Xithrius could you review

Oops sorry Chris, hit the wrong thing


Introduce ValidatorFn and BlackList implementation

Remove UnsupportedProperties which is swarm-specific
Introduce a flexible ValidatorFn option for implementation to check supported features
Offer a Blacklist helper factory to set ValidatorFn based on a JSONPath blacklist

@ndeloof this one looks good to me :+1:


Make this available to versions older than 1903

This is probably the longest shot I've done with Microsoft.
Without wanting to get into discussion about the matter, the Windows update is broken, and version 1903 will only be available for my machine September 2020, company policy.
The fact that the windows terminal requires 1903 is a deal breaker.
Any chance to get compiled binaries from somewhere else than the App Store for <1903?

We've actually discussed at length in this repo about how we're dependent upon OS features that are only available in 1903 or later. Because of them, we unfortunatelywon't be able to bring the terminal downlevel :/.

What a bummer. So no windows terminal after all. So much hype for nothing.

Anyway, thanks for the quick reply.


Game is not starting up

Geode Issue

[ ] I confirm that this crash is NOT related to a mod but directly to Geode Loader itself.

Platform
Windows
Geode Version
2.0.0 b21
Mods Installed
~ | [v1.1.1] adyagd.godlikefaces
~ | [v1.2.8] alphalaneous.deathlink
~ | [v1.8.0] beat.click-sound
~ | [v1.0.3] bitz.moregames
~ | [v1.0.5] capeling.gamemode_picker
~ | [v1.0.2] capeling.soggy-mod
| [v1.0.1] cdc.level_thumbnails
~ | [v1.0.1] cj_link.fakemoderator
~ | [v1.3.1] creatorcreepy.unixsync
| [v1.0.0] creo.music_unlock_hack
~ | [v1.3.6] dankmeme.globed2
~ | [v2.5.2] fleym.nongd
~ | [v1.5.2] geode.custom-keybinds
x | [v2.0.0-beta.21] geode.loader
x | [v1.8.1] geode.node-ids
~ | [v6.3.3] hjfod.betteredit
~ | [v1.0.5] hjfod.gdshare
~ | [v1.0.5] hjfod.gmd-api
~ | [v1.1.0] itzkiba.grandpa_demon
~ | [v1.0.0] mat.reference-image
~ | [v1.0.2] mat.run-info
~ | [v2.8.7] maxnu.gd_mega_overlay
| [v2.0.4-beta.1] muhammadgames.bettermenu
~ | [v1.1.0] sofabeddd.colored_icon_kit
~ | [v1.0.1] thesillydoggo.animatedprofiles
~ | [v1.0.0-beta.5] thesillydoggo.comment_emojis
~ | [v2.2.4] TheSillyDoggo.GradientPages
x | [v1.6.8] tpdea.betterpause-Better
Steps to Reproduce
I do not know how to reproduce the crash; it happened naturally
Crash Report
2024-03-27_15-56-38.log
Additional Information
No response

if you still have this issue, it seems to be a graphics drivers/gpu problem. please verify that you're on the latest (and correct!) drivers for your gpu


WPF app that has multiple targets

I am trying to package up a WPF application I have and I have run into a wall and can't figure out what to do next besides give up. My app targets .Net Framework 4.8 and .Net Core 3.1 and 5.0 Preview. When building the package project after adding my application to it I get an error message that "Project targets 'net48;netcoreapp3.1;netcoreapp5.0' and it cannot be referenced by a project that targets .NetCore,Version=5.0.
How can I fix this? Is there a way to tell the package project to create a package for each framework? Does this only work if your app targets a single .Net?

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

ID: d7844818-d0b7-f502-3a07-a4970401fdcf
Version Independent ID: 1acada53-18fa-68a2-4211-c4c3236c856e
Content: Package and deploy with MSIX
Content Source: hub/apps/desktop/modernize/modernize-wpf-tutorial-5.md
Product: uwp
Technology: apps
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Thank you for reaching out to us, and I'm sorry you're running into this issue. Unfortunately, this repo is set up to track feedback and bugs on the documentation, and it is not an avenue for technical support requests. To get assistance with your issue, your best bet is to post a question on the MSIX Tech Community. Members of the product team that own the MSIX and desktop packaging technology monitor this community and respond to customer questions.

Thank you for reaching out to us, and I'm sorry you're running into this issue. Unfortunately, this repo is set up to track feedback and bugs on the documentation, and it is not an avenue for technical support requests. To get assistance with your issue, your best bet is to post a question on the MSIX Tech Community. Members of the product team that own the MSIX and desktop packaging technology monitor this community and respond to customer questions.

Thank you for the link and sorry for creating the issue. I've seen the docs feedback used for questions before so i thought it was okay. I now know its not meant for that. Thanks again and sorry


Can't create complex if complex's name = name of one of its parent's children

From: https://forum.inaturalist.org/t/species-with-same-name-as-complex-prevents-creation-of-complex-at-the-correct-location/23564

Create a new species complex taxon with the name Amaranthus deflexus and parent Amaranthus.
See this warning:

Most (all?) complexes will have the same name as one of its parent's children, so this shouldn't be a blocking error when creating a new complex.

I can take a look at this @tiwane. I'm also assuming that we should only be able to create a single sibling complex with that duplicated name. For example, once the 'Amaranthus deflexus' complex is created (alongside 'Amaranthus deflexus' species), no other complexes named 'Amaranthus deflexus' should be allowed as a child of 'Amaranthus'. Sound right?


Notification should not hide content according to ux

Issue description:
Notification will hide some content when it popup.
According to ux requirement, we should make it not extend to the whole widh of UI, and should not hide the content.
Fix:
Modify the position from 'absolute' to 'relative', so that it will not hide the content.

LGTM


About version

Can you provide the version of Python and tensorflow to run you code?

Hi,
This is a very old code, it worked with python 3.6 and TF 1.12, I haven't tested with newer versions.


[BUG] vci auth login web does not work with Safari - blocks non-HTTPS even if its localhost

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
safari blocks non-https traffic even if its localhost.
https://stackoverflow.com/questions/46394682/safari-keeps-forcing-https-on-localhost
Expected Behavior
allow non-https from localhost since its not really remote
Steps To Reproduce
No response
Which operating system where you on?
Mac
Anything else?
possible fixes

[ ] add copy to warn that this does not work (non-breaking)
[ ] look into shipping the CLI with its own cert/key so that it can serve properly over https ( breaking )

https://github.com/denji/golang-tls


Remove the reference to name (username) in create_droplet.yml

Remove the reference to name (username) in create_droplet.yml

#8 covers this. closing this issue.


Announcements awaiting interaction (Enter key)

Some (all?) announcements aren't being closed, awaiting the user to press "Enter"
Eamples:
Here are two examples I noticed on my last run, awaiting the "Enter" key

xyz has assumed the position of expedition leader of abc
dwarf or animal is thirsty or starving

Version Info:

DF-AI 0.44.12-r1 from 7/17/18 Buildmaster
DFHack 0.44.12-r1-0-gc2a64b35

Extra:
This issues was also mentioned in issue 48

The first one isn't a DFHack announcement. Also, I'm not sure why you would duplicate an open issue.

Update - Removed "DFHack" from name and description. :)

Similar to #37 - warn-starving, warn-stuck-trees, x for additional options

For future reference, the ai version command should give you a pretty complete set of version information.

I tied using that yesterday, but nothing was showing up in the console. 🤔

Is this happening when the AI is performing a specific action? Is there a copy of the pause message in the AI's log?

Since I've switched to using the latest build master releases, I've not encountered this bug since reporting it


Empty style got undefined

// styles.scss
.example {
}

render () {
  return

}

When .example is empty, styles.example will become undefined.
How to avoid this? since I don't need to set styles to .example right now, but have to override its styles somewhere else, so I need to get a dedicated class name.
Also reported in https://github.com/css-modules/css-modules/issues/140

What should it generate? I wonder if this a spec issue.

It should be Card__example if the localIdentName=[name]__[local], but got undefined

I think empty string would be fault tolerant, but depends on whether you want to strip unstyled classes or not

I want to keep the className because I want to override it somewhere else.

If you follow the chain of issues from the CSS Modules one, it turns out this is actually to do with SASS removing empty styles, which can be overridden with a comment - see https://github.com/sass/sass/issues/2069.
This is working fine for me if I use the mixin in that issue.


Add user not found popup alert

Issue that this pull request solves
Closes: #123
Proposed changes
Added an alert message when the login request fails due to a not registered email address.
Types of changes
Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] My changes does not break the current system and it passes all the current test cases.

Screenshots

Thanks for your contribution @matheusmosca :)


functional: do not use omega character

this patch replace omega character with Expect to make
tests easier to read and understand
fixes #192
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

@devimc many thanks for this. Sorry I never got around to it...
lgtm

hi @grahamwhaley  no problem 😄


MM-18766 - componentWillReceiveProps is being deprecated

Summary

componentWillReceiveProps is being deprecated. Replace its usage in app/screens/image_preview/downloader.ios.js.
Bonus UI polish - The download progress component's contents shift up after the component is displayed - avoid this jitter.

Ticket Link
GH: https://github.com/mattermost/mattermost-server/issues/12353 (the ticket calls out several components - since this my first contribution I wanted to keep the PR small and started with just one).
JIRA:  https://mattermost.atlassian.net/browse/MM-18766
Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] All new/modified APIs include changes to mattermost-redux (please link)
[X] Has UI changes
[ ] Includes text changes and localization file updates

Device Information
This PR was tested on: iPhone X Simulator Running IOS 13.
Screenshots
Successfully download the image:

Canceling the download then trying again:

Commented above, @smacgregor feel free to submit a follow-up PR for the UI/UX polish you mentioned above. Thanks!

@esethna - do you mind creating the follow up tickets for me in case another contributor wants to tackle them before I get to it? Thanks!

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

@saturninoabril Quick reminder to help with QA review on this PR :)
@esethna did we create follow-up tickets on the above UX/UI polish?


Support Swipe-to-Dismiss behavior

Feature request:
I would like to ask it is possible to support Swipe-to-Dismiss behavior in Zoomable library?
I see the API like this:
val zoomState = rememberZoomState()

Modifier.zoomable(
    zoomState = zoomState,
    // enables behavior
    enableDismissGesture = true,
    // callback for progress listening, may be needed for changing background alpha
    // provides value between 0F and 1F
    onDismissProgressChange = { progress: Float -> Log.d("MyTag", "Dismiss progress $progress") },
    // triggers when progress reaches 1F
    onDismiss = { Log.d("MyTag, "Dismissed" },
),

Similar behavior you can see in Telegram or at the video:
https://github.com/usuiat/Zoomable/assets/16479387/e1769b4c-95d9-4528-b8f3-7dafcffce9ab

Thank you for your suggestion.
It’s interesting to implement swipe-to-dismiss as a library.
However, I think swipe-to-dismiss should be provided as an independent feature from zoomable, so I would not add it to this library.
FYI
Zoomable can be used in conjunction with gesture detection modifiers such as dragable, since zoomable makes PointerEvents transparent that it does not consume.
You can implement your own swipe-to-dismiss and use it in conjunction with zoomable.


[Story] Revamp Database

As a developer I need to be have all the information stored properly so that I can properly build the application and machine learning models.
Acceptance Criteria:

[x] Given the database, when the user makes changes then all the information is stored
[x] Given the application is open when the user interacts with it then the information from the database should be interactive in some way.

Finished with commit 9f571313ccc915f5b699d3f406f42fd7abb2732c


Warning about highlight.js in changelog

Hi @ef4 on the last 2 new releases you added, on top of the changelog, the following message, maybe it was a copy paste gone wrong?

Version 9 of Highlight.js has reached EOL and is no longer supported.
Please upgrade or ask whatever dependency you are using to upgrade.
https://github.com/highlightjs/highlight.js/issues/2877

If it's not a mistake can you explain better what this means?
Thank you in advance

I think that is a bug in the changelog renderer itself. 🙃
On Wed, Oct 9, 2024 at 4:55 AM João Teixeira @.***>
wrote:

Hi @ef4 https://github.com/ef4 on the last 2 new releases you added, on
top of the changelog, the following message, maybe it was a copy paste gone
wrong?
Version 9 of Highlight.js has reached EOL and is no longer supported.
Please upgrade or ask whatever dependency you are using to upgrade.
highlightjs/highlight.js#2877
https://github.com/highlightjs/highlight.js/issues/2877
If it's not a mistake can you explain better what this means?
Thank you in advance
—
Reply to this email directly, view it on GitHub
https://github.com/ember-cli/ember-load-initializers/issues/310, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACN6MWY7SCNEXIUKEXUSADZ2TVPDAVCNFSM6AAAAABPUARWNKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU3TKMRWGA2TANY
.
You are receiving this because you were mentioned.Message ID:
@.***>


[Bug]: Tests fail only when running on Github Actions (locally runs fine)

Version
28.0
Steps to reproduce

Create yarn workspace with two workspaces inside packages folder

Create test in first workspace that has a dependency on second workspace

Run yarn test locally, no issue

Run on Github Actions error: Unable to process '/home/runner/work/arch3.js/arch3.js/packages/arch3-core/src/archwayclient.ts', please make sure that outDir in your tsconfig is neither '' or '.'. You can also configure Jest config option transformIgnorePatterns to inform ts-jest to transform /home/runner/work/arch3.js/arch3.js/packages/arch3-core/src/archwayclient.ts
at TsCompiler.getCompiledOutput (../../node_modules/ts-jest/dist/legacy/compiler/ts-compiler.js:165:27)
at TsJestCompiler.getCompiledOutput (../../node_modules/ts-jest/dist/legacy/compiler/ts-jest-compiler.js:13:39)
at TsJestTransformer.processWithTs (../../node_modules/ts-jest/dist/legacy/ts-jest-transformer.js:232:37)
at TsJestTransformer.process (../../node_modules/ts-jest/dist/legacy/ts-jest-transformer.js:161:24)
at ScriptTransformer.transformSource (../../node_modules/@jest/transform/build/ScriptTransformer.js:619:31)
at ScriptTransformer._transformAndBuildScript (../../node_modules/@jest/transform/build/ScriptTransformer.js:765:40)
at ScriptTransformer.transform (../../node_modules/@jest/transform/build/ScriptTransformer.js:822:19)

Expected behavior
Should run to completion like it does on local machine.
Actual behavior
Fails saying it cannot process a typescript file. It would help to know what the precise failure in processing the typescript file is.
My outDir is set (again it works locally)
Debug log
11s
Run yarn test:core
FAIL src/archwayclient-rewards.spec.ts
● Test suite failed to run
Unable to process '/home/runner/work/arch3.js/arch3.js/packages/arch3-core/src/archwayclient.ts', please make sure that `outDir` in your tsconfig is neither `''` or `'.'`. You can also configure Jest config option `transformIgnorePatterns` to inform `ts-jest` to transform /home/runner/work/arch3.js/arch3.js/packages/arch3-core/src/archwayclient.ts

  at TsCompiler.getCompiledOutput (../../node_modules/ts-jest/dist/legacy/compiler/ts-compiler.js:[1](https://github.com/archway-network/arch3.js/actions/runs/4473280460/jobs/7860484202?pr=48#step:6:1)65:27)
  at TsJestCompiler.getCompiledOutput (../../node_modules/ts-jest/dist/legacy/compiler/ts-jest-compiler.js:13:39)
  at TsJestTransformer.processWithTs (../../node_modules/ts-jest/dist/legacy/ts-jest-transformer.js:232:37)
  at TsJestTransformer.process (../../node_modules/ts-jest/dist/legacy/ts-jest-transformer.js:161:2[4](https://github.com/archway-network/arch3.js/actions/runs/4473280460/jobs/7860484202?pr=48#step:6:5))
  at ScriptTransformer.transformSource (../../node_modules/@jest/transform/build/ScriptTransformer.js:619:31)
  at ScriptTransformer._transformAndBuildScript (../../node_modules/@jest/transform/build/ScriptTransformer.js:76[5](https://github.com/archway-network/arch3.js/actions/runs/4473280460/jobs/7860484202?pr=48#step:6:6):40)
  at ScriptTransformer.transform (../../node_modules/@jest/transform/build/ScriptTransformer.js:822:19)

FAIL src/archwayclient.spec.ts
● Test suite failed to run
Unable to process '/home/runner/work/arch3.js/arch3.js/packages/arch3-core/src/archwayclient.ts', please make sure that `outDir` in your tsconfig is neither `''` or `'.'`. You can also configure Jest config option `transformIgnorePatterns` to inform `ts-jest` to transform /home/runner/work/arch3.js/arch3.js/packages/arch3-core/src/archwayclient.ts

  at TsCompiler.getCompiledOutput (../../node_modules/ts-jest/dist/legacy/compiler/ts-compiler.js:1[6](https://github.com/archway-network/arch3.js/actions/runs/4473280460/jobs/7860484202?pr=48#step:6:7)5:2[7](https://github.com/archway-network/arch3.js/actions/runs/4473280460/jobs/7860484202?pr=48#step:6:8))
  at TsJestCompiler.getCompiledOutput (../../node_modules/ts-jest/dist/legacy/compiler/ts-jest-compiler.js:13:39)
  at TsJestTransformer.processWithTs (../../node_modules/ts-jest/dist/legacy/ts-jest-transformer.js:232:37)
  at TsJestTransformer.process (../../node_modules/ts-jest/dist/legacy/ts-jest-transformer.js:161:24)
  at ScriptTransformer.transformSource (../../node_modules/@jest/transform/build/ScriptTransformer.js:619:31)
  at ScriptTransformer._transformAndBuildScript (../../node_modules/@jest/transform/build/ScriptTransformer.js:765:40)
  at ScriptTransformer.transform (../../node_modules/@jest/transform/build/ScriptTransformer.js:[8](https://github.com/archway-network/arch3.js/actions/runs/4473280460/jobs/7860484202?pr=48#step:6:9)22:1[9](https://github.com/archway-network/arch3.js/actions/runs/4473280460/jobs/7860484202?pr=48#step:6:10))

------------------|---------|----------|---------|---------|-------------------

File
% Stmts
% Branch
% Funcs
% Lines
Uncovered Line #s

All files
0
0
0
0

archwayclient.ts
0
0
0
0
1-138

index.ts
0
0
0
0
1-2

------------------
---------
----------
---------
---------
-------------------

Test Suites: 2 failed, 2 total

Tests:       0 total

Snapshots:   0 total

Time:        5.769 s

Ran all test suites.

Error: Process completed with exit code 1.

Additional context
Github runner is ubuntu latest
Environment
Here's my local machine setup and then the github action yml.

System:
    OS: macOS 13.2.1
    CPU: (10) arm64 Apple M1 Max
  Binaries:
    Node: 18.12.1 - /usr/local/bin/node
    Yarn: 3.2.1 - /usr/local/bin/yarn
    npm: 8.19.2 - /usr/local/bin/npm
  npmPackages:
    jest: ^28.1 => 28.1.3

name: Test

on:
  push:
    branches: [main, develop]
  pull_request:
    branches: ["*"]

concurrency:
  group: ${{github.workflow}}-${{github.ref}}
  cancel-in-progress: true

jobs:
  test:
    name: Test
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v3
      - uses: actions/setup-node@v3
        with:
          node-version: 14
          cache: yarn
      - run: yarn install --immutable
      - run: yarn build:all
      - run: yarn test:core

I have a very similar issue.
Tests are running ok locally (windows and macos, not tried ubuntu), but in a Github Actions workflow, i have the following message:
FAIL src/__tests__/App.test.tsx
  ● Test suite failed to run

    Unable to process '/home/runner/work/midi-jar/midi-jar/src/__tests__/App.test.tsx', please make sure that `outDir` in your tsconfig is neither `''` or `'.'`. You can also configure Jest config option `transformIgnorePatterns` to inform `ts-jest` to transform /home/runner/work/midi-jar/midi-jar/src/__tests__/App.test.tsx

      at TsCompiler.getCompiledOutput (node_modules/ts-jest/dist/legacy/compiler/ts-compiler.js:173:27)
      at TsJestCompiler.getCompiledOutput (node_modules/ts-jest/dist/legacy/compiler/ts-jest-compiler.js:14:39)
      at TsJestTransformer.exports.TsJestTransformer.TsJestTransformer.processWithTs (node_modules/ts-jest/dist/legacy/ts-jest-transformer.js:265:37)
      at TsJestTransformer.exports.TsJestTransformer.TsJestTransformer.process (node_modules/ts-jest/dist/legacy/ts-jest-transformer.js:188:[24](https://github.com/la-jarre-a-son/midi-jar/actions/runs/6013973240/job/16312644447#step:9:25))
      at ScriptTransformer.transformSource (node_modules/@jest/transform/build/ScriptTransformer.js:545:31)
      at ScriptTransformer._transformAndBuildScript (node_modules/@jest/transform/build/ScriptTransformer.js:674:40)
      at ScriptTransformer.transform (node_modules/@jest/transform/build/ScriptTransformer.js:7[26](https://github.com/la-jarre-a-son/midi-jar/actions/runs/6013973240/job/16312644447#step:9:27):19)

The test:
import '@testing-library/jest-dom';
import { render } from '@testing-library/react';
import App from '../renderer/App';

describe('App', () => {
  it('should render', () => {
    expect(render(<App />)).toBeTruthy();
  });
});

Versions:
    "jest": "^29.6.4",,
    "ts-jest": "^29.1.1",

Config:
{
  "testEnvironment": "jsdom",
  "testEnvironmentOptions": {
    "url": "http://localhost/"
  },
  "transform": {"\\.[jt]sx?$": "ts-jest"},
  "moduleNameMapper": {
    "\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$": "<rootDir>/.erb/mocks/fileMock.js",
    "\\.(css|less|sass|scss)$": "identity-obj-proxy"
  },
  "moduleFileExtensions": [
    "js",
    "jsx",
    "ts",
    "tsx",
    "json"
  ],
  "moduleDirectories": [
    "node_modules",
    "release/app/node_modules",
    "src"
  ],
  "testMatch": [
    "**/*.test.[jt]s?(x)"
  ],
  "testPathIgnorePatterns": [
    "release/app/dist"
  ],
  "setupFiles": [
    "./.erb/scripts/check-build-exists.ts"
  ],
  "setupFilesAfterEnv": [
    "./src/__tests__/setup.ts"
  ]
}

Funny thing (not so much): Github action fails if running windows-latest or ubuntu-latest, but passes when running macos-latest. I spent 2 days trying to reproduce locally, or fixing it without success. I just enabled tests only on macos-latest, until there is another workaround (or new ideas).


AI/ChatGPT - Bill Summaries

Overview
It would be nice if it were possible to summarize bill content without manual effort.

Could implement the following on a hosted platform (such as Huggingface) for a proof of concept: https://huggingface.co/docs/transformers/main/en/tasks/summarization

The spirit of this feature may be helpful: https://github.com/codeforboston/maple/pull/871


Added isreadonly mode, which shows verbose cron setting without need to display dropdowns

@wowo thanks for the PR! What do you think about using ngDisabled on the select with isReadOnly that way we minimize the HTML we have to write and maintain? Let me know what you think.

@wowo I will close this PR if I do not hear back from you soon, thanks.

@jacobscarter  Sorry for missing your comment. I'll have a look into it during the weekend

What do you think about using ngDisabled on the select with isReadOnly that way we minimize the HTML we have to write and maintain? Let me know what you think.

What do you exactly have in mind? Can you share some example?

@jacobscarter and @wowo .
I would argue that the ngDisabled would be great as it should simply disable the select fields as one would expect and should definitely be added. What I would do instead of this PR is simply add a stringify function (or whatever you want to call it) to the cronService which could takes a cron expression string and outputs a human readable string. This way the user could do whatever they want with it and it keeps the template simpler. Also by doing it this way if we did want to add a readOnlyMode property the template would simply need to add {{stringify(cronExpression)}} and hide the rest rather than a bunch of complicated show/hide directives.

@vincentjames501 I think your "human readable" idea is a good new feature to add, I think the ngDisable is another topic.  @wowo I simple meant we can add ng-disabled="isReadOnly" to the selects. If you like that idea can you update your PR and rebase so we can merge.  Thanks.


📖 story "user reports naughty reviews via email"

we're going to defer implementation of a complex system for this,
by simply asking users to report misbehavior via simple support emails

[x] design what this should look like
[ ] #59
[ ] #98
[x] #88

explainer -- should be customizable somehow
Please send an email to the following address to submit your report against this user's comment.

email -- should be customizable

copy-instructions
Please include the following information in the email, so we can identify which comment and user.

copy-textarea
User ID: ${comment.user.id}
Comment ID: ${comment.id}
Reporter ID: ${auth.user.id}

for the customizable text,

maybe it will be done with shadow <slot> elements?
maybe it could be component html attributes?
more likely: it might have to be an installation option for namaka specified in prepare-components.ts and passed into a special "options model" or some such madness


Package MvcSiteMapProvider.MVC5 4.6.22 is not compatible with netcoreapp1.1

When adding the NuGet package to my site with a target framework of .NETCoreApp 1.1, I get the following error.
Package MvcSiteMapProvider.MVC5 4.6.22 is not compatible with netcoreapp1.1 (.NETCoreApp,Version=v1.1). Package MvcSiteMapProvider.MVC5 4.6.22 supports: net45 (.NETFramework,Version=v4.5)
One or more packages are incompatible with .NETCoreApp,Version=v1.1.
Package restore failed. Rolling back package changes

MvcSiteMapProvider doesn't support .Net Core yet see: #394

MvcSiteMapProvider.MVC5.Core doesn't support .Net Core yet.


cli: Use --use-rpc flag correctly during program deployments

Problem
As noticed in https://github.com/anza-xyz/agave/pull/928#discussion_r1576906616, the logic for --use-rpc is unfortunately backwards, since it's creating the tpu client if use_rpc is true, when we only want to create it when it's false.
Summary of Changes
Reverse the logic with a !
Fixes #

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

Project coverage is 81.9%. Comparing base (b8cc66e) to head (936ff2f).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #1008     +/-   ##
=========================================
- Coverage    81.9%    81.9%   -0.1%
=========================================
  Files         855      856      +1
  Lines      232194   232194
=========================================
- Hits       190222   190215      -7
- Misses      41972    41979      +7


Pull common data structure changes

I've made some changes to the common data structures, you might want to pull them in.
I'm going to finish all these with tests and then continue to the structures for tunnel/i2np/i2cp.

awesome


[Bug] Investigate RelayMiner Issues

Objective
Investigate & fix the issue identified during E2E PATH Testing.
Origin Document
See the discord thread here.

Goals & Deliverables

[ ] Identify the root cause in the issue above
[ ] Implement a fix
[ ] Improve the tooling and/or logs to make the issue clearer in the future
[ ] If applicable, improve the tooling

Deliverables

[ ] Deliverable #1
[ ] Deliverable #2

Creator: @Olshansk
Co-Owners: @okdas @adshmh @red-0ne

Closing as pr #1007


IREE Compile Error

With the most recent git pull (7963abb) and a clean environment, only some Model/VAE/LoRA combinations produce the compile issue below. In previous versions, this issue was not present (specifically 0361db4 was the last commit I had installed). I have tried various combinations and have failed to find a potential root cause. Any and all help is appreciated.
System Information

Windows 11 22H2
RX 7900 XTX
Adrenalin 23.10.2
Vulkan 2.0.283
SHARK v7963abb

SystemExit: Error invoking IREE compiler tool iree-compile.exe
Error code: 1
Diagnostics:
<eval_with_key>.33 from D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\torch\fx\experimental\proxy_tensor.py:507 in wrapped:28:12: error: 'func.func' op failed to get lowering configuration
<eval_with_key>.33 from D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\torch\fx\experimental\proxy_tensor.py:507 in wrapped:28:12: error: failed to run translation of source executable to target executable for backend #hal.executable.target<"vulkan", "vulkan-spirv-fb", {spirv.target_env = #spirv.target_env<#spirv.vce<v1.6, [Shader, Float64, Float16, Int64, Int16, Int8, StorageBuffer16BitAccess, StorageUniform16, StoragePushConstant16, StorageBuffer8BitAccess, UniformAndStorageBuffer8BitAccess, StoragePushConstant8, GroupNonUniform, GroupNonUniformVote, GroupNonUniformArithmetic, GroupNonUniformBallot, GroupNonUniformShuffle, GroupNonUniformShuffleRelative, GroupNonUniformClustered, GroupNonUniformQuad, VariablePointers, VariablePointersStorageBuffer, DotProduct, DotProductInputAll, DotProductInput4x8BitPacked, DotProductInput4x8Bit, CooperativeMatrixKHR], [SPV_KHR_16bit_storage, SPV_KHR_8bit_storage, SPV_KHR_integer_dot_product, SPV_KHR_storage_buffer_storage_class, SPV_KHR_variable_pointers, SPV_KHR_cooperative_matrix]>, api=Vulkan, AMD:DiscreteGPU, #spirv.resource_limits<max_compute_shared_memory_size = 65536, max_compute_workgroup_invocations = 1024, max_compute_workgroup_size = [1024, 1024, 1024], subgroup_size = 64, min_subgroup_size = 32, max_subgroup_size = 64, cooperative_matrix_properties_khr = [#spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = i8, b_type = i8, c_type = i32, result_type = i32, acc_sat = false, scope = <Subgroup>>, #spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = f16, b_type = f16, c_type = f16, result_type = f16, acc_sat = false, scope = <Subgroup>>, #spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = f16, b_type = f16, c_type = f32, result_type = f32, acc_sat = false, scope = <Subgroup>>]>>}>
<eval_with_key>.33 from D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\torch\fx\experimental\proxy_tensor.py:507 in wrapped:28:12: error: failed to serialize executables
<eval_with_key>.33 from D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\torch\fx\experimental\proxy_tensor.py:507 in wrapped:33:14: error: 'func.func' op failed to get lowering configuration
<eval_with_key>.33 from D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\torch\fx\experimental\proxy_tensor.py:507 in wrapped:33:14: error: failed to run translation of source executable to target executable for backend #hal.executable.target<"vulkan", "vulkan-spirv-fb", {spirv.target_env = #spirv.target_env<#spirv.vce<v1.6, [Shader, Float64, Float16, Int64, Int16, Int8, StorageBuffer16BitAccess, StorageUniform16, StoragePushConstant16, StorageBuffer8BitAccess, UniformAndStorageBuffer8BitAccess, StoragePushConstant8, GroupNonUniform, GroupNonUniformVote, GroupNonUniformArithmetic, GroupNonUniformBallot, GroupNonUniformShuffle, GroupNonUniformShuffleRelative, GroupNonUniformClustered, GroupNonUniformQuad, VariablePointers, VariablePointersStorageBuffer, DotProduct, DotProductInputAll, DotProductInput4x8BitPacked, DotProductInput4x8Bit, CooperativeMatrixKHR], [SPV_KHR_16bit_storage, SPV_KHR_8bit_storage, SPV_KHR_integer_dot_product, SPV_KHR_storage_buffer_storage_class, SPV_KHR_variable_pointers, SPV_KHR_cooperative_matrix]>, api=Vulkan, AMD:DiscreteGPU, #spirv.resource_limits<max_compute_shared_memory_size = 65536, max_compute_workgroup_invocations = 1024, max_compute_workgroup_size = [1024, 1024, 1024], subgroup_size = 64, min_subgroup_size = 32, max_subgroup_size = 64, cooperative_matrix_properties_khr = [#spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = i8, b_type = i8, c_type = i32, result_type = i32, acc_sat = false, scope = <Subgroup>>, #spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = f16, b_type = f16, c_type = f16, result_type = f16, acc_sat = false, scope = <Subgroup>>, #spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = f16, b_type = f16, c_type = f32, result_type = f32, acc_sat = false, scope = <Subgroup>>]>>}>
<eval_with_key>.33 from D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\torch\fx\experimental\proxy_tensor.py:507 in wrapped:33:14: error: failed to serialize executables

Invoked with:
 iree-compile.exe D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\iree\compiler\tools\..\_mlir_libs\iree-compile.exe .\unet512_1_64_512_512_fp16_tuned_lilymix_v2_SecretSauce610-fp16-pruned_vulkan_torch_linalg.mlir --iree-input-type=tm_tensor --iree-vm-bytecode-module-output-format=flatbuffer-binary --iree-hal-target-backends=vulkan --mlir-print-debuginfo --mlir-print-op-on-diagnostic=false --iree-llvmcpu-target-cpu-features=host --iree-vulkan-target-env=#vk.target_env<v1.3, r(120), [VK_KHR_16bit_storage, VK_KHR_8bit_storage, VK_KHR_shader_float16_int8, VK_KHR_spirv_1_4, VK_KHR_storage_buffer_storage_class, VK_KHR_variable_pointers, VK_EXT_subgroup_size_control, VK_KHR_cooperative_matrix], AMD:DiscreteGPU, #vk.caps< maxComputeSharedMemorySize = 65536, maxComputeWorkGroupInvocations = 1024, maxComputeWorkGroupSize = dense<[1024, 1024, 1024]>: vector<3xi32>, subgroupSize = 64, subgroupFeatures = 255: i32, minSubgroupSize = 32, maxSubgroupSize = 64, shaderFloat16 = unit, shaderFloat64 = unit, shaderInt8 = unit, shaderInt16 = unit, shaderInt64 = unit, storageBuffer16BitAccess = unit, storagePushConstant16 = unit, uniformAndStorageBuffer16BitAccess = unit, storageBuffer8BitAccess = unit, storagePushConstant8 = unit, uniformAndStorageBuffer8BitAccess = unit, variablePointers = unit, variablePointersStorageBuffer = unit, cooperativeMatrixPropertiesKHR = [#vk.coop_matrix_props<mSize = 16, nSize = 16, kSize = 16, aType = f16, bType = f16, cType = f16, resultType = f16, accSat = false, scope = #vk.scope<Subgroup>>], shaderIntegerDotProduct = unit >> --iree-stream-resource-max-allocation-size=4294967295 --iree-vm-bytecode-module-strip-source-map=true --iree-util-zero-fill-elided-attrs --iree-opt-strip-assertions=true --verify=false -iree-vulkan-target-triple=rdna3-7900-windows

Need more information? Set IREE_SAVE_TEMPS=/some/dir in your environment to save all artifacts and reproducers.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\starlette\routing.py", line 686, in lifespan
    await receive()
  File "D:\Stable\stable-diffusion-shark\shark.venv\Lib\site-packages\uvicorn\lifespan\on.py", line 137, in receive
    return await self.receive_queue.get()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.11_3.11.1776.0_x64__qbz5n2kfra8p0\Lib\asyncio\queues.py", line 158, in get
    await getter
asyncio.exceptions.CancelledError

Thank you. Will get this looked at today.
(I also note that turbine isn't preserving source location properly and that can be fixed)

Thanks for the heads up. I'm looking into this right now, trying to reproduce. If there are any patterns in which unet/vae/lora combinations I should look at, let me know.

@monorimet The specific combination used in this example was as follows:
Model: Lily Mix v2
VAE: kl-f8-animev2
LoRA: Secret Sauce - 610
Each of these links is a civitai download link directly to the versions used. That being said, many other combinations of Model, VAE and LoRA have caused this issue.

@monorimet The specific combination used in this example was as follows:
Model: Lily Mix v2
VAE: kl-f8-animev2
LoRA: Secret Sauce - 610
Each of these links is a civitai download link directly to the versions used. That being said, many other combinations of Model, VAE and LoRA have caused this issue.

Thanks, I'll find out what's confusing the pipeline.

Something changes in the unet model when we use LoRAs. It seems to work fine without the tunings, but with tunings I reproduce your error.

I'll hypothesize that the bad combo is RDNA3 tuned + SD1.X UNet + LoRA weights since the custom VAE shouldn't impact UNet compilation. I'll take your word that only certain models/loras do this and try to figure out what's happening exactly.

We can operate under the assumption that the VAE does not affect anything, since that much has been more or less unproven since I use a VAE in every generation. The theory about tuned vs. untuned is supported because, using the same setup after a --clear_all, remove the LoRA and re-generate with your prompt. The error should not occur. Does this make the LoRA the breaking point, or the tuned model the breaking point?

Probably the tuned LoRA configuration, I don't see this without the LoRA or without tuning. We probably have to make some tuning config changes, I'll narrow this down and disable tunings for the failure modes and file a separate issue for a gentle rework of the tunings to better handle some of these custom model configs.

It looks like this happens without LoRAs now, too... I've got a minimal reproducer with the following IR:
https://storage.googleapis.com/shark-public/ean/unet_lc/module_forward_dispatch_5.mlir
The error (Also on the CI now):
<eval_with_key>.12 from C:\actions-runner\w\SHARK\SHARK\shark.venv\Lib\site-packages\torch\fx\experimental\proxy_tensor.py:507 in wrapped:28:12: error: 'func.func' op failed to get lowering configuration

<eval_with_key>.12 from C:\actions-runner\w\SHARK\SHARK\shark.venv\Lib\site-packages\torch\fx\experimental\proxy_tensor.py:507 in wrapped:28:12: error: failed to run translation of source executable to target executable for backend #hal.executable.target<"vulkan", "vulkan-spirv-fb", {spirv.target_env = #spirv.target_env<#spirv.vce<v1.6, [Shader, Float64, Float16, Int64, Int16, Int8, StorageBuffer16BitAccess, StorageUniform16, StoragePushConstant16, StorageBuffer8BitAccess, UniformAndStorageBuffer8BitAccess, StoragePushConstant8, GroupNonUniform, GroupNonUniformVote, GroupNonUniformArithmetic, GroupNonUniformBallot, GroupNonUniformShuffle, GroupNonUniformShuffleRelative, GroupNonUniformClustered, GroupNonUniformQuad, VariablePointers, VariablePointersStorageBuffer, DotProduct, DotProductInputAll, DotProductInput4x8BitPacked, DotProductInput4x8Bit, CooperativeMatrixKHR], [SPV_KHR_16bit_storage, SPV_KHR_8bit_storage, SPV_KHR_integer_dot_product, SPV_KHR_storage_buffer_storage_class, SPV_KHR_variable_pointers, SPV_KHR_cooperative_matrix]>, api=Vulkan, AMD:DiscreteGPU, #spirv.resource_limits<max_compute_shared_memory_size = 65536, max_compute_workgroup_invocations = 1024, max_compute_workgroup_size = [1024, 1024, 1024], subgroup_size = 64, min_subgroup_size = 32, max_subgroup_size = 64, cooperative_matrix_properties_khr = [#spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = i8, b_type = i8, c_type = i32, result_type = i32, acc_sat = false, scope = <Subgroup>>, #spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = f16, b_type = f16, c_type = f16, result_type = f16, acc_sat = false, scope = <Subgroup>>, #spirv.coop_matrix_props_khr<m_size = 16, n_size = 16, k_size = 16, a_type = f16, b_type = f16, c_type = f32, result_type = f32, acc_sat = false, scope = <Subgroup>>]>>}>

Running the reproducer CLI input gives the dispatch numbers 5, 7, 22, [...] which are matmul ops:
https://storage.googleapis.com/shark-public/ean/unet_lc/module_forward_dispatch_7.mlir
https://storage.googleapis.com/shark-public/ean/unet_lc/module_forward_dispatch_22.mlir

Currently something that looks the same as this on anything tuned. I think with differing eval key .<number> though. (Also getting the the arith.maxf complaint on everything).
Probably irrelevant thing I noticed, is that I also get this on untuned SD 2.1base, if I set the wrong --max_length. So setting it to 77 rather than 64.

@monorimet just checking whether this is making progress?


Consumables cap not working

You're able to add as many  consumables as you want.
Max packs cap is not working. Probably due to me rewriting everything. It probably fell through the cracks.

This is in develop branch btw


Add mobile styles

Search area needs to go full-width, and be sticky (like a sticky nav bar) while the user scrolls.

This has been fixed with: https://github.com/gregrickaby/nextjs-reddit/commit/b9e44a345b79db341157c9226e93bc2ebcdc4b8b


Bug Report: 'lime' is not recognized as an internal or external command, operable program or batch file.

I'm on windows
I did everything the guide said for building and when i type lime build windows into command prompt is says "'lime' is not recognized as an internal or external command, operable program or batch file."


add SNO for AWS

Signed-off-by: Chris Ahl<EMAIL_ADDRESS>For issue https://github.com/open-cluster-management/backlog/issues/11928

There will need to be a separate PR for bare metal as these changes only cover AWS, Azure, GCP, VMware and OpenStack.


Implementation of the back-button logic in implemented screens

Acceptance criteria

[ ] In all of the currently implemented screens, The back button logic is implemented

See
https://www.figma.com/file/cvV48t0f7O2znT6QBxK0Zj/Tonomy-ID?node-id=3287%3A13928&t=rknQ0OJNwIxXfrf4-0
and
https://github.com/Tonomy-Foundation/Tonomy-ID/issues/316#issuecomment-1439757704

@MilicaSequeira we need an indication of weather you believe the default back button behaviour is not enough for each screeen

I dont understand what you mean.
On Wed, 1 Mar 2023, 17:06 Jack Tanner, @.***> wrote:

@MilicaSequeira https://github.com/MilicaSequeira we need an indication
of weather you believe the default back button behaviour is not enough for
each screeen
—
Reply to this email directly, view it on GitHub
https://github.com/Tonomy-Foundation/Tonomy-ID/issues/530#issuecomment-1450406024,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4UFGBGYVBRWXHR2CTIM4N3WZ5XYHANCNFSM6AAAAAAVLYZGJU
.
You are receiving this because you were mentioned.Message ID:
@.***>


Improve MySQL db query index usage

I found that there is a way to get rid of the filtering "txn_id < ?" which can cause perf for MySQL query.

Coverage increased (+0.8%) to 69.769% when pulling 968c747a239c3c2796df4c36f07fd593cdcb278a on longquanzheng:eventv2-sql into 6ab93242d30f2d41f3bb2b6470e94b318f21e6c7 on uber:master.

Assuming that there is a million stale records and page size is 100.
the pagination token:
	filter := &sqldb.HistoryNodeFilter{
		TreeID:    sqldb.MustParseUUID(request.TreeID),
		BranchID:  sqldb.MustParseUUID(request.BranchID),
		MinNodeID: &minNodeID,
		MaxNodeID: &maxNodeID,
		PageSize:  &request.PageSize,
		ShardID:   request.ShardID,
	}

will not change at all, meaning that the query events v2 API will end in an endless loop.
We can talk tomorrow

Talked to @longquanzheng, the min node id is actually changed by 1, so filter will be different


Combine Direct Volume Migration Client Pods enhancement document

Implementation is complete https://github.com/konveyor/mig-controller/pull/1077


Move Prometheus to Hub image

From Alibaba employee - Quai.io is banned in China, change to Hub reference.

Done and released.


filter bunch by geometry type  (#11)

add method: filter_by_geometry and enum: GeometryType

Closes #11

@martinfleis Please take a look, thanks!

@martinfleis please take an another look at your leisure.

done thanks @martinfleis

@martinfleis
sorry for all the clean up not done after moving from enum to list[str]
please take a quick look if things are okay, thanks!

Hey @sdp5,
when looking at it again, I realised that the best solution may be porting the whole filter method from xyzservices over here, so I did that. Sorry for not pointing you in this direction from the beginning... Bunch.filter is pretty robust and returns a correctly nested subset of Datasets rather than a flat dictionary.

At some point I need to refactor this generic logic from xyzservices and just use it here rather than reimplementing the same stuff in both places... no time...

Hey @sdp5,
when looking at it again, I realised that the best solution may be porting the whole filter method from xyzservices over here, so I did that. Sorry for not pointing you in this direction from the beginning... Bunch.filter is pretty robust and returns a correctly nested subset of Datasets rather than a flat dictionary.

No worries, totally agree @martinfleis learning for me!
if you can help/guide me with some pointers where I can contribute in geopandas space.

@sdp5 this would be a good issue to start with -> https://github.com/geopandas/geopandas/issues/2010

@sdp5 this would be a good issue to start with -> geopandas/geopandas#2010

thank you @martinfleis  ..


Add support for dense table

Just as material-ui dense table and bootstrap small table, Kindly support dense option to make tables more compact by cutting cell padding.

This looks like it could be done via a setTableProps option, similar to https://github.com/gregnb/mui-datatables/issues/871, if I have time, I'll see put a PR together tomorrow.

Just in case anyone else needs this, the example https://github.com/gregnb/mui-datatables/issues/871 mentioned above uses
      ...
      options: {
        setTableProps: () => {
          return {
            // material ui v4 only
            size: 'small',
          };
        },
       ...

any updates on this? MUI 5 is out so I take that this isn't compatible anymore?

any updates on this? MUI 5 is out so I take that this isn't compatible anymore?

It just works, all you need is trying ;)

Just in case anyone else needs this, the example #871 mentioned above uses
      ...
      options: {
        setTableProps: () => {
          return {
            // material ui v4 only
            size: 'small',
          };
        },
       ...

saved my day, thanks

Thanks @ltalirz.  I've updated your fix for MUI v5:
      ...
      options: {
        setTableProps: () => {
          return {
            // material ui v5 only
            size: 'small',
          };
        },
       ...

I am trying to use this for setting my table density on click of a free-action.  I am just calling the setTableProps() method in the click handler of the free action.  But no luck.  Are there css styles that may conflict with this option setting?


Only run prettier for JS files

Like this, we ensure to only pass modified JS files to prettier.

Codecov Report

Merging #1791 into canary will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           canary   #1791   +/-   ##
======================================
  Coverage    78.7%   78.7%
======================================
  Files           6       6
  Lines         263     263
======================================
  Hits          207     207
  Misses         56      56

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 655bce7...2ceebd0. Read the comment docs.


Documentation about the internal format.

Hello,
This project is good for dealing with large files manually, however it seems it’s not the best choice for code integration into other projects and/or language.
So does the exact changes in objects are documented outside the source code ?

Here are some internal docs:

Specification
Server API


Proposal: Become Biicode block

Hey guys, was just checking out Biicode and it seems to have many graphics related "blocks", wondering if BGFX && BX could be added to this awesome collection?
Edit:
It also has a nice bonus of that if you publish BGFX there you may get a free :shirt:
I'd post it myself, but that would mean I am taking away the t-shirt from you. :P

Nothing stops you from packaging it for whatever software dlivery system you happen to have. From what I've seen, biicode kind of requires a lot of effort to get anything working.

Feel free to add it there and the t-shirt. :) Once it's there I'll add info about it into README.


Bug: IntentResolution.data property not supported by addIntentListener callback

The IntentResolution type returned by raiseIntent is defined as follows:
interface IntentResolution {
  source: string;
  data?: object;
  version: string;
}

but the addIntentListener callback is defined as follows:
addIntentListener(intent: string, handler: (context: Context) => void): Listener;

which includes no way to return data back to the caller. It is therefore impossible to populate the data property with the 1.0 API, causing an API inconsistency.
There are three options:

Document and leave the inconsistency as is.
Remove the data property from the IntentResolution interface until it can be provided.
Amend the addIntentListener interface to include a way to pass data back to the caller.

my vote would be for #3 - Amend the addIntentListener interface to include a way to pass data back to the caller.

Close in favour of #201


Once world is loaded it's in memory until Minecraft restart.

Worlds are never removed from Client/Server WorldContext.

Fixed in cc0bb1aaf579275747cb414ca9dc89de50222575

Thanks. I haven't gone and updated any of these issues yet. You beat me to it. =)


Scaling issues with exhibitor

If I have 3 ZK nodes in exhibitor and I terminate one server, am I to expect exhibitor to maintain the ZK ensemble?  What appears to happen is when one server goes down all my other servers lose connectivity and takes 10 mins or so to correct itself.
Can I use AWS autoscaling groups then? Doesn't anyone have a need to scale up or down?
thanks

@stonefury by "terminate one server" do you mean terminate an AWS instance with a set of exhibitor/zookeeper processes running on it? If so, exhibitor does not have logic to launch new instances on the cluster's behalf. You will have to replace the lost instance through other mechanisms (e.g. using an ASG).

@qiangdavidliu thanks for the reply. I have 3 EC2 instances each running 1 exhibitor/zookeeper.
I use a autoscaling group to replace lost servers. If I manually terminate a server, a new one is launched.  But in the process my remaining ZK nodes stop communicating with each other, they show up as yellow. My load balancer which is checking each server on port 2181 also goes down, so I'm assuming this means I lose all zookeeper connectivity for about 10-15 mins.
Is that normal?

Exhibitor has several modes it can run in. If you enable Automatic Instance Management, Exhibitor will automatically re-configure the ZK ensemble when a new server comes up.
https://github.com/Netflix/exhibitor/wiki/Automatic-Instance-Management
On January 28, 2015 at 5:30:47 PM, bfleming<EMAIL_ADDRESS>wrote:
If I have 3 ZK nodes in exhibitor and I terminate one server, am I to expect exhibitor to maintain the ZK ensemble? What appears to happen is when one server goes down all my other servers lose connectivity and takes 10 mins or so to correct itself.
Can I use AWS autoscaling groups then? Doesn't anyone have a need to scale up or down?
thanks
—
Reply to this email directly or view it on GitHub.

Again, thanks for the reply, but just referring me to that page with a few sentences isn't that helpful, but I appreciate the reply.
My question is:  When a zk server is terminated, is it expected for all the remaining ZK instances to go offline????  Shouldn't one server going down not affect the others?

That's more of a question for the Zookeeper user group. It depends how many servers are in your cluster, etc.

Thank you, that is indeed the answer.  I was running 3 servers and one was set by default as an observer, and I lost the quorum.  Manually setting to 0 observers kept everything running.
Thanks for fielding my questions. I'm overloaded with information and hard to tell what parts control what.  Glad I got it sorted out! You're the best.


Gradle instruction readme.md fix

Hi there!
I love your library from the first sight. But it might be that there are users that won't be able to correctly install your app via gradle since the gradle compile statement is not correct in the readme.md file.
Instead of using
compile 'com.github.florent37:expectanim:v1.0.1'
the v is missing and it is stated to use
compile 'com.github.florent37:expectanim:1.0.1'
Keep on the great work!

where ?

I've made a pull request with to minor changes in the readme.md :)

:)


[LOC] "Windows こんにちは for Business" needs to be replaced with "Windows Hello for Business"

[ ] https://docs.microsoft.com/ja-jp/windows/security/identity-protection/hello-for-business/hello-manage-in-organization

ドキュメントの詳細
⚠ このセクションを編集しないでください。 docs.microsoft.com で必須です ➟ GitHub の問題のリンク。

ID: 789566d4-463a-4c64-bf89-fb963ac29f80
Version Independent ID: 02946732-bb29-d8e6-cedd-7f5e508f8798
Content: 組織での Windows Hello の管理 (Windows 10)
Content Source: windows/security/identity-protection/hello-for-business/hello-manage-in-organization.md
Product: w10
Technology: windows
GitHub Login: @DaniHalfin
Microsoft Alias: daniha

@officedocsbot assign @e0i
@officedocsbot assign @dongmicro
@officedocsbot add label localization

@dongmicro It looks like the localization software recognized a proper noun (Hello) as a common noun, resulting in a string where the literal back-translation of it would go like, "Windows Kon'nichiwa for Business".
Similar issues in this manner:
#3434 #3436 #3437 #3438 #3439 #3440 #3441 #3442 #3443 #3444 #3445 #3446 #3447 #3449 #3450
Would you kindly review?
Thanks.

[ ... ] It looks like the localization software interpreted a proper noun (Hello) as a common noun, resulting in a string where the literal back-translation of it would go like, "Windows Kon'nichiwa for Business".

Funny translation issue. "Windows Kon'nichiwa for Business" -- thank you for providing a quick chuckle break from the seriousness of troubleshooting. 👍

Changing now.  Thanks

This issue, along with all the referenced issues have all officially confirmed to be fixed.
@hyoshioka0128 Thank you for bringing these issues to attention.

@officedocsbot close


Publish a rewrite-bom Maven BOM

Just a BOM that lists all open source Rewrite jars known to be in a good state and in alignment with one another, so one dependency definition can pull them all.
We could do this for both releases and snapshots, where the mechanism to release a snapshot BOM could run on a schedule (e.g. daily) and use the Versions Maven Plugin to lock snapshot versions in advance of publishing.

This is a great idea
At the moment, I have three separate properties for each dependency that I am using :

        <rewrite-maven-plugin.version>4.21.0</rewrite-maven-plugin.version>
        <rewrite-logging-frameworks.version>1.4.0</rewrite-logging-frameworks.version>
        <rewrite-testing-frameworks.version>1.19.0</rewrite-testing-frameworks.version>
        <rewrite-migrate-java.version>1.3.0</rewrite-migrate-java.version>

And it is difficult to know if they're compatible
Additionally, I am also receiving updates for them individually (dependabot)

@jkschneider, so you're thinking a composite module with all rewrite modules being API dependencies?
Given this allows version alignment of all core rewrite modules and not all modules are required, is having a rewrite-all for the purpose of those whom want to have all languages just included automatically?

@shanman190 We'd want a separate rewrite-all from this repo that also includes all recipe jars.

This is now completed and awaiting releases.
api(platform("org.openrewrite.recipe:rewrite-recipe-bom:latest.release"))

<dependencyManagement>
  <dependencies>
    <dependency>
      <groupId>org.openrewrite.recipe</groupId>
      <artifactId>rewrite-recipe-bom</artifactId>
      <version>latest.release</version>
    </dependency>
  </dependencies>
</dependencyManagement>

For full transparency as well, there is a rewrite-bom which provides version alignment of core rewrite modules. This is specifically helpful for consumption within the various recipe jars (including custom recipe jars) AND build plugins.
Gradle
api(platform("org.openrewrite:rewrite-bom:latest.release"))

Maven
<dependencyManagement>
  <dependencies>
    <dependency>
      <groupId>org.openrewrite</groupId>
      <artifactId>rewrite-bom</artifactId>
      <version>latest.release</version>
    </dependency>
  </dependencies>
</dependencyManagement>

For full transparency as well, there is a rewrite-bom which provides version alignment of core rewrite modules. This is specifically helpful for consumption within the various recipe jars (including custom recipe jars) AND build plugins.
Gradle
api(platform("org.openrewrite:rewrite-bom:latest.release"))

Maven
<dependencyManagement>
  <dependencies>
    <dependency>
      <groupId>org.openrewrite</groupId>
      <artifactId>rewrite-bom</artifactId>
      <version>latest.release</version>
    </dependency>
  </dependencies>
</dependencyManagement>

For my own understanding, what will be the value for latest.release
A separate release for rewrite-bom or the version of rewrite ?

@yiekel, for your particular above use case, you'll be adding the org.openrewrite.recipe:rewrite-recipe-bom to your dependency management. It imports the rewrite-bom as well as provides dependency management for all current and future rewrite recipe jars, so covers the full use case that you had from a version alignment standpoint.
I believe soon the release of rewrite-recipe-bom will be available (I don't control the versions themselves, but they'll follow a standard release model and show up in maven central as they are completed), but with each update of rewrite proper, the update of a recipe jar, or the addition of a new recipe jar will result in a new version of the rewrite-recipe-bom for you to consume with everything aligned. The openrewrite team will be able to provide more information as to what they are going to select for the release version of this new BOM.

@yiekel, for your particular above use case, you'll be adding the org.openrewrite.recipe:rewrite-recipe-bom to your dependency management. It imports the rewrite-bom as well as provides dependency management for all current and future rewrite recipe jars, so covers the full use case that you had from a version alignment standpoint.
I believe soon the release of rewrite-recipe-bom will be available (I don't control the versions themselves, but they'll follow a standard release model and show up in maven central as they are completed), but with each update of rewrite proper, the update of a recipe jar, or the addition of a new recipe jar will result in a new version of the rewrite-recipe-bom for you to consume with everything aligned. The openrewrite team will be able to provide more information as to what they are going to select for the release version of this new BOM.
To answer your second question, rewrite-bom (the core one) will have the same version as other core modules (they're all published at the same time). rewrite-recipes-bom will have a separate version number that aligns everything (as an example [as of this writing]: rewrite-bom:7.22.0, rewrite-spring:4.19.3, rewrite-testing-frameworks:1.20.2, etc)

Thank you for taking the time to explain. It makes sense

Hi there, I tried this with latest release, so I have
<properties>
<openrewrite.recipe-bom.version>1.0.0</openrewrite.recipe-bom.version>
</properties>

 <dependencyManagement>
 <dependency>
<groupId>org.openrewrite.recipe</groupId>
<artifactId>rewrite-recipe-bom</artifactId>
 <version>${openrewrite.recipe-bom.version}</version>
<type>pom</type>
<scope>import</scope>
</dependency>
<dependencyManagement>

What version should I be using while declaring the plugins?

<plugin>
<groupId>org.openrewrite.maven</groupId>
<artifactId>rewrite-maven-plugin</artifactId>
<version>?</version>
<configuration>
</configuration>
<dependencies>
 <dependency>
<groupId>org.openrewrite.recipe</groupId>
<artifactId>rewrite-logging-frameworks</artifactId>
<version>?</version>
</dependency>
<dependency>
<groupId>org.openrewrite.recipe</groupId>
<artifactId>rewrite-testing-frameworks</artifactId>
<version>?</version>
</dependency>
<dependency>
<groupId>org.openrewrite.recipe</groupId>
<artifactId>rewrite-migrate-java</artifactId>
<version>?</version>
</dependency>
</dependencies>
</plugin>

If I use openrewrite.recipe-bom.version as the version then it downloads the wrong versions(for example 1.0 for rewrite-logging-frameworks)  and If I do not declare a version, it fails to compile with

 'build.plugins.plugin[org.openrewrite.maven:rewrite-maven-plugin].dependencies.dependency.version' for org.openrewrite.recipe:rewrite-logging-frameworks:jar is missing.

I think that my issue is that I had to remove it from the plugins and just add them as normal dependencies

I think the idea here is that you should not have to supply versions for the plugins OR dependencies, but we may have some more work to do before this works properly.
I suspect the versioning may not apply to the plugins (and probably needs to be added in
I will leave this issue open and move it to the next milestone.

I think the idea here is that you should not have to supply versions for the plugins OR dependencies, but we may have some more work to do before this works properly.
I suspect the versioning may not apply to the plugins (and probably needs to be added in pluginManagement
I will leave this issue open and move it to the next milestone.

I was able to use it just declaring the plugin as dependencies with no versions and that seems to be doing the job

So I personally don't use maven often, but apparently Maven doesn't support BOMs for plugins even though it's a heavily requested feature...
https://issues.apache.org/jira/browse/MNG-5588
@tkvangorder, you are correct. The intent for rewrite-bom is to ensure consistent versioning for all projects built from this repository. The various recipe jars and build tool plugins are going to get updated to use rewrite-bom to ensure version alignment for themselves in terms of the core rewrite projects that they consume. rewrite-recipe-bom imports rewrite-bom and recommends the versions for all recipe jars, so that everything is consistently controlled and aligned correctly in terms of what end users will import and use.

We have been published rewrtie-bom and rewrite-recipe-bom for a while now, closing defunct issue


vim mode can sometimes trigger snippet

First, thank you very much for developing this -- this really does make working with LaTeX in Obsidian much nicer.
I wanted to report an issue that I'm having with snippets in vim-mode. Because moving up a line uses k, if you move up a line where the character before is a k, this triggers the snippet.
As an example, suppose I am in command mode and my cursor is on the a below.
math is fun
 ^

If I move up a line with k, this goes to
$$ath is fun

Unfortunately, I'm relatively new to obsidian, so not sure how the plugins work, but this might be as simple as disabling snippets when in command mode when using vim key bindings.
Again, thanks for a very useful plugin.

Glad you find the plugin useful!
I'll try to fix this soon by only executing snippets while in insert mode.

Thanks for reporting the issue. It should now be fixed in 1.1.2.


Create 0000-01-02-Sharayu1027.main.md

This is my second branch and second request pulling.

Merging my second pull request


demo

hello!
I am trying to test the trained model on a custom picture, and I have some problems.
First of all, this is the input:

After I run the demo.sh file, I have the following picture:

It seems, the network doesn't work on this data, does it?
What could be the problem?

You might want to check if the pretrained model is loaded correctly. Otherwise, it might happen.


Support for: TilingZoeDepth

https://github.com/BillFSmith/TilingZoeDepth
Pleeeeeease. :)

Hmm it produces 16bit depth map while Comfy nodes are made to process 8bit...

They look similar to me so I assume there is no issue with using 8-bit depth map
Source image:

8-bit depth:

16-bit depth:


Java implementation

Hey - I have a java implementation of the multicodec in my github. I'd be happy to link it to the list of implementations in the readme.md if that would be of help for the multiformat community.  https://github.com/richardbergquist/java-multicodec
I've attempted to submit a PR from a branch, but need permissions to this repo do so. What the process of getting permissions to submit a PR?
Thanks - @richardbergquist

@richardbergquist just fork the repo, push your branch to your fork, then pull request from there - it'll prompt you when you push to your fork so it should be straightfoward from there.

Ok, all good - understood now. Have created a very small PR. Thanks. Will close.


Template functions with annotations / ng-strict-di

As per my GitHub comment in this angular.js issue, I'm running into issues with template functions with annotating or ng-strict-di.
I know this issue may belong in the main angular.js repo, but I just wanted to ask, in case someone here had an idea how to get around this.

Not an immediate fix, will look into this, but doing this works using $inject: https://jsfiddle.net/toddmotto/vy4ayzb8/

Just pushed a fix for this, see https://github.com/toddmotto/angular-component/commit/24264c0154121980ffe811be516377b27970b762
Demo for annotated code: https://jsfiddle.net/toddmotto/67vmx7p5

Thanks @toddmotto! Works great. :+1:

@karlhorky Sweet! I've published 0.0.5 which includes the new automatic annotations, for NPM as well. Closing, over and out.

@karlhorky FYI, I've just published 0.0.6, which aligns with further component() changes, small Object merge rather than if statements: https://github.com/toddmotto/angular-component/commit/1229b256524cb840023db96e1c086af956c33f57

:100: thanks, sounds good


Wrong capitalization for volume types

This is a Bug Report

Problem:
https://kubernetes.io/docs/concepts/storage/persistent-volumes/#mount-options lists some volume types. The capitalization is wrong.
Proposed Solution:
Update the capitalization of the volume types listed in https://kubernetes.io/docs/concepts/storage/persistent-volumes/#mount-options to match the capitalization used in https://kubernetes.io/docs/concepts/storage/persistent-volumes/#types-of-persistent-volumes
Page to Update:
https://kubernetes.io/docs/concepts/storage/persistent-volumes/

Additional Information:
/sig storage

/help
/language en

I've raised a PR for this.
/assign

The PR is merged so this can be resolved I guess.

Hello. i would like to contribute. I am a newcomer but i know some html code in order to correct the capitalization you ask for.

/close
Fixed in #31542


Difference between two examples

Hello,
What's the difference between the "DMXUSB_Simple" and the "DMXUSB_DmxSimple" examples?
Thank you in advance :)
PS: I'm looking for an emulator USB to DMX to be used with freestyler

Hi there! I'm happy to help!
Take a look at the last line of the callback function in each example. In DMXUSB_Simple, the callback dim am LED on the first universe ane first channel:
https://github.com/DaAwesomeP/dmxusb/blob/70d4ae3dfafdc91a9b7d6e4a9c4b730d281e0667/examples/DMXUSB_Simple/DMXUSB_Simple.ino#L37
In DMXUSB_DmxSimple, the callback sends thr entire first universe into the DmxSimple library, which can output a real DMX signal (with the right hardware):
https://github.com/DaAwesomeP/dmxusb/blob/70d4ae3dfafdc91a9b7d6e4a9c4b730d281e0667/examples/DMXUSB_DmxSimple/DMXUSB_DmxSimple.ino#L52
Hopefully that clears that up! Let me know if you have any more questions
I'm not familiar with FreeStyler, but it looks like a DMX controller program similar to QLC+. Assuming it can see an arbitrary Enttec device on any serial port (which this emulates), then it should work. If not, then you may need an FTDI adapter or to use OLA as an adapter.
Wbat is your intended output? Are you connecting real DMX fixtures or your own electronics?

@paoloros97 please let me know if I can be of any further assistance!

Closing due to no response.

Wbat is your intended output?

Can i use this firmware to use an arduino leonardo with a SN75176BP chip to control real dmx fictures through usb?

Hey @tonyxforce,
Yes, the DMXUSB_DmxSimple example is probably where you want to start. I would recommend looking at the DmxSimple examples first and get it working before you add in DMXUSB.


Prevent variable change on form submit

Support Request
Hi @bencroker,
I am building an addresses book for a craft commerce website.
I am using the same form include with as variable, the id of the address to edit.
My issue is that after submitting the form, the variable is reset.
How could I prevent that? I have tried with the hidden input field, but the result is not that nice.
Plugin Version
3.5.0

I’m not sure I’m following. Can you show me an example of what you mean?

Sorry for the non clear explanation
I have the following
`{% set addressToEditId = addressToEditId ?? '' %}
{% if currentUser.addresses|length %}
{{ include('_shop/_private/address/list', {
addresses: currentUser.addresses,
primaryBillingAddressId: currentUser.primaryBillingAddressId,
primaryShippingAddressId: currentUser.primaryShippingAddressId,
}) }}
{% else %}
{{ 'You don’t have any addresses yet.'|t }}
{% endif %}
{{ include('_shop/_private/address/address-form.twig', {
address: null
}) }}`

and in the list.twig, I have a for loop:
`{% for address in addresses %}
{% set isFormShow = 'false' %}
  {% tag 'div' with {
    class: [
      'customer-block',
      addressToEditId == address.id ? 'visually-hidden'
    ],
    id: "customer-block-address#{address.id}"
  } %}

      {{ address.title }}

      {{ "#{address.firstname} #{address.lastname}" }}

    {{ address | address | attr({class: 'small'}) }}
    {% if primaryBillingAddressId == address.id or primaryShippingAddressId == address.id %}

        {% if primaryBillingAddressId == address.id %}

{{ 'Primary billing address'|t }}

        {% endif %}
        {% if primaryShippingAddressId == address.id %}

{{ 'Primary shipping address'|t }}

        {% endif %}

    {% endif %}

      {{ tag('button', {
        class: ['action', 'action-customer', 'action-edit-address'],
        type: 'button',
        data: {
          anchor: "address#{address.id}",
          sprig: true,
          's-vals': {addressToEditId: address.id}|json_encode
        },
        html: m_icon.params('edit') ~ tag('span', {class: 'visually-hidden', text: 'Edit the address' | t})
      }) }}

  {% endtag %}

  {{ include('_shop/_private/address/address-form.twig', {
    address: address
  }) }}
{% endfor %}`

When I click on the edit button of one block, I set the addressToEditId variable with the value of the address' id.
It displays the form with the correct data.
But once I submit the form, the addressToEditId variable is reset.

You can use a hidden input field or s-val to maintain the variable state in your form.
{{ hiddenInput('addressToEditId', addressToEditId) }}

OR
<form sprig s-val:address-to-edit-id="{{ addressToEditId }}">

Thanks Ben!
I managed with not nice UX by extracting the flash messages from the form.
I am also using the same form to create new addresses, where the address id is null.
I am not sure that it is possible to maintain the state in this case.

It should be possible to maintain state, I can’t see why it wouldn’t be.

@bencroker
I found what was missing, the instantiation of each input variables like
{% set lastname = lastname ?? '' %}
like there https://putyourlightson.com/sprig/cookbook#contact-form
I was wondering how to handle "fields[firstname]" for special addresses' fields, but it should be doable with hiddenInput, like
 {{ hiddenInput('fields[firstname]', firstname) }}


2 Questions Only

Does hikari takes care when cpu {About machine} max out to 100% ?
In-relation to question 1, If no then will it issue a log that there is a connection leak?

thanks,
Dave

I don't understand question #1.  If a connection is out of the pool for longer than the leak detection threshold it will be logged, regardless of why it was out of the pool.  If the CPU is pegged at 100% the timer thread that logs the leak might itself be starved, and therefore be delayed from reporting the leak.

Ah i see, thanks for your time.


4.0 Release Merging Development into Master (Python)

Please note this PR does NOT contain the A2-THOR 2.5 merge.

@deanwetherby Done. (There are a lot of files containing the version number we need to update. I added a list to DEV for reference, but maybe there's a way to consolidate them all in the future.)


Remove dofmax parameter in contrast

This seems parameters does not seem useful and could hide some issues in estimation procedures

Solved. Closing.


Remove publishArtifact := false

So asked around in the Scala discord communities sbt chat to get help on this and they are quite sure that
Compile / packageDoc / publishArtifact := false

Is whats causing the isuse. Even if you don't have any apidocs, having this setting for published artifacts in sonatype causes problems. Other projects such also don't have this setting (see https://github.com/lightbend/sbt-paradox-apidoc/blob/master/build.sbt)
References https://github.com/lightbend/sbt-paradox-project-info/issues/31

Fingers crossed that this fixes the problem


ustringhash

ustring is a unique string (that lives in an internal hash table) whose
representation is the pointer to the characters that live in the table.
ustring hash is its alter ego -- it also refers to a ustring in the
internal hash table, but the representation is just the hash itself.
It presents an interface that is a subset of that of ustring, and in
many ways they can be used interchangeably.  For ustringhash,
accessing the characters requires using the hash to find the string in
the table -- which is not hard, but more expensive than accessing the
charcters with a ustring, where the representation is the pointer
directly to the characters.
For some uses where you don't need access to the characters in any
performance-critical paths, this may be a more convenient
representation.  In particular, it's well suited to a GPU that doesn't
have access to the character memory. Another interesting difference is
that from run to run, a ustring may have a different literal value,
since there's no reason to expect that the pointer to a string like
"foo" will refer to the same memory location every time the program
executes, but in contrast, the hash is guaranteed to be identical
from run to run.
In the process of adding unit tests for ustringhash, I ended up
overhauling all of ustring_test.cpp, as I saw that the ustring tests
were extremely cursory. Did a tiny bit of (non-breaking) touch-ups
on a few ustring methods as well.

@cmstein Do you think this will help you?

I have use for this; merging.


Rewrite unit tests using pytest

Unit tests were initially written using the standard library unittest module. It would be better to use pytest as it has support for parameterized tests, which would make the test code cleaner.

This has been completed with commit 9bd7f2e5ffa6667f1f01837f832db16d92642edd.


Loading huggingface textual inversions

🤗/sd-concepts-library has prettained textual inversions: https://huggingface.co/sd-concepts-library
would be nice to be able to easily load and use these
cc https://github.com/AUTOMATIC1111/stable-diffusion-webui/issues/151

The ability to read hugging face .bin files was just added today. You'll find it in the development branch, and documented in docs/features/TEXTUAL_INVERSION.md

Thank you!
https://github.com/lstein/stable-diffusion/blob/development/docs/features/TEXTUAL_INVERSION.md

Looks like training doesn't yet work on M1. Has anyone started looking into this alread?

I confirmed that this does work.
For example, download https://huggingface.co/sd-concepts-library/ugly-sonic/blob/main/learned_embeds.bin and save as uglysonic.bin
Then run scripts/dream.py --embedding_path uglysonic.bin and use "a photo of <ugly-sonic>" and you will get a photo:


How did you use the png map file to do overlay on Google Map?

Hi,
I am having difficulty trying to understand how you overlay you png file on Google Map. What did you use to obtain the values for the coordinates? As you know the map was in meters to be able to use UAV footprint.

I did not overlay the map in GoogleMaps. It is a print screen from that website.

thanks


Genie clients now support constructors with or without security

This PR changes the way genie-client classes were being constructed. Instead of taking in a configuration object and using it to instantiate security interceptors, the responsibility of creating the security interceptors now rely with the client users. This helps users plug in their own security interceptor implementation. There is a default implementation provided for OAuth authetication.

Coverage increased (+0.6%) to 82.065% when pulling 8273e5a3717b13c0bdf54d91e4407a0a7e2568dd on amitsharma1981:develop into 8f369ada128bbe35d5034889b687a67375453bc4 on Netflix:develop.


⚠ Bofrost.de has degraded performance

In a45fa08, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 842 ms

Resolved: Bofrost.de performance has improved in 5bcdf9b after 1 minute.


CI is failing.

See https://app.circleci.com/pipelines/github/bullet-train-co/bullet_train-super_scaffolding/263/workflows/e31434e4-7e2c-47b3-89c2-8dc5490694bf/jobs/1109 .

@andrewculver I kind of did this on a random PR so I apologize for that, but I got the tests to pass here by using the latest GitHub branch for bullet_train-incoming_webhooks: https://github.com/bullet-train-co/bullet_train-super_scaffolding/pull/100#issuecomment-1342146390
I think the gem just needs to be released, and then it'll require the proper files (from this PR) so we don't get any uninitialized constant errors.


Enforce promise parameters

This new module allows enforcing naming of promise parameters. We don't use a lot of promises (yet), but this seems sensible to me. Should we add this rule?

I like it, seems like good practice.

After looking into this, I'm fine with it.


Pipenv run is not finding binary in PATH (despite it being there)

Issue description
I tried to invoke pipenv run pyo3-pack which is installed on my system and retrievable by means of where and which -a, hence in my PATH variable etc.
Expected result
My global pyo3-pack installation is called
Actual result
pipenv run pyo3-pack develop
Error: the command pyo3-pack could not be found within PATH or Pipfile's [scripts].

Steps to replicate
pip3 install pyo3-pack
Try to make install  the hyperjson package: https://github.com/mre/hyperjson
Additional notes
I found out that the system_which implementation used in do_run_posix is buggy.
Switched it out with shutil.which which solved my problem.
core.py line 2271+:
 do_run_posix(script, command):
    command_path = shutil.which(script.command)
    if not command_path:

This might be fixed, see https://github.com/pypa/pipenv/blob/main/pipenv/core.py#L1567-1571


#438: Better spacing for javadoc code examples

I tried to follow https://github.com/joel-costigliola/assertj-guava/commit/841ee1b60df12a59b5e7528901a07e9d18c0f618
Once we are satisfied with this change I will create another pull request for the 3.0 classes.

Thanks a lot !
Don't create a PR for 3.x, 3.x is really just a branch containing java 8 specific stuff, I usually report issues in 2.x to 3.x by merging master (= latest 2.x) in java-8 branch (latest 3.x).

Thanks for the info. :)
I will let you merge to the java-8 branch, but I guess there could be some java 8 specific classes e.g. AbstractLocalDateAssert which might need similar changes?

You are correct indeed, e.g. AbstractOptionalAssert, if you want to address it, just create another PR specific to those classes.

Integrated, many thanks !