Access Freebase KG

Hi. Thanks for the great work!
I have opened various preprocessed data for webQSP, but I still did not understand how to map from freebase mid to entity name string. I wonder if you could provide some instructions on how I should use files like "ent2id.pickle", "subgraph_2hop_triples.npy" and "entity_name.pickle". Especially if I want to find related triplets for each question in the test set, how should I do this?

Hi,
Thanks for your attention. I would update the detailed operations about how to preprocess these data as soon as possible.


Refactor RNGCryptoRandom, clean tests

This PR refactors the logic in RNGCryptoRandom for readability while maintaining performance. Splitting up the logic in NextBytes() should help any future maintainers understand the process a bit better. I also applied the suggested Visual Studio syntax improvements in the unit tests for readability purposes.
I mainly did this as an exercise, but I am open to suggestions for improving this PR!

Thanks for the PR - I won't be merging this one. Reasons:

Changed tabs to spaces. I like my tabs where they are.
Each method has precise reasons for why it is not broken down to smaller methods (ex. I monitor exactly how many IL statements each method has and whether it's inlinable or not). When the logic is already small enough, breaking it down into smaller functions yields limited value while spreading critical-for-correctness code all over the file and making unnecessary additional method calls on a hot path.


set up icon with sidebar, let's add text in svelte

approving new changes let us know if you want help running the extension !


Make (optional) declaration of main() possible in geekest modes

if the code includes void main() then it avoids wrapping it,
which allows you to define helper functions beforehand.

Addresses #19.

@totalgee
Thanks for great PR!
I also saw the GitHub issue.
There are two matters I would like to discuss.
Can I make a further modification to use whitespace-aware regexps to determine if the main function is included?
I would like to add the following note in the readme and elsewhere, if that is OK with you.
The reason why we support the notation that does not omit the definition of the main function is to allow users to define their own functions.

Of course... it's your project! (-; Those sound like good changes; I was trying to keep my changes minimal because I didn't know whether it would be of interest to you or others.

Hi @totalgee.
your PR is merged!
I have made a few changes, such as displaying the line number of the error location in the error message, the contents of the readme, and checking using regular expressions.
Thanks!

Thanks!


Fix Sqs ErrorType not being parsed properly

Customer reporting exception error type is not properly handled after the AwsJson-1.0 migration when calling SQS.
For example, when calling
import boto3
import botocore

sqs = boto3.client('sqs')

try:
    sqs.delete_queue( QueueUrl='foo-nonexistant-queue' )
except sqs.exceptions.QueueDoesNotExist as e:
    print("Successfully caught QueueDoesNotExist")

Message does not printed, but instead exception dump does.
This is because Client is listening to an ErrorCode to throw a modelled factory exception. In the above case, self.client.exceptions.QueueDoesNotExist ErrorType should be thrown. However, as a prerequisite of the AwsJson migration, we also need to throw the correct ErrorCode to ensure a backwards compatibility. In the above case, ErrorCode should be AWS.SimpleQueueService.NonExistentQueue.
This PR should fix this.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (01dabf4) 93.46% compared to head (dec0412) 93.43%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3054      +/-   ##
===========================================
- Coverage    93.46%   93.43%   -0.03%
===========================================
  Files           66       66
  Lines        14007    14007
===========================================
- Hits         13091    13087       -4
- Misses         916      920       +4

Files
Coverage Δ

botocore/client.py
97.68% <100.00%> (+<0.01%)
:arrow_up:

botocore/parsers.py
98.61% <100.00%> (-0.01%)
:arrow_down:

... and 3 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Make Docker Scout policies configurable

Tell us about your request
Docker Scout ships with a number of software supply chain best practice policies"out of the box", but I would like it if I could configure these. For example, there is a policy that looks out for use of third-party software licensed under GPLv3. I would like to be able to configure that policy to include other license types that are not compatible with our software/product/business.
Which service(s) is this request for?
Docker Scout
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
The policies in Docker Scout are great, but in some cases I need to be able to adjust them to suit my needs.
Are you currently working around the issue?
I have to gather the information manually.
Additional context

I think this might be possible now: https://docs.docker.com/scout/policy/configure/


Add Floyd-warshall and utility to read graph from csv

Codecov Report

Merging #4 (04c3b2b) into main (4bba0d6) will increase coverage by 16.80%.
The diff coverage is 85.41%.

@@             Coverage Diff             @@
##             main       #4       +/-   ##
===========================================
+ Coverage   58.19%   75.00%   +16.80%
===========================================
  Files           1        3        +2
  Lines         122      216       +94
===========================================
+ Hits           71      162       +91
- Misses         51       54        +3

Impacted Files
Coverage Δ

src/avus.jl
61.31% <23.52%> (ø)

src/relationship.jl
98.50% <98.50%> (ø)

src/shortestpaths/floyd-warshall.jl
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4bba0d6...04c3b2b. Read the comment docs.


i get 404 error randomly

Environment

Operating System: Windows_NT
Node Version:     v18.17.1
Nuxt Version:     3.7.0
CLI Version:      3.7.3
Nitro Version:    2.6.2
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      telemetry, app, experimental, runtimeConfig, modules, devtools, css, imports, vite, build, image, pwa
Runtime Modules:  vite-plugin-vue-type-imports/nuxt<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
sorry i cannot reproduction. it happens very randomly
Describe the bug
i got 404 error very randomly when fetch data on my nuxt app.
the api path is right and it happens only on dev server

Additional context
No response
Logs
No response

seems like relative to nuxt's server catch all routes not ofetch, so i close it


Bug: SQLite3 NOT NULL prevents inserts

Describe the bug
I'm a SQLite noob so maybe someone can correct me... but I've been using it more in testing since it is so fast with :memory:. However, I just ran into this issue today that a NOT NULL field (the default, unless your migration specifies 'null' => true) will fail on inserts without data specified. Example error:

ErrorException: SQLite3::exec(): NOT NULL constraint failed: db_users.flair

Consider the following migration:
$fields = [
	'name'       => ['type' => 'varchar', 'constraint' => 63],
	'flair'      => ['type' => 'varchar', 'constraint' => 63],
	'created_at' => ['type' => 'datetime', 'null' => true],
	'updated_at' => ['type' => 'datetime', 'null' => true],
];
$this->forge->addField('id');
$this->forge->addField($fields);

$this->forge->createTable('users');

Very standard. Then my UserModel can do this:
$model->insert(['name'  => 'Zaboomafoo']);

In MySQL this will create a row with "" for flair since it is a VARCHAR with no input specified. In SQLite3 I get the above error. Best I can tell (see e.g. https://github.com/cakephp/phinx/issues/265) when a SQLite3 column is NOT NULL it needs to have a default value specified to prevent this.
Anyone?
CodeIgniter 4 version
develop
Affected module(s)
Database - Forge - SQLite3
Context

OS: Ubuntu
Web server: Apache
PHP version: 7.3

Seems like a proper behavior to me. If MySQL is running in the strict mode (and I'm quite sure it's the default setting where STRICT_TRANS_TABLES is present in sql_mode) you will get the same result (error).
Personally I would advise using strict mode whenever it's possible.

Maybe I've been trained poorly with MySQL but I only add NULL to VARCHAR fields if I need an additional state. I.e. if NULL and "" actually mean something different. When creating a MySQL field it defaults to NOT NULL unless otherwise specified, so adding a column without a specified field will set it to "" out the box, per se.
Regardless, it seems that having "standard" migrations + seeds fail on different drivers should not happen.

Of course NULL and "" (empty string) means something different but if you set the database field as NOT NULL (without DEFAULT) and at the same time you don't specify the value for that field during INSERT -  the error is something I would always expect to happen.
I believe that this is the default behavior for all databases: SQLite3, PostgreSQL, and MySQL. The problem with this last database is a bit of a CodeIgniter fault. We don't use the strict mode (default for this DB) when we establish the connection. This behavior was ported from CI3. I think that some time ago it had sense since from what I remember STRICT_TRANS_TABLES was not always the default setting - so I guess it was something introduced for compatibility reasons.
Differences between drivers in this case purely depend on a mode we want to use during connection to MySQL (strictOn). When we will use strict mode in our connection settings for MySQL, it will behave the same as other databases.

I see, that makes sense. I will do a little more testing before I make up my mind about whether this behavior merits changing.

When it comes to changes - personally I would go with a strict mode on for MySQL, but that would probably result in hiring a high-class hitman by the community to assassinate us 😅 Idk maybe this kind of change would fit the v4.1?

I imagine it would be a major breaking change and would need to be for version 5. Do we have a way to create a strict connection with the MySQL driver currently? Like an option on the Config? If not definitely we should add that. Alternatively, if it constitutes deeper changes than just a connection flag perhaps we could make a new driver that extends the current one?

Switching the variable will do the trick - it's already built-in: https://github.com/codeigniter4/CodeIgniter4/blob/develop/app/Config/Database.php#L49


The compiler is unable to type-check this expression in reasonable time

I am receiving this issue on xcode 12:
The compiler is unable to type-check this expression in reasonable time; try breaking up the expression into distinct sub-expressions
This error happens in Ease.swift line 179. Do you think you can separate c*sin(t/d * (Double.pi/2))+b into
let halfPi = Double.pi/2
let sined = sin(t/d * halfPi)
return c * sined + b

I will create a PR instead. Sorry not very familiar with the flows


[ISSUE-202] Add name of person(s) who fixed the build.

Looking good, many thanks for your time and help! 😃

Thanks!


Avoid GraphicsDevice.CreateOpenGL from using IGLContext in its parameters.

IGLContext is from Silk.NET.Core which is a third-party library. As GraphicsDevice.CreateOpenGL is a public-facing API, it wouldn't be good to leave that exposed. I would propose instead that we pass delegates and let us manually create and manage GL contexts as needed.

I have been meaning to look into this, although the implementation would be similar, just with a Pie implementation instead of relying on Silk.NET's implementation.

This has been implemented in 6a0d71a.


Remove raw spreadsheet rows and support multiple indexes

This PR changes 4 things:

removes raw spreadsheet rows (csv now loads into rye native rows)
add support for multiple indexes in a spreadsheet
adds spreadsheet columns? function
moves transforming raw values to rye values to utils (needed because of a cyclic import in spreadsheet.go otherwise)

Great, thank you!!


FetchPickAndPlace-v1

Hello!TianhongDai. Thank you very much for your code to help me! But I can't run the results given in the paper in environment 'FetchPickAndPlace-v1'.  I'm very happy to help you solve them.

oh thank you! I want to ask how I can reproduce the paper results! Why is the curve I get by running 'mpirun -np 16 python -u train.py --env-name='FetchPickAndPlace-v1' 2>&1 | tee pick.log' in 'redeme' higher than that in the paper?And it's very different！

[ Results of my run]
[ paper's]

thanks your reply

@quyouyuan Hi, this is because, the network's input in HER paper is different from my code / code from openai baselines.
In HER paper, page 7, the first paragraph
Observations: In this paragraph relative means relative to the current gripper position. The policy is given as input the absolute position of the gripper, the relative position of the object and the target4, as well as the distance between the fingers. The Q-function is additionally given the linear velocity of the gripper and fingers as well as relative linear and angular velocity of the object. We decided to restrict the input to the policy in order to make deployment on the physical robot easier.

We can find that - relative linear and angular velocity of the object are only given in the Q-function, which form an asymmetric actor-critic structure. This is because they said: "We decided to restrict the input to the policy in order to make deployment on the physical robot easier.", and access / estimation of relative linear and angular velocity of the object is difficult in the real world.
However, in most of other HER extensions, or even in openai code: https://github.com/openai/baselines/blob/master/baselines/her/actor_critic.py#L31-L39 , relative linear and angular velocity of the object are provided to both policy network and Q-function.
I think you should better refer to this paper: https://arxiv.org/pdf/1802.09464.pdf, if you don't need to deploy HER on the physical robot.

Thank you for your answer! I feel very helpful to me

May I ask how did you obtain the plot? I have a custom environment, but I'm using this pytorch implementation of HER to train my model

May I ask how did you obtain the plot? I have a custom environment, but I'm using this pytorch implementation of HER to train my model

@ollintzinlab Hi - you can refer to this script: https://github.com/TianhongDai/esil-hindsight/blob/main/plot_curves.py.

Thank you 😀


Cannot navigate to exact error line while run Python file in terminal

VSCode Version: 1.19.0
OS Version: ubuntu 16.04

Steps to Reproduce:

Run Python file in terminal (internal terminal)
Error happen, I cannot navigate to error line using ctrl + click, it only jump to error file, not exact error line

This issue was reported at vscode-python repo: https://github.com/Microsoft/vscode-python/issues/450
but one person told me report this issue here
Does anyone know about this issue?

Reproduces without extensions: No

Please provide the exact format of the link, ideally with a screenshot as well

Following is the sequence that I create this issue
For instance, I created an error at line 26

Then I move the cursor to another line, for instance, line 57

Then I run Python file by right click on this file and choose Run Python File in Terminal as below

Then error happen at line 26 as below

Then I try to navigate to error location by ctrl+click to error message in terminal, but the cursor is still at line 57, it does not move to line 26, which is error line that I have created earlier


[bug 1123744] Nix jenkins

Removes the jenkins script--we're not using it anymore.
Quick r?

burn it with :fire: r+

Landed in: 1e3bb5c [bug 1123744] Nix jenkins
Thank you!


remove bk7231s usb related file.

感谢提交 Pull Requests！
Thanks for submitting a pull request.

auto build fail,number:98

auto build fail,number:98

auto build fail,number:98

Security Check : 85777e15d2e8a53df1e0cee7a4ef411576396e88


Training SSD with a custom dataset

Hello,
I have configured Caffe-SSD and download a 300 * 300 model which contains prototxt files and a caffemodel. I use DIGITS 5. it is actually a GUI for the Caffe.
I want to train this model (fine-tune) it with my own images (dataset) but I have some challenges:

Creating LMDB files for training/validation images and labels
Annotating images

I assume annotation should comply with COCO format maybe, which a tool that can do this annotation task is unknown for me. besides I do not know how to create those 4 LMDB files after that.
Could you help and and give some tips to do these process?

You mean how to create LMDB for custom dataset? To be honest, I'm not familiar with creating LMDB for ssd either, especially not from PASCAL-VOC formats. You best bet is to organize your own data just like pascal-voc(xml labels), and process with the code provided in caffe-ssd

This question is in-relative to this project so I will close this issue, you can send me email if hopefully I can help with. However, some issues are deeply rooted in caffe implementations, which I am trying to avoid, that's the reason I reimplemented it again.

May I ask your Email sir?

cheungchih at gmail

Hi,
I'm trying to train SSD on my own data by DIGITS but I got the error that AnnotatedData layer is not supported in DIGITS. Did you manage to train SSD by DIGITS?


Fix line number offset by 1

This PR addresses these open issues:

https://github.com/microsoft/sarif-vscode-extension/issues/397
https://github.com/microsoft/sarif-vscode-extension/issues/373

The existing code assumes that the line number passed from a sarif result location is zero-indexed. However, the line numbers are already 1-indexed. See https://docs.oasis-open.org/sarif/sarif/v2.1.0/os/sarif-v2.1.0-os.html#_Toc34317690 which describes startLine, for example, as a positive integer.
See also https://docs.oasis-open.org/sarif/sarif/v2.1.0/os/sarif-v2.1.0-os.html#_Toc34317409, which indicates that line numbers are 1-based.

@lin-ll Nope. Thanks!


Form validation error on email field

When validating the email address I kept running into errors when a new user wants to sign up.
I see this errors, because I use this print statement:
print(form.errors.as_json())
When I leave email empty I get this error (which is expected):
{"email": [{"message": "This field is required.", "code": "required"}]}
When I enter a invalid email address I get this error (which is expected):
{"email": [{"message": "Enter a valid email address.", "code": "invalid"}]}
When I enter a valid email address I get this error (which is NOT expected):
{"email": [{"message": "This field cannot be blank.", "code": "blank"}]}
To fix this issue I edited the AbstractCUser class in cuser.models.py
I added blank=True, like this:
email = models.EmailField(
    _('email address'),
    blank=True,
    unique=True,
    error_messages={
        'unique': _("A user with that email address already exists."),
    },
)

Maybe I made a mistake somewhere else, not sure.
But this was my solution.

I haven't run into this. What does your form class look like?

I have struggled with exactly the same issue as above.  Same error message about the field being blank when it is not.

@67millwood What does your form class look like?

My apologies for not replying quickly.
I'll include the used form at the bottom of this comment.
My current solution is to use the standard Django User and just use an email address as username (which Django accepts) and leave email blank.
Make sure to follow these steps so you can still add new fields to the User model:
https://docs.djangoproject.com/en/2.2/topics/auth/customizing/#using-a-custom-user-model-when-starting-a-project
Not sure if django-username-email is still needed now an email address is accept as a username. But that's probably another discussion.
import django
from django import forms
from django.contrib.auth import (authenticate, get_user_model,
password_validation)
from django.contrib.auth.forms import ReadOnlyPasswordHashField
from django.utils.translation import ugettext_lazy as _
from cuser.models import CUser
UserModel = get_user_model()
class UserCreationForm(forms.ModelForm):
"""
A form that creates a user, with no privileges, from the given email and
password.
"""
error_messages = {
'password_mismatch': _("The two password fields didn't match."),
}
name = forms.CharField(
    label=_("Voornaam"),
    max_length=30,

)

email = forms.EmailField(
    label=_("Email"),
    max_length=254,
    widget=forms.EmailInput(),
)

phone = forms.CharField(
    label=_("06-nummer"),
    max_length=20,
)

woonplaats = forms.CharField(
    label=_("Woonplaats"),
    max_length=20,
)
#
# password1 = forms.CharField(
#     label=_("Password"),
#     strip=False,
#     widget=forms.PasswordInput,
#     help_text=password_validation.password_validators_help_text_html(),
# )
# password2 = forms.CharField(
#     label=_("Password confirmation"),
#     widget=forms.PasswordInput,
#     strip=False,
#     help_text=_("Enter the same password as before, for verification."),
# )

class Meta:
    model = CUser
    fields = []

def clean_password2(self):
    password1 = self.cleaned_data.get("password1")
    password2 = self.cleaned_data.get("password2")
    if password1 and password2 and password1 != password2:
        raise forms.ValidationError(
            self.error_messages['password_mismatch'],
            code='password_mismatch',
        )
    return password2

def _post_clean(self):
    super()._post_clean()
    # Validate the password after self.instance is updated with form data
    # by super().
    password = self.cleaned_data.get('password2')
    if password:
        try:
            password_validation.validate_password(password, self.instance)
        except forms.ValidationError as error:
            self.add_error('password2', error)

def save(self, commit=True):
    print('start save usesr')
    user = super().save(commit=False)
    user.set_password(self.cleaned_data["password1"])
    if commit:
        print('save the user')
        user.save()
    return user

did not create a custom User.  simply imported UserCreationForm from cuser.forms.  no modifications. creation of users via the Admin works perfectly (email field...no username as expected).

error messages post filling in a normal email address and matching pwds...

@cwazuidema @67millwood Thank you, I was able to reproduce and fix the bug. Version 2.2.4 has been released.

works like a charm.  great response on that one.  appreciate it.


Refine setup command

I changed some things in the setup method.

Load/Unload kernel modules only when necessary
Moved all setup operations to separate functions
Changed the setup to always run in the same order (pool -> kernel modules -> network)
Fixed a bug for detecting if bridge0 is already set up (removed a whitespace from the grep statement)

Looking good, thanks for the work!


Add mount points for es pages

Follow up of #4989
+CC @chalin

@chalin Added — thanks!

Follow up (FYI):

#5007


Typing issue in Tour of Heroes example for chapter 5

In chapter 5 "Services" of the Tour of Heroes tutorial,  you can find this example:
getHeroesSlowly(): Promise<Hero[]> {
  return new Promise<Hero[]>(resolve =>
    setTimeout(resolve, 2000)) // delay 2 seconds
    .then(() => this.getHeroes());
}

Not being familiar with Promises and TypeScript, this got me confused. How come the typing is done in the constructor with the <Hero[]>  type argument? The first Promise constructed actually does not return this array, it is only the then statement that is indeed typed. So I believe the following syntax would make more sense:
getHeroesSlowly(): Promise<Hero[]> {
  return new Promise(resolve =>
    setTimeout(resolve, 2000)) // delay 2 seconds
    .then<Hero[]>(() => this.getHeroes());
}

Sorry, but I come from java where this could never get past the compiler !

@rdettai Your intuition is correct. The generic type passed to Promise.constructor is actually misleading. It is used to create a promise of the desired type so that the type is propagated through .then. Your approach is clearer as to intent, and is more idiomatic.
Personally, I would prefer to leverage type inference all the way throughout the method to avoid misleading and potentially type assertions.
export interface Hero {
  name: string;
  powers: string[];
}
export class Service {

  getHeroesSlowly() {
    return Promise
      .resolve(() => setTimeout(() => { }, 2000))
      .then(() => this.getHeroes());
  }

  getHeroes() : Hero[] {
    return [{
      name: 'Wolverine',
      powers: [
        'Adamantium Claws',
        'Mutant Healing'
      }
    ];
  }
}

I beg to differ. The type-parameter passed to the Promise constructor is perfectly correct in all languages that support generics properly. It should work in Java too.
That said, the example code is horrible (which I am allowed to say because I wrote it :)).
In a PR #2991, I change it to the following:
  getHeroesSlowly(): Promise<Hero[]> {
    return new Promise(resolve => {
      // Simulate server latency with 2 second delay
      setTimeout(() => resolve(this.getHeroes()), 2000);
    });
  }

@aluanhaddad I like type inference too. Within the docs, our policy is to state public method return types explicitly on the method signature so that you can tell at a glance while reading the code what the method returns.

@wardbell that's fair since the IDE isn't there to serve up the signature in the example.
However, it leads people into the practice of placing these annotations. With generics this can be problematic because TypeScripts generics are unsafely covariant. So a Promise<void> will happily convert to a Promise<any> and mislead he caller into thinking the get a result back.

@aluanhaddad true (until TS 2.1 I think). I would hope that I didn't make that mistake. I also hope that anyone who sees explicit typing of  Promise<any> or Promise<void> realizes that the returned value is uninteresting; what matters is that it resolves or errors.
Structural typing is not as safe as nominal typing ... by design. That's why we say that TypeScript provides "static typing" not "strict typing".  It's still JavaScript after all.  On balance, I'm happy with the compromises.

@wardbell The difference between structural and nominal typing is completely orthogonal to the notion of type safety, strictness, and staticness. It certainly has nothing to do with the issue in question.


Using Phalcon\Cache\Backend\Redis got segmentation fault with latest Phalcon 2.1.x build and PHP7

Hi,
When I try to use Redis cache with latest Phalcon 2.1.x branch and PHP7 I got segmentation fault. When I use master branch and php 5.6 it works perfectly.
php index.php
Segmentation fault

If I run under debugger:
gdb php
run index.php

I got this:
Program received signal SIGSEGV, Segmentation fault.
0x00007fffe6ce0fd7 in zephir_call_user_function.3436 () from /usr/lib64/php/modules/phalcon

this is something that should be filed as a phalcon bug report as long
as there isn't any concrete connection to wrong behavior on zephir's part.

I have posted in phalcon before but someone informed me that is a zephir bug and I should post here.

generally that makes sense, but not as long as it isn't clearly linked to what's causing the failure.

Ok, I will try to investigate more this issue and come back with detailed infomations. Thank you!

I thought its zephir problem beacause last call is zephir_call_user_function

Means that I can post back this problem to phalcon bug report?

It likely is a zephir problem, but the circumstances leading to the crash aren't clear
enough for anyone to analyze why it might happen, so the issue as it is/was
is just incomplete and nobody would be able to figure out what to do to reproduce the crash and deduce why it's crashing.

just link this issue to the issue in phalcon, @andresgutierrez should get around to fixing it when he gets to work on completing PHP7 support.

@steffengy he just have to post whole backtrace using bt and some short script to reproduce the problem but the same code works on php 5, so i guess it's another zephir related problem with php 7 @AniaSoft

I have posted the issue in phalcon bug report. I also have posted there the whole backtrace using bt.

@Jurigag generally yes, but anyhow it makes sense to post phalcon specific issues with phalcon, because the people there generally have phalcon and zephir installed and are able to track it down.
I for example only seldomly grab phalcon to figure out why a bug is happening,
that's also why reduced examples of bugs are generally best, because they do not
include any dependency to another project.

He posted full stack and it's problem with di. Can your post some example code so I can this on debug version? @AniaSoft

Seg fault occurs when this code is executed:
use Phalcon\Cache\Frontend\Data;
use Phalcon\Cache\Backend\Redis as RedisCache;

$di->set('cache', function () {
    $frontCache = new Data(array(
        "lifetime" => 86400
    ));
    return new RedisCache($frontCache, array(
        'host' => '<IP_ADDRESS>',
        'port' => 6379
    ));
});

Also I'm not sure why we should post it on phalcon. Last call it's zephir function, I could possibly create some code in zephir which will produce same stack without using any phalcon classes, it can't be fix in phalcon until using reflection classes etc but that will be workaround and it's need to be fixed in zephir.

@Jurigag
Strictly speaking it would actually belong to phalcon
because simply you require phalcon to debug and analyze it that way.
Any useful bug report would be reproducable (or atleast descriptive enough)
that you'd be able to fix it without requiring any 3rd-party application.
The only reason when bug reports from phalcon are tolerated is when
a) there are when there are sufficient instructions to reproduce
b) since phalcon and zephir are pretty coupled
Some of the current phalcon bug reports include an initial hint of what might
be causing the problem and some don't.
Generally I'm not really liking such bug reports ending up here,
since in the last few months they tend to get overlooked here and cannot
be really fixed without taking a look at immense phalcon internal cascades.

So I have to create the code in zephir without using phalcon classes which will produce the same stack and then you could possibly look into it? I don't get it.

As you might have noticed the crash happend in `zephir_call_user_function´
that alone is not anywhere near useful since that mehtod is literally called by any method or
function call.
The problem with fixing it is figuring out the context in which it actually crashes, which
requires attaching a debugger and analyzing the bug.
If you do that with an extensive amount of code like with a whole phalcon framework loaded,
you might end up chasing one ghost after the other since it's just too easy to get lost in a complex
logic cycle than in a simple test program.
In the past when I had more time for bugs like these, I usually also just produced smaller test cases and fixed them and could also directly integrate them into zephir's unit tests, which prevented these bugs from occurring again.

@Jurigag
just to clarify: That doesn't mean that you have to produce a smaller test case to fix it,
only that it's advisable and that I won't be able to afford time to look into bugs
occurring in complex 3rd-party situations.

Okay, so there is context why it's happening:
https://github.com/phalcon/cphalcon/issues/11583#issuecomment-200397410
Zephir have problems when creating instance of object when he has class passed as string. This function is called here in phalcon: https://github.com/phalcon/cphalcon/blob/2.1.x/phalcon/di.zep#L235
Exactly this line is producing a problem:
'.phtml' => 'Phalcon\Mvc\View\Engine\Php

This issue is somehow related to https://github.com/phalcon/zephir/issues/1218

I´ll take a look on this, I think several issues are caused by the same reason

@andresgutierrez look on the other i posted here too, beacause they can occur in many places too :)


[wallet, react] Add Coin98 Wallet

Problem solved
Short description of the bug fixed or feature added
Changes made

[x] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[x] Manual tests: step by step instructions on how to test

/release-pr

/release-pr

/release-pr

@ MananTank their deeplink doesn't seem to work, I have reached out to their Discord
https://docs.coin98.com/developer-guide/deeplink
Also the deeplink has a chainId parameter. Checking with them if it's optional, otherwise will need to get the chainId before calling the URL

/release-pr

/release-pr

@MananTank this PR is ready to be reviewed.
I deployed a website for you too ^: https://coin98-wallet.vercel.app/

Thanks @kien-ngo!


"Error setting up entry" after full power down

I installed ECHONETLite a few weeks ago and everything has been working perfectly.
I'm just getting some solar panels installed so the whole house has had power cut temporarily so the Air Condiitioner lost power, and I gracefully shut down the Home Assistant docker.
Now the power is back on and Home Assistant is alive and well with everything working except EchonetLite.
Heres the log file:
2023-10-30 13:37:33.983 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Living Room Air Con for echonetlite
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 399, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/echonetlite/__init__.py", line 241, in async_setup_entry
    await echonetlite.async_update()
  File "/config/custom_components/echonetlite/__init__.py", line 434, in async_update
    return await self.async_update_data(kwargs=kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/echonetlite/__init__.py", line 441, in async_update_data
    batch_data = await self._instance.update(flags, no_request)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pychonet/EchonetInstance.py", line 122, in update
    response = await self._api.echonetMessage(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pychonet/echonetapiclient.py", line 168, in echonetMessage
    await asyncio.sleep(0.1)
  File "/usr/local/lib/python3.11/asyncio/tasks.py", line 639, in sleep
    return await future
           ^^^^^^^^^^^^
asyncio.exceptions.CancelledError: aiohttp connection lost
2023-10-30 13:38:41.236 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Living Room Air Con for echonetlite
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 399, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/echonetlite/__init__.py", line 241, in async_setup_entry
    await echonetlite.async_update()
  File "/config/custom_components/echonetlite/__init__.py", line 434, in async_update
    return await self.async_update_data(kwargs=kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/echonetlite/__init__.py", line 441, in async_update_data
    batch_data = await self._instance.update(flags, no_request)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pychonet/EchonetInstance.py", line 122, in update
    response = await self._api.echonetMessage(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pychonet/echonetapiclient.py", line 202, in echonetMessage
    await asyncio.sleep(0.1)
  File "/usr/local/lib/python3.11/asyncio/tasks.py", line 639, in sleep
    return await future
           ^^^^^^^^^^^^
asyncio.exceptions.CancelledError: aiohttp connection lost

Everything else is working in Home Assistant, and the crappy Mitsubishi cloud app is still able to control the aircon (I know this is a different protocol, but just confirming that the aircon has network connectivity)
What are the troubleshooting steps from here?
Thanks

Heres additional logs that might help
2023-10-30 12:18:31.775 ERROR (MainThread) [homeassistant.components.sensor] Error adding entities for domain sensor with platform echonetlite
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 593, in state
    numerical_value = float(value)  # type:ignore[arg-type]
                      ^^^^^^^^^^^^
ValueError: could not convert string to float: 'unavailable'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 507, in async_add_entities
    await asyncio.gather(*tasks)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 752, in _async_add_entity
    await entity.add_to_platform_finish()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1057, in add_to_platform_finish
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 779, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 879, in _async_write_ha_state
    state, attr = self._async_generate_attributes()
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 820, in _async_generate_attributes
    state = self._stringify_state(available)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 785, in _stringify_state
    if (state := self.state) is None:
                 ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 595, in state
    raise ValueError(
ValueError: Sensor sensor.living_room_air_con_measured_outdoor_air_temperature has device class 'temperature', state class 'measurement' unit '°C' and suggested precision 'None' thus indicating it has a numeric value; however, it has the non-numeric value: 'unavailable' (<class 'str'>)
2023-10-30 12:18:31.801 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up echonetlite platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 593, in state
    numerical_value = float(value)  # type:ignore[arg-type]
                      ^^^^^^^^^^^^
ValueError: could not convert string to float: 'unavailable'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 367, in _async_setup_platform
    await asyncio.gather(*pending)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 507, in async_add_entities
    await asyncio.gather(*tasks)

This issue appears to be related to #144. It has been fixed in the master branch, so it may resolve this issue as well. You can get the master branch using HACS, so could you try it out?

This issue may have been fixed in Version 3.7.7, so I'm closing this. If the problem continues, you can reopen it.


sanitycheck: fixed handling of retries

When re-running failed tests, do not go through any filtering, just load
the failed tests directly and execute them. The filtering was done based on
default platforms and any tests that were failing on non-default platforms
were ignored.
Fixes #13956

Codecov Report

Merging #14025 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14025   +/-   ##
=======================================
  Coverage   52.24%   52.24%
=======================================
  Files         307      307
  Lines       45467    45467
  Branches    10525    10525
=======================================
  Hits        23752    23752
  Misses      16913    16913
  Partials     4802     4802

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 605ae10...f221a5b. Read the comment docs.


pull request for e2e test at 12/21/2016 21:15:24

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Increase policy_comp_operator_test timeout

Signed-off-by: Chunxi Luo<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/23684
These test cases failed due to operator status delay.  PENDING vs RUNNING / PENDING vs AGGREGATING, etc
Increase policy_comp_operator_test timeout and continuously observe it.
Also, increase the timeout for insightsClientPod creation for https://github.com/stolostron/backlog/issues/23609

/retest

/retest


Hangs in 'COPY  FROM...'

Hi!
I'm creating a daily job that moves data from another database into MapD, it generates a .csv file that is moved into MapD server and then use pymapd (execute) to launch the "copy  from '<.csv>'" statement. The problem is that when the .csv is large enough, it hangs and never returns. Any idea how to prevent this?
>>> copy_response = list(mapd.execute("COPY FACT_USER_ACQUISITION_DAILY_MAPD FROM '/raidStorage/imports/FACT_USER_ACQUISITION_DAILY_MAPD_FINAL.csv';"))[0][0]
^CTraceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/pymapd/connection.py", line 226, in execute     return c.execute(operation, parameters=parameters)   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/pymapd/cursor.py", line 120, in execute     nonce=None, first_n=-1)   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/mapd/MapD.py", line 1030, in sql_execute     return self.recv_sql_execute()   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/mapd/MapD.py", line 1046, in recv_sql_execute     (fname, mtype, rseqid) = iprot.readMessageBegin()   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/thrift/protocol/TBinaryProtocol.py", line 134, in readMessageBegin     sz = self.readI32()   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/thrift/protocol/TBinaryProtocol.py", line 217, in readI32     buff = self.trans.readAll(4)   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/thrift/transport/TTransport.py", line 60, in readAll     chunk = self.read(sz - have)   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/thrift/transport/TTransport.py", line 161, in read     self.__rbuf = BufferIO(self.__trans.read(max(sz, self.__rbuf_size)))   File "/home/user/.conda/envs/teradata_env/lib/python3.5/site-packages/thrift/transport/TSocket.py", line 117, in read     buff = self.handle.recv(sz) KeyboardInterrupt

Hi @tcourregesJC ,
Do your COPY FROM commands succeed when run from the mapdql command line client, or do they also hang there? That will help us determine if it's a pymapd issue or if it's a mapd-core issue.

How large are the input files which succeed and fail? # rows, # columns, file size

Does it seem like it's time-based, ie does it seem to fail after 30 seconds or similar?

Do the imports otherwise seem to succeed? Is the number of rows added the same as the number of rows in the file (possibly minus one for the header).

If the number of rows is different, could you provide your mapd_server*INFO* logs? They're usually located under /var/lib/mapd/data/mapd_log or /raidStorage/mapd-storage/data/mapd_log or similar.

Are you aware of our Java-based SQLImporter tool? It allows you to move data from other databases into MapD via JDBC. Might eliminate the need to write out to a CSV depending on your use case. https://www.mapd.com/docs/latest/mapd-core-guide/loading-data/#sql-importer

Hi @andrewseidl, the data is loaded successfuly.

The file i'm trying now is 24GB, 21944274 rows and 158 columns.
Yes, I think it's time based, I've tried with smaller csv files and it works fine.
Yes, the number of rows matches the rows in the csv and in the other database.
The number of rows match.
Yes! I know it's possible to move data via JDBC but it's considerablyslower than moving the file to the server and using "copy xxxx from .csv"

Thanks! Let me know if you need more info

I'm going to close this issue, if only because so many changes have come through MapD/OmniSci in the last year. If this is still an issue, please let us know and we can re-open the issue/create a new one


Panic (0.11 rc1) "slice index starts at 21 but ends at 1

Description
When I compile my document with v0.11 rc1 instead of v0.10 I now get
thread 'main' panicked at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce\library\alloc\src\vec\mod.rs:2084:36:
slice index starts at 21 but ends at 1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Running with RUST_BACKTRACE=full gives me a stacktrace, but that looks a bit useless to me:
thread 'main' panicked at /rustc/07dca489ac2d933c78d3c5158e3f43beefeb02ce\library\alloc\src\vec\mod.rs:2084:36:
slice index starts at 21 but ends at 1
stack backtrace:
   0:     0x7ff7f8d73029 - <unknown>
   1:     0x7ff7f8a0069d - <unknown>
   2:     0x7ff7f8d6e551 - <unknown>
   3:     0x7ff7f8d72e5a - <unknown>
   4:     0x7ff7f8d757b9 - <unknown>
   5:     0x7ff7f8d75475 - <unknown>
   6:     0x7ff7f8d75d34 - <unknown>
   7:     0x7ff7f8d75c09 - <unknown>
   8:     0x7ff7f8d73bd9 - <unknown>
   9:     0x7ff7f8d75882 - <unknown>
  10:     0x7ff7f97a4a77 - <unknown>
  11:     0x7ff7f97a5134 - <unknown>
  12:     0x7ff7f93129cb - <unknown>
  13:     0x7ff7f9229b55 - <unknown>
  14:     0x7ff7f933e0c0 - <unknown>
  15:     0x7ff7f939d8b8 - <unknown>
  16:     0x7ff7f9226fca - <unknown>
  17:     0x7ff7f945670a - <unknown>
  18:     0x7ff7f9574a3a - <unknown>
  19:     0x7ff7f9559195 - <unknown>
  20:     0x7ff7f94dadf6 - <unknown>
  21:     0x7ff7f8d67ede - <unknown>
  22:     0x7ff7f95775dc - <unknown>
  23:     0x7ff7f97a1a6c - <unknown>
  24:     0x7fffc12a257d - BaseThreadInitThunk
  25:     0x7fffc28caa58 - RtlUserThreadStart

Unfortunately, I cannot determine how to reproduce the issue from scratch. When I start deleting sections in my document, the panic just randomly disappears. It can be as little as adding/removing a blank line (i.e., a new paragraph) to "toggle" the error.
I am happy to provide more information and help debugging this if anyone can tell me what information to provide or what to try (I cannot share the entire document, though). I could run a debug build with backtracing later on my private PC if that helps.
Reproduction URL
No response
Operating system
Windows
Typst version

[X] I am using the latest version of Typst

Might be fixed by https://github.com/typst/typst/pull/3607

#3607 is merged. If you could try again with Typst built from main, that'd be great.

It is pretty surely that same issue that #3607 intended to fix, so I'll close this. Please reopen if you can still reproduce it!

I will give it a shot tonight. Thanks for the help!

With the current main branch d01ccffad6470ad49d4e47b5c0884dfd23305961 the bug is fixed 🎉


Make ruby_binary compatible with container

Reimplement the runfiles resolution based on the python version

Wow, @yugui , this looks amazing!
Could you be so kind and add a tiny bit of context into the description so that it's a bit easier to follow what is being changed and why? I am going to dig into this PR later today, but any additional description would be very helpful.
Thank you so much!
Konstantin

This looks really good, I like the removal the the loadpath script and have it all in one.
I was wondering about the ruby executable in the container image. Should pkg_tar bundling the ruby binary with the tar? Or shouldn't we rely on the ruby version inside the container?
The reason for the I ask is we are pretty focused on cross-compatability, and since most of us are on OSX here, bundling the binary will not work inside the image.
Cheers for this awesome work!

@kigster @grahamjenson
Thank you for your comments.
You are right. I should have explained more about the design and future plans.
What this PR does

Correctly handles several different types of runtime directory structure discussed at an equivalent code in rules_nodejs

I have reimplemented this runfiles resolution based on an equivalent code which py_binary provides.

Makes ruby_binary work fine in container.
Stop depending on the installation path of the host interpreter.

ruby_binary built on MacOS is now able to run on Linux as far as the binary and its dependencies do not contain native extensions.
The container test does work on MacOS because currently the test scripts are pure-Ruby.

Removes extra complexity in the original implementation.

Some of them were introduced just because of my misunderstanding about Bazel's sandbox system. Some others are no longer necessary

Limitations

This PR stopped bundling host interpreter in ruby_binary. So ruby_binary now depends on the interpreter installed on the runtime environment as py_binary does.

As a side-effect, this PR tentatively stopped resolving the interpreter path from runfiles and always tries to use the host interpreter.
I am going to recover the functionality of interpreter path resolution as nodejs_binary does, just after this PR. It will be necessary to support Sorbet's interpreters.

For cross-compatibility, it is not a problem for the existing tests because they are pure-Ruby. However, we will need a cross-compiling with Sorbet's interpreters and C cross-compilers to  support native extensions.

Plans
How about working on the issues with the following steps?

Recover the interpreter bundling functionality ASAP (yugui)
Solbet Ruby toolchain (graham) -- https://github.com/bazelruby/rules_ruby/tree/graham/sorbet-ruby-toolchain
Support cross-compiling using Solbet toolchains if necessary

Make it possible to build Linux container image on MacOS.

@yugui @grahamjenson All of that sounds great!
Let's Start Using Github Project!
(CC @volkangurel) Guys — I would like to note that we started using the Github Project feature (which looks a lot like Trello) and I would love it if you added any work you plan on doing there in the appropriate column.
Basically, the Project View is a project view resembling Trello swimming lanes and is meant more for brainstorming than for project tracking. Cards we create can remain cards or be used to very easily auto-generate a new linked issue, where we should add more technical details, if needed.
Subsequent PR will reference the issue, and once merged, it will auto-fix the issue, and the associated card will move into the Done ✅ column.  Since the three of us are already moving pretty quickly, I would love to have one place where we can all see where we are all at.
I am going to take the liberty and add Yugui's bullet points as Cards there , but will leave it to her to create the issues and move each card into the In Progress when able.
Mini Sprint Planner?
If it were possible to book a 45 minute time slot later this week for the three of us to hop on Google Hangout and do a mini-planner, that would be really great. Yugui, we have 7-hour difference, so say 10am in Japan would be 5pm in San Francisco and we can certainly make that happen easily. The only problem with this week is that Graham is at AWS Reinvent Conference.


Removed Attribute from the namespace for DynamicDataAttribute

Removed Attribute from the namespace for DynamicDataAttribute

@harshjain2,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@dotnet-bot test this please.

@dotnet-bot test "Windows / Debug Build"

@dotnet-bot test "Windows / Debug Build"

@dotnet-bot test "Windows / Debug Build"

@dotnet-bot test "Windows / Debug Build"

@dotnet-bot test "Windows / Debug Build"

@dotnet-bot test "Windows / Debug Build"

@harshjain2,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Process New Song List

Implement a way for users to add new csv rows to the dataset and persist it to the source file

[x] Provide a text area for user to type or paste a string of new CSV data
[x] Upon submission, validate that the text is a valid CSV string and can be parsed successfully into one or more songs
[x] After parsing, show a summary to the user of the new data (number of songs, preview grid?)
[x] Alert the user if the new song already exists in the grid (keyed on new_file_name)
[x] Insert the new songs to the active dataset obeying the current sort rules (if no sort is specified, add songs to the top of the list)
[x] Do not allow songs to be inserted if they are already in the dataset (keyed on new_file_name)

[X] Insert the new songs to the active dataset obeying the current sort rules (if no sort is specified, add songs to the top of the list)

Original implementation skipped this bullet. This would really be more useful as an option when adding new songs, so I created #39 for that feature.


(request) Safe-mode isolation/execution

Hi all! 👋 This may be out of scope, but would be happy to contribute myself if relevant, and pointed in the right direction.
Use-case
One of my projects uses Goja for executing user-defined JavaScript. For obvious security reasons, I'd like to prevent certain snippets of code from executing, including (but probably not limited to):

eval(...)
require(...)
globalThis[...]
process

While I am currently accomplishing this with simple code-checks and assigning the aforementioned identifies as undefined, I am curious as to what the best way would be to achieve complete isolation from the underlying host in Goja (without something like accessing a Docker container over the network - the Goja invocations often range into a few hundred-thousand and the application is performance-critical).
Some helpful things would be:

Ability to pass a context for cancelation to a Callable.
Ability to deny access entirely to the underlying filesystem and environment.

Thanks for your time! 😄

Correct me if I'm wrong, but the default Goja environment (the result of running goja.New()) only has the eval method and the global this object as illustrated below. Require would only be available if you loaded a module loader into your runtime and process is a NodeJS package that you would have to re-write in Go and make available to the Goja runtime.
func TestRuntime(t *testing.T) {
	runtime := goja.New()
	// This works
	t.Run("eval", func(t *testing.T) {
		value, err := runtime.RunString(`eval(10 + 10)`)
		require.NoError(t, err)
		require.Equal(t, "20", value.String())
	})
	// This works
	t.Run("this", func(t *testing.T) {
		value, err := runtime.RunString(`this`)
		require.NoError(t, err)
		require.NotNil(t, value)
	})
	// This does not work
	t.Run("require", func(t *testing.T) {
		_, err := runtime.RunString(`require`)
		require.Error(t, err)
	})
	// This does not work
	t.Run("process", func(t *testing.T) {
		_, err := runtime.RunString(`process`)
		require.Error(t, err)
	})
}

It seems to me that the only thing you have to worry about is the functionality that you expose via your own Go binding functions, all other functionality is isolated to the runtime. What I mean by that is that you don't have to worry about someone modifying your host machine filesystem for example unless you provide a Go binding function to the Goja runtime that allows the runtime to modify the host filesystem.

@clarkmcc Nice! In my tests globalThis seemed to have also worked but I suppose most of my worries are misplaced in require and fs are unavailable by default 😄

Please see #85 and #94. Bottom line is, it depends on your definition of 'safe'. Filesystem access can be prevented by disabling source maps (rt.SetParserOptions(parser.WithDisableSourceMaps)) or providing your own source map loader (parser.WithSourceMapLoader), but it's still open to DoS attacks by starving CPU or RAM. The former can be partly mitigated by limiting execution time using Interrupt.


[CP] Remove unused co19_2 directory

Commit(s) to merge
be098f08e1df736331cbb7adf8c05637b599957b
Target
beta
Prepared changelist for beta/stable
https://dart-review.googlesource.com/c/sdk/+/339901
Issue Description
The co19_2 directory was accidentally revived as a git submodule.
What is the fix
Delete the incorrect submodule.
Why cherry-pick
The submodule should not be there, and can cause unexpected side effects with tooling that doesn't expect submodules in the Dart SDK checkout.
Risk
low
Issue link(s)
N/A
Extra Info
No response

lgtm

@athomas is this still relevant?

No, forgot to close it.


Remove "Edge lengths" docs todo

The "Edge lengths" section of cell statistics seems done, so I propose removing this "todo".

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

coverage: 98.826%. remained the same
when pulling c9274d12069e70ce56492e7d55fcf2db301fe9fe on Twixes:patch-1
into ecc0d254d2fcc93a8df807b21e116f942092d161 on uber:master.


GITOPS-2690 adding all the example used in testing from different sources

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:

adds examples for tests from https://github.com/argoproj/argocd-example-apps, https://github.com/bluengo/argocd-apps-examples, https://github.com/jaideepr97/gitops-examples, https://github.com/jannfis/operator-e2e-git used for testing of gitops-operator
~updates tests to use these example instead of examples present in different repositories.~ (I will cover this on a seperate PR)

Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes GITOPS-2690
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:

/test v4.10-kuttl-sequential

/test v4.11-kuttl-parallel v4.10-kuttl-parallel v4.12-kuttl-parallel

Some tests are failing with
        +  - lastTransitionTime: "2023-07-18T07:13:53Z"
        +    message: 'rpc error: code = Unknown desc = Manifest generation error (cached):
        +      test/examples/kustomize-guestbook: app path does not exist'

As the tests are pointed to HEAD for example path,
We can merge this PR without modification to the tests and then make changes to test on a separate PR to test if the tests are working as expected or not.
@varshab1210 @trdoyle81 @iam-veeramalla, WDYT?

/test ci/prow/v4.10-e2e

/test v4.10-e2e

/test v4.11-kuttl-sequential

/lgtm

It's just a rebase, so adding the label again.
/lgtm


Report generation OOME

Using version 6.8.8:
java.lang.OutOfMemoryError: Java heap space
        at java.util.Arrays.copyOf(Arrays.java:2367)
        at java.lang.AbstractStringBuilder.expandCapacity(AbstractStringBuilder.java:130)
        at java.lang.AbstractStringBuilder.ensureCapacityInternal(AbstractStringBuilder.java:114)
        at java.lang.AbstractStringBuilder.append(AbstractStringBuilder.java:415)
        at java.lang.StringBuilder.append(StringBuilder.java:132)
        at org.testng.reporters.Files.readFile(Files.java:28)
        at org.testng.reporters.Files.readFile(Files.java:20)
        at org.testng.reporters.FileStringBuffer.toString(FileStringBuffer.java:137)
        at org.testng.reporters.XMLStringBuffer.toXML(XMLStringBuffer.java:331)
        at org.testng.reporters.jq.ChronologicalPanel.getContent(ChronologicalPanel.java:93)
        at org.testng.reporters.jq.BaseMultiSuitePanel.generate(BaseMultiSuitePanel.java:24)
        at org.testng.reporters.jq.Main.generateReport(Main.java:63)
        at org.testng.TestNG.generateReports(TestNG.java:1115)
        at org.testng.TestNG.run(TestNG.java:1074)
        at org.apache.maven.surefire.testng.TestNGExecutor.run(TestNGExecutor.java:115)
        at org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.executeMulti(TestNGDirectoryTestSuite.java:204)
        at org.apache.maven.surefire.testng.TestNGDirectoryTestSuite.execute(TestNGDirectoryTestSuite.java:107)
        at org.apache.maven.surefire.testng.TestNGProvider.invoke(TestNGProvider.java:112)
        at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:200)
        at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:153)
        at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)

The exception appears when there are a more than a few test failures.

Looks similar to this:
https://github.com/cbeust/testng/issues/291
It says, that it fixed in latest version, but I still has similar issue with 6.8.13
The only workaround I found, is to clear Report regularly:
try {
    assert compareScreenShots(frame, actual, "build/screenshots/proxy"),
      "Too many differences between the two panes, this URL fails: ${url}"
}
finally {
    Reporter.clear()
}

Somehow, at the end report generated anyway

I found, that my advice above only partially resolve issue - memory still grows, just slower... At the end, I still getting OutOfMemory

It's odd that Reporter.clear() would help: are you logging a lot of things in the Reporter? Do you have a lot of tests? Can you break point into Reporter.clear() and see how big the two fields that get cleared are?

I do not log anything explicitly, But I found, that testNG default Reporter logs all test case properties, as I can see in Heap Walker.
So, content of screenshots are written in the Reporter - that's a lot of data. And I do not know, how I can make it NOT write this data in the Reporter. I tried to set them to null, but it didn't work.
Anyway, I tried to debug, as you suggested, and found one strange thing, which would explain, why even with Reporter.clear() memory still grows:
The code in my "finally" block gets executed ONLY if assert PASS. If it FAIL - than code in "finally" block never gets to execute. Is it intended? It certainly not intuitive.
Here is Heap Walker report:
http://slavikf.com/files/testNG1/
Note, that  java.util.HashMap$Node element created for each test run (I'm using data provider with array of URLs) and each of that element contain 2 screenshots (frameImg and actualImg). So, it grows out-of-control.
And it seems, that Reporter.clear() takes care of it.
It just, that now I realize, that I need to put it after Assert, too.

Indeed, thanks for the heap dump.
I'm guessing you are using Selenium and adding screen shots in the test result, since TestNG itself doesn't use the Reporter at all.
I think the main problem here is that ITestResult is used as a key in the m_methodOutputMap, and this is really not necessary at all: all I need is a unique identifier from a test result in order to get the logged data. This would remove all the ITestResult from the maps and replace them with Strings (most likely) as keys.
Actually, even using the ITestResult's hashCode() should be good enough, since it seems it's already doing that (and using the default one, so identityHashCode really).
I can create a patched testng.jar if you're interested, you can test it and let me know if it helps.
Please email me directly and I'll send it to you.
Thanks!

Reporter class now uses only the method id as the key for m_methodOutputMap. Please comment/add a sample that can be used to reproduce the problem if issue still exists in latest released version of TestNG as of today 7.8.0


"Add LLVM" doesn't work

I manually downloaded and installed the latest LLVM from the official website. I installed it into a non-default location. I click "Add LLVM" and select that LLVM folder, but nothing happens. I tried selecting the LLVM\bin folder, nothing happens still.
I would like a clear error message instead of a silent failure.

Hi @VioletGiraffe,
Thank you for reporting the issue. We are investigating the problem and will provide a solution along with a clear error message.
Regards,
Marina

Hi @VioletGiraffe,
What version of LLVM you trying to add?

16.0.0

Hi @VioletGiraffe,
Do you use the last version of Clang Power Tools?
I found out that our extension accepts adding just predefined versions of LLVM, but we will disable this check. If you are using a lower version of Clang Power Tools, it may be the reason why it fails.
Kind regards,
Marina

Hi Marina, it was the latest one on the day I submitted this issue. i. e. 5 days ago.

Hi @VioletGiraffe,
We cannot reproduce your issue, and we don't find something suspicious in our source code
Can you provide the full log output after compile run? (activate verbose from Settings -> Verbose mode)
Please send us the settings.json file from %appdata%\ClangPowerTools\settings.json
Kind regards,
Marina

I can see that it does work, so maybe there's no problem. It found LLVM because I added it to PATH. I'm just put off by the fact that I can't specify the LLVM installation manually, and if it wasn't on the PATH, it would likely not work. Would a log still be helpful when analysis is working?

Glad to hear that it's working
We definitely will output a message when operation will be completed successfully
Kind regards,
Marina

Hi @VioletGiraffe,
We have a bugfix for this issue in Clang Power Tools release v2023.7.0
Kind regards,
Marina


XUbuntu 18.04 SIFT install failure and fix

Hello,
I was installing SIFT via the CLI on Xubuntu 18.04 and SIFT installation fails. The saltstack.log file is attached and there are 5 failures - all related to gnome dock.
OS version details

NAME="Ubuntu"
VERSION="18.04.5 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic

Error details
$ grep "result: false" saltstack.log 2>/dev/null | wc -l
5

gsettings set org.gnome.shell favorite-apps "['gnome-terminal.desktop','firefox.desktop', 'org.gnome.Nautilus.desktop']"

On the XUbuntu machine this fails. And the root cause of this is because of two missing packages.

gnome-shell-extensions
gnome-shell-extension-dashtodock

I found this by running the gsettings and noticing that on xubuntu these are not available
$ gsettings set org.gnome.shell favorite-apps "['gnome-terminal.desktop','firefox.desktop', 'org.gnome.Nautilus.desktop']"
No such schema “org.gnome.shell”

Fix
sudo apt-get install gnome-shell-extensions gnome-shell-extension-dashtodock
I understand that the supported distro mentions 18.04 Ubuntu Desktop (only default Gnome Desktop) but it might be useful to add this a dependency so that this can work on lightweight Ubuntu distributions too. The specific failures are detailed in the siftlog-failures-20210117.txt file.
saltstack.log
siftlog-failures-20210117.txt

I'll take a look at adding these two dependencies.
Regards,
-Erik
On Sun, Jan 17, 2021 at 8:24 AM gnittala<EMAIL_ADDRESS>wrote:

Hello,
I was installing SIFT via the CLI on Xubuntu 18.04 and SIFT installation
fails. The saltstack.log file is attached and there are 5 failures - all
related to gnome dock.
OS version details
NAME="Ubuntu"
VERSION="18.04.5 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic
$ grep "result: false" saltstack.log 2>/dev/null | wc -l
5
The main cause of the error is this
gsettings set org.gnome.shell favorite-apps "['gnome-terminal.desktop','firefox.desktop', 'org.gnome.Nautilus.desktop']"
On the XUbuntu machine this fails. And the root cause of this is because
of two missing packages.

gnome-shell-extensions
gnome-shell-extension-dashtodock

I found this by running the gsettings and noticing that on xubuntu these
are not available
$ gsettings set org.gnome.shell favorite-apps "['gnome-terminal.desktop','firefox.desktop', 'org.gnome.Nautilus.desktop']"
No such schema “org.gnome.shell”
The fix is to
sudo apt-get install gnome-shell-extensions
gnome-shell-extension-dashtodock
I understand that the supported distro mentions 18.04 Ubuntu Desktop
(only default Gnome Desktop) but it might be useful to add this a
dependency so that this can work on lightweight Ubuntu distributions too.
The specific failures are detailed in the siftlog-failures-20210117.txt
file.
saltstack.log
https://github.com/teamdfir/sift/files/5826302/saltstack.log
siftlog-failures-20210117.txt
https://github.com/teamdfir/sift/files/5826308/siftlog-failures-20210117.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/teamdfir/sift/issues/506, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALZSPCSNWGMSNETP2HAB3S2L6JNANCNFSM4WGF37QA
.

I'll take a look at adding these two dependencies.
Regards,
-Erik
On Sun, Jan 17, 2021 at 8:24 AM gnittala<EMAIL_ADDRESS>wrote:

Hello,
I was installing SIFT via the CLI on Xubuntu 18.04 and SIFT installation
fails. The saltstack.log file is attached and there are 5 failures - all
related to gnome dock.
OS version details
NAME="Ubuntu"
VERSION="18.04.5 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic
$ grep "result: false" saltstack.log 2>/dev/null | wc -l
5
The main cause of the error is this
gsettings set org.gnome.shell favorite-apps "['gnome-terminal.desktop','firefox.desktop', 'org.gnome.Nautilus.desktop']"
On the XUbuntu machine this fails. And the root cause of this is because
of two missing packages.

gnome-shell-extensions
gnome-shell-extension-dashtodock

I found this by running the gsettings and noticing that on xubuntu these
are not available
$ gsettings set org.gnome.shell favorite-apps "['gnome-terminal.desktop','firefox.desktop', 'org.gnome.Nautilus.desktop']"
No such schema “org.gnome.shell”
The fix is to
sudo apt-get install gnome-shell-extensions
gnome-shell-extension-dashtodock
I understand that the supported distro mentions 18.04 Ubuntu Desktop
(only default Gnome Desktop) but it might be useful to add this a
dependency so that this can work on lightweight Ubuntu distributions too.
The specific failures are detailed in the siftlog-failures-20210117.txt
file.
saltstack.log
https://github.com/teamdfir/sift/files/5826302/saltstack.log
siftlog-failures-20210117.txt
https://github.com/teamdfir/sift/files/5826308/siftlog-failures-20210117.txt
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/teamdfir/sift/issues/506, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALZSPCSNWGMSNETP2HAB3S2L6JNANCNFSM4WGF37QA
.

After looking into this, it seems that the gnome-shell-extension-dashtodock is only available on Bionic, Hirsute and above, but not on Focal.
Technically, the deb package under Hirsute available on the Ubuntu PPA does install and appears to work on Focal, so it "could" be a viable option. However, any changes that might occur with this package which only function in Hirsute could then break future compatibility in Focal.
@ekristen , if you're interested in giving it a go for Bionic and Focal, I can do up a state which will run on both, but the Focal one will need a direct download of the deb package from the PPA for install.

We are dropping 18.04 and only supporting 20.04 and 22.04 as soon as https://github.com/teamdfir/sift-saltstack/pull/80 is merged.
Side Note: SIFT CLI will be officially deprecated as of March 1, 2023, and is replaced with cast


Make token extractions more generalized

Changes:

Choose what header(s) and what form(s) to extract it from
Make it functional options

Closed by #24


Using object for RecipientBox rather than bytearray

Create specific immutable type for RecipientBox rather than byte[].
Make list unmodifiable

Codecov Report

Merging #1100 into master will decrease coverage by 0.02%.
The diff coverage is 96.42%.

@@             Coverage Diff              @@
##             master    #1100      +/-   ##
============================================
- Coverage     99.37%   99.34%   -0.03%
- Complexity     2294     2306      +12
============================================
  Files           352      353       +1
  Lines          6391     6431      +40
  Branches        363      372       +9
============================================
+ Hits           6351     6389      +38
  Misses            5        5
- Partials         35       37       +2

Flag
Coverage Δ
Complexity Δ

#unittests
99.34% <96.42%> (-0.03%)
2306.00 <14.00> (+12.00)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ
Complexity Δ

...om/quorum/tessera/partyinfo/ResendManagerImpl.java
93.75% <86.66%> (-3.75%)
8.00 <0.00> (ø)

...n/java/com/quorum/tessera/enclave/EnclaveImpl.java
100.00% <100.00%> (ø)
17.00 <0.00> (ø)

...ava/com/quorum/tessera/enclave/EncodedPayload.java
100.00% <100.00%> (ø)
18.00 <12.00> (+11.00)

...ava/com/quorum/tessera/enclave/PayloadEncoder.java
100.00% <100.00%> (ø)
1.00 <1.00> (ø)

...com/quorum/tessera/enclave/PayloadEncoderImpl.java
100.00% <100.00%> (ø)
8.00 <0.00> (ø)

.../java/com/quorum/tessera/enclave/RecipientBox.java
100.00% <100.00%> (ø)
1.00 <1.00> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f666ada...bce13db. Read the comment docs.


chore: move dev tools to devDependencies

This moves all

babel-*
eslint-*
prettier

dependencies to devDependencies, causing a smaller download footprint for consumers.

Can you do this with a squash commit?


clients: add .env.template

upcoming pull request will be related with package.json (add @dotenvx/dotenvx) and README.md change

We already have a .env.template in clients/apps/web; so I don't think this is needed. Could you elaborate on this?

oh, I was never developing on clients/app/web, I might just do some readme update


0.28.1

Migrate to voluptuous (#2954)
Ecobee (#3055)
update ha-ffmpeg version to 0.9 (#3059)
Device tracker component & platform validation. No more home_range. (#2908)
Merge pull request #3051 from tchellomello/added_random_effect_flux_led - Added option to use effect:random for Flux Led light bulbs
Use voluptuous for smtp (#3048)
Upgrade sendgrid to 3.2.10 (#3044)
Upgrade TwitterAPI to 2.4.2 (#3043)
Fix bug in wemo discovery caused by voluptuous addition. (#3027)
Merge pull request #3036 from home-assistant/bug_fix_asuwrt
Bug fix for asuswrt device_tracker. Issue #3015
Remove units for humidity in Wundeground sensor (#3018)
Fix media_player descriptions and select_source (#3030)
Allow user to configure server id to perform speed test against (#3008)
Bug fix for asuswrt device_tracker. Issue #3015 (#3016)
Ecobee (#3025)
Map Modes to setpoint indexes (#3023)
fix voluptuous and cover autodiscovery (#3022)
Fixes wrong statevalue and problem with zwave setpoint (#3017)

Shouldn't the release be 0.27.1?

@arsaboo EXCELLENT catch :cookie:


typo in method name (but still worked cause typo was in both places)

I found this one because I was trying to call instantiate_resouRce from within an function override.
Caution about merging this one, because I haven't looked, but Loomio itself may make reference to this misnamed function at some places in the code.

You rock!


Feathers from master PR

Please merge with master repo, we need the features from there and ligature as well

yea, but i love alacritty too much, it's advantage is too much, such as leigweight

I personally don't use alacritty anymore, and there are too many commits after my last rebase.
It's really hard to rebase on the newest alacritty since the architecture and the concept of the underlying system are all changed.
The same method of rendering ligature won't work in the latest alacritty.


Incremental max value

Part of https://keboola.atlassian.net/projects/COM/issues/COM-8

@pivnicek now orderBy is allowed only with incremental fetching limit


Upgrade to Xamarin.Forms 3.5

Now that Xamarin.Forms 3.5.0 is out, we can start work to integrate this new version.
This should fix #155 and #295 as well

For reference, here's the delta between XF 3.4.009999 and <IP_ADDRESS>452
https://www.fuget.org/packages/Xamarin.Forms/<IP_ADDRESS>452/lib/netstandard2.0/diff/<IP_ADDRESS>9999/

Are there some parts which we can say we won't add them, so we can make a more clearly list? So define which things we need.
E.g. interfaces and enums we do not have to look at, right?
Are there some other parts which you know about?


Grammar and structure suggestions

Please see my comments in #61

makes sense i'll update that. thanks

ACK
thanks.


Compressing static assets

Enabling ngx_http_gzip_module to compress assets on the way out.
This fixes #491.

Thanks @davidalber!  How about compressing the HTML output, too? 😁


Add link to gocrack-ui

Codecov Report

Merging #46 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #46      +/-   ##
==========================================
+ Coverage   22.82%   22.87%   +0.05%
==========================================
  Files          68       68
  Lines        3602     3602
==========================================
+ Hits          822      824       +2
+ Misses       2712     2710       -2
  Partials       68       68

Impacted Files
Coverage Δ

server/notifications/cache.go
81.57% <0%> (+5.26%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dc14ad5...92e371d. Read the comment docs.

Awesome thanks!


Created function get_total_repositories() in github.py

Created the get_total_repositories() function which takes the username as an argument and returns the number of repositories in integer datatype.

Thanks for your contribution


Fix issue with tracing

Currently only order-service and inventory-service is printing traceId and spanId, Rest all are not printing it. Fix this

61ad3191493e483befec58d6cc2de27bdccf2a1e


Admins editing advisors/mentors are shown all lab member as options

When admins are editing the page for other lab members and are attempting to set an advisor / co-advisor / grad mentor, all lab members are available as options from the drop-down and search. For example, in the screenshot below, I see undergrads as options when trying to set an Advisor.

There is validation, so if I try to set someone to a role that they should be in, I get an error upon saving. But it shouldn't let me get that far!

OH! And when you click on the drop-down, it would be great if the cursor automatically focused on the search box so you could just start typing. As of now, you have to click on the drop down then in the search box to start searching for someone.


[AutoPR dns/resource-manager] Private-DNS swagger

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

Codecov Report

Merging #1912 into restapi_auto_dns/resource-manager will increase coverage by <.01%.
The diff coverage is 75%.

@@                          Coverage Diff                          @@
##           restapi_auto_dns/resource-manager    #1912      +/-   ##
=====================================================================
+ Coverage                               52.9%    52.9%   +<.01%
=====================================================================
  Files                                   4698     4698
  Lines                                 116020   116041      +21
=====================================================================
+ Hits                                   61383    61396      +13
- Misses                                 54637    54645       +8

Impacted Files
Coverage Δ

azure-mgmt-dns/azure/mgmt/dns/models/mx_record.py
57.14% <0%> (-9.53%)
:arrow_down:

...ure-mgmt-dns/azure/mgmt/dns/models/cname_record.py
66.66% <0%> (-13.34%)
:arrow_down:

...re/mgmt/dns/models/record_set_update_parameters.py
66.66% <0%> (-13.34%)
:arrow_down:

...zure-mgmt-dns/azure/mgmt/dns/models/aaaa_record.py
66.66% <0%> (-13.34%)
:arrow_down:

azure-mgmt-dns/azure/mgmt/dns/models/srv_record.py
44.44% <0%> (-5.56%)
:arrow_down:

azure-mgmt-dns/azure/mgmt/dns/models/txt_record.py
66.66% <0%> (-13.34%)
:arrow_down:

azure-mgmt-dns/azure/mgmt/dns/models/ptr_record.py
66.66% <0%> (-13.34%)
:arrow_down:

azure-mgmt-dns/azure/mgmt/dns/models/caa_record.py
50% <0%> (-7.15%)
:arrow_down:

azure-mgmt-dns/azure/mgmt/dns/models/soa_record.py
100% <100%> (ø)
:arrow_up:

azure-mgmt-dns/azure/mgmt/dns/version.py
100% <100%> (ø)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b0f1bbb...3521aa7. Read the comment docs.


[r/python/c++] Depend on core 2.14, tiledb-r 0.18, tiledb-py  0.20

Issue and/or context:
#752
Changes:
Only package-version dependency updates.
Notes for Reviewer:
This PR does not implement UTF-8 mods for #415, although it does unblock work in that direction.

Codecov Report
Base: 77.86% // Head: 59.01% // Decreases project coverage by -18.86% :warning:

Coverage data is based on head (4e9ec66) compared to base (04e47f3).
 Consider uploading reports for the commit 620d5b6 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #766       +/-   ##
===========================================
- Coverage   77.86%   59.01%   -18.86%
===========================================
  Files          30       40       +10
  Lines        2006     1576      -430
===========================================
- Hits         1562      930      -632
- Misses        444      646      +202

Flag
Coverage Δ

python
?

r
59.01% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

apis/python/src/tiledbsoma/util_ann.py

apis/python/src/tiledbsoma/dense_nd_array.py

apis/python/src/tiledbsoma/constants.py

apis/python/src/tiledbsoma/util_scipy.py

apis/python/src/tiledbsoma/factory.py

...on/src/tiledbsoma/options/tiledb_create_options.py

apis/python/src/tiledbsoma/experiment_query.py

apis/python/src/tiledbsoma/tiledb_object.py

apis/python/src/tiledbsoma/util.py

apis/python/src/tiledbsoma/util_iter.py

... and 60 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Replacing with #770
There were some CI jobs I cancelled earlier while https://pypi.org/project/tiledb wasn't ready; now, even with 'Re-run all jobs' they seem to not be all getting to run. #770 is a clean-slate attempt.


i18n for Farsi.

Add message for Persian language.

@aliazani Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@aliazani Thank you for signing the Contributor License Agreement!

I guess this is interesting, but it's sort of academic right, because the app contains plenty of Strings that are not translated? We probably can't really ever claim that we aim to support all (or any) languages without doing some more fundamental work on extracting more text into messages.
However, if we were to start accepting new translations for the existing minimal set of labels, is there a particular reason that Farsi should be the first?

I guess this is interesting, but it's sort of academic right, because the app contains plenty of Strings that are not translated? We probably can't really ever claim that we aim to support all (or any) languages without doing some more fundamental work on extracting more text into messages.

Yes, I think that's right about academic right things.
To answer your question about any reason that Farsi should be the first, Short answer is no.
There is no superiority for Farsi against German or any other languages.
I just try to contribute for the I18n for Farsi, the language that I know well as native person.


feat: remove quorum for evm

Closes: https://github.com/snapshot-labs/sx-ui/issues/468

hm, all of it dynamic quorum changes looks ugly, maybe there is a better way. Maybe it can be only temporary solution.

hm, all of it dynamic quorum changes looks ugly, maybe there is a better way. Maybe it can be only temporary solution.

Maybe we can avoid those ugly changes by enabling additionalProperties on the definition so we can just pass quorum in without worrying about mapping types/data.

yep, here will be the same solution

@Sekhmet looked again and decided to leave this dynamic staff, we still need to hide quorum from definition


Release with default pypi.org repository URL not working

When working with the default setup to push artifacts to the pypi.org, the following issue in encountered on push during the release workflow of 2.14.0:
Caused by: java.lang.NullPointerException
[INFO]     at org.technologybrewery.habushu.PublishToPyPiRepoMojo.addTrailingSlash (PublishToPyPiRepoMojo.java:231)
[INFO]     at org.technologybrewery.habushu.PublishToPyPiRepoMojo.getRepositoryUrl (PublishToPyPiRepoMojo.java:224)
[INFO]     at org.technologybrewery.habushu.PublishToPyPiRepoMojo.publishPackage (PublishToPyPiRepoMojo.java:162)
[INFO]     at org.technologybrewery.habushu.PublishToPyPiRepoMojo.doExecute (PublishToPyPiRepoMojo.java:122)
[INFO]     at org.technologybrewery.habushu.AbstractHabushuMojo.execute (AbstractHabushuMojo.java:304)

It appears that when using the default repo, we don't check that the url we want to ensure ends in a trailing slash is non-null before the the check.
Workaround
Can be worked around with the following configuration, if needed, in the meantime:
                <configuration>
                    <!--NB: workaround until https://github.com/TechnologyBrewery/habushu/issues/134 is fixed: -->
                    <pypiRepoUrl>https://pypi.org</pypiRepoUrl>
                </configuration>

Recreated with a new unit test:
[ERROR] Provides ability to push .dev versions to a different repository than released versions.Provides ability to push .dev versions to a different repository than released versions - Ensure trailing slash handles default case where an empty URL is passed to poetry for default handling without issue -- Time elapsed: 0.009 s <<< ERROR!
java.lang.NullPointerException
        at org.technologybrewery.habushu.PublishToPyPiRepoMojo.addTrailingSlash(PublishToPyPiRepoMojo.java:231)
        at org.technologybrewery.habushu.PublishToPyPiRepoMojo.getRepositoryUrl(PublishToPyPiRepoMojo.java:222)
        at org.technologybrewery.habushu.DeployPhaseSteps.the_repository_url_is_null(DeployPhaseSteps.java:51)
        at ✽.the repository url is null(classpath:specifications/dev-repository-deployment.feature:54)

Resolved


Fix minor crashing bug...

If you created an Incognito Window, it would crash w/ no cookies.

@shun-fan PTAL
@tswast FYI


Details: errors is an  invalid keyword argument for this function

Checklist for submitting an issue to KickThemOut:

[x] I have carefully read the README file and haven't managed to resolve my issue.
[x] I have searched the issues of this repo and believe that this is not a duplicate.
[x] I am running the latest version of KickThemOut.

OS name & version:
Python version:
Scapy version:
Nmap version:
Link of Gist:

Description:

Hey there @Thehere. This problem should be solved in the latest release. Thanks!

I do still encounter this problem? Is that correct?

All right, but have you re-installed KickThemOut and followed its new installation process?