The cache file conversion LESS to CSS ignores changes in files that have been announced @import

Check of existence is the file cache in AssetConverter
public function convert($asset, $basePath)

            if (@filemtime("$basePath/$result") < filemtime("$basePath/$asset")) {

For example, I point to the Asset Bundle file bootstrap.less, in which imported several LESS files. After changing one of the imported LESS files will not re-conversion bootstrap.less
I solved this problem in yii1 that specifies the directories in which you want to track filemtime to each file compiled LESS.

We just came up with a solution for this for Craft. We’re overriding AssetManager::publishDirectory() with the following method:
protected function publishDirectory($src, $options)
{
	// See if any of the subdirectories have a more recent modify date than $src
	$srcModTime = filemtime($src);
	$subdirs = glob($src.DIRECTORY_SEPARATOR.'*', GLOB_NOSORT | GLOB_ONLYDIR);

	foreach ($subdirs as $dir)
	{
		if (filemtime($dir) > $srcModTime)
		{
			@touch($src);
			break;
		}
	}

	return parent::publishDirectory($src, $options);
}

So basically if any subdirectories have a more recent mtime than $src, we call touch() to bump $src’s mtime. Then we let Yii’s parent method take it from there.
Seems to be working quite well, and shouldn’t have too much of a performance hit.

For reference forceCopy per AssetBunde - Yii Wiki

Setting the forceCopy option to true makes working with less nearly impossible. Each page load takes like half a minute with couple of tens of less files. Wouldn't it be faster to check @imports recursively?

This issue can be solved providing custom check for whether asset is outdated  or not.
In my project I have created a special AssetConverter for that:
class AssetConverter extends \yii\web\AssetConverter
{
    /**
     * @var boolean whether to check if already converted assets are outdated (source has been modified since conversion).
     * While enabled this option will force re-conversion of the outdated assets, but will consume extra time.
     */
    public $checkOutdated = true;

    /**
     * @inheritdoc
     */
    public function convert($asset, $basePath)
    {
        $pos = strrpos($asset, '.');
        if ($pos !== false) {
            $srcExt = substr($asset, $pos + 1);
            if (isset($this->commands[$srcExt])) {
                list ($ext, $command) = $this->commands[$srcExt];
                $result = substr($asset, 0, $pos + 1) . $ext;
                if ($this->forceConvert || $this->isOutdated($basePath, $asset, $result, $srcExt)) {
                    $this->runCommand($command, $basePath, $asset, $result);
                }

                return $result;
            }
        }

        return $asset;
    }

    /**
     * @param string $basePath the directory the $asset is relative to.
     * @param string $asset the asset file path, relative to $basePath
     * @param string $result the converted asset file path, relative to $basePath
     * @param string $sourceExtension source asset file extension.
     * @return boolean whether asset is outdated or not.
     */
    protected function isOutdated($basePath, $asset, $result, $sourceExtension)
    {
        $resultModificationTime = @filemtime("$basePath/$result");
        if ($resultModificationTime === false || $resultModificationTime === null) {
            return true;
        }
        if (!$this->checkOutdated) {
            return false;
        }
        if ($resultModificationTime < @filemtime("$basePath/$asset")) {
            return true;
        }
        foreach (FileHelper::findFiles($basePath, ['only' => ["*.{$sourceExtension}"]]) as $filename) {
            if ($resultModificationTime < @filemtime($filename)) {
                return true;
            }
        }
        return false;
    }
}

@klimov-paul thank you! It would be very useful to have something like this in core though.

We've created https://github.com/dmstr/yii2-web/blob/master/src/AssetBundle.php to overcome this limitation.
Von meinem iPhone gesendet

Am 04.08.2016 um 23:35 schrieb Nikolay Oleynikov<EMAIL_ADDRESS>Setting the forceCopy option to true makes working with less nearly impossible. Each page load takes like half a minute with couple of tens of less files. Wouldn't it be faster to check @imports recursively?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@qiangxue @klimov-paul @samdark @cebe I can't quite get it why this issue is closed. Am I missing something or the asset converter is still broken?
If you use default asset converter and set forceConvert then every page load will take like a minute with a large number of files. If you don't set forceConvert then imported assets just won't recompile. So one just can't use default asset converter.
Basically you have two options - either implement some custom asset converter or just get rid of it completely and user some external tools like watch. The funny thing is that the official framework site uses the latter.
Can we have some working asset converter in framework core? If no then what is the point of having this section if the official guide?

@Kolyunya I am using the forceConvert on small sites. If you whole compilation step takes a minute that is not a limitation of yii, but of the less compiler and you should consider using a different approach. There is seldomly one way that solves all problems. The auto conversion is for small sites, if you have a large set of less files you probably need to use other tools for it.

In my examples above we were trying just to look for modification times of files in the the asset folders. We're setting the modification time of the folder to the time of the newest file.
So we don't have to forceCopy all the time, but only when a modification time changes (by changing the hash), which is much faster.
What's about  a deepHash option, which could be turned on during development?

I see situation can be improved, so I'm reopening this. How fast is PHP on iterating over directories and getting mtimes? If that works fast enough, we could implement something that uses such an approach.

wait, this seems to be duplicate with https://github.com/yiisoft/yii2/issues/8197


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1033 ms

Resolved: Bofrost.de performance has improved in 96ada47.


(SP: ) Fix sonnar errors

Investigate and fix sonnar issues. (you can see them in develop branch commits list)

not relevant


libreoffice: add update script and 7.2.5 -> 7.3.3

Description of changes
Fixes https://github.com/NixOS/nixpkgs/issues/174063.
The 7.2.5 version is already outdated and 404'ing. This pr adds the following:

Update script with nixt tests to ease the update process: just run nix-shell maintainers/scripts/update.nix --argstr package libreoffice to update to the latest stable version from https://download.documentfoundation.org/libreoffice/stable/. Furthermore, this may even enable and trigger simple https://github.com/ryantm/nixpkgs-update package pr to proactively update to the latest stable version.
Updates from 7.2.5 -> 7.3.3
Minor general darwin improvements

@NixOS/darwin-maintainers: I tested this on aarch64-darwin. Could someone give this a test ride on x86_64-darwin?
One question remains:

What is needed so that the darwin platforms and versions also show up on https://search.nixos.org/packages? How do you handle different versions for darwin vs. linux on https://search.nixos.org/packages? Do I have to give my derivation another pname?

Edit: Answered here https://github.com/NixOS/nixos-search/issues/485.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Tested on x86_64-darwin, both the update script and libreoffice seem to work fine.

@ofborg build libreoffice

Follow-up to the discussion here https://github.com/NixOS/nixos-search/issues/485:
Is there any downside if we rename this to libreoffice-darwin in all-packages.nix because this derivation is completely different from the linux one?
The advantage would be a clear separation and visibility on https://search.nixos.org/packages.

They linked an example of a pattern that works in that issue: https://github.com/NixOS/nixpkgs/blob/5ce6597eca7d7b518c03ecda57d45f9404b5e060/pkgs/applications/networking/instant-messengers/teams/default.nix#L27-L34=.

@ofborg build libreoffice-darwin

They linked an example of a pattern that works in that issue: https://github.com/NixOS/nixpkgs/blob/5ce6597eca7d7b518c03ecda57d45f9404b5e060/pkgs/applications/networking/instant-messengers/teams/default.nix#L27-L34=.

I know because I wrote that code😄.

@ofborg build libreoffice

@7c6f434c Just for your information: I extracted the meta attribute to common.nix and added darwin to supported platforms so that it will show up on https://search.nixos.org/packages.

@SuperSandro2000 I always appreciated your detailed feedback and experience on prs.
However, I think that the discussion around update script written in nix with tests vs. update script written in bash is not going into a fruitful direction.
Feedback like that:

Then your update script is not great. Usually the version is updated and then the hash in the same run.

is not constructive and leads nowhere (especially when the update script is currently working as intended).
I propose to move forward with the current working update script written in nix. Time will tell if it is indeed as fragile as you write it to be. If it is then the burden is on me since I am the maintainer and I'll gladly fix it and convert it to a bash script using pup.

I do not think splitting is bad because yours is libreoffice-bin not really libreoffice by the common attribute naming nomenclature.

The very idea of imperative code in Nix inside Nixpkgs is not what people expect. This in itself is a valid criticism.

@7c6f434c I agree that libreoffice-bin is more accurate according to the nomenclature but it is not commonly used for darwin. Therefore I would keep it at libreoffice to keep it simple. Are you ok with sharing the meta attribute and keeping it at libreoffice or do you want me to introduce libreoffice-bin?

I do not think splitting is bad because yours is libreoffice-bin not really libreoffice by the common attribute naming nomenclature.

If there is a source build version and a downloaded and patched version, then both names should exist. If there is only one version the name is libreoffice and the information that it is not source build should eventually be in meta when people figured out how they want to do that.

I propose to move forward with the current working update script written in nix.

I don't agree with that. There are good reasons why update scripts are not written in nix and have no tests. They are a bit flimsy and especially the ones that parse websites are fragile and cannot be made future proof. Ideally we would want to use some kind of API instead.

is not constructive and leads nowhere (especially when the update script is currently working as intended).

You described a bug in the update script to me and I wanted to imply that it should be fixed because most other update scripts also can do that in one run.
PS: Just the fact that something works does not make it a good solution to a problem. It can work and at the same time be an ugly solution which shouldn't really be done.

I don't really like an outcome where I am listed among maintainers of something that I a) have never been interested in, b) find too magic (and I like tricks, it's just that this goes too far for Nixpkgs in my opinion)

I don't really like an outcome where I am listed among maintainers of something that I a) have never been interested in, b) find too magic (and I like tricks, it's just that this goes too far for Nixpkgs in my opinion)

What magic are you talking about?

If there is a source build version and a downloaded and patched version, then both names should exist. If there is only one version the name should be libreoffice and the information that it is not source build should eventually be in meta when people figured out how they want to do that.

Done I renamed the darwin version to libreoffice-bin.

I don't agree with that. There are good reasons why update scripts are not written in nix and have no tests. They are a bit flimsy and especially the ones that parse websites are fragile and cannot be made future proof. Ideally we would want to use some kind of API instead.

I completely agree that we should always use an api if available.
However, I am not convinced by your argument: We have always written an update-script in bash, therefore this is the only right way to do that and if there is no api, you should not write an update script because parsing html is fragile and not future proof. What I am doing here is merely the following:

I precompute the latest version and the sha256 by parsing some html (because I have not found an api) using nix.
I use this precomputed values in the update bash script using the recommended update-source-version script to update the version.
Since I update two platforms (aarch64-darwin and x86_64-darwin) I need to use a workaround by resetting the version first, which is also used in the sublime4 derivation.

