Fix UnhandledPromiseRejection warning when logging on an ignored path

Description
A few days ago I ran into a bug where logging via request.log would cause unhandled promise rejection warnings on routes with disabled logging. I traced it down to this line. Basically on an ignored route, current is equal to the nullLogger which doesn't have a levels property.
Steps to reproduce

Add a request.log([level]) statement within a route handler such that level is a valid log level configured within hapi-pino such as 'info', 'warn', 'error', etc.
Disable that route by adding it to ignorePaths
Make a request to that route

I've added a unit test which reproduces this bug.

Coverage increased (+0.05%) to 98.861% when pulling b8549d0fcfce4086ffbade74229969e942d9bc2d on qswitcher:ignore-path-fix into 79ad1b4612ffb0b7b1c99174bdc947f0d8dd83dd on pinojs:master.

Coverage increased (+0.05%) to 98.861% when pulling b8549d0fcfce4086ffbade74229969e942d9bc2d on qswitcher:ignore-path-fix into 79ad1b4612ffb0b7b1c99174bdc947f0d8dd83dd on pinojs:master.


mouse-controls: no update function

Problem: when we dynamically change the pointerlockEnabled property. Rotation is disabled.
Solution: add following lines of code in mouse-controls component-registor.
update: function(){
this.addEventListeners();
}

Thanks!


🤖 chore: update react to Version 16.13.1

This PR updates react to version 16.13.1
Release Notes 16.13.1
Features

Implemented foo #8

Bug fixes

fixed Baz in #7

🎉


add examples for mach-nix and conda

Add two examples on how to create a python environment from:

requirements.txt (using mach-nix 3.0.2)
environment.yml (using mach-nix conda-beta)

I'm quite sure that the checks fail because of an outdated cachix/install-nix-action

In next branch (which is currently under heavy development) all default kernels are packages with poetry2nix, but there is a planned way to use any other *2nix tool for the environment.  Documentation will be added as part of #262 issue and we would love if somebody could test it and write a python kernel using mach-nix.
For now I'm closing this PR, but thank you for putting the effort into it. I hope new API will actually make it easier to extend jupyterWith.


Wrong diagnostic for ##VSO DECODE_PERCENTS

Hello
Following pipeline wrongfully gets tagged for the DECODE_PERCENTS breaking change, but it does not make any use of the ##VSO commands.
steps:
- checkout: none
- task: PowerShell@2
  inputs:
    targetType: inline
    script: write-host "%"

Actual output:
Starting: PowerShell
==============================================================================
Task         : PowerShell
Description  : Run a PowerShell script on Linux, macOS, or Windows
Version      : 2.180.1
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/powershell
==============================================================================
##[warning]%25 detected in ##vso command. In March 2021, the agent command parser will be updated to unescape this to %. To opt out of this behavior, set a job level variable DECODE_PERCENTS to false. Setting to true will force this behavior immediately. More information can be found at https://github.com/microsoft/azure-pipelines-agent/blob/master/docs/design/percentEncoding.md
Generating script.
##[warning]%25 detected in ##vso command. In March 2021, the agent command parser will be updated to unescape this to %. To opt out of this behavior, set a job level variable DECODE_PERCENTS to false. Setting to true will force this behavior immediately. More information can be found at https://github.com/microsoft/azure-pipelines-agent/blob/master/docs/design/percentEncoding.md
========================== Starting Command Output ===========================
/usr/bin/pwsh -NoLogo -NoProfile -NonInteractive -Command . '/home/vsts/work/_temp/47a4cfd4-e7bd-4cbb-8052-5dcc793deb61.ps1'
%
Finishing: PowerShell

@max-zaytsev I believe this would be a bug.
@damccorm Thoughts? (Tagging because you appear to have been the primary committer for these changes)

Hey @fbrosseau thanks for calling this out - I think this is actually sort of right that its spitting out errors, the task itself is setting variables with %25. With that said, we've gotten a lot of false positives from this change where %25 is actually a valid thing that people don't want unescaped. I put up https://github.com/microsoft/azure-pipelines-agent/pull/3241 which should fix the issue

Thanks @damccorm for confirming.
BTW, will this open the door to being able to output multiline errors/warnings? That'd be really great and we would have multiple usecases for it.

Hey @fbrosseau - this would not be related to that. Offhand I'm not sure how we format those, but we do already handle unescaping new line characters (which should be written as %0A) - again, I'm not sure how we format those in our output offhand though. If there's an issue with those, you should file a separate issue for that though, I'd prefer to keep this one focused on the %25 issue

An agent with the fix for this issue has been deployed. Now closing this item, feel free to reopen it if it still actual for you.


NODE_ENV is not recognized as an internal or external command.

When I added NODE_ENV in build and start commands in package.json as you suggested in the last module,
"scripts": {     "dev": "NODE_ENV=development nodemon backend/server.js",     "start": "NODE_ENV=production node backend/server.js",     "build": "npm install && npm install --prefix frontend && npm run build --prefix frontend"   },
I faced this error...
` npm run start

<EMAIL_ADDRESS>start
NODE_ENV=production node backend/server.js

'NODE_ENV' is not recognized as an internal or external command,
operable program or batch file.`

