Dependency Error

Hi All,
I am facing an error while uploading an .app file on a Cloud Sandbox environment.
Its seems a dependency error.
Can anyone suggest any solution, since i uploaded all the dependency first and then trying to upload this .app file.
Below is the error.
Deployment failed. Errors: App ID : 7e996381-60d3-403c-b420-b60e7eab9b5c Message : { Cannot synchronize the extension because no synchronized extension could be found to satisfy the dependency definition for ResponsibilityPage Extension by NetsmartzLLC <IP_ADDRESS>. } - Job Id : 31b5f229-f9f2-4cdd-aeee-0590265ebbab
Below is extension upload summary for which current .app file is giving error.

Can you share with us app.json file of that extension?

This problem occurs when a newer version of the dependency (in this case "ResponsibilityPage Extension") has been synchronized on the tenant database, but is no longer published. It appears that version <IP_ADDRESS> of "ResponsibilityPage Extension" was published as a developer extension and then unpublished later. Therefore, we detect that the synchronized version of the dependency is v<IP_ADDRESS>. Since v<IP_ADDRESS> is no longer published, you'll get this error "Cannot synchronize the extension because no synchronized extension could be found to satisfy the dependency definition for ResponsibilityPage Extension by NetsmartzLLC <IP_ADDRESS>." Even though v<IP_ADDRESS> satisfies the dependency, it is not technically considered the "synchronized" version.
We are working on making a change to the platform logic to consider ALL published versions of the dependency, rather than only the "highest sync'd version". The new logic would detect that there is a published/sync'd version that is greater than or equal to v<IP_ADDRESS>. In the meantime, you can mitigate this issue by ensuring that the "highest sync'd version" is published on the server.
Mitigation: Deploy "ResponsibilityPage Extension" v<IP_ADDRESS> and then retry the Deploy operation for the dependent extension.

Hi @ShaylaG-MSFT ,
Thanks for your prompt revert.
I tried your suggested solution, but now the dependent extension giving error for another dependency extension, like below. Although it has been deployed successfully on Cloud sandbox without any problem. Problem seems related to Table Change, but i am not able to find which table change i have done in this extension and how to recover that.

Please help me out.

Back on April 24th, you deployed version <IP_ADDRESS> of the extension that you are currently trying to upload. (It was deployed as a developer extension via VS Code). That version of the extension contained a Table Extension called "AssemblyLineExt" which extended base Table 901 "Assemble Line". In your latest version of the extension, you must have removed this table extension.
The easiest mitigation for this is to follow these steps:

Deploy the latest version of your extension using VS Code. Make sure to set the "schemaUpdateMode" to "Recreate" or "ForceSync" in the Launch.json. This will get the schema to synchronize correctly and remove any database schema associated with the missing Table Extension. Here's documentation around the difference between Recreate and ForceSync: https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-retaining-data-after-publishing
Uninstall and unpublish the extension so that this developer extension doesn't interfere with your upload attempt.
Try to upload the extension as a Per-tenant extension again.

Hi @ShaylaG-MSFT
Thanks for your help!!
But, it solved after creating a new sandbox and uploading all apps on that sanbox.


Added support for Azure cosmosDB for alarms package

Added support for alarms package to use cosmos DB.
All database interactions will now go through database.js which in turn will delegate to actual DB implementations (couchdb or cosmosdb) based on dbType.
There are some TODO comments and need to get better understanding on those.

