IGNITE-3407 HTTP REST: query commands without pageSize failed with NPE

https://issues.apache.org/jira/browse/IGNITE-3407 is closed, closing the PR


thin-provisioning-tools: fix patch url

Motivation for this change
Void Linux changed its github organization from "voidlinux" to "void-linux"
See: https://voidlinux.org/news/2018/06/GitHub-Organisation-is-moving.html
Fixes https://github.com/NixOS/nixpkgs/issues/60654
cc: @nixbitcoin
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

Thanks for the quick fix. Have already tested it with my own fix and it works. You beat me to the PR :smile:


Far Cry 4 sometimes hang during startup with dxvk

When launching Far Cry 4 with dxvk it will freeze between 1/3 - 1/2 of the time at the loading screen before the company logos.
Launching with WINED3D i was able to get to the main menu each of the around 25 times i tried.
Software information
Far Cry 4 steam
System information

GPU: AMD R9 380
Driver: mesa 22.1.1
Wine version: Proton 7.0-2 & 7.0-3RC
DXVK version: 1.10.1, master and every other older versions I've tried back to 1.5

Apitrace file(s)
I was trying to get a couple API traces with dxvk both where it would freeze and where it wouldn't but the game wouldn't start with it.
If you think a WINED3D or windows trace will have value for this then i will make one.
Log file
FarCry4_d3d9.log
FarCry4_d3d11.log
FarCry4_dxgi.log
These are from a time where it froze.

For d3d11, I did not see the setting to do it. For dxgi, I saw:
dxgi.maxDeviceMemory = x
dxgi.maxSharedMemory = x
which you can try to change the values accordingly. I did not have the chance to try it out.

For the general settings outside VRAM, just change them to a more generally compatible option for most games. For example, dxgi.deferSurfaceCreation = True and/or d3d11.dcSingleUseMode = False.

It's not the same issue but thanks for the input :grin:
I have already tried all conf settings and asked the devs in the discord if i might be missing anything. Just making the issue so it isn't forgotten.

Made a mesa issue. Updated findings can be read there https://gitlab.freedesktop.org/mesa/mesa/-/issues/7050
Tho this could still just be some weird dxvk or game bug.

Closing this as i still believe it is likely some weird driver issue.


Deploy RDS Doc Commands Do Not Work

Describe the bug
The current doc suggests the following :

(https://github.com/awslabs/kubeflow-manifests/tree/v1.3-branch/distributions/aws/examples/rds-s3)
But when trying the commands from the doc Deploy Amazon RDS MySQL to setup the RDS instance, the commands do not work.
Steps To Reproduce

Create an EKS cluster
Try the commands listed here

Expected behavior
The command should return the VpcId for the first command, the list of SubnetId for the second command and the SecurityGroupId for the third command.
The returned value(s) should be only the values needed such that we can use the returned values as-is (no parsing required from user).
Environment

Kubernetes version 1.21
Using EKS (yes/no), if so version? yes 1.21
AWS service targeted (S3, RDS, etc.) RDS

I was able to figure out why the commands would not work.
Running aws configure I was able to notice that my preference for output format was set to yaml.
I will make a PR to improve the commands so that it works regardless of your output format preference.

Describe the bug The current doc suggests the following :  (https://github.com/awslabs/kubeflow-manifests/tree/v1.3-branch/distributions/aws/examples/rds-s3) But when trying the commands from the doc Deploy Amazon RDS MySQL to setup the RDS instance, the commands do not work.
Steps To Reproduce

Create an EKS cluster
Try the commands listed here

Expected behavior The command should return the VpcId for the first command, the list of SubnetId for the second command and the SecurityGroupId for the third command. The returned value(s) should be only the values needed such that we can use the returned values as-is (no parsing required from user).
Environment

Kubernetes version 1.21
Using EKS (yes/no), if so version? yes 1.21
AWS service targeted (S3, RDS, etc.) RDS

What if I'm still not getting the return values as what i expect it to be returned? Can you help?

@khalyan Could you please share more details? What is the commands you are using, what is the output, screenshot maybe etc ?

@khalyan Could you please share more details? What is the commands you are using, what is the output, screenshot maybe etc ?

I think I'm making a mistake on the setup of EKS cluster. I'll post the example of what I'm using!
$ eksctl create cluster --name=demo-eks-cluster --nodes=2 --region=ap-south-1
[ℹ]  using region ap-south-1
[ℹ]  setting availability zones to [ap-south-1a ap-south-1c ap-south-1b]
[ℹ]  subnets for ap-south-1a - public:<IP_ADDRESS>/19 private:<IP_ADDRESS>/19
[ℹ]  subnets for ap-south-1c - public:<IP_ADDRESS>/19 private:<IP_ADDRESS>/19
[ℹ]  subnets for ap-south-1b - public:<IP_ADDRESS>/19 private:<IP_ADDRESS>/19
[ℹ]  nodegroup "ng-ae09882f" will use "ami-09c3eb35bb3be46a4" [AmazonLinux2/1.12]
[ℹ]  creating EKS cluster "demo-eks-cluster" in "ap-south-1" region
[ℹ]  will create 2 separate CloudFormation stacks for cluster itself and the initial nodegroup
[ℹ]  if you encounter any issues, check CloudFormation console or try 'eksctl utils describe-stacks --region=ap-south-1 --name=demo-eks-cluster'
[ℹ]  2 sequential tasks: { create cluster control plane "demo-eks-cluster", create nodegroup "ng-ae09882f" }
[ℹ]  building cluster stack "eksctl-demo-eks-cluster-cluster"
[ℹ]  deploying stack "eksctl-demo-eks-cluster-cluster"
[ℹ]  building nodegroup stack "eksctl-demo-eks-cluster-nodegroup-ng-ae09882f"
[ℹ]  --nodes-min=2 was set automatically for nodegroup ng-ae09882f
[ℹ]  --nodes-max=2 was set automatically for nodegroup ng-ae09882f
[ℹ]  deploying stack "eksctl-demo-eks-cluster-nodegroup-ng-ae09882f"
[✔]  all EKS cluster resource for "demo-eks-cluster" had been created
[✔]  saved kubeconfig as "/Users/Bensooraj/.kube/config"
[ℹ]  adding role "arn:aws:iam::account_number:role/eksctl-demo-eks-cluster-nodegroup-NodeInstanceRole-1631FNZJZTDSK" to auth ConfigMap
[ℹ]  nodegroup "ng-ae09882f" has 0 node(s)
[ℹ]  waiting for at least 2 node(s) to become ready in "ng-ae09882f"
[ℹ]  nodegroup "ng-ae09882f" has 2 node(s)
[ℹ]  node "ip-192-168-30-190.ap-south-1.compute.internal" is ready
[ℹ]  node "ip-192-168-92-207.ap-south-1.compute.internal" is ready
[ℹ]  kubectl command should work with "/Users/Bensooraj/.kube/config", try 'kubectl get nodes'
[✔]  EKS cluster "demo-eks-cluster" in "ap-south-1" region is ready
Not working for me

@khalyan from this output, looks like the cluster has been created successfully. Can you please elaborate what is not working? i.e. which of the following outputs is empty/incorrect? - vpc_id, subnet_id or security group

I created a PR in kubeflow/website to update the current doc see this PR but this PR improves the setup and would also fix this issue.


Collider missing warnings

First commit changes cargo validator to only have a warning on missing bounding collider.
Second commit further forgives the validation by checking the children for colliders.

One more thing, it could be a good idea to change from BoxCollider to just Collider? Boxes are the best but there might be cases of capsules or spheres only.

And now, should this be also changed on the livery collider check instead of just the cargo?
The "collider will be auto-generated" leaves me unsure.

The main vehicle colliders are a bit stricter I think, if we're not running into issues there I think it might be best to leave as-is for now.

It's a bit odd, I am copying the exact setup of vanilla cars, and they often don't have a component on the [collision] object.

Maybe this was something that was changed between OVH and sim


Interception timeout when responding empty body

Simply responding empty body ("") causes the timeout error. It seems to require at least one character (e.g. " ").
Steps to reproduce

Puppeteer version: 1.15.0
Platform / OS version: macOS 10.14.4
Node.js version: v10.15.3

What steps will reproduce the problem?
const puppeteer = require("puppeteer");

(async () => {
  const browser = await puppeteer.launch();
  const page = await browser.newPage();
  await page.setRequestInterception(true);
  page.on("request", async request => {
    await request.respond({
      status: 200,
      contentType: "text/plain",
      body: "" // <------
    });
  });
  await page.goto("https://example.com");
  await browser.close();
})();

What is the expected result?
Done with no error.
What happens instead?
Raises TimeoutError.
(node:35254) UnhandledPromiseRejectionWarning: TimeoutError: Navigation Timeout Exceeded: 30000ms exceeded
    at Promise.then (/Users/jinjor/Projects/actcast/webui/node_modules/puppeteer/lib/LifecycleWatcher.js:142:21)
  -- ASYNC --
    at Frame.<anonymous> (/Users/jinjor/Projects/actcast/webui/node_modules/puppeteer/lib/helper.js:110:27)
    at Page.goto (/Users/jinjor/Projects/actcast/webui/node_modules/puppeteer/lib/Page.js:629:49)
    at Page.<anonymous> (/Users/jinjor/Projects/actcast/webui/node_modules/puppeteer/lib/helper.js:111:23)
    at /Users/jinjor/Projects/actcast/webui/tests/ts/sscce.js:14:14
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:35254) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:35254) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

A workaround for this issue it to pass a Buffer instead of string:
  page.on("request", async request => {
    await request.respond({
      status: 200,
      contentType: "text/plain",
      body: Buffer.from("") // <------
    });
  });


Make URL a variable

I'm aware of all these PRs:
#21 - doesn't check validity of generated URL
#18 - trying to add more variables at a time which makes it hard to discuss the right name per variable
#16 - doesn't check validity of generated URL
In addition to that, I was thinking about right variable names and came up with ROOT_URL because:

anything with domain is IMO not correct because we expect protocol (http/https) as well, not just domain
the prefix discovery is IMO not needed because the variable is supposed to be visible only for the Discovery container

I know that ROOT_URL may not be the best name, but it's the best I was able to come up with and I'm open for suggestions.

If this gets merged, two more PRs can be submitted: https://github.com/TimeIncUK/discovery.etcd.io/compare/configurable

We merged #25 which has a flag if you want an ENV variable I would accept a patch for DISCOVERY_HOST= along with the flag


[SUPPORT] No successful commits under path

