"Show unit path" create problems to lens

Describe the bug
When "show unit path on hover or selection" is enabled, if you open a lens and hover a unit, lens disappears and doesn't return when no longer hovering, but interfaces shows that lens is enabled.
To Reproduce
Steps to reproduce the behavior:

Set a unit on a long path (more than one turn)
Activate a lens (anyone)
Hover the unit with long path
Lens disappears
No longer hover - lens remains not visible

Expected behavior
Not sure if is intended that lens disappears; if is intended, it should return visible when no longer hovering.
Note that Religion lens has a different behavior: it seems to NOT disappear (color remains) but all other information (city religion ratio, etc) disappears.
Screenshots

Initial situation

Hovering on a unit (lens disappears)

no longer hovering (lens does NOT reappear)
General informations

OS: Win
Version <IP_ADDRESS>
Expansion: None
Other mods: none

Save game and logs
Logs.zip
TRAIANO 325 1904 d.C..Civ6Save.zip

This has been there a long time. It would be nice to see this fixed. Didn't realize it was tied to showing unit path on hover. Maybe have an option to only show unit path on select as an easier work around?

I will look into this - sorry for not commenting earlier!

The specific problem is that the Active Lens is changed and never restored.  The RealizeMovementPath function is basically a copy of the Firaxis version of it, with the addition of a little bit of logic to select a unit based on the mouseover.  Under normal circumstances, the code proceeds to this section:
            -- Turn on lens.
            if not UILens.IsLensActive(lensNameBase) then
                UILens.SetActive(lensNameBase);
            end

And calls UILens.SetActive with "MovementQueue" as the value of lensNameBase.  Once this call happens, it wipes out whatever other lens is showing.
I tried a couple of things, with some success, though it's not perfect.

Near the beginning of RealizeMovementPath, call UILens.SaveActiveLens(), which basically saves off the name of whatever lens is currently applied.  In ClearMovementPath, call UILens.RestoreActiveLens() to restore it.  The effect is the displayed lens disappears in order to show the unit path.  It works okay for everything except the Religion lens.

With Religion lens, it's funky:

Instead of the UILens.SetActive, I tried UILens.ToggleLayerOn(lensNameBase) which would not draw the dashed lines.

I fiddled around with other things, but ultimately I couldn't figure out how to get the dashed lines to appear on top of the hexes as is.  Not sure it's possible, at least with the dashed lines for the lens, as opposed to coloring those hexes, or something...
(I used ScreenToGif to capture these, uploaded to Imgur as anonymous, and then inserted here using the Markdown text styling).

From PR discussion.
These lenses (movement and city mgmt) are custom made and both utilize the same layer. Enabling one disables the other.
As for now we don’t know how to enable both of them, so to avoid “blinking” and making the code even more obscure the proposed solution is to disable Movement path while City Mgmt is open.


Extractor error on Akamai AMP feed, KeyError: 'channel'

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.04.17. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.04.17

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
youtube-dl --verbose --skip-download --write-sub "http://video.foxnews.com/v/4032976847001/"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', '--skip-download', '--write-sub', 'http://video.foxnews.com/v/4032976847001/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.04.17
[debug] Python version 3.6.0 - Darwin-16.5.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.3, ffprobe 3.3
[debug] Proxy map: {}
[foxnews] Downloading Akamai AMP feed
ERROR: An extractor error has occurred. (caused by KeyError('channel',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/Users/johria/.virtualenvs/late-night-talk-show-analysis/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 429, in extract
    ie_result = self._real_extract(url)
  File "/Users/johria/.virtualenvs/late-night-talk-show-analysis/lib/python3.6/site-packages/youtube_dl/extractor/foxnews.py", line 65, in _real_extract
    'http://%s/v/feed/video/%s.js?template=fox' % (host, video_id))
  File "/Users/johria/.virtualenvs/late-night-talk-show-analysis/lib/python3.6/site-packages/youtube_dl/extractor/amp.py", line 18, in _extract_feed_info
    'Unable to download Akamai AMP feed')['channel']['item']
KeyError: 'channel'
Traceback (most recent call last):
  File "/Users/johria/.virtualenvs/late-night-talk-show-analysis/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 429, in extract
    ie_result = self._real_extract(url)
  File "/Users/johria/.virtualenvs/late-night-talk-show-analysis/lib/python3.6/site-packages/youtube_dl/extractor/foxnews.py", line 65, in _real_extract
    'http://%s/v/feed/video/%s.js?template=fox' % (host, video_id))
  File "/Users/johria/.virtualenvs/late-night-talk-show-analysis/lib/python3.6/site-packages/youtube_dl/extractor/amp.py", line 18, in _extract_feed_info
    'Unable to download Akamai AMP feed')['channel']['item']
KeyError: 'channel'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/johria/.virtualenvs/late-night-talk-show-analysis/lib/python3.6/site-packages/youtube_dl/YoutubeDL.py", line 760, in extract_info
    ie_result = ie.extract(url)
  File "/Users/johria/.virtualenvs/late-night-talk-show-analysis/lib/python3.6/site-packages/youtube_dl/extractor/common.py", line 442, in extract
    raise ExtractorError('An extractor error has occurred.', cause=e)
