fix: min max age filter logic

Description
Replace audience age params with opposite to fix the search query
Issues
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

Codecov Report

Merging #53 (1d13e03) into main (4bdc153) will not change coverage.
The diff coverage is 80.00%.

@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage   81.38%   81.38%
=======================================
  Files         142      142
  Lines        2772     2772
  Branches      822      822
=======================================
  Hits         2256     2256
  Misses        514      514
  Partials        2        2

Impacted Files
Coverage Δ

src/domain/nextApi/graphql/generated/graphql.tsx
48.57% <ø> (ø)

src/domain/search/eventSearch/constants.tsx
100.00% <ø> (ø)

...search/eventSearch/filterSummary/FilterSummary.tsx
67.92% <66.66%> (ø)

src/domain/search/eventSearch/AdvancedSearch.tsx
87.77% <100.00%> (ø)

src/domain/search/eventSearch/utils.tsx
90.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

The PR-test-server throws an error from the proxy and cannot fetch the events:
{"errors":[{"message":"Unknown argument \"audienceMaxAgeGt\" on field \"Query.eventList\". Did you mean \"audienceMaxAgeLt\" or \"audienceMinAgeGt\"?","locations":[{"line":3,"column":5}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED","exception":{"stacktrace":["GraphQLError: Unknown argument \"audienceMaxAgeGt\" on field \"Query.eventList\". Did you mean \"audienceMaxAgeLt\" or \"audienceMinAgeGt\"?","    at Object.Argument (/app/node_modules/graphql/validation/rules/KnownArgumentNamesRule.js:46:29)","    at Object.enter (/app/node_modules/graphql/language/visitor.js:323:29)","    at Object.enter (/app/node_modules/graphql/utilities/TypeInfo.js:370:25)","    at visit (/app/node_modules/graphql/language/visitor.js:243:26)","    at Object.validate (/app/node_modules/graphql/validation/validate.js:69:24)","    at validate (/app/node_modules/apollo-server-core/dist/requestPipeline.js:233:34)","    at Object.<anonymous> (/app/node_modules/apollo-server-core/dist/requestPipeline.js:119:42)","    at Generator.next (<anonymous>)","    at fulfilled (/app/node_modules/apollo-server-core/dist/requestPipeline.js:5:58)","    at processTicksAndRejections (internal/process/task_queues.js:97:5)"]}}},{"message":"Unknown argument \"audienceMinAgeLt\" on field \"Query.eventList\". Did you mean \"audienceMinAgeGt\" or \"audienceMaxAgeLt\"?","locations":[{"line":4,"column":5}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED","exception":{"stacktrace":["GraphQLError: Unknown argument \"audienceMinAgeLt\" on field \"Query.eventList\". Did you mean \"audienceMinAgeGt\" or \"audienceMaxAgeLt\"?","    at Object.Argument (/app/node_modules/graphql/validation/rules/KnownArgumentNamesRule.js:46:29)","    at Object.enter (/app/node_modules/graphql/language/visitor.js:323:29)","    at Object.enter (/app/node_modules/graphql/utilities/TypeInfo.js:370:25)","    at visit (/app/node_modules/graphql/language/visitor.js:243:26)","    at Object.validate (/app/node_modules/graphql/validation/validate.js:69:24)","    at validate (/app/node_modules/apollo-server-core/dist/requestPipeline.js:233:34)","    at Object.<anonymous> (/app/node_modules/apollo-server-core/dist/requestPipeline.js:119:42)","    at Generator.next (<anonymous>)","    at fulfilled (/app/node_modules/apollo-server-core/dist/requestPipeline.js:5:58)","    at processTicksAndRejections (internal/process/task_queues.js:97:5)"]}}}]}

The PR-test-server throws an error from the proxy and cannot fetch the events:
{"errors":[{"message":"Unknown argument \"audienceMaxAgeGt\" on field \"Query.eventList\". Did you mean \"audienceMaxAgeLt\" or \"audienceMinAgeGt\"?","locations":[{"line":3,"column":5}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED","exception":{"stacktrace":["GraphQLError: Unknown argument \"audienceMaxAgeGt\" on field \"Query.eventList\". Did you mean \"audienceMaxAgeLt\" or \"audienceMinAgeGt\"?","    at Object.Argument (/app/node_modules/graphql/validation/rules/KnownArgumentNamesRule.js:46:29)","    at Object.enter (/app/node_modules/graphql/language/visitor.js:323:29)","    at Object.enter (/app/node_modules/graphql/utilities/TypeInfo.js:370:25)","    at visit (/app/node_modules/graphql/language/visitor.js:243:26)","    at Object.validate (/app/node_modules/graphql/validation/validate.js:69:24)","    at validate (/app/node_modules/apollo-server-core/dist/requestPipeline.js:233:34)","    at Object.<anonymous> (/app/node_modules/apollo-server-core/dist/requestPipeline.js:119:42)","    at Generator.next (<anonymous>)","    at fulfilled (/app/node_modules/apollo-server-core/dist/requestPipeline.js:5:58)","    at processTicksAndRejections (internal/process/task_queues.js:97:5)"]}}},{"message":"Unknown argument \"audienceMinAgeLt\" on field \"Query.eventList\". Did you mean \"audienceMinAgeGt\" or \"audienceMaxAgeLt\"?","locations":[{"line":4,"column":5}],"extensions":{"code":"GRAPHQL_VALIDATION_FAILED","exception":{"stacktrace":["GraphQLError: Unknown argument \"audienceMinAgeLt\" on field \"Query.eventList\". Did you mean \"audienceMinAgeGt\" or \"audienceMaxAgeLt\"?","    at Object.Argument (/app/node_modules/graphql/validation/rules/KnownArgumentNamesRule.js:46:29)","    at Object.enter (/app/node_modules/graphql/language/visitor.js:323:29)","    at Object.enter (/app/node_modules/graphql/utilities/TypeInfo.js:370:25)","    at visit (/app/node_modules/graphql/language/visitor.js:243:26)","    at Object.validate (/app/node_modules/graphql/validation/validate.js:69:24)","    at validate (/app/node_modules/apollo-server-core/dist/requestPipeline.js:233:34)","    at Object.<anonymous> (/app/node_modules/apollo-server-core/dist/requestPipeline.js:119:42)","    at Generator.next (<anonymous>)","    at fulfilled (/app/node_modules/apollo-server-core/dist/requestPipeline.js:5:58)","    at processTicksAndRejections (internal/process/task_queues.js:97:5)"]}}}]}

Relates to https://github.com/City-of-Helsinki/events-helsinki-api-proxy/pull/87


Do not follow redirects

From #109
Currently redirects are followed silently and there does not seem to be an option for disabling automatic redirect following. When using unirest for testing, this is problematic.
Is there a way to disable automatic redirect following that I'm missing or can it be added?

Added .followRedirect(boolean followRedirect)


make bacalhau resilient to flaky networks

there are lots of ways that job execution can fail if messages are dropped. make it more resilient!

related: https://github.com/filecoin-project/bacalhau/issues/487

also includes: https://github.com/filecoin-project/bacalhau/issues/320

Timeouts were introduced by https://github.com/filecoin-project/bacalhau/pull/1061, which will allow failing the job early when messages are dropped or when nodes disappear, instead of having the jobs stuck with no progress.
Note that the requester node will fail the job instead of retrying or asking for more bids if the selected compute nodes are no longer responsive. Retrying will require more significant changes that we an revisit in the future if there is a demand for it.


ARC upgrade error

