Artyom with express

Hi, I'm trying to use artyom in a express nods app. I added with the following command (inside the project folders)
npm install artyom.js
And I see it in node_modules. In app.js I add the following lines:
var artyom = require('artyom.js/build/artyom.js');
const jarvis = new Artyom();

but when I start the app I got the following error:
const jarvis = new Artyom();
ReferenceError: Artyom is not defined

Any idea why? Thanks
4b

Have you tried just require("artyom.js") ? It should work out of the box requiring only the name of the module.

of course but no chance, doesn't work with only artyom.js, is for that I have tried with full path.
Others module works out of the box.
Any others idea?

Ah, the error is not the module, check the uppercase:
var Artyom = require('artyom.js');

const jarvis = new Artyom();

yes, I found this typo error me too. Thanks
but it doesn't work anyway.... I got
const jarvis = new Artyom();
               ^

TypeError: Artyom is not a constructor

Has anyone solved this?

Are you running this in the browser?

Hello, I have the same issue :/
I want to use Artyom in a bot for Discord (made with node.js)
link imgur
const Artyom = require("artyom.js"); const artyom = new Artyom();

const artyom = new Artyom();
TypeError: Artyom is not a constructor

Does someone know what it is or what to do ?
Thanks

I ran into this issue recently in working with this after doing npm install on nodejs, and found that changing:
exports.default = Artyom;
to:
exports.Artyom = Artyom;
At the end of build/artyom.js allowed me to do the following:
var Artyom = require('./node_modules/artyom.js/build/artyom.js').Artyom; this.voice = new Artyom();
this being a class that assigns the new Artyom object to its voice member. I'm able to start it up, and now I'm just going through trying to get it to recognize any voice inputs (which may be a different issue than something inside Artyom, since I'm experimenting with node inside Electron right now).
If that issue has to do with Artyom, I'll update..

@DigitalFlux
I ran into this issue recently in working with this after doing npm install on nodejs, and found that changing:
exports.default = Artyom;
to:
exports.Artyom = Artyom;
At the end of build/artyom.js allowed me to do the following:
var Artyom = require('./node_modules/artyom.js/build/artyom.js').Artyom; this.voice = new Artyom();

this being a class that assigns the new Artyom object to its voice member. I'm able to start it up, and now I'm just going through trying to get it to recognize any voice inputs (which may be a different issue than something inside Artyom, since I'm experimenting with node inside Electron right now).

If that issue has to do with Artyom, I'll update..

In case I just managed to make it work because of your comment above to adjust the export unfortunately it is not working in the current version. I was able to make parts work on electron but I need to solve the window problem. I'll put the solution here if I can.

Same here. I also tried the solution mentioned above about changing default export.
following is the error i get :
`(base) LT6564:node-server-tutorial amreen.rajmohmad$ node server.js
/Users/amreen.rajmohmad/Documents/node-server-tutorial/node_modules/artyom.js/build/artyom.js:59
if (window.hasOwnProperty('speechSynthesis')) {
^
ReferenceError: window is not defined
at new Artyom (/Users/amreen.rajmohmad/Documents/node-server-tutorial/node_modules/artyom.js/build/artyom.js:59:13)
at Object. (/Users/amreen.rajmohmad/Documents/node-server-tutorial/server.js:11:16)
at Module._compile (module.js:577:32)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
at Module.runMain (module.js:611:10)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:160:9)`

Okay, So how i resolved this is :
const Artyom = require('artyom.js').default;
since I was facing a windows error as I mentioned in my previous comment
I saw there was already a PR addressing it.

I replaced my Artyom.js file with these updated changes.
Then to make sure that node_modules use this changes : run following command from your package. it will create patches folder.
npx patch-package <package name>
and add the following flag in your package.json so that on npm install this patch is applied again.
"postinstall": "npx patch-package"


fails while making the website responsive check

First of all thanks for this amazing plugin. It differentiates the mobile view and desktop view very efficiently but when we check using Chrome’s simulation with google chrome and made responsive(custom pixel)such as 1250 * 800, It is showing mobile view whereas it is actually a desktop view. Advance thanks for the feedback. @barnabaskecskes

@austindudley10 this package uses the User-Agent header to determine whether the client is a desktop/tablet/mobile device. When switching to responsive mode in Chrome it sends the original browser headers hence you see the desktop view. If you were to switch the option on the right hand side to imitate a mobile device, the header changes accordingly and you should be able to test your site on any screen size of your choice.

I tried what you have told me but i got the negative result. It should show "desktop view".

These are my code :
@mobile
Mobile View
@elsemobile
Desktop View
@endmobile

Ohhh my bad, sorry 😅- as I'm re-reading I see I mixed it up thinking you were after a mobile view and had desktop. OK, so then here is what you can do:
Switch that option back to 'Online', then go to the far right of that option toolbar and click the 'More options' represented by three vertical dots. Hit 'Add device type' and notice the extra dropdown in the toolbar. Select 'Desktop' and you're done. 😌

Okay! Thank you :)


add local-only login to existing login test

Right now we skip the login test entirely if SKIP_SSO is set in the environment. Let's add a local users only login test for the case when we have no SSO options

this work has been deprioritized.


Flying Scripts delay loading for recaptcha

I've implemented Flying Scripts and set it up to delay the following:

ctf-scripts (custom twitter feed)
recaptcha__en (recaptcha)
api.js (related to recaptcha)

Before:

After:

Haven't tested the forms, but presumed that will happen as part of https://github.com/lfph/lfph.io/issues/14

sfmc-forms.js requires recaptcha, but is bundled in to global.js, so delaying recaptcha can cause JS errors.
If we stop bundling sfmc-forms.js and enqueue as its own file, this would allow us to delay its loading as well as recaptcha. This would allow us to retain the speed improvements of delaying recaptcha.

The forms seem to work fine currently so maybe we don't need to do anything further.

I just tested the contact form now and it doesn't work so I've deactivated this plugin for now.

deployed

this has been done to cncfci

I re-tested the performance with and without Flying Scripts:
Homepage wo Flying Scripts

Homepage with Flying Scripts

This agrees more with improvements I'm getting on the events site.  @thetwopct how did you originally generate those performance numbers above?

I originated them via web.dev/measure - so independent of connection speed and browser extension issues.
I can only assume your before and after testing is inaccurate. Did you clear server cache?

With Flying Scripts

Without Flying Scripts
(Deactivate plugin, clear Pantheon cache, visit page in browser to rebuild any hit cache)

Here's what I got
Homepage wo FS

Homepage w FS


Report is not generated (experiment 2)

The second run of the experiment, with execution-depth=0, threw an exception for the following commits because of which the report was not generated.

Commit
Reason/Resolution

0a8a5d1d5fc3a6d0681e7344ddd773acdea27dda
https://github.com/GumTreeDiff/gumtree/issues/285

3ae853a6464baa3dd498a43e2af9638f8ba09d86
https://github.com/SpoonLabs/gumtree-spoon-ast-diff/pull/249

5db7253811426931ad8b303027542106bbe756e5
Terminated before the process was finished. Threw memory exception when ran on local without time limits.

6043bd63b93ee4565b6ef3efe0179fbb2351b9c3
Input files not generated. Multi method change so GumTree could not find method.  https://github.com/GumTreeDiff/gumtree/issues/285

737913664e57dad4768ff538a81ace8e0f0db559
https://github.com/SpoonLabs/gumtree-spoon-ast-diff/pull/249

a16399f50f84d3d82d6bb0306996961b2a82bb06
Report is generated. Path: /mnt/hdd2/khesoem/exec-diff0/output/sahab-reports/a16399f50f84d3d82d6bb0306996961b2a82bb06/left.json, /mnt/hdd2/khesoem/exec-diff0/output/sahab-reports/a16399f50f84d3d82d6bb0306996961b2a82bb06/right.json. Infinity is not being parsed by state diff computer. https://github.com/khaes-kth/PatchExplainer/issues/3

d5f817b6b3e6f63e542787f741b4ec8cb62a7662
https://github.com/GumTreeDiff/gumtree/issues/285

e707bbcc114bb0afa096454f9a57ff08323a5de8
Terminated before the process was finished. Takes around 5 hours.

sahab-left.json does not exist

Commit
Reason/Resolution

5b45b78464d5888ba60a3325f5ccc12876582cca
Input files not generated. GumTree could not find method.

21dba3d0e53f1b2d8452aca3f760d418ff470cdc
Input files not generated. GumTree could not find method.

6bf70cd6214303b773d2091b60e93ed0a52caea1
Input files not generated. GumTree could not find method.

fbf87fd5c93e46b28c3987e16b05f1d1da0b021d
Input files not generated. GumTree could not find method.

sahab-right.json does not exist

Commit
Reason/Resolution

5f99f45853b5c98bff5dbcbfeac837aa2c686d8c
Terminated before process finished.

a8ad6b9853609e86cbf9534fff05377dfdf7790a
Terminated before process finished.

6e9e91677774089db6e6b9de2945716269216327
Terminated before process finished.

047ef2fcc23f713dc2e682d8b4f2ca5f994bb7ca
Terminated before process finished.

37757413c6d69946ea68c27c12ac0f7fd38b535c
Terminated before process finished.

Entire commit folder does not exist

Commit
Reason/Resolution

af19a3800c20db42db18bc3ac51ce90236ca9517
Math-105 is not a maven project

885bbd7d0f6db843627986bf97d3f0db4cfddb02
Math-105 is not a maven project

Report exists but there are no states in left

Commit
Reason/Resolution

30709758f0b33338fa697b400112fac836e4cef1
No breakpoints or methods were encountered. @khaes-kth DIFF HTML is not generated. Sahab is reporting the states correctly. Left threw an NPE.

98dc9eef38a8a3353d110786ffed93ec83d1cc02
No breakpoints or methods were encountered.

fcadcfa2001ad4a86718e6a98856b332d37a4825
No breakpoints or methods were encountered.

36ceb2ae32c903057a1f574e1e8963c3f26a5a2f
No breakpoints or methods were encountered.

3177794e4cd6e9d09813d5e0c750c0cac5bc3da7
No breakpoints or methods were encountered.

Report exists but there are no states in right

Commit
Reason/Resolution

98dc9eef38a8a3353d110786ffed93ec83d1cc02
No breakpoints or methods were encountered.

fcadcfa2001ad4a86718e6a98856b332d37a4825
No breakpoints or methods were encountered.

92430cd021f1afc55f40d99090893eddd620ed9b
No breakpoints or methods were encountered.

5f57bcfae6e6bebc98de9d9b49d01e6240b63554
No breakpoints or methods were encountered.

1482e9e4fba71651f86299f5abd0688af176e3db
No breakpoints or methods were encountered.

APPROPRIATE COMMITS: sahab doesnt find target

https://github.com/google/gson/commit/b1fb9ca9a1bea5440bc6a5b506ccf67236b08243: MatchedLineFinder fails to find the class file. Reference: https://github.com/khaes-kth/drr-execdiff/blob/master/dspot-sdiffs/notes.txt#L100.
https://github.com/google/gson/commit/74e37110e5edf38d1c9ec1b46f42670340a0c6c7: extras is not compiled. Reference: https://github.com/khaes-kth/drr-execdiff/blob/d3ca7200d8460db15c3dd2deb4f8554809eeda87/dspot-sdiffs/notes.txt#L101
https://github.com/google/gson/commit/ada597e69a78b8b99f79f79493cbf51e16045202: works. Left version is not generated because tests do not exist for the first left. Reference: https://github.com/khaes-kth/drr-execdiff/blob/d3ca7200d8460db15c3dd2deb4f8554809eeda87/dspot-sdiffs/notes.txt#L102
https://github.com/google/gson/commit/9a2421997e83ec803c88ea370a2d102052699d3b: works. Left version is not generated because tests do not exist for the first left. Reference: https://github.com/khaes-kth/drr-execdiff/blob/d3ca7200d8460db15c3dd2deb4f8554809eeda87/dspot-sdiffs/notes.txt#L103
https://github.com/google/gson/commit/9e6f2bab20257b6823a5b753739f047d79e9dcbd: works. Left version is not generated because tests do not exist for the first left. Reference: https://github.com/khaes-kth/drr-execdiff/blob/d3ca7200d8460db15c3dd2deb4f8554809eeda87/dspot-sdiffs/notes.txt#L104
https://github.com/google/gson/commit/44cad04a639b3a53f4205f93acda37b3f7d10871: works. Left version is not generated because tests do not exist for the first left. Reference: https://github.com/khaes-kth/drr-execdiff/blob/d3ca7200d8460db15c3dd2deb4f8554809eeda87/dspot-sdiffs/notes.txt#L105

Thanks!

@khaes-kth I have revised the MatchedLineFinder according to what we discussed. All the tests are passing, but I am sure I need to revise this report again. I will open another issue to see if the failed patches pass now.


Allow custom storage slots

A function like this must be added to each storage library:
function layout(bytes32 slot) internal pure returns (Layout storage l) {
    assembly {
        l.slot := slot
    }
}

If we do it like this, we can maintain full test coverage without writing any new tests:
function layout() internal pure returns (Layout storage l) {
    l = layout(STORAGE_SLOT);
}

function layout(bytes32 slot) internal pure returns (Layout storage l) {
    assembly {
        l.slot := slot
    }
}

After some discussion, this seems less useful than I originally thought.  For it to work, each contract would have to store a reference to the STORAGE_SLOT defined in each library, and access storage via the layout(bytes32) function.  The storage slot reference would have to be overridable.
This could still be a useful change, but needs more thought.

Use cases for custom slots:

End users maintaining legacy code can override the SS storage slot if SS ever changes its storage slot seed format (solidstate.contracts.storage.xxx).
Some storage layouts (and contracts) can be reused for different purposes in the same deployment - for example, it may be useful to treat some internal value as an ERC20 and use the ERC20 storage layout for this purpose.  This would be most applicable to diamond proxies.

Would only work if storage libraries were merged into internal contracts, which is not desirable.

I'm once again considering implementing this.  If Solidstate introduces a breaking change to support EIP-7201 (#166), we will want to provide a way for existing users to continue using the legacy storage slots.


:zap: Speed up tuple_element_t compilation with __type_pack_element

This speeds up the CIB compilation benchmark by 10-15% on my laptop.


Handle Ballerina reserved literal name when generating code from proto(1.1.x)

Purpose
Fix the issue when generating the code from the proto definition which has ballerina reserved literal name as the field names.
Fixes #20837
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[x] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

There is no java API to retrieve Ballerina reserved literal names. So I hardcoded all words for now. Since those are subjected to change, this is not the ideal solution. I created issue #20856, requesting an API.

Codecov Report

Merging #20854 into ballerina-1.1.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.1.x   #20854   +/-   ##
================================================
  Coverage            15.21%   15.21%
================================================
  Files                   50       50
  Lines                 1341     1341
  Branches               208      208
================================================
  Hits                   204      204
  Misses                1121     1121
  Partials                16       16

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 c4dd756...cda7e17. Read the comment docs.


Unable to navigate streetview with keyboard arrows

Description
A regression in RapiD v2 was spotted where once in Map Data > Photo Overlays with streetview toggled on e.g. Mapillary, Bing Streetside, we are unable to use the arrow keys from the keyboard to proceed in any direction i.e. forward/backward -- this functionality was previously available in v1. Pressing the arrow key navigates the main page rather than the pop-up streetview window. Note that the buttons within the window works fine, such as the zoom -/+, auto play, etc.

Version
2.0-alpha
What browser are you seeing the problem on? What version are you running?
Edge v104.0
Steps to reproduce
No response
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid-v2-alpha#background=Maxar-Premium&datasets=fbRoads,msBuildings&disable_features=boundaries&map=20.24/37.76257/-122.48350&photo=mapillary/157412386316434&photo_overlay=mapillary

Thanks for the bug report!  Possibly a duplicate of #504, we'll have to take a closer look once we're back from FOSS4G.


TODO before version 1.0

There are 3 key aspects for a Cardano offchain library in Elm.

Wallet connection with CIP30 and CIP45
Building, submitting, watching transactions
A simulation environment to help end-to-end testing of aiken contracts

Wallet connection with CIP30 and CIP45

[ ] Connect to a CIP30 enabled wallet
[ ] Connect to a CIP45 enabled wallet (eternl)

Building, submitting, watching transactions

[ ] Define the types and CBOR codecs to build a transaction: #1
[ ] Define an API to create transactions (inputs, outputs, destinations, validators, utxo selection strategy, ...)
[ ] Being able to submit a transaction with ease (local node, remote node, blockfrost, txpipe, ...)
[ ] Being able to request information easily (wallet utxos, watch submission status, mempool, ...)

Simulation environment for end-to-end testing

[ ] Check phase-2 by being able to run a transaction in a sandbox, via aiken simulation
[ ] Check phase-1 by providing context
[ ] Ability to replay with time-travel debugging

Goal-oriented roadmap:

Wallets interactions

CIP30 support
CIP45 support
Explore improvements for Dapp-Wallet interactions

per-open-dapp personas
per-open-dapp account
per-open-dapp multi-account

Community Tooling

Transaction Explorer web app

Educative (links to relevant sources, ...)
Simple (comprehensible tx overview, ...)
Powerful / Complete (inspect all parts, convert formats, ...)

Interactive transaction Builder web app

Suite of common Tx types (send to, mint, multi-sig, ...)
Steps-by-steps guided interactive builder for custom Tx

Dapp examples

Equivalent to Radix Gumball Club https://gumball-club.radixdlt.com/


Revert "Add image field to Hobby and Resource API endpoints"

Reverts dotkom/onlineweb4#2078
This guy was not ready for prime time.
Decided to test in dev, did not go as planned.

Remember to unapply the migrations also so it won't conflict the next time
you try to roll it out.
On Thu, Nov 1, 2018, 21:55 Ole Anders Stokker<EMAIL_ADDRESS>wrote:

Merged #2166 https://github.com/dotkom/onlineweb4/pull/2166 into
develop.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dotkom/onlineweb4/pull/2166#event-1941567019, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFK96_12YsPbAIFWWe_tf5uue8aNslqOks5uq1_dgaJpZM4YJAca
.

Will try to keep it in mind


Wrong formula for the pseudo-inverse in 17.1.1

In section 17.1.1 (Smoothness prior), the matrix L has full row rank, but the pseudo-inverse wrongly uses the formula when L has full column rank. Precisely, (L^T L)^{-1} is not invertible if L is not full rank (which is the case when D-2 < D).
Is this an error, or I'm missing something?

Thanks. I've decided to move this to vol 2, where I will fix this and explain it in more detail.


[Perf] Add Relationship Index Storing Caveat

I have read the CLA Document and I hereby sign the CLA

recheck

I have read the CLA Document and I hereby sign the CLA

I have read the CLA Document and I hereby sign the CLA

recheck

I believe this PR was opened to address https://github.com/authzed/spicedb/issues/1923

I believe this PR was opened to address https://github.com/authzed/spicedb/issues/1923

Yes, you're right, thanks. I forgot to add the description.

I have concerns about this PR. The reason the issue was created in the first place is because the CRDB UI shows it as recommended. This is likely so that the query becomes served from the index, instead of having to go to disk to fetch the rest of the columns.
I have three concerns about the proposal:

it drops the previous indexes. This means that until the new indexes are created, the system is left without an index. This has the potential to cause an incident on a production system, as it can push the database over saturation level.

further increases the cost on the write path, as the system now has to write more information to disk, potentially making writes slower.

increases the amount of storage needed. While it is reasonable to trade disk for query speed, customers running CRDB Dedicated can see their bill explode, as Cockroach Labs puts a hefty price tag on the disk capacity of their managed offerings.

I don't think we can move forward with this without:

running a load test to assess how much queries improve and if it's worth the tradeoff

determining the % increase in disk used

finding a way to add those indexes without causing an incident (the easiest solution being to consider running 2 migrations, or exploring ALTER the indexes)

I would try to work on the stress test to test the two metrics you mentioned.
For the index drop and create issue, I have searched on that, (currently) it seems it is impossible to alter the standard index to the storing column index.
Moreover, maybe require to contribute this feature to cockroachDB upstream.
But, For the current workaround, I think we can create the storing column index first, after finishing indexing, then it would be safe to drop the original index.

But, For the current workaround, I think we can create the storing column index first, after finishing indexing, then it would be safe to drop the original index.

I'm not entirely sure this would work either. There are some caveats:

The docs indicate that index creation and drop are executed concurrently: see https://www.cockroachlabs.com/docs/stable/create-index and https://www.cockroachlabs.com/docs/stable/drop-index. Thus there is a risk the old indexes are dropped and the new indexes are not ready yet.
on top of that, there are some gotchas with multi-line DDL, see https://www.cockroachlabs.com/docs/stable/online-schema-changes#known-limitations, which could lead partially run migrations.

This is unfortunately a difficult migration to run without some careful planning. E.g. we have customers with Terabytes worth of SpiceDB relationships in their Cockroach clusters. This migration would:

increase the load on the cluster for a non-trivial amount of time (proportional to the amount of data stored)
increase write transaction latency
increase the disk capacity required to operate, as a new index covering all tuple columns is added.

I'd like to see these optimizations make it into main, but it's a risky change as-is. In an ideal world:

The code can estimate if there is enough disk available to run the migration
the old indexes are only dropped when the new indexes have been validated to be selected by the query planner and provide a visible performance gain
the index creation runs with low priority and limits the impact on the cluster

Perhaps a conservative approach would be:

To introduce logic that determines the size of the indexes to be dropped, and checks if at least the same amount plus some headroom (to account for the newly stored caveat columns) is available on cluster
the indexes are made NOT_VISIBLE to begin with, and a feature flag in SpiceDB enables / disables the use of new index. It's enabled by default, but can be disabled if users detect performance regressions
We can explore if using transaction priority buys us anything when it comes to running DDL statements like CREATE INDEX or DROP INDEX.
On a different subsequent SpiceDB release the new indexes can be made visible, and the old indexes can be dropped safely.

Would like to hear your thoughts on this issue @josephschorr

https://github.com/authzed/spicedb/blob/d77601bfe8b927c78d348144684bfa972b14e199/pkg/datastore/test/watch.go#L49-L53
If I change tuple num to 512, then the test 256-true would pass, while its testing name would be 512-true

If I change tuple num to 512, then the test 256-true would pass, while its testing name would be 512-true

I think that's fine, the goal of the test is to push so many changes the consumer cannot retrieve them for the channel fast enough, a timeout will be triggered if the channel is at capacity, and if enough time elapses the channel Watch API connection will be aborted. Feel free to increase it, id say this is likely a flake.


Containers may not be cleaned up after restart

When the agent has a task removed from its assignment set, it first updates the database to remove the task assignment, then attempts removal of the container (closeManager):
https://github.com/docker/swarmkit/blob/9363459166b93bd4612f8d7a36ddbf65ffc9b3cc/agent/worker.go#L292-L296
If the daemon crashes between updating the database and removing the container, I think the container won't be removed. On startup, the agent walks the database of tasks, and deletes any entries not marked "assigned":
https://github.com/docker/swarmkit/blob/9363459166b93bd4612f8d7a36ddbf65ffc9b3cc/agent/worker.go#L92-L100
It doesn't start task managers for these, which would be needed to perform the container removal.
Would it be more correct to have closeManager update the database after calling tm.ctlr.Remove?
cc @stevvooe

@aaronlehmann Is this a common occurrence? Are there any things that have changed to exacerbate this?

Would it be more correct to have closeManager update the database after calling tm.ctlr.Remove?

This may help. However, there is the issue where we could have removed the task but then it can reappear, due to various race conditions.
This kind of cleanup was original meant to be handled by a component called the "task reaper". The role of the component was to cover this behavior by removing any tasks that have the node's id but are not assigned to the node currently.
Basically, this would address the TODO at https://github.com/docker/swarmkit/blob/9363459166b93bd4612f8d7a36ddbf65ffc9b3cc/agent/worker.go#L87.


add custom azure-ml license manager to a dedicated path

unpack the dedicated monitor build to /opt/rstudio/bin, configure
rserver.conf and statup.sh to point at the new binary

How about matching the path we use for the marketplaces?  Other than that, looks good to me.  How the marketplace is setup:
echo 'server-license-manager-path=/opt/rstudio-license/license-manager'>> /etc/rstudio/rserver.conf
mkdir -p /opt/rstudio-license
mv /tmp/license-manager /opt/rstudio-license/license-manager
chown root.root /opt/rstudio-license/license-manager
chmod 0755 /opt/rstudio-license/license-manager

Fixed up for consistency! Thanks Brian!

Thanks Cole!


Get repository command

Add get repository command to list all repositories (in optional organization) where current user has access:

artictl get repository - works in a "flat" mode
articlt get repository orgname - for "org" mode

The output could include these fields: name, type, user access mode, e.g.
NAME          |    TYPE  | ACCESS
my-registry   |  docker  | RW
mvn           |   maven  | R
js            |      npm | RW

API documentation is here: https://github.com/artipie/maven-adapter/
Ticket: #2

At the moment artipie doesn't provide necessary api endpoints.
Waiting for a solution to the issue artipie/artipie#972


User pref test: change port for provider Auth

Description
Change port for provider Auth
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

user_preferences tests is all passed:

Codecov Report

Merging #4766 (61ccf91) into master (5edfc37) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4766   +/-   ##
=======================================
  Coverage   32.03%   32.03%
=======================================
  Files          61       61
  Lines        5145     5145
=======================================
  Hits         1648     1648
  Misses       3071     3071
  Partials      426      426

Flag
Coverage Δ

gointegrationtests
9.32% <ø> (ø)

integrationtests
∅ <ø> (?)

unittests
25.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 5edfc37...61ccf91. Read the comment docs.

I think this draft doesn't need review rn, considering there are no progress in the PR other than the same Rudraksh's commit. I will mark it open once I am done.

Changed the UI to send pref request as json. Getting 200 Now :+1:

@Abhishek-kumar09 is it expected that the e2e test is failing in this PR? I assume that it will continue to fail unto Rudraksh's other PR is merged, is that right?

No @leecalcote
This PR contains all the @DelusionalOptimist changes. So it is not expected to fail.
Actually user preferences UI and API is working fine and as expected if the user is logged in with meshery-provider.
It is None provider due to which server panics. as discussed.


Added a button for clearing the value

Fixes #15

 You should sign our Contributor License Agreement in order to get your pull request merged.

Reviewed 3 of 3 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

vaadin-date-picker.html, line 30 [r1] (raw file):
Why font-size: 0 is needed?

vaadin-date-picker.html, line 45 [r1] (raw file):
Add quotes around the bound value.

vaadin-date-picker.html, line 97 [r1] (raw file):
Unnecessary comma.

Comments from the review on Reviewable.io

Review status: 2 of 3 files reviewed at latest revision, 3 unresolved discussions.

vaadin-date-picker.html, line 30 [r1] (raw file):
It's not, good catch.

vaadin-date-picker.html, line 45 [r1] (raw file):
Done.

vaadin-date-picker.html, line 97 [r1] (raw file):
Done.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Connection issue between Docker containers using Qdrant client

Description
I'm experiencing a connection issue between my application container and the Qdrant service container in a Docker Compose setup. The application is unable to connect to the Qdrant service, resulting in a "Name or service not known" error.
Error Msg
Traceback (most recent call last):
...
httpcore.ConnectError: [Errno -2] Name or service not known
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
...
qdrant_client.http.exceptions.ResponseHandlingException: [Errno -2] Name or service not known
Steps to Reproduce

Run docker-compose up.
The local-search-application container fails to connect to the Qdrant service.

Expected Behavior
The local-search-application should be able to connect to the Qdrant service using the service name as the hostname within the Docker network.
Actual Behavior
The application fails to resolve the hostname "qdrant" and cannot connect to the Qdrant service.

Hi @amirhosein-vedadi,
This might happen sometimes: would you mind telling me on what OS and with what version of docker and docker-compose are you running the application?

Hi @AstraBert,

Docker version: 27.0.3
Docker Compose version: v2.28.1
Python version: 3.10
Operating System: Ubuntu 22.04

Ok thanks!
Docker and docker-compose seem ok, I'll test the application on my Ubuntu to see wether there is any problem and, if so, how we can fix it :)
Thanks for bringing that up!

Hi @amirhosein-vedadi,
I'm closing the issue for now due to inactivity and no other reported cases: feel free to reopen it if needed.
Cheers,
Astra :)


