Fix for https://github.com/Azure/azure-rest-api-specs/issues/1697#iss…

…uecomment-367167726
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-node

Automation for azure-libraries-for-java

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

@ravbhatnagar breaking change, however specifying any of these parameters (which were not meant for customer use) cause the service to fail (@naveedaz correct me if I'm wrong), rendering some SDKs unusable. See this issue for more info.

@ravbhatnagar Can we get this approved please. Customers are asking for this fix.


Feature/support infinity in json for no cycle

fixes #1482

Coverage increased (+0.9%) to 96.188% when pulling a233e1aa19921d4f282ff6ad89cc13ec00a1c69a on AndyOGo:feature/support-Infinity-in-JSON-for-no-cycle into 5e143b23f3b89096e1b6a268af723f60c1ecb22f on benmosher:master.

Coverage increased (+0.9%) to 96.188% when pulling a233e1aa19921d4f282ff6ad89cc13ec00a1c69a on AndyOGo:feature/support-Infinity-in-JSON-for-no-cycle into 5e143b23f3b89096e1b6a268af723f60c1ecb22f on benmosher:master.

Coverage increased (+0.9%) to 96.188% when pulling a233e1aa19921d4f282ff6ad89cc13ec00a1c69a on AndyOGo:feature/support-Infinity-in-JSON-for-no-cycle into 5e143b23f3b89096e1b6a268af723f60c1ecb22f on benmosher:master.

Coverage increased (+0.9%) to 96.188% when pulling a233e1aa19921d4f282ff6ad89cc13ec00a1c69a on AndyOGo:feature/support-Infinity-in-JSON-for-no-cycle into 5e143b23f3b89096e1b6a268af723f60c1ecb22f on benmosher:master.


Update traffic-coverage.md

Added 3new countries: Brunei(BRN), Nigeria(NGA), Uruguay(URY) and for each country flow and incidents.

@outinyman : Thanks for your contribution! The author, @dsk-2015, has been notified to review your proposed change.

Close/reopen for fresh validation post-migration.

@jinzh-azureiot please take a look at this. Is this coming from within the engineering team? If so, we should update the azure-docs-pr repo instead.

@dsk-2015 It is coming from our PM team.  However, this PR is already old, because we updated the page last week and this request was created by me over month ago. To sum up, can you remove or delete this pr?:)

Thank you for fixing this from docs-pr and closing this issue!


Current user

Returns user information such as first name, last name, and email address

Coverage increased (+0.02%) to 96.719% when pulling bc87d0cf51c36cd3f36f5c49be61e3b0ed0cbe02 on mbierma:current_user into c2648d1252523876a46d6c3bb69a59bbbaf4e345 on sidecars:master.

Coverage increased (+0.02%) to 96.719% when pulling bc87d0cf51c36cd3f36f5c49be61e3b0ed0cbe02 on mbierma:current_user into c2648d1252523876a46d6c3bb69a59bbbaf4e345 on sidecars:master.

Coverage increased (+0.02%) to 96.719% when pulling bc87d0cf51c36cd3f36f5c49be61e3b0ed0cbe02 on mbierma:current_user into c2648d1252523876a46d6c3bb69a59bbbaf4e345 on sidecars:master.

This also includes the feature added in PR #33. I should have split these up

Coverage increased (+0.02%) to 96.719% when pulling bc87d0cf51c36cd3f36f5c49be61e3b0ed0cbe02 on mbierma:current_user into c2648d1252523876a46d6c3bb69a59bbbaf4e345 on sidecars:master.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

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

New


Elasticsearch version update to 6.2.2

Upgrade to the new version of elasticsearch.
Note that JDK9 and JAVA_HOME environment variable is needed to build this project.

@babayotakun could you email me or PM me on discuss.elastic.co?


User profile pic is scaled, not cropped

On the main bar of ldjam.com the profile pic is scaled to be square, while it's fine in the actual profile and posts.
Top bar:

Profile and post:

User Environment:

Arch Linux (kernel 4.10.13)
KDE 5.33.0 / Plasma 5.9.5
Chromium 58.0.3029.110 (64-bit)

Ah yes, I never did add constrains to the picture in the corner, 'cause all my test avatars were square.


Transition Circom tests to WASM

Currently we test the Circom circuits using circom-helper which generates the proofs in C++.
circom-helper doesn't seem to work with Mac (I tried hacking this to work a whole lot to no avail). While it's certainly ultimately possible, it's not really worth the effort.
Instead, we should use the wasm provers, which works cross-platform, and is ultimately what we will use in production. Also an important part of designing/testing zk circuits is ascertaining their performance, and circom-helper goes a lot faster than WASM, hence the need to run them with WASM anyway.

i would like to try this one

Excellent, go for it :)

While this ticket is in progress, if anyone needs a way to make circom-helper working on mac m1, I downgraded node to v14.19.1 using nvm and downgraded python to python3.10 and it worked!
But this is indeed not viable at all lol