SOLUTION :-
The error occurs because Windows doesn't support setting environment variables in the same way as Unix-based systems. To make this work cross-platform, you can use the cross-env package, which allows you to set environment variables across different operating systems in a consistent way.
Steps to Fix this :
 **1.Install `cross-env`**

 `npm install cross-env`

 **2.Update the `start` and `dev` scripts in your `package.json`**

 ```

"scripts": {
"start": "cross-env NODE_ENV=production node backend/server.js",
"dev":"cross-env NODE_ENV=development nodemon backend/server.js"
}
    **3.Try npm run start again in your terminal**
    The error will be fixed.

Thank you.

SOLUTION :-
The error occurs because Windows doesn't support setting environment variables in the same way as Unix-based systems. To make this work cross-platform, you can use the cross-env package, which allows you to set environment variables across different operating systems in a consistent way.
Steps to Fix:
1. Install cross-env
npm install cross-env
2.Update the start & dev script in your package.json:
"scripts": {   "start": "cross-env NODE_ENV=production node backend/server.js",   "dev": "cross-env NODE_ENV=development nodemon backend/server.js" }
3. Try npm run start in your terminal
npm run start

ok the above solution works just make sure you update both Json.packages and by both I mean the internal and the external package


add AOAI tests for tools, dall-e-3, content filtering, and others

Live run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=3333567&view=results
(tests passing, one failed on a flaky service connection)

/check-enforcer override

/check-enforcer override


Extend QkNNs

Extend QkNNs SchuldQkNN and SimpleHammingQkNN to allow duplicates in the training data.
Both QkNNs use a QAM, which currently cannot load in duplicates. The solution is to load in the indices of the points to some index-qubits and add the points via the QAM's additional_bits and additional_wires option. Currently both QkNNs do not use index-qubits, thus their method get_necessary_wires needs to be updated. Also the variable point_num_to_idx in SimpleHammingQkNN will be obsolete and the index aquired from the index-qubits has to be used in the method calculate_distances.

Merging the branch "feature/extend_qknn" should have fixed this problem!


Use Spring Cache provider for Correlation Map

The Correlation Map is now implemented by a ConcurrentHashMap. The dependency on Caffeine has been removed.


PDF is corrupt

Description
https://arxiv.org/html/2403.03594v1
PDF download is rejected by Adobe Acrobat Pro with this message: "An error exists on this page. Acrobat may not display the page correctly. Please contact the person who created the PDF document to correct the problem."  The PDF display itself is blank, with bounding boxes only around links.
(Optional:) Please add any files, screenshots, or other information here.

(Required) What is this issue most closely related to? Select one.
Other
Internal issue ID
f48bce43-2918-4be9-9fa8-266b4405ef31
Paper URL
https://arxiv.org/html/2403.03594v1
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

I believe this has been independently resolved?
Also, this repository only tracks the quality of the experimental HTML format. Thanks!


[ui action input] Input compressed, crushed when multi fields or wide field

Hi,
All is in the title.
The jsfiddle :
http://jsfiddle.net/sxyvwa4y/4/
Regards

Hi @dawadam,
I think it’s supposed to be used with dropdowns initialized with $('.ui.dropdown').dropdown(); the use case seems too narrow to me to be a part of the core, therefore I’m interpreting it as a usage question. The following is a standard reply.
Thank you for posting, but although it’s a valid usage question, we’ve limited GitHub Issues to bug reports and feature requests, keeping the board more manageable for maintainers; see the contributing guidelines for more information on what kind of posts should find themselves into the GitHub Issues board.
To get answers or feedback that might allow you to repost this issue, please use one of our other support resources:

Gitter chatroom
Official forums
StackOverflow

Sorry,
For me, it was a bug.
Why should I use javascript?
Because without "multiple field" or "wide field", this works very well.
It would seem that this is only a CSS rectification to do.
Regards.

@dawadam You don’t have to, the solution is as easy as overriding one of the default rules (.ui.form select) that produces this behavior:
.ui.dropdown {
  width: initial !important;
}

Open installation_folder/src/site/modules/dropdown.overrides and put it there under a namespace, something like .ui.simple.dropdown, then rebuild the framework. I suppose you could override the mentioned .ui.form select rule in installation_folder/src/site/collections/form.overrides instead, but it’s up to you.

Thanks.


libsqlite3.so dlopen failed with sqlcipher_flutter_libs

libsqlite3.so dlopen failed with sqlcipher_flutter_libs
I found the docs on the native db for Android, and it sais:

Opening libsqlite3.so fails on some Android 6.0.1 devices. This can be fixed by setting android.bundle.enableUncompressedNativeLibs=false in your gradle.properties file. Note that this will increase the disk usage of your app. See this issue for details.

But I want to point out that this only mentions apps that use sqlite3_flutter_libs. I need my db to be encrypted, so I'm using sqlcipher_flutter_libs as suggested here
The issue mentioned in the docs (cited above) is #420. I've followed the actual suggestion to add android.bundle.enableUncompressedNativeLibs=false on my app gradle.properties but still not working. Any suggestions on how can I proceed with this?
I'm using:
dependencies:
  #...
  drift: ^2.11.1
  sqlcipher_flutter_libs: ^0.6.0
  sqlite3: any
  #...
dev_dependencies:
  #...
  drift_dev: ^2.11.1
  #...

 flutter doctor -v
[√] Flutter (Channel stable, 3.13.2, on Microsoft Windows [Version 10.0.22621.2283], locale pt-BR)
    • Flutter version 3.13.2 on channel stable at C:\src\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ff5b5b5fa6 (4 weeks ago), 2023-08-24 08:12:28 -0500
    • Engine revision b20183e040
    • Dart version 3.1.0
    • DevTools version 2.25.0

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at C:\Users\felip_0vh5fa6\AppData\Local\Android\sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: C:\Users\felip_0vh5fa6\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\222.4459.24.2221.9862592\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.4.5)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.4.33403.182
    • Windows 10 SDK version 10.0.19041.0

[√] Android Studio (version 2022.2)
    • Android Studio at C:\Users\felip_0vh5fa6\AppData\Local\JetBrains\Toolbox\apps\AndroidStudio\ch-0\222.4459.24.2221.9862592
    • Flutter plugin version 75.1.1
    • Dart plugin version 222.4582
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-b2043.56-9586694)

[√] VS Code (version 1.82.2)
    • VS Code at C:\Users\felip_0vh5fa6\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.72.0

[√] VS Code (version 1.79.0-insider)
    • VS Code at C:\Users\felip_0vh5fa6\AppData\Local\Programs\Microsoft VS Code Insiders
    • Flutter extension version 3.64.0

[√] Connected device (4 available)
    • SM S918B (mobile) • <IP_ADDRESS>:36831 • android-arm64  • Android 13 (API 33)
    • Windows (desktop) • windows              • windows-x64    • Microsoft Windows [Version 10.0.22621.2283]
    • Chrome (web)      • chrome               • web-javascript • Google Chrome 116.0.5845.188
    • Edge (web)        • edge                 • web-javascript • Microsoft Edge 116.0.1938.81

[√] Network resources
    • All expected network resources are available.

• No issues found!

I saw this on the Encryption docs, I'm doing this but I have no idea why would it be calling the libsqlite3.so. I'm actually using as well:
  firebase_analytics: ^10.4.5
  firebase_app_check: ^0.1.5+2
  firebase_auth: ^4.7.2
  firebase_core: ^2.15.0
  firebase_dynamic_links: ^5.3.5
  firebase_ui_auth: ^1.7.0
  firebase_ui_localizations: ^1.6.0
  firebase_ui_oauth_google: ^1.2.9
  firebase_ui_oauth: ^1.4.9
  firebase_storage: ^11.2.6

I saw a brief example of a crash while using one of the Firebase packages, I'm not 100% sure which one because I skipped the example as it was really brief. I can look for it later in my browser history to update this issue.

Yeah it would be great to see the stack trace to see who is trying to open libsqlite3.so.
Are you using drift with a NativeDatabase.inBackground? That one spawns a background isolate which has "fresh" settings on how sqlite3 is loaded. In that case, you could use the isolateSetup parameter to apply the override on the isolate opening the database as well.

Yeah it would be great to see the stack trace to see who is trying to open libsqlite3.so.
Are you using drift with a NativeDatabase.inBackground? That one spawns a background isolate which has "fresh" settings on how sqlite3 is loaded. In that case, you could use the isolateSetup parameter to apply the override on the isolate opening the database as well.

It worked! Thank you!
Still want to suggest that this information should be on the Encryprion setup page so that other users like me don't make the same mistake.
I ended up trying things that didn't work at all to solve my problem just because this was something I, as a user, never thought about. My drift database always worked without any issues and I didn't even stop to think that my db creation (the method I looked at only once when starting my app) was making the db on another isolate. This is only due to your remarkable work on this package, maybe with a harder API or less complete docs, I would have found my mistake 😂😂😂.

Thanks! And yeah, this should be mentioned in the docs, I'll add it.

I got the same issue currently, it just keep opening the regular libsqlite3.so regardless how I set it up, can anyone help me?

// call this method before using drift
Future<void> setupSqlCipher() async {
  await applyWorkaroundToOpenSqlCipherOnOldAndroidVersions();
  open.overrideFor(OperatingSystem.android, openCipherOnAndroid);
}

bool _debugCheckHasCipher(Database database) {
  // Check that we're actually running with SQLCipher by quering the
  // cipher_version pragma.
  return database.select('PRAGMA cipher_version;').isNotEmpty;
}

/// see: https://github.com/simolus3/drift/blob/develop/examples/encryption/lib/database.dart#L39
QueryExecutor openConnection() {
  return LazyDatabase(() async {
    final dbFolder = await getApplicationDocumentsDirectory();
    // TODO(someone): have a central place to store the db file names when we have more than one dbs
    final file = File(join(dbFolder.path, 'citiesAndTowns.db'));
    if (!file.existsSync()) {
      // TODO(someone): in the future once we decide to ship some preloaded dbs with the app, we dont need to create
      // the db file here. Instead if they are not found, that's a big problem
      await file.create();
    }
    final token = RootIsolateToken.instance;
    return NativeDatabase.createInBackground(
      file,
      isolateSetup: () async {
        BackgroundIsolateBinaryMessenger.ensureInitialized(token!);
        await setupSqlCipher();
        open
          ..overrideFor(OperatingSystem.linux,
              () => DynamicLibrary.open('libsqlcipher.so'))
          ..overrideFor(OperatingSystem.windows,
              () => DynamicLibrary.open('sqlcipher.dll'));
      },
      setup: (db) {
        assert(_debugCheckHasCipher(db),
            'The database was not opened with SQLCipher');

        // Then, apply the key to encrypt the database. Unfortunately, this
        // pragma doesn't seem to support prepared statements so we inline the
        // key.
        // TOOD: very very bad practice! this is for demo purposes only
        const encryptionPassword = 'SECURE PASS';
        final escapedKey = encryptionPassword.replaceAll("'", "''");
        db.execute("pragma key = '$escapedKey'");

        // Test that the key is correct by selecting from a table
        // db.execute('SELECT * FROM Site LIMIT 1');
        final queryResults = db.select('SELECT * FROM au_towns LIMIT 1');
        if (queryResults.isEmpty) {
          throw Exception('Failed to decrypt database');
        }
      },
    );
  });
}

From a quick look the code looks correct to me, is this failing on Linux or Android? In a real Flutter context or when running as a unit test?


[Feature] adding tensor classes annotation for loss functions

Description
followup from this pull request
copy past:
We project on using https://github.com/Tensorclass to represent losses.
The advantage of tensorclass for losses instead of tensordict is that it will help us use all the features of tensordict while preserving type annotation or even completion.
Changes:
Check the out_keys of the loss;
Create a tensorclass with the respective fields;
Type the forward as returning that class (and/or a tensordict)
Add an argument to return the class in the constructor with the False value by default;
Update the docstrings (not done)
Write a little test to check that things work as expected (this test should be new and not parametrized - if we add one more parameter to the existing tests the code will be much longer and harder to follow, and the tests will run for a long time).
Types of changes
What types of changes does your code introduce? Remove all that do not apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of examples)

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[x] I have read the CONTRIBUTION guide (required)
[ ] My change requires a change to the documentation.
[x] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.

@vmoens can you review once, build errors on resource not related to the PR.

@vmoens address most of your comments above, but doctests are failing with below error not caused by this PR changes.
File "/home/sandish/rl/torchrl/objectives/cql.py", line 128, in cql.CQLLoss
Failed example:
    loss = CQLLoss(actor, qvalue)
Exception raised:
    Traceback (most recent call last):
      File "/home/sandish/.conda/envs/torch_rl/lib/python3.9/doctest.py", line 1334, in __run
        exec(compile(example.source, filename, "single",
      File "<doctest cql.CQLLoss[16]>", line 1, in <module>
        loss = CQLLoss(actor, qvalue)
      File "/home/sandish/rl/torchrl/objectives/cql.py", line 321, in __init__
        self.convert_to_functional(
      File "/home/sandish/rl/torchrl/objectives/common.py", line 289, in convert_to_functional
        params.apply(
      File "/home/sandish/.conda/envs/torch_rl/lib/python3.9/site-packages/tensordict/nn/params.py", line 125, in new_func
        out = meth(*args, **kwargs)
      File "/home/sandish/.conda/envs/torch_rl/lib/python3.9/site-packages/tensordict/base.py", line 3824, in apply
        return self._apply_nest(
      File "/home/sandish/.conda/envs/torch_rl/lib/python3.9/site-packages/tensordict/_td.py", line 659, in _apply_nest
        out = TensorDict(
    TypeError: __init__() got an unexpected keyword argument 'filter_empty'

@vmoens made changes based on your review, I still reduction is not being added to the test_cost.py file so all of the failures are related to that.


styles should be configureable

Given that I pass a custom css object to styles, then the CSS rules will override or replace the defaults

react-command-palette is now "themeable"


Create a favicon for Doctor

ok Rushan. The requirements are as follows. We want to visually communicate that its the documentation aspect of a given company's product/service. It would be great if the icon can be versatile in the sense that it has the capability to include and live with other people's icons. Imagine a minio bird with books or something like that. How can we assimilate our docs icon into every logo so that the user will immediately know its company blah's documentation.

Not a must have for V1.0 release.

isn't the current favicon sufficient? - or is it that doctor will have its own?

yeah, doctor will have it's icon and favicon.

Re-opening this right now favicon is not visible.

ETA: 2017-06-07 12:00

ETA: 2017-06-07 12:00

ETA: 2017-06-07 12:00

ETA: 2017-06-07 12:00

ETA: 2017-06-07 12:00

ETA: 2017-06-07 12:00

ETA: 2017-06-07 12:00

ETA: 2017-06-06 12:00

ETA: 2017-06-06 12:00

ETA: 2017-06-06 12:00

ETA: 2017-06-06 12:00

ETA: 2017-06-28 11:15

ETA: 2017-06-07 12:00

ETA: 2017-06-06 12:00

ETA: 2017-06-06 12:00

ETA: 2017-06-06 12:00

@rushenn only one comment is enough :-)

ETA: 2017-06-13 12:00

@harshavardhana Testing Minio Buzz

ETA: 2017-06-06 12:00

@harshavardhana Testing Minio Buzz

Ah didn't knew! @rushenn so we standardizing a format? how exhaustive is the check?

@harshavardhana I'm working on https://github.com/minio/buzz/issues/38 and trying to set ETA for Github issues from Minio Buzz. Currently using a format for testing purposes only.


Disallow arbitrary attributes on convention

We want to start out being restrictive about what attributes are allowed on a convention method. For e.g. it's unclear what to do if the convention is decorated with an Auth attribute. We should have the runtime code throw if the attribute is not recognized.
Related item: https://github.com/aspnet/Mvc/issues/7908

Fixed as part of https://github.com/aspnet/Mvc/commit/94a7c839984b7143ba83ed2173b70ff25c401f0e


feature(ui-ux): add copy receive address toaster

What kind of PR is this?:

/kind feature
What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #2490
Additional comments?:
Developer Checklist:

[x] Read your code changes at least once
[x] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[x] No console errors on web
[x] Tested on Light mode and Dark mode*
[x] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

Here are the updated style


maskDefinitions method was missing in configProvider - issue #135

Completed configProvider tests.

I just noticed that the README.md lacks update - "Inside of ui-options, you can customize these four properties:" should be five properties, located in the Customizing section.
Should i change it and bundle with this PR, make a new PR or let this sort of change for you @lukepfeiffer10?

@RafaelTaranto I would say go ahead and change the readme as well if it's just a simple typo like that.
Also, in the config provider when it merges the default object and the new supplied object, have you considered using angular.merge. I believe it should do a deep copy/merge like is necessary in this case and should make that function even easier.

@lukepfeiffer10 the documentation says that every config should be replaced, except for maskDefinitions. If I use angular.merge won't it also merge the eventsToHandle array?

@RafaelTaranto I just tested it and you are right about that. It also merges it in an odd way in my opinion. For instance, an array of [1,2,3] and an array of [4] will be merged using angular.merge as [4,2,3].
Best to just leave as is I think then.

@lukepfeiffer10 okk, just fixed the readme!

Thanks!


Translations update from Weblate

Translations update from Weblate for Mattermost-test/mattermost-mobile_release-1.32.

@metanerd @wget ???

@enahum Yes, I added this.

@metanerd on thanks... I do see every single line mark as changed because of identation, is that expected? If so, why?

It seems like weblate handles the translation with this indentation style. Do we have a reason to keep the indentation as it was before? Other than the diff check with mmjstool?

No real reason, was just curious

/check-cla


🐛 Allowed data in collections to use specified key

refs #10434
This is finishing off the work started in the PR here:
https://github.com/TryGhost/Ghost/pull/10449

@kirrg001 I'm pretty sure this closes the #10434 issue now?

Looks good yeah. Maybe worth adding a regression test to ensure we won't break both formats again?

@kirrg001 No regression test but updated the unit tests to ensure current functionality, and added comment for removal of "old"/incorrect functionality


Error at WAC connection to EFLOW

Describe the Problem
Cannot connect to IoT Edge via Windows Admin Center
To Reproduce
Steps to reproduce the behavior:

Setup WAC
Deploy EFLOW
Reboot
Getting Error

Connection error
RemoteException: The 'Verify-EflowVm' command was found in the module 'AzureEFLOW', but the module could not be loaded. For more information, run 'Import-Module AzureEFLOW'.

Screenshots

Windows Host OS (please complete the following information):

Edition: IoT Ent 2019 LTSC
Version: 17763.2090
Virtual Machine: Local VM

Additional context
I can connect to VM using Connect-EflowVm command from elevated PowerShell and see that IoT Edge works fine.
Before VM reboot WAC worked fine.

@dmitriyteteruk,
Please try connecting to the PC Connection first on the Windows Admin Center home screen first.  It will prompt you for consent to connect to the device.  Once you have been able to access the PC Connection this should clear you for access to the Azure IoT Edge connection.
Terry Warwick
Microsoft

Thank you Terry! It works!


Add a Transport::TCP class for tcp-based communication.

Problem
CHIP requires the ability to communicate over TCP (for key negotiations, for faster data transfers when supported (e.g. OTA)).
Summary of Changes
Implements a TCP-based transport, with equivalent functionality as the current Transport::UDP.
fixes #2664

@bzbarsky-apple I have addressed all comments. Will double check on freeing paths ... I wish we had some 'smart pointer' like constructs instead of raw points. Tracking buffer usage manually seems error prone to me. Maybe topic for the future.

@bzbarsky-apple I have addressed all comments. Will double check on freeing paths ... I wish we had some 'smart pointer' like constructs instead of raw pointers. Tracking buffer usage manually seems error prone to me. Maybe topic for the future.

SecureSession does indeed Free the passed in buffer, so right now all our paths seem to behave correctly (after my updates to the test).

@saurabhst ?

I wish we had some 'smart pointer' like constructs instead of raw pointers.

Yes, indeed.  I filed https://github.com/project-chip/connectedhomeip/issues/2707

Created #2708 for better active count usage.


No results (suddenly)

We've been using Axocover for about a year and we love it.  Never had any problems.  Today I tried to check our coverage and got no results.  The 'Assemblies' item on the Report tab is not expandable.  I've done alot of google searching and gotten nowhere.  Here is my console output.
Test execution started.
   ___
  /   \
  |   |
  \   /
  /   \
 /     \

/       _
/
|    _______    |
_/       __/
AxoCover Test Runner Console
(c) Peter Major 2017

Executing tests...
| Runner version is 1.1.389.0.
| We are on x86 platform.
| File redirection is enabled for the following files:
| C:\Users\Mark\AppData\Local\Microsoft\VisualStudio\15.0_4bb82853\Extensions\ytwkotxs.xb2\MSTestAdapter\Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.dll
| C:\Users\Mark\AppData\Local\Microsoft\VisualStudio\15.0_4bb82853\Extensions\ytwkotxs.xb2\MSTestAdapter\Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.dll
| C:\Users\Mark\AppData\Local\Microsoft\VisualStudio\15.0_4bb82853\Extensions\ytwkotxs.xb2\MSTestAdapter\Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.Interface.dll
| C:\Users\Mark\AppData\Local\Microsoft\VisualStudio\15.0_4bb82853\Extensions\ytwkotxs.xb2\MSTestAdapter\Microsoft.VisualStudio.TestPlatform.TestFramework.dll
| C:\Users\Mark\AppData\Local\Microsoft\VisualStudio\15.0_4bb82853\Extensions\ytwkotxs.xb2\MSTestAdapter\Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.dll
| Setting up file redirection hooks...
| File redirection hooks are enabled.

Loading assembly from C:\Users\Mark\AppData\Local\Microsoft\VisualStudio\15.0_4bb82853\Extensions\ytwkotxs.xb2\MSTestAdapter\Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.dll...
|| Loaded executor: Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.MSTestExecutor
<< Assembly loaded.
Running executor: Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.MSTestExecutor...
|| Processing tests in: D:\Projects\MyTools\Tools.UnitTests\bin\Debug\Tools.UnitTests.dll...
Passed   MyTools.Tools.UnitTests.LoggerTest.LogTest
....
....
....
Passed   MyTools.UnitTests.XInfoTest.ValueTest
<< Executor finished.
| File redirection rules are cleared.
< Test execution finished.
Shutting down...
Exiting...
Generating coverage report...
Committing...
**No results, this could be for a number of reasons. The most common reasons are:

1) missing PDBs for the assemblies that match the filter please review the
output file and refer to the Usage guide (Usage.rtf) about filters.
2) the profiler may not be registered correctly, please refer to the Usage
guide and the -register switch.**

Test execution finished.
I saw one post here suggesting Debugger.Launch() in a test method to see if the PDB files were in the right folder.  I tried that and there was a list of things that said PDB file not found and it also showed that symbols were not loaded.  I don't know why this would suddenly stop working.  I have AxoCover <IP_ADDRESS> and I have also tried uninstalling and reinstalling it.

Also ... all the tests run and pass perfectly fine

Please check my last post in this thread: https://github.com/axodox/AxoCover/issues/172

This worked for me


error on colab

OSError: cannot identify image file 'data/omniglot/images_background/Japanese_(katakana)/character44/0639_04.png'

I am not sure, but I think Colab has PIL installed by default, whereas Torchmeta (as well as Torchvision) uses Pillow, so that creates some kind of conflict. If you are installing Torchmeta with !pip install torchmeta it should ask you to restart the runtime to use Pillow instead. If you restart the runtime, it should work.


To add a new a new b2c tenant is not clear

In my opinion the documentation on creating a new b2c tenant is not described on this page. However it would be very helpful if it was the case.
An extra tenant needs to be created within the azure active directory, it would be nice if this procedure would also be described.

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

ID: 588fc4d6-0c4d-297d-cfe1-8562728fdc4d
Version Independent ID: cf117c91-90fa-036b-9763-a3c16b398a33
Content: Tutorial - Create an Azure Active Directory B2C tenant
Content Source: articles/active-directory-b2c/tutorial-create-tenant.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@JurrianFahner
Thanks for your feedback! We will investigate and update as appropriate.

@kengaderdus
could you please review this and update as appropriate.

@JurrianFahner
I've delegated this to @kengaderdus, a content author, to review and share their valuable insights.

@JurrianFahner
Thanks for your feedback and patience. That's exactly what we're describing. The Azure AD B2C tenant is created, but you've to switch to the Microsoft Entra tenant (formally Azure AD tenant), which has a subscription. After you switch (if needed) and configure subscription, tenant creation starts in step 4. Let us know if this description helps.

@kengaderdus
Well I solved this riddle just before July 22th.
From the perspective of a new user, it might seem more logical that they can create one or multiple Azure AD B2C instances within their own billable tenant. However, it appears that this is not possible due to the way Azure AD B2C technology is implemented.
The current implementation by Microsoft can be considered counterintuitive, and I believe there is room for improvement in making it clear that an Azure AD B2C instance should always be created in its own dedicated tenant. It would be even more helpful if there were explanations provided regarding the reasons for this requirement. Additionally, if there are other advantages or benefits to using a separate tenant for Azure AD B2C, it would be valuable to share that insight with users.

@JurrianFahner
Thanks for explaining again. Now it's clear what we need to improve. For now, we'll close this issue. Please note that our updates may take some time before they go live #please-close.


DEBUGS

This PR fixes some bugs:

contrasting sessions on firstlevel
diagnostic groupmask
fully specified input for aamod_norm_write

Two questions:

What is the significance of tagging a stream as "diagnostic"?

Your modification to aamod_secondlevel_model includes the first use of spm_print I've seen in the code. Should we start using this instead of saveas or print? What are the advantages?

Ad 1.: Diagnostic streams are not deleted by garbagecollection after execution, and therefore, they are available for report generation.
Ad 2.: I am not sure. I used spm_print, because I created image on the SPM Graphics, and thought it would be saved more 'properly' with using spm_print.

HI Tibor,
So I've been testing your PR -- having some trouble with domain wildcarding. (I realize this is not new to this PR, but forget that it never worked for us).
For example, the current version of aamod_norm_write has a domain='*' which I have to change back to domain='session' to get the module to run.
Is there something we need to set up or change for this?
Cheers,
Mike
PS: Everything else with the PR seems to be working fine...

What is not working? Does it throw you an error? In general, '*' domain is substituted based on the domain of the input or the module providing input.

With domain='*', the scheduler is trying to run aamod_norm_write as 'subject'. Here is currenttask passed in:
`K>> aap.tasklist.currenttask
ans =
struct with fields:
      epiprefix: []
