Dynamic type problem

Hi all, how can I execute dynamic query on DynamicObject array?
this work (with anonymous object)
var quarableList = new[] { new { TypeName = "PROCESS.ITEM.END" } }.ToList().AsQueryable();
var resr = quarableList.BuildQuery(testRule).ToList();

but this doen't :(
var data = @"{
	'Id': 999,
	'ParentId': null,
	'TypeName': 'PROCESS.ITEM.END',
	'Name': 'END'
}";
           var testRule = new FilterRule
            {
                Condition = "and",
                Field = "TypeName",
                Id = "Id",
                Input = "NA",
                Operator = "equal",
                Type = "string",
                Value = "PROCESS.ITEM.END",
            };

            var expObjData = JsonConvert.DeserializeObject<dynamic>(data);
            var quarableList = new[] { expObjData  }.ToList().AsQueryable();
            var resr = quarableList.BuildQuery(testRule).ToList();

Thanks,
Best Regards,

Hi @tghamm , @cbrianball , @willydavidjr , @elialgranti , I found a solution and wrote it on my fork and commit. Do you think it is suitable ?
https://github.com/MasallahOZEN/dynamic-linq-query-builder/commits/master
Best regards,

Hi again @tghamm , @cbrianball , @willydavidjr , @elialgranti , I found a new solution and commit. Do you think it is suitable ?
//Json string model
var dataItem = @"{
'Id': 999,
'PersonImsId': 123,
'CreatorImsId': 123
}";
//typed model
var boundRefDataItem = new BoundedReferenceDataActionItem {
FormId = 1,
Mode = 1,
ComponentId = 12,
StateId = 1
};
=> ExpandoField = generated from dataItem.
//Adding new field to dataItem model
ExpandoField.Add("FirstName", "Masallah");
ExpandoField.Add("LastName", "ÖZEN");
ExpandoField.Add("Age", 36);
//Add new complex type to typed model
boundRefDataItem.DataItem = ExpandoField;
//Submodel filtering
var rule = new FilterRule
{
Field = "DataItem.Age",
Id = "FirstName",
Input = "NA",
Operator = "greater_or_equal",
Type = "integer",
Value = "36"
};
Thnx
Best Regards,

Hey, sorry for the slow response @MasallahOZEN .  I'd be open to some of your changes if you wanted to lay out what they accomplish and submit PRs.  Thanks!


Does this gem is still maintained/works?

Title explains my question.

:+1: would like to know!

I suspect this project is no loger being maintained because of it's last commit timestamp and the PR's and issues opened for quite a while.
I've recently tried to use this gem and it is throwing an error on rails start. After some research I've found that activeadmin project has many locales on it's github repository. I've learned on the internatilonalization section of the documentation page that you can just copy the desired locales to your project's config/locales folder and it should be automatically translated.
Please notice that some Rails internationalization issues may occur after that, and you can find the quick answer on this stack overflow question
I hope that helps others looking to use activeadmin's translations.

The globalize gem used in this project stores localized translations of Rails models in the database, and you can hardly substitute it with services like localize. You can't compare it with simple i18n gems, either. If you are an international company that can't rely on machine translation, globalize is the only way to let content managers translate model content.


[gha] update create-issue

Motivation
Use the latest create-issue in diem/actions that does not depend on event.payload in github context.  This fixes the issue in https://github.com/diem/diem/runs/3030532680?check_suite_focus=true (the same issue seen and fixed in diem/partners)
Test Plan
Canary https://github.com/diem/diem/runs/3030857049?check_suite_focus=true

/land

/cancel

/land

:broken_heart: Test Failed - ci-test

/land

:broken_heart: Test Failed - ci-test

/land

:broken_heart: Test Failed - ci-test

/land


added info section on ico/ido

fixed #799

@vanshchauhan21 please reduce to level 1 or I have to remove your project from gssoc


Fix signed/unsigned bug in Value.fromCBOR

It looks like the author meant to do:
if(reader.peek() === UNSIGNED_INT) { reader.readUInt() }

instead of
if(reader.peek() === UNSIGNED_INT) { reader.readInt() }

Hi @itsfarseen thanks for the feedback, in our typescript CBOR implementation both readInt and readUInt map to a bigint (because we dont have signed types in javascript and bigint is large enough in both directions), and particularly readInt is 'smart' enough to figure out if the number is negative or positive and do the right thing when deserializing, so this wont cause any issues here.
That being said, the readUInt version will throw if it finds a signed number which is probably semantically more correct, so I am happy to accept the change, but please fix the commit message to use conventional commits, I.E:
fix(core): deserialization of Value now checks that both coin and multiasset amount are unsigned

Not sure if the same bug is present in the following lines as well. Coins and quantity can't be negative right?

Correct, you can apply the same change here.

@itsfarseen thanks for the PR but we're unable to merge given it's incomplete. Will you be able to address @mkazlauskas feedback?

I will continue this work on https://github.com/input-output-hk/cardano-js-sdk/pull/1482 since this commit is in a fork


Move TCL test unit/expire to Go case

I'd like to work on it, can assign to me :)

@maochongxin assigned. Go ahead!

Inactive. Unassign to release the seat.

Closed by #955. Thank you @maochongxin!


Dead link

The link to "Find some supporting examples at Key Vault - Managed Storage Account Keys Samples." is dead.

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

ID: cebec73e-e7f4-75c5-2068-ae89f12d8d95
Version Independent ID: e2497a7b-3b5f-554a-1a31-da540740fcc9
Content: Azure Key Vault Storage Account Keys
Content Source: articles/key-vault/key-vault-ovw-storage-keys.md
Service: key-vault
GitHub Login: @lleonard-msft
Microsoft Alias: alleonar

@eskaufel Thanks for the feedback! We are currently investigating and will update you shortly.

@eskaufel - thanks very much for reporting this. The link has been fixed in the article you mention, as well as the SDK readme.md page.
#please-close

@eskaufel We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


For me, Azure stopped the VM when I added the disk so you must restart it before connecting.

For me, Azure stopped the VM when I added the disk so you must restart it before connecting

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

ID: aa688416-61ed-5a02-eab0-4a39b6cdc533
Version Independent ID: c4aded8c-ae13-f863-362d-cc77f72fa023
Content: Attach a managed data disk to a Windows VM - Azure - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/attach-managed-disk-portal.md
Service: storage
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

@chris-arbogast
Thanks for your feedback! We will investigate and update as appropriate.

