Experiencing ResponseNotRead errors

Hey we are currently experiencing the follow error on v0.10.3:

4-06-19T07:24:11.108Z | ERROR: Exception in ASGI application
-- | --
  | 2024-06-19T07:24:11.108Z | Traceback (most recent call last):
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/uvicorn/protocols/http/httptools_impl.py", line 399, in run_asgi
  | 2024-06-19T07:24:11.108Z | result = await app( # type: ignore[func-returns-value]
  | 2024-06-19T07:24:11.108Z | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/fastapi/applications.py", line 1054, in __call__
  | 2024-06-19T07:24:11.108Z | await super().__call__(scope, receive, send)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/applications.py", line 123, in __call__
  | 2024-06-19T07:24:11.108Z | await self.middleware_stack(scope, receive, send)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/middleware/errors.py", line 186, in __call__
  | 2024-06-19T07:24:11.108Z | raise exc
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/middleware/errors.py", line 164, in __call__
  | 2024-06-19T07:24:11.108Z | await self.app(scope, receive, _send)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 65, in __call__
  | 2024-06-19T07:24:11.108Z | await wrap_app_handling_exceptions(self.app, conn)(scope, receive, send)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/_exception_handler.py", line 64, in wrapped_app
  | 2024-06-19T07:24:11.108Z | raise exc
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/_exception_handler.py", line 53, in wrapped_app
  | 2024-06-19T07:24:11.108Z | await app(scope, receive, sender)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 756, in __call__
  | 2024-06-19T07:24:11.108Z | await self.middleware_stack(scope, receive, send)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 776, in app
  | 2024-06-19T07:24:11.108Z | await route.handle(scope, receive, send)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 297, in handle
  | 2024-06-19T07:24:11.108Z | await self.app(scope, receive, send)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 77, in app
  | 2024-06-19T07:24:11.108Z | await wrap_app_handling_exceptions(app, request)(scope, receive, send)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/_exception_handler.py", line 64, in wrapped_app
  | 2024-06-19T07:24:11.108Z | raise exc
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/_exception_handler.py", line 53, in wrapped_app
  | 2024-06-19T07:24:11.108Z | await app(scope, receive, sender)
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/starlette/routing.py", line 72, in app
  | 2024-06-19T07:24:11.108Z | response = await func(request)
  | 2024-06-19T07:24:11.108Z | ^^^^^^^^^^^^^^^^^^^
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/fastapi/routing.py", line 278, in app
  | 2024-06-19T07:24:11.108Z | raw_response = await run_endpoint_function(
  | 2024-06-19T07:24:11.108Z | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/fastapi/routing.py", line 191, in run_endpoint_function
  | 2024-06-19T07:24:11.108Z | return await dependant.call(**values)
  | 2024-06-19T07:24:11.108Z | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  | 2024-06-19T07:24:11.108Z | File "/app/powerproxy.py", line 371, in handle_request
  | 2024-06-19T07:24:11.108Z | f"Text: {aoai_response.text} "
  | 2024-06-19T07:24:11.108Z | ^^^^^^^^^^^^^^^^^^
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/httpx/_models.py", line 576, in text
  | 2024-06-19T07:24:11.108Z | content = self.content
  | 2024-06-19T07:24:11.108Z | ^^^^^^^^^^^^
  | 2024-06-19T07:24:11.108Z | File "/usr/local/lib/python3.11/site-packages/httpx/_models.py", line 570, in content
  | 2024-06-19T07:24:11.108Z | raise ResponseNotRead()
  | 2024-06-19T07:24:11.108Z | httpx.ResponseNotRead: Attempted to access streaming response content, without having called `read()`.

We temporarily could fix the issue by adding await aoai_response.aread() to the status check.
        if aoai_response.status_code != 200:             # print infos to console             await aoai_response.aread()             print(                 (                     f"Unexpected HTTP Code {aoai_response.status_code} while using target '{aoai_target['name']}'. "                     f"Text: {aoai_response.text} "                     f"Path: {routing_slip['path']} "                     f"Url: {aoai_target['url']}"                 )             )
No we get Unexpected HTTP Code 408 while using target ... Errors that are no handled by the following
`
# got 429 or 500
if aoai_response.status_code in [429, 500]:
# block endpoint for some time, either according to the time given by AOAI or, if not
# available, for 10 seconds
waiting_time_ms_until_next_request = (
int(aoai_response.headers["retry-after-ms"]) if "retry-after-ms" in aoai_response.headers else 10_000
)
aoai_target["next_request_not_before_timestamp_ms"] = (
get_current_timestamp_in_ms() + waiting_time_ms_until_next_request
)
        # try next target
        continue

`
Shouldn't this include handling of timeouts as well?

@LeonKalt I have fixed the .aread() issue and checked into main. I will create a new release later today. Thanks for reaching out and bringing this up!
The 408 timeout occurs when the client does not send the request fast enough to the server. If I add 408 to the list, we would simply hide an issue which could/should potentially be fixed. Do you have an idea why a 408 is encountered in your scenario? Could it be related to where you host PowerProxy and/or how the network connection is under pressure? What kind of prompts are you sending? Is larger prompts? Just trying to better understand the context before adding 408 to the list above. Thanks!

I have seen you have opened another timeout-related issue, rendering this as duplicate, hence closing.

@LeonKalt I have fixed the .aread() issue and checked into main. I will create a new release later today. Thanks for reaching out and bringing this up!
The 408 timeout occurs when the client does not send the request fast enough to the server. If I add 408 to the list, we would simply hide an issue which could/should potentially be fixed. Do you have an idea why a 408 is encountered in your scenario? Could it be related to where you host PowerProxy and/or how the network connection is under pressure? What kind of prompts are you sending? Is larger prompts? Just trying to better understand the context before adding 408 to the list above. Thanks!

Our managed PTU had some issues so we ran into a scenario where customers where effected instead of falling back on our paygo deployment and although I understand the issue of hiding errors, I think full filling the request with an answer is preferable to just returning an error to the application.

Ok I see. I have added the 408/Request Timeout code to the list of codes that will trigger trying the next AOAI target instead of just forwarding the 408 back. Will create a new release in a moment.


Added spanish language

I added spanish language, please merge to update my app

@recchia  Thank you , @alenoosh  did merged your commit , looking forward to work together in laravelpanel .


Applicant can create new account (register / sign up)

We have a page where the applicant can log in, but we don't have one where they can register for a new account.

Note that registering for an account requires you to visit https://login.seattle.gov/#/login.

LGTM


Add an option to list CAs

Request for feature: Add "--list" to display all active CA certificates.
This is easy to implement and allows to check, if your add operation was sucessfull. It would save people from having to face certutil itself ;-)
You just need to filter out the intermediate certificates that are saved in cert8.db also.

I did a simple implementation of a show-certs.cmd
Use it as you see fit.
show-certs.txt


fix(java): improve property override detection

The previous code only checked whether a methodd name was a get/setter
by checking if it's name started by get or set and was at least 4
characters long. This is insufficient, as a method called settings
would be misidentified.
Changed to expect getters to have zero parameters, and setters to have
exactly one parameter and to match a corresponding getter.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I don't know what caused the package-lock.json changes but they keep coming back if I try to undo them. I am actually suspecting the no-diff enforcement of the Travis build is somehow not working here...


RPC methods in PAPI

Can this query be replicated by using PAPI?

api.rpc.author.pendingExtrinsics
Reference - https://polkadot.js.org/docs/substrate/rpc#pendingextrinsics-vecextrinsic

Tried looking at the docs, but I don't see a clear example of it

RPC methods are usually not standard and are not defined by the metadata, so papi can't add types and other features on them. But they are still accessible through client._request, the client reference on the docs has a small example as part of the JSDoc:
  /**
   * This API is meant as an "escape hatch" to allow access to debug endpoints
   * such as `system_version`, and other useful endpoints that are not spec
   * compliant.
   *
   * @example
   *
   *   const systemVersion = await client._request<string>("system_version", [])
   *   const myFancyThhing = await client._request<
   *     { value: string },
   *     [id: number]
   *   >("very_fancy", [1714])
   *
   */
  _request: <Reply = any, Params extends Array<any> = any[]>(
    method: string,
    params: Params,
  ) => Promise<Reply>

by default it falls back to any types, but you can still use its generics to add types if you're certain they have specific ones.


Run dos2unix on register-number.sh file to prevent CRLF issues

I tested it on my Mac by setting the sh file to CRLF instead of LF and making sure it failed then updating the Dockerfile to ensure it succeeded after that.

Perfect!


Unable to create a valid Base64 encoded Munge key

I followed the guidance outlined here to generate a custom Munge key: https://docs.aws.amazon.com/parallelcluster/latest/ug/Scheduling-v3.html#yaml-Scheduling-SlurmSettings-MungeKeySecretArn
I tried using the mungekey utility and Base64-encoding the resulting file.
$ mungekey -c -k munge-custom.key -b 1024 -f
$ base64 -w 0 munge-custom.key

However, pasting this text into the configured Secrets ARN fails to pass the validator:
{
  "configurationValidationErrors": [
    {
      "level": "ERROR",f
      "type": "MungeKeySecretSizeAndBase64Validator",
      "message": "The content of the secret arn:aws:secretsmanager:us-east-2:[redacted]:secret:pcluster-default-munge-key-YYYYYY is not a valid Base64 encoded string. Please refer to the ParallelCluster official documentation for more information."
    }
  ],
  "message": "Invalid cluster configuration."
}

I got the same results using dd and the suggested Python code.  Any suggestions would be welcomed with open arms.
Required Info:

AWS ParallelCluster version [e.g. 3.1.1]: 3.10.1

Sorry for the late reply.
Please make sure the key is stored as "Plaintext" (instead of "Key/value") in secret manager.

Hi @hanwen-pcluste - thanks for getting back to me.
The "Plaintext" field was explicitly used to save the key.

Thank you, that did the trick.


[otbn/rtl] Consider separate reset scrambling key and nonce for imem and dmem

Currently the scrambling key and nonce for the imem and dmem in OTBN are reset to the same netlist secret (RndCnstOtbnKey/RndCnstOtbnNonce) it may be preferable to introduce separate netlist secrets for them.

@msfschaffner @moidx @andreaskurth @vogelpi my view here is whilst this is likely straight-forward it isn't important for M2.5 hence this should be postponed til after M2.5 (if we decide it's needed at all).