To Reproduce
Steps to reproduce the behavior:
code https://github.com/izhangzhihao/Real-time-Data-Warehouse/tree/hudi
create table
CREATE TABLE accident_claims
(
    claim_id            BIGINT,
    claim_total         DOUBLE,
    claim_total_receipt VARCHAR(50),
    claim_currency      VARCHAR(3),
    member_id           INT,
    accident_date       DATE,
    accident_type       VARCHAR(20),
    accident_detail     VARCHAR(20),
    claim_date          DATE,
    claim_status        VARCHAR(10),
    ts_created          TIMESTAMP(3),
    ts_updated          TIMESTAMP(3),
    ds                  DATE,
    PRIMARY KEY (claim_id) NOT ENFORCED
) PARTITIONED BY (ds) WITH (
  'connector'='hudi',
  'path' = '/data/dwd/accident_claims',
  'table.type' = 'MERGE_ON_READ',
  'read.streaming.enabled' = 'true',
  'write.batch.size' = '1',
  'write.task.max.size' = '1',
  'write.tasks' = '1',
  'compaction.tasks' = '1',
  'compaction.delta_seconds' = '60',
  'write.precombine.field' = 'ts_updated',
  'read.tasks' = '1',
  'read.streaming.check-interval' = '5',
  'read.streaming.start-commit' = '20210712134429',
);

insert from CDC change stream
INSERT INTO accident_claims
SELECT claim_id,
       claim_total,
       claim_total_receipt,
       claim_currency,
       member_id,
       CAST (accident_date as DATE),
       accident_type,
       accident_detail,
       CAST (claim_date as DATE),
       claim_status,
       CAST (ts_created as TIMESTAMP),
       CAST (ts_updated as TIMESTAMP),
       CAST (SUBSTRING(claim_date, 0, 9) as DATE)
FROM datasource.accident_claims;

Expected behavior
SELECT * FROM accident_claims;

should return results
But got:
Flink SQL> SELECT * FROM accident_claims;
[ERROR] Could not execute SQL statement. Reason:
org.apache.hudi.exception.HoodieException: No successful commits under path /data/dwd/accident_claims

But the sample code works:
CREATE TABLE t1(
  uuid VARCHAR(20), -- you can use 'PRIMARY KEY NOT ENFORCED' syntax to mark the field as record key
  name VARCHAR(10),
  age INT,
  ts TIMESTAMP(3),
  `partition` VARCHAR(20)
)
PARTITIONED BY (`partition`)
WITH (
  'connector' = 'hudi',
  'path' = '/data/t1',
  'write.tasks' = '1', -- default is 4 ,required more resource
  'compaction.tasks' = '1', -- default is 10 ,required more resource
  'table.type' = 'COPY_ON_WRITE', -- this creates a MERGE_ON_READ table, by default is COPY_ON_WRITE
  'read.tasks' = '1', -- default is 4 ,required more resource
  'read.streaming.enabled' = 'true',  -- this option enable the streaming read
  'read.streaming.start-commit' = '20210712134429', -- specifies the start commit instant time
  'read.streaming.check-interval' = '4' -- specifies the check interval for finding new source commits, default 60s.
);

-- insert data using values
INSERT INTO t1 VALUES
  ('id1','Danny',23,TIMESTAMP '1970-01-01 00:00:01','par1'),
  ('id2','Stephen',33,TIMESTAMP '1970-01-01 00:00:02','par1'),
  ('id3','Julian',53,TIMESTAMP '1970-01-01 00:00:03','par2'),
  ('id4','Fabian',31,TIMESTAMP '1970-01-01 00:00:04','par2'),
  ('id5','Sophia',18,TIMESTAMP '1970-01-01 00:00:05','par3'),
  ('id6','Emma',20,TIMESTAMP '1970-01-01 00:00:06','par3'),
  ('id7','Bob',44,TIMESTAMP '1970-01-01 00:00:07','par4'),
  ('id8','Han',56,TIMESTAMP '1970-01-01 00:00:08','par4');

SELECT * FROM t1;

So I didn't get what's wrong here...
Environment Description

Hudi version : 0.9.0 SNAPSHOT

Flink version :  1.12.2

Hive version : none

Hadoop version : 2.8.3

Storage (HDFS/S3/GCS..) : local file system

Running on Docker? (yes/no) : yes

Additional context
Add any other context about the problem here.

Taskmanager log: taskmanager.log.zip

Looking at Flink Web-ui, it seems that there is no sink. Is the checkpoint triggered?

No, there is no checkpoint history.

If it is streaming writing, please make sure to turn on checkpoint. When the checkpoint is triggered, all data in the memory is flushed out, or a bucket has accumulated to 64MB in memory, or the total buffer size is accumulated to 256MB.

Thanks a lot! after config checkpoint dir, it works!

Hi~How can I use checkpoint in Flink SQL client?

Hi~How can I use checkpoint in Flink SQL client?

Take a look at this document: https://www.yuque.com/docs/share/01c98494-a980-414c-9c45-152023bf3c17?#


Complete v0.1 mdbook

This PR completes the documentation for v0.1. It documents how to define constraints in the boundary_constraints and transition_constraints sections and makes a few other minor updates for clarity.
Remaining:

we'll update the name throughout these docs when we update the name throughout the codebase just prior to releasing v0.1

@bobbinth this PR is ready for a final review. In addition to the points previously addressed, I made a few more minor tweaks to incorporate fixes added while this was pending and also a to address a few small points @tohrnii noticed.
@tohrnii your review is optional. (I know you are unlikely to be available today/tomorrow)

Great, thanks for catching both of these! I'll fix before merging


ResourceLoader can't find resources

Describe the bug
I added Strings/en-US/Settings.resw in my packaged app, but ResourceLoader("Settings") throws an exception.
Everything works fine when using a parameterless constructor, or when referencing the resource in XAML.
Steps to reproduce the bug
Using ResourceLoader(String).
Expected behavior
Constructs a ResourceLoader that references the specified resource.
NuGet package version
Windows App SDK 1.5.1: 1.5.240311000
Packaging type
Packaged (MSIX)
Windows version
Insider Build (26090)
IDE
Visual Studio 2022-preview
Additional context
Documents:
https://learn.microsoft.com/en-us/windows/windows-app-sdk/api/winrt/microsoft.windows.applicationmodel.resources.resourceloader.-ctor?view=windows-app-sdk-1.5#microsoft-windows-applicationmodel-resources-resourceloader-ctor(system-string)
https://learn.microsoft.com/en-us/windows/apps/windows-app-sdk/mrtcore/localize-strings?source=recommendations

The old and obsolete API Windows.ApplicationModel.Resources.ResourceLoader has the correct behavior, but it's not a real solution to the problem.

I'm not sure if this is intended but you need to say which pri you want to use now. So, instead of just ResourceLoader("Settings"), you should write ResourceLoader(ResourceLoader.GetDefaultResourceFilePath(), "Settings").
If this is intended, doc should be updated.

duplicate #2406

duplicate


Allow sharing common configuration among rules

So far it seems that we have to repeat the full Alerter configuration in each rule file. This makes it tricky to keep things up to date, for instance:

Using the same rule configuration in both our dev and production environments requires us to keep a different copy for each env.
Updating configuration requires repeating the update across all rules.

It would be nice to be able to specify some sort of default values in a common file.

Hi,
I'm agree with you !
For my point of view, it could be nice to have a configuration file for each environment to centralized specific value which depends of an environment.
When ElastAlert run a rule, it will send it couple time with the value of each envirnment file configuration.
And for a most common usage it could be nice to be able to use variables into configuration rules files


ML ranking algorithm implementation

@lewismc I have searched for ML libraries that can provide "Learning to Rank" APIs for days, but I still cannot find a good one.
http://www.cs.cornell.edu/people/tj/svm_light/svm_rank.html  (Written in C++)
https://github.com/jattenberg/RankLib (Written in Java, but does not provide API, and no comment at all)
Inspired by this Python code: http://fa.bianp.net/blog/2012/learning-to-rank-with-scikit-learn-the-pairwise-transform/
I decided to implement a RankSVM by myself, which won't be very hard. I would say 2 days is enough.
So, what are your thoughts on this?

I ended up using RankSVM algorithm and Spark MLlib


[Issue-2563] Update staking APY formula for relaychain

Related issue(s)
#2563
Is your feature request related to a problem(s)? Please describe.

Update staking APY formula for relaychain

Describe the solution you'd like

No

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-2584--sw-web-runner.netlify.app


Some random stuff.

I didn't look over your entire code, but I acknowledged a few stuff. Let me know if you have any questions.

Thanks for opening this pull request. I appreciate the work and effort that you've put into it.
In some of the commits I have made issues in the repo which I'll take a look at and have a go at eventually [https://github.com/shadowolfyt/Rodney/issues/114 & https://github.com/shadowolfyt/Rodney/issues/115]


Request : remove dependency with ZendXml | add autoloader.php

Hello,
Thanks you for this lib, it helps a lot ;)
Inside your lib/PicoFeed/Parser/XmlParser, there is "use ZendXml\Security;", can you manage something to remove this dependency ?
For technical purposes, i refuse to use Composer, not adapted to fit my development guidelines. Can you add an autoloader please ?
Thanks you,
Best regards,
Vincent

No need to integrate composer, simply run:
composer dump

In the picoFeed directory and then there's an autoloader

PS: If you want to automatically install and generate the autoloader, run this in the picoFeed directory:
composer require --prefer-dist

I understand your wish to make us use Composer, it's a great project, there is no debate about this.
But have the necessity to install one "other" program is not an option for me, since i'm already dealing with hundreds. Maybe later.
If you could add an autoloader by default within your projects, it would be really appreciate.
Regards,
Vincent

No need to install it, you can download the composer.phar and run it directly, aka
php composer.phar install

Here's a one liner:
"wget getcomposer.org/composer.phar; php composer.phar install"

Okay you really dislike autoloader ;)
And about the dependency with ZendXML ?

ZendXML is there to protect against XXE. This is usually simple (libxml_disable_entities(true)) but for PHP versions lower than 5.5.22 and 5.6 and php-fpm you can exploit the fact that the function above is not threadsafe.
To still be able to let picoFeed run on that php versions you can manually scan through the code which is not trivial (there was another exploit recently related to unicode tricks). Just take a look at https://github.com/zendframework/ZendXml/blob/master/library/ZendXml/Security.php
Basically ZendXml is used for a complicated Security feature. Using our own mechanism is harder to maintain and check. In case of an RSS reader where users can potentially read out any file on your server this is a definite no-go.
However: This dependency can be removed once picoFeed requires a PHP version greater than 5.6.6 which will take a few years though (I guess).

I understand your problem but it's sad, in 2016, to impose this kind of things (bypassing very old version issues), since PHP 7 is going to be released soon.
I was watching around the web to find something about PHP version usage. I found this : http://w3techs.com/technologies/details/pl-php/5/all. I'm horrified. I hope, these statistics are false. Else, there will be pain and suffering in 2017, since the security support of PHP 5.6 will be stopped.
Presently, i'm using Debian 8 which use PHP 5.6.14. Is there a way to give us a "Zend less" version of your library ? For those who care about having an up to date information system ?
Thanks

