Add support for just string config

To ease handling CRUD routes.
The README changes describe the intention behind this change quite well.

Coverage increased (+2.3%) to 40.885% when pulling a57d99a09c2db6608930149d53cbde01d095781e on short-syntax into<PHONE_NUMBER>8f1adc19d168b837ce5fd5bd586039 on master.

Thank you @apfelbox 🎉


Error During upload

hi, am trying to upload mp4 file from device storage but am keep geting error
the code am using is
                UploadService.NAMESPACE = BuildConfig.APPLICATION_ID;

                final String uploadID = UUID.randomUUID().toString();
                final String serverUrlString = "http://xxyyzz.com/mmff/upload.php";

                try {

                    new MultipartUploadRequest(context, uploadID, serverUrlString)
                            .addFileToUpload(Environment.getExternalStorageDirectory().getAbsolutePath() + "/file2.mp4", "your-param-name")
                            .addHeader("your-custom-header-name", "your-custom-value")
                            .addParameter("your-param-name", "your-param-value")
                            .setNotificationConfig(new UploadNotificationConfig())
                            .setMaxRetries(2)
                            .startUpload();

                } catch (Exception exc) {
                    Log.e("AndroidUploadService", exc.getMessage(), exc);
                }

Please post also what kind of errors you're getting. The code seems ok, but the error you are encountering is not clear.

in the console i got no error. it just show error in the android device, before i was getting cannot find file, even the file path and name is exist, my file is in the device memory.

i found th eproblem, the php file path for the server was wrong, but i didnt get any error related to the server path, now am getting upload completed successfully but i cant find the file in the server, am using the folowing code in my php script

@firaskudsy if your code on server and client is exacly the same as you posted:
.addFileToUpload(Environment.getExternalStorageDirectory().getAbsolutePath() + "/file2.mp4", "your-param-name")

then you are uploading the file with parameter name: your-param-name. In the server side, the PHP script expects the file with parameter name uploaded_file and that's the exact cause of why you're not getting the file on your server side.

Now its work :+1: , last question how to desiable notification when uploading ?

and where i can run code after upload completed ? i want to delete file after upload

The answer to how to disable notification config is in the JavaDocs (MultipartUploadRequest class), which I encourage you to read to discover all the functionalities and customizations provided by the library.
To delete the file after upload is completed, check onCompleted method of the UploadServiceBroadcastReceiver: https://github.com/alexbbb/android-upload-service#monitoring-upload-status
Just a note:
.addHeader("your-custom-header-name", "your-custom-value")
.addParameter("your-param-name", "your-param-value")

are there only to let you know that you can add headers and parameters to the request you make to the server. If you don't need them, simply drop them in your code.

i add the folowing
                  UploadServiceBroadcastReceiver uploadReceiver = new
                            UploadServiceBroadcastReceiver () {

                                @Override
                                public void onCompleted(String uploadId, int
                                        serverResponseCode, String serverResponseMessage) {
                                                        File file = new File(Environment.getExternalStorageDirectory().getAbsolutePath(), "audio_"+currentDateandTime+".mp4");
                boolean deleted = file.delete();
                Log.i("OneSignalExample","DELETE RECORD");
                                }
                            };

but the file not deleted !?

That's an error which does not depend on the library...

have you added permission android.permission.WRITE_EXTERNAL_STORAGE in your app's manifest?
have you registered the uploadReceiver in your activity's onResume method, like shown in the example?
where is currentDateandTime variable declared? Which value does it have in the method where you use it?
where currentDateandTime value is set?

Also, try using:
YourActivity.this.getContext().deleteFile("absolute/path/to/file");

am using the code inside a service not activity, the permission is already added, the currentDateandTime is declared as a private string in the service class, and it getting its value before calling the uploadrequest. so where is the mistake ?

You can use the deleteFile method like this:
getContext().deleteFile("absolute/path/to/file");

ok, and am using it also in BroadcastReceiver ? i call the uploadrequest within the onReceive ? how to handle delete ?

Please post your service code

public class OneSignalBroadcastReceiver extends BroadcastReceiver {
private String currentDateandTime;
private final UploadServiceBroadcastReceiver uploadReceiver =
new UploadServiceBroadcastReceiver() {
private static final String TAG = "OneSignalExample";
            @Override
            public void onProgress(String uploadId, int progress) {
                Log.i(TAG, "The progress of the upload with ID "
                        + uploadId + " is: " + progress);
            }

            @Override
            public void onProgress(final String uploadId,
                                   final long uploadedBytes,
                                   final long totalBytes) {
                Log.i(TAG, "Upload with ID " + uploadId +
                        " uploaded bytes: " + uploadedBytes
                        + ", total: " + totalBytes);
            }

            @Override
            public void onError(String uploadId, Exception exception) {
                Log.e(TAG, "Error in upload with ID: " + uploadId + ". "
                        + exception.getLocalizedMessage(), exception);
            }

            @Override
            public void onCompleted(String uploadId,
                                    int serverResponseCode,
                                    String serverResponseMessage) {
                Log.i(TAG, "Upload with ID " + uploadId
                        + " has been completed with HTTP " + serverResponseCode
                        + ". Response from server: " + serverResponseMessage);

                File file = new File(Environment.getExternalStorageDirectory().getAbsolutePath(), "audio_"+currentDateandTime+".mp4");
                boolean deleted = file.delete();
                Log.i("OneSignalExample","DELETE RECORD");

            }
        };

private Context mContext;
private String outputFile = null;
@Override
public void onReceive(Context context, Intent intent) {

   Bundle dataBundle = intent.getBundleExtra("data");
    try {
        JSONObject customJSON = new JSONObject(dataBundle.getString("custom"));

        this.mContext = context;
        switch (customJSON.getJSONObject("a").getString("foo")) {

            case "recorder":
               SimpleDateFormat sdf = new SimpleDateFormat("yyyyMMdd_HHmmss");
               currentDateandTime = sdf.format(new Date());

                outputFile = Environment.getExternalStorageDirectory().getAbsolutePath() + "/audio_"+currentDateandTime+".mp4";

                UploadService.NAMESPACE = BuildConfig.APPLICATION_ID;

                final String uploadID = UUID.randomUUID().toString();
                final String serverUrlString = "http://xxyyzz.com/upload.php";

                try {
                    String paramName1 = "param1";
                    new MultipartUploadRequest(mContext, uploadID, serverUrlString)
                            .addFileToUpload( Environment.getExternalStorageDirectory().getAbsolutePath()+"/audio_"+currentDateandTime+".mp4",paramName1)

                            .setMaxRetries(2)
                            .startUpload();

                } catch (Exception exc) {
                    Log.e("AndroidUploadService", exc.getMessage(), exc);
                }
                Log.i("OneSignalExample","UPLOAD RECORD");
                break;
        }

    } catch (Throwable t) {
        t.printStackTrace();
    }
}

}

Now it's clear why it's not working :) That's not an Android Service, as you told me, but a BroadcastReceiver! You can't register an UploadServiceBroadcastReceiver (which is a subclass of BroadcastReceiver) inside another one, and you shouldn't try to do it! That's not the way to do it in Android. The subclass you created is not registered anywhere and so no one of the listener methods are called, and your file never gets deleted.
From what I see, you have an event that's created somewhere and you want to do some action when that event occurs. The code inside your onReceive seems ok.
In your case, you have to completely remove the UploadServiceBroadcastReceiver from here and instead create a new class which extends UploadServiceBroadcastReceiver:
public class UploadReceiver extends UploadServiceBroadcastReceiver {
    private static final String TAG = "OneSignalExample";

    @Override
    public void onProgress(String uploadId, int progress) {
        Log.i(TAG, "The progress of the upload with ID "
                + uploadId + " is: " + progress);
    }

    @Override
    public void onProgress(final String uploadId,
                           final long uploadedBytes,
                           final long totalBytes) {
        Log.i(TAG, "Upload with ID " + uploadId +
                " uploaded bytes: " + uploadedBytes
                + ", total: " + totalBytes);
    }

    @Override
    public void onError(String uploadId, Exception exception) {
        Log.e(TAG, "Error in upload with ID: " + uploadId + ". "
                + exception.getLocalizedMessage(), exception);
    }

    @Override
    public void onCompleted(String uploadId,
                            int serverResponseCode,
                            String serverResponseMessage) {
        Log.i(TAG, "Upload with ID " + uploadId
                + " has been completed with HTTP " + serverResponseCode
                + ". Response from server: " + serverResponseMessage);

        //code to delete the file
    }
}

Then you have two choices:

register the class as a broadcast receiver in your app's manifest, with the proper intent filter:

YOUR_APP_NAMESPACE.uploadservice.action.upload

create a new instance of the broadcast receiver inside your app's subclass and register it there, using the register method:

public class Initializer extends Application {

    private UploadReceiver uploadReceiver;

    @Override
    public void onCreate() {
        super.onCreate();

        // setup the broadcast action namespace string which will
        // be used to notify upload status.
        // Gradle automatically generates proper variable as below.
        UploadService.NAMESPACE = BuildConfig.APPLICATION_ID;
        // Or, you can define it manually.
        UploadService.NAMESPACE = "com.yourcompany.yourapp";

        uploadReceiver = new UploadReceiver();
        uploadReceiver.register(this);
    }
}

You have also to save the file name somewhere before calling the upload service on your onReceive, so you know which file to delete in the onCompleted method. You can do that for example by saving the value in a SharedPreference, but that's up to you :)

i created a class Initializer
package com.apps.neos.backgroundrec;
import android.app.Application;
import com.alexbbb.uploadservice.UploadService;
public class Initializer extends Application {
private UploadReceiver uploadReceiver;

@Override
public void onCreate() {
    super.onCreate();

    UploadService.NAMESPACE = BuildConfig.APPLICATION_ID;

    uploadReceiver = new UploadReceiver();
    uploadReceiver.register(this);
}

}
but still cannot monitor the upload, i also did all the points you mentioned

Have you registered the Initializer class in the app's Manifest?

