TCP/UDP Gateway

Hello
I want to use spring-cloud-gateway as my load balancer. The problem is that my clients have https connections and some of them have tcp/udp connections.
Can you please let me know if spring-cloud-gateway supports https and tcp/udp connections?
Best regards

HTTPS should be supported by spring framework in general. I'm unsure if a TCP/UDP gateway is in scope. We'll keep this issue open until we decide one way or another.

Any further information about this feature?

No, this will not be part of the initial release.

What's the plan to support HTTPS ?

@harryyu1018
HTTPS is already supported via Spring WebFlux and Spring Boot
here is an example
https://github.com/making/pks-master-gateway/blob/master/src/main/resources/application.properties#L7-L14

@making  Thank you.

Is it planned in future ?

@spencergibb Thank you

I hope Spring cloud gateway supports tcp protocol.

@spencergibb  the rsocket is tcp socket support module?

No it is not. It is only for rsocket not TCP in general

Does gateway-rsocket have a release plan? @spencergibb

Later this year

@spencergibb Now, Spring boot 2.2.x is released, I want to known the plan to support for TCP.

Any further information about this feature?

Any further information about this feature?

No

any good news, for implementing this feature

any good news, for implementing this feature

No

No

now,it's support tcp or not?

Not while this is still open

hello, is there any new about tcp/udp gateway?

it's been a long long time!

hello, is there any new about tcp/udp gateway?

We do use spring-cloud-gateway as our reverse proxy and would have a need to also handle TCP connections as well (mqtt broker bridge, websockets do not work here).
What is the status of this? @spencergibb

