Added new bazel build rules for c++/python

This reworks the existing flatbuffer bazel rules to be more general and handle transitive dependencies properly.
The goal is to be able to express a build graph of flatbuffer rules where each rule only needs to specify its immediate dependencies.
Example:
flatbuffer_cc_library(name = "a", srcs = ["a.fbs"])

# b.fbs imports a.fbs
flatbuffer_cc_library(
    name = "b",
    srcs = ["b.fbs"],
    deps = [":a"],
)

# c.fbs imports b.fbs
flatbuffer_cc_library(
    name = "c",
    srcs = ["c.fbs"],
    deps = [":b"],
)

In this setting, flatc should have the visibility into the tree of flatbuffer files which are dependents of c.
In the existing flatbuffer bazel rules, the user is required to specify all child dependencies of a rule, which becomes difficult to maintain in larger code bases.
The way this change works is to use a bazel aspect to propagate the flatbuffer source files which generated a rule.
This change also adds flatbuffer_py_library and an easy integration for other language toolchains.

This was committed into master but we are using "dev"

Hmm I don't see a dev for the flatbuffers repo. I did also cherry-pick this to figure/base/23_5_26 (which I'll send a PR to bump the project-x version shortly)

Hmm I don't see a dev for the flatbuffers repo. I did also cherry-pick this to figure/base/23_5_26 (which I'll send a PR to bump the project-x version shortly)

Oh dang, I totally missed that this was in the other repo :) my bad

The rules we use in our tree are our in-tree rules, mostly because they are closer to our code and easier to iterate on. Is there a reason to put the rules in this repo?

There's nothing here specific to project-x, and I'd like to upstream this at some point as the original bazel rules are pretty cumbersome. Ideally the functions here expose as many knobs as we need to tweak in our internal code without needing to fork the rules entirely, wdyt?

There's nothing here specific to project-x, and I'd like to upstream this at some point as the original bazel rules are pretty cumbersome. Ideally the functions here expose as many knobs as we need to tweak in our internal code without needing to fork the rules entirely, wdyt?

So my concerns are that now to understand what the flatbuffer_cc_library() bazel rule is doing you have to understand our file then you have to jump to a different repo and understand that file. Also having both rules with the same name complicates it a bit.
I think the benefit of being upstreamed isn't immediately clear, there are quite a few missing arguments which means there is missing functionality that would need to be implemented to upstream it. I like the fact that the current impl focuses on what we use so there isn't extra noise about other options but those would need to be added back to upstream, since the flatc flags are the API surface that we are actually interested in then I don't think we would gain much from upstream maintaining this rule, as we will still have access to any new flatc flags.
All of that said I very much like the improvements and don't want to stand in the way of them landing so feel free to keep it this way if you want but I think splitting the rule (in our perspective) across two repos is a regression.

Jumping between repos is a fair concern. I'd ideally have the library repo provide solid rules that are project-agnostic, similar to protobuf's family of bazel rules and the other bazel libraries we depend on internally; in particular if we have other internal repos that want to use flatbuffers then we don't need a copy of the build rules per repo. Perhaps we can vendor these rules into project-x in a similar fashion as our other third_party/X/ dependencies.
I agree we don't see much benefit on our end from upstreaming this. These rules remove the need for many of the flags in the original rules and I think are much simpler to use in any bazel project -- at some point we could open a discussion with the upstream maintainers if there's a path to align but otherwise I'd be happy to keep this in our fork, or move this to a separate bazel rules repo similar to pybind11_bazel.


ScreenManager:pop() doesn't return the last state

The docs and comments say that ScreenManager:pop() returns the screen on top of the stack. But this is not true, it doesn't return anything.
This error is gonna be hard to fix once #5 is merged so I think it's probably easier to delete this from the docs and suggest to use ScreenManager:peek() which is gonna hold the last state until the next frame anyway.

I can't remember ever using a screen returned by pop so chances are it's just a false / leftover comment.
Feel free to fix the documentation :)


Improvement/misc

- Added Get-Help for "_Get-ChiaKPlotCombination_"
- Added "upto" param to filter kplots by largest kplot size for combinations.
- Added "c" as swtich to clear screen before displaying info.
-

Please let me know if there's a standard process or a method to make it easier and/or more efficient for you.
I'd like to help where I can, but I don't want to cause you to waste time unnecessarily, so if what I contribute isn't helpful in the scope of your vision then please let me know.
Feel free to reach out anytime (phpfrog@gmail.com).

Sure thing.
The string was just to allow with or without the 'k'. I'd rather do it the
way you prefer.
I'll add the validate range and change it to 'int'.
On Thu, Jun 24, 2021 at 10:29 PM Syrius Cleveland @.***>
wrote:

@MrPig91 commented on this pull request.
In PSChiaPlotter/Public/Get-ChiaKPlotCombination.ps1
https://github.com/MrPig91/PSChiaPlotter/pull/129#discussion_r658465728:

 [CmdletBinding(DefaultParameterSetName = "DriveLetter")]

 param(
     [Parameter(ParameterSetName="FreeSpace")]
     [int64[]]$FreeSpace,
     [Parameter(ParameterSetName="DriveLetter")]

   [string[]]$DriveLetter = (Get-Volume).DriveLetter

   [string[]]$DriveLetter = (Get-Volume).DriveLetter,

   [string]$UpTo = "K35",

Can we do a validate range for this parameter? I think it will ensure
proper values get inputted.
[ValidateRange(32,35)]
[int]$UpTo = 35
If you want to keep it as a string with the letter K, perhaps we can
[ValidateSet("K32","K33"...)]
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MrPig91/PSChiaPlotter/pull/129#pullrequestreview-692439698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANNDQ7NRPDJOJPUVH6DP5LTUQA3BANCNFSM47I34UPQ
.

You're right. Stupid move on my part, sorry about that. I'll be sure to
re-add it with the fix.
On Thu, Jun 24, 2021 at 10:31 PM Syrius Cleveland @.***>
wrote:

@MrPig91 commented on this pull request.
In PSChiaPlotter/Public/Get-ChiaKPlotCombination.ps1
https://github.com/MrPig91/PSChiaPlotter/pull/129#discussion_r658466220:

 if ($PSCmdlet.ParameterSetName -eq "FreeSpace"){

     foreach ($space in $FreeSpace){
         $Max = Get-MaxKSize -TotalBytes $space

       $AllCombos = Get-OptimizedKSizePlotNumbers $Max | sort RemainingBytes

       $AllCombos = Get-OptimizedKSizePlotNumbers -MaximizedKSize $Max -KSFilter $KSFilter | Sort-Object RemainingBytes

I think we are missing the updated "Get-OptimizedKSizePlotNumbers" with
the parameter -KSFilter. I think that must be a new parameter you created,
but have not yet included in this Pull Request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MrPig91/PSChiaPlotter/pull/129#pullrequestreview-692440187,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANNDQ52GA2YRCXUNMM2DMDTUQBBTANCNFSM47I34UPQ
.


changing on-premise to on-premises

Hi @GarethBradshawMSFT, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

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

TTYL, MSBOT;

@GarethBradshawMSFT : Thanks for your contribution to the Azure documentation! The author, @GarethBradshawMSFT, has been notified to review your proposed change.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.
It�s easy to access the private repo version of your articles in GitHub. Just add �pr to the URL, as follows:
Public repo URL: https://github.com/Azure/azure-content/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Azure/azure-content-pr/blob/master/articles/azure-portal-overview.md
We've closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions.


Audio editor crashes when changing the bpm of a song while playing it.

If you try to change the boom of a song while it is playing, the editor starts to spam the notes and they don't stop until you reload the tab. It would be nice if we could easily hear how different tempos could affect our audio. If this isn't possible, having the bpm box grayed out or having it not take effect until the song finishes and repeats would be nice.

Could you post an example of this? We're having trouble recreating it. Screenshots, screen recordings, and code snippets would help.


Pass in ID to pulling joint

Makes handling network easier as you don't need to teardown the joint and re-create it.
Requires https://github.com/space-wizards/RobustToolbox/pull/1787

Something weird is going on so i'm making a new PR and merging that.


pdctl:fix some parameters of region not displaying

What problem does this PR solve?
when I used the command 'pd-ctl region check empty-region'，output like this:
 {
      "id": 5404114,
      "start_key": "7480000000000002FFE500000000000000F8",
      "end_key": "7480000000000002FFE600000000000000F8",
      "epoch": {
        "conf_ver": 961,
        "version": 10348
      },
      "peers": [
        {
          "id": 5404116,
          "store_id": 1
        },
        {
          "id": 5770409,
          "store_id": 11
        },
        {
          "id": 5772468,
          "store_id": 8
        }
      ],
      "leader": {
        "id": 5404116,
        "store_id": 1
      },
      "approximate_size": 1
    }

I wanted to see the true value of approximate_keys,written_bytes,read_bytes ,because their true value is zero.
What is changed and how it works?
remove the json 'omitempty' limit.
Check List
Tests

Integration test

Why not update WrittenKeys and ReadKeys too?

@disksing
At the beginning, I wanted to change these two parameters at the same time, but when I checked the output of our production environment, I found that the output of these two parameters was not found in our version 3.0.5. So I dare not modify these two values, because I didn't know the function of these two variables before. I will change these two values later, thank you!

@mantuliu Good to know. They are just like the other fields, only count by keys instead of sizes.

Thanks @mantuliu

/merge

Codecov Report

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

@@            Coverage Diff             @@
##           master    #2020      +/-   ##
==========================================
+ Coverage   77.41%   77.47%   +0.05%
==========================================
  Files         180      180
  Lines       18039    18039
==========================================
+ Hits        13965    13975      +10
+ Misses       3015     3011       -4
+ Partials     1059     1053       -6

Impacted Files
Coverage Δ

server/api/region.go
77.25% <ø> (ø)
:arrow_up:

pkg/metricutil/metricutil.go
90.62% <0%> (-9.38%)
:arrow_down:

server/region_syncer/client.go
81.57% <0%> (-3.95%)
:arrow_down:

server/tso/tso.go
77.37% <0%> (-2.92%)
:arrow_down:

pkg/etcdutil/etcdutil.go
88.4% <0%> (-2.9%)
:arrow_down:

server/schedulers/shuffle_hot_region.go
58.51% <0%> (-2.13%)
:arrow_down:

server/member/leader.go
77.55% <0%> (-2.05%)
:arrow_down:

server/server.go
82.06% <0%> (-0.59%)
:arrow_down:

server/schedule/operator_controller.go
83.57% <0%> (-0.19%)
:arrow_down:

server/cluster/cluster.go
81.17% <0%> (+0.23%)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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


Updated Container Networking topic

NOT READY TO MERGE - Just Staging so folks can review

Added new pictures to help explain topics
Removed superfluous docker CLI commands and referenced Docker networking pages
Updated different gotchas and caveats in light of recent overlay backport and new features in Windows 10 Creators Update

Other than my minor comments, really awesome updates to the docs. I wish it existed when I started working on the team.

@kallie-b  - Could you please update the FAQ section based on standard inquiries we receive? I provided some topics (albeit not phrased as a question). There's a nice TODO section for you. :-) Feel free to work out of my branch

Awesome, although with only GitHub I can't review the markdown with the embedded images.
Seems like the
<figure>
  <img src="media/windowsnetworkstack-simple.png">
</figure>

does not get interpreted by GitHub. Would be great if
![](media/windowsnetworkstack-simple.png)

would work for your documentation pipeline in the same way. Just an idea ;-)

@JMesser81 I added some "@Jason" questions to the doc itself. Aside from those questions, let me know if there's anything else you'd like added to that section. I decided to call it "Tips and Insights" because it seemed like framing the items as questions might make them harder to understand at a glance.

@JMesser81 -- I believe this is now ready to merge!


Feature request: SMS Alert when code stops running

Would be nice to receive an SMS alert when the code runs into errors that causes it to stop running. Otherwise I have to regularly check to see that it's running as it should. Don't have the knowledge to implement this myself.
Since the last hour I keep getting this error when the region is set to GBR: panic: runtime error: index out of range [0] with length 0. The error occurs occassionally, about every 5 minutes or so.

I solved the root cause of this exception, as far as I can tell all exception paths should be handled now (short of segfaults, which I don't know if I can throw during anyway). I'm going to close this in favor of cleanly handling all errors, as a side note notifications get sent before browser opening now so even in the case of a failure there you should still be notified (please see latest release for this change)


Instance method call section is overly complex.

It could at least have showed how to call methods from same component which doesn't take more than ten lines, which I consider is very common scenario. Calling a method from calling component after a promise is completed to show progress or maybe result. which are not possible by static methods.
But, It goes beyond that and shows how to call methods from different instances created later.
that section could've been simplified or at least be more elaborated.

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

ID: 49807900-0d56-e23c-1aef-d79099caeffb
Version Independent ID: feff434f-912c-0985-d575-4a23a629057d
Content: ASP.NET Core Blazor JavaScript interop
Content Source: aspnetcore/blazor/javascript-interop.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

https://github.com/aspnet/AspNetCore.Docs/issues/14879


Doesn't work on TP-Link TL-WR1043ND v2, OpenWrt 19.07.0 r10860-a3ffeb413b / LuCI openwrt-19.07 branch git-20.006.26738-35aa527

Failed to execute cbi dispatcher target for entry '/admin/network/access_control'.
The called action terminated with an exception:
/usr/lib/lua/luci/dispatcher.lua:938: module 'luci.cbi' not found:
no field package.preload['luci.cbi']
no file './luci/cbi.lua'
no file '/usr/share/lua/luci/cbi.lua'
no file '/usr/share/lua/luci/cbi/init.lua'
no file '/usr/lib/lua/luci/cbi.lua'
no file '/usr/lib/lua/luci/cbi/init.lua'
no file './luci/cbi.so'
no file '/usr/lib/lua/luci/cbi.so'
no file '/usr/lib/lua/loadall.so'
no file './luci.so'
no file '/usr/lib/lua/luci.so'
no file '/usr/lib/lua/loadall.so'
stack traceback:
[C]: in function 'require'
/usr/lib/lua/luci/dispatcher.lua:938: in function </usr/lib/lua/luci/dispatcher.lua:937>

You'll have to install luci-compat to get it working on openwrt 19.07.

You'll have to install luci-compat to get it working on openwrt 19.07.

This worked for me as well on a TP-Link Archer a7


image quality are very poor and unreadable

[Enter feedback here]
Image is not readable, please put high definition picture. the current one is of not much use

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

ID: 6c7151f8-cebf-0019-b028-b414faa1ac24
Version Independent ID: 01d4f188-aefa-5f5a-d0dc-1f4ab54ab2a4
Content: Azure Private Link - Azure SQL Database and Azure Synapse Analytics
Content Source: azure-sql/database/private-endpoint-overview.md
Service: sql-database
Sub-service: security
GitHub Login: @rohitnayakmsft
Microsoft Alias: rohitna

Hi, @nssidhu Thanks for your feedback! We will look into it and update you as appropriate.

@nssidhu Can you please share the specific image you are referring to?

it is under Azure Private Link - Azure SQL Database and Azure Synapse Analytics
|
|
|
|  |  |
|
|
|
|  |
Azure Private Link - Azure SQL Database and Azure Synapse Analytics
Overview of Private endpoint feature.
|
|
|
I have seen this kind of poor quality all across the Microsoft docs.
It is general problem.
On Friday, September 2, 2022 at 12:48:16 PM CDT, GeethaThatipatri-MSFT<EMAIL_ADDRESS>wrote:

@nssidhu Can you please share the specific image you are referring to?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @nssidhu , thanks so much for reaching out and letting us know! I've processed an update to fix the images, it should be live in the next few weeks. I'll go ahead and close out this Git Issue but please feel free to comment should you need anything further.
Thanks again and I hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#reassign:MashaMSFT
#please-close


fix(default-mock): supports an array with declarations #568

closes #568

v12.0.1 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Planned features

Correct answer management

[x] Pinyin tone variants can be warned
[x] Mandatory (required) Pinyin readings
[ ] Auto Repeat-to-Wrong => deduct to Wrong from double misses

Review/Leech management

[x] Some news after some old reviews => 10 after 40, then 0/0
(?) Daily limit (No need, imo)

UI

[x] Review history
[x] Open selected word
[x] Open selected Hanzi (parts of a word)
[ ] No console
[ ] Bundle QtWebEngine and remove pywebview

Dictionaries

[x] Bundled dictionaries
[ ] Update button to download new versions of dictionaries
Study by Hanzi and Hanzi dict (for individual Hanzi not in CC-CEDICT.)

QtWebEngine doesn't provide TTS engines (for speech synthesis). Perhaps Electron, nw.js or other alternatives may have.


Create-AlProjectFolderFromBcContainer ParameterBindingException

Hi,
I am using version <IP_ADDRESS> and running this command:
Create-AlProjectFolderFromBcContainer -alProjectFolder $alProjectFolder -containerName $bcW1LatestContainerName -useBaseLine -AddGIT
with variables for parameters are set like this:
$bcW1LatestContainerName = "zm-bc-w1-latest"
$alProjectFolder = "C:\ProgramData\NavContainerHelper\AL\zm-bc-w1-latest"

gives me the following error:
Create-AlProjectFolderFromNavContainer : Der Parametersatz kann mit den angegebenen benannten Parametern nicht aufgelöst werden.
In Zeile:1 Zeichen:1
+ Create-AlProjectFolderFromBcContainer -alProjectFolder $alProjectFold ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Create-AlProjectFolderFromNavContainer], ParameterBindingException
    + FullyQualifiedErrorId : AmbiguousParameterSet,Create-AlProjectFolderFromNavContainer

It seems to me that the Parameter list is an issue somehow but I cannot figure out what/why.
I haven't had this issue before when creating AL folder and this exact command.
What am I missing?
Thanks,
Zoran M.

The AL project created needs an id, name, publisher and version.
First version of the function created a random ID and set some default values if you didn't provide them.
Second version of the function supported to use the existing BaseApp properties as id, name, publisher and version as default values, which kind of conflicts with the original.
So, you need to specify what you want.
add -useBaseAppProperties to use existing baseapp properties
add -useBaseAppProperties:$false to use random guid, etc.
add -id, -name, -publisher and -version to add your own properties.

Works!
Thanks Freddy. :)
Zoran M.


BackgroundColor

I do something like this:
# setup for queryButton
var queryButton = newButton(" Run Query ")
queryButton.fontSize = 18
queryButton.textColor = rgb(30,144,255)     # color ok
queryButton.backGroundcolor = rgb(0,255,0)  # no effect ?
rightMenuContainer.add(queryButton)

textColor method  works for button and labels only
other controls eg textBox does not show any text or background color changes at all.
PS.: nigui is really nice  and hopefully there will be a comboBox widget
I am using it to display output from sql queries.
Here running under openSuse Tumbleweed .

As far as I know, the underlying platforms Gtk and WinAPI do not allow to change the background color of a button (resp. it has no effect). This is something which should be mentioned in the upcoming NiGui documentation.
If you want to have a button with a specific background color, you can create a custom control based on the type Button. Here an example implementation of a custom button. The main program has to use newToolbarButton() instead of newButton().
type ToolbarButton* = ref object of Button

proc newToolbarButton*(text = ""): ToolbarButton =
  result = new ToolbarButton
  result.init()
  result.text = text

method handleDrawEvent*(control: ToolbarButton, event: DrawEvent) =
  let canvas = event.control.canvas
  canvas.areaColor = control.backgroundColor
  canvas.textColor = control.textColor
  canvas.lineColor = control.textColor
  canvas.drawRectArea(0, 0, control.width, control.height)
  canvas.drawTextCentered(control.text)
  canvas.drawRectOutline(0, 0, control.width, control.height)

Thanks for your feedback :)

Well in Gtk you can do this: https://stackoverflow.com/questions/1706550/gtk-modifying-background-color-of-gtkbutton, and for WinAPI you have this: http://winprog.org/tutorial/dlgfaq.html

Is this issue still up to date?

Yes. Same issue than #152.


error when uploading files to chat

Installation Method = DOCKER
[Describe the method you used to install the project, e.g., git clone, Docker, pip, etc.]
Environment = Open WebUI Versão v0.4.8

Utilizo a API embedding da OpenAI
Operating System:  Ubuntu
Browser (if applicable): Chrome

Confirmation:

[x ] I have read and followed all the instructions provided in the README.md.
[x ] I am on the latest version of both Open WebUI and Ollama.
[x ] I have included the browser console logs.
[x ] I have included the Docker container logs.
[ x] I have provided the exact steps to reproduce the bug in the "Steps to Reproduce" section below.

Expected Behavior:
The behavior I would like is to be able to attach any PDF and be able to interact with it without any problems. Plus, you can attach PDFs as a knowledge base without error.
Actual Behavior:
[Describe what actually happened.]
Description
Bug Summary:
[Provide a brief but clear summary of the bug]
Reproduction Details
Steps to Reproduce:
When attaching a PDF file with more than 20 pages to interact with, I get the following error:
"unable to reshape array of size 20101 to format (763,1493,newaxis)"
And, due to the error I can't interact with the PDF with more than 20 pages..
Still, if I try to use the file as a knowledge base it also gives the error.
Logs and Screenshots

Additional Information
[Include any additional details that may help in understanding and reproducing the issue. This could include specific configurations, error messages, or anything else relevant to the bug.]
Note
If the bug report is incomplete or does not follow the provided instructions, it may not be addressed. Please ensure that you have followed the steps outlined in the README.md and troubleshooting.md documents, and provide all necessary information for us to reproduce and address the issue. Thank you!

Discovering that the error is caused because the PDF has images and the Extract Images (OCR) option is active, when deactivating the option the PDF is processed without any error, however no images are considered, which is not the expected behavior.


JsExecutor - Use FElement instead of Webelement

Summary
If we wanna use the JsExeutor we always have to call it with FElement.getWebelement()
It should be FElement only.
Motivation
Stop mixture of FElement and Webelement. Use FElement!

Switch to playwright.


Include jQuery in the footer

Currently, jQuery is being included in the head. This means it is render blocking.

I recall this created some load order problem.. can't remember what it was specifically but I know this wont be a one liner.

Yeah, I believe it did too. Fun stuff.

I think there are some scripts that use jQuery that we dynamically generate in the middle of the page as well.


updated to AAP 2.1 and removed prerelease