This sounds good to me.

Agreed with postponing this.

Just making sure I understand - the issue is that the DMEM and IMEM use the same key/nonce pair, not that the key and nonce are equal, right? :)
If we think of DMEM+IMEM as one scrambled object then I can convince myself that this is not nonce reuse.
I would suggest it should be changed post-M2.5 though.

Key and nonce are not equal, that is right.
As far as I understand, this issue is really just about the fact that both DMEM and IMEM start out with the same default values after a reset. The scrambling controller inside OTBN actually does request two sets of key/nonce pairs for DMEM/IMEM.

Key and nonce are not equal, that is right.
As far as I understand, this issue is really just about the fact that both DMEM and IMEM start out with the same default values after a reset. The scrambling controller inside OTBN actually does request two sets of key/nonce pairs for DMEM/IMEM.

Correct

Oh.  I somehow missed the bit about it only being the reset value.  Given what what said about requesting two sets of key/nonce pairs this is pretty unlikely to be consequential.
If we are reopening the code base later, then yes, having two different reset values is slightly more hygenic but priority is low from my perspective.

@johannheyszl the requests are only done when explicitly requested by Ibex (the secure wipe imem/dmem commands). So if software runs an OTBN operation without requesting a secure wipe first the imem/dmem will be using the netlist constants for key and nonce.

thx @GregAC !
@jadephilipoom - can you pls comment on the aspect of: ('how much / critical') do we run crypto without triggering secure wipe?
My guess is this will land on the backlog side since the additional gain in security is more on the defense in depth side.

@jadephilipoom - can you pls comment on the aspect of: ('how much / critical') do we run crypto without triggering secure wipe?

We don't run any crypto without a secure wipe first. After reset, ROM runs otbn_load_app before calling the sigverify RSA routine, which includes: https://github.com/lowRISC/opentitan/blob/c5a9cc9a788f4685c4396e9d573c06014af86d4c/sw/device/silicon_creator/lib/drivers/otbn.c#L231-L233

thx @jadephilipoom
@GregAC it seems we can close this issue :)


Using source built library - cannot find CMakeFunctions lib

Describe the bug
Linux, Ubuntu 20
I have built eCAL from source (9.5.2 tag, but problem is also on current code 10.0.0) and then installed it using cpack. Using cmake, I call the package created by providing the path to it using
find_package(eCAL 5.9.2 EXACT REQUIRED PATHS "path/to/library")
but when I build my project, I get folloswing error:
  project has asked CMake to find a package configuration file provided by
  "CMakeFunctions", but CMake did not find one.

  Could not find a package configuration file provided by "CMakeFunctions"
  with any of the following names:

    CMakeFunctionsConfig.cmake
    cmakefunctions-config.cmake

  Add the installation prefix of "CMakeFunctions" to CMAKE_PREFIX_PATH or set
  "CMakeFunctions_DIR" to a directory containing one of the above files.  If
  "CMakeFunctions" provides a separate development package or SDK, be sure it
  has been installed.

even though the CMakeFunctions-0.4.1 is located in lib/cmake.
I fixed the issue by adding  set(CMakeFunctions_DIR "${PACKAGE_PREFIX_DIR}/lib/cmake/CMakeFunctions-0.4.1")
right above the find_package(CMakeFunctions REQUIRED) in the eCALConfig.cmake
Maybe this is solved if eCAL is installed via deb file using dpkg, but doesn"t hurt adding the CMakeFunctions_DIR so the library can be accessed statically.

Hi @SimonTigo,
usually, the user is responsible for telling CMake that it needs to search for libraries/modules in non-standard paths.
The generic way to do this, is to set this on a CMake invocation:
mkdir _build && cmake _build
cmake .. -DCMAKE_PREFIX_PATH="path/to/uncommon/folder"

With a very recent CMake version (>3.20), you can also create a CMake(User)Presets.json file, where you put this information.
I would strongly recommend against doing
find_package(eCAL 5.9.2 EXACT REQUIRED PATHS "path/to/library")

as this effectively will work only on your machine. The CMAKE_PREFIX_PATH approach is a lot more generic.
If you set it, CMakeFunctions will also be found.


[Grid] _settings.scss $breakpoint-classes

changing the var
$breakpoint-classes: (small medium large xlarge);

don't do anything
if i change it in _breakpoint.scss direct
$breakpoint-classes: (small medium large xlarge) !default;

output is generated

@zurb/yetinauts
can somebody confirm that?