现在都没有一个好的解决方案
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2022年2月7日(星期一) 下午2:48
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [spring-cloud/spring-cloud-gateway] TCP/UDP Gateway (#66)
我们确实使用 spring - cloud - gateway 作为反向代理,并且还需要处理 TCP 连接(mqttbroker bridge 、 websockets 在这里不起作用）。
这个现状如何？@ spencer gib
—
直接回复此邮件，在 GitHub 上查看或取消订阅.
GitHub Mobile 上的分流通知监察办或机器人.
@.***与>

hello, there doesn't seem to be a good solution.

hello, is there any new about tcp/udp gateway?

no
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2022年9月8日(星期四) 上午9:19
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [spring-cloud/spring-cloud-gateway] TCP/UDP Gateway (#66)
hello, is there any new about tcp/udp gateway?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

we also need this one, please consider implement this

We have no immediate or medium term plan to do this. If we do decide, it will not be until next year

这是来自QQ邮箱的假期自动回复邮件。

您的邮件我已收到，我会尽快阅读并回复您。祝您工作顺利，心情愉悦！

Tomorrow a have a demo of my project that I was doing 1 year to business, but only today our team has understood that we cannot route TCP with sping-cloud-gateway. Looks like we are f***d...

Any follow-up?

https://github.com/spring-cloud/spring-cloud-gateway/issues/66#issuecomment-1240852870

I also want to know the answer

Yes it will be interesting that we can connect to mongo/rabbitmq/pg etc... behind the gateway

Why can you connect to mongodb, rabbitmq .. etc directly? Why do you have to connect via spring cloud gateway?

Why can't you connect to mongodb, rabbitmq .. etc directly? Why do you have to connect via spring cloud gateway?@hhfrancois

I use api-gateway for access to microservices in my cluster.
Usually UI target microservices by their names.
So all my microservice are in the cluster.
For example endpoint GET(/user) exposed by the microservice user-srv will be targetable by the UI with the url /user-srv/user
The API gateway redirect the request to the microservice, remove the extra path (/user-srv) and the microservices serve the response.
Everything works fine.
My microservice user-srv is in cluster with the DB. So when the microservice want to access to the DB, no problem, it can use the service name provide by the ingress.
But during the developpement maybe I would like to put user-srv outside the cluster. In this case it cannot target the database by the service name for sure. So I need to expose the db port for target it. That works. But I guess It should be interesting to target the api-gateway its port is always binded and like that, no need to bind port of db...
I hope I was clear.
In fact I developing an UI for API gateway, with dynamic route locator. I doing something like APISIX but based on spring cloud gateway.
If the community is interesting I can show what I did.

@hhfrancois understood.
i'm interested , please share it

@Divine1. I will, its too early ofr the moment
We cannot paste picture here ?
screenshot

@hhfrancois  you can

@Divine1 Thx

Does Spring cloud Gateway load balancer now support TCP client connections?

Any progress here? Thanks.


Implement Cose_Mac0

Fixed by #80 (though many small fixes will be needed before Mac0 is complete).


Can't view the variable from variable explorer

Description
What steps will reproduce the problem?

What is the expected output? What do you see instead?
Please provide any additional information below
File "/usr/lib/python3/dist-packages/spyder/widgets/variableexplorer/collectionseditor.py", line 449, in createEditor
if not editor.setup_and_check(value, title=key, readonly=readonly):
File "/usr/lib/python3/dist-packages/spyder/widgets/variableexplorer/arrayeditor.py", line 619, in setup_and_check
self.data.flags.writeable = True
ValueError: cannot set WRITEABLE flag to True of this array
Version and main components

Spyder Version: 3.2.6
Python Version: 3.6.7
Qt Versions: 5.12.1, PyQt5 5.12 on Linux

Dependencies
pyflakes >=0.6.0 :  2.1.1 (OK)
pycodestyle >=2.3:  2.5.0 (OK)
pygments >=2.0   :  2.3.1 (OK)
pandas >=0.13.1  :  0.24.0 (OK)
numpy >=1.7      :  1.16.3 (OK)
sphinx >=0.6.6   :  1.8.4 (OK)
rope >=0.9.4     :  0.12.0 (OK)
jedi >=0.9.0     :  0.13.3 (OK)
psutil >=0.3     :  5.5.1 (OK)
nbconvert >=4.0  :  5.4.1 (OK)
sympy >=0.7.3    :  None (NOK)
cython >=0.21    :  None (NOK)
qtconsole >=4.2.0:  4.4.3 (OK)
IPython >=4.0    :  7.3.0 (OK)
pylint >=0.25    :  2.3.0 (OK)

This is fixed in our latest version. Please update by opening the Anaconda Prompt and running there
conda install spyder=3.3.4


$ in Swift code incorrectly highlighted as an error

(Original issue 1080 created by qwzybug on 2015-01-26T06:19:45.253711+00:00)
$ is an acceptable character in Swift variable names, and is also used in the automatically bound variable arguments to blocks. It is also used for automatically generated variables in the REPL.
#!swift

[3, 1, 4, 1, 5].sorted{ $0 < $1 }

var $x = "foo"

1> 5 * 2
$R0: Int = 10
2> $R0 / 2
$R1: Int = 5

(Original comment by birkenfeld on 2015-01-26T06:25:47.491712+00:00)
Thanks for the report. Is $ allowed only at the start of names, as your example suggests, or in any place?

(Original comment by qwzybug on 2015-01-26T06:40:04.375758+00:00)
I believe $ is allowed anywhere in the variable name.
Here's the specification of valid identifiers in the Swift grammar: https://developer.apple.com/library/prerelease/ios/documentation/Swift/Conceptual/Swift_Programming_Language/LexicalStructure.html#//apple_ref/swift/grammar/identifier
identifier → identifier-head[identifier-characters]
identifier → `identifier-head[identifier-characters]`
identifier → implicit-parameter-name
identifier-list → identifier  identifier,identifier-list
identifier-head → Upper- or lowercase letter A through Z
identifier-head → _
identifier-head → U+00A8, U+00AA, U+00AD, U+00AF, U+00B2–U+00B5, or U+00B7–U+00BA
identifier-head → U+00BC–U+00BE, U+00C0–U+00D6, U+00D8–U+00F6, or U+00F8–U+00FF
identifier-head → U+0100–U+02FF, U+0370–U+167F, U+1681–U+180D, or U+180F–U+1DBF
identifier-head → U+1E00–U+1FFF
identifier-head → U+200B–U+200D, U+202A–U+202E, U+203F–U+2040, U+2054, or U+2060–U+206F
identifier-head → U+2070–U+20CF, U+2100–U+218F, U+2460–U+24FF, or U+2776–U+2793
identifier-head → U+2C00–U+2DFF or U+2E80–U+2FFF
identifier-head → U+3004–U+3007, U+3021–U+302F, U+3031–U+303F, or U+3040–U+D7FF
identifier-head → U+F900–U+FD3D, U+FD40–U+FDCF, U+FDF0–U+FE1F, or U+FE30–U+FE44
identifier-head → U+FE47–U+FFFD
identifier-head → U+10000–U+1FFFD, U+20000–U+2FFFD, U+30000–U+3FFFD, or U+40000–U+4FFFD
identifier-head → U+50000–U+5FFFD, U+60000–U+6FFFD, U+70000–U+7FFFD, or U+80000–U+8FFFD
identifier-head → U+90000–U+9FFFD, U+A0000–U+AFFFD, U+B0000–U+BFFFD, or U+C0000–U+CFFFD
identifier-head → U+D0000–U+DFFFD or U+E0000–U+EFFFD
identifier-character → Digit 0 through 9
identifier-character → U+0300–U+036F, U+1DC0–U+1DFF, U+20D0–U+20FF, or U+FE20–U+FE2F
identifier-character → identifier-head
identifier-characters → identifier-character identifier-charactersopt
implicit-parameter-name → $decimal-digits

(Original comment by birkenfeld on 2015-01-26T06:51:10.617777+00:00)
Hmm, if I read that correctly it only allows "$" in the form of the "implicit-parameter-name" (which is already recognized by the lexer). None of the other ranges include $...
Of course, if it appears in the wild I'd rather not second guess and ignore the specs.

(Original comment by qwzybug on 2015-01-26T07:02:12.453930+00:00)
I just noticed the same thing. Very strange. The behavior appears to vary across how you're using Swift:

In a playground: $s seem to be perfectly OK to use in identifiers.
In the REPL: variables starting with $ can be defined, but accessing them doesn't work. Variables with $ in the middle or at the end work fine.
Executing Swift code from a file: variables starting with $ cause very strange behavior. Variables with $ in the middle or at the end work fine.

I've been using pygments to highlight REPL sessions, where usage of $R0 variables is relatively common, so it would make my life easier to allow it. But it does seem like those aren't really variables according to the language spec.


Per-branch data no longer being created by cli

Under .nhost/data I no longer have folders for each of my git branches. Instead, I only have a nogit folder.
I've tried deleting the .nhost folder and running nhost down and nhost up on main and on other branches, but no new folders are created. Per discussion with Nuno and David in Discord, this may be a bug with the cli. This is in a Git repository that has not changed and has been connected for many months to 2 different cloud projects (and that deploys config and changes successfully).
Cli version: v1.5.2

Thanks for reporting, should be fixed in the next release.


using sorl-thumbnail and easy-thumbnails side-by-side

It would be really useful to be able to use both  easy-thumbnails and sorl-thumbnail in the same django project. 3rd-party apps may depend on one or the other and currently it is just impossible to install both into the same project.
As far as I can tell the only reason the two don't work together is because of variable name clashes.
Is this something both projects would consider addressing?
Problem: both libraries use the same names for things
In templates:
{% load thumbnail %}
{% thumbnail [...] %}

And in settings.py both use the same THUMBNAIL_ prefix:
THUMBNAIL_PREFIX = '...'

Suggested way to make both play nice together
My suggestion would be that both libraries switch to prefixing their settings uniquely:
SORL_THUMBNAIL_XXX
EASY_THUMBNAILS_XXX

For the transition both libraries could add a setting:
SORL_THUMBNAIL_NEW_SETTINGS_PREFIX = False
EASY_THUMBNAILS_NEW_SETTINGS_PREFIX = False

That reads the settings with the new prefix and outputs a deprecation warning if not set to True.
After a few releases the default could be set to True.
Similarly for the templatetags. Both libraries could change the templatetags to use the prefix:
{% load easy_thumbnails_tags solr_thumbnail_tags %}

{% sorl_thumbnail [...] %}
{% easy_thumbnail [...] %}

Both libraries would keep the thumbnails version around, but it would issue a deprecation warning. If both are installed in the same project, the one first in INSTALLED_APPS would be used when using the old name.

@mariocesar If we were to provide a PR for such a change, would you merge it?

@stefanfoulis Sure

Either, will be good to have an SORL_THUMBNAIL_TEMPLATETAG_NAME="thumbnail", though I will really see as beneficial to avoid a backward problem.

(y)

For anyone coming across this post, this works for me in the meantime: https://stackoverflow.com/a/35166302/396300

Thanks to this file https://github.com/jazzband/sorl-thumbnail/blob/master/sorl/thumbnail/templatetags/sorl_thumbnail.py , it is now possible to use {% load sorl_thumbnail %} instead of {% load thumbnail %}.
I don't think the remaining conflicts have been fixed, though.


No module named SpatialTransform

When I install the latest version of bvhio via pip, a ModuleNotFoundError occurs. Also, it seems like glm is not listed in the dependencies.

As a workaround you can use "pip install -r requirements.txt" to install the requirements. I gonna check this out


[feature] supports the enum/in can use object to transform value

these PR MUST BE applied first:

https://github.com/balderdashy/anchor/pull/67
https://github.com/chriso/validator.js/pull/336

see https://github.com/balderdashy/waterline/issues/52
attributes: {
  state: {
    type: 'int',
    enum: {'pending':2, 'approved':3, 'denied':4]
  }
}

@snowyu can you refresh me on what this does? What is the purpose of the object notation?

This an unit test only. the real code is https://github.com/balderdashy/anchor/pull/67.
Last year I said  the chriso (validator author) merged partly. so you have to add or enhance the interface( Anchor.match.type() to pass the additional rule information.
and the feature issue is https://github.com/balderdashy/waterline/issues/52.

Changing the data based on a validation isn't really the goal of the validation system in Waterline. Custom validations or lifecycle callbacks could be used for this currently.


Missing dist/incremental-dom-cjs.js file on npm published version 0.4.0

Hi,
the file dist/incremental-dom-cjs.js is missing on npm.
require('incremental-dom') fails as it tries to load index.js.
A problem with the prepublish task?
Thanks

Not sure how that happened. Seems like it generates it correctly locally. I'll publish a new version.

Published 0.4.1, which I did confirm has all the expected files. Will leave this issue open for a bit while I investigate how it might have happened.
Thanks for the heads up!

Haven't figured out what happened, but I didn't encounter this issue in the latest release. Will keep an eye out for it though.


This reference value, b_unet, b_vae, b_clip = -6131.5400, 17870.7051, -2097.8596 if is_clip_v1 else -8757.5630, is based on a specific standard or benchmark?

I want to know how this standard value was obtained
thks :)

its normalized so the SDv1 checkpoint has the metric (0000/0000/0000)


Incontinence in statistics summaries

I am currently in the process of adapting the Home Assistant integration to the new mytoyota version.
I noticed that the statistics summaries are inconsistent.
According to the current date (08 January 2024), the monthly and annual statistics should be the same. However, for the same period from 01.01 to 08.01.2024 I get different values for month (e.g. 90km distance) and year (e.g. 179km distance).

Fyi @GitOldGrumpy :wink:

Ill take a look later, if I can find a few spare minutes.

The code always expects there to be multiple months and it doesn't work anyway as the first month is always added twice.


404 at /build/bake/

I found a broken link : https://docs.docker.com/build/bake/
I arrived on this page through: https://github.com/docker/buildx/blob/master/docs/reference/buildx_bake.md

Thanks for sharing the issue. As this content is maintained in an upstream repo, please help create an issue in the source repo to fix https://github.com/docker/buildx/blob/master/docs/reference/buildx_bake.md


ref #1767 [Docs] Fix dropdown filter hiding when clicking on select2 search-field

Fix dropdown-filter hiding when clicking on select2 search-field
Fixes #1767
Proposed Changes

Add click event to select2 search-field which stops propagation.

still facing the same issue in orchid/platform 10.25


Iis2dlpc move drdy_int and tap configuration into device tree

(Depend on #30317).
Move some informations from KConfig to Device Tree..

drdy_int.
pulse (renamed to tap).
.
As agreed in #30317 the name has been changed from 'pulse' to 'tap' and the tap-ths property value is used to enable/disable the feature for each axis (> 0 enables it). The dts binding is describing each property with particular care for the default value..

Tested on x_nucleo_iks02a1 shield.

Added the DNM because this PR depends to #30317.

Rebased it soon after #30317 has been merged on master. And removed DNM.

Rebased it soon after #30317 has been merged on master. And removed DNM.

@mhelm Do you happen to have some tme to take a look at this one today?

@mhelm Do you happen to have some tme to take a look at this one today?


[C++] CSV writer's WriteTable failed with: Failed casting from large_string to string: input array too large

Describe the bug, including details regarding any error messages, version, and platform.
For tables which contains arrow::LargeStringArray which further contains long string elements, the following code:
arrow::csv::WriteCSV(*table, options, ofp_.get())

failed with the error: Failed casting from large_string to string: input array too large.
Component(s)
C++

Issue resolved by pull request 40271
https://github.com/apache/arrow/pull/40271


fix opacity adjustment and set it to low non-zero value

This should fix #279 and could potentially solve #271 .
This lets the selected layer barely visible (opacity = 0.2), which would not lead (I think) to the user thinking the original labels layer was broken, but still hide it quite a bit so that the clustering result does not blend much with the labels colors.
The best option in my opinion is to interactively hide and show it, which is already written in the documentation under Manual clustering.
Diving deep into git blame, it seems this was set by @thorstenwagner here, which then was later updated to setting opacity to 0. @thorstenwagner,  would it be OK for your use cases to let a small opacity like 0.2?
Best,
Marcelo

@zoccoler,
I have one question before merging: does the visbility actually change the speed of the plugin? I could imagine that by making the layer visible napari would need some more time to render the image or is this not the case? Unfortunately I do not have much time for testing this week but will reply once I have had the chance to.

@Cryaaa I could imagine that by making the layer visible napari would need some more time to render the image or is this not the case?

I just tested this a bit on my machine and it did not seem to make a difference, even though I would agree, that one might expect a difference.

@Cryaaa I could imagine that by making the layer visible napari would need some more time to render the image or is this not the case?

I just tested this a bit on my machine and it did not seem to make a difference, even though I would agree, that one might expect a difference.

Thanks for checking this so soon! I am fine with merging this from my end maybe @thorstenwagner can chime in over the next few days if he is also fine with this? otherwise I'll merge it.

I totally forgot to check that. Will do it today.

Hi, from side everything still works as it should be. Probably because I set the opacity to zero when I load the the plugin:
https://github.com/MPI-Dortmund/napari-tomotwin/blob/main/src/napari_tomotwin/load_umap.py#L79

Perfect! Then I'll merge this fixing #279


Org Page Content Epic: As org staff I can go to the dashboard and input text that will update the hero and about us Part 2

Blocked by https://github.com/rubyforgood/pet-rescue/issues/479 but if you want to take on both at the same time, feel free!
This issue is to add the views that enable an org staff to edit and update their page text.
Acceptance Criteria

Add new link 'Page Text' in the dashboard navbar under Default Pet Tasks link that goes to the PageText edit action/view that renders the form (similar to how clicking Settings takes you to the edit form for the org profile)
Create form with inputs for Hero Text and About Us
It should look similar to below, including header and breadcrumb
Update the hero text and about us section on the root page so that if hero text or about us text is not nil for PageText it renders the org's custom text, otherwise renders the existing default text.

@kasugaijin can you include a screenshot of where "Hero" and "About Us" text shows up? I see a couple places that have "About Us"

@nsiwnf yes good point...does this help enough?

@phonghpham I need to add you here so I can assign you. Done.

@phonghpham I for some reason could not assign you so assigned myself. But this ticket is all yours to do!

Copy that!


php -e causes Invalid address/ Address not supported: No error information, when yasd is enabled

Please answer these questions before submitting your issue. Thanks!

What did you do? If possible, provides a simple script and debug step to reproduce the problem.
I run

php -e test.php

test file: test.php
<?php echo "test";

What did you expect to see?
I expected to see no problems and it to work without problems.

What did you see instead?

Invalid address/ Address not supported: No error information

What is your machine environment used (show your uname -a & php -v & gcc -v) ?

~ # uname -a
Linux 3ea8ae99125a 4.19.121-linuxkit #1 SMP Tue Dec 1 17:50:32 UTC 2020 x86_64 Linux

~ # php -v
PHP 7.4.15 (cli) (built: Feb 18 2021 00:45:01) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
    with Yasd v0.3.7, Our Copyright, by codinghuang

~ # gcc -v
Using built-in specs.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/libexec/gcc/x86_64-alpine-linux-musl/10.2.1/lto-wrapper
Target: x86_64-alpine-linux-musl
Configured with: /home/buildozer/aports/main/gcc/src/gcc-10.2.1_pre1/configure --prefix=/usr --mandir=/usr/share/man --infodir=/usr/share/info --build=x86_64-alpine-linux-musl --host=x86_64-alpine-linux-musl --target=x86_64-alpine-linux-musl --with-pkgversion='Alpine 10.2.1_pre1' --enable-checking=release --disable-fixed-point --disable-libstdcxx-pch --disable-multilib --disable-nls --disable-werror --disable-symvers --enable-__cxa_atexit --enable-default-pie --enable-default-ssp --enable-cloog-backend --enable-languages=c,c++,d,objc,go,fortran,ada --disable-libssp --disable-libmpx --disable-libmudflap --disable-libsanitizer --enable-shared --enable-threads --enable-tls --with-system-zlib --with-linker-hash-style=gnu
Thread model: posix
Supported LTO compression algorithms: zlib
gcc version 10.2.1 20201203 (Alpine 10.2.1_pre1)

Figured it out... Seems that yasd does not support hostnames, only ips...

@juslintek You can use the latest yasd code of the master branch, and then use the init_file file #98 #97

I personally prefer to exclude debugging deps from code unless they inject from packages and latch onto hooks of framework. I do it like this on entrypoint.sh:
DEBUG_LISTENER_IP=$(ping -c 1 $DOCKER_HOSTNAME | grep "): " | sed -E 's/.*\((.*)\).*/\1/g')

sed -i "s/yasd.remote_host=.*/yasd.remote_host=${DEBUG_LISTENER_IP}/" /usr/local/etc/php/conf.d/20-yasd.ini

But I actually still have a problem of phpstorm ide not stopping on breakpoint...

Have you configured path mapping?

@huanghantao yes path mapping is configured. :-)

Any news on this one?

@huanghantao hasn't resolved it, configuration of yasd.remote_host is not accepting fqdn or hostname it must be IP.
So @janskopljak make sure you use IP or do As I did, resolve hostname to IP. But still for me it seems like there are other issues on top from yasd and docker side, because it cannot reach phpstorm.

@juslintek I acutally tried with the static ips, but I just get [yasd] connection refused issues no matter what I try

@janskopljak have you used -e flag on PHP. Basically, I can debug single file executions, but cannot debug swoole server. :-)
I run PHPUnit or any command-line tools with PHP -e flag and have these settings set and it works:

Do not run in it debug mode, just make sure "Start Listening for PHP Debug Connections" is enabled

Yasd is a very bad debugger :-) When you try to evaluate it with some value it allows you to do that only for a single run. After that, it returns null all the time. Or when you try to open larger data sets it crashes.
And because of that, I will quit swoole. It is too much hassle to debug, if you have a complex application it really big overhead to not be able to debug your code properly.


Question about "Evaluation Conditions" in paper

Hi, thanks for your awesome work. I have noticed that you said "DACS was evaluated as an average of three runs" in "Evaluation Conditions". Dose it mean that the reported result is the average of the best results from three runs? Or dose it mean that the reported result is the average of all results from three runs and each run contains many results?

Each run only has one result, the one from when training is completed after 250k iterations. The reported result is the average of three runs like this.
This is in contrast to several other works that we are comparing ours against, which actually use early stopping, i.e. report the best result from the course of the training, something we believe is not a fair evaluation.
I hope this answers your question! :)