The most used versions reflect the maintained distros in current usage:

CentOS 6, Ubuntu 12.04, Debian oldstable: PHP 5.3
CentOS 7: PHP 5.4
Ubuntu 14.04: PHP 5.5

As for a Zend less version: There is no downside in requiring an extra lib to handle backwards compatibility issues (Zend will only use slow parsing on vulnerable versions, see https://github.com/zendframework/ZendXml/blob/master/library/ZendXml/Security.php#L139) and there is no upside in maintaining two different picoFeed versions.
Why don't you simply execute the oneliner from bash and be happy :)?

PS: even the PHP 5.6 versions are vulnerable up to the point of 5.6.6 it all depends on how your distro backports these things. In case of Ubuntu see this one here ;) https://bugs.launchpad.net/ubuntu/trusty/+source/php5/+bug/1509817

Yes, i've noticed that point.
Well, since you are not going to make a standalone version of your library, i'll stop to annoy you about this ;)
I wish you an happy new year and hope the best for your projects !
Regards,
Vincent


test: Added unittests with Alert API

This PR introduces the following modifications:

updates test dependencies
passes down .env from the CI
adds unittests with the API
improves logging in Engine
renames client_creds into cam_creds
fixes edge case for state update
return HTTP response in private methods (for unittests)

Closes #51
Any feedback is welcome!

All good, and now we got 94% of coverage :grin:


Dropdown menu improvements

Background
Need is to have more control over menu items. Passing props to menu items' and menu item's title can now also be a JSX.element, before it could only be a string.

Modifications were made for the same purpose as https://github.com/lyytioy/lyyti-design-system/pull/336


Check that destroys actually succeed.

Original implementation did not check return value from calls to destroy, allowing failed destroys to return to the client as successful. Use existing exceptions to return errors in cases where it does fail.

:+1: I exactly need that now :smile:.

Good catch.

In our use-case we would like to require the User#current_password to be send on delete.
Would it be possible to set data.attributes on Resource#_model, maybe with something like self.destroyable_fields`?
DELETE /users/1 HTTP/1.1

{
  "data": {
    "id": "1",
    "type": "users",
    "attributes": {
      "current_password": "1234"
    }
  }
}

@lgebhardt This patch has worked well for us in production for a few months.We've updated it with latest changes from master. Are you happy to merge?  We're keen to move back to mainline rather than our fork. Thanks

@Soliah Thanks! Merged by way of #604.


Allow using one password for incoming and outgoing server

In most cases users only have one email password. It's used for authentication to both the incoming server and the outgoing server.
But right now the passwords for both servers are completely separate in K-9 Mail. While users only have to enter the password once during initial setup, they have to enter it twice when the password changes, once for the incoming server and once for the outgoing server.
The probably easiest way to link the two passwords is to have a check box in the outgoing server settings that reads "use incoming server password".

This just bit me - trying to fix an issue with gmail had me change passwords. Having forgotten that passwords need to be configured in two different places, I only updated the outgoing server. Cue confusion when K9 complained it couldn't read email. So I try changing passwords again, and blindly don't notice I'm updating a different server. So now sending doesn't work. Cue Benny Hill music.


Fix hyperlinks to source trees

These are missing links which refer to source tree instead of a blob.

Coverage increased (+0.05%) to 98.42% when pulling e39a315ffce1ac6b1818fe9f45f067026ae3f3d6 on guersam:tmp/fix-more-hyperlinks into bd437b266bb6fa963f21c0537f8379221d605e36 on mohiva:master.

Thanks


Updated README.md

A broken link was fixed

Pull Request Test Coverage Report for Build 89

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.1%) to 87.912%

Files with Coverage Reduction
New Missed Lines
%

talos/reducers/spear_reducer.py
1
86.67%

talos/metrics/performance.py
7
74.67%

Totals

Change from base Build 88:
-1.1%

Covered Lines:
640

Relevant Lines:
728

💛  - Coveralls

Thanks! :)


Possible Incorrect Operation in "05 Descriptor Sets/09 Creating an input attachment.cpp"

On line 54 in the second 'if' statement (the one regarding aspect), the operation being performed is
(aspect & (VK_IMAGE_ASPECT_DEPTH_BIT | VK_IMAGE_ASPECT_DEPTH_BIT))
Is this supposed to be
(aspect & (VK_IMAGE_ASPECT_DEPTH_BIT | VK_IMAGE_ASPECT_STENCIL_BIT))

Thanks for pointing this. Your are correct - here we are checking if image should contain color, depth or stencil data. Code is already fixed.


Include PV power data from all PV systems in region of interest

How best to represent this given that the number of PV systems will vary?  Maybe space for, say, 128 PV systems.  The compression should mean that, on disk, the 'empty slots' will take up only a few bytes.

Informally, it looks like 128 is a good number of PV systems to include.  Probably about 95% of the examples have less than 128 PV systems in the region of interest.  The highest I've seen is 160


fix(prettier): oxc_prettier drops type keyword inside import specifier braces, fixes #2708

#2708

Thank you. Formatting TypeScript is only 4% done: https://github.com/oxc-project/oxc/blob/main/tasks/prettier_conformance/prettier.ts.snap.md
There's currently no one working on this so any help is appreciated.

It seems like we are adding custom logic (import_inner_type) in this PR, I prefer not to introduce these options because this is not part of our initial goals.
I'm happy to merge if this PR only prints type without any custom logic.

It seems like we are adding custom logic (import_inner_type) in this PR, I prefer not to introduce these options because this is not part of our initial goals.
I'm happy to merge if this PR only prints type without any custom logic.

No problem. But I need to think of a way around the use of this state.
Is there now a way to store the state or some context when formatting?

prettier doesn't move the type keyword to the outside, I want to refrain from adding these kind of custom logic at this stage.

prettier doesn't move the type keyword to the outside, I want to refrain from adding these kind of custom logic at this stage.

Then that simplifies it)


Change legend information

Change legend ‘drains to sewer’ to ‘drains to creek’ for all of Durham

As a user, I find it confusing that "Drains to sewer" is listed in contrast to "Adopted" and "Adopted by you".

I think "Unadopted" or "Needs adoption" makes more sense.

Changed to drains to stream.


Use sys.exit to exit script

Use 'sys.exit' to exit script instead of 'exit'.
Yes, this change may just be pedantry. :neutral_face:

@anassinator Ok. Yes, that was likely a waste of time.
It's probably remotely related to my task of understanding how bags work.

lol this is gold


Add version to Manifest (default it to Git SHA)

It would be useful to have a stable identifier for the version of the application being run. By default, we can use the Git SHA for the version.

It's probably safer to hash the inputs as you cannot assume someone is using Git.

@swankjesse Does this end up just being the code signature?

Inside our build.gradle files we can add something that calls this: git rev-parse --verify HEAD


Properly map MMIO with Cache-Disable/Write-Through bits

Currently, we have a huge gap of functionality where all MMIO is mapped as if it was normal RAM. This means that the MMU can (and will) cache reads/writes to MMIO regions. This can result in broken functionality. We should map all MMIO with the cache-disable/write-through bits set to 1 in the PTE.

Once #266 gets merged, this will be much easier to fix. MMIO mapping types will simply need to be marked as uncacheable.


If the file share is a standard file share,  the alert rule fail without evaluation data when selecting "All Current and Future Values".

[Enter feedback here]
Hi,
The cx chose "All current and future values" because of the following text:
https://docs.microsoft.com/en-us/azure/storage/files/storage-files-monitoring?tabs=azure-portal#how-to-create-an-alert-if-the-azure-file-share-size-is-80-of-capacity
If the file share is a standard file share, select All current and future values.
As a result, the dimension was specified and the alert rule did not work properly...(The alert was evaluated successfully as a result of deleting the dimension) It may be difficult to understand the description of public information. For your information :)
Thank you.
honami akedo / hoakedo
<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ecde1ac6-43b5-ad12-695f-40cdadac4f25
Version Independent ID: 3ca5b6b9-bcb7-7b53-b473-6e69d2e88b8f
Content: Monitoring Azure Files
Content Source: articles/storage/files/storage-files-monitoring.md
Service: storage
Sub-service: files
GitHub Login: @normesta
Microsoft Alias: normesta

@hoakedo Thank you for your query . We will investigate and update the thread.

@hoakedo Thank you for your query . We will investigate and update the thread.

@hoakedo Thanks for the feedback!  Your feedback has been shared with the content owner for further review.

@hoakedo Thanks for the feedback!  Your feedback has been shared with the content owner for further review.

Thanks @hoakedo. @jeffpatt24 Is this expected behavior?

Thanks @hoakedo. @jeffpatt24 Is this expected behavior?

@jeffpatt24 - any feedback on this?

Hi @hoakedo - @jeffpatt24 has updated the content has been updated to clarify things. Those updates should appear live shortly. Thank you for bringing this to our attention! Thank you @jeffpatt24 for updating!
#please-close


[BUG] Patterns PPL showing sample log of default pattern

What is the bug?
When you run a ppl command with a specific pattern for example source = opensearch_dashboards_sample_data_logs | patterns agent`` the sample log shown is still for message, the default pattern, not the one the user is passing in
How can one reproduce the bug?
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

What is the expected behavior?
A clear and concise description of what you expected to happen.
What is your host/environment?

OS: [e.g. iOS]
Version [e.g. 22]
Plugins

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
Add any other context about the problem.

@joshuali925 @mengweieric what is the expected behavior here? Maybe this is ok for now, but we aren't getting the selected/filtered pattern from the query at all, so at some point this should be fixed - potentially after we get autocomplete functionality in.

currently we don't have synchronization between patterns in query and patterns in UI, this needs query parsing instead of regex


fix: Add idle_cycles_burned_per_day field

The field was missed in the canister status output.

@berestovskyy just to clarify, I believe that idle_cycles_burned_per_day value already includes the compute_allocation additions in it?
So if I update my canister to have a 1% compute allocation (from zero), the idle_cycles_burned_per_day seems to increase by 864 billion (per percentage point increase).
Is this correct?

Yes, this field is used internally and returned in canister_status, but it's not currently visible in dfx for some reason. By increasing the compute allocation by 1%, we should see the idle_cycles_burned_per_day field increase by 10M cycles multiplied by 86,4K seconds (one day), which equals 864 billion cycles.


Don't store plot data in the Vue instance

The plot data does not need vue's reactivity, so move it out of the vue instance for all plots. This should improve performance and at the same time allow for multiple plots to use the same data/server response.

https://github.com/lrgr/mutation-signature-explorer/commit/adcc4159926940586f99b942a034f6d026e1fdb0


Refactor/update angular school content

What does this PR do ?
Slides

