Set up the dashboard login

(You have to work on this in the feature/dashboard-login branch. (This branch already exist, do not create it, just switch to it)
You have to implement the items of the following list:

login template
login view
verify template
verify view

All users (staff and manager) can login here.
User will enter their phone in the login page (only phone field, no password needed). it has to be checked if there is a user with this phone. if there is, the phone has to be saved in the session (with key of user_phone), and user be redirected to the verify page. If the phone does not exist in the database an error has to be shown to them in the login page.
In the verify page, if the user_phone is in the session, using GET method, a 4-digit number has to be created and saved in the session (with key of "2FA"). Then a form must be shown to them (exactly like login page but they have to enter this 4-digit number), if the number is correct they have to be redirected to index page (just for now) and if the number is wrong, redirect them back to verify page again.
NOTE: add a print to see the randomly generated number instead of using a real OTP. (Do not remove this print even after your tests)
NOTE: the templates has to look like our own template. You have to customize the fields and make them look better than a plain input.
Part 2
When making the verify number, also set session for expire time (name it 2fa_expire). You have to save 1 minute ahead of current time as the expiration time. Whenever user opens verify page, beside checking for the phone and verify number and the expiration time to exists in the session, also check if the moment the page is loading is not after the expiration time. If any of these 4 cases happen, user has to be redirected to the login page and any of these keys have to be removed from the session (if any exists).
When user has successfully logged in, remove the expiration time and 2fa code from the session. Also set the authenticated session to True. In any of the forms (login or verify) when the user enters incorrect data, show an error in the same page.
As the verify page make new code everytime we redirect to this page (GET), add a resend button to the verify page which just redirect back to the page again.

completed in #109


web3.eth.syncing throws 'bool' object is not iterable

Version: 3.10.0
Python: 3.6
OS: osx

What was wrong?
Attempting to get the value of web3.eth.syncing:
web3 = Web3(HTTPProvider(endpoint_uri=f"http://localhost:8545"))
print(web3.eth.syncing)

causes the following error:
Traceback (most recent call last):
  File "/Users/abc/Projects/test.py", line 30, in <module>
    print(web3.eth.syncing)
  File "/Users/abc/.pyenv/versions/3.6.1/lib/python3.6/site-packages/web3/utils/functional.py", line 15, in inner
    return formatter(value)
  File "/Users/abc/.pyenv/versions/3.6.1/lib/python3.6/site-packages/eth_utils/functional.py", line 11, in <lambda>
    return lambda x: f(g(x))
  File "/Users/abc/.pyenv/versions/3.6.1/lib/python3.6/site-packages/web3/formatters.py", line 54, in inner
    return fn(value)
  File "/Users/abc/.pyenv/versions/3.6.1/lib/python3.6/site-packages/web3/formatters.py", line 78, in wrapped
    return wrapper(fn(*args, **kwargs))
  File "/Users/abc/.pyenv/versions/3.6.1/lib/python3.6/site-packages/web3/utils/datastructures.py", line 19, in __init__
    self.__dict__ = dict(dictionary)
TypeError: 'bool' object is not iterable

Node: Parity/v1.6.8-beta-c396229-20170608/x86_64-macos/rustc1.17.0
Used to work in 3.8.0.

A fix was pushed in 3.11.0
@reverendus can you confirm?

Just checked, seems to be fine in 3.11.0.
Thanks!


Design clutter

What's the Problem?
i think the ui is not as clean as it could be, and the side preview is missing completely, witch is anoying, and there is no way to disable annoying tabs, i hate tabs
Solution/Idea
thats why i chose to declutter visually the top portion so that its more organized, please copy this concept since its more organized while you add more features as you want, please note that when the user single click the grayed this pc on top, it will go back to the this pc area, when the user double click it, the full path appears and the user is able to copy it...
see the blank space on top of search? this is the area for tabs, but instead being always on screen, witch is useless when you have only one tab avaliable, it hides itself and they wouldnt appear as big rectagles, but one icon in a square similar to vertical tabs in edge (plus i dont like tabs sooo yea... i would leave this disabled)
the main problem with yours currently, its the ammount of vertical stuff that is cluttering the user experience... there is toooooo much that can be hidden, like lines and search boxes and settings icon and tabs, and buttons...
its starting to feel weird

Alternatives
there arent really any alternatives
Priorities
| Capability | Priority |
| Clean design | Show only whats necessary |
| This proposal will allow developers to accomplish their task faster | Must use the design i send to you|
This proposal will allow end users to accomplish  (mainly creators) to look at the design of your file explorer and be less confused/frustrated/ annoyed with the organization with everything
This proposal will allow end users to accomplish disable tabs if they dont use them, and see less of what they dont need, as well as keep the functionality of the left panel (when they click, it expands accordingly)
i also think this wont, would should to be very confusing... so i just deleted them and wrote like a normal human
Files Version
No response
Windows Version
No response
Comments

this was done right when windows 11 released, so there are ui elements that could follow win 11 ui better ...

@pedro-gt3250 thank you for your feedback, you can enable the preview pane from the toolbar button. If you have suggestions for other improvements, please open an individual feature request so we can track each idea separately.


Inability to login

Hi.
I often receive negative feedback from users due to their inability to login through standard OAuth 2.0 process because of DA provided pages.
The problem almost each time is that users click "Accept" button on the last page (when you need to allow the app to access data) and nothing happens. No reaction at all, no kind of notification, message box or anything. Some of users simply forget to tick the checkbox and I always have to remind each and every of them. But in some cases even after selected checkbox, nothing happens.
One of users informed me, that there was some applications blocked by him on the site and that didn't allow him to login in my app. Only removing them from black list allowed.
So the question is if it's possible to somehow enhance the login pages with more additional information regarding inabilities to press "Accept" button? May be some information on WHY the can't, so that users now to check the checkbox, or remove apps from blacklist.
As for now, the worst reviews I personally get from people, who can't login due to Accept button. Would be nice to improve usability.
Thanks.

I have made the checkbox bigger and bolded the text so its a little more obvious that they need to check the box. I also duplicated the error to the top so they see it without scrolling. See if that helps :)


Unauthenticated Redirect Does not Work

Go to https://app.staging.crisiscleanup.io/admin/organization/5376 without logging in
Bug: The page spins but never redirects to /login

Works!


Keeping track of qubits you've measured

@JiahaoYao wanted to take a circuit where we were measuring 1 of the qubits to a case where we're measuring all of the qubits. Sounds like there should be an obvious one way to do this? I wrote the following in response, reproduced here so that others may 1) use this advice or 2) think about why there's no idiomatic way of doing this. cc #4144
Use cirq.measure(*qubits) instead of cirq.measure(q) for q in qubits (where qubits is a list). Now the results will have a 2d numpy array where the second axis is the qubit index in the order they were listed in qubits
You still need to save a mapping between the qubit and the index. One way of doing this is a dictionary {q: i for i, q in enumerate(qubits)}. Since you can always do this on the fly, I prefer just saving the qubits ordered list alongside my circuit so I can make this "mapping" on the fly
Some people use sorted(circuit.all_qubits()) as an implicit qubit ordering everywhere, but I worry this can introduce subtle bugs if you forget to sort your qubits sometimes or e.g. qubits have different sort order after you place your circuit onto a device. I prefer having an explicit qubits list.
Some people like using cirq.measure(q, key=str(q)) for q in qubits and parsing the stringy qubit key. This feels dirty and precludes lots of numpy tricks, e.g. zz_parity = np.sum(bitstrings[:, [i0, i1]], axis=1) % 2

This is the kind of stuff we deal with all the time at the hardware level, keeping track of what data for each key corresponds to what qubits, so I have some thoughts in this area.
Note that we have a find_measurements helper function in cirq_google to extract information about all measurements in a circuit: https://github.com/quantumlib/Cirq/blob/master/cirq-google/cirq_google/api/v2/results.py#L58. It returns a list of MeasureInfo dataclasses, one for each measurement, that include the measured qubits. It's a bit specialized for google devices, e.g. using cirq.GridQubit instead of cirq.Qid, but in general it's helpful to extract this kind of info from a circuit to help interpret results.
(Aside: One problem I always run into with this kind of analysis is I'd like to be able to compute some property of a circuit, e.g. find info about measurements, and then cache the result on the circuit so I don't accidentally redo the computation.  Having a generic way to cache such analyses and then invalidate them if the circuit is mutated would be useful.)
We could have cirq.Result include some of this information, such as storing the qubits along with the data for each measure key, though if we remove the uniqueness constraint keys as proposed in #4274 it could be that each "instance" of a particular key in a circuit measures different qubits, which would complicate the tracking of what data corresponds to what qubits.

Not sure I really addressed your question, though. I agree with cirq.measure(*qubits) and keeping track of the qubit order, so maybe the one thing we might consider is including some kind of "measurement descriptor" in cirq.Result so if you save results you don't have to separately keep track of the qubit order.

I should also add cirq.measure(*qubits) keeps all your measure operation in one moment, which is really important.

Cirq cync requests an update from @dabacon whether this should be accepted.

I think storing information about the qubits that are measured on Result seems like a good idea.  I think when we've talked about this in the past, it was always pointed out that you could keep track of this yourself, but this seems like a lot of ugly code to have to write.

The simulators don't currently track what measurements correspond to what qubits. But I was planning to add this in the near future anyway so that we can support qudits in integer expressions in classical control: if we have cirq.measure(some_qudits, key='mkey'), cirq.X(q0).if('mkey > 20'), then we need to retain at least the shape of the qudits we measured so that we can calculate the int from the dits.
Once that's done we should be able to pass that info into Result easily enough.

Unassigning myself but this should be fairly doable now. All the data is stored in the simulation TrialResult class, and just needs passed into the Result class. It might require breaking change to SimulatesSamples._run return type. That currently just returns raw data. If we can have it return the collection of TrialResults instead, then those could be passed into ResultDict to satisfy this requirement. Note qsim would be affected as well.

All the data is stored in the simulation TrialResult class, and just needs passed into the Result class.