So I really don't understand why step number 1 is causing so much problems. Besides, the update-script is mostly used by maintainers (me) and if there are automated tests to help me spot changes to my source early, I really like that.

You described a bug in the update script to me and I wanted to imply that it should be fixed because most other update scripts also can do that in one run.

No I did not describe a bug. You just commented that We shouldn't need to reset the version first. This can be done in one go. and I then pointed out that I couldn't make it work without the abovementioned workaround.

PS: Just the fact that something works does not make it a good solution to a problem. It can work and at the same time be an ugly solution which shouldn't really be done.
I completely agree. But in my opinion it is always better to: first make it work and then beautiful. As I already said, if the update-script breaks in the future (because it is ugly, which you imply here), then I fix it and make it better.

I would love to get some other honest opinions on that topic. I am actively defending my rights as a maintainer to decide about the implementation details of the update script. Am I being too stubborn? @NixOS/nixpkgs-committers.

@ofborg build libreoffice-bin

What magic are you talking about?

Nix code with intentionally non-reproducible evaluation. Moreover, updateScript that determines upstream version while being evaluated, not while being run.

@tricktron You just pinged over 150 people. Please don't do that. Since GitHub does not allow bulk-unsubscribing people from issue and in order to keep spamming them at a minimum, I am going to close this PR. Please open a new one.

@piegamesde ups, sorry pinged the wrong group:sweat_smile:. Will do.

New pr is here https://github.com/NixOS/nixpkgs/pull/175160

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/updatescript-written-in-nix-considered-bad-practice-vs-bash/19379/1


Entity name with namespace change mapping