extraparameters: [1×1 struct]
       settings: [1×1 struct]
   inputstreams: [1×1 struct]
  outputstreams: [1×1 struct]
           name: 'aamod_norm_write_00001'
          index: 1
   modulenumber: 8
         domain: 'subject'
       modality: 'MRI'

aamod_norm_write then crashes on line 87 by trying to load stream_epi_outputfrom_aamod_norm_write_00001.txt
That seems wrong. Why is it trying to load an outputfrom stream?

What is the module right before aamod_norm_write? I suspect it is aamod_realign, which is on subject level; which makes aamod_norm_write also subject level. It try to load the epi stream output from the module, because it cannot find the input into it.
I am not sure about your pipeline, but aamod_norm_write is generally preceded by aamod_coreg_extended_2epi (see example), which is on session level.

You are correct. Here is the complete pipeline up to the crash:

aamod_structuralfromnifti
aamod_reorienttomiddle_structural
aamod_epifromnifti</module
aamod_reorienttomiddle_epi
aamod_realign
aamod_coreg_extended
aamod_segment8_multichan
aamod_norm_write

Is this not a valid pipeline? It worked before aamod_norm_write was modified to use domain wildcarding.

The recommended version would be:
aamod_structuralfromnifti
aamod_reorienttomiddle_structural
aamod_epifromnifti
aamod_reorienttomiddle_epi
aamod_realign
aamod_coreg_extended_1
aamod_segment8_multichan
aamod_coreg_extended_2epi
aamod_norm_write