Android setup links from the tool are broken

Links output by the tool (e.g. by flutter doctor) ending in "#android-setup", such as https://flutter.dev/docs/get-started/install/windows#android-setup are broken because of changes in the Flutter installation documentation structure.
cc @parlough

Thanks for reporting this! I'll work on something to keep these old links working for now and then likely create some sturdier redirects that the tool can use.


Update logstash-indexer.conf

We need syslog input for enterprise - it should replace redis as we dont use it

corrected branch

@adamnfish bump - need this to run update stack

:+1:
With this PR the Infosec Elk Stack has diverged quite a bit so it might be worth forking. We can think about that later though, this looks good.


Staging Release RC_Sculptor-2021-06-07

Staging
Release notes: https://github.com/DataBiosphere/duos-ui/releases/tag/staging_RC_Sculptor-2021-06-07
Deployment: https://duos.dsp-duos-staging.broadinstitute.org/
Production
Release notes: https://github.com/DataBiosphere/duos-ui/releases/tag/production_RC_Sculptor-2021-06-07
Deployment: https://duos.broadinstitute.org/
Important Changes

DUOS-622: Table Sorting
DUOS-1270: Return to Console Link
DUOS-1273: Dataset Metrics

Released


Asserting a Kafka message of interest from the consumed messages

