E5108: Error executing lua

This morning (18 August 2022, UTC+8) I got this error after update symbols-outline.
The command that I execute is :SymbolsOutline
I open golang project.
E5108: Error executing lua ...vim/plugged/symbols-outline.nvim/lua/symbols-outli stack traceback:         ...vim/plugged/symbols-outline.nvim/lua/symbols-outline.lua:307: in func         [string ":lua"]:1: in main chunk

Pretty sure it's cause you're not calling setup. This was recently changed so see the setup part of the readme

https://github.com/simrat39/symbols-outline.nvim#setup


drivers: wifi: winc1500: Use offload_context instead of user_data.

The wifi_winc1500 driver's socket id is stored in
net_context->user_data, which may be overwritten later at
the socket layer, which also uses the net_context->user_data
field to store socket flags.
This patch introduces a dedicated offload_context field
for use by offload drivers, and updates the wifi_winc1500 offload
driver to use this field instead of user_data.
Fixes #8820
Signed-off-by: Gil Pitney<EMAIL_ADDRESS>

Note: This passed sanitycheck for quark_se_c1000_devboard, but someone with a winc1500 wifi driver may want to validate also.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #8835   +/-   ##
=======================================
  Coverage   52.32%   52.32%
=======================================
  Files         195      195
  Lines       24730    24730
  Branches     5140     5140
=======================================
  Hits        12941    12941
  Misses       9715     9715
  Partials     2074     2074

Impacted Files
Coverage Δ

include/net/net_context.h
93.61% <ø> (ø)
:arrow_up:

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 4d3e13d...2222313. Read the comment docs.

net_context->user_data, which may be overwritten later at
the socket layer, which also uses the net_context->user_data
field to store socket flags.

So, I wanted to add that native API usage may also overwrite ->user_data, but turns out it's funkier: we call offloading handler before storing anything to ->user_data, pass input user data to handler, and in case of winc driver, it stores that in its own struct socket_data, pointer to which gets stored to net_context->user_data. Smart, but I wonder if that goes on the border of "premature optimization".
Consequently, we have 2 choices:

Explicitly document that in the case of offloaded net_context, its user_data belongs to the offloading driver, e.g. for the purpose of used by winc1500 driver. Then move socket flags on the net_context level.
Or apply this patch, but later probably still go for optimization in p.1.

@GAnthony, choice is yours. I'd be ready to approve this patch.

@pfalcon, thanks.  I prefer 2), to have an explicit context for use by the offload drivers.
Since I couldn't actually test the change on the winc1500 device, I'm OK to wait for @tbursztyka 's opinion/validation on the proposed change to that driver.

I'm OK to wait for @tbursztyka 's opinion/validation on the proposed change to that driver.

I believe at the last networking meeting he said he'd be away for extended time (vacation followed by parental care). So, I'm going to approve it in the hope to avoid another stall on offloading elaboration.

OK, thanks, makes sense.  The change should be innocuous.


How did you do the diagrams?

Hi, I like the ascii diagrams in SPEC.md and DESIGN.md. How were those made?

Hi @drudru, thanks for the interest!
The ascii diagrams were created by hand with Vim's block edit mode (ctrl-v). It lets you edit a text file more as a grid.
There are some minor aesthetic things I've picked up on. You have lines |- and arrows v<^> and can make nice looking rounded corners with apostrophes and periods '-.. After that it's just experimenting to make it look nice.
I like ascii art because there is no concern about file hosting, and the images will always live with the text. Though they don't work for everything. I considered using ascii art for graphs, but the low-resolution made it difficult to see what was going on. Instead I generate graphs with matplotlib on a separate branch: https://github.com/ARMmbed/littlefs/tree/gh-images.

Awesome :+1:


[Question] Limits has no effect?

I probably get it wrong, but it looks like Limits for client have no effect because we create a new httpx.AsyncClient with each AsyncClientBase.send_request call.
I want to limit number of connections to FCM by limiting the number of connections in httpx connection pool, but appereantly for this to work we need to reuse httpx.AsyncClient beetween send_request calls
btw. @akalex thank you for your great work.

Hi @svidchenkov-a,
Thanks for submitting a question. You are correct; in the current implementation, Limits have no effect. Let me fix that.


Fix Safari 9

Removing the data: part from blob url makes it load again in a new tab.
Haven't tested how it still works in Safari 8 though.

thanks! i'll do some back testing, but the change makes sense. i'll try to get it in soon if everything checks out.

Any updates?

i've been on vacation away from my mac and busy with family. you picked a bad time of year, the worst in fact... sorry it's taking so long; i haven't forgotten and i should be able to check it out pretty soon.

Thats ok :) Take your time

