Hitting an error on auto-migrate

2023-01-09T05:02:47Z   [info]2023/01/09 05:02:47 /go/pkg/mod/github.com/davecheney/pub@v0.0.0-20230108095803-866e199b8b29/automigrate.go:19 Error 1170 (42000): BLOB/TEXT column 'name' used in key specification without a key length
2023-01-09T05:02:47Z   [info][1.356ms] [rows:0] CREATE TABLE `account_markers` (`id` int unsigned AUTO_INCREMENT,`created_at` datetime(3) NULL,`updated_at` datetime(3) NULL,`account_id` bigint unsigned NOT NULL NOT NULL,`name` longtext NOT NULL,`version` int NOT NULL NOT NULL,`last_read_id` bigint unsigned NOT NULL NOT NULL,PRIMARY KEY (`id`),UNIQUE INDEX `idx_account_name` (`account_id`,`name`),INDEX `idx_account_markers_account_id` (`account_id`),CONSTRAINT `fk_accounts_markers` FOREIGN KEY (`account_id`) REFERENCES `accounts`(`id`) ON DELETE CASCADE)

Hmm, interesting. Which version of mysql are you using?

Oh, that's interesting. https://github.com/davecheney/pub/blob/main/internal/models/account.go#L62 name is an enum.
I suggest

In mysql, drop the account_markers table; use pub; drop table account_markers
Probably try building from source, rather than using go install to make sure you're using the head of main.

Tried both using latest mariadb and building from source. Don't know which one helped but it now works! https://github.com/marksteve/fly-pub


read API credentials from .env file

Hi Walid R. Rashed,
I hope this message finds you well. I noticed that the clientId and clientSecret were hard-coded, which could potentially pose a security risk. To address this issue, I made some changes to the code by adding a constructor to read these values from a .env file instead.
Specifically, I did the following:

Imported the dotenv and fs modules
Added the dotenv.config() method to read the environment variables
Replaced the hard-coded clientId with process.env.CLIENT_ID
Read the clientSecret from the .env file using fs.readFileSync()

However, because I do not have an API client and client secret, I was unable to test these changes. Please let me know if you have any feedback or concerns about the updated code.
Thank you for your time and attention.
Best regards,
Hoshmand

Hi @hoshmandctf
Thank you for your contribution!
This approach of yours has some drawbacks, here is why:

Since this is an npm package, your commits take effect on the package level, not the application, by doing this, you'r forcing the developers to use the environment variables and not giving them any other options.
If we go with your approach of getting client_secret and cleint_id directly inside the constructor from the ENV, this will also be a development flexibility issue, because in multi-tenant applications architectures, the developers cant initialize different instances by using different client_id and client_secret

From the README file's documentation, you can see that initializing the Fib instance and authenticating it can be very flexible and developers can use Env files or any other approaches and keep their credentials safe, for example:

// you can get the cleint_secret and client_id values any way you want.
const clientId = process.env.CLEINT_ID
const clientSecret = process.env.CLEINT_SECRET

import { Fib } from 'fib-sdk';
const fib = new Fib( clientId, clientSecret);

So by understanding the above example, I believe this will cover all of your worries.
Regards,

Hi @kurdi-dev,
Thank you for your prompt response and for sharing your concerns about my proposed changes. I appreciate your attention to detail and your commitment to maintaining the flexibility and security of the package.
After considering your feedback, I understand that my approach of getting the client_id and client_secret directly inside the constructor from the environment variables could limit the flexibility of the package and cause issues in multi-tenant applications.
I agree that developers should have the freedom to choose how they want to manage their credentials and that the current approach of initializing the Fib instance and authenticating it using environment variables or any other method is more flexible and secure.
Best regards,
Hoshmand

Thank you for your understanding!
I will close this pr now.


[JetNews] List-detail support for main page

Adds an initial list-detail support for the main page.
The overall approach relies on distinguishing between what the user sees as different screens, and what we implement as different navigation routes.
Navigation-wise, the "home" route is responsible for drawing the list detail screen, as well as the list screen and detail screen on their own (for small devices). Distinguishing between which one to show is now simply a matter of keeping track of whether the user is looking at the list, or a selected article.
Doing so allows meeting a couple of nice behaviors:

We keep track of the user's state (scroll state on the list/detail, as well as which one they were looking at), even while changing which side we are looking at
We avoid navigating as a side-effect of a size change
We can make size changes reversible (mostly as a consequence of the previous point)

List:
Detail:
List and Detail:

@manuelvicnt I just pushed up another commit, which makes quite a few changes.
Most notably, I've removed the article route entirely (for now), so articles are now only displayed from the home route.
I also created a HomeRoute @Composable, which is the primary component responsible for determining which HomeScreen to display (based on the size of the screen, and the available state).

Also @alexvanyo, can you add screenshots to the description of the PR for reviewers to get a sense of what's been implemented? At some point, we'll also have those mocks in the README. Thanks!

Is there a reason that you included the migration from List<Post> to PostFeed in this PR? It's making it difficult to follow the changes related to that migration vs the actual changes made for the large screen support.

@JolandaVerhoef Ah I apologize, that ended up being a refactor that happened partway through while cleaning everything up. Ideally that should have been pulled out into a separate PR that didn't touch the UI at all.
That ended up being changed to more nicely support having the topmost article selected by default. Previously, the post list was expecting a hardcoded list that was pulled apart into different sections, so the 4th post was arbitrarily chosen to be the "highlighted item." With the refactor to PostsFeed, those sections are now specified by the data object, so we can nicely choose the "highlighted item" to be selected by default.

Thanks @manuelvicnt and @JolandaVerhoef !


System Integration

I dunno how to name the title,
These are requests, A KDE user's.

[ ] Please use the Title bar button's KDE's Breeze them.
[ ] Present the app's logo in a circular frame instead of Pure rectangle or rectangle with rounded corners.

This is still a needed issue. Bot, open.
Aug 15, 2024, by @.***:

Closed > #263 https://github.com/Sandakan/Nora/issues/263>  as completed.
—
Reply to this email directly, > view it on GitHub https://github.com/Sandakan/Nora/issues/263#event-13899600555> , or > unsubscribe https://github.com/notifications/unsubscribe-auth/AYJXNKCQJ6UOVWBYXTL7O4LZRTQDTAVCNFSM6AAAAABJA646NSVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTHA4TSNRQGA2TKNI> .
You are receiving this because you authored the thread.> Message ID: > <Sandakan/Nora/issue/263/issue_event/13899600555> @> github> .> com>


Please let us know the queries which it can do and it can't do

One thing which bugged me for so long with mongo is it does not gives a clear picture of things which it can do and can't do.
I would also be interested if i can find a page where for each api the road map of the features is written for instance i have no clue whether someone is working on group by feature in sql api or not.
Another example would be $$ROOT i have not find even a single page where i can say it's a feature missing from cosmos or it is available, for now if i use $$ROOT it does not do anything no issue nothing.

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

ID: 1534d9c1-c568-f744-7cec-07f107333a3a
Version Independent ID: 0c56ac88-b762-38a3-6b1c-d086597f0c5a
Content: Azure Cosmos DB: How to query using the MongoDB API?
Content Source: articles/cosmos-db/tutorial-query-mongodb.md
Service: cosmos-db
GitHub Login: @SnehaGunda
Microsoft Alias: sngun

@kamaljoshi911 Thank you very much for your feedback. Have you had a chance to look over the following documentation: MongoDB API support for MongoDB features and syntax
It does not appear that $$ROOT is supported. Does this provide the information you are seeking?
Regards,
Mike

@kamaljoshi911 We will now proceed to close this thread. If there are further questions regarding this matter, please open a new issue and we will gladly continue the discussion.


Suppress logging in cue/rego evaluations

Summary

This removes the opinionated logging from policy.EvaluatePolicyAgainstJSON to avoid clashing with any downstream consumers logging structure (such as policy-controller).
Since these messages aren't enriched with anything the caller doesn't already have, I don't think this is a very impactful change, but let me know if there's a place these should be moved to.
Release Note

Remove internal logging for cue/rego policy evaluations
Documentation

There are some linting issues.


Grouping - suppressCount with groupUseEntireRow set to true and Custom Template for Group Row

I started implementing a Grouping feature in one of my grid and start with very basic configuration which is:
    var columnDefs = [
           {
              headerName: "Name",
              field: "name",
              width: 100,
              filter: agGrid.ColumnQuickFilter
          },
          {
              headerName: "Profession",
              field: "occupation",
              width: 120,
              filter: agGrid.ColumnQuickFilter
          },
          {
              headerName: "Telephone",
              field: "phone",
              width: 120,
              filter: agGrid.ColumnQuickFilter
          },
          {
              headerName: "Email",
              field: "email",
              filter: agGrid.ColumnQuickFilter
          }
    ];

    var gridOptions = {
            columnDefs: columnDefs,
            enableSorting: true,
            enableFilter: true,
            enableColResize: true,
            angularCompileRows: true,
            angularCompileFilters: true,
            groupUseEntireRow: true,
            groupKeys: ['matchedOn']
        }

The grouping works perfectly fine, but I want to suppressCount that appears right after the Group. How it would be done by having groupUseEntireRow: true ?
Also, Is it possible to use the custom template for Group row? Like I want to add a menu icon along with Group Name  which will be a dropdown of having couple of action buttons

How it would be done by having groupUseEntireRow: true

you would need to provide your own inner renderer for that.

Also, Is it possible to use the custom template for Group row? Like I want to add a menu icon along with Group Name which will be a dropdown of having couple of action buttons

same as above actually, if you provide your own group cell renderer, then you have total freedom as to what goes in.
there is an example of a custom cell renderer on the grouping documentation page.
PS please use the board for questions, i want to keep the github issues for issues with the grid, it items i need to work on! thanks!

OK. Is there anyway I can compile the DOM that I wrote inside groupInnerRenderer?
var gridOptions = {
            columnDefs: columnDefs,
            enableSorting: true,
            enableFilter: true,
            enableColResize: true,
            angularCompileRows: true,
            angularCompileFilters: true,
            groupUseEntireRow: true,
            groupKeys: ['matchedOn'],
            groupDefaultExpanded: true,
            groupInnerRenderer: groupInnerRenderer
        }

        function groupInnerRenderer(params) {
            return '    ' + params.node.key + '';
        }

do you have angularCompileRows=true set in your grid options?

Yes, Please review the GridOptions.

Also, I am wondering we can change the background color for a specific Group inside innerRenederer.

