Performance Issue in Object System

When performing actions that require a methods of the different object type handlers (LCL_OBJECT_* classes), method lcl_objects->create_object is processed, which includes a check for the experimental features flag
    IF lcl_app=>settings( )->read( )->get_experimental_features( ) = abap_true
        AND is_item-obj_type = 'CLAS'.
      lv_class_name = 'LCL_OBJECT_CLAS_NEW'.
    ENDIF.

This leads to select on the ZABAPGIT table to read the settings and deserialize them each time a class creation happens, which is basically at least twice per object in the repository (for is_supported check as well as to acquire the handler instance). For repositories with lots of objects this has a huge impact on performance.
By buffering the settings in a fork I did I measured a performance gain of 600-800% for a the read of objects and 150-200% for the serialization for a repository with roughly 11000 objects.
For smaller repositories, like ABAPGIT, the savings similar, although the savings on the serialization side when not as high.
Since settings are not changing frequently I could imagine buffering them would be a valid option.
Unfortuantely I haven't had time to look into the unit testing for abapGit, so I can't verify if buffering in lcl_persist_settings, like I did for the test, would in break the unit tests and therefore I did not create a pull request.
Any pointers on what needs to be considered to not break the unit tests? Or is that irrelevant?

Cool, thanks, buffering seems like a good idea


Doesn't work when editor groups are vertical

When there are multiple editor groups open, vertically, it saves all of them into a single minimised group, then closes everything.
And when restoring, they are restored in the wrong order, into two horizontal editor groups.

Could you publish and update so this issue don't happen anymore?
thanks


Kong fails to start due to serf 0.8.1

Summary
Kong (latest from homebrew) requires serf 0.8.0 – but I have serf 0.8.1 – this should work but instead it's not running. I'm not able to install serf 0.8.0 from homebrew. Shouldn't Kong accept any minor version?
Steps To Reproduce

brew update & brew install kong
kong start

Additional Details & Logs

Kong version (0.9.9)
Kong debug-level startup logs ($ kong start --vv)

Error:
/usr/local/share/lua/5.1/kong/cmd/start.lua:37: /usr/local/share/lua/5.1/kong/cmd/start.lua:22: incompatible serf found. Kong requires version 0.7.0 to 0.8.0, got 0.8.1
stack traceback:
	[C]: in function 'error'
	/usr/local/share/lua/5.1/kong/cmd/start.lua:37: in function 'cmd_exec'
	/usr/local/share/lua/5.1/kong/cmd/init.lua:89: in function </usr/local/share/lua/5.1/kong/cmd/init.lua:89>
	[C]: in function 'xpcall'
	/usr/local/share/lua/5.1/kong/cmd/init.lua:89: in function </usr/local/share/lua/5.1/kong/cmd/init.lua:45>
	/usr/local/bin/kong:13: in function 'file_gen'
	init_worker_by_lua:38: in function <init_worker_by_lua:36>
	[C]: in function 'pcall'
	init_worker_by_lua:45: in function <init_worker_by_lua:43>

Kong error logs (<KONG_PREFIX>/logs/error.log)

Kong configuration (registered APIs/Plugins & configuration file)

Operating System
macOS Sierra

due to past issues we're rather strict with those.
That said, its easy to modify the constraints, see https://github.com/Mashape/kong/blob/master/kong/meta.lua
Please try and post back the results.

I'd rather not build from source if possible. Isn't it possible to just modify the home-brew formula to just force version 0.8.0 then?

Have you tried:
$ brew switch serf 0.8.0

To use Serf 0.8.0 in the meantime? I am not sure if we can lock a dependency version from a Formula.

Thanks – I tried it, looks like the recipe itself doesn't have 0.8.0. Here's the fun part – I can manually install serf but brew complains when I try to uninstall homebrew's serf because kong requires it :) I'll force uninstall and then install the older serf manually.

I'd rather not build from source if possible.

No need to, the Lua files are compiled by LuaJIT on Kong start. So the plain files are in your system, you can simply update it there and start Kong.

@Tieske There is only one file in the homebrew installed kong: bin/kong – i didn't see any lua.
I manually installed serf 0.8.0 from serf.io and made it work.
But the issue is still valid for others: if anyone uses homebrew today to install kong, I believe it's actually going to break and there might be similar issues opened on github.

@atonse Happy to accept a patch that would lock the version here.

@thibaultcha Ok sure – will do a bit of research and look into it.

Awaiting the pull request referenced above so then we can just point to serf from the mashape-kong tap, to keep versions always in sync.

Serf was downgraded back to 0.7.0 in Kong 0.10.1, thanks for the report.


Building istio fails

OS: Mac OS X :
System Software Overview:

      System Version: OS X 10.11.4 (15E65)
      Kernel Version: Darwin 15.4.0
      Boot Volume: Macintosh HD
      Boot Mode: Normal
      Computer Name: Avinash’s MacBook Pro
      User Name: Avinash Sridharan (asridharan)
      Secure Virtual Memory: Enabled
      System Integrity Protection: Enabled
      Time since boot: 1 day 6:48

Bazel:
bazel version
Build label: 0.2.1-homebrew
Build target: bazel-out/local_darwin-fastbuild/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Fri Apr 1 00:35:17 2016<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>

Getting the following error when I compile from the istio source directory.
make build
bazel  build  //...
ERROR: package contains errors: .
ERROR: no such package '@io_bazel_rules_go//go': error loading package 'external': Could not load //external package.
INFO: Elapsed time: 0.084s
make: *** [build] Error 1
Avinashs-MBP:istio asridharan$
SHA:  12b89fddd9124014b6bd5cb899301b38482e7c6f
<!--
Please see https://istio.io/troubleshooting for potential workarounds.
Please add the correct labels and epics (and priority and milestones if you have that information)
-->

Hi,
We also haven't managed to build Istio directly on MacOS, however we've managed to build it using Docker Tools (it does not work with Docker native driver for MacOS because of lack of support for AUFS) and Docker image with Ubuntu. We just start the container in which we build Istio and publish image to docker registry. Maybe you can also try this approach.
Cheers,
Damian

You need a newer version of Bazel. I currently use:
Build label: 0.7.0-homebrew
Build target: bazel-out/darwin_x86_64-opt/bin/src/main/java/com/google/devtools/build/lib/bazel/BazelServer_deploy.jar
Build time: Thu Oct 19 09:12:48 2017<PHONE_NUMBER>)
Build timestamp:<PHONE_NUMBER>
Build timestamp as int:<PHONE_NUMBER>


don't use raining gold on nuked bosses

Stops Raining Gold from being wasted on bosses that will die to Cripples/Nukes in 0.2 seconds

Also why not add, not waste raining Gold for first 100 Levels on Bosses, because the gain is mediocre there...


[ENH] in-sample forecasting with alternative exogeneous X, e.g., for scenario analysis

As @ngupta23 pointed out in #3224, forecasters that make use of exogeneous data should support in-sample forecasts with exogeneous data different from the data seen - this is useful and desired, say, in the use case of scenario analysis.
I was under the impression that forecasters support this out-of-the-box, but it seems to depend on the forecaster.
Therefore, we should collect come examples of current failures and see how we can support this best.
Some options:

tag for forecasters that can do this vs not
testing this via specific test scenario

@ngupta23, would appreciate your examples of failures, and would very much appreciate if you could boil them down as short pieces of self-contained code pasted here.

Hi @fkiraly I have added a couple of small examples in the notebook here. I could reproduce this issue for ARIMA as well as ETS so may not be specific to pmdarima

@ngupta23, you would expect AutoETS to behave in this way since it ignores X, cf the ignore-exogeneous-X tag.

@ngupta23's example from #3224:
from sktime.datasets import load_airline
from sktime.forecasting.compose import make_reduction
from sklearn.ensemble import GradientBoostingRegressor

y = load_airline()
X = y/2
X.name = "dummy_exogenous"

regressor = GradientBoostingRegressor()
forecaster = make_reduction(regressor, window_length=15, strategy="recursive")
forecaster.fit(y, X)

# Insample predictions with original X ----
y_pred = forecaster.predict(fh=-np.arange(0, len(y)), X=X)

@ngupta23, you would expect AutoETS to behave in this way since it ignores X, cf the ignore-exogeneous-X tag.

Yes, my bad. That should be excluded, but the examples for ARIMA and reduced regression still hold.


Fix string formatting a float without explicit locale

Currently, the internal reader shows me all images as their html tags i.e. <img src=... yrel=...>. This is because of a localization bug when formatting and then later parsing yrel. I fixed this by explicitly using Locale.US, however I see that yrel isn't even used in the internal reader. Is it maybe better to simply remove it?

however I see that yrel isn't even used in the internal reader. Is it maybe better to simply remove it?

I think yes, afaik it was supposed to tell position of images on the page, but we don't really use that method to parse images, so it should be safe to just remove it.


[tf][firehose] support per firehose metric alarms for IncomingRecords

to: @austinbyers or @chunyong-lin
cc: @airbnb/streamalert-maintainers
size: large
Background
Currently, StreamAlert uses individual AWS Firehose delivery streams for each log type ingested and parsed. This enables a somewhat simple and reliable way to determine if processing of a specific log type has dropped off.
Changes

Slight refactor to StreamAlertFirehose client to make performing some actions easier.

The prior class required instantiation for performing operations such as determining enabled log types, etc, which was overkill.
Migrating a few methods to be class methods, since instantiation will create a boto client which is not needed for CLI operations.
Also renaming from StreamAlertFirehose to just FirehoseClient since it's less wordy.
Updating all areas that used this class.

Migrating payload categorization logic to FirehoseClient class instead of handling in the main StreamAlert.
Adding terraform code for minimum log count alarm support in Firehose DeliveryStreams
Updating terraform generation code for firehose log count alarms configuration

Testing

Updating/renaming old StreamAlertFirehose unit tests.
Adding unit tests for firehose terraform generation code (no tests previously existed).

Coverage decreased (-0.1%) to 97.105% when pulling ab826dcfc890544126f54d9e249f35f562eda7ad on ryandeivert-support-per-firehose-metric-alarms into e3e12c88b8ac9191e04f7b61ed84e057bfc9f25f on master.

thanks @austinbyers made the suggested changes :)


Run codegen_integration devicelab test on Windows

codegen_integration_win is not run anywhere and can be deleted.  It is identical to codegen_integration_linux.

Rename codegen_integration_linux builder and codegen_integration_linux.dart test, removing the _linux suffix, to codegen_integration.
Add a Windows codegen_integration builder that will run the new dev/devicelab/bin/tasks/codegen_integration.dart on Windows.

Make sure the releases don't break (by not removing codegen_integration_linux builder, for example).
See also https://github.com/flutter/flutter/issues/86802

\cc @keyonghan

Seems neither windows nor linux tests are running.

Seems neither windows nor linux tests are running.

Right, in #86802 I called out that all three should be running, but aren't.

[ ]  codegen_integration_linux - introduced in Add basic codegen app to be used for integration testing and benchmarks #27257, Run codegen_integration devicelab test on Windows #86830
[ ]  codegen_integration_mac - introduced in Add basic codegen app to be used for integration testing and benchmarks #27257
[ ]  codegen_integration_win - can be combined with codegen_integration_linux, Run codegen_integration devicelab test on Windows #86830

However, instead of creating a Linux codegen_integration_linux and Windows codegen_integration_win builder, with this issue's suggestion it can be a Linux codegen_integration and Windows codegen_integration builder when the test file is renamed.

@jonahwilliams says all three can be removed.
https://github.com/flutter/flutter/issues/86802#issuecomment-885245288


feat: Add fiction warning

Fixes #259
Description

Add fiction banner to landing page

@TheLabCollective/developers


Bigsim autobuild failures in checkpoint/restart test

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

tests/charm++/chkpt/ is failing the past couple of days on the Bigsim autobuild target on Charity.
./build LIBS netlrts-linux-x86_64 bigsim
cd netlrts-linux-x86_64/tests/charm++/chkpt/
make

./charmrun  ./hello +p2 +x3 +y1 +z1  ++local ++no-va-randomization
Charmrun> scalable start enabled.
Charmrun> started all node programs in 0.007 seconds.
Charm++> Running in non-SMP mode: numPes 2
Converse/Charm++ Commit ID: 57cc10f
Charm++> scheduler running in netpoll mode.
BG info> Simulating 3x1x1 nodes with 1 comm + 1 work threads each.
BG info> Network type: bluegene.
alpha: 1.000000e-07	packetsize: 1024	CYCLE_TIME_FACTOR:1.000000e-03.
CYCLES_PER_HOP: 5	CYCLES_PER_CORNER: 75.
BG info> cpufactor is 1.000000.
BG info> floating point factor is 0.000000.
BG info> Using WallTimer for timing method.
CharmLB> Load balancer ignores processor background load.
CharmLB> Load balancer assumes all CPUs are same.
Trace: traceroot: ./hello
Running Hello on 3 processors for 8 elements
Charmrun> error on request socket to node 0 '<IP_ADDRESS>'--
Socket closed before recv.

https://charm.cs.illinois.edu/autobuild/cur/netlrts-linux-x86_64-bigsim.txt

Original date: 2017-11-02 14:55:50

Buil system changes broke bigsim even worse for the past couple days but those have been fixed so this is again showing up.

Original date: 2017-11-02 17:10:28

Phil commented that git bisect shows this commit at fault: ~~https://charm.cs.illinois.edu/gerrit/#/c/381/~~ https://github.com/UIUC-PPL/charm/commit/be7ee10917004cb13a8fd0c27fd0f026bf774c43

Original date: 2017-11-02 20:01:49

This bug is not specific to the checkpoint/restart test. The hang happens when number of Charm PEs does not divide the number of bigsim PEs. For example the following run command also hangs for tests/charm++/simplearrayhello
./charmrun +p2 ./hello 10 +x3 +y1 +z1 +wth1

Original date: 2017-11-02 20:47:35

It looks like the fix for #1735 also solves this. I'll run the tests a few more times to gain a bit of confidence.

Original date: 2017-11-02 20:50:07

Looks like bgtest is happy with that patch, too.

Original date: 2017-11-02 20:51:07

~~https://charm.cs.illinois.edu/gerrit/#/c/3220/~~ https://github.com/UIUC-PPL/charm/commit/51b07e3dfe956ae0ef1691727c1174b3db021f84


I cannot compile Cargo.toml

Hi,
I cloned in my catkin workspace the relaxed_ik_ros1 code (link) and the relaxed_ik_core. After sending the comand cargo build from terminal something went wrong: the compilation stopped and showed the following error. I don't know how to solve this issue. Have you any suggestion? Thank you for your time.

Hello,
I solved this problem in my fork by updating the versions of the dependencies. You can see the commit with those changes here: https://github.com/jmpinit/crash_relaxed_ik_core/commit/057b2e5cb9f9c09ab56991e2a86750fbd3a2fc81

Thank you!


Fine-tuning protein language model？

Congratulations on your great work. In my actual experience，Fine-tuning protein language models should lead to better results. Using technologies such as LoRA should not cause a lot of memory consumption. Also, have you tested other protein language models?

Hi, thanks for your comment! We experimented with directly fine-tuning the full language model, but in our past experience this actually degrades model performance, and as you point out is more memory and compute intensive. Treating the PLM embedding as a fixed representation allows us to pre-compute them to disk and iterate much more quickly on model training.
We haven't yet experimented with more recent methods for lightweight delta tuning such as LoRA, but this is a great idea!
In the supplementary material, we show evaluations comparing ProtBert (which we ended up using in the paper) with ESM and with Prose. We find that each language model is most competitive on different benchmarks, but that ProtBert was most consistently performant. We definitely recommend trying different PLMs, and the ConPLex framework allows for this expansion as PLMa develop in parallel to our work.

Thank you for bringing me new insights. It is possible that there are significant differences in the tasks related to proteins, and fine-tuning may not always bring benefits. Different PLMs indeed have differences in many tasks, so a large-scale benchmark to comprehensively evaluate different PLMs may be a good idea. I tested our proposed protein language model (https://github.com/ISYSLAB-HUST/ProtFlash, 174M) and found that it performs well in some tasks (TAPE), even better than ESM-1b (650M). LoRA feels like a good method for efficiently fine-tuning PLMs, and in the future, we may build some demos using ProtFlash. If there is an opportunity, I would like to explore some collaboration on this field.

I'd be interested to read your manuscript on the proposed PLM! Unfortunately I don't think I'll have the bandwidth to take on additional projects this summer, but do feel free to try ConPLex replacing ProtBert with your proposed method, and I'm happy to provide feedback and technical support with the method. If you're still interested in further collaboration beyond, please send an email including my co-authors on this work to see if there is mutual interest? Thanks!


.WithJavaScript() config failing

Description
Trying to configure with javascript throws exception...
System.TypeLoadException
HResult=0x80131522
Message=Could not load type 'AngleSharp.Services.IScriptingProvider' from assembly 'AngleSharp, Version=<IP_ADDRESS>, Culture=neutral...
Source=AngleSharp.Scripting.JavaScript
StackTrace:
at AngleSharp.JsConfigurationExtensions.WithJavaScript(IConfiguration configuration)
Steps to Reproduce

Create a blank app (universal Windows)
Set target version = 2004, minimum version = 1809
Manage nuget packages; add AngleSharp, AngleSharp.Css, AngleSharp.Scripting.Javascript & AngleSharp.Js
Create a MainPage_Loaded event handler with the following code...

            // use default configuration
            IConfiguration config = Configuration.Default
                .WithDefaultLoader()
                .WithCss()
                .WithJavaScript();

Expected behavior: Ahm, this should work?!
Actual behavior: It doesn't work.
Environment details: Windows 10 Pro v2004 Build=19041.508

Hi @Gavin-Williams the AngleSharp.Scripting.Js is only for older versions of AngleSharp and deprecated. You should use AngleSharp.Js which brings a WithJs extension.
Hope that helps!
For details see the migration guide: https://github.com/AngleSharp/AngleSharp/blob/master/doc/Migration.md#scripting
This was mentioned in #841, too.


Update to ktfmt-gradle-plugin 0.16.0

#655
The new update requires some kind of code changes. These changes should be made after rustlayout is merged in, to avoid a large amount of merge conflicts.

Done


Setup Products Dynamic Links

Follow up of #50
Use dynamic links to launch the product screen(directly) when the product is shared by someone.
If the app is not installed lead to the Play/App Store.

Can I work on this issue @himanshusharma89 ? But it would be only possible after issue #50  is resolved right ?

Yeah, we need to work on #50 to resolve this and also one person can be assigned to only one issue at a time.

@himanshusharma89 please assign this issue to me


Mute status

Hello everyone! As from what i can see on the documentation from allen heath there is no hex combination that can be sent to the console for retreiving channel mute status or current fader level? Just possible to send command for setting mute status and fader level. Am i correct? Thank you

Yes.


Update endpoint throwing wrong arguements error

On the manage fosters page for a staff, changing the start date will cause the shown error.

I'm on FF and not necessarily able to reproduce this error...
https://github.com/user-attachments/assets/1c096f48-eee7-432c-a652-0a09b9492bf7

@MarchandMD Thanks for checking. I'll have a look at this again tonight.

I can't seem to reproduce this anymore with the latest changes from main. Weird.
Closing issue.


Update txt files and small code improvements

Move ~150 working tests from NotYetReady TXT files to regular TXT files
Remove duplicates
Small code improvements

✅ No public API change.


Fix multiple compiling errors on win10x64+vs2022-amd64

Change ThemeEditor.cpp PrimitiveUtils.cpp SanboxWindow.cpp file encoding method to utf8-with-bom.
Remove duplicated QlementineStyle::groupBoxBackgroundColor/QlementineStyle::groupBoxBorderColor.
Explicitly convert size_t to int since vc++ treats C4267 as an error under warning_level_4.

Since all the major issues were addressed in the last commit in a different way, this PR should be withdrawn.


Grow and Shrink using the API

Trying to figure out how to growAndShrink the cluster using the API. I am guessing it is using ClusterAPI PUT, but I can find no documentation around the payload.
I just want to be able to reduce/increase a certain type of instance groups.

Hi,
If you do a GET of the cluster endpoint, then you'll get the current data that Director is operating on. You can then modify the count of the instance group that you want to grow/shrink and provide that as the payload for PUT. You can use the API console to experiment with this. To access the API console, click on the ? symbol in the upper right corner of the Director UI.
Hope this helps.
Vinithra.

I see instances but don't see instance group in the JSON. Am I missing something?

@ujjwal Director is using a declarative approach. To grow a cluster via the API what you need to do is to retrieve the current cluster template, mutate the object to include additional virtual instances (with random UUID as identifiers) and send it back as an update request.
See the following file for more details on how the template was constructed:
https://github.com/cloudera/director-sdk/blob/master/python-client-samples/cluster.py


go1.21 -> go1.20

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Patch coverage has no change and project coverage change: -13.60% :warning:

Comparison is base (328af5f) 66.36% compared to head (a2ad84c) 52.77%.
Report is 11 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #818       +/-   ##
===========================================
- Coverage   66.36%   52.77%   -13.60%
===========================================
  Files         236      229        -7
  Lines       23153    22607      -546
===========================================
- Hits        15366    11931     -3435
- Misses       6982     9793     +2811
- Partials      805      883       +78

Flag
Coverage Δ

52.77% <ø> (-0.94%)
:arrow_down:

go-1.17.x
52.77% <ø> (-13.46%)
:arrow_down:

go-1.20.x
?

go-1.21.x
?

integration
52.77% <ø> (-0.94%)
:arrow_down:

macOS
?

ubuntu
?

unit
?

windows
?

ydb-22.5
?

ydb-23.1
?

ydb-23.2
52.77% <ø> (-0.73%)
:arrow_down:

ydb-23.2-slim
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

internal/xstring/convert.go
20.00% <ø> (-80.00%)
:arrow_down:

... and 101 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


include directories are installed to /usr/include/include

In the refactor branch wlc.h ends up in /usr/include/include/wlc/wlc.h. Is that expected? One include should be enough :) (I don't know cmake so I can't fix it myself.)

Fixed.

That was quick, thanks a lot. Working perfectly.


Add ufw allow for REST API

Just helped a user debug hooking up their raspiblitz to my chrome extension Joule (https://github.com/wbobeirne/joule-extension) which hit a snag because that extension has to use the REST API rather than the gRPC API due to browser limitations. I've added a ufw allow for 8080 to open it up. I personally haven't been able to test this, but I assume this is enough to make it accessible. Let me know if I'm wrong!

Thas a good idea. Thanks.


feat(network): mixed beahviour event

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1896  👈
#1786
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nagmo-starkware and the rest of your teammates on  Graphite


[DOCS] Minor adjustments

Minor adjustments
Just a routine job, typos and broken links fixes

can you make your branch up to date with main ? thanks

@LelouchFR rebased


[Logging] adding security logging of byzantine votes

Motivation
This adds a way to detect votes that display byzantine behavior, as was first attempted in #4997.
If two non-NIL block end up voting for two different consensus_data_hash at the same round, then it means that either:

the proposer sent different blocks to different voters
voters executed the same block differently

To detect this, we simply add a cache in pending_votes to save non-NIL votes by consensus_data_hash.
But this also require adding logic to differentiate NIL and non-NIL votes at the vote_data level.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Ideally this could be tested using Twin tests to have it trigger the logs, I guess. But I need to investigate this possibility further.

Feel free to re-open if you'd like to resume.


fix-#229:  Added Github Actions: Give a link to contributing guidelines to the assigne as soon as issue is assigned to someone

Summary
Briefly describe the purpose of this PR.
Description
Explain in detail what this PR is all about. This should include the problem you're solving, the approach taken, and any technical details that reviewers need to know.
Images
Include any relevant images or diagrams that can help reviewers visualize the changes, if applicable
Issue(s) Addressed
Enter the issue number of the bug(s) that this PR fixes

Template should be strictly Closes <issue_number>
Example: Closes #1

Prerequisites

[ ] Have you followed all the CONTRIBUTING GUIDELINES?

Kindly work on the PR description and the issue is about contributing guidelines, you haven't followed any 😅😂
Please go through it and get back , if there is any confusion let me know . https://github.com/krishnaacharyaa/wanderlust/blob/e341e913f1dde592d1819969609548abba9633ae/.github/CONTRIBUTING.md#guidelines-for-contributions

@Niranthari

creating a proper one


Cannot restore variables with Checkpoint because keys do not align

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Win10
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): tf-nightly==1.10.0.dev20180609
Python version: 3.6.5
Bazel version (if compiling from source): -
GCC/Compiler version (if compiling from source): -
CUDA/cuDNN version: -
GPU model and memory: -
Exact command to reproduce:

I get the error that is thrown here:
https://github.com/tensorflow/tensorflow/blob/r1.9/tensorflow/python/training/checkpointable/util.py#L633
I cannot provide code to reproduce it. But basically what happens is I have a class that inherits from Checkpointable. It assigns all variables to itself to make them checkpointable. It also assigns an optimizer to itself. I then save the model and restore it. When calling .assert_consumed() on the load_status object of restore(dir, session) it throws an error because some key does not match. The variable it tries to restore is actually in the saved checkpoint it just has a different key then the one it gets from enumerate(self._checkpoint.object_graph_proto.nodes).
This describes it as good as I can. Sorry for not being able to share the code. I tried to reproduce it but so far I cannot.
Output of self._checkpoint.object_by_proto_id.keys() at line 631.
You can see that some keys are missing (idk why) but theses are the ones I need to restore.
[0, 1, 2, 3, 4, 5, 6, 7, 14, 19, 10, 17, 22, 11, 18, 23, 12, 13]

Output of util._serialize_object_graph(self.checkpoint, None) after restore before assert_consumed
nodes {
  children {
    node_id: 1
    local_name: "model"
  }
  children {
    node_id: 2
    local_name: "save_counter"
  }
}
nodes {
  children {
    node_id: 3
    local_name: "_global_step_pretrain"
  }
  children {
    node_id: 4
    local_name: "embedding"
  }
  children {
    node_id: 5
    local_name: "cell"
  }
  children {
    node_id: 6
    local_name: "dense"
  }
  children {
    node_id: 7
    local_name: "_optimizer"
  }
  children {
    local_name: "_checkpoint"
  }
}
nodes {
  attributes {
    name: "VARIABLE_VALUE"
    full_name: "initialize_or_restore/save_counter"
    checkpoint_key: "save_counter/.ATTRIBUTES/VARIABLE_VALUE"
  }
}
nodes {
  attributes {
    name: "VARIABLE_VALUE"
    full_name: "ConditionedLSTMGenerator2/CONDITIONED_LSTM/pretrain/global_step"
    checkpoint_key: "model/_global_step_pretrain/.ATTRIBUTES/VARIABLE_VALUE"
  }
}
nodes {
  attributes {
    name: "VARIABLE_VALUE"
    full_name: "ConditionedLSTMGenerator2/CONDITIONED_LSTM/embedding"
    checkpoint_key: "model/embedding/.ATTRIBUTES/VARIABLE_VALUE"
  }
}
nodes {
  children {
    node_id: 8
    local_name: "kernel"
  }
  children {
    node_id: 9
    local_name: "bias"
  }
  attributes {
    name: "OBJECT_CONFIG_JSON"
    checkpoint_key: "model/cell/.ATTRIBUTES/OBJECT_CONFIG_JSON"
  }
}
nodes {
  children {
    node_id: 10
    local_name: "kernel"
  }
  children {
    node_id: 11
    local_name: "bias"
  }
  attributes {
    name: "OBJECT_CONFIG_JSON"
    checkpoint_key: "model/dense/.ATTRIBUTES/OBJECT_CONFIG_JSON"
  }
}
nodes {
  children {
    node_id: 12
    local_name: "beta1_power"
  }
  children {
    node_id: 13
    local_name: "beta2_power"
  }
  slot_variables {
    original_variable_node_id: 4
    slot_name: "m"
    slot_variable_node_id: 14
  }
  slot_variables {
    original_variable_node_id: 8
    slot_name: "m"
    slot_variable_node_id: 15
  }
  slot_variables {
    original_variable_node_id: 9
    slot_name: "m"
    slot_variable_node_id: 16
  }
  slot_variables {
    original_variable_node_id: 10
    slot_name: "m"
    slot_variable_node_id: 17
  }
  slot_variables {
    original_variable_node_id: 11
    slot_name: "m"
    slot_variable_node_id: 18
  }
  slot_variables {
    original_variable_node_id: 4
    slot_name: "v"
    slot_variable_node_id: 19
  }
  slot_variables {
    original_variable_node_id: 8
    slot_name: "v"
    slot_variable_node_id: 20
  }
  slot_variables {
    original_variable_node_id: 9
    slot_name: "v"
    slot_variable_node_id: 21
  }
  slot_variables {
    original_variable_node_id: 10
    slot_name: "v"
    slot_variable_node_id: 22
  }
  slot_variables {
    original_variable_node_id: 11
    slot_name: "v"
    slot_variable_node_id: 23
  }
}

I assume this is something for @allenlavoie ?

I found the culprit. Double assigning a variable
import tensorflow as tf
from tensorflow.python.training.checkpointable import base as checkpointable
from tensorflow.python.training.checkpointable import util

class Model(checkpointable.Checkpointable):

  def __init__(self):
    self.cell = tf.nn.rnn_cell.BasicLSTMCell(4)
    self.cell = tf.nn.rnn_cell.BasicLSTMCell(4)
    # self.cell = tf.nn.rnn_cell.BasicLSTMCell(4)
    out = self.cell(tf.constant([[1.]]), self.cell.zero_state(1, tf.float32))
    self.optimizer = tf.train.AdamOptimizer()
    self.optimizer.minimize(tf.reduce_sum(out[0]))
    self.session = tf.Session()
    self.checkpoint = tf.train.Checkpoint(model=self)

  def init(self):
    print('Init')
    self.session.run(tf.global_variables_initializer())

  def save(self):
    print('Save')
    self.checkpoint.save('./tmp/', self.session)

  def restore(self):
    print('Restore')
    latest = tf.train.latest_checkpoint('./tmp/')
    load_status = self.checkpoint.restore(latest)
    print(util._serialize_object_graph(self.checkpoint, None))
    load_status.assert_consumed().run_restore_ops(self.session)

  def print(self):
    print(self.session.run(self.cell._kernel))

m = Model()
m.init()
m.print()
m.save()
m.restore()
m.print()

It works if I double assign a tf.get_variable though.

Happy to help


Small fixes for Units 31, 32 and 33

Should this  be backported to master (plone 4) branch as well?

Currently I'm working on plone 5, and I have not kept track of the differences with master, so I don't know the context of everything that changed when the 5 training got created.  I'll finish my pass through plone 5 first, and then see what to do about 4.


ConversationalRetrievalChain doesn't work along with memory and return_source_documents

Issue you'd like to raise.
I'm trying to use a ConversationalRetrievalChain along with a ConversationBufferMemory and return_source_documents set to True. The problem is that, under this setting, I get an error when I call the overall chain.
from langchain.chains import ConversationalRetrievalChain

chain = ConversationalRetrievalChain(
    retriever=retriever,
    question_generator=question_generator,
    combine_docs_chain=doc_chain,
    memory = memory,
    return_source_documents=True,
)

