BUG: Fix itkGPUDemonsRegistrationFilterTest fails

…rse.itk.org/t/itkgpudemonsregistrationfiltertest-fails/3160)

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

Please refer to the following for details.
https://discourse.itk.org/t/itkgpudemonsregistrationfiltertest-fails/3160

Also the first line of the commit message needs to be less than 78 characters long, and for good GitHub display it should be 72 or less characters long.


Initial setup for l10ns

Hi Im trying to run the initial setup of l10ns but facing some issue
Steps i followed

npm install l10ns -g
mkdir test
$ cd test and run the l10ns init
-Created a test.js file (test/test.js)
Added  this code var requireLocalizations = require('path/to/output/all.js'); var l = requireLocalizations('en-US'); var firstname = l('SIGN_UP->FIRSTNAME'); var lastname = l('SIGN_UP->LASTNAME');
var requireLocalizations = require('path/to/output/all.js');
should i create an output folder and add all.js file in it? so the path will be require('output/all.js');
After this i tried running l10ns update command. Nothing seems to be creating

Please help me know if im missing any steps or not doing it properly
Thanks in advance

You must specify your source files in source property in your JSON.
            "source": [
                "Components/**/*.{ts,tsx}",
                "Contents/**/*.{ts,tsx}"
            ]

Be aware to not include the node_modules that would be a perf hit.

Appreciate your immediate reply. Added my json code
{   "defaultProject": "Language",   "projects": {     "Language": {       "timezones": [         "America/Los_Angeles"       ],       "currencies": [         "USD"       ],       "interface": {         "autoOpen": true,         "port": 3001       },       "languages": {         "en-US": "English",         "zh-CN": "Chinese"       },       "defaultLanguage": "en-US",       "programmingLanguage": "javascript",       "store": "",       "output": "localizations/test/output/all.js",       "source": "localizations/test/test.js"     }   } }
My folder structure is as follow
https://drive.google.com/open?id=0BxxhCCYCbjzIWURkVDQ0VmRZLUE

Just notice that the source property should represent all your source js files for whatever application you are building. This will tell the l10ns how to update all translation keys.
And I think it must be an array:
"source": [
    "test.js"
]

You can probably try the following:
"source": [
    "*.js",
    "!node_modules"
]

This will add all js files except those in the node_modules folder.

After adding "source": [     "test.js" ]
still getting this error
SyntaxError: Unexpected end of input     at Object.parse (native)     at /usr/local/lib/node_modules/l10ns/libraries/file.js:248:29     at tryToString (fs.js:414:3)     at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:401:12)
I guess this error is because the resource file en-US.json is empty
@tinganho can you explain following things

Does the resource file(en-US.jso) gets created once i run l10ns update or should i create it.
Once i create it should i define the structure in the file like this
{   "key1": "value1",   "key2": "value2" }
var requireLocalizations = require('path/to/output/all.js'); which path should i metion here...i have added
var requireLocalizations = require('output/all.js');

@tinganho can you please provide a simple working example

Does the resource file(en-US.json) gets created once i run l10ns update or should i create it

L10ns will create it and update it for you.

var requireLocalizations = require('path/to/output/all.js'); which path should i metion here...i have added var requireLocalizations = require('output/all.js');

This is the path to the compiled output.

L10ns will create it and update it for you.
Tried l10ns update but it dint create any resource file(en-US.json)

Please provide a simple example

@godfern can you check out https://github.com/tinganho/l10ns-init?
l10ns update -> updates the storage files.
l10ns compile -> compiles the localisation source files.

@godfern I think you missed to create any l('LOCALIZATION_STRING') calls in your source.
If you add in test.js
l('HELLO_WORLD');

and then run l10ns update.
It will add the the HELLO_WORLD entry in en-US.json.
[
  {
    "files": [
      "test.js"
    ],
    "id": "RopyEG7Lh9",
    "key": "HELLO_WORLD",
    "new": true,
    "timestamp": "2016-04-01T07:43:24.843Z",
    "value": "",
    "variables": []
  }
]

Thanks alot @tinganho for your help...will explore more options with L10ns

@tinganho in order to include it in the html file i did the following thing
`

</body>

`
but seems to be not able to console the output.

It should work since I do a check of the window property:
  if(typeof require === "function" && typeof exports === 'object' && typeof module === 'object') {
    module.exports = requireLocalizations;
  }
  else if (typeof define === "function" && define.amd) {
    define(function() {
      return requireLocalizations;
    });
  }
  else {
    window.requireLocalizations = requireLocalizations;
  }

[
{
"files": [
"test.js"
],
"id": "oG9zjA7GUb",
"key": "SIGN_UP->FIRSTNAME",
"new": false,
"timestamp": "2016-04-01T10:18:32.517Z",
"value": "名",
"variables": []
},
{
"files": [
"test.js"
],
"id": "L74qena7tp",
"key": "SIGN_UP->LASTNAME",
"new": false,
"timestamp": "2016-04-01T10:18:32.517Z",
"value": "",
"variables": []
}
]

This is my zh-CN.json. and when i console.log(l('SIGN_UP->FIRSTNAME')) expects the value(名) to print.
But the following function

function requireLocalizations(language) {
return (function(language) {
return function l(key) {
if(!(language in localizations)) {
return 'LANGUAGE_NOT_IN_LOCALIZATIONS: ' + language;
}
if(!(key in localizations[language])) {
return 'KEY_NOT_IN_LOCALIZATIONS: ' + key;
}
return localizations[language][key].call(localizations[language], arguments[1]);
};
})(language);
};

This line return localizations[language][key].call(localizations[language], arguments[1])  returns an empty string

Oh Sorry had not compiled it. Now getting it.

@tinganho can't i assign a value when i'm creating the key?. If i run l10ns update it creates the json file but value is en empty string.

You have the following commands:
l10ns interface // opens a web interface
l10ns log // look at all latest added keys
l10ns set @1 "something" // set the nth(first) key from the log
l10ns set @1 "something" -l en-US // set the nth(first) key for `en-US`.

Please read the docs.

And there is not default values in l10ns.

Ok @tinganho thought we could set default values. Is there a support for handlebars?

No you have to reference them as variables

Thanks @tinganho for your support


Remove LDA from KNN classification notebook

Will require substantial rewrites of both AM and PM notebooks

Fixed with https://github.com/memphis-iis/datawhys-content-notebooks/commit/f5f84bedb6ded11d96130a8784e82d1e0bee8787


Add TiDB/TiKV/PD version and Spark version supported for each latest major release

Fix #802

@ilovesoup resolved. PTAL


Error: EACCES: permission denied,

Plz help in below error at the time of install the package.
configure error
gyp ERR! stack Error: EACCES: permission denied, mkdir '/home/NodeJsApp/node_modules/ldap-client/build'

Something in that directory path was created with a bad bitmask, I'd bet - look at permissions all the way down to /home/NodeJsApp/node_modules/ldap-client/build and see where you lose rw privs.


Avoid keeping test set in memory unnecessarily