The data is stored in the optional final_step_result field, which is only available to SimulatesIntermediateStates simulators. Since there's no guarantee that a SimulatesSamples simulators implements intermediate states (qsim doesn't, for one) we can't use it here - or at least not in a generic way that covers all simulators.

Could we override SimulatesSamples.run_sweep_iter in SimulatorBase? Anything that inherits from SimulatorBase should have this final_step_result field hydrated. Thus SimulatorBase.run_sweep_iter should be able to return Results that have the qubit information (either a new Result subclass, or add the functionality to ResultDict). Then we can revert SimulatesSamples.run_sweep_iter back to expecting a 2D array, which would fix the qsim (and any others) integration.

I don't think modifying SimulatorBase is sufficient to resolve this problem, as it's not extensible to external simulators. RE use of 2D arrays, I've opened #5014 with a fairly lightweight fix.


Instructions on discovering memory offset?

This is fantastic work, I would like to replicate this for one of my projects, but I'm running Linux, and trying to figure it out for that is more fun than just booting to Windows and using this script. Can I ask how you found the memory location and values that you've hardcoded into the script?

I apologize for it being two months, I never got an email about the issue you created and only saw it because I was randomly looking at my repos.
As for your question (if you still even need an answer), take a look at this thread for a discussion about it. You could probably use Cheat Engine to narrow down the addresses if they aren't the same for you.

Thank you, I eventually saw that and tried for a few hours to reproduce the findings, with not much success. I will disable ASLR on linux and try again, thanks.


Spack package?

Hello,
Is there any chance a spack package of this could be made? I'm asking since we had a request to install it, and a spack package would be more convenient for us. No problem if this isn't possible, we just wanted to ask in case.
Thanks!

Hi Colleen,
I already have a prototype for a spack package. I just need clarify some technical details, in particular how to best model the sycl dependency.
Best regards,
Carsten

Thank you!
Colleen

A spack package is now available in the develop version of spack:
https://github.com/spack/spack/blob/develop/var/spack/repos/builtin/packages/double-batched-fft-library/package.py


Dependencies: Updated to advertise numpy v2 is not supported

Checklist:

[x] The PR title is formatted as: <Module or Topic>: <Action> <Summary> (see contributing guidelines)

Closing in favor of #4082


Access to forums

Pete, Laura, and Giustina cannot get into the ninjaforums

Fixed on Friday here https://github.com/CoderDojo/cp-users-service/pull/162


[FR] Allow displaying additional base part details when accessing a stock item.

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

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

Problem statement
When browsing a base part, the part description is displayed. When accessing a stock item for that same part, no description is displayed but only the Part (Name and IPN). If the Name/IPN does not contain any descriptive data but just a model no., this makes it difficult to identify the part, without having to view the base part again separately.
Suggested solution
Allow enabling/disabling display of fields for a stock item, eg. "description", "link" etc. Perhaps this could be enabled from the settings menu?
Describe alternatives you've considered
Modifying the Part Name Display Format directly in the "Parts settings" menu as listed below:
{{ part.IPN if part.IPN }}{{ ' | ' if part.IPN }}{{ part.name }}{{ ' | ' if part.revision }}{{ part.revision if part.revision }}{{ ' | ' if part.description }}{{ part.description }}

This is not an ideal solution, as it will affect the part name everywhere.
Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

The stock display is only meant to show stock specific information. I think it will confuse users more then anything if we duplicate inforamtion around the system. If you want information about the part click on the part name, if you want to change stuff about the stock use the stock display in my opinion.

@matmair I see your point. However, the issue occurs when I browse the stock page since our inventory contains many parts that have similar IPNs. I would need to memorize what a specific part name/IPN is related to - or click the base part and view the description of it each time. Otherwise, I risk mixing up stocks.
Therefore, it is primarily the base part description field that is missing. If I need more detailed information about the part, I can just view the base part, as you suggest.


Build fails with "error IDE0005: Using directive is unnecessary"

I have the DBus source generated added and i also have the .editorconfig which configures IDE0005 as an error
.editorconfig
dotnet_diagnostic.IDE0005.severity = error

Generated files aren't usually inspected for this error, we've been looking into why they were not and reason seems to be the // <auto-generated/> comment in the generated file. Although, the location of that comment appears to be important, it must be at the top of the file, as opposed to top of the class, as it is for DBus source generator generated files.
In other words, for the below file, analyzer still scans the file for styling errors
using System;
using System.Collections.Generic;

