Allow registering a service without specifying a plugin.

Currently, ServiceManager mandates that a plugin instance be passed in as a parameter when registering a service.
Aside from the fact that the plugin instance is never used, this creates problems for services which are provided by an implementation. Services like ConfigService, which the implementation provides, have no sensible value to pass as a plugin instance.
I propose that a method setProvider(Class<T> service, T provider) be added to ServiceManager. Additionally, the use (if any) of passing in a plugin instance should be clarified.

Perhaps there should be an ServiceProvider interface which Plugin can implement, but so can other things like SpongeServices or something in the implementation. That way, they can be registered for something, so that plugins can enumerate services registered to things that aren't plugins.

pinging @sk89q for this, he designed the API originally.
I agree with the sentiments here, except for ServiceProvider. I can't think of any specific interesting quality that something that "provides services" has, so passing in a plugin object should just be optional.

@gratimax It's not really for special qualities, it's just some kind of ID for the registrar, so plugins could find all services registered by X, plugin or not.

@AlphaModder Then wouldn't Identifiable be enough?

@zml2008
Did your changes allow this to happen?

Since SpongeMod (and any other mod) is now a PluginContainer, I don't think there is any real reason to not require a plugin when registering. Additionally, it gives a reference to who owns the service in case that becomes useful later on


Add --client, --dump, --restore and --readonly

This PR adds some functionality to the postgres entrypoint script. A couple of notes:

--dump writes to stdout
--restore reads from stdin. If supplying input with docker run aptible/postgresql --restore < dump.psql, pass docker run the -i (interactive) flag but not the -t (open pseudo-TTY) flag.
--client, --dump, --restore are wrappers around the psql and pg_dump binaries, but they only transmit one argument, expecting a postgresql:// URL. This seems sort of brittle; maybe it would be better to pass through all arguments ($@). However, it feels semantically right as it is. Thoughts?
--readonly turns on the postgres config option default_transaction_read_only. A user can override this with a SET TRANSACTION command. I'm not aware of a better way to enforce read only mode. Any suggestions? Maybe this is alright.

cc @fancyremarker @aaw

This looks perfect to me. @aaw: Comments or ideas about the SET TRANSACTION issue?

Looks great. I think default_transaction_read_only is the way to go and the rule of thumb with all of the --readonly modes for databases is that they only need to disallow data manipulation (inserts, updates) from a client. Since our users are effectively admins of their databases as well, there will probably be ways to override read-only mode for any of our databases.
I would assume default_transaction_read_only also allows DDL statements like "create table" to execute, which I also think is okay. If a migration runs while we're in readonly mode and taking a snapshot, the migration should be re-runnable when the new read/write database container comes back up. We're just trying to guard against, say, 5 minutes of lost application-level updates to the database from normal user-initiated actions.

Merging.


bugfix: 工单状态查看接口取消鉴权

Codecov Report

Merging #287 (d07a66a) into v2.6.x_develop (bcdf242) will not change coverage.
The diff coverage is 100.00%.

@@               Coverage Diff               @@
##           v2.6.x_develop     #287   +/-   ##
===============================================
  Coverage           55.27%   55.27%
===============================================
  Files                 446      446
  Lines               27564    27564
===============================================
  Hits                15237    15237
  Misses              12327    12327

Impacted Files
Coverage Δ

itsm/ticket_status/permissions.py
62.96% <100.00%> (ø)

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 bcdf242...d07a66a. Read the comment docs.


Multiplayer Morph Mod Bug

Hello, I've been having a lot of problems with the morph mod on LAN games with my friend. It started when I killed a mob, morphed into it, and got disconnected because of a timeout error. After this I joined back and all of my morphs were gone. When I killed another mob to get it's morph I couldn't morph into that mob, but the mob I killed previously, and now the mobs in my morph inventory are all random. I cannot get the whole list to show either. Another problem is that sometimes my friend cannot see when I morph, as in I see a turtle but he sees me. I'm willing to send anything, I'm just not sure how.
Thanks a million!

I think this is sorta fixed now due to commit 7da4632c1dceb47c18e9d1d2f36836fc197466a7.

@Budschie seems not fixed. I use 1.18 with 2.0.1 mod version.
Each time I rejoin the world my morph menu is empty. But when I kill mob it does not drop the soul assuming like I have it (but menu is empty, one big red X only item). When I kill mob I had not before and it drops soul, then sometimes (not each time) all my previous mobs are shown in menu too after picking soul.
For server owner all works OK, but for peers not.

@Budschie seems not fixed. I use 1.18 with 2.0.1 mod version (client and server) Each time I rejoin the world my morph menu is empty. But when I kill mob it does not drop the soul assuming like I have it (but menu is empty, one big red X only item). When I kill mob I had not before and it drops soul, then sometimes (not each time) all my previous mobs are shown in menu too after picking soul. For server owner all works OK, but for peers not.

did you ever find a fix? i have the same issue


fix regex maybe

Description
Should make the regex work with both old and new formats. My testing was limited to throwing a few test cases through it so I have no idea how it will perform in the real world. There's a high probability this will break things for FLEX users
Fixes #357
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Just tested this and so far it checks out.


No constructors works

Hi!
This is not working for me
Crouton.makeText(contexto,"Recaregado com sucesso", Style.ALERT);
i get an error, canot resolve method...

What context are you passing? You'll need to pass an Activity.


Train the model

@Jason3900
Hi, first thankyou for removing allennlp dependency from GECToR because i had problem for using that.
Actually I started to run this your version but had some problem and questions if you answer me I would be thankful.

I didn't used CUDA  and NVIDIA. Does it make problem in running. I should say I'm running the code in colab and using T4 gpu.

also I decided to just use part of the synthetic data to just see if it works but it is still about 2 hours and still running. Is it normal?

After it finished it gave this error:

Some weights of RobertaModel were not initialized from the model checkpoint at roberta-base and are newly initialized: ['roberta.pooler.dense.weight', 'roberta.pooler.dense.bias']
You should probably TRAIN this model on a down-stream task to be able to use it for predictions and inference.
2330722it [1:57:15, 26.72s/it][2023-10-25 16:50:16,631] [INFO] [launch.py:315:sigkill_handler] Killing subprocess 6518
[2023-10-25 16:50:16,632] [ERROR] [launch.py:321:sigkill_handler] ['/usr/bin/python3', '-u', 'train.py', '--local_rank=0', '--deepspeed', '--deepspeed_config', 'configs/ds_config_zero1_fp16.json', '--num_epochs', '10', '--max_num_tokens', '128', '--valid_batch_size', '256', '--cold_step_count', '0', '--warmup', '0.1', '--cold_lr', '1e-3', '--skip_correct', '0', '--skip_complex', '0', '--sub_token_mode', 'average', '--special_tokens_fix', '1', '--unk2keep', '0', '--tp_prob', '1', '--tn_prob', '1', '--detect_vocab_path', './data/vocabulary/d_tags.txt', '--correct_vocab_path', './data/vocabulary/labels.txt', '--do_eval', '--train_path', '/content/drive/MyDrive/GEC_test/gector/OUTPUT_FILE', '--valid_path', '/content/drive/MyDrive/GEC_test/gector/OUTPUT_Test', '--save_dir', 'ckpts/ckpt_20231025_14:52:16', '--use_cache', '0', '--log_interval', '1', '--eval_interval', '50', '--save_interval', '50', '--pretrained_transformer_path', 'roberta-base', '--tensorboard_dir', 'logs/tb/gector_20231025_14:52:16'] exits with return code = -9

Should I change "ckpt_path="ckpts/globalstep-xxxx"" in predict.sh ? What should it be?

Maybe my questions are too nave but I'm kind of new in GEC. So if you help me it will really help.

If it doesn't give more clues, you should probably check if there's any OOM errors in your system's log.

okay I will check this.
Actually I'm working on GEC for another language than english. Do you think fast gector can be adopted to other language and how much data it need? because i want to create the dataset myself and maybe it wouldn't have much data.

It mainly depends on what downstream data you are facing. In a common sense, a few thousand of samples may be enough. But if your application varies a lot in the real scenario, it may requires more to achieve a better performance with robustness. Meanwhile, you can try to use wiki history to generate synthetic data.

Have you experienced cpu memory increase until it explodes during training

I haven't encounter any CPU memory leaking issue for now, does it keep increase during training or in the process of loading the task dataset?

during training

I haven't encounter any CPU memory leaking issue for now, does it keep increase during training or in the process of loading the task dataset?

small data may not be obvious to observe, large data have significantly observed cpu memory increase

I haven't encounter any CPU memory leaking issue for now, does it keep increase during training or in the process of loading the task dataset?

small data may not be obvious to observe, large data have significantly observed cpu memory increase

Does it occur in the training process or data loading process, and what is the size of the data you're using?

occur in training process.
two million data may  cpu memory increase but not  explode.
five million data may  cpu memory increase until it explodes.
Total cpu memory is 128g.

It mainly depends on what downstream data you are facing. In a common sense, a few thousand of samples may be enough. But if your application varies a lot in the real scenario, it may requires more to achieve a better performance with robustness. Meanwhile, you can try to use wiki history to generate synthetic data.

Thanks

occur in training process.
two million data may cpu memory increase but not explode.
five million data may cpu memory increase until it explodes.
Total cpu memory is 128g.

occur in training process.
two million data may cpu memory increase but not explode.
five million data may cpu memory increase until it explodes.
Total cpu memory is 128g.


clean-css 4.x dependencie

We will need to delete the resolution deps of clean-css when ember-cli will upgrade his deps to ember-cli-preprocess-registry v4.x.x because it doens't use it anymore
 //package.json "resolutions": {     "clean-css": "^4.1.11"   }
it could also bring new issue with node.js 12
see: https://github.com/ember-cli/ember-cli-preprocess-registry/issues/71

The resolution to<EMAIL_ADDRESS>was removed in the pull request #126 (commit a6d6a25f03882d6d6c22e3186122144052f18988).
I am closing this issue.


Fix Storage Config

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #X, partially #Y, mentioned in #Z

License
Apache 2.0

What's in this PR?
This PR fix the configuration of kafka logs.
Why?
The current implementation is not configuring the kafka.logs property making the cluster use the default directory which is /tmp/kafka-logs. The line added was borrowed from master branch
Additional context

Checklist

[ ] Implementation tested
[ ] Error handling code meets the guideline
[ ] Logging code meets the guideline
[ ] User guide and development docs updated (if needed)

To Do

[ ] If the PR is not complete but you want to discuss the approach, list what remains to be done here

 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.Bernardo Vale seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


About - 小黄的博客 | BY Yellow

http://yellowprogramer.top/about/
True mastery of any skill takes a lifetime.

欢迎大家畅所欲言


CTD After Placing Item Shelf

I had a major crash after placing an item shelf in my world. Here is the report.
crash-2022-01-24_19.47.38-server.txt
I had to remove mod to re-enter the world.

Looks like you are running different versions for the server and client, make sure both are the latest version (20220123a)

Okay, I see. After starting a brand new world the item shelf doesn't cause a crash. Thanks! This is happening in my creative planning world that I've been using for a while. Pardon my ignorance, but is there any way to update the version or should I just avoid placing the shelf in this world?

Yes

If it makes any difference I am using the curseforge app to launch the game.

That's super odd because in the report it makes reference to both "buildersaddition-1.18-20211208a.jar" and "buildersaddition-1.18.1-20220123a.jar"

Fixed in 20220308a.


feat(textarea): resize prop

Checklist

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information
Included prop for the textarea which allows setting the resize css value.

resolves https://github.com/geist-org/react/issues/415


51560341455

### Tasks
- [ ] https://github.com/MicrosoftDocs/appcenter-docs/issues/1512

F e A r


Dropdown menu appears when typing a word

This issue has been moved from a ticket on Developer Community.

[severity:It's more difficult to complete my work]
Hello, I’m using VS 2019 Community, my environment is Python 3.7, and I’m editing a .sh script in my solution. When I type part of a word, a drop-down list of words appears. It is extremely annoying! How can I disable this drop-down list? Of note, when I type part of a word in a .py script within the same solution, the drop-down list thankfully doesn't appear. (I had previously disabled IntelliSense for Python, so maybe that explains the difference in behavior?)

Original Comments
Feedback Bot on 3/7/2021, 10:13 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

It looks like my Feedback issue was automatically added to this GitHub repository. I'm not sure it belongs in PTVS, considering the behavior is with .sh scripts in my VS solution?? Sorry if it got bot-dropped here mistakenly. But I still definitely have this issue, and I'm hoping that it can be resolved!

Microsoft, please fix this issue! If you need any more info, just let me know.

Hi @AJVincelli, you're right that this bug has been miscategorized as being related to Python for VS which is a set of tooling I manage. The fact that it moved to github was just because of the association with Python.
That being said, to try and help you anyway I'm wondering two things:

What is the intent of your .sh file? Can I assume it's a bash script?
What extensions do you have loaded in VS? By default, it appears to me that there is no auto-complete or highlighting in .sh files. However, there are extensions you can load that will enable it. I'd think that removing that extension (assuming you have one) will do the trick.

Hi @greazer, thanks for your reply! Yes, you are correct that the .sh script is a bash script. Though I don't recall explicitly installing any extensions, if I go to Extensions --> Extension Manager --> Installed --> All, I can see three extensions: Visual Studio IntelliCode, NuGetRecommender, and Visual Studio Rich Navigation. I just disabled all three, and the dropdown menu still appears. Is there anything else you can recommend??? I can't believe this is an issue, and that it hasn't been solved yet! I feel like it must just be a tiny setting somewhere that I'm not seeing. Thanks so much for your help.

Here is a screenshot of the ridiculous and annoying dropdown menu that appears whenever I type a character in a .sh script within a Python-environment solution:


Fix discrepancy between tf.placeholder and tf.sparse.placeholder

When working on sparse placeholder I noticed that if shape is presented with None in any dims, the shape information is remvoed from sparse.placeholder. So (None, 4) will be (None, None).
This is different from tf.placeholder:
root@ubuntu:/v# python
Python 2.7.15rc1 (default, Nov 12 2018, 14:31:15)
[GCC 7.3.0] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import tensorflow as tf
>>> v1 = tf.placeholder(tf.float32, (None, 4))
>>> v1.get_shape()
TensorShape([Dimension(None), Dimension(4)])
>>> v2 = tf.sparse.placeholder(tf.float32, (None, 4))
>>> v2.get_shape()
TensorShape([Dimension(None), Dimension(None)])
>>>

This fix adds the fix so that tf.placeholder and tf.sparse.placeholder
behaves the same.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

@yongtang gentle ping! Can you please address the changes requested? Thanks!

It has been 15 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

It has been 43 days that this pull-request has stalled. Please create a new pull-request with the requested changes.


Fixes and improvements to Docker stack image builds and dependencies

Fixes and improvements to Docker stack image builds and dependencies.

Add Python package version constraint to schedulerservice and requestservice Docker images
Optimizing py-sources Docker image build layers for fewer long rebuilds
Default to better versions for NetCDF packags in nwm-deps image
Bump model execution base image to be based on alpine 3.15
Fix base Docker image apk package add command
Add pandas to ngen-deps image apk packages
Fix nwm-deps image build for apk '--repository' use
Fix MPI build issue in nwm-deps in new alpine
Fix bug in nwm-deps with MPI build and job count

Curiously I checkout out this branch and tried ./scripts/control_stack.sh main build and got the following error late in the build:
 => [stage-1 7/8] COPY ./entrypoint.sh entrypoint.sh                                                                                0.0s
 => ERROR [stage-1 8/8] RUN pip install --no-index --find-links=/DIST dmod-subsetservice     && rm -r /DIST                                                                                3.7s
------
 > [stage-1 8/8] RUN pip install --no-index --find-links=/DIST dmod-subsetservice     && rm -r /DIST:
#14 0.876 Looking in links: /DIST
#14 0.907 Processing /DIST/dmod_subsetservice-0.3.0-py3-none-any.whl
#14 0.954 Processing /DIST/Flask-2.0.2-py3-none-any.whl
#14 1.024 Processing /DIST/dmod_modeldata-0.4.0-py3-none-any.whl
#14 1.076 Processing /DIST/Jinja2-3.0.3-py3-none-any.whl
#14 1.127 Processing /DIST/itsdangerous-2.0.1-py3-none-any.whl
#14 1.166 Processing /DIST/click-8.0.3-py3-none-any.whl
#14 1.209 Processing /DIST/Werkzeug-2.0.2-py3-none-any.whl
#14 1.268 Processing /DIST/numpy-1.22.1-cp38-cp38-linux_x86_64.whl
#14 1.617 Processing /DIST/geopandas-0.6.3-py2.py3-none-any.whl
#14 1.682 Processing /DIST/dmod_communication-0.4.1-py3-none-any.whl
#14 1.735 Processing /DIST/pandas-1.4.0-cp38-cp38-linux_x86_64.whl
#14 2.330 Collecting hypy@ git+https://github.com/noaa-owp/hypy@master#egg=hypy&subdirectory=python
#14 2.330   Cloning https://github.com/noaa-owp/hypy (to revision master) to /tmp/pip-install-0n67n9ik/hypy
#14 2.330   Running command git clone -q https://github.com/noaa-owp/hypy /tmp/pip-install-0n67n9ik/hypy
#14 3.518 ERROR: Could not find a version that satisfies the requirement MarkupSafe>=2.0 (from Jinja2>=3.0->flask->dmod-subsetservice) (from versions: none)
#14 3.518 ERROR: No matching distribution found for MarkupSafe>=2.0 (from Jinja2>=3.0->flask->dmod-subsetservice)
------
executor failed running [/bin/sh -c pip install --no-index --find-links=/DIST ${subsetservice_package_name}     && rm -r /DIST]: exit code: 1
ERROR: Service 'subset-service' failed to build : Build failed
Error: requested 'build' action failed (1); exiting without proceeding further

Unsure what the issue is there or why I hit it...is that the wrong thing to try? Do I need to clean something first?

After doing docker builder prune and control_stack.sh main build again I got a similar but different error:
#14 4.105 ERROR: Could not find a version that satisfies the requirement websockets>=8.1 (from dmod-communication>=0.3.0->dmod-modeldata>=0.3.0->dmod-subsetservice) (from versions: none)
#14 4.105 ERROR: No matching distribution found for websockets>=8.1 (from dmod-communication>=0.3.0->dmod-modeldata>=0.3.0->dmod-subsetservice)
------
executor failed running [/bin/sh -c pip install --no-index --find-links=/DIST ${subsetservice_package_name}     && rm -r /DIST]: exit code: 1
ERROR: Service 'subset-service' failed to build : Build failed
Error: requested 'build' action failed (1); exiting without proceeding further

@mattw-nws I've add a workaround for that issue by installing certain packages in a separate pip command (but within the same Docker build layer) that uses the standard index and not just a local directory with wheels from py-sources.

Okay! Now build py-sources and main with success!


Hybrid Azure AD join is not supported on Windows down-level devices w…

when using HAADJ with lower end devices
included mandatory profile - which kind of profile customization and customer reports this is not working when we use mandatory profiles and which is not included in this article as not supported. which cause confusion for customers. hence updated here

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

Pending feedback from PG

#sign-off


errors in OSX when from megaman.geometry import Geometry

errors in OSX when from megaman.geometry import Geometry
the errors are following:
`---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
 in ()
----> 1 from megaman.geometry import Geometry
/Users/myusername/miniconda2/envs/ir-conda/lib/python2.7/site-packages/megaman/geometry/init.py in ()
2
3 from .rmetric import RiemannMetric
----> 4 from .geometry import Geometry
5 from .adjacency import Adjacency, compute_adjacency_matrix, adjacency_methods
6 from .affinity import Affinity, compute_affinity_matrix, affinity_methods
/Users/myusername/miniconda2/envs/ir-conda/lib/python2.7/site-packages/megaman/geometry/geometry.py in ()
34 from scipy import sparse
35 from scipy.special import gammaln
---> 36 from .adjacency import compute_adjacency_matrix
37 from .affinity import compute_affinity_matrix
38 from .laplacian import compute_laplacian_matrix
/Users/myusername/miniconda2/envs/ir-conda/lib/python2.7/site-packages/megaman/geometry/adjacency.py in ()
5 from scipy import sparse
6
----> 7 from .cyflann.index import Index as CyIndex
8 from .utils import RegisterSubclasses
9
ImportError: dlopen(/Users/myusername/miniconda2/envs/ir-conda/lib/python2.7/site-packages/megaman/geometry/cyflann/index.so, 2): Library not loaded: @rpath/libflann.1.8.dylib
Referenced from: /Users/myusername/miniconda2/envs/ir-conda/lib/python2.7/site-packages/megaman/geometry/cyflann/index.so
Reason: image not found
`

what I understand, that the problem in

conda version 4.3.40 (the latest one)
Becaouse it install flann version = 1.9.1 instead of 1.8.5

with previous version, for example 4.3.20  everything works

the solution:
after installation downgrade flann package to version 1.8.5

conda install --channel=conda-forge flann=1.8.5


"In-line sprites cannot have more than 64 frames" warning occurs at >= 64 frames rather than >64 frames.

This warning exists in the file __scribble_gen_9_write_vbuffs.gml, line 304.
Code should check if  _sprite_number > 64 rather than _sprite_number >= 64

Not entirely sure if the docs are wrong or the code is wrong! Limit is based on integer packing so I'll need to investigate and test.

Limit was 63 images in reality so the code was right but the warning was wrong.
Since I was in the area, I bumped up the limit to 127 and reduced the accuracy of the image speed to compensate.
Available in 9.1.0


Revert "Revert "Fix #14 - better sounds""

Reverts awdev1/Vector24#16

No we are not reverting a revert.

nevermind ive been kidnapped back in


[IDEA] feat: Attachment.content allow streams/buffers

Putting large texts into Attachment.content multiple times might require lots of memory.
Proposal: have an option to allow Attachment.content to be streams/buffers, that are only read/processed when it comes to normalizing the data?
Attention: check compatibility for browsers. For node streams and buffers exist.
But they are not part of the common JavaScript implementation ala ES.
As long as there is no clear path, this feature stays just an IDEA.
The thing is, that the Attachment.encoding and the buffer's content might cause some interference,
when the encoding claimed to be base64, but the buffer is not in binary mode, so there is no chance to fetch and convert it properly.

done.
via v2:

Property content was widened to be any stringable, was string ([#516] via [#753])
This enables the use of Buffer and other data-saving mechanisms.


Handling of Breaking Changes

Describe the bug
Upgraded from v2.1.19 to v2.2.3. Renewal of IIS certificate silently stopped working.
Log showed: The JSON value could not be converted to System.String. Path: $.IncludeRegex
To Reproduce

Use v2.1.19 to create IIS-Certificate with option --host-regex (translates to  IncludeRegex in .renewal.json)
Update to v2.2.3 and try to renew

Expected behavior
Update to 2.2.x is marked as

Version 2.2.0 is an xcopy update for users of any 2.x.x version.

Expected would be a working renewal.
Log
The JSON value could not be converted to System.String. Path: $.IncludeRegex
Cause
Serialization changed with commit 35f00b9aee920a7cc036f23255635d4d224fb47e, changing IncludeRegex in json from Type RegEx to Type string.
Workaround
Manually change IncludeRegex serialization in .renewal.json as follows:
FROM: "IncludeRegex": {       "Pattern": "(.*)\\.externaldomain\\.(net|com)",       "Options": 0     }
TO: "IncludeRegex": "(.*)\\.externaldomain\\.(net|com)"
Suggested solution

Mark version update to 2.2.x as breaking change and include info in realese notes.
Enable error mails (taken from settings.json) for deserialization errors to inform users (currently error is silent).

I prefer another solution, which is not to have breaking changes in the first place 😄. Thanks for you thorough bug report, I've cooked up a patch that allows both flavors of JSON to be parsed correctly. You can test the build from this page: https://ci.appveyor.com/project/WouterTinus/win-acme-s8t9q/builds/46831721/artifacts


Rest Theme

Rest theme
Here is my theme "Rest" which I have created using ImThemes and am currently using in my own project.
I wanted to stay away from colours, so the entire theme is completely grey-scale.
Description 🗒
Here's some marketing jargon:

"Rest your eyes with a delightfully pleasing grey-scale theme"

Images 📷

Thanks for contributing. :heart:


Add docs service api token

I couldn't identify exactly what changed with the new z2jh to v1.0 beta, but it seems that we need to specify an api token for the docs service now. When we first added the docs service, we concluded that because the service didn't make any API calls, then we didn't need a token (ref)

@consideRatio do you know what might be causing this?

Merging to unbreak CI.

@yuvipanda @GeorgianaElena hmmm I don't know... Hmmm...
I have touched this logic in z2jh recently I recall so there may be some change there, but it may also be a change in JupyterHub itself.
https://github.com/jupyterhub/zero-to-jupyterhub-k8s/blob/30b3647591af121dc46ddc08a4c517a530f11eee/jupyterhub/files/hub/jupyterhub_config.py#L360-L372
Hmmm... Could it be that it is required to be set explicitly to None, and with my change, it isn't set at all?
https://github.com/jupyterhub/zero-to-jupyterhub-k8s/pull/2042/files#diff-22223d4c6126ae9811f49f6477a770d372019099e5fc613aeb7dcf2b5b5a5323R368-R380

I couldn't identify exactly what changed with the new z2jh to v1.0 beta, but it seems that we need to specify an api token for the docs service now.

I'd like to get a hold of the error shown again, I found this failure, but I understand we don't emit any logs after this automatically and we rely on manual inspection at the moment.
https://github.com/2i2c-org/pilot-hubs/runs/2720750369?check_suite_focus=true

Hmmm hmmm no I'm lost. I don't understand what if anything changed of relevance. Reproduction or clear logs about the error is needed.
Note that if the deployer script fails, we could given access credentials for k8s setup, consider using https://github.com/jupyterhub/action-k8s-namespace-report to get information on failures.


vitables installation problem

Hi,
the qtpy dependency is giving installation problems to some users (see https://github.com/uvemas/ViTables/issues/75). I haven't experienced the problem when installing directly the package, but if a vitables environment is created first and then the package is installed then the problem appears. Would it be possible to add this dependency to the meta.yaml file and see if it fixes the problem?
Thanks

For my part, on linux this conda-forge package fails to run in a clean conda virtualenv for all supported Python versions,

with Python 2.7,conda create -c conda-forge -n vitables27 ViTables==3.0.0 python=2.7
source activate vitables2.7
vitables

fails with,pkg_resources.DistributionNotFound: The 'PyQt5>=5.5.1' distribution was not found and is required by ViTables

where PyQt5.6.0 was pulled by conda; previously reported in https://github.com/uvemas/ViTables/issues/79
With Python 3.5 and 3.6,conda create -c conda-forge -n vitables3.5 ViTables==3.0.0 python=2.7
source activate vitables3.5
vitables

fails with,pkg_resources.DistributionNotFound: The 'ViTables==3.0.0' distribution was not found and is required by the application

previously reported in https://github.com/uvemas/ViTables/issues/74

Hi @rth. As you know many people have problems with the conda-forge package, but the same manual workaround that work for them should work for you too. If it is not the case then I don't know how to help you, sorry :-( What I'm trying to do in #3 is to make the workaround automatically applied when the package is installed. It is not the perfect solution but I think it is an improvement.

@uvemas Last time I used vitables was around 2014 and I'm quite happy it still exists and is working now. Thank you for maintaining and developing it all these years!

As you know many people have problems with the conda-forge package, but the same manual workaround that work for them should work for you too. If it is not the case then I don't know how to help you, sorry :-(

I did see that on the issue tracker, and my point was that this conda-forge package appears a bit broken.  As a result, I think it might be easier to fix it here, rather than suggesting to users in the FAQ that they manually apply some patches for conda. I know your proposed approach in #3 will fix https://github.com/uvemas/ViTables/issues/79 but it still doesn't address the above installation problem for Python >=3.5.
Essentially I think the root cause of this issues is that the install with conda is doubly constrained. The dependencies versions are not specified in recipe/meta.yaml but then they are specified in the setup.py which results in the insatisfied requirements issues. For instance, if I create a virtualenv with all the dependencies,
conda create -n vitables-new qtpy pyqt numpy numexpr pytables  python=3.6
source activate vitables-new

then download vitables 3.0.0 from the github tag, remove the requirements line from the setup.py and install it - everything work fine.
So will propose a PR to patch  setup.py at the build time here  which should also fix #3, unless @scopatz has a better idea... )

I agree with you, it is better to fix this problem on the conda side. Unfortunately conda packages are not my field. Anyway, if I can help just tell me.

I haven't tested it thoroughly yet, but a quick and dirty workaround might be to use pip to build the package instead of setup.py since pip provides an option to ignore dependencies:
pip install --no-deps .

Thanks, @mivade that will be certainly easier than patching setup.py will try that in the PR above.


Datascience with python:Introduction to plotly and cufflinks

Description 📜
Described the use of plotly and cufflinks to plot various plots and use in data visualisation.
Fixes #4983

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[x] Datascience with Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[ x ] I follow Contributing Guidelines & Code of conduct of this project.
[ x ] I have performed a self-review of my own code or work.
[ x ] I have commented my code, particularly in hard-to-understand areas.
[ x ] My changes generates no new warnings.
[ x ] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

Mam how to get mentor assigned?
On Sat, 16 Oct, 2021, 11:55 AM Prathima Kadari, @.***>
wrote:

@.**** requested changes on this pull request.
This is not sufficient. Add README.md file and be elaborate. I don't think
your code blocks are compiled. And also the .ipynb file needs more
detailing & add text and comments for understanding of your code. Kindly
work on it. Get a mentor assigned to you. Any queries, you can ask the
mentor or me. @shreyarai05 https://github.com/shreyarai05
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/5744#pullrequestreview-781334324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUIXTR4L3LQIG4WWZN2CCK3UHELGXANCNFSM5F4XA3PA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@shreyarai05 I'm the mentor

Send me link.
And what coding needed to be involved?
On Sat, 30 Oct, 2021, 2:22 PM Aishani Singh, @.***>
wrote:

@.**** commented on this pull request.
please make the suggested changes.
In README.md
https://github.com/girlscript/winter-of-contributing/pull/5744#discussion_r739624772
:

@@ -0,0 +1,4 @@
+# winter-of-contributing

You'll have to write the README file properly @shreyarai05
https://github.com/shreyarai05 . It should include proper introduction
of the topic, the approach that you're following in your code, advantages
over other methods and conclusion. Please do the same otherwise this won't
be considered as a quality PR.
In Datascience_With_Python/Data Analysis and Visualization/Introduction
to plotly and cufflinks/.ipynb_checkpoints/Untitled-checkpoint.ipynb
https://github.com/girlscript/winter-of-contributing/pull/5744#discussion_r739624898
:

@@ -0,0 +1,6 @@
+{

Please include only those files which have legitimate content in relevance
to the topic and remove everything else. This file doesn't have any code,
please remove it.
In Datascience_With_Python/Data Analysis and Visualization/Introduction
to plotly and cufflinks/Plotly and cufflinks.ipynb
https://github.com/girlscript/winter-of-contributing/pull/5744#discussion_r739627980
:

@@ -0,0 +1,9622 @@
+{

"cells": [

Your file name should be in snake case-
Introduction_to_Plotly_and_cufflinks. And you haven't concluded the coding
that you did, it's incomplete. You can go through the contributing
guidelines, let me know if you need the link to that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/5744#pullrequestreview-793549109,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUIXTR233P6C4XOENFRBARDUJOW5LANCNFSM5F4XA3PA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Tag mentor, when you ask query so that they get notified about your query. @aishanii look into this. There are conflicting files in your PR. You added your README in main branch. Kindly correct it.  @shreyarai05

@aishanii review changes

@aishanii how would i change branch of my readme file from main to other one.

@aishanii  would u change location of my readme file please.

@aishanii mam please look in the PR

This should be done from your end. It says changes have been tried to perform in main README file of the GWOC repository. Might be others contributors have pushed the files at same time as yours. So do one thing, I shall close the PR. Come up with a new PR along with changes suggested here. Ok. @shreyarai05

do i need to get mentor again for my PR?
On Fri, 5 Nov 2021 at 12:24, Prathima Kadari @.***>
wrote:

Closed #5744
https://github.com/girlscript/winter-of-contributing/pull/5744.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/girlscript/winter-of-contributing/pull/5744#event-5572692435,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUIXTRYCDEL7CAUQII7YUALUKN5QVANCNFSM5F4XA3PA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Use a lock around all operations to store reminders

Rebased into master

Should fix the remaining race conditions causing operations to fail (even if without data loss). This will help at least when the calls are made to the same Dapr runtime; if calls are hitting different runtimes, the data races will still cause write operations to fail (without data loss).
Fixes #6018 (whatever is left)
Perf impact
As for impact on perf, it doesn't look like there's any significant impact on the actors_reminder perf test. Although the lock does limit one operation at a time, before we were still using etags to essentially limit to one operation at a time too.
These are the QPS:
BEFORE:
39.43, 39.36, 39.37, 39.51

AFTER:
39.36, 39.40, 39.37, 38.98

Note that these tests were run on a different Kubernetes cluster than the one we use in our Actions, so the results are not comparable. The numbers should be looked at as relative to before and after.

/ok-to-perf

Currently the main perf test that is failing is actor reminders perf test
From scheduled run
actor_reminder_test.go:152: target dapr app consumed 1m CPU and 4.964352Mb of Memory
actor_reminder_test.go:153: target dapr sidecar consumed 2m CPU and 31.531008Mb of Memory
actor_reminder_test.go:154: target dapr app or sidecar restarted 0 times
actor_reminder_test.go:164: 90th percentile: 274.95ms
actor_reminder_test.go:164: 99th percentile: 314.51ms
actor_reminder_test.go:166: Actual QPS: 52.43, expected QPS: 500
actor_reminder_test.go:196:
Error Trace:	/home/runner/work/dapr/dapr/tests/perf/actor_reminder/actor_reminder_test.go:196
Error:      	Should be true
Test:       	TestActorReminderRegistrationPerformance
--- FAIL: TestActorReminderRegistrationPerformance (676.64s)

From /ok-to-perf run above
actor_reminder_test.go:152: target dapr app consumed 1m CPU and 4.984832Mb of Memory
actor_reminder_test.go:153: target dapr sidecar consumed 2m CPU and 28.184576Mb of Memory
actor_reminder_test.go:154: target dapr app or sidecar restarted 0 times
actor_reminder_test.go:164: 90th percentile: 273.69ms
actor_reminder_test.go:164: 99th percentile: 299.90ms
actor_reminder_test.go:166: Actual QPS: 51.59, expected QPS: 500
actor_reminder_test.go:196:
Error Trace:	/home/runner/work/dapr/dapr/tests/perf/actor_reminder/actor_reminder_test.go:196
Error:      	Should be true
Test:       	TestActorReminderRegistrationPerformance
--- FAIL: TestActorReminderRegistrationPerformance (702.88s)

QPS seems to have been reduced by 1 point potentially, not much of a change ideally.

/ok-to-perf

@artursouza I implemented the tests for race conditions as unit tests. This is done by changing the FakeStateStore to have an optional NoLock mode, so it doesn't use locks internally. This means that concurrent access by multiple goroutines will cause a race condition. When locks are used in the actor methods, race conditions should not happen.
You can now validate that race conditions do not happen by running the tests with go test -race. Right now the unit tests pass successfully. If you remove one of the locks, they fail with a race condition.
This should allow preventing (not just detecting) race conditions when requests are made to the same sidecar. With etags, we can then detect race conditions across multiple sidecars.

/ok-to-test

Closing this as it was included in #6325


Proper projection construction

The tests were missing a new when constructing a projection (since ffb7d72c905ccbb542974bbf4597197eb3f24993).  This causes a test failure when we transform the sources to modules (this is undefined in the projection function).

Actually the projection proj is not needed for this test - not for the source neither for the view. So it could as well be removed. The test is really only about different tile grids for source and view.


Programming exercises: Display code hints dynamically

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added @PreAuthorize and checked the course groups for all new REST Calls (security).
[x] I implemented the changes with a good performance and prevented too many database calls.
[x] I documented the Java code using JavaDoc style.

Client

[x] I followed the coding and design guidelines.
[x] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[x] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[x] I documented the TypeScript code using JSDoc style.
[x] I added multiple screenshots/screencasts of my UI changes.
[x] I translated all newly inserted strings into English and German.

Changes affecting Programming Exercises

[x] I tested all changes and their related features with all corresponding user types on Test Server 1 (Atlassian Suite).
[x] I tested all changes and their related features with all corresponding user types on Test Server 2 (Jenkins and Gitlab).

Motivation and Context

These changes are part of the Bachelor Thesis from @Hialus and @ole-ve about the automatic generation of code hints for programming exercises (HESTIA) in Java.
While introducing CodeHints into Artemis, we noticed that the current ExerciseHint implementation is a bit lackluster and as a result almost not used.
In this PR we want to rework the ExerciseHint system, especially on the client-side, to be more user friendly.
Description
First of all we want to make hints more prominent. The current small question mark next to a task is easily overlooked. If hints are available, there is a floating button on the bottom-right (see screenshots below). If one of these hints is not-activated yet (has not been enabled by the user), the button has a zoom animation and a green color to be more recognizable.
When clicking on this button, a modal with all newly, non-activated hints and already activated hints is displayed. Every hint's content can be expanded/collapsed and there is a component to rate the hint. If the user has already rated the hint, the stars are filled.
In addition, the relation between tasks and hints is not done via the problem statement markdown anymore, but via the creation/editing interface for an exercise hint.
Apart from that, minor UI changes have been made to improve the look.
Secondly, we implemented a system where students have to activate a hint first before they see its content. However, a student can only activate hints that are relevant to the current task and only after they have made 3 unsuccessful commits for this task. In order to decide which hint to activate we also added a short description that is always visible.
This reduces spoilers for students who are just interested in what the button does and those who just want to see if there is any helpful hints available.
Steps for Testing

Prerequisites:

1 Instructor/Editor
1 Student
1 Programming Exercise with Text and/or Code Hints

Log in to Artemis as an Editor/Instructor
Navigate to the Exercise Detail Page
Go to the hints page
Distribute the hints between the tasks and edit them however you want (Especially add a description)
Log in to Artemis as a student
Navigate to the exercise
Start the exercise
Do some submissions

3 submissions in a row that do not solve the next task in order to get hints for that task
Any other combination of submissions (be creative) to check for edge cases

Wait for the builds of the submission to finish
Check that the hint button is displayed/not displayed
Click on the hint button
Check that the available hints contain all unactivated hints of the next task (if applicable)
Activate a hint
Check that you can now see the content of the hint
Rate the hint
Close the modal
Do a submission that completes the task
Wait for the build to finish
Check that the hint button is still there
Reopen the modal
Check that the activated hint is still there, but the other hints (if there were any) are not
Check that the rating is still there
Check that hints in quiz exercises are still working by creating a new quiz exercise, adding a multiple-choice question, and use insert a hint ([hint] This is a hint). Then participate in the exercise and make sure the hint is displayed.

Lastly: Be creative and recombine any of the steps in a meaningful order.
Database Change:

We added a new database table exercise_hint_activation that persists the interaction of a user with an exercise hint. This interaction includes activation timestamp and rating. In addition, we added a new column description to provide a description on the hint displayed to the student.
(The associations to jhi_user and exercise_hint) are implicit by the columns and just added for readability)
Review Progress

Server Code Review

[x] Review 1
[ ] Review 2

Client Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

ExerciseHintService.java
85%
97%

ExerciseHintResource.java
64%
89%

exercises/shared/exercise-hint/manage

exercise-hint-detail.component.ts
80%
100%

exercise-hint-update.component.ts
86%
92%

exercise-hint.component.ts
72%
93%

exercises/shared/exercise-hint/participate

exercise-hint-button-overlay.component.ts
75%
94%

exercise-hint-expandable.component.ts
80%
100%

exercises/shared/exercise-hint/shared

code-hint-container.component.ts
77%
83%

solution-entry.component.ts
67%
95%

Screenshots
Workflow for new available hint:
https://user-images.githubusercontent.com/73833780/169666832-7abbb947-1ff4-49fc-a648-3e2c0fd7687f.mov
Selecting a task for an exercise hint:

@rhubinger Please read the PR description more carefully, next time (right at the top):

Do not deploy this to a test server yet. You can either do code reviews or test it locally, until the migration has been approved.

The database migration seems to make no problems but this could have been different.

The migration has been approved by @krusche, the PR can now be tested on the testservers.

The hint question mark button is too small on a 5K Monitor

Thanks for testing. We will implement notifications about new hints in the code editor in a very soon and small follow-up PR.


chore: support --cache-from arg

Improve docker build speed. See the repo: https://github.com/go-training/docker-in-github-actions-vs-drone
FROM golang:1.14-alpine

LABEL maintainer="Bo-Yi Wu<EMAIL_ADDRESS>
RUN apk add bash ca-certificates git gcc g++ libc-dev
WORKDIR /app
# Force the go compiler to use modules
ENV GO111MODULE=on
# We want to populate the module cache based on the go.{mod,sum} files.
COPY go.mod .
COPY go.sum .
COPY main.go .

ENV GOOS=linux
ENV GOARCH=amd64
RUN go build -o /app -tags netgo -ldflags '-w -extldflags "-static"' .

CMD ["/app"]

GitHub Actions
- name: build and push image
  uses: docker/build-push-action@v1
  with:
    username: ${{ secrets.DOCKER_USERNAME }}
    password: ${{ secrets.DOCKER_PASSWORD }}
    repository: appleboy/gin-docker-demo
    dockerfile: Dockerfile
    always_pull: true
    tags: latest

Drone CI
- name: publish
  pull: always
  image: plugins/docker:linux-amd64
  settings:
    auto_tag: true
    cache_from: appleboy/gin-docker-demo
    daemon_off: false
    dockerfile: Dockerfile
    password:
      from_secret: docker_password
    repo: appleboy/gin-docker-demo
    username:
      from_secret: docker_username
  when:
    event:
      exclude:
      - pull_request

Summary
build time

GitHub Actions: 1m10s (logs)
Drone CI: 17s (logs)

Apologies for getting back to you so slowly. Thanks for opening this PR, cache support is something we want to add.
One thing I'd suggest is to allow multiple --cache-from. The option can be used multiple times to give a list of cached images. As github actions doesn't allow for list or complex object inputs the way we've been handling these kind of options is by using a comma-delimited list (see how --build-arg is handled).

@zappy-shu Done. See the commit https://github.com/docker/github-actions/pull/17/commits/f05453caffb843189c22e0d7be18fbd12c34474c

Looks good. Could you also add a line to the build->inputs table in the readme: https://github.com/docker/github-actions#inputs-1
Afterwards I can merge and update the action include the new input

@zappy-shu Done

@zappy-shu Can you merge this feature? Maybe fix the https://github.com/docker/build-push-action/issues/7 issue?


Resizing Left, Top or, Top/Left very jumpy

Issue Type: Bug
Resize the code window using either top, left or top/left handles causes jumpy, unperdictable resizing.
VS Code version: Code 1.22.2 (3aeede733d9a3098f7b4bdc1f66b63b0f48c1ef9, 2018-04-12T16:32:53.389Z)
OS version: Darwin x64 16.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz (8 x 3100)

Load (avg)
2, 2, 2

Memory (System)
16.00GB (3.31GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Extensions (7)

Extension
Author (truncated)
Version

beautify
Hoo
1.3.0

vscode-docker
Pet
0.0.26

npm-intellisense
chr
1.3.0

jshint
dba
0.10.18

vscode-eslint
dba
1.4.8

vscode-npm-script
eg2
0.3.3

python
ms-
2018.3.1

This issue seems to originate from an upstream component and is not something VSCode can fix by making a change to the code. As such, I will go ahead and close it. If you can identify the component that is causing the issue, please report it upstream:

for UI, OS integration: https://github.com/electron/electron
for Rendering: https://bugs.chromium.org/p/chromium/issues/list
for file IO: https://github.com/nodejs/node

Thanks for understanding.


Short solution needed: "ufw xrdp allow" (ufw)

Please help us write most modern and shortest code solution for this issue:
ufw xrdp allow (technology: ufw)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

Allow xrdp with ufw


Upgrade component-contrib to v0.8.0

Description
Upgrade component-contrib to v0.8.0
Issue reference
#1619 https://github.com/dapr/components-contrib/issues/7
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

It should be 0.1.0 for components contrib, no? it's decoupled from the runtime version.


USHIFT-421: Update manifests to more closely align with OCP

Small updates to the rebase script and the produced assets to more closely match OCP's deployments.
Signed-off-by: Frank A. Zdarsky<EMAIL_ADDRESS>Closes USHIFT-421

This looks OK to me. I'll hold off the lgtm until we sort out CI to avoid unnecessary retests.

/retest-required

/lgtm

/retest


src: stream: webrtc: Move from async-std runtime to tokio

It's not clear from the commit body or the PR what is the intended reason behind this PR.

It's not clear from the commit body or the PR what is the intended reason behind this PR.

The async-std runtime was being used in the example code and I forgot to translate this part after finishing the rest, so we are switching from creating async-std runtimes to creating them using the Tokio runtime. Also, from testing I realized that we don't need more than one worker, reducing the overhead.


Easily identify deployed changes

Display the Docker image tag in the SBL Frontend to help identify what changes are currently deployed
Exploratory implementation

How it's done on HMDA

@meissadia Would this only be something we would display during the beta period?

@natalia-fitzgerald I think it would be something helpful even once things reach the Production stage.  On HMDA, clicking that tag links to the project's Github Releases page, which provides interested users with insight into what changes have been made to the site in that version.  So if they experienced and reported a bug on v1 they could see what was changed and deployed in v2.
At the same time, it provides developers with an easy visual confirmation that the deployment matches what is expected.
I would vote to keep this feature thought it would take some expansion of the DSR's <FooterCfGov /> component to enable a layout similar to HMDA's.


BENCH: Benchmarks for signal module

Benchmarks for signal module:

[x] convolution methods (convolve, correlate, fftconvolve)
[x] LTI functions (impulse, step, bode)
[x] spectral analysis tools (periodogram, spectrogram, coherence)

The benchmarks for the filter and filter design functions can be done in signal_filtering.py, that was started by @rgommers as part of #4901 . I would like to seek opinion of others before I go forward.
Cheers !!

Thanks @maniteja123, looks good from a quick browse. I'm on holiday so will take a while before I can review in detail.

For fftpack/fftconvolve benchmarks you should probably test some prime sizes in addition to the easy composite ones?
Would it also be a good idea to track numerical accuracy?  http://asv.readthedocs.org/en/latest/writing_benchmarks.html#tracking-generic says

It is also possible to use asv to track any arbitrary numerical value. “Tracking” benchmarks can be used for this purpose and use the prefix track. These functions simply need to return a numeric value.

So for instance, convolve([0, 1, 1, 0], [0, 1, 1, 0]) → [0, 0, 1, 2, 1, 0, 0], while fftconvolve produces
[ -4.44089210e-16,  -1.11022302e-16,   1.00000000e+00,
         2.00000000e+00,   1.00000000e+00,   1.11022302e-16,
         0.00000000e+00]

so you could track the rms error between the expected result and the actual result?
f = fftconvolve([0, 1, 1, 0], [0, 1, 1, 0])
c = convolve([0, 1, 1, 0], [0, 1, 1, 0])
rmse = np.linalg.norm(c-f)/np.sqrt(len(f))

(or whatever the correct statistical calculation is)

Thanks, lgtm


How to set "f", how to set different RKL/KL/JS divergence？

Hi, I notice you use 'type' in 'args' to set different kinds of divergence. But in the f-gail script. I notice you don't explicitly set a different kind of divergence. If I wanna use a specific kind of divergence. How to set?
Best,
Yuechao.

Hi Yuechao,
Thank you for pointing this out. I forgot to include other divergence implementations in this version, and will update later.
For implementation of pre-defined divergences, such as the JS, KL, RKL and TV divergences, they all have explicit f function (generator functions) that can be found in https://en.wikipedia.org/wiki/F-divergence and in the paper https://arxiv.org/abs/1606.00709. Implement the convex conjugate of the predefined generator function f's and replace with the f_conjugate network will lead you to different divergences.
Many Thanks,
Xin Zhang


Refactor to allow noConflict mode. Also add noConflict function.

By wrapping everything in a closure we create the possibility for a
no conflict mode that will not polute the global namespace.
However, it is totally backwards compatibile as window.UUID is still
the default. If you want noConflict you simply call
UUID.noConflict() which will return you a UUID object and remove
UUID from the global scope if it was not already there.

Thanks for your responsiveness in addressing my concern.
Also thanks for open sourcing your library.


Functional spec & build "Carousel with image cards"

Carousel with image cards (or "Image card carousel"? Actual component name TBD) is a section used on AEM solution template. It can be build as a variant of https://ibmdotcom-web-components.mybluemix.net/?path=/story/components-carousel--default
Acceptance criteria

[x] Finalize design. Needs to be fleshed out with all variations
[ ] Visual and functional spec
[ ] Write out some basic usage guidance :book:

"Image card carousel" component has preliminary design below. Final design would show aspect ratios aligned with existing DDS standard.

This component does not need a change as the images can be added to the cards component as is.  DDS team will add a storybook story - name TBD.  Issue will be assigned to Raph.
@sbahl The DDS focals will explain on the next AEM call.


compiled identifiers clash with names containing non-alphanumeric characters

While working on the other issue, I came across this:
> (compile '%x)
"_37x"
> (compile '-%x)
"_37x"
> (compile '37x)
"_37x"
> (compile '-37x)
"_37x"

And some possibly-surprising behavior:
> (compile "#")
"_35"
> (compile "35")
"_35"
> (eval "#")
function
> (eval "35")
function
> ((eval "35") '(a b c))
3

One way to deal with this would be to update id to check whether the first character is numeric, and if so, prepend a copy of that numeric character. So:
> (compile "37x")
"_3_37x"
> (compile "3-37x")
"_3_3_37x"

Note that leading underscores still pose a problem:
> (compile "37x")
"_3_37x"
> (compile "-3-37x")
"_3_37x"

One strategy would be for id to trim the leading - characters, generate an id, then prepend those trimmed characters just before returning a result.
> (compile "-3-37x")
"__3_3_37x"
> (compile "%x")
"_37x"
> (compile "-%x")
"__37x"
> (compile "37")
"_3_37"
> (compile "-37")
"__3_37"
> (compile "_37")
"_37"

I think that would cover all the cases, allowing - to always prefix any kind of identifier (similar to emacs) without causing conflicts with non-valid-id? characters like !#*@%, while coexisting with identifiers that start with number codes.
Leading underscores are left alone, so the user can explicitly reference compiled identifiers:
> (compile "-3-37x")
"__3_3_37x"
> (eval `(set -3-37x "abc"))
"abc"
> -3-37x
"abc"
> __3_3_37x
"abc"

The only remaining issue is that e.g. (compile 'a%b) and (compile 'a37b) compile to the same identifier, but that's probably not a big deal.
Maybe there's a simpler way to handle all of these cases, though? I don't know.

Probably irrelevant in most cases.


Improved docs for NDSL examples

Description
Tutorials are in important step in on-boarding new library users. While working through the examples, I noticed a couple of typos and broken links. In the bigger picture, we figured that examples should use the boilerplate code that the middleware already exposes in ndsl.boilerplate. I've also added a README file with a little bit of context and a quickstart guide to get people up and running with minimal overhead. And we agreed to remove the output currently stored in the Jupyter notebooks. This was one in the first commit such that we can see the actual changes when looking at the other commits.
In summary

Removed all output from the Jupyter notebooks
Moved plot_filed_at_kN from examples/NDSL/basic_boilerplate.py  to  ndsl/boilerplate.py. This adds matplotlib to the list of dependnecies.
Refactored Jupyter notebooks get all boilerplate code from ndsl.boilerplate.
Removed examples/NDSL/*_boilerplate.py
Added examples/NDSL/README.md with a bit of context.
Fixed typos and broken links in Jupyter notebooks.

It is recommended to exclude the first commit when looking at the diff, e.g. this view is much nicer to look at.
Regarding the previous addition of a .tool-versions file a python version manager: With the install instructions as simple as pip install ndsl[demos], I don't think we need this complexity anymore. The middleware's README already specifies the need for python 3.11. How developers ensure this should be up to them.
Fixes: N/A
How Has This Been Tested?
I ran all the notebooks locally against a local version of NDSL.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules
[ ] New check tests, if applicable, are included

Sorry, for yet another commit. Just realized that one of the notebooks was marked as executable


Validate that Android native layer just works

write some basic native tests to validate using just Java code (no TCP, no node) works properly with Bluetooth. The idea is to prove to ourselves that the Bluetooth layer does work reasonably well and therefore we can know that the problem isn’t Bluetooth, it’s our code.
The tests I am asking him to write are:

 Have one phone open a Bluetooth client socket to the other phone and move 200K of data and have the 200K of data reflected back. The connection should have retry logic so that if a connection fails we will retry and continue from the last data we successfully had reflected back to us. We need to time how long it takes from when we start sending the data until we successfully receive all the data back. The timer should keep running through retrying the connection if it fails in the middle.

 The same test as before but have both phones simultaneously try to send data to each other instead of having one phone sending and the other reflecting. In other words, run the previous test twice in parallel on each handset.

 Run test 1 but with 3 phones. That is, exactly one phone will connect to the other two phones simultaneously and that one phone will send data simultaneously to the other 2 phones and the other 2 phones will reflect back what they get to the sending phone.

 Same test as 2 but with 3 phones. So each phone is simultaneously opening up 2 connections to 2 other phones so we should have 6 connections all running at once.

We don’t have to have the phones perfectly in sync so it’s fine if the tests launch by someone hitting a button on each phone.
These tests are only intended to run manually and only on the phones at Evgeni’s desk.
If these tests pass reasonably then it demonstrates that Bluetooth is reasonably reliable and the issues we are fighting are in our code, they aren’t something fundamentally broken in Bluetooth.
If these tests don’t pass… well… then we need to rethink what we want to do with Android. We do have another option which is to use WiFi hot spots but there are some very scary security issues there.

We agreed with @yaronyg that this issue is not valid now since the issue #1000.

Note that we still need to do this work but we should do it in the context of the perf work that @andrew-aladev is doing. So we need to do this, just not this very moment.


message_format unsupported for in_syslog in v0.14

I believe this will be implemented in due course. It'd be very handy.

in_syslog in v0.14 supports message_format.
<source>
  @type syslog
  <parse>
    @type syslog
    message_format xxx
  </parse>
</source>


MergeTree: Revertibles and Removed Segment Tracking via Local Reference

Description
This change introduces Revertibles to the Merge-Tree package. This co-locates the core revert logic, which can be used to build undo redo in the merge tree package. This improves testability, and reduces concept leakage from the package, as would be necessary to implement elsewhere.
The primary change here is in how we track and revert removed segments. Previously it was don't via tracking the deleted segments, however this only work due to bugs in the handling of removed segments and had negative impacts on the tree itself in terms of fragmentation. For now, the previous way of tracking still works to give people time to move to the new way. Additionally, the new way requires bug fixes currently under an option (mergeTreeUseNewLengthCalculations), which also breaks the old way, so staging the release of this will need to be done carefully.
The primary change in this review is contained in the revertibles file. For insert and remove the code aligns with the existing undo redo implementation. For remove it is substantially different. The primary functions for remove are appendLocalRemoveToRevertibles and revertLocalRemove.  All other changes are in support of the remove function, or tests for revertibles in general, we were severely lacking. The most thorough comes from revertibles.farm.spec.ts. There are simple smoke tests for add remove and annotate is both merge tree, and sequence, along with regression test for issues found with revertibles.farm.spec.ts.
Remove Revertible and Local References
As before, the append functions are event based. The primary difference in the new append for remove is rather than creating a tracking group on the removed segment itself, it adds a sliding local reference to the segment. The local reference contains two properties, 1) the serialized form of the segment, so it can be re-inserted, 2) a tag (referenceSpace) so the reference can be identified as being created for revert. We also configure the afterSlide callback for this reference, and in the case that this reference is detached, we move it to a new kind of segment, the EndOfTreeSegment. This local reference is then added to a tracking group for tracking.
As mentioned above, this change brings in a new kind of segment, the EndOfTreeSegment. This is a special segment that is not bound or known by the merge tree itself, but the segment itself pretends to be a removed segment at the end of the tree. It is removed so it appears as a undefined/0 length segment. This segment adds the capability to hold local references that have been detached from the real merge tree, and give the appearance that they exist at the end of the tree. In general, local references only become detached when the tree becomes empty and the EndOfTreeSegment allows us to gracefully handle that case by giving those references a place to live. The EndOfTreeSegment is not in itself a generalized solution to the problem of detached references, as it requires special and careful understanding of its usage. Specifically, it is local only and would not lead to eventually consistent results if serialization of its contained references was attempted, as on deserialization a naive re-insertion would not put the back into the EndOfTreeSegment leading to inconsistent results with existing clients.
The last part of the revert remove lifecycle is the re-insertion of the removed segment by revertLocalRemove. The first phase of the re-insert is to find the appropriate position. For this we first get the position of the tracked local reference-- the EndOfTreeSegment give a correct result here. After we get the position, we need to see if we must shift it. We only need to shift if the local reference's segment is not removed, and the local references has been slid to its after tombstones, as if it is in the after tombstones, then the original segment was after its current position, and inserts come before by default. We don't need to shift of removed segments, as they have a length of 0, so after or before is meaningless.
After we get the position, we insert the segment as normal, but we are not done, as we need to split or shift any adjacent local references related to revertibles such that we account for the re-insertion. First, we decide if we will move forward or backward from the new segment to the local reference it was inserted for. We do this based on the ordinal, as the ordinal provides are assigned per segment such that they can be lexicographically sorted. We then walk all segments, and all references of those segments from the re-inserted segment to the original local references, and slide any encountered local references that are part of the revertible reference space to the newly inserted segment. This preserves the ordering of all the revertible references in the presence of the re-insert. Generally, this walk will only cover a small number of segments which are the result of local removes, and only segments of unacked local removes will contain any local references. In the absence of removes, the walk will simply split the references on the local refences we inserted for segment.
Lastly, we have special handling for the EndOfTreeSegment if the inserted segment was more a reference on the EndOfTreeSegment. The EndOfTreeSegment will not be seen by the above walk, as it is not part of the tree.
AB#45
AB#27

sorry for the bigger review folks, was getting tougher and tougher to break off reasonable chunks. hopefully its not to bad. ask lots of questions if you have them


Newsletters: add #32 (2019-02-05)

Todo:

[x] Add info about the Chaincode Residency (@jnewbery)
[ ] (Optional) Closer to publication, check to see if LND 0.5.2 final is released and add short note (@harding)

I've added my proposed wording for the Chaincode residency announcement: https://github.com/bitcoinops/bitcoinops.github.io/pull/106/commits/523761a3d3e94439d9bdfe64d929ab0b73644c21. As previously discussed, I'm recusing myself from whether we include this, so the decision for whether to include, edit or exclude the wording is with @harding, @moneyball and @bitschmidty.
I've had a quick scan over the rest of the newsletter and it looks good. I plan to do a full review tomorrow.

Untested ACK 523761a .  Thanks!

Untested ACK 523761a

ACK 523761a3d3e94439d9bdfe64d929ab0b73644c21

utACK. I have a few questions noted above but no showstoppers. Very educational newsletter!

Pushed an updated commit addressing the excellent feedback from @jnewbery and @moneyball (thanks!).  It makes the newsletter longer, but hopefully the extra clarity and accuracy is worth it.
@moneyball To answer your questions:

Does miniscript support functions? ie can the personal policy be in a function instead of inlined?

I don't see that in the described grammar and I wouldn't expect it to be added, as it'd complicate the language significantly.  However, when using a miniscript library in your program, I'd expect you to be able to extend the base class with custom functions or just throw normal functions into policy definitions, e.g.: policy = "thres(3,C,C,C," + myscript(foo, bar) + ")"

Will Schnorr/Taproot allow for that personal policy to be private and not broadcast to the blockchain

Presumably miniscript will allow users of Schnorr-based multiparty signature schemes (e.g. musig) to specify they want that rather than onchain multisig.  In that case, their use of multisigs will be invisible to block chain analysts.  For Taproot, the actual policy used will remain private if the funds are spent cooperatively between the people who controlled those funds; in the non-cooperative case, parts or all of the policies used will have to be revealed on the block chain.

(or even shared with the participants in the 3 of 4)?

The person or people responsible for securing the funds sent to a particular encumbrance (e.g. script, merklized script, taproot, etc...) needs to know the entirety of terms in that encumbrance in order to ensure nobody has inserted any conditions that allow them unilateral ability to spend the funds.  Miniscript doesn't change that obligation but it does make the review easier by allowing a miniscript-compatible encumbrance to be decompiled back into a policy that can be checked for equality at some level to the desired policy (e.g. the outer thres in the example in the newsletter).
Unrelated to miniscript, it might be possible for someone to use zero-knowledge proofs to prove a set of opcodes wouldn't reduce the security of a script without revealing those opcodes to any outside parties when cooperatively creating a receive address, but that's not something I'm aware of anyone working on.  Additionally, even if you did that, you might need to reveal the encumbrance when spending funds sent to the address.

If yes then will miniscript be able to support that?

I think it should be easy for miniscript to be extended to support multi-party Schnorr-based signatures, generic MAST, and Taproot-enabled MAST.  Assuming we don't lose any opcodes or flexibility along the way to implementing those features, I can't see why any existing miniscript couldn't make optimal use of those improvements for both efficiency and privacy.

Merged.

Very educational newsletter!

Agree. I really enjoyed this one. It was a great summary of a lot of very technical information. Thanks Dave!


fixed warning message in set_channel_types() that caused KeyError whe…

…n unit_old was not a key in the _unit2human dict.

this is a fix for https://github.com/mne-tools/mne-python/issues/2878

Style should be fixed, and I added the unknown unit to the dict as you suggested. However I'm not sure about this behavior: in the example I presented in #2878, the unit type is 0, and this code would result in a unit type of '0' (with value '0') being added to the dictionary. As another commenter on the issue mentioned, the underlying file reader might be the cause of the weird unit. I had simply intended to make the warning message not cause an error...

what file reader did you use?

look up FIFF constants

We actually don't have a _UNIT_ with value 0, so whatever converter wrote the FIF file wrote some unit type we don't understand. You should be able to add 0: 'unknown' to the dict around line 182 to fix the problem. Can you try it?


You can even use submit tag to make a button For Example-  <input typ…

…e="submit" value="Submit">

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

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

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


2 poses calibration for yaw offset

Aligns trackers automatically from two poses.
Replace selecting front/back/left/right setting, and make it way more precise too.
Support for upside-down trackers or firmware.

Took me so much time figuring out the Quaternion math. If you have a better way of doing it, give it to me, but this works!

After someone reported bad calibration, I found out that the calibration will be off if the tracker is tilted to the side.
Need to fix this somehow before merging into main

Update: I don't think the way I'm doing it is good enough after all.
Might need to use HMD rotation. Problem is that the HMD yaw and trackers' yaw are all different.
So it needs to align tracker yaw with HMD from the 2 poses.
What my code currently does (which is stupid) is:
/*
* Yaw of IMUs is random (determined when booting up), but pitch and roll is absolute.
* This rotates tracker to exactly 90 degrees and takes the roll. This is indeed the
* equivalent of the yaw we need but absolute.
*/

 Mounting calibration

Created a new branch: https://github.com/Louka3000/SlimeVR-Server/tree/mounting-calibration
Will clean up my code and make it work with the most recent commits and create a new PR once we got the math figured out and all :3


App Redirecting to Provider

I have used angular-oauth2-oidc(implicit flow) for my angular 7 application. But after successfully authentication on third party provider , I get redirected to my application, and my application page opens up but again there is a redirect to the third party and this continues.
I've  used sessionChecksEnabled: false and cleaned all the cookies before logout.

Doesn't sound like an issue with the library, you might have better luck asking a question on Stack Overflow, where there's more people answering questions? In either case, there or here, a repro of your scenario would be needed, otherwise we can only guess at the cause.

I've created below service :
import { OAuthService, JwksValidationHandler } from 'angular-oauth2-oidc';
export class OneService {
constructor(private oauthService : OAuthService){
}
public  configureWithNewConfigApi():void {
    this.oauthService.configure(authConfig);
    this.oauthService.tokenValidationHandler = new JwksValidationHandler();
    this.oauthService.loadDiscoveryDocumentAndLogin();
}

public init():void {
    this.oauthService.initImplicitFlow();
}

public logOut(noRedirectToLogoutUrl?: boolean)
{
    this.oauthService.logOut(noRedirectToLogoutUrl);
}

}
And called this service from my App.component:
OneService .configureWithNewConfigApi();

I'm afraid that your previous post is still a bit too little to go on.
Could you please provide a minimal  repro, e.g. a StackBlitz example to demonstrate the issue? And perhaps elaborate a it more? - Or possibly ask on StackOverflow if it's not an issue with the library, but more of a support question?

Going to tidy up the issue for now. Hoping you solved the issue somehow, if not please let us know if you were able to create a minimal repro somewhere (stackblitz, or a clone-able example) and we could reopen.


Villager GUI bug

When double clicking an item while in the villager trade GUI, the xp bar and levels reset. However after re-entering the trade, it pulls the correct data back likely because this is a client-side bug.

Duplicate of https://github.com/BTW-Community/BTW-Public/issues/291
Also, not a useful title.


added generator interface

Hi, Olaf,
Thanks a lot for implementing the generator interface! That's really great, and I really, really appreciate it!
Beste Gruesse,
Sebastian


[Event Request] Codeunit 12 Gen. Jnl.-Post Line - OnAfterSetVendProdPostingGroup

Hello,
We would like to add the event OnAfterSetVendProdPostingGroup in the procedure VendFindVATSetup in order to set other  ProdPostingGroup value, for the case of PurchLine2.Type::"Fixed Asset".
Thank you!

Please always put the full detailed signature of the requested events and the parameters passed in the call.
You help us implementing the requests faster when your request contains AL code copied from VSCode instead of screenshots.


Add basic support for Expansion

https://pubs.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_06
It sounds like this should be handled in the logic of consuming a Token from the Lexer in the Parser. (the terminal symbol grammar rules)

Some expansions can create multiple fields

It is only field splitting or pathname expansion that can create multiple fields from a single word.

The single exception to this rule is the expansion of the special parameter '@' within double-quotes, as described in Special Parameters.

It actually seems like Expansions are applied as part of Execution, not as part of parsing
Though some grammar rules are applied which do use expansions.
That's actually the exception rather than the norm, which I had backwards for a long time

I think this line in the linked section clears up this confusion:

The expansions described in this section shall occur in the same shell environment as that in which the command is executed.

Since we can add command-specific environment variables, they have to be included.
And to achieve that we have to first fork for the command, which we'll only do in the execution for the command.
Expansions happen after forking

@mraasvel I saw a comment in the executor with a suggestion as to where to implement the expansions, I'm not sure that works
What might work is creating a virtual method expand to implement on every node type.
It looks like some expansions happen directly in the parser while others are delayed until execution.

I think both can work. I was thinking the expander to be stateless, but configurable. Since depending on the context of the expansion, the behavior is different (e.g. heredoc string expansion is only quote removel, and not variable/command/parameter expansion).
I think the case where expansions can sometimes yield multiple tokens is related to field splitting, right? I'm wondering whether this is a distinct step after expansion (perhaps a separate function that yields a vector of tokens from a given input token, or iterator over tokens...)
It can both work, the executor would depend on some Expander object and configure it to only expand what it needs. The node type could also handle the expansion, but then does the node type also handle things like field splitting? I personally have a preference towards having a separate class manipulating and depending on the nodes.


[wip]Set annotations for the operator Deployment created via the ClusterServiceVersion

Description of the change:
Like https://github.com/operator-framework/operator-lifecycle-manager/pull/2054 for labels, we support annotations with this PR.
Motivation for the change:
Architectural changes:

Testing remarks:

Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Bug fixes are accompanied by regression test(s)
[ ] e2e tests and flake fixes are accompanied evidence of flake testing, e.g. executing the test 100(0) times
[ ] tech debt/todo is accompanied by issue link(s) in comments in the surrounding code
[ ] Tests are comprehensible, e.g. Ginkgo DSL is being used appropriately
[ ] Docs updated or added to /doc
[ ] Commit messages sensible and descriptive
[ ] Tests marked as [FLAKE] are truly flaky and have an issue
[ ] Code is properly formatted

/close


infinity  Installing build dependencies ...

Collecting profanity-check   Downloading profanity_check-1.0.3-py3-none-any.whl (2.4 MB)      |████████████████████████████████| 2.4 MB 906 kB/s  Collecting scikit-learn>=0.20.2   Using cached scikit-learn-1.0.tar.gz (7.8 MB)   Installing build dependencies ... \
os: macos
python: 3.10
pip: 21.2.4

thank you 😊
пт, 29 окт. 2021 г., 20:10 Hugo @.***>:

Update : Downgrade your python verson
It seem that sickit isnt supported yet
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vzhou842/profanity-check/issues/38#issuecomment-954946366,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQRCW2JZ3JRDPD6S3D3GM73UJLWSLANCNFSM5FWJ6WZA
.


PR: Add CMake build and function tracing tests

Fixes https://github.com/pytorch/vision/issues/2576

Hi,
Let me know when this is ready for review

@fmassa I'm also having several Windows compilation errors here

This probably means that the CMakeLists that we currently have are broken for Windows. I would say that we can move forward with this PR as is for now (disabling the Windows tests), and then on a follow-up PR try fixing Windows CMakeLists.
Note that there are also some other PRs trying to change / improve the CMakeLists, like https://github.com/pytorch/vision/pull/2587, so this PR will be very helpful to validate that they work.


"Clear" button for the console

It would be nice if the console had a "clear" button to delete everything that has been printed to it up to this point.

That's a darn good idea.


httpx running error

I am using this command python shadowclone.py -i domains.txt -c "/go/bin/httpx -l {INPUT}"         and getting this error
Encountered a bad command exit code!

Command: '/go/bin/httpx -l /tmp/infile'

Exit code: 127

Stdout: already printed

Stderr: already printed

nuclei and subfinder working fine only getting this error in httpx!

python shadowclone.py -i hackerone_new_domains.log --split 100 -o hackerone_new_output.txt -c "httpx -l {INPUT}"
i have the same issue I just change the /go/bin/httpx to httpx only and it work :)

python shadowclone.py -i hackerone_new_domains.log --split 100 -o hackerone_new_output.txt -c "httpx -l {INPUT}"
I have the same issue. I just changed the /go/bin/httpx to httpx only, and it works :)

not working

Anyone solved this? I got same issue with httpx only

same problem here i think the input file is not uploaded into the s3 bucket if i'm understanding this tool correctly

Resolved : i was facing the same issue , i just made sure that i used the dockerfile that have httpx tool in it before building it up

this worked for me try it !
python shadowclone.py -i /home/kali/4.txt  -c "cat {INPUT}|/go/bin/httpx " -o test1.txt -s 1000

also, try this for nuclei
 python shadowclone.py -i /home/kali/ShadowClone/nuc.txt  -c "/go/bin/nuclei -ut && cat {INPUT}|/go/bin/nuclei  -s critical,high -rl 600 -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Firefox/78.0'" -o nuc-rez.txt -s 100


iOS improve a11y sync behavior

Proposed Changes

Implement desktop-like sync logic. The render can issue undefined amount of layout and semantics invalidation, that will be coalesced into a single sync every time it happens.

Implement different logic based on kind of invalidation to avoid extra when only bounds are changed.

Fix wrong refocusing logical bug.

Testing
Test: N/A

I'll do a minor refactor.


Add new Kubernetes procedure for single-node App Dev environment

Reviewers:  The diagram in the HTML build artifact might appear to have font problems, but that's a build-artifact-only issue. Please look at the PDF (LaTeX) image instead of the SVG (HTML) version.
https://build.sawtooth.me/job/Sawtooth-Hyperledger/job/sawtooth-core/job/PR-1822/1/artifact/docs/build/latex/appdev-environment-one-node-3TPs-kube.pdf

I pushed a fixup commit with @rberg2's suggested changes, plus an alignment tweak for the diagram.

I pushed a fixup commit to fix the broken links. I also corrected a few other things, such as a typo, the name of the kubeconfig YAML file, and example prompts (made them consistent with the upcoming multi-node procedure).

I can't link to the kubeconfig YAML file, because the file isn't there yet (it's being added in a separate PR).  When that PR has been merged, I'll add the link in the Kubernetes procedure.

Added a commit that links to the new Kubernetes YAML file (from PR #1830 ).