Possibly the server has a path handling problem, possibly the URL configured is wrong.

Hi,
url is the right one and it shows me the calender in Homeassistant, but i get always this hint
Its an calDAV Calender  on a synology diskstation
Logger: caldav
Source: /usr/local/lib/python3.11/site-packages/caldav/objects.py:303
First occurred: 11:02:20 (2 occurrences)
Last logged: 11:02:21
Possibly the server has a path handling problem, possibly the URL configured is wrong. Path expected: /caldav/Dieter, path found: ['/caldav.php/Dieter/'] Please raise an issue at https://github.com/python-caldav/caldav/issues or reach out to<EMAIL_ADDRESS>include this error and the traceback and tell what server you are using. Continuing, probably everything will be fine

I should probably consider to simply remove that warning.

I have the same problem with my Synology DiskStation and Caldav integration in HA.
My calendars are displayed correctly.
However, I cannot edit, delete or add appointments.
Is this normal?

I also haven't figured out how to edit, delete or add appointments in HomeAssistant.

In my Google calendar, which I use as a Workday sensor and trash calendar, I can edit and delete appointments.
However, since I don't want to use the Google calendar for my office organization, it's a shame that the Caldav integration doesn't work here.
In my opinion, it can't be Synology's fault, as the Caldav synchronization works perfectly on two Android devices (DAVX-App) and a Windows PC (Thunderbird email), including all editing and deletion functions.

As for the "path handling problem" with synology, it may be related to my findings in #191, where the URL was configured as /caldav/, but reported back from synology as `/caldav.php/'.  It's unrelated to any editing issues however.
Writing, editing and deleting events in Synology should work - I did manage to run the test suite towards Synology while doing research into #191.
Writing, editing and deleting events in HomeAssistant using the caldav interface is antoher thing - I have the impression that this is a bit work under progress, but since I'm not directly involved in the HomeAssistant caldav integration I may be wrong on that account.

Thank you very much for the quick information.
I have changed the caldav integration in HA by adding 'caldav.php' to the path.
HA has accepted this. All Synology calendars are synchronized. Editing does not work via HA (maybe in the future?).
So far no more error messages.
What I have noticed, however, is that the HA integration in the caldav URL only accepts the IP address of the Synology. The domain name is not accepted.

I'm experiencing the same error on my Home Assistant docker setup and CalDAV package on my Synology.
Interesingly enough I saw the same error on my Android phone, which uses DAVx5 for syncing the calendar between my Synology DSM and my Android phone.
My thoughts so far is that Synology changed something in either their Calendar-package or their nginx configuration (which is used as a reversed proxy for all Synology packages with a web-interface), and that they changed the URL from caldav to caldav.php. This change got pushed to my DSM during an DSM-update or an Calendar-package-update. Not sure if this is really the case, but this is my working theory for now.

It's a warning, not an error.  :-) Interesting that DAVx5 is showing the same warning.  DAVx5 is not using this library, so probably not exactly the same error?

It's a warning, not an error. :-)
Edited ;-)

Interesting that DAVx5 is showing the same warning. DAVx5 is not using this library, so probably not exactly the same error?
Nope, it's not the exact same warning, but it does show a similar warning

Same issue here:
Logger: caldav
Source: /usr/local/lib/python3.12/site-packages/caldav/objects.py:303
First occurred: 11:43:59 (1 occurrences)
Last logged: 11:43:59

Possibly the server has a path handling problem, possibly the URL configured is wrong. Path expected: /caldav/Stiltjack, path found: ['/caldav.php/Stiltjack/'] Please raise an issue at https://github.com/python-caldav/caldav/issues or reach out to<EMAIL_ADDRESS>include this error and the traceback and tell what server you are using. Continuing, probably everything will be fine

Only happens on restart, though, and everything is indeed fine. In view of the comments above, I'll monitor a bit longer, then probably just filter it out.

@stiltjack known configuration problem, change /caldav to /caldav.php in the caldav URL in the configuration and the warning should disappear.


Use GitHub actions to build SCTP native binaries.

Also reorganize the SCTP repo to build as just a single Maven project.

Thanks for chiming in @ibauersachs ! ❤


BUG: Add Cursor Tail Effector in all pages

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I want to add a Cursor Tail Effector in all page, Events, Login, About us page
PA please assign me this task.
https://github.com/user-attachments/assets/d282517b-1f44-4141-b06b-4a084220c286
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I really like your app and want to work on your project ,please assign this issue to me under gssoc-ext 2024


time: marshaling wallclock time is lossy

What version of Go are you using (go version)?

$ go version
go version go1.16.3 linux/amd64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/user/.cache/go-build"
GOENV="/home/user/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/user/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/user/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.16.3"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3471220289=/tmp/go-build -gno-record-gcc-switches"

What did you do?
https://play.golang.org/p/xLZ0iwRQ0qu
What did you expect to see?
time.Time{wall:0xbab699fc00000000, ext:1, loc:(*time.Location)(0x5676c0)} "2009-11-10T23:00:00Z" time.Time{wall:0xbab699fc00000000, ext:1, loc:(*time.Location)(nil)}
time.Time{wall:0xbab699fc00000000, ext:1, loc:(*time.Location)(0x5676c0)} "\x01\x00\x00\x00\x0e\u008b\xe7p\x00\x00\x00\x00\x00\x00" time.Time{wall:0xbab699fc00000000, ext:1, loc:(*time.Location)(0x5676c0)}

What did you see instead?
time.Time{wall:0xbab699fc00000000, ext:1, loc:(*time.Location)(0x5676c0)} "2009-11-10T23:00:00Z" time.Time{wall:0x0, ext:63393490800, loc:(*time.Location)(nil)}
time.Time{wall:0xbab699fc00000000, ext:1, loc:(*time.Location)(0x5676c0)} "\x01\x00\x00\x00\x0e\u008b\xe7p\x00\x00\x00\x00\x00\x00" time.Time{wall:0x0, ext:63393490800, loc:(*time.Location)(0x5676c0)}

Summary
There are two pieces of information that are dropped during encoding: text drops location (this is documented in godoc), and both drop monotonic clock information. I assume the former was an unfortunate consequence of using time.RFC3339Nano, but due to hyrum's law you just documented the failure. As such, I would request this too be documented, since it caught me off guard, and I assume that it is infeasible to change.

This is documented in the package doc

Because the monotonic clock reading has no meaning outside the current process, the serialized forms generated by t.GobEncode, t.MarshalBinary, t.MarshalJSON, and t.MarshalText omit the monotonic clock reading, and t.Format provides no format for it. Similarly, the constructors time.Date, time.Parse, time.ParseInLocation, and time.Unix, as well as the unmarshalers t.GobDecode, t.UnmarshalBinary. t.UnmarshalJSON, and t.UnmarshalText always create times with no monotonic clock reading.

somewhat derivative, but what is the suggestion for interface boundaries within a process? is there a way to persist this monotonic information, say cgo or reflective use cases like testing.


TFLITE fix inconsistencies in using std/absl for variant/any_cast functions.

Fix using std::any_cast instead of absl::any_cast
Inconsistencies exist in Lite repo using std::variant/absl::variant and std::any_cast/absl::any_cast. When abseil is built with ABSL_USES_STD_ANY (Linux), everything works even with this bug. But if not (on Windows build), this uses std::any_cast on variable created by absl::variant, causing a throwing of exception at runtime.

Here are the internal errors, @radekzc can you please verify ? Thank you!
ld: error: undefined reference due to --no-allow-shlib-undefined: util_graphics_giza::BlendRgbaToRgbaIcon::DrawScanlines(int, int, int, int, int, util_graphics_giza::GizaOperator, Callback5<int, int, int, unsigned int, util_graphics_giza::GizaOperator>*) const

referenced by blaze-out/k8-opt/bin/_solib_k8/libutil_Sgraphics_Sgiza_Sinternal_Slibgiza.ifso

ld: error: undefined reference due to --no-allow-shlib-undefined: util_graphics_giza::BlendRgbaToRgbaIcon::DrawIntoIndexcolorBuffer(int, int, int, int, int, int, unsigned char const*, unsigned char*) const

referenced by blaze-out/k8-opt/bin/_solib_k8/libutil_Sgraphics_Sgiza_Sinternal_Slibgiza.ifso

ld: error: undefined reference due to --no-allow-shlib-undefined: util_graphics_giza::BlendRgbaToRgbaIcon::DrawIntoRgbaBuffer(int, int, int, int, int, int, util_graphics_giza::GizaOperator, unsigned char*) const

referenced by blaze-out/k8-opt/bin/_solib_k8/libutil_Sgraphics_Sgiza_Sinternal_Slibgiza.ifso
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Seems not relevant to the commit.

Hi @radekzc,
Could you provide the steps to reproduce the issue (breakages under Windows build)?

Will try to prepare minimal example, but it may take time.
If you are unsure what this PR fixes, its like if you wrote code like
int * a = new int;
...
free(a);   // free instead of delete

which would "work" if the compiler used
void free(void *ptr)
{
    delete(ptr);
}

which is often the case, but there is zero guarantee it works always, because the behavior is undefined.

Hi @radekzc Any update on this PR? Please. Thank you!

Hi @radekzc,
According to Google's policy, we are deprecating the usage of absl::any_cast and absl::variant in our codebase. To avoid the scenario you mentioned, Is it ok to fix all uses of absl::variant/any_cast to std::variant/any_cast?

Yes, perfectly ok.

Hi @radekzc Any update on the above comments? Please. Thank you!

Hi @radekzc Any update on the above https://github.com/tensorflow/tensorflow/pull/60797#issuecomment-1690990772? Please. Thank you!

Hi @radekzc Any update on the above https://github.com/tensorflow/tensorflow/pull/60797#issuecomment-1709240974? Please. Thank you!

Hi @radekzc Any update on the above https://github.com/tensorflow/tensorflow/pull/60797#issuecomment-1709240974? Please. Thank you!

Hello
Please also delete
#include
on top of header
"tensorflow/lite/delegates/gpu/common/operations.h"
It's broking builds with old compilers (require c++17).

Hello Please also delete #include on top of header "tensorflow/lite/delegates/gpu/common/operations.h"
It's broking builds with old compilers (require c++17).

Based on previous comments the agreed upon way is to completely remove absl and switch to std instead, which is the direct opposite of this commit. I can look into this when our next backend upgrade task (switching to newer TF version in our product) begins, which is due in April.


Hide startup messages with custom image

It would be really nice to be able to specify a custom boot logo and hide the system startup messages.
I would then start my app, which it's already doing using a systemd service, but it shows the login prompt for a couple seconds during the app startup that I'd also like to mask with the boot logo image.

