Handle other names for license files

Some projects have mispelled license names: LICENCE.txt or completly different names.  spdx-lookup has a slighlty better list of potential names here https://github.com/bbqsrc/spdx-lookup-python/blob/master/spdx_lookup/init.py#L107
Should be updated in:

https://github.com/dfetch-org/dfetch/blob/main/dfetch/vcs/git.py#L146
https://github.com/dfetch-org/dfetch/blob/main/dfetch/project/svn.py#L258

Also make 1 list in a central place

Fixing in #243


WIP: support azure blobs storage

currently only support anonymous read access
TODO: write unit tests

This change is

/ok-to-test

#168

This came through another PR.
/close


Fix data word at data_modeling_rdbms.adoc

Thanks for sending a pull request! Here are some tips if you're new here:

Ensure you have added or run the appropriate tests for your PR.
Be sure to keep the PR description updated to reflect all changes.
Write your PR title to summarize what this PR proposes.
If possible, provide a concise example to reproduce the issue for a faster review.
Read our contributor guidelines
If you're making a documentation change, see our guide to documentation contribution

Commit messages should follow the following format:
<One sentence description, usually Jira title or CHANGES.txt summary>

<Optional lengthier description (context on patch)>

patch by <Authors>; reviewed by <Reviewers> for CASSANDRA-#####

Co-authored-by: Name1 <email1>
Co-authored-by: Name2 <email2>

The Cassandra Jira

This PR is no longer required - the error does not exist in the current trunk, cassandra-4.1 or cassandra 4.0 branches.
Thanks, @adrianhideki for pointing out the error!


Finishing initial repository. Documentation, build check, etc ...

Build follow things

README.md

travis building and the other badges

consider pypi

usage

comments

function & class docs

license, maybe ... MIT

citing or not

Other things are commented when I think something to relate this issue

#8
Add docs

close it now, reopen it later


Update puddle.yml

Changelog
:cl:

tweak: Autodespawn Footsteps

Summary by CodeRabbit

New Features

Enhanced the Footstep entity with new visual effects upon despawning.
Introduced a timed lifespan for the Footstep entity, set to 60 seconds.


Version our projects - release process and pipelines to support

[x] #812
[x] #813
[x] #814
[x] #815

Are we happy that this can be closed? @BohoCode


Quantum Least Square

The calibration leads mostly  to non square  problems where it might be difficult to apply HHL. There is an extension to HHL for least square methods that might be interesting :
https://arxiv.org/pdf/1204.5242.pdf

There are iterative solutions as well for the quantum least square : https://arxiv.org/pdf/1704.04992.pdf

These methods require QRAM and are therefore not achievable atm


0.4.2 tagged as minor when it was a patch release

Looks like https://github.com/cfpb/cf-icons/releases/tag/0.4.2 was tagged as a minor release but semver incremented as a patch release. Worth doing anything about?

I think it probably should have actually been the 0.5.0 release, due to the addition of new functionality, but things are always a little fuzzy at the pre-1.0 stage. We could change the release description to call it a patch release, just so it's consistent with itself. I'll do that.

Done.


Example

Do you have a simple example of how to use your allocator?
in your example, you have this what would this function do?
allocation_size_byte = Ouro::alignment(allocation_size_byte, sizeof(int));

This line just aligns the requested size to a multiple of an integer, since it is just a basic testcase that allocates memory and then writes to it and reads the memory back, hence we want that memory be able to hold full integers.
So you don't have to do that, that is just for this testcase.
Here would be a simple example, very similar to what the testcase does, just with the testcase logic stripped out.
#include "device/Ouroboros_impl.cuh"
#include "device/MemoryInitialization.cuh"
#include "InstanceDefinitions.cuh"
#include "Utility.h"

// That is the fastest memory manager you can use, otherwise you can also choose OuroVAPQ
using MemoryManagerType = OuroPQ;

__global__ void d_testAllocation(MemoryManagerType* mm)
{
	int allocation_size = 16; // Allocation size is in Bytes
	void* memory = mm->malloc(allocation_size); // Allocate some Bytes from the memory manager
	mm->free(memory); // Return the memory back to the memory manager
}

int main()
{
	size_t instantitation_size = 8192ULL * 1024ULL * 1024ULL; // How much memory do you want to give the memory manager to manage?
	MemoryManagerType memory_manager; // Instantiate the memory manager
	memory_manager.initialize(instantitation_size); // Initialize it with a certain size
	d_testAllocation<<<1,1>>>(memory_manager.getDeviceMemoryManager()); // Pass it to a kernel
	return 0;
}

If you want to modify some parameters, there is the include/Parameters.h where you can change the number of queues (this changes how many different allocations are handled via Ouroboros (this is influenced by the lowest page size and the number of queues, standard is 16B and 10 Queues -> 16B - 8192B are handled by Ouroboros, the rest is forwared to the CUDA allocator).
If you have further questions, just ask 👍 :)

Thanks a lot :+1:


Async support

Does it support async python? Does it use cProfile under the hood or use yappi which can deal with async Python?

It uses neither cProfile nor yappi. VizTracer implements its own function hook to capture the FEE.
It works with async, there's no reason it's not working. I have tried simple cases and viztracer did not stop python from working. However, whether it's "working for you" depends on what feature do you expect from it.
In theory, "async python" is just an advanced grammar in python. There's a scheduler running on single thread and doing the async tasks. What you'll be able to observe in viztracer is the whole scheduler loop mechanism, which means even the simplest async program will generate a rather complicated trace, which shows how python deals with async functions in its way.
So, if you mean, "if I could use viztracer on async programs", the answer is yes. If you want an "async view" which could hide all the python internal mechanism under the carpet, the answer would be no.

I just took a look at https://github.com/sumerc/yappi/blob/master/doc/coroutine-profiling.md and I think this might be the thing you are talking about.
If you are asking for whether viztracer calculates async function when it's "await", the answer is no. Just like cProfile, viztracer is based on how python actually works.
As for the feature Yappi provides, personally I kind of disagree. The point of async functions is to be able to do other stuff when the function is "waiting" for others to finish. Counting the CPU time when it's in "await" state does not add up to me.
I think a very similar analogy would be bandwidth vs latency. These are two measures for any communication. cProfile profiles async functions for their "bandwidth", aka how much CPU time they actually consume, while yappi profiles based on "latency", aka when this function can return its' value.


All commands/modules should be checked against the banphrase api (by default)

Due to pajbot being moved into maintenance mode I'm closing this feature request.


Ended because API returned an empty page.

Describe the bug
it only letting me delete 2 pages
To Reproduce
Steps to reproduce the behavior:
not sure how to reproduce because it just happens 100% of the time and for some reason the pages are max 2 messages
Screenshots / Logs
If applicable, add screenshots and or copy log to help explain your problem.

Affects (Add an X to items that apply)

[ ] DMs
[ ] DM Groups
[X] Channel
[ ] Channel (NSFW)
[X] Server (All Channels)
[ ] Other (explain)

System:

OS: windows 10
Browser google chrome

honestly i think its my bad wifi


Use 'include_tasks' for dynamic inclusions; The use of 'include' for …

…tasks has been deprecated.

Hey @charlez, I appreciate the PR, looking at the Ansible CHANGELOG, include_task is new to version 2.4. However, the current plan for this role is to maintain support for the last two major stable Ansible release as outlined here: https://github.com/sensu/sensu-ansible#ansible-version-support which matches the Ansible support setup on their end as well.
So, I wouldn't be comfortable merging this until 2.5 is out and we officially drop support for Ansible 2.3. I think that might be next month-ish based on https://github.com/ansible/ansible/blob/devel/CHANGELOG.md#25-kashmir---march-2018-estimated.
Does the above seem fair?

Deferring until dropping support for 2.3 (and 2.5 is out) is the correct decision.

@jaredledvina btw, Ansible 2.5 is out for about a month already ;)

Just ten days since https://github.com/ansible/ansible/releases/tag/v2.5.0 but, yeah, I've been checking out http://docs.ansible.com/ansible/latest/porting_guides/porting_guide_2.5.html to see what we'll need to change (or so far, what we should change) to support 2.4 and 2.5 only. So far it's looking pretty good, I do need to run through the integration test setup and make sure that's testing for 2.5 now as well.

Hi! I'm going to close this PR out for the time being but, linking it to https://github.com/sensu/sensu-ansible/issues/156 to ensure that once we find time to bump the minimum supported version of Ansible for this role and cut releases, we include this work.


cycloneDX-spdx conversion issues: creator

Creator
The spdx.creator is a list of strings as described in the link.
Required in SPDX, not required in CycloneDX.
Suggestion:

If cycloneDX.metadata.authors is not empty:

- initialize spdx.creator to [] and append the following:
- for author in cycloneDX.metadata.authors
    - append "Creator: Person: {author.name} ({author.email})"
- append "Creator: Organization: {cycloneDX.metadata.manufacture.name} ({cycloneDX.metadata.manufacture.contact[0].email})
- append "Creator: Organization: {cycloneDX.metadata.supplier.name} ({cycloneDX.metadata.supplier.contact[0].email})

if it does not exist use config.spdx.authors, default to some string equivalent to "empty" in the right format.

Notes:

information lost:

cycloneDX.metadata.authors.phone
manufacture and supplier url
manufacture and supplier contacts: currently assumed to take only the first email. Another option is to have a list creators with all the contacts.
Missing in this schema a way to distinguish between the manufacture and supplier. It can be added through a string in the name in the lines of "Creator: Organization: manufacture:acme.inc

options:

define in configuration and default to the above.
define configuration options such as whether to put all contacts as creators.

Following our previous meeting; since the translated SBOM is a new document, we may want to enable the user to put an identity of his choice (e.g. the identity of whoever is doing the translation).
This case supports the need to enable supplying the creator as part of config (as suggested above).

Returning to the original suggestion above.
Any objections? let's finalize this.

Should a PR associated with this be in sbom-convert and not protobom?

I think the translation issues should continue here.

Decision:
Since this field allows for describing the tool, we stick to the decision that the SBOM we created is a new one. Thus:
The sbom Translator tool should populate this field with the string describing itself.
In addition, the user can override this string by user input.

We could also expose in the translation CLI a flag to --preserve-author or something like that.


Redis Collections Not release

https://github.com/pinpoint-apm/pinpoint-c-agent/blob/d42524d2b51b9d7de080133b75b7de28c14d81fc/plugins/PY/pinpointPy/libs/_pyRedis/PyRedisPlugins.py#L63
when i use redis pipeline function, this code pipeLine.connection_pool.get_connection('') will make new connection, but not release
i change the code to str(pipeLine) it's ok
my code

