Add public BaseModel.iter() method

Feature Request
Add BaseModel.iter() method which will behave exactly like a dict, but without converting nested values to dict by default and will be a generator instead
Why
Imagine you want to iterate through values in model, but also need to exclude some, or use aliases instead of attributes. So what you need to do is write something like this:
for attr, value in model.dict(exclude={...}, by_alias=True).items():
    do_something(attr, value)

Obviously, in this case python will firet evaluate model.dict(exclude={...}, by_alias=True). part, going through all values and only then you will be able to iterate, going through the same values again.
Also, by using dict nested values are force-converted to dict, and this might be very slow when we just want to iterate through top-level values.
We also cannot use existing _iter(), because some work is actually done in dict()
How we can solve it
By having iter() method we can bypass this double-iteration problem and converting-to-dict problems:
for attr, value in model.iter(exclude={...}, by_alias=True):
    do_something(attr, value)

Selected Assignee: @hramezani

This already exists!
It's called ._iter().

Should be documented I guess.

We also cannot use existing _iter(), because some work is actually done in dict()

I don't want to add more public method to models, best I think to implement this yourself.
If there's a simple way to achieve this with changes to existing functions of happy review a PR.

Sorry, I missed your comment when replying in a hurry.

I don't want to add more public method to models, best I think to implement this yourself.

Just to be clear, thats valid even if ._iter() itself would become public without adding any new methods?

If there's a simple way to achieve this with changes to existing functions of happy review a PR.

I guess the simplest way would be to move those excluding and aliases stuff from .dict() to ._iter(), that will also simplify .dict() too.
I'll try to play with it soon and will be happy to make a PR once it'll be done.

For what it's worth, I would be in favor of moving logic from dict into _iter -- I think it would simplify/result in a smaller diff for the implementation of dump_as_type from #812, and the related serialization performance improvements (even when not dumping as a specified type).

On V2, we can leverage .model_dump() as follows:
from pydantic import BaseModel

class Foo(BaseModel):
    a: str
    b: int

foo = Foo(a="a", b=1)

for key, value in foo.model_dump().items():
    print(key, value)

For that reason, I'll be closing this issue. 🙏

@Kludex model_dump is not equivalent to _iter if there are fields with custom serialization. Here's an example for a bson.ObjectId field serialized as string.
import bson
from pydantic import BaseModel, Field
from pydantic_core import core_schema

class PydanticObjectId(bson.ObjectId):
    @classmethod
    def __get_pydantic_core_schema__(cls, source_type, handler):
        serialization = core_schema.plain_serializer_function_ser_schema(str)
        return core_schema.is_instance_schema(cls, serialization=serialization)

class Foo(BaseModel):
    id: PydanticObjectId = Field(default_factory=PydanticObjectId)
    b: int

foo = Foo(b=1)
print(f"str: {foo}")
print(f"_iter: {dict(foo._iter())}")
print(f"model_dump: {foo.model_dump()}")

Output:
str: id=ObjectId('64e3afb883f5e2aaed388877') b=1
_iter: {'id': ObjectId('64e3afb883f5e2aaed388877'), 'b': 1}
model_dump: {'id': '64e3afb883f5e2aaed388877', 'b': 1}

So unless there is a way to prevent custom serialization, I'd suggest to reopen this issue.

no longer really relevant in V2 - _iter has gone, replaced by the underlying methods in pydantic-core.

An example of how the previous example would work by using pydantic-core methods instead of _iter would help.

@Kludex model_dump is not equivalent to _iter if there are fields with custom serialization. Here's an example for a bson.ObjectId field serialized as string.
So unless there is a way to prevent custom serialization, I'd suggest to reopen this issue.

I think there is a problem with your PydanticObjectId definition. If I use the one from here, this is the output I get running your example:
str: id=ObjectId('655f16f55d40bae3a225723f') b=1
_iter: {'id': ObjectId('655f16f55d40bae3a225723f'), 'b': 1}
model_dump: {'id': ObjectId('655f16f55d40bae3a225723f'), 'b': 1}

So what was the final verdict? Can we use model_dump instead of _iter?


[5.1] [5.2] queue:listen and queue:work not working on windows

I can't figure out why this is happening. I'm am running redis-cli monitor to check for output, and while calling $this->dispatch(JOB) works (I can see the job being RPUSHed), neither running php artisan queue:listen or php artisan queue:work appears to be doing anything in redis (no change in the monitored output).
There is no output for php artisan queue:work before it terminates (or for queue:listen before I terminate it).
I've tried specifying the connection with php artisan queue:listen/work redis but this doesn't help either.
The above description was for redis, but using the database driver has the same issue. It's inserted in the database correctly, but is never processed (attempts is always 0).
I tried upgrading to 5.2 but with the same result. I used php artisan make:job ClassName --queued in 5.1 to create the command. In 5.2 I've removed SelfHandling from the class as well.
One other thing to note, it works fine when I set QUEUE_DRIVER=sync in .env. The queue used is default (i.e. I didn't specify a different queue).

I can't replicate. I use redis in production, and I'm pretty sure Taylor uses database, and we both have no issues.

Might be worth to note that I just tried it on my Debian 8 VPS (Laravel 5.1) which works, but still does not work on my Windows 10 computer. Is there anything which would prevent the queue from working depending on what OS the script is running from?

That's very likely. The code we have is OS specific. Ping @taylorotwell.

I've updated the description.

me too, not working in MAC as well, i use database driver, and the job is not processed... even after i run the command line, the job still there...

I have the same problem.

Also, same problem

same problem on CentOS 7 x64,job records already added to the jobs tables,but queue:listen and queue:work didn't make any output,and the records in jobs didn't change too.

Same problem. Jobs are added to the jobs table but they are never excecuted

Same problem. Half of our dev team is on windows.

Aynı sorunu bende yaşıyorum. Laravel 5.3 ile tabloya veri ekliyorum fakat o verileri queue:listen ile çalıştıramıyorum.

Any updates on this issue, i'm having this problem too. thanks.

@grepollo on Windows? Laravel 5.3?

Haven't tested on 5.3, I had issues on 5.2
On Thu, Oct 13, 2016 at 9:15 AM, Mohamed Said<EMAIL_ADDRESS>wrote:

@grepollo https://github.com/grepollo on Windows? Laravel 5.3?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/laravel/framework/issues/11714#issuecomment-253525757,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAI9MyQtr2S5H21Jc8BnP3Syx98D-hfLks5qzj0dgaJpZM4G_XWs
.

I just ran into the same issue, but I knew for sure it worked before... so I did some digging and found out that the queue:work command does not return ANY errors to the console. In my case I first did not have Redis running and after that there was a problem serializing a model, because a value was not available.
Both errors showed up on the laravel.log, but queue:work never told me anything.
Not sure if this helps anyone, but do check your laravel.log just in case it's crapping out somewhere.

I do not see errors on queue:work either. that's why I use : queue:listen, for me it shows exceptions, etc. but is all this connected? isn't this another issue @TesseractG ? /cc @themsaid

@huglester I am not sure if it is the same issue at all, I am just saying that I ran into similar symptoms and this is how they were fixed. Perhaps it helps other people.

Any idea why this is happening? I'm running into the same problem and database queueing...

Having the same problem, when i do queue:work or queue:listen nothing happens.
Laravel version : 5.3.23
Mysql : 5.6.31
OS: Windows 8.1 64bits

I get this in the log :
local.ERROR: Predis\Response\ServerException: ERR unknown command 'EVAL' in C:\PROJECTS\hypesquare\vendor\predis\predis\src\Client.php:370
windows 10 here :)

@zho - that is not a bug - you must put queue:work --queue:sendNotification

@lioannou I no longer have the problem with queue:work. But I must say that I went from database queuing to SQS.
But you're right, my co-worker had the same problem and doing queue:restart and specifying the queue like you said: queue:work --queue:sendNotification (for example), worked or so I'm told :)

I have same problem my jobs are inserted into job table but never execute by php artisan queue::work or php artisan queue::listen

Almost the same here, only on Homestead.
The listen method works, but using the work method causes the jobs not to be started anymore.

Can someone confirm the issue still exists in laravel 5.4?

