[Build] Collect frontend code coverage from Cypress tests

CATEGORY

[x] Build / Development Environment

SUMMARY
College frontend code coverage from Cypress tests and add proper tagging for coverage reports.
With this, we can finally start to take code coverage numbers more seriously.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION
N/A
REVIEWERS

Codecov Report

Merging #9513 into master will decrease coverage by 27.80%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9513       +/-   ##
===========================================
- Coverage   58.57%   30.77%   -27.81%
===========================================
  Files         386       74      -312
  Lines       12287     3214     -9073
  Branches     3025      698     -2327
===========================================
- Hits         7197      989     -6208
+ Misses       4906     2130     -2776
+ Partials      184       95       -89

Flag
Coverage Δ

#cypress
30.77% <ø> (?)

#ui
30.77% <ø> (?)

Impacted Files
Coverage Δ

superset-frontend/src/utils/safeStringify.ts
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/components/Timer.jsx
12.50% <0.00%> (-84.38%)
:arrow_down:

...uperset-frontend/src/utils/getClientErrorObject.js
0.00% <0.00%> (-80.00%)
:arrow_down:

superset-frontend/src/components/Link.tsx
7.69% <0.00%> (-79.81%)
:arrow_down:

...et-frontend/src/SqlLab/components/TableElement.jsx
4.59% <0.00%> (-78.17%)
:arrow_down:

...src/components/FilterableTable/FilterableTable.jsx
4.39% <0.00%> (-76.38%)
:arrow_down:

superset-frontend/src/explore/exploreUtils.js
1.78% <0.00%> (-75.00%)
:arrow_down:

...-frontend/src/SqlLab/components/HighlightedSql.jsx
16.66% <0.00%> (-73.34%)
:arrow_down:

...end/src/SqlLab/components/ExploreResultsButton.jsx
5.00% <0.00%> (-68.75%)
:arrow_down:

...t-frontend/src/SqlLab/components/ColumnElement.jsx
33.33% <0.00%> (-66.67%)
:arrow_down:

... and 356 more

Continue to review full report at Codecov.

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

I think GitHub + Travis just works very badly with force push.


doc(router): fix data type of currentInstruction

I have not realised this is not intended: https://github.com/aurelia/router/issues/472


[SSR] Hydration mismatch when createResource throws an error

Describe the bug
When createResource throws an error (e.g. rejects a promise), the hydration mismatch error is thrown.
Another thing to notice here: there's no hydration mismatch if createResource throws an error without a promise (e.g. without async). It feels like the behavior should be unified no matter whether it returns a promise or not.
Your Example Website or App
https://stackblitz.com/edit/solid-ssr-vite-81jsyh?file=src/routes/index.tsx
Steps to Reproduce the Bug or Issue

This bugs can only be reproduced when using SSR.
Go to the console and observe the hydration mismatch error.

Expected behavior
There should be no error during hydration.
Screenshots or Videos
No response
Platform

OS: macOS 12.6.8
Browser: Chrome
Version: 118

Additional context
No response

Thank you for reporting. Synchronous resolution of values work but errors from serialization. Which means that loading states are breaking hydration. Will get this fixed up.


SBT 1.5 prompt seems broken in git-bash + Windows.

steps
Minimal reproduction
I understand this may or may not be a bug in SBT since it only seems to occur with git-bash + Windows, but I am guessing so at this point.
$ mkdir test
$ cd test
$ touch build.sbt
$ echo "sbt.version = 1.5.0" > project/sbt.properties
$ sbt
[info] welcome to sbt 1.5.0 (Amazon.com Inc. Java 1.8.0_282)
[info] loading settings for project global-plugins from global.sbt,idea.sbt ...
[info] loading global plugins from C:\Users\V8ZNX3A\.sbt\1.0\plugins
[info] loading settings for project isg-swagger-codegen-build from plugins.sbt,scalapb.sbt ...
[info] loading project definition from C:\bench\streaming-data\isg-swagger-codegen\project
[info] loading settings for project root from build.sbt,version.sbt ...
[info] set current project to isg-swagger-codegen (in build file:/C:/bench/streaming-data/isg-swagger-codegen/)
>....>....>....>....

Same behavior on a fleshed-out project.
problem
After changing build.properties to upgrade from 1.3.13 to 1.5.0, the SBT prompt is broken in git-bash on Windows.  It displays >... instead of the expected sbt:my-project>.  In addition, the enter key does not create a new line (see above).
Change the version to 1.3.13, no problem.
Change to 1.4, and you get a different problem, but that's a separate issue.
In general, whenever I upgrade SBT, the 'git-bash' experience on Windows changes significantly with regard to the terminal, keystrokes, text wrapping, etc.
expectation
Normal prompt displayed.
notes
This works fine in Powershell, but that's not an option for me, nor is switching operating systems.

Since most sbt maintainers and contributors don't run Windows regularly, to some extent these regressions are expected esp for things like git bash. The terminal in VS Code works on macOS, so maybe it's worth giving it a shot on Windows as well? Another alternative might be trying Windows Terminal (https://github.com/Microsoft/Terminal), which I think was supposed to have terminal emulation on par with those that come with Linux.

I'm experiencing the same issue.
sbt 1.6.2
git version 2.35.1.windows.2
GNU bash, version 4.4.23(2)-release (x86_64-pc-msys)
mintty 3.5.2 (x86_64-pc-msys)
I'm running Windows 10 with code page set to utf-8
I tried to investigate the cause by inspecting sbt.sh (sbt), but the file appears to be rather new in the repo :/
I experienced the problem using sbt 1.3.13 as well as sbt 1.2.8
Running  winpty sbt solves the issue of not seeing the input, instead with the lovely side-effect that all control characters are rendered:
?[0J?[0J?[2K?[1000D?[0J?[2K?[1000D?[0J?[0J?[2K?[1000D?[0J?[2K?[1000D?[0J?[?1h?=?[?2004hsbt:hello-world?[36m> ?[0
sbt runs fine using git-bash in the windows terminal
Basically, it seems to be the reverse of this coc.nvim issue
My guess is that it's a control character problem. It'd be great if control characters could be disabled.

I had the original prompt problem and then using winpty produced the same output as @CervEdin. I found a combination worked of using ansicon and using winpty:

Download and in a terminal:
unzip ansicon-(version).zip
In a terminal in the unzipped ansicon sub-directory for your chip architecture:
ansicon.exe -i
Open new terminal.
Open sbt with winpty:
winpty sbt
(or give the full path of sbt.bat if not on PATH)

The strange characters are no longer there and it works as it should.

No longer reproducible


Update timeout for .github/workflows/e2e_trigger_via_label.yaml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/e2e_trigger_via_label.yaml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

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

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

Closing in favour of already merged #5733


Revamp build system

Okay, so I revamped the build with rollup. This will result of shipping the @angular-redux/store with CommonJs, ES5 and ES6.
For some reason one of the test is failing. Not sure why cause I didn't change any logic of components! I tried to understand the logic of that unit-test, but there are too much stuff going on and I'm not sure what's what. So I ended up not being able to fix it.
I believe this PR will bring some values.
And by the way we should get rid of @angular/compiler, cause typescript-library-bundler has a dependency of that.
Would you take a look?

Thanks for the PR, taking a look into the testing issue.
@SethDavenport any ideas? noticed also for the select$ test - the second stub never excutes
fit('should reset stubs used by @select$', (done: Function) => {
    const instance = TestBed
      .createComponent(TestComponent)
      .debugElement
      .componentInstance;

    const stub1 = MockNgRedux.getSelectorStub('bar');
    stub1.next(1)
    stub1.next(2);
    stub1.complete();

    instance.barTimesTwo$
      .toArray()
      .subscribe((values: number[]) => expect(values).toEqual([2, 4]));

    MockNgRedux.reset();

    // Reset should result in a new stub getting created.
    const stub2 = MockNgRedux.getSelectorStub('bar');
    expect(stub1 === stub2).toBe(false)

    stub2.next(3);
    stub2.complete();

    instance.obs$
      .toArray()
      .subscribe((values: number[]) => {
        expect(values).toEqual([6])
        done(); // this never gets hit
      }
      );
  });


Add old docs admonition to 3.3 docs

Summary of the changes / Why this improves CrateDB
Add admonition to old docs to make readers aware that they are not looking at latest version of CrateDB reference documentation
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

After updating requirements.txt docs build successfully 🚀
Some pages are double-noted right now (e.g. https://crate--12167.org.readthedocs.build/en/12167/index.html or https://crate--12167.org.readthedocs.build/en/12167/sql/index.html) which does not look particular nice.
Something that should be addressed or is it ok?
Tests do not run. Not sure how to fix this or if it's necessary to do so.
Ps: Nice side effect: 3.3 docs now match new theme and colours ✨

Dear Johannes,
thanks for your efforts! We tried to generalize your patch and converged it into https://github.com/crate/crate-docs-theme/pull/322.
With kind regards,
Andreas.


Issue with protoc-gen-validate: Unknown Option "(validate.rules)" Error

Description:
I encountered a problem while using protoc-gen-validate.
When I execute the command protoc --go_out=. -I . hello_world.proto --validate_out="lang=go:."
it throws an error hello_world.proto:10:20: Option "(validate.rules)" unknown. Ensure that your proto definition file imports the proto which defines the option.
Directory Structure:
Copy code
├── proto
│   ├── hello_world.proto
│   └── validate.proto

Details:

The validate.proto file is copied from protoc-gen-validate.

The  hello_world.proto file is written by myself, with the following content:
protobufCopy code
syntax = "proto3";

package proto;

option go_package="./;proto";

import "validate.proto";

message Person {
    uint64 id = 1 [(validate.rules).uint64.gt = 999];
}

I have also successfully installed protoc-gen-validate in the $GOPATH/bin directory following the installation instructions.

Hello, thanks for your interest! As explained in the README, this project is in maintenance mode. We recommend upgrading to protovalidate.
I'm going to go ahead and close this issue, but feel free to open one against protovalidate if needed. Thanks!


Update docs and images

Update docs in usage.md and output.md
More extensive description of --design in usage.md
Regenerate and replace images in docs as relevant to ChIP-seq data
Change parameter names from boolean type to value type for consistency e.g. fingerprintBins > fingerprint_bins

What do you think about embedding the images in the docs instead of just linking to them? I don't like that you have to click through to each plot to see it. As a user I'd prefer to scroll down the docs to find the plot that looks like the one I'm trying to interpret... Or just for faster reading as someone interested in what the pipeline produces.

Yes, I agree. I did try that initially. However, I settled for this due to differences in the size of the images, and because it made it more difficult to follow the section headings and text. Willing to give it another shot though to see how it looks! Just needed some friendly criticism :+1:
Im trying to get this ready for a release in the next week or so. Almost there. Are you ok with the current code on master being wiped?

Go for it!

Thanks! Can you have a look at the docs now?
https://github.com/drpatelh/chipseq/blob/master/docs/output.md

Brilliant 🎉


[suggestion] Packaging mechanism

Can you give an option to "pack" our command blocks in a box ?
(I know, we can write some fill commands etc., but yeah, that would be nice)

I will add the possibility to wrap command block chains into a limited space. If that's what you meant. :)

Thank you ! :D

BTW very neat tool, I tested few others, your's the one that really fit devs needs !

Thank you. :) I've been working hard to make it as flexible and robust as possible.


[Docs]: update translation_guide.md file

what's wrong in the documentation?
We need to add some description about updating the available language list when translating the doc in a new language
Add screenshots
Add screenshots
Code of Conduct

[X] I agree to follow this project's Code of Conduct

assigned to you

Completed. Thanks for your contributions 🎉 ❤


Plot action to switch on/off OpenGL

Following #3255, this PR provides a plot action which can be used with a silx plot.
It have to be setup manually.
It only shows if OpenGL is enabled or not. I guess it is already complex for most people.
That's more or less the way i will use it in Flint.
It can be used with this example.
./bootstrap.py examples/plotInteractiveImageROI.py

Changelog:

Added an action to switch on/off OpenGL rendering on a plot

I have reworked the PR in order fix the text and the name.
I have also removed the disabled icon, and make it checkable. This allow to use it as it is with a menu.


modified newsletter

Issue that this pull request solves
Closes: #338 (DWoC)
Proposed changes
Fix Error message for Newsletter signup
Brief description of what is fixed or changed
Changed the error message for the Sign Up for our Newsletter input box.
Types of changes
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots
Please attach the screenshots of the changes made in case of change in user interface

Other information
Any other information that is important to this pull request

Please make changes for responsive design also.
You can make the message appear in center below the input box

@tejan-singh I am having some problem solving this issue. I will try to solve it and make the changes soon.

@tejan-singh @tripathyprateek I have made the changes. Please review.


Improve looks and resizing of Post Achievement metrics

Change Post Achievement User Counts bar chart to take full width on -md breakpoint
Add vertical margin between cards when each card takes 100% of the width
'Post Achievement Usage' chart should show a message when there is no data
Experiment with reducing length of titles and series names, ideas:

Wrap post achievement charts in a card with title "Post Achievement Metrics"
Change chart titles to just "User Counts" and "Usage"
Change series for "User Counts" to "Stopped After Achieving" and "Performed at least once after achieving" or maybe just "Continued using"?
Maybe also add a footer to the card that wraps these 2 charts, the footer can say something like: "** Post Achievement Metrics only account for users that have fully achieved this skill".

PR: https://github.com/NationalSecurityAgency/skills-service/pull/369

:+1:


pip, noarch, template

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Recipe maintainers should be a json list.
The recipe must have a build/number section.
The build section contained an unexpected subsection name. build is not a valid subsection name.

@conda-forge-admin please rerender

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Recipe maintainers should be a json list.
The recipe must have a build/number section.
The build section contained an unexpected subsection name. build is not a valid subsection name.


Visualize capture regions on the map

From https://mods.factorio.com/mod/TLBE/discussion/6498a5ed070072bbaa7d044c

map visualisation of capture regions - I know we can't draw rectangles onto the map, but perhaps we can use bookmarks or something to demarcate the rectangle being captured, and when panning, the target rectangle

I do not have any experience with the map API/possibilities
Drawing on the map would be nicest, but alternatively markers could be used to show the area(s). Although I have no idea how clear this would be and how much performance degradation it takes (updating only once in a while might counter performance issues)
I guess some experiment/test needs to be done, to make this request more clear and to see what is feasible

There is no API to render extra shapes onto the map. I have no idea the performance of adding and removing map markers, so as you say, this would need testing. It would be possible to use corner marker icons as the map icons though I think. It would mean shipping those icons with the mod.

I've made a start on this. I can put down a map marker at the centerPos position on the map, and it tracks about as you would expect. How do I calculate the bounding box from the centerPos and other numbers in the camera? I am guessing I need to use the zoom somehow? And possibly the width and height?

This is just a proof of concept. The box width and height are hard-coded, and I would need to replace the labels rendering the box with icons, but it basically works. As I walk about or build things, the box moves as expected.

Trackers have a size object, which is calculated into a zoom for the Camera (as factorio API requires zoom) here https://github.com/veger/TLBE/blob/master/scripts/camera.lua#L167
I guess if you reverse this you can calculate size again (where centerPos is the middle of the area defined by the size).
Alternatively, we could store size and calculate zoom on each screenshot that is taken (or both are kept), but that requires to modify the transition code, as it goes from one zoom factor to another.

Is this the code I should be getting inspiration from?
function Camera.zoom(camera, tracker)
    -- Calculate desired zoom
    local zoomX = camera.width / (tileSize * tracker.size.x)
    local zoomY = camera.height / (tileSize * tracker.size.y)
    return math.min(zoomX, zoomY, maxZoom)
end

If I'm following this, the values have the following units:
tileSize: pixels per tile
tracker.size.{x,y}: tile
camera.{width,height}: pixel
zoomX,zoomY: dimensionless
So I think to calculate my boundary in tiles I use:
local width = zoom / (tileSize * camera.width)
local height = zoom / (tileSize * camera.height)

The logic in the zoom method seems to clip the captured region. So if the region was very long and thin, it would cap the zoom to capture the height, and if it was very tall and narrow, it would cap the zoom to capture the width. Is this the intended behaviour? It may explain why I was having difficulty with some captures where I thought it would be maximal over the bookmarks I set for corners. If we are reworking the rectangles and zoom logic, perhaps this return value could become:
    return math.min(
      math.max(zoomX, zoomY),
      maxZoom)

But perhaps I have misunderstood the zoom and got it inverted in my head. Bigger zooms show fewer tiles, right?

See #48

I'll be merging https://github.com/veger/TLBE/pull/46 soon, which adds the transitionData object in the camera. This object has the start and end positions/zoom factors. So I guess it makes sense to use that info.
It also adds more info on the fields (in an attempt to start documenting the code)

The logic in the zoom method seems to clip the captured region

More to extend the captured region, I guess. So it will contain at the least the intended area of the tracker. With a perfect ratio zoomX and zoomY are equal. Otherwise the smallest zoom factor is picked (smaller zoom is bigger area screenshotted) making sure that the screenshots indeed contain the tracker area.

Yes. I learned how to make an icon today. Tomorrow I will learn how to tint
them.
On Tue, 27 Jun 2023, 19:02 Maarten Bezemer, @.***>
wrote:

I believe it is possible to change the tint of sprints? No idea whether
this is possible for map tags... If so it would be nice to have different
colors for the different areas on the map?
—
Reply to this email directly, view it on GitHub
https://github.com/veger/TLBE/issues/39#issuecomment-1609984205, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEOOXCKYNXD3KZTLVK5YWTXNMN4NANCNFSM6AAAAAAZTXMYSQ
.
You are receiving this because you commented.Message ID:
@.***>

Tints implemented for 'corners' icons.

note to self - the current code crashes if used with an old save with an old mod, as the camera doesn't have the new field for the tags - it just needs a nil check added, not an upgrade script, but if I don't document the issue I will forget it

Note that there is already a migration script setup for the 'paused camera issue' fix and enable/disable the 'take screenshot' button: https://github.com/veger/TLBE/blob/master/migrations/tlbe.1.4.5.lua
It will be run for existing games when the current changes are released

OK, I'll hook into that then as the code will be less scruffy that way.

I'll take a look at your PR in the evening (in a few hours)

It will be available in the next release

I've used this for a few hours now, and imho it is a really useful feature.

Agreed! When I read your request, I was wondering why I didn't think of it before :smile:
It sounded (and is as we now see) a very valuable addition

I just stumbled upon this mod: https://mods.factorio.com/mod/eradicators-screenshot-maker
It might be nice to show the camera/transition areas on screen/in-game? It would show that you are leaving the area with the player character or something? :thinking:
Maybe enable alt-mode drawing, so the rectangles won't end up in the screenshots, or we hide them before taking a screenshot? :thinking:

I just stumbled upon this mod: https://mods.factorio.com/mod/eradicators-screenshot-maker

I think this uses the LuaRendering API, which I discovered today via this mod :D I think it allows rendering into the game window but not onto the map view. Seeking clarification from discord.
We could use these APIs to render in-game rectangles for the cameras in-world. But as you say, we'd then have to juggle if these are visible in screenshot somehow.

Rendering to map may possibly be added in future. Mark as "when WUBE update their APIs" and park?

Rendering to map would be nice, to the game/screen I have my strong doubts.
parking until API supports such a functionality seems a good solution

Capture regions with map markers is available in release v1.5.0


Remove failing tests from test runs temporarily to get builds running: skip unittest.test.ts

Workaround for #2684 until we get to the root cause.

unittest.test.ts is failing with the ECONNRESET somewhere in the  method.

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)

[x] Title summarizes what is changing

[x] ~Has a news entry file (remember to thank yourself!)~

[x] ~Has unit tests & system/integration tests~

[x] ~Any new/changed dependencies in package.json are pinned (e.g. "1.2.3", not "^1.2.3" for the specified version)~

[x] ~package-lock.json has been regenerated by running npm install (if dependencies have changed)~

Please create a specific issue for the test that has been disabled.
Issue #2684 is too generic and that cannot be used to track the test that has been disabled.

#2692 to track the re-enabling of this test suite.


Add CI/CD to auto-deploy do Github Pages

Some info to get started: https://blog.logrocket.com/deploying-react-apps-github-pages/
A nice bonus would be to set up a domain and connect it for a nicer experience.

We're live at https://rozlicz-estateguru.pl/ 🎉


Shadow acne artifacts in highlight engine demo

Description
I'm seeing a lot of shadow acne artifacts on the highlight demo.  Can't recall what it looked like before (and therefore if theres a regression) but at the least it should probably be cleaned up.  Surely a simple scenario like this should work by default?

mac book pro m1
chrome

Thanks so much for reporting! We'll investigate. (My guess would be this is related to clustered lighting).

no worries unfortunately I can't recall what it looked like before but with the new default lighting switch, thats set off some alarm bells ringing

I reproduced this on PROD. Adding app.scene.clusteredLightingEnabled = false; fixes the shadows.

The clustered lighting shadows just appear lower precision than before. More shadowBias is required to remove acne on the example scene. I can't see or think of any obvious reason for this. Could be the difference of depth texture format (RGBA8 vs F32) or perhaps an additional rounding off error in clustered atlas depth pack/unpack.
Either way this will require @mvaligursky to investigate so handing over to him.

Should we log another issue for the underlying change?  I noticed this In my app too and have avoided shadows for this reason.  switching back to old lighting has none of those artifacts

I added a comment here: https://github.com/playcanvas/engine/issues/2780


Bot automatically offline

Authorized the bot on our server (I am the administrator, so I have priviliges for it), gave it all the rights it needed (actually, every right), but it goes offline instantly, and i can't summon it. Tried kicking even, and re-authorizing, but I didn't solve it.

Hmm I have the same problem.
Did you fix it by any chance so I could get it running myself?

Since I found another bot for this (Sidekick), I had to research some stuff about bots (also the maker say it is unsupported now), so it means it's always offline=not hosted :(


[Search v2.4] Use text style for router input

Problem
Coming from here, we should use text color for the search router input text.
Solution
Update the text color
Issue OwnerCurrent Issue Owner: @greg-schroeder

there are no contributors here, so this can be closed as soon as it's merged and deployed to prod

Closing per this as this is finished up!

Reopening so that @luacmartins and @greg-schroeder can finish the regression checklist

Done


Enhancement: do not allow Terraform version < 0.14.x

Summary
Terraform v0.14 released the sensitive input variable feature, which prevents variable values from being output in the plan.
Since this action outputs plan results, we should not allow it to run if the Terraform version is < 0.14 to prevent accidental secret leakage in the PR comment.
⚠ Note
To avoid breaking existing use of the action, this will result in a new major version of the action.

Dupe of #65


WSL2 - Windows 19041.264 - WslRegisterDistribution fails with error 0x80070057 - Incorrect parameter

Microsoft Windows [version 10.0.19041.264]

What you're doing and what's happening:
I updated to this version of Windows.
I already have docker on this machine, that detected WSL2 possibility.
I followed https://docs.microsoft.com/fr-fr/windows/wsl/install-win10 to the letter, with an apparent success.

What's wrong / what should be happening instead:
When installing Debian, then ubuntu I get the same result at the initial lauch

Also docker is having a similar issue when trying to register its distribution :
[Engines ][Error] Start failed with Failed to deploy distro docker-desktop to [...]\AppData\Local\Docker\wsl\distro: exit code: -1
 stdout: Paramètre incorrect.

i tried to uninstall WSL et install it again, I tried to reset, delete and reinstall images, reinstall docker, etc. Always stops here. Also not an issue of legacy console. I tried to stop, start, restart, and tried in between LxssManager.
Thank you for your attention !

Please, I really need some help people, i re-installed everything (wsl, wsl2, windows components, etc..)
Same result :
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070057
Error: 0x80070057 ParamÞtre incorrect.

@devsuitup - Could you please take a trace so I can figure out where the error is coming from?https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md#8-detailed-logs

Hi @benhillis , thank you for your attention.
You will find the logs attached to this post, in a zip file -- .etl isn't supported by github
lxcore_logs.zip
Thanks !

@devsuitup - Thanks for the logs.  It looks like the failure is coming from trying to create our virtual network.  Could you please also collect the networking logs detailed here:
https://github.com/microsoft/WSL/blob/master/CONTRIBUTING.md#9-networking-logs

@benhillis FYI, after your answer I uninstalled NordVPN, Hamachi, FortiClient and restarted the machine -- and got the same issue.
Here is the trace : https://aka.ms/AA8lxay

I have similar issue:
When trying to --set-version of existing Ubuntu I got:
wsl --set-version Ubuntu 2
Conversion in progress, this may take a few minutes...
For information on key differences with WSL 2 please visit https://aka.ms/wsl2
The parameter is incorrect.
When trying to install new one:
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070057
Error: 0x80070057 The parameter is incorrect.
Press any key to continue...

@benhillis Any news ?

I solved a similar problem for myself (#5660) by uninstalling HyperV under "Turn Windows features on and off". Does this happen to work for you, @devsuitup , as well?

I had this problem after installing Ubuntu or Ubuntu 20.04 either from the Store or PowerShell, the latter still complained that no distributions are installed. I had HyperV already disabled and LxssManager was running. What finally fixed it was just restarting the computer and Ubuntu 20.04 ran fine then.

Amusingly, this only was fixed for me after I turned hyper-v on. I figured it was somehow essential to the operation of WSL2, so it's funny hearing that turning it off worked.

I created a new Virtual Switch in Hyper-V Manager.
Then it works well.
0x80070057 error did not occurred.

Same issue. None of the methods above work :(
I have tried the following:
Turn on and off Hyper-V
Enable and disable WSL
Install and uninstall Ubuntu Package

Windows version 20H2

This is still not resolved by the latest update for 20H2.
I am wondering that not body encounters this issue but we?
Seems only few discussion about the 0x80070057.

I finally find a solution. Change back to WSL 1 works for me.

Happens to me as well with latest build 21364

system info

PS C:\WINDOWS\system32> wsl --status;cmd /c ver
Default Version: 2

Windows Subsystem for Linux was last updated on 22/04/2021
The Windows Subsystem for Linux kernel can be manually updated with 'wsl --update', but automatic updates cannot occur due to your system settings.
To receive automatic kernel updates, please enable the Windows Update setting: 'Receive updates for other Microsoft products when you update Windows'.
For more information please visit https://aka.ms/wsl2kernel.

Kernel version: 5.10.16

Microsoft Windows [Version 10.0.21364.1]

networking

PS C:\WINDOWS\system32> netsh interface show interface

Admin State    State          Type             Interface Name
-------------------------------------------------------------------------
Enabled        Connected      Dedicated        Wi-Fi
Enabled        Disconnected   Dedicated        Local Area Connection
Enabled        Disconnected   Dedicated        Ethernet

networking on WSL2 machine with beta channel and WSL2 working fine

PS C:\WINDOWS\system32> netsh interface show interface

Admin State    State          Type             Interface Name
-------------------------------------------------------------------------
Enabled        Disconnected   Dedicated        Ethernet
Enabled        Connected      Dedicated        Local Area Connection
Enabled        Connected      Dedicated        Wi-Fi
Enabled        Connected      Dedicated        vEthernet (Default Switch)
Enabled        Connected      Dedicated        vEthernet (Ethernet)
Enabled        Connected      Dedicated        vEthernet (WSL)
Enabled        Connected      Dedicated        vEthernet (Wi-Fi)

virtual platform/hyper-v is installed (as you see wsl --status do not complain)

I got the insider update to 21364 and was looking to try out the new GUI feature for WSl, but both my Debian and Ubuntu installations would not start with the message "The parameter is incorrect".
Tied uninstalled both and reinstalling but both still report:
WslRegisterDistibution: failed with error: 0x80070057
Error: 0x80070057 The parameter is incorrect.
At least I'm not the only one...

I got the insider update to 21364 and was looking to try out the new GUI feature for WSl, but both my Debian and Ubuntu installations would not start with the message "The parameter is incorrect".
Tied uninstalled both and reinstalling but both still report:
WslRegisterDistibution: failed with error: 0x80070057
Error: 0x80070057 The parameter is incorrect.
At least I'm not the only one...

That was precisely my reason to give WSL another try after a couple of years. Ended up in the same boat as above. Tried everything as above. Now I can't even go back to my standard Windows. Seems like a bait to attract people to debug their uncooked sh*t.
Thanks Satya.

I got the insider update to 21364 and was looking to try out the new GUI feature for WSl, but both my Debian and Ubuntu installations would not start with the message "The parameter is incorrect".
Tied uninstalled both and reinstalling but both still report:
WslRegisterDistibution: failed with error: 0x80070057
Error: 0x80070057 The parameter is incorrect.
At least I'm not the only one...

(replacing my last comment)
Success!: Here is what worked for me:
On an Administrative command line:
> wsl --set-default-version 1
Then install the distribution of your choice (e.g. > wsl --install -d kali-linux)
Error is gone, and distribution is installed successfully (takes a while)

But the error returns if I switch to wsl version 2

I am getting this behavior, "Incorrect parameter" and the shell not created with insider build 21370 for Linux 18.04.  Same behavior for WSL1 or WSL2.
From the Event Log in PerfView it looks like there's a failure creating a virtual machine due to an incorrect parameter:
ThreadID="15,496" FormattedMessage="onecore\vm\compute\shared\vmwp\lib\workerprocess.cpp(1315)\vmcompute.exe!00007FF66FD2BDF3: (caller: 00007FF66FC4E45C) Exception(50) tid(3c88) 80070057 The parameter is incorrect. CallContext:[\VirtualMachine_Start] " TraceData="onecore\vm\compute\shared\vmwp\lib\workerprocess.cpp(1315)\vmcompute.exe!00007FF66FD2BDF3: (caller: 00007FF66FC4E45C) Exception(50) tid(3c88) 80070057 The parameter is incorrect. CallContext:[\VirtualMachine_Start] " VmName=" - " VmId=" - " StackFrame="<<<EXCEPTION_DURING_VALUE_LOOKUP InvalidCastException>>>" Module="[{ Name="C:\WINDOWS\system32\vmcompute.exe", BaseAddress="140696412815360", SizeOfImage=4091904, TimeDateStamp=1438749299 },{ Name="C:\WINDOWS\SYSTEM32\ntdll.dll", BaseAddress="140703339642880", SizeOfImage=2109440, TimeDateStamp=1581716554 },{ Name="C:\WINDOWS\System32\KERNEL32.DLL", BaseAddress="140703317295104", SizeOfImage=774144, TimeDateStamp=-257251206 }]"
I was unable to attach wsl.etl.
Default Distribution: Ubuntu-18.04
Default Version: 2
Windows Subsystem for Linux was last updated on 5/4/2021
WSL automatic updates are on.
Kernel version: 5.10.16
Microsoft Windows [Version 10.0.21370.1]

I fixed this.  I was missing a .wslconfig file in my user directory.

WSL2 distro not starting since 21364 build #6839
Apparently solves this problem as well.

This worked in my setup

On an Administrative command line:

wsl --set-default-version 1
then install the Linux.
Window needs to update the wsl documentation.

I created a new Virtual Switch in Hyper-V Manager. Then it works well. 0x80070057 error did not occurred.
I was able to replicate this. Creating a new switch worked! I shared it on my site:
https://www.tidbitsfortechs.com/2022/04/wsl2-wont-start-0x80070057-the-parameter-is-incorrect/

Removing the existing install with wsl --unregister ubuntu-18.04 and installing again worked for me.

Anyone having this issue might also want to try cleaning existing ".wslconfig" file which could cause the problem.

https://stackoverflow.com/questions/64191759/wslregisterdistribution-failed-with-error-0xffffffff#comment121617854_64217462

I had that issue on Ubuntu and Ubuntu 20.04 after installing the WSL2-Linux-Kernel .  I don't know why it broke my WSL2 installation, but I realized that it was the kernel setting in the ./.wslconfig file that was creating this bug...
Skipping this setting with "#" in front of the line resolve the issue for me...

I have the same problem, tried a couple different options but no luck.

I was able to install the distro by rolling back to wsl 1. I'm not able to update to wsl2 later, it results in the same error. This happens at least on both Ubuntu, Ubuntu-22.04 and Debian, I did not try others.
There is no .wslconfig file in my %UserProfile% directory
wsl does not list (prior to the wsl1 installation) any distros installed, it's a clean setup
Going a little deeper in the point above, I've tried deleting everything. WSL, the distros and I have disabled the "Virtual Machine Platform" and "Windows Subsystem for Linux" features. Reinstalling WSL reenabled the first one automatically, the second one only comes back after I manually enable it. The error happens regardless if it's on or off after this procedure. I even tried cleaning the microsoft store cache.
This is all after I've reset my windows installation (keeping my old files).
I've tried disabling and enabling network adapters based on https://github.com/microsoft/WSL/issues/8216#issuecomment-1322993273 , since I have both ethernet and wifi. It also didn't work.

the latest update of Windows is suck, tried to rollback then it works now.

^This

Indeed, it appears the windows update Windows 11 Insider Preview 25905.1000 (rs_prerelease) was the problem. Today I installed  Windows 11 Insider Preview 25915.1000 (rs_prerelease) and the issue is gone now.


MicroK8s addon

Add an official MicroK8s addon distribution: https://microk8s.io/docs/addons

Addon enable/disable handlers in microk8s are implemented here.

Blocked by https://github.com/Kong/team-k8s/issues/55

cc @gAmUssA just wanted to point this one out to you for your awareness since you had previously worked on the kong minikube addon.

Addons Hacking Guide


[halide] use supports expression

2547/4262 halide[core,target-opengl-compute]:x64-windows-static
Installing 1/1 halide:x64-windows-static...
Building halide[core,target-opengl-compute,target-x86]:x64-windows-static...
-- Note: halide only supports dynamic library linkage. Building dynamic library.
CMake Error at scripts/cmake/vcpkg_check_linkage.cmake:25 (message):
  Refusing to build unexpected dynamic library against the static CRT.

      If this is desired, please configure your triplet to directly request this configuration.
Call Stack (most recent call first):
  ports/halide/portfile.cmake:1 (vcpkg_check_linkage)
  scripts/ports.cmake:170 (include)

As discussed previously using supports here is wrong.

Closing this PR since it seems that no progress is being made. Please ping us to reopen if work is still being done.


[BEAM-8616] Make hadoop-client a provided dependency on ParquetIO

See the JIRA issue for the motivation on this.
R: @lgajowy

Run Java PreCommit

Run Java ParquetIO Performance Test

Run Java PreCommit

Run Java ParquetIO Performance Test

Just run the ParquetIO test to be 100% sure but in general LGTM. Thanks!

Revert due to failing sql postCommit tests: https://github.com/apache/beam/pull/10087


Unify get_pos() / get_translation()

I gave a try to 3D in Godot and I got confused by the naming convention.
In 2D, you use get_pos() and get_rot(), but in 3D it's get_translation() and get_rotation().
This got me to write errors in my code everytime because of the different names.
I think the names of these functions should be unified for clarity, because they do the same thing in space.

Well, in 3D it is better to use Transform class. The concepts are too different in use to make them look similar.

I know transforms are better in 3D, but it's more about the naming, for transforms or shortcuts.

this was done already, closing


Need a better error message for plot color nodes if a node is not valid

The get_clusters method can return "-1" values to indicate points that are "noise" for a given cluster labeling scheme. If this "-1" label is passed directly to the color_nodes parameter it's a problem because this isn't a valid tree node ID, but the error message isn't very helpful.
>>> cluster_labels = tree.get_clusters(fill_background=True)
>>> labels = np.unique(cluster_labels[:, 1])
>>> fig = tree.plot(color_nodes=labels)[0]

KeyError                                  Traceback (most recent call last)
<ipython-input-27-8cae5ab76c06> in <module>()
----> 1 fig = tree.plot(color_nodes=labels)[0]

/home/brian/projects/DeBaCl/debacl/level_set_tree.pyc in plot(self, form, horizontal_spacing, color_nodes, colormap)
    313
    314         for i, ix in enumerate(color_nodes):
--> 315             subtree = self._make_subtree(ix)
    316
    317             for ix_sub in subtree.nodes.keys():

/home/brian/projects/DeBaCl/debacl/level_set_tree.pyc in _make_subtree(self, ix)
    519
    520         T = LevelSetTree()
--> 521         T.nodes[ix] = _copy.deepcopy(self.nodes[ix])
    522         T.nodes[ix].parent = None
    523         queue = self.nodes[ix].children[:]

KeyError: -1

Fixed by pull request #53 - closing.


"Determine Cancelled" no longer working on "player exits notable cuboid" event

The output of /version on my server is:

[02:21:14 INFO]: This server is running Paper version git-Paper-99 (MC: 1.15.2) (Implementing API version 1.15.2-R0.1-SNAPSHOT)
[02:21:14 INFO]: Checking version, please wait...
[02:21:15 INFO]: Previous version: git-Paper-88 (MC: 1.15.2)
[02:21:15 INFO]: You are running the latest version

The output of /version denizen on my server is:

[02:21:42 INFO]: Denizen version 1.1.3-SNAPSHOT (build 4590-DEV)
[02:21:42 INFO]: Author: The DenizenScript Team

The script in question:

https://one.denizenscript.com/haste/65148

The issue is:
Since updating to a newer Denizen dev build, a script that previously worked no longer works. The line - determine passively cancelled passes but does not stop the player from moving as it used to. The result in the above script is that the next if command executes, returns true, and runs the narrate and flag commands, but I am allowed to keep running. The intended behavior is that the walk action would be cancelled and the player is unable to move any farther.

User error.
Please read https://one.denizenscript.com/denizen/lngs/safety in events


Please add a way to format sources without semicolons

Semicolons are excess in JavaScipt sources

I agree that semicolons should be optional. Your tool is very nice, but currently we can't use it cos our style guides forbid semicolons.
Your tool adds to much stuff, and it is hard to diff commits, after your tool ads semicolons everywhere.

+1
Optional semicolons are a must for me. Your tool formats code really nicely otherwise.

This issue is a duplicate of #12. We won't be actively working on this feature, but if someone came up with a reasonable prototype/solution, we might work on adding this into Prettier.

Yes, please see the resolution at #12. You can freely code without semicolons but we will add them for consistently. It's not a simple matter of removing them because there are cases where we need to specially detect when to add them and it makes formatting harder.
This is an opinion that for now we are not going to change, much like standard will never have the option to add semicolons. Really, in the end, it doesn't matter at all and the fact that prettier will add them for you removes any annoyances you have in writing them.

for me it's not as much about writing the semicolons as reading them, but i'm one of those guys that liked coffeescript too, so take that fwiw ;)


UI polish 2

Codecov Report

Merging #47 (5e696a3) into main (276e926) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #47   +/-   ##
=======================================
  Coverage   92.47%   92.47%
=======================================
  Files          34       34
  Lines         824      824
=======================================
  Hits          762      762
  Misses         62       62

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 276e926...5e696a3. Read the comment docs.


revert!: don't rely on environment for interopDefault

Most of the users don't actually test Node.js applications, so it becomes cumbersome to setup Vitest.
I think we might return to this idea in the future, but with more checks. We should also probably change module resolution to be actually compatible with Node. Until then this is just a distraction.
Closes #2610

Not 100% sure about the context, but it seems to make sense.

The initial idea was to follow Node.js spec for non-Node environments. But we forgot about auto interop in TS files. Since this affects most people, I decided to rollback this change, until we can reliably disable it.


qBittorrent login failed with code 405

When I set the qbittorrent Url with the correct username and password, logs on the Unraid container shows:
error: qBittorrent login failed: request to http://<IP_ADDRESS>:8080/api/v2/auth/login?username=admin&password=[REDACTED] failed, reason: connect EHOSTUNREACH <IP_ADDRESS>:8080
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[1]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[2]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[3]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[4]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[5]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[6]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[7]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[8]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
warn: Failed to reach http://<IP_ADDRESS>:9117/api/v2.0/indexers/[9]/results/torznab/api?apikey=[REDACTED]&t=search&cat=&q=
Configuration file already exists.
info: Validating your configuration...
info: Your configuration is valid!
info: [server] Server is running on port 2468, ^C to stop.
Configuration file already exists.
info: Validating your configuration...
error: qBittorrent login failed with code 405
Configuration file already exists.
info: Validating your configuration...

According to this link from qbittorrent, the 405 error is from the wrong request method. Is this something with the API call or something I need to fix on my end?

Is your cross-seed version up to date?

Apparently not. I was using a version that didn't support "mmgoodnow/cross-seed" but instead "crossseed/cross-seed". Thanks.


Next steps

Hi folks!
As discussed in https://github.com/upringjs/upring/pull/27 there are some things that we want to change, since all of those are breaking changes, let's discuss here how we plan to perform them and define how we will update the Upring architecture.
Points of discussion:

Move from actual tentacoli based requests to node core http2 implementation

How we will support fire and forget request?

Change the current routing model to a more user friendly one.

find-my-way could be an option but we should use the compatibility api. We can refactor it as upring-router in a way that supports just the http2 specifc api.
if we choose to use find-my-way we can even think about an encapsulation model like the one built in Fastify

Introduce a tracing api
Change the shape of the request object

Handle requests id
Add logging support

Feel free to update the above list with other points that we should discuss.

I am using the stack of find-my-way + http2 instead of fastify for a backend file manager.
I may help on this if the project is till going.
Do you have some maintainers or time you will consume on this project on the future ?

I don't think so :(.

Oh, sad, anyway thanks for the reply ... i will find another way for solving my architecture problem ^^"


Update Translation of Async for Resource Leak/Taint

Async methods cause the compiler to spin up state machine classes around these methods; with new fields and methods. Particularly notably:

Parameters of these async methods get stored in class fields in an initializer method
The source code of this method gets put into a "MoveNext()" method within the state machine, along with some extra code.
It seems that (at least sometimes) finally blocks are executed only if the value of a __state field is >=0. This __state field is supposed to be initialized to -1 by the initializer method. This was the root cause for observing false positives on iDisposable objects allocated via using within async methods.
The return value of the method is instead set via a SetResult() method, as the MoveNext() method is actually void.

All of these technicalities are addressed in this PR, along with a QoL fix to a frequently encountered issue to do with wrong line numbers being reported, particularly for resource leaks. The changes are:
1/3 : We mock the initializer behavior: we inline Store instructions within the MoveNext() method which initialize the fields storing the parameters to the parameter values and initializes the __state field to -1, so that Infer can correctly understand that the fields store the parameter values and that the __state field is negative.
2. We take the translation of the MoveNext() method and put the signature of the original method instead. This trick lets Infer apply the spec for the MoveNext() method containing the original source code whenever that method gets invoked (async method invocations look identical to how they would be if those methods were synchronous). We have to change the signature and change all ProgramVariables associated with all parameters and all return variables to be associated with the swapped method to ensure that Infer can track information flow properly.
4. When we see a call to SetResult in a MoveNext() method, we instead treat this equivalently to a return instruction.
These changes generally enable Infer to treat the asynchronous methods almost identically to synchronous methods.
Additionally, the start node of a method is now mapped to the first real line of code, and the end node of a method is now mapped to the last real line of code. This should fix most of the issues we had encountered with line numbers.
Tested:
Added to the Examples.cs file some taint examples, focused on async methods, to validate the above changes.
A false positive that had existed in the past with an async method returning an iDisposable method is now no longer reported, as desired.

Latest Infer has skip-translation deprecated. I think we should update this line by replacing skip-translation as capture-block-list and removing language.
Please take a look at this example.

Latest Infer has skip-translation deprecated. I think we should update this line by replacing skip-translation as capture-block-list and removing language.
Please take a look at this example.

I removed the skip-translation. capture-block-list references Java/Clang specifically so not sure about compatibility with .NET. It looks nonessential anyway, so will delete for now.


Accept interface instead of struct

Problem:
Is it possible to accept interface instead of S3 client struct for initialization?
https://github.com/aws/amazon-s3-encryption-client-go/blob/8a052f61c5940cc52018c82f94e8173822c27672/v3/client/s3_encryption_client_v3.go#L45
Solution:
Accept interface that implements only necessary S3 client API for this library/operation.
Out of scope:
None

Hey @hnnazm,
We can take this as a feature request and add it to our backlog. Can you tell us more about your usecase so that we can prioritize it accordingly? Thanks!


Added zio-json instances for http4s

closes #240

Looks good to me 😄

Thank you for working on this, white wizard.


"annotation information" is slightly confusing

Normally an annotation is considered to be something done manually, so I was a bit confused when I saw the default message saying that the bot analyzed "the annotation information on this pull request".
I agree with the change away from "blame" in #142, and the motivation to defer to git terminology, but I'd offer that the annotations on git annotate are actually generated on-the-fly when the command is called, out of metadata present in the repo, rather than annotations that are themselves part of the commits (and even less so the PR, since the important thing is the history of the files prior to the PR, not the info in the commits of the PR itself).
How about By analyzing the history of the files in this PR? /cc @ZauberNerd @vjeux

@waldyrious thanks for the suggestion, I'm down for it. Can you send a pull request to update it? Thanks!

Sure, that was my intention. I'll just wait a bit to see what @ZauberNerd's thinks :)

@walyrious sure go ahead. That does indeed sound nicer than annotation information ;)

Glad you guys agree. I'll submit the PR then.


Vertical postion randomly changes when some settings modified in the Settings Editor, Sometimes gone from my sight

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.88.0
OS Version: Windows 22H2(22621.3296)

Steps to Reproduce:

Open Settings Editor window.
Change some settings like check or uncheck checkbox.
This setting item leaves it's original position in vertical.

https://github.com/microsoft/vscode/assets/24841978/739dd087-61f0-47be-8dd2-9282ec9c369e

Also happened when I switch to other tabs and return back.

This is so inconvenient when I change a lot of settings at once.

I find it inconvenient as well, but haven't had the chance to look more into it yet.
Closing in favour of https://github.com/microsoft/vscode/issues/204207


feat: add Raku programming language

Apparently, for this to work right, it has to be done in three places: here, mason.nvim, and lsp-config.nvim ... almost there...

@williambotman /fixup

three places: here, mason.nvim, and lsp-config.nvim

https://www.youtube.com/watch?v=TwYczl2yOh8


Cinder v2 returns capitalized X-Openstack-Request-Id header

This is kind of a mirror of LP#1828765.
'-d x-openstack-request-id' should actually be '-d X-Openstack-Request-Id'
$ /usr/lib/nagios/plugins/check_http -H REDACTED -p 8776 -u /v2 -e Unauthorized -d x-openstack-request-id -S -v
GET /v2 HTTP/1.1
User-Agent: check_http/v2.1.2 (monitoring-plugins 2.1.2)
Connection: close
Host: REDACTED:8776
https://REDACTED:8776/v2 is 343 characters
STATUS: HTTP/1.1 401 Unauthorized
**** HEADER ****
Date: Tue, 30 Jul 2019 13:03:20 GMT
Server: Apache/2.4.18 (Ubuntu)
Content-Type: text/html; charset=UTF-8
Content-Length: 23
Www-Authenticate: Keystone uri='https://REDACTED:5000'
X-Openstack-Request-Id: req-81208248-95f3-4c5d-acbe-4b5d78851699
Connection: close
**** CONTENT ****
Authentication required
Status line output matched "Unauthorized" -
HTTP CRITICAL: Status line output matched "Unauthorized" - header 'x-openstack-request-id' not found on 'https://REDACTED:8776/v2' - 343 bytes in 0.013 second response time |time=0.013277s;;;0.000000 size=343B;;;0

Imported from Launchpad using lp2gh.

date created: 2019-07-30T13:08:19Z

owner: aieri

assignee: None

the launchpad url

(by aieri)
Nevermind, in some clouds it's capitalized, in others it is not. I think at this point check_http should be extended to allow for case-insensitive matching.

(by eric-chen)
This charm is no longer being actively maintained. Please consider using the new Canonical Observability Stack instead.
(https://charmhub.io/topics/canonical-observability-stack)
I will close this feature request


Fixing Acadiana

Hello,
Acadiana is currently located in the incorrect place.
862CCVWP+GX
(30.4463018,-91.1126054)
This brings you to an area located in the middle of Baton Rouge, when this is far from where this needs to be.
Acadiana is a 22 Parish region of Louisiana, and Lafayette, Louisiana is considered the heart of it.
https://en.wikipedia.org/wiki/Acadiana
https://en.m.wikipedia.org/wiki/Flag_of_Acadiana
The Acadian Flag was even designed by a professor at what is now the University of Louisiana at Lafayette.
Please. Move this code to the correct latitude and longitude.
(30.2427769,-92.0127186)
As well as defining this as an area. I’ve attempted to create a fusion table to fix this issue.
https://fusiontables.google.com/DataSource?docid=1YbFImgZm9aFY3ou0Uk-KznVPK-xyRORke4rDFFK8#rows:id=1
If anything, please help correct that it is not Acadiana, Baton Rouge.
If anything, it is Acadiana, Louisiana or Acadiana, Lafayette, Louisiana.
Thank you.

This seems to be similar to an issue previously discussed on the mailing list: https://groups.google.com/forum/#!topic/open-location-code/40rTruLnAX8
Plus codes are not "assigned" to a locality, but are just a different representation of the already existing lat/long coordinates that Google Maps has for some location. If Google Maps returns the wrong coordinates when entering the search term "Acadiana", then that is a Maps issue that should be brought up via feedback from the Maps app (or, if you have, some other way of getting into contact with people responsible for the map data itself).

Hi @LafayetteTravel,
As @bocops mentioned, this is an issue with what Google Maps is returning to us, which we unfortunately have no control over. From the thread on the mailing list, it sounds like you already have a contact within Google. If not, you can either submit the feedback via Google Maps or let me know and I can file a bug internally with the appropriate team.
I'm going to mark this issue as closed since it's not caused by anything on our end and we can't do anything directly about it.


How do I implement passkey with webauthn

I tried using this package, but it just shows the biometric(face id or fingerprint), I want to implement passkey in my application, Any way I can do it with webauthn package

Please take a look at the example app for how to integrate the authenticator in your app: https://github.com/flutter-institute/webauthn/blob/master/example/lib/main.dart
This is designed to work as the authenticator portion of Webauthn, so it's expecting to receive information from a Relying Part (your back-end app). That can then be passed through the WebApi helper methods to interact with the authenticator.

Hi!
How can I obtain the public key when creating the credential? I have tried the following code:
var attestationBytes = attestation.asCBOR();
var attestationJson = attestation.asJSON();
However, I only received authData without the key.

@majdslmt For the currently used key type (ES256), the public key bytes are the last 77 bytes of the attestation's authData.
So you can retrieve it by using attestation.authData.sublist(attestation.authData.length - 77)
See https://github.com/flutter-institute/webauthn/blob/master/lib/src/models/attestation.dart#L32 where the structure of authData is documented.
However, as the authenticator you shouldn't be worrying about extracting the public key from the attestation. That's something the Relying Party will do when they validate the response. What's your use case here?

I want to save the public key in the backend, but I can't extract the public key from authData. Also, when using this code, attestation.authData.sublist(attestation.authData.length - 77) didn't work to get the public key as string data.

@majdslmt that's because authData is a Uint8List, so the sublist is the list of bytes that make up the public key data. If you want to save that data as a string, you'll need to do some encoding on it. Which format are you trying to store it in?

To be much more verbose.
The public key that is stored in the authData is encoded in COSE_Key format per https://www.w3.org/TR/webauthn-2/#sctn-attested-credential-data and https://datatracker.ietf.org/doc/html/rfc8152#section-7
The byte data you get from the authData is that encoded format, so you can parse it an re-encode it however you want. I had a script that encoded this into an RSA style format. I'll see if I can find it today.

I want to get the response as this payload to be able to use @simplewebauthn/server in server side
{"id":"ASBmohREs1OhTB6s_EJY-hIwyiQuGbklL-mXK-tmVnmR5N4rlg_TRlrOf-olrhRQlasoGZOabDM4oppcnj0fn0Y","user":"+46723115353","rawId":"ASBmohREs1OhTB6s_EJY-hIwyiQuGbklL-mXK-tmVnmR5N4rlg_TRlrOf-olrhRQlasoGZOabDM4oppcnj0fn0Y","response":{"attestationObject":"o2NmbXRkbm9uZWdhdHRTdG10oGhhdXRoRGF0YVjFSZYN5YgOjGh0NBcPZHZgW4_krrmihjLHmVzzuoMdl2NFAAAAAAAAAAAAAAAAAAAAAAAAAAAAQQEgZqIURLNToUwerPxCWPoSMMokLhm5JS_plyvrZlZ5keTeK5YP00Zazn_qJa4UUJWrKBmTmmwzOKKaXJ49H59GpQECAyYgASFYIEjOcsLFskQtV98RJaSq5QzeJ6s-JqDUZLCpGP495Ew1Ilgg07dunytQNNIHKKF5q-OwC1lbMC2b5JdBdtwlK9InYx0","clientDataJSON":"eyJ0eXBlIjoid2ViYXV0aG4uY3JlYXRlIiwiY2hhbGxlbmdlIjoiakg3MjZvV1gtdkFzLWIyUm9XZE9SZlZyQWR1aEZFWDNfbjhUeTdOUlpzZyIsIm9yaWdpbiI6Imh0dHA6Ly9sb2NhbGhvc3Q6ODAwMCIsImNyb3NzT3JpZ2luIjpmYWxzZX0","transports":["hybrid","internal"],"publicKeyAlgorithm":-7,"publicKey":"MFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAESM5ywsWyRC1X3xElpKrlDN4nqz4moNRksKkY_j3kTDXTt26fK1A00gcooXmr47ALWVswLZvkl0F23CUr0idjHQ","authenticatorData":"SZYN5YgOjGh0NBcPZHZgW4_krrmihjLHmVzzuoMdl2NFAAAAAAAAAAAAAAAAAAAAAAAAAAAAQQEgZqIURLNToUwerPxCWPoSMMokLhm5JS_plyvrZlZ5keTeK5YP00Zazn_qJa4UUJWrKBmTmmwzOKKaXJ49H59GpQECAyYgASFYIEjOcsLFskQtV98RJaSq5QzeJ6s-JqDUZLCpGP495Ew1Ilgg07dunytQNNIHKKF5q-OwC1lbMC2b5JdBdtwlK9InYx0"},"type":"public-key","clientExtensionResults":{"credProps":{"rk":false}},"authenticatorAttachment":"cross-platform"}

You'll want to check out WebAPI.createAttestationResponse here https://github.com/flutter-institute/webauthn/blob/master/lib/src/web_api.dart#L132
That will return you most of what you are looking for. The server should theoretically be able to handle this response. The WebAPI stuff are helpers I quickly threw in to test against one use case, though, so they might be incomplete.
I was looking over https://www.w3.org/TR/webauthn-2/#authenticatorattestationresponse and the response object the WebAPI helper creates has all the "attributes" available, but does not have any of the "operations" implemented.
Also, I haven't built in support for extensions, yet.

@killermonk thanks! now I am trying the WebApi but still getting this error ""type 'Null' is not a subtype of type 'Map<dynamic, dynamic>' in type cast"
"   when create the CreateCredentialOptions with fake payload to test this {
"authenticatorExtensions": "",
"clientDataHash": "LTCT/hWLtJenIgi0oUhkJz7dE8ng+pej+i6YI1QQu60=",
"credTypesAndPubKeyAlgs": [
["public-key", -7]
],
"excludeCredentials": [{
"type": "public-key",
"id": "lVGyXHwz6vdYignKyctbkIkJto/ADbYbHhE7+ss/87o="
}],
"requireResidentKey": true,
"requireUserPresence": true,
"requireUserVerification": false,
"rp": {
"name": "webauthn.io",
"id": "webauthn.io"
},
"user": {
"name": "testuser",
"displayName": "Test User",
"id": "/QIAAAAAAAAAAA=="
}
}
Do you have an example Create a New Credential and Make an Assertion

@killermonk I think the problem is in fromjson function but my solution is build the payload not from json
// Convert 'rp', 'user', and 'authenticatorSelection' directly from jsonMap.
final rp = RpEntity.fromJson(jsonMap['rp']);
final user = UserEntity.fromJson(jsonMap['user']);
final authenticatorSelection = AuthenticatorSelectionCriteria.fromJson(jsonMap['authenticatorSelection']);
// Initialize the list with a capacity for better performance since its length is known.
var keys = List.filled(2, null, growable: false);
for (var i = 0; i < keys.length; i++) {
keys[i] = PublicKeyCredentialParameters.fromJson(jsonMap['pubKeyCredParams'][i]);
}
// Construct the PublicKeyCredentialCreationOptions object.
final p = PublicKeyCredentialCreationOptions(
rpEntity: rp,
userEntity: user,
challenge: utf8.encode(jsonMap['challenge']),
authenticatorSelection: authenticatorSelection,
pubKeyCredParams: keys,
);
// Finally, create the credential options object.
final rpOptions = CreateCredentialOptions(publicKey: p);

@majdslmt sorry for my delay here, I've been swamped with work so I haven't had time to look into this yet. Thanks for the details, though. I'm hoping to be able to dig in over the next few days.

@majdslmt I need to apologize for my delay here. My personal PC's SSD died and by the time I'd gotten things all fixed up I forgot I had this pending issue that we were discussing.
I went through your payload, and you're getting the current error because it is malformed. If you take a look at the CreateCredentialOptions model at lib/src/models/create_credential_options.dart you can see the properties that are required and how the object should be structured.
First, the only key that should exist on the top level is "publicKey", and all the other options need to be moved inside that as a nested object. That's where the Null error is coming from: it can't find that key. Not the greatest error message of all time from the generated code.
Next, your options need to be restructured slightly.
None of these options are supported in this manner.
  "requireResidentKey": true,
  "requireUserPresence": true,
  "requireUserVerification": false,

They need to be moved into the "authenticatorSelection" object as follows:
"authenticatorSelection": {
  "requireResidentKey": true,
  "userVerification": "required"
}

Also, you are missing the "challenge" data. With those modifications you should be able to use the fromJson options now.
End result is options that look like this:
{
  "publicKey": {
    "rp": {
      "name": "webauthn.io",
      "id": "webauthn.io"
    },
    "user": {
      "name": "testuser",
      "displayName": "Test User",
      "id": "/QIAAAAAAAAAAA=="
    },
    "clientDataHash": "LTCT/hWLtJenIgi0oUhkJz7dE8ng+pej+i6YI1QQu60=",
    "credTypesAndPubKeyAlgs": [
      ["public-key", -7]
    ],
    "excludeCredentials": [{
      "type": "public-key",
      "id": "lVGyXHwz6vdYignKyctbkIkJto/ADbYbHhE7+ss/87o="
    }],
    "pubKeyCredParams": [
      { "alg": -8, "type": "public-key" },
      { "alg": -7, "type": "public-key" },
      { "alg": -257, "type": "public-key" }
    ],
    "challenge": "AQIDBA==",
    "authenticatorSelection": {
      "requireResidentKey": true,
      "userVerification": "required"
    }
  }
}


feat: reroute to dashboard if there is only one budget

Add a tap operator in the effect to fetch budgets data to reroute to the dashboard if there is only one budget.
Pull Request Template
Title
[Provide a succinct and descriptive title for the pull request, e.g., "Improve caching mechanism for API calls"]
Type of Change

[ ] Feature
[ ] Fix
[ ] Refactor
[ ] UI/UX improvement

Description
[Provide a detailed explanation of the changes you have made. Include the reasons behind these changes and any relevant context. Link any related issues. For UI/UX changes, include a screenshot or recording of the changes.]
Testing
[Detail the testing you have performed to ensure that these changes function as intended. Include information about any added tests.]
Impact
[Discuss the impact of your changes on the project. This might include effects on performance, new dependencies, or changes in behavior.]
Additional Information
[Any additional information that reviewers should be aware of.]
Checklist

[ ] My code adheres to the coding and style guidelines of the project as specified in CONTRIBUTING.md.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.

Your Render PR Server URL is https://ynab-custom-reports-pr-8.onrender.com/.
Follow its progress at https://dashboard.render.com/static/srv-cojhn5uv3ddc73eta2d0.

This was open and closed a bunch to test out the lightout report functionality


Format strings not being handled in grammar

Given the grammar:
https://github.com/textmate/markdown.tmbundle/blob/master/Syntaxes/Markdown.tmLanguage it seems that vscode can't properly handle the construct https://github.com/textmate/markdown.tmbundle/blob/master/Syntaxes/Markdown.tmLanguage#L105
i.e.: when parsing a markdown with '##' with this construct (markup.heading.${1/(#)(#)?(#)?(#)?(#)?(#)?/${6:?6:${5:?5:${4:?4:${3:?3:${2:?2:1}}}}}/}.markdown), it'd be expected that the result is markup.heading.2.markdown, whereas vscode is just giving markup.heading.${1/(#)(#)?(#)?(#)?(#)?(#)?/${6:?6:${5:?5:${4:?4:${3:?3:${2:?2:1}}}}}/}.markdown as the result.

@fabioz Do you happen to know how one should interpret / substitute
${1/(#)(#)?(#)?(#)?(#)?(#)?/${6:?6:${5:?5:${4:?4:${3:?3:${2:?2:1}}}}}/}  i.e. what does it mean ?
https://github.com/textmate/markdown.tmbundle/blob/7a8ff34351f96d0a2c15b0f6fe1b0c7091fb08a2/Syntaxes/Markdown.tmLanguage#L105

From your comments at https://github.com/eclipse/tm4e/issues/116

suppose the begin captured ##.
${1} is '##'.
then proceed to match ## against (#)(#)?(#)?(#)?(#)?(#)? and replace with ${6:?6:${5:?5:${4:?4:${3:?3:${2:?2:1}}}}}

${6} through ${3} are ''
${2} is '#'
so the expression resolves to 2.

whoaaa

I think this whould vastly simplify @mjbvz's https://github.com/mjbvz/vscode-comment-tagged-templates/blob/master/syntaxes/grammar.json.


Implement final design for log search filter layout

Update the logging panel layout based on final design specified here:
https://www.figma.com/file/Zzlnjk8BEWEkQusFM3rb9T/Logging?node-id=6%3A5031

fixed by #747


fix: update theme color

ok

:tada: This PR is included in version 1.2.30 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Create Database Schema

We received 0 points with the comment "Missing"

I have added the schema.


feature support select current_catalog();

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 33.33 seconds
stream load tsv:          440 seconds loaded<PHONE_NUMBER>9 Bytes, about 162 MB/s
stream load json:         23 seconds loaded<PHONE_NUMBER> Bytes, about 97 MB/s
stream load orc:          72 seconds loaded<PHONE_NUMBER> Bytes, about 14 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230328044513_clickbench_pr_121608.html

run COMPILE

run COMPILE

run buildall

run p0

run buildall

@morrySnow @morningman
pls review


nvidia-smi does not work

Nvidia driver (version 440) is already installed but when I run nvidia-smi command I get the following error.
NVIDIA-SMI has failed because it couldn't communicate with the NVIDIA driver. Make sure that the latest NVIDIA driver is installed and running.

fixed by #35


Update all scanners to output violations with inventory_index_id, full_name, inventory_data

This is for all the remaining scanners.

Some of the rules have full_name as part of the rule definition and some don't, any reason why is that? E.g. INN, IAP, IAM Firewall rules don't have that as part of the rule definition.


Demo page window expand indefinitely

Describe the bug

On page https://www.egui.rs/#demo, Table Demo and strip Demo windows' height increases indefinitely when UI code is ran.
To Reproduce
Steps to reproduce the behavior:

Visit the page https://www.egui.rs/#demo
Turn on Strip demo, or Table demo
Move the mouse, or change mode from Reactive to Continuous

Expected behavior

I expect the window's size to be fixed
Screenshots

Desktop (please complete the following information):

OS: windows 11 22H2
Browser Chrome 110.0.5481.104
Version

Smartphone (please complete the following information):

Device:
OS:
Browser
Version

Additional context

Oh probably a duplicate of https://github.com/emilk/egui/issues/1568


Support functions that return promises

This lib only supports callbacks. It would be great to support functions that return promises as well.

Tick.wrapPromise = function (name, callback) {
    if (typeof name === 'function') {
        callback = name;
        name = functionName(callback);
    }

    if (name === '') {
        name = 'anon';
    }

    return function() {
        var tick = new Tick(name);
        tick.start();

        name.apply(this, arguments)
          .then(function(results) {
              tick.stop();
              return results;
          })
    };
};

You could still do it using the current functionality:
Tick.wrap(function myFunction(done) {
      myPromise.then(done, done);
    });

... or:
var tick = new Tick("myFunction");
tick.start();
var stop = tick.stop.bind(tick);
myPromise.then(stop, stop);

... or you could create an utility function that could be reused:
var resolve = function(result) {
    this.stop();
    return Promise.resolve(result);
}

var reject = function(result) {
    this.stop();
    return Promise.reject(result);
}

var tickWrapPromise = function(name, promise) {
   name = name || 'anonPromise';

   var tick = new Tick(name);
   tick.start();
   return promise.then(resolve.bind(tick), reject.bind(tick));
}

The result would be passed whatever it is resolved or rejected. But please keep in mind I haven't run any of the above code .. so test it first if you think it is useful.
I would prefer to keep this lib very simple and this feature seams a bit out of the scope of the module.

@leftarmbandit It's been already a few weeks so I'd assume that you already found a solution. I'll close this issue just to keep things tidy.
Please feel free to reopen it if you have something to add.


recommend to add a reference to the parent component for removeElement

I am trying to adapt weex to work on platforms other than iOS and android. I found some rendering directive definitions not very convenient at work.
If I add a reference to the parent component for removeElement, I can implement this removeElement directive more easily.
After adding the parameters referenced to the parent component, the removeElement directive implementation looks like this:
callRemoveElement(instanceId, parentRef, ref)

Welcome to submit a pull request to add removeElement ，and other question what is the platform that you trying to adapter ？

@zshshr webgl

I will close this issue as the conversion happened in https://github.com/apache/incubator-weex/issues/1853


python312Packages.aiortm: 0.9.7 -> 0.9.11

Diff: https://github.com/MartinHjelmare/aiortm/compare/refs/tags/v0.9.7...v0.9.11
Changelog: https://github.com/MartinHjelmare/aiortm/blob/v0.9.11/CHANGELOG.md

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 347559 run on x86_64-linux 1

2 packages built:

python312Packages.aiortm
python312Packages.aiortm.dist


Extraction of Date Picker

Date picker dialog is now its own class.
OnDateSet method remains in the classes where date picker dialog is first opened because attempting to move it to DatePickerDialogFragment class casues cases of where datepicker is no longer responsive or causes the app to crash entirely.

[ ]  run the UI test suite

I'm getting an issue with the UI test suite where tests should pass because I can see that they pass during the run on my emulator are failing. I also went in and did the tests manually and the app is doing what it should (failing to add a habit to list when requirements are not met). Is it safe to ignore these issues?

[ ]   run the UI test suite

I'm getting an issue with the UI test suite where tests should pass because I can see that they pass during the run on my emulator are failing. I also went in and did the tests manually and the app is doing what it should (failing to add a habit to list when requirements are not met). Is it safe to ignore these UI test fails?

@JacobNguyen71, if your UI tests are failing on detecting the toast messages, this was a known issue and a fix was already put into develop a few days ago. I see you have no merge conflicts, so what I'd do is merge develop into your feature branch, and run the UI tests again.


Setting up basic trend reporting in Looker Studio

As a team
we need to monitor anomalies in website usage
so that we can identify and fix bugs or identify use cases for new features.
Reporting requirements:

Global searches conducted
Catalogue searches conducted
Image searches conducted
Story searches conducted
Catalogue works pages viewed
Catalogue item pages viewed
Image modals viewed
Image items viewed

Draft Looker Studio report

Draft Looker Studio report

Looker Studio filter definitions now added to Gitbook.


can't install this library on linux

im using linux mint on an hp probook, and i keep getting this error:
$ go get github.com/kerbyj/goLazagne
../go/src/github.com/kerbyj/goLazagne/mail/outlook.go:6:2: build constraints exclude all Go files in /home/[my user]/go/src/golang.org/x/sys/windows/registry
i tried solving it with https://stackoverflow.com/questions/55348458 but the issue is still present. hope someone can tell me what im doing wrong

Hey! Try this command - GOOS=windows go get github.com/kerbyj/goLazagne
goLazagne uses specific windows functions which are not available on Linux without changing environment

thanks, it worked!


Suppress several warnings

Summary
It is important to reduce sizes of log files for using CI tools. This PR suppresses warnings in order to prevent the sizes from growing larger than necessary.

Baseline: the recent Python 3 + TF build on the master branch

the message /home/travis/miniconda/envs/test-environment/lib/python3.6/site-packages/numpy/lib/type_check.py:546 wastes 793 lines,
the message keras/engine/saving.py:425 wastes 43 lines,
the message keras/engine/saving.py:590 wastes 6 lines.

Related Issues
#9139
#7636

The size of the log files are 351KB for before and 272KB for after.


eventsource: attempt to improve the Retry support implementation (1st attempt)

First attempt to rework retry support. The idea is to do a couple of approaches and see which one works best.

I will close this one. I believe it turned out to be a good idea to expose the interfaces directly. It forces you to think more on how to do stuff.


mqtt client panics

Trying to get the experimental MQTT client to work, but having no luck.
The following code leads to the panic below, however no actually useful error message describing the panic is printed. Decoding the backtrace leads into the mqtt client internals.
fn main() -> Result<()> {
    // Temporary. Will disappear once ESP-IDF 4.4 is released, but for now it is necessary to call this function once,
    // or else some patches to the runtime implemented by esp-idf-sys might not link properly.
    esp_idf_sys::link_patches();

    // setup logging
    esp_idf_svc::log::EspLogger::initialize_default();

    let netif_stack = Arc::new(EspNetifStack::new()?);
    let sys_loop_stack = Arc::new(EspSysLoopStack::new()?);
    let default_nvs = Arc::new(EspDefaultNvs::new()?);
    let wifi = wifi(netif_stack, sys_loop_stack, default_nvs);

    let mqtt_cfg = MqttConfig {
        client_id: Some("foobar"),

        ..Default::default()
    };

    EspMqttClient::new("mqtt://broker:1883", &mqtt_cfg);
    return Ok(());
}

Running leads to the following panic:
Guru Meditation Error: Core  1 panic'ed (Unhandled debug exception).

Decoding using addr2line gives:
~/.rustup/toolchains/esp/lib/rustlib/src/rust/library/core/src/panicking.rs:87
~/.rustup/toolchains/esp/lib/rustlib/src/rust/library/core/src/result.rs:1298
~/.cargo/registry/src/github.com-1ecc6299db9ec823/esp-idf-svc-0.36.3/src/mqtt/client.rs:371
~/.cargo/registry/src/github.com-1ecc6299db9ec823/esp-idf-svc-0.36.3/src/mqtt/client.rs:101
~/.rustup/toolchains/esp/lib/rustlib/src/rust/library/alloc/src/boxed.rs:1705
~/work/diox-rs/.embuild/platformio/packages/framework-espidf/components/esp_event/esp_event.c:145
~/work/diox-rs/.embuild/platformio/packages/framework-espidf/components/esp_event/esp_event.c:582 (discriminator 3)
~/work/diox-rs/.embuild/platformio/packages/framework-espidf/components/mqtt/esp-mqtt/mqtt_client.c:926
~/work/diox-rs/.embuild/platformio/packages/framework-espidf/components/mqtt/esp-mqtt/mqtt_client.c:913
~/work/diox-rs/.embuild/platformio/packages/framework-espidf/components/mqtt/esp-mqtt/mqtt_client.c:1371
~/work/diox-rs/.embuild/platformio/packages/framework-espidf/components/freertos/port/xtensa/port.c:168

Same here, cant get it to work either.
An example would be really nice.

Working on this. Probably will have something later today.

Should work now. Demo snippet: https://github.com/ivmarkov/rust-esp32-std-demo/blob/main/src/main.rs#L710


Choosing enconding at PostgreSql

Environment
Knex version: 0.14.4
Database + version: PostgreSql 9.4
OS: Windows/Ubuntu
Feature discussion / request

Explain what is your use case
Unfortunately I need access a postgresql database (hosted in a windows server) using a SQL_ASCII encoding. When I use the Knex I got the message: Unhandled rejection error: invalid byte sequence for encoding "UTF8": 0xc1 0x52.

Explain what kind of feature would support this
This feature will allow choose the postgresql database encoding

Give some API proposal, how the feature should work
A feature at database connection like encoding, or a function to use this in querys (select, insert, update, etc.)

How is this implemented with pg driver? Does PG driver support different encodings? I don't think this should be handled in any ways in knex level.

Lets reopen if there is something that needs to be done in knex side.


[Ubuntu Server 18.04] Cannot find module 'node-webhooks'

I downloaded the 1.3 release to my Mac, and then copied it over to my Ubuntu 18.04 Server and placed it in /usr/local/bin/. I then chmoded it according to your readme. Then I connected my hub to my server via USB, and then started up the application. This error is then thrown:
internal/modules/cjs/loader.js:775
    throw err;
    ^

Error: Cannot find module 'node-webhooks'
Require stack:
- /usr/local/bin/HarmonySpan_Linux_x64/__nexe_stdin.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:772:15)
    at Function.Module._load (internal/modules/cjs/loader.js:677:27)
    at Module.require (internal/modules/cjs/loader.js:830:19)
    at require (internal/modules/cjs/helpers.js:68:18)
    at Object.<anonymous> (/usr/local/bin/HarmonySpan_Linux_x64/__nexe_stdin.js:2:18)
    at Module._compile (internal/modules/cjs/loader.js:936:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:947:10)
    at Module.load (internal/modules/cjs/loader.js:790:32)
    at Function.Module._load (internal/modules/cjs/loader.js:703:12)
    at Function.Module.runMain (internal/modules/cjs/loader.js:999:10) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/usr/local/bin/HarmonySpan_Linux_x64/__nexe_stdin.js' ]
}

Thank you for opening the issue!
This looks like an issue with how my binary was put together - let me recompile on a Linux machine and I'll throw it back up on the releases page.

HarmonySpan v1.3.1 just released, download it at https://github.com/AShifter/harmony-span/releases/tag/v1.3.1 - compiled the Linux build on a machine running Ubuntu 20.04 using the same build script, let me know if it works any better!

Thanks, and good news: Now it works just fine! :)
On a sidenote, how do you leave it running when closing the ssh shell? Should I detach/attach as a screen session, or do you have anything else that you think should be used for the Linux server users? (as you might have understood I do not have the complete background knowledge about Linux (I am a hobby Linux-dude), so stuff like this I need to ask to make sure I am doing it as intended).

I would just detach/attach as a screen session - most of my expertise is in Windows so I haven't spent the time figuring out how to get it working in Linux haha.
Feel free to open another issue if you run into trouble! Thank you for the help!


Migration and Mixing with type providers

I’m having a problem migration away from the SqlClient type provider to Facil - Our project is so big the switch over to Facil can’t be done in one hit while also keeping up with normal development.
We have mutable databases feeding into mutable projects then mutable apps so changing one database access layer has a knock-on effect.
I was hoping to mix Facil with the SqlClient but as we have to compile our project with FSharp.Compiler.Tools to get around the Type Provider problems it means the anonymous records Facil creates break the build.
error FS0010: Unexpected symbol '|' in expression
error FS0604: Unmatched '{'

I know FSharp.Compiler.Tools is a dead project and everyone should switch away from it but in a production this is easier said than done.
I see we can use Result type to tell Facil not to create the anonymous records but adding all the types for all the queries is a bit too much.
I’m happy to do the work and add an option to the Result type which creates a new create record type but before I do I just wanted to make sure you would be happy with this as I know at this moment in time (the early stages of development) you would rather have complete control of code and any changes especially big ones.
At the moment our migration is dead in the water because we had to branch our whole project which has quickly become behind with ongoing development. This is in no way Facil fault it's a great tool just we can't use it because of our setup. As the type providers are widely used this addition could help others projects move to Facil.

This should be a fairly simple change. Thanks for the PR offer, but I think it's probably quicker for me to implement it myself than to review your PR (and that's not even including the time you must spend).
But let me first suggest a workaround: Would it be possible (during migration) to use a separate F# project for the stuff generated by Facil, and compile that using something other than FSharp.Compiler.Tools?

@cmeeren thank you so much 🥇
I did a try separate project but FSharp.Compiler.Tools throws the following errors:
FSC : error FS0229: Error opening binary file 'x.dll': Error reading/writing metadata for the F# compiled DLL 'x.dll'. Was the DLL compiled with an earlier version of the F# compiler? (error: 'lookup_uniq in table istrings out of range, n = 99, sizeof(tab) = 84').
FSC : error FS3160: Problem reading assembly 'x.dll': The exception has been reported. This internal exception should now be caught at an error recovery point on the stack. Original message: Error opening binary file 'x.dll': Error reading/writing metadata for the F# compiled DLL 'x.dll'. Was the DLL compiled with an earlier version of the F# compiler? (error: 'lookup_uniq in table istrings out of range, n = 99, sizeof(tab) = 84').)

Hopeful once all the migration is complete we can move away from all this legacy stuff and onto just net5 (or net6, 7 etc)
There could be other fun to be had down the road but one problem at a time.

Released in v0.2.5. You can now set result to "nominal".


Support for OpenJ9 milestone builds

The Eclipse handbook suggests doing milestone builds before a release. This is a pre-release build that is distinguished from typical nightly builds. It would be built from a branch and not the main line. It gives users a chance to try out a pre-release driver containing new features and report any problems. Which provides development a chance to fix the reported problems before the release is finalized.
For the OpenJ9 0.12.0 release in January 2019, milestone builds are scheduled in the plan for Dec 17 and Jan 7.
The milestone build would be named similar to a release but contain M1 in the name, i.e. _openj9-0.12.0M1. FYI the outstanding 0.11.0 naming problem #678
The OpenJ9 release process describes the format of the OpenJ9 milestone release tags, although since AdoptOpenJDK builds from an OpenjDK extensions branch, this part would be invisible to the Adopt build.

Edited the description to add the Dec 17 OpenJ9 milestone date.

Related: https://github.com/AdoptOpenJDK/openjdk-website/issues/217 (Could handle both under the same concept of "significant prerelease build")

Could handle both under the same concept of "significant prerelease build"

Works for me.

@pshipton can we close this?

@lumpfish @andrew-m-leonard do you know if this is good to be closed?

@pshipton are we happy with the current way we handle milestone builds?

I would like the milestone builds to be more prominent on the Adopt web site, so users can easily notice and find them.

@DanHeidinga

I would like the milestone builds to be more prominent on the Adopt web site, so users can easily notice and find them.

I guess this is covered by https://github.com/AdoptOpenJDK/openjdk-website/issues/217 so this issue can be closed.


*: update and unify rbac doc and rbac utils with rbac templates

Addresses #1527
This PR updates the RBAC setup guide to explain how to setup RBAC with either a ClusterRole or Role, depending on the create-crd flag.
Added RBAC templates at example/rbac so that both the RBAC doc and hack/ci/rbac_utils.sh use the same permissions.
The rbac_utils still sets up RBAC with a ClusterRole since our e2e tests test the etcd-operator for the default mode of --create-crd=true.

lgtm


ScaleBar issue or Bad coding?

I am having scale bar issues upgrading to ol 4.6.5 on many levels.
My Code:
//Check if a coordinate System Override is present i.e. if it is Override the WKT that came from Mapguide
if (display_srs !== undefined && display_srs !== null && display_srs != '') {
Mapguide.Wkt = Map_display_srs;
}
var tempproj = 'EPSG:' + Mapguide.EpsgCode;
//First see if Mapguide EPSG code is recognized
var csproj = ol.proj.get(tempproj);
if (csproj === undefined || csproj === null) {
	// Not recognized
	proj4.defs('Custom', Mapguide.Wkt);
	var x = proj4.defs('Custom');
	var newProj = ol.proj.get('Custom');
	ol.proj.addProjection(newProj);
	Map_display_srs = 'Custom';
	newProj.setGetPointResolution(function (resolution) { return resolution; });
}
else {
	//Proj4js Recognized the Mapguide EPSG Code
	csproj.setGetPointResolution(function (resolution) { return resolution; });
	Model.Config.general.display_srs = Model.MapguideSrs = tempproj
}

This Reads the Coordinate System from a Mapguide layer source and determines if the user has set an override WKT and if Mapguide EEPSG code is not in proj4js it creates one from the Mapguide WKT or Override WKT. This worked fine in my old open layers.
if I don't use newProj.setGetPointResolution(function (resolution) { return resolution; });
i get these results for a scale bar next to a 1 mile Square.

Spherical Mercator ^ (correct)

State Plane ^ (incorrect)
after adding newProj.setGetPointResolution(function (resolution) { return resolution; });

Spherical Merc ^ (incorrect)

State Plane ^ (correct)

This is probably a duplicate of #7700


run_escape module not found

Hi,
Thank you for the great work. I am trying to use the work for further research, however, I encountered errors when running the demo. It seems that some package in lib folder is missing (from lib.run_escape import names, run_escape).
Error message: ModuleNotFoundError: No module named 'lib.runescape'.
Thanks!

Hi, thank you very much for letting me know. I have now added the file. Please let me know if the problem persist.

Hi @DavideBuffelli,
The problem has been solved. Thank you very much!
Best


problem with node module

Problem with standart module NodeJS: url module, http, https, zlib,stream
my setting file:
          webpack: {
            custom: {
                entry: './src/index.js',
                output: {
                    path: 'build/',
                    filename: 'index.js'
                },
                target: 'node',
                resolve: {
                    extensions: [
                        '.js',
                    ],
                    root: '.'
                },
                module: {
                    loaders: [
                        { test: /\.js$/, loader: 'jsx-loader' },
                    ],

                }
            }
        }

After run build index.html, and see error: Uncaught ReferenceError: require is not defined
Opened details, and see:
function(module, exports) {

	module.exports = require("url");

/***/ },

require not spun. How it fixed?

It might be because your target is target: 'node', which is set for node and expects your bundle to be loaded in a node-like environment (where require exists).
If you leave that configuration out, it will default to 'web' which is likely what you want if you're loading in index.html.

thnx

Adding target: 'web' in my webpack.config.js fixed the issue indeed.


How to configure typescript 1.5 and tsconfig file

I would like to add tsconfig file and I want to use the typescript 1.5.0-beta.
I have not found any documentation how I can configure these two things together.

Same problem here, i need to use typescript 1.5.0-beta and this plugin uses 1.4.1...

You can use this:
var tsProject = ts.createProject('tsconfig.json', { typescript: require('typescript') });

And install typescript as a (dev) dependency with the version you want to use. When 1.5 (stable) is released, I'll release a new version of gulp-typescript with 1.5 as default.

Thank you for your help.
It will help until you will update the package with 1.5. Actually the 1.5 is released today.

Working on it (#167)

New version is released on NPM

Thanks


fix: Vc count logic

I ran into a few index out of bounds and divide by zero issues. I fixed those with what I expect is the right method. One thing I would like you to check is my change on the participant_network.star
I added a check on the vc_context assignment for the case where vc_count == 0. We would get an index out of bounds panic so I just set it to None. I'm unsure if there are downstream consequences but everything appears to be working on my end with that.

can you please run kurtosis lint --format to lint this PR?

vc_context = None should work fine, if we have no vc client set.