Setting formControl to FormGroup

Description
In my custom formly template, I used prePopulate hook to set the formControl to be a FormGroup. However, on any value update, it raised an error
Cannot read property 'length' of undefined. I checked that the error is thrown from FormlyForm.trackModelChanges, specifically the code
this.modelChangeSubs.push(valueChanges.subscribe((value) => {
  // workaround for https://github.com/angular/angular/issues/13792
  if ((control as any)._onChange.length > 1) {...

Looking at angular source code, _onChange is an internal variable only on FormControl, but not in FormGroup, and my custom formly template control is a FormGroup.
Minimal Reproduction

https://stackblitz.com/edit/ngx-formly-custom-template-pumeug?file=app%2Fformly-field-input.ts
Your Environment

Angular version: 8.2.14
Formly version: 5.5.8

💡you may use the following workaround until fixing the issue in our side:
// temporary workaround for https://github.com/ngx-formly/ngx-formly/issues/2016
field.formControl['_onChange'] = [];

Excellent. Thanks a lot @aitboudad

This issue has been fixed and released as part of v5.5.9 release.
Please let us know, in case you are still encountering a similar issue/problem.
Thank you!


[components]: Meshery UI components checklist

Checklist of major components to create and reuse

[ ] https://github.com/layer5io/sistent/issues/399
[ ] https://github.com/layer5io/sistent/issues/385
[ ] https://github.com/layer5io/sistent/issues/382
[ ] https://github.com/layer5io/sistent/issues/381
[ ] https://github.com/layer5io/sistent/issues/335
[ ] https://github.com/layer5io/sistent/issues/334
[ ] https://github.com/layer5io/sistent/issues/318
[ ] https://github.com/layer5io/sistent/issues/313
[ ] https://github.com/layer5io/sistent/issues/312
[ ] https://github.com/layer5io/sistent/issues/264
[ ] #424

Contributor Guide

🎨 Wireframes and designs for Layer5 site in Figma (open invite)

Would like to help with component #381

Can you give me reference to which component i should refer?
Also where should I create this Stepper component? @nebula-aac


[ExplicitModule] Don't pass PCM output path in -Xcc option

There is no need to pass output path of the PCM compilation twice, once as swift -o flag and other time as -Xcc -o flag to clang importer. This can also cause swift caching to miss when output path is different because -Xcc options are not modeled in caching model to understand -o output path doesn't affect compilation output content.
rdar://128650954

@swift-ci please smoke test


Feature: API authentication

Add the new set of permissions for admin/user
Write integration tests to support the new set of permissions
Add a new legacy admin client to support the already existing tests which might fail because of the new set of permissions.

PR submitted here - https://github.com/rancherio/cattle/pull/61

PR submitted again with the new admin_client tests and support for an internal admin client for the tests related to "resource types/fields" we don't intend to expose to the admin/user like nics, ipsec, storagePool etc. https://github.com/rancherio/cattle/pull/61

Closed in favor of rancher/rancher#3833


Raise start_test polling interval

According to the documentation
https://onsi.github.io/gomega/#eventually

the default value for the polling interval is 10 milliseconds

That is excessively fast given the observed failures in
issue #4021 are always using podman-remote.  Lower the interval to
3-seconds, which should be plenty long enough for container removal.
NOTE: I have no idea if this will help prevent the race or not, given
I've only observed it very rarely reproduce (basically once, after #4121 merged).
Signed-off-by: Chris Evich<EMAIL_ADDRESS>

LGTM

/hold

That test failure is #4021 again (or looks identical).  Worse, it should not be possible to fail in that way.
I want to try again later, once F29 is gone and our VMs are updated with proper conmon-packages.

/hold cancel

Confirmed this will/does not fix the #4021 problem, but we still need the change b/c the default interval is 10-miliseconds.  For a 90-second timeout, that's a lot of spam.

/lgtm

/approve


The telemetry object adds data to FTC Dashboard as a line instead of an actual data point.

When you telemetry.addData() it adds at as a line. Then in update() it adds it as a line. It should be using .put instead of addLine() to allow it to take the data in as a datapoint and not just a line.

Fixed in


Caddy: support h2c protocol for reverse_proxy

Important notices

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] When the request is meant for an existing plugin, I've added its name to the title.

Is your feature request related to a problem? Please describe.
I would like to have a gRPC streaming handler, with the TLS managed by Caddy. For this the upstream connection must use h2c (gRPC streams require http/2, but I don't want to have a certificate, hence "unencrypted http/2" a.k.a. h2c).
According to the Caddy docs, the resulting config should look like:
reverse_proxy {
    to h2c://<IP_ADDRESS>:7777
}

Describe the solution you'd like
I would like to be able to select h2c:// as upstream protocol (currently only http:// and https:// are available).
Describe alternatives you've considered
Using a Layer4 config should work, but I am not sure how to enable the TLS termination.
Additional context
The template should be adapted around the following lines:
https://github.com/opnsense/plugins/blob/8cd1be1adbf33c4d04d4b77406c0b84f1de0a42e/www/caddy/src/opnsense/service/templates/OPNsense/Caddy/Caddyfile#L447
The model should be expanded as well:
https://github.com/opnsense/plugins/blob/8cd1be1adbf33c4d04d4b77406c0b84f1de0a42e/www/caddy/src/opnsense/mvc/app/models/OPNsense/Caddy/Caddy.xml#L358
cc @Monviech who originally created the Caddy plugin, I believe (thanks for the great work BTW!)

That seems easy to add, I will put it on my list.
Btw TLS termination will come here:
https://github.com/opnsense/plugins/pull/4364

@oliverpool
You can test the change, execute this in the ssh shell:
opnsense-patch -c plugins ec4b328027e6e87ebab618834073fcd8f9b7bf06

Please give me feedback if it works as expected for you. :)

@Monviech thank you so much, it seems to be working fine!!


Update Jest and react-router-native dev deps

Updates Jest (and related packages) to v24.5.0
Removes deprecated React Native Babel preset babel-preset-react-native and replaces it with metro-react-native-babel-preset.

In order to get the React Native tests working, I had to bump some other dev dependencies. The react, react-native, and react-test-renderer packages are now all set to their latest versions.

Oh, this is good. Jest's Node 11 breakage is a thorn in my side right now.


How should we handle HSTS headers and HTTP redirects in the redirect chain?

🐛 Bug Report
The current pshtt logic that examines the HSTS headers gives credit if any HSTS header encountered in the redirect chain involves the initial endpoint/hostname. If the redirect chain goes on to send the user to an HTTP URL on an entirely different domain for which the browser has never seen an HSTS header, then it seems to me that the user is not being protected by the HSTS header.
How should we handle this case?
Please also see the discussion here in #193.

I believe that credit for a domain should only be given in cases where the STS header is for that domain, or in the case of a subdomain, if a superdomain serves an STS header with the includeSubDomains directive. I agree with @jsf9k that if the STS header is served on a redirect URI that does not match the above cases then the original domain should not receive credit for using HSTS.

in the case of a subdomain, if a superdomain serves an STS header with the includeSubDomains directive.

I don't think we currently check superdomains to see if they include the includeSubDomains directive.

if the STS header is served on a redirect URI that does not match the above cases then the original domain should not receive credit for using HSTS.

Yes, https://github.com/cisagov/pshtt/pull/193 already only gives credit if the original domain uses HSTS, but allows for that to be later on in the redirect chain on the same domain.  I think the question here is if a domain should get credit for HSTS when it uses HSTS itself, but then redirects to other domains that don't use HSTS. In that case, should the original domain's credit for HSTS be removed?


Jest codemods part 1

This is the first part of migrating to Jest:

remove commented out tests (Jest doesn't like these)
fix tests that contains no assertions (Jest found these)

The rest of the migration to Jest should be done by jest-codemods, I'm working on that.

Codecov Report

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

@@          Coverage Diff           @@
##           master   #3399   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files          68      68
  Lines        1529    1529
======================================
  Hits         1529    1529

Impacted Files
Coverage Δ

src/createFieldArray.js
100% <100%> (ø)
:arrow_up:

src/createField.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Replaced by https://github.com/erikras/redux-form/pull/3400


GoImports invocation ignores dependencies in a vendor/ folder

GoImports on save will not add imports that are in vendor/ but not in $GOPATH.  Running goimports from the terminal does not have this problem.

I assume this is the same issue as https://github.com/DisposaBoy/GoSublime/issues/939.
If not, you'll need to tell me how you call goimports since I don't use it.


Feature/add ttc data

In PR13 I changed how we init data, this should follow that as well
Would it be beneficial to have a list of {zoneId: distance (number)} rather storing these as discrete values?


Scheduling cron jobs programatically

Hi,
I want to schedule my cron jobs and prevent them from running on multiple nodes simultaneously using Shedlock. But I want to do this programmatically i.e without using @SchedulerLock annotation.
I checked the post https://github.com/lukas-krecan/ShedLock#running-without-spring
but 'LockingTaskExecutor' does not seem to provide a method to schedule a cron job.
Could you please let me know if this is achievable using ShedLock?

ShedLock only takes care about locking, scheduling had to be done by some other library.


big changes with Teasy element to better support non-existing element…

… case

Please note that I added few unit tests to "prove" that the approach works. @estuold please pay additional attention to the tests, to make sure they make sense


[Enhancement][Cherry-Pick]Optimize iceberg metadata

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:heart_eyes: pass : 320 / 394 (81.22%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/connector/iceberg/cost/IcebergStatisticProvider.java
101
142
71.13%
[145, 159, 160, 161, 164, 165, 171, 172, 320, 321, 343, 347, 348, 349, 350, 351, 353, 354, 355, 356, 357, 358, 359, 361, 362, 372, 373, 375, 377, 380, 381, 382, 383, 387, 390, 393, 394, 395, 397, 398, 401]

:large_blue_circle:
com/starrocks/connector/iceberg/IcebergSplitScanTask.java
9
12
75.00%
[48, 63, 68]

:large_blue_circle:
com/starrocks/connector/iceberg/IcebergMetadata.java
119
142
83.80%
[270, 294, 303, 316, 320, 327, 328, 329, 341, 424, 429, 430, 431, 432, 451, 452, 453, 454, 632, 636, 644, 646, 647]

:large_blue_circle:
com/starrocks/server/MetadataMgr.java
6
7
85.71%
[378]

:large_blue_circle:
com/starrocks/connector/iceberg/cost/IcebergFileStats.java
24
27
88.89%
[120, 246, 248]

:large_blue_circle:
com/starrocks/connector/PartitionUtil.java
10
11
90.91%
[627]

:large_blue_circle:
com/starrocks/sql/optimizer/rewrite/OptExternalPartitionPruner.java
12
13
92.31%
[316]

:large_blue_circle:
com/starrocks/catalog/IcebergTable.java
17
18
94.44%
[155]

:large_blue_circle:
com/starrocks/qe/SessionVariable.java
10
10
100.00%
[]

:large_blue_circle:
com/starrocks/connector/iceberg/IcebergFilter.java
7
7
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/statistics/ColumnStatistic.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/connector/CatalogConnectorMetadata.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/connector/ConnectorMetadata.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/connector/unified/UnifiedMetadata.java
2
2
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)


Make InboundHAProxyHandler @Sharable

#2205 introduces a regression where multiple uses of HAProxyProtocolDetectingDecoder::decode will now end with Exception:
io.netty.channel.ChannelPipelineException: org.apache.cassandra.stargate.transport.internal.HAProxyProtocolDetectingDecoder is not a @Sharable handler, so can't be added or removed multiple times.
Since InboundHAProxyHandler looks reentrant and because the original idea in #2205 was to make it shared let's do just that

What this PR does:
Which issue(s) this PR fixes:
Fixes #2232
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CLA Signed: DataStax CLA

@jakubzytka Ahhh. Sorry, I did not connect the dots here wrt your earlier question.
Yes, this makes sense, will approve.

/gcbrun


Show links contained in post

If a post contains links, it might be helpful to get a list of these links. Otherwise you would click a link without knowing the destination.

done


🎂 [RELEASE] Release v3.1.1 – Fix Search, Move DKIM in Kubernetes to Secrets

🎂 Release
Release v3.1.1 – Fix Search, Move DKIM in Kubernetes to Secrets
Todo

[x] create and set SMTP_DKIM_PRIVATKEY as multi line

[ ] does new multiline notation this work on the Kubernetes cluster?

fixed by  #10


edge.js: Unexpected Duplicate Large Edge Files and File Size Increase

Bug description
A user switched to Cloudfare Workers and noticed that they have two edge files whose sizes are very large. They are wondering why they have two edge files instead of one. They also noticed an increase in the size of the edge files and want to know if they can get rid of one.
Link to Zendesk ticket #3099
Link to discussion in Slack
How to reproduce

Expected behavior
On deployment, we should have one edge file with reduced bundle size
Prisma information
N/A
Environment & setup
Cloudfare Worker Deployment
Prisma Version
5.4.2

From looking at the screenshot, I guess one is the runtime (the one in @prisma/client/runtime/edge.js ) and the other one is the generated Client in .prisma/client/edge.js). Same name, but different files.
So question back: Is the content of those files identical?

They also noticed an increase in the size of the edge files

Increase in what context? From one version to another version? If so, which versions?

Cheers @ludralph and @janpio for looking into this.
For some more detail, it wasn't an increase in file size switching. It was that I want to reduce the bundle size as the current _worker.js file is way too large, there is a limit on the free tier for 1mb - currently using the paid version which increases the size to 10mb. When I looked into what was causing the file size using @next/bundle-analyzer it indicated that every page that I had on the edge had the duped edge.js files from the picture above. I'm hoping that removing one or the other could potentially bring the file size down.

So question back: Is the content of those files identical?

I don't think so as they look like they are two different sizes, and one of them doesn't contain the withAccelerator module.

Oh I also forgot to mention @janpio - the cloudflare team say that if the file size of the worker.js goes beyond 1mb it can cause longer coldstarts - which is exactly what I'm trying to eliminate by switching to cloudlare.

Ok, that makes sense and is of course something we want to improve.
But to double check again:
You did not observe an increase in that size between two different Prisma versions?
(Then I will remove that from the issue title as it can be misleading)

Yep, correct. I didn't notice a change in size between the versions of prisma :)

Any thoughts/updates on this @janpio

What thoughts do you expect?
We now understand that you want these files to be smaller. We will look into if we can make these file smaller when we have time for it, or this becomes a focus of ours to optimize usage of Prisma Client in edge environments.
Something you can try already, and might move things around and as side effect also lead to an overall smaller size could be to define a custom output folder: https://www.prisma.io/docs/concepts/components/prisma-client/working-with-prismaclient/generating-prisma-client#using-a-custom-output-path (Note that the import has to be adapted to the new path!) That should copy the "runtime" from @prisma/client as well and allow your bundle to skip the files from node_modules.


perf: code-splitting should not regenerate chunk

Summary
Every time when iteratoring a block, there will be a new chunk created, though the chunk is not used, but it will stay in memory

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

!bench


Naveen/zone profile gen

Pull Request doc
Purpose
What is the larger goal of this change?
solving the positive slope scenario in the heating cure regression model
What the code is doing
How is the purpose executed?
when the s_heat (slope) >0, there would be relation between temperature and load. 2 additional cases arises with this conclusion.
1.load dependent only on darkness fraction (regression model with only darkness fraction)
2.load independent on both temp and darkness fraction
Testing
How did you test this change (unit/functional testing, manual testing, etc.)?
Where to look

It's helpful to clarify where your new code lives if you moved files around or there could be confusion/
zone_profile_gen

What files are most important?

Usage Example/Visuals
How the code can be used and/or images of any graphs, tables or other visuals (not always applicable).
Current Algorithm:

Proposed Algorithm:

Time estimate
How long will it take for reviewers and observers to understand this code change?

Please see https://github.com/SEL-Columbia/PreREISE/pull/1
This repo is being moved and will be archived.


use different templating engine

https://github.com/valyala/fasttemplate
this will allow using maps instead of structs which will make the code simpler.

this is in latest release.


default_stage does not support callable as all set() do

Q
A

Issue Type
Bug

Deployer Version
4.3.0 / 5.x

Local Machine OS
Mac Sierra

Remote Machine OS
Debian 8

Description
When I set default_stage as callable there is error because it does not expect callable.
set('default_stage', function () {
    return 'local';
});

Steps to reproduce

Set:

set('default_stage', function () {
    return 'local';
});

Make
dep deploy

Output is:
PHP Catchable fatal error:  Object of class Closure could not be converted to string in /Volumes/Data/www/workspace-typo3/projects/t3base8/vendor/deployer/deployer/src/Stage/StageStrategy.php on line 67

When you set:
set('default_stage', 'local');
then everything is working as expected.

@antonmedv
If you accept its a bug I can prepare PR.

This is a tricky one. You can't run anything inside  this cb, because there is no context yet.
So this is doesn't correct:
set('default_stage', function () {
    return run('echo local');
});

In my case I do not need context but only callable.
Having callable allows me to not set the value at once when declared but later when used. This maybe has no difference when we assign simple string but when we assign value of function then this function must be called at once if not used as callable. With callable it can be called when used so function can be overwritten. This is the great power of having callable in set().
So to solve my case its enough to change:
File: src/Deployer.php
$this['stageStrategy'] = function ($c) {
            return new StageStrategy($c['servers'], $c['environments'], $c['config']['default_stage']);
        };

to
$this['stageStrategy'] = function ($c) {
           $default_stage = $c['config']['default_stage'];
           if(is_object($default_stage) && ($default_stage instanceof \Closure)) {
               $default_stage = call_user_func($default_stage);
           }
           return new StageStrategy($c['servers'], $c['environments'], $default_stage);
        };

I think it can not break anything and give more possibilities to have dynamic value of default_stage.
What do you think?
Can I prepare PR with this change ?

Yes, open PR, but please add tests and docs for this feature.

I added the docs in https://github.com/deployphp/docs/pull/94 but I need your help on writing test for this. Should I create separate test/recipe/DeployTest.php ?

I think no test for needed here


C2 27 data persistency

Small PR for adding a bind mount to sync the data/db folder inside mongo container and local data folder

Thank you sir! This looks great


Enhancement Request: Save query output to CSV from the console.

OrientDB Version, operating system, or hardware.

[X] v2.2.9

Operating System

[X] Linux
[X] MacOSX
[X] Windows
[ ] Other Unix
[ ] Other, name?

Enhancement Request
Per email thread https://groups.google.com/forum/#!topic/orient-database/rAfYoQMBs1g I am posting this enhancement request.
The ability to save the results of a query into a CSV table exists in Studio, but to my knowledge it is not currently available from the console.sh app.  The ability to execute a query and save its result into a delimited text file (CSV, TSV, etc.) from the console is desirable for environments like ours in which we would like to be able to run a query using a script that executes a query or series of queries and saves the output into CSV files that can be easily ingested into other tools.
Just a thought, this could possibly be done via the SET command to the console... since it's just a key-value pair... something like the following sequence might be reasonable:
SET OUTPUT_MODE "CSV"
SET OUTPUT_FILE "filename.csv"
SELECT FROM V

or perhaps in one command if the  entries can be parsed:
SET OUTPUT "CSV;filename.csv"
SELECT FROM V

In any case, however it's implemented it would be handy to have a way to save the output from a query to a CSV file from the console so that it could be done via scripts or other batch commands.
Thanks!

I feel this is quite essential for practical situations for people to take out data for reporting or sharing out of the database. Studio too should have, else no way to share either, just copying with clipboard isn't workable.

Just bumping this... this is becoming an increasingly important thing.
In other databases like SQLite, it's pretty trivial to create a script that can dump the results of a query to a CSV file from the console for sharing and/or to pull into excel or a stats tool like JMP.  It'd be nice to have this feature added to OrientDB.
Currently, the only option I have is to run a query and get the ASCII table and manually fix the data to be delimited text so we can pull it into other tools.

As of now I had to write query and dump data using CSV utility using Java API, not idle but no other way to extract query output which I feel is must have for Studio/Console.sh.

Duplicate of #5435

Hello, this enhancement looks like done. But I can't find how to export csv in console with query in documentation.


Use only 1 HEAD and 1 streaming GET request in RangeStream

The existing interface for RangeStream is fine, but what it does internally should change. There should be only one streaming GET request, and all ranges added to this stream will not have their own RangeResponse but a simulated version derived from modifying the original, total range.
This is motivated by the discussion for PngStream in #28 in which I noticed the slow performance of the chunk populating method which relies on sending range requests for the next chunk after determining their location:

You can imagine rewriting it by simply changing what the add method does (split off a subdivision of the total range) and what happens upon initialisation if no range is passed in (a streaming request for the total range instead of a head request).

This was essentially a misunderstanding in the design phase of how to use these streams, and should be fairly simple to change over since it’s tested pretty thoroughly.
The only weirdness may be from having a single response… so maybe that should be ‘faked’ by changing the values and making them as copies…? Unclear if worth the memory toll though
Expect will need some mechanism to ‘gate’ ranges so reading past the end behaves as expected
For testing, perhaps switch over gradually with a flag in the init (singlestream=False) that gets toggled to True once stable and eventually removed in future versions?

Oops… never mind, I did already consider it
On re-reading the motivation pages in the docs, I did already consider this and have forgotten the reason I decided against it.
If you use a single stream, then you’re required to load it linearly. You can’t skip on a single stream, seeking later than the cursor loads all bytes up to it.
The current design permits non-linear reading (which is costly for e.g. ZIP central directories but not for linear PNGs).
This is helpful to clarify and can be documented in the motivation page.
I will need to add this as a reading mode then, so all the previous code can stay as it is as long as I leave the current mode as the default 😃

Rather than simply overwrite the add method wholesale when single_request is True in the params of RangeStream, I suggest adding a new method divide, which can be used in this case, and also be available for less simple use cases (such as "single request for the first N bytes of a file but new requests for any range after that")

add is a fitting name for a method that supplies a particular range on a stream, but divide does not accurately suggest the provision of a sub-range from a pre-existing range.
Better names for this method:

partition
split
subset

Implemented with use_window=True, it all seems to work perfectly.


tune runtime build with --lock/--disable-lock

Current Behavior
Generate lock files automatically, but users cannot inspect the file details.
Proposed Behavior

swcli runtime build --lock command will dump the lock file and compare contents with locked file, give users some advice.
--lock as the default args.
Users can config --disable-lock to disable requirements lock.

https://github.com/star-whale/starwhale/pull/807


FullSaveAssociations always Trigger BeforeCreate Hooks

Your Question
I have 2 models that connected to each other I also using BeforeCreate Hooks to perform hooks that fill created_at and updated_at column. my problem is when I performing an update using gorm.Session{FullSaveAssociations: true} as mentioned in Documentation here FullSaveAssociations the BeforeCreate Hooks also executed. it breaks my created_at and updated_at.
My question is how to perform  FullSaveAssociations on update in the relational table without executing  BeforeCreate hooks
Here is a snippet of my code
MembersOnline struct {
	ID         uint64    `gorm:"primaryKey column:id" json:"ID"`
	NickName   string    `json:"nick_name" gorm:"column:NickName"`
	Email      string    `json:"email" gorm:"column:Email"`
	MemberNo   string    `json:"member_no" gorm:"column:NoAnggota"`
	CreateDate time.Time `json:"create_date" gorm:"column:CreateDate"`
	UpdateDate time.Time `json:"update_date" gorm:"column:UpdateDate"`
	Member     Members   `gorm:"foreignKey:MemberNo;references:NoAnggota" json:"member,omitempty"`
}

Members struct {
	ID                 uint64                `json:"ID"`
	MemberNo           string                `gorm:"column:MemberNo;primaryKey" json:"member_no,omitempty"`
	FullName           string                `json:"full_name,omitempty" gorm:"column:Fullname"`
	Phone              string                `json:"phone,omitempty" gorm:"column:Phone; default:null"`
	CreateDate         time.Time             `json:"create_date" gorm:"column:CreateDate"`
	UpdateDate         time.Time             `json:"update_date" gorm:"column:UpdateDate"`
}

func (u *Members) BeforeCreate(tx *gorm.DB) (err error) {
	u.CreateDate = time.Now()
	u.UpdateDate = time.Now()
	return
}

##Update function
DB.WithContext(ctx).Session(&gorm.Session{FullSaveAssociations: true}).Updates(memberOl)

The document you expected this should be explained
Expected answer

I just found solutions by adding conditional check-in if the variable with zero value or not

I believe, the way GORM works is it always tries to insert before updating. If you use the.debug() command the database, you can see exactly what GORM is querying.  This got me at first as well.


On resize the library does not work correctly

When resizing the page the sticky element does not have the correct position.

The issue was introduced by commit 727bf31fa5b081a77ebd7cc38045b2905b10c2b6 at line 458. The problem is that the handler executed in onResize() is the one before it's updated by the call to  init().


[FancyZones] "Match not found" error fix

Summary of the Pull Request
What is this about:
Changed work area initialization in the Editor. Passing all the necessary data from the C++ side.
What is include in the PR:
How does someone test / validate:
Open FancyZones Editor with different monitor scaling, verify that layouts are placed correctly. To be sure, set any template grid layout, remove spacing and observe that layout borders match screen borders.
The bug is reproducible with scaling different from 100%, so at least one of the monitors should be scaled (Settings -> System -> Display -> Scale and layout).
Quality Checklist

[x] Linked issue: #11717
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Can we merge?

Can we merge?

It would be better if Mykhailo test it too before merge.


Contributor PRs do not run TravisCI checks

For some currently-unknown reason, whenever an external contributor forks neverthrow and opens a PR from their repository to merge into this repository, the TravisCI checks do not run. The TravisCI pipeline stays yellow.
This is really annoying as it doesn't tell the PR author or me if the mandatory checks have all succeeded.
If you know why this might be happening, or you know how to solve it. I would really appreciate a PR or even a comment to guide me in how to fix this.
Screenshot

Sorry if it's off topic but I received a different screenshot from the email notification for this issue and I'm thinking it might be leaking personal information. (big WTF)

Yeahhh that was my bad :'( And now I don't have access to the image link. Would you be able to email me at<EMAIL_ADDRESS>with the url of the image? I will request github to remove it.


Create E2E tests for the Developer Resources Page

Problem statement
This page on the RS website needs End-to-end (E2E) tests created to ensure that it works properly before merging into master. Without E2E tests, some features/links may not function/load properly and we wouldn't know until a user tried to use them.
The Page is located at: https://reportstream.cdc.gov/developer-resources
What you need to know

This is based off the investigation from #11307
E2E Test Creation Tracker
Strategy discussed for checking content load: check to see if every DIV on this page loaded

Acceptance Criteria
As any user, logged in or not, check that:

[ ] The page loads
[ ] If there is an error, the error is shown on the page
[ ] Page content loads without errors (including if the footer is showing, but not its functionality) (also, not checking the content itself, but just if it loads)
[ ] Components work as expected (eg, accordion expands/collapses, side nav, etc)

E2E tests for this page already exist.


Player save does not exist?

I've tried this entire process 4 times, over the last 3 hours and this is the farthest I can get.  Either I'm not understanding something in the directions correctly, putting some file in the wrong spot, idk.  Very frustrated
ERROR: Your player save does not exist. Did you enter the correct new GUID of your player? It should look like "8E910AC2000000000000000000000000".
Did your player create their character with the provided save? Once they create their character, a file called "C:\Users\admin\Desktop\PalTemp/Players/308344D9000000000000000000000000.sav" should appear. Look back over the steps in the README on how to get your new GUID.

Hey there, I have the same problem too. I fixed it by remove ".sav" in the command.
For example, you should enter "308344D9000000000000000000000000", not "308344D9000000000000000000000000.sav"

mine doesnt work either, and I have never put the .sav at the end.... other idea?

python fix.py D:\uesave\uesave.exe D:\uesave\XXXXXX5F4472575B784671B1A41DA32B XXXXC527000000000000000000000000 XXXX9CD8000000000000000000000000
XXXXXX5F4472575B784671B1A41DA32B = savegame folder
XXXXC527000000000000000000000000 = new save
XXXX9CD8000000000000000000000000 = old save

I figured it out! Do I know what I did to make it work, no! Could I do it again if I tried, probably not! But it works.

I wourld to feedback that cuz i spend a couple hours to understand what i do wrong, as 0 in python
In first - commands need to paste into simple cmd.exe , not python-cmd window
My workspace forlder was right in player-save-folder
Then in my case full my command looks like
python "H:\SteamLibrary\steamapps\common\PalServer\Pal\Saved\SaveGames\0\ABCDEFG123456790\Players\fix-host-save.py" "C:\Program Files\uesave\bin\uesave.exe" "H:\SteamLibrary\steamapps\common\PalServer\Pal\Saved\SaveGames\0\ABCDEFG123456790\Players" "CDC3F71B000000000000000000000000" "00000000000000000000000000000001"
Also i didnt know that i must place .py files directory to command TOO
Than cmd say that i havent player save file like -
_FileNotFoundError: [Errno 2] No such file or directory: 'H:\\SteamLibrary\\steamapps\\common\\PalServer\\Pal\\Saved\\SaveGames\\0\\ABCDEFG123456790\\Players/Players/00000000000000000000000000000001.sav'
So i create extra Player folder inside Player 0\saveid\Players\Players
Cuz cmd mark directory with reverted " / "
Than i place both player save file to 0\saveid\Players\Players
Also i must to place Level.sav nearby fix-host-save.py ( H:\SteamLibrary\steamapps\common\PalServer\Pal\Saved\SaveGames\0\ABCDEFG123456790\Players\Level.sav )
And than all start works

Need new_guid file copy to Players folder

Has anyone got this working on a Mac or Linux ?

Closing this issue because it's user error and the person who created it figured it out.


How to resume the training

Hi there, thank you very much for this repo. It's very helpful.
I'm using your nerf_colab.ipynb to generate train the model. However, I don't really know how to resume from an existing checkpoint. Please, could you tell me how to resume from an existing checkpoint in order to continue the training? I'm a machinelearning/artificialintelligence newbie.
Thank you for your hard work.

If you're using colab, I'm afraid that it automatically deletes all files once you disconnect, so your models will be lost, unless you manually download the checkpoints from colab.
So did you manually save a checkpoint and want to resume from that? It is simple then, you just need to add the argument --ckpt_path ${YOUR CHECKPOINT} to python train.py.

Perfect! Yes, I'm downloading every checkpoint in order to save locally and don't lose them if I'm disconnected from colab.
Thank you for your answer!! Really grateful.
Maybe it possible to modify train.py to save ckpt files to GDrive. Anyway, thank you again!

Yes, it is possible, modify this line:
https://github.com/kwea123/nerf_pl/blob/6ea0410768b578209d3704f44baf8dec922db1e6/train.py#L160
change the string ckpts to the folder you want to save to, e.g. /content/drive/My Drive/colab
It is a really specific case so I prefer not to add this as an argument, but I'll leave it here in case someone has the same need.


[agent] Add Spark Agent

added support for spark through pyspark library.
added jupyter notebook as example.

No prob! Will change.
On Tue, May 2, 2023 at 10:27 PM Gengliang Wang @.***>
wrote:

@.**** commented on this pull request.
In langchain/agents/agent_toolkits/spark/prompt.py
https://github.com/hwchase17/langchain/pull/4020#discussion_r1183249929:

@@ -0,0 +1,13 @@
+# flake8: noqa

+PREFIX = """
+You are working with a spark dataframe in Python. The name of the dataframe is spdf.
Nit: shall we make the dataframe name as df? Probably you are trying to
make a difference from the panda dataframe, but many Spark users are using
df.
—
Reply to this email directly, view it on GitHub
https://github.com/hwchase17/langchain/pull/4020#pullrequestreview-1410203400,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO64MHZ3QKGKKSNFQCU25NLXEHUFBANCNFSM6AAAAAAXT3KIKM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Running in detectron2

I want use it in detectron2. I don't know what is "targets". Could you tell me?  for example, in line 28, 'yolox.py'.
Thank you for you awesome work.

'targets' is the groundtruth generated by dataloader/dataset. When doing inference(model.eval()) we needn't 'target'( is None)

'targets' is the groundtruth generated by dataloader/dataset. When doing inference(model.eval()) we needn't 'target'( is None)

Thx very much.

By the way.
The shape of "targets" is [2,120,5]. So what's the meaning of them, respectively? Thx.


fix: enable git-main-branch to figure out branch from subdirs

Got confused to why my branch switching alias didn't work sometimes in our monorepo 😄

ooh amazing, thanks!
I got that issue some times myself but didn't get why :D


Fixes #3507: Fix App Crashes on opening Contributions from MainActivity

Fix app Crashing
Fixes #3507 App Crashes in MediaDetailsFragment with ClassCastException
What changes did you make and why?
Implemented CategoryImagesCallback for MainActivity

Codecov Report

Merging #3512 into master will increase coverage by 0.03%.
The diff coverage is 0.00%.

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


Disable shallow clone for libbacktrace dependency

Shallow cloning breaks whenever GIT_TAG is not the current HEAD.
Also update GIT_TAG to revision of current HEAD

We don't really need the shallow clone in this case - but is there any reason this doesn't work?

I assume it only downloads the most recent revision in a shallow clone, nothing else.
This has not been a problem until today when there was a new commit upstream, HEAD now points to 4d2dd0b172f2c9192f83ba93425f868f2a13c553 but we tried to check out HEAD-1 ( 2446c66076480ce07a6bd868badcbceb3eeecc2e ), which it couldn't resolve because it only knows the most recent one.
Without the shallow clone we can still pin to a known good version and it should not break in the future.


BUG FIX: Race condition when saving poll

Adding the poll option id int to the poll option set asynchronously creates a race condition which results in a 'duplicate index error' (MongoServerError: E11000 duplicate key error collection: objects index: _key_1_value_-1).
This is because NodeBB adds a unique key-value index.

@Gadana1 thanks, published 3.1.3.
One of these days I will rewrite this to use async/await and use the setObjectBulk method.

Thanks for merging @barisusakli

Also setAdd can accept an array of values so that should work as well,
await db.setAdd(`poll:${pollId}:options`, options.map(opt => opt.id));

Interesting... Didn't know that..

@Gadana1 thanks, published 3.1.3.
One of these days I will rewrite this to use async/await and use the setObjectBulk method.

Noticed someone created a PR for the async/wait. Might be worth taking a look. 🙂


The 'clangd' language server was not found on your PATH. Would you like to download and install clangd 10.0.0?

I get this error (as above) when I install the extension on my vscode editor. Not sure how to fix this problem, it seems that
System information
Output of clangd --version: clangd --version doesn't show up, but c++ --version shows Ubuntu clang version 10.0.1-++20200708122807+ef32c611aa2-1~exp1~20200707223407.61
Editor/LSP plugin: llvm-vs-code-extensions.vscode-clangd
Operating system: Ubuntu 20.04
I am not sure what I should do,  I am sure clang is install because I can compile my C++ Programs, but the plugin itself doesn't work?

clangd.path: had to be set to the executable.


Always use the provided page URL and referrer

We've had a customer wanting to track AJAX page loads. They pass details to page(), but our internal KM.pageView() function records a Page View event with the window URL as a property rather than the URL of the specified page.
The KM.pageView() function is defined as follows:
KM.pageView = function() {
  window._kmq.push(['record', 'Page View', {
    'Viewed URL': KM.u(),
    'Referrer': KM.rf()  || 'Direct'
  }]);
}

This PR inlines and then modifies the page view tracking code so that we send the correct details.
cc @salomon

Any update on this?

@chriso sorry for the late response. I left a comment. We're currently undergoing a big rollout for our newest version of analytics.js. The rollout should be happening throughout this week. But we can definitely prep this PR so that once the migration is fully complete, we can ship these right away.
I'd expect ETA of sometime next week!

[Current coverage][cc-pull] is 98.42%

Merging [#15][cc-pull] into [master][cc-base-branch] will increase coverage by 0.01%

@@             master        #15   diff @@
==========================================
  Files             1          1
  Lines           126        127     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            124        125     +1
  Misses            2          2
  Partials          0          0

Powered by Codecov. Last updated by [9812c5f...f7ea97a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/segment-integrations/analytics.js-integration-kissmetrics/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/segment-integrations/analytics.js-integration-kissmetrics/compare/9812c5fb92df6d95fcb1580da81d88931eed5d66...f7ea97a4f87c4f0f93636a9242557467052fc4eb
[cc-pull]: https://codecov.io/gh/segment-integrations/analytics.js-integration-kissmetrics/pull/15?src=pr

@hankim813 I've rebased and added an additional test case. Let me know if you need anything else.

@chriso LGTM!
Just to reiterate, going to merge this now but we won't ship the new release including these changes until we finish the rollout of our ajs which hopefully should be done by end of this week or early next week.
I'll definitely add a note on my calendar to make sure I address KM once we complete the rollout!
Thanks again for the PR :) 💯

going to merge this now but we won't ship the new release including these changes until we finish the rollout of our ajs which hopefully should be done by end of this week or early next week.

No problem, and thanks!

@hankim813 can you let us know once analytics.js+this has been released?

@chriso absolutely! We're at the last leg of the migration so I'm hoping we can ship all these changes first thing Monday barring any setbacks! Definitely will ping on this thread 👍


List outdated, does not include Foxit

Foxit PDF reader is showing as an enlighten app under the Enterprise Context column in task manager after adding it as an allowed app in my WIP policy.
Are there any other apps that are enlighten missing from this list?

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

ID: b5ed0150-0172-f3ad-3c7f-3a7141efbe64
Version Independent ID: 6a978d2b-a0ff-1052-ce00-06f02f30cfa1
Content: List of enlightened Microsoft apps for use with Windows Information Protection (WIP) (Windows 10) - Microsoft 365 Security
Content Source: windows/security/information-protection/windows-information-protection/enlightened-microsoft-apps-and-wip.md
Product: w10
Technology: windows
GitHub Login: @DulceMontemayor
Microsoft Alias: dansimp

Hello, @avbski12
The issue is under investigation and you will be notified with any changes on the way.
Thank you for providing feedback.

@avbski12 The article you mentioned contains the list of Microsoft enlightened / unenlightened apps, not third party apps.
You can find the information needed in this article.

@avbski12
Closing via the above comment that we hope answers your question. Thank you for reaching out.


Add search bar for programs

It's pretty obvious that with time the amount of programs in the list will grow and at some point you will need to somehow find them. So  either it should be some kind searchbar or maybe pop-up explorer, . Fuzzy finder, if possible. Image as reference from packer manger in sublime text.

Dig the idea, I'll think of a way of adding a search in the future.


Login session expires too frequently

I authenticated with Github which was pretty smooth, but now I have to login again once or twice every day. That's kind of annoying and I can't see an option to change it.
Not sure if this is some token expiration thing that Github forces upon us, or if it would go away by me adding a password and email address instead, but if so that should probably be mentioned when setting up the account.

I think I also experienced this minor Issue just a few hours ago.  My CW account is linked to this GitHub account but my most recent login was with an ordinary email and password (not GitHub login).  However, after viewing "My Profile" following about 30 minutes of solving Kata, I realised I was (unexpectedly) logged out when I went back to the "Kata" page.  Not that it was a major inconvenience to me at the time but weird nonetheless.

This actually happens to be a duplicate of the first issue #1 (which remains open). It's also likely related to #762, and #462.
Anecdotally I've noticed the frequency of this seems to increase the more codewars tabs I've been using.
If I've got lots of codewars tabs open I tend to need to log in again regularly, which is a pain as after logging in you are not returned to your previous page.

Thanks @DonaldKellett and @brendanvos -- is there a way to mark this as a dupe and close it so it's still referenced by #1? Or should I just close it and add a comment to the other bug? It's not massively encouraging that the very first issue in the tracker is still open, although it's not the worst bug in the world I guess... :bomb:

You can either leave it open or close it.
Just by referencing the other issue this one shows up as 'referenced by' in the other issue's log.
I usually close my issues if I realize they're duplicates, but seeing as this has been around for so long it might be good to stir up discussion on it again.
Besides, the developers can close it and/or mark this as a duplicate when they come across it.

This issue continues to linger. I've done a lot to try to clean it up and its much better than it was when the issue was originally logged, but unfortunately its a combination of technical debt and our use of PJAX, which seems to make troubleshooting these login issues a lot harder.
The issue is definitely more reproducible when you use multiple tabs.

Closing due to duplicate.


BlocksRuntime: match the ABI specification for LLP64

The Blocks ABI v1 defined the size and reserved fields as unsigned long int rather than uintptr_t.  This breaks on LLP64 hosts where
uintptr_t is larger than unsigned long int.  Make this conform to
the specification.

CC: @MadCoder

@swift-ci please test


When loading an OBJ with a large index range, I encounter the following error. Could you suggest a solution?

When loading an OBJ with a large index range, I encounter the following error. Could you suggest a solution?
Validation Error: [ VUID-vkCmdDrawIndexed-robustBufferAccess2-07825 ] Object 0: handle = 0xa4324a00000001c5, type = VK_OBJECT_TYPE_BUFFER; | MessageID = 0x41363e42 | vkCmdDrawIndexed(): index size (4) * (firstIndex (0) + indexCount (10000)) + binding offset (0) = an ending offset of 40000 bytes, which is greater than the index buffer size (24000). The Vulkan spec states: If robustBufferAccess2 is not enabled, (indexSize {times} (firstIndex + indexCount) + offset) must be less than or equal to the size of the bound index buffer, with indexSize being based on the type specified by indexType, where the index buffer, indexType, and offset are specified via vkCmdBindIndexBuffer

I’ll have a look into this

Not been able to reproduce this. Is it for one specific obj?


Topic recognition Pull Request - Siamese s4697173

Hi, This is a pull request to merge the recognition problem for the Siamese Network classification of the ADNI brain dataset.
The code library is designed for use on the Rangpur cluster by default (Accessing the ADNI dataset from the Rangpur paths) but can be modified to work locally.

This library implements the classification of the ADNI dataset into AD and NC labels.
A 3D Resnet model was used as the backbone for the Siamese Network.
The Siamese Network was implemented with Triplet Loss.
The classification model is a simple model comprising several linear layers.

The following requirements determined by the maintainers have been met:

Reasonable Commit Log History
Fork of the topic-recognition branch.
Documentation of the problem in the README.md
Completed test-driver script (predict.py)

Requires the following libraries:

Torch 2.1.0+cu118,
pytorch-cuda | 11.7
seaborn | 0.13.0
scikit-learn | 1.3.1
opencv-python | <IP_ADDRESS>
numpy | 1.26.1
time | 1.8

Regards,
Matthew (s46971733)

This is an initial inspection, no action is required at this point
Difficulty: Hard
Readme: Good

Clear flow of information
Algortihm and usage explained well
Some plots for loss and 2D manifolds provided (only for the training set for the triplet network though)
No accuracy plots provided

Commit messages: very good, detailed
Code:

uses a triplet network with a 3D resnet backbone
good design
well commented

Functionality/Performance:

triplet network seem to have separated the two classes very well for the training set, but you could have plotted the loss/accuracy/2D manifolds for the validation set as well to understand issues
classifier accuracy is only 50% (random guessing?)
doesn't seem to have performed hyperparameter tuning and no attempts have been made to improve either networks

General comments:

solves the problem apporpriately although performance is poor
you need to have observed the performance on the validation set as well for both networks to show/understand potential issues with the algorithm

Marking
Good Practice (Design/Commenting, TF/Torch Usage)
Adequate design and implementation
Good spacing and comments
Header blocks missing -1
Recognition Problem
Solves problem, poor performance <0.6 -1
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage, accuracy plots would be good -1
Module present
Commenting
No Data leakage
Difficulty: Hard
Commit Log
Meaningful commit messages
Progressive commits used
Documentation
ReadMe good, no usage -1
Model/technical explanation
Good Description and Comments
Markdown used and PDF submitted
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required
Request Description good


Clean up EnvironmentLogger directory

After ROGER maintenance is done, @jdmaloney and I should work together to clean up the EnvironmentLogger database of files named "enviromentlogger" (missing first 'n') that should be duplicates of the corrected versions with a typo in the name.
None of the erroneous files appear to be in Clowder so there shouldn't be any issue deleting these.

This clean up has been completed today, duplicates with wrong spelled names are gone, and single files with wrong name have been moved to correct name.


ci(nano): cannot compile into executable

Received an error from Deno when attempting to compile the executable for hyper nano.
Tracking on deno repo in https://github.com/denoland/deno/issues/15705

Solved with the release of Deno v1.25.1


warning when async section not in prompt

The problem
When you have a prompt order set without async in your prompt order, you get the following flash error when it tries to render.
Warning! The 'async' section was not found. Removing it from the prompt.

I would expect that if the async section isn't present, it just quietly tries to render, until it fails, in which case an error could be displayed.
Relevant Zsh configuration
DIV="⟡ "

# Spaceship Options
# https://spaceship-prompt.sh/sections/
SPACESHIP_PROMPT_DEFAULT_PREFIX=$DIV
SPACESHIP_PACKAGE_PREFIX=$DIV
SPACESHIP_DOCKER_PREFIX=$DIV
SPACESHIP_GIT_PREFIX=$DIV
SPACESHIP_DIR_PREFIX=$DIV
SPACESHIP_AWS_PREFIX=$DIV
SPACESHIP_EXEC_TIME_PREFIX=""

SPACESHIP_CHAR_SYMBOL="➜ "
SPACESHIP_CHAR_SYMBOL_ROOT="➜ "

SPACESHIP_EXIT_CODE_SHOW=true
SPACESHIP_EXEC_TIME_COLOR=245
SPACESHIP_EXEC_TIME_ELAPSED=0

# SPACESHIP_JOBS_AMOUNT_THRESHOLD=0
SPACESHIP_GIT_STATUS_COLOR=magenta
SPACESHIP_PACKAGE_SHOW_PRIVATE=true
SPACESHIP_PYTHON_COLOR=green

SPACESHIP_PROMPT_ORDER=(
        user
        host
        dir
        git
        aws
        package
        node
        bun
        xcode
        golang
        python
        venv
        rust
        haskell
        java
        docker
        kubectl
        # async
        line_sep
        battery
        char
)

SPACESHIP_RPROMPT_ORDER=(
        jobs
        exec_time
        exit_code
)

### What is version of Spaceship are you using?

4.10.1

### What is version of Zsh are you using?

zsh 5.9 (arm-apple-darwin21.3.0)

### What operation system are you running?

macOS

### Do you use any Zsh framework or plugin manager?

no

### Which terminal emulator do you use?

kitty

### Additional information

_No response_

Thanks for reporting! I can confirm this is a reproducible bug. I'll take a look what I can do.
You're welcome to investigate and send a PR too!

Should be resolved in 0a18cb5


Complete published applications.

Currently when clicking on link to read more about an application you get a 404.

No longer an issues sense the site was reworked to a gatsby site


[WIP] Protect analytics

To capture data points for analytics reason, requestRecord is added.

[x] RequestRecord,db entry creation
[x] Store data points for each request (Right now data points are logged)
[ ] Error enum mapping
[ ] store url & query param part of RequestEntry

good start! let's review in person tomorrow :)


Can no longer ignore Samsung TV in discovery integration

The problem

The discovery integration is throwing an error when trying to use ignore with Samsung TVs.
Environment

Home Assistant release with the issue: 0.105.2
Last working Home Assistant release (if known): 0.104.3
Operating environment (Hass.io/Docker/Windows/etc.): Docker
Integration causing this issue: Discovery
Link to integration documentation on our website: https://www.home-assistant.io/integrations/discovery/

Problem-relevant configuration.yaml

discovery:
  ignore:
    - samsung_tv

Traceback/Error logs

2020-02-07 09:46:52 ERROR (MainThread) [homeassistant.config] Invalid config for [discovery]: value is not allowed @ data['discovery']['ignore'][0]. Got 'samsung_tv'. (See /config/configuration.yaml, line 93). Please check the docs at https://www.home-assistant.io/integrations/discovery
2020-02-07 09:46:52 ERROR (MainThread) [homeassistant.setup] Setup failed for discovery: Invalid config.

Additional information
I have also tried using the following, with the same error
discovery:
  ignore:
    - samsungtv

Because it's now discovered by the ssdp integration. You can ignore it within the UI.

Closing this issue as the solution is mentioned above and the discovery documentation has been updated to exclude the samsung_tv integration.


Improve documentation and integrate documentation with readthedocs.org

Documentation for ifm3d could be significantly improved -- both the content and quality as well as the presentation. On the latter topic, this issue proposes integrating our documentation with the Read The Docs website. Putting the infrastructure in place to automatically build updated documentation upon release or commit to a particular branch would be ideal.
Also, let's start a list of what documentation needs to be made available:

[x] Supported platforms
[x] Building the software
[x] Building and installing on Windows (should be treated separately from above)
[x] How to implement a new image container
[x] How to configure the imager using the JSON interface
[ ] Related to above, all parameters need to be documented by the firmware/systems team
[x] Using the ifm3d command-line tool

Please help me add to this list!
(NOTE: Is there a good way to make this check list shared so others can add to it? This is beyond my github skills).
We note that, per @graugans comment in #35 we will continue to provide code examples on using the library. The side benefit of making the examples a separate project also provides a nice way to illustrate how to integrate ifm3d into your project to include the build chain.

@graugans Would appreciate if you can lead the charge on getting the last checkbox on my list above checked off.

@tpanzarella frankly I am struggling a bit with this task. I do understand the need of documentation. I would  like to check what is provided in the official documentation to not have a duplicated information. On the other hand I understand that one is interested in this library may  want to have direct access to the documentation and not go through the whole ifm documentation first. Another point we should care about is the fact that we have different versions of firmware and each version may have a different set of parameters.
Meanwhile since quite some time we expose the underlying schema from which we generate the complete parameter set. This typically contains the type of parameter and its range plus a default parameter.
http://<IP_ADDRESS>/cgi-bin/schema.sh

The link above generates a tar ball  containing all schema information. With the help of a python script and a camera connected we can generate a list of parameters and their limits. This can be generated for each version. supporting the schema download.

I give doscon a shot.

This is my proposal:
$ wget https://github.com/lbovet/docson/archive/v1.1.0.zip
$ unzip  v1.1.0.zip
$ cd docson-1.1.0/
$ mkdir -p o3d3xx
$ curl -s http://<IP_ADDRESS>/cgi-bin/schema.sh | tar xvf - -C o3d3xx
$ python3 -m http.server

When you now start a web browser you can open the following URIs
Device configuration
http://<IP_ADDRESS>:8000/index.html#/o3d3xx/gecco/schema/device.json
Application
http://<IP_ADDRESS>:8000/index.html#/o3d3xx/gecco/schema/device.json

The information provided by the schema is good in terms of understanding what the names , data types, and valid ranges are but this does not answer the primary question of semantically what do the tunable parameters mean. For example, we get asked often things like "What is the 'SymmetryThreshold'"? What happens when I turn it up? Down? Also data types are not units of measure. Here is a simple example. The extrinsic calibration parameters exposed on the O3D are:
"ExtrinsicCalibRotX": "0",
"ExtrinsicCalibRotY": "0",
"ExtrinsicCalibRotZ": "0",
"ExtrinsicCalibTransX": "0",
"ExtrinsicCalibTransY": "0",
"ExtrinsicCalibTransZ": "0",

OK, so what is the coord frame? We know that these relate to the optical frame and not the camera frame because we establish a camera frame consistent with the Right Hand Rule (consistent with what most roboticists would expect). What are the units? First time I saw it, I just assumed meters and radians. However, I was very wrong. These are mm and degrees.
My point is, much more important than saying the parameter ExtrinsicCalibRotX takes a float is to define what it means and its unit of measure in the real world.
Unfortunately, the example parameters I use above for the extrinsics are actually the obvious ones. Others are much more obtuse. For example, what does ThreeFreqMax3FLineDistPercentage mean? What is its unit of measure? I simply see it is "80".

I guess a few point we can address with a description of the parameters for others we need more of an application note. Others can me marked as internal use. The ThreeFreqMax3FLineDistPercentage is a good example. I guess less than a handful people know how to control this parameter.

We should discuss how to communicate an internal vs. external parameter.
Application notes are always a good idea.
Could SV help in this effort?

@GarrettPlace You are right. I started an internal discussion about some of the parameters, maybe there is a specific reason why we expose them. Related to SV, I guess he can support us when it comes to the application notes.

I'd just like to add that there is nothing wrong with exposing deeply technical parameters via the XMLRPC interface (and by association the JSON serialization). However, we just need to explain what they mean. In some cases, I assume, having access to these via the ifm3d Linux command line will be desirable. Said another way, hiding the parameter is not documenting it.

@tpanzarella I agree hiding to avoid documentation. The idea behind this was, if nobody needs this than we can skip it. Today I learned we need those parameters so I'll document them.


Editor jumps to new line after formatting

After black has finished formatting, the editor window jumps to a new line. Sometimes it is just a few lines below, sometimes the very bottom of the file (could be hundreds of lines below). The cursor itself is often in the same position, but the editor window has paged down. Sometimes the cursor itself has jumped to a new location

@willjhenry Try the pre-release version, it should have a fix for this.

awesome - thank you!

It still pages down ~one page, but it no longer jumps to the bottom of the file. Is this expected?

The pre-release improves the case my doing narrow edits. But that is all this extension provides. Further improvements for this behavior should be addressed in VS Code itself.
From the extension we don't really do anything other than telling the editor that the various edits needed to be done to the file as a result of formatting. VS Code chooses to move the cursor or scroll as it sees fit.
If you want to file a bug on VS Code, I recommend setting "python.trace.server": "verbose" in your user settings. This should capture the file and edits we send back to repro. Reproing the issue, and then share it with the logs.


Feature/grading instructions/modify markdown editor

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[x] Server: I added multiple integration tests (Spring) related to the features
[x] Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)
[x] Server: I implemented the changes with a good performance and prevented too many database calls
[x] Server: I documented the Java code using JavaDoc style.
[ ] Client: I added multiple integration tests (Jest) related to the features
[ ] Client: I added authorities to all new routes and check the course groups for displaying navigation elements (links, buttons)
[ ] Client: I documented the TypeScript code using JSDoc style.
[x] Client: I added multiple screenshots/screencasts of my UI changes
[x] Client: I translated all the newly inserted strings into German and English

Motivation and Context

Given the current situation, instructors add a block of text describing the grading instructions in the text editor. However, in order to unify the assessment feedback, we have to divide the text block in specific parameters, that could, later on, be dragged and dropped by tutors when assessing an submission.
Description

Added the following domain commands :

Credits
Grading Scale
Grading Instruction
Feedback
Usage Count (which indicates how often the points should be applied for the current instruction)

to the text editor of the grading instructions and by that we divided the text block into four text fields. The above-discribed commands are all triggered by the GradingCriteriaCommand to enable users to add all parameters of instruction via one click.
Steps for Testing

Navigate to Course Administration
Navigate to Exercise
Edit Modeling/text/file-upload exercises
Add free text in the grading instructions editor and save it
Switch to structured Grading instructions -> if you switch back to grading instructions the free text inserted before should still be available
Add a new criterion/ edit existing criterion/ delete an existing criterion
Add one or multiple instructions to a criterion of your choice/ edit and delete it
Save your changes -> when you click on edit again your changes should be displayed correctly

Note: it is not possible to delete a single instruction. Only a criterion can be deleted and all its corresponding instructions will therefore also be removed. This limitation will be addressed in a separate PR
Screenshots

We decided to close this PR as it is covered in #1269


@mcfi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mcfi : Thanks for your contribution! The author(s) have been notified to review your proposed change.
Originally posted by @PRMerger18 in https://github.com/MicrosoftDocs/windowsserverdocs/pull/4997#issuecomment-724170302

Please don't add unhelpful tickets.

#please-close


[Packaging] Error prone tests and docs modules included

The tests (and docs) modules are packaged, despite it's not good practice it may alors break projects having their own tests module.
Environment details

OS type and version:
Linux mistigri 5.8.0-53-generic #60-Ubuntu SMP Thu May 6 07:46:32 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Python version: python --version :
Python 3.8.6

pip version: pip --version:
pip 20.1.1

google-cloud-recpatchaenterprise version: pip show google-cloud-recaptcha-enterprise
Name: google-cloud-recaptcha-enterprise
Version: 0.3.1
Summary: UNKNOWN
Home-page: https://github.com/googleapis/python-recaptcha-enterprise
Author: Google LLC
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /home/qbey/dev_perso/venv/test_project/lib/python3.8/site-packages
Requires: packaging, google-api-core, proto-plus
Required-by:

Steps to reproduce

new virtual env : virtualenv -p /usr/bin/python3.8  ../venv/test_project
python -c "import tests;print(tests.__file__)"
/home/qbey/dev_perso/venv/test_project/lib/python3.8/site-packages/tests/__init__.py

Code example
No real code example, this causes crash on one of our Django library that implements its own tests module and cannot be loaded through the Django magic mod = importlib.import_module(self.SETTINGS_MODULE) where self.SETTINGS_MODULE is tests.projects.settings.
Stack trace
Traceback (most recent call last):
  File "/home/jenkins/venv/2/bin/btr", line 8, in <module>
    sys.exit(run())
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/bluetils/pytest/cli.py", line 138, in run
    return_code = pytest.main()
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/config/__init__.py", line 143, in main
    config = _prepareconfig(args, plugins)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/config/__init__.py", line 318, in _prepareconfig
    config = pluginmanager.hook.pytest_cmdline_parse(
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/pluggy/hooks.py", line 286, in __call__
    return self._hookexec(self, self.get_hookimpls(), kwargs)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/pluggy/manager.py", line 93, in _hookexec
    return self._inner_hookexec(hook, methods, kwargs)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/pluggy/manager.py", line 84, in <lambda>
    self._inner_hookexec = lambda hook, methods, kwargs: hook.multicall(
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/pluggy/callers.py", line 203, in _multicall
    gen.send(outcome)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/helpconfig.py", line 100, in pytest_cmdline_parse
    config: Config = outcome.get_result()
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/pluggy/callers.py", line 80, in get_result
    raise ex[1].with_traceback(ex[2])
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/pluggy/callers.py", line 187, in _multicall
    res = hook_impl.function(*args)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/config/__init__.py", line 1003, in pytest_cmdline_parse
    self.parse(args)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/config/__init__.py", line 1283, in parse
    self._preparse(args, addopts=addopts)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/config/__init__.py", line 1172, in _preparse
    self.pluginmanager.load_setuptools_entrypoints("pytest11")
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/pluggy/manager.py", line 299, in load_setuptools_entrypoints
    plugin = ep.load()
  File "/opt/python-3.8.5/lib/python3.8/importlib/metadata.py", line 77, in load
    module = import_module(match.group('module'))
  File "/opt/python-3.8.5/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/assertion/rewrite.py", line 170, in exec_module
    exec(co, module.__dict__)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/bluetils/pytest/fixtures.py", line 8, in <module>
    from bluetils.test import helpers
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/assertion/rewrite.py", line 170, in exec_module
    exec(co, module.__dict__)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/bluetils/test/__init__.py", line 16, in <module>
    from .test import BluesysTestCaseMixin
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/assertion/rewrite.py", line 170, in exec_module
    exec(co, module.__dict__)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/bluetils/test/test.py", line 30, in <module>
    from bluetils.conf import settings
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/_pytest/assertion/rewrite.py", line 170, in exec_module
    exec(co, module.__dict__)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/bluetils/conf.py", line 15, in <module>
    CONFIG = settings.CONFIG
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/django/conf/__init__.py", line 76, in __getattr__
    self._setup(name)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/django/conf/__init__.py", line 63, in _setup
    self._wrapped = Settings(settings_module)
  File "/home/jenkins/venv/2/lib/python3.8/site-packages/django/conf/__init__.py", line 142, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
  File "/opt/python-3.8.5/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named 'tests.project'

I will draft a pull request to remove tests and docs from the Python package.

Hi @qbey,
Thanks for opening this feature request and proposing a fix! I'll get back to you next week after I gather feedback from team members to determine if there is consensus to proceed. If we decide as an organization (googleapis) to move forward with this, we would want to apply this change broadly, across most / all the googleapis repos.
cc @busunkim96
cc @tseaver

Hello @parthea,
Thank you for your quick feedback :)
Actually in my PR I initially changed the owlbot.py file which is not required after second thought, only the setup.py's change is required. I updated the commit accordingly (I only saw your answer afterward).

Thank you for the fix :) I will wait for the next release to start using it.
Have a good day!


tree-sitter: incorrect pname for some grammars

Describe the bug
The grammars tree-sitter.builtGrammars.tree-sitter-{org-nvim,markdown-inline} have a wrong pname: org-grammar and markdown_inline-grammar, respectively. While the rest of the grammars would have the correct pname; for example tree-sitter-perl-grammar.
Steps To Reproduce
Steps to reproduce the behavior:
$ nix eval .#tree-sitter.builtGrammars.tree-sitter-org-nvim.pname
"org-grammar"
$ nix eval .#tree-sitter.builtGrammars.tree-sitter-perl.pname
"tree-sitter-perl-grammar"

Additional context
Discovered during #320783.
Seems to have been introduced in #156911.
Argument language seems to never have been used, introduced in cf817454041353a235f02d34245e0873a072f31a.
PR for fixing this coming up.
Notify maintainers

@Profpatsch
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.36, NixOS, 24.11 (Vicuna), 24.11.20240704.1afc544`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.23.1`
 - channels(mightyiam): `""`
 - channels(root): `""`
 - nixpkgs: `/nix/store/xxmxxf2s5c4jq98yk5mr9crm2ig4mv28-source`

Add a :+1: reaction to issues you find important.
@stepbrobd

From these lines:
https://github.com/NixOS/nixpkgs/blob/c9d9fd0c619e2f73f94e4a79abad722e578ff7f1/pkgs/development/tools/parsing/tree-sitter/grammar.nix#L10-L11
https://github.com/NixOS/nixpkgs/blob/c9d9fd0c619e2f73f94e4a79abad722e578ff7f1/pkgs/development/tools/parsing/tree-sitter/grammar.nix#L20
It seems that the correct format should be <language>-grammar.

Recap, because we noticed nuance and changed our minds:
Problem
This is about the pnames of packages in #tree-sitter.builtGrammars (they show up in other paths, as well, by the way).
From the implementation code, it would appear as if their pnames should be <language>-grammar:
https://github.com/NixOS/nixpkgs/blob/c9d9fd0c619e2f73f94e4a79abad722e578ff7f1/pkgs/development/tools/parsing/tree-sitter/grammar.nix#L10-L11
https://github.com/NixOS/nixpkgs/blob/c9d9fd0c619e2f73f94e4a79abad722e578ff7f1/pkgs/development/tools/parsing/tree-sitter/grammar.nix#L20
But in reality, only two of them (#tree-sitter.builtGrammars.tree-sitter-{org-nvim,markdown-inline}) have these pnames. The rest have tree-sitter-<language>-grammar.
And that is because that implementation is called with the value provided as language being "tree-sitter-<language>" except for the two mentioned above. Here that is:
https://github.com/NixOS/nixpkgs/blob/c9d9fd0c619e2f73f94e4a79abad722e578ff7f1/pkgs/development/tools/parsing/tree-sitter/default.nix#L58
Suggested "solution A"
Accept the pname format tree-sitter-<language>-grammar — which all but two packages have. And refactor the implementation to make that appear intentional. Not as a means of covering up the blooper :sweat_smile:, though.


Downloading Schema

Describe the bug
Hi! This might not be a bug, but rather my noobiness. I'm using the local development setup and would like to download the generated GraphQL schema to feed it into GraphQL Code Generator.
To Reproduce
Steps to reproduce the behavior:

I ran in the database: select graphql.rebuild_schema();.
In terminal: curl --request GET 'http://localhost:54321/graphql/v1' \                   -H 'apikey: eyJhbG...' \                                                                              -H 'Authorization: Bearer eyJhbG...'
As result I get: {"code":"PGRST202","details":null,"hint":"If a new function was created in the database with this name and parameters, try reloading the schema cache.","message":"Could not find the public.graphql() function in the schema cache"}

Please give me a hint on how I should form the CURL request to get the schema. It would be great if you could put the required CURL into the API section of the docs.
Versions:

CLI: 1.11.0
Latest images for Docker swarm (downloaded today)

get the schema

A graphql endpoint is self describing through introspection. The easiest way to visualize the schema is to load it up in client. GraphiQL is a popular choice. You can open it against your supabase project using the following html

    <title>GraphiQL</title>
    <link href="https://cdnjs.cloudflare.com/ajax/libs/graphiql/2.0.4/graphiql.css" rel="stylesheet" />

or if you want to stay on the command line, there are some tools that will interact with a graphql endpoint and dump a .graphql schema file. Apollo CLI

Please give me a hint on how I should form the CURL request

The endpoint is a POST endpoint where the body of the request contains at least {"query": "<some query>"} but I'd recommend sticking with one of the tools mentioned ^ initially


Added implementation to parse substring as timestamp field

As a part fo this PR:

Added support to extract the substring from a field and use it as a timestamp.

Example:
let's suppose the timestamp field has the following value:
"lastModifiedDate" : "/Date(2020-10-01)/"
For this use-case we can pass the following parameters:
http.response.record.timestamp.parser = com.github.castorm.kafka.connect.http.response.timestamp.RegexTimestampParser
http.response.record.timestamp.parser.pattern = "yyyy-MM-dd"
http.response.record.timestamp.parser.regex = "(?:\/Date\()(.*?)(?:\)\/)"

Hi @AkshatJindal1,
Thank you very much for your contribution.
I left a review with a proposal for a change, let me know what you think.
Best regards!

Also, I don't know much about this date format, but I just noticed the example given in #69 (which has an offset), and I don't think it would be covered by this implementation, would it?
//Date(1530144000000+0530)/

Thanks @AkshatJindal1 for your new changes, I've left a couple comments in the review.
Best regards and happy new year 🥳

Thanks a lot for your contribution @AkshatJindal1,
I proceed to merge it and release a new version.

Thanks a lot for your contribution @AkshatJindal1,
I proceed to merge it and release a new version.


Support for multiple SCM plugin

I need to use this pull-request plugin in a job where 2 repos need to be checked-out:

one main project whose pull-request branch is the test target
one auxiliary project used for provisioning reason

I'm using multiple SCM plugin with 2 git scm. And as usual, I set the Advanced/refspec of the main project to be +refs/pull/*:refs/remotes/origin/pr/* and leave the auxiliary project's configs intact (so jenkins should always checks out its default branch).
The pull request triggers correctly the job build but the problem is that when checking out the auxiliary project, instead of the default branch, jenkins tries to checks out a pull-request branch as if it's the main project.
# jenkins checks out the main project's pull request branch, OK!
GitHub pull request #439 of commit 8444c7344c4f4a20928b9d8021e387660928a002 automatically merged.
Building in workspace /var/lib/jenkins/jobs/main_project/workspace
 > git rev-parse --is-inside-work-tree
Fetching changes from the remote Git repository
 > git config remote.origin.url<EMAIL_ADDRESS>Fetching upstream changes from<EMAIL_ADDRESS> > git --version
 > git fetch --tags --progress<EMAIL_ADDRESS>+refs/pull/*:refs/remotes/origin/pr/*
 > git rev-parse origin/pr/439/merge^{commit}
Checking out Revision fdaf37ae7048a4c8999471a8dc6df0ea72dbce67 (detached)
 > git config core.sparsecheckout
 > git checkout -f fdaf37ae7048a4c8999471a8dc6df0ea72dbce67
 > git rev-list 6a561e06859de8177ac504cd8c3c05315cedc699
 > git rev-parse --is-inside-work-tree

# jenkins tries to checkout the same, pull-request branch, in the auxiliary project, why?
Fetching changes from the remote Git repository
 > git config remote.origin.url<EMAIL_ADDRESS>Fetching upstream changes from<EMAIL_ADDRESS> > git --version
 > git fetch --tags --progress<EMAIL_ADDRESS>+refs/heads/*:refs/remotes/origin/*
 > git rev-parse origin/pr/439/merge^{commit}
FATAL: Command "git rev-parse origin/pr/439/merge^{commit}" returned status code 128:
stdout: origin/pr/439/merge^{commit}

So from above log, jenkins tries to get origin/pr/439/merge^{commit} of the auxiliary project, but that doesn't exists, and I've never configures it to do that. It should always checks out the default branch of the auxiliary project.
So is it a bug? Or simply because multiple scm plugin is not supported? If the latter one is true can you guys gives me some pointers so that I can try to make a contribution?
Thanks.

+1

I think I have found a workaround on this.  I create a "trigger job" that just listens for changes on the repo I want to trigger on, then create a downstream build job with multi SCM.  It seems to work ok in this scenario, but I'm not really sure why.

+1

We simply stopped using multiple SCM plugin, and use the shell to clone any additional repos we may need. This doesn't scale to all use cases of course as you lose out on other SCM features of Jenkins.


Fix a hanging query shutdown

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr/11353/

https://arangodb.atlassian.net/browse/BTS-126

http://jenkins01.arangodb.biz:8080/view/Java/job/arangodb-ANY-java-driver-matrix/1048/

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr/11353/

Tests blue.

http://jenkins01.arangodb.biz:8080/view/Java/job/arangodb-ANY-java-driver-matrix/1048/

It seems that this one doesn't work for MMFiles case.

@KVS85 : test results seem to indicate something doesn't work w.r.t. MMFiles.
But how can one find the actual errors? I download the build artifcat file but it only seems to contain logs from arangod...
So what actually went wrong here?

@KVS85 : test results seem to indicate something doesn't work w.r.t. MMFiles.
But how can one find the actual errors? I download the build artifcat file but it only seems to contain logs from arangod...
So what actually went wrong here?

These tests log activity directly into console:
...
20:28:16 [INFO] Running com.arangodb.StreamTransactionApiTest
20:28:27 [ERROR] Tests run: 3, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 10.73 s <<< FAILURE! - in com.arangodb.StreamTransactionApiTest
20:28:27 [ERROR] streamTransactionFromDifferentCoordinators[1](com.arangodb.StreamTransactionApiTest)  Time elapsed: 10.013 s  <<< ERROR!
20:28:27 org.junit.runners.model.TestTimedOutException: test timed out after 10000 milliseconds
20:28:27 	at com.arangodb.StreamTransactionApiTest.streamTransactionFromDifferentCoordinators(StreamTransactionApiTest.java:86)
20:28:27
...
20:29:19 [ERROR] Errors:
20:29:19 [ERROR]   StreamTransactionApiTest.streamTransactionFromDifferentCoordinators:86 » TestTimedOut
...

thats a new test not in the master version of the driver, does it actually work ?
Test seems to block on begin not any particular operation ?
@rashtao

There was a bug in the test code, fixed here: https://github.com/arangodb/arangodb-java-driver/commit/74933f0ae9463ee7f3bfe44c9f0e6e74deffda06
http://jenkins01.arangodb.biz:8080/view/Java/job/arangodb-ANY-java-driver-matrix/

driver test blue


Fix problem with IE9

IE9 sees .finally() as a keyword.  This gets fixed with the ".min.js" version of the build, but precludes us from using the non-minified version in development.  Using the ["finally"] form of this function fixes it and all is good with the world.

Could you please:

use single quotes to be consistent with the rest of the file / library?
change the commit message to follow our conventions. Sth like refactor(modal): don't use reserved keywords for IE9 compatibility or similar

And fix the build failure :-) https://travis-ci.org/angular-ui/bootstrap/builds/91434961

This hack is the build failure. I don't know if there are eslint rules (like in jshint) to disable it for some stuff. If not, this won't pass tests ever.

Yeah, I think I'll need to add an exception comment or something.  I'm on it.

Turns out you want the {"allowKeywords": false} option.  Turning that on found two more cases (in collapse.js) where this is happening.  http://eslint.org/docs/rules/dot-notation.html

Was going to suggest you to add it, but I see you did it already.

LGTM - will merge this in shortly.


Bring in more roles

Change-type: minor
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>
Bring the following roles into jellyfish-core:

role-user-community
role-user-operator
role-user-test

@balena-ci I self certify!


🔄 Synced file(s) with realm/ci-actions

Synced local file(s) with realm/ci-actions.

Changed files

Synced local  .github/advanced-issue-labeler.yml with remote  synced-files/.github/advanced-issue-labeler.ymlSynced local  .github/workflows/issue-labeler.yml with remote  synced-files/.github/workflows/issue-labeler.yml

This PR was created automatically by the repo-file-sync-action workflow run #4042629248

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 89.219%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3004

Relevant Lines:
3367

💛  - Coveralls


I cannot install 3.6.0 [help]

i tried
pyenv install 3.6.0

BUILD FAILED (OS X 10.12.1 using python-build 20160602)

Inspect or clean up the working tree at /var/folders/vg/zpr7sfhx309cpw3lxp4bzb840000gn/T/python-build.20170202175142.30016
Results logged to /var/folders/vg/zpr7sfhx309cpw3lxp4bzb840000gn/T/python-build.20170202175142.30016.log

Last 10 log lines:
  File "/private/var/folders/vg/zpr7sfhx309cpw3lxp4bzb840000gn/T/python-build.20170202175142.30016/Python-3.6.0/Lib/ensurepip/__main__.py", line 4, in <module>
    ensurepip._main()
  File "/private/var/folders/vg/zpr7sfhx309cpw3lxp4bzb840000gn/T/python-build.20170202175142.30016/Python-3.6.0/Lib/ensurepip/__init__.py", line 189, in _main
    default_pip=args.default_pip,
  File "/private/var/folders/vg/zpr7sfhx309cpw3lxp4bzb840000gn/T/python-build.20170202175142.30016/Python-3.6.0/Lib/ensurepip/__init__.py", line 102, in bootstrap
    _run_pip(args + [p[0] for p in _PROJECTS], additional_paths)
  File "/private/var/folders/vg/zpr7sfhx309cpw3lxp4bzb840000gn/T/python-build.20170202175142.30016/Python-3.6.0/Lib/ensurepip/__init__.py", line 27, in _run_pip
    import pip
zipimport.ZipImportError: can't decompress data; zlib not available
make: *** [install] Error 1 ```

<bountysource-plugin>
---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/41585128-i-cannot-install-3-6-0-help?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github).
</bountysource-plugin>

I'm not OSX user, though it seems you are missing zlib-devel. You have to get it before attempting to build python using pyenv.
The one solution I found is to run
xcode-select --install

Yeah, I guess he doesn't have zlib header files in /usr/include. xcode-select --install will solve.

@Qtimer actually their page already classifies this as a common build problem.
See: https://github.com/yyuu/pyenv/wiki/Common-build-problems
#UncleGoogle


"Actual" current directory and PowerShell current directory mismatch when using .NET BCL methods

When cd is used, the actual current directory of the process is not changed. This means that plain BCL methods still see the old current directory.
This creates an inconsistency with other ways of executing commands:

PowerShell cmdlets: $pwd is honored
External commands: $pwd is honored
.NET methods: $pwd is not honored

Steps to reproduce
PS C:\Users\Andrea> cd C:\temp
PS C:\temp> echo example1 > example1.txt
PS C:\temp> [IO.File]::WriteAllText('example2.txt', 'example2')

Expected behavior
C:\temp contains example1.txt and example2.txt.
Actual behavior
C:\temp only contains example1.txt. example2.txt is created in C:\Users\Andrea.
Environment data
> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      5.1.15063.0
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.15063.0
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Btw the same happens with PowerShell v6.0.0-alpha.17.

If I have process with multiple Runspaces and each one have different current location. Which one $pwd should be honored by .NET methods?

Possible solutions I can think of:

Leave the behavior as it is
Take a process-wide lock when BCL methods are directly invoked (except for cmdlets), if multiple runspaces exist (I don't know to what extent it would impact the performance)
Switch the current directory to a dummy, default directory whenever the number of runspaces > 1 (to make the mistake clear)
Only honor the $pwd of the main runspace

@antiufo Process-wide lock will not going to work, unless everyone agreed to use it and performance implication are not very good. Also, not every process host PowerShell just for sake of hosting PowerShell. Process may have its own work to do and unlikely be happy if you just change its working directory to dummy one. And as Runspaces can come and gone at process wish, it may be hard to say which one is main.
Besides, IMHO, there is good enough solution for this already:
PS C:\Users\PetSerAl> function ConvertTo-FileSystemPath {
>>     param($PSPath)
>>     $PSProvider = $null
>>     $Result = $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath($PSPath, [ref]$PSProvider, [ref]$null)
>>     if($PSProvider.ImplementingType -eq [Microsoft.PowerShell.Commands.FileSystemProvider]) {
>>         $Result
>>     } else {
>>         Write-Error 'Not filesystem provider'
>>     }
>> }
PS C:\Users\PetSerAl> New-PSDrive Win FileSystem C:\Windows | Out-Null
PS C:\Users\PetSerAl> ConvertTo-FileSystemPath qwe
C:\Users\PetSerAl\qwe
PS C:\Users\PetSerAl> cd \
PS C:\> ConvertTo-FileSystemPath asd
C:\asd
PS C:\> ConvertTo-FileSystemPath Win:\zxc
C:\Windows\zxc
PS C:\> ConvertTo-FileSystemPath HKLM:\SOFTWARE
ConvertTo-FileSystemPath : Not filesystem provider
At line:1 char:1
+ ConvertTo-FileSystemPath HKLM:\SOFTWARE
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,ConvertTo-FileSystemPath

This behavior is by design and it's been this way since version 1.0. We considered options like those proposed above but none of them were viewed as satisfactory for various reasons. Acquiring a global lock, then setting the process pwd, then calling a method is a lot of overhead when the majority of .NET APIs don't depend on pwd. Doing this also greatly complicates a host application's (e.g. exchange UI, VMM console, etc.) use of the current working directory since it can't depend on PWD anymore. The workaround for .NET APIs that take paths is to, as mentioned in another comment, resolve the path before passing It to the API which need not be onerous e.g. WriteAllText("$pwd\fileincurrentdir.txt")

Too bad that there's no good solution, because the behavior is a perennial pain point.
The "$pwd\fileincurrentdir.txt" shortcut is alluring, but - aside from assuming that the current location is a filesystem location (an assumption that will often, but not always hold) - it fails with current filesystem locations based on PS drives.
Remedying that doesn't take much extra effort, fortunately:
[io.file]::WriteAllText($PWD.ProviderPath + '/fileInCurrentDir.txt', 'foo')

If you also want to guard against the current location being on a different provider's drive:
[io.file]::WriteAllText((Get-Location -PSProvider FileSystem).ProviderPath + '/fileInCurrentDir.txt', 'foo')

@PetSerAl's ConvertTo-FileSystemPath helper function is the most robust and flexible approach, but anything that doesn't ship with PowerShell is unlikely to see widespread adoption.
However, what his function does could be folded into the standard Convert-Path cmdlet - see #7197

I think you've missed a use-case, and that's libraries that don't know anything about PS.
For example, Terminal.Gui has at least three APIs (related, but still - FileDialog, OpenDialog, SaveDialog) that initialize the displayed folder from [system.environment]::currentdirectory. So I have to set it to what PS thinks the working directory is before initializing any of those APIs.


How to compute running time as shown in your paper

Hi, I want to compute the running time of different methods which are similar to table 6 in your paper.

I have a few problems as follows:

For the other baselines, did you re-compute the running time by yourself or you get the result directly from the KITTI benchmark website? I can not get 0.08s for Sparse-to-Dense. Do you have any suggestions?
In your provided source code, did you implement this function?

Hi @TruongKhang，

I directly quote the run-time results from KITTI. You can find  0.08s for Sparse-to-Dense (gd) on KITTI website (current ranked at 42).
I implement the funtion to compute time in trainers/DepthTrainer.py(Line 210, 221, 236, and 287 ).

@anglixjtu , so I have a question about the results shown on the KITTI website. Because I don't submit the results on the KITTI website, I want to know how the running time is computed to show on the website. The authors need to submit their result or they have to run code on the KITTI benchmark server?

@TruongKhang KITTI does not provide any offical tools for computing running time or test the running time online. The authors just compute the time on their own and submit the results to KITTI.

thank you @anglixjtu , I understand. Maybe because of the different Platform, so I can not get the 0.08s for Sparse-to-Dense. I got 0.01-0.02s, that's why I'm so surprised!

Hi @TruongKhang, I'm not sure if you used the same code to measure the running time, but I just want to tell you that there is a problem with my code so as not to affect your experiment.
According to a kind pull request from jaehyuck0103, the runtime should be measured at the end of the cuda operations by using torch.cuda.synchronize(). The problem is now fixed by adding cuda synchronize code.


Installation instruction updates

Reference Issues or PRs

What does this implement/fix?
Put an x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features to not work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[x] The content is written with plain language (where relevant).
[x] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[x] All links describe where they link to (for example, check the Nebari website).
[x] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

@trallard merge conflicts have been resolved. Ready for review.

@kcpevey I've left some small typo-fix suggestions, other than that, this looks good to me!

Thanks @pavithraes! Now I think we're just waiting on final review from @trallard

I only realised this is waiting for my review - will do this tomorrow

I had a quick look, and now it seems these instructions and the ones available in https://www.nebari.dev/docs/get-started/installing-nebari and the various cloud providers conflict with each other - Asmi and @iameskild  updated these docs to match the newest CLI.
Can rebase main and/or sort through the conflicts (see above

@kcpevey I'm happy to take over this issue and push up commits to resolve conflicts, would you be fine with that?

@pavithraes yep. she's all yours! :)

Thank you, @kcpevey!
@trallard This PR is ready for your final review. :)


Extracting resource name/type/labels for webhook

Hi, I wanted to know if its possible to extract some parameters from the resource I have the listener setup to.
e.g callback for the resource type/name/labels/action/namespace and use them as parameters to the executed webhook(url/slack..)?

Not the maintainer, but I'm working on a PR for this. Right now I'm sending the whole resource as json. What is your usecase and does adding the entire resource to the json send to the webhook help?