When I create a query builder by using entity classname like:
$this->entityManager
            ->getRepository($entity->getClassName())
            ->createQueryBuilder($entity->getClassName())`

$entity->getClassName();//   ->   Cds\\Library\\Mysql\\Entity\\CdsDomain

and add a ordering
$dql->orderBy(
                sprintf(
                    '%s.%s',
                    $entity->getClassName,
                    $somestring
                ),
                $order->getParam('dir', 'string')
            );

doctrine create the query (example)
SELECT c0_.name AS name_0, c0_.id AS id_1, c0_.safeDelete AS safeDelete_2, c0_._created AS _created_3, c0_._modified AS _modified_4, c0_._deleted AS _deleted_5, c0_.domain AS domain_6 FROM CdsDomain c0_ WHERE c0_.safeDelete <> 1 ORDER BY c0_.id ASC LIMIT 10

when will use entity classname without namespace
$entity->getClassName();//   ->   CdsDomain

query looks:
'SELECT c0_.name AS name_0, c0_.id AS id_1, c0_.safeDelete AS safeDelete_2, c0_._created AS _created_3, c0_._modified AS _modified_4, c0_._deleted AS _deleted_5, c0_.server AS server_6 FROM CdsDomain c0_ WHERE c0_.safeDelete <> 1 LIMIT 10

difference is
c0_.domain AS domain_6` instead of right `c0_.server AS server_6

Annotation mapping look like this
    /**
     * @ORM\ManyToOne(targetEntity="Cds\Library\Mysql\Entity\CdsServer", inversedBy="domain", fetch="EAGER")
     * @ORM\JoinColumn(name="server", referencedColumnName="id", nullable=false)
     */
    protected $server;

@mirekratman I don't understand "when will use entity classname without namespace". What do you mean by "use" here? What is the $entity->getClassName() call?

@Ocramius Hi, $entity->getClassName() just return classname of entity simply using
return (string)get_class($this); which returns Cds\\Library\\Mysql\\Entity\\CdsDomain
Regarding - "when will use"  - wanted to say that problem happens when method ->orderBy() gets as a $sort parameter a strng which contains classname with namespace Cds\\Library\\Mysql\\Entity\\CdsDomain instead of a simple string like CdsDomain.
I will prepare a simple entities and service to show the problem and will share a link here.

@mirekratman but why are you using the class name as alias name on the query builder?

@lcobucci good question - I can use any string like abc of course, I've just used classname.
Strange thing is Ive already did a separate unit tests but there I cannot simulate the issue yet.
In my project problem still exists - thats Ive also double checking my project.

The unit test (exact copy of the entities/repositories and method from service) can be fount here - https://github.com/mirekratman/tests - it passes corectly the test. The method with existing issue in my project is located in $this->domainService->getWrong(....). Still looking how to replicate this.

Because Ive tried many times to replicate this issue and I couldnt do it, please close the ticket.
When I will find what is the problem then share more info in new ticket.
Thank you


Breaks on Django 1.9

Hi,
When using Aldjemy 0.4.3 with Django 1.9 I get the following error:
Traceback (most recent call last):
  File "/opt/pycharm-5/helpers/pycharm/django_test_manage.py", line 53, in <module>
    django.setup()
  File "/home/adam/.virtualenvs/fleetv2/local/lib/python2.7/site-packages/django/__init__.py", line 18, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/home/adam/.virtualenvs/fleetv2/local/lib/python2.7/site-packages/django/apps/registry.py", line 115, in populate
    app_config.ready()
  File "/home/adam/.virtualenvs/fleetv2/local/lib/python2.7/site-packages/aldjemy/apps.py", line 17, in ready
    prepare_models()
  File "/home/adam/.virtualenvs/fleetv2/local/lib/python2.7/site-packages/aldjemy/orm.py", line 86, in prepare_models
    tables = get_tables()
  File "/home/adam/.virtualenvs/fleetv2/local/lib/python2.7/site-packages/aldjemy/core.py", line 67, in get_tables
    generate_tables(get_meta())
  File "/home/adam/.virtualenvs/fleetv2/local/lib/python2.7/site-packages/aldjemy/table.py", line 79, in generate_tables
    typ = DATA_TYPES[internal_type](field)
  File "/home/adam/.virtualenvs/fleetv2/local/lib/python2.7/site-packages/aldjemy/types.py", line 19, in foreign_key
    parent_model = field.related.model
AttributeError: 'ManyToOneRel' object has no attribute 'related'

Can you check 0.5.0-rc1?

Hi.
I have the same problem with:
aldjemy==0.4.3
Django==1.9
SQLAlchemy==1.0.11

Ok, so will release then. Its hard to keep this project alive, since I does not work with django for a long time.

releases 0.5.0 to pypi.


Enhancement: ability to detect when operation has completed

When rendering a large page to PDF, it may take a few seconds for the browser to pop up a "save as" dialog to the user. I would like to show a spinner during this time and hide the spinner when jsPDF completes.
Currently there is no way to detect when this happens.
It would be nice if "save()" could return a Promise that will complete when the operation has finished.

Are you using the latest version of jsPDF?

Yes, this affects current master, 7d9f141

Duplicate of #540


Bump semver to fix vulnerability

CVE-2022-25883 (CVSS3: 7.5) Severity: (HIGH)
Component: semver-5.3.0 (Transitive)
Fix: Upgrade to version semver - 5.7.2, 6.3.1, 7.5.2

@rayhammond Sorry I didin't see you had done this before me. I'm happy to rebase my PR on yours, if you like

@rayhammond Sorry I didn't see you had done this before me. I'm happy to rebase my PR on yours, if you like

I'm happy either way as long as the vulnerability get patched.  If your PR gets merged first; just closed this one.

Hi guys, thanks a lot for working on this. I can see it's been a while and these didn't get in. Would it be possible to merge this? Kudos to you for maintaining this :pray:


Blender 2.8

is not working in blender 2.8, I cant enable it.

https://gyazo.com/1e8dfffe5e008bdeaa02ffda82fd39fe

Use the development version of cats for Blender 2.8

it was my bad I had it installed in the 2.79 and it made conflicts and errors somehow


[WIP] Update the AR VIR to the new prototype's code

Also updates the fonts.

          1 Error

:no_entry_sign:
No CHANGELOG changes made

          2 Warnings

:warning:
PR is classed as Work in Progress

:warning:
Needs testing on a Phone if change is non-trivial

  Generated by :no_entry_sign: Danger

OK, gonna start working on getting this mergable

Pretty sure this'll build now

Cool, will also add some comments on the ARAugmentedVIRSceneController

Nice one! 🍾


How I would do it

You can probably get a couple useful things from this just by looking at it.  If you have questions, I'd be happy to discuss the why's and wherefore's.
I don't expect you to merge this, just wanted to share how I approach things like this.

@binaryphile Awesome, thank you for taking the time to do this!

@binaryphile Awesome, thank you for taking the time to do this!

Sure thing.
Three tricks I employ.  The first is setting the field separator IFS to a newline.  This turns off word-splitting on space and tabs when a variable is expanded.  When you don't have to worry about that, you don't need to quote all of your variable expansions, which makes the code less noisy and more readable.
The second is to put all code into functions, called by a main function.  Before invoking main at the end of the script, throw in a return statement.  If the file is being called as a script, return isn't a valid command and throws an error, then script keeps executing and main gets called (the 2>/dev/null on the return statement suppresses the error message).  If the file is sourced like: source myfile, then return is a valid statement and it stops execution before calling main.  That means all of your functions and globals are loaded into the shell, and you can call them at the command line to test them without having to run the whole script.
Last trick is to turn on errexit and nounset options.  That way if the script runs into a problem, it stops and tells you where the problem happened rather than continuing on in a broken manner.  This comes after the return statement so that when you source the file, it doesn't turn on those options in your interactive session, which could accidentally exit you out of the session when you didn't mean to.

Sure thing.
Three tricks I employ.  The first is setting the field separator IFS to a newline.  This turns off word-splitting on space and tabs when a variable is expanded.  When you don't have to worry about that, you don't need to quote all of your variable expansions, which makes the code less noisy and more readable.
The second is to put all code into functions, called by a main function.  Before invoking main at the end of the script, throw in a return statement.  If the file is being called as a script, return isn't a valid command and throws an error, then script keeps executing and main gets called (the 2>/dev/null on the return statement suppresses the error message).  If the file is sourced like: source myfile, then return is a valid statement and it stops execution before calling main.  That means all of your functions and globals are loaded into the shell, and you can call them at the command line to test them without having to run the whole script.
Last trick is to turn on errexit and nounset options.  That way if the script runs into a problem, it stops and tells you where the problem happened rather than continuing on in a broken manner.  This comes after the return statement so that when you source the file, it doesn't turn on those options in your interactive session, which could accidentally exit you out of the session when you didn't mean to.


dnsmasq: Make unexpected resolvconf integration separately opt-in

Motivation for this change
Currently, if you set services.dnsmasq.resolveLocalQueries = true; (the default), then dnsmasq will not only resolve DNS queries for the local system, but it will also make it so that any resolvconf-provided DNS servers are injected into dnsmasq's list of upstream DNS servers.
This behaviour was introduced way back in #3745, but is problematic on several fronts:

It isn't documented by the option description, at all.
It probably should not be enabled by default (while potentially useful, using dnsmasq with DHCP-supplied upstreams is not that common a use-case).
It can't be separately configured -- while the user can disable use of the resolv-file directive by adding no-resolv to extraConfig, there is no way to disable reading of the resolvconf-produced /etc/dnsmasq-conf.conf file. On my system at least, this file configures dnsmasq to resolve queries for my system's search domain using the DHCP-supplied server.

This PR migrates the functionality to its own, documented, opt-in configuration option. The only downside to this is that it will change the behaviour on existing systems.
Things done

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

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

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

Makes sense to provide a new setting I believe, and I agree the default should be changed. I am a bit hesitant as to how to change it though. I would say let's keep the current behavior and wait a bit before changing it, maybe mention it in 19.09 release notes ?

I'm still carrying this patch around on several systems where the previous behaviour is unwanted.

Another issue: resolveLocalQueries enables networking.resolvconf.useLocalResolver which prepends <IP_ADDRESS> to nameserver list. Dnsmasq also sets networking.nameservers here. Isn't this a duplicate?

I'd still like to get this merged.
@bb2020 I suppose, but that's not related to the issue this solves.

Yes, right. I just wanted to mention it when I saw this issue about resolveLocalQueries.

I'd still like this to be merged...

It probably should not be enabled by default (while potentially useful, using dnsmasq with DHCP-supplied upstreams is not that common a use-case).

This is actually what most users do; using dnsmasq as DNS cache for faster DNS resolving. Thus it should be enabled by default.

@bb2020 the quoted line is not relevant to whether or not dnsmasq is being used as a DNS cache, only to whether or not it sources DNS servers from an upstream DHCP server (via resolvconf).
As for whether or not it is "what most users do": I have only anecdotal evidence, in that I have yet to see someone deliberately using resolv-conf supplied DNS servers in dnsmasq, and I have seen many people deliberately using explicitly-configured DNS servers in dnsmasq instead. Is your own assertion also based on anecdotal evidence, or do you have some sort of harder data backing it?

I agree there is a bug here because it just merges upstream DNS with local DNS.


Implement "asynchronously" part of CHA-RL1h5

Split from #28
CHA-RL1h5 says that the detaching of all non-failed channels should happen "asynchronously with respect to CHA-RL1h4" (in my current implementation, it happens as part of the ATTACH operation, thus not fulfilling this spec point)
┆Issue is synchronized with this Jira Story by Unito

Awaiting input on spec from @AndyTWF in https://github.com/ably/specification/issues/253.


Reply to: Screenshots/GIf not visible of 3D flip card issue

What type of PR is this? (check all applicable)

[ ] ♻ Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 👷 Optimization
[ ] 📝 Documentation Update
[ ] 🔖 Release
[ ] 🚩 Other

Description
Related Tickets & Documents
Mobile & Desktop Screenshots/Recordings (if there are UI changes)
Added tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 readme
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Thank you

Nice work


Snapshot compare fails with one line overriding another in snapshot

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
We perform snapshot testing with svg.

npm test --> snapshots are created
npm test --> fails

 @@ -70,12 +70,11 @@
             viewBox="0 0 20 20"
             x="0px"
             xmlns="http://www.w3.org/2000/svg"
             y="0px">
             <path
      -        d="M15,9H8.4l2.3-2.3c0.4-0.4,0.4-1,0-1.4s-1-0.4-1.4,0l-4,4c-0.4,0.4-0.4,1,0,1.4l4,4C9.5,14.9,9.7,15,10,15
      - s0.5-0.1,0.7-0.3c0.4-0.4,0.4-1,0-1.4L8.4,11H15c0.6,0,1-0.4,1-1S15.6,9,15,9z" />
 s0.5-0.1,0.7-0.3c0.4-0.4,0.4-1,0-1.4L8.4,11H15c0.6,0,1-0.4,1-1S15.6,9,15,9z" />.4-0.4,1,0,1.4l4,4C9.5,14.9,9.7,15,10,15 **Weird line here like the snapshot replaced the first chars of the previous line**
           </svg>
         </symbol>
         <symbol
           id="talend-arrowright">
           <svg

npm test -- -u --> tests pass, but snapshot do not change
npm test --> same error

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal repository on GitHub that we can npm install and npm test.
I cleaned the repo to avoid polluting the test, you can reproduce it by cloning this repo (on this branch) : repo on the cleaned branch
What is the expected behavior?
The saved snapshot is correct, but the test should not fail.
Run Jest again with --debug and provide the full configuration it prints. Please mention your node and npm version and operating system.
Operating system : Fedora 22
Npm: 3.10.8
Node: 6.9.1
Jest --debug log:
> jest "--debug"

jest version = 17.0.0
test framework = jasmine2
config = {
  "transform": [
    [
      ".*",
      "/home/jsomsanith/Documents/dev/workspace/react-talend-components/node_modules/babel-jest"
    ]
  ],
  "testRegex": "src/.*\\.test\\.js$",
  "moduleNameMapper": [
    [
      "^.+\\.(css|scss)$",
      "/home/jsomsanith/Documents/dev/workspace/react-talend-components/test/styleMock.js"
    ]
  ],
  "rootDir": "/home/jsomsanith/Documents/dev/workspace/react-talend-components",
  "name": "-home-jsomsanith-Documents-dev-workspace-react-talend-components",
  "setupFiles": [
    "/home/jsomsanith/Documents/dev/workspace/react-talend-components/node_modules/babel-polyfill/lib/index.js"
  ],
  "testRunner": "/home/jsomsanith/Documents/dev/workspace/react-talend-components/node_modules/jest-jasmine2/build/index.js",
  "usesBabelJest": true,
  "automock": false,
  "bail": false,
  "browser": false,
  "cacheDirectory": "/tmp/jest",
  "coveragePathIgnorePatterns": [
    "/node_modules/"
  ],
  "coverageReporters": [
    "json",
    "text",
    "lcov",
    "clover"
  ],
  "expand": false,
  "globals": {},
  "haste": {
    "providesModuleNodeModules": []
  },
  "mocksPattern": "__mocks__",
  "moduleDirectories": [
    "node_modules"
  ],
  "moduleFileExtensions": [
    "js",
    "json",
    "jsx",
    "node"
  ],
  "modulePathIgnorePatterns": [],
  "noStackTrace": false,
  "notify": false,
  "preset": null,
  "resetMocks": false,
  "resetModules": false,
  "snapshotSerializers": [],
  "testEnvironment": "jest-environment-jsdom",
  "testPathDirs": [
    "/home/jsomsanith/Documents/dev/workspace/react-talend-components"
  ],
  "testPathIgnorePatterns": [
    "/node_modules/"
  ],
  "testURL": "about:blank",
  "timers": "real",
  "transformIgnorePatterns": [
    "/node_modules/"
  ],
  "useStderr": false,
  "verbose": null,
  "watch": false,
  "cache": true,
  "watchman": true,
  "testcheckOptions": {
    "times": 100,
    "maxSize": 200
  }
}
 FAIL  src/IconsProvider/IconsProvider.test.js
  ● IconsProvider › should render default talend-icons

    Error
      Error: expect(value).toMatchSnapshot()

      Received value does not match stored snapshot 1.

      - Snapshot
      + Received

      @@ -70,12 +70,11 @@
             viewBox="0 0 20 20"
             x="0px"
             xmlns="http://www.w3.org/2000/svg"
             y="0px">
             <path
      -        d="M15,9H8.4l2.3-2.3c0.4-0.4,0.4-1,0-1.4s-1-0.4-1.4,0l-4,4c-0.4,0.4-0.4,1,0,1.4l4,4C9.5,14.9,9.7,15,10,15
      - s0.5-0.1,0.7-0.3c0.4-0.4,0.4-1,0-1.4L8.4,11H15c0.6,0,1-0.4,1-1S15.6,9,15,9z" />
 s0.5-0.1,0.7-0.3c0.4-0.4,0.4-1,0-1.4L8.4,11H15c0.6,0,1-0.4,1-1S15.6,9,15,9z" />.4-0.4,1,0,1.4l4,4C9.5,14.9,9.7,15,10,15
           </svg>
         </symbol>
         <symbol
           id="talend-arrowright">
           <svg
      @@ -83,12 +82,11 @@
             viewBox="0 0 20 20"
             x="0px"
             xmlns="http://www.w3.org/2000/svg"
             y="0px">
             <path
      -        d="M5,9h6.6L9.3,6.7c-0.4-0.4-0.4-1,0-1.4s1-0.4,1.4,0l4,4c0.4,0.4,0.4,1,0,1.4l-4,4C10.5,14.9,10.3,15,10,15
      - s-0.5-0.1-0.7-0.3c-0.4-0.4-0.4-1,0-1.4l2.3-2.3H5c-0.6,0-1-0.4-1-1S4.4,9,5,9z" />
 s-0.5-0.1-0.7-0.3c-0.4-0.4-0.4-1,0-1.4l2.3-2.3H5c-0.6,0-1-0.4-1-1S4.4,9,5,9z" />.4,1,0,1.4l-4,4C10.5,14.9,10.3,15,10,15
           </svg>
         </symbol>
         <symbol
           id="talend-badge">
           <svg
      at Object.<anonymous> (src/IconsProvider/IconsProvider.test.js:9:19)

  IconsProvider
    ✕ should render default talend-icons (21ms)

Snapshot Summary
 › 1 snapshot test failed in 1 test suite. Inspect your code changes or run with `npm test -- -u` to update them.

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 total
Snapshots:   1 failed, 1 total
Time:        0.961s, estimated 1s
Ran all test suites.
npm ERR! Test failed.  See above for more details.

Thank you in advance for your time, Jest rocks !

Hey @jsomsanith!
Cloned the repo and it works. Does this still happen to you?

Hi @thymikee,
Yes still happens. Are you on the branch from the link in the issue description ? bug/jest-snapshot-issue

Yup, first i tried on master, but then on the branch. Have you tried to remove node_modules and install deps once again?
For reference, I was on node v6.9.1, macOS Sierra.

Oh ? yes I still reproduce it on the branch:
$ rm -Rf node_modules
$ yarn cache clean
$ yarn
$ yarn test

I also tried again with npm to pull the last release (17.0.3)
$ rm -Rf node_modules
$ npm cache clean
$ npm i
$ npm test

Same result.
I use the same node version (6.9.1) on macOS El Capitan. But this was reproduced on Fedora and Ubuntu.

This time I got it right!
Currently snapshots have known problem with escaping single \r character, which is being inserted in 2 of your icons – talend-arrowright and talend-arrowleft. As a temporary solution, please remove \r from those icons if possible.
Here's the corresponding issue: #1890. Feel welcome to send us PR on this if you like :)

Thank you @thymikee, I'll try that.
If I have time, I'll try to look at the code :)


Updated and moved lambda scripts into examples folder

At dinner - Teddy has pull powers so I will leave it to him to review the
file changes and pull it.
On Thursday, February 26, 2015, bschwagg<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/PyClass/PyClassLessons/pull/10
Commit Summary

added a sort function as a bonus. Cleaned up some comments
added a more useful GUI example with lambdas. Changed the OO
strategy pattern to use a game instead of logging. Reordered examples
Moved files to examples folder.

File Changes

R lessons/lambda_functions/examples/ex1.py
https://github.com/PyClass/PyClassLessons/pull/10/files#diff-0 (0)
R lessons/lambda_functions/examples/ex2.py
https://github.com/PyClass/PyClassLessons/pull/10/files#diff-1 (28)
R lessons/lambda_functions/examples/ex3.py
https://github.com/PyClass/PyClassLessons/pull/10/files#diff-2 (75)

Patch Links:

https://github.com/PyClass/PyClassLessons/pull/10.patch
https://github.com/PyClass/PyClassLessons/pull/10.diff

—
Reply to this email directly or view it on GitHub
https://github.com/PyClass/PyClassLessons/pull/10.

--
(Sent from cellphone)


#259 | Profile Page

The profile page is not done yet, because of there are some service that still not finished. But it already work.
Note for myself :

user link is still not there
list of location is still not there

@meomancer can you address the concerns I raised above so we can merge?

@timlinux

@meomancer we still need migration rules in place in order to be able to merge this request.

Thanks!


Filter expectations when clearing

I tried to filter expectations when clearing them like this:
yield client.clear({
		httpRequest: {
				path: '.*',
				cookies: [
						{
								name: cookie.name,
								value: cookie.value
						}
				]
		}
});

but Mockserver log shows:
INFO o.m.m.MockServerHandler clearing expectations and request logs that match:

	null

Is it possible to clear only expectations that match certain conditions?

that looks like javascript, are you using the mockServeClient.js file or mockserver-client-node npm module?
Both of these clear functions only currently support providing the path to clear on as follows:
client.clear('/somePath');

I could however extend this if required, so that the clear function supports a path or request matcher.

I'm using `mockserver-client-node

