more indexes on system_requests

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Map printing on external button

I've seen current plugins, but nobody can get a client function for map printing
Any ideas ?

Hi, great to hear that you find Leaflet useful!
However, this issue tracker is used for reporting bugs and discussing new features. For questions on using Leaflet, please use gis.stackexchange.com or stackoverflow.
If you are really sure that this is a bug in leaflet, or a feature request, please change the wording to make it look like a bug report.


could not find driver

Hey,
I'm using your plugin for backup and using PostgresSQL with Laragon on Windows and latest filament I get error of "could not find driver" when I open Backups page.

Same here just on ubuntu, php8 and mysql.

Ok, found and solved it. This package uses the https://github.com/calebporzio/sushi packages under the hood, which itself requires sqllite. The error message is not really helpfull, but it dies because of the missing sqllite driver. So just install it on your maschine/server:
sudo apt install php8.0-sqlite

And afterwards restart your http server (apache2/nginx)..

@bernhardh Thanks its fixed after enabling pdo_sqlite and sqlite3 extensions from laragon.

Just for other people who might end up here, in case sudo apt install php8.3-sqlite didn't work for you, try this:
sudo apt install php8.3-sqlite3


ISSUE - 'in ' requires string as left operand, not Tag

I use the switches: Python KMPDownloader.py -w --date -d E:\KMP-Kemono-Downloader-<IP_ADDRESS>-pre\logs
I try using scanning this user: https://kemono.party/patreon/user/53820
And after a certain amount of downloads, this occurs in CMD:
Exception in thread Thread-102:
Traceback (most recent call last):
File "E:\Python_3.11.3\Lib\threading.py", line 1038, in _bootstrap_inner
self.run()
File "E:\KMP-Kemono-Downloader-<IP_ADDRESS>-pre\Threadpool.py", line 256, in run
todo0
File "E:\KMP-Kemono-Downloader-<IP_ADDRESS>-pre\KMPDownloader.py", line 1007, in __process_container
if not prev or (prev and container.contents[0] not in prev):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: 'in ' requires string as left operand, not Tag
Any idea what it means, or how to fix it? Thanks.

I'll take a look at the issue later today. From a glance, it seems like an issue resulting from the inconsistent structure from the website. If you would like to download without post content, use the "--omitcontent" switch. It should work after applying that switch. What will be omitted will the stuff labeled "Content" as seen below":

Any attachments such as rar files will still be downloaded so you can just use the --omitcontent switch for now while waiting on the fix if you don't care too much about not having "Content" data being downloaded.

Extra information:
At first I thought this might be a different problem and was going to open another issue, but after some testing, I think this might be related to this original post.
User = https://kemono.party/patreon/user/797556
Problem = The downloader SOMETIMES is unable to skip broken links, and enters a state of infinite loading, in which the time-remaining-counter goes up slowly forever.
Examples of skippable broken links =
https://kemono.party/patreon/user/797556/post/13755803
https://kemono.party/patreon/user/797556/post/13403479
https://kemono.party/patreon/user/797556/post/13403248
https://kemono.party/patreon/user/797556/post/13403140
https://kemono.party/patreon/user/797556/post/12493741
Examples of unskippable broken links that stop the downloading entirely =
https://kemono.party/patreon/user/797556/post/11558120
Wish you goods, friend~

I think the issue is fixed in #25, you can go to the pull request and click on <IP_ADDRESS>.1 then download it as a zip. I've been busy with work and more so I haven't had the time to properly test it but it does work now for the issues you mentioned here.

Fixed in releases <IP_ADDRESS>.1 and <IP_ADDRESS>


fix(ios) : change webView source prop that bare html to uri path

related : #7 User permission check has failed
I have found that when bare HTML is embedded inside a WebView source, the navigator's mediaDevices property is unavailable.
This appears to be an issue related to cross-browser compatibility. I have observed that changing the source to a URI path resolves the issue and allows the functionality to work correctly.

i will change something and pull request again thank you


Not required rule doesnot work

Bug Description:
Is there not a required rule?
Im tryin this rule:
params = {'otherParam': 'ABC'}
valid = Validator(params, {
        'mark': 'integer|min:1|max:3'
    })

The param mark is not required, so it can be None. But I get this errors:
{'mark': {'Integer': "Expected Type of Int or Str, Got: <class 'NoneType'>", 'Min': "Could not get size from data. type = <class 'NoneType'>", 'Max': "Could not get size from data. type = <class 'NoneType'>"}}

Currently we don't support not required rule, but we are going to implement 'nullable' rule, so it can handle cases like this.

Currently we don't support not required rule, but we are going to implement 'nullable' rule, so it can handle cases like this.

That would be awesome. Looking forward to it.

Thanks, for some time we stopped development of package, but we will be on the rails soon, if you would like to contribute I'd accept PR for this issue


Expose similarity score functions as part of custom scoring

Expose functions that evaluate the similarity score like Euclidean, cosine etc so that users can perform their own scoring.

https://discuss.opendistrocommunity.dev/t/combining-knn-score-with-keyword-query/3600/3

This will be a pretty useful feature for mixing knn and keyword search results. Is there an ETA for this feature

Any updates on if this will make it in to the next release?

Hi @timforr @VijayanB is working on the PR and we should be ready for odfe1.13 release.
As a side note we already have custom scoring exposed from odfe 1.11  https://opendistro.github.io/for-elasticsearch-docs/docs/knn/#custom-scoring


Create a Profile Model

Every user would require his own Profile. Create a ProfileModel in users/models.py
Some fields he would require include

date of birth
description
profile_image
Don't forget to add a OneToOneField to users

Do add more cool fields that you think might be important!

i will do it


Utility program probe_3d does not property provide warning when q is set too large.

compressor_3d does it.

It's improved in compressor_3d and probe_3d is no longer there.


Update Visibility docs on maintainSize

Description
To make the documentation clearer, I added a statement about maintainState
Related Issues
Fixes #53073
Tests
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read [Handling breaking changes].

[x] No, no existing tests failed, so this is not a breaking change.

This doesn't completely fixes the issue I opened, it just adds one of the recommendations I gave.
There's a also the problem with:
  /// Maintaining the size when the widget is not [visible] is not notably more
  /// expensive than just keeping animations running without maintaining the
  /// size, and may in some circumstances be slightly cheaper if the subtree is
  /// simple and the [visible] property is frequently toggled, since it avoids
  /// triggering a layout change when the [visible] property is toggled. If the
  /// [child] subtree is not trivial then it is significantly cheaper to not
  /// even keep the state (see [maintainState]).

Which goes in depth to explain the costs of maintaining the size, animations and state, but we don't actually have an option as if we select to mantainSize we are obligated to also add maintainAnimations and mantainState.
That's the part that I think needs clarification. It looks like something that was written a while ago, but the Framework evolved and added constraints that the docs don't fit anymore.


Weird "invalid default" error for a constant

Repro here

This happens because the clear command also clears constants in define. defines are desugared to functions from 0-length vectors of values to their output, so any constant will just have whatever "default" value for its sort made up when it's used after a clear. There is no such default for integers, hence the error. I agree this behavior seems wrong, but I'm not sure what we should do instead.
One option here would be to have clear skip defines. But what happens if we have:
(datatype Expr
  (Lit i64)
  (Add Expr Expr))

(define x (Lit 0))
(define y (Lit 1))
(define z (Add x y))

Clearing Expr will remove the mapping for any ids in x, y, and z.
Seems like what we want to do instead is replay any defines after a clear. That won't always work, e.g.
(function SomeInt i64 i64)
(set (SomeInt 0) 1)
(define x (SomeInt 0))

Maybe we should ~always be using push and pop rather than clear?


Improved Deck Builder UI - better visibility of meta content [mockup inside]

The current Deck Builder is fine, except:

The description and metadata are hidden behind hard to find modal dialog and only exposed when saving a deck, and some people don't save decks because they are auto-saved anyways.
The new compact menu actually makes it harder to find where to go to describe your deck

Suggestion:
Remove the keywords field (it's not very useful when building a deck, it's more useful on individual deck pages where you are not aware of the deck and want to study its synergies) and instead use the space for exposing important metadata fields.
Mockup below:

Describe Deck button will open a modal dialog as description has be written in large enough writing area, not compact one-liners that we can fit in this space.

Done


way to opt out of reference for certain packages

Hello,
here is my use case: I have two types of packages that I put in two folders, namely foo and bar, that each contain many packages. The foo packages do not contain any shared code and will never be referred to by other packages, though the foo packages will often refer to the packages within bar, where shared libs exist. Is there a way to avoid packages from foo being added as references in the base tsconfig?

I think I'm just confused, closing for now.


Create Code of Conduct

In order to make BAF a welcoming community, where expectations are clear, we need a code of conduct document.  This is largely expected to borrow from / reference the Hyperledger code of conduct, but should still have a reference within this repository.

Closed with #88 and #89


Avoid nested transaction in eda-server with new util

Speaking practically, I know for a fact that org/team modifications will hit this line, and the entire point of the ensure_transaction is to avoid further nesting unnecessary transactions. So to make this make sense, it needs to be applied here too, and it's correct. The code isn't properly generalized without it.

cherry picked into the branch


Fixed #25984 -- Corrected error message in ModelBase.new

Changed the error message in RuntimeError Exception class

merged in 692d05589061a593d4bfbdb2895eaef0386eca34, thanks.


python310Packages.sonos-websocket: init at 0.1.0

Description of changes
Library to communicate with Sonos devices over websockets"
https://github.com/jjlawren/sonos-websocke

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 228789 run on x86_64-linux 1

4 packages built:

python310Packages.sonos-websocket
python310Packages.sonos-websocket.dist
python311Packages.sonos-websocket
python311Packages.sonos-websocket.dist


fix: updated deps

:tada: This PR is included in version 1.13.149 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: tabindex inputs not supporting coercion

All the components that were implementing the HasTabIndex input weren't allowing coercion.
Fixes #20026.