Hi thijstriemstra,
i found a Script called asplashscreen with google. Maybe this hels you?
#! /bin/sh
### BEGIN INIT INFO
# Provides:          asplashscreen
# Required-Start:
# Required-Stop:
# Should-Start:
# Default-Start:     S
# Default-Stop:
# Short-Description: Show custom splashscreen
# Description:       Show custom splashscreen
### END INIT INFO

do_start () {

    /usr/bin/fbi -T 1 -noverbose -a /etc/splash/splash.png
    exit 0
}

case "$1" in
  start|"")
    do_start
    ;;
  restart|reload|force-reload)
    echo "Error: argument '$1' not supported" >&2
    exit 3
    ;;
  stop)
    # No-op
    ;;
  status)
    exit 0
    ;;
  *)
    echo "Usage: asplashscreen [start|stop]" >&2
    exit 3
    ;;
esac

:

sincelery Uwe

thanks for the suggestion @bassemolufda

Additionally you have to replace loglevel=3 with quiet in /boot/cmdline.txt.

Thanks for that addition @FooDeas. Is it possible to also hide the pi logo as described here with a new option?

Looks like video is also supported with the same asplashscreen script, very cool.

Note that self: the asplashscreen script also needs a # Required-Start: $local_fs according to this comment.

The asplashscreen script adding/customization should be done via install_files (see README.md).
The logo.nologo kernel parameter will be added to next version with the quiet_boot=1 option.

The latest raspbian has this nice 'pixel' startup image. Any way we can add an option that allows you to replace/add such an image?

Any info which way they implemented that?


[FIX] .

For working routes to work properly you need to add the base tag in your index.html document, this tag is necessary for the URL to be generated correctly. If you have problems with routes try to add the following example .
When creating a new PR, please delete this template text and reference the issue number.
To automatically close the issue when the PR is merged, cite the issue number in the first line with ...
Fixes #Issue_Number
To only address an issue with a PR and not automatically close the issue when the PR merges, cite the issue number in the first line with ...
Addresses #Issue_Number

Thanks @srburton
I'm going to make a few tiny changes to the language to conform to our style manual, and I'd like to link this over to our existing coverage for app base path.

I need to update the App base path section, too. When the change was made to relative paths, we didn't change the 'set it to an empty string' language over there. I'll set that PR up now.
Thanks again @srburton for flagging this! :rocket:


Fix: Cannot import Vmware instances

Description
This PR fixes the Vmware import unmanaged instances process, was failing due to a null detail value for SSHPublicKey:

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[x] Major
[ ] Minor

Bug Severity

[x] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
Verified fix by importing VM successfully

@blueorangutan package

@acs-robot a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6384 (SL-JID-1565)

@blueorangutan package

@shwstppr a Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan test centos7 vmware-67u3

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been kicked to run smoke tests

Trillian Build Failed (tid-4173)

@blueorangutan test centos7 vmware-67u3

Trillian test result (tid-4176)
Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
Total time taken: 41902 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr6384-t4176-vmware-67u3.zip
Smoke tests completed. 95 look OK, 1 have errors
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_02_verify_ipv6_network_redundant
Failure
175.52
test_network_ipv6.py

Tested failed test on a VMware env, error is unrelated and seems to be some intermittent issue,
[root@pr6385-t4175-vmware-67u3-marvin marvin]# cat /marvin//MarvinLogs/test_network_ipv6_TY1ONW/results.txt
Test to create network offering ... === TestName: test_01_create_ipv6_network_offering | Status : SUCCESS ===
ok
Test to create network offering ... === TestName: test_02_create_ipv6_network_offering_fail | Status : SUCCESS ===
ok
Test to create network offering ... === TestName: test_03_create_ipv6_vpc_offering | Status : SUCCESS ===
ok
Test to create VPC offering failure ... === TestName: test_04_create_ipv6_vpc_offering_fail | Status : SUCCESS ===
ok
Test to add IPv6 guest prefix ... === TestName: test_01_create_ipv6_guest_prefix | Status : SUCCESS ===
ok
Test to add IPv6 guest prefix failure ... === TestName: test_02_create_ipv6_guest_prefix_fail | Status : SUCCESS ===
ok
Test to verify IPv6 network ... === TestName: test_01_verify_ipv6_network | Status : SUCCESS ===
ok
Test to verify redundant IPv6 network ... === TestName: test_02_verify_ipv6_network_redundant | Status : SUCCESS ===
ok
Test to add IPv6 public IP range ... === TestName: test_01_create_ipv6_public_ip_range | Status : SUCCESS ===
ok
Test to add IPv6 public IP range failure ... === TestName: test_02_create_ipv6_public_ip_range_fail | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 10 tests in 1239.774s

OK

Also, tested import vm functionality with the test added in in #6385
[root@pr6385-t4175-vmware-67u3-marvin marvin]# nosetests --with-xunit --xunit-file=results.xml --with-marvin --marvin-config=./pr6385-t4175-vmware-67u3-advanced-cfg -s -a tags=advanced --hypervisor=VMware tests/smoke/test_vm_life_cycle_unmanage_import.py
/usr/local/lib/python3.6/site-packages/paramiko/transport.py:33: CryptographyDeprecationWarning: Python 3.6 is no longer supported by the Python core team. Therefore, support for it is deprecated in cryptography and will be removed in a future release.
  from cryptography.hazmat.backends import default_backend
/usr/local/lib/python3.6/site-packages/paramiko/transport.py:236: CryptographyDeprecationWarning: Blowfish has been deprecated
  "class": algorithms.Blowfish,

==== Marvin Init Started ====

=== Marvin Parse Config Successful ===

=== Marvin Setting TestData Successful===

==== Log Folder Path: /marvin/MarvinLogs/May_13_2022_08_08_47_OI6KDY All logs will be available here ====

=== Marvin Init Logging Successful===

==== Marvin Init Successful ====
=== TestName: test_01_unmanage_vm_cycle | Status : SUCCESS ===

=== Final results are now copied to: /marvin//MarvinLogs/test_vm_life_cycle_unmanage_import_M0XC3L ===


feat: use Node.js v15.6 crypto.randomUUID() functions if it's avail…

…able

Thanks for the PR! I'm not 100% sure about the current approach though 🤔
First of all, this tries to import crypto even in browser environments leading to the following error, as can be seen on the CI:

Module not found: Error: Can't resolve 'crypto' in '/home/runner/work/uuid/uuid/dist/esm-browser'

But even without that, since it takes the UUID string returned from randomUUID and turns it back into bytes I'm not sure that there is any benefit over just using the default rng which will be crypto.randomFillSync (with a pool) on Node.js? 🤔

How about falling back to randomUUID only when we don't need the bytes?
function v4(options, buf, offset) {
  options = options || {};

  if (buf == null && options.random == null && options.rng == null) {
    return randomUUID()
  }

  // ...
}

We would also need to solve the importing of crypto which I think we would do by creating two files, e.g. native.js & native-browser.js.
native.js would export randomUUID on Node.js platforms where it's available.
native-browser.js would export randomUUID once randomUUID is added to browsers.
This would then be configured the same way as e.g. sha1 and rng are package.json -> "browser".

I don't like current aproach too much too, I was thinking more about uuid working as a polyfill, but the buffer filling thing or the options made it more complex. I like your idea of using it as an optimization corner case. By the way, native randomUUID() already manages a data pool itself for up to 128 UUIDs by default.

Where are we at with this?

Implemented the changes I described in a new PR: #600
It gives an 11x speedup on my computer 🤩


NOISSUE: [release-4.13] Fix configure-vm.sh script to use release-4.13 branch by default

Necessary for QE procedures to build the right source code in the release-4.13 branch.

/assign @jogeo

How does the caller get the script to run? Could the caller clone the repo and the script require that it is present? That way we don't have to keep updating the content of the script every time we create a branch.

How does the caller get the script to run? Could the caller clone the repo and the script require that it is present? That way we don't have to keep updating the content of the script every time we create a branch.

At present, this script is copied to a brand new machine and it is the one that's cloning the repo. Since we're talking about the system bootstrap, it's a chicken-and-egg problem.

/label cherry-pick-approved

QE checks out the MicroShift repository, so we can use the configure script to setup the build machine. In one scenario  we download the RPMs from ART, so we don't build from source. In a second scenario we checkout microshift but then also builds from source. This second scenario is where we get into the situation that a release-4.13 branch has been checked out but then when the configure-vm.sh script is called it again checks out microshift but uses the main branch.

How does the caller get the script to run? Could the caller clone the repo and the script require that it is present? That way we don't have to keep updating the content of the script every time we create a branch.

QE is going to update our pipeline to make sure that ~/microshift is created the first time microshift is cloned, so configure-vm.sh will skip it's git clone call.

How does the caller get the script to run? Could the caller clone the repo and the script require that it is present? That way we don't have to keep updating the content of the script every time we create a branch.

QE is going to update our pipeline to make sure that ~/microshift is created the first time microshift is cloned, so configure-vm.sh will skip it's git clone call.

That feels like a good compromise for long-term maintenance.

/label backport-risk-assessed
/label  jira/valid-bug

/label jira/valid-bug


Suggestion for Website

I think that it would be really useful, if instead of just using kahoot-flood on the site if it would actually answer questions with random answers with kahoot-rand.
Please let me know if this is possible, and thank you for your hard work and dedication. Have a great day!

The website doesn't support kahoot-rand because that would require the site server to maintain open connections to Kahoot's server for a prolonged period of time. The website is designed to put as little strain on the server as possible (since I used to run it on my own server).
That being said, it should be possible to add kahoot-rand. I just don't plan on doing it any time in the near future, since I've pretty much abandoned the site idea.


Unable to specify folder-uri run with ./scripts/code-web.sh

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

VS Code Version:  1.73.0 71ab69d8496d11
OS Version:  Mac OS 12.2.1

Steps to Reproduce:

git clone VS Code Source code and open it with VS Code
Use  Cmd + B Command to Run Core - Build and Ext - Build these two tasks.
After the task is completed, because I don't need to use vscode-web-playground, I need to reassign folder-uri and extensionPath, So I should use ./scripts/code-web.sh --extensionPath=./extensions --folder-uri=memfs:///files/ to start VS Code Web here.
Then you will find that it cannot be started with a mistake.  It does not process this folder-uri parameter, and didn't pass it on to @vscode/test-web either.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!


Adopt the interoperability test to work on xpu

Xpu doesn't support dlpack yet, so I modified the interoperability test:

This test failed on xpu because the checks only applied to the GPU and only the CPU remained by default. I modified it so that if there is support for XPU, the modification will be very simple.
I added a test to check if for xpu the PJRT C API does not yet support these functionalities.