Updated to AAP2.1 also changed naming to Controller instead of Tower to match new naming conventions.
fixes #92

Hey @chadmf , merging this PR for now, but it would be best to remove anything that's environment specific from this central repository, like the console link (hard-coded to crc url).

thanks @pittar i actually was just changing that from the previous one that looked like a customer one. I can PR to remove it.


Test Advanced A-tag Allignment

🚀 Issue: Test AprilTag Assisted Drive Code on Robot (file name :ClaudeAiHelpedNewTry.java) // don't ask me why
📝 Description
We have a new TeleOp program integrating AprilTag detection for assisted driving and auto-alignment. We need to test this code on the robot to ensure it works as expected.
🌟 Key Features:

AprilTag Detection:
Manual and Auto-Align Modes: Switches between modes using options button on gamepad
PID Control for Alignment: Aligns the robot with AprilTags using PID control.
Real-time Telemetry: Provides feedback on the robot's state and alignment process.

✅ Task List

Deploy Code: Load the new code onto the robot.
Initial Setup: Ensure the webcam and hardware are configured correctly.
Manual Drive Test: Verify manual driving with gamepad controls.
AprilTag Detection Test: Check if the robot detects AprilTags and enters auto-align mode.
Auto-Align Test: Ensure the robot aligns correctly with the detected AprilTag.
Telemetry Check: Confirm that telemetry data is accurate.
Exit Conditions: Test manual and automatic exits from auto-align mode.

Didn't work, probably won't fix. Good concept tho.


Rename "Can't Tell" to "Unknown"

Earlier versions of the product spec used the term "Can't Tell" to describe
species that the user can't identify at observation time, but now it's using the
term "Unknown." Update the data model and API to match the official terminology.
This is a breaking API change; clients that submit observation data will need to
replace CantTell with Unknown in the species certainty payload field.

Current dependencies on/for this PR:

main

PR #1188

PR #1189   👈

This comment was auto-generated by Graphite.

@sgrimm started a stack merge that includes this pull request via Graphite.

Graphite rebased this pull request as part of a merge.

@sgrimm merged this pull request with Graphite.


Missing feature to pass Biome original flags inline or by config

First, big thanks for creating such a plugin. :)
the missing feature is that, there's no option to pass a additional arguments/flags for Biome as inline NX args or by the config, such as:
--unsafe
--write and others.
Or it's just a missing plugin docs point?
NX does allow to pass an arguments
like
nx run lib123:biome-lint -- --unsafe
but doesn't seem that biome command does consume it.
Is it going to be maintained further?

Hello @devzom
In the successive versions I will start adding the possibility to add configurations in the plugin that allow the execution of the library with those flags.
Any contribution is welcome! Thank you!

Hello @devzom
I have added the possibility to linte the files with --write and --unsafe through the exectuor configuration. These changes are in version 1.5.0 of the plugin.

@mcfdez wow, that's pretty fast! Thanks, great info, I'll implement it in project soon :)


gRPC for Relayers to query which channels are incentivized

Since channelID/portID does not include fee information, we need a rpc that tells relayers whether or not a given channel is incentivized

I think we should possibly include two queries here. One to return all fee enabled channels and one to return whether or not a channel is fee enabled given its channel/port ID.


F/babylon optimizer

As simple as that. Let's create the docker hub user, and adjust and push the image.

Created in haste.


let = typing assist doesn’t have any effect

In case it’s helpful, I’m using VS Code version 1.48.2 and rust-analyzer version 2020-08-31.

I’m not sure if I’ve understood the part of the manual on the subject correctly, but typing let = in front of an existing expression is meant to insert a semicolon. For me, it doesn’t. Here’s a video showing what I mean.

Random guess -- do you use a vim extension or something similar?

Nope. If you’d like I can try disabling all my extensions and config to see if the issue is with rust-analyzer.

Do you have "editor.formatOnType": true, in the settings?

No. Is that setting required for typing assists to work?

Update: I have tried enabling editor.formatOnType, and typing assists work now. Do you think this would be worth mentioning in the docs?

Yeah, most of the assists are implemented as the onTypeFormatting

Yup, PR to the docs would help. I think the source for docs is the comment
in typing.rs
On Tue, 1 Sep 2020 at 15:26, Aramis Razzaghipour<EMAIL_ADDRESS>wrote:

Update: I have tried enabling editor.formatOnType, and typing assists
work now. Do you think this would be worth mentioning in the docs?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rust-analyzer/rust-analyzer/issues/5925#issuecomment-684851651,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANB3M4QIH42RVF7O2VJFGTSDTY7ZANCNFSM4QRU6O7A
.


The AudioContext was not allowed to start

Hi,
I have again an issue with playing audio in the Browser Link to the game.
According to the browser console:

It seems like calling Music.play() fails if it is called at the application startup? If I control the player (WASD, SPACE) then e.g. I can here the sound for the jump effect and also the jingle when I get in contact with the finish flag.
But the normal background music of the map is not played.
Can I somehow force a gesture to also play music immediately without an interaction of the user first? Or is there another solution to this issue?
edit: by the way, I use Edge as a browser.

I dont think there is a way to force a gesture. What they recommend is to add a play button at start of the game to be the first interaction .
https://gamedev.stackexchange.com/questions/163365/html5-games-play-failed-because-the-user-didnt-interact-with-the-document-fi
https://stackoverflow.com/questions/64929753/play-failed-because-the-user-didnt-interact-with-the-document-first-issue
https://developer.mozilla.org/en-US/docs/Web/Media/Autoplay_guide

Thanks for responding. I actually did a similar solution now. After preloading some resources I show a text to the user "click to continue" and only after that touch event, I go to the real game and from there music playing is working.
Thank you xpenatan :)


Allow scripting variables with Invoke-Sqlcmd2 and Invoke-SqlDbaCmd

Problem to solve
Scripting variables allow a script to be run in different environments and change database objects in SQL scripts.
Additional information
Example sql script with scripting variables.
select   max($(col1)) as $(col1)
from     $(db).$(schema).$(table)
where    $(col2) = $(value2)
Example execution of sql script with sqlcmd utility:
sqlcmd  -S foo -d bar -i "c:\temp\test.sql" -v col1=CREATE_DATE db=MyDB schema=MySchema table=MyTable col2=MyColumn value2=1234
Example execution using Invoke-Sqlcmd:
$SqlVariables = 'col1=DateColumn','db=MyDB','schema=MySchema','table=MyTable','COL2=MyColumn','value2=1234';
Invoke-SqlCmd -ServerInstance foo -Database bar -InputFile "c:\temp\test.sql" -Variable $SqlVariables;
See documentation at
Invoke-Sqlcmd
sqlcmd utilty
scripting variables

scripting variables are not quoted. They're just replaced "as is" within the query.
There's a set of things Invoke-DbaSqlQuery won't ever do (and that's one of the main reason is aptly named NOT after sqlcmd), and this is one of them.
Having the ability to run a script that connects to multiple databases without you noticing is another one.
To sum up, parsing the whole shenanigans of sqlcmd is (and probably always be) out of scope.
For this particular usecase, you'd be just as good using interpolation within Powershell to build the query upfront.


Add script tags for Unicode 16

The following scripts are being added in Unicode 16, but don't yet have tags in the OpenType Layout script tag registry:

Script
Unicode / ISO 15924 ID
proposed tag

Garay
Gara
gara

Gurung Khema
Gukh
gukh

Kirat Rai
Krai
krai

Ol Onal
Onao
onao

Sunuwar
Sunu
sunu

Todhri
Todr
todr

Tulu-Tigalari
Tutg
tutg

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

ID: 49c1efc4-e864-0727-9663-7cec0ec4d9f1
Version Independent ID: 36ec4396-b9cb-cba9-1582-d8400296d249
Content: Script tags (OpenType 1.9 delta) - Typography
Content Source: typographydocs/opentype/spec/scripttags.md
Product: typography

See the OT 1.9.1 alpha for draft revisions addressing this issue.

Looks good – thank you!

This is fixed in OT 1.9.1. See the OT 1.9.1 beta for draft revisions addressing this issue. Closing.


列表删除bug

Laravel Version: #.#.#
PHP Version:
Laravel-admin: v1.7.8

Description:
列表删除有bug。
在form删除回调deleting已经阻止了删除了，批量删除是被阻止了，但是grid列表的右侧的那个删除，还是可以删除数据，
Steps To Reproduce:

Controller must have public function form to enabling delete batch

Controller must have public function form to enabling delete batch

The batch that is deleted now is no problem. When the deletion is blocked, the batch deletion can be blocked normally. The problem is that the right side of the list item is deleted individually and is not blocked. This is a place I don't understand.

solved. Find the root of the problem. This is the link:
权限控制
The general meaning is that the deletion operation is not defined by the form, and additional permission logic is needed inside the model. As for other usages, still exploring


Question of edit_func

Hello, thanks for providing such excellent open source projects. I encountered several puzzles when reading the code. Thanks for your help:

in editor.py, the line of code edited _ model, weights _ copy, ICL _ examples = edit _ func (request). I am confused about the return values of this: edited_model, weights_copy, where edited_model is the overall weight of the model after an edit? So every time you go through the for loop, you re-edit the original model? Or do you always add new edits to the edited model? Is weights_copy the basic model parameter of the original GPT or llama corresponding to the modified part?

Are sequential _ edit and keep_original_weight the same? I didn't find the keep_original_weight parameter used in apply_rome_to_model and apply _ mememit _ to _ model.

I don't understand the relationship between edit_func and edit_evaluation. My understanding is that edit _ func uses a prompt in requests to edit model parameters every time, and then uses edit_evaluation to verify indicators in the whole requests?

I can understand that every time you edit a prompt in requests through edit_func, the parameters of the model will be modified correspondingly. Do these edits not affect each other? Is it true that by saving metrics, edited_model, _ = editor. edit (...) in run_knowedit_llama2.py, the returned edited _ model is the model of n samples in the current data set?  Thank you for your answer.

I am a novice in knowledge editing. Please forgive me.

Yes,  edited_model is the overall weight of the model after an edit. No, your statement was not entirely accurate. In the setting of sequential editing, the model returned after each edit becomes the object for the next edit, meaning that the model is updated progressively. However, in the case of singleton editing, the object being edited is always the original model, and each edit operates on the original state. Yes, the edit_copy parameter stores the original parameters of the model's modified parts, allowing the original model to be restored after the edits.
No, these two parameters are completely different. Sequential_edit refers to a specific editing task setting, while keep_original_weight is a boolean parameter that indicates whether to restore the original parameters after an edit. However, as you mentioned, this parameter is essentially no longer in use, which is why you didn't find it in rome and memit. This is perfectly normal.
The function edit_func is responsible for performing the editing operation, while edit_evaluation evaluates the model before and after the edit, such as assessing the accuracy of the edit, as well as the model's portability, locality, and other related metrics.
I'm not quite clear what you mean the edits influencing each other. In fact, If the task setting is single editing, then these edits will not affect each other because the model reverts to its original weights after each edit; whereas if it's set to sequential editing, these edits will continuously alter the model's parameters. Regarding your last question, I suspect you might be asking whether the edited_model obtained from run_knowedit_llama2.py is the model after n edits. If sequential editing is configured, then what you get is indeed the model after n edits. However, if it is not set this way, then this edited_model would be the original model.

hi, do you have any further issues?


Add subpage for decklist card transition

When a new meta is starting a player is interested in the card transition of his leader.
He wants to know the top cards which are newly added in the decklists and the top cards which left.
TODOs:

Find a threshold, where card occurence change is seen as removal
Find a threshold, where card occurence change is seen as newly added

AK:

new page was created
visually appealing

Created subpage in this commit: 8ad3f36e4a2cb114e3eb5641f003160b973164fd


[Market Price] Refactor the query functionality in a functional style

Now a lot of inefficiencies exist due to collecting data eagerly.
Tasks:
Get rid of intermediate collections. Instead, work with Iterator-s.
Remove duplicated code in alarms::price::PriceAlarms fn query  and fn nolify

moved to money-market


The module creates a default security group that allows blanket access

The module creates a security group allowing all traffic from everywhere. Not sure that's a safe default.
There should at least be a warning about this behavior (I don't think its enough to just mention this in the allowed_cidr_blocks bullet point.

We left the default open because we couldn't predict what ports someones tasks might require. For example, a task that launches on an ephemeral port and registers to an alb.  My reading of the ECS docs https://docs.aws.amazon.com/AmazonECS/latest/developerguide/get-set-up-for-amazon-ecs.html#create-a-base-security-group  leaves me thinking that the SG on the instance would need to know what ports the tasks might need.   Also in part because of the way the SG is implanted,  because I used ingress and egress blocks rather then the TF Security Group Rule resource, refactoring this will be a breaking change. 😢   I've added a refactor to our backlog, and we could add a default of something like "open to only my VPC"  but in general I think I prefer providing an interface where the end user can apply appropriate rules rather then predict what those might be..
However, I'd be totally happy if you wanted to update the README to call that our a bit more! That would be fantastic!

Can we modify the security group to not have any rules and expose the security group so users can attach security group rules?


Same song or alternating songs sometimes spam the "Now Playing" message

Certain songs will spam their "Now Playing" message over and over, without any noticeable song transition appearing to the user.
This often happens with "special" songs playing over zone music (eg, moonfire faire festival music in its small zone in costa del sol), but is also seen in some places like Quarrymill.
There is another version of this issue where 2 songs will "fight", with the message spamming back and forth between the two constantly, even though the user perceives no change.  This can be seen in places like the twin adder gc counter.

Maybe addressed (poorly) in 1a01f37
Leaving open for testing.

Seems to work decently for both 'regular' music transitions, as well as preventing all the fighting and duplicate song spam.


[FLOC-1065] Deb repositories

https://clusterhq.atlassian.net/browse/FLOC-1065

Fix the merge conflicts.

Thanks @tomprince , I don't want to do a more thorough review until the tests pass and the potential re-arranging I suggested happens. Please re-submit when the comments are addressed.

Was https://github.com/ClusterHQ/flocker/pull/1287#discussion_r28410000 lost as an outdated comment or do you think it isn't worth worrying about? This seems like an ideal use of delayed assertions.

Thanks @tomprince , please address the comments and resubmit when the dpkg-scanpackages problem is solved.

The dev-box is failing because the zfs epel-testing repository is currently broken. I've asked @ryao to look into it.

Ok thanks @tomprince . Merge when you're happy and Buildbot is to your satisfaction. Create follow-up issues for:

The confusion of having yum names for deb logic (and preferably start working on it as a follow-up)
Switching to an upstream DPKG package when available

and link to those issues from the code. The best live testing will be in https://clusterhq.atlassian.net/browse/FLOC-1103 and as this doesn't affect anything (I hope) until that is done I'm happy for this to be merged.

The epel-testing problem? Upstream?

Modify the outcomes section and the step which mentions RPMs in the release process.

Add a section in packaging.rst similar to the current Building Omnibus RPMs.

Change the Package Hosting section in packaging.rst to describe the Ubuntu packaging. Sorry, because I've had a few thoughts on what else needs to be done please re-submit when addressed.

Thanks Tom, please address the minor comments and merge when you are happy and satisfied that there are no build failures introduced. I can't keep in my head a state of what is and what is not OK to fail.


adding DUST_AMOUNT error to errorSpec

Coverage remained the same at 93.656% when pulling de76da5d3d495d2da38dfa93d2a921c7baf2cac1 on gabrielbazan7:feat/addError into 315fc7f9e1d9878dc68837498b2617ea2dcc8abd on bitpay:master.


Add more test

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 87.163%

Totals

Change from base Build<PHONE_NUMBER>:
1.5%

Covered Lines:
2003

Relevant Lines:
2298

💛  - Coveralls


How about add DynMoE to the list?

Paper link: https://arxiv.org/abs/2405.14297
Code link: https://github.com/LINs-lab/DynMoE

Hi @QAQdev, thanks for sharing this paper - I think this would be a great one to add to the list! It's super interesting work!
Do you have a short description of your paper or key points that you think would be important here to include in the description?
My initial understanding is that the approach allows us to determine the hyperparameters for experts activated and number of experts automatically and so reduces the time and compute needed for h-param sweeps. It also has the benefit of slightly increasing performance on downstream tasks (is this FLOP-matched? Seems like maybe not?). The areas for future work at this stage look to be optimising for the hardware (currently GPU memory is taken up by rarely used experts and there seems to be work on load balancing to increase throughput). Does this feel accurate?
Again, thanks so much for being part of and sharing this work - I really enjoyed reading it and it seems likely to become a new default for the MoE paradigm. Also great job releasing DeepSpeed-MoE flavoured code!

Sorry for a late reply. I think your understanding is precise actually! As for the description, I think you can use your own language since you have capture the key points :)


Reliable updating from local

Method that reliably works for updating Macrosynergy package from local folder, independent of pip version

Assumed to be fixed by #785. Will keep the issue alive till confirmed

To be solved as part of https://github.com/macrosynergy/macrosynergy/issues/929


Add TravisCI integration

Also, for general project improvement, this PR adds the "engines" section in the package.
Closes #21.

@italopessoa One by one:

Is there no need to tell travis to install packages npm install [...]?

You can do so, or you can leave it for the Travis default install command.

Is there no need to tell travis to [...] build the npm build project?

As far as I can tell, the default test command of create-react-app (which is what we have) doesn't need anything but its declaration in the package file.

I'm assuming that npm test will build the code before the test, but is there any chance that the tests pass even if the build fails?

Only if we set the test command so wrong that it doesn't return a non-zero code from the process if the test fails, but out test frameworks abstract this job from us.
Additionally, just realized we can leave the script: npm test out of .travis.yml aswell (see here).

I'll clone this pull request to another one since I ~am stupid and~ was doing it from a fork, which made something break in the coverage configuration when it starts from scratch. ¯_(ツ)_/¯
See you in the next PR.


Getting test setup on the live tests pipeline for Blobs

We recently encountered test setup failures in our live tests pipeline while attempting to create required resources.
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=900632&view=ms.vss-test-web.build-test-results-tab&runId=19377790&resultId=100815&paneView=debug

@benbp have you seen an error like this before?

@scbedd this issue is from last May, do we still even provision storage resources via the test preparer anymore? My guess would be bad test parallelization or resource scoping.

It's also possible the group was tagged improperly, as was a frequent issue with the old python preparers, and it was deleted during a cleanup sweep.


Exception: Cannot read properties of undefined (reading 'toLocaleUpperCase')

Describe the bug
I am receiving an error on one specific entity. It is a battery sensor coming from the ambientweather2mqtt custom add-on
How to reproduce
Here is the sensor in question. It looks no different than any other battery sensor:

Expected behavior
It should pick up the battery like it was any other
YAML configuration
title: Sensors
tap_action: more-info
view_layout:
  column: 2
charging_state:
  attribute:
    - name: Battery State
      value: Charging
    - name: '*is_charging'
      value: true
    - name: '*battery_charging'
      value: true
  state:
    - charging
    - Charging
    - 'On'
    - 'on'
state_map:
  - from: 'off'
    to: 100
  - from: 'on'
    to: 10
filter:
  include:
    - name: entity_id
      value: '*_battery_level'
    - name: attributes.device_class
      value: battery
  exclude:
    - name: entity_id
      value: group*
    - name: entity_id
      value: binary_sensor.*low_battery_level*
    - name: entity_id
      value: sensor.*pixel_*
    - name: entity_id
      value: sensor.*lock*
    - name: entity_id
      value: binary_sensor.*_co2_battery
    - name: entity_id
      value: binary_sensor.*replace*(now|soon)*
    - name: entity_id
      value: sensor.bathroom_pothos_battery
    - name: entity_id
      value: sensor.african_violet_battery
    - name: entity_id
      value: sensor.plumosa_fern_battery
    - name: entity_id
      value: sensor.rabbits_foot_fern_battery
    - name: entity_id
      value: sensor.orchid_blue_pot_battery
    - name: entity_id
      value: sensor.orchid_striped_pot_battery
    - name: entity_id
      value: sensor.malfunctioning_eddie_battery
    - name: entity_id
      value: sensor.hedonismbot_battery
bulk_rename:
  - from: Level
    to: ''
  - from: Battery
    to: ''
  - from: battery
    to: ''
  - from: Upstairs Low
    to: Upstairs Temperature

Dev console errors
TypeError: Cannot read properties of undefined (reading 'toLocaleUpperCase')
    at http://homeassistant.local:8123/hacsfiles/battery-state-card/battery-state-card.js?hacstag=256292682316:35:9596
    at Vt.<anonymous> (http://homeassistant.local:8123/hacsfiles/battery-state-card/battery-state-card.js?hacstag=256292682316:35:9635)
    at Generator.next (<anonymous>)
    at http://homeassistant.local:8123/hacsfiles/battery-state-card/battery-state-card.js?hacstag=256292682316:1:633
    at new Promise (<anonymous>)
    at e (http://homeassistant.local:8123/hacsfiles/battery-state-card/battery-state-card.js?hacstag=256292682316:1:378)
    at Vt.internalUpdate (http://homeassistant.local:8123/hacsfiles/battery-state-card/battery-state-card.js?hacstag=256292682316:35:9031)
    at Vt.<anonymous> (http://homeassistant.local:8123/hacsfiles/battery-state-card/battery-state-card.js?hacstag=256292682316:35:1486)
    at Generator.next (<anonymous>)
    at http://homeassistant.local:8123/hacsfiles/battery-state-card/battery-state-card.js?hacstag=256292682316:1:633

Screenshots

Version
3.1.6

I have this error as well, three in fact. How did you determine it was your Ecowitt device causing the problem? I have an Ecowitt also. I don't see anything obvious in the error message tying the failure to a certain battery/device. Thanks.

I don't have an Ecowitt. Mine is from an Ambient weather station. The only real reason I knew it was that was that it happened right after some changes were made to it. I confirmed by adding it to the exclude list and the error went away.

Sorry, I read too fast and blurred Ambient and Ecowitt together since I have several Ambient weather sensors all reporting in via an Ecowitt GW1100. As for which entity is giving me trouble, I answered my question. If I click on the pink error box it shows me the device and then the Gear icon will reveal the entity name. In my case, it is failing on three Oregon thermostats (not my Ambient or Ecowitt stuff) that have all stopped sending data for past two weeks, therefore HA thinks they are Unknown.

I just double checked and mine is definitely not unknown. It's been getting regular updates

Weird. The rtldiscovery service on the Rpi hosting my RTL-SDR had stopped. Once I restarted it and MQTT discovery ran, I began getting temperature samples again. However, I noticed both the "Battery" and "Battery+" entity friendly names for these three devices did NOT start with the device name. Once I manually added the device to the beginning of the name, things got better with the Battery State Card. E.g. "Battery+" -> "Temperature Garage Battery+"
I have no idea why these entities were misnamed. Perhaps something to do with the recent changes where HA likes device entity names to all begin with the device name...

I confirm, exactly the same problem.

Ok I know what is the problem.
It throws exception because in your rename rules you have one to replace "Battery" with an empty string. There is a line in the card code which tries to capitalize first letter of the name. The first letter doesn't exists as the string is an empty string - because "Battery" was the actual entity name.
While this is a bug in the card which should be fixed you should consider as well changing the rules or entity name as probably it's not your intention to display entity without the name.

Fixed in v3.2.0


Forwarding ingress traffic to external resources (Azure storage / CDN)

Bug description
Currently, we are using istio's ingress gateway as the public endpoint for many of our microservices. Furthermore, we are using its reverse proxy capabilities to redirect traffic based on URI format to our web frontend or various AKS-hosted backends.
What I am trying to achieve at the moment is splitting 'regular' web requests (i.e. traffic destined for dynamic web pages) from static content requests (CSS, images, etc.). My current setup uses prefix-based URI matching to identify requests containing the subpath '/static', rewrite the subpath to the name of the CDN endpoint that the static content is hosted on, then finally forward the traffic to the Azure CDN where the said static content is hosted. For example:
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: cdn-ch
spec:
  hosts:
  - "ch-portal-test.example.com"
  gateways:
  - example-ch-app-gateway
  http:
  - match:
    - uri:
        prefix: /static
    rewrite:
      uri: "/test"
    route:
    - destination:
        host: cdn-ch.azureedge.net
    timeout: 40s
    retries:
      attempts: 5
      perTryTimeout: 3s
      retryOn: gateway-error,connect-failure,refused-stream

Furthermore, I added a Service Entry and Destination Rule for the Azure CDN, since it is an external resource:
apiVersion: networking.istio.io/v1alpha3
kind: ServiceEntry
metadata:
  name: rwe-cdn-test
spec:
  hosts:
  - "cdn-ch.azureedge.net"
  ports:
  - number: 80
    name: http-port
    protocol: HTTP
  - number: 443
    name: http-port-for-tls-origination
    protocol: HTTP
  resolution: DNS
---
apiVersion: networking.istio.io/v1alpha3
kind: DestinationRule
metadata:
  name: rwe-cdn-test
spec:
  host: cdn-ch.azureedge.net
  trafficPolicy:
    loadBalancer:
      simple: ROUND_ROBIN
    portLevelSettings:
    - port:
        number: 443
      tls:
        mode: SIMPLE

Now, while Istio does perform the basic operations described in the VS config (i.e. rewrites the request URI from /static/whatever.css to /test/whatever.css and forwards the traffic to the CDN), every test that I have ran so far yields a 404 not found despite the requested file being present in the said path (accessing the CDN URL directly downloads the test style sheet)
My first instinct was raising a support ticket with Microsoft, suspecting a possible limitation their Azure CDN / Storage Blob product. However, after a week of back-and-forth emails and log analysis, their conclusion has been that it is possible that Istio could strip away some of the request headers, resulting in the said 404.
And since I couldn't find any solid reference to header manipulation in the documentation, I thought I might move my inquiry here - did anyone attept something similar until now? does any sort of header manipulation / stripping occur when a request gets processed by Istio? I am happy to provide logs / packet captures of the forwarded requests if it helps with the troubleshooting process.
Expected behavior
Requests should be forwarded successfully, since the rewrite is correct and the requested file exists in the specified path.
Steps to reproduce the bug
Apply the manifests specified above, replacing the URLs with a different CDN / online storage service (I reckon you could use anything with a static URL)
Version (include the output of istioctl version --remote and kubectl version)
Client: 1.2.4
Server: 1.2.4
How was Istio installed?
Via a helm chart
Environment where bug was observed (cloud vendor, OS, etc)
Azure - AKS

Weirdly enough, running a tcpdump on the ingress gw's proxy container shows a 200 response for the same request that yielded a 404 in the browser:
HTTP/1.1 200 OK
date: Thu, 14 Nov 2019 09:07:37 GMT
content-type: text/plain; charset=utf-8
server: istio-envoy
x-envoy-upstream-service-time: 11
x-envoy-upstream-healthchecked-cluster: singleadpage-ch.freki
transfer-encoding: chunked
Host: ch-portal-test.example.com
Connection: keep-alive
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.97 Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3
Accept-Encoding: gzip, deflate
Accept-Language: en-US,en;q=0.9
Cookie: rub_NewAdsFirstVisit=09/26/2019 11:11:50; tsa=58fe1431f9d
host: ch-portal-test.example.com
upgrade-insecure-requests: 1
user-agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.97 Safari/537.36
accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3
accept-encoding: gzip, deflate
accept-language: en-US,en;q=0.9
cookie: rub_NewAdsFirstVisit=09/26/2019 11:11:50; tsa=58fe1431f9d
x-forwarded-for: <IP_ADDRESS>
x-forwarded-proto: http
x-envoy-external-address: <IP_ADDRESS>
x-request-id: 5251bd26-2b1b-4398-b8ca-4573f1fe5774
x-envoy-decorator-operation: cdn-ch.azureedge.net:80/static*
x-istio-attributes: CkIKGGRlc3RpbmF0aW9uLnNlcnZpY2UubmFtZRImEiRhdXRvY29tcGxldGUtd2lkZ2V0LWRlLmF6dXJlZWRnZS5uZXQKKAodZGVzdGluYXRpb24uc2VydmljZS5uYW1lc3BhY2USBxIFZnJla2kKTwoKc291cmNlLnVpZBJBEj9rdWJlcm5ldGVzOi8vaXN0aW8taW5ncmVzc2dhdGV3YXktNmZmN2ZjYzVmNC1qcjZsdy5pc3Rpby1zeXN0ZW0KQgoYZGVzdGluYXRpb24uc2VydmljZS5ob3N0EiYSJGF1dG9jb21wbGV0ZS13aWRnZXQtZGUuYXp1cmVlZGdlLm5ldA==
x-envoy-expected-rq-timeout-ms: 3000
x-b3-traceid: 0c152fe59272a76ddbf7cecf30e59f4c
x-b3-spanid: dbf7cecf30e59f4c
x-b3-sampled: 0
x-envoy-original-path: /static/header-footer/Test/generated/FooterResultPage.html
content-length: 0
Content-Type: text/html
Date: Thu, 14 Nov 2019 09:07:47 GMT
Server: ECAcc (ama/8B2F)
Content-Length: 345
content-type: text/html
date: Thu, 14 Nov 2019 09:07:47 GMT
server: istio-envoy
content-length: 345
x-envoy-upstream-service-time: 7
Host: ch-portal-test.example.com
Connection: keep-alive
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.97 Safari/537.36
Accept: image/webp,image/apng,image/*,*/*;q=0.8
Referer: http://ch-portal-test.example.com/static/header-footer/Test/generated/FooterResultPage.html
Accept-Encoding: gzip, deflate
Accept-Language: en-US,en;q=0.9
Cookie: rub_NewAdsFirstVisit=09/26/2019 11:11:50; tsa=58fe1431f9d

Turns out the issue was the 'Authority' request header, apparently the CDN rejected calls with any other value than its hostname. As illustrated in the packet capture above, the request header was set to the original hostname accessed through the browser ('ch-portal-test.example.com'), while the CDN expected its own hostname ('cdn-ch.azureedge.net').
As a test, I set the gateway and virtual service to listen for the CDN hostname, and modified the DNS records to point the CDN's hostname to the ingress gateway's IP address. The routing worked perfectly, including the URI rewrites, so the problem lies solely with the authority header of the request.
However, the only workaround I have found so far is re-setting the header from the virtual service config, which results in a hard redirect to the CDN's URL:
  http:
  - match:
    - uri:
        prefix: /static
    rewrite:
      authority: cdn-ch.azureedge.net
      uri: "/test"
    route:
    - destination:
        host: cdn-ch.azureedge.net

This doesn't really help my case however, since a) the purpose of the setup is to seamlessly integrate the CDN-served static files into the web app, not to have the session redirected to a different URL, b) this kinda messes up the SEO setup, since a good portion of the requests (basically anything associated with css or images) will be registered with a different URL than the main web app.
I also tried employing an Envoy filter with some LUA code, however the result was the same, i.e. a hard redirect:
apiVersion: networking.istio.io/v1alpha3
kind: EnvoyFilter
metadata:
  name: edge-lua-filter
spec:
  workloadLabels:
    app: example-ch-app-gateway
  filters:
  - listenerMatch:
      listenerType: GATEWAY
    filterName: envoy.lua
    filterType: HTTP
    filterConfig:
      inlineCode: |
        function envoy_on_request(request_handle)

          local authority = request_handle:headers():get(":authority")

          if authority == "ch-portal-test.example.com" then
            request_handle:headers():replace(":authority", "cdn-ch.azureedge.net")
          end
        end

It is possible that my config could be a bit off though, this is my second ever attempt at LUA code :)
Any input on the matter would be greatly appreciated. This has been a bit of a blocker for the past week, and it would be useful to know whether this approach is worth pursuing or whether I should scrap it and employ a different method.

You can rewrite the authority in your virtualservice :
https://istio.io/docs/reference/config/networking/virtual-service/#HTTPRewrite
I should at least save you the trouble of envoyfilters + lua (which is good because envoyfilters are changing a lot in 1.3)

However, the only workaround I have found so far is re-setting the header from the virtual service config, which results in a hard redirect to the CDN's URL

I don't understand why the rewrite in the virtualservice would trigger a "hard redirect".
Are you sayin you get a 301 from the istio ingress-gateway ?

However, the only workaround I have found so far is re-setting the header from the virtual service config, which results in a hard redirect to the CDN's URL

I don't understand why the rewrite in the virtualservice would trigger a "hard redirect".
Are you sayin you get a 301 from the istio ingress-gateway ?

Yep, that's what happened initially at least, I would get a 301 and had my CDN-served content opened in a separate tab / window.
However, I managed to find a working version of the config with a few tweaks here and there, including rewriting the content type response header from octet-stream to text/html. Just in case someone attempts to replicate this admittedly fringe case in the future, this is the config that worked for me:
http:
 - match:
   - uri:
       prefix: /static
   rewrite:
     authority: cdn-ch.azureedge.net
     uri: "/test"
   route:
   - destination:
       host: cdn-ch.azureedge.net
     headers:
       response:
         set:
           content-type: "text/html"

Closing this issue as it is now resolved.

@bogdansucila I am trying to achieve something similar as part of my setup. unable to do with ngnix (using external service pointing to CDN endpoint not working fine)..
So as part of your final config with istio, are you still maintaining two URL's or able to make the traffic route to CDN seamlessly with a single URL.


Support @PropertyName on Setters only

Issue
Currently in a project where we develop a service that wants to consume data from Firestore. The fields are named differently in firestore than we want them so we use @PropertyName to change them during deserialization. Since we only care about deserialization (we never write something back to firestore) I assumed that it would be enough to write setters with the annotation for this to work. This is not the case, since properties will be added by variable name (in this case because of getters without the annotation) and since the setters with annotation are not matching these, they will be ignored. So to have working deserialization it seems to be a requirement to have getters with the same annotation? The small issue this results in is that you can't generate the getters with something like lombok, instead you have to write the getters and include the same annotation.
Expected
Since we only care about creating the objects, I expected annotated setters to be enough to support the renaming from firestore to our POJO. Instead, the setters are ignored because the getters without annotation add properties with non-matching names. This is because there are no existingPropertyName matching the setters annotated value.
Finally
This is more of a quality of life improvement and not a dealbreaker. Also there might be a reason why getters with @PropertyName is required, or something else I have misunderstood. If so, an explanation to clarify why this is the case would be appreciated!

@schmidt-sebastian Do you have context or know someone who does about the existing priority of fields over setters?


make Contributing-information more precise, supply code style guidelines

The current contributing.rst is not that useful.

Suggestion for #122: GitHub has a new repo setting called "Only allow squash commits" (link). It looks like this can create a single commit based off development each time a pull request is made, without any extra effort from developers.
If a linear history is the goal, then this should be enabled and used in place of the line requesting a rebase in the developer guide IMHO.

I have seen that. However, squashing the commits creates one single feature commit, that is sometimes a bit too compressed (imho).
You are right, the guide is somewhat contradictory. The ideal pull request would contain imho logically grouped commits and would then be merged as a non-fast-forward-merge.
Maybe we should discuss first, how the proper workflow for commiting should be. Then it is easier to put it into a guide.
Do you think the current workflow should be changed? I'm always open for improvements.

And another thing to dicuss is: do we need a development-branch. We have it now, because refactoring the code was a large task and then more and more PRs came, but maybe we should switch to a "PRs against master"-workflow.

Maybe it could be re-phrased to give master some purpose (eg. latest tag), and suggest an interactive rebase for improving the clarity of a pull request where necessary.

Git
^^^
The master branch contains code that has been released to PyPi. Issues are closed
when they have been resolved in a released version of the package.
When you have a change to make, begin by creating a new branch from the HEAD of python-escpos/development. Name your branch to indicate what you are trying to achieve. Good branch names might be improve/text-handling, feature/enable-color-printing.
Please try to group your commits into logical units. If you need to tidy up your branch, you can make use of a git feature called an 'interactive rebase' before making a pull request. A small, self-contained change-set is easier to review, and improves the chance of your code being merged.


update callables to default to liteLLM and drop support for prompt, instructions and msg_history in favor of messages

Do not merge yet.  Slated for 0.6.x.


rutracker not working?

Hi
I have added my rutracker credentials and "Check" says: "Connection successfull".
But when I do "Add torrent" on the main page, and input this URL, nothing happens:
http://feed.rutracker.cc/atom/f/1997.atom
Any ideas?

It is not working with feed, it works with particular torrent like: https://rutracker.org/forum/viewtopic.php?t=5285924

Thanks for the reply.
I would like to watch the feed and do automatic downloads.
If I have to manually download individual torrents, I will look for another solution :)

Monitorrent doesn't support feed, it can watch for particular torrent and download it automatically if torrent will change.
I don't understand, what do you want from monitorrent?

Oh, I guess I misunderstood the purpose of Monitorrent.
I thought it could watch a feed from different sites and then filter and download the relevant files.
My usecase is to monitor the NBA forum feed from rutracker and automatically download the torrents from posts with "Condensed" in the name.

Ok, I got it. 😞
Sorry, but monitorrent doesn't support such feature yet 😢.
You can watch only for particular torrent.
This is first request from all our users.


RecyclerView support

i mean list-view is very old and require a lot of work ,,,

Hmm, Isn't it just creating an adapter? For now can't support RecyclerView due to other issues. But I would definitely check it out. Closing this issue for now.


Revert "Mark ECS Windows Fargate e2e tests as flaky (#30314)"

This reverts commit d52e811b7972b8592fc02a2db2178b6298ffad5e.
What does this PR do?
Un-mark the

TestECSSuite/Test00UpAndRunning/ECS_tasks_are_ready and
TestECSSuite/TestWindowsFargate
tests as flaky.

Motivation
The root cause of the test failure has been addressed by DataDog/test-infra-definitions#1220.
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes
Needs #30408.

/merge


[BUG] overflow of preview area for larger input values of grid generator

Description
The issue title is self explanatory
@Dun-sin As discussed here I am opening a new issue to fix this bug.
You may assign this issue to me. I would love to submit a PR.
Screenshots

Additional information
No response
🥦 Browser
Google Chrome
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue?
Yes I am willing to submit a PR!

ok, you know the rules, follow it

:tada: This issue has been resolved in version 2.71.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


update merge to 1.2.1

This was flagged as an automatic alert by GitHub
https://nvd.nist.gov/vuln/detail/CVE-2018-16469
NOTE: I haven't built this in Docker from this branch to test, it's worth testing before merging

I did a build to test this
kathyreid@kathyreid-ubuntu-X510UQ:~$ git clone https://github.com/MycroftAI/mimic-recording-studio.git mycroft-recording-studio-test
Cloning into 'mycroft-recording-studio-test'...
remote: Enumerating objects: 14, done.
remote: Counting objects: 100% (14/14), done.
remote: Compressing objects: 100% (14/14), done.
remote: Total 284 (delta 5), reused 0 (delta 0), pack-reused 270
Receiving objects: 100% (284/284), 4.30 MiB | 157.00 KiB/s, done.
Resolving deltas: 100% (118/118), done.
kathyreid@kathyreid-ubuntu-X510UQ:~$ cd mycroft-recording-studio-test/
kathyreid@kathyreid-ubuntu-X510UQ:~/mycroft-recording-studio-test$ ls
backend   docker-compose.yml  LICENSE.md  start-windows.bat
demo.gif  frontend            README.md
kathyreid@kathyreid-ubuntu-X510UQ:~/mycroft-recording-studio-test$ docker-compose up
ERROR: Couldn't connect to Docker daemon - you might need to run `docker-machine start default`.
kathyreid@kathyreid-ubuntu-X510UQ:~/mycroft-recording-studio-test$ sudo docker-compose up
[sudo] password for kathyreid:
Creating network "mycroftrecordingstudiotest_default" with the default driver
Building frontend
Step 1/4 : FROM node:latest
latest: Pulling from library/node
54f7e8ac135a: Pull complete
d6341e30912f: Pull complete
087a57faf949: Pull complete
5d71636fb824: Pull complete
0c1db9598990: Pull complete
89669bc2deb2: Pull complete
42f59d95957f: Pull complete
51cbe79eb934: Pull complete
Digest: sha256:72c50a1f151cf0310fc5cae53daaa2220a06d4d64204dd2f2a15d1739be0180f
Status: Downloaded newer image for node:latest
 ---> 2824ef9be5b3
Step 2/4 : RUN npm install -g yarn
 ---> Running in 826dd1a42563
/usr/local/bin/yarnpkg -> /usr/local/lib/node_modules/yarn/bin/yarn.js
/usr/local/bin/yarn -> /usr/local/lib/node_modules/yarn/bin/yarn.js
+<EMAIL_ADDRESS>added 1 package in 2.909s
Removing intermediate container 826dd1a42563
 ---> b286f2867f8e
Step 3/4 : WORKDIR /src/
 ---> Running in 70e3c4310f49
Removing intermediate container 70e3c4310f49
 ---> 787d2e1567ab
Step 4/4 : CMD ["bash", "start.sh"]
 ---> Running in 914a4befd05d
Removing intermediate container 914a4befd05d
 ---> f30d54ff251d
Successfully built f30d54ff251d
Successfully tagged mycroftrecordingstudiotest_frontend:latest
WARNING: Image for service frontend was built because it did not already exist. To rebuild this image you must use `docker-compose build` or `docker-compose up --build`.
Building backend
Step 1/7 : FROM python:3.6-alpine3.6
3.6-alpine3.6: Pulling from library/python
ab3d5dc0b96d: Pull complete
71afa6c1e317: Pull complete
665b92311f3a: Pull complete
288ef6399715: Pull complete
c0b1006d1dbd: Pull complete
Digest: sha256:777cbbdbfe81b9c5b86ccab85cf1716215423a9e40c70446b121d9d956074e6c
Status: Downloaded newer image for python:3.6-alpine3.6
 ---> af9d4081d2d7
Step 2/7 : COPY requirements.txt /src/
 ---> 9818f3665a0f
Step 3/7 : WORKDIR /src
 ---> Running in e393e723300f
Removing intermediate container e393e723300f
 ---> 339345511a66
Step 4/7 : RUN apk update
 ---> Running in 5bb715918f2a
fetch http://dl-cdn.alpinelinux.org/alpine/v3.6/main/x86_64/APKINDEX.tar.gz
fetch http://dl-cdn.alpinelinux.org/alpine/v3.6/community/x86_64/APKINDEX.tar.gz
v3.6.3-58-g8a2e635d73 [http://dl-cdn.alpinelinux.org/alpine/v3.6/main]
v3.6.3-57-gfd0de89d0b [http://dl-cdn.alpinelinux.org/alpine/v3.6/community]
OK: 8440 distinct packages available
Removing intermediate container 5bb715918f2a
 ---> 69d9909d1794
Step 5/7 : RUN apk add ffmpeg make automake gcc g++ subversion python3-dev
 ---> Running in efbfe6320ac1
(1/64) Installing perl (5.24.4-r2)
(2/64) Installing automake (1.15.1-r0)
(3/64) Installing sdl2 (2.0.5-r0)
(4/64) Installing libxau (1.0.8-r1)
(5/64) Installing libbsd (0.8.3-r3)
(6/64) Installing libxdmcp (1.1.2-r3)
(7/64) Installing libxcb (1.12-r0)
(8/64) Installing libx11 (1.6.6-r0)
(9/64) Installing alsa-lib (1.1.3-r0)
(10/64) Installing gmp (6.1.2-r0)
(11/64) Installing nettle (3.3-r0)
(12/64) Installing libtasn1 (4.10-r3)
(13/64) Installing p11-kit (0.23.2-r1)
(14/64) Installing libunistring (0.9.7-r0)
(15/64) Installing gnutls (3.5.13-r0)
(16/64) Installing lame (3.99.5-r6)
(17/64) Installing opus (1.1.4-r0)
(18/64) Installing librtmp (2.4_git20160909-r2)
(19/64) Installing libogg (1.3.2-r1)
(20/64) Installing libtheora (1.1.1-r12)
(21/64) Installing libjpeg-turbo (1.5.3-r2)
(22/64) Installing v4l-utils-libs (1.12.5-r1)
(23/64) Installing libxdamage (1.1.4-r0)
(24/64) Installing libxext (1.3.3-r1)
(25/64) Installing libxfixes (5.0.3-r0)
(26/64) Installing libxxf86vm (1.1.4-r0)
(27/64) Installing libpciaccess (0.13.4-r1)
(28/64) Installing libdrm (2.4.80-r0)
(29/64) Installing mesa-glapi (17.0.3-r1)
(30/64) Installing libxshmfence (1.2-r0)
(31/64) Installing mesa-gl (17.0.3-r1)
(32/64) Installing libva (1.8.1-r0)
(33/64) Installing libvorbis (1.3.6-r1)
(34/64) Installing libvpx (1.5.0-r0)
(35/64) Installing x264-libs (20161201-r0)
(36/64) Installing libgcc (6.3.0-r4)
(37/64) Installing libstdc++ (6.3.0-r4)
(38/64) Installing x265 (2.3-r0)
(39/64) Installing xvidcore (1.3.4-r0)
(40/64) Installing ffmpeg-libs (3.2.9-r0)
(41/64) Installing ffmpeg (3.2.9-r0)
(42/64) Installing binutils-libs (2.30-r1)
(43/64) Installing binutils (2.30-r1)
(44/64) Installing isl (0.17.1-r0)
(45/64) Installing libgomp (6.3.0-r4)
(46/64) Installing libatomic (6.3.0-r4)
(47/64) Installing pkgconf (1.3.7-r0)
(48/64) Installing mpfr3 (3.1.5-r0)
(49/64) Installing mpc1 (1.0.3-r0)
(50/64) Installing gcc (6.3.0-r4)
(51/64) Installing musl-dev (1.1.16-r14)
(52/64) Installing libc-dev (0.7.1-r0)
(53/64) Installing g++ (6.3.0-r4)
(54/64) Installing make (4.2.1-r0)
(55/64) Installing python3 (3.6.5-r0)
(56/64) Installing python3-dev (3.6.5-r0)
(57/64) Installing libuuid (2.28.2-r2)
(58/64) Installing apr (1.5.2-r1)
(59/64) Installing apr-util (1.5.4-r3)
(60/64) Installing db (5.3.28-r0)
(61/64) Installing libsasl (2.1.26-r10)
(62/64) Installing serf (1.3.9-r2)
(63/64) Installing subversion-libs (1.9.7-r0)
(64/64) Installing subversion (1.9.7-r0)
Executing subversion-1.9.7-r0.pre-install
Executing busybox-1.26.2-r11.trigger
OK: 326 MiB in 89 packages
Removing intermediate container efbfe6320ac1
 ---> c2b777781907
Step 6/7 : RUN pip install -r requirements.txt
 ---> Running in d61072857897
Collecting flask (from -r requirements.txt (line 1))
  Downloading https://files.pythonhosted.org/packages/7f/e7/08578774ed4536d3242b14dacb4696386634607af824ea997202cd0edb4b/Flask-1.0.2-py2.py3-none-any.whl (91kB)
Collecting flask-cors (from -r requirements.txt (line 2))
  Downloading https://files.pythonhosted.org/packages/65/cb/683f71ff8daa3aea0a5cbb276074de39f9ab66d3fbb8ad5efb5bb83e90d2/Flask_Cors-3.0.7-py2.py3-none-any.whl
Collecting peewee (from -r requirements.txt (line 3))
  Downloading https://files.pythonhosted.org/packages/b2/38/fe304c84dd7b771e6c313e2905d54944b9577b212e3247cfa58fac2ac935/peewee-3.8.0.tar.gz (805kB)
Collecting pydub (from -r requirements.txt (line 4))
  Downloading https://files.pythonhosted.org/packages/2f/73/bb9c093882d647437a9e6e87c7e6592d2df852f83ffac6f348b878979be0/pydub-0.23.0-py2.py3-none-any.whl
Collecting gunicorn (from -r requirements.txt (line 5))
  Downloading https://files.pythonhosted.org/packages/8c/da/b8dd8deb741bff556db53902d4706774c8e1e67265f69528c14c003644e6/gunicorn-19.9.0-py2.py3-none-any.whl (112kB)
Collecting click>=5.1 (from flask->-r requirements.txt (line 1))
  Downloading https://files.pythonhosted.org/packages/fa/37/45185cb5abbc30d7257104c434fe0b07e5a195a6847506c074527aa599ec/Click-7.0-py2.py3-none-any.whl (81kB)
Collecting Werkzeug>=0.14 (from flask->-r requirements.txt (line 1))
  Downloading https://files.pythonhosted.org/packages/20/c4/12e3e56473e52375aa29c4764e70d1b8f3efa6682bef8d0aae04fe335243/Werkzeug-0.14.1-py2.py3-none-any.whl (322kB)
Collecting itsdangerous>=0.24 (from flask->-r requirements.txt (line 1))
  Downloading https://files.pythonhosted.org/packages/76/ae/44b03b253d6fade317f32c24d100b3b35c2239807046a4c953c7b89fa49e/itsdangerous-1.1.0-py2.py3-none-any.whl
Collecting Jinja2>=2.10 (from flask->-r requirements.txt (line 1))
  Downloading https://files.pythonhosted.org/packages/7f/ff/ae64bacdfc95f27a016a7bed8e8686763ba4d277a78ca76f32659220a731/Jinja2-2.10-py2.py3-none-any.whl (126kB)
Collecting Six (from flask-cors->-r requirements.txt (line 2))
  Downloading https://files.pythonhosted.org/packages/73/fb/00a976f728d0d1fecfe898238ce23f502a721c0ac0ecfedb80e0d88c64e9/six-1.12.0-py2.py3-none-any.whl
Collecting MarkupSafe>=0.23 (from Jinja2>=2.10->flask->-r requirements.txt (line 1))
  Downloading https://files.pythonhosted.org/packages/ac/7e/1b4c2e05809a4414ebce0892fe1e32c14ace86ca7d50c70f00979ca9b3a3/MarkupSafe-1.1.0.tar.gz
Building wheels for collected packages: peewee, MarkupSafe
  Running setup.py bdist_wheel for peewee: started
  Running setup.py bdist_wheel for peewee: finished with status 'done'
  Stored in directory: /root/.cache/pip/wheels/de/de/13/a3be7f7fa6d373fe3e61224f4a508513cd2ea05695a047fef8
  Running setup.py bdist_wheel for MarkupSafe: started
  Running setup.py bdist_wheel for MarkupSafe: finished with status 'done'
  Stored in directory: /root/.cache/pip/wheels/81/23/64/51895ea52825dc116a55f37043f49be0939bcf603de54e5cde
Successfully built peewee MarkupSafe
Installing collected packages: click, Werkzeug, itsdangerous, MarkupSafe, Jinja2, flask, Six, flask-cors, peewee, pydub, gunicorn
Successfully installed Jinja2-2.10 MarkupSafe-1.1.0 Six-1.12.0 Werkzeug-0.14.1 click-7.0 flask-1.0.2 flask-cors-3.0.7 gunicorn-19.9.0 itsdangerous-1.1.0 peewee-3.8.0 pydub-0.23.0
Removing intermediate container d61072857897
 ---> dc559e5efd31
Step 7/7 : CMD ["sh", "start_prod.sh"]
 ---> Running in 033a7f45aac5
Removing intermediate container 033a7f45aac5
 ---> 508a05e9c191
Successfully built 508a05e9c191
Successfully tagged mycroftrecordingstudiotest_backend:latest
WARNING: Image for service backend was built because it did not already exist. To rebuild this image you must use `docker-compose build` or `docker-compose up --build`.
Creating mrs-frontend ...
Creating mrs-backend ...
Creating mrs-frontend
Creating mrs-backend ... done
Attaching to mrs-frontend, mrs-backend
mrs-backend | [2018-12-26 13:12:32 +0000] [7] [DEBUG] Current configuration:
mrs-backend |   config: gunicorn_conf.py
mrs-backend |   bind: ['<IP_ADDRESS>:5000']
mrs-backend |   backlog: 2048
mrs-backend |   workers: 1
mrs-backend |   worker_class: sync
mrs-backend |   threads: 1
mrs-backend |   worker_connections: 1000
mrs-backend |   max_requests: 0
mrs-backend |   max_requests_jitter: 0
mrs-backend |   timeout: 30
mrs-backend |   graceful_timeout: 30
mrs-backend |   keepalive: 2
mrs-backend |   limit_request_line: 4094
mrs-backend |   limit_request_fields: 100
mrs-backend |   limit_request_field_size: 8190
mrs-backend |   reload: False
mrs-backend |   reload_engine: auto
mrs-backend |   reload_extra_files: []
mrs-backend |   spew: False
mrs-backend |   check_config: False
mrs-backend |   preload_app: False
mrs-backend |   sendfile: None
mrs-backend |   reuse_port: False
mrs-backend |   chdir: /src
mrs-backend |   daemon: False
mrs-backend |   raw_env: []
mrs-backend |   pidfile: None
mrs-backend |   worker_tmp_dir: None
mrs-backend |   user: 0
mrs-backend |   group: 0
mrs-backend |   umask: 0
mrs-backend |   initgroups: False
mrs-backend |   tmp_upload_dir: None
mrs-backend |   secure_scheme_headers: {'X-FORWARDED-PROTOCOL': 'ssl', 'X-FORWARDED-PROTO': 'https', 'X-FORWARDED-SSL': 'on'}
mrs-backend |   forwarded_allow_ips: ['<IP_ADDRESS>']
mrs-backend |   accesslog: logs
mrs-backend |   disable_redirect_access_to_syslog: False
mrs-backend |   access_log_format: %(h)s %(l)s %(u)s %(t)s "%(r)s" %(s)s %(b)s "%(f)s" "%(a)s"
mrs-backend |   errorlog: -
mrs-backend |   loglevel: debug
mrs-backend |   capture_output: True
mrs-backend |   logger_class: gunicorn.glogging.Logger
mrs-backend |   logconfig: None
mrs-backend |   logconfig_dict: {}
mrs-backend |   syslog_addr: udp://localhost:514
mrs-backend |   syslog: False
mrs-backend |   syslog_prefix: None
mrs-backend |   syslog_facility: user
mrs-backend |   enable_stdio_inheritance: False
mrs-backend |   statsd_host: None
mrs-backend |   statsd_prefix:
mrs-backend |   proc_name: None
mrs-backend |   default_proc_name: app:app
mrs-backend |   pythonpath: None
mrs-backend |   paste: None
mrs-backend |   on_starting: <function OnStarting.on_starting at 0x7eff2b02a268>
mrs-backend |   on_reload: <function OnReload.on_reload at 0x7eff2b02a378>
mrs-backend |   when_ready: <function WhenReady.when_ready at 0x7eff2b02a488>
mrs-backend |   pre_fork: <function Prefork.pre_fork at 0x7eff2b02a598>
mrs-backend |   post_fork: <function Postfork.post_fork at 0x7eff2b02a6a8>
mrs-backend |   post_worker_init: <function PostWorkerInit.post_worker_init at 0x7eff2b02a7b8>
mrs-backend |   worker_int: <function WorkerInt.worker_int at 0x7eff2b02a8c8>
mrs-backend |   worker_abort: <function WorkerAbort.worker_abort at 0x7eff2b02a9d8>
mrs-backend |   pre_exec: <function PreExec.pre_exec at 0x7eff2b02aae8>
mrs-backend |   pre_request: <function PreRequest.pre_request at 0x7eff2b02abf8>
mrs-backend |   post_request: <function PostRequest.post_request at 0x7eff2b02ac80>
mrs-backend |   child_exit: <function ChildExit.child_exit at 0x7eff2b02ad90>
mrs-backend |   worker_exit: <function WorkerExit.worker_exit at 0x7eff2b02aea0>
mrs-backend |   nworkers_changed: <function NumWorkersChanged.nworkers_changed at 0x7eff2ab5c048>
mrs-backend |   on_exit: <function OnExit.on_exit at 0x7eff2ab5c158>
mrs-backend |   proxy_protocol: False
mrs-backend |   proxy_allow_ips: ['<IP_ADDRESS>']
mrs-backend |   keyfile: None
mrs-backend |   certfile: None
mrs-backend |   ssl_version: 2
mrs-backend |   cert_reqs: 0
mrs-backend |   ca_certs: None
mrs-backend |   suppress_ragged_eofs: True
mrs-backend |   do_handshake_on_connect: False
mrs-backend |   ciphers: TLSv1
mrs-backend |   raw_paste_global_conf: []
mrs-backend | [2018-12-26 13:12:32 +0000] [7] [INFO] Starting gunicorn 19.9.0
mrs-backend | [2018-12-26 13:12:32 +0000] [7] [DEBUG] Arbiter booted
mrs-backend | [2018-12-26 13:12:32 +0000] [7] [INFO] Listening at: http://<IP_ADDRESS>:5000 (7)
mrs-backend | [2018-12-26 13:12:32 +0000] [7] [INFO] Using worker: sync
mrs-backend | [2018-12-26 13:12:32 +0000] [10] [INFO] Booting worker with pid: 10
mrs-backend | [2018-12-26 13:12:32 +0000] [7] [DEBUG] 1 workers
mrs-frontend | yarn install v1.12.3
mrs-frontend | warning package-lock.json found. Your project contains lock files generated by tools other than Yarn. It is advised not to mix package managers in order to avoid resolution inconsistencies caused by unsynchronized lock files. To clear this warning, remove package-lock.json.
mrs-frontend | [1/4] Resolving packages...
mrs-frontend | [2/4] Fetching packages...
mrs-frontend | info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
mrs-frontend | info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
mrs-frontend | [3/4] Linking dependencies...
mrs-frontend | warning " ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^15.0.0".
mrs-frontend | [4/4] Building fresh packages...
mrs-frontend | Done in 61.60s.
mrs-frontend | yarn run v1.12.3
mrs-frontend | $ react-scripts start
mrs-frontend | Starting the development server...
mrs-frontend |
mrs-frontend | Compiled successfully!
mrs-frontend |
mrs-frontend | You can now view mimic-recording-studio in the browser.
mrs-frontend |
mrs-frontend |   Local:            http://localhost:3000/
mrs-frontend |   On Your Network:  http://<IP_ADDRESS>:3000/
mrs-frontend |
mrs-frontend | Note that the development build is not optimized.
mrs-frontend | To create a production build, use yarn build.
mrs-frontend |


[REEF-606] Generalize IPartitionedDataset

This addressed the issue by

Making interface IPartition generic
Open function is renamed to GetPartitionPointer and return pointer of generic type.
JIRA:
REEF-606

@dkm2110 Done with a pass.

@markusweimer  I have done required changes

@dkm2110 Looks good, will test and merge.


fixed ceilometer policy file to be up to date with community, and add…

…ed cloud_admin policy.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

retest


UTXO balance of xpub addresses with sats/btc toggle

UTXO balance of xpub addresses with sats/btc toggle

Codecov Report

Merging #266 (3b3adff) into master (3331dfe) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #266   +/-   ##
=======================================
  Coverage   30.21%   30.21%
=======================================
  Files          31       31
  Lines        2317     2317
=======================================
  Hits          700      700
  Misses       1617     1617

Continue to review full report at Codecov.

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


Caching in 0.10.0-rc1 is returning wrong values

This issue is not present in 0.9.2
This can be replicated with the wikiticker quickstart. Once this data is loaded, execute the following 2 queries :
{
    "queryType": "timeseries",
    "dataSource": "wikiticker",
    "granularity": "all",
    "aggregations": [
        {"type": "longSum", "name": "count", "fieldName": "count"},
        {"type": "longSum", "name": "added", "fieldName": "added"}
    ],
    "intervals": [ "2016/2018" ]
}

This return the correct values
[ {
  "timestamp" : "2016-06-27T00:00:11.080Z",
  "result" : {
    "count" : 11774265,
    "added" : 24433
  }
} ]

Then execute the following query (it's identical other than the order of the aggregations)
{
    "queryType": "timeseries",
    "dataSource": "wikiticker",
    "granularity": "all",
    "aggregations": [
        {"type": "longSum", "name": "added", "fieldName": "added"},
        {"type": "longSum", "name": "count", "fieldName": "count"}
    ],
    "intervals": [ "2016/2018" ]
}

Yields the following.
[ {
  "timestamp" : "2016-06-27T00:00:11.080Z",
  "result" : {
    "count" : 24433,
    "added" : 11774265
  }
} ]

The values returned have been "switched". With "count" incorrectly showing the "added" value in the result, and vice versa.
Disabling caching on both the historical and the broker removes the problem. If caching is on either the broker or the historical then the error returns.

Thanks for the report @gabrielpage. Confirmed locally, working on a fix.

Patch in #4094.

Fixed by #4094.


Support VM Extensions on Azure Batch pools

Feature Request Description
Currently, VirtualMachineConfiguration based pools cannot deploy VM extensions.
Describe Preferred Solution
Allow VM extensions to be deployed on VMC based pools.
Additional Context
This was originally requested by @benjzh here.

This would be useful for many scenarios. Another one is to be able to use the NVIDIA GPU Driver Extension described at https://docs.microsoft.com/en-us/azure/virtual-machines/windows/n-series-driver-setup?toc=%2Fazure%2Fvirtual-machines%2Fwindows%2Ftoc.json

Any news on this?


Test Method in Production Code

Context: Proper separation of test and production code.
Location: validator.go
Issue: GetPowers method is used only in tests but exists in production code.
Remediation: Move the method to test files.

no longer needed as we're switching to BlobstreamX


Partial read of E57 file

I would like to do a partial read of the e57 point cloud data at a specific location, avoiding the costly operation of reading the entire file. This is a follow-up to the last comment of #74, where you mentioned this e57 library could be used instead, as the rust library it's wrapping has the functionality. I'm not familiar with the underlying rust e57 library, but I imagine the skip method of PointCloudReaderSimple might be useful. Is there potential for this, and could this be on the roadmap? Thanks!
Edit: Although this might not be great for this library, I was also wondering if the e57 could be parsed directly in python itself without using a wrapper (I tried this a bit: I was able to access the xml, but I was having trouble reading the raw data at the specified locations).

Hello, I'm not very active with developing this library right now but I would definitely be interested in pull requests / working together on adding this functionality and might get around to it myself at some point !
Thinking about how this would look - This could maybe be added to the standard point reader by adding a chunksize keyword argument similar to pandas.read_csv. I think ideally this would return an iterator over chunks of points.

I was also wondering if the e57 could be parsed directly in python itself without using a wrapper

I wonder whether numpy has methods for this... there is https://numpy.org/doc/stable/reference/generated/numpy.fromfile.html
I guess in theory you could cut out just the bit of the file you want and read that using numpy.fromfile if you can specify the correct data type to match the endianness, etc...

I took a look at your library, since the wrapper code itself is written in rust (I think), and I don't know any rust, while maybe I could take a shot, sadly I'm probably not a great person for the task.
For implementing directly in python, I'm thinking you could directly use python's binary read, as you would have more direct control over how the numbers are formatted. For example, it looks like e57 uses scaledInteger which seems like a special data type for storing numbers.
I used BunnyInt32.e57 from libe57 as a test. I first read it using the e57 library, the first coordinate is:
[-0.07063, 0.04015, 0.0012259999999999999]
I then took a stab at reading the data myself. I was able to access the xml, which describes the data structure of a coordinate:
<cartesianX type="ScaledInteger" minimum="-2147483648" maximum="2147483647" scale="9.9999999999999995e-007"/>
<cartesianY type="ScaledInteger" minimum="-2147483648" maximum="2147483647" scale="9.9999999999999995e-007"/>
<cartesianZ type="ScaledInteger" minimum="-2147483648" maximum="2147483647" scale="9.9999999999999995e-007"/>
<cartesianInvalidState type="Integer" minimum="0" maximum="1"/>

I then tried reading this first coordinate:
import math

# information on the scaled integer format
# they were manually taken from the xml and of course should be retrieved dynamically
start_data_offset = 48  # the first 48 bytes are for the header
minimum_value = -2147483648
maximum_value =<PHONE_NUMBER>
scale = 10 * 10**-7

n_bits_per_int = math.log2(maximum_value-minimum_value+1)
assert n_bits_per_int.is_integer(), "should never have a noninteger value"
assert n_bits_per_int == 32, "this file is supposed to be 32bit"

n_bytes_per_int = int(n_bits_per_int/8)
n_bytes_per_coordinate = n_bytes_per_int * 3    # this doesnt include the invalidstate, i'm not sure how many bits this would be (only 1?)

def get_integer_value(int_bytes: bytes):

    """converts the scaled integer bytes into a number"""

    # the header was little endian, so my guess is the data is too
    unscaled_value = int.from_bytes(int_bytes, "little")
    # from docs for scaledInteger:  The actual data value is computed by multiplying the stored integer by the scale and adding the offset.
    scaled_int_offset = minimum_value * scale
    value = unscaled_value * scale + scaled_int_offset
    return value

with open("BunnyInt32.e57","rb") as f:

    f.seek(start_data_offset)

    # read the first coordinate values
    x_bytes = f.read(n_bytes_per_int)
    y_bytes = f.read(n_bytes_per_int)
    z_bytes = f.read(n_bytes_per_int)

    x = get_integer_value(x_bytes)
    y = get_integer_value(y_bytes)
    z = get_integer_value(z_bytes)

sadly, the values did not match at all. In binary, it's:
[b'\x01\x00\x00\x00', b'\x00\x00\x00\x00', b'\x90\xa8\x05\x00']
which I converted to:
[-2147.483647, -2147.483648, -2147.112816]
I'm a bit stuck here, I'm not sure if there's some wrinkle to the data format that I'm missing...

Do you maybe need to read the scale and offset from the node itself not from the file header ?

I think there is zero offset in this file - looking in cloud compare the "Shifted Box Centre" is equal to the "Global Box Centre" and the xml section doesn't describe an offset. That still doesn't give the right answers for me though ...
I guess you will also have the challenge of trying to read a single bit of data for the invalidstate where python only wants to read complete bytes

I think there is zero offset in this file - looking in cloud compare the "Shifted Box Centre" is equal to the "Global Box Centre" and the xml section doesn't describe an offset.

Nice, I think you're right, it's storing signed integers with a scale but no offset.

I guess you will also have the challenge of trying to read a single bit of data for the invalidstate where python only wants to read complete bytes

Yup, I think it would doable with formatting as binary strings (simpler but I'm guessing super inefficient), or some bit manipulation on the individual bytes, but that would be tricky to get right (at least for me).
I'm still stumped on the values not matching the actual coordinates. Maybe there's a place to ask about this, or maybe we are better off with a wrapper around the Rust library like you're doing...


AZ-204_lab_05.md

English:
Select (US) East US
Current Translation:
選取 (美國) 美國東部
Suggested Translation:
選取 [(美國) 美國東部]
Description:
It is a UI and should use UI format. Global check

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run bugfix tool and waiting for result ICMS QUERY