redis_client = redis.Redis(host='<IP_ADDRESS>', port=6379,password='123456', db=0)
pipe = redis_client.pipeline(transaction=True)
pipe.get('test_key')
values = pipe.execute()

@quicksandznzn
Thanks your report, we will check this problem.

When will pinpointPy 1.2.1 be released? @eeliu

@quicksandznzn
Can you help us test it ?
pip install -i https://test.pypi.org/simple/ pinpointPy==1.3.3

@quicksandznzn
Can you help us test it ?
pip install -i https://test.pypi.org/simple/ pinpointPy==1.3.3

https://github.com/pinpoint-apm/pinpoint-c-agent/blob/469dc3c353d803acc63b3c3955a41dd13521c70c/plugins/PY/pinpointPy/libs/_pyRedis/PyRedisPlugins.py#L65
pipeLine.connection is none , because pipeline.execute function will releases the connection
`    def execute(self, raise_on_error=True):
"""Execute all the commands in the current pipeline"""
stack = self.command_stack
if not stack and not self.watching:
return []
if self.scripts:
self.load_scripts()
if self.transaction or self.explicit_transaction:
execute = self._execute_transaction
else:
execute = self._execute_pipeline
    conn = self.connection
    if not conn:
        conn = self.connection_pool.get_connection("MULTI", self.shard_hint)
        # assign to self.connection so reset() releases the connection
        # back to the pool after we're done
        self.connection = conn

    try:
        return conn.retry.call_with_retry(
            lambda: execute(conn, stack, raise_on_error),
            lambda error: self._disconnect_raise_reset(conn, error),
        )
    finally:
        self.reset()`

maybe use this code
str(pipeLine.connection_pool) #ConnectionPool<Connection<host=<IP_ADDRESS>,port=6379,db=0>>  str(pipeLine.connection_pool._available_connections[0]) #Connection<host=<IP_ADDRESS>,port=6379,db=0>

@quicksandznzn
Can you help us test it ?
pip install -i https://test.pypi.org/simple/ pinpointPy==1.3.3

https://github.com/pinpoint-apm/pinpoint-c-agent/blob/469dc3c353d803acc63b3c3955a41dd13521c70c/plugins/PY/pinpointPy/libs/_pyRedis/PyRedisPlugins.py#L65
pipeLine.connection is none , because pipeline.execute function will releases the connection
https://github.com/redis/redis-py/blob/de8b5709a7d2022deff55034fe7dabc5770225a2/redis/client.py#L1529
maybe use this code str(pipeLine.connection_pool) or str(pipeLine.connection_pool._available_connections[0])

i think str(pipeLine.connection_pool) is good for me
debug log is
add clue key:dst value:ConnectionPool<Connection<host=<IP_ADDRESS>,port=6379,db=0>>

Thanks again, I will dig the source code  more deep.

Thanks again, I will dig the source code more deep.

Please @ me, if you have any progress .

@quicksandznzn
please check this https://github.com/pinpoint-apm/pinpoint-c-agent/pull/617

@quicksandznzn

@quicksandznzn
please check this #617

Test version released
pip install -i https://test.pypi.org/simple/ pinpointPy==1.3.4

Released

pip install pinpointPy==1.2.1


Unable to determine game data

Describe the bug
I am unable to use hyprpanel and overview of workspaces on hyprland.
To Reproduce
just using super+A [super=windows key] or use ags command on terminal
Expected behavior
Over view of current 10 workspaces and hyprpanel should have run.
Screenshots

Desktop (please complete the following information):

Distribution: [ Arch Linux]
Window Manager/Desktop Environment: [Hyprland]

Additional context
At first things worked fine except the workspace overview. HyprPanel was working fine except few features. so i decided to delete the folder and install again [Yes, this was bad]. And now im stuck with this issue.
I did delete ags and reinstall it multiple times but did not work. I dont really know how ags works. So please do help me solve this is simple steps.

You installed the wrong package. You seem to have ran this command:
yay -S ags

in order to install ags.
But that is the wrong package. You need to install aylurs-gtk-shell-git which is done via:
yay -S aylurs-gtk-shell-git

You should remove ags (adventure game studio) if you don't plan on using it so it doesn't cause further confusion.

Thank you Mr it started working.

@BlueDragon320 Nice, are we good to close this issue?

well hyprpanel is working but workspace overview isn't working.  Is there any way i can use both hyprpanel and the workspaceoverview [Photo attached].
This is the only issue else everything works fine.

Where is that workspace overview coming from?

ja's hyprland i guess may be its a feature. I asked this on discord the answer i got is " i can use only 1 among hyprpanel or this workspace, may be its because of ags" if i want to use that workspaceoverview i need to make that hyprland's ags file as default for ags.

At the end of the day it is really just AGS. You could bring in his Overview module if you wanted to and have it work in this system. You'd just have to do it manually but it's possible.

Thank you for letting me know i will try doing that


[EngSys] Enable Rush build cache for for common packages

See more details about rush build cache at https://rushjs.io/pages/maintainer/build_cache/
In short, this enables build cache that works even after switching branches. And CI can be configured to populate the cloud cache after each commit is merged into the main branch.

API change check
API changes are not detected in this pull request.

API change check
API changes are not detected in this pull request.

This is really cool - I assume it works with the local cache and falls back to the storage account only if needed? How big is what it puts into the container?

This is really cool - I assume it works with the local cache and falls back to the storage account only if needed? How big is what it puts into the container?

It may try downloading from cloud first then fall back to local then actually build but I could be wrong.  They are not too big (just tar balls of these output folders). The total for these packages is about 1600 KiB.
I suppose us developers can set a local secret of read-only SAS to the blob storage in our local environment, while our internal CI can override the setting to write to the cloud cache.


Xcode 13 build failed

Hi @tr3v3r
I am facing the issue with Xcode 13 Version
node_modules/react-native-esc-pos-printer/ios/PrinterSDK/libepos2.a' for architecture arm64

the same

any fixes ?

Hi All I fixed this issue by,

delete Pod file and pod.lock file
delete derived data folder (by preference -> location -> derived data folder)
XCode -> target file -> excluded architecture -> both debug and release  -> Any Ios simulator SDK -> add "arm64"
Quite XCode
right click on Xcode -> info -> check open by Rosetta
restart your macbook
clean and run it will work

Same problem, and not fixed by your suggestion @AzranAzwer.
Owner? @tr3v3r

I tried to run it on a real device and now there is no error.
I think it has a problem with the simulator.


300 |  Longest Increasing Subsequence | C++

Title - Longest Increasing Subsequence
What will change -
Adding Code for a new problem
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[ ] Improving Code
[ ] Improving Documentation
[ ] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[ ] Python
[x] C++
[ ] Java
[ ] SQL

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

I am interested in doing this problem. Can you please assign it to me?

Assign to me, please.

@SSKale1 @Yashica7 can you pls assign me this task


Add a trailing newline to fix GO 1.20 test compile error

Fixes an "syntax error: unexpected var after top level declaration" error when running 'go test -cover -covermode=atomic ./...' with GO 1.20.
Related to GO issue: https://github.com/golang/go/issues/58370

LGTM


codecov token

This should fix the build

PR-Codex overview
This PR updates the codecov.yaml workflow file to use a secret CODECOV_TOKEN and specifies the location of the Jacoco XML file.
Detailed summary

Updated codecov.yaml to use ${{ secrets.CODECOV_TOKEN }}
Specified the location of Jacoco XML file as ./target/site/jacoco/jacoco.xml
Set fail_ci_if_error to true

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 15min)


ERROR: 18: bad-txns-joinsplit-requirements-not-met

Hello,
I have quite a bit of ZEN (at least for me) that is currently locked up in my Swing Wallet.  When I try to transfer cash, I get "ERROR: 18: bad-txns-joinsplit-requirements-not-met".
I've been able to access some of my cash by installing the Arizen wallet, but the majority of my holdings are in the private (z-address) wallet, and I can't import those into Arizen - or move them.
Any ideas on how to fix this error?  Or at least access my coins?

Save your zaddress private key.
Rename your wallet.dat to old-wallet.dat and back it up too.
Start your swing wallet again, it should create a new wallet.dat
Import your z-address private key and try sending your coins to a t-address.

If that does not work
You need to run open a command line window and type zencashd -rescan
So basically you need to run the Zencash daemon with the -rescan argument.

Indeed the problem of error message:
ERROR: 18: bad-txns-joinsplit-requirements-not-met

is not a GUI problem but comes from the ZEN core/node. Actually it also exists for other Z-currencies too. Rescanning the blockchain (run the Zencash daemon with the -rescan argument) usually solves it...

@zennermac should be able to close this one.


Different sample rate in melspec computation

In the melspectrograms.py script the loading sampleRate is set to 12000. However, in get_npy.py sample rate is 16000. Is this downsampling just for faster computation or is there another reason for this that I'm missing?

This is a mistake inside get_npy.py. This value is computed for information purposes only, the duration is not used anywhere else in the code for the baseline.


rest: Workaround Http2 zero-length reply bug

What does this PR change? What problem does it solve?
The golang http client does not return an error when a HTTP2 reply includes a non-zero content length but does not return any data at all. This scenario can occur e.g. when using rclone when a file stored in a backend seems to be accessible but then fails to download.
This can cause errors like "Pack ID does not match, want $HASH, got e3b0c44" as reported in #2742.
The underlying issue is a bug in the go http client, which is also tracked as https://github.com/golang/go/issues/46071 . However, it is unknown when a fix will be available and I don't expect it to be backported to all go versions still supported by restic.  I've added a test that will fail once the underlying issue in the http.client has been fixed, to allow us to eventually remove the workaround again.
Was the change previously discussed in an issue or on the forum?
Fixes #2742 .
See also https://forum.restic.net/t/http2-stream-closed-connection-reset-context-canceled/3743/10 .
Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[x] I have added tests for all code changes.
[ ] I have added documentation for relevant changes (in the manual).
[x] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[x] I have run gofmt on the code in all commits.
[x] All commit messages are formatted in the same style as the other commits in the repo.
[ ] I'm done! This pull request is ready for review.

The cleanup looks fine now.


Collect perf report data from target process

This commit sets up a perf record sub-process that monitors the target process
if the user gives the --target-perf-data-path flag. I haven't done anything to
ensure this will actually work, which it won't depending on your Linux sysctl
settings and will never work on OS X or Windows. Still. I have had to undo some
of #156 so that perf can shutdown correctly, pull in Nix to avoid SIGKILL
signals to both the target and perf. This means the PR is blocked until #155 is
resolved.
Signed-off-by: Brian L. Troutwine<EMAIL_ADDRESS>

I think this is overly specific. Having an analysis sub-target in the configuration makes more sense to me now that I've had some time to let this sit. Allow the user to specify a path to a shell script or what not: path + args + environment variables?

I think this is overly specific. Having an analysis sub-target in the configuration makes more sense to me now that I've had some time to let this sit. Allow the user to specify a path to a shell script or what not: path + args + environment variables?

Corrected with a89637c

Oops, accidentally closed this with #167. Apparently you can't just close a comment.


Test scan

Description
Add feature flag to validate package creation and avoid installedPackage growht.
Add omit to avoid dup
Create commit_installed_package to be able to reuse installed_package_id from previous package created
Added test scenario
The goal is reduce the exponential data creation, we should do one more refactor to fix m2m relationship and clean up the 26gb of data
FIXES: THEEDGE-2648/2469
Type of change
What is it?

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Codecov Report
Base: 52.53% // Head: 53.56% // Increases project coverage by +1.03% :tada:

Coverage data is based on head (3cd9eb5) compared to base (3fcee09).
Patch coverage: 85.00% of modified lines in pull request are covered.

:exclamation: Current head 3cd9eb5 differs from pull request most recent head 3ac275c. Consider uploading reports for the commit 3ac275c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1885      +/-   ##
==========================================
+ Coverage   52.53%   53.56%   +1.03%
==========================================
  Files          70       70
  Lines        9871     9963      +92
==========================================
+ Hits         5186     5337     +151
+ Misses       4212     4145      -67
- Partials      473      481       +8

Impacted Files
Coverage Δ

pkg/models/commits.go
100.00% <ø> (ø)

pkg/clients/imagebuilder/client.go
56.58% <85.00%> (+16.11%)
:arrow_up:

pkg/common/kafka/kafkaconfigmap.go
100.00% <0.00%> (ø)

