Unified interface for Correction and CompoundCorrection

While testing the jetmet SFs for JES, I realized that working with CompoundCorrections can be a bit awkward. The issue is that depending on the correction level(s) that is (are) needed (simple or multiple), one has to work with either a Correction or a CompoundCorrection object. When writing helper code that loads the corrections, using as input the correction level requested by the user, one doesn't know in advance what will be used. Essentially I need to do e.g.:
// first load all objects
auto cset = CorrectionSet::from_file(".../jet_jerc.json.gz");
std::string key = "Summer19UL18_V5_MC_L2Relative_AK4PFchs"; // or could be "Summer19UL18_V5_MC_L1L2L3Res_AK4PFchs"
auto maybe_corr = std::find_if(cset->begin(), cset->end(), [key](const auto& elem){ return elem.first == key; } );
std::variant<Correction::Ref, CompoundCorrection::Ref> jesSF;
if (maybe_corr !=  cset->end())
    jesSF = maybe_corr->second;
else
    jesSF = cset->compound()[key];
// ...
// then, in the event loop:
float sf;
if (auto corr = std::get_if<Correction::Ref>(&jesSF)) {
    sf = (*corr)->evaluate({eta, pt});
} else {
    sf =  std::get<CompoundCorrection::Ref>(jesSF->evaluate({area,eta,pt,rho});
}

Perhaps it would be easier if both Correction and CompoundCorrection inherited from the same abstract class, that declares the evaluate method?
Then, CorrectionSet::at could automatically return either the correction or the compound correction, depending on the presence of the key in corrections_, dropping the extra compound method?

Hm I wasn't aware there would commonly be a need to switch between simple and compound objects. I don't see an obvious issue with making them polymorphic. (the interior nodes however are purposely not polymorphic)

I would simply argue that whether a correction is compound or not is an implementation detail, that the user does not necessarily need to know about.

I second the suggestion.  Whether the correction is compound or not is not of interest to the vast majority of users.  Internally we can design them as separate, but having a unified user-facing interface would be a very nice enhancement.  (Incidentally, this is also one of the points I noticed while working on the JECViewer and mentioned in the Cross-POG meeting last week)


Fix of non thread safe map reqHeaders

SOAP_ACTION is shared between different requests despite thread.local.request.context=true. Reproduced under load of 100 transactions per second.

Thanks for your feedback. Created https://github.com/apache/cxf/pull/306

@deki, thanks for your feedback. Please review

Looks like you need to rebase.


Compile Error using std::thread and objects  (this)

I can't generate a std::thread which works on an object (extra argument this).
std::thread t(&Hello::print, this);
The following program runs fine under linux, however we get an error using a 4.9.3 gcc:
mingw.thread.h:67:11: error: must use '.*' or '->*' to call pointer-to-member function in '((mingw_stdthread::detail::ThreadFuncCall<void (Hello::*)(), Hello*>*)this)->mingw_stdthread::detail::ThreadFuncCall<void (Hello::*)(), Hello*>::mFunc (...)', e.g. '(... ->* ((mingw_stdthread::detail::ThreadFuncCall<void (Hello::*)(), Hello*>*)this)->mingw_stdthread::detail::ThreadFuncCall<void (Hello::*)(), Hello*>::mFunc) (...)'            mFunc(std::get<S>(std::forward<Tuple>(mArgs)) ...);
#include <iostream>
#include <thread>
#include <mingw.thread.h>
struct Hello {
	int val;
	Hello(int val_): val(val_){
		std::thread t(&Hello::print, this);
		t.join();
	};
	void print () {
		std::cout << "Hello " << val << std::endl;
	}

};
int main(int argc, char **argv) {
	Hello h(1);
}

This is related to #34. Specifically, thread and call_once do not behave as if using std::invoke.
If you want to write a patch for it, I have the following recommendation.

When compiling for C++17 (#if __cplusplus >= 201703L), use std::invoke.
When compiling for C++14 and lower, a custom implementation equivalent to std::invoke would be required. The type_traits header allows detection of the relevant types, while a helper class will allow template specialization.

If you'd rather wait for someone else to fix it, I'll get around to it when I have time.

Fixed in #38.


Ladybird not doing proper cleanup after exiting

When closing the main Ladybird window, some processes stay active without any Ladybird windows open.
Expected behaviour should be the program closing child processes and anything in regards to itself.
See image below for the processes

This bug should've been solved by https://github.com/LadybirdBrowser/ladybird/pull/319
Please reopen if the problem is still reproducible.


66 propose adding new config entry constructor 1

Notes and Changes

New Structures Namespace to house Refueler, Neckro, Bat, and Spirit Pylons.
New Common Namespace to house classes common to multiple Namespaces/Classes
New ChebsRecipe class which can be used for any item/piece that has a recipe associated with it.
Removed SetRecipeReqs() and GetCustomPieceFromPrefab() from structures to the ChebsRecipe class to dedupe code.
Added new ModConfig() configuration settings constructor to BasePlugin for now.
ModConfig simplies setting configuration values and automatically appends [Synced with Server] or [Not Synced with Server] to the end of the description depending on whether you set serverSync to true or false.
This means the config sections will no longer be divided into Client/Server settings.  Instead all "SpiritPylon" config options will be beneath [SpiritPylon] and  the description will just annotate if it's client/server.
  So, we may want to change all of the config options to follow this format before we merge this so that users only have to deal with this once.
ModConfig has support for AcceptableValueBase classes such as AcceptableValueRange<> or AcceptableValueList<>.
A few default AcceptableValues have been set in BasePlugin for now.  I intend to move everything related to configuration into it's own Class at some point and remove it from BasePlugin all together.

Testing and Debugging

The NeckroGathererPylon isn't showing up in the build menu for me.
Instead there are two RefulerPylons (One green one orange)
I also noticed that I had no resource cost for summoning armored minions or mages.  As long as I had 1 core/metal on me I could just summon them forever.  It did consume fragments/meat though.  So, let me know if you see this as well.

These need to be investigated.  Other than that....everything "appears" to be working ok.
Let me know if you want me to go ahead and convert all configuration options once you review this.   Don't' really want to merge any of this to master until that is done so that we aren't messing with the config file over and over.

This has been accepted into #77 , so I will close this MR and we will merge it all together after some additional testing over there.


KCP sever is down been trying for over three hours anyone experience this?Unable to connect to the server: dial tcp <IP_ADDRESS>:8443: i/o timeout

Discussed in https://github.com/kcp-dev/kcp/discussions/1285

Originally posted by ppatierno June 17, 2022
I have build and started kcp as described in the landing README page and was taking a look at the last demo video.
First off all if I understood correctly (as the video mentions) I should have an Organization workspace created by default but I cannot see it.
kubectl get workspaces
No resources found

It seems I am on the "root:default" workspace anyway by running
kubectl kcp ws
Current workspace is "root:default".

But when try to create a new workspace as in the demo I got the following error.
kubectl kcp ws create --type Organization green
error: clusterworkspaces.tenancy.kcp.dev "green" is forbidden: spec.type "Organization" in workspace "root:default" does not exist

Any idea what's going on? I am just trying to follow the guide and video steps.

May be you need to do below ?
kubectl kcp ws .. (which will bring you to the root)
kubectl kcp ws create --type Organization green ?

@Bianca2-be any updates on your end?

Hi Andy, yes I got passed the server issue. However, I am still unable to create a deployment because each time I syn the kind cluster image it shows the changes. But when I do the command : k api- resources | grew deployments.
There is no deployments in the workspace.  I see that the pod is being created but I can’t find where and can’t get it to show up. So, I haven’t made it to complete the full trial of the demo. How about you ?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Andy Goldstein @.>
Sent: Tuesday, June 28, 2022 11:23:40 AM
To: kcp-dev/kcp @.>
Cc: Aisha Bah (student) @.>; Mention @.>
Subject: Re: [kcp-dev/kcp] KCP sever is down been trying for over three hours anyone experience this?Unable to connect to the server: dial tcp <IP_ADDRESS>:8443: i/o timeout (Issue #1327)
@Bianca2-behttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FBianca2-be&data=05|01|Aisha.Bah%40my.utsa.edu|fed1be28fcee4e67af4908da591a2ed4|3a228dfbc64744cb88357b20617fc906|0|0|637920266239286243|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=9CFXr3fNsELxFg2SdScy5xvdVc8vFvxWGnx82Uu%2F6E0%3D&reserved=0 any updates on your end?
—
Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkcp-dev%2Fkcp%2Fissues%2F1327%23issuecomment-1168867125&data=05|01|Aisha.Bah%40my.utsa.edu|fed1be28fcee4e67af4908da591a2ed4|3a228dfbc64744cb88357b20617fc906|0|0|637920266239286243|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=GYIbNy7XR28v4uh4I%2FKhooJrVFRmcjf6KXkwZ0L9svw%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATDA755JTD6HJQHYKEBKCZ3VRMKHZANCNFSM5ZOXICXQ&data=05|01|Aisha.Bah%40my.utsa.edu|fed1be28fcee4e67af4908da591a2ed4|3a228dfbc64744cb88357b20617fc906|0|0|637920266239286243|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=k8h0EPvP2Fapo8Dxu4I4VKVRbcCRCT%2B%2FOSFaym6%2B1Qg%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

@Bianca2-be if you're able to, please post every exact command that you are running, along with the output. Thanks.

/close
reopen if still needed


wrong "Unsaved changes" notification after duplication

Winter CMS Build: 1.2.5
Pages Plugin Version: 2.1.3
PHP Version: 8.2.0
Description:
When a Page is duplicated and saved ("The page has been successfully saved.") the little icon on the right of the tab doesn't change (always a circle, not a X) and the system seems don't understand its status.
Steps To Reproduce:

create a page and save it
duplicate this page
edit its title, url and body (image1)
save it ("The page has been successfully saved." appears) (image2)
close it and the original one
the message "Close the tab? Unsaved changes will be lost." appears (image3)

Thanks.

I've noticed that the change indicator has seemed a bit buggy in general for the past while, mostly when you have multiple tabs open. Do you think you could dig further into it and submit a PR to fix the issue?

@ericp-mrel is this something you'd be willing to take a look at?

Here's what I've found so far.
If you have multiple tabs open and you make changes to one of them, then all of the open tabs are being marked as modified.
It seems like something is causing the change event to be re-triggered after the original change event is fired since there is a slight delay between the first tab's close button is changed and when the others change.

There's also another issue related to the markup that's being added to the page when duplicating a page that's causing an issue with the modified counter not updating correctly. It seems like some of the form's hidden fields are being added as children inside a div, rather than being direct children to the actual form element itself.

This is problematic because of the following selector: https://github.com/wintercms/wn-pages-plugin/blob/728d2f0cb2f2a478b05d215f5647ede756d045af/assets/js/pages-page.js#L148

![Screenshot 2024-06-16 at 11 06 16 AM](https://github.com/wintercms/wn-pages-plugin/assets/63664924/9eb5b715-616d-4538-9bb9-e94acd34120f)

I haven't been able to figure out why this is happening yet.

This is what I've found after digging around with the JS debugger a little bit.
It seems like what's happening is when you make a change to an input field, the initial change event correctly fires and marks the active tab as modified. But then after that the FormWidget.onRefreshDependants function gets run and then triggers the change event during the refresh process and as a result of that every tab is marked as modified.
https://github.com/wintercms/winter/blob/9932f0c05f7423a50b526f05e2748185084bf143/modules/backend/widgets/form/assets/js/winter.form.js#L169-L200

@bennothommo @jaxwilko @mjauvin any thoughts?


Schema for IR measurement

Schema for IR
```python
class InfraredSpectroscopy(Measurement):
	measurements_data_file = ...
	simulation_data_file = ...
	method = ...
		default= 'Infrared spectroscopy'
	def normalize...
			# read the files and put the data in the results sub_section

			# generate a nice plot

```

In case additional parameters are needed (which may be missing from the file), the IR is plotted in the paper in Figure 3C labled as  ReRH H2-Reduced lyo, and is related to the sample ReRH_Nia-C_H lyophilizate.


Compass Results

I just noticed that the compass results in some maps are wrong. For example in 4-3, node F (according to the kancolle wiki) is node N in the compass result.
I hope this can be fixed.
Thank You.

Old bug this is.


ImageCanvas.draw_stimuli num_frames parameter stays zero

With every draw of the canvas, the frame counter should increase, but it remains zero.

The consequences of this bug is that stimuli are scheduled, but the frame on which they should be presented doesn't ever occur. So events such as the stimulus::started are never fired and probably the stimuli aren't drawn.


Save all datawallet modification payloads of a push in a single blob

Readiness checklist

[ ] I added/updated unit tests.
[ ] I added/updated integration tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

The reason for this change is that we currently observe some performance issues in the customer's environment. This is presumably due to a relatively slow blob storage. With this change, instead of saving one blob per datawallet modification, there is only one blob, which contains the payloads of all modifications in a JSON-serialized Dictionary, where the Index of the the modification is the key of the dictionary.

Datawallets/Queries/GetModifications/Handler.cs

We do: https://github.com/nmshd/backbone/blob/2dc9c17232120266a37cbf33eb133a4256fd7e6c/Modules/Synchronization/src/Synchronization.Application/Datawallets/Queries/GetModifications/Handler.cs#L95-L99
Btw: the solution I implemented is far from ideal. Especially the fact that the logic for creating modifications is duplicated across two files. But since we need to refactor the persistence of the Synchronization module anyway, I didn't spend a lot of time finding a better solution.

Sorry for the mess in the PR. Wanted to fix everything before we deploy the new version. But doing something quickly is not always quick. ;)


Ubuntu 20.04 LTS

Conducting a python3 pip install
~/untitledgoosetool$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.6 LTS
Release:	20.04
Codename:	focal
(.venv) REDACTED@REDACTED:~/untitledgoosetool$ python3 -m pip install .
Building wheels for collected packages: goosey, wxpython
Building wheel for goosey (setup.py) ... error
ERROR: Command errored out with exit status 1:
command: /home/REDACTED/untitledgoosetool/.venv/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-1vkwpido/setup.py'"'"'; file='"'"'/tmp/pip-req-build-1vkwpido/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-a4cxcpr2
cwd: /tmp/pip-req-build-1vkwpido/
Complete output (6 lines):
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: invalid command 'bdist_wheel'
ERROR: Failed building wheel for goosey
Running setup.py clean for goosey
Building wheel for wxpython (setup.py) ... error
ERROR: Command errored out with exit status 1:
command: /home/REDACTED/untitledgoosetool/.venv/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-pf3rf2i2/wxpython/setup.py'"'"'; file='"'"'/tmp/pip-install-pf3rf2i2/wxpython/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-tbiwnqpu
cwd: /tmp/pip-install-pf3rf2i2/wxpython/
Complete output (6 lines):
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: invalid command 'bdist_wheel'
ERROR: Failed building wheel for wxpython
Running setup.py clean for wxpython
Failed to build goosey wxpython
Installing collected packages: Pillow, six, numpy, wxpython, psutil, colored, pygtrie, Gooey, urllib3, idna, charset-normalizer, certifi, requests, pycparser, cffi, cryptography, python-dateutil, PyJWT, adal, attrs, frozenlist, multidict, yarl, aiosignal, async-timeout, aiohttp, portalocker, msal, msal-extensions, typing-extensions, azure-core, azure-identity, azure-common, azure-mgmt-core, isodate, azure-mgmt-compute, azure-mgmt-monitor, azure-mgmt-network, azure-mgmt-resource, azure-mgmt-security, oauthlib, requests-oauthlib, msrest, azure-mgmt-storage, azure-mgmt-web, azure-storage-blob, darkdetect, msrestazure, et-xmlfile, openpyxl, pyAesCrypt, pytz, blinker, pyparsing, kaitaistruct, pyasn1, hyperframe, hpack, h2, outcome, sortedcontainers, sniffio, exceptiongroup, trio, h11, wsproto, trio-websocket, selenium, pysocks, brotli, zstandard, pyOpenSSL, selenium-wire, soupsieve, BeautifulSoup4, lxml, appdirs, tqdm, webdrivermanager, goosey
Running setup.py install for wxpython ... error
ERROR: Command errored out with exit status 1:
command: /home/REDACTED/untitledgoosetool/.venv/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-pf3rf2i2/wxpython/setup.py'"'"'; file='"'"'/tmp/pip-install-pf3rf2i2/wxpython/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' install --record /tmp/pip-record-hjg03jev/install-record.txt --single-version-externally-managed --compile --install-headers /home/REDACTED/untitledgoosetool/.venv/include/site/python3.8/wxpython
cwd: /tmp/pip-install-pf3rf2i2/wxpython/
Complete output (138 lines):
running install
running build
WARNING: Building this way assumes that all generated files have been
generated already.  If that is not the case then use build.py directly
to generate the source and perform the build stage.  You can use
--skip-build with the bdist_* or install commands to avoid this
message and the wxWidgets and Phoenix build steps in the future.
"/home/REDACTED/untitledgoosetool/.venv/bin/python3" -u build.py build
Will build using: "/home/REDACTED/untitledgoosetool/.venv/bin/python3"
3.8.10 (default, May 26 2023, 14:05:08)
[GCC 9.4.0]
Python's architecture is 64bit
cfg.VERSION: 4.2.1

Running command: build
Running command: build_wx
wxWidgets build options: ['--wxpython', '--unicode', '--gtk3']
Configure options: ['--enable-unicode', '--with-gtk=3', '--enable-sound', '--enable-graphics_ctx', '--enable-display', '--enable-geometry', '--enable-debug_flag', '--enable-optimise', '--disable-debugreport', '--enable-uiactionsim', '--enable-autoidman', '--with-sdl']
/tmp/pip-install-pf3rf2i2/wxpython/ext/wxWidgets/configure --enable-unicode --with-gtk=3 --enable-sound --enable-graphics_ctx --enable-display --enable-geometry --enable-debug_flag --enable-optimise --disable-debugreport --enable-uiactionsim --enable-autoidman --with-sdl
checking build system type... x86_64-pc-linux-gnu
checking host system type... x86_64-pc-linux-gnu
checking for toolkit... gtk
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking whether we are using the Intel C compiler... no
checking how to run the C preprocessor... gcc -E
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking whether gcc needs -traditional... no
checking for g++... g++
checking whether we are using the GNU C++ compiler... yes
checking whether g++ accepts -g... yes
checking whether we are using the Intel C++ compiler... no
checking whether we are using the Sun C++ compiler... no
checking for ar... ar
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking for langinfo.h... yes
checking for wchar.h... yes
checking for sys/select.h... yes
checking for cxxabi.h... yes
checking for an ANSI C-conforming const... yes
checking for inline... inline
checking size of short... 2
checking size of void *... 8
checking size of int... 4
checking size of long... 8
checking size of size_t... 8
checking size of long long... 8
checking size of wchar_t... 4
checking for va_copy... yes
checking for _FILE_OFFSET_BITS value needed for large files... 64
checking if large file support is available... yes
checking for _LARGEFILE_SOURCE value needed for large files... no
checking whether byte ordering is bigendian... no
checking for std::wstring in <string>... yes
checking for std::istream... yes
checking for std::ostream... yes
checking for type_traits... yes
checking for __sync_xxx_and_fetch builtins... yes
checking for libraries directories... /usr/lib/x86_64-linux-gnu /usr/lib
checking for cos... no
checking for floor... no
checking if floating point functions link without -lm... no
checking for sin... yes
checking for ceil... yes
checking if floating point functions link with -lm... yes
checking for strtoull... yes
checking for pkg-config... /usr/bin/pkg-config
checking pkg-config is at least version 0.9.0... yes
checking for LIBPCRE... yes
checking for zlib.h >= 1.1.4... yes
checking for zlib.h... (cached) yes
checking for deflate in -lz... yes
checking for png.h > 0.90... no
checking for png.h... (cached) no
configure: WARNING: system png library not found or too old, will use built-in instead
checking whether png.c file exists... yes
checking for jpeglib.h... no
configure: WARNING: system jpeg library not found, will use built-in instead
checking whether jpeglib.h file exists... yes
checking for jbg_dec_init in -ljbig... no
checking for LIBTIFF... not found via pkg-config
checking for tiffio.h... no
configure: WARNING: system tiff library not found, will use built-in instead
checking whether tiff.h file exists... yes
checking for expat.h... yes
checking if expat.h is valid C++ header... yes
checking for XML_ParserCreate in -lexpat... yes
checking for LIBCURL... yes
checking for GTK+ version...
checking for pkg-config... (cached) /usr/bin/pkg-config
checking pkg-config is at least version 0.16... yes
checking for GTK+ - version >= 3.0.0... Package gtk+-3.0 was not found in the pkg-config search path.
Perhaps you should add the directory containing `gtk+-3.0.pc'
to the PKG_CONFIG_PATH environment variable
No package 'gtk+-3.0' found
no
*** Could not run GTK+ test program, checking why...
*** The test program failed to compile or link. See the file config.log for the
*** exact error that occurred. This usually means GTK+ is incorrectly installed.
configure: error:
The development files for GTK+ were not found. For GTK+ 2, please
ensure that pkg-config is in the path and that gtk+-2.0.pc is
installed. For GTK+ 1.2 please check that gtk-config is in the path,
and that the version is 1.2.3 or above. Also check that the
libraries returned by 'pkg-config gtk+-2.0 --libs' or 'gtk-config
--libs' are in the LD_LIBRARY_PATH or equivalent.

Error running configure
ERROR: failed building wxWidgets
Traceback (most recent call last):
  File "build.py", line 1563, in cmd_build_wx
    wxbuild.main(wxDir(), build_options)
  File "/tmp/pip-install-pf3rf2i2/wxpython/buildtools/build_wxwidgets.py", line 379, in main
    exitIfError(wxBuilder.configure(dir=wxRootDir, options=configure_opts),
  File "/tmp/pip-install-pf3rf2i2/wxpython/buildtools/build_wxwidgets.py", line 72, in exitIfError
    raise builder.BuildError(msg)
buildtools.builder.BuildError: Error running configure
Finished command: build_wx (0m2.119s)
Finished command: build (0m2.119s)
Command '"/home/REDACTED/untitledgoosetool/.venv/bin/python3" -u build.py build' failed with exit code 1.
----------------------------------------

ERROR: Command errored out with exit status 1: /home/REDACTED/untitledgoosetool/.venv/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-pf3rf2i2/wxpython/setup.py'"'"'; file='"'"'/tmp/pip-install-pf3rf2i2/wxpython/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' install --record /tmp/pip-record-hjg03jev/install-record.txt --single-version-externally-managed --compile --install-headers /home/REDACTED/untitledgoosetool/.venv/include/site/python3.8/wxpython Check the logs for full command output.

@AloneAndUnknown Can you try installing the tool on Ubuntu 22.04 LTS?

Sure,
REDACTED~/untitledgoosetool$ python3 -m pip install .
Defaulting to user installation because normal site-packages is not writeable
Processing /home/ckbrown/untitledgoosetool
Preparing metadata (setup.py) ... done
Collecting Gooey==<IP_ADDRESS>
Using cached Gooey-<IP_ADDRESS>-py2.py3-none-any.whl (680 kB)
Collecting aiohttp==3.8.5
Using cached aiohttp-3.8.5-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (1.0 MB)
Collecting async-timeout==4.0.2
Using cached async_timeout-4.0.2-py3-none-any.whl (5.8 kB)
Collecting azure-identity
Using cached azure_identity-1.13.0-py3-none-any.whl (151 kB)
Collecting azure-mgmt-compute
Using cached azure_mgmt_compute-30.1.0-py3-none-any.whl (4.9 MB)
Collecting azure-mgmt-monitor
Using cached azure_mgmt_monitor-6.0.1-py3-none-any.whl (1.3 MB)
Collecting azure-mgmt-network
Using cached azure_mgmt_network-24.0.0-py3-none-any.whl (655 kB)
Collecting azure-mgmt-resource
Using cached azure_mgmt_resource-23.0.1-py3-none-any.whl (2.5 MB)
Collecting azure-mgmt-security
Using cached azure_mgmt_security-5.0.0-py3-none-any.whl (1.1 MB)
Collecting azure-mgmt-storage
Using cached azure_mgmt_storage-21.0.0-py3-none-any.whl (2.8 MB)
Collecting azure-mgmt-web
Using cached azure_mgmt_web-7.1.0-py3-none-any.whl (4.7 MB)
Collecting azure-storage-blob
Using cached azure_storage_blob-12.17.0-py3-none-any.whl (388 kB)
Collecting colored==1.4.4
Using cached colored-1.4.4.tar.gz (36 kB)
Preparing metadata (setup.py) ... done
Collecting cryptography==41.0.2
Using cached cryptography-41.0.2-cp37-abi3-manylinux_2_28_x86_64.whl (4.3 MB)
Collecting darkdetect
Using cached darkdetect-0.8.0-py3-none-any.whl (9.0 kB)
Collecting msal==1.23.0
Using cached msal-1.23.0-py2.py3-none-any.whl (90 kB)
Collecting msrestazure
Using cached msrestazure-0.6.4-py2.py3-none-any.whl (40 kB)
Collecting openpyxl
Using cached openpyxl-3.1.2-py2.py3-none-any.whl (249 kB)
Collecting pyAesCrypt
Using cached pyAesCrypt-6.1.0-py3-none-any.whl (15 kB)
Requirement already satisfied: pytz in /usr/lib/python3/dist-packages (from goosey==1.2.4) (2022.1)
Requirement already satisfied: requests in /usr/lib/python3/dist-packages (from goosey==1.2.4) (2.25.1)
Collecting selenium-wire
Using cached selenium_wire-5.1.0-py3-none-any.whl (239 kB)
Collecting selenium==4.10.0
Using cached selenium-4.10.0-py3-none-any.whl (6.7 MB)
Collecting typing-extensions>=4.1.1
Using cached typing_extensions-4.7.1-py3-none-any.whl (33 kB)
Collecting webdrivermanager
Using cached webdrivermanager-0.10.0.tar.gz (33 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Preparing metadata (pyproject.toml) ... done
ERROR: Package 'goosey' requires a different Python: 3.10.12 not in '<=3.10.11,>=3.7'
REDACTED:~/untitledgoosetool$ python3 --version
Python 3.10.12
effin python.

Ill be honest, i dont feel like trying to go through the python3 downgrade aptitude package manager hell.


Fix Psi Warrior's Force of Will to actually use INT

Psi Warrior

Force of Will

Your psionic energy grants you extraordinary resilience. At the start of each of your turns, you gain temporary hit points equal to your Intelligence modifier (minimum of 1) if you have at least 1 hit point. In addition, you can use your Intelligence modifier instead of your Wisdom and Charisma modifier for saving throws if it's higher.

Currently, when Force of Will is triggered, all that happens is that the displayed saving throw is shown as INT, but it didn't actually use the INT mod. This change fixes that.

However, several caveats:
(1) This change considers your saving throw proficiency when doing the replacement. It compares INT [+Prof] against the original WIS [+Prof] or CHA [+Prof], rather than just INT vs WIS/CHA. I like this more because if you get proficiency through sub-classing or feats, you will gain benefit or not lose out just because the base INT vs WIS works differently.
If we'd prefer just INT vs WIS, I have a code path in there to do just that. (Set useNewBonuses = false)
(2) The base game Mana Painter Sorcerer has Mana Absorption, which has similar wording:

Your Charisma modifier is used (if higher) instead of other ability modifiers on your saving throws against spells and other magic effects.

This vanilla feature works by replacing the saving throw entirely to be Charisma if the CHA mod is greater than the original saving throw's ability mod. (This is done through an ISavingThrowAffinityProvider in RulesetImplementationManager.TryRollSavingThrow).This has interesting consequences, such as the fact that you get CHA proficiency added (to all saving throws basically), you don't benefit from cover against spells that were originally DEX saves, and that bonuses to CHA saves (like gnome adv on INT/WIS/CHA) apply for things like fireball.
This change does not do that. Rather, the save just swaps the ability scores, but is mostly still treated as a saving throw of the original type.
(3) Despite point 2, the saving throw type still does get changed to INT after Force of Will is applied, so any mod power that uses OnSavingThrowInitiated will race against it. This is currently not an issue because no Unfinished Business feat applies a bonus to INT/WIS/CHA through that handler. However, if this code will be reused for Barbarian Path of Savagery, then the race needs to be addressed for the Shield Master feat.

 You gain +2 AC while you are raging and dual-wielding two melee weapons.

Thanks for the suggestions but will probably take the vanilla approach and remove older modifier from list, add new one. Meanwhile, I’m the solo developer around.  Thanks


Single channel halos

Are you using the latest version? Is the version currently in use as reported by npm ls sharp the same as the latest version as reported by npm view sharp dist-tags.latest?
Yes<EMAIL_ADDRESS>What are the steps to reproduce?
Do multiple operations with single channel images
What is the expected behaviour?
Perform operations trim, resize, flatten, composite, resize, flatten with single channel b-w—with the same quality as multichannel srgb.  Hopefully faster too.
Are you able to provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem?
Available upon request.
Are you able to provide a sample image that helps explain the problem?
Faint halo / artifacts / glow around the letters:

Expected output:

What is the output of running npx envinfo --binaries --system?
  System:
    OS: macOS 10.15.7
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 24.40 MB / 64.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 14.17.0 - ~/.asdf/installs/nodejs/14.17.0/bin/node
    Yarn: 1.22.10 - ~/.asdf/shims/yarn
    npm: 7.19.0 - ~/.asdf/installs/nodejs/14.17.0/.npm/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman

Please can you provide a code sample that allows someone else to reproduce this.

@jlarmstrongiv Are you able to provide code sample that allows someone else to reproduce this?

@lovell I haven’t forgotten about it, I just haven’t had time to refactor and write it yet—would I be able to privately send you that portion of my project?  I’ll have more time after a deadline passes next month

No hurry, whenever you're ready, thank you for the update.

@jlarmstrongiv Were you able to make any progress with this?

Closing due to inactivity but please feel free to reopen with more details if further help is required.


[Feature Request] Sync with MyAnimeList

If there was a way to synchronize the "Watching", "Completed", etc. list with MAL accounts it would be great! With this feature users with both MAL and KawAnime wouldn't have to update two anime lists which is, in my opinion, very useful.

Nevermind, I just read that it's on track in the reddit thread


Add OpenCL backend

This adds a new runtime backend using OpenCL for SPIR-V devices. The target device must currently either support the Intel USM extensions (which are not only implemented by Intel, but also e.g. pocl or rusticl), or, as a fallback, fine-grained system SVM [1].
For SPIR-V generation, it relies on our generic single-pass compiler, i.e. --hipsycl-targets=generic.
The backend has been tested using the Intel OpenCL implementations for CPUs and GPUs. It runs fine on CPUs, iGPUs and PVC.
Other potential targets to experiment with could be pocl, rusticl or the oneAPI construction kit.
It needs some more testing, especially for multi-device setups.
[1] The requirement for fine-grained system SVM may appear very strong compared to the semantics of SYCL USM memory. However, system SVM is needed because all other forms of SVM require that pointers to all allocations that a kernel might use are passed to the runtime at submission time. In a SYCL USM scenario, we cannot however know this in general, because the compiler cannot infer all the pointers that might be used (think of pointer-based data structures like a linked list).

Works fine on multi-gpu Intel Max 1550 system. However, there seems to be a strange (driver?) issue with shared context across all the GPUs on the test system. On such systems, the environment variable HIPSYCL_RT_OCL_NO_SHARED_CONTEXT=1 must be set. This may limit the ability to directly copy between devices.

The target device must currently either support the Intel USM extensions (which are not only implemented by Intel, but also e.g. pocl or rusticl)

USM is currently entirely unimplemented in Rusticl.
https://gitlab.freedesktop.org/mesa/mesa/-/issues/9066

@biergaizi Ah, I though this was already being worked on - thanks for the clarification! Good to hear that the fallback is useful! :)


Water Whip won't pickup on return

Water whip won't catch the water in the bucket in ver 1.19.2-1.9.2

@fluffyxox Was the water bucket in your hand when the water returned?

yes

Can not reproduce. Re-open when you can reproduce it reliably.

Nevermind, I was able to reproduce this bug and add a workaround. This issue is resolved now.


replace "whitelist" with "allowlist"

https://developers.google.com/style/inclusive-documentation#features-and-users describes this pretty well. Associating "white" with good/allowed and "black" with bad/disallowed cannot be disentangled from systemic racism. In contrast, allowlist and denylist exactly and merely describe the functions of these fields and variables.
This will change CLI options - do we need to support both options for a time or are we ok with a breaking change?

--metrics-host-whitelist becomes --metrics-host-allowlist
--rest-api-host-whitelist becomes --rest-api-host-allowlist
change variable names throughout the code

Raised equivalent doc issue:
https://app.zenhub.com/workspaces/pliny-5e33b0c5274adc1d47ff7cc5/issues/pegasyseng/doc.teku/103


Why is responseJson only in fuel-android?

Writing a backend application which gets JSON kind of needs a way to deserialize JSON and I'm currently wondering why fuel core doesn't support this?

responseJson is for org.json library from android SDK. Fuel isn't depend on android, therefore it isn't included.
In addition, Fuel has a module  for GSON support: https://github.com/kittinunf/Fuel#gson-deserialization
If you can tell me which Json library you use, I can help you with writing a deserializer.

Thanks. I just hadn't figured out how to use Fuel properly :)


undefined: ws.SetLogger

Why I'm facing this issue? SetLogger is a function defined in the WS package. I took the example code for CS and CP but everything is working except this WebSockets package. What is the issue and what I'm doing wrong?

On master branch the function is defined, so the error doesn't make any sense to me. I would need to get some more context. Could you please share the offending code snippet (including the imports ideally)?

Actually, I cloned this whole repository and It worked fine. But If I use the code from the examples folder only. I face this error even without a single change in the code. However, commenting on the ws.SetLogger line makes the code work again. But I wanted to establish that WS connection between CP and CS.

Currently, master branch isn't the one, that is fetched by go get. The code with latest tag - 0.15.0 is outdated. You can tell go to directly use master branch instead of the latest tag by executing go get github.com/lorenzodonini/ocpp-go/ocpp1.6@master. this fixed the issue for me.

I just released v0.16.0, which contains all the fixes currently on master branch. That way, you won't have to specify the branch anymore.
@Shakirsadiq6 can you check, if this solves your issues?

@lorenzodonini Thanks.


Add stacktraces to errors

Codecov Report
Base: 76.46% // Head: 76.12% // Decreases project coverage by -0.34% :warning:

Coverage data is based on head (02780f5) compared to base (772eea6).
Patch coverage: 48.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #452      +/-   ##
==========================================
- Coverage   76.46%   76.12%   -0.35%
==========================================
  Files          17       17
  Lines        4598     4599       +1
==========================================
- Hits         3516     3501      -15
- Misses       1082     1098      +16

Impacted Files
Coverage Δ

src/compiler.jl
75.13% <48.00%> (-0.05%)
:arrow_down:

src/compiler/orcv2.jl
61.90% <0.00%> (-10.48%)
:arrow_down:

src/compiler/orcv1.jl
76.92% <0.00%> (-3.85%)
:arrow_down:

src/compiler/validation.jl
64.68% <0.00%> (-0.34%)
:arrow_down:

src/Enzyme.jl
86.21% <0.00%> (-0.05%)
:arrow_down:

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.


Add Play Button for Running Code in Deno Docs

Description
To enhance the user experience and interactivity of our Deno documentation, we propose adding a "Play" button next to code snippets. This button will allow users to run the code directly in an embedded Deno playground, providing immediate feedback and a hands-on learning experience.
Example:

This is not a priority right now.


Add C header files to dist

Generate individual C header files in dist.

I'm a bit of two minds on this - maybe we should just use the generated files as templates (that can and should be hand-edited); otherwise we will never cover all use cases; OTOH the constants file can be included in the user file, which is probably a good idea.

The spec should allow to define additional constants?

From: Michał Moskal<EMAIL_ADDRESS>Sent: Thursday, November 12, 2020 8:43:54 AM
To: microsoft/jacdac<EMAIL_ADDRESS>Cc: Peli de Halleux<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/jacdac] Add C header files to dist (#53)
I'm a bit of two minds on this - maybe we should just use the generated files as templates (that can and should be hand-edited); otherwise we will never cover all use cases; OTOH the constants file can be included in the user file, which is probably a good idea.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fjacdac%2Fpull%2F53%23issuecomment-725901123&data=04|01|jhalleux%40microsoft.com|c82c1c9275f74380d01708d886deb4cb|72f988bf86f141af91ab2d7cd011db47|1|0|637407638356481480|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=sDx%2Ba%2BbOPrlSiQ56HaF2tAcrA9EHLLGfMJNmboh7YH8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKPB4ZXXBZRIDS5H5JLSPOG3VANCNFSM4TSRLU3A&data=04|01|jhalleux%40microsoft.com|c82c1c9275f74380d01708d886deb4cb|72f988bf86f141af91ab2d7cd011db47|1|0|637407638356481480|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=4VfwjRs%2FQ9T41%2BBG5HTZGhY0B0JOxlembcithENEG0w%3D&reserved=0.

Let's see what's needed in jd-c

I really think we should have one source of truth for the constants and as much as possible it should be the spec.

From: Michał Moskal<EMAIL_ADDRESS>Sent: Thursday, November 12, 2020 8:48:38 AM
To: microsoft/jacdac<EMAIL_ADDRESS>Cc: Peli de Halleux<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/jacdac] Add C header files to dist (#53)
Let's see what's needed in jd-c
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fjacdac%2Fpull%2F53%23issuecomment-725903269&data=04|01|jhalleux%40microsoft.com|0a34eb37213a493eba0108d886df5e65|72f988bf86f141af91ab2d7cd011db47|1|0|637407641205492375|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=pxXY1SeC82SrcyvbYsrGaOO%2BJN9a2A%2BSYCZZ%2B8ZmBgM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKKX5RLQSIBFYKQYOYLSPOHNNANCNFSM4TSRLU3A&data=04|01|jhalleux%40microsoft.com|0a34eb37213a493eba0108d886df5e65|72f988bf86f141af91ab2d7cd011db47|1|0|637407641205497363|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=GMxyj4XLmY%2B13iLIM1%2FDlLpBZnhiDJ0osGYwIoWwvuw%3D&reserved=0.

I think we should be going a step further and generating complete JACDAC interfaces, and maybe even the code for communicating with the underlying I2C peripherals (stretch goal).

I find the fraction of code that can be generated for different services in jd-c to be very limited (other than constants and structs for binary layout).
The registers are currently defined semi-dynamically. The parsing code is 376 bytes and it handles read, write, sign and zero-extensions. I'm not sure generated code would be more efficient. One reason to do it differently would be for readability.


Blog: Using MySQL with Node-RED

Description

Related Issue(s)

Checklist

[ ] I have read the contribution guidelines
[ ] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

Hi @gdziuba I have applied all of the suggestion, could you please re-review it ?

Can't merge because of local install links.   I think @joepavitt is working on this.

Yep, ignore the test failures, they're being fixed elsewhere


Reduce the scope of fields the extension will inject into.

Type of change
Select the type of your PR

Bugfix

Description
This change restricts the extension to only injecting into fields where there are no annotations, or only annotations known to be safe which are those coming from:

io.kroxylicious
org.junit

It also prevents injection into fields which already have a non null value.
Additional Context
I've had a couple of issues with the extension trying to inject into fields without having a KafkaCluster field and thus the injection fails. In my specific tests I didn't want the extension to inject into the fields. As I was using a mockito to mock the admin client or building my own MockProducer or MockConsumer.
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

I still need to update the change log

It also prevents injection into fields which already have a non null value.

I don't see the code enforcing this.  Here's a failing test
    @Order(4)
    static Consumer<String, String> mockConsumer = new MockConsumer<>(null);

    @Test
    void fieldWithExistingValueNotInjected() {
        assertTrue(mockConsumer instanceof MockConsumer<String, String>);
    }

Hmm looks like I've failed to push a commit. Thanks for checking @k-wall

I'd add a test ensure that fields with unrecognised annotation don't get injected too.  This test passes.
    @Deprecated // poor choice of annotation.
    static Consumer<String, String> fieldWithUnrecognizedAnnotations;

    @Test
    void fieldWithUnrecognizedAnnotationsNotInjected() {
        assertNull(fieldWithUnrecognizedAnnotations);
    }

I'd add a test ensure that fields with unrecognised annotation don't get injected too. This test passes.
    @Deprecated // poor choice of annotation for test case.
    static Consumer<String, String> fieldWithUnrecognizedAnnotations;

    @Test
    void fieldWithUnrecognizedAnnotationsNotInjected() {
        assertNull(fieldWithUnrecognizedAnnotations);
    }

Yeah thats a good point, but I think java.lang annotations should probably be allowed. e.g. @Deprectated and @SupressWarnings should still be valid targets.


[proposal] support Reservation preemption

What is your proposal:
koord-scheduler supports Reservation preemption.
Why is this needed:
In our scenario, Reservations also have different priorities. The lower priority Reservations may create earlier than the higher priority Reservations, so they are assigned first. When the higher priority Reservations come, there are not enough resource to assign them. So we expect that when a high priority Reservation cannot be assigned, it can preempt a lower priority Reservation.
Is there a suggested solution, if so, please add it:

/assign


How can i have access to rasa response inside HTML file and send it to my robot?

Hi,
I don't know where is the correct place to ask this question, so if it's the wrong place, sorry about that!
I'm using rasa3.1 and everything is working with your HTML file. I want to have access to the rasa response in an HTML file and send it with URL method to my robot? so I need to define a string variable to store received messages as rasa responses and use this message in my own way. I searched a lot but I couldn't find something useful to work correctly.
I would be grateful if you give me a hand.
thanks.

By "rasa responses" are you referring to messages from user or Rasa chatbot?
There is parameter onSocketEvent (see README) which allows one to listen and react to some of the socket events, including messages from the bot, but unfortunately that does not include messages from the user. (That event could also be exposed by would require changes to widget source code.)

Dear @rajalala thanks for your response.
Base on your expalnation onSocketEvent should work for me. I need to listen to messages from rasabot.
for example:
user: "hi"
bot: "Hi there, How can i help you?"
I need use bot response "Hi there, How can i help you?" and send it to my mobile robot.
How could i use below code in my html file?
onSocketEvent={{
  'bot_uttered': () => console.log('the bot said something'),
  'connect': () => console.log('connection established'),
  'disconnect': () => doSomeCleanup(),
}}

thanks in advance.

html code:
<html>
	<body>
		<script>!(function () {
		sessionStorage.clear();// remove chats history

		  let e = document.createElement("script"),
		    t = document.head || document.getElementsByTagName("head")[0];
		  (e.src =
		    "https://cdn.jsdelivr.net/npm/rasa-webchat@1.0.1/lib/index.js"),

		    (e.async = !0),
		    (e.onload = () => {
		      window.WebChat.default(
			{
			  intitpayload: '/greet',
			  customData: { language: "en" },
			  socketUrl: "http://<IP_ADDRESS>:5005",
			  params:{

				storage: "session" // chats history location is "session", default is "local"

			  }

			},
			null
		      );
		    }),
		    t.insertBefore(e, t.firstChild);
		})();

		</script>

	</body>
</html>


Detect console.logs and its equivalent in other languages

Description
The goal of this PR is to detect console.log()s and its equivalent in other languages (System.out.println(), println(), print(), fmt.Println(), etc.). The initial goal is to successfully detect such. Then we can comment them and add them to the PR Rating score.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Chore: cleanup/renaming, etc
[ ] RFC

Notes
I am initially sending this PR with a simple console.log() to test this PR.
Acceptance

[x] I have read the Contributing guidelines
[x] I have read the Code of Conduct

This is a test, right? Does not seem to work.

As stated in the description, yes

@EstebanDalelR pls retest, I added some PRs with hello worlds in some languages to do so. A few notes:

Commenting the specific line diff would be a next step. I tried doing so with octokit.rest.pulls.createReviewComment (its equivalent on our octokit.request("....") fashion but couldn't achieve it. Right now we're adding an additional comment if console logs are detected
I tried changing the function from a void that comments to a funciton that returns a boolean but I wasn't successful with that because the call to openai is awaited and if console logs are detected, that would be later and therefore a false would be returned despite console logs being there.
Modifying the label to "don't merge" would be a remaining step to send this to prod, but let's keep testing it. I'm expecting testing this feature to be extensive.
The name of the file is detectConsoleLogs.ts which is what it's intended for at the moment. However we can and should change that to something of a broader scope in the future.


Basename: Bio design & NFTs metadata fixes

What changed? Why?

Update bio design
Fix basename links
Fix NFTs metadata and add assets for opensea

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1


[MERGE REQUEST] Translate to French

Hi,
There is a version of config.yml and the README translate to French.
⚠The command /panel don't work on my bot ⚠
There is the screenshot bellow, no error on the console.

He start correctly :

The issue with /panel would either be bad config or an error in panel.js. There is no debug logging but if you ask GBT to insert logging without modifying code structure then rerun it can help. I heavily advise to keep a fresh copy of panel.js from this repository. If you fix the issue I advise to remove all the debug console logging from the file

No i don't think it's an error with panel.js i use this version of the bot on 2 discord server and no bug there.
I don't want to start to modify the panel.js to risky for me, i'm not a JS developer 😅

I advise to add logging to the panel.js file. Make a copy of your original and put on your desktop or somewhere then get GBt to add logging to it all. If that breaks then you always have a copy of the original there.
If that doesn't work then i would recommend going over config.yml thoroughly. Noticing the bot not responding to the command but the bot being online indicates the command didnt fire. Usually invalid object/variable that isnt in a try...catch statement. Everything is fine for me apart from unknown interaction but can be fixed by adding interaction.deferReply() and replacing interaction.reply with interaction.editReply

Don't need to do all of that. I just make a simple test :

Copy the original file config.yml
Copy the general ID for run correctly the bot
Now execute /panel

And it's worked !

So it's a problem cause by my translation, i think ChatGPT go wrong when he translate the file.
I'm gonna figure out what is the problem and solve it.

Problem solve by the latest commit.
Everything works, ready for merge. @ralphkb


Questions about pilot discovery

I am trying to build a docker image for istio/pilot-discovery component and plans to run it with docker-compose. I want to know if:

pilot-discovery can be used outside the context of kubernetes?
if yes to #1, where can i find more info on how to set the options for pilot-discovery, mesh config, mesh networking etc.?

thanks.

It's very uncommon to run Istio entirely outside of Kubernetes without at least an api-server to store config. It is possible though: https://blog.howardjohn.info/posts/local-gateway/

Instead of using latest image gcr.io/istio-testing/pilot:latest or istio/pilot:1.15.0-debug@sha256:0db0d47671b03387667d5bc8583dff59ccd64a6dd67b3641a319d6638e1e2eed, i switched to istio/pilot:1.14.3-debug, now pilot-discovery service started successfully.


Add rectified depth image as output

In addition to the pointcloud format, would it be possible to return a rectified depth image as well? Looking at the NxLib API, this would be similar to the single channel pointmap from the RenderPointMap function:
https://www.ensenso.com/manual/cmdrenderpointmap.htm
The reason is that many detections can be done on the depth image instead of a pointcloud. Now, we recreate the depth image from the pointcloud result ourselves, but it seems cleaner if the driver would expose this format directly.
Is this feature of interest to others as well?

Sure, that's possible. I assume that such a depth image should be represented as a float image as it is described in REP 118 and as it is used by depth_image_proc, right? If so, I will have a look into this soon.

Correct, that's what I meant!

Any progress on this? We would be interested in adding this functionality ourselves and make a PR if this fits with the design of the driver.

I didn't get around to do it yet. If you want to implement it yourself I will gladly accept a PR.


Prior for Beta Distribution

How do we add prior to Beta Distribution.
The documentation of Beta says it takes only scalar or array.
What is the way of adding a hyperprior to the beta distribution . something like below

This gives me error
# Creating Hyper priors for the Bernouli distribution
p_beta = Beta([0.5,0.5], name='p_beta')
p_gamma = Gamma(0.1,0.1, name=['p_gamma'])
#For each bag create a Bernouli distribution of the different colors inside it
P = Beta([p_beta, p_gamma],
        plates=(D,),
        name='P_')

You can use the DirichletConcentration node as a parent node to Beta too as:
c = DirichletConcentration(2)
p = Beta(c)

Perhaps, I'll need to change the name from DirichletConcentration to Concentration.
The formulas you have written in the image cannot be implemented in BayesPy because of non-conjugacy. Will have to wait for the non-conjugacy support until exactly that can be implemented.


StyleBoxFlat aliasing looks blurry (regression between 3.2beta5 and 3.2beta6)

Godot 3.2 beta5
Same stylebox, two versions.
Godot 3.2, beta5, worked fine:

Godot 3.2, beta6 and rc1:

Now it is blurry and has a line at the bottom.
Turning off antialiasing "fixes" it, but it looks worse than before.
Project:
TabContainerTheme.zip
It might have been broken in https://github.com/godotengine/godot/issues/34830, but if you look closely at the screenshots, only the inner parts were lacking anti-alias, while the outline parts were fine all along.

What's the status on this as of 3.2.3?

Yes it's still there.

I can still reproduce this in 4.0.dev.custom_build [b5b633ff5].

I think the issue with AA should be fixed rather than trying to bypass the issue. Either that or there should be new properties that allow certain sides of borders to be blurry and sharp.

I think the issue with AA should be fixed rather than trying to bypass the issue. Either that or there should be new properties that allow certain sides of borders to be blurry and sharp.

One way to fix this would be to change StyleBoxFlat's antialiased property to use an enum instead, with three possible values:

Disabled
Crisp (what we had in 3.1.x)
Smooth (what we have now)

@Calinou That sounds like a great idea! We could do that for 3.x since proper antialiasing will be probably supported only in 4.x.

@Calinou That sounds like a great idea! We could do that for 3.x since proper antialiasing will be probably supported only in 4.x.

Switching to an enum will break compatibility, so we'll have to add a second boolean property in 3.x instead.

Switching to an enum for the antialiased property will break compatibility with existing projects, so we'll have to add a second boolean property in 3.x instead.

@Calinou Couldn't we handle compatibility in 3.x by overriding the property _set function and set a specific value when antialiased is true? And then if it's still needed on master it can be done with compatibility breaking to make the code cleaner.


After making a change to src templates, no changes take affect.

After building and tweaking a template file, all the components using that template need to be rebuilt in order to see the new changes.

To solve this:

gulp nuke
gulp watch
???
Profit


ci: fixed version until chocolatey is ready

What does this pull request do?
Fixed version since the latest 3.8 version got some issues apparantely
Related issues
closes #ISSUE

:broken_heart: Build Failed

Expand to view the summary

Build stats

Build Cause: [Pull request #887 opened]

Start Time: 2020-07-09T08:31:49.942+0000

Duration: 24 min 22 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
10219

Skipped
7914

Total
18133

Steps errors
Expand to view the steps failures

Name: Install tools

Description: .\scripts\install-tools.bat

Duration: 0 min 3 sec

Start Time: 2020-07-09T08:39:10.005+0000

log

Name: Error signal

Description: hudson.AbortException: script returned exit code 1

Duration: 0 min 0 sec

Start Time: 2020-07-09T08:39:13.917+0000

log

Name: Archive the artifacts

Description: [2020-07-09T08:56:09.884Z] Archiving artifacts
hudson.AbortException: script returned exit code 1

Duration: 0 min 0 sec

Start Time: 2020-07-09T08:56:09.870+0000

log

Log output
Expand to view the last 100 lines of log output

[2020-07-09T08:55:11.131Z] tests/instrumentation/pymemcache_tests.py                    68     63      8      0     7%
[2020-07-09T08:55:11.131Z] tests/instrumentation/pymongo_tests.py                      219    214     12      0     2%
[2020-07-09T08:55:11.131Z] tests/instrumentation/pymssql_tests.py                       57     51      2      0    10%
[2020-07-09T08:55:11.131Z] tests/instrumentation/pymysql_tests.py                       34     30      2      0    11%
[2020-07-09T08:55:11.131Z] tests/instrumentation/pyodbc_tests.py                        51     47      2      0     8%
[2020-07-09T08:55:11.131Z] tests/instrumentation/python_memcached_tests.py              40     35      4      0    11%
[2020-07-09T08:55:11.131Z] tests/instrumentation/redis_tests.py                         81     79      4      0     2%
[2020-07-09T08:55:11.131Z] tests/instrumentation/requests_tests.py                      92     90      0      0     2%
[2020-07-09T08:55:11.131Z] tests/instrumentation/sqlite_tests.py                        83     77      4      0     7%
[2020-07-09T08:55:11.131Z] tests/instrumentation/transactions_store_tests.py           251    211     22      0    15%
[2020-07-09T08:55:11.131Z] tests/instrumentation/urllib3_tests.py                      107     94      8      0    11%
[2020-07-09T08:55:11.131Z] tests/instrumentation/urllib_tests.py                       129    109      8      0    15%
[2020-07-09T08:55:11.131Z] tests/metrics/__init__.py                                     0      0      0      0   100%
[2020-07-09T08:55:11.131Z] tests/metrics/base_tests.py                                 107     90     16      0    14%
[2020-07-09T08:55:11.131Z] tests/metrics/breakdown_tests.py                            180    168     58      0     5%
[2020-07-09T08:55:11.131Z] tests/metrics/cpu_linux_tests.py                             56     43     10      0    20%
[2020-07-09T08:55:11.131Z] tests/metrics/cpu_psutil_tests.py                            27     22      4      0    16%
[2020-07-09T08:55:11.131Z] tests/middleware/__init__.py                                  0      0      0      0   100%
[2020-07-09T08:55:11.131Z] tests/middleware/tests.py                                    32     25      2      0    21%
[2020-07-09T08:55:11.131Z] tests/processors/__init__.py                                  0      0      0      0   100%
[2020-07-09T08:55:11.131Z] tests/processors/tests.py                                   145    111     12      0    22%
[2020-07-09T08:55:11.131Z] tests/transports/__init__.py                                  0      0      0      0   100%
[2020-07-09T08:55:11.131Z] tests/transports/test_base.py                               161    128     14      0    19%
[2020-07-09T08:55:11.131Z] tests/transports/test_urllib3.py                            201    164      8      0    18%
[2020-07-09T08:55:11.131Z] tests/utils/__init__.py                                       2      1      2      0    25%
[2020-07-09T08:55:11.131Z] tests/utils/cgroup_tests.py                                   7      3      0      0    57%
[2020-07-09T08:55:11.131Z] tests/utils/cloud_tests.py                                   30     21      0      0    30%
[2020-07-09T08:55:11.131Z] tests/utils/compat.py                                         4      3      2      0    17%
[2020-07-09T08:55:11.131Z] tests/utils/compat_tests.py                                  36     19      2      0    45%
[2020-07-09T08:55:11.131Z] tests/utils/encoding/__init__.py                              0      0      0      0   100%
[2020-07-09T08:55:11.131Z] tests/utils/encoding/tests.py                               124     99      8      0    19%
[2020-07-09T08:55:11.131Z] tests/utils/json_utils/__init__.py                            0      0      0      0   100%
[2020-07-09T08:55:11.131Z] tests/utils/json_utils/tests.py                              26     14      2      0    43%
[2020-07-09T08:55:11.131Z] tests/utils/stacks/__init__.py                               10      7      4      0    21%
[2020-07-09T08:55:11.131Z] tests/utils/stacks/tests.py                                 127    100     10      3    22%
[2020-07-09T08:55:11.131Z] tests/utils/test_disttracing_header.py                       42     31      0      0    26%
[2020-07-09T08:55:11.132Z] tests/utils/tests.py                                         69     45      2      0    34%
[2020-07-09T08:55:11.132Z] tests/utils/threading_tests.py                               37     31      2      0    15%
[2020-07-09T08:55:11.132Z] tests/utils/wsgi/__init__.py                                  0      0      0      0   100%
[2020-07-09T08:55:11.132Z] tests/utils/wsgi/tests.py                                    47     32      0      0    32%
[2020-07-09T08:55:11.132Z] -------------------------------------------------------------------------------------------
[2020-07-09T08:55:11.132Z] TOTAL                                                     10831   8071   1759    134    24%
[2020-07-09T08:55:11.132Z]
[2020-07-09T08:55:11.132Z] ============ 2 passed, 27 skipped, 459 deselected in 49.78 seconds =============
[2020-07-09T08:55:11.394Z] + cleanup
[2020-07-09T08:55:11.394Z] + '[' -n 1 ']'
[2020-07-09T08:55:11.394Z] + ./scripts/docker/docker-summary.sh
[2020-07-09T08:55:11.394Z] *******************************************************
[2020-07-09T08:55:11.394Z] ***************Docker Containers Summary***************
[2020-07-09T08:55:11.394Z] *******************************************************
[2020-07-09T08:55:11.394Z] CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES
[2020-07-09T08:55:11.394Z] + PYTHON_VERSION=
[2020-07-09T08:55:11.394Z] + docker-compose down -v
[2020-07-09T08:55:11.967Z] Removing network tests_default
[2020-07-09T08:55:12.228Z] Removing volume tests_pypgdata
[2020-07-09T08:55:12.228Z] Removing volume tests_pymongodata30
[2020-07-09T08:55:12.228Z] Removing volume tests_pymongodata32
[2020-07-09T08:55:12.228Z] Removing volume tests_pymongodata34
[2020-07-09T08:55:12.228Z] Removing volume tests_pymongodata36
[2020-07-09T08:55:12.228Z] Removing volume tests_pyesdata7
[2020-07-09T08:55:12.228Z] Removing volume tests_pyesdata6
[2020-07-09T08:55:12.228Z] Removing volume tests_pyesdata5
[2020-07-09T08:55:12.228Z] Removing volume tests_pyesdata2
[2020-07-09T08:55:12.228Z] Removing volume tests_pycassandradata3
[2020-07-09T08:55:12.228Z] Removing volume tests_pymssqldata
[2020-07-09T08:55:12.228Z] Removing volume tests_mysqldata
[2020-07-09T08:55:12.228Z] + [[ -n '' ]]
[2020-07-09T08:55:12.373Z] Running in /var/lib/jenkins/workspace/thon_apm-agent-python-mbp_PR-887/src/github.com/elastic/apm-agent-python
[2020-07-09T08:55:12.782Z] + chmod 755 docker-logs.sh
[2020-07-09T08:55:13.097Z] + ./docker-logs.sh Python-pypy-3-graphene-2 docker-compose.yml
[2020-07-09T08:55:13.133Z] Archiving artifacts
[2020-07-09T08:55:13.179Z] Recording test results
[2020-07-09T08:55:13.915Z] Stashed 1 file(s)
[2020-07-09T08:55:16.511Z] Post stage
[2020-07-09T08:55:16.828Z] + pip3 install --user coverage
[2020-07-09T08:55:18.217Z] Collecting coverage
[2020-07-09T08:55:18.789Z]   Downloading https://files.pythonhosted.org/packages/db/19/ad8f558e885a11a68111300ee23a632cc46a5ef6aaed875ba56460076fa7/coverage-5.2-cp36-cp36m-manylinux1_x86_64.whl (229kB)
[2020-07-09T08:55:18.789Z] Installing collected packages: coverage
[2020-07-09T08:55:18.789Z] Successfully installed coverage-5.2
[2020-07-09T08:55:19.080Z] Running in /var/lib/jenkins/workspace/thon_apm-agent-python-mbp_PR-887/src/github.com/elastic/apm-agent-python
[2020-07-09T08:55:34.802Z] + python3 -m coverage combine
[2020-07-09T08:56:06.939Z] + python3 -m coverage xml
[2020-07-09T08:56:08.882Z] [Cobertura] Skipping Cobertura coverage report as build was not SUCCESS or better ...
[2020-07-09T08:56:08.882Z]
[2020-07-09T08:56:09.336Z] + curl -sLO https://code.jquery.com/jquery-3.3.1.slim.min.js
[2020-07-09T08:56:09.646Z] + curl -sLO<EMAIL_ADDRESS>[2020-07-09T08:56:09.884Z] Archiving artifacts
[2020-07-09T08:56:10.067Z] Stage "Building packages" skipped due to earlier failure(s)
[2020-07-09T08:56:10.184Z] Stage "Integration Tests" skipped due to earlier failure(s)
[2020-07-09T08:56:10.290Z] Stage "Benchmarks" skipped due to earlier failure(s)
[2020-07-09T08:56:10.512Z] Stage "Prepare Release" skipped due to earlier failure(s)
[2020-07-09T08:56:10.557Z] Stage "Prepare Release" skipped due to earlier failure(s)
[2020-07-09T08:56:10.657Z] Stage "Prepare Release" skipped due to earlier failure(s)
[2020-07-09T08:56:10.765Z] Stage "Prepare Release" skipped due to earlier failure(s)
[2020-07-09T08:56:11.134Z] Running on Jenkins in /var/lib/jenkins/workspace/thon_apm-agent-python-mbp_PR-887
[2020-07-09T08:56:11.307Z] [INFO] getVaultSecret: Getting secrets
[2020-07-09T08:56:11.403Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2020-07-09T08:56:12.360Z] + chmod 755 generate-build-data.sh
[2020-07-09T08:56:12.360Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-python/apm-agent-python-mbp/PR-887/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-python/apm-agent-python-mbp/PR-887/runs/1 FAILURE 1462147
[2020-07-09T08:56:12.610Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-python/apm-agent-python-mbp/PR-887/runs/1/steps/?limit=10000 -o steps-info.json


chore(main): release 0.8.14

:robot: I have created a release beep boop
0.8.14 (2023-12-08)
Bug Fixes

use signs pipe (b672784)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kevinah95/hacienda/releases/tag/v0.8.14 :sunflower:


[Bug]: Button variant used is incorrect on the Quick Page Wizard

Is there an existing issue for this?

[X] I have searched the existing issues

Description
On the quick page wizard, the Generate Page button variant used is incorrect, as per the design system specs (comparing also with Prod).
Steps To Reproduce
Try to create a page with the quick page wizard. Note the Generate Page button differs from what is used in Prod.

Public Sample App
No response
Issue video log
No response
Version
Instance

@btsgh could you re-check this and let me know if you're still seeing this? I'm not able to see the other variant on the instance. It looks same to me on prod and on instance.


Why does _roi_align.so appear some problems?

Traceback (most recent call last):
File "main.py", line 18, in
from misc import utils, eval_utils, AttModel
File "D:\PycharmProjects\NeuralBabyTalk-master\misc\AttModel.py", line 10, in
from misc.model import AttModel
File "D:\PycharmProjects\NeuralBabyTalk-master\misc\model.py", line 14, in
from pooling.roi_align.modules.roi_align import RoIAlignAvg
File "D:\PycharmProjects\NeuralBabyTalk-master\pooling\roi_align\modules\roi_align.py", line 3, in
from ..functions.roi_align import RoIAlignFunction
File "D:\PycharmProjects\NeuralBabyTalk-master\pooling\roi_align\functions\roi_align.py", line 3, in
from ..ext import roi_align
File "D:\PycharmProjects\NeuralBabyTalk-master\pooling\roi_align_ext\roi_align_init.py", line 3, in
from ._roi_align import lib as _lib, ffi as _ffi
ModuleNotFoundError: No module named 'pooling.roi_align._ext.roi_align._roi_align'

Hello @jiajunhua , did you solve this problem? I have met this issue just now.

Hello @jiajunhua , did you solve this problem? I have met this issue just now.


rpc 接口 不能继承父接口

Q
A

Bug report?
yes/no

Feature request?
yes/no

Swoft version
x2.0.6

Swoole version
4.4.12

PHP version
7.3.11

Runtime environment
Ubuntu

Describe the bug
rpc 接口 不能继承父接口

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Details

Describe what you are trying to achieve and what goes wrong.

// paste output here

Provide minimal script to reproduce the issue

// paste code

尝试支持下


Analyze exec engine support

This is starting of the work to build stats collecting and storing functionality to be merged into terrier so that the cost model can be used more effectively using real stats and the optimizer's effectiveness can be tested on Join Orderings. Also this functionality is required to have the optimizer support parameter value expressions such as while optimizing prepare statements before the first execute is called.
A decision was taken to merge the analyze function inside the execution engine and to do code generation for it. This PR adds aggregator support for topK elements in aggregators.h and provides 2(3 sub tests each) tests for it.

Re-open so that we don't lose track of this.

Closing to reduce Jenkins contention. We are definitely going to want to come back to this one.


Allow tab indentation

Hi,
This is more of a question than a bug. I'm wondering whether in the future the possibility to enforce tabs instead of spaces might be an option?
In this comment on issue #457, I see that your AST doesn't allow tabs but only spaces, but I couldn't find a dedicated issue for this problem.
Thanks

The latest version of the AST that we will be including in our 1.6 update 'should' support tabs properly now. As for enforcing tabs or spaces I think that would be fine but it's also a duplication of what you'd use a .editorconfig for so in the meantime you could just use that.
Definitely an option for a rule though. PR's welcome :+1:

Once tabs work, the current rule should work fine to enforce it

On Mar 31, 2016, at 10:16 AM, Dan Purdy<EMAIL_ADDRESS>wrote:
The latest version of the AST that we will be including in our 1.6 update 'should' support tabs properly now. As for enforcing tabs or spaces I think that would be fine but it's also a duplication of what you'd use a .editorconfig for so in the meantime you could just use that.
Definitely an option for a rule though. PR's welcome
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

That's only for mixed spaces and tabs though no? I think maybe @aberonni means a rule where you would define tabs as your choice and it would enforce it across all of your scss/Sass files.

Sorry if I wasn't clear. I think both of your last comments are correct.
So once tabs work, I'm thinking of something similar to eslint's rule:
indentation:
  - 2
  -
    size: tab

@aberonni this is working for scss syntax right now, So this should hopefully be with you for our 1.8 release which I expect to land in the next few days.
If you're using Sass syntax then it may be a little while longer but if i get time to look at it prior to 1.8 I will.

@DanPurdy
Any sign of this going into sass syntax?

@DanPurdy Same issue with 1.9.1 :(

@theskillwithin it should already be in. That comment wasn't updated once I'd actually worked on the rule. Due to a bug in the AST there are however some caveats which are explained in the original PR and possibly the documentation if I remembered to add it. If you're seeing any problems with Sass syntax could you report it in a new issue, thanks!


Add link libraries to generate_resource_spec cmake

Description
Adds -lpthread -lrt -ldl link library/options to the generate_resource_spec.cmake to resolve link errors when building the resource_spec generator on some systems.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

Looks good overall minus the quoting issue. In the long run, I'd like to see the resource spec generator built by CMake's build graph if possible - that's what CMake is for :)

We are due this year to bump the minimum required CMake version, and moving this to the build graph and leveraging the ability to generate the ctest spec at runtime is part of it :)

/merge


Feature request: Seal interface

I don't like @Builder much. It makes the .class bigger, it doesn't allow inheritance, the resulting code is short but not too nice.
Often the only argument for @Builder is that it ensures immutability, for which the only other option is all args constructor (which @Builder uses internally).
Hereby I suggest another approach, a "seal" interface - which would only have getters. It would work like this:
@Sealable
@Getter
@Setter @Accessor(chain = true)
class Foo { private int x; }

The @Sealable would create this:
interface Foo { public int getX(); }

class FooImpl implements Foo {
   private int x;
   ...
   public Foo seal() { return this; }
}

And could be used this way:
Foo foo = new FooImpl().setX(1).seal();

Obviously, the seal() method is not necessary, but creates dev-friendly way to get the type rather than having to look it up / write the type / change the type. Also, it could be overriden to perform some additional operations before returning the object (validation, ...).
Also, I assume it's not possible / advisable to swap the class names like this. It could possibly rather be the other way - class Foo and interface FooSeal, or maybe Foo.Seal as a inner type.
Foo.Seal foo = new Foo().setX(1).seal();

I think that this is doable by using @Value, @Wither and @AllArgsConstructor.

An annotation that changes class Foo into interface Foo and magically creates a class FooImpl whose source is in Foo.java would be, to me, extremely surprising behavior.  It's one thing to add nested classes, but please don't completely violate conventions for top-level types.

@OndraZizka I guess, a better starting point would be the interface and you'd just generate the immutable implementation of it. But that's actually simpler than what Lombok does, as it only requires creating a new class rather than modifying existing code. See AutoValue and FreeBuilder, if that's what you want.

I don't see this happening.

@wrprice  Thanks for the comment. Have you read the whole post? If not, let me point to the relevant part:

Also, I assume it's not possible / advisable to swap the class names like this. It could possibly rather be the other way - class Foo and interface FooSeal, or maybe Foo.Seal as a inner type.

Ok I admit I haven't put the idea the right way. Seems like the rejection isn't about the idea, but about the particular implementation I suggested.
Since I can't reopen this, I will reword the RFE in a new issue, unless @rspilker confirms that he doesn't like the idea of a Lombok feature to provide a method to "seal" an object (to return an immutable representation of it, whether it's a limiting interface, a proxy, an internal flag for setters...).

Thanks for the comment. Have you read the whole post?
If not, let me point to the relevant part ...

@OndraZizka, no, I read it.  My comment regarded the part(s) I find most objectionable; I meant what I said, and I said what I meant. (And no more than that.)  If you'd like to know my opinion on the portion you quoted, it is:

Your initial proposal probably would have been a better had you applied one of these alternatives rather than listing them as afterthoughts at the end.
That said, I also don't particularly care for those specific alternatives.
There probably is a reasonable approach; I'm not arguing against immutable objects in the abstract.

Finally, my opinion doesn't mean terribly much.  I'm just a user.


Replace frontend component from vis.js to antvis/G6

G6 Repo：https://github.com/antvis/G6

Regarding this issue, a PR #4  has been submitted. Please review these changes and let me know if further adjustments are needed.


CID 1375916 Missing break in TSHttpTxnConfigStringSet

Haha, just like #2044. But you missed the double break.

[approve ci]

This looks good to like #2044

[ci approve autest]

[approve ci autest]


feat[pie/donut]: custom tooltips for aggregate

Explanation About What Code Achieves:
This code fixes the issue I had with tooltips with pie charts and donuts charts, like I mentioned it here: frappe#304
Considering the fact that these kind of graph have only 1 value I choose the naming of the function as tooltipOptions.formatTooltip without X or Y like it's possible to do with other kind of graphs.
Screenshots/GIFs:

TODOs:

maybe it would be nice to document this in the docs with a simple example too?
might be usefuel for the "progress" chart too to anyone?

Any news on this @scmmishra, would you like me to edit something?

Hey @riccardogioratoatoms so sorry for the delay. I've been very busy lately with a few stuffs, I'll definitely review this tomorrow. On the surface it looks good to go, there won't be any changes required it seems.
I'll update the documentation for it as well.


Something wrong with paginate.getArrayPages

When I exec the following code in my express demo
var pages = paginate.getArrayPages(req)(3, 2, 2);
pages.forEach(function(page) {
    console.log(page);
});

I get the following ouput
 { number: 0, url: '/products?page=0&limit=10' }
 { number: 1, url: '/products?page=1&limit=10' }
 { number: 2, url: '/products?page=2&limit=10' }

@IchiNiNiIchi does this resolve your issue? http://stackoverflow.com/questions/39676397/error-express-paginate-limit-is-not-a-number-0

@IchiNiNiIchi does this resolve your issue? http://stackoverflow.com/a/39689389

@niftylettuce thank you for your reply. My question is the result include a "page 0" when I call paginate.getArrayPages(3, 2, 2). Finally I get around this by changing
 var start = (currentPage < (limit - 1)) ? 1 : (end - limit) + 1;
to
 var start = (currentPage <= (limit - 1)) ? 1 : (end - limit) + 1;
in https://github.com/expressjs/express-paginate/blob/a34a719fcc308bd8d82b46a9d51dc3bf05faae09/index.js#L72
but I don't know does the change works for other input.

This is still an issue. Changing the code in local copy cannot be a solution. Please reopen this bug.
@niftylettuce , SO Link does not fix anything.

@rajivnarayana I reopened the issue - just saw this now

feel free to submit a PR!  I don't use Express anymore - would you like to help maintain?

Will give this a try and submit a PR. It will fix an outstanding bug in one of our projects.
Would like to maintain this plugin!

Great @rajivnarayana!  Added you as a collaborator.  Once your code is in ping me and I will release a new version.  When I'm comfortable I'll add you as an NPM owner!

Thanks @niftylettuce for merging v0.3.0. This issue should now be fixed and could be closed.


Reimplementation of known examples from other packages

Following issue #5 it would be interesting to see if we can already reimplement a couple of well-known examples from other packages, for instance Keras?

We have MNIST, but not anything convolutional or CIFAR.  These are important to have in papers comparing NMT behavior vs other tasks.

Tomasz is working on convolutions in convnet branch. This will soon be added. I close this issue as it was sort of inactive. I would actually plan to reimplement both, the facebook convnet and the google transformer stuff.

Maybe it would make more sense to have specific issues for concrete models. This one is about all and nothing.

So, I am closing this one again. And if you have needs for a specific model or an interesting idea, open a new issue for that.


DES: Added Data Encryption Standard in README.md

Have you read the Contributing Guidelines on Pull Requests?
Yes
Checklist

[X] I've read the contribution guidelines.
[X] I've checked the issue list before deciding what to submit.
[X] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Issue Number: #1553

@HarshCasper please review the pull request once, if any changes/improvements/modifications needed please let me know about the respective.
Sequence Number: 13
Issue Number: #1553
Pull Request Number:  #2090

@HarshCasper please review the pull request once, if any changes/improvements/modifications needed please let me know about the respective.
Sequence Number: 13
Issue Number: #1553
Pull Request Number:  #2090

Please have a look at my review here and make the same changes: https://github.com/TesseractCoding/NeoAlgo/pull/2078

@PuravUdayDesai Are you still working on this ?

@PuravUdayDesai Are you still working on this ?

Sorry @atarax665 I just saw the comments and changes requested today, I'll update the following as soon as possible by tomorrow.
Really sorry for keeping you waiting

Closing this PR due to inactivity.

@atarax665 please reopen the pull request


Got Multiple Styles to work in MPS

I was doing bad things.

Fixes #2723

Overview
Multiple styles wasn't working in TuriCreate because of this silly problem.
Issue

The instance norm layers which determine the stylistic feature of an image are supplied by a class called the InstanceNormDataLoader. This data loader takes care of all of the beta and gamma weights during inference as well as the velocity and momentum buffers during training.

Two methods exist that need to be overridden on this data loader: the gamma and the beta method.

The instanceNorm layer in mps only supports one style meaning one gamma and beta pair.

To circumvent this limitation, in the InstanceNormDataLoader class we created NSArrays to store the different gamma and beta values.

We used an indexing variable styleIndex to select the different gamma and beta values for different styles.

To export the weights after training I added a concat to the gamma and beta methods so that all of the gamma and beta weights for all styles are concatenated into one contagious block of NSData

This worked during export because the part of the style transfer application TuriCreate owned was aware that these were concatenated gamma and beta values.

this didn't work during predict because these concatenated values were truncated by MPS since they had no awareness that these exported gamma and beta values were for multiple styles.

Since the gamma and beta values were contagious the truncation happened to the gamma and beta values for the first index only.

The Solution

Separating the methods that exported the gamma and beta weights and the methods that gave the current gamma and current beta weights for the selected style, solved this issue.

calling gammaWeights and betaWeights gave the callee the concatenated values of gamma and beta weights for all styles

calling gamma and beta only gave the gamma and beta values for the current selected style.

In conclusion MPS was happy and TuriCreate exported all the weights it needed for multi style style-transfer.


Doesn't seem like the JVM arguments make their way into the Application.

When trying to run the test using GuiceOneAppPerSuite doesn't seem like the JVM arguments set in Build.scala are being passed into the Application instance of the JVM.

         implicit val ec = scala.concurrent.ExecutionContext.global
         val cc = Helpers.stubControllerComponents()

         val controller = new Model(cc, new Authenticated(cc.parsers.defaultBodyParser))
         val result: Future[Result] = controller.getPage(1L).apply(FakeRequest())

@dk8996 I'm not sure I understand what the pasted code is demonstrating.
Also, are you running the tests in forked JVMs?


Add webpack to the yo extension generator

I have generated a new extension using yo code. I would have expected for the generator to generate a state of the art configuration, including with webpack.

@jrieken @alexandrudima
Is this something we want to recommend to all users? Or should there be an opt-in option to make it webpacked?

I would make it a question in the flow, but one which is default selected


Unhandled Exception: MissingPluginException(No implementation found for method currentUser on channel plugins.flutter.io/firebase_auth)

I'm facing this issue after upgrading Flutter to 1.17, it worked fine on 1.12..
E/flutter (15385): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: MissingPluginException(No implementation found for method currentUser on channel plugins.flutter.io/firebase_auth)
E/flutter (15385): #0 MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:154:7)
E/flutter (15385):
E/flutter (15385): #1 MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:329:12)
E/flutter (15385): #2 MethodChannel.invokeMapMethod (package:flutter/src/services/platform_channel.dart:356:48)
E/flutter (15385): #3 MethodChannelFirebaseAuth.getCurrentUser (package:firebase_auth_platform_interface/src/method_channel_firebase_auth.dart:43:10)
E/flutter (15385): #4 FirebaseAuth.currentUser (package:firebase_auth/src/firebase_auth.dart:316:45)
And this Note as well
Note: C:\flutter-sdk\flutter.pub-cache\hosted\pub.dartlang.org\cloud_firestore-0.13.5\android\src\main\java\io\flutter\plugins\firebase\cloudfirestore\CloudFirestorePlugin.java uses unchecked or unsafe operations. Note: Recompile with -Xlint:unchecked for details.
pubspec.yaml
environment:
sdk: ">=2.2.0 <3.0.0"
dependencies:
flutter:
sdk: flutter
firebase_core: ^0.4.0+9
firebase_database: ^3.1.3
firebase_auth: ^0.16.0
cloud_firestore: ^0.13.5
google_sign_in: ^4.0.4
firebase_messaging: ^6.0.13
http: ^0.12.1
Flutter Doctor
C:\flutter-sdk\flutter\bin\flutter.bat doctor --verbose
[√] Flutter (Channel stable, v1.17.1, on Microsoft Windows [Version 10.0.18362.836], locale en-IN)
• Flutter version 1.17.1 at C:\flutter-sdk\flutter
• Framework revision f7a6a79 (5 days ago), 2020-05-12 18:39:00 -0700
• Engine revision 6bc433c6b6
• Dart version 2.8.2
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at C:\Users\xxxxx\AppData\Local\Android\sdk
• Platform android-29, build-tools 29.0.2
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
• All Android licenses accepted.
[√] Android Studio (version 3.6)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin version 45.1.1
• Dart plugin version 192.7761
• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
[√] Connected device (1 available)
• Nexus 5X • 01fxxxxxxxxd69 • android-arm64 • Android 8.1.0 (API 27)
• No issues found!
Process finished with exit code 0
Everything seemed to work perfectly fine before and I did not change any dependency after upgrading flutter. Kindly review this as let me know what's going wrong

Found a solution here! But not sure if that's a solution or a workaround. Helped anyway!
https://github.com/flutter/flutter/issues/56548#issuecomment-625239992


Convert to xunit

This takes the rest of the tests to xunit.  It builds but some tests are definitely failing.  I mostly just want to see the test report from the PR build :smiley:
I expect to rebase/squash/generally rewrite history before I'm done here, so don't base any changes off these commits.

Do we still need Microsoft.VisualStudio.QualityTools.UnitTestFramework reference in *.csproj file?

That's a lot to look through, but looks good! Had one comment, and yeah what Maxwe11 said. Can't wait to get this into xplat so we can have one test framework!

Ok, CI build is passing now, if maybe not reliably.  I suspect we'll be hunting bugs and unreliabilities in tests for a while :persevere:.
@Maxwe11 @AndyGerlicher I think I addressed your concerns.
All: I know this is huge, but can you take a look now?

In TaskRegistry_Tests.cs, what's the reason for compiling an assembly on the fly instead of just building it as part of the test build?

In AssemblyTaskFactory_Tests, xUnit's Assert.IsNotType has a different behavior.  It checks if the types are exactly identical, not whether the type is assignable from type of the object.

Looks like several of the commits for the automated conversion to xUnit have the wrong first line for the commit description: Convert Utilities unit tests to xunit

Left some comments but LGTM! :shipit:

LGTM

Filed #211 to stop building things unnecessarily in our unit tests (@dsplaisted).

@dsplaisted I couldn't see any place where duplicate node request IDs would be interesting, and it doesn't seem to have caused any functional problems.  Leaving as is for now.


"Călugăreni", "Valea Scheilor" are listed in the drop-down menu

Description:
When user writes ""Călugăreni" and "Valea Scheilor" the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Călugăreni"
Expected results
The location appears in the drop-down menu as "Sat Călugăreni, PRAHOVA (Călugăreni)"

Step 2
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Valea Scheilor"
Expected results
The location appears in the drop-down menu as "Sat Valea Scheilor, PRAHOVA (Călugăreni)"


GH-2588: ARBP support batch recoverable

Resolves #2588
This is a draft of support DLT batch recover for DefaultAfterRollbackProcessor.

add opt-in property batchRecoverAfterRollback at ContainerProperties
add method processBatch at AfterRollbackProcessor
deprecated setTransactionManager(), getTransactionManager(), transactionManager at ContainerProperties

batch recoverable

batchRecoverAfterRollback is true
set transcation DLT to enable isCommitRecovered
enable kafka transcation

exception handler logic

in retry, seek to begin
out of retry exception recover entire recods
recover fail, seek to begin

Can you provide a valid use case where this would be useful?
Any transactional sends for the records before the failed one would have been rolled back, so it doesn’t make sense to to skip those records on the retry. If you are not doing transactional sends, then why use transactions?
I tried to convey this in my comments on the issue.

Any transactional sends for the records before the failed one would have been rolled back, so it doesn’t make sense to to skip those records on the retry. If you are not doing transactional sends, then why use transactions?

Exactly right, this is the most important point in this issue.
I had a rethink in the BatchListenerFailedException sence. Simple implementation is

in retry range that seek to reocrds begin offsets
retry exhaustion that return remaing record exclude recovered records.

One more time - we cannot support BatchListenerFailedException here - the transaction will have rolled back by now so any transactional sends for the records before the failed record will have been ignored; it is not correct to discard those records.
Yes, we could support recovering (sending to DLT) the entire batch, but we can't discard a partial batch here. Even that would have to be an opt-in feature.

Hi, @garyrussell
Could you do me a favor for build failed?
I didn't modify code at GlobalEmbeddedKafkaTestExecutionListenerTests#testGlobalEmbeddedKafkaTestExecutionListener
and ran ./gradlew check success locally.
but build failed and following exception is reported

Re-running now; that test is sometimes flaky.

Thanks

@Wzy19930507 ,
please, rebase your branch to the latest main.
Then we will start looking into your change to incorporate it into a new version.
Thanks

@artembilan @garyrussell @sobychacko Thanks for review, this issue could not have been closed without your guidance.


K8S 1.25 Update

PR to Bump K8s version to 1.25 and golang to 1.19
Additionally bumped versions for the linter and underlying support packages to be compatible with the latest golang.

So I think the sanity check in the test action is failing because of the testdata not matching up. I think doing a make generate should fix this

So I think the sanity check in the test action is failing because of the testdata not matching up. I think doing a make generate should fix this

That's exactly hat it was, but Ish' go was all jacked up. He had to reinstall :)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.223%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1631

Relevant Lines:
1828

💛  - Coveralls


Field "OR" is not defined by type for $where

I'm following this tutorial: https://www.howtographql.com/graphql-js/8-filtering-pagination-and-sorting/
I'm using latest prisma (1.26) install locally in docker container with connected to MongoDB

This is similar issue that was closed 2874 without any proper solution

MongoDB doesn't support OR right now as it is in preview.
Closing as a duplicate of https://github.com/prisma/prisma/issues/3897

@pantharshit00 Okay, thanks!


Error: No credentials at GraphQLAPIClass, I suddently can't do any graphql request for no reason

Hi everyone,
Today I restarted my computer and had a good surprise, I couldn't do any graphql operation. I had the following issue, anytime I did a graphql request, wether with Cognito or IAM access on my web app :
Error: No credentials
    at GraphQLAPIClass.<anonymous> (GraphQLAPI.ts:138)
    at step (tslib.es6.js:237)
    at Object.next (tslib.es6.js:237)
    at fulfilled (tslib.es6.js:237)

Before restarting my machine everything was working well, and sudently nothing works.
I tried :

Reinstall amplify libraries for reactjs
update CLI
reinstall npm
update my graphql and push it
remove auth and recreate it (because my auth had dependencies with graphql subscription I can't delete it so I feel I will be up for a nice "amplify delete" thanks god I'm not in production yet...)

I tried the solution proposed there : https://github.com/aws-amplify/amplify-js/issues/3489#issuecomment-504290892
replacing Amplify.configure(awsExports); by
Amplify.configure({
  ...awsExports,
   Analytics: {
       disabled: true
   }
})

Because I have deadlines I will remove my amplify project and restart it again, as I always do, but I wanted to let you aware that this can happen to you any day of your amazing weekend.
The problems seems that the credentials for graphql access are not generated, I don't really understand how this works...
The problem is only on my web app, everything works normally on APPSync
I'm working on macOS with amplify CLI v 4.44.0
EDIT : As I said I deleted all my amplify project and this time my problem is still there... It might me related to some libraries I installed from npm.

The problem seemed to come from the CLI. Simply reinstalling it didn't work for me but running the following commands to clean the CLI before reinstalling it solved my problem :
rm -rf package-lock.json yarn.lock node_modules
yarn add aws-amplify@latest

Hope this hepls.