@sandeep-paliwal - have you tested this with couchdb?  fails for me
"namespace": "whisk.system",
    "name": "alarmWebAction",
    "version": "0.0.2",
    "subject": "whisk.system",
    "activationId": "5d9dfa5f98cb4f629dfa5f98cb2f6232",
    "start":<PHONE_NUMBER>476,
    "end":<PHONE_NUMBER>495,
    "duration": 19,
    "response": {
        "status": "application error",
        "statusCode": 0,
        "success": false,
        "result": {
            "error": {
                "message": "Cannot read property 'getWorkerID' of undefined",
                "stack": "TypeError: Cannot read property 'getWorkerID' of undefined\n    at common.verifyTriggerAuth.then (/nodejsAction/dUGHWGoL/alarmWebAction.js:135:26)\n    at process._tickCallback (internal/process/next_tick.js:109:7)"

@sandeep-paliwal  Please run all tests to verify it works once you fix this issue.  The getDatabase function in alarmWebAction needs to be a promise that you wait on when you call it.  You are returning the database but only after you wait on the initDB.  The call to getDatabase right before you try to getWorkerID is undefined because you are not waiting on it.

@sandeep-paliwal I also see some alarms functionality that exists with couchdb that was not made available with cosmosDB as part of this PR:

The internal monitoring/health code that creates and deletes trigger db entries found in provider/lib/health.js
The trigger feed cleanup that occurs for the fire once alarm when the deleteAfterFire param is not set to false (see the handleFiredTriggers function in provider/lib/utils.js)

Thanks for the comments @jasonpet . I will update the PR and run the tests you have mentioned.


Do 5396 return dispatch fix failing workflow

@dan-rosenbrock seequent aye? Your office is 10 minutes away from my house ;)
I'd recommend merging origin to your fork as I have fixed some things here and there, although if the only difference is an additional vuln scanner then perhaps it's easier for maintenance to make requests here as I'm fairly active 🤷


all: reopen tree for Go 1.23 development

Current Tree Status: Tree open for Go 1.23 general development (see announcement)
This is a tracking issue for the upcoming task of reopening the tree for Go 1.23 development. (It's created a little early to create room for planning early CLs/branches to land during tree reopening.)
As usual, the tree will initially be open to changes that must land early:

[ ]  Bump internal/goversion.Version to 23 and update the go directive in the go.mod files —this should be the very first CL to be submitted as it marks the start of master branch representing Go 1.23 (rather than Go 1.22). (Example CL.)

[ ]  Any CLs associated with issues labelled as early-in-cycle.

[ ]  Submit CLs that are already approved and marked AutoSubmit, but blocked on wait-release

[ ] Finally, open the tree for all general Go 1.23 changes.

CC @golang/release.

Closing as a duplicate of the earlier #64340.


Getting a translation error for an 'option' picker

Hello, I'm getting started with Bullet Train and I'm testing out creating a simple crud model that has  name:text_field and category:options. When trying to create one of these I get the following error:
ActionView::Template::Error (translation missing: en.strengths.fields.category.options):
And am not sure why. Any tips on where to check / how to fix?

Hello @TheColorPlum, I used the following commands and was able to Super Scaffold options without any errors:
rails g model Foo team:references name:string category:string
bin/super-scaffold crud Foo Team name:text_field category:options

You can read the field partial docs in case you have any mismatches between your model's data types and our field partial types.


socketDelay broken by v9.0.8

PR #802 may have broken socketDelay.
As reported by https://github.com/node-nock/nock/pull/802#issuecomment-282994787
Steps

Repro
If reproducible, revert
Fix

@matteocontrini & @lpinca, what version of request were you using when you encountered this bug?

I'm got@latest but the request library has probably nothing to do with it.

@lpinca PR #802 was introduced specifically because nock timeouts were failing with certain versions of the request module. We have previously seen issues with nock restricted to specific HTTP libraries/versions.

I'm using request 2.79.0, which is the latest version

I was able to repro, will revert.

Just released v9.0.9, should be fixed. Thank you for reporting @matteocontrini, @lpinca & @matteocontrini. Please let me know if you continue to see issues.
We will reattempt #802 once #832 is complete.

@ianwsperber sorry I wasn't clear. What I meant is that I think the issue is not caused by a specific request library. Thanks for the quick "fix".

@lpinca I think you're right with that assessment, but I wanted to check just because the cause was related to a specific lib. The tests evidently didn't cover this use case 😢

Here is a test case to reproduce the issue.
'use strict';

const http = require('http');
const nock = require('nock');

nock('http://my.server.com')
  .get('/')
  .socketDelay(200)
  .reply(200, '<html></html>');

const req = http.get('http://my.server.com');

req.on('socket', (socket) => {
  socket.on('connect', () => {
    req.setTimeout(100, () => req.abort());
  });
});

req.on('error', () => {});

req.on('response', () => {
  throw new Error('fail');
});

This is what got does under the hood. Hope it helps.

It's better to add a test case into nock's test suite to cover it.

@aleung Feel free to open a new PR with @lpinca's test case and a reattempt at your fix.

@ianwsperber @lpinca @matteocontrini Could you read this and comment on it? I try to analysis this issue.

Please review PR #859, which is a new fix.


Error when deploying Icebreaker

I'm getting this error when trying to deploy icebreaker app.
{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"BadRequest","message":"{\r\n "code": "BadRequest",\r\n "message": "Database account icebreaker-storage-ik4e36oz5me72 is not online\r\nActivityId: d8d656fb-a5cc-4b07-aced-943029dbd476, Microsoft.Azure.Documents.Common/2.10.0"\r\n}"}]}

Are you using free trial ??? Azure is facing heavy traffic due to the global pandemic, Hence this issue on free subscriptions.

See #59

We are using a Paid subscription.

Received the same error message when deploying to a Pay-As-You-Go subscription for (US) East US. Any workarounds?

Received the same error message when deploying to a Pay-As-You-Go subscription for (US) East US. Any workarounds?

Please try North Europe as it worked for us

West Europe worked for me this time


Exited with 255

Error: Failed to execute driver, arguments: apps 9dd0b5ddc12a2bb1236191c5db5d8c849facbab4
enumerate_applications() got an unexpected keyword argument 'scope'
Exited with 255

m1 Mac OS: Darwin arm64 20.5.0

sudo pip3 install -U frida-tools
更新后解决


Created a Teach button to link to the Educator's site.

Karen wants this button and the educator's site is ready now.
@afmckinney @jisqyv @halatmit

@afmckinney @jisqyv @jsheldonmit Is there enough room on the page to include this, when people are logged in as administrators and have the additional admin and moderation headings?  Also, "teach" might need a little explanation.   If I were a userI'd assume that would lead to some kind of tutorial resource.   Maybe something like "instructors" would be better?   How does Scratch link to ScratchEd?

Scratch links to ScratchEd through a subtle "For Educators" link that is on the main landing page before you log in, but it isn't in the top menu bar. The link takes you to a page that explains what ScratchEd is, and then you click another link to get to the ScratchEd site.
Once you're logged in, it's at the bottom in the site footer in the About section.

I'd be ok with Instructors instead of Teach.

I would prefer the interstitial page, served off the main site (main
website or App Inventor service). I am very concerned about people
curiously clicking the link. Teach is a "bare" wordpress website, it cannot
handle the same load their our normal website can handle or the service can
handle.
-Jeff
On Mon, Jun 20, 2016 at 4:08 PM, kdclang<EMAIL_ADDRESS>wrote:

I'd be ok with Instructors instead of Teach.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/802#issuecomment-227254241,
or mute the thread
https://github.com/notifications/unsubscribe/ABhdOC_PQ7i4KA2IVqd7tRIla9GiF8Keks5qNvM9gaJpZM4I6BG-
.

--

Jeffrey I. Schiller
Information Systems and Technology – MIT App Inventor
Massachusetts Institute of Technology
77 Massachusetts Avenue  Room 32-386
Cambridge, MA 02139-4307
617.910.0259 - Voice
<EMAIL_ADDRESS>http://jis.qyv.name
http://appinventor.mit.edu

Hmm ...   Something like that might be better.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Mon, Jun 20, 2016 at 3:56 PM, ajcolter<EMAIL_ADDRESS>wrote:

Scratch links to ScratchEd through a subtle "For Educators" link that is
on the main landing page before you log in, but it isn't in the top menu
bar. The link takes you to a page that explains what ScratchEd is, and then
you click another link to get to the ScratchEd site.
Once you're logged in, it's at the bottom in the site footer in the About
section.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/802#issuecomment-227251220,
or mute the thread
https://github.com/notifications/unsubscribe/AAwLtyg9wgsRusb_RCZ-pnBnw2sBi4xvks5qNvBygaJpZM4I6BG-
.

Or maybe something more like what Natalie says Scratch does.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Mon, Jun 20, 2016 at 4:08 PM, kdclang<EMAIL_ADDRESS>wrote:

I'd be ok with Instructors instead of Teach.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/802#issuecomment-227254241,
or mute the thread
https://github.com/notifications/unsubscribe/AAwLt4oQiObZduBgYNHzeCUX_ByRHo7Mks5qNvM8gaJpZM4I6BG-
.

I liked the interstitial page anyway.  Your point about capacity just
reinforces that preference.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Mon, Jun 20, 2016 at 4:18 PM, Jeffrey Schiller<EMAIL_ADDRESS>wrote:

I would prefer the interstitial page, served off the main site (main
website or App Inventor service). I am very concerned about people
curiously clicking the link. Teach is a "bare" wordpress website, it cannot
handle the same load their our normal website can handle or the service can
handle.
-Jeff
On Mon, Jun 20, 2016 at 4:08 PM, kdclang<EMAIL_ADDRESS>wrote:

I'd be ok with Instructors instead of Teach.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/mit-cml/appinventor-sources/pull/802#issuecomment-227254241
,
or mute the thread
<
https://github.com/notifications/unsubscribe/ABhdOC_PQ7i4KA2IVqd7tRIla9GiF8Keks5qNvM9gaJpZM4I6BG-
.

--

Jeffrey I. Schiller
Information Systems and Technology – MIT App Inventor
Massachusetts Institute of Technology
77 Massachusetts Avenue Room 32-386
Cambridge, MA 02139-4307
617.910.0259 - Voice
<EMAIL_ADDRESS>http://jis.qyv.name
http://appinventor.mit.edu

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/802#issuecomment-227256875,
or mute the thread
https://github.com/notifications/unsubscribe/AAwLt6rqHGB3Eefvw_2Ztv-ISbUtpUdSks5qNvWzgaJpZM4I6BG-
.

I didn't think about the people who might just click out of curiosity. Feedback from teachers has been they seldomly go to the main site once they are teaching but are often in the developer site so we thought a button from there would be helpful to them. If you think it would put a strain on the teach site we can drop the idea. I already changed the main site teach link under resources to go to the teach site (per Josh's suggestion).

It's great to use the developer site to popularize the teach site.  But
people on the developer site should know that they are going to the teach
site and what it is.  That's the kind of info that the Scratch page (and
the proposed App Inventor interstitial page) could provide.
== Hal
Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
On Mon, Jun 20, 2016 at 6:01 PM, kdclang<EMAIL_ADDRESS>wrote:

I didn't think about the people who might just click out of curiosity.
Feedback from teachers has been they seldomly go to the main site once they
are teaching but are often in the developer site so we thought a button
from there would be helpful to them. If you think it would put a strain on
the teach site we can drop the idea. I already changed the main site teach
link under resources to go to the teach site (per Josh's suggestion).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/802#issuecomment-227283310,
or mute the thread
https://github.com/notifications/unsubscribe/AAwLt3dIdqCaS0OrMSt43rXxcv3w5PaUks5qNw2ugaJpZM4I6BG-
.


Test_run_execution class operator variable are lazy loaded from DB

From chip-certification-tool-backend created by sholagi: CHIP-Specifications/chip-certification-tool-backend#296
In #289, we introduced a workaround to ensure operator was explicitly loaded (wasn't before due to lazy loading).
This should be refactored, to use from_orm in pydantic directly on the new model, once they allow us to pass additional values for the new model eg. test_run_stats. This will let the schema handle the loading of the operator.
see: https://github.com/samuelcolvin/pydantic/pull/3375
Related PR: https://github.com/CHIP-Specifications/chip-certification-tool-backend/pull/289
Related Issue: https://github.com/CHIP-Specifications/chip-certification-tool-backend/issues/276

Deprioritized.


Accuracy issue

When I run this model on video, i've noticed the prediction changes pretty wildly from frame to frame, depending on the face dimensions, lighting, position, etc. But as soon as the face becomes closed to the camera, the predictions are more consistent.
Two questions

the default 32x32... why was that chosen?
could prediction be helped by training / running on larger dimensions? (64x64 or even much larger)

Hi @taewookim,

I simply chose 32x32 for efficiency in development and testing. It is not appropriate for deployment for its poor accuracy as you pointed out.
I think the image size should be larger if accuracy matters. The image size 64x64 is used in the sample code in README.md as python3 create_db.py --output data/imdb_db.mat --db imdb --img_size 64. In the original paper [1], the VGGNet is used; the input image size is 224x224.

[1] R. Rothe, R. Timofte, and L. V. Gool, "DEX: Deep EXpectation of apparent age from a single image," ICCV, 2015.

ah oops.. completely forgot about that. Thank you @yu4u

By the way, how do you figure out the upper limit on how far this architecture can be taken? I'm sure this is more of a hardware limitation as well as how much time I'm willing to wait on, but are there any guidelines?

Did you mean how to determine the size of the network?
I like WideResNet with moderate depth (16-22); it provides good trade-offs between accuracy and computational cost.

I tried training on 224x224 .. and the training process froze. When I check nvidia-smi, volatile GPU-util is @ 0% (screenshot: https://s8.postimg.org/uw615qxyd/image.png)
My setup
google compute instance
4vCPU with 30 gb RAM
Tesla k80 w/12gb VRAM
ubuntu 16.04 LTS
TF 1.30
Keras 2.0.8
cv2 3.2.0
cudnn v8
Using your exact same wide resnet, but with 3 outputs.
any ideas where it might've gone wrong? (Currently trying img size of 128x128)

@yu4u
I tried changing img_size to 224 and the model is performing really bad.   For each of the 3 dense layers  (i.e. classifying 3 things), loss was anywhere between 9 and 20 .. and accuracy  didnt go anywhere even above 0.25.  I didn't even bother finishing 30 epochs (i stopped at 28)
I am re-training with img_size at 64, but do you have any idea where I might have gone wrong and what are some things that I can tweak?

Currently, my implementation is not suitable for larger sizes of images (e.g. 224) for several reasons: 1) it saves and loads all raw images; it requires large amount of memory, 2) the stride of the network is only 4 (before pool (8, 8)) because small image size is assumed.
If you use the image size 224, it seems good idea to use officially supported pre-trained models:
https://keras.io/ja/applications/

so what's the largest  possible that would you'd recommend w/your implementation?

I think 64x64 is good enough because there seem to be many low resolution images in the dataset.

Hi, I tried this project with your pretrained model with default depth and width but it is giving the wrong prediction. for example it is predicting age 43 F of a 80-year female lady. It is predicting wrong on each and every image.

Please confirm that your backend and channel order are consistent with the pretrained model.

I am using your pretrained model and my hardware specs are different from your one. Is that an issue?

What is your backend?

Hi, I'd like to ask you where I can see the calculation of this model? because its accuracy is bad from 0.2 to 0.4

See:
https://github.com/yu4u/age-gender-estimation/blob/master/demo.py#L101-L105

Thank you for your reply. But  I mean train and test(validation) accuracy.
When I set the learning rate = 0.001 by all epochs, It would be trained for train dataset. but val_loss increased so there was an over-fitting(did not decrease val_loss)

The validation accuracy is automatically calculated by Keras using validation_data=(X_test, [y_test_g, y_test_a]).
Overfitting would be alleviated by data augmentation.

@yu4u Hi, so i'm running the model with a different face detector(mtcnn) but made sure the img_size is 64 and the input to the model is of the form (batch_size, 64,64,3), but the age prediction is bad compared to using dlib. The only difference with mtcnn is, its detecting more number of faces and the bounding boxes points vary. Can this affect the prediction of age?


Add support for "full screen" mode

This is more of an enhancement then a "issue".
It would be great to have an option to full-screen the HDMI stream. Some of the installers makes the display really small and not really visible (alpine linux being one of them...)

this would be awesome.. would save me zooming on my tiny laptop screen.

Fixed in #191


Epochs don't interrupt binaries with long time.Sleep durations

Hi!
I'm trying to execute the following go snippet compiled to WASM:
package main

import time

func main() {
    time.Sleep(10 * time.Second)
}

and am using an epoch deadline (set to "3"), which is being incremented every 100 milliseconds. In other words, the deadline for this is 300 milliseconds.
However, this isn't behaving as I expect using wasmtime-go: with the wasm trap only firing after the time.Sleep call has completed.
Is this expected behaviour?
Thanks!

This is sort of expected behavior, sort of not. It's expected in that, yes, with wasmtime-go that's expected. This is a caveat of epochs where epochs are used to cancel executing wasm code but don't assist in cancelling host code. For cancelling host code that's left up to the embedder. The primary way of doing that is to use the async support that Wasmtime has, for example https://github.com/bytecodealliance/wasmtime/pull/9184 and https://github.com/bytecodealliance/wasmtime/issues/9188 are similar issues to the one here but for the wasmtime CLI (which switched over to using async).
So the "not expected" part is that Wasmtime does natively have support to fix this, but that comes back to the "this is expected" part in that the async parts of Wasmtime aren't exposed in Go at this time. There's a C API but it's not bound in Go yet.

Hey @alexcrichton, thanks! Do you see a workaround to this issue: for imports that I can control, I think the workaround is straight-forward, we just need to ensure that we are respecting some timeout. However, AFAIU time.Sleep reduces to a call to the wasi syscall poll_oneoff which is not something I have control of. I can stub it out, but that raises the question generally of how we enforce timeouts on the wasi imports defined by wasmtime-go.
And just for my understanding: are you anticipating that support for these async parts of wasmtime will be released soon for wasmtime-go?
BTW if it's helpful I'd be happy to take a look at binding the relevant C API to Go if you can point me in the right direction?

