[v1.9] Update K8s to v1.19.12

k8s: Update libraries to 1.19.12

test-backport-1.9

test-backport-1.9

test-only --focus="K8sVerifier.*" --k8s_version=1.18 --kernel_version=419

test-1.17-5.4

test-1.18-4.19

Branch needs to be rebased, but since these are just library changes, it should be okay to merge.


Long lines in CodeSnippets increase the width beyond 50%

Description
Given Long lines in CodeSnippets increase the width beyond 50% we should set a max-width.

This seems to be fixed. Feel free to reopen if this is still an issue!


Folding a Piece of Paper

Before jumping into Pr first comment for assign. All Problem from Edabit. Link is mandatory to add
Problem statement:  Create a function that returns the thickness (in meters) of a piece of paper after folding it n number of times. The paper starts off with a thickness of 0.5mm.
Problem Link: https://edabit.com/challenge/4t6YAJS8dtT7RQjta

Before submitting a PR please Check some details. Check Edabit for problem task

[x]  fileName format should be in camelcase
example- returnTheNextNumber.py
[x]  Add label for Problem
[x]  In file there should be problem statement and link to problem
example -

'''
    Problem Task : This program will add two numbers
    Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
'''
*/
def main():
    return a + b

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

@aayushi-droid can i work on this ?


Bug(ish): Kernel buff/cache usage prevents downscaling

This is also kind of a feature request.
Basically, buff/cache memory usage is not counted separately from other usage, which means that VMs with non-trivial read loads will tend to balloon in memory usage.
There's two parts to resolving this:

The vm-monitor needs to allow downscaling when that may require evicting some memory that's currently being used by the kernel for buff/cache

Note: We should be careful here to appropriately factor in "dirty" cache usage, because otherwise downscaling might block on writeback

The autoscaler-agent needs to take buff/cache mem usage somewhat into account, so that we can maintain some cache available to be used, but not too too much.

# Tasks
- [ ] Change vm-monitor to ignore buff/cache memory usage when considering downscale
- [ ] Change autoscaler-agent to _partially_ consider buff/cache usage when deciding target memory usage

Probably easiest to implement this after #371.

This should actually be mostly fixed by neondatabase/neon#5524, now that #371 is merged so the agent retries downscale requests with progressively higher CU.
That fix will still generate a lot of extra traffic, so we'll still want the changes in the agent, unfortunately.

I think this is actually already fixed. There's still imperfections (agent often disagrees with vm-monitor), but they're mostly intentional for now, pending a larger redesign.


[FIX] Replaced hardcoded Storage Account, cleaned up Script

Simple fix to swap out the hardcoded data store value.  Cleaned up the script while I was looking at it.
Ran locally against provider crud and worked fine.

Lint report for commit c90edf91858dbb0782fce3633a7a1db5df691d6d:
Everything seems all right :smile: .
CFME QE Bot

@jteehan So the storage_account is already present in our yamls, is that correct?

Yes.  I added that on June 15th to the yamls when I first noticed the issue.  Now I'm actually using it.

So I will check it out and merge if it works, thanks for the answer Jeff.


How do I change the coco iou thr in tools/test.py?

Hello, I train with a custom dataset and run test.py.
python test.py ... --eval mAP

This is the existing coco test standard, so the results are very strict. Which part should I modify to control this? Thank you

Is the default 0.5? Which file should I refer to?
I wonder if the mAP test is based on the base coco test that (0.5:0.95:0.05).

default is 0.5:0.95, you can refer to coco.py.


WFCORE-597 Where an ObjectTypeAttributeDefinition is in use respect t…

…he ResourceOnly setting on contained types for add operations.

Full integration - Windows Build 216 outcome was FAILURE using a merge of 515b44f191e342f2ba28d6f235dc1296b4d1199d
Summary: Execution timeout (new); tests passed: 1101, ignored: 290 Build time: 01:16:18

Full integration - Windows Build 225 outcome was FAILURE using a merge of be5cb9e17911044f2697c12f681cc5500f95a42b
Summary: Tests passed: 3122, ignored: 364; swabra cleanup failed (new) Build time: 00:51:52


feat(#124): don't move method for string decoding

I've checked the code and there is no need to move that method into XmlData because it serves different purpose.
So I just close that puzzle.
Closes: #124

History:

feat(#124): rename and move the hex convertation method
feat(#124): move static method back

PR-Codex overview
Detailed summary

Replaced the hexToString method with decodeHexString in the XmlInstruction class.
Removed duplication between XmlInstruction and XmlData classes.
Refactored the code to improve readability and remove duplication.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)


LXD wrong paths on ZFS

tank/lxd on /tank/lxd type zfs (rw,noatime)
tank/lxd/containers on /tank/lxd/containers type zfs (rw,noatime)
tank/lxd/deleted on /tank/lxd/deleted type zfs (rw,noatime)
tank/lxd/deleted/images on /tank/lxd/deleted/images type zfs (rw,noatime)
tank/lxd/images on /tank/lxd/images type zfs (rw,noatime)
tank/lxd/images/4c779c2f068cfa819c27e3775f98ec88b098b75053b68dd0ece8630b6ba1278f on /var/lib/lxd/images/4c779c2f068cfa819c27e3775f98ec88b098b75053b68dd0ece8630b6ba1278f.zfs type zfs (rw,noatime)
tank/lxd/images/e9ba98e963c89a5be3e11ab55f0d6eff93cc47e88a75d00fc900c38bc4a5aba4 on /var/lib/lxd/images/e9ba98e963c89a5be3e11ab55f0d6eff93cc47e88a75d00fc900c38bc4a5aba4.zfs type zfs (rw,noatime)
tank/lxd/images/8bc41c8af0a852d674551772a3f0c6afe893b13adfd4ca1c5b9a37a0d90c51e1 on /var/lib/lxd/images/8bc41c8af0a852d674551772a3f0c6afe893b13adfd4ca1c5b9a37a0d90c51e1.zfs type zfs (rw,noatime)

Why is not mounted everything under /var/lib/lxd? Is that because ZFS backend was configured (obviously) after lxd get installed?
Why are images mounted by default? With these long hashes the mount output gets ruined.
Anyway, my current problem is that lxc publish does not work, because it intents to write and image file under /var/lib/lxd/images, which is not on zfs and there is no enough space this way.
BTW why does it want to write an image and not create a zfs dateset?
Am I lost in space somehow?

My current problem was solved by
zfs set mountpoint=/var/lib/lxd/images tank/lxd/images

But it's just a workaround.

It's intentional.
LXD uses a zpool regardless of where it's mounted and just sets mountpoint= as needed.
It keeps the containers and images mounted at all time because that's what the other backends do and having to special case zfs would have made things more difficult. I did write a branch which would do opportunistic mounts/unmounts of zfs but it started hitting a lot of races, including a bunch inside the zfs kernel code when running scale tests with a few thousands containers.
The LXD image hash is the sha256 of the compressed image so we need to produce that compressed image regardless of storage backend, it's also required for further exports or exchange with remote servers.
As to why we don't mount /var/lib/lxd/images or even the whole /var/lib/lxd directly on the zfs pool you provided, it's because we support "upgrading" to zfs, where old containers and images get converted to zfs as they get copied (for containers) or used (for images).
We also plan on supporting the use of multiple zfs pools or mixing storage backends in the future, which will also require that the main LXD directories themselves aren't tied to a specific backend.
Your workaround is perfectly fine so on systems that are similarly space constrained, doing what you did is quite fine. Just make sure you move any existing content over to the zfs filesystem (or do it before you import any image).


Support for Local Account username in Azure B2C with simple names as as "joe", "john" and no domain name association

Documentation refers to support of simple usernamaes for login - when using Azure B2C. But there are not steps mentioned on to show to configure this. User creation process either via Graph API or via Azure portal screen, always ask for verifiable domain name to be attached to username.
Please confirm the steps required to have basic usernames - and not domain string.

@yrvverma Can you provide the document link you are referring to? Thank you!

Hi Mike, Please check this link
https://docs.microsoft.com/en-us/azure/active-directory-b2c/active-directory-b2c-faqs
This is the text in the link that is causing confusion across multiple implementations.
"We also support arbitrary user names as local accounts (for example, joe, bob, sarah, or jim). You can choose one of these two local account types by configuring Azure AD B2C in the Azure portal."

@PatAltimore The user is looking for details on configuring simple user names but is unable to find that content.

@Mike-Ubezzi-MSFT Can you assign to @davidmu1?  David works on B2C now.  @davidmu1  Can you verify the author metadata on B2C articles?  Thanks!  FAQ might need to be saeeda.

#in-process

@DavClk : please can you help if simple usernames are supported and the steps required to configure.

@yrvverma.... i am of no help on this thread. I think you want @davidmu1

@yrvverma This text will be added to the FAQ article:
You can choose one of these two local account types when configuring identity providers for Azure AD B2C in the Azure portal. In your Azure AD B2C tenant, click Identity providers and then select Username under Local accounts.
User accounts for applications must always be created through a sign-up policy or by using the Azure Graph API. User accounts created in the Azure portal are only used for administering the tenant.

@davidmu1 : Thanks for the response. It does not answer in clear form that questions that every B2C AD implementer is seeking to get answered. if we chose "username" under local accounts; the FAQ says that B2C AD sign in get be done without the format<EMAIL_ADDRESS>As per our testing, this is not the case to be. Even if username is chosen under "Local Account" the sign in credentials will required<EMAIL_ADDRESS>as the format.
I would like simple confirmation that explicitly clarifies if Azure B2C AD allows login using simple usernames without @ symbol.

According to the engineering team, this works as designed and is documented accordingly.
#please-close

@yrvverma We will now close this issue. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Add option to not generate "Since: [ver]" in docs

We were not using the version-info in a project at work, so i added an option to not generate them in the first place. Keeping the default to the current behaviour, i thought maybe upstream could want this change as well.

Pull Request Test Coverage Report for Build 98

0 of 0   changed or added relevant lines in 0 files are covered.
45 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.1%) to 89.0%

Files with Coverage Reduction
New Missed Lines
%

/home/travis/virtualenv/python3.5.6/lib/python3.5/site-packages/doxyqml-0.5.0-py3.5.egg/doxyqml/main.py
15
66.36%

/home/travis/virtualenv/python3.6.3/lib/python3.6/site-packages/doxyqml-0.5.0-py3.6.egg/doxyqml/main.py
15
66.36%

/home/travis/virtualenv/python3.4.6/lib/python3.4/site-packages/doxyqml-0.5.0-py3.4.egg/doxyqml/main.py
15
66.36%

Totals

Change from base Build 97:
-0.1%

Covered Lines:
1068

Relevant Lines:
1200

💛  - Coveralls

Looks good, thanks!
Can you add a functional test for this option? I just added support for testing command line arguments to Doxyqml functional test "framework". It's documented in https://github.com/agateau/doxyqml/blob/master/tests/functional/README.md.
You can start from the "namespace" test as an example.

rebased onto latest master and added a functional test to test the absence of the "Since: .." version entry.

Sorry for the delay, looks great now, merging it. Thanks again!


[GPU] LightGBMError: bin size 3142 cannot run on GPU

Description
Hi, I encountered an error which has been posted by many people while training a LightGBM model using the GPU: LightGBMError: bin size 3142 cannot run on GPU. This error seems to be related to the max_bin parameter setting. I tried reducing the bin size, but it has not resolved the issue.Just wanted to know if there is any fix for this or how I can train lgb model with categorical features having large number of categories.
Reproducible example

params = {
    'boosting_type': 'gbdt',
    'objective': 'binary',
    'metric': 'binary_logloss',
    'device': 'gpu',
    'max_bin': 200,
    'num_leaves': 31
}

train_data = lgb.Dataset(X_train, label=y_train)
model = lgb.train(params, train_data)

Environment info
lgb version : 4.5.0
cuda version : 11.8
Python 3.7.13
Thanks in advance.

Hi team, just a gentle reminder.Any update on this ?

我也有碰到了这个问题，正在寻求解决办法。

@crs1910
I've also encountered this issue. It seems there is a solution in this pull request, but I don't know how to try it out. If you succeed, could you please let me know how to do it?
https://github.com/microsoft/LightGBM/pull/6019

@crs1910
lgbm version: <IP_ADDRESS>
When using the GPU version of LightGBM, I encountered the error:
LightGBMError: bin size 257 cannot run on GPU.
After checking the issue, I found that a fix for this problem was merged 5 months ago in [https://github.com/https://github.com/microsoft/LightGBM/pull/6019].
However, I am still encountering this issue.
Has anyone else encountered this problem? How should I resolve it?

Thanks for reporting the issue. We will verify to see if it still exists in latest commit. We will report back here soon.


Object doesn't support property or method 'map' in ZeroClipboard.js

Hi,
When I am trying to copy text to clipboard from IE I am getting the following exception in ZeroClipboard.js..
Object doesn't support property or method 'map'..
My code looks like this...

  <script type="text/javascript" src="JQuery/jquery-1.3.2.js" />
  <script type="text/javascript" src="ZeroClipboard/ZeroClipboard.js" />
   <script type="text/javascript">
    if(!window.location.origin) {
    	window.location.origin = window.location.protocol + "//" + window.location.host;
    }
    $(document).ready(function(){
    	try{
    		var clip = new ZeroClipboard($('.clipcopy'));

    	clip.on("copy", function(event) {
    	var clipData = $(event.target).attr('data-clipboard-text');
    	clipData = window.location.origin + clipData;
    	event.clipboardData.setData("text/plain", clipData);
    	});
    	}catch(err)
    	{
    		alert(err.message);
    	}
    });

Please see README.md#support

P.S. Even if you need to support "oldIE", please consider upgrading your version of jQuery to the latest 1.x version... your version is 4 years and 11 months old. :weary:


dns/bind: allow signed zone transfers

This allows you to add a TSIG key to your secondary zones so the primary can authenticate your request.

@fichtner @fraenki I submitted the original PR > 8 months ago, any update on when it can be merged? I'd really like to get our offices off pfSense routers!

@miken32 thanks for ping... I was hoping @fraenki could take a brief look again and indeed it should have been done sooner. sorry about that.

Sorry, looks like I accidentially deleted the notification. :-/
Regarding the review, that's probably not very useful, because I don't use signed zone transfers in BIND. Besides that I like the effort to make the template more readable by splitting this huge single-line thingy into multiple lines :+1:

I'd merge it. @fichtner Any objections?

Merged, thanks!

Thanks @fichtner ! When can I expect to see this show up as an update on my installation?

@miken32 should find its way into 21.7.7 next week

@fichtner FYI I've created #2874 to fix an issue with this commit. Not sure if it's been spotted or not yet.

@miken32 slipped through, sorry. gentle reminder is always good... hoping for a review by @mimugmail


Partial amendment

Partial amending has been problematic during the development of the 4 series as well as 4.2. The problem is that by resetting the line and column but not the offset, the caret is rendered in the wrong position, even if it has the correct "win" behaviour against sibling errors.
If we want to make this work properly, we need to distinguish between the trueOffset and caretOffset of an error message. A full amend will adjust both of these offsets, but a partial amend will only adjust the caretOffset. By doing this, we'll be able to support partial amending behaviours on filtering combinators in 5.0.0 when the semantics can be modified.

As an added extra, we can also extend the caret from the true offset to the caret offset: this is actually what filtering combinators do, which is an added plus!


Feat/921 change icons doc

Hey @veilvokay , I am ready to approve the task, but we should merge the PR #952  from @davidken91 first. This may lead to some conflicts in your branch, as David also worked on the readme. This is why I will wait with the approval.

Hey @veilvokay , #952 has just been merged and leads to some conflicts. Once these are resolved  and we have 2 approvals again, we can also merge your branch :-)


.Net 8.0 install is not installing latest SDK

Type of issue
Outdated article
Description
Running sudo apt-get install -y dotnet-sdk-8.0 is installing 8.0.100-rc.2.23502.1 [/usr/lib/dotnet/sdk]
Page URL
https://learn.microsoft.com/en-gb/dotnet/core/install/linux-ubuntu-2304
Content source URL
https://github.com/dotnet/docs/blob/main/docs/core/install/linux-ubuntu-2304.md
Document Version Independent Id
c43f280d-ee30-6092-b48c-93b58eb1dea1
Article author
adegeo

Hello. It may take time for packages to show up in the distro feeds. If you're using the Microsoft feed, I've verified it is in the feed, you may just need to update your feed data.


initial syntax analyzer

added language description and generated parser
added ast classes(not final)
improved lexer
integrated lexer tokens with parser tokens
slightly changed project structure

also added round bracket support for expressions in 46345ee


Release 22.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.406%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
590

Relevant Lines:
602

💛  - Coveralls


Include public workspaces

adds a checkbox on the workspaces page that includes public workspaces into the list.

@katevoss I think clicking on the text to check the box is the standard behavior we have


chore: adhere to licenses

Extract 3rd party licenses and include them in the bundle.

Not necessary as dependency source code is installed by consumers. However, adding license scanning with a denylist of restrictive licenses would be considered due diligence.


Windows Enviroment: useref not injecting the files that are in "bower_components"

Explanation:
When gulp-inject adds the references from the bower components folder, it creates them with this path:
    <!-- build:css styles/vendor.css -->
    <!-- bower:css -->
    <link rel="stylesheet" href="../../bower_components/bootstrap/dist/css/bootstrap.css" />
    <link rel="stylesheet" href="../../bower_components/font-awesome/css/font-awesome.css" />
    <link rel="stylesheet" href="../../bower_components/toastr/toastr.css" />
    <!-- endbower -->
    <!-- endbuild -->

After running the optimize task, the vendor.css and vendor.js where not created in the build folder. The issue was that gulp-useref couldn't locate correctly the bower_components folder because was looking in a different path:
C:\Users\eliashdezr\bower_components\jquery\dist\jquery.js
C:\Users\eliashdezr\bower_components\angular\angular.js

Instead of this
C:\Users\eliashdezr\Desktop\pluralsight-gulp\bower_components\jquery\dist\jquery.js
C:\Users\eliashdezr\Desktop\pluralsight-gulp\bower_components\angular\angular.js

Solution/Workaround
I just added another pipe in the inject task using gulp-replace
gulp.task('inject', ['wiredep', 'styles', 'template-cache'], function () {
    log('Wire up the bower css and js references into the index.html');

    return gulp
        .src(config.index)
        .pipe($.inject(gulp.src(config.css)))
        .pipe($.replace(/="..\/..\/bower_components\//g, '="/bower_components/'))
        .pipe(gulp.dest(config.client));
});

This works for me, thank you very much.

this worked for me as well on linux(ubuntu), thanks

Awesome, thanks for this

Awesome !!! thank you very much
I was trying to figure out why the useref look  two folders before like if we would used something like this:
../../

weird....

just downloaded the zip folder...it has no bower_components directory in the root..

@Dcolonel6 try and install with bower type bower install in your command line or terminal


A way to reset expect?

Hope it's ok to ask a question in the Issues.
Is there a way to drop an expectation that was set ealier?
My case for context: I call an expect in the setup block, but then for a specific test I want to override that, so it doesn't expect that call. I tried to cancel it with reject but it didn't work (and maybe it shouldn't after all).

First, totally fine to ask questions here :+1:
No way to drop expectations. Supporting such feature would mean that the user would need to have some way to reference an expectation to then remove it and this would complicate quite a bit the codebase.
My suggestion would be to use ExUnit setup blocks. One example that can be done per test is using module attribute @tag and specify which test you want the expectation to be set-up or not.
Something like this:
defmodule ExampleTagModificationTest do
  use ExUnit.Case

  setup %{mimic: expect} do
    if expect do
       expect(MyModule, ....)
    end
    :ok
  end

  @tag mimic: true
  test "tags modify context" do
     # test here with expectation
  end

  @tag mimic: false
  test "tags modify context" do
     # test here without expectation
  end
end

That's a nice trick, with the tags ❤ Another work-around that I ended up using so far was to actually split my tests into describe blocks that do and don't define the mock via the related setup.


dependency: added shorthand for specifying version tag

Adds : syntax for specifying dependency versions. The two lines now do
the same thing:
acbuild dep add example.com/debian:jessie
acbuild dep add example.com/debian --label version=jessie

Partially fixes https://github.com/appc/acbuild/issues/128.

can we use this? https://github.com/appc/spec/blob/master/discovery/parse.go#L45

sure
On Mon, Nov 30, 2015 at 2:33 PM, Jonathan Boulle<EMAIL_ADDRESS>wrote:

can we use this?
https://github.com/appc/spec/blob/master/discovery/parse.go#L45
—
Reply to this email directly or view it on GitHub
https://github.com/appc/acbuild/pull/137#issuecomment-160783210.

Uses appc/spec/discovery.NewAppFromString now. How's that look?

a nit but otherwise lgtm

nits solved. Last call for more nits.

Yup makes sense. I just moved all of the string parsing and related sanity checking to acbuild/dependency.go. How's that look?

I guess the help for the add subcommand should be tweaked a bit too to reflect it's not strictly an image name..

Double last call for more nits.

el gee tee emm


Specifying values for "capabilities" parameter?

Hi there. I have a template that creates some IAM objects, and I apparently need to specify --capabilities CAPABILITY_IAM but I'm not clear on how to do this or if it's even possible. I have tried:
name: Deploy to AWS CloudFormation
        uses: aws-actions/aws-cloudformation-github-deploy@v1
        with:
          name: stack-name
          template: master-packaged.template.json
          parameter-overrides: >-
               capabilities=CAPABILITY_IAM

and
name: Deploy to AWS CloudFormation
        uses: aws-actions/aws-cloudformation-github-deploy@v1
        with:
          name: stack-name
          template: master-packaged.template.json
          parameter-overrides: >-
               Capabilities=CAPABILITY_IAM

and also created a parameters.json file containing
[
	{
	  "ParameterKey": "Capabilities",
	  "ParameterValue": "CAPABILITY_IAM"
	}
]

and this action:
name: Deploy to AWS CloudFormation
        uses: aws-actions/aws-cloudformation-github-deploy@v1
        with:
          name: stack-name
          template: master-packaged.template.json
          parameter-overrides: >-
               [Capabilities=CAPABILITY_IAM](parameter-overrides: "file://${{ github.workspace }}/parameters.json")

In all cases the action fails and I get this error:
Error: Parameters: [Capabilities] do not exist in the template
Am I barking up the wrong tree here?

Hi Lamont,
You can specify capabilities at the same level as parameter-overrides with a comma delimited list
name: Deploy to AWS CloudFormation
        uses: aws-actions/aws-cloudformation-github-deploy@v1
        with:
          name: stack-name
          template: master-packaged.template.json
          capabilities: CAPABILITY_IAM, CAPABILITY_NAMED_IAM


Added pems_clearinghouse_start_date variable

Variable added to dbt_project.yml and staging__station_raw model to minimize computation resources while we continue to develop/refine our downstream models. The variable is currently not added to station_meta and station_status based models due to their relatively small size and limited equipment updates by some districts (primarily district 11). This is associated with #181.

This PR closes #181.

This will be good to go in once CI is passing. If you make further edits while this is still a draft please re-request review. Thank you @kengodleskidot!

@britt-allen we can discuss which other models may need to add this variable. My thought is that applying the variable to the staging_clearinghouse__station_raw model will limit the data being reprocessed for all associated downstream models but if you can think of any other models this would be good to implement with let me know.

This looks good to me, thanks @kengodleskidot! I agree that stg_clearinghouse__station_raw is the most important one to apply this to.
Since it's been in place for a week, have you noticed any improvements in development iteration times?


sample project needs more

I find it dissapointing that the sample code does not show how to extend the applicationuser with extra properties.

New documentation is here: https://docs.microsoft.com/en-us/aspnet/core/security/authentication/customize_identity_model?view=aspnetcore-2.1


ci(test): split into multiple machines

🎉 Thanks for submitting a pull request! 🎉
Summary
This is a PR to split our tests across multiple machines.
ava sorts the files by name, then split them to chunks across machines. See here
Due to ava's behavior I grouped our unit tests to run during the build job and our integration tests to run during the test job. Integration tests will get splitted across machines.
Since there isn't a good way to load balance the tests, I renamed the specs to better distribute them (yes I know this is ugly).
I also divided the dev command tests into 6 specs to help with load balancing (the dev spec had over 60 tests).
To stabilize the tests I changed some specs to use test.serial.
Also see https://github.com/avajs/ava/issues/2947 for disabling worker threads
Todo

[ ] Update branch protection rules once PR is approved

For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[x] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[x] Update or add tests (if any source code was changed or added) 🧪
[x] Update or add documentation (if features were changed or added) 📝
[x] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

This is awesome @erezrokah!


Importing project in Intellij or eclipse.

Hi all,
Can anyone point me to the documentation on importing this project into any IDE. I am looking to debug the source code from the beginning.
Right now in my current project i am using Spark JobServer and the  UI is build by us.
I am thinking of integrating this into our current system. Will need some confidence on the code and how it works.

If you are using Livy, you should look at https://github.com/cloudera/livy
If you are working on the Hue UI, generally people just create an existing project based on the root, or just other text editors


Custom HTML Head not applied to /settings page

The custom HTML head is valid (e.g. customized font) for most pages but not the /settings page, while /settings/maintenance, /settings/audit, /settings/users, /settings/roles pages are using the customized font.

Hi @wxrl,
This is very much purposeful, and explained under the setting input:

This is to ensure that any troublesome custom code can always be reverted if needed.


Create a code editor and use it on the nanos Script's code property.

The editor was just copied from codemirror.net. Here's what it looks like in nanos:

Kinda sucks that it's pulling in so much code so I won't be heart broken if we don't wanna go forward with this but I'm finding it useful for writing scripts.

How big is it?

11k lines
On Tue, Apr 2, 2019 at 2:36 PM Kevin Glen Roy Greer <
<EMAIL_ADDRESS>wrote:

How big is it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/foam-framework/foam2/pull/2010#issuecomment-479138552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHuySODrOYxnvcMTnRNuQFauixRVsDiVks5vc6MZgaJpZM4cY08Z
.

<500k if we only load JS and Java support.


Forked relx scripts

I was wondering why some scripts from relx are forked and modified. Like https://github.com/bitwalker/exrm/commit/9d0e53878bf73b82e60121c986a5a4af1903a890 could have been merged into relx. Instead we both have different fixes for the same bug.
We should really be sharing scripts like this not only to dedup work we have to do but also so users of both relx and exrm benefit from improvements.

@tsloughter That commit in particular I'd gladly revert, since there's really no point in having a forked version of it. If I recall I did that one in a bit of a hurry, which is why I forked it instead of upstreaming it.
In general I absolutely agree, however I've had to keep the boot scripts forked because of the modifications I made to use an Elixir shell vs Erlang one, and to add in the hooks for conform. If there was a good way to add support for those things from within relx itself, I'd gladly work on that, but I'm not sure what that approach would be. If you have thoughts though, I'm open.
My intention with the forked scripts has been to merge changes from relx into the forked versions exrm uses, but it's difficult because the changes I've made since forking have been quite extensive, so fixes are often a translation instead of a straight merge, and I've fallen behind.
What I'd really like to do is have a modular boot script which permits users to set which shell (Elixir, Erlang, or something else) console/remote_console/attach will use, and provides a way to hook extensions into it (for things like conform). I'm decent at sh, but I'm lacking the know-how to write a script like that, assuming it's possible, at least not without some research.

Cool, no need to revert, just wanted to find out what happened.
And yes, I agree about the boot script. The more modular or easily replaced boot script is something I -- or @joedevivo :) -- have to figure out to integrate cuttlefish support into relx. So hopefully we'll have something to make that possible in the not to distant future.
Feel free to close this now, you weren't on irc so I couldn't just ping you there about it. Or leave it open as a place to discuss the modular boot script functionality.

@tsloughter I'll close this for now, but keep me posted on where you guys go with the boot script, if you want to talk through it sometime, just ping me on IRC or shoot me an email (it's in my GitHub profile).


Added synology

RE-CHECK!

Automated adapter checker
ioBroker.synology

[ ] :heavy_exclamation_mark: [E503] "iob_npm.done" found in repo! Please add it to .gitignore
[ ] :heavy_exclamation_mark: [E818] file iob_npm.done found in repository, but not found in .npmignore
[ ] :heavy_exclamation_mark: [E914] file iob_npm.done found in repository, but not found in .gitignore

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.synology

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.synology

:thumbsup: No errors found
Add comment "RE-CHECK!" to start check anew


How to export the scene generated by "prolificdreamer-scene.yaml" to a 3D mesh format?

We use the following command to convert the scene into a 3D mesh format:
<EMAIL_ADDRESS><EMAIL_ADDRESS>
CUDA_VISIBLE_DEVICES=9 python launch.py --config $config_path --export resume=$resume_path system.exporter_type=mesh-exporter system.exporter.context_type=cuda system.geometry.isosurface_threshold=10.
The generated MP4 video results are excellent, but when using the above command to convert them into 3D mesh, the results are as shown in the following image:

Is there an issue with the command for converting to 3D mesh mentioned above? How should we modify it to generate 3D mesh data corresponding to the video in MP4 format?

github提问的咋全是中国人lol

This issue is a duplicate of [#329], so I'm closing it to keep discussions centralized.


Request to add CODE OF CONDUCT

A Code of Conduct helps to establish clear guidelines for expected behavior within the community, ensuring that it remains inclusive, respectful, and productive for everyone involved.
Here are some reasons why adding a Code of Conduct is important:
(i) It promotes respectful and constructive interactions between contributors.
(ii) It ensures that all contributors feel welcomed and included, regardless of their background or experience.
(iii) It helps maintain a positive community environment, encouraging collaboration.
Contributor Covenant is widely adopted for open source projects.

Hello Admin and Mentor, I'm a WOB Contributor-2024. I would like to add Code of Conduct to this repository. Please assign this issue to me. Thank you!

Hii @neyhere07 Its been added already.


Coil extension readme followup

If you aren’t signed in to your Coil account, the extension still checks each page for the meta tag, but does not inject the Web Monetization API nor stream payments from Coil.

This is not technically true.
It actually always injects the document.monetization object when the extension is installed.

Development workflow
For quick iterating, see the repo root readme section for info on speeding up webpack by skipping type checking.

This link is stale

Good catches, can you update the README to reflect those?


Add an test case for rat

Description of feature
A test case with small test and truth files will be added into test case to show the usability of the pipeline for non-human species.

@3nrique0 I wonder if you have any update on this :)


erase_configuration and power cycling doesn't fully erase configuration

Occasionally I'll call erase_configuration, power cycle the ODrive, run a full recalibration and then when I set a very low velocity setpoint, the motor will spin at what seems like max speed.
I don't have any more of a concrete pattern than that, but it certainly seems to be erase_configuration-related.
FW v0.4.1 and HW v3.5

What settings are non-default through an erase_configuration and power cycle?
What settings do you set after the config erase?
What is the effect if you don't erase config but do a power cycle?
Try to read all the configs to see what is different in the different scenarios: just looking at a spinout or not isn't enough information.
Thanks.

I have also had similar problems in the past. However, ever since I have been using odrv0.reboot() directly after odrv0.erase_configuration() the has been no problems.
For the time being how about changing odrv0.save_configuration() and ``odrv0.erase_configuration()toodrv0.save_configuration_reboot()toodrv0.erase_configuration_reboot() or similar?

The issue @Capo01 is describing is tracked in #349 and now fixed on devel.
It's well possible that that's what @chachi was observing. Since there's been no new information on this for a while I'm gonna close this now. Feel free to open a new issue if this happens again after updating to the latest devel.


⚠ PrivateHD has degraded performance

In cde66d3, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 4016 ms

Resolved: PrivateHD performance has improved in a0c9f37.


Map gets stuck on mouse moving

On map moving some times its stuck. After release and click again its continue moving.
it happens randomly with a probability of 1/10
Console is empty.
Reproduced on demo page too.

Found reason. It happened after upgrade to mapbox-gl 0.44
https://github.com/alex3165/react-mapbox-gl/commit/4eaf7bb416e93ce6926d61829944030aeeeca060

Cannot reproduce.


Extension doesn't work in WSL

Extension version: 1.4.0
VS Code version: 1.53.2
OS Version: Microsoft Windows [Version 10.0.19042.746]

The extension doesn't appear to work under WSL as can be verified by the screenshot below:

I'm using the  "ligaturesLimited.selectionMode": "line" setting.

Can you see anything in the VSCode logs? There's a log called "Log (Extension Host)". My best guess is that the extension failed to load, since the arrow ligature in the comment is also not disabled.
WSL is definitely not an environment I spend much time in at all.

This is what is under Remote Extension Host (WSL runs as a remote instance):
[2021-02-12 20:41:23.892] [exthost] [info] extension host started
[2021-02-12 20:41:24.151] [exthost] [info] ExtensionService#_doActivateExtension ms-vscode.cpptools {"startup":false,"extensionId":{"value":"ms-vscode.cpptools","_lower":"ms-vscode.cpptools"},"activationEvent":"onLanguage:c"}
[2021-02-12 20:41:24.152] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/extensions/ms-vscode.cpptools-1.2.0/dist/main
[2021-02-12 20:41:24.349] [exthost] [info] ExtensionService#_doActivateExtension vscode.debug-auto-launch {"startup":true,"extensionId":{"value":"vscode.debug-auto-launch","_lower":"vscode.debug-auto-launch"},"activationEvent":"*"}
[2021-02-12 20:41:24.350] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/debug-auto-launch/dist/extension
[2021-02-12 20:41:24.371] [exthost] [info] ExtensionService#_doActivateExtension vscode.git {"startup":true,"extensionId":{"value":"vscode.github","_lower":"vscode.github"},"activationEvent":"*"}
[2021-02-12 20:41:24.372] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/git/dist/main
[2021-02-12 20:41:24.464] [exthost] [info] ExtensionService#_doActivateExtension kshetline.ligatures-limited {"startup":true,"extensionId":{"value":"kshetline.ligatures-limited","_lower":"kshetline.ligatures-limited"},"activationEvent":"*"}
[2021-02-12 20:41:24.464] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/extensions/kshetline.ligatures-limited-1.4.0/dist/extension
[2021-02-12 20:41:24.960] [exthost] [info] ExtensionService#_doActivateExtension vscode.github {"startup":true,"extensionId":{"value":"vscode.github","_lower":"vscode.github"},"activationEvent":"*"}
[2021-02-12 20:41:24.960] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/github/dist/extension.js
[2021-02-12 20:41:25.171] [exthost] [info] ExtensionService#_doActivateExtension vscode.configuration-editing {"startup":false,"extensionId":{"value":"vscode.configuration-editing","_lower":"vscode.configuration-editing"},"activationEvent":"onLanguage:json"}
[2021-02-12 20:41:25.171] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/configuration-editing/dist/configurationEditingMain
[2021-02-12 20:41:25.212] [exthost] [info] ExtensionService#_doActivateExtension vscode.extension-editing {"startup":false,"extensionId":{"value":"vscode.extension-editing","_lower":"vscode.extension-editing"},"activationEvent":"onLanguage:json"}
[2021-02-12 20:41:25.212] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/extension-editing/dist/extensionEditingMain
[2021-02-12 20:41:25.256] [exthost] [info] ExtensionService#_doActivateExtension vscode.json-language-features {"startup":false,"extensionId":{"value":"vscode.json-language-features","_lower":"vscode.json-language-features"},"activationEvent":"onLanguage:json"}
[2021-02-12 20:41:25.256] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/json-language-features/client/dist/node/jsonClientMain
[2021-02-12 20:41:25.426] [exthost] [info] ExtensionService#_doActivateExtension ms-vscode.js-debug {"startup":false,"extensionId":{"value":"ms-vscode.js-debug","_lower":"ms-vscode.js-debug"},"activationEvent":"onLanguage:json"}
[2021-02-12 20:41:25.441] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/ms-vscode.js-debug/src/extension.js
[2021-02-12 20:41:26.112] [exthost] [info] ExtensionService#_doActivateExtension vscode.npm {"startup":false,"extensionId":{"value":"vscode.npm","_lower":"vscode.npm"},"activationEvent":"onLanguage:json"}
[2021-02-12 20:41:26.113] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/npm/dist/npmMain
[2021-02-12 20:41:26.307] [exthost] [info] ExtensionService#_doActivateExtension vscode.emmet {"startup":false,"extensionId":{"value":"vscode.emmet","_lower":"vscode.emmet"},"activationEvent":"onStartupFinished"}
[2021-02-12 20:41:26.307] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/emmet/dist/node/emmetNodeMain
[2021-02-12 20:41:26.386] [exthost] [info] ExtensionService#_doActivateExtension vscode.merge-conflict {"startup":false,"extensionId":{"value":"vscode.merge-conflict","_lower":"vscode.merge-conflict"},"activationEvent":"onStartupFinished"}
[2021-02-12 20:41:26.387] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/merge-conflict/dist/mergeConflictMain
[2021-02-12 20:41:26.416] [exthost] [info] ExtensionService#_doActivateExtension vscode.testing-editor-contributions {"startup":false,"extensionId":{"value":"vscode.testing-editor-contributions","_lower":"vscode.testing-editor-contributions"},"activationEvent":"onStartupFinished"}
[2021-02-12 20:41:26.416] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/testing-editor-contributions/dist/extension.js
[2021-02-12 20:41:26.446] [exthost] [info] ExtensionService#_doActivateExtension eamodio.gitlens {"startup":false,"extensionId":{"value":"eamodio.gitlens","_lower":"eamodio.gitlens"},"activationEvent":"onStartupFinished"}
[2021-02-12 20:41:26.446] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/extensions/eamodio.gitlens-11.2.1/dist/gitlens
[2021-02-12 20:41:26.814] [exthost] [info] ExtensionService#_doActivateExtension GitHub.vscode-pull-request-github {"startup":false,"extensionId":{"value":"GitHub.vscode-pull-request-github","_lower":"github.vscode-pull-request-github"},"activationEvent":"onStartupFinished"}
[2021-02-12 20:41:26.814] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/extensions/github.vscode-pull-request-github-0.23.1/media/extension
[2021-02-12 20:41:26.989] [exthost] [info] eager extensions activated
[2021-02-12 20:41:27.159] [exthost] [info] ExtensionService#_doActivateExtension vscode.typescript-language-features {"startup":false,"extensionId":{"value":"vscode.typescript-language-features","_lower":"vscode.typescript-language-features"},"activationEvent":"onLanguage:jsonc"}
[2021-02-12 20:41:27.159] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/ariel/.vscode-server/bin/622cb03f7e070a9670c94bae1a45d78d7181fbd4/extensions/typescript-language-features/dist/extension

I don't see any sign of an error loading the extension in here. Perhaps it's a problem of WSL CSS handling or font support. Ligatures are broken into separate characters using the Unicode U+200A Hair Space character. Maybe in WSL that's not available?

Interesting. When highlighting a ligature with the "block" cursor enabled, every time it blinks, it reveals the actual character.

The block cursor behavior is totally expected. To show a character under the cursor in inverse mode, different CSS styling has to be applied to that character. The change in CSS style interrupts the font rendering for the ligature, so you see the individual isolated character instead. The Ligatures Limited extension takes advantage of the same font rendering phenomenon, but it uses an invisible CSS change rather than one so distracting as inverse colors.
Since it appears that the extension is running, not crashing, my best guess is that the CSS trick doesn't work in your WSL environment for some as-yet-unknown reason.
Is this WSL 2? My personal experience with WSL is very limited, only WSL 1, and never for anything other than the command line interface.

Yes, this is WSL2.

Which desktop environment are you using?

I tried to set up Gnome following these instructions: https://gist.github.com/Ta180m/e1471413f62e3ed94e72001d42e77e22
And just got stuck with endlessly repeating "Sleeping for 1 second to let systemd settle" when I tried to start WSL. The only suggested solution was to uninstall and reinstall Ubuntu, but that set me back to square one, and I now see no reason why, if I try the same stuff over again, I won't get stuck the same way. Did you perhaps use Xfce, as is also suggested?

I tried the Xfce approach, and just get errors too. "Cannot open /dev/tty0 (Input/output error)". Just trying to even see this issue happen is proving to be a nightmare.

I got Xfce running, but it's a really crude desktop, no web browser even available by default, so I doubt that's what you're using.

I'm trying to install Chromium, but get

Before I go further, please tell me how you set this up in the first place. As far as I can tell at this point, running an Ubuntu desktop under WSL 2 is no straight-forward thing.

You got confused. I don't use WSL2 with any desktop environment, as that is not a really supported use case (there are some hacks to run an X Server, but it doesn't work so well).
VS Code has a special feature that allows it to communicate with a WSL2 distro, all you need to do is install this extension (in Wndows):

then either run code . inside a linux directory, or click :

and select "Open folder in WSL...":

That issue I can recreate. It also happens with Remote SSL connections. Hopefully I can figure out why VSCode treats editors for remote connections differently, but there's unfortunately no "ah, hah! that's what's wrong!" apparent to me at the moment.

I've found a solution, but unfortunately badly screwed up at the same time, in a way that's beyond my ability to fix.
You'll have to delete the old extension, then re-install it, because I accidentally wiped out the entire history of the old extension. Old installations of Ligatures Limited won't know that the new Ligatures Limited is an update for the same extension. As far as VSCode is concerned, the updated version is an entirely different extension, one that merely happens to have the same author and same name.
I'm hoping someone at Microsoft can help me fix this, because it's all about the inner workings of the Visual Studio Market Place over which I have no control. For now, only you'll know there's an update.

The new version appears to be working correctly, I will close this issue. Thanks.


Add support for illumos target (0.6.x)

With support for an illumos target on its way into rust, the cfg guards for solaris should be updated to include illumos as well.

This will help untangle illumos from Solaris, as noted in #1263.

Updated to require minimum working libc version bump and reflect alphabetical sorting request made in #1295.

Let's focus on getting #1295 merged first, I don't know if we want to add support for a new OS to v0.6.

Hi @Thomasdezeeuw!
From our perspective in the illumos community, we're pretty keen to get 0.6 support done before 0.7 support.  Here is a small shell script that collects all of the requested versions of mio from https://crates.io:
#!/bin/bash

set -o errexit
set -o pipefail

crate="mio"
url="https://crates.io/api/v1/crates/$crate/reverse_dependencies?page="

n=1
while :; do
	if ! res=$(curl -s -f "${url}${n}"); then
		printf 'ERROR: request page %d failed\n' "$n" >&2
		exit 1
	fi

	if ! len=$(jq '.versions | length' <<< "$res"); then
		exit 1
	fi

	if (( len == 0 )); then
		exit 0
	fi

	for (( i = 0; i < len; i++ )); do
		if ! name=$(jq -r ".versions[$i].crate" <<< "$res") ||
		    ! req=$(jq -r ".dependencies[$i].req" <<< "$res"); then
			exit 1
		fi

		# printf '%-24s %s\n' "$req" "$name"
		printf '%s\n' "$req"
	done

	(( n++ ))
done

Based on the output of this script, it's pretty plain that the 0.6 branch is by far and away the most common in the ecosystem:
$ ./assess.sh > /tmp/mio_versions.txt
$ wc -l /tmp/mio_versions.txt
259 /tmp/mio_versions.txt
$ grep -c '^\^0.6' /tmp/mio_versions.txt
215

215 of the 259 crates currently depend on a 0.6 version of mio, perhaps most prominently the flagship module tokio!  While we're keen to help out with 0.7, as evidenced by Patrick's other PR, we'd like to be able to build and run all that existing software on our platform today.
Please let us know how we can help to move this forward!

Ok we've decided to accept this for v0.6 once rustc is updated (https://github.com/rust-lang/rust/pull/71145).

Please let us know how we can help to move this forward!

For the long term we need two things:

CI: preferably where we can run cargo test, but being able to run cargo check is sufficient. Seeing how its a new target I don't expect tier 1 support in Rustc so we can't (easily) cross check from another OS at the moment.
Maintainer(s): as far as I know no-one on the Mio team currently has access to a Illumos machine, so if we can ping someone for Illumos specific bug reports that would be great.

Ok we've decided to accept this for v0.6 once rustc is updated (rust-lang/rust#71145).

Thank you very much!  Deeply appreciated.

For the long term we need two things:
* CI: preferably where we can run cargo test, but being able to run cargo check is sufficient. Seeing how its a new target I don't expect tier 1 support in Rustc so we can't (easily) cross check from another OS at the moment.

We're definitely conscious of not being supported by the big CI/CD providers, and can look into what we can do for mio.

* Maintainer(s): as far as I know no-one on the Mio team currently has access to a Illumos machine, so if we can ping someone for Illumos specific bug reports that would be great.

I have created a team, @illumos/rust, to house a list of people you can ask about issues!  I've added @pfmooney and myself to start off.

Looking closer at this, I'm not sure if people outside an organisation can see a particular team.  If you want to create a GitHub team in the tokio-rs organisation to track the list of maintainers you can ping for illumos issues, Patrick and I would be happy to be listed there, too.

@jclulow @pfmooney I've created @tokio-rs/illumos and invited both of you. I couldn't make it a sub-team of Mio so you might get people pinging that team on other tokio repos.
As for this pr, it LGTM, but the CI is having some problems (I'm trying to fix it in #1297).

As for this pr, it LGTM, but the CI is having some problems (I'm trying to fix it in #1297).

Is there something I should do to re-trigger the CI build?

@pfmooney If you can merge master branch in to this, the FreeBSD check was just updated and should not hang anymore.

Edit: Nevermind this is merging into 0.6 branch. May have to do the same update there. I will get that change in.

I'm happy to rebase my bits once the 0.6.x branch is in the desired state.

@pfmooney You should be okay to rebase now. Thanks for the patience on this!

@pfmooney You should be okay to rebase now. Thanks for the patience on this!

Thanks for getting the 0.6.x CI situation squared away!

@pfmooney @jclulow thanks!


Adding an initial protobuf v3 template

Signed-off-by: Frederick F. Kautz IV<EMAIL_ADDRESS>An initial PR to initiate discussion on what should go into a network service mesh.
A few notes:

We should determine whether grpc's error mechanism will be sufficient, or whether we need to separate mesh errors from transport errors by adding in error handling explicitly into the proto definitions.

We will need to work out how expressive we want service discovery queries to be.

grpc is bi-directional. This initial commit deals only with the mesh itself. We should also set up rpc services that may be present on the service requester and service provider.

Errors are set as a tuple: (code, errormsg)
We are able to set our own status codes and error messages. These codes do not appear to be enumerable in the specification itself, so we will need to be consistent with our implementations.
There are a default set of status codes you can read about here:
https://github.com/grpc/grpc-go/blob/master/codes/codes.go
My recommendation is we define our own set of error codes to supplement the default codes. We may need to add machine readable error messages if more context is required.


integration with redux-saga

Hello,
i have redux-actions and redux-saga in one project
I tried to find way get string constant of action for saga subscription.
Could you help me?

We don't support redux-saga, which is entirely different middleware.
I'm not sure what the use case is, but I think I'd advise standardizing on one middleware in your project; I've never a project that use both simultaneously, they have completely different philosophies and styles.

@yangmillstheory hi, your post confused me. What are those philosophies and styles?
IMO you can use sagas as side effect handlers and redux-actions to do other boilerplate stuff

I agree with @satispunk . They may have different philosophies, but these are philosophies that you would normally use together in your application. One that handles side-effect and one that handles state.

For those who are still looking for an answer to this issue, basically you could call .toStrong() on your action creator and it will return the action type.
For example:
import { createAction } from 'redux-actions';

const getUserRequest = createAction('GET_USER_REQUEST');

// in your saga
function* getUserSaga({ payload }) {
// side-effect
}

takeLatest(getUserRequest.toString(), getUserSaga);


Large loops cause stack overflow

Current implementation is recursive, failing for large loops.
It may not be possible with the SDK to implement in a different way...

Fixed in 1.0.1


[rust] give node downcasts the proper lifetimes

This PR is sort of the opposite of #1625: here, the code currently returns structures that are bound to the lifetime of &self.  But that's not correct; the structures are actually bound to 'pr, the lifetime of the parse result.  There's no reason that they should be bound to the lifetime of &self; doing so makes some kinds of programs difficult to write.
e.g. The added test gets errors like:
error[E0597]: `path` does not live long enough
   --> ruby-prism/src/lib.rs:804:38
    |
802 |         impl<'pr> Extract<'pr> {
    |              --- lifetime `'pr` defined here
803 |             fn push_scope(&mut self, path: Node<'pr>) {
    |                                      ---- binding `path` declared here
804 |                 if let Some(cread) = path.as_constant_read_node() {
    |                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |                                      |
    |                                      borrowed value does not live long enough
    |                                      argument requires that `path` is borrowed for `'pr`
...
811 |             }
    |              - `path` dropped here while still borrowed

cc @ngroman

Sorry, had to fight with clippy a little bit, but I think we got there in the end.


chore: WIP - replace nps with npm scripts

PR Checklist

[x] Addresses an existing open issue: fixes #5126
[x] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview
Work in progress draft branch to keep #5042 cleaner and more focused

Yes please

Related: https://github.com/mochajs/mocha/issues/5159
I would suggest moving the version scripts to GitHub Actions along with that.

Waiting for #5177 before I continue on this one, as they overlap

coverage: 94.523%. remained the same
when pulling f0243d0c308be8e95f04910b7edf8c8f5282af14 on voxpelli/issue5126
into a8f8c8b0f741094a61ca900353aff01b14f67c6a on main.

Created a related feature request for VSCode: https://github.com/microsoft/vscode/issues/224581 Requesting syntax highlighting of npm scripts


Cuda 10.2 mxnet installation error mxnet-cu102mkl

Hi, I have followed this post #17678 , I want to install the mxnet-cu102mkl package by pip install mxnet-cu102mkl==1.6.0 on the windows,
but it didn't work with ERROR: Could not find a version that satisfies the requirement  mxnet-cu102mkl==1.6.0 (from versions: none) ERROR: No matching distribution found for mxnet-cu102mkl==1.6.0
I changed other package resourse but it always can't work, does this package only have nightly build version now? How should I do?

We do have that for linux. https://pypi.org/project/mxnet-cu102mkl/#files
What operating system are you using?

We do have that for linux. https://pypi.org/project/mxnet-cu102mkl/#files
What operating system are you using?

I am using python compiler of anaconda on the windows10, will you have a windows version soon?

The windows version was too large in size so we only have it available in our own repo https://dist.mxnet.io/python. You can install with
pip install mxnet-cu102mkl==1.6.0 -f https://dist.mxnet.io/python

Let's keep this issue open until the installation doc is updated.

The windows version was too large in size so we only have it available in our own repo https://dist.mxnet.io/python. You can install with
pip install mxnet-cu102mkl==1.6.0 -f https://dist.mxnet.io/python

Did the option -f equal to --find ？ I can't find this option in the user guide documentation of pip..

We do have that for linux. https://pypi.org/project/mxnet-cu102mkl/#files
What operating system are you using?

@szha What does the post0 suffix signify?

We do have that for linux. https://pypi.org/project/mxnet-cu102mkl/#files

What operating system are you using?

@szha What does the post0 suffix signify?

It's just that after the release the pip package document needs to be patched and hence a new version. Since pypi versions are immutable, such patch happens on a post-release version, hence the suffix.

It's just that after the release the pip package document needs to be patched and hence a new version. Since pypi versions are immutable, such patch happens on a post-release version, hence the suffix.

Is there a reason a patch does not get its own patch version?

@ianhellstrom there's no code change other than a readme update. I think this is a proper use of the post-release version numbers according to PEP440

Thanks a lot, it works well. Could you please update the command on "https://mxnet.apache.org/versions/1.6/get_started?platform=windows&language=python&processor=gpu&environ=pip&" either? People may get frustrated when they run the command from offical website.


Update the tutorial notebooks in the fundamentals directory

This PR aims to address Issue #193, updating all notebooks apart from the visualisations notebook.
To echo my comments in the issue referenced above, I do not believe that much of the contents of these notebooks requires updating, as it is still relevant to the inner workings of NanoVer and is useful to those wishing to develop their own applications using NanoVer. However, I have edited the wording in some parts of the notebooks to attempt to make things clearer, and have (where possible) added text to link the concepts being explained to the current classes used by NanoVer to run iMD (i.e. the OmniRunner).
The only things that I cannot update on my own is the "GRPC Services" and "TLDR" sections of the servers.ipynb notebook (as I don't know enough about the inner workings of the current version of the iMD server to edit the text around the new figure). @Ragzouken, would you be able to do this? The rest of the notebook probably doesn't need altering further. We can talk about this together if you like.

Your changes seem good to me. I have clarified some things in the servers notebook, let me know if it all sounds okay.

Looks good to me, as long as you're happy with those descriptions! Thanks for getting on this quickly 😄


Update README.md

📝 Description
Adding space before the line-break backslashes in the installation command such that a parsing error does not occur in the shell.
✔ How to Test
Run command in shell to make sure it parses correctly.

Feel free to merge it by Squash and merge after 2 approvals from the team


Edge border on popover body cut off

This is also causing the x scroll bar to come alive

(But I should add, other than this, everything else looks fantastic! Pleasant to find a very solid popover component)

Thanks for reporting this! Seems like it will be easy to reproduce. Hopefully easy to solve. Unsure when a fix will land though.

👍🏻
On Thu, Nov 19, 2015 at 5:51 AM Jason Rose-Kuhrt<EMAIL_ADDRESS>wrote:

Thanks for reporting this! Seems like it will be easy to reproduce.
Hopefully easy to solve. Unsure when a fix will land though.
—
Reply to this email directly or view it on GitHub
https://github.com/littlebits/react-popover/issues/49#issuecomment-158061721
.

@KyleAMathews Can you give me the minimal repro case? I was unable to reproduce via simply putting a border on the .popover-body. Screenshot:

No feedback so closing for now.


Pao operator

This installed a very old version of the operator.  I updated it to use the newer pao operator.

I'll just recreate this PR


GuestAdditions seems to be installed (5.0.14) correctly, but not running.

Every time I run vagrant up|reload I get the following error.
GuestAdditions seems to be installed (5.0.14) correctly, but not running.

stdin: is not a tty
Usage: service < option > | --status-all | [ service_name [ command | --full-restart ] ]
bash: line 4: vboxadd: command not found

stdin: is not a tty
Removing existing VirtualBox DKMS kernel modules ...done.
Removing existing VirtualBox non-DKMS kernel modules ...done.
Building the VirtualBox Guest Additions kernel modules ...done.
Doing non-kernel setup of the Guest Additions ...done.
You should restart your guest to make sure the new modules are actually used

Restarting VM to apply changes...

If I restart as it says to do I get the same as above.
When the vm has started:
$> vagrant vbguest --status
GuestAdditions 5.0.14 running --- OK.

Any idea what's happening here?
Host: OS X 10.11.3
Guest: Ubuntu 15.04
Vagrant version: 1.8.1
Virtualbox version: 5.0.14

:+1:

Seeing the same issue with the same specs as @atadams. The only thing I can add is my error message is slightly different some times (other times it is the same as reported):
==> default: Machine booted and ready! Got different reports about installed GuestAdditions version: Virtualbox on your host claims:   4.3.36 VBoxService inside the vm claims: 5.0.14 Going on, assuming VBoxService is correct... GuestAdditions seems to be installed (5.0.14) correctly, but not running. Got different reports about installed GuestAdditions version: Virtualbox on your host claims:   4.3.36 VBoxService inside the vm claims: 5.0.14 Going on, assuming VBoxService is correct...

No change with the 5.0.16 update.

@ChrisWiegman ditto
as before I add
  config.vbguest.auto_update = false

to my Vagrantfile, but comment out whenever I upgrade VirtualBox, then uncomment between upgrades to avoid this annoying behaviour.

also anyone can run
vagrant vbguest
and get
GuestAdditions 5.0.16 running --- OK.
between Virtualbox updgrades.
Of course this seems to mean that vbguest contains the correct logic to check for the current version of GuestAdditions, but the check process during auto_update is faulty.
Not sure why it wouldn't get fixed. This problem has been around for a couple of years now...

The same here (Mac OS 10.11.3 - VBox 5.0.16)
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.2.0
VBoxService inside the vm claims: 5.0.16
Going on, assuming VBoxService is correct...
GuestAdditions seems to be installed (5.0.16) correctly, but not running.
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.2.0
VBoxService inside the vm claims: 5.0.16
Going on, assuming VBoxService is correct...
stdin: is not a tty
Usage: service < option > | --status-all | [ service_name [ command | --full-restart ] ]
bash: line 3: vboxadd: command not found
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.2.0
VBoxService inside the vm claims: 5.0.16
Going on, assuming VBoxService is correct...
stdin: is not a tty

:+1:

@alejandrofcarrera I don't think it's actually installing an old version, just mis-reporting
after vagrant up, do vagrant vbguest to see

@HenrikBechmann Yes, you are right, it is not going to install that version.
BTW, i updated additions from 5.0.14 to 5.0.16 and it did not updated it, it reported that message.
My solution was to destroy the machine and re up to install 5.0.16 from beginning (bad move with each update) and after that i got the same message with the next "vagrant up".
Right now i have auto_update to false like you.

same here

I'm experiencing same problem for quite a long time with different vagrant and vbox versions. For almost every vagrant up, I can see this behaviour. I think, it got worse after moving my boxes from SSD to HDD. It seems at least in my case it is just a matter of timing. SSH of the box gets available, but vbguest check for running virtualbox service happens to early while box is still booting.
Does this make sense?
If yes, it would maybe be convenient to introduce a customizable (via Vagrantfile) delay?

Is it safe to assume you all have the vagrant-vbguest plugin installed??
$ vagrant plugin list
vagrant-share (1.1.5, system)
vagrant-triggers (0.5.3)
vagrant-vbguest (0.11.0) <---------------------- This one.

That is required for automatic updates of the guest OS services.

Yup
On Wed, May 4, 2016 at 4:25 AM, Richard Scott<EMAIL_ADDRESS>wrote:

Is it safe to assume you all have the vagrant-vbguest plugin installed??
$ vagrant plugin list
vagrant-share (1.1.5, system)
vagrant-triggers (0.5.3)
vagrant-vbguest (0.11.0) <---------------------- This one.
That is required for automatic updates of the guest OS services.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/dotless-de/vagrant-vbguest/issues/202#issuecomment-216782050

--
internetcommons.ca

Having the same problem with this. Added
if Vagrant.has_plugin?("vagrant-vbguest")
    config.vbguest.auto_update = false
end

which at least suppresses the error.

#191 appears to fix this error:
Usage: service < option > | --status-all | [ service_name [ command | --full-restart ] ]
bash: line 4: vboxadd: command not found

I applied the change from that issue manually to this file, after which the error disappeared:
~/.vagrant.d/gems/gems/vagrant-vbguest-0.11.0/lib/vagrant-vbguest/installers/linux.rb

I'm pretty sure that modifying files manually like that will void any warranty.

Seeing this in VB 5.1.2
==> default: Machine booted and ready!
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.3.36
VBoxService inside the vm claims: 5.1.2
Going on, assuming VBoxService is correct...
[default] GuestAdditions 5.1.2 running --- OK.
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.3.36
VBoxService inside the vm claims: 5.1.2
Going on, assuming VBoxService is correct...
==> default: Checking for guest additions in VM...

Result of vbguest version
neilg@[~/vdev]:(develop) vagrant vbguest
[default] GuestAdditions 5.1.2 running --- OK.

May be linked to https://github.com/dotless-de/vagrant-vbguest/issues/224#issuecomment-275181898

I copied VBoxGuestAdditions.iso from VirtualBox installation directory to local directory and added below line in VagrantFile and it worked
config.vbguest.iso_path = "VBoxGuestAdditions.iso"
Not sure why this happens. But in case this helps

I had a whole lot of trouble with this issue before, but I've recently tried again and it "just works" now.
VirtualBox 5.1.18
Vagrant 1.9.3
Using ubuntu/trusty64 dated version 20170313.0.7 on an Ubuntu 16.04 host.
I did get a "do not match" after initially provisioning the VM, but it appears that the vbguest plugin correctly installed the updated version automatically because I have received no further errors or problems.

Same here. Mis-report during auto update:
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.3.28
VBoxService inside the vm claims: 5.1.22

$ vagrant vbguest --status
[default] GuestAdditions 5.1.22 running --- OK.

Same as @vanquang9387 on OSX.

after vagrant up
do vagrant vbguest --do install
then vagrant reload
to check vagrant vbguest --status should be ok

@javinc - When running vagrant vbguest --do install I get:
Cancelling installation.
An error occurred during installation of VirtualBox Guest Additions 5.1.24. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.
vboxadd: unrecognized service
vboxadd-service: unrecognized service

In my case after seemingly successfully installation of vbguest, the automatic update of the guest additions on vm box didn't work. So the way I fixed it is I ssh into the guest box which is an arch linux machine and upgraded all packages with pacman -Syu command.

In my case this problem seems to be caused by a bug in VirtualBox. The problem was with virtualbox-5.2_5.2.8-121009~Ubuntu~zesty_amd64.deb and disappeared after switching to virtualbox-5.2_5.2.12-122591~Ubuntu~zesty_amd64.deb  - See vagrant-packet-builder issue #9 for details.


Create intake logs ETL

Does it make sense to run CI for 3 versions of Python in the usage metrics repo? Seems like it's just for internal use and 3.10 would be totally fine.


Missing tags

rustc_version is currently at 0.2.3, but the last tagged release was 0.1.7. Not having tags makes it hard to see what's changed, as there's also no changelog. Fortunately the project is fairly small, so examining all commits is doable. But I think it'd be helpful if all the releases were tagged.

I've backfilled all of the tags since 0.2.0.

I've backfilled all of the tags since 0.2.0.


HTTP failures browsing site with iPhone/IOS

Current behavior
Http failure with iPhone/IOS is being reported in our error logs. (using stacktrace)

ERROR:
Http failure response for /assets/i18n/en.json: 0 Unknown Error

AGENT:
Mozilla/5.0 (iPhone; CPU iPhone OS 13_2_2 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.0.3 Mobile/15E148 Safari/604.1

Expected behavior
No Http errors reported
How do you think that we should fix this?
Perhaps setup the same environment and use same user agent to re-create the issue.
Minimal reproduction of the problem with instructions
We have had this issue since converting to Angular 7 from AngularJS.   Only happens when site is browsed using IPhone/IOS and is possibly intermittent.  We have error reporting in place and the above is what gets reported.   Why the failure I do not know.
Environment

  "dependencies": {
    "@angular/animations": "~8.2.11",
    "@angular/cdk": "~8.2.3",
    "@angular/common": "~8.2.11",
    "@angular/compiler": "~8.2.11",
    "@angular/core": "~8.2.11",
    "@angular/forms": "~8.2.11",
    "@angular/material": "^8.2.3",
    "@angular/platform-browser": "~8.2.11",
    "@angular/platform-browser-dynamic": "~8.2.11",
    "@angular/router": "~8.2.11",
    "@ng-bootstrap/ng-bootstrap": "^5.1.1",
    "@ngx-translate/core": "^11.0.1",
    "@ngx-translate/http-loader": "^4.0.0",
    "angular-gtag": "^1.0.3",
    "bootstrap": "^4.3.1",
    "core-js": "^2.5.4",
    "express": "^4.17.1",
    "hammerjs": "^2.0.8",
    "jquery": "^3.4.1",
    "moment": "^2.24.0",
    "ng2-truncate": "^1.3.17",
    "ngx-bootstrap": "^3.2.0",
    "ngx-moment": "^3.4.0",
    "path": "^0.12.7",
    "popper.js": "^1.16.0",
    "pusher-js": "^5.0.2",
    "rxjs": "~6.5.3",
    "stacktrace-js": "^2.0.1",
    "tslib": "^1.9.0",
    "zone.js": "~0.9.1"
  },

Browser:
Mozilla/5.0 (iPhone; CPU iPhone OS 13_2_2 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/13.0.3 Mobile/15E148 Safari/604.1

For Tooling issues:
- Node version: 10.16.0
- Platform:  Heroku Platform (Linux)

I have the same issue

I see the same thing, we're getting 100k+ reports every day with this error.
All from Safari and Mobile Safari.
Angular 11.2.0
@ngx-translate/core 13.0.0
@ngx-translate/http-loader 6.0.0

Were you able to resolve that issue?

I have the same issue. Any idea what to do?

Same:
@ngx-translate/core: 15.0.0
@ngx-translate/http-loader: 8.0.0


World Record Badges not correlated

I did two Zwift races this weekend, and the segment results are at most Cat5 Badged, but the Analysis Page shows Cat4 on each of them.
Sunday
Saturday
I assume this is just matching to some Coggan Tables, but I'm not typically sure how the overall would be higher than the parts.

Hi @ph4r probably it's just the time domain and spatial bounds of the segments vs the peak power table which is unbounded spatially.  I just goes through the power stream looking for the highest power averages of each time period.  For example, while it's possible that your peak 1 minute power happened to line up exactly with a 1 minute segment it is unlikely.  However there may be a segment inside that 1 minute period that takes say 45 seconds.  And like you suspected the rank is based on w/kg / time.  So the same power (or even slightly higher) for 45 seconds of that selection will not rank as high as the same power held for 1 minute (or at least evaluated for a minute).  I hope that makes some sense.
TL;DR;  The rank badges for segments are bound to a fixed time domain.  The time period for the Peaks table and the time taken for each segment in the segments list.
I hope that helps.  Also, if you haven't already, click on the rank badge.  There is a little widget that lets you see more details.

Hi @ph4r Let me know if my reply resolves this for you so I can close this issue or if you wanted to discuss further.


How to use $t on a page in the 'export default' object?

Last time I used nuxt it was during version 1 and I remember I had to add an import statement to use this.$t().
I have a component displaying a table with a lot of keywords. I'm assuming every time the component render it'll need to grab the translation again if I use it in the template part, so I'd like to grab the translation in the script part of the component.
I checked in the documentation but couldn't find what I was looking for.
data: () => {
  let word = this.$t('keyword');
  return { word }
}

I feel like this is dumb but I can't find the answer, thank you!

I'm not sure it's possible. You would need to have access to the vue-i18n instance that has all the translations loaded. You can import 'vue' and look into its prototype and find the I18n object there but not sure it will have translations loaded.
But in any case that's not a proper solution as you won't get reactivity this way. If user would change the language on your page (dynamically) then translations wouldn't update.
So you can maybe hack your way in but I would not recommend it.

Oh wait... You want to do that from script part of the component and not external script (I've just noticed your code snippet)?
Well, the problem in your snippet is that you are using an arrow function. The overrides this so don't do it. Generally avoid it in Vue components. It should work then.
But doing it that way is probably not any more efficient than using I18n directly in the template. The content will still be reactive and translations will be re-resolved on component render.

I use this: Vue.prototype.$nuxt.$options.i18n.t('keyword')

I think my answer addresses initial problem so closing.


Data Integration - Identity Lifecycle (HR -> IT) with Meltano

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2584
Originally created by @vischous on 2021-02-16 01:13:45

What
Vision - All organizations should have a fully automated HR -> IT process. Manually doing the same thing over and over again is insanity.
Identity Lifecycle management in a nutshell is how an organization creates, updates, and removes employees in their systems. Generally when an organization gets to ~50 users they'll have a HR (Human Resources) system (Workday, BambooHR, Rippling, ADP WorkforceNow, etc etc), and they'll also have some kind of IT system(s) for identity's (Google Workspace, Azure AD, Active Directory) some may even go as far as (Okta, Onelogin, Jumpcloud, etc). Enterprises generally have multiple IDPs that are integrated in different fashions.
Today most SMBs 50-500 employees have a manual process for creating IT accounts. Their HR team contacts their IT team via email and the account's get created, updated or deleted. Manual processes lead to mistakes, check out https://www.onelogin.com/press-center/press-releases/new-research-from-onelogin-finds-over-50-of-ex-employees-still-have-access-to-corporate-applications.
Businesses that have this automated have multiple commercial options, but generally I believe Meltano could be very attractive to businesses that are maintaining their own scripts.
After employees are in an organizations IDP they can generally take advantage of SCIM for applications that support SCIM. Applications that doesn't support SCIM are another interesting avenue but a bit outside the scope of this issue.
Here's a short 3 minute video of me going over BambooHR -> Active Directory and should give the general gist of the idea. https://www.youtube.com/watch?v=9kO4St2TRk0 .
How
Identity Lifecycle is pretty much an ETL job. We could Pull data from BambooHR (tap), push it to Active Directory, or Google(target). I understand that Meltano, Fivetran, Stick, etc are designed for loading data from systems to populate a Datawarehouse. I think the tooling overlaps enough that doing a data integration for identity is an interesting overlap.
To start I'll look at using a datawarehouse as an intermediate DB. Ideally all transformations will happen via DBT, and the google-admin-target can pull from one of the materializations.

bamboohr-tap, google-admin-tap, postgres-target
postgres-tap, google-admin-target

There's more work that's needed for Matching data between the two systems, and making the DBT transformations easy to work with but this seems like a good start.
Does anyone have any pointers or examples of people doing this with Data warehouses? I have a bunch of experience doing these integrations, but very little experience on the "Big data" side.
My next steps right now:

[x] Run through Python tutorial - https://docs.python.org/3/tutorial/index.html
[x] Connect Meltano to Google Analytics
[x] Run through DBT fundamentals https://courses.getdbt.com/courses/take/fundamentals/lessons
[x] Setup a tap / target with a DBT transformation in Meltano (can just follow the main website for this with gitlab / postgres)
[x] Map out what an integration would look like to Google Workspace (AdminSDK) with DBT doing the transformation. High level (Write a google-admin tap?, Write a target? etc) Write next steps

@DouweM here's the write up, this is basically my business model but I firmly believe everyone should automate their Identity Lifecycle. I believe open sourcing this will lead more organizations automating their processes and will lead to a community of people working together to remove the old manual HR to IT processes.

View 13 previous comments from the original issue on GitLab


Adding Icon on drawer

Conflicts:
mobile/lib/base/constants.dart

Not needed on main branch as flutter code is in other branch now.


Large performance hit if using custom actionEventMapper

Describe what happened
We noticed that our applications felt sluggish when using DD RUM with user interaction tracking enabled. We then noticed if we disabled our custom actionEventMapper the sluggishness went away. Tracing the code lead me here.
This deepClone is adding an additional 50 - 250ms to every action event! 💀 That's a huge penalty for using a custom mapper.

Steps to reproduce the issue:

Enable user interaction tracking and add a custom actionEventMapper
Track time to complete simple onPress handler

Compare with implementation where custom actionEventMapper configured.
Describe what you expected:
I expect no additional performance impact when adding a custom actionEventMapper outside of what is added by the mapper itself.

Hi @justinadkins, thanks for reaching out!
We can reproduce some slowness in the actionEventMapper, I apologize for this.
I've already identified ways to make the deepClone function faster, I'm investigating into how far we can go and will make a pull request soon.
I will let you know once the PR is opened so you can patch the SDK on your end without having to wait for the release.
Thank you again for reporting this.

Hi @justinadkins,
This has been fixed in release 1.8.3.
The release will be promoted to latest in a few weeks and is already safe to use as the changes in this one are quite minimal :)
I'm closing the issue, feel free to reopen if you experience any further performance issue with this version!


file upload failure

Describe the Bug
when i want to paste or upload an image i get this error

Steps to Reproduce
create page paste image or upload image
Expected Behaviour
image should upload
Screenshots or Additional Context

Browser Details
No response
Exact BookStack Version
BookStack v23.12

just for added context
this is the handler mapping on the *.php fast cgi

I also tried with All verbs

Hi @jonty-esterhuizen,
I'm afraid I'm not too familiar with IIS, so can't provide much insight.
The error message if fairly explicit though.
Are you sure there are not other "handlers" or things acting on the request?
Especially since your screenshot reflects a "request path" of "*.php", since these requests are not direct to PHP files (at least in terms of request path) I'm wonder if these requests are picked up by another handler first.
I'll re-categorize this as a support thread rather than a bug since there's nothing to indicate this is a bug in the core project here.

Since there's been no further follow-up I'll close this off.


validators called twice

from @dmfigol on gitter:
I am failing to understand why validator is applied several times on the complex field:
import json
from typing import List, Dict, Any

from pydantic import BaseModel, ValidationError, validator

class DemoModel(BaseModel):
    data: Dict[str, Any] = {}

    @validator('data', pre=True, whole=True)
    def json_decode(cls, v):
        print(f'value is {v!r}, type: {type(v)}')
        if isinstance(v, str):
            try:
                return json.loads(v)
            except ValueError:
                pass
        return v

print(DemoModel(data='{"foo": {"bar": 123}}'))

# > value is '{"foo": {"bar": 123}}', type: <class 'str'>
# > value is 'foo', type: <class 'str'>
# > DemoModel data={'foo': {'bar': 123}}

I would appreciate if you could explain :) because of this I spent a lot of time debugging my code and after I found the problem, I still don't understand why it is working this way :(

Sorry you had trouble. I can understand that this was confusing.
First of all, for parsing json you would be much better off using the Json meta type:
from typing import Dict, Any

from pydantic import BaseModel, Json

class DemoModel(BaseModel):
    data: Json[Dict[str, Any]] = {}

print(DemoModel(data='{"foo": {"bar": 123}}'))

This is documented here unfortunately I broke that section of the docs when last deploying so it wouldn't have made much sense, but it's now fixed
On the particular problem you're having, the reason is that the validator is being called twice is that it's also being called when validating the dict keys:
https://github.com/samuelcolvin/pydantic/blob/c7c15bc41533551c3ad2a3655c06cc12ca68d6e9/pydantic/fields.py#L241
I guess this should be fixed so there's another keyword for validators key which means they're called on values, and whole validators shouldn't be called on keys or values.

@samuelcolvin thanks for the hint with JSON type and fixing docs!

I guess this should be fixed so there's another keyword for validators key which means they're called on values, and whole validators shouldn't be called on keys or values.
Yeah, I'd expect that the whole validator is applied once and is not invoked later when we are checking the type defined in the annotation.

Your check on duplicate validators breaks validators in jupyter notebooks if a cell is executed more than once.
from pydantic import BaseModel, validator
import yaml
from ipaddress import ip_address

class Config(BaseModel):
    ip: str
    name: str
    port: int

    @validator("ip")
    def check_ip(cls, value):
        return ip_address(value)

with open("config") as f:
    d = yaml.safe_load(f)
    c = Config(**d)
    print(c)

Returns
Config ip=IPv4Address('<IP_ADDRESS>') name='asdf' port=2392
when the cell is called first and raises

ConfigError                               Traceback (most recent call last)
 in ()
3 from ipaddress import ip_address
4
----> 5 class Config(BaseModel):
6     ip: str
7     name: str
 in Config()
8     port: int
9
---> 10     @validator("ip")
11     def check_ip(cls, value):
12         return ip_address(value)
~/anaconda3/lib/python3.7/site-packages/pydantic/main.py in dec(f)
449         ref = f.module + '.' + f.qualname
450         if ref in _FUNCS:
--> 451             raise ConfigError(f'duplicate validator function "{ref}"')
452         _FUNCS.add(ref)
453         f_cls = classmethod(f)
ConfigError: duplicate validator function "main.Config.check_ip"
The second time the same cell is called.

I was somewhat surprised by this behaviour and when I checked the code, I didn't really get why this exception is raised since we're dealing with a set here, so behaviour shouldn't change.
I would suggest to replace this exception with a logger message, warning level, which won't mess with people's code and is well supported by jupyter.

@amogorkon this is not related to the issue being discussed here. Would be great if you could create a new issue and delete your comment.

Sorry, didn't notice your reply here. Opened a new issue and deleted comment as requested.


⚠ Bofrost.lu has degraded performance

In f372029, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1401 ms

Resolved: Bofrost.lu performance has improved in 7849049 after 26 minutes.


Nuget package for version 7.X not being published

No nuget  7.X version is being published

@letstrodrigues Can you elaborate? The NuGet packages are at https://www.nuget.org/packages/NEST

Thank you. I was expecting on the same nuget.


usually crash at [ZXCapture dealloc]

(void)dealloc {
if (_lastScannedImage) {
CGImageRelease(_lastScannedImage); // crash here
}
if (_session && _session.inputs) {
for (AVCaptureInput *input in _session.inputs) {
[_session removeInput:input];
}
}
if (_session && _session.outputs) {
for (AVCaptureOutput *output in _session.outputs) {
[_session removeOutput:output];
}
}
} // or crash here

malloc: *** error for object 0x1666cc90: pointer being freed was not allocated
*** set a breakpoint in malloc_error_break to debug

I am facing the same problem and could not found any help yet. Any Body here please help. We need your urgent attention. I am stuck at this point. Please help.

Same here but commenting dealloc does not solve the problem. I guess it's the memory management of the last scanned image which is not correct. I'm investigating...

OKAY ! I found a fix !
You should remove the ZXCapture.layer from your layer hierarchy in your dealloc.
This is how my dealloc now looks like :
- (void)dealloc
{
    [self.capture.layer removeFromSuperlayer];
    self.capture.delegate = nil;
    [self.capture stop];
}

@nverinaud Thanks for the fix! Recently my app started crashing but your fix has helped.


Publish javadoc to GH pages

PR summary
Relates to: sdks/doc-generation
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the
Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

There were no javadoc publishment.
What is the new behavior?

This enhancement publish the generated javadoc to GH pages.
A new gh-pages branch was created, from where the GitHub Pages being built.
Can be configured from Settings -> GitHub Pages
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

This PR will be not merged, I made this to make my changes more transparent.
Here you can review the full documentation deployment process.
These changes will be generated by sdks.
Currently the document publish runs in stage, so each time the stage is executed with tag test-doc.
In the end, the document will be created only when new release published with the release tag, this change will be made later.

Source branch is merged.

Source branch is merged.


Bug/1645/add missing table assets

TABLES in backend schema is missing the assets table
Resolves #1645

Codecov Report

Merging #1646 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #1646   +/-   ##
=======================================
  Coverage   98.46%   98.46%
=======================================
  Files          59       59
  Lines        2862     2862
=======================================
  Hits         2818     2818
  Misses         44       44

@CsterKuroi You've submitted a pull request before, but it was just a small documentation change so we didn't require you to agree to our CLA. This one is code, however so...
Before we can merge this pull request, we need you or your organization to agree to one of our contributor agreements. One of the big concerns for people using and developing open source software is that someone who contributed to the code might claim the code infringes on their copyright or patent. To guard against this, we ask all our contributors to sign a Contributor License Agreement. This gives us the right to use the code contributed and any patents the contribution relies on. It also gives us and our users comfort that they won't be sued for using open source software. We know it's a hassle, but it makes the project more reliable in the long run. Thank you for your understanding and your contribution!
If you are contributing on behalf of yourself (and not on behalf of your employer or another organization you are part of) then you should:

Go to: https://www.bigchaindb.com/cla/
Read the Individual Contributor Agreement
Fill in the form "For Individuals"
Check the box to agree
Click the SEND button

If you're contributing as an employee, and/or you want all employees of your employing organization to be covered by our contributor agreement, then someone in your organization with the authority to enter agreements on behalf of all employees must do the following:

Go to: https://www.bigchaindb.com/cla/
Read the Entity Contributor Agreement
Fill in the form "For Organizations”
Check the box to agree
Click the SEND button

We will email you (or your employer) with further instructions.

This is to confirm that I agreed to and accepted the BigchainDB Individual Contributor Agreement at https://www.bigchaindb.com/cla/ and to represent and warrant that I have authority to do so.
oXrrKIXhYg8bEzTNrCEuzjhzg7o9ez9apCZNxgKViB4lhQQLN9HZWf7N7OpMx5V

Thanks @CsterKuroi --- that completes the CLA hurdle. I will merge this PR now, since @r-marques already approved it.


Use a lowercase "t" for "thoughtbot"

This is a little bit insane, but "thoughtbot" is always lowercase, even when
starting a sentence.

Sorry that took me so long - somehow I wasn't watching the repo any never got email for those two new PRs. Thanks @gabebw!

Thank you!