AC:
As a developer/tester(SDET) I want to assert a particular Kafka message from the number of messages consumed.
e.g.
incoming messages(sequence not deterministic):
Message 1  (key1, value1)
Message 2  (key2, value2)
Message 3  (key3, value3)
Now I want to assert only "Message 2" based on "key2" or some matching fields in "value2".
Just checking,
if anyone has done this kind of automation?
Any pointers would help.

Closed by https://github.com/authorjapps/zerocode/pull/546


Extracts essential CSS off the responsive-grid.css file

There's essential CSS on the responsive-grid.css file that I extracted to layout-commons.less and a module for visibility classes. They should not conflict with bootstrap 4.
I have only done a smoke test on the hidden-* ones and they work, but it'd be good to have more confirmation.

Closing this one as I have created a standalone PR against on https://github.com/jenkinsci/jenkins/pull/4697

Ok, this makes sense.


Add RootRepositoriesTempDir to local copier

HSDFCopier does not write directly into the output siva file. It instead does the copy to a temporary directory and does a rename. This is done so no siva files are corrupted in case of borges error or stop.
Do the same in LocalCopier.
https://github.com/src-d/core-retrieval/blob/master/repository/copier.go#L55

For gluster it would be much better copying to file.siva.tmp than copying it to a different folder, as that would imply Gluster would need to do some magic to avoid file being moved to a different brick
Borges makes sure that no two workers are handling at the same time the same rooted repository. It would be great to add the cleanup of .tmp files to avoid doing housekeeping anywhere else.

Most likely this should be implemented above LocalCopier to make sure that a new copier type will work the same way... Especially if the graceful shutdown depends on this to make sure no half copied files exist.

Superseded by #58


Kodi crashes after song is finished

My kodi crashes every time one soundcloud song is done playing.
Since I haven't really payed attention when it happened the two times that I was listening to music on soundcloud. But what I can tell you it that it at least doesn't play the second song of the playlist/artist/search results.

Technical Info

Kodi version: 19 (Matrix)

OS: bullseye on raspberry pi 4

Crashlog: kodi_crashlog-20231101_204248.log

Can you try changing the audio format in the addon settings?

Which one should I try?

You can try "mp3 (HLS)".

That seems to have fixed it! Thanks


diff.Diff() took too long

to := time.After(time.Second * 3)
but, why only set 3 seconds?

Pull request with change welcome!


Expand README

Write some more stuff for da people

@nikkischnelle Is this story/issue still relevant?


Pluggable ScanItems

This PR introduces pluggable scan items in a similar way to dialogs and product features.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1773640
@bdunne Please review
cc @agrare @pemcg @jrafanie

Technically this improves seeding performance as well, but barely any :)

first seed
second seed

before
161.8 ms
111.4 ms

after
104.8 ms
24.0 ms

savings
35.2%
78.5%

