Profile edit fix

Users should be able to change their alias without being required to enter something for the date of birth field as well.

@saeedmb8 @miltontony @peterp Ready for review

:+1 :cake: :cake:  :cake: :cake:  :cake: :cake:


GRPC error with ubuntu 12.04

This is my system architecture

This is bug I am facing

Thanks Manohar M R
Originally posted by @manuurdevang in https://github.com/grpc/grpc/issues/13908#issuecomment-425002901

@manuurdevang the strange thing that pops out here is that you seem to be using the one of the pre-compiled x86_64 grpc packages on what looks like a 32 bit architecture.
I'm wondering how the x86_64 grpc package was downloaded here in the first place. Normally, the package manager should be fetch the 32-bit pre-compiled grpc package when it recognizes that it's installing for a 32-bit platform.
One more thing I'm wondering, can you please run:
$ irb
irb> RUBY_PLATFORM

and share the output here? thanks


Run app on .NET and .NET Core using VS

Need a doc which tells a user to switch between .NET and .NET Core for VS

We should wait to do this until we know how this is going to look after we move to the .NET CLI.

This should be handled as part of the .NET Core docs.


🚀[Feature]: Class for IPv4 addresses

Description
Create a class for an IPv4 address

Might be worth a look:
https://www.itprotoday.com/powershell/working-with-ipv4-addresses-in-powershell


Drop duplicated tests

Closes #92.
It seems I accidentally added the TestTransfer unit tests twice in #105 - this PR drops the copies.
I also found that #92 is fixed locally following a fresh Firedrake install so the temporary hack is no longer needed :)

Sorry for missing the duplicate test!

No worries


[test-triage] OTBN timeouts with multiple tests

Hierarchy of regression failure
Block level
Failure Description

otbn_rf_base_intg_err with seed:<PHONE_NUMBER>24054756254243263758896087906517619155141568323899745862435846495702
UVM_FATAL @ 931274490 ps: (otbn_rf_base_intg_err_vseq.sv:32) uvm_test_top.env.virtual_sequencer [uvm_test_top.env.virtual_sequencer.otbn_rf_base_intg_err_vseq] Timeout while waiting for register file A to be used

otbn_ctrl_redun with seed:<PHONE_NUMBER>2524754647320219056712357229922074576056242959858639090007540897643
UVM_FATAL @  69456697 ps: (otbn_ctrl_redun_vseq.sv:31) uvm_test_top.env.virtual_sequencer [uvm_test_top.env.virtual_sequencer.otbn_ctrl_redun_vseq] Never found a time to inject an error.

Steps to Reproduce

GitHub Revision: a182fcef27866fb502d8cceb810bdf76cfa3ef42
dvsim invocation command to reproduce the failure, inclusive of build and run seeds:

util/dvsim/dvsim.py hw/ip/otbn/dv/uvm/otbn_sim_cfg.hjson -i otbn_rf_base_intg_err -t xcelium --fixed-seed<PHONE_NUMBER>24054756254243263758896087906517619155141568323899745862435846495702

util/dvsim/dvsim.py hw/ip/otbn/dv/uvm/otbn_sim_cfg.hjson -i otbn_ctrl_redun -t xcelium --fixed-seed<PHONE_NUMBER>2524754647320219056712357229922074576056242959858639090007540897643

Tests with similar or related failures
It might be related to the same root cause due to recent changes:

[ ] otbn_rf_base_intg_err
[ ] otbn_ctrl_redun

otbn_rf_base_intg_err - back to 100%
otbn_ctrl_redun - 91.7%

I suspect that these are different issues.
Firstly, let's consider the otbn_rf_base_intg_err message. The test tries to inject an error by one or two bit error into a register read. To do this, it waits until a cycle when an instruction is trying to read from the given side of the register file, and then uses the force statement to bodge the value.
The message is saying that we've waited ages and haven't actually seen any instructions coming through that read from the register file. Here, "ages" is defined as a time in clock cycles (currently 20,000). My guess is that we're actually blocked, waiting for a seed from the EDN, so aren't running any instructions at all in that time period.
There's a reasonably obvious solution: tweak the vseq so that it times out when the operation finishes. If the device completely hangs then the test will (eventually) time out from a phase timeout. I've just pushed #23582 which should implement this.

For the second test, it looks like we sometimes fail to find a "good time" and a re-run can perturb things so that we do. A local run doing so gets the failure rate from roughly 4/50 to 1/50 (and failing with a different behaviour), so I think this triage issue should be solved by that change: #23583.

Removing the M4 milestone association. I think this will be fixed by the two PRs mentioned above, and am also certain that the issue is unrelated to the M4 exit criteria.

Thanks Rupert!

The first vseq passed at 100% over the last 9 nights. The second test passes at a reasonable rate and the sporadic failures that I see when running locally aren't the same sort of timeout as described above.
Closing this issue because I think the problem that it describes is fixed.


APIResponseError: query failed validation

Hello! I have (I think) really read through the instructions and followed the documentation but am finding an error when I try to sync to my Notion Database. I am getting an error that states APIResponseError: query failed validation: query.v should be not present, instead was '"b722......"'. I have checked I set up the integration correctly with the appropriate permissions, used the 32 id link and the token code as stated, and deleted any possible other notion tabs that may be connected to zotero. Both zotero and notion are up to date. Any thoughts on how to fix this error? It only shows up with I click "sync to notion" on any folder in my zotero workspace.

Hi @sepowers94! This error shows up when you have some extra characters in the Database ID field of the Notero preferences. It's saying there's a v character in the field, and you'll need to grab a smaller piece of the link as specified in step 5 of the Configure Notion section of the readme.
For example, when I copy the link to my database, I get this full link:
https://www.notion.so/dvanoni/5ba9956716ac4218be77d2b4655911f5?v=7e74bd8988cd42308211329ab491545f

The Database ID is just the part in between /dvanoni/ and ?v=:
https://www.notion.so/dvanoni/5ba9956716ac4218be77d2b4655911f5?v=7e74bd8988cd42308211329ab491545f
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

So, the value that I'll want to enter in the Notero preferences is:
5ba9956716ac4218be77d2b4655911f5

Let me know if this helps!

Hello!
First, thank you so much for the quick response! Secondly, that makes total sense! I just did as you requested and it worked seamlessly! I appreciate it.
Another question I had is when I close out of Zotero and Notion, I have to re-add the plug-in in order to be able to sync to notion.
To explain, say I was opening both applications and then wanted to make some updates and resync some additional papers that I have added to a folder in Zotero. When I right-click, there is no longer the option to sync to notion, and when I go to Zotero--Tools, "Notero Preferences..." no longer shows up. It does appear in the Add-Ons and it is not disabled. What I have to do in order to sync, is click in the Add-Ons, Install Add-on from File... and re-install the notero-0.4.6 in order to have notero preferences, and the ability to click on a Zotero folder and have "Sync Items to Notion" appear. Is there some setting I have not configured correctly?
Thanks,
Shannon

Hi Shannon, I'm glad you were able to get the syncing to work!
It is strange that the Notero plugin stops working when you restart Zotero. My best guess right now is that there may be some sort of conflict between Notero and either Zotero itself or another one of the plugins.
Would you be able to provide me with the following extra information to help me debug?

The Zotero version
The names of any other plugins you have installed
A screenshot of the error console (which can be opened via Tools > Developer > Error Console)

You may need to scroll down through the logs a bit to try to find any mention of "Notero"

Thanks!

Hello!
Of course!
Zotero Version: 6.0.23
The current plugins installed:

The Error Console:
<img width="650" alt="Screen Shot 2023-04-07 at 9 35 53 AM" src="https://user-images.githubusercontent.com/67433730/230636423-ac83bd16-1015-49c1-9115-4

c1bc1622b01.png">
Please let me know if you need anything else!
Shannon

Thanks so much for sending that info! I'm not seeing anything out of the ordinary yet...
Could you take another screenshot of the error console for me, but this time can you capture it just after restarting Zotero and before you reinstall Notero? I want to see if there's anything weird happening with Notero when Zotero starts.

That actually is the image after I restart them (here again), since sync to notero doesn't pop up, I can't get an error message.
I just restarted it and opened both, going direct to the error console!

Ah interesting. Okay, I have a hunch as to what's going on.
I misread your original message where you said "close" Zotero rather than "quit"—which are two different things on macOS. Can you confirm that you were closing the Zotero window rather than quitting the entire Zotero application? If so, I think you may be running into the issue described in #90. Does that sound accurate?

Ah--great point, I am "closing" the application by pressing the red x on my Mac. I see from the discussion that seems to be a common issue, I can you keep it open and minimize it in the background!

Sounds good! Are you good with closing this issue?

Yes! Thank you so much!


feat(redux): refactor guest orders

Description

This reviews the guest orders redux storage

Dependencies

None.
Checklist

[x] The commit message follows our guidelines
[x] Tests for the respective changes have been added
[x] The code is commented, particularly in hard-to-understand areas
[x] The labels and/or milestones were added

Disclaimer
By sending us your contributions, you are agreeing that your contribution is made subject to the terms of our Contributor Ownership Statement

Do not forget to add the BREAKING CHANGE label and the BREAKING CHANGE footer in the commit and in the PR description for all modules that were removed/changed.


A gap appears at the top published in obsidian publish.

There were no bugs until Minimal 3.3.3.
Now the latest version, 5.1.8, shows gaps.
Firefox and Safari on Mac and Safari on iPhone also show the gap.
After inspecting the CSS, the gap no longer appears after removing the following statement
body:not(.hider-frameless):not(.is-fullscreen):not(.is-mobile) {
--titlebar-height: 28px;
padding-top: var(--titlebar-height)!important
}
Is this a bug? I don't see any practical problem, but it bothers me, so if it is possible to fix it, I would like to have it addressed.
Translated with www.DeepL.com/Translator (free version)

Minimal does not support Publish yet and is not licensed for that usage. A version of Minimal for Publish will be available soon


Maybe Fulfill Order Password Field Should be Viewable/Require Confirmation

This is more of a tweak, but we require the merchant to enter a password that is obscured and not confirmed/validated. They could easily type a bad password and not even know if it was typed properly. This could just be a nuisance. I'd suggest we either show the password or have an option to view it to confirm.

Ironically we actually show it clear text on the next page.
/facepalm

@morebrownies Should we just show it in the clear on the form?

Ironically we actually show it clear text on the next page.
/facepalm

It's also in the contract and viewable by the moderator. /facepalm

Ironically we actually show it clear text on the next page.