Replace some code sample images with code blocks (easier to maintain, update and version)
Move Databinding section after Component section (so the exercise 4 is linked to databinding section)
Replace "annotation" term with "décorateur"  (term is more used on documentation/posts/article and is transparent in english)
Add some file name with extension on some code example for better contextualization
Fix bind, on and bindon syntax on databinding section
Refactor Concept slides on Component section
Rewrite Attribut vs propriété slide to make it clearer
Add slides number by default

Exercises

Rename PeopleAppComponent to AppComponent to match file name and convention.
Remove deprecated Material directive (align...) and replace them with CSS code.
Remove _ prefix on certain class properties when it is not necessary to match most common standards.
Remove $ suffix on EventEmitter to avoid confusion with Observable naming convention.
Remove unused modules imported in AppModule.
Use EventEmitter emit() function directly in HTML file for card deletion.

@Nicoss54 @jefBinomed @tinesoft

EventEmitter (emit function) return an Observable, so maybe you can keep the $ convention. Above all some library use it :)

EventEmitter (emit function) return an Observable, so maybe you can keep the $ convention. Above all some library use it :)

From what I saw, angular official repo, ngrx, ng-bootstrap, angular material or projects like ngx-admin or storybook don't use this convention for Output. I think it is less confusing for beginners (who don't know Rxjs well) to remove this suffix.
Maybe, other reviewers could settle this debate :)  (democracy for the win)

PR is not in draf anymore, ready for review


Clean the lib/java target before copying artifacts: [ECR-3286]

Overview
Always clean the lib/java target before copying Java artifacts.
Also clean the whole Cargo target directory before packaging,
as it will recompile things anyway. Cleaning can be skipped,
if desired, with the --skip-cargo-clean flag.

See: https://jira.bf.local/browse/ECR-3286
Definition of Done

[x] There are no TODOs left in the code
[ ] Change is covered by automated tests
[x] The coding guidelines are followed
[x] Changelog is updated if needed (in case of notable or breaking changes)
[x] The continuous integration build passes

Coverage remained the same at 85.398% when pulling dc806dad46731ac8f4007230f3463157fb14ec34 on dmitry-timofeev:fix-packaging-ECR-3286 into d4fb457a0cec6d67363c3993a8e822ed2169e990 on exonum:master.


[SPARK-6639]Create a new script to start multiple masters

start-slaves.sh script is able to read from slaves file and start slaves node in multiple boxes.
However in standalone mode if I want to use multiple masters, I’ll have to start masters in each individual box, and also need to provide the list of masters’ hostname+port to each worker. ( start-slaves.sh only take 1 master ip+port for now)
So I added several files including sbin/start-masters.sh , sbin/stop-masters.sh , conf/masters, and also modified few files so that start-all.sh/ stop-all.sh is going to start/stop all masters and slaves.
start-slaves.sh is also going to read masters file in order to fill master node list in spark connection string.
If conf/masters file does not exist, start-all.sh is going to start a master node on localhost.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

 I don't think that this has been a widely-requested feature and I'm not sure that we have the review bandwidth to look at a bunch of bash code for such an uncommonly-used feature. Therefore, would you mind closing this PR for now in order to de-clutter the review queue? Thanks!


Rel int 1.0

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


Code tags are not displaying properly on this page

The < code > tags are not working (using most up to date version of Chrome).
This is what I see (without the spaces before and after ampersand):
Gets or sets & lt;code & gt;true</code & gt; to enable client affinity;  & lt;code & gt;false & lt;/code & gt; to stop sending session affinity cookies, which route client requests in the same session to the same instance. Default is
& lt;code>true & lt;/code & gt;.

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

ID: c89f7c02-ca6c-3a95-b967-b8ca3e80e269
Version Independent ID: d1c50b62-bbc7-304f-1313-65266458bdaa
Content: Site.ClientAffinityEnabled Property (Microsoft.Azure.Management.WebSites.Models) - Azure for .NET Developers
Content Source: xml/Microsoft.Azure.Management.WebSites.Models/Site.xml
Service: app-service
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Thank you for reaching out to us about the Azure SDK for .NET. Unfortunately, this issue is related to the service side and not the SDK side, so we are not able to help you with it. Therefore, we are closing this issue for now. If you need more assistance, please open an Azure support request. We apologize for any inconvenience this may cause you.
We value your feedback and we want to make sure that your problem is solved. If you think that we have misunderstood your issue or closed it incorrectly, please feel free to comment on this thread and reopen the issue. We will be happy to assist you further. Thank you for your understanding and cooperation.

I'm not sure what you mean by service side/SDK side, I'm talking about an issue on the website.
https://learn.microsoft.com/en-us/dotnet/api/microsoft.azure.management.websites.models.site.clientaffinityenabled?view=azure-dotnet has a link at the bottom that says "Submit and view feedback for This Page".
That is what I have done. There is an error on that page to do with the display of the code, so I am providing feedback about that.


Null Safety

Hi, is null safety going to be supported soon?

Hi @aytunch!
Yes, it will be. It's currently in the pipeline.


Fix text baseline broken in recent version bump

As I noted in #603, PR #591 bumped a bunch of crates across what are
effectively major version boundaries, because the crates are in the
leading-zero phase of semver -- so an 0.6 to 0.7 update is breaking.
This has broken at least text rendering.
This commit fixes the text baseline calculations to use the method
expected by the embedded-graphics 0.7 crate, which has (quite
reasonably) switched to using baselines instead of bounding box top to
place text, but opted to leave the old implicit-baseline new operation
in place and change its behavior, which allowed all of this to break
without any compile errors.
 I'm new here and I'm not sure if these worked before
the version bump, or if they were also broken by the merge of this
untested PR, but at this point, only clock and usb_serial_display work.
Everything else displays blinking garbage on the display and hangs.
Summary
[describe your changes here]
Checklist

[ ] CHANGELOG.md for the BSP or HAL updated
[ ] All new or modified code is well documented, especially public items
[ ] No new warnings or clippy suggestions have been introduced (see CI or check locally)

If Adding a new Board

[ ] Board CI added to crates.json
[ ] Board is properly following "Tier 2" conventions, unless otherwise decided to be "Tier 1"

To partially answer my own question: I've rewound past the version bump commit to the commits preparing for the 0.15 release, and to the 0.14 release, and the Wio Terminal examples were already broken, though the screen garbage looks different than after the version bump. So, that commit is off the hook for part of the breakage.

@jessebraham @twitchyliquid64 this is what happens when you leave your boards unattended :stuck_out_tongue:

FWIW it turns out some of the display corruption is because the graphics examples have, for some time, been attempting to clear a 320 x 320 screen; the Wio Terminal's screen is 320 x 240. It looks like the behavior of this changed recently somewhere in the web of dependencies. Fixing the screen sizes for e.g. the qspi example causes it to at least correctly clear the screen, though it still doesn't work.

And just in case anyone's riveted by these updates: so it turns out you can get the qspi example display updates to work by raising the display clock frequency to 58 MHz, which I tried because it's the speed used by usb_serial_display, one of the demos that worked.
With that and the display size fix, the qspi demo on master works.
I assume the qspi demo did work at some point, so, not sure what changed. Maybe your clock setup code can't achieve 24 MHz exactly and its fallback behavior is different than it used to be?

Aaand fwiw, with the display clock change to 58, all of the demos work on a newly unboxed Wio Terminal except the wifi ones, which hang.
The sdcard demo begins by drawing on an uninitialized display, so the first pass tends to look like garbage, but after 2.5 seconds it clears the screen properly and tries again.
So, I conclude that something is wrong with the clock setup routine.

@cbiffle , thanks for looking into all of this. Do you want to pull in any more of these changes into this PR, or keep it to just the text baseline change?

Thanks for asking! No, I'm really hesitant to offer any fixes when I don't understand the problem. After you find a working clock frequency, nudging it up or down by 1MHz can cause the display driver to hang, which seems... suspicious.

Lol agreed, quite sus. We can just take this for now then.

@cbiffle Thanks for your diagnosis and fix.  I've rebased this on current master, added the fix in https://github.com/atsamd-rs/atsamd/pull/604#issuecomment-1099736323, and done one more in my main.  Now I'm just down to wifi making the wio act like a brick.


[Sql Server] Add support for datetime2

Codecov Report

Merging #1251 into master will increase coverage by 2.15%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1251      +/-   ##
==========================================
+ Coverage   72.32%   74.48%   +2.15%
==========================================
  Files          35       35
  Lines        5551     4738     -813
==========================================
- Hits         4015     3529     -486
+ Misses       1536     1209     -327

Impacted Files
Coverage Δ

src/Phinx/Db/Adapter/SqlServerAdapter.php
0% <0%> (ø)
:arrow_up:

src/Phinx/Migration/AbstractTemplateCreation.php
73.33% <0%> (-4.45%)
:arrow_down:

src/Phinx/Console/Command/Breakpoint.php
36% <0%> (-3.29%)
:arrow_down:

src/Phinx/Seed/AbstractSeed.php
46.34% <0%> (-1.58%)
:arrow_down:

src/Phinx/Console/Command/AbstractCommand.php
50.89% <0%> (-1.49%)
:arrow_down:

src/Phinx/Console/Command/Test.php
24.13% <0%> (-0.87%)
:arrow_down:

src/Phinx/Db/Table.php
95.87% <0%> (-0.37%)
:arrow_down:

src/Phinx/Db/Adapter/AbstractAdapter.php
80.32% <0%> (-0.27%)
:arrow_down:

src/Phinx/Db/Adapter/TablePrefixAdapter.php
79.54% <0%> (-0.07%)
:arrow_down:

src/Phinx/Db/Table/ForeignKey.php
97.91% <0%> (-0.05%)
:arrow_down:

... and 25 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 454238f...e915f67. Read the comment docs.

An error reported by PHPUnit. It seems related to this change.
C:\var\contributing\phinx>vendor\bin\phpunit tests\Phinx\Db\Adapter\SqlServerAdapterTest.php
PHPUnit 5.7.25 by Sebastian Bergmann and contributors.

...........................E................

Time: 38.31 seconds, Memory: 10.50MB

There was 1 error:

1) Test\Phinx\Db\Adapter\SqlServerAdapterTest::testGetColumns
RuntimeException: "sql_type" is not a valid column option.

C:\var\contributing\phinx\src\Phinx\Db\Table\Column.php:639
C:\var\contributing\phinx\src\Phinx\Db\Table.php:354
C:\var\contributing\phinx\tests\Phinx\Db\Adapter\SqlServerAdapterTest.php:410

ERRORS!
Tests: 44, Assertions: 125, Errors: 1.

Hello, what happened with this?

@diego-sorribas Never got around to it. We used native queries because there were several other things not supported in this package that we needed.
Feel free to pick up this PR if you want to.

Hello @lorenzo ! Our problem is when phinx call the getColumns method that has a getPhinxType method that not support the datetime2

@diego-sorribas what do you use the getColumns method for?

