signal-desktop: gnome2 cleanup

Motivation for this change
Removed/updated gnome2 references (Part of #39976)
Things done

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

[x] NixOS
[ ] 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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg  build signal-desktop

Success on x86_64-linux (full log)
Attempted: signal-desktop
Partial log (click to expand)
  /nix/store/5bvxg06wfz1vczzllqxy1hki2hl061qh-gconf-3.2.6
  /nix/store/89bqw5bqbmmab49gikrgfz4ddyw2i9m2-dpkg-<IP_ADDRESS>
  /nix/store/qhsv58mfxi8bqdi26753fmq4ppq56503-signal-desktop_1.11.0_amd64.deb
copying path '/nix/store/qhsv58mfxi8bqdi26753fmq4ppq56503-signal-desktop_1.11.0_amd64.deb' from 'https://cache.nixos.org'...
copying path '/nix/store/89bqw5bqbmmab49gikrgfz4ddyw2i9m2-dpkg-<IP_ADDRESS>' from 'https://cache.nixos.org'...
copying path '/nix/store/5bvxg06wfz1vczzllqxy1hki2hl061qh-gconf-3.2.6' from 'https://cache.nixos.org'...
building '/nix/store/ibvnbd8h10v3c6127qgk2nwq6nkgxfhh-signal-desktop-1.11.0.drv'...
unpacking sources
installing
/nix/store/0rxqx810n1j9ccvjmmnv2ng9ggipldpd-signal-desktop-1.11.0

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: signal-desktop
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: signal-desktop
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.


Halfway on #20

This implements the multiple base directories discussed on issue #20.
Please review.

This PR was rolled back. See #24 for corrected version


Move Amazon.Corretto.21 <IP_ADDRESS> to Amazon.Corretto.21.JDK <IP_ADDRESS>

Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/pull/159802


fix cast 255 as int8 error not as 127

Here's the error I found (this should probably produce an error, not insert 127 into an int(8) field).
gluesql> create table myi8 (id int(8));
gluesql> insert into myi8 values (1);
1 row inserted

gluesql> select cast(255 as int(8)) from myi8;
╭─────────────────────╮
│ CAST(255 AS INT(8)) │
╞═════════════════════╡
│ 127                 │
╰─────────────────────╯
gluesql>

If the code looks good, I can look into converting the other (*, Literal::Number) cases so that they rely on BigDecimal to do the heavy lifting to convert from bigdecimal to i8, i64, f64, etc.

Codecov Report

Merging #546 (326ddfd) into main (01a55e2) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 326ddfd differs from pull request most recent head 8f534d5. Consider uploading reports for the commit 8f534d5 to get more accurate results

@@           Coverage Diff           @@
##             main     #546   +/-   ##
=======================================
  Coverage   93.11%   93.12%
=======================================
  Files         178      178
  Lines       11644    11644
=======================================
+ Hits        10842    10843    +1
+ Misses        802      801    -1

Impacted Files
Coverage Δ

core/src/data/value/literal.rs
95.23% <100.00%> (+0.34%)
:arrow_up:

test-suite/src/function/cast.rs
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

resolves issue #545

resolves issue #545

This phrase should be at the top of your PR conversation. 👍
e.g) https://github.com/gluesql/gluesql/pull/512

This PR is still officially a draft, but I would like to receive comments about it.  I believe we can use the same basic code for other number types (int, float, decimal), and let bigdecimal do the heavy lifting by using bigdecimals to_i8, to_i64, to_f64 functions to convert the bigdecimal to the appropriate data type.   I am also looking at implementing solutions for i32, i128, and the unsigned versions (u8, u32, u64, and u128), in which the cast function would need code for each data type.


replace auto deduction for master session close member function

Can one of the admins verify this patch?

(Closing for the same reason as the other change)


awscli2: cryptography package version mismatch

Steps To Reproduce
Steps to reproduce the behavior:

build awscli2 on the master branch
see error