ah, let's change the input to plain text for now.  We can circle back and improve on this if people report it being an issue.


[Bug report]: FF AppImage instantly crashes on Linux (Pop!OS 22.04)

Platform
Desktop
Version
1.6.2, 1.6.5
Description
Maybe dublicate of: https://github.com/iotaledger/firefly/issues/3783
Today I wanted to open Firefly again (last time I used Pop!OS 21.10) and out of no reason it does not start anymore. An empty window pops up for under a second and then disappears.
What I tryed:
I have deleted the $home/.config/Firefly folder, but that does not help. This occurs on 1.6.2 and 1.6.5
$home/.config/Firefly/logs/main.log [1.6.5]
[2022-08-17 12:55:12.271] [info] Checking for update
[2022-08-17 12:55:12.353] [info] Generated new staging user ID: xxxxxxx
[2022-08-17 12:55:12.460] [info] Update for version 1.6.5 is not available (latest version: 1.6.5, downgrade is disallowed).

$home/.config/Firefly/logs/main.log [1.6.2]
[2022-08-17 12:59:17.071] [info] Checking for update
[2022-08-17 12:59:17.155] [info] Generated new staging user ID: xxxxxx
[2022-08-17 12:59:17.243] [info] Found version 1.6.5 (url: firefly-desktop-1.6.5.AppImage)

I got the hint to disable sandbox by using the argument "--no-sandbox", which does fix my issue. So when I start FF through console with this argument, FF opens as expected. As I use the usual behaviour by double clicking the AppImage file, the explained issue comes up.
Expected behaviour
Double clicking the AppImage file and Firefly opening normal.
Actual behaviour
Firefly opens an empty window for under a second and crashes.
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
Starting the AppImage file.
Operating System
Linux
Diagnostics
Version 1.6.5
Language: en
Currency: USD
Node selection: Automatic
Platform: linux
Platform Version: 5.18.10-76051810-generic
Platform Architecture: x64
CPU Count: 8
Total Memory: 64217.3 MB
Free Memory: 47610.7 MB

Developer Profile
No
Profile Type
Software (Stronghold)
Staking
Previously staked
Crash Status
Application closed
Error Messages
None
Error Logs
empty

Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

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

I have the same issue.
Here is the terminal output when trying to run the appimage:

15:34:51.638 › Checking for update
15:34:51.873 › Update for version 1.6.5 is not available (latest version: 1.6.5, downgrade is disallowed).
[6071:0818/153451.897490:FATAL:gpu_data_manager_impl_private.cc(415)] GPU process isn't usable. Goodbye.
[appimage-binfmt-bypass/bin] ERROR: child exited with code 5

System:

openSUSE Tumbleweed
Kernel: 5.19.1
DE: GNOME 42.4
GPU: NVIDIA GeForce RTX 2080 Ti
CPU: Intel i7-10700K

I confirm that running firefly with --no-sandbox parameter fixes the problem

I have the same problem running AppImage Firefly on Ubuntu 22.04:
[2327:1001/124555.689920:FATAL:gpu_data_manager_impl_private.cc(415)] GPU process isn't usable. Goodbye.
Trace/breakpoint trap (core dumped)
Seems like this is an issue with Electron:
https://github.com/electron/electron/pull/31091

Update: Version 1.7.0 has this issue, as well.

Hi, is this still an issue in the latest version?

Yes, this is still an issue on Ubuntu 22.04 with version 1.7.5 of firefly. The shimmer version (2.1.5) is running without problems.
$ ./firefly-desktop-1.7.5.AppImage
16:46:40.161 › Checking for update
16:46:40.387 › Update for version 1.7.5 is not available (latest version: 1.7.5, downgrade is disallowed).
[5687:0920/164640.886631:FATAL:gpu_data_manager_impl_private.cc(415)] GPU process isn't usable. Goodbye.
Trace/breakpoint trap (core dumped)

Yes, this is still an issue on Ubuntu 22.04 with version 1.7.5 of firefly. The shimmer version (2.1.5) is running without problems.
$ ./firefly-desktop-1.7.5.AppImage
16:46:40.161 › Checking for update
16:46:40.387 › Update for version 1.7.5 is not available (latest version: 1.7.5, downgrade is disallowed).
[5687:0920/164640.886631:FATAL:gpu_data_manager_impl_private.cc(415)] GPU process isn't usable. Goodbye.
Trace/breakpoint trap (core dumped)

I use Ubuntu 22.04 and I cant reproduce the issue, the app works well for me. Searching for the issue you have it seems like you can try running Firefly with the --no-sandbox flag, it looks like is a common Electron issue, could you please try it out? In any case, we will release very soon a new version which has code parity with Firefly Shimmer so i would guess the issue will be gone very soon

If I use the --no-sandbox parameter, Firefly starts as expected, so this is working as a workaround. I'am looking forward for the merge of both Firefly versions. :-)

If I use the --no-sandbox parameter, Firefly starts as expected, so this is working as a workaround. I'am looking forward for the merge of both Firefly versions. :-)

Unfortunately we will not be able to merge both apps anytime soon because, even though iota will be upgraded to stardust, we then need to start working on iota 2.0 for shimmer 🙈 so we have to keep both apps separate for now.
I'll close the issue, thanks for replying and testing 🌷


Maps threshold bug

Hi,
I believe there's a bug in the maps threshold.
As you can see in the following screenshot.

In the t_stats , we have some negative zones while in Raw z map, they are positive.
Best,

Can you send me the code you used ? I can't reproduce the issue.

OK, the problem comes from the fact that you obtained the p-values from a ttest_onesamp of scipy.stats that is a two-sided tests, and implicitly takes the absolute value.
You should use nistats functions, not ttest_onesamp, in that context (copy-pasting from old notebooks has some drawbacks !).
Best,


introduce dataset level setting to hide votes during annotation

Some annotation tasks might benefit from hiding other annotator's tags from users with owner/curator roles while annotating.
This should be introduced as a setting for the overall dataset (or potentially task if implemented after those models are separated).


Packages

Tensorflow
Flax
Tensorflow-addons
Tensorflow-text
tensorflow-graphics
openexr

@dusty-nv i have an error with llvm.
I don’t know why.
with this configuration I can compile outside docker

@dusty-nv it is ready to merge

@dusty-nv you can merge if you want. All is working

tensorflow whls: https://drive.google.com/drive/folders/1hmlRUHO8qNhObpTozDB4815COc2PcbZB?usp=drive_link


[1.12] Bump Mesos to nightly 1.7.x 414805d

High-level description
This is a routine bump to the latest Mesos and mesos-modules.
Related JIRA Issues
Checklist for all PRs

