mathoid: init at 0.7.2

mathoid is a service that renders mathematical formulae on the
server-side. As such it can be used to e.g. prerender MathJax formulae
rather than relying on doing so on the client. This is very useful if
you want to include them in a non-webpage format.
Of course one does not just Add A Single Node Package. I ran generate.sh and it updated various things a little bit, but I think that should be fine.

Motivation for this change
Things done

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

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

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

isn't it possible to just split into different commits the mathoid update from the rest  via git add -p for instance ? would make reverting this peculiar package easier or the node update easier. You may even not want to include the rest of node updates.

That would require me to know which (transitive) dependencies are the mathoid ones. I'm not sure there's any way for me to know that?
FWIW this seems to be what's happened historically when people have added new node packages.

another way would be to run an update first, commit and then rerun the update with your commit. One could assume no update in-between.
If node maintainers are fine without the change, then that's ok, I was just trying to increase your odds :p

I'm no longer using this.


Add global variable definition improvement

Note that, with this implementation, there may be issues with concurrent access of global variables. we may need to support a approach for that as well.


CI: retrieve keys for LLVM repo

Signed-off-by: Daniel Maslowski<EMAIL_ADDRESS>

yea I'll make it "will be" ;)


Android Explanation Text Size

The explanation text is crowding the edges (left edge in example) of the artboard for Android, I had to create a new symbol for Android text using 75px font, instead of using the iPhone 6 Plus settings (85px).
For the 'Android 2' artboard, the text was bleeding off the screen, didn't notice it happening on other Android artboards.
Example of issue:

Yes, that's why android ones are considered a bonus set. It's not perfect, but hopefully helpful to some folks


Created a new tool for creating Incidents with email

A detailed explanation is here.
https://github.com/samikroy/Azure-Sentinel/blob/patch-32/Tools/Create Incidents with Email/Readme.md
This will be a part of the readme after the PR merge.

@v-mchatla  - Request your help for the approval.

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva is checking with @shainw.

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

@rushriva : Please have a look and provide an update. Thanks!

Hi @samikroy  -
Thank you for this content. Based on the functionality I see this is automation playbook and can we move this to Playbook folder, so that we can get into Content hub subsequently.  While moving, can you please add following elements, so that it will be ready for content hub

Metadata section:

You can find the instructions here - https://github.com/Azure/Azure-Sentinel/tree/master/docs/New Playbooks Contribution Guide#contribution-guidelines
For examples you can refer this playbook https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Okta Single Sign-On/Playbooks/OktaPlaybooks/Okta-EnrichIncidentWithUserDetails/azuredeploy.json

As a best practice, we recommend sentinel connection in playbooks uses "ManagedSecurityIdentity".
Please refer Sample Template for sample template. For more details, refer this. Make sure to do changes at 3 places

@samikroy : Please address the @anki-narravula comments.

@anki-narravula : Please have a look and provide your feedback @samikroy addressed your comments.

@anki-narravula @v-spadarthi - Please have a look at the updated code and share your reviews !

@samikroy  Metadata object is still missing, example one

@samikroy : Please address the @anki-narravula comments.

@samikroy : Please address the @anki-narravula comments.

hi @samikroy,
Please add metadata object to your playbook, so that @anki-narravula can review and merge your changes.
Thanks

@v-mchatla - Could you please help with the merge.

@anki-narravula , @v-spadarthi & @v-mchatla - Could you please help with the merge as the comments are addressed now.

Hi @anki-narravula,
Author has addressed your comments. Can you please review and provide your approval.
Thanks

@v-mchatla , @anki-narravula, @v-spadarthi - Could you please help with the merge.

Hi @samikroy -
Thank you for this content. Based on the functionality I see this is automation playbook and can we move this to Playbook folder, so that we can get into Content hub subsequently. While moving, can you please add following elements, so that it will be ready for content hub

Metadata section:

You can find the instructions here - https://github.com/Azure/Azure-Sentinel/tree/master/docs/New Playbooks Contribution Guide#contribution-guidelines
For examples you can refer this playbook https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Okta Single Sign-On/Playbooks/OktaPlaybooks/Okta-EnrichIncidentWithUserDetails/azuredeploy.json

As a best practice, we recommend sentinel connection in playbooks uses "ManagedSecurityIdentity".
Please refer Sample Template for sample template. For more details, refer this. Make sure to do changes at 3 places