query = 'dummy query'
chain({"question": query})

The error message says:
File [~/.local/share/virtualenvs/qa-8mHXn5ez/lib/python3.10/site-packages/langchain/chains/base.py:354](https://vscode-remote+wsl-002bubuntu.vscode-resource.vscode-cdn.net/home/my_user/~/.local/share/virtualenvs/env-8mHXn5ez/lib/python3.10/site-packages/langchain/chains/base.py:354), in Chain.prep_outputs(self, inputs, outputs, return_only_outputs)
    352 self._validate_outputs(outputs)
    353 if self.memory is not None:
--> 354     self.memory.save_context(inputs, outputs)
...
---> 28         raise ValueError(f"One output key expected, got {outputs.keys()}")
     29     output_key = list(outputs.keys())[0]
     30 else:

ValueError: One output key expected, got dict_keys(['answer', 'source_documents'])

It works if I remove either the memory or the return_source_documents parameter.
So far, the only workaround that I found out is querying the chain using an external chat history, like this:
chain({"question": query, "chat_history":"dummy chat history"})
Thank you in advance for your help.

Same issue, please help

Nvm I found the fix, you need to define output_key='answer' inside ConversationBufferMemory
ConversationBufferMemory(memory_key="chat_history", output_key='answer', return_messages=True)

output_key='answer'

Great, now it's working. Thanks!

I tried this with output_key='answer but get the same error
qa_chain = ConversationalRetrievalChain.from_llm(llm=llm,
                                   chain_type="stuff",
                                   retriever=set_retriver(retriever),
                                   memory=memory,
                                   output_key='answer',
                                   return_source_documents=True,
                                   verbose=True)

ValueError: One output key expected, got dict_keys(['answer', 'source_documents'])
When I try to remove the return_source_documents=True setting, the chain works but does not refer to the chat history when answering the follow up question.
Any insights or ETA for when this can be resolved ?

Same error for the following code
prompt
def get_prompt():
    template = """Use the following pieces of context to answer the question at the end.
    ---
    CONTEXT: {context}
    QUESTION: {question}
    FINAL ANSWER: {answer}
    SOURCES: {sources}"""
    qa_prompt = PromptTemplate(
        template=template, input_variables=[
            'question', 'context', 'answer', 'sources'
        ],
    )
    return qa_prompt

memory
memory = ConversationBufferMemory(
        memory_key='chat_history', return_messages=True,
        input_key='question', output_key='answer'
    )

chain
model = ConversationalRetrievalChain.from_llm(
    llm=llm,
    retriever=retriever,
    memory=memory,
    return_source_documents=True,
    verbose=True,
    chain_type="stuff",
    output_key='answer',
    combine_docs_chain_kwargs={'prompt': prompt},
)

error
An error occurred: Missing some input keys: {'sources', 'answer'}


Fixed invalid variable name in generic host snippet

Snippet ConfigureAppConfiguration in the generic host fundamental page was referencing an invalid variable name in the context of the scope. Updated to use the correct variable name.

@roketworks Nice catch! Thank you for the PR.


Fix metric counting incorrect Content-Length headers

[x] Read the Contributing document.

Summary

Go 1.22 introduced stricter checks for Content-Length headers. The header, if set, must not be empty.
This new strict check can be worked around with debug.httplaxcontentlength, which will allow empty Content-Length headers. In preparation for users disabling this debug flag that Gorouter turns on by default, a metric is emitted to collect requests that would break when the workaround is disabled.
The metric introduced in https://github.com/cloudfoundry/gorouter/pull/412 was erroneous, see https://github.com/cloudfoundry/routing-release/issues/411.
Backward Compatibility
Breaking Change? Yes(ish)

The metric that was recently introduced, called missing_content_length_header is misleadingly named and renamed in this PR to empty_content_length_header instead.

@mariash, @MarcPaquette, maybe you can have a look as well.


Fix #7350 Expandable Html Attributes

Add new static property expandableAttributes which will be html attributes that will be allowed to receive array values and will expand them

#7350

Closed via https://github.com/yiisoft/yii2/commit/a3ea7b774d0cecf09776755c04f6703a71b22ec4
Thanks!


test: streamline the fix infra for config tests

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#127  👈
#126
#113 : 1 other dependent PR (#114 )
#111
#110
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ArniStarkware and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 44.05%.  Consider uploading reports for the commit 1687911 to get more accurate results

Additional details and impacted files
@@                                Coverage Diff                                @@
##           arni/config/stateless_validator/create_serialize     #127   +/-   ##
=================================================================================
  Coverage                                             44.05%   44.05%
=================================================================================
  Files                                                    16       16
  Lines                                                   597      597
  Branches                                                597      597
=================================================================================
  Hits                                                    263      263
  Misses                                                  325      325
  Partials                                                  9        9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

crates/gateway/src/config_test.rs line 15 at r1 (raw file):
Previously, ArniStarkware (Arnon Hod) wrote…
Maybe just? @giladchase

No longer relevant.


Enable module stability support

In order to support module stability feature, I switched Chatto & ChattoAdditions to Swift 5 source compatibility mode and enabled BUILD_LIBRARY_FOR_DISTRIBUTION option for release configurations.

Codecov Report

Merging #625 into master will decrease coverage by 0.03%.
The diff coverage is 74.07%.

@@            Coverage Diff             @@
##           master     #625      +/-   ##
==========================================
- Coverage   60.73%   60.69%   -0.04%
==========================================
  Files          92       92
  Lines        4892     4898       +6
==========================================
+ Hits         2971     2973       +2
- Misses       1921     1925       +4

Impacted Files
Coverage Δ

...BaseChatViewController+AccessoryViewRevealer.swift
0% <ø> (ø)
:arrow_up:

.../Chat Items/BaseMessage/BaseMessageViewModel.swift
89.65% <100%> (ø)
:arrow_up:

ChattoAdditions/Source/Common/AnimationUtils.swift
100% <100%> (ø)
:arrow_up:

...ws/BaseMessageCollectionViewCellDefaultStyle.swift
96.47% <100%> (ø)
:arrow_up:

...ditions/Source/Common/UIEdgeInsets+Additions.swift
100% <100%> (ø)
:arrow_up:

...s/PhotoMessageCollectionViewCellDefaultStyle.swift
84.09% <100%> (ø)
:arrow_up:

...ttoAdditions/Source/Common/UIColor+Additions.swift
100% <100%> (ø)
:arrow_up:

...ws/TextMessageCollectionViewCellDefaultStyle.swift
91.78% <100%> (ø)
:arrow_up:

.../Input/Photos/Camera/LiveCameraCellPresenter.swift
80.41% <33.33%> (-1.51%)
:arrow_down:

...ttoAdditions/Source/Common/UIImage+Additions.swift
30.3% <33.33%> (ø)
:arrow_up:

... and 2 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 6cb0868...3ca6ff7. Read the comment docs.


"invalid_request" when sending opaque access token to Introspect endpoint

Background
It would be super useful to have a full example and or guide to setting up spring authorisation server for use with opaque tokens.
I am currently working on migrating my existing app (Old spring auth server using opaque tokens with a custom PKCE Implementation) to this project and would be happy to write up an example / how-to guide after I have it working.
So far (through a little trial and error) I have gotten to the point where my SPA directs users to login and auth server directs them back with the code. My SPA then exchanges the code for an opaque access token which seems to work as expected.
The part I am currently stuck on is the Introspect endpoint.
Whenever I make an introspect request to the auth server (/oauth2/introspect) I am getting the error response shown below.
{
    "error": "invalid_request"
}

The annoying part of this is I can not see in the logs any reason for this and I can see that the token does exists in my DB.
Questions:
Is there any extra configuration needed for the Introspect Endpoint that isn't enabled out of the box?
Does anybody have a working example they can share?
Does anybody have any idea where I am going wrong?
Thanks,
Carl

@CarlRobinsonRAM

Whenever I make an introspect request to the auth server (/oauth2/introspect) I am getting the error

The Introspection Endpoint requires client authentication so this might be the issue? See Configuring Client Authentication for more details.

The annoying part of this is I can not see in the logs any reason for this

You can enable TRACE logging and then you will see where the issue is. Logging was added in gh-159.
I'm going to close this as these are questions that are better suited to Stack Overflow. We prefer to use GitHub issues only for bugs and enhancements.
However, if you feel there is something specific missing in the documentation or you would like to propose a How-to guide, please log a new issue. Thank you.


Link in /parallel-computing leads to wrong destination

Describe your problem and how to reproduce it:
Link referenced by "Help our community expand it" in /computer-science/parallel-computing leads to /computer-science/databases/index.md instead of /computer-science/parallel-computing/index.md
Add a Link to the page with the problem:
https://guide.freecodecamp.org/computer-science/parallel-computing
Tell us about your browser and operating system:

Browser Name: UC browser
Browser Version: 7.0
Operating System: windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

Hi @juniorbuba Thanks for reporting this.  This issue has already been resolved but not moved to production.  You can visit are beta site which shows this guide article is no longer a stub, so the link no longer appears anyway.  The beta site shows all the current changes available on the master branch, but may not have been shipped to production.
I am closing this issue for now.  If you have any questions, let me know.


Add diskSize, repositoryId, imageFullTag settings, and several fixes

Hi zhazha, I would like to add this PR to your excellent project, thank you!

Hi @Senorsen @reAsOn2010 . Would you like to take the owner of this project to pintia organization?

Yes of course!


Release v2.4.0

@c-rack Since this is a major problem for many users, I'd like to go forward with this one. Can I get your approval soon? :)


No fold found Error

When restoring session, the below error happens time to time. Not always.
E5108: Error executing lua ...ck/packer/opt/possession.nvim/lua/possession/session.lua:176: Vim(normal):E490: No fold found
stack traceback:
	[C]: in function 'nvim_exec'
	...ck/packer/opt/possession.nvim/lua/possession/session.lua:176: in function 'fn'
	...possession.nvim/lua/telescope/_extensions/possession.lua:77: in function 'run_replace_or_original'
	...k/packer/opt/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'
	...ack/packer/opt/telescope.nvim/lua/telescope/mappings.lua:341: in function 'execute_keymap'
	[string ":lua"]:1: in main chunk

My session options
vim.opt.sessionoptions =
	{ 'buffers', 'curdir', 'folds', 'help', 'tabpages', 'globals', 'winsize', 'winpos', 'terminal' }

I tried to reproduce the issue but could not. I don't think it's related to this plugin specifically, I would rather say it might be related to :mksession and your folding configuration. It seems like something is trying to open a fold that is no longer valid (e.g. lines in the file have been deleted). Are you creating folds manually using zf? If not then I would suggest removing folds from sessionoptions, maybe this would help.

It mostly occurs when loading session using :Telescope possession list or :PossessionLoad mysession, but loading the same session using :PossessionLoad without any session name (load last session) the error does not occur.

Fyi I was seeing the same issue and removing folds from vim.opt.sessionoptions seems to have solved it for me 👍

Thanks for info, I think I can close this issue.

For anyone still experiencing this issue after removing folds try adding:
vim.api.nvim_create_autocmd({ "BufEnter" }, {
    pattern = { "*" },
    command = "normal zx",
})

This worked for me.
From this issue on Telescope


Add Scribus

add new document "Scribus"
add link to official website for scribus
add short description for scribus
add contribute link + shieldimage to "https://www.scribus.net/contribute/"

Do not use this! it is not right
i made a mistake with the comparison. will fix and make a new pull request


Redirection not working: 504 Gateway Time-out

I have set up a test container in a public VPC listening on http://<IP_ADDRESS>:32769 . I can access the container directly using this url in the port indicated. I have then set up the Proxy Host in NginxProxyManager as follows:
Domain Names: ifconfig.mydomain.com
Scheme: http
Forward IP: <IP_ADDRESS>
Forward Port: 32769

I don't need https access for this test for the time being so I left he SSL tab blank: None. Navigating to "http://ifconfig.mydomain.com" does show up in the container's nginx proxy log, but the there is no sign of actually forwarding the request in the log tail -f /config/log/proxy_host-4.log:
[27/Mar/2021:13:19:40 +0000] - 504 504 - GET http ifconfig.mydomain.com "/" [Client <IP_ADDRESS>] [Length 164] [Gzip -] [Sent-to <IP_ADDRESS>] "curl/7.58.0" "-"
I was testing this operation with curl (to disregard fat browsers' attempts to silently redirect http to https).
... and neither is the forwarding of packets to port80 or 443 showing up in tcpdump -n -i eth0 'port 32769 || port 80 || port 443' in the container's hosting VPC (no trace of the port 32769 packets). IP addresses and domain names are edited to protect the innocent...
Does anyone have any pointers for how to troubleshoot? I suspect the issue might be with the nginx config

Thanks for answering! Yes, this is what's puzzling for me. From the same docker host I can access the target container (using curl http://<IP_ADDRESS>:32769) but not sure why nginx from the container is unable. This is the same for multiple containerised web apps. When "tcpdump" ing on the host I can see the request coming in the to the NPM app but tit is not forwarded to the target (no further tcp packe seen). How can I figure out why is this? I tried setting the docker network to be the standard bridge network or various container specific "bridges" (with inter-porocess-communicaton enabled). None helped and am a bit stuck. Do you have any suggestion for what to look for?

So is the same host running both NPM and the target container ?

Yes, in this case, they are both on the same host. But this host is also Internet accessible as well. I tried to use the default bridge network (Internet accessible) and the private bridge (also internet accessible). Tried to refer to the target container by the external (public) IP address and the socket "file". Neither worked: I seem to be able to hit nginx but it doesn't forward (reverse proxy) traffic to the intended destination.

To keep things simple, let's configure both containers to use the default bridge network, with proper port mappings.
In this scenario, you should set in the proxy host the IP address of the host itself, with the port of the target container.
Also, is the host also has a private/LAN IP address, or only a public one ?

I have changed the docker network used for both (NPM/nginx-proxy-manager and a test container called ifconfig) to the standard "bridge" network. Both containers get a private IP address on this network, and I can ping from the NPM container all private IP addresses.
KEY/Explanation:
redey@npm:             is my username on the docker host computer (SAAS in the Cloud)
redey@mylaptop:     is my username on my laptop
Here are the diagnostics I attempted:
#1 on docker host VPS:
There are 3 containers connected to the standard "bridge" network and all pingable from the NPM container:
docker ps | grep -E 'NAMES|nginx|ifconfig'
CONTAINER ID        IMAGE                                  COMMAND                CREATED             STATUS              PORTS                                                               NAMES
3c8fa1d95118        jlesage/nginx-proxy-manager:latest     "/init"                3 weeks ago         Up 45 minutes       <IP_ADDRESS>:443->4443/tcp, <IP_ADDRESS>:80->8080/tcp, <IP_ADDRESS>:81->8181/tcp   nginx-proxy-manager
0e8f5b78e79e        mpolden/echoip:latest                  "/opt/echoip/echoip"   4 weeks ago         Up 52 minutes       <IP_ADDRESS>:32769->8080/tcp                                             ifconfig

redey@npm:~$ docker network inspect bridge | grep -E 'Containers|Name|Subnet|IPv4Address'
        "Name": "bridge",
                    "Subnet": "<IP_ADDRESS>/16",
        "Containers": {
                "Name": "ifconfig",
                "IPv4Address": "<IP_ADDRESS>/16",
                "Name": "nginx-proxy-manager",
                "IPv4Address": "<IP_ADDRESS>/16",

redey@npm:~$ for i in 5 6 ; do docker exec -it 3c8fa1d95118 ping -c 1 172.17.0.$i ; done | grep -E '172.17|received'
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.060 ms
--- <IP_ADDRESS> ping statistics ---
1 packets transmitted, 1 packets received, 0% packet loss
PING <IP_ADDRESS> (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: seq=0 ttl=64 time=0.100 ms
--- <IP_ADDRESS> ping statistics ---
1 packets transmitted, 1 packets received, 0% packet loss

#2 NPM proxy configuration http://<IP_ADDRESS>:81/nginx/proxy (this is the web interface):
SOURCE                   DESTINATION                 SSL             ACCESS    STATUS
ifconfig.mydomain.com    http://<IP_ADDRESS>:32769    Let's Encrypt   Public    Online
npm.mydomain.com         http://<IP_ADDRESS>:81       Let's Encrypt   Public    Online

#3 Web access (with curl) of both containers from the NPM host computer and from my laptop via the Internet:
redey@npm:~$ curl http://<IP_ADDRESS>:32769
<IP_ADDRESS>

redey@npm:~$ curl <IP_ADDRESS>:81
<!doctype html>

    <meta charset="utf-8">
<snip>...</snip>

#4 but trying to access the reverse proxy from the Internet (my laptop) it fails:
##ifconfig.mydomain.com:
redey@mylaptop:~$ curl ifconfig.mydomain.com
^C
redey@mylaptop:~$ curl https://ifconfig.mydomain.com
curl: (60) SSL certificate problem: self signed certificate
More details here: https://curl.haxx.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

##npm.mydomain.com
redey@mylaptop:~$ curl npm.mydomain.com

<title>301 Moved Permanently</title>

301 Moved Permanently

openresty

redey@mylaptop:~$ curl https://npm.mydomain.com
^C

In summary: same result. The Internet connected host/browser can send a packet to the NPM host but it is not "proxied" to the target web server...

Massive facepalm situation: the docker host was running a host based firewall (ufw), for some reason (well for some earlier experimenting). I was going to run curl (not installed), telnet (not installed) then nc from within the NPM container, unsuccessfully. This gave me to prompt to check if iptables was enabled. Rest is history, many thanks for looking into this!!!


Add disableToc parameter to not show TOC on a page

disableToc: true can be set in page frontmatter to not show the table of contents for that page.

I'd like to suggest using enableToc: false to achieve this same result. It seems more intuitive.
What do you guys think?
Talking about myself:
When I saw enableToc: true in data/config.yaml, I intuitively thought that using enableToc: false in a page's frontmatter would be the way to disable ToC for that page.
I had to look into the partials code to figure out that it's done via disableToc: true.

This makes sense! Not super urgent for me right now but feel free to open a PR for this if you so wish :))


wt.exe split-pane doesn't work in quake mode

Windows Terminal version
1.15.1863.0
Windows build number
10.0.22000.739
Other Software
No response
Steps to reproduce
Open a quake terminal and execute wt.exe -w 0 split-pane
Expected Behavior
The quake terminal to split
Actual Behavior
If a non-quake terminal is already open it is split. Otherwise a new non-quake terminal is opened and immediately split

So, because the quake window is "special" (it can be hidden in a way that normal windows can't), it's excluded from the list of recent windows that -w 0 consults.
To split the quake window, you may want to target it specifically: -w _quake will do it.

Is there any way to detect whether you're in a quake window programatically?

... frick. No. Excellent question.

Huh. Yea maybe that was an oversight when doing #9956. Maybe -w 0 should still work for quake, but "Attach to the most recently used window" should not. Not sure if there's a good way to differentiate between the two, however.

discussion: Yep, we should totally pull these two code paths apart.


refactor: make the pin-input component headless

Closes #1889
Description
Currently, the @yamada-ui/pin-input component has minimal styles set within the component's source code.
We want to make this style transferable to the theme and make the component headless, enabling more flexible UI customization.
Is this a breaking change (Yes/No):
No

Thanks!


[core] core_basic_window.c maxes out one cpu core

Is this because of the SUPPORT_BUSY_WAIT_LOOP define? Can someone explain why this is the default?

Related issue https://github.com/raysan5/raylib/issues/415
It's the default because is the safest and most accurate way in most platforms instead of dealing with system interrupts... but so the define, in case someone needs to change that behaviour. By the way, a 100% CPU usage doesn't mean that CPU can run other 1000 processes smoothly at the same time... lots of games work this way.

Idk, why I didn't find that related issue, thanks!


Situational Bestie

Enable the ability to select any two PAX and see how many BD's they have attended together.

Maybe this would be a better fit for backblasts page and for building customer reports when you pick pax and other filters


Bug with deleting line of code after make zz or z. commands

Description
After making zz on line in the bottom of code navigation screen and next removing this line, your screen is moving up on just about a half of general number of lines, covered your screen.
Also, i can reproduce this bug only in files, where the number of lines of code is more, that can fully fit on screen.
Operation
Pressed on zz or z. combinations. Then delete line by dd keymap. In moment, when you press on d second time, your screen with code move up, approximately, on the virtual center of code navigation screen (as the same, if you will press zz on central line before first step).
Expected behaviour
Line removing only (without screen moving up).
Environments

Xcode Version [ 7.3.1 ]
XVim branch and revision [ master ]
Keyboard Language [ English ]
Input source language []

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/34202098-bug-with-deleting-line-of-code-after-make-zz-or-z-commands?utm_campaign=plugin&utm_content=tracker%2F252770&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F252770&utm_medium=issues&utm_source=github).

Actually, the same behaviour after o or O commands.


do not hardcode six version

I'm closing this since #95 has been merged. Thanks for the work though!


Ignore arm-linux-androideabi on bors

r? @ghost
Signed-off-by: Yuki Okushi<EMAIL_ADDRESS>

@bors r+

:pushpin: Commit c04ecd499de3c084579045e4278830b8c0213292 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit c04ecd499de3c084579045e4278830b8c0213292 with merge 80b2ede6770c52621f9c9f5488a6385f39f0e30f...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 80b2ede6770c52621f9c9f5488a6385f39f0e30f to master...


toast.custom leave animation does not play, and padding error

I can make this custom toast, and the "enter" animation plays, but the "leave" animation does not.  When I click the button on the toast, there is a short delay and then the toast disappears without the animation.
I have confirmed that the animations work, by using them on divs in a page.
The second problem I have is that if I specify p-5 for padding, the padding becomes 0.  However, p-4 works.
Setup:
"postcss": "^8.4.5",
"tailwindcss": "^3.0.7",
"react-hot-toast": "^2.2.0",
"next": "latest",

my code to launch a toast
return toast.custom((t) => (

<XCircleIcon className="h-8 w-8 text-white" aria-hidden="true" />

          {message}

      <button
        onClick={() => toast.dismiss(t.id)}
        className="w-full border border-transparent rounded-none rounded-r-lg p-4 flex items-center justify-center text-white font-medium focus:outline-none focus:ring-0"
      >
        Close
      </button>

), {
  duration:Infinity
})

tailwind config
module.exports = {
  content: [
    './pages/**/*.{js,ts,jsx,tsx}',
    './components/**/*.{js,ts,jsx,tsx}',
  ],
  theme: {
    extend: {
      boxShadow: {
        't-sm': '0 -1px 2px 0 rgba(0, 0, 0, 0.05)',
        't-md': '0 -4px 6px -1px rgba(0, 0, 0, 0.1)',
        't-lg': '0 -10px 15px -3px rgba(0, 0, 0, 0.1), 0 4px 6px -2px rgba(0, 0, 0, 0.05)',
        't-xl': '0 -20px 25px -5px rgba(0, 0, 0, 0.1), 0 10px 10px -5px rgba(0, 0, 0, 0.04)',
        't-2xl': '0 -25px 50px -12px rgba(0, 0, 0, 0.25)',
        't-3xl': '0 -35px 60px -15px rgba(0, 0, 0, 0.3)',
      },
      animation: {
        enter: 'enter 200ms ease-out',
        leave: 'leave 200ms ease-in',
      },
      keyframes: {
        enter: {
          '0%': { transform: 'scale(0.9)', opacity: 0 },
          '100%': { transform: 'scale(1)', opacity: 1 },
        },
        leave: {
          '0%': { transform: 'scale(1)', opacity: 1 },
          '100%': { transform: 'scale(0.9)', opacity: 0 },
        },
        'slide-in': {
          '0%': { transform: 'translateY(-100%)' },
          '100%': { transform: 'translateY(0)' },
        },
      },
    },
  },
  plugins: [
    require('@tailwindcss/forms'),
    require('autoprefixer'),
  ],
}

I guess it's something to do with Tailwind JIT plus toasts rendering. Try to add "animate-enter" somewhere else on the page (in addition to toast itself) and see what happens. Where "code to launch a toast" is located, in the components folder?

Hi.  I tried adding forwards to both, but it did not help.  The problem is not the jumping, but that the leave animation doesn't play at all.  Is there a reason my react toast is not behaving the same as the demo code?
There is also the padding problem.  Suddenly it resolved itself, i dont know why.

I guess it's something to do with Tailwind JIT plus toasts rendering. Try to add "animate-enter" somewhere else on the page (in addition to toast itself) and see what happens. Where "code to launch a toast" is located, in the components folder?

yeah, i tried that.  animate-enter and animate-leave function without issue when used outside of a toast

`

I guess it's something to do with Tailwind JIT plus toasts rendering. Try to add "animate-enter" somewhere else on the page (in addition to toast itself) and see what happens. Where "code to launch a toast" is located, in the components folder?

yeah, i tried that. animate-enter and animate-leave function without issue when used outside of a toast

Could you reproduce that issue in a sandbox? I've just made this leave-enter animation in my current project and it works without the issues you mentioned. In my case, it was I tried to create the component in /lib folder, out of the default
content: [
    './pages/**/*.{js,ts,jsx,tsx}',
    './components/**/*.{js,ts,jsx,tsx}',
  ],

the animation isnt the problem.  because i think i took it off the github repo for hot toast examples. it must be with my config or something else is interfering.
you brought up an interesting point.  what is the "content" section in tailwind config?  i also call toast.custom from a file inside ./lib
i have an idea what it is--im guessing i need to add
content: [
    './pages/**/*.{js,ts,jsx,tsx}',
    './components/**/*.{js,ts,jsx,tsx}',
    './lib/**/*.{js,ts,jsx,tsx}',
  ],

@timolins I think this one is resolved.


Update data loading to use datetime object representations, rather than epoch time.

When loading data (mainly model data), wwe use iris constraints to only load a particular time for a particular field. The times are converted from the number in the netcdf metadata, which is hours since 1970, to datetime objects which are used with iris constraints. This is done using datetime.datetime.fromtimestamp, which is not the correct way of doing this.
Instead, we should not be using explicit conversions at all, but rather than using my_cube.coord('time').points, we should be accessing the data times available using my_cube.coord('time').cells, where the conversion to a datetime object is done by iris behind the scenes.

No longer relevant to latest implementation of FOREST.


[SMALLFIX] make masks auto update, and make permission checking respect ACL mask

Before this change, we were ignoring ACL masks. After this change, we will take ACL masks into consideration when we do permission checking. Also we automatically sets masks when updating other relevant entries.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/28/
Test FAILed.

@gpang PTAL.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/29/
Test FAILed.

jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/31/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/32/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/57/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/58/
Test FAILed.

Merged build finished. Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/63/
Test PASSed.

@gpang addressed comments, PTAL

@gpang comments addressed. PTAL.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/78/
Test PASSed.


fix doc link

Motivation
Fix doc link
Modification
fix doc link in README.md and configs/seesaw_loss/README.md.
BC-breaking (Optional)
Use cases (Optional)
Checklist

 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.


updated s3cmd kb with tips to improve speed

Added improve speed section with --no-progress option

Please check links referenced in the details of the failed check.

@gavin-lai links appear to still be failing. Pls fix.

Hi Rich,
Can you please cancel this change?  I messed up on how I am branching locally.
Thanks
Gavin

Gavin Lai | Sr. Cloud Solutions Architect
CenturyLink Cloud
Phone:<PHONE_NUMBER>
Email<EMAIL_ADDRESS>www.ctl.iohttp://www.centurylinkcloud.com/
[Description: Machine generated alternative text: CenturyLink]http://www.ctl.io/
From: Rich DuBose<EMAIL_ADDRESS>Sent: Thursday, October 15, 2015 1:54 PM
To: CenturyLinkCloud/PublicKB<EMAIL_ADDRESS>Cc: Gavin Lai<EMAIL_ADDRESS>Subject: Re: [PublicKB] updated s3cmd kb with tips to improve speed (#474)
@gavin-laihttps://github.com/gavin-lai links appear to still be failing. Pls fix.
—
Reply to this email directly or view it on GitHubhttps://github.com/CenturyLinkCloud/PublicKB/pull/474#issuecomment-148471985.


[FEATURE] use yeoman instead of rosaenlg-boilerplate repo

Is your feature request related to a problem? Please describe.

forking an existing repo is painful when creating a new RosaeNLG project
using yeoman would ease it

Describe the solution you'd like
npm install -g yo
create a folder to contain your project, and go in that folder
npm install -g generator-rosaenlg
yo rosaenlg

Describe alternatives you've considered
yeoman seems the most up to date solution

done


[FLINK-34215] FLIP-377: Support fine-grained configuration to control filter push down for JDBC Connector

This improvement implements FLIP-377 Support fine-grained configuration to control filter push down for Table/SQL Sources

@Jiabao-Sun could you please rebase

Thanks @eskabetxe for the reminder.
Rebased.


Send Batch example does not work

Expected Behavior
From the README.md the following example is provided:
batch := &EventBatch{
            Events: []*eventhub.Event {
                eventhub.NewEventFromString("one"),
                eventhub.NewEventFromString("two"),
            },
        }
err := client.SendBatch(ctx, batch)

Actual Behavior
The example provided on the README.md for using SendBatch does not actually work. It looks like SendBatch was changed with this commit:
https://github.com/Azure/azure-event-hubs-go/commit/e7ca69248db0ae51aa6c8ca86ed9fd1c60b1a21f
It is not clear how to use the new version of SendBatch().
Environment

OS: Windows 10
Go version: 1.12.7
Version of Library: v2.0.0

@bryanklewis you could try something like this:
	events := make([]*eventhub.Event, batchSize)
	for i := 0; i < batchSize; i++ {
		// init events
	}
        err = hub.SendBatch(ctx, eventhub.NewEventBatchIterator(events...))

@gavinfish that does not seem to work either. I got a panic.
Code:
package main

import (
	"context"
	"fmt"
	eventhub "github.com/Azure/azure-event-hubs-go/v2"
	"time"
)

func main() {
	connString := "{your_connection_string_here}"
	batchSize := 3

	hub, _ := eventhub.NewHubFromConnectionString(connString)

	events := make([]*eventhub.Event, batchSize)
	for i := 0; i < batchSize; i++ {
		events = append(events, eventhub.NewEventFromString("one"))
		events = append(events, eventhub.NewEventFromString("two"))
		events = append(events, eventhub.NewEventFromString("three"))
	}

	ctx, cancel := context.WithTimeout(context.Background(), 10*time.Second)
	defer cancel()

	err := hub.SendBatch(ctx, eventhub.NewEventBatchIterator(events...))
	if err != nil {
		fmt.Println(err)
	}
}

Results:
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x8e9693]

goroutine 1 [running]:
github.com/Azure/azure-event-hubs-go/v2.(*EventBatch).Add(0xc0002920f0, 0x0, 0x24, 0xc, 0x6635623700000010)
        C:/Users/myuser/go/pkg/mod/github.com/!azure/azure-event-hubs-go/v2@v2.0.0/batch.go:115 +0x33
github.com/Azure/azure-event-hubs-go/v2.(*EventBatchIterator).Next(0xc000004fe0, 0xc0002a6300, 0x24, 0xc000298180, 0x0, 0x0, 0xf524b0)
        C:/Users/myuser/go/pkg/mod/github.com/!azure/azure-event-hubs-go/v2@v2.0.0/batch.go:82 +0xfb
github.com/Azure/azure-event-hubs-go/v2.(*Hub).SendBatch(0xc00004c720, 0xb11560, 0xc00004c7e0, 0xb08160, 0xc000004fe0, 0x0, 0x0, 0x0, 0x0, 0x0)
        C:/Users/myuser/go/pkg/mod/github.com/!azure/azure-event-hubs-go/v2@v2.0.0/hub.go:668 +0x2b0
main.main()
        C:/Users/myuser/Desktop/event-adapter/main.go:26 +0x5a1

I think this is caused by the missing eventID for each event, but the error message here is unclear.

Thats seems to be the right track, in my application, the actual line the panic referred to was here. Is that a bug in the library or do i need an eventID waiting for each event added? Since it gets over-written by e.PartitionKey = eb.PartitionKey, seems like the library should account for no ID field.

Setting eventID when constructing events should mitigate the issue. But you are right, I think we should  consider no ID field situation. If the eventID is necessary when creating an event, I prefer to treat it as a constructor parameter, otherwise users probably will miss it.
What do you think? @devigned

@gavinfish constructing the slice with this line:
events := make([]*eventhub.Event, batchSize)

for example with batchSize 3 to the make() statement causes the constructed object to be filled with 3 nil Events. Its those nil events that are then causing the error when passed to func (eb *EventBatch) Add(e *Event) (bool, error).

@bryanklewis you are right. The error is not related with the eventID stuff. I have just double checked your sample code. The init code should be some like this if you want to use append:
package main

import (
	"context"
	"fmt"
	eventhub "github.com/Azure/azure-event-hubs-go/v2"
	"time"
)

func main() {
	connString := "{your_connection_string_here}"

	hub, _ := eventhub.NewHubFromConnectionString(connString)

	var events []*eventhub.Event

	events = append(events, eventhub.NewEventFromString("one"))
	events = append(events, eventhub.NewEventFromString("two"))
	events = append(events, eventhub.NewEventFromString("three"))

	ctx, cancel := context.WithTimeout(context.Background(), 10*time.Second)
	defer cancel()

	err := hub.SendBatch(ctx, eventhub.NewEventBatchIterator(events...))
	if err != nil {
		fmt.Println(err)
	}
}

otherwise you can use make like this:
package main

import (
	"context"
	"fmt"
	"time"

	"github.com/Azure/azure-event-hubs-go/v2"
)

func main() {
	connString:= ""

	batchSize := 3

	hub, _ := eventhub.NewHubFromConnectionString(connString)

	events := make([]*eventhub.Event, batchSize)
	for i := 0; i < batchSize; i++ {
		events[i] = eventhub.NewEventFromString(string(i))
	}

	ctx, cancel := context.WithTimeout(context.Background(), 10*time.Second)
	defer cancel()

	err := hub.SendBatch(ctx, eventhub.NewEventBatchIterator(events...))
	if err != nil {
		fmt.Println(err)
	}
}

Thanks @gavinfish, that helps. Either way, need some better examples of batch send in the docs :-)

The iterator does not work, (partition key is missing in the batch event). Batch ends up in random partition even when you specify PartitionKey for each event.

A batch can only be committed to a single partition. If more than one event in a batch has a different partition, the batch will fail.
The batch iterator may send events to a different partition on each iteration, but for each iteration all the events in that iteration must be committed to the same partition (per the Event Hubs service).


Mediatr throws exception when invoked in Parallel

Hi,
I'm not sure if this is na error but https://github.com/jbogard/MediatR/blob/master/src/MediatR/Mediator.cs#L38 this will throw an exception when invoked like this (at least for few starting threats) :
Parallel.ForEach(someList, (someElement) =>  {       _mediator.Send(new Request() { Element = someElement})  });
Method GetOrAdd of ConcurrentDictionary is not atomic. But I'm not sure if this is by design and I should not use mediator like this.

Typically not - we usually scope the IMediator instance per request, so in the above code you'd create the instance inside the Parallel.ForEach.
How are you hosting this? What is running MediatR?

Thanks for promp response. I thought I'm using this wrong (it was just simple to reuse what I've got in code)  :) Regarding hosting it is inside hangfire task and _mediator it is injected there as singelton.

But when you think about it _requestHandlers   is static field so no matter how many instances of mediator I will create they all will point to the same dictionary. So in the context of a web application, there could  be a situation where two requests (for the same handler) will hit serwer at the same time (like one in a million cases but it could happen) and we can get en exception inside GetOrAdd right ?

Ah you’re right!
On Thu, Oct 3, 2019 at 9:22 AM Beczka<EMAIL_ADDRESS>wrote:

But when you think about it _requestHandlers
https://github.com/jbogard/MediatR/blob/master/src/MediatR/Mediator.cs#L17
is static field so no matter how many instances of mediator I will create
they all will point to the same dictionary. So in the context of a web
application, there could be a situation where two requests (for the same
handler) will hit serwer at the same time (like one in a million cases but
it could happen) and we can get en exception inside GetOrAdd right ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jbogard/MediatR/issues/444?email_source=notifications&email_token=AAAZQMQ3FJAV6LLVULGBRLTQMWTVFA5CNFSM4I4S22DKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAHNB6A#issuecomment-537841912,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAZQMWNZDTJ354AKSZOB4LQMWTVFANCNFSM4I4S22DA
.

GetOrAdd is not atomic, but it ís thread-safe.
See Remarks on https://docs.microsoft.com/en-us/dotnet/api/system.collections.concurrent.concurrentdictionary-2.getoradd?view=netframework-4.8#System_Collections_Concurrent_ConcurrentDictionary_2_GetOrAdd__0_System_Func__0__1__
Because it is not atomic, You COULD get the value back which was inserted by another thread, and your value is discared. But since it is a cache, it doesn't matter here.

@remcoros Could you share why you vote for removal of static?

There is no control over the lifetime of the cache.
I was playing around with dynamic assembly loading/unloading and type generation, and there was no way for me to clear the static cache, so MediatR would hold on to the types in that assembly. And (from what I remember), when I load two times the same assembly (trough Assembly.LoadFrom(byte[]), it still holds references to the types in the first assembly.
It's very much an edge case. But I'd like to have some control.
e.g. trough a IHandlersTypeCache, with a default implementation like it is now (using static).
Haven't given it much thought though.

@remcoros ehh you are right :) problem was with how I set up IoC :) Thanks for a clear response with GetOrAdd

@remcoros if you can figure a way to add it without affecting existing users, I'm open to a PR. I don't want to make DI registration more complicated.

@remcoros if you can figure a way to add it without affecting existing users, I'm open to a PR. I don't want to make DI registration more complicated.

Yea figured that. It's so edge casy, that it's probably not worth the trouble. And if anyone still needs to get rid of that dictionary, they can fork (that's what I did back then).
This issue can be closed.

Another option would be to allow you to subclass and override like we did with the Publish options.


Add JUnit report

No, I don't know how to test it without releasing the package and referencing it from swrve/swrve repo.

Merging to give this a go in the ui/ project


add and refactor testcase for filter and exchange

1.add and refactor testcase for event.exchange
2.add and refactor testcase for multi OR filter
3.output error info for unquote
Signed-off-by: yason<EMAIL_ADDRESS>

Codecov Report

Merging #1900 into master will increase coverage by 0.27%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1900      +/-   ##
==========================================
+ Coverage   47.29%   47.56%   +0.27%
==========================================
  Files          89       89
  Lines        8830     8830
==========================================
+ Hits         4176     4200      +24
+ Misses       3968     3943      -25
- Partials      686      687       +1

Flag
Coverage Δ

#linux
50.95% <100%> (+0.29%)
:arrow_up:

#windows
42.55% <100%> (+0.31%)
:arrow_up:

Impacted Files
Coverage Δ

filters/parser.go
85.88% <100%> (+5.52%)
:arrow_up:

events/exchange/exchange.go
68.91% <0%> (+6.08%)
:arrow_up:

filters/filter.go
64.58% <0%> (+12.5%)
:arrow_up:

Continue to review full report at Codecov.

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

The testing for exchange filters is good but the refactoring of filters_test.go is completely unnecessary.

@stevvooe
I add a testcase for filter. The testcase is for "multi name", and the relation of multi name is ORed.
We cann't test it in the original test framework.
and The original test framework is too difficult to maintain. It depend with the order of  config data.
So I think it is necessary. PTAL.
		{
+			name: "Multi name",
+			input: []string{
+				"name==baz",
+				"name==bazo",
+			},
+			expected: []interface{}{
+				corpusS["key6"],
+				corpusS["key7"],
 			},
 		},

We cann't test it in the original test framework.

The case you are proposing is just two separate test cases. Its not really adding anything to the test.
If you want to test ParseAll, write a new test, but it is really not necessary, as you are just testing the compiler.

The original test framework is too difficult to maintain.

Not really. Just go based on the names.
This test also gets it wrong because it changes the type of the target from interface{} to cEntry, so the actual invariant we are checking at the end is getting broken.

OK, you decide it. Thanks for review @dnephin @stevvooe

LGTM

LGTM


Engine Refactor about the use of Pointers

Refactor performed with the purpose of discarding unnecessary Smart Pointers and substitute them with Raw or Unique Pointers.

@markmeliaa Thank you very much for your feedback Mark! It's important to discuss the changes, I really appreciate it.

Thank you both for your reviews!


Reupload stream closed marker in case of state mismatch

Sometimes we end up in a situation where the closed.json for a stream does not exist in object storage but local state says the stream has been closed. This condition should not be possible because object storage is updated before updating local state and any error in updating object storage should cause the local state not to be updated.
To recover from this supposedly impossible situation, have controller pass the closed timestamp read from object storage when it creates a backup stream object and if backup stream sees that the object storage closed marker is not present even though it should be then it uploads it again, hopefully resolving the issue.
About this change: What it does, why it matters
Fixes (or works around) invalid state that the service can sometimes end up in.

Codecov Report
Base: 76.14% // Head: 76.94% // Increases project coverage by +0.79% :tada:

Coverage data is based on head (510536d) compared to base (9d4d6de).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #142      +/-   ##
==========================================
+ Coverage   76.14%   76.94%   +0.79%
==========================================
  Files          16       16
  Lines        4226     4229       +3
  Branches      960      960
==========================================
+ Hits         3218     3254      +36
+ Misses        771      740      -31
+ Partials      237      235       -2

Impacted Files
Coverage Δ

myhoard/controller.py
79.61% <ø> (ø)

myhoard/backup_stream.py
77.93% <100.00%> (+0.38%)
:arrow_up:

myhoard/restore_coordinator.py
76.67% <0.00%> (+0.35%)
:arrow_up:

myhoard/basebackup_operation.py
89.51% <0.00%> (+10.48%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

This is not the correct fix for the problem. Apparently the files are uploaded successfully but the cloud provider API is currently broken and can return incomplete results. Uploading again does not fix the situation, the files can be missing from the responses no matter how hard you try uploading them. The last commit here would probably be valid, though, but closing the entire pr for now.


[FEATURE] Add OpenSearch Slack chat link to the website

Is your feature request related to a problem? Please describe.
ODFE project has a Slack chat. It is still alive, but not referenced on OpenSearch. Sometimes chat is more convenient than a forum.
Describe the solution you'd like

[ ] Add Slack link to the website
[ ] Rename Slack server ODFE -> OpenSearch
[ ] Invite developers, maintainers and contributors to discuss ongoing tasks there
[ ] Voice call Slack feature could simplify interaction between developers

odfe-workspace.slack.com

Hello @Yury-Fridlyand - we appreciate the idea. There is talk going on to add a public Slack for the OpenSearch community but that's all the details I have for now.

Woohoo, it is launched!
Ref: #1491, #1493.


Latest version frame false invalid

Latest version frame false invalid
版本: 1.92.2 (user setup)
提交: fee1edb8d6d72a0ddff41e5f71a671c23ed924b9
日期: 2024-08-14T17:29:30.058Z
Electron: 30.1.2
ElectronBuildId: 9870757
Chromium: 124.0.6367.243
Node.js: 20.14.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

I want to achieve this effect

"apc.electron": {
"frame": false,
},
"apc.menubar.compact": true,
"apc.listRow": {
"fontSize": 18,
"height": 25,
"lists": [
"customview-tree",
"results",
"open-editors",
"explorer-folders-view",
"outline-tree",
"scm-view",
"debug-view-content",
"debug-breakpoints",
"tree"
],
"parts": {}
},
I reopened it and it worked again. I don't know the problem. I kept restarting it and it didn't work before.

The effect I want to achieve is as shown above.

This extension is gone, but it works again.

After restarting the system, it has no effect


Migrate cluster settings modal to React

Cluster settings modal uses raw bootstrap and jquery to get user input, we're migrating it to the new React setup.
This PR introduces some new things, I'll explain them briefly:
A new webpack bundle
The most trivial of the news: a new webpack JS bundle to load our new settings modal into the HANA cluster page.
react-bootstrap
As we go forward with the implementation of React components across the application, we'll need an helper to translate our design system's jquery lingo to React. I identified react-bootstrap for this: a widely-used component library that wraps and reimplements Boostrap components in plain JS using also React internals. It came in handy and we'll have all the components in EOS automatically broght to the React ecosystem this way.
New API endpoints
I needed a couple additional API endpoints. See the code for the details.
A new Checkbox component
The checkbox component that react-bootstrap offers wasn't enough. We need something that we can drop inside any context, and after a couple evenings fighting with the toolkit I found a way that satisfies me.
The settings modal needs refactoring
I know, a 200-LOC-long React component is not a big deal for us. Thing is, the refactoring is pretty easy and I'd like to pair with someone to take them on board.

@arbulu89 I addressed almost everything now. Two things are open:

The connection default user is hardcoded now. This is retrieved from the cloud data currently. You could get it even from consul

It is, and I wanted to wait until we had the hosts projection, so I could easily take the cloud provider/user from there. For now, I was opting for a fake it till you make it strategy :grimacing:

Do we have some user notifications when the Save changes button is clicked? Positive or negative?

We don't, but I want to have them! I'm struggling finding a library that lets us render notification toasts, but it's harder than I thought since nearly no library supports rendering custom toasts, and I really just want to render our own EOS toasts.


SmartCard c++ sample does not work

I tried the c++ sample with external USB smartcard reader and attempt to run scenario 7 - List all smartcards fails with access denied. c# sample works fine because their Package.appxmanifest specifies the "Shared User Certificates" permission. When I set the same permission to c++ sample it fails slightly further with exception "The application called an interface that was marshalled for a different thread.". Is c++ code actually tested/working?

I asked the feature team to investigate.

Fix will be in the next update. Thanks for reporting it.


sample prometheus alerts

I'm using this alarms on prometheus to monitor my fluentd nodes. It helped me a lot to identify zombies nodes, when fluentd is running in a high load node and when the fluentd job is not running.
Thanks @kazegusuri for this plugin

Thanks for good examples!


GafferDispatchUI fixes and improvements

Removed reliance on a compatibility config file that will one day be removed.
Simplified nodule metadata for ExecutableNodes. This will mean that IE internal ExecutableNodes (RenderManNetRender, IERenderPass and LiveSceneCache at least) will need their UI files updating to explicitly request a nodule for anything they wish to be visible in the graph (typically the incoming scene). Adding a duplicate registration will do no harm in the current version of Gaffer, so we could update in advance of the release of the next version.
Fixed UI appearance for individually promoted task plugs.

Travis tests are failing, but it seems to be a problem with Travis itself.

Travis is happy again.


When updating /docusarus it causes the /data-access project to build.

Not sure if this is still an issue, a docusaurus build was done today to update the Edit link, and no data-access build was triggered.

Let's keep an eye on it - maybe I just triggered something on the data-access side for some reason.


irsdkClient::getVarFloat hits assert when building debug

Hey there,
I build the project from source and while the .exe, as well as release(x64) work fine, the debug(x64) asserts in irsdk_utils.cpp:297 when trying to get the TireChange in the DDU. Happens in other places too, so I think it's not directly related to the TireChange line, but just the first one to trigger it.
In release builds that assert is ignored, so the whole thing is just failing silently. Any idea if there is some check or condition missing before being allowed to access the data? Or is this just failing all the time and no one is noticing? I'm trying this on a Test Drive (maybe the data is not available here?).
Kind regards,
Cedric

Okay, I managed to fix it. All three variables: ir_dcBrakeBias ir_dpLTireChange and ir_dpRTireChange seem to be invalid (m_idx is -1), so just calling getFloat on them will cause an assert when debugging. I added an isValid() call (part of the irsdkCVar class) before accessing them and that seems to solve the issue. Just started out with it, so not sure if that's correct, or just hiding a more significant problem, but at least I can start in debug mode now.

@eXifreXi @lespalt Was this fix/temporary fix committed to this repo?


add support got global settings

cc @johanste @annatisch

closes #5126

Absent other considerations, I created a tool I would personally use in a project. A few notes:

As it stands, None is not a valid value for "immediate" values (it signals the absence of a value). I  considered using Unset for this purpose, but that places a burden on all users of this class, and absent optional function arguments are regularly denoted by None anyway.

There's an asymmetry between user-setting a value, and getting the computed value:
settings.log_level = logging.DEBUG  # set, assignmemt

level = settings.log_level(...)  # get, function

This is necessitated by having settings arbitrate case 4) itself. It's well within comfort zone  (especially since I suspect case 3) is not common), but thoughts welcome.

The currently added log_level and proxy_settings are just demonstrative, until there are real requirements for what settings should exist, and what values they need to accept.

With this descriptor approach, it would be simple to add an argument to PrioritizedSetting to add a __doc__ for the attributes.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Codecov Report

Merging #5192 into azure-core will increase coverage by 0.01%.
The diff coverage is 100%.

@@              Coverage Diff               @@
##           azure-core    #5192      +/-   ##
==============================================
+ Coverage       49.63%   49.64%   +0.01%
==============================================
  Files            6371     6372       +1
  Lines          179193   179248      +55
==============================================
+ Hits            88936    88991      +55
  Misses          90257    90257

Impacted Files
Coverage Δ

azure-core/azure/core/settings.py
100% <100%> (ø)

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 86b54eb...1832412. Read the comment docs.

cc @alzimmermsft @JonathanGiles -- @johanste said you might be interested to look at this PR


Release abi copying update

Add ABIs to npm package for consumption in subgraph environment

How does wagmi get the ABIs currently?

How does wagmi get the ABIs currently?

when you run "yarn wagmi" in the root, it connects to the foundry project and builds all the solidity, then grabs the abi from the out folder.

Current dependencies on/for this PR:

main

PR #252   👈

This comment was auto-generated by Graphite.


Kusto language

Add kusto language support to extra/.
Fix path handling in tools/lib/language.js on windows machines.

We don't merge languages into core anymore because of lack of developer time to maintain them in the long-run.  If you make your own repository though you can make a PR that adds a link to it so those who'd like to use your language as a 3rd party plugin can easily do see.

Fix path handling in tools/lib/language.js on windows machines.

If there is something here though we'd probably welcome a PR for a fix just for that.


Intrusion detection dashboard shows wrong Pod

Describe the bug
I have the feeling -- but I was wrong before -- that the Intrusion Detection / Falco dashboard uses the wrong label (see screenshot). Instead of labeling events by the misbehaving Pod, it labels events by the reporting Falco Pod.
If this is intended, please leave a comment and close the bug report.
To Reproduce
Steps to reproduce the behavior:

Install CK8s.
Install Falco Event Generator.
Look at the Intrusion Detection / Falco dashboard.

Expected behavior
I can see the name of the misbehaving Pod (e.g., falco-event-generator-deployment-7bf5f7d76b-mrbcb).
Actual behavior
I see the name of the reporting Pod (e.g., falco-4xsfn).
Screenshots

Version (add all relevant versions):

Compliant Kubernetes apps v0.13.0

Additional context
Happens on the demo cluster.

Not sure what the intention is, but it sounds good to report the misbehaving pod instead of the reporting pod.
I did a quick check and the metrics do have labels for which pod in which namespace that is misbehaving, but only for some of the events (I guess some events might not be possible to trace to a pod or they originate from something else). So it should be pretty easy to fix, then you could also filter on namespaces.


Expand the home message to other flavors.

Description
Made a new OpenFoodAPIService in HomeFragment with a static foodFact URL since all tagline falls under this URL. Example (https://world.openfoodfacts.org/files/tagline/tagline-opf.json). And made changes in the OpenFoodAPIService getTagline method so now the changed getRequest is  @GET("/files/tagline/tagline-"+BuildConfig.FLAVOR+".json"). So now all flavors show the tagline on homeFragment.
Related issues and discussion
#fixes {#2193}
Screen-shots, if any

Looks good!