A test set is created whenever "train" is included as an op and test_holdout is specified in the config file. A test set should only be used and kept in memory when "evaluate" is listed as an op in the config file, as the test set will only be evaluated by a call to evaluate in TrainModel (which subsequently calls _evaluate_on_data(self._test_data)). Unfortunately, this test set is kept in memory throughout training, regardless of whether the test set is actually used (i.e. irrespective of whether "evaluate" is included as an op in the config file). I believe that it would be optimal to avoid keeping the test set in memory unnecessarily (i.e. delete it if evaluate() is never going to be called). To test the possible improvements that such a change would make, I ran case 2 from the manuscript examples with "train" as the only op specified in the config file. When I set self._test_data = None in TrainModel, it decreased memory usage by about 10 GB.

There are two simple fixes for this. In either case, we must pass a parameter to the TrainModel indicating if a test set is to be used for evaluation after training.
The first solution would be to add a function that allows us to explicitly call for the destruction of the data associated with testing. It is more complicated than it sounds. I considered only deleting the test data held by TrainModel, but this does little to reduce the memory overhead. As such, it is necessary to delete the sampler's information about the test data as well. To do this, we need to implement many sampler-specific methods for deleting loaded test data.
The second approach is to never create the test set, and not allow saving of a test set without testing the model. This is by far the simplest approach, and doesn't require any modifications of the sampler classes.

I'll make this modification in my next PR - TrainModel will not create self._test_data until the evaluate method is called.


Are automatic app-local dependencies supposed to work reliably for all packages?

This is my first time using CMake so sorry if this is a dumb question.
I took the minimal example of usage with CMake from https://github.com/microsoft/vcpkg/blob/master/docs/examples/installing-and-using-packages.md and replaced sqlite3 with SDL2. I end up with this:
# CMakeLists.txt
cmake_minimum_required(VERSION 3.17)
project(test)

find_package(SDL2 CONFIG REQUIRED)

add_executable(main main.cpp)

target_link_libraries(main PRIVATE SDL2)

// main.cpp
#include <SDL2/SDL.h>
#include <stdio.h>

// An old SDL2 hack
#undef main

int main()
{
	SDL_Init(0);
	return 0;
}

First things first: Neither <SDL2/SDL.h> or <SDL.h> can be found, so I add find_path and target_include_directories:
# CMakeLists.txt
cmake_minimum_required(VERSION 3.17)
project(test)

find_package(SDL2 CONFIG REQUIRED)
find_path(SDL2_INCLUDE_DIR SDL2/SDL.h)

add_executable(main main.cpp)
target_include_directories(main PRIVATE ${SDL2_INCLUDE_DIR})

target_link_libraries(main PRIVATE SDL2)

Now I can compile, but linking fails because SDL2.lib cannot be found. Okay, I add find_library and replace "SDL2" with an absolute path:
# CMakeLists.txt
cmake_minimum_required(VERSION 3.17)
project(test)

find_package(SDL2 CONFIG REQUIRED)
find_path(SDL2_INCLUDE_DIR SDL2/SDL.h)
find_library(SDL2_LIBRARY_DIR SDL2)

add_executable(main main.cpp)
target_include_directories(main PRIVATE ${SDL2_INCLUDE_DIR})
target_link_libraries(main ${SDL2_LIBRARY_DIR})

target_link_libraries(main PRIVATE SDL2)

Now everything compiles, but the program does not run, because the dependencies (SDL2.dll) were not copied to the program's output directory. From what I've seen, vcpkg should do this automatically, and indeed it does for the sqlite3 example. What am I doing wrong here that breaks it?
Another thing I feel I must point out: The exact same behavior occurs with glbinding (a dependency I use in my actual project). I know that's not a terribly large sample size... but it seems to me as if sqlite3 is some kind of "blessed" package that happens to work perfectly out of the box. Would it not be better to provide a more "real-world" example? As a CMake novice, I spent several hours trying to figure out why the behavior between sqlite3 and SDL2 was so different, only to come to the conclusion that the scripts were just written that way.

Here is a simple example project that works: https://github.com/qis/sdl2-test
Open it by right-clicking into the directory and selecting "Open in Visual Studio" or enter the directory in cmd.exe and run make, make run, or make install.

# CMakeLists.txt
cmake_minimum_required(VERSION 3.0)
project(test)

find_package(SDL2 CONFIG REQUIRED)

add_executable(main main.cpp)

target_link_libraries(main PRIVATE  SDL2::SDL2) # also  SDL2::SDL2main?

Using VCPKG does not simplify using CMake. So you still need to know which CMake targets a package export and correctly link it into your code.

@Neumann-A that works perfectly! Thanks! I didn't come across any examples online that used that notation. It's not even in the documentation for target_link_libraries but at least I know about it now.

vcpkg install sdl2 printed it out…

Correct, I misread that because everywhere else I was looking online (including the examples here) don't use that form.


Misbehaving gms on post location

Can't enter post location because of insufficient settings.
To reproduce:

android<9.0
set location settings to only device
try entering post location activity

This is a bug in gms 🤦♂:
https://issuetracker.google.com/issues/118347902
https://issuetracker.google.com/issues/117090563

🤦♂


add sugested identifier in doxygen for return values of functions

add suggested identifier in doxygen for return values of functions

feature available


No such module 'background_locator'

when run app from xcode it says :
No such module 'background_locator'
import UIKit
import Flutter
import GoogleMaps
import background_locator
func registerPlugins(registry: FlutterPluginRegistry) -> () {
if (!registry.hasPlugin("BackgroundLocatorPlugin")) {
GeneratedPluginRegistrant.register(with: registry)
}
}
@UIApplicationMain
@objc class AppDelegate: FlutterAppDelegate {
override func application(
_ application: UIApplication,
didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?
) -> Bool {
GMSServices.provideAPIKey("XXX")
BackgroundLocatorPlugin.setPluginRegistrantCallback(registerPlugins)
GeneratedPluginRegistrant.register(with: self)
return super.application(application, didFinishLaunchingWithOptions: launchOptions)
}
}

Hi @heshesh2010
Thank you for opening an issue;
Try run flutter pub get, if it didn't solve the issue run pod install in iOS directory;
As always clean and build in Xcode might help;


Fix #140 - Prevent docker networks accumulations during test runs.

Type of change

Bugfix

Description
Docker networks were being leaked during the test runs, one network instance per test, when run in TEST_CLUSTER_EXECUTION_MODE=CONTAINER mode.  The change prevents the leaks by closing the instance scoped network on KafkaCluster#close().
Additional Context
This was noticed as I added more tests to the test suite.  All of the sudden, the test suite started to become unstable.
Checklist
Please go through this checklist and make sure all applicable tasks have been done

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

I haven't reproduce it in the past as well, the networks were removed automatically (I'm using podman on Fedora). But TBH it's been a while since I don't run the test


[SPARK-44468][BUILD] Add daily test GA task for branch3.5

What changes were proposed in this pull request?
The pr aims to add daily test GA task for branch3.5.
Why are the changes needed?
After the branch 3.5 is cut out from the master, we need a daily GA task to ensure the smooth release of version 3.5.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manually test.

cc @zhengruifeng @HyukjinKwon FYI

should we also make some changes in build_and_test ?

yeah should also change:

https://github.com/apache/spark/blob/master/.github/workflows/build_and_test.yml#L34 (fix branch-3.5)
https://github.com/apache/spark/blob/master/.github/workflows/build_and_test.yml#L521 (fix master branch)

cc @xuanyuanking who actually were going to fix soon :-).