https://github.com/jamesdbloom/mockserver-client-node/commit/a5772a5bc8d13050886748ece12d1eb86ef30d50
I have added this support to both javascript clients (see mockserver-client-node 1.0.6) so I'm going to close this issue.

Cool. Thanks a lot for your quick response. I'll try very soon.

Working great for me. Thank you very much, @jamesdbloom.


request: Modify the Windows clipboard in some way? (xclip/xsel)

Would it be possible, in one way or another, to from within msys2 modify the Windows clipboard in the same way xclip and xsel can do?

I'm unsure about what xclip nor xsel, but I usually simply do:
$ echo 'Wow, clipboarded!' > /dev/clipboard
$ cat /dev/clipboard
Wow, clipboarded!

# Copy something in Windows uisng Ctrl+C
$ cat /dev/clipboard
# Copied content here

Note: Don't copy the cat /dev/clipboard line, type it directly instead. Otherwise, you will have overwritten clipboard content and you would end up with this:
$ cat /dev/clipboard
 cat /dev/clipboard

Don't make same mistake I made :D
The special /dev/clipboard is like any input/ouput device, so you can pipe it to make some nice stuff like cat /dev/clipboard | jq '.users[0]'.
Maybe it's enough for your use case.

Oh yeah; I love you! Suddenly my life became that much easier! Cheers man!

Glad I could make someone happy and hope and will help others :D
Regards,
Matt


Fix python key tracing errors with quantized models

Repro
./torchbench.py --no-skip --python-key -n 1 -k mobilenet_v2_quantized

Partial output:
ERROR:torchdynamo.optimizations.python_key:exception running call_function torch.quantize_per_tensor (inputs_0_, mod_features_0_0_input_scale_0, mod_features_0_0_input_zero_point_0, torch.quint8) {}
Traceback (most recent call last):
  File "/home/jansel/torchdynamo/torchdynamo/optimizations/python_key.py", line 77, in run_node
    result = super().run_node(n)
  File "/home/jansel/pytorch/torch/fx/interpreter.py", line 147, in run_node
    return getattr(self, n.op)(n.target, args, kwargs)
  File "/home/jansel/pytorch/torch/fx/interpreter.py", line 219, in call_function
    return target(*args, **kwargs)
  File "/home/jansel/functorch/functorch/_src/python_key.py", line 112, in __torch_dispatch__
    return wrap_with_proxy(real_out, proxy_out)
  File "/home/jansel/functorch/functorch/_src/python_key.py", line 104, in wrap_with_proxy
    return PythonTensor(e, proxy)
  File "/home/jansel/functorch/functorch/_src/python_key.py", line 60, in __new__
    proxy.node.meta['tensor_meta'] = _extract_tensor_metadata(r)
  File "/home/jansel/pytorch/torch/fx/passes/shape_prop.py", line 48, in _extract_tensor_metadata
    qscheme = result.qscheme()
RuntimeError: toIValue() cannot handle converting to type: QScheme

This effects 2 quantized modules with the same error.
cc @Chillee @anijain2305

1.13 P0 is to fall back quietly.
Long term we want this to actually work / speed up.

Root cause here is fairly bizarre. It appears that for some of the operators in the TorchDynamo resulting graph, operator(PythonTensor) is resulting in a regular tensor instead of a PythonTensor.
Ah, might have found it.
https://github.com/pytorch/pytorch/pull/50859
This PR breaks(?) composite compliance, as it forces aten::flatten (a composite op) to call native::reshape, which prevents redispatching.
But... this seems to work fine with normal tensors, it's only when it's a quantized tensor that it fails. I'm guessing there's an operator somewhere that has some different implementation for QuantCPU? not sure.
Minimal repro
def f(x):
  out = torch.flatten(x)
  return out

val = torch.quantize_per_tensor(torch.randn(2,3,4,5), 0.1, 10, torch.quint8)
print(make_fx(f)(val))

results in
def forward(self, x_1):
    _reshape_alias = torch.ops.aten._reshape_alias(x_1, [120], [1]);  x_1 = None
    _tensor_constant0 = self._tensor_constant0
    return _tensor_constant0

on hold for now.

Not sure why, but this seems to work for me now.


Drawing vectors?

Hi. This is really neat!  I thought I'd try re-doing in @iooxa/article an RLC circuit simulator that I once wrote in Java.
Here's an image of the UI I had:
.
I've been able to set up the sliders, the displayed variables and the graphs.
My question is, how do we draw vectors, i.e. lines with arrows on the ends?
Thanks.  (I tried defining defs and marker but couldn't get it to work.)
PS- While we're at it: Logarithmic axes for graphs?

PS - I got it mostly working the way I like!  Just would like the vectors to look like vectors:
Demo: https://hedges.belmont.edu/~shawley/physapplets/seriesrlc.html

So cool that you got this working -- saw that you converted a few other things over as well! Are these used in university courses? Would love to know more!
Absolutely agree that vector heads are necessary, will leave this open!

Hey Rowan.  Yes to college courses: over ~15 years I've written various interactive teaching tools but over time things like Flash & Java applets have stopped running in browsers.
This utility will be used in my Electronics & Circuit Theory course at Belmont University in Spring 2021.
Actually, that was so easy that yesterday I re-did a similar demo on Simple Harmonic Motion for my Acoustics course: https://hedges.belmont.edu/~shawley/physapplets/shm.html
I appreciate how iooxa/article allowed me to focus on the behind-the-scenes math instead of me having to build all the graphics & interaction, all so that my students can focus on the graphics & interaction to gain insights that the math doesn't easily show! ;-)
Working on an interactive eBook next. Not sure how many JS libraries I'll need (e.g. using vis.js for 3D), but I appreciate how your "scoping" functionality allows one to keep things separate.

I found that one in browsing through your apps pages, I really love the hiding and showing intro text and equations. I have tried to do something similar here:
https://row1.ca/probability-of-a-spill
By having a slider that changes from a 2min interactive read/graphic to an in-depth proof.
Glad you found the scoping easy, and makes sense to you!
One of the other things I am in the process of getting off the ground is iooxa.com. It aims to support these sort of interactive textbook-like environments, and also take care of some of the collaboration aspects that come with re-using, writing and getting help from other folks. We found that github was a bit too challenging to most of the people who would actually write content, learning from developing a geophysics textbook.
I would love to chat/hear more about your eBook plans if you would be up for it!
If you are up for slack: https://slack.iooxa.dev or<EMAIL_ADDRESS>

Sounds great! Emailing...


Delete Imported Folder Too?

Hello, first of all thank you for this awesome tool, I got a lot of archived downloads and this really helps me a lot :)
Everything works as expected until now, but I realized the folder.delete_after doesn't delete the imported folder after 5m (only the files) and I ended up with a lot of empty folders in my download folder. Why is the folder not also be deleted with the files? Thank you.

If you set move_back to false it should delete the folder too. Can you share your folder config?

If you set move_back to false it should delete the folder too. Can you share your folder config?
I don't modify move_back, so it should be defaulted to false right?
I'm using kubernetes and here are the environment variables:

apiVersion: v1
kind: Secret
metadata:
  name: unpackerr-secrets
  labels:
    app.kubernetes.io/name: unpackerr
type: Opaque
stringData:
  UN_SONARR_0_URL: http://sonarr-svc.default.svc.cluster.local:8989
  UN_SONARR_0_API_KEY: secret
  UN_SONARR_0_PATHS_0: /downloads/Completed
  UN_RADARR_0_URL: http://radarr-svc.default.svc.cluster.local:7878
  UN_RADARR_0_API_KEY: secret
  UN_RADARR_0_PATHS_0: /downloads/Completed
  UN_LIDARR_0_URL: http://lidarr-svc.default.svc.cluster.local:8686
  UN_LIDARR_0_API_KEY: secret
  UN_LIDARR_0_PATHS_0: /downloads/Completed
  UN_FOLDER_0_PATH: /downloads/Completed

I guess I'm confused about what empty folders are being left behind? Your download client downloads stuff into a folder, unpackerr extracts that stuff, Sonarr imports it, and then .... your torrent app seeds it. What folder is left empty?

I guess I'm confused about what empty folders are being left behind? Your download client downloads stuff into a folder, unpackerr extracts that stuff, Sonarr imports it, and then .... your torrent app seeds it. What folder is left empty?

Sorry I wasn't clear enough. Those downloaded stuffs are no longer seeding because they have exceeded the seeding ratio I set in the download client (I use Transmission), the status in download client is Paused. I set up Unpackerr after I have so many archived files failed to get imported in Radarr.
So here's what happened after I set up Unpackerr:

Unpackerr found a lot of archived files in the download folder.
Unpackerr queued them and started to extract them.
Unpackerr finished extracting and Radarr imported the files and delete the files from torrent client list.
Unpackerr delete the files in that folder after 5m.
Now the download folder contains only the empty folder.

Unpackerr delete the files in that folder after 5m.

I don't think this is right. Unpackerr only deletes the files it extracted. It won't delete the files that your client downloaded (with your settings). Your download client likely deleted the files it downloaded, and then left you an empty folder.

Unpackerr delete the files in that folder after 5m.

I don't think this is right. Unpackerr only deletes the files it extracted. It won't delete the files that your client downloaded (with your settings). Your download client likely deleted the files it downloaded, and then left you an empty folder.
EDIT: The Unpackerr logs always tell you what it's deleting. Please check those and provide them if they indicate Unpackerr deleted rar files.

Yes, point number 3 Radarr imported and deleted the files that got downloaded by Transmission. Point number 4 Unpackerr delete the remaining files it created but left the folder there.

Radarr imported and deleted the files that got downloaded by Transmission.

If Radarr imported what Transmission downloaded, then Unpackerr did not extract anything. If Unpackerr extracted the files that Transmission downloaded, then Radarr imported what unpackerr extracted, not what Transmission downloaded.

Point number 4 Unpackerr delete the remaining files it created but left the folder there.

That is accurate behavior. Unpackerr does not delete things it did not create.

But that's what happened. Sorry I don't have the logs because I restarted the container and I only logged in the console. But these are the files that Unpackerr deleted after 5minutes (from my nfs server terminal history):
╰ ls -la /export/Kubernetes-Volume/default-shared-downloads-pvc-778cb67f-123e-4611-96b9-48f9d0447583/Completed/The.Iceman.2012.LIMITED.1080p.BluRay.x264-GECKOS
total 15M
drwxr-xr-x 1 budiman budiman  306 Apr 18 15:07 .
drwxr-xr-x 1 budiman budiman  336 Apr 18 15:07 ..
-rw-r--r-- 1 budiman budiman 137K Apr 18 15:03 The.Iceman.2012.1080p.BluRay.x264-GECKOS.idx
-rw-r--r-- 1 budiman budiman  15M Apr 18 15:03 The.Iceman.2012.1080p.BluRay.x264-GECKOS.sub
-rw-r--r-- 1 budiman budiman 9.0K Apr 18 15:03 _unpackerred.The.Iceman.2012.LIMITED.1080p.BluRay.x264-GECKOS.txt

Those all look like files it extracted, so it would have deleted them. The question, I think, is, what deleted your rar files? You hypothesized that Radarr did it, but I find it highly unlikely that Radarr would delete a bunch rar files that it obviously didn't import. The Radarr logs would tell you that, those should be saved to appdata. Unpackerr didn't delete the rar files, so.. that leaves your download client. Does it have a lot you can look at?

and, pro tip, set the log file to /download/unpackerr.log

Like I said though you can totally make Unpackerr delete your Transmission files but it will do so even if Transmission is still seeding. There is no way to control that. If you want that configuration set UN_SONARR_0_DELETE_ORIG=true Same for the other apps. This setting will make Unpackerr delete the download folder instead of the extracted files within it.

Those all look like files it extracted, so it would have deleted them. The question, I think, is, what deleted your rar files? You hypothesized that Radarr did it, but I find it highly unlikely that Radarr would delete a bunch rar files that it obviously didn't import. The Radarr logs would tell you that, those should be saved to appdata. Unpackerr didn't delete the rar files, so.. that leaves your download client. Does it have a log you can look at?

Looks like you are right, Radarr didn't delete the archived files.
[Info] MovieService: Assigning file [The Iceman (2012).mkv] to movie [[The Iceman (2012)][tt1491044, 68812]]
[Warn] DownloadedMovieImportService: RAR file detected, will require manual cleanup
[Info] DiskScanService: Scanning disk for The Iceman
[Info] DiskScanService: Completed scanning disk for The Iceman
[Info] ExistingMetadataImporter: Found 0 existing metadata files
[Info] ExistingSubtitleImporter: Found 1 existing subtitle files
[Info] ExistingOtherExtraImporter: Found 0 existing other extra files
[Info] ExistingExtraFileService: Found 0 extra files

and Transmission logs shows nothing too:
╰ kubectl logs transmission-5df9d8c467-5m8k9 | grep Iceman
[2021-04-18 02:20:46.394] The.Iceman.2012.LIMITED.1080p.BluRay.x264-GECKOS Scrape error: Could not connect to tracker (/home/buildozer/aports/community/transmission/src/transmission-3.00/libtransmission/announcer.c:1392)
[2021-04-18 02:20:46.394] The.Iceman.2012.LIMITED.1080p.BluRay.x264-GECKOS Retrying scrape in 0 seconds. (/home/buildozer/aports/community/transmission/src/transmission-3.00/libtransmission/announcer.c:1401)
[2021-04-18 15:07:34.393] The.Iceman.2012.LIMITED.1080p.BluRay.x264-GECKOS Removing torrent (/home/buildozer/aports/community/transmission/src/transmission-3.00/libtransmission/torrent.c:2078)
[2021-04-18 15:07:34.393] The.Iceman.2012.LIMITED.1080p.BluRay.x264-GECKOS Pausing (/home/buildozer/aports/community/transmission/src/transmission-3.00/libtransmission/torrent.c:2022)

I don't know how did the RAR files gone.
I will try to set up a log file now and observe for a little while.

Yeah that's strange. I'm not sure what to say here. The unpackerr logs would tell us the whole story. :( Definitely get that setup!

Debug set to on, Log files set to download folder. Will update when I found something. Thank you for answering :)

Any updates?

Sorry I forgot to follow this up. I can't seem to reproduce it anymore. I will close this issue.


REPOSITORIES.md file

Description
Adds the REPOSITORIES.md file. This document provides definitions of the scopes and statuses of Kubewarden repositories.
Fix https://github.com/kubewarden/kubewarden-controller/issues/684

I don't think this fixes https://github.com/kubewarden/kubewarden-controller/issues/684, while it explains the repository groups, we are still missing a list table of them as in https://github.com/falcosecurity/evolution/blob/main/README.md#repositories.

I don't think this fixes kubewarden/kubewarden-controller#684, while it explains the repository groups, we are still missing a list table of them as in https://github.com/falcosecurity/evolution/blob/main/README.md#repositories. I think we could start by putting everything into kubewarden/community, without separating things into yet another repo as falco does with falcosecurity/evolution.

I'm aware that the table is missing. However, as we are following the falco example, I decided to keep the table for later when  we add the README file. I haven't done this here because I'm writing the README file and noticed that we miss other documentation that I would like add later to keep the PR small enough to be quickly reviewed.


Tag releases as Github releases

Hello,
I think it would be nice if you could tag your releases in git and mark them als releases on Github. That way it would be easier to find a specific release.
Kind regards
Tobias

Sure, I can start tagging new releases moving forward. What's your particular use-case if you don't mind me asking? @tobiasge

I wanted to start an app from the latest release. So I had to look through the commit history to find the corresponding commit.
If they are tagged it's easier to find them.
Thanks.


Warning text does not support multi-line

Description of the issue
https://design-system.service.gov.uk/components/warning-text/
The warning text component does not support text that drops onto multiple lines
Steps to reproduce the issue
The icon looks a bit strange when the element is showing multiple lines of information

    !

        Notice
        Lorem ipsum dolor sit amet, consectetur adipisicing elit. Accusamus alias dolores illo praesentium, quaerat ratione reiciendis voluptas. Corporis, delectus, enim error et illum maxime, officiis quam qui quibusdam repudiandae tempora?

Actual vs expected behaviour

Hi @Jamesking56,
Can you elaborate on what you mean 'the icon looks a bit strange'? As far as I can tell from your screenshot this is working as designed.

Hi @Jamesking56,
Can you elaborate on what you mean 'the icon looks a bit strange'? As far as I can tell from your screenshot this is working as designed.

It looks high up stuck in the top left?

This is currently working as designed – there was a change made back in 2019 to vertically align the icon with the first line of the text:

The primary purpose of the this PR was to align the icon with the top line, so that when the content wraps the icon doesn't vertically align with the entire block and potentially end up out of sight when the user starts reading the warning text.

Have you seen any evidence that users are confused by this? What would you expect the component to do when there are multiple lines of text?

I would expect it to look like this:

I haven't seen evidence of user confusion but if there was text above that isn't included in the warning, wouldn't the high up icon then cause confusion which parts of the text it is for?

Having the icon vertically centred is fine if you assume a certain quantity of text and a certain amount of horizontal space. It starts to quite goofy in situations where those circumstances are not ideal—such as on mobile devices or for users who use zooming or magnification tools for accessibility purposes.

Hi @Jamesking56
Thanks for raising this, you're not the first to bring it up!
We placed the icon at the top left because it ensures the warning icon is within the path of people using screen magnification tools. The icon is important for understanding that the text is a warning, and not just some bold text. For longer blocks of warning text, you could miss that context.
I personally agree that it looks a bit strange when it sticks out above the top of a large block of text. I think ideally it'd be vertically centre-aligned when there's just one line of text, and aligned to the top of the text block when it runs over 2 or more lines. But we're not aware of a way to do this reliably in CSS. So we decided the accessibility consideration trumped the aesthetic one in this case.
Hope that makes sense.

@christopherthomasdesign Excellent explanation! I agree accessibility should triumph here :+1:

@Jamesking56 Given the information above, are you happy for us to close this issue?

@Jamesking56 Given the information above, are you happy for us to close this issue?

Yes can be closed


3.1.b2 issues with Sierra

3.1.b2 stopped working properly for me on a 10.12.6 machine, so I reverted back to an old Spark version 3.0b11 (470) I luckily found on a backup and all is well again.  The 3.1.b2 version didn't respond when I clicked "Start Spark Demon" and many shortcuts I made stopped working including this custom one for Textedit to quickly change colors of sections of text I have selected:
I had it run this with shift-command-B:
tell application "TextEdit"
end tell
tell application "System Events" to tell text area 1 of scroll area 1 of window 1 of process "TextEdit" to if exists then
	set {x, y} to value of attribute "AXSelectedTextRange"
	if x ≤ y then tell application "TextEdit" to set color of document 1's characters x thru y to {0, 0, 65535}
end if

Nothing happens when I did the shortcut with 3.1.b2, but works perfectly with 3.0b11 (470).  I guess I should just stick with the old version that works for now.
I should also mention the interface in 3.1.b2 was jacked up with the "All Application's Hot Keys", etc. button riding up to where the top vertical half of the button was missing.
Does anyone know where one can find older versions of Spark?

also found the new version not working for met today in sierra. version 3.0b10 (470).....
same thing spark daemon button not working, no hotkeys working.
it must have updated automatically and now I can't go back, any links to older versions or Cowicide can you upload the version that works for you??
thanks


As a user, when I select a leaf to read more details, I want the leaf and label highlighted in the tree view so that I can see and remember what I selected.

reviewed and tested in PR #43


question about stft function

When I ran gen_babble_speech.m, there was an error. no stft function . My matlab version is R2016a, Signal Processing Toolbox version is 7.2. which version is your matlab?

The stft function was introduced in MATLAB version R2019a.
Commit 1a5f1a1 contains the stft and istft functions.


UWP Renderer renders inputs regardless of the supportsInteractivity field in HostConfig

Actions are properly ignored, but all inputs are rendered regardless of the configuration.

If the SupportInteractivity setting is false, should it not still render the UI declared, but set the input fields to Read Only, so they can not be interacted with, but are still visible?

Rendering read only input fields would confuse users, and the blame would end up being directed to the host. The current desired design is that it should be up to card authors to know the guidelines for the specific host that will be rendering their cards as to know that they don't support use any of the action or input elements.


Restore original behavior

In prior to #19, #{prefix_highlight} was replaced with empty prompt or prefix prompt or copy prompt. However, #19 changed the behavior so that (empty prompt or prefix prompt) and (copy prompt or sync prompt or none) are displayed.
This PR restores the behavior to show only one of the four possible prompts: prefix, copy, sync, and empty (prioritized in this order).
In addition to this change, , and } in prompts and attribute strings are now escaped as #, and #} to simplify the script. Now ${prefix,copy,sync,empty}_highlight are included in ${prefix,copy,sync,empty}_with_optional_affixes.

I'm so sorry for the delay. That's a great contribution.
Could you please rebase you branch with master? We can merge it afterwards.

Thank you for the reply and I rebased the branch with master.

Thank you!


Various typo fixes

Corrections to various typos discovered whilst browsing https://substrate.io/ecosystem/projects.

@SBalaguer could you review on the improvements?


Add an implementation of Stop-Computer for Linux and OS X.

PR Summary
Adds an implementation of the Stop-Computer cmdlet for Linux and OS X
PR Context
Closes #5448
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None

User-facing changes

[X] Documentation needed

[X] Issue filed: #5448

Testing - New and feature

[X] N/A or can only be tested interactively

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.

I'm pretty sure the Windows CI run is a flake.

@brendandburns I had the CI system rerun the flaky tests and filed: https://github.com/PowerShell/PowerShell/issues/11168

Comments addressed, please re-check.

comments addressed, please take another look.

@SteveL-MSFT comments addressed, please take another look.
Thanks!

@TravisEz13 any chance you can take another look here, I'd love to see this merged since I have a follow up PR to share once it is merged.

@brendandburns Sorry, I've been out of the office.


Add identifier when there are no results

Questions
Answers

Description?
Add identifier when there are no results

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

How to test?
Relative to PrestaShop/PrestaShop#29683

Possible impacts?
N/A

@Progi1984 I believe we could validate this PR by running the tests https://github.com/PrestaShop/PrestaShop/pull/29683 wdyt?

Affirmative, it's dependent.

Thank you @Progi1984 for your work! :-)


Add tests and doc

Hi! As we struggled to use extensions, we added tests + docs. I hope you'll like it!
Thanks for the repo!

Could you please rebase? @aceol Thank you.

This is great. Thank you @aceol


Filtering variables

It would be nice to have an option to somehow select what variables are displayed, so sensitive information like passwords, tokens and API keys could be hidden.

@inovosel All that remains is to figure out what criteria can be used to identify confidential information ))

@yarkovaleksei either using a whitelist for allowed variables or a blacklist for forbidden :)

@inovosel Hmm... That's not a bad idea. I'll probably do that))


FR: Share only email title date from

Hi
I use email sharing (passing content to othr apps) with other apps alot but one issue I have is that it shares the whole body of email and if the whole body is a long email then it is a mess given that it also shares the html code. For instance I share an email to my tasks apps to keep track of it and in most cases the title and the date of email etc is enough information for such tasks. And given that most apps wont be able to render html inside the whole email body looks like a mess in the apps.
My feature request is for having one more share item where it says "share basic" and the other would be "share all/advanced" this way the user can decide what kind of email content he wants to share with other apps.
Or the other solution would be some kind of settings that dictaces how much info should be passed to the other apps.
One solution would be to not to send any html and makse sure only rendered text passed along, if none of the ideas above is implementable.
thanks

I'd rather not add complexity to the share feature. You could write an app that acts as a share target, strips everything from the text that you don't want, then re-shares (maybe directly to the note taking app).

I am not a dev so I cant do that but thanks for the follow up.

@gerroon It might be better to switch to a tasks app that automatically converts emails forwarded to a designated email address into tasks in that app (with the subject line of an email becoming the task name, the body of the email becoming task notes and even the attachments of the email become task attachments).
http://www.makeuseof.com/tag/9-instant-ways-turn-emails-tasks/


Files uploud issue

Hello great program but i have one problem, evrything works fine on my minecraft server but i  just cant seem to uploud plugins,world files in to the server files anyone have a fix for that?  chown for that doesnt work for me.

This is not for issues with the panel. Please visit our Discord.


ReadYamlSequence.java:10-11: Continue implementing and unit...

The puzzle 64-6c1b602e in src/main/java/com/amihaiemil/camel/ReadYamlSequence.java (lines 10-11) has to be resolved: Continue implementing and unit testing this class, one method at a time.
The puzzle was created by amihaiemil on 22-Feb-17.
Estimate: 30 minutes, role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD.

@amihaiemil please pay attention to this new issue.

The puzzle 64-6c1b602e has disappeared from the source code, that's why I closed this issue. //cc @amihaiemil

The puzzle 64-6c1b602e has disappeared from the source code, that's why I closed this issue. //cc @amihaiemil


Add option to make npm install optional

First of all, thanks for the action. It's awesome 🙂
We had a problem using it, though. We use a lerna monorepo and the npm install that you do inside the docker container fails, so I've added an option to avoid running npm install in the container and thus allow people to run their necessary install and/or build scripts in a previous step of their workflow, like this:
jobs:
  eslint:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout Repo
        uses: actions/checkout@v2
      - name: Install dependencies
        run: npm ci
      - name: Run a build step
        run: npm run build
      - uses: bradennapier/eslint-plus-action@v2
        with:
          github-token: ${{secrets.GITHUB_TOKEN}}
          npmInstall: false

https://github.com/frontity/frontity/blob/refactor-analytics-event/.github/workflows/eslint.yml
The option is called npmInstall and has a default of true.
Let me know what you think.

Yeah it makes sense that this is an issue - my goal was to reduce the steps needed to get this working but it makes sense to set this up better - going to see if there may be a more automated way of doing it.

@luisherranz you may want to see https://github.com/bradennapier/eslint-plus-action/issues/6 as well for now

Ok will merge these in, however I am going to see about waiting until this major release I am working on that should make it possible to lint remote forks securely.  I will let you know when released (as v3 most likely).

No problem. I just subscribed to the repo 🙂


More Language Changes

Changes Unknown when pulling 56459bd00e0c4ab05739c833041fbf7a29820587 on blowmage:language-fixes into ** on GoogleCloudPlatform:master**.

Changes Unknown when pulling 56459bd00e0c4ab05739c833041fbf7a29820587 on blowmage:language-fixes into ** on GoogleCloudPlatform:master**.

Changes Unknown when pulling 56459bd00e0c4ab05739c833041fbf7a29820587 on blowmage:language-fixes into ** on GoogleCloudPlatform:master**.

Coverage decreased (-0.02%) to 95.642% when pulling 56459bd00e0c4ab05739c833041fbf7a29820587 on blowmage:language-fixes into f7e239d364b0de2852d794d564e31d8032ae7474 on GoogleCloudPlatform:master.

@quartzmo This contains the fixes needed to get the acceptance tests passing on Travis-CI. Can you review ASAP?

Coverage decreased (-0.01%) to 95.655% when pulling ca5a020920d26a742deff9692bac51b39d052124 on blowmage:language-fixes into f7e239d364b0de2852d794d564e31d8032ae7474 on GoogleCloudPlatform:master.

chan_creds changed to channel. Good sleuthing.


Recommended App "Recommended-Denied-Office-365-ProPlus-1810.xml"

What is this application? Do we have to add it to our WIP policy?

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

ID: 7df27add-73fe-4f0b-7e30-face0f95cda3
Version Independent ID: 3ec647d9-3470-8d8b-a941-d56f6522a35e
Content: Create a Windows Information Protection (WIP) policy with MDM using the Azure portal for Microsoft Intune (Windows 10)
Content Source: windows/security/information-protection/windows-information-protection/create-wip-policy-using-intune-azure.md
Product: w10
GitHub Login: @Justinha
Microsoft Alias: justinha

@Martony78 The Office engineering team recommended that Intune add that app to the recommended list. It denies older versions of Office and other unenlightened Office apps from getting even accidentally allowed through some other rule, as they could cause app compat problems and over-encrypt data if allowed.


Implemented new way of holding the context within the session

Session now holds an Arc pointing to a Pkcs11 instance
Pkcs11::new now returns an Arc

Resolves #57

This works, but just for the sake of clarity I was thinking an internal Arc around just the function_list element, which is what gets cloned into the Session. An outer Arc allows user code to clone the context. That wasn't possible before this change, and I don't know if that behavior goes against any threading restrictions you were assuming. I'm also not totally clear on what role the _pkcs11_lib member is actually playing that might require it to be transferred too.

threading restrictions you were assuming

