Mongo fix

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #27 into master will increase coverage by 0.02%.
The diff coverage is 28.38%.

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   40.91%   40.93%   +0.02%
==========================================
  Files        2590     2589       -1
  Lines       78724    78674      -50
  Branches    10664    10653      -11
==========================================
- Hits        32207    32205       -2
+ Misses      43541    43490      -51
- Partials     2976     2979       +3

Impacted Files
Coverage Δ

...avercorp/pinpoint/plugin/mongo/MongoConstants.java
0% <ø> (ø)
:arrow_up:

...rofiler/context/provider/TraceContextProvider.java
100% <ø> (ø)
:arrow_up:

...pinpoint/profiler/context/DefaultTraceContext.java
56.75% <ø> (+0.91%)
:arrow_up:

...filer/context/module/ApplicationContextModule.java
100% <ø> (ø)
:arrow_up:

...avercorp/pinpoint/web/service/SpanServiceImpl.java
0% <0%> (ø)
:arrow_up:

...rcorp/pinpoint/plugin/mongo/MongoTypeProvider.java
0% <0%> (ø)
:arrow_up:

...m/navercorp/pinpoint/plugin/mongo/MongoConfig.java
0% <0%> (ø)
:arrow_up:

...p/pinpoint/test/PluginVerifierExternalAdaptor.java
0% <0%> (ø)
:arrow_up:

.../interceptor/MongoDriverConnectInterceptor3_7.java
0% <0%> (ø)
:arrow_up:

.../mongo/interceptor/MongoCUDSessionInterceptor.java
0% <0%> (ø)
:arrow_up:

... and 36 more

Continue to review full report at Codecov.

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


BUGFIX: Venafi issuer and clusterissuer checks were failing due to nilpointer exception

Introduced in https://github.com/cert-manager/cert-manager/pull/7036 (did not run the Venafi tests on the last commit).
Adds a missing nilpointer check that is causing panics for Venafi issuers.
Kind
/kind bug
Release Note
NONE

/test ?

/test pull-cert-manager-master-e2e-v1-30-issuers-venafi-cloud
/test pull-cert-manager-master-e2e-v1-30-issuers-venafi-tpp

/area venafi

/approve

/lgtm
/approve


Feature request: fuzzer program

Hi,
I was wondering if you could support a custom fuzzer program in your job config file in an upcoming release? The use case I am thinking of is doing A/B testing between afl and aflfast. I can wrap your support inside Orthrus.
Cheers,
Bhargava
Something like:
[
...
"fuzzer_prog" : "afl" // OR "aflfast"
..
]
that would be picked up by afl_path, like so:
if "fuzzer_prog" in conf_settings:
afl_path = shutil.which(conf_settings["fuzzer_prog"])
I suppose afl-utils already supports optional args to AFL via "afl_margs", so fine tuned fuzzing shouldn't be a problem.

Nice idea! Yeah, I think this should be doable.

Alright, implemented this on experimental.
@bshastry may I ask you to test this a bit?

Wow, thanks for a quick turnaround! Will test and report back! :-)

Hi @rc0r
Took me a while to clean up my codebase and test this feature, but I am happy to report that it works as expected. A minor glitch on my system (python dist bug) is that this line in afl-multicore (experimental):
try:
    import simplejson as json
except ImportError:
    import json

lead to simplejson being imported succesfully but not having any methods defined i.e., dir(simplejson) shows no methods (I have simplejson 3.8.2)...
This lead to the json.load aborting. Fixed this temporarily on my local installation by forcing import json (using native python json lib).

Thanks for your feedback @bshastry!
This simplejson bug is strange. I do have 3.8.2 as well and it is behaving normal.


Fix issues with zero sized VLAs

To address issue https://github.com/ElementsProject/simplicity/issues/25 we begin our removal of VLAs starting with all VLAs used in sizeof.
We also fix an issue with possible zero sized allocations for 'extra_var' within mallocTypeInference.

This looks right and tests out okay for me.

ACK from me too. A bit hard to follow the logic changes but I think the final result is followable (as much as allocation logic can be) on its own.


Consider update ILRepack to 4.0 framework

Hi,
What's your need for a 4.0 ILRepack version ?
Thx

Time is passing, 3.5 is not installed by default on Windows 8 anymore and it may be annoying to install.

I was pretty sure runtime 4 and 4.5 could run 3.5 assemblies w/o issue (e.g. I can use a 3.5 DLL in a 4.0 executable), are you saying it doesn't ?

Looks like no. Windows 8 requires .NET 3.5 installed to run ILRepack.exe

http://msdn.microsoft.com/en-us/library/jj152935(v=vs.110).aspx
What if you add the proper app.config to allow 4.0 ?

It should work, I think. Going to try that.


Unable to handle multiple stylings

The kit currently is unable to handle multiple stylings of the same text in a Rich Text field.
For example, if you put everything in bold, and one part of the text in italic on top of the already bold, it renders only the strong tag, the em tag is missing.
Along with this if you put text in bold, and I set a link on 2 different words on top of the bold, only the last link will be rendered to an anchor tag.

Hey @levimykel , this one has probably been solved in prismic-richtext ✌


Fraction-less floats are coerced into integers

I noticed when using dts that floats are mangled when outputting hcl. I found this:
https://github.com/martinohmann/hcl-rs/blob/796707ab4ebfe6b83996aef4cce8d58ae2d51ced/crates/hcl-primitives/src/number.rs#L53-L58
Would it be an option not to do that and instead preserve the type?

hcl spec doesn't say anything about number implementation
cty.Number seems to be used, which is effectively an f64

Looking at the coercion logic I start to think that maybe it's better to accept that in the end everything is kind-of f64 and accept that .0 may not be written.
https://github.com/martinohmann/hcl-rs/blob/796707ab4ebfe6b83996aef4cce8d58ae2d51ced/crates/hcl-primitives/src/number.rs#L17-L22
JSON (and by extension probably YAML) also don't care about the underlying data type


Java 9 .tar.gz downloads have extra "." toplevel folder

Summary
All AdoptOpenJDK .zip/.tar.gz downloads have a single toplevel folder named something like "jdk-XX.Y.ZZ". In case of the .tar.gz downloads for JDK 9 an extra toplevel folder was inserted named ".". This means that if you simply unpack the archive it will look correct, but if you, like us, use an option like tar --strip-components=1 it will not correctly remove the toplevel folder anymore.
Steps to reproduce
Simply download any .tar.gz for any non-9 version (8, 10, 11, etc) and compare the contents, eg using tar tvf OpenJDKblah.tar.gz.
Expected results
For any version the result should look like:
drwxr-xr-x root/root         0 2021-04-20 23:09 jdk8u292-b10/
-r--r--r-- root/root      1522 2021-04-20 23:09 jdk8u292-b10/ASSEMBLY_EXCEPTION
-rw-r--r-- root/root  52758560 2021-04-20 23:09 jdk8u292-b10/src.zip
drwxr-xr-x root/root         0 2021-04-20 23:09 jdk8u292-b10/lib/
drwxr-xr-x root/root         0 2021-04-20 23:09 jdk8u292-b10/lib/amd64/
-rwxr-xr-x root/root     29008 2021-04-20 23:09 jdk8u292-b10/lib/amd64/libjawt.so
drwxr-xr-x root/root         0 2021-04-20 23:09 jdk8u292-b10/lib/amd64/jli/
-rwxr-xr-x root/root<PHONE_NUMBER>-04-20 23:09 jdk8u292-b10/lib/amd64/jli/libjl
.
.
.

Actual results
But for 9 it actually looks like:
drwxrwxr-x jenkins/jenkins   0 2018-10-08 15:33 ./jdk-9.0.4+11/
drwxrwxr-x jenkins/jenkins   0 2018-10-08 15:33 ./jdk-9.0.4+11/man/
lrwxrwxrwx jenkins/jenkins   0 2018-10-08 15:33 ./jdk-9.0.4+11/man/ja -> ja_JP.UTF-8
drwxrwxr-x jenkins/jenkins   0 2018-10-08 15:33 ./jdk-9.0.4+11/man/ja_JP.UTF-8/
drwxrwxr-x jenkins/jenkins   0 2018-10-08 15:33 ./jdk-9.0.4+11/man/ja_JP.UTF-8/man1/
-rw-rw-r-- jenkins/jenkins 13698 2018-10-08 15:33 ./jdk-9.0.4+11/man/ja_JP.UTF-8
.
.
.

Pay special attention to the ./ in front of all the paths.

Unfortunately, we won't be respinning 9 builds for this.


MDS refactoring

As discussed briefly a little while back.
Refactored metadata related to services with the following goals:

Separate the concerns of the various sources of data: FIDO2 MDS, static, filesystem and conformance testing
Provide caching support leveraging standard infrastructure in ASP.Net Core
Pleasant developer experience
Intuitive DI behavior, reduced reliance on static singletons
No breaking changes within the existing core library (i.e. keep IMetadataService the same)

The result is:

IServiceCollection fluent extensions to guide registration (see Fido2NetLibBuilderExtensions)
Simple and caching (via IDistributedCache) implementations of IMetadataService that can accept a collection of priority-ordered metadata repositories. Cache aligns expiry with MDS TOC expiry.
All additional dependencies are in a new Fido2.AspNet project that should be easy to package up in nuget