I have a scenario in which I have to change the background color of Group to Red. As the DOM is not ready before the groupInnerRenderer function execute, I setup a logic like this
function groupInnerRenderer(params) {
            setTimeout(function () {

                if (params.node.children[0].data.isInDiscussion) {

                    angular.element(".ag-body-container div.ag-row-group[row=" + params.rowIndex + "] span.ag-group-cell-entire-row").css('background-color', 'red');
                } else {
                    angular.element(".ag-body-container div.ag-row-group[row=" + params.rowIndex + "] span.ag-group-cell-entire-row").css('background-color', '#aaa')
                }
            }, 500);

            return '    ' + params.node.key + '';
        }

didn't I ask you to raise question about the grid on the board and not github issues????
i feel if i keep answering, you are just going to keep using the github issues so am gong to stop! :)

I am sorry, but I thought as this issues have already been closed and I was in the same context so still continue here. Again sorry, I will open this question on Board.


Add config spec

What does this PR do?

Add config spec for MySQL
Motivation

Config specs everywhere! Also I might need to add a charset option for a support case, so having the config spec done beforehand would be nice.
Additional Notes

Not super happy with optional sections such as ssl, custom_queries or queries showing up as un-commented in the generated conf.yaml.example - but we can look at tweaking ddev later.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

/azp run

Can you bump checks base on the setup to 11.0.0? (to include service features)


Add ability to adjust resolution

Please add ability to adjust "resolution" of the output image. I don't necessary mean the output size of the brick, more that the image size of the output changes to include less or more detail of the original.

Could you explain a bit more? like how lego brick size and amount of bricks would work? Because if I were adding more detail, the bricks would need to become smaller, or you're going to get some huge ram usage from this. At some point when the bricks become smaller and smaller, they don't look that great

I agree, that would be very useful. I converted a 200 MB image of Starry Night by Van Gogh and it's barely recognizable. I converted a painting I actually own and I didn't recognize it.
For a work event the graphic designers actually built legos into famous works and photographed those and ended up with great images. They look a lot like these, but with far more brick pixels - on the order of 80 wide and 60 tall for this painting. I can't share the example with you (because of licensing) but can confirm getting up to ~80 pixels per side makes a really gorgeous image.

OK, once I implement it, the bricks will not be that good of quality, but later it might

I think we are talking about two separate options here.
First, at least because it's more simple to describe (!), an option to set the brick size. ie. change the definition of what our "pixel" is. The resulting images would be a multiple of this brick size by definition.
Second, an option to set the "scaling" or "resolution" - whilst my source image might be 600x600, I want the final image size to be much larger in order to "capture" more details of the original. Putting it another way, "legofying" an image is obviously information-destructive, and this would be the setting that controls how much information is in the final image. I believe this is what @carschar was referring to when he made Lego out of Van Gogh.
Perhaps we don't need the brick size option - as you imply, it would not necessarily result in a good image as the brick PNG is somewhat "pixellated" and these images never look good when resized from their designed dimensions.. I mostly just mention it here for completeness and to ensure we are are on the same page.

Ok, working on this, fixed ram usage. this 7680x7680 (baseImage: 256x256) image was made using a consistant 235MB of ram. Usually it would have increased every time a lego block was drawn, but I fixed that. Expect some updates soon.

That is really cool! :+1:

forgot to close this, 25f5c88dd84ce8b3405d306063c6fe46aa52a585

beautiful


Rename WorldMarginShape to WorldBoundaryShape

Following up on https://github.com/godotengine/godot/pull/52265#issuecomment-912969778
2D line shape and 3D plane shape are now replaced with world boundary shape in nodes and physics servers.
Supersedes #52265.

Ok, let's merge then since it's just renaming and we already have a consensus.


Docs say TGENV_AUTO_INSTALL is true by default but it's actually false

https://github.com/cunymatthieu/tgenv/blob/dd720f62051bb748f75eee2d83d6768171d45830/libexec/tgenv-exec#L31
Can you make up your mind. thanks.

Hey @oofdog. We've forked this project as tgenv/tgenv and a fix for this issue has been merged as https://github.com/tgenv/tgenv/pull/9

Thanks @iokiwi


Also import/keep notes from ABOUT files in a Package

We should also import and keep notes from ABOUT files in a Package

Added in https://github.com/nexB/scancode.io/pull/1349


It won't maximize.

I was working just fine till someday, when I execute all I get is the tray icon and when I double click it, it shows only the title bar with the minimaze, maximize and close buttons besides the java icon. I tried extract it again, download it again, update my java but nothing worked. Also I tried to active the "clipboard autorip" function you can access through the tray icon menu to see if it would save anything on the last destination folder I set but still nothing. Heres what it looks like:

It was running just fine till some time ago, the last log entrance was:
2016-02-09 20:54:30,184 DEBUG ripper.AbstractRipper .setWorkingDir() Set working directory to: C:\Users\....(just the folder from the last rip)

Looks like an issue with Java, I'd say.
If nothing else works, try a clean installation of the Java JRE.

Try:

Exit the program
move the rip.properties file to another name (e.g. rip2.properties)
Restart the program

If that fixes it please post the rip.properties file to this issue. People have had this problem before and I'd like to detect it within the program during startup to avoid it in the future.
If that doesn't work, try running it from a different folder.

#371

Could be related to https://github.com/4pr0n/ripme/issues/166


Pocket Article View API

I hope to get access to Pocket Article View API. After that i will implement it as well into my API.

Nevermind, I figured out, how to get access.


fix translations

some translations are missing, e.g.

In the account page it shouldnt say that language settings are coming soon

In the database the user actually has a preference for languages.

This preference should be used in the frontend. When the user has the language preference "DE" the frontend should be in german. If the user clicks on the button to change languages in the frontend, the preference in the database should be adjusted accordingly.

also, pls remove the underscores here


EmbedArt Plugin: remove_art_file doesn't seem to work

I'm running beets version 1.15. The EmbedArt plugin isn't removing the art file from the file system.
Logfile: http://pastebin.com/n10bbdpS
Config: http://pastebin.com/ztrjd16C

+1
Config: http://pastebin.com/bqTDtUg6
Exemplary Log (via beet -vv import): http://pastebin.com/DxGabj0C

have you tried with maxwidth commented out?  I know there was a time embedart was failing for me because I didnt install imagemagik or pils, not sure if it relates though

Thanks for the reports! It looks like @jackwilsdon only added the functionality for an explicit beet embedart command in #1591. Also covering the automatic import hook would be a good idea.

great, thanks! :)


Tags refactoring

Tags refactoring
Currently, tags are hardcoded and it lacks flexibility. We had many requests around many subjects, and this draft aims to support them.
General principle
Tags should be categorized
We have several categories of tags and we should keep them:

Drivers (VL, VPSP)
First Aiders (PSC1, PSE1, ...)
And some tags that are not categorized

Tags should be hierarchized
In a single category, we should keep the tags hierarchy in order to reduce the list of tags we show to users.

If a volunteer can ride an ambulance (VPSP), he logically can ride a car (VL)
Thus, if a volunteer has both VL and VPSP skills, we'll only render "VPSP" in views

Tags should be togglable
We will integrate the hundreds of tags, but don't want to have huge forms to tick tags or not when filtering the audience to trigger.
Thus, tags should keep a "visible" state, togglable in the administration interface. It will be their default value in the application.
And then, structures should be able to toggle the tags for their scope only, to make them visible and usable both in forms and views.
Tags should be customizable
Finally, structures should be able to create custom sets of people they are used to trigger.
These tags are not available / shareable with any other structures nor rendered in the tags management.
Related tickets

#373 Integrate TCAU skill
#289 Création de déclenchement: audience par nomination pegass pour les Multi Structure (DT...)
#248 New Feature: Triggers at national level with specific group of people
#238 Ability to create custom list of volunteers

#491

#491


Format integration documentation pages via markdownlint

PR Checklist

[x] A description of the changes is added to the description of this PR.
[x] If there is a related issue, make sure it is linked to this PR.
[x] If you've fixed a bug or added code that should be tested, add tests!
[x] If you've added or modified a feature, documentation in docs is updated

Description of changes
Update the formatting of the integrations documentation pages by using markdownlint. Updated the custom markdownlint configuration file with rules as necessary to comply with MkDocs. User-facing changes have been kept minimal.
Resolves #655

Thanks @HorizonNet - will review this ASAP!


If top info container is closed, the share button overlaps with the reopen box icon

get's fixed in #238


Fix for python3.11?

Is any release for python3.11 planned?
It is currently no possible to directly install this package under python3.11, and even from sources without diving into the code.
The big package scikit-fda depends on it, and so accnot be installed on python3.11 :(
Thank you in advance.

https://github.com/SheffieldML/GPy/pull/1020 fixes, closing. As the fix is merged, it'll be part of 1.13.0 when released. Please re-open the issue if you run into the problem again after the release.

Thanks!
Tested installation from latest devel rev, it works fine for me :)


Update cspell and textlint config

Move textlint config out of package.json
Add more words to cspell list, a lot of names of authors that have published multiple times on the blog.
@chalin PTAL

LGTM
As for blog author names, as a general rule, we should have an ignore rule in the blog post itself. Otherwise over time we'll end up adding every single blog author name to the global list. My 2cents, though I don't feel strongly about it.
As for author names, IMHO, we should leave the ignore rule in the blog post.

I don't want to have all authors in that list, I only picked names of people that show up multiple times because the have created multiple blog posts already, or are mentioned within the text e.g. as candidates or elected members of the GC or speakers at a conference. But if the overall tendency is that we should not do that I can revert the changes.

@svrnm - your call.

as discussed yesterday during the SIG call with @chalin, I will merge this in: we can go backwards if we want anyways.
I will also write down some updates to the contributors guidelines that updates to that cspell file should not be made for individuals that write a blog post once but for names that came up (and will come up) again and again


Add Explain screen

Added a screen to visually display score explanations and be able to drill down into explanations

By the way, I couldn't make my unit test work yet...
If you're interested, feel free to comment, and tell me how to improve.

:+1:

Unit tests are OK now.
At the beginning, I made a specific page/route for this "explain" feature but in the end it was very similar to the Rest Client page. So I remove this page and added an Explain button on the Rest Client page.

@gquintana hey, thanks for the PR :)
Kind of busy with other issues at the moment, will try to take a look at the PR this next week.


Form filter not PHP 8.1 compatibille

Hi, seems that Filetrs have problems with named parameters, therefore it's not working for PHP 8.1.
Definition in form::init() (example):
// phone         $this->addElement('text','customerPhone', array (                                                 'label'=> 'Phone',                                                 'required' => TRUE,                                                 'filters'   => Array(                                                                 'StringTrim',                                                                 ['PregReplace', ['match' => '/\s+/', 'replace' => ''] ] // remove white spaces                                                 ),                                                 'validators' => Array(                                                                     Array('StringLength', false, array(2))                                                                 )                                             )         );
Error:
Unknown named parameter $match
Any idea how this should be solved? Thank you.
Ivan