I believe this is related to https://github.com/zurb/foundation-sites/pull/9396 - the in-framework build process does not actually reference the settings file (which is a generated file updated on deploy).
We have an open issue to look again at how we handle settings (https://github.com/zurb/foundation-sites/issues/9492), I think I will close this in favor of that.


unable to have model fields with name process

Creating a model name with a field named process will result in the error listed below.
I'm using a revison as the current release doesn't work with the latest version of sql alchemy models https://github.com/flask-admin/flask-admin.git@2c2c98e06e59f0988f34618d6cab7deb6533a61f#egg=Flask_Admin
class User(Base):
    __tablename__ = 'user'
    process = Column(String(120), nullable=True)

builtins.TypeError
TypeError: __call__() takes 1 positional argument but 3 were given

Traceback (most recent call last)
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/app.py", line 1997, in __call__
return self.wsgi_app(environ, start_response)
File "/var/www/current/web/venv/lib/python3.6/site-packages/raven/middleware.py", line 100, in __call__
iterable = self.application(environ, start_response)
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/app.py", line 1540, in handle_exception
reraise(exc_type, exc_value, tb)
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/_compat.py", line 33, in reraise
raise value
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/var/www/current/web/venv/lib/python3.6/site-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functions[rule.endpoint](**req.view_args)
File "/var/www/current/web/venv/src/flask-admin/flask_admin/base.py", line 69, in inner
return self._run_view(f, *args, **kwargs)
File "/var/www/current/web/venv/src/flask-admin/flask_admin/base.py", line 368, in _run_view
return fn(self, *args, **kwargs)
File "/var/www/current/web/venv/src/flask-admin/flask_admin/model/base.py", line 2046, in edit_view
form = self.edit_form(obj=model)
File "/var/www/current/web/venv/src/flask-admin/flask_admin/model/base.py", line 1292, in edit_form
return self._edit_form_class(get_form_data(), obj=obj)
File "/var/www/current/web/venv/lib/python3.6/site-packages/wtforms/form.py", line 212, in __call__
return type.__call__(cls, *args, **kwargs)
File "/var/www/current/web/venv/src/flask-admin/flask_admin/form/__init__.py", line 16, in __init__
super(BaseForm, self).__init__(formdata=formdata, obj=obj, prefix=prefix, **kwargs)
File "/var/www/current/web/venv/lib/python3.6/site-packages/wtforms/form.py", line 278, in __init__
self.process(formdata, obj, data=data, **kwargs)
TypeError: __call__() takes 1 positional argument but 3 were given

This looks like a wtforms problem. Even though it's not documented in their documentation, process is called by the form core library.


Update README.md

Adding usage inspired from https://gist.github.com/SudoCat/7c1323eeff1e83ed2a69b004b8c6024b

Woohoo! Thanks for being the first to contribute, hopefully this extension has been useful to you.
I'm happy to go ahead and merge this, however I thought that other extensions should be possible (albeit a limited selection) through the test option. Admittedly, I primarily only use .njk files, so I can't remember for certain.
This section of the code looks like I did try to consider other templates:
https://github.com/SudoCat/Nunjucks-Isomorphic-Loader/blob/master/src/node-loader.js#L54
Did your experience suggest otherwise?

Thank to you I finally managed to use nunjucks templates with the html-webpack-plugin (no luck with the other loaders out there that could not provide data from webpack to the templates)
I didn't test other extensions than .njk but read it was the only supported ones. I could be wrong on this one.
I just felt frustrated not to have a simple usage docs on the readme so I added it for future users \o/
PS: This is my first PR ever haha

That's excellent! I'm glad to hear it, I encountered the exact same issue, so I wrote this to try and solve that. Stoked to hear it's been useful for someone else 😄
Well, I'll leave it in there then, as .njk is the only one that we know for certain is working! Hopefully I'll get an opportunity to test that and update it.
Thanks a bunch for taking the time to do this - I really should have done it sooner, but honestly still a little surprised to see other people finding this!
Haha, congratulations! This is the first PR I've ever received on any OSS, so it's an exciting time for the both of us! 🎆 🎉

WooHoo :p Here's a little history that lead me here https://stackoverflow.com/questions/47335302/using-nunjucks-with-htmlwebpackplugin-using-dynamic-vars-in-the-template
Glad it's as exiting for you as it is for me


Proxy request filter not working for localhost

The problem
I added a request filter to log all requests.
It works for requests to external sources but not localhost.
Details
in my test I setup the filter
Configuration.proxyEnabled = true;
open();

SelenideProxyServer proxy = getSelenideProxy();
proxy.addRequestFilter("request-logger", new RequestFilter() {
  @Override
  public HttpResponse filterRequest(HttpRequest request, HttpMessageContents contents, HttpMessageInfo messageInfo) {
    log.info("{} {}", request.method(), request.uri());
    return null;
  }
});

My tests run against http://localhost:9000
I open a page (html, css, js etc), click a button which triggers an ajax request (to an endpoint served by http://localhost:9000)
I would expect log entries for html page, associated css, js etc and ajax endpoint
I tried both Firefox and Edge and get the same results.
Both browsers make their own external calls, which I get logging for.
In my JavaScript, where I make the AJAX call I made the following changes:

change URL to https external address (https://www.google.com/) - I get logging
change URL to http external address (http://www.bom.gov.au/) - I get logging
change URL to http://<IP_ADDRESS>:9000/path/to/endpoint - I don't get any logging

Tell us about your environment

Selenide Version: 7.6.1
Edge Version: 129.0.2792.52
Firefox Version: 128.3.1esr
Selenium Version: 4.27.0
OS Version: Windows 10.0.19045

workaround is to set baseURL to  format("http://{0}:9000", InetAddress.getLocalHost().getHostAddress()); or explicitly open pages based on that baseURL.

Hi @Daniel-F-Garcia
Sorry, but I cannot reproduce the problem. I coped your code and added open("http://localhost:9000") after it, and it runs well. Request to "http://localhost:9000" gets intercepted and logged.

Feel free to reopen the issue if we get more information.


Consider a rework to remove the need for a base class for specifications

Currently, the specification interfaces have a number of methods and properties. In each case, only one of those is important, the rest is all supporting logic.
Rather than pushing that supporting logic into a base class, it would be far better to move it elsewhere. Then specifications will not need to implement a base class at all.

The API I really want for implementers is:
public interface ISpecification<T>
{
    Func<T, bool> GetPredicate();
}

public interface ISpecificationExpression<T>
{
    Expression<Func<T, bool>> Get Expression();
}

Other functionality should be implemented either as extension methods or other services.

List of extension methods to replace base class functionality
Here is a list of the extension methods which I would like to provide.  I'd like to provide these for both kinds of specification, function and expression based ones.  Where the following mentions spec functions, I'd like another version which is identical but works with expressions.
bool Matches<T>(this ISpecificationFunction<T> spec, T object);
Predicate<T> AsPredicate<T>(this ISpecificationFunction<T> spec);
ISpecificationFunction<T> Not<T>(this ISpecificationFunction<T> spec);

Also, just for expressions, I'd like the following:
ISpecificationFunction<T> AsSpecificationFunction<T>(this ISpecificationExpression<T> spec);


Implement Newsletter Subscription Feature Using Firebase

This PR solves the issue #118 by adding the functionality for users to subscribe to the newsletter, allowing them to receive important updates. The following changes have been made:

Integrated Firebase to handle newsletter subscriptions.
Created a subscription form where users can enter their email addresses.
Implemented form validation to ensure valid email input.
Added error handling for failed subscriptions.

Please review the changes and let me know if there are any modifications needed.

@jeevan10017 Changes have been reverted, and the original Firebase credentials have been restored. Please check and let me know if everything looks good!


MIC-110 add get csip endpoint with filters

Codecov Report
Attention: Patch coverage is 96.61017% with 2 lines in your changes missing coverage. Please review.

Project coverage is 87.40%. Comparing base (6df507e) to head (6357361).

Files
Patch %
Lines

...challengesupportinterventionplanapi/entity/Plan.kt
50.00%
0 Missing and 1 partial :warning:

...rtinterventionplanapi/service/CsipRecordService.kt
95.23%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #124      +/-   ##
============================================
+ Coverage     87.03%   87.40%   +0.37%
- Complexity      685      706      +21
============================================
  Files           133      135       +2
  Lines          2106     2160      +54
  Branches        141      147       +6
============================================
+ Hits           1833     1888      +55
+ Misses          191      188       -3
- Partials         82       84       +2

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


DOC: Simplify study dictionary creation

Instead of assigning each individual dictionary key-value pair in its own Python expression, more concisely and clearly assign them all in one combined Python expression, using the standard dict(...) constructor.  (This is related to Issue #94 but does not close it).

The second commit shortens tile_overlap_width to overlap_width and tile_overlap_height to overlap_height.
Additionally it records these values in the slide dictionary so that the user need not try to derive the overlap values from the resulting tile positions.  This closes Issue #93.

The recent force push of the second commit additionally records these overlap values in the study and slide dictionaries so that the user need not try to derive the overlap values from the resulting tile positions.  Slides can have and record different overlap values from each other.  The study-wide value is either set directly by the user, or is taken from the first use of TilesByGridAndMask in the study.  Creation of a TilesByGridAndMaskobject without specifying overlaps now defaults to the study-wide values, if available, else zero.  Previously the default was simply zero.  This closes Issue #93.

Force push just now, rebases to current master.

The third commit closes Issue #104.


Mapping implementation

I'm trying to implement Mapping on Android and I am not sure I am doing it right.
Is it okay to create another Platform in DatabaseSource#onCreateMapping? It looks like it is already created in the constructor, but is marked private. Should this field be protected instead?
Also, could a Mapping be provided at configuration time using the Builder pattern? It seems unnecessary to create a DatabaseSource just to set a Mapping.
public class ExampleDatabaseSource extends DatabaseSource {
    // Constructors...

    @Override
    protected Mapping onCreateMapping() {
        return new ExampleMapping(new SQLite());
    }
}

public class ExampleMapping extends DefaultMapping {
    public ExampleMapping(Platform platform) {
        super(platform);
        // Joda Time converters
        addConverter(new LocalDateConverter());
        addConverter(new DateTimeConverter());
    }
}

Yes it's ok to create another platform instance. Yup I think this could be improved some will look into it thanks!


New install method - shell_binary, upstart script tweak

Changes:

New install method - shell_binary: Faced problems with ark dependency with the binary module on ubuntu 14.04. The new module aims to provide same functionality using basic curl and shell.
Upstart script tweak: Console template does not work as it tries to search for console pages in root (/). The default should be the installation directory of prometheus. This is where the consoles/ directory is present.

Thanks!

@elijah Sorry, forgot to update earlier. Rebase done.
Thanks.

@elijah Is this PR good to merge?
I use the suggested changes myself for maintaining Prometheus clysters since the time I raised the first pull request. It works as expected.

Merged.


Add docker build github action

I would like to work on this please assign it to me @xamfy

Assigned it to you, @ChellappanRajan maybe able to guide you as well, if you have specific questions.
More context on this:
We'll have a new file called Dockerfile in the root directory, you can refer Dockerfile.example, we need exactly same thing but with the ability to replace the container enviroment variables during the build of the image from github secrets(this needs to be figured out).

@SaumyaBhushan Do let me know if you're facing any issue. You can raise a PR with few changes or a new Dockerfile and we can discuss further over there if there's any doubt or question.

Sure @xamfy


Changed format_string doc string to have accurate parameters and para…

…meter descriptions

What changes were proposed in this pull request?
The parameters doc string of the function format_string was changed from col, d to format, cols which is what the actual function declaration states

Why are the changes needed?
The parameters stated by the documentation was inaccurate

Does this PR introduce any user-facing change?
No

How was this patch tested?
N/A: documentation only

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Thank you for your first contribution, @darrentirto !

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14437/
Test PASSed.

Test build #109374 has started for PR 25506 at commit 3851cef.

Test build #109374 has finished for PR 25506 at commit 3851cef.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109374/
Test PASSed.

I attached the screenshot to the PR description, @darrentirto . In general, we recommend to attach the screenshot of the generated document or Spark web UI.

@dongjoon-hyun thanks for the help. I'll keep the picture attachment and the PR tags in mind in the future.

Thank you, @darrentirto . Yep. I followed the written guideline. Please take a look at https://spark.apache.org/contributing.html when you have some time. :)


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 888 ms

Resolved: Bofrost.be performance has improved in 3582906.


User-reported unhelpful error message

Program: https://code.world/haskell#PIwzB1kXnxKyDEKfR2TWFNQ
Comment:
i was typeing the heading corectly but it underlined it with red
Message:
program.hs:1:1-15: error:
    Parse error: module header, import declaration
    or top-level declaration expected.

Hard to see how to do better here.  Also, this is haskell mode.  I should really just disable the report link in haskell mode


Break up stage 1 work into issues

The stage 1 tasks should be reduced into coding tasks:

[x] data catalog
[x] nodes
[x] pipelines

Work broken up into features -- though these will need subtasks defined.


Added incompatibility flag to prepare for flip in future Bazel versions

A quick query seemed to prove the repo is compliant with incompatible_disallow_empty_glob but it'd be good to add regardless to avoid future regressions.

/gcbrun

/gcbrun

Looks like there's some RPM related error https://app.buildbuddy.io/invocation/554468ea-d9a8-11ee-8ecb-0242c0a80a02

Looks like there's some RPM related error https://app.buildbuddy.io/invocation/554468ea-d9a8-11ee-8ecb-0242c0a80a02

Yeah, I was definitely wrong when I mentioned the repo looked compliant. But the way the repo is tested is also slightly flawed. It shouldn't be checking @rules_pkg//... and things like that. Instead there should be sufficient uses of each external repo within rules_hdl to cover what needs to be tested. I think this might be blocked by https://github.com/bazelbuild/rules_pkg/pull/825

@QuantamHD this change is finally ready for review!


Syntax error

This is in the error log for PHP 5.5:
PHP Parse error: syntax error, unexpected '.', expecting '&' or variable (T_VARIABLE) in /var/www/.../mysql.php on line 553
I've also seen other errors like "memory exceeded" on my local environment (PHP 7 XAMPP). Looks like I need to replace those mysql_ to mysqli_ calls myself. Need it working tomorrow.

The lib requires php 5.6
https://github.com/dshafik/php7-mysql-shim/blob/master/composer.json#L13

Oh. Maybe this should be noted in the readme. 5.5 to 7.0 is the latest Ubuntu LTS transition and might not be that uncommon. I've fixed my code manually now.

It is noted in the readme

Are you referring to this?

You may safely include the file in a PHP 5.6 project, it will do nothing.

I interpret this as "You may safely include the file in a PHP ≤ 5.6 (i. e. < 7.0) project", including 5.5. The readme does not say anywhere that 5.6 is required.

Oh, you are right. Its not explicitly mentioned.
Since mysql_* was dropped with php7 the only versions this shim actually has a effect is 7.0+. The compat with 5.6 is a bonus so you dont need version checks in your code.

Pull request #22 addresses this issue.


User/password support in url

Is there way to pass username and password for SSH connection using url address itself? We are using this in our internal administration. So when our administrator is logged in and opens special webpage with webssh2 in iframe, he will be logged automatically.

we need it too!

Interested in taking a PR for this, but it's not something on my roadmap at this time.

+1 for this feature

+1 for this one.

+1 for this feature

+1 for this feature

+1 for this feature

where can we donate?

+1 for this feature

so, i'm on the fence on this one. exposing the password on the URL isn't great... especially for an administrative system... going to close for now, but willing to discuss a more secure way to implement this.


Maven 3.5.2 - libjansi.so not a dynamic executable

linuxbrew@marvin:~ $ brew upgrade maven -v
==> Upgrading 1 outdated package, with result:
maven 3.5.2
==> Upgrading maven
rm /home/linuxbrew/.linuxbrew/bin/mvn
rm /home/linuxbrew/.linuxbrew/bin/mvn.cmd
rm /home/linuxbrew/.linuxbrew/bin/mvnDebug
rm /home/linuxbrew/.linuxbrew/bin/mvnDebug.cmd
rm /home/linuxbrew/.linuxbrew/bin/mvnyjp
==> Downloading https://www.apache.org/dyn/closer.cgi?path=maven/maven-3/3.5.2/binaries/apache-maven-3.5.2-bin.tar.gz
Already downloaded: /home/linuxbrew/.cache/Homebrew/maven-3.5.2.tar.gz
==> Verifying maven-3.5.2.tar.gz checksum
tar xzf /home/linuxbrew/.cache/Homebrew/maven-3.5.2.tar.gz
ln -s ../Cellar/maven/3.5.0/bin/mvn mvn
ln -s ../Cellar/maven/3.5.0/bin/mvn.cmd mvn.cmd
ln -s ../Cellar/maven/3.5.0/bin/mvnDebug mvnDebug
ln -s ../Cellar/maven/3.5.0/bin/mvnDebug.cmd mvnDebug.cmd
ln -s ../Cellar/maven/3.5.0/bin/mvnyjp mvnyjp
Error: Failure while executing: /home/linuxbrew/.linuxbrew/opt/glibc/bin/ldd /home/linuxbrew/.linuxbrew/Cellar/maven/3.5.2/libexec/lib/jansi-native/freebsd64/libjansi.so
        not a dynamic executable

Thanks for the bug report, Torsten.
This bug will be resolved by PR https://github.com/Linuxbrew/brew/pull/517

I've merged PR #517. Could you please try…
brew update
brew --version
brew upgrade maven

It installs now.

I still have the problem described by this issue. I have the latest linuxbrew installed (1.3.6).
Is this fix released?

Looks like it isn't and brew stashes changes of it's own repo so I can't apply the patch manually.

No, it's not released yet. If you apply the patch manually and commit it, it should survive through a brew update.

It's here: https://github.com/Linuxbrew/brew/commit/44564b47d63b492af4b5bd0452f6716cfac778e9
It will be available when we tag 1.3.7

Try…
git -C $(brew --repo) cherry-pick 44564b4

Sorry to comment on a closed issue, I'm hitting this in Docker builds that extend FROM linuxbrew/linuxbrew, e.g. https://hub.docker.com/r/heuermh/dsh-bio-dev/builds/bqs4mma27hkut5dybofabea/.  Is there something I need to do to pick up the fix?

The linked build is from over a month ago, before PR https://github.com/Linuxbrew/brew/pull/517 was merged, which resolved this issue. Your most recent build of today is successful. https://hub.docker.com/r/heuermh/dsh-bio-dev/builds/

Your most recent build of today is successful.

Well, look at that.  Nice when things work themselves out.  :)
Sorry for the noise.

No worries. Glad it worked for you. =)


[WIP] Simplify JDBC code sample

Fixes #4621, #3578, #4399.
Summary of changes:

Update 'Build a Java App with CockroachDB' to use the Java DAO pattern

Add a 'Recommended Practices' section that includes information about
IMPORT, recommended batch size, and the JDBC INSERT rewriting flag

TODO:

Update "secure" code sample to use certs.  Not worth doing yet until
this code is reviewed by someone with Java expertise and updated to
match good style, etc.  Once that is done, the secure code sample in
this PR will be updated with the necessary cert munging.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/70ecb331b4798e47a5237766fbf434c9d34447d2/
Edited pages:

v19.1/build-a-java-app-with-cockroachdb.md
v19.1/savepoint.md
v19.1/transactions.md

@jordanlewis can you please help me find someone on our engineering team who has Java expertise and would be willing to review this PR for:

Idiomatic Java/JDBC usage
Errors/mistakes in how it interacts with the DB
Better ways to describe what this doc is trying to describe, if possible
General smurfiness (💙 )

I am expecting that the code and possibly the doc surrounding it will need updates according to the criteria above, hence the WIPness.
My initial plan was to ask around but I spoke to @awoods187 and he recommended coming directly to you as a team lead / engineering manager type person.
Happy to discuss offline if that's easier/quicker.

Hey @rmloveland, will follow up with you offline. Thanks for bringing this up!

Hey @rafiss, could you please take a look at this code sample while your Java knowledge is still fresh?
The context is that we're trying to improve our docs for Java users in particular. We'd like to show a canonical way of interacting with the database. Your review would be valuable, as you have recent Java experience.

Sure thing! Some of the suggestions I had might not really be worth doing. Some of the idioms or conventions I suggested probably are not important for this simple example.

Thanks for the review @rafiss!  Working on making these changes now.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e3fcc9ede7749efa963d0f6c8258da637fb08948/
Edited pages:

v19.1/build-a-java-app-with-cockroachdb.md
v19.1/savepoint.md
v19.1/transactions.md

Whoa, looks like Reviewable really messed up the Github comment output on this one.  Everything looks OK to me on the actual review page tho

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b5dd9cef4cbae31ba18aa69fb8ef4fddb081dd31/
Edited pages:

v19.1/build-a-java-app-with-cockroachdb.md
v19.1/savepoint.md
v19.1/transactions.md

Thank you @rafiss !!!!!!1!
Apologies in advance for a bit of forthcoming noise, I have to make some uninteresting updates re: secure connections / certs now that we have nailed down the main code samples -- then it goes in for docs team internal review.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a739028437823a2fbaeaa289ef0a531b90811a19/
Edited pages:

v19.1/build-a-java-app-with-cockroachdb.md
v19.1/savepoint.md
v19.1/transactions.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/13c50fa650ef2f963333d6fcfded6bfec8d7fc65/
Edited pages:

v19.1/build-a-java-app-with-cockroachdb.md
v19.1/savepoint.md
v19.1/transactions.md
v19.2/build-a-java-app-with-cockroachdb.md
v19.2/savepoint.md
v19.2/transactions.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e69a0d06937c0d8623ee4da0d07e2930c76fefb6/
Edited pages:

v19.1/build-a-java-app-with-cockroachdb.md
v19.1/savepoint.md
v19.1/transactions.md
v19.2/build-a-java-app-with-cockroachdb.md
v19.2/savepoint.md
v19.2/transactions.md


Site updated to include a mailing list, dates updated and some new layouts

new plain page layout
new form layout
404 page
dates in 2018

oops


ufunc 'create_collection' not supported for the input types

This is a recent issue identified using nightly numpy binary wheels with the main branch of shapely.
The error looks like this:
=========================== test session starts ============================
platform linux -- Python 3.10.12, pytest-8.2.2, pluggy-1.5.0
benchmark: 4.0.0 (defaults: timer=time.perf_counter disable_gc=False min_rounds=5 min_time=0.000005 max_time=1.0 calibration_precision=10 warmup=False warmup_iterations=100000)
rootdir: /home/mtoews/src/shapely/shapely
configfile: setup.cfg
plugins: syrupy-4.6.1, anyio-4.4.0, dotenv-0.5.2, flaky-3.8.1, hypothesis-6.108.5, cov-5.0.0, xdist-3.6.1, benchmark-4.0.0
collected 9 items / 1 error

================================== ERRORS ==================================
__________ ERROR collecting shapely/tests/geometry/test_coords.py __________
shapely/tests/geometry/test_coords.py:5: in <module>
    from shapely.tests.common import line_string, line_string_z, point, point_z
shapely/tests/common.py:26: in <module>
    multi_point = shapely.MultiPoint([(0, 0), (1, 2)])
shapely/geometry/multipoint.py:62: in __new__
    return shapely.multipoints(subs)
shapely/decorators.py:77: in wrapped
    return func(*args, **kwargs)
shapely/creation.py:433: in multipoints
    return lib.create_collection(geometries, typ, out=out, **kwargs)
E   TypeError: ufunc 'create_collection' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''
========================= short test summary info ==========================
ERROR shapely/tests/geometry/test_coords.py - TypeError: ufunc 'create_collection' not supported for the input types,...
!!!!!!!!!!!!!!!!!!!!!!!! stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!
============================= 1 error in 0.41s =============================

I've bisected the issue to https://github.com/numpy/numpy/commit/f3b669d391cfe660b064af052973b9bff9372dd8 from https://github.com/numpy/numpy/pull/26905 (ping @seberg) which is related to NEP 50.

Here is a short reproducer:
>>> import shapely
>>> from shapely import GeometryType
>>> pts = shapely.from_wkt(["POINT(1 2)", "POINT(3 4)"])
>>> shapely.lib.create_collection(pts, GeometryType.MULTIPOINT)

where the last line would normally return <MULTIPOINT ((1 2), (3 4))> before the referenced numpy commit or the error:

TypeError: ufunc 'create_collection' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

Makes sense, I guess intenums may be int subclasses, is it a problem to cast to int() to make it i compatible (rather than int64/intp?)

Thanks @seberg for taking a look; a few follow-up items/questions:

Firstly, yes casting int(GeometryType.MULTIPOINT) before sending to the ufunc is a possible change we can make.
But why does (e.g.) np.sqrt(GeometryType.MULTIPOINT) cast the enum to float64 without error? Is there a change that we could  make on the C-side?
For what it's worth, I see np.result_type(GeometryType.MULTIPOINT) return dtype('int64').
Should there have been a warning that this API change was about to happen? I didn't see any before the breaking commit.

You can support the dfault int in C mainly.  It's a pretty niche change and a warning is hard, because it is rare that it will trigger any change.  Since most ufuncs will support the default integer explicitly.

@mwtoews thanks for the investigation and @seberg for the explanation.
I was quickly checking in which places we should need such a cast, and opened a PR for that -> https://github.com/shapely/shapely/pull/2099
We do have a set of other enums for modelling parameter options, and in all those cases it seems we were already casting the enum to an int. For example:
https://github.com/shapely/shapely/blob/9a8bf73ecf2ef51d684b5866ab5718220b402640/shapely/_geometry.py#L842-L852
or the various buffer arguments:
https://github.com/shapely/shapely/blob/9a8bf73ecf2ef51d684b5866ab5718220b402640/shapely/constructive.py#L181-L190
(although I see we consistently use np.intc(..) for the casting while in the PR I just opened I used a python integer int(..), will change that for consistency)

Sounds right.  Is this a big enough problem to worry about delaying things in NumPy (also since 2.1 is probably be going to be relatively soon for Python 3.13 support).

I don't think so, we can always include this small fix in another 2.0.x release

Same problem with numpy 2.1.0 (released yesterday)

@AlessandroMondin - as a short-term solution, reverting back to numpy 2.0.1 has at least patched the issue for me. Hope that helps!

Shapely 2.0.6 is released that includes the fix (it's on pypi, conda-forge can still take a few hours)

I jsut found this issue with:
python 3.11
shapely 2.0.5
numpy 2.1.0

all conda-forge builds.
updating to shapely 2.0.6 fixed it -- nice work!
But maybe the 2.0.5 builds on conda-forge should be marked as broken?
https://github.com/conda-forge/admin-requests?tab=readme-ov-file#mark-packages-as-broken-on-conda-forge

They're not broken, just not compatible with the latest shapely. Following this logic, we would have to mark as broken all releases prior to 2.0.6.

Got it -- but the dependencies are wrong, yes?
That can be updated with:
https://prefix.dev/blog/repodata_patching
https://github.com/conda-forge/conda-forge-repodata-patches-feedstock
I think shapely 2.0.5 should have:
`numpy >=1.19,<2.1`

rather than:
`numpy >=1.19,<3`

I could help with this if need be, though I don't have permissions on the feedstock, so maybe not?

Yeah, on conda-forge the numpy 2.0 support was added later, and for packages built before that, conda-forge automatically ads a numpy<2 constraint (even though shapely itself didn't have that).
So in practice, it seems that indeed only shapely 2.0.5 could be patched with a numpy<2.1 (shapely 2.0.4 still has a numpy<2 upper pin). I suppose opening a PR for that in https://github.com/conda-forge/conda-forge-repodata-patches-feedstock is welcome (I don't have any permissions their either, it will need to be reviewed by a core conda-forge team member anyway)


bug: slot fallback broken in components built with 2.6.14 running in 2.6.12

Version
2.6.14
Reproduction link
Slot fallback broken running in 2.6.12:
https://jsfiddle.net/hirokiosame/dotsp9ak/
Slot fallback working in 2.6.14:
https://jsfiddle.net/hirokiosame/eb16s0hx/
Steps to reproduce

Build the following component with vue-template-compiler v2.6.14:

<template>
	<div>
		test component
		<slot>
			- slot fallback works!
		</slot>
	</div>
</template>

Run component in a Vue.js v2.6.12 app:

<template>
    <test-component />
</template>

<script>
import TestComponent from './TestComponent.js';

export default {
  components: {
  	TestComponent,
  },
}
</script>

What is expected?
For a component built with vue-template-compiler v2.6.14 to be backwards compatible with a Vue.js v2.6.12 runtime.
It's not only no longer working, it actually displays "undefined" in place of the slot fallback.
Since there is no new feature being leveraged, I think it's expected for behavior to be preserved.
What is actually happening?
Slot fallback not working in component built with vue-template-compiler v2.6.14, but running in a Vue.js 2.6.12 app

I see. If it's not possible because it's a side-effect to fixing an underlying bug then that's understandable too.
Thanks for explaining and referencing the PR.


Strong naming

Could you please sign the assembly you publish on NuGet? Else projects with strong naming enabled aren't able to use your library.

@Toemsel hi! Thanks for reaching out. I'll work on this in the following days, so stay tuned, don't loose hope.

Else I have to clone and ditch the NuGet ;)

@Toemsel I released a new version 4.5.0. Please check. Please toss a star for this repo in order to support my work. Thank you!


python311Packages.teslajsonpy: 3.9.5 -> 3.9.6

Diff: https://github.com/zabuldon/teslajsonpy/compare/refs/tags/v3.9.5...v3.9.6
Changelog: https://github.com/zabuldon/teslajsonpy/releases/tag/v3.9.6
Description of changes

Things done

Built on platform(s)

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 260624 run on x86_64-linux 1

4 packages built:

python310Packages.teslajsonpy
python310Packages.teslajsonpy.dist
python311Packages.teslajsonpy
python311Packages.teslajsonpy.dist


BUG: Duplicate Content and Logo in Free eBook Section Header

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
In the Free eBook section of the website, the header section contains duplicate content and a repeated logo, resulting in a cluttered and visually unappealing display.
Screenshot:

@anu Please add the GSSOC and appropriate level label to this issue and assign this to me. I would love to contribute.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!

Thankyou @huamanraj  for assigning the issue to me. I will work according to the guidelines.


Inter-Stack-Communication not working after 1.4.0 update

Rancher Versions:
Server: v1.4.0
healthcheck: 0.2.3
ipsec: 0.0.4
network-services: 0.0.10
scheduler: 0.4.0
network-policy-manager: 0.0.1 (installed manually after the update)
Docker Version:
Docker version 1.12.3, build 6b644ec
OS and where are the hosts located? (cloud, bare metal, etc):
AWS
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node rancher, external DB
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Spin up 1.3.3, upgrade to 1.4.0
Results:
Expected: Inter-Stack-Communication works as before
Actual: Inter-Stack-Communication does not work at all
Problem:
My guess (!) is that during the update, defaultPolicyAction is not touched for an existing environment. When the rancher-agent is then upgraded, it doesn't know about it and restricts communication.
Workaround:

Set defaultPolicyAction to allow in rancher-server:8080/v2-beta/projects/1a5/networks (should be the first env you've created; if you have multiple set it for every cattle env at least).
restart rancher-agent on every host using docker restart rancher-agent

Fix:
Eiter set the default to allow or make sure that no value actually means allow in code.

@off , i am not able to reproduce the issue where inter-Stack-Communication is broken after upgrading from 1.3.3 to 1.4.0. I am testing with hosts from GCE with OS version - Ubuntu 16.04.1 LTS (4.4.0) and docker version  - 1.12.6.
As you have mentioned in the bug , i do see defaultPolicyAction  and policy set to null even after the following steps where done:

Upgrade rancher-server to 1.4.0
Do not ugrade any of the infra stacks.
inter-Stack-Communication works.
Upgrade all infra stacks (to 1.4)
inter-Stack-Communication works.
Install network-policy-manager
inter-Stack-Communication works.

Even though I see  defaultPolicyAction  and policy set to null  , intercommunication between stacks continues to work in my case as expected.
Can you please check , if in your case ipset is empty ? You can get to it from the network-policy-manager container  - ipset -L

@0ff I see your point that after upgrade defaultPolicyAction: null. But that shouldn't cause any problems in the environment.
Looking at the code: https://github.com/rancher/network-policy-manager/blob/v0.1.0/policy/watcher.go#L456, in a 1.3.3 setup, len(np.Rules) = 0 causing to take the else block and since I am checking for an explicit deny for defaultPolicyAction there shouldn't be any rules programmed at all.
I am curious as to how you could have run into the problem of communication not working.
Can you please share your OS details?
Also if you are planning on trying to reproduce the issue, before doing any workaround, can you please check the output of the following commands on all the hosts?
iptables-save
ipset -L

Thanks for both of your responses! I will try to reproduce the setup tomorrow and get in touch with the results.
Just to clarify why I am suspecting something with the defaultPolicyAction: I tried pinging (and curling) 3 different kinds of hosts in my ipsec network:

hosts in the same stack on the same host -> worked
hosts in a different stack on the same host -> didn't work
hosts in a different stack on a different host -> didn't work

Unfortunately, I didn't test same stack/different host so I can't say for sure that it was only related to the 1.4.0 changes. Also, at some point, the rancher-agent on one (and only one) of the hosts (out of 3 in this setup) went kind of rouge, which means that for some reason (most certainly not related to this) even after restarting it wouldn't assign IP Addresses to new containers which lead to all containers breaking with Timeout getting IP address.
By this time (though I can't be sure if it was like this before or not), the healthcheck container on this host (and again, only this host) went into failure saying that some services couldn't be reached. The other healthchecks were always green (which, I realise, is quite in contrast to 1. above).
So I really hope that we can replicate the error so that there may be a fix for other users. I'll update you tomorrow!
If anything else comes to mind which might explain the above, please let me know!

@0ff I think you were hitting a different issue altogether. May be: https://github.com/rancher/rancher/issues/6858
When you try to reproduce, please make sure the environment upgrade went fine and all of your stacks are reachable before even you install network-policy-manager.
Let us know how it goes.

@0ff Did you get a chance to reproduce the issue?

I run into the same issue. We've got a HA Rancher setup with two hosts and a stack consisting of two services which got deployed one on each host. If I execute a shell on one host and try to ping the service on the other host it will timeout. Although the DNS resolves correctly.
There's no additional setup or configuration in rancher and the two services are not configured further as well.

I was able to resolve my issue by recreating the rancher-agent container using the correct IPs. Our hosts are behind a firewall with NAT, so I was confused about which IP I should use.
docker run -e CATTLE_AGENT_IP="<INTERNAL_IP>"  -e CATTLE_HOST_LABELS='io.rancher.host.external_dns_ip=<EXTERNAL_IP>'  -d --privileged -v /var/run/docker.sock:/var/run/docker.sock -v /var/lib/rancher:/var/lib/rancher rancher/agent:v1.2.0 http://rancher-rulez:8080/v1/scripts/idAnDStUfF

@0ff Could this be the problem for you as well? If not I was just facing a similar issue.

Hi @einfallstoll - our hosts are in fact also running behind a NAT but unfortunately I wasn't able to reproduce the error when replicating the setup and upgrading in the same way.
I guess @leodotcloud was correct in that the whole problem was caused by the Timeout getting IP address error instead of the new network-policy-manager.
In the end, restarting rancher-agent fixed it for us too (our hosts did know about their correct IPs) and so I think the issue can be closed.

Ok, great. So this issue can be closed now :)


Custom Layout Renderers other then GDC,MDC any other?

Can you please advise except the below contexts for  Custom Layout Renderers?
1)GlobalDiagnosticsContext.Set("myDataBaseColumName","someValue");

MappedDiagnosticsContext.Set("myDataBaseColumName","someValue");

Do you mean how to build custom layout renderers ? Or choosing between MDC and GDC?

Actually i am using the below contexts for layout render pass the value to the datbase. but my boss is not approved the below code. bacause it's Globalcontext . Please let me know any other solution for this.
GlobalDiagnosticsContext.Set("sessionid", '154265887');

I think you're looking for the event properties: https://github.com/NLog/NLog/wiki/EventProperties-Layout-Renderer

Has your question being answered?

Closing this due to inactivity. Please let us know if this still an issue and please provide the requested info.


Erro - Captcha

Hi, Gilson.
I hope you're well.
Great package for data extraction. It has helped me a lot in my doctorate.
It has been a few days since the captchar has been found. Did you notice this error? Since the changes you made, this error has appeared to me.

Hello, Great to know the package is helping you.
I downgraded dependencies to avoid conflicts in Google Colab and tested the package on Google Colab and on my computer running Ubuntu 20.04.3 LTS through WSL 2.
You can try again on Colab if you are using it, otherwise I recommend you to uninstall the package from your personal computer and reinstall before using it.
If the error persists, let me know and provide as much detail as you can, so I can try to fix it as soon as possible.
Tests on Colab:

Hi.
I tested it again and it worked.
Thanks.


Use mutiple EditorForm in ValidateForm

As Title,  OnInvalidate not active when I use,
Is this the correct way to use it ?

@a0926019412 hi, It is recommended to use one EditorForm in one ValidateForm


Move to use algoliasearch lite version?

Just take a look at different algolia v3.21.1 releases:
algoliasearch.jquery.min.js 59K
algoliasearchLite.min.js 37K
algoliasearch.min.js 58K

I wonder if we can use algoliasearchLite instead, 56% size diff, could make the website lighter, should compare the different parts and trace the related codes and then we'll know if we can move to the lite version.

Hi @redox ! It's been a while since our last conversation, would you be interested in take a look at this one? (Since I think you're the one most familiar with it 😄 ) Thanks!

Hi @redox ! It's been a while since our last conversation, would you be interested in take a look at this one? (Since I think you're the one most familiar with it 😄 ) Thanks!

Oh wow, sure 👍

Thanks @redox


What is private.js

From what I can tell, private.js runs AES encrypted JavaScript code, in a password protected manner. It also includes the source code for CryptoJS to do this operation. However, what is the purpose of this and why is it included in this repo?
I assume it has something to do with GimKit, but I'm still curious as to why it's here. What is it used for here?

In order to prevent Gimkit from constantly updating and implementing more anti-cheat features I decided to make the updated Gimkit script password protected to prevent Gimkit developers from easily viewing how the cheat works. I am currently only giving away access to the password through request.
In a couple days I will be releasing a * hopefully groundbreaking* extension which should solve all current and future issues.

Okay thank you for clarifying

Can I get the password?

I'm new to GitHub so I don't understand much. Is there a way you can private message the code to me?

I don't think I know of any way to PM a user on github. I'll be releasing an extension soon for Gimkit that will be publically available.

I'd be interested to see how powerful it is. Good luck on it


Spell checking tool

Introduce a tool to catch spelling and common typing problems, such as "the the", "decomission", etc.
In the meantime, we can use Checkstyle's Regexp pattern, but that's pretty inefficient(?) and may slow down the build.
Inspired by #7154.

I'm not sure if a generic spell check is worth it -- there might be way too many false positives and it would just get annoying -- but something that looks only for specific misspellings may be worth it. (Although, of limited usefulness, since it wouldn't be checking very much.)
I'd suggest doing something that can be applied through Maven (rather than IntelliJ inspections) since it's more accessible. Everyone working on Druid has Maven, not everyone has IntelliJ, and it's a better experience when people can verify their own stuff before submitting it up as a PR. The IntelliJ inspection errors won't be obvious until TC runs, which introduces delay in the PR cycle.

Btw, did you just mean docs? I was figuring this suggestion was about adding a spell checker specifically for docs, but just realized the top comment didn't mention that.

I'm not sure if a generic spell check is worth it -- there might be way too many false positives and it would just get annoying -- but something that looks only for specific misspellings may be worth it. (Although, of limited usefulness, since it wouldn't be checking very much.)