I like this. However, build is failing due to:
error CS0246: The type or namespace name 'ServiceCollection' could not be found (are you missing a using directive or an assembly reference?) [D:\a\1\s\Test\Test.csproj]
Would you mind fixing those last errors so that it works as intended and I will do a full checkour review for it?
I appreciate your work, thank you.

Sure, no problem. Will sort this out in the next day or two.

FYI,  I tested that the conformance testing client could initialize properly but haven't attempted to run the conformance testing tool against the demo app.

Part of the conformance testing requires downloading some metadata statements from the conformance application and pre-populating the conformance statement cache with those files out of the file system, to demonstrate that the server is capable of reading metadata statements that are not included in MDS.  How can we do that with IDistributedCache?

metadata.zip
Attaching actual conformance statements that need to be pre-loaded into the cache.

I think I want to add some logic to deal with the possibility that the TOC "NextUpdate" value could be in the past if we assume the time is "00:00:00" when parsing the date and the cache service will throw an exception if the absolute expiry time is in the past. Watch this space.

@mackie1001, I made a few changes to get the conformance testing to work again and sent a PR to your fork.

LGTM!


[BUG] While the mod is installed, Mission Mode cannot be joined.  Players kicked with error message: "Client requested equality check too soon"

Bug Description & Context
When starting a game in mission mode, any players who attempt to join the game will be kicked with an unhelpful error message.
In particular, the server will show the error message "Client requested equality check too soon!", and the player will see either that message again, or something about failing authentication.
This issue can be found described and solved in the bugs thread on Steam, but I'm posting it here as well because it took me a long time to actually find the solution :p
While in hindsight this behavior makes sense, it was really inexplicable and puzzling, especially because a google search for that error turns up exactly zero results.  If nothing else, I'm hoping that this issue gets indexed by Google and other people who encounter this error are able to figure out their mistake quicker.
Expected Behavior:
I think it'd be very nice if either:

The server and client were shown an error message which more accurately described the problem.  Even if this means just kicking the player from the game with a custom error message and printing something to the console, it would make the problem a lot easier to notice and diagnose (it definitely would have saved me an hour or so :sweat_smile:)
The server somehow disable mission mode now that it isn't supported.

Steps to recreate:

Install MoreLevelContent correctly on a dedicated server
Have a player join with all content correctly installed on their client as well
Start a mission.

Solution
If you're encountering this bug, just start the game in campaign mode rather than mission mode.

Thanks for this report! I totally missed this, however I've now added this to both the main page on the workshop as well as the FAQ. I've also updated the readme on this github and a future version will block access to mission mode while the mod is installed.


Clarification of Paragraph in Paper

Section 4.2 p7, I read:

NAO seems to constantly outperform random search in the reduced space. Nevertheless, the final architecture chosen by NAO is always one of the architectures from the initial pool, which were sampled uniformly randomly. This indicates that the ranking of NAO is not correctly updated throughout the search and that, in practice, in a reduced space, NAO is similar to random search.

I do not understand this paragraph, in particular the second sentence and why you claim NAO is similar to random while seemingly outperforming it. Could you help clarify it ?

Sorry for my late reply. If you check the standard deviation, you could see that the confidence interval heavily overlaps. So from pure statistical point of view, the performance of random search and NAO is not really different.


Hide fill missing data prompt

Hide fill missing data for now from playground edit mode, since it doesn't work well right now

@hhpang can you remove this one instead of commenting it out? JSON files don't like comments, unfortunately. We can find it in the git history to bring it back pretty easily when we're ready.


added missing Endpoint.AUXILIARY_HEATING and refactored

Have you tested this?

Have you tested this?

I just forgot to add Endpoint.AUXILIARY_HEATING in the get_endpoint() method when implementing Auxiliary Heating in my previous PR.. For get all it was already trying to retrieve data from this endpoint but like I wrote already before, Superb iV doesn't use this endpoint but air-conditioning is used instead.. From discord I got some sample response so I have tested it at least with that

Have you tested this?

and yes I have tested get specific fixture using the get "keyword" option and it work as before, there is just much less if conditions in the code ;)

Thanks for your contributions! 🍻


Add shared role managing event handler and event listener

Purpose
$Subject
SharedRoleMgtHandler handles:

On creating a role with application audience in the primary organization, created the shared roles in the organizations where the app is already shared
On shared app creation in sub-organization

Create application audience roles defined in the parent application
Create organization audience roles if the shared app uses them and if the same organization role is not availble in the sub-org due to any other shared app

config to enable handler:
[[event_handler]]
name= "SharedRoleMgtHandler"
subscriptions =["POST_ADD_ROLE_V2_EVENT", "POST_SHARE_APPLICATION"]

SharedRoleMgtListener handles:

On deleting a shared app(may be due to unsharing the app from given org/ delete the main app and due to that deleting shared app) deleting roles with organization audience in app-shared organizations if no other shared app use them

On main application's role audience/role list update, track the change in preUpdate listerner and set into the threadlocal varibales. Add/ Delete roles from shared organizations

Part of: https://github.com/wso2/product-is/issues/16363
How other Shared Role management happens:

On Rolename update of main role -> Rename shared roles names at role mgt DAO layer
On deletion of main role -> Handled shared role deletion in role mgt DAO layer
On changing permissions of the role in the parent organization -> Permission always assigned to the main role not to the shared role
On unsharing an application from a sub-organization -> Role mgt service level method to delete roles by app id
On deleting a shared B2B application in a parent organization with roles -> Role mgt service level method to delete roles by app id

Depends on:

[ ] https://github.com/wso2/carbon-identity-framework/pull/5001
[x] https://github.com/wso2/carbon-identity-framework/pull/4965
[x] https://github.com/wso2-extensions/identity-organization-management/pull/260
[ ] Bump framework version

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6614732606

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6614732606
Status: failure

PR builder failure is an intermittent failure. passing locally. Hence merging


Add minimum AHK version

Fixes #31

I have made the relevant changes in the my newest patch. I also included your name in my readme. Thanks for pointing this out. And continue to offer up pull requests if you like.


What's the difference between BPC and cross-entropy loss?

↑

For a given sequence x, there is a mathematical relationship between the total cross entropy loss and the BPC (Bits Per Character), which is a constant coefficient difference and does not affect linear regression. However, for the loss per token, since different models have different tokenizers, and the information contained in each token varies, the loss per token is not directly comparable.

To add based on the discussion on X, different denominators on the cross-entropy loss due to varying tokenizers do make a big difference.


Module 5 - Errors when Training the Model