Ivanchuk backport details:
$ git log -1
commit 75d79eadf0c5a291e926e6db5557d2d6bccaf5e8
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Mon Oct 14 14:15:39 2019 -0400

    Merge pull request #19388 from Fryguy/pluggable_scan_items

    Pluggable ScanItems

    (cherry picked from commit c0664f72920d8c50019144288e8dff11466bab23)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1773703


Does not eork anymore...!?

It looks like it has stopped working... does anyone know if google is blocking iframes in web receivers now?

Not as far as I know, but sites can stipulate that they not be loaded in an iframe.
This similar project is still working: https://github.com/google/generic-webdriver-server/tree/main/backends/chromecast
Here are the docs on iframe limitations: https://github.com/google/generic-webdriver-server/tree/main/backends/chromecast


Create Pop function from Stack

I have added a program for popping the last value in the stack in Ruby. Please Consider it. Thanks!!

@SiddharthBihola
Great work!
You can follow me and star this repository for further updates.
Have fun 🙌🏼


NameError: name 'plt' is not defined

I'm trialing tracktor on a machine running Xubuntu 16.04, on a jupyter notebook. When running the fish example, everything seems to be going smoothly, except when I try to plot (last section). In that case, the following output error is produced:

NameError                                 Traceback (most recent call last)
 in ()
----> 1 plt.scatter(df['time'][index], df['cum_dist'][index], c='#FF7F50', s=8, alpha=0.5)
2 plt.xlabel('Time (s)')
3 plt.ylabel('Cumulative distance (cm)')
4 plt.tight_layout()
5 plt.savefig('imgs/ex1_fig2a.eps', format='eps', dpi=300)
NameError: name 'plt' is not defined