I'd suggest doing something that can be applied through Maven (rather than IntelliJ inspections) since it's more accessible. Everyone working on Druid has Maven, not everyone has IntelliJ, and it's a better experience when people can verify their own stuff before submitting it up as a PR. The IntelliJ inspection errors won't be obvious until TC runs, which introduces delay in the PR cycle.

Yes, that's exactly what I meant - a Maven plugin which is feeded with a list of stop words (BTW we can include profanity there, too). My only concern with Checkstyle's Regexp rules is that they are regular expressions and having a lot of them to check in the whole repo may slow down the build. The stop word plugin should do simple exact match scan.


fix incorrect snapshot size

Codecov Report

Merging #135 (ca179f7) into main (e14a68e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #135   +/-   ##
=======================================
  Coverage   11.18%   11.18%
=======================================
  Files          17       17
  Lines        1681     1681
=======================================
  Hits          188      188
  Misses       1474     1474
  Partials       19       19

Flag
Coverage Δ

unittests
11.18% <ø> (ø)

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

Continue to review full report at Codecov.

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


Build failure on Windows 10 when adding DefaultPlugins

Bevy version 0.4
Operating system & version
Windows 10 Professional 64-bit
What you did
Attempted running the simple hello world app with DefaultPlugins added. Code:
use bevy::prelude::*;