@samikroy , did you checked the 2-point mentioned by anki , regarding "managedserviceidentity" , As i am not able to see the changes being done,
Kindly check again

@samikroy: Please address the @manishkumar1991 comments. Thanks!

@samikroy: Please address the @manishkumar1991 comments. Thanks!

@samikroy: Please address the @manishkumar1991 comments. Thanks!

@samikroy: Please address the @manishkumar1991 comments. Thanks!

Hi @samikroy, we still waiting your response for @manishkumar1991 comments. Thanks

Hi @samikroy, I would appreciate it if you shared the details with @manishkumar1991 that he asks for. Thanks.

Hi @samikroy, we still waiting your response for @manishkumar1991 comments. Thanks

Hi @samikroy, please address the @manishkumar1991 comments. Thanks!

Hi @samikroy, we wanted to check on the status of PR https://github.com/Azure/Azure-Sentinel/pull/6409. PR is pending for more than 20 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation.

Hi @samikroy, we wanted to check on the status of PR https://github.com/Azure/Azure-Sentinel/pull/6409. PR is pending for more than 22 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation

Hi @samikroy, we wanted to check on the status of PR https://github.com/Azure/Azure-Sentinel/pull/6409. PR is pending for more than 24 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation

@v-atulyadav  & @manishkumar1991 - Please have a look the updated comment and revert for any queries.

Hi @manishkumar1991, @samikroy has responded on your comments please check and respond accordingly. Thanks.

Hi @manishkumar1991, @samikroy has responded on your comments please check and respond accordingly. Thanks.

Hi @manishkumar1991, check @samikroy's response to your comments. Thanks.

Hi @manishkumar1991, @samikroy has responded on your comments please check and respond accordingly. Thanks.

Hi @manishkumar1991, please provide your feedback. Thanks.

Hi @manishkumar1991, @samikroy has responded on your comments please check and respond accordingly. Thanks.

Hi @manishkumar1991, @samikroy has responded on your comments please check and respond accordingly. Thanks.

checking

@samikroy ,
Please address comments given by @anki-narravula regarding the use of "managed service identity " while making azure sentinel connection in playbook .
@v-atulyadav : Readme file changes has been taken care with minor correction needed, but first comments mentioned by anki is still not addressed .

Hi @samikroy -
Thank you for this content. Based on the functionality I see this is automation playbook and can we move this to Playbook folder, so that we can get into Content hub subsequently. While moving, can you please add following elements, so that it will be ready for content hub

Metadata section:

You can find the instructions here - https://github.com/Azure/Azure-Sentinel/tree/master/docs/New Playbooks Contribution Guide#contribution-guidelines
For examples you can refer this playbook https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Okta Single Sign-On/Playbooks/OktaPlaybooks/Okta-EnrichIncidentWithUserDetails/azuredeploy.json

As a best practice, we recommend sentinel connection in playbooks uses "ManagedSecurityIdentity".
Please refer Sample Template for sample template. For more details, refer this. Make sure to do changes at 3 places

@samikroy These comments still not addressed, can you please make the changes

Hi @samikroy, please addressed comments above. Thanks

Hi @samikroy, could you please take a look at the above comments. Thanks

Hi @samikroy, I would appreciate if you could take a look at the above comments Thanks.

@v-atulyadav & @manishkumar1991 - Have fixed the mentioned ones.
As there a lot of back and forth could you pease mention the exact error with line numbers mentioned to fast track this
Thank you.

@v-atulyadav & @manishkumar1991 - Have fixed the mentioned ones. As there a lot of back and forth could you please mention the exact error with line numbers mentioned to fast track this Thank you.

@samikroy :As PR approver, We can only suggest you good practice, so that good quality content gets merged in our branch.
I see that if you can scroll up and see that Anki from our team, has already told you on the first day of the PR itself, and mentioned what needs to be changed. If proper action has been taken at that time, we could not have gone back and forth too much.

Kindly change the AzureSentinelConnection with MicrosoftSentinelConnection at all places in your arm template.

Thanks
Manish Kumar

Hi @samikroy, please respond on remaining fixes suggested by Anki. Thanks

Hi @samikroy, please respond on remaining fixes suggested by Anki. Thanks

@v-atulyadav - Have address  the comments. Please let me know for any further changes .

Hi @manishkumar1991, @samikroy has responded to comments please check. Thanks

@samikroy  , , Changes are not done, as asked .
Kindly refer to me previous comments

