Assign vhost-user-blk device to Kata container as ephemeral or persistent storage #2380

With this patch set, kata container can run with vhost-usre-blk device provided by SPDK.
Fixes: #2380

Here are the steps to assign SPDK vhost-user-blk device to container.
1. set enable_hugepage=true and enable_vhost_user_store in kata toml file.
2. run spdk_tgt and create a vhost-user-blk device
* file vhost-blk.conf is <
[AIO]
AIO /home/vhost-blk-file AIO1 4096
[VhostBlk0]
Name vhostblk0
Dev AIO1
>
*spdk_tgt -c vhost-blk.conf -S /var/run/kata-containers/vhost-user-socket/
3. Create fake block node and uevent file
* mkdir -p /var/run/kata-containers/vhost-user-mknod/block/
* mkdir -p /var/run/kata-containers/vhost-user-mknod/block/241:0/
* file /var/run/kata-containers/vhost-user-mknod/block/241\:0/uevent is: <
MAJOR=241
MINOR=0
DEVNAME=vhostblk0
DEVTYPE=vhost-user-blk
>

4. mknod for vhostblk0 device
* mknod /tmp/vhostblk0 b 241 0

5. Run docker with vhost-user-blk
* docker run --runtime kata-runtime --device=/tmp/vhostblk0:/dev/vda -it busybox sh

Thanks for your comments @jodh-intel .
Let me add UT and fix vendor qmp's change later.
 It only record Major/Minor number for block devices. But vhost-user devices is specified by path of its socket. So I have to use the trick to emulate a sys directory for  uevent, and use Major number 241 and 242 as the signs of vhost-user-blk and vhost-user-scsi devices.
Yes, the prepare operations doesn't seem ideal to users. So I'm going to first add a docker volume plugin to automaticly handle these steps, just referencing kata's SRIOV and VPP's cases.

@devimc @jodh-intel Can you take a look as well.

travis is failing, may be this needs a rebase

Hi, @devimc
It's really a strange failure there -- it reported:
ERROR: Document vendor/github.com/containernetworking/plugins/plugins/main/windows/CONTRIBUTORS.md is not referenced.
I extracted the related markdown checking script out, and applied the current master branch. Seems Document vendor/github.com/containernetworking/plugins/plugins/main/windows/CONTRIBUTORS.md is also not referenced.
Pulled master branch and rebase on it, but the error is still there.

Hmm, vendor documents should not be getting checked in the markdown check scripts... didn't we see this somewhere else recently ?

uhmm, yeah, this error is new, @GabyCT @chavafg @jodh-intel have you seen this before?

Hi, @devimc
The extra check is triggered by change in ./virtcontainers/documentation/api/1.0/api.md.
Seems some other added configurations also aren't in api.md. So I just removed that change.

@dong-liuliu seems like you need to rebase this, I see a commit that shouldn't be here

@devimc rebased :)

Hi, @devimc @amshinde @c3d @jodh-intel
Any other review comments? :)

/test

/test

/test

Hi @dong-liuliu - please can you update this branch as it is conflicted?

Hi @dong-liuliu - please can you update this branch as it is conflicted?
Ack. Sorry, didn't notice the confilct.

Ping @kata-containers/runtime.

/test

Thanks for your approval, @devimc
Just found and fixed the extra test failure in which a added case should be skipped as requiring root privilege.
Please help me retrigger the extra test :)

/test

/test

/test

Added more unit tests to improve code coverage :)

hi, @amshinde
need your help to trigger /test again 👍

/test

hi, @chavafg,
Seems a test retrigger is still required.
The failures were some random errors from their error log:
one was caused by GPRC Failed to check if grpc server is working: rpc error: code = Unavailable desc = transport is closing;
Or  Trying to pull registry.fedoraproject.org/fedora-minimal:latest... 04:25:54   received unexpected HTTP status: 503 Service Temporarily Unavailable;
And ERROR: kata-log-parser: missing pid

@dong-liuliu - I've re-triggered the F31 CI.
@GabyCT - is the F31 CI stable? I see a lot of fails there - is it something we can/should currently ignore?

Seems that F31 started to fail when running with podman, I saw the same issue on other PRs.
About the other failures, yes, they seem to be random, I have retrigger those jobs.

Thanks @grahamwhaley and @chavafg,
we've gotten debian-10 passed~.
Seems F31 with Podman really failed weirdly many times.
And for U1804 with initrd also failed weirdly. I saw for its one successful build log showed same content with  this failure's log

Seems CI-F31 with podman and CI-U1804 with initrd failed a lot.
Is it possible to just ignore them currently? @GabyCT

@dong-liuliu , the previous failure with the podman CI, it is a bug that we currently have, however, I re-trigger the jenkins job and now it seems we have a failure but this is related with this PR
13:46:55      BUILD    /tmp/jenkins/workspace/kata-containers-runtime-podman-fedora-31-PR/go/src/github.com/kata-containers/runtime/kata-runtime
13:47:06 # github.com/kata-containers/runtime/virtcontainers/device/drivers
13:47:06 ../virtcontainers/device/drivers/vhost_user_blk.go:76:16: devReceiver.DecrementSandboxBlockIndex undefined (type api.DeviceReceiver has no field or method DecrementSandboxBlockIndex)
13:47:08 make: *** [Makefile:539: /tmp/jenkins/workspace/kata-containers-runtime-podman-fedora-31-PR/go/src/github.com/kata-containers/runtime/kata-runtime]

Please see http://jenkins.katacontainers.io/job/kata-containers-runtime-podman-fedora-31-PR/66/console

@dong-liuliu , the initrd CI is also failing, and it also seems related with this PR
# github.com/kata-containers/runtime/virtcontainers/device/drivers
../virtcontainers/device/drivers/vhost_user_blk.go:76:16: devReceiver.DecrementSandboxBlockIndex undefined (type api.DeviceReceiver has no field or method DecrementSandboxBlockIndex)
Makefile:539: recipe for target '/tmp/jenkins/workspace/kata-containers-runtime-ubuntu-18-04-PR-initrd/go/src/github.com/kata-containers/runtime/kata-runtime' failed
make: *** [/tmp/jenkins/workspace/kata-containers-runtime-ubuntu-18-04-PR-initrd/go/src/github.com/kata-containers/runtime/kata-runtime] Error 2
Failed at 67: "${cidir}/install_runtime.sh" "${tag}"

Thank you, @GabyCT
I just rebased the patch set on recently merged patches, and fixed the issue.

/test

/test

hi, @amshinde, @devimc
What's the next step I should do to get this patchset merged into branch? :)


add support for binding symfony command

fixes #209
cc @tacman and @tucksaun
simpler diff: ?w=1

I dont' know or use Castor to have a strong opinion about this but code looks legit 🤓
On the UX side I'm wondering if that makes sense to declare the console entrypoint in your tasks? maybe being able to configure it in castor.php would be more practical?

Hmm, I've added the AsSymfonyTask to one of my commands, but it's not being recongized.  I'm using castor  v0.11.1, installed manually and using the symfony-command branch.
Is there something else I need to do to the castor.php file?  I re-initialized it, same issue.
bin/console make:command castor:test

Add the attribute
#[AsSymfonyTask(name: 'testing', console: [__FILE__])]
#[AsCommand(
    name: 'castor:test',
    description: 'Add a short description for your command',
)]

castor list

castor v0.11.1

Usage:
  command [options] [arguments]

Options:
  -h, --help            Display help for the given command. When no command is given display help for the list command
  -q, --quiet           Do not output any message
  -V, --version         Display this application version
      --ansi|--no-ansi  Force (or disable --no-ansi) ANSI output
  -n, --no-interaction  Do not ask any interactive question
  -v|vv|vvv, --verbose  Increase the verbosity of messages: 1 for normal output, 2 for more verbose output and 3 for debug

Available commands:
  completion  Dump the shell completion script
  hello       Welcome to Castor!
  help        Display help for a command
  list        List commands

I dont' know or use Castor to have a strong opinion about this but code looks legit 🤓

HAN 😱 (joke)

On the UX side I'm wondering if that makes sense to declare the console entrypoint in your tasks? maybe being able to configure it in castor.php would be more practical?

we don't have way to configure castor globally
It's not possible to configure it globally, since in one repository managed by castor, you can have N Symfony application. So it mush be at the task level. BUT, you can extends the attributes, and preconfigure it, then use this child class to avoid repetition.

Is there something else I need to do to the castor.php file? I re-initialized it, same issue.

You need to import(/path/to/symfony/command/);

import? I've used require and include, but I'm not sure what import means. the castor import didn't work either. No matter what I do I get

Like all other task, you need to import them. Example with your reproducer
import(__DIR__ . '/src/Command');

Cache key "symfony-console-definitions-/home/tac/ca/mus/src/Command/CastorTestCommand.php" contains reserved characters "{}()/@:".

Oups, I'll fix that asap.

I was hoping that it would do a reflection of all the symfony commands, find the ones that were tagged, and auto-register them.

Yes, that what my PR does :)

And of course cache all this so it only did the scan if a file changed, like the CompilerPass of a bundle.

Yes, there is also cache :)

A reproducer would be trivial:

I'm on it. Thanks

I'm not able to reproducer any of the error you reported.
my history, and the diff
symfony new --webapp symfony-command-and-castor
cd symfony-command-and-castor
bin/console make:command Hello
# Fix the MAILER_DSN=null
castor
code castor.php
castor hello

diff --git a/.env b/.env
index d7c75ee..5bc2089 100644
--- a/.env
+++ b/.env
@@ -37,5 +37,5 @@ MESSENGER_TRANSPORT_DSN=doctrine://default?auto_setup=0
 ###< symfony/messenger ###

 ###> symfony/mailer ###
-# MAILER_DSN=null://null
+MAILER_DSN=null://null
 ###< symfony/mailer ###
diff --git a/castor.php b/castor.php
new file mode 100644
index 0000000..851ed20
--- /dev/null
+++ b/castor.php
@@ -0,0 +1,9 @@
+<?php
+
+use Castor\Attribute\AsTask;
+
+use function Castor\io;
+use function Castor\capture;
+use function Castor\import;
+
+import(__DIR__ . '/src/Command');
diff --git a/src/Command/HelloCommand.php b/src/Command/HelloCommand.php
new file mode 100644
index 0000000..445605a
--- /dev/null
+++ b/src/Command/HelloCommand.php
@@ -0,0 +1,51 @@
+<?php
+
+namespace App\Command;
+
+use Castor\Attribute\AsSymfonyTask;
+use Symfony\Component\Console\Attribute\AsCommand;
+use Symfony\Component\Console\Command\Command;
+use Symfony\Component\Console\Input\InputArgument;
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Input\InputOption;
+use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Style\SymfonyStyle;
+
+#[AsCommand(
+    name: 'Hello',
+    description: 'Add a short description for your command',
+)]
+#[AsSymfonyTask()]
+class HelloCommand extends Command
+{
+    public function __construct()
+    {
+        parent::__construct();
+    }
+
+    protected function configure(): void
+    {
+        $this
+            ->addArgument('arg1', InputArgument::OPTIONAL, 'Argument description')
+            ->addOption('option1', null, InputOption::VALUE_NONE, 'Option description')
+        ;
+    }
+
+    protected function execute(InputInterface $input, OutputInterface $output): int
+    {
+        $io = new SymfonyStyle($input, $output);
+        $arg1 = $input->getArgument('arg1');
+
+        if ($arg1) {
+            $io->note(sprintf('You passed an argument: %s', $arg1));
+        }
+
+        if ($input->getOption('option1')) {
+            // ...
+        }
+
+        $io->success('You have a new command! Now make it your own! Pass --help to see your options.');
+
+        return Command::SUCCESS;
+    }
+}