When I look at the corresponding issue for this PR, it seems to me that the common pattern for static tabindex values should be to set the tabindex attribute (instead of the tabIndex property). I think we did that in a couple of components using @Attribute, but probably not consistent and the use of a basic input (through the mixin) is easier to maintain/consume by users.


MRG: Organise NIRS specific preprocessing

What does this implement/fix?
Address issues raised  by @agramfort in #7035

@agramfort please let me know if I missed anything.

@rob-luke can you please open an ENH: Agenda for fNIRS processing issue or so? There are things I'd like to put in / discuss there. For example whether or not TDDR is something worth adding. Let's discuss that over in the new issue, though, so we don't hijack this.
(I would create the issue myself, but if you create it then we'll both have edit permissions on the top comment.)

@larsoner & @agramfort this is ready for review again. Thanks

Pushed license+authors, thanks @rob-luke !


added mocked tests for databases - part 1

Refer Issue: #204
Added additional mocked tests for databases.create_cluster, databases.create_replica

Thank you so much for your contribution! This has already been addressed in #215 so no need to merge.


Planning for Susy v3

Philosophy
The goal of Susy v3 is to separate the math layer from the css-output layer. At the core, Susy is a grid calculator that can be used for any CSS layout technique. That simplicity and flexibility is too easily overshadowed by attempts to provide every common shortcut. By reducing the core to functions, Susy can truly shine as a grid-system system, and the shortcuts can be provided in a more modular ecosystem of plugins designed to handle specific needs.
1. The Core
The core will focus on two functions and five global settings — only what is necessary for describing grid math, but nothing to do with actual CSS output. Developers can use the core to create their own grid-systems on the fly.
$susy: (
  'columns': 12,
  'gutters': 1/4,
  'column-width': null, // optional width for static math
  'math': 'fluid' or 'static',
  'spread': 'narrow' or 'wide' or 'wider',
);

.functions {
  width: span(1 of 4);
  margin-right: gutter(of 4);
}

2. Plugins
Plugins can be created to provide common shortcuts for different CSS layout techniques (float, isolation, flex-box, inline-block, table-cell, gutters before or after or inside, etc.) — as well as different pre-built systems (oocss, 960gs, etc).
@mixin susy-float ( // names are obviously not final...
  $span,
  $flow: ltr
) {
  float: if($flow == ltr, left, right);
  width: span($span);
  margin-right: gutter();
}

@mixin susy-flex (
  $span
) {
  flex: 1 0 span($span);
  padding-left: gutter()/2;
  padding-right: gutter()/2;
}

3. Tutorials
It will be important to provide lots of examples on how to create your own grid system using the Susy core engine.

Things to consider:

how do we make it flexible and easy for users who need 2 or more different output techniques (plugins) in their code? Do the plugins all play well together?
do the plugins live in a folder inside the repo, or in external repos? I think it will be most flexible to keep things separate - and hope that the community also starts creating plugins to share.

how do we make it flexible and easy for users who need 2 or more different output techniques (plugins) in their code? Do the plugins all play well together?

Plugins can be imported on demand so I don't think there will be any specific issue with people wanting different output techniques in the same code base.

do the plugins live in a folder inside the repo, or in external repos? I think it will be most flexible to keep things separate - and hope that the community also starts creating plugins to share.

In the same repository might be just easier and as convenient for code maintenance than several repositories.

what are the naming patterns for plugins?

It should be different from core functions and variables in order not to prevent any future enhancement. An idea would be to have susy-- instead of susy-. Not a fan but that could work.

ok, sounds like both Hugo and Zell voting for a single repo. Let's go with that for now, unless anyone has a strong argument the other way.

do the plugins live in a folder inside the repo, or in external repos?

I don't see why can't we have both, in Drupal we have core modules and contrib modules, core modules (or plugins) should be relatively basic and satisfy the most common use-cases. These core modules should also provide an example as to how a module should be constructed and written for contrib modules authors to explore and learn from.
One thing to keep in mind is that core and contrib plugins must be separated into different folders to allow easy update of core whenever it is available.

Sure, that makes sense. Plugins we maintain live inside, contributed plugins can live elsewhere - and we can link to them.

@tsi Because it's a lightweight grid system. Not a full-blown CMS powering millions of sites. :)

Hey y'all. Singularity already does a bunch of this, including offering a plugin system for different output styles (and input options) as well as having the core math layer. If we can agree on architecture and stuff, I'm happy to try the "merging Singularity and Susy" thing again.

Hey @Snugug, I'm open to the idea, but also don't see a strong need for it. I'm happy with people having various options in the Sass layout world. :) If you do want to get in on this effort, we're already a fair way into structuring the architecture. The Su grid engine is pretty stable/powerful, and we've only done minor cleanup on that front. I don't plan to abandon it unless there's a real compelling reason. From there I see the structure happening in a few additional layers:

Susy Core provides a slim API layer for the math engine, using only functions. The core does not care about CSS output, or different approaches to layout, or even a "system" for installing plugins. This should be as small and simple as possible. The entire goal is to push people towards using grid math themselves, rather than relying on pre-built mixins.
Plugins can be built on top of either Su or Susy, as needed — but Su and Susy are simply dependencies. No plugin can touch the core - only use it. Plugins can provide anything they want — new API's for the math, mixins for output CSS, etc. Where Su and Susy remain completely grid-agnostic, plugins can be as opinionated as you want.
Starter Templates are like plugins, but much smaller. Like the HTML5bp of Susy. Pretty basic sass templates that just give you good default settings for different grid types. Maybe they also include a mixin or two, showing how Su/Susy can be used to create a simple grid system?

I'm personally not that interested in maintaining any massive output plugins, but I'm very excited about teaching people to build their own systems, and helping any way I can. I mostly want to see people building their own per-project plugins, that do exactly what they want for the project at hand. Then sharing the code, so others can see what options exist.

Hey @ericam, will Susy 3 work with the Ruby version of Sass or LibSass? (or both)

Both. At this point they are nearly identical, and getting closer. There's no reason to drop support for either one.

Any further conversation can happen feature-by-feature.


fix(website): CSS of jest card

Summary
At some point, the elements on the back of the jest cards on the website's main page started to overlap on the front. When I first saw it, I didn't really care. Because it was a problem with the website's main page, I thought it would be fixed soon. But it seems to have been left in that state for several months, so I fix it myself and send this PR.
Test plan
Please check in the Chrome browser.
Before:

After:

Codecov Report

Merging #12127 (83425de) into main (3093c18) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #12127   +/-   ##
=======================================
  Coverage   67.71%   67.71%
=======================================
  Files         328      328
  Lines       16990    16990
  Branches     4817     4817
=======================================
  Hits        11505    11505
  Misses       5452     5452
  Partials       33       33

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 3093c18...83425de. Read the comment docs.


Extend CI/CD

Infrastructure as Code
GitHub Environments
PR Actions by comments

current solution does not trigger workflow on label

each PR should auto-get comment with available actions. Or use PR template that contains that


Multiple self-hosted fonts on Theme

What is the best way to customize the theme with multiple self-hosted fonts? I have followed the instructions from the documentation on how to override the global font with one self-hosted font and it works well. However, am not sure how to use multiple self-hosted fonts. Adding an array of two font objects on @font-face seems to do exactly what I want but my linter complains since @font-face expects an object.
I want to be able to use the Regular font on typography.fontFamily and the Bold on typography.h1.

Potentially a duplicate of #15771


Is this project still active?

Hi gentleman,
Is this project still active?

Hey, yes it is still active. I am currently working on this branch.

Is this project initialized by Microsoft?

Yes.

Could we trying to keep the syntax same as much as possible?
Like TensorFlow.NET.

Hi @interesaaat, sounds great that the project is still active 👍
Will your work eventually be pull requested back into the master branch in xamarin/TorchSharp/? So we can have a new release/nuget package?

Hey @mdabros, yes that's my plan. @Oceania2018, what do you mean? I believe we are following PyTorch API (or at least the c++ API) plus some C# notation.

@interesaaat I've raise a new dissucss issue #76


[sil] Teach the verifier that an upcast of an address isn't a consuming use.

Just a missing case.

@swift-ci smoke test


feat: Page Templates.

Okay! Been playing with https://github.com/vapid/vapid/issues/158 over in a private branch and I think I have it in a happy enough state to open as a draft here so we can keep playing with it and get a feel for the proposed changes 🙂
(Fun fact: once complete, this should be fully backward compatible.)
I've been using this in production for the last couple weeks and it feels very natural. Giving control to the users for new page creation and navigation management feels natural correct.
I think I'm going to hold off on "collections as a field" features that are crossed out at the end of the features list for another PR. They're technically independent and can be implemented as a follow up to this PR if it lands.
I'm going to spin up a demo site with this dashboard up and running for y'all to play with hands-on 👍

Wrapped up in this PR is:
Data Model Updates

[x] Rename sections table to templates
[x]  Add a type column in the templates table of type ENUM(setting, form, page, collection).
[x] Set all multiple=1 templates to type collection
[x] Set all form=1 templates to type form
[x] Set all other template types to setting
[x] Drop form and multiple columns from templates
[x] Rename the records.section_id column to records.template_id
[x] Generate type=page template entries for each discovered /[^_]*.html file.
[x] Generate a default page record instance for each template entry
[x] Add a new metadata field to records to cusomize page name and URL slug.

Template Parser Updates

[x] Use the Handlebars parser ASTs for improved template parsing, hash validation, template rewrites, partials support, and page rendering
[x] Discover all data paths that start with this. inside of page templates and add them to the page's template entry as fields.
[x] Discover all data paths with no context prefix and log a deprecation warning. Automatically insert general. as the path context to avoid this. conflicts.
[x] Discover all data paths with the _ pseudo-private prefix and log a deprecation warning. Automatically rewrite _ paths to @ for proper Handlebars data attribute rendering support.
[x] Add the {{#collection name as |value|}} block helper to the runtime and analyzer.
[x] Update the {{#section name multiple=true}} helper to analyze a type=collection
[x] Deprecate and the {{#section name multiple=true}}
[ ] Rewrite the {{#section name multiple=true}} to {{#collection name as |value|}} (Edit: this is hard and requires children path rewrites... may not be reasonably possible.)

Dashboard Updates

[x] Display Pages in the admin dashboard as single records.
[x] Allow users to create new Pages, selecting from the project Page templates as data structure options.
[x] Allow users to delete Pages (including the default created ones)
[x] Add UI for customizing page metadata.
[ ] ~Allow users to create new Collection instances, selecting from the available templates.~
[ ] ~Allow users to delete Collection instances – including the default Collection created.~
[ ] ~Add collection directive to the dashboard that will render on all pages that reference the collection – users may select from a list of collections that use the same data model.~
[ ] ~Add a filter, orderBy, and limit features to the collection directive. This way, users can customize how collections are rendered on the page. In-template values will be considered defaults.~

Note: Came up with an idea as I was working here: What if we allow {{#collection this.name as |value|}} and {{#section this.name}} (note: this.name instead of name). This would make a page-specific collection. It won't be shared between other pages and will show up as a direct child of the page in the dashboard. Essentially, each page instance will get a private collection or section to modify in-place.

Additional code cleanup / features:

Updated analyzer to user Handlebars AST.
Updated runtime to use Handlebars AST.
Use native @ data props for metadata values instead of pseudo-scoped _.
Added the @navigation data property for site nav management
Add full partials support.

I have a number of known regressions and "must-do"s that need to land before this can be considered code complete (hence the draft PR):

[ ] Collection sort and order-by is broken – needs to be added to the HBS runtime
[ ] Slug validation should be moved to a metadata field validator (like content) to ensure slugs are globally unique. Currently, it is implemented as a unique index in the Record.slug column, but this breaks the metadata JSON config independence, complicates the data model, and doesn't capture the complexity of actual record slug uniqueness. This needs a little remajigger.
[ ] Delivering @data properties deep into the data model needs a better construct. Currently using fake __data__ properties to pass them down.
[ ] Database migrations need a little massaging. They can be unified into one, need to ensure that existing pages / collections are configured correctly on migration, and they need a proper downgrade script.
[x] I believe there are issues with newly generated projects with this change – need to get that initial site working nicely.
[ ] Also.... tests 😅

Also, have this queued up which will help improve the dashboard experience. Currently dependent on this PR because I branched off of this one and there are a lot of file changes 😬 https://github.com/amiller-gh/vapid/pull/3

@amiller-gh Is this still a WIP? Let me know when you're ready for comments..
Re: amiller-gh#3, I might need to push back a little on the color scheme changes, since it affects the current branding.


Reduce dependence on ARCH_NAME in dev_msgs.h

Ticket
#14643
Problem description
dev_msgs.h is commonly included in host code, and poisons any translation unit with ARCH_NAME specific code.
What's changed

Macro GET_MAILBOX_ADDRESS_DEV has been changed to accept an additional parameter mailbox_base

This allows the macro caller to bring in the ARCH_NAME specific mailbox base address, and gets the problem out of host code.

Static asserts that lived in dev_msgs.h are moved to Hal.

Doing this required the introduction of two new files:

mailbox_base.h: Gives ARCH_NAME / Processor specific mailbox_base address
hal_asserts.hpp:  Gives ARCH_NAME / Processor specific static asserts to hal.

Checklist
https://github.com/tenstorrent/tt-metal/actions/runs/11786676019

[ ] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

When I first started this PR, I thought GET_MAILBOX_ADDRESS_DEV was called in host code.
It is not. Instead, this is called: GET_MAILBOX_ADDRESS_HOST.
This means that I could have solved the inclusion problem with another ifdef...
Something like:
#if defined(KERNEL_BUILD) || defined(FW_BUILD)

.. Existing code that has GET_MAILBOX_ADDRESS_DEV

#endif

However, I think the changes I made by adding a parameter to GET_MAILBOX_ADDRESS_DEV are good cleanup anyways.
They make dev_msgs.h more readable, and they highlight that not all calls to the macro are the same in the various program points where the macro is called.


[Bug - next-pwa]: Deployment with vercel

Provide environment information
I am currently using the 9.7.2 version and have the latest node version
Link to reproduction - Issues with a link to complete (but minimal) reproduction code help us address them faster
https://github.com/DuCanhGH/next-pwa/issues/107
To reproduce
Deploy the app using vercel with a custom service worker.
Describe the bug
Hey again, I am not sure if its related, but with the modification of babel due to the custom service worker not being recognized when specifying it, deploying the app using vercel provides this error:

Installing babel by itself using npm install make the deployment work but the custom service worker we want to use is then not recognized. Not sure if this is expected behavior but I was hoping if you could help us with this or maybe the new repo provides a solution for this.
Expected behavior
Expected behavior: to be able to deploy our application using vercel while also having the custom service worker being targeted and used.
Screenshots (if relevant)
No response
Additional information (if relevant)
No response

That's just your average ESLint moment. We don't make use of Babel at all, so I'm not sure why you think this plugin is related. I don't even know why you'd use ESLint in a bundled file either...

I see, would you happen to know how to have the service worker registration working while deploying the app on vercel ? I thought the error would be related to babel but I guess I was wrong. This is the error we are getting:

As a result, this is what we see in dev mode:

@MrMelon1232 you are precaching "/~offline" multiple times:

Recheck your precache list. Looking at your source, I noticed this line: t7.push({ url: "/~offline" }), which was unnecessary because the provided precache list already has "/~offline": { revision: "dKSb-EVmPw8vg-p8-Zh0w", url: "/~offline" }. One more thing, it is recommended that you provide a revision unless this revision is in the URL itself, which is not the case for "/~offline".
Next time, if your service worker has an issue, you can go to Developer Tools > Application > Service Workers to see why. Hope this helps!

It works thanks! Weird how the error does not come up locally. I'll be closing it now.


Cleanup of on-top-of-Kubernetes solutions

Motivation
I think there are too ways to deploy Falco on top of Kubernetes (deployment VS daemonset + with-RBAC VS without-RBAC + Falco slim container image VS Falco "full" container image).
Feature
I'd maintain only the daemonset solution since the driverloader should run on every node, with RBAC enabled since it is the default authorization mechanism for apps in k8s.
Furthermore, since this is not an official method to deploy Falco (kernelspace and userspace components), I think that it's better to keep it simple.
Additional context
The Falco images are going to be under review; see here for details.
/kind feature

Has that been fixed?
I believe so, except for the slim image that should be fixed by https://github.com/falcosecurity/contrib/issues/5
Furthermore, integrations/k8s-using-deployment seems to be a different use case that we should still keep, IMO.

Yes, I agree. As the deployment solution should still be in place this issue can be considered fixed for me too.


Not working W10 22H2 19045.4894

I have Mouse Jiggler active, set on and have tried both with Zen and without .. 5s interval.
Used to work until the last windows update. My use case is keeping GeForce Now running. Post update this no longer works for that timeout (admittedly GFN has had a couple of updates too so maybe they patched this out of spite).

This is happening to me after the latest update(s) as well, but I am on W11 23H2 22631.4169

For me it stopped working with W10 update 22H2 19045.5131

Since this program is temporarily not working, I took the liberty of forking it and completely rewriting it in F#. If anyone is interested, feel free to test it. Everything should work without any issues:
https://github.com/MaxGripe/mouse-jiggler-console
You can download the file by clicking "Auto-release" on the right side. The build is done automatically on GitHub's side


Laravel 10.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 9.x.
Before merging, you need to:

Checkout the l10-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: Shift detected a dependency for the laravel/framework. Unless you have a special requirement for depending on the entire Laravel framework, it's recommended to add dependencies for the specific illuminate packages instead.

:warning: Shift updated your build matrix for Laravel 10. However, since you are supporting older versions of Laravel, you need to add the following exclusions:
exclude:
  - laravel: 10.*
    php: 8.0

:warning: Shift updated your build matrix for Laravel 10. However, since you are supporting older versions of Laravel, you need to add the following exclusions:
exclude:
  - laravel: 10.*
    php: 8.0

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 10. However, you should review these changes to ensure the desired combination of versions are built for your package.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-eloquent-spatial.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "matanyadaev/laravel-eloquent-spatial": "dev-l10-compatibility",
    }
}

Finally, run: composer update


Getting "TypeError: cannot pickle '_thread.RLock' object" error when trying to send emails using AsyncTask

I can this error when trying to use django-q2 package when sending emails:
INFO 2024-04-06 14:33:40,148 worker 22676<PHONE_NUMBER>75520 Process-0b549de9691d49f9b9dd40206f0ff9cf processing delaware-uranus-lemon-seventeen '<bound method EmailMessage.send of <django.core.mail.message.EmailMultiAlternatives object at 0x73831f582440>>' [email]
Traceback (most recent call last):
  File "/usr/lib/python3.10/multiprocessing/queues.py", line 244, in _feed
    obj = _ForkingPickler.dumps(obj)
  File "/usr/lib/python3.10/multiprocessing/reduction.py", line 51, in dumps
    cls(buf, protocol).dump(obj)
TypeError: cannot pickle '_thread.RLock' object

My code looks like this:
        with get_connection(
            host=settings.EMAIL_HOST,
            port=settings.EMAIL_PORT,
            username=settings.EMAIL_HOST_USER,
            password=settings.EMAIL_HOST_PASSWORD,
            use_tls=True,
        ) as connection:
            subject = data["subject"]
            body = data["message"]
            recipient = data["recipient"]
            email = EmailMultiAlternatives(
                subject=subject,
                body=body,
                from_email=settings.DEFAULT_FROM_EMAIL,
                to=[recipient],
            )
            message = prepare_message(subject, body)
            email.attach_alternative(message, "text/html")
            a = AsyncTask(email.send, group="email")
            a.run()

When I tried this, it worked fine without error:
a = AsyncTask('math.floor', 1.5, group="math")
a.run()

Also, I tried to send email without using AsyncTask and it worked fine also.
I'm using Python  3.10.12, Django 4.2.11, and django-q2 1.6.2.
Do you have any idea about this problem or how can fix it?
Let me know if I should provide more code/details.

It's sometimes tricky to pickle functions based on objects and can lead to errors. Pickling works best for standard things, like arrays, strings, integers...
I would suggest moving the whole email sending thing into a separate function and triggering that through AsyncTask.

It's sometimes tricky to pickle functions based on objects and can lead to errors. Pickling works best for standard things, like arrays, strings, integers... I would suggest moving the whole email sending thing into a separate function and triggering that through AsyncTask.

That worked for me. Thanks!


⚠ Processors has degraded performance

In 1cad773, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 151 ms

Resolved: Processors performance has improved in b19a2ad.


proposal: Check if migrations Needed

Use Case:
I am using ent with a external migration tool for sql that ent does not currently support.
I would like to know if Migrations need to be run as a way to check if the external created schema matches what ent expects.

Hey (again) @cliedeman and thanks for proposing this issue 😃
I actually want to add support  for this, and it should be easy. I'll add more details about it in the upcoming days.

@a8m Thanks, I looked over the code briefly. I think would just need to refactor the migration code int a plan and apply phases and this feature should be very easy to implement after that. Also somewhat related to offline migrations

pretty sure this is superseded by atlas


[New Exercise]: Simple Cipher

Looking to add the Simple Cipher exercise for Object Oriented October.

To simplify the exercise creations, we generate from a template file, it often also makes the creation of the test file faster compared to writing it manually. And we can make changes with very little effort. Here is a doc on the Crystal test generator: https://github.com/exercism/crystal/blob/main/docs/generator.md. When writing the template, I recommend looking at how the other templates have been written and we do use ecr, which is the built-in template engine in Crystal. So there are no dependencies for running the script.

I could write you a template for this exericse if you like, but I would recommend to try it yourself.

I'll have a go at creating the template ;)


Maintenance

Release featuring build fixes and memory leak fixes.
@nagineni could you PTAL?

seems like we really need 18bdb59.

LGTM


fix: handle non-hex-encoded utf8 values

Description
Sometimes we receive non-hex-encoded utf8 values in a payload. This does not conform to the standard, but we should handle this anyway, since the conversion is only used for display purposes.
Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
N/A - helps with Mercle.
Risk
Small, since this is mainly focused on preventing the app throwing when a malformed request is received.
Testing
Log in, and mint a Mercle NFT at https://app.mercle.xyz/shapeshift/.
Engineering
☝
Operations
☝
Screenshots (if applicable)
N/A

Current dependencies on/for this PR:

develop

PR #4600   👈

This comment was auto-generated by Graphite.


Feature: Python code answers for FreeCodeCamp Data Analysis certification

Is your feature request related to a problem? Please describe.
Python code files for the projects you need to submit in the FCC Data Analysis certification
Describe the solution you'd like
I can upload the code that I submitted for the course if this falls under hacktoberfest

You can upload any code file you want

I would like to work on this issue as I worked extensively in the field of data analysis. On freecodecamp, 5 data analysis projects statements are available. Is it ok to submit the solution of any 1 project?


Why place f5_text_embed layer in F5_Preprocess (instead of F5_Transformer)

Hello, thank you for this great work! I'm wondering why you decided to put f5_text_embed in F5_Preprocess instead of F5_Transformer.

Firstly, the f5_text_embed process runs only once, so it should not be included in the NFE Steps loops.

Secondly, the text embedding process involves CPU-only operators like torch.arange() and torch.zeros(). To enhance execution speed, it would be better to split this process to utilize the GPU or NPU.

In summary, we aim to minimize the required computation and internal hardware communication as much as possible.


When exporting E2E keys, it isn't clear that you are creating a new password

The interface makes you think that you are entering your existing Matrix account password, not creating a new one. https://matrix.to/#/!GnEEPYXUhoaHbkFBNX:matrix.org/$1538229553932472jftKg:matrix.org
Instead of "Enter passphrase", maybe "Create passphrase" is better. Or add in some of the text Riot web uses like "To help with this, you should enter a passphrase below, which will be used to encrypt the exported data."

Done


Unable to copy to a public ECR repository

Hello,
I have been trying to copy an image from private ECR repo to a public ECR repo, within the same account. I did both modify the dest/public repository's access policy to allow lambda to perform ecr-public actions, and give the lambda role permission to do the same.
However, I'm getting:
2022/10/27 03:20:12 sending status failed: copy image failed: trying to reuse blob sha256:af353cabf27cab47a389872bcfcd8f93308ee7e298d843e01e420fb991f394fa at destination: checking whether a blob sha256:af353cabf27cab47a389872bcfcd8f93308ee7e298d843e01e420fb991f394fa exists in public.ecr.aws/alias/test-ecr: unauthorized: authentication required

Below is the code:
        this.repository = new CfnPublicRepository(this, 'TestPublicECR', {
            repositoryName: 'test-ecr',
            repositoryPolicyText: {
                "Version": "2008-10-17",
                "Statement": [{
                    "Sid": "ECR Public Repository Policy",
                    "Effect": "Allow",
                    "Principal": {
                        "Service": "lambda.amazonaws.com"
                    },
                    "Action": [
                        "ecr-public:*"
                    ],
                }]
            }
        });

        this.image = // image here
        this.ecrImageLocation = this.image.bind(this);

        const deployment = new ECRDeployment(this, 'DeployDockerImage', {
            src: new DockerImageName(this.ecrImageLocation.friendlyImageUri),
            dest: new DockerImageName('public.ecr.aws/alias/test-ecr:latest'),
        });

        deployment.addToPrincipalPolicy(new PolicyStatement({
            sid: 'PublicEcrSid',
            effect: Effect.ALLOW,
            actions: [
                "sts:GetServiceBearerToken",
                "ecr-public:*",
            ],
            resources: ['*']
        }))

Any pointers?

I updated it to pass a custom role to the ECRDeployment construct, and add the role to the repository's access policy, stack failed with same cause, lambda invocation somehow returned success, but no CloudWatch log group was created, no log found.
        const statement = new PolicyStatement({
            sid: 'TempSid',
            effect: Effect.ALLOW,
            actions: ["sts:GetServiceBearerToken", "ecr-public:*"],
            resources: ['*']
        });

        const deploymentRole = new Role(this, 'ECRDeploymentRole', {
            assumedBy: new ServicePrincipal('lambda.amazonaws.com'),
            roleName: 'ECRDeploymentRole',
            inlinePolicies: {
                'PublicEcrPolicy': new PolicyDocument({
                    statements: [statement]
                })
            }
        })

        this.repository = new CfnPublicRepository(this, 'TestECR', {
            repositoryName: 'test-ecr',
            repositoryPolicyText: {
                "Version": "2008-10-17",
                "Statement": [{
                    "Sid": "ECR Public Repository Policy",
                    "Effect": "Allow",
                    "Principal": {
                        "AWS": [deploymentRole.roleArn] // also tried including aws account, and lamba SP here.
                    },
                    "Action": [
                        "ecr-public:*",
                        "ecr:*"
                    ]
                }
                ]
            }
        })

        const deployment = new ECRDeployment(this, 'DeployDockerImage', {
            src: new DockerImageName(this.ecrImageLocation.friendlyImageUri),
            dest: new DockerImageName('public.ecr.aws/alias/test-ecr:latest'),
            role: deploymentRole
        });

Getting the same issue. Any update on this ??
Some more information from the logs
2023/12/30 17:46:59 SrcImage: docker://<accountId>.dkr.ecr.ap-southeast-1.amazonaws.com/<repo>:qa DestImage: docker://public.ecr.aws/<>>/<repo>:prod
2023/12/30 17:46:59 ECR auto login mode for docker://<accountId>.dkr.ecr.ap-southeast-1.amazonaws.com/<repo>:qa
Getting image source signatures
2023/12/30 17:47:02 sending status failed: copy image failed: trying to reuse blob sha256:<some hash> at destination: checking whether a blob sha256:<some hash> exists in public.ecr.aws/<name>/<repo>: unauthorized: authentication required
END RequestId: <RequestId>
REPORT RequestId: <RequestId>	Duration: 2779.77 ms	Billed Duration: 2780 ms	Memory Size: 512 MB	Max Memory Used: 64 MB


Respect umask in WriteFile

Fixes #33 by taking the umask into consideration in renameio.WriteFile.

Just checking that this won't change the current behavior for existing users of renameio.WriteFile. My specific case:

I use renameio.WriteFile.
I handle umask already, including allowing the user to set a my-application-specific umask, which may be more permissive than the system's umask.

This PR is clearly an improvement. If this PR does change the default behavior of renameio.WriteFlie then it should be accompanied by a major version bump in the next release of renameio.

I handle umask already, including allowing the user to set a my-application-specific umask, which may be more permissive than the system's umask.

I'm assuming you don't set the application-specific umask via umask(2) (e.g. because it may vary per user and goroutine). With this PR there will be a change in behaviour depending on what the system/process umask is. That umask will be applied on top of your permissions, regardless of whether you applied an application umask or not—the syscall just sees the permissions given to open(2). Before this PR the file permissions would always be overwritten using fchmod(2) and that's where renameio.WriteFile differed from ioutil.WriteFile (see #33).
So yes, as you suggest, a major version bump is in order. @stapelberg, when reviewing and merging, can you tag a new release?

I'm assuming you don't set the application-specific umask via umask(2)

You're correct :) I use the umask at startup as the default value, but the user can override it in their config file. The "umask at startup" is read in a package's init function, and I rely on hope that no other package modifies it beforehand.
Random other umask factoids that I discovered the hard way:

Debian-based and RedHat-based Linux distributions use different default umasks (you already knew this).
Most distributions created new users' home directories by copying /etc/skel and the files in /etc/skel are not always consistent with that distribution's umask (specifically, group permissions can differ).
There is a bug in go test: it re-uses cached test results even if the umask has changed, which is incorrect. Specifically, umask 002 ; go test ./... ; umask 022 ; go test ./... should re-run the tests on the second invocation of go test ./... but instead go test re-uses the cached test results (I haven't reported this because I haven't dived deep enough to provide a fix along with the bug report).

While I appreciate proceeding with caution, I’m not sure we really need a major version bump here?
Citing from https://golang.org/doc/modules/version-numbers:

A module developer should increment this number past v1 only when necessary because the version upgrade represents significant disruption for developers whose code uses function in the upgraded module. This disruption includes backward-incompatible changes to the public API, as well as the need for developers using the module to update the package path wherever they import packages from the module.

I’d like to avoid everyone having to switch their imports to /v2, because I suspect that few people actually care about the umask.
Do we have any indication that this change might break existing setups? Can we identify repositories that use both renameio and umask handling?
Or, asked differently, what guidance would we give developers who are upgrading from the current version to the new version? How should they change their code, and/or verify things are working?

Do we have any indication that this change might break existing setups? Can we identify repositories that use both renameio and umask handling?

I think it will break chezmoi in some edge cases. google/renameio is imported by at least 64 importers, which in turn may be imported themselves. I don't think it's reasonable to audit so many packages for dependence on the current behavior.

Or, asked differently, what guidance would we give developers who are upgrading from the current version to the new version? How should they change their code, and/or verify things are working?

Maybe something like:
"Since version X, renameio.WriteFile changes the way that permissions are handled. Before version X, the file was created with the permissions passed to the function. From version X onwards, these permissions are further modified by the user's umask. If you were relying on the pre-version X behavior, you must change your code to explicitly set the permissions you want after calling renameio.WriteFile, for example with os.Chmod. Note that the call to os.Chmod will not be part of the atomic replacement, i.e. the file may be read, modified, or deleted between the calls to renameio.WriteFile and os.Chmod."
I actually want the current behavior and I want the precise permission setting to be part of the atomic operation, and it is not possible to achieve the current behavior with the change proposed here. Some options:

Modify the renameio.WriteFile to allow the user to pass in renameio.Options which are then passed to NewPendingFile.
Add a new function that behaves as the current renameio.WriteFile for users who want the current behavior, although they will have to modify their code to use the new function.
Bump the version of this repo to v2 when this PR is merged.
Tell users to use a fork of this repo instead of google/renameio.
Tell users to exclude all versions of google/renameio that implement the new behavior in their go.mod files (this only works for the main module of course, users will have to audit their go.mod to see if they actually depend on google/renameio).
Of these, my preference order is 3, 2, 1, 4, 5.

I’d like to avoid everyone having to switch their imports to /v2, because I suspect that few people actually care about the umask.

Bumping major versions of Go packages really isn't a big deal. https://github.com/google/go-github is already a v39, for example. Major version bumps are for exactly these sort of changes. Silently changing the default behavior without providing a work-around for users to even achieve the current behavior is pretty hostile.

Thanks for the details.
I agree that bumping the major version isn’t that big a deal, but it also sounds like there are other reasons to change this PR to be backwards-compatible (keep current behavior), probably by introducing an optional options struct.

I'd like to have a confirmation before I touch the code again.

WriteFile is documented as follows:

WriteFile mirrors ioutil.WriteFile, replacing an existing file with the same name atomically

We should clarify that WriteFile doesn't mirror ioutil.WriteFile fully, but rather always sets the mode as given in the perm parameter ignoring the umask.

Add another function receiving options, e.g. WriteFileWithOptions(path string, data []byte, opts ...Option) error. Users can then invoke that with WithPermissions(perm) to have the umask respected.

Is this correct?

My $0.02c:

We should make renameio.WriteFile behave like ioutil.WriteFile by default, i.e. respect the umask.
We should add opts ...Option to renameio.WriteFile so WithPermissions(perm) can be passed as @hansmi proposes above. Note that this does not change the use of renameio.WriteFile for existing users.
We should bump google/renameio to v2 to reflect the change in default behavior.

@stapelberg, do you have an opinion on which path we should take?

Thanks for the details. I would prefer:

Make renameio.WriteFile mirror ioutil.WriteFile, because I consider that the cleaner long-term goal. The shorter the docs can be (without any extra thought that users need to spend on whether the umask affects them), the better.
We can add opts ...Option to renameio.WriteFile (no need for a WriteFileWithOptions), because we need to bump the major version anyway.
We should document (in README.md, I suppose) how to switch from the current version to the v2 version. Maybe something along the lines of (adapting @twpayne’s suggestion):

With major version renameio/v2, renameio.WriteFile changes the way that permissions are handled. Before version 2, files were created with the permissions passed to the function, ignoring the umask. From version 2 onwards, these permissions are further modified by the user's umask.
If you were relying on the umask being ignored, add the renameio.IgnoreUmask() option to your renameio.WriteFile calls when upgrading to v2.

@stapelberg, @twpayne, the latest revision of my changes include the IgnoreUmask option mentioned in https://github.com/google/renameio/pull/36#issuecomment-929102354 and update the README file. Please take another look.

This looks great @hansmi. Thank you so much for your work and patience!


Consider changing return type of ISpendingCondition.verify

Issue Type

[ ] bug report
[x] feature request

Current Behavior
ISpendingCondition.verify type signature indicates that the function returns a boolean value. Currently our implementation reverts transaction instead or returning false. If verification succeeds true is returned.
Expected Behavior
Either false is returned on unsuccessful verification or we change return type to void.
Suggested Fix
Make the function return void.

Closing this, as it is ingrained in our code. Changing that would not benefit the design greatly.


Do not execute irrelevant checks on certain pull requests

How to categorize this issue?

/area quality cost
/kind enhancement
What would you like to be added:
There are certain PRs that modify only documentation related files. There is no need to run all test suites in such cases. We can save time and computing resources by skipping the irrelevant suites in such cases. See this PR for example.
cc @oliver-goetz
Why is this needed:
Save time and computing resources.

Last time I checked, there have been only very few PRs (like 3 in a quarter) which changed docs only.
If we will exclude certain files from several tests, we cannot not enforce these tests via github branch protection anymore. We should not do this for this low number of PRs.

Thanks for the explanation! Feel free to close the issue if you think it is irrelevant for the time being.


Using full window space for pop up?

Is it possible to do something like this with MP?
http://www.jqueryscript.net/demo/jQuery-Plugin-To-Create-Fullscreen-Popup-Windows/
Thanks

Size and position of inline popup is defined just via CSS, feel free to modify default styles of  .mfp-content and .mfp-inline-holder, or force size via JS based on window width/height,


Feat Request (AngularFire Docs on sub domain of angular.io)

Feature request!
Wouldn't it be awesome to show case all of the Google supported plugins for Angular in one place? Well there is already a place, the angular.io domain!
AngularFire is not currently there.
no docs exist at firebase.angular.io or angularfire.angular.io
It would be awesome if Angular fire be properly sub domain-ed with the angular project.
Material.angular.io is a thing, why not firebase.angular.io ?
This would allow Google to showcase all the projects maintained for the angular framework in one place.
It would also allow a little more cross-team communication, which we could all benefit from. It might also help expose developers to the wonders of Firebase!
What would be required

[ ] Tooling to auto generate docs
[ ] Communication with the angular team to get the sub-domain
[ ] Examples that work in the browser instead of building a project.

Who can help

[ ] Firebase team needs to be on board as well as the angular team. Adding the sub-domain and basic build tooling that the material team has done.
[ ] Community can help make docs (the community has helped the Material 2 with docs a lot already)

Why

[ ] Firebase might be able to get a little bit more exposure. Which would be good for GCP and Firebase.
[ ] The community gets to learn about an awesome tool that might fit their use-case perfectly.

@davideast is the AngularFire 2 project suppose to just be support for Angular2 or is it support for Angular X ?

@M-a-c it's supposed to be for Angular version 2+. I believe there is a major re-write to the Module coming soon.

@markgoho Thanks

Follow #854 for all upgrade tracking.


Add support for HTTP(S) option

I run Identityserver locally for developing purposes too. Would be good to be able to use HTTP (and set HTTPS by default). #1

Integration test seems to fail to import the ids test certificate. I have not at all touched that part and it does run locally. Somehow an AppVeyor issue I'm affraid.

Hello,
@PeterButzelaar Thanks for the contribution.  I don't think we have to require https by default - it will create regression with the actual codebase. The default value should be false.


fixes #211: Add AVRO Support to the Streams Sink Plugin

Fixes #211
One sentence summary of the change.
Proposed Changes (Mandatory)
A brief list of proposed changes in order to fix the issue:

Added AVRO+Schema Registry support to sink and streams.consume procedure
updated documentation

Needs to be rebased again, sorry.

and the nested conversion has to be handled correctly (if needed).


Log error in compileTree.

Issue #391
There is a long chain of Promise in compileTree dealing with modules and plugins which also means there is a high chance of things could go wrong here but there is no log or any other information available  when it happens.

Thanks for the PR. Promise APIs should return error as rejections and not automatically log them though - that is up to the consumer to manage.

Sorry for the rushed PR, that was really bad.


fix: increase default listers on abort signals

To prevent spurious warnings appearing in the console, increase the number of listeners on created abort signals.
Change checklist

[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation if necessary (this includes comments as well)
[ ] I have added tests that prove my fix is effective or that my feature works

I agree it's a shame this is necessary but it seems unlikely to stop being so given that we have a top-level abort signal being passed into a non-trivial set of operations, all of which need to be torn down if the signal fires it's "abort" event.
I don't think node should emit this warning, evidence of a memory leak is memory going up and not coming down, not listeners being added to an event emitter.


[BACKPORT][v1.0.2] - Fix postgres-exporter service errror:  Error opening connection to database

Is your feature request related to a problem? Please describe.
When creating cluster with  postgres-exporter role enabled, the result is that the postgres-exporter service is not exporting metrics at all due to the Error opening connection to database:
Jan 04 09:26:57 cicharka-postgres-test-centos-postgresql-vm-0 postgres_exporter[23084]: time="2022-01-04T09:26:57Z" level=error msg="Error opening connection to database (postgresql://:PASSWORD_REMOVED@): malformed dsn \"\"" source="postgres_exporter.go:1474"

Describe the solution you'd like
Solution is to remove additional quotation marks from the systemd unit template for Environment field:
Environment="{{ specification.systemd_environment|default("DATA_SOURCE_NAME=postgresql://postgres@:5432/postgres?host=/var/run/postgresql/&sslmode=disable") }}"

Environment
Found in epicli v1.0.1, only on machines that run CentOS

DoD checklist

[ ] Changelog updated
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Schema updated / doesn't need to be updated
[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Idempotency tested
[ ] Documentation added / updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Solution meets requirements and is done according to design doc
[ ] Usage compliant with license
[ ] Backport tasks created / doesn't need to be backported

:heavy_check_mark: The service template has been fixed.


Stopped working?

Out of the blue, it seems this isn't working anymore.
I have power cycled my harmony hub several times without any luck.
% ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: icmp_seq=0 ttl=64 time=2.267 ms
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=64 time=12.470 ms
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=64 time=3.374 ms
^C
--- <IP_ADDRESS> ping statistics ---
3 packets transmitted, 3 packets received, 0.0% packet loss
round-trip min/avg/max/stddev = 2.267/6.037/12.470/4.571 ms
% telnet <IP_ADDRESS>  5222
Trying <IP_ADDRESS>...
telnet: connect to address <IP_ADDRESS>: Connection refused
telnet: Unable to connect to remote host
%

map scan report for harmony-hub (<IP_ADDRESS>)
Host is up, received user-set (0.020s latency).
Scanned at 2018-12-21 18:31:13 PST for 5s
Not shown: 65533 closed ports
Reason: 65533 conn-refused
PORT     STATE SERVICE    REASON
8088/tcp open  radan-http syn-ack
8222/tcp open  unknown    syn-ack

I tried passing those two ports to the constructor, but nothing.

awesome, that worked.
Thank you!
Maybe a comment in the README.md about this issue, so people aren't so easily lost (like me!).

The XMPP option is now easy to enable using the Harmony app, and future firmware upgrades will apparently honor the opt-in setting.


[DOC] Sass - add custom boosted Sass-variables to documentation

Following Bootstrap improvement : https://github.com/twbs/bootstrap/commit/4c7a3e8adf73bf55a5fba5905b356468be17bc0e
As pointed out by @julien-deramond
adds Sass docs (variables, mixins, and loops) to most pages. Some of them specific to Boosted aren't they missing? I'm thinking about @each $name in map-keys($btn-social-networks) { in _button.scss for example.

[ ] List custom boosted sass vars candidate
[ ] Add relevant ones to documentation

Closing it. We've done it in one of the v5.x in the whole documentation to ensure consistency.


fixpaimon support more data type

Proposed changes
Issue Number: close #xxx
Support for more data types and predicate pushdown of more types
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.23 seconds
stream load tsv:          508 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run buildall


Cannot move cursor up after soft wrap on punctuation

Steps to reproduce:

Go to http://codemirror.net/demo/variableheight.html (which has soft line wrapping turned on).

Enter this string
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa-bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbccccccccccc

It will wrap:

Put your cursor after any of the cs and press the up arrow.

Expected behaviour: cursor goes onto the line above (with the as).
Observed behaviour: cursor goes to the start of the line, and pressing the up arrow again does not move the cursor (cursor gets stuck).
If the cursor starts after any of the bs, the behaviour is as expected.
If, once the cursor is stuck, you press left or right, the behaviour is as expected after that.
The behaviour seems consistent across Firefox, Chrome and Safari on Mac.
The original use case was a long LaTeX equation with no spaces, and the problem originally occurred when the line soft wrapped on }{, rather than a hyphen; it seems that parentheses )( will also cause the problem. So, these strings show the same behaviour:
aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa}{bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbccccccccccc

aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa)(bbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbccccccccccc

It's clearly not the end of the world, but it does seem like a bug. HTH!

This is now fixed in the current master branch.


BoundingBox Script- Proximity Scaling of Handles- Shell Parity

Overview
changes to Boundingbox.cs script and HandInteractionExamples scene. Added Proximity scaling behaviors to boundingbox handle affordances to match the Shell Boundingbox behavior.
Near interaction

Far interaction

Changes

Fixes: #4415

Verification
When either handRay or hand is 'far' away(as defined by an added Property) each handle scales to a 'far' scale- zero by default. When hand is at a medium distance, an initial scale animation plays. Then when close, a final close animation plays. during each animation, the property-specified material is applied. The behavior is intended to mimic the Shell boundingbox behavior - an initial handle increase in scale, followed by a final scale with a 'blue' material applied.

Default values in the script are based on the use cases illustrated in the HandInteractionExamples scene.

John is looking into a material related memory issue.

Looks like there are some merge conflicts -- could you please resolve them?

Hi John, it looks like there are 229 files changed in this PR -- could you please double check that you are merging from the correct branch? It looks like you might be using the wrong base branch.

John it loks like there are conflicts again, could you please resolve?

Closing this PR - New PR opened #4895


aspectRatio + undefined width/height

Environment
React Native Environment Info:
    System:
      OS: Linux 4.15 Ubuntu 18.04.1 LTS (Bionic Beaver)
      CPU: (12) x64 AMD Ryzen 5 2600X Six-Core Processor
      Memory: 15.10 GB / 31.34 GB
      Shell: 2.7.1 - /usr/bin/fish
    Binaries:
      Node: 8.10.0 - /usr/bin/node
      npm: 3.5.2 - /usr/bin/npm
    npmPackages:
      react: 16.6.3 => 16.6.3
      react-native: 0.57.8 => 0.57.8
    npmGlobalPackages:
      react-native-cli: 2.0.1
      react-native-git-upgrade: 0.2.7

Description
aspectRatio and height: 100% seem to work only when width is set to undefined, but with aspectRatio and width set, then height is not required at all. (See Demo)
Why is that?
Reproducible Demo
Working fine:
aspectRatio: 1,
width: '100%',

aspectRatio: 1,
height: '100%',
width: undefined,

Hey @santiph thanks for filing
If I understand correctly, this is the way aspectRatio is designed to work since aspectRatio controls the size of the unset dimension. If I'm misunderstanding, can you provide a snack or repro that demonstrates?


Add openapi multiple examples

Hey @anukuljha, appreciate you closed this but here's a heads up on how examples work. They already exist via the mostRecent type https://github.com/AndrewWalsh/openapi-devtools/blob/main/src/utils/types.ts#L58.
This type is unknown. If you wanted to include multiple examples, then it would be best to simply change this to unknown[]. There is no need for a new Example type. Appreciate the code and architecture is not documented.
The implementation is pretty straightforward beyond this. If users enable the feature, then the option enableMoreInfo becomes true, and is included in the MediaTypeObject https://github.com/AndrewWalsh/openapi-devtools/blob/main/src/lib/endpoints-to-oai31.helpers.ts#L97.
An issue with this approach is that the data structure will become very large, and the size becomes unbounded. This could compromise speed, and would affect the import/export feature. Would you be open to raising an issue before raising a PR regarding this matter?

Hey @anukuljha, appreciate you closed this but here's a heads up on how examples work. They already exist via the mostRecent type https://github.com/AndrewWalsh/openapi-devtools/blob/main/src/utils/types.ts#L58.
This type is unknown. If you wanted to include multiple examples, then it would be best to simply change this to unknown[]. There is no need for a new Example type. Appreciate the code and architecture is not documented.
The implementation is pretty straightforward beyond this. If users enable the feature, then the option enableMoreInfo becomes true, and is included in the MediaTypeObject https://github.com/AndrewWalsh/openapi-devtools/blob/main/src/lib/endpoints-to-oai31.helpers.ts#L97.
An issue with this approach is that the data structure will become very large, and the size becomes unbounded. This could compromise speed, and would affect the import/export feature. Would you be open to raising an issue before raising a PR regarding this matter?

@AndrewWalsh Hey thanks for replying.
I accidentally raised PR on main repo instead of my forked version. I am working on developing some ways to detect ENUM fields based on all recorded samples during the session, hence I have tried to modify the leaf to store all examples of request/response for the given path.
It is still a work in progress and I didn't mean to raise the PR on main repo yet. I am able to build something usable on this, will surely raise PR when it's ready.

Ok cool, sounds interesting. Server based tools like Akita can achieve this because they receive tens or hundreds of thousands of requests. They have enough data to sift through and determine patterns such as fields that are enums. This tool doesn't have that advantage as it is only aware of requests that the client has triggered. It doesn't have anywhere near that much data, and works on the assumption that an endpoint may be defined by only a single request. Collecting samples en masse would be problematic and wouldn't suit this tool, but an alternative data structure might work. Just a friendly heads up, as data validation is very difficult to determine in the context this tool works with given a very limited set of data.


Stereopy input (cellbin)

Hello,
I have a question about input of stereopy.
I ran successfully the stereoPipeline_v7.1 pipeline with cellbin.
Now, I want run the Quick Start (cellbin) tutorial.
Which input should I use between adjusted.cellbin.gef and cellbin.gef ?
Thanks for help!

I guess stereoPipeline_v7.1 is a pipeline containing Cell-Correction function, adjusted.cellbin.gef is the corrected GEF, so, generally, we should use adjusted.cellbin.gef to do subsequent analysis.

Thanks a lot for the reply


[D&D_Moldvay] Number of Attacks

Requesting a dedicated space on the monster tab (possibly near the weapon stats) to mark how many attacks a monster can make.

This issue has been resolved as of the most recent update.


[FEATURE] Check order of methods in spec

Is your feature request related to a problem?
Coming from https://github.com/opensearch-project/opensearch-api-specification/pull/271 where we reordered a DELETE method.
What solution would you like?
A linter that orders methods (probably GET, POST, PUT, PATCH, DELETE).
What alternatives have you considered?
Alphabetical order.

This should be applied to entire spec folder right?
We should also alow --fix option in the linter to auto fix issues like this.

@nhtruong Yes.

If someone else is picking this up: We already have a helper function that can do in-place object sorting with a custom priority list. (Added this previously so I can add custom sorting for operation properties for the linter later)

We should also alow --fix option in the linter to auto fix issues like this.

https://github.com/opensearch-project/opensearch-api-specification/issues/327


Support for exporting whole folder contents at once

Is it possible to implement a feature to export all the files inside a folder to image in one go? Exporting them one by one seems impractical when you have large collections of notes.

It's possible. Could you discuss why there is such a requirement? If this scenario is realistic and important, I can prioritize implementing this feature.

For example, I am making a collection of notes for my math, physics and chemistry classes. In case i ever want to share them, my classmates would need to use obsidian to view them. It's impractical to make them install an app just to view a couple documents. It's also more convenient to store export images in an archive for future viewing instead of markdown.
Additionally, I am using a tikz obsidian plugin, and that doesn't get rendered outside Obsidian without the plugin. For example:
Ignore the content of the note, it's currently incomplete

I'm pleased to tell you that I've implemented the feature you wanted, and you can download the latest version to try. You can right-click on a folder in the Obsidian file list and click export all notes to image.
I think sometimes, when sharing course materials, perhaps a PDF file is better than many image files. Today, I spent a few hours specifically supporting this type of PDF, guessing it might be useful to you.

Thank you so much for this! Also, thing is, I'm not going to sharing the whole thing at once, instead either maintaining an archive or send whichever notes I need to. But the PDF feature will probably come in handy later, so much appreciated regardless.

I just tested the plugin, and I noticed a bug. While the tikz plugin renders in Live Preview mode, it doesn't render anything when exporting the entire folder. I assume this is because of some inter-plugin issues but I'm not sure.

I just realized I should mention which plugin I'm using, sorry.

Could you share the content of the note that's causing issues, so I can replicate the problem?

Okay, wait one second. I realized that it's not working AT ALL for image export and works for select files with PDF export.
Intervals.md

Please try again with the latest version. The issue likely occurs because there isn't enough time given for charts to render completely during batch image export. I've added a bit of delay, which should solve the problem.


Test avro and json dataset schema.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.DataFactory

@wangbwn please fix the errors

@ArthurMa1978 This PR introduce the new serialization model. https://github.com/Azure/azure-sdk-for-net/commit/579850647ecb9b7d21223559c3770ffcb078bea2 . Seems the Serialization is changed, will this have impact on our customization code?

The recording file is updated, in our local test all test case passed. But the CI check still failed. Maybe the failed test case is pipeline infra issues.


Utility of lane_number: time to deprecate?

The optional lane_number property on the Lane object was left in the major v3.0 release as there were reservations about giving a lane with a not typically driveable lane type (e.g. shoulder or sidewalk) a number. For clarity and to remove connection to what individuals think of when they see "lane number", the required lane order property was added. order is used as an index to identify the lanes positions (order) relative to each other. For many contributors/producers, this was the utility of lane_number, so with order, lane_number has no value. I support that conclusion and think that lane_number remaining in the spec just adds confusion.
In summary, it is not clear that there is any utility in lane_number.
If there are any producers or consumers who see value in lane_number aside from identifying the lane's position, please raise your opinions!
If there is no value, the next step is to deprecate lane_number.

Every time I present the spec, I always gloss over lane_number as for new adopters it adds confusion rather than value.
Seeing again no new comments here, I'll move forward with the PR.

Thumbs up.

Completed in #175.


Feedback for netserver Plug-ins

The plugins subcategory is duplicated:

Document Details
⚠ Do not edit this section. It is required for docs.superOffice.com ➟ Docs Team processing.

Content Source: netserver_patterns

Fixed by https://github.com/SuperOfficeDocs/superoffice-docs/commit/24739c12f6b571451e4eb3604b36cf03b8bfbed2


Wrong directory in updated settings.gradle

When integrating the react-native-vector-icons a wrong directory is in the updated settings.gradle file.
react-native init TestProject
cd TestProject
npm install --save react-native-vector-images
rnpm link
cat android/settings.gradle

The output is
rootProject.name = 'TestProject'
include ':app'
include ':react-native-vector-icons'
project(':react-native-vector-icons').projectDir = new File(rootProject.projectDir, '../../node_modules/react-native-vector-icons/android')

The last line should actually be:
project(':react-native-vector-icons').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-vector-icons/android')

Thanks for reporting. That is indeed our fault, it's been already fixed - waiting for @Kureev review and landing later today :)

(Already in npm - version 1.5.3. npm install -g rnpm and check again when you have time!)

thanks :heart:


Fix subject number in aas_addcontrast

I'm not 100% sure if this is a correct fix (or if it is optimal, as I am not a huge fan of MATLAB so my code may not  be the best MATLAB implementation) and/or if it may fail for some situations.
But this code works fine for me. Can you please review it?
I mentioned the issue here before: https://github.com/automaticanalysis/automaticanalysis/issues/336#issuecomment-1160843159
And I can elaborate if my explanation there is unclear.

I have looked at your changes, and I think it will not work if subjects have different sets of series because subj_id is set outside/before the loop.

I have looked at your changes, and I think it will not work if subjects have different sets of series because subj_id is set outside/before the loop.

I see. True. I forgot that subjname is a list of subjects. Let me see what I can do.
However,  there is an issue in this code: particularly when you call addsubject one by one for subject at a time in a loop.

I am not sure how adding subjects one at a time in a loop is a problem.

I will try explaining with screenshots later.
Assume we are looping over our 3rd subject, using aas_addcontrast.
The function receives an array of subject names. The loop (https://github.com/automaticanalysis/automaticanalysis/blob/331702ec9c23307c44012ad2b12b9cf735b36c17/aa_engine/aas_addcontrast.m#L265) loops over each subject in the array. When you pass only one subject, the numel is 1.
The loop initializes with subj value 1. And will always stay as that because I have provided only one subject. The variable subj is passed onto aas_getseries (https://github.com/automaticanalysis/automaticanalysis/blob/331702ec9c23307c44012ad2b12b9cf735b36c17/aa_engine/aas_addcontrast.m#L279)
aas_getseries receives subj and indexes subject list like this: aap.acq_details.subjects(subj).seriesnumbers;
or when using literals instead of variables, like this: aap.acq_details.subjects(1).seriesnumbers;
The series numbers of subject 1 is taken instead of the subject's real series number.

I see. Yes, it is an issue, indeed.

We could replace [~, mriser] = aas_get_series(aap,'functional',subj,sess); with [~, mriser] = aas_get_series(aap,'functional',strcmp({aap.acq_details.subjects.subjname},subjname{subj}),sess);

We could replace [~, mriser] = aas_get_series(aap,'functional',subj,sess); with [~, mriser] = aas_get_series(aap,'functional',find(strcmp({aap.acq_details.subjects.subjname},subjname{subj})),sess);

Aha, indeed. Let me try this.

I haven't tested this yet. I committed to test for myself on my other computer. I will tag you when I replicate previous analysis.

The code is fine for me. I was able to replicate the same analysis I've done before with that loop I'd implemented on my local branch.

Please merge the latest changes for your checks to be successful.

Please merge the latest changes for your checks to be successful.

Done. Though we only changed one line, since I have been sending commits in between, the fork registers as This branch is [4 commits ahead]
If this is a big problem (uselessly increasing the commit count whereas it's just 1 new line added) I guess I can re-base the branch and re-commit.


show white box for selected neurons

create white rectangle based on box size for selected neurons

@ArjunPutcha can you test sometime this week and then I will merge

@ArjunPutcha can you test sometime this week and then I will merge
Sure! Will do tomorrow

Works on my end.


Better user navigation with the header

Overview
This issue aims to provide better navigation through the header on teloscan.
Acceptance Criteria

Search bar should have highest affordance other than the burger menu
Light/dark theme toggle should be in the dropdown menu
Hide Connect Wallet from unsupported browsers (ie. mobile browsers outside of metamask app built-in browser)

Originally posted by @EJSG in https://github.com/telosnetwork/teloscan/issues/260#issuecomment-1355624163

See also: https://github.com/telosnetwork/teloscan/issues/175

Connect Wallet is supported by all browsers. It is just the Metamask part that isn't. But Anchor should work for example.

@tomtomcrypto Karyne is designing a new header overall. connect button acceptance criteria is out of date

@tomtomcrypto also metamask login already deeplinks to metamask app on mobile browsers

Will be addressed in https://github.com/telosnetwork/teloscan/issues/274


digraph not being displayed

Tried the digraph example and the graph isn't rendering at all.
https://github.com/maaslalani/slides/blob/main/examples/slides.md
Desktop (please complete the following information):

OS: OSX 11.4 Big sur
Browser ???? this is a terminal tool
Version No idea, the command doesn't have a version command.

It only displays the content As is instead of the Graph.

Can you try this example. https://github.com/maaslalani/slides/blob/main/examples/preprocess.md. The digraphs are a feature of pre-processing. You will need to copy this text to a local file and make it executable with chmod +x preprocess.md. You will also need GraphViz and graph-easy installed.

Oh, so sorry, I didn't really realise that. I will try.

Would you mind linking me to these tools, I'm absolutely oblivious. :D

https://graphviz.org/download/
https://metacpan.org/pod/Graph::Easy

Thanks!

Hey! I have GraphViz and graph-easy installed. But still cannot figure out how to use digraph. https://github.com/maaslalani/slides/blob/main/examples/preprocess.md doesn't provide any example with digraph...


Please provide a Docker version or guide me on how to compile it on CentOS 7. Your GLIBC version is too high.

Please provide a Docker version or guide me on how to compile it on CentOS 7. Your GLIBC version is too high.

# ./schemadiff
./schemadiff: /lib64/libc.so.6: version `GLIBC_2.32' not found (required by ./schemadiff)
./schemadiff: /lib64/libc.so.6: version `GLIBC_2.34' not found (required by ./schemadiff)

This should be fixed in #16.
For self-compiling, you should be able to run make to build it.

There's now a 1.1 release which is built with CGO_ENABLED=0 which means there's no glibc dependency anymore.

thanks


Test LiveServiceManager once dagger 2 can replace modules in test code

See issues:
https://bitbucket.org/hvisser/android-apt/issue/22/dagger-robolectric-gradle-apt-source-not
and
https://code.google.com/p/android/issues/detail?id=154651

Not sure what this has to do with lint, are you sure you didn't mean to put this on https://github.com/evant/gradle-retrolambda? Can you also explain exactly what the issue is? gradle-retrolmaba should already support unit tests.

sorry :( opened the wrong repo, this was an issue for myself :-1:


fix sed and update job

Summary

the sed was missing to parse the KO_PREFIX env var brackets so that fail as well

Step #3: sha256:5f67c995503680f34105fae4d825c057dbed16056ceabc4926de2c3ab82574ef
Step #3: sed -i -e 's/gcr.io/projectsigstore\/fulcio@.*/gcr.io/projectsigstore\/fulcio:v0.2.0/g' fulcio-v0.2.0.yaml
Step #3: sed: -e expression #1, char 38: unknown option to `s'

update the github user to the bot, which is the token key we use in cloudbuild
remove duplicate step in cloudbuild
update image to do the image copy to have cosign

rehearsal using this change https://github.com/cpanato/fulcio/releases/tag/v99.99.02
Ticket Link
n/a
Release Note
NONE

Codecov Report

Merging #470 (a3edb56) into main (e88278c) will increase coverage by 0.91%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #470      +/-   ##
==========================================
+ Coverage   45.66%   46.58%   +0.91%
==========================================
  Files          14       14
  Lines         957     1024      +67
==========================================
+ Hits          437      477      +40
- Misses        460      478      +18
- Partials       60       69       +9

Impacted Files
Coverage Δ

pkg/config/config.go
51.90% <0.00%> (+0.36%)
:arrow_up:

pkg/api/ca.go
46.59% <0.00%> (+0.61%)
:arrow_up:

pkg/challenges/challenges.go
18.77% <0.00%> (+13.80%)
:arrow_up:

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 e88278c...a3edb56. Read the comment docs.


Revised and improved examples.

This PR improves the existing examples and updates them to work with the current version of the model.
There were also a couple of minor changes:

Output writer API: Filename suffix is now called a file extension. No more inserting underscores for empty names and filename_prefix. User should include underscore if they want one.
Model metadata:  Use :CPU and :GPU instead of :cpu and :gpu. This is so that Oceananigans.jl agrees with the symbols used in GPUifyLoops.jl.

Model metadata: Use :CPU and :GPU instead of :cpu and :gpu. This is so that Oceananigans.jl agrees with the symbols used in GPUifyLoops.jl.

We probably should make these types

We probably should make these types

I like that idea! As simple as
abstract type Device end
struct CPU <: Device end
struct GPU <: Device end

in GPUifyloops.jl?


Go: Implementing PING command

Implemented PING command

Something went wrong with the CI, please have a look and fix 🙏


Everyone can open locked things

LWC Version (/lwc admin version):  2.1.2
Server version (/version): 1.13.2
Short five word summary of issue:  Even after locking things up other players can open it
Detailed description of problem: Hello i have a problem that when im trying to anyhow lock items on my server it just doesnt work, it says its locked but everyone can open the locked things...
If there was an error in console, put it here:
There was no error...

I can think of 2 possible issues off the top of my head:

You have a conflicting plugin (try to remove all plugins and see if the problem persists)
Your permissions are set incorrectly (check the wiki for permissions)

Also, be sure to update to 2.1.3, which is the latest version for 1.13.2

Closing this issue due to lack of response.


MuiTextField inputProps for "form" will disable onSubmit handler.

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Steps:

set default props for MuiTextField in the theme with the following props...
MuiTextField: {      defaultProps: {        inputProps: {          autoComplete: 'new-password',          form: {            type: 'submit',            autoComplete: 'off',          },        },        autoComplete: 'off',        size: 'small',        margin: 'dense',        variant: 'outlined',        fullWidth: true,      },    },

Current behavior 😯
Form will lose the onSubmit() event on Enter
Expected behavior 🤔
I expected to be able to submit my form after I hit enter.
Context 🔦
No response
Your environment 🌎
"react": "^18.2.0",
"@mui/material": "^5.9.3",

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://mui.com/r/issue-template), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


Playing albums and songs via albums doesn't work (same songs via playlists are fine)

🧐 Have you read the FAQ and viewed similar issues?

[x] I have read the FAQ
[X] I have checked older issues, open and closed
[X] I did a clean install of SpotX as described in the FAQ

Сountry of your account
United Kingdom
What is your Spotify Plan?
Free
Do you have Spicetify installed?
No
ℹ Computer information
- Spotify: <IP_ADDRESS>4.gf42f9f0a
- Windows: Windows 10 Pro (10.0.19045)
- PowerShell: 5.1.19041.2364

📝 Description

Fresh install of Spotify via SpotX
Click play on an album or song from an album: nothing happens and no song plays.
Click play on a playlist: songs play as usual, including any song on an album that doesn't play when attempted to play via playing the album or song on the album.
...

This has been happening for a while now, across different Spotify versions, and I have been 'getting around it' by just adding the albums I want to listen to at that moment to a playlist and then deleting the playlist afterwards lol. Strange thing is that it's inconsistent. Now and again I open Spotify and albums and songs played via albums work perfectly. I come back later, open up Spotify and it isn't working again.
Very weird!
📸 Screenshots
No response

+1

I could not reproduce this error, it seems like a temporary problem on the Spotify servers.
@Spottyfriend
what input parameters do you use to install spotx or did you use a ready installer ?

I have experienced this bug using Spotify installed using SpotX with no custom settings, and SpotX with updates turned off. And across a number of months. Though it is inconsistent, it seems to be the case more than it isn't.

specify which ready installer you used

I used to use an old one which didn't include custom settings (can't remember exactly which one it was sorry), but more recently I've been using "Usual installation (New theme)".

if you ran Usual installation, then you were offered 3 options for parameters, answer them in the same way as you did with that installation:

"Do you want to disable podcasts, episodes and audiobooks from the main page? [Y/N]"
"Want to block Spotify updates? [Y/N]"
"Do you want to install automatic audio cache cleaning for Spotify? [Y/N]"

Happened to me aswell not so long ago, with genuine Spotify and SpotX Spotify, and it looks like it was (or still) an issue with Spotify services, not "SpotX version". Unable to play songs on albums or artists, and even some artists pages had issue loading properly, while playlists were fine, but couple hours later it was fine (did nothing to my spotify installation, just waited).

@nomis51  hi, thanks for the feedback, what country is your account located in ?

@nomis51  hi, thanks for the feedback, what country is your account located in ?

Canada

Hi folks
any news about this issue? Maybe the problem has disappeared or vice versa, share fresh info

Happened to me aswell not so long ago, with genuine Spotify and SpotX Spotify, and it looks like it was (or still) an issue with Spotify services, not "SpotX version". Unable to play songs on albums or artists, and even some artists pages had issue loading properly, while playlists were fine, but couple hours later it was fine (did nothing to my spotify installation, just waited).

Doing some Googling this seems to be a recurrent problem for some people's genuine Spotify so I'm satisfied it's not to do with SpotX. Still happening for me irregularly.

@Spottyfriend @nomis51 i don't about that, but for only SpotX patched is having issue.
i've have tried both genuine spotify and spotx everyday and everytime I have a chance since this issue opened, only spotx having issue for me.
I have since found this crack version (v<IP_ADDRESS>5) on another site: https://crackshash.com/spotify-pc/; it should be in 43rd in the spotx official spotify exe file google forum list. this one works perfect for without any issue all day long.
I planned to installed every version between this one and the lastest version with spotx patch to see which works without any issue, haven't done much but I having same issue with 1.2.5.x's and 1.2.4.x's...will try the remaining within this week, and to see what happens.

it has been happening to me as well. has there been a fix yet ?

It appears to be related to the native Spotify client as complaints about a similar issue have started coming in on their forums, if you're experiencing something like this post it here.


unable to parse question

I'm trying to add simple facts then query the system, and it says "unable to parse question". Can you please provide a simple example how to work with it?
My simple example:
./executive_test console
read all humans are mortal.
rerank
read Socrates is a human.
rerank
research is Socrates mortal?

Sure, in order to parse custom inputs, you first need to make sure the parser's coverage includes those inputs. To do so, you need to make sure every word other than proper nouns is in at least on example in seed_training_set.txt. For your inputs, we just need to make sure "human" and "mortal" is in the seed training set. We can do so by adding the following:
:
{(N "human"/"human")}
    {human[index:sg]};

:
{(ADJ "mortal"/"mortal")}
    {?[x]:mortal(x)};

I can explain this notation in detail if you'd like.
But once you've added these entries, you can then run:
./executive_test concole
read All humans are mortal.
add 0
read Socrates is human.
add 0
read Is Socrates mortal?
answer 0 1000

Note that the parser is sensitive to case and punctuation, so don't forget to capitalize the first word of each sentence and the period at the ends.
Also the sentence "Is Socrates mortal?" is not covered by the current version of the grammar, since the grammar doesn't currently implement subject-auxiliary inversion or wh-movement.
For an example that is covered by the grammar, consider running:
./executive_test console
examples

Anyway, I hope this helps, and please let me know if you have any further questions.


README: Add test and coverage badges

Coverage remained the same at 100.0% when pulling f0dad2c4de8561b15d776aeceb6657889ff94a3c on badges into 1196a55849ba6d150352104e604f5bed332f48ab on master.