config/config.go
58.86% <0.00%> (+15.61%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

/retest

/retest


TaskSet should not be sampled by TaskSet_Object.x method, it shotcut task specific transformations

Indeed, furthermore .x should be made private ._x.

Done in https://github.com/Continvvm/continuum/pull/29


OAI-PMH Object Converter

Convert the a collection into OAI-PMH compliant format.
Take the metadata (possibly in JSON) and convert it to the format needed by OAI-PMI

Do not include metadata inherited from collections.


Cost calculation: John Jasa's updates to the WISDEM are merged.

Purpose
Merging cost calculation feature from John Jasa's WISDEM dev
Type of change

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (non-backwards-compatible fix or feature)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Documentation update
[ ] Maintenance update
[ ] Other (please describe)

Testing
None
Checklist
Put an x in the boxes that apply.

[ ] I have run existing tests which pass locally with my changes
[ ] I have added new tests or examples that prove my fix is effective or that my feature works
[ ] I have added necessary documentation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 9 (77.78%)  changed or added relevant lines in 3 files are covered.
23 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 80.308%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

weis/aeroelasticse/openmdao_openfast.py
3
5
60.0%

Files with Coverage Reduction
New Missed Lines
%

weis/aeroelasticse/LinearFAST.py
4
90.0%

weis/aeroelasticse/runFAST_pywrapper.py
19
69.92%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
16839

Relevant Lines:
20968

💛  - Coveralls


ValueError: Must install dask-expr to activate query planning

Occurs here when importing pyam, via (among others)

pyam-iamc 2.1.0,
ixmp4 0.7.2 (the bug may be properly targeted there; I'm not sure),
pandera 0.18.3, and
dask 2024.3.0.

This distinct package was announced in December: https://docs.dask.org/en/latest/changelog.html#logical-query-planning-now-available-for-dask-dataframes in 2023.12.1. This error appears to be new with 2024.3.0, which was released today.
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/genno/compat/pyam/__init__.py", line 2, in <module>
    import pyam  # noqa: F401
    ^^^^^^^^^^^
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/pyam/__init__.py", line 18, in <module>
    from pyam.iiasa import read_iiasa, lazy_read_iiasa  # noqa: F401
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/pyam/iiasa.py", line 27, in <module>
    import ixmp4
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/ixmp4/__init__.py", line 4, in <module>
    from ixmp4.core import Model as Model
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/ixmp4/core/__init__.py", line 2, in <module>
    from .iamc.variable import Variable as Variable
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/ixmp4/core/iamc/__init__.py", line 3, in <module>
    from .data import RunIamcData, PlatformIamcData
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/ixmp4/core/iamc/data.py", line 4, in <module>
    import pandera as pa
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/pandera/__init__.py", line 4, in <module>
    import pandera.backends
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/pandera/backends/__init__.py", line 6, in <module>
    import pandera.backends.pandas
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/pandera/backends/pandas/__init__.py", line 5, in <module>
    import pandera.typing
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/pandera/typing/__init__.py", line 9, in <module>
    from pandera.typing import (
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/pandera/typing/dask.py", line 9, in <module>
    import dask.dataframe as dd
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/dask/dataframe/__init__.py", line 87, in <module>
    if _dask_expr_enabled():
       ^^^^^^^^^^^^^^^^^^^^
  File "/home/runner/actions-runner/_work/_tool/Python/3.11.2/x64/lib/python3.11/site-packages/dask/dataframe/__init__.py", line 24, in _dask_expr_enabled
    raise ValueError("Must install dask-expr to activate query planning.")
ValueError: Must install dask-expr to activate query planning.

This is indeed an issue related to ixmp4, https://github.com/iiasa/ixmp4. Note that the issue can’t be transferred because it belongs to a different organization. Please close and open a corresponding issue there.

Taking care of this with this PR: https://github.com/iiasa/ixmp4/pull/61

Should be resolved by ixmp4 v0.7.3.


Add: TextVenture - A text based terminal game

Description
A summary of what is included in your Pull Request.
Fixes #(issue_no)

[ ] Issue_no: <insert your issue no here>
[x] Not applicable

Type of change
You are choosing one or more options from the following per the nature of your Pull request.

[x] New project insert
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation Update

Project

Your Project Name:

TextVenture

Short Description: Provide a short description of your Python project

A simple text-based game to crush your boredom and that too in your terminal.
Checklist:
Please tick all the boxes that are fulfilled by your Pull Request.

[x] I have named my files and folder, according to this project's guidelines.
[x] My code follows the style guidelines of this project.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy-to-understand README.md, according to the given README_TEMPLATE.
[x] My changes do not produce any warnings.

Could you please add HacktoberFest tags?
Thanks for accepting!


Final Notes

[x] Please capitalize data

[ ] Please order columns:

UID

First name

Last name

If demographic info exists

Start/end dates

Employers

Reason for separation if exists

[x] Please add to dropdown:

Georgia Discipline

Oregon

South Carolina

Idaho

[x] Please have "National Police Index" header bring you back home

[x] Birth year decimals:

Florida has a birth year decimal

Kentucky has a birth year decimal

Ohio has a birth year decimal

[x] Tennessee "reason for separation" should be "employment change"

[x] Utah "reason for separation" should be "employment status"

[x] Vermont "reason for separation" should be "employment change"

[x] Wyoming "reason for separation" should be "employment status"

Holding for launch:

[x] New Mexico

On further review, I think we should keep Maryland. It's ok that it might show multiple rows within the same department for an officer and that it's a little inconsistent with the other states. It's still useful and accurate.

Done except for column order. We should re-discuss column order bc of the inclusion of discipline data for GA and FL.

New items noticed week of:

[ ] Dropdown cuts off the bottom, doesn't allow for scrolling down
[ ] I still see birth year decimals for Kentucky, Ohio


is there a way to restart?

if this subscription gets interrupted or my server restarts/etc, that's running this...
I'll get an error when trying to restart this again.
It says the "id" already exists and errors out.
Is there any way to restart this program and have it continue with the next incremental id in the mysql table?

Currently you have to archive the original table into another name and restart the application.
In the earlier version the program will restart from the last id from the existing table. But then I removed it for performance and simplicity. Perhaps it is the right time to implement it back.

is this something that I can change on my end with the code, so you don't have to spent time with this?
Thank you again for the advice and support!

Sorry that it might not be an easy task to recover the last id from the table, as it involves reading back the database.
However, off my head, the workaround to create a "mostly" unique key may work. The table id can be created as
return self.exch_snapshot_id + (datetime.now() - datetime.strptime("2017-01-01", "%Y-%m-%d")).total_seconds() * 100

Awesome! Thank you for the quick reply and suggestion! Much appreciated.


Consider perl Data::Frame

Zaki Mughal
18:31
@p6steve
, I'll have to check this out! I wrote the Perl5 Jupyter kernel and Data::Frame library. We can probably share some ideas. Glad to see Raku being used for stats.
p6steve
18:35
hi
@Zaki Mughal
as mentioned in my talk this is early days for the raku Dan module(s) and I welcome any and all guidance, collaboration and feature requests! this is the time to shape how it looks…
18:37
I will check out perl Data::Frame and have a think about how to bring together...

Hi, will take a look!


Wrong message on the 'component groups' tab when empty

Just a minor issue. When you are at the 'component groups' tab, but there isn't a component group yet, it says "You should add a component.". I think this should be "You should add a component group", since there were actually components added.

Good catch!


the CLI usage of this tool is broken - Missing required keys: owner, repo

$ npx github-action-readme-generator
Missing required keys: owner, repo
$ npx<EMAIL_ADDRESS>--readme /home/denis/huma/huma-cloud-infrastructure/github-actions/dump-mongodb-to-gcp-bucket/README.md --action /home/denis/huma/huma-cloud-infrastructure/github-actions/dump-mongodb-to-gcp-bucket/action.yaml
Missing required keys: owner, repo

@denist-huma  I have addressed this in version v1.2.3 - but you need to use --owner  --repo
You only need to do this once, as it will save the details in the config file.
{
  "paths": {
    "action": "action.yml",
    "readme": "README.md"
  },
  "show_logo": true,
  "versioning": {
    "enabled": true,
    "override": "",
    "prefix": "v",
    "branch": "main"
  },
  "owner": "bitflight-devops",
  "repo": "github-action-readme-generator",
  "title_prefix": "GitHub Action: ",
  "pretty": true
}

@Jamie-BitFlight thank for your responce, I can give a new try with this new keys.
I still don't see any mention about "--owner --repo" keys in the https://github.com/bitflight-devops/github-action-readme-generator#cli-usage README.md.

Ah yes. In most cases it detects those values. Like if the process is run from git root, or in a GitHub action, it automatically grabs those values.
I’ll update the readme, but also I can make it also look for a package.json in the location of the action.yaml and try to get the values from there.
Regards,
Jamie Nelson
On Aug 10, 2022 at 5:56 PM, <Denis Trofimov @.***> > wrote:
@Jamie-BitFlight thank for your responce, I can give a new try with this new keys.
I still don't see any mention about "--owner --repo" keys in the https://github.com/bitflight-devops/github-action-readme-generator#cli-usage https://github.com/bitflight-devops/github-action-readme-generator#cli-usage README.md.
—
Reply to this email directly, view it on GitHub https://github.com/bitflight-devops/github-action-readme-generator/issues/162#issuecomment-1211322835 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AF7J6MBWRBW2UNHJQM5SMCLVYQQQFANCNFSM5YXWDMHQ .
You are receiving this because you were mentioned.Message ID: @.***>

fixed: #225


Remove auth fallback

resolves #464

[x] Remove fallback auth system for all endpoint
[ ] Remove old system to get user credentials to perform batch jobs

Is this PR ready for CR?


Secret 4MB update from Valve 🤫

Yep, you heard me right. 100% some pattern changed as the dumper provides the current uploaded offsets.

Issue on my end, my bad


Update storage-import-export-service.md

Hi @lakasa-msft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


AssertionError: rtail value for 1 in 2 year flood

When running an L1 assessment for multiple flood scenarios (1:2yr, 1:5yr, 1:10yr, 1:20yr, 1:100yr), I receive an AssertionError: passed rtail value (0.50) not > max aep (0.50) error. (I suspect it would also fail for L2, but haven't tested that.)
Exact error as follows:
AssertionError: passed rtail value (0.50) not > max aep (0.50)
Traceback (most recent call last):
  File "C:\Users\XXX\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\canflood\model\dialog.py", line 164, in run_risk1
    res_ttl, res_df = model.run(res_per_asset=self.checkBox_r1_rpa.isChecked())
  File "C:\Users\XXX\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\canflood\model\risk1.py", line 268, in run
    res_df = self.calc_ead(bres_df)
  File "C:\Users\XXX\AppData\Roaming\QGIS\QGIS3\profiles\default\python\plugins\canflood\model\riskcom.py", line 807, in calc_ead
    assert aep_val > df.columns.max(), 'passed rtail value (%.2f) not > max aep (%.2f)'%(
AssertionError: passed rtail value (0.50) not > max aep (0.50)

Removing the 1:2yr scenario and updating related elements clears the error.
Workaround:
Before running the L1 assessment, change the value for the following line in the control file from 0.5 to 0.6:
rtail = 0.5   #EAD extrapolation: right trail treatment (high prob low damage)
While this workaround helps, we will be conducting several coastal flood studies with modelling for the 1:2yr event for several locations across Canada and it would be nice to get this working by default for high probability, low impact events.

Hi Ryan, sorry to hear you're having issues with the rtail parameter.  This tells CanFlood how to complete the right tail of the risk curve. By specifying a value, you're telling CanFlood "at this probability there is zero damage"... so it throws an error when you also include an event of equal or greater probability in the event table.  Therefore, you should specify a higher value...as you've done. i.e., this is not a workaround but working as intended. We'll try and clarify the documentation.
the parameter is described here.
Let me know if this clarifies things and I'll close the issue.

Hi Seth, thanks for the quick feedback!
I agree that an update to the documentation to clarify would be very helpful. That said, there is nothing the user experiences at any point leading up to the error that would lead them to expect such an error. There is also nothing in the error message that would provide them hints on where to go to address the issue themselves. So, might I suggest the following changes as well:

This could be flagged on the 'Event Variables' tab for ARI values <= 2 (not sure if that's an option within the UI)
This should be flagged during the validation stage, with a message providing details on how to address it.
The error message should also be updated to more clearly describe what is happening and how to address it.

Beyond the above, while 0.5 seems quite a reasonable limit for most situations, it is arbitrary and can therefore be changed. Depending on how things are set up on the backend, using a greater-than-or-equal check instead or equals (i.e, rtail value (0.50) ≥ max aep (0.50)) or changing to a slightly higher default value will also guarantee this issue doesn't occur at all without any significant impact on other useres. This would help user experience to better align with user expectations. Just my 2 cents.

Hi Ryan, thank you for the feedback. This is very useful. I added a suggestion to the error message and a tooltip to the build dialog - event variables tab (see PR 70). A nicer fix would probably be to add the parameter (rtail and ltail) directly to the UI (but this would take more effort). Adding a check to the validation tool (which is a good idea as this is what it's there for) would also take more effort as this tool is under-developed.
As for the parameter itself, it needs to be greater than the largest AEP (most probable) event specified in 'evals' (> not >=).
0.5 (2-year) is the value in the control file template, which acts as a default when building a new model from the UI (you can customize this by editing this file). Along with changing this value, there are four other key word options (see control file summary) the user could specify in the control file.
Apart from this, I'm a bit suspicious of using such a small/probable event in your model. Do you really have impacts in half of all previous years?
Cheers,

Thanks Seth, those sound like reasonable future changes. I'll definitely modify the pars file.
Unfortuntely we are seeing impacts from these "regular" events in our work in flat-laying areas in the Arctic today, resulting in damage to homes or the need to relocate homes as coastal erosion increases. Of greater concerns is that a 1:2yr flood starts to cover the same areas that a 1:5, 1:10, or even 1:20 year flood does today as we incorporate sea level rise into our future-looking scenarios in these areas.
Thanks again!


chore(superset-frontend): remove unused @superset-ui/plugin-chart-period-over-period-kpi package

SUMMARY

DEPS: Remove  @superset-ui/plugin-chart-period-over-period-kpi package from the dependency list (Related to #27993)

ADDITIONAL INFORMATION
Fixes #27993

@michael-s-molina, you are welcome! And thanks for your approve. But can you do it again?
Because I changed the PR's title, and the link-check workflow waited an approval from a maintainer again:

Thanks!


meta: add release-please-action for automated releases

Checklist

[x] npm install && npm test passes
[x] documentation is changed or added (sort of, see the updated Pull Request template)
[x] commit message follows commit guidelines

Description of change

Adds release-please-action to this repository. release-please-action automatically creates/updates release proposal PRs when there are new commits on a given branch of the repository (currently configured as master). If the release proposal PR is merged, release-please-action will publish a corresponding entry in GitHub Releases, and add the new git tag for that version to the repository.
@gengjiawen and I added various NodeJS-style commit prefixes to be recognized in addition to the usual chore:, fix:, and feat: prefixes defined in Conventional Commits. (Conventional Commits is the commit message spec release-please is based around.) These (and any arbitrary) prefixes are valid per Conventional Commits and release-please, but would be ignored by default -- configuring them as recognized ensures they are not excluded from the generated Changelog.
Semver hints in commit messages can let release-please know whether to draft a minor or major release; without those, the release proposal PR will be for a semver patch-level release by default.
This PR also updates the MarkDown formatting of CHANGELOG.md so that older entries will not be overwritten (deleted) in the new release proposal PRs.
Finally, this PR updates the "commit guidelines" link in the PR template to point to instructions for release-please-style commits, rather than pointing to the NodeJS commit guidelines.
Resolves: https://github.com/nodejs/node-gyp/issues/2286
Closes: https://github.com/nodejs/node-gyp/pull/2330

Thanks for dedicated contribution :)


Add Streaming mechanism for ChatBedrock Class

ChatBedrock currently implements _streamResponseChunks however the _call function always calls _signedFetch and I don't see how _streamResponseChunks is used anywhere.
This is a feature request to allow ChatBedrock to be configurable for streamed or non-streamed responses (in the same way the streaming parameter is used in ChatOpenAI) and use _streamResponseChunks/_signedFetch accordingly.
Happy to assist if you like

Hey @ServerlessSam, you should be able to call the .stream() method on ChatBedrock.
The *_streamResponseChunks method is called inside a parent class of ChatBedrock. If you go into BaseChatModel class and search *_streamResponseChunks you'll see it's called by *_streamIterator which is in turn called by stream

@bracesproul
Hey [@ServerlessSam](https://github.com/ServerlessSam), you should be able to call the .stream() method on ChatBedrock.
The *_streamResponseChunks method is called inside a parent class of ChatBedrock. If you go into BaseChatModel class and search *_streamResponseChunks you'll see it's called by *_streamIterator which is in turn called by stream

Thanks for the response (did you delete it btw? Not sure why it's not here?).
I see *_streamIterator in the BaseChatModel but not stream that is calling it.
For clarity my use case is to pass ChatBedrock into a ConversationChain so I can also provide the memory argument to keep the conversation's context. I'm then calling ConversationChain.stream and I can see the HTTP request to AWS is for the invoke method and not invoke-with-response-stream because we are calling ChatBedrock._call which calls ChatBedrock._signedFetch with a hardcoded bedrockMethod: "invoke".
Is there a way to provide memory to ChatBedrock.stream?
The docs (https://js.langchain.com/docs/api/chat_models_bedrock/classes/ChatBedrock) don't make it clear to me what args are accepted in .stream

Hey @ServerlessSam, _streamResponseChunks is called from the base class here (has to do with some iterator vs. stream concerns): https://github.com/langchain-ai/langchainjs/blob/main/langchain/src/chat_models/base.ts#L134
It sounds like you're trying to use ChatBedrock in a ConversationChain? If so, you'll need to use our streaming method that works with legacy chains - the new .stream() method is currently just for expression language sequences:
https://js.langchain.com/docs/modules/chains/foundational/llm_chain#usage-in-streaming-mode
https://js.langchain.com/docs/expression_language/

Closed but reopen if the above isn't the case

Seems like this issue is still not fixed.
Here is my code,
  constructor(chatConfig: ChatBedrockConfig) {
    this.chatBedrock = new ChatBedrock({
      model: chatConfig.model || config.bedrockModelName,
      region: config.awsRegion,
      streaming: true,
      temperature: chatConfig.temperature || 0.6,
      maxTokens: chatConfig.maxTokens || parseInt(config.maxTokens.toString()),
    });

    this.chatPrompt = ChatPromptTemplate.fromMessages([
      [
        'system',
        'You are an assistant that answers questions based on text {text}.',
      ],
      ['human', '{prompt}'],
    ]);
  }

  async handleConversation(context: string, text: string): Promise<void> {
    const chain = new LLMChain({
      prompt: this.chatPrompt,
      llm: this.chatBedrock,
    });

    const stream = await chain.stream({
      text: context,
      prompt: text,
    });
    // TODO: send to websocket
    console.log('streaming started');
    for await (const chunk of stream) {
      console.log(chunk);
    }
  }

and the log is:
streaming started
{
  text: ' Here is one way to rewrite that text:\n' +
    '\n' +
    'My AI assistant has the remarkable capacity to swiftly analyze enormous amounts of patient data, including medical records and imaging results. This enables the AI to diagnose complex conditions in seconds, whereas it could take human doctors hours or even days to do the same. As a result, the AI facilitates earlier detection of conditions and prompt medical intervention, which can greatly improve patient health outcomes.'
}

you can see that the output is still not a streaming message.


deepEqual broken when comparing objects without enumerable properties or null

sinon.deepEqual returned true for (null, {}), (/.*/, {}) and (new Error(), null).

Sweet, thanks!


Fixed #25903 -- Fixed the admin's list_editable add/change buttons.

https://code.djangoproject.com/ticket/25903

buildbot, test on selenium.


memory cgroup count leak?

Description
Every time a new pod is created, the counter for memory slice in /proc/cgroups is increased by 2
Steps to reproduce the issue:
Running the following command
cat /proc/cgroups;crictl rm crictl ps|grep filebeat|awk '{print $1}';sleep 3; cat /proc/cgroups
Replace filebeat with the pod of your choice
Describe the results you received:
#subsys_name	hierarchy	num_cgroups	enabled
cpuset	12	16	1
cpu	5	25	1
cpuacct	5	25	1
blkio	6	25	1
memory	8	32	1
devices	10	55	1
freezer	11	16	1
net_cls	3	16	1
perf_event	7	16	1
net_prio	3	16	1
hugetlb	2	16	1
pids	9	15	1
rdma	4	1	1
abfd4d71b4a58
#subsys_name	hierarchy	num_cgroups	enabled
cpuset	12	16	1
cpu	5	25	1
cpuacct	5	25	1
blkio	6	25	1
memory	8	34	1
devices	10	55	1
freezer	11	16	1
net_cls	3	16	1
perf_event	7	16	1
net_prio	3	16	1
hugetlb	2	16	1
pids	9	15	1
rdma	4	1	1
Describe the results you expected:
I'd expect the memory number to remain the same
Additional information you deem important (e.g. issue happens only occasionally):
I've tried all combination of the following configurations:
crio - 1.10.5 and 1.11.6
kubelet - 1.10.7 and 1.11.3
cgroup manager: systemd and cgroupfs
All results in the same behavior
Output of crio --version:
crio version 1.11.6
Kubernetes v1.11.3

Additional environment details (AWS, VirtualBox, physical, etc.):
We are running bare metal with centos 7
uname -r
4.14.35-1818.2.1.el7uek.x86_64

but I've tried it with various kernels as well (el and uek)

That number is the number of cgroups on the system -- it doesn't have anything to do with memory usage (other than the couple of bytes in memory that the kernel needs to represent the cgroups). Why is it 2? I'm not sure, but I'm sure that a quick look inside /proc/self/cgroup inside a container would probably tell you whether 2 cgroups are being created for each pod (I think that there's a pod-level cgroup and then a container-level one below it).

So only one is created, but my guess goes for the pause container for creating one as well

@aalubin @mrunalp Is this still an issue?

@rhatdan we no longer use cri-o so I don't know


OpenTracing support for grpc c++

What version of gRPC and what language are you using?
1.6
What did you do?
Is the census code ready to use with OpenTracing / Jaeger? Are there any examples to use the census code in C++?
https://github.com/grpc/grpc/tree/master/src/core/ext/census
What did you expect to see?
Documentation or status report of the census extension. gRPC, OpenTracing and Jaeger are all CNCF projects, and I expect to see them integrate with each other. Thanks.

Does https://github.com/grpc-ecosystem/grpc-opentracing provide what you need?

https://github.com/grpc-ecosystem/grpc-opentracing currently has only gRPC, Java, and Python implementations.

Thanks @calder, grpc-opentracing does not have a c++ implementation yet.

By the way, I found that all census code were removed in bb7d088ed2ea79bdad2917b7be86bad622d1f790.

It seems like work on opentracing-cpp and opentracing-csharp have finally started :)

#12613 got merged today. I will look into doing the grpc-opentracing implementation for C# when the next release including this version (v1.10.x) is done.
I think, once #9035 is takled, an grpc-opentracing implementation on grpc-core level would be much appreciated. Until then, it's hard since the C++ implementation doesn't have any interceptor functionality AFAIK. OpenTracing itself has a C++ implementation, so the only thing missing are the interceptors.

Waht a pity.
I think it was useful to add interceptor in grpc-cpp for user.

I +1 this feature request.

https://github.com/opentracing-contrib?tab=repositories is the main account for OpenTracing implementations. So better to look there for packages. Java and Python also made references on grpc-ecosystem but new once won’t appeare there.
The C++ Version is still not possible until #9035 is implemented :) so maybe also track that issue.

I'm looking into adding OpenTracing support for C++.
What's the plan with gRPC's channel_filter interface?
I believe it would be possible to add OpenTracing support with it, and it's what OpenCensus was using before its plugin was moved into gRPC's repository.
It looks like the header is available from gRPC's bazel build, but not installed by gRPC.
Is the intention to make channel_filter part of gRPC's public interface at some point and available from not just bazel?

Maybe add a filter for OpenTracing @rnburn? I imagine this is dynamically load so it won't need to be part of the gRPC source tree.

I guess that is probably based on the channel filter interface anyway. Ya I'd love to see it moved to the public API as well.

@isaachier - that's what I'm considering. But adding such a filter would require you to include the channel_filter.h header, which seems to be only available if you're using bazel. (I don't see it installed anywhere).

@mehrdada, @Vizerai - could you let us know that the future of the channel filter interface is?


refactor: make issue list buttons look better on mobile

Also removes the I also have something to say! text on desktop for consistency.

Kamilla and I don't really like this

It's better UX on mobile though. What exactly do you not like?

The button feels just too big.
And separating the tab toggle and the button into 2 lines takes up more vertical space until you reach the actual important part of the page - the issues.
On desktop we like the I also have something to say!. It gives the site some flavour, so it's not so bland.

Can you take a look at how it looks like on the phone? Sometimes the desktop makes it look bigger than it actually is. Imo the size is fine.

And what about the vertical size? It wastes a whole issue row of space

It's not that drastic since we don't have such a huge amount of issues open at the same time you can still see pretty much everything.

But we worked so much on current version i think it's perfect

No the current version was thrown together quickly

Not even true we used all our knowledge

Suuuuuure

This page was thrown together in like 10 minutes :laughing:

Hey if you want contributors don't talk shit about their contributions especially after merging them

This page was created by us during a meeting. It was the best solution back then because we needed it quickly, but this does not mean that we need to stick to this solution all the time. We can still improve it.

That is true, the reason to not merge it should not be the effort we spent on it.
We can just vote on discord what we prefer.

But the I also have something to say! button on desktop should really stay...

Since I like consistency (not displaying different content on different screens) it should be the same text.
Since the mobile button is bigger now it can also show the text I also have something to say!.
So both mobile and desktop can show the fancy text.


added docker support

added two new install_methods:  auto and docker (neither set to default).
docker now uses the official docker container to deploy but still calls config_hash and mounts the config_dir for consistency.  Image can be chosen, but network and certain key variables are hardcoded per consul recomendations for use in a container by the host.  See: https://hub.docker.com/_/consul/
auto will look for the [docker0] network interface.  If found, it uses docker, on failure it falls back to the url default.
The default remains at "url", disabling this feature.

changes made including removal of auto.  Had to make some "minor tweaks" as you can't rewrite variables in puppet but should be closer inline to what you're looking for.

took way to long to get things to pass, but there you go.

Running internally at the company against Ubuntu 14, 16, and CentOS 7.9 in both docker and non-docker configs.  Found a minor bug where a container env variable was writing a new local.json.  Removing to allow the user complete control over config through the module.

Spaces removed and data_dir added.

This should be ready for another review.  Please let me know.

This is still on my radar and I will review.
Can you confirm: are you running this code (in the PR) for real in your infrastructure? or is it a variant of it?

This code is the code deployed at my company.  We are using a mix of Ubuntu (14 and 16) as well as a few CentOS boxes, and I deploy both Docker and URL mode so I have a pretty good cross section, but we are a small shop of only ~80 boxes on premise.  It's currently used in tandem with Registrator to do most of the service registration, but I do have a few specific service callouts being done.

Alright, hitting the button...

Sweet.  Ill remove my branch and open a new one If i come up with more ideas.


Test r1curve tls

bringing up to date with k1 cluster fixes

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: joshuafernandes:white_check_mark: Ezzahhh:x: siladuYou have signed the CLA already but the status is still pending? Let us recheck it.

This isn't intended for master, my bad


Document for binary data

Most of the usage could be inferred via beanstalkd. But the encoding control needs more documents.
sample
It is controled by the encoding of Client.__init__().
When encoding is not None, the input of Client.put() should be str, and the output of Client.reserve() would be str too. Consistent encoding for both producer and consumer is neccessary.
When encoding is None, the input of Client.put() should be bytes, and the output of Client.reserve() would be bytes too. Consistent encoding (None) for both producer and consumer is neccessary.
Behaviours under other settings are undefined.

Hey @zhengwx11, I'm working on improving the docs and should hopefully have something up soon. I'll definitely make sure to mention this, thank you!

Added some docs: https://greenstalk.readthedocs.io/en/latest/quickstart.html#body-encoding


[zh] Fix change PV reclaim policy task

The original English text is missing ...

/assign @xichengliudui

/approve


Typo fix

Thanks @Swader for spotting that one. It is addressed in a merged PR#662 https://github.com/0xPolygon/polygon-docs/pull/662


noSsr

In options property of useMediaQuery, it's noSsr, while in withWidth it's noSSR. I know withWidth will be deprecated but this difference may confuse developers and lead to bugs.

@mkermani144 Thanks for the feedback. This is a broader concern regarding how we handle acronyms. noSsr is consistent with the rest of our API.


Agent did not support on node latest LTS (v18.14.2)

Update
it seems like a bug of node

Environment
ofetch: v1.0.1
node:v18.14.2
Reproduction
here is a minimum recurrence demo
import { getProxySettings } from "get-proxy-settings";
import { ofetch } from "ofetch/node";
import httpAgent from 'http-proxy-agent'
import httpsAgent from 'https-proxy-agent'

let agent
await (async function setSystemProxy() {
  const { http, https } = (await getProxySettings()) || {}
  function agentGenerator(ProxySetting) {
    const { protocol, host, port } = ProxySetting
    const agentUrl = `${protocol}://${host}:${port}`
    if (protocol === 'http') {
      return new httpAgent(agentUrl)
    }
    if (protocol === 'https') {
      return new httpsAgent(agentUrl)
    }
    return false
  }
  if (http) {
    agent = agentGenerator(http)
  }
  if (https) {
    agent = agentGenerator(https)
  }
})().then(async () => {
  const response = await ofetch.raw('https://google.com', { agent })
})

Describe the bug
Nodejs has supported fetch api since v17.5.0
This caused current issue:
export const fetch = globalThis.fetch || createNodeFetch();

I'm going to submit a PR to fix this bug!
Additional context
No response
Logs
No response

it seems like a bug of node

I bumped into this problem today when using Node 16 the "agent" option works as expected but when using Node 18 the "agent" option does not seem to work. Is this a known issue with ofetch or am I missconfiguring something? :)

Node.js 18 uses undici for fetch. It seems they use dispatchers (like this). PR for docs is welcome if anyone can investigate further.

Hey @everchanger @pidanmeng, this isn't 100% your situation(s) but here's a stripped-down example of how I solved a localhost self-signed certificate issue in development:
import { Agent } from 'undici'

try {
  const response = await $fetch('/my/api/call', {
    ...(process.env.NODE_ENV === 'development' && {
      dispatcher: new Agent({
        connect: {
          rejectUnauthorized: false
        }
      })
    })
  })
  console.log(response)
} catch (e) {
  console.log(e)
}

I hope this helps set you on the right path!

Agent did not support on node v21.4.0

You should use dispatcher in modern node (https://github.com/unjs/ofetch#-adding-https-agent)


CMake mxnet with cpp-package compile  Failed on MacOS

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System: MacOS
Compiler: gcc(clang)
Package used (Python/R/Scala/Julia):
MXNet version: 0.11r3
Error Message:
nstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin
 "/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/ld" -demangle -lto_library /Applications/Xcode.app/Contents/Developer/Toolchains/X
codeDefault.

Minimum reproducible example
cmake -D USE_CUDA=OFF -D USE_OPENMP=OFF -D USE_OPENCV=OFF -D USE_CUDNN=OFF -D USE_CPP_PACKAGE=ON ../
make

What have you tried to solve it?
build mxnet without cpp-package succeed.

commit c1beac566f898a122d53a19fef32d69497180251 build with cpp-package succeed, close this issue.


Inconsistent types reported by IntelliSense

Using type hints:

The mouse hover tooltip reports a different type for the vector parameter Vector vs. the method return type list.
The signature info in the second tooltip has vector parameter type list.

If I eliminate all calls to scale, reload project then this is the signature I get, with no type for vector parameter, as if it ignored the type hint and only went by usage:

It looks good now

For reference, the code:
from typing import List

Vector = List[float]

def scale(scalar: float, vector: Vector) -> Vector:
    return [scalar * num for num in vector]

new_vector = scale(2.0, [1.0, -4.2, 5.4])

Hello,
I am having some trouble with a similar situation. I am interested in type-hinting a property of a Generic[T] container, but it looks like Generic[T] does not work properly (or its my fault).
Based on the example in the comment above:
from typing import TypeVar, Generic, Iterable, List

T = TypeVar('T')

class MyCollection(Generic[T]):
    def __init__(self, col:Iterable[T]):
        self.col = col

    @property
    def first(self) -> T:
        return self.col[0]

    def get_last(self) -> T:
        return self.col[-1]

def to_list(iter: Iterable[float]) -> List[float]:
    #return list(iter)
    pass

def to_collection(iter: Iterable[float]) -> MyCollection[float]:
    #return MyCollection(iter)
    pass

original = [1.0, -4.2, 5.4]

new_list = to_list(original)             # hints as: list[float]
list_last = new_list[-1]                 # hints as: float

new_collection = to_collection(original) # hints as: MyCollection
                                         # expected: MyCollection[float]

first = new_collection.first             # hints as: T
                                         # expected: float

last = new_collection.get_last()         # hints as: T
                                         # expected: float

Note that the return ... lines are disabled on purpose. This is to prevent the type hints from originating from the original iterable.
In the case of -> List[float], the new_list and list_last type hints are inferred from the hint as expected.
In the case of -> MyContainer[float], the type is not hinted as expected.
So I'd say this is either a bug (missing support for Generic[T]?), or I am doing something wrong.
Your thoughts?
cheers & thank you!
Michael


Python latest vis-a-vis IronPython

I am looking at Python which is currently in 3.9.x whereas IronPython is baselined with Python 2.7.x
Is there a plan for the latter to use the newest Python engine?

Work for a Python 3 compatible version is in progress at https://github.com/IronLanguages/ironpython3


Color facial features, like eyes, lips, cheek bones

Firstly, this is a fantastic library.. really well implemented and documented. Although i did get stuck in the following places.

Using the faceLowPoly.json can i highlight the meshlines in the video, currently when i try to do it. I get the entire mesh as 1 solid object.

Further extension of the one above, can i color certain triangles(facial markers) like lips eyes with colors? Basically detect the lips and then color them..

If so how do i go about doing this

Hi,

You need to use another mesh, a textured mesh or to split the geometry into several mesh if you want to use the parts separately.

Then you won't have a good rendering because to outline the lips we should be able to get the coords of the lip points and this is not possible:

Currently the detection is global, without going through key points like the methods of Active Shape Model of BRFV4 for example (https://github.com/Tastenkunst/brfv4_javascript_examples).
So we do not have the opportunity to finely cut the lips eg.
However I think it would be possible to learn the position of key points. Instead of the network outputting the 3D transformation of the head + 2D framing it would give the position of the key points.
This is not yet in our pipeline (we switch a little presta mode with the integration of virtual fitting of glasses and facial filters for the next few months) but it could be

Before doing 3d programming with this library, I suggest you to learn the basics of 3D programming and THREE.JS : mesh parsing, texturing, positioning, projection... http://www.webglacademy.com can be a good point to start.

I close the issue because there is nothing new.
We are working on a landmark based version of the library, FaceFilterLM, but there is still a lot of work to do :)


Add information on Ubuntu 22.04

The table lacks information on Ubuntu 22.04 LTS

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

ID: 84d52296-198d-7c8a-db99-e8cb336e355f
Version Independent ID: 77670399-109f-3233-0466-c13d84542039
Content: Supported Ubuntu virtual machines on Hyper-V
Content Source: WindowsServerDocs/virtualization/hyper-v/Supported-Ubuntu-virtual-machines-on-Hyper-V.md
Product: windows-server
Technology: hyper-v
GitHub Login: @BenjaminArmstrong
Microsoft Alias: benarm

Hey debuglevel,
Thanks again for contacting Microsoft. We value your feedback.
Ubuntu 22.04 LTS was added into the table to this documentation a while ago and this issue was left open. We're now closing this thread.
Thank you and have a most pleasant day.
#please-close


 A complete log of this run can be found in:>
when the installation is done it shows an error and it is not added to the package.json
i have been looking for the best solution fix this start node.js reinstallation and others

Could you try with a clean installation? Delete your node modules and run npm/yarn install again ?

it's the same sir, I have tried many times

Could you try with a clean installation? Delete your node modules and run npm/yarn install again ?

it's the same sir, I have tried many times

It looks like your npm installation got into a weird state. Can you try:
    npm cache clean -f
    rm -rf node_modules
    npm i

@joaquim-verges
npm cache clean -f
rm -rf node_modules
npm install @thirdweb-dev/react @thirdweb-dev/sdk @thirdweb-dev/auth ethers

Results:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: nextjs-followatch@undefined
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@thirdweb-dev/sdk
npm ERR!   @thirdweb-dev/sdk@"^3" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@thirdweb-dev/auth
npm ERR!   @thirdweb-dev/auth@"^1" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

@0xdunston interesting, can you try installing each thirdweb package one by one? I think that might be the problem

@joaquim-verges Installing auth first then the others after seem to solve it. Thanks!

@artur-eth is your problem resolved as well?

i have men I will close the case because there is no activity or answer


update.py: /usr/bin/env python

Hi Rado,
again problem with python installation: admin can not install 2.7.x in order not to break current system installation.
Instead he provided alias python=.../python2.7 workaround, which is not in work for /usr/bin/env python of update.py - this command always catches system installed python of lower version.
Any help, please ?

Hi Miro, what I would do here is to install a virtual environment with --python=<path to the one you want> and source that one. Then it should work. Please let me know whether this worked for you.

I consider this one as solved. If not, please reopen.


Add rvslots:main to page query to avoid a deprecation warning from MediaWiki 1.32

Retrieving content from page revisions needs an "rvslots" parameter to avoid a deprecation warning from MediaWiki 1.32. I haven't checked if it's needed anywhere other than edit.go: getPages.

Well, that gets rid of the warning, but also seems to break something. I'll look into it.

Adjusted for a change in the json response when the slots are present.

Thanks! Not sure what this slots thing is, but I see that it was introduced here: https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/core/+/07842be379ca3d4d0bc0608c217dd0e8cd7cbe4b

Multi-Content Revisions, a somewhat complex mechanism https://www.mediawiki.org/wiki/Requests_for_comment/Multi-Content_Revisions. I suppose once additional slots are used for something, a different library function would be needed for retrieving them.


Filesystem.mkdir fails due to read-only file system for local loop

Error message
Unhandled promise rejection GoError: mkdir helloworld: read-only file system

Hello @kdouglass-oliveai - Can you demonstrate how you are making the call to filesystem.makeDir()?
For reference, the interface for makeDir function (LDK v3.0.0-beta.1) is as follows:
/**
 * Makes a directory at the specified location.
 * @param destination - destination of where directory needs to be created at
 * @param writeMode - file mode and permission bits
 */
makeDir(destination: string, writeMode: WriteMode): Promise<void>;

Updated original issue, but it appears to be a webpack issue since __dirname is just converted to /

Hello @kdouglass-oliveai, I have couple of thoughts in here. First of all you cant depend on node capabilities as I'm not sure how well will it transpile with the webpack. We have an enhancement request which may help users with supplying better paths: https://github.com/open-olive/loop-development-kit/issues/144. Also, please make sure to specify octal number as permission bits (ex. 0o755) so it will properly propagates through the system.


Return top level error for results

This was causing the shell to not report top level errors from parsing, etc.

Nice, +1.

:shipit:


Last updates for consultations

This update contains the diagnosis consultation.

Looks okay @Benilde. Somebody who understands the form schema will need to review that, since I have no idea about that :).

Also, you can see by the grey pull request icon above that you have merge conflicts that need to be resolved. Let me know if you need help with that.

@Benilde can you try resolving the conflicts and push changes?

I still see conflicts. Does anyone else see the same?

I can see conflicts as well. @Benilde please fetch/pull the last changes from esaude/master then run git status to see the conflicting files

Closing this PR because target branch was merged.


[Feature]: adding timeout to bindr and windowclass to all binds

I would be really nice to have a timeout option for bindr, after which the bind even would expire:
For example, if using the example from the documentation, rressing and immediately releasing Alt_L yields the desired behavior:
bindr=ALT,Alt_L,exec,amongus

However, pressing then keeping Alt_L down for over 10 seconds will also yield the same result, while it's likely the intend changes more by every second that passes between the key press and the key release: for example: the user may have prepared to do a Alt Tab, but then change their mind after noticing a notification about an email.
A better behavior could be achieved with a timeout starting at the press event, then cancelling the bindr action if the timeout is reached without a bindr event, like for 2 seconds:
bindr_bindl_timeout = ALT,Alt_L, 2

In this case, holding the Alt_L for longer than 2 seconds would inhibit the bindr action: among would not have been started past 2 seconds of holding the key down.
It would also be desirable to add window rules as both inclusions and exclusions, for example to never start amongus while reading email in thunderbird:
bindr_inhibit= ALT,Alt_L, class:^(thunderbird)$

The same approach could be used require that a given application be focused, for example to only process bindr events for the SHIFT modifier when using xwayland applications:
bindr_require= SHIFT,, xwayland:1

Alt_L would work as before, but using something xwayland would prevent the bindr for SHIFT modifiers.
This would allow application specific shortcuts, as it could also possible more generally to only process regular bind events for say the CONTROL modifier when using kitty:
bind_require = CONTROL,, class:^(kitty)$,title:^(kitty)$

I would be really nice to have a timeout option for bindr, after which the bind even would expire

fair

It would also be desirable to add window rules as both inclusions and exclusions, for example to never start amongus while reading email in thunderbird:

No

The same approach could be used require that a given application be focused, for example to only process bindr events for the SHIFT modifier when using xwayland applications:

No

tysm for the timeout!
As for the inclusion and exclusion rules, they are to improve hyprland shortcuts handling and support more scenarios: for example, if I try to bind Control_L to "navigation-history-next" (alt-right) and Alt_L alone to "navigation-history-previous" (alt-left):
# Control_L=37 -> Send Alt=56 Right=106
bindr=CONTROL,Control_L,exec, YDOTOOL_SOCKET=/run/user/1000/.ydotool_socket ydotool key 56:1 106:1 56:0 106:0

# Alt_L=64 -> Send Alt=56 Left=105
bindr=ALT,Alt_L,exec, YDOTOOL_SOCKET=/run/user/1000/.ydotool_socket ydotool key 56:1 105:1 56:0 105:0

Then pressing on Control_L causes the Alt_L bindr to also be executed, so the browser receives Alt Left Right (visible in wev, or even vim where the cursor goes quickly to the left then back to the original position)
This can't be fixed by specifying the keycode instead of the keyname:
# Control_L=37 -> Send Alt=56 Right=106
bindr=CONTROL,37,exec, YDOTOOL_SOCKET=/run/user/1000/.ydotool_socket ydotool key 56:1 106:1 56:0 106:0

# Alt_L=64 -> Send Alt=56 Left=105
bindr=ALT,64,exec, YDOTOOL_SOCKET=/run/user/1000/.ydotool_socket ydotool key 56:1 105:1 56:0 105:0

Inhibitors and required-bys seemed the easier solution to this problem, while also extending the possibilities for shortcuts.
Is there another way to have this mapping work with the existing options?
It there a specific reason why it would be bad idea?
Like, would it make a constraint tree too large or too slow that would degrade the performance or the user experience?

No, it's just an extremely niche feature that can be accomplished by a script. IMHO it doesn't belong in the core.

Ohh yes, that makes sense. I'm coming from AHK which allowed very advanced configuration/scripting of the whole UI.
It'd just be nice to have the bindl timeouts , and maybe if you can, something to replace ydotool with hyprctrl: between these  2 simple things, a lot of advanced keymapping will become simple!
The more precise shortcuts (classes and inhibitors) are not exactly urgent lolol


WIP: Preview of oneAPI distributed intrefaces

Changes proposed in this pull request:

Preview for SPMD interfaces in oneAPI
Implementation for MPI & CCL
Test engine update
Backend update for SPMD
Updated oneapi DPC++ samples for K-Means (mpi & ccl)
Updated bazel tests for communicators
Basic statistics and K-Means updates
Updated Bazel build

/intelci: run

/intelci: run


How are requests, internal & remote calls stubbed?

Description
We are trying to get Polly to use a HAR recording it has previously saved. The code below will illustrate my latest attempt after a day of working with the library. We have cloned this repo and have installed the ./examples/puppeteer example and ran it then began to modify it's source code.
What I was expecting to be able to do was that it would download the network archive (HAR) and save it and then any incoming requests would be intercepted and that archive's data would be used instead. The dummy-app is really simple as it just makes requests to https://jsonplaceholder.typicode.com and those are the specific calls I was hoping to intercept.
That is where my issues stem. As I learned PollyJS doesn't automatically stub/intercept these calls so I needed to do them myself. I have reached an impasse as I don't know where the CORs issue is coming from or if I can avoid it all together.
Shareable Source
(async () => {
  const browser = await puppeteer.launch({ headless: false });
  const page = await browser.newPage();
  await page.setRequestInterception(true);

  const { server } = polly;
  server.any().on('request', (req, res) => {
    // None of these work.
    // req.headers['no-cors'] = false
    // req.headers['no-cors'] = true // At this point I'm just trying anything.
    // req.headers['Access-Control-Allow-Origin'] = 'jsonplaceholder.typicode.com'
    // req.headers['Access-Control-Allow-Headers'] = 'Origin, X-Requested-With, Content-Type, Accept'
  });

  server.host('http://localhost:3000', () => {
    server.get('/favicon.ico').passthrough();
    server.get('/sockjs-node/*').intercept((_, res) => res.sendStatus(200));
  });

  server.get('https://jsonplaceholder.typicode.com/*').intercept((req, res) => {
    const rawHar = fs.readFileSync('./recordings/puppeteer_2155046665/recording.har');
    const data = JSON.parse(rawHar);
    res.status(200).json(rawHar)
  });

  await page.goto('http://localhost:3000', { waitUntil: 'networkidle0' });
})();

Error Message & Stack Trace
(newlines added)
Access to fetch at 'https://jsonplaceholder.typicode.com/posts?_end=10&_order=ASC&_sort=title&_start=0'
from origin 'http://localhost:3000' has been blocked by CORS policy:
 No 'Access-Control-Allow-Origin' header is present on the requested resource.
If an opaque response serves your needs, set the request's mode to 'no-cors' to
fetch the resource with CORS disabled.

Config
Copy the config used to setup the Polly instance:
new Polly('puppeteer', {
    adapters: ['puppeteer'],
    adapterOptions: {
      puppeteer: {
        page,
        requestResourceTypes: ['xhr', 'fetch']
      }
    },
    persister: 'fs',
    persisterOptions: {
      fs: {
        recordingsDir: path.join(__dirname, 'recordings')
      }
    }
  });

Full source code but collapsed

Click to expand!
/* eslint-disable */
console.clear()
const path = require('path');
const { bgGreen } = require('chalk');

const puppeteer = require('puppeteer');
const { Polly } = require('@pollyjs/core');
const PuppeteerAdapter = require('@pollyjs/adapter-puppeteer');
const FSPersister = require('@pollyjs/persister-fs');
const fs = require('fs');

console.log(bgGreen('I MADE IT INTO HERE'))
Polly.register(PuppeteerAdapter);
Polly.register(FSPersister);

(async () => {
  const browser = await puppeteer.launch({ headless: false });
  const page = await browser.newPage();

  await page.setRequestInterception(true);

  const polly = new Polly('puppeteer', {
    adapters: ['puppeteer'],
    adapterOptions: {
      puppeteer: {
        page,
        requestResourceTypes: ['xhr', 'fetch']
      }
    },
    persister: 'fs',
    persisterOptions: {
      fs: {
        recordingsDir: path.join(__dirname, 'recordings')
      }
    }
  });

  const { server } = polly;
  server.any().on('request', (req, res) => {
    // None of these work.
    // req.headers['no-cors'] = false
    // req.headers['no-cors'] = true // At this point I'm just trying anything.
    // req.headers['Access-Control-Allow-Origin'] = 'jsonplaceholder.typicode.com'
    // req.headers['Access-Control-Allow-Headers'] = 'Origin, X-Requested-With, Content-Type, Accept'
  });

  server.host('http://localhost:3000', () => {
    server.get('/favicon.ico').passthrough();
    server.get('/sockjs-node/*').intercept((_, res) => res.sendStatus(200));
  });

  server.get('https://jsonplaceholder.typicode.com/*').intercept((req, res) => {
    const rawHar = fs.readFileSync('./recordings/puppeteer_2155046665/recording.har');
    const data = JSON.parse(rawHar);

    res.status(200).json(rawHar)
  });

  await page.goto('http://localhost:3000', { waitUntil: 'networkidle0' });

  // Commented out so that I can use the browser.
  // await polly.flush();
  // await polly.stop();
  // await browser.close();
})();

Dependencies
{
  "@pollyjs/adapter-puppeteer": "*",
  "@pollyjs/core": "*",
  "@pollyjs/persister-fs": "*",
  "puppeteer": "*",
  "start-server-and-test": "*"
}

Environment
Tell us which operating system you are using, as well as which versions of Node.js and npm/yarn. If applicable, include the browser and the corresponding version.
Run the following to get it quickly:

Software
Version

node
8.15.1

OS
16.7.0

npm
6.4.1

yarn
1.1.0

There's a lot here to unpack.

The dummy-app is really simple as it just makes requests to https://jsonplaceholder.typicode.com and those are the specific calls I was hoping to intercept.

Were you hoping to record these requests and replay them back?  If so, Polly should be able to do this for you without explicitly recording a HAR file yourself and attempting to use that in place.
What's going on here appears to be CORs isn't something that jsonplaceholder implements.  Basically your client is asking for permission to hit jsonplaceholder.typicode.com and it's not responding with the correct header set.
To disable same-origin policy within the chrome instance that puppeteer is spinning up you can pass the disable-web-security flag like:
const browser = await puppeteer.launch({
  args: [
    '--disable-web-security',
  ],
  headless: false
});

Thank you for the help. Somewhere along the lines I made some mistakes, which I cannot divulge because I don't know where I went awry, but your help helped me fix my issue. it is working as it should so thank you!


Connection disposal NRE

So, we've had a bit of an odd issue on one of our azure instances this morning .... where we got a load of errors saying we had no connections available for sql.
Some investigation later revealed that one of the instances was having an issue disposing it's session and was throwing this every 1minute (so possibly related to the alias refresh?)
Exception information:      Exception type: NullReferenceException      Exception message: Object reference not set to an instance of an object.    at Orchard.Data.TransactionManager.DisposeSession()    at Autofac.Core.Disposer.Dispose(Boolean disposing)    at Autofac.Util.Disposable.Dispose()    at Autofac.Core.Lifetime.LifetimeScope.Dispose(Boolean disposing)    at Autofac.Util.Disposable.Dispose()    at Orchard.Mvc.Routes.ShellRoute.HttpAsyncHandler.<ProcessRequestAsync>d__7.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Web.TaskAsyncHelper.EndTask(IAsyncResult ar)    at System.Web.HttpApplication.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute()    at System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously)
One assumes because it was failing to dispose the session, it was leaving db connections open.
Recycling the app pool fixed the issue for now, this was using 1.9.2.

I remember I made the alias refresh a separate feature for this reason. I could see the connection pool getting filled quickly with its queries. I assumed at that point that the query was not well constructed and would lock the table.
And you need this feature as you have multiple nodes ... I will check what it's doing to see if I can find something wrong with the code.
If you want to help it might be interesting to trigger the issue on purpose by shortening the background task delay, and having multiple instances of the site do the same work (to simulate multiple nodes, which by the way can be done by creating multiple tenants on the SAME DATABASE tables, nice trick). You can also add some log on the transaction instantiation and disposal, to ensure they are correctly handled.

Interesting tip about the multiple tenant :) will investigate further.

After a quick look in the dev branch around this, first, 2 things that seems weird to me.
When starting the app, _aliasHolderUpdater.Refresh(); is called 2 times. One in AliasHolderUpdater.cs through AliasUpdaterEvent IOrchardShellEvents.Activated(), and one in AliasHolderUpdaterTask.cs through its IOrchardShellEvents.Activated().
In background tasks, _aliasHolderUpdater.Refresh(); is executed through AliasHolderUpdaterTask.Sweep() that also inherit from IBackgroundTask, and if i enable the Alias Updater feature, then it is executed a 2nd time in AliasUpdaterBackgroundTask through its Sweep() method.
Best

That could explain the deadlock on the tables. I will look at it.

Just wondering if this could be due to the MessageBus bug (distributed shell restarts).

@carlwoodhouse looking at your trace, it happens at the end of a web request and in TransactionManager.DisposeSession().
Here, without analysing nhibernate, it would mean that _session is null or _session.Transaction is null. The 1st one is checked at the beginning of the method, this would mean a concurent access on the same TransactionManager instance.
So, just for testing, without disturbing you, before re-checking _session itself before closing and diposing, can you do this 1st test by replacing this:
if (_session.Transaction.IsActive) {

With:
if (_session.Transaction != null && _session.Transaction.IsActive) {

Best.

sorry had zero time to look at this since! will try to find time toward the end of the week


[terra-form-select] Searchable fields' ID moved to input element

Summary
Fixes #2674

Is this change non-passive?

This change will likely break existing test cases for consumers, but because this is not an exposed API and the change is being made to fix an accessibility concern I lean towards this being considered passive change. We should get a few maintainers to weigh in before making the final decision.
cc: @mjhenkes @ryanthemanuel @tbiethman @dkasper-was-taken

@StephenEsser  I think this is now passive with updating the prop name from id to inputId

htmlFor mapped to inputId and prop made private in 48d7bda
@StephenEsser

@saket2403  you should add the jest test cases for all the searchable select fields variants here.

JAWS with Windows 10 is not reading the value that was selected via keyboard.

Functional review complete. No issues identified.


Docs tasks cloud

Applies the updated descriptions in tasks to the new cloud spec for
tasks.
Merged from master.

🎉 excellent! make generate should fix the error, btw

heh, again...


Allow compiling with the CUDA 11.4 compiler

Hello,
This pull request allows sol2 to be compiled by version 11.4 of the CUDA C++ compiler nvcc. It essentially consists of workarounds for compiler bugs, and I certainly understand the annoyance of changing one's code due to bad compilers, but if this is merged then the mainline version of sol2 will be able to support our High Performance Computing codes with no modifications.
Thank you for any time you spend reviewing this and considering it for merging, please let me know if there is anything I can do to help.
I don't think it should change any behavior, although the construction of a pointer by forwarded arguments change might be worth a look by someone with deeper C++ knowledge.

For the time being we've decided to go a different route that doesn't use sol2, so I'll close this to not take up developer time.


Move .config/env.local to .config/env.template

The original intention of adding .config/env.local to .gitignore was to prevent local changes to the file from getting checked in. However, .gitignore will not ignore files actually checked into the repository.
With this change, the convention is now to copy .config/env.template to .config/env.local before making changes. ./go now checks for this file before attempting to source it.

Coverage remained the same at 97.742% when pulling 8b2dd2f4e01354f7d19866adec960ce17b8d534e on env-template into 27da4f1665f632d1732e71d56ce61932ecdb672e on master.


Feature/new full text search

Please target the develop branch (you don't have to recreate the PR, simply change theis current one).

@hajarghaem Why do you want merge this? I think it is mistake. Can you please close this pull request.


Disable review status change on the WEB UI

Please review the tests and rebase to the master branch.


vehicle_types_available in all/station_status.json uses wrong format

The form-factor filtered feeds have this in the correct format:
        "vehicle_types_available": [
          {
            "vehicle_type_id": "Renault ZOE 360km",
            "count": 1
          }
        ],

while in the "all" feed it looks like this:
        "vehicle_types_available": {
          "Renault ZOE 360km": 1
        },

Ie. update_availability_status would need the same logic as station_with_available_vehicles_array in moqoToGBFS.py, but that's slightly more than I can reasonably do blindly without a way to test this :)

related: @hbruch Would it be possible to check MOQO API responses into this repo, to have something like an integration test? This would make it much easier for others to contribute.


How to obtain token embeddings from GraphCodeBERT

Can you give me some examples?

You can obtain token emdeeings using self.encoder(inputs_embeds=inputs_embeddings,attention_mask=attn_mask,position_ids=position_idx).last_hidden_state in https://github.com/microsoft/CodeBERT/blob/1008a9a9698e4ad909ebabeb4a1e5c2a282bc304/GraphCodeBERT/codesearch/model.py#L19

Can you tell me the detailed steps? I don't know how to do it.

I want to get the graphcodebert  token vectors to do downstream tasks.

I suggest you can use one of downstream tasks we release (code search, clone detection, code translation, code refinement) as the base code, and then modify the input data. For classification task, you can use clone detection. For retrieva task, you can use code search. For generation task, you can use code translation.
What you need to do?

Choose a similar task we release
Change input data to your data
Modify  loss calculation. For example, you can obtain token vectors in clone detection, codesearch(change [1] to [0]), and code translation. And then use token vectors to calculate loss.

Okay, thank you for responding so quickly.


Fix url encoding for Dataset url.

I found a small bug on the web module. It's missing encoding for Dataset urls.

Fixed version.

Sorry, This is not completely fixed. The problem recurred when I clicked the breadcrumbs.

Could you help me?
Thank you!

I don't have a good idea, so I close this PR.
Thank you.


[ Full Stack Cert ] - update intro text for event hub lab

Let’s update the description to the following:
 "intro": [
          "In this lab, you will review working with semantic HTML elements by building an event hub.”, “This lab will give you an opportunity to review working with the header, nav, article elements."
        ]

Here is the file you need to update
https://github.com/freeCodeCamp/freeCodeCamp/blob/main/client/i18n/locales/english/intro.json#L1756-L1761

This has been opened for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum
Happy coding


cannot get assets from module folder

I have uploaded image to my module path Modules/Blog/Resources/assets/blog.png but any function from documents i use i cannot get that image in my app (In app not in module itself)
Sample code
@foreach ($modules as $module)
    {{$module->asset('blog:blog.png')}}
@endforeach

Tried and errors
$module->asset('blog:blog.png')

// Method Nwidart\Modules\Laravel\Module::asset does not exist.

$module->assetPath('blog')

// Method Nwidart\Modules\Laravel\Module::assetPath does not exist.

$module->getAssetsPath()

// Method Nwidart\Modules\Laravel\Module::getAssetsPath does not exist.

$module->getExtraPath('Resources/assets/blog.png')

// /Users/xxxx/xxxxx/xxxx/Modules/Blog/Resources/assets/blog.png
// _This is wrong by all means! there shouldn't be any path of my PC user or folders, just the app path i need._

Any suggestion on how to get my module image in main app?

Solved
add to module config file
'image' => 'modules/blog/blog.png',

run following command
 php artisan module:publish

use following code in blade to get modules images
{{url(Config::get($module->getLowerName().'.image'))}}


How accept tag if  he not have in table?

Hello.

How make that tags not delete if it tag not exist in table dtabase an accept too when click enter?
How make that tag accept “_” instead of “enter”?

I @adamtester did you developed this feature?


click dependency not bounded

OS: Windows 10 21H2
Python: Python 3.10.4
Steps:

pip install "click<7"
pip install pyquotes

example.py
print("Hello, World!")

pyquotes --check example.py
pyquotes -D --check example.py
Both fail with the error
  File "C:\Python310\lib\site-packages\pyquotes\settings.py", line 96, in __init__
    super().__init__(**settings, project_root=project_root)
TypeError: _Config.__init__() got an unexpected keyword argument 'check'

This is resolved when using click>=8.
Thank you for your time, this library has saved my many hours.

Thanks, but I do not plan to support click 6. It's ancient (5 years old).
If there's a problem with recent click versions (7 or 8) that'd be something more worth fixing (ideally supporting both).

Thanks, but I do not plan to support click 6. It's ancient (5 years old).
If there's a problem with recent click versions (7 or 8) that'd be something more worth fixing (ideally supporting both).

Absolutely correct, but could you put a requirement on >=8? (the issue happens on 7.x as well) incase someone's environment has click already installed, it's currently an unbounded requirement.