if(typeof safari !=="undefined" ){  seems to be false in safari9, so the change won't actually affect that, but could break safari8, which kinda worked the last time i checked, but i can no longer test it out. i'll look into the whole situation more, maybe somebody out there has found a solution we can incorporate.
thanks for the heads up and the code, sorry it doesn't fix safari (the new IE?); i was excited...

Hi, weird, library with this patch is running live on: https://my.teleport.org/runway/
Also my Safari 9 console output:
![Screenshot]
(https://photos-1.dropbox.com/t/2/AABAPixkuBEM6jX3i3nu-S_2JnZYsPzIVjPz0aZGtbz-9Q/12/463359/png/32x32/3/1452438000/0/2/Screenshot 2016-01-10 12.24.12.png/EJOnRRimwbb1AyACKAI/Wxouc3XMDdIdlNISyUrQAWE6wItCTuQP29e5CELX4aw?size_mode=5&size=32x32)

Booted up Yosemite on Virtualbox, works on Safari 8 too - Screenshot
Of course in a fallback mode, by opening a plain text blob in new tab.

Also window.safari is the same on vanilla Safari 8.

I'm facing the same issue. The patch proposed by @msokk works perfectly on Safari 9. By playing around I find that we could also redirect to the blob URL on this browser:
var blobUrl = window.URL.createObjectURL(myBlob);
window.location = blobUrl;

what am i missing in my testing? it didn't do anything for me when i tried. it seems the patch cannot affect safari9 because safari9 will never make it to line 110 because safari9 will not satisfy the conditional on line 109: if(window.safari), which stopped being set in safari8. therefore, if the patch works on safari9, then so too should the code without the patch, but it doesn't not. msokk showed a shot of the userAgent, but since safari allows changing that, we can't reliably use it.
that said, if a blob routine is now workable in safari, then it might be possible to fix somehow, it's just that this patch changes a part of the code safari9 can't see.
thanks for the feedback and info about URL, i will definitely check this out and see if it can be used to fix safari.
oh, and if i am misreading the patch somehow, i apologize in advance; i tried it without improvement and then realized that window.safari is not set in safari9, so i stopped testing/debugging; be insistent if that was in haste...

i would really like to get this fixed, but i don't want to just apply a patch that didn't work for me in testing...
the test page i'm using is at http://pagedemos.com/nqcrj2uy6ku4/ if you want to kick the tires...

@rndme I found out why window.safari is undefined for you.
Given that it contains only pushNotification property, some Safari security system for this disables the whole window.safari property, when accessed in an iframe (pushnotifications should not be registered in a frame)
and your demo link is in an <iframe>.
So I agree, we should have a better detection of Safari.

Updated the pull request with Safari user agent test from uap-core project.

kudos on solving the iframe mystery, that explains a lot. i'm not in love with sniffing userAgent strings, and i suppose an iframe in an iframe is a edge case. let me test again without an iframe in play see what results. i might have you revert to the window.safari version and merge that.
tldr; thanks, one final round of testing and i'll get it pulled...

I agree with UA sniffing being unreliable, I tried to use as tested source for that regexp as I could, uap-core is used by user-agent package and cdn.polyfill.io
If we're reverting back to window.safari, there should be a disclaimer in README.

:+1:
Could you release a new version with this fix?

done. must have slipped my mind last night, thanks for the heads-up!


SSH rake tasks

This adds new rake tasks to SSH into servers. Each task adds the local computer's public IP address to the corresponding security group, looks up the corresponding server's public DNS name, and execs ssh with the username and public DNS name. In the first commit, I wrapped the file in a module, so there's noise from the indentation and variable scope changes.

Very nice!


fix Data labeler role

we need to add following two actions in Data labeler.

Microsoft.MachineLearningServices/workspaces/labeling/projects/summary/read (To display a check box)
Microsoft.MachineLearningServices/workspaces/labeling/labels/read (To read the label itself)

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

ID: 0ca3ea12-4f8c-72e4-4703-ea2ef9dcee0f
Version Independent ID: a4b7c496-22f9-0fd6-cd7d-0fd613f1a1ec
Content: Manage roles in your workspace - Azure Machine Learning
Content Source: articles/machine-learning/how-to-assign-roles.md
Service: machine-learning
Sub-service: enterprise-readiness
GitHub Login: @johnwu0604
Microsoft Alias: johwu

@KazuOnuki Thanks for the feedback! We will review and update accordingly.

Thanks, Raised Pull request #https://github.com/MicrosoftDocs/azure-docs/pull/90935 for the Data labeler role.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Add layer related methods in ProjectSettings

Closes https://github.com/godotengine/godot-proposals/issues/1572
Closes https://github.com/godotengine/godot-proposals/issues/3930
This PR adds two methods to ProjectSettings:

get_layer_count(type): The only way to avoid hard-coding layer count as a magic number. Useful when the user decide to do something layer related.
find_layer(type, name): Helper method to find layer number by name. Toggling a layer dynamically in code is not uncommon. This avoids using magic numbers or boilerplate code.

I expected to be able to access layers via a global enumerator, I was surprised when I couldn't.
It makes sense to me to make layer names have a global enum that allows users to do something like:
set_collision_mask_value(Layers.My_Layer_Name, true)
It avoids string searching and double handling, if you need the bitmask maybe something like LayerMask.My_Layer_Name

Layer names can be changed at runtime.

The intention is to replace hard-coded layer values. That is, instead of writing this:
# Globals.gd
var COLLISION_LAYER_PLAYER = 3
var COLLISION_LAYER_HITBOX = 5

users can now write this instead:
# Globals.gd
var COLLISION_LAYER_PLAYER = ProjectSettings.find_layer(ProjectSettings.LAYER_PHYSICS_2D, "player")
var COLLISION_LAYER_HITBOX = ProjectSettings.find_layer(ProjectSettings.LAYER_PHYSICS_2D, "hitbox")

Users don't have to make sure whether the hardcoded number should be the number on that layer button, or the number in the button's tooltip.

 In which case, you still would want to store the names in a constant as for best practice.

It's the same problem for hard-coded layer numbers. If you decide to use layer 2 instead of layer 3 for player collision, you will have to look up every single 3 and make sure it's a layer. It's not a find_layer() problem.
Anyway, it's not related to the problem this function wants to solve.


docker 13 breaks docker::compose, compose uses static version and requires an internet connection...

class {'docker::compose':
    ensure       => present,
    install_path => '/bin',
}

I had to hack docker::compose in the meantime to be a file resource, but that's gross on my part because I have to update docker-compose in my puppet repo every time they change docker.
Additionally, the module currently defaults to using 1.7.0 in params.pp without any logic as to the installed version of docker.  I had to override that with some logic.  It would be better if that pulled the latest version somehow.

I found another fun bit - docker::compose seems current broken even with the new docker-compose.  I'm guessing they changed a docker command it used.  My resource:
  docker_compose { $composefile:
    ensure    => present,
    require   => File[$composefile],
    subscribe => File[$composefile],
  }

Info: Checking for compose project data
Debug: Executing: '/usr/bin/docker ps --format {{.Label "com.docker.compose.service"}} --filter label=com.docker.compose.project=data'
Error: /Stage[main]/Solr::Docker/Docker_compose[/data/docker-compose.yml]: Could not evaluate: Execution of '/usr/bin/docker ps --format {{.Label "com.docker.compose.service"}} --filter label=com.docker.compose.project=data' returned 1: template: :1:2: executing "" at <.Label>: Label is not a method but has arguments

Hey yet more information - looks like the broken docker::compose is a bug:
https://github.com/docker/docker/issues/28337

Specify the docker 12 version in your role as a workaround:
class {'docker':
version => '1.12.6-0~ubuntu-xenial',
}

Fixed for me with docker 1.13.1.


serde::Deserialize for tls::cli::ClapArgs plus additional bugfixes

Adds serde support for ClapArgs, making it easier for downstream users to incorporate it into a larger app configuration struct that deserializes from a configuration file. zbx_sender already depends on serde, so this is a low-cost additional feature.
Also, two bug fixes:

Add Send + Sync to the internal AsyncStream trait constraints, which allows it to be included in downstream async tasks.
Remove spaces from the IP range specified in CI. Zabbix docs say that spaces are supported, but CI broke on validating the specified CIDR. A review of Zabbix's parsing code looks like there is no allowance for spaces.

Okie dokie, that seems to have fixed the spurious CI problem that my previous, closed, PR had. Also some ergonomic improvements here for library use. (Selfish, of course. :-) )

Thanks!


Add the ability to use other fields in names of @Step

Add the ability to use other fields in names of @Step
eg. global variables, className

Relates #387

@lanwen I don't think we will get an extension point for #387 in nearest releases but this one can be easy solved

will not be resolved in Allure 1. Moved to https://github.com/allure-framework/allure-java/issues/14


BugFix: left arrow on loans, reservations, and lists returning to previous page instead of list books

Fixed with d92d30b


Removed flip faces for MergeMeshes

I tested this fix with:

https://playground.babylonjs.com/#E99FHV (glb merging)
https://playground.babylonjs.com/#YIU90M#782 (glb merging)
https://playground.babylonjs.com/#KBS9I5#76 (babylon file merging)

We do not need to flip the faces as the initial world matrix is merged as well so it is actually important to keep the indices the way they are.
Related forum discussion: https://forum.babylonjs.com/t/babylon-5-0-2-mesh-mergemeshes-flips-backface-of-glb-models/29043/11

As reference - fixed versions should be:
https://playground.babylonjs.com/?snapshot=refs/pull/12360/merge#E99FHV (glb merging)
https://playground.babylonjs.com/?snapshot=refs/pull/12360/merge#YIU90M#782 (glb merging)
https://playground.babylonjs.com/?snapshot=refs/pull/12360/merge#KBS9I5#76 (babylon file merging)

How was this not totally broken if it was flipping faces but shouldn't be?

Wait..something is really wrong with the build or something
I need to double check with @RaananW

How was this not totally broken if it was flipping faces but shouldn't be?

This is a very interesting case where the mix could work on some situation thanks to the overrideMaterialOrientation but like you I'm amazed that we did not see it earlier

Ok so I confirmed the fix now :)
https://playground.babylonjs.com/?snapshot=refs/pull/12360/merge#E99FHV (glb merging)
https://playground.babylonjs.com/?snapshot=refs/pull/12360/merge#YIU90M#782 (glb merging)
https://playground.babylonjs.com/?snapshot=refs/pull/12360/merge#KBS9I5#76 (babylon file merging)