@manishkumar1991 , @v-atulyadav , @v-dvedak  - Thank you for the your time throughout the PR timeline with the detailed reviews till the approval & merge.
Apologies for the back and forth while addressing the comments.


bonsai fails silently

I have some code like this:
    info!("Uploading memory image");
    let client = bonsai_sdk::alpha::Client::from_env()?;
    let img_id = {
        let program = Program::load_elf(PXP_BLOCK_ELF, risc0_zkvm::MEM_SIZE as u32)?;
        let image = MemoryImage::new(&program, risc0_zkvm::PAGE_SIZE as u32)?;
        let image_id = hex::encode(image.compute_id());
        let image = bincode::serialize(&image)?;

        client.upload_img(&image_id, image)?;
        image_id
    };

    let input_data = to_vec(&input)?;
    let input_data = bytemuck::cast_slice(&input_data).to_vec();
    let input_id = client.upload_input(input_data)?;

    // Start a session running the prover
    info!("Starting session");
    let session = client.create_session(img_id, input_id)?;

    info!("Bonsai session UUID: {}", session.uuid);

    loop {
        let res = session.status(&client)?;
        if res.status == "RUNNING" {
            tokio::time::sleep(std::time::Duration::from_secs(POLL_INTERVAL_SEC)).await;
            continue;
        }
        if res.status != "SUCCEEDED" {
            panic!("Workflow exited: {}", res.status);
        }

        // Download the receipt, containing the output
        let receipt_url = res
            .receipt_url
            .expect("API error, missing receipt on completed session");

        let _receipt_buf = client.download(&receipt_url)?;

        let snark_session = client.create_snark(session.uuid)?;
        let snark_receipt: SnarkReceipt = (async move {
            loop {
                let res = snark_session.status(&client)?;
                match res.status.as_str() {
                    "RUNNING" => {
                        tokio::time::sleep(Duration::from_secs(POLL_INTERVAL_SEC)).await;
                    }
                    "SUCCEEDED" => {
                        return res
                            .output
                            .ok_or(anyhow!("output expected to be non-empty on success"));
                    }
                    _ => {
                        bail!(
                            "SNARK proving session exited with bad status: {}",
                            res.status
                        );
                    }
                }
            }
        })
        .await?;

        return Ok(bincode::serialize(&snark_receipt)?);

it works for several times this morning but later this afternoon it just fails silently

[2023-10-24T12:16:08Z INFO  pxp_lib::prover] Uploading memory image
[2023-10-24T12:16:11Z INFO  pxp_lib::prover] Starting session
[2023-10-24T12:16:13Z INFO  pxp_lib::prover] Bonsai session UUID: 2df701ac-3003-4cff-91e7-0b4a665af903

I didn't find a way to generate snark receipt without using bonsai. Currently I'm just relying too much on bonsai and this is not good when it doesn't work well.

seems it's because that I try to use it to verify the proof and it failed, I'll double check

The sessionID 2df701ac-3003-4cff-91e7-0b4a665af903 completed successfully in 40 seconds (as well as all the SNARK workflows), additionally while bonsai is running you can check the state that the proof is in via:
        if res.status == "RUNNING" {
            eprintln!(
                "Current status: {} - state: {} - continue polling...",
                res.status,
                res.state.unwrap_or_default()
            );
            std::thread::sleep(Duration::from_secs(15));
            continue;
        }

I didn't find a way to generate snark receipt without using bonsai. Currently I'm just relying too much on bonsai and this is not good when it doesn't work well.

Currently STARK2Snark is only available via bonsai but we will soon be releasing the tooling into the open source codebase.
If a workflow fails you can also print the error message via:
            panic!(
                "Workflow exited: {} - | err: {}",
                res.status,
                res.error_msg.unwrap_or_default()
            );

Let me know if there is a specific error I can help with.

hi @mothran @flaub it's my error, proof is generated.

Currently STARK2Snark is only available via bonsai but we will soon be releasing the tooling into the open source codebase.
looking forward to it!


Fix flask-server script to be run through pipenv

If pipenv was selected during cookiecutter setup, then npm run flask-server should run pipenv run flask run. This commit changes package.json npm scripts accordingly.

Thanks!


[DA] chunk compression and multi-version supports

Add chunk builder: collect batch and compress batch in stream, output is bytes for making segments in different DA servers
multi version chunks: the major issue is the compression algorithm, the batch version could do the multi version chunk job. we could get first byte from segment to decide which way to decompress compressed chunk. Which means we share version between chunk and batch. For version_0: each batch is chunk without compression, version_1: chunk is 1 or more batches using the ZLIB algorithm (as specified in RFC-1950) with no dictionary
batch encode: bcs
chunk size: at starting the threshold will be 0, which means each batch will be a chunk avoiding blocking for too long time (in present, DA share the same actor with execution because of the tx buffer)

batches -> chunk(multi version + compression) -> segments(multi version) has been implemented


Background in center layout not shown

Describe the bug
I'm starting to learn how to use slidev and am following the guide. For some reason the background image is not show using the center layout as the example shows, ie if I do:
---
layout: center
background: '/background.png'
class: 'text-white'
---

However both:
---
layout: image
image: '/background.png'
class: 'text-white'
---

and
---
layout: cover
background: '/background.png'
class: 'text-white'
---

work.
To Reproduce
See above.
Desktop (please complete the following information):

OS: arch-linux
Browser: Firefox
Slidev version: 0.38.3

In fact the center layout does not accept an image as parameter.

@tonai, does it accept a background?

I see, thank you for explaining!


Public initializer for DefaultJSONSerializer

DefaultJSONSerializer doesn't need to be public. It is called implicitly when you call json.description.

json.description is perfect for convenience, but if you need performance, I don't think it's a good idea to initialize new object every time (even if it doesn't store any properties)

In that case, could you please update to the style:
public init() {}

It's the style used throughout Zewo. I saw it was your first take. I'm always unsure of which style to use it too. But I think this is better. And while you're at it. Could you create a public initializer for PrettyJSONSerializer too? And if it's not too much to ask, make it final? haha, this way you get even more contrib kudos! (:
Thanks, man!

PrettyJSONSerializer inherits initializer from DefaultJSONSerializer, so it's not necessary.
And thank you for doing these libraries.

Thanks, mate!


A tail function that would take the last n results

We could use something like
LIMIT n OFFSET ROW_COUNT-n
If we could get the ROW_COUNT for the last transformation , which I was thinking might be possible if we took the entire generated query and used it with a COUNT subquery to get ROW_COUNT

This was said on discord:
aljazerzen — 14/06/2022

DataFrame libraries usually do have head and tail, but I think this goes against the "only one way of doing things" so i would be better to refrain from adding so similar functions

max-sixty — 14/06/2022

I think it could still be useful — "reverse sort and use head" is not that easy for a user.
My main concern is that it might be difficult to implement, and it's unclear what to do when there's no sorting — would we just take head with the assumption that sorting isn't guaranteed?

I also agree that in SQL databases, order of unsorted table is not guaranteed, so we can just take head.
Regarding the implementation:
head is actually exactly the same as take
from employees
sort salary
tail 100

could be expanded to:
from employees
sort [-salary]
head 100
sort salary

which would translate to:
WITH (
  SELECT * FROM employees
  ORDER BY salary DESC -- inverse the sort here
  LIMIT 100 -- take head
) table_0
SELECT table_0.* FROM table_0
ORDER BY salary ASC -- now correct the order


Holiday Admin

Implement CRUD support for holidays

[ ] Holiday list
[ ] Holiday create/edit
[ ] Holiday delete
[ ] Holiday import

Create a unique constraint on Date + Jurisdiction

Look at ProjectActivityConfiguration for an example of how to configure unique constraint on multiple columns.


Last and latest position

Because of tracker's weird behavior and some other circumstances last added device position can be not latest. Therefore device position in web client can be wrong. https://github.com/vitalidze/traccar-web/issues/97
It is believed that to get the latest position in the web client scales badly.
It would be nice to implement latest position in device properties. And update it on every position addition.

Good idea. I will implement it at some point.

Websocket mybe good option for this... since version 4.x netty support websocket server...

Don't get how web sockets may help here. I have spent some time and came with the following SQL query to update positions:
    <entry key='database.updateLatestPosition'>
        UPDATE devices d SET d.latestPosition_id = IF(d.latestPosition_id IS NULL OR EXISTS(SELECT 1 FROM positions p1, positions p2 WHERE p1.id = d.latestPosition_id AND p2.id = :id AND p1.time < p2.time), :id, d.latestPosition_id) WHERE d.id = :device_id;
    </entry>

This works fine on MySQL and should be quite fast without any modifications to java code. To use it replace (or comment out) existing setting in traccar.cfg file:
    <entry key='database.updateLatestPosition'>
        UPDATE devices SET latestPosition_id = :id WHERE id = :device_id;
    </entry>

Hope this helps.

I've tried your query with default H2 db and got:
2015-02-28 14:15:05  WARN: Function "IF" not found; SQL statement:

        UPDATE devices d SET d.latestPosition_id = IF(d.latestPosition_id IS NULL OR EXISTS(SELECT 1 FROM positions p1, positions p2 WHERE p1.id = d.latestPosition_id AND p2.id = ? AND p1.time < p2.time), ?, d.latestPosition_id) WHERE d.id = ?;
     [90022-184] - org.h2.jdbc.JdbcSQLException (DbException.java:345)

I think latestPosition should just point to the latest record by time.

@Abyss777 I haven't tried it on H2, so looks like it's MySQL specific.
@tananaev that's what I intend to do with such update query. It may be processed in java, but I think it will require additional select to look up latest position for device to make decision whether latestPosition_id should be updated or not. Maybe I am missing something.

Masterkit BM8009GL (protocol Wialon IPS) sends stored points (after period without connection to server) in the reverse order - latest first.
And there is logic here, sometimes it's better to know where is device now than wait until it will send whole track.
In this case it's more important to have latest position than last received.

As I understand it was implemented long far ago and issue can be closed


Parity for Rancher support for docker-ce and docker-ee (Enterprise Edition)

Further to https://github.com/rancher/rancher/issues/8547
As Docker is now releasing CE and EE versions of Docker, supported docker versions should probably show some parity between the two streams as they are effectively the same releases:
http://docs.rancher.com/rancher/v1.6/en/hosts/#supported-docker-versions
Current:
17.03.x-ee
17.03.x-ce
17.04.x-ce
17.05.x-ce

Should be:
17.03.x-(ce|ee)
17.04.x-(ce|ee)
17.05.x-(ce|ee)

A) They are not the same release.  They are not supported for the same amount of time by Docker and will not be supported for the same amount of time by us.
B) There exists no public download link for EE.
C) EE is quarterly, 17.04 and 05 EE do not exist.