Solution proposed here https://github.com/google/jax/pull/17593/files#diff-7676567c0e266ec77721c81de9d2c0d4298f473b4f1e9fb74835107e5ecff9b2


Add support for float configs in VerifiableProperties.

This PR add support for method getGloatInRange in VerifiableProperties. This will be used in future PRs (specially by QuotaConfig).

Codecov Report

Merging #2042 (d4d119d) into master (339fcec) will decrease coverage by 4.24%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##             master    #2042      +/-   ##
============================================
- Coverage     73.65%   69.41%   -4.25%
+ Complexity    10065     9534     -531
============================================
  Files           722      723       +1
  Lines         54931    54935       +4
  Branches       6796     6797       +1
============================================
- Hits          40460    38133    -2327
- Misses        12316    14675    +2359
+ Partials       2155     2127      -28

Impacted Files
Coverage Δ

.../com/github/ambry/config/VerifiableProperties.java
72.60% <75.00%> (+0.13%)
:arrow_up:

...java/com/github/ambry/notification/UpdateType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...m/github/ambry/accountstats/AccountStatsStore.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...thub/ambry/notification/BlobReplicaSourceType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../cloud/LeaderStandbyHelixVcrStateModelFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...om/github/ambry/named/MySqlNamedBlobDbFactory.java
0.00% <0.00%> (-94.74%)
:arrow_down:

...ry/accountstats/AccountStatsMySqlStoreFactory.java
0.00% <0.00%> (-93.34%)
:arrow_down:

...a/com/github/ambry/network/NettyServerRequest.java
0.00% <0.00%> (-90.91%)
:arrow_down:

...y/network/http2/Http2ClientStreamStatsHandler.java
0.00% <0.00%> (-90.48%)
:arrow_down:

...ava/com/github/ambry/network/http2/Http2Utils.java
0.00% <0.00%> (-90.00%)
:arrow_down:

... and 110 more

Continue to review full report at Codecov.

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


String.Empty is manage as a space

Hi John,
In version 2.0.6149.33145 a empty string is manage such as a space and return only string containing a space.
Expected ==> all results.
context.Companies.Search(Company => Company.CIE_NAME.ToUpper())
.Containing(searchString.ToUpper())
.Select(Company => new
{
CIE_ID = Company .CIE_ID,
CIE_NAME = Company .CIE_NAME,
})
.Distinct()
.OrderBy(Company => Company.CIE_NAME)
.ToList();
Regards,

Sorry,
Error in version. Project updated and it's work.


Flag to set the edition of Visual Studio being installed

Hi,  awesome work on the chocolatey packages to install Visual Studio!  My present goal is to have a single chocolatey package with no parameters that installs Visual Studio, sets the license key and installs a pre-defined set of workload/components.
Making a modified copy of our visualstudio2017professional package that also inserts our corporate license key was easy enough.  I really love your idea of a 1:1 mapping of a chocolatey package with a workload.  So I figured I could write a chocolatey package with no powershell scripts and just declare the dependencies of visualstudio and the workloads that I needed.  The problem that I ran into is that dependencies are not installed in the order they are listed in the nuspec file.  So chocolatey tried to install some workloads before it installed chocolatey.  So I figured OK, I'll make variants of the workload packages - but I didn't want to make one for professional and one for enterprise, so I updated my visualStudio2017 package to decided on edition via a pass parameter.  That failed because your code is routing all of the chocolatey pass parameters to the visual studio installer.  I totally get why, but I'm wondering if your Parse-Parameters.ps1 code could ignore pass parameters with a "/" in them which would allow custom packages that use your code to support parameters without interfearing with the process.
Let me know if this is a possibility and I'll submit a pull request.  My present solution is to have the package that installs the workloads to have a dependency on the visualStudio and then iterate through a list of workloads and components in the chocolateyinstall.ps1/uninstallchocolatey.ps1 files.  I think having dependencies on the workloads is a more elegant solution - but I don't want maintain sets of workload packages that are identical except for the edition of VisualStudio they depend on.

The feature you are requesting, if I understand correctly (not passing certain package parameters to the  VS Installer), is aready present, in a fashion. The extension recognizes --no-xyz as a directive to remove the --xyz parameter, if one is defined, from the list passed on to the VS Installer. So you can write, for example:
cinst (...) --package-parameters "--visual-studio-edition Professional --no-visual-studio-edition"
and neither of those parameters will be passed on to the native installer. You can try it even now with the packages published on chocolatey.org.
This feature is implemented in the Remove-NegatedArguments function.
As for your suggestion for recognizing the "/" prefix, I'm afraid it would complicate the package parameters parsing code, which is currently very simple and robust, thanks to the existing rule that all parameters must start with  --, which is a rather uncommon sequence of characters in contexts such as paths. This makes it possible to avoid quoting paths and other parameter values containing spaces, for example.

If the "negated arguments" feature did not satisfy your needs, then instead of defining a new parameter symbol, I would be more inclined to recognize a special prefix of parameter names, such as --package:xyz (e.g. --package:visual-studio-product Professional).
Also, could you describe your planned package design in more detail? Are you planning to create a completely new package, depending only on chocolatey-visualstudio.extension, with a ChocolateyInstall.ps1 script that would:

invoke Install-VisualStudio, setting the Product parameter to a value extracted from package parameters,
invoke Add-VisualStudioWorkload/Add-VisualStudioComponent for each workload and component you wish to add?

Thanks for the response.  I think the 'ignored' parameter could work for some of my use case; however, I'm not as confident that I can use pass parameters at all.  I'll share some of the journey.
As a starter, I did implement the logic to filter out args with the "/".  I chose the "/" options because according to the online chocolatey documentation for parsing pass paramters parse-packageparameters-argument, they are recommending using "Get-PackageParameters" and show an example choco install <pkg_id> --params "'/LICENSE:value'".  This led me to introduce a parameter of "/edition:Enterprise" with a default of "Professional"
I did hack the Parse-Parameters.ps1 to filter out the "/" options with the code below.  The first line removes any "/" options followed by the "--" options.  The second line removes any "/" options at the end of the line.  The third removes any leftover space.
 9 Write-Debug "Starting 'Parse-Parameters' with s:'$s'"; 10 $s = $s -replace '/.+?--','--' 11 $s = $s -replace '/.+(?!--)$','' 12 $s = $s.trim() 13 Write-Debug "Running 'Parse-Parameters' with filtered s:'$s'";
This worked for the following test cases:
Parse-Parameters '/ignoreArgNoVal' Parse-Parameters '/ignoreArg SomeVal' Parse-Parameters '/ignoreArg2:SomeVal2' Parse-Parameters '--arg someval /ignoreArgNoVal' Parse-Parameters '--arg someval /ignoreArg val1' Parse-Parameters '/ignoreArgNoVal --arg someval' Parse-Parameters '/ignoreArgNoVal --arg someval /ignoreArgNoVal2' Parse-Parameters '/ignoreArg val1 --arg someval' Parse-Parameters '/ignoreArg val1 --arg someval /ignoreArg val2' Parse-Parameters '/ignoreArg:val1 --arg someval /ignoreArg:val2' Parse-Parameters '--arg someval1 /ignoreArgNoVal --arg2 someval2' Parse-Parameters '--arg someval1 /ignoreArg val1 --arg2 someval2' Parse-Parameters '--arg someval1 /ignoreArg:val1 --arg2 someval2'
Now, with all of that out of the way, let me describe a bit more about my need.
We're a fairly large organization with a lot of development teams using Visual Studio.  Some of them do C#, some do C++, some do python.  We have lengthly documents written by IT on where to find a local share drive to get the VS installer and install the corporate license key.  Then each team needs to write up it's own lengthly docment on which workloads and components to install.  This just results in every setup being an adventure and I want to make it boring.
To make it dead simple for development teams, I want a chocolatey package such as "visualStudio2019professional-myteam" which will install Visual Studio 2019 professional, the corporate license key and the workloads and components required by the team.  When a developer is installing the package is possible to ask them pass in pass parameters, but that starts to make it an adventure.  One of my visions is to enable teams to create their own 'meta' chocolatey package that has no playload but rather  declare the packages they need in the dependency list.  So a team member can just execute "choco install myteam-software" and the rest is automatic.  So far, the only way I've found to pass parameters to the dependencies is to require the user to provide the --paramsglobal option along with the parameters which defeats my goal of having the settings 'baked' into the package.
So, my current implementation is that I wrote 'k.visualStudio2019professional' which looks identical to your 'visualStudio2017professional' but also calls "StorePID.exe $licenseKey $visualStudioMpc" at the end.  So this bakes in our license key.  Then I created 'k.visualStudio2019professional-myteam' which declares a dependency on 'k.visualStudio2019professional'.  I want to also declare dependencies on packages to install for each of the workloads required by a team; however, since the workloads don't depend on the core visualStudio package and chocolatey does not install dependencies in the order they are listed in the nuspec file, I bump into problems where chocolatey is trying to install the workloads before installing visualstudio which doesn't work.  My present solution is to have the 'k.visualStudio2019professional-myteam' package iterate 'Add-VisualStudioWorkload' and 'Add-VisualStudioComponent' over the desire lists of workloads and components.
As I'm writing this, one argument for filtering out the '/' options is that I know I need to install "openssh" which uses '/' options.  If I create the metapackage with all of the software a team needs, I might need to pass --paramsglobal --package-parameters='"/SSHServerFeature /KeyBasedAuthenticationFeature"' for the openssh package - and that would tank your visual studio installer.
Anyway, you've done some great work.  I really appreciate it.

Thank you for the kind words :) I'm really glad you find the packages useful.
A couple of thoughts:

(some historical background) The "--" package parameter format actually dates as far back as 2016, when it initially appeared in the VS 2015 packages. Back then Chocolatey did not provide any facility for parsing parameters and there was no established guidance. When I saw that format, I liked its simplicity and lack of need for quoting or other special characters (in contrast to the format introduced later with the Get-PackageParameters helper). The machinery used for VS 2017 and 2019 evolved out of the scripts from VS 2015 packages, so the parameter format was carried over, too (and, by happy concidence, it matched the format used by the new VS Installer, which appeared with VS 2017 Preview 4).

If you are using only one edition of Visual Studio (Professional, I presume), then copying the workload packages might actually be the simplest way of implementing your idea of "-myteam" metapackages. There aren't that many workloads and the packages only need updating in case of a bugfix (and mostly in the form of bumping the extension dependency version), so performing a clone of the packages, changing the ids and adding dependency on the VS 2019 Professional package would be a really quick, easily automated, one-time task. You could also add some code to adjust the defaults, for example pass -IncludeOptionalComponentsByDefault or add something to $env:chocolateyPackageParameters.