@themsaid - for me - running Windows 10, on Homestead v1.0, using both database and redis drivers, it is working for me on Laravel 5.1 -> 5.4 (I've tried all versions).

Having same issue,
Laravel version=5.4
QUEUE_DRIVER=beanstalkd
OS = MacOS

You need to be specific about which queue(s) your worker is going to process.
The default queue is called default. If you run php artisan queue:work it will only process the default queue. Laravel does not magically know what queues you want to use. So, if for example, you dispatched a job on the emails queue, then make sure to pass --queue=emails,default to the worker/listener command.
Although I would agree that the documentation could be clearer on this topic.

I think you should be define queue name first then process to queue.
$job = (new ProcessPodcast($podcast))->onQueue('email');
Then check your code within constructor of job if every thing do well then you can add your queue code below in handle function.
Sometime if your code have any error then queue not working.

@hafeez1042 - can you please paste your queue config, and also what command you are using to run the queue?

I did it!
Working example:
artisan queue:work database --queue=emails
Required parameter: database and --queue

@themsaid - I feel this should/can be closed.
When you read many of the comments in this thread saying theirs is not working - many of them were incorrectly not applying the right queue name to be processed. I count at least 5 of them.
No one is able to provide any config that truly fails despite repeated requests. I've run lots of tests on Windows - and it works fine.
If it was a framework/Windows issue - we've had hundreds of people reporting it by now.

Having the same problem:
Laravel 5.4.23
Windows 10 Home
Redis 2.8.17
There is no output for php artisan queue:work, or for queue:listen
When using QUEUE_DRIVER=database everything works fine
.env
BROADCAST_DRIVER=redis
CACHE_DRIVER=file
SESSION_DRIVER=file
QUEUE_DRIVER=redis

REDIS_HOST=<IP_ADDRESS>
REDIS_PASSWORD=null
REDIS_PORT=6379

in redis queues->default
{
  "displayName": "App\\Jobs\\MailMessage",
  "job": "Illuminate\\Queue\\CallQueuedHandler@call",
  "maxTries": null,
  "timeout": null,
  "data": {
    "commandName": "App\\Jobs\\MailMessage",
...

@ursuleacv just to confirm, could you paste the exact command(s) which you run in redis-cli (or whatever you use to get that JSON you provided) to check what's in the queue?
Also, check if it works when you pass the queue name, and try to experiment with different queue names:
php artisan queue:work redis --queue=default

@mrliptontea I am using Redis Desktop Manager. I tried with the queue name and it's the same, still not working.

@ursuleacv only just now I have noticed Redis version which you use. Generally, Redis isn't officially supported on Windows, so where you got that actual build might be important.
I've just tried it on Redis 3.0.504 from MSOpenTech, and it all definitely works for me (Windows 10, Laravel 5.4.23).

Reading all comments in this thread I find that it's most probably an environment misconfiguration since many people report that it's working or got it to work themselves. For that I'm closing this issue since I don't see a point in keeping it since no laravel core changes would be applied.
I suggest that people visiting this thread later to read all the comments and make sure they applied all the suggestions.

@mirko77 No errors on these commands will mean that queues you listen to are empty/don't exist, and that is expected. So far nothing proved there is an actual issue with the framework, it's always misconfiguration/misunderstanding.
Please, quadruple check:

Queue driver you use.
Queue name(s) you dispatch jobs on.

If you still don't find the cause:

Trace what's being executed when you run queue:work (using a remote debugger or dd(); in the code).
Whether you can manually see jobs in your queue driver.

@mrliptontea the same exact code and commands run flawlessly on Ubuntu 16.04, just locally on a Mac High Sierra it does not.
While I could chase ghosts and witches for hours, I would just use sync when testing locally and live with that.

But that's the point; it's not the code that's at fault but your environment, dependencies and their configuration, which is why I suggested checking your queue driver.
Maybe try Docker. If you're trying Redis then literally all you need is docker run -p 6379:6379 redis.

@pmochine just like back in February, please check the following:

Trace what's being executed when you run queue:work (using a remote debugger or dd(); in the code).
Whether you can manually see jobs in your queue driver.

Honestly, I just tested this:

Install Laravel 5.2git clone https://github.com/laravel/laravel.git laravel52
cd laravel52
composer install
composer require predis/predis
cp .env.example .env
php artisan key:generate

Create a test jobphp artisan make:job TestQueue

Edit app/Console/Commands/Inspire.php - add inside handle():dispatch(new \App\Jobs\TestQueue);

Edit app/Jobs/TestQueue.php:- add inside handle():\Log::info('Hello from the queue');

Rundocker run --rm -p 6379:6379 -d redis
php artisan inspire
php artisan queue:work

Check the logtail storage/logs/laravel.log

Should show something like[2018-07-04 20:20:20] local.INFO: Hello from the queue

In my case php artisan queue:listen/work  not worked while site in maintance mode
https://laravel.com/docs/5.6/configuration#maintenance-mode

@CoderFish8 ... it was the same reason... stupid me

If you specified the connection or the queue when you add the job,you should run the listen or work command with connection and --queue,examples here:
php artisan queue:listen database --queue emails

This worked for me, Thanks!

@themsaid It's not a Bug, but there's something missing.  --queue parameters like optional but the default value is heavily assigned to 'default'.
https://github.com/laravel/framework/blob/5042846eb0bdd0ade2e760015b1e5c57013225d5/src/Illuminate/Queue/DatabaseQueue.php#L50
When people give this optional parameter they intentionally only want certain queues to run. At the same time, it is developing in order of priority. doc: https://laravel.com/docs/5.4/queues#queue-priorities
expected: if a priorty is not specified, first in first out
happening: run only default queue. Not exist default; people googling

laravel queue not working

Neded wildcard support and defalut queue name = "*"
php artisan queue:work --queue=super,high,*  1. super 2. hing 3. other...
php artisan queue:work  run All queue

The issue still exists on Windows in Laravel 5.8. Have no problems on MAC.

In my case php artisan queue:listen/work not worked while site in maintance mode
https://laravel.com/docs/5.6/configuration#maintenance-mode

Thank you @CoderFish8 that worked for me like magic after 5 hours of searching the problem was just in the maintenance mode

I have the same problem. Wildcard support is really needed. Good solution @Qh0stM4N. The current behavior is not obvious. I use a lot of queues and why specify the name of each queue?

In my case php artisan queue:listen/work not worked while site in maintance mode
https://laravel.com/docs/5.6/configuration#maintenance-mode

you saved my day, thanks, after more than 3 hours searching for.

same issue, no tips helped, also i am not in maintenance mode

I have the same problem. Wildcard support is really needed. Good solution @Qh0stM4N. The current behavior is not obvious. I use a lot of queues and why specify the name of each queue?

@M0nik Is there still the same problem in newest version of laravel?
If still same, maybe I can create a PR to end this.

php artisan optimize:clear on laravel 8 solved my issue

If you specified the connection or the queue when you add the job,you should run the listen or work command with connection and --queue,examples here: php artisan queue:listen database --queue emails

Literally saved my day.


Updated UWP Behaviors SDK (Managed) version to 2.0.0

Hi @bhavyau, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Bhavya Udayashankar). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Looks good.


Pip install datashader[examples] fails because cartopy requires numpy and cython at install time

It won't be possible to pip install datashader[examples] until https://github.com/SciTools/cartopy/issues/1035 is addressed. Ideally (from datashader's point of view) there would be wheels for cartopy. Alternatively, datashader could declare numpy and cython as install time dependencies when pip 10 is commonly available. For now, users installing via pip will have to do something like pip install numpy cython && pip install datashader[examples].

Actually, cartopy has install-time dependencies beyond those two. I'll come back to this later.

Ok, thanks.

This issue's now out of date (there's no datashader-examples package; the difficulties will instead move to (some topic of) pyviz...).


Fix unbuffered segments bug and naming issue

I think we should remove the cout statements. They don't help the user and are for internal debugging.

FYI: I've validated the fix on a couple of designs.

What units does MAX_TOLERANCE have?
There is no unit. It is a normalized value.
Why is the magic number 10 baked into your solution?
This seems like a kludge more than a fix, which is fine if it solves the flow for now but does not
look like a long term solution.
I also don't understand what the point of renaming the internal variable name is.
That is fine in development, but there is no way it fixes a bug, and only bug fixes belong in alpha2.1
This name change was requested by Prof. Kahng, so I am pushing for it...


Update production info in README

Minor updates to the README to improve the info about the current production setup.

In the Apache2 example configuration we have an external static directory.

In the Apache2 example configuration we have an external static directory.

In the Apache2 example configuration we have an external static directory.

But how will the static files end up there? I think it makes sense to add an explanatory step to the guide if there is something which has to be done.

In the Apache2 example configuration we have an external static directory.

But how will the static files end up there? I think it makes sense to add an explanatory step to the guide if there is something which has to be done.


Add Peifeng's thesis

Fix #116

Fix month.
Remive random line breaks in abstract.

Technically the degree conferral is in August. I don't know when the dissertation will be actually published, though.

Where do these get published btw? I thought it's done, when I approve it to be done as the chair.

They are collected in the Deep Blue library. I can find Juncheng's: https://deepblue.lib.umich.edu/handle/2027.42/169955, but not mine yet. I suppose it takes some time for it to appear.

Cool. I didn't know. Anyway, I'm merging. Congrats!


[How-To] remove devices trackee

Hi!
First off thanks for this 🙂
Is there a way to remove the tracked devices under the integration? I don't want to link my Livebox to other entities / devices (I have a second router).

Thanks!

``
First , you have to clean the history of connected devices in the live box
Second, you have to suppress the livebox in the integration and to recreate it.
The tracked devices are not suppressed as they are not connected, this is the aim of that: track the devices connected once to the livebox...

`` First , you have to clean the history of connected devices in the live box Second, you have to suppress the livebox in the integration and to recreate it.
The tracked devices are not suppressed as they are not connected, this is the aim of that: track the devices connected once to the livebox...

Hi,
Thanks for your help, I didn't know there was a history in Livebox.
Mersi bras et kenavo! ;)


Make changes to only run PR when target branch is main or release-0.3

@msajidmansoori12 @mguetta1 I'm not sure if there is relevant reason for keep testing release-0.3 when there are several newer releases including latest (not-yet released branch) 0.6, just created https://github.com/konveyor/go-konveyor-tests/issues/217 that might be refrected also in jenkins. Let's discuss relevant releases for testing there, thanks!

@mguetta1 made the changes to only run for main branch , please merge.


Remove all time.Sleep calls during tests

It slows down tests, nobody wants that.

Codecov Report

Merging #204 into master will decrease coverage by 0.03%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##           master     #204      +/-   ##
==========================================
- Coverage   84.52%   84.48%   -0.04%
==========================================
  Files          66       66
  Lines        5511     5511