@lorenzo I dont know, I have this migration and give me the problem with the datetime2
$table = $this->table('employees');
  if ($table->exists() && $table->hasColumn('document_type_id')) {
     $table->changeColumn('document_type_id', 'integer', ['null' => true]);
     $table->update();
  }

what's the error exactly?

@diego-sorribas can you open a separate issue for that, please?


Added Swedish translation

Added Swedish translation strings to brew.sh website.

Thanks!

No problem! Just heads up, the 'ä' character in one of the titles isn't bolded right now. Does it need any extra html tags?
Link: http://brew.sh/index_se.html

My guess is that's a limitation of the webfont we're using.

If needed I can just revert that specific title back to English, as I noticed the Norwegian translation has done. There is really no other way of writing it without using non-English characters.

I'd be open to switching to a new font, if we can find a similar free one with better character support.

Yeah, that could be necessary. I just realized this issue has already been brought to attention, with no resolution: https://github.com/Homebrew/homebrew.github.io/issues/41

I'll open a PR soon to switch to this font https://www.google.com/fonts/specimen/Roboto+Slab unless we come across a better alternative.

Looks like a good replacement. Thanks you!


Feature integrate qiniu oss service

add qiniu oss service

integrate qiniu oss service ,merge this branch to master

close #1


No output for duration less than a second

If I format a duration that is less than a second, I get no output.

Fixed in https://github.com/hako/durafmt/pull/5. Thanks!


ProduceService and ConsumeService should log exception from produce() and consume() at WARN level

Looks good to me.

@smccauliff Thanks. Can you take another look? I removed check for _running in TopicManagementService.awaitShutdown()

Yeah.  The TopicManagementService running means that someone no longer needs to wait for it to stop running.


RowGroupReader.get_row_iter() fails with Path ColumnPath not found

Describe the bug
I am trying to read a Parquet file generated by a Hadoop MapReduce job. The schema is a bit complex, with a minimal repro looking something like this:
message schema {
    REPEATED group level1 {
        REPEATED group level2 {
            REQUIRED group level3 {
                REQUIRED INT64 value3;
            }
        }
        REQUIRED INT64 value1;
    }
}