Yeah unfortunately there's not a great solution at this time because as you're seeing the imports in question are the ones provided by Wasmtime. You could theoretically define your own implementation of WASI and skip adding Wasmtime's own implementation, but even then you'd still have to orchestrate timeouts and such yourself (e.g. wake up the host function that's blocked on sleeping once the timeout elapses).
Currently there's no one working on adding support for the async APIs to Go, so that's an open project which if you'd like you can pick up. Exactly how it would integrate within Go I'm not sure myself, though, so it'll probably require a chunk of design work to figure out how to best fit within Go's idioms.


Dynamic Lock IDs

More a question of use case then an issue -- unless the answer is "Cant do that"
I have some small 'microservice' type processes (running in docker ECS on AWS).  One thing they do is pocess S3 filels on change notificaiton.  Im subscribing to a SNS topic which then gets routeed and load balnced to an avoaible node.   For some operations I want to 'lock' a segment of code identifyed by the S3 Path -- Like a pool of ionotify workers.
Since the spae of possible key values is large and unknown upfront I cant pre-allocate locks.
Rather when a node gets an event I want it to 'lock' such that other nodes if they get the ssame event can be (cooperatively) kept from operrating on the same file at th3es ame time.
(inthe same process or a differnt process) In a period of an hour -- hundreds or thousands of unique keys/files may occur.  The operations while locked may vary from ms to munltiple minutes.  I realize I will likely have to treat these cases differnt -- (keep a map of locked node)
My question is this.  I tried a 'simplistic' approch which carates a NEW lock structure on eveyr eent, passing it the unique IID vailel Then after 'work' is complete releasing the lock.
Other apps may be doing the same thing and they should block in some way sothey dont both operatioe on the same file.   This seems to work --but I dont know if its effective.
For exmaple -- how 'expensive' is it to allodate a new lock.
If I simply 'unlock a loc does that delete it from Redis ?   What do I need to do to 'release' the lock  fully?  THen assumign all went well -- what is the latency to examime a lock ot sed if its sill in use ?
There will be hundreds of uniquie Ids --  Is there a better dataa type for this ?  Maybe a combonation of a Map and Lock ?
Any suggestions welcom.

how 'expensive' is it to allodate a new lock

Each lock allocates pubsub channel and holds until lock acquisition moment.

If I simply 'unlock a loc does that delete it from Redis ?

Yes, it does.

Is there a better dataa type for this ?

I'm afraid no. if you have an idea I'm ready to consider it.


fix casting list of non-proxy hosts from ScriptObjectMirror to List

Description
Thanks for contributing this Pull Request. Make sure that you send in this Pull Request to the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

Relevant Issues : (compulsory)
Relevant PRs : (optional)
Type of change :

[ ] New feature
[x] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

thanks ! merged.


Add Unix Domain Socket support

Adds the ability of configuring a Unix Domain Socket as the HTTP transport layer, instead of traditional network-based TCP.
UDS is useful for gRPC-based IPC solutions, where all communication between clients and servers is happening on the same machine, or even in the same process. UDS avoids networking overhead and can be made secure by fine-grained filesystem permissions.
These changes were validated on a Linux-based system, with a Unity gRPC client talking to a tonic-based Rust gRPC server. UDS is only supported on Unix-based systems, I'm not sure what happens on Windows - let's see CI results.
I tried to document everything properly and list all known caveats. Please let me know if there are questions or any changes needed.

Thank you for your contribution! It looks good, but may I add unit tests to this branch?

Thank you for your contribution! It looks good, but may I add unit tests to this branch?

Sure! Unit tests are something I left out, apologies.

@yzsolt
I have made some changes to the build, and have now released it as 1.9.0. Thanks!


random rec change, warnings, association rules refactoring

random rec changed:

random generator as a new attribute to provide recommendations diversity and reproducibility for model with fixed seed
fit for uniform model distinguished
probabilities for uniform predict are not passed to udf (speed up)
np.random.choice is replaced with default_rng().choice() (new way of sampling)
tests added

warnings for neural models and wrappers added
association rules code refactoring
dataframe non-equality check added to utils

@AleXXL1986 please have a look


[docker] add local run for validator dynamic

Now we can support running an arbitrary number of nodes locally in
docker. This helps facilitate test development rather than immediately
jumping into cluster-test, etc.
Usage:
install docker
build dynamic validator:
cd libra &&
./docker/validator/build.sh &&
./docker/validator-dynamic/build-dynamic.sh
run the dynamic validator:
./docker/validator-dynamic/run.sh

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

@ericnakagawa, this CLA signing thing seems broken.

@bors-libra r+

:pushpin: Commit 78c33f4 has been approved by bothra90

:hourglass: Testing commit 78c33f44207dbdbc8b6f67fe9c0ec8deb29e9ca0 with merge 92bef1594629424b7f817300c82f373e9551a859...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: bothra90
Pushing 92bef1594629424b7f817300c82f373e9551a859 to master...


Sampling rate of sensors not changed

When changing the REST path sampling rate e.g. /Meas/Acc/13 -> /Meas/Acc/52, the output from the device does not change. This might be a firmware related issue, that the chosen firmware does not provide the the path in that format and requires the usage of different path instead.
Needs to be checked and tested.

Due to above, mismatched sampling rates can't be tested when the data is being merged at the end of a data collection session.

It appears that higher sampling rates do not send packets at a higher rate, but instead simply create larger packages. This functionality is now fixed, and the timestamps for new instances after the first one are inferred (as they are not included in the data packets).


Remove analysis codes that are no longer used

Four analysis codes are no longer generated by Istio.  See https://github.com/istio/istio.io/issues/7844#issuecomment-726350837 for list.
Removing these dead codes will remove them from http://istio.io/latest/docs/reference/config/analysis/
cc @frankbu

In response to a cherrypick label: #28830 failed to apply on top of branch "release-1.8":
Applying: Remove analysis codes that are no longer used
.git/rebase-apply/patch:255: new blank line at EOF.
+
warning: 1 line adds whitespace errors.
Using index info to reconstruct a base tree...
M	galley/pkg/config/analysis/msg/messages.gen.go
M	galley/pkg/config/analysis/msg/messages.yaml
Falling back to patching base and 3-way merge...
Auto-merging galley/pkg/config/analysis/msg/messages.yaml
CONFLICT (content): Merge conflict in galley/pkg/config/analysis/msg/messages.yaml
Auto-merging galley/pkg/config/analysis/msg/messages.gen.go
CONFLICT (content): Merge conflict in galley/pkg/config/analysis/msg/messages.gen.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Remove analysis codes that are no longer used
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #28889


typo on bulletproofs.md

Like erciccione said, all of your commits should be squashed into a single commit with a more detailed message (e.g. "moneropedia: fixed typo in bulletproofs.md").

Ohh sorry ! Is it better now ?


Cli argument parsing seems broken

The following command leads to an error. However the order, flags and arguments match the required scheme.
npx create-near-app my-app --contract ts --frontend vanilla --tests ts

Arguments error
Run npx create-near-app without arguments, or use:
npx create-near-app <projectName> --frontend next|vanilla|none --contract rs|ts|none --tests rs|ts|none

Nice catch, addressed by #2113

Thanks. The error is clear now


[WIP] Revisit documentation

TODO mandatory (to check before merging):

[ ] add redirections for guides that have been moved to guides folder
[ ] installation page
[ ] make index / quickstart / installation pages consistent
[ ] remove all mentions of TO_BE_COMPLETED
[ ] update href links in index (the 4-cells part)
[ ] ...

TODO optional (pages to revisit or write):

[ ] Guide to download / upload a model
[ ] HfApi.upload_file vs hf_hub_download vs Repository: what should I use? When?
[ ] Difference HTTP-based / GIT-based
[ ] How to use endpoints and private models ?
[ ] (?) End to end guide for a MLOps engineer: entire ML lifecycle using just code.
[ ] Manage a repo (create, delete, add tag, add branch, rename, change_visibility,...)
[ ] Manage discussion on a repo (list discussion, reply, change status,...)
[ ] Manage a local clone (Repository object)
[ ] Download a file / a folder / all files (get file metadata, file existence)
[ ] Upload a file, upload a folder, update some files, delete some files
[ ] How to build an integration ? (push_to_hub, from_pretrained) (programmatically, mixin, cli)
[ ] Modelcards
[ ] Inference API
[ ] utils:

[ ] Make HTTP request
[ ] Handle cache
[ ] Runtime
[ ] Logging
[ ] Validators
[ ] TQDM

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Most of the task points in the description have now being handled. I'm closing this PR without merging.


No extraction sometimes

Some strings do not get extracted as expected. I cannot tell whether it's specific to svelte, and I've tried to narrow it down to the following:

{__('Settings')}

{@html __('Settings')}

<button type="button" aria-label="{__('I agree')}">{__('I agree')}</button>

{__('Other Settings')}

<button type="button">{__('I still agree')}</button>

Parser and config:
import { callExpressionExtractor } from 'gettext-extractor-svelte';
import { GettextExtractor } from 'gettext-extractor';

const options = {
	arguments: {
		text: 0
	}
};

const extractor = new GettextExtractor();

extractor
	.createJsParser()
	.addExtractor(callExpressionExtractor(['__'], options))
	.parseFilesGlob('src/**/*.svelte');

extractor.savePotFile('./src/translations/source.pot');
extractor.printStats();

Thanks in advance for looking into this!

Gonna close it, since it does not seem to be a svelte specific.

Gotta reopen, since according to the mentioned issue, my previous assumption was incorrect.

After  a couple of tests I've ended up using a stock JS extractor with some source transforming:
import { GettextExtractor, JsExtractors } from 'gettext-extractor';

let extractor = new GettextExtractor();

extractor
	.createJsParser([
		JsExtractors.callExpression(['__'], {
			arguments: {
				text: 0
			}
		})
	])
	.parseFilesGlob('src/**/*.svelte', undefined, {
		transformSource: (text) => text.replace(/{__\(\'/gm, "{@__('")
	});

extractor.savePotFile('./src/translations/source.pot');
extractor.printStats();


Fix script targets

This allows me to use these scripts in the project I'm working on.
Thank you for the fantastic and convenient crate!

Thank you for fixing my bug!

Thank you for fixing my bug!


Gg 8142 record support

Thank you for taking time to contribute this pull request :smile:
You have probably already checked the following points, but as a reminder, please ensure that:

your branch is rebased on the latest version of the master branch
all tests pass
code is formatted with IntelliJ Idea default settings (if you use another IDE, never mind, ignore this point)

You may add the reference of the issue in your commit message.
Don't hesitate to add your name to the contributors list in the README.md file.
Many thanks upfront!

It's a bit a weird fork structure now. easy-random -> Yspadadden -> BertschiAG. I didn't realize this during our call. I guess this make staying up to date with the parent a bit more complicated

Do you have it used in a Bertschi project? Example: You could readd the test in SP: https://bitbucket.org/bertschiagch/opsshipmentplanner/commits/647bb7cadb5f06f676b4d25af003ee1afc7a418c


Bug: test::version_log_snap_shot often errors occur in macos ci

Bug Report
manually cycled 30,000 times in window and ubuntu without error, not tested on macos yet

I think the problem is here: there are more than one log file, and you get wrong log file when list the files.
Maybe you could delete or rename the old log file here:
https://github.com/tonbo-io/tonbo/blob/564fb3f4e99236290fc0127a74e70eeec0f08ca5/src/version/set.rs#L200-L209

The strange thing about this problem is that it only occurs in macos. I think it is probably related to FileProvider::list, and the existence of multiple logs at the same time is within the scope of the design, because there may be a sudden downtime, choose the second latest one log (this is an overlooked point now, I will improve it now)

This problem is mainly caused by the order of the list, and it no longer appears in the new version.


feat: prompt update if available and provide links and instructions

We have auto-update mechanism implemented for Windows & macOS. For Linux there is UI prompt in Dashboard notifying about new release and link to GH Release page, so IMHO this is done ;-)


Gitattributtes not showing proper languages used in projects

Literally unusable

Thanks for letting me know, it's already fixed.


Refactor CSRF protection modules, make sure CSRF tokens can be up-to-date.

The CSRF protection modules have been broken for a while. For example, comment said: // Csrfer maps CSRF to each request. If this request is a Get request, it will generate a new token., however, it doesn't!! That's why a lot of users complain about their work lost.
Related issues:

https://github.com/go-gitea/gitea/issues/17850
https://github.com/go-gitea/gitea/issues/17282
https://github.com/go-gitea/gitea/issues/11797
https://github.com/go-gitea/gitea/issues/19126

And maybe more.
This PR does a refactoring to the CSRF related code, removes most unnecessary functions.
It introduces ParseCsrfToken to parse the generated token's issue time, and regenerate the token every minute.
And it fixes some incorrect code in old code, for example: ctx.Data["CsrfToken"] = html.EscapeString(ctx.csrf.GetToken()), the EscapeString is incorrect.
The error handling logic in Validate has been simplified, remove many duplicate code.
Totally this PR is +119 −195

I wonder if it would make sense to move this into a separate package?

What's the preferred package name and package path?

Codecov Report

Merging #19337 (850a9fd) into main (1843026) will increase coverage by 0.00%.
The diff coverage is 83.54%.

@@           Coverage Diff           @@
##             main   #19337   +/-   ##
=======================================
  Coverage   47.50%   47.51%
=======================================
  Files         944      943    -1
  Lines      131556   131492   -64
=======================================
- Hits        62497    62478   -19
+ Misses      61545    61500   -45
  Partials     7514     7514

Impacted Files
Coverage Δ

modules/web/middleware/cookie.go
68.54% <ø> (+4.15%)
:arrow_up:

routers/api/v1/api.go
76.43% <0.00%> (ø)

routers/web/auth/auth.go
26.72% <ø> (ø)

routers/web/auth/oauth.go
20.71% <ø> (ø)

modules/context/context.go
63.77% <40.00%> (-0.49%)
:arrow_down:

modules/context/csrf.go
77.67% <84.78%> (+12.85%)
:arrow_up:

modules/context/xsrf.go
92.68% <91.30%> (+8.06%)
:arrow_up:

modules/context/api.go
73.60% <100.00%> (ø)

modules/context/auth.go
20.00% <100.00%> (ø)

modules/git/repo_base_nogogit.go
70.58% <0.00%> (-3.93%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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

What's the preferred package name and package path?

modules/csrf maybe?

What's the preferred package name and package path?

modules/csrf maybe? Not sure if that's what @zeripath means.

I do not think csrf can be a proper package name. CSRF means Cross site request forgery, it's an attack vector/method. The package is protecting against CSRF attack.

It's code to handle CSRF, seems pretty clear to me. And a short name is easy to work with.

TBH, I still think the name csrf is not ideal enough.
And this PR has done what it should do. Moving code into a new package is too much for this PR. Reasons:

If we do moving code and changing code at the same time, this PR would become more difficult to be reviewed, and it may introduce potential bugs.
Moving code into a new package is not an easy work at the moment. Because modules/context.go need modules/csrf.go  and csrf also needs context. Without more related refactoring, there will be a cycle-import error, the downside of Golang. For reason 1, it's not ideal to make these more argument renaming/changing refactoring into this PR.

@zeripath @silverwind

bot


feat: solve year 2022, day 15

Closes: #31

:tada: This PR is included in version 1.15.0 :tada:
The release is available on:

v1.15.0
GitHub release

Your semantic-release bot :package::rocket:


BUG getting 403 while attempting to download docker-ce

Problem description
I am getting 403 while following the instructions https://docs.docker.com/install/linux/docker-ce/centos/ to install the docker-ce

Problem location

I saw a problem on the following URL:https://docs.docker.com/install/linux/docker-ce/centos/

I couldn't find the information I wanted. I expected to find it near the following URL:

Other:

Project version(s) affected

Suggestions for a fix

Noticed this for Ubuntu as well, looks like all of the linux directory is 403ing: https://download.docker.com/linux

+1 on trying to update.
E: Failed to fetch https://download.docker.com/linux/ubuntu/dists/bionic/InRelease  403  Forbidden [IP: <IP_ADDRESS> 443] E: The repository 'https://download.docker.com/linux/ubuntu bionic InRelease' is no longer signed. N: Updating from such a repository can't be done securely, and is therefore disabled by default. N: See apt-secure(8) manpage for repository creation and user configuration details.
Also trying to clean /etc/apt/source.list and re-add pgp key and source doesnt work.
On adding the pgp key
curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo apt-key add - curl: (22) The requested URL returned error: 403 gpg: no valid OpenPGP data found.

Problem seems solved again.
curl -fsSL https://download.docker.com/linux/ubuntu/gpg | sudo apt-key add - OK
sudo add-apt-repository "deb [arch=amd64] https://download.docker.com/linux/ubuntu bionic stable" Hit:1 http://security.ubuntu.com/ubuntu bionic-security InRelease Get:2 https://download.docker.com/linux/ubuntu bionic InRelease [64.4 kB] Hit:3 http://azure.archive.ubuntu.com/ubuntu bionic InRelease Hit:4 http://azure.archive.ubuntu.com/ubuntu bionic-updates InRelease Hit:5 http://azure.archive.ubuntu.com/ubuntu bionic-backports InRelease Get:6 https://download.docker.com/linux/ubuntu bionic/stable amd64 Packages [8880 B] Fetched 73.3 kB in 1s (113 kB/s) Reading package lists... Done
p.s. how do you create new lines in code blocks :S

Issue should now be resolved, this can be closed

This issue is back again.


Read pkcs#8 private keys encrypted using FIPS-compliant openssl

Describe the enhancement:
Currently, the tlscommon.ReadPEMFile function is able to read encrypted PKCS#8 private keys created using a non-FIPS-compliant openssl. Such keys look like this:
-----BEGIN EC PRIVATE KEY-----
Proc-Type: 4,ENCRYPTED
DEK-Info: AES-256-CBC,065B7536137462AFB30E8508CAA2EE88

BASE64 ENCODED DATA
-----END EC PRIVATE KEY-----

However, if the private key is encrypted using a FIPS-compliant openssl, the tlscommon.ReadPEMFile function is unable to parse it. Such keys look like this:
-----BEGIN ENCRYPTED PRIVATE KEY-----
BASE64 ENCODED DATA
-----END ENCRYPTED PRIVATE KEY-----

Describe a specific use case for the enhancement or feature:
To allow FIPS-compliant openssl created PKCS#8 private keys to be used with Beats.

After a conversation with @cmacknz, this issue should be reopened but we do not expect it to be in scope for our current efforts.


Acquire silent token refresh strategy does not return an access token

Please follow the issue template below. Failure to do so will result in a delay in answering your question.

Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[x]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Important: Please fill in your exact version number above, e.g<EMAIL_ADDRESS>Framework
React
Description
When targeting multiple resources, acquireSilentToken does not return an access token for a resource that was not part of the acquireTokenRedirect scope.
Scopes:
A.0 - Resource A
B.0 - Resource B
This request redirects to idp and returns an id_token
acquireTokenRedirect({
   scopes: ['openid','profile']
});

This request calls the token endpoint using refresh strategy and does not return an access token
acquireTokenSilent({
   scopes: ['A.0']
})

This request calls the token endpoint using refresh strategy and does not return an access token
acquireTokenSilent({
   scopes: ['B.0']
})

While tracing the network calls, both of the acquireTokenSilent request are appending the scopes ['openid', 'profile'] to the refresh endpoint, causing it to not fail and in effect does not proceed to request for an access token using the other strategies (authorization code, iframe)

Error Message
N/A
Security

[ ] Is this issue security related?

Regression

[ ] Did this behavior work before?
Version:

MSAL Configuration
// Provide configuration values here.
// For Azure B2C issues, please include your policies.

Reproduction steps
// Provide relevant code snippets here.
// For Azure B2C issues, please include your policies.

Expected behavior
acquireTokenSilent should not append the scopes openid and profile to the refresh token request so that it fails and proceed to the other strategies.
This however works fine in the compiled version 2.1.0 and the npm package version 2.1.0
Browsers/Environment

[ ] Chrome
[ ] Firefox
[x] Edge - Chromium
[ ] Safari
[ ] IE
[ ] Other (Please add browser name here)

This issue was addressed here https://github.com/AzureAD/microsoft-authentication-library-for-js/pull/1962/files, but i think it regressed for version 2.3.0

@judedaryl This is a known issue with B2C and is being tracked in #2315. Please see that thread for the suggested workaround until a fix is rolled out. Closing as duplicate.


Multicast multiple interface binding

When trying to use node-coap for some multicast traffic today on multiple interfaces, I found that it cannot bind to multiple interfaces for multicast. The nodejs socket.addMembership documentation was recently updated (https://github.com/nodejs/node/pull/7244/files) and I suppose it's now necessary to iterate the available interfaces and bind to each one explicitly, probably using os.networkInterfaces().

Unfortunately, I do not have time to address this issue. I'll be happy to merge any contribution in this area.

So, I've got a start on this but trying to figure out how to add tests for it now: https://github.com/dwarburt/node-coap/tree/multi-multi


[CVE-2020-7731] CWE-476: NULL Pointer Dereference?

https://ossindex.sonatype.org/vulnerability/CVE-2020-7731?component-type=golang&component-name=github.com%2Frussellhaering%2Fgosaml2
This affects all versions of package github.com/russellhaering/gosaml2. There is a crash on nil-pointer dereference caused by sending malformed XML signatures.
The document mentions https://github.com/russellhaering/gosaml2/issues/59 which was fixed with https://github.com/russellhaering/gosaml2/pull/90
Maybe https://ossindex.sonatype.org/vulnerability/CVE-2020-7731?component-type=golang&component-name=github.com%2Frussellhaering%2Fgosaml2 is not up-to-date.

You're right, but I'm not quite sure what is causing this. The Snyk report, which the CVE seems to reference, correctly notes that this was resolved in v0.7.0.
I've opened a GitHub Security Advisory reflecting the status of this - perhaps that will propagate to the CVE?
I'll keep this open for tracking.

Maybe, you could also Report advisory or correction for ossindex.sonatype.org.


run_unit_tests.py script improvements

Ensure that the user's machine has Git installed as the script requires it to clone the osirixgrpc repository
Create the required "osirixgrpc_tests" folder on the user's machine which is required to store the test outputs

Possibly a note to mention about deleting the "Test Patients" database in Osirix if they are running it again

Ability to check for output directory now added as of dev72.


Fixes 3608: switches in repo modals cause infinite loading

Summary

In the add/edit repository modals, when a user rapidly toggles the switches to enable/disable snapshots or modularity filtering, this triggers an infinite loading state of the Save button and it never becomes clickable
This change fixes that

Testing steps

Toggle the snapshot or modularity filtering switches quickly, the Save button should become clickable

https://issues.redhat.com/browse/HMS-3608


⚠ Bofrost.be has degraded performance

In 6064ea4, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1110 ms

Resolved: Bofrost.be performance has improved in 23bab4b after 12 minutes.


Fix ActiveModel#validates_with; block argument can be omitted

Without this fix, I got [error] The method cannot be called without a block with Validator class.

Well, I encountered a bug in GitHub... I've clicked the merge button then the commit has been merged into the main branch, but it didn't change the PR status to "merged".
So I'll close this PR unfortunately, sorry for that and really thanks for your pull request!


Customize Textarea (rows attribute)

Sorry to reach out so soon again.
How would I override the number of rows for a textarea? I can't find any examples of it not hardcoded to 5.
e.g.
<textarea rows="15"></textarea>

Hey, @jjanyan! The more issues you create, the more you contribute to the community! Don't sweat 😄
You can create your own component and pass it to the multilineComponent prop. You can do that generically, passing it in your project's Form component, or locally, passing this prop to the specific form you want to have a different number of rows. Here's an example of a text area component.
In case you already have a multiline component and don't want to create a component for each form you render, you can pass children to Field, access the Multiline component, and pass the rows prop to it. Something like this:
const schema = z.object({
  name: z.string().nonempty(),
  bio: z.string().nonempty(),
})

export default () => (
  <Form schema={schema}>
    {({ Field, Errors, Button }) => (
      <>
        <Field name="name" />
        <Field name="bio">
          {({ Label, Multiline, Errors }) => (
            <>
              <Label />
              <Multiline rows={12} />
              <Errors />
            </>
          )}
        </Field>
        <Errors />
        <Button />
      </>
    )}
  </Form>
)

I'm closing the issue for now but feel free to reopen it at any time!


#628 Cards: Match Welcome card design AND #638 Welcome card: Update the text

#628 Cards: Match Welcome card design + text to wireframes
Changed text

#638  Welcome card: Update the text

@LiavGut approved?
The exclamation icon, is that the current version?

The exclamation icon, is that the current version?

@StanislavShevchenko build build build

@sirpy @StanislavShevchenko
No. This is not the right icon for the info cards, in fact, I don't know where they took it from.
There are 2 system icons:

info (i)
warning (!)

And they are not just a flip version of one another...

Added new icon (info)

The new ! icon doesn't seem to be aligned in height with the avatar icon.
make sure the new icon is same size as other icons

I added icons from the design, they have the right size. There is another task for aligning the icon and avatar

@StanislavShevchenko the task for aligning the icons has been done and merged

@StanislavShevchenko the task for aligning the icons has been done and merged

This master branch, merge you task, it has a bug


Raise a compile error if self is referenced from a static method

Currently code like
type Foo
    Value x
    bar = self.x*self.x

yields no error at compile time because every method has the 'synthetic self'.
But of ourse this is most of the time an oversight - I forgot to add self and created a static method instead of a member - and it's my mistake.
So if I do:
o = Foo.Value 42
o.bar

which I wanted to work, it will fail with Unresolved_Symbol because the method is static by accident.
To prevent this, we can report a compile time error whenever self is referenced in method body but the method does not declare to be taking self.
The only issue is if the synthetic self reference ever needs to be used by user code. I'm not aware of such situations however. @hubertp do you know any obstacles here?
If not, I think reporting an error could make it easier to catch issues as essentially a self reference without a self argument is most likely just a mistake.
(As mentioned on Discord)

The only issue is if the synthetic self reference ever needs to be used by user code. I'm not aware of such situations however. @hubertp do you know any obstacles here?

Can't recall any at this point. This sounds like a desired improvement.


Adding support for ssl connections to rethinkdb in migrations

Currently rethinkdb-migrate doesn't support any SSL connections while running migrations.
This PR introduces support for it as we're using in our own setup too.

Coverage increased (+0.02%) to 98.241% when pulling 965350616d3d5a521d55528e7d13e2233bca1d17 on lirantal:fix/migrations-ssl-support into f38c4e2fd07cef1499cb2575cf510c4232b6c39f on vinicius0026:master.

@vinicius0026 ping

@lirantal I'll review it today, maybe add some tests for this use case.

@lirantal Thank you for your continued contributions for this repo. Just added you as a collaborator.
Since your PR with the option SSL is simply forwarding the option to rethinkdbdash driver, I think there is no need to additional tests. But I was wondering if we should allow other ssl options, such as cert, key and passphrase, and also if we should make possible to set ssl option simply to true, as in rethinkdbdash example.

@vinicius0026 I'm honored, thanks! Would love to continue the good work with you on this.
On SSL - both rethinkdb and rethinkdbdash allow ssl: true and indeed rethinkdb declares support for other tls options. Since we are anyway just passing the ssl config around and it can be either a primitive one or a reference object then all that is left to do to support both cases is update the Joi validation. What do you think?

Agreed. Something like:
ssl: [Joi.object(), Joi.boolean()],

Pushed the suggested change, thanks.
Also updated the README with an example for SSL that would work well for most setups. Maybe we should also have a sub-section to explain the possible options there.

Coverage increased (+0.02%) to 98.241% when pulling 924c4c3d0d0a8f04aca4411747906872bc7ac80c on lirantal:fix/migrations-ssl-support into f38c4e2fd07cef1499cb2575cf510c4232b6c39f on vinicius0026:master.

Coverage increased (+0.02%) to 98.241% when pulling 924c4c3d0d0a8f04aca4411747906872bc7ac80c on lirantal:fix/migrations-ssl-support into f38c4e2fd07cef1499cb2575cf510c4232b6c39f on vinicius0026:master.

Coverage increased (+0.004%) to 98.228% when pulling 994b6c2672f2c399485c93c352af255dab4b4337 on lirantal:fix/migrations-ssl-support into f38c4e2fd07cef1499cb2575cf510c4232b6c39f on vinicius0026:master.

@lirantal after merging I noticed that ssl option is not being passed to rethinkdb driver (this should be done here). I'll do this and update the docs, if you have no concerns.

Of course, good catch.


Add a Preloader

Add a well-designed Preloader for this gaming website project.

Want to give it a try. Please assign it to me for jwoc

Sure, Go ahead.


this cannot be used in macros

(alternative title: this should be hygiene-bending, not breaking)
macro "foo" : term => `(have := 1; this)
#check foo  -- unknown identifier 'this✝'

@leodemoura I believe we had a proposal once that "anaphoric" identifiers like this should be keywords instead, with an elaborator that unhygienically accesses the scope-less this assumption. I don't remember what became of that proposal though :) .

@Kha Yes, this is a problem, and you are correct that we have discussed it before. Unfortunately, I don't recall the details :(
I will try to record these discussions/solutions as GitHub issues in the future.

I believe we had a proposal once that "anaphoric" identifiers like this should be keywords instead, with an elaborator that unhygienically accesses the scope-less this assumption.

Unfortunately, this breaks subst this and other uses of this as an identifier... https://github.com/Kha/lean4/commit/1a80beaf59e1f1b3f08da09ea4b6442c709674d6

Unfortunately, this breaks subst this and other uses of this as an identifier...

For subst it would make sense to extend it to terms so that you can write stuff like subst show x = 0 from Subsingleton.elim _ _ (like cases).  Which would still leave clear and revert though.  On second thought, clear ‹Nat› could also be useful.
Another idea is to change name resolution specifically for this:
import Lean

open Lean Elab Term in
@[termElab ident]
def elabThis : TermElab | stx, _ => do
  unless stx.getId.eraseMacroScopes == `this do
    throwUnsupportedSyntax
  for decl in (← getLCtx).decls.toArray.reverse do
    if let some decl := decl then
      if decl.userName == `this then
        return decl.toExpr
  throwError "this not found"

macro "foo" : term => `(have := 1; have := 3; this)
#eval foo

Mmh yes, widening idents to terms even if the tactic elaborator then asserts that the elaboration results must be a single fvar seems generally useful. @leodemoura What do you think?

Another idea is to change name resolution specifically for this:

I'd like to avoid this one :) . Especially since it doesn't solve the ident tactic issue unless those also add special cases. Apart from this specific issue, I really like the idea of signalling anaphoric identifiers' special behavior by making them... not identifiers but keywords.

Mmh yes, widening idents to terms even if the tactic elaborator then asserts that the elaboration results must be a single fvar seems generally useful. @leodemoura What do you think?

It looks reasonable to me.


cannot adapt margin›

How to decrease the gap in between? I tried different paddings but not working.


The trick to not have any distorsion when pics loading is to use a default video thumbnail with the same borders.
Do you know how to get rid of the borders on the thumbnails ? Does it comes from Picasso ? The size ?
Thanks in Advance.

I fixed it using another library using Picasso API.
Check out : Picasso Transformations
The aim is to crop the image to get rid of the black borders, using CropTransformation method.
Example of my use:
CropTransformation(the same width : , the new height : 170, GravityHorizontal.CENTER, GravityHorizontal.CENTER);
GravityHorizontal are used to define the point where the crop will be executed.
The height of the border is 35px each, so 70px in total to remove to your initial height to get the new one.


issues with installing causal-convd1 and mamba-ssm

Thank you for your hard work. I'm really interested in your model.
I hope this issue finds you well.
I'm currently struggling with installing causal-convd1 and mamba-SSM; the attached photos are respectively.
My nvidia-smi and nvcc-V are matching with cuda 11.7 but still have problem installing them.

Are these two mandatory to run Swin-UMamba model? If so, please let me know if you have any solution for this issue.
Thank you.

Hi @tash46. We use ubuntu 22.04 LTS + nvidia-smi==535.161.07 + cuda 12.2 on our server. I found an similar issue from mamba. Hope it helps.

Are those answers from the issue helpful?

@tash46 You can try it. We have never encountered this issue before.


What does the uninstall target do / how can we not create it

Hi,
I asked the same question on stack overflow but apparently nobody there knows.
What does the uninstall target do that is generated in Visual Studio when using this with CMAKE?
How do we disable it?
It is not mentioned in the documentation, nor any of the code within the "install" target CMakelists itself.
If it must exist can it be automatically added inside the GLFW3 folder instead of outside it?

What does the uninstall target do that is generated in Visual Studio when using this with CMAKE?

It uninstalls the GLFW library, headers and package files from where the install target placed them (or will place them).

How do we disable it?

It's disabled along with the install target. Use the GLFW_INSTALL CMake option to disable them.

It is not mentioned in the documentation, nor any of the code within the "install" target CMakelists itself.

True, and it really should be included. There isn't really a section on installation at all and the rest of the compilation guide isn't all that great either.

If it must exist can it be automatically added inside the GLFW3 folder instead of outside it?

Absolutely! Fixed with aef4edadd0b580fd55a93299ac171caade586fe8.

Awesome answer. Great work.


feat: re-add Chinese translations

@roj1512 what can we do about the memory consumption? This did not happen back in the days, did it?

@KnorpelSenf It’s my first time seeing it. Maybe we can do something to stop pending builds when there’s a new commit pushed. I think they do something like that in Deno’s https://github.com/denoland/deno repository.

https://github.com/denoland/deno/blob/211f49619afd915a2ce56cf0b5da50f640c90f73/.github/workflows/ci.yml#L18

Huh it seems not related

Let's just throw more memory at the problem.

I have listed the pull requests that have the label "ready for translation" from commit 48aa7020 (the first commit since ZH was removed) to date. Hope this helps.
To do list:

[ ] #530
[ ] #502
[ ] #513
[ ] #506
[ ] #518
[ ] #536
[ ] #545
[ ] #442
[ ] #505
[ ] #511
[ ] #558
[ ] #560
[ ] #561
[ ] #564
[ ] #563
[ ] #557
[ ] #520
[ ] #497
[ ] #568
[ ] #572
[ ] #576
[ ] #556
[ ] #579
[ ] #578
[x] #594
[ ] #589
[x] #597
[x] #626
[x] #559
[x] #587
[x] #585
[ ] #642
[ ] #641
[ ] #590
[ ] #625


[Browser Bookmarks] It often crashes when I search for a bookmark.

Extension
https://www.raycast.com/raycast/browser-bookmarks
Description
Error:
Error: Worker terminated due to reaching memory limit: JS heap out of memory

NodeError:errors:406:5
[kOnExit]:worker:313:26
Worker.<computed>.onexit:worker:229:20

Steps To Reproduce
It often crashes when I search for a bookmark.
Current Behaviour
Something went wrong.

Expected Behaviour
It should search for bookmarks normally every time and be able to open bookmarks by hitting enter in the default browser.

Thanks for bringing this up. It appears that this is a duplicate of issue #6810, so I'll mark it as such.


Have an explicit Solver::name function

I find this cleaner than requiring Display and it allows the trait to
be automockable.
Test Plan
still compiles

Just an internal change. Now you do
println!("{}", solver.name());

Instead of
println!("{}", solver);

The reason I prefer it like this is that:

I find the solver.name() code easier to read: "print the solver name" is less ambiguous than "print the solver".
I think Solver: Display was abusing the Display trait a bit.


Error While Scrolling and Region Creation

Ive been having an issue when zoomed in and trying to scroll by grabbing the scrollbar. Each time I try to scroll it creates a new region. I want it to only create a region when clicking in the canvas area, not when trying to scroll.
I have found within the region enableDragSelection there is an eventDown click event is firing for both clicking in the waveform area and clicking on the scrollbar. I then determined when clicking on the scrollbar the tag type is Wave and when clicking within the waveform area the tag type is Canvas.
So I created a method within enableDragSelection called isCanvas
`isCanvas: function (evt) {
    if ( evt.target.tagName == 'CANVAS' ) {
        return true;
    }
    return false;
}`

I then added in an if to check when its NOT the canvas just return out of the eventDown method
`if (!my.wavesurfer.Region.isCanvas(e)) { return; }`

Is there a better way to handle this oddity? Or is it a wanted feature?

maybe related to https://github.com/katspaugh/wavesurfer.js/issues/843 ?

@mspae you are right it is exactly that. I will read that thread over and close this when I find some time over the weekend.

@mspae the issue is related but has no solution and is worthless to me, would have been better to let the other issue know of my solution.

Unfortunately this doesn't solve the issue. When clicking ahead of the cursor, the event tagname is "CANVAS", but when clicking behind the cursor, the tagname is "WAVE". However, the two wave nodes (the actual waveform and the scrollbar) are slightly different. So I can use these differences to maybe pinpoint when the user clicks the scrollbar and prevent it from doing anything.

Ok cool, I can check e.target.childElementCount and if it's greater than zero, ignore the eventDown. Will submit a PR shortly, after testing.


feat: package vulnerabilities sourcing

This PR adds sourcing of verified npm package vulnerabilities from github advisory db

I was wondering what happens if the package source code is not hosted on github (for example, on gitlab or bitbucket instead)? Will we be able to pull vulnerabilities for such packages?

I was wondering what happens if the package source code is not hosted on github (for example, on gitlab or bitbucket instead)? Will we be able to pull vulnerabilities for such packages?

Yes, github advisory db doesn't exclude packages not hosted on github itself.


Add reference to ducks modular redux proposal to README.md

closes #15

thank you! @tanertopal :) glad you're back


Kraken getKrakenWebsocketToken Permission denied

Hello, while setting up the KrakenExchange with apikey and secret and trying to get a kraken websocket token via the KrakenAccountService, I get permission denied. I have already tried manually with a rest request with the same api key and secret to get a websocket token and that succeeds, so I do not know what the problem is with this function.

For Kraken, you to also add your username to the exchangeSpecification.
exSpec.setUserName("abc");
Some exchanges also require a  passphrase (but not kraken)
exSpec.setExchangeSpecificParametersItem("passphrase", "****");


Make getProperties method public in BaseEntity

Addresses issue: https://github.com/googleapis/google-cloud-java/issues/5451

I signed it!

@igorbernstein2 Hi, I'm struggling to see the build failure here for formatting, mind pointing me in the right direction?

Please run
mvn com.coveo:fmt-maven-plugin:format

Thanks @igorbernstein2, formatted and updated

Hi @sduskis, seems like all checks have passed, is there someone in specific I request a review from?


Doesn't work with node > v10.0.0

Using node v10.2.1 and get the following error:
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=8.0.0 <10.0.0".

Fixed.


Can't create handler inside thread that has not called Looper.prepare()

网络请求时遇到：
Can't create handler inside thread that has not called Looper.prepare()

Note: DataLoader runs within a threadpool


how to manual add the vcpkg include/lib path to visual studio

I know vcpkg integrate install, but I do not like it.
I want to add the include path and lib path manually, for example, through Microsoft.Cpp.Win32.user.
What makes me confused is how can I set for x86-windows and x86-windows-static, and debug and release? All these projects settings use Microsoft.Cpp.Win32.user. I think maybe we should have Microsoft.Cpp.Win32.Debug.user and Microsoft.Cpp.Win32.Static.Debug.user?

Here is our documentation on the raw structure: https://github.com/Microsoft/vcpkg/blob/master/docs/users/integration.md#manual-compiler-settings.
Otherwise, this looks like an issue with Visual Studio and unrelated to vcpkg. Please post this over on the Visual Studio User Voice[1] so the right people will hear about it!
[1] https://visualstudio.uservoice.com/forums/121579-visual-studio-ide


Better error handling

Now is little bit tricky to eg. log errors in zf-console.
I like the idea of new error handler in stratigility. Especially error response (message) generation, triggering listeners (useful for logging) etc.
Do you have any idea how this implement in zf-console? Your opinions?

This would be a great addition for this project. I've been thinking of a similar feature for a while now - a ConsoleView.
My goal is, that the object covers verbosity, colorful output and (Error)Output as well.
Error Handling and Message Output
Your idea to wrap a  Error handler around a Dispatcher is great and could be done here.
Verbosity
To provide a configuration of the verbosity by the developer it could be injected as parameter in the constructor of the View class. I think the level could be re-used from the PSR-3.
$view = new View($writer, View::INFO);

// Somewhere in a Command
public function __invoke(Route $route, Console $console) {
    $this->view->write(View::DEBUG, 'Start processing Foo Action');

    try {
        // ... Do Some Stuff
    } catch (InvalidArgumentException $exception) {
        $this->view->write(View::ERROR , $exception);
    }
    $this->view->write(View::DEBUG, 'End processing Foo Action');
}

As I am thinking about it, the View could accept the Psr\Log\LoggerInterface as a writer to allow an injection of all PSR-3 implementations.
A major advantage are the already available implementations to write the messages that we can re-use.
But it would be a conflict with the next feature.
Colorful Output
It is really annoying to read Log-Files that have symbols like this (\e[42m10/\e[41m1). The important information aren't visible anymore because they are hidden in the control characters for the console. On contrary the colors could be really helpful to highlight important information on the console. So the color should be passed into the writer and it's decide itself if the color is used or discarded.
I propose these interface to allow the features:
namespace ZF\Console\View\Writer;

interface WriterInterface
{
    /**
     * @param int $priority Priority of the Message
     * @param string|\Exception|\Throwable $message Message that should be logged
     * @param int|null $color Color code to control the color of the message
     * @param int|null $backgroundColor Color code to control the background color of the message
     */
    public function write($priority, $message, $color = null, $backgroundColor = null);
}

The View Class could look like this:
namespace ZF\Console\View;

class View {

   /**
    * @param WriterInterface $writer Target to write the message
    * @param int $priority Messages with a smaller priority will be discarded
    */
    public function __constuctor(WriterInterface $writer, $priority)
    {

    }

    /**
     * @param int $priority Priority of the Message
     * @param string|\Exception|\Throwable $message Message that should be logged
     * @param int|null $color Color code to control the color of the message
     * @param int|null $backgroundColor Color code to control the background color of the message
     */
    public function write($priority, $message, $color = null, $backgroundColor = null)
    {
        if ($this->isPriorityMatched($priority)) {
            $this->writer->write($priority, $message, $color, $backgroundColor);
        }
    }
}

A ServiceFactory for the View could either automatically detect if the current environment supports a TTY or it is defined by the developer in a local Configuration file.
My idea currently does not cover the following things

Additional Information to a message, like in the Zend\Log Repository (I think this would be helpful)
Coloring Parts of a Message (A workaround is to use multiple write calls)

@vaclavvanik What do you think about this changes? Is there a chance to cover these features in a View Class?


use the gtp port configured in s11 config file

Current code does not use the port configured in config file

Need similar changes in paging branch as well.

Need similar changes in paging branch as well.

Need similar changes in paging branch as well.
also pushed to paging branch.

retest this please

tested with ng40+staging pod.