The app errored during latest upgrade.
Logs
Initializing upgrade
[1016] Checking if upgrade is required.
[1016] Deleting new databases if exists.
[1016] Initializing upgrade
[1016] Getting data from old database
Failed to execute 'transaction' on 'IDBDatabase': One of the specified object stores was not found.
Error: Failed to execute 'transaction' on 'IDBDatabase': One of the specified object stores was not found.
    at Promise (chrome-extension://epgngalmiadbjnoompchcohonhidjanm/scripts/libs/app.importer.js:941:28)
    at arc.app.importer.getDatabase.then (chrome-extension://epgngalmiadbjnoompchcohonhidjanm/scripts/libs/app.importer.js:940:12)
[1016] Reading database takes longer than expected. You must have a lot of data in the datastore. Still working...
[1016] Reading database takes longer than expected. You must have a lot of data in the datastore. Still working...

Versions
App: <IP_ADDRESS>9-beta
Chrome: 55.0.2883.52
Platform: Linux (-)

Hi,
Thank you for issue report.
It's happening on fresh install. It's trying to access data from the database that do not exists. I'll fix it right away.


Exposing ObjectStoreInterface.autoIncrement / ObjectStoreInterface.keyPath

Can ObjectStoreInterface expose autoIncrement and keyPath? I use introspection on the database. They are currently gettable from store.ctx.objectStore but I have to circumvent typechecking to get them

That's not the case at the moment. I'm adding it to my todo list.
I think it'll be done in a day or two. I'll keep you updated.

Done ! Available in v4.0.5

awesome, works!


mem,main: add warning about missing locking

this is a proposal ...
the warning text can be discussed ...

didnt work:
FAILED: CMakeFiles/re-objs.dir/src/main/main.c.obj
[86](https://github.com/baresip/re/runs/6175251275?check_suite_focus=true#step:4:86)
C:\PROGRA~1\MICROS~2\2022\ENTERP~1\VC\Tools\MSVC\1431~1.311\bin\Hostx64\x64\cl.exe  /nologo -DARCH=\"AMD64\" -DHAVE_ATOMIC -DHAVE_INET6 -DHAVE_IO_H -DHAVE_SELECT -DOS=\"Windows\" -DUSE_DTLS -DUSE_OPENSSL -DUSE_OPENSSL_AES -DUSE_OPENSSL_DTLS -DUSE_OPENSSL_HMAC -DUSE_OPENSSL_SRTP -DUSE_TLS -DVERSION=\"2.2.2\" -DVER_MAJOR=2 -DVER_MINOR=2 -DVER_PATCH=2 -DWIN32 -D_CRT_SECURE_NO_WARNINGS -D_WIN32_WINNT=0x0600 -ID:\a\re\re\. -ID:\a\re\re\include -I"C:\Program Files\OpenSSL-Win64\include" /WX /MDd /Zi /Ob0 /Od /RTC1 /W3 -std:c11 /showIncludes /FoCMakeFiles\re-objs.dir\src\main\main.c.obj /FdCMakeFiles\re-objs.dir\ /FS -c D:\a\re\re\src\main\main.c
[87](https://github.com/baresip/re/runs/6175251275?check_suite_focus=true#step:4:87)
D:\a\re\re\src\main\main.c(203): fatal error C1021: invalid preprocessor command 'warning'
[88](https://github.com/baresip/re/runs/6175251275?check_suite_focus=true#step:4:88)
[49/199] Building C object CMakeFiles\re-objs.dir\src\main\init.c.obj
[89](https://github.com/baresip/re/runs/6175251275?check_suite_focus=true#step:4:89)
[50/199] Building C object CMakeFiles\re-objs.dir\src\main\method.c.obj

is there a portable preprocessor command for "warning" ?

do you think that error is too aggressive in mem.c ?
the problem is that for the debug build there is a list of allocated objects,
and if the application is multi-threaded, it needs to have a lock otherwise
it might crash.
this is now solved platforms with pthread and also for Windows.
any other platform is undefined.

yes, error is fine for mem.c I think.

if the person building re got the build flags wrong, the build process will not stop.
the alternative is that he will be using a buggy re.
please note that the main.c also needs updating regarding locking for WIN32,
adding INIT_ONCE and CRITICAL_SECTION etc...


dmonitoring: refactor to c++ class

Over the last year, we've hit some great milestones in cleaning up openpilot and making it more generic while bringing up the comma three, like fully moving the UI over to Qt. Now that the codebase is pretty clean, improving openpilot's quality looks more like speeding up camerad launch (#21925) than another minor camerad refactor (#23179). I updated CONTRIBUTING.md to reflect this. Our policy going forward will be to close PRs that have unclear goals, such as this one. Let's focus more on reliability and stability rather than C++-ifying the codebase. We'll be more active about opening issues that will help us push on the reliability and stability front.


[QUESTION] How do the kvvsink framerate and key-frame-fragmentation properties work?

I am using the kvssink gstreamer plugin to stream from rtsp cameras to kvs and then consume them in a mobile app (ios and android).
I note that kvssink has a framerate property that defaults to 25. How does this affect the stream. For example - if my camera is set to 10 fps, is kvssink then doubling up the frames to try and get to 25 fps? Do I need to manually set the fps property to the same value as my camera?
Or does this property not actually do anything to the framerate... From reading the source it seems that its only being used in buffer size calculations (but I may be wrong).
I notice that in all the provided gst-launch-1.0 example you don't set the property.
key-frame-fragmentation - the default is TRUE. When or why would you set this to false? I'm just trying to understand its purpose... I have issue #441 and am wondering how this setting interplays with that?

@js1972 you are correct, the KVS specific parameter frame rate is only used for buffer optimization. The actual frame rate should be configured upstream in the video source and possibly the encoder.


Specify required_ruby_version >= 1.9.3

PR 666 :metal:

Why 1.9.3? Shouldn't 1.9.2 work as well?

Why 1.9.3? Shouldn't 1.9.2 work as well?

Good question. I don't know. There were a lot of changes between 1.9.2 and 1.9.3 (http://svn.ruby-lang.org/repos/ruby/tags/v1_9_3_0/NEWS).


Add MabelTerm Release Manifest, Attempt 2

0x94
Microsoft Reviewers: Open in CodeFlow

Hi,
I'm Not 100% Sure What's Causing This Error.
Can Someone Look Into What's Going On Here?
Thanks In Advance


Docker installation failure

Running the provided commands from the README.md to install docker. Getting error relating to internationalization.
root@rush:~# curl -fsSL https://get.docker.com -o get-docker.sh
root@rush:~# sh get-docker.sh
# Executing docker install script, commit: 2f4ae48
+ sh -c apt-get update -qq >/dev/null
E: Failed to fetch http://deb.debian.org/debian/dists/stretch/main/i18n/Translation-en  BZ2_bzread: /var/lib/apt/lists/partial/deb.debian.org_debian_dists_stretch_main_i18n_Translation-en.bz2 Read error (-5: DATA_ERROR_MAGIC)
E: Some index files failed to download. They have been ignored, or old ones used instead.

root@rush:~# uname -a
Linux rush 4.4.0-134-generic #160-Ubuntu SMP Wed Aug 15 14:58:00 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

I tested in a different machine(arm machine), did get some errors with locale setting but it ended up  using fallback settings and managed to install docker successfully.
root@rush2:~# curl -fsSL https://get.docker.com/ | sh
# Executing docker install script, commit: 2f4ae48
+ sh -c apt-get update -qq >/dev/null
+ sh -c apt-get install -y -qq apt-transport-https ca-certificates curl >/dev/null
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = (unset),
	LC_ALL = (unset),
	LC_CTYPE = "en_NZ.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("en_US.UTF-8").
debconf: unable to initialize frontend: Dialog
debconf: (No usable dialog-like program is installed, so the dialog based frontend cannot be used. at /usr/share/perl5/Debconf/FrontEnd/Dialog.pm line 76, <> line 3.)
debconf: falling back to frontend: Readline
+ sh -c curl -fsSL "https://download.docker.com/linux/ubuntu/gpg" | apt-key add -qq - >/dev/null
+ sh -c echo "deb [arch=arm64] https://download.docker.com/linux/ubuntu xenial stable" > /etc/apt/sources.list.d/docker.list
+ sh -c apt-get update -qq >/dev/null
+ [ -n  ]
+ sh -c apt-get install -y -qq --no-install-recommends docker-ce >/dev/null
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = (unset),
	LC_ALL = (unset),
	LC_CTYPE = "en_NZ.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("en_US.UTF-8").
debconf: unable to initialize frontend: Dialog
debconf: (No usable dialog-like program is installed, so the dialog based frontend cannot be used. at /usr/share/perl5/Debconf/FrontEnd/Dialog.pm line 76, <> line 3.)
debconf: falling back to frontend: Readline
+ sh -c docker version
Client:
 Version:           18.09.7
 API version:       1.39
 Go version:        go1.10.8
 Git commit:        2d0083d
 Built:             Thu Jun 27 18:04:32 2019
 OS/Arch:           linux/arm64
 Experimental:      false

Server: Docker Engine - Community
 Engine:
  Version:          18.09.7
  API version:      1.39 (minimum version 1.12)
  Go version:       go1.10.8
  Git commit:       2d0083d
  Built:            Thu Jun 27 17:23:19 2019
  OS/Arch:          linux/arm64
  Experimental:     false
If you would like to use Docker as a non-root user, you should now consider
adding your user to the "docker" group with something like:

  sudo usermod -aG docker your-user

Remember that you will have to log out and back in for this to take effect!

WARNING: Adding a user to the "docker" group will grant the ability to run
         containers which can be used to obtain root privileges on the
         docker host.
         Refer to https://docs.docker.com/engine/security/security/#docker-daemon-attack-surface
         for more information.

root@rush2:~# uname -a
Linux rush2 4.15.0-46-generic #49~16.04.1-Ubuntu SMP Tue Feb 12 17:45:52 UTC 2019 aarch64 aarch64 aarch64 GNU/Linux

I am not aware of any preset required when using this script, if there is anything I am missing, please advise.

I tested this again, and the error seems to be gone!
closing this!


Feedback responsive

@mdazfar2
BUG: Feedback is not responsive #229
close #229
check this, i made feedback responsive

@dhruv8433 feedback and search is not responsive please create pr again on it before raise issue I'll assign you; do fast

@mdazfar2
I created another issue for that please assign me


Switching episode pauses audio

According to @aabounegm: "Playing another episode while one is already playing switches the current episode correctly, and leaves paused as false also correctly, but for some reason the audio element itself stops playing."
We need to fix this and add unit tests to cover this behavior.

Testing this is likely related to #120. I can take it too.


Empty PDF file on IOS

please provide the version of installed library and RN project. : RN 0.64.2, RNBlobUtil 0.13.7
a sample code snippet/repository is very helpful to spotting the problem.

When downloading a PDF file through the above method, it works on Android but on IOS, an empty files opens and nothing is to be found inside the iPhone filesystem

What is your downloadedFilePath ?
Did you try changing it? In iOS only DocumentDir and CacheDir will work.

What is your downloadedFilePath ?
Did you try changing it? In iOS only DocumentDir and CacheDir will work.

I am using this:
const downloadedFilePath = ReactNativeBlobUtil.fs.dirs.DownloadDir + '/' + fileName;

Ok it works with documentDir ! But is there a way to download a file without opening/displaying it like on Android ?

What do you mean without displaying?
You don't have to call "openDocument". It would be possible to simply save the file to disk without opening it.

When I dont use openDocument, I don't find any file on the iPhone (to have the file on the iPhone while using openDocument, I have to click on 'save to files')

The file is saved within the apps directory, since all apps on iOS live in their own sandbox.
There is currently no possibility to save it directly to downloads / photos / ...

Oh I see !
So then, how can the app access the saved documents (to delete them for exemple, or open them for the user)

Oh I see ! Thanks a lot

In my case,  The file path contains more than 255 characters


FLOPS = 0

I added --debug and --profile_flops to compute the FLOPS, but failed.
I implemented on CIFAR10.
The following is the output.
-------------------------- DeepSpeed Flops Profiler --------------------------
Profile Summary at step 1:
Notations:
data parallel size (dp_size), model parallel size(mp_size),
number of parameters (params), number of multiply-accumulate operations(MACs),
number of floating-point operations (flops), floating-point operations per second (FLOPS),
fwd latency (forward propagation latency), bwd latency (backward propagation latency),
step (weights update latency), iter latency (sum of fwd, bwd and step latency)
params per gpu:                                               1.09 M
params of model = params per GPU * mp_size:                   1.09 M
fwd MACs per GPU:                                             0 MACs
fwd flops per GPU:                                            0
fwd flops of model = fwd flops per GPU * mp_size:             0
fwd latency:                                                  57.42 s
fwd FLOPS per GPU = fwd flops per GPU / fwd latency:          0.0 FLOPS
----------------------------- Aggregated Profile per GPU -----------------------------
Top 1 modules in terms of params, MACs or fwd latency at different model depths:
depth 0:
    params      - {'DataParallel': '1.09 M'}
    MACs        - {'DataParallel': '0 MACs'}
    fwd latency - {'DataParallel': '57.42 s'}
depth 1:
    params      - {'ResNet': '1.09 M'}
    MACs        - {'ResNet': '0 MACs'}
    fwd latency - {'ResNet': '0'}
depth 2:
    params      - {'Sequential': '1.09 M'}
    MACs        - {'Conv2d': '0 MACs'}
    fwd latency - {'Conv2d': '0'}
depth 3:
    params      - {'BasicBlock': '1.09 M'}
    MACs        - {'BasicBlock': '0 MACs'}
    fwd latency - {'BasicBlock': '0'}
depth 4:
    params      - {'Conv2d': '1.07 M'}
    MACs        - {'Conv2d': '0 MACs'}
    fwd latency - {'Conv2d': '0'}
------------------------------------------------------------------------------

Hello @fanlyu,
Thanks for creating this issue.

Could you please provide the script you used to run this?
Are you running this on multiple GPUs? I encountered a similar issue when running on 2 GPUs. This could be a limitation of the FlopsProfiler tool.

Additionally, I have just added new commits to enable profiling of OCL methods that use deepcopy, such as LwF, RWalk, and MIR. Please pull the repository to obtain the latest code. I have also created a tutorial on how to use the FlopsProfiler tool to calculate FLOPs, which can be found here:
https://github.com/Yasir-Ghunaim/RealtimeOCL/blob/main/measuring_delay.md
Please keep in mind that the FlopsProfiler tool has some limitations, as outlined in the tutorial. Thus, it is strongly recommended that you manually calculate the forward/backward passes for each method and then use this tool for verification purposes.

Hello @fanlyu, Thanks for creating this issue.

Could you please provide the script you used to run this?
Are you running this on multiple GPUs? I encountered a similar issue when running on 2 GPUs. This could be a limitation of the FlopsProfiler tool.

Additionally, I have just added new commits to enable profiling of OCL methods that use deepcopy, such as LwF, RWalk, and MIR. Please pull the repository to obtain the latest code. I have also created a tutorial on how to use the FlopsProfiler tool to calculate FLOPs, which can be found here: https://github.com/Yasir-Ghunaim/RealtimeOCL/blob/main/measuring_delay.md
Please keep in mind that the FlopsProfiler tool has some limitations, as outlined in the tutorial. Thus, it is strongly recommended that you manually calculate the forward/backward passes for each method and then use this tool for verification purposes.

Yes, I comment the multi GPU, and it works. Thank you!
I will close this issue.


Update IsNullOrEmpty.cs

Simplified one line of code.

Hi! Thanks for the PR, but it doesn't improve anything... it's just a matter of coding style, and I prefer the current style, which I find more readable.


Submission search should find assignments in same course

Previously, searching for submissions for fa15 Ants was broken, because it takes the first assignment named "Ants", which happened to be a previous semester's Ants project.
Tested on staging

Oops, forgot to ./run_tests

Aside from the broken tests it looks good to me

@alvinwan ready to merge!
I need this for Ants regrades - I'll just use the staging version for now.

LGTM. Merging.


Adding localization files and targets. Using Multilingual App Toolkit…

… (MAT) for RESX file and custom tool VsctToXliff (sources added) for VSCT. XLF files are the loc files that vendors localized for us. We translated from RESX and VSCT to XLF, then vendors localize, then we tranlsate it back from localized XLF files to localized RESX and VSCT.

Yep, we will add vsixlangpack after I come back.

Thanks all!


[RRFC] Keeping and reverting to default values for properties

[x] I searched for an existing RRFC which might be relevant to my RRFC

Motivation
Currently the common pattern of setting default values for properties to do it in the constructor or class field instantiation. These are really more initial values that we lose after instantiation. Users may actually want some behavior where "unsetting" a property will revert to a default value.
It's possible to manually add this behavior by falling back to some value in render(), willUpdate(), or custom accessors[1] or even attribute converters if we wish to consider removing an attribute as "unsetting" as well.
There have also been calls for this behavior for or React wrapper[2] which we could add on at the wrapper level but would also be nice to have it for the underlying Lit element.
We do need to decide what "unsetting" means

Setting property to undefined
Setting property to null (probably not this?)
delete el[prop]; (unlikely to happen?)
Removing associated attribute
If we implement spread, encountering a missing property from a previously rendered spread.

How
Eventually when the only way to declare a property is with standard decorators, I believe we get access to the initial value set on the class field (with auto-accessor). Until then, we would need to add a property option for the default value.
Once we collect the default value, we can add it as a fallback value in an appropriate place like property setter or attribute converter.
References

[1] https://open-wc.org/blog/some-things-to-know-when-developing-with-lit/#default-values
[2] https://github.com/lit/lit/issues/4227

Having an alternate way of specifying a default value will also allow not reflecting initial property value to attribute.


timedatectl status different output on SUSE

The timedatectl output is not printing 'NTP enabled: yes'  .
Instead it is showing 'NTP synchronized: yes' on SLES 12 /15
Please include this.
Also the pyhton script is not returning anny rc code even if there are errors found.

~/hana-os-healthchecker-master # timedatectl status
Local time: Wed 2020-09-30 15:50:12 CEST
Universal time: Wed 2020-09-30 13:50:12 UTC
RTC time: Wed 2020-09-30 13:50:13
Time zone: Europe/Berlin (CEST, +0200)
Network time on: no
NTP synchronized: yes
RTC in local TZ: no

Which version are you using of this script because the code is there for both RH and SuSE on current version
Lines 227..240

I close pls reopen when you add the info. Thanks


components: configurable record access display

components: configurable record access display

Closes https://github.com/inveniosoftware/invenio-app-rdm/issues/2022.
:heart: Thank you for your contribution!
Description
Please describe briefly your pull request.
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[ ] I'm aware of the code of conduct.
[ ] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[ ] I've marked translation strings (for relevant code).
[ ] I've followed the CSS/JS and React guidelines (for relevant code).
[ ] I've followed the web accessibility guidelines (for relevant code).
[ ] I've followed the user interface guidelines (for relevant code).
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[ ] I've NOT included third-party code (copy/pasted source code or new dependencies).

Third-party code
If you've added third-party code (copy/pasted or new dependencies), please reach out to an architect.
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

note for release: major version, changes in the component interface


[One Driver Download]

Please do not submit this issue.

[x] I understand

Currently, Playwright downloads all supported browser drivers (Chromium, Firefox, and WebKit) by default, even if only one browser is needed. This behavior can be inconvenient, especially in scenarios where storage or bandwidth is limited, or when only one specific browser (e.g., Chromium) is required for testing.

You can skip browser downloads as per here and install them as a separate step. Would that work for you?

Closing per the response above, feel free to open a new issue if it doesn't work.


does not get the list or loads the file formats

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.07.17. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2016.07.17

Before submitting an issue make sure you have:

[ ] At least skimmed through README and most notably FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

youtube-dl --verbose -F https://www.youtube.com/watch?v=TucKUi0jmHk
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'-F', u'https://www.youtube.com/watch?v=TucKUi0jmHk']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.02.22
[debug] Python version 2.7.12 - Linux-4.4.0-31-generic-x86_64-with-LinuxMint-18-sarah
[debug] exe versions: ffmpeg 2.8.6-1ubuntu2, ffprobe 2.8.6-1ubuntu2
[debug] Proxy map: {}
[youtube] TucKUi0jmHk: Downloading webpage
[youtube] TucKUi0jmHk: Downloading video info webpage
[youtube] TucKUi0jmHk: Extracting video information
[youtube] {22} signature length 43.40, html5 player en_US-vflbQww0A
[youtube] TucKUi0jmHk: Downloading player https://s.ytimg.com/yts/jsbin/player-en_US-vflbQww0A/base.js
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 905, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 820, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 883, in _parse_sig_js
initial_function = jsi.extract_function(funcname)
File "/usr/lib/python2.7/dist-packages/youtube_dl/jsinterp.py", line 241, in extract_function
raise ExtractorError('Could not find JS function %r' % funcname)
ExtractorError: Could not find JS function u'tr'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by ExtractorError(u"Could not find JS function u'tr'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 905, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 820, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 883, in _parse_sig_js
initial_function = jsi.extract_function(funcname)
File "/usr/lib/python2.7/dist-packages/youtube_dl/jsinterp.py", line 241, in extract_function
raise ExtractorError('Could not find JS function %r' % funcname)
ExtractorError: Could not find JS function u'tr'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 666, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 316, in extract
return self._real_extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1407, in _real_extract
encrypted_sig, video_id, player_url, age_gate)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 915, in _decrypt_signature
'Signature extraction failed: ' + tb, cause=e)
ExtractorError: Signature extraction failed: Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 905, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 820, in _extract_signature_function
res = self._parse_sig_js(code)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 883, in _parse_sig_js
initial_function = jsi.extract_function(funcname)
File "/usr/lib/python2.7/dist-packages/youtube_dl/jsinterp.py", line 241, in extract_function
raise ExtractorError('Could not find JS function %r' % funcname)
ExtractorError: Could not find JS function u'tr'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
(caused by ExtractorError(u"Could not find JS function u'tr'; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.",)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

You are using an outdated version of youtube-dl. Read our FAQ if you have troubles updating.


chore(DeferObservable): add benchmarks & minor perf

This is the benchmark result on Node v5.9.0,
MacBook Pro (Retina, 15-inch, Mid 2014, 2.8 GHz Intel Core i7, 16 GB 1600 MHz DDR3).
Before (https://github.com/ReactiveX/rxjs/pull/1393/commits/7116d1d5415404a2b69f591c1bfe0d00ab5bf4dc)
                                         |                     RxJS 4.1.0 |              RxJS 5.0.0-beta.3 |          factor |      % improved
---------------------------------------------------------------------------------------------------------------------------------------------------
                       defer - immediate |             1,117,475 (±1.08%) |             1,979,565 (±1.09%) |           1.77x |           77.1%
                                   defer |               905,413 (±1.15%) |             1,187,348 (±1.42%) |           1.31x |           31.1%

After changeset https://github.com/ReactiveX/rxjs/pull/1393/commits/09cd8d0e5ca587a4f383d5fd6dd821977eb18fc0
                                         |                     RxJS 4.1.0 |              RxJS 5.0.0-beta.3 |          factor |      % improved
---------------------------------------------------------------------------------------------------------------------------------------------------
                       defer - immediate |             1,070,341 (±0.96%) |             2,003,761 (±1.21%) |           1.87x |           87.2%
                                   defer |               944,100 (±1.30%) |             1,234,359 (±1.25%) |           1.31x |           30.7%

After changeset https://github.com/ReactiveX/rxjs/pull/1393/commits/70c32478007faf577ec46314556c82c0a7d2c7c1
                                         |                     RxJS 4.1.0 |              RxJS 5.0.0-beta.3 |          factor |      % improved
---------------------------------------------------------------------------------------------------------------------------------------------------
                       defer - immediate |             1,058,502 (±1.15%) |             2,035,010 (±1.06%) |           1.92x |           92.3%
                                   defer |               914,642 (±1.26%) |             1,219,651 (±1.89%) |           1.33x |           33.3%

Can you try adding this as a member of DeferObservable?

okay. there is no regression after doing it.
                                         | RxJS 5.0.0-beta.2 |
----------------------------------------------------------------
                       defer - immediate | 2,440,423 (±0.54%) |
                                   defer | 1,471,945 (±0.44%) |

@saneyuki , appreciate for effort. Recent changes in defer to support promise as well includes changes for this PR of remove tryCatch (https://github.com/ReactiveX/RxJS/commit/0cb44e16b46660bad72468944a35872dd20871d8) - would you mind update this PR for benchmark test?

@kwonoj
I updated this pull request and the description.

@saneyuki, can you flatten the last two commits?

Adding "Blocked" until the last two commits are flattened, Then it's good to go. Attn @kwonoj

I'll keep eye on this. :eyes:

@kwonoj squashed!

I tried this before check in, seeing results as below
 defer |     659,651 (±1.48%) |     603,249 (±1.83%) |      0.91x |     -8.6%

I've been experienced sometimes number shows differently on my machine only and maybe this is same cases, but would like to confirm once again. @saneyuki , would you mind provide number again with latest changes?

@kwonoj
This is the newly benchmark result (the based master is ec4f2b76b81e924ba22adebbdad119e57039f621)

Environment

Node v5.10.1
OSX 10.11.4
MacBook Pro (Retina, 15-inch, Mid 2014)

CPU: 2.8 GHz Intel Core i7
RAM: 16 GB 1600 MHz DDR3

Result
Before
                                         |                     RxJS 4.1.0 |              RxJS 5.0.0-beta.5 |          factor |      % improved
---------------------------------------------------------------------------------------------------------------------------------------------------
                       defer - immediate |             1,010,478 (±1.09%) |             1,537,480 (±1.37%) |           1.52x |           52.2%
                                   defer |               874,775 (±1.34%) |               969,339 (±1.62%) |           1.11x |           10.8%

After
                                         |                     RxJS 4.1.0 |              RxJS 5.0.0-beta.5 |          factor |      % improved
---------------------------------------------------------------------------------------------------------------------------------------------------
                       defer - immediate |             1,049,825 (±0.74%) |             1,595,495 (±1.33%) |           1.52x |           52.0%
                                   defer |               843,224 (±1.18%) |               997,643 (±1.33%) |           1.18x |           18.3%

Merged with 566f46b, thanks @saneyuki  for confirmation. Seems it's my machine again.


Update configuration.md

Added the information about the limitation for the number of IP addresses that can be added in the staticwebapp.config.json

@craigshoemaker
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I am ok adding this to the configuration article. Note that this is already included in the quotas table https://learn.microsoft.com/en-us/azure/static-web-apps/quotas, so perhaps we should refer to that as the single source of truth @craigshoemaker

to be honest, the reason why I created the PR was because we had multiple customers not finding the limitation. (at least 4).

I am ok adding this to the configuration article. Note that this is already included in the quotas table https://learn.microsoft.com/en-us/azure/static-web-apps/quotas, so perhaps we should refer to that as the single source of truth @craigshoemaker

to be honest, the reason why I created the PR was because we had multiple customers not finding the limitation. (at least 4).

I definitely understand that, that is why I recommend pointing folks to the quotas table. That table does a good job of listing the limitations for each SKU

Agreed. I like how you're trying to surface the limitations better, @rapopescu16! Let's go with the link to the quotas page to make sure we don't have changes that get out of sync.

done, sorry totally missed it. :(

@craigshoemaker  - is this ready for merge? Thanks.

#sign-off


Fixing the server FAQ link

The server FAQ link was using WIKI link style which does not seems to work from the README file. This PR changes the link to a relative file link as suggested in https://stackoverflow.com/questions/23789542/github-link-to-wiki-page-from-readme.

Reopening the PR. It was closed when I deleted my fork.

Thank for the PR
:)


Remove the need for encoding attributes on pack ops

In #11603 we started copying the tensor encoding attributes onto the pack op created in MaterializeEncoding.  That was only needed for one pattern, LowerDispatchWorkgroupCountFromSetEncodingOp, which runs later (TileAndDistributeToWorkgroups) and needs to evaluate dynamic tile sizes because it needs to know outer sizes, which depend on those tile sizes (as a quotient).
What's the long-term fix? @MaheshRavishankar  (not urgent, when you come back), @hanhanW

@MaheshRavishankar Setting this as a P2 'needs scheduling' - please edit as needed.


changefeed resolved ts can not go on after restart all pd、tikv and tidb

What did you do?
1、create changefeed with simple protocol
sink_url_options: simple-test?protocol=simple&max-message-bytes=1024000&partition-num=3&replication-factor=3
config: "[integrity]\nintegrity-check-level = "correctness"\ncorruption-handle-level = "warn"\n[sink]\nsend-bootstrap-interval-in-sec = 120\nsend-bootstrap-in-msg-count = 10000\n[sink.kafka-config.codec-config]\nencoding-format = "avro"\n[sink.kafka-config.large-message-handle]\nlarge-message-handle-compression = "lz4"\nlarge-message-handle-option = "claim-check"\nclaim-check-storage-uri = "s3://tmp?access-key=minioadmin&secret-access-key=minioadmin&endpoint=http%3a%2f%2fminio-peer%3a9000""
2、run sysbench and  cdc boundary workload
3、kill all pd、tikv and tidb
What did you expect to see?
after fault recover, changefeed resolved ts can go on
What did you see instead?
changefeed resolved ts can not go on after restart all pd、tikv and tidb

Versions of the cluster
./cdc version
Release Version: v8.2.0-alpha
Git Commit Hash: 8d223402ae20ed3042444582eb3c6503d61103e9
Git Branch: heads/refs/tags/v8.2.0-alpha
UTC Build Time: 2024-05-08 11:36:18
Go Version: go version go1.21.6 linux/amd64
Failpoint Build: false
2024-05-09T08:21:41.925+0800
current status of DM cluster (execute query-status <task-name> in dmctl)
No response

/remove-area dm
/area ticdc
/severity major

/assign hongyunyan


Add import quick fix does not add import

From https://github.com/halohalospecial/atom-elmjutsu/issues/65

@pacbeckh, I already found the cause.  I will include the fix in the next release.  Thanks!


How to change the Caller frame in Entry object

type Entry struct {
	Logger *Logger

	// Contains all the fields set by the user.
	Data Fields

	// Time at which the log entry was created
	Time time.Time

	// Level the log entry was logged at: Trace, Debug, Info, Warn, Error, Fatal or Panic
	// This field will be set on entry firing and the value will be equal to the one in Logger struct field.
	Level Level

	// Calling method, with package name
	Caller *runtime.Frame

	// Message passed to Trace, Debug, Info, Warn, Error, Fatal or Panic
	Message string

	// When formatter is called in entry.log(), a Buffer may be set to entry
	Buffer *bytes.Buffer

	// Contains the context set by the user. Useful for hook processing etc.
	Context context.Context

	// err may contain a field formatting error
	err string
}

I implement a log interface by logrus, but the file name and line num is not as expected.
below is the code section

func (cl *CustomerLogger) ErrorWithContext(ctx context.Context, format string, args ...interface{}) {
	logrus.WithContext(ctx).Errorf(format, args...)
}

how to change the Caller frame in the entry object?

I believe this issue can be solved by https://github.com/sirupsen/logrus/pull/1215

I believe this issue can be solved by https://github.com/sirupsen/logrus/pull/1215

There is no method to currently change the current caller. It's computed by taking the first package outside of the logrus world. So it will always report the wrapper if any is used.
I will have a look at the mentioned PR to see if it's still relevant.


Opening the dropdown should scroll to the selected item.

Currently the drop down opens at the first item. It would be better if it scrolled to ensure the current selection is visible.
A second issue exists in that moving the mouse over the dropdown to scroll clobbers the original active item so you can no longer tell what the selected item was (when using the selectize theme).
It would probably be better if the hover state was separate from the active state as mentioned in #282.

It should indeed scroll to the selected item. +1

For anyone in a pinch, the "_ensureHighlightVisible()" function is called whenever the user strikes a "vertical" key (e.g. up/down arrow).  While I haven't run the tests yet, I've seen success executing the same logic inside the "ctrl.activate()" function's $timeout where focus is also set.  Again, this is merely a bandaid.

Will be solved with PR https://github.com/angular-ui/ui-select/pull/1208
https://github.com/angular-ui/ui-select/commit/d2ce54c0cc41fbfcdf2badb5ed45b45246d12026

When reset-search-input is set to false in ui-select directive, the dropdown does not highlight/select nor scroll to the previously selected item.

same as @DerekDomino

The PR #1208 has broken multiple selection with close-on-select=false, if the user use the mouse to select an item, then it goes back to the highlighted element.
For instance, the user wants to select X, Y and Z. He selects X with the mouse and the dropdown goes back to A.

This does not work on Angular 1.5.7. I dont know if that is a problem. I am trying to debug the code, I dont understand at this line. Can you explain to me ? It always goes into 'if' condition with animate = true, so the else statement never triggered and _ensureHighlightVisible () never be called.
`if (ctrl.$animate && ctrl.$animate.on && ctrl.$animate.enabled(container[0])) {
var animateHandler = function (elem, phase) {
if (phase === 'start' && ctrl.items.length === 0) {
// Only focus input after the animation has finished
ctrl.$animate.off('removeClass', searchInput[0], animateHandler);
$timeout(function () {
ctrl.focusSearchInput(initSearchValue);
});
} else if (phase === 'close') {
// Only focus input after the animation has finished
ctrl.$animate.off('enter', container[0], animateHandler);
$timeout(function () {
ctrl.focusSearchInput(initSearchValue);
});
}
};
                  if (ctrl.items.length > 0) {
                      ctrl.$animate.on('enter', container[0], animateHandler);

                  } else {
                      ctrl.$animate.on('removeClass', searchInput[0], animateHandler);
                  }
              } else {
                  $timeout(function () {
                      ctrl.focusSearchInput(initSearchValue);
                      if (!ctrl.tagging.isActivated && ctrl.items.length > 1) {
                          _ensureHighlightVisible();
                      }
                  });
              }`

+1
Same as @sonphnt
You can try this:
`if (ctrl.$animate && ctrl.$animate.on && ctrl.$animate.enabled(container[0])) {
var animateHandler = function (elem, phase) {
if (phase === 'start' && ctrl.items.length === 0) {
// Only focus input after the animation has finished
ctrl.$animate.off('removeClass', searchInput[0], animateHandler);
$timeout(function () {
ctrl.focusSearchInput(initSearchValue);
});
} else if (phase === 'close') {
// Only focus input after the animation has finished
ctrl.$animate.off('enter', container[0], animateHandler);
$timeout(function () {
ctrl.focusSearchInput(initSearchValue);
});
}
};
              if (ctrl.items.length > 0) {
                  ctrl.$animate.on('enter', container[0], animateHandler);
                 $timeout(function () { _ensureHighlightVisible();});
              } else {
                  ctrl.$animate.on('removeClass', searchInput[0], animateHandler);
              }
          } else {
              $timeout(function () {
                  ctrl.focusSearchInput(initSearchValue);
                  if (!ctrl.tagging.isActivated && ctrl.items.length > 1) {
                      _ensureHighlightVisible();
                  }
              });
          }`

Hello,
I've tracked down 2 cases that prevent the dropdown to scroll to to the selected item on open
Here is a plunk that reproduces the issue: https://plnkr.co/edit/n9jwLCyUv87Ba3vPgOvu
1. In combination with ngAnimate

Add ngAnimate as module dependency
Create an ui-select with ng-model that has initial value
Click to toggle the dropdown open
Not scrolled to active item

2. When configured with uiSelectConfig.resetSearchInput = false

Configure ui-select with uiSelectConfig.resetSearchInput = false  (or define on component)
Create an ui-select with ng-model that has initial value
Click to toggle the dropdown open
Not scrolled to active item

Did anyone find a solution ? I have the same problem with the same angular and ui select versions as you

You can use the fork I made for the pull request until they release a new version:
In package.json replace:
"ui-select": "^0.19.8"
with:
"ui-select": "git@github.com:kidroca/ui-select.git#v0.19.8-fork"
The run an npm-install
Fork: https://github.com/kidroca/ui-select/tree/v0.19.8-fork

@kidroca thank you, the fork works perfectly fine. Any word from the maintainers as to when we can expect this merged/solved on master?

Glad you like it. We almost merged the changes last week, maybe track the pull request topic to know for sure: #2034


Request: Support for running multiple chapters

I made a scuffed mock up here , but I would love to see this be fledged out. What if like to see are:

Support for other games/categories
Input validation
Check if the category already exists before trying to create it again

I also forgot to add the offsets oops


topology

Original issue: https://charm.cs.illinois.edu/redmine/issues/604

No body.

Original date: 2015-02-12 20:58:14

Test runs fine and tests distinct functionality (topo manager). I've added it to the Makefile so that it's now part of our test suite.
[[http://charm.cs.illinois.edu/gerrit/#/c/520/]]


use underscored columns rather than camelcase?

When using PostgreSQL editors like pgadmin iii to write and/or edit queries, it can be a pain if the db does not use lowercase characters.
Note the double-quotes around the Table Name, the createdAt, updatedAt, and the constraints.
CREATE TABLE "Roles"
(
  id serial NOT NULL,
  role character varying(255),
  "createdAt" timestamp with time zone NOT NULL,
  "updatedAt" timestamp with time zone NOT NULL,
  CONSTRAINT "Roles_pkey" PRIMARY KEY (id),
  CONSTRAINT "Roles_role_key" UNIQUE (role)
)
...

Sequelize offers a way to define a model or the entire db so that underscores are used, thereby avoiding the double-quotes issue.
http://stackoverflow.com/questions/29828676/change-default-column-name-sequelize
Any opinion on forcing columns to be underscored?

Sounds good to me :)
Are there any drawbacks to this approach that you know of?

Possibly. I have to get more familiar with Sequelize, and in the meantime I wonder what happens in this scenario:
Would the foreign key reference to userId here have to refer to the actual column name in the db (potentially user_id) or its representation in the Sequelize model for a User?
  var Article = sequelize.define('Article', {
    title: {
      type: DataTypes.STRING,
      allowNull: false,
      defaultValue: null
    },
    content: DataTypes.TEXT
  }, {
    classMethods: {
      associate: function(models) {
        Article.belongsTo(models.User, {
          foreignKey: 'userId',
          foreignKeyConstraint: true
        });
      }
    }
  });

Hmmm... According to the docs, "foreignKey" would refer to the actual column name:
http://docs.sequelizejs.com/en/latest/docs/associations/

Hmmm... looking at this again. In PSQL, unquoted identifiers automatically fold to lowercase - hence the need for quotes on camelCase identifiers.
I kinda prefer snake_case/underscores myself but I think most JS applications go with camelCase as, idiomatically, it makes more sense not having to switch between contexts. As well, it would be a PITA to revamp the whole of the application for the sake of preference.
My vote is to stay with the default camelCase and leave the rest to user preference.
Thoughts?

Closing as this seems to be a "non-issue".
Please feel free to "reopen" if you would like to discuss the topic further.


fix: Validate and sanitize certificate for logpipeline (#675)

Description
Changes proposed in this pull request (what was done and why):

Cherry-picked from: https://github.com/kyma-project/telemetry-manager/pull/675

Changes refer to particular issues, PRs or documents:

Traceability

[ ] The PR is linked to a GitHub issue.
[ ] New features have a milestone set.
[ ] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[ ] The corresponding GitHub issue has a respective area and kind label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] Adjusted the documentation if the change is user-facing.
[ ] The feature is unit-tested
[ ] The feature is e2e-tested

closing this in favour of: https://github.com/kyma-project/telemetry-manager/pull/686


Support using pathlib.Path with markupwriters

This fixes a simple omission for supporting Path objects

Looks ok. Could you nevertheless tell the use case for this change?

e.g. when doing post-processing of execution results:
from pathlib import Path

from robot.api import ExecutionResult
from robot.result.visitor import ResultVisitor

my_old_result = Path("location/to/old/report.xml")
my_postprocessed_result = Path("location/to/new/result.xml")

result = ExecutionResult(my_old_result) # Already works
result.visit(SomeResultVisitorClass())
result.save(my_postprocessed_result) # Internally creates and uses an xml-markupwriter, currently fails because a Path is not a string (workaround is to wrap the path in str)

Sounds good and safe enough to be included in 4.0.1 as well. Could you please submit an issue aa well so that this change gets listed in the release notes? Creating a unit test that save() works with Path would be good too to make sure we don't accidentally break this later. I believe there are tests already for creating ExecutionResult using Path.

For the unit tests, I notice that none of the tests appear to actually print the output (presumably to avoid cleanup). Since the goal is simply to test taht the save function doesn't throw, would it be acceptable to target os.devnull?

Looks good to me. The new test could have possibly been included into the existing suite using Path but then at least the class name needs to be changed. I could to that myself after merging. Thanks for the PR!


@PathParam Annotation not interpreted correctly

Hello,
i found a little bug in the @PathParam Annotation.
If i write @PathParm("testParam"), the generated swagger.json is not valid and cannot read by swagger ui (i get an error: TypeError: undefined is not an object (evaluating 'property.$ref') in model.js:176
But if i write @PathParam(value="testParam") everything is correct.
IMHO both variants are correct.
Thanks
Marcel

@marcelalburg - considering that Java-wise, those two are equivalent, it's unlikely that's the cause of the issue you're experiencing. Do you see any difference in the swagger.json produced by both variants?

@marcelalburg?

Sorry, but i cannot reproduce it in my project anymore. Maybe it's not the problem of the PathParam.
Marcel

Hello,
i found something and have more information about that.
If i run my project into eclipse, it works, but if i run it in the console with "java -jar" not.
i compare the genereaded swagger.json and see one difference:
working version:
      "Category":{
         "type":"object",
         "properties":{
            "id":{
               "type":"integer",
               "format":"int32"
            },
            "name":{
               "type":"string"
            },
            "parentId":{
               "type":"integer",
               "format":"int32"
            },
            "urlPath":{
               "type":"object",
               "additionalProperties":{
                  "type":"string"
               }
            },
            "sortOrder":{
               "type":"integer",
               "format":"int32"
            },
            "categories":{
               "type":"array",
               "items":{
                  "$ref":"#/definitions/Category"
               }
            }
         }
      },

not working version:
"Category":{
         "type":"object",
         "properties":{
            "id":{
               "type":"integer",
               "format":"int32"
            },
            "name":{
               "type":"string"
            },
            "parentId":{
               "type":"integer",
               "format":"int32"
            },
            "urlPath":{
               "type":"object",
               "additionalProperties":{
                  "type":"string"
               }
            },
            "sortOrder":{
               "type":"integer",
               "format":"int32"
            },
            "categories":{
               "type":"array"
            }
         }
      },

in the swagger ui, i get the same error like before..
Thanks Marcel

That's an entirely different issue than the original one.
The difference is output is due to a difference in versions. I don't know how you manage your dependencies, but something is messed up there if you get different results in different environments. Make sure you use 15.2.

Hello,
your're right. With Version 1.5.0 it's not working, but with 1.5.2 it works.
Thanks
Marcel

Same issue got me with 1.5.0, and upgrading worked.  Thank you!


Root cause pipes issue on SQLClient

This has been an issue for years:

System.Data.SqlClient.SqlException : A connection was successfully established with the server, but then an error occurred during the login process. (provider: Named Pipes Provider, error: 0 - No process is on the other end of the pipe.)
---- System.ComponentModel.Win32Exception : No process is on the other end of the pipe

After 2.0 is locked down, let's sync with the SqlClient team to try to get to the root cause and see if it can be fixed.

@divega Can you find your simple repro for this?

@divega @ajcvickers @glennc - Should we do something about Triage to Ready back & forth transitions?

@glennc this actually doesn't look right. Is it a bug in some script? Otherwise, can you just remove EF repo from the projects?

I think @glennc should definitely do something about this! 😺

This continues to be an issue as seen here.

Still seeing this occasionally, though less frequently.

Assigned to @smitpatel to add some mitigations to the database creation tests, which is where this commonly shows up. This code doesn't change much, so one option is to not run them on the normal C.I. runs. Another is to add try-catch as appropriate to stop the tests from failing when this issue pops up.

While searching for error online, I found some suggestions which said, it could happen if tcp connection has higher priority than np (named pipes). And putting np first could resolve the issue. I checked efci agent to see configuration (where this error does not happen) and found that it does not have tcp enabled. I looked at the history of the build config on aspnetci to see agents where it failed last. I updated on agent 107 (where it failed only once in last 2 month or so) and tried to update aforementioned settings. In theory it should fix it.
Closing this issue as resolved for now.
If it fails again on different agent then we can apply same settings to other agent. If it fails on agent after above settings then we can look into disabling/skipping tests.

This is still happening. A list of these failures can be found here.


ants mass suicide

Hi!
My ants keep running into water and dying. Is that OK?)
Kind Regards,

Hey again! Yeah, that's expected due to how the code works, but obviously not ideal. This would require however writing a complex AI for ants, instead of the current "oooga booga let's just move to a random tile around us" one, which would slow down the game whenever high amounts of ants are present, so I don't think I'll do anything about it in the near future. Still, thanks!


Add labels to tutorial code chunks

This addresses the labels-point in https://github.com/liesel-devs/liesel-internal/issues/141.
I also took the opportunity to remove some leftover compiled tutorial files and the makefile. Since now the CI is rendering the tutorials, we do not need the makefile anymore.

@hriebl would you do the review?

Sure! Is it a good idea to have spaces in the labels? In tutorial three and four I used dashes, because I assumed that was better for the file names...


Single choice and multichoice support needed !

we need Single / multi Choice RecyclerView support for this adapter !

What is the Single choice and multichoice ？

Beyond the scope of the adapter, so sorry


secret: add secret provider interface and use it for TlsCertificates

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>Description:
Add secret provider interface which is part of DynamicTlsCertificateProvider interface proposed in #3700. Make static and inline TlsCertificate utilize it.
Risk Level: Low
Testing: unit test, integration test
Docs Changes: N/A
Release Notes: N/A

cc @JimmyCYJ @qiwzhang

@ggreenway and/or @mattklein123, PTAL. For a non-Google review.

Yes I will look. Please assign me as a reviewer on all SDS PRs.

I'm probably missing something, but what's the reason for having TlsCertificateConfigProvider, which is effectively a wrapper around TlsCertificateConfig?

It is to extract the callbacks for dynamic secret, which will swap the holding TlsCertificateConfigPtr, like this: https://github.com/envoyproxy/envoy/pull/3700/files#diff-93a68bfee9eaf7ed295925ab50201db8R16


Layout error for simple XAML with ProgressBar

Describe the bug
Possibly related to #1441 and #477. A simple window with the following XAML will crash instantly on startup with a layout cycle error.
<Window
    x:Class="LayoutCrash.MainWindow"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
    xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
    mc:Ignorable="d">

  <StackPanel Orientation="Horizontal">
    <ProgressBar MinWidth="200" Value="100" BorderThickness="3"/>
  </StackPanel>
</Window>

The crash requires ProgressBar.Value to be large (98, 99 or 100 in my case) and BorderThickness to be greater than 0. It appears there is some layout fighting between the StackPanel and the ProgressBar with the BorderThickness.
My DPI is set to 125%, but a coworker was able to reproduce the issue with DPI set to 100%. I'm running this in Unpackaged mode.
Steps to reproduce the bug
Start a fresh WinUI 3 project, and paste in the XAML above. Start the project and it should crash instantly with the exception in the screenshot above.
Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.2.1: 1.2.221116.1
Windows version
Windows 11 (21H2): Build 22000
Additional context
No response

I have a XAML parsing error in WinUI3 project each time when ProgressBar is on the page, even no need for set a value or thickness. Had to remove them all. The same error is for ProgressRing.

A fix for this issue has just been completed for WinAppSDK 1.5. ProgressBar didn't handle if BorderThickness or Padding was set on it. Moving these properties up to a parent element would be a workaround on older builds.


Paste format when pasting from Excel file

When pasting values from an Excel file, the Grid is using the HTML format, instead of Plain Text. This is troublesome as Excel can have many visual formats and the Grid is not using the raw value.
Take as an example a value "12345.67" in Excel. Depending on the cell width, Excel will show the value trimmed, rounded or even replaced with "####".

These are the values used if HTML format is first, which are not correct:

However, Plain Text contains the correct raw values:

Proposal:
Change the evaluation to consider clipboard format text/plain first and text/html second.
Second proposal:
Control the order of evaluation with a prop. Some prop called clipboardFormatPriority, with an array of values text/html,text/plain, text.

@nick-keller I have created the PR #368 for your consideration 👍

@nick-keller that is a fair question. Agree. I will have a deeper look at this. A very high level search lead me to Sheetjs. Their approach is to use all of the formats, which I think is interesting:
https://docs.sheetjs.com/docs/demos/local/clipboard/#browser-reading-paste


Class:Institution

Parent
ObjectGroup

Definition
The institution that maintains the collection

Repeatable
Yes

Relationships
CollectionDescription (1 to many) <-> Institution (0 to 1)

Potential standards/vocabularies/ontologies to adopt
"AIISO (http://vocab.org/aiiso/), FOAF (http://xmlns.com/foaf/0.1/)"

Notes

I'm not sure we actually need this as a separate class because we have OrganisationalUnit.
This would be a record with OrganisationUnitType = Institution. Especially for collections that sit within a multi-institutional hierarchy. For example NMNH is an stitution that sits with Smithsonian which is also an institution. I vote to get rid of this!

Closed, as institution is covered by organisationalUnit


Reports

May I be taken off of this list
Dave Singh
University of Michigan '18 | School of Information
Pi Kappa Phi Fraternity | Vice President
(585) 732-6672

On Jun 2, 2016, at 2:31 PM, Conner Brennick<EMAIL_ADDRESS>wrote:
Merged #29.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

You have to remove yourself from the list. You are following the repository.


Implementation of bionic tiny stack

Summary

Adds initial stack definition and tests for the Bionic Tiny Stack. This stack should be functionally identical to what's published in the paketo-buildpacks/tiny-stack-release repo.
Differences from Legacy Bionic Tiny Stack Implementation
Stack image filesystem diffs generated with container-diff
Build
container-diff diff paketobuildpacks/build:1.3.55-tiny-cnb daemon://updated-bionic-build:latest --type file | tee pbcopy

-----File-----

These entries have been added to paketobuildpacks/build:1.3.55-tiny-cnb: None

These entries have been deleted from paketobuildpacks/build:1.3.55-tiny-cnb:
FILE                          SIZE
/etc/group-                   446B
/etc/gshadow-                 374B
/etc/passwd-                  926B
/etc/shadow-                  501B
/etc/subgid-                  0
/etc/subuid-                  0
/home/cnb/.bash_logout        220B
/home/cnb/.bashrc             3.7K
/home/cnb/.profile            807B

These entries have been changed between paketobuildpacks/build:1.3.55-tiny-cnb and updated-bionic-build:latest:
FILE                                 SIZE1         SIZE2
/var/log/lastlog                     285.4K        28.8K
/var/log/dpkg.log                    246.6K        246.6K
/var/log/tallylog                    62.6K         6.3K
/var/log/faillog                     31.3K         3.2K
/var/log/apt/term.log                25.9K         25.9K
/var/log/apt/history.log             12.6K         12.6K
/var/log/apt/eipp.log.xz             9.2K          9.2K
/var/cache/ldconfig/aux-cache        8.2K          8.2K
/var/log/alternatives.log            6.3K          6.3K
/var/lib/apt/extended_states         4K            4K
/etc/shadow                          526B          501B
/etc/gshadow                         382B          374B
/etc/apt/sources.list                225B          208B
/etc/default/locale                  92B           92B
/etc/subuid                          17B           0
/etc/subgid                          17B           0

There are a couple of differences that stem from jam create-stack not using useradd to create the cnb user:

the user does not have a default.bashrc, .profile, or .bash_logout files in its home directory. This means that aliases like ls -al ==> ll are not available by default in the new build images. (See StackOverflow for discussion of this useradd behaviour.)
/etc/subuid and /etc/subgid are empty because useradd populates these automatically

/etc/apt/sources.list is different because we no longer use the multiverse package repository component. There were no packages on the tiny image that required that source.

/etc/shadow and /etc/gshadow are encrypted versions of /etc/passwd and /etc/group, respectively. Since the unencrypted files have no diff, I'm not worried about the encrypted versions being slightly different.

/var/lib/apt/extended_states is modified by apt-get and apt-mark. The diff reflects the fact that libexpat1 is NOT explicitly installed in the updated version of the bionic tiny stack build image. It's auto-installed as a transitive dependency of git.

The files suffixed with a dash (e.g. /etc/group-) are simply backups of these important /etc files. It's ok that they're not included in the new stack.

the /etc/default/locale file routinely changes between stack builds

log and cache files routinely change between builds of stacks; nothing to see here

Run
container-diff diff paketobuildpacks/run:1.3.55-tiny-cnb daemon://updated-bionic-run:latest --type file

-----File-----

These entries have been added to paketobuildpacks/run:1.3.55-tiny-cnb: None

These entries have been deleted from paketobuildpacks/run:1.3.55-tiny-cnb:
FILE                        SIZE
/var/lib/dpkg/status        1B

These entries have been changed between paketobuildpacks/run:1.3.55-tiny-cnb and updated-bionic-run:latest:
FILE                                          SIZE1        SIZE2
/var/lib/dpkg/status.d/openssl                1.1K         1006B
/var/lib/dpkg/status.d/libc6                  1K           963B
/var/lib/dpkg/status.d/ca-certificates        990B         883B
/var/lib/dpkg/status.d/libssl1.1              972B         875B
/var/lib/dpkg/status.d/base-files             911B         813B
/var/lib/dpkg/status.d/tzdata                 858B         767B
/var/lib/dpkg/status.d/zlib1g                 715B         616B
/etc/os-release                               452B         509B
/var/lib/dpkg/status.d/netbase                445B         372B
/etc/passwd                                   193B         190B
/etc/group

/var/lib/dpkg/status was empty in the old bionic run image. It's been intentionally removed.
The status.d/* files are different because we write slightly different metadata into the files for each dependency. A notable difference is that we no longer include upstream source info (Source-Package, Source-Version, and Source-Upstream-Version fields)
the /etc/os-release file is different since the "support URL" and other human-readable information about the stack has changed. See my other comment on the PR.
some changes caused by not using useradd:

the change in the /etc/passwd field is the removal of the User ID Info (GECOS) field in the file. This is an optional, "comment" field.
the change in /etc/group is the removal of a trailing colon, with no effect on behaviour

TODO

[x] compare outputs of this repo to what we currently ship as bionic tiny stack; ensure package list is the same, images are "same enough"
[ ] create custom push-image.yml workflow that pushes to the appropriate repos/tags; add push-image.yml to .syncignore
[ ] don't hardcode lifecycle version

Checklist

[ ] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[ ] I have reviewed the styleguide for guidance on my code quality.
[ ] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@fg-j  Re: the differences in the stack images.
From your research, are any of these differences red flags to you in that they may change behavior for users? Or is more research needed to understand that? From your (very helpful) descriptions, the status.d/* differences in the run image are of interest to understand ramification, and additionally I’d also maybe want to check that the lack of a .bashrc, .profile, and/or .bash_logout files have no effect.

RE your TODO list: create custom push-image.yml workflow that pushes to the appropriate repos/tags; add push-image.yml to .syncignore
We'll need the same for base and full Bionic stacks too. I wonder if it would make more sense to extend the existing github-config stack push-buildpackage workflow to perform this special case for any bionic-* stacks?

From your research, are any of these differences red flags to you in that they may change behavior for users?
All in all, I don't see any red flags in the diff.

The default of .profile, .bashrc and .bash_logout all impact the behaviour of the bash shell in the build image, but only when the shell is running in interactive mode. This means a user might see a difference if they exec into a build/run container using bash, but it won't impact how bash scripts (including bash buildpacks) run in the build container. I don't think it's a big deal, since our stacks/app containers aren't intended to provide an interactive bash experience.
The change in the status.d files makes syft generate slightly different package metadata when it scans the run image. All the package versions and key information is the same, but syft notices the source version metadata when it's present. I don't think this is a huge deal.


HHH-15509 correctly support @NotFound @ManyToMany

error if a non-@ManyToMany collection is @NotFound
disable constraint generation for @NotFound @ManyToMany
allow lazy fetching for @NotFound @ManyToMany
rework a completely bogus test so it makes sense

This is a good any rather low-impact change and I'm going to merge it.


chore: add linting and formatting

This PR adds eslint, prettier, and lint-staged to the repo as well as linting and formatting scripts to more easily catch errors and format changed files
For reviewers, should we combine this PR with the changed files that will result from the lint and formatting scripts or should I separate the work? Ideally I'd want to avoid massive merge conflicts with any existing PRs or work in progress

This looks important but I have no idea how to answer Andrew's question! Tagging @jeffchew to review.


.TX texture cache implementation.

It would be great if LuxCore would be able to cache .tx textures! It could save lots of VRAM and speed up the render times a bit!
https://docs.arnoldrenderer.com/display/A5AFMUG/Textures

A CPU implementation is possible but writing the GPU support is extremely complex due the need to read from the GPU something on the file system (i.e. GPUs have no direct access and everything must be mediate by the CPU).

A CPU implementation is possible but writing the GPU support is extremely complex due the need to read from the GPU something on the file system (i.e. GPUs have no direct access and everything must be mediate by the CPU).

Good! After we create a bounty for Persistent data I think we can organize ourselves to help this one get implemented as well.


Create BYOnetwork.md

Initial file used to capture BYO resources that can be consumed by the IAC code.

Abandoning this pr. Information is provided for the user in the CONFIG-VARS.md file.


Make the registerShutdownHook optional

May be using a system properties or similar.
It is nice to turn on shutdown hook in managed environment, where there are explicit start/stop lifecycle events.
Thanks

Dear Edgar,
I will make the registerShutdownHook opional in the next release, planned for July.
Best regards,
Wolfgang


Add Disk 2018-09 to Python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5562
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5562/commits/4ed3f2a637ee144f5f26d6bf386dbb2ea1523dfe


weave mantain connection to stale apiserver

What you expected to happen?
weave restart when node restart
What happened?
if not all masters are up, some weave pods doesn't restart , because cannot connect to apiserver ( yeah 1/3 of apiserver is down, but 2/3 are up! so he should recover aswell! ) this leave whole node networking down causing cascading issues..
How to reproduce it?
see here down in description
Anything else we need to know?
bare metal servers, installed manually (everything else works, weavescope is ok, service account are ok no other errors in kubelets ( except those caused by weave networking missing when I trigger the problem )
Versions:

$ weave version 2.0.4
$ docker version 1.12.6
$ uname -a  4.4.0-93-generic #116-Ubuntu SMP Fri Aug 11 21:17:51 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
$ kubectl version v1.6.7+coreos.0

I'm trying to perform some HA tests with multiple master nodes,
The problem that I'm encountering is that when I shudown master components on master nodes (shutdown docker service and leave down), and I leave those down for some minutes, then when I re-start those masters( but not all, just 2/3 ( start docker service again )  the apiserver cluster come back online and all other components too, BUT some instances of weave not...
in some nodes I get  the weave pod in "CrashLoopBackOff" with only 1 log line such as :
# from kubectl describe pod...
  2m    1m      5       kubelet, v5s2.net      spec.containers{weave}  Normal  Pulling         pulling image "weaveworks/weave-kube:2.0.4"
  1m    1m      1       kubelet, v5s2.net      spec.containers{weave}  Normal  Created         Created container with docker id 9a57ad8973ab; Security:[seccomp=unconfined]
  2m    1m      5       kubelet, v5s2.net      spec.containers{weave}  Normal  Pulled          Successfully pulled image "weaveworks/weave-kube:2.0.4"
  1m    1m      1       kubelet, v5s2.net      spec.containers{weave}  Normal  Started         Started container with docker id 9a57ad8973ab
  2m    6s      17      kubelet, v5s2.net      spec.containers{weave}  Warning BackOff         Back-off restarting failed docker container
  1m    6s      7       kubelet, v5s2.net                              Warning FailedSync      Error syncing pod, skipping: failed to "StartContainer" for "weave" with CrashLoopBackOff: "Back-off 1m20s restarting failed container=weave pod=weave-net-0cbf3_kube-system(d81fd395-9f76-11e7-afe6-fa163e8ea49c)"

### from kubectl logs ..
2017/09/22 14:40:58 Could not get peers: Get https://<IP_ADDRESS>:443/api/v1/nodes: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused
Failed to get peers

#from  docker logs -f 509f13f67733
2017/09/22 14:46:06 Could not get peers: Get https://<IP_ADDRESS>:443/api/v1/nodes: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused
Failed to get peers

on the nodes where weave doesn't restart, if i curl to https://<IP_ADDRESS>:443/ I get the timeout reponse from the node which is down , on nodes where weave recover correctly instead I get the correct reply from one of the  apiserver which is up.
now, if I delete the service kubernetes in default namespace , it gets recreated with only the two apiserver endpoints  that are online , BUT the weave still stand in this CrashLoopBackOff state...
the situation is even worse..
even if I delete the pod from kubectl it doesn't work.. I'll perform more tests on monday probably,
when I then start the missing master that was still down, weave connect again,
I suspect that weave connect to one master and remain to this, or maybe watch for changes in default/kubernetes ?
I think that  because when one of the master nodes is down, the default/kubernetes endpoints still remain 3, including the node that is down.. so shouldn't be trusted too much ...
Name:                   kubernetes
Namespace:              default
Labels:                 component=apiserver
                        provider=kubernetes
Annotations:            <none>
Selector:               <none>
Type:                   ClusterIP
IP:                     <IP_ADDRESS>
Port:                   https   443/TCP
Endpoints:              --IP1--:6443,--IP2--:6443,--IP3--:6443
Session Affinity:       ClientIP
Events:                 <none>

a solution I can think about is providing the apiserver endpoint manually,
each kubernetes HA  system to be really HA should have a loadbalancer ( nginx/hap/... ) pointing to apiserver, why I cannot tell weave to use those endpoints  instead of default.kubernetes.svc.... ?
there is a way to specify different apiserver endpoint/service ?  I haven't found it in docs..
Thank you,
Francesco

Ok, probably I've found where is the problem.. which anyway I think should be fixed...
the problem is that weave configure iptables/nat to redirect randomly between apiservers ip,
BUT, it's a bit sticky about which ip to use,
this is the like that weave create
-A KUBE-SVC-NPX46M4PTMTKRN6Y -m comment --comment "default/kubernetes:https" -m recent --rcheck --seconds 10800 --reap --name KUBE-SEP-QGSNCIMMVIEBJLAU --mask <IP_ADDRESS> --rsource -j KUBE-SEP-QGSNCIMMVIEBJLAU

as you can see the timeout of the ip redirection is about  3 hours,
this means that in HA clusters, when a node is down, connections that was directed to the failed node/nodes will remain attached to it for 3 hours.. ( and maybe even more if weave doesn't try other apiserver destination when find a failed connection and still re-connecting to the failed one )  And because the service that configure iptables doesn't test the connection neither do the service that want to connect to the apiserver everything stays down
I think that the service that connect to weave-apiserver should not rely on this iptables redirection because at the moment the failower logic for services doesn't seems to work for this specific service ( ip are not updated fast enough ) instead would be better if weave use the ip on dns resolution and attempt to connect to them directly ( maybe refreshing at given interval ) so he can failower correctly ,
summary:
At this point when one/some of the master nodes crashes , some client nodes crashes too because weave stay connected to the ip of this master nodes(even if more than half of masters are up ), and this is VERY VERY VERY bad...
Thanks,
Francesco

Thanks for the update.  I agree it shouldn't be behaving as you describe.
It would be kube-proxy creating those iptables rules.
The could not get peers message comes from a program that starts, does its job and exits, so it cannot be maintaining a connection.

Another solution that I can propose, is leaving a way to configure the apiserver IP address on weave ( with an env variables/cli-args  ) and this ip will point to my nginx loadbalancer that I have already have to setup for the kube-proxy instances in the cluster ( HA kubernetes require a custom webserver with failower  torwards api ip address, this is specified also in kubernetes official guides for HA cluster setup ) and so you could use this same services that anyway is required, to point weave to..
this will be a solution with very low coding, very effective and optional only to ones that are willing to use it.. ( those with a HA kubernetes cluster )
Thanks,
Francesco

Weave Net connects to the Kubernetes api-server as recommended here; the address comes from environment variables configured within Kubernetes; you are free to set them as you wish.

Hi,
Maybe I've found what are you talking about.. and  experimenting for an elegant solution at the moment I have created a service to expose the nginx-ha which provide failover over various api-servers
apiVersion: v1
kind: Service
metadata:
  labels:
  name: nginxha
  namespace: kube-system
spec:
  selector:
    k8s-app: kube-nginx
  ports:
  - name: https
    port: 443
    protocol: TCP
    targetPort: 6444 ## port exposed by nginx-ha and which reverse-proxy to api-servers
  sessionAffinity: ClientIP
  type: ClusterIP
  clusterIP: <IP_ADDRESS>

then created a redirect to the just created services (wich exists in kube-system namespace ), this redirect will replace the kubernetes.default service which in all cluster is the default link for api-servers endpoints
apiVersion: v1
kind: Service
metadata:
  labels:
  name: kubernetes
  namespace: default
spec:
  type: ExternalName
  externalName: nginxha.kube-system.svc.cluster.local

then ,  AND I DON'T KNOW WHY weave started crashing with this error..
2017/09/25 12:26:41 Could not get peers: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined
Failed to get peers

which is strange because this is supposed to be provided by the kubernetes by default.. and the service is in place.. even restarting, and even recreating docker containers of weave and deleting the whole daemonset and recreating it didn't fixed.. at the end I've changed the default image of weave adding to both weave and weave-npc  those envs
              env:
                - name: KUBERNETES_SERVICE_HOST
                  value: "<IP_ADDRESS>"
                - name: KUBERNETES_SERVICE_PORT
                  value: "443"

now with this all in place the cluster can survive a master failure in a multi-master cluster,
probably setting a KUBERNETES_SERVICE_HOST directly to the ip of nginx-ha without overriding the default kubernetes.default service would be easier I'll try maybe in the future..
anyway I hope this thread could be helpful to someone else with same issue :)
Bye,
Francesco

Found a problem in the solution  here above, when a service with type type: ExternalName is used, the injected env variables become the first name of the redirected service name, not the name of the service which has the redirect,  this cause the problem with env variables in weave, aswell as missing kubernetes.default services in kubedns and other services that need it..
solution:
the nginx-ha service should be name "kubernetes" , so :
apiVersion: v1
kind: Service
metadata:
  labels:
  name: kubernetes
  namespace: kube-system
spec:
  selector:
    k8s-app: kube-nginx
  ports:
  - name: https
    port: 443
    protocol: TCP
    targetPort: 6444 ## port exposed by nginx-ha and which reverse-proxy to api-servers
  sessionAffinity: ClientIP
  type: ClusterIP
  clusterIP: <IP_ADDRESS>

and the second service in default namespace should then be :
apiVersion: v1
kind: Service
metadata:
  labels:
  name: kubernetes
  namespace: default
spec:
  type: ExternalName
  externalName: kubernetes.kube-system.svc.cluster.local

and everything works ;)

@fvigotti great to hear you got it working.  If you'd like to turn this into a blog post we'd be happy to host it or put a pointer on our website.

I would be happy to do that but I'm one man trying to do the job of 10 ;) I didn't have time to blogpost :( but you can reuse this discussion as you prefer if you want to do so :)
Have a nice day,
Francesco


using camera more than one time on android

i created a content page with the barcode reader.
on first use it works great.
if i leave the page and come back the camera does not activate and i have to close the app to get it back.
i am trying to see how to solve this but so far i am not able to as i do not know how the android camera control works.
several people have posted with comments but so far i have not been able to make this work.
can someone please help ?

just want to add a bit of info in case it helps anyone:
i have a content page that is navigated to why the flyout menu.
first time in the page works perfect.
but if i navigate back to the page the camera preview is closed and i cant scan.
i have tried some code posted here to try and get the camera back and it almost works.
i have just not been able to figure out the right set of commands to send to the device to get the preview back under control.
if the content page could be fully re-initialized or the control ....
the issue is somewhere in how to control the camera but i am at a loss on where to look.
if a switch from the app and back it works. if i close the app and open it that works..... but i need to stay in the app and just allows the user to open the page and go.

SOLVED!
credit to @RoyM33-T  for the clues to what i had to do!
the current published  view and or the Maui Content page / flyout navigation and the android camera do not work well together.
if you have flyout navigation then the Content page is not disposed of and the camera view is left in a state where if you leave the page and later come back it can not open the camera and preview a new image.
the answer with the current code is to use the code from: Camera is not released when using the CameraBarcodeReaderView in xaml #164
create a content page that is using flyout but then have that page invoke a second page that uses the camera view.
the flyout page needs to create its own local copy of the page with the camera view and get rid of that instance when it has the result back.
that keeps the MAUI runtime from reusing the old instance of the view and start over each time.
ideally the camera view control would detect the changes in the state and re-initialize the view.


Programming exercises: Fix an issue where the clone process does not work for new users

Checklist
General

[x] I implemented tests such that the functionality is completely covered and there's no need to test this on a server (see testing section for more info).
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data.
[x] I followed the coding and design guidelines.
[x] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Motivation and Context

When a user participates in an exercise for the first time after being created, the VCS access token might be missing, and therefore the resulting repo URL is not correct and cannot be used to clone the repository.
Description

The missing token might happen in case the vcsAccessToken feature in Artemis was not active/existing but already users are created. In that case the access token is only created when participating in a new exercise for the first time. In that case the token is not yet present in the client, since the data flow is like

log in, load user from server
start exercise participation
server creates repositories and notices token is missing, creates it
client wants to access URL for the new repositories, now notices that token should be there

token is missing since user object is from the time the user logged in and the token was created afterwards
to fix that: now explicitly fetch user object again to get the updated one that also contains the token

If for some reason the token could not be retrieved, a error message is displayed, detailing the next steps.
Steps for Testing
This issue can only be reproduced with old accounts that were present before the vcsAccessToken feature was activated. Because of this, the PR also doesn't change anything for the test servers.
What you can check:
On any test server check that the clone URL is still shown when participating as student in a programming exercise. Ideally one test for a testserver connected to GitLab/Jenkins and another one to either Bamboo or LocalVCS.
Review Progress

Performance Review

[x] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[x] Code Review 1
[ ] Code Review 2

Manual Test

[ ] Review 1
[ ] Review 2

Test Coverage

Screenshots

This is the text that is displayed when the token was initially missing before the click, but is now being fetched from the server.

If something went wrong during the communication with the server or the token doesn't exist, this text is displayed.

i tried to deploy on ts3_Legacy and ts11_Legacy and it didn't work. I couldn't reach the servers (We’ll be back soon!).

i tried to deploy on ts3_Legacy and ts11_Legacy and it didn't work. I couldn't reach the servers (We’ll be back soon!).

@Gusti2010 Thanks for the info!
That might have been due to a different database schema on develop that was not yet merged into this branch but already deployed by some other PR before. I just merged develop back into this branch to make sure it is up to date. Since this PR does not make changes to the database, it should hopefully work whenever the build in the CI has finished.

Tested on Gitlab and Bamboo server. Both times I can see the Clone URL -> Working as expected

Thank you for this contribution. However, after careful consideration, we decided that this PR cannot be merged, because the code is too specific and hard to test and maintain in the future. If this issue is problematic for you, we kindly ask you to develop a different solution that is more generic and requires less maintenance effort


[stacked_cli] error when deleting service

modification are not getting removed from app.dart file when using stacked cli to delete service

Interesting. Are you using windows or Mac?

I am on Ubuntu 22.04 using stacked_cli 1.1.6

in this image i create test service using cli and also trying to delete it using cli
creating and deleting works for view, just failing on service.

I see. Thanks for the details @JeetChhatrala we'll look at this in the week


Lazy message field access is dominated by <instance_members_initializer>

This chart is only really meaningful in context with time spent elsewhere, but I wanted to zoom in far enough to show that a significant portion of Foxglove Studio's 3D rendering loop is spent on the CPU accessing lazy message fields. This may be the same issue as https://githubmemory.com/repo/ChainSafe/ssz/issues/187

This is related to https://bugs.chromium.org/p/v8/issues/detail?id=12242&hotlist_id=12935&sort= rank -ID
Maybe a short-term fix is to not using instance members and do this old-school. Tho I am not sure how instance member initializer is impacted by the getters/setters.

Btw you probably meant to open this on https://github.com/foxglove/rosmsg-serialization

Fixed by https://github.com/foxglove/rosmsg-serialization/commit/99255811b45ff730bce4c2909bce1e14300fe872


Update siteConfig.json

I'm happy that you use my repo as reference. But please, remove the blog posts and the ackee data. I'm tracking your visits ;)

@rubenRP  Gracias :), yes your blog is good, I am removing your posts and infos step by step,because it is first time gastsby and type script sorry bother you :)


Doesn't serve a URL unless a transform is provided

asset.url and asset.getUrl() are empty unless a transform is provided as described in the docs.

Hi @owe,
Thank you for reporting this. I'll have a look over the weekend. Which version of the plugin are you on?

Latest version: 2.0.0


[SCB-1368] Scalability using Sharding from Akka Cluster

manage to distribute actors across several nodes in the cluster
SagaActor automatically recovers when JVM crashes

Thanks @coolbeevip for this wonderful contribution and I just request some minor changes.

The CI looks like failing with the FSM unit tests.

The CI looks like failing with the FSM unit tests.

I have no problem with local testing. I think it may be that the use of Akka's asynchronous causes the timeout setting to be unreasonable. I will try to modify the timeout period.

Just check the CI failure, it is caused by Report submission to Coveralls API failed.
All  the test are passed.

Coverage decreased (-2.5%) to 80.439% when pulling d521afd3699f623a5434aa2a9cffb4191ff9cf1d on coolbeevip:SCB-1368 into 6d5bef913b49d221cfc755e3e7eb3e9d7e6d38fe on apache:master.


Extend subscription rhp with ability to subscribe by registry entry id

This MR adds another type of request to the subscription rhp to allow for subscribing only via registry ID.
The result is the same as for regular subscriptions except for the initial values we send to the client after subscribing. These now contain the public key to allow the client to verify that the responses belong to the values they requested.

TestNewProgramLowCollateralBudget is failing -- any connection to this?

Looks like another NDF