Configure Style/NumericPredicate for performance

Hello,
While the Style/NumericPredicate cop recommends using .zero?, this can lead to performance degradation in gems where zero checks are frequent and speed-critical; for such performance-oriented libraries, using == 0 remains preferable despite not adhering to the style guide.
This will not lead to performance improvements for the current implementation of memo_wise, I'm rather submitting this as a "reminder"

Replace predicate methods like v.zero? with direct comparison v == 0 for better performance.
While the current implementation may not show visible improvements, frequently called methods using .zero? instead of == 0 can lead to significant performance losses.
Benchmarks across Ruby versions 2.5 to 3.3 consistently show comparisons  outperforming predicates:
Comparison (Ruby 2.6 to 3.2, v = 0):
              v == 0: 21479329.3 i/s
             v.zero?: 17979885.4 i/s - 1.19x  (± 0.00) slower

Comparison (Ruby 2.5 and 3.3, v = 0):
              v == 0: 23652215.7 i/s
             v.zero?: 21843174.0 i/s - 1.08x  (± 0.00) slower

Comparison (Ruby 2.5 to 3.3, v = 1):
              v == 0: 23227474.2 i/s
             v.zero?: 21675200.9 i/s - 1.07x  (± 0.00) slower

Before merging:

[ ] Copy the table printed at the end of the latest benchmark results into the README.md and update this PR
[x] ~If this change merits an update to CHANGELOG.md, add an entry following Keep a Changelog guidelines with semantic versioning~

Hi @tagliala! First I want to thank you for what appears to be an awareness campaign to help the community.
Can you explain why we would make this change here, in these two lines of code? One is in benchmark set up, the other is in one-time instrumentation of an object with MemoWise.
In neither case, is it yet clear to me that our trade-off would be for performance at the cost of (arguable) readability. Is that trade-off clear to you here? If so, can you help explain and make it clear?

Hi,

the other is in one-time instrumentation of an object with MemoWise.

In the specific case of memo wise, it is not going to provide any visible improvement, because as you mentioned it is a one time initialization. This is also stated in the opening post: "This will not lead to performance improvements for the current implementation of memo_wise, I'm rather submitting this as a "reminder"

Is that trade-off clear to you here? If so, can you help explain and make it clear?

It is more than clear. The fact is that this cop is globally configured for readability, so it may lead to an usage of numeric predicates in contexts where they are not optimal, like loops. Again, this is not the case. I see memo_wise as a gem oriented to performance, so I proposed this change here in specific, but as a reminder and also to read other thoughts

Thanks for making this PR, @tagliala! I appreciate the thoughtful commit message and interest in making future gem changes more performant.
Personally, I'm indifferent to this change. I generally like having performant defaults, but I also struggle to see a world in which we would change this gem to include .zero? or .one? in a hot path at all, let alone have it slip through code review and our benchmarks. I'm not opposed to merging this however, since you took the time to make it.
@ms-ati do you have a preference?

Welcome

I'm not opposed to merging this however, since you took the time to make it.

Thanks, and don't worry about the time.
I was also proposing to understand if that method could have made the difference in some circumstances, like large codebases, or if the new default may positively have impact on future development, but as far as I understand this is not a concern for memo_wise. And if it will ever be, it will be possible to make the change in the future

I was also proposing to understand if that method could have made the difference in some circumstances, like large codebases

On this point specifically I think it won't make a noticeable difference. That .zero? will typically run at load time (when memo_wise is called), and based on your benchmark it seems like both .zero? and == 0 can execute many billions of times per second. I can't see a codebase ever having close to that many memo_wise calls so I don't think this will matter.

I'm going to close this, but please let me know if anyone feels this should not be closed.


todo-python-mongo-aca has func/.devcontainer.json

Examine the file: https://github.com/Azure-Samples/todo-python-mongo-aca/blob/main/.devcontainer/func/devcontainer.json
Not sure why this file exists. The sample isn't a Azure Function sample, so I would not expect an Azure Function related devcontainer to show up

Looks like a bug in the repo.yaml file


Restarting specific jobs

Is your feature request related to a problem? Please describe.
A development branch of Psi4 currently uses a straightforward interface to QCFractal via,
r = client.add_compute("psi4", self.method, self.basis, self.driver, keyword_id, [mol])
self.result_id = r.ids[0]
if self.result_id in r.existing:
    print("Job already completed {}".format(self.result_id))
else:
    print("Submitting Single Result {}".format(self.result_id))

The results are collected with,
result = client.query_results(id=self.result_id)

If a calculation fails for some reason, so that the job status becomes ERROR, it cannot be restarted with the above calls.
Describe the solution you'd like
I know job restarts are possible from the server-level, but it would be convenient to somehow rerun any job regardless of its current status. Maybe something like,
client.restart_compute(id = self.result_id),

where self.result_id is the hash for a particular job. Alternatively, a function instead could change/remove the job status so that it gets rerun automatically.

Looks good! I think we might do something more like:
client.modify_tasks(ids, action)

Where action may be hold, restart, etc. Would this still work for you while allowing more capabilities in the future?

I think the modify_tasks is a better solution. This could even provide a simple interface to remove queued jobs.


Add a legend

It would be nice if you could create a pull request

@mhmdanas
The new version 1.1.8 published recently has options for legends.


[fix] UnboundLocalError: local variable 'has_comment_priv' referenced before assignment

I found out that by reversing those snippets of code, the function to comment on profiles is now working again.

LGTM! Sorry for the delay and thanks for your contribution!


v7.6.1 release

I'm using this good plugin.
Do you have any plans to support v7.6.1?
Thank you

Checkout the new 7.6 branch.
We still do not know how to change the throbber image in kibana 7.6+, but the rest of the changes should work.

I'm using opendistro and kibana.
opendistro path is  /usr/share/kibana/src/core/server/rendering/views/styles.js
I changed background-image:
.kibanaWelcomeLogo {
            width: 60px;
            height: 60px;
            margin: 10px 0 10px 20px;
            background-repeat: no-repeat;
            background-size: contain;
            /* SVG optimized according to http://codepen.io/tigt/post/optimizing-svgs-in-data-uris */
            background-image: url('data:image/svg+xml;base64,PD94bWwgdm');
          }

also I think you can use this way :

go to : src/core/server/rendering/views/styles.tsx
adding welcomelogo

refrence url : https://discuss.elastic.co/t/customize-logo-in-kibana-7-6/234771/5


add testcontainer as integration testing tool

What this PR does:
Which issue(s) this PR fixes:

Fixes #103 .
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #119 (4f55b06) into master (e03c2a9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   35.39%   35.39%
=======================================
  Files          69       69
  Lines       11002    11002
=======================================
  Hits         3894     3894
  Misses       6665     6665
  Partials      443      443

Continue to review full report at Codecov.

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

I think the test source file (test_container.go) should be renamed to testcontainer_common_test.go, for example. Because it's for test scope only.


Update codecov badge

@matthewfarwell

Codecov Report

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

@@          Coverage Diff          @@
##           master   #336   +/-   ##
=====================================
  Coverage       0%     0%
=====================================
  Files          62     62
  Lines        1495   1495
  Branches      146    146
=====================================
  Misses       1495   1495

Continue to review full report at Codecov.

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


This really needs some color (especially the JSON output)

Just a thought - would be nice to have a --color parameter (which could be the default or not - or at least be configurable).
Using underscore-cli to simulate desired behaviour here:

@snobu: have you tried -o jsonc as an option to see colorized Json output?

A-ha! Cool stuff.
Can i make that the default through something like .azrc, you know like .bashrc or .vimrc?

@snobu you can set your default output format using az configure (my default is also JSONC 👍 )


interfaces: VIPs sometimes reorder, taking higher priority as "real" interfaces IPs

Perhaps it was the order in which things have been configured on my machine but I ran into a bug where the interface displays one subnet while in reality another is being stored into config.
I took the following steps:
->configure public IPv6 adress on interface
->enable RA
->create virtual IP for interface with ULA adress
->try to enable DHCPv6 for interface, see that it defaults to public subnet and allows no choice for ULA subnet (perhaps in this step enable and leave enabled DHCPv6 server)
->switch public and ULA adresses (ULA becomes interface adress, public one becomes virtual adress)
-> enable DHCPv6 server
-> configure ULA range and get confronted with message that range is outside of subnet, while it is displayed above as available range.
I've commented out the subnet checking bits to see what was happening, and I noticed that instead of my private subnet being written to file my public one was being written to file. So for some reason the public subnet overrides everywhere but in the interface.
Repeatedly fidling switching and deleting interfaces has allowed me to get a correct config written out, however I'm now unsure whether it will remain there after reboots.

@marjohn56 the issue is not with VLAN afaik but with virtual IP (the OPNSense lingua for FreeBSD alias, Firewall->Virtual IP's), which allows multiple IP adresses to be configured on the same device.
My guess would be that by setting a ipv6 adress on the device, going to the dhcpv6 server and enabling it it stores already that subnet in a config.
Then I guess changing the main interface subnet to a different one, but at the same time configuring configuring the old subnet on a virtual ip and then modifying the dhcpv6 settings will not result in the stored subnet getting changed, as it is still associated with the device.
While at the same time the dhcpv6 configuration page pulls it's info from the configured main subnet.
Hope it all makes sense ;)

@gizahNL still an issue?

@fichtner I'm having this issue as well. I've posted it on the pfSense issue tracker as well. Hopefully you guys can take a better look at it then they do.
Basically the current workaround seems to be this:

change the order of the IP's on the interfaces so the ULA is below the tracked IP
update the radvd.conf file
restart radvd

As I said in their issue tracker, I don't really know how to do step 2 from the CLI. It seems to have to do wit the services.inc file and the first function.

@draggeta The IPv6 side of things has given us trouble in that area before, the sense of a "primary" address on the interface is already trashed by the concept of link-local addresses, but that's a bit of a side story. Since dhcp6 kicks in later we get an address too late and the alias is already pushed to the interface... the only solution that seems doable is make IPv6 aliases "second class" addresses and flush + reapply them on new IPv6 address delegations. How does that sound?

@fichtner That is a completely reasonable implementation. In those cases you may notice a small hiccup when using the ULAs for communication, but it's better miles better than having the connection to the internet be unavailable after a reboot/PD change.
I'll be happy to help with this issue, but I'm not sure I could contribute directly.

@draggeta no worries. Testing is appreciated as well... hopefully can look at it next week in more detail.

I'm unable to see an issue at the moment. No new data since Dec 2018.


Fix test path for test-experimental-torchlib-tracing

Update the file path in the test command after the file was moved