How about putting the license key installation in a package by itself? If that is the only change you needed to make to the chocolatey.org visualStudio2019professional package, then with this approach you could just put a dependency on the license key package in your "-myteam" package and use the package from chocolatey.org unmodified, reducing maintenance work.
A function such as Set-VisualStudioProductKey would also make a fine addition to chocolatey-visualstudio.extension, if you were in the mood for making it robust and sharing with the community ;-)

Yes, the "paramsglobal" feature quickly runs into problems when you start adding completely unrelated packages (which themselves may depend on other packages). Myself, I've never really found that feature useful. It is also brittle - even if you manage to achieve a set of packages and global parameters that works one day, it may break tomorrow if a dependent package suddenly gains an optional feature and starts looking at parameters.
Creating a custom package, which sets parameters appropriately and invokes the installation code from the original package, seems to be the only option available in choco right now. The drawback is that the original package is not seen by choco as installed, so the user loses the ability to upgrade the software via choco, unless your custom package follows the updates of the original, which naturally generates maintenance burden.
One solution would be to extend the nuspec to enable providing package parameters for dependencies, e.g.
<dependency id="openssh" version="1.2.3" parameters="/SSHServerFeature /KeyBasedAuthenticationFeature" />

It might be worth suggesting on https://github.com/chocolatey/choco/issues (I'm rather surprised that nobody seems to have done it yet).
In my company we have solved the problem of "a standard team toolset" not via a custom choco package, but rather via a script, which essentially uses choco to install a list of packages, with appropriate parameters where needed. The advantages are simplicity and flexibility, at the cost of people not being able to detect upgrades to the toolset using choco only (without out-of-band information, such as a team- or company-wide announcement) - but that has never been a real problem for us in practice (whenever something is added, we just tell people to run the script again).

This is an unrelated matter, but perhaps as a user of the VS packages you could provide your opinion. I'm thinking of changing the package versioning scheme in order to accomodate both "latest" packages (which always install the latest released VS version, the way the current packages work) and new, "specific version" packages (which always install the specific VS version) under the same package id. The idea is described in this thread: https://github.com/jberezanski/ChocolateyPackages/issues/100. Does it look reasonable?

Closing due to inactivity.


feature: only log progress from stderr

Would it be possible to alter the library, so it can be set up to log progress only from stderr and not also from stdout ? I have an executable which I'm calling and it logs the correct output to the stdout and progress messages to stderr. Problem is, this gets mixed together in the .progress callback of a promise.

What about both get notified but with a type? We would have to modify notify to pass an object with maybe { type: 'stderr|stdout', data: buffer }

Yes, sure, that'd solve the same problem.

Can you do PR with this stuff?

Sure, can do that.

Thanks


Mobile Grid > Too Small

The main puzzle grid looks too small on the mobile phones screen. For desktop and tablet not an issue.

Ahh good callout, I want to make sure the keyboard is big enough to use and its pretty hard to fit all this information on the screen, but I'll see what I can do

Great. Thanks for your marvelous effort on this.

This is how it's look on iOS. The bottom got plenty of space after keyboard. Even if the puzzle board increase the size, the keyboard still could maintain the current siZe. Just my thoughts..

Hi @mmmewk Thanks for the PR add. I’m closing this as the issues resolved.


Test added for standalone tasks filter, rootTasksOnly test uncommented

Related to https://github.com/Activiti/Activiti/issues/2439
Depends on
https://github.com/Activiti/activiti-cloud-query-service/pull/181
https://github.com/Activiti/activiti-cloud-acceptance-tests/pull/110

@CTI777 I think that this one requires rebase.. can you please rebase it from master?


Fix errorformat of markdownlint

Some error messages of markdownlint contain also a character. This
caused the character to be interpreted as line and the line to be part
of the filename.
Example:
README.md:1:81 MD013/line-length Line length [Expected: 80; Actual: 129]

Closes #2480

Thank you!. @blueyed can we merge this one?

When exploding the list it shows that there is a duplicate entry now, which should not be necessary:
--- i/autoload/neomake/makers/ft/markdown.vim
+++ w/autoload/neomake/makers/ft/markdown.vim
@@ -29,8 +29,10 @@ endfunction
 function! neomake#makers#ft#markdown#markdownlint() abort
     return {
                 \ 'errorformat':
-                \ '%f:%l:%c %m,%f: %l: %m,' .
-                \ '%f:%l %m,%f: %l: %m'
+                \   '%f:%l:%c %m,' .
+                \   '%f: %l: %m,' .
+                \   '%f:%l %m,' .
+                \   '%f: %l: %m'
                 \ }
 endfunction

I don't know how the order should go.  Some example output might be helpful, and could be added to a test then also.

My bad with the duplicate in the errorformat.
I'm not sure about the %f: %l: %m format. I haven't seen it before but there is probably a reason why it has been added to neomake.
I've also added a test case for markdownlint.


Resurrect S4TF

@compnerd some time after I finish reviving the Swift kernel for Google Colab (link), I’m making a PR to this repo with major changes to the README. Since this would be going against Google's decision to archive it, I expect that you can't approve of the commit yourself. Is there any way we could contact people leading TensorFlow to get them to approve of un-archiving?
@BradLarson @rxwei @dan-zheng @lattner also pinging you, although @compnerd is the only person who seems to be able to make commits now.

For starters, the update date of the differentiable programming manifesto and implementation overview are out of date and misleading. Also, the guidance to try vanilla Swift in a broken Colab notebook is misleading.

We can wait to postpone merging the PR until after autodiff is fixed.

On the Testing section of Swift-Colab's README, most tutorials that don't directly depend on TensorFlow are salvageable. It's just Custom Differentiation that's the problem (it was also discussed when transforming the Differentiable Programming Implemenation into Markdown).


You should be able to create custom PluginDescriptors

The default visibility of the setters in org.pf4j, coupled with the lack of any kind of access to its private fields makes it impossible to create plugin descriptors for, for instance, YAML.  I'd propose moving org.pf4j.PluginDescriptor to an interface, creating a DefaultPluginDescriptor with (a) protected constructor(s) providing access to its fields.

See: https://github.com/josiahhaswell/pf4j/tree/PF4J-179

Resolved in https://github.com/decebals/pf4j/commit/5b192003f42c99b85131387017292955278eb0a8


No networking between two hosts in cattle

Rancher Version: v1.0.1
Docker Version: 1.10.3, build 20f81dd
OS: Debian 8.3 (Linux 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt20-1+deb8u3 (2016-01-17) x86_64 GNU/Linux)
Steps to Reproduce:

Start a new Cattle Environment
Join two nodes (one of them sharing)
Run any container
Network agent (rancher/agent-instance:v0.8.1) is added to the same host by Rancher
Start another container on another host and link them together using Managed network
Try to ping the first container by name: the name resolves to an 10.42 IP-address

Results:
# ping iperf-server
PING iperf-server (<IP_ADDRESS>): 56 data bytes
^C
--- iperf-server ping statistics ---
3 packets transmitted, 0 packets received, 100% packet loss

100% packet loss

Expected:
Network connectivity between the nodes
Errors:
two STDERR records are logged by Network Agent container:
4/25/2016 6:41:41 PM INFO: Downloading http://<IP_ADDRESS>:8080/v1//configcontent//ipsec current=
4/25/2016 6:41:41 PM INFO: Running /var/lib/cattle/download/ipsec/ipsec-1-bd2fca96de41d6264c4160a4099fb092f5f057e4960a9265420c85c497503d37/apply.sh
4/25/2016 6:41:41 PM RTNETLINK answers: No such file or directory

This one is caused by the following lines in the script:
if ! ip rule list | cut -f1 -d: | grep -q 200; then
        ip rule del iif $NETDEV table $TABLE pref 200 || true
    fi

Since there is no ip rule with "200", it cannot be deleted, so I guess it's not the real reason of issues we are having.
This one, however:
4/25/2016 6:41:41 PM INFO: Downloading http://<IP_ADDRESS>:8080/v1//configcontent//iptables current=
4/25/2016 6:41:41 PM INFO: Running /var/lib/cattle/download/iptables/iptables-2-939596f223e63c2702faba894f3cbf0b12e0953a2fff2c8e603c058407d80d4e/apply.sh
4/25/2016 6:41:41 PM SIOCSARP: Invalid argument
4/25/2016 6:41:41 PM arp: cannot set entry on line 2 of etherfile content-home/etc/cattle/ethers !

is caused by a recond in content-home/etc/cattle/ethers, trying to add arp record for an ip address, which is already assigned to eth0 inside rancher/agent-instance:v0.8.1 container:
root@0283f7baba25:/# cat /var/lib/cattle/download/iptables/iptables-2-939596f223e63c2702faba894f3cbf0b12e0953a2fff2c8e603c058407d80d4e/content-home/etc/cattle/ethers
<IP_ADDRESS> 02:40:a0:3e:b8:5d temp
<IP_ADDRESS> 02:40:a0:47:1a:00 temp
root@0283f7baba25:/# ip addr
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever
210: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default
    link/ether 02:40:a0:47:1a:00 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/16 scope global eth0
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/16 scope global eth0
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/32 scope global eth0
       valid_lft forever preferred_lft forever
    inet6 fe80::40:a0ff:fe47:1a00/64 scope link
       valid_lft forever preferred_lft forever

Have you looked at the docs about how to troubleshoot your cross host communication?
http://docs.rancher.com/rancher/faqs/#cross-host-communication


Setup CI to automatically build, run tests and publish NuGet packages

This is currently on-hold, because I want to use AppVeyor for this and they don't support VS2017 yet.


smartResize is causing General error: 1205 Lock wait timeout exceeded issue

I found the project really promising and testing revealed nice results. But in production the following code is causing multiple issues.
$image = new nwtn\Respimg($input_filename);
$image->smartResize($output_width, 0, true);
$image->writeImage($output_filename);
nwtn\Respimg::optimize($output_filename, 0, 1, 0, 0);

Second line, i.e. smartResize is often causing SQLSTATE[HY000]: General error: 1205 Lock wait timeout exceeded; try restarting transaction in my symfony application.
I know it is because my session table is causing trouble but it is due to the processing time of smartResize (exceeding to more than a minute for resolutions like 1000x1000 or even smaller)
Any clue how to reduce this time, I am using Ubuntu 14.04 4GB RAM (never found a high usage on new relic so the server is enough strong it seems).

Closing because this repo is not maintained


Role of international bunker emissions / global aggregates