fn main() {
    App::build()
        .add_plugins(DefaultPlugins)
        .run();
}

What actually happened
Project failed to compile, siting error: linking with link.exe failed: exit code: 1120
Full console output: link_error.txt
Additional information
@DJMcNab Suspected an issue with glsl-to-spirv

Probable duplicate of https://github.com/bevyengine/bevy/issues/788

Looks like I was able to fix my issue. The problem was that I had VS 2017 with game build tools, but VS 2019 without game build tools. The system (apparently) defaults to the newest version, but will supplement with an older version if available. However, these two versions of VS are not compatible, hence the linking error.


Blur event handler causes the virtual dom to lose track of nodes

Here is a somewhat minimal example:
https://gist.github.com/larribas/b47e3f63e8134aa855595f11f429ef56
This is the initial state

This is the state after writing a new item

And then if we repeat that action many times goes on

This is what Chrome's console says each time it happens:
Uncaught DOMException: Failed to execute 'replaceChild' on 'Node': The node to be removed is no longer a child of this node. Perhaps it was moved in a 'blur' event handler?
    at _VirtualDom_applyPatchRedraw (demo.js:3355:14)
    at _VirtualDom_applyPatch (demo.js:3276:11)
    at _VirtualDom_applyPatchesHelp (demo.js:3262:17)
    at _VirtualDom_applyPatches (demo.js:3253:9)
    at demo.js:3651:14
    at demo.js:3686:8
    at sendToApp (demo.js:1494:3)
    at HTMLTextAreaElement.callback (demo.js:2552:3)