I believe this is a matplotlib issue (see, e.g., https://github.com/konstantint/matplotlib-venn/issues/33). How can it be corrected?

Are you running an example notebook?
plt is defined when you say import matplotlib.pyplot as plt


fix(client/async): replace asyncify with asyncio.to_thread (#1827)

[x] I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested
This Pull Request addresses an issue where AnyIO worker threads do not terminate properly when using asyncify(get_platform) in an asynchronous context. This can cause tests to hang, especially when using pytest, due to non-daemon worker threads remaining alive.
Changes Made:

Replaced asyncify(get_platform) with asyncio.to_thread(get_platform) in the _request method. This ensures that worker threads terminate correctly after completing their tasks, preventing the hanging issue in asynchronous environments.
Updated the pythonVersion setting in pyright from 3.7 to 3.9.18 to match the current Python version and to support asyncio.to_thread, which is available from Python 3.9 onwards. This change allows linters to recognize asyncio.to_thread and pass linting checks.

Additional context & links
#1827

thanks for the PR, can we instead close the worker threads after they're done? It also sounds like this could be an underlying issue with anyio or just that we're using it incorrectly... do you have any insight there?

The issue occurs because AnyIO's worker threads are non-daemon threads. When AnyIO creates a worker thread, it does not set the daemon attribute, so it inherits the daemon status from the current thread, which is the main thread. Since the main thread is non-daemon (daemon=False), the worker threads are also non-daemon.
Non-daemon threads prevent the program from exiting until they have completed. In AnyIO, these worker threads remain alive even after their tasks are done, causing the program to hang, especially in testing environments like pytest.
AnyIO does not provide an API to close or terminate these worker threads. According to the AnyIO documentation:
"Note, however, that the thread will still continue running – only its outcome will be ignored."
Source: AnyIO Documentation on Threads
To resolve this, I replaced asyncify(get_platform) with asyncio.to_thread(get_platform). The asyncio.to_thread function creates daemon threads, which terminate when the main program exits, preventing the hanging issue.
Regarding Deleting asyncify:
I've confirmed that the asyncify function is only used in this part of the code. If it's acceptable, I can remove the related utility function and update the commit accordingly. Please let me know if this is okay.

Any update on this? It's caused issues on a github actions we're using and this just makes it hang.

I have been suffering from the same bug as @jeongsu-an when using the openai python library in github workflows.  Specifically, the process stalls at the end of the script I'm running and if I print out threads there is a leftover AnyIO thread that is not closing.
I tested the behavior in github workflows using @jeongsu-an's PR branch and it fixes the issues I was having and does not introduce any other issues.

would anyone be able to share a reproduction for this issue? I can't reproduce it with this script running both standard python and pytest
import asyncio
import anyio
import anyio.to_thread

from openai._base_client import get_platform

async def test_main() -> None:
    result = await anyio.to_thread.run_sync(get_platform)
    print(result)

asyncio.run(test_main())

Okay thanks @spokeydokeys, I can confirm that reproduces the issue.

closing in favour of #1853


Multidimentional items

Initial draft for multidimentional items.
The idea is that item can have 0..N controls together.
Maybe item level should keep simple and move specific to controls.
It allows to combine int+float, int+int+int or float+string control in single item.
Previous items implementations become to controls, for exaple ItemList -> MenuItem + ControlList and so on.
https://github.com/user-attachments/assets/02e7f0d0-2918-4d1a-b028-f8c428bd24b9
MENU_SCREEN(mainScreen, mainItems,
    new ItemControl<float>("Temp",
        new NumberControl<float>({.min=-30.0, .max=40.0, .initial=26.5, .step=0.5, .format="%+2.1f\002C", .cycle=false, .blinkerOffset=2}),
        cb1),
    new ItemControl<float>("Price",
        new NumberControl<float>({.min=0.0, .max=10.0, .initial=1.0, .step=0.1, .format="$%2.2f", .cycle=true}),
        cb1),
    new ItemControl<float, int>("Dist",
        new NumberControl<float>({.min=0.0, .max=10.0, .initial=1.0, .step=0.1, .format="%2.1fm", .cycle=false, .blinkerOffset=1}),
        new NumberControl<int>({.min=0, .max=100, .initial=10, .step=1, .format="\003%d%%", .cycle=false, .blinkerOffset=1}),
        cb2),
    new ItemControl<int, int, int>("Time",
        new NumberControl<int>({.min=0, .max=23, .initial=0, .step=1, .format="%02d", .cycle=true}),
        new NumberControl<int>({.min=0, .max=59, .initial=0, .step=1, .format=":%02d", .cycle=true}),
        new NumberControl<int>({.min=0, .max=59, .initial=0, .step=1, .format=":%02d", .cycle=true}),
        cb3));

For now it's not stable, so it's draft, need to check overflow of line length.

MenuItem's hierarchy:

So for different items we can use composition of Controls/Widgets instead of inheritance, should be mo flexible.

You can change your target branch to master and re-open the PR

@forntoh Can you kindly create new target branch for me?)

You can target master, the only reason why we split the other one was because it had so many changes (especially breaking ones), this change is just to include a new item type, right!

FYI @ShishkinDmitriy

For now it's not stable, so it's draft, need to check overflow of line length.

Fixed in #233, you can use it as your new target.


Rewrite the transport for Netty 5

This is a first draft of rewriting the transport to work with Netty 5.
There are many changes; IoHandler, AbstractChannel, Buffer, ... all very different in Netty 5.
The io_uring JNI integration code remain largely the same.
Another change is being attempted in this rewrite: to use blocking file descriptors and rely on FEAT_FAST_POLL,
where blocking IO operations are handled efficiently with a poll-arming mechanism inside the kernel.
This means we don't have to issue separate poll-in/-out and read/write operations; we can instead just
submit our IO operations and wait for them to complete.
The state of this is that basic socket IO works. Datagrams and domain sockets are broken or missing,
TCP fast-open is missing, SSL handshake times out, many tests are failing or timing out.
But many tests are also passing, so progress is being made.

Next week I will take a deep breath and have fun reviewing this, if you feel it's ready bud!

I would drop support for async tasks too: they are dangerous for capacity planning (how many kernel threads are spawned?) And recently read some Jen's A. comment that suggest not using it blindly.
I would prefer instead to have an per-event loop counter to be used to trigger batched early drains from the ring, interleaving users tasks/read to not cause HOL to caught us.
I will given more detail next week if you are interested

@chrisvest Let me know when I should have a first look

@normanmaurer You can start reviewing this now. The SslEchoTests are failing because writability updates are not communicated to the ChunkedWriteHandler, which I'm looking into.

One more thing that needs to happen before we merge this, is switching the log level back to debug, and removing the default test timeout.

I've been working on the UDP packet segmentation support; GRO/GSO. I think epoll is not actually doing things correctly, but I also learned that the Linux kernel don't want to segment packets that go over loopback (localhost, or the self-ip) address. It only segments when packets actually go out to the real network, and this is probably a consequence of GSO doing segmentation as late as possible, which ends up being at a stage that loopback is able to skip.
So that means I can't test it automatically; only manually.

@normanmaurer The only thing missing now is doing the read-loop correctly. The broken read-loop is not caught by the tests, though. So I think it'd be better to fix that as part of, or immediately after, moving this into Netty Core. How about that?

@chrisvest whatever you feel is best. I can spend some time later this week to work on the read-loop stuff as well


splashsceen not work

I fllow you step to config the splashscreen ,but it not work, it has ios version limit?

please dray SplashScreenResource to your project [if you want change image, replace splash.png]

Still does not works, i have image assets should i delete that?

+1 platform ios.
Can  not find the default splash image In the ios program

find the SplashScreenResource file in node_modules>@remobile>react-native-splashscreen>iOS, then drag it to your project

@wangyueg thx,it's work☺

Still does not, i want to add a delay. i want the splash screen to show for a longer period.

@satyagupta Maybe you can control this function-> ‘splashscreen. Hide ()’
componentDidMount() {     setTimeout(()=>{       SplashScreen.hide()     },2000)   }

componentDidMount: function(){     this.setTimeout( function(){       SplashScreen.hide();       this.getLoginStatus();       },5000);   },
this does not works for me :(

@satyagupta Do you use Skype?This is my Skype account：Aran Hu (chinese)

satya.bdt

See https://github.com/mehcode/rn-splash-screen/blob/master/docs/ios.md for iOS documentation; there is also a working example

@wangyueg I can't find the SplashScreenResource file.
My @node_modules>@remobile>react-native-splashscreen>ios folder only contains:

RCTSplashScreen.xcodeproj
RCTSplashScreen/

RCTSplashScreen.h
RCTSplashScreen.m

SplashScreenResource folder was removed in commit 83125b5. If you clone this project yourself and checkout to the d349e55 commit, you can get the files.


Fix 80

This should fix #80 by allowing Event XML that does not contain some of the non-required elements to convert to an object. Also fixes a typo in the event_identifier_type mapping.

Nice chunk of code you've eliminated :thumbsup:

:+1:


Add link the HalGet/SetBusDataByOffset functions to help users find the right method send the IRPs in the "Accessing PCI Device Configuration Space"

Microsoft confirmed HalGet/SetBusDataByOffset functions are helper functions which send the IRPs detailed in the "Accessing PCI Device Configuration Space" document and we can also request to update this document to link the Hal functions to help users find them.
Please help to add link the Hal functions to help users find them.
https://learn.microsoft.com/en-us/windows-hardware/drivers/ddi/ntddk/nf-ntddk-halgetbusdatabyoffset
https://learn.microsoft.com/en-us/windows-hardware/drivers/ddi/ntddk/nf-ntddk-halsetbusdatabyoffset

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

ID: bd0a490c-5642-3863-596f-a0e5d7ba9644
Version Independent ID: f06461a3-83eb-7d2e-e3e7-dfcfd2e69fa8
Content: Accessing PCI Device Configuration Space - Windows drivers
Content Source: windows-driver-docs-pr/pci/accessing-pci-device-configuration-space.md
Product: windows-hardware
Technology: pci
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks @guliteb for the feedback!  I've added a warning to https://learn.microsoft.com/en-us/windows-hardware/drivers/pci/accessing-pci-device-configuration-space (at the bottom).  I also cleaned up https://learn.microsoft.com/en-us/windows-hardware/drivers/kernel/accessing-device-configuration-space and added the links to the sample code.  Finally, added a warning to the top of https://learn.microsoft.com/en-us/windows-hardware/drivers/ddi/ntddk/nf-ntddk-halgetbusdatabyoffset and https://learn.microsoft.com/en-us/windows-hardware/drivers/ddi/ntddk/nf-ntddk-halsetbusdatabyoffset.
The bottom line is we highly recommend using BUS_INTERFACE_STANDARD or IRP_MN_READ_CONFIG / IRP_MN_WRITE_CONFIG, but those two HAL routines are there for backward compatibility if those methods cannot be used.
Hope this helps!


Add comments re: secret_key on Rails 4+ apps

Adding a couple comments in initializer that explain that Devise will use secret_key_base on Rails 4+ applications as its secret_key by default.

Thanks!


Code profiling: identify bottlenecks

See examples here: https://github.com/israelmcmc/cosipy-profiling

Duplicated #27


Too many open files error from UDF server

Describe the bug
E0410 04:40:33.165965010  321185 tcp_server_posix.cc:205]    Failed accept4: Too many open files

Not sure if it has anything to do with the upper limit of the number of file descriptors.
Probably there is leak of connections.
To Reproduce
No response
Expected behavior
No response
Additional context
Reported by one design partner.

Seems to no longer happen after #9240.

The error seems to still exist, in our cluster and a user's cluster.

It should have been fixed by #9916.


Skip NPN tests when libressl 2.6.1+ is used.

Motivation:

LibreSSL removed support for NPN in its 2.6.1+ releases.

Modifications:

Skip NPN tests in libressl 2.6.1+

Result:

Be able to run netty tests against libressl 2.6.1+ as well.

@ejona86 done

Verified it works with https://github.com/netty/netty-tcnative/pull/334


Use systemd unit file to create dummy network

In 4.8 nightly builds, it is observed that the changes for dummy
network not presisted during the reboot. Looking into the networkManager
logs doesn't provide any suspious activity. This patch will create
a systemd unit file which suppose to create the dummy network for
every start.

Has there been a definitive answer explaining why the previous approach is no longer working with 4.8? Without knowing why this is broken, this feels more like a workaround for an undetermined issue than a fix.

Has there been a definitive answer explaining why the previous approach is no longer working with 4.8? Without knowing why this is broken, this feels more like a workaround for an undetermined issue than a fix.

@cfergeau no definitive answer, I tried whatever conversation we had on slack but doesn't find the reason why it is happening.


[Bladerunner - Sleek]

Prerequisites
Before opening an issue, make sure

You have the most recent spicetify version. Yes
Your Spotify version is compatible with the spicetify version you are using. Yes
You have the most recent version of the themes. Yes
You followed the installation instructions for the theme you are using (please go through the entire theme README). Yes

Describe the bug
Bladerunner colour scheme for Sleek theme has black icons on black background.
To Reproduce
Steps to reproduce the behavior:

Go to any playlist or view the playbar at the bottom of the screen
Heart button, next track, previous track, loop, lyrics, volume, fullscreen, devices icons will be black on the black background.

Expected behavior
They should be not the same colour as the background, probably the yellow that goes with the theme.
Screenshots
If applicable, add screenshots to help explain your problem.

Other
From inspecting in the console, by changing the color attribute of the various button elements from rgba(var(--spice-rgb-selected-row),.7); to var(--spice-text), the colour is fixed. I don't know how to make a PR though so uhhhhhh...
Specifics (please complete the following information):

OS: Windows 10
Spotify version: <IP_ADDRESS>2.g94554d24
Spicetify version: 2.10.2
Theme name: Sleek - Bladerunner

Can't replicate issue on 1.1.89/2.10.2, run spicetify restore backup apply and try again.

Running spicetify restore backup apply did not work. Although, I reinstalled spicetify marketplace and that seemed to do the trick.


Improve mix description

ExMachina currently doesn't show up on hex.pm if you search for "factory" or
"factories". I'd like to at least appear on that list. There are some
other libraries that do show up and mention that they are clones of
FactoryGirl. That seems to be something people are interested in, so it
may be worth saying that this is from the same people who created
FactoryGirl.

I think it's worth the experiment to see how it affects traffic!


Gem slows rails boot

Expected behavior
Conceptually, I'd expect bourbon would mostly be a few static sass files and shouldn't appreciably affect rails start up time (e.g. firing up rails console).
Actual behavior
When I ran bumbler, a gem which analyzes how the gems in a rails app's Gemfile affects boot time, I saw that the bourbon gem was one of my slowest gems, adding about 400ms to boot. (For comparison, rails takes about 600ms and the neat gem takes just 6ms).
Steps to reproduce behavior
$ gem install bumbler
$ bumbler -t 10

Bourbon version
We're using the 4.2.3 version of the gem. I tried upgrading to 4.7 but it was even slower.
Environment info
Rails <IP_ADDRESS>. In addition to using bourbon ourselves, we also use activeadmin, which also depends on bourbon.
Screenshots (if relevant)

Hey @losvedir! This is interesting; thanks for looking into it.
I haven’t had a chance to dive into what could be causing this, but out of curiosity have you tried v5.0.0.beta.6?

I’ve run Bumbler on a handful of Rails apps that use Bourbon, and I’m not seeing anything near the ~400ms mark (more like 20ms).
I noticed Active Admin is listed above, as well, which has a dependency on Bourbon. Maybe that dependency is causing a slow boot?
Either way, I’m going to close this as I’ve been unable to track down any specific action items. Bourbon is indeed just some static Sass files.


Misconfiguration Exception: The provided lr scheduler 'StepLR' doesn't follow PyTorch's LRScheduler API.

Greetings
I have been trying to go through your tutorials and keep getting this MisconfigurationException (below). I have tried running it locally and on Google Colab. Please let me know if there is an easy fix.
Thanks!
Ryan

import numpy as np
import pandas as pd
import pytorch_lightning as pl
import matplotlib.pyplot as plt
from neuralforecast import NeuralForecast
from neuralforecast.models import MLP
from neuralforecast.losses.pytorch import DistributionLoss, Accuracy
from neuralforecast.tsdataset import TimeSeriesDataset
from neuralforecast.utils import AirPassengers, AirPassengersPanel, AirPassengersStatic
AirPassengersPanel['y'] = 1 * (AirPassengersPanel['trend'] % 12) < 2
Y_train_df = AirPassengersPanel[AirPassengersPanel.ds<AirPassengersPanel['ds'].values[-12]].reset_index(drop=True) # 132 train
Y_test_df = AirPassengersPanel[AirPassengersPanel.ds>=AirPassengersPanel['ds'].values[-12]].reset_index(drop=True) # 12 test
model = MLP(h=12,
input_size=24,
loss=DistributionLoss(distribution='Bernoulli', level=[80, 90], return_params=True),
valid_loss=Accuracy(),
stat_exog_list=['airline1'],
scaler_type='robust',
max_steps=200,
early_stop_patience_steps=2,
val_check_steps=10,
learning_rate=1e-3)
fcst = NeuralForecast(models=[model], freq='M')
fcst.fit(df=Y_train_df, static_df=AirPassengersStatic, val_size=12)
forecasts = fcst.predict(futr_df=Y_test_df)
Plot quantile predictions
Y_hat_df = forecasts.reset_index(drop=False).drop(columns=['unique_id','ds'])
plot_df = pd.concat([Y_test_df, Y_hat_df], axis=1)
plot_df = pd.concat([Y_train_df, plot_df])
plot_df = plot_df[plot_df.unique_id=='Airline1'].drop('unique_id', axis=1)
plt.plot(plot_df['ds'], plot_df['y'], c='black', label='True')
plt.plot(plot_df['ds'], plot_df['MLP-median'], c='blue', label='median')
plt.fill_between(x=plot_df['ds'][-12:],
y1=plot_df['MLP-lo-90'][-12:].values,
y2=plot_df['MLP-hi-90'][-12:].values,
alpha=0.4, label='level 90')
plt.legend()
plt.grid()
plt.plot()

Error:
MisconfigurationException
Traceback (most recent call last)
 in <cell line: 28>()
26
27 fcst = NeuralForecast(models=[model], freq='M')
---> 28 fcst.fit(df=Y_train_df, static_df=AirPassengersStatic, val_size=12)
29 forecasts = fcst.predict(futr_df=Y_test_df)
30
10 frames
/usr/local/lib/python3.9/dist-packages/pytorch_lightning/core/optimizer.py in _validate_scheduler_api(lr_scheduler_configs, model)
348
349         if not isinstance(scheduler, LRSchedulerTypeTuple) and not is_overridden("lr_scheduler_step", model):
--> 350             raise MisconfigurationException(
351                 f"The provided lr scheduler {scheduler.__class__.__name__} doesn't follow PyTorch's LRScheduler"
352                 " API. You should override the LightningModule.lr_scheduler_step hook with your own logic if"
MisconfigurationException: The provided lr scheduler StepLR doesn't follow PyTorch's LRScheduler API. You should override the LightningModule.lr_scheduler_step hook with your own logic if you are using a custom LR scheduler.

Hey @cash-mckeeman,
Is your bug is related to this past Issue?
https://github.com/Nixtla/neuralforecast/issues/509

I tried the solution listed in Issue #509. After installing the latest from the GitHub main branch, I got a new error:
NotImplementedError: Support for validation_epoch_end has been removed in v2.0.0. NHITS implements this method. You can use the on_validation_epoch_end hook instead. To access outputs, save them in-memory as instance attributes. You can find migration examples in https://github.com/Lightning-AI/lightning/pull/16520.
2023-04-15 19:54:13,782	ERROR tune.py:794 -- Trials did not complete: [train_tune_f5a97_00000, train_tune_f5a97_00001, train_tune_f5a97_00002, train_tune_f5a97_00003, train_tune_f5a97_00004]
2023-04-15 19:54:13,789	WARNING experiment_analysis.py:621 -- Could not find best trial. Did you pass the correct metric parameter?

Hey @kdgutier - I tried the rest of the solution on Issue #509 and I think I got the dependencies fixed. I am now getting a new error:
UserWarning: resource_tracker: There appear to be 1 leaked semaphore objects to clean up at shutdown.
Based on my research, this could mean that my local machine ran out of memory trying to run the nf.fit() method for AutoNHITS and AutoTFT models. I see in your documentation that you have run nueuralforecast on ECR in AWS. Is there a link to review the configuration of the ECR and the Docker image that was used?


Android export failing with custom keystores and no JDK setup in the OS environment

Tested versions

Reproducible in:  v4.3.rc1.official e343dbbcc
Tested on: Windows 11 Pro, Version 23H2, Build 22631.3880

System information
Windows 11 - Godot  v4.3.rc1.official
Issue description
Export for Android fails on Windows if there is no path to a JDK set in the PATH environment variable and debug and/or release keystores have been set in the options dialg of the export window.
The error message printed in the console looks like this:
Could not create child process: keytool -list -keystore ******** -storepass ******** -alias ********

The reason for this is that the static function has_valid_keystore_credentials in platform/android/export/export_plugin.cpp runs the hardcoded "keytool" command instead of using get_keytool_path to get the correct path that uses the export/android/java_sdk_path setting.
Steps to reproduce
Tested on Windows

Ensure your PATH variable doesn't contain any JDK binary directoy
In Godot open Project > Export ... and select an Android export
Specify a Debug and/or Release keystore
Execute Export Project...

Minimal reproduction project (MRP)
Should be reproducable with any project

This error happens. Tried in RC1

 This error happens. Tried in RC1. Same happened in Beta 2 & 3. not sure about B1.

You can workaround this issue by adding the path to the bin folder of a JDK is in your PATH environment variable.

same here


[Feature]: Talkback/Two-way audio implementation

Describe the solution you'd like
Hi!
I've implemented a proof-of-concept talkback or two-way audio feature for this repository and would love to hear your input/recommendations.
But first let me thank you for the amount of work you have put in this project, so that we can make so much more out of our eufy devices.
While I was working on this I did some network and software analysis on the native eufy app. It seems to me that this repository completely relies on the 1.0 api version of the eufylive network.

do you have plans to move to the latest api version?
do you believe that eufy will eventually let the old api version die, so that this project will no longer work? (since the old eufy apps already show a big warning message that an upgrade is required to use eufy).

But now to the talkback feature:
You can find it in my fork under the feature-talkback branch: https://github.com/thieren/eufy-security-client/tree/feature-talkback
I have a battery doorbell from eufy.

T8210 (firmware:<IP_ADDRESS>)
Homebase T8010 (firmware <IP_ADDRESS>h)

With this setup and a modified homebrigde-eufy-plugin (you can find it here: https://github.com/thieren/Homebridge-eufy-plugin/tree/feature-talkback) I am able to use the speech feature inside of HomeKit.
If you want I can open a draft pull request. But I don't think that it is quite ready to merge it into this project yet, since some things are just missing.
What it does is quite simple. You can start and stop a talkback session while a p2p-stream is running. When the talkback session is confirmed from the station side a writeable stream is returned via EventEmitter.
At last you can check whether talkback is supported for the device.
Since I own only a battery doorbell (T8210) this is the only tested and therefore supported device at the moment. But I suppose this approach should work for other devices too.
The input stream has to be of a specific format:

AAC_LC (format: adts)
1 channel (mono)
16000 kHz
~16kbps/s (other bitrates might work, but I had best results with this)

An example ffmpeg command would be:
ffmpeg -i input.mp3 -acodec aac -ac 1 -ar 16k -b:a 16k -f adts pipe:1
The stdout of this process can directly be fed into the talkback stream.
Still ToDo:

Test other devices
Test other codec formats
implement for more than one device per station (for now all is running for channel 0 -> first device)
sanity checks for incoming audio data (is it the right codec/format, ...)
...

Open questions on my side:

I found that you use another packet sequencing for energy saving devices. I didn't dig really deep into this, since I don't use that feature for myself. Is there a quick explanation, so that I can check whether this has to be accounted for here?

For me this works for now. If there is a demand I can try and improve this up to the point where it could be merged.

Hi @thieren,
thank you for sharing your solution :)

do you have plans to move to the latest api version?

Just checked the Android Eufy App v4.2.4_1304 and I still don't see any use of the V2 api (at least in the EU; security-app-eu.eufylife.com), except for authentication (which I have already implemented).

do you believe that eufy will eventually let the old api version die, so that this project will no longer work? (since the old eufy apps already show a big warning message that an upgrade is required to use eufy).

As soon as the Android Eufy app uses this, I will try to implement it.
Where do you see the use of the V2 api?

I found that you use another packet sequencing for energy saving devices. I didn't dig really deep into this, since I don't use that feature for myself. Is there a quick explanation, so that I can check whether this has to be accounted for here?

Yes with these devices additional ping packets are sent and for this I had to introduce this hack with the additional energySavingDeviceP2PDataSeqNumber.
But I think that no additional sequence is necessary for the talkback feature (I have not analysed the network traffic yet).
As soon as I have a bit more time, I will take a closer look at this talkback feature. I've always left it out until now because I don't use it myself :P

Hi @bropat,

Where do you see the use of the V2 api?

Maybe i understood something wrong, or explained myself a bit unfortunate. I mostly analyzed the network traffic from an older android app (which i meant when i said that it uses v1 of the api) and for comparison i have the network trafic from an iphone app which is the newest version.
What i saw, was that the new app had another form of udp packets send to and from the station. To me it seemed as if the packets were encrypted. At least i couldn't decipher these packets with your code. And I didn't dig really deep in the apps code for that matter.
The old version of the android app (would have to look up the exact version if it is important) had network traffic that fits perfectly with your version of the code.
So that is what i meant by v2 of the api. I suppose you understand something else under 'v2 api'.

As soon as I have a bit more time, I will take a closer look at this talkback feature. I've always left it out until now because I don't use it myself :P

Ok. When I have a little more time, i'd like to include this in a PR, maybe as a beta-feature if you'd like.
Just out of curiosity: do you own any devices from eufy, that support talkback?
As said i only own a battery doorbell and so can't really test this for other devices.

So that is what i meant by v2 of the api. I suppose you understand something else under 'v2 api'.

I meant the Eufy Cloud REST Api. ;)
I am aware of the P2P protocol and the encryption. I could not spend much time on it.
If you could be helpful, it would be great ;)