I created the SnowballTarget.yaml file and proceeded with the tutorial.  When I get to "Train the Agent" section and run the code, I get a long list of errors.  I am not sure how to correct this and complete the exercise.
Version information:
ml-agents: 0.31.0.dev0,
ml-agents-envs: 0.31.0.dev0,
Communicator API: 1.5.0,
PyTorch: 1.11.0+cu102
[INFO] Connected to Unity environment with package version 2.1.0-exp.1 and communication version 1.5.0
[INFO] Connected new brain: SnowballTarget?team=0
2023-04-03 20:10:50.913408: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
RuntimeError: module compiled against API version 0xf but this version of numpy is 0xe
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
RuntimeError: module compiled against API version 0xf but this version of numpy is 0xe
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
ImportError: numpy.core._multiarray_umath failed to import
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
ImportError: numpy.core.umath failed to import
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
RuntimeError: module compiled against API version 0xf but this version of numpy is 0xe
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
ImportError: numpy.core._multiarray_umath failed to import
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
ImportError: numpy.core.umath failed to import
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
RuntimeError: module compiled against API version 0xf but this version of numpy is 0xe
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
ImportError: numpy.core._multiarray_umath failed to import
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
ImportError: numpy.core.umath failed to import
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 42, in tf
from tensorboard.compat import notf  # noqa: F401
ImportError: cannot import name 'notf' from 'tensorboard.compat' (/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/bin/mlagents-learn", line 33, in
sys.exit(load_entry_point('mlagents', 'console_scripts', 'mlagents-learn')())
File "/content/ml-agents/ml-agents/mlagents/trainers/learn.py", line 264, in main
run_cli(parse_command_line())
File "/content/ml-agents/ml-agents/mlagents/trainers/learn.py", line 260, in run_cli
run_training(run_seed, options, num_areas)
File "/content/ml-agents/ml-agents/mlagents/trainers/learn.py", line 136, in run_training
tc.start_learning(env_manager)
File "/content/ml-agents/ml-agents-envs/mlagents_envs/timers.py", line 305, in wrapped
return func(*args, **kwargs)
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer_controller.py", line 172, in start_learning
self._reset_env(env_manager)
File "/content/ml-agents/ml-agents-envs/mlagents_envs/timers.py", line 305, in wrapped
return func(*args, **kwargs)
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer_controller.py", line 107, in _reset_env
self._register_new_behaviors(env_manager, env_manager.first_step_infos)
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer_controller.py", line 267, in _register_new_behaviors
self._create_trainers_and_managers(env_manager, new_behavior_ids)
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer_controller.py", line 165, in _create_trainers_and_managers
self._create_trainer_and_manager(env_manager, behavior_id)
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer_controller.py", line 125, in _create_trainer_and_manager
trainer = self.trainer_factory.generate(brain_name)
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer/trainer_factory.py", line 58, in generate
return TrainerFactory._initialize_trainer(
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer/trainer_factory.py", line 105, in _initialize_trainer
trainer = trainer_type(
File "/content/ml-agents/ml-agents/mlagents/trainers/ppo/trainer.py", line 52, in init
super().init(
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer/on_policy_trainer.py", line 44, in init
super().init(
File "/content/ml-agents/ml-agents/mlagents/trainers/trainer/rl_trainer.py", line 50, in init
self._stats_reporter.add_property(
File "/content/ml-agents/ml-agents/mlagents/trainers/stats.py", line 322, in add_property
writer.add_property(self.category, property_type, value)
File "/content/ml-agents/ml-agents/mlagents/trainers/stats.py", line 283, in add_property
self._maybe_create_summary_writer(category)
File "/content/ml-agents/ml-agents/mlagents/trainers/stats.py", line 259, in _maybe_create_summary_writer
self.summary_writers[category] = SummaryWriter(filewriter_dir)
File "/usr/local/lib/python3.9/dist-packages/torch/utils/tensorboard/writer.py", line 220, in init
self._get_file_writer()
File "/usr/local/lib/python3.9/dist-packages/torch/utils/tensorboard/writer.py", line 250, in _get_file_writer
self.file_writer = FileWriter(self.log_dir, self.max_queue,
File "/usr/local/lib/python3.9/dist-packages/torch/utils/tensorboard/writer.py", line 60, in init
self.event_writer = EventFileWriter(
File "/usr/local/lib/python3.9/dist-packages/tensorboard/summary/writer/event_file_writer.py", line 72, in init
tf.io.gfile.makedirs(logdir)
File "/usr/local/lib/python3.9/dist-packages/tensorboard/lazy.py", line 65, in getattr
return getattr(load_once(self), attr_name)
File "/usr/local/lib/python3.9/dist-packages/tensorboard/lazy.py", line 97, in wrapper
cache[arg] = f(arg)
File "/usr/local/lib/python3.9/dist-packages/tensorboard/lazy.py", line 50, in load_once
module = load_fn()
File "/usr/local/lib/python3.9/dist-packages/tensorboard/compat/init.py", line 45, in tf
import tensorflow
File "/usr/local/lib/python3.9/dist-packages/tensorflow/init.py", line 37, in
from tensorflow.python.tools import module_util as _module_util
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/init.py", line 42, in
from tensorflow.python import data
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/data/init.py", line 21, in
from tensorflow.python.data import experimental
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/data/experimental/init.py", line 97, in
from tensorflow.python.data.experimental import service
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/data/experimental/service/init.py", line 419, in
from tensorflow.python.data.experimental.ops.data_service_ops import distribute
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/data/experimental/ops/data_service_ops.py", line 22, in
from tensorflow.python.data.experimental.ops import compression_ops
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/data/experimental/ops/compression_ops.py", line 16, in
from tensorflow.python.data.util import structure
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/data/util/structure.py", line 22, in
from tensorflow.python.data.util import nest
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/data/util/nest.py", line 34, in
from tensorflow.python.framework import sparse_tensor as _sparse_tensor
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/framework/sparse_tensor.py", line 25, in
from tensorflow.python.framework import constant_op
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/framework/constant_op.py", line 25, in
from tensorflow.python.eager import execute
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/eager/execute.py", line 21, in
from tensorflow.python.framework import dtypes
File "/usr/local/lib/python3.9/dist-packages/tensorflow/python/framework/dtypes.py", line 37, in
_np_bfloat16 = _pywrap_bfloat16.TF_bfloat16_type()
TypeError: Unable to convert function return value to a Python type! The signature was
() -> handle

I'm getting this really nasty error myself. I tried the suggested fix seen below, but no luck:
https://github.com/NVlabs/stylegan3/issues/181

Digging into this a bit further, it looks like the issue has something to do with Tensorboard logging...something?
If you comment out this line in the in ml-agents/ml-agents/trainers/trainer/learn.py file:

Colab seems to run and train, just without the Tensorboard logging. I haven't waited long enough to see what the end results are, however, it is running. Not sure if this affects loading the model to the hub or not.
This might be more of an issue with dependencies in the ml-agents library though.
Pinging @simoninithomas. Seems to be a hard stop blocker on Unit 5.

Hey there 👋 a simpler solution was to uninstall tensorflow 2. We updated the notebooks (Huggy and Unit 5). It should work fine now 🤗


Update microflow-timer.md

Add context options description.

@leonardomendix I assume this feature is already shipped?

@leonardomendix I assume this feature is already shipped?

That's right!
We forgot to describe this part of the widget in the documentation.

Solves support-connected issue: https://mendix.atlassian.net/browse/TW-2114


Tidyverse to data table for simulations

Will test integration of sampling and estimation functions again, and merge when successful.
The re-write of the remaining sampling functions will be a new pull-request from this branch to main I guess.

Will test integration of sampling and estimation functions again, and merge when successful. The re-write of the remaining sampling functions will be a new pull-request from this branch to main I guess.

@till-tietz . I am actually working on some edits for the sampling functions and will try to convert multi-/meta- study handlers to data table too for the meta-analyses we are planning. So closing this pull request for now.

@gerasy1987 sounds good...I'm about to wrap up the pps re-write using grouping and data.table. Will push this week.


Adding Teams Meeting ID join changes

Purpose
Adding Teams meeting ID and passcode join feature along with necessary sdk version update
Does this introduce a breaking change?
[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?
[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid
Other Information

#sign-off


Review current approach of sourcing and building packages with Github Actions

What would you like to be added:
The package pipelines should be moved from Gitlab to Github as stated in #1788
Thereby, there was already a PoC created to test the package pipeline in Github.

Centralized package repo for providing a build container: https://github.com/gardenlinux/package-build
Examples for package repos:

https://github.com/gardenlinux/package-htop
https://github.com/gardenlinux/package-iproute2
https://github.com/gardenlinux/package-libyang
https://github.com/gardenlinux/package-systemd

Why is this needed:
We must check if the PoC already covers all the requirements to replace the Gitlab pipeline for sourcing, packaging and deploying Garden Linux packages.

We reviewed the current sourcing and building approach and created issues for enhancing some parts of the package pipeline. All improvements can be found here:

https://github.com/gardenlinux/package-build/issues

As a result, I'm closing this issue now.


X1, X2, X3, X4 Part is missing mounting hole

B2B-XH-AMK(LF)(SN) is missing a mounting hole in the board

Fixed in 6538cdf7a0786e43772336199ad8634c6ffd7720


Con 2837 favorites undo

Carina's comment: When 2 items are removed from the list, fast after one another, the toasts are created correctly. But when you now click the undo for both the toasts, only one addFavorites pipeline request is done. (I think it's due to the blocking of the add action)
Answer: fixed.

Coverage increased (+0.08%) to 32.294% when pulling 8e0fe8e27788915139c500d4e207f4c5fe24883b on CON-2837-favorites-undo into 1c84953edcd0334e8c7f3814e9ba01ad604e58e3 on master.

Style files renamed.


prioritize transport

with a advance technology brain

reply here. talk here i don't understand you..


Add gross carrying amount

We note here that we should be using the ELDM as a base, not the LDM, so we will change this.
For now we will just add it to the appropriate LDM entities.

We note that we actaully added this as part of issue #3


moodle-dl: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for moodle-dl is: A Moodle downloader that downloads course content fast from Moodle
meta.homepage for moodle-dl is: https://github.com/C0D3D3V/Moodle-Downloader-2
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/9nqc4xikqmf3dprskhd450a7sxra8y04-moodle-dl-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/9nqc4xikqmf3dprskhd450a7sxra8y04-moodle-dl-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
moodle-dl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/9nqc4xikqmf3dprskhd450a7sxra8y04-moodle-dl-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A moodle-dl https://github.com/r-ryantm/nixpkgs/archive/6ea3a541d58d4d0c4cf1a823af570d4607fdcbaa.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/6ea3a541d58d4d0c4cf1a823af570d4607fdcbaa#moodle-dl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/9nqc4xikqmf3dprskhd450a7sxra8y04-moodle-dl-<IP_ADDRESS>
ls -la /nix/store/9nqc4xikqmf3dprskhd450a7sxra8y04-moodle-dl-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

moodle-dl

Maintainer pings
cc @kmein for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for moodle-dl is: A Moodle downloader that downloads course content fast from Moodle
meta.homepage for moodle-dl is: https://github.com/C0D3D3V/Moodle-Downloader-2
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/csvkk1hyh7rcky4sm6ykka2fwsx25s1l-moodle-dl-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/csvkk1hyh7rcky4sm6ykka2fwsx25s1l-moodle-dl-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
moodle-dl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/csvkk1hyh7rcky4sm6ykka2fwsx25s1l-moodle-dl-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A moodle-dl https://github.com/r-ryantm/nixpkgs/archive/b19b7edf65a14ae6c3bf7192d620e27b0d80eb51.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b19b7edf65a14ae6c3bf7192d620e27b0d80eb51#moodle-dl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/csvkk1hyh7rcky4sm6ykka2fwsx25s1l-moodle-dl-<IP_ADDRESS>
ls -la /nix/store/csvkk1hyh7rcky4sm6ykka2fwsx25s1l-moodle-dl-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

moodle-dl
moodle-dl.dist

Maintainer pings
cc @kmein for testing.

Superseded by #307421


Ideas

Thank you a lot for your study :)
As a hobby I have been training sisr models, and made some curated versions of datasets. So this study is very interesting to me.
I thought Id just write some ideas/inputs I had (extending this pipeline with tiling, iqa and ic9600):
I see that you use blockiness scoring to filter out jpg compressed images. Its a great idea. I think this can even be extended upon by using IQA metrics such as HyperIQA to score the image tiles for the sisr training dataset to also filter out noisy, blurry etc tiles. (I like my tiles clean. We can still add noise, blur, compression etc into the lr images for the model to learn to deal with them, but for that we need our hr to be clean)
Which brings me to another step, including tiling in this automated process. This will improve i/o speed during training, since it will crop during training anyway. There is no need to load a 4k input image to take a 256x256px crop out of it just as an example. The LSDIR dataset is all over the place with image sized. I like to use 512x512px tiles as my dataset, which is big enough to increase gt size during training to get better results, but also small enough to have good training speeds.
Maybe this pipeline could also additionally be extended by using IC9600 for automatic image complexity assessment. In the end we could have a very good pipeline producing good sisr training datasets automatically. Im just missing tiling, iqa scoring currently in this pipeline (and maybe complexity assessment).
For an example of a recent dataset I curated with all the steps I took would be in the Nature Dataset
I just thought id mention that I think LSDIR has some bad quality images in it that I saw when inspecting it, which will affect model training negatively and it might not be the best reference as a comparison point for a high quality dataset. (This is why a year ago i tried to make a curated version of it called SSDIR)
Something else I might mention, there are small sisr networks to train like Compact, medium like RealPLKSR and big ones like ATD. From experiencs id say small datasets like <3k image tiles is enough to train a small network, between 3-6k is enough for a medium network, and 6k-20k is enough for a big network. Or in other words, I was able to achieve a similiar quality with 8k images in comparison to LSDIR with like 84k input images. As long as we make sure that it is high quality and high variety dataset. Also ensuring these quality requirements on a smaller dataset is more realistic than a huge one.
A good example would be the Nomosv2 dataset musl curated I think, 6k image tiles he distilled out of multiple datasets (LSDIR included).
Those are just my thoughts, inputs and insights. Thank you for this study and your work in this topic :)
(PS I just remembered something else I like to do is to check the dataset for similiar images and delete them so there are no duplicate images or visually similiar ones since thats just redundancy in the training process, Im using Czkawka for that currently)

Thank you for sharing your ideas and insights, and I apologize for the delayed response!
We appreciate your interest in our study and your valuable suggestions.
By incorporating IQA metrics, we would be able to filter out noisy and blurry images, which could further improve the Quality Estimation part. We would like to give this a try.
Additionally, since ImageNet and PASS are mainly composed of low-resolution images, we have not applied tiling so far, but as you mentioned, tiling is certainly effective when working with high-resolution images.
We would also like to explore the idea of using IC9600 for assessing image complexity.
Your other ideas and insights are also helpful, and we will take them into consideration.

Nice :D
Thats actually something im working on / trying to evaluate myself; you can have a loot at my video, probably the timestamp/section from HQ-50K Dataset Curation would be interesting to you, your paper is also referenced in there. I simply applied dataset curation techniques like i often do for my sisr datasets, but then i was like I wanna test these things more so i can proove that these steps help or dont help, and what their effects are, and what ideal thresholds might be etc, which i talk a bit about in the testing section there.
First I started with div2k but the thing is, is has not well enough score distribution for my tests, or in other words, i switched to df2k, because, for example, div2k's highest blockiness score was 33 while in df2k there was still over 100'00 tiles that scored worse in blockiness than that, so blockiness filtering would have way less on an effect on that div2k in comparison
Also div2k had less tiles of course and df2k had more tiles.
Anyway i think some things i explained in the video. Anway im still running tests, there is morre i want to test still. Like filtering what effect is 0.6, 0.7, 0.8 .. scored on complexity alone, since complexity might have a big influence already as im seeing on my graphs, äh and so forth. So which filtering has what effect on which thresholds (is it just higher is better or is there a threshold where the model improves not or only marginally for us to sacrifice tiles and therefore content with it).
Also i think its fair if i just run some iqa tests in the sense of qaling got some best result son benchmarks but does that translate so sisr training output results? How does for example a model trained on the 5000 best scored qaling filtered tiles compare to a model trained on the 5000 best scored hyperiqa filtered tiles which is way faster and less resource intensive to run. So also such tests. Then if higher patch sizes for training profit more from such or certain filtering techniques. If bigger networks with higher patches during training profit even more. Then what if multiscaling is used with these filtering techniques. And also of course, the iqa metrics used influence the results like i show in the video, graphs on psnr would show that the model trained on qaling filtered (score >=4) did worse but dists shows that this model did better, than the baseline model (no filtering applied just tiled df2k).
Äh yeah maybe too much i wanna test, its fascinating, here the video:
https://youtu.be/Nv7fSreanzc?si=C4FhLIXZyNrdCngs


Include cache headers in query responses

It would be great if Influx could include caching headers with query responses. Here are some thoughs I had:

Presumably, once data is loaded, queries on past date ranges won't change, so queries with a time boundary in the past could be cached for a long time (if not indefinitely).  For this, the cache time should probably be configurable.
Alternatively, I'm not sure if it would be cheap to calculate an etag value for a query (perhaps based on a last modified attribute of the shards that would service the query?), but that might also enable caching queries.

I'm going to close this issue due to inactivity. Please comment on the issue if you are still interested in this feature and we will look at it again to see if we should reopen and prioritize it for a future release. I don't think this feature would be able to get in before 1.0 though.
Thanks.


Meaningful Error Output for Gateway HTTP Transport

With RPC over HTTP as the gateway transport, we get meaningful error codes that can be used to distinguish between incorrect passwords, expired passwords, and so on. For example, Status Code 503 means the user's password is expired:
[21:25:08:415] [3416:3418] [ERROR][com.freerdp.core.gateway.rpc] - error! Status Code: 503
[21:25:08:415] [3416:3418] [ERROR][com.freerdp.core.gateway.http] - HTTP/1.1 503 RPC Error: 773
[21:25:08:415] [3416:3418] [ERROR][com.freerdp.core.gateway.http] - Content-Type: text/html
[21:25:08:415] [3416:3418] [ERROR][com.freerdp.core.gateway.http] - Server: Microsoft-IIS/10.0
[21:25:08:415] [3416:3418] [ERROR][com.freerdp.core.gateway.http] - X-Powered-By: ASP.NET
[21:25:08:415] [3416:3418] [ERROR][com.freerdp.core.gateway.http] - Date: Sat, 16 Feb 2019 03:25:09 GMT
[21:25:08:415] [3416:3418] [ERROR][com.freerdp.core.gateway.http] - Content-Length: 27
[21:25:08:416] [3416:3418] [ERROR][com.freerdp.core.gateway.tsg] - tsg_check failure
[21:25:08:416] [3416:3418] [ERROR][com.freerdp.core.nego] - Protocol Security Negotiation Failure
With HTTP as the transport, the error output printed to console isn't particularly useful.  It doesn't seem to be much more than a binary - auth failed or auth succeeded.
Would it be possible to implement the more detailed output we received with RPC when using HTTP transport?

Duplicates #5208


Fix fakeintake port override

What does this PR do?
#incident-26323
The fakeintake is not listening to the port 80 because the port is overriden

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge -p incident

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable DEPLOY_AGENT=true --variable RUN_UNIT_TESTS=on


See the index page with current stored state

The index page of the web UI should output all the current state for the application. For now it should be something simple, probably, just text area or multiple nested lists.
For the initial page rendering we can use the first test file in the tests folder.

@nergal-perm 2 puzzles #12, #13 are still not solved.

@nergal-perm 5 puzzles #12, #13, #14, #15, #16 are still not solved.

@nergal-perm 3 puzzles #12, #13, #14 are still not solved; solved: #15, #16.

@nergal-perm all 5 puzzles are solved here: #12, #13, #14, #15, #16.


GH2799: Bump .NET Core SDK to 3.1.301

Fixes #2799.

@gitfool your changes have been merged, thanks for your contribution 👍


where to put mapleader

-- init.vim
vim.g.mapleader = ','
require("config.lazy")

this doesn't actually work.. my mapleader is still <space> after everything loads. i can get it working by putting my mapleader in a plugin/keys.vim file but then i get this message:
You need to set `vim.g.mapleader` **BEFORE** loading lazy

i'm not sure where to put it to make it work without getting that error message every startup

i will just disable flash.nvim until you can tell me how to override it :)

Have you tried the options.lua file? https://github.com/briandipalma/iac/blob/main/dotfiles/nvim/lua/config/options.lua if you cloned this repo you should have one in the same location.

thanks!


On pypi.org, keywords are a list of strings

Example:
https://pypi.org/project/wiki/
Code:
https://github.com/django-wiki/django-wiki/commit/688999ebce5cc2ddeb65c9383fd375c2f7b9dc26

Has this changed recently?
From a quick google research, I only find it as a space separated string, including the PyPA sample project and setuptools basic example.

It's a bit confusing... but after this [1] change, keywords need to be either comma/semicolon-separated or as a list. I don't think docs are updated correctly. I can try verifying the actual state of things.. if what has happens is what I think it is, then PyPi.org development should assure with their changes that they're 100% docs-driven.
[1] https://github.com/pypa/warehouse/pull/975

From a quick look at that code, it looks like space-separated is still supported: https://github.com/pypa/warehouse/pull/975/files#diff-387b0e7ca289cf745391093d2e5f6f1dR108
And it should, it would be silly to require every package to change their setup.py :)

Okay, I had the issue not more than a couple of weeks ago on pypy.org so I figured the PR from february wasn't it.. but the issue is fixed now, verified it... spaces do work. The issue was that pypi.org would display "k, e, y, w, o, r, d, 1"
Thanks for following up @eliasdorneles
Closing this, since it's not decided to formalize the list version in the docs (it does work on the old pypi.python.org, too).

Gotcha!
Thanks for digging it, anyway! :+1:


Bug mail

When $sendverification = true; I do not receive such email for $forgot

you need to configurate mail server in order to use mail function in php


Table: Date column filters validation

Feature
The date range picker used in date filters should prevent users from picking a start date later than the end date or picking an end date earlier than the start date.
Examples
It could work like the new iTwinUI range date picker component. With the current implementation, it could alternatively have the dates earlier than start disabled when picking an end date and vice versa.

Currently blocked by #905


Install Open-policy-agent and create a ingress-conflicts policy.

This PR is to install OPA(open-policy-agent) using helm chart and create a ingress-conflicts policy.
This helm chart installs OPA as a Kubernetes admission controller. Using OPA, which can enforce fine-grained invariants over arbitrary resources in Kubernetes cluster.
Created a ConfigMap that contains the main OPA policy and default response. This policy is used as an entry-point for policy evaluations and returns allowed:true if policies are not matched to inbound data.
Created a policy ingress-conflicts that prevents Ingress objects in different namespaces from sharing the same hostname.
Also updated the default values in Helm chart to get the ingress-conflicts policy necessary permissions and resource access.

This is been applied to test-1.cloud-platform.service.justice.gov.uk cluster
This can be tested by created the same ingress on a different namespace, use the below for guidance.
https://www.openpolicyagent.org/docs/kubernetes-admission-control.html

Overall LGTM!
I've commented on a few things that I feel we might want to adjust.
Additionally, what do we want to do about =, == and :=? Should we convert the policies to use them as in that handy guide or stick with what is in the examples?

No we can't :(
https://github.com/open-policy-agent/kube-mgmt/issues/11

@vijay-veeranki-moj I've just pushed two commits with my recommended changes. This should work fine now I but let's get someone else to test as well.

@vijay-veeranki-moj I've just pushed two commits with my recommended changes. This should work fine now I but let's get someone else to test as well.

@alkar Agree with you


Video/Audio file Captioning support

Is your feature request related to a problem? Please describe.
In our customized DSpace 6, we used JWplayer streaming the video/audio and provided WEB VTT or SRT captioning support. We are planning move to DSPace 7 and we would like continue support captioning for our media files.
Describe the solution you'd like
Add captioning support in current DSpace 7 media viewer. The idea is to add track information within video tag of media viewer. The HTML 5 video only support WEB VTT format. So the implementation is going to focus on this format only. I propose to use filename to match the media file with associated VTT file and it's language. For example, if you have a media file called "greenteam.mp4", the VTT filename would be "greenteam.mp4-en.vtt", while -en is the language code with .vtt suffix.
Describe alternatives or workarounds you've considered
N/A
Additional context
I implemented one on DSpace 7. Here is some screen shot -

I only need to update front angular code to have this done.
Any suggestions or a better idea of implementing it?
Best,
Ying

Thanks @yingjin for the ticket & PR. I've pulled the PR over to our 7.5 project board so that we can find a reviewer/tester to provide feedback.


Mouse settings crashes local game

https://gist.githubusercontent.com/Cloak-Bot/e9440c5517dc0fa02a7f07a7c1960b5f/raw/a867c57f1aa08a9e5b955407352af9431af000ab/crash.txt
Clicking the 'Mouse Settings' sub-menu should open the mouse settings GUI but instead crashes.
Playing on All the Mods Fabric, launched via MultiMC. ATMF issue here: https://github.com/AllTheMods/ATM-Fabric/issues/23

I have the same problem.

Does this still happen in the 1.16 version ?


ARROW-2974: [Python] Replace usages of "source activate" with "conda activate" in CI scripts

If you do this, then you need to undo the path modification in
https://github.com/apache/arrow/blob/master/ci/travis_env_common.sh#L23
and replace it with something like
source $MINICONDA/etc/profile.d/conda.sh

Well, I forgot that. Thanks!

This contains a temporary fix for ARROW-3186, i.e. the mesonbuild bug we've been seeing


Spike:  Aggregate device failures into a new section called 'All results' which shows all tests across all devices

Note - this is a spike PR, so placing it here to get some feedback. The code provided solves the problem but is not necessarily well architected
At my company we have started distributing UI tests to run them in parallel and get faster feedback. We coagulate results using XCTestHTMLReport and they come back as a per device report. The command to generate the report will take test results from each run is xchtmlreport -r TestResults-0 -r TestResults-1 ... where the directory for the test run was copied off of a node that performed the run.
The generated report can have many separate devices that need selecting to be able to see failures, this is time consuming. The PR adds a new section which will show all tests on all devices to alleviate the single device selection. See screenshot. The main difficulty with this implementation was keeping the UUID unique in the generated HTML, the current solution to that is to prefix each UUID with ALL and so the javascript and HTML still work.

Hello!
We're facing the same problem - a large amount of parallel runs produces a hard-to-read report.
Are there any plans to support/merge this?


If source repo ends with / hosting fails

Reproduction steps
Repository URL
https://github.com/vzamoragbmc/compuware-topaz-for-enterprise-data-plugin/ <- removing trailing / fixed it
https://github.com/jenkins-infra/repository-permissions-updater/issues/2558#issuecomment-1151185093
Expected Results
Hosting worked
Actual Results
14:23:51.430 [main] INFO  i.j.i.r.hosting.Hoster - Approving hosting request 2558
14:23:54.056 [main] INFO  i.j.i.r.hosting.Hoster - Forking compuware-topaz-for-enterprise-data-plugin/
14:23:54.322 [main] WARN  i.j.i.r.hosting.Hoster - No such repository: compuware-topaz-for-enterprise-data-plugin/

Anything else?
No response

hey timja , I check the url, the source repo is not present in particular location, in thses issue?

This field here:
https://github.com/jenkins-infra/repository-permissions-updater/blob/0a6d1db4708f32ae0a48ee7edea2e5cc25b29594/src/main/java/io/jenkins/infra/repository_permissions_updater/hosting/HostingRequest.java#L12
should have trailing slashes stripped.
GitHub displays it in the UI without a trailing / which also didn't help with this issue

This field here:
https://github.com/jenkins-infra/repository-permissions-updater/blob/0a6d1db4708f32ae0a48ee7edea2e5cc25b29594/src/main/java/io/jenkins/infra/repository_permissions_updater/hosting/HostingRequest.java#L12
should have trailing slashes stripped.
GitHub displays it in the UI without a trailing / which also didn't help with this issue

So all the changes are done at this file right ? Or we should raise a another issue for this?


Double click drawio file open empty page

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
Opening .drawio file shown as empty Untitled page
To Reproduce
Steps to reproduce the behavior:

Double click drawio file or right click and and choose open with drawio

Expected behavior
Drawio should read the file and load the diagrm
Screenshots
If applicable, add screenshots to help explain your problem.
draw.io version (In the Help->About menu of the draw.io editor):

draw.io version [13.6.2]

Desktop (please complete the following information):

OS: [Xubuntu 18.04.4 using snap]

Additional context

None

I have the same problem. I'm using Ubuntu 20.04lts. The diagram opens perfectly in windows but it shows empty file when opened in Ubuntu.

I have the same problem. I'm using Ubuntu 20.04lts. The diagram opens perfectly in windows but it shows empty file when opened in Ubuntu.


ERROR:  Error installing cf-uaac:

ERROR:  Error installing cf-uaac:
The 'eventmachine' native gem requires installed build tools.
Can you please help to install uaac. I am getting above error

@nagamanikandank Please provide more information about the environment where you are trying to install the gem

@nagamanikandank  we faced same issue Please guide to us if you problem is solved .


New symbols: $InterfaceNameShort$, $InterfaceNameShortLower$, $Interf…

…aceNameBaseShort$, $InterfaceNameBaseShortLower$, $InterfaceMockNameShort$, $InterfaceMockNameShortLower$
This resolves issue: #17 #

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

I don't think the "Short" ones make a lot of sense here. The code you're calling will for example change "LoggingService" to "Service", which isn't what we want to happen when we're trying to come up with unique variable names for the mock fields.


Wrong AST for conditional that looks like arrow

I noticed a bug in flow with this snippet, while trying to fix the same bug in babel/babylon.
let y = a ? (b) : c => d;

Flow creates an ast that looks like this:
VariableDeclerator
    -> ConditionalExpression
        -> consequent: ArrowFunctionExpression((b) : c => d)
        -> alternate: Literal(null)

but it should be
VariableDeclerator
    -> ConditionalExpression
        -> consequent: Identifier(b)
        -> alternate: ArrowFunctionExpression(c => d)

Tested with flow-parser 0.26 (the latest version on npm)

Yeah, it looks like we disagree with babylon on this parse. It seems pretty ambiguous to me, but we should at least ensure consistency.

Flow actually doesn't parse this successfully. The Literal(null) is flow trying to recover from a parse error. Since it greedily interpreted it as an arrow function with an annotation, there was no : and no alternate.

Looks like both flow (v0.28.0) and babylon (6.8.4, latest on npm) agree now with an unexpected token on the semicolon.
var a, b, c, d;

let y = a ? (b) : c => d; // Fails with "Unexpected token ;"
let y = a ? (b) : (c => d); // Parses as desired
let y = a ? (b) : c => d : "foo"; // Also valid

There is a PR for babylon that makes this valid: babel/babylon#59

Fixing this might introduce an ambiguity in the language:
a ? (b) : c => (d) : e => f

How should that expressions be parsed?
a ? function (b) : c {
    return (d)
} : function (e) {
    return f
}

// or

a ? (b) : function (c) {
    return function (d) : e {
        return f
    }
}

I'm fixing this in babylon (babel/babylon#573); is it ok to throw in ambiguous cases?


[fix](partial update)allow delete sign column in partial update in planForPipeline

Proposed changes
fix doris-6532

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

pls add related PR in your description

run clickbench

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.84 seconds
stream load tsv:          543 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          98.9 seconds inserted 10000000 Rows, about 101K ops/s
storage size:<PHONE_NUMBER>0 Bytes


Frontend Project "PrimeFlix" 'Even better with Fire TV Stick' Section Improved Alignment

Title and Issue number
Frontend Project "PrimeFlix" 'Even better with Fire TV Stick' Section Improved Alignment
Close #3643
Output
Before

After

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md

@MohitGupta121 Kindly review this pull request.

Hi @MohitGupta121
Kindly review this pull request so that I can work on other issues.


feat: deprecate jest-jasmine2

Description:
We discussed that a few times. Maintenance cost is getting higher for jest-jasmine2, and the planned features cannot be implemented using the old runner. Moreover, the Jest team also plans to switch to circus in the next major version, so let's start moving.
In this PR, I have:

Documentation updates.
CI scripts are targeting jest-circus by default. jest-jasmine is tested only for the edge case with detox.init() timeout.
DetoxJasmineAdapter prints a warning once on every next suite with the link to the migration guide.

You are my hero

@noomorph sorry for repeating myself, but really - do we really need to keep jasmine as 'legacy'? Can we not just deprecate in a major version and move on?

@d4vidi, I really really want to rewrite adapter in a very breaking manner, so I would go an extra mile postponing a major release to not breakchange two times in a row, with jasmine and with adapters.

Pull Request Test Coverage Report for Build 2172

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

Files with Coverage Reduction
New Missed Lines
%

detox/src/artifacts/screenshot/ScreenshotArtifactPlugin.js
1
81.82%

detox/src/devices/drivers/android/AndroidDriver.js
1
93.72%

detox/src/devices/drivers/android/tools/ADB.js
1
65.83%

Totals

Change from base Build 1158:
-0.07%

Covered Lines:
3166

Relevant Lines:
3825

💛  - Coveralls

@noomorph I still see there is no documentation re the huge change in #2027, just making sure its going to be a part of this PR.

@rotemmiz, that's right, because #2027 was being treated as a part of this one. I will be working on the docs today. I am not bananas not to supply such PRs without docs, hehe, don't worry.
I'll ask you to review user-facing places on Sunday.

This is so important, we should have done this kind of thing a long time ago, tapping in at the configuration space instead of the user space.
Can you please clarify regarding point no 2, why would this type of setup disregard the module cache?

side note -

Initiate Detox as early as possible, even before we know what tests we are going to run.

That would also enable a fix for #2097, as the device's platform would already be known before it calls register tests

With this new integration with jest-circus. How should I launch the app with the permissions I need?
I got my tests working again by overriding the initDetox function in my CustomDetoxEnvironment like this:
  async initDetox() {
    const instance = await this.detox.init(undefined, {launchApp: false});
    await instance.device.launchApp({permissions: {notifications: 'YES'}});
    return instance;
  }

Is this the best way to define permissions for my device?

@wlindner, I have what to say. I'll answer tomorrow.


T279550: Plugin branding and presentation

https://phabricator.wikimedia.org/T279550
Updates header and adds readme.txt
Notes:

Using this as icon: https://foundation.wikimedia.org/wiki/File:Wikipedia's_W.svg#/media/File:Wikipedia's_W.svg
Looking for a high quality banner image

This is what I expect the current banner and icon to look like once it's published. Sort of bland? We can always update if we receive feedback from design

This is the official account: https://profiles.wordpress.org/wikimediafoundation/


[Auth] Migrate app-backend to use new auth services

Hey, I just made a Pull Request!
Migrate app-backend to use new auth APIs:

[x] Cookie refresh

[x] Backend endpoint, /.backstage/v1-cookie
[x] Frontend provider

[ ] ~Remove issueUserCookie from HttpAuthService and move to auth-node~
[x] Move RedirectToRoot to auth-react
[x] Logout? How do we clear the cookie?
[x] Document index-public-experimental & how to use

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Tried rebasing again to see if that was just a flaky install in CI 😅


Add support for other tracking camera types

Hi. I just received a old camera from a friend that no longer need it : it is the logitech quickcam deluxe (model number V-UBV49) and here's the specs from logitech website.
.
I read at the wiki that we can use pretty much any camera as long as they support opencv (which I'm pretty sure this camera do as in earlier versions of psmoveservice the program was sometime using this camera instead of my pseye but I do no longer have this issue) and I've done the same setup (with libusb drivers) that I did with my pseye but it looks like the program doesn't detect this camera.. As you got already a lot of work to do, I just ask if the support for cameras other that the pseye cam will be added at some point and if this camera would be supported.
Thanks a lot !

I actually want to work on this very issue this week. I actually broke support for other non pseye camera's to try and fix an issue with the OSVR tracking camera getting opened. You can effective re-enable support for other cameras by un-commenting this commented out code:
https://github.com/cboulay/PSMoveService/commit/cb6d5109c9258d8ee60934951655618937e188b4
For non-PS3Eye cameras you don't need to install libusb drivers. We can use the driver that came with the webcam and access the video feed using OpenCV's directshow layer instead.

Oh also the other reason other cameras aren't officially supported yet is that we don't have a way to calibrate the lenses yet, so you're stuck with focal lengths that will only work for the PS3Eye. This will result in tracking positions that are almost certainly wrong. That's actually the real meat of the feature.

To support arbitrary cameras (but not BAD arbitrary cameras) we might need to store config files for every found camera regardless if we intend to use it or not, but add a variable to enable/disable using that camera. The variable can be toggled in the config tool.
This is more complicated than it seems on the surface because we don't want the service to take over cameras that are meant to be used by other VR systems, so the service has to ignore them, EXCEPT when the config tool wants to set whether or not they should be used by PSMoveService.
Should we start off assuming all cameras should not be used then require users to enter the config tool and explicitly toggle on cameras they want?
When the service searches for cameras...

Any cameras found that don't have config files will have config files generated and be defaulted to off
Any cameras found that are not currently in use but are config'd to be used will be turned on
Any cameras found that are currently in use but are config'd to not be used will be turned off

If this is done gracefully then it could even be used to fix accidental unplugs by going into config and toggling off, wait a moment, toggle on.

@cboulay I like the opt-in model. I think making the user enable a camera also provides a good UI hook for running through calibration as well (i.e. running color calibration and pose computation).
Do you know if there is a way via OpenCV to get any identifying information about a camera at a given index. The OSVR guys don't even use OpenCV to open the cameras. They just use libuvc to open them so they can have complete control of camera iteration. But it sure seems a lot easier if we could just get a camera name from opencv and filter that way.

Hi, is there anything new yet for other camera support ? Because I just noticed the focal length is adjustable with that camera :) Otherwise I will probably try to compile from source and see what I can get :)

I got part way into other camera support in the generic_camera branch, specifically the lens calibration tool and the video undistortion back in July.
The calibration involves holding a checkerboard pattern in front of the screen in different orientations. It then takes those samples and tried to figure out what the focal length and lens distortion of the camera is. Unfortunately the current implementation doesn't make it easy to get a reliable calibration (you are given no guidance on where the best places to hold the checkerboard are).
I then had to set aside this work to focus on getting the DualShock4 controller tracking and Kalman filtering working. In a few weeks hopefully I can get back to the distortion calibration tool.
In the mean time your best bet is to compile the code yourself and make the change I mentioned.

I was happy to find cboulay/generic_camera branch is now merged into master 💯
https://github.com/cboulay/PSMoveService/commit/b6e4a8ad998370e9169b51b8a0cb664affcd891e

Unfortunately that branch wasn't complete when it was merged. It was just necessary to merge everything together because there were multiple diverging paths that would have become a nightmare to merge later.
That being said, the camera calibration tool is almost complete. The only things I think are missing is a GUI element to type in the width of the checkerboard boxes on the image (currently hardcoded to 24 mm because that's what I use), and extensive testing.

For the moment would it be possible to simplify settings and calibration of a generic camera to a measurement of it's FOV? Even if the position is a little off due to lense distortion it would still allow for a reasonable approximation.
Also, another option could be to use any psmove cameras being used in tandem with their known specs to progressively calibrate other cameras being used. I understand this may be a more complicated approach, though.

Actually, is this something I would have to recompile for or is there some setting that can be changed for what part is implemented so far?


Disable flaky test

@swift-ci Please test

@slavapestov - I assume that there is a rdar or SR to track this?

Yes, https://bugs.swift.org/browse/SR-9981


Fix module specification hashtable in ModuleSpecification.Tests.ps1

PR Summary
Change Name to ModuleName in module specification hashtable.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[x] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

@sethvs  The broken names was created specially for test "Cannot create from invalid module hashtables".

@sethvs Thanks for your contribution. I have left a comment to make the test even more effective.


Add support for using cookie for auth

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 13 (69.23%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 60.773%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/client/client.go
3
7
42.86%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
818

Relevant Lines:
1346

💛  - Coveralls

@djak250 thank you very much for your contribution.
Can you please have a look at the lint checks?

@djak250 thank you very much for your contribution.
Can you please have a look at the lint checks?

Whoops. Didn't even mean for that to get committed. Fixed.

released with https://github.com/bakito/adguardhome-sync/releases/tag/v0.4.15


chore(CPlusPlus): add linked list implementation

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] New algorithm
[x] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[ ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR

Added to the Linked List implementation. It is menu based now, and does not terminate unless the user gives that command. Also added an option to remove an element from the end of the linked list.
Other information:

Alright, I have tried to solve all the issues that you have pointed out. Since I'm just adding on to an already existing file, I thought the description in README.md is sufficient.

Does the updated destructor do the job?

Thanks so much. This is a major experience for me as an open source contributor and a major confidence booster.
And so happy I could realize what production level code meant. I have never done anything like this before.
Thanks :)

So now you will merge it into your repo or there is something else I should do?


Allow use command name other than default name

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
-

👍

👍


⚠ Bofrost.be has degraded performance

In e8b8f70, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1203 ms

Resolved: Bofrost.be performance has improved in 5f010b2 after 6 minutes.


Update windwp/nvim-spectre

Repo URL:
https://github.com/windwp/nvim-spectre
Checklist:
This is an existing entry, simply moved to the correct category

[x] The plugin is specifically built for Neovim, or if it's a colorscheme, it supports treesitter syntax.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] The title of the pull request is Add/Update/Remove `username/repo`  (notice the backticks around `username/repo`) when adding a new plugin.
[x] The description doesn't mention that it's a Neovim plugin, it's obvious from the rest of the document. No mentions of the word plugin unless it's related to something else.
[x] The description doesn't contain emojis.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Vim is spelled as Vim (capitalized), Lua is spelled as Lua (capitalized), Tree-sitter is spelled as Tree-sitter.
[x] Acronyms should be fully capitalized, for example LSP, TS, YAML, etc.

Thanks for the PR!


Adds Matching resource

This resource lets you define arbitrary data to match and report against. I use this when piping in a generated file and using templates to match against its contents:
Let's say we have a data.json file that gets generated as part of some testing pipeline:
{
  "instance_count": 14,
  "failures": 3,
  "status": "FAIL"
}

This could be piped into goss: goss --vars data.json validate
And then validated against:
---
matching:
  check_instance_count: # Make sure there is at least one instance
    content: {{ .Vars.instance_count }}
    matches:
      gt: 0

  check_failure_count_from_all_instance: # expect no failures
    content: {{ .Vars.failure_count }}
    matches: 0

  check_status:
    content: {{ .Vars.status }}
    matches:
      - not: FAIL

This PR should be merged after #225 and #233

Once #256 is merged, the tests for this should be passing

@aelsabbahy do you think this will make it into the next release?

Was doing some thinking about this PR last night. Some high level thoughts on a V2 implementation of this.
What if this was a validator type, that can be used on any JASON/YAML file or url pointing to one. An optional attribute for both the http and file resource. The tricky part would be the template portion, or maybe we can see if go provides a library that does something like jq.
This implementation works, but feels limited and awkward to use the vars file for this.
Any thoughts?

@aelsabbahy (in response to your question...)
IMO, this starts to feel like feature-creep. I like the idea of JSON/YAML file validation, but I think that work should be externalized to a different tool/library that could then be called from GOSS. I know of tools that already do this e.g. yaml_bot. Incidentally, I think @skippyPeanutButter has started work on a Golang version of that tool ☝(and is looking for collaborators).
Personally, I would like to see this functionality moved to a validator for the file resource. GOSS could then call custom rules validation engines then be be called to validate its structure/contents.  Here's a rough example in my head...
file:
  /path/to/my.json
    exists: true
    filetype: file
    validator:
      jsonValidatorLibrary:
        with_tests: /path/to/json/file/tests

To summarize, I think it's a good idea, but it feels slightly out-of-scope and should probably be handled in a separate library/project.
That is my two cents, but I may be in the minority here.


SyncList.FindIndex added

Had this extension in all my assets since day one. I use this function a lot, probably useful for others too.

Why not use the extension from Linq?
https://docs.microsoft.com/en-us/dotnet/api/system.collections.generic.list-1.findindex?view=netframework-4.7.2
why reinvent the wheel?

It is weird that microsoft added this to List  instead of making it an extension in Linq

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


d.dd is not working

I tried https://www.d.ddinstagram.com/reel/DAoAS7evhdX and it didn't work

omit the www and you are golden!


Sharing pack content: Bundle custom st2packs image vs Shared NFS storage?

Bundling custom st2 packs as immutable Docker image
In current implementation for this chart to use custom st2 packs we rely on building a dedicated Docker image with pack content and virtualenv, pre-installed and bundled beforehand, see: https://docs.stackstorm.com/latest/install/ewc_ha.html#custom-st2-packs
As a downside means any writes like st2 pack install or saving the workflow from st2flow won't work in current HA env.
Share content via ReadWriteMany shared file system
There is an alternative approach, - sharing pack content via read-write-many NFS (Network File System) as High Availability Deployment doc recommends.
Examples
For example, There is a stable Helm chart nfs-server-provisioner which codifies NFS in an automated way.
https://kubernetes.io/docs/concepts/storage/persistent-volumes/#access-modes
lists K8s supported solutions for ReadWriteMany (RWX) volumes.
From that list of volumes, additional interest apart of NFS, goes to CephFS and https://github.com/rook/rook Rook, a CNCF-hosted project for storage orchestration.
Feedback Needed!
As beta is in progress and both methods have their pros and cons, we’d like to hear your feedback, ideas, experience running any of these shared file-systems and which way would work better for you.

I would be very interested to understand what would be the best way to update/add/remove or in general the lifecycle of the packs, given the proposed above approaches.
For the shared content file system I guess we can spin a k8s job that would mount the shared filesystem, update it, and then update mongodb through st2 pack.load packs=some_pack.
For the customs pack Docker image it seems that we need to kill and recreate all containers (which generally should be fine for a immutable infrastructure) in order to rerun the initcontainers that are responsible for coping the pack/virtualenv data into the container, but then what would happen with long running actions (we have actions running from 30mins up to 2h+) inside st2actionrunner containers? Is there a way to ensure that the st2actionrunner container is idle, before recreating it? And then it comes the question on how do we update mongodb - I guess we need to rerun the job-st2-register-content job as part of the pack update process.

I was a bit away from k8s but now it seems providing RWX capable PV become much easier than before. AKS natively support RWX PV via AzureFile plugin. EKS/GKE, they both provide managed NFS services (Amazon Elastic File System/Google Cloud Filestore) and we can use nfs-client-provisioner  BUT, the NFS backed PV may suffer in terms of performance especially when the pack has many dependent pip packages, and/or the package that needs native binary to be built.
IMO though the current approach seems much stable since the only problem is mongodb part.

which generally should be fine for a immutable infrastructure

I feel this is the reasonable way for now. We can just put a load balance in front, then switch traffic once the new st2 deployment become ready. But this is only possible if you can just through away any data like execution logs stored in mongodb. And of course you need some extra work like exporting/importing st2kv data between new/old clusters. This approach is which exactly called as "Content Roll-Over" method in the doc.

Indeed "Roll-Over" deployments seem like the way to go and the mongodb "problem" can most probably be fixed, by externalising it, as discussed in https://github.com/StackStorm/stackstorm-ha/issues/24.

Commenting to provide some additional feedback on this PR, given our current use case.
We are deployed to EKS, and have an EFS volume used for mounting "packs" and "virtualenvs".  We are currently using the outstanding NFS pull request and it has been working like a champ.  We have an in-house API/UI built that allows the construction of user workflows.  Those workflows are ultimately written to the shared EFS volume by a Lambda function so we can update things on the fly.  For our use case, any sort of pack-sharing mechanism that requires "bake and deploy" is not going to do the trick.
And to make things a bit more fun, we do have a core set of internal packs that are essentially shared libraries utilized by the custom packs that we develop with our API/UI.  We're considering a build job that takes our "shared library packs", assembles a package (maybe simply a tarball), and deploys them to the cluster using a Lambda that writes to the EFS volumes and issues the "register" and "setup_virtualenvs" API calls.  We could use a custom pack image for this piece, but then I need to make some significant changes to the Helm chart to support both a custom pack image AND NFS.  I truly do not want to do this, because we'd like to stay more in-line with master for easier updates and ability to contribute back.
Cheers!

I am working on converting an old 1ppc k8s install to stackstorm-ha charts.
We use Ceph + rook to handle packs, configs, and virtualenvs with approximately this:
volumeMounts:
- name: st2-packs
  mountPath: /opt/stackstorm/packs
- name: st2-configs
  mountPath: /opt/stackstorm/configs
- name: st2-virtualenvs
  mountPath: /opt/stackstorm/virtualenvs

volumes:
- name: st2-packs
  flexVolume:
    driver: ceph.rook.io/rook
    options:
      fsName: fs1
      clusterNamespace: rook-ceph
      path: /st2/packs
- name: st2-configs
  flexVolume:
    driver: ceph.rook.io/rook
    options:
      fsName: fs1
      clusterNamespace: rook-ceph
      path: /st2/configs
- name: st2-virtualenvs
  flexVolume:
    driver: ceph.rook.io/rook
    options:
      fsName: fs1
      clusterNamespace: rook-ceph
      path: /st2/virtualenvs

This has been working quite well for some time. I'm happy to put together a PR to add support for this to stackstorm-ha.

I started to add charts for the rook-ceph operator, but I think that setting up a storage backend should be out-of-scope for the StackStorm chart because:

There are many different in-tree k8s volume types plugins, and even more out-of-tree volume plugins that can use flexVolume (the older standard) or CSI (the newer standard). rook-ceph + flexVolume happens to be the one that I need, but others are very likely to need NFS or some other storage solution in their cluster.
Our current solution, copy packs from images into emptyDir volumes, makes for a good robust cluster-agnostic default.
Setting up an operator like rook-ceph ideally uses namespaces separate from the st2 namespace. This becomes problematic when considering installations like helm install --namespace st2 stackstorm-ha . because that ends up putting all of that storage infrastructure in the st2 namespace.

So, I think we should allow something like this in values.yaml:
st2:
  packs:
    images: []
    use_volumes: true
    volumes:
      packs:
        # volume definition here (required when st2.packs.use_volumes = true)
      virtualenvs:
        # volume definition here (required when st2.packs.use_volumes = true)
      configs:
        # optional volume definition here

Translating the example from my previous into this context I would put this in my values.yaml:
st2:
  packs:
    images: []
    use_volumes: true
    volumes:
      packs:
        flexVolume:
          driver: ceph.rook.io/rook
          options:
            fsName: fs1
            clusterNamespace: rook-ceph
            path: /st2/packs
      virtualenvs:
        flexVolume:
          driver: ceph.rook.io/rook
          options:
            fsName: fs1
            clusterNamespace: rook-ceph
            path: /st2/virtualenvs
      configs:
        flexVolume:
          driver: ceph.rook.io/rook
          options:
            fsName: fs1
            clusterNamespace: rook-ceph
            path: /st2/configs

Then we can translate that into volume definitions that get mounted to /opt/stackstorm/packs, /opt/stackstorm/virtualenvs, (and maybe /opt/stackstorm/configs).
What does everyone think of this approach? Is this simple enough but flexible enough for most requirements?

I think that looks like a fantastic, flexible approach!


TEIIDTOOLS-817: documentation about local build and deployment instru…

…ctions

@shawkins this from the conversation we had with Bilgin yesterday. Take look and suggest any modifications. surely been a long day :)
https://issues.jboss.org/browse/TEIIDTOOLS-817

@shawkins can we resolve this as I need to take this to 7.5 document possibly

Post commit, I can submit a pr to address the minor stuff.
Bilgin is still commenting though.  I'll follow up on his comment.

I have no further comments


New PR to test SignCLA at 2020-05-05 11:08:18.315068296 -0700 PDT m=+5.425510312

Body of PR

unsigned-fail-staging: test succeeded


sleep issue.

thx for this guide, I might buy this masterpiece, before that, did u solve the sleep issue? thx in advance

I never bothered with it and will not be continuing to update this repo.
I sold the machine and replaced it with a Dell PowerEdge T30 (WIP EFI repo here).

thx. a lot for the nice guide, I just bought this pc yesterday.
for the open core, shall I just copy and paste the OC folder since this is a fresh install not staging?

It's not quite that simple as OC is a lot more picky about configuration. You will also need to generate the SMBIOS data (serial number, UUID, etc) and add that to the config.plist.
If you have Windows on the computer you can run a program to capture a report from your hardware and then I can use that to help you get things configured correctly. Once we get your machine running I would also be willing to do another batch of updates to the repo with your help testing that everything works properly.
If you'd like to do that send me an email and I'll send you instructions on how to take the next steps.
The hardest part of working with the T3610 is USB3 support since they used a 3rd party controller which has not had driver development in a long time and doesn't work with Mojave or Catalina. If you can get by with USB2-only things are a lot easier. If you need USB3 you really have no choice but to buy a well-supported USB3 PCIe controller.
The downside to add-on USB3 controllers is that most of the affordable ones can't be used to boot the system since they require firmware and driver support to be loaded by the OS.

thx u so much, My setup is almost identical to urs, I will test it when big sur comes to the final build, I used to have clover, but never used oc
how to get ur email?
I Will use the USB3 Later not a big deal now

@aelsharawi My email is on my Github profile.

I just ran across some old notes saying that enabling the loading of legacy ROMs is required for sleep to work. Have you tried that?

Checkout the new EFI. It solves the USB issues, sleeps fine, and works with Big Sur.


#13441 [Clojure] Add Spec Validations for the Random namespace

Description
[Clojure] Add Spec Validations for the Random namespace
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
[x] Unit tests are added for small changes to verify correctness (e.g. adding a new operator)

Changes
optimizer.clj: fix a previously added spec
random.clj: added clojure specs
operator_test.clj: expect ctx to be in a map
random_test.clj: update context key to ctx, added tests for random/normal, random/uniform  and random/seed

Thanks @hellonico ! I'll take a look shortly

Thanks for making this better!
I ran some of the examples and found some bugs with the improved validation:
In examples/neural-style
Caused by: clojure.lang.ExceptionInfo: Incorrect adam optimizer options {:clojure.spec.alpha/problems ({:path [:learning-rate], :pred clojure.core/float?, :val 10,  ...
and
 Incorrect random uniform parameters #:clojure.spec.alpha{:problems [{:path [], :pred map?, :val #object[org.apache.mxnet.Context 0x752b69e3 "cpu(0)"],  ...
Would you mind correcting the problems in the example as well?
Thanks 💯

having a look ...

@gigasquid
I see.
1/ The first one is due to the spec below:
The spec is defined as:
(s/def ::learning-rate float?)

while  the received parameter for the optimizer is: (note the 10 instead of 10.0)
(opt/adam {:learning-rate 10
                             :wd 0.005
                             :lr-scheduler lr-sched})

If I make sure we cast to float all the different parameters, I can make the specs to be more flexible with:
(s/def ::int-or-float (s/or :f float? :i int?))

That works ?
2/ Second one is
(random/uniform 0 255 content-np-shape dev)

Where the dev should be passed inside the map (as I understand it) and which probably means the context was being ignored.
Is that ok to change the examples to use a map for the context ?

It's definitely fine to change the example to make it work correctly :)
I'm ok with either changing the example code to be a float or we can think about loosening the spec to be a number?.  What do you think would be more user friendly? I don't have strong feelings on it.

What do you think would be more user friendly?
number? was a better choice, and is nicer to the user.
I have updates the specs.

It's definitely fine to change the example to make it work correctly :)
I am glad you agreed ;)

Thanks @hellonico for your work in improving this. I ran through the examples and everything looks good. It is good to go when CI is green.


Java Script

JavaScript
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I can see the four topics I learned listed along with a suggestion on what I should study first

If you are a beginner in programming, it is important to not only focus on reading but also on doing projects and learning from them. When I was learning JavaScript, I worked on basic projects such as creating a to-do list which covered concepts like dynamic appending and local storage, and also creating a Tic Tac Toe game. It is also important to learn about REST APIs and concepts like PUT, GET, POST, and DELETE.

W3 Schools is a great resource to learn from:  https://www.w3schools.com/js/.
can also check out some of my basic projects on my LinkedIn profile:  https://www.linkedin.com/in/sohan-guptha/.