That will be fixed in https://github.com/apache/spark/pull/42055

Merged to master.


Status of test suite is not updated when all tests pass

Create test as follows:
tests/test_one.py
import unittest

class TestOne(unittest.TestCase):
    def test_dividing(self):
        self.assertEqual(4/2, 2)

    def test_dividing2(self):
        self.assertEqual(4/2, 2)

    def test_dividing21(self):
        self.assertEqual(4/2, 2)

Run all tests, the status of the suite is still unknown

@karrtikr
Please merge latest master, and it doesn't work with parameterized tests:
Here's some sample python code for parameterized tests.
testdata = [
    (datetime(2001, 12, 12), datetime(2001, 12, 11), timedelta(1)),
    (datetime(2001, 12, 11), datetime(2001, 12, 12), timedelta(-1)),
]

@pytest.mark.parametrize("a,b,expected", testdata)
def test_timedistance_v0Hmm(a, b, expected):
    # import time
    time.sleep(1)
    diff = a - b
    assert diff == expected


Turns 90% of vomit into actual vomit

About the PR

Instead of vomiting pure reagents, 90% of them are now turned into a vomit reagent upon vomiting.
This will prevent chemical duping through vomit.
Media

https://github.com/space-wizards/space-station-14/assets/137322659/058bdb51-ffc8-4292-9466-97fa964bd1ea

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

Changelog

:cl: Dygon

tweak: Vomit now only contains 10% of the reagents removed from the chem stream

I believe you can still dupe it. Foams can max out the bloodstream (so 250u) with 1u, then get 10% back, so 25u. Not sure how efficient it'll be, though, so this might kill it regardless.

I believe you can still dupe it. Foams can max out the bloodstream (so 250u) with 1u, then get 10% back, so 25u. Not sure how efficient it'll be, though, so this might kill it regardless.

You would get 12.5 u of chemical per 250u chem stream "puke cycle"
The foam/smoke multiplies by roughly 20x over the full duration of the cloud(10 seconds)


Section Width Defintions need adjusting

Instead of making the section a max width with margin; it will need to be the children wrapper divs that have the max with. This is necessary to support the experience when a section has a background color. The color should extend the full width of the viewport, not the section.
Make sure when updating these CSS definitions that any blocks which override the section width are updated to adjust to the wrapper instead.
Example of incorrect experience:

Invalid bug.


Bumping vectordb to v0.1.3

Did you run the script at ci/bump_versions.sh? This PR seems to be missing the native dependency bumps, which that script should handle.
https://github.com/lancedb/lancedb/blob/391de1644b6caf306d1aea8f1e97ca1eeac8bd47/node/package.json#L74-L77


Bugfix/208

Fixes form input height on Android

Lgtm. Merge it in when you get a chance. @binoy14


WIP: 282-Create InFlightNatsMsg to improve channel buffer usage.

Here is a Draft PR based on our preliminary discussions in #282  as well as observations I've made, at least as far as how to keep this from being a breaking change or involving too much hard to maintain code (Putting a DU-esque object inside NatsMsg<T> results in a -lot- of painful code, especially if fields get added in the future.)
This does, however, result in a possibly (minorly) breaking change I was not initially aware of; the constructor for NatsJSMsg<T> was made public, and one of the optimizations that was made there was to use _context.Connection (since that way, we can get away with not converting to a message and also make NatsJSMsg<T> smaller).
I would not expect this to break anything internally; however if end users have tests that depend on the behavior of a NatsJSMsg's connection being set to something that isn't what the consumer has... it may cause them problems.
We can always just not take the improvement on NatsJSMsg for now if this is too worrisome.
I'll also note, we -can- also DU NatsMsg<T> as well if we wanted, however for a record struct of that style it is a bit more problematic.

Numbers so far are promising... With one exception there is a 5-20% boost on read/write to the channels.
The case where we are suffering badly is ReadAsync where we have to wait on the internal reader; At the moment the custom ChannelReader doesn't have any pooling for it's async operations. I tried to add pooling however... I'm definitely doing something wrong there since alloc numbers didn't change. 😅 Not sure if it's my lack of full understanding of how our pooling works or something I'm doing wrong in the implementation... or if something more thought out is required.
BenchmarkDotNet v0.13.8, Windows 10 (10.0.19045.3803/22H2/2022Update)
Intel Core i7-8750H CPU 2.20GHz (Coffee Lake), 1 CPU, 12 logical and 6 physical cores
.NET SDK 8.0.100
  [Host]   : .NET 6.0.16 (6.0.1623.17311), X64 RyuJIT AVX2
  .NET 8.0 : .NET 8.0.0 (<IP_ADDRESS>103), X64 RyuJIT AVX2

Job=.NET 8.0  Runtime=.NET 8.0

Method
Mean
Error
StdDev
Gen0
Allocated

RunNatsMsgChannel_Sync
61.41 us
0.623 us
0.552 us
-
-

RunInFlightNatsMsgChannel_Sync
55.17 us
0.225 us
0.199 us
-
-

RunNatsMsgChannel_Sync_Pulse
59.99 us
0.315 us
0.263 us
-
-

RunInFlightNatsMsgChannel_Sync_Pulse
54.54 us
0.256 us
0.239 us
-
-

RunNatsMsgChannel_Async_Pulse
87.39 us
0.311 us
0.276 us
-
-

RunInFlightNatsMsgChannel_Async_Pulse
82.23 us
0.368 us
0.326 us
-
-

RunNatsMsgChannel_Async_Pulse_Unhappy
178.55 us
1.279 us
1.196 us
31.2500
147456 B

RunInFlightNatsMsgChannel_Async_Pulse_Unhappy
784.90 us
8.744 us
8.179 us
80.0781
377081 B

RunNatsMsgChannel_Async_Unhappy
119.70 us
1.230 us
1.091 us
0.1221
887 B

RunInFlightNatsMsgChannel_Async_Unhappy
100.23 us
1.514 us
1.416 us
0.2441
1308 B

RunNatsMsgChannel_Async_Happy
117.07 us
0.640 us
0.599 us
-
408 B

RunInFlightNatsMsgChannel_Async_Happy
92.05 us
1.206 us
1.069 us
-
408 B

Fixed the pooling issues, however I'm still seeing a huge perf loss in the 'nightmare' case (even though memory usage is now better than the old Reader.) Need to profile to see what's going on.


Allow to disable AutoSorting

The PropertyGrid now has a button to toggle sort by name - actually it works only for editors derived from MemberwisePropertyEditor, such as ComponentProprertyEditor and ResourcePropertyEditor.
Not sure if the tooltip on the new button is clear enough.
Requires https://github.com/AdamsLair/winforms/pull/2
EDIT: please take care of .csproj files, as I had to reference the modified AdamsLair.WinForms dlls on my machine in order to create this feature.

Changes requested in the WinForms dependency.

Requested AdamsLair.WinForms changes have been implemented. Merging this will need to be done manually and locally, so some fixes can be applied.

Merge and edit AdamsLair.WinForms PR locally, then push it back.
Publish a new AdamsLair.WinForms package
Update Duality dependencies (both VS project and .nuspec)
Merge and edit this PR locally, then push it back.
Release new Duality packages.
Test the new dependency situation as usual with a fresh binary download + install plus some sample packages.

Progress