==========================================
- Hits         4658     4656       -2
- Misses        722      724       +2
  Partials      131      131

Impacted Files
Coverage Δ

devices/ds248x/dev.go
7.24% <0%> (ø)
:arrow_up:

devices/tm1637/tm1637.go
87.8% <100%> (ø)
:arrow_up:

devices/bmxx80/bmp180.go
96.42% <100%> (ø)
:arrow_up:

devices/bmxx80/bmxx80.go
100% <100%> (ø)
:arrow_up:

devices/lepton/cci/cci.go
100% <100%> (ø)
:arrow_up:

devices/ds18b20/ds18b20.go
63.46% <50%> (ø)
:arrow_up:

devices/lepton/lepton.go
88.34% <0%> (-1.23%)
:arrow_down:

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 4294eaa...1b60036. Read the comment docs.

gohci


[BACKLOG-22182] Decouple checksum step from Kettle engine

@pentaho/x-wing Please review

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'engine,plugins/core/impl,plugins/core/ui' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine,plugins/core/impl,plugins/core/ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/resources/kettle-steps.xml
  engine/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSum.java
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSumData.java
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSumMeta.java
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_en_US.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_es_AR.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_fr_FR.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_it_IT.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ko_KR.properties
  plugins/core/impl/src/test/java/org/pentaho/di/trans/steps/checksum/CheckSumMetaTest.java
  plugins/core/impl/src/test/java/org/pentaho/di/trans/steps/checksum/CheckSumTest.java
  plugins/core/ui/src/main/java/org/pentaho/di/ui/trans/steps/checksum/CheckSumDialog.java
  plugins/core/ui/src/main/resources/CSM.png
  plugins/core/ui/src/main/resources/CSM.svg
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_en_US.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_es_AR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_fr_FR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_it_IT.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_ko_KR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_zh_CN.properties

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes.  They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'engine,plugins/core/impl,plugins/core/ui' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine,plugins/core/impl,plugins/core/ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/resources/kettle-steps.xml
  engine/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSum.java
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSumData.java
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSumMeta.java
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_en_US.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_es_AR.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_fr_FR.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_it_IT.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ko_KR.properties
  plugins/core/impl/src/test/java/org/pentaho/di/trans/steps/checksum/CheckSumMetaTest.java
  plugins/core/impl/src/test/java/org/pentaho/di/trans/steps/checksum/CheckSumTest.java
  plugins/core/ui/src/main/java/org/pentaho/di/ui/trans/steps/checksum/CheckSumDialog.java
  plugins/core/ui/src/main/resources/CSM.png
  plugins/core/ui/src/main/resources/CSM.svg
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_en_US.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_es_AR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_fr_FR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_it_IT.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_ko_KR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_zh_CN.properties

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
:new: org.pentaho.di.trans.steps.checksum.CheckSum

Branch Coverage: 69% :exclamation:
Complexity Coverage: 61.4% :exclamation:
Instruction Coverage: 75.8% :exclamation:
Line Coverage: 72% :exclamation:

:new: org.pentaho.di.trans.steps.checksum.CheckSumMeta

Branch Coverage: 38.5% :exclamation:
Complexity Coverage: 50.7% :exclamation:
Instruction Coverage: 60.6% :exclamation:
Line Coverage: 58.5% :exclamation:

org.pentaho.di.trans.step.StepInitThread

Branch Change: -25% :small_red_triangle_down:
Complexity Change: -8.3% :small_red_triangle_down:
Instruction Change: -15.2% :small_red_triangle_down:
Line Change: -6.4% :small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -2.1% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.4% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.4% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.pgbulkloader.PGBulkLoaderMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Branch Change: -2.6% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.systemdata.SystemDataMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -2% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:
Line Change: -0.8% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.Trans

Branch Change: -1.6% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -1.6% :small_red_triangle_down:
Line Change: -1.3% :small_red_triangle_down:
Method Change: -1.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/plugins/core/ui/src/main/java/org/pentaho/di/ui/trans/steps/checksum/CheckSumDialog.java

:heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 507 Column: 89

:heavy_exclamation_mark: ')' is not preceded with whitespace. at Line: 507 Column: 102

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl 'engine,plugins/core/impl,plugins/core/ui' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine,plugins/core/impl,plugins/core/ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/resources/kettle-steps.xml
  engine/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSum.java
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSumData.java
  plugins/core/impl/src/main/java/org/pentaho/di/trans/steps/checksum/CheckSumMeta.java
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_en_US.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_es_AR.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_fr_FR.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_it_IT.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/impl/src/main/resources/org/pentaho/di/trans/steps/checksum/messages/messages_ko_KR.properties
  plugins/core/impl/src/test/java/org/pentaho/di/trans/steps/checksum/CheckSumMetaTest.java
  plugins/core/impl/src/test/java/org/pentaho/di/trans/steps/checksum/CheckSumTest.java
  plugins/core/ui/src/main/java/org/pentaho/di/ui/trans/steps/checksum/CheckSumDialog.java
  plugins/core/ui/src/main/resources/CSM.png
  plugins/core/ui/src/main/resources/CSM.svg
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_en_US.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_es_AR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_fr_FR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_it_IT.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_ja_JP.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_ko_KR.properties
  plugins/core/ui/src/main/resources/org/pentaho/di/ui/trans/steps/checksum/messages/messages_zh_CN.properties

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes.


Remove stray backticks from doc comments

Description of the change
Just removed stray backticks that caused an empty code block to show up in the generated documentation.
Do I need to add an entry in the changelog for this? It seems unnecessary.

Checklist:

[ ] Added the change to the changelog's "Unreleased" section with a reference to this PR (e.g. "- Made change (#0000)")
[ ] Linked any existing issues or proposals that this pull request should close
[ ] Updated or added relevant documentation
[ ] Added a test for the contribution (if applicable)

No need for a changelog for this one — thanks!


Nice work! Few questions...

Can I use your method without providing a text prompt? My requirement is to simply segment different parts of a 3D object, without the need for labeling each segmented part. So, can I eliminate the requirement of passing any part names to the network? Additionally, do you think it's possible to achieve this without any fine-tuning, considering that the correspondence between part names and segments is not required?

Yes, I think using SAM's everything function is a good choice for your requirement. What you can do is to change the masks generated by GLIP+SAM to SAM everything output. However, the result is highly dependent on the granularity of SAM and I think it's somehow uncontrollable.


Efficient alternative to the update notifier

Currently, installing TSLint via npm also pulls the update-notifier module, which then pulls >65 other npm packages as its dependencies. This update-notifier module is only used in src/updateNotifier.ts for giving a reminder of an available update, arguably a non-essential feature. In the mean time, the most essential feature of TSLint requires only about a half dozen (dependent) packages.
It is far more efficient to grab https://registry.npmjs.org/tslint, JSON.parse it, and then check for the latest version.

Agree, I don't think we realized how many dependencies that package used

Actually it is not as easy as you think.
You need to read the .npmrc file to find out if the user has a custom registry.
You should also use a http library that can work with user's proxy settings.
I don't know which of the above is covered by update-notifier, but that's the minimum requirement to make it work for users inside big corporate networks.

IMHO such a feature does not need to be bundled with TSLint. Nowadays, there are already various services out there to help tracking updates: Greenkeeper, David, libraries.io, etc.

For a data point, for our usage in Google we disabled this dependency and patched the source to avoid pulling these in.

I agree with @ariya, I'm removing the dependency in #2262.


supports umanoPanelHeightOffset

this makes panel's height relative to SlidingUpPanelLayout's height, for a example, it SlidingUpPanelLayout is 600dp high, and umanoPanelHeightOffset is 200dp, then the panel is 400dp high.

Hi, any ideas about this PR?

I'm not sure I understand why this is need. Can you explain what you are trying to accomplish?

I'm using this when the panel height is not sure, but space for collapsed panel is set. Like attached picture above, if I slide up, the map will be hidden.


Refactor HandleQuotedLineTerminators trait

I implemented the handling of newlines that fall within quoted strings using a trait called CSVelte\Traits\HandleQuotedLineTerminators. While there is not necessarily anything wrong with that in and of itself, I had wanted to avoid providing my Input classes with too much knowledge of WHAT they are reading. There may just not be any other way. My initial idea was to implement either a stream wrapper that buffered data being read from a CSV source and encoded special characters such as newlines, delimiters, etc. I also tried implementing something similar using PHP's stream filter functions. Neither one of those solutions really panned out though, unfortunately and I ended up going with this trait solution. It works for now, but eventually I'd like to come back to this and see if I can find a solution I feel better about.

Take a look at this C code that reads CSV data. Might help I dont know I didnt look that hard at it.
https://dev.w3.org/cvsweb/csvtotab-vv/csvtotab.c?rev=1.1;content-type=text%2Fx-cvsweb-markup

[ ] Move readLine() method's first argument to the end of the argument list and assign null as its default


Simplify id space management

[ch15456]

I think we should remove the consensus module from the repo if it's essentially dead code now.

@dominiklohmann do you mind taking another look at the latest commit. (The rest is only rebase.)

Looks fine from a taking a look at the code only. I'll check this out and run some tests locall.

I think we should revert #906 as part of this PR. It's useful for monitoring whether this change works.

I'd prefer not to, as the algorithm to build the JSON value is still extremely inefficient. (Will give problems with large archives regardless.)

I've noticed a crash when setting table-slice-size to 0, which should probably get a nicer-looking error message instead.