// <auto-generated/>
#pragma warning disable
#nullable enable
namespace Tmds.DBus.SourceGenerator
{
    internal static class ReaderExtensions

but for the below, it does not
// <auto-generated/>
using System;
using System.Collections.Generic;

#pragma warning disable
#nullable enable
namespace Tmds.DBus.SourceGenerator
{
    internal static class ReaderExtensions

Shall we move that comment at the top ?
Note:
As far as .editorconfig goes, i tried dozens of ways to disable IDE0005 for generated file, as an example, one of the ways should be below
# Disable all inspections for generated files
[**\GeneratedFiles\*.cs]
# IDE0005: Using directive is unnecessary
dotnet_diagnostic.IDE0005.severity = none

but it does not work

@affederaffe thanks for accepting the PR. Do you know when i can expect new nuget package to be release ? thanks

I just uploaded the new package, it may take a while to be indexed, but should be available soon.
Thank you for the PR!


I want to run SQL file, how do I need to do it

Execute the SQL file directly without reading the SQL file
db = await Database.load(sqlite:${dbPath})
Can I use db.** or fs.** command
maybe other

If you want to initialize your database with this file then this is what migrations are for: https://github.com/tauri-apps/tauri-plugin-sql/blob/dev/examples/vue-todos/src-tauri/src/main.rs#L15-L23.
If you want to execute .sql files at runtime you need to read it first (fs.readBinaryFile) and use db.execute(content_of_sql_file). Keep in mind that you have to manually include the .sql file in the bundled app then! (for example via tauri.conf.json's resources)


Password field silently missing on new admin creation if no auth enabled

If you neglect to enable an auth backend, everything seems fine except there is no place to create a password for your new admin login. That section of the page is missing.
Maybe show a message warning that no authentication method is enabled? I test installed twice before I noticed I hadn't been asked to create a password. When I logged out, I couldn't login again. The login page shows "You look familiar" but no login fields or button.

Yeah, we should have a "no auth backends enabled" error page, that one gets redirected to both from /register and /login.

@zulipbot claim


Adjust e2e browser based tests

New feature description
The Dev-2-3 environment introduces a restriction on the clients that can issue an Access Grant, breaking e2e tests whose target environment is Dev-2-3 since they're not acting as an allowed client.
The adjustment replaces the use of access grants with access requests, which don't have any restriction and don't need AMC to be involved in the process: instead of approving and revoking an access grant, an access request is issued and denied.
The test-app UI has also been modified according to the changes.
Checklist

[ ] All acceptance criteria are met.
[ ] Relevant documentation, if any, has been written/updated.
[ ] The changelog has been updated, if applicable.
[ ] New functions/types have been exported in index.ts, if applicable.
[ ] New modules (i.e. new .ts files) are listed in the exports field in package.json, if applicable.
[ ] New modules (i.e. new .ts files) are listed in the typedocOptions.entryPoints field in tsconfig.json, if applicable.
[x] Commits in this PR are minimal and have descriptive commit messages.

This has been incorporated into https://github.com/inrupt/solid-client-access-grants-js/pull/999


gulp.watch writing variable number asterisk(*) signs instead of code changes

I'm running a gulp task that concatenates few JS files. It works great when i run it once. But the issue is when i include gulp.watch: Whenever i save a watched file It writes a variable number of asteriks in my output file in places where changes were supposed to be and i get JS "ILLEGAL TOKEN" error. If i manually delete the output file and then create it again (either by re-running the gulp task or by simply saving a watched file) the result is generated properly.
I'm running gulp in laravel's homestead (virtual machine).
Gulp version is 3.9.0
This is my gulpfile.js
var gulp  = require('gulp'),
    gutil = require('gulp-util'),
    concat = require('gulp-concat'),
    uglify = require('gulp-uglify');

gulp.task('mix-jquery', function() {
    return gulp.src([ 'jquery_deps/*.js']).pipe(concat('frontend.js')).pipe(gulp.dest('./public/'));
});

gulp.task('mix-angular', function() {
    return gulp.src([
        'bower_components/angular/angular.min.js',
        'bower_components/angular-ui-router/release/angular-ui-router.min.js',
        'bower_components/ng-dialog/js/ngDialog.min.js',
        'main.js',
        'controllers/*.js'
    ]).pipe(concat('app.js')).pipe(gulp.dest('./public/'));
});

gulp.task('default', ['mix-jquery', 'mix-angular'], function() {
    return gutil.log('all done!')
});

var watcher = gulp.watch(['main.js', 'controllers/*.js'], ['default']);

watcher.on('change', function(event) {
    console.log('File ' + event.path + ' was ' + event.type + ', running tasks...');
});

Support questions should be directed to StackOverflow.


Fix Issue Number: 24 - Git Siblings

Fix for the Issue Number 24 in the Cam Repo.
Problem Description:
For each .java class, let's count the number of other .java classes which it was updated together with, in the Git history.
Solution Description:

Use the git log command with --name-only and --pretty=format: options to list committed files per commit.
Filter the output for .java files.
For each .java file, track the commits it appears in.
Calculate the number of times each .java file is committed with other .java files.
Output the result for each file, showing its "siblings".

@tanmaysharma2001 looks like a very good start, thanks! Now, please, try to run make clean install and then make test lint, in order to check that the code doesn't break anything. Then, please, create a unit test for this script, similar to the tests we already have for other scripts.
Also, if it's a "step", we should name it like all other steps: XXX-name.sh, where XXX is a number.

Hi @yegor256 thanks a lot. I have fixed one the 1 test where my script was failing. Although, i had few problems with installing make clean install, which includes installing dependencies. I'm working on that. Meanwhile I request you to approve the PR so that it can run the tests again. Also i have written the unit test for the script too.
Also I wanted to ask about the step number. I didn't quite understand where should i name it cause other steps are named according to their purposes. For example: aggregate-join.sh.
Sorry if i misunderstood anything. Thanks.

@tanmaysharma2001 this should be a metric, not a "step". Put your file into the metrics/ directory and that's it.

Done @yegor256 Thanks a lot.


Profiling

Is it possible to profile your rsmpi using application with a MPI aware profiler like VampirTrace or Valgrind?
Excuse me, if this is not the place to ask such questions.

I just did some experiments and was able to successfully link mpiP into some of my examples. It seemed to pick up the MPI function calls performed by Rust code.
To build the examples with mpiP, I did
cargo rustc --verbose --example vector -- -L native=/directory/containing/libmpiP.so/ -l dylib=mpiP

I guess other profilers might achieve similar amounts of success. Does this answer your question?

Yes, thank you. I've tried Callgrind and VampirTrace with dynamic instrumentation, which seems to work. It would be interesting, if tools would work, that do the instrumentation on a binary level.
I probably will investigate this further. For now, I'll close this issue.

Good to know, thanks for reporting this. I would be interested to hear about the results of your further investigations also.


Crime shouldn't start until some amount of time after landing

Related to #1228, criminal activity should probably not start until some amount of time after the population has landed when the colonists have had some time to settle into their new home.
The idea behind this is that 1) colonists are going to be too busy adjusting to their new life to plan and commit crimes and 2) from a gameplay standpoint we want to give the player at least a few turns to establish themselves before crime is something they need to worry about.
This buffer period should be adjusted based on difficulty level. Default ('hard mode') should be 15 turns with no crime updates from the time that colonists are landed (just over 'one year')

Related issue that probably deserves its own issue number - Morale also starts dropping before colonists have landed.

Fair point. Thinking it should stay consistent while there's no population on the ground. Will open a new ticket for that. Thanks :D


Appstudio update test-component-pac-bqjv

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-bqjv-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
appstudio-init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
13 seconds
appstudio-configure-build

✅ Succeeded
15 seconds
sast-snyk-check

✅ Succeeded
37 seconds
build-container

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
15 seconds
sanity-inspect-image

✅ Succeeded
18 seconds
sbom-json-check

✅ Succeeded
14 seconds
clair-scan

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
12 seconds
sanity-label-check

✅ Succeeded
10 seconds
sanity-optional-label-check

✅ Succeeded
8 seconds
show-summary

Pipelines as Code CI/test-component-pac-bqjv-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
appstudio-init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
15 seconds
sast-snyk-check

✅ Succeeded
13 seconds
appstudio-configure-build

✅ Succeeded
37 seconds
build-container

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
15 seconds
sanity-inspect-image

✅ Succeeded
1 minute
clamav-scan

✅ Succeeded
14 seconds
clair-scan

✅ Succeeded
18 seconds
sbom-json-check

✅ Succeeded
12 seconds
sanity-label-check

✅ Succeeded
10 seconds
sanity-optional-label-check

✅ Succeeded
8 seconds
show-summary


Pre registration

Codecov Report
Patch and project coverage have no change.

Comparison is base (3b5fdbf) 39.26% compared to head (65c5a1d) 39.26%.
Report is 15 commits behind head on main.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage   39.26%   39.26%
=======================================
  Files          20       20
  Lines        1859     1859
=======================================
  Hits          730      730
  Misses       1129     1129

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

@JuliaRegistrator register


Update vaultwarden default to 1.26.0

Update vaultwarden default to 1.26.0 and the web vault to 2022.10.0, the version mentioned in the release notes.
https://github.com/dani-garcia/vaultwarden/releases/tag/1.26.0
https://github.com/dani-garcia/vaultwarden/releases/tag/1.25.2
https://github.com/dani-garcia/vaultwarden/releases/tag/1.25.1
https://github.com/dani-garcia/vaultwarden/releases/tag/1.25.0
https://github.com/dani-garcia/vaultwarden/releases/tag/1.24.0
https://github.com/dani-garcia/vaultwarden/releases/tag/1.23.1
Tested and works for me.

Yeah, the web vault version is only part of the docker build information. I've built this for the updater over in nixpkgs, feel free to reuse it.
#!/usr/bin/env bash
set -euxo pipefail

VAULTWARDEN_VERSION=$(curl --silent https://api.github.com/repos/dani-garcia/vaultwarden/releases/latest | jq -r '.tag_name')

URL="https://raw.githubusercontent.com/dani-garcia/vaultwarden/''${VAULTWARDEN_VERSION}/docker/Dockerfile.j2"
WEBVAULT_VERSION=$(curl --silent "$URL" | grep "set vault_version" | sed -E "s/.*\"([^\"]+)\".*/\\1/")


what is the instruction to cause segment fault??

I have two piece of code,one cause segment fault,one don't.
here the are.

.text
.globl atoic_test

atoic_test:
    addi    sp,sp,-8
    sd      s0,0(sp)
    ori     s0,sp,0
    addi    sp,sp,-16
    ori     t4,a1,0
    mv      a1,sp
    sw      a0,0(a1)
    amominu.w       a4,t4,(a1)
    lw      a0,0(a1)
    addi    sp,sp,16
    ld      s0,0(sp)
    addi    sp,sp,8

.text
.globl atoic_test

atoic_test:
    addi    sp,sp,-8
    sd      s0,0(sp)
    ori     s0,sp,0
    addi    sp,sp,-16
    ori     t4,a1,0
    mv      a1,sp
    sw      a0,0(a1)
    amominu.w       a4,t4,(a1)
    lw      a0,0(a1)
    addi    sp,sp,16
    ld      s0,0(sp)
    addi    sp,sp,8

they are almost the same , why cause segment fault??

sorry about the typo "atoic"


fix: targetTo should return invert mat4 of out

it seems targetTo function have some problems, according to my analysis, we should invert the 'out' matrix. Besides, I replace Math.sqrt(Math.pow()+...) with Math.hypot() which is more elegant.

Regarding targetTo, it's supposed to be this way. For more information, check out https://github.com/toji/gl-matrix/issues/225
However, the Math.hypot optimization seems to be pretty interesting. 😄

👍 Thank you for the great pull request! 😄 I'll merge it ASAP.

Math.hypot is interesting indeed 👍 A concern is that is gl-matrix supposed to be compatible with IE? hypot  is landed in ES2015 so it may not work on IE11.

Yes, that is a valid concern indeed. There are a few options here:

Drop IE support: Sub-optimal
Reject the PR: Also sub-optimal, as gl-matrix is supposed to be super fast
Transpile Math.hypot to Math.pow(Math.sqrt(...
Ship a Math.hypot polyfill with gl-matrix

How about detecting the browser's Math.hypot support and use our current implementation as fallback? In this way we don't have to include the heavy official polyfill libs, neither do we need extra transpile process.

heavy official polyfill libs

I didn't suggest adding a fully blown polyfill library. We only have to polyfill, Math.hypot, which is rather easy and lightweight. 😄
However, the polyfill option does have a disadvantage: gl-matrix is going to be slower on IE. But I suppose that's an acceptable downside.

How can you be so patient, it's my pleasure to do my bit!


Allow more than one field to be used for the filename

Use an optional javascript specification for the filename, e.g. ${part1field}-${part2field}  or maybe even ${part1field}/${part2field}

Implemented in 0.35.0


Failed to fetch secret: 'secret_name'. Error: TimeoutError.

Can someone help why i see intermittent timeouts . My session duratiuon is 1 hour at I am role. I am using OIDC with AWS
  - name: Get Secrets from AWS Secrets Manager
    id: get-secrets
    uses: aws-actions/aws-secretsmanager-get-secrets@v2
    with:
      secret-ids: |
        secret_name
      parse-json-secrets: true

Run aws-actions/aws-secretsmanager-get-secrets@v2
Building secrets list...
Your secret names may be transformed in order to be valid environment variables (see README). Enable Debug logging in order to view the new environment names.
Error: Failed to fetch secret: 'secret_name'. Error: TimeoutError.

Are you using self-hosted or github hosted runners?
Would you mind sending me arn and exact timestamps when these timeouts happened? I would advise not publishing the ARN to the comment here, but sending via mail<EMAIL_ADDRESS>

Any update on this? Seeing this all of a sudden in the ap-southeast-2 region

+1 the same issue there, reading secret in ap-southeast-2 from eu-west-1
temporary workaround (for plaintext secrets)
      - name: Set up AWS CLI
        run: |
          curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64.zip" -o "awscliv2.zip"
          unzip awscliv2.zip
          sudo ./aws/install
          rm -r awscliv2.zip ./aws

      - name: Get secret
        run: |
          secret_value=$(aws secretsmanager get-secret-value --secret-id "your/secret/name" --query 'SecretString' --output text)
          echo "SECRET=$secret_value" >> $GITHUB_ENV

The https://github.com/aws-actions/aws-secretsmanager-get-secrets/pull/205 should resolve the problem. Closing the ticket - please let me know if you continue to have problems.


Continuous Query fanout causes panic

query:
CREATE CONTINUOUS QUERY availability_customer2_overall_30m ON statsd BEGIN SELECT sum(available) as available, sum(nonavailable) as nonavailable, customerId INTO "availability.customer.[customerId].30m" FROM "statsd"."default"."availability" group by time(30m), customerId END
webcall: http://prd-influx-1:8086/query?database=statsd&q=CREATE CONTINUOUS QUERY availability_customer2_overall_30m ON statsd BEGIN SELECT sum(available) as available, sum(nonavailable) as nonavailable, customerId INTO "availability.customer.[customerId].30m" FROM "statsd"."default"."availability" group by time(30m), customerId END
panic: runtime error: index out of range

goroutine 180863 [running]:
github.com/influxdb/influxdb.(*Server).convertRowToPoints(0xc208076200, 0xc209d815c0, 0x22, 0xc2091d2120, 0x0, 0x0, 0x0, 0x0, 0x0)
        /root/.gvm/pkgsets/go1.4.2/global/src/github.com/influxdb/influxdb/server.go:4061 +0x7b7
github.com/influxdb/influxdb.(*Server).runContinuousQueryAndWriteResult(0xc208076200, 0xc2099c3880, 0x0, 0x0)
        /root/.gvm/pkgsets/go1.4.2/global/src/github.com/influxdb/influxdb/server.go:4013 +0x158
github.com/influxdb/influxdb.(*Server).runContinuousQuery(0xc208076200, 0xc2099c3880)
        /root/.gvm/pkgsets/go1.4.2/global/src/github.com/influxdb/influxdb/server.go:3992 +0x8c1
github.com/influxdb/influxdb.func·050(0xc2099c3880)
        /root/.gvm/pkgsets/go1.4.2/global/src/github.com/influxdb/influxdb/server.go:3941 +0x34
created by github.com/influxdb/influxdb.(*Server).RunContinuousQueries
        /root/.gvm/pkgsets/go1.4.2/global/src/github.com/influxdb/influxdb/server.go:3942 +0x269

goroutine 1 [chan receive (nil chan), 21 minutes]:
main.(*RunCommand).Run(0xc20803d380, 0xc20800a010, 0x4, 0x4, 0x0, 0x0)
        /root/.gvm/pkgsets/go1.4.2/global/src/github.com/influxdb/influxdb/cmd/influxd/run.go:264 +0x806
main.main()
        /root/.gvm/pkgsets/go1.4.2/global/src/github.com/influxdb/influxdb/cmd/influxd/main.go:81 +0x495

More info: the customerId is a tag

Running: http://prd-influx-1:8086/query?database=statsd&q=select customerId as customerId, sum(available) from "statsd"."default"."availability" group by customerId
Gives:
{ 	"results": [{ 		"series": [{ 			"name":"availability", 			"tags":{"customerId":"1"}, 			"columns":["time","customerId","sum"], 			"values":[["1970-01-01T00:00:00Z",0]] 		}] 	}] }
looks like the tag is not in the columns group where the continuous query is looking for it

@annymsMthd can you provide a sample statement for writing points to this series? It helps to understand how the data in availability is organized.

Believed fixed by https://github.com/influxdb/influxdb/issues/2499, please try with RC31.


Case insensitive ethnicity matching

For this ticket: https://trello.com/c/7Q6nf0rg/1448-chart-and-table-ethnicity-lookups-shouldnt-be-case-sensitive
The first commit here is a refactor to avoid "standardising" the exact same set of "raw values" multiple times for no good reason.
The second commit here addresses the issue itself, which is that the csv file used to generate the lookup dictionary for reducing input ethnicities to our small "standard" set was missing rows for a few of the ethnicities in themselves.  The lookup dictionary maps lowercase strings to "standard" ethnicities if input strings don't match a standard ethnicity exactly.  So we need entries to map e.g. "Irish traveller" to "Irish Traveller" if we want case-insensitive matching.

Aargh!  I have no idea why the diff here is showing the whole csv file as changed. There are only a handful of rows added, honest.


Add flow to configure BDI QA metadata in a namespaced scratch org

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

@skristem Can you validate that this branch fixes the gack you were observing related to help text mapping?
@davisagli I'd love to know if there's a better way to achieve this - for deployment into a namespaced org, we need to merge the org's namespace only on the Help Text mappings for BDI. As far as I could tell I can't use NAMESPACED_ORG tokens without also injecting other tokens we don't want injected in that context.


Bump wasm-encoder to 0.34.0 and wasmparser to 0.114.0

I think this is the cause of https://github.com/bytecodealliance/wit-bindgen/issues/697 - two crates got minor version bumps, but all the other crates got patch bumps.
Then today wit-component got a patch release, but bumped its dependency on wit-parser by a minor version. That means that any dependent that uses both (even indirectly) is hosed until the rest of the ecosystem cuts new releases.
Another way to avoid this would be to not expose foreign types in the APIs, but that ship has likely sailed.

Yes thanks for the heads up, and this is my mistake as I didn't catch this. I posted https://github.com/bytecodealliance/wasm-tools/pull/1250 to fixup the versions and additionally future-proof this (hopefully). I'll yank the broken versions once those are published to avoid future breakage.

Thanks.

I'll yank the broken versions once those are published to avoid future breakage.

Has this happened?

I've just now done the yanks, yes.


Add "Single Point" to add_eddy_current_sweep

Description of the feature
Dear pyaedt team!
It seems that I can't add a single point, only single points (corresponding to Single Point Sweep):

I tried this, but it doesn't add frequency sweep:
sim_setup = m2d.create_setup()
sim_setup.add_eddy_current_sweep(range_type="SinglePoints", start = 0.001)
Thanks in advance for looking into this.
Thanks
Irene
Steps for implementing the feature
sim_setup = m2d.create_setup()
sim_setup.add_eddy_current_sweep(range_type="SinglePoints", start = 0.001)
Useful links and references
No response

@IreneWoyna-HE I'll close this one since I fix the issue in #5492.
It will be available in today's new release.
Please do let me know!
Giulia

Many thanks Giulia! I'll test asap.


Bug: user card sometimes persists on using browser back button

To reproduce this:
Click open a user's profile card for the 1st time (so the avatar image isn't cached, I guess) and immediately click the back button. Often (but not always) the profile card stays open even after the previous page has loaded in.
More details and debugging suggestions at this CZO thread

@zulipbot add "bug"

@zulipbot add "help wanted" "area: popovers"

i think i can work on this issue.

@zulipbot claim

https://user-images.githubusercontent.com/98907006/216810094-ff20e43a-f708-4a79-b106-63535f4c2bfc.mp4

This is the bug right ? @N-Shar-ma @alya

@rahul-maurya11b Correct, the 1st and 3rd time the user card stayed open even on page change, which is the bug

@zulipbot claim

@rahul-maurya11b Are you still interested in solving this issue? It seems to be a quick and easy fix to just make the popovers.hide_all() function call after every on-click event in recent conversations.

@zulipbot claim

Closing in favour of #25395


Example for grafana does not work

Describe the bug
The example available for Argo CD with grafana does not seem to work. Operator is running into the error state.
Example:
https://raw.githubusercontent.com/argoproj-labs/argocd-operator/master/examples/argocd-route.yaml
apiVersion: argoproj.io/v1alpha1
kind: ArgoCD
metadata:
  name: example-argocd
  labels:
    example: route
spec:
  grafana:
    enabled: true
    route:
      enabled: true
  prometheus:
    enabled: true
    route:
      enabled: true
  server:
    route:
      enabled: true

To Reproduce
Steps to reproduce the behavior:

Install the operator from Operator Hub or make run.
Create Argo CD Instance with grafana and openshift route as mentioned above.
Verify that the Argo CD workloads are not created and operator throws the below error.

2021-10-13T09:22:20.927Z  ERROR controller-runtime.manager.controller.argocd  Reconciler error  {"reconciler group": "argoproj.io", "reconciler kind": "ArgoCD", "name": "example-argocd", "namespace": "default", "error": "open /var/lib/grafana/templates: no such file or directory"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
  /remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/internal/controller/controller.go:253
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
  /remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.5/pkg/internal/controller/controller.go:214

Expected behavior
No error.
Additional context
Add any other context about the problem here.

I am experiencing this issue as well. Is there a missing configmap entry?

If u start Prometheus Operator in local-machine (not container) - need to copy files from directory grafana to /var/lib/grafana like it's do in DockerFile.
Or use environment variable GRAFANA_CONFIG_PATH to change dir path from /var/lib/grafana to <PROJECT_PATH>/grafana
Of change ArgoCDDefaultGrafanaConfigPath in common/defaults


Bug 1838372: Allow image push after postCommit script completes

Builds which use postCommit hooks run in a container that references the image digest sha
directly. To buildah these images are "pulled" using the containers-storage transport, i.e
from local storage. Previously, we did not set any policy for containers-storage transport,
which meant we fell back to the default policy. The default policy is "reject all" in clusters
with an AllowedRegistries setting in image.config.openshift.io/cluster.
This fixes the generated signature policy.json file in builds so that images in
containers-storage can always be "pulled" and pushed.

/hold
IMO this is a risky change that is difficult to test in our CI environment. I'd prefer to not merge this until master is open for 4.6

/assign @gabemontero
/cc @nalind

/bugzilla refresh

/hold cancel
BZ bot will do the job of blocking merge.

LGTM.  I'd be tempted to pull the transport names of the ImageTransport objects in the various subdirectories of the github.com/containers/image package, but that would require us to pull in a lot of dependencies that we're not currently pulling in.

/lgtm

/lgtm

/cherry-pick release-4.5

/cherry-pick release-4.4

/cherry-pick release-4.3


Begin declarative-izing NiA - adapt the core/common subproject

Document the setup in top of README.

https://github.com/gradle/nowinandroid/pull/5


dangling comma

Fresh install:
     ERROR in ./app/src/App.vue

       ✘  http://eslint.org/docs/rules/comma-dangle  Unexpected trailing comma
       /Users/Tektango/projects/stack/dev-studio/app/src/App.vue:39:6
         },
             ^

     ✘ 1 problem (1 error, 0 warnings)

Removed comma in my build and all well....

Thanks for posting. What plugins did you select during the project scaffolding?

This is a elinter error.you can check your code and just delete a extra comma,then every thing goes well


No hide backup / drop root password text

Closes #573 #572

Screenshot?

Nice

Coverage decreased (-0.01%) to 84.85% when pulling 73e50eb62fcf1ab06caf7d5e846f71da5e016099 on briansteffens:no-hide-backup into d1b3457edb18330f88eab96409eb37c26125714c on linode:master.


dataset

Where is the relevant data set in the code? Can you provide it?

Spaghetti error detection uses a pre-trained YOLO network taken from The Spaghetti Detective, you can se more details on their page, but I don't think they provide de dataset. There are a few test images in my repository to try it.


fix(54283): Provide better UX when an invalid file is selected for 'move to file' refactoring

Fixes #54283

@typescript-bot cherry-pick this to release-5.1


Multithreading in Java

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Proposed title of article
Multi-threading in Java
Introduction paragraph (2-3 paragraphs):
A thread is a lightweight process. They can be used to perform complicated tasks in the background without interrupting the main program. Any process can have multiple threads running concurrently in it that is referred to as multi-threading. It enables us to improve the responsiveness of a system.
For example in a web browser, we can have one thread which will load the user interface and another thread which will actually retrieve all the data that needs to be displayed in that interface.
We will learn how to create threads in your programs. It can be created by extending the Thread class or implementing the runnable interface.
Key takeaways:

What is a thread and multi-threading
Why we need Multi-threading
Create threads by extending the thread class
Create threads by implementing the runnable interface.

References:
N/A

sounds like a good one - approved @Jeff-mwangi

merged

https://github.com/section-engineering-education/engineering-education/pull/2432


TI API: LBO with PDU session already active

To discuss if the current TI API v0.8.1 is already compatible with a Use Case where it is invoked for already active sessions.

The focal point is to have clear the coordinates for the active session if we are considering a single user. So we need to know the UE IP and the EAS IP.
In the current version of the TI API, the EAS IP is retrieved by the OP starting from the applicationId input parameter. The current scope of the TI API is indeed related to Application onboarded with the CAMARA API and deployed in the Telco Infrastructure so they all have an applicationId and the OP is aware if the IP addresses of each instance.
For the UE IP the TI API has the userId parameter. It can be: IPAddress, MSISDN, IMEI, MDN, GPSI. From MSISDN and GIPSI the OP is able to identify the UE IP to be used in the UPF (also discriminating between possible Private or Public IP because of CGNAT). What about MDN and IMEI, anyone knows?

There was this question whether the Application needs to provide the IP Address in case of a traffic influence for an active session. The IP Address could also be a NAT'd one. There are ways in which the OP can fetch the IP Addr using the UE Id like GPSI/SUPI, for example using Event Monitoring service.
Another service has also been introduced in R17 called Nnef_UEId service, which is however for retrieving the external UE Id (GPSI) using an IP Address. In this case, the specs notes that the NATed (Network and Port Address Translation) IP address is not supported in the release.
We can make appropriate recommendations based on the above.

one note: the 3GPP TI API gets, as input to identify the UE, the GPSI. So we should understand if the list of possible options we have to identify the UE (IPAddress, MSISDN, IMEI, MDN, GPSI) let the OP to retrieve the GPSI.

I discussed this topic in OPAG and the suggestion is to avoid to use IMEI and MDN. IMEI is a sensitive information and it does not allow to identify the network that that device is connected to. It is indeed related to the device rather than actually providing an identification of the network used by the device. This prevent to use it in roaming scenarios and federation. MDN is considered obsolete when working on 4G and 5G networks.
I would also suggest to update the userid parameter to match the CAMARA guidelines on EU Identification: https://github.com/camaraproject/WorkingGroups/blob/main/Commonalities/documentation/UE-Identification.md
The guidelines suggest to use GPSI, IP or MAC.
A general alignment with the other APIs in CAMARA on UE Identification is needed.

Yes, I agree. The API can be used for the purpose of influencing active sessions too, by giving the appropriate parameters that will help OP identify the specific traffic in a PDU Session to be influenced. If UE IP Addr is provided, the PDU session can be identified directly with it. If GPSI (MSISDN or Externa identifier) is given, additional info like the App Id can help OP identify the correct session among the multiple that may have been established from the UE.

To align the TI API with the Glossary (https://github.com/camaraproject/WorkingGroups/blob/main/Commonalities/documentation/Glossary.md) it is better to rename UE as Device. Also MSISDN should be renamed as PhoneNumber. For this reason it is proposed to adopt the implementation done in QoD for the parameter Device.

Closed with the UE parameter modified as Device containing arguments that represent GSPI.
PR TI V0.9.0: https://github.com/camaraproject/EdgeCloud/pull/105


[Backport 2.x] [Segment Rreplication] Adding CheckpointRefreshListener to trigger when Segment replication is turned on and Primary shard refreshes

Description
Backport https://github.com/opensearch-project/OpenSearch/commit/fd5a38de12cbb0fe5fca40aa833cd77ad3efdb8e from https://github.com/opensearch-project/OpenSearch/pull/3108
Issues Resolved
#3108
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4044 (e120686) into 2.x (fb4f96f) will decrease coverage by 0.05%.
The diff coverage is 61.65%.

@@             Coverage Diff              @@
##                2.x    #4044      +/-   ##
============================================
- Coverage     70.63%   70.58%   -0.06%
+ Complexity    56487    56480       -7
============================================
  Files          4527     4534       +7
  Lines        271798   271930     +132
  Branches      39975    39987      +12
============================================
- Hits         191985   191939      -46
- Misses        63718    63806      +88
- Partials      16095    16185      +90

Impacted Files
Coverage Δ

...c/main/java/org/opensearch/index/IndexService.java
74.82% <0.00%> (-0.18%)
:arrow_down:

.../replication/checkpoint/ReplicationCheckpoint.java
24.24% <24.24%> (ø)

...eplication/checkpoint/PublishCheckpointAction.java
29.26% <29.26%> (ø)

...plication/checkpoint/PublishCheckpointRequest.java
36.36% <36.36%> (ø)

.../indices/replication/common/ReplicationTarget.java
63.41% <63.41%> (ø)

...in/java/org/opensearch/index/shard/IndexShard.java
70.06% <66.66%> (+0.90%)
:arrow_up:

...ch/indices/recovery/PeerRecoveryTargetService.java
52.18% <68.00%> (-0.18%)
:arrow_down:

...rg/opensearch/indices/recovery/RecoveryTarget.java
70.83% <69.56%> (-4.59%)
:arrow_down:

...ices/replication/common/ReplicationCollection.java
71.56% <71.56%> (ø)

...nsearch/index/shard/CheckpointRefreshListener.java
100.00% <100.00%> (ø)

... and 476 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


ValueError: '\xe2' is not in list error when i try to train with color parameter

Hi @emedvedev
I get this error after a first few steps when I train with --color parameter (set channels to 3). Could you please help?
Please find the error log below:

This error means that your image labels have a character that's not in the model charmap. Check your labels, because by default the supported charset is numbers + uppercase only: https://github.com/emedvedev/attention-ocr/blob/7bb17af211de60e0fff5d56925f73d9018def744/aocr/util/data_gen.py#L23.

Thank you for the prompt response. I used --full-ascii and --no-force-uppercase flags as well, I assumed --full-ascii covers all characters. Am I missing something?

--full-ascii covers the ASCII range (uppercase, lowercase, symbols, etc.). There's no flag for covering the entire Unicode range, and you'd have to manually modify the CHARMAP (at the line I've mentioned before). Not sure about the performance in that case though—you might have to consider modifying the dataset labels unless Unicode plays significant part in it.

thank you for the reply. I am able to train the model. However, I now see a very weird issue. I added synthetic images (generated using GANs) to the training data, and few cropped COCO images. So about 1M synthetic images and Synth90k and about 60k coco images. The training loss is improving but when I test the model for prediction, it performs very poorly. It just prints "cccc" or "aaaaa" etc.
Am I doing something wrong?
Here's the training log:

Hi @kulkarnivishal ,
Did you manage to solve this problem?
If yes, please share a hint)

Not really. Although, I continued the training for 3 more weeks and results look better, not that great though. The main issue I am facing is predicting symbols. No matter how I train the inference seems to be getting it always wrong.

You can try to manually change the dictionary instead of using keys. At the same time I have no idea how you managed to add a non-ASCII labeled targets to the train-set .tfrecords file. When I tried to do this it used to throw an error of encoding all the time(

you mean manually adding symbols instead of using full-ascii flag?
And I used python's string.printable to filter non-ascii characters

@emedvedev , I han an error even though I chaned as you mentioned.
(@kulkarnivishal, how did you change?)
*error message.
ValueError: '!' is not in list

changed code
DataGen.CHARMAP = ['', '', ''] + list('!#$%&()*+-.0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ?@')

This error means that your image labels have a character that's not in the model charmap. Check your labels, because by default the supported charset is numbers + uppercase only:
  attention-ocr/aocr/util/data_gen.py

     Line 23
  in
  7bb17af

       CHARMAP = ['', '', ''] + list('0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ')

.

This error means that your image labels have a character that's not in the model charmap. Check your labels, because by default the supported charset is numbers + uppercase only:
  attention-ocr/aocr/util/data_gen.py

     Line 23
  in
  7bb17af

       CHARMAP = ['', '', ''] + list('0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ')

.

@emedvedev , in my case, aocr doesn't look to catch changed code.
I nn defaults.py, I changed max_prediction 8 to 18. but it still show 8.


Update watson-developer-cloud

OpenWhisk is using a 1.* version of the watson-developer-cloud package, which is now at 2.*. Please see if you can upgrade to that version.

Issue moved to apache/incubator-openwhisk-runtime-nodejs #10 via ZenHub


defalt key-map "ctrl+shift+k" can't be override by user setting

Dear team,
As we all know the "ctrl+shift+k" shortcut is set default as "delete one line" in sublime's key-map,
But  "ctrl+shift+k" has been override by Javatar's default key-binding.  I can't use "ctrl+shift+k" to delete one single line after installed Javatar.
I tried to override Javatar's default key-binding by setting user  key-binding like this:
{
"keys": ["ctrl+shift+t","ctrl+shift+m"],
"command": "javatar_util",
"args": {
"util_type": "parse",
"text": "#"
}
But still can't use "ctrl+shift+k" to delete lines.
could you pls help with this?

If you override your delete line to Ctrl+Shift+K, this may not work since Javatar key bindings always begin with Ctrl+Shift+K. So why the overriding key does not work? Because when you press Ctrl+Shift+K, Sublime Text is still confuse of which commands you are going to use since delete line has bind with Ctrl+Shift+K but also many remaining Javatar commands still begin with Ctrl+Shift+K so Sublime Text might decide to wait for another key combination. Hence, the override is not working.
To solve this issue you have 2 options...
First option, you need to override delete line to a new key binding that is not Ctrl+Shift+K.
Second option, you need to override all Javatar commands to a new key binding that is not Ctrl+Shift+K.
You can see what commands Javatar has in the Default (Platform).sublime-keymap file.


Add basic_string_view

This file should provide us with a c++17 compatible string_view object to allow use with std::string.
Prevent malloc() and or extra new() when using char* or std::string for read-only, and in the cases of copy alloc memory with std::string with emplace and std::forward only once.
I have tests for this class that are still being finished, but did not add it here as we are waiting for the https://github.com/apache/trafficserver/pull/2257 to finish first. Once that PR is pulled in I will add tests here or as a different PR to make sure we have good code coverage

ok @SolidWallOfCode  minus the BSD issue I think we can finalize everything else

I think we're close enough now, as nothing else depends on this.


Release Binary Assets CI is failling

Workflow release-binary-assets.yml is failing for all the targets with:
Error: Found argument '-V' which wasn't expected, or isn't valid in this context

I tried looking at what could be causing this issue but I haven't found any changes

this is going to be fixed with the v0.15.1 that is about to go out

Great, thanks!

fixed with v0.15.2


[WIP] Re-enable plugin filter

Fix #1231. It was disabled in #1224, due to lazy loading by the engine.
Using the plugins API, it is possible to list installed plugins.
One issue is that we don't have a way of storing arbitrary plugins in the TaskSpec, from which to match them against the node on which PluginFilter is applied. For volumes and networks, this happens via a separate code path that allows for attaching to networks or mounting volumes. This is something we might want to build to support general plugins, and will require proto changes. This PR will work for volume and network v2 plugins.
Signed-off-by: Nishant Totla<EMAIL_ADDRESS>

Current coverage is 55.56% (diff: 100%)

Merging #1517 into master will increase coverage by 0.38%

@@             master      #1517   diff @@
==========================================
  Files            82         82
  Lines         12952      12952
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           7147       7197    +50
+ Misses         4822       4778    -44
+ Partials        983        977     -6

Powered by Codecov. Last update 21b5af0...8bddc81

Isn't the issue that there's a catch-22? I'm not sure my memory is correct, but I thought requiring a certain plugin wouldn't schedule the task on a node until the plugin was loaded on that node, but the plugin might not get loaded until a task which uses it runs there.

@aaronlehmann I wasn't aware of this catch-22 situation. I was under the impression that a plugin is loaded externally on a node (say by an operator) and has nothing to do with SwarmKit. I'll try to get this clarified.

@stevvooe @mrjana what do you think about this change?

The preconditions for adding this back are in the TODO:

We can add this back when we can schedule plugins in the future.

I think the problem is that there are cases where the plugin isn't reported until it is actually in use.
What testing have we done to confirm this works?

Welp @tiborvass @vieux

After some discussion with @vieux, here's an update:
In the experimental part of engine-api, we have a function called PluginList, which lists all installed plugins on the engine (loaded or not). This is what we want.
There are two caveats though

PluginList will only list v2 plugins, there is no way to get a list of v1 plugins that haven't been loaded. So as long as users aren't using v1 plugins, we're good.
The v2 plugin functionality is still experimental, so nodes will need to be running engine with the experimental flag. We could either wait for this to get out of experimental, or else tell users that if they want plugin filter to work, they will have to run engines with experimental flag.

WDYT @aluzzardi @stevvooe @aaronlehmann?

cc @anusha-ragunathan

@nishanttotla : PluginsV2 can be queried using PluginList API endpoint. Also pluginsV2 will be stable in 1.13, so you dont need to worry about experimental support only.

@anusha-ragunathan thanks! Is there a PR to track, that's moving PluginsV2 out of experimental? We can then update this PR and merge after that one is.

@nishanttotla : There's no PR yet, but an issue tracking it. https://github.com/docker/docker/issues/26760

Also related: #1545

Picking this up again, after https://github.com/docker/docker/pull/28226 was merged.
I will update this PR using PluginList to list plugins instead of relying on docker info. This will make installed plugins available. We will need to vendor in docker/docker for that though. Should I do it, or will it cause other issues? cc @aaronlehmann @LK4D4 @mrjana

In 1.13, we will support pluginv1 (where plugins are queried via docker info) as well as pluginv2 (where plugins are queried via docker plugin list). So the correct approach in swarm would be to query both, rather than use pluginv2 alone.

@anusha-ragunathan thanks for the heads up. In the case of Swarm mode though, v1 plugins are tricky because they won't show up in docker info unless they're actually loaded/running, but the scheduler will not schedule until the plugin actually is running (which won't happen unless a task uses it).
So we might have to stick to v2 plugins for Swarm mode.

What about the case where the v1 plugin was loaded and running before the node joined the swarm? Node had some containers using the plugin. So plugin was loaded and running and docker info shows the plugin. Then node joins a swarm. For such a node, we should honor plugin v1.

@anusha-ragunathan you're right, I hadn't considered that case. Fortunately, it won't be hard to support both, so I will update the PR accordingly.

Are these test failures related? @LK4D4

Looks similar to https://github.com/docker/swarmkit/issues/1662
It is unrelated.

@nishanttotla This needs testing (at the very least manual). e.g. install a plugin on a machine and then deploy a service using that plugin (and make sure it gets deployed ONLY to that machine)

@tiborvass @stevvooe @anusha-ragunathan Updated the PR to save plugins based on driver type. PTAL.

I think there may be fundamental problems with the plugin data model that need to be worked out before going forward with this.
For example, the plugin list reported via NodeDescription is actually a list of drivers (we should probably rename this), but the new plugin model creates "capabilities". What are these capabilities? How do they work? I have seen a few examples in discussion but the results look foreign. Do we have a list of capabilities for a plugin?
For this to work properly, we really need a feedback loop of installation of a plugin to whatever the result is for a running plugin. This only lists the plugins and reports them, and doesn't provide any indication if they are the right state or the correct plugin.

@nishanttotla @aluzzardi : Any updates on this? Are you blocked on something? Is there something I can help with. We need this PR for 1.13-rc5.

I am going to carry this PR. I've fixed some of the outstanding issues, but I've discovered a big problem that prevents this from being practically useful for volumes.

Creating a volume is supposed to be idempotent. Some parts of the engine code assume this, for example registerMountPoints. However, an inconsistency around tags causes problems here. If I try to create a volume using the driver vieux/sshfs, using the same parameters I used to create it before, it will fail because the volume is already associated with vieux/sshfs:latest. This is a big problem because it means I can't create a service using vieux/sshfs as a volume driver without including the latest tag.

...but including :latest doesn't work either. In this case, no suitable node will be found to run the service, because the vieux/sshfs plugin reported by the agent (based on the Name field) won't match vieux/sshfs:latest.

This is tricky to get right. We should be consistent in the way we refer to plugins. We shouldn't just strip the tag off, because that loses information. We also need to make sure that we handle other cases like name:tag@digest matching name@digest (since the tag is purely advisory in this case).
What we probably need as a starting point is a canonical function that compares plugin references, adding a default tag where necessary and handling digest cases properly. However, even this won't fix (1), since the volume code is where this comparison is happening, and it does a simple string comparison on the volume driver name, which already had the tag included. I'm not sure what the right solution to this is.
cc @anusha-ragunathan @vieux @dmcgowan

Discussed with @anusha-ragunathan and @tonistiigi and learned that digests will never be used in plugin naming. The canonical name of a plugin is name:tag. If no tag was specified at pull time, latest is implicit.
I will fix (1) in docker/docker by fixing the driver comparison in the volumes code.
I will fix (2) in this PR (or a carried version) by comparing tags in the plugin filter, and assuming latest if no tag is specified.

Carried in #1808


NameError: name 'RuntimeValueString' is not defined

I ran into the following error when trying to extract types into a separate .wit files:
Traceback (most recent call last):
  File "/home/danielbevenius/work/wasm/wasmtime-py/demo-module/demo.py", line 12, in <module>
    main()
  File "/home/danielbevenius/work/wasm/wasmtime-py/demo-module/demo.py", line 8, in main
    ret = wit.something(store)
          ^^^^^^^^^^^^^^^^^^^^
  File "/home/danielbevenius/work/wasm/wasmtime-py/demo-module/dist/exports/wit.py", line 27, in something
    variant = RuntimeValueString(list)
              ^^^^^^^^^^^^^^^^^^
NameError: name 'RuntimeValueString' is not defined

I've created a reproducer for this might help troubleshoot this and the readme contains a note about this perhaps being an issue with a missing module/namespace. The same .wit files work with JavaScript and Rust.

Thanks for the report! This is probably a mismatch in how variants are represented and how they're lifted, and probably shouldn't be too hard to fix. I don't have time right now myself to tackle this but I'd be happy to review a PR!


Improve loading of G4 physics lists

In gitlab by @simonspa on Jul 21, 2017, 23:26
Playing around and also reading http://geant4.cern.ch/collaboration/working_groups/electromagnetic/physlist10.3.shtml we currently only allow loading of emstandard lists, while emstandard_optX or others might provide interesting results.
We should support that, e.g. https://www.research-collection.ethz.ch/handle/20.500.11850/164813 suggests the PAI physics list for thin scatterers as we usually have them.

In gitlab by @simonspa on Jul 24, 2017, 10:54
I needed to explicitly set
cmake -DGEANT4_INSTALL_DATA=ON ..

otherwise it would find the emstandard but no other physics lists...
Should we somehow make that even more clear in the manual? (I think it is already fine)

In gitlab by @simonspa on Jul 21, 2017, 23:29
changed title from Im{-rp-}ove loading of G4 physics lists to Im{+pr+}ove loading of G4 physics lists

In gitlab by @simonspa on Jul 24, 2017, 11:14
Aye.

In gitlab by @Koensw on Jul 24, 2017, 11:14
Bit surprising indeed. We can explicitly mention it, but then we probably also have to explain a lot of other Geant4 installation issues. I would also just go with specifying a list of flags as we have now.

In gitlab by @simonspa on Jul 24, 2017, 11:14
closed


branch-2.1 fix some tests in external p0

Also move the analysis exception of "Not support insert with partition spec in hive catalog."
from create sink phase to bind sink phase.
So that when set enable_fallback_to_original_planner=false;, the return error will be correct.

run buildall

run buildall


24w10a

Todo:

[X] Banner patterns
[X] Wolf variants
[X] Components in recipes
[ ] Misc. item component changes

That should be all of the component changes, just remembered the slot field was added so I'll work on that next and that should be everything

Since there are so many possible slot values (161 total I think?), I currently have the keys for the slot field accept any string but have the collection slots (container.*, enderchest.*, etc.) suggested by default. Might not be the best solution, I can change it if needed.


Update google spreadsheet with prod dates

Update this spreadsheet after a cache has successfully been pushed to production.

This has been done for a while.


credview: quick fix for password glitch

Sets currentDeviceValidated  to true for all devices that does not have OATH enabled, need to clean this up a bit later to make it more clear what it means.

Which pw glitch is this?

When inserting a PW protected device, the "homeview" is visible for a second before the password prompt


[Feature Request] Anchor tags in pagination

Existing Component
Yes
Component Name
Pagination
Description
Currently, pagination uses by default buttons.
For SEO reasons using anchors with a different link for every page is a much better option.
It would be great if such a feature existed

:+1:
Also there should be a way to use  in this way.


How to specify a GP-LVM with p(x)~GP

the original setting of latent space p(X) is i.i.d Gaussian distribution.
However, I want to place a Gaussian Processes prior on the latent space p(X)~GP(0,k_{t,t'})
Is GPflow able to do this?
If can, how?
Please!

There's a paper that does exactly this http://papers.nips.cc/paper/4330-variational-gaussian-process-dynamical-systems.pdf, but now we'd refer to this as a deep gp. If you apply natural gradients to https://github.com/ICL-SML/Doubly-Stochastic-DGP you get very nearly the same inference (it doesn't use the kernel expectations, but including them is fairly straightforward). I'm planning to put up an example of this soon.

Thank you for your response!
As far as I know, the latent space of GPDM is under Markov assumption.
That's a natural choice to make t+1 prediction.
However, in the V-GPDM paper, the latent space is a GP prior for smoothing, how to make t+1 prediction?
I don't know if this is a naive question because I am not familiar with recent GP papers.

From memory (it's been years since I read that paper) you should use the usual GP predictive distribution to get a the distribution in the latent space for the next t+1, and then pass this through the GP from the latent space to the observations and the likelihood. Hope this is a reasonable hint.
Feel free to reply again, but I'm closing this issue since it's more of a general GP question rather than an actual issue with the software.


WIP: Derive Default for builders

Possible replacement for #905.
cc @antoyo

CI pass with just Derive (modulo regen_check).
As I remember @antoyo want derive Clone too, so I try to add it too.
PS. Was in wrong PR 😉

Seems CI like derive Clone too.
@antoyo You prefer with Clone or just derive Default?

@EPashkin: Both.

Updated

CI passed

Thanks!


Improvements to Color

Adds support for System.Windows.Media.Color (WPF).
Casting between Color <=> {System.Drawing.Color, System.Windows.Media.Color} is now implicit since alpha values are now preserved.
0xFF is now treated as opaque internally in the Color struct, and 0x00 as transparent*.
Added PublicAPI attributes to some properties, constructors, and methods.
Moved color definitions to their own file for easier management.
Alpha values are now considered in conversion and comparison.
Updated unit tests for new changes.

*: Since Razer has said they do not support alpha, it should be safe for us to store alpha information in the 4th bit of the color value for conversion purposes, as Razer should be ignoring this value anyway due to them having said they do not currently support alpha.
Input/more information on the above note is appreciated.
Dependency on PresentationCore
Will the dependency on PresentationCore (System.Windows.Media) create problems?

This is a lot of work and from reading through looks fine.  Have you tested this implementation?

The Color struct has 100% unit test coverage.

I know that, but test coverage is only a measure.  We should do exploratory testing also.

Like what?

Cool, I'm happy with this and brief testing.  I think the dependency is fine as WPF is quite popular.  Happy to go ahead?

Yes. People will have to add PresentationCore in projects using Colore. I wonder if there's a way to make NuGet add dependencies.

There is. https://docs.nuget.org/create/nuspec-reference#specifying-dependencies-in-version-2.0-and-above


Bug + Feature

Hello @cybersteel8
I'm using your gamechange.sp file
I did found a bug:
As I typo: !gm comp de_dust2 it doesn't switch.
Cyka Means Cat: !gm comp de_dust2
[GM] Changing gamemode to Competitive...
[GM] Changing to de_dust2
[GM] Calling changelevel de_dust2de_dust2de_dust
L 02/17/2021 - 12:38:55: [SM] Exception reported: Tracker stack is out of bounds
L 02/17/2021 - 12:38:55: [SM] Blaming: gm.smx
L 02/17/2021 - 12:38:55: [SM] Call stack trace:
L 02/17/2021 - 12:38:55: [SM]   [1] Line 71, plugin.sp::Command_Gamechange
CModelLoader::Map_IsValid:  No such map 'maps/de_dust2de_dust2de_dust.bsp'
changelevel failed: de_dust2de_dust2de_dust not found

And features; Could you add retakes?
game_mode 0 / game_type 0
sv_skirmish_id 12

Seems the author is inactive. /closed.

Hey mate, sorry, yeah I didn't actually realise anyone would use this, I made this for personal use but thought I'd publish it here in case it was useful.
I think I have already fixed this in the dev branch, I'll publish a new version soon after I do a bit of testing and you can try that next one.

Hey, @cybersteel8 sounds cool! =)

Hey @cybersteel8
I did tested your dev. nothing happened but I did found this:
L 03/29/2021 - 15:26:13: [SM] Exception reported: Tracker stack is out of bounds
L 03/29/2021 - 15:26:13: [SM] Blaming: gm.smx
L 03/29/2021 - 15:26:13: [SM] Call stack trace:
L 03/29/2021 - 15:26:13: [SM]   [1] Line 55, plugin.sp::Command_Gamechange
L 03/29/2021 - 15:26:22: [SM] Exception reported: Tracker stack is out of bounds
L 03/29/2021 - 15:26:22: [SM] Blaming: gm.smx
L 03/29/2021 - 15:26:22: [SM] Call stack trace:
L 03/29/2021 - 15:26:22: [SM]   [1] Line 71, plugin.sp::Command_Gamechange

That's also a weird one, I wasn't sure what used to cause it but that one kind of magically resolved itself in my latest version. This is the exact copy of my mods that I'm running on my server, can you give them a try?
mods.zip
Also, I highly recommend updating your copy of sourcemod to the latest one. I just tested it now on build 6502 (stable) and I am able to change gamemode and include the map in the command and everything fine. Please let me know if the attached version works for you

Hi. @cybersteel8
The compiled gamechanger from mods.zip it works perfectly.
Do you think; you can add "Retake" to the menu? VALVe's has own retake function.
Retake (if you could add it)
game_mode 0
game_type 0
sv_skirmish_id 12

Thanks, I'm glad that worked for you.
Yeah I'll consider that as a feature request, it's a good idea!

@cybersteel8 let me know, once you have updated it. :) could be nice, since I use your plugin on my services =D

Any update about this? :D


Decaying learning rate and other schedules

Pass "reporter" around to optimizers and models in order to be able to implement decaying learning rates and other decays, for instance used for guided alignment.
OpenNMT allows to mutiply the learning rate with a decay-factor for each new epoch or whenever the cost on the validation set does not improve by at least a given factor.
Maybe we should also rename "reporter" if we use it for things like that.
Options to be added would be for example:
--learning-rate-decay 0.7 --start-decay-epoch 5

What happens at each epoch after epoch 5 is:
lr = lr * 0.7

Done and looks good.


Added support for ResearchPal Forked and Fluffys Research Trees queue function

I got a request of updating a similar mod, Pause On Research, but since this already was maintained Ill just notify the requestor about it instead.
The other mod however had a request that when using a research-tree mod that allows for queueing of projects, that it should only pause when there are no projects left.
So this PR will add support for that, with the appropriate mod-option.
It also adds support for pausing when showing the tree when using Fluffys ResearchTree as it overrides the window-type.
ResearchPal Forked already has its own setting for pausing so if that is loaded it will inform about that instead of showing the setting in this mod.
Hope you find it useful!

Apologies on the very very long delay getting back to you on this PR..
I've merged in some of the syntax changes, but for now I'm going to keep the other changes related to supporting other mods out for now. I'd like to keep this mod as simple as possible and avoid referencing other mods as much as possible, mainly to avoid having to test all the possible combinations.
Hope you understand and thanks for taking the time to put this PR together originally!


Does the Manifolds extension maybe not work?

Yesterda< I removed the cache from the Quarto tutorials CI over on Manifolds.jl and merged said PR today. Today the rendering (even in the notebook) there an error
https://juliamanifolds.github.io/Manifolds.jl/latest/tutorials/getstarted.html
I am not sure why that is even happening over there since the tutorial environment and the tutorial do not use ManifoldDiff I think, but the error sounds like it is coming from here?
edit: I get the same error, then I start an empty environment and install Manifolds.jl

Hm, we probably should move the manifolds extension to Manifolds.jl.

I have not yet understood what the actual error here is and why ProductManifold is not found.
I am not sure for this one, where it fits better. Maybe in Manifolds is better if we consider that the central place.

The error makes little sense to me. Manifolds.jl depends on ManifoldDiff.jl though so moving these parts to Manifolds.jl seems reasonable and would solve this issue.

sounds good, then lets move that ext.

I will do that then.


Atom 1.40.1 [Linux]: find in buffer does not work

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Find of find-and-replace Atom core package does not search in current buffer in Atom 1.40.1. Find-in-project still works. Everything is fine in Atom 1.39.1.
Steps to Reproduce

Open a file in Atom
Run find-and-replace:show command
Try searching for something

Expected behavior:
Find and replace finds text present in the file
Actual behavior:
Find and replace never finds anything
Reproduces how often:
100%
Versions
Atom    : 1.40.1
Electron: 3.1.10
Chrome  : 66.0.3359.181
Node    : 10.2.0
apm  2.4.3
npm  6.2.0
node 10.2.1 x64
atom 1.40.1
python 2.7.16
git 2.22.1
Additional Information
A video showing the issue https://youtu.be/IdpzJDsIyos

Thanks for the report!  I was unable to reproduce with 1.40.1 on ubuntu 19.04 just now - have you already tried temporarily resetting to factory defaults to see if that makes any difference?
Also what distro are you using?


Yubikey 2 factor

I can't use your service because I sign into google with 2FA via a yubikey, and that only works because of a built-in extension in Chrome. I've seen apps force text codes to be sent before so if the former is difficult, perhaps this might be an easier solution.

Of course as I write this post I find the try something else button at the bottom of the yubikey error and got logged in. Nonetheless, would it be possible to make yubikey happen? I like it a lot. Much more fun than typing things in. NBD regardless.


3+-dimensional arrays

The Type::validate method has a check that rejects multidimensional arrays with more than 2 dimensions, e.g. Array(Array(Array(T))): https://github.com/Protryon/klickhouse/blob/master/klickhouse/src/types/mod.rs#L811
Comparing
https://github.com/Protryon/klickhouse/blob/master/klickhouse/src/types/deserialize/array.rs#L25
with the python implementation
https://github.com/ClickHouse/clickhouse-connect/blob/main/clickhouse_connect/datatypes/container.py#L37
I was not able to find what breaks down for 3 or more dimensions. The Python implementation is more efficient in that it reads all the data at once rather than recursing, but this is all I see for now.
The following test passes both with integers and with strings as inner type (after commenting out the check):
#[derive(Debug, Clone, klickhouse::Row, PartialEq)]
struct Row {
    a: Vec<Vec<Vec<u8>>>,
}
#[tokio::test]
async fn test_3d_array() {
    let client = super::get_client().await;
    let items = vec![
        Row {
            a: vec![
                vec![vec![1, 2], vec![3], vec![4, 5, 6]],
                vec![vec![7], vec![8, 9]],
            ],
        },
        Row {
            a: vec![
                vec![vec![10], vec![11, 12, 13], vec![14]],
                vec![vec![15, 16], vec![17]],
                vec![vec![18]],
            ],
        },
    ];

    client
        .execute("drop table if exists test_serialize")
        .await
        .unwrap();
    client
        .execute(
            r"
    CREATE TABLE test_serialize (
    a Array(Array(Array(UInt8)))
    ) ENGINE = Memory;
    ",
        )
        .await
        .unwrap();
    client
        .insert_native_block("insert into test_serialize format native", items.clone())
        .await
        .unwrap();

    let items2 = client
        .query_collect("SELECT * from test_serialize")
        .await
        .unwrap();
    assert_eq!(items, items2);
}

Do you remember what the reason for this restriction was?

My previous implementation for array parsing was quite broken and I fixed it a while ago. Should be fine to remove the restriction.

Done in #38


Better handling of the AD auth part

admin group for human users
admin group for API users
user group for read-only users
require_groupmembership to true by default

Hello,
I still don't understand why you have 3 commits, could squash them?
Regards,
JM


Better error handling for HTTP responses

It seems that Threat Intel does not handle very well HTTP error responses, e.g. 403 Forbidden.
They are rather ignored silently.
Some error handling, e.g. throwing an exception, might be a better way of dealing with these situations.

If you point me to the sections of code you're worried about, I can take a look too.

So when I was investigating this recent errors in OSXCollector LookupDomainsFilter test I was supplying wrong OpenDNS API key and I was getting 403 from their service.
This apparently was treated like an empty response because it was getting filtered out by this list comprehension.
But to be very honest I have no idea how to distinguish 403 in this case with some other empty responses that were particularly  annoying and made you add the retry thing.

Would it make sense to break the loop and raise a connection error if response.status_code == 403? I think it's safe to assume that if authentication is the problem, retrying won't make the request succeed.

I think this is a great idea :)

Closed in #54.


Logging prefix set to EDGE_SDK_

All logging macros now include EDGE_SDK as a prefix
Resolves: OLPEDGE-407
Signed-off-by: Sergii Vostrikov<EMAIL_ADDRESS>

This is a huge change. Please split into multiple small commits.

Change Log.h macros, add #defines to keep old macros alive until all classes are adapted.
Adapt module based macros.
Remove old macros.

Your proposition would generate unnecessary many new pull-requests. The ticket stands for adding log prefix for all macros, without a
The changes in the review if quite repetitive and easy to follow (despite many files affected).
I doubt if there any benefits of spending resources on generation additional 58 self-dependent pull-requests.

I think we need to make a conversation on necessity of such splits.

Discarded as per discussion above


Cleanup and refactoring

Description
Why was this PR created?
Development notes
What have you changed, and how has this been tested?
Checklist

[ ] Read the contributing guidelines
[ ] Open this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Update the documentation to reflect the code changes
[ ] Add a description of this change and add your name to the list of supporting contributions in the CHANGELOG.md file. Please respect Keep a Changelog guidelines.
[ ] Add tests to cover your changes

Notice

[ ] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (c29efcc) compared to base (c2e8055).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #382   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           27        27
  Lines          922       922
=========================================
  Hits           922       922

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Just for future reference

Likely not worth look into those, because they seem very rare corner cases. But if it happens again to my syncs, I'll solve them.
Traceback (most recent call last):
 File "<censored>/./md5sum", line 119, in <module>
   main()
 File "<censored>/./md5sum", line 21, in main
   check_checksums(args.method, args.check, name)
 File "<censored>/../md5sum", line 62, in __init__
   lines = f.readlines()
           ^^^^^^^^^^^^^
 File "<frozen codecs>", line 322, in decode
UnicodeDecodeError: 'utf-8' codec can't decode bytes in position 4421-4422: invalid continuation byte

and
Traceback (most recent call last):
  File "<censored>/./md5sum", line 128, in <module>
    main()
  File "<censored>/./md5sum", line 21, in main
    check_checksums(args.method, args.check, name)
  File "<censored>/./md5sum", line 74, in __init__
    for stdout, stderr in pool.imap(self.check_checksum, lines):
  File "/usr/lib/python3.11/multiprocessing/pool.py", line 873, in next
    raise value
  File "/usr/lib/python3.11/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
                    ^^^^^^^^^^^^^^^^^^^
  File "<censored>/./md5sum", line 99, in check_checksum
    checksum, filename = line.rstrip("\n").split(maxsplit=1)
    ^^^^^^^^^^^^^^^^^^
ValueError: not enough values to unpack (expected 2, got 1)

For now, I just replace
https://github.com/99991/parallel-md5sum/blob/ecd1467ac4a22421b7b86073e1ce919e5ba0a6b5/md5sum#L61 with
       with open(checksum_filename, errors='ignore') as f:
and
https://github.com/99991/parallel-md5sum/blob/ecd1467ac4a22421b7b86073e1ce919e5ba0a6b5/md5sum#L90 with
        try:
            checksum, filename = line.rstrip("\n").split(maxsplit=1)
        except ValueError:
            pass

The checksum file probably contains text which is not using UTF-8 encoding, but uses some other encoding instead. Could you run file -i <your_checksums.txt> to check?
If that is the case, you can try converting your checksum file to UTF-8 encoding like so:
iconv -t <old_encoding> <your_checksums.txt> -o <your_checksums_utf-8.txt>


[Feat]: Add Backend

What feature?
I have an idea i.e.
we need to add a backend
we will store data in mongoDB
google auth i.e. login with google
API
etc
Add screenshots
Add screenshots
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hello @JasonDsouza212 ,
What's your take on this
If you want the task to done
Do assign me under gssoc'32.

Okay


1.16.0 / 1.16.1 / 1.17.0 : @remix-run/v1-route-convention do not save new route

What version of Remix are you using?
1.17.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce

Install 1.17.0
Configure v2_routeConvention with @remix-run/v1-route-convention
Run project
Add a route
After a very long rebuild time (issue already created before) the route isn't registered

Precision : Do not use the new dev server but the historic one.
Expected Behavior
The route is registered and accessible without full reload of dev server.
Actual Behavior
The route is not registered.

Is there any update ?

Can you provide a reproduction?  This works fine for me on a brand new app, you can see in the logs that I created a new file and it was picked up in 91ms.
Here's the remix.confg.js:
const { createRoutesFromFolders } = require("@remix-run/v1-route-convention");

/** @type {import('@remix-run/dev').AppConfig} */
module.exports = {
  ignoredRouteFiles: ["**/*"],
  serverModuleFormat: "cjs",
  future: {
    v2_dev: false,
    v2_errorBoundary: true,
    v2_headers: true,
    v2_meta: true,
    v2_normalizeFormMethod: true,
    v2_routeConvention: true,
  },
  routes: (defineRoutes) => createRoutesFromFolders(defineRoutes),
};

> remix dev

 💿  remix dev

 warn  The `remix dev` changing in v2
┃ You can use the `v2_dev` future flag to opt-in early.
┃ -> https://remix.run/docs/en/main/pages/v2#dev-server
┗
💿 Building...
💿 Rebuilt in 175ms
Remix App Server started at http://localhost:3000 (http://<IP_ADDRESS>:3000)
GET / 200 - - 55.512 ms
💿 File created: app/routes/page.tsx
💿 Rebuilding...
💿 Rebuilt in 91ms
GET / 200 - - 4.885 ms
GET /page 200 - - 11.220 ms

I'm going to close this out due to the lack of a reproduction but if you can prepare one please let us know and we'll get this opened back up.  Thanks!


🐛 Button's font not changed

Hi,
thanks for this great package first of all.
I noticed that changing the font of an app doesn't change the font of some buttons (TextButton, Button).
The result is an inconsistent look and feel.

Can you provide a reproducible example?

Sure, here you go: https://github.com/slimyjimmy/wrong_font_example

Sorry, but I can not reproduce this on the lastest version.


Do not require object identifiers for add function

Have the code figure out what type the object is.
Currently, code works like this: add <type> $object to <type> $target, which isn't very nice.
Either set up $target.add($object) or add $object to $target

Supports all data types as of 24 April 2015.

Note: Using add $object to $target method.