Oh wow, good to know!
If it turns out we require faster than wasm testing, there are two other
projects that might work: circom-tester and circomkit (no link because I'm
on mobile & it's a hassle, might edit later)
On Mon, Jul 10, 2023, 19:17 Matteo @.***> wrote:

While this ticket is in progress, if anyone needs a way to make
circom-helper working on mac m1, I downgraded node to v14.19.1 using nvm
and downgraded python to python3.10 and it worked!
But this is indeed not viable at all lol
—
Reply to this email directly, view it on GitHub
https://github.com/norswap/0xFable/issues/52#issuecomment-1628652830,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABRLP6U46KI633ER2EAUY3XPPJDTANCNFSM6AAAAAAZGPRKJM
.
You are receiving this because you authored the thread.Message ID:
@.***>

Any update on this?

@postmeback Haven't heard from @GrapeBaBa on this yet. Are you interested in helping out, or reusing some kind of testing setup? Let us know!

Thanks for mentioning me, I am interested in helping out. I want to upskill myself in solidity, hopefully this project can help me.
@norswap .

@postmeback This project does not really involve Solidity at all however 😅
If you're interested, go for it though, @GrapeBaBa told me he's busy with some stuff at the moment.

I’d like to work on this.

May I try my hand at this?


Add performance tests for Swift compilation

We need to have a baseline for how long it takes to compile swift projects with Gradle so we can understand how adding caching affects our build performance.

PR: https://github.com/gradle/gradle/pull/4270
Adds perf tests for build cache, clean assemble, assemble (up-to-date) builds.  Need to add one more test for incremental compile


fix compiler warnings, add documentation

Signed-off-by: Stamer, Daniel<EMAIL_ADDRESS>

For this old project, thank you.


Beckybanbury

This merge should complete data entry from DeAngelis 1981; all data from the digitized and raw files has now been added.

Thanks! It looks like there are some conflicts that need to be resolved. Could you please see if you can sort that out?


Add error handling for site improve code

Problem Description:
The window._sz functionality is causing errors in Firefox.
Expected Behavior/Solution:
Add error handling around the Site Improve code to prevent white screens.
Requirements:

[x] Check if window._sz is null or undefined

Needed for promotion:
If an item on the list is not needed, it should be crossed off but not removed.

[x] Wireframe/Mockup - Kam
[x] ~Committee discussions - Sarah~
[x] ~Feasibility/Team discussion - Sarah~
[x] ~Backend requirements - TBD~
[x] Frontend requirements - Kam
[x] ~Are new regression tests required for QA - Amy~
[x] ~Questions~

UAT/LUX Examples:
Steps to test:

Open a private window in Firefox
Navigate to the landing page
It should render and not show a white screen

Dependencies/Blocks:
N/A
Related Github Issues:
N/A
Related links:
N/A
Wireframe/Mockup:
Current:

Expected should just be the normal landing page.

In addition to the null check, please print an error message (console.error) to the console including the parameters passed to the function, which could help us determine in the longer run whether the problem can be fixed so the error would never occur.

Deploying to DEV.
cc: @jffcamp @azaroth42 @roamye @prowns

Closing as this was a hot fix and is in all environments.


Error: PlotPrincipalNodalStress

When I call PlotPrincipalNodalStress, the coutour overlapped with white element and some nodes seem like have no value. (please see picture below.)

rst file I used is down below.
(Cyclic symmetry condition applied)
file.zip
Thanks in advance!

Still working on this.  Please be aware that as this is a cyclic result you should be using:
result = pyansys.ResultReader('file.rst')

pyansys automatically determines if it's a cyclic result or not based on the number of sectors parameter in the result file.  I've added support for user defined coordinate systems as this example uses CSYS 12, but I'm having trouble with getting the cyclic results to match ANSYS for this model.  Your result will display a "bit" better in pyansys==0.34.0, but there's still a bug in the reader due to the way ANSYS 18.2+ saves cyclic results

@dangit6413,
pyansys>=0.34.7 now plots this result property.  I've been able to get the static and modal displacements to agree with ANSYS outputs results for cyclic results, as well as static stresses.  Modal stresses are still giving me trouble.
The latest version uses open_result instead of ResultReader to clear up the confusion between the Result and CyclicResult objects.  It's also a function, so it should be underscore by PEP8 standards.  For example:
result = pyansys.open_result('file.rst')

Let me know if you still encounter issues.

Thanks, I really appreciate your help!
But it looks like problem is not solved.

If full_rotor=False, it yield error like below;
"plot_nodal_stress() takes from 3 to 10 positional arguments but 11 were given"

Even if the full_rotor=True, the plotted value is wrong.
I expect same repetitive result on every sector, but it's not.(see below picture)
And the other model shows white empty result same as before

I feel sorry for keep bothering you and thanks in advance!

Thanks for identifying that bug in (1).  I'll have that fixed on the next release.
For (2), the full solution will not be identical for each sector.  Depending on the loading and the stress component being plotted, the stress will indeed vary throughout the part. Here's a screenshot from ANSYS:

Can you please remind me which issue the other result file was?

I got it. I was misunderstood. Thanks!
For the issue(1), Here is my test rst file which shows only white element not plotting scalar values.
import pyansys
result = pyansys.open_result('file_test.rst')
result.plot_nodal_solution(0, full_rotor=True)
file_test.zip
Thank you!

As of pyansys==0.36.3, your example works now:
import pyansys

rst = pyansys.read_binary('./file_test.rst')
rst.plot_nodal_solution(0)
# rst.plot_nodal_stress(0, 'Sx')
rst.plot_principal_nodal_stress(0, 'seqv')


Automatically update homebrew every 24 hours by default

Provide a detailed description of the proposed feature
Homebrew should adopt https://github.com/DomT4/homebrew-autoupdate as the default behavior, instead of making users wait for brew update before brew install commands.
This behavior can be implemented manually by running the following commands:
brew tap domt4/autoupdate
brew autoupdate --start
echo 'export HOMEBREW_NO_AUTO_UPDATE="1"' >> ~/.zshrc
# (or ~/.bashrc)

Some care should be taken to avoid large spikes in brew update requests at exactly the same time (e.g. midnight.) The auto-update time should be randomized so that brew update commands are spread out.
Otherwise, it should be straightforward to integrate the code from DomT4/homebrew-autoupdate and run brew autoupdate --start during the Homebrew installation.
What is the motivation for the feature?
brew update can be quite slow, and it can be frustrating when homebrew decides to fetch updates before installing a new package. It would be great if the Homebrew formula index could be updated in the background, so the latest formulae are always ready to use whenever we run brew install .... (However, I wouldn't recommend actually upgrading installed formulae in the background.)
How will the feature be relevant to at least 90% of Homebrew users?
100% of Homebrew users would benefit from this becoming the default behavior.
What alternatives to the feature have been considered?
It's currently possible to opt-in to automatic updates by using this third-party script: DomT4/homebrew-autoupdate
brew tap domt4/autoupdate
brew autoupdate --start
echo 'export HOMEBREW_NO_AUTO_UPDATE="1"' >> ~/.zshrc
# (or ~/.bashrc)

cron does the same job without installing extra stuff, and is cross-platform to boot (I also run Homebrew on my Linux boxes). My crontab contains, among other things:
3 0 * * * { /usr/local/bin/brew update ; /usr/local/bin/brew bundle dump --global --force ; } > /tmp/brew_maint.log 2>&1

which both updates Homebrew and dumps the current list of installed formulae, every night at 12:03am.

We're not going to make this default behaviour but, if @DomT4 was game, we could consider making this an official tap and point the documentation to it.

Me and @MikeMcQuaid held some initial discussions on making my autoupdate tap part of the Homebrew organisation last month. That's likely to happen in the near future, and likely would have already if I hadn't been briefly derailed by some very minor surgery & the aftercare around that. Remains very much on my radar and should see a resolution soon 🤞🏻.

if I hadn't been briefly derailed by some very minor surgery & the aftercare around that

Oof, get well soon @DomT4 ❤


ExTermbox in escript

Hi, I'm trying to make a command-line app with Ratatouille and ExTermbox and having a lot of fun of it.
So far everything works fine in iex, and mix run, but in an escript.
When running the escript binary, I got error messages like this:
13:58:42.454 [warn]  The on_load function for module Elixir.ExTermbox.Bindings returned:
{:error, :bad_name}

** (EXIT from #PID<0.98.0>) shutdown: failed to start child: Ratatouille.Window
    ** (EXIT) an exception was raised:
        ** (UndefinedFunctionError) function ExTermbox.Bindings.init/0 is undefined (module ExTermbox.Bindings is not available)
            (ex_termbox 1.0.2) ExTermbox.Bindings.init()
            (ratatouille 0.5.1) lib/ratatouille/window.ex:104: Ratatouille.Window.init/1
            (stdlib 3.11) gen_server.erl:405: :gen_server.init_it/2
            (stdlib 3.11) gen_server.erl:373: :gen_server.init_it/6
            (stdlib 3.11) proc_lib.erl:249: :proc_lib.init_p_do_apply/3

FYI, here is a minimum repository to reproduce this issue.
Why does not it work?
In an escript binary, ExTermbox is unable to load the NIF file from its priv dir.
This is because priv dir of dependencies are stripped during the escript compilation.
Should we fix it?
I suggest so.
While this is certainly not ExTerbox's fault, and there are other libraries with the same problem too, I think it's a special case for ExTermbox, because ExTermbox IMO is supposed to be used in command-line applications, and CLI applications on Elixir are most likely packed in escripts.
My attempt to fix it
So I took an attempt to solve this problem by changing the way of loading NIF. The idea is to embed NIF binaries into the Elixir module, and generate a temporary file for loading.
I forked and made a proof of concept commit here:
https://github.com/qhwa/ex_termbox/commit/971cfe6e726cea81338cb5746dc1f12c0e2dd0fe
and it works. In case you want to see the result, you can check out the embed_nif branch of the example repo to give it a try.
Help needed
Although I got it working as I wished, I need some discussion and your opinions. I wonder if there is a better approach to fix this problem.
Please drop your ideas. :)

I would also love to see this! Currently trying to add a Ratatouille based dashboard in an existing escript CLI tool.


Crafting

For being based on a program named "Craft", the functionality of crafting itself is missing

The crafting logic itself should be straightforward, but implementation of the UI opens a new can of worms.
There are not yet any GUI windows, should there be? One alternative is to use sub-block interaction within the world on the blocks themselves, such as in the mod Vanilla Immersion:

but would this in-world UI be enough, completely fulfilling the needs of any GUI window? GUI windows are familiar and intuitive, and powerful: various combinations of mouse clicks and modifier keystrokes to perform specific actions, splitting stacks, decrementing stacks, painting stacks, merging stacks. Could an in-world UI do all this, as conveniently? Would the angled perspective (on top of the workbench) get annoying versus a flat 2d window? Maybe move the camera right up against the side of a block, no angle or rotation, so the "window" is on the block itself?


Define extension story for tasks

Out of the box VS Code ships with a set of predefined task templates. Since VS Code 1.9 we also support defining more than one command in the tasks.json. Hence the value of a template is not huge anymore since most users will use different commands in a tasks.json.
However extension like C# or C++ want to contribute to the tasks.json to ensure that a build tasks for example executes a C++ specific build script.
We need to define how extensions best participate in running builds and tests.

The current plan to address this is to provide API so that extensions can provide tasks.


about showinfo

AlertView of showinfo has only one textFiled, can you support a custom to add more.  we want to use two    textFiled to edit name and pwd...

let alert = SCLAlertView()
let textField = alert.addTextField(title:"Enter your name")
textField.secureTextEntry = true

you should be able to set any property to UITextField

For Swift 4 is:
textField.isSecureTextEntry = true


mWeb - Long press on hyperlink not always displayed Copy URL to clipboard title

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

Issue was found when executing PR: https://github.com/Expensify/App/pull/8815
Action Performed:

Go to https://staging.new.expensify.com/ and log in with any account
Navigate to chat conversation
Send any hyperlinked text
Long press in hyperlinked text

Expected Result:
Long press on hyperlink displayed Copy URL to clipboard title
Actual Result:
Long press on hyperlink not always displayed Copy URL to clipboard title
Workaround:
Unknown
Platform:

Where is this issue occurring?

Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No 9new feature)
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/167973090-303fb1cd-a47c-464c-a95f-6bf1c596eb85.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause
Slack conversation:
View all open jobs on GitHub

It can be External.

CC: @eVoloshchak as it seems like this PR didn't quite work.

This issue isn't connected to my PR, it was present before
It's still open https://github.com/Expensify/App/issues/8311
Before submitting my PR i raised a concern here, but was told i could submit a PR since that's a separate issue

Thanks for looking into it. It is a different issue than the one we are trying to solve here. So you don't have to fix that in this PR.

I also stated that in the PR description, in the Screenshots/mWeb section
I also researched https://github.com/Expensify/App/pull/8815, it is present on mWeb and Native platforms, but it's somewhat different on these platforms: on mWeb it shows wrong context menu for all links, on native it shows wrong menu only if message contains text and a link

Okay got it, thanks for that added context. If we've got #8311, do we need this issue open as well?

Okay got it, thanks for that added context. If we've got #8311, do we need this issue open as well?

Not sure if you you were asking me, but imo this one is a duplicate

Cool, cool. I'm going to close this out. Thanks! @danieldoglas, let me know if you disagree.


Double nested form, 2nd level links_to_add do not create 3rd level links_to_add

Hi Nathan,
I've been trying for several days to make double nested forms work with cocoon, but I cannot seem to tack my issue.
I have three hierarchical models in my view, on the first load everything looks fine, all the add and remove links are there and working, but when I add another 2nd level field, it won't show the link_to_add_association for its 3rd level fields.
It seems to me as if the link_to_add should somehow build the next level objects, but it doesn't. At the first load, the build methods are invoked per the new action in the controller, but when clicking the cocoon links, this method is not invoked again and I'm not sure where I should go for building the objects.
I have no controller for the 2nd level objects, but I checked your demo repository and you also have just one controller there.
Sorry to bother you with this, but do you see what I might be doing wrong?
Using Rails 4, strong params are all fine, with ids and _destroys and stuff.
The code>>
_form.html.erb
<%= simple_form_for @ucetni_report do |f| %>
  <%= f.input :datum %>
  <%= f.input :uzavreny %>

Účetní balíček

  <%= f.simple_fields_for :ucetni_baliceks do |ucetni_balicek| %>
    <%= render 'ucetni_balicek_fields', f: ucetni_balicek %>

      <%= link_to_add_association 'Pridat balicek', f, :ucetni_baliceks %>

  <% end %>

  <%= f.button :submit %>
<% end %>

_ucetni_balicek_fields.html.erb (2nd level partial)

  <%= f.input :nadpis %>
  <%= link_to_remove_association "Odstranit balicek", f %>

  <%= f.simple_fields_for :ucetni_polozkas do |ucetni_polozka| %>
  	<%= render 'ucetni_polozka_fields', f: ucetni_polozka %>

	    <%= link_to_add_association 'Pridat polozku', f, :ucetni_polozkas %>

  <% end %>

_ucetni_polozka_fields.html.erb (3rd level partial)

  <%= f.input :nazev %>

  <%= link_to_remove_association "Odstranit polozku", f %>

The controller new method
  def new
    @ucetni_report = UcetniReport.new
    @ucetni_report.ucetni_baliceks.build
    @ucetni_report.ucetni_baliceks.each do |b|
      b.ucetni_polozkas.build
    end
  end

And the models
class UcetniReport < ActiveRecord::Base
  has_many :ucetni_baliceks, :dependent => :destroy
  accepts_nested_attributes_for :ucetni_baliceks
end

class UcetniBalicek < ActiveRecord::Base
  belongs_to :ucetni_report
  has_many :ucetni_polozkas, :dependent => :destroy
  accepts_nested_attributes_for :ucetni_polozkas
end

class UcetniPolozka < ActiveRecord::Base
  belongs_to :ucetni_balicek
end

Thanks and take care!

Mmmm. You only show a link_to_add_association if there are already nested items???
You write
<%= f.simple_fields_for :ucetni_baliceks do |ucetni_balicek| %>
    <%= render 'ucetni_balicek_fields', f: ucetni_balicek %>

      <%= link_to_add_association 'Pridat balicek', f, :ucetni_baliceks %>

  <% end %>

but it should be
  <%= f.simple_fields_for :ucetni_baliceks do |ucetni_balicek| %>
    <%= render 'ucetni_balicek_fields', f: ucetni_balicek %>
  <% end %>

    <%= link_to_add_association 'Pridat balicek', f, :ucetni_baliceks %>

I guess you are not familiar with haml. Check the erb examples (but I prefer haml as it is much cleaner and shorter and you are less likely to make html errors --and it is really easy to learn imho).

Nathan, you are absolutely awesome. I've read through the cocoon issues forum & cocoon on SO and it is unbelievable that you put so much care in this piece of code, doing your best to help everyone who just opens his mouth...thank you so much for your advice and for your passion!
I'm definitely going to switch to haml, it seems to be so clear .)
Oh and of course you've solved my issue.

Thank you for those kind words :smile: :heart: :heart: :+1:


BlazorWebView

Fixes #12

@Eilon Can you take a look at these BlazorWebView docs?

@Eilon
Closing here, and created a new PR that merges into the release branch.


Possibility for in-game configuration menu on Forge/NeoForge versions?

Hey, I'm wondering there are any plans to implement an in-game configuration menu for the Forge/NeoForge versions of Wild Tool Access. There's a couple of options that can be used:

YetAnotherConfigLib, works on Forge up until 1.20.1, and all NeoForge versions (and even Fabric!) that this mod supports
Configured, works on all versions of NeoForge, Forge, and Fabric that this mod supports

Thank you for the extensive question. Makes me sorry to say, that that is just a result of me not bothering to find out how I make the Forge config button open the config screen, I already made for Mod Menu using Cloth Config, which has a Forge version. 😅
Going to look into it. 🙂


Cannot view pack when Private

This is a shame, because it means users cannot see their weight breakdown, etc unless they make it Public.

I'll be adding the weight breakdown info to private packs eventually.
The primary motivation is to encourage users to make their packs public to benefit people who may be searching for information about similar trips.

I understand that this is to encourage users to make their pack public, and I have no problem with that. But I think this should be mentioned a bit more explicitly on the pack creation page (I never saw the hover information next to the public toggle). I just discovered Packstack today and searched for 30min after a pack overview page, I thought this feature was missing until I discovered this issue.
Some suggestions of how this can be made more clear for new users:

Setting the packs public by default (on the creation page), and notify the user when he/she sets it to private, saying that they will only have access to restricted features.
Disabling (graying out) the view button instead of completely hiding it, and by hovering over it inform that this feature is not available for private packs.


Incorrect documentation for check_point_order_equals_group_order

In the documentation for check_point_order_equals_group_order,
https://github.com/ZenGo-X/curv/blob/44537a746292be65961e9c38191319c17a3d2977/src/elliptic/curves/traits.rs#L185
should presumably read
/// P ≠ 0 ∧ qP = 0

The implementation appears to be correct, so just a simple doc fix should suffice.

In addition,
https://github.com/ZenGo-X/curv/blob/44537a746292be65961e9c38191319c17a3d2977/src/elliptic/curves/bls12_381/g2.rs#L249
should say G2 instead of G1

Thanks @tjade273 , feel free to open a PR :)