Will do.


add to resources

https://hutchdatascience.org/data_snacks/r_snacks/sas2r.html

possibly this: https://hutchdatascience.org/data_snacks/r_snacks/janitor.html


Revert "PL-releasenotes-782xx" (merged too early by mistake, before prod deployment time)

Reverts harness/developer-hub#367

Preview environment: https://ci.pr.harness.io/pr-370


cuda out of memory error while reproducing the result

i am getting the following error while reproducing the result.
CUDA out of memory. Tried to allocate 96.00 MiB (GPU 0; 10.76 GiB total capacity; 8.86 GiB already allocated; 64.94 MiB free; 8.99 GiB reserved in total by PyTorch).
raise subprocess.CalledProcessError(returncode=last_return_code, cmd=cmd)
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
I have tried to reduce the batch size, image resolution and cleared pytorch cache as well; still the issue exists. Any suggestion on this would be a great help.

Hi, what's your hardware configuration, and which script are you running?

I am running the script on Ubuntu 18.04 machine with 2 Nvidia rtx 2080 graphic cards. The 1st stage of training goes well. I am facing issue at the 2nd stage of training while running the command bash script/train/a2_train_stage2_res50.sh  res50_stage2 . Tried setting the OMP_NUM_THREADS to values other than the default value. Still the issue persists.

The 2nd stage of the LETR training requires more GPU memory. To reduce memory usage, you are welcome to try to replace self-attention with efficient attention (e.g. our Conv-Attention module in CoaT https://github.com/mlpc-ucsd/CoaT).

how much memory should be enough for training the network ?


Fix gui structure restoring breaking tabs and tags

Uh, yea. My bad. I can also see exaclty how i missed that.
Anyway, fixed!

Heh, oops.

Almost like someone deliberately used these names as trap. ;)


Add a readable log and expose it to subsequent activities

Boban @bobanco 00:01

@creyke wouldn't be easier to implement the saga on a top of EventSourced grain?

Roger Creyke @creyke 00:02

if the saga is a "true" saga, the result of the work is inconsequential
for me, that would be "adding a log"
which is sure, optional
and probably a good feature

Not required currently.


Fonts are blurred in both editor and pdf preview

Issue Type: Bug
Both the pdf preview and the main editor fonts appear blurred and difficult to read.

Similarly to other bug reports such as #97682 and #85154, there's a gradient of blurriness in the pdf preview tab. Conversely to these cases, the blur is way more evident while moving away from the center of the preview were the text is somewhat readable.

At the same time the fonts in the classical editor view seem to be affected by a constant blur effect (no gradient in this case) that makes the editor really hard to use after some time.

The terminal, once set to dom renderer, doesn't seem to have any blur gradient like reported in the above issues. However (i might be wrong) it doesn't seem as "clear" as previous VSCode versions.

The issue happens no matter having tested the following:

interface zoom level to 0 or -1 (any zoom level doesn't seem to fix the issue);
--disable-gpu;
--disable-extensions;
even with a new config profile

code-insiders (1.56) also seems to be affected.
And here two screenshots:

--> Especially in the case of the second screenshot, notice how the text like All the defense mechanisms seems way more readable than the rest (e.g. In light of the above, from our point of view some lines after). It's like the the more you move from the center of the webview the more the blur is evident.
Here's the data of the normal version:
Version: 1.55.2
Commit: 3c4e3df9e89829dce27b7b5c24508306b151f30d
Date: 2021-04-13T09:37:02.931Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.11.12-1-default

VS Code version: Code 1.55.2 (3c4e3df9e89829dce27b7b5c24508306b151f30d, 2021-04-13T09:37:02.931Z)
OS version: Linux x64 5.11.12-1-default

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5200U CPU @ 2.20GHz (4 x 2700)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
1, 1, 1

Memory (System)
7.67GB (0.42GB free)

Process Argv
--disable-gpu --no-sandbox --crash-reporter-id 97fbb8d5-1730-4015-988a-ca7eca3d4640

Screen Reader
no

VM
0%

DESKTOP_SESSION
/usr/share/xsessions/plasma5

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

Extensions (26)

Extension
Author (truncated)
Version

QML
bbe
1.0.0

bracket-pair-colorizer-2
Coe
0.2.0

doxdocgen
csc
1.2.0

arm
dan
1.5.0

githistory
don
0.6.16

todo-tree
Gru
0.0.211

asciidecorator
hel
0.2.0

output-colorizer
IBM
0.1.2

latex-workshop
Jam
8.16.1

cortex-debug
mar
0.3.12

python
ms-
2021.3.680753044

vscode-pylance
ms-
2021.4.1

jupyter
ms-
2021.5.745244803

cpptools
ms-
1.3.0

veriloghdl
msh
1.4.2

subtitles-editor
pep
1.1.4

material-icon-theme
PKi
4.6.0

tcl
ras
0.1.0

vscode-modern-vhdl-support
rjy
1.0.5

code-settings-sync
Sha
3.4.3

rewrap
stk
1.14.0

hdl-checker-client
suo
1.1.1

vhdl-formatter
Vin
1.0.5

markdown-all-in-one
yzh
3.4.0

riscv
zhw
0.0.8

linkerscript
Zix
1.0.1

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspyt653:30270858
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vstry244:30276681
pythonvsdeb440:30248342
pythonvsded773:30248341
pythonvspyt875:30259475
pythontb:30283811
openwslfolder:30282073
vspre833:30267464
pythonptprofiler:30281270
vshan820cf:30276953
vscorecescf:30290706
pythondataviewer:30285071
vscus158:30286553

Issue Type: Bug
Ok I tested an older version which, IIRC, did not had this problem: 1.52
(I was on Manjaro KDE just a week ago however, I really hope it's not distro dependent)
Now I am running OpenSUSE an both versions (1.55 and 1.52) were installed from Microsoft Official Repository for OpenSUSE
The problem with the pdf preview seems to persist with the exact same behavior.
However, I noticed this: if I reset the zoom level the editor text seems on point and readable with no issues, no matter how text zoom I apply to it (ctrl+mouse_wheel). The pdf preview problem seems to disappear only if I set a 130% zoom. See the following screenshot:

However, such an interface zoom level is quite unusable, so as the pdf preview zoom.
If I set the interface zoom level to -1 (the value I normally use) while keeping the pdf preview @ 130%, the preview seems maybe a little bit off but still way more readable than before (I would say the problem seems to be solved). See the folliwing screenshot:

Again, if I move from this state changing the preview zoom (say, from 130% to 110%) then the non-constant blurriness appears again, see next screenshot:

Finally, if I move from this state back to a 130% zoom level of the preview, one could think the problem to disappear again but no, the blurriness persists, see next screenshot.

So in summary:

1.56: readability issue with both the editor text and the pdf preview text
1.55: readability issue with both the editor text and the pdf preview text
For both the above: the problem persists at any interface zoom level and preview zoom level, no particular combination has been found to solve the problem

1.52: readability issue only with the preview text, the editor seems ok
1:52: Setting the interface zoom level to 0 and preview zoom to 130% solves the problem.
1:52: From the above state, setting the interface to -1 while keeping 130% on the preview seems to leas to a correct visualization
1.52: From the above state, as soon as I change the preview zoom, the readability is lost and is regained only by jumping again to point 2)

VS Code version: Code 1.52.1 (ea3859d4ba2f3e577a159bc91e3074c5d85c0523, 2020-12-16T16:32:10.090Z)
OS version: Linux x64 5.11.12-1-default

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-5200U CPU @ 2.20GHz (4 x 2494)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: unavailable_offvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
7.67GB (0.91GB free)

Process Argv
--no-sandbox --unity-launch --enable-gpu /home/sem/Syncthing/Politecnico di Torino/01 - Magistrale/Tesi/Thesis --crash-reporter-id 97fbb8d5-1730-4015-988a-ca7eca3d4640

Screen Reader
no

VM
0%

DESKTOP_SESSION
/usr/share/xsessions/plasma5

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
KDE

XDG_SESSION_TYPE
x11

Extensions (26)

Extension
Author (truncated)
Version

QML
bbe
1.0.0

bracket-pair-colorizer-2
Coe
0.2.0

doxdocgen
csc
1.2.0

arm
dan
1.5.0

githistory
don
0.6.16

todo-tree
Gru
0.0.211

asciidecorator
hel
0.2.0

output-colorizer
IBM
0.1.2

latex-workshop
Jam
8.16.1

cortex-debug
mar
0.3.12

python
ms-
2021.3.680753044

vscode-pylance
ms-
2021.4.1

jupyter
ms-
2021.5.745244803

cpptools
ms-
1.3.0

veriloghdl
msh
1.4.2

subtitles-editor
pep
1.1.4

material-icon-theme
PKi
4.6.0

tcl
ras
0.1.0

vscode-modern-vhdl-support
rjy
1.0.5

code-settings-sync
Sha
3.4.3

rewrap
stk
1.14.0

hdl-checker-client
suo
1.1.1

vhdl-formatter
Vin
1.0.5

markdown-all-in-one
yzh
3.4.0

riscv
zhw
0.0.8

linkerscript
Zix
1.0.1

(1 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspyt653:30270858
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492cf:30256860
vstry244:30276681
pythonvsdeb440:30248342
pythonvsded773:30248341
pythonvspyt875:30259475
pythontb:30283811
openwslfolder:30282073
vspre833:30267464
pythonptprofiler:30281270
vshan820cf:30276953
vscorecescf:30290706
pythondataviewer:30285071
vscus158:30286553

I am a contributor to the PDF viewer. https://github.com/James-Yu/LaTeX-Workshop
Notice that you have to reload the PDF viewer after changing the zoom level 0 of VS Code. Please call Developer: Reload Window or Developer: Reload Webviews.

Oh I see! And in fact, with 1.52 zoom level 0 reload webview is sufficient to solve the issue. However, I have to do that every time I change the zoom of the preview (say, from 130% to 110% or more).
Sadly, with an interface at -1 reloading the webview or the window does not solve the problem as the text is blurred at any zoom level. It seems to be effective only when the interface zoom is at 0

I had this problem pop up after updating to Windows 11. In my case the editor appears fine, but there is a blur in the pdf preview, and there is a quite noticeable gradient (notice the text is much clearer near the center of the page).

Versions:

Code:                         1.68.1      30d9c6cd9483b2cc586687151bcbcd635f373630
OS Name:                  Microsoft Windows 11 Home
OS Version:                10.0.22000 N/A Build 22000

Notable settings:

VS Code zoom level: problem persists across various settings
PDF preview zoom: page width, notably the problem disappears after zooming to very high levels (170+%)
Windows display scaling: 125%

Is the editor font issue still present with latest stable 1.73 ?

I can confirm the pdf preview is still blurry for me as of 1.74
Version: 1.74.0 (user setup)
Commit: 5235c6bb189b60b01b1f49062f4ffa42384f8c91
Date: 2022-12-05T16:38:16.075Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: No

For pdf viewer, please file an issue with the extension https://github.com/James-Yu/LaTeX-Workshop.


implement using LOCAL STORAGE

local storage is used in the omdb.js file in order to filter out movies that have been marked as watched


thank you / hint for http 400 response on windows

thank you for your work. I am using this bot now in my prometheus installation.
I had to change the deserialization of alerts for me. Though it worked it always hat an 400 http error set - and at least with my windows environment the subsequent "c.AbortWithStatus(http.StatusOK)" would not work - so alertmanager would get an 400 back und retry the message endlessly.
So I changed:
		var alerts Alerts
		//		c.BindJSON(&alerts)
		binding.JSON.Bind(c.Request, &alerts)

@man-at-home sorry for late response, I just got back from my vacation. Thanks for this note, I'll check http codes.

I have the same problem.
Alarm manager Log, taked from journactl of my server.
Oct 07 14:23:38 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:23:38Z" level=warning msg="Notify attempt 2 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:23:39 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:23:39Z" level=warning msg="Notify attempt 3 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:23:40 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:23:40Z" level=warning msg="Notify attempt 4 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:23:42 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:23:42Z" level=warning msg="Notify attempt 5 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:23:44 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:23:44Z" level=warning msg="Notify attempt 6 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:23:49 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:23:49Z" level=warning msg="Notify attempt 7 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:23:52 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:23:52Z" level=warning msg="Notify attempt 8 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:23:58 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:23:58Z" level=warning msg="Notify attempt 9 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:24:05 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:24:05Z" level=warning msg="Notify attempt 10 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193" Oct 07 14:24:21 myMachineName.mydomain alertmanager[18803]: time="2016-10-07T14:24:21Z" level=warning msg="Notify attempt 11 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-mycharID" source="notify.go:193"
LOG of bot telegram:
Alert: {"alerts":[{"annotations":{"description":"<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)","summary":"Istance <IP_ADDRESS>"},"sendsAt":"","generatorURL":"http://mylocalmachinedomain:9090/graph#%5B%7B%22expr%22%3A%22raspb_temperatura%20%3E%2050%22%2C%22tab%22%3A0%7D%5D","labels":{"alertname":"Temperature","instance":"<IP_ADDRESS>","job":"StatsD","severity":"Critical"},"startsAt":"2016-10-07T14:22:54.803Z"}],"commonAnnotations":{"description":"<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)","summary":"Istance <IP_ADDRESS>"},"commonLabels":{"alertname":"Temperature","instance":"<IP_ADDRESS>","job":"StatsD","severity":"Critical"},"externalURL":"http://alert.greco.cf/alert-manager","groupKey":946614883222831012,"groupLabels":{"alertname":"Temperature"},"receiver":"Telegram","status":"firing","version":0} message: %!(EXTRA string=[FIRING:1] grouped by: alertname=
Temperature
 labels: job=
StatsD
, severity=
Critical
, instance=
<IP_ADDRESS>
 description:
<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)
 summary:
Istance <IP_ADDRESS>
 <IP_ADDRESS>[StatsD]) [GIN] 2016/10/07 - 14:23:37 | 200 |  119.322156ms | <IP_ADDRESS> |   POST    /alert/-154461500 Alert: {"alerts":[{"annotations":{"description":"<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)","summary":"Istance <IP_ADDRESS>"},"sendsAt":"","generatorURL":"http://mylocalmachinedomain:9090/graph#%5B%7B%22expr%22%3A%22raspb_temperatura%20%3E%2050%22%2C%22tab%22%3A0%7D%5D","labels":{"alertname":"Temperature","instance":"<IP_ADDRESS>","job":"StatsD","severity":"Critical"},"startsAt":"2016-10-07T14:22:54.803Z"}],"commonAnnotations":{"description":"<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)","summary":"Istance <IP_ADDRESS>"},"commonLabels":{"alertname":"Temperature","instance":"<IP_ADDRESS>","job":"StatsD","severity":"Critical"},"externalURL":"http://alert.greco.cf/alert-manager","groupKey":946614883222831012,"groupLabels":{"alertname":"Temperature"},"receiver":"Telegram","status":"firing","version":0} message: %!(EXTRA string=[FIRING:1] grouped by: alertname=
Temperature
 labels: instance=
<IP_ADDRESS>
, job=
StatsD
, severity=
Critical
 description:
<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)
 summary:
Istance <IP_ADDRESS>

I also disable gin DEBUG option. via export variable.
Here i see many retry of Alertmanager, i think that bug stay in this program because i don;t find anythink big issues, in main repo of Prometheus, alert manager, and man-at-home have the same issues.
I try his code but go compiler return error.
Is there some fix for this?

@AndreaGreco I didn't have time to check @man-at-home's code, what error did you get?

Tanks for reply,
I try to explain better:
I think that this is chain:
alertmanager, send alert to telegram_bot, telegram_bot recive alert and send telegram messages, i recive message in Telegram Chat, telegram_bot, return 400 to alertmanager.
alertmanager return error in log, and retry send messagges.
Result:
Telegram chat receive 100.000 messages, alert result not send.
This is log better formatter sorry last time was terrible formatted:
host_name alertmanager: time="..." level=error msg="Error on notify: context deadline exceeded" source="notify.go:152"
host_name alertmanager: time="..." level=error msg="Notify for 1 alerts failed: context deadline exceeded" source="dispatch.go:238"
host_name alertmanager: time="..." level=warning msg="Notify attempt 1 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 2 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 3 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 4 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 5 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 6 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 7 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 8 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 9 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 10 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 11 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 12 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 13 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 14 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"
host_name alertmanager: time="..." level=warning msg="Notify attempt 15 failed: unexpected status code 400 from http://<IP_ADDRESS>:9087/alert/-154461500" source="notify.go:193"

Telegram_bot in log return 200, but alertmanager get 400 maybe problem is GIN take look of this bug in GIN: #133, In telegram bot is there the same log.
I report here, my log, from telegram bot:
Oct 10 10:50:09 host_name prometheus_bot[31505]: Alert: {"alerts":[{"annotations":{"description":"<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)","summary":"Istance <IP_ADDRESS>"},"sendsAt":"","generatorURL":"http://host_name:9090/graph#%5B%7B%22expr%22%3A%22raspb_temperatura%20%3E%2050%22%2C%22tab%22%3A0%7D%5D","labels":{"alertname":"Temperature","instance":"<IP_ADDRESS>","job":"StatsD","severity":"Critical"},"startsAt":"2016-10-07T14:22:54.803Z"}],"commonAnnotations":{"description":"<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)","summary":"Istance <IP_ADDRESS>"},"commonLabels":{"alertname":"Temperature","instance":"<IP_ADDRESS>","job":"StatsD","severity":"Critical"},"externalURL":"http://alert.host_name.cf/alert-manager","groupKey":946614883222831012,"groupLabels":{"alertname":"Temperature"},"receiver":"Telegram","status":"firing","version":0}
Oct 10 10:50:09 host_name prometheus_bot[31505]: message: %!(EXTRA string=[FIRING:1]
Oct 10 10:50:09 host_name prometheus_bot[31505]: grouped by: alertname=
Temperature

Oct 10 10:50:09 host_name prometheus_bot[31505]: labels: job=
StatsD
, severity=
Critical
, instance=
<IP_ADDRESS>

Oct 10 10:50:09 host_name prometheus_bot[31505]: description:
<IP_ADDRESS> Temperature of CPU : (current value:<PHONE_NUMBER>67°C)

Oct 10 10:50:09 host_name prometheus_bot[31505]: summary:
Istance <IP_ADDRESS>

Oct 10 10:50:09 host_name prometheus_bot[31505]: <IP_ADDRESS>[StatsD])
Oct 10 10:50:09 host_name prometheus_bot[31505]: [GIN] 2016/10/10 - 10:50:09 | 200 |  158.262881ms | <IP_ADDRESS> |   POST    /alert/-154461500
Oct 10 10:50:09 host_name prometheus_bot[31505]: [GIN-debug] [WARNING] Headers were already written. Wanted to override status code 400 with 200

Thank you for help
Andrea

Hi,
yes this is the behavior I had too - the bot tries to set the return code to 200 at the end, that does not work and the 400 code set by c.BindJSON(&alerts) will be returned instead. Alertmanager will keep alerting on 400 response again and again.
I hacked the fix in 2 lines (avoided c.BindJSON() so I have the bot working on my installation), but the fix is kind of ugly, so I did not submit this as a patch.

@man-at-home I try copy past your code it not compile, but i don't know golang then i miss somethink.

I am at work now, I try to put it into a small patch after work, but it should be one line change only.

@man-at-home Return that is simbol is not defined:
# command-line-arguments
./main.go:106: undefined: binding in binding.JSON

Yes.  add below line ""github.com/gin-gonic/gin" (line 7)
"github.com/gin-gonic/gin/binding"

@man-at-home why don't you fire a PR?

I'm gonna testing my configuration of Prometheus, but has just stop to send 100.000 messages, over Telegram.
We waiting for your PR.
Thank you all for help.
Andrea

ok, try https://github.com/inCaller/prometheus_bot/pull/2 , hope it helps.

Closing.


fixed width & height of video when it is in fullscreen mode

When video is at a fullscreen mode at demo page. The video is not actually in fullscreen mode. Because the width the height passed in from the props. So that I changed width from 480 x 270 => 100% x 100% and wrapped outside a div (480x270). Now it is worked fine.

@nndung179 excellent. yeah was going to suggest this too


Issue 3: Array of objects?

point 3 of my old todolist:
https://code.google.com/p/infectorpp/source/browse/trunk/todo.txt

Construct arrays of objects? (it's implicit that "Single" types can't be
costructed into arrays, so there will not be "buildSingleArray" in any case).
auto myArray = ioc.buildArray(50);
for(int i=0; i<50; i++)
myArray[i]; //would that be usefull? would that be misused?

I see no reason for that living in C++ world (even if it would make it very Ninject-like). Actually the few cases in wich that would be usefull are more clear if done properly (using factories, creating instances inside a class).
If you can show one case for wich that is usefull to you I'll add the feature.

Definitely not, it could be easily workarounded with ad hoc class, implementing that would cause some performance hit and increase in complexity of code for an edge case.


Consolidate canShare and canEdit in the getFullContentProfile method

There's some room for optimisation here as currently the roles are being fetched twice.

Can you also submit a PR to oae-rest for the editors changes to createCollabDoc ?

I couldn't see a way to cleanly fetch the roles only once without a major refactor, this commit does reduce it to a single call when the user is a manager of the content though.

Coverage decreased (-0.03%) to 90.1% when pulling 442f194fb3af07b6604ca745a99062fbc7616570 on stuartf:editorRole into cdb29f7b120a03ee4919c040f4cba2f3885375dd on oaeproject:master.

If relevant, the pattern of resolveEffectiveContentAccess similar to those of Discussions and Folders could be useful here: https://github.com/oaeproject/Hilary/blob/master/node_modules/oae-folders/lib/authz.js#L271-L301


tweaks to stata

Two tweaks -- on installation, realized (a) stata not in path by default (so added to readme) and stata now using stata as default command (not xstata-se, as on my system). Sorry for late additions -- didn't realize till installed updated version!

Shipped in 3.8.0 (and 3.8.1) :ship:


add resume at the end of the execution

test this please

test this please

test this please

test this please

test this please

/retest

/retest


verilog: unstable-2019-08-01 -> unstable-2020-08-24

The build would previously fail due to a change in bison's behavior (see
https://github.com/steveicarus/iverilog/commit/5b699c1be73e789831db01e779a41478c0c62309
for more information). Updating to a more recent version fixes this
issue.