youtube_dl.utils.ExtractorError: An extractor error has occurred. (caused by KeyError('channel',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description of your issue, suggested solution and other information
Ran into an error while trying to download a video from foxnews using FoxNewIE. Seems to be a generic error related to the AMPIE.

Thanks @remitamine!


Samples with very low frequencies of certain cell types

Dear Maria,
When annotating cell types in our scRNA dataset I realised that our samples do not contain many different cell type populations. We have around 60% B cells (healthy or malignant), 25% of CD4+ T cells and 12-15% CD8+ T cells. The other cell populations (NK, DC, Monocytes..) are less than 1% each.
Would this be enough for Community analysis?
Best,
Verena

Hi Verena,
You don't have to annotate the malignant B cells in any special way, you can put them as the B cell category even if there is a mixture of malignant B cells and a small rest of the healthy B cells in your tumor samples, the algorithm will "notice" that there are two populations by checking the "active fraction" component.
Best,
Maria


Fix resty.upload issue

when use lua-resty-upload , ngx.req.read_body() should not be used

Hi, @lhmwzy ,Thank you very much about your PR, but I think it's so many codes and I suggest that we should not use standard lua functions such as string.find.


Remove note that is contradicted in the subsequent paragraph

Why: remove contradictory information
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):
Remove a note stating that security reports cannot be made to repositories that do not opt into receiving them

Check off the following:

[ ] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[ ] For content changes, I have completed the self-review checklist.

The paragraph following the deleted line says that reports can be made to public repositories even if they have not turned on this feature.

@jmandel Thanks so much for opening a PR! I'll get this triaged for review ✨

Thanks for your contribution @jmandel ✨
We will review this PR this week!

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@jmandel -I am not sure that there is a contradiction between both the note and the paragraph, but I agree that you've highlighted confusing content, that we should improve on. Thanks you so much for that 💖 ✨
We will fix this internally as I need to discuss with the team in charge how they'd like to proceed with this.
As a result, I will close this issue.

Thanks! It sounds like I misunderstood the content. Here are some suggestions for how to make it clearer.

If you do not have admin or security permissions for a public repository, you can still privately report a security vulnerability

If I understand correctly, this could be rewritten more clearly as something like

If a public repository has enabled "Private vulnerability reporting," anyone can report an issue by clicking "Security" ...


[BUG] Do now show warning messages from werkzeug

Describe the bug
Running pf flow test ... --ui starts local PFS. Command line prints out a bold warning message from werkzeug. The message is confusing and misleading. PF should consider dismiss the warning message.

@luigiw This requires to install an WSGI container to disable the message, which means we need to add some dependency like gunicorn. I think we should not introduce this dependency just to avoid this warning.


fixes #255 so we auto-configure the ability to deploy a maven site to the content-repository

fixes #255 so we auto-configure the ability to deploy a maven site to the content-repository

[merge]


🐛 BUG: 'wrangler@2 pages publish' returns 504 or 503 failure

What version of Wrangler are you using?
2
What operating system are you using?
ubuntu-latest
Describe the Bug
RUNS
GITHUB-ACTION<EMAIL_ADDRESS>Run<EMAIL_ADDRESS>  with:
    apiToken: ***
    accountId: ***
    projectName: xxx-yyy-gitaction
    directory: ./public
    gitHubToken: ***
npm WARN exec The following package was not found and will be installed: wrangler@[2](...)

wrangler COMMAND inside of<EMAIL_ADDRESS>Executing: npx wrangler@2 pages publish "./public" --project-name="xxx-yyy-gitaction" in /home/runner/work/xxx-yyy/xxx-yyy

ERROR LOG
504
🌎  Uploading... (701/708)

✘ [ERROR] ✘ [ERROR] Received a malformed response from the API

🌎  Uploading... (701/708)

    <!DOCTYPE html>


added support for files on gcp

tested here: https://connection.europe-west3.gcp.keboola.com/admin/projects/220/queue/335678

(if the above is not clear) There should be a test created that uses an ENV variable that contains token to a GCP project. That env variable will be stored as secret in this repo. In the test you will use the client authorized with the token from env variable to upload and assert the file's existence.


wrapped operation iteration block in try/finally to ensure logs are captured

added libjpeg-dev to docker install to satisfy pillow dependency (http://pillow.readthedocs.org/en/3.0.x/installation.html#external-libraries)

Fixes #53

emm... I thought I had already pushed the libjpeg-dev yesterday... maybe not.

sure enough, I branched off before that got merged -- surprised it's still showing as a diff but doesn't seem to be in conflict

yeah, that's the part I don't understand

lgtm


Added options and timeout for Modbus RTU

Added: Option to print result (currently it's always printed and spams the logfile) with -p or --print-result
Added: Timeout was missing for the Modbus RTU function and therefore not applied
Changed: Fixed some Flake8 errors

Thank @mr-manuel for your contribution.

Thanks for merging!


Difficulty

Overview
This PR introduces the notion of difficulty

Game now has difficulty mode which then identifies its grid size.
3 modes of easy, normal, and hard have been added which correspond to a grid size of 6, 7, and 8, respectively.
Replace unnecessary return of pointers to avoid pointer chasing.
Modified the tests
Fixed the problem of Error being always present in JSON responses.
Removed unnecessary CurrentGame field from Player

After your review, I'll push this to staging


Xcode 9.2 shows build error but compiles

This may not be an issue, for my part, so I confirmed the problem in the example in this repo.
Whether or not the code conversion process is performed, the example code (using either the original version 0.5 or the latest 0.7), there is an error, but the build will still compile. Using the pod in a custom project has this same issue:
import SwiftySound results in "Cannot load underlying module for 'SwiftySound'"
The example works in the simulator, however, as does a custom project also runs on a device.

Building the project removes the error until the next edit...


Cannot rename when path in compile_commands.json is not absolute

Try renaming T to U in the following code:
template <class [[T]]> struct Foo {
  using type = T;
};

Expected: template parameter is renamed.
Actual: code is unchanged, no error or warning is shown.

@hokein if the fix is small, it would be nice to merge it into the 9.0 release.

@hokein  reports everything works fine on his machine.
After digging a bit more, the problem comes from the fact that compile_commands.json I was using had a relative path:
{"file": "/home/ilya/project/tmp/foo.cpp",
 "directory": "/home/ilya/project/tmp",
 "command": "/usr/bin/c++ -xc++ -c foo.cpp"}

The changes are (incorrectly) filtered out here

Can confirm that gn, followed by ninja -t compdb cxx produces compile_commands.json with relative paths, so this issue could be classified as high-to-medium priority.

i believe a similar issue also occurs with bazel, since compile commands
refers to same file via a different path due to symlinked execroot path
On Tue, Aug 6, 2019, 12:23 PM Ilya Biryukov<EMAIL_ADDRESS>wrote:

Can confirm that gn, followed by ninja -t compdb cxx produces
compile_commands.json with relative paths, so this issue could be
classified as high-to-medium priority.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/clangd/clangd/issues/117?email_source=notifications&email_token=AAG3EE62Y77COOOTMIJOJ73QDE7JLA5CNFSM4IJTXQ4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3UQM3I#issuecomment-518588013,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAG3EEYAJJ6TZWIXGN3VORLQDE7JLANCNFSM4IJTXQ4A
.

I believe the issue exists for a long time (and in the previous release). One option to fix this is to check the absolute file path (but it'd not work for the symlink case). Given the fact that we only get Replacement result from the rename library, it is non-trivial to fix it using other options e.g. checking the replacement source location is in main file.
And we don't have this issue internally as we use a filesystem wrapper in clangd.

Even better, we should check on FileID being equal to the main file id. @sam-mccall suggested we just replace calls to rename with whatever it calls internally to get all ranges of references (or even use capacities we have in clangd's XRefs instead?).
Does the rename function does anything extra than finding all places where a name is references?

fixed in upstream rL368429.


Add client side schematic

Update client side schematics and Gerber files to match Rev 1b

Lol. The breadboard we tested was the test.


Use FloatArray everywhere

I took the liberty to use FloatArray everywhere where applicable.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/15124/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15124/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15124/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15124/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15124/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15124/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15124/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15124/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15124/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15124/merge/testResults/webgpuplaywright/index.html


Implement Ascii2NcWrapper

This wrapper has been implemented and tested on a few cases using the MET test data. The work is found in branch feature_263_ascii2nc. I still need to update the documentation.

Documentation has been added to sphinx transition branch.


CU4 Adding fields using in designer crashes web client

I am using CU4 on an upgraded database. I am able to publish and table / page extension from VS code in an upgraded DB.
When I go to add a field using the web client-designer the application crashes when dragging the field from the available fields to the list page.
Error accessing Website
Raw Url:
Url:
Type: Microsoft.Dynamics.Nav.Types.GenericException
Message: Could not find target of extension objects during compilation. Make sure that system symbols are published.
The name 'Search Name' does not exist in the current context
The target Page "Chart of Accounts" for the extension object is not found
StackTrace:
at Microsoft.Dynamics.Nav.Client.ConnectionStateManager.CallServer[T](BeginCallServerMethod beginCallServerMethod, EndCallServerMethod1 endCallServerMethod, ServerCallOptions options) in S:\Depot\NAV\Platform\Client\Shared\Prod.Client.ServiceConnection\ConnectionStateManager.cs:line 151    at Microsoft.Dynamics.Nav.Client.ServiceConnection.CallServer[T](BeginCallServerMethod beginCallServerMethod, EndCallServerMethod1 endCallServerMethod) in S:\Depot\NAV\Platform\Client\Shared\Prod.Client.ServiceConnection\ServiceConnection.cs:line 2011
at Microsoft.Dynamics.Nav.Client.FormBuilder.NavDesignerService.AddLogicalControlCore(LogicalControl parentControl, Int32 newIndex, RowEntry controlInfo) in S:\Depot\NAV\Platform\Client\Shared\Prod.Client.Builder\BuilderSession\UIServices\NavDesignerService.cs:line 264
at Microsoft.Dynamics.Framework.UI.InteractionInvoker.Invoke[T](String interactionName, Action1 invokeMethod, T input) in S:\Depot\NAV\Platform\Client\Shared\Prod.ClientFwk\Interactions\InteractionInvoker.cs:line 42    at Microsoft.Dynamics.Framework.UI.LogicalInteraction1.Invoke(T logicalInteractionInput) in S:\Depot\NAV\Platform\Client\Shared\Prod.ClientFwk\Interactions\LogicalInteraction.cs:line 40
at Microsoft.Dynamics.Framework.UI.InteractionManager.<>c__DisplayClass8_0.b__3() in S:\Depot\NAV\Platform\Client\Shared\Prod.ClientFwk\Interactions\InteractionManager.cs:line 125
Source: Microsoft.Dynamics.Nav.Client.ServiceConnection----------------------------------
Type: Microsoft.Dynamics.Nav.Types.GenericException
Message: Could not find target of extension objects during compilation. Make sure that system symbols are published.
The name 'Search Name' does not exist in the current context
The target Page "Chart of Accounts" for the extension object is not found
StackTrace:
Source:

Looks similar to #1853

You mean you that if you run Get-NavAppInfo -servername  -symbolsonly then you get something lime this too besides  teh System symbols.
Id            : e070cd4c-f452-46b7-b572-545b37df1e33
Name          : Application
Version       : 13.0.21845.0
Publisher     : Microsoft
ExtensionType : ModernDev
Scope         : Global

We believe this is fixed on CU6. Please contact regular support for further assistance.


Timeout issue with marathon-wait-for-listen.sh on underpowered control node

When I spun my test cluster I ran into an issue with marathon-wait-for-listen.sh timing out.  My control node was relatively under powered (t2-small in AWS).  I found that 60s wasn't long enough.  I changed the timeout to 300s and was able to get passed this.  Perhaps a longer timeout would be worth adding to help out all us poor folks playing with small instances.

@chrholme do you think 120 seconds as in pull #659 would be sufficient?

I'd have to check but my guess is that that would be perfectly fine.  :)

Closing with the merge of #659. Reopen if it is too short.


fix: add "key" attribute to suppress react "key" warnings

b953beb5735fd522fe345ce7ef1a402dbe16360a
As @yusukebe pointed out in https://github.com/honojs/honox/issues/233#issuecomment-2475227165, template elements are written out as arrays, but without the key attribute, which caused a warning, has been fixed.
be543f3b49000480b7fad144439e6c11d7552b56
There was no key attribute left in the content of the template element, which was exported by SSR, but when the element was restored by creatingElement on the client side, a warning was displayed due to the absence of the key attribute.
It may seem strange to add the key attribute with a sequential number, but these elements are not overwritten after rendering by the client, so there is no problem if the warning is avoided by adding a sequential number.

I am investigating the failure of the test...

bcb8fdc
We are using jsx directly for the performance, but with this change, we needed to remove the “key" from the props as follows, so we should stop using jsx directly and started using createElement.
https://github.com/honojs/hono/blob/44a50bdf55850ead9d36caf31d4b0a0b2586f08b/src/jsx/dom/index.ts#L52-L55

@yusukebe
Would you please review?

Hi @usualoma
These commits make sense. Looks good. I'll merge and release a new version. Thank you!


ngx-formly does not update the model

I have created a dynamic template that work well with a little mockup, but when i call service does not show nothing. It looks like the template its not taking the model from call.
----file1.html---
`<form [formGroup]="form" (ngSubmit)="submit()">
<formly-form [model]="modelTable1" [fields]="fields" [options]="options" [form]="form">

`
-----file1.ts----
constructor( private service: service, ) {

   this.service.getPrenotazioni().subscribe(
     (result) => {
       if (result.success) {
         this.modelTable1 = {
           data:result.data
         },
        // this.modelTable1 = {...this.modelTable1}
         console.log('data from service', this.modelTable1);         }
     },
     (error) => console.log('errore')
   );
 }

-----dynaimc-form.html------

<div *ngFor="let field of field.fieldGroup; let i = index;">
  <formly-group [model]="model[i]" [field]="field" [options]="options" [form]="formControl">

  ====some content===

  </formly-group>
</div>

--------dynamic-form.ts--------

constructor(builder: FormlyFormBuilder) {
    super(builder);}

Hi, can you replicate your code using our stackblitz starter example https://stackblitz.com/edit/ngx-formly-ui-bootstrap
Thanks!

Hi, can you replicate your code using our stackblitz starter example https://stackblitz.com/edit/ngx-formly-ui-bootstrap
Thanks!
Sure. This is demo.
When i try to call from function with some delay the model is not shown.
Thank you
https://stackblitz.com/edit/ngx-formly-ui-bootstrap-rdny1v

The context this when the function is invoked will not update the expected model instance defined in the AppComponent so either:

use Arrow function:

callFunction() {
  setTimeout(() => { this.modelTable = ...; }, 1000);
}

use bind:

callFunction() {
  setTimeout(function() { this.modelTable = ...; }.bind(this), 1000);
}

Thank you

The context this when the function is invoked will not update the expected model instance defined in the AppComponent so either:
1. use Arrow function:

callFunction() {
  setTimeout(() => { this.modelTable = ...; }, 1000);
}

1. use bind:

callFunction() {
  setTimeout(function() { this.modelTable = ...; }.bind(this), 1000);
}

thank you


#626 - Added required to description in Volunteer

Added validation via the viewmodel. Returns an error on view if Description is empty

thanks @wmay1991


CoinbaseWalletSDK throws an error on coinbase wallet selection on desktop (Google Chrome) w/ Vanilla JS

The Coinbase option shows up fine in the list, rendered properly.
Clicking it pops up this up as the modal, consistently spinning on the connecting animation, and the modal is non dismissible:

In the console, I get this error:

The fix is to take this code, and put it above the Coinbase script.
<EMAIL_ADDRESS>Then you want to define this as a global.
const Buffer = buffer.Buffer;
I'll close out the issue, but this should be added to the docs I would think.


there is a white line at the SnackBar bottom

The following photo shows that

code:
new MySnackBar(this,intent.getStringExtra(Config.TASK_DATA)).show();
Thanks
Your widgets are awesome !

Hi, sorry but I don't know what is that line, I use the SnackBar in some applications and that line never has appeared .

Same problem here.
That line appears because the snackbar is a little above the controls. Thus there's a little gap between the lower controls and the snackbar.


Add golden test for InkWell clipping

For https://github.com/flutter/flutter/issues/20483

@jonahwilliams here's a duck sized horse :duck:


Viz: Render reconstructions in CCF space

In the end, the topmost view of the cells in the dataset would be to plot them together in The Allen's CCF i.e. position them in the brain, to whatever accuracy is possible. 10 test neurons would be ok, 105 training neurons might get a bit crowded/overplotted.

BrainRender seem like a good tool. It's not interactive but it can render purdy.

Spatial registration of serial microscopic brain images to three-dimensional reference atlases with the QuickNII tool

QuickNII v2.2 supports standard web-compatible image formats, 24-bit PNG and JPEG. Images can be loaded up to the resolution of 16 megapixels (e.g. 4000x4000 or 5000x3000 pixels), however QuickNII does not benefit from image resolutions exceeding the resolution of the monitor in use. For a standard FullHD or WUXGA display (1920x1080 or 1920x1200 pixels) the useful image area is approximately 1500x1000 pixels, using a similar resolution ensures optimal image-loading performance.
Preprocessing of images (downsampling, rotation, renaming) can be achieved with open access software tools (e.g. ImageMagick, Matlab scripts) or python scripts found in many open source libraries (e.g. PIL) to fulfill these requirements (converting to PNG or JPEG and downscaling to screen-like size), but QuickNII allows storage of original image dimensions as part of its series descriptor. Serial section images should be assigned consecutive serial numbers, preferably indicated by three-digit numbers at the end of the file name, e.g. Sample_ID_s001.tif. Section sampling is given by the serial numbers. The section images are collected in a folder.


Allow to fold a specific level

It should be possible to fold all sections in the current file that have a specific indentation level.

it should be possible to right click on a code folding icon and select (Un)Fold all on this level
it should be possible to trigger this action from command palette or shortcut to fold all on the level of the section the cursor currently is in.

Thanks for the request, I had 'Fold Level' already planned. Not all of what you request, but IMO the most useful:

Ctrl + J, Ctrl + 1: Folds all regions of level 1, except if a cursor is placed in it
Ctrl + J, Ctrl + 2: Folds all regions of level 2, except if a cursor is placed in it
...
Ctrl + J, Ctrl + 5: Folds all regions of level 5, except if a cursor is placed in it

The other features you mention (unfold to level, context menu, fold to level at cursor) are currently not planned. Please file separate issues if you think they are important.


Password Protection

Feature Request: Would like simple password protection. Wouldn't mind hardcoding it into the code either.

Perhaps we can add a very simple auth module, just using a user:passwd config file or command option. I don't want it to be complicated.
Actually, this is a base project, and I will add more business-related features in production.


Add benchmarks to the repository - [merged]

In GitLab by @perarnau on Jul 25, 2018, 20:42
Merges benchmarks -> master
Merge the great work done by @suchyb, with a few cleanups in the commit format/msgs :-)
Will further improve the code in the following days, mostly for style issues.
Of course the important work is in the dgemm_* files.

In GitLab by @perarnau on Jul 26, 2018, 06:45
merged
By Swann Perarnau on 2018-07-26T11:45:25 (imported from GitLab project)

In GitLab by @perarnau on Jul 25, 2018, 20:43
enabled an automatic merge when the pipeline for 2c452094b60e902556739246ef5db8cf4f30454f succeeds
By Swann Perarnau on 2018-07-26T01:43:34 (imported from GitLab project)

In GitLab by @perarnau on Jul 25, 2018, 20:43
Will restart the KNL runner tomorrow for merging.
By Swann Perarnau on 2018-07-26T01:43:29 (imported from GitLab project)


3.7.2

@rultor release tag is 3.7.2

@rultor release tag is 3.7.2

@portlek OK, I will release it now. Please check the progress here

@rultor release tag is 3.7.2

@portlek Done! FYI, the full log is here (took me 11min)


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/pkg 438776b...18589ac:
  > 18589ac Bump assorted dependencies to their latest releases (# 2371)
  > d37811b Update community files (# 2370)
  > 0c482f1 Update actions (# 2369)
  > 07fb0dd Update community files (# 2368)
bumping knative.dev/reconciler-test ae9e2e2...80584b5:
  > 80584b5 Test scoped logs in wait functions (# 261)
bumping knative.dev/hack 813559c...e11ac12:
  > e11ac12 Update community files (# 126)
  > 4ec2223 Update community files (# 124)

/hold
Build is failing. https://github.com/knative-sandbox/control-protocol/pull/136 fixes it with including this PR.


Update to semver ≥ 7

Node Version: 10.17.0
Platform: Linux/Debian
Compiler: gcc 9
Module:

node-gyp is not compatible with semver ≥ 7. I found this little fix that seems to fix the problem:
--- a/lib/find-python.js
+++ b/lib/find-python.js
@@ -226,7 +226,7 @@
       }
       this.addLog(`- version is "${version}"`)

-      const range = semver.Range(this.semverRange)
+      const range = new semver.Range(this.semverRange)
       var valid = false
       try {
         valid = range.test(version)

Also affects Arch linux.

Can you send a PR for this ? Thanks.

Done in #2006 :wink:

so, we have "semver": "^5.7.1", how is >=7 getting involved and why is this a problem for folks?

Without the patch above people encounter this error: https://gist.github.com/goddessfreya/90babde3d51401c51ef9e3d717bd5f05#file-gistfile1-txt-L181

This seems to be Arch Linux issue. They for some reason decided not to respect the npm version defined in package.json as @rvagg pointed out but use a system-wide installed semver package, see https://git.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/node-gyp#n28:
 # Experimental dedup
  rm -r "$pkgdir"/usr/lib/node_modules/$pkgname/node_modules/semver

@vladimiry This explanation does not explain why it is also broken on Debian.
I checked the Debian experimental package and apparently they apply this patch to get node-gyp to work:
Description: fix for semver 7
Author: Xavier Guimard<EMAIL_ADDRESS>Forwarded: <no|not-needed|url proving that it has been forwarded>
Last-Update: 2019-12-26

--- a/lib/find-python.js
+++ b/lib/find-python.js
@@ -226,7 +226,7 @@
       }
       this.addLog(`- version is "${version}"`)

-      const range = semver.Range(this.semverRange)
+      const range = new semver.Range(this.semverRange)
       var valid = false
       try {
         valid = range.test(version)

This bug obviously effects more than one distribution.

apparently they apply this patch to get node-gyp to work:

It's because instead of respecting npm versioning they prefer to maintain a single/system-wide set of npm modules that they put to respository. This is, in my opinion, a vicious way when it comes to installing/using the npm modules.

Is there are way to do this conditionally so that
const range = semver.Range(this.semverRange) is executed if semver < 7 and const range = new semver.Range(this.semverRange) with new added, is executed if semver >= 7 so that our code can continue to support both?

By the way, here is the workaround that should be working on any Linux system where the npm v5.2.0+ package is installed:

Uninstalling the node-gyp package from the system.
Running npx node-gyp instead of node-gyp. What is the "npx" tool?

And well, it's better not to install/use any npm module from the at least Arch/Debian repositories since they don't respect npm versioning.

so, we have "semver": "^5.7.1", how is >=7 getting involved and why is this a problem for folks?

Hi @rvagg,
I'm Debian Developer. For security reason, our policy does not accept embedded code. That's why there is only one semver in our distribution. I posted here my patch, published in our package.
Cheers,
Xavier

what on earth ...
  # Experimental dedup
  rm -r "$pkgdir"/usr/lib/node_modules/$pkgname/node_modules/semver

Linux package managers 🤦♂
@guimard I know all about these policies, they're outdated and conflict with Node.js' packaging style and these kinds of problems have been popping up for years. This is no surprise and why I always recommend people (a) never install Node.js from their distro, but use https://github.com/nodesource/distributions instead and (b) never ever use packaged Node.js packages from their distro, because these kinds of breakages are inevitable.
OK, so semver.Range in pre-7 uses the instanceof trick to make new optional: https://github.com/npm/node-semver/blob/v5.7.1/semver.js#L856-L858
In post-7 it uses an ES class, so there's no way to pull off the instanceof trick: https://github.com/npm/node-semver/blob/master/classes/range.js
But this means that we should be able to just stick a new in and it'll be compatible with the current and future version(s) of semver, for this specific instance. I would normally put my foot down and tell Debian & Arch to suffer from the problems their policies create, but this one seems easy enough.
Even with this fix it's extremely probable that using node-gyp (and any other bundled Node.js package) from your Linux distro will make more problems show up that we have zero test coverage for and could be extremely risky--and yes @guimard even "security" concerns because they create these unknown edges where we have zero understanding of the interactions between packages outside of the specific ranges that we test for.
So anyone reading this -- ditch your distro version of node-gyp and anything else you've installed from their and use npm directly to install them, npm install node-gyp -g is easy.

For security reason, our policy does not accept embedded code. That's why there is only one semver in our distribution.

Thanks for the info. That must be a pain maintaining npm versioning through the manual patches. I personally think it's very hard to impossible to do not being a core maintainer of all the involved npm modules. Asking as a curiosity matter, do you use some sort of script that scans all the npm packages in the dist repo and then lists all the cases of npm versioning breaking (automated versioning breaking detection)?

Hi @vladimiry,
for node.js modules packaged in Debian, we never use npm but our dependency system (debian/control)


Typing '<' or '>' in repl doubles that character

Happens in VS 2015 Update 3 and VS 15.
So typing this:
'<'
(or >)
Gives you this:
'<<'

I updated to latest sources and turns out a fix that went in today took care of that already.


.find unable to find dynamic nodes

Describe the bug
mount .find is unable to find a dynamically created dom node within the tree.
I know this is an edge case since, in the majority of cases, react components render other reacts components all the way down. I just wanted to point this out in case this is something you wanted to support. I came across this when trying to write some tests for react-codemirror, which is a react wrapper around a code mirror object.
To Reproduce
Here's a test case that showcases the error:
import React from 'react';
import { mount } from 'enzyme';

const myClassName = 'my-great-class';

class ComponentWithDynamicChild extends React.Component {
  constructor() {
    super();
    this.container = React.createRef();
  }

  componentDidMount = () => {
    this.container.current.append(`<div class="${myClassName}"></div>`);
  }

  render() {
    return (<div className="container" ref={this.container} />);
  }
}

describe('enzyme dynamic class', function() {
  let mountedComponent;

  it('should find the dynamically inserted div', function() {
    mountedComponent = mount(<ComponentWithDynamicChild />);
    expect(mountedComponent.find(myClassName)).to.have.length(1);
  });
});

Expected behavior
I  expected .find to return a reference to a node? Maybe a stripped down wrapper that has a text() method.
Desktop (please complete the following information):

OS: macOS
Browser Chrome
Version 67

I just found the Common Issues section. Does this issue fall under https://github.com/airbnb/enzyme/blob/master/docs/common-issues.md#testing-third-party-libraries?

I think it fits, so I'm going to close this issue.

Sounds right to me, unfortunately.


STEK share plugin using Raft

This is a plugin used to share STEK among a cluster of ATS instances, it replaces the current session reuse plugin. This new plugin uses the Raft consensus algorithm (NuRaft library by Ebay https://github.com/eBay/NuRaft) for leader election.
The reason for creating this plugin is:

The current session reuse plugin needs redis to run, and the leader election mechanism is somewhat messy.
There is less and less need for sharing Session ID's, on our hosts it session reuse based on ID's are as low as 0.2%.

Have been testing this in our production hosts for a few weeks now, everything runs great other than the weird #8734 problem we were seeing.

@moonchen is going to take a look at this

Cherry-picked to v9.2.x


fix(cli): prefer require dependencies from local node_modules

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

When using nuxt global binary or NPX tool, CLI should prefer using locally installed packages.
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #4337 into dev will decrease coverage by 0.07%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##              dev    #4337      +/-   ##
==========================================
- Coverage   91.49%   91.41%   -0.08%
==========================================
  Files          56       56
  Lines        1857     1863       +6
  Branches      468      468
==========================================
+ Hits         1699     1703       +4
- Misses        148      150       +2
  Partials       10       10

Impacted Files
Coverage Δ

packages/cli/src/imports.js
80% <80%> (-20%)
: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 ae8dc96...aa70e72. Read the comment docs.


DrupalVM and nginx with multi language private files / image styles

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 1.9.1
VirtualBox 5.1.10r112026
ansible <IP_ADDRESS>
  config file = /etc/ansible/ansible.cfg
  configured module search path = Default w/o overrides

Your OS

macOS (10.12.4)

Summary
DrupalVM with nginx lacks support for private files and / or private multingual image styles. The following nginx rule works for me:
location ~ ^(/[a-z\-]+)?/system/files/ {
    try_files $uri /index.php?$query_string;
}

Perhaps we could add something like this.
Originally from here : https://github.com/geerlingguy/ansible-role-nginx/issues/104#issuecomment-301122637

+1 from me. Would need a PR and maybe someone else manually testing it out to confirm :)


Adding GetFieldSelector() to client.EventInterface

This is required so that kubectl can call client.EventInterface.List() with the appropriate field selector.

Verified that e2e tests pass :)

Looks like @lavalamp is out this week.
@bgrant0607 @smarterclayton ?

Sorry, I'm swamped. I suggest @saad-ali.

I'll take a look

LGTM mod a few nits

Thanks!
Replied to all comments and updated the code accordingly. Ready for another look.

LGTM. Will merge on green.


v2.0 - Running a Command for a container doesn't work

Rancher versions:
rancher/server:
rancher/agent:
Infrastructure Stack versions:
healthcheck:
ipsec:
network-services:
scheduler:
kubernetes (if applicable):
Docker version: (docker version,docker info preferred)
Containers: 90
 Running: 69
 Paused: 0
 Stopped: 21
Images: 22
Server Version: 17.09.1-ce
Storage Driver: zfs
 Zpool: turtle
 Zpool Health: ONLINE
 Parent Dataset: turtle/_ubuntu/rancher/docker
 Space Used By Parent: 27648
 Space Available:<PHONE_NUMBER>0624
 Parent Quota: no
 Compression: on
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
 Log: awslogs fluentd gcplogs gelf journald json-file logentries splunk syslog
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 06b9cb35161009dcb7123345749fef02f7cea8e0
runc version: 3f2f8b84a77f73d38244dd690525642a72156c64
init version: 949e6fa
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.4.0-98-generic
Operating System: Ubuntu 16.04.3 LTS
OSType: linux
Architecture: x86_64
CPUs: 12
Total Memory: 17.64GiB
Name: bigma
ID: UWKU:2Q36:44LW:HH6M:5DI3:YERH:WHCZ:YT5X:LDP2:QP4F:7Y4W:BHS2
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

WARNING: No swap limit support

Operating system and kernel: (cat /etc/os-release, uname -r preferred)
4.4.0-98-generic

Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)
Bare-metal

Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Ubuntu 16.04 server
Docker-CE (latest)
docker run -d --restart=unless-stopped -p 8080:8080 rancher/server:preview

Environment Template: (Cattle/Kubernetes/Swarm/Mesos)
Rancher 2.0 default

Steps to Reproduce:
Logging in and running /usr/bin/apt update from the shell works, but running it from the UI doesn't
Results:

This was probably just my first time using docker.


Fix pytest tests/benchmarks --benchmark-enable

Without this it tries to sort by the name and the name can be none. I think we should fix that issue in the pytest-speed library but anyway this makes it work locally. Ran into this while running all benchmarks for the sake of testing perf of the tuple refactor


Using example with latest tamago

I'm currently porting tamago to the sabrelite board, and I would like to be able to test the example using the latest
tamago version. When I  replace github.com/f-secure-foundry/tamago with a local copy of the master tamago
branch, the build fails with:
../tamago-git/arm/exception.go:134:17: undefined: runtime.MemRegion
../tamago-git/arm/exception.go:154:17: undefined: runtime.MemRegion
../tamago-git/arm/mmu.go:60:17: undefined: runtime.MemRegion
../tamago-git/arm/mmu.go:87:16: undefined: runtime.MemRegion

Should MemRegion be defined in the example somewhere?

The latest tamago HEAD is meant to work with https://github.com/f-secure-foundry/tamago-go/tree/tamago1.16.3 which hasn't a binary release yet.
This because this commit took out MMU initialization away from tamago-go and within tamago.
Please compile this branch on your own in order to use tamago HEAD.
Thanks!

Thanks will give this a go tomorrow an report back!

Got this working, thank you!


incomplete list of command using get-command -module AnyBox

Hello
after installing the module, it appear that the command get-command give an incomplete list:
`PS C:> get-command -module AnyBox
CommandType     Name                                               Version    Source

Function        Show-AnyBox                                        0.3.1      AnyBox`
if we do, we got mode command but the version and source is empty
`PS C:> get-command box
CommandType     Name                                               Version    Source

Function        New-AnyBoxButton
Function        New-AnyBoxPrompt
Function        Show-AnyBox                                        0.3.1      AnyBox`
:) not cool when trying to see what are the commands used by the module.
one more thing:
I had PSCX module, when i first execute "Install-Module -Name AnyBox" it blocked because there is a conflict on the function ConvertTo-Base64, PSCX had a function with the same name.
after removing it  i can install anybox module.
thanks for the work done.
Tonic8

Tonic8, this should be resolved in the latest version. Also, ConvertTo-Base64 is now Get-Base64. Thanks for bringing that to my attention. Let me know if any issues persist.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-30-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4590

Questions ?
Please refer to the Backport tool documentation


'The "center" property is not defined in the main scope'

I dont find a way to create a directive that embed leaflet directive.
I have created a plunker to isolate the problem:
http://plnkr.co/edit/zgnzG3fqHxJ13DoiNGy8?p=preview
console shows many errors:

[AngularJS - Leaflet]  The "center" property is not defined in the main scope(anonymous function)
[AngularJS - Leaflet] The 'tiles' definition doesn't have the 'url' property.

This seems different from #362

Thank for forking my plunk.
I have updated my plunk.
Please check it.

This issue was moved to angular-ui/ui-leaflet#135

I'm going to rework&redesign angular-leaflet-directive to be compatible with Leaflet v1.0. It will mantain almost all its functionality, and will be compatible with the current features of the directive, but I must start from a fresh point, so I'm going to close this issue. If you think it must be worked with the new version, please reopen it.


python310Packages.weconnect: 0.40.0 -> 0.41.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.weconnect is: "Python client for the Volkswagen WeConnect Services"
meta.homepage for python310Packages.weconnect is: "https://github.com/tillsteinbach/WeConnect-python"
meta.changelog for python310Packages.weconnect is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.41.0 with grep in /nix/store/l1m92djaznza36xrqq2biszq9bxxry71-python3.10-weconnect-0.41.0
found 0.41.0 in filename of file in /nix/store/l1m92djaznza36xrqq2biszq9bxxry71-python3.10-weconnect-0.41.0

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

4 package rebuild(s)

4 x86_64-linux rebuild(s)
4 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.weconnect
python310Packages.weconnect-mqtt
python39Packages.weconnect
python39Packages.weconnect-mqtt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l1m92djaznza36xrqq2biszq9bxxry71-python3.10-weconnect-0.41.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.weconnect https://github.com/r-ryantm/nixpkgs/archive/215515ad8ae22670f6cd47cb796e093690ef83a1.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l1m92djaznza36xrqq2biszq9bxxry71-python3.10-weconnect-0.41.0
ls -la /nix/store/l1m92djaznza36xrqq2biszq9bxxry71-python3.10-weconnect-0.41.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages failed to build:

python310Packages.weconnect-mqtt
python39Packages.weconnect-mqtt

2 packages built:

python310Packages.weconnect
python39Packages.weconnect

Maintainer pings
cc @fabaff for testing.

Superseded by #174705


SREP-1037: Add Tests, and fix namespace controller defect

Adds tests for SREP-1037 and adds a fix in the namespace controller to
allow for the case when the configuration ConfigMap is absent, or does
not otherwise have the expected project_blacklist key. The fix is to
requeue the request after 5 seconds.
This change also modifies the Makefile to add a test target, and to
force build to depend upon test.
Signed-off-by: Lisa Seelye<EMAIL_ADDRESS>

@jewzaam

Not seeing a test for modifying (if possible) or deleting the dedicated-project-admin rolebindings.

Unless I'm missing something, these (update/delete) aren't operations performed by the operator?

Hmm.. saw a comment from @lisa but now it's gone:

Unless I'm missing something, these (update/delete) aren't operations performed by the operator?

There are 2 controllers.  The rolebinding controller recreates the project rolebindings if they're deleted.  Not sure if it will reapply if they're changed, but that should be tested too.

Sorry. 1678c75bef0cb2ff7bd8757f48fdf884b20dfff7 updated deps and has a ton of changes in vendor.

Hmm.. saw a comment from @lisa but now it's gone:

Unless I'm missing something, these (update/delete) aren't operations performed by the operator?

There are 2 controllers. The rolebinding controller recreates the project rolebindings if they're deleted. Not sure if it will reapply if they're changed, but that should be tested too.

Yes, I deleted it because I noticed the Rolebinding controller.

I just realized that this PR and commits have been for SREP-1039, not SREP-1037. I'll edit the commit message when it's merged.

No more concerns once we deal with https://github.com/lisa/dedicated-admin-operator/pull/1

This is going to be a bit of an odd merge, given the outstanding comments both here and @jewzaam's proposed PR in https://github.com/lisa/dedicated-admin-operator/pull/1 but in light of #16 it makes sense.
As noted in https://github.com/lisa/dedicated-admin-operator/pull/1#discussion_r270998409 we have a potential issue where an invalid regular expression is specified in the configuration and we then have weird behaviour when trying to match against it (and how do we handle it?), and dealing with what we do when users edit the ConfigMap at runtime?
With #16, however, we're going to move away from a ConfigMap entirely, in favour of a hardcoded set of compiled regular expressions in the operator's source. Thus, if the compilation fails the operator won't start, and we won't have unexpected behaviour. Plus, this moves some guarantees to make test and make build) to ensure the provided (static) config is safe.
This PR is being merged because of the large number of files changed (thanks vendor 😦 ) and how painful a rebase after #16 is completed. We know that these tests will have to change for #16 and that's okay.


[BEAM-9160] Update AWS SDK to support Pod Level Identity

Upgraded AWS SDK and added support to pod identity when running in AWS EKS.

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

R: @aaltay @kennknowles

R: @jbonofre @lukecwik @chamikaramj @timrobertson100

Can we run the linkage checker to perform analysis on whether we increased the number of linkage errors?
Example PR of running the linkage checker:
https://github.com/apache/beam/pull/10769

The code changes look fine otherwise.

Thanks again @andeb !

Thanks for reviewing and merging it, @iemejia!
With regards to the linkage errors, they may be false positives considering that there are lines of code to ignore classes from GraalVm? https://github.com/GoogleCloudPlatform/cloud-opensource-java/blob/master/dependencies/src/main/java/com/google/cloud/tools/opensource/classpath/LinkageChecker.java#L208-L213
Also, I thought it made sense to expose WebIdentityTokenCredentialsProvider directly to be consistent with other exposed credential providers. Happy to do any cleanup though but it seems that @ecapoccia is already on it!

The linkage errors from svm is false positives.


Add UpdateRequiredStatusChecks function

Fixes #937

Thanks for the review! Addressed most of the feedback -- will wait to hear back on whether I should change the function parameter or keep it consistent with the other ones.

Ping -- still waiting on response to latest question (all other PR feedback has been addressed in update)

Ping for another reviewer (@juliaferraioli or anyone else actively doing reviews) to unblock

Thank you, @nmiyake and @juliaferraioli!
Merging.


Error for unknown custom rules dependencies

Here is an example.
schema = Dry::Validation.Schema do
  required(:x).filled(:int?)

  validate(with_option: [:x, :foo]) do
    raise 'will not be raised'
  end
end

result = schema.(x: 1)
puts result.success? #=> true

Would be nice to check dependencies and warn user about unknown keys. It may be hard to notice typo in deps without any errors.

high level rules in schemas will be gone in 1.0.0 so I'll make sure this will be addressed in the new API

This won’t be addressed as this dsl was removed in dry-schema which will be schema backend in dry-validation 1.0


Document all the RESIN_SUPERVISOR_ env vars available

Rather than having them spread across various blog entries:
https://resin.io/blog/device-bandwidthdata-usage-how-low-can-we-go/
https://resin.io/blog/updating-config-txt-remotely/
https://resin.io/blog/customize-the-way-your-devices-are-updated/

https://github.com/resin-io/resin-supervisor/issues/87

fixed in #702

Current config variables documented here: https://docs.resin.io/learn/manage/configuration/


Auto wrap callable returns from Realm Functions?

@littledan @caridy The current Readme allows autowrapping callable returns from the Realm functions. It just felt more natural for the workflow.
I believe we set in #9 that only importBinding would autowrap returns and we would also autowrap given arguments. Like in this example:
const red = new Realm();

await red.importBinding('./file.js', myImportRedFn); // wrapped

const redFunction = red.Function('cb', 'return cb(2);');
redFunction(x => x * 3); // the arrow function is wrapped

My question is regarding to block or not autowrap of returns:
const redOtherFunction = red.Function(secret, `
  globalThis.secret = secret;
  return (x) => globalThis.secret + x;
`);

const wrapped = redOtherFunction('foo');

wrapped('bar'); // yields 'foobar';

Maybe I just captured it wrong in #9, it seems fair enough to have this, but my memory might be failing here.

Yes, I misread the issue I wrote myself. Some coffee helped me put my brain back on track.


Allow nulls in migration name column

This is because existing users of Phinx will not have values for this column, and so it should correctly specify that it is expecting to have null values. Should fix #803.

@coatesap thanks for fixing this!


Is it possible to change the background color of the video when it is in fullscreen mode?

Is it possible to change the background color of the video when it is in fullscreen mode?

Maybe this helps: https://developer.mozilla.org/de/docs/Web/CSS/:fullscreen

Thanks @AgentSmith0 ,
It is working for normal videos.But if i am using a vimeo url the video is coming as an iFrame and the background itself coming from iFrame We dont have control over it. Is there a solution to tackle this?

In worst case you could parse the mp4 file from vimeo and then play it, but I think the vimeo terms do not allow this method.

Thanks for the reply @AgentSmith0 ,
We hold this requirement for now as it is difficult to do in iFrame.


Possible to get the power usage of pump?

Appears to be publishing it to the app experience but wasn't sure if it was coming back from their API or not...?
Would be interesting to be able to track power usage of the pump in my HA

.

I haven't seen the data in the feeds from the API. Can you run the diagnostics (see the main page for instructions) and post the config and telemetry files from your setup so I can see if that has been updated? Thanks!


Rename CimCmdlet resources

PR Summary
Rename CimCmdlet resource file to common format.
PR Context
It is for #12355.
The commit was checked in #12355.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

The test failure in macOS CI is the known issue with GetHostEntry, which has nothing to do with this PR.


Unable to create certificate using ingress annotations - MalformedCSRException: CSR must mark the SAN extension critical when it has an empty subject

Kubernetes: 1.19 (v1.19.6-eks-49a6c0)
aws-pca-issuer: Helm release 0.1.0
certmanager: 1.3.1

I have a working setup of this addon but currently unable to use it to auto-generate certificate using annotations such as below:
apiVersion: extensions/v1beta1
# apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  annotations:
    cert-manager.io/cluster-issuer: myprivate-ca
  name: podinfo
  namespace: podinfo
spec:
  rules:
  - host: podinfo.infrastructure.com
    http:
      paths:
      - path: /
        backend:
          serviceName: podinfo
          servicePort: 80
  tls:
    - hosts:
      - podinfo.infrastructure.com
      secretName: podinfo10

Creating certificate using the CRD works, e.g with:
kind: Certificate
apiVersion: cert-manager.io/v1
metadata:
  name: podinfo-cert
  namespace: podinfo
spec:
  commonName: podinfo.infrastructure.com
  dnsNames:
    - podinfo.infrastructure.com
  issuerRef:
    group: awspca.cert-manager.io
    kind: AWSPCAClusterIssuer
    name: myprivate-ca
  renewBefore: 360h0m0s
  secretName: podinfo-cert

The error I get when using the ingress approach to autogenerate the certificate is:
2021-04-28T18:42:12.883939412Z {"level":"error","ts":1619635332.883713,"logger":"controllers.CertificateRequest","msg":"failed to request certificate from PCA","certificaterequest":"podinfo/podinfo3-rxrmp","error":"MalformedCSRException: CSR must mark the SAN extension critical when it has an empty subject.","stacktrace":"github.com/go-logr/zapr.(*zapLogger).Error\n\t/go/pkg/mod/github.com/go-logr/zapr@v0.2.0/zapr.go:132\ngithub.com/jniebuhr/aws-pca-issuer/pkg/controllers.(*CertificateRequestReconciler).Reconcile\n\t/workspace/pkg/controllers/certificaterequest_controller.go:171\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.8.3/pkg/internal/controller/controller.go:298\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.8.3/pkg/internal/controller/controller.go:253\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func1.2\n\t/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.8.3/pkg/internal/controller/controller.go:216\nk8s.io/apimachinery/pkg/util/wait.JitterUntilWithContext.func1\n\t/go/pkg/mod/k8s.io/apimachinery@v0.20.2/pkg/util/wait/wait.go:185\nk8s.io/apimachinery/pkg/util/wait.BackoffUntil.func1\n\t/go/pkg/mod/k8s.io/apimachinery@v0.20.2/pkg/util/wait/wait.go:155\nk8s.io/apimachinery/pkg/util/wait.BackoffUntil\n\t/go/pkg/mod/k8s.io/apimachinery@v0.20.2/pkg/util/wait/wait.go:156\nk8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/go/pkg/mod/k8s.io/apimachinery@v0.20.2/pkg/util/wait/wait.go:133\nk8s.io/apimachinery/pkg/util/wait.JitterUntilWithContext\n\t/go/pkg/mod/k8s.io/apimachinery@v0.20.2/pkg/util/wait/wait.go:185\nk8s.io/apimachinery/pkg/util/wait.UntilWithContext\n\t/go/pkg/mod/k8s.io/apimachinery@v0.20.2/pkg/util/wait/wait.go:99"}

The approach of using ingress works with other issuers as we currently use vault issuer but we want to migrate to AWS ACM.
Does this work for anyone?

Hi,
I haven't actually tested this before. Can you post both of the CertificateRequest CRs that are created?
I can probably check why it's not working from there.

Not sure I know exactly what to post whilst keeping sensitive information off. It is easy to test it as documented here https://cert-manager.io/docs/usage/ingress/#how-it-works

I'll have to try and reproduce it later this week then.

@jniebuhr - any chance you got round to looking at this?

Turns out that this is resolvable by doing below:
apiVersion: extensions/v1beta1
# apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  annotations:
    cert-manager.io/issuer: myprivate-ca
    cert-manager.io/issuer-kind: AWSPCAClusterIssuer
    cert-manager.io/issuer-group: awspca.cert-manager.io
    cert-manager.io/common-name: podinfo.infrastructure.com
  name: podinfo
  namespace: podinfo
spec:
  rules:
  - host: podinfo.infrastructure.com
    http:
      paths:
      - path: /
        backend:
          serviceName: podinfo
          servicePort: 80
  tls:
    - hosts:
      - podinfo.infrastructure.com
      secretName: podinfo10

Main change being adding the extra annotation cert-manager.io/common-name: podinfo.infrastructure.com
Happy for this to be closed perhaps with an example of generating cert this way.

Thanks for the update.

Just ran into the same issue and adding the cert-manager.io/common-name annotation fixed it. Thanks!


nerfing .vimrc's setrelativenumber and undodir options in case of old versions of vim

Nice little feature detection to avoid errors like this:

(ignore your commit in here. GH is having problems)

Thanks!


pyCUDA updates

Tests were done on Jetson Xavier AGX with image demo-image-full
Add the following lines into your conf/local.conf

EXTRA_IMAGE_FEATURES:append = " tools-sdk dev-pkgs"
CORE_IMAGE_BASE_INSTALL:append = " python3-pycuda"

Running pycude sample applications

1- Running demo.py sample application:
root@jetson-agx-xavier-devkit:~# python3 ./demo.py
original array:
[[ 0.8893958   0.88322926  0.15596126 -0.23357607]
 [ 0.6795663   0.34148544 -0.3477962   0.8451891 ]
 [ 1.091979   -0.21781361  0.19186608 -0.69381344]
 [-1.9413309   0.13207184 -1.717314    0.0613003 ]]
doubled with kernel:
[[ 1.7787915   1.7664585   0.31192252 -0.46715215]
 [ 1.3591326   0.6829709  -0.6955924   1.6903782 ]
 [ 2.183958   -0.43562722  0.38373217 -1.3876269 ]
 [-3.8826618   0.26414368 -3.434628    0.12260059]]
doubled with InOut:
[[ 1.7787915   1.7664585   0.31192252 -0.46715215]
 [ 1.3591326   0.6829709  -0.6955924   1.6903782 ]
 [ 2.183958   -0.43562722  0.38373217 -1.3876269 ]
 [-3.8826618   0.26414368 -3.434628    0.12260059]]
original array:
[[ 0.6561598   0.1759764  -0.79895955  0.24399514]
 [-0.7802414  -0.6623423   1.2609524  -1.452552  ]
 [ 1.5975146  -1.8651625  -1.4724625   0.3414911 ]
 [ 0.1420012  -1.2412921   1.3443182   0.6225943 ]]
doubled with gpuarray:
[[ 1.3123196  0.3519528 -1.5979191  0.4879903]
 [-1.5604827 -1.3246846  2.5219047 -2.905104 ]
 [ 3.1950293 -3.730325  -2.944925   0.6829822]
 [ 0.2840024 -2.4825842  2.6886363  1.2451886]]
root@jetson-agx-xavier-devkit:~#

2- Running demo_cdpSimplePrint.py sample application:
root@jetson-agx-xavier-devkit:~# python3 ./demo_cdpSimplePrint.py
starting Simple Print (CUDA Dynamic Parallelism)
***************************************************************************
The CPU launches 2 blocks of 2 threads each. On the device each thread will
launch 2 blocks of 2 threads each. The GPU we will do that recursively
until it reaches max_depth=2

In total 2
+8
=10 blocks are launched!!! (8 from the GPU)
***************************************************************************

Launching cdp_kernel() with CUDA Dynamic Parallelism:

BLOCK 0 launched by the host
BLOCK 1 launched by the host
|  BLOCK 5 launched by thread 0 of block 1
|  BLOCK 3 launched by thread 0 of block 1
|  BLOCK 4 launched by thread 0 of block 0
|  BLOCK 2 launched by thread 0 of block 0
|  BLOCK 6 launched by thread 1 of block 1
|  BLOCK 7 launched by thread 1 of block 1
|  BLOCK 8 launched by thread 1 of block 0
|  BLOCK 9 launched by thread 1 of block 0
root@jetson-agx-xavier-devkit:~#

Thanks @ichergui this works for me on demo-image-full and jetson-xavier-nx-devkit as well.


Enabeling/disabling zwave logging does not take effect on save.

When enabling or disabling the logging toggle and hitting safe, logging is not enabled|disabled.

I know this, I'm still sure that this is something wrong with node-openzwave-shared connection management. Something strange in connect and disconnect methods. I'm not very friendly with C++, could you check connect and disconnect methods of node-openzwave-shared and check if options are inited/reset correctly? I think that when something is not reinited on disconnect and this causes to connect using the previous options...

On 19 Mar 2019, at 18:51, Jorge Schrauwen<EMAIL_ADDRESS>wrote:
When enabling or disabling the logging toggle and hitting safe, logging is not enabled|disabled.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Check the OpenZwave constructor too, here is where I pass the options and seems that options can be set only once:
https://github.com/OpenZWave/node-openzwave-shared/blob/master/src/openzwave.cc#233
Daniel

On 19 Mar 2019, at 18:51, Jorge Schrauwen<EMAIL_ADDRESS>wrote:
When enabling or disabling the logging toggle and hitting safe, logging is not enabled|disabled.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Check this, it is the constructor of OpenZwave and seems that options can be inited only once:
https://github.com/OpenZWave/node-openzwave-shared/blob/master/src/openzwave.cc#233
How can this be fixed?

On 19 Mar 2019, at 18:51, Jorge Schrauwen<EMAIL_ADDRESS>wrote:
When enabling or disabling the logging toggle and hitting safe, logging is not enabled|disabled.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I think the only way to do the change is to destroy the entire object and create a fresh one?
Like do the cleanup as you do now, then complete reinitialize the entire client object? Seems a bit wasteful resource wise though... then again it is not something you will do often.

This is not possible in javascript, the only way to do this would be start zwave in a separate process an kill the process but this is such an hard solution for a simple problem in my opinion. I think things need to be fixed on the library side, I will open an issue on node-openzwave shared about this problem

SGTM

@sjorge  the response from the author:

quick short answer: the C++ plugin currently only reads config upon init
time. It will need some refactoring to make it able to reread and apply new
options at runtime.

Need to check the code and see we can do something

Hi @sjorge. I have tried to check the code here: https://github.com/OpenZWave/node-openzwave-shared/blob/master/src/openzwave.cc#L203 but sincerly I don't understand why options can be inited just one time. Maybe you are more familiar than me with C++ could you check that code? I don't think the fix would be much hard just need to know where to start...

Aside from forcing it to compile I have limited understanding of c++ code.
This bit is very confusing as it seems to be building and arg list for a process... which does not make much sense.

Are there any clues in the other program where you got the other fix from?

Just submitted a PR that seems to work from my tests, let's wait to check what happens
https://github.com/OpenZWave/node-openzwave-shared/pull/277

Error on app veyor:
C:\projects\node-openzwave-shared\lib\install-ozw.js:147
  throw err;
  ^
Error: invalid signature: 0x4f44213c
    at C:\Users\appveyor\AppData\Local\Temp\1\ozwinstall-0pm1a\node_modules\unzip\lib\parse.js:59:13
    at runCallback (timers.js:810:20)
    at tryOnImmediate (timers.js:768:5)
    at processImmediate [as _immediateCallback] (timers.js:745:5)


build, deploy & undeploy individual actions using a flag

current
can build/deploy/undeploy all or nothing
cool to have
a flag to do by action e.g. aio app (un)deploy --only-actions a,b,c
how
needs changes in https://github.com/adobe/aio-app-scripts build/deploy/undeploy

build & deploy has been done


clangd respects -Werror in compile_commands.json

clangd now respects -Werror or /WX when present in compile_commands.json
as seen in these 2 logs, in defer.hpp at lines 3, 6, and 8. 18.3 produces warnings, whereas 20 produces errors.
18-clangd.log
20-clangd.log

This appears to be fixed in more recent trunk.

A regression test for exactly this issue was added in https://github.com/llvm/llvm-project/commit/9e65dcac660723a06039c7e9b30f305b9b8ca652, though it doesn't mention what changes regressed and subsequently fixed the issue.

It looks like the regressing change was https://github.com/llvm/llvm-project/pull/70976, and the fix was its revert in https://github.com/llvm/llvm-project/commit/2ad435f9f6fb792d9b010ddf56ca3ea26fbf5f15.
Closing this issue as there's nothing further to do here (the regression on trunk has been fixed, and the added regression test will make sure it does not regress again).


[Bug] tree shaking problem

Describe the bug
Some hooks will import all hooks.
For example: useInfiniteScroll
https://github.com/InhiblabCore/vue-hooks-plus/blob/2a8fd77d5adab1f59d042dbacc7b407631aefa92/packages/hooks/src/useInfiniteScroll/index.ts#L2

Steps to reproduce
import useInfiniteScroll from 'vue-hooks-plus/es/useInfiniteScroll';

System Info
no

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a VueHooks Plus issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

change them like:
// packages/hooks/src/useInfiniteScroll/index.ts

import { useBoolean, useEventListener, useRequest } from "../index";

Into:
// packages/hooks/src/useInfiniteScroll/index.ts

import useBoolean from "../useBoolean";
import useEventListener from "../useEventListener";
import useRequest  from "../useRequest ";

@Airkro Thank you for your feedback. I will check and optimize the code, and I will be updating and improving it soon. Please stay tuned for the latest version.


Test nginx as a Deployment with NodePort service

DaemonSets have some issues with DNS reachability of services e.g. Ingresses with services of type ExternalName don't work
in theory, it should be possible to set --service-node-port-range=80-33000 on the api-server and then use a Deployment with a NodePort service to achieve the same result, but with DNS working 100%

An alternative might a DaemonSet with host-network=false and hostPort=80,443


Update publish.yml

Added code example URL

All good, thank you for adding the link with the documentation


Submitting JSON as body?

I'm trying to submit a JSON post to my local dev server and having issues. I haven't seen much documentation on sending complex parameters or JSON in the body for a POST request in the documentation (unless I'm missing something). I've been browsing through issues with no success unfortunately on how to get this to work, I have a complex bit of JSON that I need to send in a .POST request to my Rails API. Here is the code for sending the JSON
 `@IBAction func SessionSubmit(_ sender: Any) {
    let selected_session_type = session_type_ids[self.SessionTypePicker.selectedRow(inComponent: 0)]
    let selected_session_duration = session_type_durations[self.SessionTypePicker.selectedRow(inComponent: 0)]
    let dateFormatter = DateFormatter()
    let selected_start_time = DateTimePicker.date
    let start_time = dateFormatter.string(from: DateTimePicker.date)
    let end_time = dateFormatter.string(from: selected_start_time + selected_session_duration.minutes)
    let refreshToken = current_user.refreshToken as String
    let accessToken = current_user.accessToken as String
    let params: [String: Any] = ["access_token": accessToken, "refresh_token": refreshToken]
    let body: [String: Any] = ["data": [
                    "type": "session",
                    "attributes": [
                        "title": "",
                        "description": "",
                        "start": start_time,
                        "end": end_time,
                        "session_mode": "video_session"],
                        "relationships": [
                            "users": [
                                "data": [
                                    ["type": "user",
                                     "id": current_user.userId],
                                    ["type": "user",
                                     "id": clientId]
                                ]
                            ],
                            "session_type":[
                                "data": [
                                    "type": "session_type",
                                    "id": selected_session_type
                                    ]
                                ]
                            ]
                        ]
                    ]
    do {
        let jsonData = try JSONSerialization.data(withJSONObject: body, options: .prettyPrinted)
        print(jsonData)
        let decoded = try JSONSerialization.jsonObject(with: jsonData, options: [])
        print(decoded)

        if let dictFromJSON = decoded as? [String:Any] {
            print(dictFromJSON)
            self.sendData(parameters: params, body: dictFromJSON) {
                self.stopAnimating()
            }
        }
    } catch {
        print(error.localizedDescription)
    }
    startAnimating(GlobalVariables().LoadingSize)`

   `    func sendData(parameters: [String:Any], body: [String:Any], finished: @escaping () -> Void) {
    DataController().Post(parameters: parameters, path: "/v1/sessions/"+current_user.userId).perform(withSuccess: { (FetchedArray) in
        if let SessionTypesArray = FetchedArray.response["data"].array {
            self.session_types += SessionTypesArray
        }
        finished()
    }) { (FetchFailed) in
        AuthenticationController().logout(user: self.current_user)
        self.dismiss(animated: true, completion: {});
        finished()
    }
}`

I'm thinking maybe I need to make these embedded in the document and only have my refresh and access tokens as parameters? so I tried working on doing it that way and yet still another road block because of not seeing any ways to add to the body, I know there is appendBody but I'm not sure how that would work either, the json seems to not be flowing to the rails api correctly either
`Started POST "/v1/sessions/122" for <IP_ADDRESS> at 2017-05-19 13:52:32 -0400
Error occurred while parsing request parameters.
Contents:
access_token=90d2befd58912f978dc681f2f0d1eaf3d217dbe25fdac3d1ec4ab45e9bfff41a&data%5Btype%5D=session&data%5Battributes%5D%5Bdescription%5D=&data%5Battributes%5D%5Btitle%5D=&data%5Battributes%5D%5Bend%5D=&data%5Battributes%5D%5Bsession_mode%5D=video_session&data%5Battributes%5D%5Bstart%5D=&data%5Brelationships%5D%5Bsession_type%5D%5Bdata%5D%5Btype%5D=session_type&data%5Brelationships%5D%5Bsession_type%5D%5Bdata%5D%5Bid%5D=379&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Btype%5D=user&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Bid%5D=122&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Btype%5D=user&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Bid%5D=6&refresh_token=6b60a52534e1fa2ed436a9542a6c27b7f05e56e7b306f48df12b286b8fe5f624
JSON::ParserError - 822: unexpected token at 'access_token=90d2befd58912f978dc681f2f0d1eaf3d217dbe25fdac3d1ec4ab45e9bfff41a&data%5Btype%5D=session&data%5Battributes%5D%5Bdescription%5D=&data%5Battributes%5D%5Btitle%5D=&data%5Battributes%5D%5Bend%5D=&data%5Battributes%5D%5Bsession_mode%5D=video_session&data%5Battributes%5D%5Bstart%5D=&data%5Brelationships%5D%5Bsession_type%5D%5Bdata%5D%5Btype%5D=session_type&data%5Brelationships%5D%5Bsession_type%5D%5Bdata%5D%5Bid%5D=379&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Btype%5D=user&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Bid%5D=122&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Btype%5D=user&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Bid%5D=6&refresh_token=6b60a52534e1fa2ed436a9542a6c27b7f05e56e7b306f48df12b286b8fe5f624':
json (1.8.6) lib/json/common.rb:155:in parse'
any help here would be greatly appreciated guys! thanks in advance!

Just do:
request.parameters = body
request.parameterEncoding = JSONEncoding.default

You don't need to explicitly create JSON using JSONSerialization, since it's already implemented in Alamofire ParameterEncoding, and therefore, in TRON as well.

Ok I tried adding that to my api controller for the request, but instead of using body I just went back and added everything to my main params
let params: [String: Any] = ["access_token": accessToken, "refresh_token": refreshToken,                         "data": [                         "type": "session",                         "attributes": [                             "title": "",                             "description": "",                             "start": start_time,                             "end": end_time,                             "session_mode": "video_session"],                             "relationships": [                                 "users": [                                     "data": [                                         ["type": "user",                                          "id": current_user.userId],                                         ["type": "user",                                          "id": clientId]                                     ]                                 ],                                 "session_type":[                                     "data": [                                         "type": "session_type",                                         "id": selected_session_type                                         ]                                     ]                                 ]                             ]                         ]
my api controller code is this
    func PostCreate(parameters: [String:Any], path: String) -> APIRequest<Fetched, FetchFailed> {         let request: APIRequest<Fetched, FetchFailed> = DataController.tron.request(path)         request.method = .post         request.parameterEncoding = JSONEncoding.default         request.headers = ["Accept":"application/vnd.api+json"]         request.headers = ["Content-Type":"application/vnd.api+json"]         print(request.path)         print(request.parameters)         print(request.headers)         print(request.urlBuilder)         return request     }
But now I'm getting an invalid media type, I'm thinking that JSONEncoding.default is messing up the ""application/vnd.api+json" content-type, right?

It should not reset Content-Type, if you set it before sending a request
https://github.com/Alamofire/Alamofire/blob/master/Source/ParameterEncoding.swift#L313-L314

func PostCreate(parameters: [String:Any], path: String) -> APIRequest<Fetched, FetchFailed> {
    let request: APIRequest<Fetched, FetchFailed> = DataController.tron.request(path)
    request.method = .post
    request.parameterEncoding = JSONEncoding.default
    request.headers = ["Accept":"application/vnd.api+json"]
    request.headers = ["Content-Type":"application/vnd.api+json"]
    print(request.path)
    print(request.parameters)
    print(request.headers)
    print(request.urlBuilder)
    return request
}

when I submit it with that I get
"errors" : [
{
"code" : "406",
"status" : "406",
"title" : "Not acceptable",
"detail" : "All requests must use the 'application/vnd.api+json' Accept without media type parameters. This request specified 'application/json'."
}
]
but if I comment out the JSONEncoding.default I get the issue I had originally
Started POST "/v1/sessions/" for <IP_ADDRESS> at 2017-05-21 11:37:17 -0400
Error occurred while parsing request parameters.
Contents:
access_token=9856a83b1a286343482a729e5ec7c909f838e43fc0623586e38c13ff0bb72c10&data%5Btype%5D=session&data%5Battributes%5D%5Bdescription%5D=&data%5Battributes%5D%5Btitle%5D=&data%5Battributes%5D%5Bend%5D=&data%5Battributes%5D%5Bstart%5D=&data%5Battributes%5D%5Bsession_mode%5D=video_session&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Btype%5D=user&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Bid%5D=122&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Btype%5D=user&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Bid%5D=6&data%5Brelationships%5D%5Bsession_type%5D%5Bdata%5D%5Btype%5D=session_type&data%5Brelationships%5D%5Bsession_type%5D%5Bdata%5D%5Bid%5D=679&refresh_token=6b368c95d13e7ecece91dfe1ade30e80ccb7d7311ffcf0771086f227e1de72fc
JSON::ParserError - 822: unexpected token at 'access_token=9856a83b1a286343482a729e5ec7c909f838e43fc0623586e38c13ff0bb72c10&data%5Btype%5D=session&data%5Battributes%5D%5Bdescription%5D=&data%5Battributes%5D%5Btitle%5D=&data%5Battributes%5D%5Bend%5D=&data%5Battributes%5D%5Bstart%5D=&data%5Battributes%5D%5Bsession_mode%5D=video_session&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Btype%5D=user&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Bid%5D=122&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Btype%5D=user&data%5Brelationships%5D%5Busers%5D%5Bdata%5D%5B%5D%5Bid%5D=6&data%5Brelationships%5D%5Bsession_type%5D%5Bdata%5D%5Btype%5D=session_type&data%5Brelationships%5D%5Bsession_type%5D%5Bdata%5D%5Bid%5D=679&refresh_token=6b368c95d13e7ecece91dfe1ade30e80ccb7d7311ffcf0771086f227e1de72fc':
Another catch 22 unfortunately, does this seem like its something wrong with our backend api?

You should append, but not reassign headers
request.headers["Accept"] = "application/vnd.api+json"
request.headers["Content-Type"] = application/vnd.api+json"

Awesome, I think that got it, now I'm having an authentication issue with our server, but I don't think it has anything to do with what we were trying to solve, thanks again for all your help!


CLI updates for preview 6 - Part 2

Fixes #7146
Contributes to #9893

@@karann-msft please review!

@mairaw - we don't need to change the dotnet nuget push *.nupkg
see https://github.com/dotnet/docs/issues/7146#issuecomment-506918400

Please see my comments on the related issue. The current doc is accurate.

@karann-msft added the note. Does it look good to you?

lgtm 🚢


Support read image

This pull request is indented to resolve #83
using this vision api

@seratch , how can I add more commits to fix the check errors?

please resolve the lint errors on your branch first; you can do so with your fork as these validations do not require any extra settings

Do you think we should use another PR for the refactor?

@gnuser Thanks for updating the PR! Before doing refactoring, it seems your change still need updates. I will check the behavior and apply a few additional changes. As for refactoring work in general, I am open to any improvements, but it does not mean I always accept them.

Closing this in favor of #88


Pan and zoomstart

Trigger pan and zoomstart events

Been a busy month & just got to this. I really appreciate your interest & thank you so much for the suggestions! Looking it over now.


import pyopal causes "Illegal instruction (core dumped)"

This happens when running on old cpus which do not support AVX2.
archspec.cpu.host().cpu.features returns:
{'ssse3', 'popcnt', 'aes', 'rdrand', 'sse4_2', 'pclmulqdq', 'sse4_1', 'f16c', 'mmx', 'avx', 'sse2', 'sse'}
Thanks for wonderful package :)
(Too small to warrant an issue:  add typing for scoring_matrix.alphabet)

Fixed in v0.6.1 (I checked on the cluster node with AVX only).


Lauching the game through VRCX does not use the EAC launcher

I am unable to join friends instance via VRCX because it launches the game without EAC, which makes me not load into a world.

ah yes I forgot about that I'll fix that.

Might be caused by VRCX calling vrchat.exe?
the new launcher uses start_protected_game.exe.

Could use "VRChat Path Override" in "Launch Options" to point to that exe instead for now.


When binds include localhost, try to bind ::1 as well

On some platforms (at least Windows) accessing ports via localhost is very slow because the address ::1 will be tried first and will be allowed to timeout. This patch generalises the port bindings a little, allowing a single user-requested binding to be mapped to a list of listening sockets. When the user requests an address <IP_ADDRESS> or <IP_ADDRESS> we perform a best-effort bind to ::1 also to speed up accesses on these IPv6 platforms.

I think this approach is OK. It's in line with what Linux does when net.ipv6.bindv6only is set to 0 (default). See also: https://github.com/docker/docker/issues/2174

CircleCI build failure looks like an actual problem:
File "src/bind.ml", line 37, characters 7-51:
Error: This expression has type
         (Lwt_unix.file_descr list, [> `Msg of bytes ]) Result.result Lwt.t
       but an expression was expected of type
         (Lwt_unix.file_descr, [> `Msg of bytes ]) Result.result Lwt.t
       Type Lwt_unix.file_descr list is not compatible with type
         Lwt_unix.file_descr

oops, yes indeed -- I was too focused on Windows. Will fix!

Seems reasonable though we may need to tweak later, not sure if we should respect the Linux side settings for ipv4 only and so on later...

There may also be an assertion failure which shouldn't happen but which did just happen in my tests. Investigating.

Darn, I've managed to trigger the system lacked sufficient buffer space error -- there might be a leak.

The resource leak was actually due to using an older version of hvsock. With the latest hvsock.0.7 it seems to be fast and no sign of a resource leak after 10k connections.
I think this one is ready, assuming the CI passes.

That sounds good to me. :+1:


Add fuzzing tests

Use os-fuzz https://github.com/google/oss-fuzz or similar and write tests to fuzz every exported API function.

Experience with fuzzing FreeRDP: Fuzzing and Exploiting Virtual Channels in Microsoft
Remote Desktop Protocol for Fun and Profit.

@akallabeth probably it can be closed


Implement Go drop-in lib

:+1:

https://github.com/GoogleChrome/http2push-gae/tree/master/examples/go


Archive tarballs are created without the top-level directory

Command
https://{host}/{user}/{repo}/archive/{branch,tag,40-char SHA}.tar.gz

creates the tarball without the top directory, with the project files in the root of the tarball.
Please compare with
https://codeload.github.com/gogits/gogs/tar.gz/09c9818?dummy=/gogits-gogs-v0.151218-09c9818_GH0.tar.gz

which creates the top level directory gogs-09c9818/ with project files under it.
The top-level directory is usually expected in UNIX tarballs. Please note that 'tar' command has --strip option that allows to remove the top level directories while extracting.

Thanks your feedback!
But this is duplicated to https://github.com/gogits/gogs/issues/518


FlashAttention2 has been toggled on, but it cannot be used due to the following error: the package flash_attn seems to be not installed. Please refer to the documentation of https://huggingface.co/docs/transformers/perf_infer_gpu_one#flashattention-2 to install Flash Attention 2.

What is the reason and how to solve it? Thank you

It's not installed. I wouldn't worry about it when it comes to Florence2 though, I don't notice any performance difference to "sdpa" myself.

I have compiled a whl of flash-attn for python 3.11.8 + PyTorch 2.3.0+cu121 in case you still want to try it with flash attention 2 for some reason.


AVRO-2603: Provide PYTHONPATH in Environment

Blocked by AVRO-2624.
Avoid the need to generate a script in a temporary file by providing the python path in the environment.
Jira

[x] Addresses part of AVRO-2603
[x] References it in the PR title
[x] Adds no new dependencies

Tests

[x] Improves an existing test

Commits

[ ] Commits reference the Jira issue.
[x] Commits follow guidelines from "How to write a good git commit message"

Documentation

[x] No new documentation needed.

Thanks for the assist, Ryan!


Class names randomly changed from ie to y suffix

In my generated classes, some type-names are randomly changed from ending in ie to ending in y.
Uh, yeah, I know - but look:

I swear, this is not a prank - some types (not all of them) in my input JSON schema are somehow getting renamed.
It does not appear to have anything to do with the output format - this happens with at least both TypeScript and C# output.
Again, this is not a prank - but maybe some joker introduced this into your codebase as a prank?
I gotta say, it is kind of funny. 😉
What could possibly cause this?

Uh, and here it is now, apparently cutting a class-name in half:

Come on now, guys - joke's over!

Looks like it could be quicktype's use of pluralize. Doesn't seem like this is configurable in quicktype?
Your second comment, I cannot at all see how/why this should happen. In general, I think the way pluralize is used here is flawed.

Yup, looks like JSONSchemaInput unconditionally attempts to singularize type-names - here:
https://github.com/quicktype/quicktype/blob/daffe53101c4d329c0647fb8310aded585e3808e/src/quicktype-core/input/JSONSchemaInput.ts#L662
And here:
https://github.com/quicktype/quicktype/blob/daffe53101c4d329c0647fb8310aded585e3808e/src/quicktype-core/input/JSONSchemaInput.ts#L790
Lucky for me, I'm using the API rather than the CLI - so for my project, I should be able to reconfigure pluralize.
For people using the CLI, not so much. There probably should be an option to bypass name mangling.
Though, to begin with, I wonder, why is it, by default, we don't "trust" the names in JSON schema? Whether those names "look pretty" as class-names or not, there are those of us who would prefer consistency with the names in the original API specification - I personally don't think this behavior makes sense as a default for JSON schema.
(It might not make sense as a default under any circumstances, given how poorly pluralize works in this practical case.)

Ah, I understand now - for nested types in JSON schema that don't have type-names, it has to come up with a name, in order to generate additional classes in e.g. C# and other languages that (unlike e.g. TypeScript) do not support structural subtyping.
So this is a feature, not a bug.
The issue remains though, that this doesn't actually work well.
Might we consider adding a warning or something? When it has to make up missing names on the fly.
If there was a warning, and an option to manually inject singular/plural word pairs, that would seem a lot more robust.
Having to break into third-party libraries and rely on global state and side-effects ain't pretty. We should not be coupled to pluralize on the consumer side - it should be an implementation detail in the library, I think.


Bug/filter ignores sort

User changes sort, table gets sorted. He then selects filter value, and sort gets ignored. I've put together select-filter-with-sort example where you can see it.

@just-paja, thanks your contribution, I thinks it's a big bug since we supported filter on header and your solution is not bad. Actually, the root cause is on here.  We always use this.data to filter, you can see the other function in TableDataStore(ex: remove, search, add  etc.), The correct way is like following:
const currentDisplayData = this.getCurrentDisplayData();
this.filteredData = currentDisplayData.filter( row => {
   //....
}

Anyway, You did a great job and do you want to keep to fix it? I can let you fix this issue follow above-mentioned :)

@AllenFang, thanks, but your suggestion actually does not work. You would filter filtered data and eventually get to empty array. I think that my way is efficient. It sorts only when requested and right after user filtered data.

Sorry, I got a misunderstand on filter mechanism. I think your solution is well!

Hey @AllenFang this would be really useful.  I've encountered this issue too. (full disclosure, I work with @just-paja)
Keep up the good work, this is a really useful project.

Release on v2.1.2 :)