Seems that
['match' => '/\s+/', 'replace' => '']
..is going into Filter constructor and that's where issue pops up.

Did more research, problem is related to the
Zend/Form/Element, line 2119
There is code
$instance = $r->newInstanceArgs((array) $filter['options']);
If it's single array, it's decoupled into named args and therefore it fails.
So workarrout is to wrap it inside another array in the Zend_Form::init() definition.
Not sure if we can solve this on the ZF1 level. Seems to me like PHP8 bug. Or undocummented feature of ReflectionClass::newInstanceArgs()

As I wrote in #246 (since I didn't read this issue here, shame on me), this bug was reintroduced with the current release 1.21.3, and I am not sure if reverting back to the behaviour of 1.21.2 causes any problems somewhere.

I did exactly the same change, but in the INIT(), not in the ZF lib and running currently under 7.4 to test. Seems to be working OK.

After reading the reason for the change back to the original ZF1 version in https://github.com/Shardj/zf1-future/pull/186#issuecomment-1060931720, I think that we have to deal with passing the options as individual arguments instead of as an array.
I propose the following solution instead (untested, but I think it should work): pass the options through array_values(), i.e. $r->newInstanceArgs(array_values((array) $filter['options']));. Since this passes the arguments "normally", and not named, the error should go away. For pre-PHP 8 code it does not make any difference, since the option names were irrelevant anyways, but the order mattered. For new PHP 8 code it could have an impact, but I doubt that this feature is currently in use in ZF1 projects (at least at this point).
The only disadvantage is that this solution removes the possibility to use named arguments in this case in the future, but I think that that is not really a problem.
I can test that on monday and create a PR if it works.