blurry results

Hi!
Thanks for such an impressive work!
I've tested it on some videos, and while it works well on most of them, some turn out blurry edges. Do you have any insights on why this might be happening?

input
output

Hello, thank you for your attention.
We have also noticed this issue. We have not completely resolved it yet.
We partially attribute it to using the cv.inpainting function with added noise for initialization. However, considering that this initialization method generates better results than directly using Gaussian noise, we have adopted the former method.


Immediately fetch initial chat data in adapter (+fix customDataModel flakey UI test)

What
When creating chatAdapter, immediately call chatAdapter->fetchInitialData()
Why
Otherwise you can create a chat composite with missing information

Chat composite will load with no topic header (then suddenly populate it)
Chat composite will load with no previous messages (then suddenly populate them)
We have had flakey UI tests from this for over a month now and tripped us up - likely will trip up others.

Caveats

Chat adapter will now takes longer to construct as it has to wait for fetchInitialData to complete

Fixes
Flakey ui test where previous messages hadn't loaded before screenshot was taken:

Completing this, cc @PorterNan for any issues this may cause


Files (XML/XSD) Drag and Drop does not work!

This issue seems to occur on windows system with UAC enabled. Hence likely to do with windows security framework. As a workaround UAC needs to be disabled in order to use drag and drop.


Wide table runs into caption contingent on label prefix when using tinytable

If I create a table using tinytable that is very wide, the caption is at risk of being struck through:
---
title: "Test Document"
author:
  - name: Test Author
    affiliations:
      city: Los Angeles
      region: CA
lang: en
format:
  apaquarto-pdf:
    documentmode: man
keep-tex: true
---

```{r}
#| label: "tbl-descriptive-statistics"
#| echo: false
#| tbl-cap: "Summary Statistics"

# Install dev version of `tinytable`
# remotes::install_github("vincentarelbundock/tinytable")

library(tinytable)
tt(head(iris))
```

Curiously, removing the tbl- prefix from the label resolves the issue:

Initially raised at vincentarelbundock/tinytable#327, where the following was also noted:

And for reference, here's the LaTeX that this produces. Note that the only code that tinytable generates is the \begin{tblr} block, and that the issue stems for a conflict between a vanilla tblr environment and the table and caption commands inserted by Quarto and/or apaquarto.
\begin{table}

{\caption{{Summary Statistics}{\label{tbl-descriptive-statistics}}}
\vspace{-20pt}}

\centering
\begin{tblr}[         %% tabularray outer open
]                     %% tabularray outer close
{                     %% tabularray inner open
colspec={Q[]Q[]Q[]Q[]Q[]},
}                     %% tabularray inner close
\toprule
Sepal.Length & Sepal.Width & Petal.Length & Petal.Width & Species \\ \midrule %% > TinyTableHeader
5.1 & 3.5 & 1.4 & 0.2 & setosa \\
4.9 & 3.0 & 1.4 & 0.2 & setosa \\
4.7 & 3.2 & 1.3 & 0.2 & setosa \\
4.6 & 3.1 & 1.5 & 0.2 & setosa \\
5.0 & 3.6 & 1.4 & 0.2 & setosa \\
5.4 & 3.9 & 1.7 & 0.4 & setosa \\
\bottomrule
\end{tblr}

\end{table}

This was caused by a hacky solution of mine to a problem with longtable (the default environment for quarto tables). The longtable environment does not play nicely with the apa7 package that apaquarto depends on to make .pdf via LaTeX. I am not sure what to do about it, but I have inserted a possible fix. By default, apaquarto removes 20pt  of extra space (i.e., \vspace{-20pt}) between captions and tables when in manuscript mode. Using tinytable, longtable is not used, and space should not be subtracted. If I can find a way to take care of the problem automatically, I will do it. In the meantime, you can now add after-caption-space: 0pt as a code-chunk option. For example:
---
title: "Test Document"
author:
  - name: Test Author
    affiliations:
      city: Los Angeles
      region: CA
lang: en
format:
  apaquarto-pdf:
    documentmode: man
keep-tex: true
---

```{r}
#| label: "tbl-descriptive-statistics"
#| echo: false
#| tbl-cap: "Summary Statistics"
#| after-caption-space: 0pt

library(tinytable)
tt(head(iris))
```


Data Type Difference Between Document Snapshot and QuerySnapshot

Since, the latest update Cloud Firestore, The Query and Dcoument Snapshots switched to <Map<String, dynamic>> data type from Object, But Firestore Cache Still using Object as Datatype for Query Snapshot, that causes conflicts between Document Snapshot and Query Snapshot.

Here's Image of Cloud Firestore's new Query Snapshot Data Type

Thanks for reporting this issue, I have just released a new version to fix it 🙂


Handle boolean case

@jhermsmeier @themasch
Convert boolean values to 0/1.
I wasn't sure if this was "proper" behavior and something that should be included since booleans are not really supported. It helped in my case since I didn't want to have to convert all boolean values before sending the bencode message. I think at the very least there should be some error handling for this case.

Thanks for making this happen so quickly!

I think at the very least there should be some error handling for this case.

Was my initial thought as well, but then I think convenience should be the more important factor in this case – as people fiddling with bencode probably know what they're doing (and I wouldn't want to have to convert every boolean that's in a nested object somewhere to a number either). But I think we should definitely mention it in the README or something. Thoughts, @themasch?

We could just add a section to the readme stating that the lib automatically encodes booleans as [ 0 ; 1] because the spec does not specify anything about booleans.
Should we add a way to configure (enable / disable) this behavior? enabled would be default, disabled would case an exception when encoding an integer?

@kaelar since you can't restore boolean value because spec doesn't has this type, why you want encode it?
I think that this package should follow up spec.

@kaelar thanks again! Published as 0.8.0 now.


Add initial jupyter cmdlets for PowerShell kernel

This adds the following cmdlets (brackets mean optional):
Get-HtmlContent [-HtmlString] <string>

# returns IHtmlContent (calls HTML() under the hood)

Show-JupyterContent [-InputObject] <object> [-MimeType <string>] [-PassThru]

# returns DisplayedValue if PassThru is specified, nothing if it's not specified
# (calls display() under the hood)

Use-JavaScript [-JavaScriptString] <string>

# returns nothing (calls Javascript() under the hood)

New-PlotlyTrace [-TraceType] <string> [-Name <string>]

# returns a XPlot.Plotly.Graph+Trace

New-PlotlyChart [-Trace] <Trace[]> [-Title <string>]

# returns a XPlot.Plotly.Chart

Here are some examples:
$a = Show-JupyterContent @(3,4,5)
Start-Sleep -Seconds 3
$a.Update(@(1,2,3))

Get-JupyterHtml '<b style="color:blue">Hello!</b>' | Show-JupyterContent

@("alert('1!');","alert('2!');","alert('3!');") | Invoke-JupyterJavaScript

And a gif of some of this all mixed in together:

Here is a plotting example:

I've also rewritten the HTML notebook in the C# folder for PowerShell.

@TylerLeonhardt Can you please update the PR description to reflect the cmdlet names that were finally used in this PR?


add code comments to clarify requirements for custom geocoders

looks like you refactored at some point and made this change irrelevant.


Depletes inotify limit on remote session

Issue Type: Bug

Behaviour
Expected vs. Actual
Since 2-3 weeks, my each remote session (while I use Python extension for year/s in that scenario) depletes the inotify limit pretty quickly.
Seems like Python extension tries to inotify-watch a Python lib.
Steps to reproduce:

Kill server processes on a target, close VS Code editors on client
Open new VS Code on client, connect to a repository with 2 files, one being python
Python extenstion launches
After a while "inotify limit" popup can be seen

inotify consumers:
   INOTIFY   INSTANCES
   WATCHES      PER
    COUNT     PROCESS   PID USER         COMMAND
------------------------------------------------------------
    8042         1      7427 mmatczak    ~/mmatczak/.vscode-server/bin/c3511e6c6
      41         1      8862 mmatczak    ~/mmatczak/.vscode-server/bin/c3511e6c6
       4         1      7123 mmatczak    ~/mmatczak/.vscode-server/bin/c3511e6c6

The process is:
node,7427,6680 ~/mmatczak/.vscode-server/bin/c3511e6c69bb39013c4a4b7b9566ec1ca73fc4d5/out/bootstrap-fork --type=fileWatcher

remoteagent.log shows it's a Python lib, that couldn't be watched because of a limit:
───────┬───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
       │ File: ~/.vscode-server/data/logs/20220603T154041/remoteagent.log