Just out of curiosity: do you own any devices from eufy, that support talkback?

Yeah, T8114 and T8410.

I meant the Eufy Cloud REST Api. ;)
I am aware of the P2P protocol and the encryption. I could not spend much time on it.

Yeah, I figured that.
Somehow I believed that the P2P encryption had something to do with the v2 api. My bad.

If you could be helpful, it would be great ;)

Sure, would love to, but don't count on it any time soon :D
Unfortunately I have a lot of other stuff to do at the moment and things like the talkback feature are of more use for me. But I'll try to have a look when it suits better.

@thieren
I have added the talkback feature in the development branch. I could not test it yet. Maybe you can test it @thieren? ;)
Let me know.

@bropat
Cool thanks. Will test as soon as I can but could well be a week or so.

Hey @bropat
tested the feature and created a PR for some fixes. Without them the feature won't work.
No rush, but any idea when to expect this in the main branch?
If anyone wants to test it you could download a cmd line tool here: https://github.com/thieren/eufy-test-client (start a livestream for a device and then start the talkback).

Hi @thieren,
thank you for testing and contributing.
I think this weekend should be doable.

@thieren released ;)

Hi @thieren
I'm implementing talkback using this library and was successful to stream an mp3 file to my T8200 doorbell. But I notice heavy stuttering that. Therefore I'm reaching out to you if you had similar issues during implementing this feature.
Also: how did you figure out how the talkback stream must be encoder?
I wonder if different cameras might expect different codecs.
Any help would be really appreciated

Hi @thieren I'm implementing talkback using this library and was successful to stream an mp3 file to my T8200 doorbell. But I notice heavy stuttering that. Therefore I'm reaching out to you if you had similar issues during implementing this feature.
Also: how did you figure out how the talkback stream must be encoder? I wonder if different cameras might expect different codecs.
Any help would be really appreciated

Hi @oischinger
stuttering audio would be most likely caused if you stream an incorrect bitrate or maybe sample rate to the device. I found that these values must not be too high.
For me a bitrate of 20 kbps and a sample rate of 16kHz works just fine. If I set a bitrate of 64kbps the audio begins to stutter for example.
As for the codec: I analysed the network traffic and found an adts stream in the udp packages sent to the camera. You can deduce some codec settings from the adts header and I fiddled a bit with the encoder settings to get it working.
However I did not extract the audio stream from the network traffic and analysed it for bitrate and sample rate. That was mostly trial and error until it worked. Since other devices might be good with a higher bitrate there is also no limitation on what you can send to the device.
In theory it is possible that other devices use another codec but I think this is unlikely. Also if you have stuttering audio the codec seems to be fine, since you can hear at least something. But the encoder settings seem to be off, like explained above.

Hi  @thieren thanks for the useful information.
In the end my main problem was that I was assuming the output track to be mono but instead it needs to be stereo -ac 2 instead of  -ac 1.
Maybe some doorbells can do stereo..

Interesting that this should be an issue. The channel information is something that is actually present in the adts header and it was mono in my network analyzation. So seems that you're right and some devices may need stereo encoding.
We use talkback with default settings (mono, 20kbps, 16kHz) in our homebridge plugin and haven't any reports of this kind for now. But maybe it's not just released long enough.


Add the config preset features to the view

When new code is added that is conditional on a feature, that code needs access to the config object. This can mean passing the config object as a parameter through several layers of calls. Adding the preset features to the view solves this problem.

Pushed a commit that changes presets_ from a shared_ptr to a ref after feedback from Vinnie. Rational: if someone is using the features after the config object goes away we'd rather know about it (and using a shared_ptr introduces a memory fence as well)

Current coverage is 67.12% (diff: 85.18%)

Merging #1981 into develop will decrease coverage by 0.03%

@@            develop      #1981   diff @@
==========================================
  Files           686        686
  Lines         49319      49301    -18
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits          33120      33091    -29
- Misses        16199      16210    +11
  Partials          0          0

Powered by Codecov. Last update 7799957...3766e32

Rebased onto 0.60.0-b3 and added two commits.
@miguelportilla Can you review the Remove Config dependency commit I cherry picked from Ed? (I reviewed it, thumbs up from me)
@miguelportilla and @ximinez Can you review the last commit Rules.impl_ is never null.
Thanks.

@seelabs On it! I'll review the cherry pick and the last commit today.

@seelabs The cherry pick from Ed and your last commit look good to me.

squashed

Merged as https://github.com/ripple/rippled/pull/2005/commits/f5af8b03de8d770d49e444b60ffe2b9063f0ad91


Add OAuth endpoints using League oauth-server package.

