Example not working on Windows

Edit: the basic template works, I think the problem is with my nodejs install, I will investigate it.
Sorry for the noise.
~I tried to run the example (as in the docs), but getting an ErrorException:~
Microsoft Windows [Version 10.0.17763.557]
(c) 2018 Microsoft Corporation. Minden jog fenntartva.
C:\Users\Ipse\Documents\Code>julia
               _
   _       _ _(_)_     |  Documentation: https://docs.julialang.org
  (_)     | (_) (_)    |
   _ _   _| |_  __ _   |  Type "?" for help, "]?" for Pkg help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 1.1.0 (2019-01-21)
 _/ |\__'_|_|_|\__'_|  |  Official https://julialang.org/ release
|__/                   |

(v1.1) pkg> add JuDoc
  Updating registry at `C:\Users\Ipse\.julia\registries\General`
  Updating git-repo `https://github.com/JuliaRegistries/General.git`
 Resolving package versions...
 Installed LiveServer ───── v0.1.1
 Installed JuDoc ────────── v0.1.3
 Installed JuDocTemplates ─ v0.2.1
  Updating `C:\Users\Ipse\.julia\environments\v1.1\Project.toml`
  [4ca9428c] + JuDoc v0.1.3
  Updating `C:\Users\Ipse\.julia\environments\v1.1\Manifest.toml`
  [4ca9428c] + JuDoc v0.1.3
  [6793090a] + JuDocTemplates v0.2.1
  [16fef848] + LiveServer v0.1.1

julia> using JuDoc
[ Info: Recompiling stale cache file C:\Users\Ipse\.julia\compiled\v1.1\JuDoc\RFfo3.ji for JuDoc [4ca9428c-4c75-11e9-2efb-bf5cb6c1e8f8]
✘ Couldn't find highlight.js (`node -e "require('highlight.js')"` failed).
→ It is required for pre-rendering highlight.js but is not necessary to run JuDoc (cf docs).
✘ Couldn't find python3 (`python3 -V` failed).
→ It is required for minification but not necessary to run JuDoc (cf docs).
→ After installing any missing component, please re-build the package (cf docs).

julia> newsite("test", template="pure-sm")
✓ Website folder generated at "test" (now the current directory).
→ Use serve() from JuDoc to see the website in your browser.

julia> serve()
→ Initial full pass...

... error processing 'menu1.md' in ...\Code\test\src\pages.
Verify, then start judoc again...

err = ErrorException("Command '\\output' was not defined before it was used.")
-1

julia> versioninfo()
Julia Version 1.1.0
Commit 80516ca202 (2019-01-21 21:24 UTC)
Platform Info:
  OS: Windows (x86_64-w64-mingw32)
  CPU: Intel(R) Core(TM) i3-3227U CPU @ 1.90GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.1 (ORCJIT, ivybridge)

(v1.1) pkg> st
    Status `C:\Users\Ipse\.julia\environments\v1.1\Project.toml`
  [537997a7] AbstractPlotting v0.9.8
  [c52e3926] Atom v0.8.7
  [6e4b80f9] BenchmarkTools v0.4.2
  [5ae59095] Colors v0.9.5
  [31a5f54b] Debugger v0.5.0
  [0c46a032] DifferentialEquations v6.4.0
  [67fdca37] Displaz v1.0.0
  [add2ef01] GDAL v0.2.0
  [f7f18e0c] GLFW v2.3.0
  [e9467ef8] GLMakie v0.0.6
  [4d00f742] GeometryTypes v0.7.5
  [7073ff75] IJulia v1.18.1
  [6218d12a] ImageMagick v0.7.4
  [916415d5] Images v0.18.0 ⚲
  [4ca9428c] JuDoc v0.1.3
  [e5e0dc1b] Juno v0.7.0
  [ee78f7c6] Makie v0.9.4
  [dbd62bd0] MakieGallery v0.0.8
  [66fc600b] ModernGL v1.0.0
  [510215fc] Observables v0.2.3
  [1dea7af3] OrdinaryDiffEq v5.8.1
  [9b87118b] PackageCompiler v0.6.3
  [c46f51b8] ProfileView v0.4.1
  [df47a6cb] RData v0.6.0
  [ce6b1742] RDatasets v0.6.2
  [dee08c22] RegionTrees v0.2.0
  [295af30f] Revise v2.1.6
  [6038ab10] Rotations v0.11.1
  [90137ffa] StaticArrays v0.11.0
  [65254759] StatsMakie v0.0.6
  [8603256b] ZChop v0.3.3

~Am I missing something and/or Windows is not fully tested/supported? I'm happily helping to investigate further, but just started looking into the package. (Also I installed the python and node.js dependencies, but JuDoc couldn't find them. But I will investigate these things, before asking for further help.)~

No worries, please let me know if you encounter other problems.
Also you may want to use JuDoc#master for now as for instance it plays better with the python call on windows. You can also specify how you want either python or node to be called. See https://github.com/tlienart/JuDoc.jl/blob/master/src/build.jl (I still need to update the doc about this https://github.com/tlienart/JuDoc.jl/issues/167) so if you have a specific way of calling python or node, you can specify this in ENV["PYTHON3"] or ENV["NODE"].

Thanks for the environment variables link/source. Python started working without further debug, and I think that #178 solves the node.s part.

Awesome, thanks for letting me know


DGIdb should support drug classes and combinations.

While importing DoCM, we found many entries that were either combinations of drugs or a type of drug "BRAF inhibitors." Currently, the data model assumes a single drug with a single name as part of each interaction. We may want to expand to schema to accommodate these types of entries.

I'm surprised that this is the first time we encountered this.  I think some "drugs" according to some sources might actually be 'compounds' that are in some cases combinations of drugs. Anyway, I agree that more formal support for this concept is desirable. :)

Plan is to leverage drug normalization service in development at VICC

see related #344


WFS GetFeature via POST

Is there support for sending WFS GetFeature requests via POST? I'd like to query the WFS service using a prepared WFS GetFeature filter in XML, but I can't get it to work with the current version.
The 'method' parameter seems to suggest it should work:
https://github.com/geopython/OWSLib/blob/d2a9722685de09e4f70189cdc034a23f5ecccffa/owslib/feature/wfs110.py#L154-L157
But changing this to method='Post' and providing a filter as XML doesn't seem to work.
I think the culprit is in the
https://github.com/geopython/OWSLib/blob/d2a9722685de09e4f70189cdc034a23f5ecccffa/owslib/feature/wfs110.py#L247
I can prepare a PR to support a (fourth) way of using the GetFeature method, with just a filter (in XML) and method = 'Post'? Like so:
wfs = WebFeatureService(
                url="https://www.dov.vlaanderen.be/geoserver/wfs",
                version="1.1.0")

filter = """<wfs:GetFeature service="WFS" version="1.1.0"
  xmlns:wfs="http://www.opengis.net/wfs"
  xmlns:ogc="http://www.opengis.net/ogc"
  xmlns:gml="http://www.opengis.net/gml"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xsi:schemaLocation="http://www.opengis.net/wfs
                      http://schemas.opengis.net/wfs/1.1.0/wfs.xsd">
  <wfs:Query typeName="gw_meetnetten:meetnetten">
    <ogc:Filter>
      <ogc:And>
      <ogc:Within>
        <ogc:PropertyName>geom</ogc:PropertyName>
        <gml:Polygon srsName="http://www.opengis.net/gml/srs/epsg.xml#31370"><gml:exterior><gml:LinearRing><gml:posList>145507.928000719<PHONE_NUMBER>69683<PHONE_NUMBER>22679<PHONE_NUMBER>48491<PHONE_NUMBER>02172<PHONE_NUMBER>05293<PHONE_NUMBER>48721<PHONE_NUMBER>45256<PHONE_NUMBER>82115<PHONE_NUMBER>52685<PHONE_NUMBER>6093<PHONE_NUMBER>2839<PHONE_NUMBER>18193<PHONE_NUMBER>64443<PHONE_NUMBER>95669<PHONE_NUMBER>93785<PHONE_NUMBER>02482<PHONE_NUMBER>49089<PHONE_NUMBER>65871<PHONE_NUMBER>95865<PHONE_NUMBER>76775<PHONE_NUMBER>11176<PHONE_NUMBER>09693<PHONE_NUMBER>22265<PHONE_NUMBER>0153<PHONE_NUMBER>32092<PHONE_NUMBER>55128<PHONE_NUMBER>9942<PHONE_NUMBER>59324<PHONE_NUMBER>08396<PHONE_NUMBER>94333<PHONE_NUMBER>40651<PHONE_NUMBER>00719<PHONE_NUMBER>69683</gml:posList></gml:LinearRing></gml:exterior></gml:Polygon>
      </ogc:Within>
      <ogc:PropertyIsEqualTo>
      <ogc:PropertyName>gemeente</ogc:PropertyName>
      <ogc:Literal>Mechelen</ogc:Literal>
      </ogc:PropertyIsEqualTo>
      </ogc:And>
    </ogc:Filter>
  </wfs:Query>
</wfs:GetFeature>"""

result = wfs.getfeature(filter=filter, method='Post')

Are we in favor of supporting such a use-case?

We're hitting size limits on our GET feature requests, so having support for POST would be great.

Hello
Post is not working for me. Is this issue solved in current branch?

This can be closed now with #706.


[Question]: Should the Action Input always include the query?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I'm using a React Agent over some of my documentation, one of my sources is Confluence and the others are PDF files. Both of these have separate agents and engines setup respectively. These agents use Llama2:13b via Ollama running on my laptop.
When I query the Confluence agent, it will go through the Thought/Observation loop as expected. The first instance of the Thought will have Action Input: {"input": "<My Query>"}.
However the agent that uses the PDF agent returns the name of the document in that input like this Action Input: {"input": "<Name of file>"}.
It still answers my question but I feel like something is amiss here. Is there a proper way or tool/function to check what is the cause of this? I'm thinking of looking at the contents of my index and engine but I want to do this more scientifically than just looking manually.
Thanks!

Turns out the problem was the that my engine wasn't labelled very specifically in my tool defiition. I had the name labelled as "customer_report" but it needed to be something like "<company_name>_report". I'm not sure why this happens but just in case it helps someone.


Standardizing resource and column names

For the background, see https://github.com/CellMigStandOrg/CMSO-Samples/pull/18. After discussing at today's meeting, the consensus for now seems to be:
Agree on fixed names to refer to files within datapackage.json. For instance, objects_table for the objects table. Note that it's not the file name that should be fixed, but rather the name that the JSON file uses to refer to it.
For instance, the JSON data could be:
{
  "resources": [{
    "name": "objects_table",
    "path": "objects.csv",
    ...
    }]
  ...
}

Or:
{
  "resources": [{
    "name": "objects_table",
    "path": "tracking_output_0001.csv",
    ...
    }]
  ...
}

While the actual "path" of the tabular file changes from datapackage to datapackage, the resource's "name" is fixed. This could be implemented with a global variable defined somewhere in the API, e.g.:
OBJECTS_NAME = "objects_table"

which would be used both by data package generators and readers. The same goes for other files that make up the data package.
Readers also need a standard way to refer to column names. Options for solving this include:

agree on fixed names for columns, such as "x_coord", "frame_idx", etc. In this case, the code that generates data packages would need to convert the CSV files during the process, replacing column names on the basis of a mapping file such as the .ini we are currently using.

allow columns to have arbitrary names. In this case the datapackage should include the mapping file so that the reader can resolve the actual names.

In any event, keys in the mapping file (e.g., "x_coord_cmso") should be fixed.
A third option that has not been discussed at the meeting is to use the "name" property of table schema fields for the fixed CMSO keys and the "title" property for the actual column name in the file, e.g.:
{
  "resources": [{
    "name": "objects_table",
      "schema": {
        "fields": [{
          "name": "cmso_frame",
          "title": "FRAME",
          ...

Note that the spec imposes a restriction on "name" attributes (Lower case characters with ., _, - and / are allowed). This does not seem to be currently enforced for table schema fields in the Python API, but it is enforced for other "name" attributes at upper levels. In future versions of frictionless APIs they might add this missing restriction, so this third option:

is probably the safest long-term
does not require CSV conversions
does not require the inclusion of a mapping file in the data package

As for the agreement on fixed names to refer to files within datapackage.json, perhaps best to have something like option 2:
{
  "resources": [{
    "name": "objects_table",
    "path": "tracking_output_0001.csv",
    ...
    }]
  ...
}

so we fix the resource's "name" (objects_table), while the actual "path" stays flexible to change across data_packages.
As for a standard way to refer to column names, I do agree that the third option seems to be the cleanest and preferable ("name" property is fixed (something like x_coord_cmso), while the "title" changes and can be used to write the CSV files and read them back for further plotting, analytics etc.)

Briefly discussed with @simleo. Overall, I like the usage of a well-defined property (here name) in both resources and schema to provide an unambiguous mapping between the tables/columns in the file format and the specification. Coming back to the  initial discussions on the Positions, this have similar functionality as the prefixed CMSO:xxx column title.
Maintaining the flexibility on the column naming also means existing CSV tables could be unaltered under some conditions with all the metadata layer expressed at the level of the JSON file. Also this approach might simplify both the implementation but also the validation of the file format.

See also https://github.com/CellMigStandOrg/cell_track_dpkg/issues/14 (not entirely orthogonal to this one)

@simleo is this partly or fully addressed via #15?

@simleo is this partly or fully addressed via #15?

Only partly. createdp takes data from dictionaries created in readfile, where many fields are still hardwired to the specific values we have in our examples.

I have updated the original description by appending a new "UPDATE (2017-05-22)" section. This stems from considerations emerged after working on several pull requests that required more extensive review of the exisiting code. I haven't altered the text above that, so that the original discussion can still be followed for future reference, but I will probably close this proposal and open a new one to avoid confusion.

Fixed in #41.


Does non responsive designed application developed for desktop only can be uploaded in Microsoft Store ?

In this section, we are presented the responsive techniques, but I cannot find anywhere I it's possible to upload a non responsive application developed for desktop only to Microsoft store

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

ID: 7371c6d4-c3b8-f6ed-dc2c-11e62504adde
Version Independent ID: 9f674922-6d8b-4293-756f-0036ad9cbcff
Content: Responsive design techniques - UWP app developer
Content Source: windows-apps-src/design/layout/responsive-design.md
Product: windows
GitHub Login: @mijacobs
Microsoft Alias: mijacobs

Thanks for the question, @buricanu! Response design is a recommendation, not a requirement. You can upload an application designed only for the desktop to the Microsoft Store. However, just because an app is only for the desktop doesn't meant it shouldn't be responsive--there are a variety of monitor sizes and resolutions. Also, the user can make the app window smaller and larger.


Add Kelp dependency in README

The plugin depends on kelp however there is no mention of that and where to get the kelp plugin

added in commit 7df747c71b24e142c26eafc8ec0b6cb458828360


Provide skygear config for plugin init events

connect skygearIO/py-skygear#99

@ben181231, thanks for your PR! By analyzing the annotation information on this pull request, we identified @rickmak to be a potential reviewer

Merge via 4c51ffe


Move the SSL ALPN to a separate thread

Description
Don't run the Application Level Protocol Negotiation in the thread that accepted the new socket. Instead run it in a different thread.
Motivation and Context
Prevent lock ups when there are problems with the SSL negotiation. Fixes Issue IBM-Swift/Kitura#1143
How Has This Been Tested?
Have run the Kitura unit tests and load tested by @djones6.
Checklist:

[ ] I have submitted a CLA form
[ ] If applicable, I have updated the documentation accordingly.
[ ] If applicable, I have added tests to cover my changes.

Codecov Report

Merging #218 into master will decrease coverage by 0.04%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #218      +/-   ##
==========================================
- Coverage   73.86%   73.81%   -0.05%
==========================================
  Files          30       30
  Lines        4082     4086       +4
==========================================
+ Hits         3015     3016       +1
- Misses       1067     1070       +3

Flag
Coverage Δ

#CHTTPParser
56.04% <ø> (-0.07%)
:arrow_down:

#KituraNet
73.81% <88.88%> (-0.05%)
:arrow_down:

Impacted Files
Coverage Δ

Sources/KituraNet/HTTP/HTTPServer.swift
81.29% <88.88%> (-0.83%)
:arrow_down:

Sources/CHTTPParser/http_parser.c
55.84% <0%> (-0.07%)
:arrow_down:

Continue to review full report at Codecov.

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

This didn't cause performance regression, but also appears not to have fixed the hang,  which I believe is happening at the acceptClientConnection() stage (which happens before the handleNewClient() part)
The code flow seems to be:
HTTPServer calls listenSocket.acceptClientConnection() (Socket)

calls self.delegate?.onAccept(socket: newSocket) (SSLService)
calls SSL_accept(sslConnect) which then blocks

I wonder if the fix belongs in Socket or SSLService? The issue seems to be that, if Socket has a delegate and calls onAccept(), the delegate is probably going to try to read some more bytes off the wire, which we now know could block.
A fix in either place would add a dependency on Dispatch.  Either that, or somehow decoupling the TCP-level socket accept from the protocol-level handshaking done in onAccept so that the application (Kitura) can decide to run it on a separate thread.

@shmuelk I believe I have implemented this change in #219:  since ALPN requires SSL, the changes from #219 will cause the rest of the socket initialization / handling to be performed on another thread.
SocketManager.handle() will still be called on the main listener thread if there is no delegate, but I think it's safe to assume that this case can only be vanilla http/1.1.

@djones6 can we close this?

I think so as per my comments above. We can always raise it again if necessary.


fix(server_openvr): :bug: Make SteamVR chaperone consistent with ALVR

Often times, recentering using SteamVR, breaks ALVR calibration. With this PR we force SteamVR to revert to stage tracking any time it gets reset as standing or seated.
Not tested yet. The code that listens for recentering could cause an infinite loop.

@The-personified-devil I wanted to discuss this. Do you think it's ok to override the game preference and re-reset the guardian space to stage when the game actually requested to reset to standing or seated? The benefit of forcing reset to stage is that we can remove the linux zero pose workaround. But this will cause glitches, for example when a game wants to reset to seated, then ALVR would undo it and you get teleported up. instead maybe we should keep track of which chaperone mode it's set to, and when pressing recentering we ask to zero pose with the current SteamVR chaperone mode


Should be able to create worker pools in AWS

Currently there is no provision to create worker pools in AWS similar to the configuration in Packet.

Fixed by https://github.com/kinvolk/lokomotive/pull/151. Closing.


Added ability to localize "On" and "Off" strings for toggle switches.

Clean up two unused variables.
See issue #10668488

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

It looks like you're a Microsoft contributor (Ale Contenti). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


responsive typography and vertical rhythm

about responsive typography in tailwind... sure one could set it up using the utility classes in some cases. i usually create a component and it get some html parsed content from a cms. like h1 and p in example below.

Hello

world

lorem ipsum

i ended up using @apply and @screen but thats a bit of a hassel to get done responsive imho. especially calulating the margin-bottom for vertical rhythm myself.
.my-component {
  h1 {
    @apply
      .text-xl
      .leading-normal
      .mb-6;
      /* 20px, 1.4 unitless lineheight, 1.4rem margin */
  }
  h2 {
    @apply .text-lg .leading-tight .mb-5; /* 18px, 1.2, ~1.2rem */
  }
  p {
   @apply .text-base;
  }
  p + p {
    @apply .pt-4; /* 1rem padding top */
  }
}

/* skipping @screen sm to keep it brief */

@screen md {
  .my-component {
    h1 {
      @apply .text-3xl .leading-tight .mb-7;
      /* 28px, 1.2 lineheight, 1.2 margin based on lineheight.
     now to get vertical rhythm right...
      margin bottom: not 1.2rem but 28/20*1.2 = 1.68 rem ! */
    }
   /* ... and lots more :-( */
  }
}

since i am using tailwind as a webpack postcss plugin i can no longer make use of sass mixin typi and its awesome vertical-rhythm function.
what i would love to have instead is something like this.
tailwind.config.js
// sm = 600px
// md = 800px
// define named font and vertical rhythm pairs for each breakpoint
typo: {
  'base': { // applied to root html element using @typo-init; like @include typi-init; does in typi.
    'base' : [ 16px, 1.4 ], // font-size, line-height aka vr. enter in px but will be converted to rem
    'sm': [ 18px ], // keeps 1.4 from above
    'md': [ 20px ],
  },
  'headline' : {
    'base' : [ 20px, 1.4 ],
    'sm': [ 24px ], // skipping line-height
    'md': [ 28px, 1.2 ],
  },
  'subtitle' : {
    'base' : [ 18px, 1.2 ],
    // skipping sm
    'md': [20px ],
  }
}

any tailwind css file
html {
  @typo-init;
}
.my-component {
  h1 {
    @typo headline; // will create responsive font-size and line-heights
    @vr headline margin-bottom; // will create responsive margin-bottoms based on rem of line-height for headline
  }
  h2 {
    @typo subtitle;
    @vr subtitle padding; // will create responsive versions of padding set to line-height of subtitle in rem
  }
  p {
   @typo base; // this is redundant since calling @typo-init on html, but it explains it usage.
  }
  p + p {
    @vr base padding-top; // i think you get where i am going by now ;-)
  }
}

result of @typo-init
html {
  font-size: 100%; /* this means 16px */
  line-height: 1.4;
}

@media all and (min-width: 600px) {
  html {
    font-size: 112.5%; /* this means 18px */
  }
}

@media all and (min-width: 800px) {
  html {
    font-size: 125%; /* this means 20px */
  }
}

the resulting css is what i did before by hand but way shorter and solving the vertical rhythm problem.
maybe tailwind could be enhanced for better responsive typography and vertical rhythm based on typi? or any suggestions how to get it done better using tailwind?

the main problem is how @typo and @vr should know their context and "bubble up" the @media rule.

would love to figure out how we could do more vertical rhythm stuff with tailwind. has anyone made personal progress on this?

did anyone move forward with this idea? @bnomei

maybe we can do similar to Bootstrap 4.3? They are using this code https://github.com/twbs/rfs#how-does-it-work


Package manager UI

This PR doesn't address @huguesv's changes from yesterday, so "avaliable packages" tab and install/uninstall buttons aren't working yet.

I can't find the string for the 'Each package is licensed...' in the screenshot above. But it should say 'Microsoft is not responsible...' rather than 'RTVS'

Done

:+1: pending Pavel's CR. Also, may need to consider adding some telemetry in 0.4

You'll have to handle RPackageManagerException at some level, right now I don't see it being caught anywhere.

As i said in the description, your changes from yesterday aren't addressed :)

:+1:, let's get it in for the bash


Implement nvim_create_user_command

This moves the creation of the :CmpStatus command to the new vim.api.nvim_create_user_command API
This requires neovim 0.7 so it should be done along with #844

LGTM. Thank you!


Web export html doesn't work offline, only works if uploaded to a website/itch

Describe the bug
I'm guessing this is considered a bug, but it could be a missing feature.
The exported web html version only works if connected to the internet through a website or itch.io, but it doesn't work if just opening the html file offline. I just get a blank screen. Online works 100% (I've deployed to itch and my own netlify site). The Grant Galitz's JavaScript GameBoy Color Emulator (https://github.com/taisel/GameBoy-Online) which is the base emulator does work offline, so I feel the fix is possible. Maybe some caching needs to be done? I tried comparing the javascript files of the original emulator and the GB Studio fork to find the difference causing the problem, but couldn't find a solution (my js and html knowledge is pretty basic).
To Reproduce
Opening the built index.html from desktop.
Expected behavior
It would be nice to have offline compatibility for different options of publishing and delivering the game.
Additional comments
I realize it may not be within the scope of GB Studio or really needed by many people, but I would like the functionality so if someone has a workaround I would greatly appreciate it!

It does work from a local server, just not from "file://" (double clicking index.html)

Recommend for closing @chrismaltby

try wrapping in a paw

PAW? Do you mean PWA?
The existence of a PWA manifest won't fix the 'local files' issue.
I think the user wants to distribute the game offline and without requiring an emulator.
You could bundle the web page into a wrapper like https://nwjs.io/ - though there's probably a might lighter-weight way of bundling the games with a desktop-native emulator for distribution on places like steam/itch/etc


HBASE-25439 Add BYTE unit in PrettyPrinter.Unit

HBASE-25439
Add a new unit 'BYTE' to pretty print value of size, such as MAX_FILESIZE, MEMSTORE_FLUSHSIZE

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 28s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 22s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 56s
master passed

+1 :green_heart:
checkstyle
1m  6s
master passed

+1 :green_heart:
spotbugs
1m 48s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 28s
the patch passed

-0 :warning:
checkstyle
0m 23s
hbase-common: The patch generated 1 new + 4 unchanged - 0 fixed = 5 total (was 4)

-0 :warning:
rubocop
0m  9s
The patch generated 4 new + 343 unchanged - 4 fixed = 347 total (was 347)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
17m  2s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
1m 59s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

39m 56s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle rubocop

uname
Linux 17f371b2da04 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dcb38f47db

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-common.txt

rubocop
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/artifact/yetus-general-check/output/diff-patch-rubocop.txt

Max. process+thread count
94 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=3.1.12 rubocop=0.80.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 34s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 21s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  0s
master passed

+1 :green_heart:
compile
1m  8s
master passed

+1 :green_heart:
shadedjars
7m 10s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 56s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 48s
the patch passed

+1 :green_heart:
compile
1m 10s
the patch passed

+1 :green_heart:
javac
1m 10s
the patch passed

+1 :green_heart:
shadedjars
7m 44s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  8s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 48s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 24s
hbase-client in the patch passed.

+1 :green_heart:
unit
7m  5s
hbase-shell in the patch passed.

41m  4s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 5fc455d96690 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dcb38f47db

Default Java
AdoptOpenJDK-1.8.0_232-b09

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/testReport/

Max. process+thread count
1490 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 22s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 39s
master passed

+1 :green_heart:
compile
1m 27s
master passed

+1 :green_heart:
shadedjars
8m 50s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 16s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 17s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
5m 22s
the patch passed

+1 :green_heart:
compile
1m 28s
the patch passed

+1 :green_heart:
javac
1m 28s
the patch passed

+1 :green_heart:
shadedjars
8m 19s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 10s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 55s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 16s
hbase-client in the patch passed.

+1 :green_heart:
unit
7m  7s
hbase-shell in the patch passed.

45m 42s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c7b84aa78628 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / dcb38f47db

Default Java
AdoptOpenJDK-11.0.6+10

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/testReport/

Max. process+thread count
2384 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 14s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 30s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 57s
master passed

+1 :green_heart:
compile
1m  7s
master passed

+1 :green_heart:
shadedjars
7m 18s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 53s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 52s
the patch passed

+1 :green_heart:
compile
1m  6s
the patch passed

+1 :green_heart:
javac
1m  6s
the patch passed

+1 :green_heart:
shadedjars
7m 12s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 52s
the patch passed

_ Other Tests _

-1 :x:
unit
1m  1s
hbase-common in the patch failed.

+1 :green_heart:
unit
1m 14s
hbase-client in the patch passed.

+1 :green_heart:
unit
6m 57s
hbase-shell in the patch passed.

39m  0s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux ebbea8240433 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d963342f8a

Default Java
AdoptOpenJDK-1.8.0_232-b09

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-common.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/testReport/

Max. process+thread count
1494 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 34s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 22s
master passed

+1 :green_heart:
compile
1m 20s
master passed

+1 :green_heart:
shadedjars
7m 10s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  4s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 16s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 22s
the patch passed

+1 :green_heart:
compile
1m 22s
the patch passed

+1 :green_heart:
javac
1m 22s
the patch passed

+1 :green_heart:
shadedjars
7m  9s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  4s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 52s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 10s
hbase-client in the patch passed.

+1 :green_heart:
unit
7m  4s
hbase-shell in the patch passed.

40m 57s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 1c07eb8441fe 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d963342f8a

Default Java
AdoptOpenJDK-11.0.6+10

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/testReport/

Max. process+thread count
2384 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 42s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 25s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 49s
master passed

+1 :green_heart:
checkstyle
1m  4s
master passed

+1 :green_heart:
spotbugs
1m 44s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  0s
the patch passed

-0 :warning:
checkstyle
0m 25s
hbase-common: The patch generated 1 new + 4 unchanged - 0 fixed = 5 total (was 4)

-0 :warning:
rubocop
0m  9s
The patch generated 4 new + 343 unchanged - 4 fixed = 347 total (was 347)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
20m 22s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
2m 14s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 34s
The patch does not generate ASF License warnings.

46m 11s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle rubocop

uname
Linux 96bf31f63b79 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / d963342f8a

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-common.txt

rubocop
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/artifact/yetus-general-check/output/diff-patch-rubocop.txt

Max. process+thread count
94 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/2/console

versions
git=2.17.1 maven=3.6.3 spotbugs=3.1.12 rubocop=0.80.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

PR looks good. The failure seems unrelated in hbase-common (it timed out). There maybe an issue recently introduced unrelated to this PR.

PR looks good. The failure seems unrelated in hbase-common (it timed out). There maybe an issue recently introduced unrelated to this PR.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 23s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 33s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  1s
master passed

+1 :green_heart:
compile
1m 17s
master passed

+1 :green_heart:
shadedjars
7m 36s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 59s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 11s
the patch passed

+1 :green_heart:
compile
1m 18s
the patch passed

+1 :green_heart:
javac
1m 18s
the patch passed

+1 :green_heart:
shadedjars
7m 48s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  3s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 44s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 30s
hbase-client in the patch passed.

+1 :green_heart:
unit
7m  5s
hbase-shell in the patch passed.

42m  1s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 6fa428feb9f3 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 1ca49ea50c

Default Java
AdoptOpenJDK-1.8.0_232-b09

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/testReport/

Max. process+thread count
1486 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 23s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 33s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m  1s
master passed

+1 :green_heart:
compile
1m 17s
master passed

+1 :green_heart:
shadedjars
7m 36s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 59s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 14s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 11s
the patch passed

+1 :green_heart:
compile
1m 18s
the patch passed

+1 :green_heart:
javac
1m 18s
the patch passed

+1 :green_heart:
shadedjars
7m 48s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  3s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 44s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m 30s
hbase-client in the patch passed.

+1 :green_heart:
unit
7m  5s
hbase-shell in the patch passed.

42m  1s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 6fa428feb9f3 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 1ca49ea50c

Default Java
AdoptOpenJDK-1.8.0_232-b09

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/testReport/

Max. process+thread count
1486 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
4m 33s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 26s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
5m  3s
master passed

+1 :green_heart:
compile
1m 24s
master passed

+1 :green_heart:
shadedjars
6m 52s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  7s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  9s
the patch passed

+1 :green_heart:
compile
1m 22s
the patch passed

+1 :green_heart:
javac
1m 22s
the patch passed

+1 :green_heart:
shadedjars
6m 43s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  5s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 47s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m  5s
hbase-client in the patch passed.

+1 :green_heart:
unit
6m 55s
hbase-shell in the patch passed.

44m 19s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 228e745bcf90 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 2e21da7597

Default Java
AdoptOpenJDK-11.0.6+10

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/testReport/

Max. process+thread count
2388 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
4m 33s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+0 :ok:
mvndep
0m 26s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
5m  3s
master passed

+1 :green_heart:
compile
1m 24s
master passed

+1 :green_heart:
shadedjars
6m 52s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  7s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 15s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m  9s
the patch passed

+1 :green_heart:
compile
1m 22s
the patch passed

+1 :green_heart:
javac
1m 22s
the patch passed

+1 :green_heart:
shadedjars
6m 43s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  5s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 47s
hbase-common in the patch passed.

+1 :green_heart:
unit
1m  5s
hbase-client in the patch passed.

+1 :green_heart:
unit
6m 55s
hbase-shell in the patch passed.

44m 19s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 228e745bcf90 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 2e21da7597

Default Java
AdoptOpenJDK-11.0.6+10

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/testReport/

Max. process+thread count
2388 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 10s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 24s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 45s
master passed

+1 :green_heart:
checkstyle
1m  7s
master passed

+1 :green_heart:
spotbugs
1m 49s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 30s
the patch passed

-0 :warning:
checkstyle
0m 23s
hbase-common: The patch generated 1 new + 4 unchanged - 0 fixed = 5 total (was 4)

-0 :warning:
rubocop
0m  8s
The patch generated 4 new + 343 unchanged - 4 fixed = 347 total (was 347)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
18m 46s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
2m 27s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

44m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle rubocop

uname
Linux c47b27e3ebf4 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 1ca49ea50c

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-common.txt

rubocop
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-general-check/output/diff-patch-rubocop.txt

Max. process+thread count
94 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/console

versions
git=2.17.1 maven=3.6.3 spotbugs=3.1.12 rubocop=0.80.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 10s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 24s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 45s
master passed

+1 :green_heart:
checkstyle
1m  7s
master passed

+1 :green_heart:
spotbugs
1m 49s
master passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 13s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 30s
the patch passed

-0 :warning:
checkstyle
0m 23s
hbase-common: The patch generated 1 new + 4 unchanged - 0 fixed = 5 total (was 4)

-0 :warning:
rubocop
0m  8s
The patch generated 4 new + 343 unchanged - 4 fixed = 347 total (was 347)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
18m 46s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
2m 27s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 38s
The patch does not generate ASF License warnings.

44m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2812

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle rubocop

uname
Linux c47b27e3ebf4 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 1ca49ea50c

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-general-check/output/diff-checkstyle-hbase-common.txt

rubocop
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/artifact/yetus-general-check/output/diff-patch-rubocop.txt

Max. process+thread count
94 (vs. ulimit of 30000)

modules
C: hbase-common hbase-client hbase-shell U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2812/3/console

versions
git=2.17.1 maven=3.6.3 spotbugs=3.1.12 rubocop=0.80.0

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Typescript support for V2 orders

There is typescript autocomplete for the V3 orders endpoint but not for V2 orders. Adding this would allow for more details on orders that aren't in the V3 endpoint
https://developer.bigcommerce.com/api-reference/store-management/orders/orders/getanorder

It looks like there are types for that endpoint just as there are for other endpoints. I think the issue you're having might be with some changes we made a while ago to allow requests to unknown paths (e.g. private beta endpoints).
The effects of the above change are:

No autocomplete on the params for any request
If the specified params do not match what is expected, Promise<unknown> is the resolved return value of the function

Support for requests to unknown paths probably needs rethinking to avoid these issues.

Apologies, looks like I was using an older version of the package (0.1.5) and so the orders endpoint wasn't in the autocomplete suggestions

Upgrading to 0.2.5 has resolved the issue


[regresstion][external]fix case test_show_where and es_query 0811

Proposed changes
Issue Number: close #xxx

fix case test_show_where and es_query 0811
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run external

LGTM


question: Is there an image to video option?

Like the video to video option Is there an image to video option or workflow?

There is no image2video version of the model released publicly, only text2video. It's something that needs to be trained in.


input[url] validation regex is not valid

In v1.4.8 the validation scheme for url inputs was changed, causing valid URLs with semicolons, which previously passed the filter, to become invalid
Our application handles the generation of long urls with messy query strings to be used in advertising, so it's important to us that esoteric characters remain supported. Semicolons are valid URL characters
You can reproduce the error at the angular code example for a url input, try entering
http://www.test.com/asdf;df
http://www.test.com/asdf?asd=f;df=df

The fix is simply to add a semicolon to the last chunk of the validation regex here

Sounds like a real bug. Apart from ; there seem to be a few more characters that should be allowed in the URL_REGEXP: [, ], $, ', (, ), *, ,, ;, ~
I will take a closer.
--
In the meantime, a little context: The ; character wouldn't have been allowed by our URL_REGEXP in previous versions, but another bug in the regexp shadowed this issue. That other bug was fixed as part of cb51116 in v1.4.8, revealing the ; issue (bug?).

@gkalpak @Narretz I've opened a PR against angular:v1.4.x - please let me know if it should be against a different branch and I'll create a separate PR.

Thx @andypatterson !
For future reference:
Usually, you should submit the PR against the master branch, which is where development is mainly taking place, and we will cherry-pick it into specific older version branches if necessary/appropriate.
You should still submit against a concrete version branch, if it's a bug/feature specific to that version (e.g. it is already solved or not applicable on master).
(You don't need to submit a new PR for this, we'll take care of cherry-picking it into master.)


With API create invoice w/ description hash.

Discussion: https://github.com/ACINQ/phoenixd/discussions/45

Superseded by #50.


[ux-lazy-image] data_uri_thumbnail does not recognize relative path

This does not work:
src={{data_uri_thumbnail('public/res/images/backgrounds/zzz.jpg', 100, 75)}}
Both of these (absolute url and absolute path) work:
src={{data_uri_thumbnail(absolute_url(asset('res/images/backgrounds/zzz.jpg')), 100, 75)}}

src={{data_uri_thumbnail('/Users/xxx/dev/projects/yyy/public/res/images/backgrounds/zzz.jpg', 100, 75)}}

Error:
An exception has been thrown during the rendering of a template ("Warning: file_get_contents(public/res/images/backgrounds/zzz.jpg): Failed to open stream: No such file or directory").
Versions:
Symfony 5.3
"symfony/ux-lazy-image": "^1.3"

My bad, no need for absolute_url, the code that works is:
src={{data_uri_thumbnail(asset('res/images/backgrounds/zzz.jpg'), 100, 75)}}


Multi-Binary action space

Are you planning on supporting games with multi-binary action space anytime soon?

I guess this refers to #44.

Helped a lot, thanks!


Display Revert On My Lcd Screen

i ported lvgl to my board by followed the official guide,  it did show the demo image but the image's orientation is invert on the horizon, for example string "Open" shows out to be “nepO”
what's wrong?

Hi,
Are you using a display controller (like ILI9341) or the MCU's LCD driver periphery?

AS there wasn't activity here for months I close this issue.
If you have a similar question feel free to comment here  and reopen the issue!


Can't start machine anymore

PS C:\Users\someb> docker-machine start default
Starting "default"...
(default) Check network to re-create if needed...
Error setting up host only network on machine start: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe modifyvm default --nic2 hostonly --nictype2 82540EM --nicpromisc2 den
y --hostonlyadapter2 VirtualBox Host-Only Ethernet Adapter #3 --cableconnected2 on failed:
VBoxManage.exe: error: Code E_FAIL (0x80004005) - Unspecified error (extended info not available)
VBoxManage.exe: error: Context: "LockMachine(a->session, LockType_Write)" at line 507 of file VBoxManageModifyVM.cpp

There were 2 virtual network adapters in the windows network center. When i tried to open settings in virtualbox it throw unknown error and provided completely blank fields on all tabs.
Probably some configuration files were corrupted, maybe implement some configs and important files backups to restore to previous versions and re-try requested operation? Notifying user with y/n option about launching backup version etc.


#1: add support for horses in database initialization

Nuevo-cambios

/paulo - can you please review this pull request?

/paulo - can you check now?


Correction for raw affiliation UFR Droit et Sciences Politiques, University of Nantes, Nantes, France

Correction needed for raw affiliation UFR Droit et Sciences Politiques, University of Nantes, Nantes, France
raw_affiliation_name: UFR Droit et Sciences Politiques, University of Nantes, Nantes, France
new_rors: 014nzh720;03gnr7b55
previous_rors: 014nzh720;05fe7ax82;03gnr7b55
works_examples: W4205255980;W4297911869
contact: 8289190e249fd128e0c6c02c466775ca:a8b0db8b509dcbc50cb2d12ffa464f @ univ-nantes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


feat: add Text.toLowercase, Text.toUppercase with tests and doc

(tested locally against https://github.com/dfinity/motoko/pull/4216)

Marked DO-NOT-MERGE until weird wasi bug on https://github.com/dfinity/motoko/pull/4216 squashed


Installation via NPM won't work

Running
npm install eslint-plugin-checked-exceptions --save-dev

results in
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/eslint-plugin-checked-exceptions - Not found
npm ERR! 404
npm ERR! 404  'eslint-plugin-checked-exceptions@*' is not in this registry.

Hi, thanks for creating an issue!
I have not pushed this to NPM as there are some issues to be ironed out. I'm not planning on maintaining this project, unfortunately.


SPC w C-c is bound to undefined command ace-delete-window

What did you expect to happen?
I expected to be able to select a window to delete.
What actually happened?
"Undefined command" error.
Additional details:
It looks like ace is either not installed or not active, because none of its commands are defined in my Emacs session. I've just recently installed Doom and don't have any customizations in my .doom.d.
Messages buffer:
## after pressing SPC w C-c:
command-execute: Wrong type argument: commandp, ace-delete-window
## after pressing F1 k SPC w C-c:
user-error: SPC w C-c is bound to ace-delete-window which is not a command

System information:
https://pastebin.com/70XFKbEk

Looks like ace-window is installed as part of the :ui window-select module. Maybe that code was leftover, and should be ported to the :ui window-select module as well?


🐞 [BUG] - NavBar alignment and about page

Issue Verification

[X] I have verified that this is a new bug report.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Describe the bug
Fix navbar alignment and add content in about page

Expected behavior
No response
Steps to reproduce
No response
Additional Context
No response

Hello @jfmartinz  , I want to work on this issue. Can you please assign it to me?


Faster batch scaler tests

As discussed on slack the two slowest test has to do with the batch scaler.
This PR tries to bring their test time down.

@Borda I moved some of the batch scaling test from cpu to gpu (where they probably should have been all the time). It reduces the time of the two longest test by around 40 sec each, so total reduction by 1:30 min.

@SkafteNicki can we also make the test smaller, just less data or larger steps?

we could limit the size of the training set since after PR https://github.com/PyTorchLightning/pytorch-lightning/pull/3271 the search will stop when batch_size>=len(train_dataset), meaning lower test time

@SkafteNicki I was thinking the same yesterday but then on the other hand, part of the test is also to check that the OOM handling goes correctly. so we would sacrifie that.

@awaelchli also what I was thinking. Only other way to make the search stop earlier if we artificially filled up the gpu memory before running the search. Else let keep it as it is now (still reduced significantly compared to before).


Separate NAT-T hint string from hostname internally

Hint string is a "hack" used to provide additional information for NAT traversal.
The common ones are:

example.com/tcp
Used to indicate no NAT-T should be used
example.com/<private ip>
Useful when multiple servers are under the same global address

However, since the internal process does not separate the real hostname from the hint string, it is necessary to consider the existence of the hint string everywhere the hostname is used. The current approach is cumbersome and error-prone.
For example, I have seen these strange behaviors:

example.com/tcp is sent for DNS resolution
example.com/tcp is present in TLS SNI

Changes proposed in this pull request:

Separate NAT-T hint string from hostname internally
User-facing interface is unchanged

Some illustrations that may help you better understand my proposal.
Current:
                                 vpn_client.config
                                Host: example.com/tcp
                                         ^
                                         |
                          RPC            |            TCP/UDP
  Client Manager / CMD   <--->         Client          --->         Server
 Host: example.com/tcp          Host: example.com/tcp       TLS SNI:    example.com/tcp (BAD)
                                                            DNS Lookup: example.com example.com/tcp (mixed) (BAD)
                                                            NAT-T Hint: tcp

New:
                                 vpn_client.config (UNCHANGED)
                                Host: example.com/tcp
                                         ^
                      (UNCHANGED)        | 2
                          RPC            |            TCP/UDP
  Client Manager / CMD   <--->         Client          --->         Server
 Host: example.com     1      1 Host: example.com           TLS SNI:    example.com
 Hint: tcp                      Hint: tcp                   DNS Lookup: example.com
 To user: example.com/tcp 3         (SEPARATED)             NAT-T Hint: tcp
 (SEPARATED INTERNALLY)

Changes are made at these points:

RPC communication
InRpcClientOption  Read from RPC
OutRpcClientOption Write to RPC
I added a hint field in ClientOption. The RPC format is unchanged to be compatible with old clients.
When we read from RPC, hint string is stripped. When we write, they are put back.
Client configuration file
CiLoadClientOption  Load from configuration
CiWriteClientOption Write to configuration
Similar to 1, the configuration file is unchanged. Only internally we now have a hint string field.
User operations
CtEnumAccount          Get account list
CmEditAccountDlgUpdate Save account in client manager
CmEditAccountDlgInit   Load account in client manager
PcAccountGet           Load account in vpncmd
PcAccountSet           Save account in vpncmd
PsCascadeGet           Same as PcAccountGet
PsCascadeSet           Same as PcAccountSet
These are operations performed by users. The idea is that the user will continue to see and enter example.com/tcp but internally we separate store them.
Now that host and hint are separated. The DNS resolution and TLS SNI will only take the real hostname. Hint is only passed to the NAT-T server.

Closing in favor of #1522


Battery status methods

Is it possible to methods to retrieve the scanner's battery statuses?
i.e. scanner battery level and charging status

I'm not sure. I don't have access to a scanner anymore, so I can't update anything


Fix inter-asset margins

Before:

After:

Native Photos.app:

Let me know what you think!

Thanks!


Scanning images so slow

I dont know why I register by posting Images and it scanning for so long

What does "so long" mean? 1 min? 10 min? 2 hours?
Anyways the first scan takes a bit longer because the model is loaded into the browser cache, the next scans are very fast! So it depends also on the memory cache you have available. The tests I did on my old Mackbook Pro (2019) 16GB RAM, took 4/5 secs for the first scan and less than 1 sec for the next ones.

yeah, it take about 3-5 mins on windows, but when i run it on Linux, it's much more faster. Thanh you so much


⚠ Bofrost.at has degraded performance

In 603bfa5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1352 ms

Resolved: Bofrost.at performance has improved in d272c9e.


Allow to set custom template path

By default is sao.js exists the template path would be ./template otherwise it would be ./, I want to introduce a templatePath option for setting this manually, for example I want to copy ./ but also have sao.js

Has this been done or when is it scheduled for? Should be a very small/easy change to make?

@kristianmandrup ah yeah, but as templateFolder https://sao.js.org/#/create?id=templatefolder


When would you release the TensorFlow Implemention?

Expect the tensorflow interface

@guyibang @SoulCoderCN Hi all, we have implemented the TensorFlow version of our fast MPN-COV, but the code is not evaluated for reproducing the results as reported in our paper. We'll finish it as soon as possible. :)

@jiangtaoxie Hi,
May I ask if the Tensorflow implementation is ready ? I believe it will help many researchers who are following your great work. Thanks.

@WeitaoVan Thanks for your attention to our work! There are some problems in our current Tensorflow implementation, which cannot reproduce the results as our paper reported. As such, we are not ready to release Tensorflow version until it works fine.

TensorFlow implementation (https://github.com/XuChunqiao/Tensorflow-Fast-MPNCOV)


Node 11 support

Hi,
Could we expect a version bump soon now that https://github.com/googleapis/nodejs-text-to-speech/pull/132 has landed and "grpc": "1.16.1" has been published (node 11 support)?
Cheers,
Daan

This was fixed in the 0.4.0 release


ADD: get maturing height (js + andr)

@marcosrdz can you pls add swift counterpart

@marcosrdz can you pls add swift counterpart

on it


Drop dependency on routecore.

This PR drops routecore as a dependency and includes all the code from that crate we need for RPKI.
Having a shared dependency between the RPKI and BGP projects was a bit premature and the overlap is relatively small. Keeping things separate removes some administrative burden on the BGP projects during their early stages, so this seems the right move. We can always reconsider later.
The PR moves what used to be routecore::{addr, asn} to crate::resources::asn androutecore::bgpsec::KeyIdentifier to crate::crypto::keys::KeyIdentifier (where it was re-exported already previously).

Good point. Let’s do this in a separate PR. I also want to update the base64 dependency which will probably result in a util::base64 module.

Sounds good! Then I can probably also remove krill's dependency on the base64 crate.


suppress errors in the fast loading path

we might call redirect_stderr at https://github.com/Suzhou-Tongyuan/jnumpy/blob/65cea6cc6a750911ab2c1545d94af26008e7b240/jnumpy/init.py#L172

It seems error message comes from:
https://github.com/Suzhou-Tongyuan/jnumpy/blob/1d7e2007abb10fcc29cb4efb0e190c4829bd40f6/jnumpy/apis.py#L87-L89
use_gil=False should not print out error.

done by #58


Missing content for flat pages, various error fixes

This is a relatively small branch that:

Adds missing content from the Guidelines page
Migrates Bridgewater landing page from old codebase
Fixes linter warnings related to flat pages

@elisescu I'm guessing the backend one is an unrelated error, should I wait until it's fixed on main?

Yeah, it should not be related. Probably somebody merged their broken PR before letting the checks to finish. If this is high prio, feel free to merge, as it should be harmless.


Test PR to see if mentionUser works

Internal previews

📄 File
🔗 Preview link

docs/fundamentals/code-analysis/categories.md
docs/fundamentals/code-analysis/categories

So it seems that having requestReview in the policy (which didn't work) prevented mentionUser from working.

This is weird - there was a comment @mentioning Dina yesterday, but now it is gone!


add node style shape rectangle

Click on the edge of the element to drag the area Bug

I release v1.1.1 with this change. Sorry for the delay.


Add project charter to the repo

Created charter.md that contains AEDES project charter
Closes #3

Looks good. Closing.


Saddle Finance Withdraw event

The From & To field of the Withdraw event should be swapped. Documentation reads:
Address that received the tokens
to: String!
Address that sent the tokens
from: String!
I would expect a with draw from (pool address) to (user address). this is backwards as currently implemented.

@spitko would you be able to take a look at this?


Generic writer docs update

added timeout and ssl verification features
@hhanova

@odinuv porfavor can you review?


Add new app - showing todays name in Sweden

A small app that shows this days name that celebrate "Nameday" in Sweden

I have read the CLA Document and I hereby sign the CLA

recheck

It is good to rename it ti "Swedish Name Day" and change the id, can you do it?

I have renamed the app to Swedish Today's name and also changed the id to swedish-todays-name


Allow subdir serving

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I'm not 100% sure what's going on here but it seems like you're adding a feature that is not part of openvscode-server. Wouldn't it make sense to open a PR in https://github.com/gitpod-io/openvscode-server instead?

It is very unlikely to be merged there. If it works, I will still open a PR over there but for now, I would first like to get the build working.


Flyway cannot run on a newly created AWS RDS PostgreSQL

Which version and edition of Flyway are you using?
7.12.0
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Maven plugin.
Which database are you using? (Type & version)
PostgreSQL on Amazon RDS.
Engine version 12.
Which operating system are you using?
Linux
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
run Flyway migration when the application starts up.
What did you expect to see?
Flyway migration finished successfully.
What did you see instead?
Caused by: org.flywaydb.core.api.FlywayException: Found non-empty schema(s) "public" but no schema history table. Use baseline() or set baselineOnMigrate to true to initialize the schema history table
There is a function called db_session_replication_role(text) created by Amazon RDS, which causes Flyway to regard the public schema as non-empty.

Closing until further activity


Refactor logs capture and faster logs functional testing

Overview of changes:

Refactor logs artifact production into a new service that implements a factory and operation interface. This pattern can be used across all artifact types in a future change, which would allow simplification of the creation of egress operations and http response writing to not have any specific knowledge or callback implementations.
The logs functional tests execution time is greatly reduced due to the introduction of a debug logging event notifying when logs capture has started and issuing sentinel logging events at appropriate timings to tell the test infrastructure to end. On my local machine, the logs functional tests used to take 12-13 seconds per test invocation in Visual Studio Test Explorer whereas with this change they take about 2-3 seconds per test invocation, a 77-83% reduction in execution time.

/azp run dotnet-monitor-ci

/azp run dotnet-monitor-ci

/backport to release/7.x

/backport to release/6.x


Web - LHN last message shows as multiline after the last message is flagged as Harassment/Assault

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

Action Performed:

User A sends a multiline message to B
User B flag the message as Harassment/Assault
Observe the LHN last message

Expected Result:
Last message shows as a single line text
Actual Result:
Last message shows as a multiline text
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:  1.3.37-6
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/e036ca84-337f-4f7f-999d-128e996a2224
https://github.com/Expensify/App/assets/93399543/98d1efa0-b9ec-462b-be3d-aa62a2b7d3f0
Expensify/Expensify Issue URL:
Issue reported by: @bernhardoj
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688727004082269
View all open jobs on GitHub

Lots of high priorities. Getting to this tomorrow


Configuring Deploy Widget

Hi Team,
I have configured the deploy widget successfully, currently, we are using 2 environments Dev and QA.
a.) How to configure more than 2 environments in deploy widget .
b.) In Deploy widget we find two arrows(up & down) could you please explain us what value need to be populated , currently it shows us 0.
Please clarify .
Thanks in Advance .

PrinceVijay ,can you please help me for initial setup, unable to bring the api itself up, took latest & did build

@princevijayakumar are you using the Jenkins plugin?

Hi @princevijayakumar It will be helpful if you would like to share the deploy widget configuration process in detail. Thank you.

@princevijayakumar  @supremex1
a) if using Hygieia jenkins plugin, you can configure Post build actions for Jenkins job -> Configure -> Post build actions -> select Hygieia Publisher, for each Jenkins job specific to environment. This will help Hygieia plugin to push all the data to your dashboard.
Check Publish Deployment Resulyts
Check Deploy Start Status
Application Name:
Environment Name : Dev/Int/Perf/Prod/QA
Artifact Directory :  //
Artifact File Name Pattern : -.jar
Atrifcat Group : details found in pom.xml
Artifact version : details found in pom.xml
b)  Up & down arrows specify the count of servers online/offline.

@princevijayakumar Closed. Reopen if still an issue.

@nireeshT the Up & down arrows in deploy widget specify the count of servers online/offline,but I dont understand what are the servers online/offline.Could u please explain this?

@tabladrum the Up & down arrows in deploy widget specify the count of servers online/offline,but I dont understand what are the servers online/offline.Could u please explain this?


Update to a stable version of the dep

Coverage remained the same at 100.0% when pulling 06e30bc778c013ffc27e6e45d8a9614a54e0b4d4 on mikeSimonson:patch-1 into 6a90213c57596e90f36b05424220d819a2853580 on nickel715:master.

thanks, merged!


I want to display the grid cell.

I found this image from a search, but I want to display the detection results in a grid cell like this one.
Can someone please tell me how to do this?

Don't know


RadioGroup not setting form touched on focus out

Detailed description

Describe in detail the issue you're having.

On focus out the form is not getting set as touched, only on value change

Is this a feature request (new component, new icon), a bug, or a general issue?

bug

Is this issue related to a specific component?

RadioGroup

What did you expect to happen? What happened instead? What would you like to see changed?

on focus out without any value change the form should result as touched

What browser are you working in?

any

What version of the Carbon Design System are you using?

latest

Closing this as it's fixed in https://github.com/IBM/carbon-components-angular/pull/1971
(forgot to link it in the PR)


nodePackages: update package set

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

I support this, but I can't test this due to the fact that it takes for ever to build on my machine.
Personally this is good to merge.


How to correctly apply griffin-lim vocoder?

Hi,
I wanted to add some example audio to the tensorboard during training to help monitor quality and so I thought I'd add this using griffin-lim.
One thing that I'm a little unclear about is how to get the linear spectrogram needed for griffin-lim from the mel spectrogram.
The mel-spectrogram is created by multiplying the magnitude part of the  linear-spectrogram by the mel-basis.  i.e mel_spec = torch.mm(mel_basis, magnitudes).
This seems to imply that to get back the linear spectrogram you should multiply the spectrogram by something like the pseudo-inverse of the mel-basis? But that isn't what seems to be done in the example.
Please could you help me with my misunderstanding.
many thanks!

https://github.com/NVIDIA/tacotron2/issues/41#issuecomment-397439290


Improvements | String Literals on the left on comparisons (SonarQube)

Transforming from someString.equals("string")  to "string".equals(someString). This avoids checking for null and also prevents NullPointerException being thrown.
These fixes multiple SonarQube violations of the rule: Strings literals should be placed on the left side when checking for equality

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@156267c). Click here to learn what that means.
The diff coverage is 80%.

@@          Coverage Diff           @@
##             dev    #1077   +/-   ##
======================================
  Coverage       ?   53.07%
  Complexity     ?     3167
======================================
  Files          ?      119
  Lines          ?    28118
  Branches       ?     4694
======================================
  Hits           ?    14925
  Misses         ?    10961
  Partials       ?     2232

Impacted Files
Coverage Δ
Complexity Δ

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
68.36% <0%> (ø)
169 <0> (?)

...oft/sqlserver/jdbc/SQLServerPreparedStatement.java
53.31% <100%> (ø)
216 <0> (?)

...m/microsoft/sqlserver/jdbc/SQLServerDataTable.java
81.75% <100%> (ø)
42 <0> (?)

...oft/sqlserver/jdbc/SQLServerParameterMetaData.java
73.86% <100%> (ø)
103 <0> (?)

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 156267c...2fcff87. Read the comment docs.

Hi @dvmarcilio
There are more occurrences of such comparisons that you may wanna fix altogether for this rule.

I'm covering all occurrences that showed up in Sonar. Unfortunately, the building is failing, and the error message does not provide much information:

[ERROR] testDatabaseMetaData  Time elapsed: 1.512 s  <<< ERROR!
com.microsoft.sqlserver.jdbc.SQLServerException: Transaction (Process ID 56) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.	at com.microsoft.sqlserver.jdbc.connection.DBMetadataTest.testDatabaseMetaData(DBMetadataTest.java:52)

Is that on my end? Any directions? Thanks!

Nevermind the previous comment. The building is passing now.

hi @dvmarcilio,
There are some conflicts in this branch if you could please resolve it we can review and look into merging it. Thanks!

Hi @lilgreenbird. I resolved the conflicts. I think the build error is not on my end.
Please let me know if there is anything else I can do.

Thanks for the contribution, @dvmarcilio 👍.


remove augmentation in data loader

deepmind implementation does not use data augmentation.

Misread your README. Seems like things already work and the differences lie in init?


macOS Ventura (and related) test fixes

fix check_for_config_scripts test, no idea why this wasn't working
don't autoretry tests if focus or byebug are enabled
add a ARM64 Ventura bottle symlink

Review period skipped due to critical label.


filezilla: Update manifests and change source

Readd checkver/autoupdate by switching the sources to Chocolatey. This would remove the need for someone to actively upload the binaries to the Binary repo.

Relates to https://github.com/ScoopInstaller/Binary/issues/20

[x] I have read the Contributing Guide.

/verify

Make a PR for delete old version of them in binary repo as well.


User wants to search for address

I want to be able to search for an address to find a relevant.

As an example -  there is a search function in Zoning Index Map page - http://www1.nyc.gov/site/planning/zoning/index-map.page. You can search for a location using BBL (Borough, Block, Lot), BIN (Building Identification Number) or Address. It uses Geoclient API from NYC Developer Portal.

Recommend using a consumer-ready geocoder with autocomplete suggestions such as Mapzen's or Mapbox's if we want to do address lookup.
I support BIN and BBL lookup in general, but wonder if these are relevant to users of the community portal.  Do we have any stats on how much these types of lookups occur on the existing pages that use them?
I think requiring the user to enter a borough prior to address search is a poor user experience for quickly locating an address on the map.

Duplicate of #133


Users management

This cookbook could use a users.rb recipe that handles the creation and modification of users. Including the support for ssh, ssh authorized keys, and passwords. Perhaps even the modification of sudoers.

Addressed by https://github.com/copious-cookbooks/users and https://github.com/copious-cookbooks/sudoers


⚠ ClamAV has degraded performance

In 4626a1e, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in 437ed8a.


Visualization in the rectangular section does not work

When I click visualize, the inscription "error" just appears at the bottom. Here are the logs from the console:
Traceback (most recent call last):
File "G:\stable-diffusion-webui\venv\lib\site-packages\gradio\routes.py", line 422, in run_predict
output = await app.get_blocks().process_api(
File "G:\stable-diffusion-webui\venv\lib\site-packages\gradio\blocks.py", line 1326, in process_api
data = self.postprocess_data(fn_index, result["prediction"], state)
File "G:\stable-diffusion-webui\venv\lib\site-packages\gradio\blocks.py", line 1260, in postprocess_data
prediction_value = block.postprocess(prediction_value)
File "G:\stable-diffusion-webui\venv\lib\site-packages\gradio\components.py", line 4457, in postprocess
file = self.img_array_to_temp_file(img, dir=self.DEFAULT_TEMP_DIR)
File "G:\stable-diffusion-webui\venv\lib\site-packages\gradio\components.py", line 355, in img_array_to_temp_file
return self.pil_to_temp_file(pil_image, dir, format="png")
TypeError: save_pil_to_file() got an unexpected keyword argument 'format'

Me too.  I'm a new installer.  The rest seems to work ok, just the visualizer doesn't.

daam, same problem, I hope someone help... Has anyone tried installing an older version?

Temporary solution:
Under your Stable Diffusion path, find the venv/Lib/site-packages/gradio/components.py
Modify line 355
return self.pil_to_temp_file(pil_image, dir, format="png")
to
return self.pil_to_temp_file(pil_image, dir)
Save and restart
Now it works fine

Temporary solution: Under your Stable Diffusion path, find the venv/Lib/site-packages/gradio/components.py Modify line 355 return self.pil_to_temp_file(pil_image, dir, format="png") to return self.pil_to_temp_file(pil_image, dir) Save and restart Now it works fine

Thanks! Now it works


Count metric for completed jobs

Here you go @JohnLZeller . This is the pull request for the 'count' metric.
The PR also includes the changes for the tags...which this requires. The relevant part to review would then be in the second commit of this PR.

Hey @MadsNielsen!
Okay, so this PR works, but it is not the ideal way of doing this. First of all, it's not entirely accurate to use a gauge in such a way, and it'll require using rollups and things on a dashboard to make it work.
Additionally, multiple jobs finishing at the same second will not be counted correctly.
This works fine as a forked hack if someone wants to utilize it, but I have an alternative suggestion.
Let's shoot out a UDP packet to the local Dogstatsd (packaged with the datadog agent). If it's not there, the UDP packets will fall on nothing, if it is there, then the counts will be pushed up to Datadog.
Let me know if I can clarify anything here for you.

@JohnLZeller Sounds fine by me. I assume this is documented in your API docs so i can implement it?

@MadsNielsen It's not in the API docs actually, because it's an agent feature, not a feature of the API.
http://docs.datadoghq.com/guides/dogstatsd/

Hey @JohnLZeller
I've started working on this, and a couple of questions popped up.

We probably want an url // port configuration for the statsdaemon jenkins should communicate with. We can the just default to localhost // 8125

What about tags, i assume we need to attach all the same tags as we do to the build finished. Atleast job name so you can group by job name with this counter.

But i think i got the overall idea on how this works so i'm working on getting this done :)

Hey @JohnLZeller
Another question popped up: What is your thought of using one of the excellent client libraries you're linking to in the plugin?
More specifically im lookin at this: https://github.com/indeedeng/java-dogstatsd-client

@MadsNielsen Yeah, but let's use our own client fork, https://github.com/DataDog/java-dogstatsd-client

Yeah let's add this as an optional global configuration, with those defaults. Let's hide this behind an Advanced options dropdown.

Yeah, let's include our tags.

@MadsNielsen once you've implemented the new functionality we spoke of, would you rebase this PR from master (I merged your other changes)? Thanks :)

@JohnLZeller Okay. First draft for this feature is ready. I used your client fork for this part. The configuration now includes an Advanced section where you can specify the address to your daemon.
I found the Client reasonably ok to work with, one issue i had initially was that i just couldn't just create a new client in the onCompleted method. Since a run listener is global (one instance per Jenkins) i found that if i created the client there, and stopped is at well...it would cancel out the udp requests... :)
So...i think i figured out how to do it properly. When the global configuration page is saved. We need to pick up eventual configuration changes and create a new client for our run listener to use. And when we use the Client in the run listener, we can ask the descriptor to give us the current configured instance...

@MadsNielsen okay there is my first round of feedback.  I tried as best to test this, i think i had a setup with like 10 jobs running at intervals pushing events to datadog. I'll wait ammending this PR until you've finished your validation inhouse so that we can finish it up in a single go.

@MadsNielsen looks good, other than the nitpicks :)

@JohnLZeller great! Im on vacation now and Will be back after next weekend. I'll fix it then. Is this ok for your release schedule.?

@MadsNielsen we were hoping to release this a bit sooner actually. Would you mind if I rebased your commit and made the small changes? The commit will show up as a pair commit, but you'll still have attribution on the things you changed, and in the commit user.

@MadsNielsen here is the final PR, with your permission I'd like to merge this https://github.com/DataDog/jenkins-datadog-plugin/pull/43

Merged #43

Great you got it fixed! @JohnLZeller im perfectly fine with you fixing the issues you pointed out in your review while i am away😁


Fuel syntax updates etc

Here is another bunch of fuel fixes, mostly in the syntax highlighting area.

Merged!!


feat: extend the linter against broad imports

Also lint on importing Tactic.{Have,Replace} or any file in the Deprecated directory.
Both of these are not supposed to be imported; this linter enforces it.

This PR should be rewritten on top of the header linter: let's land that one first, then this PR can be re-considered.

@grunweg what is the header PR you mentioned? The bot automatically removed the blocked label because there was no depends on line in the PR description.

@j-loreaux That was #16574: that PR has since landed, and this PR has been adapted accordingly. (I removed it from the PR description after that, so it's not visible any more; sorry for the confusion.)
In other words: from my side, this PR is ready for review.

LGTM.
maintainer merge

bors merge


Clarification on usage of https://auth.acme-dns.io

I couldn't find any information if https://auth.acme-dns.io is official and can be used in production? Or is it just a demo and could go away anytime? Would be great to have this clarified in the readme or at  https://www.acme-dns.io. Thank you.

Pretty sure you can use it in production, but if you do you're technically giving the owner of that domain the ability to issue certs for your website.
In the future it might be possible to fix that using ACME CAA extensions, but until that's implemented I'd recommend against using auth.acme-dns.io in production.

Let's Encrypt is just a CA. They don't develop ACME clients or run servers for third-party tools.
I don't know who runs auth.acme-dns.io. Probably @joohoi, I assume.

Yes. I'm running acme-dns.io and the statement in README.md stands true. However there is something brewing that would address the trust issue once and for all. validationmethods extension to CAA, RFC draft of which you can check out. It's not yet deployed to Let's Encrypt production, but it available on staging environment. You can read more about this from Let's Encrypt community forum post.
TL;DR: CAA extension allows you to redstrict certificate issuance to a specific ACME account, making it feasible to use third party service for providing the TXT record as the issuance is restricted to your specific local ACME account.

@joohoi Thank you for providing auth.acme-dns.io in the past as a simple solution for the ones that don't want to run the whole system. I noticed it is not working now. The DNS isn't resolving. Does that mean you've discontinued service and we should migrate?
nslookup auth.acme-dns.io <IP_ADDRESS>
Server:		<IP_ADDRESS>
Address:	<IP_ADDRESS>#53

** server can't find auth.acme-dns.io: SERVFAIL

I've been considering offering a low-cost hosted acme-dns option for a while now, mainly to service users of https://certifytheweb.com but it could work for other clients. If auth.acme-dns.io is permanently offline that may become more of a necessity, I'd possibly also be willing to host it depending on the type of costs you're seeing in your current implementation.

auth.acme-dns.io is back up now. It was left offline because of hurried maintenance tasks before midsummer festives (a big thing in Finland). When the CAA extensions draft gets finalized and enforcing of it is turned on for Let's Encrypt, I'm planning on hosting a public instance on a more fault tolerant infrastructure.
That said, I'm planning to keep the auth.acme-dns.io running even in its current state regardless.

Thank you sir

@joohoi  Thank you for providing auth.acme-dns.io service. However it is not working now (DNS isn't resolving). Can you please tell me if it is planned to resume its operation, and if so, when? Thank you!

It works! :)


Allow multiple target_pools or backend_services on a instance group

To support cross-regional deployments, it would be nice if I could pass a list of target_pools or backend_services and the CPI would attach any of them that makes sense based on the region and/or AZ of each VM.
For example:
instance_groups:
 - name: cloud_controller
   azs: ["west1a", "west1b", "central1a"]
   instances: 3
   cloud_properties: # via vm_extensions
     target_pools: ["west", "central"]  # with central likely being a backup pool in the load balancer
 - name: router
   azs: ["west1a", "west1b", "central1a", "central1b"]
   instances: 4
   cloud_properties: # via vm_extensions
     backend_services: ["westsvc", "centralsvc"]  # this time we'd probably go global distribution between svcs

When the CPI booted the 3 cloud controllers, it should obviously only attach the pool from the "west" region to the VMs actually in the west region and same with central.  Likewise with the backend services.

@evandbrown we now have another request for 2 load balancers on 1 job.  What do you think?