───────┼───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
   1   │ [2022-06-03 15:40:41.665] [remoteagent] [info] Extension host agent started.
   2   │ [2022-06-03 15:40:42.047] [remoteagent] [info] [<IP_ADDRESS>][69f3143f][ManagementConnection] New connection established.
   3   │ [2022-06-03 15:40:42.122] [remoteagent] [info] [<IP_ADDRESS>][b0d31593][ExtensionHostConnection] New connection established.
   4   │ [2022-06-03 15:40:43.664] [remoteagent] [info] [<IP_ADDRESS>][b0d31593][ExtensionHostConnection] <7647> Launched Extension Host Process.
   5   │ [2022-06-03 15:44:00.388] [remoteagent] [error] [File Watcher (parcel)] Inotify limit reached (ENOSPC) (path: /....../Python-3.8.0/lib/python3.8)
───────┴───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.8.0
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Unknown
Value of the python.languageServer setting: Pylance

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

Experiment 'pythonaacf' is active
Experiment 'pythonTensorboardExperiment' is active
Experiment 'PythonPyTorchProfiler' is active
Experiment 'pythonDeprecatePythonPath' is active
Experiment 'pythonInterpreterInfoPinned' is active
> conda info --json
> /XXX/Python-3.8.0/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py
> /usr/share/anaconda/bin/conda info --json
> /file:/XXX/Python-3.8.0/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py
[ERROR 2022-5-3 15:40:50.382]: [Error: Command failed: /file:/XXX/Python-3.8.0/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py
/bin/sh: /file:/XXX/Python-3.8.0/bin/python: No such file or directory

	at ChildProcess.exithandler (node:child_process:397:12)
	at ChildProcess.emit (node:events:390:28)
	at maybeClose (node:internal/child_process:1064:16)
	at Socket.<anonymous> (node:internal/child_process:450:11)
	at Socket.emit (node:events:390:28)
	at Pipe.<anonymous> (node:net:687:12)] {
  killed: false,
  code: 127,
  signal: null,
  cmd: '/file:/XXX/Python-3.8.0/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py'
}
Python interpreter path: /XXX/Python-3.8.0/bin/python
> /bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py
> /usr/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py
> /usr/local/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py
[ERROR 2022-5-3 15:40:51.220]: [Error: Command failed: /usr/local/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py
Traceback (most recent call last):
  File "~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py", line 26, in <module>
    runpy.run_path(module, run_name="__main__")
AttributeError: 'module' object has no attribute 'run_path'

	at ChildProcess.exithandler (node:child_process:397:12)
	at ChildProcess.emit (node:events:390:28)
	at maybeClose (node:internal/child_process:1064:16)
	at Socket.<anonymous> (node:internal/child_process:450:11)
	at Socket.emit (node:events:390:28)
	at Pipe.<anonymous> (node:net:687:12)] {
  killed: false,
  code: 1,
  signal: null,
  cmd: '/usr/local/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/get_output_via_markers.py ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/interpreterInfo.py'
}
Starting Pylance language server.
[ERROR 2022-5-3 15:40:51.994]: Active interpreter type is detected as Unknown {"id":"/XXX/Python-3.8.0/bin/python","sysPrefix":"/XXX/Python-3.8.0","envType":"Unknown","envName":"","envPath":"","path":"/XXX/Python-3.8.0/bin/python","architecture":3,"sysVersion":"3.8.0 (default, Jun 27 2020, 21:48:56) \n[GCC 4.8.5 20150623 (Red Hat 4.8.5-4)]","version":{"raw":"3.8.0","major":3,"minor":8,"patch":0,"build":[],"prerelease":["final","0"]},"displayName":"Python 3.8.0 64-bit","detailedDisplayName":"Python 3.8.0 64-bit"}
> /XXX/Python-3.8.0/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/linter.py -m pylint ./run.py
cwd: .
> /XXX/Python-3.8.0/bin/python ~/.vscode-server/extensions/ms-python.python-2022.6.3/pythonFiles/linter.py -m mypy --follow-imports=silent --ignore-missing-imports --show-column-numbers --no-pretty ./run.py
cwd: .
##########Linting Output - mypy##########

Success: no issues found in 1 source file

##########Linting Output - pylint##########

[
    {
        "type": "convention",
        "module": "run",
        "obj": "",
        "line": 7,
        "column": 0,
        "path": "run.py",
        "symbol": "trailing-newlines",
        "message": "Trailing newlines",
        "message-id": "C0305"
    },
    {
        "type": "convention",
        "module": "run",
        "obj": "",
        "line": 1,
        "column": 0,
        "path": "run.py",
        "symbol": "missing-module-docstring",
        "message": "Missing module docstring",
        "message-id": "C0114"
    }
]

User Settings

languageServer: "Pylance"

linting
• mypyEnabled: true

formatting
• autopep8Args: "<placeholder>"

Extension version: 2022.6.3
VS Code version: Code 1.67.2 (c3511e6c69bb39013c4a4b7b9566ec1ca73fc4d5, 2022-05-17T18:15:52.058Z)
OS version: Windows_NT x64 10.0.18363
Restricted Mode: No
Remote OS version: Linux x64 3.10.0-1127.10.1.el7.x86_64

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1185G7 @ 3.00GHz (8 x 1805)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.39GB (5.59GB free)

Process Argv
--crash-reporter-id 3c01249c-cb3d-4c5d-87ad-4909624b55b9

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: XXX

OS
Linux x64 3.10.0-1127.10.1.el7.x86_64

CPUs
Intel(R) Xeon(R) Gold 5218R CPU @ 2.10GHz (80 x 2899)

Memory (System)
503.41GB (3.76GB free)

VM
0%

Closing a duplicate of #19233.

@brettcannon, I am not sure if that's a duplicate.
Python extension still seems to start a watcher with this path for me:
/.../Python-3.8.0/lib/python3.8

I am getting the same error as in a first post. That directory has, for me, almost 100k files, so no surprise watcher limit is depleted.

@Jiayuan-Gu, do you still have an issue with newest version of Python extension, 2022.08.1?

@maciejmatczak may I ask what is in that directory to lead to it containing 100k files? It isn't site-packages so that doesn't look like a virtual environment. Looking at your Output Panel results we can't actually identify what type of Python installation it is.
And are you putting that directory on the search path somehow? I don't believe we have ever watched anything in someone's lib directory. Have you tried this with all extensions deactivated and just with the Python extension activated?

@brettcannon, I am guessing, that the type of the installation is closest to be a custom built Python from source. And it's kind of OS installation, but, well, on shared disk ;).
In lib I have libpython3.8.so, lib/python3.8/site-packages, etc.
Well, for sure we didn't have this issue in the past, and it does not appear with I turn off just a Python extension, but I will also check clean session with Python extension only.

In remoteagent.log I found additional trace log, so something did requested this path to be watched:
[2022-07-05 10:52:23.639] [remoteagent] [trace] [File Watcher (parcel)] Request to start watching: /.../Python/Python-3.8.0/lib/python3.8 (excludes: **/.git/objects/**,**/.git/subtree-cache/**,**/node_modules/*/**,**/.hg/store/**,**/venv/**,/u/mmatczak/.vscode-server/extensions/**, includes: <all>),/u/mmatczak/.local/lib/python3.8/site-packages (excludes: **/.git/objects/**,**/.git/subtree-cache/**,**/node_modules/*/**,**/.hg/store/**,**/venv/**,/u/mmatczak/.vscode-server/extensions/**, includes: <all>)
[2022-07-05 10:54:04.512] [remoteagent] [error] [File Watcher (parcel)] Inotify limit reached (ENOSPC) (path: /.../Python/Python-3.8.0/lib/python3.8)

I left Python, Pylance and remote's ssh and ssh-edit extension on. After launching an extremely small folder on a target in VS Code, leaving session a bit, I once again got:

Pylance is still active on remote, it's actually caused by Pylance watching the Lib folder: https://github.com/microsoft/pylance-release/issues/2914. Closing in favor of that.


Cannot add a recursive entry to PageTable without unsafe

I want to add a recursive entry to the 511th entry of PageTable. This struct has the set_addr method, but we cannot use this because PhysAddr assumes the 52..64 bits are 0, while the recursive address is 0xffff_ffff_ffff_f000. Although we can create a PhysAddr containing such an address with PhysAddr::new_unsafe, I'd like to avoid this as it is unsafe. Is there a way to create a recursive page table without using unsafes?

Sorry I was confused about the addressing. This was totally wrong. I don't set the address 0xffff_ffff_ffff_f000. I just had to set the address of PML4.


Phone Calls Dataset: Query returns with incomplete results

Description
Using grakn console loads huge data, but GC overhead.
Environment

OS (where Grakn server runs): { e.g. Mac OS 10, Windows 10, Ubuntu 16.4, etc. }
Ubuntu 16.4
Grakn version (and platform): { e.g. Grakn Core 1.4, or Grakn KGMS 1.4.3 on Google Cloud }
Grakn Core 1.5.7
Grakn client: { e.g. client-java, client-python, client-nodejs, or console }
console
Other environment details:
Nope

Reproducible Steps
Steps to create the smallest reproducible scenario:

Run query on  phone_calls dataset which given by you.
query is:

 match
$customer isa person, has first-name $cna;
$company isa company, has name "Telecom";
$contract (customer: $customer, provider: $company) isa contract;
$target isa person, has phone-number "+86<PHONE_NUMBER>", has first-name $tna;
$call (caller: $customer, callee: $target) isa call, has started-at $started-at;
$min-date == 2018-09-14T17:18:49; $started-at > $min-date;
get $customer, $company, $contract,$call, $target, $cna, $tna;

Expected Output
there are many calls between $customer and $target person, so it should return some data
Actual Output
$target person doesn't have first-name($tna) attribute in graph, no data return.
Additional information

@shanghai-Jerry
on this line: $target isa person, has phone-number "+86<PHONE_NUMBER>", has first-name $tna;,
you've added the , has first-name $tna to the statement, whereas there is no instance of type person with the given phone-number AND a first-name. The person who has phone-number of +86<PHONE_NUMBER> owns no instance of type first-name. This is why the query is correctly returning no answer.

@shanghai-Jerry
here is the original query:
match
  $customer isa person, has phone-number $phone-number;
  $company isa company, has name "Telecom";
  $contract (customer: $customer, provider: $company) isa contract;
  $target isa person, has phone-number "+86<PHONE_NUMBER>";
  $call (caller: $customer, callee: $target) isa call, has started-at $started-at;
  $min-date == 2018-09-14T17:18:49; $started-at > $min-date;
get $customer, $company, $phone-number, $contract, $call, $target;


Multi-language redirects

We've got a live site running 1.3.1 which has the re-direct /about-us/company: /about-us. Our new site has an upgraded version of Grav (I've tried 1.5.10 and also 1.6-rc4).
On the old site that re-direct would match for all languages i.e. only the slug part is important - as per the docs (https://learn.getgrav.org/15/content/routing#site-level-routes-and-redirects). On the upgraded site however, whilst the root https://example.com/about-us/company redirects it no longer matches the language versions i.e. https://example.com/en-gb/about-us/company and https://example.com/en-us/about-us/company
If I modify the redirect to use/en-(gb|us)/about-us/company: /about-us it does correctly redirect.
Not sure if the logic has changed in this area - but any help you could give would be very much appreciated!

Bumping this up: I can confirm it is still happening in 1.6.27.
I have a multi-language website (en and fr). When setting up site.yaml like this, redirects for requests to /en/foo/2020 won't work:
redirects:
  /foo/(\d+): /bar/$1

But they do with the following:
redirects:
  (/.*)?/foo/(\d+): /bar/$2

I expected the first to work, as the documentation mentions:

All redirect rules apply on the slug-path beginning after the language part (if you use multi-language pages)

Investigation
I noticed there was a big refactor in Grav 1.5 around multi-language redirects, perhaps explaining why OP had it working in prior versions. I tried to skim through it, but couldn't make out how the changes would impact matching source URL with language prefixes.
Instead, I had a look at the dispatch() code responsible for handling redirects. Here is the code for regex redirects:
https://github.com/getgrav/grav/blob/75cb4e392d1bebc62e56f2601fc93a72efb97212/system/src/Grav/Common/Page/Pages.php#L537-L556
As we can see, it tries to match redirect source $pattern (after sanitizing) against $source_url retrieved via $uri->uri(false). However, $source_url contains the language prefix, and thus cannot match $pattern as-is.
So I tried to use $uri->path() instead, as it does not contain the language prefix, and hooray! It does seem to work. Considering the $route parameter to the dispatch() function is actually already supposed to be $uri->path() (as per PagesServiceProvider), we can directly use $route for matching purposes instead of $source_url (which becomes an unused variable and can be cleaned up).
Thus, the question is: why are regex redirects and routes relying on $uri->uri(false) rather than the given dispatch() $route? Is there any specific reason? If not, then I am ready to submit a PR to fix this.

I agree that based on the documentation, your proposal seems to be the better choice.


WIP: Semi supervised

Summary:
Work in progress: Add support for semi-supervised learning for Pytorch-Lightning
Features:

Added SemiSupervisedIterator (
Added SSLModule

TODO

Test Pi model

Checklist:

[ ] Your code is documented (To validate this, add your module to tests/documentation_test.py).
[ ] Your code is tested with unit tests.
[ ] You moved your Issue to the PR state.

Coudl also move the ssl experiment files in their own folder. Ex experiments/ssl_experiments

Alright, I think we are ready for review @parmidaatg :)


Howto definine globally accessible BackgroundSocketProcessor?

Given that a server-side implementation is described, a common next step would be to implement some sort of ClientManager in order to later message connected clients.  In the case where SignalR is not a natural fit, it would be helpful to describe a way to instantiate a ClientManager in such a way as to be globally accessible.  As in, accessible outside of 'Configure'.  An answer more than "use services.AddSingleton would be helpful.".  I believe I've found a way using services.AddSingleton to share a 'ClientManager' throughout the webapp but the solution does not appear to be accessible within 'Configure'.

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

ID: a3b0cac1-adc7-434c-d867-6e28e39bdae7
Version Independent ID: 762efeed-010e-f422-03d0-22d6681b2851
Content: WebSockets support in ASP.NET Core
Content Source: aspnetcore/fundamentals/websockets.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

The answer can be found here:
https://stackoverflow.com/questions/39005861/asp-net-core-initialize-singleton-after-configuring-di

@karezza  Thanks for the info.


configuring taxonomy root

It's not really necessary right now, but in the future DILA will probably want their own taxonomy. So we should probably have a config instead of the hardcoded value at https://github.com/buda-base/lds-pdi/blob/master/src/main/java/io/bdrc/taxonomy/Taxonomy.java#L37

as well as https://github.com/buda-base/lds-pdi/blob/master/src/main/java/io/bdrc/taxonomy/TaxModel.java#L21

Fixed as of commit 4de6ae


removed limit on character count

Instead uses line break to determine length

Karan I think it will be better if you just create another file and add this solution then both the solutions will be present i.e. using line break and using function. After that you generate another pr as I am closing this one .

Sure


Add missing 'be' word

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

Your suggested changes
a) Would result in incorrect English
b) Are to the original source files which we've clearly stated that we won't accept changes to.


Flatpak support

I've been working on a Flatpak package for easy distribution across different Linux distros. I noticed that this was mentioned before in #225, but it seems like it didn't get much traction. My manifest is for the latest v3.6.1-1 release and in my testing it works great!
Flathub
Packaging as a Flatpak allows for distribution on Flathub, the go-to Flatpak repository. This would allow easy distribution across several Linux distros (including those without .deb or .rpm support). I've personally been waiting for a Jupyter Flatpak to be available on Flathub for a while, and I figured this would be a great opportunity to fill the void.
My repo is ready for submission for Flahub's approval and I'd be happy to maintain the Flathub repository if needed.
Link to repo: https://github.com/kpthedev/flathub/tree/org.jupyter.JupyterLab

@mbektas FYI
@kpthedev Thank you for taking the initiative on this!

@mbektas Would you like to move forwards with this? I noticed that it was "approved" in the meeting, but I'm not quite sure what that means.
It's all ready to be submitted, and again I'd be happy to to maintain the repo.

hi @kpthedev github releases page is our main distribution channel but you can go ahead with your Flatpak distribution integration. Similarly someone volunteered to integrate with winget. Please make sure you update it regularly. Your configuration uses 3.6.1-1 but 3.6.1-2 is the latest now.

Sounds good!


Resolves dependabot warnings and errors

This PR resolves the dependabot alerts for the project.

@skyarkitekten can you sign the contributor agreement above?


Button to switch which subject's data is shown

Switch subject in 3D window.

Maybe this can be a radio button next to the table

@akeshavan could this be managed through a simple database that allows a user to select what is displayed in the 3d window? Or does this go beyond the scope of this project?

Per discussion with @arokem and @anotherjoshsmith, we decided to take this off of the revision milestone.


ISSUE: Child locks do not seem to work...

First, thanks for creating Keymaster. I have been using it for ~2 years (with a single lock on a different install).
I am running Keymaster 0.0.83, Z-Wave JS UI 1.5.0, and HA 2023.1.6.
I am trying to add 3 locks, 2 of them parents of the first. They are all Kwikset Home Connect 620s.
When I start adding the child locks, I get errors (like described in https://github.com/FutureTense/keymaster/issues/304).
The difference is that none of the child locks seem to be getting the pins.

304 is a missing notification script as found here in the wiki.

Hi, I think this (and #304) are a distinct issue from the missing notification script.  Note the 'not found' script in the error message is a different script than the notification one.
I just created the notifcation script, restarted HA and not fixed.
The script in the error message exists and shows up in HA's automations & scripts UI.  So it's weird that it's claimed not to exist.

Found the bug and confirmed a fix locally.  Will open a PR.


update template references

Our workflow has been to add CFN template files to aws-infra repo let the
CI deploy to S3 bucket then reference the file from the S3 bucket.  This
changes it so that we reference the file directly from GH instead of the S3
bucket.

Decided against this change because it would introduce two ways to reference files, GH and Buckets.  Since we can only  reference built SAM lambda templates from buckets it's just easier to standardize on one way to reference files.


ENYO-2533, ENYO-2696, ENYO-2514: Overhaul refresh / reset in VDR, NewDataList

This change moves the logic governing deferred refresh() and reset() behavior up the inheritance chain from NewDataList to VirtualDataRepeater and reintroduces async refresh for performance reasons.
In the process, it addresses ENYO-2533 and ENYO-2696, and replace the original fix for ENYO-2514.
See the notes on the individual commits for more detail.
Enyo-DCO-1.1-Signed-Off-By: Gray Norton<EMAIL_ADDRESS>

That's what I get for reviewing a commit at a time.  All clear now!


Implementation of some missing features in the SvgGdi Bridge

added "Rect-aligned text" GDI test and enhanced the "Lines" test in
SvgGdiTest application
replaced dependency on discontinued Adobe's SVG control with
dependency on WebBrowser control in test applications
implemented support for horizontal and vertical text alignment for
single-lined text with known bounding rectangle
implemented support for complex (GraphicsPath-based) CustomLineCap pen
style
minor bug fixes

Thanks a lot!

@monoman while playing with the code I've noticed the copyright header inconsistency. In my opinion the license header in all files should be equipped with Copyright © 2010 SvgNet & SvgGdi Bridge Project... (assuming this actually means "monoman's repository at GitHub") as the original copyright holder Ben Peterson from RiskCare Ltd. became somehow net-inactive

That copyright came in the original code I copied into GitHub.
I can accept a PR to fix that. :)
Really I could not find Ben Peterson also. But the original licensing allow us to bring it back alive.
Thank you for your help!


Can't pull metrics from Confluent Cloud. Bootstrap Broker disconnected.

Hello,
I'm trying to run the kafka-lag-exporter against a confluent cloud cluster. Below is my configuration
kafka-lag-exporter {
reporters.prometheus.port = 9999
clusters = [
{
name = "a-cluster"
bootstrap-brokers = "pkc-XXXXX.westus2.azure.confluent.cloud:9092"
labels = {
location = "chennai"
zone = "asia-south-east"
}
admin-client-properties = {
client.id = "admin-client-id"
ssl.endpoint.identification.algorithm = "https"
sasl.mechanism = "PLAIN"
sasl.jaas.config = "org.apache.kafka.common.security.plain.PlainLoginModule required username='XXXXXX' password='xxxxxxxx';"
security.protocol = "SASL_SSL"
request.timeout.ms = 60000
retry.backoff.ms = 500
  }
}

]
}
This is the problem in the log
2020-09-23 14:33:49,990 INFO  c.l.k.ConsumerGroupCollector$ akka://kafka-lag-exporter/user/consumer-group-collector-a-cluster - Collecting offsets
2020-09-23 14:33:51,150 WARN  o.apache.kafka.clients.NetworkClient  - [Consumer clientId=consumer-kafkalagexporter-1, groupId=kafkalagexporter] Bootstrap broker pkc-xxxxxx.westus2.azure.confluent.cloud:9092 (id: -1 rack: null) disconnected
I also tried this - using double quotes around username and password with "
sasl.jaas.config = "org.apache.kafka.common.security.plain.PlainLoginModule required username="XXXXXX" password="xxxxxxxx";"
P.S I'm able to list and describe the consumer groups using kafka-consumer-groups cli tool using the same configuration. I'm on a Ubuntu server 18.01 LTS and Java openjdk version "11.0.8" 2020-07-14

In my setup we specify the credentials twice, once for admin-client and again for consumer.
kafka-lag-exporter {
  clusters = [
    {
      name = "default"
      bootstrap-brokers = "..."
      admin-client-properties = {
        security.protocol = "SASL_SSL"
        sasl.mechanism = "..."
        sasl.jaas.config = "..."
      }
      consumer-properties = {
        security.protocol = "SASL_SSL"
        sasl.mechanism = "..."
        sasl.jaas.config = "..."
      }
    }
  ]
}

@dylanmei Thanks a lot. Worked.

Wow, the same question twice in one day.  One way to make this less of an issue would be to have a base-properties, or connection-properties that could be merged into both.  I guess for most cases you would only need to set the connection-properties.


Issue with proxy configuration for elemental/download route

There's an issue with proxy configuration for elemental/download route on elemental v1.0.0, which prevents the correct download of the registration YAML. It can be downloaded, but the content is gibberish.
Let's try to release a new shell version, update it on elemental and check if it fixes the problem

With rancher 2.7.2-rc2 and elemental-ui 1.0.1, I was able to download the registration file and the content is fine.


update changelog for v9.0.2

Description

closes: #XXXX

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to GitHub issue with discussion and accepted design, OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/).
[ ] Added relevant godoc comments.
[ ] Provide a conventional commit message to follow the repository standards.
[ ] Include a descriptive changelog entry when appropriate. This may be left to the discretion of the PR reviewers. (e.g. chores should be omitted from changelog)
[ ] Re-reviewed Files changed in the GitHub PR explorer.
[ ] Review SonarCloud Report in the comment section below once CI passes.

I guess should be on main as well as we talked about

Yes, will add it there too.


Add run_without_submitting to simple functions, datasinks

While resuming a mostly-completed workflow, I noticed a number of simple nodes being scheduled, running in <2 seconds, and thus causing the scheduler to wait unnecessarily.
I don't know if this will even buy more than a minute speedup for a large, multi-subject run, so consider this a super low-priority review.

I wonder if this would make sense for niu.Merge MapNodes.

Probably. Most of the niu nodes, really, except for Function nodes that do real processing. (My rule was: If it reads and writes a file bigger than an affine matrix, leave it alone.) I didn't think to look for Merge.


Add support to transformed_data for reconstructed (from_dict) charts

Relates to #313. For layered and multi-view charts, Altair takes a bit of a shortcut and does not use the exact classes which map to the Vega-Lite schema but instead uses the API convenience classes such as alt.Chart, alt.LayerChart, etc. For example for the layered chart below, the elements of layer are alt.Chart instances:
import altair as alt
import vegafusion as vf
from vega_datasets import data

source = data.wheat()

bar = alt.Chart(source).mark_bar().encode(
    x='year:O',
    y='wheat:Q'
)

rule = alt.Chart(source).mark_rule(color='red').encode(
    y='mean(wheat):Q'
)

layer_chart = (bar + rule).properties(width=600)
type(layer_chart.layer[0])  # altair.vegalite.v5.api.Chart

However, when reconstructing this chart from a dictionary, the layers are of type UnitSpec which is the correct class following the VL schema:
reconstructed_layer_chart = alt.Chart.from_dict(layer_chart.to_dict())
type(reconstructed_layer_chart.layer[0])  # altair.vegalite.v5.schema.core.UnitSpec

When calling transformed_data on the reconstructed chart, it fails as the isinstance checks do not accept UnitSpec as an equivalent to alt.Chart although it should be:
vf.transformed_data(reconstructed_layer_chart)

ValueError: transformed_data accepts an instance of Chart, FacetChart, LayerChart, HConcatChart, VConcatChart, or ConcatChart
Received value of type: <class 'altair.vegalite.v5.schema.core.UnitSpec'>

All classes which I think can be treated as equivalent to the first one mentioned in every bullet:

Chart: TopLevelUnitSpec (parent class of Chart), FacetedUnitSpec, UnitSpec, UnitSpecWithFrame, NonNormalizedSpec (used in concat charts for chart objects in e.g. hconcat attribute)
LayerChart: TopLevelLayerSpec (parent class of LayerChart), LayerSpec
RepeatChart: TopLevelRepeatSpec (parent class of RepeatChart), RepeatSpec
ConcatChart: TopLevelConcatSpec (parent class of ConcatChart), ConcatSpecGenericSpec
HConcatChart: TopLevelHConcatSpec (parent class of HConcatChart), HConcatSpecGenericSpec
VConcatChart: TopLevelVConcatSpec (parent class of VConcatChart), VConcatSpecGenericSpec
FacetChart: TopLevelFacetSpec (parent class of FacetChart), FacetSpec

Spec classes which I'm not sure about but we can probably ignore as repeat charts are not supported yet by transformed_data anyway:

LayerRepeatSpec
NonLayerRepeatSpec
GenericUnitSpecEncodingAnyMark

Just a reminder that the same changes would need to be applied to https://github.com/altair-viz/altair/blob/master/altair/utils/_transformed_data.py.
Btw, my use case for the above is that a frontend sends the chart specifications as a JSON to a backend API which executes vf.transformed_data(alt.Chart.from_dict(spec)) and then returns the extracted data as an Excel file to the user of the web application.

Thanks for the report @binste, I think this all makes sense. And now that chart.transformed_data is in Altair master, we need to fix it there as well!

I don't have a working development setup for VegaFusion right now. Is it easy to set it up to work purely on the Python part without Rust or do I need to compile the Rust code anyway? I assume the later. Let me know if I should land a hand on implementing the changes in Altair.

Let me know if I should land a hand on implementing the changes in Altair.

You can develop the Python part of VegaFusion on its own. But if you want to work on fixing it in the Altair implementation of chart.transformed_data (which doesn't use the vf.transformed_data implementation), I could pretty easily copy the fixes into the VegaFusion implementation afterward.

Great, I'll work on the Altair implementation probably early next week.


running azure bus sampler  amqp over websocket not working in NON GUI mode.

Operator called default onErrorDropped
reactor.core.Exceptions$ErrorCallbackNotImplemented: com.azure.core.amqp.exception.AmqpException: Connection refused: no further information, errorContext[NAMESPACE: ERROR CONTEXT: N/A]
Caused by: com.azure.core.amqp.exception.AmqpException: Connection refused: no further information, errorContext[NAMESPACE: ERROR CONTEXT: N/A]
at com.azure.core.amqp.implementation.ExceptionUtil.toException(ExceptionUtil.java:85) ~[jmeter-plugins-azure-servicebus-0.3.1.jar:?]
at com.azure.core.amqp.implementation.handler.ConnectionHandler.notifyErrorContext(ConnectionHandler.java:325)
NOTE- its working in GUI mode

Its does not set proxy

When set proxy in jmeter , in NON GUI mode its not picking that proxy settings

Hello tarung121,
In my environment I can send messages to Azure Service Bus.
Please elaborate on your environment and situation.
JMeter version:
OS/distribution and version:
Java runtime provider and version:
If your OS is Linux, how you installed Java:
JMeter GUI or CLI mode: (I am aware of problems with CLI, GUI is OK)
Are you using proxy?: (I am aware that you do not use it.)
Queue or Topic?:
Protocol: (AMQP / AMQP over Web Sockets)
Auth type:
Note that if you select "AMQP" for Protocol, you cannot use Proxy; if you want to use proxy, you must select "AMQP over Web Sockets".

JMeter version: 5.6.3
OS/distribution and version: Windows 10 enterprise 10.0.19045 build 19045
Java runtime provider and version: jre <IP_ADDRESS>
If your OS is Linux, how you installed Java:
JMeter GUI or CLI mode: (I am aware of problems with CLI, GUI is OK) running using command line jmeter -n -t
Are you using proxy?: (I am aware that you do not use it.) yes i am using proxy so plugin dont able to use plugin
Queue or Topic?: queue
Protocol: (AMQP / AMQP over Web Sockets) - AMQP over Web Sockets
Auth type: shared access signature
Problem is plugin dont able to use proxy defined in jmeter or you pass in command line also using -H -P

Even if AMQP over Web Sockets was selected, when running in Cli mode, it would run as AMQP protocol, and Proxy was not being used accordingly.
Fixed in Azure Service Bus Sampler v0.3.2.
Thanks for your patience.


Print version info

This PR adds support for printing the build version and the OpenTelemetry SDK version on Beyla startup.
I'm not sure there's a better way to do this, I looked into debug/BuildInfo, but it's not as nice as git describe --tags and it has this restriction https://github.com/golang/go/issues/51264.
The one unfortunate sideffect is that it adds 10 seconds build time for me, the link time change of the value seems slow. Maybe we need better default make for development?
Closes #343

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

Comparison is base (cfbd80c) 80.02% compared to head (463b0bc) 42.12%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #365       +/-   ##
===========================================
- Coverage   80.02%   42.12%   -37.90%
===========================================
  Files          51       49        -2
  Lines        3999     3879      -120
===========================================
- Hits         3200     1634     -1566
- Misses        619     2139     +1520
+ Partials      180      106       -74

Flag
Coverage Δ

integration-test
?

unittests
42.12% <ø> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 37 files with indirect coverage changes

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

LGTM


Start adminMongo with context path

At the moment it's not possible to start adminMongo with a context path (e.g. 'http://localhsot:1234/').
Is there any way to configure a context path?

I'm not really sure what you are wanting here? Can you explain further?

Sorry, my example is wrong.
I'd like to configure adminMongo to run with a context path like "http://localhost:1234/adminMongo/".
At the moment, when I start your app, it's alsway on the root path.

Ok I understand. I think though that adding the /adminMongo directive would be the job for a web server like Apache or Nginx and not the application.

I understand that.
We need to run adminMongo in a docker container. We habe some applications in docker container running and a proxy before them. We need all docker not to run in the root context. Most of them running a tomcat, so there is no problem.
My hope was not to install a web server in the docker container for adminMongo.

There is no built in way to achieve this in Express. It's not impossible but I feel it would be best to leave it to the web server. I understand this doesn't help in your case but I would think (I could stand corrected) that the need for this would be pretty low with the majority of users.


Tilt correction wrong if up != Vec3::Y

I think there's an error here: the tilted_up vector should be calculated as the tracked rotation applied to the y-axis rather than to self.up:
https://github.com/idanarye/bevy-tnua/blob/5fdc7c284109880a6e29dfeacb6eee07ac5c184f/src/builtins/walk.rs#L282-L284
The rotation_required_to_fix_tilt line makes sense - the desired "equilibrium" position is that where tilted_up == self.up. However, computing tilted_up as tracker.rotation * self.up means this condition becomes self.up = tracker_rotation * self.up, so the resulting equilibrium character transform is a rotation around self.up (in world-space) rather than a rotation of the "character-space" up vector (the y-axis) onto the desired world-space up vector self.up.
This leads to some odd-looking effects in-game when rotating the desired_forward direction of a character with self.up set to the normal of an angled floor.
Changing line 282 to
let tilted_up = ctx.tracker.rotation.mul_vec3(Vec3::Y);

solved these issues for me in local testing.
Hopefully this makes sense and I'm not just misinterpreting the intended usage of the up vector or something. Happy to open a PR if you'd prefer but that felt a bit overkill for a +/- 7-character proposed change

Originally up was accompanied by another field - forward - and together they decided the orientation - which directions are considered up and the forward. They were both part of the configuration struct - unlike desired_forward, which was part of the controls struct.
When I did the big refactor of version 0.10, I did many changes - two of them I think are relevant to the confusion:

I got rid of automatic rotation based on movement direction, I also decided to get rid of forward - ruling that in 3D it'll always be negative Z and in 2D it won't be used.
I unified the configuration and the controls fields - so up is now in the same struct as desired_forward.

So if previously it was clearer that up as a similar purpose to forward - now it looks like it has a similar purpose to desired_forward. I think what you need is a desired_up field - which Tnua does not currently have.
The current up is not "where the head should be facing" - it's "which direction is considered as 'up'". For example - that's the direction opposite to the gravity. I think (didn't actually test this) that if you change it it'll cause problem because the spring force will no longer be parallel to the gravity.
Maybe I really should replace up with a proper desired_up that has the correct semantics...

Now that I think about it - maybe the up direction should be set from the gravity? Instead of actions getting it from the basis, I can add it to the context and both basis and action will get it from there.
As for the desired_up - currently the ray's cast direction is affected by the rotation, which is not that noticeable but a desired_up will make it noticeable. I assume that even if the character is tilted by desired_up we still want to cast the ray straight down?

I think I'm encountering a similar issue.  I've been adjusting gravity in my world to be a line from my character to the center of the globe (origin), but my character wants to stand upright to Vec3::Y no matter what I do.
Is there a way I can change the up direction when I'm also adjusting gravity?

Awesome, thanks for this fix!
Note that I'm close on having an update to the demo that allows you to switch between the flat world and a spherical world.

@rltvty When you change the gravity to fit the player's position, what happens to the other rigid bodies? I assume your spherical world is small enough to have a noticeable curve. but AFAIK both Rapier and Avian only support a uniform gravity...


GH-43284: [Release] Fix variable name on post-12-bump-versions.sh script

Rationale for this change
The script fails to bump deb package names at the moment due to a wrong variable name.
What changes are included in this PR?
Update the variable name to the correct one.
Are these changes tested?
It has been tested locally to bump the versions for the debian package names on main for 17.0.0
Are there any user-facing changes?
No

GitHub Issue: #43284

I am not sure why update_deb_package_names is failing on the git mv here: https://github.com/apache/arrow/blob/main/dev/release/utils-prepare.sh#L239-L241
I was doing some tests locally but I don't understand what the issue is:
+ git mv 'debian*/lib*1701.install' debian/libarrow1800.install debian/libarrow-acero1800.install debian/libarrow-cuda1800.install debian/libarrow-cuda-glib1800.install debian/libarrow-dataset1800.install debian/libarrow-dataset-glib1800.install debian/libarrow-flight1800.install debian/libarrow-flight-glib1800.install debian/libarrow-flight-sql1800.install debian/libarrow-flight-sql-glib1800.install debian/libarrow-glib1800.install debian/libgandiva1800.install debian/libgandiva-glib1800.install debian/libparquet1800.install debian/libparquet-glib1800.install
fatal: destination 'dev/tasks/linux-packages/apache-arrow/debian/libparquet-glib1800.install' is not a directory

cc @kou

I am not sure why update_deb_package_names is failing on the git mv here: https://github.com/apache/arrow/blob/main/dev/release/utils-prepare.sh#L239-L241 I was doing some tests locally but I don't understand what the issue is:
+ git mv 'debian*/lib*1701.install' debian/libarrow1800.install debian/libarrow-acero1800.install debian/libarrow-cuda1800.install debian/libarrow-cuda-glib1800.install debian/libarrow-dataset1800.install debian/libarrow-dataset-glib1800.install debian/libarrow-flight1800.install debian/libarrow-flight-glib1800.install debian/libarrow-flight-sql1800.install debian/libarrow-flight-sql-glib1800.install debian/libarrow-glib1800.install debian/libgandiva1800.install debian/libgandiva-glib1800.install debian/libparquet1800.install debian/libparquet-glib1800.install
fatal: destination 'dev/tasks/linux-packages/apache-arrow/debian/libparquet-glib1800.install' is not a directory

This is happen because there are no files that are matched against debian*/lib*1701.install.
The pattern at https://github.com/apache/arrow/blob/4161898d5cd7d05ad0fd9f23e1e9df9439460c98/dev/release/utils-prepare.sh#L238 is used as-is for git mv. So https://github.com/apache/arrow/blob/4161898d5cd7d05ad0fd9f23e1e9df9439460c98/dev/release/utils-prepare.sh#L231 value is strange.

Could you share the command line you tried?
I want to try it too.

Ah, we need to run current_version before we call update_versions:
diff --git a/dev/release/post-12-bump-versions.sh b/dev/release/post-12-bump-versions.sh
index 422821a66b..206da6ccb4 100755
--- a/dev/release/post-12-bump-versions.sh
+++ b/dev/release/post-12-bump-versions.sh
@@ -41,6 +41,8 @@ version=$1
 next_version=$2
 next_version_snapshot="${next_version}-SNAPSHOT"

+current_version_before_bump="$(current_version)"
+
 case "${version}" in
   *.0.0)
     is_major_release=1