My only thought on threading restrictions would be that the user would be able to create new sessions in threads they create, based off of the same context.  So the context itself would need to be shared across any thread opened for this purpose.  I made the entire thing an Arc in order to capture all elements of the context.

I'm also not totally clear on what role the _pkcs11_lib member is actually playing that might require it to be transferred too.

This effectively holds open the shared library (either a .so or a .dll)

I was looking at the cryptoki-sys crate for the first time. Curious why doesn't it hold its own function list? It just seems fragile in ways that it doesn't need to be.
And this still works with this somewhat common pattern if you don't have a strong need to expose the Arc:
#[derive(Clone)]
#[repr(transparent)]
pub struct Pcks11(Arc<Inner>);
struct Inner { // private
   // old struct contents
}

I was looking at the cryptoki-sys crate for the first time. Curious why doesn't it hold its own function list? It just seems fragile in ways that it doesn't need to be.

The majority of that crate was auto-generated by bindgen.

And this still works with this somewhat common pattern if you don't have a strong need to expose the Arc:

I can look into this

I was looking at the cryptoki-sys crate for the first time. Curious why doesn't it hold its own function list? It just seems fragile in ways that it doesn't need to be.

Because it's supposed to be an FFI-only layer crate, as described here:

The task of the sys crates is expose a minimal low-level C interface to Rust (FFI) and to tell Cargo how to link with the library. Adding higher-level, more Rust-friendly interfaces for the libraries is left to "wrapper" crates built as a layer on top of the sys crates

Yes, I understand its role. But doesn't that role include owning the function list it exposes? They have the same lifetime, but rust can't detect that because of the FFI boundary. So the list should either need explicit lifetime annotations to prevent it from being used unsafely or the library should maintain ownership so they're invalidated at the same time.

The library does own the function list it exposes.  It's returning a pointer to it.  The 'lifetime', if you will, is maintained by keeping an instance of cryptoki_sys::Pkcs11 as _pkcs11_lib.

Maybe instead of "owns" I should have said "manages". As in, the callbacks are cached inside the FFI layer struct and accessed through it, rather than having two members in the wrapper. This is effectively the self-reference problem described in the stackoverflow link in #57. The wrapper struct contains both a thing (the FFI lib struct) and a "reference" into that thing. Except instead of an actual reference that the borrow checker could get angry about, it's a raw pointer whose safety you have to guarantee manually.
Also, in case I"m coming across as argumentative, I'm really just curious. I haven't had the chance to work with bindgen or deal with FFI issues, so I'm trying to make sure I understand why things that look "bad" are maybe necessary.

I think this is one of those things that will always look bad because of how the C library implements things.  Normally, the C library contains a static struct that gets defined at compile time and you are just receiving a pointer to some piece of memory within the library.  I guess you could change how the crate is implemented to hold it by creating a wrapper struct around the crypto_sys::Pkcs11 struct, but then I think it would like the same as cryptoki::Pkcs11.  But I think that gets away from the pure FFI only crate ideal.

Indeed, I think a big part of the issue is down to how the spec defined things, which in the end puts limits on how you can (sanely and nicely) deal with it in Rust. Something like the TPM spec, where you don't have to retrieve a list of pointers to functions that form the API would've been nice, alas such is PKCS11.

Also, in case I"m coming across as argumentative, I'm really just curious. I haven't had the chance to work with bindgen or deal with FFI issues, so I'm trying to make sure I understand why things that look "bad" are maybe necessary.

No, it's always good to have people challenge the design/interface/implementation/... - it might not always get a speedy fix if we want to maintain things stable for a while, but at least being aware of alternatives and best practices is certainly better than not being aware :) And this is a prime reason for having a project open sourced anyway. Apologies if I've been curt, there's been a deluge of stuff coming up this week, difficult to keep track of everything.

Shall I merge this? I don't think we'll get any other reviews for a while


az login --identity

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 603, in execute
    raise ex
cli/core/commands/__init__.py, ln 661, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 654, in _run_job
    six.reraise(*sys.exc_info())
lib/python3.6/site-packages/six.py, ln 693, in reraise
    raise value
...
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 600, in set_token
    token_entry = self._vm_msi.get_token(self.resource)
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 649, in get_token
    token_entry = self._retrieve_token_from_imds_with_retry(resource)
python3.6/site-packages/msrestazure/azure_active_directory.py, ln 704, in _retrieve_token_from_imds_with_retry
    raise HTTPError(request=result.request, response=result.raw)
requests.exceptions.HTTPError

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login --identity

Expected Behavior
Environment Summary
Linux-5.0.0-1014-azure-x86_64-with-debian-buster-sid
Python 3.6.5
Shell: bash

azure-cli 2.0.72

Additional Context

im so sorry， I forgot to switch the cloud area.
configure： az cloud set -n ChinaCloud

After 3 years, while I am reviewing this issue. I don't think this issue has anything to do with the cloud.
As indicated by https://github.com/Azure/azure-cli/issues/15656#issuecomment-716372982, the managed identity on the VM should be configured properly.


Include a custom .profile and/or .mksh file

Although we do mostly focus on the end-user that uses graphical software, it's not a bad idea to also make the Terminal look and feel beautiful too.
Some custom features to include where appropriate:

Try to adapt Powerline to work with mksh.
Pre-configured Linuxbrew paths.

# Linuxbrew setup.
if [ -d /home/linuxbrew/.linuxbrew ]; then
    LINUXBREW=/home/linuxbrew/.linuxbrew
else
    LINUXBREW=$HOME/.linuxbrew
fi

export PATH="$LINUXBREW/bin:$PATH"
export MANPATH="$LINUXBREW/share/man:$MANPATH"
export INFOPATH="$LINUXBREW/share/info:$INFOPATH"

export PATH=$HOME/go/bin:$PATH

mksh autocomplete (necessary)
Others?

We switched to zsh for the interactive user shell, and we include a default zshrc configuration file.


Fix syntax error in empty object with dangling comment

Fixes #524

Thanks for jumping it and fixing it!


[AA-1180] remove deprecated reset to minimal template

The "Setup" tab, hidden in on-prem deployments, was intended to be removed back when we removed the Staging vs Production concepts. Instead of being removed, it was left in a state where it was accessible in Cloud ODS deployments, though it did no meaningful work and had little to say to the user. It's no longer necessary since Descriptors would already be present at database deployment time. So, we had a fairly large set of code that was in practice unreachable and would not perform meaningful work if it ever were reached.
This PR phases out the deprecated code that was intended to be removed previously. Although you may want to review the overall diff as a whole, it may help to see the technique used by scanning the individual commits in order. In general, I removed code "bottom up" starting with things we knew had no meaningful use, relying on R# and compiler errors to guide me towards removing all the unreachable code.

Developer testing

Went through the individual commits
Local build testing
Unit tests pass successfully


Fix highlightedIndex_ not being set in setHighlightedIndex

As discussed in https://github.com/google/closure-library/issues/731

I'm not sure I understand why the The Travis CI build failed

@ghosttie well it's because if setHighlightedIndex(-1) is called the highlight should be removed. Obviously no child at the index -1 is found and this.highlightedIndex_ > -1 will always evaluate to false in that case and because of your change. Therefore you can't remove the current highlight anymore and the test fails.
You obviously only want to set the highlighted index to something > -1 if there's actually a child that will be highlighted anyway so that's another issue.

@mashedcode thanks, that was very helpful - I'll be more self-sufficient next time

I don't think

Failed: Modal dialog present: The Acrobat DC PDF browser plug-in is missing.

is my fault...

Sauce Labs shakes fist!
That failure is unrelated to this CL. It's a Sauce Labs infra platform issue.

How about

Failed: ERROR The test with session id c52b1bf454734499adc752c66df1edcd has already finished, and can't receive further commands.

Same. It means we ran out of VMs on Sauce Labs. It typically happens when multiple PRs/commits come in at the same time.

Does it automatically retry or do I need to do something?

I restarted it.

closure/goog/net/streams/jsonstreamparser_test.html timed out after 45s!

This is less fun that I was expecting

I wouldn't be too concerned getting Travis green. It's pretty obvious your change is introducing any new breakages.

PR looks good. Can you please squash your commits into 1 and add a unit test to https://github.com/google/closure-library/blob/master/closure/goog/ui/container_test.js?

@ghosttie waiting on the last comment before merging.

Sorry, I do intend to do squash the commits and add a unit test

When I was writing the tests I realized it was working even without my changes.  highlightedIndex_ is set in goog.ui.Container.prototype.handleHighlightItem so it doesn't need to be set directly in setHighlightedIndex.  I can't think why I had the problem in the first place.


chore(driver-adapters): bump @neondatabase/serverless peer dep version to ^0.10.0

Changelog:

Capture stack traces in NeonDbError, if Error.captureStackTrace is available.
Allow authentication through JWT by adding a authToken property to the neon HTTP connection options.

https://github.com/neondatabase/serverless/blob/main/CHANGELOG.md#0100-2024-10-07

Thanks! @andyjy


Introducing sig-etcd

Thanks to @logicalhan @serathius for the effort of making etcd a Kubernetes special interest group, @fsmunoz for helping putting together the doc, and @jmhbnz for posting the blog in etcd.
This post is to celebrate introducing etcd as a k8s sig.
cc @ahrtr @jberkus @kaslin @ptabor @mitake

This is a mirror of https://etcd.io/blog/2023/introducing-sig-etcd/
Usually we like to sync them, but I think we may have missed the boat on that. Are they otherwise identical?

We’ll at least need to change the publication date to a future date
/hold

Nov 6 may be  a good date, will give us some time to review it.
Board https://github.com/orgs/kubernetes/projects/93/views/3

Nov 6 may be a good date, will give us some time to review it. Board https://github.com/orgs/kubernetes/projects/93/views/3

Thanks, updated.

This is a mirror of https://etcd.io/blog/2023/introducing-sig-etcd/
Usually we like to sync them, but I think we may have missed the boat on that. Are they otherwise identical?

Yes, this is the exact the same as the etcd blogpost https://etcd.io/blog/2023/introducing-sig-etcd/. Thanks for the reminding, we will do the sync instead next time.

Sorry, i should have mentioned earlier, could you also add a canonical URL field in the front matter as well?
See these upcoming blog entries for the format:

https://github.com/kubernetes/website/pull/43775/files
https://github.com/kubernetes/website/pull/43474/files

Sorry, i should have mentioned earlier, could you also add a canonical URL field in the front matter as well? See these upcoming blog entries for the format:

https://github.com/kubernetes/website/pull/43775/files
https://github.com/kubernetes/website/pull/43474/files

Thanks for reminding. Added.