Motivation for this change
ZHF: #97479
Saw this was failing when looking for packages to fix for ZHF, and realized I need it for one of my classes this semester... Anyways, I tested this by compiling the first assignment (an 8-to-3 decoder) and verifying the output was the same. Test source can be found here: https://gist.github.com/cole-h/d499079a8ea0c58a80f415e9b5e2bafa.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I'll backport this to 20.09 in the morning.
@NixOS/nixos-release-managers

Also fixes #97110

Yosys works with this iverilog version, compiled some litex with it, without issues


doc(kic) remove CREATE from Secret hook

Summary
Remove CREATE Secret from the admission webhook watched actions. Due to
changes in implementation, this is no longer used. We only check Secret
UPDATEs.
Reason
This is tied to the changes in https://github.com/Kong/kubernetes-ingress-controller/pull/2015. It should not be merged until that is actually released (not just merged). This will remain a draft until we're ready to release.

Superseded by https://github.com/Kong/docs.konghq.com/pull/3518


[C#]:ERROR DotNetAstGenRunner:  - running dotnetastgen failed!

"When I use Joern to traverse a web form project in C#, the console prompts the following issue. How can I resolve it?"

@tuxology Are you able to reproduce something like this? It seems like it appears on Windows

@chengkenyong Please let us know what project you tried to import or how we can reproduce this, as well as which version of Joern you're using? C# is currently in an alpha state

sample project:
https://github.com/chengkenyong/web-Application
Okay, I just simulated a simple ASP.Net Web Form project, Upon examining the log information, it seems that all files ending with ". cs" have been parsed, but only four nodes have been generated.

Desktop (please complete the following information):
Win 11
Joern 2.0.306


updated deets

Added my project section which will list out the apps i am working on in the neog camp.

PR looks good. Proceed with merge.


Sketch Palettes not importing on OS Catalina 10.15.7 and Sketch v69

I have a brand new install of Sketch and this plugin on a brand new Mac Pro.  I've had success exporting/importing in the past - specifically success importing the file i'm trying to import now.  With this setup, when I point to a file, I get the 'Load Palette' dialog to define the source.  I've tried all options in the dialog.  End result is no imported palette.

I would point to Sketch v69 being the issue. Worked flawlessly on v68. After updating I experience the same said behaviour, tho on Mojave.

Update:
Seems Sketch now wants us to use their libraries feature. Saving all into colours into one library and including that helped. Not sure I'm happy about this.

I have the same issue, getting this error in the plugin log:
TypeError: app.refreshCurrentDocument is not a function. (In 'app.refreshCurrentDocument()', 'app.refreshCurrentDocument' is undefined)     at loadPalette (~.../Plugins/Sketch Palettes.sketchplugin/Contents/Sketch/sketchPalettes.js:412:28) No WebView present. Event cannot be posted to the web UI.

Please Help! Same issue i had tons of favourite colors collected in years!!! This is a disaster!!!

Same :( Issue with Sketch 69

And Sketch 70.

Anxiously awaiting a fix for latest versions of sketch 🎨👀
Sketch Color Variables really took away from this awesome plug-in.

I took a look at the code and the new functionality. Tech wise it has changed considerably, and it wasn't clear how to access colors now without majorly reworking the plugin. If that is necessary it can be done, but it will take time. But in addition to the tech aspect, there is the question of functionally it should work alongside color variables. Also, only colors has moved to variables, but images and gradient fills still remain as global and document presets. How would you want or expect saving or loading palettes to interact with variable functionality?

There's a workaround.  Create a sketch file with all of your swatches.
Save the sketch file as a template.  Then navigate to Sketch's preferences
and import and enable the template.
On Wed, Nov 25, 2020 at 10:56 AM Andrew Fiorillo<EMAIL_ADDRESS>wrote:

I took a look at the code and the new functionality. Tech wise it has
changed considerably, and it wasn't clear how to access colors now without
majorly reworking the plugin. If that is necessary it can be done, but it
will take time. But in addition to the tech aspect, there is the question
of functionally it should work alongside color variables. Also, only colors
has moved to variables, but images and gradient fills still remain as
global and document presets. How would you want or expect saving or loading
palettes to interact with variable functionality?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/andrewfiorillo/sketch-palettes/issues/93#issuecomment-733793460,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACE22AN7BWCFQA2RW7O5WM3SRUSKZANCNFSM4SH3LMCQ
.

Having the same problem, is a fix planned / possible for this soon?


Syntax style tweaks

denote bool literals in JSON.
YAML is now green like JSON.  The lexer does not distinguish keys and values,
so we can't even color them differently.

self-merging to make progress on other experiments


Adding Apache v2 license

The code is currently unlicensed, which leaves in question how it can be used. This branch adds an OSI-approved Apache v2 license.
https://opensource.org/licenses

Thank you @tmajor86!


chore(package): update rollup to version 1.27.14

Coverage remained the same at 87.261% when pulling ea9c4003032705e3f4914f6d1c4e35ed220f3b53 on greenkeeper/rollup-1.27.14 into 071eb19a306f8c22516c0f33a8e9bbbb28480a3e on master.


Update project page

Describe the changes you've made
The current projects page's header does not match the blog and contributors page ui.. so i just remove the blue background

##my change⬇

I upgrade the repo cards
added fork counter and open issues. and added topic labels

Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update.
[ ] This change requires a documentation update

How Has This Been Tested?
tested by localhost
Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)
|      Original       |      Updated       |
|

|

|
Code of Conduct

[x] I agree to follow this project's Code of Conduct

LGTM!!!! Love it ❤

Merging, thank you for your contribution :)


Theme: Meeting Developer Expectations

.NET 5 introduced more incremental improvements for applications to realize faster startup times and smaller exe sizes using single file and app trimming. As we look ahead on how we continue to make .NET a compelling platform for developers to build their modern apps on, we will continue to evolve the platform by building on top of what .NET 5 has brought to the ecosystem.
Epics under this theme:

[x] https://github.com/dotnet/runtime/issues/43538

[x] https://github.com/dotnet/runtimelab/issues/248
[x] https://github.com/dotnet/runtime/issues/43543

[x] #5395
[x] https://github.com/dotnet/machinelearning-modelbuilder/issues/1053
[ ] Enable APIs to be discoverable and integrated in the Power Platform

[x] #5367

[x] https://github.com/dotnet/runtime/issues/44314
[x] https://github.com/dotnet/runtime/issues/43767
[x] https://github.com/dotnet/runtime/issues/43545
[x] https://github.com/dotnet/runtime/issues/43616
[x] https://github.com/dotnet/runtime/issues/43620
[ ] https://github.com/dotnet/efcore/issues/24107
[x] https://github.com/dotnet/runtime/issues/48253

Bulk closing .NET 6 epics and user stories. If you think this issue was closed in error, please reopen the issue and update it accordingly.


Feature request: Default exports in individual files

What is the new or updated feature that you are suggesting?
It would be useful if hooks were provided as default exports when importing directly from a given file.
For example, currently to directly import useUnmountEffect I have to use a statement like:
import { useUnmountEffect } from '@react-hookz/web/cjs/useUnmountEffect/useUnmountEffect';

It would help if I could also use an import like this:
import useUnmountEffect from '@react-hookz/web/cjs/useUnmountEffect/useUnmountEffect';

Why should this feature be included?
There's various build tooling, intended for cases like this library, Lodash, etc, that can rewrite imports like this...
import {
  thingA,
  thingB,
  thingC
} from 'some-library'

to look like this at build time, using string matching.
import thingA from 'some-library/thingA'
import thingB from 'some-library/thingB'
import thingC from 'some-library/thingA'

This tooling generally assumes that any individual exports will be the default export in the specific file, since that's the pattern followed by major libraries like Lodash. Having default exports would allow using that tooling.

I would be able to set this up if the general consensus agrees with the feature. Not a lot of discussion on it, so not sure if there is a reason it hasn't been done yet.

aaabout this one. it is kinda janky theme to start, but here we are.

adding default export will increas overall package size for @react-hookz/web
the fact lodash uses it does not mean it is coreect and\or the proper way
i cant really see whyimport {  thingA,  thingB, thingC } from 'some-library';

considered worse thanimport thingA from 'some-library/thingA';
import thingB from 'some-library/thingB';
import thingC from 'some-library/thingA';

Second approach will lead to larger bundle than first one (even after munging)


README.md Update

Changes proposed
Installation Steps

Clone the project git clone <url>

This change only demonstrates cloning with https. I wonder if instead a screenshot would be better of where to find the git clone instruction under the code button?


multiple datasets

hello! I would like to define my own buckets, without ar_buckets, because I have different widths and heights that I would like to preserve, would it be possible to do multiple datasets similiar how to multiple eval_datasets are possible?

I'm not sure exactly what you're trying to do. Use some specific ar_buckets instead of the range-style specification? Not currently supported but easy to add. Just have different ar_buckets for different datasets? You should be able to set num_ar_buckets, min_ar, and max_ar per directory, i.e.
[[directory]]
path = '/home/anon/data/images/something'
min_ar = 0.5
max_ar = 2.0
num_ar_buckets = 9

If these fields are specified at the directory level they will override whatever is at the top level. Does this do what you want?

Thank you for the reply! I suggested multiple datasets because that appeared simpler to me than functionality for specifying buckets.
yes, what I needed ideally is not ar_buckets as they currently are but completely self-specified ar_buckets.
for example, I had training data of 520x520x33, 520x320x33, 320x520x33, but using ar_buckets sets them to the pixel resolution of 512x512, even though I made the resolution and time-step as I already needed them.
my option then was either scale the 512x512 samples down to the same pixel-resolution as the other videos (which would involve figuring out what I would have to set in the ar_bucket value to properly have the 520x320 resolution) or accept that the 520x320 and 320x520 clips got scaled to 640x384 and 384x640
I like automated bucketing functionality, but personally I always make sure to have training data and buckets as I need them for balancing reasons and knowing on what I've trained exactly.

I pushed commit 3cd447dbd5f223c875e6dd8edf437aafaec37664 which I think supports your use case. You can directly specify the aspect ratios and resolutions using (width, height) pairs, and do so per-directory. For example:
[[directory]]
path = '/home/anon/data/something'
ar_buckets = [[512, 320]]
resolutions = [[512, 320]]
frame_buckets = [33]

Note that for these settings and all AR calculations, it's always width first in the pair (the opposite of how the tensor dimensions usually are).

wow thanks a ton!


Add libvirt and vagrant plugin to the server we use for sandbox e2e test

Can we get libvirt and vagrant plugin vagrant-libvirt installed alongside the server that runs Virtualbox? I would like to run the same e2e test we run with Vagrant VirtualBox but with libvirt

@mmlb pointed out we will need 2 different runners because libvirt and Virtualbox modules don't like each other, I think we have to test it out!

@mmlb is right, I had this issue when testing basebox, libvirt and vbox are not happy to run side by side https://www.agix.com.au/kvm-error-ioctlkvm_create_vm-failed-16-device-or-resource-busy/

Thanks @rawkode for helping us out!


Remove unnecessary references in resource.go

genruntime can be used by interfaces, which will be automatically
included via InterfaceImplementer. It can also be included in
specs for properties such as Owner, but in that case it will
already be included on the Spec as a requirement (via the property
itself).
fmt can be used in functions, but those will report their need for
it directly.

Coverage remained the same at 73.086% when pulling 3f598f0502c7bc2bec61a8a5eaccc0b53955e6fc on matthchr:feature/cleanup-resource-refs into f73c3b4518ca7e5b6594aaeabe61fdc5e885876c on Azure:master.


Create tutorial for new Gittip Developers

We need a proper tutorial on what programming environment and programs/files/software that each new developer for Gittip will use to more easily integrate with.
So far the list of software needed to program in Gittip includes:
Windows:
Github
Git Shell
Git
Vagrant
Virtual Box
Macs:
???
Linux:
??? (Help me out here guys)
python
gcc
make
postgres server

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Merged with "Lower the barrier for hacking on Gratipay" #2927


testing: prow/gpu-operator: test v1.8 in PRs while v1.9.0-beta cannot be tested

/test gpu-operator-e2e

/test gpu-operator-e2e

/lgtm

/test gpu-operator-e2e

/test gpu-operator-e2e

/lgtm

/test gpu-operator-e2e
On Thu, Nov 25, 2021 at 3:23 PM openshift-ci[bot] @.***>
wrote:

New changes are detected. LGTM label has been removed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/289#issuecomment-979255883,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQIXSJEWHFT7LWJYZLNTUNZBEXANCNFSM5IYBUQCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

/test gpu-operator-e2e
On Thu, Nov 25, 2021 at 4:36 PM openshift-ci[bot] @.***>
wrote:

@kpouget https://github.com/kpouget: The following test failed, say
/retest to rerun all failed tests or /retest-required to rerun all
mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/gpu-operator-e2e e0fbe4a
https://github.com/openshift-psap/ci-artifacts/commit/e0fbe4a28e31c9a9d2c9843c32ac7fabce462a7d
link
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift-psap_ci-artifacts/289/pull-ci-openshift-psap-ci-artifacts-master-gpu-operator-e2e/1463877133175623680
true /test gpu-operator-e2e
Full PR test history
https://prow.ci.openshift.org/pr-history?org=openshift-psap&repo=ci-artifacts&pr=289.
Your PR dashboard
https://prow.ci.openshift.org/pr?query=is:pr+state:open+author:kpouget.
Instructions for interacting with me using PR comments are available here
https://git.k8s.io/community/contributors/guide/pull-requests.md. If
you have questions or suggestions related to my behavior, please file an
issue against the kubernetes/test-infra
https://github.com/kubernetes/test-infra/issues/new?title=Prow issue:
repository. I understand the commands that are listed here
https://go.k8s.io/bot-commands.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift-psap/ci-artifacts/pull/289#issuecomment-979308536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZVQIWBOH6JVA6XDK277WDUNZJWTANCNFSM5IYBUQCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@omertuc good for merging,
I validated the "master" branch testing code path in https://github.com/openshift-psap/ci-artifacts/pull/286#issuecomment-979377278, by ensuring that my monitoring PR got properly tested
/approve

/lgtm


json config file is inconsistent with lists

fp_pdn_macro_hooks must be a string: "wrapped_rgb_mixer_3 vccd1 vssd1 vccd1 vssd1, wrapped_frequency_counter_2 vccd1 vssd1 vccd1 vssd1",
but extra lefs and gds must be lists

please pick one and let's have an example. It's taken me far too long to work out how to do it.

Related to: https://openlane.readthedocs.io/en/latest/reference/configuration.html#on-comma-delimited-variables
However, I do agree that there is confusion around it and I will look if something can be done about that.

Hi Kareem, maybe a link to that doc from the config?
That would have been super helpful for me if I'd known about it.


REVIEW FORM IS VERY MONOTONOUS

review form seems very plane and doesnt suit the purple theme of the site hence here is a before and after of some changes i suggest!
BEFORE

AFTER

ive added the hover effect for the ratings

@07sumit1002  kindly assig me this.

@07sumit1002  kindly assign me this issue

@07sumit1002 pls review