@@ -64,7 +66,7 @@ if [ ${BUMP_VERSION_POST_TAG} -gt 0 ]; then
 fi

 if [ ${BUMP_DEB_PACKAGE_NAMES} -gt 0 ] && \
-     [ "${next_version}" != "$(current_version)" ]; then
+     [ "${next_version}" != "${current_version_before_bump}" ]; then
   update_deb_package_names "${version}" "${next_version}"
 fi

Could you share the command line you tried? I want to try it too.

I was just trying what the test was doing on the branch:
BUMP_DEFAULT=0  BUMP_DEB_PACKAGE_NAMES=1 dev/release/post-12-bump-versions.sh 17.1.0 18.0.0

I also added the set -x to the post-12-bump-versions.sh:
diff --git a/dev/release/post-12-bump-versions.sh b/dev/release/post-12-bump-versions.sh
index 2eeb715..5ffc7d3 100755
--- a/dev/release/post-12-bump-versions.sh
+++ b/dev/release/post-12-bump-versions.sh
@@ -17,7 +17,7 @@
 # specific language governing permissions and limitations
 # under the License.
 #
-set -ue
+set -uex

 SOURCE_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
 source "${SOURCE_DIR}/git-vars.sh"

(I'm writing a test for this case...)

Could you apply this?
diff --git a/dev/release/post-12-bump-versions-test.rb b/dev/release/post-12-bump-versions-test.rb
index 2bd1458746..f31e1a3122 100644
--- a/dev/release/post-12-bump-versions-test.rb
+++ b/dev/release/post-12-bump-versions-test.rb
@@ -358,8 +358,15 @@ class PostBumpVersionsTest < Test::Unit::TestCase
   def test_deb_package_names
     omit_on_release_branch unless bump_type.nil?
     current_commit = git_current_commit
-    stdout = bump_versions("DEB_PACKAGE_NAMES")
-    changes = parse_patch(git("log", "-p", "#{current_commit}.."))
+    stdout = bump_versions("VERSION_POST_TAG", "DEB_PACKAGE_NAMES")
+    log = git("log", "-p", "#{current_commit}..")
+    # Remove a commit for VERSION_POST_TAG
+    if log.scan(/^commit/).size == 1
+      log = ""
+    else
+      log.gsub!(/\A(commit.*?)^commit .*\z/um, "\\1")
+    end
+    changes = parse_patch(log)
     sampled_changes = changes.collect do |change|
       first_hunk = change[:hunks][0]
       first_removed_line = first_hunk.find { |line| line.start_with?("-") }


Clarity on URLs

This page defines the URLs required as
https://login.microsoftonline.com
https://login.windows.net
https://login.microsoft.com
https://sts.windows.net
https://login.partner.microsoftonline.cn
https://login.chinacloudapi.cn
https://login.microsoftonline.de
https://login.microsoftonline.us
https://login.usgovcloudapi.net
https://login-us.microsoftonline.com
The page https://docs.microsoft.com/en-us/azure/active-directory/develop/msal-ios-shared-devices lists the url as
https://login.microsoftonline.com
Which page is correct?

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

ID: f3aa0933-44ee-6c0f-caf2-defe2b90c3f9
Version Independent ID: 7a281e52-2e8d-cae3-34b8-8ce5f6b78c03
Content: Microsoft Enterprise SSO plug-in for Apple devices - Microsoft identity platform | Azure
Content Source: articles/active-directory/develop/apple-sso-plugin.md
Service: active-directory
Sub-service: develop
GitHub Login: @brandwe
Microsoft Alias: brandwe

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

Hi @bubbletroubles, the correct list is defined in this article, and the other article has been updated in https://github.com/MicrosoftDocs/azure-docs/commit/7f10ecd6abc3d8be9ed815d7857a65a6948d1379 to reflect same.

URLs:

https://login.microsoftonline.com
https://login.microsoft.com
https://sts.windows.net
https://login.partner.microsoftonline.cn
https://login.chinacloudapi.cn
https://login.microsoftonline.de
https://login.microsoftonline.us
https://login.usgovcloudapi.net
https://login-us.microsoftonline.com

Thanks for the notification of the discrepancy!
#please-close


AttributeError from checker.py

Hi,
I just freshly installed this and flake8 on python3.7 and on any use I'm getting an error on simple flake8 file.py . Last call of the traceback is:
  File "c:\python37\lib\site-packages\flake8_type_checking\checker.py", line 325, in visit_FunctionDef     for i in range(node.lineno, node.end_lineno + 1):  # type: ignore AttributeError: 'FunctionDef' object has no attribute 'end_lineno'
bug-report info:
{
"dependencies": [],
"platform": {
"python_implementation": "CPython",
"python_version": "3.7.3",
"system": "Windows"
},
"plugins": [
{
"is_local": false,
"plugin": "flake8-type-checking",
"version": "1.0.3"
},
{
"is_local": false,
"plugin": "mccabe",
"version": "0.6.1"
},
{
"is_local": false,
"plugin": "pycodestyle",
"version": "2.7.0"
},
{
"is_local": false,
"plugin": "pyflakes",
"version": "2.3.1"
}
],
"version": "3.9.2"
}

Hi @Yobmod, thanks for submitting the issue!
It looks like end_lineno was implemented in Python 3.8 (see here) 🤔
Since mapping the start and end of different ast objects is pretty integral to how the plugin functions I'm not sure we'd be able to create a workaround for python versions <= 3.8, but you're definitely free to try to if you want.
If not, I guess I should probably change the package metadata to only support Python 3.8+ 🙂 Sorry about that

Ah, ok. Thanks for the fast reply.
I seem to spend all my time dealing with typing changes between versions, lol.

Updated python versions in https://github.com/snok/flake8-type-checking/commit/ae69260264bd024a762d1b2adbfb809ddacf2b1e 🙂

:wave: Guys, please consider releasing metadata fix to PyPI, it's still advertised as 3.7-compatible there and people run into this issue.
By the way, isn't this note from ast docs a little worrying?

Note that the end positions are not required by the compiler and are therefore optional.

It implies you are relying on something that does not necessarily exist, even in 3.8+. This is confirmed by ast's typeshed typings - end_lineno is nowhere to be seen.

Good point. I'll update the metadata tonight after fixing broken tests in v3.8 👍
Regarding the optionality of end_lineno - I've not personally seen any problems on Python 3.8+, and we've not had any issues as far as I can remember, so I think I'll assume it's fine until proven otherwise 🙂 Happy to be convinced otherwise though.

Metadata has been updated in v1.0.4 🙂

In case you'd like to support Python 3.7 again (at least until mid-2023 when it reaches EOL), here's a solution (roughly stripped down) I've been using in a project of mine:
import inspect
import tokenize
from pathlib import Path

def tokens(path: Path) -> tuple[list[tokenize.TokenInfo], defaultdict]:
    with open(path) as fd:
        tokens = list(tokenize.tokenize(fd.readline))
    token_table = defaultdict(list)  # mapping line numbers to token numbers
    for index, token in enumerate(tokens):
        token_table[token.start[0]].append(index)
    return tokens, token_table

def endlineno(lineno: int, filepath: Path) -> int | None:
    tokens, tokens_by_line = tokens(filepath)
    first_token_index = tokens_by_line[lineno][0]
    blockfinder = inspect.BlockFinder()
    with suppress(inspect.EndOfBlock, tokenize.IndentationError):
        for token in tokens[first_token_index:]:
            blockfinder.tokeneater(*token)
    return blockfinder.last

Thanks @vect-tmazzucotelli, that's very helpful 👏
I personally won't use this in a Python 3.7 project, so don't think I'll be able to prioritise this myself, but for anyone who would, PRs are welcome 🙂


print_to_stdout inconsistent

It seems that run_design.py and run_experiment.py pass bool print_to_stdout to the flow_fcn(). Not all flow functions accept this parameter. Also, is this parameter needed? Maybe multi-threading could be edited to always print each thread to a different file and never to stdout (not sure why you would want all the threads printing to stdout). Otherwise, if the user does not want the results of the run then they should use >> to redirect stdout to a file.

I agree. Sound good!

Resolved in PR #70


chore: add browserslist config "fully support es6"

Technically a breaking change but I believe it makes sense to only support es6 supported browsers which get's rid of a lot of the polyfills that babel need to insert which is inflating the bundle a lot. It also allows babel to use object deconstruction internally in the transformers and other newer syntax so further reduce the bundle.
The minified file in the top level of the repo went from 7 540 byte to 6 341 byte. A reduction of ~16%.
ES6 is supported by 97.1% of the browsers right now: https://browsersl.ist/#q="browserslist"%3A+[
++++++++"fully+supports+es6"
++++++]
I didn't see any reference to which browsers or JS versions that are supported so I don't believe this requires any documentation updates but please let me know if it does.
PS: If you prefer this as a option inside the .babelrc file instead I'd be happy to change it but in general I think it's best to set it in the package.json so all tooling picks it up.
Checklist