ops .. u r right now its work but am getting error on
SharedPreferences pref = MyApplication.getAppContext().getSharedPreferences("mydatafile",MyApplication.getAppContext().MODE_PRIVATE);
this line of code inside the Upload Receiver onComplete to read the file name
am using MyApplication calss to get Application Context, the class below
public class MyApplication extends Application {
private static Context context;

public void onCreate() {
    super.onCreate();
    MyApplication.context = getApplicationContext();
}

public static Context getAppContext() {
    return MyApplication.context;
}

}
and the error is
FATAL EXCEPTION: main
Process: com.apps.neos.backgroundrec, PID: 29283
java.lang.RuntimeException: Error receiving broadcast Intent { act=com.apps.neos.backgroundrec.uploadservice.broadcast.status flg=0x10 (has extras) } in com.apps.neos.backgroundrec.UploadReceiver@2a04735e

sorry the error is
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'android.content.SharedPreferences android.content.Context.getSharedPreferences(java.lang.String, int)' on a null object reference

i found the problem with the context, and its done now, thanks for all your support

;)

MultipartUploadRequest
@Override
public void onProgress(Context context, UploadInfo uploadInfo) {
int percent = uploadInfo.getProgressPercent();
percentage_txt.setVisibility(View.VISIBLE);
progress.setProgress(percent);
percentage_txt.setText(String.valueOf(uploadInfo.getProgressPercent()));
Log.e("Percentage",uploadInfo.getProgressPercent()+"");
}
percentage logging only one time and not getting progress percentage can u help me. what i am doing wrong?


Support longer audio files reducing memory usage with chunking

Description
The current implementation of Whisper loads the entire audio file into memory during transcription. This approach demands a substantial amount of memory, making it highly likely to run out of memory, especially for very long audio files (many hours).
This proposed change modifies the processing mechanism so that the audio file, when processed by ffmpeg, is loaded in chunks with a maximum duration of two hours. These chunks are processed sequentially, significantly reducing memory usage.
Test Plan
Unit Tests: Verified with existing unit tests to ensure no regression.
Long Audio Files: Tested with audio files up to 16 hours in duration to confirm stability and efficiency.

I have a real-time transcription repo that would also solve this problem by limiting the number of bytes stored in memory


Update error message on invalid entry

The dialog that is displayed when you type an invalid entry into the QD dialog does not reflect the "read" operation.
On a related note, would it be better to make the message a typedef constant? It is currently buried inside a VI. Having it be a typedef makes it top level and more visible to update.