Build log
https://gist.github.com/michielboekhoff/9ff1208201775ee8efb4e751c5af2c55
Additional context
This seems to have been introduced via a combination of commit 5377b75a52156cf3cc94cb0fa3e8237ef76a0856 & a bump of the version of the cryptography package in 4561bf0464c55d4871ca7d32042bf8f2ec81cad4. I've confirmed that reverting 5377b75a52156cf3cc94cb0fa3e8237ef76a0856 & correcting the version fixes the problem, and it might make sense to keep this in because of this comment.
My (confirmed) diff:
diff --git a/pkgs/tools/admin/awscli2/default.nix b/pkgs/tools/admin/awscli2/default.nix
index 305a1e719f5..8ba850b6ead 100644
--- a/pkgs/tools/admin/awscli2/default.nix
+++ b/pkgs/tools/admin/awscli2/default.nix
@@ -70,7 +70,8 @@ with py.pkgs; buildPythonApplication rec {
       --replace "docutils>=0.10,<0.16" "docutils" \
       --replace "ruamel.yaml>=0.15.0,<0.16.0" "ruamel.yaml" \
       --replace "wcwidth<0.2.0" "wcwidth" \
-      --replace "distro>=1.5.0,<1.6.0" "distro"
+      --replace "distro>=1.5.0,<1.6.0" "distro" \
+      --replace "cryptography>=3.3.2,<37.0.0" "cryptography"
   '';

   checkPhase = ''

Notify maintainers
@bhipple @davegallant @bryanasdev000
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.36, NixOS, 22.05 (Quokka), 22.05.20220502.fb679d3`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.0`
 - nixpkgs: `/nix/store/yw1synf6pp4hvc8zjlmr25ay7k8ikhf3-source`

Thanks for reporting!
Closed by https://github.com/NixOS/nixpkgs/pull/178313.

Brilliant, thanks @bryanasdev000!

Looks like this same breakage is happening again, except this time due to some combination of:

awscli2 version update from 2.10.0 -> 2.10.1: https://github.com/NixOS/nixpkgs/commit/a0e76d80c628d634e7bf346ef7fa778c2899d54e
cryptography version update: https://github.com/NixOS/nixpkgs/commit/8e69a97f5101c64286ba23d4ce830a66ef4883ad

This specific build failure should be fixed
Bigger picture: Shouldn't an issue like this be caught before getting merged into master or promoted to nixos-unstable branch? I'm confused and surprised that this keeps happening.


[Bug]: Unable to download multiple files parallelly in different tabs of puppeteer browser

Bug description
I want to download multiple files parallelly by puppeteer and for which, I tried logging in to one tab and then opening multiple tabs by for loop and downloading the file from each tab separately.
Download behavior is like this:

When I try downloading a separate single file from multiple tabs, Only the file of the last tab is downloaded in a different download path.

@mathiasbynens @jrandolf Can you help with this issue?
const promises = [];
for (let j = 0; j < 3; j++) {
      try {
        promises.push(this.getData(j, accountId));
      } catch (err) {
        this.logger.error(`Downloading PDF error: ${err.message}`);
      }
    }
await Promise.all(promises);

private async getData(
    j: number,
    accountId: string
  ): Promise<void> {
    const { billDetailsPage } = this.config.getData;
    try {
      const page1 = await this.browser.newPage();
      await page1.goto(billDetailsPage, {
        waitUntil: 'networkidle0',
      });
      const billsDataObject = await this.getBillsData(page1);

      // billsData is array of BrowserInterface.ElementHandle.
      const { billsData, billsDateString } = billsDataObject;

      const relativeDownloadFilePath = `service_acccounts/${accountId}/bills/${billsDateString[j]}`;
      const downloadPath = path.resolve(
        __dirname,
        `/home/download/${relativeDownloadFilePath}`,
      );
      const client = await page1.target().createCDPSession();
      await client.send('Page.setDownloadBehavior', {
        downloadPath,
        behavior: 'allow',
      });
      await billsData[j].click();
      await new Promise((resolve) => setTimeout(resolve, 20000));
    } catch (err) {
      console.log(err.message);
    }
  }

Puppeteer version
^12.0.0
Node.js version
14.17.3
npm version
6.14.13
What operating system are you seeing the problem on?
Linux
Relevant log output
No response

It appears that you run your function in parallel but modify the global shared state (the browser's download behaviour). Unfortunately, there is no way to isolate the download behaviour in Chromium. Try using https://github.com/thomasdondorf/puppeteer-cluster and see https://github.com/puppeteer/puppeteer/issues/299 where the support for downloads is discussed.

Below is the sample code which is failing to download multiple files in a separate download path by a puppeteer.
@OrKoN can you look at this?
const puppeteer = require('puppeteer');
const path = require('path');

async function download(i, browser) {
  const page = await browser.newPage();
  await page.goto('https://vinay-jtc.github.io/test-pdf', {
    waitUntil: 'networkidle2',
  });
  const billsData = await page.$$('.pdf');
  const downloadPath = path.resolve(`/home/vanni/download/${i}`);
  await page._client.send('Page.setDownloadBehavior', {
    behavior: 'allow',
    downloadPath: downloadPath,
  });
  await billsData[i].click();
  await new Promise((resolve) => setTimeout(resolve, 5000));
}

async function simplefileDownload() {
  const browser = await puppeteer.launch({ headless: false });
  const promises = [];
  for (let i = 0; i < 4; i++) {
    promises.push(download(i, browser));
  }
  await Promise.all(promises).then(() => {
    browser.close();
  });
}

simplefileDownload();

please ask on https://stackoverflow.com/questions/tagged/puppeteer (I am sure someone will be able to help you)


fix: add registerWith method for Windows

- What I did

Add registerWith() method for Windows

- How I did it
- How to verify it
- Description for the changelog

@sachins-geekyants @sonle-geekyants the e2e tests are being a bit flaky, I am manually re-running the test packs. Will re-run these tests once the current merge action's checks have completed


Unable to add multiple new user in teams page

On the team's page, I am not able to add multiple new users it gives a 500 Internal Server Error, but on other hand, I am able to add a single new user. I have explain the scenario to @darth-coder00

single-user API call

Multi-user API call

{"responseMessage":"An exception with message ['[{\"id\":\"2808eb23-a209-4847-b2d9-035b580c0fd6\",\"type\":\"user\",\"name\":\"amanda_phillips7\",\"description\":\"Amanda Phillips\"},{\"id\":\"95a3c28f-67f2-4237-9f16-8e24349b8de6\",\"type\":\"user\",\"name\":\"aaron_mcgee8\",\"description\":\"Aaron Mcgee\"}]' contains no element for index 3 and for '3'.] was thrown while processing request."}

I think it is because of the reverse sorting done in the applyPatch. I am happy to take this up and provide a PR soon.

I am unable to assign it to myself. Can someone please assign this issue to me?


All uploaded IPA files are invalid

After many tests, IPA uploaded with this tool indicates that IPA is invalid, while other official uploads have no problem

Hi @xml520. Are you saying that Apple says it is invalid after the upload is completed or are you getting that as an error from ios-uploader? Can you post the exact error you are getting?

Hello. I think the same thing happens to me, ios-uploader fulfills its function but minutes later I receive an email saying:
ITMS-90161: Invalid Provisioning Profile - The provisioning profile included in the bundle com.ghostsoft.acceso [Payload/Proyect] is invalid. [Missing code-signing certificate]. A Distribution Provisioning profile should be used when submitting apps to the App Store. For more information, visit the iOS Developer Portal.

Hola @xml520. ¿Está diciendo que Apple dice que no es válido después de que se complete la carga o lo está recibiendo como un error de ios-uploader? ¿Puede publicar el error exacto que está recibiendo?

Hi @ABD2k , thanks for the additional information on this issue.
So this looks like an issue with code signing when the .ipa file was built. I'm guessing a Development Provisioning profile was used instead of a Distribution Provisioning profile. How you configure and set this up depends on the platform used to build the app (Xcode, Xamarin, Expo, e.t.c.) and is outside the scope of this tool.
ios-uploader does not validate this before uploading, hence why the error is received from App Store post upload.
See this question on Stack Overflow for more information: https://stackoverflow.com/questions/28960751/error-code-90161-invalid-provisioning-profile

Closing this issue as this most likely is not a problem with ios-uploader. ios-iploader makes no modifications to the actual IPA.


menteslibres.net is down

The server behind the Cloudflare setup of menteslibres.net is down. Now menteslibres.net is constantly displaying a "502 Bad Gateway". Thus menteslibres.net/gosexy/to is not installable by go get or go mod.
If the site is a static site, please consider to move to Github page or other free hosting service so the packges can be installable again. Thanks.

Moved to Github! thanks

Both the domain menteslibres.net and www.menteslibres.net do not resolve to anything right now. I can see you're using Cloudflare as your DNS.


Cleanup Throttle Flow mechanism from CRMP

Problem
Throttle Flow mechanism in CRMP was inherited from Weave, but this feature was dropped in CHIP spec.

Summary of Changes
Cleanup Throttle Flow mechanism from CRMP

@jelderton @hawk248 @saurabhst @BroderickCarlin ?

@jelderton @hawk248 @saurabhst @BroderickCarlin ?


Add typescript support for step_definition files

I was wondering whether we should add test for this.. but I think we are testing whether it works for javascript.. and it would be hard to miss if someone broke this line in a way that it kept js working, but not ts. And it seems to me that the complexity of adding the whole typescript ecosystem here just to test those three characters is too much. So.. I think I'l pass this one as is :) Thanks for the issue and the PR!

Cool!
You can always choose to fully support TypeScript as well. But it's what you say: You have to include the complete TypeScript circus to do so...Let me know what you want to do :)


set development to default branch

We have to rename this way:
master ===> development
and
production ===> master
Right ?

Development set as default and branches renamed


Menubar went black

In the newest version (152) on macOS 10.12.6, the titlebar went black. Occasionally all of the text will go black, but 'select all' will fix it. I downgraded to 150 and it works fine.

Duplicate of #799. Will be fixed soon, track it using https://github.com/macvim-dev/macvim/milestone/1.


Nil as part of where interface

Your Question
Hello. I have a db request with "where" param, which accepts an interface value, and table model one of the columns of which could be int or null:
type ArticlesThreads struct {
	ID          uint              `gorm:"primary_key;column:id;type:INT4;"`
	Title       string            `gorm:"column:title;type:VARCHAR;size:255;"`
	Parent      uint              `gorm:"column:parent;type:INT4;"`

result := []models.ArticlesThreads{}
where  := make(map[string]interface{})

DB.Model(models.ArticlesThreads{}).Where(where).Find(&result)

And If I pass multiple values everything works fine until one of the values is equal nil:
//All this variants return correct value:
where["parent"] = 1
where["parent"] = nil
where["parent"] = []interface{}{1, 2}

//But noone of this:
where["parent"] = 0
where["parent"] = []interface{}{1, nil}
where["parent"] = []interface{}{1, 0}

Expected answer
Please tell me how can I list several values in the interface, one of which will be identical to NULL in database?
I tried to change the field type in the model to sql.NullInt, string, omitempty, json, but nothing helped..

where["age"] = []interface{}{1, sql.NullInt64{}}
DB.Where(where).Find(&users)

 SELECT * FROM `users` WHERE `age` IN (1,NULL) AND `users`.`deleted_at` IS NULL


Sep by comp

make plots that show the separation of SLE cases across different components. Ex:

@pascoesean Looks good to me now!


Page offset reset to 0 on server side sorting

I'm submitting a ...
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support request here

Current behavior

If I enable server side sorting and navigate to page 2, the offset is 1. If I click one of the sorting buttons, the offset is set to 0 again and I get wrong results.
Reproduction of the problem
Bind datatable to this.offset and view the output:
onSort(event) {
        console.log(this.offset);
}

Sorry - my bad. I accessed a wrong property :-|


How to do FPS controls collisions

FPS Controls collisions
Version

Master

Platform

Windows [X]

Hi
Please take a look at the portal game example.
Cumps

Hi tentone. Is me again...
Im working in a fork of nunuStudio in which i've implemented a system like blueprints, make grow the post processing and im changing the material editor to make it with nodes, i would like when it be public you can implemente many of these things 😄

Fork the repository on github. https://guides.github.com/activities/forking/
Create a branch from that fork on git. Add your own code there.
Open pull requests to suggest adding your code to the main repo.

Ok... I'll implement the blueprints and i'll open a pull request


Inline Button inside of top-bar

Hello,
I'm using Foundation 6. See my code below

   <input class="input-group-field" type="text">

   <input type="submit" class="button" value="Submit">

I discovered that the margin-right property of. .top-bar input gets applied to the button causing it not to display as expected (margin to its left)

I appreciate the code snippet, but I'm not 100% clear what the issue was and what you expected to happen, or which version of 6 you were using.  Going to close, if you're still seeing this with foundation 6.2.1 please reopen with the info requested here: https://github.com/zurb/foundation-sites/blob/develop/.github/ISSUE_TEMPLATE.md


How can I save a Group as PNG?

the pixelData in the Group layer is []
So I dont know if there is any othrer ways to save a Group as png

hey, I also have this demand  in my project and is looking for solution. happened to see this related issue yesterday, seems not accomplished yet.

i also need this solution.
anyone have solve ?

This library does not contain a built in renderer, so any images exported are raw pixel data (or a full preview, but that's already generated by Photoshop). The answer is that you can't do that with this library.
You can go through each layer in the group, extract its pixelData and combine them using a canvas (keep in mind layer positioning). This will not however preserve things such as patterns, layer styles, adjustment layers etc., doing that would require building a psd renderer.

This library does not contain a built in renderer, so any images exported are raw pixel data (or a full preview, but that's already generated by Photoshop). The answer is that you can't do that with this library.
You can go through each layer in the group, extract its pixelData and combine them using a canvas (keep in mind layer positioning). This will not however preserve things such as patterns, layer styles, adjustment layers etc., doing that would require building a psd renderer.

Thank for your suggestion!

Thank for your suggestion!


PendingOperations returned value is not correct

Describe the bug
When checking the PendingOperations for a table then it returns the wrong value
To Reproduce
Have multiple table updates while you are offline. Kill the app then turn on internet and launch app. Push the changes to server and verify results
Expected behavior
After successful push it should return correct pendingOperations count which is 0
What platforms?
iOS/Windows with offline store

Server:

.NET7
VS2022
6.1.0

Client:

.NET7
VS2022
6.10
Azure K8S

Additional context
I have added my additional logging in OperationQueue.cs to see if pendingOperation variable is set properly or not
InitializeAsync:: [PageCount]: 2 [PendingOperations]: 2

[12:59:30 INF] Syncing [report]: "PushStarted" Success: null Pulled: -1
EnqueueOperationAsync:: [PendingOperations - BeforeDecrement]: 2
EnqueueOperationAsync:: [PendingOperations - AfterDecrement]: 1
[12:59:32 INF] Syncing [report]: "PushFinished"

[12:59:32 INF] Syncing [userreport]: "PushStarted" Success: null Pulled: -1
EnqueueOperationAsync:: [PendingOperations - BeforeDecrement]: 2
EnqueueOperationAsync:: [PendingOperations - AfterDecrement]: 1
[12:59:33 INF] Syncing [userreport]: "PushFinished"

I had 2 items in operation queue which are correct in InitializeAsync method and when reportis being pushed then in EnqueueOperationAsync method it had  count of 2 pendingOperations for which are so far good and after decrement with this
 Interlocked.Decrement(ref pendingOperations);
it shows the correct count to 1. Still so far all good and happy at this point
Now second table userreport start pushing data to server and it shows in EnqueueOperationAsync method pendingOperation as 2 which should be 1.
So here is the problem now rather than 1 its using 2 and decrement value so you will always get wrong result.

Thanks for the report.  I'll see if it's an easy fix.  If not, I'll defer it to 6.1.2, but I'm going to be releasing 6.1.1 this afternoon, so I'll take a look this morning.

I've added the following test to ensure PendingOperations is updated properly:
    [Fact]
    public async Task KS7_EndToEndSyncTest()
    {
        const int insertedItems = 5;

        await InitializeAsync();

        Assert.Equal(0, client.PendingOperations);

        // Add a couple of new items, and ensure pending operations are updated.
        for (int i = 0; i < insertedItems; i++)
        {
            var insertion = new KitchenSinkDto { StringValue = $"Item {i}" };
            await offlineTable!.InsertItemAsync(insertion);
            Assert.Equal(i + 1, client.PendingOperations);
        }

        await offlineTable!.PushItemsAsync();
        await offlineTable!.PullItemsAsync();

        Assert.Equal(0, client.PendingOperations);
        var nItems = await offlineTable!.CountItemsAsync();
        Assert.Equal(insertedItems, nItems);
    }

This test passes, so basics of pending operations works.  I'm wondering if it's a multi-table issue (so that's my next task - to add a new test for multiple tables).
In the interim, if you have an integration test that demonstrates the failure you experience, that would be immensely helpful.

... and the equivalent multi-table version:
    [Fact]
    public async Task KS7_EndToEndSyncTest()
    {
        const int insertedItems = 5;

        await InitializeAsync(true);

        Assert.Equal(0, client.PendingOperations);

        // Add a couple of new items, and ensure pending operations are updated.
        for (int i = 0; i < insertedItems; i++)
        {
            var insertion = new KitchenSinkDto { StringValue = $"Item {i}" };
            await offlineTable!.InsertItemAsync(insertion);
            Assert.Equal(i + 1, client.PendingOperations);
        }

        // Also add a movie, in case there is a multi-table issue
        var movie = GetSampleMovie<ClientMovie>();
        await offlineMovieTable!.InsertItemAsync(movie);
        Assert.Equal(insertedItems + 1, client.PendingOperations);

        await offlineTable!.PushItemsAsync();
        Assert.Equal(1, client.PendingOperations);
        await offlineTable!.PullItemsAsync();
        Assert.Equal(1, client.PendingOperations);

        await offlineMovieTable!.PushItemsAsync();
        Assert.Equal(0, client.PendingOperations);
        await offlineMovieTable!.PullItemsAsync();
        Assert.Equal(0, client.PendingOperations);

        Assert.Equal(0, client.PendingOperations);
        var nItems = await offlineTable!.CountItemsAsync();
        Assert.Equal(insertedItems, nItems);
    }

... also works.  Thus I need a test that shows the problem off before I can proceed.

Thanks @adrianhall for checking. Sure I will try with integration test.


Adds Arena Evac shuttle

Changelog:

Adds Uncle Charlie's Luxury Boat, an Evac vessel for Arena Station.

:cl: Velcroboy, Old Dance Jacket, Colin-Tel

add: Added Uncle Charlie's Luxury Boat!

Without context, the way the player-facing changelog reads makes it sound like a new ship is available from the shipyard dock console. If all this does is change the evac shuttle for Arena, is it really necessary for a changelog entry?

Without context, the way the player-facing changelog reads makes it sound like a new ship is available from the shipyard dock console. If all this does is change the evac shuttle for Arena, is it really necessary for a changelog entry?

It's Uncle Charlie's Luxury Boat.

I don't know though. I think it's pretty cool and will be noticed when it shows up. But I'm not at my pc anymore for the night to change anything

Without context, the way the player-facing changelog reads makes it sound like a new ship is available from the shipyard dock console. If all this does is change the evac shuttle for Arena, is it really necessary for a changelog entry?

Yeah, I agree, on the other, this is kinda a sick af evac so


Baget doesn't allow setting Mirror options via environment variables

Describe the bug
The default production appsettings.json file explicitly disables mirroring—which is a sound, sensible default. However, due to the order in which configuration is applied in ConfigureBagetConfiguration', with environment variables being added first, followed by the settings in appsettings.json`, this effectively means that when running Baget in a docker container, it's impossible to change this setting.
To Reproduce

Run the following commands:docker pull loicsharma/baget:latest
docker run `
>> --name  baget-server -d -p 8800:80 `
>> -e Mirror__Enabled=true -e Mirror__PackageSource=https://api.nuget.org/v3/index.json
>> -e AllowPackageOverwrites=true `
>> --mount type=bind,src=D:\var\opt\baget\pkgs,dst=/var/baget loicsharma/baget:latest

Navigate in your browser to the baget-server.
No NuGet.org packages show up in the interface when searching for them.

Expected behavior
I expected to see packages from Nuget.org show up when I search for them (or even when just simply navigating to the baget server).
Additional context
The only way to effectively do this right now is to create your own docker image based on the official baget image and add a layer which adds a modified appsettings.json file.
But I think this could be fixed by reordering the code here in ConfigureBagetConfiguration. Basically, lines 15 and 17 ought to be swapped such that the appsettings.json is loaded first, then the environment variables are loaded. In this way, if no environment variables are supplied, then the prevailing configuration in appsettings.json applies. But if environment variable are supplied, they can override the settings in appsettings.json.
I can put in a PR for this a little bit later today.

Hmm, what doesn't make sense about this, though, is that in my docker container instance, I was able to change AllowPackageOverwrites to true, and it does work. So why isn't the mirroring configuration being reflected?

Ah, never mind. Apparently I need to read more. This is handled natively by ASP.Net Core already. So I must have something misconfigured.

@fourpastmidnight Have find fox for this?

Sorry for the late reply. Like @fourpastmidnight mentioned, this should be supported natively. You should be able to use the environment variable Mirror__Enabled to enable mirroring (notice that there are TWO underscores between Mirror and Enabled).


improved retry policy

When an NTP query fails the client retries after 5 seconds.
On every failed retry the client doubles the restart delay,
but does not wait longer than 600s.

https://github.com/input-output-hk/ouroboros-network/pull/1596

bors r+


Disallow move action for namespace if target project has quota set

Till we put a proper implementation for resource quota on a move action as a part of https://github.com/rancher/rancher/issues/15549, we need to disallow moving the namespace to a project with resource quota set.

Putting to merged, but not ready to test because the CLI needs fixed for this too. When the CLI PR is merged, will move it to test

Tested with master from Sep 14th.
Create a project "P1" without any resource quota assigned and create a namespace "N1" in P1
Create a project P2 with resource quota assigned.
Move N1 to project P2
Move is not allowed as expected.
Server logs:
2018/09/14 18:39:54 [ERROR] Unknown error: can't move namespace. Project P2 has resource quota set

Error message below:

https://github.com/rancher/rancher/issues/15612


Added KeyPrinter project

Developer Checklist

[ ] Followed guidelines mentioned in the readme file.
[ ] Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
[ ] Starred ⭐ the Repo (Optional)

Summary
add a summary here
Screenshot

Live Project Link
add a working project link here

@PBJI Hey bro I hope you're doing well. I have added my simple javascript project in this repo please check it and give feedback about it. thank you

@kohi9noor I rarely request features to people, yet I would like to request one wherein the application also shows the amount of time a user kept that button pressed and how many times. This data should be stored in the localstorage using localstorage API and then we can download the data in CSV format.


Markdown Linting

PR to add markdown linting so that we can keep the formatting nice and clean. This includes the following changes:

Update all *.md files to be compliant with .markdownlint.yaml, excluding LICENSE.MD.
Add GitHub's Super-Linter action to run linting on any modified markdown files in PRs/pushes to main/master

An example of a failed run can be found here.
Couple of things worth noting:

There were a number of dead anchor links in AWS IAM. All have been updated except the ones for #Groups and #instance-metadata-service as there weren't relevant section present
There were a fair few bare URLs which have mostly been changed to [here](https://url)

@x-delfino can you merge in all the filename changes you actioned in the other PR, to fix the merge conflicts?


DSND-3112: Modify delete behaviour and add delta_at into endpoint

Describe the changes

Add new headers: entityID and deltaAt for delete endpoint
Modify the uri path for delete endpoint
Remove Transactional annotation from delete
Change ResourceChangeRequestMapper to action when document is null
Route to new service method in processor when delete returns empty

Related Jira tickets
DSND-3112
Developer check list
General

[x] Is the PR as small as it can be?
[x] Has the Jira ticket been updated with any relevant comments?
[x] Is the README up to date?
[x] Has the POM been updated for the latest versions of dependencies?
[x] Has the code been double-checked against main?
[x] Does the code adhere standards in this repository, including SonarQube checks?

Testing

[x] Do the code changes have unit and integration tests with code coverage > 80%?
[x] Has the code been tested locally and/or passed the API Karate tests on Tilt?
[x] Have mandatory manual test cases been run?
[x] Are extra Karate tests required?
[x] Are all the test data resources up to date?

Release preparation
Where possible, add links to the respective Jira tickets when an item is checked

[x] Have these changes been included in a release ticket?
[ ] Are changes required to the docker-chs-development deployment or test data?
[ ] Are any changes required to the environment added to deployment repositories?

Notes to the tester
Add any testing hints or instructions here.

CI: Security warnings found!

CI: No security warnings found

CI: No security warnings found

CI: No security warnings found

CI: No security warnings found


Documentation improvements and Tutorial Index

Documentation changes:

Streamlined installation instructions
Added tutorial index and links
Added SPC page for improved doc access
Fixed citation
Improved doc sidebar

Signed-off-by: Maria Masha Shugrina<EMAIL_ADDRESS>

@Caenorst pls take another look


nimplex install errors

Issue with the nimplex in AMMap installation.
Current instructions have added the previously lacking:
nim c --d:release --threads:on --app:lib --out:nimplex.so nimplex/src/nimplex.nim
nim c --d:release --threads:on --app:lib --out:utils/plotting.so nimplex/src/nimplex/utils/plotting.nim
Issue is that current fix I found was to manually move nimplex.so, nimplex.nim, and plotting.nim to respective locations in the AMMap folder. Please help update the install instructions for automated cmd line to install

The above commands use relative paths (like utils/plotting.so), so they will depend on where you run them.
The commands you posted look good in terms of placing executables in the root directory. Is that not the goal?
We can consider adding a simple install.sh script to automate everything for the users.

The way to explain it would be that following the install instructions installs nimplex in a nimplex directory. It does not make the importable nimplex for AMMap to call
Essentially, the step that joins the now built nimplex to the python AMMap so it can run nim commands

In the AMMap folder, with the env active after installing nim run this and it works. Updated README and marking as resolved.
git submodule update --init --recursive && nim c --d:release --threads:on --app:lib --out:nimplex.so nimplex/src/nimplex.nim && nim c --d:release --threads:on --app:lib --out:utils/plotting.so nimplex/src/nimplex/utils/plotting.nim


SSTP authentication bug with Windows 10 1803

Prerequisites

[ Yes ] Can you reproduce?
[ Yes ] Are you running the latest version of SoftEtherVPN?

SoftEther version: v4.27-9668-beta-2018.05.29
Component: Server
Operating system: Ubuntu 16.04
Architecture: 64 bit
Description
Can not authenticate users connecting from Windows built-in SSTP client on Windows 10 1803. It works with Windows 10 1709 and below.
Expected behavior:
The username and password are correct and the user is active; the user should be authenticated
Actual behavior:
Error code 691 is returned to the client.
In the server_log_vpn, "User authentication failed." (code 9). And then "Connection has been disconnected." (code 3).
Steps to reproduce

Setup SoftEther server with MS-SSTP enabled.
Create a user with Auth Type of "Password Authentication" or "RADIUS Authentication" (For RADIUS, "Use RADIUS Authentication" and appropriate RADIUS server info are supplied).
Set "Encryption Algorithm Name" to "AES256-SHA".
Create a new Server Certificate with CN as the IPv4 address of the SoftEther server.
Export the X509 Certificate (.cer) from the SoftEther server.
Import the X509 Certificate (.cer) above to the client's certificate store under Local Computer\Trusted Root Certification Authorities\Certificates.
Try to connect to the VPN server via SSTP on Windows 10 1803 (Specifying the IPv4 IP address of the Softether server).

Actually when I tried to reproduce this problem on another Windows 10 1803 computer, it did not appear. Then I ran the following commands to repair the computer with this issue:
dism /Online /Cleanup-image /Restorehealth
sfc /scannow

After that, SSTP worked correctly now. I guess this was because Windows was corrupted. Please close this issue.

Actually when I tried to reproduce this problem on another Windows 10 1803 computer, it did not appear. Then I ran the following commands to repair the computer with this issue:
dism /Online /Cleanup-image /Restorehealth
sfc /scannow

After that, SSTP worked correctly now. I guess this happened because Windows was corrupted. Please close this issue.

Well i was having same problem with 1809, i managed to solve my problem with your hint. I'm not in any domain but sstp disconnecting after a seconds, still i was able to solve this by changing network identity to home computer from system settings.


Allow for environment specific tf templates in the config dirs.

Right now, every .tf file in a config dir will be combined, unlike those in the leaf.  We should allow for the same level of granularity in the config dirs ('env') so you can group config environment templates just like you can in the leaf.
ie:
foo/
foo/env/
foo/env/remote.tf.dev
foo/env/remote.tf.base
etc...
that would be combined all the way up depending on the same rules for the leaf.

old / irrelevant.  - closing


feat(node): add forge node

Motivation
Add support for forge node

Solution

[x] general RPC support
[ ] Eth API bindings
[x] axum HTTP server
[x] axum WS server

[ ] ws specific handler

[x] axum RPC handlers
[x] EthApi interface

[ ] EthRequest dispatch

[x] Node revm Database Backend
[ ] revm call transactor
[x] Transaction Pool
[ ] Mining strategies
[ ] forge node command

should we rebase this to master so that it doesn't have the revm commits?

yeah will fixup all revm related commits

I see a tracing service mentioned in the commits. Fwiw Besu has support for open telemetry if your looking for a reference that is useful.

Random mind dump from my accumulation of building this in cevm back in the day (see here):

We should not match eth nodes 1:1 in capabilities, we should have additional nonstandard features such as:

forge_setBlock: used for setting reported block number
forge_setRpcUrl: used for forking - forward requests to addresses that we dont have to mainnet
forge_mining: bool to set either time based block number increment
forge_setTimestamp: set block timestamp
forge_enableTraces: turn on call traces for transactions that are returned to the user when they execute a transaction (instead of just txhash/receipt)
eth_sendUnsignedTransaction: execute a transaction regardless of signature status

ideally this is used for dry runs of a solidity scripting system (the backbone that will be behind forge deploy, that will allow for arbitrary scripting capabilities, and testing of the transaction system)

Random mind dump from my accumulation of building this in cevm back in the day (see here):

We should not match eth nodes 1:1 in capabilities, we should have additional nonstandard features such as:

forge_setBlock: used for setting reported block number
forge_setRpcUrl: used for forking - forward requests to addresses that we dont have to mainnet
forge_mining: bool to set either time based block number increment
forge_setTimestamp: set block timestamp
forge_enableTraces: turn on call traces for transactions that are returned to the user when they execute a transaction (instead of just txhash/receipt)
eth_sendUnsignedTransaction: execute a transaction regardless of signature status

ideally this is used for dry runs of a solidity scripting system (the backbone that will be behind forge deploy, that will allow for arbitrary scripting capabilities, and testing of the transaction system)

thanks for this.
will add them to the EthApi type to track this and keep this in mind moving forward with the impl.

Not explicitly required, but we should probably strive for compatibility with trace_replayTransaction etc for logging our traces. A separate thing would be to allow our traces to print state diffs, which is probably something we'd PR into the Tracing inspector

Random mind dump from my accumulation of building this in cevm back in the day (see here):

We should not match eth nodes 1:1 in capabilities, we should have additional nonstandard features such as:

forge_setBlock: used for setting reported block number
forge_setRpcUrl: used for forking - forward requests to addresses that we dont have to mainnet
forge_mining: bool to set either time based block number increment
forge_setTimestamp: set block timestamp
forge_enableTraces: turn on call traces for transactions that are returned to the user when they execute a transaction (instead of just txhash/receipt)
eth_sendUnsignedTransaction: execute a transaction regardless of signature status

ideally this is used for dry runs of a solidity scripting system (the backbone that will be behind forge deploy, that will allow for arbitrary scripting capabilities, and testing of the transaction system)

Perhaps its a good idea to work with the hardhat team to standardize some RPCs for these kinds of actions so that tests can be written in a portable way. Right now hardhat has a hardhat RPC namespace that includes similar methods

re: some comments on one of my ethers changes I've been diving into this PR, I wanted to see how much we could share with ethers, and comment on what's missing w.r.t transaction types.
Based on some messing around with types (link to a rough PoC on top of this PR), I think the node presents a need for a signed request type, as opposed to the existing Transaction and TransactionRequest types in ethers.
Quick primer on the differences between these types, and what we need
The ethers Transaction type:

Represents a response from a provider for APIs such as eth_getTransactionByHash.
Contains a signature via r, s, v fields, although not a Signature.
Has fields like transaction_index, block_hash, block_number in case the tx is in a block.
Is not an enum type like the ethers TypedTransaction, instead using a transaction_type field.
Must contain a hash, also includes a method to compute it.

The ethers TypedTransaction type:

Represents a transaction request and is used in some part as input to the following APIs:

eth_sendTransaction
eth_call
eth_estimateGas
eth_createAccessList
eth_createAccessList
trace_call
trace_callMany
If we want, the discussed eth_sendUnsignedTransaction

Is used as input to signers
Implements transaction types by being an enum
Does not include a signature

In the node, we need to work with signed transaction requests in a ton of places, and TypedTransaction wouldn't work because it's unsigned. While the Transaction type would probably work, I would prefer something more like this (use in the node can be seen here):
use ethers_core::types::{
    transaction::eip2718::TypedTransaction,
    Signature,
};

/// Represents a signed transaction request
pub struct SignedTransaction {
    pub tx: EthersTypedTransaction,
    pub signature: Signature,
}

We could introduce this type into ethers and move the decoding changes in ethers-rs#1096 onto that type.
The methods required for the node to work are implemented in the PoC, with notes and questions for implementation.
The PoC removes most of the transaction types introduced in the node.
If we decide to introduce a type like SignedTransaction we could remove the rest of the transaction types.
What do you all think about adding something like this in ethers so we can simplify things in the node?
I can help integrate into the node ofc if we decide to introduce something!

fork mode is currently blocked by #1212

Update

there are probably still lots of bugs in there and somethings are missing

[ ] various rpc endpoints
[ ] nice logs with produced blocks, tx etc...
[ ] better test coverage
[ ] ...

We should mention Anvil in the README as well :smile:

update #3
todo

[ ] e2e test support for existing repos
[ ] support tracing endpoints, parity+geth
[ ] pubsub support

[x] add rpc endpoints

[ ] finish configurable logging
[ ] cleanup and proper abstractions

What's missing? Can take another pass if we're good - excited

What's missing? Can take another pass if we're good - excited

I think this is pretty complete and should be usable, there could be some quirks with fork mode but those we can fix when they come up.
all major and most common things should be supported, including EIP-1559 transactions.
we have/lack

[x] http
[x] ws + subscriptions
[x] filter subscriptions, poll based
[x] most common endpoints
[x] logs (probably need to add some additional logs like tx info)
[x] fork mode (reset impl missing)
[x] mining modes: auto, interval, none
[ ] evm_snapshot
[x] parity style traces

some parts of the codebase could deserve another pass and could be implemented in a nice way with additional unification effort in ethers but are fine for now.
would appreciate another pass

to run locally from this branch
cargo build --bin anvil --profile local --all-features
./target/local/anvil --help

baseFee can not be lower than 7 wei fwiw
will try and build from source again as it failed earlier
💯🔥🧑🍳

Metamask has a hardcoded limit of<PHONE_NUMBER>370476 for chainId, see https://github.com/MetaMask/metamask-extension/blob/12a088e638759b709b475a723157622f946de063/shared/constants/network.js#L36

Random mind dump from my accumulation of building this in cevm back in the day (see here):

We should not match eth nodes 1:1 in capabilities, we should have additional nonstandard features such as:

forge_setBlock: used for setting reported block number
forge_setRpcUrl: used for forking - forward requests to addresses that we dont have to mainnet
forge_mining: bool to set either time based block number increment
forge_setTimestamp: set block timestamp
forge_enableTraces: turn on call traces for transactions that are returned to the user when they execute a transaction (instead of just txhash/receipt)
eth_sendUnsignedTransaction: execute a transaction regardless of signature status

ideally this is used for dry runs of a solidity scripting system (the backbone that will be behind forge deploy, that will allow for arbitrary scripting capabilities, and testing of the transaction system)

To add to @brockelmore suggestions:
forge_setAccessList see example https://github.com/manifoldfinance/golang-eth-patches/blob/master/AccessListOnState.go.patch
forge_multicall see example https://github.com/manifoldfinance/golang-eth-patches/blob/master/EthMultiCall_RPC.go.patch
for forge_enableTraces - can this be forge_enableGethTraces / forge_enableParityTraces wrt those clients?

approved - modulo my comments on integration tests against uniswap v3-core not passing + how should we be thinking about the waffle patch required in v3-periphery

The only tests failing for uni v3 are
a) tests that expect a tx to consume a certain amount of gas
b) tests that expect a specific revert message
currently not sure why a) happens, could just be differences in how revm/hh measure.
since these all test against snapshot files, they could also be automatically migrated.
b) happens because waffle has hardhat decoding baked in https://github.com/TrueFiEng/Waffle/blob/master/waffle-chai/src/matchers/revertedWith.ts#L80
I think we could just get this fixed in there as well
rn the hardhat-waffle plugin is only compatible with hardhat due to this
https://github.com/NomicFoundation/hardhat/blob/fd1f7f8cdb06d8e4ee7fb5fce15c4c48730ca45c/packages/hardhat-waffle/src/waffle-provider-adapter.ts#L16-L23
I made a drop-in replacement plugin here:
https://github.com/mattsse/hardhat-anvil this currently lacks proper anvil config
that could be massively simplified if the network name would not be enforced in hardhat-waffle

maybe forge create  could deploy contracts to anvil by default?


Disable inside games would be nice.

I usually strafe in World of Warcraft and other similar games, I use QWE to move, and when I jump with space bar E + Space activate the PowerAccent. Thank you.

OK, perfect, I will wait, thank you very much.


tide::Result should default to Into as the Ok-type

In order for inference to work we must now write:
async fn list(req: Request<State>) -> tide::Result<impl Into<Response>>;

This should be:
async fn list(req: Request<State>) -> tide::Result;

We should make Into<Response> part of the default alias. Thanks!

This is looking harder than it seems. Published a branch here https://github.com/http-rs/tide/tree/generic-result, but it's looking like the lang may not support this yet. Going to go ahead and close this. If folks want inference to work they'll have to spell out Result<impl Into<Response>>.


Refactor lexer tests to expose test function

Summary
Use the example here to refactor the lexer tests so that the test function is exposed in those suites. This allows the IDE to detect tests for better integration and runnability. Also determine if Ava has native support for repeating tests on different data.
Other Information
No response
Code of Conduct

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

A major drawback of the approach in the link is that all lexer assertions would be consolidated into a single test function, making it difficult to diagnose multiple lexer issues simultaneously. To that end, do not use the above example.

It seems that using Ava's macros is the ideal approach here. We have the benefit of both reusing test implementation logic and being able to execute an individual test via the IDE.

:tada: This issue has been resolved in version 2.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update ms.date field.

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

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

TTYL, MSBOT;

#sign-off

@ahgyger : Thanks for your contribution to the Azure documentation! The author, @ahgyger, has been notified to review your proposed change.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It�s easy to access the private repo version of your articles in GitHub. Just add �pr to the URL, as follows:
Public repo URL: https://github.com/Azure/azure-content/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Azure/azure-content-pr/blob/master/articles/azure-portal-overview.md
We've closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions.


Deprecated: Return type of Ramsey\Uuid\Uuid::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize():

Deprecated: Return type of Ramsey\Uuid\Uuid::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange]
attribute should be used to temporarily suppress the notice in /sugar/vendor/ramsey/uuid/src/Uuid.php on line 216
PHP Version - 8.1.13
Sugar - 12.3.0
OS - Mac

What version of ramsey/uuid do you have installed? Run composer info ramsey/uuid to find out which version is installed in your project.
I'm not familiar with "Sugar." What is that?

Hi,
Ramsey/uuid version is 3.9 and Sugar is nothing but CRM(SugarCRM).
Regards,
Pankaj
On Tue 7 Mar 2023 at 04:07, Ben Ramsey @.***> wrote:

What version of ramsey/uuid do you have installed? Run composer info
ramsey/uuid to find out which version is installed in your project.
I'm not familiar with "Sugar." What is that?
—
Reply to this email directly, view it on GitHub
https://github.com/ramsey/uuid/issues/498#issuecomment-1457493070, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHNH2TQVR4EN2EZL6PZ3E63W22YA3ANCNFSM6AAAAAAVRR6P64
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
ThanksPankaj Sapkal

Thanks. It looks like SugarCRM is using an old version of ramsey/uuid. Do you know where I can find their "community" edition so I can see if it's feasible to submit a pull request to them to update the library?

I have the same issue with v4.7.4 using php version 8.2.6
I looked at the old issue https://github.com/ramsey/uuid/issues/439 which mentioned this being fixed in v3.9.7 - which it was, however I'd rather use the newest version.
3.9.7 Uuid::jsonSerialize function
/**
     * Converts this UUID object to a string when the object is serialized
     * with `json_encode()`
     *
     * @return string
     * @link http://php.net/manual/en/class.jsonserializable.php
     */
    #[ReturnTypeWillChange]
    public function jsonSerialize()
    {
        return $this->toString();
    }

v4.7.4 Uuid::jsonSerialize function - this was changed in 4.0.0
/**
     * Converts the UUID to a string for JSON serialization
     */
    public function jsonSerialize(): string
    {
        return $this->toString();
    }

I have the same issue with v4.7.4 using php version 8.2.6

Are you saying that you're seeing the deprecation message when using ramsey/uuid v4.74 on PHP 8.2.6?

deprecation already mentioned on v4.7.5 on PHP 8.3.2
It for eg. occurs on mocking at in phpunit
/current/vendor/phpunit/phpunit/src/Framework/MockObject/Generator/MockClass.php(50) : eval()'d code:3 TestStub_UuidInterface_d9d94739 implements the Serializable interface, which is deprecated. Implement __serialize() and __unserialize() instead (or in addition, if support for old PHP versions is necessary)
if we check the SerializableInterface of PHP which is used in UuidInterface it mentioned:
As of PHP 8.1.0, a class which implements Serializable without also implementing__serialize()and__unserialize()will generate a deprecation warning
solution would be to remove the interface or implement the functions


Static linking

What was wrong?
How was it fixed?
To-Do

[ ] OPTIONAL: Update Spec if applicable

[ ] Add entry to the release notes (may forgo for trivial changes)

[ ] Clean up commit history

Codecov Report

Merging #180 (b1f3de9) into master (8b00838) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #180   +/-   ##
=======================================
  Coverage   77.93%   77.93%
=======================================
  Files          45       45
  Lines        3300     3300
=======================================
  Hits         2572     2572
  Misses        728      728

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 8b00838...b1f3de9. Read the comment docs.

Codecov Report

Merging #180 (b1f3de9) into master (8b00838) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #180   +/-   ##
=======================================
  Coverage   77.93%   77.93%
=======================================
  Files          45       45
  Lines        3300     3300
=======================================
  Hits         2572     2572
  Misses        728      728

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 8b00838...b1f3de9. Read the comment docs.


When cargo can't run because it doesn't have permissions to a file, error messages should mention which file is causing the problem

I'm running the pre-packaged Windows GNU version of Rust on Windows 7 64-bit. I use Avast Antivirus.
When my antivirus is enabled, I cannot use any other cargo commands but new. All other commands return Access is denied. (os error 5).
After disabling my antivirus' processes, other Cargo commands work fine.
Is this something which could be prevented inside Cargo or is it something that all users should "know" and fix themselves? Is this a known problem?
Building and running exes created with plain rustc worked fine when AV was running, so it is related to something Cargo is doing.

This may just be a case where we need to provide a better error message. Could you gist a full log perhaps? I'd also be curious to figure out which function is actually returning this error...

Sometimes this issue prevents running Cargo commands at all, as in the executed Rust application hangs and does not output anything when using cargo run. Ctrl-C breaks the execution though. Running cargo build does nothing (no output) but doesn't hang either.
I can't see documentation on logging anywhere. Any idea where to look for log files or how to debug the execution? Should I clone and build to be able to do any proper debugging? I could not find logs anywhere where applications usually do logs.

Strange, I have now disabled my antivirus as much as possible (uninstalling is the only option left right now), and still the Cargo commands result in
> Access is denied. (os error 5)

Running without Cargo works fine:
$ cd target/debug && ./app.exe
> Hello, world!

How are you seeing the Access is denied. (os error 5) error message? Is Cargo printing that? If cargo build works (which it looks like it did if target/debug/app.exe was created), then if you run cargo build --verbose or perhaps cargo run --verbose then the output should appear on stdout.

cargo build worked temporarily after juggling AV settings, hence the built exe. Both cargo run --verbose and cargo build --verbose are broken, displaying the os error 5 message. Leaving out --verbose does not print the error message, but instead tells me to use --verbose to get a better error message.
Examples of what I see in my terminal:
$ cargo clean --verbose
> could not remove build directory
>
> Caused by:
>    Access is denied. (os error 5)

$ cargo run --verbose
>     Compiling app v0.1.0 (file:///D:/Projects/Rust/app)
> Access is denied. (os error 5)

$ cargo build --verbose
>     Compiling app v0.1.0 (file:///D:/Projects/Rust/app)
> Access is denied. (os error 5)

$ cargo build
>     Compiling appv0.1.0 (file:///D:/Projects/Rust/app)
> An unknown error occurred
>
> To learn more, run the command again with --verbose.

I found that the os error 5 comes from Windows itself: https://msdn.microsoft.com/en-us/library/windows/desktop/ms681382(v=vs.85).aspx
Cargo is running to some permission wall and uses the Windows error message there.
Most searches I made on that error relate to SQL server running on Windows Server. People have been told to fix it by running the SQL server process itself in elevated mode.
The directory permissions for my project directory matches what I've always used for my whole system. All other tools which have not explicitly stated that elevated permissions are needed have worked fine (e.g. build tools, compilers, software installers, and so on). The above error messages appear on an elevated cmd.exe prompt too, so it is not about the permissions of the terminal trying to run Cargo.
I'll attempt to elevate cargo.exe itself and see if that makes it work.

Note that access can also be denied if it tries to open a file which another process currently has a lock on. It's not just about permissions. Sure would be nice if Cargo would tell you what operation it was trying to do which caused that error.

Apparently there were four(!) ghost processes that did not want to die until I suffocated explorer.exe. After I managed to get rid of the ghost processes, Cargo commands worked fine after I disabled Avast Antivirus to the brink of uninstallation.
Does Cargo shove processes it cannot "run" to the background for explorer.exe (or the terminal parent process) to handle? Sometimes when a process is tied to explorer.exe it cannot be terminated as is, but the explorer.exe process needs to be rebooted first (Windows stability protection at work).
I think the issue here is the behaviour Cargo does when a Rust program cannot be run properly through cargo run: it releases it to the wild, hence the users themselves have to find out how to end the process properly. Perhaps add a test to verify that Cargo has the permissions to operate on the files and then fail with a more descriptive message in case Cargo does not have the permissions? Is this possible without major modifications to the Cargo logic?

At the very least whenever Cargo tries to access or execute a file, if it errors, it should include the path to the file in the error.

Did not see such behaviour in my setup. It either hanged or exited with that os error with no information on what exactly caused the error. Not sure if this relates to me using the pre-built binaries instead of a custom built Cargo binary.

I believe this can be closed. I get:
error: cannot read "/home/restioson/IdeaProjects/vertex/client/gtk/.flatpak-builder/cache/objects/54/2b14975bb28ba3a0b02b1145c19f9a4ee911e95e586e3b6897687530139e5f.file/containerd"

Caused by:
  Permission denied (os error 13)

Which clearly mentions the file that is causing the problem.

@Restioson This issue is for some situation on Windows where "Access is denied" is encountered, not "Permission denied". Possibly related to anti-virus. Cargo has dozens (hundreds?) of places where it accesses files, and any one of them could have raised that error without printing the path.

Ah I see.

Cargo is difficult to use on my system because of this ghost process problem:
Downloaded rayon-core v1.9.0
  Downloaded miniz-sys v0.1.12
  Downloaded png v0.5.2
  Downloaded 21 crates (901.4 KB) in 9.71s
error: The process cannot access the file because it is being used by another process. (os error 32)

c:\dev\ash\examples>cargo run --bin triangle
error: The process cannot access the file because it is being used by another process. (os error 32)

c:\dev\ash\examples>cargo run --bin triangle
error: The process cannot access the file because it is being used by another process. (os error 32)

c:\dev\ash\examples>cargo run triangle
error: `cargo run` could not determine which binary to run. Use the `--bin` option to specify a binary, or the `default-run` manifest key.
available binaries: texture, triangle

@seanvikoren Can you run with the RUST_BACKTRACE=1 and  CARGO_LOG=trace environment variables set and copy the output?

Hello guys,
I just went through this and, if someone has the same problem, it can help:
I use the mongodb database locally running on docker, this action created a directory named data within the structure of my project. For some reason, the cargo watch only came back to work now that I ran a chown on data directory.
Then for diagnostics, check the permissions of the files that are at the same level as your project.
The image below is an example of how it didn't work:

And below, an example of how I made it work:

I'm going to close this as probably fixed.  Extra context was added to error messages in #8232 among other PRs.  Otherwise, without additional information on the specific cases reported here, I don't think any progress can be made.
For anyone encountering similar problems, please open a new issue with as much detail as you can provide.

Same problem:
when "cargo run":
error: failed to link or copy C:\Users\Martin\projects\rust\hello-cargo\target\debug\deps\hello_cargo.exe to C:\Users\Martin\projects\rust\hello-cargo\target\debug\hello-cargo.exe
Caused by:
Acceso denegado. (os error 5)


sparkDriver Exception while crawling

Hi
As soon as I hit the command to crawl, I see the exception which will result in stop of the execution of the code
aditya@aditya-Inspiron-5520:~/gitRepos/sparkler.git$ bin/sparkler.sh crawl -id sjob-1487441495596 -m local[*] -i 1
2017-02-18 10:14:03 WARN  NativeCodeLoader:62 [main] - Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 WARN  Utils:70 [main] - Service 'sparkDriver' could not bind on port 0. Attempting port 1.
2017-02-18 10:14:03 ERROR SparkContext:95 [main] - Error initializing SparkContext.
java.net.BindException: Cannot assign requested address: Service 'sparkDriver' failed after 16 retries!
at sun.nio.ch.Net.bind0(Native Method)
at sun.nio.ch.Net.bind(Net.java:433)
at sun.nio.ch.Net.bind(Net.java:425)
at sun.nio.ch.ServerSocketChannelImpl.bind(ServerSocketChannelImpl.java:223)
at sun.nio.ch.ServerSocketAdaptor.bind(ServerSocketAdaptor.java:74)
at io.netty.channel.socket.nio.NioServerSocketChannel.doBind(NioServerSocketChannel.java:125)
at io.netty.channel.AbstractChannel$AbstractUnsafe.bind(AbstractChannel.java:485)
at io.netty.channel.DefaultChannelPipeline$HeadContext.bind(DefaultChannelPipeline.java:1089)
at io.netty.channel.AbstractChannelHandlerContext.invokeBind(AbstractChannelHandlerContext.java:430)
at io.netty.channel.AbstractChannelHandlerContext.bind(AbstractChannelHandlerContext.java:415)
at io.netty.channel.DefaultChannelPipeline.bind(DefaultChannelPipeline.java:903)
at io.netty.channel.AbstractChannel.bind(AbstractChannel.java:198)
at io.netty.bootstrap.AbstractBootstrap$2.run(AbstractBootstrap.java:348)
at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:357)
at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:357)
at io.netty.util.concurrent.SingleThreadEventExecutor$2.run(SingleThreadEventExecutor.java:111)
at java.lang.Thread.run(Thread.java:745)
Exception in thread "main" java.lang.reflect.InvocationTargetException
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at edu.usc.irds.sparkler.Main$.main(Main.scala:47)
at edu.usc.irds.sparkler.Main.main(Main.scala)
Caused by: java.net.BindException: Cannot assign requested address: Service 'sparkDriver' failed after 16 retries!
at sun.nio.ch.Net.bind0(Native Method)
at sun.nio.ch.Net.bind(Net.java:433)
at sun.nio.ch.Net.bind(Net.java:425)
at sun.nio.ch.ServerSocketChannelImpl.bind(ServerSocketChannelImpl.java:223)
at sun.nio.ch.ServerSocketAdaptor.bind(ServerSocketAdaptor.java:74)
at io.netty.channel.socket.nio.NioServerSocketChannel.doBind(NioServerSocketChannel.java:125)
at io.netty.channel.AbstractChannel$AbstractUnsafe.bind(AbstractChannel.java:485)
at io.netty.channel.DefaultChannelPipeline$HeadContext.bind(DefaultChannelPipeline.java:1089)
at io.netty.channel.AbstractChannelHandlerContext.invokeBind(AbstractChannelHandlerContext.java:430)
at io.netty.channel.AbstractChannelHandlerContext.bind(AbstractChannelHandlerContext.java:415)
at io.netty.channel.DefaultChannelPipeline.bind(DefaultChannelPipeline.java:903)
at io.netty.channel.AbstractChannel.bind(AbstractChannel.java:198)
at io.netty.bootstrap.AbstractBootstrap$2.run(AbstractBootstrap.java:348)
at io.netty.util.concurrent.SingleThreadEventExecutor.runAllTasks(SingleThreadEventExecutor.java:357)
at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:357)
at io.netty.util.concurrent.SingleThreadEventExecutor$2.run(SingleThreadEventExecutor.java:111)
at java.lang.Thread.run(Thread.java:745)

Caused by: java.net.BindException: Cannot assign requested address: Service 'sparkDriver' failed after 16 retries!

This seems a temporary issue in your environment.
Looks like the spark driver is unable to get a port it wanted. May be there are some security constraints or maybe there is some other process using it.
Just check if you have any other processing using port 4040. I usually do lsof -i :4040 and kill that process if it is not required.
If there isn't any other process, then I don't see a reason why it was unable to get that port. Maybe you have older JDK? Is your JDK version is 1.8?
If this problem remains in your environment, I recommend trying bin/dockler.sh, that will give you ubuntu container.

I a, trying on Ubuntu and also on Google Cloud. Issue is same. No other process is running on 4040.

Solved this issue. Was related to local environment.


Make Package Load faster

(@v1.10) pkg> activate --temp
  Activating new project at `/var/folders/ds/_tfd5bfd0bn51v_5xqxy821h0000gn/T/jl_Ey9COp`

julia> using Pkg

julia> @time Pkg.add("FinanceModels")
  2.605898 seconds (6.90 M allocations: 679.492 MiB, 4.80% gc time, 36.52% compilation time: 34% of which was recompilation)

julia> @time using FinanceModels
  1.909036 seconds (4.09 M allocations: 337.071 MiB, 3.66% gc time, 15.32% compilation time: 53% of which was recompilation)

julia> versioninfo()
Julia Version 1.10.2
Commit bd47eca2c8a (2024-03-01 10:14 UTC)
Build Info:
  Official https://julialang.org/ release
Platform Info:
  OS: macOS (arm64-apple-darwin22.4.0)
  CPU: 8 × Apple M3
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-15.0.7 (ORCJIT, apple-m1)
Threads: 4 default, 0 interactive, 2 GC (on 4 virtual cores)
Environment:
  JULIA_NUM_THREADS = auto

with #178, about 21% of the using time has been eliminated:
julia> @time using FinanceModels
  1.516557 seconds (2.96 M allocations: 192.880 MiB, 4.40% gc time, 22.95% compilation time: 59% of which was recompilation)

removing UnicodePlots does quite a bit:
julia> @time using FinanceModels
  1.024806 seconds (2.03 M allocations: 136.206 MiB, 5.04% gc time, 14.48% compilation time: 96% of which was recompilation)


Device.serialNumber cannot be searched on

HAPI v4.0.0
As already supported by HAPI is the attribute "serialNumber" of Device. But you cannot search of filter for it.
GET /Device?serialNumber=123 produces:
{   "resourceType": "OperationOutcome",   "text": {     "status": "generated",     "div": "

Operation Outcome

ERROR	[]
Unknown search parameter \"serialNumber\". Value search parameters for this search are: [_id, _language, device-name, identifier, location, manufacturer, model, organization, patient, status, type, udi-carrier, udi-di, url]
	\n\t\t\t\t\t\n\t\t\t\t\n\t\t\t
\n\t\t
\n\t
"   },   "issue": [     {       "severity": "error",       "code": "processing",       "diagnostics": "Unknown search parameter \"serialNumber\". Value search parameters for this search are: [_id, _language, device-name, identifier, location, manufacturer, model, organization, patient, status, type, udi-carrier, udi-di, url]"     }   ] }

The FHIR Spec doesn't include a serialNumber SearchParameter: https://www.hl7.org/fhir/device.html#search
You have to create a custom SearchParameter and Upload it to your hapi instance to enable searching for serialNumber:
https://blog.smilecdr.com/2017/02/08/custom-search-parameters-in-hapi-fhir/
and/or you can discuss the adding that parameter to Device on chat.fhir.org in the Implementers stream: https://chat.fhir.org/#narrow/stream/179166-implementers
Closing, as this is not a hapi issue.

Alright, thanks!


Improve wording of adminconsent target

@omondiatieno - can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


feat(dashboard): inspect dispatchers in dashboard

Signed-off-by: Bugen Zhao<EMAIL_ADDRESS>I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
We'll show the all dispatchers for each actor in the fragment. Take the delta-join plan as an example.

What's it like before?

It was not clickable. 🥺


CreateWallet: allow disabling wallet until it's confirmed by another …

…endpoint

A new inspection was created.

The inspection completed: No new issues


HW counters/lower-bound-overhead

Piggybacks on perf to aggregate hardware counters and compute lower-bound-overhead from Cai et al ("Distilling the Real Cost of Production Garbage Collectors").
Example of usage:
../julia_master/julia hw_measurements.jl "../julia_master/julia" "../julia_pr/julia" "./benches/serial/bigint/pollard.jl"

Example of output:
Running pollard.jl on ../julia_master/julia
Measuring cycles
Measuring instructions
Measuring cache-misses
Measuring page-faults
Running pollard.jl on ../julia_pr/julia
Measuring cycles
Measuring instructions
Measuring cache-misses
Measuring page-faults
┌─────────────────────┬───────────────────────┬──────────────────────┐
│                     │ ../julia_master/julia │    ../julia_pr/julia │
├─────────────────────┼───────────────────────┼──────────────────────┤
│       LBO in cycles │  1.4050708230999985e9 │ 1.1315522388999996e9 │
│ LBO in instructions │  2.0782619566000023e9 │ 1.9208291789000015e9 │
│ LBO in cache-misses │   6.552438599999998e6 │ 5.8233013999999985e6 │
│  LBO in page-faults │     17403.79999999999 │   10940.899999999994 │
└─────────────────────┴───────────────────────┴──────────────────────┘

Cycles, instructions, cache-misses and page-faults are measured for now, but can easily be extended to a larger set of hardware counters measured by perf.
Only works on linux.

I think this could be double-counting write-barriers and unrelated GC functions. Will replace with a callback that counts corresponding HW events.


Check array length before access the item in redirectToCache

Pelican Version:
Run ./pelican --version to get version number.
Pelican Service:

[ ] Client
[ ] Plugin
[ ] Registry
[ ] Director
[ ] Origin
[ ] Cache
[ ] Other (please give the detail)

Describe the bug
A clear and concise description of what the bug is.
I saw these panics spam the OSDF production director log:
2024/06/06 16:13:58 [Recovery] 2024/06/06 - 16:13:58 panic recovered:
runtime error: index out of range [0] with length 0
/usr/local/go/src/runtime/panic.go:114 (0x43927b)
/pelican/director/director.go:327 (0x15940d7)
/pelican/director/director.go:576 (0x1622c8e)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/context.go:174 (0xb4246a)
/go/pkg/mod/github.com/zsais/go-gin-prometheus@v0.1.0/middleware.go:364 (0x1297064)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/context.go:174 (0x129ab9a)
/pelican/web_ui/authorization.go:82 (0x129ab88)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/context.go:174 (0x12aa049)
/pelican/web_ui/ui.go:535 (0x12aa030)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/context.go:174 (0xb4e499)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/recovery.go:102 (0xb4e487)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/context.go:174 (0xb4d44a)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/gin.go:656 (0xb4d437)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/gin.go:649 (0xb4d1a4)
/go/pkg/mod/github.com/gin-gonic/gin@v1.9.1/gin.go:576 (0xb4cb3c)
/usr/local/go/src/net/http/server.go:2938 (0x7dcc2d)
/usr/local/go/src/net/http/server.go:3546 (0x7dfcd0)
/usr/local/go/src/net/http/h2_bundle.go:6131 (0x7b0637)
/usr/local/go/src/runtime/asm_amd64.s:1650 (0x4720e0)

Looking at the code, we didn't seem to check the length of originAds before accessing the item:
https://github.com/PelicanPlatform/pelican/blob/a61898943c980b4b14d2e3dd511c79c2a0da6a16/director/director.go#L327C2-L327C62
To Reproduce
Steps to reproduce the behavior.
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Linux, MacOS]
Browser: [e.g. chrome, safari, if the bug relates to any web user interface]

Additional context
Add any other context about the problem here.

Assigned to @jhiemstrawisc as this is taken care of in his PR.


Rule idea: jsx-unique-key

I'd like to propose a rule that triggers when duplicate key properties are defined in on the same JSX nesting level or in arrays containing JSX at the top-level element.
React depends on unique keys and will render very unexpected results when duplicate keys are present which shoult to my knowledge almost never be intentional.
Examples where the rule would trigger an error:
const div = (

);

const spans = [
  ,
  ,
];

The first example shouldn't warn because key is irrelevant there, because it's not inside an array.

I'm going to solve this by adding a warnOnDuplicates option.

The first example shouldn't warn because key is irrelevant there, because it's not inside an array.

I think any HTML sibling nodes can be affected by key conflicts.

@silverwind can you provide a react codesandbox that illustrates such a problem? if so, I can try to include it in the warnOnDuplicates option prior to release.

I think the worst that can happen with non-array cases is unnecessary re-renders. That said, React does warn at runtime for the first example I provided above.

Warning: Encountered two children with the same key, notunique. Keys should be unique so that components maintain their identity across updates. Non-unique keys may cause children to be duplicated and/or omitted — the behavior is unsupported and could change in a future version.

Ok thanks, I’ll reopen this then to cover the first case with the new option.

@ljharb: Since https://github.com/yannickcr/eslint-plugin-react/commit/f51ec4551b3fc85d971fc4d65414684bfabe3974, jsx-key is now being triggered for all elements inside a React.Fragment. Is that intentional? Our CI failed with 7.29: https://github.com/microsoft/rnx-kit/runs/5333921004?check_suite_focus=true
The code that it trips on: https://github.com/microsoft/rnx-kit/blob/main/packages/test-app/src/App.native.tsx#L39

It triggers now on all elements inside my React app - also on elements that are not in an array.

With this, I have now 7195 new lint errors for things that are completely not related to an array that never tripped before. For example:
            <Fragment>
              <div data-testid={`${testId}-message`}>{error.message}

              <div data-testid={`${testId}-stack`}>{error.stack}

            </Fragment>

Causes:
  49:15  error  Missing "key" prop for element in array  react/jsx-key
  49:15  error  Missing "key" prop for element in array  react/jsx-key
  50:15  error  Missing "key" prop for element in array  react/jsx-key
  50:15  error  Missing "key" prop for element in array  react/jsx-key

@stefcameron yes, thank you. that's #3215.

@tido64 if you're getting a different problem than #3215, please file a new issue.


NLTK relation extraction

I am trying to extract relations from a text
the example that is provided in the book uses ieer corpus
I tried to make it on a single text but it's not working , I am not getting anything
Can anyone help ?
Here is my code :
import nltk
import re
sent = "Barack Obama is the president of America"
tokens = nltk.word_tokenize(sent)
tagged = nltk.pos_tag(tokens)
grammar = r"""
NP : {<DT|PP$>?}
{+}
{+}
}<VBD|IN>+{
"""
cp = nltk.RegexpParser(grammar)
result = cp.parse(tagged)
IN = re.compile(r'.\bin\b(?!\b.+ing)')
headline=["test headline for sentence"]
for i,sent in enumerate(tagged):
text = nltk.ne_chunk(sent)
for rel in nltk.sem.relextract.extract_rels('ORG', 'LOC', sent, corpus='ace', pattern=IN):
print(nltk.sem.rtuple(rel) )

Please take a look at https://www.nltk.org/book/ch07.html
Here's an example:
import nltk
import re

sent = "Barack Obama is the president of America"

# Tokenize and POS tag the sentence
tokens = nltk.word_tokenize(sent)
tagged_sent = nltk.pos_tag(tokens)

# Chunk the pattern determiner <DT>,
# followed by noun <NN*>
# then a prepositional phrase made up of
# a preposition and proper noun <IN><NNP>+
grammar = "NP: {<DT><NN*>+<IN><NNP>+}"

cp = nltk.RegexpParser(grammar)
print(cp.parse(tagged_sent))

[out]:
(S
  Barack/NNP
  Obama/NNP
  is/VBZ
  (NP the/DT president/NN of/IN America/NNP))


Build to blank screen on osx

Every time I build my electron application it is successful but when I run the application Im presented with a blank screen. Im positive that it is finding my main.js and using it as modifications here result in changes in the built application. Any help with this would me much appreciated it.
Here is my build command
$ sudo electron-packager . foo --platform=all --arch=x64 --version=0.31.2 --overwrite

Here is my main.js
var app = require('app');  // Module to control application life.
var BrowserWindow = require('browser-window');  // Module to create native browser window.

// Report crashes to our server.
require('crash-reporter').start();

// Keep a global reference of the window object, if you don't, the window will
// be closed automatically when the JavaScript object is garbage collected.
var mainWindow = null;

// Quit when all windows are closed.
app.on('window-all-closed', function() {
  // On OS X it is common for applications and their menu bar
  // to stay active until the user quits explicitly with Cmd + Q
    app.quit();
});

// This method will be called when Electron has finished
// initialization and is ready to create browser windows.
app.on('ready', function() {
  // Create the browser window.
  mainWindow = new BrowserWindow({width: 1400, height: 700});

  // and load the index.html of the app.
  mainWindow.loadUrl('file://' + __dirname + '/templates/index.html');

  // Open the DevTools.
  // mainWindow.openDevTools();

  // Emitted when the window is closed.
  mainWindow.on('closed', function() {
    // Dereference the window object, usually you would store windows
    // in an array if your app supports multi windows, this is the time
    // when you should delete the corresponding element.
    mainWindow = null;
  });
});

Try doing console.log('file://' + __dirname + '/templates/index.html') and make sure the path is correct

The correct file path is returned. The application runs correctly when use  electron . just not when compiled.

I have the same problem... Would like some help...
Also running on mac, and the directory exists. When i run with electron, my app opens, but when packaging it starts a blank screen with no error.

I have the exact same problem as gillyb running electron-packager 5.2.1 and using this command:
electron-packager . myapp --ignore="node_modules/(electron-packager|electron-prebuilt)" --platform=win32 --arch=ia32 --version=0.36.3 --overwrite --asar=false

It seems that we're going to need someone to debug this for us.

Need a failing test case from someone first

Mind having a look at Console on OS X and see if any messages appear when the app launches?

Hi guys,
I'm running in to something similar too.
I'm using using shelljs in my main process (and sending data using ipc)
When I start the app from a terminal:
/Applications/dcapp.app/Contents/MacOS  ./Electron

After poking around with your app, it appears that running your app via doubleclicking in Finder does not run with nearly the same PATH that running from a terminal does, so depending on where the drupal command ends up installed in your case, it could be that it ends up not finding it when run that way?
I'm not sure if the other users reporting this issue are in the same boat - do they use shelljs or child_process.exec/etc. in their apps?  Is there a difference running the packaged app via terminal vs. Finder for them?

Normally you'd run the app by typing open /Applications/dcapp.app, not /Applications/dcapp.app/Contents/MacOS/Electron.

@erkyrath in my testing, both have the same effect (both work, as opposed to opening directly from Finder).

I don't think this is a path issue. The which command should return the location of the arg you pass, if it's in your PATH..... Running my code with npm start works, and from the terminal. Thanks guys for taking a peek at this, I'll be back to poking at it tomorrow.
@erkyrath thx, I'm not that familiar with OSX :)

@willwh I agree with @kfranqueiro, because last month I helped troubleshoot a similar environment variable problem on OSX. If you look at https://github.com/maxogden/electron-packager/issues/232#issuecomment-170225945, you will note that the PATH variable differs between opening it via Finder and via the terminal. In particular, the /usr/local/{bin,sbin} paths are added to PATH in the terminal. If the Drupal Console docs are any indication, drupal is likely installed in /usr/local/bin on your system. If this is the case, you should add /usr/local/bin to PATH before running which.

@kfranqueiro @malept thanks for sticking with me and explaining that!
Of course I had the binary for this aliased in my .zshrc.
alias=~/DrupalConsole/bin/drupal
However, after removing this (and source ~/.zshrc) - when I run which drupal in my terminal, I see the correct path: `/usr/local/bin/drupal' - although drupal console still doesn't seem to be working in my packaged app.
I'm going to try repackaging, do you have any other advice here? :)
Thank you once again!

For your app, the short-term fix is to append /usr/local/bin to process.env['PATH'] before you call which('drupal'). For a longer-term fix, I suggest creating a configuration option to set how you execute it, given that the docs I linked earlier say that it only exists in /usr/local/bin if you want to be able to execute it system-wide.

@malept thanks so much for the advice :)
I tried just setting shell.env.PATH += shell.sep + '/usr/local/bin'; in my main,js but still the same problem.
Any other ideas on how I might debug what's going on?

The only advice I can give at this point is to start putting console.log statements everywhere.

You said you tried setting shell.env.PATH, but AFAIK shell is not a thing that exists...  Did you try setting process.env.PATH as @malept said?  That works for me.

@kfranqueiro thank you, sorry typo late at night :)
The interesting thing is, that still doesn't work for me, and indeed, if I drop some console.log(process.env.PATH) before and after that - I end up with /usr/local/bin in my $PATH twice (if I run open /Applications/dcapp.app
i.e.
function createWindow () {
  // Create the browser window.
  mainWindow = new BrowserWindow({width: 800, height: 600});
  console.log(process.env.PATH);
  process.env.PATH += '/usr/local/bin';
  console.log(process.env.PATH);

Results in /usr/local/bin in my path twice.
I can't use console.log() to check things from the main process, so I am not sure what to do here.
@kfranqueiro when you say that "works for me" - you're able to open dcapp.app from finder / launch from the Applications directory by double clicking and it shows a desktop notification and the version of Drupal Console?

I can't use console.log() to check things from the main process, so I am not sure what to do here.

I believe you can if you open the DevTools Window. (Look at the tutorial and search for DevTools.) You may have to do some IPC to get the data where you want it, though. I haven't tested.

This is what I am doing now :)
The interesting thing is, I have the following:
mainWindow.loadURL('file://' + __dirname + '/index.html');
mainWindow.webContents.openDevTools();

and I am trying to use IPC to send data between the main and renderer process.
ipcRenderer code
ipcMain code
As I've said, this works when opening the application from a terminal using open /Applications/dcapp.app - but not when run from finder.
So now I am thoroughly confused :)
I really appreciate your help.

I meant, send the contents of process.env.PATH (for example) over IPC (over, for example, a debug channel) so you can console.log() and see it in the DevTools window.

@malept something like this? https://github.com/willwh/dcapp/commit/a758d9e66e768ca635dc7df27ff92a8e5b0a8f38
I get nothing in the DevTools window. I wonder if this is an issue with my implementation of the IPC stuff.

Thanks guys for all of your help here. I got IPC stuff working correctly now, and this indeed a path issue.
@malept I actually misread your first note; re: /usr/local/{bin,sbin}
So when launched from Finder, or the Icon, my path is /usr/local/{bin,sbin}
I tried adding the following inside my createWindow() function; process.env.PATH = ['/usr/local/bin', process.env.PATH].join(':') - but that doesn't seem to work.
Once I figure out where I should be setting the process.env.PATH in my code, I think I'll be able to call this done :)
(Just for testing purposes: sudo ln -s /usr/local/bin/drupal /usr/bin/drupal and my app works as expected, I am not sure why setting the process.env.PATH like above is not working.

I'm going to give this a quick test: https://github.com/sindresorhus/fix-path it sounds like it may solve my problem :)

Ah well, back to the drawing board I guess, this doesn't seem to work.

This issue has kind of turned into a catch-all, so I'm going to close it.
IF YOU HAVE THIS PROBLEM

Read our docs on filing bugs and see if your specific problem already has another issue created for it (whether it's open or closed)
If you believe you have a different set of problems, file a new issue and follow the directions in the GitHub issue template.

Not sure if this is related to your problem, but please check Console it shows some errors log if any strange things occur. Mine is because of Uncaught Error https://github.com/onmyway133/blog/issues/145


How Do I Stop Generation of  and  elements?

This line in my template
{{input searchTerm action="search" placeholder="Search..." id="searchThing"}}

creates a  and a
, and then creates the <input> inside the
.
Search term

   <input id="searchThing" class="ember-view form-control ember-text-field" placeholder="Search..." type="text">

I don't want the  or the
 elements, just the <input>. How do I stop them from being generated?

You can register a "wrapper" with a custom template in for the label and input. Please check this section of the README: https://github.com/dockyard/ember-easy-form/tree/1-0-stable#wrappers

The library was updated to be compatible with Ember 1.13 and 2.*.
You can get the most recent version of ember-easy-form on the branch 1-0-stable or as an ember-addon here.
Make sure to read the README and CHANGELOG on the 1-0-stable branch.


Fixed the stray validation error message in Resource Deployment wizard

This PR fixes #13679 and fixes #13719
I have already extensively tested this change locally and I found 1 bug: Table 'withValidation' does not work. @Charles-Gagnon  what can be a potential fix for this issue?
@ranasaria and @Charles-Gagnon,
Please run this branch locally to see if it has fixed all the other edge cases. If not then please report the issues to this PR.
Initial State when a tool is not installed: (@Charles-Gagnon, @ranasaria   shouldn't this be an error message?)

If you click next without installing the tool:

If you click install button:

Once the tool installed:

When you click the next button:

Pull Request Test Coverage Report for Build 412602444

2 of 18 (11.11%)  changed or added relevant lines in 2 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 41.495%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/resource-deployment/src/localizedConstants.ts
2
3
66.67%

extensions/resource-deployment/src/ui/toolsAndEulaSettingsPage.ts
0
15
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/resource-deployment/src/ui/toolsAndEulaSettingsPage.ts
1
2.84%

extensions/notebook/src/jupyter/serverInstance.ts
4
73.81%

Totals

Change from base Build 412058638:
-0.02%

Covered Lines:
22056

Relevant Lines:
48761

💛  - Coveralls

From Description:
Initial State when a tool is not installed: (@Charles-Gagnon, @ranasaria shouldn't this be an error message?)
Issues that can be resolved using automation provided in the current ui are considered 'warnings'.


Add config overrides for ironic

This adds the necessary variables for ironic to communicate with
glance for image upload/retrieval.
Issue: RI-313
Issue: RI-313

also, are these being tracked for removal as well (for when the fix is upstreamed)?

Works now.


Added the zns_file_writer class, can be compiled, not tested yet

Added the zns_file_writer class, can be compiled, not tested yet

你下次先test了 再merge吧


Change details can't display well if updating is known after apply in-place update

The terraform-j2md get diff as Change details from before and after of plan result. But if update is known after apply in-place update, after doesn't have known after apply but afterr unknown.

This is resolved by v0.0.7 https://github.com/reproio/terraform-j2md/releases/tag/v0.0.7


Not downloading private account posts

Describe the bug
Back few days we can download posts of private accounts by login but now it's always skips posts and start downloading directly stories [ I'm not using --no-posts switch]. So it should download them too as it does before.
I've try with --stories and --highlights with both it skips posts
To Reproduce
Steps to reproduce the behavior:
instaloader --no-metadata-json --no-video-thumbnails --no-compress-json --filename-pattern "{profile}-{date}s" --stories
targetusername -l myusername -p mypass
(e.g. Instaloader command line)
It tries Retrieving posts from profile user
But failed I guess.
Expected behavior
It should retrieve posts of profile it does if I don't use login then it download public profiles.
I think here problem with login.
Error messages and tracebacks
There is no error it just skip them.
Instaloader version
4.12.1 from pip
Additional context
Add any other context about the problem here.

Duplicate of #2319


Stats stop sending after a while in v4.1.1

This was a known bug in v4.1.1, please update to v4.1.2+ as soon as possible! I have left this here in case somebody does a search for this issue.

Resolved by: https://github.com/hazardfn/riemannx/commit/ab7adc6abb5e7785ab448dc5f6ceac992614b1c1


Issue 438 - Build on JDK 17 targeting Java 11

Resolves #438

Still need to update the lambda deployments and the docker images


Avoid normalizing headers in maybe_add

maybe_add already assumes that the arguments are normalized, so we shouldn't repeat that work.

This was included in https://github.com/fastly/js-compute-runtime/pull/844.


How to validate values and keys?

I'm having trouble with a DB structure that looks like this:
{
  "post-flag": {
    "-KvXU5eiR4oZM-vBihib": {
      "6t5KEJjAOSYbf8dleabDlTyxhux2":<PHONE_NUMBER>604,
      "6t5KEJjAOSYbf8dleabDlTyxhux3":<PHONE_NUMBER>605
    }
  }
}

I'd like to validate that the data is valid and that the keys follow a certain format.
I've tried several variations of the following, but none of them really work.
path /post-flag is Map<PushID, Map<UserID, UnixTimestamp>>;
path /post-flag/{post}/{user} is UnixTimestamp {
  write() {
    return isCurrentUser(user) && root.post[post] != null
  }
}

type PushID extends String {
  validate() { return this.test(/^[-_0-9a-zA-Z]{20}$/); }
}

type UserID extends String {
  validate() { return this.test(/^[0-9a-zA-Z]{27}[0-9]$/); }
}

type UnixTimestamp extends Number {
  validate() { return this >= 0; }
}

For the example above I get the following JSON:
{
  "rules": {
    "post-flag": {
      "$key1": {
        ".validate": "newData.hasChildren() && $key1.matches(/^[-_0-9a-zA-Z]{20}$/)",
        "$key2": {
          ".validate": "$key2.matches(/^[0-9a-zA-Z]{27}[0-9]$/) && newData.isNumber() && newData.val() >= 0"
        }
      },
      ".validate": "newData.hasChildren()",
      "$post": {
        "$user": {
          ".validate": "newData.isNumber() && newData.val() >= 0",
          ".write": "auth != null && auth.uid == $user && newData.parent().parent().parent().child('post').child($post).val() != null"
        }
      }
    }
  }
}

Which yields the following error in Firebase simulator:

Cannot have multiple default rules ('$key1' and '$post').

I'm not sure if I'm writing my rules wrongly or if this is a limitation of Bolt.

Possibly you want something like:
function isCurrentUser(user) {
  return auth != null && auth.uid == user;
}

path /post-flag is Map<PushID, Map<UserID, UnixTimestamp>> {
  write() {
    return isCurrentUser(user) && root.post[key1] != null
  }
}

type PushID extends String {
  validate() { return this.test(/^[-_0-9a-zA-Z]{20}$/); }
}

type UserID extends String {
  validate() { return this.test(/^[0-9a-zA-Z]{27}[0-9]$/); }
}

type UnixTimestamp extends Number {
  validate() { return this >= 0; }
}

Slightly tricky but you have to know the first map gets a key generated of $key1. Bolt doesn't really understand how to merge paths between types and wildcards.

@rockwotj Thanks for the quick reply.
I thought that might be the case... user should then become key2, right?
path /post-flag is Map<PushID, Map<UserID, UnixTimestamp>> {
  write() {
    return isCurrentUser(key2) && root.post[key1] != null
  }
}

Oops yes, good catch. I've updated my answer as well.

So the only way I've made it to work is I duplicate the rule and use keyX as variable name:
path /post-flag is Map<PushID, Map<UserID, UnixTimestamp>>;

path /post-flag/{key1} {
  write() {
    return root.post[key1] != null
  }
}

path /post-flag/{key1}/{key2} {
  write() {
    return isCurrentUser(key2)
  }
}

Isn't there any sort of aliasing to instruct Bolt to map variables to certain names? Like:
path /post-flag is Map<PushID as post, Map<UserID as user, UnixTimestamp>>;

Currently there is not, but feel free to open a PR :smile:

I'll have a go at it. :)