What's this PR do?
This PR introduces OAuth authorization endpoints using a v5 release candidate of the League's oauth2-server package. It also introduces a new v2 namespace, where the new OAuth versions of the authentication endpoints will live.
There's three major components things that are worth taking a peek at here:

The AuthServiceProvider now configures the OAuth server so it can be resolved from the service container elsewhere in our application. It's responsible for registering repositories (how the package knows how to store/retrieve data) and enabling & configuring which grants we're going to use.
The repositories allow us to customize how the OAuth package talks to the database.
The entities are returned by the OAuth repositories... they're our Eloquent models translated to a format the OAuth package expects.

References #202.
How should this be reviewed?

...mwahahah!
Okay, but really it's mostly scaffolding to glue together our models and the OAuth server! I'd recommend just skimming the whole diff. I also added tests and documentation for each of the new endpoints. This PR does not handle validating JWTs or include middleware for these new tokens.
For the time being, the new endpoints are only available in debug mode so that we can safely continue developing these features without creating a massive feature branch. :construction_worker:
Checklist

[x] Documentation added for changed endpoints.
[x] Tests added for new features/bug fixes.

For review: @weerd

So. Many. Files. 😱
Anyhoo, some minor DocBlock edits but overall looks super cool! 👍

Thanks @weerd! :bow:

👍


SRE-16: Add nix dockerTools image, build and push to dockerhub per commit

This completes point 1:

base image with just the binaries

This PR provides a method to build docker binaries that cane be used with an appropriate set of CLI flags.
Images are tagged using the the commit in which they are built.

In this case I don't see must benefit in supplying command-line parameters to cardano-node in the entrypoint script. Just make the entrypoint cardano-node and let the user do the rest.
Set up /data and /config using extraCommands.
Have you got it to automatically push images to Dockerhub from CI?

I agree on using cardano-node as the ENTRYPOINT (this was our intent), however the syntax of the  causing problems, and the script worked as expected. @craigem having another look at this now is a good idea.
I disagree on removing the arguments though. it's very nice to simply mount the volumes by convention.

Also, to ensure the scope of the change is captured so the merge makes sense, maybe rename this PR to "Add nix dockerTools image, build and push to dockerhub per commit"

We should only push on tagged releases, not every commit.

@disassembler pushing every commit is part of the tagging strategy, as it's a requirement to target any commit for the purpose of pinning of dependencies in production or working with the services outside of the released versions.

If we do think this is fine for a first pass, maybe it makes sense to use forEnvironments to create docker config directories with nix for every environment that the user can map to a directory on the container and execute cardano-node against that if the directory exists.
Something like nix-build -A dockerConfigs.mainnet
docker run -v ./result:/config cardnao-node
Which would run:

I leave it up to @rhyslbw if he thinks this is a decent first pass worth merging, but no end user will use this.

Agree with you @disassembler, it's intended to be the base we build from, where now to make the supported network extensions is simply a matter of mounting files into the containers using a standard file structure. @craigem let's present the full offering in this PR.

I really think it makes more sense to embed the configs for all the environments in the docker image and provide a way to override the default configs with your own.

I disagree because the container is only concerned with only the network it's configured for (See CAD-556). From a consumers perspective this makes sense to me and forms a decent descriptive standard:
dockerImage.cardano-node
dockerImage.cardano-node.mainnet
dockerImage.cardano-node.byron-testnet
dockerImage.cardano-node.some-other-future-public-network

eg
docker load -i $(nix-build -A dockerImage.cardano-node.mainnet --no-out-link)

docker run cardano-node
# or override complete config.yaml
docker run cardano-node -v ../custom-mainnet-config.yaml:config/config.yaml

yeah, if we build separate images for each network, I fully agree. I thought one of your requirements was a single image that could support multiple environments, but maybe I'm mistaken.

I made some progress on this. It now builds images for each environment in dockerImages. Although the tagging is only dependent on the gitrev which needs to be resolved. it should probably be env:gitrev || env:master or similar.

 docker load -i $(nix-build -A dockerImage --no-out-link)

The commonLib is removed wth commit 5ba30ad. However as it's now trying to build multiple images the load fails.
I'm unsure why we've gone backwards on all the other tests. Looking into them now.

bors r+

bors r+

bors r+

bors r+

bors r+

bors r+


problem starting up

npm start -->

is there a live demo or some screenshots of how the result should look like ?

Please look at the Readme file for the project, under "Firebase configuration"

you need to add a config.js file in /src with your firebaseConfig data and Mapbox API key


Ackowledge empty statement at end of function to "specify" void return type

To stop a function from returning implicitly, we can use
some-fn = !->
    do-shit()

I think it would be very neat if we could also:
some-fn = ->
    do-shit()
    ;

The last semicolon should count as an empty statement, and therefor be 'undefined', and therefor return nothing.

You can use void for this purposr

That was indeed better :)


Updating dependencies with @types/paho-mqtt and @types/zen-observable

Issue #, if available:
Issues with angular #629
Description of changes:
Updating dependencies with @types/paho-mqtt and @types/zen-observable
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #689 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #689   +/-   ##
=======================================
  Coverage   87.28%   87.28%
=======================================
  Files          72       72
  Lines        3428     3428
  Branches      652      652
=======================================
  Hits         2992     2992
  Misses        419      419
  Partials       17       17

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 8780ec5...ff6ed02. Read the comment docs.


add apm-spring-annotation-plugin to dependency  not work when deploy in tomcat

Why do you submit this issue?
[ ] Question or discussion

Question

What do you want to know?
In order to trace local spring span, I  can put apm-spring-annotation-plugin-6.0.0-GA.jar from optional-plugins to plugins. But I wonder if I can add apm-spring-annotation-plugin to  dependency for tracing spring bean?  It is OK when I test this by staring my springboot application with inner tomcat, but it fails when I start my application by puting it in wabapp of tomcat 。
I wonder if this is the result of classloader of tomcat? Or can you explain it, thanks.

I think it should work in Tomcat. It has no difference with other plugins. They all should work in Tomcat. No classloader issue in default Tomcat.

I think it should work in Tomcat. It has no difference with other plugins. They all should work in Tomcat. No classloader issue in default Tomcat.

You mean put  apm-spring-annotation-plugin-6.0.0-GA.jar in TOMCAT_PATH/webapps/APPLICATION/WEB-INF/lib  will work as put it in AGENT_PATH/plugins ?

I think it should work in Tomcat. It has no difference with other plugins. They all should work in Tomcat. No classloader issue in default Tomcat.

Put  apm-spring-annotation-plugin-6.0.0-GA.jar to TOMCAT_PATH/bin works.
I think AgentClassLoader's parent is AppClassLoader, and it's class path is TOMCAT_PATH/bin.
Thanks for your reply.

In my mind, in AGENT_PATH/plugins should work too.

In my mind, in AGENT_PATH/plugins should work too.

Yes


Change rewrite -> master

rewrite is master now!
So I thought it would be cute to edit the old one to tell about this change.

LGTM, thanks.

No problem.


Django 1.9 incompatibility.

When launching project this happens. For now it's Django 1.8 this will work, but in Django 1.9 this will fail to load.
/home/djangocms/djangocms/local/lib/python2.7/site-packages/imagestore/utils.py:8: RemovedInDjango19Warning: django.utils.importlib will be removed in Django 1.9.
from django.utils.importlib import import_module

I'm plan to add support of 1.9 after next release of django CMS, because django CMS is required for tests.

:+1:

Closed via 3269ebdb477a76b41b4e7e85612701c6d5490f12.


I failed to change locale to "en"...