[x] only relevant code is changed (make a diff before you submit the PR)
[x] run tests npm run test
[x] tests are included
[x] commit message and code follows the Developer's Certification of Origin

Checklist (for documentation change)

[x] only relevant documentation part is changed (make a diff before you submit the PR)
[x] motivation/reason is provided
[x] commit message and code follows the Developer's Certification of Origin

An alternative to this which might be preferable is to just use @babel/plugin-transform-modules-commonjs instead of preset env and let the consumers build system handle any transformations. Especially for the ESM builds.

I would also be happy to change this to what matches the main i18n package but that requires installing @babel/preset-react as well.
Might make sense to have them all unified though.

should be in line with react-i18next

at the end it's: "1kb of size reduction" vs. "dropping support for a lot of older environments"... is this really a huge benefit?

at the end it's: "1kb of size reduction" vs. "dropping support for a lot of older environments"... is this really a huge benefit?

Considering we have to load i18next and all plugins for it in our index bundle making that as small as possible is very much something that would benefit us.
There is also a small environmental impact to this as it would save just us tens of gigabytes in lowered data transfers per year from just this simple change.
I haven't checked how it would look with the i18next config in terms of numbers but I'd say dropping support for 2.9% of the users with this current config is not really a huge deal. Especially since the consumers of this package don't have to update if they really need it. For example 100% of our own user base at @electricitymaps would be supported and there would only be upsides to this for us.