Couldn't we simply add a session domain header alias for aamod_norm_write (call it something like "aamod_norm_write_epi.xml")?
I'm still trying to understand whether domain wildcarding was added for convenience or added to fix a problem.

Yes, we can.
However, domain wildcarding was added to avoid aliases.
Another fix for your issue can be to write a (generic) function to specify/change domain for modules.

However, domain wildcarding was added to avoid aliases.

That's a reasonable design philosophy.
A competing philosophy is to ask: "what makes aa easier to use?" Clearly-named alias headers are self-documenting and help a user -- particularly a new user -- have confidence in designing a proper pipeline. When the behavior of a module is determined by the other modules that appear in a pipeline (or ordering thereof), this is a potential source of confusion. A confused user is an angry user.
M
PS: I'll reiterate this issue is not related to this PR, which as I mentioned passed my testing* aside from the wildcarding issue. So this PR can be merged as far as I am concerned (* the full aa_test suite still doesn't run under OS X, which we have a student looking into, who is currently out for winter break).

I think this discussion would be more suitable for a separate thread, then.
The aim to "make aa easier to use" was mine, too. And there was an agenda to reduce redundancy across modules.
I agree that some of my simplifying changes may not be intuitive for other but me (because I have deveopped them independently); however, the were introduced according to the 'best practice' we had agreed upon.


Is it possible to use it client-side or in-memory?

I would like to use cspell to spellcheck a string in memory entered by a user in a web app.
I could use web-workers.
Would like to know if this is possible with the current cspell implementation.
Thanks

Great question.
It would be easier to setup a service that did the spell checking.
cspell does a few things that are file system based:

Searching for files to check (this can be skipped).
Loading configuration settings based upon the file to check (file type, path etc).
Reading dictionaries necessary for checking the file.

very cool demo and thanks for the reply! Do you have your demo's source code in github?
I was wondering if using something like xterm and webworkers could emulate nodejs FileSystem on the browser so I could use cspell or something similar.


Mindustry

Module Request
Game Information:
Game Title: Mindustry
How to obtain: https://github.com/Anuken/Mindustry/releases
Suppoted OSs: Windows, Linux, Mac
Supports RCON/Console input? Yes
I confirm:

[x] that I have searched for an existing module request for this application.
[x] that the server is an official server distributed by the games publishers/developers.
[x] that the server is available to the general public.
[x] that the server can be run headless (without a GUI).

I successfully got the Mindustry server-release.jar working using a custom Minecraft module with custom startup arguments.

@werethless12 would you mind sharing the modules settings ? i would love to help/test it

@werethless12 would you mind sharing the modules settings ? i would love to help/test it

I don't quite remember how I did it. But I know at the time, the only way I could get it to work was to change the name of the Mindustry jar to Minecraft and let AMP run it like it's Minecraft

Hi !
Is anyone knowing how to use the AMP Template Generator (https://config.getamp.sh/) to create a Mindustry Server instance ?
To many parameter for a beginner :(
Thx !

Read the above comment

Read the above comment

Yes it still work with the minecraft java template but thats wasn't the question :) thx

Open a request in the AMPTemplates repo and we can look into it. This isn't the place for requests discussion anymore.

Ill do that thx !

I've already get a working template going, mostly finished.

I've already get a working template going, mostly finished.

Awesome ! I'll keep in touch with your fork ! 👍

It's in my dev branch


Allow user to specify Trino Connection object conn and cur

A user may want to specify their own Trino connection in a notebook and add this to the Trino configuration. Presently the Trino configuration does not allow specifying the conn and cur attributes.

A user can configure the magic via %config, for example
%config Trino.host='localhost'
%config Trino.port=8080
%config Trino.httpScheme='http'
%config Trino.auth=None
%config Trino.user='the-user'

Are there missing options.

Catalog and schema could be included and would be useful given issues with the Trino python connection not persisting session info (such as USE catalog.schema).

okay sounds good. I'll add these configurations. At least a user will be able to switch default catalogs using this approach. A stop gap until the library properly handles USE.

added two additional configs to the magic
%config Trino.catalog="cat1"
%config Trino.schema="schema2"

fixed


Invalid assignment causing CI failures in skills-client

fix for bug introduced in #604

PR: #613


Update javafx-8-tutorial-id.md

Very nice work! It's live here: http://code.makery.ch/library/javafx-8-tutorial/id/
If you want you could also translate the description sentence in the header. Those are one or two sentences that sometimes appear in search engines and thus makes it easier for people to find what they are looking for.

This is my first time using github and as a translator.
Still dont know what i'm doing, and what part should i translate, need more
learning.
I'll check my translated article one by one after finishing part 7.
On Sat, Jan 3, 2015 at 8:06 PM, Marco Jakob<EMAIL_ADDRESS>wrote:

Very nice work! It's live here:
http://code.makery.ch/library/javafx-8-tutorial/id/
If you want you could also translate the description sentence in the
header. Those are one or two sentences that sometimes appear in search
engines and thus makes it easier for people to find what they are looking
for.
—
Reply to this email directly or view it on GitHub
https://github.com/marcojakob/code.makery.ch/pull/41#issuecomment-68592923
.

Thank you. I've also integrated part 3, 4, and 5 of your translations.
You are doing great the way you work with GitHub and how you send me the pull requests.
In the zip-File you sent me I saw that you also translated the Update to JavaFX 8 - What's New? blog post. This is not part of the tutorial series in the library section but in the blog. I currently don't plan to have blog articles in multiple languages because when people subscribe to the blog via RSS they get confused when there are multiple languages. So, if you think the Indonesian translation of the blog article would be useful for other people you are free to put it on a blog of your own, if you want. Just put it on some website and create a link back to my original article. Then let me know when it's ready and I would be glad to also put a link to your website on my page.

I dont have any blog for, maybe later i'll make one and make back link
to your.
In the future maybe i'll change some part of the translated article if i
or some one else find my translated words weird haha.
All done, and thank you for letting me translated articles in your blog.
Regards
Triawarman
On 05/01/15 17:32, Marco Jakob wrote:

Thank you. I've also integrated part 3, 4, and 5 of your translations.
You are doing great the way you work with GitHub and how you send me
the pull requests.
In the zip-File you sent me I saw that you also translated the Update
to JavaFX 8 - What's New?
http://code.makery.ch/blog/update-to-javafx-8-whats-new/ blog post.
This is not part of the tutorial series in the library section but in
the blog. I currently don't plan to have blog articles in multiple
languages because when people subscribe to the blog via RSS they get
confused when there are multiple languages. So, if you think the
Indonesian translation of the blog article would be useful for other
people you are free to put it on a blog of your own, if you want. Just
put it on some website and create a link back to my original article.
Then let me know when it's ready and I would be glad to also put a
link to your website on my page.
—
Reply to this email directly or view it on GitHub
https://github.com/marcojakob/code.makery.ch/pull/41#issuecomment-68685155.

Wow, thank you very much. I will add the changes probably tomorrow.


Cant edit keywords

I had to edit the .plist
This is what the workflow looks like in alfreds settings.
Seems it's missing most workflows.

@durandom They are under the Script Filter, Open URL and Push Notification, because workflow is automatically generated. I don't use any offset to add some spaces between the set of items. Just drag them out.

Ahh, thats it. Maybe add some offset?
Otherwise, thanks for this great workflow and fast response. Really appreciated 🙇

@durandom I will prepare a separate fix. I will update info.plist and it will contain a data dictionary with position of elements and you don't need to do that manually.

@durandom Please update to the latest version https://github.com/mdreizin/chrome-bookmarks-alfred-workflow/releases/tag/0.1.6


Connection state is coming as closed in First itteration when selecting table data

Issue
Our IBM EntityFramework Core Provider is based MS EF Core 1.0.1 . We have written MVC EF Application using IBM EntityFramework Core provider by following below link:
https://docs.microsoft.com/en-us/ef/core/get-started/aspnetcore/existing-db
It just selects data from table. When we run application we are getting error:
AggregateException: An error occurred while writing to logger(s). (Invalid operation. The connection is closed.)
On Debugging, we figured out that From MS EF layer to IBM EF layer we have 3 times call for
 public Db2QuerySqlGenerator(
            [NotNull] IRelationalCommandBuilderFactory relationalCommandBuilderFactory,
            [NotNull]ISqlGenerationHelper sqlGenerationHelper,
            [NotNull] IParameterNameGeneratorFactory parameterNameGeneratorFactory,
            [NotNull] IRelationalTypeMapper relationalTypeMapper,
            [NotNull] SelectExpression selectExpression,
                      IRelationalConnection connection)
            : base(relationalCommandBuilderFactory,
                sqlGenerationHelper,
                parameterNameGeneratorFactory,
                relationalTypeMapper,
                selectExpression)

And in first iteration Connection state is coming as closed. Please note Connection Object is created but its state is closed.
Since We have one common provider for all DB2 Servers, We require to get database server type. We already had discussion about the implementation here https://github.com/aspnet/EntityFramework/issues/2636
In user\test application scenario Connection state is closed and it throws error.
If we change our user\test application context class from
 public class TestContext: DbContext
    {

        public TestContext(DbContextOptions<TestContext> options) : base(options) { }
........
.......
}

to
 public class TestContext: DbContext
    {

        public TestContext() : base() { }
       protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
                        optionsBuilder.UseDb2(ConnectionString);
        }
...
...
}

Application works perfectly because Connection state is coming as Opened.
Why is Connection state coming as closed in First call of method Db2QuerySqlGenerator(....) from MS layer when we have context class implementation like below:
public class TestContext: DbContext
    {

        public TestContext(DbContextOptions<TestContext> options) : base(options) { }
........
.......
}

Regards,
Archana

@archanasoni Can you post the code that creates the DbContextOptions object passed to the constructor and also the implementation of the UseDb2 method?

@ajcvickers, We are not creating DbContextOptions object any where. we register the service and as per the article dependency injection takes care:
https://docs.microsoft.com/en-us/ef/core/get-started/aspnetcore/existing-db
"
You'll notice that the controller takes a BloggingContext as a constructor parameter. ASP.NET dependency injection will take care of passing an instance of BloggingContext into your controller."
Implementation for UseDb2 is below:
public static DbContextOptionsBuilder UseDb2 (
[NotNull] this DbContextOptionsBuilder optionsBuilder ,
[NotNull] string connectionString,
[CanBeNull] Action Db2OptionsAction = null )
{
Check.NotNull(optionsBuilder, nameof(optionsBuilder));
Check.NotEmpty(connectionString, nameof(connectionString));
        var extension = GetOrCreateExtension(optionsBuilder);

        extension.ConnectionString = connectionString;

        ((IDbContextOptionsBuilderInfrastructure)optionsBuilder).AddOrUpdateExtension(extension);

        Db2OptionsAction?.Invoke(new Db2DbContextOptionsBuilder(optionsBuilder));

        return optionsBuilder;

    }

@archanasoni In that case it could you show the code that sets up the service provider and calls AddDbContext? Also, are you using this in a standard web app with MVC and having the context injected into the controller? If not, how are you managing the service provider and creating scopes? Can you show code for that?

@ajcvickers, We are using standard Web App with MVC. Controller and DBContext  are defined, same as what is described in article. Database and object already exist.
 There should be no difference in behavior between using OnConfiguring and AddDbContext. Can you post the full code for your implementation of the RelationalOptionsExtension class and the class that contains the UseDb2 method?  Also, can you check that your implementation of QuerySqlGeneratorFactoryBase is registered as scoped?

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.

Still we have same issue and we are using EF Core 1.0.1.
To proceed further, Which path would you prefer- Either we can provide the data points which were asked Or we can also share some sample repro with Nuget pkg. This sample repro would require DB2 database to connect and repro the issue. Do you have any existing DB2 server ?

@archanasoni please open a new issue. Including some code we can run is great. Here is the info on the DB2 provider https://docs.microsoft.com/en-us/ef/core/providers/ibm/.

Opened issue #8125  to discuss further.


copy/paste problem in EN doc Setup Kubeadm

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
i think there was a copy / paste problem during writing
Proposed Solution:
If you see Pods in the RunContainerError, CrashLoopBackOff or Error state after deploying the network solution and nothing happens to coredns (or kube-dns), it’s very likely that the Pod Network solution and nothing happens to the DNS server, it’s very likely that the Pod Network solution that you installed is somehow broken.
Page to Update:
https://kubernetes.io/docs/setup/independent/troubleshooting-kubeadm/#pods-in-runcontainererror-crashloopbackoff-or-error-state

/language en
/sig docs


Alternate presentation of multi-cluster exec plane names

Signed-off-by: Meg McRoberts<EMAIL_ADDRESS>This is an alternative presentation to https://github.com/keptn/keptn.github.io/pull/1382 , which fixes https://github.com/keptn/keptn/issues/8576 .  It makes adjusting the exec plane name a step in the process of implementing a multi-cluster environment rather than an afterthought.
The original draft had the following issues.  Giovanni responded and I believe these are fixed:

I made up the examples without much knowledge -- please suggest better examples
The text says that this is so the distributor can handle event subscriptions correctly, then says that the nameOverride value is available only if the integration uses the distributor.  This does not sound right.
What happens if someone sets different values for nameOverride and K8S_DEPLOYMENT_NAME?
Should we add something to the Troubleshooting section to describe but it looks like if you don't handle the exec plan names correctly?

Once we get the text correct here, we need to implement these changes in the 0.19.x tree as well.

Thanks, @thisthat !  I think I incorporated your information, also a bit from the original issue, and this might be okay.  I also added a little section to "Troubleshooting" about this issue.  It isn't wonderful but perhaps good enough; we can always improve it later if necessary.

The documentation around exec plane names definitely needs to get added to the upgrade guide. Anyone with a multi-cluster setup running through the upgrade steps will end up with a broken situation, where in the worst-case the helm-service in production picks up the events for a dev stage.


Seahash has poor performance with integer keys

My performance profile has ~4% spent in <seahash::stream::SeaHasher as core::hash::Hasher>::write and ~2% spent in <seahash::stream::SeaHasher as core::hash::Hasher>::finish, despite using a HashMap with a simple u64 key.
With a different hasher such as AHash or FxHash, this whole process consists of just a handful of instructions (~5) which are inlined directly into the hash table lookup function.
Changing the hasher would be a breaking change in the rkyv format, but perhaps this could be considered under a feature flag?

I think that's sensible to provide as a feature flag. Do you know whether AHash or FxHash are portable across platforms? The primary motivation for choosing seahash was its strong guarantee to get the same hash from the same data on all platforms.

Unfortunately not. FxHash produces different results on 32-bit vs 64-bit platforms, and aHash uses AES instructions when available which produces a completely different hash.
Perhaps the 64-bit variant of FxHash could be directly included in rkyv? HashDoS guarantees aren't needed since perfect hashing is used (though it might still hurt serialization time).

As long as FxHasher64 always gives the same results regardless of the target platform, it's fine to include.

Wrote an implementation of fxhash with accommodations that should produce the same hash on all platforms in be38ebe926bf8bb4898af472ba7ce90f72df74bd


[GHSA-hfxp-p695-629x] abomonation transmutes &T to and from &[u8] without sufficient constraints

Updates

Affected products

Comments
this advisory is a duplicate of https://github.com/advisories/GHSA-5vwc-r48g-wj6c

Hey there. I'm seeing just a timestamp update on this PR. Can you expand on what should be changed?

Hi @darakian!
it seems that https://github.com/advisories/GHSA-hfxp-p695-629x is a duplicate https://github.com/advisories/GHSA-5vwc-r48g-wj6c
isn't it?
thanks!

Indeed it does look like a dupe! Thanks for raising this. At the moment we have some technical blockers preventing us from de-duping, but I've added this to our list to do once we can. If you're good with that we can close out the issue for now.

@darakian sure, us your wish, let's close it.


[Icon Request]: License folder

Housekeeping

[X] I'm sure this issue is not a duplicate

Icon Type

[ ] File Icon
[X] Folder Icon

Icon Type

[ ] Extension
[X] Filename
[ ] Language ID

Original Icon
This icon should be a folder icon with a small license icon in the lower right corner,
Like this:

But I don't know how to make nice icon in SVG format 😞. please help me.
Icon Information
Icon for folder containing third-party licenses.
The name of folder could be "license" or "licenses".
More Information:
No response

Why did you close it?

Because multiple licenses can be contained in one file...
MIT License

Copyright (c) 2022 somebody
Copyright (c) 2021 another
Copyright (c) 2020 ...

Ok, makes sense! 👍🏻


Updated least privileged role for PTA

Updated least privileged role for PTA

https://docs.microsoft.com/en-us/azure/active-directory/hybrid/how-to-connect-pta-quick-start#in-the-azure-active-directory-admin-center

GitHub Issue: https://github.com/MicrosoftDocs/azure-docs/issues/90334
cc: @rolyon

@JamesTran-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @JamesTran-MSFT
Sent to team to review

#label:"aq-pr-triaged"

Hi @JamesTran-MSFT
Leaving doc as is now. Will update if team decides to update.
thanks

#please-close


opencv debug version for CameraWithCVAndCalibration

Hi authors,Thanks a lot for the great repository. I have successfully built the CameraWithCVAndCalibration sample in debug version. But the it seems that the program lack some dll after deploy.

see #96

see #96


Using Dream with effects

Consider the following snippet (it's not a strictly minimal example; I hope that's fine):
open Dream
open Effect
open Effect.Deep
open Ppx_yojson_conv_lib.Yojson_conv.Primitives

type user_object = {
  email : string;
  token: string;
  username: string;
  bio: string;
  image: string option;
} [@@deriving yojson]

type login_user_object = {
  email : string;
  password: string;
} [@@deriving yojson]

type login_object = {
  user: login_user_object;
} [@@deriving yojson]

type _ Effect.t += User_login : login_object -> user_object Effect.t

let user_login x = User_login x |> perform

let with_handlers f x =
  try_with f x
    { effc = fun (type a) (eff: a t) ->
          match eff with
          | User_login _ -> Some (fun (k: (a, _) continuation) ->
              continue k { email = "test"; token = "test"; username = "test"; bio = "test"; image = None })
          | _ -> None }

let main() =
  run ~error_handler: debug_error_handler
  @@ logger
  @@ router [
    get "/" (fun _ -> html "Hello, world!");
    post "/api/users/login" @@
    (fun request ->
       let%lwt body = Dream.body request in
       body
       |> Yojson.Safe.from_string
       |> login_object_of_yojson
       |> user_login
       |> yojson_of_user_object
       |> Yojson.Safe.to_string
       |> json);
  ]

let () = with_handlers main ()

Making a POST /api/users/login request with a valid payload fails because the effect appears to be unhandled. As well as the following variation:
(* ... snip ... *)
let () =
  run ~error_handler: debug_error_handler
  @@ logger
  @@ router [
    get "/" (fun _ -> html "Hello, world!");
    post "/api/users/login" @@
    with_handlers (fun request ->
       let%lwt body = Dream.body request in
       body
       |> Yojson.Safe.from_string
       |> login_object_of_yojson
       |> user_login
       |> yojson_of_user_object
       |> Yojson.Safe.to_string
       |> json);
  ]

Lwt seems at fault here, as moving with_handlers inside let%lwt or not parsing the request body at all works as expected with the second variation. Are there any tips on how to use Dream with some top-level effect handlers? Either by installing them in a way that works with Lwt or duplicating them more ergonomically than just carefully spotting all use of async API from Dream and manually inserting effect handler inside the promise handlers.

I minimized this example (please do so! :)) to confirm that this is indeed an Lwt issue:
type _ Effect.t += E : unit Effect.t

let () =
  Effect.Deep.try_with
    begin fun () ->
      Lwt_main.run begin
        Lwt.bind (Lwt.pause ()) @@ fun () ->
        Effect.perform E;
        assert false
      end
    end
    ()
    {
      effc = fun (type a) (e : a Effect.t) ->
        match e with
        | E ->
          Option.some @@ fun (k : (a, _) Effect.Deep.continuation) ->
            prerr_endline "handling E";
            Effect.Deep.continue k ()
        | _ -> None
    }

I've opened https://github.com/ocsigen/lwt/issues/1003 to ask about it.

I am also unable to use algaeff with dream, which does not make use of Lwt:
https://github.com/RedPRL/algaeff/blob/main/src/State.ml
Here is a minimal example which compiles but fails at runtime:
module S = Algaeff.State.Make (Int)
let () =
  S.run ~init:100 @@ fun () ->
  Dream.run (fun _ -> Dream.html (Printf.sprintf "%i" (S.get ())))

15.02.24 09:12:55.204      dream.http ERROR Unhandled algaeff effect; use Algaeff.State.run


Parsing error

Hi,
I'm getting this error for many md files which I exported with Exporter from Apple Notes:
WARNING:root:Parsing error <class 'lxml.etree.XMLSyntaxError'> occurred with file dhl packstationskarte.md
WARNING:root:No declaration for attribute aria-hidden of element figcaption, line 3, column 152 (, line 3)
WARNING:root:Parsing error <class 'lxml.etree.XMLSyntaxError'> occurred with file esta usa.md
WARNING:root:No declaration for attribute src of element embed, line 3, column 132 (, line 3)
WARNING:root:Parsing error <class 'lxml.etree.XMLSyntaxError'> occurred with file handy--mobilfunkvertrag freenet<PHONE_NUMBER>0.md
WARNING:root:No declaration for attribute src of element embed, line 3, column 109 (, line 3)
Also in the .enex output file attachments don't seem to be included at all. Is this by design?
Thank you,
Helge

I'm not sure why this is happening, but it may have something to do with the capttions or embedded elements in those files. Maybe try simplifying the formattings, removing attachments/embedded images in those files, and trying again.
This tool does not support file attachments currently, I didn't need it for my use case, if you'd like to extend it to support them please do so I would welcome PRs.

@karloskalcium Thank you for your remarks! For now I'm not if it would make valid sense to invest into such enhancements since there's currently no working export available from Apple Notes, at least none which really exports all data (images, attachments). The best tool at the moment seems to be "Exporter" app but unfortunately it also doesn't cover all data in my notes. It's a shame that Apple doesn't provide interoperability despite GDPR Data Subject Rights. Even the export via https://privacy.apple.com/ is of terrible quality (especially the notes don't contain links to the attachments). Hopefully someone will tackle this sometime soon...

Sorry to hear that. Re: Exporter tool, you might want to send a note to the developer (https://star-lord.me/), and provide some examples of notes that don't export, maybe he can fix the tool accordingly. I have reported bugs in the past and he was responsive.

@karloskalcium Thank you for the suggestion. Today I found out that Applescript provides options to also export pdf attachments from Notes. There has also been some interesting work from Bear App and Appe Cloud Notes Parser. I lack a bit of capacity right now to spend more time on this but I will certainly follow-up on this topic. Thank you again.


[Bug report] Unexepcted message when refresh the web page

Version
main branch
Describe what's wrong
When I restart Gravitino server and visit the web page, the following error occur:

Error message and/or stacktrace
Please see above.
How to reproduce
Please see above.
Additional context
No response

@LauraXia123 ,Please help to verify this is a bug.

This should be an interface exception. Can you open the console and see which interface threw the exception? @yuqi1129

This should be an interface exception. Can you open the console and see which interface threw the exception? @yuqi1129

https://api.github.com/repos/apache/gravitino
It's the unstable github interface that's causing the problem.
Not an issue right now.

The API is not unstable. You can only make a limited number of calls to it before you are blocked. The information should be cached to avoid this issue or not checked as frequently.

This information is already cached, and the exception occurs because of repeated forced refreshes during the testing process.


[BUG] LNURLp broken

Describe the bug

Currently LNURLp is not working, neither in LNURLp plugin nor in offline shop, so it seems to be a deeper issue. Tested on my own instance and Legends.

LNbits version: 0.11.1

Offline shop is completely broken, as shown in the screenshot.

Moved the issue to the offlineshop repo then.

Offline shop is completely broken

Please post the output of your LNbits terminal so we can figure out what went wrong here. Thanks!

That’s the only output, basically the same as the screenshot. This is after using fjatjafs decoder, because wallets don’t even decode it. Same issue with legends.


Difficulty dropdown

Similar to how quizdb does it, where you select (multiple items) from a dropdown to choose which difficulties you want. Has the added benefit of having labels next to each number to signify what they mean.

Close as duplicate


Domain name

The domain name listed on the repository is for sale and I assume it wasn't paid for, should modify the repository accordingly.

Network issues caused duplicate entries


Added a new topic

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a perovskite.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

There are few repositories in this topic for now but they have potential for growth. A topic related to this field will help to gather all related repositories in a commonplace.

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

Hi @paramphy, thanks again for the contribution. However, after further investigation, this topic does not match our requirements for curation. See here in our contribution docs: https://github.com/github/explore/blob/master/CONTRIBUTING.md
specifically:

Valuable topics, for example, include those that are already widely used by repositories and could benefit from the addition of important information.

currently, this topic is only used on 4 public repositories.

Hi @paramphy, thanks again for the contribution. However, after further investigation, this topic does not match our requirements for curation. See here in our contribution docs: https://github.com/github/explore/blob/master/CONTRIBUTING.md
specifically:

Valuable topics, for example, include those that are already widely used by repositories and could benefit from the addition of important information.

currently, this topic is only used on 4 public repositories.


Fix All TODO Items

Fix ALL the TODO items found throughout the project

Closed all of the TODO items.


Rendering of in-line stage directions

Hi team,
In-line stage directions are rendering in odd ways. Here, in 1HW, for example:

The stage directions should be rendered in brackets and on the same line as the dialogue. The code is:
<sp who="#emd1HW_M_Hippolito" xml:id="emd1HW_M_a1_s1_sp6">                   <speaker>Hippolito</speaker>                   <l part="I"><stage type="delivery" xml:id="emd1HW_M_a1_s1_sd4">To the Duke</stage> I do arrest thee, murderer.</l>                   <l part="F"><stage type="delivery" xml:id="emd1HW_M_a1_s1_sd5">To Attendants</stage> Set down,</l>                   <l><anchor xml:id="emd1HW_M_anc_4332"/>Villains, set down <anchor xml:id="emd1HW_M_anc_37"/>that sorrow<anchor xml:id="emd1HW_M_anc_38"/>; ’tis all mine<anchor xml:id="emd1HW_M_anc_4333"/>.</l>                </sp>
Two further examples:

The style file modernized.scss line 35 specifies that stage directions are by default display: block; stage directions which are direct children of the speech (<sp>) element are to be rendered inline. @LEMDO-PM @pszpak: is this the intended behaviour? Or was the intention that <stage> elements that are descendants of <sp> elements should be inline?

That looks like an omission in the modernized.scss file, then. I'll tweak it.

I'll be on this tomorrow afternoon.

I've committed a fix for this, and Martin has committed a fix for an XSLT issue to restore parentheses. Leaving this open for now, but will close once we see the new build.

This looks good in this last build. Closing this issue :)


refactor: remove navigation tabs theming

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#464  👈
#463
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


Swift 4 support

When will there be swift 4 support?

I'm also interesting about that, any info about Swift 4 support would be great. Thanks

Any idea?

Hey there, I just pushed up a new branch called swift-4. Please let me know if you come across any issues. I had to add some special handling for Float types as it seems they can no longer be casted from Any as they could previously.

Hey, how do i get this swift 4 update?

Codable is the way to go w/swift 4.

...but that's not an answer to the swift 4 question.

@tristanhimmelman I appreciate the swift-4 branch, but I have pods using ObjectMapper as dependency, and since .podspec doesn't allow ":git" / ":branch" tags, I can't reference the new branch.
Is there any ETA when the swift-4 branch gets released on cocoa pods?

The swift 4 branch has now been released


[#106317732] Added Certificate based authentication for SP

moved markcowl's code from his fork which was based on Master to AutoRest branch

@hovsepm We need to plug in the ServiceClientCredentials support as well in AuthenticationFactory, as indicated

@hovsepm One change.  Feel free to merge after you address this (and please file the story I mention)


Form helpers

[ ] Investigate http://guides.rubyonrails.org/form_helpers.html and check what most important helpers do we need.
[ ] Investigate https://github.com/kylef/Stencil#customisation and findout how it can help

Might want to check out simple_form as well. That's the most popular form helper gem for Rails, which enhances the default helpers.

Yes, we need to analyse popular Ruby gems for form helpers https://www.ruby-toolbox.com/categories/rails_form_builders and then figure out what to start with.

The form helper I prefer is formtastic. It's well supported and has a very comprehensive API. I hope this helps. https://rubygems.org/gems/formtastic


dokku-redeploy.service is not enabled on Debian jessie

On a fresh install of Dokku on Debian Jessie, containers are not restarted after a reboot.
The dokku-redeploy service is present but not enabled.
~$ sudo systemctl status dokku-redeploy
● dokku-redeploy.service - Dokku app redeploy service
   Loaded: loaded (/etc/systemd/system/dokku-redeploy.service; disabled)
   Active: inactive (dead)

After enabling the service
sudo systemctl enable dokku-redeploy
containers are restarted after a reboot.
I'm not sure where this can be added (bootstrap.sh, dokku-standard/install, ...) or if it's just a matter of updating the doc regarding Debian installation.
I don't know if the service is enabled on openSUSE.
Infos:

uname -a:Linux actarus 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt11-1+deb8u6 (2015-11-09) x86_64 GNU/Linux
dokku version: 0.4.5
dokku plugin:

  00_dokku-standard    0.4.0 enabled    dokku core standard plugin
  20_events            0.4.0 enabled    dokku core events logging plugin
  apps                 0.4.0 enabled    dokku core apps plugin
  backup               0.4.0 enabled    dokku core backup plugin
  build-env            0.4.0 enabled    dokku core build-env plugin
  certs                0.4.0 enabled    dokku core certificate management plugin
  checks               0.4.0 enabled    dokku core checks plugin
  common               0.4.0 enabled    dokku core common plugin
  config               0.4.0 enabled    dokku core config plugin
  docker-options       0.4.0 enabled    dokku core docker-options plugin
  domains              0.4.0 enabled    dokku core domains plugin
  enter                0.4.0 enabled    dokku core enter plugin
  git                  0.4.0 enabled    dokku core git plugin
  logs                 0.4.0 enabled    dokku core logs plugin
  named-containers     <IP_ADDRESS> enabled    dokku core named containers plugin
  nginx-vhosts         0.4.0 enabled    dokku core nginx-vhosts plugin
  plugin               0.4.0 enabled    dokku core plugin plugin
  ps                   0.4.0 enabled    dokku core ps plugin
  shell                0.4.0 enabled    dokku core shell plugin
  tags                 0.4.0 enabled    dokku core tags plugin
  tar                  0.4.0 enabled    dokku core tar plugin

Environment: Vagrant box debian/jessie64 8.2.2 on Virtualbox 5.0.10
How was dokku installed?: Debian package unattended installation with web_config=false and vhost_enable=true

Hi,
Yes. I have the same problem with dokku 0.4.5 on Debian Jessie.
~$ sudo systemctl status dokku-redeploy
● dokku-redeploy.service - Dokku app redeploy service
   Loaded: loaded (/etc/systemd/system/dokku-redeploy.service; disabled)
   Active: inactive (dead)

Probably worth enabling here. Do either of you mind making a PR for it?


Not completely working

Hey there! Thanks for writing this plugin.
It is partially working, as ESBuild does not return any error when I use it on this kind of syntax :
import 'core/*/js/__*.js';
import 'components/*/*/js/__*.js';
import 'flexibles/*/js/__*.js';

But my files are still not loaded.
They are auto instanced classes that looks like that:
export default new class {
	constructor() {
		console.log( 'hello you' );
	}
};

When I try to debug it this way:
import cores from 'core/*/js/__*.js';
console.log(cores);

I get an infinite object with each part of the path, never seeing the expected class.

Any idea what could be causing it?
Thanks a lot for your help!

Thanks for reporting this! I'll definitely look into it. If you've made any headway keep me updated. I've had a next branch with quite a few updates that I never quite wrapped up last year that may have covered this. I'll try to set up some unit tests and see where I'm at.

Hey Dustin, thanks for your comment.
I did fork it here: https://github.com/lesanimals/esbuild-plugin-glob-import
It works for my use case, but I didn't particularly  tested it, and it is the first time I have a look at esbuild plugins.
If it can be of any help to you, I'll be more than happy!
Thanks  again!


Implement forgot password endpoint

This involves sending a token to the user to their email.

Maggie tar den här issue

Maggie takes this issue


Lakeformation integration

Add lakeformation integration to the adapter https://aws.amazon.com/about-aws/whats-new/2022/11/amazon-athena-support-lake-formation-fine-grained-access-control/
Maybe as an idea, if lakeformation=true, the adapter take care of enabling lakeformation, and register the buckets used by the adapter on it.

Hey @nicor88, what kind of integration do you have in mind?
I am not sure if this adapter should modify any Lake Formation settings. Only Data lake administrators (or Lake Formation admins in form of IAM User or IAM Role) should be able to make changes.
I see dbt "jobs" as consumers of the data platform where settings, data locations and permissions on DBs and tables are provided as part of the data platform provisioning.

@mattiamatrix in the context of data mesh, with a centralized catalog I agree with you. I was thinking that the adapter could have some little helpers (disabled by default), that allows to register the s3 bucket used by dbt in lakeforkation.

I see the concept of grants already exists. It might be extended to support buckets too

@mattiamatrix

I am not sure if this adapter should modify any Lake Formation settings. Only Data lake administrators (or Lake Formation admins in form of IAM User or IAM Role) should be able to make changes.

I would not fully agree with your statement.
Lakeformation offers detailed permissions settings, i.e. permissions given to non-administrators can be given just as is or be given as grantable, so the role itself can grant the exact same scope at max which it was granted itself.

I see dbt "jobs" as consumers of the data platform where settings, data locations and permissions on DBs and tables are provided as part of the data platform provisioning.

I would also disagree with this statement slightly. The main determinant whether something falls into the platform or rather the "jobs" category is frequency of change.
So for example, I would put datalake locations, s3 buckets, firmly into the platform domain and as something not managed by dbt. There are fundamental security aspects, the N of managed objects is probably small and the concept of a location works per se pretty independent of if and how data is transformed after ingestion.
This is not the case with grants, to the concept you linked to @mattiamatrix.
I see this much closer and useful when managed by dbt for a couple of reasons:

Grants on a dataset are a very closely related artefact of a dbt job, hence they are implemented for the other platforms
Managing grants somewhere else, in Terraform, is rather cumbersome, both when it comes to coding, as well as the code living in two completely separate places
Typically, you'd want a potentially large number N of grants if the datalake is used and again, paired with frequent changes, leading to a high number of frequent changes. Again, if the changes live in two places, deploying them accurately together is cumbersome
I'd see very basic grants (who is datalake admin, which roles have what kind of maximum permissions) as manageable by the data platform team in Terraform. These base settings are expected to not change frequently
Operational changes within this scope should be manageable via Lakeformation within dbt

This is a rough outlook on my thoughts on the topic.
Happy to discuss further and I'd also ping the community to ideally get some more perspectives to make the tradeoffs and prio clearer.
What do you think @mattiamatrix @nicor88 ?

For using Lake Formation Tag Based Access Controls (TBAC) I think encoding the tags in DBT YAML could be really useful. i.e. Categorizing the data produced by the model to drive the appropriate grants elsewhere. The TBAC grants themselves and other infra setup does seem better managed in TF.

@dacreify I totally agree with you, the key is to:

let dbt-athena create "schemas" aka databases and then tag them based on what is specified from in a yml definition
as tables inherit from databases the tags, the only case where we need to re-tag a table is when its tags is different from the parent, this can be done with a post hook macro

In both case to do this operations, the role used for dbt needs to be have associate tag permissions in lf.

Groundwork done in this PR: https://github.com/dbt-athena/dbt-athena/pull/159
What is missing is the possibility to add tags to specific columns

Closing this one.
The most relevant work has been implemented:

dbt athena nows supports assigning tags to database, tables and columns
dbt athena supports "overwriting" inherited tags from the parent relations, e.g. if a table inherits tags from the parent db, we can overwrite them using dbt.

At the momement there is no plan for dbt-athena to provide access control for the AWS principals, this means that you cannot use dbt-athena to grant select/insert/delete or create/drop in tables and databases with specific tags.
It's recommended to have such permission control using other tooling e.g. terraform/cdk or boto3.


Printing text output while prompt is active?

If I create a new engine and run it and then start printing output to the console, the prompt (> )  is not re-written. This makes it unclear if the window is still accepting input. Is there a way to make it so that while the prompt is shown, subsequent WriteLine calls will re-write the prompt after the output is displayed? Something like this:
Currently, if output is written to the console while the prompt is shown (and the user is typing) it does this:
> inputinputOUTPUT
moreinputmoreinput

Once that output is written, the user is left to finish their typing without a prompt. I'm looking for this instead:
> inputinput
OUTPUT
> inputinputmoreinputmoreinput

In this case, the new output was put on a new line. At the end of that output, a new prompt was written followed by the unfinished input.

Getting that to work would really be great. Then it would be possible to really take advantage of async as well.
I will give it a try. Feel free to try it out as well!

My attempt was to add System.Console.Write("> ") after the WriteLine call in the ClientConsole (see here). This almost worked, but...

It would sometimes print 2 prompts
I couldn't figure out how to re-print the entered text

I'm still trying to fully understand how this library works, so don't expect any genius contributions coming from me anytime soon :wink: But I'll see what I can figure out.

I think I would try to somehow move the line with the input buffer and cursor down and print the output above it. Not sure that this would work, but I think that would be my first try.

When using the Output... methods in the commands this should work now. If using the regular "System.Console.WriteLine(.." it will not work.
Give it a try and tell me what you think.

The Output... methods don't seem to have any dependency on being inside a command; could they be separated into the generic console? I'd like to be able to use them for things like debug info and warnings from other code that isn't part of a command.

You are very right. I moved that thing to the console part. So, you can use the console object instead.
_console.WriteLine("SOME DATA", OutputLevel.Default);
Then it works for commands as well, since it uses the console.
I also found two other issues now. When pressing ESC, the cursor moves back up to the lines above, where input started. And, if the input is above two lines then only the last line is moved.
I am not sure how to fix that. Have to sleep on that. :)

This is looking great! One more thing: can it be added to the Write() method as well?

Testing some more I realised that this is really tricky. There are so many special cases where it does not work as it should.

The Write() function
When the output is more than one line.
When the entry text is more than one line.
When using the special parameter input functions

I have started to write a completley new input manager. I have the basic stuff in place but there is still some work to do. I will try to have a new version out later today so that you can try it out if you like.

This is fixed with version <IP_ADDRESS>. Now Write will get a new line to write on (above the input buffer). The Write() function will continue to write there, until a WriteLine() is used. Then the line is "closed". And a new line will be used for writing.


Disconect from database every minute

System information:

Operating system (distribution) and version:  Win 7 and Win 10
DBeaver version:  6.1 and early
Additional extensions:  Local server: Openserver, XAMP , WAMP, Remote servers  with Cpanel

Connection specification:

Database name and version; all mysql 5.5-8.0 and mariadb 10-10.0.3
Driver name: mysql and mariadb
Do you use tunnels or proxies (SSH, SOCKS, etc)?  no

Describe the problem you're observing:
Disconect from database every 1 minute. This problem 3 years old. Always i test new DBeaver and always this problem. I used ALL sql programm in the world. Nobody dont have this problem.
I must click this smaaaaaaaal icon - reconect every 1 minute. Imposible use programm.
Settings "Database - Error handle" dont work absolutly.
Error Image:
http://bit.ly/2lQ4OwR
How Heidisql and others absolutly never has`t this problem ?
Steps to reproduce, if exist:

Connect to db
Change table field
Wait 1 minute
Change table field  - no connection

Include any warning/errors/backtraces from the logs
db.log
-->

I can't reproduce this. But there can be several different causes of this problem.
It may be the server configuration (close idle connections after 1 minute). In this case configuring Kepp-Alive in connection settings may help (it is on Initialization page of connection settings dialog). Set it to 10-20 seconds.
It may be the driver problem (MariaDB). You could try another driver (MySQL, MySQL 8).
Did you try to enable connection auto-recovery option? It usually works (if server returns proper error code) and you don't need to click reconnect manually.
Please send debug.log file (https://github.com/dbeaver/dbeaver/wiki/Log-files) - it may contain some additional clues.

@andreysneg Please tell is this issue still actual for you?

there is no update on ticket for a long time. ticket closed.
if issue is still actual for you - feel free to reopen the ticket


Missing parameters cause consume request to fail

While building the rtpCapabilities the stack assembles various elements. IMHO it misses hereby to take care of the matchingLocalCodec.parameters, which are packed into the localParameters element of ExtendedCodec.
Those are not taken over before this line https://github.com/skymaze/pymediasoup/blob/6fd3564ed690fe59ab886334fe136bca84648a7e/pymediasoup/ortc.py#L162
and will later on make a consume request fail due to non matching parameters here: https://github.com/versatica/mediasoup/blob/0b4d4ef9f2bb0b9d117e48f707c970d406ee5022/node/src/ortc.ts#L1458
This doesn't harm with audio and VP8 streams, but is essential to make H.264 subscriptions work.
I "hotfixed" it by inserting a line to take over the localParameters in ortc.py:
        codec: RtpCodecCapability = RtpCodecCapability(**extendedCodec.dict())
        codec.parameters = extendedCodec.localParameters
        rtpCapabilities.codecs.append(codec)

Thanks, I released version 0.2.3, which should fix this issue.

Thanks for the quick response and good that this nice lib is still alive

I am no longer using mediasoup, but I wrote this lib according to the official mediasoup-client structure, so it can be easily maintained. Feel free to open a issue or make a PR.

@skymaze May I ask you if simulcast is supposed to work?
In analogy to my web solution I was adding "encodings" parameters to the sendTransport.produce() call. Track is the default VideoStreamTrack(), encoding H.264. It works w/o "encodings" parameter, but doesn't work with encodings, even though the consumer is notified about simulcast tracks and L3T3. The server spits warnings, continuously:
 mediasoup:WARN:Channel [pid:42091] RTC::Producer::ReceiveRtpPacket() | no stream found for received packet [ssrc:2458002806] +31ms

The video does not appear on the remote side.
                highestBitrate = 6000000
                mediumBitrate = highestBitrate / 2
                lowestBitrate = highestBitrate / 10

                encoding = RtpEncodingParameters(
                    maxBitrate=lowestBitrate,
                    priority='very-low',
                    rid="r0",
                    scalabilityMode='L3T3',
                    scaleResolutionDownBy=4
                )
                encodings.append(encoding)

                encoding = RtpEncodingParameters(
                    maxBitrate=mediumBitrate,
                    priority='low',
                    rid="r1",
                    scalabilityMode='L3T3',
                    scaleResolutionDownBy=2
                )
                encodings.append(encoding)

                encoding = RtpEncodingParameters(
                    maxBitrate=mediumBitrate,
                    priority='high',
                    rid="r2",
                    scalabilityMode='L3T3',
                    scaleResolutionDownBy=1
                )
                encodings.append(encoding)

            videoProducer: Producer = await self._sendTransport.produce(
                track = track,
                encodings = encodings,
                stopTracks = True,
                appData = appData
            )

Maybe it is related to this code,, aiortc does not implement the corresponding API. Or related to default scalabilityMode here


Classless html feature needs testing + demo site

reconfig/update the css properties and values
update html for a demo of the classless functionality

classless.css updated and tested!
classless.html is now a complete demo website for the classless functionality!


bug / issue with button callback query answering

https://github.com/TheHamkerCat/WilliamButcherBot/blob/dev/wbb/modules/spam.py#L110
https://github.com/TheHamkerCat/WilliamButcherBot/blob/dev/wbb/modules/admin.py#L57
get_chat_member will raise UserNotParticipant if any user not a member of the group tries to click on the buttons. which would result in a just a spinning circle, and eventually would result in a notification from @BotFather, saying Too few answers to callback queries.

Fixed in last commit, thanks.


Option to resolve bundle from executable file location

I would like to write applications that use bundler, have local gems installed using bundler install --deployment and have the executables of the application be executable on their own from any working directory.
Currently my executable starts with require 'bundle/setup' which should populate the LOAD_PATH with the locations of the bundle's gem bins, but this looks for the Gemfile from the current working directory. It would be nice to have some option to search up from the location of the executable file so that the related bundle for that file is used.
Doing this sort of thing hacks the functionality in by overriding the BUNDLE_GEMFILE variable, but it would be nice to pass a flag into setup instead or have some alternate 'bundle/executable' require
require 'pathname'
filepath = Pathname.new __FILE__
filepath.realpath.ascend do |path|
    if (path += 'Gemfile').exist?
        ENV["BUNDLE_GEMFILE"] = path.to_s
        break
    end
end
require 'bundler/setup'

It's not a large feature but maybe I'm missing something in how to use bundler properly, so some sort of feedback would appreciated before I try to submit any pull request.

Setting BUNDLE_GEMFILE is indeed the official way of doing this

require 'bundle/setup' will already search up (see Bundler::SharedHelpers.find_gemfile).


[Playground] Implement tag based filtering feature on the frontend

What needs to happen?
Improve type filter (all, example, kata, unit test) to the tag based filter, that can use any custom tags.

Issue Priority
Priority: 3
Issue Component
Component: beam-playground

.take-issue

@miamihotline's work in progress: pg_22074_tag_based_filtering

.take-issue

.take-issue


Can't build multi arch images on M1 chips (Jib 3.4.1+) (Works on x64 chips)

Environment:

Jib version:  3.4.2
Build tool: Gradle 7.6.4
OS: macOS and Linux

Description of the issue:
When running ./gradlew jibDockerBuild on macOS (M1 chip), it returns an error saying the the from image is not a manifest file, but this base image is a manifest list, and the same command when executed on my CI environment running on containerized Debian (Intel x64) just works.
Problem started after upgrading jib to 3.4.1 and persists on 3.4.2. It was not happening on 3.1.2
Expected behavior:
Image to be built successfully on both OS and Processor Architecture.
Steps to reproduce:

On a macOS system with M1+ chips
Run the command ./gradle jibDockerBuild

jib-gradle-plugin Configuration:
// Projects that generate a JVM based Docker image
apply(plugin = "com.google.cloud.tools.jib")
jib {
    to {
        image = "local/project"
    }
    from {
        image = "europe-docker.pkg.dev/<redacted>/repo-docker/base-17-jre"
        platforms {
            platform {
                architecture = "amd64"
                os = "linux"
            }
            platform {
                architecture = "arm64"
                os = "linux" //also tried darwin here.
            }
        }
    }
    container {
        creationTime.set("USE_CURRENT_TIMESTAMP")
    }
}

Log output:

What went wrong:
Execution failed for task '::jibDockerBuild'.

com.google.cloud.tools.jib.plugins.common.BuildStepsExecutionException: cannot build for multiple platforms since the base image 'europe-docker.pkg.dev/redacted/repo-docker/base-17-jre' is not a manifest list.

Additional Information:
This base image is based on gcr.io/distroless/java17-debian11, hosted internally without modifications. But I also tried from the original distroless repo and also tried eclipse-temurin:17 image. None of them worked on my macOS env.

Hmm... just to rule out something: can you try with ./gradlew jibDockerBuild -Djib.useOnlyProjectCache=true? This way, Jib won't use centrally cached images.

Same result unfortunately.

What if you specify the SHA pointing to the manifest list as the from image reference?
You can take note of the correct SHA when you run the following command on x64 (not on Mac). Mind clean and -Djib.useOnlyProjectCache=true. And it's be interesting to see if the same command reports a different SHA on Mac.
$ ./gradlew -Djib.useOnlyProjectCache=true clean jibDockerBuild
...
Containerizing application to Docker daemon as ...
The base image requires auth. Trying again for alpine@sha256:185518070891758909c9f839cf4ca393ee977ac378609f700f60a771a2dfe321...
Using credentials from Docker config (/home/chanseok/.docker/config.json) for alpine@sha256:185518070891758909c9f839cf4ca393ee977ac378609f700f60a771a2dfe321
Using base image with digest: sha256:185518070891758909c9f839cf4ca393ee977ac378609f700f60a771a2dfe321

And make sure the SHA points to a manifest list. In my case above,
$ docker manifest inspect alpine@sha256:185518070891758909c9f839cf4ca393ee977ac378609f700f60a771a2dfe321
{
   "schemaVersion": 2,
   "mediaType": "application/vnd.docker.distribution.manifest.list.v2+json",
   ...

I tried informing the image sha256, but it didn't work. But I noticed something. Although the sha256 I'm providing is a manifest (when I check on the registry itself), the downloaded version on my laptop is the linux/arm64 version.

About the command you recommended, the error was the same.
My docker version is 4.29.0, and I tried with containerd enabled and disabled, both returns the same error and the same result when running docker inspect <image>.
PS: Randomly during my tests I saw a message about not being able to build multi-arch using docker, but it doesn't show-up all the time.

Quick update. If I run docker manifest inspect the result is the same as the registry.

As a last resort, I'd like to see all the raw HTTP requests and responses between Jib and your registry. Can yo follow these instructions to capture the traffic? Make sure you include both clean and -Djib.useOnlyProjectCache=true. (That is, it should not use the cached image.) You may also want -X (debug logging) instead of --info, but actually I am not sure if debug logging will be useful.
You may redact the registry address, but please do not redact SHA strings.

PS: Randomly during my tests I saw a message about not being able to build multi-arch using docker, but it doesn't show-up all the time.

Yeah, regardless of this issue, jibDockerBuild will not work anyway, because the Docker engine doesn't support storing a manifest list.

Sorry for the late reply. I'll try this as soon as possible and get back with the results.

Hey guys, I've noticed that if I remove
platform {
                architecture = "amd64"
                os = "linux"
            }

or the entire platforms section from the configuration above, then it works fine. Hope it helps.

It should be fixed in release 3.4.3, closing for now. Please re-open it is not fixed.


JS error when interacting with the toolbar

Hi everyone, someone had this error before? Everytime I interact with the redactor component, specially clicking the toolbar options this error appears.
I've added in the layout the {% framework %}, {% scripts %} and {% styles %} tags. What im doing wrong?

What version of jQuery have you included with your site?

I'm using version 3.1.0.
I didn't thought about that actually. I'll downgrade the jQuery version and let you know the results.

@daftspunk I created a pull request #33 fixing the issue.

Cheers! Fixed by #33


As a user, I want to have zero page of the administrative part to select an object for settings.

For administrator role.
Precondition:
User with Administrator role pressed [Адміністрування] button in the dropdown list of the Header.
Page includes:

Header
Footer
Leftside menu
Text "Оберіть об'єкт налаштування у меню"

Leftside menu
has name "Налаштування:"  and
chapters according to the names of pages.
Each chapter has buttons to open page with settings of corresponding object.
For current release left side menu has chapter
Головна and [Важливе] button.

Epic #260

22/07/21
BA check.
Accepted.