What is MLC

This shows up in the about menu of the latest update. What is it, @Fleker?

Ah, that was a quick hack I threw together about a month ago and then never
followed up on. It's long term goal was like those music channels shown on
cable, although I only built a proof of concept.
On Sat, Feb 13, 2016, 12:34 AM Brandon Giesing<EMAIL_ADDRESS>wrote:

This shows up in the about menu of the latest update. What is it, @Fleker
https://github.com/Fleker?
[image: screenshot_2016-02-12-23-31-13]
https://cloud.githubusercontent.com/assets/7895463/13026087/12f0a65c-d1e1-11e5-8019-69df6b915f41.png
—
Reply to this email directly or view it on GitHub
https://github.com/Fleker/CumulusTV/issues/88.

@Fleker I wonder how Pluto is doing it as they just added those style channels also powered by Dash Radio.
It would be pretty cool to make your own.

Yeah I imagine they're taking a similar approach.
I may add in these music channels at some point, although I guess it is
something I should hide for the moment as it's not active.
On Sat, Feb 13, 2016, 12:42 AM Brandon Giesing<EMAIL_ADDRESS>wrote:

@Fleker https://github.com/Fleker I wonder how Pluto is doing it as
they just added those style channels also powered by Dash Radio.
It would be pretty cool to make your own.
—
Reply to this email directly or view it on GitHub
https://github.com/Fleker/CumulusTV/issues/88#issuecomment-183598935.