i think it's relatively simple... we can update it... but if there will be complaints... we will consider to revert it

I'll update this PR to match the react-i18next config as soon as I have some time then.

I have matched the react-i18next config where it made sense, @babel/plugin-transform-runtime is already included in the @babel/preset-env and @babel/preset-react is not needed as this package don't have any jsx source code.
The size of the generated minified code is now 6 370 bytes. Likely cause it is using const and let as the source code did and not converting everything to var.

BTW: Once this is merged I plan on opening a follow up PR that changes code to take advantage of these changes.

do note that supports es6 and defaults are a different audience reach.


##About building Chinese datasets

Hello blogger, I found that this tool can only build a dataset for English text. When I try to use Chinese text, it cannot mark the aspect words in it.
I would like to ask if you have any tools here suitable for building Chinese datasets！

https://github.com/yangheng95/ABSADatasets/issues/14

thank you very much！


Added Logic Apps template for ingesting docs into Azure Cosmos DB for NoSQL for RAG

This Logic Apps template makes it easy to take new and modified documents from blob storage, vectorize the text, format it the text, vectors, and other data into data items, and insert them into Azure Cosmos DB for NoSQL.

@Elaina-Lee, can we please have a review of this template?


Update CONTRIBUTERS.MD

What type of PR is this? (check all applicable)

[YES] 🚀 Added Name
[ ] ✨ Feature
[YES] 🌟 stared the repo
[ ] 🐛 Grammatical Error
[YES] 📝 Documentation Update
[ ] 🚩 Other

Description
Added my name into the CONTRIBUTORS.md file
Add Link of GitHub Profile
https://github.com/upadhyay8844

Updated my name in CONTIBUTORS.md file


45kb/s

My Internet speed is 8MBPS, But download activity on any series stand alone is just between 45-73 KB/S?
Software restriction?

Now the download has gone to ZERO.

Now the download has gone to ZERO.

Ya , same problem here with differnt numbers

Ya , same problem here with differnt numbers

confirmo

Actually sometimes if you've other things using internet on your computer for example you're downloading something on chrome and downloading lectures at the same time, then there is a fair amount of chance that your speed will go down


Setting an alert to whisper uses prevents opening of the turn alerts window

When manually creating a turn alert that includes a whisper, the alert does not appear in the window and further attempts to re-open the combat alert window result in the following error in foundry
TypeError: An error occurred while rendering CombatAlertsApplication 74: Cannot read property 'data' of undefined     at CombatAlertsApplication.js:104     at Array.map (<anonymous>)     at CombatAlertsApplication._createAlertDisplayData (CombatAlertsApplication.js:104)     at Array.map (<anonymous>)     at CombatAlertsApplication.js:78     at Array.map (<anonymous>)     at CombatAlertsApplication._turnData (CombatAlertsApplication.js:70)     at CombatAlertsApplication.getData (CombatAlertsApplication.js:60)     at CombatAlertsApplication._render (foundry.js:2042)     at CombatAlertsApplication.render (foundry.js:2004) (anonymous) @ foundry.js:2006
Steps to recreate:

Create an active combat with at least one combatant
Click on the bell icon
Create a new combat alert that includes a whisper
Attempt to re-open turn alert window

I tested this with the 5e system and with no other modules enabled.
The alerts still output to chat appropriately on the turn they are supposed to, and once the offending alert completes, the window can be opened again.

I can confirm this also happens in my game using Foundry 0.8.9.


add support for responsiveness when peaks are pre-rendered

Hey,
Thanks for the great lib.
Since wavesurfer allows to pass pre-rendered peaks, responsiveness should be enabled before the audio has been loaded (aka before wavesurfer's "ready" event). So instead of hooking to the window's resize after the "ready" event has been raised, hook to it after component mount.
Also removed empty() call before redrawBuffer on resize, because it works without it, and with it there is a bug, see here: https://github.com/katspaugh/wavesurfer.js/issues/943
Omer

Thanks @omerts !! – I'm currently working on an extensive refactor of the wavesurfer.js lib itself and I will take this into account when I add a responsive option to the core lib (then we can just use for react-wavesurfer that instead of doing our own resize-event-listening) – I will publish a new version of react-wavesurfer shortly. (I'm assuming you've run into this issue while using the lib and would appreciate not having that issue anymore :) )

Ahh. Only just remembered about the release, my bad folks!
FYI the wavesurfer responsive feature API is being discussed here https://github.com/katspaugh/wavesurfer.js/pull/963

Released as v0.8.3 – Sorry it took so long.


Add an option to avoid building binary when used as a library

When Ninja is used as libraries (libninja & libninja-re2c), we actually need to build them but not a ninja binary. Therefore, I added a CMake variable: NINJA_NO_BINARY to prevent building it if users of Ninja's libraries explicitly set it as ON (default is OFF for backward compatibility).

You can specify which targets to build, so if you only want to build libninja run ninja libninja in the build directory.

Ah I see! Is there any standard CMake way for this or a convention how the variable should be named? In any case I would prefer it if you'd drop the double negative, i.e. name the variable NINJA_BINARY and default to 1.

@jhasse
Sure! In this case AFAIK, using NINJA_BUILD_BINARY with the option command would be much clearer :)
I have updated the code, but what do you think about this name?


remote media control fixes

IOS on lock screen media control buttons are enabled now

This don't really fix the problem. You're just commenting the logic and hardcoding it.


MultiCompilerOptions is not exported in types.d.ts

Bug report

What is the current behavior?
MultiCompilerOptions is declared in types.d.ts but it is not exported and can't be used when typing webpack.config.js
Result (without webpack.MultiCompilerOptions type):
Property 'parallelism' does not exist on type 'Configuration[]'.ts(2339)
If the current behavior is a bug, please provide the steps to reproduce.
Example:
/**
 * @param {NodeJS.ProcessEnv} env
 * @returns {webpack.Configuration[] & webpack.MultiCompilerOptions}
 */
function createWebpackConfigurations (env) {
  /// returns array of webpack.Configurations
}

export default (/** @type {NodeJS.ProcessEnv} */ env) => {
  const allConfigurations = createWebpackConfigurations(env)
  allConfigurations.parallelism = parallelism
  return allConfigurations
}

What is the expected behavior?
MultiCompilerOptions type should be usable in webpack.config.js when webpack is imported

Other relevant information:
webpack version:  5.88.2
Node.js version: v16.15.1
Operating System: Windows 10
Additional tools: Typescript 5.2.2

This is still valid with unmerged pull request.


Issue3140 - log and replay unconfirmed blocks

While cleaning this up I discovered a new issue, #3172 that can probably be resolved at a future point in time.


Added TravisCI configuration file

I set up CI on the travis-ci.com side of things.  If all is correct, it is configured to run tests on every pull request or push after this one.

So, perhaps I was wrong!  It looks like it worked fine!
If you click one of the above "Details" links and then click on one of the Build Jobs, you can see the console output of the build/test.

Looks great man, good call on allowing failure for the nightly.  Looks like everything is working perfectly.