Minor UI Fix to School/Dept Cards

Summary

Comment cards on both School and Department can now be opened by clicking the text box, in addition to the already existing dropdown icon.

Test Plan
Before:

After:

Issues
Closes #583
Additional Notes
Frankly, if I knew how to implement this without directly declaring the color in the styles, I would 😅. Hopefully my solution is chill enough :)

I know I approved this, but I changed my mind. The clicking is good. I'm not completely sold on the style change though. It makes the top look cluttered, and it isn't that distinguished in light mode. Can you just remove (reset and rebase) that change?
That resolves my other concern of not-exactly-atomic PR as well. For the sake of procedural completeness, can you create an issue (something along the lines of "comments card should be clickable") and link this PR? That's the usual workflow; a PR solves an issue.
As for the distinction, if it bothers you, put up an issue, and we can all discuss it.

Had to do a new push rather than a rebase since my initial commits weren't atomic (I can see why atomic PRs are preferred now). Other than that, everything should be good now; an issue was created and linked here


Organizing data for use cases

We have many different use cases which require data to be captured at each layer of a stack deployment.  These use cases require data to be organized for collection, storage, use, and retention.  We'd like to define these elements and determine the best structure and storage method for each:

Project
AWS Accounts
AMIs
Instances
AWS Services

Need to figure out a standard for how to store information as part of the DevSecOps model.


Why performSelfAnalysis is a static method, not a self method?

ChevrotainParser.performSelfAnalysis(this);
Why performSelfAnalysis is a static method, instead self method?
why not this.performSelfAnalysis() ?
simple implementation:
selfAnalysis() {
  this.constructor.performSelfAnalysis(this)
}

usage:
class MyParser extends Parser {

  constructor() {
    super()

    // rules
    // rules
    // rules

    this.selfAnalysis()
  }

}

much more convenient, is not it?

I don't remember why I used a static method when I wrote that code that later became Chevrotain ~4 years ago. 😄
I agree that your suggestion looks better and is easier to use and I think it should be implemented.
Maybe I should focus version 4 road on some polishing and ease of use improvements.

@bd82 Actually,  you can automatically call this method if a person tries to use a parser.
Object.defineProperty(Parser.prototype, 'input', {
  set: function(){
    if(!this._selfAnalysied) {
      Parser.performSelfAnalysis(this);
      this._selfAnalysied = true;
    }

    this.reset()
    // e.t.s.
  }

})

Actually, you can automatically call this method if a person tries to use a parser.

Yes that is very good, "input" is already a setter so it would be trivial.
But there are other APIs on the parser that expect the self-analysis to have finished.
So those would have to be taken of too:

https://github.com/SAP/chevrotain/blob/9c4f487b3b05335adbd54b1ffb3be59aa39ce958/src/parse/parser_public.ts#L822
https://github.com/SAP/chevrotain/blob/9c4f487b3b05335adbd54b1ffb3be59aa39ce958/src/parse/parser_public.ts#L841

Also we will need to remove the option to pass the input as part of the parser constructor
because the parser base constructor would be called  before the rules have been defined.
This would be a breaking change which I am fine with, but it probably mean modifying many tests and examples and would also require some kind of runtime check to easily spot incorrect usage of the deprecated APIs and providing a clear error message.

Just reconsider the whole concept, and using the experience already available, break back compatibility, and make the 4th version. It will be unsurpassed and perfect. It will take into account all your achievements and mistakes. Have not you thought about it?

I think you may have missed my last comment before writing that last response

No, I read it.

Lets chat on gitter, I think we may have a communication shortage so it would be easier to clarify it there.


How to support php 7.1

How to support php 7.1

Theoretically, it is already supported. If you are experiencing issues, feel free to report here (if it is related to this library) or report to mpdf repo (in case it is related to mpdf directly)

No, PHP 7 is not actually supported. It is required to update mpdf first, but then yii2-pdf is not compatible with mpdf v7.  To make it compatible one need to change class name from mPDF to Mpdf and Mpdf accepts single input parameter that is actually array of mPDF parameters. See post:
https://harrybailey.com/2018/08/yii-1-php-7-1-yii-pdf-and-mpdf/
I would like to update in a clean way yii2-pdf and mpdf v7. Is there an easy way to do it? Shall I fork yii2-pdf and make it work for mpdf v7?

Feel free to submit a PR, or to become a contributor :-)

I am not big git master, but I think I managed to send two pull requests with changes that might do the job. And you can mark it as new version after merge...

@robregonm i have made a PR for this issue #29


 This is almost already hackable from group nodes on your face embed, but we dont need to feed multiple insightface or face embeds, so at that point it's just a matter of accepting n images.

Yes, I have considered it.
I decided it would be too expensive considering the time I would spend writing the UI.
At first glance, it seems simple, but if you think about UX, the fact is that such added images need to be edited (removed).
Provide an option to somehow mark facial images where face can't be found (they can't just be skipped), even if they seem OK at first glance, etc.
All this seems to be at odds with the authors' vision of a comfy, "the way of spaghetti hell".
If someone is bothered by inserting and connecting all the nodes just use the script to generate the workflow, that's what it was created for.

Fair fair. Any chance you might be able to give me a bit more of an outline of your mega nodes process so I can see about porting the functionality as closely as possible to the native instantid implementation? I was under the impression inpainting generally requires an inpainting model to not generally make a mess but you see be doing quite well to get around that requirement with what you do here. With that I can use some of the standard image stacking methods since it accepts batches of images just as easily.


Prepare stage failing while trying to change the Audit account email address

Hi,
I need to change the email address for the audit account. I changed the email address inside the accounts-config.yaml file, and after that, I changed the email address on the stack "AWSAccelerator-InstallerStack" parameter "AuditAccountEmail". I ran the "AWSAccelerator-Pipeline', and it failed at the prepare stage with a following error message.

I am not creating a new account. I was only trying to change the email address of audit account. Why the error message is pointing out to "Validate that the emails in the parameter ManagementAccountEmail" The email address I was changing it is belongs to "AuditAccountEmail" parameter in "AWSAccelerator-InstallerStack" stack, not the "ManagementAccountEmail".

While I was trying to change the email address for Dev Account. I got following error at Prepare Stage