Adds Django 3.0 support

This PR adds Django 3.0 support and fixes #314

Looks wrong and unneeded. Can you explain the change (specifically why you think it has anything to do with Django 3.0)?

argh, scratch that, I've forked the wrong version. Indeed, not needed


[image_picker] Image picker phpicker impl

This PR adds the PHPicker implementation. PHPicker is the new approach to pick image from the photo library for iOS 14 and higher versions.  But we still need to keep UIImagePickerController to use camera (PHPicker does not support) and for the lower iOS versions.
PHPicker implementation fixes the rotation problem which is explaining in the below issue. The issue is based on "Select Photos" option that gives the limited access to the photo library. With PHPicker implementation  the limited access is also handled and to get PHAsset will not fail anymore.
Related issues:

Fixes  flutter/flutter#74014
Fixes flutter/flutter#77281
Fixes flutter/flutter#45148

If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides. (Note that unlike the flutter/flutter repo, the flutter/plugins repo does use dart format. See plugin_tool format)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@googlebot I signed it!

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

@googlebot I signed it!

Tests are failing tho, seem legit.

Looking at our test targets to make sure some tooling changes I'm planning will work with our repo, I came across RunnerUITestiOS14 from this PR. This is the only non-standard testing target we have, and will cause issues for the tooling I was planning on writing. Can you elaborate on why that separate target is necessary (especially given that it appears to be doing availability checks internally anyway)?
Apologies if this is in the discussion somewhere above, there was a lot to try to sift through.


