Add scalars with tests

Hello again,
I'm using this lib for validating ids, for example const UserId = v.string().assert((v) => regex.test(v), "Wrong userId"). Its all good and I love it, but I don't like that I can assign UserId to any other id in my project. The reason for that is UserId is an alias for string.
Example time:
const UserId = v.string().assert((v) => regex.test(v), "Wrong userId")

// this is an alias for string meaning type UserId = string
type UserId =  v.Infer<typeof UserId>;

By adding this amazing scalar feature, I'm able to pass a type to which I want it to cast when parsed, therefore I'm able to differentiate different ids types in the project (could be any strings, numbers, etc). For example:
const aScalar = v.scalar<string & { __aScalar: unknown }>();
const bScalar = v.scalar<string & { __bScalar: unknown }>();

const aValue: v.Infer<typeof aScalar> = aScalar.parse("hearts");
const bValue: v.Infer<typeof bScalar> = bScalar.parse("hearts");

// error here
if (aValue === bValue) { ...

// error here (pay attention to "a" and "b" inside the line)
const assigningWrongVariable: v.Infer<typeof aScalar> = bScalar.parse("hearts");

// all good here
if (aValue === aValue) { ...

// all good here
const assigningRightsVariable: v.Infer<typeof bScalar> = bScalar.parse("hearts");

I know this is a terrible practice to hide something behind a type which it is not, but it makes life much better. I will understand if this gets rejected, but maybe, just maybe for few people who does this in their daily work, this could be an easy answer...
If for some reason this is consider to be something legitimate, I don't know how to write tests that check types... (I have commented out lines that should test typings)
If this will be closed and forgotten I would like to talk about extending Valita API from child project. I could have an init place for valite where I could inject this stuff.
Thoughts?
Cheers,
Arturs

First of all: thank you for this pull request. I also use branded types for emulating nominal typing in TypeScript. However, I think v.scalar would duplicate (a subset of) .assert's functionality and it would be best to just address this in the documentation. Your first example can be implemented like so:
type UserIdType = string & { __brand: unknown };

const UserId = v.string().assert<UserIdType>((v) => regex.test(v), "Wrong userId");

Now v.Infer<typeof UserId> is UserIdType.
This approach allows asserting the type after an arbitrary chain of validation steps. As .assert also supports type predicates, you can even package assertions as reusable helpers (or reuse pre-existing type predicates):
type UserIdType = ...

function isUserId(value: string): value is UserIdType {
  return regex.test(value);
}

// Then later:

const UserId = v.string().assert(isUserId);

Now again v.Infer<typeof UserId> is UserIdType.


Cannot find video error in livestream

~/.local/bin/youtube-dl http://livestream.com/ChessCast/events/4298205/videos/99776320 --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'http://livestream.com/ChessCast/events/4298205/videos/99776320', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.09.22
[debug] Python version 2.7.9 - Linux-3.19.0-28-generic-x86_64-with-Ubuntu-15.04-vivid
[debug] exe versions: ffmpeg 2.5.8-0ubuntu<IP_ADDRESS>, ffprobe 2.5.8-0ubuntu<IP_ADDRESS>
[debug] Proxy map: {}
[livestream] 99776320: Downloading webpage
ERROR: Cannot find video 99776320; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/home/tbudiman/.local/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 660, in extract_info
    ie_result = ie.extract(url)
  File "/home/tbudiman/.local/lib/python2.7/site-packages/youtube_dl/extractor/common.py", line 287, in extract
    return self._real_extract(url)
  File "/home/tbudiman/.local/lib/python2.7/site-packages/youtube_dl/extractor/livestream.py", line 190, in _real_extract
    raise ExtractorError('Cannot find video %s' % video_id)
ExtractorError: Cannot find video 99776320; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

It doesn't load on a browser, that it works for you?

The link works in my browser, it leads to:
http://api.new.livestream.com/accounts/7928738/events/4298205/videos/99776320.m3u8
on my chrome in Android.

It's because this video with id "99776320" and the corresponding mp4- and m3u8-urls is part of the json-element "post" instead of "feed". Also it seems youtube-dl doesn't detect the m3u8-urls at all.
  "post": {
    "id": 99776320,
    "caption": "In the studio with Vladimir Kramnik",
    "duration": 1032000,
    "broadcast_id": 99766365,
    "original_duration": 20446747,
    "copied_from": 99766364,
    "description": null,
    "asset": {
      "qualities": [
        {
          "name": "Mobile Quality",
          "bitrate": 174000,
          "width": 480,
          "height": 270,
          "angle": 1
        },
        {
          "name": "Medium Quality",
          "bitrate": 614000,
          "width": 768,
          "height": 432,
          "angle": 1
        },
        {
          "name": "High Quality",
          "bitrate": 1628000,
          "width": 864,
          "height": 486,
          "angle": 1
        },
        {
          "name": "HD Quality",
          "bitrate": 2160000,
          "width": 1280,
          "height": 720,
          "angle": 1
        }
      ],
      "akamai_stream_id": 125242,
      "stream_name": "7928738_4298205_227ff20f_1",
      "workflow_status": "downloaded",
      "state": null,
      "workflow_retry_count": 0,
      "workflow_error": 0,
      "workflow_progress": 0,
      "downloader_progress": 0,
      "transcoded": true,
      "transcode_started_at": "2015-09-20T15:34:44.844Z",
      "inserted_to_download_queue_at": "2015-09-20T15:31:55.354Z",
      "inserted_to_transcoding_queue_at": null,
      "received_video_from_user_at": null,
      "application_name": "Livestream Producer"
    },
    "in": 10955,
    "out": 11987,
    "thumbnail_width": 1280,
    "thumbnail_height": 720,
    "draft": false,
    "tags": null,
    "created_at": "2015-09-20T13:19:36.273Z",
    "updated_at": "2015-09-20T16:33:25.588Z",
    "streamed_at": null,
    "publish_at": "2015-09-20T13:19:35.955Z",
    "views": 17084,
    "likes": {
      "total": 0,
      "data": []
    },
    "comments": {
      "total": 0,
      "data": []
    },
    "thumbnail_url": "http://img.new.livestream.com/events/00000000004195dd/d3d9711a-a8ca-4e9d-8269-cef891b963fe_11280.jpg",
    "thumbnail_url_small": "http://img.new.livestream.com/events/00000000004195dd/d3d9711a-a8ca-4e9d-8269-cef891b963fe_11280_150x84.jpg",
    "progressive_url": "http://pdvod.new.livestream.com/events/00000000004195dd/d3d9711a-a8ca-4e9d-8269-cef891b963fe_614.mp4?start=10955&end=11987&__gda__=1443834614_6e96dd24d399500a32470d83530191ef",
    "progressive_url_hd": "http://pdvod.new.livestream.com/events/00000000004195dd/d3d9711a-a8ca-4e9d-8269-cef891b963fe_2160.mp4?start=10955&end=11987&__gda__=1443834614_7c0aecfdc32688f55e400347fcc9c58d",
    "smil_url": "http://api.new.livestream.com/accounts/7928738/events/4298205/videos/99776320.smil",
    "secure_smil_url": "https://api.new.livestream.com/accounts/7928738/events/4298205/videos/99776320.secure.smil",
    "secure_progressive_url": "https://pdvod.new.livestream.com/events/00000000004195dd/d3d9711a-a8ca-4e9d-8269-cef891b963fe_614.mp4?start=10955&end=11987&__gda__=1443834614_6e96dd24d399500a32470d83530191ef",
    "secure_progressive_url_hd": "https://pdvod.new.livestream.com/events/00000000004195dd/d3d9711a-a8ca-4e9d-8269-cef891b963fe_2160.mp4?start=10955&end=11987&__gda__=1443834614_7c0aecfdc32688f55e400347fcc9c58d",
    "m3u8_url": "http://api.new.livestream.com/accounts/7928738/events/4298205/videos/99776320.m3u8",
    "secure_m3u8_url": "https://api.new.livestream.com/accounts/7928738/events/4298205/videos/99776320.secure.m3u8",
    "f4m_url": "http://api.new.livestream.com/accounts/7928738/events/4298205/videos/99776320.f4m",
    "secure_f4m_url": "https://api.new.livestream.com/accounts/7928738/events/4298205/videos/99776320.secure.f4m",
    "type": "video"
  },


chat: fix loading for multidms

Little drive-by PR. Not sure how we missed this, but it was an easy fix. Thought we had an issue for this, but this is the issue reported by Jack in Tlon Public

I can't seem to find evidence of the issue; what was the symptom?


LCFS - Fuel codes, add Contact name and Contact email

Description: The business area would like to add optional contact information (name and email) to fuel codes to help notify fuel producers when fuel codes are near expiry.
Wireframe:
https://preview.uxpin.com/da68049c0af15edd7ce4e47e4bee431fed40c30d#/pages/168739509?mode=m
Purpose and benefit to user: The business area would like to investigate having LCFS automate 90 alerts for expiring fuel codes. Fuel producers would receive an email to update their fuel codes.
Acceptance Criteria:

[ ] Given I am a IDIR users, When I am adding fuel code info, then I can add an optional Contact name and Contact email.
[ ] Given I am a IDIR users, When I am adding fuel code info, when I am filling out these fields, it should display predictive text based on what is already available in the database
[ ] Given I am a IDIR users, When I am adding fuel code info, when selecting a fuel code, the contact name and contact email should be autofilled based on the last fuel code data

Dev Notes

[ ] add Contact Name and Contact Email to the schema
[ ] use autocompleteEditor for those fields
[ ] add the data to the table-options endpoint
[ ] change the fuel code auto populate to include the above fields

Notes: Fuel producers are not the same as fuel suppliers. They are not users of TFRS/LCFS, they may not operate in BC and may not be eligible to get a Business BCeID.

@AlexZorkin @kevin-hashimoto receiving a 500 error accessing the fuel code page and the new fuel code button also gives an error so these cards can't be tested

@airinggov I believe this should be fixed on dev now. we had a problem with the seeders

The 'Contact name', 'Contact email' columns should be optional but they are currently showing as required.
'Former company' is also showing as required and it should be optional instead.

The column "Former company" is still showing up as a required field. This should not be a required field:

contact email is still not saving when null


Update README.md

added MIT License icon in README.md
Summary

With the purpose of adding information and easy look-up access for others
Test plan

@lauraguevara97 thanks! Can you make the linter happy by running yarn lint:md?

Thanks!


HttpRequest.Protocol always / only reports HTTP/1.1

"If an HTTP/2 connection is established, HttpRequest.Protocol reports HTTP/2."
That's not true. I have the following setup in Program.cs:
`            .ConfigureKestrel((context, options) =>
            {
                options.Listen(IPAddress.Any, 5002, listenOptions =>
                {
                    listenOptions.UseHttps("/tmp/localhost.pfx", "4xLabs.com");
                    listenOptions.UseConnectionLogging();
                    listenOptions.Protocols = HttpProtocols.Http1AndHttp2;
                    //listenOptions.UseHttps("testCert.pfx", "testPassword");
                });
            });

And I log HttpRequest:
            app.Use(async (context, next) =>
            {
                // Request method, scheme, and path
                _logger.LogInformation(JsonConvert.SerializeObject(new RequestLog(context?.Request?.Method,
                                    context?.Request?.Scheme,
                                    context?.Request?.PathBase,
                                    context?.Request?.Path,
                                    context?.Request?.Host.ToString(),
                                    context?.Request?.ContentLength,
                                    context?.Connection?.RemoteIpAddress?.ToString(),
                                    context?.Request?.QueryString.ToString(),
                                    context?.Request?.ContentType,
                                    context?.Request?.Protocol,
                                    context?.Request?.Headers
                                    )));
                await next();
            });

I use Chrome to browse to my application. Logs are collected by Elasticsearch and shown in Kibana. Application is deployed to Kubernetes on cloud behind Load Balancer which uses HTTPS. Unfortunately ONLY HTTP/1.1 is recorded. There is no other protocols observed.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f22dcddb-0c25-be1a-9a02-9e71dce86bdb
Version Independent ID: 3e0e12c2-e544-5655-e001-d2ec180b11a8
Content: Kestrel web server implementation in ASP.NET Core
Content Source: aspnetcore/fundamentals/servers/kestrel.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @khteh ... please open this for the engineers at https://github.com/aspnet/AspNetCore/issues. If we need to make a docs change after they react to that, then we can re-open over here.


add fastemebed under a feature flag

[x] add fastembed under a feature flag as this can help save compile time or bring unnecessary dependencies if not used.
[x] remove duplicate example/fastembed.rs
[x] add how to run the cargo example in some of the files
[x] convert ollama samples to use English instead of Spanish

LGTM


Add ifndef guard for Option

I don't really understand what this is for, but it is missing compared to the other bindings. This makes the code in cxx.h get pasted into the expanded C++ binding code

This means that the user does not need to #include "rust/cxx.h" in their code. This is particularly important when there is no C++ file in the build, just a rust library exposing stuff to C++


Graphql lib problems with gql_exec

Hi, I'm facing this error:
../go_core/go_network/lib/src/client/logger_http_link.dart:67:23: Error: Required named parameter 'response' must be provided.
    yield gql.Response(
                      ^
../../../flutter/.pub-cache/hosted/pub.dartlang.org/gql_exec-0.3.2-alpha+1635885531651/lib/src/response.dart:23:9: Context: Found this candidate, but the arguments don't match.
  const Response({
        ^^^^^^^^
../../../flutter/.pub-cache/hosted/pub.dartlang.org/graphql-5.0.0/lib/src/core/query_manager.dart:427:23: Error: Required named parameter 'response' must be provided.
              Response(data: cachedData),
                      ^
../../../flutter/.pub-cache/hosted/pub.dartlang.org/gql_exec-0.3.2-alpha+1635885531651/lib/src/response.dart:23:9: Context: Found this candidate, but the arguments don't match.
  const Response({
        ^^^^^^^^
../../../flutter/.pub-cache/hosted/pub.dartlang.org/gql_link-0.4.0/lib/src/response_parser.dart:10:64: Error: Required named parameter 'response' must be provided.
  Response parseResponse(Map<String, dynamic> body) => Response(
                                                               ^
../../../flutter/.pub-cache/hosted/pub.dartlang.org/gql_exec-0.3.2-alpha+1635885531651/lib/src/response.dart:23:9: Context: Found this candidate, but the arguments don't match.
  const Response({
        ^^^^^^^^
../../../flutter/.pub-cache/hosted/pub.dartlang.org/gql_http_link-0.4.0/lib/src/link.dart:128:19: Error: Required named parameter 'response' must be provided.
    yield Response(
                  ^
../../../flutter/.pub-cache/hosted/pub.dartlang.org/gql_exec-0.3.2-alpha+1635885531651/lib/src/response.dart:23:9: Context: Found this candidate, but the arguments don't match.
  const Response({
        ^^^^^^^^

I have these following deps on my pubspec:
graphql: 5.0.0
http: ^0.13.3
Have already tried to fix the gql_exec version with override, but nothing works.

Important note, the problem can be fixed by add the override_dependency of gql_exec to 0.3.0, but if you have multiple packages in your project you must change in all your projects the graphql version and remove the ^.

Hello
I am sorry for this breaking change, the issue is being tracked on https://github.com/gql-dart/gql/issues/287 and currently you need to use dependency override.


AnimationTree doesn't work right anymore godot 4 beta 14,15

Godot version
4.0 beta 14,15
System information
Ubuntu 22.04 1 LTS
Issue description
After update from godot4 beta 13 to beta 14 AnimationTree doesn't work right anymore.
_animation_tree.set["parameters/move_state/current"] = 1 #<= NOT WORKING
ERROR:  Invalid set index 'parameters/move_state/current' (on base: 'Callable') with value of type 'int'
Change code to:
_animation_tree.set("parameters/move_state/current", 1) #<= IT WORKS BUT NOT RIGHT
Control of AnimationTree is corrupt. See appended project.
Steps to reproduce
Please download and start "Minimal reproduction project"
Minimal reproduction project
game_character_test.zip

It's not a bug. The parameter control has changed a lot, but unfortunately the documentation has not been updated, sorry. See also https://github.com/godotengine/godot/pull/71418.

Thank you, for your comment. Works perfect!

I tried the minimal reproduction project above and already updated the code to the latest implementation.
I think the reset is only working in the editor but not on runtime. It doesn't reset the animation on runtime, instead, it is stuck at the last frame. But if you turn off the reset on the AnimationNodeTransition, then the animation works on runtime.
Not sure if this is how it is supposed to behave.

You can test the attached sample project.
game_character_test (new).zip

@metanoia83 The request should not be spammed. So it should be:
if _animation_tree.get("parameters/move_state/current_state") != "jump":
	_animation_tree.set("parameters/move_state/transition_request", "jump")

@metanoia83 The request should not be spammed. So it should be:
if _animation_tree.get("parameters/move_state/current_state") != "jump":
	_animation_tree.set("parameters/move_state/transition_request", "jump")

Do you mean like this?

Below is the output for the code above.
https://user-images.githubusercontent.com/39092464/215300117-e1d938e5-a5ac-437f-ae6f-aa2b3aa4ef8c.mp4

func handle(is_on_floor: bool, _motion: Vector2) -> void:
	if not is_on_floor:
		if _animation_tree.get("parameters/move_state/current_state") != "jump":
			_animation_tree.set("parameters/move_state/transition_request", "jump")
	else:
		if _animation_tree.get("parameters/move_state/current_state") != "move":
			_animation_tree.set("parameters/move_state/transition_request", "move")
	# Blend position for walk speed based on motion.
	_animation_tree.set("parameters/move/blend_position", _motion.length())

https://user-images.githubusercontent.com/61938263/215300871-73ba1c9b-8b51-419f-9f7d-cd02be5c5a7a.mp4

@TokageItLab Thanks for the help.


Compile time checks

I didn't find any information about compile-time checks. If message in locale file is missing then string passed to macro will be displayed: https://github.com/longbridgeapp/rust-i18n/blob/ba2c7cb8e1083ccc12c8887623a43c25bf9ee4d6/crates/macro/src/lib.rs#L183-L186
Will it be hard to create a separate macro or add option for t!() to make sure the message I use exists in the locale file at compile time (like sqlx query! does)? This can be tricky, so maybe an easier way is to create xtask (or test) to check that all the messages I'm using in the code are defined. This way it can be easily integrated with CI.
All I want to achieve is if my app has passed CI, the locale files will have the appropriate content and the user won't see "en.wron_mesag".
Probably internationalization-rs can be used to see how they achieve compile time verification (using build.rs).

Can we implement this by parsing all the Rust source code, and for the macro invocation, check if its ident is t, if so, extract the translation key, then we can do the needed checks. 🤔
This should work for code like:
t!("translation key");

However, usages like:
use rust_i18n::t as the_t_macro;

the_t_macro!("translation key");

and
let translation_key = "translation key";
rust_i18n::t!(translation_key);

need us to do more things with the AST.

I gave this a go today, in Topgrade's locale file checker:
$ ./topgrade_i18n_locale_file_checker --locale-file ../topgrade/locales/app.yml --rust-src-to-check ../topgrade/src
Errors Found:
  UseOfKeysDoNotExist
    file '../topgrade/src/self_update.rs' / line '52' / column '27' / key 'Respawning...'
    file '../topgrade/src/steps/os/windows.rs' / line '202' / column '21' / key 'Could not find Topgrade in any WSL disribution'
    file '../topgrade/src/steps/os/windows.rs' / line '209' / column '20' / key 'Windows Update'
    file '../topgrade/src/main.rs' / line '488' / column '19' / key '\n(R)eboot\n(S)hell\n(Q)uit'

It works well, though I am not sure if we can do this in build.rs 🤔


Automatic documentation doesn't discover authentication method

Following the implementation of multiple auth backends, I didn't find a way yet to make OpenAPI discover the authentication method (even at least one).
I guess we have to go deeper in fastapi implementation to understand how to do this.

https://fastapi.tiangolo.com/tutorial/security/simple-oauth2/#authenticate

My 2cents as I don't know what you've tried or not.
From what I've seen, to display the "Authorize" button, OpenAPI needs an 'Authorization' header. (https://stackoverflow.com/questions/32910065/how-can-i-represent-authorization-bearer-token-in-a-swagger-spec-swagger-j).
In FastAPI, it seem's open-api schema are generated 'using routes', each route can integrate is own Security_schemes (https://github.com/tiangolo/fastapi/blob/4e8080f29013dcb6850518bb9011bf82c9081828/fastapi/openapi/utils.py)
Here is a test: https://github.com/tiangolo/fastapi/blob/025b38df40a18f94322fab445aee13024fe783da/tests/test_security_oauth2_password_bearer_optional.py
Using this kind of fastapi class 'OAuth2PasswordBearer' injected with Depends() (or Security()) in routers seems to add oauth2 security scheme in the open-api schema generation resulting in the 'Authorize' button showing is face.
oauth2_scheme = OAuth2PasswordBearer(tokenUrl="/token", auto_error=False)
async def get_current_user(token: str = Depends(oauth2_scheme)):
    credentials_exception = HTTPException(
        status_code=status.HTTP_401_UNAUTHORIZED,
        detail="Could not validate credentials",
        headers={"WWW-Authenticate": "Bearer"},
    )

[...]
But how/where to integrate/mimic this inside fastapi-users ?

Thank you @MariusMez for digging through this 👍 The problem here is that we "break" the dependency chain for various reasons (support of several authentication backends, configuration...).
Our best bet would be if we could hook into the OpenAPI schema generation to add our own logic 🤔

I hard code in
fastapt_users/autgebtication/init.py
from fastapi import Depends, HTTPException

async def get_current_active_user(self, request: Request ,check = Depends(OAuth2PasswordBearer(tokenUrl='/auth/users/login/thaijo21', auto_error=False))) -> BaseUserDB:
        user = await self.get_current_user(request)
        if not user.is_active:
            raise self._get_credentials_exception()
        return user

I add this
check = Depends(OAuth2PasswordBearer(tokenUrl='/auth/users/login/thaijo21', auto_error=False))
and in
fastapt_users/autgebtication/jwt.py

async def get_login_response(self, user: BaseUserDB, response: Response) -> Any:
        token = await self._generate_token(user)
        #return {"token": token}
        return {"access_token": token}

I return access_token instead of token
Then I can use auth on fastapi swagger ui.
Note. I use router prefix as /auth and jwt name as thaijo21

but I can't apply to your jwt class. T_T

Currently fastapi-users uses httpx-oauth for the OAuth flow, would it be interesting perhaps to add the use of the authlib library to facilitate this flow?
Examples: https://github.com/authlib/demo-oauth-client
Documentation: https://docs.authlib.org/en/latest/client/fastapi.html

Hi @martincolladofab! I deliberately put authlib aside because of its license : https://authlib.org/plans
You cannot use it for free for commercial projects, which is a totally no-go for me.
That's why I created httpx-oauth which is a simple and pure async OAuth client.

wow! I did't know about the pricing licence! I think that it's no-go for alot of people hehe


Date format for Germline related objects

AlleleDescription.release_date and GermlineSet.release_date currently lack a date format. OpenAPI uses RFC3339 section 5.6 (format: date), which identical as the ISO8601 notation we have used so far in other fields.

Fixed via PR #593


CSS issue: List spacing is very tall between top-tier items, but crowded on second tier items:

http://kubernetes.github.io/v1.1/docs/user-guide/known-issues/

any tighter than this latest felt really cramped to me

Agreed, looks good


Serialized class to JArray

Hello! Great work on the library! I am having a simple issue that you could probably answer in your sleep.
Serializing an object works really really well, but I would like to add each of my serialized strings to a JArray so that I can form myself a valid JSON array. However upon doing so, I am getting a double-serialized output.
[   "{\"Lot\":\"SA12345678\",\"Inspector\":\"Inspector\",\"RPN\":\"Something\",\"Date\":\"11/16/18\",\"Time\":\"7:00 a.m.\",\"Sample\":\"1\",\"Displacement\":\"0.12\",\"Force\":\"1.14\"}"]
I populate my object with values, then I serialize the object to get the Lot, RPN, etc, valid JSON string. Upon getting that I use jArray.Add(JsonConvert.SerializeObject(newRow));, which generates that double-quoted string above.
The only way I can resolve this issue is to use jArray.Add(JsonConvert.DeserializeObject(JsonConvert.SerializeObject(newRow)));, which seems redundant. Would you know of a method that works better than serializing and serializing? Thank you.

In order to add a non-primitive POCO to a JArray, you need to explicitly serialize it to a JToken.  Your deserialize/serialize combination accomplishes this, but a better solution is to use JTokenFromObject(newRow):
jArray.Add(`JTokenFromObject(newRow));

See: Could not determine JSON object type for type “Class” and With NewtonSoft, how to create an array of strings with JArray.


Properly define non-integer single PKs when using SQLite

Non-integer single primary keys aren't defined as such in the table CREATE statement when using SQLite as a database.

Thanks!


chore(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-resource-group
module
patch
v1.1.0 -> v1.1.4

terraform-ibm-modules/cbr/ibm (source)
module
minor
1.15.1 -> 1.16.0

terraform-ibm-modules/resource-group/ibm (source)
module
patch
1.1.0 -> 1.1.4

Release Notes

terraform-ibm-modules/terraform-ibm-resource-group (github.com/terraform-ibm-modules/terraform-ibm-resource-group)
v1.1.4
Compare Source
Bug Fixes

fix variable name in catalog validation json template (#535) (6cadceb)

v1.1.3
Compare Source
Bug Fixes

fix typo in catalog validation json template (#534) (c4eb24f)

v1.1.2
Compare Source
Bug Fixes

cleanup for catalog onboarding (#533) (ec38fcc)

v1.1.1
Compare Source
Bug Fixes

create ibm catalog manifest which will be used for onboarding to the IBM catalog module registry (#528) (87bd79c)

terraform-ibm-modules/terraform-ibm-cbr (terraform-ibm-modules/cbr/ibm)
v1.16.0
Compare Source
Features

The KMS CBR context are now set to target HPCS in the fscloud module by default, starting with version 1.16.0, instead of Key Protect.

Important Changes:

This update replaces the previous default Key Protect rule with HPCS. As a result, when you upgrade to v1.16.0, the context will no longer be the set for the key protect CBR rule.
To maintain the behavior from previous versions when upgrading to v1.16.0, you should set the new variable kms_service_targeted_by_prewired_rules variable to include 'key-protect.' This ensures that the 'key-protect' CBR context continues to be set. For detailed instructions on configuring this variable, please refer to the fscloud module's variables.tf file.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.4.13 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add pytest_splinter_screenshot_dir hook

Fixes #119

The splinter_screenshot_dir fixture now pulls its value from this hook instead of the config option directly.
For non-xdist runs, this hook can be used instead of the splinter_screenshot_dir fixture. It will make no real difference.
For xdist runs, this hook allows the screenshot directory to be overridden from a conftest file instead of only as a command-line option

Coverage remained the same at 88.07% when pulling b9702f690f7f12ebc6d11dd77c3fbb37775b9255 on jsfehler:add_hooks into 7162de64b71d5c24a6ebd074ae09b085b573351c on pytest-dev:master.

I am very much against using hooks for configuration. IMO configuration should really be in the pytest.ini or via command line arguments. I think we should also remove the configuration via fixtures.

@youtux I agree hooks are not meant for this, but fixtures as a config are not bad, you know it from the paylogic right?

@youtux I don't disagree. Fixtures aren't perfect for configuration either. This situation shows the limitation of fixtures to modify config data.
Should the pytest-splinter plugin allow configuration to be set and modified in python code? If so, there's only the choice of hooks or fixtures.
In this case, the splinter_screenshot_dir fixture is used by the rest of the plugin. This hook acts as an in-between for the command line argument and the fixture. The advantage of keeping the fixture is easily retrieving the value inside a test, or for use by other fixtures.
Off the top of my head, a different pattern would be:

Replace fixtures with hooks
Expose the results as a single fixture with a dict of the results, ie:

@pytest.fixture(scope='session')
def splinter_configuration(request):
    config = {
        'splinter_screenshot_dir': '...',
        'splinter_headless': True,
        etc, etc
    }
    return config

Of course, overriding such a fixture would be pointless.

I would propose another approach then, assuming it would work (I cannot check it by my self at least for the next week, so maybe you can help me).
I would move all the configuration to the ini file, and remove the configuration fixtures.
To solve the problem with having an ad-hoc configuration for the slaves, I would instead use the hook from pytest-xdist that is invoked on slave initialization. In that hook, I would modify the pytest config object, upgrading the value of the configuration key that you want to change (splinter_screenshot_dir in this case).
This should fix the issue, and it’s a user space hook implementation, so pytest-splinter doesn’t have to be aware of this edge case. Maybe we can just mention the trick in the documentation.


Core: add random numbers generators

Similar to #4315.
Original discussion here: #4305.
Let's first discuss the syntax here @Amerousful @shoaib42.

Original proposal from @Amerousful in https://github.com/gatling/gatling/pull/4311:
"#{randomNumber(10)}"
"#{randomNumber(5,10)}"
"#{randomNumber(5, max)}"
"#{randomNumber(5, maxL)}"

My comments on this:

all those would generate Long. Shouldn't we have distinct randomInt and randomDouble?
I'm not found of the magic max and maxL. I'd rather stick to explicit values.

I would suggest:
"#{randomInt()}" // full int range
"#{randomInt(10)}" // between 0 and 10 inclusive
"#{randomInt(5, 10)}" // between 5 and 10 inclusive
"#{randomLong()}" // full long range
"#{randomLong(10)}" // between 0 and 10 inclusive
"#{randomLong(5, 10)}" // between 5 and 10 inclusive

WDYT?

I want to add something from my side:
I think that randomDouble will be beter but how to handle a precision if someone want to have 3.14 (not 3.1415...) - do you have thinking about using BigDecimal to handle all numbers ?
How about negative values ?
I don't thik that full range is a good idea I thik that between X and Y it's only good solution - this will be clear how it works when someone will see it.

Late to join the party, here are my thoughts. I've listed them in the table below

func/method
use case
other requirements

randomLong()
full range -ve to +ve, counts/random ids

randomLong(min, max)
enumerated codes, "device type" etc, randomLong(0, 10) covers randomLong(10)

randomInt(min, max)
same as above

randomDouble(min, max)
longitutde, latitude, we need negative floating points (OOTB is not covered), Device send slocation and response depends on this, might be a hot key that gets involved that needs stressing
needs +ve and -ve

randomDouble(min, max, round)
currency, ending up in a DECIMAL in db
needs +ve and -ve

I feel that adding too many flavours fo randomLong/randomDouble will turn into a long list for the endUser to know. That's my opinion, thought I throw that in.
If it were not for gRPC I would have said randomLong(min,max) is enough since all of it gets converted to TEXT anyways. What do you guys think about keeping the following list only
(Let's avoid the word precision with floating point numbers in this context. Someone will nit pick on that.)

func/method
defaults

randomLong()

randomLong(min, max)

randomInt()

randomInt(min, max)

randomDouble(min, max, round)
round = -1, no rounding

randomFloat(min, max, round)
round = -1, no rounding

Now for naming the EL function names, should we do randomInt(), randomLong() or something shorter, yet understandable, like randInt() or rInt()

@slandelle

Yep, you're totally right about Long. I also realized this after a while, so locally I have the version which returns AnyVal.
About max and maxL I thought that will be preferable for the user instead pass the explicit value. I mean case when your min above zero. And in my case, I use one unified method instead methods for each number type.
But on the other side, we pay for that by performance and extra logic. It's all about choice convenience vs performance.

Nonetheless, your suggestion is good and I like it.
Also, there is a question about realization. I tried to find another solution instead of good well known like ThreadLocalRandom.current()...  and I didn't find...
I found only SplittableRandom which has +- the same performance as ThreadLocalRandom

@shoaib42 @gemiusz Initial task didn't contain any info Double or any another type instead Int and Long... Nevertheless, we can generate Double in this way: 😅
#{randomInt(1,10)}.#{randomInt(100,999)}

@Amerousful I pointed to the fact that sometimes you'll have to use the correct type. I am not a 100% sure how this would work with gRPC plugin, where the proto buff has a float and you send #{randomInt(1,10)}.#{randomInt(100,999)} to construct a payload. I'll find time today to try this out with the gRPC plugin. Something like having an int32 and passing the #{currentTimeMillis()}.
Another scenario could be the latitude, longitude generation in a given range. say for example 11.987 to 12.123 , the end user would have to juggle with this while generating with string concactination.
I'm trying to focus on the use cases and requirements, to help this effort :)

Yep, I understand. My example was just a workaround :)

Re https://github.com/gatling/gatling/issues/4316#issuecomment-1215076367
That won't let you generate 3.01.

@slandelle exactly my point, and my thoughts are gatling would see issues open up for "can we get randomDouble" etc. Just trying to avoid the misses before this roles out. The call isn't mine, just throwing out ideas :)

randomDouble(min, max, round) | round = -1, no rounding

I really don't like magic values.
We could have randomDouble(min, max) and randomDouble(min, max, fractionalDigits).

randomFloat

Let's postpone this one and see if someone ever needs it.

do you have thinking about using BigDecimal to handle all numbers ?

@gemiusz This feature is for generating HTTP payloads, not for performing scientific or financial computations. I don't think we need BigDecimal support. Just like above, I might be wrong, but let's wait for a use case to show up. YAGNI.

@shoaib42 Yep, you're totally right)
I understand and don't deny necessity functions for random Double and Float. I just to basic on initial task which contains info only about Int and Long

@shoaib42 I think it's fair if you will implement random for Double :)

@Amerousful There is an edge case with rounding double. A quick hacky way is Math.round(nextDouble(min,max) * pow(10, digits)) / pow(10, digits), but this can break near double max value. @slandelle I'll assume that the use of randomDouble will not be used to generate values near max values or epsilon.

@shoaib42 There are 2 different cases:

randomDouble(min, max) => no brainer
randomDouble(min, max, fractionalDigits) => your hacky way will overflow for large number or large number of digits. AFAIK, the only correct way to do that is to use BigDecimal, eg:

double rounded = BigDecimal
      .valueOf(10.12345)
      .setScale(2, BigDecimal.ROUND_HALF_UP)
      .doubleValue();

And let's use the term scale instead of digits as in BigDecimal's javadoc.
Does this sounds good to you?

I'm aware of the BigDecimal and can make the changes. My only concern is performance as internally it will convert to string/char array. I thought of doing string with formatter and back to double, but again I thought of performance. Should I first check what performance better? There's always a catch with floating points with precision.

I suspect any correct solution would have poor performance.
Actually, this makes me wonder if this scale feature is really something we want for numbers. Shouldn't actually such generator output text? I suspect the main (sole?) usage of this generator would be to populate forms/JSON payloads.
Maybe that's something we might want to save for later, or never implement ourselves but just tell users to use dedicated libraries for advanced generators such as java-faker.


Add catalogue-api cluster

This change is to move towards partitioning read / write load on the catalogue-api following up on a recent incident retro.
This PR adds a catalogue-api cluster and adds a script to provision necessary roles & users.
Future PRs will add scripts to initiate creation of follow indexes and to wire this up to the catalogue-api service. This change simply adds the cluster/secrets.

Will we have to (or should we) make changes to the other places that read from the catalogue cluster? ie items & requests and maybe some other things too
Is there any way of avoiding the local-exec provisioner? eg a third party provider like this one

Will we have to (or should we) make changes to the other places that read from the catalogue cluster? ie items & requests and maybe some other things too

We will need to update all the API apps to read from the new cluster eventually.

Is there any way of avoiding the local-exec provisioner?

The linked provider seems only to support AWS elasticsearch - i'm more comfortable waiting for this functionality to appear in the official provider.


float value fnt file

Please ensure you have given all the following requested information in your report.
Issue details
Please provide the details of your issue
Reproduction steps/code
Even if you think your issue is trivial to reproduce, please supply a SSCCE that demonstrates your issue.  This saves time on our end, and makes it much more likely that your issue will be fixed.
You can find barebones templates here
Version of LibGDX and/or relevant dependencies
Please provide the version(s) affected.
Stacktrace
//Please provide the stacktrace if applicable

Please select the affected platforms

[ ] Android
[ ] iOS (robovm)
[ ] iOS (MOE)
[ ] HTML/GWT
[ ] Windows
[ ] Linux
[ ] MacOS

@DragonKing91-jych please close this yourself; you should be able to tell that this is a duplicate of #6065 .


[Dubbo-1983] Support Protobuf Serialization

What is the purpose of the change
close #1983
Brief changelog
This Protobuf support is based on Protostuff, so we don't need to write proto files by hand.
And this PR is based on @vangoleo 's demo here.
Maybe some benchmarks will be helpful, and I will add them later.
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn clean install -DskipTests & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Why it necessary?
Pls test it yourself, but submit simplest.

I find several class in dubbo-serialization module.
I will start a work on optimize it. Issue:
https://github.com/apache/incubator-dubbo/issues/2623
After that, we can make this pr's test simply.

I find several class in dubbo-serialization module.
I will start a work on optimize it. Issue:
#2623
After that, we can make this pr's test simply.

This is very good, these serialization modules have almost the same unit tests, it's good to simplify it.

@satansk
I will optimize it ASAP. Let us make this pr pending before I finish the ut module. Thx for your contribution.

@satansk When can you finish your pull request? I would like to use it.

@jliu666 I'm glad you like this feature, but this PR still needs some improvements as @Jeff-Lv commentted, and beside that, I will use the new UT instead of the duplicated pojoes.
I will try to finish it as soon as possible :)

I have finish the test of serialization module.
Pls remove the unnecessary POJO class.

@satansk failed to response, I decide to merge his change first, then follow up with the pending review comments, what do you think?

Hi @beiwei30 , this PR still need the following improvements:

the unit test failed on java.sql.time and pojo which has loop reference;
use cache to improvement performance just as @Jeff-Lv commented;

And sorry for the delay, because I'm preparing a release in my company :)

Hi @beiwei30 https://github.com/beiwei30 , this PR still need the following improvements:
the unit test failed on java.sql.time and pojo which has loop reference;
use cache to improvement performance just as @Jeff-Lv https://github.com/Jeff-Lv commented;
And sorry for the delay, because I'm preparing a release in my company :)
I found you are enhancing this PR today, and thanks a lot. Some days later I can be back to the Dubbo community and do some more contributions.
Best regards.

在 2018年10月26日，上午10:42，Ian Luo<EMAIL_ADDRESS>写道：
@satansk https://github.com/satansk failed to response, I decide to merge his change first, then follow up with the pending review comments, what do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/apache/incubator-dubbo/pull/2618#issuecomment-433267786, or mute the thread https://github.com/notifications/unsubscribe-auth/AD0y76Ft6QzNL-VRphc_HNpTyxdIxJ8Jks5uonaGgaJpZM4XQ9wC.


Merge PR to Create Version 0.6.0

Add Filtering. #88
Add Order By. #91
Add Template Null check. #94
Fix destroy of appropriate deckgrid. #108

@akoenig Please.


Update "Active Directory" to "Entra ID"

(Active Directory was always incorrect, should have been "Azure Active Directory")

@bandersmsft

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@SvenAelterman @Jak-MS I appreciate you opening this PR, but please close it.
Our official guidance was to replace all mentions of Azure Active Directory, Azure AD, and AAD with Microsoft Entra ID, so Microsoft is missing from the offical name. We had automated processes that were supposed to make all needed changes, Azure-wide, but somehow a few were missed in the Cost Management + Billing doc articles.
I'll manually make the needed updates by end of day today. The this article should get refreshed live by 10:30 AM PST tomorrow.


Checking checkboxes introduces search

Thanks for the plugin!
Checking checkboxes with WorkbenchToggleCheckbox (and also adding them with the other command) starts highlighting brackets like foo[1] in the code below the notes window.
My fix right now is to change the mapping to
<Plug>WorkbenchToggleCheckbox <bar> :nohl<RETURN>
but I guess there could be a better fix.
Thanks

@BoGGoG Hi, thanks for raising this issue, this problem has been there since the beginning, but because my current setup has the highlighting turned off by default so this is not bugging me. Recently, there are a few people raising similar issues. So I decided to spend some time and try to fix this. And I am happy to say it is fixed now. On top of that, I have combined the functionality, so that you no longer need to have a second mapping to add a checkbox. By calling the mapping below, it will either add or toggle the checkbox on that line.
nmap <leader><CR> <Plug>WorkbenchToggleCheckbox

I have just deployed the fixes, can you try updating the plugin and see if it works? If you don't mind, I would love to hear some feedback from you about the plugin. Have a great day.

@marcushwz cool, thank you! Works for me.
Feedback: The plugin does exactly what I was searching for and I love that it works project specific! Not sure what else I would want. Great job =)
If you are looking for more features, maybe automatically adding TODO etc from the code, similar to this plugin (I also found yours on Reddit I think), but I don't know how useful that would be.


SWIFT 3 not supported anymore by Apple

We got a message that Apple will not accept apps anymore, which use SWIFT 3 code. Furthermore the newer Xcode versions (forced upgrade by AppStore!!) also don't support SWIFT 3 code anymore.
https://developer.apple.com/documentation/xcode_release_notes/xcode_10_2_release_notes/swift_5_release_notes_for_xcode_10_2
Those files must be upgraded to SWIFT 4.x or 5.
https://swift.org/migration-guide-swift5/
Since we are not experienced with native coding we would appreciate if someone could update the code so that it can again be compiled with XCode 10.3 or higher.

@patrick-stirnemann
Did you find solution? Were you able to update this plugin with SWIFT 4 or 5?

@patrick-stirnemann
Did you find solution? Were you able to update this plugin with SWIFT 4 or 5?

No. We tried to patch the plugin but we were not able to get it running even when the project compiled successfully. In the end we switched to similar plugin from mauron85:
"@mauron85/cordova-plugin-background-geolocation": "^3.1.0",
"cordova-plugin-advanced-http": "^2.2.0",
Note that it is NOT possible to make standard Angular http calls with $http when the app is in background for longer than about 5 minutes (on Android and iOS). The first 5 minutes everything works. So we also had to use another library for that (cordova-plugin-advanced-http). Also - we had to make sure that no promises were used in this case because the promise response was somehow blocked by Angular or the browser (we didn't find out why). So we had to replace all promises by standard JavaScript callback patterns. In the end we got everything to work but it was a "fight".
Good luck!

No. We tried to patch the plugin but we were not able to get it running even when the project compiled successfully. In the end we switched to similar plugin from mauron85:
"@mauron85/cordova-plugin-background-geolocation": "^3.1.0",
"cordova-plugin-advanced-http": "^2.2.0",
Note that it is NOT possible to make standard Angular http calls with $http when the app is in background for longer than about 5 minutes (on Android and iOS). The first 5 minutes everything works. So we also had to use another library for that (cordova-plugin-advanced-http). Also - we had to make sure that no promises were used in this case because the promise response was somehow blocked by Angular or the browser (we didn't find out why). So we had to replace all promises by standard JavaScript callback patterns. In the end we got everything to work but it was a "fight".
Good luck!


Run rspec explicitly in random order for seed output

Codecov Report

Merging #144 (e289c18) into main (7fbd583) will increase coverage by 30.52%.
The diff coverage is n/a.

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

@@             Coverage Diff             @@
##             main     #144       +/-   ##
===========================================
+ Coverage   62.70%   93.23%   +30.52%
===========================================
  Files          67      107       +40
  Lines        2027     2659      +632
===========================================
+ Hits         1271     2479     +1208
+ Misses        756      180      -576

Impacted Files
Coverage Δ

lib/inferno/utils/preset_template_generator.rb
56.25% <0.00%> (ø)

lib/inferno/apps/web/serializers/input.rb
100.00% <0.00%> (ø)

lib/inferno/dsl/http_client_builder.rb
81.81% <0.00%> (ø)

lib/inferno/apps/web/serializers/test_session.rb
100.00% <0.00%> (ø)

...pps/web/controllers/test_sessions/results/index.rb
100.00% <0.00%> (ø)

...b/inferno/apps/web/controllers/test_suites/show.rb
100.00% <0.00%> (ø)

.../inferno/apps/web/controllers/test_runs/destroy.rb
61.90% <0.00%> (ø)

lib/inferno/apps/web/serializers/test_suite.rb
100.00% <0.00%> (ø)

lib/inferno/dsl/fhir_client_builder.rb
88.88% <0.00%> (ø)

...suites/dev_infrastructure_test/serializer_group.rb
100.00% <0.00%> (ø)

... and 74 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 7fbd583...07dd2db. Read the comment docs.


Weekend boundary adjacent hours

There may be an issue with boundary hours.
Even though Friday hour 23 is a week day it is being grouped with Saturday hours 0 and 1 when training the model. It's likely that Friday hour 23 is actually behaving like a weekend because demand is dropping off by then. Maybe try reclassifying Friday hour 23 as a weekend?
Could have similar things going on for other boundary hours.

Competition over. Closing old issues.


[Feature Request] Submixes

Around a month ago now the official GoXLR app got updated with submixes, and I was curious if this feature is intended to be released for the GoXLR Utility? I can't find details anywhere on the existing pages.

So long as your GoXLR has the submix firmware installed, the utility already supports submixes.
To grab the firmware, you'll need to run the updates through the official app, once done you can startup the utility and the submix checkbox will be available.
Thanks.


[BUG] If a user hibernates his computer when a request is performing, AppInsights sends duration of the request equals the duration of the computer's sleep

Description/Screenshot
AppInsights' portal shows us that the duration of some requests equal to a few days. It cannot be true, the maximum timeout for requests we have is 5 minutes.

We investigated the issue and found that it happens when a user ends his web session by hibernating the computer. If the hibernation happens in the middle of performing a request, AppInsights sends a request duration equals the duration of the sleep. For example, here are a few sessions when a user worked with a particular page and finished the sessions by hibernating his computer:

Steps to Reproduce
Needs to run a request which will last at least 30 seconds (to catch the issue easier) and hibernate the computer in the middle of performing the requests. Wake up the computer after 10 minutes and AppInsights will send telemetry that the request lasted 10 minutes.

OS/Browser: Windows 10/Edge 18.18362 (actually it happens in any browser including mobile browsers on iOS and Android)
SDK Version [e.g. 22]: 2.2.2 (latest)
How you initialized the SDK: by using a snippet from the documentation https://docs.microsoft.com/en-us/azure/azure-monitor/app/javascript#snippet-based-setup

Expected behavior
Exclude telemetry which measures the longevity of a computer sleep

For this, I'd recommend implementing a telemetry initializer which you can use to preprocess your telemetry before it is sent over the wire. For example, you can set it to drop any telemetry item (return false) which is of type DependencyData  and has a duration that exceeds some limit


Possible bug in VQ loss gradient

Hey, nice and clean implementation!
I think I might have spotted a bug in the gradient of the commitment loss. It's supposed to pull the embeddings to the inputs that mapped to them. But you're computing it after patching the straight-through gradient onto the embeddings, so they actually now have the gradient of the inputs.
In other words, the current code:
quantize = self.codebook[encoding_indices]
quantize = inputs + jax.lax.stop_gradient(quantize - inputs)
encoding_loss = jnp.mean((jax.lax.stop_gradient(quantize) - inputs)**2)
commit_loss = jnp.mean((quantize - jax.lax.stop_gradient(inputs))**2)

Should have the commitment loss computed before applying the straight-through gradient:
quantize = self.codebook[encoding_indices]
encoding_loss = jnp.mean((jax.lax.stop_gradient(quantize) - inputs)**2)
commit_loss = jnp.mean((quantize - jax.lax.stop_gradient(inputs))**2)
quantize = inputs + jax.lax.stop_gradient(quantize - inputs)

Thank you so much for pointing out the bug! What you said is indeed correct.
I have fixed the bug with commit 38c057e146309e9123e7b94e818b925407d36d10, and have retrained the models.
The sample images in the README has also been updated.
I seem to observe more stable training of the GPT-like sampler after fixing the bug, but no clear perceptual improvement
in the quality of the VQVAE itself.
I will close the issue, but feel free to reopen it if the solution is not satisfactory.


bot detection email issue

Description:
I am following below URL
https://apim.docs.wso2.com/en/latest/learn/api-security/threat-protection/bot-detection/
the document says to configure email.
  extension:
        name: email
        namespace: sink
        properties:
          username<EMAIL_ADDRESS>          address<EMAIL_ADDRESS>          password: ******

but getting error:

@kalevivek11  It seems the issue happens due to bad credentials, could you please check if it's correct?

@Krishanx92 credentials are perfectly fine

@kalevivek11,
Could you please check by turning "ON", the  "Allow less secure apps" option of the gmail account?

yes i have checked
On Thu, Nov 12, 2020 at 5:48 PM hisanhunais<EMAIL_ADDRESS>wrote:

@kalevivek11 https://github.com/kalevivek11,
Could you please check by turning "ON", the "Allow less secure apps"
option of the gmail account?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wso2/product-apim/issues/7918#issuecomment-726043495,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACTSMCFEQUT4MAY2J2MGNATSPPHDFANCNFSM4MJNMO5A
.


Fix being able to shake yourself from tackles

About the PR

Title.
Why / Balance

Bug!
Technical details

Added a check to return if target and user are the same.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

no
Changelog

:cl:

fix: You can no longer shake tackles off yourself

Smug has a fix ready already. REJOICE!!


[eigen3] update to 3.4.1 from 3.4.0

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

The fix-vectorized-reductions-half.patch is no longer necessary since version 3.4.1 incorporates that fix itself.

@microsoft-github-policy-service agree


Update _config.yml file details

Config file for Github pages should match the details of ClickConnect

Duplicate of #23


New landing page for DATSU

New landing page for DATSU under development
https://www.ices.dk/data/tools/Pages/datsu.aspx
Review text document - for main description
Datsu for web.docx

Shorten the description of the different datasets. Also more datasets have separate access points (not only Intercatch).

Shorten the description of the different datasets. Also more datasets have separate access points (not only Intercatch).

@cekv I have updated the text and made a change to the link description, is it ready to publish?

Add DATSU to the sub-menu of 'Data tools' on the left-hand side under the Data Tools menu

Add DATSU to the sub-menu of 'Data tools' on the left-hand side under the Data Tools menu

@neil-ices-dk to action


Don't create the class when loading the dependency

This was causing a React deprecation warning to print anytime a grunt task was run.
We should probably fix that warning at some point, but we also probably shouldn't be creating the React class when the dependency is loaded.

Simple fix, so merging.


[WIP] Japanese translation

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

This pull request is an on-going effort to provide the Japanese translation.

Great work here @yuichielectric! Give me a shout when you consider it ready to merge.

Thanks @MikeMcQuaid!

#701

#provided ✔

I do not understand.

@MikeMcQuaid Thank you for letting me know. That helps a lot.

@yuichielectric
Awesome work!
Although it seems that it's almost done, please tell me if there is something I can help.（fixing typo, reviewing, etc.）

Hi @gotchane, really appreciate your help :heart: Could you review the Japanese translation? Any feedback is welcome!
p.s. Currently the build fails because one of the external links is broken. That is out of scope of this PR, so I’ll wait until that issue would be foxed at master branch.

@yuichielectric
Thanks for your reply 😄
I understood about the failure of build.

Could you review the Japanese translation?

OK, I'll do it!

@gotchane Really appreciate your detailed review ❤ I've accepted most of your suggestion at 1f23875, and replied some of your comments.

Hi @gotchane, sorry for my late reply and thanks a lot for your comments. I'll check them soon.

@gotchane Really appreciate your continuous contribution 😍 I've applied your suggestion. Thanks!

yuichielectric:ja

@yuichielectric Is this ready to 🚢?

Great work here @yuichielectric! Give me a shout when you consider it ready to merge.

謝謝你

Hi @silverskyvicto , the review process is on hold, but I'll restart that soon 👍 If you can help reviewing, that would be a great help!

Thanks @yuichielectric, great work! Live at https://opensource.guide/ja/

Yay, thanks for merging this @mlinksva.
Really appreciated for your reviewing @gotchane !

@yuichielectric
Sorry for not being able to complete the review...
Thanks for publishing!


bring the code in SiteAnalytics.config up to date

Description of problem
The GA code in SiteAnalytics.config is using ga.js, which is Legacy code.
https://developers.google.com/analytics/devguides/collection/gajs
Description of solution (edited on 21 oct, input David Poindexter)
Change the code to analytics.js
https://developers.google.com/analytics/devguides/collection/analyticsjs
Additional context (edited to new solution)
It would perhaps be useful to also include 'anonymize_ip' as a default setting, as this is suggested for GDPR compliance by "various" sources. Feel free to scout around for validness of this suggestion.
https://developers.google.com/analytics/devguides/collection/analyticsjs/ip-anonymization

This to be a good enhancement for a GA default.
Any Google Analytics specialists around who can comment?

Google Analytics (GA) and Google Tag Manager (GTM) are two separate methods.  The newer version of GA is actually documented HERE.

@david-poindexter You're absolutely right, I even have used that one, I'm not sure why I so blindly took the wrong script, my fault.
I will edit my original post.

It appears my original post wasn't so wrong after all...
https://developers.google.com/analytics/devguides/collection/gtagjs/migration

@dieterdtx If I understand this correctly, it is possible to replace the GA code with the gtag code and specify to measure Page Views?

@EPTamminga That's how I am reading it. As I see it gtag is more prepared to use for other features but can track simple page views just as well.

End the end, there are several ways to implement Google Analytics or Google Tag Manager.  We need to make clear distinctions between these methods.  We also need to take into consideration some of the differences in privacy, especially as it relates to GDPR.  My vote would be to provide multiple connectors  (one for each implementation method) or update the one we have to give the admin a choice as to which method they prefer to use.  CC @mikesmeltzer (what are your thoughts on this?)

I agree, in general, with @david-poindexter, but think it would end up being confusing to have multiple GA connectors.  I think adding options to the GA connector would be an easier experience for the administrator, and then having a second connector for GTM would be provide an easier choice to consider.

I would lean on towards this approach as well @bdukes - much clearer to the admin.

We could simplify this even further and future proof ourselves by building a really basic implementation that wasn't a GA or GTM connector but simply a Universal Analytics Connector.
Behind the scenes it could be virtually identical to what we have today just without pulling from the file and from a Portal Setting (this would prevent the current issue of all portals doing the same thing).
It could have simply a multi-line textbox that lets you put the code that you want.
I'm thinking from a pragmatic standpoint that this would satisfy a lot use cases, be a minimal feature to add and stop the various workarounds of manually editing files, etc.
Thoughts?

I like this approach (one caution I considered mentioning with the other approach is making sure we continue to allow customizing the script).
My only concern is that, for instance, Google Tag Manager requests that you add a script block to the head and a <noscript> to the body.  Would we want to introduce the complexity of having multiple inputs that can be injected into multiple places within the markup of the site?

Good point, Brian. There would have to be a fine balance on that as you wouldn't want scope to creep which I could see happening easily. Two fields, InjectInBody and InjectInHeader, would work but you are opening up the door to misuse (which I believe the SiteAnalytics.config is misused today too) by potentially having non-analytics things injected.
If we are looking at the ability to change between Google codes, etc. then we are probably getting into modifying the SiteAnalytics.config file, etc. in code (in the current state) which may open up other challenges like potentially over writing someones custom modifications, etc. unless we change the way it's built or do some tricky business.
Some food for thought... Are we even aware of any extensions (commercial or OSS) that implement a site analytics engine bedsides the Platform itself? I've never come across one, could be an opportunity to deprecate that functionality and do it a bit differently.
I'm not sure where the "happy spot" is. If we continue to include Google specific snippets in the Platform then we need a solution to keep them up to date and something that gives a tracking code that works for as many as possible (there is lots of GA questions on DNN online, people wanting to add specific things, etc..) vs. just giving someone the ability to add whatever code they want from a central location.
My believe is that the user is going to log into GA/GTM anyway to get their tracking ID which is usually in the same screen (or very close) as the complete tracking snippet they tell you to inject in your site..
Other thoughts?

This is the modified file that I am currently using for GTM. It works well, so far. https://pastebin.com/e8qqBnVg

@skamphuis and I created a GTM connector (David already linked). Use cases for me:

DNN instance that supports GA on portal 1 and supports GTM on portal 2.
The snippet has multiple parts so you can  not just add it in the site settings like Hotjar. My customers need something really simple like just pasting the GTM-code not the entire snippet
If people search for GTM, they will type 'GTM'. Not 'GA with option to behave like GTM' :-) So, it is marketing for DNN as well.

Now for the versions of the snippet.
What if we load a stable/current/suppoorted version but we offer to  change that? I know it might be quite a bit of work but this way we offer a version that works but allow innovators to use newer versions.


⚠ Orpheus has degraded performance

In 427b905, Orpheus ($OPS) experienced degraded performance:

HTTP code: 200
Response time: 4639 ms

Resolved: Orpheus performance has improved in 34371e5 after 6 minutes.


Parameterize AES proof over s-box

The AES s-box can be implemented in a wide variety of ways, with different timing properties and side-channel resistances. However, all of these implementations can easily be tested exhaustively, because the s-box has only 256 cases. Therefore, it seems to me that it's low-hanging fruit to write our AES proofs in such a way that any s-box implementation can be plugged in. We can still provide an end-to-end proof that plugs in the LUT or Canright s-box, but that proof would be just a specialization of one that can take any s-box.
Some of the s-box implementations take multiple cycles, so this would involve changing the cipher implementation to accept subroutines -- at least sub_bytes -- as Circuits instead of purely combinational components. Likewise, sub_bytes would need to allow a multi-cycle s-box.

We're prioritizing HMAC development instead, so I'm downgrading the priority of this change.


Support gomega without ginkgo

This PR adds support for cases when using gomega directly, without ginkgo.
For example:
func TestSomething(t *testing.T) {
    g := NewGomegaWithT(t)

    var err error
    g.Expect(err).ToNot(BeNil())
}

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

12 of 65 (18.46%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-6.8%) to 79.935%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

gomegahandler/handler.go
12
65
18.46%

Totals

Change from base Build<PHONE_NUMBER>:
-6.8%

Covered Lines:
490

Relevant Lines:
613

💛  - Coveralls


fix: add --unstable-sql flag to formatter options

This pull request introduces the addition of the --unstable-sql flag, as introduced in https://github.com/denoland/deno/pull/26750, to the options of the formatter.

Before proceeding with the merge, I would like to discuss whether the values used in the deno.json file should also be included, in addition to the flags. Currently, attempting to use these values results in a parsing error:
kotkifotki@laptop deno-project % deno fmt
error: Failed to parse "fmt" configuration

Caused by:
    unknown field `single-quote`, expected one of `useTabs`, `lineWidth`, `indentWidth`, `singleQuote`, `proseWrap`, `semiColons`, `options`, `include`, `exclude`, `files`

I would appreciate feedback on whether including these values is reasonable.

I would like to discuss whether the values used in the deno.json file should also be included, in addition to the flags.

There's a link below the table, which links to this page that describes those deno.json fields.


License clarification

Can someone clarify the license of this project? It appears to have been released under the MIT license for nearly a year, but has recently been updated to name the BSD license.
I would assume the MIT license either still applies or applies to versions of the code up until the new license was added.

Oh, that was added by mistake. It was always considered to be BSD-2-Clause.
Fixed https://github.com/yeoman/stringify-object/commit/6468246134e3cebae245b5d428bcd6e144959526

This should be documented as part of the readme file.


v0.2.0 is broken: niko is not shown on geomap

neofs_net_monitor_netmap{location="Vassilevsky Ostrov/St Petersburg"}
monitor: can't fetch 02a4da4d4dd51a682ab1a58caf0903361d3fd6b56622421d32bd140b37732c5cb8 geoposition, could not get record by provided locode

mona is downgraded to 0.1.0 now

I confirm it works fine with 0.2.1


fix: Links in the footer to the guides

Links in the footer to switch the tabs instead of redirection
Fixes #14

Latest changes will fix #34 as well

These links in the footer appear to scroll or change the guide when clicked on the guide pages. Probably need to add a hashchanged listener https://developer.mozilla.org/en-US/docs/Web/API/Window/hashchange_event

@RoboticMind please revisit


Use response synthesizer in context chat engines

The context chat engines do not handle token overflows well. It requires the user to

limit the token limit on their memory
limit the overall top-k on each retrieval

By using a response synthesizer, we can make this a little more painless.
This needed up being a decent refactor, but I think it simplified a lot too

Hi @logan-markewich, we just noticed this (seemingly) behavioral change by upgrading llama-index to the latest version. In previous versions of this class, there was no "refine" prompt. Is the additional refine prompt essential to preventing token overflows well? The implication for us is that we were unintentionally using the new variant of the class including the "refine" prompt, meaning extra tokens and suboptimal configuration (default refine prompt) in our production system.
As downstream users of the API, it's hard to be aware of behavioral changes like this, especially because llama-index has stopped providing detailed release notes since version v0.11.0). Are the release notes available somewhere else?

@shadyelgewily-slimstock our changelog is up to date. Updating the github release notes is manual process and sometimes gets missed
Before, there was no refine, and the chat engine would error out with a token/context window error. Now it doesn't

Hi @logan-markewich, thanks for your response and I can indeed see the changelog is up to date. I will check there in the future. We also pinned the versions of all llama-index (sub) packages.
I would like to give the feedback that to us, "CondensePlusContext" has a specific meaning: it condenses a prompt and adds context from a knowledge base (potentially with self-managed chat history). I have looked through the code, and "CondensePlusContext" is now "CondensePlusContext" and "compactAndRefine" intermixed: condense a prompt, add context, and then do a (potentially iterative) refine and compact loop. To me, nothing in the name CondensePlusContext indicates that this will happen. So is it really still "CondensePlusContext" now? It seems almost like a compactAndRefine with additional condense prompt.
If we wanted to use compactAndRefine strategy, we would have used that strategy. It's your library and I know it is well before version 1 and we are simply users, so in the end it's your choice. But please do note that from a users' (and also from a potential contributor standpoint) perspective changing behavior to strategies that have particular meaning is very confusing. In this case, the "refine" part could have been optional while achieving the same goals.

CondensePlusContext means condense the chat history, use it for context retrieval, use the context for responding -- the name is still representative of what it's doing imo?
Most people are surprised when token limits are encountered, because query engines automatically handle token limits, so why shouldn't a chat engine? That was the motivation for the change in any case.
The refine step would only be hit in cases where an error would be raised previously, and with the current context window sizes on most LLMs, would be rarely hit

Based on your response I checked what actually happened as the complete prompt (which does not exceed the token limits) enters the CompactAndRefine instance, and I indeed see that only the _text_qa_template is used when everything fits into a single token limit buffer. The refine prompt is only used when the buffer is exceeded.
Thanks for your answer and we resolved the issue on our end.


JDK 11

Hello!
Since we would like to start using Java 11 sooner or later, I've started fixing some of the cleanups and tests that were failing with JDK 9 (#52). I would appreciate if you could take a look at my changes and tell me if what I'm doing makes sense :)
Thank you!

Tests that are still failing:

ShutdownHookCleanUpTest.cleanUpAfterTriggeringLeak ClassLoader has not been garbage collected
I don't know how I can fix this one, since it fails in the Travis build, but works in my local Eclipse. (See #88)

LdapPoolManagerInitiatorTest.firstShouldLeak
SunGCInitiatorTest.firstShouldLeak
ReplaceDOMNormalizerSerializerAbortExceptionCleanUpTest.triggerLeakWithoutCleanup
JavaUtilLoggingLevelCleanUpTest.triggerLeakWithoutCleanup
These tests fail with the message "ClassLoader has been garbage collected, while test is expected to leak", so I would assume the leaks have been fixed.

BeanELResolverCleanUpTest.cleanUpAfterTriggeringLeak
JavaServerFaces2746CleanUpTest.cleanUpAfterTriggeringLeak
These two tests still fail, because I can't get the cleanup to work with Java 11


Reverse order of first two parameters of render function

Can first parameter of render be container? This would help with currying. I am more likely to know the render target first.

This would also make it more compatible with yo-yo. Context: I am trying to make a component lib that can use either yo-yo or lit.

Dupe of #351, which is also closed.
The current ordering is consistent with React and Reactalikes, and reads (to me, of course) like "render this template to this container".
Note, you can in effect do currying with:
const myRender = (template) => render(template, container);


[CentOS7 SharedLibrary] Program received signal SIGSEGV

on exit process
Program received signal SIGSEGV, Segmentation fault.
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff6770d58 in CloseHandle (handle=0x60d0c0) at /root/corert_3253263/src/Native/Runtime/unix/PalRedhawkUnix.cpp:583
583     /root/corert_3253263/src/Native/Runtime/unix/PalRedhawkUnix.cpp: No such file or directory.
(gdb) bt
#0  0x00007ffff6770d58 in CloseHandle (handle=0x60d0c0) at /root/corert_3253263/src/Native/Runtime/unix/PalRedhawkUnix.cpp:583
#1  0x00007ffff672fd25 in PalCloseHandle (arg1=0x60d0c0) at /root/corert_3253263/src/Native/Runtime/Full/../PalRedhawkFunctions.h:14
#2  Thread::Destroy (this=0x6046d0) at /root/corert_3253263/src/Native/Runtime/thread.cpp:375
#3  0x00007ffff6730e23 in ThreadStore::DetachCurrentThread () at /root/corert_3253263/src/Native/Runtime/threadstore.cpp:181
#4  0x00007ffff677196e in TlsDestructionMonitor::~TlsDestructionMonitor (this=<optimized out>) at /root/corert_3253263/src/Native/Runtime/unix/PalRedhawkUnix.cpp:468
#5  0x00007ffff792cb19 in (anonymous namespace)::run (p=<optimized out>) at ../../../../libstdc++-v3/libsupc++/atexit_thread.cc:64
#6  0x00007ffff7024b69 in __run_exit_handlers (status=0, listp=0x7ffff73b16c8 <__exit_funcs>, run_list_atexit=run_list_atexit@entry=true) at exit.c:77
#7  0x00007ffff7024bb7 in __GI_exit (status=<optimized out>) at exit.c:99
#8  0x00007ffff700d3dc in __libc_start_main (main=0x400abd <main>, argc=1, argv=0x7fffffffe528, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>,
    stack_end=0x7fffffffe518) at ../csu/libc-start.c:300
#9  0x00000000004009f9 in _start ()

shared library
using System;
using System.Runtime.InteropServices;

namespace SharedLibrary
{
        public class Class1
        {
                [NativeCallable(EntryPoint = "add", CallingConvention = CallingConvention.StdCall)]
                public static int Add(int a, int b)
                {
                        return a + b;
                }
        }
}

sample(c++) code
#include <stdio.h>
#include <dlfcn.h>

typedef int(*f_add)(int, int);

int main(int argc, char* argv[]) {
	void* handle = dlopen ("SharedLibrary.so", RTLD_LAZY);
        f_add add = (f_add)dlsym(handle, "add");
        printf("%d\r\n", add(5, 10));
	dlclose(handle);
        return 0;
}

CoreRT libraries are not unloadable. There are number of problems that would need to be fixed to make them unload cleanly. https://github.com/dotnet/corert/pull/7011#issuecomment-519403322 lists some of the issues.

Should I ignore it using a signal handler?
Is there any other way?

CoreRT libraries are not unloadable. There are number of problems that would need to be fixed to make them unload cleanly. #7011 (comment) lists some of the issues.

Preventing the library from unloading is the only reliable way.


Manual lists of CVEs to include and exclude

Closes #775.
Adds data/cve_include.json and data/cve_exclude.json which can be used to manually add CVEs for any package releases or ignore CVEs that the osv.dev API returns. Here are what the json formats would look like, using the grafana package as an example (should we include examples like this somewhere to show how to make changes to these two files?)
In data/cve_exclude.json:
{
    "grafana": [
        {
            "id": "CVE-2018-15727",
            "releases": [
                "grafana (Build 2266) - (STABLE)",
                "grafana (Build 2267) - (STABLE)"
            ]
        }
    ]
}

In data/cve_include.json:
{
    "grafana": [
        {
            "id": "CVE-2018-15727",
            "details": "Grafana 2.x, 3.x, and 4.x before 4.6.4 and 5.x before 5.2.3 allows authentication bypass because an attacker can generate a valid \"remember me\" cookie knowing only a username of an LDAP or OAuth user.",
            "severity": "CRITICAL",
            "affected_releases": [
                "grafana (Build 2266) - (STABLE)"
            ]
        }
    ]
}

I also switched the bin/zopen-audit script from using commit hashes to find CVEs to using release timestamps. For example, if we manually add a vulnerability for one release of a package that has the same commit hash as another release of that package, then using release timestamps instead of commit hashes ensures that the audit script is checking for CVEs under the correct release in docs/api/zopen_vulnerability.json.

Excellent! A couple of comments:
Can data/cve_include.json use the community commit sha rather than or in addition to affected_releases
This way we dont need to update data/cve_include.json every time we do a rebuild of the port project.
Are you able to remove zopen_releases.json and zopen_releases_latest.json from the PR?
We update that automatically so it will quickly become outdated, causing the current set of conflicts.

Hmm, do you think it would work to use the package version rather than community commitsha? I was thinking that the version might be easier and also allow for adding CVEs for package releases that were built from tarballs (especially since these are the packages where we don't get CVEs from the osv.dev database). Even once all packages are migrated to git repos, there will still be the old releases without community commitsha, right?
Also, I guess there is never any circumstance where we would want to add a CVE to one specific release of a package? The original reason I chose the release name was so we can target specific releases that use the same package version as other releases, but on second thought that probably isn't necessary since the releases should have the same CVEs if they use the same package version.
I'll remove the releases json files from the PR along with these other changes.

Hmm, do you think it would work to use the package version rather than community commitsha? I was thinking that the version might be easier and also allow for adding CVEs for package releases that were built from tarballs (especially since these are the packages where we don't get CVEs from the osv.dev database). Even once all packages are migrated to git repos, there will still be the old releases without community commitsha, right?
Also, I guess there is never any circumstance where we would want to add a CVE to one specific release of a package? The original reason I chose the release name was so we can target specific releases that use the same package version as other releases, but on second thought that probably isn't necessary since the releases should have the same CVEs if they use the same package version.
I'll remove the releases json files from the PR along with these other changes.

That's a good idea. Let's go with that


Thumbnail Preview in the Feed?

before it was possible too get his done with some small edits i notices in the forum @ tt-rss. but they do not seem too work anymore.. so is there a way to get this working now? with newest tt-rss and this theme ?

+1. Would be a game changer feature in my opinion. @levito, in the point of view of a theme dev, do you think it'd be simple enough to implement this on individual themes, or should there be some changes in ttrss code?
The clean-creader theme seems to have something like this.


drop gulp-dereserve

IE8 have been dead for a long time...
I also added this to match the version that are being tested in the github workflow.
"engines": {
  "node": ">=14"
},

The resulting code can run in some strangely old versions of node.

I just thought it would be best to just simply add a version based on the current LTS.
That way we could write things without having to worry about making any breaking changes.
I suppose that if you install "alasql": "*", or run "npm install alasql" then npm will notice that you could install any version of alasql and if you have a older nodejs version then it would pick a previous release.

Good point

This is a breaking change, so will bundle a few changes before we release v3 now.


update fs docs

General docs in the fs module
show examples
show what can be done with very minimal code

I have added examples and update the docs
except

rmtree needs python 3.7
copy_stat
lchmod  got errornot available on this system
last four functions as there were already docs with example

commit
https://github.com/js-next/js-ng/commit/52bbccf02305b1d0d6500c841b41f90149abac01

can not verify till now waiting https://github.com/threefoldtech/js-sdk/issues/107

works a below


Weekly Competition 11: Square-1 scrambles are bugged

scramble image shows up instread of scramble and image is not found on the server

For some reason for Weekly Competition 1-11 all scrambles ONLY for Square-1 got replaced by their img names and image files are empty, so Square-1 scrambles for first 10 weeks lost :/
But I generated new ones for Weekly Competition 11 and Weekly Competition 12 has the scrambles correctly in the database, so likely something went wrong during migration from #5, but only affected previous scrambles.
Replaces each img text in scramble column on previous scrambles with Scramble missing text :cry:


Create Ruby counter

This plugin tracks your attacks, procs and rate of ruby bolt procs.
There is options to separate Armadyl/Zaryte crossbow special attacks from the normal counter.
You must have sounds enabled for the plugin to track ruby bolt procs.
There is 2 commits by accident. I noticed a small typo when i was making 1st commit and had to replace it. So the newer is the correct one

Okay took few more attempts than just 2 commits, was doing some pointless mistakes during commits and naming


Reconfigure for documentation tooling

Consolidate documentation under a new docs directory
I haven't removed the old readmes. Over time, the content in your READMEs should diverge from the documentation.
Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

ping @aluzzardi I'd like to merge this before you cut the release branch.  Let me know if you have any questions.

I wouldn't include "docs/token.md", in my opinion it's focused for devs, it should stay as a "README.md" only.

@vieux Ok, I can pull that pretty easy.  You'll note I didn't touch y'alls README's :-D anticipating just the statement.

@vieux done

thanks, LGTM
ping @aluzzardi

Just a few minor comments otherwise LGTM

ping @vieux @aluzzardi ready to go! Thanks for your help getting this in place. I really appreciate it.


 We think that paper-masked-input is really cool.
Check out the blog here -> https://vaadin.com/blog/top-five-web-components-for-password-inp-1
It would be awesome if you can drop a quick comment in the comment section below, or on this tweet so that people can know who the mastermind is.
Cheers!

This project id no longer maintained.


Update README.md

correct docker-compose command

Hey, docker compose up is also valid:
https://docs.docker.com/engine/reference/commandline/compose_up/
 $ docker compose up [OPTIONS] [SERVICE...]


[Feature Request] Help put keynote slides into the keynote page

Is your feature request related to a problem?
We should be able to put keynote slides on the keynote page.
Describe the solution you'd like
Add a item to the original json data structure.
Describe alternatives you've considered
Nope.
Additional context
This was once on the website, but it was taken out through these years.
And, the first one to put is the slides for Mariatta's talk https://speakerdeck.com/mariatta/oops-i-became-an-open-source-maintainer
Sample: https://github.com/pycontw/pycon.tw/blob/master/src/static/pycontw-2018/assets/keynotes/info.json

close by #946 & #970

close by #946 & #970


Using --namespace displays namespace does not exist

What steps did you take and what happened:
When starting Octant and providing the namespace at runtime, with no access to list namespaces, Octant displays information but shows a warning that the namespace does not exist.

I'm experiencing the same issue, when user only has access to a single namespace different than the default namespace, and no access to list namespaces.

@GuessWhoSamFoo pointed out that you may be able to work around this issues by also setting the --namespace-list flag. In this caes, you would provide it with the single namespace the user has access to.

Using --namespace-list solved the problem for me testing with the Mac client on a chrome browser.
FYI I was using --namespace all the time before. Without it, no information was displayed at all in any section

If we fail to list namespaces, but namespace is provided, lets set namespace-list to namespace.


Issue with installing extension on macOS

I seem to install TyperTools, but it won't show in my PS 2023.
My process:

open Terminal
cd to directory
sudo chmod +x install_mac.sh
It got me an error with CLRF line endings so I changed them to CL endings
Reopened Terminal and sudo bash ./install_mac.sh
I got all the normal prompts from install_mac and finished the installation
Installation completed.
Open Photoshop and in the menu click the following: [Window] > [Extensions] > [Typer Tools]

It doesn't appear in my PS Legacy extensions window

And I don't see it in extensions folder

My first assumption was that it maybe because my OS is in Spanish. So I tried renaming DESTDIR in install_mac.sh to DESTDIR="${HOME}/Biblioteca/Application Support/Adobe/CEP/extensions/typertools"
But it didn't help either. Please help, I don't know what to do next.

You have to open Photoshop in Rosetta for the Legacy Extensions to be available


Add card-white for white backgrounds in cards

Fix #17117

I'm fine with the background color, though probably use lowercase for the hex value. My only issue with naming it "white" or similar is that it could be (should be?) a variable value.

ah ok. We can certainly use lowercase hex codes @zacechola — I've pushed an update addressing that.
Honestly, the reason I didn't make it a variable value is to avoid stepping on #17069

Let's nix the card-white variation and go with a default background on .card instead.

OK.

@mdo updated to include it on .card

@kkirsche The commit message doesn't make sense anymore

@cvrebert — Changed. Figured you guys would have changed it when merging honestly to be what you wanted but no big deal

Thanks for all the back and forth! <3

Anytime @mdo — just trying to help out :+1:


feat(native-filters): add optional time col to time range

SUMMARY
Adds an optional time column option to the filter config modal to achieve full feature parity with Filter Box.
BEFORE
Currently time ranges can only be applied to the default time column:

AFTER
When no time range is specified, the time column control is hidden:

When a time range is specified, the optional time column control is exposed (when left unpopulated, the default time column is used):

In addition, the sort metric is moved to a row of it's own, and a tooltip is added to the control:

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[x] Has associated issue: closes #14744
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

A couple of points to discuss:

Should the time range picker and time column be in 1 row? In my opinion that would strongly suggest to the user that those 2 fields are closely related. In my opinion, due to rther large vertical distance between time range and time column, they seem like separate values at first glance
Should we prefill time column with the default value rather than leaving it empty? That would make it clearer which column is being used, especially when user doesn't know which temporal column is default

Should the time range picker and time column be in 1 row? That would strongly suggest to the user that those 2 fields are closely related. In my opinion, due to a rather large vertical distance between time range and time column, they seem like separate values at first glance

I initially had it like that, but it seemed to look better when moved to the next row (this is a similar solution as that whoch we've opted for in Explore). But I don't mind having it on the same one if it causes less confusion.

Should we prefill time column with the default value rather than leaving it empty? That would make it clearer which column is being used, especially when user doesn't know which temporal column is default

That's not a bad idea. I left the option so that it would be possible to fall back to the default dttm column in the dataset. But I agree that it would be more explicit to always specify the time column, especially as Filter Box does it like this.

change request: select show only when there are more than 1 time columns

change request: select show only when there are more than 1 time columns

Great idea! Thanks all for the feedback, I'll implement all changes proposed here and request a new round of reviews once done.

One nit: the sort type and sort metric have different widths.

Merging to unblock work on a related issue (will address the test comment in a separate PR)


[DRAFT] MultiDB support

This PR is to add support to mass-migration to multiple DBs.
Assumptions

There's a requirement to be able to force a migration to run on specific databases
By default, the migration can run on any DB
There's always the option to run the migration letting django decide which database to use (so even when allowed_db_aliases = [] the migration can still be run).
Migration records might be stored on a different DB from the migration one.
Migration records are stored on the default database by default.
For the first iteration of this, all the different migrations against different dbs are all listed in the manage view.

Todo (in this ticket):

Update documentation to explain how this all works
Find a better name for _get_queryset_without_namespace

Notes:

This is a breaking change.
This currently doesn't work because of this bug in djangae. defer_with_finalize always use the default DB.

Improvements (different tickets):

Allow to pick a specific DB from manage to filter only those migrations.

Closing in favour of #2


Wrong link in introduction on how to setup BYOK with new storage

In the introduction, we see:
"To learn how to configure customer-managed keys for a new storage account, see Configure customer-managed keys in an Azure key vault for an existing storage account."
It should rather be:
"To learn how to configure customer-managed keys for a new storage account, see Configure customer-managed keys in an Azure key vault for a new storage account."

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bbfc0cf0-c0c0-c7ba-c0a1-a371fb84998e
Version Independent ID: 67ac10b1-902f-b037-50e7-3efdd063aa9b
Content: Configure customer-managed keys for an existing storage account - Azure Storage
Content Source: articles/storage/common/customer-managed-keys-configure-existing-account.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@cweo
Thanks for the question! We are investigating and will update you shortly.

@cweo The PR has been initiated! We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


variable-length-quantity: create test case generator

For #605
Generate two test case arrays, one for encode and
one for decode, from the canonical-data.json.
Update example solution to pass the tests.
The API changed to use slices in order to match
the canonical-data.json test cases.
Retain the old functions but rename to decodeInt and encodeInt
and utilize these as helpers to support the new slice based API.
Update test program to use generated test case arrays.
Put FAIL and PASS in test result output.
Increment testVersion and targetTestVersion in
example solution and test program.

I pushed an update which has a simpler generator.

Well, I did wonder about the 0-length array, whether it was clear enough.  Do you think it is worth adding an errorExpected bool for clarity in the test data ?  Will be fairly easy to do.

I would add it - I'm looking at the example of https://github.com/exercism/xgo/blob/master/exercises/largest-series-product/cases_test.go where we could find out via expected < 0 whether it's an error, but we still add an ok boolean anyway.
However, just to make sure I'm not falling to confirmation bias, I looked for a counterexample, and I found it in Forth: https://github.com/exercism/xgo/blob/master/exercises/forth/cases_test.go#L15 there is a comment there:

// nil slice indicates error expected.

So it looks like there are two possible ways of making it clearer, so we have a choice.

Ok. I chose to add the nil slice comment, and I updated test program to compare against nil as well.

I just remembered something... forgot to check that err is an error? (#275)
Maybe we should only check it in the first few exercises that have error and not worry about it for the rest.

Accidental close earlier. Sorry.
I updated the PR commit without the +build tag and with the var _ error check.


[RealmSwift] Swift 1.2 Compatability

Closes #1744.
This is not ready to merge, at least until we have CI set up with Xcode 6.3.

@jpsim next week, let's work to get CI set up with Xcode 6.3 so we can get this merged in?

CI has completely been migrated to Xcode 6.3.1 and the build is all green.