here's my reproducer:
symfony new castor-command-demo --webapp && cd castor-command-demo
sed -i "s|# MAILER_DSN|MAILER_DSN|" .env
bin/console make:command app:castor-test
cat > castor.php <<'END'
<?php

use Castor\Attribute\AsTask;

use function Castor\io;
use function Castor\capture;
use function Castor\import;

import(__DIR__ . '/src/Command/CastorTestCommand.php');

#[AsTask(description: 'Welcome to Castor!')]
function hello(): void
{
    $currentUser = capture('whoami');

    io()->title(sprintf('Hello %s!', $currentUser));
}
END

Add attribute to AppCastorTest.php
#[\Castor\Attribute\AsSymfonyTask()]

castor list

Thanks for the reproducer.

I didn't get an error about the cache. Are you sure your vendor are up-to-date ? composer i ; anyway, I will-ensure we don't put such char in the key
I found a cache issue ! I'll fix it ASAP
Otherwise, everything is working as expected

@tacman I pushed a new version. Can you tried it?
Note, you may have to clear the cache manually rm ~/.cache/castor/ -rf on my computer


Create schedule_algorithm file

@davidopp
This document explains briefly the schedule algorithm of Kubernetes and can be complementary to scheduler.md.
The purpose of writing this doc is to let new Kubernetes users have a general idea about the schedule algorithm easily before wondering through the source code. Since the schedule algorithm could be vital in their product, this doc is helpful for them to quickly decide to what extend they would have to change the scheduler algorithm if they want to adopt Kubernetes.
This is only a very first version, and it is quite open to any changes. So, please leave comments if you have any suggestions.
Cheers.

cc @erictune

Please rename to docs/devel/scheduler_algorithm.md

@davidopp
The PR is updated and your suggestions are considered.
PS: How come the shippable test fails? Would you give a link on how to fix this? I always get "Build not found" when checking the details.

LGTM
Let's merge this, since we're getting very close to the cutoff, but once it's merged can you send a follow-up PR adding a sentence to the "LeastRequestedPriority" description explaining that this policy has the effect of spreading pods across nodes (with respect to resource consumption)?

Oh wait, actually, not OK to merge -- please move this file to the docs/devel/ directory (that's where scheduler.md lives now). When you do that you can also do what I suggested in the previous comment.

Oh and also please squash the commits into one. :) Thanks!

Oh and also please run hack/run-gendocs.sh
I promise that's the last request!

@davidopp
Hi, thanks for the kindly and quickly review! I will change/run-gendocs.sh/rebase ASAP.

@davidopp
New patch sent, PTAL.

LGTM
Thanks!


Something is wrong with speed limits tool

Describe the problem
I slapped down a bunch of roads while testing airport stuff:

Note how they all have 'custom override' speeds 20 mph.
However, I didn't set those overrides. I've done nothing to those roads other than splat them down on the map.
The default speeds of those roads are 30 mph.

Where did the 20 mph come from?

The number comes from SpeedLimitManager API returning the numbers. As that's a dictionary with segmentId as a key, it should not return anything for a new segment that did not exist before. But then also this code was touched by @kianzarrin should get his opinion too.

Lane flags etc for image in previous comment, lanes ordered from left to right:
Road on the left:

Road on the right:

Added some debugging to the DrawSpeedLimitHandles_SegmentCenter() method:
if (isHoveredHandle) {
    try {
        Log.Info(
            $"overrideSpeedlimitForward = {overrideSpeedlimitForward}, " +
            $"overrideSpeedlimitBack = {overrideSpeedlimitBack}, " +
            $"drawSpeedlimit = {drawSpeedlimit}, " +
            $"defaultSpeedLimit = {defaultSpeedLimit}");
    }
    catch (Exception ex) {
        ex.LogException();
    }
}

Here's what came up when hovering over one of the speed icons (note: log is in km/h whereas images earlier in this issue are showing MPH):
overrideSpeedlimitForward = ,
overrideSpeedlimitBack = 30 km/h,
drawSpeedlimit = 30 km/h,
defaultSpeedLimit = 50 km/h

30 MPH = 47 km/h (I assume this gets rounded to 50 km/h)
20 MPH = 31 km/h (I assume this gets rounded to 30 km/h)

So it seems defaultSpeedLimit is correct, but for some reason I'm getting back invlaid speeds from CalculateCustomSpeedLimit() in the lines below:
SpeedValue? overrideSpeedlimitForward =
    SpeedLimitManager.Instance.CalculateCustomSpeedLimit(segmentId, finalDir: NetInfo.Direction.Forward);
SpeedValue? overrideSpeedlimitBack =
    SpeedLimitManager.Instance.CalculateCustomSpeedLimit(segmentId, finalDir: NetInfo.Direction.Backward);
SpeedValue? drawSpeedlimit = GetAverageSpeedlimit(
    forward: overrideSpeedlimitForward,
    back: overrideSpeedlimitBack);

Will now start digging in to those to find out where the error creeps in.

Results of extended logging:
Info<PHONE_NUMBER>: CalculateCustomSpeedLimit Forward
Info<PHONE_NUMBER>: CalculateCustomSpeedLimit Backward
Info<PHONE_NUMBER>: laneInfo.m_speedLimit = 0.6, as SpeedValue = 30 km/h
Info<PHONE_NUMBER>: GetDefaultSpeedLimit = 30 km/h
Info<PHONE_NUMBER>: validLanes = 1, meanSpeedLimit = 30 km/h
Info<PHONE_NUMBER>: laneInfo.m_speedLimit = 0.6, as SpeedValue = 30 km/h
Info<PHONE_NUMBER>: GetDefaultSpeedLimit = 30 km/h
Info<PHONE_NUMBER>: validLanes = 2, meanSpeedLimit = 60 km/h
Info<PHONE_NUMBER>: overrideSpeedlimitForward = , overrideSpeedlimitBack = 30 km/h, drawSpeedlimit = 30 km/h, defaultSpeedLimit = 50 km/h

The Info<PHONE_NUMBER>: laneInfo.m_speedLimit = 0.6, as SpeedValue = 30 km/h is coming from this code:
        public SpeedValue GetDefaultSpeedLimit(NetInfo netinfo, NetInfo.Lane laneInfo, bool db = false) {
            if (customNetinfoSpeedLimits_.TryGetValue(netinfo, out float speedLimit)) {
                if (db) {
                    Log.Info($"customNetinfoSpeedLimits_ = {speedLimit}");
                }
                return new SpeedValue(speedLimit);
            } else {
                if (db) {
                    Log.Info($"laneInfo.m_speedLimit = {laneInfo.m_speedLimit}, as SpeedValue = {new SpeedValue(laneInfo.m_speedLimit)}");
                }
                return new SpeedValue(laneInfo.m_speedLimit);
            }
        }

So 20 MPH (30 km/h) is the actual prefab speed of the road.... But why is it not getting the 30 MPH (50 km/h) custom default speed for the road if that value is shown when viewing/editing default speeds?
Now I need to dig in to how defaultSpeedLimit (mentioned in previous comment) is being determined...

var defaultSpeedLimit = new SpeedValue(
    gameUnits: SpeedLimitManager.Instance.CalculateCustomNetinfoSpeedLimit(info: neti));

Calls (some code trimmed for brevity)....
        public float CalculateCustomNetinfoSpeedLimit(NetInfo info) {
            return !customNetinfoSpeedLimits_.TryGetValue(info, out float speedLimit)
                       ? GetVanillaNetInfoSpeedLimit(info)
                       : speedLimit;
        }