iPhone simulator unable to boot on Jenkins Mac machine

Tests are failing with:
xcodebuild[10509:78312023]  iPhoneSimulator: Timed out waiting 120 seconds for simulator to boot, current state is 1.
Apparently xcodebuild won't start the simulator if run via ssh. People have solved it using launch agents. @nicolasnoble, do you know what those agents are?
Here's what Travis did to solve the issue.

Step 4 of this tutorial seems to be what we need.

That should be now in. Please verify and close if that's the case.

A run ~1h after your message didn't get it right :( Maybe wait till the next?

After some more digging, it seems you have to go with the full 9 yards of running Xcode once, building a dummy iOS app once, and running it once, before everything's set up to run it automatically from the command line. Done that now. Let's see what will happen for the next build.

https://grpc-testing.appspot.com/job/gRPC_master_unstable/1361/config=opt,language=objc,platform=macos/console
This seems to be actually running the simulator now.

The log of the failure is similar to the one in these two StackOverflow questions. There are two suggested solutions:

create a plist file in ~/Library/LaunchAgents, and have that user continuously logged in: I verified that the machine has a plist file there, but it's unrelated to any of this. I'm not sure what should the contents of the file be.

execute sudo /usr/sbin/DevToolsSecurity --enable: Although you might already have gone through that dialog?

From these intructions, there should be a org.jenkins-ci file in a LaunchDaemons/ directory, which is what we'd have to move to LaunchAgents/.

I don't think this is relevant anymore, since we switched to testing iOS on Travis.


bug: remove cyclic deps

Bug Report
Current behavior:
Cyclic deps between kakarot rpc test utils and kakarot rpc core breaks rust-analyzer. It is now loading a long time and it sometimes doesn't allow you to jump to structure, function, trait definitions.

Expected behavior:
Rust-analyzer should not be broken.

Steps to reproduce:
Open current repo, wait a while and try a couple of times to jump to definitions. It should fail after a while.

Other information:
This seems to be a known issue: https://github.com/rust-lang/rust-analyzer/issues/3390

fixed!


Improve Bridje HTTP (connectors and ssl)

Allow multiple connectors in the servers configuration see the HttpServerConfig, and HTTPS protocol by using Ssl handler from netty.

Resolved, only the https feature was added, connectors can be implemented later, for now ssl can be configure in the http-server.xml configuration file.


docs/security: fix peer TLS communication example

I think the command line in the example is wrong.
/cc @kelseyhightower

lgtm

The example is just to support Transport security & client certificates in peer communications, so I think there is no need to add TLS flags for etcd clients.

ping @kelseyhightower

lgtm


Custom Pagination Production Problem

Issue Type (delete the irrelevant ones)

[x] Bug
[x] Enhancement Request
[ ] Question

Specs
v 2.16.0
What browser?
Google Chrome
Expected Behavior
this.pageChange should be defined in development and production
Actual Behavior
What actually happened?
this.pageChange  is un defind when i run npm run production
Steps to Reproduce the Problem
<template slot="pagination-bottom" slot-scope="props">
        <pagination v-if="props.total > perPage"
                    :total="props.total"
                    :perPage="perPage"
                    :pageChanged="props.pageChanged"
                    :perPageChanged="props.perPageChanged">
        </pagination>
</template>

