View isn't updating

getting an error in onDataSetChanged while updating my view

You should write what is the error, and a POC code. Nobody can help you if you just write that you have an error.
That's like saying "I have a problem, please help", and that's it.

ok I got it

so close the issue.

I fixed my issue thanx.


docs(Search): debounce search change handler

Added a debounce around the search change event as to not execute the callback on every key press. Dropped the timeout to be just under the debounce. This includes standard and category search examples.

Codecov Report

Merging #2650 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2650   +/-   ##
=======================================
  Coverage   99.74%   99.74%
=======================================
  Files         160      160
  Lines        2751     2751
=======================================
  Hits         2744     2744
  Misses          7        7

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 2bd1c3d...8705321. Read the comment docs.

Released in<EMAIL_ADDRESS>


modulesync 4.0.0 & puppet-lint updates

modulesync 3.0.0

Also fails on Debian 9 and all Ubuntu versions.

Not sure if you want to cherry-pick #3 into here? Alternately, I can try cherry-picking these commits into there.


Following redirects drops cookies.

Reported by @oz: https://github.com/httprb/http/issues/263#issuecomment-156260388
Upon following redirects, redirector should send cookies that were sent in response.

Note that this needs to be done carefully with respect to cookie domain scoping rules or it's a potential security vulnerability (e.g. in conjunction with an open redirect)

Good catch. Bright side is that Cookie::Jar has that feature already ;)) So I'll just need to make sure to use it :D

Hi, looking for some feedback in https://github.com/httprb/http/pull/613 that addresses this issue.

@Kache thank you, sorry for being a bit slow on this, will tackle your PR this weekend.

We have #705 complaining about the opposite problem now, that cookie scoping rules aren't implemented correctly.
Regardless, #613 seems to have implemented this, so closing.


Stop using the Bootstrap form-horizontal CSS class

As noted by @timabbott  in this comment:
We should try to excise is the Bootstrap form-horizontal CSS class. It's pretty likely we're overriding it ~everywhere that it applies.
We should do careful auditing for this next batch -- check pages in both normal and mobile sizes, and perhaps use the inspector to try to see what styles are applied by it.

Hello @alya would love to work on this but I need some clarifications, do you mean we should stop using bootstrap class names in our custom classes?

We should rewrite our CSS so that it does not depend on the Bootstrap form-horizontal CSS class, with the eventual goal of dropping our dependency on Bootstrap altogether.

Hello @alya, I would like to work on this issue but first I would like to know if the logic of my solution would be acceptable: I was thinking of creating a standardized class for form-horizontal in our CSS as, for example, .horizontal-pattern and then just call it in our HTML as <form class="horizontal-pattern">. Is it what you want or did I get it wrong?

@isabellalcarmo Please post your question in #frontend in the Zulip development community (no need to tag me). Thanks!

Yes, we no longer have form-horizontal class.

Closing this as completed in #24057.


Update validate_ib_nic.yml

Signed-off-by: Shubhangi-dell<EMAIL_ADDRESS>Issues Resolved by this Pull Request
Fixes #1513
Suggested Reviewers
@sujit-jadhav

@all-contributors please add @Shubhangi-dell for talk


Adds 'useIpFromEnv' parameter to registrator

Why does it require to use IP from environment variable?
The registrator will register the docker containerized services with the service discovery tools by grabbing the container IP address and exposed ports from the docker inspect information (low-level information of docker object).

The docker containers which are managed by the kubernetes will not show up the container IP address under docker inspect information.  In such environments, the docker containers network information will be offloaded to another container who's reference will be given under HostConfig.NetworkMode value.

In GKE (Google Container Engine), the docker containers will not show up container IP address either in the container inspect information or in the offloaded container inspect information.
If the registrator is provided with the capability to use IP address from the environment variables, then it helps to pass the service IP address (in the consul perspective) whether the pod or node IP address as an environment variable based on the requirement.
How to use the flag useIpFromEnv?
This example information is given based on the consul service discovery perspective.
The POD_IP is the environment variable.
Run the registrator service as shown:
$ registrator -useIpFromEnv=POD_IP consul://<consul-agent-ip>:8500
Add the following k8s spec configuration to the service to be registered:
...
    env:
    - name: POD_IP
      valueFrom:
        fieldRef:
          fieldPath: status.podIP
...

When the registrator run with the flag -useIpFromEnv=POD_IP, the registrator uses this IP address (the value of POD_IP env) as container IP address and registers with consul agent service.

I never liked that we never switched to purely env based configuration. I would prefer we use environment. As such, I think this could be simplified by just having an environment variable that, if set, would be used for IP. No need for argument to specify what environment variable. Also, thanks for adding to the docs.

do we have an image for this pull request?


setMemorySlot sets memoryslot value to "undefined" if passsed an undefined value

https://github.com/naxxfish/pfinterface/issues/1
This needs to be fixed in this package.

Added some logic so that if you pass the setMemorySlot function an undefined msValue or msSlotName, it won't try to complete it and will fire the callback with an error.


Pact lang api cookbook

Can you make a PR for the Kadena docs

@Randynamic Here's the PR: https://github.com/kadena-community/docs/pull/21


@craigspaeth => add more specific class for bid+registration button

resolves #562
I worked with @sweir27 on this- there are 2 buttons in this force app with the same elan class, but only one actually should be affected by the auction closed event. the other is being triggered by live auctions which trigger the same clock:is-over event when if they are live. this changes the event to only effect the artwork bid form, not the registration form.

Easy peasy, thank you!


Support for PW getBy locators

Since PlayWright Version 1.27 there's PlayWrights own take on writing locators, as documented prominently here. These, you can then filter and more.
These we could implement using something like:
Click    pw=getByRole('listitem') -- hasNot: getByText('Product 2')

The -- would be treated a bit like there is custom >> for chaining, but -- would be for filtering.
filtering can consist of: has or hasNot and is followed by either text or any other locator.

While I like the idea to implement the getBy Locators and filters , I worry that the readability of our Keywords will suffer too much by using the syntax above (e.g. using -- as an equivalent for filter).
One huge advantage of Robot Framework Keywords/Libraries for me was always, that they were easier to read and understand than the equivalent in Playwright, Selenium, etc.
I don't have a good idea how to approach this right now, I just wanted to share my opinion.

Can you @manykarim elaborate on how this feature will let readibility suffer more than using >> as a way of chaining locators?
Locators in general can look terrible, so the way playwright attempted to make an alternative to xpath or css is only to make it more readible I think. Therefore adding the feature in a way that is as close to the original implementation as I could come up with will eventually add to readibility of locators instead.
But writing neat locators will probably always remain a form of art, as writing them badly is so easy

you can easily find out which selector is similar to what expression by just printing the Locator object in Playwright Python or Playwright Node
Like this:
abs = page.locator("div").filter(has_text=re.compile(r"^ABS 990€$")).get_by_role("checkbox")
print(abs)

prints:
<Locator frame=<Frame name= url='xxxxxxxx'> selector='div >> internal:has-text=/^ABS 990€$/ >> internal:role=checkbox'>

Writing docs is good idea, let’s at least do that.
Also it would be interesting to see would it be possible write a keyword, for experimenting, that would eat PW locator, like page.locator("div").filter(has_text=re.compile(r"^ABS 990€$")).get_by_role("checkbox") Then it would return <Locator frame=<Frame name= url='xxxxxxxx'> selector='div >> internal:has-text=/^ABS 990€$/ >> internal:role=checkbox'> but as a valid css locator.

Will be fixed by #2920