/hold
We usually avoid mirroring articles that appear more than 24 hours after initial publication.
With this not having been planned in, I think we should stick to that convention and not mirror the article here.

Maybe we can have a (different) post-kubecon article that wraps up recent announcement and hyperlinks to the SIG etcd article?

cc

/retitle Add mirrored copy of blog article “Introducing sig-etcd”
I don't think we'll merge this as-is @wenjiaswe; we'd have been happy to mirror the article with notice ahead of publication, but that didn't happen.

I'm going to close this PR, as we wouldn't merge it.
Thanks for the proposal @wenjiaswe; it's unfortunate that the timing wasn't right.
/close

@sftim I'm going to ask you to reconsider.  It's a new SIG, being announced here at Kubecon; it would be valuable to have content on the Kubernetes blog in a more timely fashion.  It's one thing if there's content that's not appropriate for the Kubernetes blog, or if nobody has time to review.  But if the only problem is that they missed the mirroring window by 24 hours, that seems like "rules over spirit".

I agree with @jberkus here. We have a new SIG and this is the opportunity to get the most eyes on it. Posting it after KubeCon will not be nearly as impactful as getting it up before or (now,) during.

Reopening to facilitate conversation.
/reopen

Holding for publishing decision
/hold

While I agree with Tim that this isn't per the norm and should have, ideally, been submitted earlier for review and approval. However, taking into cognizance that this is a new SIG that is being introduced at KubeCon we should consider making an exception.

Hello,
As co-author of the article I would just like to add that I was under the assumption that it would be published here, and was unaware that it was planned to be published elsewhere - not that I have anything against it! I would have tried to make things sync better otherwise.

Thanks everyone for chiming in. I apologize,  being a new SIG, there is indeed a bit gap there. We weren't aware of this 24hours mirroring policy, will definitely follow the rule in future.

Ideally there would have been better coordination efforts to mirror this blog article. In my opinion, we can make an exception in this case with the timing of the introduction of SIG etcd at KubeCon and publishing a mirror of this blog that introduces SIG etcd
/lgtm

Updated to remove commit error.

/approve
The SIG Docs leads have been able to have conversations with @wenjiaswe and @nate-double-u about process, and while we've already commented about the process preferably being better followed in future, we agree that it's worthwhile publishing due to contributor wrangling for etcd, specifically at KubeCon NA.
@sftim We want you to have the final call here. If you're amenable to this decision, it would be great for you to remove the hold in order for the blog to publish by Tuesday. A huge thanks in advance 🙏

/hold cancel

/lgtm

Thanks everyone!


Fix deployment errors

Summary
This PR fixes the deployment issue caused by the incorrect syntax for styling. Unfortunately, our setup right now only supports the React syntax (ie. style={{"class": "value"}})
We will support traditional syntax shortly

Merged in https://github.com/BuntingLabs/mundipy/commit/b71ab5383e63da0b20dc2d474d97b61f4f40015c, thank you!


[BUG] useCartItem composable returns invalid values

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Hello, currently itemRegularPrice and itemSpecialPrice in useCartItem composable returns the same single unitPrice value.

Expected Behavior
Since cartItem can have different itemQuantities:

I expect that itemRegularPrice will return the price of unit * itemQuantity.
I expect that itemSpecialPrice will return discounted price of unit * itemQuantity.

hey @ignasradzius, thanks for reporting the issue 💙
I've added new property named itemTotalPrice within #168 which is responsible now for keeping the total price for quantities of a product added to the cart.
I left the regular and special (discounted) price as it was because we don't wont to do any calculations on browser side due to known issues with float and rounding numbers, especially for the prices in ecommerce. So I could only expose what comes from API, as it's our single source of truth. The precision and other currency settings can be also set in admin panel, so it would be hard to mainain the client and backend side - it's easy to forget 🤔
for special price case you can always do the calculation on your as you have an access to cartItem object as well - using the library you can trust in that matter.
Please leave your feedback if it's that OK 👍🏻 for you or not 👎🏻


Updating app_requested payload to include new field

Updating app_requested payload to include is_granular_bot_app
Category (place an x in each of the [ ])

[x] bolt (Bolt for Java)
[x] bolt-{sub modules} (Bolt for Java - optional modules)
[x] slack-api-client (Slack API Clients)
[x] slack-api-model (Slack API Data Models)
[x] slack-api-*-kotlin-extension (Kotlin Extensions for Slack API Clients)
[x] slack-app-backend (The primitive layer of Bolt for Java)

Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

This Java class needs to be updated and you can run the unit test that auto-generates the sample JSON file: https://github.com/slackapi/java-slack-sdk/blob/main/slack-api-model/src/main/java/com/slack/api/model/admin/AppRequest.java

Thank you, I was able to add to the java file. This is my first time working on this, which test do I run and how do I run them?

@gugutagli You can run this script for it, but it may take time. For this specific pull request, running only this test should be fine.

@gugutagli Thanks for sending this PR!


Metadata field caching hibernate performance

Description
We noticed the hibernate cache is not working well when searching for a metadata field. When using org.dspace.content.service.MetadataFieldService#findByElement(org.dspace.core.Context, java.lang.String, java.lang.String, java.lang.String), a large amount of time is spent querying the database for the field
Instructions for Reviewers
This is a very technical problem.
The hibernate implementation did use the level 2 hibernate cache. This is not the fastest cache, and does in fact still query the database.
This implementation will use the hibernate level 1 cache (when possible). This is done by storing the ID of the metadata field in a Java cache (only the ID), and retrieving the MetadataField from the hibernate cache (which is always up-to-date). This passes all tests, including the tests where new fields are made, all the flyway resets, …
I was able to use the GitHub runners to verify the current performance:

https://github.com/atmire/DSpace/runs/1934911128#step:5:737 displays the duration of finding a metadata field is 0.06188 ms per operation
https://github.com/atmire/DSpace/runs/1934911128#step:5:694 displays the duration of adding a metadata value is 1.5202 ms per operation

Hereafter I've included the actual performance improvement https://github.com/atmire/DSpace/commit/4f821ca417673067673c31160232bc53695611a0 :

https://github.com/atmire/DSpace/runs/1934914170#step:5:692 simply succeeds, and complies to the max durations

Hereafter I've reduced the max duration to see how much faster it really is (no longer in a branch, just to check the amount)

https://github.com/atmire/DSpace/runs/1934948108#step:5:737 displays the duration of finding a metadata field is now 0.0022 ms per operation => only 3.5% of the duration
https://github.com/atmire/DSpace/runs/1934948108#step:5:898 displays the duration of adding a metadata value is now 0.0538 ms per operation => only 3.5% of the duration

I've now set the expected durations relatively midway the current master performance and the new performance, which is an expectancy margin of roughly a factor 5 on either side
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

@abollini This PR doesn't change that behavior, the hibernate cache was already used, and is still used
This implies:

If you change a metadata field in a single node deployment, the hibernate level 1 cache is aware of this change, and the changed version will be returned in this PR. The only difference is that it doesn't perform an additional search, unless the data returned from the cache doesn't match the expected value. In that case the current code is used
If you change a metadata field in a clustered deployment, the hibernate level 1 cache is not aware of this change, also not in the current main. This behavior has not been changed in this PR

@abollini Samuel has added tests verifying there's no problem when changing the metadata schema
About the clustering scenario, we tested the following scenario using the main branch (not this PR):

Call MetadataFieldService#findByElement(org.dspace.core.Context, java.lang.String, java.lang.String, java.lang.String) to search for dc.subject and dc.identifier.issn
Outside the tomcat (it was done directly in the DB, to represent to other cluster), change the schema of these fields to dspace (so it’s now dspace.subject and dspace.identifier.issn
Call MetadataFieldService#findByElement(org.dspace.core.Context, java.lang.String, java.lang.String, java.lang.String) using the tomcat again, but using a new context. Searching for dspace.subject and dspace.identifier.issn => This resulted in the dc.subject and dc.identifier.issn fields (Hibernate's cache did not see the change)
Call MetadataFieldService#findByElement(org.dspace.core.Context, java.lang.String, java.lang.String, java.lang.String) using the tomcat again, but using a new context. Searching for dc.subject and dc.identifier.issn => This resulted in the dc.subject and dc.identifier.issn fields (Hibernate did match the new names, but Hibernate's cache still did not see the change)

Since the clustering scenario is not working in main, we didn't try to fix it in scope of this PR

thanks @benbosman the added test matches my request


Multiple Trainloaders in a Sequential Mode.

🚀 Feature

Add new mode to multiple_trainloader_mode of the Trainer in order to support training on multiple dataloaders in a sequential way.
Motivation

Say we want to train on two dataloaders A, and B.
Currently, the modes support max/min size of all trainloaders and provide a list/dict containing batches from loader A and B at each step. But it is not possible to define loaders such that it returns batches from loader A until it is exhausted, and then returns batches from loader B until exhausted.
Pitch

Adding a new mode, called sequential loading to the Trainer. At each time, the batch list/dict can have only one batch from one of the loaders.
Example application: Finetuning
Say we want at each epoch, train a model on one dataset and then finetune it on another.
loaders = {'train': loader_train, 'finetune': loader_finetune}

for epoch in epochs:
   train(model, loaders['train'])
   train(model, loaders['finetune'])

Alternatives

Chaining dataloaders together as a single dataloader seems possible but then additional book-keeping is required to figure out which dataloader a specific batch belongs to.
If we can call the train_loop of the LightningModule, we can add a callback that is called at the end of each training epoch and then finetune the model on the finetune loader.

Additional Context
To the best of my knowledge, there is a CombinedLoader that takes care of parallel sampling. I suppose a new mode can be added to this class to support sequential loading.
But I could be wrong and there is a better alternative. In that case, please let me know.
Thanks.

Dear @imirzadeh,
Great apology for your issue going stable ...
We plan to investigate such ideas with Lightning Flash to find the best way to support this and then up-stream components to Lightning.
Would you be interested in collaborating with us ?
Best,
T.C


[question] Page localization.

Hi, thanks for reading this.
Well, i'm practicing making AppImage, and understanding docs.
But one day i thought "what if a person whose English skill are bad?" and this person will suffer from reading.
So what if someone translate docs?
Can someone's translation be accepted?
Thanks for reading.

Hi @BeyondLimitation, thanks for your question. Since AppImageKit is mainly for developers who want to develop AppImages of their software, we assume that the main language is English. It would be a lot of work to translate everything not just once, but keep the translations updated all the time.
I could imagine that we translate the part "how to make executable and run an AppImage", but not all the developer documentation.
Would you volunteer for Korean?

@probonopd Thanks for answer!