Custom Pagination component
    export default {
        template: require('./Pagination.html'),
        props: {
            total: {type: Number},
            perPage: {type: Number, default: 10},
            pageChanged:{type: Function},
            perPageChanged:{type: Function}
        },
        data() {
            return {
                current: 0,
                pages: 1
            }
        },
        created() {
            console.log(this.pageChanged);  // <---------- This will return undefined on production
            this.calPageCount();
        },
        methods: {
            setPage(page) {
                this.current = page;
                this.pageChanged({currentPage: this.current + 1});
            },
            previous() {
                if (this.current > 0)
                    this.setPage(this.current - 1)
            },
            next() {
                if (this.current < this.pages)
                    this.setPage(this.current + 1)
            },
            setPerPage(perPage) {
                this.perPage = perPage;
                this.calPageCount();
                this.perPageChanged({currentPerPage: this.perPage});
            },
            calPageCount() {
                if (this.total > this.perPage) {
                    this.pages = Math.round(this.total / this.perPage);
                    console.log(this.pages);
                }
                else {
                    this.pages = 1
                    this.setPage(0);
                }
            }
        }
    }

### According to the solution of this issue the docs need to be updated

I found the solution buy changing the properties into kebab-case. But I don't understand why this is happening in production environment?
<template slot="pagination-bottom" slot-scope="props">
        <pagination v-if="props.total > perPage"
                    :total="props.total"
                    :per-page="perPage"
                    :page-changed="props.pageChanged"
                    :per-page-changed="props.perPageChanged">
        </pagination>
</template>

hey @enghelewa the reason it is happening for you in production is because your production build probably uses in-dom templates. here's some relevant documentation:
https://vuejs.org/v2/guide/components-props.html
In general with vue i think it's always better to use kebab case for your templates. This is a vuejs characteristic, so i'm going to close this.


python3Packages.mlflow: added setuptools dependency

Description of changes
MLFlow has a runtime dependency on setuptools. For more information refer to https://github.com/mlflow/mlflow/issues/11330
I've just added setuptools to propagatedBuildInputs. The discussion in the mlflow issue seems to imply they don't like this dependency, but have not found a replacement as of yet, so maybe the dependency is temporary. Nonetheless it is currently needed.
Before this change,
nix-shell -p python3Packages.mlflow --run "mlflow server"

will exit with a ModuleNotFoundError for pkg_resources which is provided by setuptools. After the change, mlflow server starts successfully.
Extra information
I started debugging this issue through the mlflow-server package. Interestingly, adding the setuptools dependency there produces a different ModuleNotFoundError for the package gunicorn. The derivation for mlflow-server does something weird with the gunicorn package which I wasn't able to figure out. But it turns out the server can just be run through the python3Packages.mlflow package, so I'm not sure if that package is still needed. The commit history of the package implies it was originally created in response to some problem with gunicorn, however gunicorn seems to work fine in mlflow now.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

Please squash.

Looks like https://github.com/NixOS/nixpkgs/pull/330747 beat me to it.
closing...


Add --no-tty option

This enables running docker run without the -ti options, which allocates a pseudo-tty and keeps STDIN open.
The motivation for this is to be able to spawn container from systemd, which provides no TTY.

Thanks @wngr. Merged.


Pcq1026 spring framework upgrade

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/PCQ-1026
Change description
Spring framework upgrade to 2.5.3
Azure library upgrade 12.13.0
Pcq loader functional test change, deleting the blob before creating one.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x
<img width="744" alt="Screenshot 2021-08-20 at 14 06 24" src="https://user-images.githubusercontent.com/88436206/130237736-a603fb84-7035-4e6d-a7b8-c003cda8c961.png">
] No

PCQ Loader functional test worked on nightly build


Requested type change to lightning.Component.Template

It would be nice if the template pulled the extends of keys from TemplateSpecType, instead of from Element.TemplateSpec in the else case.
The reason for this is that you have public properties that are defined in the template based on bound properties.
If we switched to using a setter, we would have to change other locations that used the same bound property, but directly onto Element.TemplateSpec values.
static _template() {
  return {
     publicProp: this.bindProp('isLive', () => { logicToSetProp }),
     Container: {
        visible: this.bindProp('isLive'),
     }
  }
}

Right now, typescript fails and prevents the usage of the pattern.
/**
   * Type used for the return result of _template().
   *
   * All TemplateSpec properties are made optional. Nested TemplateSpec properties are also made
   * optional, except for the `type` propety which is made required.
   */
   export type Template<TemplateSpecType extends Element.TemplateSpec = Component.TemplateSpecLoose> = {
    [P in keyof TemplateSpecType]?:
      P extends ValidRef
        ?
          TemplateSpecType[P] extends Component.Constructor
            ?
              TemplateRequireType<TemplateSpecType[P]>
            :
              TemplateSpecType[P] extends Element.Constructor
                ?
                  Template<InstanceType<TemplateSpecType[P]>['__$type_TemplateSpec']>
                :
                  Template<Element<InlineElement<TemplateSpecType[P]>>['__$type_TemplateSpec']>
        :
          P extends keyof Element.TemplateSpec
            ?
              TemplateSpecType[P] // P is a Element property key
            :
              string extends P
                ?
                  any // Support Loose Elements: keyof loose Elements `P` will always be a `string`, so let anything go
                :
                  undefined // Otherwise allow the property to only be undefined
  };