Thanks for your reply! :-)


Update google-cloud-bigquery to 1.135.3

Updates com.google.cloud:google-cloud-bigquery from 1.122.2 to 1.135.3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-bigquery" } ]

labels: library-update, semver-minor

Superseded by #240.


Revive legacy contentqa engine

Revive the deployment and configuration of the legacy contentqa rails engine, inside the platform app.  Also improve some of the configuration that used to be there.
The contentqa engine is irrelevant to most installations and is disabled by default.

@moltude: I have changed the role name to loadbalancer in https://github.com/dpla/automation/pull/203/commits/24e535462698f4d373b7789ad72807f926b2e2cd but have left the name of dev_loadbalancer.yml alone, per https://github.com/dpla/automation/pull/203#discussion_r84755412

This looks 👍 to me and I'm ready to merge it in


Create releases via CircleCI

Currently, Bitbucket Pipelines generate releases for new tags pushed to the repository. I need to update the CircleCI configuration to perform this release, now that we use GitHub.

Done.


Use EXT_mesh_features to indicate batch IDs

I'm going to use upgrade-feature-metadata as a staging branch for the metadata extension changes, since it's going to involve a larger overhaul and I want to make changes as incrementally as possible.
In this PR, batch IDs are stored EXT_mesh_features instead of an EXT_feature_metadata extension on the mesh primitive. This change only affects b3dm / pnts; a separate class should be written to convert EXT_feature_metadata to EXT_mesh_features for glTFs.
I also prematurely renamed the file before deciding to split up PRs. In this PR it doesn't handle EXT_structural_metadata yet.

This looks good to me @j9liu. Do you want me to merge PRs like this into the staging branch, or do you want to do it yourself when you're ready?

I'll merge this one since I have your approval, but in the future you can merge!


Create project directory

Create project directory from the given name.
path should be ./<project_name> from the location that supaquick was ran from.

21f7091 closes this issue


add Array classifications and its Applications

Classification of Arrays based on Dimensions is described along with this Complexity for some Array operation and some Applications of Arrays are also mentioned.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@amit2307 This article has been flagged as plagiarism. Please reword in your own words and resubmit.


How to snap non-redistributable, portable Windows application?

The target application provides a portable executable that doesn't require installing, how to snap it?

You can use the "RUN_EXE" environment variable for this. Just add the exe to your snap and point RUN_EXE to that location.
This is what the notepad-plus-plus snap does. Taqi recently ported it to sommelier-core: https://github.com/mmtrt/notepad-plus-plus/blob/master/snap/snapcraft.yaml
docs contributions are welcome! :)

You can use the "RUN_EXE" environment variable for this. Just add the exe to your snap and point RUN_EXE to that location.
This is what the notepad-plus-plus snap does. Taqi recently ported it to sommelier-core: https://github.com/mmtrt/notepad-plus-plus/blob/master/snap/snapcraft.yaml
docs contributions are welcome! :)

Apologies for the ignorance, I finally am able to come back to this.
My implementation is to download the executable in runtime using a launcher script, then set the RUN_EXE environment variable to the path
https://github.com/brlin-tw/xnote-stopwatch-snap/blob/4ee68ea0e8b8b6d6ddaa13b44ece7b54bce8a183/snap/local/launchers/xnote-stopwatch-launch
https://github.com/brlin-tw/xnote-stopwatch-snap/blob/4ee68ea0e8b8b6d6ddaa13b44ece7b54bce8a183/snap/snapcraft.yaml#L110
Not sure how to improve the documentation, though, my thought is to simply enumerate different kinds of applications and their corresponding snap recipe in the Example Snaps section of the README for reference.  I've quickly made up #30 for the idea.


Refresh the compiler versions for Python Linux builds

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


Tracker logic

Context
The consumer now processes all events and turns them into validator_metadata_events when it thinks there are validators attached to a beacon node. Sentries will try to redo succesful handshakes & metadata exchanges every epoch, which means that in the ideal case we have data points on each beacon node every epoch.
The next step is to build a view on live validators based on these events. The table will look like this (stored in sqlite DB inside consumer)

PeerID
ENR
Multiaddr
IP
Port
LastSeen
LastEpoch
PossibleValidator: bool
AverageValidatorCount: int
NumObservations: counter

Bold properties are ones that we have to derive. The rest can just be extracted from the metadata events. Note that we maintain a single row per PeerID and PeerID can thus be the primary key.
Methodology
For each metadata_event that results in a validator_metadata_event, we do the following:

If shortLivedSubnets > 0 we set PossibleValidator = true
Increment NumObservations
Derive estimated number of validators: AverageValidatorCount will be a cumulative average of all the validator count estimations so far. The ValidatorCount entry for the current event will be shortLivedSubnets / 2, because validators at any point will be subscribed to 2 attnets: one for the current epoch and once for the next. NOTE: we should round up not down when doing integer division! I.e. q := 1 + (x - 1) / y
Set the new AverageValidatorCount to be the cumulative average of the previous ones:
$$new average=current average+\frac{new count}{(new value−current average)}$$
Insert or modify row

@namn-grg no, this new table will not be a table of events (which the validator event is), but a live view of active validators based on these events. So no need to change the existing events.

Now that we will estimate the validator count through attestation aggregates, I think AverageValidatorCount  will not make sense as the validator will be subscribed to the short-lived attnets for certain epochs only. Instead of this, I propose to have a MaxValidatorCount, which will keep the max count of validators observed.
We also want to have a client version to get more accurate data analysis.
Updated schema -

PeerID
ENR
Multiaddr
IP
Port
LastSeen
LastEpoch
ClientVersion
PossibleValidator: bool
MaxValidatorCount: int
NumObservations: counter

@namn-grg makes sense, good call!


Add Basic Wasm Bindings and add CI Workflow to Measure Size