Is the internal:has-text keyword documented anywhere?
(Just wondering. I've already figured out how to use it 🙂)


fix: explicit HTTPRequest.resourceType type defs

Fixes #6854.

@jschfflr please take another look - I discovered the Lowercase util so we don't have to duplicate the list :)


Documentation

Currently there is none.

The documentation should include:

README.md

Description of the overall concept
Installation and usage
Design and Security considerations

In-function documentation or doxygen style documentation (including links to keycloak documentation or source code)

I've added some documentation. Is it enough? What is missing?


Balance discrepancy of 719 Mina for public key B62qqa9g4CFfkSuX2j22S52z6UfcDcS9tMTgQrFKZ21v7GrEP6Zu5T3c

At block height 359604, there is a balance discrepancy of 719 Mina for the specified public key. This discrepancy corresponds to the coinbase reward minus the account creation fee.
I suspect this discrepancy is visible at block height 2.

The Indexer's ledger has the account balance at:<PHONE_NUMBER>02957 where as the mainnet state dump has it at<PHONE_NUMBER>02957. We have 719 more Mina.

Still a problem?

I suspect it is but we won't know for sure until we have the ledger export at height 359604 from the prod indexer.

This is still a problem. I asked Gareth and Jon Washburn for some help.


Add copy next to the account address

This is in original Rivet:

Hey @ivpavici can I get assigned to this? Also would love to space hard as an engineer but don't know the right steps.


Please help me

Hello i have a problem i want to install the intel hd4000 patcher but it doesn't allow the installation because csrutil authenticated-root is set to enable but when i enter csrutil status in the terminal it shows me that everything is set to disable. When I try to disable Gatekeeper via hackingtool I get the error: mount failed with 66. Then I found a program that means bigsurmountrw because I can get into the system folder but the patcher still doesn't work because SIP is activated. I am already despairing.

Change csr-active-config to EF0F0000 as instructed on the HD4000 Patcher Repo.

I changed csr-active-config to EF0F0000. I even disabled csrutil again even in recovery mode in the terminal and type csrutil authenticated-root disable but still doesn't work

Enter  sudo spctl --master-disable in Terminal to disbale gatekeeper.
HD4000 patcheris only compatible wirh macOS Monterey , no Ventura support. Report any issues related to HD 4000 to the author of the patche.  The last time I used it it worked without issues.

No. NONE of this has doen in Recovery.

Hello, sorry for contacting you again, I have a problem that Monterey only starts when the USB stick is plugged in. I have already copied the EFI folder from the USB stick and pasted it into the system EFI, but it does not start from the hard disk and shows in the BIOS He doesn't show me any more entries in the boot menu. Only when the USB stick is plugged in again can I change back in the bios from where to start. I do not know how to continue

I tested this EFI with macOS Monterey 12.5 again yesterday! It works!
Intel HD4000 Patcher = wroking!
Drives are detected in Bootpicker!
Installing macOS 12 on this machine is not recommended (as mentioned in the Readme)! Install macOS Catalina
Try some troubleshooting on your own!
Look for a different EFI!


feat(ci): make flamegraph generation nexmark-query configurable [WIP}

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Closes #9420
Pulled queries from kube-bench using this: https://github.com/risingwavelabs/kube-bench/pull/132.

Checklist For Contributors

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[x] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[x] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

Nexmark flamegraphs generated for nexmark-q0 and nexmark-q17, since they are tagged in this PR:


Yaxis crosshair tooltips of grouped charts unexpected behaviour, bug

Bug report
Codepen
CodePen example of issue:
https://codepen.io/benben_ik/pen/WNjmWOb
Explanation

What is the behavior you expect?
When crosshairs are enabled for grouped charts: Only the Y-axis tooltip of the chart a user is hovering is shown on a grouped chart. I do not expect to see Y-axis tooltips of both the chart I'm hovering and Y-axis tooltips of other grouped charts that do not belong to the chart I'm hovering.

What is happening instead?
Instructions (see codepen): Create two or more grouped charts and enable crosshairs and Y-Axis tooltips on each chart. This results in all Y-Axis showing all Y-Axis tooltips of all grouped charts on each Y-Axis when hovering one of the charts. This overlap makes Y-Axis tooltips unreadable and confusing. It seems that the Y-axis crosshair keeps on drawing outside its frame.
Example:

What error message are you getting?
There is no error message, there is just unexpected behaviour.

Thank you for your time.

I tried out eCharts, https://echarts.apache.org/en/index.html . Grouped charts, hurray!


Danganronpa V3: Killing Harmony Demo [PCSE01105]

Game summary

Game name: Danganronpa V3: Killing Harmony Demo
Game serial: PCSE01105

Vita3K summary

Version: 0.01
Build number: 404
Commit hash: https://github.com/vita3k/vita3k/commit/d4cd595

Test environment summary

Tested by: waterflame
OS: Windows 7
CPU: Intel i7-2600
GPU: AMD Radeon HD 6700 Series
RAM: 8 GB

Issues
Crashes after Unimplemented sceNgsSystemRelease import called.. No noticeable regressions against 328-https://github.com/vita3k/vita3k/commit/6525c1c. No use of addcont0 like it's full game version.
Screenshots

Log
vita3k-404.log
Recommended labels

Crash

Vita3K summary

Version: v0.1
Build number: 571-70599d3
Commit hash:  https://github.com/Vita3K/Vita3K/commit/70599d3

Danganronpa3_Demo.log

Vita3K summary

Version: v0.1
Build number: 685-2eee7b1
Commit hash: https://github.com/vita3k/vita3k/commit/2eee7b1

#Issues
Loading ngs seems to allow it to get further, or... at least it gets past the ngs block...
#Log
Danganronpa3_Trial.log
vita3k_DR3_ngs.log

Vita3K summary

Version: v0.1
Build number: 984
Commit hash: fb0798e of pull/456 (I don't want to append every issue to the PR)

Danganronpa V3  Killing Harmony Demo version - [PCSE01105].log
Issue
Unicorn error 0x7 (used to crash without unicorn error)

Vita3K summary

Version: v0.1
Build number: 1315
Commit hash: https://github.com/vita3k/vita3k/commit/5b070aac

Danganronpa3_Trial.log

Vita3K summary

Version: v0.1.5
Build number: 2630
Commit hash: https://github.com/vita3k/vita3k/commit/18dacae2
CPU backend: Dynamic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:

Game goes ingame- with good performance but major graphical issues and crashes soon after going ingame

Screenshots:

Log
vita3k.log
Recommended labels

Ingame-
Graphical bug

as of https://github.com/Vita3K/Vita3K/commit/c541d8eb45e445d8741e5344d2cb3f9017607212, the game doesn't crash anymore, but there's a regression that the text doesn't show.
Screenshots

Log
vita3k.log
Recommended Labels

Ingame +
text missing


How to set Addin Icon32

I am trying out Addin Maker for the first time and cannot seem to figure out how to set the addin icon.
normally I would do this:
<Addin ...>
    <Header>
        <Icon32>...</Icon32>
    </Header>
</Addin>

How do I do this with Addin Maker?

I don't think it's currently possible to do this with Mono.Addins attributes /ccc @slluis.

This should do it:
[assembly:AddinProperty ("Icon32","TheIcon.png")]

Or you can add an add-in manifest resource with that xml (but using ExtensionModel element as root instead of Addin).


docker run causes env: ‘python’: No such file or directory

With the latest presto-server-336 tarball, on doing docker run, getting

[[ ! -d /usr/lib/presto/etc ]]
[[ -d /etc/presto ]]
ln -s /usr/lib/presto/default/etc /usr/lib/presto/etc
set +e
grep -s -q node.id /usr/lib/presto/etc/node.properties
NODE_ID_EXISTS=1
set -e
NODE_ID=
[[ 1 != 0 ]]
NODE_ID=-Dnode.id=d30d44f455d9
exec /usr/lib/presto/bin/launcher run -Dnode.id=d30d44f455d9
env: ‘python’: No such file or directory

The Dockerfile does not include RUN command for python installation and setting environment variable.

@MohammedRashidKP The docker image in https://hub.docker.com/r/prestosql/presto contains Python. Could you share the steps to reproduce?

Right. But the Docker file at 'presto/docker/Dockerfile' does not contain the step to install python. I had to customise the Dockerfile to have base image rhel-ubi instead of the CentOs that comes in the file.
And I believe CentOs doesn't come with python and we need to install it, like jdk-11 is installed.
I added the following lines and it worked:
yum install -y python36 python3-pip &&
ln -s /usr/bin/python3.6 /usr/local/sbin/python &&
python -V && \

Centos docker image comes with Python preinstalled.
Of course, you can create your own Dockerfile, just the way you did, changing the base image or the installed components.
It's reasonable to expect that such a change may imply some other changes. For example, changing the base image may require adding more packages, using different names of packages, or changing the way packages are installed.
I happy you were succesful creating your own image!
btw please make sure you're on the #dev channel of our slack.


Set utimes from original files

This fixes issue #78

Onlyvinyl and vinyl-fs updates are meaningful and all the other chnages are not needed. https://nodesource.com/blog/semver-tilde-and-caret

And squash the commits into a single commit.

Done, commits cleaned and squashed.

Thanks. We'll publish a new version soon.

https://github.com/kevva/download/commit/cbe4c6cba19a600bffb0ac010db1df78263cdb05

Thank you :-)


Exclude slack notification from PR builds

Right now, we use the branch name to determine if a message should be sent on failure. Based on the docs branch will be the base when the type of build is a PR. So this limits the build type to push, excluding PR builds from triggering the notifications.

Thanks for the pull request @ggetz!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.


Missing requirments.txt

Describe the bug
the frontend directory is missing the requirements.txt
To Reproduce
Steps to reproduce the behavior:

run start-windows.bat

Expected behavior
Runs without the error missing requirements.txt
Log files
If possible, add log files from /var/log/mycroft/ to help explain your problem.
You may also include screenshots, however screenshots of log files are often difficult to read and parse.
If you are running Mycroft, the Support Skill helps to automate gathering this information. Simply say "Create a support ticket" and the Skill will put together a support package and email it to you.
Environment (please complete the following information):

Device type: [e.g. Raspberry Pi, Mark 1, desktop]
OS: [e.g. Ubuntu, Picroft]
Mycroft-core version: [e.g. 20.08]
Other versions: [e.g. Adapt v0.3.7]

Additional context
Add any other context about the problem here.
Please think carefully about whether you have modified anything in Mycroft's code or configuration files.  If so, can you reproduce this on a clean installation of Mycroft? Many "bugs" turn out to be non-standard configuration errors.

Hi there, thanks for logging some bug reports!
If I'm not mistaken, the backend is a Python Flask app which is why it has a requirements.txt file. Where as the frontend is  a Javascript React app so its dependencies are defined in the package.json file.
Let me know if I've missed something.

Interesting - I just saw that start-windows.bat tries to install python dependencies for the frontend - that will never work...
I don't have a Windows machine to test with but can you delete line 81 and try again?


Add option for separating out technical replicates

PCR duplicate should happen on each technical replicate
The the bams should be merged with a RG assignment atatched

we should allow the user to specify custom read group meta data

@nboley: Just to clarify: a technical replicate here is a unique sample/lane combination? So a library sequenced on multiple lanes has a technical replicate for each sample in each lane?

I think it's dangerous to do excessive inference from read names etc.
Platforms or conventions often change and then all of this breaks down.
Best to have users be explicit about what tech reps, lanes, bioreps and
adapter sequences.
On May 24, 2016 4:57 PM, "Chris Probert"<EMAIL_ADDRESS>wrote:

@nboley https://github.com/nboley: Just to clarify: a technical
replicate here is a unique sample/lane combination? So a library sequenced
on multiple lanes has a technical replicate for each sample in each lane?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/kundajelab/bds_atac/issues/17#issuecomment-221436863

@akundaje Yeah, I think some type of sample sheet to specify parameters for all replicates would be a good solution, perhaps combined with automated sanity checking (e.g. if the user-specified adapter sequence is clearly wrong).

Agreed
On May 24, 2016 5:07 PM, "Chris Probert"<EMAIL_ADDRESS>wrote:

@akundaje https://github.com/akundaje Yeah, I think some type of sample
sheet to specify parameters for all replicates would be a good solution,
perhaps combined with automated sanity checking (e.g. if the user-specified
adapter sequence is clearly wrong).
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/kundajelab/bds_atac/issues/17#issuecomment-221438252


RFC: techmap to recognise wires named "TECHMAP_REPLACE."

... and to substitute the original cell name and alias them to the auto-generated wire.
Useful for creating wires that have a predictable name (not $techmapNNN...) that can be recovered by other passes.
Could be used to create (* hierconn *) signals, for example, too.

hmm.. quickly summarizing the current behavior, using the following top-level (from your test case):
module top(input i, output o);
sub s0(i, o);
endmodule

The following will create a foobar instance called s0. Everything else in that module is mapped using the prefix $techmap%d. The reason for not using the prefix s0 is so that when the new s0 is techmapped to something else, and that something else would create s0.asdf that would conflict with s0.asdf frtom sub:
module sub(input i, output o, input j);
foobar _TECHMAP_REPLACE_(i, o, j);
wire asdf = i ;
endmodule
EOT

And the following would create s0.xyz and s0.asdf, as no _TECHMAP_REPLACE_ cell is present, and therefore no s0 cell exists after techmap, so no risk of name collisions down the road when a s0 is mapped.
module sub(input i, output o, input j);
foobar xyz (i, o, j);
wire asdf = i ;
endmodule
EOT

What is the use-case for this PR where that latter version without _TECHMAP_REPLACE_ is insufficient? I think it makes sense to avoid creating entities with names like s0 and s0.foo that would potentially result in name collisions when s0 is techmapped to something else.

What is the use-case for this PR where that latter version without TECHMAP_REPLACE is insufficient? I think it makes sense to avoid creating entities with names like s0 and s0.foo that would potentially result in name collisions when s0 is techmapped to something else.

The use case is that I need to use _TECHMAP_REPLACE_. Specifically, I have a techmap module that wraps (using -max_iter 1 to prevent infinite recursion) an existing cell with existing cell + new cell, while preserving everything about the existing cell:
https://github.com/YosysHQ/yosys/blob/1b96d29174d7c56a14031bc117a7da5fa5192c81/techlibs/xilinx/abc_map.v#L38-L54
In this manner, I really want to have a predictable name for (specific) wires inside, as then I can do a lookup in a downstream pass to get that generated wire, based only on information from the cell. With the stringf($techmap%d, autoidx++) prefix, I don't believe I can.
An alternative would be to preserve the value of autoidx using a mechanism like:
https://github.com/YosysHQ/yosys/blob/d963e8c2c6207ad98d48dc528922ad58c030173f/passes/cmds/scc.cc#L248-L251

Okay. SGTM.
Maybe it would make sense to add a similar functionality for cells named _TECHMAP_REPLACE_.* here:
https://github.com/YosysHQ/yosys/blob/369652d4b99181e2f7b875b6c458c1a5a3b0381e/passes/techmap/techmap.cc#L380-L388

The alternative is that we only use the $techmap%d prefix /iff/ there is going to be a name collision (i.e. do so lazily). Are there any problems with that? I think we should at least try and behave the same whether or not there is a _TECHMAP_REPLACE_ present...

We have recently added functionality to techmap to deal with name collisions for wires. We would just need to do the same for cells, and then we can get rid of the $techmap%d prefix mechanism.


Make other langs

Add a rule for making pdf files for languages other than english. Also closes #55.

Great documentation too, thank you!


The setting [color] doesn't exist.

hi,
no matter what setting i create, i get the error setting doesn't exist.
Laravel Version: 8.61.0
PHP 8
I can see the settings in table, they are enabled. Everything looks ok. Cache is disabled.
this is my user model
...
use DarkGhostHunter\Laraconfig\HasConfig;

class User extends Authenticatable implements LdapAuthenticatable
{
    use AuthenticatesWithLdap;
    use HasFactory, Notifiable;
    use HasLdapUser;
    use HasRoles;
    use HasConfig;
...

this is the settings/users.php file
<?php

use DarkGhostHunter\Laraconfig\Facades\Setting;

Setting::name('color')->string()->default('black');

This is how i am trying to get it. it always comes blank, even the value exists in database.
$user = User::find(1);
echo "Your color is: {$user->settings->get('color')}.";

if i try to set it i get setting doesn't exists, no matter how i try to set it.
$user = User::find(1);
$setting = $user->settings->get('color');

$setting->value = 'red';
$setting->save();

or

$user->settings->set([
    'color' => 'red'
]);

i have investigated more and found out that in database the settable_type is getting saved without slashes.
like: AppModelsUser
it should be App\Models\User
Users already exists in database and the setting is initialized slashes are missing.

Same here. Also see issue #29


defer Eureka + Health linking until after Injector creation

NetflixOSS » runtime-health » runtime-health-pull-requests #5 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » runtime-health » runtime-health-pull-requests #6 FAILURE
Looks like there's a problem with this pull request


[Enhancements] Minor security enhancements

Encrypt SNS topics at rest
Encrypt CWL Kinesis streams at rest
Optionally enable IMDSv2 on all instances
set termination protection on perimeter firewalls
fix json schema package and dev script sdk versions

Encrypt SNS topics at rest - in v1.5.1
Encrypt CWL Kinesis streams at rest - in v1.5.2
Optionally enable IMDSv2 on rsyslog and RDGW instances - in v1.5.1
Optionally enable IMDSv2 on firewall instances - in v1.5.2
set termination protection on perimeter firewalls - in v1.5.1
fix json schema package and dev script sdk versions - in v1.5.1


Render previews as doc examples

We have the ability now to render and assert against previews in tests, and in general we're thinking quite a bit about reusing previews in more places to avoid duplication. It would be really cool to be able to use previews as examples when generating docs as well.

We have the opposite of what you're suggesting right now where we generate previews from the YARD doc examples in the component files, then run tests against those. 😆
If you run bundle exec rake docs:preview, it generates component previews by grabbing the code examples in our YARD docs.   (docs.rake preview task)
We then have a browser test that runs accessibility checks against by visiting the generated preview URL. (doc_examples_axe_test.rb).


Add minimal isitgr install instructions to readthedocs

It would be useful to have some pointers here:
https://ccl.readthedocs.io/en/latest/source/installation.html
as we do for FAST-PT.

Agreed! I added some instructions on how to install isitgr via pip or conda in:
https://github.com/LSSTDESC/CCL/pull/778.
@elisachisari, let me know what you think.

@mishakb @elisachisari has this been resolved now? can we close this issue?

Great


Swift4.2 Hasher support

swift-protobuf supports Swift 4.2 in the latest master version. but Hasher does not seem to support yet.
swift-protobuf generates hashValue with its own logic, but I think that it is better to generate hashValue using Hasher.
This is valid for instances that generate hashValue with multiple properties.
Is there a clear reason not to use Hasher?
If there is no problem,  I will submit pull request.
Reference
The stdlib has switched to a randomly seeded, high-quality hash function for all hashing.

Proposal: SE-0206

We just haven't gotten to it (or opened an issue for it, I thought we had).  I have a PR out for CaseIterable, and I'll likely work on the hashing changes next (they are a little trickier).  The good news is 4.2 isn't GM yet, so we really aren't behind from that  POV.


Documentation on Firmware

Hi,
The documentation could do with a bit more of a hint/warning about the firmware that people have on their device, it absolutely requires the official firmware from ESP to work properly (I couldn't get the http server working, so spent ages figuring out how to flash the ESP firmware on it)
After flashing, the HTTP server works correctly.    While bits may work fine with the AI-THINKER firmware not all of it will as I found out!

http://majerle.eu/documentation/esp_at/html/page_faq.html

I think it could do with being repeated here:
http://majerle.eu/documentation/esp_at/html/page_update_at_software.html#sect_update_process_preparations
and more importantly in the readme.md
I missed the FAQ...I suspect others might too.

Now I added new event, which is triggered if ESP AT software version is less than minimum supported by library. Check examples, main.c files.

Cool, always best to make it as explicit as possible what's required!


Turning off HTTPs in API-Umbrella

Hi,
Just curious with this doc on HTTPs configs: http://api-umbrella.readthedocs.io/en/latest/server/https-config.html and this old issue chain: https://github.com/NREL/api-umbrella/issues/112
It was mentioned that for:

SSL Termination: If you’re placing API Umbrella behind a load balancer in a multi-server setup, you can handle the SSL termination with that external load balancer

I was wondering if there was a way to turn off HTTPs/SSL checking in the .yml file. I have a similar setup with a load balancer in front of API-Umbrella where I'd like to handle SSL termination there. But it is unclear to the details on how you're doing so there when the default API-Umbrella has its own self signed certificates.
Thanks

Sorry for the delay. We've added some additional documentation about SSL termination in a couple places:
https://api-umbrella.readthedocs.io/en/latest/server/https-config.html
https://api-umbrella.readthedocs.io/en/latest/server/listen-ports.html
Generally speaking, you should be able to perform SSL termination, and things should just work as long as your SSL terminator sends back X-Fowarded-Proto and X-Forwarded-Port headers. I know Amazon ELBs do this by default, and we use ELBs for SSL termination in some of our uses. But let us know if that doesn't answer your question, or you're still having any issues.


luci on nginx

Is it possible to make luci work on nginx?
luci.on.nginx page is missing on OpenWRT website, and a few configs I have found don't work (OpenWRT 14.07) and return 404 Not Found for <IP_ADDRESS>/cgi-bin/luci
Configs I have tried:
    location /cgi-bin/luci {
        fastcgi_split_path_info ^(/cgi-bin/luci)(/.+)$;
        fastcgi_pass  unix:/var/run/luafcgid.sock;
        fastcgi_param PATH_INFO $fastcgi_path_info;
        fastcgi_param SCRIPT_FILENAME /usr/lib/lua/luci/sgi/fcgi.lua;
        include       fastcgi_params;
        client_max_body_size 16M;
    }

     location /cgi-bin/luci {
            fastcgi_split_path_info ^(/cgi-bin/luci)(/.+)$;
            fastcgi_pass  unix:/tmp/fcgi-cgi.sock;
            fastcgi_param PATH_INFO $fastcgi_path_info;
            set $script_name '/cgi-bin/luci';
            fastcgi_param SCRIPT_FILENAME $document_root$script_name;
            include fastcgi_params;
            add_header Cache-Control no-store;
    }

do you solve this problem? i also want to run luci on nginx

Also keen on this, having the same problem.

I'd suggest you prefer using uhttpd on a different port than nginx and keep the admin UI separate from you nginx instance.  There is no advantage to nginx as your UI server.

I think we really need to set luci as default for serving luci as uhttpd take like 5-10 seconds to load any page(100% CPU in top for just one user!),
I think first we need to solve this
https://github.com/openwrt/packages/issues/2408
so nginx can serve old plain cgi scripts.

Any progress about this?


Camera2Video crash when recording with Back camera in Android 11

Tried with Android 11 and Pixel 3a emulator. Every time I try to record a video with back camera the app will crash. Rest of Android versions work as expected.
How to reproduce:

Open the app
Select any back camera resolution
Record a video

Output:
2020-09-14 12:43:40.172 13317-13370/com.android.example.camera2.video E/MediaRecorder: stop failed: -1007 2020-09-14 12:43:40.235 13317-13370/com.android.example.camera2.video E/AndroidRuntime: FATAL EXCEPTION: DefaultDispatcher-worker-1     Process: com.android.example.camera2.video, PID: 13317     java.lang.RuntimeException: stop failed.         at android.media.MediaRecorder.stop(Native Method)         at com.example.android.camera2.video.fragments.CameraFragment$initializeCamera$1$1$2.invokeSuspend(CameraFragment.kt:297)         at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)         at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:56)         at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:571)         at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:738)         at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:678)         at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:665)