[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[x] Read the DC/OS contributing guidelines
[x] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Changelog:
https://github.com/apache/mesos/compare/9c68e2a532352a89f02e9352714a4f42ab76e82a...414805d337d1ac1996065995d860c2ec5ad50887

[ ] Test Results: [link to CI job test results for component]

[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot bump-ee

Closed in lieu of https://github.com/dcos/dcos/pull/3894


How to use pretrained weights for inference on custom datasets?

All the scripts mentioned are training the NeRF on a dataset and evaluating them.
Is it possible to run the pretrained model on any set of images captured by a camera?
Or is the pipeline such that the end-end training has to happen for every set of camera images?

I understood your question as to the way of importing trained camera paths before the training stage.
Well, you can touch the parameter initialization part if you want to import your custom camera paths. But the current code only supports importing from the COLMAP camera poses.
Did I get the right point?

Please reopen the issue if the response was not sufficient.


yifysubtitles subtitles by dexter21767

Addon Manifest URL
https://2ecbbd610840-yifysubtitles.baby-beamup.club/manifest.json
Addon Description
Addon for getting subtitles from yifysubtitles, by Dexter21767.

not working for series


Fixes #16 Tests with decorators do not appear in the test explorer view

Fixes #16 Tests with decorators do not appear in the test explorer view
Enhances the code path that extracts test names using analysis to also use the current AST.

Needs an additional test that ensures that the scenario this is trying to fix will not regress.

When merged we should remember to update https://developercommunity.visualstudio.com/content/problem/92871/tests-with-decorators-do-not-appear-in-the-test-wi.html

@huguesv Minor updates that should address your request.

👍


C#: Run source generators tests in CI

Follow-up to https://github.com/godotengine/godot/pull/82955

Runs the test project in CI so that changes to the C# source generators are tested. Not sure which job this should be added to but since it requires building the .NET assemblies, I chose the w/ Mono job.

Tests were broken by #87890, because we now generate [ScriptPath(...)] for all generic types.

@akien-mga

These warnings should be addressed by https://github.com/godotengine/godot/pull/80343.

Thanks!


:bug: fix limiter node behaviour

Closes #247

@rxlecky I have adjusted the PR - it was previously not clear when exactly a TimeLimiter and Limiter would work. I clarified in the docs that it only works with children that return RUNNING and in other cases it resets.


Negative CRF loss

Hello,
I'm trying to use the crf loss in my code, but when I run the training I receive a negative loss value.
Here's how I defined the loss:
crf_criterion = DenseCRFLoss(weight=1e-8, sigma_rgb=15, sigma_xy=100, scale_factor=0.5)

croppings = Variable(torch.ones(2, ws, ws).float())
crf_loss = crf_criterion(255 * x.cpu(), out.cpu(), croppings.cpu())

"x" is a [batch, 3, height, width] image
"out" is the softmax segmentation [batch, 2, height, width]
"croppings" is the binary mask
I receive a negative loss value between -0.6 and -0.5.
How can I deal with it? Is it normal to have a negative crf loss?
Thank you

Hi,
I've got same problem. Is there any specific condition to use CRF pytorch version or bilateral filter? @meng-tang

I meet the same problem, have you solved it ?

Hello,
I'm trying to use the crf loss in my code, but when I run the training I receive a negative loss value.
Here's how I defined the loss:
crf_criterion = DenseCRFLoss(weight=1e-8, sigma_rgb=15, sigma_xy=100, scale_factor=0.5)

croppings = Variable(torch.ones(2, ws, ws).float())
crf_loss = crf_criterion(255 * x.cpu(), out.cpu(), croppings.cpu())

"x" is a [batch, 3, height, width] image
"out" is the softmax segmentation [batch, 2, height, width]
"croppings" is the binary mask
I receive a negative loss value between -0.6 and -0.5.
How can I deal with it? Is it normal to have a negative crf loss?
Thank you

I meet the same problem, have you solved it ?

This is normal due to the specific continuous relaxation for the discrete CRF energy.
See https://github.com/meng-tang/rloss/issues/3


Can I have the documentation for Sil IR?

Thank you for the great analysis tool.
I am currently trying to understand the internal, and so I want to check any documentation (or maybe paper?) regarding the language design of Sil IR.
Can I have this documentation? Or, is the source code the best materials to understand the Sil IR?
Thank you!

SIL is a graph with 5 kinds of instructions. The instructions are explained in the API document.
https://fbinfer.com/odoc/next/infer/IR/Sil/index.html#type-instr
Could you let us know which kinds of information about the IR you want?

Thank you, I will check the documentation!


How to escape special chars? e.g @

I am having some trouble extracting the value of a field in an Odata query, a field named @odata.count.
I have tried the following, but without success:

<EMAIL_ADDRESS>- parse error
<EMAIL_ADDRESS>- still parse error
$['\x40odata.count']  - no parse error, but returns nothing

It seems it clashed with some type feature of JSONPath plus, which specifies that you use ` (backtick) to escape.


Payload serialization & removal of the queue names

Changes:

Payload serialization
Queue names removed from initialization
Minor code refactoring

Sorry for multiple PR, still not ready, methinks


setup: pin moto to under 2.0

I didn't had chance to debug it fully though moto>2.0 is incompatible with the tests as is (2 version check tests are failing) intake/filesystem_spec/pull/557

( cc @mariusvniekerk , in case you know how versioning is different in moto 2.0 - no need for any action here)

Thanks!


http: add support for dynamically injecting filters into the chain

Adds support for a set of new filter callbacks that allows a filter to
inject a filter into the filter chain immediately following itself. This
allows a filter to inspect the incoming headers and dynamically adjust
what filters are being executed.
Filters can only be injected before headers have made it to the next
filter; this ensures that the injected filters will be treated exactly
as any other filter.
See https://github.com/envoyproxy/envoy/compare/master...snowp:composite-filter?expand=1 for a working example of a filter making use of this API.
Part of #12968
Signed-off-by: Snow Pettersen<EMAIL_ADDRESS>Risk Level: Medium, new unused feature but changes to FM
Testing: UTs
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]

@snowp can you explain a bit more on how the the lifecycle of filters and filter factories works in this setup? Would be helpful to have some documentation on this beyond just the patch linked.

@snowp can you explain a bit more on how the the lifecycle of filters and filter factories works in this setup? Would be helpful to have some documentation on this beyond just the patch linked.

Sure, let me add some comments to the callbacks that talks about expected usage

Sure, let me add some comments to the callbacks that talks about expected usage

So to elaborate on the use case here, the goal behind this change is to allowing a remote server a lot of flexibility to influence the filter iteration (see https://docs.google.com/document/d/1dOa_v1eMA4F61f0jdFkxtbcO-_TYD76oMfGoY7nzZhk/edit#heading=h.5wabejoc1qk5 for the original doc).
The immediate use case for us is being able to express
match on header X:
if value == y => use filter config A
if value == z => use filter config B

as a way to facilitate fault injections based on a header identifying which experiment the current request belongs to.
Some possible solutions to this:

extend the fault filter to support being passed fault configuration as part of a match action
provide a filter that can create an inner filter based on the provided config, delegating HCM callbacks to the inner filter
provide a hook into the FM that allows to inject filters on demand, using a filter to create the delegated filters but hand over management to the FM (which this PR is a part of)

Solution 1 works but ends up being very specific to the fault filter, which means that this flexibility is not afforded in a generic way.
Solution 2 & 3 are conceptually similar, the main difference being whether we have to manage the delegated filters ourselves and whether we can support injecting multiple filters (this becomes very tricky to do in a delegated manner).
They have the same implications when it comes to making the list of filters dynamic, though solution 3 ensures that the FM is aware of the final filter ordering. The addition of a filter callback to add a dynamic filter does increase the complexity of the filter API, but does open up for a lot of possibilities (e.g. https://github.com/envoyproxy/envoy/issues/11690)
Let me know if you have any thoughts on whether this is the right approach here, totally open for suggestions! @htuch @alyssawilk

So to elaborate on the use case here, the goal behind this change is to allowing a remote server a lot of flexibility to influence the filter iteration (see https://docs.google.com/document/d/1dOa_v1eMA4F61f0jdFkxtbcO-_TYD76oMfGoY7nzZhk/edit#heading=h.5wabejoc1qk5 for the original doc).
The immediate use case for us is being able to express
match on header X:
if value == y => use filter config A
if value == z => use filter config B

as a way to facilitate fault injections based on a header identifying which experiment the current request belongs to.
Some possible solutions to this:

extend the fault filter to support being passed fault configuration as part of a match action
provide a filter that can create an inner filter based on the provided config, delegating HCM callbacks to the inner filter
provide a hook into the FM that allows to inject filters on demand, using a filter to create the delegated filters but hand over management to the FM (which this PR is a part of)

Solution 1 works but ends up being very specific to the fault filter, which means that this flexibility is not afforded in a generic way.
Solution 2 & 3 are conceptually similar, the main difference being whether we have to manage the delegated filters ourselves and whether we can support injecting multiple filters (this becomes very tricky to do in a delegated manner).
They have the same implications when it comes to making the list of filters dynamic, though solution 3 ensures that the FM is aware of the final filter ordering. The addition of a filter callback to add a dynamic filter does increase the complexity of the filter API, but does open up for a lot of possibilities (e.g. https://github.com/envoyproxy/envoy/issues/11690)
Let me know if you have any thoughts on whether this is the right approach here, totally open for suggestions! @htuch @alyssawilk

@snowp when I read the original doc, I had in mind the flow in https://docs.google.com/document/d/1dOa_v1eMA4F61f0jdFkxtbcO-_TYD76oMfGoY7nzZhk/edit#. I.e. we are just selectively enabling/disabling filters. That's super easy to understand but also limited for your use case.
In the flows above, who actually is doing the filter chain injecting? Is the fault filter injecting a new filter for config A or B? Or is some other special matcher filter making the choice and injecting different fault filters? This model I think gives a lot of rope; we can easily get into a place where the actual filter execution is very hard to predict. I don't know if that's actionable but just an observation.

@snowp when I read the original doc, I had in mind the flow in https://docs.google.com/document/d/1dOa_v1eMA4F61f0jdFkxtbcO-_TYD76oMfGoY7nzZhk/edit#. I.e. we are just selectively enabling/disabling filters. That's super easy to understand but also limited for your use case.
In the flows above, who actually is doing the filter chain injecting? Is the fault filter injecting a new filter for config A or B? Or is some other special matcher filter making the choice and injecting different fault filters? This model I think gives a lot of rope; we can easily get into a place where the actual filter execution is very hard to predict. I don't know if that's actionable but just an observation.

@htuch My idea would be for a special filter to handle the injection of the filter: it would snap the filter factories for each possible configuration during its own creation (which means filter factories are created at the same time as regular filters), and then the match result would indicate which factory to use to create the filter to inject. This means that filters like the fault filter doesn't have to know anything about this, making it possible to mix and match filters if desired.
One of the problems with this approach (as you point out) is that the filter API surface is pretty huge: any enforcement of which filters to inject, when to inject them, etc. is left up to the filter doing the injection. As a result the possible usages of the API might be hard to predict.
So while I like the flexibility this gives us, I'd like to make sure that we'd be okay with supporting this API in the future (@alyssawilk might have thoughts here? or @mattklein123). I can imagine more elaborate APIs that would lock things down a bit (e.g. using the idea of declaring dynamic slots in the filter config that can be populated during filter chain iteration with a filter from an allowlist), but the alternatives comes with their own set of complexities.
The most conservative approach is one of the options I mentioned where we don't mess with the filter chain at all and just teach the fault filter (and any other filter that we want this kind of dynamic behavior on match results) to understand match filter-specific match actions.

I think what you have makes sense when it's used by these special filters implementing matchers. Definitely interested in hearing @alyssawilk and @mattklein123 take on this though.
Part of me wishes this was only available in allowlisted scenarios where we actually need this flexibility, but if we're adding it to the public interface, I'm not sure this is something that makes sense. I have a non-specific unease with self-modifying filter chains, but nothing I can pin down to say we shouldn't do this.

@snowp asked me to take a look at this and I should have time to look through tomorrow.

Paging all of this back in, I had thought that we were effectively going to go with your solution (1) here: https://github.com/envoyproxy/envoy/pull/14722#issuecomment-769217255.
I agree it's not optimal in the sense that the each filter has to implement it's own logic, at the same time, it feels like a fairly natural pattern that we could embed filter specific and have it be passed as part of the match action, much like how we allow route specific config today. We could probably even type check this in exactly the same way to make sure the config lines up with the filter it's being passed to and is already pre-loaded and verified.
IMO this is simpler to understand and still quite flexible. We could do something like this PR later but my preference would be to avoid it until if/when we really feel that we need it?
I'm happy to be swayed but that's my 2c.

IMO this is simpler to understand and still quite flexible. We could do something like this PR later but my preference would be to avoid it until if/when we really feel that we need it?

Yeah given the push back I'm inclined to go that direction. I'll close out this PR and then get something working using the delegated approach, and then we can revisit this if we need this flexibility down the line.


QiskitRuntimeService url argument from env variables.

What is the expected feature or enhancement?
Ability to construct QiskitRuntimeService object with url argument from env variable (let's say QISKIT_AUTH_URL).
This will enable to set auth url from env variables in different dev environments.
Acceptance criteria
When env variable is set one can construct QiskitRuntimeService class without specifying url argument.
Example:
print(os.environ)
# environ({"QISKIT_AUTH_URL": "https://my_awesome_url/api"})

service = QiskitRuntimeService() # same as I would call QiskitRuntimeService(url="https://my_awesome_url/api")

We already have this env var - QISKIT_IBM_URL. Perhaps we need to document it better.

There is documentation regarding QISKIT_IBM_URL in contributing.md - I think this is sufficient, but we could also update the setup-channel doc. If that's the case feel free to open this issue in https://github.com/Qiskit/documentation


CSR tables in priv section 2.2 are rendered incorrectly

See original vs. current.  The third column is wrapping.  All columns should have the same number of rows.

Fixed with https://github.com/riscv/riscv-isa-manual/commit/3c5943d67bc2c9f3ed1f85d730964e9845e942b5

Thanks!


[MERGE] 1.7 -> master

No conflicts

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18841/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18843/
Test PASSed.


Added translateSettings, for using different delimiters for translate.

Hello,
I really like your plugin implementation for localization.
But I have other separators for variables in my project localization files.
And I made a small addition :) added translateSettings
// for %{}, 'Hello, %{name}'

underi18n.translateSettings = {
    regexp: /\%\{(.*?)\}/g,
    left: '%{',
    right: '}'
};

Thanks!
I also use different separators in certain projects. So if for example I want to use {{ and }} I would just do:
underi18n.templateSettings = {
    translate: /\{_([\s\S]+?)_\}/g,
    i18nVarLeftDel: '{{',
    i18nVarRightDel: '}}'
};

Isn't that enough? I am not sure why it is necessary to introduce something new. Can you please explain?

Hello,
underi18n.templateSettings used only for templates.
You used predefined regexp for replacing in translation catalogs - '${'+key+'}'and /\$\{(.*?)\}/g.
My fix, allows use of custom delimiters for translate catalogs,
in MessageFactory.translate and underi18n.template functions.
// chnage prefix `$` => `%` `%{count}`
underi18n.translateSettings = {
    regexp: /\%\{(.*?)\}/g,
    left: '%{',
    right: '}'
};

var catalog = {
    energy: "%{count}/%{maxCount}"
};

var t = underi18n.MessageFactory(catalog);

t('energy', {count: 2, maxCount: 5}); // 2/5


scroll research

resolves issue: #2
A detailed investigation and documentation of the data messaging process between Layer 1 (Ethereum) and Layer 2 (Scroll) using the Scroll L2 platform. The document covers Scroll's architecture, including the Scroll Bridge, Gateway Architecture, and the Settlement, Sequencing, and Proving Layers.

Thanks for the good work!

thank you chief @dragan2234


Migrate TOSA input conversion to a compiler plugin.

Progress on https://github.com/openxla/iree/issues/15468
Using the plugin API lets us avoid plumbing #ifdef IREE_HAVE_TOSA_INPUT across the source tree. The IREE_INPUT_TOSA CMake define now just chooses whether to include the plugin or not.
Future PRs will migrate other components like the StableHLO input conversion. I chose to start with the TOSA conversion because it has fewer files.
Build system details
This is mostly a code move. The Bazel and CMake configurations were a little tricky though. Note that some CMake files here were authored manually, not via bazel_to_cmake.
The Torch input conversion has files located at compiler/plugins/input/Torch/torch-iree/**, with includes rooted on torch-iree (e.g. #include "torch-iree/InputConversion/Passes.h"). However, Bazel wants include paths to match the (unambiguous) WORKSPACE-relative paths (e.g. #include "compiler/plugins/input/Torch/torch-iree/InputConversion/Passes.h"). Solutions for working around that in Bazel are documented here.
Similarly, the Torch input conversion roots its CMake targets at torch-iree::InputConversion. In cases where we use relative targets that isn't an issue, but bazel_to_cmake does not understand that for absolute target labels.
We have a few options:

Use WORKSPACE-relative paths and generate CMake files from Bazel files
(this PR) Use plugin-relative paths and manually author a few CMake files
Use plugin-relative paths and teach bazel_to_cmake about include copts
Use something like //compiler/src:defs to propagate include paths (I tried this a few ways already)
Use relative include paths (./...)

Future work

This new plugin depends on iree::compiler::Dialect::Flow::Transforms for IREE::Flow::createStripSignednessPass(). I think this is the only use of that pass, so it could be moved in to the plugin.
Mixed input dialect conversion is worth trying now

Nice. Could you double check that the lit tests that moved are actually running?

Bazel: https://github.com/openxla/iree/actions/runs/6817676816/job/18541690731?pr=15495#step:4:677
//compiler/plugins/input/TOSA/tosa-iree/InputConversion/test:convert_i48_to_i64.mlir.test PASSED in 0.6s
//compiler/plugins/input/TOSA/tosa-iree/InputConversion/test:tosa_to_linalg_ext.mlir.test PASSED in 0.6s
//compiler/plugins/input/TOSA/tosa-iree/InputConversion/test:verify_compiler_tosa_input_legality.mlir.test PASSED in 0.8s

CMake: https://github.com/openxla/iree/actions/runs/6817676816/job/18541913066?pr=15495#step:6:1185
          Start  792: tosa-iree/InputConversion/test/convert_i48_to_i64.mlir.test
 517/1417 Test  #718: iree/tests/e2e/tosa_ops/check_vulkan-spirv_vulkan_logical_right_shift.mlir .................................   Passed    0.27 sec
          Start  793: tosa-iree/InputConversion/test/tosa_to_linalg_ext.mlir.test
 518/1417 Test  #724: iree/tests/e2e/tosa_ops/check_vulkan-spirv_vulkan_mul_shift.mlir ...........................................   Passed    0.25 sec
          Start  794: tosa-iree/InputConversion/test/verify_compiler_tosa_input_legality.mlir.test

VSCode CMake Test Explorer:

Ah, I should also update CODEOWNERS for the new paths: https://github.com/openxla/iree/blob/3ef0ea1127c196029dfbd8764c94412778d592b7/.github/CODEOWNERS#L75-L77


Documentation :: Invalid link to cukes.info/step-definitions from Multiline Step Arguments page

the page https://github.com/cucumber/cucumber/wiki/multiline-step-arguments
has link to cukes.info/step-definitions, which cannot be found.
Suppose the correct link should be: https://cukes.info/docs/reference#step-arguments

Thanks, I've updated the wiki page.
I'm curious - were you unable to edit the wiki page yourself, or did you not want to for some reason?

Hi Matt.
I was not sure that the supposed url was correct. That is why I decided not to edit it myself.
I'm new to cucumber and did not want to mess something in the documentation.

Well you may be new but you’re already contributing, thanks!


Automatically detect application release?

While digging through this integration, I noticed that the docs include a release: config.
That config is not set by default, but since this package relies on ocramius/package-versions, it may be trivial to set a default, by using $defaultVersion = \PackageVersions\Versions::getVersion(\PackageVersions\Versions::ROOT_PACKAGE_NAME);
That is much simpler than requiring the downstream consumers of the package to invent a release/deployment version environment variable.

Proposed the implementation for this in #292


Add Support generator to build a template webpack setup

> rails generate react_on_rails:install

Created generators for:

React and Redux folders
Assets folder
Symlinks for images and fonts of they exist
Blank global jsx files
Update gitignore to ignore node modules
Insert generated and react_on_rails includes into application.js
Copy react_on_rails.rb config to app/initializers
Nodejs server files for the hot reload
Webpack config files
npm package

I moved rails engine to its own file to be more explicit.
Regarding the npm package.json, I copied it from the tutorial repo. I'm not sure if all of those packages should be included in the gem, or if you want to slim it down to the barebones react libraries.

Checking this out now! @rstudner can you give this a try? or any others on the team. I'll take a quick look as well.

For file: lib/generators/react_on_rails/templates/client/app/startup/clientGlobals.jsx and some of the other files, let's put in a few useful doc links on what is supposed to go in these files. We can provide:

URL to example file in the tutorial
A few sample lines

Wow! Great start! Let's consider getting this in v1.0.0, or very shortly thereafter. @aaronvb Don't hesitate to ask us if you have any questions. If you need any help with the suggestions, please let us know. This is very useful.

@justin808 Thanks for the feedback. I'll be going through this over the next few days and will join you guys in Slack for any questions. A lot of what I have in here was extracted from the webpack tutorial, so I expected lots of changes and fixes.

I've pushed updates to all the files noted above and added:

Made changes to the generators to separate react and redux.
Added the JS linter files so npm run lint works now
Readme's

Install stays the same, except it doesn't create the redux folders or include the redux packages in package.json
rails generate react_on_rails:install

Install with redux will do the same as the regular install, but create the redux directories and include redux in the package.json.
rails generate react_on_rails:install_with_redux

Please review the copy in both README files.

Do we want to add a command line parameter for indicating 0.13 vs 0.14 setup (exposing ReactDOM and ReactDOMServer as appropriate)

@rstudner @aaronvb Regarding any command line param for the old react, there's probably no need if we're doing a generator. As far as I can tell, React 0.14 gives warnings for the deprecations, so those are relatively easy to fix.
That being said, there could be cross dependencies that might not yet like React 0.14.
So if it's easy to add a param, then sure, but this is not a huge priority, at least not until at least one person really wants it.

Correct -- is definitely not necessary.  Just suggested it since some people get all upset when their chrome console has warnings in it they don't understand ;)

@rstudner You mean terminal, right?

@aaronvb would you say this is ready to go? I'll take a look at this tomorrow, and give it a spin.

@justin808 In my opinion, yes. If all looks good to you, I will update the README.md in the gem with instructions on using the generate install.

@justin808 Sorry I missed this in the email.  yeah, I meant terminal sorry.  The use of the word 'console' was an extremely poor choice on my part :)

So my first run.. npm 2.14.4 and node v0.12.7 'failed'
     run  cd client && webpack --config webpack.client.rails.config.js from "."

sh: webpack: command not found
run  cd client && webpack --config webpack.server.rails.config.js from "."
sh: webpack: command not found
Basically webpack didn't get installed etc.  Checking/looking into why now

@aaronvb I tested this PR on my project, using Rails 3.2.22/Ruby 2.2.2, it works very nice. Saving A LOT of boilerplate. Thanks for this :+1:
I will report if I find anything useful.

Does the Procfile.dev etc need to get created (is that part of the boilerplate)
I do realize there are other ways to startup the rails & node servers in parallel etc.
But since the react-webpack-rails-tutorial project has the 'opinion' to do Procfile.dev should this?

Procfile.dev is very handy and well worth adding.

I'm going to take this over for now. Thanks @aaronvb!

From CI build. Rubocop errors.

I recommend setting the .babelrc stage to 2 or 1 as stage 0 is not considered recommended for production code.

@gugl We're on the stage 0 mainly because of static properties. Moreover base libs (like redux & react-router) are compiled with stage 0, so anyway we're using it in production.

Next up:

Add linter gems to Gemfile
Add hello world react component -- let's use the same simple setup we use for the spec/dummy app
Don't run webpack in the fresh generator install, as it doesn't pick the version just installed. P

Per running steps:
From directory of react_on_rails, with a test app named "react_on_rails_gen"
cd ..
rails new react_on_rails_gen
cd react_on_rails_gen
git init
git add .
git commit -m "Rails new plus react_on_rails"

Edit the Gemfile, adding these 2 lines:
gem 'react_on_rails', path: '../react_on_rails'
gem 'therubyracer'

Note the relative path to the react_on_rails gem.
bundle
git commit -am "added react_on_rails gem"
rails generate react_on_rails:install

Then you can see the changes that the generator did, ready for a commit.
Then run:
foreman start -f Procfile.dev

Then visit ports 3000 and 4000.
That's it!

For extra credit:

Add bootstrap sass options
2 Add some tests.

However, we can do those in a separate PR.

@aaronvb Let's run some basic sanity check that nvm, npm, node are installed and abort the generator with a link to get those installed. I think this is going to be the basic stumbling block for most of the ruby community when they first see this.

Added linter gems to Gemfile.
Added a simple react hello world component that demonstrates client rendering, server rendering, and hot reload.
You can generate the hello world files by using rails generate react_on_rails:hello_world After generating the initial install.
Removed running webpack after initial install.
Added a check for npm and node before installing, will abort with a message containing a url to the libraries.

Fixing CI errors now.

@aaronvb
Just pulled latest. Looking AWESOME

ci.rake file is empty
clientGlobals.jsx should have the hello world setup
putting an empty images directory. That should probably be a symlink: https://github.com/shakacode/react-webpack-rails-tutorial/tree/master/client/assets Same for fonts.
Need to put in current versions of node and npm: https://github.com/shakacode/react-webpack-rails-tutorial/blob/master/client/package.json#L7
Nice job on client/README.md
Let's have server globals be there
Let's have the hello world example do server rendering.

Overall, very awesome!

@justin808

ci.rake file is empty
Fixed

clientGlobals.jsx should have the hello world setup
The clientGlobals.jsx gets updated with the hello world setup when using the rails generate react_on_rails:hello_world generator. I can have the hello world example code created with the main install generator.

putting an empty images directory. That should probably be a symlink: https://github.com/shakacode/react-webpack-rails-tutorial/tree/master/client/assets Same for fonts.
Images and font directories are symlinks and can only be symlinked if they exist. In a new Rails app, the font directory does not exist so a symlink is not created. See https://github.com/aaronvb/react_on_rails/blob/generators/lib/generators/react_on_rails/setup_files_generator.rb#L8

Need to put in current versions of node and npm: https://github.com/shakacode/react-webpack-rails-tutorial/blob/master/client/package.json#L7
Fixed

Let's have server globals be there
See 2

Let's have the hello world example do server rendering.
https://github.com/aaronvb/react_on_rails/blob/generators/lib/generators/react_on_rails/templates/hello_world/app/views/pages/index.html.erb#L4

The work for this PR branch has been merged to a new branch on the original repo: https://github.com/shakacode/react_on_rails/tree/add-generators

@justin808 I propose that we close this so as to ensure all future progress is made on #70, right?


Byte aligned memory access

RISC-V allows unaligned memory reads
li x1, 0x11223344
li x2, 0x55667788
sw x1, 0(zero)
sw x2, 4(zero)
lw x3, 2(zero)

should have the value of x3 be 0x77881122
This is not necessary for the exercises, but in order to be spec compliant the byte aligned memory branch may be utilized where this feature has been implemented. This will not be merged until the semester is over at I don't like testing in production.

This is also necessary for halfword and byte reads, which we are not implementing at the moment.


BOTTOM Wrong Comment

Sorry, the first pr is so low-level. I am looking at your source code. I hope I can optimize it later. I had an idea to write a similar library before, but I shelved it.

Hey, no such thing as low-level PR, thanks for your time.


Add nojekyll file

Add a .nojekyll to the gh-pages branch so the correct action is used when building and deploying.

@jni It looks like the action for deploying from the gh-pages branch did not run. I think you might need to configure it in settings > pages after merging this PR because I think it will fail otherwise.


Direction

Issue description
Can't find direction service like from source to destination and can't draw curve path like in google map
Steps to reproduce and a minimal demo of the problem
_Use https://plnkr.co or similar -- try this template as a starting point: http://plnkr.co/edit/YX7W20?p=preview
What steps should we try in your demo to see the problem?
Current behavior
Only provide polyline.
Expected/desired behavior
angular2 & angular-google-maps version
Curved path can't be drawn in from source to destination
Other information

I would also love to use curved Polylines in AGM!


Feature request: Password protect transferred files

Is your feature request related to a problem? Please describe.
Hello and thank you for reading my request.
I was thinking that password protecting transferred files would allow for greater security.
I know that by default transferred files are end-to-end encrypted, but what if you accidentally select the wrong person? This would make it sure that only the person that you intend to share the files with can actually open them.
Describe the solution you'd like
I'd like an implementation where the user needs to put the password on a per-transfer basis, such that people who don't want to use this feature, can do as they please.
Maybe a little checkbox to toggle and that prompts you to add a password might be a good idea?
The encryption with this toggle enabled would be: first, password encrypting the files, then end-to-end encrypting them.
Describe alternatives you've considered
A potential alternative that would require more effort to pull off is verifying with a qr code: basically, you don't choose a password, but a qr code is automatically generated and only after scanning it the files would get decrypted.
Additional context
No response

For security focused users, you can enable "PIN" on the receiving device. Have you tried this out? This is a new feature introduced in 1.15.0


Date replace and/or get processes

After we've added date_shift, there are some more options that seem obviously useful:

date_replace(date, value, unit) -> date - replace parts of a date, e.g. set a date to the first of a month date_replace("2020-05-13", 1, "day") -> "2020-05-01"
date_get(date, unit) -> value - get a part of a date, e.g. get the year date_get("2020-05-13", "year") -> 2020 or get the day date_get("2020-05-13", "day") -> 13 (name could also be date_element in accordance to array_element? 😅)

With date_get and date_shift one could emulate date_replace to some degree, but not fully (e.g. can't do the "snap" behavior) so they have both their use cases.

I just had a use case that requires this: for best available pixel compositing, one weighting score is distance to the middle of the compositing period. In the context of a callback, we have access to the time dimension label (date), but we don't necessarily know for which month we are computing a composite.
With date_replace I can do something like:
abs(date_difference(label, date_replace(label, value=15,component="day")))
Seems like we can take the definitions from the PR for an initial implementation!

@jdries sure. @soxofaan are there any further suggestions for PR #433 ?

added some more suggestions


cpaulisi-train-1: added basic training notebooks for recurrent baseline

[X] Add notebook for practice training
[X] Create Basic RNN object
[x] Create training examples with local data

Early training looks fruitful for vanilla RNN

Larger Hidden Dimension shows promise for improving results


UX Problems in an Old Getting Started dark

Summary

This file does not perform as expected and a little confusing as far as UI.
Steps to Reproduce it

Follow the documentation
Steps to Fix

Remove the https://github.com/whylabs/whylogs/blob/mainline/docs/getting_started.rst
Should this be kept? https://github.com/whylabs/whylogs/tree/mainline/src/whylogs/cli
This example is better -  https://github.com/whylabs/whylogs-examples/blob/mainline/python/WhyLabs_Platform.ipynb

For number 2:
whylogs/examples/Analysis.ipynb References the whylogs init and would need to be updated


Client side route transition support?

After loading the first page, every navigation to a new page will trigger a full server side rendering. It seems like Fresh doesn't have client side routing right now or did i just miss something?
And thanks for the hard work, it didn't take long for me to migrate a nextjs project to Fresh :)

Fresh is primarily a SSR framework, but does include some client side interactivity with islands; as such, I personally would say client-side routing is out of scope for fresh, and something like aleph is more appropriate if such functionality is necessary

It seems like Fresh doesn't have client side routing right now or did i just miss something?

Correct, all pages are server side rendered only. Only "island" components are shipped to the client to be rendered there.


Web - Search string in WorkspaceMembersPage is lost when moving to another page and coming back

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to settings
Click on workspaces
Click on a workspace
Click on Members
Search for a string, click on invite.
Click back and find that search field is empty.

Expected Result:
Search field should persist when navigating back to Members page from Invite page
Actual Result:
Search field becomes empty
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.38-3
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/244f9f7c-e2fd-4cd6-94a7-b4bbf05d28d3
https://github.com/Expensify/App/assets/93399543/39e5d88c-a387-40d7-ad42-1e9e83cdcd16
Expensify/Expensify Issue URL:
Issue reported by: @c3024
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688896241730799
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Search string is lost when clicking on Invite members from WorkspaceMembersPage and coming back to WorkspaceMembersPage
What is the root cause of that problem?
When we are navgiating to Invite page, we are setting searchValue as '' in WorkspaceMembersPage
const inviteUser = () => {
        setSearchValue('');

so it is lost.
What changes do you think we should make in order to solve the problem?
We should remove the setSearchValue line above
What alternative solutions did you explore? (Optional)

This feels like a dupe of https://github.com/Expensify/App/issues/22534
So I think we should just close this one out.

dupe, closing!


Remove duplicate customize_install function from get-poetry.py

Looks like there were two identical copies of the customize_install function
in the installer script. I removed one and fixed a couple minor typos
along the way.

Thanks!


Example gulpfile is not working with an error "$.sprockets.declare is not a function"

gulpfile.babel.js in Usage section doesn't work because of errors like "$.sprockets.declare is not a function".
For a workaround, I have to change $.sprockets to $.sprockets.default to make it work because exported sprockets is an object kind of { default: { css: [Function],  js: [Function],  precompile: [Function]  scss [Function], declare: [Function] } }.
However, for my understanding it should work as long as you set __esModule as an property and export it as exports.default = sprockets that exactly I see in your codes.
Furtheremore, if you do import sprockets from 'gulp-sprockets' without using gulp-load-plugins, it seems to work.
So it might be a problem related to gulp-load-plugins, but the example gulpfile is not working anyway.

Hi @5t111111 yeah Im having an error $.sprockets.default is not a function any work around?
Thanks

@delacruzjames as I wrote, try changing $.sprockets to $.sprockets.default


Language Client is not ready yet

Hi, i'm having some trouble to get coc-volar running after the update 0.5.30
It just occur when i type anything inside any vue file.

Is there any config im missing?

In addition to this error, when its initialized throws this message:

@guihig Hi, Thanks for the issue report.
I can't reproduce the error in my environment.

Can you please share the output of :CocCommand volar.doctor?
What kind of input will cause this error?
I've just released coc-volar v0.5.31 (Sync VSCode's volar v0.27.21). Do you get the same error after updating to v0.5.31 with :CocUpdate?

Hello, i've just updated and the error is gone, thank you


[Bug]: ShaftEngine_ElementActions_shaft waited long time before clicking an existing element.

Describe the bug
before clicking the element shaft waited the defined duration in timeout property file > defaultElementIdentificationTimeout ,
the element :

the locator that cause the problem :
By.xpath("//app-checkbox[@name='prepaid']//input")
another locator for same element but didn't cause the issue
By.xpath("//app-checkbox[@name='prepaid']//mat-checkbox")
Link to SHAFT_Engine Console logs
https://gist.github.com/zanoon2020/ed8cfbfb312582988eabde07b6795699
time stamps for the issue :
[INFO ] 2023-02-25 14:47:18.211
[INFO ] 2023-02-25 14:48:19.552
Environment

SHAFT_Engine version that exhibits the issue (Note: We support only the latest release): 7.1.20230222
Last SHAFT_Engine version that did not exhibit the issue (if applicable):
Desktop OS: windows 10
Desktop Browser (if applicable): [e.g. chrome, safari] chrome
Desktop Browser Version (if applicable): [e.g. 22] 110.0.5481.178

To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots, gifs, or video recordings to help explain the issue you're facing.
Additional context
Add any other relevant context about the problem here.

first of all "THANK YOU" for the detailed description and the debug logs! you even enabled debugging and sent me a timestamp! I really appreciate that :D that's the best bug report I've even received.
now, investigating the debug logs, I see that SHAFT kept trying to:
findElement > which was succeeding
and then clickElement > which was consistently failing like the following example
[DEBUG] 2023-02-25 14:48:18.480 [AsyncHttpClient-1-2] ChannelManager - Adding key: http://localhost:23406 for channel [id: 0x64ed172b, L:/<IP_ADDRESS>:51246 - R:localhost/<IP_ADDRESS>:23406]
[DEBUG] 2023-02-25 14:48:18.481 [Forwarding clickElement on session 691d718f31216c12474b42a0b35fc261 to remote] NettyRequestSender - Using pooled Channel '[id: 0x64ed172b, L:/<IP_ADDRESS>:51246 - R:localhost/<IP_ADDRESS>:23406]' for 'POST' to 'http://localhost:23406/session/691d718f31216c12474b42a0b35fc261/element/11bafa21-d932-4fcc-b4fd-a510336fd267/click'
[DEBUG] 2023-02-25 14:48:18.481 [Forwarding clickElement on session 691d718f31216c12474b42a0b35fc261 to remote] NettyRequestSender - Using open Channel [id: 0x64ed172b, L:/<IP_ADDRESS>:51246 - R:localhost/<IP_ADDRESS>:23406] for POST '/session/691d718f31216c12474b42a0b35fc261/element/11bafa21-d932-4fcc-b4fd-a510336fd267/click'
[DEBUG] 2023-02-25 14:48:19.509 [AsyncHttpClient-1-2] HttpHandler -

Request DefaultHttpRequest(decodeResult: success, version: HTTP/1.1)
POST /session/691d718f31216c12474b42a0b35fc261/element/11bafa21-d932-4fcc-b4fd-a510336fd267/click HTTP/1.1
User-Agent: selenium/4.8.1 (java windows)
Content-Length: 50
Content-Type: application/json; charset=utf-8
host: localhost:23406
accept: */*

Response DefaultHttpResponse(decodeResult: success, version: HTTP/1.1)
HTTP/1.1 400 Bad Request
Content-Length: 1739
Content-Type: application/json; charset=utf-8
cache-control: no-cache

finally after the timeout expires, the engine tries to use JavaScript and succeeds, so I'm assuming that you have "forceClickUsingJavascript" enabled
[DEBUG] 2023-02-25 14:48:19.520 [AsyncHttpClient-1-2] ChannelManager - Adding key: http://localhost:23406 for channel [id: 0x64ed172b, L:/<IP_ADDRESS>:51246 - R:localhost/<IP_ADDRESS>:23406]
[DEBUG] 2023-02-25 14:48:19.520 [Forwarding executeScript on session 691d718f31216c12474b42a0b35fc261 to remote] NettyRequestSender - Using pooled Channel '[id: 0x64ed172b, L:/<IP_ADDRESS>:51246 - R:localhost/<IP_ADDRESS>:23406]' for 'POST' to 'http://localhost:23406/session/691d718f31216c12474b42a0b35fc261/execute/sync'
[DEBUG] 2023-02-25 14:48:19.520 [Forwarding executeScript on session 691d718f31216c12474b42a0b35fc261 to remote] NettyRequestSender - Using open Channel [id: 0x64ed172b, L:/<IP_ADDRESS>:51246 - R:localhost/<IP_ADDRESS>:23406] for POST '/session/691d718f31216c12474b42a0b35fc261/execute/sync'
[DEBUG] 2023-02-25 14:48:19.551 [AsyncHttpClient-1-2] HttpHandler -

Request DefaultHttpRequest(decodeResult: success, version: HTTP/1.1)
POST /session/691d718f31216c12474b42a0b35fc261/execute/sync HTTP/1.1
User-Agent: selenium/4.8.1 (java windows)
Content-Length: 228
Content-Type: application/json; charset=utf-8
host: localhost:23406
accept: */*

Response DefaultHttpResponse(decodeResult: success, version: HTTP/1.1)
HTTP/1.1 200 OK
Content-Length: 14
Content-Type: application/json; charset=utf-8
cache-control: no-cache

From the tree you shared, I think the issue is as follows:

SHAFT is designed to wait until an element is interactable, and this element is not interactable, it's throwing an invalidelementstate exception, which is why SHAFT keeps looping until the timeout expires.
If you can, let's disable "clickUsingJavaScript" and try again, SHAFT should fail and provide a staktrace that should show the invalidelementstate as the root cause.
This is usually caused by FE developers not propagating actions properly... the action (method) is actually placed on the wrapper element (the locator that's working for you) and not on the input element, because a regular user wouldn't be able to click an element that they cannot see, so this issue is never reported... but it impacts automation.
The correct solution is to adjust your locator, and always use the upper most element locator. HOWEVER, I think it makes sense to add a public "clickUsingJavascript()" method in SHAFT's element actions, so that users can use it whenever they need to.

In conclusion, I think this is working as designed, so I'll be closing this bug. However, I will refer to this bug and open a new feature request to expose clickUsingJavaScript() in the fluent element actions class. This will be an easy thing to do since the function is already in the code, it's just not exposed.


updating header for mobile view

#7328
[UI]-kubernetes notification drop down is cut from the left side in mobile mode at https://github.com/meshery/meshery/issues/7328
Notes for Reviewers
This PR fixes #7328
video of changes:
https://user-images.githubusercontent.com/91942072/233275607-d9802e78-eaae-4d0e-afcf-277f4f2d8c92.mp4
Signed commits

[ ] Yes, I signed my commits.

@leecalcote can you review

@leecalcote can you review

@leecalcote could you review?

@parmishh, do be sure to reach out to other maintainers so I don't slow you down. @meshery/meshery-site-maintainers @meshery/meshery-site-contributors

@thisiskaransgit Can we get this merged.


audit/manifests: log requests to Events

Re-enable audit logs about Events to be able to identify misconfigured clients.

/retest-required

/approve
/hold
in case you would like to test something.

Verified on the proof PR.
/unhold


Divergent branches

Hey @RyanDavies19 and @AlexWKinley !
For some reason (probably we rebased on master instead of merging) the dev branch has diverged from the master one. I rebased and the problem was automagically gone, so it is not really a big deal. However, if I do that I will break all your on-going work (including the active PRs).
Thus, we can proceed in two ways (I really think the first way is the good one):

Launch a new release forcily pushing the current dev on master
Rebase dev on master

The merit of way 1 is that everything will work just fine for all us. The problem is that we are forcily pushing on master, so some remote forks  on the wild might have problems to create PRs. I think the risk is rather low because we are actually the only active devs of this project. If someone complains on the close future I will take full responsibility on fixing the issue.

I'm fine with force pushing to master. I am not aware of anyone doing and development, and if they are I would think they would be working off the dev branch.

OK, I force pushed. Patch release is on its way!


README edge-case: --ca-cert from <(bosh int) doesn't work if BOSH_LOG_LEVEL=DEBUG

If you are trying to run the command from the README:
bosh -e <IP_ADDRESS> --ca-cert <(bosh int ./creds.yml --path /director_ssl/ca) alias-env bosh-1

And get the error:
[CLI] 2017/08/22 09:55:20 ERROR - Validating Director connection config: Parsing certificate 2: Missing PEM block
Validating Director connection config:
  Parsing certificate 2: Missing PEM block

Exit code 1

Check your BOSH_LOG_LEVEL -- if it's "DEBUG" then the process-substitution in the shell command won't quite work because the output has extra lines

Closing as it's just an fyi -- too much of an edge case to be in the readme itself.

@cppforlife, I think logger should be sending output to STDERR instead of STDOUT. Objections? I think it's more correct, and would also happen to fix this particular scenario.

Actually, closing in favor of https://github.com/cloudfoundry/bosh-cli/issues/301


Update scodec-bits to 1.1.18

Updates org.scodec:scodec-bits from 1.1.17 to 1.1.18.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scodec", artifactId = "scodec-bits" } ]

labels: library-update, semver-patch

Codecov Report

Merging #1509 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1509      +/-   ##
==========================================
+ Coverage   85.84%   85.98%   +0.13%
==========================================
  Files          58       58
  Lines        2197     2197
  Branches      107      107
==========================================
+ Hits         1886     1889       +3
+ Misses        311      308       -3

Impacted Files
Coverage Δ

...d/src/main/scala/io/circe/NonEmptySeqDecoder.scala
91.66% <0.00%> (-8.34%)
:arrow_down:

...les/core/shared/src/main/scala/io/circe/Json.scala
79.32% <0.00%> (-1.12%)
:arrow_down:

.../core/shared/src/main/scala/io/circe/Decoder.scala
85.32% <0.00%> (+0.27%)
:arrow_up:

...re/shared/src/main/scala/io/circe/JsonNumber.scala
97.59% <0.00%> (+6.02%)
:arrow_up:

Continue to review full report at Codecov.

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


[feat] Member Card popup on submitting from

@hparashar27 you can assign this issue to me.

@alokVerma749 yes you can start working it 👍

completed


Run NAT diagnostic in it's own freedom module.

NAT classifying is currently planned to be added into generic_core (https://github.com/uProxy/uproxy/pull/1173)
Ideally we would want diagnose.ts (and messages.ts) to run in their own freedom module, similarly to how uproxy-probe's Chrome app runs the NAT test (https://github.com/uProxy/uproxy-probe/tree/master/src/chrome-app).
In order to do this, we should:

cleanup the diagnose code and add its API to the .json file (export appropriate functions and class?)
publish uproxy-probe to NPM
include uproxy-probe in uproxy's dependencies
edit core.ts to use the communicate with the new module instead of calling Diagnose.doNatProvoking as is happening in the above PR.

Two comments:

I don't think the probe needs to run in its own freedomjs module.
Maintaining another NPM will be a pain; I would be happy to see this code move to uproxy-networking.

Thoughts?

I like Lucy's original plan. Perhaps Lucy can better clarify what the impetus is for moving the NAT classifying into a separate freedom.js module.
In my mind, it removes complexity from core (is a separate unit of function), removes core's dependence on raw socket APIs, and makes it reusable for others.

We can also achieve those three things if the re-useable code is in uproxy-networking, no?

I dont' know much about this but in general I'm in favor or having less
repositories. it'd make sense to run it in its own freedom module though.
On Tue, Apr 7, 2015 at 1:37 PM, trevj<EMAIL_ADDRESS>wrote:

We can also achieve those three things if the re-useable code is in
uproxy-networking, no?
—
Reply to this email directly or view it on GitHub
https://github.com/uProxy/uproxy/issues/1175#issuecomment-90663179.

NAT diagnosis code has been moved into uproxy-lib. I don't see any reason to give it its own freedomjs module.


config.componentFactoryResolver not been passed to ComponentPortal constructor in MatDialog._attachDialogContainer method

Bug, feature request, or proposal:
Bug.
What is the expected behavior?
The custom componentFactoryResolver of Material Dialog config parameter should be passed to ComponentPortal in _attachDialogContainer and _attachDialogContent methods.
What is the current behavior?
The custom componentFactoryResolver of Material Dialog config parameter is only passed to ComponentPortal in the _attachDialogContainer method.
What are the steps to reproduce?
Just provide a custom componentFactoryResolver to dialog.open.
What is the use-case or motivation for changing an existing behavior?
Can´t use the custom componentFactoryResolver to instantiate new components.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular: 8.2.13
Material: 8.2.3
All browsers affected
Is there anything else we should know?
I belive that to solve the problem we just need to change the line 264 (method _attachDialogContent) of dialog.ts, passing the custom componentFactoryResolver to ComponentPortal constructor. I did it locally and worked properly:
dialog.ts:
264: new ComponentPortal(componentOrTemplateRef, undefined, injector));

to
264: new ComponentPortal(componentOrTemplateRef, undefined, injector, config.componentFactoryResolver));

The method _attachDialogContainer was updated with this feature in
https://github.com/angular/components/commit/565bd7da67ea5c789f14ae37d0f7fb6c85221593

Hi - Could we get the fix for this merged? It looks like it's been approved. Thanks.

Hi - Could we get the PR for this released? It looks like it's been approved. Thanks.


Add LICENCE

Signed-off-by: Doğukan ÇELİK<EMAIL_ADDRESS>

Related issue #47

LGTM, thanks!

Actually @dgkncelik one more thing, do you mind squashing those commits? Thanks :)

sorry for the inconvience :(
just opened new pull request with correct naming
#48


WIP: Update MARK-V to version 1.01

Includes:

Multi-sequence move-to-front

Coding by id descriptor

Statistical coding of non-id words

Joint coding of opcode and num_operands

Removed explicit form Huffman codec constructor
The standard use case for it is to be constructed from initializer list.

Fixed long compilation time with clang/ninja

Moved markv_autogen.inc to a separate comp module
Shrinked markv_autogen.inc to a smaller file (for now)
Fixed assertion

Removed some unused move-to-front sequences

This is squashed and rebased form of #759. I'm doing this to see if the bots pass.
FYI: @atgoo

MSVC bot fails.  Closing.  @atgoo needs to rebase the original PR.
Closing.


Godef doesn't build now that 9fans has moved to github?

I was setting up godef today on a new laptop and upon running go get github.com/rogpeppe/godef I received an error:
[user@localhost]$ go get code.google.com/p/rog-go/exp/cmd/godef
package code.google.com/p/rog-go/exp/cmd/godef
        imports 9fans.net/go/acme: unrecognized import path "9fans.net/go/acme"

seeing what appeared to be the issue, I changed the imports to in acme.go to:
"github.com/9fans/go/acme"

And that started to help, but then all the [9fans] (https://www.github.com/9fans) packages are still importing things like 9fans.net/go/plan9 so the same problem kept arising. Eventually I changed all the 9fans.net/* imports to github.com/9fans/* and go get command completed successfully.
I may just be building this incorrectly or my setup could be broken, so let me know if that's the case! I'm running go 1.4.2 built from source on Fedora 21 64-bit. Let me know if there's any other info I can provide that will help.

too

Same problem here.

:+1:

same issue here
@medvednikov
can you give me an workaround for Vundle install ?

yeah, the same problem came to me too.

I need to set up a notification on godef issues! Buried amongst a pile of github emails, I haven't seen this before. I will vendor its dependencies forthwith. Thanks for the report.

@rogpeppe fwiw i think it's fixed now

Fixed by #35


Enhance Input AST with namespace and encoding properties.

The identifier nested under a prefix must have the namespace property set to the namespace source.
The identifier or string nested under value must have the encoding property set to the namespace value's encoding

Add bel.rb as dependency to access rdf namespaces.


Support ~

https://www.styled-components.com/docs/api#supported-css don't have ~ information.
I need:
.input:focus ~ .highlight {
  -webkit-animation: inputHighlighter 0.3s ease;
  -moz-animation: inputHighlighter 0.3s ease;
  animation: inputHighlighter 0.3s ease;
}

I have input and highlight.

Hard to say without your code, probably something like this:
const Highlight = styled.div`
${Input}:focus ~ & {
  -webkit-animation: ${inputHighlighter} 0.3s ease;
  -moz-animation: ${inputHighlighter} 0.3s ease;
  animation: ${inputHighlighter} 0.3s ease;
}
`

@mxstbr this is an example: -> https://codepen.io/anon/pen/rGYeWE
How to make Input and Label?

In CSS the sibling selector (~) can only target siblings after the current one. To fix your issue you'll likely just have to render the label after the input:
<Input />
<Label />

That's unfortunately just the way CSS works.

The way to go here would be to abstract the two separate components inside a structural component that renders them alongside each other. It seems that a wrapper around the two is missing in your example, but I guess it's easy to add of course :wink:

Btw check out my answer to a staggeringly similar question (and solution) here: https://github.com/styled-components/styled-components/issues/74#issuecomment-334005641


Issue 184

Description
Added a HttpClient implementation of the Dapr invoke endpoints following the same pattern as the StateClient.
Issue reference
This issue is related to issue #184
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[X] Extended the documentation

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: jeastham1993 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @jeastham1993  for the contribution, this looks good. I see that you have added unit tests which is great. Have you validated it with invoking another service using Dapr.

Happy to help @amanbha . I can go through and add clients for the other endpoints as well if you like?
Yep, I built an application locally against my service and invoked a method using the API calls. All seems good to me 👍

Happy to help @amanbha . I can go through and add clients for the other endpoints as well if you like?
Yep, I built an application locally against my service and invoked a method using the API calls. All seems good to me 👍

Thats great, I will merge this. Yes, your other contributions to the project are welcome too.


Fix link colour on app start page

Context
Imported overly aggressive link styling from Register, which turns all the links on the provider start page white – hiding most of them 😭
Changes proposed in this pull request
Scopes link styling only to phase banner link.
Guidance to review

Link to Trello card

Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

@duncanjbrown That margin is correct, but will look broken in environments where the header border isn't blue. Think that's okay though.

Might see if there's a quick fix that works regardless of header border colour.


Adding a summary of used operations

Hey, thanks for netron!
How about adding a statistics of used operations in the graph (see an example below)?
One particular moment why I am interested in this - TF has quite dangerous ops like WriteFile, RPC, etc., and searching for them in a sorted list is sometimes more convenient than looking through the whole graph.

Should the list be sorted alphabetically or by count?

I definitely prefer by name, as from security point number of times these operations present does not matter.

#204

Just came across this issue when I wanted to make this feature request. Is this feature available in the latest Netron? I do not find a button to open a page of summary of operators.


Compatibility with other promise libraries

In order to show the power of being compatible with Fantasy Land and Promises/A+, allow Pacta promises to be combined with other compatible promises and monadic types.
This will probably involve some feature sniffing to determine whether another promise responds to map or then and then using that as appropriate. At the moment, Pacta assumes everything is Fantasy Land compatible which means it can't be combined with, say, the Elasticsearch client's promises.

Fixed in Pacta 0.9 with Promise.resolve which will convert any thenable to a Pacta promise.


data explorer api

@WalternativE keep an eye on this PR as will change few things for you integration with sanddance


convert console_supported? to supports?

part of https://github.com/ManageIQ/manageiq/pull/21990

update:

fixed rubocop issues


Added python lru cache

repoze is really old and python supports lru. I kinda trust python more than an old package.

@GuillemGSubies Thanks for the PR 🎉
I agree I would be much rather be using the stdlib version of lru. However, textstat currently supports Python2 and lru isn't available in the Python2 stdlib, which is why a third-party package is being used.
Dropping Python2 support is planned, I'm just not sure on the roadmap.
Thank you for your work on this, but I'm going to close this for now and raise an issue to be addressed in the future.

rebased on master and added lru_cache to the new functions.


Unable to get the apply function to work

Hello, on previous version, the apply function syntax as below
#undef EOSIO_ABI
#define EOSIO_ABI( TYPE, MEMBERS )
extern "C" {
void apply( uint64_t receiver, uint64_t code, uint64_t action ) {
auto self = receiver;
TYPE thiscontract( self );
if( action == N(onerror)) {
/* onerror is only valid if it is for the "eosio" code account and authorized by "eosio"'s "active permission /
eosio_assert(code == N(eosio), "onerror action's are only valid from the "eosio" system account");
}
if( code == self ) {
if (action != N(transfer)) {
switch( action ) {
EOSIO_API( TYPE, MEMBERS )
}
/ does not allow destructor of thiscontract to run: eosio_exit(0); */
}
}
else if (code == N(eosio.token) && action == N(transfer) ) {
execute_action(&thiscontract, &dice::dosomething);
}
}
}
the above code work on the previous compiler where when people send EOS into the smart contract, the smart contract will execute the action "dosomething"
I am not able to get this old code to work in the new CDT, any example for the apply function to work in new compiler?

nvm i found the solution as below
extern "C" {
void apply(uint64_t receiver, uint64_t code, uint64_t action) {
if(code==receiver)
{
switch(action)
{
EOSIO_DISPATCH_HELPER( example, (func1)(func2) )
}
}
else if(code=="eosio.token"_n.value && action=="transfer"_n.value) {
execute_action( name(receiver), name(code), &example::transfer);
}
}
};


Add a cli bin file

This tool should have a command line interface executable, like vulcanize, polybuild and crisper.

Particularly since the Polymer documentation claims such a thing already exists:
https://www.polymer-project.org/1.0/tools/overview.html

@jcgregorio it doesn't exist if you install it globally
also the example in the new documentation does not work

Same problem, do you have a workaround to use it with CLI?


Add pandas extra for google-cloud-bigquery to pick up missing `py…

Backport #1153

Integration tests: https://github.com/dbt-labs/dbt-bigquery/actions/runs/8461503711


[TEVA-1283] Add database seed for review apps and local development

Jira ticket URL
https://dfedigital.atlassian.net/browse/TEVA-1283

Our names are all backwards.. but looks good to me 👍

adjusted, and put the names in alphabetical order


Question regarding: Cluster configuration best practices (SQL Server on Azure VMs)

[Do all VMs that are part of the Always On Availability set need to be the same Size/Sku?]

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

ID: 66bd43a6-c244-750c-4ff1-dd12e35ecb1c
Version Independent ID: 087efa4c-7d01-0a48-b368-2d4d442be05a
Content: Cluster configuration best practices - SQL Server on Azure VM
Content Source: articles/azure-sql/virtual-machines/windows/hadr-cluster-best-practices.md
Service: virtual-machines-sql
Sub-service: hadr
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Hello @ToddGe , thanks for the question! We are taking a look into this and will get back to you soon.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@kfarlee  would you be able to assist?
Thanks!
Masha from the SQL Docs team
#assign:kfarlee

@kfarlee  would you be able to assist?
Thanks!
Masha from the SQL Docs team
#assign:kfarlee


Added BSZ-Wiesau

Institution/School Name
Staatliches Berufliches Schulzentrum Wiesau
Instiution/School Website
www.bsz-wiesau.de
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[x] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@str0yd First stage of verification passed, official domain is bsz-wiesau.de as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@str0yd Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Rule Management: add Aristotle

Aristotle was mentioned during SuriCon2023 as a complementary tool for managing rule sets and sounded like a good addition to this repo.
Disclaimer: I have not used or tested it.

Is the linting tool complaining about itself in the CI check, or am I reading that wrongly? 🤔

Thanks for the contribution!

Is the linting tool complaining about itself in the CI check

That's what I think, I need to fix it. Will take a look later.

Fixed! 👍🏻


Enable cors for COVID-19 API

🙇♂ Thank you for creating this API. Been wanting to play with the data using observable notebooks, but looks like cors is not enabled:
Test: https://observablehq.com/d/894750d4be37b21a
Enabling cross origin requests for the APIs will allow it to be used more widely in any web apps 🙏

Have got it working, looks like the issue might have been in how the JSON body was encoded. https://observablehq.com/d/894750d4be37b21a


checkDatabaseEncryptionKey returns "Failed to parse JSON object as TDLib request: Wrong padding length"

Hello,
I use JSON interface to interact with tdlib and with some keys (for example, changeme123 :) ) checkDatabaseEncryptionKey method returns Failed to parse JSON object as TDLib request: Wrong padding length (checked with tdlib v1.6.0):
{
    "@type": "error",
    "code": 400,
    "message": "Failed to parse JSON object as TDLib request: Wrong padding length"
}

Should encryption_key be passed as a base64 encoded string? I see this message in tdlib in base64 related code. Also, in td_api.tl encryption_key parameter has type bytes, but in the documentation it's string.
Also, the python example uses key parameter, not encryption_key. Which one is correct?
Thank you.

Should encryption_key be passed as a base64 encoded string?

Yes, in JSON interface there is no way to pass arbitrary binary data, so all bytes fields must be encoded with base64.

Also, in td_api.tl encryption_key parameter has type bytes, but in the documentation it's string.

The documentation available on-site is generated for native C++ interface, which uses std::string for arbitrary binary strings of type bytes. I will try to fix this and to keep in the documentation the difference between string and bytes.

Also, the python example uses key parameter, not encryption_key. Which one is correct?

The Python example is incorrect. Thanks for noticing this. I will fix the example.

Thank you!


Display normal notification on Android when app is in foreground

In the README, it says:

When app is running in foreground
IOS will receive notification and android won't (better not to do anything in foreground for hybrid and send a seprate data message.)

This doesn't seems like it's the default android behavior and I would prefer the app not to "swallow" the notification.
Is it possible to disable this behavior ?

FCM sdk swallowed it, not me

Thanks for the feedback, gonna report that to the FCM sdk then and implement the local notif.

cool BTW, local notification for this repo is on the way. should release this week

Saw the report through our docs feedback button!  https://firebase.google.com/docs/cloud-messaging/android/receive has the receiver code - if the app is in the background, it'll automatically create a notification, otherwise you'll get the message to the onMessageReceived of that service.
If you're keen on the ability to disable it, you can request the feature here: https://firebase.google.com/support/contact/bugs-features/
Let me know if I'm completely misunderstanding the intention here though!

Response from the Firebase team for the feature request:

Hi Damien,
I'm very sorry for the delay here. Our internal discussion for this took quite some time.
Right now, this is just how "notification messages" are designed. They are received by the system tray when app is in background, then in the onMessageReceived() method when app is in foreground. Thus, for now, you'd need to manually build your own notification to display your message while app is in foreground. Reference here.
We can't share why it's designed like this, but we're continuing to work on the framework based on user feedback. So thanks for bringing this up!
Let me know if you have any additional concerns.
Regards,
Kat

@evollu @mdamien   is this still the same for latest release? that notification don't show up when app is open and local notification is required.

@mdamien now we are in 2019, approximately 2.5 years ahead of the time when you wrote you are working on framework. Have you guys did it so that notifications can be received in notification tray when app is in foreground.

@devabhi36 I'm a simple peasant library user like you, what you see pasted is just a reply I got from the Firebase team.


add-debug-and-trace-to-function

[ ] Bugfix (non-breaking change which fixes an issue)
[ x] New feature (non-breaking change which adds functionality)

Proposed changes
Adds node.deb and node.trace to function node so they can be called if needed.
Checklist

[ x] I have read the contribution guidelines
[ x] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x ] I have run grunt to verify the unit tests pass
[ x] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 87.755% when pulling 47ca4d17fe6f2fd958cc373a3143489d394b43a3 on add-debug-and-trace-to-function into 4ebb5d099e9f1589b142bd374645eb6ed7167f0b on master.