Dear @IAMconsortium/common-definitions-emissions team,
I would be glad if you could decide about the treatment of international bunkers.
As far as I know, In the AR6 template, Emissions|* variables for BC, CO, CO2, NOx, OC, Sulfur and VOC had the international bunkers added only at the global level, leading to a situation that the sum over all regions did not match World results. In the NGFS project, we found out that the MESSAGE team also used this treatment for Final Energy|Gases and Primary Energy|Gas, while REMIND did it only for Emissions, and GCAM not at all. So there is definitely much room for confusion.
So I would be pleased if the variable description clarified how this should be treated. Thanks!
If I know that correctly, in NAVIGATE, the international bunkers were meant to be distributed to the regions, and I would propose to stick to that but make it explicit in the variable definition.

@volker-krey @strefler, please chime in here.

Yes, I think it makes sense to distribute the international bunkers (emissions and energy alike) to regions. If we don't do that, as Oliver noted the sum of regions will not equal the World total, and we still have emission and energy totals without bunkers if we need those values to compare to national inventories.

@volker-krey: An answer would be appreciated. We would like to "World = sum(regions)" for all variables in NGFS as well, and if we knew in the REMIND team that this is the future for the community template, that would make all our lives easier.

Making explicit the bunkers and summing regions = World, makes sense to me, because otherwise this is the type of insider information that few people know, confuses others and makes it look like we can't sum...

Maybe less of a quick-fix, and I assume this has been discussed already before, but wouldn't the addition of a "Unassigned"/"Other"/"International" region make sense, for cases like international bunkers, such that {REGIONS}+UNASSIGNED = WORLD?

I like the idea of adding an "International" region. We already use "Other" for model-regions that cannot be mapped easily to the R5/R9/R10 regions, so I'd not use that for bunker emissions.

Yes, agreed. Should not use "Other" again.
Okay, would there be anything against introducing the "International" region?


Cant close Lightbox

So i have lightbox added to my page but everytime i click on an image i cant close it.
The only things i changed in the lightbox.js file is that i have setted every quit value (except quitOnEnd) to true.
test.txt

Hi,
Did you use the proper selector? From the file you've included it doesn't seem so... Take a look at the docs included in this repo to find out how to do it...


[bug] Freezing on bootup

When I run VRCT, It freeze like image.
there is no log file in logs folder
It happens both installer & archive version.
before window open, black-backgrounded-texts show up left top corner and some of them are my font's name.
Windows 11 Edu / 22631 Build.
AMD 5800X / 64GB / RTX 3080Ti

add

cannot run install.bat with error below

i installed python with installer but similler error occured

Thank you for reporting. Okay, this happened again. I see...
We actually don't know why this happens yet.
I'll mark this issue as a duplicate and continue the conversation over there. Anyway, thanks.
Duplicate of #13

Duplicate of #13

When I run VRCT, It freeze like image.
there is no log file in logs folder
It happens both installer & archive version.
before window open, black-backgrounded-texts show up left top corner and some of them are my font's name.
Windows 11 Edu / 22631 Build.
AMD 5800X / 64GB / RTX 3080Ti

add

cannot run install.bat with error below
i installed python with installer but similler error occured

Errors occurring during the installation of "sentencepiece==0.1.99" seem to be due to using Python version 3.12 instead of the recommended Python version 3.11.5 by the developer. It appears to be a different issue from the error where the GUI does not load, as shown in the first picture.

When I run VRCT, It freeze like image.
there is no log file in logs folder
It happens both installer & archive version.
before window open, black-backgrounded-texts show up left top corner and some of them are my font's name.
Windows 11 Edu / 22631 Build.
AMD 5800X / 64GB / RTX 3080Ti

add

cannot run install.bat with error below
i installed python with installer but similler error occured

Errors occurring during the installation of "sentencepiece==0.1.99" seem to be due to using Python version 3.12 instead of the recommended Python version 3.11.5 by the developer. It appears to be a different issue from the error where the GUI does not load, as shown in the first picture.

yes it is, so I tried 3.11.9 and worked fine excuting install.bat


Add more projects to ProjectGrid

One  being the Portfolio-Page itself and then I'll just have to make a few other projects

Lgtm


npm install

Operation System:win x32
Beaker version:0.8.8
$ npm install
npm WARN deprecated<EMAIL_ADDRESS>gulp-util is deprecated - replace it, following the guidelines at https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5

Is there any reason to prioritize this?
On Mon, May 27, 2019 at 4:37 AM senlinmilelu<EMAIL_ADDRESS>wrote:

Operation System:win x32
Beaker version:0.8.8
$ npm install
npm WARN deprecated<EMAIL_ADDRESS>gulp-util is deprecated - replace
it, following the guidelines at
https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/beakerbrowser/beaker/issues/1408?email_source=notifications&email_token=AAJWCU3QZ24CQH7IVCFLLSDPXOTUNA5CNFSM4HP2IG6KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GV7RJLQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJWCU55EMZ6JWHEWYZ4RMLPXOTUNANCNFSM4HP2IG6A
.

I can chime with actual experience on this one. I'm having a... heck of a time trying to install anything Github  on dialup.  I was hoping for a how-to that could be shared to try to help out. Instead found them advising users to contact git-type project leaders to ask for help correcting that message within each of the half-gabazillion git-type projects that exist these days.
@senlinmilelu, because you're able to get this far in and understand the implications, is there any way you could please consider being Paul's #hero by finding something out there that helps short track what will fix this? Someone's pre-existing step-by-step of how they fixed this would be awesome. I gave it a real shot about a month ago... but got... we'll just called it "unfortunately sidetracked".. :sob:

From a brief look it would appear that gulp-util is not used directly. It was there in the original commit, so is probably from an electron app template.
However, the version of gulp-watch currently used includes gulp-util. Updating to<EMAIL_ADDRESS>and removing gulp-util, in the development environment's package.json will remove all mention of of gulp-util.

Is there any reason to prioritize this?
…
On Mon, May 27, 2019 at 4:37 AM senlinmilelu @.> wrote: Operation System:win x32 Beaker version:0.8.8 $ npm install npm WARN deprecated @.: gulp-util is deprecated - replace it, following the guidelines at https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1408?email_source=notifications&email_token=AAJWCU3QZ24CQH7IVCFLLSDPXOTUNA5CNFSM4HP2IG6KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GV7RJLQ>, or mute the thread https://github.com/notifications/unsubscribe-auth/AAJWCU55EMZ6JWHEWYZ4RMLPXOTUNANCNFSM4HP2IG6A .

有没有理由优先考虑这个？
...
在2019年5月27日星期一上午4:37，senlinmilelu @.>写道：操作系统：win x32 Beaker版本：0.8.8 $ npm install npm WARN已弃用 @ ** 。**：gulp-util已弃用 - 请按照https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5上的指南进行替换 - 您收到此消息是因为您订阅了此主题。回复此电子邮件直接，查看它在GitHub < ＃1408？email_source =通知＆email_token = AAJWCU3QZ24CQH7IVCFLLSDPXOTUNA5CNFSM4HP2IG6KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GV7RJLQ>，或静音螺纹< https://github.com/notifications/unsubscribe-auth/AAJWCU55EMZ6JWHEWYZ4RMLPXOTUNANCNFSM4HP2IG6A >。

New problems
$ npm install
npm WARN rollback Rolling back<EMAIL_ADDRESS>failed (this is probably harmless): EPERM: operation not permitted, scandir 'D:\dayanghang\beaker\node_modules\fsevents\node_modules'

beakerbrowser@ postinstall D:\dayanghang\beaker
cd app && npm install

<EMAIL_ADDRESS>install D:\dayanghang\beaker\app\node_modules\spellchecker
node-gyp rebuild