Just noticed that if I change in emulator back camera preferences from VirtualScene to Emulated it just works. Not sure if it's a code problem or an emulator bug.

I got the same issue with both android 9 for real device and android 11 for the emulator.
@buntupana How you change in emulator back camera preferences from VirtualScene to Emulated?

@hho114  sorry for the delay
AVD Manager -> Edit (Desired emulator) -> Show Advanced Settings -> Camera secction

Emulator had some issues, which should be fixed now. Please update Android Studio and the emulator to the latest version.


height fix issue

height you have given is 30px for "Li" and newssection class as well, so i have changes that, so after change the height, scrolling have some jerks, and its overlapping, please check this issue once.

Below css i have changed, according to my space:
.newsticker .newsticker-list .newsticker-item {
height: 30px;
/* overflow: hidden; /
/ white-space: nowrap; */
}
.newsticker{
width: 130px;
height:300px;
}

Hi srinivashappy,
Thank you for opening this issue, and I fixed it this morning.
I am not sure I fully understand what your meaning, so check the latest release.
Let me know if you have any feedback.
Cythilya

i tried to change of parent (newsticker) height and width, so i am able to see only one item(li html) at a time, i am not able to see all the lists that are scrolling.

am also changed the properties of li(height: 30px;
/* overflow: hidden; /
/ white-space: nowrap; */)

please look in to that above image and according check the issue.
Thanks,
Sreenivas


Nuemark2.0  *** Bold Italic *** & HR

Doing a quick code review of your dev branch, I am confused if the Blocks or Inline is scanned first?  If Blocks, then it seems this code:
 *** Bold Italic ***

Would render as an
 since you are just scanning first few characters.?
Also be aware that CommonMark is VERY flexible for
.  I personally don't think you need to cover every edge case, but it could help to document "Only the common Markdown/CommonMark rules are covered."

@tipiirai
[![](someimagelink)](somelink) in nuemark2 breaks the nue server and generator.
I used [![](/img/blog-build.png)](/) on a random page in the docs, and the server just hangs indefinitely
I know, that this is possible using markdown extensions ([! /img/blog-build.png href="/"]), but the default Markdown should still work imo.

 *** Bold Italic *** renders to nothing in my test.
I also tested *** Bold Italic *** and it generates
* Bold Italic*
 but I think it should generate
*** Bold Italic ***

@tipiirai just so you know, dev branch fails to build the docs (when there's no current .dist) since table changes:
227 |
228 |   const html = rows.map((row, i) => {
229 |     const is_head = head && i == 0
230 |     const is_foot = table.foot && i > 1 && i == rows.length - 1
231 |
232 |     const cells = row.map(td => elem(is_head || is_foot ? 'th' : 'td', renderInline(td, md_opts)))
                            ^
TypeError: row.map is not a function. (In 'row.map((td) => elem(is_head || is_foot ? "th" : "td", renderInline(td, md_opts)))', 'row.map' is undefined)
      at .../nue/packages/nuemark/src/render-tag.js:232:23
      at map (1:11)
      at renderTable (.../nue/packages/nuemark/src/render-tag.js:228:21)
      at table (.../nue/packages/nuemark/src/render-tag.js:98:18)
      at map (1:11)
      at renderBlocks (.../nue/packages/nuemark/src/render-blocks.js:18:17)
      at renderPage (.../nue/packages/nuekit/src/layout/page.js:128:23)
      at .../nue/packages/nuekit/src/nuekit.js:125:22

@tomByrer @nobkd fixed all issues mentioned on this article

Thank youu!
I'll check later today, if I find more unexpected results :)

InputExpected (Commonmark Spec Implementation)Nuemark 2

*test**

**test***

***test****

- - -

****test***

***test**

**test*

test*

test*

test*

*test

*test

*test

test*

test*

test*

*test

***test**

**test*

The first half (before hr) is correct, the second half is wrong.

Oh, i just let the cmark tests run through without expect once, and the following Markdown tests let nuemark hang (this completely ignores all the other failing test):
Hanging tests by test number: [573, 576, 577, 582, 583, 584, 585, 586, 587, 588, 589, 590, 591, 629]

show tests: category; test number; md
Images; 573
![foo *bar*]

[foo *bar*]: train.jpg "train & tracks"

Images; 576
![foo *bar*][]

[foo *bar*]: train.jpg "train & tracks"

Images; 577
![foo *bar*][foobar

[FOOBAR]: train.jpg "train & tracks"

Images; 582
![foo][bar]

[bar]: /url

Images; 583
![foo][bar]

[BAR]: /url

Images; 584
![foo][]

[foo]: /url "title"

Images; 585
![*foo* bar][]

[*foo* bar]: /url "title"

Images; 586
![Foo][]

[foo]: /url "title"

Images; 587
![foo]
[]

[foo]: /url "title"

Images; 588
![foo]

[foo]: /url "title"

Images; 589
![*foo* bar]

[*foo* bar]: /url "title"

Images; 590
![[foo]]

[[foo]]: /url "title"

Images; 591
![Foo]

[foo]: /url "title"

Raw HTML; 629
foo <![CDATA[>&<]]>

No more loops with unclosed image tags such as ![foo *bar*]

Is image reflinks like ![foo][bar] supported in Markdown?

btw: Nuemark will not support raw HTML, because that violates the separation of concerns principle

Are image reflinks like ![foo][bar] supported in Markdown?

The commonmark reference implementation does: https://spec.commonmark.org/dingus/?text=![img][tag]

[tag]%3A %2Fimg.png
#result
(PS: did you do the strike through implementation with one tilde (~) or with two? ~strike~ oder ~~strike~~?)

Found more problems:
1. **bold** not bold **bold**
2. **test**:

expected:

bold not bold bold

test:

reality:

bold** not bold **bold

**test**:

Nuemark also doesn't support line breaks with two or more spaces at the end of a line or a backslash at the end of a line:
foo
bar

foo\
bar

expected:

foo

bar

foo

bar

Escaping seems to not work properly: E.g. https://nuejs.org/docs/content-syntax.html#code-blocks

or

I also tried using code blocks with more than three backticks to wrap the md code block, but that didn't work. I tried this:
````md
```md
// here is a CSS code block
:root {
  --base-100: #f3f4f6;
  --base-200: #e5e7eb;
  --base-300: #d1d5db;
  --base-400: #6b7280;
}
```
````

But it results in this:

:root { }

@tipiirai I am wondering what was the original decision to write own parser in contrast to extending/adopting an existing one?
It seems to me, that micromark could be a great foundation that supports CommonMark out of the box hense does not have a behaviour mentioned in https://github.com/nuejs/nue/issues/379#issuecomment-2426528771

Are image reflinks like ![foo][bar] supported in Markdown?

Yes, image reflinks are supported by CommonMark. You may test it directly on GitHub.

Ooooh, thank you so much @viktor-yakubiv I didn't know CommonMark behaved differently in that case.
(But nuemark creates paragraphs in both cases. I don't know if it's better if it does it in both cases or not, but at least you can style all lists the same...)

what was the original decision to write own parser in contrast to extending/adopting an existing one?

It was explained here : https://nuejs.org/blog/nue-release-candidate/#new-markdown-parser
I don't really know if the same applies to micromark

I don't really know if the same applies to micromark

Thank you for the provided extra information. No, none of the listed problems applies to micromark (remark, unified project). Moreover, the unified project was started to solve

I have discovered from the https://github.com/nuejs/nue/issues/414 that Nuemark renders lists differently from the standard

I'm here for the same reason 😅

Users write other Markdown much more than Nue's Markdown. This means that users would prefer Nuemark work in the same way as CommonMark (or GitHub Flavoured Markdown).
— (paraphrased) Jakob's Law

Comming from https://github.com/nuejs/nue/issues/414#issuecomment-2506687179 and continuing the previous discussion, a few remarks on that point below.

Markdown by definition is a superset of HTML, i. e. it empowers the users write a simpler form of HTML but it still is an HTML code. Forbidding HTML in Markdown can be acceptable due to security reasons but it still violates this original intention.
Although, I support Nue's team intetion to invent a simple yet more powerful syntax than CommonMark currently is (provide blocks etc.). There is a relevant directive proposal and mast-util-directive.
However, while forbidding HTML seems to be a technically practical decision and a good enforcment for the user to write nicer looking markup, it takes the power from the user disabling them doing things they might want in some exceptional cases. And it's important to remember: "you are not the user". As an example, there might be a hightly technical post explaining some bits of HTML and a need to provide live examples in place, or some small bits where providing ARIA would be necessary. These are out of my mind but I am sure there will be very unpredicted real use cases.

Markdown in 2024 is well specified. CommonMark is the most popular and well adopted standard of Markdown syntax. The website explains why it exists and provides a lint to versioned specification.
There is another popular standard that we all use — GitHub Flafoured Markdown (GFM) that is an extension of CommonMark and has own specification.
There is also Multimarkdown that has own specification and toolkit but is less adopted. It aims to provide more powerful markup toolset: footnotes, citations, abreviations etc. Multimarkdown still supports all basic Markdown features (I am not sure about HTML) but it's diverges a lot and therefore provides own file extension .mmd.
There might be others that can be found in the Interner and Wikipedia but I didn't dig into the topic further.

Coming from the point 2, Nuemark can head in one of the following directions:

diverge from Markdown, provide a (significantly) different markup, an independent toolkit (parser, compiler, syntax highlighters, editor extensions) for it and use unassociated with Markdown file extension like .nm;
adopt the Markdown (Commonmark) fully, either extend an existing toolkit or provide an independent one, and keep using common file extension .md providing and optional one .nmd if a proper distinguishing is necessary.

In any of those cases, Nuemark requires a proper specification, in my opinion. Currently, it is vaguely defined.

I personally think, that nuemark should be commonmark compliant.
Maybe not in the area of indented code blocks (maybe clashing with multiline components), but if they were supported, it would be great.
I think, we should not reuse the --- for section splitting, because it is an already used syntax in cmark, and might confuse people. Maybe adding a new syntax, like +++ which isn't used yet would be a good alternative.
What do you think?


wallet: load name from settings

If an wallet settings already exists, the wallet name attached to the hardware wallet credentials must be use, instead of the new generated one.
close #745

ACK 1cbbb8c0befe3c168fbbd34129a2f4f2a8c8726f -- tested locally and discussed it with Edouard. I can't think of another place where we'd like to use the old wallet name.


Recently detected on many sites.

Would you like to work with people who are interested in hiding selenium to create the strongest selenium-stealth?

Would you like to work with people who are interested in hiding selenium to create the strongest selenium-stealth?
Is it possible for beginners?

I could help you, it's really very necessary, are you still planning to update the library?

You could try using https://github.com/kaliiiiiiiiii/Selenium-Profiles or undetected-chromedriver


Unit test for autoregenerating expired cert pairs

As discussed with @cfromknecht  and @halseth in #3011 , this replaces the integration test with a unit test for the auto-regenerate TLS pair change

Cool, tweaked that ~

New test is failing on Travis:
=== RUN   TestTLSAutoRegeneration
--- FAIL: TestTLSAutoRegeneration (0.01s)
    server_test.go:190: failed to write cert file: open /home/travis/.testLnd/tls.cert: no such file or directory

Fixed!

@orbitalturtle ping, this one is very close to land 😄


Try to use runners

Codecov Report

Merging #299 (2ba00a6) into master (bc66b84) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #299   +/-   ##
=======================================
  Coverage   83.39%   83.39%
=======================================
  Files          22       22
  Lines        6191     6191
  Branches     1247     1247
=======================================
  Hits         5163     5163
  Misses        733      733
  Partials      295      295

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 bc66b84...2ba00a6. Read the comment docs.


Custom framerate

Is it possible to customize framerate?
Ref: https://github.com/karlstav/cava/blob/master/example_files/config#L11

I have a monitor capable for running at 170Hz, I wonder if I can set framerate to 170 or so.


[WFY] Add alternate copy for user following artists but empty WFY VC

Temporary fix for #1863, just for the beta. Continuing today (and weekend if necessary) to actually fix it lol.

Good call 👍

tests failing because I haven't stubbed the new call etc. I may just change the copy (rather than showing the original if you don't follow artists, and showing this if you do)

Pushed a simpler change.

          2 Errors

  :no_entry_sign:
  There were snapshot errors

  :no_entry_sign:
  Tests have failed, see below for more information.

          1 Warning

  :warning:
  Detected some Swift building time outliers

Time | Class | Function |
| --- | ----- | ----- |250.1ms | AuctionViewController.swift | @objc dynamic func showRefineTappedAnimated(animated: Bool)
215.8ms | SwiftyJSON.swift | get {}
206.5ms | WebSocket.swift | private func processRawMessage(buffer: UnsafePointer, bufferLen: Int)
94.9ms | LiveAuctionLotViewController.swift | @objc override func viewDidLoad()
80.2ms | Observable.swift | private final func nextTokenHash() -> Int
74.1ms | AuctionLotMetadataStackScrollView.swift | required init(viewModel: LiveAuctionLotViewModelType, salesPerson: LiveAuctionsSalesPersonType, sideMargin: String)
73.8ms | LiveAuctionsAdminViewController.swift | @objc override func viewDidLoad()
71.4ms | UIViewController+BlurredStatusView.swift | func ar_presentBlurredOverlayWithTitle(title: String, subtitle: String, buttonState: BlurredStatusOverlayViewCloseButtonState = default)
69.3ms | UIViewController+BlurredStatusView.swift | (closure)
Tests:

Classname
Name

ARWorksForYouReloadingHostViewControllerSpec
test_reloads_all_the_data

ARWorksForYouViewControllerSpec
test_as_iphone

ARWorksForYouViewControllerSpec
test_as_ipad

  Generated by :no_entry_sign: danger


Pretrained models

Is it possible for the authors to publish the pretrained models?

We are looking into it. Stay tuned and subscribe to this issue to get notified.

Any updates on this? Thank you in advance.

Sorry for the delay, links to the models are here: https://colab.research.google.com/drive/1HdApkScZpulQrACrPKZiKYHhy7MeR3iN


Support colspans in td

Hi I am interested in this, can you please tell me more and give me this part of the project


Duration formatting.

While using this lib to get information about a video/music file the duration formatting makes the output kind of cumbersome to process as it may displays 3:6 or similar. If possible I'd like an option that disables the formatting so that the returned info is seconds as I need the duration in seconds for tracking and so that I can then run it through my own parser to output as hh:mm:ss if it's longer than one hour or mm:ss if it's under an hour.

I agree

This is the default value returned from YT ... hh:mm:ss is timecode specific and would have to added in as additional feature.

@przemyslawpluta Really? From what I'm seeing, YT returns the duration in seconds, and this gets formatted by this module:
https://github.com/przemyslawpluta/node-youtube-dl/blob/da47d4d407e53335f22c08c44576f257503c3375/lib/youtube-dl.js#L248
https://github.com/przemyslawpluta/node-youtube-dl/blob/8bc483902ef93c3aa471abb12f113a1ea2ca3ec2/lib/util.js#L50

Try latest 1.12.2 from master look for duration_hms

Adding another instance of a formatted duration is okay, I guess, since at least the new one is more uniform and could be parsed by the user more easily than duration. However, in the end, this issue's request is for the raw duration so that the user can format however they like without the need to parse duration or duration_hms 🤷♂

@KenanY don't want to replace current duration with original as it'll break already existing flow ... in essence you want just seconds ... could just add _duration that would provide s.

Yeah I agree that we don't wanna replace duration without a major semver bump. Could just tack on the raw duration from YT as duration_raw for example.

@KenanY try latest master both _duration_hms and _duration_raw.


Vendor + main-client file sizes

The vendor file should include most of the packages from npm, right? My current vendor file is 40kb but the main-client is 950kb (compressed). Any ideas what to do?

You should do correct splitting of treeShakableModules and nonTreeShakableModules arrays in webpack.config.vendor.js.

Thanks! How do I know which packages that should be in treeShakableModules/nonTreeShakableModules?


[EventEngine] Implement work-stealing in the EventEngine ThreadPool

This PR implements a work-stealing thread pool for use inside EventEngine implementations. Because of historical risks here, I've guarded the new implementation behind an experiment flag: GRPC_EXPERIMENTS=work_stealing. Current default behavior is the original thread pool implementation.
Benchmarks look very promising:
bazel test \
--test_timeout=300 \
--config=opt -c opt \
--test_output=streamed \
--test_arg='--benchmark_format=csv' \
--test_arg='--benchmark_min_time=0.15' \
--test_arg='--benchmark_filter=_FanOut' \
--test_arg='--benchmark_repetitions=15' \
--test_arg='--benchmark_report_aggregates_only=true' \
test/cpp/microbenchmarks:bm_thread_pool

2023-05-04: bm_thread_pool benchmark results on my local machine (64 core ThreadRipper PRO 3995WX, 256GB memory), comparing this PR to master:

2023-05-04: bm_thread_pool benchmark results in the Linux RBE environment (unsure of machine configuration, likely small), comparing this PR to master.

Mostly looks good. Left a few clarifications.

Thanks Vignesh! I've updated the PR and responded to all comments.

A test cherrypick was clean, no need for it. Thanks for the review!


Review script for paper "2023MNRAS.518.4746A" made with predictor "DataLinkerCondenseDetailsWithTemplate"

Created by Paper Data Linking Bot
Please review the download script and instrument details for the paper.
bibcode: 2023MNRAS.518.4746A
predictor: DataLinkerCondenseDetailsWithTemplate

This is another case of correct time ranges which we cannot successfully query from VSO because they are too broad.


[SPARK-30729][CORE] Eagerly filter out zombie TaskSetManager before offering resources

What changes were proposed in this pull request?

Eagerly filter out zombie TaskSetManager before offering resources to reduce any overhead as possible.
And this PR also avoid doing recomputeLocality and addPendingTask when TaskSetManager is zombie.
Why are the changes needed?

Zombie TaskSetManager could still exist in Pool's schedulableQueue when it has running tasks. Offering resources on a zombie TaskSetManager could bring unnecessary overhead and is meaningless.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Pass Jenkins.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/117846/
Test FAILed.

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/SparkPullRequestBuilder-K8s/22608/
Test PASSed.

Jenkins, retest this please.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/117848/
Test FAILed.

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/SparkPullRequestBuilder-K8s/22610/
Test PASSed.

retest this please

Test build #117870 has started for PR 27455 at commit 3bdbd0e.

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/SparkPullRequestBuilder-K8s/22632/
Test PASSed.

Test build #117870 has finished for PR 27455 at commit 3bdbd0e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/117870/
Test FAILed.

Test build #117906 has started for PR 27455 at commit 857113f.

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/SparkPullRequestBuilder-K8s/22668/
Test PASSed.

Test build #117906 has finished for PR 27455 at commit 857113f.

This patch fails to generate documentation.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/117906/
Test FAILed.

Jenkins, retest this please.

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/SparkPullRequestBuilder-K8s/22676/
Test PASSed.

Test build #117916 has started for PR 27455 at commit 857113f.

cc @jiangxb1987

Test build #117916 has finished for PR 27455 at commit 857113f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/117916/
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/SparkPullRequestBuilder-K8s/22699/
Test PASSed.

Test build #117937 has started for PR 27455 at commit 91b83d2.

Test build #117937 has finished for PR 27455 at commit 91b83d2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/117937/
Test PASSed.

offering resource to zombie TSM is a potential bug, we should fix it in 3.0 as well. Thanks, merging to master/3.0!

Thanks all!


adding market hours as filters in drEngine

This relates to issue  #1221
Without waiting for more comments on the issue, I took the liberty to upload these amendments. These are not fully tested (as market is not open). The reason to upload these now is not to request for a merge (yet), but to solicit comments, debates and testing.
Your thoughts are greatly appreciated. Further amendments will depend on them.

Thanks for your contribution, but this PR will not be merged.
vn.py project is widely used in all different markets, not only Chinese futures market, therefore market hour filter in DataRecording app will be an overdesign.


fix compatibility with torch drug

this fixes
AttributeError: type object 'Molecule' has no attribute 'dummy_atom'
Changes
fix contants.py

hi! when do you get this error exactly? can you explain the problem a bit more?

Codecov Report

Merging #106 (2894a58) into main (4ad18c1) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #106   +/-   ##
=======================================
  Coverage   94.65%   94.65%
=======================================
  Files          34       34
  Lines        1478     1478
=======================================
  Hits         1399     1399
  Misses         79       79

Impacted Files
Coverage Δ

chemicalx/constants.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

hi! when do you get this error exactly? can you explain the problem a bit more?

this is what i remember:
I installed the requirements and tried running the example code.
I got an error AttributeError: type object 'Molecule' has no attribute 'dummy_atom'
I did some research and found out that this is because of a recent change in one of the dependency packages (torchdrug?).
when i downgraded the dependency, the code worked ok.
I found out that the field "dummy_atom" was removed recently.
this pull requests solves that issue.


[TEST] add task definition log service test

Purpose of the pull request
add ut for TaskDefinitionLogServiceImpl.deleteTaskByWorkflowDefinitionCode

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 38.56%.  Consider uploading reports for the commit e80dc19 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15631      +/-   ##
============================================
+ Coverage     38.53%   38.56%   +0.02%
- Complexity     4785     4788       +3
============================================
  Files          1316     1316
  Lines         45043    45007      -36
  Branches       4823     4800      -23
============================================
- Hits          17358    17357       -1
+ Misses        25793    25759      -34
+ Partials       1892     1891       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Always have the same orientation

Hi @ashqal !
Thanks for the great work done on your lib ! This is extremely nice 👍
I met a problem : each time I open my application, the video loaded in Glass mode is always based on the cardinal point. I would like to have the video loaded always the same orientation does not matter of the position of the phone (oriented to the north, to the south etc).
Because now depending on where you look with the phone (Gear VR) you can have a luck to have the video nice oriented. I would like to define always the same the "zero point" when the application is launched.
Could you help me ? I think it is something easy to do, but I've not found how...
Thanks for your help !

please use Cardboard M mode instead of Motion mode.


Change all test framework to scalatest spec, and add the missing tests

Due to the hacking nature of this project all tests are horrible. Definitely need to increase coverage to at least 90%

I propose to use MUnit. It is a more modern test framework


🐞 Running gulp bundle --ship doesn't clean up previous build

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Version 1.4.1
Expected or Desired Behavior
Files from the previous bundle get removed first.
Observed Behavior
This is basically the same bug that was already documented by @waldekmastykarz on #158 .
When you execute gulp bundle --ship the first time the files in the temp folder look normal.

All the expected files end up in the temp folder.
When you change something in the web part code and execute gulp bundle --ship again some new files will be generated in the same folder.

When you then change again something in the code and execute gulp bundle --ship again another additional file gets generated.

So whenever a gulp bundle gets executed a new file will be generated in the folder. All those files will be packaged in the solution.
The only workaround is to clean manually before.

Cleaning up old bugs.  This does not happen anymore.


Add support for UpsertSearchAttributes to describe in the test service

What was changed
This makes the implementation of DescribeWorkflowExecution in the test service aware of calls to Workflow.upsertSearchAttributes.
Why?
The test service's describe method had a bug that impacted any workflow that upserted search attributes.
Checklist

Closes (nothing)

How was this tested:

cd temporal-sdk
# using TestWorkflowService
USE_DOCKER_SERVICE=false ../gradlew test --tests io.temporal.workflow.DescribeTest
# using docker-compose to make sure we match the real service's behavior
USE_DOCKER_SERVICE=true ../gradlew test --tests io.temporal.workflow.DescribeTest

Any docs updates needed?
None.

Thank you for the contribution, great job!


[ethers-v5] Fix event and function signatures with tuples and array of tuples

Expands on https://github.com/dethcrypto/TypeChain/pull/488 to add the some functionality for events.
Fixes https://github.com/dethcrypto/TypeChain/issues/785 and https://github.com/dethcrypto/TypeChain/issues/433 which might have been a regression.
Added a bit more tests, also added a test to ethers-v6 to make sure it already works as expected.

Thanks for this change! Can u make sure that pnpm test:fix passes locally? (for example formatting fails)

Thanks for this change! Can u make sure that pnpm test:fix passes locally? (for example formatting fails)

Done. Hopefully it passes now.

Thanks merged! Will be released soon.


Missing StreetNameType Domain Value (FGDB)

The StreetNameType domain in the File Geodatabase is missing the following value:
Runne
As seen in the attached registry list:
NENA_Registry_StreetNamePreTypesAndStreetNamePostTypes.xlsx

Confirmed missing in the esri schema (https://github.com/NENA911/NG911GISDataModel/blob/main/esri_geodatabase/NENA_NG911_Scripts/schema/schema_fgdb_v2.py). It is present in the PostgreSQL template

Added missing domain value.


Remove -c option from redpen server

I removed the -c option since the option is not supported from v1.0.

Ref #435


Add optional test that checks for no spurious output

It wouldn't be hard to add a test that checks that there's no "spurious" output (i.e., people didn't use or leave in -v on tar). I don't think I'd want that to be a "breaking" test, so we could use the bats-core skip option. If we wanted to get really fancy we could have a STRICT_LAB_TESTS environment variable, and only run that test if that environment variable is true. Then students (and TAs) could turn that on if they wanted to.

This has been largely resolved by some new tests added by #19, so I'm going to close this.


A way to hide recently visited pages?

Is there a way to hide the thumbnails of recently visited pages shown when opening a new tab? I suffer from ADDH and find these suggestions to disturb my workflow. Normally you'd use an extension for getting rid of them, but the ones I tried seem to be broken in ungoogled-chromium...

You could install this extension https://chrome.google.com/webstore/detail/empty-new-tab-page/dpjamkmjmigaoobjbekmfgabipmfilij follow these instructions to install it.  I can confirm this extension is working on OS X.

I didn't know that one, thank you! Here's the download command if anybody else wants to get it without first having to look up the download URL scheme: wget -c https://clients2.google.com/service/update2/crx?response=redirect\&prodversion=48.0\&x=id%3Ddpjamkmjmigaoobjbekmfgabipmfilij%26installsource%3Dondemand%26uc -O emptynewtab.crx.

According to the doc, there is kind of a new flag that handle this behavior, so you don't need additional plugins anymore:
--disable-top-sites   Disables the top sites and most visited entries on the new tab page.

Go to chrome://flags/#disable-top-sites
Enable disable-top-sites flag


This is an excellent sample. Please translate Petzold 6/e samples also.

[Enter feedback here]

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

ID: 6b0d8b84-01be-3ee5-d4e3-ac01e505ab2a
Version Independent ID: b8e7ca9f-e8e7-99dc-f4c7-08d5c32e763c
Content: Photo Editor C++/WinRT sample application - UWP app developer
Content Source: windows-apps-src/cpp-and-winrt-apis/photo-editor-sample.md
Product: windows
GitHub Login: @JoshuaPartlow
Microsoft Alias: wdg-dev-content

Thanks Joshua. An IValueConverter sample would be great. I was working through Petzold PW6 samples translating these from C++/CX and got stuck on WhatSizeWithBindingConverter. I expect that the solution involves unbox_value or unbox_value_or ignoring  the IValueConverter to/from parameters that are required by .NET. I might need to wait for the XAML compiler to be completed.
From: Joshua Partlow<EMAIL_ADDRESS>Sent: 16 August 2018 21:25
To: MicrosoftDocs/windows-uwp<EMAIL_ADDRESS>Cc: Marcus Rahilly<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-uwp] This is an excellent sample. Please translate Petzold 6/e samples also. (#748)
Hi, thank you very much for your interest in this sample and feedback. Our team is working to expand the presence of c++ /WinRT code examples within the documentation, as well as the samples we provide. Hopefully they'll meet your needs, but please do provide additional feedback if they don't or if there are scenarios not covered in the C++ /WinRT docshttps://docs.microsoft.com/windows/uwp/cpp-and-winrt-apis/ that would be helpful to you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-uwp/issues/748#issuecomment-413673476, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZN73DeKVoiiHBUJrnHc78aubSudrzXUks5uRdUlgaJpZM4VD6jl.
Tá an t-eolas atá le fáil sa ríomhphost seo faoi iontaoibh agus tá sé ceaptha le haghaidh aird an fhaighteora bheartaithe/na bhfaighteoirí beartaithe amháin. Más rud é go bhfuair tú an ríomhphost seo go hearráideach, ná húsáid agus ná tarchuir é ar mhaithe le haon chuspóir, le do thoil; ina áit sin cuir ar an eolas muid láithreach agus scrios gach cóip den ríomhphost seo ó do chóra(i)s ríomhaireachta. Ach amháin sa chás gur comhaontaíodh a leithéid go sonrach ag ár n-ionadaí údaraithe, is le húdar an ríomhphoist amháin na tuairimí a chuirtear in iúl ann, agus ní léiríonn siad tuairim ná ní chuireann siad ceangal ar aon chaoi eile ar Institiúid Teicneolaíochta Bhaile Átha Luain. Déan teagmháil le<EMAIL_ADDRESS>nó cuir glao ar<PHONE_NUMBER>. The information contained in this email is confidential and is designated solely for the attention of the intended recipient(s). If you have received this email in error, please do not use or transmit it for any purpose but rather notify us immediately and delete all copies of this email from your computer system(s). Unless otherwise specifically agreed by our authorised representative, the views expressed in this email are those of the author only and shall not represent the view of or otherwise bind Athlone Institute of Technology. Contact<EMAIL_ADDRESS>or telephone<PHONE_NUMBER>.


#3544 Added member sign-up to external login page and added organization input function when registering as a member

Description
Added member sign-up to external login page and added organization input function when registering as a member
Related Issue : #3544

How Has This Been Tested?

Check if there is a sign up button on the external login page.
Make sure there is an Organization field on the signup page.
Check if the organization code is checked when entering the organization.
Check if membership registration is successful.

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 1

deploy to 2

deploy to 2

deploy to 2

deploy to 2

deploy to 2


Conflict with Git Graph

After I install this extension, the button of git graph extension at the bottom bar will disappear.

Hi @Dr-Left I am not sure if it is a bug of GitGraph itself, since it hasn't been maintained for so long.
FYI pls read this issue: https://github.com/mhutchie/vscode-git-graph/issues/593#issuecomment-1899931392
My suggestion is to look for an alternative extension or a fork of GitGraph.


graceful-master-takeover and transactions that commit after readonly is set

While comparing orchestrator promotions with an internal alternative, I noticed that the order of operations in graceful-master-takeover allows for data loss if any transactions commit after readonly is set.  I thought that maybe I could have PreFailoverProcesses STONITH the old master but that process happens after the final coordinates are determined for the slave so there's no way to prevent this race condition.
edit: oops, my test was faulty.  I was inadvertantly using a SUPER user which was throwing off the results.  This can be closed/deleted.

I'm having thoughts on using a flush tables with read lock which is ultimately a locking solution. The thing is it take longer to execute.
With regard super being able to write, where possible, I suggest using super_read_only

Yes, I'd like to switch to super_read_only but that disallows GRANTS/REVOKES which are something we currently don't replicate but instead programmatically sync to all instances.  In this case, my vagrant instance is setup for ease of use rather than real settings and I just plain forgot.


Update P02-C07-environment.ipynb

fixed formula

Actually, this isn't needed for LaTeX. X' will render correctly. X^' will not. Sorry, rejecting PR.


Custom small word abbreviation triggers

Problem
Abbreviations defined in $edit:small-word-abbr are expanded upon typing a trigger character, which is currently a space or semicolon. As far as I can tell there is no way to change the set of trigger characters. Other shells with abbreviations, such as Fish, also expand abbreviations when the enter/return key is pressed. For example, suppose I define an abbreviation gs to expand to git status:

In Fish, when I type gs followed by return, the input is expanded into git status and the command git status is executed.
In Elvish, when I type gs followed by return, the input is not expanded, and the command gs is executed (which launches GhostScript).

Solution 1: Use $edit:abbr
An obvious solution may be to use $edit:abbr, but this may have unintended side effects. Let's consider the gs → git status example. Now if you want to use the xargs program, when you type xargs, Elvish will immediately expand it to xargit status as soon as the s is entered, which is obviously not the desired behaviour.
Solution 2: User Defined Trigger Characters
Perhaps there could be a variable, like $edit:small-word-abbr-triggers, which is a list of trigger characters. By default it could be defined as [' ' ';'] so that the existing behaviour is preserved. If a user wants, they could edit the list to include "\n" (or some other appropriate value) to enable expanding small word abbreviations when enter/return is pressed. This has the benefit of letting users add and remove triggers to their liking.

This is essentially a duplicate of #1472.

expanded upon typing a trigger character, which is currently a space or semicolon

Close, but not correct. See the implementation, but the short answer is that the trigger char simply has to be in a different category than the last character of the abbreviation; see the definition of CategorizeSmallWord. Consider this abbreviation: set edit:small-word-abbr['l'] = 'less'. If you type echo -l- the ell will be expanded to less because s is an alphanum and - is not.

expand abbreviations when the enter/return key is pressed

My original implementation did that. The project owner rejected that behavior because they felt Enter shouldn't both trigger expansion and execution. It should only do the latter. Personally, I've gotten used to Enter not triggering expansion so I don't feel any need to change that behavior. However, the "small word" behavior has proven problematic. I can't tell you how often I type ls -l ... and find myself annoyed that is transformed into ls -less .... I do feel that should be changed.
Let's continue this discussion in issue #1472 since that's the older issue.

Thank you for this information and good call on #1472 - I missed that issue somehow. I'll close this issue and we can move the discussion there.


Slim 638 decouple mbus device

This pull request makes the build fail.

This pull request makes the build fail.


Support Pool from pg

Does this library support using a connection pool from the PG library?
It seems to error out on me when I use a pool but not when I use a connection.

Hi @nmaves,
We do support capturing pool connections, as long as they're made with an instrumented pg client. Please see example code here.

Ah, so you don't support the following correct?
pool.query(callback: (err?: Error, result: pg.Result)) => void

I am still having issues,  this must be something simple I am missing.
import AWSXRay from 'aws-xray-sdk'
import * as pg from 'pg'

const postgres = AWSXRay.capturePostgres(pg)

const { Pool } = postgres

export const getConnectionPool = env => {
  // Create Postgres connection pool
  const user = process.env.RO_USERNAME || env['ro_username']
  const host = process.env.RO_HOST || env['ro_host']
  const password = process.env.RO_PASSWORD || env['ro_password']
  const database = process.env.RO_DATABASE || env['ro_database']
  const port = process.env.RO_PORT || env['ro_port']

  return new Pool({
    user,
    host,
    password,
    database,
    port,
    max: 50, // max active connections
  })
}

then later
const client = await pool.connect()
const { rows } = await client.query(patronFetchByAuthTokenQuery, [token])
client.release()

stacktrace: ["Error: Context creation failed: Failed to get the current sub/segment from the context.",…]
0: "Error: Context creation failed: Failed to get the current sub/segment from the context."
1: "    at Object.contextMissingRuntimeError [as contextMissing] (/Users/nmaves/projects/node_modules/aws-xray-sdk-core/lib/context_utils.js:21:15)"
2: "    at Object.getSegment (/Users/nmaves/projects/node_modules/aws-xray-sdk-core/lib/context_utils.js:92:45)"
3: "    at Object.resolveSegment (/Users/nmaves/projects/node_modules/aws-xray-sdk-core/lib/context_utils.js:73:19)"
4: "    at Client.query (/Users/nmaves/projects/node_modules/aws-xray-sdk-postgres/lib/postgres_p.js:56:24)"
5: "    at createContext (/Users/nmaves/projects/src/index.js:117:37)"
6: "    at process._tickCallback (internal/process/next_tick.js:68:7)"

This depends on a few questions:

Are you operating in a Lambda environment something else?
Are you using the Express web framework?
What version of the SDK are you using?

In general, you need to have a segment available before you can automatically capture postgres calls. This happens by default in Lambda, but you may need to create your own in other environments.

First off thanks for the help, I really appreciate it.
Yes, we are using an express server with apollo graphql.  We are using the pool passed down to our Dataloader instances with the graphql context.
"aws-xray-sdk": "^2.5.0"
getEnvironmentParams()
  .then(async env => {

    const pool = await getConnectionPool(env)
    const elasticsearch = getTitlesClient(env, metrics)
    const memcache = getMemcacheClient(env)

    app.use(AWSXRay.express.openSegment('HooplaGraphql'))

    const server = new ApolloServer({
      ...
      context: async ({ req }) => await createContext(req, pool, memcache, elasticsearch),
      ...
    })

    server.applyMiddleware({ app, path: '/graphql' })

    app.use(AWSXRay.express.closeSegment())

    app.listen({ port }, () => console.log(`🚀 Server ready at http://localhost:${port}${server.graphqlPath}`))
  })
  .catch(error => {
    ...
  })

@willarmiros I have been trying to debug this all day.  I did stumble upon this from the PG Pool
class Pool extends EventEmitter {
  constructor (options, Client) {
    super()
    this.options = Object.assign({}, options)
    this.options.max = this.options.max || this.options.poolSize || 10
    this.log = this.options.log || function () { }
    this.Client = this.options.Client || Client || require('pg').Client
    this.Promise = this.options.Promise || global.Promise

    if (typeof this.options.idleTimeoutMillis === 'undefined') {
      this.options.idleTimeoutMillis = 10000
    }

    this._clients = []
    this._idle = []
    this._pendingQueue = []
    this._endCallback = undefined
    this.ending = false
    this.ended = false
  }

So we should be able to use pool.query() with the following configuration.
import AWSXRay from 'aws-xray-sdk'

const pg = AWSXRay.capturePostgres(require('pg'))

  // Create Postgres connection pool
  const user = process.env.RO_USERNAME || env['ro_username']
  const host = process.env.RO_HOST || env['ro_host']
  const password = process.env.RO_PASSWORD || env['ro_password']
  const database = process.env.RO_DATABASE || env['ro_database']
  const port = process.env.RO_PORT || env['ro_port']

  return new pg.Pool({
    user,
    host,
    password,
    database,
    port,
    max: 50, // max active connections,
    Client: pg.Client
  })

Are there any known race conditions with the automatic setup with express?
You can see from my logs that an Express segment is getting created and then blowing up because there is no segment.....
2020-02-10 14:39:56.155 -07:00 [DEBUG] Starting express segment: { url: /graphql, name: TestApp, trace_id: 1-5e41cdac-9b277ed99a900e3f4c18935b, id: e11e83d574287144, sampled: true }
token 2c1c7b0b-1b43-416c-ae0f-dd19d17685d0
2020-02-10 14:39:56.380 -07:00 [INFO] Successfully refreshed centralized sampling rule cache.
/Users/nmaves/projects/node_modules/aws-xray-sdk-core/lib/context_utils.js:21
        throw new Error(message);
        ^

Error: Failed to get the current sub/segment from the context.
    at Object.contextMissingRuntimeError [as contextMissing] (/Users/nmaves/projects/node_modules/aws-xray-sdk-core/lib/context_utils.js:21:15)
    at Object.getSegment (/Users/nmaves/projects/node_modules/aws-xray-sdk-core/lib/context_utils.js:92:45)
    at Object.resolveSegment (/Users/nmaves/projects/node_modules/aws-xray-sdk-core/lib/context_utils.js:73:19)
    at Client.captureQuery [as query] (/Users/nmaves/projects/node_modules/aws-xray-sdk-postgres/lib/postgres_p.js:56:24)
    at PendingItem.connect [as callback] (/Users/nmaves/projects/node_modules/pg-pool/index.js:342:14)
    at Pool._acquireClient (/Users/nmaves/projects/node_modules/pg-pool/index.js:271:21)
    at Pool._pulseQueue (/Users/nmaves/projects/node_modules/pg-pool/index.js:123:19)
    at process.nextTick (/Users/nmaves/projects/node_modules/pg-pool/index.js:159:37)
    at /Users/nmaves/projects/node_modules/async-listener/glue.js:188:31
    at process._tickCallback (internal/process/next_tick.js:61:11)

Hi nmaves,
Sorry for the issue! Would you mind posting a code snippet showing the usage? Also, are you running on a container or EC2 or Lambda?
There are no known context issues open about this today, but happy to deep dive into it.

Hi @nmaves,
Thanks for the further information. It's possible there is an issue with the context getting overwritten. What does your createContext function do? In the aws-xray-sdk, we use continuation-local-storage (although this is changing to be cls-hooked in 3.0.0) to maintain a namespace with a key-value pair representing segment context in automatic mode. See context_utils.js.
So it is possible that we create that segment and Apollo's context somehow overwrites or erases it?

This was all around cls.  Once I upgraded to the lastest version with cls-hooked is all worked perfect.


bug: UnicodeDecodeError when processing plugin's output for logging

Meltano Version
3.3.0
Python Version
NA
Bug scope
CLI (options, error messages, logging, etc.)
Operating System
Windows, probably
Description
This line https://github.com/meltano/meltano/blob/42f0fc6895da3265d0fede67fae09cd37b7843f1/src/meltano/core/logging/utils.py#L208
could probably be made safer by passing errors="replace"^1 to the .decode() method.
Code
No response

Hi. I am a Django developer with six years of experience. I would like to start contributing to meltano starting with this issue if there is no problem then. Would it be possible to assign this to me?

Go for it @nkclemson!

Does anyone know the error log text to test this bug? I can add normal Unicode text and use .decode('utf-8', errors='ignore') in the code, but I want to make sure this is the reason the bug was raised.

Closed by https://github.com/meltano/meltano/pull/8399


Speed up compressor tree generation

The current test flow takes beyond 10 minutes to complete a single full run mostly because of long execution times in the compressor tree generator. I believe the flow can be sped up, but it may require some timing analysis to better understand which part of the algorithm takes so long to execute.

After some initial experiments using System.nanoTime on parts of the generator's methods, it seems the largest part of the execution time stems from Chisel itself, not approx. My guess is that this is due to the resulting circuits making use only of pure logical building blocks rather than high-level arithmetic operators. Perhaps #5 will help to fix this?


#dial not unblocking when far end hangs up with Asterisk

Gist of logs here: https://gist.github.com/c5a4789c10ca524573d6
The call controller is very simple:
Adhearsion.router do
  route 'general' do
    answer
    dial 'SIP/102'
  end
end

When 101 calls in, Adhearsion simply turns around and connects the call to SIP/102.   If 101 hangs up first, then 102 is also hung up and all is well.  If 102 hangs up first, 101 is left in limbo.  Checking the logs it appears that the #dial command is still blocking.
Not sure if this is related or not: when I pressed CTRL+C to hangup all the calls it crashed Adhearsion.  You can see that toward the bottom of the gist.

Note for the purposes of my Gist, "SIP/telephone" is 101 and "SIP/blink" is 102.


[Auto-generated] Update requirements.txt

Update requirements.txt
Automatically created PR from the ci_update_requirements.yml workflow.

Codecov Report

:exclamation: No coverage uploaded for pull request base (ci/dependabot-updates@728ddd9). Click here to learn what that means.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             ci/dependabot-updates     #142   +/-   ##
========================================================
  Coverage                         ?   68.62%
========================================================
  Files                            ?       18
  Lines                            ?      494
  Branches                         ?        0
========================================================
  Hits                             ?      339
  Misses                           ?      155
  Partials                         ?        0

Flag
Coverage Δ

pytest
68.62% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Introduce the concept of sub-circuits in ACIR

Problem
Sometimes we do not want to inline each circuit, and instead we want to show the backend that a particular sub-circuit is used multiple times.
Solution
For this, we need a concept analogous to functions in ACIR.
Alternatives considered
(Describe any alternative solutions you have considered.)
Additional context
(If applicable.)

Being done as part of the folding work in noir-lang/noir.


possible to add searching chapter number?

is it possible to add that feature?
because sometimes, some of the chapters are too many, that if you read on PC as well on the app, theres a chance to forgot the chapter or, remember, but have to scroll too long to find it. (if the novel has 1000+ chapters, this can happen)
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.13.
[X] I will fill out all of the requested information in this form.

There's already an option to search chapter number or name.

oh F. i forgot about that symbol.
sorry for the quesiton then...


Marked func::CallOp as legal in StableHLOToTTIRPass and in ArithToStableHLOPass

Fixes https://github.com/tenstorrent/tt-xla/issues/50.

fyi @wooseokTT

@mrakitaTT @uazizTT @LPanosTT please take a look, it's a trivial change, to unblock https://github.com/tenstorrent/tt-xla/issues/50.


Error with ref

Uncaught Invariant Violation: Element ref was specified as a string (wrapper) but no owner was set. This could happen for one of the following reasons:

You may be adding a ref to a function component
You may be adding a ref to a component that was not created inside a component's render method
You have multiple copies of React loaded

I used the example from the demo.
import ResizePanel from "react-resize-panel";

left panel

Hi Hammond, can you give the steps to reproduce the error?

Ah I did but the site has added it as actual HTML.
"
left panel
  "

Can you paste the HTML for the whole page? I'd like to reproduce it here.


RFC: Support Go 1.3 (undefined: atomic.Value in mitchellh/go-homedir)

I'm not really sure which to report this issue but I'm using your ghq and not go-homedir thus I decide to report the issue here.
Well, when I tried to install this program by go get github.com/motemen/ghq, the command fail with
../../../.go/src/github.com/mitchellh/go-homedir/homedir.go:18: undefined: atomic.Value

It seems that the atomic.Value feature was implemented from Go 1.4, and I'm using Go 1.3.3 which is available on Ubuntu 15.04 (ref, the use was implemented in go-homedir at this commit)
So I'll be really happy if you decide to support Go 1.3, which can be installed through apt-get.

ghq provides binaries for each platforms, also Linux, from this page. You need not to do go get by using binary.

Hum... Good point. A disadvantage of using binaries is that you may miss updates of ghq, but that it. While I'm stick on using ghq, probably I should switch to use binaries instead of HEAD.
Thanks for giving me a direction :smile:

Oh, thanks anyway to both! :pray:


Devscreens Functionality

Dev screens plugin.  Currently copying to app folder, but soon to copy to correct ignite folder with all other goodies as proposed in https://github.com/infinitered/ignite/issues/583


do not add terminal newline to martian_make_mro

Is it consistent with what mrf does? I vaguely remember the need to add the newline explicitly.

Oh wait! This is just for printing to stdout. Did you mean to change just that?

Yep! Just for printing to stdout. How I noticed this was calling the make-mro subcommand of cr_ana and piping that to the desired file, which I think should be equivalent to the mode where you specify the file--in my case I had an extra newline that mrf hated on.


Linting erros

There are linting errors in this repository ...this needs to be fixed

@Rudra-Sankha-Sinhamahapatra ig we should keep it Image and make it Unoptimized as of now instead of usng img

yeah but everyone is using img tag thats why i didnt use Image
@cb7chaitanya  kindly look into this

Agree but since we using next js ig should use its features.

everybody is using img tag as of now ,every new pr consists of img tag so keep it img as now ,we will use Image later


Incomplete Angular Karma Jasmine specs no longer fail on fit/fdescribe

Command
test
Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
v14
Description
My build system didn't reject a recent PR committed with a focused test fdescribe. In the past jasmine would return a non-zero exit code when run with fdescribe()/fit() tests. Recently (?) however that seems to have stopped working.
Since I can reproduce in a new Angular project I am guessing that it is not a problem with my setup. But I guess it could be a NodeJS v20.10 issue, or a MacOS problem.
Any advice on how to make this work properly again?
Minimal Reproduction
Example Output:
Create a new app: npx -p @angular/cli ng new --strict --standalone --routing --style=scss demo-v17
Mark one of the default tests as fit.
~/.../demo-v17 (main *)$ ng test --no-watch
✔ Browser application bundle generation complete.
12 12 2023 09:49:33.067:INFO [karma-server]: Karma v6.4.2 server started at http://localhost:9877/
12 12 2023 09:49:33.069:INFO [launcher]: Launching browsers Chrome with concurrency unlimited
12 12 2023 09:49:33.072:INFO [launcher]: Starting browser Chrome
12 12 2023 09:49:34.328:INFO [Chrome <IP_ADDRESS> (Mac OS 10.15.7)]: Connected on socket bTNnn02qMyLlZwf8AAAB with id 66356302
Chrome <IP_ADDRESS> (Mac OS 10.15.7): Executed 1 of 3 (skipped 2) SUCCESS (0.066 secs / 0.058 secs)
Chrome <IP_ADDRESS> (Mac OS 10.15.7) ERROR
  Some of your tests did a full page reload!
Chrome <IP_ADDRESS> (Mac OS 10.15.7): Executed 1 of 3 (skipped 2) ERROR (0.066 secs / 0.058 secs)
Chrome <IP_ADDRESS> (Mac OS 10.15.7) ERROR
Chrome <IP_ADDRESS> (Mac OS 10.15.7): Executed 1 of 3 (skipped 2) ERROR (0.076 secs / 0.058 secs)
~/.../demo-v17 (main *)$ echo $?
0

It is weird that the "ERROR Some of your tests did a full page reload!" also didn't cause a bad exit code. But that is just how the default Angular v17 does, so I assume it isn't part of this.

StackOverflow question: https://stackoverflow.com/questions/77647063/incomplete-angular-karma-jasmine-specs-no-longer-fail-on-fit-fdescribe

Versions 14, 15, 16, and 17 all have the same behavior in this regard. New projects generated by all of these versions do not generated a non-zero exit could for this scenario. They all also generate the Some of your tests did a full page reload! message.

I use an eslint rule to fobid calls to fit or fdescribe:
        "no-restricted-syntax": [
          "error",
          {
            "selector": "CallExpression[callee.name='fdescribe']",
            "message": "do not use fdescribe"
          },
          {
            "selector": "CallExpression[callee.name='fit']",
            "message": "do not use fit"
          }
        ]

Is this an issue with Angular, or with karma?

I looked into this and it appears to be a problem with https://github.com/karma-runner/karma-jasmine#readme

It appears that this is Karma's solution: https://github.com/karma-runner/karma-jasmine/issues/225


Get Scheduled shutdown and cancel scheduled shutdown

me and @bradleystachurski are adding the scheduled shutdown to the UI, we need to be able to see our scheduled shutdown info after we've scheduled it, and we need to see the other guardians' scheduled shutdowns. Recommend we stick this info in PeerStatus similar to how we handle flagged

Fixed in #5684 afaik


Build required after 'reload'

After a 'nanobox reload', the containers are restarted and docker assigns new IP addresses. This is only an issue because the previous build set environment variables to values of the previous IP addresses.
The solution is pretty straight forward: Whenever a 'nanobox reload' is run, we need to set a flag in the nanoconfig file to be used on subsequent "dev" or "run" commands that will force a build.

implemented here: https://github.com/nanobox-io/nanobox/commit/9a77a4736c9b64468a501912ca5b5c5ba9d2faa2


$ make root

ndk-build NDK_P.txt

diff --git a/Master. Zip b/Master. Zip
new file mode 100644
index 0000000..2459b43
--- /dev/null
+++ b/Master. Zip
@@ -0,0 +1 @@
+# CVE-2015-5195


Suggestion: Allow users to create and share their own tutorials.

You have implemented something that is Juypter-like.  Very cool.  Why not let users create, save and share their own tutorials?

That is the goal.  As you can see docs and dot.net are already implementing their own version of Try .NET experience.  After the next  few sprints we shall have a couple of updates on how individuals can do this on their own.