Which is recognisable from previous comment...
if (customNetinfoSpeedLimits_.TryGetValue(netinfo, out float speedLimit)) { ...

So assuming there isn't an entry in customNetinfoSpeedLimits, it must be calling GetVanillaNetInfoSpeedLimit() (some code removed for brevity)...
        private float GetVanillaNetInfoSpeedLimit(NetInfo info) {
            float? maxSpeedLimit = null;

            if (info.m_lanes != null) {
                foreach (var laneInfo in info.m_lanes) {
                    float speedLimit = laneInfo.m_speedLimit;
                    if (maxSpeedLimit == null || speedLimit > maxSpeedLimit) {
                        maxSpeedLimit = speedLimit;
                    }
                }
            }

            return maxSpeedLimit ?? 0f;
        }

So, that's not checking lane types or vehicle types... Example, if there was a None, None lane but with highest speed, it would be returned as the netinfo speed. I suspect that may be the case with some of the UK road assets (and potentially any network asset from workshop).... time to add more logging/trace....

Annnnd there's the culprit....
- Info<PHONE_NUMBER>: m_vehicleType = None, laneInfo.m_laneType = None, speedLimit = 1
- Info<PHONE_NUMBER>: m_vehicleType = None, laneInfo.m_laneType = None, speedLimit = 1
- Info<PHONE_NUMBER>: m_vehicleType = None, laneInfo.m_laneType = None, speedLimit = 1
- Info<PHONE_NUMBER>: m_vehicleType = None, laneInfo.m_laneType = None, speedLimit = 1
- Info<PHONE_NUMBER>: m_vehicleType = None, laneInfo.m_laneType = None, speedLimit = 1
+ Info<PHONE_NUMBER>: m_vehicleType = Car, laneInfo.m_laneType = Vehicle, speedLimit = 0.6
+ Info<PHONE_NUMBER>: m_vehicleType = Car, laneInfo.m_laneType = Vehicle, speedLimit = 0.6
Info<PHONE_NUMBER>: m_vehicleType = None, laneInfo.m_laneType = Pedestrian, speedLimit = 0.1
Info<PHONE_NUMBER>: m_vehicleType = None, laneInfo.m_laneType = Pedestrian, speedLimit = 0.1

This would also explain all the weirdness I was getting with speed overlays a while back - and why changing default speed for a network was solving those issues.
Should be easy to fix, will send PR shortly.

Code that calls GetVanillaNetInfoSpeedLimit() for future reference:

if (laneInfo.MayHaveCustomSpeedLimits()) { ...

:)


Update CustomConfigurationProvider sample app

Addresses #4208
:key: Takes sample to 2.0

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Create a solutions architecture explaining multi cloud deployment pattern

Is your feature request related to a problem? Please describe.
Multi cloud strategy is becoming more and more popular within enterprises due to the advantages it brings to the enterprises. It is an essential pattern have in this repository

Fixed with commit a50f6b78c2d3fdeccbaab8f3f41089350eb54f78


Contribution to "Regular expression denial of service vulnerability (ReDoS) in date"

Updates

Description

I'm really happy with this functionality of improving the advisory database
:+1: @KateCatlin @katblag

Just a friendly ping for any feedback @darakian

Hey there @SValkanov, sorry for the delay, this got away from me. I've gone ahead and accepted your contribution with some edits. Thank you.


Add check for public Redshift instances

Similar to a couple of other informational checks to help identify public exposure - would be good to have one for public redshift instances as well.

Good idea, thanks @sovchinn


Add a gulp task for compiling SCSS files.

Addresses pattern-lab/pattern-lab-php#16.
Summary of changes:

Adds a gulp task for compiling scss from source/css/*.scss to output directory's css folder.
Adds a watch rule for watching when .scss files are modified, to rebuild as necessary and reload css files.

Thank you @jwir3 for this PR :) I was about to do it myself.

Editions are meant to be easily extensible. While I see this as a common request, not everyone uses SCSS and therefore I don't want to bake that decision into Pattern Lab.
A core tenant is to be tool agnostic.


Update docs

Current documentation is outdated.

[x] add the new API (wait for #100 )
[x] update the existing parts
[x] add some basic overview, reasoning, how it works to the intro

docs is updated by:
https://github.com/packit/requre/pull/134
https://github.com/packit/requre/pull/133


Allow for analyzing containers

Allow for pointing dive at a container instead of an image to see what files have changed relative to the underlying image.

Hello! Any news regarding the implementing of this feature?


Wrong parsing of United States address

Hi!
I am trying to parse address "132 Lessay, Newport Coast, California, 92657, USA" and I'm getting output as
house_number: 132 city: lessay newport coast state: california postcode: 92657 country: usa
While the expected output is
 road: 100 lessay, city: newport coast state: california postcode: 92657 country: usa
Road is not getting parsed correctly. 110 lessay should be a road instead 110 lessay is parsed as house_number: 100 and city:  lessay newport coast, which is wrong.

My country is

USA

Here's how I'm using libpostal

Normalizing the address

Here's what I did

parse address with default options
Here's what I got

house_number: 132 city: lessay newport coast state: california postcode: 92657 country: usa

Here's what I was expecting

 road: 100 lessay, city: newport coast state: california postcode: 92657 country: usa
For parsing issues, please answer "yes" or "no" to all that apply.

Does the input address exist in OpenStreetMap? yes :  https://www.openstreetmap.org/node/7884175123

Do all the toponyms exist in OSM (city, state, region names, etc.)? yes

If the address uses a rare/uncommon format, does changing the order of the fields yield the correct result? No

If the address does not contain city, region, etc., does adding those fields to the input improve the result? address contains srtreet, city, state, postalCode and country

If the address contains apartment/floor/sub-building information or uncommon formatting, does removing that help? Is there any minimum form of the address that gets the right parse? No

225 cedar st, seattle, washington, 98121, usa

Here's what I think could be improved

house_number should not be extracted from the street. for example 225 cedar st is a street name, but libpostal parses 225 as house_number

I am also facing an issue with a similar address, it seems that Newport Coast is not parsed correctly, Newport is parsed as part of the street and the city is parsed as just Coast
Input: 37 S Sur, Newport Coast, CA, 92657
Output:
[
  { value: '37', component: 'house_number' },
  { value: 'south sur newport', component: 'road' },
  { value: 'coast', component: 'city' },
  { value: 'ca', component: 'state' },
  { value: '92657', component: 'postcode' },
  { value: 'usa', component: 'country' }
]

Any workarounds or news on when this could be fixed?


Some images to jpg, some to png

This is not a task, but rather a question.
Use case. Some images are better (smaller size and less compression artifacts) to be converted to PNG (graphics), some to JPG (photos). Usual scenario is like this:

Have a look at images
Specify which output as JPG, which as PNG. Right now the best solution is to make this decision before going to gulp-responsive in file extensions. So *.PNG → *.PNG, *.JPG → *.JPG. Alternative is configuration in JSON (either gulpfile.js or each folder), which is more complicated to setup.

How do you config such cases? Maybe there is another hidden option inside next branch, that should be documented :)
To make it more automatic, we could use https://github.com/Huddle/Resemble.js to analyse artefacts (difference) from different output. And select optimal. But this sounds like a huge task with big risks.

Hi @dandaka
Yep, it is good question :smile:
I think it is task for some kind of config-generators.
For example I use this one gulp-responsive-config. It parses image filename (by this module parse-image-dimensions) and produces per-image configuration.
Image<EMAIL_ADDRESS>became:
{
  name: 'image.png',
  width: 200,
  height: 400,
  rename<EMAIL_ADDRESS>}

I really use them in production but I haven't enough time to prepare them to release and document usage. Sorry :cry:

Any plans to do it some time? Any help needed?

Maybe you can throw few example of production code, so I can guess from that.

It is one of planed tasks https://github.com/mahnunchik/gulp-responsive/issues/23 :smile:
I will try to do it today.

Hi @dandaka
Usage examples: https://github.com/mahnunchik/gulp-responsive/tree/master/examples

That's cool, thank you!


Add tests for ExitCommand #327

Part of #327

v1
@Zhiyuan-Amos submitted v1 for review.

[v1 1/1] Add tests for ExitCommand

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/478/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

@limmlingg APPROVE


Feature Request: Include IDs in principal().calendars()

With the varying URL formats that each caldav server has, a more reliable way of accessing calendars could be by their UIDs, In that respect, I'm requesting the ability for the principal.calendars function to include the ids of the calendars in its output (perhaps by parsing the url).
If this feature is implemented then searching for a particular calendar via its ID could be as easy as the following
next(calendar for calendar in client.principal().calendars() if calendar.id == id)

I have been a bit confused myself (I didn't first read the standards and then write up the caldav library - I found the existing caldav library in a hope that I wouldn't have to read the standards, found that the library didn't work out well enough, tried to hammer on it until it was working for my use cases, and eventually ended up with the maintainer hat).
It's a while since last time I read through the standards, but as far as I remember ...

A CalDAV calendar does not have a UID or an ID.  It is uniquely identified only by it's URL (an event may be copied across distinct servers and hence needs a globally unique UID, but there is no such need for a calendar).
A CalDAV calendar may have a "display name" property.  It's possible to read it and set it through the caldav library.
The caldav client library has the concepts of id and name.  (I will create a new API in version 1.0, and it will have no references to name, only display_name).  cal_id is understood to be the last part of the URL.
Principal.calendar() and CalendarSet.calendar() takes two parameters, name and cal_id.  In cal_id it's possible to pass either a full URL, the path to the calendar or just the last part of the URL.  If name is passed and no cal_id is passed, the server will look up the calendar by name.

Hence, if I'm understanding you correct, the code already does what you want.  Just use Principal.calendar() instead of Principal.calendars().


Cannot comment after around 5 writes

I think it's too fast. gotta set a time interval or sleep a while for each 2-3 comments left.

I can add an configurable interval between each post but does that make a difference (maybe instagram allows more comments if you don't post so fast. I'll test it later) ?

It can increase the number of comments if there is a random time interval between each comment because Instagram doesn't allow DDOS attacks or robotic actions.

Thanks for your help. I'll try to find out a way to work on finding a nice interval between comments (they vary from user to user as I stated in README.md). And I'll do a login using cookies so Instagram doesn't take action because of too many logins.

@fmchan Option added as you requested. In a future will try to make the script find the best interval between comments for your account. Thanks for your collaboration :)


Mobile devices

This awesome plugin works in any way on mobile devices?

Maybe. For now it doesn't on most, because most mobile devices don't support rendering to floating point textures, a feature which is crucial at the moment. I have something in mind which might make it work on mobile devices, but I'm not really sure about it. I may give it a shot next week when I have quite some time to spare.

So yesterday I discovered that some mobile devices do support rendering to half-floating point textures, making mobile device support much more trivial to implement than I previously thought. Expect a new version with mobile support very soon.

And done! See https://github.com/sirxemic/jquery.ripples/releases/tag/v0.5.3

Cool, probe it. Congratulations man.
2016-09-11 15:04 GMT-06:00 Pim Schreurs<EMAIL_ADDRESS>
And done! See https://github.com/sirxemic/jquery.ripples/releases/tag/
v0.5.3
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sirxemic/jquery.ripples/issues/26#issuecomment-246204502,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGN8Fp6eJD1GO4ejGjWsiXcsyoN_VwSwks5qpGzMgaJpZM4J3fxf
.

Hi, it still does not work for mobile browser. I have tested with Samsung Mobile and chrome browser. I am using the latest package.
Please advice.

@mahedihasannoman If this doesn't work in your browser, then your browser just isn't supported. This plugin requires WebGL and support for rendering to floating point textures.


[Feature-1906][git] add git project

Purpose of the pull request
add git project

Comments need to use java doc
and need to use English


Server status  is error

Server status say top: Invalid user, how can i do it

It seems your system doesn't have user for haproxy which name haproxy


Bug: Putting "Move xx" into a comment doesn't get the right link

I see that "Move xx" automagically translates into a link to the move xx.
Unforunately, it doesn't reliably work.
I had to change all the "Move xx" other than "Move 57" into "Turn xx" in the comment at https://gokibitz.com/kifu/Sk2iKSlkZ?path=63&comment=5903293e579aff1b70d5b20d
because no matter what xx I put in there, the link always points to Move 57!
(Move 57 was the first one I created in that comment, as it happens - the other ones were edited in afterwards)

Ah, interesting. That bug makes sense. I'll try to get it fixed this week.
--
Nate Eagle

On Apr 28, 2017, at 10:29 PM, GreenAsJade<EMAIL_ADDRESS>wrote:
I see that "Move xx" automagically translates into a link to the move xx.
Unforunately, it doesn't reliably work.
I had to change all the "Move xx" other than "Move 57" into "Turn xx" in the comment at https://gokibitz.com/kifu/Sk2iKSlkZ?path=63&comment=5903293e579aff1b70d5b20d
because no matter what xx I put in there, the link always points to Move 57!
(Move 57 was the first one I created in that comment, as it happens - the other ones were edited in afterwards)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Should be good to go! Thanks for reporting the bug, @GreenAsJade!


feat: adds app open [id] command

Description
Adds apps open <clientID> command for opening the app settings page in Manage (browser).

Think this looks good, and works well enough for today's use case. Not sure how it would work with private installs, but that doesn't work well altogether anyway, so maybe we can tackle all that later on.

Thanks! 🎉


./simulate is not working on RHEL 7.9

I am a novice user of Mujoco, I try to run the simulator on the remote cluster but I met the following problem. I tried some of the online solutions, but no help. Although it reports problems related to GLX, the command "glxinfo" and "glxgears" works well.
@mercury2 bin$ ./simulate
mujoco pro version 2.10 x
error of failed request: badrequest (invalid request code or no such operation)
major opcode of failed request: 151 (glx)
minor opcode of failed request: 0 ()
serial number of failed request: 9
current serial number in output stream: 9
segmentation fault

FYI: works for me on a remote CentOS-7 x86_64 using Nice DCV (https://download.nice-dcv.com/) with mujoco210-linux-x86_64.tar.gz.

Can you try again with the new 2.1.1 release?

Closing this for now. @truatpasteurdotfr feel free to open if you are still running into issues.


BelongsToMany relationship when extending Eloquent Model

Ran into this issue when upgrading. My BelongToMany relationships for models are no longer working due to the following
Jenssegers\Mongodb\Eloquent\HybridRelations.php
if ($related instanceof Model)

I have a Project model that extends Jenssegers\Eloquent\Model because it has a relationship with a Mongo collection. However, it also has a many to many relationship with an Actor model in MySql. Actor extends Illuminate\Database\Eloquent\Model.
When getting to the if statement above, Actor is not being seen as an instance of Model.
Mongodb\Model
use Illuminate\Database\Eloquent\Model as BaseModel;
Model extends BaseModel

Not sure if this is a PHP issue or "instanceof" simply not being used right. From what I read, it looks like it should work but does not.
I looked back at a previous version...
if (!is_subclass_of($related, 'Jenssegers\Mongodb\Model'))

I used that instead and it works correctly.
Edit: I've confirmed this is an issue with any relationship where one model uses "Jenssegers\Eloquent\Model" and subsequent models with relations use "Illuminate\Database\Eloquent\Model"
My Project model needs Jenssegers\Eloquent\Model to interact with another MongoDB relationship but also interacts with other MySql relationships. All MySql relationships are broken until fixed with the previous version code above. Seems kind of funky to require all my other models to use Jenssegers\Eloquent\Model when they have nothing to do with MongoDB.

Facing same issue. Need to extend Jenssegers\Eloquent\Model in all model

Can you check if dev-master is working for you?

Yes, it works. I see you changed back to ( ! is_subclass_of) and all my tests work accordingly.


Write OpenIFS grid cell fractions in oasis files

I think that it would make an improvement if rdy2cpl could write the grid cell land/water fraction (using the pyoasis set_frac method) into the grid definitions files, at least for the OpenIFS grid. This information can improve the coupling of some fields. For example runoff from the ILCL grid to the runoff mapper.

I do not want to put pressure on this, but it would make it easier for me for the implementation of another project (a replacement of the runoff-mapper for ECE4), if it would be possible to get the effective fraction of land (instead of going to the initial conditions for that).
I saw that there is indeed a call to set_frac in https://github.com/uwefladrich/rdy2cpl/blob/b63655c48cd772a9ef2fe446255e928a8b2a0436/rdy2cpl/grids/couple_grid.py#L63 which is only executed if the fractions attribute is defined. I can make a proposal for this, at least for the OIFS grids.

In pull request #26, I have proposed a modification that would allow to get the frc information just for the OIFS grid that has the ocean masked (used for the communication with runoff-mapper).
I am unsure about what should be the fraction of grid cells that should be action for the coupling with Nemo.


fix: Docker Compose Elastic Search volume configuration, resolve permission denied issue

Summary of Changes
Currently .local and example folders are ignored by .gitignore. When docker-compose starts, it create these folders with root permission. Elastic Search will throw a Caused by: java.nio.file.AccessDeniedException: /usr/share/elasticsearch/data/nodes issue.

Documentation
Installation Doc and Developer Docs

CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely, including a title prefix.
[x] PR includes a summary of changes.
[ ] My commits follow the guidelines from "How to write a good git commit message"

NVM, the issue is noted in the Trouble Shooting Guide.


android-translation-layer: init at 0-unstable-2024-11-15

Fixes https://github.com/NixOS/nixpkgs/issues/312040
Original work by @balsoft https://github.com/NixOS/nixpkgs/issues/312040#issuecomment-2449263830

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This seems cleaner and more promising than my attempt: https://github.com/madushan1000/atl-nix/


[Catalog] Publish Configuring Redis using a ConfigMap

Desired Behavior

Publish Configuring Redis using a ConfigMap to Meshery Catalog.
Implementation

Acceptance Tests

Mockups

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hii @theBeginner86 I want to work on this issue.

@theBeginner86 I have created the Design with name Redis_using_ConfigMap in Meshmap can you please check that is correct and I wanted to know that when we do Publish. Is it  automatically create the design pattern on meshery catalog  OR we have create the design Manually  through meshery.io repo.

@theBeginner86 I am not able to published the because  there the problem in input field . i am not able to to seclect the technology section and not able to write the Description field . this field are not working properly.

@theBeginner86  I have submitted the design for Approval please check

This issue has been open for some time with no recent activity, unassigning to open it up for new contributors to give it a go.


Convert itp files to ff files

The new utility allows users to extract ff-files automatically from itp files by defining fragments as SMILES. Currently the code only works for linear polymers. Let's look at some examples for how to use this Program.
Say we have a itp that describes any number of PEO oligomers then to extract the parameters for PEO we'd run:
polyply itp_to_ff -i <itp_file.ipt> -sm [CH2]O[CH2] -rn PEO -o new.ff -charge 0

Now of course with PEO there is the challenge of how we treat the termini. They can for example be terminated with an OH group. By default the algorithm will group all atoms that cannot be assigned to a fragment as given by the smile into the next connected fragment. Thus it creates a new bigger fragment. In this example, we would obtain the PEO fragment but also PEOter fragment that describes the termini. If they are asymmetric the code produces 2 new residues describing the termini.
Of course this might not be what we want, since we may want to build a modular ff format. In this case we can also specify the OH terminal explicitly as a fragment. For example, like so:
polyply itp_to_ff -i <itp_file.ipt> -sm [OH][CH2] [CH2]O[CH2] [OH][CH2] -rn OH PEO OH -o new.ff -charge 0

Note that we need to provide the terminal definitions on both ends, because the code works in blocks. Using this command we obtain definition for the termini as a separate block.
@ricalessandri
To Do

[x] Remove print and draw statements
[ ] sort the atoms in a block definition according to their name
[ ] deal with chirality/tacticity
[x] improve the charge equalisation; also round charges so they are at most 2 significant digits
[x] test on charged molecules
[x] make pysmiles optional dep. and raise Error if not installed
[x] How many repeat units are needed???
[ ] add a warning that extract links only work with linear molecules atm
[ ] add help text to CLI

Known Issues

sometimes the first match picks a repeat unit that is not at the start of the chain. In that case, all other matches are likely messed up. The workaround is to provide the first smile with all the correct hydrogen atoms
if there are too few repeat units to extract all representative interactions between monomers the small fragment will work correctly but the larger ones will be incorrect
only works with all-atom at the moment

@fgrunewald current version (commit a39c3acb06614d99fa6181a702e62123f823561f) gives the following error:
Traceback (most recent call last):
  File "/Users/alessandri/miniconda3/envs/itp_to_ff/bin/polyply", line 281, in <module>
    main()
  File "/Users/alessandri/miniconda3/envs/itp_to_ff/bin/polyply", line 277, in main
    subprogram(**args_dict)
TypeError: itp_to_ff() got an unexpected keyword argument 'tolerance'

with commands like:
polyply itp_to_ff -i PS_n4_AA_oplsLigParGen.itp -sm '[CH](c1ccccc1)[CH2]' -rn PS -o new_OK.ff

Just created a fresh conda environment, switched to branch, installed but still get the same.
Commands:
conda create -n itp_to_ff_2  python=3
conda activate itp_to_ff_2
git clone https://github.com/marrink-lab/polyply_1.0.git
cd polyply_1.0.git
git remotes/origin/itp_to_ff
git switch -c itp_to_ff
pip install -e .        # in the polyply folder
pip install pytest
pip install pysmiles

These are the installed packages and versions:
Successfully installed decorator-4.4.2 networkx-2.8.8 numpy-1.26.2 pbr-6.0.0 polyply-1.5.1.dev102 scipy-1.11.4 tqdm-4.66.1 vermouth-0.9.6
Successfully installed iniconfig-2.0.0 packaging-23.2 pluggy-1.3.0 pytest-7.4.3
Successfully installed pysmiles-1.1.2

I get the same error as in the previous post.

@ricalessandri should be fixed now; the problem was on my end where an old polyply was giving the correct result. I now removed the decimals option again.

@fgrunewald Now the code runs fine but the charge equalization (and rounding) doesn't seem to be working for me. For example, these are the charges of a CH3 termini:
[ moleculetype ]
CH3ter 3
[ atoms ]
1 off_001 1 CH3ter C0 1 -0.089577318182 12.01078
2 off_004 1 CH3ter H1 1  0.032722681818 1.007947
3 off_004 1 CH3ter H2 1  0.032722681818 1.007947
4 off_004 1 CH3ter H3 1  0.032722681818 1.007947

with -0.089577318182+0.032722681818*3 =0.008590727271999984 . Same for the block (this is polypropylene):
[ moleculetype ]
PP 3
[ atoms ]
1 off_001 1 PP C0 1 -0.065677318182 12.01078
2 off_001 1 PP C1 1 -0.090077318182 12.01078
3 off_001 1 PP C2 1 -0.075377318182 12.01078
4 off_004 1 PP H3 1  0.048722681818 1.007947
5 off_004 1 PP H4 1  0.033389681818 1.007947
6 off_004 1 PP H5 1  0.033389681818 1.007947
7 off_004 1 PP H6 1  0.033389681818 1.007947
8 off_004 1 PP H7 1  0.040722681818 1.007947
9 off_004 1 PP H8 1  0.040722681818 1.007947

total charge is -0.0007948636380000207. Also, in here you indicated rounding to at most 2 significant digits as done but this seems not to be the case.

@ricalessandri can you provide the original itp-file? What are the total charges in these blocks? 2 decimals was my intended goal but that's not possible given that the charge neutrality needs to be given for systems sizing up to millions of residues
Perhaps a different optimizer yields better properties. Could you try to map out how well it works for the polymers you already have?

I sent the itp to you by email. I will try it out on some more examples.
On Sun, Jan 21, 2024, 12:51 Dr. Fabian Grunewald @.***>
wrote:

@ricalessandri https://github.com/ricalessandri can you provide the
original itp-file? What are the total charges in these blocks? 2 decimals
was my intended goal but that's not possible given that the charge
neutrality needs to be given for systems sizing up to millions of residues
Perhaps a different optimizer yields better properties. Could you try to
map out how well it works for the polymers you already have?
—
Reply to this email directly, view it on GitHub
https://github.com/marrink-lab/polyply_1.0/pull/327#issuecomment-1902726352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACREAS44XABQLW47TI2UKNLYPVPSNAVCNFSM6AAAAAAZJIGHNKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBSG4ZDMMZVGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@ricalessandri I see what's going wrong. There is no charge balancing being done because at the moment it requires the entry point to be a toplogy file (i.e. .top). See here:
https://github.com/marrink-lab/polyply_1.0/blob/39f7ad4e9b5923c18ff8a529294cabe877b44a9c/polyply/src/itp_to_ff.py#L65-L74
You can test if it works by removing the if statement in Line 66. Otherwise, I can have a look tomorrow morning. Just note that processing itp files is at the moment pointless because charges won't get balanced.

Small update. I tried commenting that line out, this is the error I get:
Traceback (most recent call last):
  File "/Users/alessandri/miniconda3/envs/itp_to_ff_2/bin/polyply", line 279, in <module>
    main()
  File "/Users/alessandri/miniconda3/envs/itp_to_ff_2/bin/polyply", line 275, in main
    subprogram(**args_dict)
  File "/Users/alessandri/polyply_1.0-itp_to_ff_2/polyply/src/itp_to_ff.py", line 70, in itp_to_ff
    topology=top,
             ^^^
UnboundLocalError: cannot access local variable 'top' where it is not associated with a value

I also tried to just pass an actual top file instead of the itp file to itp_to_ff but then I get:
Traceback (most recent call last):
  File "/Users/alessandri/miniconda3/envs/itp_to_ff_2/bin/polyply", line 279, in <module>
    main()
  File "/Users/alessandri/miniconda3/envs/itp_to_ff_2/bin/polyply", line 275, in main
    subprogram(**args_dict)
  File "/Users/alessandri/polyply_1.0-itp_to_ff_2/polyply/src/itp_to_ff.py", line 39, in itp_to_ff
    mol = top.molecules[0].molecule
          ~~~~~~~~~~~~~^^^
IndexError: list index out of range

@ricalessandri should be fixed, give it a try

@ricalessandri that's it; except for the improvements on labeling edges the code is done and has all tests required. I ran it on my CHARMM database and will try some OPLS tomorrow. Please give it a try and see if there are any problems.


use free variable for native fetch

i have ABSOLUTELY NO IDEA how this passed CI in the first place, but apparently #5041 broke a bunch of stuff somehow? my browser suddenly started complaining that you can't call fetch if it's a property of something that isn't window. seems like a weird constraint but who am i to judge.
no changeset because #5041 wasn't released yet, so no need
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

Huh. Are the failures on Safari/Firefox? We were fooled by green checks pretty hard.

No, just running pnpm test locally. Truly weird

Somewhat recently, there was a change to have Playwright use whatever version of Chrome it could find on your system, rather than using separate downloaded binaries. I wonder whether that could be related to the difference in behavior in the tests?
I'd be in favor of reverting that change anyway. Requiring the user have Chrome installed on their system makes local dev less portable, and not using a specific version makes tests less reproducible.

My latest local builds failing for perhaps the same reason:
> Using @sveltejs/adapter-vercel
✘ [ERROR] Could not resolve "@sveltejs/kit/install-fetch"

    .svelte-kit/vercel-tmp/serverless.js:1:29:
      1 │ import { installFetch } from '@sveltejs/kit/install-fetch';
        ╵                              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

  The path "./install-fetch" is not exported by package "@sveltejs/kit":

    node_modules/@sveltejs/kit/package.json:47:13:
      47 │   "exports": {
         ╵              ^

  You can mark the path "@sveltejs/kit/install-fetch" as external to exclude it from the bundle,
  which will remove this error.

> Build failed with 1 error:
.svelte-kit/vercel-tmp/serverless.js:1:29: ERROR: Could not resolve "@sveltejs/kit/install-fetch"

Hope this helps.


Outline view from Sections/Synopsis/notes

Hi superlou,
Thank so much for writing this fountain package it's great to have it in Atom.
I have a feature request for you: an outline view based upon section/synopsis/notes entries in fountain (#section one, ## section two, =synopsis, [[note under symposis]] [[second note]], etc).
This page gives a good example:
http://slugline.co/blog/outlinefromfountain
I don't expect a rendered outline like in the example (right side of the image in the example), more of a working outline navigator (left side of the image in the example) similar to the work you have done with the scene list navigator except containing sections, synopsis, and notes maybe).
I'm trying to write my own version/branch, but I'll be honest I'm not sure I fully understand how things are rendered in Atom(spacepen is depreciated, or it isn't.. I cant tell!! I've also written a few loops that froze Atom :p) I'll keep taking stabs at it, but you'd probably do a better job so I'm tossing this up to you if you feel inclined.  A foldable combo of sections+synopsis+notes+scenelists would be amazing (using a tree-view navigator) anyway I'll leave considerations and methods to you if you get inclined to pursue this.
If you really wanted to get crazy you'd look into rearranging order by dragging sections up/down in the tree-view and reflect in the source as block moves, thereby turning the view into a sortable 'index card' like system and making all the other fountain implementations jealous.

Thanks for the idea!  That outline looks pretty slick.  I was hoping to start integrating something like that into the Atom package, but the parsing engine being used is pretty limited with respect to linking the extracted sections to the actual text.  Being able to rearrange sections by dragging was on the list of goals, but this limitation made it difficult.  As you noticed, Spacepen is deprecated, and I was hoping to switch to whatever's the best practice templating system before I make too many changes.  I don't know if I'm going to be able to look into it in the near term, but I'd be happy to apply pull requests.  This isn't abandoned, but might be a little bit before I can get back to it.

For me the outline is the best use of fountain. Portability is nice, but being able to sketch and outline within the text without using some hokey notes system is a feature that sets it above standard screenplay editors.
No rush on development. I'm still figuring things out. so there is a chance you'll still beat me to implementation.
I spent a good deal of last night trying to make heads of what to use for a framework. The going consensus was vue.js or etch, but there are a good number of people that want to use custom elements in raw js/coffeescript. Nothing seems firm.
I did notice the block level limitations of the editor. Seems like they should have an API to abstract and catalog things so you can make manipulations on the text without needing to scan through it all the time.
I'll see if I can make some production ready code. If nothing else I'm learning somethings and your start has been valuable for that.

I essentially created all of these features except the reordering (which is a bit over kill since it's a text format).
Could I get write access to get my fork as a branch to yours?

Awesome!  Added you as a collaborator.  If you create a branch, I'll try it out on my end.

Great, Thanks!
On Sat, Mar 26, 2016 at 10:05 AM superlou<EMAIL_ADDRESS>wrote:

Awesome! Added you as a collaborator. If you create a branch, I'll try it
out on my end.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/superlou/fountain-atom/issues/2#issuecomment-201893448

Created the branch. There are just some modifications to /lib/fountain.coffee. I added a /lib/fountain-outline-view.coffee file and added a few styles to the /styles/style.css file.  The Last Birthday.fountain is added to /lib as a test. It's actually from the creator of fountain.
I really wish there was a way to easily resize panels(I see panes, but no panels got a resize feaure -- not that I can make heads or tails of actually implementing). I'd also prefer this on the left side with a  macro to set the file view to auto hide or something. No idea how I'd do that. I wish they would update the docs for stuff like that.

@morekitsch, that looks really slick.  There're other packages doing resizable panes, but I think they're manually watching for mouseclick events to perform dragging.  I'm really hoping that Atom builds the pane dragging into core sometime soon.  Would really fix some consistency issues between packages.
Given the huge overlap in functionality between the scene list and the outline view, would you want to merge the two tools together, especially as my quick scene list parser seems to incorrectly identify some action in The Last Birthday as scenes?  I think you're outline view can replace the scene list functionality just by using Atom toggle boxes in the upper right of your pane to toggle listing of scenes and sections.  That way someone can get your hide scenes functionality by turning off scenes, or getting a traditional scene list by turning of sections, or get the full outline by leaving both on.

Thanks! I had fun making it. I'd be happy to merge. Anyway it would give
ctrl+alt+o to the outline.
I tried to steal the resize pane from the tree view, but decided it was
becoming a bit too crufty and jumbled together.
On Mon, Mar 28, 2016 at 5:53 PM superlou<EMAIL_ADDRESS>wrote:

@morekitsch https://github.com/morekitsch, that looks really slick.
There're other packages doing resizable panes, but I think they're manually
watching for mouseclick events to perform dragging. I'm really hoping that
Atom builds the pane dragging into core sometime soon. Would really fix
some consistency issues between packages.
Given the huge overlap in functionality between the scene list and the
outline view, would you want to merge the two tools together, especially as
my quick scene list parser seems to incorrectly identify some action in The
Last Birthday as scenes? I think you're outline view can replace the scene
list functionality just by using Atom toggle boxes in the upper right of
your pane to toggle listing of scenes and sections. That way someone can
get your hide scenes functionality by turning off scenes, or getting a
traditional scene list by turning of sections, or get the full outline by
leaving both on.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/superlou/fountain-atom/issues/2#issuecomment-202648321

Hi,
I was I saw that you have a few updates, but the view I made isn't in them. We're you expecting some more work from me? Happy to do it I just thought the integration stuff fell under your auspices. The code I have in my branch just needs the ctrl-alt-o call changed to point to my ctrl-alt-h call then the -h one deleted. Agin I'm happy to do it I just thought it would be better for you since it's your codebase.

I actually thought you were working on merging your new outline view with the scene view.  Sorry!  If you update the branch from the master with the integration, it can be pulled in so at least you have the outlining.  I haven't had the time to do much other than really minimal tweaks like the one that just got released.

Updated and made a pull request. This will make Outline View replace Scenelist (since they do the same thing). Should be painless.
I should have made a comment sooner we could have done this three months ago.  Thanks!

Looks good.  Can you update the pull request with the latest grammar which implements some of the emphasis markdown?

Sure. I never touched the grammar so I figure it would just merge past it.  If that is the case I'll Just redo my branch with what I have local instead of just changing the few files I know of. Just a minute. This will be based on 0.4

Thanks.  I've gotten a bit skittish about how APM updates, since last time I went up two minor versions unintentionally.

That should do it. local version has italics, underline, and bold working. Bold is sort of reddish, underline and italics look the same. that's correct if I recall.

I can see why I deleted a full section of yours and it said it could auto update I thought that was weird. I should read up on this more. This is actually my first time doing much with a shared repo. hence not knowing what I was allowed to do.

Same here.  I think I merged in the pull request.  I had to do it manually because I really ticked off APM when I tried to undo the double minor version bump.  Version 0.5.0 should be released with your outline.  Thanks for the great contribution!

@morekitsch Just checking to make sure this is working as you expect.  If so, I'll close this issue.  Thanks!

Closing unless there's further feedback.

Nope thanks!
On Sun, Nov 20, 2016, 9:24 AM superlou<EMAIL_ADDRESS>wrote:

Closed #2 https://github.com/superlou/fountain-atom/issues/2.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/superlou/fountain-atom/issues/2#event-865663441, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIVf1ItT8i1pSqzq775uKFbWqfKaS6Pyks5rAIJlgaJpZM4HDgnL
.


Remove unneccessary AppsandFeaturesEntries * manifests/p/PPSSPPTeam/PPSSPP/1.15.3/PPSSPPTeam.PPSSPP.installer.yaml

Microsoft Reviewers: Open in CodeFlow

URL:
Invalid URI: The hostname could not be parsed.
(Automated message - build 899)


强制路由，高并发  no database route info

Question
使用强制路由，按照官方说明使用，高并发查询时 报错 no database route info
For English only, other languages will not accept.
Before asking a question, make sure you have:

Googled your question.
Searched open and closed GitHub issues.
Read documentation: ShardingSphere Doc.

Please pay attention on issues you submitted, because we maybe need more details.
If no response more than 7 days and we cannot reproduce it on current information, we will close it.

For English only, other languages will not accept.


Allow specifying cache outputs outside of package folder

This allows people to specify outputs outside of the package folder from which the command originated, which is especially useful for code-generation tools and polyglot repos. To enable this, we store artifacts in the cache relative to the root of the monorepo (see screenshot) below for before (top) and after (bottom).

Since we are changing how cache is structured, I updated the global cache key in turbo in order to prevent prior old artifacts from ever ever being cache hits with after this change
The only downside to this change, which might not be a downside at all, I guess, is possibility that artifacts are now tied to the directory structure of outputs relative to the root of the monorepo.

Would this mean when I have a package package1 which part of its build or deploy uploads files to a CDN like Google Cloud Storage which final url would depended on my package2 I could save the url to a file and the package2-package directory and have the cache recover the file when a cache hit occurs?

Could this change also be used in the future to enable adding files outside the package dir to a specific task's dependencies (see #706)?
I realize this isn't it, but does this change make it easier?


Development build of styled-components not liking compiled script containing styled components being embedded

Environment
System:

OS: macOS 10.15.2
CPU: (8) x64 Intel(R) Core(TM) i7-8559U CPU @ 2.70GHz
Memory: 95.38 MB / 16.00 GB
Shell: 5.7.1 - /bin/zsh

Binaries:

Node: 10.17.0 - ~/.nvm/versions/node/v10.17.0/bin/node
Yarn: 1.21.1 - /usr/local/bin/yarn
npm: 6.11.3 - ~/.nvm/versions/node/v10.17.0/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman

npmPackages:

babel-plugin-styled-components: ^1.10.0 => 1.10.6
styled-components: ^3.2.3 => 3.4.10
`

Reproduction
When importing a compiled react app into a seperate div on the page, the development versionn of styled-components is going crazy. I do not get an error or warning but some rollover styles are broken.
This doesn't seem to happen when building production builds.
I have looked through other issues but since this is a compiled app being imported it is not quite the same.
Is this safe to do in production builds?
Will this PR fix? https://github.com/styled-components/styled-components/pull/2954

Sorry for the late reply! I think this hasn’t received a more prompt response because there are a lot of other issues with the same question being answered. I know this is a little confusing since the answer has changed across versions though 😅
Either way, as the issue template states, this is a good one for StackOverflow or Spectrum, but I think the search may also bring up previous issues.
Basically our current note on this is:

If you’re using SSR you may only have one version at a time on a given page. That’s a limitation due to Rehydration.
Otherwise you should be able to use multiple versions of styled-components at the same time.

In case you need SSR you do need to ensure that only one instance is active on the page. You could achieve this either by bundling with a DLL shared bundle in a Webpack or by always falling back to the UMD bundle.
Now, this advice applies to v4/5. Have you tried maybe using a newer version, I see you’re on v3 there.


Bug fixing.

I went through and tested most commands. These are the fixes that were immediate and necessary.

Looks good :+1:


Add dialog for unknowns in disparity bar chart

Addresses #512
I made up this UX

Note: this PR tackles UI related changes and doesn't tackle data issues described in issue #291


Optimize adler32 using rvv

We use upstream toolchain and run it on SiFive internal FPGA /w p470 bitstream:

Benchmark
SpeedUp (RVV / Scalar)

adler32/2048
1.662077018

adler32/4096
1.797862857

adler32/32768
1.601873082

adler32/262144
1.701205598

adler32/2097152
1.799026509

adler32/4194304
1.790676414

Is this implementation maybe held up by a slower multiply accumulate? Other architectures tend to fair bigger improvements from this. It is possible to do it with pure widening adds, saving the multiply accumulate for the end.

It cause data error (Z_DATA_ERROR) in 2 unit tests.
731 - minideflate-file_compress (Failed)
746 - fuzzer_example_dict (Failed)

It's strange..., let me try to fix it first.

Is this implementation maybe held up by a slower multiply accumulate?

Yes, I believe that the functionality of vector add & mul would not be slow generally.

It is possible to do it with pure widening adds, saving the multiply accumulate for the end.

Sorry, I cannot move mul to the end in the formula.

Looking at your implementation, it looks like you are deferring the multiply-accumulate to the very end (just prior to rebasing to modulo adler base).
One other thing you could try, assuming your implementation can gang the vector units that wide, is a 512 bit wide version.  Unrolling the loop this way was beneficial for the PowerPC implementation and the AVX512 one as well.

One other thing you could try, assuming your implementation can gang the vector units that wide, is a 512 bit wide version. Unrolling the loop this way was beneficial for the PowerPC implementation and the AVX512 one as well.

Thank you for your advice. But we don't have enough vector register to do unroll, I almost used all vector register ...


[Core] Add a function to cache results from coroutine functions

Description
What - Adding a function to cache results from coroutine functions
Why - The currently caching utils cache_iterator_result caches results from async generators only. If third party's want to cache results from a coroutine, they result to using the event context directly, by introducing this we improve QOL a step further and adhere to DRY.
How - The addition is basically a replica of the existing cache_iterator_result decorator, however, this is designed to accommodate coroutines by awaiting results as opposed to iterative yield with async for.
Type of change
Please leave one option from the following and delete the rest:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] New Integration (non-breaking change which adds a new integration)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Non-breaking change (fix of existing functionality that will not change current behavior)
[ ] Documentation (added/updated documentation)

Functionality has been added in another PR. https://github.com/port-labs/ocean/pull/974


Security Issues

There are a couple security issues with this library that I want to talk about.
Ultimately, I think 6 digit TOTPS should NOT be used for magic links (if it's your sole means of authentication).
6 digits is far too less entropy to be effectively secure, even with "security" measures such as storing maxAttempts in the session (which you can easily clear on the client).
If you want to use it as a magic link you'd need an extremely bulletproof rate-limiting mechanism amongst other things. But even then I wouldn't recommend going for 6 digit codes. With 6 digits you only need a maximum of 1,000,000 attempts to brute-force a TOTP.
It would be awesome to use a 6 digit code for logging in users / signing them up, this would be a great win for the user experience but it is sooooo easy to get something, even the tiniest parts wrong and it will nullify your whole authentication mechanism.
I've seen people in issues trying to use a 10 minute period for a 6 digit code. DO NOT DO THAT UNDER ANY CIRCUMSTANCES.
Don't get me wrong I like this library, I actually went through the exact same process of trying to implement totps for logins / sign ups but it is simply not secure secure and it took me a while to really figure out.
Scenario:
TOTP Validity Period: 5 minutes (300 seconds)
Attack Rate: 100 attempts per second
Total Attempts in Validity Period: 100 300 = 30,000
With 30,000 attempts, an attacker has a 3% chance (30,000 / 1,000,000) of guessing the correct TOTP within a 5-minute window. This probability increases with longer validity periods or higher attack rates.
You might say 3% is reasonably low but in the world of security it should be effectively impossible to brute force a magic link.
In reality, an attacker wouldn't necessarily need all 1,000,000 attempts. The average number of attempts needed to guess a 6-digit code is actually around 500,000 (half of the total possibilities). This is because each guess has a chance of being correct, and on average, you'll find the correct code halfway through all possibilities.
And while the code becomes invalid at some point, the attacker can just simply request a new token and do it all over again. With distributed attacks this attack vector becomes even more serious. Totps are simply not suitable for magic links (if it's your sole means of authentication as in not used as 2fa) no matter what you do and most people probably won't even think about rate limiting or how much they increase the chance of a successful attack if they increase the period to 10 minutes.
I don't mean to offend, I went through the exact same learnings but tis library is not secure and should not be used in the way it is currently intended.

You don't mean to offend, but you either offer solutions.
Also, not sure if you are aware that you will probably not need to complain here first, but instead on @epic-web/totp, as it's the base TOTP generation we are using and all the library works based on it.
Deleting this in 24 hours.


Added a  static function that calculates the difference between two d…

…ates.  Returns and integer that represents the difference between two dates entered by the user.

#10 resolved
Thanks for your contribution @JaxTurboNerd


Regression bug: Email parameter missing in CustomMailer interface

While implementing #383, an unnecessary change was only partially reverted. This left the CustomMailer interface with a missing Email instance parameter. This should be restored.

v7.7.1 released.


Button

it's add button in nowplaying back + skip + disconnect + pause + resume + loop

button to change queue page

@Spiderjockey02

This has been completed.


Add STAC_BROWSER_URL env var

We missed a currently required ENV variable.

@anilnatha give me a few this PR has an error.

@anilnatha so this will prevent the error, however for the feature to correctly work we need https://github.com/MAAP-Project/stac_ipyleaflet/pull/117 and a new release.

@wildintellect The env var addition here looks good to me, can I go ahead and merge this PR and handle the 0.3.4 update separately?

Yes that should be ok, this add will at least prevent the widget from throwing an error.


Ability to add custom words to the dictionary

Would you accept a PR for this?
Any pointers?

I guess without changes to your code I could do
dict.dictionaryTable["newword"] = [[]];


Multiple Class parameters?

I want to use multiple Class objects as parameters for a test, but I can't seem to perform the correct series of incantations to keep JUnitParams happy. My main issues are:

@Parameters("blah.class") doesn't work. I'd expect it to produce a Class<blah>, instead I get an java.lang.IllegalArgumentException: argument type mismatch.

If I use source = or method = and return  blah.class then it works. But I can't return multiple .class types.

Some example code below. The test currently works, but try switching the provideData method to the commented-out one and it doesn't.
Am I doing something wrong or is it a problem in JUnitParams?
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.containsString;
import static org.hamcrest.Matchers.is;
import static org.hamcrest.core.IsEqual.equalTo;

import org.hamcrest.Matcher;
import org.junit.Test;
import org.junit.runner.RunWith;

import junitparams.JUnitParamsRunner;
import junitparams.Parameters;

@SuppressWarnings("TestMethodWithIncorrectSignature")
@RunWith(JUnitParamsRunner.class)
public class BugTest {

    @Test
    @Parameters(method = "provideData")
    public void class_source(Class myClass) throws Exception {
        assertThat(myClass.toString(), containsString("My"));
    }

//    @Test
//    @Parameters({"MyFirstClass.class"})
//    public void class_string(Class myClass) throws Exception {
//        assertThat(myClass.getSimpleName(), is(equalTo("MyFirstClass")));
//    }

//    public static Object[] provideData() {
//        return new Object[]{new Object[]{MyFirstClass.class, MySecondClass.class}};
//    }

    public static Object[] provideData() {
        return new Object[]{new Object[]{MyFirstClass.class}};
    }

    class MyFirstClass {

    }

    class MySecondClass {

    }
}

According to PR #42 you must provide a fully qualified class name for the automatic conversion to work. Check the test to the PR, it has an example.

Well that solves problem 1, which is enough for me to be able to use this stuff in my tests. So thanks :)  (I'm surprised I didn't try fully qualified name in the first place, given that a third party class is trying to convert a string into a type name!)
But I can't see how to solve problem 2. i.e. in the code below class_string now works and passes, thanks to your info, but class_source fails with java.lang.IllegalStateException: While trying to create object of class class java.lang.Class could not find constructor with arguments matching (type-wise) the ones given in parameters.
package lol;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.hamcrest.Matchers.containsString;
import static org.hamcrest.Matchers.is;
import static org.hamcrest.core.IsEqual.equalTo;

import org.hamcrest.Matcher;
import org.junit.Test;
import org.junit.runner.RunWith;

import junitparams.JUnitParamsRunner;
import junitparams.Parameters;

@SuppressWarnings("TestMethodWithIncorrectSignature")
@RunWith(JUnitParamsRunner.class)
public class BugTest {

    @Test
    @Parameters(method = "provideData")
    public void class_source(Class<?> myClass) throws Exception {
        System.out.println(MyFirstClass.class);
        assertThat(myClass.toString(), containsString("My"));
    }

    @Test
    @Parameters({
            "lol.BugTest$MyFirstClass",
            "lol.BugTest$MySecondClass"
    })
    public void class_string(Class<?> myClass) throws Exception {
        assertThat(myClass.getSimpleName(), is(equalTo("MyFirstClass")));
    }

    public static Object[] provideData() {
        return new Object[]{new Object[]{MyFirstClass.class, MySecondClass.class}};
    }

    class MyFirstClass {

    }

    class MySecondClass {

    }
}

Your provider method should be
public static Object[] provideData() {
        return new Object[]{new Object[]{MyFirstClass.class}, new Object[]{MySecondClass.class}};
    }

Every inner array is one test run and every array element is a method parameter. You want two runs with one parameter each

PEBKAC in both cases.
Thanks for the help 👍


DRAFT: Bookend tests with latest & oldest Python dependencies

Need a PR to get actions running.
edit from (@mosesyhc):
This PR continues the effort of #134 as a clean PR after the merge/revert problems.
The file changes in #134 should be considered as well as the ones in here.

This PR mainly contains bookend tests and test_new_cal, test_new_emu test suite modules.
#134 mainly contains test_emu_cal test suite module.

@jared321:
To me, this PR is ready for v0.3.
Instead of asking for a full review for this PR, I suggest merging and request for a full review on #121, which contains the combined changes of the collection of branches relevant to updating the test suite.

This PR has 12 files changed.  For me that's already a somewhat large PR, which can make it harder for a reviewer to find all issues, especially subtle issues or single changes that span multiple files.  The review of #121 will be even harder since it is massive.  In my mind, that review will be most effective at trying to get the global picture of all that was done, checking that all necessary tasks were performed, checking for issues at that level, rechecking at a low-level as an extra level of security, and convincing ourselves that the release is ready.
If you don't mind, I'll review this as best as I can here as the "official" review and then re-review with #121 as more of a sanity check.
Food for thought, what would these layers of checks look like if your workflow has release branches that don't merge into a branch like main?

I do not mind at all. Please review this PR as best as we can.
I intended to reduce the number of reviews one has to do, but I agree with your analysis about the size of this PR and #121 being even larger.  It may be then helpful to outline also the expectation as we move forward to reviewing #121.
My impression is that after a release is done, PRs to those release branches should be minimal / patches, instead of any big changes of functionality like in this PR. If such changes are indeed needed, then there should be a new release, instead of working on that release branch.

From: jared321 @.>
Sent: Monday, August 26, 2024 11:26 AM
To: bandframework/surmise @.>
Cc: Moses Chan @.>; Mention @.>
Subject: Re: [bandframework/surmise] DRAFT: Bookend tests with latest & oldest Python dependencies (PR #135)
This PR has 12 files changed. For me that's already a somewhat large PR, which can make it harder for a reviewer to find all issues, especially subtle issues or single changes that span multiple files. The review of #121https://urldefense.com/v3/__https://github.com/bandframework/surmise/pull/121__;!!Dq0X2DkFhyF93HkjWTBQKhk!SEtWrEAgUjNWKwiY4PTnJDpDWPJsoDwnLeRjMvdA6eqHRe2cbTkUAOEr2Ea5Vg0Iu1WHHJGr5eD7A0RXPqtS6nPY79bjf0FG$ will be even harder since it is massive. In my mind, that review will be most effective at trying to get the global picture of all that was done, checking that all necessary tasks were performed, checking for issues at that level, rechecking at a low-level as an extra level of security, and convincing ourselves that the release is ready.
If you don't mind, I'll review this as best as I can here as the "official" review and then re-review with #121https://urldefense.com/v3/__https://github.com/bandframework/surmise/pull/121__;!!Dq0X2DkFhyF93HkjWTBQKhk!SEtWrEAgUjNWKwiY4PTnJDpDWPJsoDwnLeRjMvdA6eqHRe2cbTkUAOEr2Ea5Vg0Iu1WHHJGr5eD7A0RXPqtS6nPY79bjf0FG$ as more of a sanity check.
Food for thought, what would these layers of checks look like if your workflow has release branches that don't merge into a branch like main?
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/bandframework/surmise/pull/135*issuecomment-2310596221__;Iw!!Dq0X2DkFhyF93HkjWTBQKhk!SEtWrEAgUjNWKwiY4PTnJDpDWPJsoDwnLeRjMvdA6eqHRe2cbTkUAOEr2Ea5Vg0Iu1WHHJGr5eD7A0RXPqtS6nPY70ChLJ_M$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AM5N5YR33W6WRQ4YYKGYLITZTNJJZAVCNFSM6AAAAABM66IBBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJQGU4TMMRSGE__;!!Dq0X2DkFhyF93HkjWTBQKhk!SEtWrEAgUjNWKwiY4PTnJDpDWPJsoDwnLeRjMvdA6eqHRe2cbTkUAOEr2Ea5Vg0Iu1WHHJGr5eD7A0RXPqtS6nPY79o3ecZU$.
You are receiving this because you were mentioned.Message ID: @.***>


Center title in completion view, fix build by upgrading version of flutter_platform_widgets, and fix a test

Sorry for the late response and much thanks for your contribution!
The reason why the linked PR is not merged yet is that the conflicts were not resolved. I will  update the dependencies soon, sorry for the inconvinience.


Remote storages support: added container runtime package and refactor stages storage for distributed builds

refs https://github.com/flant/werf/issues/1614


Failed to build project

Hi, since I recently encounter a requirement for visualizing large datasets. Google Facets gives a link for this facinating project. But when I try to build the project for the first-time use, the errors pop up in my Intellij IDEA, below are sample errors:
Error:(18, 8) not found: object featureStatistics
import featureStatistics.feature_statistics.FeatureNameStatistics.{Type => ProtoDataType}

Error:(34, 36) not found: type ProtoDataType
                     `type`      : ProtoDataType,

I'm new to scala and maven, could you please help me for this issue? Thank you in advance.

problem solved


Fixed issues with blend modes in OpenGL 3 renderer

This changes OpenGL behaviour to match the other renderers:

Changed GL_FUNC_REVERSE_SUBTRACT to GL_FUNC_SUBTRACT
Added uses_blend_alpha = true for ADD, SUBTRACT, and MULTIPLY blend modes

Fixes #76334

Thanks for taking a look at this. It is super helpful to have others working on the opengl backend

Thanks for taking a look at this. It is super helpful to have others working on the opengl backend

I appreciate this! There's always some challenge with approaching a new open source project, so this feels good to hear. I also believe that one of the top 2 or maybe the top value of a game engine is portability across all devices, including old ones, so the OpenGL backend remains very important.

Thanks!


[PyTorch] Fix backward compatibility for checkpoint API

A follow-up to #740

/te-ci pytorch

I tested this with both pretraining and LoRA.
In case of LoRA, this is one of the input types passes ad

/te-ci pytorch


source: isn't a consistent source of truth

As noted in #462 our license texts don't all match precisely the texts their source: fields point to. I'd like to fix that eventually, probably by treating license texts curated by SPDX as canonical.

Implementation needs tests. Would detect and help us avoid issues such as #310 #486 #488
If we can manage to use SPDX as the canonical source of license texts, then we don't need the source field at all, as SPDX URLs can be constructed from spdx-id.

If we can manage to use SPDX as the canonical source of license texts, then we don't need the source field at all, as SPDX URLs can be constructed from spdx-id.

See https://github.com/github/choosealicense.com/pull/489. I wrote a quick script to vendor in the license text from the SPDX API. The biggest change is the lack of line wrapping (doesn't affect the site or Licensee), and we'll need to normalize the merge fields, if we wanted to go that route.


Attempt to remove unnecessary CA/SA suppressions in Blazor projects

🧹 Tech Debt
We've introduced a number of CA/SA suppressions particularly in the Blazor example projects. Locally, I was able to remove the majority of them and successfully build in Release, so it's unclear why they're there at the moment. We should wait until the CI/CD pipeline is established for the Blazor projects, and then attempt to remove them.

There don't seem to be many instances of suppressions anymore (assuming searching for "pragma" is how to find them. Closing as low ROI. @atmgrifter00 @msmithNI tagging you as FYI.


Release 3.20.40

Release 3.20.40 (c72e3f9961)

Closing as the changes intended to be in this release were actually included in 3.20.39, those will result in empty release.


Receiver class com.mysql.jdbc.Statement does not define or inherit an implementation of the resolved method abstract isClosed()Z of interface java.sql.Statement.

Description
I am using ontop 3.0.0 for my SPARQL project. I got a successfully running with OntopCLI, but when I tried to quest the SPARQL the above error name occurred. I am new to SPARQL and Ontop. Can anybody please help me to solve this problem?
Attached material

Versions
I use:

Protege v5.0.0 to create OWL
Ontop-cli-3.0.0
Java 1.8.0_231

Hi, which version of the JDBC driver are you using?
On Tue, Nov 26, 2019 at 3:57 PM Ms Panda <notifications<EMAIL_ADDRESS>wrote:
Description
I am using ontop 3.0.0 for my SPARQL project. I got a successfully running with OntopCLI, but when I tried to quest the SPARQL the above error name occurred. I am new to SPARQL and Ontop. Can anybody please help me to solve this problem?
Attached material
[Capture]https://user-images.githubusercontent.com/16506688/69609688-73e3a580-106d-11ea-875d-d45da6732df2.PNG
Versions
I use:

Protege v5.0.0 to create OWL
Ontop-cli-3.0.0
Java 1.8.0_231

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/ontop/ontop/issues/314?email_source=notifications&email_token=AAORVKYPO3CAS44DD5USSXDQVTJGLA5CNFSM4JRT6LH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H4BB5KA, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORVK4UC5R75CVCWY4FVI3QVTJGLANCNFSM4JRT6LHQ.
--
Guohui Xiao, PhD
Assistant Professor
KRDB - Faculty of Computer Science
Free University of Bozen-Bolzano
Piazza Domenicani, 3
I-39100 Bolzano, Italy
http://www.ghxiao.orghttp://www.ghxiao.org/
According to the Regulation EU 2016/679, you are hereby informed that this message contains confidential information that is intended only for the use of the addressee. If you are not the addressee, and have received this message by mistake, please delete it and immediately notify us. In any case you may not copy or disseminate this message to anyone. Thank you.

Hi, which version of the JDBC driver are you using? On Tue, Nov 26, 2019 at 3:57 PM Ms Panda <notifications<EMAIL_ADDRESS>wrote: Description I am using ontop 3.0.0 for my SPARQL project. I got a successfully running with OntopCLI, but when I tried to quest the SPARQL the above error name occurred. I am new to SPARQL and Ontop. Can anybody please help me to solve this problem? Attached material [Capture]https://user-images.githubusercontent.com/16506688/69609688-73e3a580-106d-11ea-875d-d45da6732df2.PNG Versions I use: * Protege v5.0.0 to create OWL * Ontop-cli-3.0.0 * Java 1.8.0_231 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#314?email_source=notifications&email_token=AAORVKYPO3CAS44DD5USSXDQVTJGLA5CNFSM4JRT6LH2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H4BB5KA>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAORVK4UC5R75CVCWY4FVI3QVTJGLANCNFSM4JRT6LHQ.
…
-- Guohui Xiao, PhD Assistant Professor KRDB - Faculty of Computer Science Free University of Bozen-Bolzano Piazza Domenicani, 3 I-39100 Bolzano, Italy http://www.ghxiao.orghttp://www.ghxiao.org/ According to the Regulation EU 2016/679, you are hereby informed that this message contains confidential information that is intended only for the use of the addressee. If you are not the addressee, and have received this message by mistake, please delete it and immediately notify us. In any case you may not copy or disseminate this message to anyone. Thank you.

This kind of error happened because I naively use the wrong mysql JDBC driver. When I checked it again the mysql JDBC driver that I used was released in 2011.

I resolved this problem by changing to the recent mysql JDBC driver one, then it WORKS

BUT STILL the similar error happened again. It said that com.mysql.jdbc.Driver is deprecated and suggested to use com.mysql.cj.jdbc.Driver instead. So I had to change it into com.mysql.cj.jdbc.Driver inside ontology .properties file before running Ontop CLI.
.

@sothearin-ren Thanks for the feedback. I am happy that it works for you with the correct version of the JDBC driver and driver class.


InvalidGraph: [ONNXRuntimeError] : 10 : INVALID_GRAPH : This is an invalid model. Error in Node:Unsqueeze_3 : Node (Unsqueeze_3) has input size 1 not in range [min=2, max=2].

Hello,
I want to try this notebook tutorial https://github.com/Xilinx/finn/tree/master/notebooks
I have install brevitas:
pip install brevitas
and finn by git clone this repo and:
pip install .
and other necessary python packages. However, I got this error when executing this notebook (https://github.com/Xilinx/finn/blob/master/notebooks/end2end_example/bnn-pynq/tfc_end2end_example.ipynb)
---------------------------------------------------------------------------
InvalidGraph                              Traceback (most recent call last)
<ipython-input-24-84a83ad50a7e> in <module>
      5
      6 model = model.transform(InferShapes())
----> 7 model = model.transform(FoldConstants())
      8 model = model.transform(GiveUniqueNodeNames())
      9 model = model.transform(GiveReadableTensorNames())

~/.local/lib/python3.8/site-packages/finn/core/modelwrapper.py in transform(self, transformation, make_deepcopy, cleanup, fix_float64)
    137         model_was_changed = True
    138         while model_was_changed:
--> 139             (transformed_model, model_was_changed) = transformation.apply(
    140                 transformed_model
    141             )

~/.local/lib/python3.8/site-packages/finn/transformation/fold_constants.py in apply(self, model)
     52                 # this node has no dynamic inputs, only constant ones -- so we can
     53                 # do constant folding.
---> 54                 oxe.execute_node(n, execution_context, graph)
     55                 # use the execution result as an initializer
     56                 model.set_initializer(node_out, execution_context[node_out])

~/.local/lib/python3.8/site-packages/finn/core/onnx_exec.py in execute_node(node, context, graph, return_full_exec_context)
    128                 input_dict[inp] = context[inp]
    129
--> 130             sess = rt.InferenceSession(node_model.SerializeToString())
    131             output_list = sess.run(None, input_dict)
    132

~/.local/lib/python3.8/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py in __init__(self, path_or_bytes, sess_options, providers, provider_options)
    278
    279         try:
--> 280             self._create_inference_session(providers, provider_options)
    281         except RuntimeError:
    282             if self._enable_fallback:

~/.local/lib/python3.8/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py in _create_inference_session(self, providers, provider_options)
    307             sess = C.InferenceSession(session_options, self._model_path, True, self._read_config_from_model)
    308         else:
--> 309             sess = C.InferenceSession(session_options, self._model_bytes, False, self._read_config_from_model)
    310
    311         # initialize the C++ InferenceSession

InvalidGraph: [ONNXRuntimeError] : 10 : INVALID_GRAPH : This is an invalid model. Error in Node:Unsqueeze_3 : Node (Unsqueeze_3) has input size 1 not in range [min=2, max=2].

I change nothing to the notebook. How to deal with this issue? thanks in advance

Hi Oskar. The only supported way to install FINN is in Docker at the moment. From the README:

Getting Started
Please see the Getting Started page for more information on requirements, installation, and how to run FINN in different modes. Due to the complex nature of the dependencies of the project, we only support Docker-based execution of the FINN compiler at this time.

FINN is sensitive to the versions of the packages it depends on, in this case the onnx package is probably the culprit. Please use the provided Docker infrastructure instead.

Hello,
thanks for the reply. I am sorry I didn't read it previously. Now I am able to run it in Docker.
However, I got this error when executing https://github.com/Xilinx/finn/blob/master/notebooks/end2end_example/cybersecurity/3-build-accelerator-with-finn.ipynb

Yes, I am sure that I didn't set the VIVADO_PATH previously. Could you tell me how to set the VIVADO_PATH? because I didn't see a clear explanation on this page https://finn.readthedocs.io/en/latest/getting_started.html
All I do is just:
sh run-docker.sh
from finn repo as instructed.

Can you try:
export VIVADO_PATH=/home/oskar/Xilinx/Vivado/2019.1
bash run-docker.sh

Shown this warning/error?
mv: cannot move '/workspace/finn/bdf/deprecated' to '/workspace/finn/board_files/deprecated': Directory not empty
mv: cannot move '/workspace/finn/bdf/microzed_7010' to '/workspace/finn/board_files/microzed_7010': Directory not empty
mv: cannot move '/workspace/finn/bdf/microzed_7020' to '/workspace/finn/board_files/microzed_7020': Directory not empty
mv: cannot move '/workspace/finn/bdf/minized' to '/workspace/finn/board_files/minized': Directory not empty
mv: cannot move '/workspace/finn/bdf/picozed_7010_fmc2' to '/workspace/finn/board_files/picozed_7010_fmc2': Directory not empty
mv: cannot move '/workspace/finn/bdf/picozed_7015_fmc2' to '/workspace/finn/board_files/picozed_7015_fmc2': Directory not empty
mv: cannot move '/workspace/finn/bdf/picozed_7020_fmc2' to '/workspace/finn/board_files/picozed_7020_fmc2': Directory not empty
mv: cannot move '/workspace/finn/bdf/picozed_7030_fmc2' to '/workspace/finn/board_files/picozed_7030_fmc2': Directory not empty
mv: cannot move '/workspace/finn/bdf/ultra96v2' to '/workspace/finn/board_files/ultra96v2': Directory not empty
mv: cannot move '/workspace/finn/bdf/ultrazed_3eg_iocc' to '/workspace/finn/board_files/ultrazed_3eg_iocc': Directory not empty
mv: cannot move '/workspace/finn/bdf/ultrazed_3eg_pciecc' to '/workspace/finn/board_files/ultrazed_3eg_pciecc': Directory not empty
mv: cannot move '/workspace/finn/bdf/ultrazed_7ev_cc' to '/workspace/finn/board_files/ultrazed_7ev_cc': Directory not empty
bash: /home/oskar/Xilinx/DocNav/.settings64-DocNav.sh: No such file or directory
bash: /home/oskar/Xilinx/SDK/2019.1/.settings64-SDK_Core_Tools.sh: No such file or directory
bash: /home/oskar/Xilinx/Model_Composer/2019.1/.settings64-Model_Composer.sh: No such file or directory

However, when I open the notebook again and execute all of them. It works!
So, what are these warnings/errors mean?
And also, I check only 1 core out of 12 cores is currently working. I read that the default worker is set to 1. Thus, if I want all cores to work, can I just do this? so that it will process the conversion quickly.
export NUM_DEFAULT_WORKERS=12
export VIVADO_PATH=/home/oskar/Xilinx/Vivado/2019.1
bash run-docker.sh

I have tried to set NUM_DEFAULT_WORKERS=6 in run-docker.sh but the working CPU core is still 1.

Regarding the "no such file or directory" errors for DocNav etc, you can ignore them: https://finn-dev.readthedocs.io/en/latest/faq.html#i-am-getting-docnav-and-model-composer-errors-when-launching-the-docker-image
For the multi-core transformations, setting the environment variable should be enough. You may want to verify that the environment variable is propagated correctly into the Docker container. Please be aware that not all transformations support parallel execution.

I see. thanks


Add text for registering and help with dockers, before install

Trying to address the RHEL/SLES registration & dockers, without bloating the quick start instructions.  It makes the page a bit messy, visually.
Can use some wordsmithing by writers.

Moved prereq to register Enterprise Linux distro into RHEL / SLES tabs

Register RHEL and SLES are now a prerequisite
Quick start has links that tell you to register your RHEL or SLES
The end of quick start has link to Troubleshooting
Docker FAQ/troubleshooting mentions suggested packages for dockers


Kubernetes by default

[x] Installs cleanly on a fresh build of most recent master branch
[ ] Upgrades cleanly from the most recent release
[x] Updates documentation relevant to the changes

Add Kubernetes by default, incorporating many of @zogg's changes. Docs are at https://github.com/CiscoCloud/mantl/commit/3af143e112777648da642a370bff4590fa48604c

Just rebased onto lastest master.

After rebasing onto #1296, this upgrades cleanly from 1.0.3 :+1:

@BrianHicks is this ready to merge now that we know it upgrades cleanly?

I thought it was, but it looks like Kubelet is killing the logstash and nginx-consul containers on the box when it starts up. This leads to an unhealthy status in Consul (because of the dsitributive check.)
I think it's going to require a few changes to nginx-consul and logstash (specifically to restart them when they fail), which may fall outside the scope of this PR. @ryane do you have a preference for where that should go?

I added it in 48d0058, but we can revert that commit.

I suppose we could move it to a new PR but I don't have a problem with leaving it in this one...

Cool, then it's ready to merge

On Apr 6, 2016, at 5:37 PM, Ryan Eschinger<EMAIL_ADDRESS>wrote:
I suppose we could move it to a new PR but I don't have a problem with leaving it in this one...
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub


Second print broken on Ipad Pro

Hey. This is fixed in the latest App Store release. Please update the app.

Thank you very much!El 26 sep. 2021 13:40, Ken Chung @.***> escribió:
Hey. This is fixed in the latest App Store release. Please update the app.
—You are receiving this because you modified the open/close state.Reply to this email directly, view it on GitHub, or unsubscribe.


添加创建符号链接的接口

see in https://help.aliyun.com/document_detail/45126.html

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.xiaohui.lam seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

多谢您的分享，该接口功能已经发布。

不用谢