I discovered the issue while upgrading this library: https://github.com/larribas/elm-multi-input
Which worked fine for elm 0.18

Here is a shorter SSCCE:
https://gist.github.com/hecrj/ce06033689dca3eb603acdbbdaf2eb51

Switching onBlur to be on requestAnimationFrame fixes this. It seems that onblur does some strange things such that asynchronous seems to be a safer default.


fix imported shapes validity

Last update introduced issues with imported shapes validity that went unnoticed because the test did not check for that :(
The issue seemed to mostly come from the detection and enforcement of closed wires used to build faces.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.24%. Comparing base (fdc1b4c) to head (63165b7).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #30   +/-   ##
=======================================
  Coverage   99.23%   99.24%
=======================================
  Files           8        8
  Lines        1177     1192   +15
=======================================
+ Hits         1168     1183   +15
  Misses          9        9

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


Pull

WHY
BEFORE - What was wrong? What was happening before this PR?
??
AFTER - What is happening after this PR?
??
HOW
How did you achieve that, in technical terms?
??
Is it a breaking change?
??
How can we test the before & after?
??
If the PR has changes in multiple repos please provide the command to checkout all branches, eg.:
git checkout "dev-branch-name" &&
cd vendor/backpack/crud && git checkout crud-branch-name &&
cd ../pro && git checkout pro-branch-name &&
cd ../../..

Thanks @akhtar-techrayslabs
It seems you got it working, keep those PR's coming 🙏
Cheers


Clear up handling of UIDs for 32-bit when logging

Make sure we don't overflow any 32-bit numbers when looking up a user's login UID, their name, or making an audit log entry.

Broke this in two and update syslog and auditing patch with it.


SpinnerDialog not working in windows ionic app

I have created an ionic app for android, ios & windows. Spinner Dialog is working for android & ios but it is not working for windows app. I have used exactly same code for all the 3 apps. Can anyone please help.

this plugin do not support windows 10 platform

@AgNm you can use this fork https://github.com/greybax/cordova-plugin-native-spinner which supports win10 instead of it.

i have configure in config file for phonegap build, but i don't understand that, where i do keep show and hide code.. anybody tell me...

@greybax does this plugin supports windows desktop? I used SpinnerDialog.show(); but it is not working. Please help.

@AgNm yes. https://github.com/greybax/cordova-plugin-native-spinner supports windows 10. You can see signature for show function here

Could this be made compatible with windows 10 / UWP? that other plugin does not work with Ionic native's V3.


didScrollToDateSegmentWith working when click to cell in iphoneX

**(Required) Version Number:**8.0.3
Description
In Iphone X, when I click to cell in collectionView, function didScrollToDateSegmentWith working
cause some actions to go wrong
Steps To Reproduce
Step 1  : Setup normal Calendar
Step 2 :  Click to cell in collectionView on device iphone X
Step 3 : Debug in function didScrollToDateSegmentWith
Expected Behavior
didScrollToDateSegmentWith not working when click to cell
Additional Context
didScroll do the task of calling API , when click duplicate calling API

What code are you executing when the user clicks on the cell?

func calendar(_ calendar: JTACMonthView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {         print("work in here")     }

when i click to cell in collection View, function this working in only device iPhone X
@patchthecode  can you review it?


Concurrency: include missing header

Include the new header for placement new.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test


[rancher-compose] Add support for env_file similar to docker-compose

One of the biggest missing features right now when it comes down to managing tons of similar containers sharing 99% of their configurations. https://docs.docker.com/compose/compose-file/#env-file. Shouldn't take much effort to implement.
P.S. I wonder if it makes sense to talk to Docker team and extract parsing/combining/etc. logic for docker-compose.yml files into standalone library and make its output to be some kind of well-defined JSON structure consumable by whoever needs it. In this case there will be no need to implement the same features, but re-use what is provided.

Hi @xaka, the rancher-compose command does support environemnt files -e or --env-file should work. The command is built off of docker/libcompose which was contributed by us back to Docker. It is still a work in progress, but it is a Golang implementation of compose (which is python based). There is a PR on libcompose to add strict checking.
Rancher compose does make some subtle changes in behavior that we feel help with managing production workloads. We also extend it to add support for some concepts that are not in the upstream docker compose, like load balancers, health checks etc.

Wow! That's nice. The reason I opened issue is because I couldn't find it in documentation nor on --help. Is it out of date or I'm too drunk?

rancher-compose --help shows it for me.. I think it was added in rancher-compose v0.4.x maybe you have a really old version? :)
I also couldn't find it in the docs, so we should fix that for sure.  Carry on with the :beer: :smile:

Alright alright, it's my fault (well, not actually mine, but :beers:), but I just double checked http://docs.rancher.com/rancher/rancher-compose/commands/ and couldn't find anything useful :stuck_out_tongue: . I'm going to change the title...

@xaka Thanks for pointing out the outdated docs. I've updated them now.
Rancher-compose is a bit trickier for me to catch updates cause I"m not following the docker/libcompose repo. Maybe I should be ;)


<EMAIL_ADDRESS>Download via aria2 failed

Download failed!

Output of the github action says "INFO: Download failed" it load the file from cache.

shovel.
I've aria2c configured too. But i've tried with it disable too, it not working neither. Also using the URL within a Browser give 404, so I think there is really something changed at techpowerup.

shovel but i don't think it's revelant here.

It is totally relevant. Use scoop. Shovel cannot download stuff from techpowerup.

True, thanks for spotting it!


[Android] borderStyle does not work with specific borderWidth (borderBottomWidth, etc).

borderStyle does not work with specific borderWidth.
borderBottomWidth: 1,
borderBottomColor: '#979797',
borderStyle: 'dotted',

shows no dotted border.

I assume this is a duplicate of #8236.

If set
{   borderWidth: 1,   borderTopWidth: 0,   borderLeftWidth: 0,   borderRightWidth: 0,   borderBottomColor: '#979797',   borderStyle: 'dotted' }
It's work

:+1: repro'd this

Thanks for reporting. Any chance you could send a PR with a fix? :)
@brentvatne I believe this was owned by someone who doesn't work on RN anymore (possible to find out using blame info).