This seems reasonable.
This was purposely disallowed because the setters are be executed before any children on the Component have been created and before the Component sub-classes' own constructor is called (which might set up special child accessor properties that might be needed by the setters). Often public setters on Components manipulate the children of the Component and so using them in the _template often results in exceptions because the expected children or properties haven't been created yet.
However the use cause of bindProp() was not considered. I'd rather take this a step further and only allow the special structure that bindProp() produces to be set in this case. What do you think about that?

Absolutely. Just having the bindProp signature would be more than adequate. Otherwise, it should use the constructor or __init, or default properties.


Moved the js-module blocks into the platform blocks

I got an error when I tried to use this in a browser while developing a android and ios app. This should remove the error.

Hi @dominicrico. Thanks for this interesting comment.
This plugin is Android and  iOS only.
It does not support the browser platform or any other platforms.
I don't know what's the best practice between repeating js-module in all platform blocks
or just informing that a platform is not supported.
Can you provide me with the error message that you get?
Thanks

from the doc
The js-module tag specifies the path to the common JavaScript interface.
Nesting <js-module> elements within <platform> declares platform-specific JavaScript module bindings.

release 1.2.0
thanks @dominicrico


Refactor redis connection

Currently we have exactly 1 redis connection per request (even if unused). This has been done due to issue with the Gnosis provided redis when using multiple connections in a single request.
ToDos

[ ] Investigate why this is happening
[ ] Revert/ refactor changed if possible: https://github.com/gnosis/safe-client-gateway/pull/86

We don't see any problem

We don't see any problem


Cannot get partial hash and continue

I wanted to compute hash the way that I would compute hash for one chunk, get it, and than continue computing hash for chunks 1+2, get it; then 1+2+3 etc. Basically I need the incremental hash computing and as I'm computing it I need the intermediate hashes. But this is not working. I'm getting wrong hashes.

Can create a failing test?

Closing due to no feedback.


Add truncation warning

Add truncation warning, as it can be kind of rough to find this through trial and error or by noticing the context numbers.

I don't think this is the right approach. Truncation is now a regular part of how the context window is adjusted, so it would spam hundreds of lines in the console. I've been meaning to add a feature to the UI that will visually indicate how much of the history is being used as context, but I haven't gotten around to it, and the front-end code probably needs to be cleaned up a bunch first.

That approach would work just as well. Some kind of a return notification on the API would be nice too!


missing !{} (unescaped interpolated variables) support in attributes

when using
script(src='/js/page/!{pageName}.js')

pyjade complied to
<script src="/js/page/!{pageName}.js"></script>

so the !{} support in attributes is missing.
and when using
script(src='/js/page/#{pageName}.js')

pyjade escape the string, and finally the html will be
<script src="/js/page/landing.js"></script>

This still seems to be an issue. Is there a workaround?

@rufman are you using jinja?

I'm using Mako


test(utils): fix timeout in error message

Codecov Report

Merging #4412 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #4412   +/-   ##
=======================================
  Coverage   91.29%   91.29%
=======================================
  Files          56       56
  Lines        1919     1919
  Branches      482      482
=======================================
  Hits         1752     1752
  Misses        155      155
  Partials       12       12

Impacted Files
Coverage Δ

packages/server/src/jsdom.js
90% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Keep track of local dependencies in files task/goal

E.g. in our simple test project, there is a libs/dependency-1.0.0.jar file that is not a snapshot/project dependency, but should still be watched by skaffold since its contents/name may change without anyone changing the build file.
We should check if this is possible in Maven as well.
Related to #1021

It's actually pretty annoying and complicated to do this in Maven, but pretty easy in Gradle, so we might need to support it for gradle. Having said that, I think we should hold off doing this. Large projects can usually be combined using maven-artifacts or project artifacts.

Someone can open a new issue for this if they want to. I don't think there's a reason to support this at the moment, it's an obscure way of building.