@07sumit1002  kindly reply


Extract Golang Service Fabric SDK

Extract the Service Fabric client from the Traefik PR and host as stand-alone. Using this opportunity to clean up the SDK, expose more capabilities such as Service Manifest extensions and support for different authentication models (Windows Auth).

Work started here


Fix Seikan

NOTE: Before you begin working on this, please assign this issue to yourself. If you do not do this, then there's the possibility that 2 people will end up doing the same task, which is not a good use of time.
There is a clarifying statement on Seikan Tunnel so this can be resolved:
                // TODO: Revise wording once we get a clarification on https://boardgamegeek.com/thread/1937834/seikan-tunnel-artifact-rules-question.

I'll take this. Right now this card is making a lot of SQL queries, so I'm going to write a custom query so that we can optimize this.


add rpc method to enumerate contract storage

Describe the feature
there is a little-known method called parity_listStorageKeys that returns a paginated view of a contract storage at a certain block height.
it could be a good starting point, but the method itself is not ideal, because the user would most likely also want to retrieve the associated values and not only keys. some kind of streaming or pagination would make sense to account for contracts with massive storage.
Additional context
No response

+1. This is a very helpful RPC for security work. ETHBMC is an example where it's used for storage access. Love the idea of returning key:val pairs.
For massive storage testing, univ2factory has 202450 pairs, which IIRC is doubled in storage for redundant pair mappings pair[token][token]. Largest storage I know of

there is an even bigger one with over 4 million slots used https://twitter.com/austinadams10/status/1669869209520680965

This is interesting one, it did require fatdb flag for OE.
On impl side, I am not sure about performance but It can be done in reth, this would mean we would iterate over StorageHistory table to get all storages that got changed until that block, and second step is to extract values of those slots from StorageChangeSet table. Just to note that extracted value can be zero as it represent not existing (storage got removed) slot.
Offset storage key from which the batch should start in order makes sense.

From #3517 there is geth opcode that does similar thing: https://geth.ethereum.org/docs/interacting-with-geth/rpc/ns-debug#debugstoragerangeat
The difference (other than naming) is that geth additionally takes the tx index. This means we must execute the previous transaction from block to get the updated state and intertwine that state with state form database (both overrides and ordering new values).


open /run/config/pki/apiserver-kubelet-client.crt: no such file or directory

Any idea what I'm doing wrong?
cockroachdb:
  tls:
    enabled: true
  statefulset:
    replicas: 1
    resources:
      limits:
        memory: "4Gi"
      requests:
        memory: "2Gi"
  conf:
    single-node: "yes"
    cache: "1024Mi"
    max-sql-memory: "1024Mi"

Logs:
Namespace
iron-backend-dev
Pod
temporal-cockroachdb-0
Container
db
Search...
Failed to load logs: Get "https://<IP_ADDRESS>:10250/containerLogs/iron-backend-dev/temporal-cockroachdb-0/db?tailLines=502&timestamps=true": open /run/config/pki/apiserver-kubelet-client.crt: no such file or director

What version of the helmcharts are you using? Can you share your values.yml file?

It's 6.1.2, the above is my values.yaml where I specify cockroachdb as a Chart dependency.


Runtime typos fixes from various maintainers

I've contacted all maintainers and sent them the patch they need to fix. Some emails were invalid (yes, I removed the NOSPAM), some responded back by saying that they don't contribute to Vim anymore and so on and so forth. This PR is just to ensure that the fixes are actually done in case the maintainers doesn't maintain.
Can squash the commits if you'd like.

Could you make a list of files that maintainers indicated they were no longer maintaining?  Then the headers can be changed to indicate this status.
Thanks.

In hindsight the "some" was a bit of an exaggeration: only John Cook explicitly responded saying he doesn't contribute to vim anymore. He maintained
runtime/syntax/abel.vim
runtime/syntax/ahdl.vim
runtime/syntax/cupl.vim

Thanks for doing this.  I'll include the patch and send out a runtime file update soon.


move ABM in its own namespace

Changes
Moved all of ABM into its own namespace mio::abm.
Closes #280
Merge Request - GuideLine Checklist
Guideline to check code before resolve WIP and approval, respectively.
As many checkboxes as possible should be ticked.
Checks by code author:

[x] There is at least one issue associated with the pull request.
[x] The branch follows the naming conventions as defined in the git workflow.
[ ] New code adheres with the coding guidelines
[ ] Tests for new functionality has been added
[x] A local test was succesful
[ ] There is appropriate documentation of your work. (use doxygen style comments)
[ ] If new third party software is used, did you pay attention to its license? Please remember to add it to the wiki after successful merging.
[ ] If new mathematical methods or epidemiological terms are used, has the glossary been updated ? Did you provide further documentation ?
is present or referenced. Please provide your references.
[ ] The following questions are addressed in the documentation*:  Developers (what did you do?, how can it be maintained?), For users (how to use your work?), For admins (how to install and configure your work?)
For documentation: Please write or update the Readme in the current working directory!

Checks by code reviewer(s):

[ ] Is the code clean of development artifacts e.g., unnecessary comments, prints, ...
[ ] The ticket goals for each associated issue are reached or problems are clearly addressed (i.e., a new issue was introduced).
[ ] There are appropriate unit tests and they pass.
[ ] The git history is clean and linearized for the merge request.
[ ] Coverage report for new code is acceptable.

Codecov Report

Merging #283 (a51d516) into main (dcf077b) will increase coverage by 0.06%.
The diff coverage is 96.25%.

@@            Coverage Diff             @@
##             main     #283      +/-   ##
==========================================
+ Coverage   91.36%   91.42%   +0.06%
==========================================
  Files          79       79
  Lines        4852     4854       +2
==========================================
+ Hits         4433     4438       +5
+ Misses        419      416       -3

Impacted Files
Coverage Δ

cpp/models/abm/random_events.h
100.00% <ø> (ø)

cpp/models/abm/simulation.h
100.00% <ø> (ø)

cpp/models/abm/testing_scheme.cpp
100.00% <ø> (ø)

cpp/models/abm/testing_scheme.h
100.00% <ø> (ø)

cpp/models/abm/time.h
100.00% <ø> (ø)

cpp/models/abm/trip_list.cpp
100.00% <ø> (ø)

cpp/models/abm/trip_list.h
100.00% <ø> (ø)

cpp/models/abm/world.h
100.00% <ø> (ø)

cpp/models/abm/world.cpp
79.66% <33.33%> (ø)

cpp/models/abm/migration_rules.cpp
85.13% <92.85%> (ø)

... and 10 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 dcf077b...a51d516. Read the comment docs.


Functional Annotation Integration

2 Parts

Integration with DAVID and other functional annotation tools
A purpose of my tool can be to determine if the input gene interacts with other genes with unknown interaction

For linking to DAVID, DAVID provides a API to redirect to their page: https://david.ncifcrf.gov/content.jsp?file=DAVID_API.html

For retrieving XML results, DAVID provides a web service and Java client: https://david.ncifcrf.gov/content.jsp?file=WS.html


Make main login/logout buttons reload server side props

That means that if we create server-side redirects, they will trigger the redirects if appropriate (i.e. redirecting from / to /home if logging in, redirecting from random logged-in-users-only pages to / if logging out.) So now we can set up server side redirects when desired and not worry about also doing it on the client side in the React components anymore.

@jahooma here's my "actual suggestion" for what to do about the problem you noticed yesterday. This should resolve it everywhere.


Batch API Support and Incremental Data Addition/Subtraction?

Hey AutoRAG devwlopers,
Thanks for the great work on AutoRAG. It's been a valuable tool for optimizing RAG pipelines. I thought there are a couple of feature requests that could make it even more powerful and efficient:

Batch API Support:
With OpenAI's recent release of the Batch API, there's an opportunity to reduce the cost of generating QA data from corpus data by up to 50%. Integrating Batch API support into AutoRAG would be good for users working with large datasets and limited budgets.

Incremental Data Addition/Subtraction:
The current AutoRAG pipeline requires creating corpus data and generating QA data from it sequentially, as I understood. However, there are scenarios where users may have additional data to incorporate into an existing RAG pipeline, or need to remove or replace it(like renewed docs) It would be great to know if there are any plans or ezisting methods to support incremental data addition in the future or how this could be implemented.

These feature requests may require significant development efforts, but they would greatly enhance the usability and efficiency of AutoRAG. If there's anything that can be done to help or provide further clarification, please let me know.
Thanks for considering these feature requests. Looking forward to hearing your thoughts.

Hi @kirillocha Thanks for making new issue.
First, batch API looks great. We will consider to implement it to data creation process.
Second, there are some cases that users may want to use AutoRAG with their pre-ingested corpus.
Like if there are millions of embeddings on their own vector DB already, they may want to use it for benchmark.
From now, we do not support that features.
We can make some migration option of local Chroma, or vectorDB option via LlamaIndex. Users certainly do not want to embed all documents again.
Plus, if you changed your QA or corpus data, we strongly recommend to make new project folder. It will occur make embeddings on whole corpus again. It is inefficient, so I think we might find how to resolve it.
I'll talk about this issue with my teammates and add updates for it.  (Check out this issue @bwook00 @Eastsidegunn )
Thanks again @kirillocha 👍

Thank you for addressing my suggestions and feedback in detail. I appreciate your openness to enhancing AutoRAG and your dedication to the project, as shown by your frequent commits. Your hard work is appreciated, and I'm grateful for your commitment to making AutoRAG better. Thank you for considering my input.