Reading this schema fails with Path ColumnPath { parts: [\"value1\"] } not found. The error message is correct: value1 does not exist. It is off by one level and should be level1.value1.
Looking into the code, it seems like there is a double path.pop() happening in reader_tree().
To Reproduce
Minimal unit test reproing the issue:

Cargo.toml

[package]
name = "parquet_complex_type"
version = "0.1.0"
edition = "2021"

[dependencies]
bytes = "1.5.0"
parquet = "48.0.0"

src\lib.rs

#[cfg(test)]
mod tests {
    use bytes::Bytes;
    use std::sync::Arc;
    use parquet::data_type::Int64Type;
    use parquet::file::reader::{FileReader, SerializedFileReader};
    use parquet::file::writer::SerializedFileWriter;
    use parquet::schema::parser::parse_message_type;

    #[test]
    fn test_read_write_parquet2() {
        // Create schema
        let schema = Arc::new(parse_message_type("
            message schema {
                REPEATED group level1 {
                    REPEATED group level2 {
                        REQUIRED group level3 {
                            REQUIRED INT64 value3;
                        }
                    }
                    REQUIRED INT64 value1;
                }
            }").unwrap());

        // Write Parquet file to buffer
        let mut buffer: Vec<u8> = Vec::new();
        let mut file_writer = SerializedFileWriter::new(&mut buffer, schema, Default::default()).unwrap();
        let mut row_group_writer = file_writer.next_row_group().unwrap();

        // Write column level1.level2.level3.value3
        let mut column_writer = row_group_writer.next_column().unwrap().unwrap();
        column_writer.typed::<Int64Type>().write_batch(&[30, 31, 32], Some(&[2, 2, 2]), Some(&[0, 0, 0])).unwrap();
        column_writer.close().unwrap();

        // Write column level1.value1
        let mut column_writer = row_group_writer.next_column().unwrap().unwrap();
        column_writer.typed::<Int64Type>().write_batch(&[10, 11, 12], Some(&[1, 1, 1]), Some(&[0, 0, 0])).unwrap();
        column_writer.close().unwrap();

        // Finalize Parquet file
        row_group_writer.close().unwrap();
        file_writer.close().unwrap();
        assert_eq!(&buffer[0..4], b"PAR1");

        // Read Parquet file from buffer
        let reader = SerializedFileReader::new(Bytes::from(buffer)).unwrap();
        assert_eq!(3, reader.metadata().file_metadata().num_rows());
        let row_group_reader = reader.get_row_group(0).unwrap();
        let mut rows = row_group_reader.get_row_iter(None).unwrap();
        assert_eq!(rows.next().unwrap().unwrap().to_string(), "{level1: [{level2: [{level3: {value3: 30}}], value1: 10}]}");
        assert_eq!(rows.next().unwrap().unwrap().to_string(), "{level1: [{level2: [{level3: {value3: 31}}], value1: 11}]}");
        assert_eq!(rows.next().unwrap().unwrap().to_string(), "{level1: [{level2: [{level3: {value3: 32}}], value1: 12}]}");
    }
}

Expected behavior
Reading the Parquet file succeeds.
Additional context
A fix may be to push back the value just popped so that it can be popped again right after. This 2-line change in reader_tree() got the test to pass. I am very new to that codebase but happy to send that as Pull Request along with the test if that fix makes sense.
	_ if repetition == Repetition::REPEATED => {
		let required_field = Type::group_type_builder(field.name())
			.with_repetition(Repetition::REQUIRED)
			.with_converted_type(field.get_basic_info().converted_type())
			.with_fields(field.get_fields().to_vec())
			.build()?;

		let value = path.pop().unwrap();        <--- keep the path value here

		let reader = self.reader_tree(
			Arc::new(required_field),
			path,
			curr_def_level,
			curr_rep_level,
			paths,
			row_group_reader,
		)?;

		path.push(value);                        <--- push the path value back here

		Reader::RepeatedReader(
			field,
			curr_def_level - 1,
			curr_rep_level - 1,
			Box::new(reader),
		)
	}

Possibly a duplicate of https://github.com/apache/arrow-rs/issues/2394
Currently repeated fields are only supported via the arrow interface

That indeed looks related to #2394. Reading through the issue conversation, I also hit the error called `Option::unwrap()` on a `None` value mentioned in related #3745 (although I don't have a good repro right now).
It sounds like there isn't enough volunteer time to expand get_row_iter(). Would reviewing pull requests in that code be an option? If issues can be tackled as byte-size chunks, I can likely contribute. So far testing decoding REPEATED actually looked pretty good. For instance, this non-trivial schema seems to be handled fine (at least in the case of one repetition):
message schema {
    REQUIRED BYTE_ARRAY value1 (UTF8);
    OPTIONAL BYTE_ARRAY value2 (UTF8);
    REQUIRED INT64 value3;
    REPEATED group level1 {
        REQUIRED BYTE_ARRAY value4 (UTF8);
        OPTIONAL INT64 value5;
        OPTIONAL BYTE_ARRAY value6 (UTF8);
    }
}

Doc/example contribution could also be fair game (I was also thinking about adding something around ObjectStore + Parquet async -- I had a hard time figuring this one out but it actually works really well and I think I understand that part enough now to expand the docs).

I can't promise speedy reviews, but I can try to review PRs. I would ask though if you plan on large feature work in this area you file tickets first to get feedback on what you propose.
That being said, I want to just set expectations that these APIs will always be orders of magnitude slower than their columnar brethren. There is also a huge amount of subtlety to correctly handling nested schema in all the weird forms it comes in, which the current row readers don't currently handle. If you want a hobby project to work on, happy to help, but if you're looking to base an application around these APIs I would strongly encourage using the arrow interface

Sounds fair. I am going to run tests on more datasets to see if the scenario can be unblocked through a targeted fix, and if not I'll resolve as Won't Fix. The call to get_row_iter() happens in a codebase I do not own and switching it to arrow seems unlikely. Thanks for the speedy replies, and all the work everyone has done on that crate. Very appreciated.

To get started, I sent PR #5093 to expand the crate docs with a summary of the discussion here and in #2394.

@tustvold : would you be able to take a look at PR #5102 ? I tried to make the change minimal, with 2 lines of actual code change and the rest of tests.

Apologies, it is on my list, but I've been a bit swamped recently and I need some time to sit down and learn how that code is working before I can review effectively

label_issue.py automatically added labels {'parquet'} from #5093


If IDL files fail to parse, output the pegged error message.

…instead of just announcing "failed to parse".
A necessity if you're changing WebIDL by hand (I'm not, but none the less).

The reason I didn't output the message is because it was the same every time, and wasn't much of a help.
Still, lets add it. It might be an incentive to get better error messages from pegged.


bug fix fold constant rule can't handle variable expr

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


[EMB-137] Add loader to file-renderer (and TypeScripted it)

Purpose
Add a loading indicator to the file-renderer (used on the file detail page).
Summary of Changes

converted file-renderer component to TypeScript w/ decorators
added loading indicator and styled it to float behind file iframe

Side Effects / Testing Notes
Use a large file to see the loader.
Ticket
https://openscience.atlassian.net/browse/EMB-137
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

Passing build: https://travis-ci.org/jamescdavis/ember-osf-web/builds/359025259

Coverage decreased (-0.3%) to 20.842% when pulling 24871449af6124f705cf2b14037dc9961beb2915 on jamescdavis:EMB-137_quick_files_Add_loading_indicator_to_file_detail_page into d7d2dfbff7113530a40dc12c2de5d85c30f5c718 on CenterForOpenScience:develop.


fix: openstack scsi data volume name

When using SCSI volumes in OpenStack the volumes can have a mixed order which sometimes results in a data volume called /dev/sda even if OpenStack says it's available at /dev/sdb. The partitioning defined in the cloud-config file has no effect in this case and no persistent data partition is created.
To be on the safe side, we should format the device using the ID of the OpenStack volume.

I'll check if we can use the same approach for a non-scsi data volume (/dev/vdb).


Migrate error_pages.test.ts to North Star

Description
Please explain the changes you made here.
Release notes
Remove this section if the title of the pull request can be used as the default release notes description. If more detail is needed to communicate to partners the scope of the PR's changes, use this release notes section (and remove this placeholder text).
Checklist
General
Read the full guidelines for PRs here

[ ] Added the correct label: < feature | enhancement | bug | under-development | dependencies | infrastructure | ignore-for-release | database >
[ ] Assigned to a specific person, civiform/developers, or a more specific round-robin list
[ ] Added an additional reviewer from outside your organization as FYI (if the primary reviewer is in the same organization as you)
[ ] Removed the release notes section if the title is sufficient for the release notes description, or put more details in that section.
[ ] Created unit and/or browser tests which fail without the change (if possible)
[ ] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[ ] Extended the README / documentation, if necessary. For user-facing features, consider updating the user docs. For "under-the-hood" changes or things more relevant to developers, consider updating the dev wiki.
[ ] Ensured PII wasn't added to any new logs, unless it was guarded by isDevOrStaging

Database evolutions
Read the guidelines here

[ ] Assigned two reviewers
[ ] Guarded against already existing resources using IF NOT EXISTS and IF EXISTS
[ ] Downs created to undo changes in Ups
[ ] Every comment in script should begin with -- and not # --- unless it denotes Ups or Downs. See here for details.
[ ] Tested both the Downs and the Ups scripts manually (When testing, include all comments from the evolution in your test script to ensure any syntax errors in the comments are caught.)
[ ] Data migrations aren't being done (please use a Durable Job if doing a data migration)
[ ] Update the model documentation in our wiki if necessary

Durable jobs
Read the docs here

[ ] Assigned two reviewers
[ ] Included a rollback plan and a job to undo the data changes if appropriate

User visible changes

[ ] Followed steps to internationalize any new strings

[ ] Added context strings to new messages
[ ] Didn't use a message in applicant facing code that isn't translated yet (unless behind a flag)

[ ] Wrote browser tests using the validateAccessibility method
[ ] Made equivalent changes in Thymeleaf for applicant-facing features and changes (or created an issue in the Northstar epic to track that it's missing from Thymeleaf)
[ ] Tested on mobile view. See mobile device mode
[ ] Manually tested at 200% size
[ ] Manually evaluated tab order
[ ] Manually tested with a screen reader if the feature is applicant-facing. See screen reader testing

New Features

[ ] Add new FeatureFlag env vars to server/conf/helper/feature-flags.conf
[ ] Conditioned new functionality on a FeatureFlag
[ ] Wrote browser tests with the feature flag off and on, etc.

Instructions for manual testing
If instructions are needed for manual testing by reviewers, include them here.
Issue(s) this completes
Fixes #<issue_number>; Fixes #<issue_number>...

#9206  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ssandbekkhaug and the rest of your teammates on  Graphite

Is this error page part of the North Star? If it isn't, then I question whether we need a new test file.

You're right, the old error page is not part of North Star. According to #8873 we're going to keep the old error pages until the new page(s) are ready in M3.5. At that time, we should create new tests for the new error page(s).
Should we close https://github.com/civiform/civiform/issues/9074?

Is this error page part of the North Star? If it isn't, then I question whether we need a new test file.

You're right, the old error page is not part of North Star. According to #8873 we're going to keep the old error pages until the new page(s) are ready in M3.5. At that time, we should create new tests for the new error page(s).
Should we close #9074?

Yes, I think so.  I just closed it with a note that the error page isn't specific to the North Star yet.


Adding a "delete project" function to the CLI.

I created a cli/delete_project.py which makes it possible for me to create projects, run them. Once I've checked the results, I can then run another script to delete all of those projects, as expected by the interface.

I mistakenly used my master branch for this one. I think it will not work well. Removing. Creating another PR in a sec.


Use ObjectManager interface to allow for decorated EntityManagers

We decorate the EntityManager. When doing so the symfony console command doctrine:generate:form or generate:doctrine:form complain of being passed an object of the wrong type.
With this change the form is properly generated without issue.
I should note this isn't an issue in doctrine 2.4+ since the EntityManagerInterface is type hinted instead.

Doctrine ORM 2.3.x is EOL and won't be patched anymore unless for security-related issues.


Impro 740 remove automatic adjustment

This PR is the product of IMPRO-740, which calls for the removal of the automatic adjustment of neighbourhooding radius when working with ensemble forecasts. This adjustment is being set within the suite.
Testing:

[x ] Ran tests and they passed OK
[x ] Added new tests for the new feature(s)

Note also that several nbhood acceptance tests appear to be failing, which suggest something somewhere is not quite right (or was previously not quite right).
e.g.
[08:39:01] ..../tests/improver-nbhood/07-external-mask-square.bats ...............................
1..1
not ok 1 nbhood 'square' --radius=20000 input output
# (from function `improver_compare_output' in file ..../improver/tests/lib/utils, line 52,
#  in test file ..../improver/tests/improver-nbhood/07-external-mask-square.bats, line 48)
#   `improver_compare_output "$TEST_DIR/output.nc" \' failed
# DIFFER : VARIABLE : thickness_of_rainfall_amount : POSITION : [0,0,132] : VALUES : 7.81273e-06 <> 0
Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests


bug with event dispatcher and BlOverlays

See: https://github.com/OpenSmock/Pyramid/issues/104

It is working


transitions Section Ignored When Trying to Change Validators in a Network That Uses IBFT 2.0 With Besu v24.8.0

Description
Using the instructions described here https://besu.hyperledger.org/24.8.0/private-networks/how-to/configure/consensus/add-validators-without-voting, we are trying to change the validators in a network running with IBFT 2.0 consensus. The nodes are all restarted (tried both rolling update and all at once) before the network reaches the block number in which the change should take place, and still, the change is ignored.
Acceptance Criteria
The validators in the network should change at the specified block number.
Steps to Reproduce (Bug)

Start network with 6 nodes, 4 of them are validators and 2 are regular members.
Update the genesis file to change the validators to 2 of the original validators and the 2 nodes that were previously regular members.
Restart the nodes.

Expected behavior: The validators should change at the specified block.
Actual behavior: Validators remain the same
Frequency: Every time we tested it
Versions (Add all that apply)

Software version: 28.4.0
Java version: What ever is used in your official docker image

Additional Information (Add any of the following or anything else that may be relevant)
Example of transitions configuration we used:
{
  "config": ...,
  ...
  "extraData": ...,
  "transitions": {
    "ibft2": [
      {
        "block": 44971000,
        "validators": [
          "0x528684eb5133dbf9ec3672a71e95b0e1cde8282d",
          "0xea80db5c770bdd377e04902a6d068ac850f35649",
          "0xc39f16eef58a60f203522706d5e6872ea0f9e4d5",
          "0x311730291af2fbd112cb778bfbaf16ead80fdada"
        ]
      }
    ]
  }
}

Any ideas on why it might get ignored would be very helpful!!

Misplaced the transitions section, it should be under config and not the same level.
Closing.


Source evaluation plots enhacenments

Flip RA in all plots, so it's from right to left.
Flux is actually Flux Density

Also change naming of the labels in the ra_dec_error_plot

Fixed it using invert_xaxis().


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1182 ms

Resolved: Bofrost.de performance has improved in 8134893 after 6 minutes.


Thumbs.db file error when loading extension

There is an error when loading the Chrome Extension that a file named "thumbs.db" is illegal.

Fixed in Version 1.3 branch.


Add inspection and insurance details

Todo

[x] Support for reading data out of the .fieldGroup class on the inspection and insurance pages
[x] Add inspection and insurance to AdditionalSearchSheets
[x] Add inspection and insurance to DetailsResult
[x] Read the inspection and insurance page in DmrService
[x] Add XML documentation
[ ] Test performance

Performance improvements are needed before merging.

Greatly improved DmrHtmlReader read performance by adding a ReadStradegy

Note: The CurrentInsurance gives an empty enumerable because it wasn't supported in the old/current version


John Leteys awesome icons

@johnletey Some open/old PRs from your old account in the feather repo have some awesome icons!
I think we can add them to Lucide.
Here is a list of the icons I think are welcome to add to the Lucide repo!

[ ] Currency icons pr
[ ] code-2 icon pr
[ ] file-check, file-times , folder .. pr
[x] webcam pr

More PR from John

Also the file-check, file-times , folder .. pr is already resolved :)

Checked it


crossjoin/Browscap is worth a look

I've found another browscap parser: https://github.com/crossjoin/Browscap
It is faster and less memory usage than browscap-php. Worth a look.
My benchmark at PHP 5.5.11 (Linux 32bit)
browscap-php:       time 0.13754391670227,    memory 8388608
crossjoin-browscap: time 0.00087594985961914, memory 524288

Adding https://github.com/yzalis/UAParser to the list

Just wanted to note, that a stable version 1.0.0 of Crossjoin\Browscap has been released in the meantime.

Good to know, thanks.

I've updated my benchmarks: https://github.com/kenjis/user-agent-parser-benchmarks

parser
time
peak memory

get_browser()
59.665177
524288

browscap-php
4.9851598
50069504

crossjoin\Browscap
4.6786639
1310720

Previous benchmarks did not suit for most use cases, because they parsed multiple user agents with single instance. See https://github.com/kenjis/user-agent-parser-benchmarks/issues/1 , if you want details.


Fix dates to force correct format

The dates were being formatted in a way that google wasn't happy with. This commit uses the specced format.

Thanks! Could we have the time also, e.g. '2004-12-23T18:00:15+00:00' like in http://www.sitemaps.org/protocol.html ? Thanks in advance :)

Done.

Thanks :)

Hey Lasse,
Thanks for merging all my pull requests! Is there any chance you could do another release with these changes? I'd love to no t have to use my fork any more.
Cheers,
Tom

Hey Tom,
Thanks for all your pull requests, they really are a great help.
I just discovered that the %:z part in strftime("%Y-%m-%dT%H:%M:%S%:z") doesn't work. It just returns :z. WIthout the colon it works fine but without colon between the hour and minutes.
What to do?
/Lasse

Ahh, I think this is a ruby 1.8 thing (I'm on 1.9).
You could try adding .to_datetime before calling .strftime, ie.
page.last_mod.to_datetime.strftime("%Y-%m-%dT%H:%M:%S%:z")

And see how that goes? (seems to work for me on a 1.8 console at least).
Cheers,
Tom
On 07/01/2012, at 8:29 AM, Lasse Bunk wrote:

Hey Tom,
Thanks for all your pull requests, they really are a great help.
I just discovered that the %:z part in strftime("%Y-%m-%dT%H:%M:%S%:z") doesn't work. It just returns :z. WIthout the colon it works fine but without colon between the hour and minutes.
What to do?
/Lasse

Reply to this email directly or view it on GitHub:
https://github.com/lassebunk/dynamic_sitemaps/pull/6#issuecomment-3391137


Required reason APIs

Hi, a new Apple policy is coming in the fall, tell me, do you plan to add required reason APIs?
I saw that you have 2 API categories in Pulse
File timestamp and User Defaults.

Yes, I'm planning to address it once it becomes mandatory. I hope Apple reverses course.

Any update on this please?

@kean  it is mandatory now ...


Subtitles

Not really a bug but curious if it would be possible for you to add a toggle in settings to disable subtitles by default? It's frustrating that subtitles turn on automatically every time I start something.

I'm planning on changing that behavior. Meanwhile you should be able to disable that in advanced settings).

Some advanced features can be enabled or configured in settings. To access it, long press the ⚙ gear icon. (Alternatively, you can also enter this settings from App info screen.)

I'm planning on changing that behavior. Meanwhile you should be able to disable that in advanced settings).

Some advanced features can be enabled or configured in settings. To access it, long press the gear gear icon. (Alternatively, you can also enter this settings from App info screen.)

I think it should be kept as it is. It's probably more convenient for most users. The respective advanced setting is just fine.

@VasilisKos I will disable automatic selection only for default subtitle tracks if system captioning preferences are turned off. This should make it work just like YouTube. Forced subtitles will still be always selected.

@VasilisKos I will disable automatic selection only for default subtitle tracks if system captioning preferences are turned off. This should make it work just like YouTube. Forced subtitles will still be always selected.

I think it'll be even better, if only the subs of the language that is chosen as default in the app settings are loaded automatically.


DELETE action auditing fails on pre-existing entities

I've enabled the JHipster auditing module on a pre-existing application.
When a pre-existing entity gets deleted without being subject to any prior modification, so without having its lastModifiedDate and lastModifiedBy fields valued by a preceding update action, the EntityAuditEventRepository fails to save the EntityAuditEvent for the delete action.
The auditing framework relies on the fact that lastModifiedDate and lastModifiedBy properties have values in the audited entity, when it's time to persist the audit event, but those properties are null in the situation described above.
This is the corresponding console error:
2020-09-15 12:33:21.236 ERROR 2200 --- [ bludesk-task-2] o.h.i.ExceptionMapperStandardImpl        : HHH000346: Error during managed flush [Validation failed for classes [it.blutec.bludesk.domain.EntityAuditEvent] during persist time for groups [javax.validation.groups.Default, ]
List of constraint violations:[
	ConstraintViolationImpl{interpolatedMessage='must not be null', propertyPath=modifiedDate, rootBeanClass=class it.blutec.bludesk.domain.EntityAuditEvent, messageTemplate='{javax.validation.constraints.NotNull.message}'}
]]

This happens because the audit entity modifiedDate and modifiedBy properties gets initialized with the aformentioned entity properties, which are null since the entity never changed before.
I think that the delete action audit entity should have modifiedDate and modifiedBy properties valued "dynamically", based on the current timestamp and principal, rather than copying the values from the entity.

I came across the same problem. Is there a reason, why the fields created_by an created_date have default values in the liquibase addColumn section and last_modified_by and last_modified_date don't? In my opinion the easiest solution to the problem would be to assign them the same default values as for CREATE.

I came across the same problem. Is there a reason, why the fields created_by an created_date have default values in the liquibase addColumn section and last_modified_by and last_modified_date don't? In my opinion the easiest solution to the problem would be to assign them the same default values as for CREATE.

Closing, since we migrate to a new blueprint version.
If you found any issue with latest version, please open a new issue.


Blur support for Android

Would be really happy to set blur for Fast Image on Android 😌

While waiting for the official support of this feature, I patched the library this way to get a "Blur" effect and a "Pixel" effect:

Add this dependency to build.gradle: https://github.com/wasabeef/glide-transformations
Pass extra property blurRadius: number and pixel: number to the source object (not happy with this but it was the quickest way for me)
Apply transformations inside method getOptions in file FastImageViewConverter.java:

if (blurRadius != null) {
            options = options.transform(new BlurTransformation(blurRadius, 3));
  } else if (pixel != nul) {
      options = options.transform(new PixelationFilterTransformation(pixel));
  }

While waiting for the official support of this feature, I patched the library this way to get a "Blur" effect and a "Pixel" effect:

Add this dependency to build.gradle: https://github.com/wasabeef/glide-transformations
Pass extra properties blurRadius?: number and pixel?: number to the source object (not happy with this but it was the quickest way for me)
Apply transformations inside method getOptions in file FastImageViewConverter.java:

if (blurRadius != null) {
    options = options.transform(new BlurTransformation(blurRadius, 3));
}

if (pixel != null) {
    options = options.transform(new PixelationFilterTransformation(pixel));
}

oh, thank you! I will try 🙂

what about the ios looks like https://github.com/HyopeR/react-native-fast-image is working for few people can't we include it in official release??


feat: add gpt-4-turbo support

Two new models, 'gpt-4-turbo' and 'gpt-4-turbo-preview', have been added to the modelInfo array in the constants file. Both have a maximum prompt length of 128k tokens.

included in 0.1.8


Dependabot throws an unknown error for expanded dependency specification syntax in Poetry

Context
Poetry supports an expanded dependency specification syntax for defining additional dependency attributes.
While the Python ecosystem can correctly identify dependencies listed with this syntax and attempt to update them, the update fails:
/home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater/poetry_file_updater.rb:82:in `block in updated_pyproject_content': Content did not change! (RuntimeError)

This is because the parser for the manifest file, TomlRB, adds dependencies defined using the expanded syntax to the dependencies hash as an object rather than a key. Thus, regex that normally detects a version change fails on the object, throwing the error.
Package ecosystem
Python/Poetry
Manifest location and content prior to update
path: /pyproject.toml
contents

[build-system]
requires = [ "poetry>=0.12",]
build-backend = "poetry.masonry.api"

[tool.poetry]
name = "downforeveryone"
version = "1.0.0"
description = "checks if a website is really down via isup.me"
authors = [ "Ryan Delaney <ryan.patrick.delaney@protonmail.com>",]
readme = "README.rst"
repository = "https://github.com/rpdelaney/downforeveryone"
homepage = "https://pypi.org/project/downforeveryone"
classifiers = [ "Environment :: Console", "License :: OSI Approved :: Apache Software License", "Programming Language :: Python :: 3 :: Only",]

[tool.black]
line-length = 79
target_version = [ "py38",]

[tool.isort]
combine_star = true
indent = "    "
length_sort = false
line_length = 79
multi_line_output = 3

[tool.poetry.dependencies]
python = ">=3.7, <4"
requests = "^2.22"

[tool.poetry.dev-dependencies]
mock = "^4.0"
pytest = "^6.2"
pytest-cov = "^2.11"
pytest-mock = "^3.5"
flake8 = "^3.9"
black = "20.8b1"
mypy = "^0.812"
flake8-bugbear = "^21.4"
flake8-docstrings = "^1.6"
flake8-breakpoint = "^1.1"
pytest-icdiff = "^0.5"
toml = "^0.10.2"
responses = "^0.13.2"
semver = "^2.13.0"

[tool.poetry.scripts]
isup = "downforeveryone.isup:main"

[tool.pytest.ini_options]
console_output_style = "progress"
addopts = "--cov=downforeveryone"
mock_use_standalone_module = true

[tool.poetry.dev-dependencies.isort]
version = "^5.4"
extras = [ "pyproject",]

Updated dependency
Bumping isort from v5.7.0 to v5.8.0
What you expected to see, versus what you actually saw

Expected Dependabot to upgrade isort from v5.7.0 to v5.8.0.

Received a traceback for poetry_file_updater.rb

traceback
=== isort (5.7.0)
=> checking for updates 16/16
=> latest available version is 5.8.0
=> latest allowed version is 5.8.0
=> requirements to unlock: own
=> requirements update strategy: widen_ranges
=> updating isort from 5.7.0 to 5.8.0
Traceback (most recent call last):
	10: from bin/dry-run.rb:594:in `<main>'
	 9: from bin/dry-run.rb:594:in `each'
	 8: from bin/dry-run.rb:682:in `block in <main>'
	 7: from /home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater.rb:32:in `updated_dependency_files'
	 6: from /home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater.rb:92:in `updated_poetry_based_files'
	 5: from /home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater/poetry_file_updater.rb:33:in `updated_dependency_files'
	 4: from /home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater/poetry_file_updater.rb:50:in `fetch_updated_dependency_files'
	 3: from /home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater/poetry_file_updater.rb:67:in `updated_pyproject_content'
	 2: from /home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater/poetry_file_updater.rb:67:in `reduce'
	 1: from /home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater/poetry_file_updater.rb:67:in `each'
 /home/dependabot/dependabot-core/python/lib/dependabot/python/file_updater/poetry_file_updater.rb:82:in `block in updated_pyproject_content': Content did not change! (RuntimeError)

🕹 Bonus points: Smallest manifest that reproduces the issue
[tool.poetry.dev-dependencies.isort]
version = "^5.4"
extras = [ "pyproject",]

Reproduce
bin/dry-run.rb pip "rpdelaney/downforeveryone" --dir="/." --commit=8729649e0552284de145a9a4b840b14a1f99d941 --cache=files

Related issues:

#2855
#2715

Related:

#3290


[QUESTION] How to delete widget from widget list?

Hi, this is a bit of a weird question...
I have been testing out the app and so far absolutely loving it!
I have added several widgets from the Homescreen and later deleted them from the home screen as well. Now they are still being listed in the My Widgets list.
I've tried long tap to delete but I don't have an option there and when selecting edit there isn't an option either which would let me remove it from the list. Am I missing something or is that not a function yet?

Hi @Der-Flip, thanks for reaching out and I'm glad to hear you're liking the app so far!
Have you used different launchers and placed widgets on them by any chance? That list displays widgets placed on all launchers, not only your active one.
There's no option to delete a widget from within the app because that action can only be taken by the user from the home screen, so apps cannot do it. You have two options to work around the issue you're seeing:

Try editing them, adding a photo and then placing them on the home screen. Next you should be able to remove them if you so wish;
Alternatively, you can long press the app icon and tap to view app info. Look for the storage option and clear the app storage. Beware that this will delete all current widgets and preferences (like widget defaults), so it can be a bit inconvenient

Let me know if either of these work.

Hi, thanks for responding
So I tried adding a new source / photo to the one displayed in red, then confirming at the bottom by placing on home screen, but they never appeared on the home screen. For testing purposes I then created another new widget, confirmed it showed up in the my widgets list, then deleted and it was removed from the list as well. So something about the earlier ones seems to be bugged. I remember creating the widgets before having opened the app itself, could that be connected?
I then tried to delete all data (Settings, Apps, manage apps, Material Photo Widget, clear data, clear all data). Contrary to expectations this did not remove the widget that was (is) currently on the Homescreen but in the app they are all shown with the red 'missing' icon.
I only ever used the phone as stock as it was (android 10, miui12). (First android after blackberry meant to be a temporary phone, wanted to experiment once I had a new primary daily driver).
I'll try reinstalling next. :'D
Thanks for your help though!

@Der-Flip thanks for sharing the details. The app has some issues with MIUI but I haven't gotten to the bottom of it as I don't have a device with it myself, making it tricky to troubleshoot.

I remember creating the widgets before having opened the app itself, could that be connected?

By this you mean creating the widgets directly from the home screen, correct? That's interesting to know. So far I thought that the issues were tied to creating a widget from the app and requesting to add them to the home screen.

I then tried to delete all data (Settings, Apps, manage apps, Material Photo Widget, clear data, clear all data). Contrary to expectations this did not remove the widget that was (is) currently on the Homescreen but in the app they are all shown with the red 'missing' icon.

That's interesting as well and appears to be particular to MIUI as other ROMs do not behave like that. When clearing the app data all placed widgets would be removed. I'll try researching this a little bit to see if there's anything I can do on my end.
Good to hear that reinstalling works, at least that did the trick 😅

By this you mean creating the widgets directly from the home screen, correct?

Correct basically

Install app
Tap and hold home screen
Select "add widget"
Add widget.

All of this without opening the app after install and before adding a widget.
I haven't done much more experimenting yet tbh. If you need any more details I'm happy to see if I can help, just note, I'm not super savvy, so if you need me to check something, please take it slow and eli5 :'D
Tbh, I don't really mind the issues too much, as I will only have one widget anyways so if I need to reinstall every once in a while (once in several years) to clear the my widgets list, I can perfectly live with that :'D


[NFC] Start of support for multiple hashes.

Note: this also fixes build issues for Metal, which no longer runs anyway due to running out of registers.

Abandoned in favor of another approach.


[InfiniteScrollingLocal UI Test] Type filters in Evidence view have become slow

This is a manual UI test based on this template. Other instances of this test are here. Tests from the template that aren't mentioned hereby are intended as passed.
Testing the evidence table
...

Filter on Genes, 331 genes are filtered, as the icon shows
Select Trait and ProteinDomain, 36 rows are filtered (as the numbers in the icons)

These still work, but have become slow in reacting. Perhaps, it's rendering the whole table every time, instead of the current window only? There is a TODO comment on the code possibly related to this.

Please add bugs (and tickets) which need to be fixed as part of a version to their appropriate projects...

Lawal has improved this, but it is still a bit slow. We decided to opt out of fixing it entirely, and instead it can be dealt with in Nova. The functionality does at least work, although a bit slow at times when a large amount of types are being filtered on.
Thanks :)


add non executed transactions to the indexer

Time spent on this PR:
Part of #1111
Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Testing

What is the new behavior?

As we need, at least at the start of the testnet, more visibility on our transactions coming out of the sequencer, the filter on transaction_executed event is removed and integrated directly into the transform of the indexer to ensure that all executed transactions are stored in the main collections (transactions, logs, receipts) and failures in separate collections.
Once we have this indexing achieved, it would be more convenient to deal with failures at the rpc level according to the error messages on which we wish to provide user feedback (like the run out of resources recently).
I made the choice, as we are in the start-up phase of the testnet, to store all transactions, I think that it will be useful to us at least initially to improve the feedback, we will always be in time to restrict the filtering whenever we want.
Does this introduce a breaking change?

[ ] Yes
[ ] No

let me know when you want to try this out @tcoratger, we can PP


work.ink not bypassed

Domain
https://work.ink/
Link
https://work.ink/en/l/Xi/JailbreakProjectEvolution
Version
0.0.0
What browsers are you seeing the problem on?
Firefox
What OS are you seeing the problem on?
Windows
(Optional) Anything else?
No response
Code of Conduct

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

Closed because it's a duplicate issue of #284.


Johns Hopkins: contradictory record counts

This relates to the general question in #753, but is more specific:
The default stream for Johns Hopkins has these numbers:
Records: 2,065,546
Unique records: 1,700,.240
There are 27 files in the stream, of which 15 are labelled "Valid MARC".  The individual totals for those 15 add up to 1,700,240.
But the providers index page has these numbers:
Johns Hopkins
Files: 27
Unique records: 0
Total records: 0
I want to go back to Johns Hopkins and ask whether this is a full set of their records.  (They have over 2 million in their local catalog.). Before I do, though, I want to be sure the records they've already sent were successfully ingested.  If they were, why does the providers page list 0 records?

@bobpersing these statistics are generated in a background job that runs once a day. I kicked off the job manually for JHU and now the counts are appearing as expected on the providers page. I'll do more digging to see if I can figure out why they weren't generated during the daily job.

@bobpersing I found that the organization statistics (the ones shown on the page that lists all the providers) are a day behind the statistics shown on the individual provider/default stream page. This was due to the bug in the stats processing code. I've put in a pull request with a fix for this so the stats should be more consistent once this change is rolled out. We're also considering adding some messaging near the stats to indicate they're calculate daily and do not reflect the most recent uploads.

Also, the generate stats job successfully completed and the stats are now displaying and up to date for Johns Hopkins.


Added new article under learn about Sitecore XM item types

Description
Added a new article called Sitecore XM item types. I added a new section and link to an article on the Experience Manager page.
Motivation
Helpful content for learning Sitecore XM.
How Has This Been Tested?
Verified page rendered correctly in dev and that link displayed on Experience Manger page.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation update (non-breaking change; modified files are limited to the /data directory or other markdown files)

Checklist:

[x] I have read the Contributing guide.
[x] My code/comments/docs fully adhere to the Code of Conduct.
[ ] My change is a code change.
[x] My change is a documentation change and there are NO other updates required.
[ ] My change is a documentation change and it requires an update to the navigation.
[ ] My change has new or updated images which are stored in the /public/images folder that need to be migrated to Sitecore DAM

Great article Thomas! Long overdue that someone actually wrote down the definitions of the different item types. I removed the title from the article text itself since its also visible in the header


Expiry for Stock Items

Considers any item with an expiry that is past today will have an EXPIRED status and it to be not available for sale.
Any item whose expiry is within 30 days will have a status of STALE and but is otherwise not changed.
Created in collaboration with @Simage, that is, he wrote it months ago and I have code janitored it into submission :smile:

Hey @nwns this is a great idea! I've got a few additional thoughts below:
Status Field
The existing status field should not encompass the "staleness" state of a StockItem - it is really for the physical state of the stock item. Storing the expiry date of the stock item is already enough data to determine if the item is "stale" or "expired".
As you have the date stored, it can always be evaluated at run-time to determine if the particular stock item is stale (or not). In the API we can add a new boolean field called 'stale' (for e.g.) which is computed at run-time for each object in the query.
Default Part Lifetime
If a stock item can "expire" I think it would make sense for the Part model to have a corresponding field "stock expiry duration" (or similar). This could be specified in units of days, for example. Whenever a new stock item is created, if the corresponding part has a default expiry duration, that is added to the current date and used as the default value.
For example, stock of ACME Widget expires 30 days after creation by default.
Filtering
The stock table can be updated to add a filter for "stale" stock - allowing the user to quickly determine what stock is out of date.
Let me know your thoughts on the points above. Cheers.

I believe I added the "staleness" to the status for a couple reasons.

First I have to disagree with you, I believe 'staleness' is a physical state, akin to destroyed, a moldy loaf of bread for instance is expired, If I'm looking at my inventory and see a "stale" loaf that I know is about to expire, I may (should?) prioritize its use.
Semantically it makes sense to display a product as "Stale" and/or "Expired" rather than "Available"

That having been said, My largest concerns with the way the status is being handled are two-fold

The state is generally set by the user, The current implementation silently overwrites and discards the user set state
The state is set during object instantiation by the ORM, This just feels really hacky

Default part lifetime and filtering both make sense to me,

After discussion with @Simage, I think the best way to get this merged would be to:

[ ]  Add the stock item api to return a set of statuses, which may include Expired/Stale
[ ]  Display only statuses when viewing stock items
[ ]  Per Part default stale time (in days)
[ ]  Per Part default expired time (in days)
[ ]  Add to the Global Settings framework, Do not allow sale of expired Items.
[ ]  If Global Setting set and if an expiry is defined and the StockItem is expired, do not allow the StockItem to be sold
[ ]  Add an index on expiry field
[ ]  Enable the stock table to filter on expiry

@SchrodingersGat would this work?

The list of points above make sense. I would also suggest the following items:

[ ] List "expired" stock on the index page
[ ] Calculate "default" expiry date for StockItem objects based on the "expired time" of the Part

In terms of how the expiry data are stored - in general it is bad practice to store calculated values in a database - especially when the calculations are very simple.
Here, the calculated value is "has this stock item expired". As the expiry date is a single static datefield, it can be inexpensively compared to the current date. All the data we need to determine if an item has expired (or is about to expire) is already there.
Also, the "expired" state is orthogonal to the "physical" state. For example, a StockItem could be damaged, but not out of date, or destroyed, and also out of date. Overloading the existing status field means that these combinations cannot be encoded.
To accept, this is what I would like to see:

[ ] Add an "is_expired()" function to a StockItem which simply compares the expiry date to the current date
[ ] Use this function to display expiry status on StockItem detail page
[ ] Remove extra status codes from StockStatus class
[ ] Remove after_init_stock_item hook which updates the database model

If you can address these four items above, I'm happy to merge and then work on the other improvements (such as default Part expiry, table filtering, etc).
Thanks!

Hey @nwns please check out the implementation made in #1202  :)

Hey @nwns please check out the implementation made in #1202  :)

@SchrodingersGat Thank you.  The implementation works well.


fix: cap multiplier to 1

Resolves #302

oh.
does it mean to just output a warning if someones sets a multiplier more than 1?

ill assume that as there werent any replies

@pavlovcik is this what you meant?


Add Category to provide filename or translated filename

Re: https://github.com/ResearchKit/ResearchKit/pull/324
Re: https://github.com/ResearchKit/ResearchKit/issues/323
Add a method in a category on ORKFileResult to provide a filename derived from the concatenation of the ORKResult identifier and ORKStep identifier.
Provide for translation of the derived filename via FilenameTranslation.json.
This would serve our purposes for recovering the original filenames in Parkinson and Cardio without adding the step identifier back into the logName in ORKRecorder.

I assume FilenameTranslation.json and the method caller will be in the apps?

@YuanZhu-apple  Yes, as needed. Sage has a filename schema which we need to follow, else they will have fragmented data in their database. This will also help insulate the app from further changes to the url which we are currently using to identify the file and form a filename.

@YuanZhu-apple  To clarify further on your question about the caller.
Currently we have an APCDataArchiver class which includes a friendlyFilenameForFile() method for filename translation via regex. I am currently refactoring / deprecating this class by moving the Parkinson specific code into subclasses of APCBaseTaskViewController and using my encapsulated upload classes to upload. The filename translation will be initially be called from APCDataArchiver, and after refactoring from APCBaseTaskViewController.

r=me.


init infra cluster controller

Signed-off-by: hejianpeng<EMAIL_ADDRESS>What type of PR is this?

/kind feature
What this PR does / why we need it:
Which issue(s) this PR fixes:
xref #145
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/lgtm


Added new rebus

Associated Issue: #
Summary of Changes

change 1
change 2

Pull Request Test Coverage Report for Build 897

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

Totals

Change from base Build 514:
0.0%

Covered Lines:
158

Relevant Lines:
180

💛  - Coveralls

If u are ok with fix can u please add hactoberfest-accepted label to it ???