Hi @chris-arbogast
To add an ultra disk to an existing virtual machine, you need to follow these steps:
Stop the virtual machine.
Select Disks.
Select Additional settings.
Select Yes for Enable Ultra Disk compatibility.
Select Save.
Select Create and attach a new disk and fill in a name for your new disk.
For Storage type select Ultra Disk.
Change the values of Size (GiB), Max IOPS, and Max throughput to ones of your choice.
After you are returned to your disk's blade, select Save.
Start your virtual machine again.
You can also use Azure CLI to enable ultra disk compatibility on an existing virtual machine.
Please refer below document, it might be helpful.
https://learn.microsoft.com/en-us/azure/virtual-machines/disks-enable-ultra-ssd?tabs=azure-portal
https://learn.microsoft.com/en-us/azure/site-recovery/azure-to-azure-troubleshoot-replication
https://learn.microsoft.com/en-us/azure/virtual-machines/windows/tutorial-manage-data-disk
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Hi,
I feel like you misunderstood my comment.  This page https://learn.microsoft.com/en-us/azure/virtual-machines/windows/attach-managed-disk-portal gives step by step directions but there is an omission that should be corrected.
Before step 1 in “initialize a new data disk”, there should be a step to start the VM.
-Chris
From: Saibaba Balapur Hireka @.>
Sent: Monday, February 27, 2023 11:35 PM
To: MicrosoftDocs/azure-docs @.>
Cc: Arbogast, Chris @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] For me, Azure stopped the VM when I added the disk so you must restart it before connecting. (Issue #105868)
WARNING: This is an external email. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Hi @chris-arbogasthttps://github.com/chris-arbogast
To add an ultra disk to an existing virtual machine, you need to follow these steps:
Stop the virtual machine.
Select Disks.
Select Additional settings.
Select Yes for Enable Ultra Disk compatibility.
Select Save.
Select Create and attach a new disk and fill in a name for your new disk.
For Storage type select Ultra Disk.
Change the values of Size (GiB), Max IOPS, and Max throughput to ones of your choice.
After you are returned to your disk's blade, select Save.
Start your virtual machine again.
You can also use Azure CLI to enable ultra disk compatibility on an existing virtual machine.
Please refer below document, it might be helpful.
https://learn.microsoft.com/en-us/azure/virtual-machines/disks-enable-ultra-ssd?tabs=azure-portal
https://learn.microsoft.com/en-us/azure/site-recovery/azure-to-azure-troubleshoot-replication
https://learn.microsoft.com/en-us/azure/virtual-machines/windows/tutorial-manage-data-disk
If there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/105868#issuecomment-1447710186, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQOHMERRABBP4KYNZPEJZATWZWTBHANCNFSM6AAAAAAVJ6XDUQ.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@chris-arbogast
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


⚠ Bofrost.fr has degraded performance

In c3ebb88, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1582 ms

Resolved: Bofrost.fr performance has improved in 8905f52 after 6 minutes.


NFF based BNG-PGW primed w/ S11 simulator :: handling GTP-U traffic

Implemented in fixed_bng branch.


youki fails to run in docker-in-docker with cgroups v1

Reproduction:
cargo install cross --git https://github.com/cross-rs/cross
git clone --branch dind git@github.com:jprendes/youki.git
cd youki
cross build --features systemd,v1,v2 --bin youki
./dind.sh

Removing --runtime=youki from dind.sh, the example runs fine, but with youki it fails.

Old error message
The error I receive is
docker: Error response from daemon: failed to create task for container: failed to create shim task: OCI runtime create failed: unable to retrieve OCI runtime error (open /var/run/docker/containerd/daemon/io.containerd.runtime.v2.task/moby/278419ddb31fde6701841c5b302a632751de2a5a6aa051a62530b12ee8ab4763/log.json: no such file or directory): /youki did not terminate successfully: exit status 1: unknown.
ERRO[0003] error waiting for container:

This originates from tying to run runwasi on docker-in-docker. In that case it works fine on cgroups v2 but fails on cgroups v1, and the error is while mounting cgroups.

New error message
After workind around the issue with journald, I get an error when using youki with dind + cgroups v1.
libcontainer::rootfs::mount: failed to canonicalize "/sys/fs/cgroup/systemd/docker/af0c557cb9806654e1a9eac1de4a12afda57bf9f60952e4c3db016668c554ea6": No such file or directory (os error 2)

See https://github.com/containers/youki/issues/2528#issuecomment-1809985629 for log context.

From the dockerd log files
cleanup warnings
    time="2023-11-14T09:19:48Z"
    level=warning
    msg="failed to remove runc container"
    error="/youki did not terminate successfully: exit status 1: failed to initialize observability: No such file or directory (os error 2)\nError: No such file or directory (os error 2)\n"
    runtime=io.containerd.runc.v2
    time="2023-11-14T09:19:48Z"
    level=warning msg="failed to read init pid file"
    error="open /run/docker/containerd/daemon/io.containerd.runtime.v2.task/moby/0ad434dd1d96105d31e416174c5c4d3b6a721c32586b612218dcad3cb68bb9f6/init.pid: no such file or directory"
    runtime=io.containerd.runc.v2
    namespace=moby

I believe fixing this particular error would be the first step, and not the final solution.
As I mentioned before, runwasi fails on cgroups mounting, so eventially we should hit the same issue.

I believe fixing this particular error would be the first step, and not the final solution.

I think this is a side effect of failure to create container, just showing up as different error. The failed to read init pid file is mostly because the mounting failed, hence the init process creation didn't happen, hence there is no pid file created. I think this would get automatically fixed by addressing the original issue, not a separate problem.

The above error is due to journald logging.  BrokenChannel
2023-11-14T10:35:57.086701Z ERROR libcontainer::container::builder_impl: failed to run container process err=Channel(ReceiveError { msg: "waiting for init ready", source: BrokenChannel })

I think this is a side effect of failure to create container, just showing up as different error. The failed to read init pid file is mostly because the mounting failed, hence the init process creation didn't happen, hence there is no pid file created. I think this would get automatically fixed by addressing the original issue, not a separate problem.

I think the interesting part of that error message was the failed to initialize observability part.
Working around that issue, I get to the same issue as in runwasi.

What happens if we use the cgroup v2 driver instead of the systemd driver?

I might be missunderstanding you, but the host has cgroups-v1, using a v2 driver would not be correct, right?

it looks like when I run it, it's taking the path of setup_emulated_subsystem instead of setup_namespaced_subsystem.
That branch is controlled by cgroups_ns.
If I ignore cgroups_ns and unconditionally use setup_namespaced_subsystem everything works fine.
I'm not familiar with the need for emulation here. Why do we need it and how does it relate to cgroups_ns?

IIUC, runc doesn't have an "emulated" mode.
https://github.com/opencontainers/runc/blob/main/libcontainer/rootfs_linux.go#L253

I think I have a fix for it: https://github.com/containers/youki/compare/main...jprendes:youki:fix-dind
It still needs some kind of test. Unfortunately the docker image is built on alpine so it needs a musl build of youki.

I think I have a fix for it: main...jprendes:youki:fix-dind
It still needs some kind of test. Unfortunately the docker image is built on alpine so it needs a musl build of youki.

Thanks for this fix:
https://github.com/containers/youki/compare/main...jprendes:youki:fix-dind#diff-7593ca57a3e963ffce568c39b57a4632daf6378db43c6f66b0f3eac191abaf94R83-R91
I also experience this issue when developing youki in a dev container.

I'll create a PR after adding some test

I might be missunderstanding you, but the host has cgroups-v1, using a v2 driver would not be correct, right?

Yes, you can see these information using youk info.

What happens if we use the cgroup v2 driver instead of the systemd driver?

How would I do that?

youki info:

Output
$ sudo ./target/x86_64-unknown-linux-musl/debug/youki info
DEBUG youki: started by user 0 with ArgsOs { inner: ["./target/x86_64-unknown-linux-musl/debug/youki", "info"] }
Version           0.3.0
Commit            c7567ab4
Kernel-Release    5.15.0-88-generic
Kernel-Version    #98~20.04.1-Ubuntu SMP Mon Oct 9 16:43:45 UTC 2023
Architecture      x86_64
Operating System  Ubuntu 20.04.6 LTS
Cores             8
Total Memory      7936
Cgroup setup      hybrid
Cgroup mounts
  blkio           /sys/fs/cgroup/blkio
  cpu             /sys/fs/cgroup/cpu,cpuacct
  cpuacct         /sys/fs/cgroup/cpu,cpuacct
  cpuset          /sys/fs/cgroup/cpuset
  devices         /sys/fs/cgroup/devices
  freezer         /sys/fs/cgroup/freezer
  hugetlb         /sys/fs/cgroup/hugetlb
  memory          /sys/fs/cgroup/memory
  net_cls         /sys/fs/cgroup/net_cls,net_prio
  net_prio        /sys/fs/cgroup/net_cls,net_prio
  perf_event      /sys/fs/cgroup/perf_event
  pids            /sys/fs/cgroup/pids
  unified         /sys/fs/cgroup/unified
CGroup v2 controllers
  cpu             detached
  cpuset          detached
  hugetlb         detached
  io              detached
  memory          detached
  pids            detached
  device          attached
Namespaces        enabled
  mount           enabled
  uts             enabled
  ipc             enabled
  user            enabled
  pid             enabled
  network         enabled
  cgroup          enabled
Capabilities
CAP_BPF           available
CAP_PERFMON       available
CAP_CHECKPOINT_RESTORE available

Thanks. It looks like using cgroup v2. In that case, if you enable the systemd feature, you use systemd
https://github.com/containers/youki/blob/c538361c2e3b86531fd350a1dcf94e341308f3fa/crates/libcgroups/src/common.rs#L363-L372

Cgroup setup      hybrid

So you mean building without the systemd feature, and trying agian?
cross build --features v1,v2 --bin youki


[msal-browser] Add option to use Secure Cookies to cache configuration

This PR:

Adds the secureCookies flag to CacheOptions type

Adds logic in setItemCookie to check said flag and add the "Secure" flag to the cookie string

Adds unit tests for the secureCookies configuration option

Adds documentation on the secureCookies configuration option

Addresses #2854

Coverage increased (+13.3%) to 95.975% when pulling 376628fa0d91debae5041358a57419d8d14eb107 on secure-cookies into 0e0dd8f1270f4bd3d5687bab6a834fd33453d3f3 on dev.

@hectormmg Good to merge?

Migrating from msal 1.3.2  to msal-browser impact on code and need to move to auth code flow. Cant we provide this option on msal itself for those who are using msal with explicit flow?


Fix 2 errors and protect credentials.json file

@nadirhamid Please review and merge this whenever possible

Please describe the use of .htaccess as I am having trouble understand its use.
--
credentials.json can be set in any directory. as a result this would always need to
be moved.

@nadirhamid The htaccess file will make sure that, if the user is using apache, no JSON file will be server. In other words, trying to access any JSON file via the web browser will result in a "forbidden" error, thus protecting the json file contents. That works no matter in what directory the json file is.


Automatic invenio rdm record creation/updates

Completed via updating the DOI creation GitHub Action to push registered metadata to InvenioRDM 0fd076c8.


Migration never finishes, stalls on ipfs.cat call

I ran the migration script from a git checkout and the process was initially very slow but seems to have completely stalled at the last step:
➜  wnfs-migration git:(issue-2) npm start

><EMAIL_ADDRESS>start
> node --no-warnings --loader=ts-node/esm src/index.ts

Looking up data root for jeffg
Data root is bafybeigcrrjp5vnr77tuh3eo45xhvtv2rsxhwvylxrtxzozp6ncmg7pmra
Loading IPFS...
Connected to local ipfs node, version 0.13.0
Connected to the Fission IPFS Cluster
Looking up your filesystem version (https://jeffg.files.fission.name/version)
Your filesystem currently is at version 1.0.0
Processing public/Apps
....
Processing private/Documents
Finished migration: bafybeia54qqjhbzgp32wzo6phj4olt3fgybn3npzhbnli6cdvil6ulh4zy
?  Yes

After agreeing to overwrite, the terminal sits for a very long time ( 1+ hour and still going ) with no feedback.

I haven't found a solution for this yet, but I did add an extra message about what it's doing. Which is creating a UCAN based on your root proof, which is retrieved via IPFS (IPFS is the slow part).

@icidasset / @matheus23  Is there a way to make the ipfs connection provide verbose logging?

That's really hard unfortunately. It taking more than... instant, means that for whatever reason what you're requesting isn't on your machine. But it should be!
So we should tell js-ipfs to time out, or even force it to only look locally, because that's where it's supposed to find it.
I'm not sure why it can't find it.
So what exactly is happening is this: You locally have a UCAN that authenticates you against your account's root DID.
A CID to that UCAN can be found in your local fission config file at ~/.config/fission/config.yml in the root_proof key.
If you run ~/.config/fission/bin/fission-ipfs -c ~/.config/fission/ipfs/ cat /ipfs/<your_root_proof>/bearer.jwt you'll see that UCAN.
However, exactly that IPFS command isn't resolving in your case.
I've talked to Brooke about this a while ago, we agreed that it would be better to store that UCAN somewhere else: https://talk.fission.codes/t/webnative-on-nodejs/2240/5
What I'm saying is that we've been planning to do something that would've mitigated this issue for you!
I've create a github issue that captures this. https://github.com/fission-suite/fission/issues/578

I have performed some local testings based on the information on this comment and found that the CID never resolves and the DHT search revealed that no peer is providing it. I have established the connection manually to all the address i believe to be the Fission IPFS Cluster inside the config.yaml before running this test.
C:\Users\xxx>ipfs dht findprovs bafy...li

C:\Users\xxx>ipfs dht findprovs bafy...li

C:\Users\xxx>ipfs dht findprovs bafy...li

C:\Users\xxx>ipfs dht findprovs bafy...li

C:\Users\xxx>ipfs dht findprovs bafy...li

C:\Users\xxx>ipfs dht findprovs bafy...li

C:\Users\xxx>

Besides the local daemon, I have tried to access it through the CID directly through the ipfs.io however it also end up with 504 Gateway TImed Out. Which IPFS node(s) is supposed to provide it?

Yeah thanks for confirming that @ngeojiajun.
To be clear: It shouldn't be pinned on any fission IPFS infra and it's unlikely to be served by any IPFS instance in the network unless you're running the fission CLI's IPFS on your machine. It should "just" be in your local IPFS blockstore.
Keep in mind that the fission CLI executable downloads its own ipfs executable and configures that with its own blockstore. So if you've installed ipfs on your system additionally, that'll use a different blockstore.

The ipfs executable that the fission CLI installs can be found at ~/.config/fission/bin/fission-ipfs.
The blockstore and configuration for fission's IPFS can be found at ~/.config/fission/ipfs/*

I'm thinking the root cause of this issue was switching from re-using the fission CLI's go-ipfs executable in wnfs-migration to running its own js-ipfs instance (Introduced here: https://github.com/fission-suite/wnfs-migration/pull/3/commits/40d54c8baa5fef809b2b61c290ca8e6895deea20).
This causes this issue, because then js-ipfs doesn't use the same blockstore that go-ipfs uses.
I think it's not possible for js-ipfs to reuse that blockstore, because they're in different formats.
We can fix this by specifically calling go-ipfs for resolving the root_proof and nothing else. I'll do that.

In my situation, the fission-ipfs also do not resolves it. Is it pinned by default during the fission setup?

Yeah exactly, that's what should've happened @ngeojiajun :thinking:

@ngeojiajun can you please try out wnfs-migration at this branch: #10 :pray:

@matheus23 In progress.... and the fission setup does do its job by pinning it

@matheus23 Update: the UCAN is created and the actual request is seems sent. just my network is bottlenecking the actual upload.
.....
Processing private/マンガ/......pdf
Finished migration: bafybeicfmoflskgdnoohikppuknovsb5jc2dz33b2jxspu7a7q5x2mk6q4
?  Yes
Creating authorization UCAN.
Created authorization UCAN. Updating data root...

@ngeojiajun Okay, this process can time out. However, it makes progress even if it's interrupted, so I've uploaded a new version which re-tries to update the data root in case it timed out. Please try it with the updated branch :pray:

Ok, will try it if it is timed out later on

@matheus23 can i know how long it should times out because the updating process tooks over an hour and it do not finish and the iftop shows that there are no network activity

@therealjeffg can you make sure this issue is resolved?


Corporate Console - Tool tip is not displayed for + icon to add cla manager in Manage White list page

Background

Launch Corporate Console
Login as nirupamavmentee1
Click on Company Issue327Comp40
Click on Manage Whitelist for project
Mouse hover on + icon under CLA Managers section and observed that tool tip is not displayed for user to understand the + icon is to add cla managers

Verified that tool tip is displayed


Upgrade Flow to 0.66.0

Warnings

:warning:

Please add a changelog.md entry for your changes.

  Generated by :no_entry_sign: dangerJS


CORS permissions on JSON feeds - relax?

The JSON feed of /boosts (and /streams when that one comes) currently has standard (if restrictive) CORS settings.
My use-case is that I'd quite like to be able to have a method of pulling my boost messages off my Umbrel and publishing them on the web. My initial thought was to just have a JavaScript page that works internally in my network, that would read the boosts JSON and post them to an API on my public server; but I'm foiled by the CORS policy.
It would be good - and I think without any security issue, given that this page is only viewable on a local network (but that's up to you) - if the CORS policy was open for these JSON feeds.
It's a case of adding the following header, below.
Access-Control-Allow-Origin: *

This might need preflight as well.

Could this be configurable?

Could this be configurable?

What do you mean?

An environment variable for the right side of header so whoever hosts the service can specify the value. None/null means don't include the header and you can set the default to localhost or *.

Ah, I see. Sure that can be done.

All endpoints will be CORS enabled in v0.1.9


#184638220 Integrate HoundCI with badge attached to readme

What does this PR do?
Integrate HoundCI with badge attached to readme
How should this be manually tested?

Check that the badge code is correctly inserted into your README file. You should see the badge displayed in your README file, with a link to HoundCI.
Push your changes to your repository and wait for HoundCI to complete the analysis. You should see the HoundCI status on your pull request or commit.

What are the relevant pivotal tracker/Trello stories?
#184638220

6 commits

The five later commits were done to test if HoundCI checks the commits, kindly consider my first commit
yeah i know it and 6 commits are too much , please make research how to squash commit into one or 2 can better for you

6 commits

The five later commits were done to test if HoundCI checks the commits, kindly consider my first commit
yeah i know it but  6 commits are too much , please make research how to squash commit into one or 2 can better for you

Noted with Thanks


Added useEventSource

This adds useEventSource

Could you rebase the commit from the current master commit, the file changed tab is overhauling. Thanks


dew point

Hey,
I hope this is the right place for a feature request. How difficult would it be to plot dew point? RH is nice, and the % in the location popup is clickable, which makes me think someone once hoped that one could switch into displaying dew point.
Dew point is, to me, a much more useful measurement. It represents more closely heat index (100% humidity at 5°C feels ok, but 100% humidity at 30°C is agony). It also gives a better idea of how much water is in the air. If the dew point is 12°C at 0800 and then the sun rises and raises the temperature by 10°C, the RH goes down, but the dew point might not, depending on whether water is being added to the air.
Please implement a dew point overlay.
Thanks!
Bruce

@bwduncan Do you able to find the temperature folder for finding the temperature based on the location. I am executing this github project on window 10 localhost and it has limited functionality of wind and its speed only.


KAFKA-3378 Fix for instantly connecting SocketChannels (v3)

Added call to channel.prepare() to handle SSL handshake and authentication.

@llowrey : Thanks for the new patch. We plan to cut the 0.10.0 release branch this Monday. Since this is marked as an 0.10.0 blocker, would you have time to finish this up (including the time to address review comments) this week?

@llowrey Would you mind closing this PR since we merged the change via a different one in the end?


Non-fatal error, possible soft-ban?

Received the following failure in the log, bot logged back in without completely crashing, but can't spin. Looks like it had a moment where it thought it was back at the start point of my loop and tried to spin the stops there and then moved on, but still getting spin errors as it passes other pokestops....
2016-08-07 05:18:13,228 [RecycleItems] [INFO] [item_discarded] Discarded 1x Potion (maximum 0).
Traceback (most recent call last):
File "/usr/lib/python2.7/logging/init.py", line 851, in emit
msg = self.format(record)
File "/usr/lib/python2.7/logging/init.py", line 724, in format
return fmt.format(record)
File "/usr/lib/python2.7/logging/init.py", line 464, in format
record.message = record.getMessage()
File "/usr/lib/python2.7/logging/init.py", line 328, in getMessage
msg = msg % self.args
TypeError: not all arguments converted during string formatting
Logged from file init.py, line 550
2016-08-07 05:18:34,700 [PokemonGoBot] [INFO] [login_started] Login procedure started.
2016-08-07 05:18:35,196 [PokemonGoBot] [INFO] [login_successful] Login successful.

Exactly the same problem for me. Happened two times after 29 minutes.

Hrm. Mine was almost 34 minutes... I'll see if it happens again...

I think it is discussed here #2698 and solved with the latest commit.

Yep, 30 minutes and it popped the error.... I'll do a pull and see if that commit is in and resolves things...

Attempted to apply suggested fix, no change, error after 30 min runtime.

@raxxon Which fix did you apply? Fixing the logging turned out to be pointless. The second change of activate_signature did the trick for me. Haven't had the issue since adding that in.

The activate signature call in init

Did you also fix the logging call that causes the exception? Might need both to be fixed for it to work. Would make sense that if the exception is thrown first, then activate signature doesn't get called.

Seems to happen after ~30 min. I can confirm this.
2016-08-07 13:59:00,412 [MoveToFort] [INFO] [moving_to_lured_fort] Moving towards pokestop Huntington Falls Rock - 0.08km (attraction of lure 0.08km)
Traceback (most recent call last):
  File "/usr/lib/python2.7/logging/__init__.py", line 850, in emit
    msg = self.format(record)
  File "/usr/lib/python2.7/logging/__init__.py", line 723, in format
    return fmt.format(record)
  File "/usr/lib/python2.7/logging/__init__.py", line 464, in format
    record.message = record.getMessage()
  File "/usr/lib/python2.7/logging/__init__.py", line 328, in getMessage
    msg = msg % self.args
TypeError: not all arguments converted during string formatting
Logged from file __init__.py, line 550
2016-08-07 13:59:12,660 [PokemonGoBot] [INFO] [login_started] Login procedure started.
2016-08-07 13:59:14,487 [PokemonGoBot] [INFO] [login_successful] Login successful.

Looks like the session has ended, and extended, however throwing that error isn't nice.


[Guestconfiguration] Update atuorest.md to ensure most of the methods could be generated.

This is just a temporary work around.
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.GuestConfiguration


Implement hipGetSymbolAddress and hipGetSymbolSize

@gargrahul, thanks for the comments. I have added a test case and introduced an ihipModuleGetGlobal. hipMemcpy(To|From)Symbol(|Async) were also calling hipModuleGetGlobal, which I have changed to the new function.

@gargrahul, is there anything else you need me to do before this can be meeged?

Jenkins is failing due to build issue on NVCC path. I will try to fix that.
Otherwise LGTM, though I would like to have another test to use memcpy to/from symbol after getting symbol address. I have already tried that and can be added as a new PR.

@mangupta I think we can merge this unless you have any review comment.


Adds data param to pass information to events.

Adds the ability to set the data param for the handler events conforming to the documentation at http://api.jquery.com/on/#passing-data

@usmonster What are your thoughts on tackling this one for v1.9.0?

I think v1.9.0 is focusing more on adding a few fixes and improvements that don't change or extend the API. It's a new major version because some of these changes may initially break certain use cases (e.g. #11), though I want to keep the API as-is for now. This will be considered for the next major version, though, if there is enough user demand for it. Thanks for the pull request though!

This has been moved into a branch, https://github.com/roydukkey/jquery-hoverIntent/tree/data-param.


YOLOXPredictor memory leak when using async functions.

When creating an application I am able to use the Predict method without problems, this predicts like expected. But predicting, blocking the main thread of execution, isn't ideal. That's why I looked into "threading" it. Wrapping this Predict function in a Task, causes it to leak memory at a very fast rate. Remarkable this is only present on iOS (haven't tested Android) and not when used inside the simulated environment. Then it runs like it's supposed to.

The following code shows my first way of doing things:
// Causes memory leak
private async void PredictAsync(){
    await Task.Run(() => {
        Predict(currentFeature, (d) => {
            // Do something with the detections
            this.detections = d;
            isDetecting = false;
        });
    });
}

private void Predict(MLImageFeature feature, Action<(Rect rect, string label, float score)[]> callback){

    (Rect rect, string label, float score)[] detections = predictor.Predict(feature);

    if(callback != null)
        callback(detections);
}

Calling await PredictAsync() causes problems.
Then I got told there was a stock implementation for this, so I decided to give this a try.
var predictor = new YOLOXPredictor(...).ToAsync();

...
...

// Triggered on event by XRCpuImageFetcher
private async void OnImageAvailable(XRCpuImage image){
    if(predictor.readyForPrediction && predictor != null){
        // Might be moved to the image fetcher
        if(currentFeature == null){
            Debug.Log("Creating new feature");
            var conversionParams = new XRCpuImage.ConversionParams(image, TextureFormat.RGBA32, XRCpuImage.Transformation.None);
            currentFeature = new MLImageFeature(conversionParams.outputDimensions.x, conversionParams.outputDimensions.y);
            Debug.Log("Feature created");
        }

        if(cameraTexture == null){
            XRCpuImageToTexture(image);
            visualizer.image = cameraTexture;
        }

        Debug.Log("Copying image to feature");
        MLXRExtensions.CopyFrom(currentFeature, image);

        this.detections = await predictor.Predict(currentFeature);
    }
}

This results in the exact same behaviour.
I also verified it is probably not the XRCpuImage memory management since it works with the syncronous Predict method and it get's disposed by the event that provides it.
using UnityEngine;
using UnityEngine.XR.ARFoundation;
using UnityEngine.XR.ARSubsystems;

public class XRCpuImageFetcher : MonoBehaviour
{
    public ARCameraManager cameraManager;

    void OnEnable()
    {
        cameraManager.frameReceived += FrameChanged;
    }

    void OnDisable()
    {
        cameraManager.frameReceived -= FrameChanged;
    }

    private void FrameChanged(ARCameraFrameEventArgs args){

        if (cameraManager.TryAcquireLatestCpuImage(out XRCpuImage image)){
            ImageAvailable?.Invoke(image);
            image.Dispose();
        }
    }

    public delegate void ImageAvailableDelegate(XRCpuImage image);
    public event ImageAvailableDelegate ImageAvailable;
}

If you need any more information, let me know and I'll look into it!

Fix dropping today:

Fixed in NatML 1.1.2.

Awesome! Will verify asap. Thanks again!

Verified memory usage is stable now! Thanks @olokobayusuf.

Perfect!


document v.0.7.0

[x] Global property substitution https://github.com/tangrams/tangram/pull/263
[x] Uv y meters https://github.com/tangrams/tangram/pull/275
[x] Scene imports https://github.com/tangrams/tangram/pull/278
[x] Selection event handlers https://github.com/tangrams/tangram/pull/279
[x] Raster tile sources w/"attachments" https://github.com/tangrams/tangram/pull/282
[x] Remove 'properties' section from Layer page https://github.com/tangrams/tangram-docs/pull/74
[x] setDataSource()

https://github.com/tangrams/tangram-docs/tree/v0.7.0
PR: https://github.com/tangrams/tangram-docs/pull/77

Global property substitution added with https://github.com/tangrams/tangram-docs/commit/a44fd9b92a186ea28a63c99267b538d52fe1e0ac

line uvs added with https://github.com/tangrams/tangram-docs/commit/05a1b0dd12d503d6249e2062c6dea3f051c3180a

Raster tile sources added with https://github.com/tangrams/tangram-docs/commit/c84a579921ca010465be646abd12a1475a0e8815 , cc @bcamper

Import added with https://github.com/tangrams/tangram-docs/commit/d39decff5da200bc70c273c42e69c53df5c0e1d9

Selection event handlers added with https://github.com/tangrams/tangram-docs/commit/6e8d8c25904ef4a4136e778d20c1d84464e9ae07

setDataSource() updated with https://github.com/tangrams/tangram-docs/commit/d2fb66fcc6a5e6eaf384965b0595118eb8f5b420

Review requested – cc @bcamper

resolved by tangrams/tangram-docs#77


Implement and Test Genre and MusicAlbum Classes

Title: Implement and Test Genre and MusicAlbum Classes
Description:
This PR introduces the Genre and MusicAlbum classes and their associated tests.
The Genre class has three attributes: id, name, and items. The add_item method adds an item to the items array and sets the item's genre attribute to the genre instance.
The MusicAlbum class is a subclass of the Item class with an additional attribute on_spotify. The can_be_archived? method in MusicAlbum overrides the method in Item and returns true if the album is on Spotify and the publish_date is older than 10 years.
The tests cover the initialization of the Genre and MusicAlbum classes, the functionality of the add_item and can_be_archived? methods, and the behavior of the to_json and json_create methods in MusicAlbum.
Changes:

Add Genre class in genre.rb
Add MusicAlbum class in music_album.rb
Add tests for Genre class in genre_spec.rb
Add tests for MusicAlbum class in music_album_spec.rb

Test Plan:

Run the test suite to verify that all tests pass
Create instances of the Genre and MusicAlbum classes in IRB to manually verify the class methods and attributes

Please review and let me know if there are any areas for improvement.

Hi @fmanimashaun
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


bug happens in asl_recognition_demo

https://github.com/opencv/open_model_zoo/blob/c9407ad365340f369a837334777dd6f7d01994a3/demos/python_demos/asl_recognition_demo/asl_recognition_demo/person_detector.py#L57-L59
The code will be check when other detection model is used, e.g. person-detection-retail-0013.
But, when I use person-detection-retail-0013 model, the code get wrong bounding box, which is a little offset. If I change the code as follows, the bounding box is correct.
out = np.array([[1.0, initial_w, initial_h, initial_w, initial_h]]) * result[0, 0, :, 2:]

@evgeny-izutov could you please take a look?

This PR includes fix.

PR has been merged. @kotomiDu, you can check it. Note, the quality of other person detectors (different from person-detection-asl-0001) is expected to be low due to domain shift (different viewpoint angle).

PR has been merged. @kotomiDu, you can check it. Note, the quality of other person detectors (different from person-detection-asl-0001) is expected to be low due to domain shift (different viewpoint angle).

Yes, For this demo, we donnot need to use other person detector.  I find this bug just because I try to use this sample for other person detection models. Anyway, I checked the PR and will close the issue. Thanks.


make 0-arg writeln @trusted: Fix Issue 13681

Following the new guidelines on @trusted use, I hope :)
https://issues.dlang.org/show_bug.cgi?id=13681

LGTM

Technically, I think this looks fine. But I don't understand the boundary here. Why only no-arg writeln? Why not also writeln with a string or integer parameters?
I'd like to find a more correct mechanism to enable @safe writeln calls. While this pull is not incorrect, I feel it's not comprehensive enough. One possibility is making @safe wrappers for the C I/O functions.

@schveiguy writeln is already callable from @safe for other argument types. The problem only exists because writeln does it own special shortcut for 0-arg calls.

Ha! Ok, then, I didn't realize it was already working for others.
Ironically, how you have it is the same as it was until 2012:
https://github.com/d-programming-language/phobos/commit/95c49e3762f729fd256e14578f8bb6e2b171c965#diff-7945921c543e1ab7460797aeb759d7c5L1546
Now, I looked back, and @andralex changed it to fputc a long long time ago (2010). Testing this, I see that writeln() is more than twice as fast as write('\n').
I wonder, can we just mark fputc as @trusted?

I wonder, can we just mark fputc as @trusted?

Yes I think so, would be a better solution.

I wonder, can we just mark fputc as @trusted?

The question is whether or not fputc is guaranteed to be memory safe for any valid FILE* pointer, including null. A quick look at the relevant entry in the C standard doesn't seem to say.

fputc has been marked as @trusted in core.stdc.stdio since 2012. Here is the correct fix: https://github.com/D-Programming-Language/phobos/pull/3046


AeroVariablePropeller3D not rotating in some cases, along with no force being generated.

This issue is occurring really randomly, and is hard to reproduce.
In some cases, when i, or other people, test my game, in some cases the propeller just wont spin.
We can confirm at thrust is to 100% as the propeller sounds get louder, and the thrust stick gets pushed in.
Relevant scenes/nodes (ask if you need anything else)

My game is open source @ https://github.com/cyteon/PlenFS if you wanna look at code

Binaries for game can be found at https://cyteon.itch.io/plenfs?secret=aUyXBK35Mi5j4Fm59uarFMOBNKs

From my testing of your project, it seems that the airplane is falling asleep. The quick workaround is to set can_sleep to false on the AeroBody. However, this does bring up an important point that currently, aerodynamic forces aren't being calculated to wake the body from sleep. The aerodynamic calculations are disabled when the body falls asleep, for performance reasons, however I guess it would make sense that a change in air velocity, or a change in influencer transform should cause the body to wake.

I'll try to change the can_sleep to false, and see if that works. Also thanks for showing a interest in my project :D

Appears fixed

21dd2deac87fe9a6fdba4af448cc5486157c468f Should be a better solution than disabling can_sleep on AeroBody nodes.


Invent a way to find 64-bit dotnet.exe for out-of-process

We already check bitness of dotnet found using where.exe and fallback to appropriate program files directory. We don't have tests for these scenarios thought.


Implement standard zip behavior by default

Fixes #2381 and #1283. Note: I also found and fixed some other issues while testing.

Added Cake.Core.IO.PathCollection (mirroring DirectoryPathCollection and FilePathCollection)
Added Cake.Common.IO.GlobbingAliases.GetPaths (currently internal use only)
Changed Cake.Common.IO.ZipAliases.Zip to use Cake.Common.IO.GlobbingAliases.GetPaths

For zipping a directory only; not used by all other file centric aliases

Refactor Cake.Common.Tests.Unit.IO.ZipperTests to use Cake.Core.Tests.Fixtures.CakeContextFixture

Exercise Cake.Common.IO.GlobbingAliases.GetPaths
Fixed Cake.Testing.FakeFileSystem to work with simple root directory (/)

Cake.Core.IO.Globbing.GlobVisitor.VisitRecursiveWildcardSegment
Cake.Common.IO.Zipper.GetRelativePath
Cake.Testing.FakeFileSystemTree.ctor

Verify zip archive structure, including stored directories
... and stored empty directories (when zipping a directory only)

@devlead do the integration tests run as part of the automated build? I don't see them in the Azure Pipelines logs. I ran them manually and following is the zip archive created by the new integration test:

PS. Should I rebase my branch with develop?

Looks like Build Cake Ubuntu failed with an unrelated error?

I've got one more tweak I want to do: I was being conservative when I initially added GlobbingAliases.GetPaths with internal access only, but since then I added a lot of unit tests, so I think it would be nice to make public.
Besides that, @devlead @mholo65 do you want me to do anything else to keep moving this PR forward?

@gitfool Rebase against latest deveop and squash commits, and I'll review again.

@devlead rebased and squashed; ready for review.

@gitfool your changes have been merged, thanks for your contribution 👍


Bootable image: build bootable image on Fedora.

Later on, we can execute the bootable image and test that it, in fact, boots correctly.

This is in response to https://github.com/zfsonlinux/zfs/issues/4555 .

Awesome!  This is definitely along the right lines but let me make some suggestions inline.

I believe the incoming commits address your comments.

Replaced with #26 where I've squashed your changes and made a few additional fixes.  However, this still doesn't quite work when I ran through the process manually details in the new PR.


[stable/spinnaker] Clean spinnaker deployment only if halyard is available.

Fixes #22554
Signed-off-by: Nirmalya Sen<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #22554

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/needs-ok-to-test

/assign @paulczar

/ok-to-test

@markyjackson-taulia

@Nirmalyasen The conflicts need to be corrected so the tide test can pass

Got messed up in new merge.
Closing this and will raise a new one.


Please add a code block support (or at least PrismJS support)

Hi ! I really enjoy your theme but I can't install prismjs with your theme so I can't highlight my code in my articles. I think it is a good feature to add to your theme :)

Hej! Thanks for opening this issue! Sounds interesting to me ☺
Do you know what exactly needs to be added so that PrismJS can be supported? Perhaps you have some sort of docs?

I don't really know how can PrismJS be supported but I saw many errors when installing it on this theme. I will build a basic website without this theme but only with PrismJS to see how can it be supported

Gatsby actually has its own prismJS plugin. It does not use the default prismJS plugin since it assumes you’re running things client side. I have successfully added code highlighting by following this documentation page: https://www.gatsbyjs.com/plugins/gatsby-remark-prismjs/
Note that after the npm install I was getting errors developing the site, running gatsby clean, then redeveloping resolved any issues.
You will need to update the gastby-config.js as described in the documentation page, as well as add a gatsby-browser.js file in the same location. Instead of adding the line require("prismjs/themes/prism-solarizedlight.css") as pointed out in the documentation, I instead used require("./src/prism.css"), where the src folder is the default src folder in the gastby site's directory. That prism.css file is the css file downloaded from the prismJS website with my desired prismJS theme.
The way the documentation says to do this part may work, I just never tried as I'd rather have the css referenced locally.
After all of this was done, you can highlight code in the markdown files for the blog post articles. The way its done is different than the default prismJS way, so refer to the documentation, it is actually quite simple.
This is a sample of what I was able to produce.

Thanks for the details, @tjoab! Unfortunately, I don't have time to implement it in the near future - so if anyone wants to add it, PR are always welcome!

Hey @konstantinmuenster I think you should close this issue now haha.


X-Transmission-Session-id Header Stripped

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Great work on this project, it's certainly beautified my end users experience when they get an error however...
As transmission uses rpc to serve its pages to protect against CSRF it expects a valid session-id header in the request, otherwise it returns a 409 page.
Without error-pages defined on the entrypoint in Traefik, the request hits the rpc endpoint and on the initial try it obtains a session-id and then retries resulting in a successful response and the page loads.
With error-pages defined, all consequent hits to the rpc endpoint result in a 409 as this error page strips the X-Transmission-Session-id header and therefore gets rejected.
Is there anyway of setting "Pass-through request/response headers" or an abilitiy to set a list of headers that are allowed to be kept?
Thanks in advance.

No response

Hi @jonwilliams84!
I think you should ask your question in the traefik community as most relevant.

Traefik community - https://community.traefik.io/

Hi @jonwilliams84!
I think you should ask your question in the traefik community as most relevant.

It's not related to traefik though. If I create a custom error page in nginx and add config to allow the header described, it works fine.

Could you please show nginx configuration that works in your case as expected?
And anyway, I suppose that taking care of the proxying custom HTTP headers from the requests to the responses is out of this project's tasks (and it must be done at the layer above - that's why I said about the traefik community).

If you will not find options with the Traefik, please, check the changes in PR #67 - I believe this is exactly what you need.
After your feedback & tests, I will make a release with those changes 🤝

Docker image for a tests: tarampampam/error-pages:proxy-headers
Run it with passing environment variable PROXY_HTTP_HEADERS=X-Transmission-Session-id

Ping @jonwilliams84

Released in v2.7.0

Apologies...been flat out the past 2 days. I will test tonight, but thank you for the swift (potential) fix.

Hmmm....still not playing nice with transmission:
error-pages log:
{"level":"info","ts":1645645599.2564173,"msg":"HTTP request processed","useragent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.80 Safari/537.36 Edg/98.0.1108.43","method":"GET","url":"/409.html","referer":"https://transmission.domain.com/transmission/web/","status_code":200,"content_type":"text/html; charset=utf-8","connection_close":false,"duration":0.00021726,"request_headers":["Host: transmission.domain.com","Content-Type: text/plain;charset=UTF-8","User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.80 Safari/537.36 Edg/98.0.1108.43","Cookie: _forward_auth=redacted=|redacted|redacted@gmail.com","Accept: */*","Accept-Encoding: gzip, deflate, br","Accept-Language: en-GB,en;q=0.9,en-US;q=0.8","Dnt: 1","Origin: https://transmission.domain.com","Referer: https://transmission.sirfragalot.com/transmission/web/","Sec-Ch-Ua: \" Not A;Brand\";v=\"99\", \"Chromium\";v=\"98\", \"Microsoft Edge\";v=\"98\"","Sec-Ch-Ua-Mobile: ?0","Sec-Ch-Ua-Platform: \"macOS\"","Sec-Fetch-Dest: empty","Sec-Fetch-Mode: cors","Sec-Fetch-Site: same-origin","X-Forwarded-Host: transmission.domain.com","X-Forwarded-Port: 443","X-Forwarded-Proto: https","X-Forwarded-Server: redacted","X-Forwarded-User: redacted@gmail.com","X-Real-Ip: re.da.ct.ed","X-Transmission-Session-Id: null"],"response_headers":["Content-Type: text/html; charset=utf-8","X-Robots-Tag: noindex","X-Transmission-Session-Id: null"]}

Here is an nginx config that is working with transmission:
server {
    listen       80;
    listen  [::]:80;
    server_name  localhost;

    error_page  500 502 503 504  /50x.html;

    location = /50x.html {
        root   /usr/share/nginx/html;
    }

    location / {
        proxy_read_timeout 300;
        proxy_pass_header  X-Transmission-Session-Id;
        proxy_set_header   X-Forwarded-Host $host;
        proxy_set_header   X-Forwarded-Server $host;
        proxy_set_header   X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_pass         http://transmission:9091;
    }
}

Oh, your nginx works as a proxy server, and not as an HTTP service for the error pages only... Why you didn't say it before?
As you can see the header X-Transmission-Session-Id proxied as expected, but I have no idea how to help you, sorry 😞
If (when) you find the solution, please, tell me.

Apologies, I should have been clearer around nginx.
No worries, thank you for looking into it. For now I will just have the 409 error unhandled.
- "traefik.http.middlewares.error-pages-middleware.errors.status=400-408,410-599"
Regards,
Jon


Navigator 2.0 Updates

To support better URL/Deep linking support I'm going to add navigator 2.0 support to dart board
acceptance:
In an app with routes "store", "category" and "item=[value]"
/store/category=food/item=123
Should resolve 3 Pages and be able to work up the route
1.Routes with data (EqualityNamedRoute)
2. BREAKING! Drop prefix "/" from the route names across sample
3. MaterialApp.router (routerDelegate + routerInformationParser)
4. DartBoardPage class to handle navigator 2.0 routing
5. Example loads a deep route and navigates back correctly
6 . Ability to use your own routerDelegate + routerInformation Parser (personalized state)

Integration into Example
Standalone routing example (port of books)

Tracking this in git so I can document status since it's a big upgrade.

the basics are in place, and initialRoute loads correctly

Have a demo working with a new Routing type, PathedRoute that allows you to set up complex pathing routes.
Currently just building out the API necessary, and then integrating it into the main code.
The Navigator 1.0 API and named routes will be deprecated in favor of a new navigator with a similar API, and ideally should also be able to handle custom router delegate/information parsers for people who want to do navigation themselves.

The demo is working with many useful nav features.
I'm still adding a few features, e.g. keyed route for something like store/cakes/21 so that you can pull by ID
also QueryString route e.g. store/cakes/details?id=21
depending on how you like to manage your routes.

also the ability to push routes and get a result.

It's pretty much feature complete as far as nav 2.0 goes in a branch.
Still need to add 1 feature

Anonymous/Runtime routes

As well as refinement

Documentation
Verify / Fix tests
Update SpaceX demo to be "UI" complete
Merge and Deploy demos

Anonymous Routes work.
However return codes are going to get added to the heap as well, and dialogs verified

I'll address results in a follow up PR, everything looks like it's working and it's sundary night/monday morning, time to push/merge and release.


Features: h1 and h2 backgrounds do not fill entire width on IE11

This is probably related so I'll leave it here, the h1 texts can wrap:


Primary refresh token missing in WebView2, unable to authenticate

I have an enterprise app reg in Azure that is used as authentication to another enterprise app reg that exposes an API. I have been using MSAL.PS with Windows PowerShell 5.1 for a long time as a means of reaching this API directly from PowerShell, which works flawlessly (even with SSO). This produces the following WebView, which allows me to one-click sign-in;

If I try the exact same flow in PowerShell Core (any version), the primary refresh token seems to be completely missing inside Edge's WebView2, as it shows the Device State is "unregistered" and I have to manually type in my details.

Because the PRT is somehow missing, typing in your credentials manually is not gonna get you anywhere. Our conditional access policies require an AAD managed device, which blocks me from signing in as the request thinks my device is "unregistered" (hence why I think the PRT is missing). dsregcmd /status shows that my AzureAdPrt is fine.

I'm definately in over my head with this one, any feedback and/or attempts to help is greatly appreciated!

This is a bug in MSAL, where we shouldn't be using WebView2 because it cannot get SSO with Windows and cannot prove that the device is managed by your organization. We were not able to fix this issue, so we had MSAL use to Webview1 when talking to AAD.

@Snaddr - I agree that using another module is the best path forward, but I would suggest that you use - https://learn.microsoft.com/en-us/powershell/microsoftgraph/get-started?view=graph-powershell-1.0 - mainly because it's officially supported by Microsoft and signed by Microsoft.

For anyone running into this, you can also set UseEmbeddedWebView to false, it'll open a browser window instead of a popup.
Unfortunately it does mean you'll have to reauth on every token request.
I'm still on the lookout for a solution for that last part, be it using this module or a different solution.


A lot of erors appears using dependency check in gitlab ci

I'm trying to integrate dependency_check with gitlab ci , when i run the command directly in the vm it works properly , but using the gitlab ci much erors encountered , please any help ?
$ mkdir -p Dependency_check_reports/
$ dependency-check.sh --scan $CI_PROJECT_DIR --project ****** --out Dependency_check_reports/ --failOnCVSS 7 --format JSON --log verbose
[WARN] Unable to update 1 or more Cached Web DataSource, using local data instead. Results may not include recent vulnerabilities.
[ERROR] Unable to continue dependency-check analysis.
[ERROR] One or more fatal errors occurred
[ERROR] Unable to obtain an exclusive lock on the H2 database to perform updates
[ERROR] No documents exist
Uploading artifacts for failed job

You likely should use an API Key from the NVD and take a look at https://jeremylong.github.io/DependencyCheck/data/cacheh2.html

@jeremylong im using this config but im always encountering the same error which is unable to Unable to obtain an exclusive lock on the H2 database to perform updates
$ dependency-check.sh --nvdApiKey $NVD_API_KEY  --scan $CI_PROJECT_DIR/requirements.txt --out Dependency_check_reports/azue_report.json --failOnCVSS 7 --format JSON --data /opt/dependency-check/data --noupdate


python311Packages.llama-index-agent-openai: 0.10.18 -> 0.10.19

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.llama-index-agent-openai is: Data framework for your LLM applications
meta.homepage for python311Packages.llama-index-agent-openai is: https://github.com/run-llama/llama_index/
meta.changelog for python311Packages.llama-index-agent-openai is: https://github.com/run-llama/llama_index/blob/0.10.19/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.10.19 with grep in /nix/store/akv30fbc1x898340jzzaqn85hvymw53l-python3.11-llama-index-agent-openai-0.10.19
found 0.10.19 in filename of file in /nix/store/akv30fbc1x898340jzzaqn85hvymw53l-python3.11-llama-index-agent-openai-0.10.19

Rebuild report (if merged into master) (click to expand)

20 total rebuild path(s)

20 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.clarifai
python311Packages.llama-index
python311Packages.llama-index-agent-openai
python311Packages.llama-index-cli
python311Packages.llama-index-core
python311Packages.llama-index-embeddings-gemini
python311Packages.llama-index-embeddings-google
python311Packages.llama-index-embeddings-openai
python311Packages.llama-index-indices-managed-llama-cloud
python311Packages.llama-index-legacy
python311Packages.llama-index-llms-openai
python311Packages.llama-index-multi-modal-llms-openai
python311Packages.llama-index-program-openai
python311Packages.llama-index-question-gen-openai
python311Packages.llama-index-readers-file
python311Packages.llama-index-readers-json
python311Packages.llama-index-readers-llama-parse
python311Packages.llama-index-readers-weather
python311Packages.llama-index-vector-stores-chroma
python311Packages.llama-parse

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/akv30fbc1x898340jzzaqn85hvymw53l-python3.11-llama-index-agent-openai-0.10.19 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.llama-index-agent-openai https://github.com/r-ryantm/nixpkgs/archive/a1a3092cad143e5ede9700190a0a3f677babee71.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a1a3092cad143e5ede9700190a0a3f677babee71#python311Packages.llama-index-agent-openai

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/akv30fbc1x898340jzzaqn85hvymw53l-python3.11-llama-index-agent-openai-0.10.19
ls -la /nix/store/akv30fbc1x898340jzzaqn85hvymw53l-python3.11-llama-index-agent-openai-0.10.19/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

4 packages failed to build:

python311Packages.llama-index
python311Packages.llama-index-readers-file
python311Packages.llama-index-readers-file.dist
python311Packages.llama-index.dist

36 packages built:

python311Packages.clarifai
python311Packages.clarifai.dist
python311Packages.llama-index-agent-openai
python311Packages.llama-index-agent-openai.dist
python311Packages.llama-index-cli
python311Packages.llama-index-cli.dist
python311Packages.llama-index-core
python311Packages.llama-index-core.dist
python311Packages.llama-index-embeddings-gemini
python311Packages.llama-index-embeddings-gemini.dist
python311Packages.llama-index-embeddings-google
python311Packages.llama-index-embeddings-google.dist
python311Packages.llama-index-embeddings-openai
python311Packages.llama-index-embeddings-openai.dist
python311Packages.llama-index-indices-managed-llama-cloud
python311Packages.llama-index-indices-managed-llama-cloud.dist
python311Packages.llama-index-legacy
python311Packages.llama-index-legacy.dist
python311Packages.llama-index-llms-openai
python311Packages.llama-index-llms-openai.dist
python311Packages.llama-index-multi-modal-llms-openai
python311Packages.llama-index-multi-modal-llms-openai.dist
python311Packages.llama-index-program-openai
python311Packages.llama-index-program-openai.dist
python311Packages.llama-index-question-gen-openai
python311Packages.llama-index-question-gen-openai.dist
python311Packages.llama-index-readers-json
python311Packages.llama-index-readers-json.dist
python311Packages.llama-index-readers-llama-parse
python311Packages.llama-index-readers-llama-parse.dist
python311Packages.llama-index-readers-weather
python311Packages.llama-index-readers-weather.dist
python311Packages.llama-index-vector-stores-chroma
python311Packages.llama-index-vector-stores-chroma.dist
python311Packages.llama-parse
python311Packages.llama-parse.dist

Maintainer pings
cc @fabaff for testing.

duplicate of https://github.com/NixOS/nixpkgs/pull/295493


Added support for http-interop/http-middleware 0.5.0 (No BC Break)

We keep also support for http-interop/http-middleware 0.4.1. We use webimpress/http-middleware-compatibility to keep compatibility with all versions of http-middleware. So customer of the library can use any version of http-middleware and migrate at any time to the latest version.
composer.lock is currently with version 0.4.1 of http-interop middleware, so tests run with:

lowest - 0.1.1
locked - 0.4.1
latest - 0.5.0

Thanks, @webimpress


Trouble logging in

Expected content
Actual content
Link to page:

Hello @clementobi52
We're sorry you're having this issue. You can access support contact options by going to https://support.sendgrid.com and clicking "Trouble Logging in?". From there, you can tell us a little more about your problem without having to log in, and one of our fantastic support people will get back to you. Hope this helps!
Twilio SendGrid Docs Team


sensors unavailable at night

Hi, when micro inverters shut down when sun at dusk, that provide multiple errors in HA's logs.
So, is there a chance to put an option that disable logger when micro inverters going down, and reenable it when they become available ?
It could be part of an automation, but integrated in this component simplify things 😅
Thx
Note that all working very well when sun is shining, big up for your work !! 👍

Note to myself: It's an error in the python-library and only affects the switch.
Error:
Update for switch.apsystems_solar_power_status fails

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 696, in async_update_ha_state
    await self.async_device_update()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 957, in async_device_update
    await self.async_update()
  File "/config/custom_components/apsystemsapi_local/switch.py", line 59, in async_update
    status = await self._api.get_device_power_status()
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/APsystemsEZ1/__init__.py", line 237, in get_device_power_status
    return Status(int((await self._request("getOnOff"))["data"]["status"]))
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: ''


Document is not clear for "step 1" pre-requisite steps

On section https://docs.microsoft.com/en-us/azure/azure-monitor/platform/metrics-store-custom-rest-api#create-and-authorize-a-service-principal-to-emit-metrics you refer us to the "step 1" tutorial part - where is it?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 78411727-ea56-aef5-4f04-b016c25afba9
Version Independent ID: 8687b7bc-a196-6fd0-fa57-9fa5dd2d4545
Content: Send metrics to the Azure Monitor metric database using REST API - Azure Monitor
Content Source: articles/azure-monitor/platform/metrics-store-custom-rest-api.md
Service: azure-monitor
Sub-service: metrics
GitHub Login: @anirudhcavale
Microsoft Alias: ancav

@adibiton Thanks a lot for bringing this to our notice , will review the feedback and fix the document accordingly.

@adibiton Apologies for delay in getting back to you on this, I am further reviewing this with our document author. Will circle back to you at the earliest.

@adibiton Thanks for being patient on this and apologies for the delayed response.
Hope the below information helps and gives more clarity on what you are looking for. Please review and get back to us if you have any further queries.
Step 1 instruction which has been called out in the document is to create a service principal, starting from here  and further save the tenant id and client id as called out in the next section of the doc. Regarding the part on giving the app the right permissions it is also referred in that AAD article in this  section.
We would also have a backlog item to improve the document to further clarify and avoid this confusion regarding step 1.
Hope the above information helps. We will go ahead and close this issue, however if you have any further queries please tag us in your reply and will be happy to reopen the thread and continue the conversation. Thank you


fix(filter): throw error if not used with an array

Throw error if filter is not used with an array.
Closes #9992
BREAKING CHANGE: Previously, the filter was not applied if used with a non array.
Now, it throws an error.

lgtm, will land today

a good followup bug would be to improve the error document a bit


how to run test?

sorry
I am new in Rust
I compile your project using cargo build
and I do get a file named libdevicetree.rlib
but how to run your test?
I tried rustc --test .\tests.rs
and it says :
error[E0433]: failed to resolve: there are too many leading super keywords
--> .\tests.rs:14:5
|
14 | use super::{DeviceTree, DeviceTreeBlob};
|     ^^^^^ there are too many leading super keywords
error: aborting due to previous error
For more information about this error, try rustc --explain E0433.

sorry my bad .
just use cargo  test is OK


Runs cargo fmt

Only merge this if it doesn't conflict with anything, and after landing #5

Superseded by #9


Make serde pickle version configurable

See https://github.com/pinterest/pymemcache/pull/156 for more info :)

My quick hack was reverted. This provides similar functionality while maintaining backwards compatibility with ancient pythons or a mix of py2/py3 and python-memcached.

fin

How's this look everyone? Any thoughts?

Hi @akatrevorjay ,
After internal discussion:

we really appreciate the contribution, and believe that making the cpickle version configurable is a good (great even) change.
we want the change to make it to master, but we would like it to be as atomic and small as possible before we perform the merge.

Another way of saying this is that this is a rather low level library that many people depend on, and few people actually open up and modify. This means that the benefits of a refactor are rather low, whereas the risks of something breaking because of it would impact a large number of actors in a bad way.
Such a refactor would require us to probably bump a major version for the library, which doesn't make sense given the feature set change.
All of this being said, we all agree that your code is good and the new interface and naming make sense.
Course of action:
We would like to open a new PR with as little changes as possible allowing for the configurable CPickle version. We can then bump that as a minor patch.
Since you've already done a lot of the work here, we're happy to make the PR changes on our side, but if you would like to do them on yours, let us know.
Cheers!

@nichochar How's that? Addressed all concerns, I believe?

The failure is due to #178 , ie nothing to worry about

Updated to latest master so it can pass tests (my other PR fixed that)

Hello again, thanks for bumping! We definitely appreciate the work.
Since there is a lot of changes, and I'm no longer making big decisions related to this project, I will defer the review decision to @jogo .
In of itself, the code looks fine, the arguments against it I believe are that the project is very mature and stable, this isn't fixing any bugs but refactoring quite a bit, and losing blame history with a file rewrite. The benefits need to be measured well, and I'll defer to @jogo

Closing this in favor of the approach implemented in #190.


Do not return internal enum values

This PR updates the enum validation protocol tests to exclude returning enum values that have been marked as internal with the @internal trait or tagged as internal.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

There was some concern on using internal tag as well in this (along with @internal trait), but we side towards using internal tags for this purpose too, as tags are also commonly used to mark shapes for internal use, and we'd rather not have a surprise for such users. If one wanted to use the internal tag for another purpose, they can use a different label.


Add a license

MIT?

Easy to do. What is the name/organization that should be put on the license?

Organization is Orcasound, see
https://github.com/orcasound/orca-action-workflow/blob/main/LICENSE

A Github trick I learned this summer: "Add file->Create new file" from top of repo, then type "license" as new filename and an "Choose a license template" button appears in upper right of editor.

@scottveirs this has been merged, see https://github.com/orcasound/aifororcas-livesystem/pull/27. Closing.


metrics.rollingPercentile.enabled do not disable percentile calculation when set to false

The configuration property metrics.rollingPercentile.enabled do not disable anything when set to false. This seems to be changed since 1.5.x version.

Anything new about that?

Could anyone from maintainers take a look at this, and confirm whether it is known bug, or we are missing something?

I have prepared a fix for this. Could please someone take a look: https://github.com/Netflix/Hystrix/pull/1840

+1 Anything new about that?

Hi! Sorry to 'revive' such an old thread, but we are encountering issues realated to this... Not only is a quite big chunk of ram (75 MB aprox) going into latency metrics (even trying to get them disabled as per this issue), but this chunk is not being garbage collected on application (but not server) restarts...
I see the PR was not merged in the end... @NZvereff did you manage to apply it successfully and ketp it in use in your case? Thanks!


Form Validation

What happened?
the user registration form in the project accepts numerical in place of full name whereas the expected input is always in characters.
Add screenshots

What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

@Sanchitbajaj02 please assign this issue to me.

Yeah, I didn't noticed this until now. Please take up and fix this issue.

@Sanjana787 Any updates on your work?

@Sanjana787 I am unassigning you this issue, you made no progress to it.

I am sorry for the delay @Manishak798, @Sanchitbajaj02  and @thevirengarg.
if possible please reassign this issue to me.

@Sanjana787 I am only assigning again but I need a PR in 24 hours or else I will assign you

@Sanchitbajaj02 I would like to work on this issue

@Sanchitbajaj02 I would like to work on this issue

Assigned to you

Currently, Password validation is considering only two characters as special characters, reducing the capability of forming a strong password.
@Sanchitbajaj02 Should I change it to accept other special characters as well?

But how would the user get to know that they can only use those 2 special characters ?
I was trying to register but couldn't register because I was using other special characters in my password. and it was showing the same alert that "Oops!, Password must contain a character, a number and a special character", even though I was using correct combination as per the alert message. I think it would be better to take other special characters as well. What do you say?

only use those 2 special characters?

We can provide a small text under the password field.

@monoMonu update the pr for merge. It is reviewed but there are merge conflicts that needs to be resolved


Make sure the SA1130 exception also works on other invocable expressions

This was found using the Tester on the monodevelop source.

@sharwell I updated the PR


Scroll to top on each website pages

I have added a scroll to top on each page of my website.
So, each time I click on a link, I'm on the top of the new page.
It works well.
But, when I use the back button, I can see an anormal behaviour.
In fact, before to be redirected to the last view, I can see the current view scrolling to the top and, AFTER, I'm redirecting to the last view.
=> I would like to be redirected on the last view AND apply the scroll...
I'm using ui-router 1.0.0-beta.3 and Angular-js 1.
To do this, I have implemented this code:
$transitions.onSuccess({}, function (transition) {         $anchorScroll();       });
Do you have some suggestions ?

Try putting that anchor scroll into a timeout.
If that doesn't work, try using the standard DOM code:
    document.body.scrollTop = document.documentElement.scrollTop = 0

Thanks for your answer.
But this is the same behavior..
On back button, I can see the current view scrolling top before the change of state

It looks like a native scroll (browser native behaviour) is made before the onSuccess callback execution.
How to prevent or delay it ?

Browser native behavior? Can you show me in a plunker?

Yes: http://plnkr.co/edit/RGu8vfo1JgDVwPDHEz5d?p=preview
TO reproduce the behavior, you have to
=> click on home.foo.detail
=> scroll bottom of the page
=> Click back button
==> You will see the detail view scrolling to top before the onSuccess callback execution.

fascinating.  I don't know of any way to disable the native behavior.  There's a horrible workaround I found on stack overflow:
http://stackoverflow.com/questions/6407178/prevent-browser-from-snapping-to-previous-scroll-position-on-back-button
https://plnkr.co/edit/Y8VTEhOzzDFR7CYsrQpT?p=preview
Unfortunately, I think this is outside the scope of ui-router.  I hope you find some solution!


Injecting an object returned by a TyphoonTypeConverter

I have a plist as follows defining a mapping to a custom object:
<dict>
<key>myObject</key>
<string>MyObject()</string>
<key>myObject.myURL</key>
<string>NSURL(http://myurl.com )</string>
</dict>
It seems that on implementing the TyphoonTypeConverter protocol, only a concrete object can be returned from the convert method rather than another TyphoonDefinition. Is there any way to inject the custom object returned from the TyphoonTypeConverter with another property defined in the plist?
e.g. the MyObject returned by the TyphoonTypeConverter implementation would have a property myURL set to the value defined in the plist?

Yes only a custom object. When you create definitions in a TyphoonAssembly subclass to thing happen:

It is instrumented so that each definition is allocated a key. (To support code completion and refactoring without any special tool support).
Upon activation the definition is registered with a TyphoonComponentFactory.

So if you wanted a type converter to return a TyphoonDefinition those steps would need to happen too.

Thanks for the info. If I drop the plist mapping and configure in the assembly e.g.

(ViewController *)viewController {
return [TyphoonDefinition withClass:[DataManager class] configuration:^(TyphoonDefinition *definition) {
[definition injectProperty:@selector(dataManager) with:[self dataManager]];
}];
}

Is there a way to inject properties into the dataManager which is already being injected into the view controller?

Yes of course. The definition for the data manager can have initializer, property or method injections. And the injections can be any of these - references to other definitions, scalar values (int, BOOL, etc), inline objects or something resolved from TyphoonConfig.

Let's continue this chat on StackOverflow! http://stackoverflow.com/questions/tagged/typhoon


Implement async generators.

Implement async generators.
Handle throw in for-on bodies.
Allow a yield expression after await.

Please review (and contribute more tests if you like :-)).

I couldn't find the Observable implementation in here? Which async generator proposal are you implementing?

I am implementing https://github.com/jhusain/asyncgenerator. As far as I understand, the core of the proposal is a new protocol (the observable and the observer interface) and two syntactic additions, async generators and for-on loops that conform to the protocol. Both have been implemented.
According to https://github.com/jhusain/asyncgenerator#observable-composition, the various functions for observables are not proposed for standardization but left to library writers. Thus they are not included in this PR.
Any observable can be easily extended by adding methods in the prototype chain of the prototype property of any async generator function.

I think I understand, thanks.

Only a few nits.

Adressed the issues and also removed an unnecessary try-catch in the generated code.

LGTM. A few nits

Thanks for reviewing! Pushed the corrections.

I noticed that our tests are broken so I'm holding off this one until that has been resolved.

No problem. I'll check again tomorrow (it's now 10 pm here).
Once this is accepted, we can start adding documentation and maybe a demo.

Committed as c8815199e74551f064ff5bb5b5fa122df8e5cbf3


Adding v1.1 of RY-Storm USD example assets

cc @RY-Storm as FYI
Thanks for bringing this one home Andy!


Abstract PvlSource to MerkleStore

Abstracts PvlSource into a generic MerkleStore where the client can fetch some data from the server and verify the hash is what we expect in the merkle root. This paves the way for parameterized proofs since we store the hash known configurations in the merkle root. The original code is unchanged besides naming except for:

bug where we were not respecting the dbVersion variable correctly
removed async db write (we don't do this elsewhere, i don't think it's a performance concern)
dbKey is abstracted to be different for each store based on the tag the store is initialized with

going to apply the feedback changes to the next PR https://github.com/keybase/client/pull/13791


[Bug] abscissa type not supported, from sqlancer test

Cloudberry Database version
No response
What happened
Request to confirm if it is as designed?
BTW, "abscissa type" is vague,  if the statement not valid,  I believe customer does not know how to fix it according to "abscissa type" error.
UPDATE t0 SET c0 = DEFAULT WHERE (pg_catalog.float8div('2.2946442861439234E-33'::float8, pg_catalog.interval_interval_div(('15:01:06'::time) -(pg_catalog.linear_interpolate(('true') ||('false'), pg_catalog.cash_words(CAST(1.4549639 AS money)), ('209820 day 11 hour 25 min 41 sec'::interval) +('09:05'::time), ('false') ||('true'), ('483158 day 5 hour 20 min 21 sec'::interval) +('00:56:56'::time))), '909172 day 16 hour 39 min 49 sec'::interval))) <(pg_catalog.float8(((-8521399760881100626::numeric) %(1011845922400518285::numeric)) /(pg_catalog.trunc((-5182015971170804306::numeric) ^(4186530105024981936::numeric), 465134229))));
ERROR:  abscissa type not supported
What you think should happen instead
No response
How to reproduce
CREATE TABLE t0(c0 date );
UPDATE t0 SET c0 = DEFAULT WHERE (pg_catalog.float8div('2.2946442861439234E-33'::float8, pg_catalog.interval_interval_div(('15:01:06'::time) -(pg_catalog.linear_interpolate(('true') ||('false'), pg_catalog.cash_words(CAST(1.4549639 AS money)), ('209820 day 11 hour 25 min 41 sec'::interval) +('09:05'::time), ('false') ||('true'), ('483158 day 5 hour 20 min 21 sec'::interval) +('00:56:56'::time))), '909172 day 16 hour 39 min 49 sec'::interval))) <(pg_catalog.float8(((-8521399760881100626::numeric) %(1011845922400518285::numeric)) /(pg_catalog.trunc((-5182015971170804306::numeric) ^(4186530105024981936::numeric), 465134229))));
Operating System
centos7
Anything else
No response
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

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

@gfphoenix78 Hi Hao,   this is the issue we talked before. I get a simple recreate today.  Thus I 'd like to use this issue for tracking this discussion.

UPDATE t0 SET c0 = DEFAULT WHERE
(pg_catalog.float8div
('2.2946442861439234E-33'::float8, pg_catalog.interval_interval_div(
(pg_catalog.linear_interpolate(('true'), pg_catalog.cash_words(CAST(1.4549639 AS money)),
('09:05'::time), ('true'),
('00:56:56'::time))), '909172 day 16 hour 39 min 49 sec'::interval)))
<0;

It was an unsupported type in linterp_abscissa(). The error is expected.
I'll close this issue. Nothing needs to do.


integration: calculate required decimal precision from parameters

ceil(log10(1 / dx))

@klaeufer
Although I agree with this thinking, I think in this context you are talking about the printed result. So can there be an option to indicate either: (a) the format string for the result (for use with fmt::) or (b) --precision N (places after decimal point).  This gives users the most control.
Of course, your formula can amount to a sensible default in the spirit of convention over configuration.


Can protectedResourceMap of MSALInterceptor be updated after application start up?

Core Library
MSAL.js v1 (@azure/msal or msal)
Core Library Version
^1.1.1
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
None
Description
I have a scenario where some of the endpoints will come from a protected backend api. So from client, I should be calling the backend api which is protected by making use of MSALInterceptor and have to update protectedResourceMap with these new endpoints and scopes. Is this supported?
MSAL Configuration
No response
Relevant Code Snippets
No response
Identity Provider
No response
Source
Internal (Microsoft)

@pradeepKaatnam While we have not provided specific methods in msal-angular to directly update the protectedResourceMap, this shouldn't be a problem. The easiest way is to provide the configuration options via factory providers, and write your own function to update the protectedResourceMap. The docs on configuring via factory providers can be found here.


Heroic protocol not working

Describe the bug
So basically when I try launching games from the Heroic launcher nothing happens. When I checked the .desktop file, Exec set to: xdg-open heroic://launch/legendary/0a2d9f6403244d12969e11da6713137b.
I tried running the command in the console I got:
[tatsuuya@nixos:~]$ xdg-open heroic://launch/legendary/0a2d9f6403244d12969e11da6713137b
gio: heroic://launch/legendary/0a2d9f6403244d12969e11da6713137b: The specified location is not supported

Running xdg-open heroic gives:
[tatsuuya@nixos:~]$ xdg-open heroic
gio: file:///home/tatsuuya/heroic: Error when getting information for file “/home/tatsuuya/heroic”: No such file or directory

Steps To Reproduce
Steps to reproduce the behavior:

Serch for the game
Click on it
Nothing happens

Expected behavior
The game should launch normally.
Additional context
[tatsuuya@nixos:~]$ neofetch
          ▗▄▄▄       ▗▄▄▄▄    ▄▄▄▖
          ▜███▙       ▜███▙  ▟███▛
           ▜███▙       ▜███▙▟███▛             tatsuuya@nixos
            ▜███▙       ▜██████▛              --------------
     ▟█████████████████▙ ▜████▛     ▟▙        OS: NixOS 23.11.1209.933d7dc15509 (Tapir) x86_64
    ▟███████████████████▙ ▜███▙    ▟██▙       Host: ASUSTeK COMPUTER INC. GU502LU
           ▄▄▄▄▖           ▜███▙  ▟███▛       Kernel: 6.1.64
          ▟███▛             ▜██▛ ▟███▛        Uptime: 2 hours, 6 mins
         ▟███▛               ▜▛ ▟███▛         Packages: 1502 (nix-system), 1854 (nix-user), 185 (nix-default)
▟███████████▛                  ▟██████████▙   Shell: bash 5.2.15
▜██████████▛                  ▟███████████▛   Resolution: 1920x1080
      ▟███▛ ▟▙               ▟███▛            DE: GNOME 45.1 (Wayland)
     ▟███▛ ▟██▙             ▟███▛             WM: Mutter
    ▟███▛  ▜███▙           ▝▀▀▀▀              WM Theme: Adwaita
    ▜██▛    ▜███▙ ▜██████████████████▛        Theme: Adwaita [GTK2/3]
     ▜▛     ▟████▙ ▜████████████████▛         Icons: Adwaita [GTK2/3]
           ▟██████▙       ▜███▙               Terminal: kgx
          ▟███▛▜███▙       ▜███▙              CPU: Intel i7-10750H (12) @ 5.000GHz
         ▟███▛  ▜███▙       ▜███▙             GPU: NVIDIA GeForce GTX 1660 Ti Mobile
         ▝▀▀▀    ▀▀▀▀▘       ▀▀▀▘             GPU: Intel CometLake-H GT2 [UHD Graphics]
                                              Memory: 5636MiB / 15789MiB

Notify maintainers
@aidalgol
Metadata
[tatsuuya@nixos:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.64, NixOS, 23.11 (Tapir), 23.11.1209.933d7dc15509`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos-23.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I think there are two parts to this problem.
First, Heroic is using the Electron protocol API to register heroic:// as a protocol for xdg-open to find, and that is failing for some reason.
WARNING: [Backend]:          Failed to register protocol with OS.

Second, Heroic is run inside an FHS environment wrapper (see buildFHSEnv in the Nixpkgs manual), which may be why Electron fails to register the protocol.
I just tried creating a desktop shortcut from Steam to see what that looked like for comparison, and nothing happened, not even a log message, although Steam uses Chromium Embedded Framework, not Electron, so that may not have been of much use anyway.
I might need help from someone more familiar with Electron and buildFHSEnv to figure out exactly why this operation is failing.

I think this is resolved. Not sure when or why, but for me it works correctly now.