Revert "webkitgtk: 2.34.6 → 2.36.0"

Description of changes
This reverts commit e2b791c31db2cff9964a2d5a3141e0fef7563000, since it broke reports in gnucash. That is, all reports simply showed a blank screen.
CC @jtojnar
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

We cannot revert since the update is security critical (fixes some CVEs). As a workaround, you can set WEBKIT_DISABLE_COMPOSITING_MODE=1 to disable hardware accelerated compositing on the platforms that are buggy.

We shouldn't break stuff on the stable branch, and I doubt gnucash's usage of webkitgtk as a renderer is very security-critical.
The hardware is nvidia btw, not exactly an obscure platform that we can just break.

We shouldn't break stuff on the stable branch.

Agreed, but we cannot just leave webkitgtk insecure since it is used in untrusted contexts (e.g. web browsers).

So we would either need to set

environment.variables.WEBKIT_DISABLE_COMPOSITING_MODE = "1";
in some NixOS module,

Pass the variable to all apps using webkitgtk (see e.g. PR that disabled the HW-accelerated compositing in Evolution e-mail client.

Re-introduce the insecure webkitgtk and have packages that work with HTML that breaks the compositing use that.

Yet another alternative to environment variables is patching WebKit to disable hardware acceleration as proposed in https://github.com/NixOS/nixpkgs/pull/169058

Right, forgot about that. Though that will likely make element-heavy sites like YouTube unusable in WebkitGTK-based browsers.

if gnucash is the one of the exceptions, we can just wrap it so that the application has WEBKIT_DISABLE_COMPOSITING_MODE = "1" before running

Unfortunately webkitgtk 2.36.1 does not fix the problem. However, the WEBKIT_DISABLE_COMPOSITING_MODE=1 workaround does work, so I'll use that.


Packaging

I think this is good enough to test in practice.
Changes include

add poetry for dev environment management
add nox, flake8, black, isort, etc for development
update all existing files according to flake8 reports
add simple test to ensure preservica developed scripts are compatible with new versions of python

Suggested workflow in this new structure

develop tests and module
If new packages needed, poetry add python-module if new packages needed
Test with poetry run pytest tests/new_test.py
Once feature is complete poetry run nox
Adjust according to output
Git add, commit, push

nox currently runs black, isort, flake8, pytest, and coverage. It doesn't run mypy but that can be run with poetry run nox -rs mypy. Still need to decide if these are the right defaults.


Doesn't work with webpack-dev-server live update

This is for http://localhost:****/webpack-dev-server/, where the page is loaded to an iframe and reloaded on file changes.
Vue DevTools don't see Vue, as it is in the iframe.

You can use webpack-dev-server without the iframe, just go to the root URL instead.

Yes, but it won't live-reload on changes then.

It would...

Ok, thanks!


Rendering is off in PR reviews

When trying to add a gif to a PR review, the following is rendered:
Initial state:

After clicking GIF button:

Browser version: Version 120.0.6099.199 (Official Build) (arm64)
Extension version: <IP_ADDRESS>4
Possibly related to the same underlying problem as #68?

Hey, thanks for submitting an issue!
I can't seem to reproduce this unfortunately, but I have a hunch, and made a PR that should hopefully fix this for you here: https://github.com/N1ck/gifs-for-github/pull/71
I have submitted the updated extension for review, it should be version number 24.1.8 when it's active. If you could try again when it's updated and let me know that would be great.

Thanks for the quick and friendly response!
It's still broken in my end unfortunately. Works as expected for gifs in regular comments, but for PR reviews it looks as pictured above. Strangely enough, in PR comments it works as expected, only when adding the final verdict things are off.
I noticed similar behaviour when narrowing the screen to <1112 pixels, i.e. when the GIF-button gets out of bounds, maybe that's easier to reproduce?
Width = 1112px

Width < 1112px (expanded)

Width < 1112px (minimized)

No sweat if this isn't a prioritised bug, workarounds are available after all (e.g. finding the perfect gif in a regular comment and then copying the link), just happy to report a finding for my legitimately no. 1 most used Chrome extension. 👏

For me it is not fixed on 24.1.8 regardless of the screen width. It is worth noting that I'm also using Refined Github extension, but it is not fixed even if I disable the extension.
Refined Github on:

Refined Github off, default state:

Refined Github off, opened state:

I've hit this too, but can work around it if I increase the vertical height of the comment textarea. Not sure why that is relevant though

Thank you for all of the screenshots and helping debug 🎉

Sorry I haven't had a lot of time to look at this, after further digging I was finally able to reproduce it.
It wasn't happening for me because I had my scroll settings set to:

But when changing it to "Always" I could reliably replicate it.
I think I have a fix now which should be in version 24.1.16

GitHub has made another, unrelated change to their layout which has caused it to break again 😭 I have already fixed it but have to wait for another approval before it's live.

The latest version 24.1.18 should resolve this issue. Sorry it took so long, it took over 2 days to be reviewed 😢

Confirmed!


Update README.md: gym also builds OS X projects 💪

Changelog:

gym builds OS X projects perfectly fine, so I'd be nice to have that in the README.md.

Thanks @tjosten :+1:


Performance on android is slow :(

Yeah, it takes about 20 seconds to open lightbox on android when I test with Expo, I see every frame, fame time is more than 1 second. That's unusable at all.

For anyone stumbling into this, try to go to react native developer menu and disable debugging (shacking the device or in the case of an emulator ctrl+m). I've had the issue of some animations being slow while debugging and this one appears to be due to the same reasons. I don't consider this a major issue since this will never happen on production builds


More byte digests

Improve and some back-and-forth transformations between Byte and Text.
var b = "abcd".to_bytes
assert b.chexdigest == "\\x61\\x62\\x63\\x64"
assert b.chexdigest.unescape_to_bytes == b
assert b.binarydigest == "01100001011000100110001101100100"
assert b.binarydigest.binarydigest_to_bytes == b

assert "1".binarydigest_to_bytes.hexdigest == "01"
assert "1 0 1".binarydigest_to_bytes.hexdigest == "05"
assert "7".hexdigest_to_bytes.hexdigest == "07"
assert "a B cd \\xe".hexdigest_to_bytes.hexdigest == "0ABCDE"

FFFFUUUUUUU literal char-like bytes are not supported by the booststrap compiler.

Update: make Text::hexdigest_to_bytes more permissive
Reroll: use hexanumerical byte literals

Seems good, :shipit: !


Multiple entries stopped working

Bug
What is the current behavior?
With Webpack 3.0 this worked well:
var glob = require("glob");

module.exports = {
	entry: {
		tests: ["./tests/testBootstrap.ts"].concat(glob.sync("./tests/**/*.ts"))
	},
...}

You get
[10:47:32] WebpackOptionsValidationError: Invalid configuration object. Webpack has been initialised using a configuration object that does not match the API schema.
 - configuration.entry should be one of these:
   object { <key>: non-empty string | [non-empty string] } | non-empty string | [non-empty string] | function
   The entry point(s) of the compilation.

However using just glob.sync works, but I need the testBootstrap running before other files.
var glob = require("glob");

module.exports = {
	entry: {
		tests: glob.sync("./tests/**/*.ts")
	},
...}

If the current behavior is a bug, please provide the steps to reproduce.
See the code above.
What is the expected behavior?
If there is an alternative syntax, I would like to know.
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Node latest
Webpack 3.5.2

The issue is with webpack-stream package upgrade.


[Resource] Drawable scaling from other density-qualified resource

As Android Devices Compatibility guide shows,

To provide good graphical quality on devices with different pixel densities, provide multiple versions of each bitmap in your app—one for each density bucket, at a corresponding resolution. Otherwise, Android must scale your bitmap so it occupies the same visible space on each screen, resulting in scaling artifacts such as blurring.

Which indicates that if drawable with qualifier matches current screen is not found, Android will fetch drawable with other qualifier and scale it to fit.
In our prectice, we only provide drawable-xxhdpi to keep the product apk small. But in the generated compose resources, it just fail if the best-match drawable not found.
IMO, it would be nice to have features that:

Scaling drawable from other density-qualified resource if best-matched not found,
Attaching qualifiers info in the resources returned by Res.

Thanks for this feature request! We will discuss it.

Could you elaborate why the default drawable dir is not suitable for you?

Your question bring me to a rethought of my usage.
In some of our usages, we don't specify the size of drawable or ImageView, just set it to wrap_content and apply a drawable to it. The drawable itself, with size in pixel and density qualifier info can determine the size in Dp. But in compose resources, the size modifier will be required for variant devices density.
That is what I requested in feature 1. But I am not sure if it is a good prectice in UI coding.
Anyway, feature 2 will be helpful not only in this case, but also any other if more customization needed.

I need more examples for "if more customization needed". I guess it should be covered by https://github.com/JetBrains/compose-multiplatform/issues/4197

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Fully qualify directory for correct placement

In the POST INSTALL section, the cp commands should probably have the source directories fully qualified (can be obtained at startup of the script).  Doing this script manually as I did have the risk of not being in the right place (working directory).  Similarly, if someone changes the script elsewhere in the future.  I guess you know that 'ressources' is misspelled, but that may be hard to change.

No problems if no changes in directory structure after git clone ....
I installed from /tmp, from /root - no issues.
Script "knows" the path to the ressourses or any other directory, because these path is RELATIVE from "script root" - where script was started.


Analytics on Google searches

We learned from analytics analysis that 22% of all traffic to the BC Gov site hosting the public data came from Google. What keywords are people using to look up our data? And how often are these searches occurring? This ticked documents an exploration into commonly googled searches related to our data.
Acceptance Criteria:

[ ] Consolidate common keywords from site analytics in Miro

[ ] Research how often the following searches are performed:

BC Emissions Data

BC GHG Data

BC Industry Emissions

Emissions Data in BC
(Etc! We can get creative.)

[ ] Document findings in Teams channel

Notes:

Currently, the only known tool we have is Google Trends which only works for searches that are more popular. It's better than nothing, but won't yield very detailed data (and likely only for some searches)
WordTracker also allows 10 free searches/month

Good note, thanks @LindsayMacfarlane! We did get some of this information from the analytics report.  It doesn't tell us how often those terms are looked up but it does tell us which keyphrases are common to get to the webpage. I'll add in a step in the acceptance criteria to bring in this relevant info.

Removed from Sprint 86 (July 25 - Aug 5) based on re-prioritization conversation with @jaimiebutton.

Note: not completed during initial research and discovery phase. Removed from Internal Research EPIC to capture in potential future work. Moved ticket to icebox.


Modify measure module to support updated env and config json files format

Some important modification in env.json

"info.cache" -> "griffin.checkpoint"
"persist" -> "sinks"
"log" -> "console"
"http" -> "elasticsearch"
remove "cleaner"

Some important modiffication in dq.json

add "baseline" in data source
add "dataframe.name" in data connector
in data source,"cache" -> "checkpoint"
in rule and pre-proc rule, add "in.dataframe.name" and "out.dataframe.name", remove "name"
in rule, add "out" param array, move "metric", "record" param inside "out" array
add "sinks" string array as filter of sinks in env.json

reviewed.
We need to update service and ui , and verify on dev branch first.
Thanks,
William


Add step definitions needed for enable / disable buttons on Project Setup module

Implemented step definitions for enabling/disabling surveys and longitudinal mode, see commit: https://github.com/cdebacker/redcap_cypress/commit/dbe907d58d051613378502ece6b890a509afaefd
Currently working on step definition for clicking on checkboxes selected by their proximity to matching text, which will be used for configuring project setup within the "Additional customizations" dialog box.

I click on the checkbox nearest the text {string}

For other configurables, I think it would be best to take a modular approach that involves more explicit Gherkin, like:

I click on the button nearest the text "Designate an email field for communciations"
I select "email_v2" from the dropdown nearest the text "Choose an email field to use for invitations to survey participants:"

... instead of a "macro" approach:

I designate the field "email_v2" for email communications

While the first approach isn't always as concise, it eliminates ambiguity of what user input actions are actually being taken, and reduces step definition bloat, which should help Gherkin authors (fewer steps to memorize or search through) and improve code maintainability. Since you mentioned that it's preferable to have to update feature files over step definitions, I think it's best to have the step definitions be more abstract and the Gherkin more concrete in this regard, rather than vice-versa. Let me know what you think about this. It might be worthwhile to create another issue to discuss and document design decisions of this sort, but I'll leave that at your discretion. @aldefouw

@cdebacker  - I agree with most of your assessment.
I definitely think the "macro" approach would be a mistake.  If I'm understanding your approach, you'd have to create a separate step definition for every unique field (e.g. "email communications").
One thing I'd like to mention though ... for the dropdowns, I'd like to see an approach of selecting the option by the value that visible to the end user rather than the "value" attribute specified in the option HTML.
In other words, I'd like to see you select the option by Y rather than x:
<option value="x">Y</option>
Side note ... would your method supersede what Tintin did or would it supplement what he did?
https://aldefouw.github.io/redcap_cypress/#i-select-string-from-the-dropdown-identified-by-string-labeled-string
(Also noticing that Tintin and Rushi have almost the same step definitions .. so we'll need to consolidate.)

@aldefouw I believe it would only supplement Tintin's step definition, since mine would achieve the same result but without using selectors, which per my understanding, we are trying to get away from. I say supplement because it might be useful to keep his around as a backup option for development/testing purposes. My goal is to get this approach to work for any situation that isn't already covered by other step definitions, and by adding in "n-th selector" support, it should cover the vast majority of cases (though it might occasionally require some trial and error from test writers, trying different values of n).
For others, we might still need a specific step definition, e.g. if the email communications field dropdown didn't have any nearby text, we could just make an exception by having a step definition 'I designate the field "email_v2" for email communications. If these exceptions become excessive, we can rethink ways to generically identify elements. In the worst case scenario, a simple 'I select  from the  dropdown' would work, but I'm thinking we can do better if it becomes necessary.


Event Hubs Readme issues

Section Link:

Suggestion:
a.  Add hyperlink for Including the package https://github.com/Azure/azure-sdk-for-java/tree/master/sdk/eventhubs/azure-messaging-eventhubs#include-the-package
b.  Delete Default SSL library
2.
Section Link:

Suggestion:
Add hyperlink for  Java Development Kit(JDK)  https://docs.microsoft.com/en-us/java/azure/jdk/?view=azure-java-stable
@jongio for notification.

Looks like the Default SSL Library link was already removed long ago; and @conniey recently fixed the package link https://github.com/Azure/azure-sdk-for-java/commit/5fa67a5c18d619f83e6a15e992c64f780627232a#diff-c582aeac546cea0e843017227f72b2d3


Reading ESRI GeoDataBase using 'OpenFileGDB' results in Python Crash

I need to read a geodatabase feature layer into fiona, however everytime I do so, it crashes python. Is this a bug?
dir_temp = 'E:\GIS_Data\Final_Neighbourhoods.gdb'
dir_data = (dir_temp.replace("/","\\"))
#use fiona to read the GDB and its layers and return a list
if feature_type == 1:
    layer_list = []
    with fiona.drivers():
        print "The script found the following layers in the GDB: "
        print "\n"
        for i, layername in enumerate(fiona.listlayers(dir_data)):
            with fiona.open(dir_data, layer=i) as src:
                layer_list.append(layername)
                print (i + 1), " - ", str(layername)
    print "\n"
    #print layer_list
    layer_choice = layer_list[3]
    print "Check1"
`with fiona.open(dir_data, driver="OpenFileGDB") as src:
    print "Check2"
    print src`

So the script prints the list, prints 'Check1' and then crashes.
FYI, I am actually trying to use geopandas (which uses the fiona file read) to manipulate the attribute columns in the feature layer inside the GDB. But I am battling to get past the reading of the GDB layer. I realise that input is a ESRI FileGDB, and not sure if this is the problem, but I'd like to solve this issue without having to install/copy the ESRI FileGDB.dll manually.
Please assist. The file in question is 200mb so let me know if you would like me to share via dropbox.
Thanks in advance

"Crashes" is not enough information @jens2121. Without the Python traceback, we have nothing to go on. Additionally, we need the following information: what is your operating system (Windows maybe?), what is the Fiona version, and from where did you get the Fiona package that you installed?

Apologies @sgillies. Not sure if there is a way to get a error report when there is no traceback? If so then please advise and I will do so.
System: Windows 10
Python: 64-bit Python 2.7.12
Fiona version: 1.7.4
GDAL installed: v2.1.3
Thanks for the quick response!

If there's a Python crash with no traceback, this is because the Fiona extension modules, GDAL DLL, and OpenFileGDB driver are not compiled in a compatible manner. I'm sorry, but I can't fix this. You'll need to report this issue to the maintainer of the Windows packages you downloaded, whether that's http://www.lfd.uci.edu/~gohlke/pythonlibs/#fiona or Anaconda or whatever.

I've used conda install fiona to install fiona. Would you recommend installing another windows package, or perhaps a previous version of fiona? Could you give your recommendation on this? thanks @sgillies

Can you share the geodatabase @jens2121?

@jens2121 OpenFIleGDB is working for other users, so I must ask you to take this issue to one of the
following trackers depending on whether you installed using the conda-forge channel or not:

https://github.com/ContinuumIO/anaconda-issues/issues
https://github.com/conda-forge/shapely-feedstock

Of course @snorfalorpagus, it is approximately 200mb.
https://www.dropbox.com/sh/gsabvammp5g6ob2/AAAewYayeiVIz91gUfBQtdQma?dl=0

@jens2121 For what it's worth, I have no problems reading that geodatabase using Python 3.6 on macOS with Fiona installed from conda-forge. As @sgillies suggests you should post an issue on the relevant tracker. If you're not already, you should check out conda-forge: https://conda-forge.github.io/#about

I can reproduce this issue with binaries from http://www.lfd.uci.edu/~gohlke/pythonlibs.
Only the 64-bit builds of Fiona using the OpenFIleGDB driver crash when reading Final_Neighbourhoods.gdb:

fiona using the OpenFIleGDB driver segfaults in all 64-bit builds.
fiona using the OpenFIleGDB driver works in all 32-bit builds.
fiona using the ogr_FileGDB plugin works.
ogrinfo.exe using the OpenFileGDB driver works in all builds.
osgeo.ogr using the OpenFileGDB driver works in all builds.

@cgohlke  thanks for checking and confirming where the crash is coming from. @snorfalorpagus, thanks for checking.

@cgohlke can you be more specific about "when reading?" Does it crash on open()? On iterating over features? In @jens2121 original bug report, there was no mention of iterating over features, only calls to listlayers() and fiona.open().

I am using the same test code as OP. The crash is in fiona.open. Unfortunately enabling faulthandler does not give any traceback either. I'll do a debug build of Fiona and report the stack trace...

Here's the call stack (I don't have debug builds of GDAL):
 	ntdll.dll!00007ffb574d8283()	Unknown
 	ntdll.dll!00007ffb574d8bda()	Unknown
 	ntdll.dll!00007ffb57485b9a()	Unknown
 	ntdll.dll!00007ffb5741cf93()	Unknown
 	AcLayers.dll!00007ffb20d97583()	Unknown
 	[External Code]
 	gdal201.dll!00007ffb1c180c42()	Unknown
 	gdal201.dll!00007ffb1b03f144()	Unknown
 	gdal201.dll!00007ffb1c170a22()	Unknown
 	gdal201.dll!00007ffb1c17084b()	Unknown
 	gdal201.dll!00007ffb1c07e83b()	Unknown
 	gdal201.dll!00007ffb1bf59e79()	Unknown
 	gdal201.dll!00007ffb1bf59bd4()	Unknown
 	gdal201.dll!00007ffb1bf5aa9c()	Unknown
 	gdal201.dll!00007ffb1bea28c6()	Unknown
 	gdal201.dll!00007ffb1ba08bb4()	Unknown
>	ogrext.cp35-win_amd64.pyd!__pyx_pf_5fiona_6ogrext_7Session_4start(__pyx_obj_5fiona_6ogrext_Session * __pyx_v_self, _object * __pyx_v_collection) Line 8816	C
 	ogrext.cp35-win_amd64.pyd!__pyx_pw_5fiona_6ogrext_7Session_5start(_object * __pyx_v_self, _object * __pyx_v_collection) Line 8098	C
 	python35.dll!call_function(_object * * * pp_stack, int oparg) Line 4705	C
 	python35.dll!PyEval_EvalFrameEx(_frame * f, int throwflag) Line 3254	C
 	python35.dll!_PyEval_EvalCodeWithName(_object * _co, _object * globals, _object * locals, _object * * args, int argcount, _object * * kws, int kwcount, _object * * defs, int defcount, _object * kwdefs, _object * closure, _object * name, _object * qualname) Line 4033	C
 	python35.dll!function_call(_object * func, _object * arg, _object * kw) Line 627	C
 	python35.dll!PyObject_Call(_object * func, _object * arg, _object * kw) Line 2168	C
 	python35.dll!method_call(_object * func, _object * arg, _object * kw) Line 331	C
 	python35.dll!PyObject_Call(_object * func, _object * arg, _object * kw) Line 2168	C
 	python35.dll!slot_tp_init(_object * self, _object * args, _object * kwds) Line 6288	C
 	python35.dll!type_call(_typeobject * type, _object * args, _object * kwds) Line 906	C
 	python35.dll!PyObject_Call(_object * func, _object * arg, _object * kw) Line 2168	C
 	python35.dll!do_call(_object * func, _object * * * pp_stack, int na, int nk) Line 4951	C
 	python35.dll!call_function(_object * * * pp_stack, int oparg) Line 4749	C
 	python35.dll!PyEval_EvalFrameEx(_frame * f, int throwflag) Line 3254	C
 	python35.dll!_PyEval_EvalCodeWithName(_object * _co, _object * globals, _object * locals, _object * * args, int argcount, _object * * kws, int kwcount, _object * * defs, int defcount, _object * kwdefs, _object * closure, _object * name, _object * qualname) Line 4033	C
 	python35.dll!fast_function(_object * func, _object * * * pp_stack, int n, int na, int nk) Line 4833	C
 	python35.dll!call_function(_object * * * pp_stack, int oparg) Line 4745	C
 	python35.dll!PyEval_EvalFrameEx(_frame * f, int throwflag) Line 3254	C
 	python35.dll!_PyEval_EvalCodeWithName(_object * _co, _object * globals, _object * locals, _object * * args, int argcount, _object * * kws, int kwcount, _object * * defs, int defcount, _object * kwdefs, _object * closure, _object * name, _object * qualname) Line 4033	C
 	python35.dll!run_mod(_mod * mod, _object * filename, _object * globals, _object * locals, PyCompilerFlags * flags, _arena * arena) Line 977	C
 	python35.dll!PyRun_FileExFlags(_iobuf * fp, const char * filename_str, int start, _object * globals, _object * locals, int closeit, PyCompilerFlags * flags) Line 929	C
 	python35.dll!PyRun_SimpleFileExFlags(_iobuf * fp, const char * filename, int closeit, PyCompilerFlags * flags) Line 396	C
 	python35.dll!PyRun_AnyFileExFlags(_iobuf * fp, const char * filename, int closeit, PyCompilerFlags * flags) Line 81	C
 	python35.dll!run_file(_iobuf * fp, const wchar_t * filename, PyCompilerFlags * p_cf) Line 318	C
 	python35.dll!Py_Main(int argc, wchar_t * * argv) Line 768	C
 	[External Code]

This is the Fiona code that crashes:
                /* "fiona/ogrext.pyx":422
 *                         flags = ogrext2.GDAL_OF_VECTOR | ogrext2.GDAL_OF_READONLY
 *                         log.debug("GDALOpenEx({}, {}, {})".format(path_c, flags, [name_b]))
 *                         ds = ogrext2.GDALOpenEx(path_c,             # <<<<<<<<<<<<<<
 *                                                flags,
 *                                                drvs,
 */
                __pyx_v_ds = GDALOpenEx(__pyx_v_path_c, __pyx_v_flags, __pyx_v_drvs, NULL, NULL);

These are the values passed to the GDALOpenEx function:
+		__pyx_v_path_c	0x000001f733635c50 "Final_Neighbourhoods.gdb"	const char *
-		__pyx_v_drvs	0x000001f73325c650 {0x000001f73354a110 "OpenFileGDB"}	char * *
+			0x000001f73354a110 "OpenFileGDB"	char *
		__pyx_v_flags	4	int

papszAllowedDrivers passed to GDALOpenEx was not properly NULL terminated.
This should be fixed by #431

🙏 @cgohlke! I've fixed this in 95726af in maint-1.7 and will merge into master as well.


unexpected behavior in kfc

Environment
Device and OS:
App version:
Kubernetes distro being used:
Other:
Steps to reproduce

npx kubernetes-fluent-client crd [src] [dest] does not seem to work on local files

*Fails
┌─[cmwylie19@Cases-MacBook-Pro] - [~/pepr-excellent-examples/pepr-operator/capabilities] - [2024-01-30 07:38:53]
└─[1] <git:(operator c1fcf70✱✈) > npx kubernetes-fluent-client crd WebApp.yaml .
Attempting to read WebApp.yaml from the current Kubernetes context

❌ Failed to read WebApp.yaml as a file, url or K8s CRD: customresourcedefinitions.apiextensions.k8s.io "WebApp.yaml" not found

Passes
┌─[cmwylie19@Cases-MacBook-Pro] - [~/pepr-excellent-examples/pepr-operator/capabilities] - [2024-01-30 07:40:09]
└─[130] <git:(operator c1fcf70✱✈) > npx kubernetes-fluent-client crd https://gist.githubusercontent.com/cmwylie19/69b765af5ab25af62696f3337df13687/raw/53a9504fe865db4e10b2f56f8467eb2d7ed2181d/WebApp.yaml .
Attempting to load https://gist.githubusercontent.com/cmwylie19/69b765af5ab25af62696f3337df13687/raw/53a9504fe865db4e10b2f56f8467eb2d7ed2181d/WebApp.yaml as a URL

- Generating pepr.io/v1alpha1 types for WebApp

✅ Generated 1 files in the . directory

Expected result
Actual Result
Visual Proof (screenshots, videos, text, etc)
Severity/Priority
Additional Context
Add any other context or screenshots about the technical debt here.

This may be a non-issue.


system should accept transactions in closed loop, return error during overload

Currently the system will run in closed loop mode when using noops consensus, by virtue of how noops is implemented.  The PBFT consensus uses internal data structures and does not limit the number of queued requests.
The system should have a means to:

delay responses to incoming requests to enable a closed loop feedback
reject new requests during overload

In any case, once a positive response has been delivered to the client, a message should never be dropped.  This statement requires deliberation on when a positive response can even be delivered; is persistence on stable storage enough?

@binh @markparz @cca88 @bcbrock

Yes: In other words, the consensus layer (aka total-order broadcast channel) can "fill up" when under load. Just like a TCP channel we will need some "flow control" to make it "closed-loop".
Suppose the consensus layer interfaces to a client application with asynchronous SEND and DELIVER events:
*SEND is an input event (to consensus) for submitting a tx/msg for broadcast
*DELIVER is an output event (generated by consensus) that carries an ordered tx/msg
One very simple way to add flow control is via two more events:
*SEND-ACK: an output event from consensus to indicate that it is ready to handle a next SEND event; before that arrives the application must not invoke SEND again
*DELIVER-ACK: an input event (from the application) where the application says that it has consumed the delivered tx/msg and is ready to accept the next one; the consensus will not issue another DELIVER before the application ask invoked DELIVER-ACK
That was implemented in our SINTRA and some of that is described also in the paper --  http://www.zurich.ibm.com/~cca/papers/sintra.pdf
You will not find this PBFT, since it focused on data-center deployment with UDP communication. Under load those packets are dropped and will automatically limit the load.

Throughout the stack we will use only bounded queues; request rejection will be handled on the ingress side (REST/Devops).

this is a stack-wide issue; the consensus label is incorrect.

@corecode  are you still working this issue?

I have not been working on this - this is a stack-wide issue that needs addressing in many places.

@gennadylaventman @bmandler - I would classify this as a flow-control issue we have talked about.  this should fit into the future work once we clean up all of the communication code


Can't install release 0.3.2

I see the 0.3.2 release on PyPI, but running into trouble installing:
~ > pip install shellinford==0.3.2
Collecting shellinford==0.3.2
  Could not find a version that satisfies the requirement shellinford==0.3.2 (from versions: 0.3.1)
No matching distribution found for shellinford==0.3.2
~ > pip search shellinford
shellinford (0.3.2)  - Wavelet Matrix/Tree succinct data structure for full text
                       search (using shellinford C++ library)

Thank you for reporting.
I upload shellinford to PyPI as version 0.3.4.
Please try pip install shellinford

Thank you for fixing so quickly!


Fix breakages from girder-client v2.0

Also remove usage of a private symbol in favor of a public one.
@jeffbaumes PTAL

Environment variable GIRDER_API_KEY is blank. Cannot upload images.

I'm assuming this is because I'm making this PR from a fork.

Yeah I think so. I'll create a remote to track your branch just to allow this test to run.

It makes sense, otherwise any old adversary could make a PR that just prints the value from your travis script.
Thanks!

Hey, look, it builds! https://travis-ci.org/Kitware/candela/builds/166774970

@zachmullen I added you as a collaborator so you should be able to push the button.

Thanks @zachmullen!


clientv3: fix balancer update address bug

FIX #7189

all fixed. PTAL @heyitsanthony

All fixed. PTAL cc\ @heyitsanthony

lgtm after removing that extra SetEndpoints. Thanks!

merge once CI passes


Example code update

Update readme. And connect to demo environment in example code.

Codecov Report

Merging #15 (ac36806) into main (5582d0b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #15   +/-   ##
=========================================
  Coverage     78.61%   78.61%
  Complexity      266      266
=========================================
  Files            32       32
  Lines           926      926
  Branches         62       62
=========================================
  Hits            728      728
  Misses          162      162
  Partials         36       36


⚠ Bofrost.lu has degraded performance

In 73725d8, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1805 ms

Resolved: Bofrost.lu performance has improved in 61384f1 after 9 minutes.


How do I chain multiple proxy

According to the wiki, we can add reference to a proxy (apache or an Nginx)  by setting an header in them.
https://github.com/naver/pinpoint/blob/master/doc/proxy-http-header.md
But what about a scenario where we chain multiple proxy?
Example
User -> WAF -> SSL Terminator -> Load Balancer -> Web Server -> App Server
How can we add information regarding such a chain of proxy from various vendors?

We are planning to change the header handler to a plugin.
The header plugin can be used to change the desired header processing format.


Deployment issues with s3 buckets

Trying to deploy to a s3 bucket that doesn't exist yet
next config
 config.target = 'serverless'
    config.assetPrefix = `https://serverless-cowsay-v3-${stage}.s3-us-east-1.amazonaws.com`
    console.log(config.assetPrefix)
    return config

It reports that it found a bucket
https://serverless-cowsay-v3-dev.s3-us-east-1.amazonaws.com
Serverless Nextjs: Found bucket "serverless-cowsay-v3-dev"

Then it errors out because the bucket does not exist. No bucket is created in s3
Serverless: Uploading service cowsayify.zip file to S3 (4.09 MB)...
Serverless: [AWS s3 200 0.938s 0 retries] putObject({ Body: <Buffer 50 4b 03 04 14 00 08 00 08 00 00 00 21 00 00 00 00 00 00 00 00 00 00 00 00 00 18 00 00 00 73 6c 73 2d 6e 65 78 74 2d 62 75 69 6c 64 2f 5f 65 72 72 6f ... >,
  Bucket: 'cowsayify-dev-serverlessdeploymentbucket-f5qg4i30jpdt',
  Key: 'serverless/cowsayify/dev/1555102192807-2019-04-12T20:49:52.807Z/cowsayify.zip',
  ContentType: 'application/zip',
  Metadata: { filesha256: 'chsg3nZggrgUZUClb2nQ40pe7GDK+L7juFnjj5VHQVY=' } })
https://serverless-cowsay-v3-dev.s3-us-east-1.amazonaws.com
Serverless Nextjs: Uploading static assets to serverless-cowsay-v3-dev ...
Serverless: [AWS s3 404 0.244s 0 retries] putObject({ Body: <Buffer 7b 22 76 65 72 73 69 6f 6e 22 3a 33 2c 22 66 69 6c 65 22 3a 22 73 74 61 74 69 63 2f 63 68 75 6e 6b 73 2f 30 2e 6a 73 22 2c 22 73 6f 75 72 63 65 73 22 ... >,
  ACL: 'public-read',
  Bucket: 'serverless-cowsay-v3-dev',
  Key: '_next/static/chunks/0.js.map',
  ContentType: 'application/json' })
Serverless: [AWS s3 404 0.253s 0 retries] putObject({ Body: <Buffer 28 77 69 6e 64 6f 77 5b 22 77 65 62 70 61 63 6b 4a 73 6f 6e 70 22 5d 20 3d 20 77 69 6e 64 6f 77 5b 22 77 65 62 70 61 63 6b 4a 73 6f 6e 70 22 5d 20 7c ... >,
  ACL: 'public-read',
  Bucket: 'serverless-cowsay-v3-dev',
  Key: '_next/static/chunks/0.js',
  ContentType: 'application/javascript' })
Serverless Nextjs: ServerlessError: The specified bucket does not exist

If I create the bucket first, it complains that the bucket already exists!

@bushmango Did you add the assetPrefix to next.config.js after the initial creation of the serverless stack?
The bucket can't be added after the serverless stack was provisioned the first time. The bucket has to be created as part of the initial deployment. Wouldn't make sense to create on stack update because your Lambda will update / point to the new code without the static assets uploaded.
You'll have to delete the serverless stack and recreate from scratch with the assetPrefix in your next.config.js

I am also running into this issue.
Serverless Nextjs: Copying next pages to tmp build folder
Serverless Nextjs: Found 1 next page(s)
Serverless Nextjs: Creating compat handler for page: index.js
Serverless Nextjs: Found bucket "assets.example.com"
Serverless: Packaging service...
Serverless: Excluding development dependencies...
Serverless Nextjs: Cleaning up tmp build folder ...
Serverless: Uploading CloudFormation file to S3...
Serverless: Uploading artifacts...
Serverless: Uploading service example-site.zip file to S3 (25.42 MB)...
Serverless Nextjs: Uploading .next/static to assets.example.com ...
Serverless Nextjs: Uploading static to assets.example.com ...

I have tried these different options

new stack, assetPrefix defined with a non existent bucket
new stack, assetsBucketName defined with a non existent bucket
existing stack, assetPrefix defined with a non existent bucket
existing stack, assetsBucketName defined with a non existent bucket

Serverless Nextjs: Copying next pages to tmp build folder
Serverless Nextjs: Found 1 next page(s)
Serverless Nextjs: Creating compat handler for page: index.js
Serverless Nextjs: Found bucket "assets.example.com"
Serverless: Packaging service...
Serverless: Excluding development dependencies...
Serverless Nextjs: Cleaning up tmp build folder ...
Serverless: Uploading CloudFormation file to S3...
Serverless: Uploading artifacts...
Serverless: Uploading service example-site.zip file to S3 (25.42 MB)...
Serverless Nextjs: Uploading .next/static to example.com ...
Serverless Nextjs: Uploading static to example.com ...
Serverless: Validating template...
Serverless: Creating Stack...
Serverless: Checking Stack create progress...............................
Serverless: Operation failed!

And with an existent bucket

new stack, assetPrefix defined with a existent bucket
new stack, assetsBucketName defined with a existent bucket
existing stack, assetPrefix defined with a existent bucket
existing stack, assetsBucketName defined with a existent bucket.

It looks like uploading the artifacts is taking place before the bucket creation
Commenting out
"after:aws:deploy:deploy:uploadArtifacts": this.uploadStaticAssets,

in index.js for the first deployment works on creating the bucket, and then uncommenting works for deploying the static assets after.

Closing as duplicate of https://github.com/danielcondemarin/serverless-nextjs-plugin/issues/45


Laravel Horizon

Hi,
someone have installed Laravel Horizon?
I've installed, but sometimes getting many issues like:

Can somebody help me with this?
Thank you.

Please add more details if you're still having this issue. I can't really do much debugging with what's been presented, sorry!


Auto-create py.typed File(s)

Most useful for PyPI-bound pacakges, py.typed files need to be included in each package directory. They are empty files that tell Python's typing module to use/look for the package's typehints. This PR automates this boilerplate process.

works:

works:

https://github.com/icecube/skyreader/pull/3
https://github.com/WIPACrepo/SkyDriver/pull/27


Method getContent does not exist.

Using this as described at the documentation:
// Only related categories and posts
$category = Taxonomy::where('taxonomy', 'category')->with('posts')->get();
$category->each(function($category) {
echo $category->getContent('name');
});
It gives this error:
Method Illuminate\Database\Query\Builder::getContent does not exist.

It seems the Multilanguage Trait is not added to the Taxonomy model:
namespace Orchid\Platform\Core\Models;
use Orchid\Platform\Core\Traits\MultiLanguage;
class Taxonomy extends Model
{
use MultiLanguage;
}
and it should be:
echo $category->term->getContent('name');

I, there really is an error. Taxonometry does not have a field for localization.
It is necessary to use the term
$category = Taxonomy::where('taxonomy', 'category')->with('posts')->get();

$category->each(function($category) {
    echo $category->term->getContent('name');
});


Use report plugins?

https://github.com/postcss/postcss-reporter
https://github.com/postcss/postcss-browser-reporter

Closing this issue as people didn't miss this feature.


Dependency graph issue with sub folders

So first of all this plugin was very easy to use. Thanks..
Im seeing one issue which i think may not be supported but asking the question.
Im trying to do a setup where some java libs live in a sub folder but saw that the dependency graph doesnt pick up these projects as dependencies when listed
Steps:
- create a lib in a sub folder
 > nx generate @jnxplus/nx-boot-gradle:library messages/contacts-service

- add dependency in gradle
- start `nx graph`

My sample gradle file excerpt
// build.gradle for core-api

dependencies {
  	implementation 'org.springframework.boot:spring-boot-starter-web'
  	testImplementation 'org.springframework.boot:spring-boot-starter-test'

	implementation project(":libs:offer-parser") // only this is shown as dependency (sometimes not)

        // these are unlinked always
	implementation project(":libs:messages:contacts-service")
	implementation project(":libs:messages:message-draft-service")
}

workspace,json
{
  "version": 2,
  "projects": {
    "client": "apps/client",
    "client-e2e": "apps/client-e2e",
    "core-api": "apps/core-api",
    "messages/contacts-service": "libs/messages/contacts-service",
    "messages/message-draft-service": "libs/messages/message-draft-service",
    "offer-parser": "libs/offer-parser",
    "ui-buttons": "libs/ui/buttons",
    "ui-typography": "libs/ui/typography"
  }
}

Graph when nothing is shown as linked to core-api.

and occasionally

folder structure

Questions:

Am i listing the dependencies incorrectly? (or incompatible with the lookup)
Are sub folders currently supported by the dep-graph lookup for this library?

https://github.com/khalilou88/jnxplus/blob/main/packages/nx-boot-gradle/src/dep-graph/lookup-deps.ts#L86
only considers last part of the project name

Hi @lwhiteley Thank you for this detailed bug.
I just published a fix.
Hope it will fix your issue.

Thanks I'll test again and let you know

Hi @lwhiteley Could you please tell me how you create a lib or an app inside a folder? which command you use?

I'm trying to generate a lib inside a subfolder, but getting the following error:
yarn run v1.22.17
$ /Users/vladimir.drayling/dev/web/apps/eon/node_modules/.bin/nx generate @jnxplus/nx-boot-maven:library --name=api --directory=greeting --groupId=de.eon --language=kotlin --projects=api --tags=scope:greeting,type:domain --no-interactive --dry-run
Cannot create Project 'api'. It already exists.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
The terminal process "/bin/sh '-c', 'yarn nx generate @jnxplus/nx-boot-maven:library --name=api --directory=greeting --groupId=de.eon --language=kotlin --projects=api --tags=scope:greeting,type:domain --no-interactive --dry-run'" terminated with exit code: 1.

In fact I do have a project called api but it is inside of apps folder.
Is it possible to instruct the plugin to generate a lib called greeting-api with the folder structure like libs/greeting/api? It could make it possible to add angular libs e.g. libs/greeting/feature or libs/greeting/ui.
angular|workspace.json
{
  "version": 2,
  "projects": {
    "api": "apps/api",
    "greeting-api": "libs/greeting/api",
    "greeting-feature": "libs/greeting/feature",
    "greeting-ui": "libs/greeting/ui",
    "greeting-utils": "libs/greeting/utils",
    "web-client": "apps/web-client",
    "web-client-e2e": "apps/web-client-e2e"
  }
}

nx report
 >  NX   Report complete - copy this into the issue template

   Node : 16.13.1
   OS   : darwin x64
   yarn : 1.22.17

   nx : 13.8.4
   @nrwl/angular : 13.8.4
   @nrwl/cli : 13.8.4
   @nrwl/cypress : 13.8.4
   @nrwl/detox : undefined
   @nrwl/devkit : 13.8.4
   @nrwl/eslint-plugin-nx : 13.8.4
   @nrwl/express : undefined
   @nrwl/jest : 13.8.4
   @nrwl/js : undefined
   @nrwl/linter : 13.8.4
   @nrwl/nest : undefined
   @nrwl/next : undefined
   @nrwl/node : undefined
   @nrwl/nx-cloud : undefined
   @nrwl/react : undefined
   @nrwl/react-native : undefined
   @nrwl/schematics : undefined
   @nrwl/storybook : 13.8.4
   @nrwl/tao : 13.8.4
   @nrwl/web : undefined
   @nrwl/workspace : 13.8.4
   typescript : 4.5.5
   rxjs : 7.4.0
   ---------------------------------------
   Community plugins:
         @angular/animations: 13.2.5
         @angular/common: 13.2.5
         @angular/compiler: 13.2.5
         @angular/core: 13.2.5
         @angular/forms: 13.2.5
         @angular/platform-browser: 13.2.5
         @angular/platform-browser-dynamic: 13.2.5
         @angular/router: 13.2.5
         @angular-devkit/build-angular: 13.2.5
         @angular/cli: 13.2.5
         @angular/compiler-cli: 13.2.5
         @angular/language-service: 13.2.5
         @jnxplus/nx-boot-maven: 1.4.1

@khalilou88
when creating the lib in a subfolder, you just define the path as the lib name
for eg.
nx generate @jnxplus/nx-boot-gradle:library {folder}/{project-name}

i have not tried this with apps but it should work the same

a thing to note is that the subfolder process for this lib is inconsistent with the core nx plugins..
where the core nx plugins hyphenate the project names. see workspace example below
{
  "version": 2,
  "projects": {
    "client": "apps/client",
    "client-e2e": "apps/client-e2e",
    "core-api": "apps/core-api",
    "messages/contacts-service": "libs/messages/contacts-service", // java plugin create names with `/`
    "messages/message-draft-service": "libs/messages/message-draft-service",
    "offer-parser": "libs/offer-parser",
    "ui-buttons": "libs/ui/buttons", // core nx plugins create project names with hyphens `-`
    "ui-typography": "libs/ui/typography"
  }
}

i had added a PR that searched for the real name so maybe take another look at the logic: https://github.com/khalilou88/jnxplus/pull/16

@vladimirdrayling maybe unrelated but there is a migration bug with<EMAIL_ADDRESS>so be careful until 13.8.5 is released
https://github.com/nrwl/nx/issues/9211

The nx generate @jnxplus/nx-boot-maven:library --name=greeting/api --groupId=de.eon --language=kotlin --projects=api --tags=scope:greeting,type:domain --no-interactive generates the following content:

As you can see the file names are suffixed with (1).
angular.json
{
  "version": 2,
  "projects": {
    "api": "apps/api",
    "greeting/api": "libs/greeting/api",
    "web-client": "apps/web-client",
    "web-client-e2e": "apps/web-client-e2e"
  }
}

As you @lwhiteley mentioned, the project's name could be hyphenate.
Is it possible to name the files with the directory's name? E.g. nx generate @jnxplus/nx-boot-maven:library --name=api --directory=greeting --groupId=de.eon --language=kotlin --projects=api --tags=scope:greeting,type:domain --no-interactive --dry-run would generate the following structure:

with adjusted dependencies in apps/api/(api.iml,pom.xml) and libs/greeting/api/(greeting-api.iml,pom.xml). Right now I just replaced the occurrence greeting/api with greeting-api.
nx dep-graph

we should not create a lib with a folder like this nx generate @jnxplus/nx-boot-gradle:library {folder}/{project-name}
the correct command is nx generate @jnxplus/nx-boot-gradle:library {project-name} --directory {folder}
I am fixing that, if the name contains a / I will change it to be -.

@lwhiteley @vladimirdrayling
check this branch https://github.com/khalilou88/jnxplus/tree/feature/names
here I will make things simple and like nx generators

took a quick look was not sure all cases are being handled.
Concerns to test for:
Concern 1
the sub director can be multiple layers deep
libs/messages/shared/string-utils

nx generate @jnxplus/nx-boot-gradle:library  string-utils --directory messages/shared

Concern 2
Need to test/handle case where developer wants to move project to a new folder
https://nx.dev/workspace/move#examples
This can also be multiple layers deep
Concern 3 (nice to have)
Can the initial java package name be formatted nicely when directory given.
for eg created a lib with folder path libs/messages/shared/string-utils
package name can account for folders like
package com.myapp.messages.shared.stringutils;

Hi @lwhiteley
I will tackle point 1 and 3.
for point 4, it's hard to test it in dev env
point 2, lets put it on another task

@lwhiteley  I published the version 1.0.5 with this changes

Tested with gradle
// Works
- [x] Concern 1
- [x] Concern 3

not working
- [ ] Concern 4 - not working

current workspace.json
{
  "version": 2,
  "projects": {
    "client": "apps/client",
    "client-e2e": "apps/client-e2e",
    "core-api": "apps/core-api",
    "messages-shared-string-utils": "libs/messages/shared/string-utils",
    "messages/contacts-service": "libs/messages/contacts-service",
    "messages/message-draft-service": "libs/messages/message-draft-service",
    "offer-parser": "libs/offer-parser",
    "ui-buttons": "libs/ui/buttons",
    "ui-typography": "libs/ui/typography"
  }
}

the project name lookup still isnt trying to find the real name but assuming it based on pattern
function getDependencyProjectName(gradleProjectPath: string) {
  return gradleProjectPath.split(':libs:').pop().replace(/:/g, '-');
}

wont work for all cases / be backward compatible
https://github.com/khalilou88/jnxplus/blob/main/packages/nx-boot-gradle/src/dep-graph/lookup-deps.ts#L86

@lwhiteley you are right about the graph. Could you please activate your PR for getting the exact project name ?

sure no prob..
im also getting this error:
Error: Unable to access jarfile<EMAIL_ADDRESS>

@lwhiteley maybe you need to install deps? (npm install)

I'm using pnpm . The dependency is there but not with Checkstyles folder

Maybe you can remove this   "implicitDependencies": ["checkstyle", "pmd", "ktlint"] from project.json?

ah i get it now.. its just the install process seems different. it didnt install the other dependencies. ill have to check why

ok i figured it out. the file structure for pnpm is different ,
it uses a virtual store for transitive dependencies so may need a way to define the location of the jar or resolve the location better.
relative path:
node_modules/.pnpm/@jnxplus+checkstyle<EMAIL_ADDRESS>

@khalilou88

dependency issues in PR #24
you can review and test #23 for graph project name look up

@khalilou88 added one last PR.
let me know when tested and released

@lwhiteley thank you so much. I published your PRs in version 1.5.1

nice .. i tested and all issues work great and is now backward compatible.
screenshot shows both types of project names are linked


[FEAT]: API ENDPOINT TO ASSIGN USER'S ROLE IN AN ORGANISATION

Description
Develop an API endpoint that allows only superadmins or admins including organisation owner to assign roles to other active users in an organisation. Users can only have a role such as admin, guest, or member.
Purpose
The endpoint provides superadmins, including organistion owners with the right to assign roles to users in an organisation.
Acceptance Criteria

[ ] The endpoint should be accesible at  /api/v1/organisations/{org_id}/users/{user_id}/roles.
[ ] The endpoint should accept HTTP POST request for assigning a role.
[ ] Only superadmins, admins and owners are authorized to assign a role to a users in an organisation.
[ ] The endpoint should ensure that a role is provided in the request.
[ ] The role, user_id and org_id fields must be validated.
[ ] A 201 OK status code must be returned when the request is sucessful.
[ ] Handle cases where the provided data is not valid.
[ ] Handle all failed request with the appropriate error messages and status code.

Requirements

Superadmins, admins and owners must be authenticated and authorized to assign roles.
The organisation ID and user ID and a valid role must be provided.

Database Schema
Table Roles {
    "id": String,
    "user_id": String,
    "organization_id": String,
    "role_name": String,
    "created_at": Timestamp,
    "updated_at": Timestamp,
}

Endpoints
Requests
Assign a role to a user in an organisation
Method: POST
Endpoint URL: /api/v1/organisations/{org_id}/users/{user_id}/roles
Authorization Header: "Authorization": "Bearer abc123ABC123abd...."
Request Body:
{
    "role": String
}

Response
Successful Response when a user has been assigned a role.
{
    "message": "[firstName] [lastName] has successfully been added to the [role] role",
    "status_code": 201,
    "data": {
        "user": {},
        "org": {}
        "role": "[role]"
    }
}

Failed Validation Response
{
    "message": "Invalid data",
    "status_code": 403,
    "data": [
        {
            "field": String,
            "validationMessage": String
        },
    ]
}

Failed Authorization Response
{
    "message": "You are not authorized to manage user roles",
    "status_code": 401,
}

Testing

[ ] Write unit test to cover various scenarios.
[ ] Write integration test to cover various scenarios.

Documentation
The endpoint should be documented containing use cases with accurate response and error messages
Expected Outcome

Admins in an organisation can securely assign roles to members and guests.

the endpoint is wrong, as you have two different functionalities, one to manage roles and another to assign roles to users. this is ambigious

@olamstevy This is still two endpoints in one Ticket. Please break into two and go into more detail.
See #235
also add the Schema so we see what you are working with before second approval

Description
Develop an API endpoint that allows only superadmins or admins including organisation owner to assign roles to other active users in an organisation. Users can only have a role such as admin, guest, or member.
Purpose
The endpoint provides superadmins, including organistion owners with the right to assign roles to users in an organisation.
Acceptance Criteria

[ ] The endpoint should be accesible at  /api/v1/organisations/{org_id}/users/{user_id}/roles.
[ ] The endpoint should accept HTTP POST request for assigning a role.
[ ] Only superadmins, admins and owners are authorized to assign a role to a users in an organisation.
[ ] The endpoint should ensure that a role is provided in the request.
[ ] The role, user_id and org_id fields must be validated.
[ ] A 201 OK status code must be returned when the request is sucessful.
[ ] Handle cases where the provided data is not valid.
[ ] Handle all failed request with the appropriate error messages and status code.

Requirements

Superadmins, admins and owners must be authenticated and authorized to assign roles.
The organisation ID and user ID and a valid role must be provided.

Database Schema
Table Roles {
    "id": String,
    "user_id": String,
    "organization_id": String,
    "role_name": String,
    "created_at": Timestamp,
    "updated_at": Timestamp,
}

Endpoints
Requests
Assign a role to a user in an organisation
Method: POST
Endpoint URL: /api/v1/organisations/{org_id}/users/{user_id}/roles
Authorization Header: "Authorization": "Bearer abc123ABC123abd...."
Request Body:
{
    "role": String
}

Response
Successful Response when a user has been assigned a role.
{
    "message": "[firstName] [lastName] has successfully been added to the [role] role",
    "status_code": 201,
    "data": {
        "user": {},
        "org": {}
        "role": "[role]"
    }
}

Failed Validation Response
{
    "message": "Invalid data",
    "status_code": 403,
    "data": [
        {
            "field": String,
            "validationMessage": String
        },
    ]
}

Failed Authorization Response
{
    "message": "You are not authorized to manage user roles",
    "status_code": 401,
}

Testing

[ ] Write unit test to cover various scenarios.
[ ] Write integration test to cover various scenarios.

Documentation
The endpoint should be documented containing use cases with accurate response and error messages
Expected Outcome

Admins in an organisation can securely assign roles to members and guests.

Please share second ticket with someone else or create it and share with me

I have comment on the link to the second ticket above


WIP [master] Add a Jenkinsfile to run the CLI Integration tests ENGCORE-1094

Carrying #627

Found https://github.com/moby/moby/pull/40043 while running the tests.


Snowflake: support for extended column options in CREATE TABLE

Snowflake supports several extended options for column definitions:
-- Column definition
    <col_name> <col_type>
      [ inlineConstraint ]
      [ NOT NULL ]
      [ COLLATE '<collation_specification>' ]
      [
        {
          DEFAULT <expr>
          | { AUTOINCREMENT | IDENTITY }
            [
              {
                ( <start_num> , <step_num> )
                | START <num> INCREMENT <num>
              }
            ]
            [ { ORDER | NOORDER } ]
        }
      ]
      [ [ WITH ] MASKING POLICY <policy_name> [ USING ( <col_name> , <cond_col1> , ... ) ] ]
      [ [ WITH ] PROJECTION POLICY <policy_name> ]
      [ [ WITH ] TAG ( <tag_name> = '<tag_value>' [ , <tag_name> = '<tag_value>' , ... ] ) ]
      [ COMMENT '<string_literal>' ]

This merge request implements support for these additional column options.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

279 of 284 (98.24%)  changed or added relevant lines in 6 files are covered.
1410 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.07%) to 89.392%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/dialect/snowflake.rs
55
56
98.21%

src/parser/mod.rs
14
15
93.33%

tests/sqlparser_mssql.rs
11
12
91.67%

src/ast/ddl.rs
39
41
95.12%

Files with Coverage Reduction
New Missed Lines
%

tests/sqlparser_mssql.rs
1
93.38%

src/lib.rs
3
12.2%

src/tokenizer.rs
49
94.14%

src/ast/mod.rs
394
82.44%

src/parser/mod.rs
411
93.39%

tests/sqlparser_common.rs
552
89.58%

Totals

Change from base Build<PHONE_NUMBER>7:
0.07%

Covered Lines:
30428

Relevant Lines:
34039

💛  - Coveralls


No music on anny of the versions

Im using an old3ds and have 0 sound on anny of the versions

All versions require the NDSP firmware in order to play any audio. Have you tried dumping the NDSP firmware as described here?

no, but thank you. I couldnt see anny reference to this in the readme file


Questions about testing

Hello, I would like to ask which model should be used for inference during testing? I am getting some errors while loading the model.
RuntimeError: Error(s) in loading state_dict for IHN:
Unexpected key(s) in state_dict: "update_block_4.cnn_weight.layer0.0.weight", "update_block_4.cnn_weight.layer0.0.bias", "update_block_4.cnn_weight.layer0.1.weight", "update_block_4.cnn_weight.layer0.1.bias", "update_block_4.cnn_weight.layer0.3.weight", "update_block_4.cnn_weight.layer0.3.bias", "update_block_4.cnn_weight.layer0.4.weight", "update_block_4.cnn_weight.layer0.4.bias", "update_block_4.cnn_weight.layer1.0.weight", "update_block_4.cnn_weight.layer1.0.bias", "update_block_4.cnn_weight.layer1.1.weight", "update_block_4.cnn_weight.layer1.1.bias", "update_block_4.cnn_weight.layer2.0.weight", "update_block_4.cnn_weight.layer2.0.bias", "update_block_4.cnn_weight.layer2.1.weight", "update_block_4.cnn_weight.layer2.1.bias", "update_block_4.cnn_weight.layer3.0.weight", "update_block_4.cnn_weight.layer3.0.bias", "update_block_4.cnn_weight.layer3.1.weight", "update_block_4.cnn_weight.layer3.1.bias", "update_block_4.cnn_weight.layer4.0.weight", "update_block_4.cnn_weight.layer4.0.bias", "update_block_4.cnn_weight.layer4.1.weight", "update_block_4.cnn_weight.layer4.1.bias", "update_block_4.cnn_weight.layer4_global.0.weight", "update_block_4.cnn_weight.layer4_global.0.bias", "update_block_4.cnn_weight.layer4_global.1.weight", "update_block_4.cnn_weight.layer4_global.1.bias", "update_block_4.cnn_weight.layer5.0.weight", "update_block_4.cnn_weight.layer5.0.bias", "update_block_4.cnn_weight.layer5.1.weight", "update_block_4.cnn_weight.layer5.1.bias", "update_block_4.cnn_weight.layer6.0.weight", "update_block_4.cnn_weight.layer6.0.bias", "update_block_4.cnn_weight.layer6.1.weight", "update_block_4.cnn_weight.layer6.1.bias", "update_block_4.cnn_weight.layer7.0.weight", "update_block_4.cnn_weight.layer7.0.bias", "update_block_4.cnn_weight.layer7.1.weight", "update_block_4.cnn_weight.layer7.1.bias", "update_block_4.cnn_weight.layer8.0.weight", "update_block_4.cnn_weight.layer8.0.bias", "update_block_4.cnn_weight.layer8.1.weight", "update_block_4.cnn_weight.layer8.1.bias", "update_block_4.cnn_weight.layer9.0.weight", "update_block_4.cnn_weight.layer9.0.bias", "update_block_4.cnn_weight.layer10.0.weight", "update_block_4.cnn_weight.layer10.0.bias", "update_block_4.cnn_weight.layer10.1.weight", "update_block_4.cnn_weight.layer10.1.bias", "update_block_4.cnn_weight.layer10.3.weight", "update_block_4.cnn_weight.layer10.3.bias".

Hi, please check if you load the wrong model. For example, you load the model with weight prediction into the code setting with no weight prediction.

Hello, I tried with all the models in the repo, and i have the same error, do you know why ?

in 'myevaluate.py' file at line 75, try to edit the code to this:
parser.add_argument('--lev0', default=True, action='store_false',
                        help='warp no')


bull-board not working with AWS serveless

I have a problem with the bull-board, I created my bull-board application with nest and using AWS Lambda, it works normally, but the bull-board does not, when I access the route only the text "Loading..." appears, but If I start without serverless, the bull-board works normally, does anyone have any idea what it could be?

Hi @GBShadow I think that if you will ask in English, it will approach more ppl.
Anyway, I've translated your message, and looks like you have an issue with Nest & aws serverless, I'm not familiar with this setup, so If someone knows it, I will be more that happy for the help.

This is a duplicate of #444, in my opinion.
bull-board does not work in serverless environments because it uses full paths to load some files.

In serverless env the path to static assets of the ui is not calculated properly,
You can change the path to a the proper value by calling
const uiBasePath = YOUR_CORRECT_VALUE
serverAdapter
  .setViewsPath(path.join(uiBasePath, 'dist'))
  .setStaticPath('/static', path.join(uiBasePath, 'dist/static'));

after you pass it to createBullBoard function.
Can you try this and let me know if it helped you?

same error, i believe it's throwing on this line:
https://github.com/felixmosh/bull-board/blob/06a1c9828c5c4873172981e60bf85d7d2fc96264/packages/api/src/index.ts#L18
which is executed when running createBullBoard
if we could pass a uiBasePath to createBullBoard instead of it being calculated, that would fix it?

Probably... But i think that serverless is not a good fit anyway, since the board polling its api, you will connect to redis for each request (since serverless env is stateless)
What is the error that you get?

same error as before: https://github.com/felixmosh/bull-board/issues/444#issuecomment-1235734277

@samuelcole, Can you try v5.2.0 which allows you to pass uiBasePath from the outside?
createBullBoard({
    queues: [new BullMQAdapter(exampleBullMq)],
    serverAdapter,
    options: { uiBasePath: 'YOUR_PATH_TO_@bull-board/ui' } // <---
  });

@samuelcole this doesn't seem to work because node_modules isn't included into the resulting Vercel build source.
I also tried https://github.com/vercel/vercel/discussions/4624
!node_modules/@bull-board/ui
!node_modules/@bull-board/ui/**

and https://github.com/vercel/next.js/discussions/14807
// vercel.json
  "functions": {
    "src/pages/api/bull/*": {
      "includeFiles": "node_modules/@bull-board/ui/**"
    }
  }

or worse: tried to brute-force include the node_modules/@bull-board/ui directory into public (which makes Vercel include it into the build source), but none of these approaches worked, always failing w/ a message similar to:

Failed to lookup view "index.ejs" in views directory "node_modules/@bull-board/ui/dist"

Let me know if you had any success.

For anyone using Nuxt 3, and Vercel who winds up here...
Here's how I got around the issue.
In your vercel.json add this (credit to spb-web)
{
  "buildCommand": "nuxt build && cp -R ./node_modules/@bull-board/ui .vercel/output/functions/__nitro.func/node_modules/@bull-board/ui"
}

Then in your bull-board config
options: { uiBasePath: `${process.cwd()}/node_modules/@bull-board/ui`, },

I tried for a while to use the includeFiles option in my vercel config, but Nuxt bundles everything into nitro so it wasn't working.


Use latest version of YTPlayerView

Thank you for the PR! It looks like this fixes some issues with videos on iOS 14 as well. @colejd we should look into using the library from the source, although it looks like Google hasn't updated it in months


Update to remove the need to install istio via the istio operator

Currently to be able to avail of the rate limit and auth policy in Kuadrant you need to have an istio operator cr. To be able to have an istio operator cr you have to install istio operator which is now discouraged by the istio team.  Istioctl and Helm installation methods are recommended instead.

Personally, I don’t think that’s the direction we should be going, and neither should Istio IMHO.

While I agree, Istio is already going that way as IstioOperator will not have new features and is discouraged (https://istio.io/latest/docs/setup/install/operator/)

When you use the istioctl or helm approach, is there no configuration resource created that kuadrant could hook into if it was told where that resource was? I guess my question is where is the config stored if not when not in the operator resource?

@maleck13
There is a ConfigMap that by default looks like this:
apiVersion: v1
kind: ConfigMap
metadata:
  namespace: istio-system
  name: istio
  labels:
    install.operator.istio.io/owning-resource: istiocontrolplane
    install.operator.istio.io/owning-resource-namespace: istio-system
    istio.io/rev: default
    operator.istio.io/component: Pilot
    operator.istio.io/managed: Reconcile
    operator.istio.io/version: 1.13.2
    release: istio
data:
  mesh: |-
    defaultConfig:
      discoveryAddress: istiod.istio-system.svc:15012
      proxyMetadata: {}
      tracing:
        zipkin:
          address: zipkin.istio-system:9411
    enablePrometheusMerge: true
    rootNamespace: istio-system
    trustDomain: cluster.local
  meshNetworks: 'networks: {}'

The ConfgiMap can be patched in the following way to inject the extension provider info:
apiVersion: v1
kind: ConfigMap
metadata:
  namespace: istio-system
  name: istio
  labels: […]
data:
  mesh: |-
    defaultConfig:
      discoveryAddress: istiod.istio-system.svc:15012
      proxyMetadata: {}
      tracing:
        zipkin:
          address: zipkin.istio-system:9411
    enablePrometheusMerge: true
    extensionProviders:
    - envoyExtAuthzGrpc:
        port: 50051
        service: authorino-authorino-authorization.kuadrant-system.svc.cluster.local
      name: kuadrant-authorization
    rootNamespace: istio-system
    trustDomain: cluster.local
  meshNetworks: 'networks: {}'

The change is not reconciled back to the IstioOperator custom resource that Istio will store in the cluster regardless. So we'd need to reflect the change there as well.
We may also need our operator to have permissions to redeploy Istio, as I'm not sure if the changes to the ConfigMap will take effect otherwise.

Ok. Redeploying Istio might not be needed in the end, which BTW is consistent with the instructions here.
I've tested locally with the following steps from the Kuadrant Operator repo (even though not really using Kuadrant here):
❶ Create a cluster with Gateway API and Istio:
make kind-create-cluster gateway-api-install istioctl
./bin/istioctl install -f config/dependencies/istio/istio-operator.yaml
make deploy-gateway

❷ Deploy a sample API:
kubectl apply -f https://raw.githubusercontent.com/kuadrant/authorino-examples/main/talker-api/talker-api-deploy.yaml

kubectl apply -f - <<EOF
apiVersion: gateway.networking.k8s.io/v1alpha2
kind: HTTPRoute
metadata:
  name: talker-api
spec:
  parentRefs:
  - name: istio-ingressgateway
    namespace: istio-system
  hostnames:
  - talker-api.local
  rules:
  - backendRefs:
    - name: talker-api
      port: 3000
EOF

❸ Deploy Authorino (without Kuadrant):
kubectl apply -f https://raw.githubusercontent.com/Kuadrant/authorino-operator/main/config/deploy/manifests.yaml

kubectl apply -f -<<EOF
apiVersion: operator.authorino.kuadrant.io/v1beta1
kind: Authorino
metadata:
  name: authorino
spec:
  listener:
    tls:
      enabled: false
  oidcServer:
    tls:
      enabled: false
EOF

❹ Edit the mesh config:
kubectl edit configmap istio -n istio-system

Add the following bit to the defaultConfig:
extensionProviders:
- name: authorino
  envoyExtAuthzGrpc:
    port: 50051
    service: authorino-authorino-authorization.default.svc.cluster.local

❺ Wire ext-authz up in the gateway for the sample API:
kubectl apply -n istio-system -f - <<EOF
apiVersion: security.istio.io/v1beta1
kind: AuthorizationPolicy
metadata:
  name: talker-api-ext-authz
spec:
  selector:
    matchLabels:
      app: istio-ingressgateway
      istio: ingressgateway
  action: CUSTOM
  provider:
    name: authorino
  rules:
  - to:
    - operation:
        hosts: ["talker-api.local"]
        paths: ["/secure*"]
EOF

❻ Send requests:
kubectl port-forward service/istio-ingressgateway 8000:80 -n istio-system 2>&1 >/dev/null &

curl -H "Host: talker-api.local" http://localhost:8000/secure -i
# HTTP/1.1 404 Not Found
# x-ext-auth-reason: Service not found  <=== It means the request hit Authorino


HMR broken using Angular 2.4

Create a new Angular project with the experimental patch (v0.8.1). The app runs fine the first time but if you update a file (say home.component.html) the browser goes blank and you see a bunch of errors in the console:
EXCEPTION: Error in :0:0 caused by: The selector "app" did not match any elements

Did you rename the root component from <app>?
Check your Views/Home/index.cshtml, and also what your app.component.ts selector is.
Wasn't able to reproduce this one locally.

Nope.
This generator can also be run with: yo aspnetcore-spa

     _-----_     ╭──────────────────────────╮
    |       |    │  Welcome to the ASP.NET  │
    |--(o)--|    │   Core Single-Page App   │
   `---------´   │        generator!        │
    ( _´U`_ )    │                          │
    /___A___\   /│      Version: 0.8.1      │
     |  ~  |     ╰──────────────────────────╯
   __'.___.'__
 ´   `  |° ´ Y `

? Framework Angular 2.4.5 (experimental)
? What type of project do you want to create? project.json (compatible with .NET Core tooling preview 2 and Visual Studio 20
15)
? Do you want to include unit tests? Yes

Ran: cross-env ASPNETCORE_ENVIRONMENT=Development dotnet run
Then changed home.component.html header to:
Hello, world! XXX

Save => blank screen with error.

When HMR updates modules, it detaches and reattaches changed components. DomRenderer.prototype.detachView removes all component elements which have parentNode defined (for app element it will be body element). As a temporary bypass I use
ngOnDestroy(): void {         document.body.appendChild(document.createElement("app"));     }

I also am seeing this.  Thank you @nikitadmitry  - I added that line in the ctor of app.component and this has helped resolve the issue for me also.
I also have not changed the default name app as the selector.

@nikitadmitry Thank you for the interim fix - I'm back in business over here as well

For me HMR broke with angular 2.4.6.
We reverted back to 2.4.5 and it works.

Thanks for the report. I've published generator-aspnetcore-spa 0.8.2 which pins to Angular 2.4.5. I'll look at relaxing this versioning restriction when I get a chance to investigate the issue with 2.4.6.

The template is now updated to support HMR with newer Angular 2.x (tested with 2.4.7).


Tags for Nested Structures Are Not Properly Reflected in OpenAPI Specification Generation

When extracting a structure and using it as an attribute in another structure, the OpenAPI specification generation does not consider the tags for the external structure. Specifically, tags such as example and description for the nested structure (References in this case) are not being included in the generated OpenAPI specification.
Example:
Here is a sample Go package illustrating the issue:
package models

type Pets struct {
    ID         string     `json:"id" validate:"required" example:"pet-123456"`
    Name       string     `json:"name" validate:"required" example:"Napoleon"`
    Age        int        `json:"age" example:"2" description:"Age of the pet, in years"`
    IsAdopted  bool       `json:"is_adopted" description:"Is the pet adopted"`
    References References `json:"references"`
}

type PetsCreate struct {
    Name       string     `json:"name" validate:"required,min=1,max=100" example:"Napoleon"`
    Age        int        `json:"age" validate:"min=0,max=100" example:"2" description:"Age of the pet, in years"`
    IsAdopted  bool       `json:"is_adopted" description:"Is the pet adopted"`
    References References `json:"references"`
}

type PetsUpdate struct {
    Name       string     `json:"name,omitempty" validate:"min=1,max=100" example:"Napoleon" description:"Name of the pet"`
    Age        int        `json:"age,omitempty" validate:"max=100" example:"2"`
    IsAdopted  *bool      `json:"is_adopted,omitempty" description:"Is the pet adopted"`
    References References `json:"references"`
}

type References struct {
    Type  string `json:"type" validate:"required" example:"pet-123456"`
    Value string `json:"value"`
}

Observed Behavior:
The OpenAPI specification generated does not include the example and description tags for the nested structure References.
Expected Behavior:
The generated OpenAPI specification should reflect all tags (e.g., example and description) for nested structures, ensuring the documentation is complete and accurate.

Well found!
@dylanhitt I'll be unavailable until Jan 2nd 2025 so if there is any PR related to this issue, I let you in full control of merging it!
This "bug" is simply related to the fact that we don't search for nested structure tags. This can be fixed quickly when I come home in 6 days.
In the meantime, I hope it's not a blocker for you. Validation should still be working even if it's not mentionned in the spec.

Cool, I should hopefully have time tomorrow assuming travel goes as planned.

Should be fixed once #319 is merged.


Show casters not showing💀

I've tried EVERY single version of mono director to see if this bug will fix but it hasn't. When I click on show casters it doesn't show up anywhere, I often mess up my fight recordings and I know there is an option called "recast" that lets you redo your recording. But I can't go into "show casters", I'm starting to think that it's the way I put in the files of mono director. And when I look at the README file its still confusing. I'm 80% sure I did it the right way though. I was wondering if there was a video tutorial that someone has for PCVR, because the only tutorial up is for quest. Anybody know why this is happening?
@adamd3v do u know why this is?

i fixed it as of recent but i don't believe i pushed it, it's been a while, sorry

i fixed it as of recent but i don't believe i pushed it, it's been a while, sorry

its fine just wanting to know when its gonna be published lol


Ensure a sensors is updated after each read.

The energy sensors based on an output (ending with _wh) may not contain the correct result due to how the iotawatt is calculating them (such as a typical Import and Export energy rule).
It may be required to use the integration integration to calculate the energy the power device used over a given amount of time.
For the integration to rule, the sensor state must be updated frequently.
The Home Assistant value is to only update the value of a sensor if a change occurred which isn't what we want.
So we configure the HA DataUpdateCoordinator for forced updates.
Fixes #15

I assume you have made this change to your system. How is it working?

So far so good, now integration with the right method is correct.

Nice! Hopefully this gets accepted soon!


Canvas: Signing fails when a query string is present

I'll take a look at this one myself, basically if a query string is present in the request body it is not being correctly accounted for per the OAuth spec. Shouldn't be that difficult of a fix.

Is this not fixed by/related to ba05862f341aefa784a69aa67b6bd131df991628 and #4?

I actually have been debugging and this may be a Canvas specific issue, upon inspection I find my query params listed twice. Turns out those query parameters were added to the request body. I'm doing one last test to verify that is the case but I'm actually finding no issues with this extension itself.

Just checked the raw request body and it looks like it's including the query params. I'll speak with a Canvas tech and see if there is something we're overlooking. Doesn't appear to be in the spec though.

Going to cross reference the ticket: instructure/canvas-lms#600

Great, will let you run with it

Going to go ahead and implement this as a "fix" since their choice of implementation is a little off. I'll go ahead and put in some code to special case incoming Canvas requests.

Fixed by #41


Attach to Chrome functionality fails

I am trying to attach to Chrome that is located in /home/faraui/chrome-linux64/chrome. The code is the following:
#!/bin/bash

./chrome-linux64/chrome --remote-debugging-port=9222 --remote-allow-origins=*

perl -I "/home/faraui/extlib/lib/perl5" << 'lrep'

use WWW::Mechanize::Chrome;

my $mech = WWW::Mechanize::Chrome->new(
    launch_exe => '/home/faraui/chrome-linux64/chrome',
    port => 9222,
);

$mech->get("https://google.com");

lrep

$mech->get("https://google.com"); command does not execute. It seems that there is a problem with creating $mech object.
When I launch the code above, the result is:
$ ./main.sh

DevTools listening on ws://<IP_ADDRESS>:9222/devtools/browser/71dacf0b-d337-4764-8bf4-40d97afb60f1
Created TensorFlow Lite XNNPACK delegate for CPU.
Attempting to use a delegate that only supports static-sized tensors with a graph that has dynamic-sized tensors (tensor#141 is a dynamic-sized tensor).

and nothing happens later.
The problem is still present if there is no launch_exe, and even if I launch Chrome with chromium-browser command instead.
I am using:

Fedora Linux v38; kernel v6.8.9-100.fc38.x86_64
Perl v5.36.3, built for x86_64-linux-thread-multi
WWW::Mechanize::Chrome v0.73
Chrome v126.0.6478.55 (r1300313) (that is /home/faraui/chrome-linux64/chrome)
Chromium 125.0.6422.60, Fedora Project (that is chromium-browser)

The solution I have found was hilarious: Launch chrome with sudo!!! =)
Entschuldigung... The issue is closed. Thanks for this module!


Improve GraphQL error messages

💩 How it is currently developed
Our GraphQL errors have not customization while returning to the request. They're like NestJS returns it.
🌟 What is the optimal solution
We should improve the format of GraphQL errors. We could follow this suggestion: https://github.com/nestjs/graphql/issues/1053#issuecomment-740739410
😓 Why we did that way
To move fast.
💣 What are the possible impacts
We can lose time debugging because our error is not formatted accordingly.

Closed since we're now using ApolloErrors.


Using Error as a cause loses backtrace.

The AsFail implementation for Error only looks for a backtrace on the underlying Fail, and skips backtraces attached directly to the Error. This means that Errors constructed from implementors of the std::error::Error trait lose their backtraces when they are used as a cause.

Confusion around this resulted in rust-lang/crater#455. It would be ideal if Error implemented Fail, but this is impossible due to coherence issues (#99).


merge package from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template
chore(package): set $.ava.extensions='js,mjs' as in template

Codecov Report

Merging #384 into master will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #384      +/-   ##
==========================================
- Coverage   83.64%   83.56%   -0.08%
==========================================
  Files           1        1
  Lines         214      213       -1
==========================================
- Hits          179      178       -1
  Misses         35       35

Impacted Files
Coverage Δ

src/service-health-check.js
83.56% <0%> (-0.08%)
:arrow_down:

Continue to review full report at Codecov.

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

Coverage decreased (-0.07%) to 84.322% when pulling c77ae3971e6a9b69e01de22db888a1056f87486c on template-sync-1 into ed90b4bef9d1aad90b2697ce6b75001cb0a7e30c on master.

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


It does not work normally paging.

Hi. I can not speak English well, please forgive me.
Describe the bug
Occurs when there are many articles.
Paging will not work.
My Site and Sample too.
top menu change social account bar.
How to fix it ?
Your project
https://github.com/SasakiYuki/bibo-log
my project is here.
PS: Thanks for this awesome theme !!

@SasakiYuki I don't have any issues with your English. From what I think, you must be facing issues navigating between pages on your site, right? The pagination is working fine since I'm able to access all the pages via the address bar, so the issue here is that the navigation buttons get hidden. This, I suppose arises from the theme itself, I'll fix it ASAP, and notify you when it's fixed.
If there are some other issues, which I might have missed, state them in the comments.

@UtkarshVerma Thanks!!
Yes, I also think that paging is going well.
I like this theme, so I am looking forward to fixing the bug.

@SasakiYuki It's done. If something still fails, which is quite unlikely, feel free to reopen this issue or create a new one.

Thanks @UtkarshVerma .
Fixed it.
I will use this theme forever! !


Selecting derived variable and statistical data error

When in Select panel, if a user selects the following in order, a error traceback is prodcued:

Initialize panel
Select daily for "derived index" to show up
Select statistical
Panel breaks

This is because derived indices are not hooked up for LOCA2 data.
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/pyviz_comms/__init__.py", line 346, in _handle_msg
    self._on_msg(msg)
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/panel/viewable.py", line 292, in _on_msg
    doc.unhold()
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/bokeh/document/document.py", line 799, in unhold
    self.callbacks.unhold()
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/bokeh/document/callbacks.py", line 396, in unhold
    self.trigger_on_change(event)
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/bokeh/document/callbacks.py", line 373, in trigger_on_change
    invoke_with_curdoc(doc, event.callback_invoker)
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/bokeh/document/callbacks.py", line 408, in invoke_with_curdoc
    return f()
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/bokeh/util/callback_manager.py", line 191, in invoke
    callback(attr, old, new)
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/panel/reactive.py", line 392, in _comm_change
    self._schedule_change(doc, comm)
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/panel/reactive.py", line 376, in _schedule_change
    self._change_event(doc)
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/panel/reactive.py", line 370, in _change_event
    self._process_events(events)
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/panel/reactive.py", line 311, in _process_events
    events = self._process_property_change(events)
  File "/srv/conda/envs/notebook/lib/python3.9/site-packages/panel/widgets/select.py", line 488, in _process_property_change
    msg['value'] = list(self.values)[index]
IndexError: list index out of range```

I assume this is something we want to do for LOCA2?

I assume this is something we want to do for LOCA2?

@elehmer We actually don't want to provide the derived variable/indices for LOCA2 data at all - the raw variables needed for those derived variables aren't available in LOCA2

Hmm, I am not able to reproduce this error. Panel transitions to "Statistical" and removes the "Derived Index" variable options. @vicford Did you encounter this the first time through the code or did you select several options first?

I'm also now not seeing it pop up... I wonder if it was just a one time weirdness then... I'll close this issue then, crisis averted!


Telemetry Tool Install Need Patches/Improvements

Describe the bug
Following task needed changes/improvements in telemetry install
1.Alert manager is currently running as a binary. Is it poosible to make it as a service like other tools.
2. Prometheus config need to be updated after pushgateway installation
Add below entry  in to /etc/prometheus/prometheus.yml

job_name: 'pushgateway'
scrape_interval: 5s
honor_labels: true
static_configs:

targets: ['localhost:9091']

To Reproduce
Insatll hotpot using ansible and check Prometheus configuration file /etc/prometheus/prometheus.yml
Expected behavior
Alert manager will run as a service
/etc/prometheus/prometheus.yml will have scrape config for  pushagateway also
Desktop (please complete the following information):

OS: [e.g. iOS] Ubuntu 18.04

/assign @joseph-v


New side panel feature breaks previously possible window layout of: code editor | terminal | side bar

VS Code Version: 1.64.0
OS Version: MacOS 12.1

In version 1.63.2, it was possible to have a window layout with the code editor on the left, followed by the terminal, followed by the side bar on the right, as seen here:

In version 1.64.0, with the new side panel, this is the default view:

The only option I see now is to put the terminal in the side panel (using "Move Views From Panel To Side Panel"). But the side panel appears to only be able to be positioned on the side opposite the side bar.

So it appears to be impossible to place the terminal between the code editor on the left and the side bar on the right.
This may not seem like a big deal, but this is the layout I've been accustomed to for years, going back to before VSCode, and it has resulted in a lot of muscle memory of where to look on the screen, which makes rearranging things pretty disruptive. I've tried moving the side bar to the left and also moving the terminal to the left but it's just not right. When I'm editing my code I don't want to be distracted by anything to the left. But I still need to see the terminal prominently in a column since I'm using it as a REPL, and I still need to see the side bar.
Ideally the editor would continue to be flexible enough to accommodate the code editor | terminal | side bar layout.

/duplicate of #141349


Media: Integrated with New Search API

Switch from ISearchable to the new search API

Moved:  https://github.com/DNNCommunity/DNN.Media/issues/8


convert to pnpm

Converts the project to pnpm :)
For users who have corepack enabled, I've added the packageManager field to the package.json. While migrating from yarn, users will notice that yarn refuses to run when packageManager is set:

I also bumped setup-node, as the v4 README is what I followed to ensure pnpm cache compatibility. While I was there I bumped actions/checkout as well.

i tested this locally; lgtm :DDD tysm


DirectPartitioner - directly use integer keys when partitioning is pre-computed.

Add DirectPartitioner uses pre-computed partition numbers, provided as keys.
Update RDD.treeAggregate to use DirectPartitioner when it is sufficient (avoid computing hashes when not necessary).

Can one of the admins verify this patch?

Have a look at https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark please; this isn't how to propose a change like this. Do you mind closing this PR until then?

Got it, thanks Sean! Let's pause this until later.

Opened a ticket to consider this change: https://issues.apache.org/jira/browse/SPARK-7342


Add AfterBindValue to FluentInputBase (Fixes #472)

Moved CurrentValue setter to SetCurrentValue method so event-driven attempts to set CurrentValue can await ValueChanged etc.
Moved CurrentValueAsString setter to a method for the same reason.
Added AfterBindValue event to circumvent a .Net 6 lack of @bind-Value:after that was introduced in .Net 7 - this ensures a consumer can execute code after a databound value has been updated.
Ensured all demos use AfterBindValue.

Good work. I wonder if a comment should be added to the AfterBindValue property indicating that this parameter is equivalent to bind:after of .NET7+. What do you think?

Good work. I wonder if a comment should be added to the AfterBindValue property indicating that this parameter is equivalent to bind:after of .NET7+. What do you think?

Done. Thanks for the feedback!


[DOCS] Adding contributing guidelines.md and code of conduct.md

Describe the documentation update

As an open source project the code_of_conduct and contributing file should be present in repository to guide contributor about rules and regulation for working on codebase.

Why is this needed?

This are the part of opensource project environment.

Refactoring Readme  to include:

license badge
proper alignment and including links and redirection to contributing,md and code of conduct file
showing star history for project repository

@07sumit1002 let me know

@Khushalsarode @07sumit1002 as i'm new to open source contributions i will take a lead on this, assign me the task i will open pr till 4'O clock.


Add more documentation

Newly added pages:

[x] Migrating from Flask
[x] Swagger UI and Redoc
[x] Authors

Codecov Report

Merging #28 (d7ed52f) into master (0441292) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head d7ed52f differs from pull request most recent head fc931b9. Consider uploading reports for the commit fc931b9 to get more accurate results

@@            Coverage Diff            @@
##            master       #28   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           12        12
  Lines          673       673
  Branches       124       124
=========================================
  Hits           673       673

Continue to review full report at Codecov.

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


Updates Post Detail and Profile screens for logged-out views

Removes the ellipsis button from the navigation bar.
Also noticed that the grid/list toggle was not the same for "meta" categories and "actual" categories, and we had an unused StreamKind (categoryPosts).
Running specs now.

I had some specs to fix, they're all good now, and I added the "background color change" from Lucian, w/ updated snapshot specs.


Update RegexMaskingOperator.cs

This to override the MatchEvaluator in a specific Operator to test each value with more complex validaters.

@j9850s thank you for your contribution.
Based on the discussion we've had via e-mail I've added a new extension point that should provide the options you need.
It's been implemented in 382349d41b043cde05f6cd5ee04248ca7ddf067c and will be available on NuGet shortly as version <IP_ADDRESS>
See https://github.com/serilog-contrib/Serilog.Enrichers.Sensitive#extending-to-additional-use-cases for the documented extension points, you'd be interested in ShouldMaskMatch and an example is provided in this test case


Unknown file extension ".json" when using @unocss/preset-icons

This is reproduce https://stackblitz.com/edit/github-rgqsff?file=vite.config.ts
I cannot import 'json' file when using dynamic import

for example
collections: {
  mdi: () =><EMAIL_ADDRESS>{ assert: { type: "json" }}).then(i => i.default),
},

I'm having the same error. I cant seem to make this work with Typescript/Vite project using ESModules

I'm having the same error. I cant seem to make this work with Typescript/Vite project using ESModules

import() is brower's api, not nodejs. you can use required() instead


Allow configuration of list options

We're chucking an empty metav1.ListOptions when we look up resources.
People may want to filter based on a LabelSelector/FieldSelector, etc.

Don't need this anymore


Ubuntu install guide: can't execute Podman

Following the install guide for Ubuntu (18.04.2), I end up in a rather unusable state as some configs are not installed.
valentin@uvm:~$ podman run alpine ls
Error: error creating libpod runtime: Error running podman info while refreshing state: exit status 1
valentin@uvm:~$ sudo podman run alpine ls
Error: unable to pull alpine: error getting default registries to try: unable to parse the registries.conf file: open /etc/containers/registries.conf: no such file or directory

I think the package must be updated. Let's use the issue as a tracker.

@lsm5, can you have a look? I am not sure if it's a known issue or not. I just installed an Ubuntu VM for some performance tests.

Known issue, but I don't think we've collated all of this in the same place before.
I walked someone on IRC through getting things working a few weeks back, and we encountered the following issues:

No dependency on uidmap package (needed for rootless)
Missing slirp4netns (no package)
Missing registries.conf

@lsm5 was looking at this package today.  Is this issue fixed?

sorry, been fighting some rhel fires, but this is on my todo for today, closing this as duplicate of https://github.com/containers/libpod/issues/2116


Desktop Widgets

I propose that we add back the desktop Widgets with more productivity in mind. Like a analog clock, or the news from CNN, or a mini command prompt. I think this would help developers and people get more done with the system. I've never done C# or C but I'd love to learn!

I think Rainmeter already covers the desktop widget niche pretty well, although the way you create skins (widgets) and manage them is a bit limited.


Symbol resolution refactor

The symbol table logic started out convoluted and became a nightmare while I was thrashing about trying to figure out closures. It should be refactored and distinct data structures extracted. A clean design is essential for:

Scope-aware autocompletion
Static typing and further compiler phases
Scope-aware search
Scope visualisation

It would also likely be helpful to simplify before separating the GUI and interpreter threads.

I'm closing this issue with https://github.com/JohnDTill/Forscape/pull/8. It could still be cleaner, but it's not immediately obvious how, and separating the passes was the most pressing issue.


Inbound calls not working

My inbound calls are not working. I have implemented everything and when i call from my phone it just ends the call and when I call from the unverified number it shows me in the error logs that this call came from the unverified number but from my verified phone number it just shows nothing and call ends. Can anyone please tell me why this is happening.

Check out this updated Version https;//github.com/d10xi24/voicelink .


AVG Thinks it a virus

Describe the bug
A clear and concise description of what the bug is.
AVG Thinks it a virus IDP.Generic
To Reproduce
Steps to reproduce the behavior
1 Download AVG
2 Download App
3 Open App
4 Virus Warning
Expected behavior
A clear and concise description of what you expected to happen.
It works
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Windows 7]
Version [e.g. 1.2.1]

Additional context
Add any other context about the problem here.

App's exe file is built using Launch4j, which often creates false positives.
The file itself is safe to execute, but if your antivirus prevents you from opening it, try downloading the .jar version of application.

Resolved in release 1.5.5