cc @ahmedre as you're likely the most knowledgable person about this

i am not really sure, but the border code in Android is here.
it looks like we have two cases - drawRectangularBackgroundWithBorders and drawRoundedBackgroundWithBorders. it seems that in this case, we go to the latter (because the style is not solid), which doesn't seem to support non-uniform borders. the easiest fix here would be to replace the if on line 105 with if (!roundedBorders). this should work because styles can be applied on Paths, and drawRectangularBackgroundWithBorders also generates a Path to draw the borders now.
if no one gets to this before Tuesday, i can give it a shot.

i am not really sure, but the border code in Android is here.
it looks like we have two cases - drawRectangularBackgroundWithBorders and drawRoundedBackgroundWithBorders. it seems that in this case, we go to the latter (because the style is not solid), which doesn't seem to support non-uniform borders. the easiest fix here would be to replace the if on line 105 with if (!roundedBorders). this should work because styles can be applied on Paths, and drawRectangularBackgroundWithBorders also generates a Path to draw the borders now.
if no one gets to this before Tuesday, i can give it a shot.

same problem.

+1 why is this closed?

+1
Border turned solid after upgrading from RN 41.2 to 43.3. Styling:
  imagePlaceholder: {
    borderStyle: "dashed",
    margin: 10,
    padding: 30,
    alignItems: "center",
    borderColor: Colors.gray,
    borderWidth: 3
  },