I need to add i18n support in Laravel 9/Inertiajs2/vuejs3 app and I try to use
"laravel-vue-i18n": "^1.4.4". I added 2 dirs AppPath[/lang/en](file:///_wwwroot/lar/TodoList/lang/en) and  AppPath/lang/ua
and in resources/js/app.js I added i18n support :
import { i18nVue } from 'laravel-vue-i18n'
...
createInertiaApp({
    title: (title) => `${title} - ${appName}`,
    resolve: (name) => require(`./Pages/${name}.vue`),
    setup({ el, app, props, plugin }) {
        return createApp({ render: () => h(app, props) })
            .use(plugin)
            .use(i18nVue, {
                resolve: lang => import(`../../lang/${lang}.json`),
            })

            .component('inertia-link', Link)
            ...
            .mixin({ methods: { route } })
            .mount(el);
    },
});

and in webpack.mix.js :
require('laravel-vue-i18n/mix');

mix.js('resources/js/app.js', 'public/js')
    .i18n()
    .vue(3)

As result I have 2 generated files in public/js/lang_php_en_json.js  and public/js/lang_php_ua_json.js with all
labels from AppPath/lang/ and english label is applyed in vue file with {{ $t(' method.
I need to set ua as default language. I set in config/app.php :
        'locale' => 'ua',
it works.
But I failed to change locale to en with control method :
    public function change_locale(Request $request)
    {
        App::setLocale($request->lang);
        session()->put('locale', $request->lang);

        $app_locale = app()->getLocale();
        \Log::info(  $app_locale ); // it shows “en”

        return response()->json([
            'lang'    => $request->lang,
            'message' => 'Language was successfully changed',
        ], 200);

    }

On client side I still see "ua" labels
have I to set changes locale on js side somehow ?
Thanks in advance!

Hey Petro yes, since you are doing that with an AJAX request, you may need to use the: https://github.com/xiCO2k/laravel-vue-i18n#loadlanguageasynclang-string functions as it is shown on the README.
If you want to control all from the backend you just need to make sure the <html lang=""> is set to the correct one.
Hope that helps.


Enhance contact us part

What happened?
As the contact us section is not properly managed as I saw the icons were not properly managed.

How can we reproduce this bug?
I can make it organize well by make icons separated by some space and make it in two rows.
Desktop Information (Optional)
No response
Urgency (Optional)
None

Hey please assign me the issue. Managing the social media icons in a convenient and modern way is all you want right?

Hello ,
I'm harsh. I want to work on this issue under Gssoc'23. I am really eager to work on open source projects and contribute to this community.I've taken a look at the codebase and I believe I can make some valuable improvements. I'll start working on it right away.
Skills
HTML5, CSS3, JS, Bootstrap | node js mongodb | Php, mysql | MERN | Responsive Design
Can you please assign this issue to me.
@agamjotsingh18

Hey, this issue is already raised before


Support blake2 and blake3

BLAKE2b (128, 256, 512) will be supported but had to ask for clarification if length made any sense for BLAKE3. See https://github.com/BLAKE3-team/BLAKE3-specs/issues/3
Outcome: CycloneDX v1.2 will support:

BLAKE2b-128
BLAKE2b-256
BLAKE2b-512
BLAKE3


AAD support for WPS and new API from swagger to generate token

Moving the PR https://github.com/Azure/azure-sdk-for-java/pull/23861 to release branch to allow pipelines to run.

/azp run java - webpubsub - ci


Fix: Annoying gap in details layout browser.

fixes https://github.com/files-community/Files/issues/13819
Validation
How did you test these changes?

[X] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

Open app ...
Click settings button ...

Screenshots

We're not against removing the gap from the divider line, but this PR seems to disable using the context menu/selection rectangle in the space to the left of the file list.

Closing for now, but please reopen if you have an idea on how to achieve this.


Support for Unicore

We should look into how we can support the upcoming .Net Core version of Umbraco.
My initial thoughts is to move the Umbraco specific parts to their own projects and be able to share as much code as possible.

There seems to have been some activity around this by @benjaminc I'm interested to what the road map for this looks like and how I may be able to help. I've got the v9/dev branch up and running locally but I can't and queries working so I wondered if there where are areas could help out with?

Yes, I did start a v9/dev branch for anyone who wants to help out.  My first step was migrating the projects, and just getting things compiling.  I haven't had time to get back to try and get things working, so if you wanted to start troubleshooting it, that would be great.  More broadly, we have a few items we are planning on working on in the project as a whole, which you can see on the issues list.  From security, to filtering, to multi-lingual support to name a few.  Feel free to dive in an pick up any of them, just throw a note on the issue saying you are working on it so if someone else wants to help they can collaborate.

Great, I'll have a look and see if I can work out whats going on. Initially I could not get the example site up and running so I have a couple of changes that might help with that.
I'm not overly familiar with GraphQL but I'll see if I can work out how all the query parsers work and why data is not being returned.


Dimension error on metrics argument when updating to keras 2

I had a keras model that ran fine with keras 1, but having updated to keras 2 (and updated tensorflow) I now get an error trying to run this:
vae.compile(optimizer="adam", loss=vae_loss,metrics=["categorical_accuracy","top_k_categorical_accuracy"])
The following error shows up.
Dimension (-1) must be in the range [0, 2), where 2 is the number of dimensions in the input. for 'metrics_4/categorical_accuracy/ArgMax' (op: 'ArgMax') with input shapes: [?,?], [].
The keras release notes doesn't mention anything about arguments of the categorical_accuracy metric to have changed. What am I missing?

I have the same issues. how to deal with it

It is fixed if your loss function is not using the "objectives" class from Keras 1, instead you should use the losses class.

I still get the same error after changing to losses class:
model.compile(loss=losses.categorical_crossentropy, optimizer='sgd', metrics=['accuracy'])
Do you know how to fix this? Thanks

Did anyone solve this problem?

Same problem here. Any solution?

I ran into this problem only after I had updated my Tensorflow to the GPU optimized version that used CUDA. It worked properly before the upgrade.
Maybe there is an issue with the %path% environment variable? I cannot think of anything else. Any solution would be greatly appreciated.

I have the exact same Problem as jshobhit. After updating to tensorflow-gpu i get this "ValueError: Dimension (-1) must be in the range [0, 2), where 2 is the number of dimensions in the input. for 'metrics/acc/ArgMax' (op: 'ArgMax') with input shapes: [?,?], []."
Help would be greatly appreciated :)

Maybe try to update your tensorflow library to 1.x.y.

it worked for me by changing the syntax to
model.compile(optimizer=keras.optimizers.Adadelta(),loss=keras.losses.categorical_crossentropy)

@change2014 If you remove metric then it will go away but you can't evaluate the matrix accuracy, only losses.
import keras.backend as K
def get_categorical_accuracy_keras(y_true, y_pred):
return K.mean(K.equal(K.argmax(y_true, axis=1), K.argmax(y_pred, axis=1)))
model.compile(loss=losses.categorical_crossentropy, optimizer=sgd, metrics=[get_categorical_accuracy_keras])
I tried this and it works for me to calculate both losses and accuracy.


Compatibility with Azimuth

With this PR, we can boot against the latest version of the contracts as deployed on Ropsten.
This also applies the rebranding to the stdlib arms and associated faces.
I want to do a re-organizing of the ++ethe and ++ethereum cores, too. Here's the structure I want (and tried) to use, but currently doesn't actually work:
++  azimuth
  =>  ::  everything under ++azimuth:ethe (azimuth-related structures)
  ::  everything under ++azimuth:ethereum (logic for those structures)
::
++  ethereum
  |%
  ++  key
    ::  address-from- logic
    ::  signing logic
  ::
  ++  rlp
    ::  rlp logic as it currently exists
  ::
  ++  abi
    =>  ::  ethereum abi structures from ++ethe
    ::  everything under "encoding"
    ::  everything under "decoding"
  ::
  ++  rpc
    =>  ::  ethereum rpc structures from ++ethe
    ::  everything under "making calls to nodes"
    ::  everything under "parsing responses from nodes"
  ::
  ::  misc utils that aren't *strictly* ethereum-related, but have nowhere else to go
  --

Problem here is that zuse is split up into two cores: models and engines. This is why ++ethe existed as separate from ++ethereum, a reason I completely forgot.
We strictly need the models to be in the first core, because they're used in the models for jael. We strictly need to engines (ie old ++ethereum) to be in the second core, because it uses arms from that engine core.
There's probably reasons why we care about separating models and engines so strongly, rather than just having them be in separate sections within the same core. That would mean we do want to keep the ++ethe and ++ethereum split. Please confirm, and any advice about making that split (accounting for the ++ethereum ++azimuth split too) slightly more elegant would be appreciated. I'm probably just being dumb/stubborn about it.
(There's also a couple todo's here I still want to try touching up, but I'll get to that after the re-org.)

First, I want to note that all the arms in the |dawn core exist to be called by the runtime (see vere/dawn.c). Unfortunately, there's also a couple places where we grab these types by name. So these changes must be mirrored in that file before this could go in. It might be a good idea to make the |dawn interface more generic, perhaps just +give and +take gates that operate on a list of RPC requests or responses, respectively,  but that is a larger task.
I like your proposed refactor, and think we can get close to something like it. A smattering of thoughts on the subject:
I believe the core separation is for performance (constant-folding, and I'm not sure what else). I don't know if separate chapters of the same core provide the same benefit, but that could be an option if so.
Is there are a technical distinction we could leverage for namespace-ing? +solidity vs +ethereum, or something along those lines? Also, it seems like +jael only depends on a few of these types, so I bet we could lift those specifically into the models core.
And we can always put two cores in the same namespace:
++  foo
  =>  |%  ...  --
  |%   ...  --

First, I want to note that all the arms in the |dawn core exist to be called by the runtime

I am aware. Was pleasantly surprised to find that no C changes were needed at all here! I also haven't touched, and don't plan to touch, the organization or naming of ++dawn. It seems to serve its purpose well enough for now.

Is there a technical distinction we could leverage for namespace-ing?

Aside from what I'm already proposing? Not really. What you imagine as ++solidity lives under ++abi, since that's the only place outside of the blockchain you really ever interface with solidity concepts (that is, its types and functions).

And we can always put two cores in the same namespace

So I'm already doing that for the type-logic distinction within the proposed cores. How sane would it be to suggest:
|%  ::  zuse structures core
++  azimuth-types
  |%  etc  --
--
::
|%  ::  zuse engines core
++  azimuth
  =>  azimuth-types
  |%  etc  --
--

and similar for all the other types cores. Then just use ++azimuth always, outside of zuse's structures core?

I think your proposed core structure(s) would work. I would try to limit +azimuth-types to the bare minimum necessary for the %jael moves.

It would, of course, have been nice if I actually tested this thing after doing the renaming. (^:
See urbit/urbit#1111 for the C-level changes, just renaming.

Not 100% satisfied with the reorg of the cores, but it's good enough for now. Happy to incorporate feedback if there is any.
@joemfb I touched up some logic in ++point-from-eth, ensuring two things:

Don't crash in the not-uncommon case of key revision 0 (usually as in "no keys configured").
Activated ships always have networking state that can be modified.

The latter was already there, and an implementation error on my part. The former comes from needing the product of ++pass-from-eth, which returns ~ if the crypto suite is not 1 (or if either pubkey is not 32 bytes, but they're always parsed and stored as if they were, so that's practically never hit).
There's other TODOs remaining in the Ethereum and Azimuth code, but none of them are strictly necessary. I'll put them on the release-candidate punchlist, maybe I'll poke at a couple over the holidays.
For convenience, to test against latest Ropsten deploy, use this key for ~zod:
-G '0wln.i~qct.VWDJf.DCmIw.GQnvj.QUE4n.ZcbkZ.yclBJ.rSaER.9-wxe.EZf3G.1pMDq.Am6pu.kjaK3.1Zw6d.RoFeF.h2pNq.ZtTEg.4w0sp'

Tested with the latest Ropsten contracts, everything looks good (barring the style requests above).

@Fang-, please temporarily hold of merging this. I'm fixing pill creation in CI, and would like to get that in before this.


Download problem for ZMQ

While compiling mxnet I run into a certificate issue with Github. Maybe the right thing to do is not to download binaries, but to recompile zmq?
hive:/data/mxnet (master *)> make
make CXX=g++ DEPS_PATH=/data/mxnet/deps -C ./ps-lite ps
make[1]: Entering directory `/data/mxnet/ps-lite'
rm -rf zeromq-4.1.2.tar.gz zeromq-4.1.2
wget https://raw.githubusercontent.com/mli/deps/master/build/zeromq-4.1.2.tar.gz && tar -zxf ze
romq-4.1.2.tar.gz
--2015-11-16 15:02:00--  https://raw.githubusercontent.com/mli/deps/master/build/zeromq-4.1.2.t
ar.gz
Resolving raw.githubusercontent.com... <IP_ADDRESS>
Connecting to raw.githubusercontent.com|<IP_ADDRESS>|:443... connected.
ERROR: certificate common name “www.github.com” doesn’t match requested host name “raw.githubus
ercontent.com”.
To connect to raw.githubusercontent.com insecurely, use ‘--no-check-certificate’.
make[1]: *** [/data/mxnet/deps/include/zmq.h] Error 5
make[1]: Leaving directory `/data/mxnet/ps-lite'
make: *** [ps-lite/build/libps.a] Error 2
hive:/data/mxnet (master)>

I just add '--no-check-certificate' in the script, and everything goes fine.

Yeah, but from a security perspective downloading software without at least
verifying the host and the executing it on a system is a bit no-go.
On Mon, 16 Nov 2015 22:44 董国盛<EMAIL_ADDRESS>wrote:

I just add '--no-check-certificate' in the script, and everything goes
fine.
—
Reply to this email directly or view it on GitHub.


Downloading Files from Dashboard not always possible

When I try to download the zip files from the dashboard, I often get a message telling me the following (even if it is the first time after a while I try to download files). Anything that I can do to get access?
Error 1015 Ray ID: 697583dbda8fee44 • 2021-10-01 11:57:43 UTC
You are being rate limited
What happened?
The owner of this website (dl.fbaipublicfiles.com) has banned you temporarily from accessing this website.

This happens frequently, I am trying to setup a new dashboard somewhere else.
I've sent CSV files directly to you.
Looking for a solution for hosting the dashboard. Maybe I can make it compact (like PDF export with medium resolution) e.g. on the facebook group nevergradusers and just send detailed files on demand.