So, I created a PR (but assigned it to the other issue, #246, since I had the tab still open... sorry for that).
The solution should be a viable way to go, but with the named drawback of not supporting named parameters there.
It might be better to rewrite the use of the filters in each project to pass the options really as array (so the options have to be wrapped in another array), since then the order of the arguments is irrelevant, as the use of the keys already suggests.

Merged https://github.com/Shardj/zf1-future/pull/249


Apple Silicon - Brew bundle for custom formulae skipped without bottle

Hello
I have custom formulae for our dev team to use here , we have to upgrade old PHP systems sometime so need to maintain functioning installs for older versions beyond what are provided in the core formulae.
I have got them installed and working okay
$<EMAIL_ADDRESS>--version
PHP 7.4.22 (cli) (built: Aug 10 2021 10:10:30) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies

But the brew bundle command is not starting the service properly because it is being skipped, this would also prevent the installation of the module if I had not manually installed it
Here is the example brewfile
$ cat Brewfile
tap "ampersandhq/php"
brew<EMAIL_ADDRESS>restart_service: true

Output of brew bundle to see the issue
$ brew bundle -vvv
Skipping install of ampersandhq/php tap. It is already installed.
Using ampersandhq/php
Skipping<EMAIL_ADDRESS>(no bottle for Apple Silicon)
Homebrew Bundle complete! 1 Brewfile dependency now installed.

Looking at this commit message here https://github.com/Homebrew/homebrew-bundle/commit/ab36dc9d1dbcca401e7ff9df1ee5baaa6ba28cf5 it seemed to suggest the intention was only for this to happen for the core homebrew formulae, but it is happening on our tap.
Am I missing something here? I was always able to put them in like this and just wait for the compilation to complete (which is now much faster on the arm64 chipset)

Thanks @MikeMcQuaid 🍻


With some searches i get a php Fatal Error

I think it happens when in the results i have a configurable product.
In the result page i have:
Fatal error:
Call to a member function getId() on a non-object in
/home/mysite/public_html/magento/app/code/core/Mage/Catalog/Model/Product/Type/Configurable/Price.php
on line 85

I have Magento <IP_ADDRESS>, solr 5.1.0 and lucene-impl 5.1.0, php 5.5.6

It was a problem in Magento of attribute set for configurable products, not related to magsolr. I have resolved!


'less' command is featureless

Need to update the command less, paging does not work. I was able to get it to work with:
apk --update add less
In this case, less is not more :-/

/move home-assistant/hassio-addons


Over resource limits on Streamlit Cloud

Hey there :wave: Just wanted to let you know that your app on Streamlit Cloud deployed from this repo has gone over its resource limits. Access to the app is temporarily limited. Visit the app to see more details and possible solutions.

Hi @alessandroCiciarelli,
Thanks for the notification! The app is live again, so I'll close this.


Mismatch between local and hosted Bot API servers can corrupt files when sending

General
I observe a mismatch between the hosted Bot API on https://api.telegram.org and the one I am running locally.
More precisely, some files get corrupted when they are sent via the hosted Bot API server. They can be sent just fine using a local Bot API server.
Much like in #178, I tried out a variety of combinations to see how exactly the issue can be reproduced. In particular, I tried those combinations:

sending the request to the Bot API server vs sending the same request to dummy server which only stores the posted file
contacting the Bot API server at https://api.telegram.org vs http://localhost:8081 (v6.0) vs http://localhost:8081 (v6.0.1)
sending a small PDF file vs sending a small text file
using curl vs grammY on Deno vs grammY on Node.js

The results are:

occurs only with the Bot API server
occurs only on https://api.telegram.org
occurs only with the PDF file
occurs only when using grammY on Deno

Note that grammY uses almost identical code on both runtimes. The main difference are
In order to make reproduction of this feasible, I wrote a bash script. It will:

Create source files for both clients
Install grammY from npm
Run both curl, grammY on Deno, and grammY on Node.js.

Reproduction
Setup
Install Deno and Node.js.
Create a file /tmp/reproduction.sh with the following content.
#!/bin/bash

set -e

HOST=$1
if [[ -z $HOST ]]; then
    echo "Pass the hostname as first argument, e.g. https://api.telegram.org or http://<IP_ADDRESS>:8081"
    exit 1
fi
echo "Will use $HOST as hostname"

BOT_TOKEN=$2
if [[ -z $BOT_TOKEN ]]; then
    echo "Pass the bot token as second argument"
    exit 1
fi
echo "Will use $BOT_TOKEN as bot token"

CHAT_ID=$3
if [[ -z $CHAT_ID ]]; then
    echo "Pass the chat identifier as third argument"
    exit 2
fi
echo "Will use $CHAT_ID as chat identifier"

FILE=$4
if [[ -z $FILE ]]; then
    echo "Pass the file path as fourth argument"
    exit 3
fi
echo "Will send $FILE using curl, grammY (Deno), and grammY (Node.js)"

echo "Creating Deno file at ./bot.deno.ts"
cat >bot.deno.ts <<EOF
import { Api, InputFile } from<EMAIL_ADDRESS>const api = new Api('$BOT_TOKEN', { apiRoot: '$HOST' })
await api.sendDocument($CHAT_ID, new InputFile('$FILE', 'deno-$FILE'), { caption: 'Sent from Deno' })
EOF

echo "Creating Node.js file at ./bot.node.js"
cat >bot.node.js <<EOF
const { Api, InputFile } = require('grammy')
const api = new Api('$BOT_TOKEN', { apiRoot: '$HOST' })
void api.sendDocument($CHAT_ID, new InputFile('$FILE', 'node-$FILE'), { caption: 'Sent from Node.js' })
EOF

echo "Installing grammY for Deno"
deno cache bot.deno.ts >/dev/null

echo "Installing grammY for Node.js"
npm install grammy

echo
echo "    +++ Performing file sending test +++"

echo "Step 1: curl ..."
curl -s "$HOST/bot$BOT_TOKEN/sendDocument?chat_id=$CHAT_ID&caption=Sent%20from%20curl" -F "document=@$FILE;filename=curl-$FILE"
echo

echo "Step 2: Deno ..."
deno run --allow-env --allow-net --allow-read=$FILE bot.deno.ts

echo "Step 3: Node.js ..."
node bot.node.js

echo
echo "Test complete."

The created file expects four arguments:

The hostname including protocol and port, e.g. http://<IP_ADDRESS>:8081
A Telegram bot token
A chat identifier to send the files to
A file path to the PDF file which you can download from here (md5sum: cd2dd0ead15d0376f3966fd8a77b9a3e)

In order to verify that the files are posted correctly with all three clients, we can set up a dummy server. Its sole purpose is to accept arbitrary POST requests (multipart/form-data) and to store the first contained file on disk. I used this short Deno script:
// server.ts
import { Application } from<EMAIL_ADDRESS>
const app = new Application();

let i = 0;

app.use(async (ctx) => {
    console.log("req");
    const body = ctx.request.body();
    if (body.type === "form-data") {
        const r = await body.value.read();
        const file = r.files?.[0].filename;
        if (file !== undefined) {
            const out = `/tmp/out-${i++}.dat`;
            await Deno.rename(file, out);
            console.log("wrote to", out);
        } else console.log("no file contained");
    }
    ctx.response.body = JSON.stringify({ ok: true });
});

console.log("listening on port 8000");
await app.listen({ port: 8000 });

Execution
Start a local Bot API server.
Start the dummy server using deno run -A server.ts.
Execute the shell script using various parameters.

Hosted Bot API server (reproduction): ./reproduction.sh https://api.telegram.org <token> <chat_id> file.pdf
Local Bot API server (works): ./reproduction.sh http://<IP_ADDRESS>:8081 <token> <chat_id> file.pdf
Dummy server (works): ./reproduction.sh http://<IP_ADDRESS>:8000 <token> <chat_id> file.pdf
A text file on a hosted server (works): ./reproduction.sh https://api.telegram.org <token> <chat_id> file.txt

In each case, the script will send three files to the specified chat.
Download the files. Compare their MD5 sums.
$ # Here, I first sent the files locally and then via the hosted server, so the (2) files are the hosted ones.
$ md5sum *
cd2dd0ead15d0376f3966fd8a77b9a3e  curl-file (2).pdf # curl on hosted
cd2dd0ead15d0376f3966fd8a77b9a3e  curl-file.pdf # curl on local
5ac08256951d8c9cfb55ad9b2ebcaceb  deno-file (2).pdf # Deno on hosted
cd2dd0ead15d0376f3966fd8a77b9a3e  deno-file.pdf # Deno on local
cd2dd0ead15d0376f3966fd8a77b9a3e  node-file (2).pdf # Node on hosted
cd2dd0ead15d0376f3966fd8a77b9a3e  node-file.pdf # Node on local

As you can see, the file sent using Deno via the hosted Bot API server has changed.
When sending the files to the dummy server, all checksums stay unchanged.
There is a potential that this actually is an issue on my side, perhaps some oversight. Please let me know if you can reproduce this, or if I should provide further information.
/cc @dcdunkan @trgwii

It seems like this is a bug in the Deno runtime. You can ignore this issue for now. I'll close it as soon as I can confirm that this is not a problem with the Bot API server.

This is indeed a bug in the underlying runtime, so I'll close this. For reference, I have reported the bug here.


feat(module:slider): support nzTooltipVisible

close #2373
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #2373
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@vthinkxie Fixed. Please review again.


Typo in https://github.com/sudoplatform-labs/ffi-tutorials/blob/main/Quick_FFI_Intro/readme.md#building-the-sample-from-github

In Step 2, the following libraries don't exist after the cargo build ...perhaps they are misnamed?

The created libraries (libcharcount.a and libcharcount.dylib) will be created in ./target/debug

In Step 1, it would be nice to caution that developer to ignore the warning (assuming they are brand new to Rust):

warning: unused import: std::ffi::CString
--> src\lib.rs:3:5
|
3 | use std::ffi::CString;
|     ^^^^^^^^^^^^^^^^^
|
= note: #[warn(unused_imports)] on by default

Fixed


Plugin ui support

Hi oznu,
hi all,
at first I'd like to say a big thanks for the hard work.
Second I'd like to relate to https://github.com/normen/homebridge-platform-maxcube/issues/8 and ask whether it is planed to implement a plugin ui support.
So when config-x is installed another homebridge plugin could display some additional stuff there.

Hi @marck1991,
Plugins can define their own UI settings by publishing a config.schema.json.
See https://github.com/homebridge/Documentation/blob/master/Schema.md for details.


🐛 Uplift docker/distribution to v2.8.0

github.com/docker/distribution suffers <2.8.0 suffers from https://github.com/advisories/GHSA-qq97-vm5h-rrhg

/test-v1b1-integration

/lgtm

/cancel lgtm

/approve

/lgtm


Add linter for pycrypto use

The pycrypto library is no longer maintained, and has known CVEs. We should detect its usage altogether and encourage usage of cryptography, or at least the maintained fork pycryptodome.
We should be careful not to detect that pycryptodome is being used since they can have the same API, e.g. Crypto vs. Cryptodome. BadModuleUseLinter would be a good start for this, but we may need something more to detect pycrypto vs. pycryptodome.

Closed by 6b41f8c6f975eadc795551a9884a89c3dcb6c898.


Update license from deprecated LGPL-3.0 to MIT

Needs to be reflected in LICENSE file, package.json and a badge in readme.md.

Covered by #8


Added test cases for Service Discovery use cases

@alena1108 @will-chan , can you please review?

LGTM

LGTM

@alena1108 @will-chan I have incorporated the review comments.


Missing Crucial Information To Send JSON Data With CapacitorHttp

URL
Capacitor Official Plugins HTTP
What is missing or inaccurate about the content on this page?
I only found the issue at Android platform. The API server does not receive JSON data if you try to request it without explicity send header Content-Type application/json.
I have read the Java base code and it actually skips the process of writing data if you try to request data without Content-Type header.
If you want to send data JSON with Android platform, you need to add header Content-Type with application/json like this.
const ret = await CapacitorHttp.post({
    ...
    data: {
        message: 'Hello World!'
    },
    headers: {
        "Content-Type": "application/json"
    },
    ...
});

For other type of data, if I had time, I would try one by one.

This is a bug in the plugin, not in the docs
https://github.com/ionic-team/capacitor/issues/7245


One month delay from Simple Calendar to Small Time

Hi,
When I used both these modules, all is great except the month displayed in Small Time isn't the good one.

Looks like your Simple Calendar isn't quite in sync with About Time (which is what provides the date to SmallTime). Try hitting the Configure button there, and click the Export to About Time button, and make sure your sync type is set to "Mixed".

Thanks for your response. I already done that multiple time, before creating this issue. I'm going to make another tests.

Hey, we're running into this same issue.
Did you ever find a fix?

No, I didn't. If you find one, I'm interested because it's annoying 😉.

Hmm. I still can't get this behaviour to happen on my end. Can either of you determine a set of steps to reproduce this offset?

Interesting. I'll throw a ping at Tim, see what he thinks.

It could be some timezone weirdness.
We're using non-american date, so that's DAY/MONTH/YEAR.
Could that be the source of our woes?

I think this might be falling afoul of the new about time offset field - which is meant for non-standard world times. Make sure the about-time setting is blank.

There was also a bug in Simple Calendar that the author just fixed yesterday. It caused about a month discrepancy between Simple Calendar and About-Time (which would show up in SmallTime since it's just displaying About-Time's time).

Thanks for the help tracking this one down, everyone. Think we'll call it closed at this point.


AppRTC like RTCMultiConnection does not work in firefox

Hello since the last update of firefox, this demo is not working
https://www.webrtc-experiment.com/RTCMultiConnection/AppRTC-Look.html
In Google Chrome it works without problems.
Please, you can solve this problem and update the RTCMultiConnection.js
Thank you

Yes . Current version 2 not working on fire fox please fix thank you

Not Working with Firefox .

Same here.. please look into the issue


Feature wcm lis

Description
Replace this text with a concise description of what was changed and why.

Testcase

accepted pull request for Sara's WCM implementation into intermediate feature branch


Fix MinionPoller when no nodes have been observed

Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

Good catch!
Maybe we can add a test here. Other than that 👍 .

Sure thing. I'll add a test.

@mssola can you rebase? let's get it merged

@mssola can you rebase this, so we can merge it if it's ok?

Closing in favor #136.


libuv: 1.41.0 -> 1.41.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libuv/versions.
meta.description for libuv is: "A multi-platform support library with a focus on asynchronous I/O"
meta.homepage for libuv is: "https://libuv.org/"
meta.changelog for libuv is: "https://github.com/libuv/libuv/blob/v1.41.1/ChangeLog"
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.41.1 with grep in /nix/store/vx9582hlc0msh7rhrcknjvmz4rm1icvr-libuv-1.41.1

found 1.41.1 in filename of file in /nix/store/vx9582hlc0msh7rhrcknjvmz4rm1icvr-libuv-1.41.1

directory tree listing: https://gist.github.com/1a36954f76ee69dbce1a57e941756c5b

du listing: https://gist.github.com/a2ebcbe56464d35d425f4f82fdd0b9ee

Rebuild report (if merged into master) (click to expand)

88732 total rebuild path(s)

25243 package rebuild(s)

25100 x86_64-linux rebuild(s)
23250 i686-linux rebuild(s)
17376 x86_64-darwin rebuild(s)
23005 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath

AusweisApp2
CHOWTapeModel
DisnixWebService
EBTKS
EmptyEpsilon
Fabric
LAStools
LASzip
LASzip2
Literate
MACS2
MIDIVisualizer
MMA
NSPlist
OSCAR
OVMF
OVMF-CSM
OVMF-secureBoot
PlistCpp
QuadProgpp
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer
SDL2_net
SDL2_ttf
SDL_Pango
SDL_gfx
SDL_gpu
SDL_image
SDL_mixer
SDL_net
SDL_sound
SDL_stretch
SDL_ttf
SPAdes
SkypeExport
Sylk
Xaw3d
_0x0
_1oom
_1password-gui
_20kly
_389-ds-base
_3mux
_4ti2
_90secondportraits

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vx9582hlc0msh7rhrcknjvmz4rm1icvr-libuv-1.41.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libuv https://github.com/r-ryantm/nixpkgs/archive/17bd52d02452889d03e55594cc56e6e71541104b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vx9582hlc0msh7rhrcknjvmz4rm1icvr-libuv-1.41.1
ls -la /nix/store/vx9582hlc0msh7rhrcknjvmz4rm1icvr-libuv-1.41.1/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @cstrahan for testing.

@GrahamcOfBorg eval

@GrahamcOfBorg build nodejs neovim


Suggestion: Default values in functions

Suggestion for Go v2.x - Default values in functions (not like in C++).
Like so...
func Example(value1 int, value2 string = "default", value3 float64 = 3.14) {
...
}
Function calls...

Example(1, "string", 1.11)
value1 = 1
value2 = "string"
value3 = 1.11

Example(1, _, 1.11)
value1 = 1
value2 = "default"
value3 = 1.11

Example(1, "string", _)
value1 = 1
value2 = "string"
value3 = 3.14

Example(1, _, _)
value1 = 1
value2 = "default"
value3 = 3.14

Example(_, _, _)
Compile error (value1 must be set).

I think this is quite the "Go-way".
Thx for attention.

Thnx, but it is not the same!
In #21909 - "not Go-way", because are allowed skipping values when calling function.
Skipping values must be indicated explicitly.

robpike talks here about default arguments https://github.com/golang/go/issues/21909#issuecomment-330443402 and references this:

One feature missing from Go is that it does not support default function arguments. This was a deliberate simplification. Experience tells us that defaulted arguments make it too easy to patch over API design flaws by adding more arguments, resulting in too many arguments with interactions that are difficult to disentangle or even understand. The lack of default arguments requires more functions or methods to be defined, as one function cannot hold the entire interface, but that leads to a clearer API that is easier to understand. Those functions all need separate names, too, which makes it clear which combinations exist, as well as encouraging more thought about naming, a critical aspect of clarity and readability.

I don't see how using explicit skips gets around this underlying point that rob is making.
Separately, imo, one of the big benefits of default values in functions is that it makes the calling code more readable to humans. The reader sees fewer arguments and has less information to think about. In your proposal, the reader will see _ and will be distracted and want to know what defaults are being used instead of just trusting the library.

I have already read that robpike stands primarily for a clear code definition, simplicity and understanding of the code.
In my opinion, the omission of arguments in the form of "_" fits into this concept.
I do not want to argue and firmly defend this idea, this is just a discussion.
There is a request from the community for such functionality.
It seems to me sooner or later it will take place.

func Example(value1 int, value2 string = "default", value3 float64 = 3.14) {
...
}
var f = Example

What is the type of f?  Can you call it with _ as one of the arguments?
It seems to me that default arguments just don't mesh very well with Go's type system.

I do not like this proposal, because the result code is not easy to read.
If you have too many parameters of a function, struct should be a better option than passing a lot of parameters one by one when you consider the read of the code.
Better proposal https://github.com/golang/go/issues/12854:

I can use default value parameter of function right now.

type ExampleReq struct{
   value1 int
   value2 string
   value3 float64
}
func Example(req ExampleReq) {
   if req.value2==""{
     req.value2 = "default"
   }
}

So I like the untyped composite literals proposal https://github.com/golang/go/issues/12854 which can save some sytax from the caller.

I do not like this proposal, because the result code is not easy to read.
If you have too many parameters of a function, struct should be a better option than passing a lot of parameters one by one when you consider the read of the code.
Better proposal #12854...

An example of bronze1man has the right to exist.
But if you bring the example closer to a more practical application, it will look something like this ...
type ExampleReq struct{    value1 int    value2 string    value3 float64 } func Example(req *ExampleReq) {    if req == nil {       return    }    if req.value2 == "" {      req.value2 = "default"    }    if req.value3 == 0 {      req.value3 = 3.14    } ... }
Total: 10 lines of code instead of just 1 line.
Also, with this approach, which now has to be applied in one form or another, the behavior of the less function arguments is transparent and more difficult to understand than the definition of default values in the function declaration.
This is my opinion, of course.

@Biter777
I admit this req struct way to improvement code readability is not so elegant or tidy. But I tried serval ways, and this is the best way to fit in current golang sytanx.
Use pointer of struct in the function request will make the code more difficult to read and an alloc per function call. I have tried use it, but now i use struct of request everywhere.

remain equal to 0??? problem:
I always try to make the zero value as not pass in value, so I solve it at the data struct design stage.

type ExampleReq struct{
   value1 int
   value2 string
   value3 float64
   value3Zero bool
}
func Example(req ExampleReq) {
   if req.value2 == "" {
     req.value2 = "default"
   }
   if req.value3 == 0 &&req.value3Zero ==false {
     req.value3 = 3.14
   }
...
}

@bronze1man ok...
And if i want to use req.value2 = "" I need to add a "value2Zero bool" (or better value2Default) in stuct?!

This is all not good. Or just one me it seems so?

@Biter777
Yes ,this function req struct way is not good.
But I think function req struct way has better code readability than Example(1, _, _) , if you have 10 parameters to pass in a function, function req struct way should be better code readability.  Example(1, _, _) may be better when number the function parameters is less then 3.
As an example, please see the windows api CreateProcessW (c syntax) https://stackoverflow.com/a/42544/1586797 https://docs.microsoft.com/en-us/windows/desktop/api/processthreadsapi/nf-processthreadsapi-createprocessw .
If the golang syntax can be changed, I think something like following may be better than both ways:
func Example(value1 int,value2 string,value3 float64,value3Zero bool) {
   if req.value2 == "" {
     req.value2 = "default"
   }
   if req.value3 == 0 &&req.value3Zero ==false {
     req.value3 = 3.14
   }
...
}

func main(){
   Example(value1: 0);
   Example(value2: "string");
   Example(value3: 0, value3Zero: true);
}

In stead of having default parameters, just make new wrapper functions that fill in the defaults in various reasonable ways. This is the Go way.
Just look at the os package. Unlike C where you have a single fopen(), in Go we have func Create(name string) (*File, error),  func Open(name string) (*File, error) and func OpenFile(name string, flag int, perm FileMode) (*File, error). The former two are wrapper functions around the latter where perm anf flag are filled in with reasonable defaults. The resulting API is far easier to use.

@beoran, yes is a way out of situation.
But for our Example(...) func we must write a 4 wrappers instead of 1.
And if value1 also takes on the default value, then already 8 wrappers.
And the 3 parameters in the function is not a difficult case, but already have up to 8 wrappers.
And the amount here is even secondary.
Each developer will give different names to the additional wrappers, i.e. without a single standard.
The resulting API is more difficult to use.

I see what you mean, but this is a synthetic example, and unlikely too appear in real code.
Do you have any examples from a large Go code base where such a proliferation of wrappers is an actual problem? Or other examples from real in production Go code where default arguments would have helped?

@beoran, I did not specifically look for it, but for example, I encountered such an option.
github.com/coreos/etcd/pkg/transport
func LimitListener(l net.Listener, n int) net.Listener
func NewKeepAliveListener(l net.Listener, scheme string, tlscfg *tls.Config) (net.Listener, error)
func NewListener(addr, scheme string, tlsinfo *TLSInfo) (net.Listener, err error)
func NewTLSListener(l net.Listener, tlsinfo *TLSInfo) (net.Listener, error)
func NewTimeoutListener(addr string, scheme string, tlsinfo *TLSInfo, rdtimeoutd, wtimeoutd time.Duration) (net.Listener, error)

Sure there are many overloads, but I think that doesn't look like a problem at all. Rather it looks quite clear, because I can see from each function name what it does.
We could do something like
listen1 := transport.NewTimeoutListener("<IP_ADDRESS>", "ipv4", tlsInfo, 10, 20)
listen2 := transport.LimitListener(listen1, 16)
I can see immediately that in listen2 I have a time out listener that is limited. I don't have to go read the documentation.
With default arguments we get:
func NewListenerEx(addr String, scheme String = "ipv6", tlsinfo * TLSInfo = null, rdtimeout time.Duration = 0, wtimeout time.Duration = 0,  limit int = 0)
Then when calling the functions:
listen1 := transport.NewListenerEx("<IP_ADDRESS>", _, _, _, _, _)
listen2 := transport.NewListenerEx("<IP_ADDRESS>", "ipv4", tlsInfo, 10, 20, 16)
It's not immediately clear that in listen2 I have a limited timeout listener in listen2. I have to read the documentation of the function to find out what is going on. So I would say this example is actually an example of how default values make the API less easy to understand.

@beoran, you suggested an idea.
It is possible to allow the use of default values only for "internal" functions that are in small letters.
Those. would look something like this ...
`go
func NewListener(addr, scheme string, tlsinfo *TLSInfo) (net.Listener, error) {
return newListenerEx(addr, scheme, tlsinfo, _, _, _)
}
func newListenerEx(addr String, scheme String = "ipv6", tlsinfo * TLSInfo = nil, rdtimeout time.Duration = 0, wtimeout time.Duration = 0, limit int = 0) (net.Listener, error) {
...
}
'

Yes, but now I don't even see that default parameters were used.
listen1, err := transport.NewTimeoutListener(addr : "<IP_ADDRESS>", scheme: _,  tlsinfo: _,  rdtimeoutd : 10, wtimeoutd : 20)
Above would solve that, but that is much longer than just
listen1, err := transport.NewTimeoutListenerIpv4NoTls("<IP_ADDRESS>", 10, 20)

@beoran
I think that add NewTimeoutListenerIpv4NoTls can solve the default parameter problem.
But for a function with 7 parameters, there are too many way to use default values, So there will be a lot of function. The cost of change those functions will increase a lot.
So I think function req struct way should be better than write a lot of function to do one thing like NewTimeoutListenerIpv4NoTls or NewTimeoutListener.

Yes, then you get something like this:
listen1, err :=  transport.NewListener(transport.Settings{Hostname: "<IP_ADDRESS>"})
which is also quite good.


feat(pixels): add my new pixel

Checklist

[x ] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

Switched to a new pixel!


linux: add in kernel 5.4 lockdown feature

Motivation for this change
Add kernel lockdown feature - https://lkml.org/lkml/2019/9/10/856
To activate, need add to configuration
boot.kernelParams = [ "lockdown=integrity|confidentiality" ];
Things done

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

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

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

Notify maintainers
cc @

I checked with the links mentioned to open a kernel PR at https://nixos.wiki/wiki/Linux_kernel and seems like it's enabled on debian and arch


Synopsis property on 'Search' endpoint bad formatting

The synopsis property also comes with the 'type' and 'rating' in the beginning.
I hope you can in someway fix this. Thanks!
{
synopsis: "Anime 4.7Segunda parte de Sword Art Online: Alicization"
}

Hello @Shompi ,
The values of the Synopsis and poster properties were fixed in this commit.
I should also mention that the values of the poster property at some API entrypoint were showing me the same issue related to #1.


Unresolved resource dependencies [WarmUpPluginDefaultLogGroup] in the Resources block of the template

After upgrading from 5.1.2 to 5.2.0 using Serverless deployment to AWS Lambda, now seeing the following error on deployment.

Error: The CloudFormation template is invalid: Template format error: Unresolved resource dependencies [WarmUpPluginDefaultLogGroup] in the Resources block of the template

Hi @JeffersonSchuler ,
I haven't been able to reproduce the issue. Can you provide an example repo that I can use for that?

If I get some time I will try to do that.  Here are my environmental settings as reported by Serverless.
Your Environment Information ---------------------------
Operating System:          darwin
Node Version:              14.15.4
Framework Version:         2.25.2
Plugin Version:            4.4.3
SDK Version:               2.3.2
Components Version:        3.7.0

@juanjoDiaz Same issue here. Plugin version: 5.2.0
serverless.yml
`provider:
name: aws
runtime: nodejs12.x
iamRoleStatements:
- Effect: 'Allow'
Action:
- 'lambda:InvokeFunction'
Resource: '*'
custom:
warmup:
default:
enabled: true
role: IamRoleLambdaExecution`
Github CD

Same error with version @5.2.0

Same error with version @5.2.0

The issue looks like it is resolved now with the 5.2.1. release.  Thank you!

The error has reapeared for me when updating to Serverless v3.
Environment

Framework core: 3.1.1
Plugin: 6.0.0
SDK: 4.3.1
Running on Serverless CI/CD
Deploying on AWS
serverless-warmup-plugin: 7.0.1

Build Log
`
Warning: Invalid configuration encountered
at 'functions.offerUploadBulkFile': unrecognized property 'iamRoleStatementsName'
at 'functions.offerUploadBulkFile': unrecognized property 'iamRoleStatements'
at 'functions.filesInitiateUpload': unrecognized property 'iamRoleStatementsName'
at 'functions.filesInitiateUpload': unrecognized property 'iamRoleStatements'
Learn more about configuration validation here: http://slss.io/configuration-validation
Warning: WarmUp: Skipping warmer "default" creation. No functions to warm up.
× Stack offers-api-service-feat-gvm-600 failed to deploy (139s)
Environment: linux, node 14.15.5, framework 3.1.1 (local), plugin 6.0.0, SDK 4.3.1
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issues
Error:
The CloudFormation template is invalid: Template format error: Unresolved resource dependencies [WarmUpPluginDefaultLogGroup] in the Resources block of the template
`

Hi @GVMafeedme ,
Can you provide an example repo to replicate?
This is related to the lifecycle events hooks.
It's proving pretty hard to have hooks that work for all possible use cases and combinations of plugins and serverless pro functionality....

Hi @GVMafeedme , Can you provide an example repo to replicate?
This is related to the lifecycle events hooks. It's proving pretty hard to have hooks that work for all possible use cases and combinations of plugins and serverless pro functionality....

It works correctly on version 7.0.2 thank you!


Allow interning objects with unknown size

Addresses issue #15.

Add T: ?Sized bound to most impls.
Added impls:

impl<'a, T> From<&'a T> for ArcIntern<T>
where
    T: Eq + Hash + Send + Sync + 'static + ?Sized,
    Arc<T>: From<&'a T>,

and
impl<T: Eq + Hash + Send + Sync + 'static + ?Sized> From<Box<T>> for ArcIntern<T>

Ohh actually this isn't that useful since Arc, in contrast to Box, is not #[fundamental] so Arc<T>: From<&T> can only be implemented by things in the standard library. I guess that is why serde's Arc / Rc support also uses Box<T>: From<T> as a bound.
Can this be done here as well, using Box<T>: From<&T> and going through the new / adjusted Box impl? I guess that will be a little less efficient unless it's optimized away, but will allow custom types.

You can totally write  impl From<&T> for Arc<T> for any user-defined type: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=d3926ee3231747b60b0af42d995de009.
Perhaps I misunderstand what the issue is.

Ah sorry, I mixed things up. TryFrom<&str> for Arc<TypeThatWrapsStr> was the thing that didn't work where it does work for Box.
ruma_identifiers::RoomId is the kind of type that Jonathan wanted to experiment with putting into ArcIntern. I'm pretty sure it should work now.

I don't know about the &'static T bond, but other than that, this looks pretty good.

I don't know about the &'static T bond

This is unfortunately required for TypeId stuff to work.

Yes, 'static makes sense since you could otherwise have ArcIntern<ThingThatReferencesLocalLifetime<'_>> and extend that local lifetime through interning.


SQL Server/PostgreSQL conflict: ambiguous reference for IndexBuilder

Starting from version 6.0.4 there is an issue with IncludeProperties method in IndexBuilder.
Ambiguous invocation: Microsoft.EntityFrameworkCore.Metadata.Builders.IndexBuilder<TEntity> IncludeProperties<TEntity>(this Microsoft.EntityFrameworkCore.Metadata.Builders.IndexBuilder<TEntity>, System.Linq.Expressions.Expression<System.Func<TEntity,object?>>) (in class SqlServerIndexBuilderExtensions) Microsoft.EntityFrameworkCore.Metadata.Builders.IndexBuilder<TEntity> IncludeProperties<TEntity>(this Microsoft.EntityFrameworkCore.Metadata.Builders.IndexBuilder<TEntity>, System.Linq.Expressions.Expression<System.Func<TEntity,object>>) (in class NpgsqlIndexBuilderExtensions) match

The problem is that the EFCore.BulkExtensions now refers Microsoft.EntityFrameworkCore.SqlServer and Microsoft.EntityFrameworkCore.PostgreSQL which has completely the same set of extensions for EF builders.
Looks like it will be better to create separate packages for Postgres and SQL Server implementations.

How do you reproduce this problem?
In RunUpdate test it works without issue:
PropertiesToInclude = new List<string> { nameof(Item.Description) },

public class MyEntityConfig : IEntityTypeConfiguration<MyEntity>
{
    protected override void ConfigureEntity(EntityTypeBuilder<MyEntity> builder)
    {
        builder.HasIndex(it => it.UserId)
            .IncludeProperties(it => new { it.ReadAt, it.DeletedDate });
    }
}

The problem is that even if I specify the extensions class implicitly there is an autogenerated snapshot that have the same issue.

Try this fix at the moment:
https://github.com/dotnet/efcore/issues/20780

Use explicit call like this:
protected override void ConfigureEntity(EntityTypeBuilder<MyEntity> builder)
{
	builder.HasIndex(it => it.UserId);
	Expression<Func<MyEntity, object?>> expr = it => new { it.DeletedDate };
	SqlServerIndexBuilderExtensions.IncludeProperties(builder, expr);
}

Thank you for the references.
These solutions are ok as work-arounds, but is there any plans to fix this issues?

Will consider all options.

As a workaround you can install Npgsql.EntityFrameworkCore.PostgreSQL explicitly and give it some kind of alias. Npgsql will be virtually disabled by doing that.
For more info see: extern-alias
<Project Sdk="Microsoft.NET.Sdk">

   <ItemGroup>
      <PackageReference Include="EFCore.BulkExtensions" Version="6.2.9" />

      <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="6.0.2">
         <Aliases>EfCoreNpgsql</Aliases>
      </PackageReference>
...

As a workaround you can install Npgsql.EntityFrameworkCore.PostgreSQL explicitly and give it some kind of alias. Npgsql will be virtually disabled by doing that.
For more info see: extern-alias
<Project Sdk="Microsoft.NET.Sdk">

   <ItemGroup>
      <PackageReference Include="EFCore.BulkExtensions" Version="6.2.9" />

      <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Version="6.0.2">
         <Aliases>EfCoreNpgsql</Aliases>
      </PackageReference>
...

The workaround works, although I'd still prefer for this package not to force me to load a specific provider I am not using.

We also just ran into this issue. I guess the common approach is to have a core package like EFCore.BulkExtensions which is provider independent, and then you have additional provider specific packages like EFCore.BulkExtensions.SqlServer and EFCore.BulkExtensions.PostgreSQL.
I'm fully on @FrankHoogmans side: I would like EFCore.BulkExtensions not to pull providers. According to this comment it is considered unusual to pull multiple providers.

I have created a proof of concept for splitting the library into multiple ones:
https://github.com/FrankHoogmans/EFCore.BulkExtensions/tree/proof-of-concept/split-into-provider-packages
This would seperate the explicit dependencies and their respective code into seperate libraries.
The only breaking change is that is is needed to call provider specific registration code on startup of your app (once), to ensure everything is setup correctly:
            SQLServer.BulkExtensions.EnableSQLServer();
            SQLite.BulkExtensions.EnableSQLite();
            PostgreSql.BulkExtensions.EnablePostgreSql();

There are some things I am not completely satisfied with, but it's a proof of concept that I think works and showcases the possibilities. Refactoring would always be an option. All units tests (except for integration tests, did not have all providers available locally) are working.
@borisdj Would any future development in this direction (multiple packages) be considered as an option?

I've been considering that option.
Another approach that come to mind is to keep main NuGet with all providers but to have separate packages that would just exclude other providers and certain files. This would be done with some (semi)automated procedure when doing publish.
Reason for this is to keep maintaining easier. Lately don't have much spare time.
Another thing is, I have planned to do some larger refactoring first (no timeframe yet), which would include more code separation that would then enable easier implementation of either of these 'splitting' methods.
@FrankHoogmans will analyze your repo more when time comes for splits.

@borisdj You could potentially set PrivateAssets="all" for the package references related to providers. This should then exclude the reference when packing and publishing. Found this blog post illustrating it nicely: https://www.jocheojeda.com/2019/07/22/how-to-exclude-package-dependencies-in-a-nuget-package/
But this would require that the code itself in the library is separated enough that no access to the provider is attempted as it would lead to TypeLoadExceptions unless you actually install the other provider packages as integrator.

I've encountered a similar problem after upgrading my application that uses DB Functions (DateDiff, etc.).
I'm getting the error about EF.Functions.DateDiffDay being ambiguous between Microsoft.EntityFrameworkCore.SqlServerDbFunctionsExtensions.DateDiffDay and Microsoft.EntityFrameworkCore.MySqlDbFunctionsExtensions.DateDiffDay.
I've tried the workaround suggested above, by directly referencing the Pomelo MySql library that is used by this BulkExtensions project, and aliasing it, so my project file now looks like so:
<PackageReference Include="Pomelo.EntityFrameworkCore.MySql" Version="6.0.2">
			<Aliases>PomeloMySql</Aliases>
		</PackageReference>

But this has not resolved my problem.  I still get the same build error.
Please advise what I may be missing? (For clarity, my application only uses SqlServer, and NOT MySql.)


simple golang server setup

user story:
We want our IPFS gateway to be easy to clone. A user should be able clone this repo, and run a single command to have a little server up and running.
Description:
An easy to run http server running go.  With the outline ready for handling simple requests.
Acceptance criteria: Create the outline of a simple server using go. Should be able to handle basic GET request and serve an empty html file.
Ideally localhost will be hosted at something like gateway.storj.io  later
Basically a simple GET api which handles requests like
localhost:443/ipfs/abcd1234efgh5678
First step of this is just receiving request, acknowledging with a 200, and console.log(key)

just print out abcd1234efgh5678 for requests


Variant adds zeros at each price update

Lunar version: ^1.0.0-beta.3
Laravel Version: 11.27.2
PHP Version: 8.3
Database Driver & Version: PostgreSql 16.2

Expected Behaviour:
When I update the price of a product variant, it adds a zero to the end of each variant price.
https://github.com/user-attachments/assets/17a83e20-3aac-4fe2-beea-06ff3e483f7c
Actual Behaviour:
Steps To Reproduce:

Solution:
I think that the price should be stored in a raw_price or equivalent column, and every time the price is updated it should reference the raw data in this column. ProductOptionsWidget.php#L405


implement gpu usage for use with #566

Codecov Report

Merging #686 into master will decrease coverage by 1.09%.
The diff coverage is 11.36%.

@@            Coverage Diff            @@
##           master     #686     +/-   ##
=========================================
- Coverage   90.54%   89.45%   -1.1%
=========================================
  Files          48       48
  Lines        3640     3679     +39
=========================================
- Hits         3296     3291      -5
- Misses        344      388     +44

Impacted Files
Coverage Δ

vent/helpers/meta.py
70.63% <11.36%> (-13.55%)
:arrow_down:

vent/api/actions.py
91.6% <0%> (-2.27%)
:arrow_down:

vent/menus/tools.py
88.62% <0%> (+0.59%)
:arrow_up:

vent/api/plugin_helpers.py
82.17% <0%> (+0.9%)
: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 b158a64...4f5dee7. Read the comment docs.


Generating deprecated code when used with Play 2.5.x

I am trying to use plugin with latest Play 2.5.x but unfortunately it is generating deprecated code.
Do you have any suggestion/advice to make this plugin compatible with the latest PlayFramework ?

It was actually just in the Client side generation.
Thanks for the notify!

Thanks :)


Added lib_git.cmd

Lib it: Git Version Comparing
Status: Needs to be tested and edited before merging.
Please do not merge until the edits are committed.

@daxgames I'd appreciate you writing the docs, but please wait first before I add some more commits here so I'll be sure it's final. Thanks!

Ok not sure what else it needs because it does already work.  Waiting for your reply.

@daxgames Done. Yeah, it was just some minor edits that I wanted to do before you merged them. Could you fill the docs for all the subs now?
Great work BTW, "Libbing" Cmder has been nicely done!

Cool will do.  I am getting close on fixing cmder.exe.  I am not a c++ guy so it takes me a while...


ATAC-seq Data Integration

Cell by bin
Visualize in higlass
Cell by peaks (in BED + snap files)
Annotated peaks (genomic intervals) per cell
Genome-wide (not necessarily tied to a gene)
Our Pipelines:

[ ] We need to create all the new pipelines.

TMC:

CalTech
Stanford

Outstanding Issues:

Creation of new pipelines

Notes:
We have many visualization options:

Cell by gene

A result of the alignment? Perhaps less valuable since derived from the cell by bin matrix?
Would be useful if our heatmap viewer can scale

Quality control

Potentially view in a table

Motifs

In higlass: potentially nucleotide track or epilogos?

Variability scores for each motif

Which transcription factors are enriched per cell

Quality control scores motifs
Probably more helpful to visualize cluster profiles rather than individual cell profiles

Is there a clustering available in the snap output files already?
Also would want to show variability within clusters

Can we summarize each cluster into a single profile (using the bin data for profiles)

Link from cell sets to clusters (e.g. with colors)

Ilan says:

This just kind of fell by the wayside.  I’ll have a look again, I don’t remember where this was left.


[python] fix main thread raise error, process no quit

I have the following problem in production environment.
I need to do something(A) after starting the grpc server, at this time, 'A' throws exception.
The main thread needs to be shutdown, so the thread pool of grpc is also shutdown.
'_serve' thread raise RuntimeError, and shutdown.
'state.server' alive. process alive, but not work.
I hope stop 'state.server' when '_serve' was closed.

python version: 3.9.5

grpcio -- 1.44.0

server.py

from concurrent import futures
import time

import grpc

import test_pb2_grpc
import test_pb2

class OrderDispatcherServicer(test_pb2_grpc.TestServiceServicer):
    def testFunc(self, request, context):
        time.sleep(0.5)
        return test_pb2.TestRespnse()

server = grpc.server(futures.ThreadPoolExecutor(20))
test_pb2_grpc.add_TestServiceServicer_to_server(OrderDispatcherServicer(), server)
server.add_insecure_port("<IP_ADDRESS>:50000")
server.start()
print("Start...")

time.sleep(8)
raise Exception('test')

client.py

from concurrent import futures

import grpc

import test_pb2_grpc
import test_pb2

t_pool = futures.ThreadPoolExecutor(30)
channel = grpc.insecure_channel("localhost:50000")
stub = test_pb2_grpc.TestServiceStub(channel)

def request_test_func():
    stub.testFunc(
        test_pb2.TestRequest()
    )

for _ in range(1000):
    t_pool.submit(request_test_func)

test.proto

syntax = "proto3";

package test;

message TestRequest {}

message TestRespnse {}

service TestService {
    rpc testFunc(TestRequest) returns (TestRespnse);
}

error

Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 954, in _bootstrap_inner
    self.run()
  File<EMAIL_ADDRESS>line 892, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/libra/Desktop/python-grpc-env/python-grpc-env/lib/python3.9/site-packages/grpc/_server.py", line 883, in _serve
    if not _process_event_and_continue(state, event):
  File "/Users/libra/Desktop/python-grpc-env/python-grpc-env/lib/python3.9/site-packages/grpc/_server.py", line 847, in _process_event_and_continue
    rpc_state, rpc_future = _handle_call(event, state.generic_handlers,
  File "/Users/libra/Desktop/python-grpc-env/python-grpc-env/lib/python3.9/site-packages/grpc/_server.py", line 757, in _handle_call
    return _handle_with_method_handler(rpc_event, method_handler,
  File "/Users/libra/Desktop/python-grpc-env/python-grpc-env/lib/python3.9/site-packages/grpc/_server.py", line 733, in _handle_with_method_handler
    return state, _handle_unary_unary(rpc_event, state,
  File "/Users/libra/Desktop/python-grpc-env/python-grpc-env/lib/python3.9/site-packages/grpc/_server.py", line 637, in _handle_unary_unary
    return thread_pool.submit(_unary_response_in_pool, rpc_event, state,
  File<EMAIL_ADDRESS>line 163, in submit
    raise RuntimeError('cannot schedule new futures after '
RuntimeError: cannot schedule new futures after interpreter shutdown

result
'state.server' alive, receive request.
'_serve' shutdown, no work.
process work.

So is the issue that the server keeps on running even after the interpreter has begun to shut down? In that case, it seems like an atexit handler might be more appropriate.

So is the issue that the server keeps on running even after the interpreter has begun to shut down? In that case, it seems like an atexit handler might be more appropriate.

thanks for your advice, but state.server.start() is still running, state.server is still maintaining connection with some clients, if these reuqests are not set timeout, state.server will not shutdown forever.
atexit need wait for state.server shutdown to work.

Thanks for all the digging and reproduction information. Can you add a unit test for this behavior?
https://github.com/grpc/grpc/tree/a3acf400d94cc72b17027c751a8d196f0f2fc0ba/src/python/grpcio_tests/tests_py3_only/unit

I will complete these reviews later, thanks

There will be some problems in using _stop and _begin_shutdown_once, I will think about it again

Thanks for all the digging and reproduction information. Can you add a unit test for this behavior?
https://github.com/grpc/grpc/tree/a3acf400d94cc72b17027c751a8d196f0f2fc0ba/src/python/grpcio_tests/tests_py3_only/unit

I modified the unit test.

I don't know how to write better unit tests in this case.


Enable Support for Zstd new Codec for Plugins which registers CodecServiceFactory

Is your feature request related to a problem? Please describe.
This issue is in continuation of bug reported in this issue: https://github.com/opensearch-project/OpenSearch/issues/7012.
Describe the solution you'd like
The solution I can think of is, in the interface of EnginePlugin, if we can add a parameter which can provide the Delegate Codec which can be used, the plugins can use that Delegate Codec to build their codec service and return back to OpenSearch to be used on an Index.
This will allow the plugins to take the benefit where if optimizations are done at the OpenSearch Codec Level, plugins are also able to take advantage of that.
Not sure of this is the best solution but atleast this will allow OpenSearch to continue improving the base/delegate codec and then plugins adding functionalities on top of that.
Describe alternatives you've considered
There is no alternative.

:wave: Hey @navneet1v , is there any progress on this issue regarding knn indices ?


how to make the node not to determine its size by parent, but instead by children?

root (100, 100)
-> root_child0 (100, undefined)
----> root_child0_child0(100, 200)
I need the root_child0's layout height to be 200, how? thanks in advance.

Use the following jsfiddle to play around with styling https://jsfiddle.net/emilsjolander/jckmwztt/. If you have general flexbox questions I suggest asking them on stack overflow. If you have any yoga specific issues please re-open this issue.

Ah, the code above in my post got layout height 200 in chrome, but not in yoga, i don't know which one is wrong or there isn't a specification about that.

Could you please post the jsfiddle you are using so I can see the styles?

https://jsfiddle.net/dolmens/6cj72vbw/

And what is the result running this in yoga?

height of
root: 100
root_child0: 100
root_child0_child0: 200

Thanks! Reproduced locally with the following test fixture.

After set attribute overflow: scroll, the root_child0's layout height becomes 200, which is what I want. So there was only one caveat: the default setting is not compatible with chrome.

@dolmens I have a fix for this internally but because it is a breaking change we are rolling it out slowly starting Monday.

verified


Conflict w/ nativescript-google-maps-sdk plugin

There seems to be a conflict between the nativescript-google-maps-sdk plugin and the push-plugin.  The push plugin is referencing the google play services 8.4.0 but the nativescript google maps sdk plugin is referencing the newest google play services version installed.
I talked w/ the nativescript-google-maps-sdk plugin developer @dapriett and he thought it might be best to add the following line to your plugin's gradle file so it always targets the latest installed version?
compile 'com.google.android.gms:play-services-maps:+'
Thoughts?  Here's the open issue I have on his plugin:
https://github.com/dapriett/nativescript-google-maps-sdk/issues/34#issuecomment-222698981

@dklinzing Thanks, you are correct, for better compatibility the plugins should be referencing the latest version of the SDK available on the server.
I have proposed this to the collaborators of the plugin as an enhancement. Meanwhile you may manually modify the plugin config.

Do you have any sense of an ETA when/whether this change can/will be made?  We use the Telerik Platform and are referencing your npm package currently, so we'd have to figure out a work-around.

@dklinzing
Daniel, my apologies for the delayed reply.
Given that the latest Google Play Services SDK (9.0.0) requires the explicit applicationId in the gradle file, the collaborators of the plugin postponed this for a future version when there is officially added support for this in NativeScript. Please see https://github.com/NativeScript/android-runtime/issues/454
Otherwise adding manually the applicationId will cause similar issues to all users, not only when conflicting with other plugins.
Meanwhile the plugin package can be referenced from a forked repository of the plugin where the dependency is changed to build against the latest (or compatible) Google Play Services SDK. After this point your app to the edited module using the instructions from here.

I am having the same problem with the push notifications plugin.  I believe it has to do with GPS sdk but have tried + for version number fix which didn't work.  It might also be a gradle issue and the format of the gradle as android studio is complaining when I open the project and want to automagically update the gradle format.

Fixed in 0.0.17.

my AVD stopped showing google MAP

pl help

@swinn77 This error sounds that you have updated a framework (probably the Google Play Services SDK) in your app but the device does not contain this version. That is why the error message is advising to update the Google Play services on the device. To do so in the emulator - you should update the image to include a higher Google Play services version support.

@AntonDobrev Thanks, ur reply is much appreciated, can u pl advice if i can downgrade the framework or how can i get update image to match the requirement to mitigate the issue. Pl elaborate

This also happened tonight on a colleagues android device.  He needed to sign into his account on the device so the device would install updates from the play store.  One of those updates was the play store service.  Once we has up to date on his updates the page worked fine.
Same happened on my genymotion emulator.
I followed these steps and boom I got my map back:
* Visit opengapps.org
* Select x86 as platform
* Choose the Android version corresponding to your virtual device
* Select nano as variant
* Download the zip file
* Drag & Drop the zip installer in new Genymotion virtual device (2.7.2 and above only)
* Follow the pop-up instructions

you need to change the versions;
this is what finally worked for me:
in (app/App_Resources/android/app.gradle ) add the following code at the top
project.ext {
    googlePlayServicesVersion = "10.0.0"
}

then in (platforms/android/build.gradle)
replace this line
classpath "com.google.gms:google-services:+"
with this line:
classpath "com.google.gms:google-services:3.0.0"


Fix select UI bug

Codecov Report

Merging #234 (2ff8264) into master (f776bb0) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #234      +/-   ##
============================================
- Coverage     58.45%   58.42%   -0.03%
  Complexity      524      524
============================================
  Files           113      113
  Lines          2065     2066       +1
  Branches        186      186
============================================
  Hits           1207     1207
- Misses          762      763       +1
  Partials         96       96

Impacted Files
Coverage Δ

...c/main/java/seedu/travelr/ui/SelectedTripCard.java
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Lint/UselessMethodDefinition: false positive(?) with mixed in method

Expected behavior
Given the test script
require 'action_controller'

class A
  include ActiveSupport::Configurable
  include ActionController::RequestForgeryProtection

  def valid_authenticity_token?(session, encoded_masked_token)
    super
  end
end

class B
  include ActiveSupport::Configurable
  include ActionController::RequestForgeryProtection
end

the method definition in class A seems necessary
$ irb -I. -rtest.rb
2.5.8 :001 > A.new.respond_to?(:valid_authenticity_token?)
 => true
2.5.8 :002 > B.new.respond_to?(:valid_authenticity_token?)
 => false

and therefore is not an offense.
Actual behavior
$ rubocop --only Lint/UselessMethodDefinition test.rb
Inspecting 1 file
W

Offenses:

test.rb:7:3: W: Lint/UselessMethodDefinition: Useless method definition detected.
  def valid_authenticity_token?(session, encoded_masked_token) ...
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

1 file inspected, 1 offense detected, 1 offense auto-correctable

Steps to reproduce the problem
See above
RuboCop version
$ [bundle exec] rubocop -V
0.90.0 (using Parser <IP_ADDRESS>, rubocop-ast 0.3.0, running on ruby 2.5.8 x86_64-linux)

You are correct it isn't useless in the sense of having no side-effect, so maybe we should change the wording, but the only thing this code does is make the method public (and slow it down a bit).
It should be replaced by public :valid_authenticity_token?


build hooks package with vite and stop bundling react

This ensures that consumers use their own version of React instead of bundling potentially multiple, conflicting versions. Addresses https://github.com/automerge/automerge-repo/issues/228
Additionally, I've added a yarn visualize command that allows you to visualize the bundle's size. You can see from these screenshots. As you can see here, we've reduced the bundle's size by roughly 10x via excluding react and react-dom out of the bundle.

Thank you! I've been focused on some other work for a bit but I should have some time this week to put into this and a few other PRs that have been accumulating.

Landed; thanks!


Correct non-fast-failing behavior to match other tasks such as run

dbt build is exiting earlier than it should when faced with a broken project.  This leads to an inconsistent user experience, and an incomplete process artifacts (run_results.json).  Work done to correct this seemed to lead into larger discussions about the expected behavior of the fail-fast flag.
References:

https://github.com/dbt-labs/dbt/pull/3490#pullrequestreview-705849186
https://github.com/dbt-labs/dbt/pull/3490#pullrequestreview-709735773

dbt build is exiting earlier than it should when faced with a broken project

It seems to exit as soon as it encounters any runtime error, including SQL errors when running models—so not a broken project per se. That means its logging output looks different from all other tasks, and it never writes run_results.json. Not good!
Sometimes that "instant exit on error" is a feature, not a bug—it should only be when the -x, --fail-fast flag is passed. Incidentally, dbt build --fail-fast seems to work just fine (and differently!) when it encounters an error. (Today, --fail-fast errors for all commands prevent writing the result artifact. Is that good as a general rule? Fair question: https://github.com/dbt-labs/dbt/issues/3600.)
What's surprising to me is how/why dbt build has different behavior from standard-issue tasks like dbt run and dbt snapshot, when I figure it inherits so much of its behavior from the base tasks.


Integration with serving frameworks

Are there plans to integrate with serving frameworks such as text-generation-inference, vLLM etc?

Currently our main focus is to release more pre-trained HMM checkpoints and provide more handy functions for DFA constructions. After that we would consider integrations with LLM serving frameworks. Thank you for your suggestion!


feat(all): Move post-success scripts to separate files

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6797d3d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #264   +/-   ##
=========================================
  Coverage          ?   91.16%
=========================================
  Files             ?       11
  Lines             ?     1143
  Branches          ?        0
=========================================
  Hits              ?     1042
  Misses            ?      101
  Partials          ?        0

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 6797d3d...7b6ac2e. Read the comment docs.


implement a multistream transport protocol

@jbenet The peerstream tests for this fail. but i'm really not sure why. Everything passes fine when i integrate it into go-ipfs... (which doesnt mean much, but still).
Maybe you could take a look at this for me?

beautiful PR. :) :+1: :heart: composability.

fixed a small bug.

the big test still fails though, And i'm not sure why...

@whyrusleeping i force-pushed

@whyrusleeping can you run this in some beefy machine? running the tests without -race all pass on my machine. with -race it hoses my laptop and travis

LGTM


Custom Authorisation with Graph API

Is there already an example that works with Microsoft Graph API? There are a lot of case where AAD might be intimidating for the end user. Using Graph API I feel will be better for the end user as they are already familiar with the authorisation screen from using office.com. I was trying to figure out if I can do the approach using the custom approach but I am getting confused as to what I should put in the configuration settings.

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

ID: 6355b9cc-780a-c5e9-e24a-aa6593b37ae1
Version Independent ID: aa20c4a6-3243-d7b5-8e12-54e044b99cc2
Content: Authentication and authorization for Azure Static Web Apps
Content Source: articles/static-web-apps/authentication-authorization.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

Thank you for your feedback! We will review and update as appropriate.

mmoser-adrianv, after the initial review, your feedback has been shared with the content owner for further review. Thanks for taking time to share your feedback!

Thanks for your feedback and your contribution to Azure docs.
Feedback for this repository is moving away from GitHub to a system specific to the Microsoft Learn platform. Issues for this repository will soon be disabled, and additional comments from GitHub will no longer be possible. However, we are now tracking and triaging this issue in the new feedback system.
To learn more about our feedback systems, please see Provide feedback for Microsoft Learn content.
#please-close


Dynamic links creation. Refer to file title and accounts title

Hello, beancount has an option "title" for beancount file. For example,"title" "MO Wealth"which generates links mo-wealth instead of beancount
So, your hardcoded link /beancount/balance_sheet/ in my case /mo-wealth/balance_sheet/
It will be cool to receive some variable like ledger.title to use in links creation
Another options is names for root accounts, it useful in other languages, for example in Russian it will be looks like:
option "name_assets"      "Активы"
option "name_liabilities" "Пассивы"
option "name_income"      "Доходы"
option "name_expenses"    "Расходы"
option "name_equity"      "Капитал"
So it will be cool to use variables name_assets instead of real names: Assets, etc.

Where is /beancount/balance_sheet/ hardcoded? As far as I'm aware it is only given as an example in the https://github.com/andreasgerstmayr/fava-dashboards/blob/main/example/dashboards.yaml file. This example dashboard is meant to be adapted based on your individual ledger.
Instead of templating every aspect of this file, I find it much simpler the way it is. I suggest to run a search & replace over it to adjust the links and asset names to your specific configuration.

First of all, your work is great and yes, links not hardcoded but they are not variables, so you can't create universal template for testing for example. Sure, I could search &replace but if I use multiply files it could be rather easy to use links for root accounts and for file name. Thank you


error: attribute 'libubox20220927' missing

I put together a simple repo here:
$ nix build github:jfly/nix-openwrt-imagebuilder-bug-demo/missing-libubox20220927#my-router
error:
       … while calling the 'derivationStrict' builtin

         at //builtin/derivation.nix:9:12: (source not available)

       … while evaluating derivation 'openwrt-23.05.0-rc1-nix-mediatek-mt7622-linksys_e8450-ubi'
         whose name attribute is located at /nix/store/p57nnwjhfvmsn75y9l6hn00pl2xv7ivm-source/pkgs/stdenv/generic/make-derivation.nix:303:7

       … while evaluating attribute 'configurePhase' of derivation 'openwrt-23.05.0-rc1-nix-mediatek-mt7622-linksys_e8450-ubi'

         at /nix/store/xkw95d1jbwm42pmvwm7pmcvkk3788c58-source/builder.nix:70:3:

           69|
           70|   configurePhase =
             |   ^
           71|     let

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: attribute 'libubox20220927' missing

       at /nix/store/xkw95d1jbwm42pmvwm7pmcvkk3788c58-source/files.nix:180:20:

          179|             with_new_dep = current_deps // { ${new_dep} = true; };
          180|             deps = packages.${new_dep}.depends;
             |                    ^
          181|           in

There is a new OpenWRT prerelease "23.05.0-rc1" which is still incomplete. This caused trouble before!
Fix by pinning to the stable API:
--- flake.nix
+++ flake.nix
@@ -8,7 +8,7 @@
       let
         pkgs = nixpkgs.legacyPackages.x86_64-linux;

-        profiles = openwrt-imagebuilder.lib.profiles { inherit pkgs; };
+        profiles = openwrt-imagebuilder.lib.profiles { inherit pkgs; release = "22.03.5"; };

         image = profiles.identifyProfile "linksys_e8450-ubi";
       in

Should I rather filter out -rc versions?

Ah, thank you! What if I want to follow the latest, non-prelease version?

Given the problems we encounter with the prelreleases, I am starting to think that nobody wants that in their unattended image building pipeline. Stable OpenWrt releases only from now on!
I just hope this won't repeat itself while the .0 release is being uploaded... :-)


[Feature] Extract DB files to a subdirectory within temporary folder

Is your feature request related to a problem? Please describe.
The library performs a cleanup routine of the /tmp directory at the start of every execution.
https://github.com/xerial/sqlite-jdbc/blob/61fda53dd39bc31dbf46f5f4523d85334f6c0fb9/src/main/java/org/sqlite/SQLiteJDBCLoader.java#L90C48-L90C58
This means that the duration of time to perform this action depends on the number of files in the /tmp directory at the time of startup.
Describe the solution you'd like
This can be mostly mitigated by first creating a sub-directory in the /tmp directory (e.g., "xerial-sqlite-jdbc") and only store JDBC native libs in this directory. The size of this directory will be much less at risk of having large numbers of files present.

PR welcome

I thought about that, and we already have org.sqlite.tmpdir, so this seems redundant. What do you think?


Instagram not working

Ripme version: 1.7.24
Java version:  8 update 151
Operating system: Windows 7
Exact URL you were trying to rip when the problem occurred: https://www.instagram.com/privatekanye
Please include any additional information about how to reproduce the problem:

Instagram ripper isn't working at all for me for any instagram account. The ripper just gives me an error message: "JSONObject["TagPage"] not found." I've tried different versions of the ripper and still have the same issue. Anyone know what the issue could be and how to fix it?

It looks like they changed their JSON structure.

Was coming here to report the same thing