I feel like we have had a few regressions with merge mesh. Would it be good to add some unit tests?

oh yes! I will

@ryantrem would be amazing if you could test in your app as well as adding one of your "crazy tests" in the test list as well :-)

I should be able to test these changes out later today.

There is no need to wait for me.

I tested these changes (by commenting out the three calls to _FlipFaces), and it definitely breaks merging with some of our models. Shall we sync directly?

Ideally a repro in the PG would be useful so I can compare with other (which are working)

Moving to Draft until we figure out why it is not working for @ryantrem models

Ok this was a very wrong fix..The code was actually correct and the problem was coming from the PG we used :(


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1735 ms

Resolved: Bofrost.at performance has improved in c6de185.


Support Typst 0.12

I'm not sure if it is okay to create a new issue but i cannot get some of the locate functions to not cast warnings.
Message:
locate with callback function is deprecated
Hint: use a context expression instead
Code:
  set page(
    paper-size,
    margin: (left: 25%),
    header: locate(loc => {
      if(loc.page() == 1) {
        let headers = (
          if (open-access) {smallcaps[Open Access]},
          if (doi != none) { link("https://doi.org/" + doi, "https://doi.org/" + doi)}
        )
        return align(left, text(size: 8pt, fill: gray, headers.filter(header => header != none).join(spacer)))
      } else {
        return align(right, text(size: 8pt, fill: gray.darken(50%),
          (short-title, short-citation).join(spacer)
        ))
      }
    }),

....more code.....

Hi @JeppeGivskud!
Thanks for the report! It seems to already be tracked in the typst repository: https://github.com/LaPreprint/typst/issues/7 😊 This repository is for the original LaTeX version.

Oh I'm so sorry haha. Thanks for the fast reply though!


Add test for get_package_id in Package class

Add a test to verify that the get_package_id
method in the Package class returns a proper
string containing the package name.version.
Resolves: #251
See also: #241
Signed-off-by: Rose Judge<EMAIL_ADDRESS>

Thank you!


Conflict with Autoptimize plugin

Hi, everybody!
I would like to provide a solution to a problem that some users may have when using the Google Web Stories WordPress Plugin on their websites. After installing the plugin, the editor works fine, but when validating the web page with my created story it did not pass Google's AMP Test validation. While investigating the problem I found the elements in the code that were automatically added by another cache plugin - Autoptimize and Cloudflare. It turns out that there was the conflict with Autoptimize plugin and Cloudflare Rocket Loader script.
I've fixed this as follows:

In the functions.php file of my theme I have added the following filter that disables the Autoptimize plugin for pages with a "stories" prefix.

// Disable autoptimize on pages with the word "stories" in the URL add_filter('autoptimize_filter_noptimize','my_ao_noptimize',10,0); function my_ao_noptimize() { if (strpos($_SERVER['REQUEST_URI'],'stories')!==false) { return true; } else { return false; } }

In Cloudflare dashboard in the "Page rules" tab I have added a rule for "stories" URLs.

For example:
websitename.com/stories/*
Then the settings are:
Disable Security > Security is disabled
Disable Performance > Performance is disabled
Summary
By doing this, I have finally validated my "stories" page with AMP Test in Google Search Console.
Here is my published page: https://casinoduende.com/stories/luckiacasino/
Thank you and I wish you much success with this useful tool!

@duendecasino thanks so much for reporting and sharing your solutions! We'll look into this further as well.
Also, fantastic job on the web story!

I've opened a PR to ensure that Autoptimize doesn't try to apply its optimizations to responses that WordPress serves which are initially AMP, as in the case of Web Stories: https://github.com/futtta/autoptimize/pull/340.

Just a quick follow up that I haven't been able to reproduce an issue with just Rocket Loader (but we'll keep an eye out should it be reported again). Updating this issue to just track Autoptimize compatibility.

This looks to be resolved in version 2.7.8 of Autoptimize. Closing.


docs(memory): Remove extra backtick

@ankitm123 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Add auav-x21_default  to the qgc firmware for distribution

This will not actually make it available. You still need to send a QGC PR. Will merge this one when the QGC PR is in.

@pkocmoud is working the QGC side of things.

I am working on this with Don. Getting this merged will be one more item off my list.

Travis-ci keeps getting worse. Might be time to move the QGC firmware build + upload to s3 over to semaphore.

@LorenzMeier
https://github.com/mavlink/qgroundcontrol/pull/4501
Can you please merge so I can finish my QGC testing?


Dragging a File into a Folder Doesn't Work

Often, when I try to move a file by dragging it over a folder, my mouse pointer changes to a crossed red circle, and the file refuses to be moved. I have to double click the folder to enter it, return back, then it will work.
This doesn't happen to every folder at once, some folders will still accept files at the same time as other exhibit this behaviour.

Hi @BeholderVee , thank you for filling this issue! However this repository is for the website of Files, please create issues for Files on the Files GitHub repository. Thank you!


How do you play your game? you press " A " or " D " to move left or right and to try and catch the star.

Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[x] The image is in the /games/img directory.
[x] The name of the image matches the name of your file.

Thanks for your PR!

Catch the pink star
Get Outlook for iOShttps://aka.ms/o0ukef

From: Lucas Honda @.>
Sent: Thursday, March 21, 2024 12:47:56 PM
To: hackclub/sprig @.>
Cc: Raul Montano @.>; Mention @.>
Subject: Re: [hackclub/sprig] catch the pink star (PR #1543)
CAUTION: [EXTERNAL EMAIL] This email originated outside Deer Park ISD. Please do not click links or open attachments unless you recognize the sender and know the content is safe.
@LucasHT22 requested changes on this pull request.
Thanks for the PR! What's the name of your game?
—
Reply to this email directly, view it on GitHubhttps://github.com/hackclub/sprig/pull/1543#pullrequestreview-1953005119, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHGDZCWQW7KQ4RWMDZBQX63YZMMMZAVCNFSM6AAAAABFB2CWLWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNJTGAYDKMJRHE.
You are receiving this because you were mentioned.Message ID: @.***>
DEER PARK ISD CONFIDENTIALITY NOTICE: This email/facsimile message, including all attachments and accompanying documents, is for the sole use of the intended recipient(s) and may contain confidential student and/or employee information. Unauthorized use and/or disclosure is prohibited under the federal Family Education & Privacy Act (20 U.S.C. section 1232g, 34 CFS Part 99, 19TAC 247.2, Texas Government Code 552.023, Texas Education Code 21.355, 29 CFR 1630.14(b)(c)). If you are not the intended recipient, you may not use, disclose, copy or disseminate this information. Please call the sender immediately or reply by email and destroy all copies of the original message, including attachments and accompanying documents. Disclaimer by DPISD77536

I'm pretty sure, it's something like that
Get Outlook for iOShttps://aka.ms/o0ukef

From: Raul Montano @.>
Sent: Thursday, March 21, 2024 12:58:47 PM
To: hackclub/sprig @.>; hackclub/sprig @.>
Cc: Mention @.>
Subject: Re: [hackclub/sprig] catch the pink star (PR #1543)
Catch the pink star
Get Outlook for iOShttps://aka.ms/o0ukef

From: Lucas Honda @.>
Sent: Thursday, March 21, 2024 12:47:56 PM
To: hackclub/sprig @.>
Cc: Raul Montano @.>; Mention @.>
Subject: Re: [hackclub/sprig] catch the pink star (PR #1543)
CAUTION: [EXTERNAL EMAIL] This email originated outside Deer Park ISD. Please do not click links or open attachments unless you recognize the sender and know the content is safe.
@LucasHT22 requested changes on this pull request.
Thanks for the PR! What's the name of your game?
—
Reply to this email directly, view it on GitHubhttps://github.com/hackclub/sprig/pull/1543#pullrequestreview-1953005119, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHGDZCWQW7KQ4RWMDZBQX63YZMMMZAVCNFSM6AAAAABFB2CWLWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTSNJTGAYDKMJRHE.
You are receiving this because you were mentioned.Message ID: @.***>
DEER PARK ISD CONFIDENTIALITY NOTICE: This email/facsimile message, including all attachments and accompanying documents, is for the sole use of the intended recipient(s) and may contain confidential student and/or employee information. Unauthorized use and/or disclosure is prohibited under the federal Family Education & Privacy Act (20 U.S.C. section 1232g, 34 CFS Part 99, 19TAC 247.2, Texas Government Code 552.023, Texas Education Code 21.355, 29 CFR 1630.14(b)(c)). If you are not the intended recipient, you may not use, disclose, copy or disseminate this information. Please call the sender immediately or reply by email and destroy all copies of the original message, including attachments and accompanying documents. Disclaimer by DPISD77536

Sorry for the delay! So could you rename the file? You can do it going in Files changed > ... > edit :)

Thank you for sharing your project! I think this game needs some refinement. It never seems to end and the music can become quite chaotic fairly easily.
I'm going to close this PR for housekeeping but feel free to resubmit when you are ready.


Update Github action for mdbook

What was wrong?
book.yml didn't trigger a Github action, and the mdbook didn't deploy to Github pages.
How was it fixed?
Update book.yml to test/build/deploy the mdbook. It will run test/build when PR are for the master branch, and it will deploy a page only when commits push to the master branch. Deployed page.

Rename branches from main to master.
Add book.yml to paths.
Update some steps in jobs. Reference: running-rust-tests, starter-workflows, workflow to publish your site.

To-Do

[x] Add entry to the release notes (may forgo for trivial changes)
[x] Clean up commit history

It looks like we need to enable the GitHub page setting. GitHub page > Build and deployment > Source > Github Actions


Alfred 4 support

Hello,
tldr-alfred doesn't work in Alfred 4 as was noted in #9 issue.
I updated Alfy to v0.9.x to support Alfred 4 and upcoming patches.
Also, there was an issue with Buffer() deprication. Not sure if my solution is suitable, but it works at least 😸

Thanks for the PR!


Determine cache name based on formatted date

The cache name was based on dateTime objects, but since Google Analytics reports work with days as smallest granularity, it makes more sense to create cache names based on a formatted date (and not dateTime).

thanks


add version number in variable

__VERSION__ should be a global variable containing the current YASL version.

added in v0.8.2


Update README with poetry instructions and fix typos

Also, update example.env to have the development link instead of to prod.

Love good docs 🚀


Openssl uefi

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Description of change

These are bugfixes for a supported build config, so 1.1.0 as well as master.
But @qloong we'll need you to sign an individual CLA as well.  And if you're not LuGong you need to get Dan Zimmerman to add you to the Intel corporate CLA as well.

dang button mistake!

Done on ICLA & CCLA.
From: Rich Salz<EMAIL_ADDRESS>Sent: Wednesday, March 15, 2017 11:54 PM
To: openssl/openssl<EMAIL_ADDRESS>Cc: Long, Qin<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [openssl/openssl] Openssl uefi (#2961)
These are bugfixes for a supported build config, so 1.1.0 as well as master.
But @qloonghttps://github.com/qloong we'll need you to sign an individual CLA as well. And if you're not LuGong you need to get Dan Zimmerman to add you to the Intel corporate CLA as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/openssl/openssl/pull/2961#issuecomment-286786976, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFlyP47xHdN-ztjNPGxstjPxmPYSBmC6ks5rmAm0gaJpZM4MeImn.

@qloong, there's one detail missing, as explained in the mail you replied to: you need to get Dan Zimmerman to add you to the Intel corporate CLA as well.

CLA now ok. Ping @levitte

Merged into master: cff55b90e95e1fa6c90154f93f12363e761d88c7 and dc8da7b1b8ba276f179ea1232c9df764dcd38775
1.1.0 needs a separate PR, the statem_lib.c change doesn't cherry-pick cleanly, sorry.


Notification if device is disconected

I would like this feature, but i think that actualy this funcionality is not posible.

This is the same as this: https://github.com/intel-iot-devkit/tinyb/issues/49. It needs extra support in BlueZ.


switch to module build for vitest

TODO:

[ ] make sure this still works with Jest

fixes: https://github.com/solidjs/solid-testing-library/issues/8

An updated version of moving to ESM (tested with uvu, vitest and jest) has already been merged.


As admin, I want to be able to delete chat rooms, so that there isn't unwanted spread of bad content

The admin can actually delete things from the database already, so it's technically possible


Connection Error

Bot works fine for a few hours then i get a failed to establish a new connection error and have to restart the bot.
ConnectionError: HTTPSConnectionPool(host='<IP_ADDRESS>', port=63778): Max retries exceeded with url: /lol-gameflow/v1/gameflow-phase (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x000002A29922FDD0>: Failed to establish a new connection: [WinError 10061] No connection could be made because the target machine actively refused it'))

screenshot

Took a better screenshot this time.

Hmm can you comment out line 12 in bot.py and uncomment line 13 and then run the bot? If when it runs into this issue again can you drop the log file here for me to see (it should be on your desktop called log.txt). It will help me figure out the issue. Looks like league is maybe running into connection issues after a while or something.

Hi, same problem, it seems to always happen when it disconnects from a game and it doesnt reconnect again ( i always catch it on the end-game screen showing me as Leaver with 0-0 score )
i did that with line 12 and 13 in bot.py but i didnt get the log file

I believe @Aeikir's issue is related to #13, rather than this issue. I will still need logs from @saulcanelo if this issue happens again in order to pin down exactly what is going on.


Exception in thread "main" java.io.FileNotFoundException: Can not find core jar file in path

Exception in thread "main" java.io.FileNotFoundException: Can not find core jar file in path

检查一下这个path的值是什么

This issue has been too long to review it and reopen new one if necessary.


Does this also works for Italy or Spain visa slot check?

Hey @Hongbo-Z , Thanks for reaching out. Sorry, I'm not sure. You can try it by changing the VFS url.

Added support for Italy


Selecting single spec from spec list causes all tests to rerun

Issue description
running ng test in angular project and selecting single spec from spec list causes all tests to rerun
Steps to reproduce the issue

npm install -g @angular/cli
ng new test-app
cd test-app
ng test
clic on a single spec

What's the expected result?

only selected spec gets to run again

What's the actual result?

all the specs are re run

Additional details / screenshot

Dependencies:

"@angular-devkit/build-angular": "~0.1001.4",
"@angular/cli": "~10.1.4",
"@angular/compiler-cli": "~10.1.4",
"@types/node": "^12.11.1",
"@types/jasmine": "~3.5.0",
"@types/jasminewd2": "~2.0.3",
"codelyzer": "^6.0.0",
"jasmine-core": "~3.6.0",
"jasmine-spec-reporter": "~5.0.0",
"karma": "~5.0.0",
"karma-chrome-launcher": "~3.1.0",
"karma-coverage-istanbul-reporter": "~3.0.2",
"karma-jasmine": "~4.0.0",
"karma-jasmine-html-reporter": "^1.5.0",
"protractor": "~7.0.0",
"ts-node": "~8.3.0",
"tslint": "~6.1.0",
"typescript": "~4.0.2"

Specs and spec selection is done by plugins, not by the core code. The core code has no idea what a "spec" even is ;-)

Hi guys,
I was running into the same issue with angular versions 9 and 11. I updated all karma/jasmine related packages to the latest version:
    "jasmine-core": "~3.6.0",
    "jasmine-spec-reporter": "~5.0.0",
    "karma": "~5.2.3",
    "karma-chrome-launcher": "~3.1.0",
    "karma-coverage-istanbul-reporter": "~3.0.2",
    "karma-jasmine": "~4.0.0",
    "karma-jasmine-html-reporter": "^1.5.0",

Using Windows 10 / Chrome Version 87.0.4280.88.
When clicking on a single spec the whole page reloads and all tests are re-run.
Somewhere it was mentioned that using the debug mode helps. However, clicking a spec in debug mode also redirects me to the regular karma page and thus reruns all tests.
I found a workaround by changing the toQueryString function to always include debug.html.
To apply the fix: Open node_modules\karma-jasmine-html-reporter\src\lib\html.jasmine.reporter.js and look for the toQueryString function at the bottom of the file.
Change the following line:
return '?' + qStrPairs.join('&');

to
return 'debug.html?' + qStrPairs.join('&');

For<EMAIL_ADDRESS>this is in line 794.
This leads to the expected behaviour: Clicking on a component or spec just reruns the tests of that component or spec.
I'm not sure why this is happening with different angular / karma versions. Maybe someone can clear up which package is causing this problem. I hope that this problem can be fixed so that we don't have to rely on a workaround.

Nice, we shoud definitely report to karma-jasmine-html-reporter

karma-jasmine-html-reporter


Feature request: keyCode accept string as argument

The keymap dict can be find here : https://github.com/wesbos/keycodes/blob/gh-pages/scripts.js. With this feature, the api will be much more intuitive.

Pretty good idea, next version is coming soon :)

That's will be really nice.
By the way, why the latest commit time is 2017 December...

So sad about that, my system clock has been broken last year...

New version & documentation has arrived, enjoy it.

https://dafrok.github.io/react-shortcut-key/


Install process duplicates Imports of targets files

This happens only if the targets file is directly imported with __paket__ properties like:
<Import Project="..\..\..\GitInfoPlanter\build\GitInfoPlanter.targets" Condition="Exists('..\..\..\GitInfoPlanter\build\GitInfoPlanter.targets')"  />

The reason is that we don't have this tagged with any paket specific markers and can't delete it.
So we need to tag it. Ideas?
/cc @damageboy

I added a Paket attribute like:
<Import Project="..\..\packages\GitInfoPlanter\build\GitInfoPlanter.targets" Condition="Exists('..\..\packages\GitInfoPlanter\build\GitInfoPlanter.targets')" Paket="true" />

This seems to be ignored by Visual Studio and works as a flag.

@forki Verified!

@forki New Bug... You seem to have added a Paket=True attribute for the <Import>s that you add.
Unfortunately, VS doesn't seem to like loading those...
It does seem though, that it's legal to specify a Label attribute,
So, instead of:
<Import Project="..\..\..\GitInfoPlanter\build\GitInfoPlanter.targets" Condition="Exists('..\..\..\GitInfoPlanter\build\GitInfoPlanter.targets')"  Paket="True" />

You could do:
<Import Project="..\..\..\GitInfoPlanter\build\GitInfoPlanter.targets" Condition="Exists('..\..\..\GitInfoPlanter\build\GitInfoPlanter.targets')"  Label="Paket" />

Argh. Will check later today.
If you are interested: it's probably an easy fix in only two locations. ;-)
On Feb 8, 2015 12:32 PM, "damageboy"<EMAIL_ADDRESS>wrote:

@forki https://github.com/forki New Bug... You seem to have added a
Paket=True attribute for the s that you add.
Unfortunately, VS doesn't seem to like loading those...
It does seem though, that it's legal to specify a Label attribute,
So, instead of:

You could do:

—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/585#issuecomment-73407100.

@forki Good idea, it's time I got off my fat ass, wait for my PR, I'll feel more productive that way


Test on io.js on Travis

Testing io.js on Travis

@rvagg What's up with pangyp and node 0.8? https://travis-ci.org/rvagg/nan/jobs/49679768


Remove duplicate Slycot error handling, require Slycot >=0.4

There is a lot of unused code in mateqn.py, because Slycot itself warns and raises its own exceptions in case of invalid inputs or failing numerical routines.
I guess this also overlaps a bit with #670.

Coverage increased (+2.5%) to 92.716% when pulling af4c51d579341164f458189f62f36cc149e11b90 on bnavigator:slycot-handling into 81a4f38e39efcb2d1fdf1388f5c1567a4aa2f791 on python-control:master.

LGTM.

Thanks.


allow construction inside a separate effect from that of the circuit breaker

CircuitBreaker as currently written doesn't support the pattern described by Ref.in[F, G], in which the unsafe Ref is constructed safely inside effect F while exposing its own state via effect G. I propose implementing this pattern by generalizing the existing constructor to use Ref.in instead of Ref.of.

thank you for the merge!
any chance of a release with this sometime soon?


Function utf8_encode() is deprecated in PHP 8.2

Testing under PHP 8.2 gives a deprecation notice when signing an event here:
https://github.com/swentel/nostr-php/blob/f4e3b60fe2610f431ea503453eebf67bb7ed7ec4/src/Sign/Sign.php#L28

I saw that yesterday. I will make a PR

@swentel see my comment here: https://github.com/swentel/nostr-php/pull/25#issuecomment-1486671849


Bugfix

Add fix for https://github.com/guardrails-ai/guardrails/issues/479

Bug description: The validated_response object gets passed to schema verification logic here. While this verification happens, that object gets over-written on the following lines:

https://github.com/guardrails-ai/guardrails/blob/33bd89d8c9bbd0ea55990f3dcb02a15e7cd6d30c/guardrails/utils/json_utils.py#L180
https://github.com/guardrails-ai/guardrails/blob/33bd89d8c9bbd0ea55990f3dcb02a15e7cd6d30c/guardrails/utils/json_utils.py#L228

This causes the optional null values to be over-written accidentally by the boolean that is being returned by the function. Unsure whether this is necessary - as the entire point of schema verification is to return only a boolean and not update the raw LLM response in any way.  Updates/re-writes to the LLM response only take place during field validation.
@CalebCourier @zsimjee to confirm if this is correct.
Solution

Comment-out/remove the 2 lines where the json_value is being updated.

Result

This solves the original issue
All existing tests still pass

Other updates to code:

Add bug-fixes in OpenAI v1 implementation

Correctly parse OpenAI output - either using content or function_call arguments

Other minor bug-fixes

If these changes look okay, TODO @thekaranacharya : To remove the commented lines altogether.


Post error status in exception

Once an exceptional error is returned, report it by API and reboot the device.
This make be able to explicitly fire alert with reasons.

raise may be available for this
https://uxmilk.jp/39845

import utime

def timer():
    if utime.ticks_diff(utime.ticks_ms(), start) > 2000:
        raise ValueError
    time = utime.ticks_diff(utime.ticks_ms(), start)
    print(time)
    utime.sleep(1)

start = utime.ticks_ms()

while True:
    try:
        timer()
    except ValueError:
        start =+ 1
        print(start)
    except RuntimeError:
        print("Run time err")
        raise
    except Exception:
        # print(f"Unexpected {err}, {type(err)}")
        print("Unexpected")
        raise


variable replacement bug

var x = 5
var y = 7
var z

compare x < y -> z

var z compare x
x doesn't get replaced here because t-3 is var

fixed 7b76335ee506b0436020f04f1fe0ad8404544764


Progress Indicator?

Would be good if it had some rudimentary way to track progress or to at least indicate activity.

From further discussion in IRC:

install-update needs progress indication
would like to see when each KBID starts or completes

but even dots or a spinbar showing that the script has not frozen would be an improvement.

used in Win7/8 (and later, 10) deployments

Well, I've been doing some research and so far it doesn't look good.
I access WU via the Windows Update Agent API  which is a set of COM interfaces.
WUA exposes the IUpdateDownloader interface for downloading updates. With this we can user the Download to download all pending update synchronously. That's what the script does now.
But this method's drawback is that it downloads them all as one big lump, tying up the script until it is done. It doesn't emit any status. So there's no data to base a status bar on. I could display a spinning bar or something but it would be totally fake: if the download job hung, the spinbar would just spin forever.
I experimented with telling the IUpdateDownloader to get only one of the available updates, print the update's KBID number, then go download the next one. This allowed the script to report progress more frequently, but it made the overall download take almost twice as long.
IUpdateDownloader has an asynchronous BeginDownload method which would emit status at regular intervals, but alas, PowerShell currently has no way to subscribe to asynchronous COM object methods. (Well, I didn't find one, anyway.)
The IUpdateInstaller works basically the same way as the downloader works. Synchronous does it all in one lump without status; asynchronous emits status as it goes but is not accessible from PowerShell.
All of this has baked my noodle! I'll keep thinking on it, and ask a couple of PS gurus I know. You can call Install-Update with the -Verbose parameter, which will give you a tiny bit more output, but for now I think that is as good as it gets.

It may be possible to put this technique to use: http://goo.gl/TUsFd3, http://goo.gl/QwcT3S
Stick the IUpdateDownloader.download and IUpdateinstaller.install jobs in their own threads, then check the $updatelist object from time to time. Man I wish there was a .NET interface to WUA!
I'll fiddle with that sometime soon.

from IRC:
S_SubZero: mota: yeah, the actual update info is not important, just something spinning or
          otherwise "still going"
S_SubZero: they used to make that spinny thing with \ / - |    8)

May take another run at using the async methods of IUpdateDownloader and Installer.. https://msdn.microsoft.com/en-us/library/windows/desktop/aa386132(v=vs.85).aspx

This guy has async with progress indication working ... in vbscript. https://github.com/vattay/WuF


Set up PR vs production configs for the deploy action

We are currently building the full recipe for every commit (if a label is set).
I think we should have some things set in two different config jsons based on the event type:

Push to main:

Prune set to false
Set a permanent directory in leap-persistent-rofor each recipe and subfolders for each action run

PR:

Prune set to true
Write resulting data to temporary storage (e.g. leap-scratch)

This could be achieved with conditionals in the workflow file (and no changes needed to the action). Or it could be a PR to the action. Either seems doable.

I would think the first option is more flexible right now? I guess we really need some more users of the action to see how we might properly generalize this?

I would think the first option is more flexible right now?

Agreed. A relatively manageable path forward would be to use a github workflow step that conditionally exports either env vars (or workflow step outputs) based on the github event type, which can then be inserted into the pangeo-forge-runner JSON config.

In #33, I currently have the following parts of this working:
Push to main:

[x] Prune set to false
[ ] Set a permanent directory in leap-persistent-rofor each recipe and subfolders for each action run

PR:

[x] Prune set to true
[ ] Write resulting data to temporary storage (e.g. leap-scratch)


Problems with QOpenGLWidget as embedded widget

Thanks for this great library!
I saw that there was an issue related to nodes with OpenGL and 3D but it has been closed without solutions (https://github.com/paceholder/nodeeditor/issues/261).
I'm trying to embed a QOpenGLWidget and I'm having some problems with the refresh, update() doesn't seem to work while I move the mouse (the content should rotate), it only refreshes once after a while, when I trigger other events not directly related in the "view" or leave the MainWindow with the pointer.
I inserted the QOpenGLWidget in a resizable node, sometimes it is drawn black (usually it draws an rgb triangle on a black background), today it doesn't happen but during the tests I had also seen some graphic glitches (like if it blit dirty/random vram in some parts), is OpenGL supposed to work? (see edit2)
I used QOpenGLWidget for convenience but if there are other more compatible 3D visualization solutions I'm open to suggestions (preferably hw accelerated). (QOpenGLWidget seems to work just fine)
Also, is there any way to filter the Wheel event for the embedded widget? I'm failing, I would like to zoom the content in/out with the wheel, but instead the event zooms the scene/view.
Here a gist with the standalone version of the widget I'm using (https://gist.github.com/xenogenesi/0dd16048e8b753e907abfb58b4e5ee04) for testing.
edit: I noticed just now that the mouse move event and update() behave as expected when the scene/view is zoomed in
edit2: The resize problems were due to the aspect ratio computed with integers in gluPerspective() (should also be fixed in the gist).
As for the mouse move events, it's not clear to me where the problem is, it seems to work when the widget has a certain size or the scene/view has a certain zoom factor. Probably something wrong with how I handle events and force refresh?
bool NaiveDataModel::eventFilter(QObject *object, QEvent *event)
{
    if (object == _triangle) {
        if (event->type() == QEvent::MouseButtonPress) {
            QMouseEvent *mouseEvent = static_cast<QMouseEvent*>(event);
            qDebug() << "filter mouse press:" << mouseEvent->position();
            _triangle->mousePressEvent(mouseEvent);
            return true;
        } else if (event->type() == QEvent::MouseMove) {
            QMouseEvent *mouseEvent = static_cast<QMouseEvent*>(event);
            qDebug() << "filter mouse move:" << mouseEvent->position();
            _triangle->mouseMoveEvent(mouseEvent);
            QRect r = _triangle->rect();
            qDebug() << "filter _tirangle rect:" << r;
            MyApplication *app = qobject_cast<MyApplication *>(QCoreApplication::instance());
            if (app) {
                app->view->invalidateScene();
            }
            return true;
        } else if (event->type() == QEvent::Wheel) {
            QWheelEvent *wheelEvent = static_cast<QWheelEvent*>(event);
            qDebug() << "filter mouse wheel delta angle:" << wheelEvent->angleDelta();
            return true;

Clearly it's not a QOpenGLWidget problem that seems to work, instead it's my problem to trace the coordinates in the scene/view of the node and/or the instance of the node to refresh. For the wheel event I think it will be necessary to subclass the view.


Defer support

This builds on #29 by implementing a "more correct" approach. Readable.from is used to convert the asyncIterable to a Readable which is then forwarded to Koa with little fuss. This implementation should have slightly better error and cancellation propagation semantics.
Also next is no longer invoked on error, since this was definitely not correct and prevents error middleware from working correctly. This would cause the server to respond with a status of 200 and the default body of "OK". Since GraphQL will return JSON error responses (instead of throwing) this change should only affect the code path where a multipart response was rejected due to lack of support. Therefore, a major semver change is not needed.
@matthew-gordon @trevor-scheer

Hey @laverdet - I checked out the PR and it looks like the defer integration tests are failing. This isn't obvious...they're skipped locally by default and CI isn't running against your PR.
For local dev you'll need to set the environment variable to enable them (which the workflow does) like so:
INCREMENTAL_DELIVERY_TESTS_ENABLED=t npm test

Thank you for pointing that out. These tests are also failing in the core apollo-server package.
$ git clone git@github.com:apollographql/apollo-server.git
$ cd apollo-server
$ npm i -D
$ INCREMENTAL_DELIVERY_TESTS_ENABLED=thanks npm test
[...]
  ● integration tests › httpServerTests.ts › incremental delivery › tests that require graphql@17 › basic @defer working with accept: multipart/mixed; deferSpec=20220824

    expect(received).toEqual(expected) // deep equality

    Expected: 200
    Received: 406

      2350 |                 query: '{ first: testString ... @defer { testString } }',
      2351 |               });
    > 2352 |             expect(res.status).toEqual(200);
           |                                ^
      2353 |             expect(res.header['content-type']).toMatchInlineSnapshot(
      2354 |               `"multipart/mixed; boundary="-"; deferSpec=20220824"`,
      2355 |             );

      at ../integration-testsuite/src/httpServerTests.ts:2352:32
[...]
  ● integration tests › httpServerTests.ts › incremental delivery › tests that require graphql@17 › basic @defer working with accept: multipart/mixed; deferSpec=20220824, application/json

    expect(received).toMatchInlineSnapshot(snapshot)

    Snapshot name: `integration tests httpServerTests.ts incremental delivery tests that require graphql@17 basic @defer working with accept: multipart/mixed; deferSpec=20220824, application/json 1`

    Snapshot: "multipart/mixed; boundary="-"; deferSpec=20220824"
    Received: "application/json; charset=utf-8"

      2351 |               });
      2352 |             expect(res.status).toEqual(200);
    > 2353 |             expect(res.header['content-type']).toMatchInlineSnapshot(
           |                                                ^
      2354 |               `"multipart/mixed; boundary="-"; deferSpec=20220824"`,
      2355 |             );
[...]

and so on. The result of executeHTTPGraphQLRequest is not correct according to the tests but that's certainly out of the scope of this PR. #29 fails the new tests as well.

Oversight on my part. I'll be at my desk in a bit to try this out but you might beat me to it. My instructions also require you to install graphql@alpha locally for those tests to pass. Sorry about that! Yours very well may be in good shape.

I was able to validate that tests are passing here, sorry for the confusion!


Check if an integer is a power of 2

What the program do ?
Simple algorithm using a bit manipulation trick to check if an integer is a power of 2.
In what programming language it is written?
Python 2.x

About which program you are asking

I do not understand you


can not open .xlsx or .xls  files in Code.

i need to handle the data using python

This is expected. Use a spreadsheet app like Excel to open them instead.
To read excel files in Python, refer to this: https://pandas.pydata.org/docs/reference/api/pandas.read_excel.html

It helps a lot. Thanks!


Show free space in drives

Hello!
Please make a bar of free space usage of drives in menu. Like this:

If we add a page to see all your drives as shown in #210 would that solve this issue?

Im look on your concept of this drive page, yes, it would be good. Please make it!

I will close this issue and we will continue tracking the issue in #210. It is something we got a lot of requests for and is part of the plans for the future.


More accurate validation on labels and annotations

Codecov Report
Attention: Patch coverage is 87.14286% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 94.27%. Comparing base (d9e7b3b) to head (a138844).

Files
Patch %
Lines

internal/parser/parser.go
87.14%
8 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #834      +/-   ##
==========================================
- Coverage   94.60%   94.27%   -0.34%
==========================================
  Files          92       92
  Lines       10036    10095      +59
==========================================
+ Hits         9495     9517      +22
- Misses        402      435      +33
- Partials      139      143       +4

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


Add Kubernetes authentication option based on EKS

This introduces a new authentication scheme for kubergrunt, where you can provide an EKS cluster ARN. In this auth scheme, we will do the following:

Obtain a temporary authentication token from EKS for the k8s API.
Create a new temp file to act as a kubeconfig file.
Setup the default context in the temp kubeconfig to point to the EKS kubernetes API.
Setup the default context in the temp kubeconfig file to use the auth token pulled down in step 1.

This allows you to do something like the following without setting up the kubeconfig with eks configure:
kubergrunt k8s kubectl --kubectl-eks-cluster-arn $EKS_CLUSTER_ARN -- get pods -n kube-system

Thanks for review. Will merge after doing integration testing in the EKS repo.

Ok ran through some regression testing in the EKS repo, found a few bugs, but now this is fixed and working so going to merge + release. Thanks for review!


DOC: Fix incorrect docs for annotate_movement [ci skip]

Units were incorrectly documented for both of these. Could change the code to reflect the SI unit for rad/sec for rotation velocity limit, but given that degrees/s was the existing behavior this seems like a less invasive change (and also the units are wrong in docs for translation and must be changed anyway).
Pushed with [ci skip] because we don't need CIs for this I think :)

Thanks @larsoner


Is there any limit on word count per request, per day, per month etc.?

Is there any limit on word count per request, per day, per month etc.?

This will not be fixed, it depends on the risk factor of your IP.


文档补充

What problem does this feature solve?
对应yunzai的YunzaiBusines业务模块，需要在ng.yunzainfo.com网站生成时生成对应文档
原则上不破坏原有文档，只有附加内容
What does the proposed API look like?
暂无

Closed https://github.com/hbyunzai/yelon/commit/a502680ca3e3d645774734ff405a95e94c58f4d2


Add support for flat repositories with no Release file.

In flat repositories, the Release file is supported, but not mandatory,
according to the specification here:
https://wiki.debian.org/RepositoryFormat#Flat_Repository_Format
An example for such a repository is the MapR ecosystem:
http://archive.mapr.com/releases/ecosystem-5.x/ubuntu/binary/
Before this patch, it was not possible to create a mirror of that
repository. The new behavior is to ignore the missing metadata, and
to implicitly disable checksum checks, since there are no checksums to check.

Do you know if this change will allow aptly to work with http://archive.mapr.com/releases/ecosystem-5.x/ubuntu/ ?
Also any plans to merge this soon?

@jschwartz79 I didn't have a chance to look into that yet, but http://archive.mapr.com/releases/ecosystem-5.x/ubuntu/ looks more like regular repo. Just try putting literally apt-sources.list line to aptly mirror create xxx and see where it goes ;)

@ffrank are you still interested in working on this?
From a casual glance, this would benefit greatly from either a system or a unit test.

Sure, I can see about adding some tests. Thanks for picking this up.
From experience, unpatched aptly will not play well with the MapR repositories (or, at all).

After rebasing, the patched code doesn't compile anymore (see Travis output). I can't seem to be able to make any sense of that error. Help?

@ffrank this identifier was renamed to http.Error (https://github.com/aptly-dev/aptly/blob/master/http/http.go#L9)

Thank you @smira. Missed the forest for all the trees.
Looks much better know. LMK if you feel those tests are inadequate or need refinement.

Is the code coverage check a blocking issue?

Code coverage decrease shouldn't be blocking for us, but I wonder if this new behavior is good - the test you modified for suslik is actually hitting a distribution which doesn't exist at all.  So it's not expected that aptly mirror create should succeed. I think we might at least try to HEAD Packages file to make sure it's there?
And probably we need some system test with flat repository which does mirror update to make sure there's no regression for flat repositories in the future.

Very good points. Not quite sure how that test should look right now, but it will probably become clear once I sit down and try.
As for checking the Packages file, I looked for an available function for making a HTTP HEAD call from the http package, but didn't find one. Should I just make one? (Or just use stdlib from Fetch, ignoring the downloader object?)

@ffrank probably better adding HEAD method to the downloader, it shouldn't be that complicated as existing Download methods


Github Polling Extension

Should we try out this github polling extension thing?
/polls Option1 'Option 2' "Option 3"

This is a failed test


fix: Function.prototype.bind was failing on new BoundFn()

add a new test
use the standard es5-shim polyfill without any changes BUT include
the necessary helpers/variables

This error could be spotted by using ESLint for example (undeclared
variable).

cc @triblondon let me know what you think, this is keeping me from using polyfill-service for now :)

(ran the tests locally and it was fine).
Also I have seen projects like ember making their sauce keys public for ease of collaboration, what do you think?
thanks

@triblondon @jonathantneal @samgiles would love to have your input on this one, thanks!

I see you released 2.0.0 4 days ago, let me know if you need anything from this PR.
Current bind implementation is broken


Extracting npz weights file after training own model

Hi,
I trained the resnet-v1-50 using the train.py given here. It produces checkpoint.index, checkpoint.meta and checkpoint.data files after completion of training. However, I wanted to have the weights file as .npz file similar to the weights file provided by you for the MARS and MARKET dataset so that I can use my model with the trinet_embed.py file. Can you please suggest how can I get the .npz file after my training?
Thanks in advance.

These two codes are incompatible. The .npz file and trinet_embed.py is only for our original experiments we did in Theano+Lasagne. When you train your own model here in TensorFlow, you need to use embed.py, but this is all described in the README, so closing this issue.


sudo: policy plugin failed session initialization

Something wrong with sudo, I update my arch on wsl today(2017/09/13) and I get this error when use sudo.
sudo -i
sudo: getrlimit: Function not implemented
sudo: setrlimit: Function not implemented
sudo: setrlimit: Function not implemented
sudo: pam_open_session: Critical error - immediate abort
sudo: policy plugin failed session initialization
and I find that
warning: fakeroot: ignoring package upgrade (1.21-2 => 1.22-1)
warning: pacman: ignoring package upgrade (5.0.1-4 => 5.0.2-2)
when Syyu.
Otherwise there is no warning in my another arch on PC.

The warnings are by design, this keeps the alwsl-patched pacman alive.
The other missing implementations seem like missing syscalls, not really something I think alwsl could be responsible for.

I see , thank u anyway , I'll try to find some other help

@daihaovigg have you find a solution ? i have the same problem with many software


Fix typos and format in comments

Fix typos and format in comments about the registry manager of
packed functions.
Signed-off-by: Gustavo Romero<EMAIL_ADDRESS>

@comaniac Thanks for the review. The CI linter caught a line with 101 chars and complained about it, so I've just pushed a fix for that. It must be alright now.

@gromero @comaniac Thanks!