Merged the AdamsLair.WinForms changes and published a new package.

Immediate ToDo

Update Duality dependencies (both VS project and .nuspec)
Merge and edit this PR locally, then push it back.
Release new Duality packages.
Test the new dependency situation as usual with a fresh binary download + install plus some sample packages.

Done, package release in progress. Check for updates in about 30 minutes.


Error 409: The resource '...' already exists, alreadyExists

I'm using kube-lego to dynamically fetch certs. It attempts to update existing ingress when saving a new cert. In the last month, something seems to have changed: when it tries to update the existing ingress, errors show up on the K8S Ingress (obtained from the Events section when describing the Ingress):
Events:
  FirstSeen     LastSeen        Count   From                            SubObjectPath   Type            Reason  Message
  ---------     --------        -----   ----                            -------------   --------        ------  -------
  10d           13s             14385   {loadbalancer-controller }                      Warning         GCE     googleapi: Error 409: The resource 'projects/foo/global/sslCertificates/k8s-ssl-1-default-qux--c2cd235f2196d4d5' already exists, alreadyExists

Is this a bug in GCE ingress controller?

The only way to recover currently is to delete and re-create the ingress. Recovering from this means downtime in production and likely requires a DNS update, which takes a few minutes to propagate.

Dupe of #609
You should be able to recover by deleting the certificate k8s-ssl-1-... without any downtime.

The certificate cannot be deleted while the ingress is up:
The deletion of the SSL certificate failed. Error: The ssl_certificate resource '...' is already being used by '...'

yes, delete whichever one that isn't used. Version 0.9.3 has the fix for this... Currently being cherrypicked into K8s.

I only see one certificate listed, the one that is currently being used.


Custom skins for NPC Player entities

Describe the problem related to your feature request.
It's common to have custom NPCs on minigame servers with custom skins.
What solution would you like?
There should be an optional component that you can attach to a Player entity to change it's texture.
What alternative(s) have you considered?
None
Additional context
Apparently, this is possible in Bukkit by setting the textures property on GameProfile. I haven't found a good specific code example yet other than this forum post.

A new example would be good, unless we can work it into another large example similar to parkour or ctf.

I'm not really fond of having examples for literally every little thing, I think we can just demonstrate it on an existing example that uses an entity and we could having a mapping of certain feature to certain example if someone search something

He's talking about this component: https://github.com/valence-rs/valence/blob/f8a01b4c7b91120c9ae206d7074f47c9873e0d60/crates/valence_client/src/lib.rs#L494-L506

I was able to get a custom skin on a player entity, but it's unclear to me where the signature comes from, and whether or not it's required.
In the base64, there's a field called signatureRequired that would imply you could make it optional.
{
  "timestamp" :<PHONE_NUMBER>156,
  "profileId" : "e88274eb3f514d60bc1ad95481218303",
  "profileName" : "AnimalTheGamer",
  "signatureRequired" : true,
  "textures" : {
    "SKIN" : {
      "url" : "http://textures.minecraft.net/texture/1de2c83f8f4fb38069f55f52e4f8ee5f0867229ead91b7e79def754b70ea9433",
      "metadata" : {
        "model" : "slim"
      }
    }
  }
}

@dyc3 https://wiki.vg/Mojang_API#UUID_to_Profile_and_Skin.2FCape


svg/html conflict

Howdy,
I recently added svg support to a module. I was already exposing all of HTML, so I added svg support function-by-function until the code compiled. After compilation, to my surprise, no svg was rendered. The reason for this, it turns out, is that the code compiled using the svg function from Html. It feels to me like this kind of substitution should either not compile or work. Any thoughts?
Here is a reduced version of the code that demonstrates this issue:
import Html exposing (..)
import Html.Attributes as HA exposing (..)
import Svg exposing (circle, polygon, rect)
import Svg.Attributes as SA exposing (viewBox, id,x, y, width, height, rx, ry, fill, cx, cy, r, version, points, transform)