+1

same problem. +1

i got the same problem

Same problem here, when I'm using borderTopRightRadius and to make a semi circle. When I'm put borderTopWidth, borderRightWidth and borderLeftWidth with value and borderBottomWidth 0, the border disappear. And if I'm just put borderWidth: value and borderBottomWIdth: 0 the border bottom showing again:

and if I remove the borderTopRightRadius and borderTopLeftRadius:

still happening as you can see here
https://github.com/facebook/react-native/issues/13464

@adrianlemess Did you solve your problem? I have got same issue creating semi circle without bottom boarder.

Anyone know if this issue still open or if it's been resolved? Looks like it may have originated over a year ago. I'm still on 0.44, so just wondering if this was addressed in any of the later releases?
I think I can workaround it by just adding a <View> element to act as a faux-border, but would rather avoid that if an upgrade to the latest RN will fix it.

@gregblass Still opened.
I have the same issue when using some style like
{
    backgroundColor: APP_COLOR,
    borderTopWidth: 4,
    borderRadius: 5,
}

As long as borderRadius is set, no top border at all

I can confirm that this issue is still present in version 0.50.4.

I have the same problem(v0.47.2 on android)

+1 "react": "16.2.0",
"react-native": "0.52.0",

+1


DOC-3389

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[* ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

https://harness.atlassian.net/browse/DOC-3389

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Issue with groups containing emoji in group name

When running
for group in client.groups.list_all():
        print(group.name)

threw a groupy.exceptions.InvalidJsonError: The JSON was incomplete/invalid
Upon leaving all groups with emoji in the group name, the code runs as intended.
Python Version: 3.7.3
Groupy Version: 0.10.3
OS: Windows 10, Version 10.0.17134

This seems very similar to the know problem involving responses that exceed the max size... does it work if you omit the memberships field?
client.groups.list_all(omit='memberships')

I tested this locally. I was able to create a group with an emoji in the name and then list all group names:
>>> group_name
'Emoji group name 👍'
>>> group = client.groups.create(name='Emoji group name 👍')
>>> group
<Group(name='Emoji group name 👍')>
>>> for g in client.groups.list_all():
...     print(g.name)
...
Emoji group name 👍
<snip>
>>>

@johnluetz is this still giving you issues?


Calculator should prints entered number by user

Use case:

user starts calculator app
user enters a number
user clicks enter
calculator prints the number
user exits from calculator

Fixed by #5


Consider registering as a bower package?

Hi there, it'd be nice to be able to import this thru bower too.
I see that striptags has been taken, may be striptags.js?

Sure why not, I'll close this when I register it. It'd be nice if I could get the name striptags as it looks like the original project is 2ish years old and uses regular expressions, which are not safe for stripping HTML.
striptagsjs is probably fine though.

Late update, I released v2.1.2 on http://bower.io - it's not showing up in the search at the moment but bower info striptagsjs works.


manifest for vault:1.5 not found

Blackbox-testing security-enabled tests fail because there is no tag 1.5 for vault.
Related: https://github.com/edgexfoundry/developer-scripts/pull/312

add @hutchic for awareness


Add teams published app to AAD users via API

We have created an azure bot and linked it with the team's app. Later on, we published this app to our internal organization.
Is there any way we can automatically add the published app to our AAD users via API?
Any doc reference would be highly appreciated.

@Chang025 - Thanks for reporting your issue.
We will check this at our end and will get back to you.

@Chang025 -There is no API available to automatically add the published app to AAD users.
However to proactively install app in Teams you can follow below doc:
Use Microsoft Graph to authorize proactive bot installation and messaging in Teams - Teams | Microsoft Learn

hi @sayali-MSFT
Thank you for sharing the documentation reference. We have gone through this, until collecting user information there is a detailed API provided, but to trigger sms there is no API mentioned.
Is there any restful API to trigger the message after collecting all the related user information?
The doc reference only provided a way via framework SDK.
reference: https://learn.microsoft.com/en-us/microsoftteams/platform/graph-api/proactive-bots-and-messages/graph-proactive-bots-and-messages?tabs=dotnet#code-snippets

@Chang025 - We are checking with the engineering team, if there is any API available for this or not and get back to you.
Thanks!!

Hello @Chang025 - Sorry for delay in response.
We are checking this with engineering team and let you know the updates.

Hello @Chang025 - Sorry for delay in response.
You can use Graph API to install a Teams App for AAD users: https://learn.microsoft.com/en-us/graph/api/userteamwork-post-installedapps?view=graph-rest-1.0&tabs=http&preserve-view=true#example-1-install-an-app-for-a-user

thanks @ChetanSharma-msft for the response, yes we have added it via the same approach as suggested above.


Increase reward by rank up ?

Hi!
How can I change the configuration files to increase the rewards at each rank up? To force players to farm.
*At the same time, players can't be rewarded when they took their wheat, cause there is the anti-cheat, how to solve that ?

You could just create another job with slightly altered sets to increase the rewards.

Wut ? I mean, I want to increase by levels, not change the job bro
Cause now, u can rank up wow cool, but what do u have ? U see ?

You could create "ladders" this way. With "elite-miner" requiring one to be at least a level 10 "experienced-miner" for example.
If you do not want that and would love the project to move on faster than our free time allows, feel free to submit changes using a PR :)

Okay... why not? But there will be severals jobs when players will /job...
By the way, I can't rename jobs ? I mean, miner to Miner and after a restart of the server (not just /job reload) if I click on Miner the command says "job miner doesn't exist" ? Why ?

that's because we lowercase letters in the code. With the current implementation, user sanitized job names are not possible.
That's why we will change to UUIDs for jobs which will make names freely modifiable.