Roll Flutter Engine from 22f03ffdc290 to a9da7212eacf (4 revisions)

https://github.com/flutter/engine/compare/22f03ffdc290...a9da7212eacf
2023-08-15<EMAIL_ADDRESS>Roll Skia from c7c9e85a8657 to b07a6964a1cf (2 revisions) (flutter/engine#44733)
2023-08-15<EMAIL_ADDRESS>[Impeller] Make porter duff blended atlas calls faster. (flutter/engine#44059)
2023-08-15<EMAIL_ADDRESS>Roll Skia from 4beb6fd05f4f to c7c9e85a8657 (7 revisions) (flutter/engine#44729)
2023-08-15<EMAIL_ADDRESS>Roll Dart SDK from 19c87486f0a9 to b36934aae968 (1 revision) (flutter/engine#44728)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jsimmons@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


feat: double type interface

Good evening,
I've made a modification to the way double types are handled, in particular with the null.
I have an interface with several string | null or number | null fields.
and the swagger handled example and schema by returning :
field: {},

What's more, the schema didn't handle the nullable.
So I made the changes to default to the first type specified for the example, and to specify nullable correctly if one of the types is null.
I don't know if this is the best way to do it, so I'll wait for your feedback!

I'm planing on implementing the oneOf functionality based on the swagger specs.
If you're motivated, you can help me out. This would be cleaner imho

It's bound to be cleaner!
I'll do what I can with the time I have, have you started anything yet?

have you started anything yet?

yes, I added a comment 4 months ago 🤣


Is it possible to merge two pdfs?

So I have a scenario where I have to get a pdf from a third party API (Xero) and merge it on my own PDF, is that possible?
if yes, could you give me an example how?
and if no, do you know a package that does that?

@kempsteven Pass the resulting pdfs to combine-pdfs:
const { promises: fs } = require("fs");
const combinePdfs = require("combine-pdfs");

await fs.writeFile("output.pdf", await combinePdfs([pdf1, pdf2]));

so i guess there is no way to merge two pdfs on this package? Thanks for the answer will try that!

In short, this package converts HTML to PDF. Its purpose is not to merge them.


Package NLsolve.jl no longer exports not_in_place(...).

Whilst running UNSflow LAUTAT_example2.ipynb the following problem is encountered:
UndefVarError: not_in_place not defined
I believe this is because package NLsolve (https://github.com/JuliaNLSolvers/NLSolve.jl) no longer exports not_in_place as of commit #125 (Dec 25 2017). The version with not_in_place is 0.13 (Nov 2017). A modification (bug fix) to not_in_place required a new version 0.12.1 - whether this affects the expected behavior I don't know.
not_in_place is used in the following locations:
UNSflow.jl (1 hit)
	Line 13: export nlsolve, not_in_place
UNSflow\src\solvers.jl (34 hits)
	Line 46:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 155:         soln = nlsolve(not_in_place(kelv), -0.01*ones(nsurf))
	Line 225:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 570:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 1110:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 1150:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 1268:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 1300:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 1607:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 1651:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 1778:                 soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 1847:                         soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 1854:                         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 1966:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 1998:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 2087:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 2119:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 2215:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 2247:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 2342:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 2374:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 2469:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 2501:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 2604:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 2648:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 2744:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 2774:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 2889:         soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 2926:             soln = nlsolve(not_in_place(kelvkutta), [-0.01; 0.01])
	Line 3345:         soln = nlsolve(not_in_place(kelv), [-0.01*ones(surf.nspan); zeros(surf.nspan)])
	Line 3503:                 soln = nlsolve(not_in_place(kelv), [-0.01])
	Line 3728:                 soln = nlsolve(not_in_place(kelv), [-0.01])

A short term solution may be to update the REQUIRE file. Perhaps (whether version 0.12.1 breaks anything I don't know):
julia 0.5
Dierckx
ForwardDiff
PyPlot
NLsolve 0.11 0.14
IJulia

HJAB

The not_in_place function is still present in the updated NLsolve package, it's just not being exported. The best solution (IMO) is to add the following to  /.julia/v0.x/NLsolve/src/NLsolve.jl
export not_in_place