This PR adds minimal Wasm bindings and example code, and a github action workflow that measures the size of the built bundle (currently around 1.2MB uncompressed, 450kB gzip'd). It fails when it gets significantly larger.
This is a draft. One test started failing after I merged in the current main, and I am not sure what the issue is. I think it might be related to #1495, but I'm not sure. Can you take a look and see if this is familiar @raphaelrobert?

I can't merge this because of CI failing due to the expired key package. Can you override this @franziskuskiefer?

I can't merge this because of CI failing due to the expired key package. Can you override this @franziskuskiefer?

Lets not have a failing CI. Can you disable the failing test for now and file an issue to re-enable it as soon as the kats were updated?

I disabled the test in #1530. @keks can you merge when the CI is through?


Line Numbers are not aligned with code

Version 0.3.0
Line numbers and folding handles are not vertically aligned with code.
See screenshot.

This is true

Give explicit text style with height in it

@deva666 @Somtobro can you please check again with v0.3.1 and Flutter 3.13.9?
If it persists, please tell your platform and send the whole minimal reproducible example.

Same problem with the latest version
Reproducible on Android and iOS (both simulator and real device)
Run your own example 02.

This help for me:
https://github.com/akvelon/flutter-code-editor/issues/261#issuecomment-1963836490


Working Dependencies 2024?

Does anyone have a working version of this repo in 2024?

I second this.


Tx Sender NULL for 11 records - Create Bespoke Test

The below test has been failing since 10/29 with 1 record on 10/29, 9 on 10/30 and 11 today. The overall impact is marginal with just 11 records over a 3 day period but still needs fixing. The assigned analyst(s) should investigate the records with this error and work to determine root cause as well as a potential fix.
08:02:37  Completed with 1 error and 1 warning:
08:02:37
08:02:37  Failure in test not_null_core__fact_transactions_TX_SENDER (models/core/core__fact_transactions.yml)
08:02:37    Got 11 results, configured to fail if != 0
08:02:37
08:02:37    compiled SQL at target/compiled/terra/models/core/core__fact_transactions.yml/not_null_core__fact_transactions_TX_SENDER.sql
08:02:37
08:02:37    See test failures:
  -------------------------------------------------------------
  select * from TERRA.not_null_core.fact_transactions_TX_SENDER
  -------------------------------------------------------------

I checked all those null txs in the explorer, they  all are failed txs. I am not sure if  that's the the reason or not. I tried to exclude them but the tx_gap failure will occur.

Thanks for investigating @robel91
So - the column can legitimately be null. Let's do 2 things:

Turn off the null test so it stops alerting
Write a bespoke test checking for not_null only when the tx succeeded.

We can keep this issue open and use it to solve 2, but I'll remove the bug tag.

Ok ! sounds great @forgxyz

From the PR:
Terrascope can extract the tx_sender. I think the test holds up and the implementation should be fixed. That, unless we don't have that information at all from the chainwalker.


Add beforeSubmit hook

There are many times where we may want to normalize data before we validate/submit

@simonihmig, I made another change to pass available arguments to their actions. This is necessary if you're dependent on a server returned error message or some specific post validation check.

Thanks again @offirgolan! The hook and passing the results of validations to the actions does make sense. Why is passing the raw event object e necessary? It is a rather low level thing I would tend to not pass around to the app unless absolutely necessary for some use cases...
Things that should be added before merging:

[ ] mention the beforeSubmit action and the passing of the results to docs ("Submitting the form")
[ ] add a simple test for asserting that beforeSubmit is called

@simonihmig I wasn't sure it was really necessary or not tbh but ill remove it. To follow your naming convention, do you want to call this action before or keep it as beforeSubmit?

@simonihmig Added docs and tests. Also removed e from action calls and renamed beforeSubmit to before.

@simonihmig fixed it by exposing action as a public action handler.

Travic build was failing because of some trivial JSCS code styling issues, please fix them if still present....

Should be fixed now


Questions about how to calculate an index for registration quality

Hello!
I used antsRegistrationSyN.sh to register，how can I calculate the registration quality, I don't mean to check the figure of registration.

Image registration assessment is an ongoing research question.  We don't have any specific tools within ANTs to do such automated analyses but, rather, generally rely on visual inspection.


Demo video should have the thing that shows what keys you're pressing

First, thank you for the plugin! Looks awesome, I'm trying it out.
The demo video is great and I'm impressed by the selection of all the function parameters to add a type definition to all of them at once, but how do you do that?

using the example config you can press M to match cursors. this will prompt for a regex and then add a cursor for every match in your visual selection(s).
you can press S to split cursors. it will prompt for a regex and split your visual selection(s) into multiple. sort of like how string.split works.
i just woke up so this might not be explained well. i plan on making a tutorial when i have time since there are a lot of hidden powerful features


Create Dockerfile

Really basic Dockerfile that should pretty well match the vagrant environment for all intents and purposes.  Obviously not audited or hardened, etc.
The link in the documentation is to my own personal (albeit public) docker hub image that is an automated build from my fork-- feel free to update that with an official account-- or not.
Sorry for the rushed PR, leaving town for the weekend, feel free to let it sit til Monday.

Hi @colinross ,
Thank you for putting in the effort to build this docker container/image.
We considered using docker for this bootcamp and I experimented with Docker for this bootcamp a few weeks ago, but we decided to go with a Virtual Machine and Vagrant instead for multiple reasons:

In order for Docker to work on Mac OS X and Windows, users still needed to install Virtual Box. Therefore maintaining a docker container inside a linux VM on a Windows or OS X host added unnecessary complexity (especially networking - mapping ports between docker containers and the linux guest VM and then mapping the ports between the linux guest VM and the host)
Docker introduces a steeper learning curve compared to Vagrant. That is to say it is easier to manage virtual machines through VirtualBox and Vagrant than it is to manage docker containers.
Docker has not been whitelisted yet. There are security issues and gaps that need to be addressed before we can endorse using containerization technology.

@pmbenjamin - colin and i discussed this.  If we can get the docker file running with the right level of security and others want to use it, its a good second option to Vagrant, not necessarily a replacement.  The other reason we decided not to use it is its lack of native Mac support.

@slietz I don't mind testing & experimenting with this Dockerfile over the next few days, but if we merge it in, then we assume the responsibility for supporting it.
Just to expand on your last statement:

The other reason we decided not to use it is its lack of native Mac support.

This is the biggest reason why I opted for a simple virtual machine approach. Docker had a networking bug in their OS X offering that is a pain to resolve.
The issue would occur every time I switch networks (e.g. using the OS X at home, then bringing it to work the next day). To resolve it, I would have to terminate the running Docker daemon and re-initialize it again using Kitematic (GUI-based Docker manager).
As a result, Docker announced plans to reduce their dependence on 3rd party virtualization software (e.g. Virtualbox) a few months ago, but now Docker will be managing a linux VM directly (reference). This is still in beta and I would hardly call it native Mac support.

Sorry all for the late reply, I was out of town and traveling.
@pmbenjamin: I fully understand your trepidation over support issues. You are right that native docker support is beta in the full sense of the term on both windows [hyper-v based] and mac [ https://github.com/mist64/xhyve ] as they transition over to utilizing the native hypervisiors mentioned.
Currently, where I feel like docker support (in the context of the devsecops bootcamp repo) fits is at a user-contrib type level. It is available to those whom wish to utilize it as their own environment and skills/knowledge allow.
I think we can at least all agree that docker has enough critical mass of usage, even with it's 'beta' support that the project can support it this much (have a Dockerfile)
And lastly  -- I meant this purely as a PR at the moment simply to have a central location of discussion and so that others knew it was an idea/known enhancement.  I would really want to get some group testing/usage on it before marking it as working for the whole course work.

Sorry all for the late reply, I was out of town and traveling.
@pmbenjamin: I fully understand your trepidation over support issues. You are right that native docker support is beta in the full sense of the term on both windows [hyper-v based] and mac [ https://github.com/mist64/xhyve ] as they transition over to utilizing the native hypervisiors mentioned.
Currently, where I feel like docker support (in the context of the devsecops bootcamp repo) fits is at a user-contrib type level. It is available to those whom wish to utilize it as their own environment and skills/knowledge allow.
I think we can at least all agree that docker has enough critical mass of usage, even with it's 'beta' support that the project can support it this much (have a Dockerfile)
And lastly -- I meant this purely as a PR at the moment simply to have a central location of discussion and so that others knew it was an idea/known enhancement. I would really want to get some group testing/usage on it before marking it as working for the whole course work.

Thanks @colinross,  I think we are in agreement that docker is useful.  It would really helpful to continue to dev and provide user-contrib so that we can make it possible to complete the course using docker.


ZCLWrite should return "INVALID_VALUE" instead of success & truncation in case of invalid payloads.

Problem
zclwrite error.txt
zclwrite error2.txt
When sending zclwrite command with invalid value, command returns EMBER_ZCL_STATUS_SUCCESS. Subsequent zclread of the attribute would return values such as 0, truncated string that matched requirements, and etc.
Expecting zclwrite to return EMBER_ZCL_STATUS_INVALID_VALUE if invalid value is given.
example: (zclwrite error2.txt)
"zclwrite OnOff OffWaitTwrite OnOff OffWaitTime 1111 1 0 -1" returns success, and "zclread OnOff OffWaitTwrite OnOff OffWaitTime 1111 1 0" returns 65535
in addition, writing valid value does not always return expected result from zclread. (zclwrite error.txt)
"zclwrite Basic Location 8067 0 0 US" wrote successfully; however, zclread returns "value='U'"; expecting value to be "US"

Problem
zclwrite error.txt
zclwrite error2.txt
When sending zclwrite command with invalid value, command returns EMBER_ZCL_STATUS_SUCCESS. Subsequent zclread of the attribute would return values such as 0, truncated string that matched requirements, and etc.
Expecting zclwrite to return EMBER_ZCL_STATUS_INVALID_VALUE if invalid value is given.
example: (zclwrite error2.txt)
"zclwrite OnOff OffWaitTwrite OnOff OffWaitTime 1111 1 0 -1" returns success, and "zclread OnOff OffWaitTwrite OnOff OffWaitTime 1111 1 0" returns 65535

Currently, IM just reencodes the value and passes the value to ember to store. Ember does not check the length of value so it does not raise an error.
Solution is to add extra check on im-compatibility-functions.

in addition, writing valid value does not always return expected result from zclread. (zclwrite error.txt)
"zclwrite Basic Location 8067 0 0 US" wrote successfully; however, zclread returns "value='U'"; expecting value to be "US"

This is another issue, after checking the zap files, the Location is a string which size is 2, however, it is expected to be 3 since there is one byte length preceed. @vivien-apple I would expect the ZAP tool can add the extra one or two bytes to the size of string? (i.e. It will display 2 on GUI, but it will emit 3 (or 1+2 for better readibility) in code.


Dropdown directive shouldn't fire $rootscope.$digest() on every dropdown close

Bug description:
In "dropdown.js" line 112 there is call to openscope.$apply(), which triggers $rootscope.$digest(). This adds additional and not needed (I could be wrong here) full app digest cycle. That would be great if this call could be replaced with openscope.$digest().
Same applies to lines 324-326. Instead of wrapping "dropdownCtrl.toggle();" in scope.$apply, it can written as chaining:
dropdownCtrl.toggle();
scope.$digest();
Version of Angular, UIBS, and Bootstrap
Angular:
1.5.9
UIBS:
1.3.3
Bootstrap:
?

I do not think this can be done safely - toggling of the dropdown state with the open binding means that changes can flow up the $scope tree, resulting in potentially app-wide changes, and this behavior is almost assuredly relied upon in production applications.
Going to close this as something that will not be supported due to the regressions/bugs this request is likely to spawn.


Package is not compatible with ARM64

Is there an existing issue for this?

[X] I have checked for existing issues https://github.com/getsentry/profiling-node/issues
[X] I have reviewed the documentation https://docs.sentry.io/
[X] I am using the latest SDK release https://github.com/getsentry/profiling-node/releases

How do you use Sentry?
Sentry Saas (sentry.io)
SDK Version
0.2.2
Link to Sentry event
No response
What environment is your node script running in?
FROM arm64v8/node:18-buster-slim as build
How is your code deployed and bundled?
arm64 docker image
Steps to Reproduce
docker run -it arm64v8/node:18-buster-slim sh
mkdir example
cd example
npm init -y
npm i @sentry/profiling-node

Expected Result
Expecting the package to be installed on arm64 docker containers
Actual Result
[4/4] Building fresh packages...
130 | error /tmp/node_modules/@sentry/profiling-node: Command failed.
131 | Exit code: 1
132 | Command: node scripts/check-build.js
133 | Arguments:
134 | Directory: /tmp/node_modules/@sentry/profiling-node
135 | Output:
136 | @sentry/profiling-node: Precompiled binary found, attempting to load...
137 | @sentry/profiling-node: Compiling from source...
138 | gyp info it worked if it ends with ok
139 | gyp info using<EMAIL_ADDRESS>140 | gyp info using<EMAIL_ADDRESS>| linux | arm64
141 | gyp ERR! find Python
142 | gyp ERR! find Python Python is not set from command line or npm configuration
143 | gyp ERR! find Python Python is not set from environment variable PYTHON
144 | gyp ERR! find Python checking if "python3" can be used
145 | gyp ERR! find Python - "python3" is not in PATH or produced an error
146 | gyp ERR! find Python checking if "python" can be used
147 | gyp ERR! find Python - "python" is not in PATH or produced an error
148 | gyp ERR! find Python
149 | gyp ERR! find Python **********************************************************
150 | gyp ERR! find Python You need to install the latest version of Python.
151 | gyp ERR! find Python Node-gyp should be able to find and use Python. If not,
152 | gyp ERR! find Python you can try one of the following options:
153 | gyp ERR! find Python - Use the switch --python="/path/to/pythonexecutable"
154 | gyp ERR! find Python   (accepted by both node-gyp and npm)
155 | gyp ERR! find Python - Set the environment variable PYTHON
156 | gyp ERR! find Python - Set the npm configuration variable python:
157 | gyp ERR! find Python   npm config set python "/path/to/pythonexecutable"
158 | gyp ERR! find Python For more information consult the documentation at:
159 | gyp ERR! find Python https://github.com/nodejs/node-gyp#installation
160 | gyp ERR! find Python **********************************************************
161 | gyp ERR! find Python
162 | gyp ERR! configure error
163 | gyp ERR! stack Error: Could not find any Python installation to use
164 | gyp ERR! stack     at PythonFinder.fail (/tmp/node_modules/node-gyp/lib/find-python.js:330:47)
165 | gyp ERR! stack     at PythonFinder.runChecks (/tmp/node_modules/node-gyp/lib/find-python.js:159:21)
166 | gyp ERR! stack     at PythonFinder. (/tmp/node_modules/node-gyp/lib/find-python.js:202:16)
167 | gyp ERR! stack     at PythonFinder.execFileCallback (/tmp/node_modules/node-gyp/lib/find-python.js:294:16)
168 | gyp ERR! stack     at exithandler (node:child_process:427:5)
169 | gyp ERR! stack     at ChildProcess.errorhandler (node:child_process:439:5)
170 | gyp ERR! stack     at ChildProcess.emit (node:events:513:28)
171 | gyp ERR! stack     at ChildProcess._handle.onexit (node:internal/child_process:289:12)
172 | gyp ERR! stack     at onErrorNT (node:internal/child_process:476:16)
173 | gyp ERR! stack     at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
174 | gyp ERR! System Linux 4.14.248-189.473.amzn2.aarch64
175 | gyp ERR! command "/usr/local/bin/node"<EMAIL_ADDRESS>"configure"
176 | gyp ERR! cwd /tmp/node_modules/@sentry/profiling-node
177 | gyp ERR! node -v v18.14.2
178 | gyp ERR! node-gyp -v v9.3.1
179 | gyp ERR! not ok
180 | npm notice
181 | npm notice New patch version of npm available! 9.5.0 -> 9.5.1
182 | npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.5.1
183 | npm notice Run npm install -g<EMAIL_ADDRESS>to update!
184 | npm notice
185 | @sentry/profiling-node: Failed to build from source, please report this a bug at https://github.com/getsentry/profiling-node/issues/new?assignees=&labels=Type%3A+Bug&template=bug.yml
186 | /tmp/node_modules/@sentry/profiling-node/scripts/check-build.js:51
187 | throw e;
188 | ^
189 |
190 | Error<EMAIL_ADDRESS>cannot open shared object file: No such file or directory
191 | at Module._extensions..node (node:internal/modules/cjs/loader:1338:18)
192 | at Module.load (node:internal/modules/cjs/loader:1117:32)
193 | at Module._load (node:internal/modules/cjs/loader:958:12)
194 | at Module.require (node:internal/modules/cjs/loader:1141:19)
195 | at require (node:internal/modules/cjs/helpers:110:18)
196 | at Object. (/tmp/node_modules/@sentry/profiling-node/scripts/check-build.js:24:3)
197 | at Module._compile (node:internal/modules/cjs/loader:1254:14)
198 | at Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
199 | at Module.load (node:internal/modules/cjs/loader:1117:32)
200 | at Module._load (node:internal/modules/cjs/loader:958:12) {
201 | code: 'ERR_DLOPEN_FAILED'
202 | }
203

@agoldis we do not provide a precompiled bindings for arm64v8, so the package attempts to compile from source and fails as it cannot find python3.

163 | gyp ERR! stack Error: Could not find any Python installation to use

You will need to install python3 before running npm install (this is a standard requirement for any package that uses native modules) and I suspect you will might also need to install make and g++. After you install the build dependencies, the package should build from source and npm install should no longer fail.
Let me know how it goes, we'll be happy to help you

@agoldis any updates?
If you have not yet managed to resolve this, mind sending us the output of running npm install --verbose --foreground-scripts @sentry/profiling-node in an empty directory?

Nope, I didn't want to bloat the Docker image. Ended up switching to datadog 😝

@agoldis oh dang, that's not nice to hear, but glad you got it working at least!

@agoldis I will close the issue and see if we can provide prebuilt images for arm64v8 in the future


wslservice.exe has product name "Windows (R) Win 7 DDK Driver"

Greetings,
I was cleaning up my Win 10 installation and looking at the Sysinternals Autoruns tool to check for unwanted autostart applications.
There I noticed this entry in the Services tab, where the third column is titled "Publisher":

It's the Windows Store version of WSL.
Looking at the Details pane of the file's properties shows this:

I'm not entirely sure about the meaning of this, but it mentioning Windows 7 is probably in error?

This is fixed in 1.1.0 and later.

Fixed with https://github.com/microsoft/WSL/releases/tag/1.1.2.

And the wsl service name is 'ms-resource:AppName', which is very strange.

hmm??


feat(vue-query): support injectable contexts

This feature requires Vue 3.3.0, which has been out for a while now. It allows using vue-query APIs in places where it is valid to use them but currently throws an error.

hasInjectionContext(): https://github.com/vuejs/core/pull/8111
app.runWithContext(): https://github.com/vuejs/core/pull/7451

The only thing that I think is left to do is to add extra warnings in some APIs that rely on the Reactivity API (ref, watch, onScopeDispose etc) by checking if there is a current effect scope. I think this was previously achieved by checking for a current instance within the useQueryClient which seems to be used in functions relying on those functions. But since it wasn't tested, I didn't add them. Let me know if I should add them.
This change requires vue-demi to also be updated, if the package manager doesn't do it for their users, they shall run an pnpm update or similar, they can also install vue-demi and debug with pnpm why (or similar).
I can also replicate the PR for the beta branch

I would prefer to not introduce this in the current v4.x version if it requires other libraries to be updated as well.
Even though bumping two packages minor version should not be a problem i would prefer to avoid people updating vue-query realize that it does not work via compilation errors and then trying to figure out what is wrong.

I understand but for Vue 3.3 users, not being able to use the query client within a navigation guard would be a bug. So you can also see this as a bugfix. In practice, users update multiple libraries at the same time and they should also see the requirement in peer deps.

@posva I'm not sure if PR #5703 is a progressive transition option?

it should make it work in most cases yes

Build should be fixed with https://github.com/vueuse/vue-demi/pull/241

I released a new version of vue-demi 🙂

If anyone has issue with getting this to work and comes across this PR / comment, be aware that it doesn't always work like you might think in async functions - if you await anything, the injectable context doesn't exist anymore afterwards and the library will throw an error.
I worked around this by getting the query client before my async stuff and then passing it in to my query function:
  const queryClient = useQueryClient();

  const { isClient } = useCurrentOrg();
  if (!(await isClient())) {
    next('/');
    return;
  }

  const { trial, isTrialEnded } = useTrial({}, queryClient);

(where useTrial passes queryClient through as the second option to useQuery)
btw @posva, thanks for adding this! it's super useful :)

https://github.com/TanStack/query/discussions/5783 and #5703 brought me here.
So by adding the client as a second param everything works (again) but the warning continues to show up, right?
This helped. Thanks!

Do you think this will work on Nuxt plugins? I'm trying to use useQueryClient, but still getting that error.

It works but can we get rid of the warning somehow?


Current build has lost solo ability

Rolled back to 2569 to restore.

That is odd. There haven't been any changes since 2555.

Not sure how, but the current build 2593 has restored this.


Fix license in cvxcore.i (#834)

This should be the correct license paragraph as it is taken from the corresponding cvxcore init file

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Dyck, Alex seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I believe SWIG uses C-style comments; please prefix each line with  // and delete the quotes. Once that's done, we'll merge it in.


Polishing the core module by adding missing Javadoc and do some refactoring

Description
Some changes for the core module, i.e. adding missing Javadoc, refactoring the ProcessKeystoregenerator class, and moving the auto provisioning tooling to the provisioners module
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change
[ ] Chore (non-breaking change which doesn't affect codebase;
test, version modification, documentation, etc.)

Checklist

[x] Code contributed by me aligns with current project license: Apache 2.0
[x] Pull Request contains a description of the changes
[x] Pull Request does not include fixes for multiple issues/topics
[x] Code is self-descriptive and/or documented
[ ] I have implemented unit tests to cover my changes
[ ] I tested my code in OpenShift

/test

/and-test-again

test-me

test-again

ok-to-test

/test

Intersmash PR CI check results
Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests,
which were triggered by @fabiobrz comment, see https://github.com/Intersmash/intersmash/pull/180#issuecomment-2307493520.
CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#72> FAILED:
Intersmash integration tests job <eap-8.x-intersmash-integration-tests-community#28> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.
Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap#41> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.
Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#45> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.

/test

Intersmash PR CI check results
Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests,
which were triggered by @fabiobrz comment, see https://github.com/Intersmash/intersmash/pull/180#issuecomment-2317544511.
CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#74> FAILED:

Intersmash integration tests job <eap-8.x-intersmash-integration-tests-community#29> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.

Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap#42> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.

Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#46> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.

Intersmash PR CI check results
Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests,
which were triggered by @fabiobrz comment, see https://github.com/Intersmash/intersmash/pull/180#issuecomment-2317544511.
CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#76> FAILED:

Intersmash integration tests job <eap-8.x-intersmash-integration-tests-community#31> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.

Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap#43> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.

Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap-xp#47> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.

/test

/test

/test

Intersmash PR CI check results
Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests,
which were triggered by @fabiobrz comment, see https://github.com/Intersmash/intersmash/pull/180#issuecomment-2326400427.
CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#105> FAILED:

Intersmash integration tests job <eap-8.x-intersmash-integration-tests-products-jboss-eap#59> reported failures.
Please verify your changes locally or reach out to maintainers in order to investigate the issues.

Intersmash PR CI check results
Hi @fabiobrz, this comment is meant to report the outcome of Intersmash integration tests, which were triggered by @fabiobrz comment, see #180 (comment).
CI checks job <continuous-testing-umb-listener-for-intersmash-pr-checks#105> FAILED:
* Intersmash integration tests job <_eap-8.x-intersmash-integration-tests-products-jboss-eap#59_> reported failures.
  Please verify your changes locally or reach out to maintainers in order to investigate the issues.

<eap-8.x-intersmash-integration-tests-products-jboss-eap#59> failed due to a Jenkins error, and was executed again as <eap-8.x-intersmash-integration-tests-products-jboss-eap#60>, which in turn reported 2 failures:

org.jboss.intersmash.testsuite.provision.auto.AutoProvisioningTests.testAppIsLive
org.jboss.intersmash.testsuite.provision.openshift.ProvisionerCleanupTestCase.testProvisioningWorkflowCleanup(OpenShiftProvisioner)[1] (related to InfinispanOperatorProvisionerTest

Both are intermittent failures, and actually reported as fixed in the subsequent run, i.e. <eap-8.x-intersmash-integration-tests-products-jboss-eap#62>.
Based on the above findings, the changes in the PR are deemed to pass the integration tests and can now be reviewed.


unit tests wrong namespace deletion

As discussed here:
https://github.com/facebook/php-webdriver/issues/242

Going to leave this open for now, would like some community comment here if is the right thing to do or not.


Networking Requirements for local debugging (Firewall rules and corporate networks)

It is not entirely clear, how the TcpCheck of the Skype Media packages works.
I am unable to debug the local media bots like HueBot locally due to restricted/militarized networking in my corporate network.
I can temporarily circumvent the networking restrictions by tethering via LTE and I can successfully run the bot samples.
Is there any schematic on which ports need to be open (incoming/outgoing) for a bot to work properly.
Are ngrok generated ports required to be available from the bot itself? In general ngrok works fine in my network, but connecting outwards to that tcp port would be a problem if required.
That would enable me to create a ticket with my IT service to poke the necessary holes into our corporate network to facilitate me debugging bots on my machine.

I found this comment of an earlier issue:
https://github.com/microsoftgraph/microsoft-graph-comms-samples/issues/51#issuecomment-474014222
However, it does not specify, which UDP ports will be utilized for ICE candidate gathering :(

@ssulzer Bump!

So it appears, that the TryCheckTcpConnectivity works by the bot instance itself trying to connect to it's own loadbalancer (Public IP + Port).
Unfortunately that means, that I will have to fix my public port in place, which will require a paid plan of ngrok. Then I can mess with my IT service to open the port for outgoing traffic, so my local instance can speak to the public address (of my ngrok process) which will route back to my ngrok client and into the instance/local port of my media bot.
Can somebody approve the correctness of this hypothesis?

@zihzhan Bump!


feat: vscode-settings

Define build task and problem matcher for tsup
Add build task to preLaunchTask
Remove stopOnEntry as its value is the default value.

Will be testing this PR tomorrow


Add self-supervised learning capabilities to InnerEye

This PR adds support for training of self-supervised models in InnerEye. The implementation is based on the new bring-your-own model framework. We provide two examples to get started with training of SSL models: CIFAR10 SSL training and Chest-Xrays based models. See SSL doc for more details.

I see dozens of classes added, but only a small number of tests. maybe there's something you could add here? (I know that we've also been historically sloppy when it comes to testing new model architectures...)

Have added tests to encoder creation functions, for the various datamodule classes, for the transformation pipeline initialization


Document command for Homebrew plugin install

This is a small change to the Homebrew install docs to recommend a one-liner instead of the manual approach:
$(brew --prefix certbot)/libexec/bin/pip install certbot-s3front

@dlapiduz Just a friendly reminder to have a look at this PR when you have some time. Let me know if you need more info or if something needs to change. 😄

Thanks for the PR. I added a quick comment, once that is clear we can merge it.


issue cheato2xdag for display

https://explorer.xdag.io/block/oKw26wfTgKAsPJ5yXNB4f1g5VN8ZVUU6
所有input相加值为71.359999927；output值却是71.359999932，input竟然还＜output

How he did xfer of 0.0000 ? It should be disabled by wallet and in pool payment

@sgaragagghu  It should be history issue I think. The old version still handle cheatos less than 4 which will be displayed as 0 xdag.

Not an issue with new versions.


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2021-05-12T15:59:40+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio_release-1.8
e48aceb262ccf3eb99be1f18273cbdeffaf15abe
link
/test integ-telemetry-k8s-tests_istio_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio_release-1.8
e48aceb262ccf3eb99be1f18273cbdeffaf15abe
link
/test integ-telemetry-k8s-tests_istio_release-1.8

integ-pilot-k8s-tests_istio_release-1.8
e48aceb262ccf3eb99be1f18273cbdeffaf15abe
link
/test integ-pilot-k8s-tests_istio_release-1.8

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Default value to be placed in Currently selected box

Hey,
I'm struggling to achieve this. I want my default value to be placed in Currently selected box.
https://github.com/truckingsim/Ajax-Bootstrap-Select#optionspreserveselected
This is default state when form and default value is loaded:

But i want to have it like this ( this is after click event  )

This is my configuration block ( types are there because i m using TypeScript ):
selectObject.ajaxSelectPicker({
        ajax: {
          url: `${config.apiEndpoint}${url}`,
          type: 'GET',
          headers: {
            'Authorization': `Bearer ${localStorage.getItem(config.tokenName)}`
          },
          dataType: 'json',
          data: {
            searchBy: '{{{q}}}',
            maxResults: 5
          }
        },
        emptyRequest: false,
        preprocessData: function (data: Object[]) {

          return data.map((item: any) => {
            return Object.assign({}, {
              'value': item[key],
              'text': item[value],
              'disabled': false
            });
          });
        },
        preserveSelectedPosition: 'before',
        preserveSelected: true
      });

@vladotesanovic this has been a bug for a while #41 is basically the same thing.  Thanks for the new use case gives me one more thing to try to narrow it down.  I've tried multiple times to fix this, but we're going to try again for the 2.0.0 rewrite.

duplicate of #41


Wouldn't it be helpful to have the build dependencies listed in the install instructions?

Is your feature request related to a problem? Please describe.
I'm new to building from source, so I don't know what dependencies I need to build with.  I see something in instructions, I assume that's all I need, but invariably, there's always more, and people expect you to just know what is needed to do it.  That's unrealistic.  For example, I'm trying to install libratbag.  I've been at this for 3 hours now, and still have unresolved dependencies, like the wonderfully named "check".  Where do I get that?  Do you know how many hits you get that aren't a linux package when searching "install check ubuntu"?
Describe the solution you'd like
I would like to see a list of package dependencies that are required to build libratbag in the install instructions.

https://github.com/libratbag/libratbag/wiki/Building-from-source
Is this what you're looking for? I'm not sure if every dependency is list, but they should.

Agreed.  With an ambiguous name like "check", I never would have found that.  Not in a million years.


feat(NumberTheory/LSeries): abstract functional equations

A general framework for proving analytic continuation & functional equations via Mellin transforms.

(Note to reviewers: this is a generalisation of the existing argument used in the Riemann zeta function code; but I have not refactored the existing code using this just yet, since I will shortly be PR'ing more general results for Hurwitz and Dirichlet L-series, and Riemann zeta will eventually just be a special case of these.)

LGTM!

This looks like a great work, thanks! I will try to have a closer look later.

I agree!

I think there is not much more to do with this PR and it seems ready to go.

Thanks!
bors merge


Update netty-buffer, netty-codec-http to 4.1.67.Final

Updates

io.netty:netty-buffer
io.netty:netty-codec-http

from 4.1.66.Final to 4.1.67.Final.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.netty" } ]

labels: library-update

0.22


feat(app):  Calibration dashboard wizard data invalidation

Overview
Adds a warning message to the Deck and Tip Length calibration wizards when recalibrating to let the user know pipette offset calibrations will be wiped and adds the functionality to perform the calibration invalidations
closes RAUT-321
Deck
https://user-images.githubusercontent.com/66637570/217022915-83f69a55-f38a-4b02-8355-3834b4673054.mov
Tip Length
https://user-images.githubusercontent.com/66637570/217021685-f32abd20-8865-4567-ae17-1adcb7d52f00.mov

Test Plan
Manually tested by completing calibrations then performing recalibrations for the Deck and ensuring pipette offset calibrations were deleted for both mounts. Completed the deleted calibrations then tested tip length recalibration and ensured the offset calibration for the matching mount was deleted

Changelog

Reworked the logic of the InvalidationWarning component
Added a deleteCalibrations callback to the Wizard launchers to allow for calibration invalidation when necessary
various new tests and updates to old tests

Review requests
Ensure the calibration deletion behavior functions as expected

Risk assessment
med - low, fairly contained behavior, but has the potential to annoy the user if the incorrect calibrations were somehow deleted

@b-cooper I think I made the updates we talked about earlier, take a look when you get a chance and let me know if anything here doesn't match up with what you were expecting. Working on the table filtering in the meantime


Check if openstack user exists before trying to create it

Jira: https://issues.redhat.com/browse/OSPRH-11669

/lgtm

/recheck

recheck
content provider failed

/cherry-pick 18.0-fr1


Poetry doesn't have a group name in controller selection

Testing #9434
Controller list:

Interpreter list:

There haven't been any upvotes on this issue, hence closing this for now


Re-consider how default values are reverse engineered

Feedback on 2.0 (e.g. #9507 #9617) has made me think about what the real intent is behind having columns with default values. Specifically, I think it is likely wrong to assume that the EF model should have a store-generated property just because there is some default on the column. It may well be the cause the the application will always supply a value and so, when using EF, the default is never used. The default may instead be there because:

It was needed when adding a new column (either by Migrations or manually)
It is there for legacy reasons--old/non-EF app made use of it in hand-crafted SQL, but EF never will
It's just part of the way all columns are created by the DBA or similar--the column gets a default regardless of whether it is needed or not

Going beyond the most general case, if the CLR default would generate the same value as the database default, then having EF do store generation is a no-op anyway. If an app was relying on this for bools in 1.1, and we now generate nullable bools, everything will still work, but the app needs to be updated to use nullable bools.

Triage: we will update reverse engineering to not make the property store-generated if the default value from the column matches the default value of the CLR property.

This change break with the model, if you need use the default value in a entity, the logical is set these in the constructor of the class or in the property how default value. The problem is when the database modify the record after insert with a trigger, calculate fields, another procedure or function of transact-Sql that EF don´t know. For example if your default value is a function in sql server how default([dbo].[NEWDOC_Order]()) EF never to know this. In this case the default value will be lost and you always must to read the records inserted before return it to EF, but if EF know the function, numeric values, getdate() or other basic functions these can be set in the constructor or in the properties if I have a bool field and in my model the field refer to bool?, the validations an other functions are different. In numeric class If I make a sum, I would be forced to change the function to not read the null values. If I have a default value in a table with not null in my model must be the same because I must to change my validations and programs for this cause. I´m sure that this behavior is not correct.
For me the use of default values should be something like the following:
public partial class Customer
{
    public Customer()
    {
        Date = DateTime.Now;    // Default Value "Getdate()"
        Total = 100;	        // Default Value 100
    }

    public string CustomerId { get; set; }
    public string Name { get; set; }

    [DefaultValue("Getdate()")]
    public DateTime Date { get; set; }

    [DefaultValue(100)]
    public decimal Total { get; set; }
}

// Or using default values in properties
public partial class Customer
{
    public Customer()
    {

    }

    public int CustomerId { get; set; }

    public string Name { get; set; }

    public DateTime Date { get; set; } = System.DateTime.Now;  	    // Default Value "Getdate()"

    public decimal Total { get; set; } = 100;                       // Default Value 100
}

Can you guys just hotfix this nullable bool scaffold behavior instead of shipping on 2.1.0? Maybe ship a 2.0.1? Just add a quick flag when invoking the command line scaffold tool or something perhaps?

I mean, 2.1.0 / Q1 2018 is quite a long wait...

@ryanelian you can use my SQLite Toolbox to generate the v1 scaffolding, without installing anything in your solution 😀 - also works for SQL Server

Triage: we plan to make the following change in 2.0.1: for non-nullable bool columns in SQL Server that have a column default of false, we will scaffold a non-nullable bool property without any store generation flags.
Post 2.0.1 we will do what is described in the triage comment above.
We will not change the behavior where the default in the database is not the CLR default. Specifically, for bool columns that have a default of true, we will still scaffold a nullable bool. This is because non-nullable bool properties with a store-generated value of true are not useful--the value in the database will always be true--see #8400 and #7163 for more details.

I can´t understand. 'non-nullable bool properties with a store-generated value of true are not useful'. If you want to insert a same fields and have only two values (only true o false) without null values. I have this configuration in more than hundred forms. For me it´s the same problem. In the database you have 'bool not null default value(1)' and in the model you have bool?....  Please at least set a parameter in the scaffolding for control this behavior.

@JuanIrigoyen This is the scenario I am talking about. Entity with a non-nullable bool:
public class Fooo
{
    public int Id { get; set; }
    public bool Flag { get; set; }
}

Configured in EF to have a store-generated default of true:
modelBuilder
     .Entity<Fooo>()
     .Property(e => e.Flag)
     .HasDefaultValueSql("(1)");

With this configuration, Flag will always be saved as true; never false, and hence is not useful. For example:
using (var context = new TestContext())
{
    context.Database.EnsureDeleted();
    context.Database.EnsureCreated();

    context.Add(new Fooo { Flag = false });
    context.SaveChanges();
}

using (var context = new TestContext())
{
    Debug.Assert(context.Fooos.First().Flag);
}

This behavior hasn't changed from EF Core 1.0--there's lots of discussion in the linked issues that talk about this. The most complete is probably in #7089. Essentially, the issue is that the sentinel to tell EF to use use a store-generated value is the same as the value 'false'. The ways to make this work are:

Stop having EF make use of the store generated value--e.g. by removing the HasDefaultValue call.
Make the property nullable so that the sentinel ('null') is different from the valid value 'false'.

ok, I understand, thanks for the explanation, In this case I prefer the first option, because a bool value? is different from bool and it is important that the model sea similar to the table structure in the database.
Even the bool nullable in a forms is represented differently.
Bool? can show three states.

This is because non-nullable bool properties with a store-generated value of true are not useful--the value in the database will always be true--

I can't understand.
This is my table:
CREATE TABLE Employee
(
	EmployeeId VARCHAR(64) CONSTRAINT PK_Employee PRIMARY KEY,
	Username VARCHAR(256) CONSTRAINT UQ_Employee_Username UNIQUE NOT NULL,
	Password VARCHAR(256),
	PasswordUpdatedAt DATETIME2,
	Name VARCHAR(256) NOT NULL,
	Email VARCHAR(256),
	IsEnabled BIT NOT NULL DEFAULT 1,
)

This is from EF Core 1.1.2 scaffold (navigation properties and some stuffs are omitted for brevity):
public partial class Employee
{
    [Column(TypeName = "varchar(64)")]
    public string EmployeeId { get; set; }
    [Required]
    [Column(TypeName = "varchar(256)")]
    public string Username { get; set; }
    [Column(TypeName = "varchar(256)")]
    public string Password { get; set; }
    public DateTime? PasswordUpdatedAt { get; set; }
    [Required]
    [Column(TypeName = "varchar(256)")]
    public string Name { get; set; }
    [Column(TypeName = "varchar(256)")]
    public string Email { get; set; }
    public bool IsEnabled { get; set; }
}

public partial class MyDbContext : DbContext
{
    public virtual DbSet<Employee> Employee { get; set; }

    protected override void OnModelCreating(ModelBuilder modelBuilder)
    {
        modelBuilder.Entity<Employee>(entity =>
        {
            entity.HasIndex(e => e.Username)
                .HasName("UQ_Employee_Username")
                .IsUnique();

            entity.Property(e => e.CreatedAt).HasDefaultValueSql("getutcdate()");

            entity.Property(e => e.CreatedBy).HasDefaultValueSql("'SYSTEM'");

            entity.Property(e => e.IsDivisionHead).HasDefaultValueSql("0");

            entity.Property(e => e.IsEnabled).HasDefaultValueSql("1");

            entity.Property(e => e.UpdatedAt).HasDefaultValueSql("getutcdate()");

            entity.Property(e => e.UpdatedBy).HasDefaultValueSql("'SYSTEM'");
        });
    }
}

This is the C# code for setting IsEnabled to false. Nothing fancy:
e.IsEnabled = form.IsEnabled;
await DB.SaveChangesAsync();

The field is clearly set to false when I view the data directly using SQL Server Management Studio. (Forgive me for not showing the other data, they're abit confidential):

Flag will always be saved as true; never false, and hence is not useful.

Please help me understand this phenomena. Sorry if I misunderstand the previously shown case / example above.

@ryanelian It can be updated to false, but never inserted as false.

Ahh. That explains it.
Perhaps to maintain compatibility with database context created with version 1 scaffold, BIT NOT NULL DEFAULT X properties should be scaffolded with Auto-Property Initializers as an interim solution?
public partial class Employee
{
    public bool IsEnabled { get; set; } = true; // or false
}

i.e. For now, no need to implement that for each and every data type. Just make a compatibility flag to scaffold boolean property like that, please?
That way you won't break compat for 2.0.0 and 1.X.X users.

Hi @ryanelian. We are gathering information on the use of EF Core pre-release builds. You reported this issue shortly after the release of 2.0.0 RTM. It would be really helpful if you could let us know:

Did you consider testing your code against the pre-release builds?
Was there anything blocking you from using pre-release builds?
What do you think could make it easier for you to use pre-release builds in the future?

Thanks in advance for any feedback. Hopefully this will help us to increase the value of pre-release builds going forward.

Did you consider testing your code against the pre-release builds?

Yes

Was there anything blocking you from using pre-release builds?

When I used 2.0.0 preview1 SDK, I was not able to compile my 1.1.X project from Visual Studio 2017 Update 2 even when global.json exists for unknown reason. (AKA I cannot work.)
So I stopped playing with pre-release builds...

What do you think could make it easier for you to use pre-release builds in the future?

Don't break Visual Studio 😕

Impact: Refer to first post.
Risk: Low. Only in SqlServer, Columns with type bit and default value of (0) will not get default value constraint and will scaffold non-nullable bool which would match with 1.1 behavior.

Created #9826 for the 2.0.1 part of this; marking this issue for re-triage

Triage: for 2.1, we will do other simple types (e.g. ints) like we did in 2.0.1, and also look at what we might do for other providers. We may also choose to do more (such as a flag to turn off generating store-generated properties for columns with default constraints), but that probably requires more discussion

@ryanelian Thanks for the feedback; much appreciated.

You're welcome.
Also, +1 to completely turning off default values for generated properties using a flag...
Off-topic but related, is there a way to disable the warnings when using 1.1 generated entities with EF Core 2?
2017-09-02 02:56:01.522 +07:00 [Warning] The 'bool' property '"IsEnabled"' on entity type '"Employee"' is configured with a database-generated default. This default will always be used when the property has the value 'false', since this is the CLR default for the 'bool' type. Consider using the nullable 'bool?' type instead so that the default will only be used when the property value is 'null'.

This is distracting me from the actual warning logs in the application...

@ryanelian Yes, for example:
optionsBuilder
    .ConfigureWarnings(wc => wc.Ignore(RelationalEventId.BoolWithDefaultWarning))
    .UseSqlServer(...);

Thinking forward on this, what are the long term implications? C# is moving towards not null reference types. So later on I'll have my strings defaulting to not nullable. Databases like Oracle use strings for bools. So if we follow this approach, will EF make these strings nullable too is they have a default of True?
Overall I think the team is doing a great job on EF, but it seems that sometimes the decisions are based on a Code-First dictate. In a db heavy shop that uses DBA's to develop their databases, when a developer comes to them and says EF is going to ignore your mandatory column with a default of True, (but by the way, you can use default is False) and change it's type, shops my rethink the use of EF.

Personally I would love to just use the model classes and simple annotations. Right now I get this bool default value warning and I am forced to use the modelBuilder for simple default values on table creation within SQL Server. I see no reason top set the value on the class property as it does nothing for IntialCreate so I was forced to use a config for every model, which is OK, I like the control but since we are looking into the warning why not fix it all, aye? I would like to use one or the other, right now I am using both e.g. [Required, StringLength(100)] and IEntityTypeConfiguration.
public void Configure(EntityTypeBuilder<SystemUser> builder)
{
    builder.HasKey(o => o.Id);
    builder.Property(o => o.Id).HasDefaultValueSql("newsequentialid()");
    builder.Property(o => o.Created).HasDefaultValueSql("getutcdate()");
    builder.Property(o => o.Disabled).HasDefaultValue(false);
 }

So can we safetly ignore "This default will always be used when the property has the value 'false', since this is the CLR default for the 'bool' type. Consider using the nullable" when setting default value on not nullable bool?

This does work but seems like a workaround.
// Add DbContext SSOContext using connection string from settings.
services.AddDbContext<SSOContext>(options =>
{
	options.UseSqlServer(_configuration.GetConnectionString("DefaultConnection"));
	options.ConfigureWarnings(wc => wc.Ignore(RelationalEventId.BoolWithDefaultWarning));
});

Note from triage: post 2.1 we should consider adding a flag that will cause reverse engineering to ignore column defaults when defining the EF model. However, some thought needs to go into how this would interact with update-from-database. Leaving it up to @bricelam to decide how to track this--using an existing issue or creating a new one.

Filed #10881 (Option to ignore DEFAULT constraints)

Notes from implementing this:

Some tools remove DEFAULT constraints by setting them to NULL. This doesn't remove the constraint, but effectively disables it.
Adding NOT NULL column requires specifying a DEFAULT (typically 0). We do this in Migrations.

Both these classes will now be ignored by Reverse Engineering. 🎉

@bricelam Are the default values generated by scaffolding used by EF anywhere?
Or are they there for roundtrippabilty

By reverse engineering .HasDefaultValueSql("something") EF will not specify a value in the INSERT statement (causing the default value to be generated), and after inserting it will propagate the generated value back into the entity.

So the actual value only really matters for round-tripping, but since we have to put something...

OK, so it is used by the Update pipeline - thanks! I will add them, then:
https://github.com/ErikEJ/SqlCeToolbox/blob/master/src/GUI/ReverseEngineer20/ReverseEngineer/SqlServerDacpacDatabaseModelFactory.cs#L263
And what about sequences in the Model?

Sequences, constraint names, non-unique indexes, and index filters aren't currently used by the runtime.

I ran in to this today too. I was wanting to set a default value with my code first DB generation, so that the column would end up with XXX bit NOT NULL DEFAULT 0
The reason I do this is if you are inserting manually (not through EF), you don't want to have to specify the default values. But if there is no default, you cannot omit them in your insert. Hence, a useful default.
Your line of reasoning seems flawed in that it centers on EF being the ONLY access path to the DB to consider. But what about my case where you want to generate a schema that is friendly to other access patterns, without a bunch of warnings?

I thought the newer versions were going to fix the need for BoolWithDefaultWarning on bool properties within a model.

@cgountanis That warning should not be generated when reverse engineering with the 2.1 tools. Can you please create a new issue describing specifically when you are seeing this? Please include the column definition and the generated code.

@ajcvickers https://github.com/aspnet/EntityFrameworkCore/issues/12198


Error thrown when trying to create a 'single-day' rental any day other than today

When I try to create a 'single-day' rental on any day except the current day it throws an error.
I believe this is because the rental is being created using a 'hidden-field' for the end date thats value is always the current date. So when the start date is changed to be after this set end date, the rental validations kick in and an error is thrown.

Going to write some javascript to update the end date when start date is changed


Iota

Codecov Report

Merging #328 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #328      +/-   ##
==========================================
+ Coverage   95.71%   95.73%   +0.01%
==========================================
  Files          60       60
  Lines        3595     3607      +12
==========================================
+ Hits         3441     3453      +12
  Misses        154      154

Impacted Files
Coverage Δ

...ch/src/details/DTK_DetailsTreeConstruction_def.hpp
100% <ø> (ø)
:arrow_up:

...ackages/Search/test/tstDetailsTreeConstruction.cpp
98.76% <100%> (-0.02%)
:arrow_down:

packages/Search/src/DTK_LinearBVH_def.hpp
100% <100%> (ø)
:arrow_up:

packages/Search/test/tstDetailsUtils.cpp
100% <100%> (ø)
:arrow_up:

packages/Search/src/details/DTK_DetailsUtils.hpp
100% <100%> (ø)
:arrow_up:

packages/Utils/src/DTK_KokkosHelpers.hpp
100% <0%> (ø)
: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 fad02b1...1c7b4ef. Read the comment docs.

@aprokop

I think that

iota should be KOKKOS_FUNCTION
It should not call Kokkos::fence or otherwise sync.

I disagree. iota(), as well as exclusivefPrefixSum() and lastElement() in the same header file, was meant to be called from the host.

OK. But even if it's called from host, there is not need to include fence in it.

So would you want to remove the fence() from exclusivePrefixSum() as well?
I am not convinced at this time, I would rather have it and consider removing it in the future.

Can you change the message of the second commit for_parallel() -> parallel_for()

@Rombur Done


Work out way to detect "partial" clips

For example:

I don't remember this happening in the past, but it happens when doing drag selection in Chrome 45.0.2454.85.

One possible way to do is this to check if the previous paste was a substring of the incoming one, although I worry that that might induce false positives.

Fixed in 45ac24d6255bef0b8d694567f61fbc7262bb9911


Question about multiple version compatibility of skywalking components (agents/OAP/UI)

Please answer these questions before submitting your issue.

Why do you submit this issue?
[√] Question or discussion

Question

What do you want to know?
I didn't find compatibility information for each component version in the official documentation and release notes, below are some of my understandings, could you please help confirm if my understands are correct? Thanks.

1， OAP Server version must be the same version as the agent version.
it means that if I upgrade OAP Server(6.4 to 6.6)，I must also upgrade all agents version as well.
is it correct?
2， UI version can be different with OAP Server.
ex: UI 6.6 query against OAP Server 6.4

There are different versions for protocols. You could check the difference between releases.
Such as agents in the whole 6.x  are compatible, just have enhancement after 6.4. Go check skywalking-data-collect-protocol
UI and OAP have query protocol too. skywalking-query-protocol.


feat: initial infrastructure work for api deployment.

Context
API Implementation for Data pipeline
Change proposed in this pull request
Adds the API to support extracting data created by the data pipeline
Guidance to review
Follows the same pattern as the data pipeline implementation.
Checklist

[ ] Work items have been linked (use AB#)
[ ] Your code builds clean without any errors or warnings
[ ] You have run all unit/integration tests and they pass
[ ] Your branch has been rebased onto main
[ ] You have tested by running locally

@Chris-Lorch would you mind casting an eye over this, as a 2nd pair of eyes. Thanks


Alias persists outside of rez environment

We noticed that an alias persists after leaving a rez environment.
Example:
Let's assume we have set up up a package 'alias_test' in which we define alias('alias_command', 'echo alias successfully excecuted')
In cmd:
C:\Users\me>rez-env alias_test

C:\Users\me>alias_command
C:\Users\me>alias successfully executed
C:\Users\me>exit
C:\Users\me>alias_command
C:\Users\me>alias successfully excecuted

Good catch. This is because it is created with doskey and a doskey macros are applied to parent shells it seems (just tested). This should not be the case with powershell. Doskey poses quite some challenges and we may have to rethink cmd generally i guess.

@instinct-vfx is right - it is doskey behavior.
There are no exit traps in batch - so it might get ugly to make this work properly - if possible at all.
This might not always be possible but I would also recommend using PowerShell instead for various reasons. cmd should be avoided since it is in maintenance mode and does suffer from severe limitations. That does not mean that you can not launch things that rely on batch files within the environment. You can (and will suffer the consequences 😜 ) - but PowerShell seems more suited to be the main driving shell (for rez) on Windows.

Thank you for the quick reply. I will look a bit more into using the powershell instead of cmd.

@nerdvegas I think we can close this as "wontfix" for now.
At most we could add documentation in the alias wiki section.

Agreed, cheers.
On Sat, Sep 14, 2019 at 3:43 AM Blazej Floch<EMAIL_ADDRESS>wrote:

Closed #709 https://github.com/nerdvegas/rez/issues/709.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nerdvegas/rez/issues/709?email_source=notifications&email_token=AAMOUSQU5SGHIP4M2KZ2D7TQJPGL3A5CNFSM4ISOJJ42YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTTY2VGI#event-2633083545,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMOUSTE5W7IVU6KTG4MEM3QJPGL3ANCNFSM4ISOJJ4Q
.


What is the right way to pass data to click event handlers?

Sorry. I've been looking around for a while, and can't figure this out. I have a list of links with unique IDs or whatever. When I click one, it want to pass some info along -which button got clicked.
With an input field, you can use name: 'foo' to populate the 2nd parameter (commonly called 'data' in the examples) with data.foo...
this works but might be the wrong thing to do:
'ev-click': hg.send(channels.selectUser, user_name)
thank you

That looks right to me.


Deleting collected counters though failed to createsocket or failed to send stats

In buffered client, if failed to createsocket or failed to send events due to connection errors, counters are being deleted in flush function.

8236369809bff53c304a986a4bbf8cfc93e78c46