main =  svg
    [ SA.id "center", version "1.1", x "0", y "0", viewBox "0 0 323.141 322.95"
    ]
    [ polygon [ fill "#F0AD00", points "161.649,152.782 231.514,82.916 91.783,82.916" ] []
    , polygon [ fill "#7FD13B", points "8.867,0 79.241,70.375 232.213,70.375 161.838,0" ] []
    , rect
        [ fill "#7FD13B", x "192.99", y "107.392", SA.width "107.676", SA.height "108.167"
        , transform "matrix(0.7071 0.7071 -0.7071 0.7071 186.4727 -127.2386)"
        ]
        []
    , polygon [ fill "#60B5CC", points "323.298,143.724 323.298,0 179.573,0" ] []
    , polygon [ fill "#5A6378", points "152.781,161.649 0,8.868 0,314.432" ] []
    , polygon [ fill "#F0AD00", points "255.522,246.655 323.298,314.432 323.298,178.879" ] []
    , polygon [ fill "#60B5CC", points "161.649,170.517 8.869,323.298 314.43,323.298" ] []
    ]
                                                                                ```

In the next release of these libraries, there will be no more Html.svg, only the one in Svg.svg


Fixed pointers bug in PATCH verb in the API

I fixed the bug which gave an error if the data structure which was trying to be updated was a Pointer in the PATCH verb command. Added tests to check for the pointers as well now.
Fixes #8489

@ghodss

After minor comments addressed, this LGTM.

LGTM. Thanks!

@bgrant0607 Can you verify this is ok for 1.0

Yes, we should take this.

@k8s-bot ok to test

@k8s-bot ok to test


As an audio recorder, I would like the export page to have a default (and/or recently used) location to export to in order quickly export repeatedly rather than have to navigate every time

As an audio recorder, I would like the export page to have a default (and/or recently used) location to export to in order quickly export repeatedly rather than have to navigate every time.
J.S.
Be able to confirm whether or not this is possible for SD cards, and if not, prompt user if it could not save.

The default location can be changed, but it cannot be to an SD card. If this is attempted, the app will continue to prompt the user to pick a valid location to save to.
I suppose this raises the question of whether or not this feature should be allowed at all? If the default location cannot be on an SD card, is it useful to change the save location to be outside of the app's folder?


“Exploring Your App” tutorial instructions aren't quite right

Following the instructions on creating the deployment, the resulting pod template has no Port mapping. It looks like this:
> kubectl create deployment kubernetes-bootcamp --image=gcr.io/google-samples/kubernetes-bootcamp:v1
deployment.apps/kubernetes-bootcamp created
> kubectl describe deployments
Name:                   kubernetes-bootcamp
Namespace:              default
CreationTimestamp:      Fri, 04 Nov 2022 19:20:34 -0700
Labels:                 app=kubernetes-bootcamp
Annotations:            deployment.kubernetes.io/revision: 1
Selector:               app=kubernetes-bootcamp
Replicas:               1 desired | 1 updated | 1 total | 1 available | 0 unavailable
StrategyType:           RollingUpdate
MinReadySeconds:        0
RollingUpdateStrategy:  25% max unavailable, 25% max surge
Pod Template:
  Labels:  app=kubernetes-bootcamp
  Containers:
   kubernetes-bootcamp:
    Image:        gcr.io/google-samples/kubernetes-bootcamp:v1
    Port:         <none>
    Host Port:    <none>
    Environment:  <none>
    Mounts:       <none>
  Volumes:        <none>
Conditions:

Without a port mapping, the application cannot be accessed:
> curl http://localhost:8001/api/v1/namespaces/default/pods/$POD_NAME/proxy/
{
  "kind": "Status",
  "apiVersion": "v1",
  "metadata": {},
  "status": "Failure",
  "message": "error trying to reach service: proxy error from <IP_ADDRESS>:6443 while dialing <IP_ADDRESS>:80, code 503: 503 Service Unavailable",
  "reason": "ServiceUnavailable",
  "code": 503
}

But it is running fine:
> kubectl exec $POD_NAME -- curl localhost:8080
Hello Kubernetes bootcamp! | Running on: kubernetes-bootcamp-d9b4bdd78-gj656 | v=1

Additional info:
> kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.3", GitCommit:"434bfd82814af038ad94d62ebe59b133fcb50506", GitTreeState:"clean", BuildDate:"2022-10-12T10:57:26Z", GoVersion:"go1.19.2", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.4+k3s1", GitCommit:"c3f830e9b9ed8a4d9d0e2aa663b4591b923a296e", GitTreeState:"clean", BuildDate:"2022-08-25T03:45:26Z", GoVersion:"go1.18.1", Compiler:"gc", Platform:"linux/amd64"}
> k3d version
k3d version v5.4.6
k3s version v1.24.4-k3s1 (default)

Page: https://kubernetes.io/docs/tutorials/kubernetes-basics/explore/explore-interactive/
/label team/katacoda
I tried to reproduce this.
In 'Step 2 of 4', if you click the first darkgray-boxed command,
(echo -e "\n\n\n\e[92mStarting Proxy. After starting it will not output a response. Please click the first Terminal Tab\n"; kubectl proxy)
a new terminal tab opens, but the command is not executed unless I click the command box once again.

If I proceed with the command not executed properly, then I can meet the error that you countered.

On the other hand, if I click that command box once again in the terminal tab 2 and proceed in the terminal tab 1, then I can access the application.

Yes, it seems weird that:
the guide says that

Click on the command below to automatically open a new terminal and run the proxy:

but actually it opens a new terminal but does not run the command.

/retitle “Exploring Your App” tutorial instructions aren't quite right

/close
Won't fix because we're removing Katacoda.


framebuf driver for embedded graphics

Added a framebuf driver for embedded graphics

Hey @arunvijayshankar, I took a look and this looks really good!
One question - is it possible to move the simulator components to ONLY exist in melpo, not the kernel?
The kernel needs to be no_std, which means it should only hold the embedded-graphics dependency.

also, iit looks like rustfmt needs to be run on this branch to make CI happy.

As a note @arunvijayshankar, you can run cargo fmt --all from the root of the project, and it'll format all of the crates :)


Running tests on multiple browsers using MiniTest::Spec

I'm using the mini-spec-rails gem and when I configure sauce to run on multiple browsers, it only uses the first one.  Do you plan on making this gem compatible with MiniTest? If not, what's the easiest way to get this working?

There are no current plans to extend the Sauce gem for MiniTest; At the moment it's probably a better idea to roll your own integration for MiniTest.


Add guidance for images tab on Case Studies, Speeches and Fatality Notices

Trello ticket link
What
Adds guidance for using the new images tab on Case Studies, Speeches and Fatality Notices for users who have the Preview images update permission.
Why
These content types only support image attachments - not file attachments - and hence do not benefit from the recent additions to guidance for content types which support both image and file attachments.
Screenshots
Case Studies

Fatality Notices

Speeches

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

@dnkrj yea you're right, I've just pushed some changes to correct this


[4.2-04-30-2018][pred-memopts] Rather than asserting on recursive initialization, jus…

…t return false and bail.
Until the beginning of the ownership transition, DI and predictable mem opts
used the same memory use collector. I split them partially since I need to turn
on ownership for predictable mem opts at one time, but also b/c there was a huge
amount of special code that would only trigger if it was used by DI or used by
predictable mem opts. After I did the copy some of the asserts that were needed
for DI remained in the predictable mem opts code. When pred-memopts was only run
in the mandatory pipeline keeping these assertions were ok, but pred-memopts was
recently added to the perf pipeline meaning that it may see code that breaks
these DI invariants (and thus hit this assertion).
We should remove this limitation on predictable-memopts but that would require
some scheduled time to read the code (more than I have to fix this bug = p). So
instead I changed the code to just bail in these cases.
rdar://40032102
(cherry picked from commit 269f8e8d562d7ca6871f01132fc06aa0dd7e6232)

@swift-ci test

@shajrawi Joe, since this that doesn't affect anything, I am going to first go through CCC without the newline and then after it gets through CCC add the newline and do a smoke test. I want to make sure that CCC approval sees that this passes the full tests.

Explanation: Eliminates a DI specific assert that pred-memopts could hit. This assert was a vestigal remnant in pred-memopts from when DI and pred-memopts shared common utility code. We did not hit it before since we only used to run pred-memopts right after DI (implying the invariants would still be true). We are now running it in the perf pipeline as well where these invariants are no longer true. Rather than assert or have pred-memopts handle these new untested cases, we are instead conservative in this PR and just bail.
Scope: This makes the code more conservative and reduces risk by ensuring that we do not go down this untested code path in a non-asserts build.
Radar/SR Issue: rdar://40032102
Risk: Low risk. This just causes the optimizer to not handle these specific cases.
Testing: Compiler regression tests.
Reviewer: @shajrawi

@swift-ci please nominate

Approved by Ted via email. I am think it makes more sense to just merge this now and just add the newline on master.


Fix Comment

Commit for issue #1 Fix Comment
Thanks for this resource by the way, it's helping me a lot in my studies. 👍

@codingDr Awesome thanks for the fix and glad this code has been helpful! :)


[Fixes #1977]Implements the pdf download of complete calendar

Checklist

[x] I have read the Contribution & Best practices Guide.
[x] My branch is up-to-date with the Upstream development branch.
[ ] I have added necessary documentation (if appropriate)

Short description of what this resolves:
Implements the pdf download of the complete calendar.
Server link: https://openeveweb.herokuapp.com/
Test API endpoint: https://raw.githubusercontent.com/fossasia/open-event/master/sample/GoogleIO17
Fixes #1977

@geekyd @shubham3211 @mayank8318 @pradeepgangwar please review.

Great work @agbilotia1998. Can we reduce the size of the generated PDF?

Thank you @geekyd  :)  I tried reducing the size, but since canvas is the only way to handle this it is not possible with that. Since the PDF is generated through the browser scripts only, there is no data consumption while saving the PDF to the machine. Anything else that can be followed up from here?


Multiprocess: Shared Memory not always cleaned up when process crashed

Describe the bug
The MultiprocessRGBPublisher creates several shared memory files to operate. These are prefixed with what we call a namespace, e.g. "zed_top" or "camera", they can be seen by running:
ls -l /dev/shm

Output:
total 9024
-rw------- 1 victor victor 3686400 Dec  8 10:11 camera_depth
-rw------- 1 victor victor 2764800 Dec  8 10:11 camera_depth_image
-rw------- 1 victor victor      24 Dec  8 10:11 camera_depth_image_shape
-rw------- 1 victor victor      16 Dec  8 10:11 camera_depth_shape
-rw------- 1 victor victor       8 Dec  8 10:11 camera_fps
-rw------- 1 victor victor      72 Dec  8 10:11 camera_intrinsics
-rw------- 1 victor victor 2764800 Dec  8 10:11 camera_rgb
-rw------- 1 victor victor      24 Dec  8 10:11 camera_rgb_shape
-rw------- 1 victor victor       8 Dec  8 10:11 camera_timestamp

I've noticed during development when that publisher process is terminated incorrectly, these files are not released/removed.
This is a problem when you try to restart the publisher, because you get:
FileExistsError: [Errno 17] File exists: '/camera_rgb'

Possible solution
We could check whether these SHM files exist on creation of a new publisher and close them. However this could lead you to accidentally closing another camera's shared memory while it's still publishing. I'm not sure wheter this is an issue or not.

One solution would be to allow publishers to .unlink() any already existing shared memory in its namespace. This does however seem to cause processes that read from that memory to freeze. So maybe the best solutions is: detect -> unlink -> raise exception. Then everything should be cleaned for when you start new publishers/receivers.

One solution would be to allow publishers to .unlink() any already existing shared memory in its namespace. This does however seem to cause processes that read from that memory to freeze. So maybe the best solutions is: detect -> unlink -> raise exception. Then everything should be cleaned for when you start new publishers/receivers.

I have for now implemented this solution on the shared_memory_file_exists branch for centrifuge.
Another solution is to catch the exception and set create=False when making the shared memory object.
Will look into this more when we get some time to look at mp in depth.


"computeFault": {"message": "Version 2.79 is not supported by the API. Minimum is 2.1 and maximum is 2.65.", "code": 406

I'm getting this error when call the "select * from openstasck_instasnce", could you help to take a look?
"computeFault": {"message": "Version 2.79 is not supported by the API. Minimum is 2.1 and maximum is 2.65.", "code": 406

Adding more test results with the similar issue:
`select * from openstack_instance
Error: Expected HTTP response code [200 204 300] when accessing [GET http://dev.nova.cloud.example.net:8774/v2.1/9e761808ff1c45ac86577f614fdcccda/servers/detail?all_tenants=true], but got 406 instead
{"computeFault": {"message": "Version 2.79 is not supported by the API. Minimum is 2.1 and maximum is 2.65.", "code": 406}}
Expected HTTP response code [200 204 300] when accessing [GET http://snalab1a.nova.cloud.example.net:8774/v2.1/3e96a8cc70f04795b0f182231eb8e62f/servers/detail?all_tenants=true], but got 406 instead
{"computeFault": {"message": "Version 2.79 is not supported by the API. Minimum is 2.1 and maximum is 2.65.", "code": 406}} (SQLSTATE HV000)
select * from openstack_attachment
Error: Expected HTTP response code [200 204 300] when accessing [GET http://dev.cinder.cloud.example.net:8776/v3/9e761808ff1c45ac86577f614fdcccda/attachments/detail?all_tenants=true&project_id=0037359fe6ed4f66999831b5c952f9c8], but got 406 instead
{"computeFault": {"message": "Version 3.59 is not supported by the API. Minimum is 3.0 and maximum is 3.55.", "code": 406}}
Expected HTTP response code [200 204 300] when accessing [GET http://snalab1a.cinder.cloud.example.net:8776/v3/3e96a8cc70f04795b0f182231eb8e62f/attachments/detail?all_tenants=true&project_id=080450a09b78a164349b8602cb36b660], but got 406 instead
{"computeFault": {"message": "Version 3.59 is not supported by the API. Minimum is 3.0 and maximum is 3.55.", "code": 406}} (SQLSTATE HV000)
select * from openstack_volume
Error: Expected HTTP response code [200 204 300] when accessing [GET http://dev.cinder.cloud.example.net:8776/v3/9e761808ff1c45ac86577f614fdcccda/volumes/detail?all_tenants=true], but got 406 instead
{"computeFault": {"message": "Version 3.59 is not supported by the API. Minimum is 3.0 and maximum is 3.55.", "code": 406}}
Expected HTTP response code [200 204 300] when accessing [GET http://snalab1a.cinder.cloud.example.net:8776/v3/3e96a8cc70f04795b0f182231eb8e62f/volumes/detail?all_tenants=true], but got 406 instead
{"computeFault": {"message": "Version 3.59 is not supported by the API. Minimum is 3.0 and maximum is 3.55.", "code": 406}} (SQLSTATE HV000)
`

This should not be a issue, can be configurable.


GPT-4?

Where's GPT-4 support?

Just so happens I started working on this last weekend. Hopefully I can get a new release out with GPT-4 and GPT-3.5 16K within a couple days!

Terrific. I tried to do it myself, but hit a snag with the wasm file. Will be watching for it. I love this tool!

Yeah the wasm is giving me trouble as well. The particular tiktoken implementation we're using supports GPT4, but claims the model fails to initialize (even though it successfully encodes/decodes during use).
I pretty much have this implemented (with a few extra bonus features I couldn't resist implementing). I'll wrap it up soon and put out a release!

Thanks, Rob!! Can't wait. I'm happy to help contribute to this project as well if you could use a hand.

I appreciate the offer. I just pushed GPT4 support to release/0.8.4. It would be great if you could give it a go and let me know that things are working (or not).
Note that you will have to copy the tiktoken_bg.wasm file from node_modules/... to Resources/tiktoken_bg.wasm (dev-only procedure).
No worries if you can't, I'll do more extensive testing on the weekend.

It's working great. Obviously slower that 3.5, but the responses are smarter.
I found this discussion regarding copying the wasm file on build (albeit with nextjs):
https://github.com/transitive-bullshit/chatgpt-api/issues/570#issuecomment-1700313359
Thanks for getting this out!


fix(Activity Tracker): re-gen service after recent service name change

PR summary

[x] service name is changed from activity tracking to activity tracker, therefore update apidoc as well as sdk
[x] update atracker GA api endpoints, for GA we only support us-east / us-south public and private endpoints

PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

yings-mbp:atrackerv1<EMAIL_ADDRESS>go test
Running Suite: AtrackerV1 Suite
===============================
Random Seed:<PHONE_NUMBER>
Will run 91 of 91 specs

•••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••
Ran 91 of 91 Specs in 3.474 seconds
SUCCESS! -- 91 Passed | 0 Failed | 0 Pending | 0 Skipped
PASS
ok  	github.com/IBM/platform-services-go-sdk/atrackerv1	4.981s

yings-mbp:platform-services-go-sdk<EMAIL_ADDRESS>go test github.com/IBM/platform-services-go-sdk/atrackerv1 -tags="integration" -v
=== RUN   TestAtrackerV1
Running Suite: AtrackerV1 Suite
===============================
Random Seed:<PHONE_NUMBER>
Will run 129 of 129 specs

Service URL: http://localhost:6969
•••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••••
Ran 129 of 129 Specs in 11.779 seconds
SUCCESS! -- 129 Passed | 0 Failed | 0 Pending | 0 Skipped
--- PASS: TestAtrackerV1 (11.80s)
PASS
ok  	github.com/IBM/platform-services-go-sdk/atrackerv1	(cached)

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


Experiment name

Calling mlflow.create_experiment() without passing a name does not generate a new unique name for the experiment but the docstring says a random UUID name will be generated (which it does not). If a name is not set for an experiment the UI throws a 500 and raises the following exception:
TypeError: None has type <class 'NoneType'>, but expected one of: (<class 'bytes'>, <class 'str'>)

This will now generate a human readable experiment name such as:
Experiment 2018-06-10T21:34:21.539164

which may be desirable over something like uuid.uuid4().hex. I had originally only worked with the FileStore then realized it would probably be an issue for the RestStore as well. Currently this creates a name within the store backends but maybe it would be more appropriate to create/assign the name in tracking/__init__.py:create_experiment. Open to discussion to hear what is preferred

Codecov Report

Merging #31 into master will decrease coverage by <.01%.
The diff coverage is 71.42%.

@@            Coverage Diff            @@
##           master     #31      +/-   ##
=========================================
- Coverage    69.6%   69.6%   -0.01%
=========================================
  Files          44      44
  Lines        2527    2533       +6
=========================================
+ Hits         1759    1763       +4
- Misses        768     770       +2

Impacted Files
Coverage Δ

mlflow/store/file_store.py
90.99% <100%> (+0.12%)
:arrow_up:

mlflow/store/rest_store.py
46.06% <50%> (-0.45%)
:arrow_down:

Continue to review full report at Codecov.

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

Hi @MarkAWard : Thank you. This is an excellent catch. There was a bug with docstrings regarding UUID names for experiments. Intension is for experiments to have meaningful names and usually map 1:1 with project initiatives and used by one or more people working on a project.
I will fix both FileStore and RestStore backend to enforce a non-null and non-empty name for create_experiment API as well as enforce that in python mlflow.create_experiment.
Thanks again for flagging this.

#37 : added checks for None argument for create_experiment APIs.


[4.1] InlineCreateOperation - custom message when creating an entry.

it allow relationship ajax field to get custom message base on their response message key.

On success: if message key exists get their message or default message
On Fail:

if laravel errors key exists get their default message
if message key exists get message from message key
or get default message

A new inspection was created.

The inspection completed: No new issues

@tereaksmey123 hello :)
Thank you very much for the PR.
I agree with you that allowing custom messages after creation is something we could/should allow the developer to do.
My concern here is that we are introducing another hidden parameter, with a very common name message and I don't feel safe merging this.
I think the best way we could provide developer messages instead of backpack ones is adding a configs to the to the inline operation definition in field.
What I am thinking is something like:
$this->crud->addField([
....
'inline_create' => [
....
'success_message' => 'developer custom success message',
'error_message' => 'developer custom error message', //this one could be shown along with the errors
'show_validation_error_messages' => true //by default, but developer could set it to false so it would only show his «error_message»
]);

@tabacitu what do you think ?
Best,
Pedro

just got a trip to island and comeback,
For me manage the response message from the API response is the best choice instead of extra parameter.
as we can't guess how many situation developer want to change their message. @pxpm @tabacitu

Hope you had a good trip.
Indeed, working out the message from the API could get you more ganular control.
Maybe a mix of our two proposals could be the best of both worlds. I will let @tabacitu pitch in and give his opinion on this one.
Thanks in advance,
Pedro

This is a nice one! Thanks @tereaksmey123 . I agree with both of you 🤣

I think it's technically better to specify the custom messages inside the InlineCreate operation; APIs should respond with messages depending on what happened, it's easier for everybody;
but I also think it would be better in terms of DX (developer experience) to specify the custom messages as InlineCreate attributes, since normally for InlineCreate you don't customize anything, you just add the trait;

So it's a tricky one 😄
Now, to further complicate things, there's a third option 🤣 Using the SettingsAPI, inside the InlineCreate operation. That would basically allow the developer to:

overwrite all inline create success/error messages, for all CRUDs, inside config/backpack/crud.php, by setting the operations.inlineCreate.successMessage to something;
overwrite the inline create method for just some CRUDs by calling CRUD::set('inlineCreate.sucessMessage', 'bla-bla') inside the setup() method;
customize the messages for one CRUD inside its setupInlineCreate() method by doing CRUD::setOperationSetting('sucessMessage', 'bla-bla')

Now... using the Settings API has one big benefit: it lets us change how messages are sent/received in the future. We can keep the current implementation of echoing the messages in the blade file, inside the JS. We just output the setting (with a fallback). But in the future, if we want to move the message to the InlineCreate operation, we can do that, and it can still use the same string, the same API to change it. So personally I think option 3 is better, using the the SettingsAPI. This might have been exactly what @pxpm had in mind when he said "a mix of your two proposals" 🤣
I think this is a cool feature, and I'd like to have it. But since we're now hard at work at squashing all 4.1 bugs (for ever!!!) I'm going to postpone this until 4.2 (in 1-2 months).
Cheers!

Heads-up, this will be affected by the open-core-split in 4.2. Since this file is moving to backpack/pro the diff will probably be wonky. I still agree to provide this feature, I think it's a good idea, but I still don't have a good HOW for it. Let's leave this issue open and revisit after the 4.2 launch.


Worker_Threads: Unable to find dialect

Hello, I'm running a worker_thread that needs to read a copy of my encrypted database.
Unfortunately when I try to connect to my database I get the following error: UnhandledPromiseRejectionWarning: Error: Unable to find dialect at @journeyapps/sqlcipher.
This is my database setup
sequelize = new Sequelize(null, null, key, {
    dialect: 'sqlite',
    dialectModulePath: '@journeyapps/sqlcipher',
    storage: path,
    logging: false,
    transactionType: 'IMMEDIATE'
  });

I have been reading about this issue and it might be related to Enabling Context-Awareness, here's some info about it Context-Awareness and Example of Context-Awareness Fix
I'm running Electron 8 which runs Node 12.13.0 and I'm using Sequelize 5.21.2 and @journeyapps/sqlcipher 4.0.0
I'm able to import any other module I use on the rest of my electron app but this one, any help would be much appreciated

The current version does not support running with worker_thread.
The upstream node-sqlcipher3 project had some recent changes that could fix the issue. The changes will be included here soon.

This now does seem to work with worker_thread on Node 14 / N-API 6. Earlier versions may appear to work, but are likely to run into crashes.
Electron 10 is based on Node v12.16.3, so worker_threads will not work yet.

Also note that there are more unresolved upstream issues related to worker_threads:
https://github.com/mapbox/node-sqlite3/issues/1381
Best is to track the issues there, and update the project here if there are any fixes.


[Bug] Cognito email validation is too strict (TLD can be more than 3 letters)

Describe the bug
PreSignUpLambda for verifying user emails in AWS Cognito is too strict, and doesn't allow all gTLDs.
To Reproduce

Deploy SWB, go to the AWS Cognito user pool
Create a new user using an email address such as<EMAIL_ADDRESS>This fails PreSignUpLambda due to the regex requiring the last component of the domain to be 2 or 3 characters:
https://github.com/awslabs/service-workbench-on-aws/blob/fec1c92a108f275fa68dffd308acff518c33c43d/main/solution/post-deployment/config/infra/cloudformation.yml#L526-L527
Note there is no error when creating a user in SWB using the same email address, only when adding the user in Cognito.

Expected behavior
All email addresses should be accepted by the Cognito verifier, include gTLDs longer than 3 characters.
Screenshots
Versions (please complete the following information):

v5.0.0

Additional context

@manics -- fix has been merged into mainline from #928. Closing issue.


It's alive!

It's has been a tough week, but as we advanced on our previous post, finally we got a great milestone leading to our first alpha release to raise lower the entry barrier to new NodeOS users: now NodeOS has pre-build images automatically generated both for QEmu, CD live ISOs images (one of the most requested issues, also on Twitter), and disk images (with a writable usersfs partition! permanent storage! yay!:-D )...

...that work on real hardware!!! :-D
To create the CD-live ISOs it was more-or-less easy thanks to the fact that now the root filesystem is read-only (so it can boot from a CD-ROM) and using grub-mkrescue. Problem is, that it requires to define a usersfs partition by hand, or if not the changes would only be done on memory. To allow to make them persistent the best option was to make a partitioned disk image with both a read-only rootfs and a read-write usersfs, and allow to write it to an USB pendrive. At first I though about continue using grub, but after two days I left it: it's difficult to use on disk images, bloated and require to use sudo, so I decided to think different (I did already by using musl instead of glibc) and ended using syslinux. Result? Easy to configure (and similar to the one used on the Raspberry Pi boot process), works natively with disk images, don't need sudo and it was ready in just two hours :-D I'll later from grub-mkrescue to use isolinux so I can use only one boot system.
Oh, and if you want to write it to an USB pendrive, there's also a tool to do this and automatically expand the full disk space... Would be interesting to see this on a hard disk... ;-)
There are some more issues to fix. For example, the errors you see on the previous image are regarding that the image is prepared to mount usersfs from /dev/sda2, while as you can see it has set the USB drive as /dev/sdb (and /dev/sda is my internal hard disk), so I'll need to add support on nodeos-mount-filesystems to mount them based on UUIDs to prevent this. Another more annoying was that the first test on real hardware on my work MacBook didn't booted at all, just to get to the point later that Macs are using EFI instead of BIOS, so I'll need to investigate how to do a disk image that can boot on both systems.
Anyway, now I can say it:

:-P

:O!!!

:+1:


5 Different issues created for the same problem and yet no solution

Hi There
I am facing a problem with mxnet.mod.module API where I try to run my code and the code throws an error which is:
ValueError: Data provided by data_shapes don't match names specified by data_names ((DataDesc[_0_data,(16,) vs ['data'])
Now there are 5 different issues created for the same problem such as:
Issue 7681
Issue 7422
Issue 6801
Issue 6301
Issue 8339
Some of them are open and others have been closed by contributors for the lack of activity. I dont understand why are you guys closing the issue without a concrete solution. You do realize that you are not actually "contributing" anything by closing the issue midway without a solution.
I would like somebody to help with this problem
Thanks

Hi @SrivastavaKshitij. I checked the above five listed issues and they all come from different usage (i.e. different code), so I think it's premature to claim them to be the "same problem". Thus, I don't think an issue that aggregates them would be logical or helpful. If you have evidence to support such claim, please do share. Otherwise, if you have specific error that you need help with, please follow the issue template and provide necessary information. Thank you for your cooperation.

This is an exception the tells you you are using the API incorrectly.
You need to fix your code, but other people can't help you with that if you don't post any useful information like your code and data.
See this nice post explaining how to ask a question: http://www.catb.org/esr/faqs/smart-questions.html

#9460 should have fixed this.

Proposed Labels: "Python", "Bug", "Module",


cannot import name 'MigrateCommand' from 'flask_migrate'

Following the instructions of the README.md I get an error in step "3. Create the tables with python manage.py db create"
Traceback (most recent call last):
  File "manage.py", line 9, in <module>
    from flask_migrate import MigrateCommand
ImportError: cannot import name 'MigrateCommand' from 'flask_migrate' (/home/user/.cache/pypoetry/virtualenvs/spkrepo-vBTx5oS9-py3.7/lib/python3.7/site-packages/flask_migrate/__init__.py)

Doing a little bit of research I found https://stackoverflow.com/questions/68527489/cant-import-migratecommand-from-flask-migrate telling that Flask-Script is no longer supported. See also: https://github.com/miguelgrinberg/Flask-Migrate/issues/407

Fixed in 4640a220dfaef507d05f1ea89065c8c94a9c81ba


[feat]: Improve ring/outline behavior when NOT using keyboard

Feature description
When the keyboard isn't used for navigating the app, the ring/outline should remain hidden when selecting an item from a menu or clicking outside the menu without making a selection. In the video below, you can see that the ring appears even though I didn't use the keyboard to navigate. This forces the user to make an extra click to remove the ring. Unfortunately, I can't find a way to display the ring only on keyboard focus.
Here's an example:
https://github.com/shadcn-ui/ui/assets/43446836/5c8aee85-9b26-4912-8ac1-6d2c37de9cf5
Ideally, it should behave like in libraries such as Catalyst UI, where the ring appears only if the component was initially focused via keyboard. In that case, the ring remains even after selecting an item. However, if the component is selected using a mouse, the ring should never appear.
https://github.com/shadcn-ui/ui/assets/43446836/bace0425-52e9-4b65-aca9-cdb2b9422839
Affected component/components
Select, Dropdown
Additional Context
Additional details here...
Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues and PRs

It turned out to be a radix issue.
https://github.com/radix-ui/primitives/issues/1803


[Virtual Keyboard API] Return promise for Show/Hide calls?

In the TPAC VK API session, there were comments about returning a promise when show/hide is called so the web author knows if the call was successful or not. There are also platform specific restrictions that prevent the VK to come up even after app calls show explicitly to raise the VK. Not sure if we can deterministically determine if the show/hide call was successful, but should we return a promise indicating that the call was successful? Also if there was an error (e.g. no user activation or no editable element in focus?) then what is the author supposed to do in that case?

Tagging few folks to comment on this issue. @whsieh @rniwa @othermaciej @mustaqahmed

This has been copied into the Editing WG issues list as https://github.com/w3c/editing/issues/324


Update additional-scenarios.md - "Attach tokens to outgoing requests"

With the PR, I have simply added a using directive to the "Attach tokens to outgoing requests" sections in the first sample.
I have noticed that it is not possible to call builder.Services.AddHttpClient() without the additional using.
It is not very hard to figure out the missing packages NuGet name but maybe referring to it in the documentation, makes it easier for beginners to get started.
If there is a better way to mention it, feel free to change the request as you want.
Details:
The Call HttpClientFactoryServiceCollectionExtensions.AddHttpClient Method is part of the NuGet Package Microsoft.Extensions.Http.
The Http Extension Package is not used in my vanilla Blazor Project template, so I needed to add it manually.
Feel free to reach out for further questions.
Thanks for your support and the great Documentation!

Hello @abeckDev ... It's in the Microsoft.Extensions.DependencyInjection namespace. Because that namespace is provided by the project template for any WASM app, the topic doesn't call it out. It only lists the ones that the dev is required to add.
https://github.com/dotnet/runtime/blob/master/src/libraries/Microsoft.Extensions.Http/src/DependencyInjection/HttpClientFactoryServiceCollectionExtensions.cs#L11


Add dep update status badges

Hi, does this look like what you had in mind?

@abhisekp I've applied the changes, can you check again please?