You must be logged in with the root user to change an AWS account email address (https://docs.aws.amazon.com/IAM/latest/UserGuide/root-user-tasks.html)
Cannot change it via LZA by just updating accounts-config.yaml.

Thank You Erik.
Once I change AWS account email by login as root user. Do I still have to update email address on accounts-config.yaml? Or running the [AWSAccelerator-Installer] and [AWSAccelerator-Pipeline] pipeline will fix the accounts-config.yaml. Another question is do I need to change email in Service Catalog as mention on
https://docs.aws.amazon.com/controltower/latest/userguide/change-account-email.html
"Update the provisioned product in Service Catalog using the email address that previously belonged to the account. The process for updating the provisioned product includes associating the new email address with the provisioned product. This way the email address change takes effect in AWS Control Tower. "


root_link is not set in

from adam.pytorch import KinDynComputations
KinDynComputations takes in "root_link" as an argument but never actually sets it for use with self.rbdalgos.
For example, here: https://github.com/ami-iit/adam/blob/76c8457d8b19c7f834e9e6178fb5a6232069564a/src/adam/core/rbd_algorithms.py#L132C9-L132C21
The forward kinematics utilizes self.root_link but it is never set when initializing KinDynComputations.
Repro:

https://pastebin.com/6QfEYjh8
https://pastebin.com/8fmfdLiw

@Giulero

By the way, the feature of "rotating" the URDF by changing its base link is indeed supported in iDynTree, so one could use iDynTree to rotate an URDF, and feed the resulting URDF to adam.
I do not think we ever used that API a lot on Python, so it is not particularly ergonomic, but this is a simple example in which chest is used as a base link instead of root_link for an iCub model:
import adam
import casadi as cs
from adam.casadi import KinDynComputations
import icub_models
import numpy as np
import idyntree.bindings as idyntree
import tempfile

# if you want to icub-models https://github.com/robotology/icub-models to retrieve the urdf
model_path = icub_models.get_model_file("iCubGazeboV2_5")
# The joint list
joints_name_list = [
    'torso_pitch', 'torso_roll', 'torso_yaw', 'l_shoulder_pitch',
    'l_shoulder_roll', 'l_shoulder_yaw', 'l_elbow', 'r_shoulder_pitch',
    'r_shoulder_roll', 'r_shoulder_yaw', 'r_elbow', 'l_hip_pitch', 'l_hip_roll',
    'l_hip_yaw', 'l_knee', 'l_ankle_pitch', 'l_ankle_roll', 'r_hip_pitch',
    'r_hip_roll', 'r_hip_yaw', 'r_knee', 'r_ankle_pitch', 'r_ankle_roll'
]

# Rotate the model in idyntree
idyntree_model_reducer = idyntree.ModelLoader()
idyntree_model_reducer.loadReducedModelFromFile(str(model_path), joints_name_list)

# URDF has some peculiar convention, so we need to move the link frames to be compatible with a different base link
original_model_idyntree = idyntree_model_reducer.model()
original_model_idyntree.setDefaultBaseLink(original_model_idyntree.getLinkIndex("chest"))
rotated_model_idyntree = idyntree.Model()
idyntree.moveLinkFramesToBeCompatibleWithURDFWithGivenBaseLink(original_model_idyntree, rotated_model_idyntree)

# Export the rotated model to a URDF file
idyntree_model_exporter = idyntree.ModelExporter()
idyntree_model_exporter.init(rotated_model_idyntree)

# exportModelToString would be the perfect function to call here, but it doesn ot have Python-friendly signature,
# so we call exportModelToFile instead
with tempfile.NamedTemporaryFile(delete=False) as temp_file:
    temp_file_name = temp_file.name
print("Saving temp file to ", temp_file_name)
idyntree_model_exporter.exportModelToFile(temp_file_name)

kinDyn = KinDynComputations(temp_file_name, joints_name_list)
# choose the representation you want to use the body fixed representation
kinDyn.set_frame_velocity_representation(adam.Representations.BODY_FIXED_REPRESENTATION)
# or, if you want to use the mixed representation (that is the default)
kinDyn.set_frame_velocity_representation(adam.Representations.MIXED_REPRESENTATION)
w_H_b = np.eye(4)
joints = np.ones(len(joints_name_list))
M = kinDyn.mass_matrix_fun()
print(M(w_H_b, joints))

# If you want to use the symbolic version
w_H_b = cs.SX.eye(4)
joints = cs.SX.sym('joints', len(joints_name_list))
M = kinDyn.mass_matrix_fun()
print(M(w_H_b, joints))

# This is usable also with casadi.MX
w_H_b = cs.MX.eye(4)
joints = cs.MX.sym('joints', len(joints_name_list))
M = kinDyn.mass_matrix_fun()
print(M(w_H_b, joints))

If there is an interest in this, we could even add an helper function to simplify this change of base functionality in adam itself.


CropCircleTransformation can't work on glide4.0

when i use the glide 4.0 ,it can't show the circle imagview

you can just call the method apply  and you will get all things done like this:
      RequestOptions requestOptions = new RequestOptions();         CropCircleTransformation transformation = new CropCircleTransformation(HomeActivity.this);         requestOptions.bitmapTransform(transformation);         Glide.with(this).load("url").apply(requestOptions);

@lizihanglove  I try do like this ,but it seem not avaliable
the code:
RequestOptions requestOptions = new RequestOptions();
requestOptions.bitmapTransform( new CropCircleTransformation(context));
Glide.with(context).load(imgUrl).apply(requestOptions).into(imageView);

I've tried it also. If there's an answer, please let me know

@2391645784 @494293346
The new glide 4.0 transform implementation is different than before. These glide-transforms need to get updated.
Create your own class which extends the CropCircleTransformation and override new methods
public class MyCropCircleTransformation extends CropCircleTransformation {
@Override
    public int hashCode() {
        return getId().hashCode();
    }

    @Override
    public boolean equals(Object obj) {
        return (obj instanceof MyCropCircleTransformation);
    }
    @Override
    public void updateDiskCacheKey(MessageDigest messageDigest) {
        try {
            byte[] ID_BYTES = getId().getBytes(STRING_CHARSET_NAME);
            messageDigest.update(ID_BYTES);
        } catch (UnsupportedEncodingException e) {
            e.printStackTrace();
        }
    }
}

This will get the job done.
Alternatively if you are only looking for circle crop, I'll suggest you use Glide's own CircleCrop Transformation. No need to use external glide-transformations

I just released glide-transformations 3.0.0. So could you check that?


jcb 19 digits PAN

fixes #30

Coverage remained the same when pulling 1a84da8bc28b5a9e101af4b0a7564f03c0f137ae on jcb_19 into 64ab520c2fe7b186e71f0de166640d4a917f7cf8 on master.


i18n(ko-KR): update api-reference.mdx

Description (required)

update api-reference.mdx
#9128

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
reference/api-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


virtualgl: 2.6.2 -> 2.6.5

Motivation for this change
Update and fix virtualgl. Also added instructions to wiki
Things done

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

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

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

Edit: copy of instructions since struggling with wiki update...
OpenGL
OpenGL must break purity due to the need for hardware-specific linkage. Intel, AMD, and Nvidia will have different libraries for example. On NixOS, these libraries are mounted at /run/opengl-driver/lib and /run/opengl-driver-32/lib.
When a program is installed in your environment, these libraries should be found automatically. However, this is not the case in a nix-shell. To fix, add this line to your shell.nix:
LD_LIBRARY_PATH="/run/opengl-driver/lib:/run/opengl-driver-32/lib";
VirtualGL
X11 forwarding does not work for OpenGL applications. Instead, we can use VirtualGL. You'll need to install VirtualGL on both client and server (nixpkgs.virtualgl). Then, from client
  > vglconnect <server>
  $ export LD_LIBRARY_PATH=/run/opengl-driver/lib/
  $ vglrun -c proxy  glxgears

For nvidia hardware, an additional workaround is needed to avoid 1 fps framerate. First, run nvidia-settings and save a xorg.conffile in /tmp.
  environment.etc = {
    # needed for virtualgl to fix 1 fps problem, see:
    # https://wiki.archlinux.org/index.php/VirtualGL
    "X11/xorg.conf.d/20-nvidia.conf".text = ''
      <text from xorg.conf here>
      Section "Screen"
        [...]
        Option         "HardDPMS" "false"
      EndSection
    ''';
  }

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/virtualgl-server-on-nixos/10212/2

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

3 packages built:

bumblebee
virtualgl
virtualglLib


Remove duplicates in trackers @ encoding - Fixes #8

Closing, see comment: https://github.com/feross/webtorrent/issues/563#issuecomment-170265323


hunspell-dict-de-any-igerman98: init at 20160407

Things done

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

[X] NixOS
[ ] OS X
[ ] Linux

[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

The derivation's buildPhase requires ispell for which we reuse pull request #3320 by @bennofs.

Isn't ispell deprecated? This page (https://www.gnu.org/software/ispell/) suggests to use aspell instead. (I had to look it up, because of all the ispell patches that are included in this PR.)
Hm, Archlinux has ispell, and it's smaller (almost no patching):
https://git.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/ispell
Seems a lot of those patches belong to upstream. Are they dead?

Yes, ispell is deprecated. However this hunspell dictionary requires the buildhash executable which is included with ispell to build. That is the only reason for adding ispell to nixpkgs.
I had trouble to get a more minimal ispell to build. Therefore I just reused @bennofs' earlier pull request. Maybe we can slim down the ispell derivation.

@seschwar: So you plan to work more on this PR? (I'm OK with merging as is, if you want.)

No, I'd rather not spend more time on ispell than required as it's only a build dependency.

After having a closer look, some comments:

Those commented out stuff in the Nix file ("XXX" and "sed"). Keep or discard?
ispell is missing "meta" attributes.

Needed to solve this issue: https://github.com/NixOS/nixpkgs/issues/14430

How about the following solution:

Make a derivation called ispell-buildhash:
Build only buildhash by doing make buildhash
skip all patches because they do not affect buildhash

Closing because there is no progress.


Train to upscale low quality JPEG

My goal is to train/fine-tune the waifu2x model to upscale low quality jpeg photos (quality factor around 70). How do I set the training parameter? Is --da-jpeg-p related?

waifu2x already has GAN based denoise+upscale models for photo.
You can check the results of the pre-training model for each noise level at https://unlimited.waifu2x.net/
(select swin_unet/photo for Model option.)

waifu2x (art) models has pretrained models for each noise level.
noise level 0(Low):  JPEG quality 85~95
noise level 1(Middle):  JPEG quality 65~85, +noise level 0
noise level 2(High):  JPEG quality 27~70, +noise level 0, 1
noise level 3(Highest):  JPEG quality 27~70

They are separated by --noise-level  [0-3] option.
For example, use --method noise_scale --noise-level 1 option for 2x noise-level 1.
JPEG quality 70 corresponds to noise-level 1, but in my experience, noise-level 3 is the best look.
You can check the results for each noise-level at the above website.
For training photo models, use --style photo option (--style art by default).
When --style photo is specified, additional noise types related to photo, such as grain noise, will be added in addition to JPEG noise.
The options I used for pretrained models can be found at https://github.com/nagadomi/nunif/tree/master/waifu2x/appendix

waifu2x already has GAN based denoise+upscale models for photo. You can check the results of the pre-training model for each noise level at https://unlimited.waifu2x.net/ (select swin_unet/photo for Model option.)
waifu2x (art) models has pretrained models for each noise-level.
noise-level 0(Low):  JPEG quality 85-95
noise-level 1(Middle):  JPEG quality 65-85, +noise level 0
noise-level 2(High):  JPEG quality 27-70, +noise level 0, 1
noise-level 3(Highest):  JPEG quality 27-70

They are separated by --noise-level [0-3] option. For example, use --method noise_scale --noise-level 1 option for 2x noise-level 1. JPEG quality 70 corresponds to noise-level 1, but in my experience, noise-level 3 is the best look. You can check the results for each noise-level at the above website.
For training photo models, use --style photo option (--style art by default). When --style photo is specified, additional noise types related to photo, such as grain noise, will be added in addition to JPEG noise.
The options I used for pretrained models can be found at https://github.com/nagadomi/nunif/tree/master/waifu2x/appendix

Thanks. So which arch should I use in training?

Current answers is: waifu2x.swin_unet_2x for 2x, waifu2x.swin_unet_4x for 4x.
You can find these pre-trained models under waifu2x/pretrained_models/swin_unet/.
I am currently developing a new arch in the dev branch and it can achieve PSNR +1 for art 2x.
However, it is still under development and the pre-training model will be released 1-2 months later.

Current answers is: waifu2x.swin_unet_2x for 2x, waifu2x.swin_unet_4x for 4x. You can find these pre-trained models under waifu2x/pretrained_models/swin_unet/.
I am currently developing a new arch in the dev branch and it can achieve PSNR +1 for art 2x. However, it is still under development and the pre-training model will be released 1-2 months later.

I want to 2x upscale. Can I train waifu2x.swin_unet_2x with  noise-level 1? Which loss should I use?

Can I train waifu2x.swin_unet_2x with noise-level 1?

Yes.
For noise-level, I think it is better to use the option value that you feel is the best result from the above website.

Which loss should I use?

The pre-training model was trained with --loss lbp5(--loss lbp for 2x) for PSNR optimized model and --loss l1lpips --discriminator l3v1c for GAN model (see train_photo_psnr.sh and train_photo_gan.sh ).
However, the recently added --loss y_l1fftgrad should be faster and gives better results than --loss lbp and --loss lbp5 for PSNR optimized model.

Can I train waifu2x.swin_unet_2x with noise-level 1?

Yes. For noise-level, I think it is better to use the option value that you feel is the best result from the above website.

Which loss should I use?

The pre-training model was trained with --loss lbp5(--loss lbp for 2x) for PSNR optimized model and --loss l1lpips --discriminator l3v1c for GAN model (see train_photo_psnr.sh and train_photo_gan.sh ). However, the recently added --loss y_l1fftgrad should be faster and gives better results than --loss lbp and --loss lbp5 for PSNR optimized model.

I'm a bit confused about the size parameter. What's the difference between the size in create_training_data.py and size in train.py?

--size option in train.py is the image(patch/tile) size to input for the model. For swin_unet arch, --size 64 is recommended. (112 is also possible, but very slow.)
--size option in create_training_data.py is the image size for the dataset.
During training, input images are generated with random cropping from the dataset images.
For 4x training it must be higher than --size 320.
scale_factor = 4
required_margin = 16 * scale_factor
size_in_train = 64
upscale_target_size = size_in_train * scale_factor # 256
required_dataset_image_size = upscale_target_size + required_margin # 320

For 2x training it must be higher than --size 160.
If you are using fast SSD disk, 640 is recommended.
However, 320 is also fine (it speeds up the image loading).

--size option in train.py is the image(patch/tile) size to input for the model. For swin_unet arch, --size 64 is recommended. (112 is also possible, but very slow.)
--size option in create_training_data.py is the image size for the dataset. During training, input images are generated with random cropping from the dataset images. For 4x training it must be higher than --size 320.
scale_factor = 4
required_margin = 16 * scale_factor
size_in_train = 64
upscale_target_size = size_in_train * scale_factor # 256
required_dataset_image_size = upscale_target_size + required_margin # 320

For 2x training it must be higher than --size 160. If you are using fast SSD disk, 640 is recommended. However, 320 is also fine (it speeds up the image loading).

Just to be sure. If --num-samples is smaller than the number of images in the train folder, will the program only feed the first num-samples images to the model? Is the image order shuffled on each epoch?

Train data loader uses random sampler. Each input image is randomly selected from all training images.
For eval images, --num-samples option is not used and all images are used in one epoch.

It is difficult to give advice for GAN model training.
I myself don't feel that is working very well.
Here is a small working example. --data-dir should be changed for your env.
DEBUG=1 python train.py waifu2x --arch waifu2x.swin_unet_2x --data-dir ./data/photo --model-dir ./models/gan_2x_test --method noise_scale --noise-level 3 --style photo --loss l1lpips --discriminator l3v1c --size 64 --batch-size 16 --hard-example none --update-criterion all --learning-rate 0.00003 --scheduler step --max-epoch 4 --checkpoint-file ./waifu2x/pretrained_models/swin_unet/art/noise3_scale2x.pth

If you are using Windows, you need to set environment variables with set DEBUG=1 command.
Also, there was a bug with a recent change I made, I fixed it https://github.com/nagadomi/nunif/commit/9b21a91ebe232807dbf1a476d309d15bae01cb4d. If you get an error, update the code.
The above command takes only a few minutes because --max-epoch 4 is specified.
You can check the upscaling results of the trained model with the following command.
python -m waifu2x.cli -m noise_scale --noise-level 3 --model-dir ./models/gan_2x_test -i tmp/test_images -o ./tmp/gan_2x_out/ --yes

-i tmp/test_images is the directory containing the test images.
Basics

GAN training must start with a pretrained model. In the above example, art psnr model is used, but it is better to use photo psnr model. (but i have not released it)
It is normal that PSNR goes down. However, if it is too low, some artifact may be occurring.

When DEBUG=1 environment variable is set, the following line will be printed.
[   DEBUG] recon: 0.9735, gen: 0.2725, disc: 0.8867, weight: 0.765913, disc skip: 0.033
[   DEBUG] recon: 0.8631, gen: 0.2788, disc: 0.8994, weight: 0.755511, disc skip: 0.001
[   DEBUG] recon: 0.8291, gen: 0.1819, disc: 0.9214, weight: 0.848656, disc skip: 0.0
[   DEBUG] recon: 0.8302, gen: 0.4509, disc: 0.8535, weight: 0.652829, disc skip: 0.116
...

recon: reconstruction loss
gen: generator loss
disc: discriminator loss
weight: discriminator weight

When training is stable,

disc wave around 0.5-0.9

Failing cases

disc keeps decreasing. If disc is less than 0.2, GAN training is failing.
weight keeps decreasing. If weight is less than 1e-5, the trained model will be high PSNR model

That is also a difficult question to answer.
Typically, perceptual quality metric such as FID and LIPIS are used for GAN model evaluation.
However, we currently use L1+LPIPS loss, so if minimizing LPIPS or PSNR is the goal, Discriminator is not needed.
We expect GAN training will give better results than directly minimizing L1LPIPS.
So personally, I think it should not be used as stop criteria.
I did not define stop criteria, I defined a training schedule.

Enough iteration number to train the Discriminator
Learning rate schedule

If you want to use some criteria to model selection, I would recommend a metric other than LPIPS.
Also you may be concerned about overfitting, but waifu2x models are small and it is basically underfitting. So I am not concerned about overfitting for waifu2x models.


сannot create new vm/ct

running "Proxmox Full update" not create new vm/ct
in log i have
ноя 03 11:24:15 netbox gunicorn[272299]: Bad Request: /api/virtualization/virtual-machines/
ноя 03 11:24:29 netbox gunicorn[272299]: Bad Request: /api/virtualization/virtual-machines/
ноя 03 11:24:48 netbox gunicorn[272299]: Bad Request: /api/virtualization/virtual-machines/
ноя 03 11:24:49 netbox gunicorn[272299]: Bad Request: /api/virtualization/virtual-machines/
ноя 03 11:24:52 netbox gunicorn[272299]: Bad Request: /api/virtualization/virtual-machines/
ноя 03 11:24:53 netbox gunicorn[272299]: Bad Request: /api/virtualization/virtual-machines/
ноя 03 11:24:55 netbox gunicorn[272299]: Bad Request: /api/virtualization/virtual-machines/
ноя 03 11:25:11 netbox gunicorn[272298]: [Proxbox - Netbox plugin | Update All]
ноя 03 11:25:11 netbox gunicorn[272298]: CLUSTER...
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] CLUSTER created. -> pve-company
ноя 03 11:25:11 netbox gunicorn[272298]: NODES...
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] NODE already updated. -> pve-04
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] NODE already updated. -> apollo-pve-15
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] NODE already updated. -> apollo-pve-06
.....
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] NODE already updated. -> apollo-pve-12
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] NODE already updated. -> apollo-pve-03
ноя 03 11:25:11 netbox gunicorn[272298]: VIRTUAL MACHINES...
ноя 03 11:25:11 netbox gunicorn[272298]: UPDATE ALL...
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] VM already updated. -> zabbix.company
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] VM already updated. -> mssql.company
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] VM already updated. -> win2012srv
....
ноя 03 11:25:11 netbox gunicorn[272298]: [OK] VM does not exist on Netbox -> testvm-03
ноя 03 11:25:11 netbox gunicorn[272298]: [proxbox.create.virtual_machine] Creation of VM/CT failed.
ноя 03 11:25:11 netbox gunicorn[272298]: [ERROR] VM not created. -> testvm-03
......
ноя 03 11:25:16 netbox gunicorn[272298]: [OK] VM already updated. -> ns02.company
ноя 03 11:25:16 netbox gunicorn[272298]: REMOVE UNUSED DATA...
ноя 03 11:25:16 netbox gunicorn[272298]: [OK] VM exists on both systems (Netbox and Proxmox) -> 1c-lic.company
ноя 03 11:25:16 netbox gunicorn[272298]: [OK] VM exists on both systems (Netbox and Proxmox) -> 1c.lic.company

netbox version 3.3.1

Hi, could you please try testing it again, if possible? There was several new updates to code since then. Thank you already!

No interaction since Issue creation, probably fixed already. Closing it.


Initial prototype with documentation

This issue signifies the initial prototype work for the Transport CO2 library, as part of the initial work on the project roadmap. The deliverables so far have been:

initial research for CO2 models
initial research for competing or similar libraries
prototype library
initial documentation
first round of user feedback

We will continue building the project roadmap based on user feedback.

Closing this, as completed by @MarkusShepherd


Client connects but connection drops after few seconds

Dear community,
I have just set up a OpenVPN server on an Ubuntu 16.04.3 LTS VPS using the install script here:
https://github.com/Nyr/openvpn-install
Now, I set up 5 clients and the clients are all able to connect successfully to the server.
But, for all the clients first the connection is established and all connection is routed through VPN (i can also traceroute <IP_ADDRESS>), but after some seconds (connections stays established), the client is not able to reach any host (traceroute <IP_ADDRESS> gives no result).
Here you can find my configuration
port 1194
proto udp
dev tun
sndbuf 393216
rcvbuf 393216
push "sndbuf 393216"
push "rcvbuf 393216"
ca ca.crt
cert server.crt
key server.key
dh dh.pem
auth SHA512
tls-auth ta.key 0
topology subnet
server <IP_ADDRESS> <IP_ADDRESS>
ifconfig-pool-persist ipp.txt
push "redirect-gateway def1 bypass-dhcp"
push "dhcp-option DNS <IP_ADDRESS>"
push "dhcp-option DNS <IP_ADDRESS>"
keepalive 10 120
cipher AES-256-CBC
comp-lzo
user nobody
group nogroup
persist-key
persist-tun
status openvpn-status.log
verb 3
crl-verify crl.pem

One exemplary client.ovpn
client
dev tun
proto udp
sndbuf 0
rcvbuf 0
remote SERVER-ADDRESS 1194
resolv-retry infinite
nobind
persist-key
persist-tun
remote-cert-tls server
auth SHA512
cipher AES-256-CBC
comp-lzo
setenv opt block-outside-dns
key-direction 1
verb 3

traceroute directly after connection is established:
traceroute to <IP_ADDRESS> (<IP_ADDRESS>), 64 hops max, 52 byte packets
 1  <IP_ADDRESS> (<IP_ADDRESS>)  32.836 ms  32.139 ms  32.258 ms
 2  XX.XX.XX.XX (XX.XX.XX.XX)  31.958 ms  32.125 ms  32.249 ms
 3  XX.XX.XX.XX (XX.XX.XX.XX)  32.111 ms  35.302 ms  31.870 ms
 4  XX.XX.XX.XX (XX.XX.XX.XX)  32.062 ms  32.382 ms  32.076 ms
 5  google.mix-it.net (<IP_ADDRESS>)  32.668 ms  121.038 ms  88.067 ms
 6  * * *
 7  <IP_ADDRESS> (<IP_ADDRESS>)  32.903 ms
    <IP_ADDRESS> (<IP_ADDRESS>)  101.067 ms
    <IP_ADDRESS> (<IP_ADDRESS>)  37.394 ms

traceroute after some time (around 30 sec):
traceroute to <IP_ADDRESS> (<IP_ADDRESS>), 64 hops max, 52 byte packets
 1  * * *
 2  * * *
 3  * * *
 4  * * *
 5  * * *
 6  * * *
 7  * * *

Also no host can be resolved.
Any advice how to resolve this? Is a really strange behavior.

Hey, I would try changing the sndbuf and rcvbuf and see if that makes any difference.  Also, are you using the same client file for all 5 devices or you generated 5 different client configs?
sndbuf 393216
rcvbuf 393216
push "sndbuf 393216"
push "rcvbuf 393216"

to
sndbuf 0
rcvbuf 0

Its 5 client configs, the sndbuf and rcvbuf was to 0 before and it didnt work too.

As @zenithteq has noticed, either you aren't using my script but a fork of it, or you have manually modified the configuration.
Whoever wrote those configuration lines doesn't understand what he's doing and I can't provide support for third party experiments. This is not a problem with the script, so if you need help with configuration I suggest to try in the OpenVPN forums instead.


Fix osx wheel

Checklist

[ ] Added a CHANGELOG.rst entry

@xhochy, I saw that you were recently involved with some PR on the llvm-openmp feedstock. Do you have any idea why the 18.1.6 release seems to have bumped the minimum target version to 10.13?

Recap: we use conda-forge to install some build dependencies (including llvm-openmp). Conda-forge recently bumped the deployment target to 10.13, and the latest version of llvm-openmp was built using macos 10.13.
I think the best solution is to use 10.13 as the deployment version. This will avoid more issues with other dependencies.


Add release.yaml to ignore bot authors

x-ref to: https://github.com/mne-tools/mne-python/pull/12635

Thanks @mscheltienne !


Refactor frontend main page and badge-example code

The goal of this PR is:

Consume the new service-definition format. (#2397)
Make the frontend more readable.

Behavior changes:

I changed the Image field in the markup modal to show only the path.
I added another click-to-select field below that shows the complete URL.
This made it easier to suppress the live badge preview while it contains placeholders like :user or :gem, a minor tweak discussed at https://github.com/badges/shields/issues/2427#issuecomment-442972100.
The search box now searches all categories, regardless of the current page. (This is an improvement, I would say.)

I did not deliberately address performance, though I ripped out a bunch of anonymous functions and avoided re-filtering all the examples by category on every render, which I expect will not hurt. I haven't really tested this on a mobile connection and it'd be worth doing that.
It would be great to have some tests of the components, though getting started with that seemed like a big project and I did not want to make this any larger than it already is.

It's a medium-sized refactor:

Replace BadgeExamples, Category and Badge component with a completely rewritten BadgeExamples component which renders a table of badges, and CategoryHeading and CategoryHeadings components.
Refactor ExamplesPage and SearchResults components into a new Main component.
Rewrite the data flow for MarkupModal. Rather than rely on unmounting and remounting the component to copy the badge URL into state, employ the getDerivedStateFromProps lifecycle method.
Remove prepareExamples and all-badge-examples.
Rewrite the $suggest schema to harmonize with the service definition format. It's not backward-compatible which means at deploy time there probably will be 10–20 minutes of downtime on that feature, between the first server deploy and the final gh-pages deploy.  🤷♂ (We could leave the old version in place if it seems worth it.)
Added two new functions in make-badge-url with tests. I removed most of the uses of the old functions, but there are some in parts of the frontend I didn't touch like the static and dynamic badge generators, and again I didn't want to make this any larger than it already is.
Fix a couple bugs in the service-definition export.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

:book:

Thanks for contributing to our documentation. We :heart: our documentarians!

  Generated by :no_entry_sign: dangerJS

This pull request introduces 1 alert when merging debfa4667d846c26b47653952aee7ffba739fd53 into 0a6afb447877e9b9d973146be65cb1b23fe7e588 - view on LGTM.com
new alerts:

1 for Unused or undefined state property

Comment posted by LGTM.com

This pull request introduces 2 alerts when merging b475529238a804ea04845ecad45507c69226cd0e into 0a6afb447877e9b9d973146be65cb1b23fe7e588 - view on LGTM.com
new alerts:

1 for Useless assignment to local variable
1 for Unused or undefined state property

Comment posted by LGTM.com

AFAICT the LGTM warning is a false positive.

This pull request introduces 2 alerts when merging 44095635f667cf6c23b5114085132e900de69a3b into 8a8311d931b9949ae9fdff72c96abb60019825da - view on LGTM.com
new alerts:

1 for Useless assignment to local variable
1 for Unused or undefined state property

Comment posted by LGTM.com

This pull request introduces 1 alert when merging 74741f70621a53614b5db927869c064ada8d7ff0 into 8a8311d931b9949ae9fdff72c96abb60019825da - view on LGTM.com
new alerts:

1 for Unused or undefined state property

Comment posted by LGTM.com


Question about RowDataPacket

Hi,
try out this:
connection.connect();

connection.query('select *  from user', function (error, results, fields) {
  if (error) throw error;
  let data = results[0]
  console.log(data)
});

and got this:
RowDataPacket {
  Host: '%',
  User: 'root',
  Password: '*',
  Select_priv: 'Y',
  Insert_priv: 'Y',
  Update_priv: 'Y
}

just curious what data type of results[0]? and RowDataPacket seems not a json property, and yet I can access it like this:
console.log(data.Host)

without referencing to RowDataPacket, can anybody enlighten me on this? Thanks

Hi @sqllyw this is just how Node.js formats objects when using console.log. If you want to see it as the standard JavaScript object that it is, you can use console.log(JSON.stringify(results, null, 2)).

Hi @sqllyw this is just how Node.js formats objects when using console.log. If you want to see it as the standard JavaScript object that it is, you can use console.log(JSON.stringify(results, null, 2)).

wow, so far the fastest response I ever got in github, thanks!

wow, so far the fastest response I ever got in github, thanks!


Fix dates

I am fixing tests, but I would say that it is ready

@kprzerwa @zzacharo let's go through together on Monday!


Blocking main thread can cause low thread utilization

When the main thread takes longer than most worker threads, it leads to low thread utilization, which is unfortunate.
import malebolgia

import std/[os, atomics]

var
    threadpool = createMaster()
    busyThreads: Atomic[int]
busyThreads.store 0

let mainThreadId = getThreadId()

proc findStartPositionsAndPlay() =
    atomicInc busyThreads
    echo "Started. Busy threads: ", busyThreads.load

    sleep(1000)
    if getThreadId() == mainThreadId:
        sleep(10_000)

    atomicDec busyThreads
    echo "Finished. Busy threads: ", busyThreads.load

threadpool.awaitAll:
    for i in 1..10000:
        threadpool.spawn findStartPositionsAndPlay()

Now works well with activeProducer=true:

import malebolgia

import std/[os, atomics]

var
    master = createMaster(activeProducer=true)
    busyThreads: Atomic[int]
busyThreads.store 0

let mainThreadId = getThreadId()

proc findStartPositionsAndPlay() =
    atomicInc busyThreads
    echo "Started. Busy threads: ", busyThreads.load

    sleep(1000)
    if getThreadId() == mainThreadId:
        sleep(10_000)

    atomicDec busyThreads
    echo "Finished. Busy threads: ", busyThreads.load

master.awaitAll:
    for i in 1..100:
        master.spawn findStartPositionsAndPlay()


Update to<EMAIL_ADDRESS>

There's an issue with the current version of Jest running on latest version of Node.
https://github.com/facebook/jest/issues/8069
<EMAIL_ADDRESS>fixes the issue.

Pull Request Test Coverage Report for Build 156

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 146:
0.0%

Covered Lines:
84

Relevant Lines:
84

💛  - Coveralls


report: Background (part of Introduction) [6h]

This section has been merged with the introduction. Closing.


Pytest fixtures that yield should show the yielded result

I have a test using a fixture like so:

The app parameter is from a fixture that yields a Flask object. That's what the test will get during the test running (assuming there's a single yield).
If there was more than one yield, it would be a union of the possible types yielded.
Instead of showing Generator[Flask..., this should just show the Flask type.

This issue has been fixed in prerelease version 2023.2.41, which we've just released. You can find the changelog here: CHANGELOG.md


Errors in rancher logs when delete cluster

Version: v2.2.4-rc3 (HA Setup)
What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Have an HA setup with many clusters. In my case one of every type.
Once they are ready, delete them all at once. Observe that it takes 4-10 mins to delete them and the many errors in logs. In the UI the Removing state will change to error-type (red in UI)
Optionally delete slowly one by one and you will still see errors in logs. But Removing state does change to error-type (red in UI)

Result:

Clusters can take longer to delete than normal if deleting many at once
Errors in log repeat many times (this repeats in some cases such as when delete many clusters -- i.e. the 4-10 minute scenario)
[rancher-754467b5b8-vm89j] 2019/05/16 20:34:05 [ERROR] problem retrieving user for CRTB map[projectId:c-sz5st:p-vch79 transitioningMessage:waiting on cluster-prtb-sync_c-sz5st uuid:e627cc53-7809-11e9-a602-02af34995612 userId:u-pcx42yvjt2 type:/v3/schemas/projectRoleTemplateBinding annotations:map[lifecycle.cattle.io/create.cluster-prtb-sync_c-sz5st:true lifecycle.cattle.io/create.mgmt-auth-prtb-controller:true] roleTemplateId:project-member namespaceId:p-vch79 transitioning:yes name:u-pcx42yvjt2-member .selfLink:/apis/management.cattle.io/v3/namespaces/p-vch79/projectroletemplatebindings/u-pcx42yvjt2-member userPrincipalId:local://u-pcx42yvjt2 labels:map[cattle.io/creator:norman] state:removing id:p-vch79:u-pcx42yvjt2-member removed:2019-05-16T20:34:05Z created:2019-05-16T18:39:12Z] from cache during CRTB transformation: user.management.cattle.io "u-pcx42yvjt2" not found
If delete many clusters at once UI shows red "Removing" state.

I tested this again. This time keeping an eye on the time. Adding this as may help to add context to the issue.
t M:SS
t 0:00
Delete all the clusters.

t 9:00
All clusters are still in the normal "Removing state" and the error is still "spamming" logs.

t 9:45
We start to see some clusters show error-state in UI for "Removing" (they changed to red)

t 10:00
More clusters in the "red" error state. One got deleted.
t 12:30
Some clusters got deleted eks is still in error state for removal in UI

t 13:00
All clusters are finally deleted.

I also observe this issue in a single-node rancher install on latest master. I at least observed that it took much longer to delete the clusters than normal. I'm pretty sure if I have many clusters and try to delete all at once in my single-node master rancher I too would see the exact issue.

These issues were introduced in 804b65d681dbb0abe9eaf8881ba3e42f850a9b9b.
We now iterate over a large number of objects to check for dangling finalizers.
The error mentioned is a not found error, which is a result of the cluster role template binding not being found since the cluster is being deleted.

Version: Master (v2.3) (5/29/19)
I am no longer seeing the issue. A very small handful of [ERROR] entries are seen in the logs but this has been significantly improved. Error entires no longer loop and removal of clusters is significantly faster now. I only had to wait up to 30 seconds to remove many clusters at once (delete all at once).
I consider the issue resolved.


feat(store): add evga 3090 to newegg canada

Description
Adds EVGA 3090 xc3 ultra, xc3, ftw3 ultra and ftw3 to newegg.ca
Fixes #388  kind of, there are currently no evga 3090 cards listed on bestbuy.ca
Testing
Ran snatcher only looking for evga 3090 products and verified all links work.

had wrong issue linked, fixed now

Looks good to me 👍


ENH: Format sensor data after it's been extracted

We have a script that extracts all of the data from the csvs, but now we need to transform it.
Expected Behavior
The data frame is transformed from wide to long, with modality, unit, and value columns replacing the current sensor value columns:
cols_to_melt = ['CO2__ppm', 'CO__ppm', 'Humidity__pct', 'PN1Plus__counts_per_L',
'Pressure__mbar', 'Temperature__deg_C', 'humidity_2_rel_true_pct_avg', 'light_lux',
'room_type', 'sound_dba_avg', 'temperature_2_celsius_avg', 'vibration_ms2_avg']

A modality is something like CO2 whereas the unit is something like pct or counts_per_L.
The final data frame should look something like this:

Datetime
Building
Floor
Room
Room Type
Modality
Unit
Value

2017-02-24 00:00:00
1800 f
2
2240
None
CO
ppm
0.827

2017-02-24 00:00:00
1800 f
2
2240
None
CO
ppm
0.827

Current Behavior
Data is long: 😢

New functions added to reformat data as requested. Please review and let me know if any adjustments are needed.

@kstreepy
I made a few edits, mostly so that we're not relying on global variables (like final_df) and so that the unnamed column is dropped before the reshape.


Regex fix and swtiched to UTC date methods

There was a regex bug on master that would only capture the year and month of the filename instead of the date as well.
I also switched all the getters on the date object to use UTC, this way if someone is not in same day as the Prime Meridian it will still render the correct date that was entered in the filename.

This works perfectly!


DEP: typing.get_args new in 3.8, require python minimal version is 3.8

require typing.get_args

It seems could get by .__args__, see #206 e65796c5fe87bc7c23d05f0f0ccf615b6f74fa7a


Provide means to pass kwargs to policy in PPO2

Hello,
I'm not sure why this extra PR is needed (because related to #83 ).
Also I see some issues here:

only PPO2 is supported
adding kwargs mask what is really pass to the algorithm, I would rather use partial , or define a custom policy as in https://stable-baselines.readthedocs.io/en/master/guide/custom_policy.html

My main concern with passing kwargs as you did it is that if the use pass wrong kwargs, it will still work... (e.g. passing learnning_rate instead of learning_rate)

Hello,
I'm not sure why this extra PR is needed (because related to #83 ).
Also I see some issues here:
* only PPO2 is supported

* adding kwargs mask what is really pass to the algorithm, I would rather use `partial` , or define a custom policy as in https://stable-baselines.readthedocs.io/en/master/guide/custom_policy.html

My main concern with passing kwargs as you did it is that if the use pass wrong kwargs, it will still work... (e.g. passing learnning_rate instead of learning_rate)

I was inspired by what is done in baselines where a **network_kargs is used. It is useful to control the hyper-parameters. The problem with CustomPolicies (correct me if I am wrong) is that since the object is instantiated from inside the model, we cannot control the network hyper-parameters and I will need a CostumPolicy for each difference in hyper-parameters.
Regarding the support only for PPO2, I think it should not be difficult to extend to other algorithms.

The problem with CustomPolicies (correct me if I am wrong) is that since the object is instantiated from inside the model

I think you can have better way of changing the kwargs using python partial  (see https://docs.python.org/3.6/library/functools.html#functools.partial)
How it is used for instance with DQN: https://github.com/hill-a/stable-baselines/blob/a02ead2244bce0c3527a7ddad8349b03b2433317/stable_baselines/deepq/experiments/run_atari.py#L30


Warning iOS9, XCode 7.1.1

Mattress/Source/URLProtocol.swift:198:27: 'init(request:delegate:)' was deprecated in iOS 9.0: Use NSURLSession (see NSURLSession.h)
Could you remove this warning please?

Solved by PR ^^


Error when run 'python -m designer'

This occurs if the width is less than 589 in config.ini
CLI info
user@computer:~/kivy_projects/kivy-designer$ python -m designer
[INFO   ] [Logger      ] Record log in /home/user/.kivy/logs/kivy_18-02-02_16.txt
[INFO   ] [Factory     ] 194 symbols loaded
[INFO   ] [Image       ] Providers: img_tex, img_dds, img_sdl2, img_pil, img_gif (img_ffpyplayer ignored)
[INFO   ] [Text        ] Provider: sdl2
[INFO   ] [Kivy        ] v1.10.0
[INFO   ] [Python      ] v2.7.12 (default, Dec  4 2017, 14:50:18)
[GCC 5.4.0 20160609]
[INFO   ] [OSC         ] using  for socket
[INFO   ] [Window      ] Provider: sdl2(['window_egl_rpi'] ignored)
[INFO   ] [GL          ] Using the "OpenGL" graphics system
[INFO   ] [GL          ] Backend used
[INFO   ] [GL          ] OpenGL version <3.0 Mesa 12.0.6>
[INFO   ] [GL          ] OpenGL vendor
[INFO   ] [GL          ] OpenGL renderer <Mesa DRI Intel(R) Sandybridge Mobile >
[INFO   ] [GL          ] OpenGL parsed version: 3, 0
[INFO   ] [GL          ] Shading version <1.30>
[INFO   ] [GL          ] Texture max size <8192>
[INFO   ] [GL          ] Texture max units <16>
[INFO   ] [Window      ] auto add sdl2 input provider
[INFO   ] [Window      ] virtual keyboard not allowed, single mode, not docked
[WARNING] [IOError     ] [Errno 2] No such file or directory: '/home/danilo/.kivy/garden/garden.xpopup/xpopup.mo'. Switch to the defaults.
[INFO   ] [Clipboard   ] Provider: gtk3(['clipboard_dbusklipper'] ignored)
xclip version 0.12
Copyright (C) 2001-2008 Kim Saunders et al.
Distributed under the terms of the GNU GPL
[INFO   ] [Cutbuffer   ] Provider: xclip
[INFO   ] [CutBuffer   ] cut buffer support enabled
[INFO   ] [Base        ] Start application main loop
[INFO   ] [GL          ] NPOT texture support is available
[WARNING] [Accordion   ] not enough space for displaying all children
[WARNING] [Accordion   ] need 220px, got 100px
[WARNING] [Accordion   ] layout aborted.
[WARNING] [Accordion   ] not enough space for displaying all children
[WARNING] [Accordion   ] need 176px, got 100px
[WARNING] [Accordion   ] layout aborted.
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[WARNING] [Accordion   ] not enough space for displaying all children
[WARNING] [Accordion   ] need 220px, got 100px
[WARNING] [Accordion   ] layout aborted.
[INFO   ] [Base        ] Leaving application in progress...
[INFO   ] [Base        ] Start application main loop
[WARNING] [Accordion   ] not enough space for displaying all children
[WARNING] [Accordion   ] need 176px, got 100px
[WARNING] [Accordion   ] layout aborted.
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[CRITICAL] [Clock       ] Warning, too much iteration done before the next frame. Check your code, or increase the Clock.max_iteration attribute
[INFO   ] [Base        ] Leaving application in progress...
Exception KeyError: (2030,) in <functools.partial object at 0x7f3d82b29e68> ignored
Exception KeyError: (2214,) in <functools.partial object at 0x7f3d82af3890> ignored
Exception KeyError: (2256,) in <functools.partial object at 0x7f3d82ac12b8> ignored
Exception KeyError: (2298,) in <functools.partial object at 0x7f3d82642260> ignored
Exception KeyError: (3029,) in <functools.partial object at 0x7f3d824182b8> ignored
Exception KeyError: (3215,) in <functools.partial object at 0x7f3d822d5578> ignored
Exception KeyError: (3401,) in <functools.partial object at 0x7f3d82190838> ignored
Environment Info
kivy: 1.10.0
pygments: 2.2.0
docutils: 0.14
watchdog: 0.8.3
jedi: 0.11.1
gitpython: 2.1.8
six: 1.11.0
kivy-garden: 0.1.4
Platform: Linux-4.8.0-36-generic-x86_64-with-Ubuntu-16.04-xenial
Python: 2.7.12
Traceback
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/kivy/core/window/window_sdl2.py", line 663, in mainloop
    self._mainloop()
  File "/usr/lib/python2.7/dist-packages/kivy/core/window/window_sdl2.py", line 405, in _mainloop
    EventLoop.idle()
  File "/usr/lib/python2.7/dist-packages/kivy/base.py", line 348, in idle
    Clock.tick_draw()
  File "/usr/lib/python2.7/dist-packages/kivy/clock.py", line 588, in tick_draw
    self._process_events_before_frame()
  File "kivy/_clock.pyx", line 405, in kivy._clock.CyClockBase._process_events_before_frame (kivy/_clock.c:8093)
  File "kivy/_clock.pyx", line 445, in kivy._clock.CyClockBase._process_events_before_frame (kivy/_clock.c:7965)
  File "kivy/_clock.pyx", line 443, in kivy._clock.CyClockBase._process_events_before_frame (kivy/_clock.c:7887)
  File "kivy/_clock.pyx", line 167, in kivy._clock.ClockEvent.tick (kivy/_clock.c:3211)
  File "/usr/lib/python2.7/dist-packages/kivy/uix/boxlayout.py", line 309, in do_layout
    c.size = (w, h)
  File "kivy/properties.pyx", line 478, in kivy.properties.Property.__set__ (kivy/properties.c:5171)
  File "kivy/properties.pyx", line 1342, in kivy.properties.ReferenceListProperty.set (kivy/properties.c:20421)
  File "kivy/properties.pyx", line 494, in kivy.properties.Property.set (kivy/properties.c:6054)
  File "kivy/properties.pyx", line 516, in kivy.properties.Property.set (kivy/properties.c:5933)
  File "kivy/properties.pyx", line 571, in kivy.properties.Property.dispatch (kivy/properties.c:6614)
  File "kivy/_event.pyx", line 1225, in kivy._event.EventObservers.dispatch (kivy/_event.c:13524)
  File "kivy/_event.pyx", line 1131, in kivy._event.EventObservers._dispatch (kivy/_event.c:12723)
  File "/usr/lib/python2.7/dist-packages/kivy/uix/actionbar.py", line 644, in on_width
    self._layout_group()
  File "/usr/lib/python2.7/dist-packages/kivy/uix/actionbar.py", line 538, in _layout_group
    self._clear_all()
  File "/usr/lib/python2.7/dist-packages/kivy/uix/actionbar.py", line 501, in _clear_all
    self.clear_widgets()
  File "/usr/lib/python2.7/dist-packages/kivy/uix/widget.py", line 605, in clear_widgets
    remove_widget(child)
  File "/usr/lib/python2.7/dist-packages/kivy/uix/actionbar.py", line 494, in remove_widget
    self._list_action_items.remove(item)
ValueError: list.remove(x): x not in list

End of Traceback

Duplicate of #318


tests: arch: Add a testcase to verify the timing of x86

Adding a testcase to verify arch interface by calling
timing API to get CPU freq and verify it can transfer
cycle/counter to ns.
Signed-off-by: NingX Zhao<EMAIL_ADDRESS>

Hi @Zhaoningx , I checked this test case pass in ACRN ehl_crb, could you please also check if this passes on up_squared and ehl_crb boards, thanks!

Hi @Zhaoningx , I checked this test case pass in ACRN ehl_crb, could you please also check if this passes on up_squared and ehl_crb boards, thanks!

Hi @enjiamai , I tested it on up_squared board, it works. But the ehl_crb board is hold by others, so I'm still not testing for it. Thanks.

The new additions are not NMI related. Please do not put those here.

Hi @dcpleung , thanks for your comment, I have moved the changes to another directioy.

The new additions are not NMI related. Please do not put those here.

Hi @dcpleung ,  how do you think if we changes the testsuite and directory name instead of nmi? For example, arch_funcs or something like that, then we can put other arch relevant cases here besides nmi? This means we can put more testcases into this testsuite, thanks!

Hi @dcpleung, you suggested me to remove the testcase of testing timing APIs, because timing APIs are tested in benchmark dirctory, and I double check it, it works well, but I have an open for this, the testcase of latency_measure can not be used to generate code coverage. the error log as below:
timing init subsys
arch_timing_x86_init
START - Time Measurement
Timing results: Clock frequency: 999 MHz
Average thread context switch using yield                   :   54428 cycles ,    54428 ns
 Error: tick occurred
Switch from ISR back to interrupted thread                  :   26944 cycles ,    26944 ns
Time from ISR to executing a different thread               :   23616 cycles ,    23616 ns
Time to create a thread (without start)                     :   16512 cycles ,    16512 ns
Time to start a thread                                      :   62720 cycles ,    62720 ns
Time to suspend a thread                                    :   60224 cycles ,    60224 ns
Time to resume a thread                                     :   61632 cycles ,    61632 ns
Time to abort a thread (not running)                        :   10272 cycles ,    10272 ns
Average semaphore signal time                               :   21282 cycles ,    21282 ns
Average semaphore test time                                 :    7874 cycles ,     7874 ns
Semaphore take time (context switch)                        :   63040 cycles ,    63040 ns
Semaphore give time (context switch)                        :   98368 cycles ,    98368 ns
Average time to lock a mutex                                :    8802 cycles ,     8802 ns
Average time to unlock a mutex                              :   29009 cycles ,    29010 ns
Average time for heap malloc                                :  116128 cycles ,   116128 ns
Average time for heap free                                  :  117728 cycles ,   117728 ns
===================================================================
PROJECT EXECUTION FAILED

the command is twister -p qemu_x86 -T tests/benchmarks/latency_measure/ -C
is it normal? And need I to report a bug? @dcpleung  do you have any comments here? Thanks.

latency_measure is not failing on my side with timing functions turned on. Here are what I have:
qemu_x86:
SeaBIOS (version zephyr-v1.0.0-0-g31d4e0e-dirty-20200714_234759-fv-az50-zephyr)
Booting from ROM..*** Booting Zephyr OS build v2.7.99-2330-g903c78c207fc  ***
START - Time Measurement
Timing results: Clock frequency: 999 MHz
Average thread context switch using yield                   :   11558 cycles ,    11558 ns
Average context switch time between threads (coop)          :   21053 cycles ,    21053 ns
Switch from ISR back to interrupted thread                  :    4928 cycles ,     4928 ns
Time from ISR to executing a different thread               :    3008 cycles ,     3008 ns
Time to create a thread (without start)                     :    4224 cycles ,     4224 ns
Time to start a thread                                      :   10784 cycles ,    10784 ns
Time to suspend a thread                                    :   10400 cycles ,    10400 ns
Time to resume a thread                                     :   10688 cycles ,    10688 ns
Time to abort a thread (not running)                        :    1536 cycles ,     1536 ns
Average semaphore signal time                               :    3424 cycles ,     3424 ns
Average semaphore test time                                 :    1344 cycles ,     1344 ns
Semaphore take time (context switch)                        :   12736 cycles ,    12736 ns
Semaphore give time (context switch)                        :   17568 cycles ,    17568 ns
Average time to lock a mutex                                :    1632 cycles ,     1632 ns
Average time to unlock a mutex                              :    4738 cycles ,     4738 ns
Average time for heap malloc                                :   19136 cycles ,    19136 ns
Average time for heap free                                  :   15968 cycles ,    15968 ns
===================================================================
PROJECT EXECUTION SUCCESSFUL

qemu_x86_64:
SeaBIOS (version zephyr-v1.0.0-0-g31d4e0e-dirty-20200714_234759-fv-az50-zephyr)
Booting from ROM..*** Booting Zephyr OS build v2.7.99-2330-g903c78c207fc  ***
START - Time Measurement
Timing results: Clock frequency: 999 MHz
Average thread context switch using yield                   :    9060 cycles ,     9060 ns
Average context switch time between threads (coop)          :    9503 cycles ,     9503 ns
Switch from ISR back to interrupted thread                  :   14240 cycles ,    14240 ns
Time from ISR to executing a different thread               :    9664 cycles ,     9664 ns
Time to create a thread (without start)                     :    3968 cycles ,     3968 ns
Time to start a thread                                      :   12064 cycles ,    12064 ns
Time to suspend a thread                                    :   12640 cycles ,    12640 ns
Time to resume a thread                                     :   12096 cycles ,    12096 ns
Time to abort a thread (not running)                        :    2208 cycles ,     2208 ns
Average semaphore signal time                               :    8960 cycles ,     8960 ns
Average semaphore test time                                 :    2048 cycles ,     2048 ns
Semaphore take time (context switch)                        :   13472 cycles ,    13472 ns
Semaphore give time (context switch)                        :   18400 cycles ,    18400 ns
Average time to lock a mutex                                :    3072 cycles ,     3072 ns
Average time to unlock a mutex                              :    9283 cycles ,     9283 ns
Average time for heap malloc                                :   13056 cycles ,    13056 ns
Average time for heap free                                  :    7872 cycles ,     7872 ns
===================================================================
PROJECT EXECUTION SUCCESSFUL

latency_measure is not failing on my side with timing functions turned on. Here are what I have:
qemu_x86:
SeaBIOS (version zephyr-v1.0.0-0-g31d4e0e-dirty-20200714_234759-fv-az50-zephyr)
Booting from ROM..*** Booting Zephyr OS build v2.7.99-2330-g903c78c207fc  ***
START - Time Measurement
Timing results: Clock frequency: 999 MHz
Average thread context switch using yield                   :   11558 cycles ,    11558 ns
Average context switch time between threads (coop)          :   21053 cycles ,    21053 ns
Switch from ISR back to interrupted thread                  :    4928 cycles ,     4928 ns
Time from ISR to executing a different thread               :    3008 cycles ,     3008 ns
Time to create a thread (without start)                     :    4224 cycles ,     4224 ns
Time to start a thread                                      :   10784 cycles ,    10784 ns
Time to suspend a thread                                    :   10400 cycles ,    10400 ns
Time to resume a thread                                     :   10688 cycles ,    10688 ns
Time to abort a thread (not running)                        :    1536 cycles ,     1536 ns
Average semaphore signal time                               :    3424 cycles ,     3424 ns
Average semaphore test time                                 :    1344 cycles ,     1344 ns
Semaphore take time (context switch)                        :   12736 cycles ,    12736 ns
Semaphore give time (context switch)                        :   17568 cycles ,    17568 ns
Average time to lock a mutex                                :    1632 cycles ,     1632 ns
Average time to unlock a mutex                              :    4738 cycles ,     4738 ns
Average time for heap malloc                                :   19136 cycles ,    19136 ns
Average time for heap free                                  :   15968 cycles ,    15968 ns
===================================================================
PROJECT EXECUTION SUCCESSFUL

qemu_x86_64:
SeaBIOS (version zephyr-v1.0.0-0-g31d4e0e-dirty-20200714_234759-fv-az50-zephyr)
Booting from ROM..*** Booting Zephyr OS build v2.7.99-2330-g903c78c207fc  ***
START - Time Measurement
Timing results: Clock frequency: 999 MHz
Average thread context switch using yield                   :    9060 cycles ,     9060 ns
Average context switch time between threads (coop)          :    9503 cycles ,     9503 ns
Switch from ISR back to interrupted thread                  :   14240 cycles ,    14240 ns
Time from ISR to executing a different thread               :    9664 cycles ,     9664 ns
Time to create a thread (without start)                     :    3968 cycles ,     3968 ns
Time to start a thread                                      :   12064 cycles ,    12064 ns
Time to suspend a thread                                    :   12640 cycles ,    12640 ns
Time to resume a thread                                     :   12096 cycles ,    12096 ns
Time to abort a thread (not running)                        :    2208 cycles ,     2208 ns
Average semaphore signal time                               :    8960 cycles ,     8960 ns
Average semaphore test time                                 :    2048 cycles ,     2048 ns
Semaphore take time (context switch)                        :   13472 cycles ,    13472 ns
Semaphore give time (context switch)                        :   18400 cycles ,    18400 ns
Average time to lock a mutex                                :    3072 cycles ,     3072 ns
Average time to unlock a mutex                              :    9283 cycles ,     9283 ns
Average time for heap malloc                                :   13056 cycles ,    13056 ns
Average time for heap free                                  :    7872 cycles ,     7872 ns
===================================================================
PROJECT EXECUTION SUCCESSFUL

Hi @dcpleung, I think you didn't add the option --coverage, because I find that the case will be failed, only when I try to add --coverage to generate a coverage report. And @enjiamai has helped to fix this, see https://github.com/zephyrproject-rtos/zephyr/pull/41139 , so closed this PR. Thanks.


Ensure content is comprehensive on media page

Grant and I should do a cross-check with the media page to be sure we haven't missed anything.  Grant has a ton of stuff on linktree, for example, and perhaps the youtube video between the Quakers and Coyotes might fit well in the video section (in addition to a direct link to the townball in the quad at hillsdale video done by the Hilssdale Daily News)

Requested on Discord.

Relates to #27 and #28

Feature this link somewhere, probably at top of page (short Townball overview)
instagram.com/p/CYdDraSJrPk/

After reviewing the mentioned video (suggested by Jones) I think it's too long to be a featured video on the media page, because that page (as it currently is) is very busy already and the video isn't very short.


Added KMP and Z Algorithm

Solved issue #446

Please review the code and update me if something needs to be changed.

Please review the code and update me if something needs to be changed.

I have added sample input/ouput in the code. Please review the changes

I have added sample input/ouput in the code. Please review the changes


@:Access  fails with subdirectory

@:Access  fails with subdirectory, works fine if they are in the same directory.
platform: Win7
hxCpp; 3.2.94
haxe: 3.2.0
error message:
Test.hx:18: characters 14-39 : Cannot access private field rainbowPencilColors
directory structure required to reproduce error.
testFolder2
          Test2.hx
compile.hxml
Test.hx

contents of classes
package;
import testFolder2.Test2;

class Test {

    static function main() {
        new Test();
    }
    public function new(){
        indirect();
    }

    inline function indirect(){
        testAccess();
    }
     @:access(Test2.rainbowPencilColors)
    function testAccess(){
        trace(Test2.rainbowPencilColors().length);
    }
}

package testFolder2;

class Test2
{
     static inline function rainbowPencilColors(){ return [ 0xD2D0C1
, 0xCD8028
, 0xD29D11
, 0xE37128
, 0xF06771
, 0xD23931
, 0xAF2C31
, 0x90333E
, 0x863D50
, 0x584A5D
, 0x549EC3
, 0x2C709D
, 0x457AAE
, 0x364D6D
, 0x378C6D
, 0x6EA748
, 0x365DA4
, 0x456E42
, 0xC1882E
, 0x813424
, 0x402E24
, 0x292420
, 0x525751
, 0x1B1B19
]; }
}

-cpp cpp
-main Test

	Don't you mean @:access(testFolder2.Test2.rainbowPencilColors)?
Ok that works fine, but I think the import should mean that you don't have to define the full qualified path?
Sorry about that I guess you can close I will try to rem to do this.


[AutoPR notificationhubs/resource-manager] chore: jsonfmt notificationhubs

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/6188

This PR has been merged into https://github.com/Azure/azure-sdk-for-js/pull/3751


Version 3.21.0 LIVE updates

Since few hours the 3.21.0 has been released in LIVE, you can find changed lines in the PR #101 that contains 1,777 lines and 179 new that has been added to the english global.ini
As of a few hours ago, version 3.21.0 has been released live.
You can find the list of changes in the PR #101 (thanks to @Auhrus ), which contains a total of 1,956 lines, including 179 new lines added to the english global.ini file.
From now on, new PR will be targeting the 3.21.0 live.
Update status:

[x] English
[ ] French - France
[x] German - Germany
[ ] Portuguese - Brazil
[ ] Spanish - Spain

German #121

Spanish #120

Italian #135


Document arp cache fix.

The arp cache issue described in the docs has a fix in bosh now--this patch updates the docs to explain how to avoid the issue.

 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.


Add API for making HTTP requests

Description
This is needed to fix CORS in WebDAV sync in the Android app
Issues Resolved
https://github.com/johannesjo/super-productivity/issues/1847
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable.

Thank you very much for this! I made some comments.

Thank you! This great (as I already mentioned in the other PR :))!
Since you clearly seem to know much more about android dev than I do may I ask you question? I have a hard time figuring out a problem. When I am not using the app for a longer time it seems like the task list widget and the status bar widget both get detached from the main app instance. I have no clue what could cause this. Do you maybe have an idea, what the problem might be?

Since you clearly seem to know much more about android dev than I do may I ask you question? I have a hard time figuring out a problem. When I am not using the app for a longer time it seems like the task list widget and the status bar widget both get detached from the main app instance. I have no clue what could cause this. Do you maybe have an idea, what the problem might be?
Probably it's linked to the wrong context or Android is killing the app. Running a foreground service forever is not really sustainable, so my advice would be to remove this feature or leave it unreliable. To fix notifications for scheduled tasks, AlarmManager can be used (I haven't used this API for a while so it might have been deprecated, maybe WorkManager these days). Regarding the task list widget, it's certainly because you're connecting the background widget to the foreground activity. Android manages these two lifecycles separately so the activity can be destroyed leaving only the widget. I haven't looked at the relevant code but this is the generic answer.

@penn5 thank you so much! This is very helpful (and would have cost me a lot of time to figure out)!
Would you by any chance be interested in working on making the app better in that department? I would love to see the whole android part of the application work better as it currently does. So much so that I would put some of my own money into this getting developed by someone who actually knows what he is doing.

It's a nice offer. I'd love to take it up in the future, but for now I'm quite busy. Perhaps in July?

It's a nice offer. I'd love to take it up in the future, but for now I'm quite busy. Perhaps in July?

Sounds good to me! :) You can contact me via email at<EMAIL_ADDRESS>(if I don't respond, which might be because the email went into the spam folder, please just drop me line somewhere on github).

Just tested this with nextcloud. Unfortunately I receive the following error:
Error: Invalid response: -2 SSL Handshake Error

Does your nextcloud use a self-signed cert? Is it trusted on your phone?

My test instance uses a standard letsencrypt certificate. Not sure if it is trusted on my phone, but I don't see why it shouldn't be.

Can you send me the URL that you're using? I don't need the username/password.

I'll send it to you via mail.


Add container ID attribute for Resource constants

Is your feature request related to a problem? Please describe.
I am currently implementing AWS ECS resource detector, following the Java implementation Java ECS Reource Detector
Since Java implementation parsed certain file and assign the parsed value to ResourceConstants.CONTAINER_ID, when I implement this part, I notice that there is no container ID attribute in JS resource constant.
Java Resource Constant: https://github.com/open-telemetry/opentelemetry-java/blob/f28396b812046bb69daf9899d777e140ff8de0e9/sdk/src/main/java/io/opentelemetry/sdk/resources/ResourceConstants.java#L54
JavaScript resource Constant: https://github.com/open-telemetry/opentelemetry-js/blob/master/packages/opentelemetry-resources/src/constants.ts#L34
Describe the solution you'd like
Should I add a container ID attribute for JS repo to make it consistent?
Describe alternatives you've considered
Is there other consideration about this issue like this is a JavaScript specific constant?
Or there is other JS attribute corresponding to container ID in Java version?

Should I add a container ID attribute for JS repo to make it consistent?

Yes

Of course!

Seems like this is fixed. @dyladan


Adds support for float

https://github.com/AeneasVerif/charon/issues/332
https://github.com/AeneasVerif/charon/issues/321

Please increment the version number in Cargo.toml, do make test, and commit all the changed files.


Relationship and usage with dbt?

I see dbt is listed in the topics of this repo. I was wondering how you can use soda together with dbt and how it relates to dbt? I might be missing this in the docs, but haven't found it so far.

Soda SQL is complimentary to dbt. There is only slight overlap in terms of functionality with dbt tests, but dbt testing is fairly limited.
As an example, in an ELT pipeline, you will use Soda SQL after extraction phase to test raw data before you trigger dbt. So Soda SQL here  helps to prevent feeding bad data to dbt that builds your analytics model. You can use soda sql tests that fail when there are data quality related issues, so the T(ransform) step will not run unless all tests pass.
After dbt builds your analytics model, you use Soda SQL to capture metrics - think of all the calculations that your analysts want, and you can store these metrics to calculate change over time, or even add more tests here.
Apart from using Open source Soda SQL, you can also register for free Soda Cloud account, which will act as your metrics repository and gives your organisation a nice visual overview of all the metrics and tests, where people can configure monitors. Further to this with Soda Cloud account, you can also empower your analysts for self-service data monitors without data engineers adding code.
I hope this clarifies things!

@janet-can  Can you see if my comment can be "refactored" and added to FAQs in the docs?

@sdebruyn If you have more questions, feel free to open a https://github.com/sodadata/soda-sql/discussions topic or ping us on https://join.slack.com/t/soda-community/shared_invite/zt-m77gajo1-nXJF7JtbbRht2zwaiLb9pg :)


tsconfig.base.json does not exist error when using ng update on a new project

Description and Replication
Run ng new testing to create an Angular 8.2.0 project
Run ng update @angular/core @angular/cli to update
Output and error:
testing git:(master) ng update @angular/core @angular/cli
Using package manager: 'npm'
Collecting installed dependencies...
Found 30 dependencies.
Fetching dependency metadata from registry...
Updating package.json with dependency @angular/cli @ "10.0.0" (was "8.2.2")...
Updating package.json with dependency @angular/core @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency @angular-devkit/build-angular @ "0.1000.0" (was "0.802.2")...
Updating package.json with dependency @angular/language-service @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency @angular/animations @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency @angular/compiler-cli @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency @angular/common @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency rxjs @ "6.5.5" (was "6.4.0")...
Updating package.json with dependency zone.js @ "0.10.3" (was "0.9.1")...
Updating package.json with dependency @angular/forms @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency @angular/platform-browser-dynamic @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency @angular/router @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency @angular/compiler @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency @angular/platform-browser @ "10.0.0" (was "8.2.14")...
Updating package.json with dependency typescript @ "3.9.5" (was "3.5.3")...
UPDATE package.json (1293 bytes)
npm WARN deprecated<EMAIL_ADDRESS>fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.

<EMAIL_ADDRESS>postinstall /Users/bracicot/dev/testing/node_modules/core-js
node -e "try{require('./postinstall')}catch(e){}"

<EMAIL_ADDRESS>postinstall /Users/bracicot/dev/testing/node_modules/@angular/cli
node ./bin/postinstall/script.js

npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":"<8.10.0"} (current: {"node":"12.16.0","npm":"6.14.5"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS><10.0.0 || >9.0.0-beta <10.0.0 || >9.1.0-beta <10.0.0 || >9.2.0-beta <10.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS><10.0.0 || >9.0.0-beta <10.0.0 || >9.1.0-beta <10.0.0 || >9.2.0-beta <10.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
added 417 packages from 179 contributors, removed 54 packages, updated 134 packages, moved 6 packages and audited 1473 packages in 24.239s
found 2 low severity vulnerabilities
run npm audit fix to fix them, or npm audit for details
** Executing migrations for package '@angular/core' **
** Executing migrations for package '@angular/cli' **
ERROR! tsconfig.base.json does not exist..
The Schematic workflow failed. See above.

Hi @BenRacicot,
I tried to reproduce this locally on new project with Angular CLI version  8. However, I wasn't able.
ng update @angular/core @angular/cli
Your global Angular CLI version (10.0.0-rc.0) is greater than your local
version (8.3.28). The local Angular CLI version is used.

To disable this warning use "ng config -g cli.warnings.versionMismatch false".
The installed Angular CLI version is older than the latest stable version.
Installing a temporary version to perform the update.
Installing packages for tooling via npm.
Installed packages for tooling via npm.
Using package manager: 'npm'
Collecting installed dependencies...
Found 30 dependencies.
Fetching dependency metadata from registry...
    Updating package.json with dependency @angular/cli @ "10.0.0" (was "8.3.28")...
    Updating package.json with dependency @angular/core @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency @angular-devkit/build-angular @ "0.1000.0" (was "0.803.28")...
    Updating package.json with dependency @angular/compiler @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency @angular/language-service @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency @angular/animations @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency rxjs @ "6.5.5" (was "6.4.0")...
    Updating package.json with dependency @angular/router @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency @angular/forms @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency @angular/platform-browser @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency @angular/platform-browser-dynamic @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency zone.js @ "0.10.3" (was "0.9.1")...
    Updating package.json with dependency @angular/common @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency @angular/compiler-cli @ "10.0.0" (was "8.2.14")...
    Updating package.json with dependency typescript @ "3.9.5" (was "3.5.3")...
UPDATE package.json (1295 bytes)
✔ Packages installed successfully.
** Executing migrations of package '@angular/cli' **

❯ Angular Workspace migration.
  Update an Angular CLI workspace to version 9.
UPDATE angular.json (3589 bytes)
UPDATE tsconfig.app.json (272 bytes)
UPDATE package.json (1297 bytes)
✔ Packages installed successfully.
  Migration completed.

❯ Lazy loading syntax migration.
  Update lazy loading syntax to use dynamic imports.
  Migration completed.

❯ Replace deprecated 'styleext' and 'spec' Angular schematic options.
  Migration completed.

❯ Update Browserslist configuration file name to '.browserslistrc' from deprecated 'browserslist'.
RENAME browserslist => .browserslistrc
  Migration completed.

❯ Update tslint to version 6 and adjust rules to maintain existing behavior.
UPDATE package.json (1296 bytes)
UPDATE tslint.json (3348 bytes)
  Migration completed.

❯ Remove deprecated 'es5BrowserSupport' browser builder option.
  The inclusion for ES5 polyfills will be determined from the browsers listed in the browserslist configuration.
  Migration completed.

❯ Replace deprecated and removed 'styleext' and 'spec' Angular schematic options with 'style' and 'skipTests', respectively.
  Migration completed.

❯ Remove deprecated options from 'angular.json' that are no longer present in v10.
  Migration completed.

❯ Add "Solution Style" TypeScript configuration file support.
  This improves developer experience using editors powered by TypeScript’s language server.
  Read more about this here: https://v10.angular.io/guide/migration-solution-style-tsconfig
RENAME tsconfig.json => tsconfig.base.json
CREATE tsconfig.json (427 bytes)
UPDATE tsconfig.app.json (277 bytes)
UPDATE tsconfig.spec.json (275 bytes)
UPDATE e2e/tsconfig.json (219 bytes)
  Migration completed.

❯ Add the tslint deprecation rule to tslint JSON configuration files.
  Migration completed.

❯ Update library projects to use tslib version 2 as a direct dependency.
  Read more about this here: https://v10.angular.io/guide/migration-update-libraries-tslib
  Migration completed.

❯ Update 'module' and 'target' TypeScript compiler options.
  Read more about this here: https://v10.angular.io/guide/migration-update-module-and-target-compiler-options
UPDATE e2e/tsconfig.json (222 bytes)
  Migration completed.

❯ Update workspace dependencies to match a new v10 project.
UPDATE package.json (1296 bytes)
✔ Packages installed successfully.
  Migration completed.

** Executing migrations of package '@angular/core' **

❯ Static flag migration.
  Removes the `static` flag from dynamic queries.
  As of Angular 9, the "static" flag defaults to false and is no longer required for your view and content queries.
  Read more about this here: https://v9.angular.io/guide/migration-dynamic-flag
  Migration completed.

❯ Renderer to Renderer2 migration.
  As of Angular 9, the Renderer class is no longer available.
  Renderer2 should be used instead.
  Read more about this here: https://v9.angular.io/guide/migration-renderer
  Migration completed.

❯ Undecorated classes with DI migration.
  As of Angular 9, it is no longer supported to use Angular DI on a class that does not have an Angular decorator.
  Read more about this here: https://v9.angular.io/guide/migration-undecorated-classes
  Migration completed.

❯ Missing @Injectable and incomplete provider definition migration.
  As of Angular 9, enforcement of @Injectable decorators for DI is a bit stricter and incomplete provider definitions behave differently.
  Read more about this here: https://v9.angular.io/guide/migration-injectable
  Migration completed.

❯ ModuleWithProviders migration.
  As of Angular 10, the ModuleWithProviders type requires a generic.
  This migration adds the generic where it is missing.
  Read more about this here: https://v10.angular.io/guide/migration-module-with-providers
  Migration completed.

❯ Undecorated classes with Angular features migration.
  In version 10, classes that use Angular features and do not have an Angular decorator are no longer supported.
  Read more about this here: https://v10.angular.io/guide/migration-undecorated-classes
  Migration completed.

At which point of the migration you are seeing the ERROR! tsconfig.base.json does not exist.. error? Are you able to provide a reproduction?

When updating from the Angular CLI 8.2 or earlier, it is recommended to first update to the latest version of 8 first (ng update @angular/core@8 @angular/cli@8).
For additional details on the recommended update process, please see https://update.angular.io/#8.2:10.0

Thank you @alan-agius4 and @clydin. I do see now that update.angular.io explains to update one major version at a time.
I believe the difference between your replay(s) above and my own is that my ng new projects version matched my globally installed Angular versions @angular/cli = 8.2.14. From there updating to v10 succeeds but the error is shown at the end of the process.
Just thought I'd report.


Error installing with npm

I tried installing the package with npm:
$ npm install ink-docstrap

but i kept getting this error: (this is v0.5.2)
npm ERR! tar.unpack untar error /home/gocho/.npm/ink-docstrap/0.5.2/package.tgz
npm ERR! Linux 3.16.0-41-generic
npm ERR! argv "/home/gocho/.nvm/versions/node/v0.12.4/bin/node" "/home/gocho/.nvm/versions/node/v0.12.4/bin/npm" "i" "ink-docstrap"
npm ERR! node v0.12.4
npm ERR! npm  v2.10.1
npm ERR! path /home/gocho/node_modules/ink-docstrap/themes/paper/module-documents_probe.This%2520is%2520not%2520actually%2520a%2520class,%2520but%2520an%2520artifact%2520of%2520the%2520documentation%2520system.html
npm ERR! code ENAMETOOLONG
npm ERR! errno -36

npm ERR! ENAMETOOLONG, open '/home/gocho/node_modules/ink-docstrap/themes/paper/module-documents_probe.This%2520is%2520not%2520actually%2520a%2520class,%2520but%2520an%2520artifact%2520of%2520the%2520documentation%2520system.html'
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /home/gocho/npm-debug.log

To solve this I had to install the v0.4.12
⇒ npm i<EMAIL_ADDRESS><EMAIL_ADDRESS>node_modules/ink-docstrap
└──<EMAIL_ADDRESS>
What caused the installing the latest version to error?

I'm having this issue as well, but could not solve it even with the above suggested fix, a project's dependency seems to always go and try to use 0.5.2

@digital-abyss I was actually installing grunt-jsdoc when I encountered this bug. I ended up cloning the grunt-jsdoc repo instead of npm-installing it.

@digital-abyss since I needed to build my packages remotely, I ended up using an older version of grunt-jsdoc:
⇒ npm i<EMAIL_ADDRESS>
So in package.json:
  "devDependencies": {
    "grunt-jsdoc": "0.6.4"
  }

Explicitly use the old version.

See the possible reason behind this error at issue #109

Has there been any progress on this? I'm encountering the same issue, and it seems many others are posting similar issues:
https://github.com/terryweiss/docstrap/issues/103
https://github.com/krampstudio/grunt-jsdoc/issues/116

This is due to the ink-docstrap dependency running docs against this line, which generates the filename causing the ENAMETOOLONG error:
https://github.com/terryweiss/docstrap/blob/master/fixtures/documents/probe.js#L1004
See also:

https://github.com/krampstudio/grunt-jsdoc/issues/142
https://github.com/terryweiss/docstrap/issues/109

should be fixed in this release, please report back if not.


TS-3485: Support ip_allow config for HTTP2

Actually enforce the ip-level ACL checks for HTTP2 and move to the accept logic to make the decision before creating the session object.

Where is the check on the method?  Each individual stream would need to be verified.

The method checks are done in HttpTransact.  The acl_record is stored on the ProxyClientTransaction object for that later check.

Don't you need to check what is being denied at the higher level?

Just pushed another commit that moves the IP-based ip allow checking to the SessionAccept superclass.  Verified for IP-based and method-based policies.  The method-based denies will return an error header and body.  The IP-based denies just closed the connection.
@bryancall I'm not following your question about checking what is denied at a higher level.

Looks like it needs to be clang-formatted there are a few formatting issues.

Yes, planning on running clang-format and squashing before mergin.

Besides the formatting it looks good.  I was meaning from above that you would have to check the allowed methods at the at the session level.  I missed the isEmpty() before.


[BUG] Segfault when profiling a module compiled by Cython on 3.11

Describe the bug
It seems like the profiling function will cause CPython to crash with Cython compiled modules. I have reproduced this issue with spaCy, but according to their devs(https://github.com/explosion/spaCy/issues/12105) it would reproduce on any Cython modules.
This will only happen on Python 3.11, which changed the layout of frames significantly. It was originally found in viztracer(https://github.com/gaogaotiantian/viztracer/issues/307).
Code to reproduce the behaviour:
import sys
def func(*args):
    frame = args[0]
    while frame:
        frame = frame.f_back

sys.setprofile(func)
import spacy

The segfault happens at <frame at 0x7f2c80f1fa30, file 'thinc/backends/numpy_ops.pyx', line 1, code __Pyx_PyMODINIT_FUNC PyInit_numpy_ops(void)>.
Expected behaviour
No segfault.
Environment
OS: [Linux]
Python version [3.11]
Cython version [Not sure what they used to compile the module]
spaCy version [3.4.4]
Additional context
No response

This is probably related to (and might be a duplicate of) https://github.com/cython/cython/issues/4207