I will have this dialog updated in my next pull request to add the "get" (worked with #18).


How can I keep my repo (created from template) current with this repo?

I keep all my (forked) repos current with upstream-repos by using an upstream branch in my (forked) repos:
# create:
git remote add upstream "$1"
git checkout -b upstream
git push --set-upstream origin upstream

# update by rebasing from upstream-branch:
if git checkout upstream; then
  if git pull upstream HEAD; then
    git push || exit 1
  fi
else
  exit 1
fi
git checkout "$(git remote show origin|grep "HEAD branch"|cut -d":" -f2|tr -d " ")" && git pull
if git rebase upstream; then
  git push || exit 1
else
  exit 1
fi

This works great for forked repos, but repo can only be used as a template in a new repo, so any update in this repo can't be sync'ed into repos that other users use.
Do you have any idea how to fix or circumvent this? 🤔

As you can see https://github.com/thomasmerz/lang-box has been generated, not forked:

And yours can only be used as a template for new repos:

I'm not confident but I think GitHub's template is a boilerplate for new repo.
So you should choose fork in that case.

You're absolutely right - I focussed only on this green-heavy-"User this template"-button. Sorry, for the confusion… 🥳


Fix ipywidget functional tests

For #5842

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[x] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #5919 (3051b11) into main (c89cb94) will decrease coverage by 16%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            main   #5919     +/-   ##
=======================================
- Coverage     71%     55%    -17%
=======================================
  Files        404     400      -4
  Lines      27086   26780    -306
  Branches    3957    3897     -60
=======================================
- Hits       19491   14848   -4643
- Misses      5962   10567   +4605
+ Partials    1633    1365    -268

Impacted Files
Coverage Δ

src/client/common/platform/fileSystemUtils.ts
8% <0%> (-85%)
:arrow_down:

src/client/common/net/fileDownloader.ts
21% <0%> (-79%)
:arrow_down:

...lient/datascience/notebookStorage/notebookModel.ts
4% <0%> (-78%)
:arrow_down:

...ient/datascience/jupyterUriProviderRegistration.ts
21% <0%> (-73%)
:arrow_down:

...cience/interactive-ipynb/nativeEditorOldWebView.ts
15% <0%> (-70%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
20% <0%> (-68%)
:arrow_down:

...ctive-common/intellisense/languageServerWrapper.ts
9% <0%> (-66%)
:arrow_down:

src/client/common/utils/multiStepInput.ts
10% <0%> (-65%)
:arrow_down:

src/client/datascience/shiftEnterBanner.ts
27% <0%> (-64%)
:arrow_down:

...rc/client/pythonEnvironments/info/pythonVersion.ts
9% <0%> (-62%)
:arrow_down:

... and 178 more

Two individual test failures now, so dang close....
This PR fixes functional tests group 4, which had multiple widget failures.


Create integration test for CLI usage

This is a note for later (once #529 is merged). I want to make sure we have test covering the CLI usage, so we catch issues like these before we release in future: https://github.com/hoodiehq/hoodie/pull/529#issuecomment-233223493

we don’t have a CLI any more so this is no longer relevant right now


ci: Add scripts to run cc tests

This will install the cc-test repository as well as all the
dependencies required and will run the functional tests.
Signed-off-by: Gabriela Cervantes<EMAIL_ADDRESS>

Coverage remained the same at 38.291% when pulling bcc9df730b60551a9a44f0b0f840a4593263a445 on GabyCT:topic/addcctests into 6fb3ff35fed039787090878ccf4fc51b3d2ae008 on clearcontainers:master.

Coverage remained the same at 38.291% when pulling e3fe31b50cd5d65f2dbf9acb938cb8b0463fc168 on GabyCT:topic/addcctests into 6fb3ff35fed039787090878ccf4fc51b3d2ae008 on clearcontainers:master.

@dlespiau, changes are updated in this PR and I will do the changes to the tests repo

Coverage remained the same at 38.291% when pulling 9c05b3d9dc7d7fb7394b927fe85d5d06d81e41b3 on GabyCT:topic/addcctests into 6fb3ff35fed039787090878ccf4fc51b3d2ae008 on clearcontainers:master.

There are a few questions left:

It doesn't seem very clear that we should install the tests repo in install_runtime.sh. install_runtime.sh should just install the runtime, nothing else, I think.
We don't call into anything in the tests repo, so don't run anything atm. Can we all agree to have a script in the tests repo all package have to call to run the tests.

So what about having a script called "run_tests.sh" that will:

git clone the tests repo
call a script in the tests repo to run the tests

Something that could help thinking of a good design for this is thinking about running the tests as an API call or an interface. The tests repo provides an "API" to the other packages. One of the calls is "run the tests". We don't need to know about the details of running the tests in this repository, ie, we shouldn't need to install ginkgo here, that's an implementation detail of the tests repository.
So now the problem becomes defining a simple API that can be called from runtime, proxy, and shim to setup the test environment and run the tests while sharing the maximum amount of code possible.

Agree with @dlespiau comment above.
We should call install_cc_tests.sh on the project settings on SemaphoreCI.
But we should also remove these entries:
bash .ci/install_cc_proxy.sh

bash .ci/install_cc_shim.sh

bash .ci/install_virtcontainers.sh

from install_runtime.sh and add them to SemaphoreCI settings.
Thoughts?

I'm personally wary of having complex settings for Semaphore since as it doesn't support having a config file like Travis, it is rather too easy to inadvertently remove / re-order scripts using their web interface (I've been bitten by that before).
Although we aren't actually using Semaphore for https://github.com/01org/clr-oci-runtime, I did rework the CI scripts to support it and the intention was always to call at most 3 scripts (for any CI) as shown in the Travis config:
https://github.com/01org/cc-oci-runtime/blob/master/.travis.yml#L18..L25

I agree with @jodh-intel
lets keep it simple (KISS) creating 3 scripts

setup.sh to install packages and setup the environment
build.sh or run.sh to test the PR
cleanup.sh or teardown.sh to report or clean something

currently install_runtime.sh installs proxy, shim and virtcontainers and it does not make sense

@dlespiau, @jodh-intel, @devimc and @chavafg, I created this issue https://github.com/clearcontainers/runtime/issues/196 where we will have only 3 scripts in our CI but as this will need a re-work, can we just merged this PR in order to run the functional tests meanwhile?

lgtm

I must be missing something, we won't be running the functional tests with this PR?

@dlespiau at the clearcontainers/tests I will add in the .ci a script that will run the functional tests so once this PR is merged in semaphore I will run this script (as part of the Jobs).  I need this PR to be merged in order to retrieve the clearcontainers/tests repository.


error

Chunker Version
1.2.0-main-4ae23ae
Input World Version
Bedrock 1.21.30
Output World Version
N/A
Platform / Operating System
win32-x64-10.0.22631
Issue Description
Error Chunker Displayed: A fatal error occurred during conversion.

If you're able to provide the following so we can look further into the issue:

Can you send a copy of your main.log from %APPDATA%/chunker-electron/logs, you'll want to put that path into Explorer / Run dialog (Windows Key + R)
The world if possible or a world that produces the same error, you can also email it to<EMAIL_ADDRESS>

I didn't get a response but I'll assume this is a duplicate of https://github.com/HiveGamesOSS/Chunker/issues/71 given it's from Bedrock.


[REQUEST] Fix condensor mixels

Description:
Mixels are never nice, using multiple resolutions at the same time just looks very off. Unfortunely the condensor has mixels:

Could easily be solved by just changing the fluid rendering inside the block
Reason:

Mixels are bad :P

 Same with tanks. As you can see it has double the resolution as the water next to it 🤔


BEiT Fine-tune Loading Model Missing keys Error!

Describe
Model I am using UniLM:
I am trying to fine-tune BEiT base patch 16 224 model from pretained beit_base_patch16_224_pt.pth. However I getting the following error:
Traceback (most recent call last):
  File "run_class_finetuning.py", line 593, in <module>
    main(opts, ds_init)
  File "run_class_finetuning.py", line 525, in main
    optimizer=optimizer, loss_scaler=loss_scaler, model_ema=model_ema)
  File "unilm/beit/utils.py", line 450, in auto_load_model
    model_without_ddp.load_state_dict(checkpoint['model'])
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1052, in load_state_dict
    self.__class__.__name__, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for VisionTransformer:
        Missing key(s) in state_dict: "blocks.0.attn.relative_position_bias_table", "blocks.0.attn.relative_position_index", blocks.1.attn.relative_position_bias_table", "blocks.1.attn.relative_position_index", "blocks.2.attn.relative_position_bias_table", "blocks.2.attn.relative_position_index", "blocks.3.attn.relative_position_bias_table", "blocks.3.attn.relative_position_index", "blocks.4.attn.relative_position_bias_table", "blocks.4.attn.relative_position_index", "blocks.5.attn.relative_position_bias_table", "blocks.5.attn.relative_position_index", "blocks.6.attn.relative_position_bias_table", "blocks.6.attn.relative_position_index", "blocks.7.attn.relative_position_bias_table", "blocks.7.attn.relative_position_index", "blocks.8.attn.relative_position_bias_table", "blocks.8.attn.relative_position_index", "blocks.9.attn.relative_position_bias_table", "blocks.9.attn.relative_position_index", "blocks.10.attn.relative_position_bias_table", "blocks.10.attn.relative_position_index", "blocks.11.attn.relative_position_bias_table", "blocks.11.attn.relative_position_index", "fc_norm.weight", "fc_norm.bias", "head.weight", "head.bias".
        Unexpected key(s) in state_dict: "mask_token", "rel_pos_bias.relative_position_bias_table", "rel_pos_bias.relative_position_index", "lm_head.weight", "lm_head.bias", "norm.weight", "norm.bias".

What should I do to solve this problem?

It looks like there exist some checkpoints in the directory --output_dir, such as checkpoint-799.pth.
The code will auto-reload the checkpoint for resume fine-tuning.
You may need to change --output_dir to a new directory.


fe-homework-14

Drift -> https://video.drift.com/v/abYnxFnl2lPuP4Rf04IIYztKcmDEtm55Jy1ghbcfeTH7s/ (Shown working of all)

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


[BUG]: Angular v18 support

Description
I get an error during npm install with angular v18
...
Error
npm ERR! peer<EMAIL_ADDRESS>|| ^15.0.0 || ^16.0.0 || ^17.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/@state-adapt/angular
npm ERR<EMAIL_ADDRESS>from the root project

Okay I need to test it and add it to npm for version 18. It should work fine. I think you might be able to do npm install --force in the meantime

thanks mike,
I used --force to make it work and there are no issues on my end, I can see its just a version check that needs the update.
Cheers

Finally published this in Angular 2.0.8
Also worked on some scripts to make publishing faster


VGG reduced weights

Hi, may I know how you generated the reduced weights for VGG? I understand that you followed the SSD paper and used atrous version. Did you train the model on Imagenet to get the weights?

Hey, did you find out how he generated reduced weights? On which dataset was VGG trained?


Added performance userparameter.yml on Windows

Description of PR

Type of change

Feature Pull Request
Bugfix Pull Request
Docs Pull Request
Fixes an issue

Thanks! 👍


Failed to launch astyle on format

Failed to launch astyle. (reason: "Cannot open options file ${workspaceRoot}/astylerc")
Using vscode 0.6.1 and just upgraded to Astyle extension 0.6.1.
It works if I changed to the full path. i.e /Users/XXX/Project/astylerc.

This is fixed in 0.6.2.


TPU - build a total uncertainty calculator based on the Rob Hare model

This is kind of in progress, should allow the inclusion of kongsberg/applanix rms error sources.

should be able to autodetermine static manufacturer-specific values
should be able to determine sound velocity values based on cast distribution

First attempt in 41d7cbf8092ab2d7b51d6254bedc7edfa1f6ce3c
Built on sonar uncertainty, SBET uncertainty and scalar sep model uncertainty
I suspect this is not a 100% complete interpretation of the Hare model.  The sounder uncertainty elements that
rely on attitude/sound velocity (numbers 3-6 on page 67) and the horizontal elements that are not based on the
positioning system (lines 2-4 in number 10) probably need to be included.

Kluster now features a tpu module that will generate total horizontal and vertical uncertainties (6772a2b7560e2532751336d3b6e29a3239106178).  You have to use SBET error and Sonar error as inputs, I do not currently have any functionality to build tpu from entered stdev values.


Always show type hints in docs

See https://github.com/Qiskit/qiskit-addon-obp/pull/31 for context.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
53

Relevant Lines:
53

💛  - Coveralls


Provide a method to allow reading advice from file

Adds an extra, optional command line option which allows to initialize FixedAdviceProvider to the binary content of the file. This will allow easier test framework building.

is there a test for this?
I thought there was a security concern, which is why we are only accepting input from stdin atm. We need to check more things for files to avoid security risk.

is there a test for this?
I thought there was a security concern, which is why we are only accepting input from stdin atm. We need to check more things for files to avoid security risk.

I don't see a concern about security by using file as inputs? But I do see it is possible to pipe binary file to stdin for a same effects.

I don't see a concern about security by using file as inputs? But I do see it is possible to pipe binary file to stdin for a same effects.

I guess that would be scriptable too. I don't really like any of this actually >.<

I agree this is kinda hard to test, but I also don't have any major concerns about merging this. I'm not sure what Marty's security concerns are but maybe that can be clarified.


Add action to check if GPU extensions can be built

Fixes #44

No idea why the OSX tests are failing, maybe a caching issue.
I know you're working on proper GPU CI, but in the meantime this shouldn't hurt.

jaxlib is broken on OSX at the moment. So we can just wait this out.


Unsupported DeviceType

This looks like a neat solution, but unfortunately I'm not able to start the container. When I try to do so I get the following message:
Traceback (most recent call last):
  File "run.py", line 30, in <module>
    device = get_device()
  File "/app/common/wifi.py", line 248, in get_device
    [(x.DeviceType, x) for x in Pnm.NetworkManager.GetDevices()]
  File "<string>", line 8, in GetDevices
  File "/root/.local/lib/python3.8/site-packages/NetworkManager.py", line 550, in to_python
    val = fixups.base_to_python(val)
  File "/root/.local/lib/python3.8/site-packages/NetworkManager.py", line 607, in base_to_python
    return [fixups.base_to_python(x) for x in val]
  File "/root/.local/lib/python3.8/site-packages/NetworkManager.py", line 607, in <listcomp>
    return [fixups.base_to_python(x) for x in val]
  File "/root/.local/lib/python3.8/site-packages/NetworkManager.py", line 620, in base_to_python
    return globals()[classname](val)
  File "/root/.local/lib/python3.8/site-packages/NetworkManager.py", line 348, in __new__
    klass = device_class(obj.Get('org.freedesktop.NetworkManager.Device', 'DeviceType', dbus_interface='org.freedesktop.DBus.Properties'))
  File "/root/.local/lib/python3.8/site-packages/NetworkManager.py", line 368, in device_class
    return {
KeyError: dbus.UInt32(30, variant_level=1)

I'm not very familiar with both NetworkManager and Python. Is this something that needs to be fixed in python-networkmanager? I've experienced a similar issue with wifi-connect for which I did find this issue. I've scanned my network using the nmcli on the host, and there are no WPA3 networks in the list.
Thanks for your effort and please let me know if there is anything I can check or try!

What is your setup? Are you using balenaOS? Which device type?

Yes, using Balena. I've tried with both 2.106.2 and 2.108.18+rev2 on a Raspberry Pi 4.
Here is the output of nmcli device with just an ethernet connection.
DEVICE           TYPE      STATE                   CONNECTION
eth0             ethernet  connected               Wired connection 1
supervisor0      bridge    connected (externally)  supervisor0
wlan0            wifi      disconnected            --
p2p-dev-wlan0    wifi-p2p  disconnected            --
balena0          bridge    unmanaged               --
br-1e80d1716af5  bridge    unmanaged               --
br-9c6e4fe0fd8b  bridge    unmanaged               --
resin-dns        bridge    unmanaged               --
veth2fe6a60      ethernet  unmanaged               --
veth5e5ec4c      ethernet  unmanaged               --
veth72dcb9c      ethernet  unmanaged               --
vetha72df49      ethernet  unmanaged               --
lo               loopback  unmanaged               --
resin-vpn        tun       unmanaged               --```

My guess is that is has something to do with the `wifi-p2p` device. As can be seen [here](https://developer-old.gnome.org/NetworkManager/stable/nm-dbus-types.html#NMDeviceType), this is the device that has id 30.

Yep, looks like you are running in to this: https://github.com/seveas/python-networkmanager/issues/68
You could bump Python network manager to 2.2, but then will run in to this issue: https://github.com/balena-labs-research/python-wifi-connect/issues/1
Your best bet may be to try this fork: https://github.com/seveas/python-networkmanager/pull/85

Is there also something specific about your setup that makes this interface appear? I am trying to understand how likely it is for someone to encounter this.

If I update to 2.2 I indeed get that issue, I'll try that fork later. There isn't much special about the setup, it's a regular Raspberry Pi 4. But as that's a popular board a lot of other users would probably also have this issue.
For me it would also be fine to somehow disable this WiFi-P2P device, but I haven't found a quick way to do so with Balena (without access to the sd-card). Changing the wpa_supplicant.conf isn't possible as this is a read-only filesystem. Making the device unmanaged didn't solve the issue.

There isn't much special about the setup

Perhaps there has been some kernel update, or network manager update.
You could try installing the forked version directly from GitHub by amending the requirements.txt: https://stackoverflow.com/a/47719177/16019434
To test, you could try pointing it at this guys fork: https://github.com/AndersBlomdell/python-networkmanager

This fork gave me this:
  File "/root/.local/lib/python3.8/site-packages/NetworkManager.py", line 620, in to_python
    val['ipv4']['dns'] = [fixups.addr_to_python(addr,socket.AF_INET) for addr in val['ipv4']['dns']]
KeyError: 'dns'

I've quickly forked the repo from AndersBlomdell and applied this fix. When I install the fork given your instructions (thanks!) everything works!

Nice investigation. Thanks for the fix. I am going to leave this issue open and apply it to the main repo too.


EmailPasswordAuth: Add support for callResetPasswordFunction

Closes https://github.com/realm/realm-kotlin/issues/744
This is branched off of ct/app-functions so to make this less confusing for review purposes I am making the PR with that target, but will need to be change once that is in master. This is also dependent on updating to the latest version of Core which includes https://github.com/realm/realm-core/pull/6081.

Looks like something is off with the Core commit since RealmInterop fails to compile

I think the issue is that this is still branched off of the app functions branch so it uses old core and not the latest main branch one. This will have to wait until https://github.com/realm/realm-kotlin/pull/1110 is merged after which it can be rebased and pass CI without a divergent core.


ol.style.icon - Dependency between 'scale' and 'displacement'?

Hi,
is there any dependency between the options "scale" and "displacement" for ol.style.icon?
I created an example at https://codepen.io/gott18/pen/poLpbex
As you can see, the icons should have the same Y position for the bottom of the icons, however, they change their Y position when modifying their scale. Is this the expected behavior or is there an issue in the library?
In case this behavior is expected, where is it documented? On https://openlayers.org/en/latest/apidoc/module-ol_style_Icon-Icon.html it seems to be not mentioned.
Thanks,
Thomas

Looks like it with displacement: [0, 100 / (0.2 + 0.2*markernumber)] they are aligned at the bottom. Unexpected indeed.

It also happens with regular shape and circle styles https://codepen.io/mike-000/pen/vYRpXNp  These have no anchor option so it may have been intentional.

Thanks for your responses which show that also others can see this behavior.
Could someone who is deeper in the topic (maybe the developer of this code) give a hint if this is the expected behavior, a bug, or how exactly those options should be used / combined (how the different options work together)?

@Razi91 Can you comment on this? Thanks.

The lower one is without displacement, the higher one is with.
Scaling is done after displacing, so the [0, 100] is multiplied by the scale. It could be fixed by doing @MoonE 's calculation in those classes, I could do that, but it would be a breaking change for some projects. Maybe introducing new property like absoluteDisplacement would be a better option.

For regular shapes it make sense, e.g. circle style with radius 50 and displacement [0, 50] is always anchored bottom center regardless of scale

Maybe that should be called anchor for consistency with icon style (although for regular shapes the anchor origin would always be center-center) and a separate option displacement should not be scaled?

The difference is that displacement is given in pixels, while anchor is relative to the size value. And yes, it's a bit messed up. I think we should fix it. Without scale the displacement works fine.

Anchor units can be specified as pixels, but are still scaled with the image https://codesandbox.io/s/icon-forked-ku5ly3?file=/main.js

Anchor units can be specified as pixels, but are still scaled with the image
This is very confusing for the user.

@Razi91 do you have a rough idea until when a fix / improvement will be available? (just asking, because it directly influences if I have to scale my displacement in the code or if I - maybe - will wait a few days and can directly use an improved library)

This is very confusing for the user.

No, that would be expected as anchor is typically set within or on the edge of the image.
Also note that like anchor (where it is expected) the displacement option is affected by rotation (which might not be expected) e.g. with rotation: Math.PI/2, in first example

I'm not sure if it's enough and safe, but a method in Image.js:
  getScaledDisplacement() {
    const scale = this.scale_;
    if (typeof scale == 'number') {
      return this.displacement_.map((p) => p * scale);
    } else if (Array.isArray(scale)) {
      return this.displacement_.map((p, idx) => p * scale[idx]);
    }
    return this.displacement_;
  }

to return scaled displacement, and then changing usage of getDisplacement to getScaledDisplacement in RegularShape.js (Circle.js extends it) and Icon.js should fix it.
I need to check it because right now I have other things to focus on. I also noticed that the calculation of displacement moved from getOrigin to getAnchor, it might also change its behavior, I'm not sure if I tested it with scale right now.
Well, it definitely requires defining what displacement does, that it should move icon in absolute units.

Okay, this solution works (division, not multiplication, I've edited the code in the comment), it makes displacement translating the symbol with absolute values.
I'm not sure if it should be done in RegularShape.js, or maybe there should be a different name there, now I remember I needed this feature for them, to build more complex styles with that.
Right now it acts that way:

after fix it would be:

Any suggestions about changing it, without breaking it in any way? Adding different parameters like displacementUnit, where would be by default current 'relative', and could be set 'absolute' (just like now there is for Icon properties anchorXUnits)?

I think it should be fine if one additional parameter would be added which defaults to the current behavior ('relative') and can be set to 'absolute'. This makes it smoother for existing and new projects.

Why not just clarify the meaning of displacement, saying displacement of the symbol in css pixels, and fix the behavior accordingly. Then at least for this option, it is clear what it does.

Right now for Icon we are setting anchor, with either fraction of size or pixels. This is scaled by scale. displacement should be "above" that, I mean it should be absolute, then I think the current behavior for Icon is wrong and should be fixed (and documentation should be written clearer). So in case of Icon, this is the 'relative' setting.
But what about RegularShape, as I posted in the last comment?
I just need to know how (I mean: how those parameters should work, code is easy) to fix it, and I can make PR with a fix for that

Okay, this solution works (division, not multiplication, I've edited the code in the comment), it makes displacement translating the symbol with absolute values.
I'm not sure if it should be done in RegularShape.js, or maybe there should be a different name there, now I remember I needed this feature for them, to build more complex styles with that.
Right now it acts that way:
after fix it would be:
Any suggestions about changing it, without breaking it in any way? Adding different parameters like displacementUnit, where would be by default current 'relative', and could be set 'absolute' (just like now there is for Icon properties anchorXUnits)?

To make the handling the same over all modules the behavior in your example makes sense to me. However, this could 'kill' some more complex drawings.
Just as an idea: How would it behave if we scale it 'relative without offset'. Meaning, that we scale it relatively as it is currently, but determining the lowest displacement value and subtracting this value from all displacement values. This would scale the drawing as it is currently, but removing the offset from the baseline. Could this be a solution or do I miss a huge drawback?

@Razi91 The behavior for RegularShape should be the same. The difference between Icon and RegularShape is only that the former is an external graphic, wheres the latter is created by OpenLayers.

@thn80 You know the scale, because you configure it yourself.  So you can easily multiply the displacement by the scale on the application level, if you so wish.

That's true, but as @Razi91 also mentioned a 'displacement' is not assumed to be relative. If something like 'displacementUnit' would be added which defaults to the current behavior 'relative' it would from the usage be similar to anchor and would also be backwards compatible with existing code.

@thn80 Exactly. Displacement as I defined it above (i.e. displacement in css pixels) is not relative. We don't need to be backwards compatible - we'll be releasing v7.0 next, and it is confusing to have two similar properties (anchor and displacement). So let's fix this rather than adding more confusion.

A further observation is that pixel measurements are generally made from top left, so a positive Y value is downward (as it is in ol/style/Text offsetY property).  However a positive Y value for displacement currently represents an upward displacement.


About to start working on Learn course

I'm about to start working on a course for the MSFT Learn platform.  I'll likely need some art elements, though I have no idea what they might be.  Are you interested, or should I contract them out?

Is this still something you might need @kiki-lee ?


Adding v0.2.0 mds

The git rev is pointing at master, not the tag

Good catch. I manually typed that in to be 'master' too. Whoops. Meant 'v0.0.2'


"Operation Unavailable" error - win32com

Hey,
Upon configuring the config.ini file and starting the main.py file, I get an error. The screenshot of the error can be found below:

One note, I extracted the SDK into the project root directory and ran the installer there, I wonder would that be something that could have kickstarted this issue.
Additionally, this is my config.ini:

Thanks in advance!
Supra

could you try running both as non-admin in case one of them are running as
administrator?
On Sat., Feb. 5, 2022, 7:20 a.m. supraaxdd, @.***>
wrote:

Hey,
Upon configuring the config.ini file and starting the main.py file, I get
an error. The screenshot of the error can be found below:
[image: image]
https://user-images.githubusercontent.com/54616428/152641776-31dd7a52-3b2a-434b-a40d-a38ce973d996.png
One note, I extracted the SDK into the project root directory and ran the
installer there, I wonder would that be something that could have
kickstarted this issue.
Thanks in advance!
Supra
—
Reply to this email directly, view it on GitHub
https://github.com/vidhanio/virtualbox-rich-presence/issues/15, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ4FDENL3ZZZ5KRMJ2TXTYTUZUIXJANCNFSM5NT4FELQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

What do you mean by running both? py main.py or?

Sorry, I meant the python script and virtualbox itself. too early in the morning, haha

Same thing is happening, regardless of whether I have it on Admin or not


#161488544 Create a db-powered endpoint for an administrator to update other users' roles

What does this PR do?
This PR creates the endpoint for store owners update the roles of other users to administrators
Description of Task to be completed
The main tasks to be completed are:
Creation of an endpoint for store owners to update other users
Connect it to the database
How should this be manually tested?

Create  a virual enviroment with the command
$ virtualenv -p python3 env

Activate the venv with the command
$ source env/bin/activate

Install git
sudo apt install git

clone this repo
$ git clone https://github.com/codeMarble254/Store-Manager-v1.git"

cd into the folder Store-Manager-V1

export required enviroment variables
$ export FLASK_APP=run.py
$ export DB_NAME="storemanager"
$ export DB_HOST="fill in your localhost"
$ export DB_USER="fill in the db user"
$ export DB_PASSWORD="fill in your db user password"
$ export SECRET_KEY="<your key>"

create the dev database
createdb storemanager

create the testing database
$ createdb test_db

Switch to the current branch
$ git checkout ft-update-user-v2-161488544

install requirements
$ pip install -r requirements.txt

now we are ready to run.
8.now we are ready to run.
for tests
export APP_SETTINGS="testing"
pytest app/test/v2 -v --cov=app/test/v2

for the application run
export APP_SETTINGS="development"
$ flask run
If you ran the application you can test the  api end points using postman as follows

Endpoint
functionality
contraints(requirements)

put /api/users/userId
Update users
User authentication

What are the relevant pivotal tracker stories?
#161488544

Pull Request Test Coverage Report for Build 136

253 of 255 (99.22%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.0%) to 99.216%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/test/v2/test_users.py
83
85
97.65%

Totals

Change from base Build 67:
2.0%

Covered Lines:
253

Relevant Lines:
255

💛  - Coveralls

Pull Request Test Coverage Report for Build 136

253 of 255 (99.22%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.0%) to 99.216%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/test/v2/test_users.py
83
85
97.65%

Totals

Change from base Build 67:
2.0%

Covered Lines:
253

Relevant Lines:
255

💛  - Coveralls


[CDR-1466] Fix AQL on Media file cluster with inline data

Changes
TBD ...
Pre-Merge checklist

[ ] New code is tested
[ ] Present and new tests pass
[ ] Documentation is updated
[ ] The build is working without errors
[ ] No new Sonar issues introduced
[ ] Changelog is updated
[ ] Code has been reviewed

@alexlehn - https://github.com/ehrbase/ehrbase/pull/1380/files#diff-f3bf042f595bb7cd2f582786da0bb18ee5da158fbda5d21be1d5cbc72096c296R47 - to be changed back to latest tag, before merging to develop branch.


Mobile routing

I was going to go for something more elaborate with routes, but it was a mess to carry the query string between them and redirect away on desktop. This makes the Covince super component less necessary but it's still a good place to add things that the app definitely needs.

Nice!

LGTM!


Declare detect_encoding(…) as inline

Without marking this function as inline, I get duplicate-symbol errors at link-time for the compiled artifacts of any code that had made use of IOD. This one change fixed the problem.

Thank you @fish2000

My pleasure @matt-42, yes!


localStorage Access

I really have no clue what it would actually take to get this implemented but it looks like it's supported on Selenium.
https://github.com/SeleniumHQ/selenium/blob/trunk/java/client/test/org/openqa/selenium/html5/LocalStorageTest.java#L42
It would be awesome to have access to that to help test bad login issues.

Looks like there's commands for accessing it https://github.com/SeleniumHQ/selenium/blob/da5331b162b65c70b6367d789c1d2bc1d2114a48/java/client/src/org/openqa/selenium/remote/html5/RemoteLocalStorage.java
This must not be part of the official webdriver spec.
Ruby implementation https://github.com/SeleniumHQ/selenium/blob/64447d4b03f6986337d1ca8d8b6476653570bcc1/rb/lib/selenium/webdriver/common/driver_extensions/has_web_storage.rb


KeyError: 'defaultnamingcontext'

Hello,
I just install the tool with pip but i keep getting this error :
ldapdomaindump ldap://<IP_ADDRESS>:389                                      20:24:08
[*] Connecting as anonymous user, dumping will probably fail. Consider specifying a username/password to login with
[*] Connecting to host...
[*] Binding to host
[+] Bind OK
[*] Starting domain dump
Traceback (most recent call last):
  File "/home/bonclay/.local/bin/ldapdomaindump", line 3, in <module>
    ldapdomaindump.main()
  File "/home/bonclay/.local/lib/python2.7/site-packages/ldapdomaindump/__init__.py", line 892, in main
    dd = domainDumper(s, c, cnf)
  File "/home/bonclay/.local/lib/python2.7/site-packages/ldapdomaindump/__init__.py", line 158, in __init__
    self.root = self.getRoot()
  File "/home/bonclay/.local/lib/python2.7/site-packages/ldapdomaindump/__init__.py", line 171, in getRoot
    return self.server.info.other['defaultNamingContext'][0]
  File "/home/bonclay/.local/lib/python2.7/site-packages/ldap3/utils/ciDict.py", line 68, in __getitem__
    return self._store[self._case_insensitive_keymap[self._ci_key(key)]]
KeyError: 'defaultnamingcontext'

Do you get the same error when you supply a username and password? And are you running this against an Active Directory LDAP server?

my bad, I was using the wrong ldap server ^^ working fine now! thx

@mpgn Could you tell us how did you solve it ? I still get the same error as @fasetto

me too.


Taking 3-4 minutes per result on LinkedIn

All the other websites work great! However, when doing searches on Linkedin, it takes 3-4 minutes per result. Does anyone else have the same issue?

try updating, should be 1 minute max for 100 results

Great, it's mostly working now. The only time it hangs now is if you set a job criteria that is too focused and doesn't have as many results as you specify in results_wanted
For instance
jobs= scrape_jobs(
    site_name=["linkedin",],
    search_term="dancer on a boat",
    location="Ontario",
    country_indeed="canada",
    results_wanted=30
)

ah nice catch, see if it's good now

It's working great now, thanks for updating.


serializers: added xml transformer

closes https://github.com/inveniosoftware/invenio-swh/issues/20

Closing this PR now, we don't want to add anything in flask-resources yet. The serializer refactoring (E.g. adding the formats in this PR) should be done in a bigger task that involves more design.


[Android] How to keep data stored after application updates?

Sometimes after a device is updated the data is being deleted from SQLite. How can I ensure this doesn't happen?

Sounds like the app data was cleared. That's normal for an android app so you should be storing any data on a server, or use the android data back up api on your database.


feat: kpm sample app

I added a simple barebones KPM sample app that currently only utilises the kpm sdk on the Android side
Features include:

Capturing message
Capturing exception
Manual & Auto init

@buenaflor

Manual & Auto init

How do you configure that? How would you do the manual init? I guess this is not done or not working yet.

@buenaflor

Manual & Auto init

How do you configure that? How would you do the manual init? I guess this is not done or not working yet.

The manual init is done by extending the Application and calling the init manually as per documentation on Sentry. So calling Sentry.init(...) in MainActivity will call the Sentry.init(...) function in shared/androidMain/.../Sentry and in this function it will call the SentryAndroid.init(...) from the KMP SDK.  This is only a barebones implementation so I will have to extend it so options can also be set.
public class SentryApplication extends Application {
  public void onCreate() {
    super.onCreate();
    Sentry.init("dsn_key") { this }
  }
}


support opening a Pod terminal for another container than the first

Describe the problem/challenge you have
The terminal view in octant is awesome. However it is currently hard coded to only support the first container:
https://github.com/vmware-tanzu/octant/blob/master/internal/modules/overview/terminalviewer/terminalviewer.go#L56-L59
It would be nice to be able to open a terminal in any container. Sometimes the first container doesn't have a shell binary like sh/bash (e.g. tekton pipelines)
Describe the solution you'd like
it would be awesome if there was a way to pass in the container name to open a terminal via a URL path or something; so that either octant itself or a plugin could link to a pods terminal page for a named container. If no name is specified defaulting to the first container sounds a reasonable thing to do.
e.g. on the main Pods page there could be a terminal link on each Container so that folks can decide which container they want to use to open the terminal for a Pod
BTW I tried opening terminals in a plugin and nothing seemed to work; I've a feeling terminals only work inside octant itself? I see this if I try:
api/content_manager.go:130  generate content  {"client-id": "63f1e32c-8fb0-11ea-934d-acde48001122", "err": "generate content: unmarshal content response: unknown view component \"terminal\"",

A use case for this is when using Tekton; you often want to open a terminal in the last container (as earlier containers may have terminated).

Closing this as a duplicate of #901
cc: @GuessWhoSamFoo

Linking to a terminal is something discussed in the past. The way angular handles routing currently makes this difficult. We can revisit this as this is a reasonable feature to support.


investigate: autocomplete not working if cursor at end of document

fixed: another feature was interfering with it


Only publish required files

.github,spec, and .eslintrc.yml are currently shipping with the node modules. This should remove them.
--edit --
Wasn't able to find it in the npm docs, will types be white listed like main?
Also just saw https://github.com/epoberezkin/json-schema-traverse/pull/8, a duplicate, but might be outdated now.

Coverage remained the same at 100.0% when pulling 988031daaa5009f59127c100db0ef43e1a56386c on willfarrell:patch-1 into 6b45983cd76270042cc79527da5c8972f13599ec on epoberezkin:master.

Coverage remained the same at 100.0% when pulling 988031daaa5009f59127c100db0ef43e1a56386c on willfarrell:patch-1 into 6b45983cd76270042cc79527da5c8972f13599ec on epoberezkin:master.

Wasn't able to find it in the npm docs, will types be white listed like main?

I believe so - I’ll need to check after it’s released...

Wasn't able to find it in the npm docs, will types be white listed like main?

I believe so - I’ll need to check after it’s released...

https://docs.npmjs.com/cli/v6/configuring-npm/package-json#files
They don’t mention types here though - probably better to whitelist it

https://docs.npmjs.com/cli/v6/configuring-npm/package-json#files
They don’t mention types here though - probably better to whitelist it

done

done

I just found the same issue and realized there is already a PR with the fix.
However, this was never published to npm. The last publish was 1.0.0 in December 2020.


HCL8

Use the preview tab to see the example given below. Edit the relevant information and preview the changes before submitting. You may delete these instructions.

Key
Value

ID
HCL8

Source
Vaaran Bhai Gurdas Ji

Page
26

Line
lKmI dwshu DrmcMd; poqw huie kY Awpu gxwieAw [

Correction
THIS ≠ THAT

PROOF (EDITION)
Image:

Correction:
lKmI dwshu ≠ lKmIdwshu
Proof:
Vaaran Bhai Gurdaas Ji | November 2011 | Shamsher Singh Ashok | SGPC, Sri Amritsar
Image:


[mod_cgi] consolidate CGI cleanup code

(more consistent behavior)
reference:
"Race leads to incorrect output in some circumstances"
https://redmine.lighttpd.net/issues/2542

Thanks! Applied in dde7bea99a259ab760c1eadab01ea0ea6f5bdb6f.

@stbuehler, there is a whole lot more in this patch than just the last commit of the 4 commits in this branch.  Would you take another look?  Would you rather me squash them all together?

Sorry, I only listed the final commit (the first of these four was #30 anyway); the other two were applied in e5e66f791f43e64c3b3d82c05aad8f6e845dd29a and f2cbd0a3aaf3db34404de35919397a7e15dccaca) - with some minor changes by me.

oh, I see.  Thanks!


Migrate some release info from blog to wiki

We have a pattern at Snowplow where the blog post documentation is much better than the wiki doco as an example, the weather enrichment blog post is far better than the weather enrichment wiki page. I think it would be amazing if you could spend time pulling content from the release posts back into the wiki
-- @alexanderdean

moved to docs, closing


[SPARK-40579][PS] GroupBy.first should skip NULLs

What changes were proposed in this pull request?
make GroupBy.first skip nulls
Why are the changes needed?
to fix the behavior difference
In [1]:
   ...: import pandas as pd
   ...: import numpy as np
   ...: import pyspark.pandas as ps
   ...:
   ...: pdf = pd.DataFrame({"A": [1, 2, 1, 2],"B": [-1.5, np.nan, -3.2, 0.1],})
   ...: psdf = ps.from_pandas(pdf)
   ...:

In [2]: pdf.groupby("A").first()
Out[2]:
     B
A
1 -1.5
2  0.1

In [3]: psdf.groupby("A").first()

     B
A
1 -1.5
2  NaN

Does this PR introduce any user-facing change?
yes, updated GroupBy.first will skip NULLs
How was this patch tested?
added UT

Merged to master.

Thanks you @HyukjinKwon @dongjoon-hyun @itholic


updated switch case example

updated switch case example

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi thanks for the PR. Can you please make separate PRs? Why are you editing the client code, when your PR is otherwise.


Color correction now uses a 3D texture.

Breaks games that use the existing color correction.
demo usage:
https://gfycat.com/mammothlivelyblacklab
Model from Desmond Man

Would it be possible to implement this in a way that still makes GradientTexture usable for this? (Or perhaps there should be a 3D GradientTexture in a future version?)

@CptPotato I implemented your change, but in order to co-author you I need your email :<

@QbieShay You can leave the commit as is if you'd like. I'd prefer to keep my mail address private.
Though, make sure to test the change as I haven't done that. It should be really obvious with a neutral 2³ or 3³ texture.

@CptPotato Will test this evening, thank you very much for the tip!

@QbieShay Did you test?

@fire I did! Works!

There was some difficulty converting a png lookup table to a 3d texture. I have to do 16 horizontal and 1 vertical. It would probably be better to support a .cube to 3d texture directly, but it works for me too!

@QbieShay Is this still desired? If so, it needs to be rebased on the latest master branch, and reworked to support Vulkan.
If not, abandoned pull requests will be closed in the future as announced here.

I still desire this. It required some engine investigation though.

Hi @aaronfranke ! I haven't looked into the Vulkan renderer yet. I'm sad that this PR didn't make it into 3.x, but I expected it due to compat problems. I'll close this PR and maybe open another one for Vulkan in the future!


Add the ability to increment retry counts

Description
Currently, we are facing SOs installation issues,  the error messages is : Cannot update or delete a picklist while a picklist background job is being executed. The current command automatically retries installation of a package on a failed deployment (when "retryOnFailure"is set to true) which normally solves the problem. However, it seems in this scenario more default retry count is needed.
Therefore, I would like to request a new feature that allows users to specify the number of retries for failed deployments when using the sfpowerscript command. This would enable users to increase the retry count beyond the default 2 attempts and would help to avoid package installation issues we currently having with our SOs.
Proposed solution
Add a new parameter to the sfpowerscript command that allows users to specify the number of retries for failed deployments. For example, a new parameter called "retryCount" could be added in the config files, which accepts an integer value indicating the number of retries to attempt.
Screenshots

Platform Details
Operating System: Windows 10
sfpowerscript version: 20.20.0
Salesforce API version: 53.0
This feature would greatly improve the deployment process and make our validation process faster. Thank you for your consideration.

@Cherie-she  Scheduled for Feb 23

Added a maxRetryCount  to pool config that will help with similar issues
https://github.com/dxatscale/sfpowerscripts/blob/a5e42d63926b0ac6f888e81038e227edb5b26e63/packages/core/resources/pooldefinition.schema.json#L95


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_0d868l6ian/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_0d868l6ian/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_0d868l6ian/clone/recipe/meta.yaml:26: missing_pip_check: For pypi packages, pip check should be present in the test commands
Errors were found


Usage of NodeJS 18 LTS

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
Other
🔖 Feature description
The install guide for non-Docker installs in the wiki recommends NodeJS 14 or higher, but the easy install script installs 14 when using the non-Docker way.
NodeJS 14 LTS will be End-of-Maintenance in April 2023, not that far off.
Are there major headaches to be expected when using NodeJS 18 LTS today?
✔ Solution
Usage of NodeJS 18 LTS
❓ Alternatives
No response
📝 Additional Context
No response

Uptime Kuma should be working fine on Node.js 18, you can update your Node.js manually.

Uptime Kuma should be working fine on Node.js 18, you can update your Node.js manually.

In that case, are there plans to refresh the install script to use 18 going forward?

Node18 would be great. And if you update install scripts / docker containers, please consider using the user node and not root inside the container! (Security)

If you get the error:0308010C:digital envelope routines::unsupported on website running on an old IIS+.net with node 18, I found that running NODE_OPTIONS=--openssl-legacy-provider pm2 restart uptime-kuma --update-env fixes the error.

@RalfSoft

And if you update install scripts / docker containers, please consider using the user node and not root inside the container! (Security)

This is one of the features slated for the v2 release.
See #2720

Installation script has been sunset and node 14 is tracked in #3746
=> closing as resolved


Fix typos

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @mosheavni !
LGTM.
@noam-codefresh ?


Use go-control-plane's NewServer() and SnapshotCache system to improve ADS

Leverage go-control-plane's SnapshotCache.
The snapshot cache allows us to set a single-versioned snapshot of all xDS responses (for each proxy) in a single location, while the snapshot cache implementation maintains all state and processes for responding to the actual gRPC requests. It allows us to not worry about the implementation details of delta, vs aggregate updates, streams vs fetch, etc.
From @shashankram: we have had issues in the past with the state logic for responding to proxy Stream Resource requests, and it is very difficult to debug. Offloading this to an upstream implementation should lighten our load substantially

incoming WIP pr this afternoon!

btw a working branch for this has been avaialble since immemorial times (https://github.com/eduser25/osm/commit/f4de97bae1d91445978e6fc9d3939eca703606ee), devel and exploration has been put on hold as there are some shortcomings on the ADS cache implementation of go-control-plane that have to be properly addressed first.

@eduser25 could update this issue with details on "here are some shortcomings on the ADS cache implementation of go-control-plane" ?

context: https://github.com/envoyproxy/go-control-plane/issues/399

Update:
Initial SnapshotCache implementation support is merged https://github.com/openservicemesh/osm/pull/3530
The issue with go-control-plane is being acknowledge and worked on: https://github.com/envoyproxy/go-control-plane/issues/431

I think we should look to start this up again. I found some issues in the current xDS eventing system that can lead to: signficant delays in updates to all proxies due to a single slow proxy (head of line blocking)
The pubsub channel is a 0 capacity queue. The first message gets pulled off an internal queue immediately, to begin sending out to each subscribers queue (also 0 capacity). The second publish may block, if a single queue is slow, which will queue up all prior update messages across the entire system.

@steeling Can we use this issue as the root of subtasks of snapshot cache?

@shashankram moving the conversation about the snapshot cache here.  Looking at the github description https://github.com/envoyproxy/go-control-plane, I'm not entirely sure the snapshot cache is smart enough to do internal diffs to determine when a change is needed. Instead, we set a new version on each update (even if configs haven't changed), and I believe the snapshot cache will push out the requested changes.
So in that sense the snapshot cache will not reduce the load on the osm-controller. Happy to be wrong about this, but that's my understanding upon some brief reading

@shashankram moving the conversation about the snapshot cache here. Looking at the github description https://github.com/envoyproxy/go-control-plane, I'm not entirely sure the snapshot cache is smart enough to do internal diffs to determine when a change is needed. Instead, we set a new version on each update (even if configs haven't changed), and I believe the snapshot cache will push out the requested changes.
So in that sense the snapshot cache will not reduce the load on the osm-controller. Happy to be wrong about this, but that's my understanding upon some brief reading

Yes, I believe that's fine because the Envoy instance will compute the diff anyway. In that regard, the value of using the snapshot cache is to avoid using our controller implementation of the XDS state machine and let the cache deal with it. We've seen quite a few bugs in the past with the XDS state machine implementation in the controller, and using the snapshot cache will simplify what we need to worry about after generating the config. Moreover, I see value in simply writing the config to the cache and forgetting about what happens after that, which isn't the case currently.

@shashankram moving the conversation about the snapshot cache here. Looking at the github description https://github.com/envoyproxy/go-control-plane, I'm not entirely sure the snapshot cache is smart enough to do internal diffs to determine when a change is needed. Instead, we set a new version on each update (even if configs haven't changed), and I believe the snapshot cache will push out the requested changes.
So in that sense the snapshot cache will not reduce the load on the osm-controller. Happy to be wrong about this, but that's my understanding upon some brief reading

Yes, I believe that's fine because the Envoy instance will compute the diff anyway. In that regard, the value of using the snapshot cache is to avoid using our controller implementation of the XDS state machine and let the cache deal with it. We've seen quite a few bugs in the past with the XDS state machine implementation in the controller, and using the snapshot cache will simplify what we need to worry about after generating the config. Moreover, I see value in simply writing the config to the cache and forgetting about what happens after that, which isn't the case currently.

agreed! curious on your opinion on once we migrate to snapshot cache, whether we should keep the messaging.Broker, to know to trigger config generation on changes, or to purely rely on a continuous loop to constantly trigger config generation

@shashankram moving the conversation about the snapshot cache here. Looking at the github description https://github.com/envoyproxy/go-control-plane, I'm not entirely sure the snapshot cache is smart enough to do internal diffs to determine when a change is needed. Instead, we set a new version on each update (even if configs haven't changed), and I believe the snapshot cache will push out the requested changes.
So in that sense the snapshot cache will not reduce the load on the osm-controller. Happy to be wrong about this, but that's my understanding upon some brief reading

Yes, I believe that's fine because the Envoy instance will compute the diff anyway. In that regard, the value of using the snapshot cache is to avoid using our controller implementation of the XDS state machine and let the cache deal with it. We've seen quite a few bugs in the past with the XDS state machine implementation in the controller, and using the snapshot cache will simplify what we need to worry about after generating the config. Moreover, I see value in simply writing the config to the cache and forgetting about what happens after that, which isn't the case currently.

agreed! curious on your opinion on once we migrate to snapshot cache, whether we should keep the messaging.Broker, to know to trigger config generation on changes, or to purely rely on a continuous loop to constantly trigger config generation

I don't think we should have a continuous loop to trigger config generation, but rather a combinarion of event driven approach with coalescing and the addition of a periodic reconciler such as Ticker if necessary to recover from transient inconsistencies in the system. Constantly generating config in a tight loop when there's no change in the system has the downside of both controller and Envoy operating in a tight loop all the time at 100% CPU. This is especially evident in scale environments with 1000s of pods.
The broker is really meant to provide a streamlined mechanism for event pub-sub, with additional metrics and checks in place regarding which events are broadcast, multicast, or unicast . The broker is still necessary in that regard given the snapshot cache isn't smart enough to drop events it doesn't need.

@steeling Does #5056 close this issue?

Yes it does!

Fixed in #5056


Modified The CSS, changed the font, added transitions

Description 📜
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
Fixes # <issue_number_here>
upgrade the css , fonts , card transitions , and color of text

This would be great if you provide any of this   📝

[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[ ] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Navbar design
[ ] Footer
[x] Single page website
[x] Animation design
[ ] parallax effect design
[x] Others

Checklist ✅
]

[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm Hacktoberfest contributor

Any tutorials or documentation have you added
(made by you only not from google)

[ ] Tutorial
[ ] Documentation

Screenshots / Gif (Compulsory) 📸

conflicts is there @ishan-pf pls resolve that

i resolved the conflicts


Noble Numbat stemcell no longer uses runit to launch the agent

Currently the bosh-agent is started using a separate process manager runit.
This issue proposed removing runit in favor of the existing process manager on Ubuntu, systemd.
Having a single process management system allows that system to optimize startup across all processes in the system hopefully speeding up the time-to-ready for the whole system.
This is related to though not dependent on replacing monit (also with systemd).

As discovered by Ramon: the agent also invokes runit directly to start monit: https://github.com/cloudfoundry/bosh-agent/blob/main/platform/monit_retryable.go#L20
This should become configurable similar to: https://github.com/cloudfoundry/bosh-agent/pull/324

We should consider what we want to do with bosh-lite too. We could keep runit just on the warden stemcells or we could come up with an alternative (since systemd doesn't run under warden stemcells)

in the meantime i have removed the runit units for monit and the agent and converted them to systemd services
https://github.com/cloudfoundry/bosh-linux-stemcell-builder/commit/c27910fdf1ee48187ae767edb27df3f5d666b17b

This is still open because of https://github.com/cloudfoundry/bosh-linux-stemcell-builder/issues/352. Or @ramonskie is there anything else open for this topic?

yes its still open due to warden bosh-lite. as runc is still installed currently also to support chpst commands used in most bosh releases

can be closed after we merge #397

https://github.com/cloudfoundry/bosh-linux-stemcell-builder/pull/397 has been merged.


Address issues with windows AMD K10 architecture - Build with clang for portable SHA256

Use clang on windows and a target git reference for BLST to force BLST to compile SHA256 into portable mode.
This addresses the problem with AMD K10 architecture on Windows.

close and re-open for CI

close and reopen


CODE RUB: InvalidConfigurationAIModelException Class Update v2.10.1

Updated InvalidConfigurationAIModelException class by removing constructors in which the error message was hard coded in. Updated all relevant service files to include message.

Fixed Branching issue. All relevant updates can be reviewed in the following Pull Requests:
#507
#508
#509
#510
#511
#512
#513
#514
#515


Adds Gondolin to Prompt Themes section

I've added my personal configuration for anyone interested in a prompt with extensive git support

Thanks!


2D Tracker

Like a slider, but for tracking scroll in x & y direction
@bind x, y Tracker(1:50, 1:50)

to create a canvas with given ranges
https://github.com/fonsp/PlutoUI.jl/issues/9#issuecomment-692298456

Let's call it Slider2D?

HTML+CSS will look much better than a canvas for this


Archetypes

Instead of creating entities, adding components and setting the values by hand in the editor we should create "Archetypes" / "Prototypes" that clump up a series of component together with predefined field values, so that they can be easily created with one click.
For example, a Static Model archetype could have ModelInstance, ColliderTag, StaticTag and components.
These archetypes could be stored as scenes inside .arch files inside resources/archetypes and loaded by editor, when user clicks on Add Entity.
Later, that archetype can be used to create a "Prototype" - .prot, an instance of archetype with concrete model, material and collision shape assigned to it.
The difference between archetypes / prototypes and Unity's prefabs is that once archetypes changes (component is removed or it's field is changed), it's instances will not be updated.
Keeping track of it and referencing prefabs is probably too much work for this project, whereas archetypes are quick and messy "copy and pastes", but easy to implement.
Archetypes / prototypes can be saved just like scenes, but with restriction, to have only one entity.
User can navigate to archetypes folder inside Content Browser and click on a archetype to create a new prototype scene.
User can navigate to prototypes folder inside Content Browser and click on a prototype to instantiate it inside active scene.
We save archetypes by clicking Save as archetype... inside menu bar.

Maybe archetypes and prototypes can be the same thing. Fundamentally they have same functionalities, but they differ in usecase, for ex. archetype is "idea" mesh with collision, prototype is concrete realisation of that idea with concrete mesh and collision shape selected.
Perhaps we can reuse their logic in code and keep the difference merely in the extension of file and where it's located?

I added some TODOs inside imgui_menu_bar().
@Kurogamin will do them.
https://github.com/cladur/silence/blob/8011d6b212c43e5837a5995f4732b41868d6aaa4/editor/gui.cpp#L31
In the end, menu bar should only have one "Save as...". "Save as archetype / prototype" won't be necessary.
We also can't allow archetype / prototype scenes to create entities.
When creating archetype scene, add one entity into it by default.
In imgui_scene() check if scene is archetype, and if so, don't display "Create Entity" button.

This feature is almost finished.
The only thing left to do is to fix loading of the prototypes.
Currently, if you create a prototype and then drag and drop it onto a scene, it crashes the program.


Base32-encode only invalid runc format substrings

Also, prepend "bpm-" to the container ID.
Close #109

Thanks for the PR! I should have time to review it next week.


feat: update chat ui to match mockup

Changes

change the chat UI to match mockups

Known Bugs/Differences from mockups to be addressed later

bubble.dart does not allow for outlines, this means that the chat bubbles were filled in with colour rather than just outlined. We may need to make a custom bubble class to fix this
Several logos are different than the mockup, waiting for the logos from @laiiibah
The user profile picture was moved to above the user name
The new match's ano is the incorrect ano, waiting for the correct ano from @laiiibah
The bracket is missing from the top right of the chat
The new matches should have a horizontal fade in/fade out to prevent the harsh clipping against the ano

UI

Also, how do I test this PR? I pulled it locally and the chat screen is empty. I'm assuming it's because I have no dummy data?

Also, how do I test this PR? I pulled it locally and the chat screen is empty. I'm assuming it's because I have no dummy data?

you may have to manually mock a match on Firestore, look at some of the other matches as an example


Question regarding signature verification

Hi,
I'm trying to implement Trezor Connect on my blog. For that, I need to verify a signature of a message to check that it was generated by a given Trezor device (thus, private key).
I have tried to use the parameters provided for testing from here: https://github.com/trezor/connect/blob/gh-pages/examples/server.py
And my (stripped down) code as follows:
string visual_challenge = "2015-03-23 17:39:22";
string random_challenge = "cd8552569d6e4509266ef137584d1e62c7579b5b8ed69bbafa4b864c6521e7c2";
string signature = "20f2d1a42d08c3a362be49275c3ffeeaa415fc040971985548b9f910812237bb41770bf2c8d488428799fbb7e52c11f1a3404011375e4080e077e0e42ab7a5ba02";

SHA256 sha = SHA256Managed.Create();
var hiddenChallenge_Sha = BitConverter.ToString((sha.ComputeHash(TrezorUtils.HexStringToByteArray(random_challenge)))).Replace("-", "").ToLower();
var visualChallenge_Sha = BitConverter.ToString((sha.ComputeHash(new ASCIIEncoding().GetBytes(visual_challenge)))).Replace("-", "").ToLower();

string message = hiddenChallenge_Sha + visualChallenge_Sha;
var m2 = TrezorUtils.HexStringToByteArray(message);
PubKey pubKey = new PubKey("023a472219ad3327b07c18273717bb3a40b39b743756bf287fbd5fa9d263237f45"); //new PubKey(ConfigurationManager.AppSettings["TrezorPublicKey"]);
bool verified = pubKey.VerifyMessage(new ASCIIEncoding().GetString(m2), TrezorUtils.ConvertHexStringToBase64(signature));

The call to pubKey.VerifyMessage always returns false with this code. I have verified that the SHA256's returned are identical to what the Python code generates for the same values, as well as the base64 of the signature. So all the input to the VerifyMethod is the same as in the Python example. However, verification of the signature still fails.
Any ideas?

I just performed a byte-for-byte comparison of the messages passed in to the pubKey.VerifyMessage and the what the Python example uses (see my test here: http://goo.gl/VkAea8). They are, indeed, the same.

I am verifying with bitcoin core verifymessage rpc

mmh bitcoin core verifymessage from rpc returns me also false... VerifyMessage from NBitcoin replicate Core fuctionalities, I'm trying to make your code pass in another way

Here's my TrezorUtils class if you need it:
public static class TrezorUtils
{
    public static byte[] HexStringToByteArray(String hex)
    {
        int NumberChars = hex.Length;
        byte[] bytes = new byte[NumberChars / 2];
        for (int i = 0; i < NumberChars; i += 2)
            bytes[i / 2] = Convert.ToByte(hex.Substring(i, 2), 16);
        return bytes;
    }

    public static string ConvertHexStringToBase64(string hexString)
    {
        byte[] buffer = new byte[hexString.Length / 2];
        for (int i = 0; i < hexString.Length; i++)
        {
            buffer[i / 2] = Convert.ToByte(Convert.ToInt32(hexString.Substring(i, 2), 16));
            i += 1;
        }
        string res = Convert.ToBase64String(buffer);
        return res;
    }
}

I don't manage to make your sig work, neither with bitcoincore, nor by using Bouncy castle directly... can you try to make verification pass with bitcoin rpc verifymessage ? did not managed to.

Your code seems correct, and the format used of the signature and message (from https://github.com/trezor/connect/blob/gh-pages/examples/server.php) match with bitcoin core verifymessage implementation and NBitcoin VerifyMessage you are using.
My sole explanation is that the example they give have a wrong signature, is that possible ?

You can verify with coinig.com/ I converter pubkey in address, and it does not check.

Well, even in code, if I check the signature with the pubkey directly (not using the address) it does not pass...  I don't think they are using a good signature, can you ask them about it ?

I just tried with the values from my Trezor, using the same challenges as before.
The public key is 02fb9ea3f2388128cf2001170795e7a05936f61427955e8d4f9c82ae6eb29c8522
and the signature comes out of the Trezor as 1feef0718b6c5d9331de4cb5eb54085d19e387da29df9bb339248160738472013419c4916f0a65c656f4e73daed6034fea94d491c107723edeb2045929b5239fea
This signature is returned from the Trezor. It's all a bit weird if you ask me :) I'm converting the sha array to ASCII bytes, giving all sorts of weird characters. The example Python script gives me the exact same message byte sequence as my C# code. I'm kind of thinking it's something to do with how I convert the byte array back to ASCII before passing it into VerifyMessage...

you are just using the python script of trezor documentation... trying with your value one moment

Are you using same visual and random challenge ?

Yes

Just checked the ascii values of the message from my code against the python version and they come out identical again, with the new signature and public key.

No more luck with your sig  :/

This is too weird! Funny thing is my Trezor works fine when I log in to coinmap.org, so I know the functionality should work :)
How are you testing the signature on my data? The message results in all sorts of funny byte sequences that aren't that easy to enter into the form on coinig.com?

So basically, verifymessage from bitcoincore, NBitcoin, and coinig wants the same thing: Your address, the message you are signing, and the signature, and tell you if it is ok.
The message you are signing is the concatenation of the 2 hashes, and the signature is the base64 string of the sig.
For having the address you need to convert the pubkey to an address: pubkey.GetAddress(Network.Main).ToString()

To be sure the problem was not in the "conversion to address", I modified VerifyMessage of NBitcoin where I did the conversion, into a version where I did not. And still, it does not validate.

Can you give me an entire new set of pubkey, sig, and challenges, without special character (like punctuation) for the visual challenge ?

Sure, just a sec :)

visual challenge: 123456
hidden challenge: abcdef
pubkey: 02fb9ea3f2388128cf2001170795e7a05936f61427955e8d4f9c82ae6eb29c8522
signature: 205222527c10407211576b15ef59ddd1a83eae4cd25ef1680d5484ac2583ecbade47f86609463dc5ea2d1de08ccc07516e7650e96a14261d262efca38f26d9c576

Oh wait... something is weird with the pubkey I'm using it seems. Just give me a sec.

Hmm weird, the Trezor returns the public key I posted, but the key it's actually using to sign with is another one.
When I do a GetAddress on the pubkey I posted I get 1FhoECMfPK4XU1jELuxxtSb8HovBdrQi4n
But the key in the returned signature is 1ABJsytP6iJkbHiiUbodVLA7LANxa9AhCD, according to coinig.org.
Guess I'll have to dig a bit and find out what's going on...

Thanks for your help so far! :)

Well, I have the same result with NBitcoin that coinig.org, and you can try to validate with bitcoin core, will be same error... but the thing that is surprising, is that I also have the error on the sample data of trezor in their server implemenation.
Can you ask to trezor if it is possible to validate with verifymessage of bitcoincore or coinig ? (or whatever desktop wallet can do that)
Maybe they are using a different format, but they are not documenting...

Notice the error message here... the Addresses are different.

I know... NBitcoin is running the same algorithm, and indeed if I verify the message/sig with this address, it returns true. You should ask to Trezor how to verify with bitcoincore their signature, so we understand better how they are doing. Let me know.

When I get home I'll post our findings on the Trezor Github page and we'll see what they say.
On Tue, Dec 15, 2015 at 9:01 AM -0800, "Nicolas Dorier"<EMAIL_ADDRESS>wrote:
I know... NBitcoin is running the same algorithm, and indeed if I verify the message/sig with this address, it returns true. You should ask to Trezor how to verify with bitcoincore their signature, so we understand better how they are doing. Let me know.
—
Reply to this email directly or view it on GitHub.

@NicolasDorier it seems the verification needs to happen on the raw bytes and not on a hex string. Any chance you can overload PubKey.VerifyMessage to take a byte array?
See here:
https://github.com/trezor/connect/issues/12#issuecomment-165076588

I just did the implementation locally, I'll send you a PR once it's cleaned up :)

I am curious, because this is what I tried, but I still had false.

Ok I pushed a commit that makes things a bit better.
I also added a test which show how to use trezor with nbitcoin based on the issue.
Look at: https://github.com/MetacoSA/NBitcoin/commit/e32a52890f57bd5cdb9c004940bbc1f7d168bffc#diff-206198143ffd19de2bb2ff239b980abbR92
I'm closing this, thanks for the contribution, I'll push that to nuget when I have some connectivity. (later today)

I just push <IP_ADDRESS> here is how to use :  e32a528#diff-206198143ffd19de2bb2ff239b980abbR92

Much appreciated. Thanks! :+1:

Made a PR on trezor : https://github.com/trezor/connect/pull/13

You beat me to it :D

I'm on fire today !!!


Add deployment to measure free space in hub home dir

This adds a deployment with a single pod of
node_exporter running, configured only to report filesystem statistics. We mount the same home directory configuration that is used in the hub to this pod, so it will provide us with information on total disk size and used disk. This would allow us to have a Grafana panel with 'disk used %' for each of these hubs! We can even alert on those later.

https://github.com/jupyterhub/grafana-dashboards/pull/51 adds the graph.


Services for creating, retrieving , updating and deleting instances

Services for CRUD operations in broccoliService.js.
Seperated the CSS code from index.html to index.css and images in the images folder.

Can we go through in person when you have some basic tests @IshmeetKaur?

@IshmeetKaur you are trying to merge tons of commit I did. Where did you get them from? 😈

Actually , I tried to commit my work and then pulled the changes from the forked repository because of which all your commits are also shown.

Something went wrong :D Let's look at it together on Monday?

On 04 Nov 2016, at 10:24, IshmeetKaur<EMAIL_ADDRESS>wrote:
Actually , I tried to commit my work and then pulled the changes from the forked repository because of which all your commits are also shown.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/FRosner/cluster-broccoli/pull/96#issuecomment-258380618, or mute the thread https://github.com/notifications/unsubscribe-auth/ADRMQu6mHy72jXzNn6v7ZvG0XRO1ohdHks5q6vnigaJpZM4KRLqQ.

Okk..Sure
Thanks :)

Okk..Sure
Thanks :)


create a CI build for blst

I've had a chat with Danno on this topic. Below there are some extracts which might be useful as a starting point:
Anton Nashatyrev

Hey Danno!
I'm now near the final stage of integrating Supranational Blst BLS native library (https://github.com/supranational/blst)
Speaking shortly we would like to build and publish cross-platform Java wrapper maven artifact. This looks pretty similar to besu-native project: https://github.com/hyperledger/besu-native/

Danno Ferrin:

Since it’s C based the seckp256k1 bindings are the closest - https://github.com/hyperledger/besu-native/blob/master/secp256k1/src/main/java/org/hyperledger/besu/nativelib/secp256k1/LibSecp256k1.java
secp256k1/src/main/java/org/hyperledger/besu/nativelib/secp256k1/LibSecp256k1.java

Here is the JNA cheat sheet on type mappings - https://github.com/java-native-access/jna/blob/master/www/Mappings.md
www/Mappings.md

Anton Nashatyrev

I think the central topic I'd love to ask about is the building infrastructure, how to better organize the repository and how you do native shared libs packaging? Do you use some tool for the latter or did you do it manually (I mean packing them during build and then extracting them on runtime) ?
BTW we are currently not using JNA but stick to SWIG bindings as they are claimed to be supported by the Blst team

Danno Ferrin:

Swig has java support?

Anton Nashatyrev

Yep.

Danno Ferrin:

For building we do n+1 passes, where n is the number of binaries to produce.  First we run a pass where the native .so/.dll/.dylib is generated, and store that in CircleCI somehow.  Then we run a java step where we gather the binaries and store them in the jar, along with the java code for the wrapper.

https://github.com/hyperledger/besu-native/blob/master/.circleci/config.yml is the CircleCI config for all
.circleci/config.yml

we run the final jarring step directly from gradle.

Anton Nashatyrev

CircleCI may spin up Win/MacOS environment?

Danno Ferrin:besu:

Yes.  Although we haven’t taken the time to make windows builds work yet.
Looks like mac comes with xcode built in.
@ben.burns did most of the CI magic
JNA has support for loading jars from magic resource names, so we follow their pattern on where to put it in a jar.

Submitted the Windows related issue/question to blst team: https://github.com/supranational/blst/issues/53

Submitted the Windows related issue/question to blst team: https://github.com/supranational/blst/issues/53

Also filed an issue to SWIG: https://github.com/swig/swig/issues/1932

Also filed an issue to SWIG: https://github.com/swig/swig/issues/1932


fix: reproduction for issue (#2395)

Closing as this test is now added via https://github.com/mikro-orm/mikro-orm/commit/7a47151d363ba162a51edade0125dbe34aa44adb and passing


  I have not changed my password between client open and closing.
To Reproduce
Steps to reproduce the behavior:

Turn airplane mode
Start spot attempt to play some music
Get confused no music is playing but the bar is scrolling
Close spot
Turn off Airplane mode
Open spot again
Sign in (if you can remember your (stupidly assigned spotify) username and password)

Expected behavior
I expect it to remember my password until I inform it to forget it or "sign-out".  Maybe at sign in there should be a menu to select to sign-in with the same user but also delete that "profile" from "memory"
Screenshots
Nothing really to screenshot
General information:

Distribution: Debian 11
Installation method [e. g. built from source, installed from Flathub...]: gnome-software (flatpak)
Version [e.g. 0.1.0]: 0.3.3
Device used [e. g. desktop, phone...]: laptop (desktop)

Stack trace:
I'm not replicating it.
Additional context
You created the bug by fixing this one. Instead check if sign-in fail is due to connectivity?
add "profile" feature maybe and never forget them unless the user "deletes" them

I can indeed reproduce it. Thanks for the report!

@xou816 thank you for fixing this