D:\dayanghang\beaker\app\node_modules\spellchecker>if not defined npm_config_node_gyp (node "D:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "D:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" rebuild )
gyp ERR! configure error
gyp ERR! stack Error: Can't find Python executable "D:\Python", you can set the PYTHON env variable.
gyp ERR! stack     at PythonFinder.failNoPython (D:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\configure.js:484:19)
gyp ERR! stack     at PythonFinder. (D:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\configure.js:509:16)
gyp ERR! stack     at D:\Program Files\nodejs\node_modules\npm\node_modules\graceful-fs\polyfills.js:284:29
gyp ERR! stack     at FSReqWrap.oncomplete (fs.js:153:21)
gyp ERR! System Windows_NT 6.1.7601
gyp ERR! command "D:\Program Files\nodejs\node.exe" "D:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd D:\dayanghang\beaker\app\node_modules\spellchecker
gyp ERR! node -v v10.15.3
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN<EMAIL_ADDRESS>No license field.

Looks as if you are missing part of part of the node.js native build environment, see node-gyp.

@senlinmilelu ... have a closer look at: https://github.com/beakerbrowser/beaker#building-from-source


Move a3u blueprints to develop branch

Move a3u blueprints along with all required files to develop branch to enable integration testing

Quick question: Why are we moving away from the older format of having the blueprint in examples folder directly? For instance, the way a3-highgpu blueprints are organised. Is this a long term refactoring that we would follow for future releases as well?

Quick question: Why are we moving away from the older format of having the blueprint in examples folder directly? For instance, the way a3-highgpu blueprints are organised. Is this a long term refactoring that we would follow for future releases as well?

Yes we should refactor to allow for saving the nccl tests as well, however we may want to dedup the nccl files in a single folder for GKE so we need to reorganize for sure.


Data with no id attribute

Hello everyone,
I'm using Material-Table to display a bunch of data I extract from my DynamoDB table. Each row of data has an id field but it's not called "id", it's actually called "SK".
Is there a way I can tell Material-Table to use "SK" (or any other name) rather than "id"? The other solution would be to add the "id" field when I get the data from DynamoDB but I wanted to avoid doing that.
Thanks in advance for your help!

That would be an interesting prop to add. Would you mind creating a PR to adress this?


CF Bridge overloads Abacus

On startup the CF Bridge POSTs enormous amount of usage reports that triggers Abacus circuit breakers. On non-loaded systems this usually involves around 100 000 events/posts.
We can delay the submission of data and offload Abacus by implementing some soft of back-off logic.

On startup the CF Bridge POSTs enormous amount of usage reports that triggers Abacus circuit breakers.

Do we know which ones are moving to open state?

We can delay the submission of data and offload Abacus by implementing some soft of back-off logic.

I believe throttling the submissions and batching the submissions could help.. you may want to look at the performance test as an example client that submits usage using throttle and batch [1].
[1] https://github.com/cloudfoundry-incubator/cf-abacus/blob/master/test/perf/src/test/test.js#L276-L279

Thanks for the hint. We already added a simple exponential back-off policy when the POST fail. However we'll definitely need to batch the requests as well to reduce the latency, traffic and the load on Abacus.
Is there an easy way to check all circuits? Something like the Hystrix's monitoring tools? For sure we saw an open circuit in dbclient module, but that does not identify the process/app, thresholds and all.
Even with batching we may need to address this issue by creating a sizing guide for Abacus. There is a question in this direction here<EMAIL_ADDRESS>

Adding batching helped to reduce the errors from Abacus. However we found out that with the default configuration we soon reached:

OOM in Pouch (dbserver) - increased memory to 4GB
OOM in cf-bridge - increased the memory to 2GB

We probably need to spend some time running the performance tests and tuning the apps, including the default values for batch sizes, throttling, retries, timeouts and instance numbers, to get to a point where a client sending a peak of usage docs doesn't cause errors or timeouts.

@hsiliev, we actually have support for Hystrix in Abacus.
@sasrin, could you point us to the work you did with Turbine and Hystrix, and maybe add a little doc describing how to set it up to monitor Abacus. Thanks.

@sasrin, maybe create a doc/monitoring.md file...

Pushed improved version of the bridge with batching and exponential back-off is requests fail.

https://github.com/cloudfoundry-incubator/cf-abacus/commit/938dbf52ce6dfc2f9e82e3c0a61f547a422b35ab closes this bug


ServiceWorkerContainer.controller prose says to return undefined, but the IDL doesn't allow that

The IDL says:
readonly attribute ServiceWorker? controller;

and undefined is not a valid ServiceWorker? value.
What you would probably want is for the property to not exist in a secure context at all, right?

We plan to discuss this in web app security WG meeting what the expected behavior for getter in non-secure context should be. I'll update the spec depending on the result.

F2F: update spec. Property won't exist in non-secure contexts

Closed by 5264a72e5336b7440e3a3b3cc9e84c05d3dd3a89


Sync via Drive

I hope a Google Drive sync will become reality.

This is a demo. We have no plans to do these things.


*** No rule to make targe**

*** No rule to make target /xxxxxx/build-android/build/qgis/7d58502b3b0a615ba1f6a4a8adab5465e18caf56/resources/function_help/json/$$area', needed by src/core/qgsexpression_texts.cpp'.  Stop.
build on macOS catalina, QT 5.15.0

See https://github.com/opengisch/OSGeo4A/issues/63#issuecomment-646250171

Likely related to a recent macOS issue of QGIS which has been fixed in qgis master.


Bastion NSG deployed when parAzBastionEnabled -eq $false

Describe the bug
When setting Bastion to disabled the NSG is still deployed
To Reproduce
parAzastionEnabled set to false

Deploy
Check NSG

Expected behaviour
Expect that the NSG is not deployed; and can be fixed in hubNetworking.bicep line 368
resource resBastionNsg 'Microsoft.Network/networkSecurityGroups@2021-08-01' = if (parAzBastionEnabled) {
}

Thanks @woutermation for the issue
Would you like to submit a PR to fix this that we can merge in?
Let me know
Thanks
Jack


Me too,but i have downloaded it from other website.So you can try too!

Not only the software itself... all the extension market is helll slowww... i can never download any extension at first.. after a whole day trying.... eventually one or two get install....

  We want to make this better.

@seanmcbreen Using google chrome on mac at Shanghai ->
Download doesn't go up 20kbps

and.. the extensions.. i am still trying to install and i still couldnt...

the extensions  cannot  install now

@seanmcbreen  we download vscode need 40 mins or 1 hour, amazon s3 server very very slow in china. maybe you can put the zip file on microsoft server.

when I recommend vscode to my workmate, it's so slowly to install extension so that I give up

@Milewski, @twang281314,
We have fixed the slowness issue for extensions. I believe it is pretty fast now. Please check and reach out if that is not the case.

Vishnu

Great! download completes instantaneously now ~

Closing as per CDN improvements from the Marketplace.

The download speed is at about 0-10K/s in China using the CHINATELECOM broadband, the system is Ubuntu 16.04...  and after 2 hours only finished 59% using uGet....

cc @hamenon @modigrv

@solenovex - Is there a particular extension that takes longer to download (or) it is taking longer for all the extensions?


Consider adding slug to the current/latest documentation

Consider adding slug to the current/latest documentation, so that

https://duckdb.org/docs/data/csv/overview becomes
https://duckdb.org/docs/XX/data/csv/overview

As discussed in #941, our options for XX are:

current (was changed in #544)
dev (is currently used in the version list. It is also used for the dev documentation)
latest
next
nightly
snapshot

Hannes suggests dev

Superseded by #1732


Question: Can I add create my own window object for browserify functions?

I'm trying to use browserify to pack up all the modules for my React Application, but I want to be able to use the bundle on the server-side.  I know this is a stretch, but there's a few things I want to do during the browserify process and I'd rather write browserify transforms than webpack ones.  Anyway, when I try to run my bundle through express, I (obviously) get window reference errors.   So I was wondering if it would be a problem to use jsdom for my server-side window?  I tried something like this at the top of my entry file, but to no avail.
var jsdom = require('jsdom').jsdom;
global.document = jsdom('<!doctype html>',{
    features: {
        ProcessExternalResources : ['script', 'iframe'],
        FetchExternalResources : ['script','img','css','frame','iframe','link'],
        MutationEvents: true,
        QuerySelector : true
    }
});
global.window = global.document.defaultView;
global.navigator = window.navigator;

Any suggestions?  Thanks.

@adjavaherian Thanks for your interest in browserify! This question is really outside the scope of the browserify issue tracker -- it's better suited to somewhere like stackoverflow. But fundamentally, yes, I don't see what would stop you from adding properties like window|document to global when running in node, although I'd expect some of that code to crash when run in the browser.


A build for Scala 2.10 in maven central

Is it possible to upload a build for Scala 2.10 in maven central repository?

Hi @dnrusakov! Sorry for the (very) late reply, I totally forgot this issue...
There was some time ago a pull request for adding a Scala 2.10 build, but it had some problems and it had to be reverted. I'm willing to cross build for 2.10 if it compiles without major modifications to the existing source code. Feel free to submit a pull request if you can get it to work :)
Even if a build gets to be published for the current version, please be aware that I won't guarantee that future versions will also support 2.10. As a rule of thumb, I'll drop support as soon as it keeps me from using a feature available only on newer Scala versions (either in the standard library itself or in a library that is only released for Scala 2.11+).

Hi! Actually i no need a Scala 2.10 support anymore. But anyway, thanks!

Good! :) Sorry again for the delay.


[Bug]: SSR - popover not showing up on click

Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.

Bug type
Functionality
It affects the following packages
Components
Which version of Lyne Components are you using
1.7.0
Operating system
MacOS 14.3
Browser / Browser version
Chromium 127.0.6533.120
Input mode
Mouse/Keyboard
Additional settings
No response
What happened?
When using SSR, popover does not show up on click.
It works perfectly without SSR.
Steps to reproduce the issue

open https://codesandbox.io/p/devbox/sbb-popover-hover-trigger-v575qq
make sure the server is running, otherwise run npm start on the terminal.
click on the popover trigger
notice that the popover is not showing up

@jeripeierSBB so fast 😎 i guess you have some sort of AI-Coder in your team who's creating a fix and a PR as soon as an issue is reported? ;)

@feerglas pssst, don't tell anybody ;-). In this case, it was an easy fix. The main work was to create tests :D


On Notebooks, add a "Copy to another workspace" option

We want to give users the ability to send interesting notebooks from example workspaces like those in the Analysis Library to their own destination workspace.
Users should have this option from the tools card/list view, or from the tool details page. I'm suggesting this be a push, since we expect one of the main paths to be from the analysis library. We'll have other affordances in the workspace for to guide users to go find code.
Selecting the "Copy..." should allow users to select an existing workspace, or create a new one using the standard modal.
The result of this should be that the notebook file of interest is copied to the destination workspace bucket, creating a second artifact that can subsequently diverge from the source.

Check with @lstrano-broad before starting RE design.

If the workspace being copied into already has a notebook of the same name, warn user and give the option to override or to cancel/rename before copying


Can console.log(...) support multiple params?

dart has console.log(..) ?

@Silentdoer dart:html

We'll likely wait until we have https://github.com/dart-lang/language/issues/1014 – same for print

Thanks for filing the issue.
Until this is addressed, note you can use JS-interop to circumvent the problem:
@JS()
library foo;
import 'package:js/js.dart'
...

@JS('window.console')
external myConsole(arg1, arg2, arg3);

...
main() {
  myConsole(a, b, c);
}

or:
import 'dart:js_util' as js_util;
import 'dart:html';
...
main() {
  js_util.callMethod(window, 'console', [a, b, c]);
}

I was able to find a very similar and old request here: https://github.com/dart-lang/sdk/issues/19977, so I'll close this as a duplicate of that.


Wolf logo?

Anyone wants to craft some nice logo for the project?

Does it have to be a wolf ? They're quite connotated in the art world I would say, though I like FF7 Cloud's wolf design https://finalfantasy.fandom.com/wiki/Fenrir_(Final_Fantasy_VII)

Tried something on inkscape just now

That's cool, it looks straight out of a Monster Hunter icon

Originally the name started as the wolf that howls at the moonlight, I think it would be cool to add that to some hypothetical logo.

I quite like it!
Would you like to open a PR and drop the file in the docs/images somewhere?

No problem. It will need work to be readable at low resolution.

Done in https://github.com/games-on-whales/wolf/pull/31


The value of coordinate in .json file

I used 640×480 UnityEyes image data, and  check the size of output data is 55×35.
Next, I want to correct the value of coordinate in .json files.
I read gaze_data.py, but I only checked correction of coordinate point in crop part.
Will .json file be modified to 55x35 when the output image is refined?

@KentaroOtomo
Hi,
Did you figure this out? Would you please give me an example of your json files?

What should I have as "*_cropped.png" self.synthetic_data_paths. These are cropped versions of the synthetic images?


Timer doesn't look working precisely if mobu goes background

Problem
mobu's timer doesn't work precisely if the browser tab that shows mobu is not active.
For example, even if the interval is 15min, sometimes the alert will be notified about 30 or 40 minutes later.
I think typically this occurs a user who is the mobu facilitator who is also the mobu driver.

This is reported from @mpls104, thanks!

Nice report!
I think we have 2 ways.

Worker: 🤔 It cant keep many current implementation, however make complex and annoy js file management.
Calculate with actual duration instead of increment: I 👍 this way, because it can drop many state handling. However it might make hard to integrate pause future.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_05s4kydf0h/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_05s4kydf0h/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_05s4kydf0h/clone/recipe/meta.yaml:47: http_url: http://pyahocorasick.readthedocs.io/en/latest/ is not https
Errors were found


[BUG]: Fix PyRight GitHub Action to Use Conda Environment

Operating system
Whatever Operating System Github Actions use (linux?)
Bug description
Before running the pyright github action that runs a static type check on the backend, have it install the conda environment and poetry packages first.
Steps to reproduce
PyRight currently will complain about basically every python package rn bc it doesn't install the conda environment yet.

this should be fixed by the github action workflow revamp project @dwu359

https://github.com/mamba-org/setup-micromamba
mamba setup github action

https://github.com/mamba-org/setup-micromamba/issues/148

@dwu359 I updated the backend build check action to add poetry env to path before running pyright action. The changes are currently implemented in Prisha's PR (training-test-migration)


NVIDIA GeForce Experience telemetry not blocked

Hi y'all:
Just noticed that telemetry.gfe.nvidia.com is not blocked by this list or any of the other lists Pi-hole uses. Probably a good candidate to add to the list!

is not blocked by this list or any of the other lists Pi-hole uses
I guess there is a reason for that.If you using GeForce Experience some of the features will stop working without it(i've seen complaints in the past).
If you not using it,just unzip(unrar) the .exe and get rid of the whole telemetry folder before install the driver.

Just following up on this one( i felt that i had to).Well...looks like as of their latest driver 430.39 Nvidia backed up their telemetry deep into the driver and whatever worked in the past(unrar+remove some of the folders),does not anymore. Plan B is to block the reporting domains but i'm still not sure if there will be any side effects for whoever is using GeForce Experience.I don't use it personally and i blocked them on my end and so far so good.More feedback is needed if Steven @StevenBlack  decide to add them,for sure.Attaching here screenshot from the created Nvidia telemetry log,which points all the domains for reference and testing:

NV telemetry seems to be necessary for updating profiles especially with their new AI features.

new domains or some might say...addition:

Everybody happy if I close this?

👍


[BACKLOG-8635] - RequireJS manager dependencies working without path workaround on index.html.

@francisco-guilherme @candentira @plagoa Please review

Tested this and it works. Don't have permissions for merging though. Can someone else merge this?


Feature: Champions page with pictures

A simple page for all the years with a picture of the winning team and a link to their team page
Might have to be a bit ad-hoc

Moving to discussion for now


Investigate buildpack-deps/alpine

Can we make an image that works on this instead? Do we need to recompile a custom flavour of swift that'll work on a non-debian distribution?

First step – let's try to build this on alpine and see about the file size difference.

I gave it a shot. While the whole thing builds, there is an ABI mismatch. Swift is not runnable in Alpine at this point.

We need to recompile it against muslibc I guess?

I might have posted this in the wrong issue, but I made a comment re: compiling Swift against musl here @swizzlr: https://github.com/swiftdocker/docker-swift/issues/47#issuecomment-275280880

@aidansteele cheers. closing this in favour of #47


Adding welcome message to the interpreter

After this it prints at the starting point of the repl the following message:
Crystal interpreter 1.6.0-dev [a4704af21] (2022-09-22).
EXPERIMENTAL SOFTWARE: if you find a bug, please consider opening a ticket in
https://github.com/crystal-lang/crystal/issues/new

I would like to have an environment variable to disable this message
It might be good to set it in bin/crystal to avoid noise.


Introduce Zone#maintenance?

When I was profiling, I noticed that it was looking up the maintenance Zone when inserting an MiqQueue record.
This is probably a moot point as this will only happen.

But I did like how Zone#maintenance? read.
And I wondered if we wanted to /^__maintenance__/.match?(zone.name) instead of the MiqRegion.my_region.maintenance_zone == zone?
And I did find an issue with the cached Zone#maintenance_zone in one case.

There were a reasonable number of tests on the maintenance_zone concept.

In general I like this PR because I agree it's much cleaner, so let's continue with it.

update:

uncache maintenance_zone (since my_region is cached elsewhere)
s/Zone/described_class/g
updated Zone.maintenance?() to use case

Still wondering if we want to just compare the string with /^__maintenance__/ and not force the maintenance_zone to be loaded at all.

Still wondering if we want to just compare the string with /^maintenance/ and not force the maintenance_zone to be loaded at all.

I recall having this discussion in the original PR, but I can't remember how we got the current code - you should probably check that out and see if there was a reason there.


tests: fixed mising rhc_state parameter in release tests

test release setting with force registration should have rhc_state: "reconnect" defined

Signed-off-by: Archana Pandey<EMAIL_ADDRESS>

[citest]


sql 2.0.0 release

vapor/sql rewrite:

SQLQuery is now completely protocolized
specific components (such as SQLBind) can be overridden with concrete components as desired for custom behavior
most protocols have Generic... concrete types supplied by this package that conform to standard SQL
no more generic serializers, each type is SQLSerializable and worries about serializing itself.

Hey @tanner0101, you just merged a pull request, have a coin!
You now have 805 coins.


Select widget doesn't respect "required" config option

Describe the bug

Setting required: false in the configuration options of a select widget seems to have no effect. You will still get and error saying the value is required.
Applicable Versions:

Netlify CMS version: 2.3.0
Browser version chrome 70

CMS configuration

      - {
          label: 'Building Nr',
          name: 'building',
          widget: 'select',
          options: { 'gebouw 1': 1, 'gebouw 2': 2, 'gebouw 3': 3 },
          required: false,
        }

Additional context

@0x80  try something like so:
      - label: 'Building Nr'
        name: 'building'
        widget: 'select'
        options:
          - { label: 'gebouw 1', value: 1 }
          - { label: 'gebouw 2', value: 2 }
          - { label: 'gebouw 3', value: 3 }

Does the issue persist after updating to the correct syntax as described here https://github.com/netlify/netlify-cms/issues/1939#issuecomment-445036530?


Razor SDK shouldn't generate the MVC recompilation attributes unless its necessary

Describe the bug
A web SDK project that targets netcoreapp3.0 with no razor files will unconditionally have these attributes:
[assembly: Microsoft.AspNetCore.Mvc.ApplicationParts.RelatedAssemblyAttribute("SampleApp.Views")]
[assembly: Microsoft.AspNetCore.Razor.Hosting.RazorLanguageVersionAttribute("3.0")]
[assembly: Microsoft.AspNetCore.Razor.Hosting.RazorConfigurationNameAttribute("MVC-3.0")]
[assembly: Microsoft.AspNetCore.Razor.Hosting.RazorExtensionAssemblyNameAttribute("MVC-3.0", "Microsoft.AspNetCore.Mvc.Razor.Extensions")]

This adds extra runtime references to types that are probably not used.
This logic needs to be updated to consider whether you have any .cshtml files, and/or be based on some explicit gesture in the project file, or just obsoleted.
Basing this on the presence of .cshtml files could cause issues in the case will cause issues in the following cases:

You have no cshtml files at build time, and you are using runtime compilation when published
Youe application is an F#/VB project, referencing class libraries for views, and you are using runtime compilation

It's also an impactful change, because it requires evaluating an item group to evaluate a property, which means that it has to be in a target.

The RelatedAssemblyPartAttibute is purely for discovery of compiled views/pages and should be made conditional without reservation.
The Razor.Hosting attributes are for the settings used by runtime compilation. We could consider obsoleting these now that runtime compilation is outside the shared framework. However - we need to consider the impact on downlevel. If this logic is inside the Razor.Design package in 2.1/2.2 we can remove it from the SDK and say that it's not needed/supported in 3.0.

/cc @pranavkm

Closing this based on the work happening in the linked issue, where we will introduce a dedicated flag on the project level, which will indicate whether those attributes should be generated or not.

That doesn't address this issue.

This is needed in preview4 (at least with the current state of things).The WebSDK specifies AddRazorSupportForMvc. Some Blazor templates reference the WebSDK, so you now get compilation errors stating it can't find Mvc.


Bug in PelletCount visualizer?

The x-axis here doesn't make sense.

is there a way you can confirm whether the x-axis values are resetting back to zero on block transition?

yes, this does appear to be the case


Dynamic attribute itemExtent in ListView

Hi, below is my Flutter version

Flutter 0.5.8-pre.184 • channel master • https://github.com/flutter/flutter.git
Framework • revision 3a6228bead (2 hours ago) • 2018-07-30 18:07:23 -0700
Engine • revision af855e063a
Tools • Dart 2.0.0-dev.69.4.flutter-08f59e5de3

I have a long heavy list is placed on a tab (I use DefaultTabController, not CupertinoTabView) and have a demand that the list have to be kept as last scrolling position when tab is reopened.
It can be done with set PageStorageKey to ListView, but the problem is performance. Suppose the list is being at bottom of scrolling position, when tab reopen then all items of list have to be built and that affect to performance and make my application is slow.
I can solve if I set itemExtent value explicitly, but the problem is my row height is dynamic base on its content. Can I ask Flutter to support to able to set row height on demand base on item index like IOS UITableViewController mechanism? Some thing like:

ListView.builder（
　itemHeightBuilder: (int index) {
　　if (index % 2 ==0) {
　　　return oddHeight;
　　}
　　return evenHeight;
　},
),
Thank you!

If the list usually is scrolled to the bottom, you probably want to use reverse: true

Thanks, I know about the reverse, I just want to take an example to describe about performance problem when change scroll position without setting itemExtent

https://github.com/flutter/flutter/issues/24791#issuecomment-442566058 seems related

Is reverse: true fixing the issue for you? If not, what solution would you like to see in the framework?


Modified user model to include 'teamName' and 'orgName'

Description
A summary of the change
Modified the user model so that it now has fields for team name and organisation. Removed the previous fields for given name and family name as users no longer have their own account.
Note: All the existing users in our database have to be removed or modified manually as they have the givenName and familyName fields present over teamName and orgName.
Type of change

[ ] Bug fix
[ ] New feature
[x] Refactoring
[ ] Documentation

Problem
Briefly describe the root cause and analysis of the problem
Users from the same team now use a team account to log in to Researchify. Previously we assumed each user will have their own account so we had givenName and familyName fields in our model.
Solution description
Describe your code changes in detail for reviewers.
Explain the technical solution you have provided and how it fixes the issue.
Removed givenName and familyName fields and replaced them with teamName and orgName.
Tests
All unit tests pass?
N/A
Builds successfully?
Yes
Attach screenshot of the expected result (e.g. UI of the website/ postman api call result/ database changes)
Tested all the api requests we had for the user model on insomnia and they all work fine. Can attach screenshots if requested.
Provide steps in detail for reviewers to reproduce the expected result of the code changes.
Relevant document/ link (if any)
Any related documents could help reviewers to review this PR
Risk
Identity the risk level (low, medium, high) and indicate which component(s) will be affected if bugs exist.
Low

Looks good to me! Just wondering do we not need fake team name and org name in the seederScript anymore?

The seederScript will just populate the table with data every time we run the application, assuming that data does not already exist in the database. We can just manually add in a fake user instead so this is not an issue.


Close session banner should warn clearly if the session is not persisted

This issue is about this close session modal right?

What is the idea, it will be just a change on the text?

Cool, I would like to try to make this change, can I?

So, i ended up with this message when the session has a file attached to it

And this message if the file property on assigns is nil

The new message is good? We can improve that or it's okay to mantain your suggestion @jonatanklosko?

Looks good to me! @josevalim should we show "not persisted" in bold or something, just to catch more attention?


Make accessibility consistent (code fix for CS0051)

If a class references another class that is less accessible you will get CS0051. For example
public class Dependent
{
    public Dependent(DependendedUpon d)
    {
    }
}
internal class DependendedUpon
{
}

This will raise CS0051 on the first caracter of the constructor identifier (the first letter D).
The code fix should simply change the accessibility of the less accessible class. It would change the above code to (showing only the changed code, the first class stays the same, note the public modifier):
public class DependendedUpon
{
}

There is no need for diagnostic id (we are using the CS one, category or visibility, as this is only a code fix provider.

@giggio it´s a very common error. Will be a very useful fix.

@carloscds Ok, I am setting it as Ready, then.

I missed the idea here. Please, man explain it a little bit more.

@ElemarJR Try the first code and you will see CS0051 raised because the DependentUpon class is less accessible than the Dependent class, that references it. A public class cannot reference in a public way an internal class, for example, in a constructor argument.

@giggio, can I work on this one?

Sure, go ahead.

@giggio maybe we should open issues for other inconsistent accessibility related errors? Or at least code fix for method return type?
    public class Dependent
    {
        public DependedUpon Method()
        {
            return null;
        }
    }

    class DependedUpon { }

@baks I thought this kind of problem would also be raised with CS0051, but I tried your example and it raises CS0050. As the code fix for both CS0050 and CS0051 is the same, which is to open up the dependent class, we can fix both of them in this issue.
Is there any other we should consider? I found CS0052, that is also the same problem. See:

CS0050
CS0051
CS0052

(There is no CS0049 or CS0053, so maybe this is it)

Yes, there are other compiler errors connected with incosistent accessibility :

[CS0053] (https://msdn.microsoft.com/en-us/library/532wtsbc(v=vs.90).aspx)
[CS0054] (https://msdn.microsoft.com/en-us/library/0efy9xb8(v=vs.90).aspx)
[CS0055] (https://msdn.microsoft.com/en-us/library/6b0x5w9f(v=vs.90).aspx)
[CS0056] (https://msdn.microsoft.com/en-us/library/7zdw9dt0(v=vs.90).aspx)
[CS0057] (https://msdn.microsoft.com/en-us/library/1he6k2zs(v=vs.90).aspx)
[CS0058] (https://msdn.microsoft.com/en-us/library/9c065z8s(v=vs.90).aspx)
[CS0059] (https://msdn.microsoft.com/en-us/library/20f51zz7(v=vs.90).aspx)
[CS0060] (https://msdn.microsoft.com/en-us/library/5732hs83(v=vs.90).aspx)
[CS0061] (https://msdn.microsoft.com/en-us/library/7f82xxf1(v=vs.90).aspx)

Weird thing, two sites with different info, both on MSDN.
Ok, then, from CS0050 to CS0061, we fix them all. :)
Agreed?

I agree with that we should create code-fix for all :+1:
I just think it will be better if we can group and split it into few issues. Thanks to that it will be much easier to review code and to do it. We can use the same code fix but for every error there will be different code to find type that caused the error.
I propose to make this issue code-fix for CS0050, CS0051.
And create new issue for :

CS0052 and CS0053
CS0055 and CS0054
CS0057 and CS0056
CS0059 and CS0058
CS0060 and CS0061

@giggio, what do you think?

@baks
I just took a look at each of them. I don't think we should group them randomly. If we subscribe to CS0050, we will be expecting some type of expression with a Type on a return statement. If we subscribe to CS0051 we will be expecting a parameter. This can be hard or easy to fix, depends on exactly where Microsoft is raising the diagnostic.
On CS0050 they do it on the method identifier, so you would have to navigate to the parent, find the method declaration, and then find they type.
On CS0051 they also raise it on the method identifier, but you would have to navigate to the parameters, and it does not explicitly tell you which of the parameter is the problem.
Either way, at the end, we have a Type Symbol. And a simple method that would make the type public, internal, or public internal, fixing all the diagnostics. We could do that simply by adding this method to some of the extension helper classes, and creating a ChangeAccessibility method.
I think it makes sense to have one code fix for each of these diagnostics and not group anything. We can create an umbrella issue to group all the issues, and we go on solving each. We can leave this issue as the first, so you can work on it, and decide the best approach, after it is done we create the others.
What do you think?

@giggio Sounds good for me :+1:
I first create code fix for CS0051 and I will try to prepare code for other errors. We'll se how it goes.
We also can fix CS0051 (and other errors) if we change method accessibility (or other code construct) to accessibility same as accessibility of parameter type that generated error.
So, this is also proper code fix for CS0051:
    public class Dependent
    {
        Dependent(DependendedUpon d)
        {
        }
    }
    class DependendedUpon
    {
    }

Ok, as soon as you have the first one open a PR so we discuss it.
The code fix would make the DependedUpon class public, as is on this issue's description.

@baks Ok, we can now open the other issues as the first PR (#374) is in.


Cannot load Java class mirah.impl.MirahParser?

Run on java 1.8 #mirah -e 'puts "Hello, World!"'! I thought what problem in version jvm, but when I tried use java 1.7.0_95 problem not solved.
In requirements write that need java 1.7 or higher.
Help please, how to solve this problem?

What version of mirah are you using?
It looks like gem misconfiguration.  I do not use mirah with jruby so could not help with it, but vanilla java is enough to start with mirah. Find mirah jar somewhere (build from sources, download from maven, extract from gem).
Example below assumes java and wget is in your path:
wget.exe http://central.maven.org/maven2/org/mirah/mirah/0.1.4/mirah-0.1.4.jar
java -jar mirah-0.1.4.jar --silent -e "puts 'Hello world!'" && java DashE

Note, that -e switch just compile DashE class. Check more command line options:
java -jar mirah-0.1.4.jar --help

Yes, *.jar file is work. But for me it is uncomfortably. I want to use mirah in command line. I cloned repo and write path in $PATH. Why is not work? Or how else I may use mirah whithout jar-file?


added C# examples

added examples for C#

I was unable to get my visual studio working so I didn't test if it prints the correct result yet. I will try and do that tomorrow.

Added C# linting and formatting to the CI and Makefile. Unfortunately, because of how .NET projects work, the only linting it can do with a loose collection of .cs files (instead of an actual .NET .csproj project or .sln solution) is whitespace cleanup (spaces, indents, newlines, etc.)

@jchen293 wanna take another look?

@nwithan8 I would expect people to copy/paste these. We'll do a second pass prior to implementation where our team blitzes each example for any stragglers like the items you mentioned. For now, I think this is a good start.


142/namespace changes

Changes namespace such that import ecoscope imports only base and io
Moves daynight_ratio out of Trajectory into analysis/astronomy (fixes circular import)
Moves datashader logic out of EcoMap into analysis/geospatial

closes #142, #149, #169

This needs updating now that #170 is merged

Overall LGTM @atmorling! Thanks so much for jumping on this so fast. 🙏
Left one question (which is genuine, not rhetorical) about __all__.
So how does this PR affect import time, did you measure that at all? Guessing it's quite fast now 😄
Also, kudos on manually going through all the notebooks, not easy!

Import time looks like this now :nerd_face:

Awesome!!
And remind me again what that profiler is? Such a useful chart!

Awesome!!
And remind me again what that profiler is? Such a useful chart!

https://github.com/nschloe/tuna


⚠ Secret Website 3 has degraded performance

In f1f4d9d, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2608 ms

Resolved: Secret Website 3 performance has improved in ac03091.


Adding updates to tests

Needed to update Pest syntax to upgrade to Pest V3 and get tests working on latest version of Laravel.


feat(stdlib): implement ContentNode

Change Summary
This implements ContentNode.
ContentNode is a bit unique in how it manages its fields. Like most components, it has a set of built-in fields that it supports. Unlike most other components, these fields are all "hidden" from the node by default. Then, when a field is accessed in any way, it becomes "visible". The unit tests show this behavior as well, but for clarity, here's an example:
node = createObject("RoSGNode", "ContentNode")
print node
' Notice that even though ContentNode has dozens of fields, none of them display.
'
' <Component: roSGNode:ContentNode> =
' {
'    change: <UNINITIALIZED>
'    focusable: false
'    focusedchild: invalid
'    id:
'}

' Getting a field makes it show up (in all caps)
print node.title
print node.count()  ' => 5 (the node now includes title)
print node
' ...
' TITLE: ""
' ...

' Setting a field makes it show up (in all caps)
node.description = "some description"
print node.count()  ' => 6 (the node now includes description)
print node
' ...
' TITLE: ""
' DESCRIPTION: "some description"
' ...

Because of this unique behavior, ContentNode had to override some functions from RoSGNode, along with a few other changes to support the overrides.

I'm pretty sure I know how to resolve those conflicts now that #438 , but would you mind taking a look when you get a second?

I'm pretty sure I know how to resolve those conflicts now that #438 , but would you mind taking a look when you get a second?

done!


cmd/akash cleanup: use context.Client()

Pull Request Test Coverage Report for Build 131

4 of 5 (80.0%) changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 66.197%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/akash/provider.go
0
1
0.0%

Totals

Change from base Build 129:
0.0%

Covered Lines:
3147

Relevant Lines:
4754

💛  - Coveralls

bit by #124


Server: Validate session ticket (in http server) - check if ticket (and user) exists and has access (thus site)

Done!


Add a description field for the Error model

Is your feature request related to a problem? Please describe.
An error's name should be concise and canonical, but sometimes it's hard to come up with a concise name that accurately describes what an error means. It would be great to express what an error means with a longer text field that can change between versions and not break any programs built against the API.
Describe the solution you'd like
Add an (optional) description field to the Error model. This can be used for the purpose of describing what an error means. It is not machine-checked and only there to help developers.
Describe alternatives you've considered
Add no new field and use code comments to express error descriptions. It seems preferable to include the descriptions in the payload of the error so clients wouldn't need to read source code.
Additional context
One version of the Mina implementation of the Rosetta spec included a misunderstanding of the details field that was co-opted to serve this purpose.

Thanks for posting this, @bkase! This is a great idea.