ImportError while importing test module '/home/runner/work/onnxscript/onnxscript/tests/function_libs/torch_lib/ops_test.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/opt/hostedtoolcache/Python/3.10.14/x64/lib/python3.10/importlib/__init__.py:126: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/function_libs/torch_lib/ops_test.py:41: in <module>
    from tests.function_libs.torch_lib import (
E   ModuleNotFoundError: No module named 'tests'


Add command to delete investment projects

Description of change
This adds a management command that can delete investment projects found in the spreadsheet.
Checklist

[x] Has a new newsfragment been created? Check changelog/README.rst for instructions
[ ] Have any relevant search models been updated?
[ ] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[ ] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[ ] Has the admin site been updated (for new models, fields etc.)?
[ ] Has the README been updated (if needed)?

Output is misleading.
It says its deleting 6 records but uk region locations is metadata which I presume you don't want to delete? And I think because its a many-to-many it doesn't anyway.

@marcofucci but there are actually records in many to many tables linking the metadata to Investment Project, no? That would tell you that these investment projects had some of those assigned. Now I am not sure how useful it is but I think it is better to show all that is deleted otherwise it would be a little misleading too.


Error: path should come before headers

Bug report

What is the current behavior?
Recieve Error: path should come before headers after following the readme and running ntl dev
If the current behavior is a bug, please provide the steps to reproduce.
Clone repo, follow readme.

What is the expected behavior?
No errors should be displayed

Other relevant information:
Fix:
_headers files should be: (?)
/static/*
  # Headers for that path:
  Cache-Control: public, max-age=360000

Syntax presumably has been updated in netlify-cli

Node.js version: v13.10.1
Yarn version: 1.19.1
Operating System: macOS Catalina 10.15.4 (19E287)

See: https://github.com/netlify/cli/issues/766

Thank you both for providing this information. I've opened a PR to fix the syntax in the repo https://github.com/netlify-templates/gatsby-starter-netlify-cms/pull/546


Fix: spelling accents errors and broken links (Portuguese)

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Thank you for opening this pull request. This is a standard message notifying you that we have reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you. ** Reason: ** Since your PR was created, the master branch content has been updated making some of your unnecessary changes or need to be reorganized to fit the current version. Please re-sync your fork so you are looking at the latest version of master and then create a new PR if you find any changes warranted. Thank you and happy coding.
There is not an english folder in the docs/181n-languages folder.


LEDs WS2815 , Red and Green inverted

Hi,
Managed to make a version that use 12v WS2815 leds strip .
Noticed that the Red and Green colors are inverted.

From lines 243 to 252 :

work fine


Bad running in Jupyter Notebook

Issue Type: Bug
Hi
I have a problem with running program in Jupyter NoteBook, when running, sometimes the VS Code doesn't show the printed stuff at the current moment, and if I go forward and one step forward, it will show the previous printed line, and I can't see that line, at that correct moment, and I have to go and one step forward in the program running to see the details of the previous step.
Therefore please solve this issue.
The issue video URL:
https://mega.nz/file/2uB0SRQA#QiRSf96dmpa4A0Dx1YfqlPBAJig5SgSJlTxzRVcnYSE
(Once I typed this issue and I didn't get any answer)
Thanks.
VS Code version: Code 1.62.2 (3a6960b964327f0e3882ce18fcebd07ed191b316, 2021-11-11T20:56:38.428Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz (8 x 1800)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
6.18GB (1.29GB free)

Process Argv
--crash-reporter-id 27f72fee-8da4-4129-bb28-02f0269ce2ab

Screen Reader
no

VM
0%

Extensions (8)

Extension
Author (truncated)
Version

theme-monokai-pro-vscode
mon
1.1.19

python
ms-
2021.11.1422169775

vscode-pylance
ms-
2021.11.2

jupyter
ms-
2021.10.1101450599

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.3

polacode
pnp
0.3.4

material-theme
zhu
3.13.2

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythontb:30283811
pythonvspyt551cf:30345471
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30384385
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
dockerwalkthru:30377721
bridge0723:30353136
pythonrunftest32:30373476
pythonf5test824:30373475
javagetstartedt:30391933
pythonvspyt187:30373474
vsqsis200:30402083
vsaa593:30376534
vssld246cf:30401891

Sorry, I really don't understand the problem. Can you explain in more detail?

That's all, why you guys keep saying I don't understand?, I even uploaded a video for you.
Is the mistake from my English grammar? or you don't try to understand?

I don't understand what the video is showing, and I don't understand what you mean by "one step forward/back". Please explain what you are expecting to happen

Now good to hear it, you want me to say exactly what you didn't understand,
When my code is running, it wants an input for continuing, so when I enter a manual input some texts should be printed after that for example say thanks for input, but we can't see it printed, OK? but actually they have been executed, after that lines, in my code, it gets an input again and then another printed stuff, so if I enter the next input (this means one step forward) then in the output section, NOW we see the previous lines that were executed but not printed and shown, with the lines printed at the current moment, that's the problem, and what we are expecting to be is to see the printed lines at their correct moment (the time that they should be in the output).
Thanks
And let me know where you didn't get.

Can you share the notebook?

def includedornot(l, s):     for i in range(0, len(s)):         if s[i]== l:             return 1     return 0 def removeLetter(l, s):     sl= []     for i in range(0, len(s)):         sl.append(s[i])     for j in range(0, len(sl)):         if sl[j]== l:             sl[j]=""     return sl def main():     print("Please enter the stirng you want to remove a letter from:")     string= input()     print("Now please enter the letter you want to be removed from the string you entered:")     while True:         letter= input()         firstconfirm= includedornot(letter, string)         if firstconfirm== 1:             break         elif firstconfirm== 0:             print("Please enter again, the letter is not in the string!:")             continue     finalstring= removeLetter(letter, string)     print("The final string without the letter", letter, "is:")     fins= ""     for i in range(0, len(finalstring)):         fins+= finalstring[i]     print(fins)     print("End of the program.")     return #start0: # welcome and goodbye print("Hello and welcome, this program is used for you to be able to delete a particular letter from a string you have entered.") main() while True:     print("Do you want to start again? 1 for yes / 0 for no")     choice= int(input())     if choice== 1:         main()     elif choice== 0:         print("Thanks for using the app.")         break print("Goodbye.")
This is the code which I showed in the video as well.

I get it now, thanks.
Can you try downloading the Insiders' build and tell me whether it happens there? https://code.visualstudio.com/insiders

@roblourens Yes when you told that, I installed the Insiders app and I tried the Insiders build, but nothing happened and the issue isn't solved, I also tried update the python to version 3.10.0, but the issue's still happening.

Any idea how to troubleshoot it @rebornix, an output from print("...") is not showing up at the right time until some more output is added

ping @rebornix

Hi, the issue is still happening and the team has not yet solved it, I think you are responsible for this, if you guys don't know what to do, please refer to someone else.

Sorry for the slow response but lots of changes in the past year. Please let me know if it's still an issue


[Bug]: The Default value is not displayed when user click on JS for toggle

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
In Check, box widget click on js for Inline toggle  the data is displayed empty for the first time

Steps To Reproduce
1. Got to canvas
2. Drag and drop the check box group widget
3. Inside the property pane Click on JS for Inline toggle
4. The Data is displayed blank

Environment
Release
Version
Cloud

@somangshu This happens not due to the issue with JS Convertible functionality.
This happens since the default value for isInline is not provided here

can i get it? @somangshu

Hi @tauantcamargo,
When I checked this, I'm unable to reproduce this issue. So, I'm closing this.
Thanks for showing interest Please check out other open issues.


Add bindings for renderer discoverer

A user asked for this on the LibVLC community discord. This would allow to find and stream to chromecasts on the local network.
https://www.videolan.org/developers/vlc/doc/doxygen/html/group__libvlc__renderer__discoverer.html

I see you have the events for it already
https://github.com/adrg/libvlc-go/blob/93f9c6defcde4bd3074498f080c4c17e06fd0ff9/v3/events.go#L120-L130

I see you have the events for it already
https://github.com/adrg/libvlc-go/blob/93f9c6defcde4bd3074498f080c4c17e06fd0ff9/v3/events.go#L120-L130

@mfkl Thank you for letting me know. I'll work on implementing the renderer discoverer set of bindings next. Hopefully, I'll release a new version with them early next week.

@mfkl Thank you for letting me know. I'll work on implementing the renderer discoverer set of bindings next. Hopefully, I'll release a new version with them early next week.

Hi @mfkl. The renderer discovery functionality has been added in v3.1.0. I also added a stream media to Chromecast example.


4 packages had stderr output: fri_vendor lbr_demos_fri_ros2_advanced_python lbr_demos_fri_ros2_python lbr_demos_ros2_control_python

Hi, when I install lbr_fri_ros2_stack, I find packages had stderr output including fri_vendor lbr_demos_fri_ros2_advanced_python lbr_demos_fri_ros2_python lbr_demos_ros2_control_python.
Here is my environment:
Python: 3.10.12
ROS version: Humble

Here is the Error Message
--- stderr: lbr_demos_fri_ros2_advanced_python
/home/thy/.local/lib/python3.10/site-packages/setuptools/_distutils/cmd.py:66: SetuptoolsDeprecationWarning: setup.py install is deprecated.
!!
    ********************************************************************************
    Please avoid running ``setup.py`` directly.
    Instead, use pypa/build, pypa/installer or other
    standards-based tools.

    See https://blog.ganssle.io/articles/2021/10/setup-py-deprecated.html for details.
    ********************************************************************************

It seems to fail to install KDL, showing the message after vcs install:
lbr_demos_fri_ros2_advanced_cpp: Cannot locate rosdep definition for [orocos_kdl]

Try to install orocos using
sudo apt-get install ros-humble-orocos*
and
sudo apt-get install ros-humble-python-orocos*
They didn't work.

It works after giving:
pip install setuptools==58.2.0
Now this message is gone:
--- stderr: lbr_demos_fri_ros2_advanced_python
/home/thy/.local/lib/python3.10/site-packages/setuptools/_distutils/cmd.py:66: SetuptoolsDeprecationWarning: setup.py install is deprecated.
But still have stderr on fri_vendor

Refering to This issue:  https://github.com/ros2/ros2/issues/1310
It shows orocos_kdl is removed in favor of orocos_kdl_vendor.

To skip orocos-dependt part,  try:
colcon build --cmake-args -DCMAKE_BUILD_TYPE=Release --symlink-install --packages-skip lbr_demos_fri_ros2_advanced_python
but still meet this error
--- stderr: fri_vendor
HEAD is now at 785a8d8 updated logo
Found nanopb of version: 0.2.8
Found FRIClient of version: 1.15

It seems to work though having errors in fri_vendor.
So this is fine if we only see the stderr.
You can treat this as a warning.(Tested from my side.)

Refering to This issue: ros2/ros2#1310
It shows orocos_kdl is removed in favor of orocos_kdl_vendor.

The orocos dependency requires an update in:

CMake: https://github.com/lbr-stack/lbr_fri_ros2_stack/blob/f4bed49acd33ed4a8c954638eff83d2f76296dff/lbr_demos/lbr_demos_fri_ros2_advanced_cpp/CMakeLists.txt#L21

package.xml https://github.com/lbr-stack/lbr_fri_ros2_stack/blob/f4bed49acd33ed4a8c954638eff83d2f76296dff/lbr_demos/lbr_demos_fri_ros2_advanced_cpp/package.xml#L20

For the advanced_cpp demo https://github.com/lbr-stack/lbr_fri_ros2_stack/tree/humble/lbr_demos/lbr_demos_fri_ros2_advanced_cpp


restructure typechain

I don't think it needs to be 4 separate packages. 2 at most, one for parsing the abi, the other for writing ts

Is this a dupe/should it live in the typechain repo?


Tenant caching

Hi,
this is my code to cache a tenant in the DistributedMemoryCache once resolved, in order to skip the more expensive database call.
It seems to work but, since I have assembled this code from various issues here on GitHub, @AndrewTriesToCode can you please tell me if it complies with best practices? Thank you.
builder.Services.AddDistributedMemoryCache();

builder.Services.AddMultiTenant<MyTenantInfo>(config =>
{
    config.Events.OnTenantResolved = async (context) =>
    {
        if (context.StoreType != typeof(DistributedCacheStore<MyTenantInfo>))
        {
            var sp = ((HttpContext)context.Context).RequestServices;
            var distributedCacheStore = sp
                .GetService<IEnumerable<IMultiTenantStore<MyTenantInfo>>>()
                .FirstOrDefault(s => s.GetType() == typeof(DistributedCacheStore<MyTenantInfo>));

            await distributedCacheStore.TryAddAsync((MyTenantInfo)context.TenantInfo);
        }

        await Task.FromResult(0);
    };
})
.WithDistributedCacheStore(TimeSpan.FromMinutes(60))
.WithEFCoreStore<MultiTenantStoreDbContext, MyTenantInfo>()

// some more stuff

@devalot76 this looks good to me. I'd like to make this more seamless in the future and it's on the list...

Would certainly be a good addition to prevent silly situations like the one I have just discovered. While trying to figure out why I was getting so many (Azure) Redis timeouts, I realized that I forgot to add a check the likes of if (context.StoreType == typeof(DistributedCacheStore<MyTenantInfo>)) return; in the OnTenantResolved callback, resulting in a massive and utterly unnecessary inefficiency in the synchronization process between the database and Redis cache.

@hbulens I'm glad you caught that. I definitely would want it to be smart enough to avoid that type of issue.


Fix Regression: retrieving policy id before creating a new policy group on keycloack...

(Disclaimer: this is a simple PR to test the git workflow)
Description:
After the merge of commit fc778c5838eaef49f84f621af504136ab81afb3c, when calling the API to create a new policy, the id wasn't been retrieved, generating an error while creating a new policy group on keycloak. Thus, in the first time the group created was policy-None-owners, and in the second attempt an error message was sent back.
This PR retrieves the id added with db.session.flush() before creating the policy group.
Fix regression in commit fc778c5838eaef49f84f621af504136ab81afb3c
https://github.com/resource-hub-dev/rhub-api/blob/fc778c5838eaef49f84f621af504136ab81afb3c/src/rhub/api/policies.py#L76-L90

Rebased!


Release Event Hub Track 2 Library Preview 2

#339 tracked the work needed to ship the first Preview for Event Hubs Track 2 across the 4 languages. This issue is to do the same for the next preview

.Net
Java
TS
Python

The work here will span across 2 sprints 155 and 156

This is for the board review of the API changes necessary for preview-2?  It doesn't read that way.

@adrianhall No, this is the uber epic to track the work across languages for Event Hubs Preview 2. It is similar to what @AlexGhiondea has been creating for Keyvault and Core, see #435 and #436


FATAL CLI ERROR Error: EACCES: permission denied kibana/elasticsearch/logstash

Problem description
I cloned the master branch and run "docker-compose up -d", so I'm facing these issues below kibana, elasticsearch, logstash

Extra information
Stack configuration

Docker setup

$ docker version

[OUTPUT HERE]

$ docker-compose version
[OUTPUT HERE]

Container logs

$ docker-compose logs

[OUTPUT HERE]

I tried a lot chmod 755, chown but still cannot do anything

Looks like the configuration files aren't readable by the containers' user(s). Try to chmod -v go+r each problematic config file and see if that helps.
Read permissions should be present by default when you clone the repository, but maybe you're using a custom umask in your terminal session.
Also some heads up: docker-elk requires Compose 1.26 as indicated in the README. With Compose 1.23, the quotes inside the .env file are included in the password (it's a Compose bug).

Thanks @antoineco


Update google-java-format version to 1.10.0

Resolves #73.

Thank you 🙂

Would you be so kind to publish a release? :)


Update exact_sampling.rst

Clarify chain rule is for correlation functions (not marginal probabilities)

This change is

Hi @feynmanliang,
Thanks for reaching out!
I'd be happy to modify the docs to make things clearer :)
Can you detail what you found wrong or unclear in the chain rule for projection DPPs ?
Below is a attempt to clarify the reason of the = instead of the C.
More generally, I should take some time to refresh the documentation using a simpler flow...

For a projection DPP(K), samples have (a.s.) identical size r = rank(K).
To describe the chain rule one must express the joint distribution of a vector of points (x_1, \dots, x_r) forming a potential sample i.e. a set S = {x_1, \dots, x_r}
Applying Proposition 19 of HKPV06 to the finite case yields that:
If one can sample
(x_1, \dots, x_r) ~ 1/r! det K_S,
then one can forget the order in which the points x_1, x_2, \dots x_r were drawn and consider S as a valid sample of DPP(K)
Now, going somehow backwards, since the set S has cardinality r (the size of any sample X of this DPP), one can write
det K_S = P[X C S] = P[X = S],
since it is invariant by permutation, the joint density of (x_1, \dots, x_r) is indeed
1/r! det K_S

Thanks for explaining :). Yes, I agree P[X C S] = P[X = S] because of equation (9) modulo indicator functions.
However, I think that the chain rule is in terms of marginal probabilities P(x_1,...,x_r C S) rather than joint probabilities P(X = S). More specifically, the Schur complement formula is giving the probability P(x_1,...,x_{r-1},x_r C S | x_1,...,x_{r-1} C S) as P(x_1,...,x_{r-1},x_r = S | x_1,...,x_{r-1} = S) = 0. Also, I think this chain rule holds for all finite DPPs: it's just that for projection DPPs the rank K joint probabilities equal the rank K correlation functions which justifies terminating iterative bottom-up sampling after rank K iterations.

Thanks for explaining :). Yes, I agree P[X C S] = P[X = S] because of equation (9) modulo indicator functions. So nothing is incorrect; my suggestion is just to clarify in a way that is more clear (for me at least) where the unique properties of projection processes (rank K correlation function equals marginal density) shows up.

Agreed, thanks you again for pointing this out :)
I am eager to make things clear, to avoid any confusion and make sure the DPP model is fully preserved.

I acknowledge the current version of the doc may not reveal all the subtleties nor clear up all potential confusion
we should provide more than just = becomes C which brings more confusion (for me at least)

Please read what follows and let me know what you think would be better to write :)

The chain rule is in terms of marginal probabilities P(x_1,...,x_r C S) rather than joint probabilities P(X = S). More specifically, the Schur complement formula is giving the probability P(x_1,...,x_{r-1},x_r C S | x_1,...,x_{r-1} C S)
since the other conditional is trivial: P(x_1,...,x_{r-1},x_r = S | x_1,...,x_{r-1} = S) = 0.
This chain rule holds for all finite DPPs: it's just that for projection DPPs the rank K joint density equals the rank K correlation function which justifies sampling by performing iterative bottom-up sampling and terminating after rank K iterations.

To me the chain rule as described by [HKPV06] is on joint probabilities for a projection correlation kernel K
A sequential draw yields a vector; there is an order.
You must show that the distribution of the vector is exchangeable i.e. invariant by permutation to be allowed to forget the order and consider the points you've sampled as a set with the appropriate distribution.
Otherwise one may fall in the same trap as the paper of HeGa16 you are mentioning bellow.

I'm sorry I don't understand what you mean

Could you clarify what you mean by "applies to all DPPs" and by "bottom up sampling" ?
To me,

applying the chain rule blindly to any (valid) correlation kernel K is wrong in the general and this is the pitfall in which HeGa16 fell
bottom up sampling corresponds exactly to what Pou19 does (efficiently) by exploring the binary tree of possibilities:
(i) take item 1 ? Yes/No with proba P[1 \in X] = K_11
(ii) take item 2
given 1 was selected ? Yes/No with proba P[2 \in X | 1 \in X] = K_22 - K_12K_21/K_11 cf Proposition 3
given 1 was not selected ? Yes/No with proba P[2 \in X | 1 not \in X] = K_22 - K_12K_21/(1-K_11) cf Proposition 4
(...) take item N, given ...

It happened recently (https://arxiv.org/pdf/1903.03571.pdf Appendix D, citing https://arxiv.org/pdf/1609.06840.pdf equation (2), citing HKPV06) that the projection DPP requirement required for this formula was omitted. I suspect notation (using P(x_1,...,x_r) to denote P(x_1,...,x_r C S) which equals P(x_1,...,x _r = S) only for projection DPPs) contributed to this, and that using P(x_1,...,x_r C S) for the formulas could have helped clarify there.

I agree, totally!
@rbardenet and I had noticed that HeGa16 was wrong, for some time now.
This distortion of the DPP model was a motivation for us to set up this toolbox along with the documentation.
In particular, here is an attempt to explain why the chain rule only works for projection DPPs (and projection k-DPPs), this was written way before https://github.com/guilgautier/DPPy/commit/3da6d17a4875fa996d469acf8d06cc6d26ea798b#diff-3e642918a95c6c1b52ac0098e1fad542
FYI, the first article of [BuRaWi19] has been revised after some discussions with the authors at ICML19
HeGa16 provide only a heuristic that seem perform well in practice (this was confirmed by the first version of [BuRaWi19]) but applying blindly the chain rule doesn't yield a valid DPP nor k-DPP sample.
Note that it doesn't even appear on the first author's webpage.

To me the chain rule as described by [HKPV06] is on joint probabilities for a projection correlation kernel K
A sequential draw yields a vector; there is an order.
To be allowed to forget the order and consider the points you've sampled as a set with the appropriate distribution, one must show that the distribution of the vector is exchangeable i.e. invariant by permutation.
Otherwise one may fall in the same trap as the paper of HeGa16 you are mentioning bellow.

Yes, I agree. I'm now understanding there are two separate things at play here:

Exchangability is what allows us to state that the r! possible sampled sequences returned by sequentially running the chain rule all have the same probability. This is due to telescoping in the product of conditional probabilities (i.e. the ratio of Schur complement terms, theorem 2.2 proof in DW18), and is violated in e.g. DW18 section 4 "the analogous closed form of the sampling probability P(S) given in Theorem 2.2 is not recovered because the paths from node {1..n} to node S in the graph of Figure 2.1 do not all have the same probability" (although this paper is addressing top-down sampling).

Being a projection process means that P({x_1,...,x_r} C S) = P({x_1,...,x_r} = S), so it is correct to terminate the sampling after r steps. This is what is wrong with blindly applying the chain rule to sampling a k-DPP which is not a projection process: you end up sampling probabilities P({x_1,...,x_k}  C S) which are not equal to P({x_1,..,x_k} = S).

I'm sorry I don't understand what you mean

The event S = {x_1,...,x_{r-1},x_r } and the event S = {x_1,...,x_{r-1}} are mutually exclusive, so conditioning on the second makes the first have probability zero. I brought this up to make the point that the chain rule is for the r-point correlation function and not the joint probability mass function (P(S = {x_1,...,x_r})).

Could you clarify what you mean by "applies to all DPPs" and by "bottom up sampling" ?

"applies to all DPPs": you can always write the decomposition P({x_1,...,x_r} C S) = P({x_1} C S) P({x_1, x_2} C S | {x_1} C S) ... P({x_1, ..., x_r} C S | {x_1, ..., x_{r-1}} C S) which becomes a product of Schur complement terms.
"bottom up sampling": sorry, this is bad word choice from me. This refers to starting with an empty set and adding points (as opposed to top-down / reverse iterative sampling in DW18). I hadn't seen the Pou19 reference, but it looks paths in their tree correspond to both inclusions and exclusions (i.e. bit-vector set representations) and require O(n) steps until terminating whereas the chain rule iterative sampling requires exactly r steps.

FYI, the first version of [BuRaWi19] has been revised after some discussions with the authors at ICML19

Thank you. I will let my colleagues know.
BTW, I'm happy to do an initial pass at revising this part of the documentation if you are happy with it and would like help :)

Exchangability is what allows us to state that the r! possible sampled sequences corresponding to the same set (from applying the chain rule sequentially) all have the same probability. This is due to telescoping in the product of conditional probabilities (i.e. the ratio of Schur complement terms, theorem 2.2 proof in DW18) which holds for any DPP

That's only one tip of the iceberg!
For projection kernel K, if you write the n-th conditional that appears in the chain rule as, see also (14),
P[x_n  | S_{n-1} ] = 1/(r-(n-1)) det K_{S_{n-1}+x_n} / det K_{S_{n-1}}

you are mentioning that prod det K_{S_{n-1}+x_n} / det K_{S_{n-1}}  = det K_S is invariant by permutation
But more importantly, cf the Caution section the normalizing constant of P[x_n  | S_{n-1} ] is equal to r-(n-1) which is independent of the previous sampled points S_{n-1}.
So the their product is equal to 1/r! which is in turn independent of the final set of points S

Finally, combining 1. AND 2. proves exchangeability of the distribution of (x_1, ..., x_r)
But for generic K, one must show that 2. holds, and a priori this is not the case cf Caution section

I'm sorry I don't understand what you mean

The event S = {x_1,...,x_{r-1},x_r } and the event S = {x_1,...,x_{r-1}} are mutually exclusive, so conditioning on the second makes the first have probability zero. I brought this up to make the point that the chain rule is for the r-point correlation function and not the joint probability mass function (P(S = {x_1,...,x_r})).

This is still fuzzy to me especially because the event S = {x_1,...,x_{r-1}} has probability 0 and the r-point correlation coincides with the joint density for projection kernels

Thank you. I will let my colleagues know.

See also the email I sent you yesterday ;)

BTW, I'm happy to do an initial pass at revising this part of the documentation if you are happy with it and would like help :)

I'm glad you give me some feedback and I appreciate you help a lot!
Feel free to contribute and propose revisions of the docs :)
I am actually quite busy with writing unit tests ...
Btw, I haven't worked enough with PRs to know if you are able to amend this one or if it's better that I accept or reject it and you open a new one ?

Hi Guillaume, sorry I haven't had a chance to work on it yet. We can close
and I will resubmit a completed copy of changes when I find the time.
On Fri, Jul 19, 2019 at 3:45 AM Guillaume Gautier<EMAIL_ADDRESS>wrote:

Hi @feynmanliang https://github.com/feynmanliang 🙂
Any news about this PR ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/guilgautier/DPPy/pull/40?email_source=notifications&email_token=AAHRW5OOVGI3LIL3RKIPUHLQAFWKVA5CNFSM4H4I7QWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2K3Y3A#issuecomment-513129580,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHRW5NJUBIP34CUDA6VLFTQAFWKVANCNFSM4H4I7QWA
.

Hi @feynmanliang,
I've spent some time reshaping the exact sampling section entirely.
In particular, I've written a specific Caution section on why the chain cannot be applied blindly to sample k-DPP(L).
Feedback is welcome 😃
Enjoy the rest of the summer 🌴


Laravel-10-compatibility

#15 fixed.

Thanks for the PR, I'll merge it and publish a new version once the tests pass!


Devops

Note: The verbiage expressed here are that of the author and are not representative of State Farm.
This project requires implementation of ci-cd.
A proper devops pipeline will validate code quality, run tests, publish documentation, build the package, and deploy to pypi such that the package is pip installable.

@StateFarmIns/sponsored-project-maintainers checking in, we're trying to get our CI in gitlab and github married up; are there already standard processes for working with bot accounts?
Thanks for any help you can give!


Check corrections in Rudolph on Raku Advent

https://github.com/Raku/advent/blob/master/raku-advent-2020/authors.md

huh?


Do not printout basename usage

When users command "bacardi" only, basename usage shouldn't come out.
ISSUE=#204

@romandev, if no subcommand, (ex: ./bacardi) doesn't it go through the default task then print out all the commands? If I get the wrong point, please fix me.

@nadongguri
I think you can write the code as follows (pseudo code):
if there is a subcommand:
    processSubcommand()

processGulp()

@romandev, I got your point. :)


Mission X Failure

I have been trying to complete Mission X, but it isn't going. All other marks are checked except my username

Same with me


Don't include RKMappingTestExpectation.h and RKMappingTest.h in the Rest...

...Kit target since those are testing files.

Wrong branch. Sorry


feat(python): Support reranking for vector and fts

solves https://github.com/lancedb/lancedb/issues/1086
Usage Reranking with FTS:
retriever = db.create_table("fine-tuning", schema=Schema, mode="overwrite")
pylist = [{"text": "Carson City is the capital city of the American state of Nevada. At the  2010 United States Census, Carson City had a population of 55,274."},
          {"text": "The Commonwealth of the Northern Mariana Islands is a group of islands in the Pacific Ocean that are a political division controlled by the United States. Its capital is Saipan."},
        {"text": "Charlotte Amalie is the capital and largest city of the United States Virgin Islands. It has about 20,000 people. The city is on the island of Saint Thomas."},
        {"text": "Washington, D.C. (also known as simply Washington or D.C., and officially as the District of Columbia) is the capital of the United States. It is a federal district. "},
        {"text": "Capital punishment (the death penalty) has existed in the United States since before the United States was a country. As of 2017, capital punishment is legal in 30 of the 50 states."},
        {"text": "North Dakota is a state in the United States. 672,591 people lived in North Dakota in the year 2010. The capital and seat of government is Bismarck."},
        ]
retriever.add(pylist)
retriever.create_fts_index("text", replace=True)

query = "What is the capital of the United States?"
reranker = CohereReranker(return_score="all")
print(retriever.search(query, query_type="fts").limit(10).to_pandas())
print(retriever.search(query, query_type="fts").rerank(reranker=reranker).limit(10).to_pandas())

Result
                                                text                                             vector     score
0  Capital punishment (the death penalty) has exi...  [0.099975586, 0.047943115, -0.16723633, -0.183...  0.729602
1  Charlotte Amalie is the capital and largest ci...  [-0.021255493, 0.03363037, -0.027450562, -0.17...  0.678046
2  The Commonwealth of the Northern Mariana Islan...  [0.3684082, 0.30493164, 0.004600525, -0.049407...  0.671521
3  Carson City is the capital city of the America...  [0.13989258, 0.14990234, 0.14172363, 0.0546569...  0.667898
4  Washington, D.C. (also known as simply Washing...  [-0.0090408325, 0.42578125, 0.3798828, -0.3574...  0.653422
5  North Dakota is a state in the United States. ...  [0.55859375, -0.2109375, 0.14526367, 0.1634521...  0.639346
                                                text                                             vector     score  _relevance_score
0  Washington, D.C. (also known as simply Washing...  [-0.0090408325, 0.42578125, 0.3798828, -0.3574...  0.653422          0.979977
1  The Commonwealth of the Northern Mariana Islan...  [0.3684082, 0.30493164, 0.004600525, -0.049407...  0.671521          0.299105
2  Capital punishment (the death penalty) has exi...  [0.099975586, 0.047943115, -0.16723633, -0.183...  0.729602          0.284874
3  Carson City is the capital city of the America...  [0.13989258, 0.14990234, 0.14172363, 0.0546569...  0.667898          0.089614
4  North Dakota is a state in the United States. ...  [0.55859375, -0.2109375, 0.14526367, 0.1634521...  0.639346          0.063832
5  Charlotte Amalie is the capital and largest ci...  [-0.021255493, 0.03363037, -0.027450562, -0.17...  0.678046          0.041462

Vector Search usage:
query = "What is the capital of the United States?"
reranker = CohereReranker(return_score="all")
print(retriever.search(query).limit(10).to_pandas())
print(retriever.search(query).rerank(reranker=reranker, query=query).limit(10).to_pandas()) # <-- Note: passing extra string query here

Results
                                                text                                             vector  _distance
0  Capital punishment (the death penalty) has exi...  [0.099975586, 0.047943115, -0.16723633, -0.183...  39.728973
1  Washington, D.C. (also known as simply Washing...  [-0.0090408325, 0.42578125, 0.3798828, -0.3574...  41.384884
2  Carson City is the capital city of the America...  [0.13989258, 0.14990234, 0.14172363, 0.0546569...  55.220200
3  Charlotte Amalie is the capital and largest ci...  [-0.021255493, 0.03363037, -0.027450562, -0.17...  58.345654
4  The Commonwealth of the Northern Mariana Islan...  [0.3684082, 0.30493164, 0.004600525, -0.049407...  60.060867
5  North Dakota is a state in the United States. ...  [0.55859375, -0.2109375, 0.14526367, 0.1634521...  64.260544
                                                text                                             vector  _distance  _relevance_score
0  Washington, D.C. (also known as simply Washing...  [-0.0090408325, 0.42578125, 0.3798828, -0.3574...  41.384884          0.979977
1  The Commonwealth of the Northern Mariana Islan...  [0.3684082, 0.30493164, 0.004600525, -0.049407...  60.060867          0.299105
2  Capital punishment (the death penalty) has exi...  [0.099975586, 0.047943115, -0.16723633, -0.183...  39.728973          0.284874
3  Carson City is the capital city of the America...  [0.13989258, 0.14990234, 0.14172363, 0.0546569...  55.220200          0.089614
4  North Dakota is a state in the United States. ...  [0.55859375, -0.2109375, 0.14526367, 0.1634521...  64.260544          0.063832
5  Charlotte Amalie is the capital and largest ci...  [-0.021255493, 0.03363037, -0.027450562, -0.17...  58.345654          0.041462

LGTM!
Could we add an example of using rerank in the search function docs?

some points that can be made better :

support for passing string query for vector search by default
if the user passed in a vector directly, then the user needs to pass in the query string for reranking directly
error handling for above and documentation

Thanks. Makes sense. I'll add that

LGTM , test added , enabling squash


[SPARK-40774][CONNECT] Add Sample to proto and Connect DSL

What changes were proposed in this pull request?

Add sample to proto and connect DSL.
Add the missing plan comparison for column alias test case.

Why are the changes needed?

Improve API coverage on the connect proto.
Does this PR introduce any user-facing change?

No
How was this patch tested?

UT

R: @cloud-fan

Can one of the admins verify this patch?

thanks, merging to master!


Add config item to set base URL for UI when using media library

Fixes #1934
I am not 100% sure this is the right way to add this functionality, but it allows setting display_base_url to be used when output_filename_only=true, so that the UI can render images from external media libraries.

I'd like to see a zero config solution for this, see my comment on the issue: https://github.com/netlify/netlify-cms/issues/1934#issuecomment-459911932
What do you think?

Closing as stale due to lack of activity and/or interest. Please comment if you feel this should be reopened.

Hi @erquhart this was be closed without merging, right?
I am looking for even temporary/not ideal solution to get this working. Some hardcoded value like display_base_url is fine...
Any idea how?
Except the option to create and maintain own build.

@pavelbinar can you give me a quick explanation of your use case?


Test disappears from test explorer

From https://developercommunity.visualstudio.com/content/problem/271082/python-test-disappears-when-trying-to-run-selected.html
When I try and run the selected test from test explorer the test disappears from the list.
My problem is that sometimes all of my python unit tests for a given project disappear when I edit the code such way that it renders the resulting python code (temporarily) invalid. Which of course is the way people write code. I guess it happens most of the time when the unit test function is empty (I delete the auto-generated comments and the pass statement) and the function signature is immediately followed by the next one. Only restarting visual studio help.

Fixed for 16.3. Discovery is no longer triggered on typing (triggers on file save and settings change)


RuntimeError: generator raised StopIteration

I installed all the dependencies but i dont know why i am getting the runtime error
C:\Users\home\Fake-Apache-Log-Generator>python apache-fake-log-gen.py
Traceback (most recent call last):
File "C:\Users\home\Fake-Apache-Log-Generator\apache-fake-log-gen.py", line 29, in iter
raise StopIteration
StopIteration
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\home\Fake-Apache-Log-Generator\apache-fake-log-gen.py", line 62, in
for case in switch(output_type):
RuntimeError: generator raised StopIteration

Hi, as a short-term fix, you can run in Python 3.6 or earlier and this error won't be thrown (the error may be related to new behaviour in Python 3.7 and later, as per https://stackoverflow.com/questions/51700960/runtimeerror-generator-raised-stopiteration-every-time-i-try-to-run-app)


Setup fails with Error: The process '/usr/bin/bash' failed with exit code 1

Describe the bug
Setup fails, see:
Run shivammathur/setup-php@v[2](https://github.com/pimcore/poeditor-export-action/actions/runs/4541612220/jobs/8003997392#step:3:2)
  with:
    php-version: 8.2
    ini-file: production
/usr/bin/bash /tmp/runner/work/_actions/shivammathur/setup-php/v2/src/scripts/run.sh

==> Setup PHP
✗ PHP Could not setup PHP [8](https://github.com/pimcore/poeditor-export-action/actions/runs/4541612220/jobs/8003997392#step:3:9).2
Error: The process '/usr/bin/bash' failed with exit code 1

==> Setup PHP
✗ PHP Could not setup PHP 8.2
Error: The process '/usr/bin/bash' failed with exit code 1
Version

[ ] v2

Runners

[ ] GitHub Hosted

Operating systems

PHP versions
8.2
To Reproduce
            - name: "Install PHP"
              uses: "shivammathur/setup-php@v2"

@brusch
Please try to rerun the workflow and it should work correctly. This might have been a one-off issue as GitHub was down earlier today.
If you are still facing the issue on rerun please reproduce this on a public repo and share the workflow link.

We're also sometimes running into this (happens frequently, but not on every run)
I just switched to the verbose action and this is our logs on a failed attempt:
Run shivammathur/setup-php@verbose
  with:
    php-version: 8.1
    extensions: rdkafka
    ini-file: production
  env:
    fail-fast: true
    RDKAFKA_LIBS: librdkafka-dev
/usr/bin/bash /home/runner/work/_actions/shivammathur/setup-php/verbose/src/scripts/run.sh

==> Setup PHP
update-alternatives: using /usr/sbin/php-fpm8.1 to provide /usr/sbin/php-fpm (php-fpm) in auto mode
✓ xml Enabled
✓ php-config Added php-config
✓ PECL Added PECL 1.10.12

production
date.timezone=UTC
memory_limit=-1
opcache.enable=1
opcache.jit_buffer_size=256M
opcache.jit=1235
xdebug.mode=coverage
php-version=8.1.17
✓ PHP Updated to PHP 8.1.17

==> Setup Extensions
ID=ubuntu
VERSION_ID=22.04
VERSION_CODENAME=jammy
deb [arch=amd64 signed-by=/usr/share/keyrings/ondrej-php-keyring.gpg] http://ppa.launchpad.net/ondrej/php/ubuntu jammy main
Get:1 http://ppa.launchpad.net/ondrej/php/ubuntu jammy InRelease [23.9 kB]
Get:2 http://ppa.launchpad.net/ondrej/php/ubuntu jammy/main amd64 Packages [110 kB]
Get:3 http://ppa.launchpad.net/ondrej/php/ubuntu jammy/main Translation-en [34.2 kB]
Fetched 168 kB in 1s (161 kB/s)
Reading package lists...

[apt-fast 11:35:37]
[apt-fast 11:35:37]Working... this may take a while.
E: Unmet dependencies. Try 'apt --fix-broken install' with no packages (or specify a solution).
[apt-fast 11:35:37]Package manager quit with exit code.

E: Unmet dependencies. Try 'apt --fix-broken install' with no packages (or specify a solution).
[apt-fast 11:35:47]Package manager quit with exit code.
✓ xml Enabled
✓ php-config Added php-config 8.1.17
config-set succeeded
Updating channel "pear.php.net"
Update of Channel "pear.php.net" succeeded
config-set succeeded
Updating channel "pecl.php.net"
Update of Channel "pecl.php.net" succeeded

✓ PECL Added PECL 1.10.12
[apt-fast 11:35:49]
[apt-fast 11:35:49]Working... this may take a while.
E: Unmet dependencies. Try 'apt --fix-broken install' with no packages (or specify a solution).
[apt-fast 11:35:49]Package manager quit with exit code.
[apt-fast 11:35:50]
[apt-fast 11:35:50]Working... this may take a while.
E: Unmet dependencies. Try 'apt --fix-broken install' with no packages (or specify a solution).
[apt-fast 11:35:50]Package manager quit with exit code.
✗ librdkafka-dev Could not install librdkafka-dev
Error: The process '/usr/bin/bash' failed with exit code 1

@nicklasmoeller I have rebuilt the cache, that error should be fixed now.

@brusch
Closing this, Please let me know if it is failing again.


Initramfs  to load modules/firmware and mount rootfs

@ndechesne This is based upon @lumag PR, https://github.com/ndechesne/meta-qcom/pull/232 .

I don't like a 60MB initramfs.. that's too big..

@ndechesne Ack, that's because all firmware/modules built are installed inside the ramdisk.
Modules  52M     ./lib/modules
Firmware 48M     ./lib/firmware
We will need a manner to set which ones are needed to both for example UFS in DB845c.

I don't like a 60MB initramfs.. that's too big..

@ndechesne Ack, that's because all firmware/modules built are installed inside the ramdisk.
The uncompressed ramdisk is 169M where,
Modules 52M ./lib/modules
Firmware 48M ./lib/firmware
We will need a manner to set which ones are needed to both for example UFS in DB845c.

@ndechesne Removing the firmware and modules the ramdisk size is ~9MB compressed.

I don't like a 60MB initramfs.. that's too big..

@ndechesne Ack, that's because all firmware/modules built are installed inside the ramdisk.
The uncompressed ramdisk is 169M where,
Modules 52M ./lib/modules
Firmware 48M ./lib/firmware
We will need a manner to set which ones are needed to both for example UFS in DB845c.

@ndechesne Removing the firmware and modules the ramdisk size is ~9MB compressed. We can start from there and then figure out which/how-to add needed firmware/modules in maintainable way.

debian maintains a list of modules to include in their initramfs..
https://salsa.debian.org/kernel-team/initramfs-tools/-/blob/master/hook-functions#L600
not sure how that can be applied to OE.. but perhaps we are doing something generic and usptreamable here..

I don't like a 60MB initramfs.. that's too big..

@ndechesne Ack, that's because all firmware/modules built are installed inside the ramdisk.
The uncompressed ramdisk is 169M where,
Modules 52M ./lib/modules
Firmware 48M ./lib/firmware
We will need a manner to set which ones are needed to both for example UFS in DB845c.

@ndechesne Removing the firmware and modules the ramdisk size is ~9MB compressed. We can start from there and then figure out which/how-to add needed firmware/modules in maintainable way.

debian maintains a list of modules to include in their initramfs..
https://salsa.debian.org/kernel-team/initramfs-tools/-/blob/master/hook-functions#L600
not sure how that can be applied to OE.. but perhaps we are doing something generic and usptreamable here..

I added NO_RECOMMENDATIONS to initramfs-rootfs-image to avoid install all firmware/modules.

This is now implemented in an alternative way.


embedded relationship data does not honor relationship's object_method_name

Expected Behavior vs Actual Behavior
When using has_many, I would expect both the included, related objects and the source object's relationship data to use the same specified object_method_name, however only the "included" objects seem to use the method. Using the block definition for has_many works as I would expect.
Steps to Reproduce the Problem
class Location < ApplicationRecord
  has_many :deals
  has_many :active_deals, -> { where(active: true) }, source: :deals
end

class LocationSerializer
  include FastJsonapi::ObjectSerializer
  has_many :deals, object_method_name: :active_deals
end

# create 1 deal for the location with active as false

pp LocationSerializer.new(@location).to_json

{"data"=>
  [{"id"=>"xxx",
    "type"=>"location",
    "attributes"=>
     {},
    "relationships"=>
     {"deals"=>{"data"=>[{"id"=>"xxx", "type"=>"deal"}]}}}],
 "included"=>[]}

The 'inactive' deal is included in the location's relationship data.
Specifications

Version: 1.6.0
Ruby version: 2.6.5

@waymondo may I ask you what ActiveRecord/Rails version are you using? I remember Rails 4 would have issues with the scoped relationships. But at least it will be easier to replicate. Thanks!

We are on rails 5.1.7

I have the same issue on rails 6.0.3

There are 2 workarounds:

Use the object_block instead: has_many :deals, &:active_deals
Set polymorphic: true wherever you use object_method_name

Seems like a bug in Relationship - object_method_name is only called in one place: #fetch_associated_object. However, that method is never called if you have a @static_record_type. Hence the polymorphic workaround.


revert: Revert SearchAPI version

Description
Contest-isolation was not merged to R55.2 so reverting this changes
Solution Approach
N/A
Related PRs
N/A

@VishwasShashidhar If we merge the context-isolation PR on the SFE in R55.2 we will not be needing this PR.
And if that is not the case please merge the PR and will be changing this back to 1.55.x.

closing this in favor of #681


2024.03

Added support for 2024 March release of HA.
Will keep in draft as it depends on 4 patches I have send to OE for upgrading of several recipes.

Patches have been merged into master and scarthgap. Will trigger build again.


bugfix: when tm commit or rollback failed,xid need to be clean

Ⅰ. Describe what this PR did
fix bug #1734
Ⅱ. Does this pull request fix one issue?

fixes #1734
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1735 into develop will increase coverage by 0.03%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##             develop   #1735      +/-   ##
============================================
+ Coverage      48.17%   48.2%   +0.03%
  Complexity      1846    1846
============================================
  Files            365     365
  Lines          13308   13308
  Branches        1645    1645
============================================
+ Hits            6411    6415       +4
+ Misses          6233    6231       -2
+ Partials         664     662       -2

Impacted Files
Coverage Δ
Complexity Δ

...java/io/seata/tm/api/DefaultGlobalTransaction.java
67.74% <50%> (ø)
15 <0> (ø)
:arrow_down:

...server/store/file/FileTransactionStoreManager.java
47.38% <0%> (+1.39%)
19% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Wait for deletion, exception fix

Updated exception message

Making few other changes, don't merge for now.


Issue with notification

I am getting the following error.
Error:==> Enabling notifications failed
Error:==> Error 6: The request is not supported.
Can anyone let me know what's wrong?
Code:
        guard let firmware = firmwareProvider?.firmware else {
            Logs.shared.log.value.append("No firmware found")
            return
        }

        let initiator = DFUServiceInitiator(queue: DispatchQueue(label: "microbit_flushing"))
        initiator.logger = self
        initiator.delegate = self
        initiator.progressDelegate = self
        initiator.forceDfu = true
        initiator.enableUnsafeExperimentalButtonlessServiceInSecureDfu = true

        let dfu_uuid = [DFUUuid(withUUID: GATTKey.DFUServiceUUID, forType: .buttonlessService),
                        DFUUuid(withUUID: GATTKey.DFUCharacteristicUUID, forType: .buttonlessCharacteristic)]

        initiator.uuidHelper = DFUUuidHelper(customUuids: dfu_uuid)

        if #available(iOS 11.0, macOS 10.13, *) {
            initiator.packetReceiptNotificationParameter = 0
        }

        firmwareProvider?.applyModifier(to: initiator)

        if let _ = self.getPath(of: "abc"), let target = self.blePeripheral {
            //            let firmware = self.didSelectFirmwareType(.softdeviceBootloaderApplication, selectedFileURL: url) {
            //            print(firmware.parts)
            self.dfuController = initiator.with(firmware: firmware).start(target:target)
        }

for performing DFU

I understand that your device is using non-standard UUIDs. Are you sure you are setting correct services for buttonless service and its characteristic? Seems like you are setting the DFU service and DFU characteristic UUIDs for buttonless service.
Does your characteristic have Client Characteristic Configuration Descriptor (CCCD) and supports Indications/Notifications?

Hi Philips,
Thanks for replying.
I am afraid. What do you mean by non-standard UUIDs? and how shall I know characteristic have CCCD.

Hi @philips77 ,
I figure out what is CCCD. Now I am getting errors like below.
application:==> Buttonless DFU notifications enabled
warning:==> Application with buttonless update found
State changed to: Enabling DFU Mode
verbose:==> Writing to characteristic E97D3B10-251D-470A-A062-FA1922DFA9A8...
debug:==> peripheral.writeValue(0x01, for: E97D3B10-251D-470A-A062-FA1922DFA9A8, type: .withResponse)
error:==> Writing to characteristic failed
error:==> Error 3: Writing is not permitted.
verbose:==> Disconnecting...
debug:==> centralManager.cancelPeripheralConnection(peripheral)
debug:==> [Callback] Central Manager did disconnect peripheral
info:==> Disconnected

@philips77
Thanks in advance.
I would like to know, what am I doing wrong?

Hi, sorry that you had to wait that long. Did you manage to solve your problem?

Closing the issue due to inactivity.


minor bug fix: properly await AsyncRunManager's  method call in MulitRouteChain

This simply awaits AsyncRunManager's method call in MulitRouteChain.  Noticed this while playing around with Langchain's implementation of MultiPromptChain.  @baskaryan
cheers

thanks @fielding!


Add feature to exclude a team from mentions on PR's made by a member

This provides a new option, skipTeamPrs, to the configuration options for alwaysNotifyForPaths and fallbackNotifyForPaths.
This option works with GitHub teams. The idea is, if you specify that a team should watch a certain path, but then a member of that team modifies one of their own files, then that team does not need to be included in the resulting comment because they already know about it. However, any other teams the PR creator is not a member of should still get notified.
This only applies to alwaysNotifyForPaths and fallbackNotifyForPaths since a team will never be mentioned by parsing the blame history. Nevertheless, it's hidden behind a config flag to cut down on the number of HTTP requests made in the case where individual users, rather than teams, are watching certain file paths and for the benefit of teams that would prefer they get mentioned in all cases.

Sounds useful! Thanks!
I haven't done a good review but I trust you and the unit test you wrote :)


Problem reading UPS power when value is negative

Hi,
I have a Deye SUN SG01HP3-EU
I am using the metrics
DEYE_METRIC_GROUPS=deye_sg01hp3,deye_sg01hp3_battery,deye_sg01hp3_ups,settings
This model has three load connections, one named UPS (which is normally used for backup loads), one for smart loads and/or a generator, and the main connection which not necessarily has to be connected directly to the grid, as there can be loads connected in between the inverter and the grid.  If there are other solar inverters in the system, here is where they should be connected. If so, there should be a set of CT's or a modbus power metter conected to the entry grid for the Deye to know about the feed-in/buying power to/from the grid.  When in this last setup, you should select the Work Mode "Zero Export to CT" .
Problem arises when selecting "Zero Export to CT", and there are other inverters on the system. It looks like when in this mode, the server consider UPS load not the loads on the UPS line, but the sum of the loads on the UPS line and the main line, as it considers grid connection where the CT are, not at the main line connection.  Having other inverters in the system, the UPS powers reported can be negative. And this is correctly reported by the Deye screen.
But when negative UPS power happens, deye-inverter-mqtt does not understand it, reporting something around 65KW in each on the negative phases (something around 65535, or 2^16)
The expected behabiour should be reporting the negative power correctly.
Regards.

@fakehec It means that this register is currently mapped as unsigned int, while it should be mapped as signed int. Which metric is it exactly?

ac/ups/l1/power
ac/ups/l2/power
ac/ups/l3/power
Guess they are at deye_sg01hp3_ups

@fakehec Fixed in the new release

Thanks a lot!


Lang file for sponge plugin

Hi! Add the lang file/class pleace. I wanna translate your plugin ^_^

Localization files are split up by module, for core, bukkit, and Sponge.
Core: https://github.com/PEXPlugins/PermissionsEx/blob/master/permissionsex-core/src/main/po/keys.pot
bukkit: https://github.com/PEXPlugins/PermissionsEx/blob/master/permissionsex-bukkit/src/main/po/keys.pot
Sponge: https://github.com/PEXPlugins/PermissionsEx/blob/master/permissionsex-sponge/src/main/po/keys.pot
They aren't actually used at the moment since gettext is a bit broken, but those files are kept up to date with changes to PEX's messages.


Change URL

I guess in the future this should also come from config. callback url can probably be generated from flask using url_for(callback_route, _external=True)


ARROW-4543: [C#] Update Flat Buffers code to latest version

Update FlatBuffers code to latest version.
Mark all FlatBuffer types as internal.

Note: I didn't use the latest .fbs file version because it included the SparseTensor support. Using the latest .fbs change caused problems with the Tensor generation. I have a question if the latest .fbs file is correct.
/cc @chutchinson @stephentoub - FYI

@wesm @xhochy - how do I re-run the travis-ci? I don't think the failure is related to my proposed change.

I restarted the failed job manually.
You can re-run all jobs by force push.

Can you add a line to the README on how to update these files?

Done.  Thanks for the review.

Thanks @eerhardt!


Deployment Documentation

I couldn't find any relevant deployment docs, how should permitta be deployed?

Hi Harshit,
Thanks for your interest in permitta!
I can help with some deployment docs, but can you let me know what your
environment is?
Are you planning to run in kubernetes? Where would your user metadata be
stored, do you have active directory or something similar? Also what about
the catalog of your data objects, do you have a data catalog or would you
want them to come from trino directly?
Joel
On Thu, 3 Oct 2024 at 6:28 pm, Harshit @.***> wrote:

I couldn't find any relevant deployment docs, how should permitta be
deployed?
—
Reply to this email directly, view it on GitHub
https://github.com/permitta/permitta/issues/40, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABI2L7KPDZIQOUHQOEN7IDTZZT5ZTAVCNFSM6AAAAABPJIZC5GVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU3DGNBTGUZTANA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


View queue, topic and subscription message counts (including scheduled)

I would like to be able to obtain the active, pending and dead-letter message counts from a queue, topic or subscription
Either via ServiceBusAdministrationClient or the receiver itself?
You used to be able to do this with the old library?

@alexespencer Thanks for your question!
The active, scheduled, and dead-letter message counts are all provided by the *RuntimeProperties objects returned by the list_*_runtime_properties methods on the ServiceBusAdministrationClient.
For example, you can use the get_queue_runtime_properties  method to obtain the QueueRuntimeProperties object:
import os
import uuid
from azure.servicebus.management import ServiceBusAdministrationClient

CONNECTION_STR = os.environ['SERVICE_BUS_CONNECTION_STR']
QUEUE_NAME = os.environ['SERVICE_BUS_QUEUE_NAME']

servicebus_mgmt_client = ServiceBusAdministrationClient.from_connection_string(CONNECTION_STR)

# below returned object is a QueueRuntimeProperties object
queue_info = servicebus_mgmt_client.get_queue_runtime_properties(QUEUE_NAME)

# access the active, scheduled, and dead-letter message counts
print(queue_info.active_message_count)
print(queue_info.scheduled_message_count)
print(queue_info.dead_letter_message_count)

You can also find similar methods for subscription and topic here and here.
Thanks!

@alexespencer - You mentioned that you were able to find the active, scheduled, and dead-letter message counts in the "old library". Out of curiosity, were you referring to azure-servicebus==0.50.3 or a different version? This would be useful to know so that we can update our migration guide with this info to help others that have the same question as you. Thanks!

Yes, that is correct. I also checked through the examples you have in the
git repo (like receiving / sending etc) and there isn't an example that
does this. So that could help too?
On Thu, 19 Aug 2021, 18:53 swathipil, @.***> wrote:

@alexespencer https://github.com/alexespencer - You mentioned that you
were able to find the active, scheduled, and dead-letter message counts in
the "old library". Out of curiosity, were you referring to
azure-servicebus==0.50.3 or a different version? This would be useful to
know so that we can update our migration guide
https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/servicebus/azure-servicebus/migration_guide.md#working-with-administration-client
with this info to help others that have the same question as you. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-python/issues/20245#issuecomment-902120607,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYQQPIUD4WWNKF2BKULBDTT5VAJ7ANCNFSM5CCFQNRA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Very helpful to know, thanks!


Training failed.  Failed to create trained image after successful training run.

Hi,
after a pair of successful training (SDXL), all my trainings are ending with this message: "Training failed. Failed to create trained image after successful training run."
I've tried to create a new destination model and create a new API Token to start the creation using the Nodejs Client Library.
I've started a dozen of trainings using the web interface but the response is always the same.
Can someone please have a look into this?
Thank you very much!

Hi @khenzo. Can you please share a link to a failed training? Any code you can share would also be helpful for understanding what the problem might be.

Closing this for now. Happy to reopen if you can share more information to troubleshoot this.

HI
I HAVE THE SAME PROBLEM
ANY ADVICE PLEASE

Training failed.
Prediction input failed validation: {"detail":[{"loc":["body","input","images"],"msg":"'' is not a valid URL scheme. 'data', 'http', or 'https' is supported.","type":"value_error"}]}

{
"completed_at": "2024-10-20T18:23:35.925477Z",
"created_at": "2024-10-20T18:23:35.920000Z",
"data_removed": false,
"error": "Prediction input failed validation: {"detail":[{"loc":["body","input","images"],"msg":"'' is not a valid URL scheme. 'data', 'http', or 'https' is supported.","type":"value_error"}]}",
"id": "9es8e34zy1rj20cjncns14fny4",
"input": {
"steps": 1000,
"images": "marymway",
"repo_id": "marymway/flux.1-dev-lora",
"hf_token": "[REDACTED]",
"batch_size": 1,
"model_name": "marymway/flux.1-dev",
"resolution": "512,768,1024",
"lora_linear": 16,
"learning_rate": 0.0004,
"lora_linear_alpha": 16
},
"logs": null,
"metrics": {
"total_time": 0.005477
},
"output": null,
"started_at": "2024-10-20T18:23:35.925477Z",
"status": "failed",
"urls": {
"get": "https://api.replicate.com/v1/trainings/9es8e34zy1rj20cjncns14fny4",
"cancel": "https://api.replicate.com/v1/trainings/9es8e34zy1rj20cjncns14fny4/cancel"
},
"version": "06e50f60983aa9ad9e4c13ba1d56ee235925ee6bd1c604d94c26a3322aeb8d47"
}


Question about vnc client browser and flash games

Hi,
When I run the sample code:
env = gym.make('flashgames.DuskDrive-v0')
env.configure(remotes=1)  # automatically creates a local docker container
observation_n = env.reset()
print observation_n

while True:
  action_n = [[('KeyEvent', 'ArrowUp', True)] for ob in observation_n]  # your agent here
  observation_n, reward_n, done_n, info = env.step(action_n)
  print observation_n
  print reward_n
  print info
  env.render()

I get a window with a browser that looks like chrome or firefox where the flashgames are running.
I was expecting it to be running inside the docker instance, and was also expecting perhaps the browser command to be specified inside the ENTRYPOINT directive inside the Dockerfile, but examining the Dockerfile I don't see anything specifying what command is run inside the docker instance, other than the go vnc client inside universe/envs/vnc_env.py
Where is the browser command being specified?

Hi @CharlesJQuarra,
Indeed, the window is a Chrome browser running inside a Docker instance on your local machine.
Just to be clear, what Dockerfile are you looking at? The universe repository does not include the code that runs inside the Docker container - we plan to tidy up the code that's inside the Docker for a fully supported public release within the next few months.
The command that gets run inside Docker does more than just start a browser - it configures a more substantial environment-side control plane that we wrote, which handles the rewarder channel connection to the agent, revives the browser if it crashes (you can try doing open vnc://localhost:5900 after running that command, and then manually clicking "close" on the browser tab... within a few seconds the browser comes back!), and more.
If you let us know what it is you'd like to do with the browser command inside the Docker container, we can help you figure out how to do what it is you'd like to do!

I'm going to close this github issue, as there is no bug report here for us to fix, but if you have followup questions please don't hesitate to ask us on https://discuss.openai.com/. Good luck with Universe!


would be nice - demo php

Nice script! It would be nice if you could include a sample php file like countries.php that you use in your demo.

Just create a php file that return a json like this:
[
  {
    "id": "BR",
    "label": "Brazil"
  },
  {
    "id": "IO",
    "label": "British lndian Ocean Territory"
  },
  {
    "id": "BN",
    "label": "Brunei Darussalam"
  }
]

http://getwebhelp.com/bootcomplete/ajax/countries.php?query=br


Adding OperationResults API to Microsoft.Capacity 2019-04-01 preview

…existing examples
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

The validation error that we are getting is unrelated to the swagger changes that I'm making in this PR. The problem is that existing swagger is incorrect, and does not match the correct example file. To fix this, we will have to change the swagger and it will count as a breaking change. Is it possible to check in this PR and take the fix for GET available scopes as a separate PR?


How can I transfer an image pointer obtained by librealsense  via DMA?

Required Info

Camera Model
D400

Firmware Version
<IP_ADDRESS>

Operating System & Version
ubuntu 20.04

Kernel Version (Linux Only)
5.15.3(xilinx)

Platform
xilinx zynq Mpsoc (armhf)

SDK Version
2.53.1

Language
C++

Segment
Robot

Hi, I'm a beginner in systems engineering and I'm not familiar with the camera system. Our team is designing an AI platform for robots.
I have installed and used Ubuntu-20.04 and librealsense on the ARM embedded board (Xilinx FPGA, Zynq MPSoC).
I was able to successfully get a pointer of the color image using librealsense functions such as wait_for_frames(), get_color_frame(), and get_data().
Now, I want to transfer this image to the HW_AI_CORE via a DMA, where both DMA and AI_CORE are implemented on the FPGA side.
I'm considering two ways to achieve this:

Convert the pointer value of librealsense's get_data() function return to a physical address and input it to the DMA.
Command librealsense to use my pre-created DMA pool memory when creating a buffer for frameset.

I have some questions:

Is RealSense's memory buffer created in kernel space or user space?
Is the pointer returned by the get_data() function a continuous memory available to the DMA, like using GFP_DMA?
Can I use my pre-made DMA pool memory when creating a buffer for frameset in librealsense?
Do you have any other recommendations for a better way to do this?

Thank you.

Hi @muonkmu  This subject has not previously been discussed on this support forum.  Whilst I do not have direct answers to the questions or suggestions for a better way to implement your goal, I can highlight some references that may be helpful.

The RealSense SDK uses V4L2_MEMORY_USERPTR, as advised at https://github.com/IntelRealSense/librealsense/issues/7098#issuecomment-673615494

The SDK's frames have a 'frame handle' that acts as a smart pointer, and get_data() can retrieve data from the frame handle - see https://github.com/IntelRealSense/librealsense/issues/5087#issuecomment-550459833

I do not have information about DMA pool memory to answer this question.  However, SDK frame buffer management documentation is provided at the link below.

https://github.com/IntelRealSense/librealsense/wiki/Frame-Buffering-Management-in-RealSense-SDK-2.0
When using a pipeline, the pipeline has a frame queue that buffers the received frames.
https://github.com/IntelRealSense/librealsense/wiki/Frame-Buffering-Management-in-RealSense-SDK-2.0#pipeline

Hi @muonkmu  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


Fix different error message in source and package.nls for npm package

@qcz Thanks for the fix!

@egamma: This PR is missing from the Thank You section of v1_18.md :)

@qcz sorry, I forgot to assing a milestone to the PR, this is why we didn't find it. I will add it.

@egamma: No problem! Thanks!


potential issue with write_summary_tsv

in line 272 in ui.py we have
    # FIXME remove once MostDistantCandidateSpace exists...
    method = candidate_space.method
    if (
        candidate_space.governing_method == Method.FAMOS
        and candidate_space.predecessor_model.predecessor_model_hash is None
    ):

however, the predecessor_model can be either a Model, or str. In the case of str this code will fail with an AttributeError.

Hi Frank, thanks! Should be fixed in #38 and available here and via PyPI.

thanks that fixes this issue.


workaround

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Not needed anymore.


Screenshots and terminology like Directory ID needs to be updated

[

The screenshot with Directory ID should be updated to Tenant ID]

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

ID: 186c0697-792a-a913-1baa-0af86f7ef6de
Version Independent ID: 330e7927-57fd-75e2-26b1-05bd2e8f30c1
Content: VPN Gateway: Azure AD tenant for P2S VPN connections: Azure AD authentication
Content Source: articles/vpn-gateway/openvpn-azure-ad-tenant.md
Service: vpn-gateway
GitHub Login: @anzaman
Microsoft Alias: alzam

@SakethTubati Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thank you 👍

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Tig10 Charge

The 125 integration seems to have popped out (or in) to tig10 processing again.
That is to say my energies are now x125 too large.
Is there a .cal file flag or something else easy that can be set to get the old behaviour back?
TSiLi has some internal energy dependent functions (or will soon) so the quick, just divide the output, isnt good enough.

No I don’t think there is anything in there right now. We could add a ScaleCalibration(Mnemonic) function which takes a mnemonic and scales all channels with that mnemonic. Is there an offset that also needs to be scaled?

Yeah that is still there. I guess Tigress doesn't deal with variable integration lengths so that would work.

It does not.  The tig10 are all 125 and the tig64's are all 25.
On Fri, Oct 28, 2016 at 9:16 AM, Ryan Dunlop<EMAIL_ADDRESS>wrote:

Yeah that is still there. I guess Tigress doesn't deal with variable
integration lengths so that would work.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/GRIFFINCollaboration/GRSISort/issues/808#issuecomment-256917399,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAacI-1McnvMgM7tEQTQn7wNT2w4fWAVks5q4fW0gaJpZM4Ki-m6
.

@r3dunlop Is it still there? Could you give me more specific directions I couldnt see it in TGRSIRunInfo.h

Yep. Theres a function in TChannel called SetIntegration. This sets the denominator of the charge/integration calculation. The equation is basically gain*raw_charge/integration + offset.
To change all of the channels with a specific mnemonic, for example GRIFFIN, you can start typing that mnemonic in the static TChannel function:
TChannel::SetIntegration("GR",700) //for all griffins
TChannel::SetIntegration("GR12",700) //for griffin clover 12

If I were you I would run this session:
grsisort <my_calfile.cal>
TChannel::SetIntegration("GR",700)//Change to your chosen integration
TChannelWriteCalFile("<my_new_calfile.cal>")
please keep in mind that you likely already have default integration (whatever fKValue is set to) and this will overwrite this. You will need to account for that by multiplying them together.

That's some neat functionality, has rewritten the cal file nicely. However it still seems to be being ignored.
I've event tried calling TChannel::SetIntegration at the top of my integration script but it doesn't seem to matter if I put 1, 125 or 15000 the output is the same.

Hmm ok. There might be some logic mistakes in the TChannel. Give me a minute. I think I finally got the analysis tree proof working... > 300 MB/s sorting...

If you want to go hunting while I'm figuring this out it should be in TChannel::CalibrateENG There will be some if statements that are based on KValue etc. Your kValue will be something non-zero. Not sure off the top of my head when integration kicks in.

So the fKValue is in TGRSIDetector hit, not TChannel, Set in TFragmentMap::Add
TGRSIDetectorHit::GetEnergy only looks at the TChannel integration settings if TGRSIDetectorHit::fKValue is zero.
However contrary to the comment,
    return SetEnergy(chan->CalibrateENG(Charge(),0));  // this will use the integration value
                                            // in the TChannel if it exists.

You first need to TChannel::SetUseCalFileIntegration() as the default for fUseCalFileInt is false;
Is this the desired behaviour? I would have said that if the cal file gives an integration, it wants to be used.

I agree with you. We can probably do away with the SetCalFileIntegration flag for now.

Did we reach a conclusion on this? Currently it is complicated to do (what should be) simple checks using the command-line interface because the integration isn't being treated properly in GetEnergy().

I think this has been fixed by now. At least no-one complained about it for more than a year.


Create a Cast[T] Codec wrapper function

If it works, this would allow removal of many of the factory functions. Along with #67, Instead of lexy.Int16[MyIntType](), you would have lexy.Cast[MyIntType](lexy.Int16).
Internally, Cast[T]() could create a new Codec if the argument is a known Codec type.

This isn't possible, constraints of the form [T ~U, U bool | uint8 | ...] will not compile. The underlying type signifier ~ cannot be applied to a type parameter. It can be applied to a pointer to, slice of, ... a type parameter, but that's a different usage.

However, it might be possible to add a CastTo method to some Codecs. Reopening, worth experimenting with.

Nope, won't work, which I would have realized if I'd just stopped to think about it for a second. Any generic type in an interface method signature must be present in the interface definition.
However, NilsLast() will work. Just need an additional interface.


Language QA

As of Staging release 1.2.0 / Server 1.3.6 there are some strings that need to be QA'd in all non-English languages.
Acceptance Criteria:
Onboarding:

[x] Arrival Advisor is your go-to guide to getting settled in your new community.
[x] Answer a few optional questions to get tailored recommendations for your needs.
[x] Find service providers near you that can help you through your settlement journey.
[x] Bookmark the topics that you find helpful for future use.
[x] Start

About page:

[x] The Arrival Advisor app is created by PeaceGeeks in partnership with settlement organizations in Metro Vancouver, as well as immigrants and refugees like you. This project is possible thanks to funding by the Province of British Columbia and Google.org Canada in partnership with LEAP | Pecaut Centre for Social Impact. Special thanks also goes to individual donors who make this project possible. Arrival Advisor is an open-source project. You can view it on <0>GitHub</0> and <1>contact us</1> to learn how you can support this project.

This is waiting on Arabic and Tagalog QA:

[x] Arrival Advisor is a free, multilingual, mobile app that helps immigrants and refugees in British Columbia, Canada find information and services to navigate their settlement journey. Arrival Advisor is offered in English, Arabic, Chinese Simplified, Chinese Traditional, French (Canadian), Korean, Punjabi, and Tagalog.

done for all languages and updated in weblate


No hint path defined for [xxx]

Laravel Version: 6.x
PHP Version: 7.2

Description:
i have this error when i try to send a mail to confirm register .
No hint path defined for [components/errors]. (View: /home/fotsing/Bureau/eschool-gabon/vendor/laravel/framework/src/Illuminate/Foundation/Exceptions/views/404.blade.php)
Steps To Reproduce:
i use this code in my RegisterController
public function register(Request $request)
    {
        $this->validator($request->all())->validate();

        event(new Registered($user = $this->create($request->all())));

        //$user = User::all()->last();
        //dd($user);

        $user->notify(new RegisteredUser());

        //$this->guard()->login($user);

        return $this->registered($request, $user)
                        ?: redirect('/login')->with('success','Votre compte a bien ete creer ,vous devez confirmez avec l\' mail que vous allez recevoir');
    }

and a code of notification is:
public function toMail($notifiable)
   {

       return (new MailMessage)

                   ->success()

                   ->subject('Incription sur mon site xxxxx')

                   ->line('votre compte a bien ete cree ,merci de cliquer sur le lien suivant pour confirmer')

                   ->action('Confirmez mon compte', url("/confirm/{$notifiable->id}/" . urlencode("{$notifiable->confirmation_token}"))))

                   ->line('Merci pour votre confiance!!!');
   }

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


callable reported when more specific Closure is used

Using narrow_param: true, method looks like this:
public function map(Closure $mapper): array

And get wrongly reported with wider suggestion:
Parameters should have "callable" types as the only types passed to this method

@janedbal could you give more precise code when it shown the error? thank you.

The map method is called only with Closures. But it gets reported to be changed to callable, which is more wider type than used.
$object->map(function () {});

It seems between Closure and callable make flip flop in case of this check removed:
https://github.com/rectorphp/type-perfect/blob/df83404dda3364cae1b4f5251420f95ef20fa493/src/Printer/CollectorMetadataPrinter.php#L177-L179
then, create a method with callable method:
final class SkipCallable
{
    public function map(callable $mapper): array
    {
        return [];
    }

    public function run()
    {
        $this->map(function () {});
    }
}

then become error:
+'09: Parameters should have "Closure" types as the only types passed to this method


Backport acceptance changes

Signed-off-by: Scott Hain<EMAIL_ADDRESS>Description
https://github.com/chef/chef/pull/6673
Issues Resolved
[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussion that's relevant]
Check List

[ ] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

@tas50 @lamont-granquist


Fix for Custom icons don't work

Story details: https://app.clubhouse.io/cartoteam/story/143030

Adding @amiedes in case you see any conflict in tests


Add option to replace route when 'beaming'

It would be nice to have an option to replace the current item in 'beam history' when beaming to a new location. Something like a replaceCurrent option on beamTo.
Example:
Default location / route is splash screen
SplashScreen (finishes loading) -> HomeScreen
If back is pressed then it would currently bounce back to the SplashScreen.

Resolved by #128


Update cdng-upgrade-faq.md

Added delegate section for CD upgrading to Next gen.
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-483


browse not working in video mute settings

Microsoft PowerToys version
0.67.1
Installation method
WinGet
Running as admin
Yes
Area(s) with issue?
Video Conference Mute
Steps to reproduce
go to the video conference mute setting -> video settings -> display image when video is mute -> browse (not working)
✔ Expected Behavior
explorer window to select an image
❌ Actual Behavior
No response
Other Software
No response

This should be resolved now with a newer build


Retire mambaforge - addendum to 3774

Description

A few more refrences to Mambaforge/mambaforge that I missed in #3774 - was in a hurry get that over the fence so we get our tests back, @bettina-gier pointed to me we have a few more changes to make :beer:
Note the RTD configuration is currently unable to get the Miniforge3 variant, see https://github.com/ESMValGroup/ESMValCore/issues/2557

Before you get started

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[x] 🛠 This pull request has a descriptive title
[x] 🛠 All checks below this pull request were successful

yay! Many thanks @bettina-gier :beer: