Does this implementation survive app restarts?

Is the time association persisted to disk in case the user later opens the app while disconnected from the internet?

Not currently, but this is actually the next feature in the pipeline. TrueTime for Android currently does (as-of https://github.com/instacart/truetime-android/pull/15) so we'll probably be copying the method used there.

Awesome. I implement something similar to TrueTime in my game but this appears much more robust so I'll be stoked to drop this in instead :)

@warpling  @msanders Is this currently supported?

@otymartin don't think so. I've added it on my own by adding an abstraction around TrueTime that saves a time from TrueTime associated with a bootTime and uptime. With that you can check if the association is valid by verifying that the saved bootTime matches the current bootTime (since it'll only change when the device is restarted) and then calculate the current time (if TrueTime can't) by taking the saved time in the association and adding the interval that is the difference between the current uptime and the uptime saved with the time association.

This is the method I use to get uptime:
(A simpler one I used previously that read kern.boottime seemed to drift?)
// Alternate method that doesn't drift
// Source: https://github.com/jmah/MyLilTimer/blob/master/Classes/MyLilTimer.m
+ (NSTimeInterval) timeIntervalSinceBoot {
    // TODO: Potentially a race condition if the system clock changes between reading `bootTime` and `now`
    int status;

    struct timeval bootTime;
    status = sysctl((int[]){CTL_KERN, KERN_BOOTTIME}, 2,
                    &bootTime, &(size_t){sizeof(bootTime)},
                    NULL, 0);
    NSCAssert(status == 0, nil);

    struct timeval now;
    status = gettimeofday(&now, NULL);
    NSCAssert(status == 0, nil);

    struct timeval difference;
    timersub(&now, &bootTime, &difference);

    return (difference.tv_sec + difference.tv_usec * 1.e-6);
}

Hi @msanders,
First of all thanks for open sourcing this useful lib.
Secondary I have question which should help me to pick best NTP lib for iOS:
Do you have any rough estimates when could we expect this feature to be implemented?


May you please support openstack icons?

[Y] I agree to follow the Code of Conduct that this project adheres to.
[Y] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

I need to have openstack icons as I found docker and vagrant icons.
Thanks

Where are these sets and where are the licenses to the sets?

https://www.openstack.org/brand/openstack-logo/logo-download/
https://www.openstack.org/project-mascots/

Logos are licensed under the Creative Commons CC-BY-ND, similar to other Foundation materials. That means you can use them freely, but can’t make derivative work, such as adding a for-profit company’s logo into your community project’s logo.


Error on failed COPY FROM

Fixes https://github.com/invenia/LibPQ.jl/issues/99

Merging so I can move on with dependent async changes; comments still welcome.

Haven't tested yet, but this is hugely appreciated, thanks.

No problem! It was an oversight for sure.


I have to type in my passcode whenever I start the terminal

Is there any way to avoid having to type in my key passphrase every time I launch the terminal?

I guess this one is not for me, found another one that "just works" for me. Thanks for the info :)


Adding alt attribute for teaser images

Without an alt attribute, HTML validators like HTML-Proofer will consider the
generated HTML to be incorrect. Also adds a strip filter to the title because
otherwise titles have a trailing newline.

This is somewhat subjective. Get why you'd want an alt description, but in it's current state it offers little value to screen readers here. Just adding the post's title to the image is redundant to the post title on the page already. If you turned on a screen reader or assistive technologies it would sound silly hearing it read the post title, then post title again as it hits the image.
Ideally it would describe the image, which does benefit screen reader users.

Ah, that's true. Closing.


Modeling a HP Steam generator with TESPY

I am trying to model a high pressure steam drum from a water tube boiler, in which the saturated liquid (out1) flows into an evaporator and then back into the drum as saturated steam (in2).

Is it actually possible to recreate this connection using tespy component "Drum" ?  When trying, the program shows the following error:

Hi @ibuzzo,
could you attach a minimal code example to reproduce the issue? The drum component would be the one you are searching for, so I guess there is an issue in your model.
Thank you

Please open a discussion on the oemof/tespy repository, thanks


build examples with cmake and modern compilers (clang-18 and gcc-14)

Is there anyone who can take a look?

The idea of adding CMake or makefiles for everything is a little bit against "single-C-file" idea. TLSe is meant to just be added to your project, not to be built as a stand-alone library.


6.0.0 Rewrite

DO NOT MERGE - WORK IN PROGRESS
Please run a local instance of the bot and do additional testing before you review.
Changes

Added dependabot
Added CodeQL
Switched to gradle build system
Updated license
Updated Readme
Updated a shit load of dependencies
Removed JDA-Utilities as it was abandonware
Use brigadier command parser
The console can now execute commands
All commands migrated to brigadier
All commands (and their arguments) have suggestions
Commands which send web requests now defer, this should hopefully prevent "Application did not respond" errors.

TODO:

[x] Move non-RC24 specific brigadier implementations to their own library. (This will help shrink the PR by quite a bit)
[x] Command descriptions
[x] Command permissions
[x] Test commands that use database
[x] Register commands globally
[x] Fix bot presence

KNOWN ISSUES:

https://github.com/discord/discord-api-docs/issues/5878 prevents suggestions from appearing after you type a space, until you type more after the space

+6.0.0

[ ] #114
[ ] Something about /GameTDB
[ ] /Help command
[ ] Automatically add license headers to all java files
[ ] Implement a code style
[ ] Perhaps add a workflow that can check for updates to source dependencies in settings.gradle
[ ] https://github.com/Gamebuster19901/Roll/issues/43
[ ] A ConsoleEmbedPrinter would be nice for pinting embeds to the console, but this might be out of scope.

Ready for testing and review.
Please note: code style inconsistencies are out of scope for this PR. Will be in the next PR.


Add annotations to the code

I was working on dbt-spark yesterday, and it was sometime a bit tricky to see what's going on. Having some additional annotations (especially in the adapter part) would really help for newcomers to get around.

Sorry for the CLA spam - should be fixed now 😅. Thanks for signing that @Fokko and really cool PR!  @cla-bot check

Thanks @drewbanin I've signed the CLA about two hours ago. No worries about the spamming.

Looks good! I've kicked off the full test suite for this, I don't anticipate any failures. Once it passes we'll get this merged (likely for 0.15.1)

The successful azure tests for this PR are here: https://dev.azure.com/fishtown-analytics/dbt/_build/results?buildId=1142

My pleasure @beckjake
Thank you all for developing DBT, I starting to fall in love with it :-)


operator project-quay (3.9.0-nightly.20230302)

Add 3.9.0-nightly.20230302 to community operators

/merge possible

/merge possible


[close #383] fix integration test case flow_control & stop_downstream

What problem does this PR solve?
Issue Number: close #383, close #393
Problem Description: integration test case flow_control/stop_downstream is flaky
What is changed and how does it work?

Decrease record size of go-ycsb workload from 1KB to 100B
Retry PD connection with unlimited count.

Code changes

Check List for Tests
This PR has been tested by at least one of the following methods:

Integration test

Side effects

No side effects

Related changes

No related changes

To reviewers

Please follow these principles to check this pull requests:

Concentration. One pull request should only do one thing. No matter how small it is, the change does exactly one thing and gets it right. Don't mix other changes into it.
Tests. A pull request should be test covered, whether the tests are unit tests, integration tests, or end-to-end tests. Tests should be sufficient, correct and don't slow down the CI pipeline largely.
Functionality. The pull request should implement what the author intends to do and fit well in the existing code base, resolve a real problem for TiDB/TiKV users. To get the author's intention and the pull request design, you could follow the discussions in the corresponding GitHub issue or internal.tidb.io topic.
Style. Code in the pull request should follow common programming style. (For Go, we have style checkers in CI). However, sometimes the existing code is inconsistent with the style guide, you should maintain consistency with the existing code or file a new issue to fix the existing code style first.
Documentation. If a pull request changes how users build, test, interact with, or release code, you must check whether it also updates the related documentation such as READMEs and any generated reference docs. Similarly, if a pull request deletes or deprecates code, you must check whether or not the corresponding documentation should also be deleted.
Performance. If you find the pull request may affect performance, you could ask the author to provide a benchmark result.

(The above text mainly refers to TiDB Development Guide. It's also highly recommended to read about Writing code review comments)

/test ?

/test pull-integration-kafka-test

/test pull-integration-kafka-test

/test pull-integration-kafka-test

/test pull-integration-test

/test pull-integration-kafka-test

/test pull-integration-test

/test pull-integration-test

/test pull-integration-kafka-test

/test pull-integration-kafka-test

/test pull-integration-kafka-test

/test pull-integration-test

/test pull-integration-kafka-test

/test pull-unit-test

/test pull-unit-test

/test pull-unit-test

/test pull-unit-test


is there a method to get all the regions in the subscription ?

Hi
is there a method to get all the regions in the subscription ?
Thanks

Hi @madhugilla
Using the azure-mgmt-resource package, subscription sub-client you have list_locations method.
I don't have a sample, but the unittests is here if it helps.
Thank you

I close this issue, I think I answered correctly to the initial question. Feel free to comment to re-open it if necessary, or open a new issue if you have any new problems.
Thank you,

Appreciate your help !!.


feat: make a list of important hints for our precompiles and implement them in cairo-rs (cairo vm in rust)

Feature Request
Describe the Feature Request
Current: Starkware VM has access to python runtime -> hints can be arbitrary python (just need to whitelist them to get to testnet)
Soon in Mainnet -> full migration to rust infra: Lambdaclass Rust VM does not have access to python runtime -> hints have to be hardcoded in rust and matched on string (and cairo1 hints are matched as well, but idk how, investigate)
references:

https://github.com/lambdaclass/cairo-vm/blob/main/vm/src/hint_processor/builtin_hint_processor/hint_code.rs#L1408
https://github.com/lambdaclass/cairo-vm/blob/main/vm/src/hint_processor/builtin_hint_processor/builtin_hint_processor_definition.rs#L812
https://github.com/lambdaclass/cairo-vm/blob/main/vm/src/hint_processor/builtin_hint_processor/skip_next_instruction.rs#L8C4-L8C4

Plan:

For Kakarot cairo0 codebase -> we may need to add manually the hints used in our precompiles in cairo-vm code (colocate them with the above example)
For Kakarot cairo1 codebase -> we'll need to add the hints are libfuncs of cairo1

I believe that implementing hints directly in cairo-vm for the cairo0 codebase is overkill, as we're going to migrate soon.
We need to create a strategy for each precompile for the cairo1 migration:

implement it in cairo1
implement it as libfunc
ask SW to implement it as core libfunc

Closing in favor of #856


Multiple clients from localhost

Hello.  OK, this is not really an issue, it's more of a brain fart that I am having.  So I have this WSS server up for over a year now and it is working great but now I need to make it where the local host runs a few scripts on it as well.  The problem is sometimes multiple scripts will start at the same time and because they both have client IP of <IP_ADDRESS> the script that runs first will usually receive the answer for both scripts. These scripts send queries to different cameras and need their own distinct responses.
Is there a way I can have the server distinguish between the two requests other than using the IP address?  I have been using the IP of the clients as the user_id and has worked great as they have always been different, but now that these local scripts all have an IP of <IP_ADDRESS> there could be a race issue.  I know I can delay the running of concurrent scripts, and if I have to I will, but hoping not.
Is IP the only way to distinguish the different clients or can it also use a remote port or something else?
Am using something like the following in my server (user_id is the IP of client):
await connected_clients[user_id].send(message)
Thanks for any help and for the great software..

remote_address gives you the IP address & port.
In general, I would recommend that you give connections unique ID. Perhaps you can leverage connection.id?
If you need to share the ID between the client and the server, then there's many way to do it. For example, clients could connect to ws://<your-server>/<a-unique-id>, then the server would extract the unique id from connection.request.path and use it as the key for identifying clients. You can also send an initial message on the connection to exchange an ID, one way or the other. (NB: I don't have details of your architecture and use case; I don't know your constraints; maybe that doesn't make sense.)

I had been using an 'initt' initial message so I just added the differentiating element as an ID instead of using the IP.  Worked like a charm.  Minimal revisions to some things and DONE.
Thanks for your help and rapid response.   :)


[engsys] upgrade dev dependency puppeteer to ^19.2.2

Issue #22371

API change check
API changes are not detected in this pull request.

core-lro browser unit test failure.  Likely different error or error message for broken json
    LRO Sad scenarios
      ? should handle put200InvalidJson
        Chrome Headless 108.0.5351.0 (Linux x86_64)
      AssertionError: expected 'Expected \',\' or \'}\' after propert?' to match /Unexpected end of JSON input/
          at assertError (test/utils/utils.ts:80:14 <- dist-test/index.browser.js:26568:28)
          at async Context.<anonymous> (test/lro.spec.ts:1920:17 <- dist-test/index.browser.js:29739:21)

      ? should handle putAsyncRelativeRetryInvalidJsonPolling
        Chrome Headless 108.0.5351.0 (Linux x86_64)
      AssertionError: expected 'Expected \',\' or \'}\' after propert?' to match /Unexpected end of JSON input/
          at assertError (test/utils/utils.ts:80:14 <- dist-test/index.browser.js:26568:28)
          at async Context.<anonymous> (test/lro.spec.ts:1965:17 <- dist-test/index.browser.js:29779:21)

Probably latest browser made improvements to JSON.parse?

I see, I think it makes sense to update the expectation to either drop checking for the error message or to do conditional checking.

I see, I think it makes sense to update the expectation to either drop checking for the error message or to do conditional checking.

I changed them to just verify the error name.


⚠ Bofrost.de has degraded performance

In ac6ea4d, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1262 ms

Resolved: Bofrost.de performance has improved in 030afe8 after 38 minutes.


MWPW-158014 - [Notification] bugs

As an author, I would like to be able to have the current Aside notification functionalities to be available in the new Notification block.
Bugs:
1 & 2 the notification.ribbon block (when nested in section with .sticky-bottom style) is not using the same default functionality as the original aside.promobar block that it was created to replace.

The notification ribbon block should hide when the user scrolls to the bottom of the page.
The notification ribbon block should only show up once the user scrolls to the element position in the document.
The multi view port decorator wasn't setup to differentiate between a two column layout and a three column layout.

Solutions:

Adjusted selector in the section for sticky functionality to query for all notifications where the class list does not contain the no-hide class name.
Same as 1.
Adjusted multi viewport decorator to consider number of columns in layout and subsequently use proper viewport logic in setting child content.

Resolves: MWPW-158014
Test URLs Bug 1 and 2:

Before: https://main--milo--adobecom.hlx.page/drafts/klee/test-aside-notification-sticky-bottom?martech=off
After: https://sartxi-mwpw-158014-notifies--milo--adobecom.hlx.page/drafts/klee/test-aside-notification-sticky-bottom?martech=off

Test URLs Bug 1 and 2:

Before: https://main--milo--adobecom.hlx.page/drafts/sarchibeque/test-sticky-notification-nohide?martech=off
After: https://sartxi-mwpw-158014-notifies--milo--adobecom.hlx.page/drafts/sarchibeque/test-sticky-notification-nohide?martech=off

Test URLs Bug 3:

Before: https://main--milo--adobecom.hlx.page/drafts/klee/test-notification?martech=off
After: https://sartxi-mwpw-158014-notifies--milo--adobecom.hlx.page/drafts/klee/test-notification?martech=off

@NadiiaSokolova I have added some testing URLs for the no-hide variant functionality for your review. Thanks!


Move from nix-shell to flakes

I think this is done?
@domenkozar, @DavHau: am i missing something?


Missing values in iobrpker logfile

Describe the bug
I get error message in ioBroker Log
To Reproduce
see logfile
Expected behavior
Read all data from meter
Screenshots & Logfiles
iobroker.2021-03-29.log
Versions:

Adapter version: 0.5.4
JS-Controller version: 3.2.16
Node version: 14.15.4
Operating system: Ubuntu 20.04

Additional context
Got new Discovergy meter right these days

I've had the same problem.
This is fixed in version 0.5.7 (2021-03-19), which isn't in stable right now.
You can install it manually or wait until @DutchmanNL pushes it to stable (i couldn't find any problems).

0.5.7 is indeed in latest for 12 days al will be published soon to stable.
I see that 17% of all user already have 0.5.7 (beta) and I don't get issue in sentry or additional issue on git so looks good,  give it time.  to the weekend. Will keep the issue open until it's published

0.5.7
53
17.32%

0.5.6
3
0.98%

0.5.4
149
48.69%

0.5.0
89
29.08%

0.4.9
4
1.31%

0.4.0
8
2.61%

https://github.com/ioBroker/ioBroker.repositories/pull/1443


The statements below present in loc_scan_missing_dirs.md  do not generate corresponding partition cleanup commands in mysql_missing_parts.sql

Partitions with special character '|' don't parse the correct partition details.

fixed.


React 16

Is React 16 fully supported? If so can we add the non-alpha version as a peerDependency? Thanks!

No need to change the peer dependency. The current one (^15.3.0 || ^16.0.0-alpha) is inclusive of all alpha, beta, RC, and final versions of React.


Incorrect Closing Time

Hi - browsed through the existing issues before creating this. The closing time seems to display a date in the past (see screenshot). Not sure what the date is meant to indicate but I would have expected the current date + time that the exchange would be closed.

There's no problem here. The market was deployed on September 11, 2017 and was set to expire a year from that date.


Add Maintainers

It seems like this project is more or less dead. There are plenty of issue and PRs (aka interest).
Can @yudai add a few maintainers to help move this project forward?

@jasondeutsch Any chance to take on the job to fork this project?

Here is a request for everyone willing to take over, fork  and maintain this (very usefull) project.
If you're gonna do so, make sure you have well kept "Releases Page" providing prebuild up to date binaries for different architectures (Linux, win, x64, arm etc)
Especially the Raspberry Pi, where it runs fabously.
This is such a usefull tool, and it would help spread it's use.

But not so in tiny devices with small memory and cpu.
Nowadays there's plentiful of free option for automating the builds and releases for Go
projects. Github Actions is a also a wonderfull tool. So there's no excuse for not providing
https://github.com/yudai/gotty/issues/291
https://github.com/yudai/gotty/issues/258

is there an up to date fork that I can use? https://github.com/yudai/gotty/issues/301

It's been some time since I opened this issue. At this point, I'm happy to fork and get others involved with getting issues resolved, prs reviewed and merged, etc...
I'm currently dealing with the winter storm pandemonium in Austin but I'll leave it to the replies/emoji's to tell me if there is enough interest in the idea above.

I've forked it here
Currently dealing with the winter storm/power issues in Austin.  Hope to begin merging some of the open PRs.
Maintainers welcome.

I'm sorry, I didn't see your comment here, so I've also forked it: https://github.com/sorenisanerd/gotty
I've merged some of the PR's from here and will go through the rest of them shortly and will also start addressing issues filed here.

What about creating a "gotty" organization (or something like this as gotty is picked) and grouping with sibling projects like gotty-client?


Implement evaluation of output Gibbs state probabilities on various backends.

The goal of this issue is to sample the probability distribution encoded in a Gibbs state. Sampling depends on the backend because for certain cases we only get counts that need to be normalized.

https://github.com/Qiskit/qiskit-terra/issues/7480#issuecomment-1632068350


Azure storage dev upload functionality

Co-authored by @shubha-rajan
Description
Begin work on file upload for Azure storage.
This PR

creates an Azure storage client for prod, dev (Azurite) and unit testing.
abstracts away the logic of file uploads from hardcoded AWS to an interface that can use AWS or Azure, depending on application.conf setting

universal-application-tool-0.0.1/app/modules/CloudStorageModule.java contains the logic for determining which storage client to use

enables the dev file upload view to use Azure by adding a Typescript file to handle the file upload logic.

To test, set STORAGE_SERVICE_NAME environment variable to azure-blob, then run bin/run-dev --azure.

TODOs (not implemented in this PR)

integrate azure blob storage for ApplicantProgramBlockEditView and ApplicantProgramBlockEditController
Make CORS rules more secure
Use the user-set file names instead of UUIDs (we're setting the filename on the backend because the filename is needed to generate a blob for a single blob, and giving container-level access is a bad idea). Maybe look into Directory Scoped Access
Configure Azurite emulator to use HTTPS

Checklist

[X] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
Towards #1686

Great work! A few additional overall thoughts:

Please add a browser test that exercises the Azure upload logic with Azurite
The PR description mentions "Make CORS rules more secure", where is that done?
Some additional whitespace throughout would make the code a bit easier for my eyes at least. In particular empty lines separating multi-line statements (e.g. conditional blocks, loops, try/catch blocks etc) from code at the same level of indentation
For future reference, please break the work into smaller PRs. Examples of things that on their own should be a single PR:

adding a new dependency (Java or JS)
adding a new docker container to the dev environment compose
backend logic for a feature
front-end logic for a feature

Aha, thanks!
On Tue, Dec 21, 2021 at 11:07 AM Shubha Rajan @.***>
wrote:

@.**** commented on this pull request.
In
universal-application-tool-0.0.1/app/services/cloud/azure/BlobStorageUploadRequest.java
https://github.com/seattle-uat/civiform/pull/1752#discussion_r773379329:

@Override

public abstract String serviceName();

public static Builder builder() {
return new AutoValue_BlobStorageUploadRequest.Builder()

   .setServiceName(StorageServiceName.AZURE_BLOB.getString());

}

@AutoValue.Builder
public abstract static class Builder {

/**

* Get file name for the blob. This is used to build the canonicalized resource part of the

* signature string.

*/

abstract String fileName();

For some reason, sometimes github doesn't show the latest version by
default. you'll have to go to this menu on the top left and choose "show
all changes"
[image: image]
https://user-images.githubusercontent.com/19631367/146984473-78047042-fc89-4b3d-a46b-75c0c26bad04.png
—
Reply to this email directly, view it on GitHub
https://github.com/seattle-uat/civiform/pull/1752#discussion_r773379329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADTUR4MX2FNTUZBHW3NK33USDF67ANCNFSM5KCJRSLA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because your review was requested.Message ID:
@.***>

Wrt: https://github.com/seattle-uat/civiform/pull/1752#issuecomment-995326391
Instead of writing a new browser test, I updated the browser test environment so that it can run either in the aws or azure dev environment. Currently, the only browser test that passes when run with Azurite is the dev_file_upload.test.ts test (as expected). You can run this by starting the browser test env with bin/run-browser-test-env --azure and then in a separate terminal running bin/run-browser-tests dev_file_upload.test.ts. Let me know if this is different from what you had in mind.


Add support for running periodic tasks

Implement the ability to specify tasks in the rorolite.yml file and translate that into a cron file on every deploy.

What is the format of cron specification?  I mean will it be in the standard cron syntax or natual language?

It'll be specified in a natural language in rorolite.yml. For example:
tasks:
    - name: retrain
      command: python train.py
      when: "everyday at 10:00 AM"

The rorolite deploy command should translate that into appropriate crontab file.

@anandology  i created a pull request for this issue

@anandology  i created a pull request for this issue


Add a test for CPB

keys.robot is included in the pcengines.robot file because keys are actually used there.

Closing, source branch is not OK.


Ability to Enable/Disable Packagecloud.io OS Repo

ZenDesk  3731
As a customer doing monthly semi-automatic updates of our OS packages, I would like the option to disable the configuration of the packagecloud.io repos so that an upgrade of the reset of the packages on the OS does not include the chef server software.

@sean-horn Is this taken care of with your most recent PR related to the repo being enabled? https://github.com/chef/chef-server/pull/374

@stevendanna Yes, I'll close it out.
Fixed in #374


Codeunit 7301 Whse. Jnl.-Register Line Change Parameter OnInitWhseEntryCopyFromWhseJnlLine

Object: Codeunit 7301 Whse. Jnl.-Register Line
Publisher Event: OnInitWhseEntryCopyFromWhseJnlLine
Param: WarehouseJournalLine
Could you please pass the parameter WarehouseJournalLine as Var ?

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Is it possible to set returnTo?

Is it possible to set returnTo parametr via $_GET or something like that?
I have url /saml/login. I want to redirect always from page, where I click on login button. I would like to set something like /saml/login?returnTo=/account/me and after login (or logout) get to page /account/me etc.

There is a workaround you could to on your side.
When user clicks /saml/login you can save to session their current route.
You can then extend the DefaultAuthenticationSuccessHandler if you don't have your own.
Then load the route from session and return a RedirectResponse

@jirinapravnik hi!
If your /account/me page there is behind the firewall that use this bundle, you'll be redirected for it automatically after successful log in.
\Symfony\Component\Security\Http\Firewall\ExceptionListener persist the request URI at the moment when redirects you for login and then we pass this URI to the SAML server, which returns it in RelayState (if not, this URI will still be used by SamlAuthenticationSuccessHandler).

I had the same sort of requirement. My application allows anonymous browsing of many pages and provides extra functionality (bookmarking items) when a user is logged in. I wish to provide login links on each page that bring the user back to the previous page they were looking at after login.
I was able to do this by adding a controller to my application that creates a session and then executes the bundle's login controller:
<?php

namespace App\Controller;

use Nbgrp\OneloginSamlBundle\Controller\Login as NbgrpLogin;
use OneLogin\Saml2\Auth;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Routing\Annotation\Route;
use Symfony\Component\Security\Http\FirewallMapInterface;

class Login extends AbstractController
{
    public function __construct(
        private readonly FirewallMapInterface $firewallMap,
    ) {}

    #[Route('/login', name: 'login')]
    public function __invoke(Request $request, Auth $auth): RedirectResponse
    {
        if ($request->get('returnTo')) {
            $returnTo = $request->get('returnTo');
            // Ensure that returnTo is the same as our host.
            // This could be more thorough checking to ensure that the URL
            // is below the application's base URL or take into account
            // other routing details, but this seems like a basic sanity check.
            if ($request->getHttpHost() != parse_url($returnTo, PHP_URL_HOST)) {
                throw new \InvalidArgumentException("returnTo must match the current host");
            }
            // Set the session variable that the Nbgrp/OneloginSamlBundle is
            // looking for.
            $firewallName = $this->firewallMap->getFirewallConfig($request)?->getName();
            if (!$firewallName) {
                throw new ServiceUnavailableHttpException(message: 'Unknown firewall.');
            }
            $request->getSession()->set('_security.'.$firewallName.'.target_path', $returnTo);
        }
        // Execute the Nbgrp/OneloginSamlBundle's login action.
        $nbgrpLogin = new NbgrpLogin($this->firewallMap);
        return $nbgrpLogin($request, $auth);
    }

}

I can then put links in my template like the following:
{% if is_granted('IS_AUTHENTICATED_FULLY') %}

Bookmark this item

    ...
{% else %}
    Log-in to save
{% endif %}

This technique has the desired effect of redirecting the user back to their previous page after login.
The one downside to this technique (and reliance on the session generally) is that if a user follows a login link but fails the authentication process and returns to the application, they will have a session cookie set which will prevent them from utilizing cached responses for anonymous visitors as stored in a reverse-proxy cache.
Maybe there are alternative ways of passing a return-URL into the RelayState that gets sent to the IdP....


com.github.tomakehurst.wiremock.verification.Diff.formatIfJsonOrXml bug

If you try to e.g. verifyThat with equalToXml and there is a non xml request, the method blows up:
org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 1; Content is not allowed in prolog.

	at com.sun.org.apache.xerces.internal.parsers.DOMParser.parse(DOMParser.java:257)
	at com.sun.org.apache.xerces.internal.jaxp.DocumentBuilderImpl.parse(DocumentBuilderImpl.java:339)
	at com.github.tomakehurst.wiremock.common.Xml.read(Xml.java:60)
	at com.github.tomakehurst.wiremock.common.Xml.prettyPrint(Xml.java:41)
	at com.github.tomakehurst.wiremock.verification.Diff.formatIfJsonOrXml(Diff.java:128)
	at com.github.tomakehurst.wiremock.verification.Diff.toString(Diff.java:112)
	at com.github.tomakehurst.wiremock.client.VerificationException.<init>(VerificationException.java:64)
	at com.github.tomakehurst.wiremock.client.VerificationException.forUnmatchedRequestPattern(VerificationException.java:39)
	at com.github.tomakehurst.wiremock.client.WireMock.verifyThat(WireMock.java:353)

Thanks. Replicated.

If you have the time, I'd appreciate this fix, it is blocking some tests I am writing :)

Thanks

I've pushed a release. It usually takes a while to sync to M2, but you can grab it from oss.sonatype.org if you need it immediately.


Bump request, express, pouchdb, express-pouchdb, and memdown

Fixes #506
[delivers #137651811]
Signed-off-by: Hristo Iliev<EMAIL_ADDRESS>

Coverage decreased (-3.6%) to 89.584% when pulling 195f19b48acf9961bd15d0ed02fb34d6352b1033 on SAP:bump-request into 93a49c76c30f38ba1243a003a6321fa4383df546 on cloudfoundry-incubator:master.

Coverage increased (+0.08%) to 93.299% when pulling eb758d140cf544d75159bb8234a8ea505685f8a3 on SAP:bump-request into 93a49c76c30f38ba1243a003a6321fa4383df546 on cloudfoundry-incubator:master.


feat: Add Option to Suppress Pre- and Post-Retrieve Events

What does this PR do?

Adds a property to ComponentSet RetrieveOptions to suppress the pre- and post- lifecycle events that are emitted when the ComponentSet's retrieve operation runs.

What issues does this PR fix or reference?

In the VSCE repo, work is underway to add source-tracking (STL) into the deploy* and retrieve* commands so that they can be used against scratch orgs (#4597).

In order to complete that work, two capabilities are needed:

VSCE needs to retrieve a file to a tmp location outside of the project

(Currently STL complains about the path used by os.tmpdir)

VSCE needs to be able to retrieve a file that has known conflicts after a deploy attempt that failed because of those conflicts (for line-by-line file diffing)

@W-12551198@
Functionality Before

No way to retrieve a ComponentSet without emitting pre- and post- retrieve events.

So STL is always being notified and reacting to the events

Functionality After

Using the suppressEvents option, a retrieve call can be made without emitting the events, therefore bypassing STL behaviors

Alternatives Considered:

Have STL not track files that are retrieved to a location outside of the project

Don't really want to make changes that try to tell STL what to track/not track

Turn off listening for the events in STL temporarily

VSCE uses long running processes, would have to remember to re-enable listener, could get into trouble in cases of exceptions

@klewis-sfdc looks good to me.  Can you have someone from VSCode QA and merge since y'all have a use case for it and we don't?

Yep, we can do QA and merge.  Thanks, Shane.


Installation / Runtime Instructions

For those of us who aren't python heads, would it be possible to include a snippet of how to install and run this package? Thanks for building it!

Install anaconda mini https://docs.conda.io/en/latest/miniconda.html and brew install rust

Ok I ran brew install rust and then went to the link and found this page: https://docs.conda.io/en/latest/miniconda.html
I downloaded the pkg file for ARM and install it.
Now what?


question: about global content training images

Thank you very much for your work. I have a question: local training images are easier to understand and create. However, what is the relationship between the images of the content trained globally and the corresponding images? How do you select the training images? I am using my own industry-specific data for training. Thank you.

Global conditions are the clip embeddings extracted by the ContentDetector in /annotator/content/. That means during training, the input global condition is the clip embedding, and the output (or ground truth) should be the corresponding original image.

Thank you very much for your reply. Am I understanding this right? The data (*.npy) of the content comes from the feature data extracted from the pictures in the images directory.

Yes, you are correct.

thanks very much!

Hi, good work!
I want to confirm whether the global conditions' clip embeddings are generated by raw rgb images or single style condition images(i.e. depth/pose/etc)?


number_of_squeaks_with_address_in_block_range db query should only count unliked squeaks

This function should use a WHERE filter so that it only counts squeaks that are not liked:
https://github.com/yzernik/squeaknode/blob/1fb979ef6a257f18f6f854e3e4cee58d298e452a/squeaknode/db/squeak_db.py#L666-L686

Probably not going to do this.


MSVC tool doesn't work

I compiled the incbin.c file and made an executable if I run it with help I get:
 ./a.exe phong_lighting.glsl  -I../../Src/Engine/EmbeddedShaders/ -o ../../buildvs/data.c  -help
C:\Users\Makogan\Documents\neverengine\libraries\incbin\a.exe [-help] [-Ipath...] | <sourcefiles> | [-o output] | [-p prefix]
   -o         - output file [default is "data.c"]
   -p         - specify a prefix for symbol names (default is "g")
   -S<style>  - specify a style for symbol generation (default is "camelcase")
   -I<path>   - specify an include path for the tool to use
   -help      - this
example:
   C:\Users\Makogan\Documents\neverengine\libraries\incbin\a.exe source.c other_source.cpp -o file.c
styles (for -S):
   camelcase
   snakecase

If I try to run it normally (without the help flag) this is the data.c file that gets generated:
/* File automatically generated by incbin */
#include "incbin.h"

#ifdef __cplusplus
extern "C" {
#endif

#ifdef __cplusplus
}
#endif

No errors or warnings are thrown by the tool.
I am on windows 10

You need to list the source files which use INCBIN you want preprocessed by this tool, you don't list the data files, those are read from the INCBIN macro in the source file which are found via -I. So for instance you'd have:
// source.c
INCBIN(data, "phong_lighting.glsl");

Then you'd run the tool with ./incbin.exe source.c -I../../Src/Engine/EmbeddedShaders/ -o ../../buildvs/data.c
Which would emit a data.c file containing the phong shader source code as a byte array named data, now in source.c you can use data to refer to that embedded GLSL.
If you have multiple source files which use INCBIN you'd specify them all on the command line here.

Ah I see, I was confused, thank you!

I was confused too. I think the description of how to use it should be clearer. I didn't understand that source.c was suppossed to be the source file that contains the INCBIN macros.
Maybe you should add a little counter so it's easier to understand that nothing was found?
diff --git a/incbin.c b/incbin.c
index fd1e72b..ca6abe4 100644
--- a/incbin.c
+++ b/incbin.c
@@ -104,7 +104,7 @@ static const char *styled(int style, int ident) {
 }

 int main(int argc, char **argv) {
-    int ret = 0, i, paths, files = 0, style = kCamel;
+    int ret = 0, i, paths, files = 0, style = kCamel, written = 0;
     char outfile[FILENAME_MAX] = "data.c";
     char search_paths[SEARCH_PATHS_MAX][PATH_MAX];
     char prefix[FILENAME_MAX] = "g";
@@ -263,6 +263,7 @@ usage:
                     fprintf(out, i != size - 1 ? "0x%02X, " : "0x%02X", data[i]);
                     count++;
                 }
+                written++;
                 free(data);
                 fclose(f);
             }
@@ -282,6 +283,7 @@ end:
         fprintf(out, "#endif\n");
         fclose(out);
         printf("generated `%s'\n", outfile);
+        printf("number of files included: %d\n", written);
         return 0;
     }


lapack: add Dlatdf

Please take a look. Part of #1651.
@vladimir-ch  There is something real fishy here. Like you said:

This says that the slice rhs given to Dlaswp is too short. Based on the call above, rhs should be an (n-1)x1 matrix (that is, a column vector) with the stride ldz.

Though this is not the case! If rhs has stride ldz then this routine operates on non-acessible data. So rhs probably has stride 1 though I can't for the life of me figure out how those Dlaswp calls work.

@vladimir-ch Thank you for taking the time to explain what was off in the implementation. I have applied your suggestions to the code.
Please take a look at the first error check lines of Dlatdf, I have added a few new checks.
I have not completed the tests as I am having a hard time replicating the conditions given in Dtgsy2 (I suspect, not sure about this either). What about testing this routine via Dtgsy2 (which I've already implemented)?

Sounds good to me. Here's the PR.


[minor] unnecessary steps during uninstall

it seems helm uninstall already clean up those 2 steps.

It is currently so for the purpose of clarification. There should eventually be a script for safe uninstallation.


Text placement slightly too high

Here's a screenshot of BitBar next to two text-oriented menu bar apps (ItsyCal and Next Meeting).

I think the BitBar text is slightly too high?

Dupe https://github.com/matryer/bitbar/issues/268 & #105
Solved in #513


Error when quitting a project

After confirming the quit dialog, the following appears in the command line.
freed code id 4
freed code id 5
freed code id 2
ERROR: free: Method/Function Failed.
   At: servers/visual/visual_server_raster.cpp:4252.
ERROR: free: Method/Function Failed.
   At: servers/visual/visual_server_raster.cpp:4252.
freed code id 6
ERROR: free: Method/Function Failed.
   At: servers/visual/visual_server_raster.cpp:4252.
ERROR: free: Method/Function Failed.
   At: servers/visual/visual_server_raster.cpp:4252.
ERROR: set_stream_peer: Condition ' p_peer.is_null() ' is true.
   At: core/io/packet_peer.cpp:228.
base resources missing 2
ERROR: finish: Condition ' instance_dependency_map.size() ' is true.
   At: servers/visual/visual_server_raster.cpp:6891.
ERROR: cleanup: ObjectDB Instances still exist!
   At: core/object.cpp:1715.

Built and running on Arch Linux.

Thanks for the report. Some of those errors are still valid in the current master branch, while others are not and there are new ones. I'll close this issue and open new issues for each kind of error, so that they can be investigated.


Users are attempting to install via github clone instead of npm

Update main readme:

[ ]  move legacy instructions to README.legacy.md
[ ] indicate that you should install from npm to consume the sdk
[ ] Refer to the ciscospark README
[ ] provide links to getting started with npm

The README has been revamped since this issue was created/last updated.


Unauthorized

I did not allow any of these codes or any other apps to be added on my phone

I did not allow any of these codes or any other apps to be added on my phone

Don't use this tool then


improve coverage of Instance Composer (#5538)

Pull request opened by the merge tool on behalf of #5538

Merged in 89409f6095c48d6955390d48a6f0740b5b0d13fc


Add original TX Hash to message

Is your feature request related to a problem? Please describe.
I want to have an option in the UI to be able to see the "origin" txhash of the message.
Describe the solution you'd like
Something akin to a light version of email client "show original", showing the fields from rpc getTransactionByHash (or equiv) example tx.
The following would be added to the message headers object returned via the API. This aligns with the optional headers guidelines for RFC2822 header standards https://tools.ietf.org/html/rfc2822#section-3.6.8 ).
"transaction": {
  "blockHash": "0x94fa39de3950f87d9225d2146b19981e5824f271d1049348462cc2ee2f20f94f",
  "blockNumber": "0x81a753",
  "from": "0x4ad2b251246aafc2f3bdf3b690de3bf906622c51",
  "gas": "0x77c4",
  "gasPrice": "0x4a817c800",
  "hash": "0x2bf261a81e624d649450a3851df2d0639a8b98ed3bce39da14e8f318adf3edb9",
  "input": "0x6d61696c636861696e010a82012ee10c59024c836d7ca12470b5ac74673002127ddedadbc6fc4375a8c086b650060ede199f603a158bc7884a903eadf97a2dd0fbe69ac81c216830f94e56b847d924b51a7d8227c80714219e6821a51bc7cba922f291a47bdffe29e7c3f67ad908ff377bfcc0b603007ead4bfd87ff0acc272528ca03d6381e6d0e1e2c5dfd24d521",
  "nonce": "0x8",
  "to": "0x92d8f10248c6a3953cc3692a894655ad05d61efb",
  "transactionIndex": "0x40",
  "value": "0x0"
}

NOTE: The fields above map intersection fields from Geth & Parity getTransactionByHash calls.
Describe alternatives you've considered
Considered including the encrypted message object. This would be nice to have.
Additional context
Add any other context or screenshots about the feature request here.

Include block-number,  transaction-hash, block-number-encoding  and transaction-hash-encoding on the response for get messages.
This involves returning a struct instead of just the encrypted content in  // Receiver gets encrypted data from blockchain. type Receiver interface { 	Receive(ctx context.Context, network string, address []byte) ([]cipher.EncryptedContent, error) }
The struct would be something like ```
type Transaction struct {
Data []cipher.EncryptedContent
BlockNumber []byte
Hash []byte
}

Hi @robdefeo. This seems pretty straightforward, I believe I can have it done during next week.
What is not clear to me, though, is the meaning of block-number-encoding and transaction-hash-encoding.
Also I see that you propose []byte for the new fields, however the related txResult struct stores those field values as string. Do you mean that the []byte will contain both the original value and the encoding one?

Hi @nikosk686 That's excellent 😄
We are looking to include encoding-type into all of the http responses, as bytes need to be encoded by something usually hex but protocols such as substrate use a mixture of hex and base58 so we need to be explicit.  However the choice of encoding would happen when retuning the response in the API layer. The encoding consts are not yet merged to master (I hope to complete this in the next few days) https://github.com/mailchain/mailchain/blob/39991b28fbeab0b499314e246eef209b9dcbcde6/internal/encoding/consts.go#L31-L36
For the interface []byte is needed as it allows any value, the []byte contains only the value not the encoding.

@nikosk686 this changes have now been merge so you have access to them in the main branch. Hopefully this should unblock you.  This merge does mean that there are changes required in the UI to make every work and you will need to re-add your private keys if you are building from code. The work was largely done to the keystore to support different key types

Hi @robdefeo thanks for the update.
I will pull the changes and continue working on this.

@nikosk686 😄looking forward your pull request. In the meantime if there is anything I can do to help, just reach out.

Hi @robdefeo, I have pushed some changes related to this.
I am not sure I have understand the context completely, so
I' ve created a pr, though it is wip, for you to have a first look
so we can catch early any misunderstandings from my side.
Regarding the use of []byte in the interface, while I believe understand
the concept, in the specific case seems unreasonable as we would have to
cast from string at first and then to cast again back to string for the api
response, without any other modification taking place.
Probably if the Transaction struct gets moved somewhere more centrally
we can change it.
Also while I made changes in various files no tests failed, meaning that the
specific flow is untested. I will add tests needed for the final version of this pr.

Hi @nikosk686 glad you created the PR for this and we can work through it more iteratively. Your reasoning is correct in this case, however we are integrating new protocols and using a []byte removes this problem.  The other reason for requesting a []byte is that we will allow the front end to in some cases request the encoding type to use, making []byte the raw data that can be encoded as needed.
In the PR I will leave some feedback on the Transaction struct.
You are correct about the missing test for this part of the code base. This is something that certainly needs to be resolved. I will raise a separate issue if you prefer to keep this PR more tightly scoped.

Hi @robdefeo thanks for the comments, i ll have a look asap.
unless you need this merge to be done quickly I prefer to have tests done
in this pr so not to leave tech debt behind.

@nikosk686 very happy to wait and have you put the tests inplace. Putting tests in place will be a big benefit and make the PR more reliable process. Thank you!! 👍 👍

closed by #340


Update StorageDriver IPC subsystem for storage driver changes in docker/docker-registry#820

IPC Subsystem build has been disabled after PR docker/docker-registry#820. There are two issues that led to this:

The initial implementation of the new API methods caused the IPC system to hand on certain calls.
docker/docker-registry#814 needs to be fully worked out before the IPC implementation is updated.
Time type is not support by libchan (docker/libchan#75 or docker/libchan#79).

Once these are resolved, we can reintegrate the IPC support.
This issue supersedes docker/docker-registry#831

IPC has been disabled. Closing.


[Outlook] (preview) New Mac UI now supports 1.9

Docs Build status updates of commit 01ba86a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/compare-outlook-add-in-support-in-outlook-for-mac.md
:white_check_mark:Succeeded
View

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded
View

docs/reference/requirement-sets/outlook-api-requirement-sets.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Switching tabs method

Anyone knows a way to pragmatically switch which tab is displayed using JS?
I've been scratching my head on this and I can't get anything it to actually work.

Answered my own question. Here's how you can do it, considering your tabs look like this:


Remove celery integration

I'd like to remove celery integration from LFS in favour of using generic Django solutions for asynchronous emails like django-celery-email.
There is no need to have special code within LFS to support celery based emails. Also, current celery integration is broken. I just got

EncodeError
Can't pickle <type 'uwsgi._Input'>: attribute lookup uwsgi._Input failed

after enabling celery for LFS yesterday. This error is possibly due to the fact that whole request objects is passed as a parameter to celery task in LFS.
The way we have current celery integration makes it impossible to use celery in the project without using LFS celery integration - LFS just checks if celery module is available.
Moreover, current LFS documentation for celery is outdated. Celery has changed a lot since these docs were written. Again, I see no point in maintaining these docs within LFS, unless in very simple way (as it's done in this pull request).

Any thoughts on this?


Include of non-modular header inside framework module #2

The proposed solution for #143 breaks my other pods.
The right fix is probably http://stackoverflow.com/a/28552525/1435156:

the location of the imports needs to be changed by the library owner

will do

@btoueg, #143 also breaks my other pods but the "right fix" doesn't fix it for me. What needs to be done beyond changing the boolean? I'm not clear on the SO post as I'm not an iOS person.

@ssomnoremac Reverting that https://github.com/evollu/react-native-fcm/commit/bb17da237f9caf737bbda0c9fba4391f0ee0cf51 fixes the issue
@evollu I don't understand what I am losing by commenting this line
My pod:
  pod 'Firebase/Messaging', '~> 3.9.0'
  pod 'Firebase/Core', '~> 3.9.0'
  pod 'Firebase/Crash', '~> 3.9.0'

I have done a pod update Firebase. Since then i have four "Include of non-modular header inside framework module." and FIRNmessaging.h not found.
I try use_framework in the podfile but it's broking all my other pods.
I try "the right fix", it doesn't work for me as well.
This is blocking for us for now we can't upgrade firesbase anymore...

I'm moving the module import into .m file and hope it will fix your issue
try to use branch fix-194
note that you need to add
@import Firebase in your appDelegate

Still the same for me, the same error.

did you remove use_framework?

yes because it's broking my other pod

do you have any public repo that I can debug with? I'm not using cocoapod in my project

seems like use_frameworks! is necessary for swift projects. So is it possible for you to change other pods to support frameworks?

i build an exemple  :
https://github.com/MatthieuPeyrot/Firetest.git
i'm not using swift.

@evollu If you are still looking for an example of a repo this breaks, see this : https://github.com/transistorsoft/react-native-background-geolocation/issues/174

Here is another relevant thread: https://github.com/ParsePlatform/ParseFacebookUtils-iOS/issues/2

try setting this to yes:
CLANG_ALLOW_NON_MODULAR_INCLUDES_IN_FRAMEWORK_MODULES = YES

in the RNFIRMessaging.xcodeproj build settings

it's working when we set yes the CLANG_ALLOW_NON_MODULAR_INCLUDES_IN_FRAMEWORK_MODULES
in react-native-fcm/ios/RNFIRMessaging.xcodeproj

@MatthieuPeyrot so adding this flag solves the issue? are you using the master code or fix-194 branch?

yes on the master code , i just set yes to the flag , and it's work.

Setting the flag works for me too. Context: I'm using React Native 0.36.0, with react-native-firebase-analytics 0.36.0.

Also, @evollu, you want to update the README.md because the Firebase settings needs to be stored into ios/GoogleService-Info.plist, as oppose to ios/googleServices-info.plist.

Latest code has been switched to use header file so you don't need to set the flag anymore.
However that has a dependency of 0.40 react native

Closing this as it is solved in latest code

Looking at https://github.com/evollu/react-native-fcm/blob/master/ios/RNFIRMessaging.xcodeproj/project.pbxproj#L211 it looks like it's been resolved by flagging Allow non-modular imports to true rather than fixing header files
Issue is fixed anyway, thanks !


Not logging in correctly

Prerequisite

[X] Yes

Accounts

[X] Yes

Country Code
1
Region Code
AZ (West US)
Logs
"dev_lang": "javascript",
  "dev_channel": "homebridge",
  "devVersion": "1.7.0-beta.53"
}
body = {
  "country_code": 1,
  "username"<EMAIL_ADDRESS>  "password": "88fa5d40b7c1fb740b87c2bda9ca50bd",
  "schema": "smartlife"
}
[1/14/2024, 1:39:10 PM] Homebridge v1.7.0 (HAP v0.11.1) (Tuya) is running on port 47906.
[1/14/2024, 1:39:10 PM] [Tuya] [TuyaOpenAPI] Response:
path = /v1.0/iot-01/associated-users/actions/authorized-login
data = {
  "code": 1106,
  "msg": "permission deny",
  "success": false,
  "t":<PHONE_NUMBER>483,
  "tid": "34efeaa5b30c11eeaf0a66078db13d13"
}
[1/14/2024, 1:39:10 PM] [Tuya] Login failed. code=1106, msg=permission deny
[1/14/2024, 1:39:10 PM] [Tuya] Please make sure your countryCode, username, password, appSchema is correct, and app account is linked with cloud project.

JSON:
"name": "Tuya",
    "options": {
        "projectType": "2",
        "accessId": "5nwxwy4j7kp5fgsspw4j",
        "accessKey": "c69a98884a29430fb72cc1b6452a5a22",
        "countryCode": 1,
        "username"<EMAIL_ADDRESS>        "password": "ABCDEF",
        "appSchema": "smartlife",
        "debug": true
    },
    "_bridge": {
        "username": "0E:09:8E:F2:D1:3D",
        "port": 47906
    },
    "platform": "TuyaPlatform"

Other Infomations
The password being picked up is the device code not the password - see the json

The password is salted so it won't match your plaintext password that you define in the config.


Target the parent of an element using jQuery - red background

Challenge http://www.freecodecamp.com/challenges/waypoint-target-the-parent-of-an-element-using-jquery has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
I've read the multiple issues classified as bugs where the child (#target1) wasn't identified in the code but the parent (#left-well) was.  I believe I have the correct code and the background of the well is appearing red but "Your left-well element should have a red background" is not checked as correct.  If I change the child to "#target3" my answer is accepted.

I can clear this up with the the PR I submit for some test changes in that file for single and double quotes as well I see there is a typo on that same challenge

@dev-diva actually your issue is 11th duplicate issue. See this comment for more info. I'll leave it open because @natac13's PR will automatically close it.

@natac13 I've just verified that your fix is working. Thank you for contributing!
P.S. A little advice from @benmcmahon100. For example, $(" #button ") will not pass your test because you force campers to use $("#button"). You can consider using whitespaces by adding \\s*? in your code for example:
editor.match(/\\$\\s*?\\(\\s*?\\'\\s*?button\\s*?\\'/g). Of course this will make your RegExp more complex but users won't be confused.


Make Sandcastle line markers more visible.

Some of the markers were very faint.

Looks good.  Thanks @emackey


Library audit

Can explain in more detail later, but a bit short on time to write a detailed description.
The shared library is getting to be quite large. And certain library files are either becoming to unfocused (e.g., the general library) and there are patterns of library functions that probably should be grouped together but aren't. In fact, one of the reasons I added the docs search a few weeks ago to the site was to be better able to find different library functions.
I think it's about time to do an audit of all the library functions and recategorize them. I imagine the majority of functions should stay in their current location, but I think having better organization overall would make things simpler to use.
To do this, I'd suggest we first hash out what the different files should be, then which functions need to be moved to which file. Then once we're in agreement, we should do the entire migration in one PR and quickly review/merge it. Since this migration will touch many if not most of the scripts, doing it over multiple PRs or taking a long time to review/merge the giant PR would cause many merge conflicts and more headaches. By making the actual changes swift, we can minimize the downsides of such a migration. This will require decent coordination on when the actual migration will take place to ensure we have one person to write the PR and another to merge it.
Thoughts?
@rpatters1 @ThistleSifter @cv-on-hub @CJGarciaMusic @jwink75

So many of the library function take a PDK Framework instance as the first argument. It seems to me that much of our library is crying out to be melded into FCX mixins for each of those classes.

I would agree with that.

much of our library is crying out to be melded into FCX mixins

This sounds true, but is code bloat a potential problem? I still don't understand the mechanics of mixin inclusion, but it seems that every sub-class is included automatically, and just using FCXLuaWindow adds around 9,000 lines of code to a script. A bunch of that is comment so there might be little performance hit, but could this magnify if more routines shift to mixin?

I also agree. The only concern I have is that our current bundling method involves including the whole mixin directory.
Haha, you beat me to it just as I was typing it.

I still don't understand the mechanics of mixin inclusion

To chime in on how things work. Essentially, if you write require("library.mixin"), all mixins will be bundled into the final file.
With static dependencies like require("library.clef"), it's really easy to analyze all import statements. However, once you add dynamic imports like below, things get drastically more complex.
local clef_library = "library.clef"
local clef = require(clef_library)

To do that properly, I'd have to keep track of every single reference to the clef_library variable, and all of its changes. Since this could potentially go through many layers of indirection (for instance, the mixin library), covering all the edge cases essentially requires running the code at bundle-time—something we're not able to do and something I'm not interested in writing.
The only reliable way ensuring the the dependency is included is to include all the mixins.
Now one could make some assumptions and look for the code mixin.FCM*** or mixin.FCX***, but that has similar issues that we ran into with the old bundler. One of the reasons for rewriting the bundler a month ago (aside from being able to support mixins) was to eliminate some of the edge cases the previous heuristics required.

A bunch of that is comment

The comments would be possible to remove in future versions of the bundler. This would likely need to be custom-built, though. I tried using a Lua minifier to solve this and to make the code smaller in general, but all the minifiers I found produced buggy code.

could this magnify if more routines shift to mixin?

It could, but I'm not too worried about that. Lua is a very fast interpreted language that gets fairly close to the speed of C++. To keep Lua fast, lines of code is really not an issue. What's far more important is making sure the time complexity of our algorithms stays low.
As far as I know, no one (aside from Jan Angermüller) has had real-time performance issues in JW Lua or RGP Lua. And Jan's performance issues were solved by creating a custom framework that reduces C++ interop with Lua and Finale PDK API calls, not necessarily by running less Lua.

When it comes to bundled code bloat, the more pressing concern I'd have is the the actual download size. The largest file is ~300kb of Lua code. In the grand scheme of things, that's not a lot. But it is starting to get large enough to make me nervous (I've written a lot of code for websites, and there every single kilobyte counts, so anything in double-digit KBs causes me concern). In reality, though, no one will care about that. Even if it gets to a few MBs, that shouldn't cause issues.
Regardless, limiting the file size is still likely a good idea. And this should probably be solved at the bundler-level, not necessarily the source code level. I am working on a side project to this side project that should help with this.

Something that will need to be considered if we convert (most of) the whole library to mixins is, is it possible to wrap an instance of, e.g., FCXNoteEntry from an existing instance of FCNoteEntry? A primary case where I could see a need is that the eachentry and eachentrysaved iterator functions feed plain FCNoteEntry. (Unless you want to take on overwriting those as well, which I would be loath to do.)

I think stripping out docblocks from library includes would go a long way to helping reduce file size. Maybe even something simple like this would work: str.replace(/--\[\[.*?\]\]/gm, '')
@rpatters1
Yes, that's already possible with mixin.subclass. Eg mixin.subclass(note_entry, "FCXNoteEntry")

FYI, I tried removing comments in #334, however after merging I noticed some scripts were broken so I rolled back the change in #336. Going to go to bed so I won't be able to look at this any further tonight.
In case anyone wants to explore this more before I can, here's the actually function I wrote to remove all the comments:
https://github.com/finale-lua/lua-scripts/blob/master/.github/actions/bundle/src/remove-comments.ts
And the test cases:
https://github.com/finale-lua/lua-scripts/blob/master/.github/actions/bundle/src/remove-comments.test.ts

Your first regex misses any multiline comment with a ] in it, and there are many of them. Eg:
    --[[
    % get_raw_finale_version

    Returns a raw Finale version from major, minor, and (optional) build parameters. For 32-bit Finale
    this is the internal major Finale version, not the year.

    @ major (number) Major Finale version
    @ minor (number) Minor Finale version
    @ [build] (number) zero if omitted
    : (number)
    ]]

Since static analysis isn't a viable option for the time being, what about runtime analysis? I could write something that would output the names of all included mixins after a script has finished running. That could then be used to insert bundler directives at the top of a script file which would instruct the bundler to only include the listed mixins.
For example:
-- #INCLUDE_MIXINS: __FCMBase FCMControl FCMCtrlEdit FCMCustomLuaWindow FCMCustomWindow FCMResourceWindow __FCMUserWindow __FCMCollection __FCMIteratorBase

I know there are a couple of class hierarchies that differ between Mac and Windows but that can be handled by the output function which would list both possibilities.
If a bundler directive is not present, the whole mixin library would be included, as is currently the case.

I would be concerned that you can't guarantee that every path of the script has been executed. That means you might not be able to guarantee that every dependency was loaded.

I understand that. It would be up to the author to ensure that they had run all of the paths. And if there were too many paths to consider or the author was not confident, they don't have to use it and the entire mixin library will be included.
I guess my thinking was that it might be better than nothing to have the option there.

Oh, I see. This would be a tool for developers. (I was imagining an automated process.) Even then there would still be some risk, but it might be useful. It does seem like a lot of clutter the way it is now.


Components without color on vue 3.

The quotation marks doesn't have color neither.
VSC

WebStorm

Thanks, this is a nice theme to use in vscode.

Yes, noticed this too. Sadly. Has anyone been able to fix it?

Have a same problem with props color mismatch


[Question] Create old version manifest file

I have created a manifest file to retrieve DeltaTable data from athena.
Is it possible to create a point-in-time manifest file to retrieve old version data without rollback the data?
Thank you

Currently we don't support this. Delta has a Presto connector which supports time travel: https://prestodb.io/docs/current/connector/deltalake.html Would you be able to push Athena to support it as well? A native support would be much better than the manifest files.

Thanks for your reply.
We will request it as a new feature through the AWS Support Case.
thank you

Cool. Closing this ticket. We will focus on improving the Presto native support rather than improving the manifest files.


Unnecessary Files

Untrack pycache/* and the unnecessary .idea/[...] files
There seems to be some weird tracking issues happening with Pycharm and I'm tired of dealing with it for now

Bleh still happening, gotta sit down and just wildcard the hell out of this stuff one day. Should also add other common config stuff (like vscode)

https://github.com/ClaytonTurner/power-hour-generator/pull/16


Exec tachikoma update<PHONE_NUMBER>3323

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

great


Test Coverage  For VDC API

Description
test-suite coverage for VDC API
generate api key

[ ]  list nodes (kubernates info), add node, delete node.
[ ] list zdbs, add_zdb, delete_zdb.
[ ] get wallet info.
[ ] list pools.
[ ] list alerts.
[ ] list all admins, add admin, remove admin.
[ ] create backup.
[ ] list backups.
[ ] delete backup.
[ ] restore backup.

for test VDC API you need to generate API Keys
in case we used selenum for generate API Keys we need to login by connect app

for test VDC API you need to generate API Keys
in case we used selenum for generate API Keys we need to login by connect app

No, You don't need selenium, you can generate a key using the APIKeyFactory
https://github.com/threefoldtech/js-sdk/blob/9eed06065e5f31040f35ff5696c6dc87785b3334/jumpscale/packages/vdc_dashboard/bottle/models.py


Allow natural=cave_entrance on nodes that are part of ways

At least some caves can be correctly tagged on nodes belonging to ways
For example see https://www.openstreetmap.org/changeset/90314885#map=19/50.20174/19.81875 - cave visited each (covidless) year by 90 000 people. And yes, there is footway leading to its entrance.
iD claims that it must be always on a standalone node.

.

@matkoniecz Thanks, sounds fine.


remove base stop attribute

I'm not sure if adding bluesky as a dependency is something that I actually want to do for this package...
I do want to pin the version of bluesky however, and I think this is the best way to do it

I guess I'm wrong, the changes haven't been released---keeping this open until upstream releases


Students page added and contributing.md updated.

Students page was added and the contributing.md page was updated.

Conflicts issue, please resolve

Conflicts issue, please resolve


Visual Studio does not hit breakpoints when debugging tests

When trying to run unit tests with Mocha set as the test framework, everything is fine. However, when I try to debug the tests, VS does not hit any of my breakpoints in the tests. It shows the white centered breakpoint markers with the exclamation marks, indicating, that the symbols could not be loaded. Furthermore the test I was trying to debug will fail, regardless of its actual result!
Could you please advise on this? Is this a config issue on my side?
Expected Behavior
When running a test in debug mode, VS should hit the first breakpoint
Actual Behavior
When running a test in debug mode, no breakpoints are hit, and the test will fail, regardless of its actual result

NTVS Version: 1.4.11027.3
Visual Studio Version: 15.6.1
Node.js Version: v9.5.0
Mocha Version: 5.0.4

Also experienced the same in VS 2015 (14.0.25431.01 Update 3) with NTVS 1.2.40726.00
Steps to Reproduce
I followed this guide: https://github.com/Microsoft/nodejstools/wiki/Test-Explorer

Create a new "Blank Node.js Console Application" project (any other type should do as well)
Install the Mocha npm package (locally)
Add a new "JavaScript Mocha UnitTest file" - the default contents will do (I tried putting the file in the root of the project, and in a "test" folder, but did not made any difference)
Make sure, the "TestFramework" property of the file is set to "Mocha"
Set a breakpoint on the first line (but any other will do)
Build the app
Open the test explorer, and run the first test ("Test 1") - it should pass
Debug the same test - the breakpoint will not be hit, and the test will fail

Debugging node 8+ is not supported on Visual Studio 2017 Update 6. This should be fixed in the near future, but I unfortunately don't have much influence on the release schedule.
Debugging should work with Node 6.

Do you mean only for tests? Because I can debug my code without any problem with the setup mentioned above. The only debugging issues I'm having is with the tests.

yes, only for tests... they have a different code path, because of the way the tests are started, vs. the way an app is started.


[FIX] Safe exit cuda

Fixes #395 with a first try/except on nvidia-smi command check output.

Codecov Report

Merging #396 (01b1140) into main (6b78211) will decrease coverage by 0.10%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #396      +/-   ##
==========================================
- Coverage   55.49%   55.38%   -0.11%
==========================================
  Files          42       42
  Lines        2528     2533       +5
  Branches      476      476
==========================================
  Hits         1403     1403
- Misses       1025     1030       +5
  Partials      100      100


Failed to save draft constantly pops up.

Describe the bug
The message:  "Failed to save draft. Ensure you have internet connection before saving this page" pops up all the time even though I have a good connection to the page.
Steps To Reproduce
Steps to reproduce the behavior:

Start editing a page.
A few seconds later it pops up and keeps popping up in regular intervals.

Expected behavior
I'd expect for this not to always pop up as the connection to the server is fine.
Screenshots
https://i.ibb.co/T15qC6d/Capture.png
Your Configuration (please complete the following information):

Exact BookStack Version (Found in settings): BookStack v0.31.6
PHP Version: I used the install script for Ubuntu 20.04. I don't know the PHP version.
Hosting Method (Nginx/Apache/Docker): Install script on Ubuntu 20.04.

Additional context
n/a

I'm going to try installing it manually on Windows and see if that fixes the issue.

That fixed it :)

Yeah I have the same problem now. Installed it on an Ubuntu webserver.

@kevinblackdesk sorry I have no idea. I'm using XAMPP on windows so it could be whatever the web server it's using on Ubuntu causing it maybe?
Wish I knew more about the issue.

@kevinblackdesk on Ubuntu the formatting for editing a page was all screwed up as well. For example, it wouldn't center text and code blocks were huge.


Start updating frames in viewWillAppear, rather than in viewDidAppear.

Looks great, thanks! I think it makes sense to keep the focus event in viewDidAppear because until the view transition is complete, the user can't interact with the app, so it doesn't really have focus.


Need help on github action System has not been booted with systemd as init system (P\ID 1). Can't operate. [LocalTest/Test k8s backend & chart] | Failed to connect to bus: Host is down

https://dask.discourse.group/t/getting-starting-with-act-locallly-with-podman-need-help-system-has-not-been-booted-with-systemd-as-init-system-p-id-1-cant-operate/2251 I wrote a bug report here

cri-dockerd does not support podman and it never will. This is a CRI for dockerd. You will need to use a CRI for podman or dockerd as your runtime.

same error occurs with docker as well, I think it relates to nektos/act


groups creations

Greetings, thanks for the excellent work, would it be possible for groups to be created automatically when creating a client? Or that the clients also appear in the Device search engine? that would allow a faster search for a device (or a client) currently the group can be done by hand, but it is repetitive and necessary if you wants to be able to easily search for someone's devices, another option would be for the clients to come in the devices tab and its equipment as a tree view, thanks for your time

There's no concept of a "client". Do you mean a user? Why can't you just log in a the client to see their devices?

yes, user, sorry, my mistake, most admins leave a copy of all the devices in their account, so if for example the user "jesus christ" tells me that he has a problem with his equipment I have to look for him, (normally I should have created the group with that name to be able to see it)
if I have to review, for example, 20 or 30 devices I have to be able to go from one to another easily (I repeat, if I create the groups I can search for them in the search engine) but in the current way If I don't create them, I have to enter the client's account, exit, go back to mine, go back to another client, (and repeat and repeat)
it takes much more time than being able to look for it directly in my account and be able to go from one to another, ( That is why if the groups were created automatically or being able to search by client, it would greatly speed up the work of the admin)
In this way, with a quick review, one can see the status of the clients and devices in general organized, in the current way, to see it easily, one has to remember the ID (normally the license plate) that the vehicles of each client have.

Maybe you can provide more details on the use case. Why do you need to review multiple clients one by one?

Of course, let me tell you a little about my story so that you understand that I have come across these cases many times, I am not a programmer at all, but for 20 years I have been from installer to administrator of more than 7 companies, I have seen many platforms of tracing. I am trying to start my own life in this medium.
As a manager or administrator one has to ensure the proper functioning of the devices, the first thing one sees is the general list to know which ones are reporting or not, but today visually you do not know who each device belongs to,
The first line to know this is the users themselves, who communicate to say that they have not received an sms or email, so you have to check the web first, and send sms or status commands to see if it is a bug. gsm, or satellite, or devices, and of course wait for the response and return several times to the device
this is the common start of any day for an admin, depending on that he would coordinate the visits to the equipment, and even when he does, he will have to see the device several times to confirm that it went well,
When installing, the same thing happens, checking several times that it reports ok, that it emits alerts, that it has a good signal
Assuming you have a system of 1,000 vehicles and 10% of them require (albeit minimal) service, we would be servicing 100 vehicles per day,
not to mention that the equipment that you checked yesterday and the day before yesterday you check again today to confirm that they are fine and keep the user calm
It is more than common that when a user communicates with the administrator for 1 device, the two start to review all of them (and my wife's is good? and my mom's? and my cat's?)
All this means that not only several have to be reviewed, but many more since it is all about the user being functional, also in billing cases when charging one, all the others are reviewed.
with the current system, you either create the groups to be able to see all the users or you have to log in and log out (when you log out, you don't go back to the admin account, but you have to log out and back into the system of course) when review it's a matter of 2 seconds it gets long in the process of reloading and reloading the page,
that would be the most basic case, but there are many variants where a tree view or being able to search for them by users in the main tab helps a lot with system administration

Based on the explanation it still sounds like you're working on a single client at a time, even if you need to check multiple devices they all belong to the same client. Maybe I misunderstood something.

I was putting 1 case, but (and it is daily) imagine that scenario with at least 10 users and 20 vehicles every morning. enter, exit, verify, device, exit, re-enter search for another user, verify device, return to the previous one and see if it worked, exit again, etc, (this added to the fact that the users tab is organized randomly) is solved a lot by create a group with each user and be able to search for it, but creating them is quite repetitive if you have 200 or 300 users, if the groups were created at the same time or you searched for the clients in the initial tab (where they are already device name , last connection and groups) everything would be much more fluid

I guess it makes sense. Let's see what others think about it.

thank you very much for taking it into consideration

users lack a search magnifying glass. groups missing search magnifying glass description and amount of device that the group has


Design Documentation should be separated from user documentation

Please see https://github.com/ROCmSoftwarePlatform/rocFFT-internal/tree/develop/docs/design for an example design docs not separated from end user docs.
An audit is needed and reorganization of design docs.

https://diataxis.fr/


CMake package config tries to include non-existent SDL_ttf-targets.cmake file

The cmake package config file SDL2_ttfConfig.cmake tries to include the file SDL_ttf-targets.cmake (at the bottom of that file). But, the file generated by the build is called:  SDL2_ttf-${sdl2_ttf_install_name_infix}-targets.cmake (https://github.com/libsdl-org/SDL_ttf/blob/main/CMakeLists.txt#L276)
As a result, the include fails with an error similar to the following:
CMake Error at <SDL_ttf install prefix>/cmake/SDL2_ttfConfig.cmake:18 (include):
  include could not find requested file:

    <SDL_ttf install prefix>/cmake/SDL2_ttf-targets.cmake
Call Stack (most recent call first):
  CMakeLists.txt:276 (find_package)

A possible solution is to change the name of the target file inside SDL_ttfConfig.cmake like so:
include("${CMAKE_CURRENT_LIST_DIR}/SDL2_ttf-<EMAIL_ADDRESS>
The configure_package_config_file call should then expand this to the right target file name.

Good find, I'll fix asap!


npm build silently fails, instead running  gulp --production produces error?

How can we reproduce this bug?
Running npm build is doing nothing for me, so I run gulp --production .
What did you expect to happen?
gulp --production to run as expected and add inline css to dist/index.html and minify it.
What happened instead?
[09:20:55] Requiring external module babel-register
[09:20:56] Using gulpfile ~\Desktop\work\SB-170\sb-170\gulpfile.babel.js
C:\Users\Acer-Pc\AppData\Roaming\npm\node_modules\gulp\bin\gulp.js:129
    gulpInst.start.apply(gulpInst, toRun);
                  ^

TypeError: Cannot read property 'apply' of undefined
    at C:\Users\Acer-Pc\AppData\Roaming\npm\node_modules\gulp\bin\gulp.js:129:19

    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)
    at Function.Module.runMain (module.js:449:11)
    at startup (node.js:146:18)
    at node.js:404:3

Are you trying to run the production and start the server? If so that is npm run build

Well what I wanted was to run only build`` like npm build, but this command does not work for me. npm run build``` it does the job pretty well. Thank you for your help


Getting started for newcomers

Hi, I'm a CS student in Padova (Italy) on Master Degree and I was interested in working on a thesis about serverless computing and I found this project. I have personal good experience with JavaScript and frontend technologies, but I'm not sure where to get started with WebAssembly and this project in particular. Do you have any advice/resource to share?
I'd also love to see if you have any roadmap. I'm currently evaluating the idea of focusing the whole thesis on this cool stack Serverless on WASM and I'd like to help with some development hopefully, once I get my head around the technologies and the source code.
I opened an issue because others might be interested in contributing, serverless is an hot topic!
Thanks!

Hi,
In terms of roadmap, it is currently quite HPC oriented at the moment. I'm currently doing my own thesis on using supporting existing OpenMP programs in a dynamic environment. And at the same time the codebase is being extended for MPI support and is being corrected for WASI compatibility instead of the current more custom Faasm host interface.
There are many new things you could use Faasm for to give you a head start. If you're a JS developer your could look at https://github.com/Shillaker/faasmjs and tune Faasm for edge computing workloads for example. If you're interested in graphics processing, the OpenMP API is currently adding support for GPUs but nobody is looking at how to integrate this with WebAssembly (see my failed PoC #197 - getting this to work would be great). I have a dream that the current integration with Rust will be extended with Rust futures in a super composable API efficiently ran by building computational DAGs internally but you'd have to be a wizard to get this to work.

Thanks for the reply. Yup, after I posted the issue I found the paper about Faasm and realised it's focused on HPC whereas my interest is more on edge computing and replicating/analysing the work done by Fastly and Cloudflare.
I'm also interested in the integration with Rust, but I didn't understand your statement about composable API with DAGs. I suppose I need to study a lot yet :)

Hi @jiayihu, thanks for the interest!
If you're coming from the browser/ Javascript world then you might be interested in the cross-over between client-side wasm and server-side wasm as @mfournial mentioned. Faasm.js was a PoC I did to experiment with offloading client-side wasm tasks to a back-end serverless wasm cluster. The idea is to have exactly the same code able to execute in the browser and on the server so that the runtime or user can choose where it gets run, potentially making the decision dynamically. A client-side library could choose to offload work to a serverless back-end to access a much higher level of parallelism, or to execute code closer to large datasets stored in the cloud (e.g. for video/ image processing). This is essentially a standard client-server model, so isn't really that interesting in itself, but there's potential in looking at the programming model when you have a uniform client-server execution environment, and when/ how you dynamically offload work to other devices. It could also be interesting to extend this to other execution environments such as phones or IoT/ embedded devices, creating a sea of computing power, all able to execute the same code. Taking that idea a step further, you could also think about volunteer computing, offering a wasm-capable device or browser up to join part of a larger cluster of things all working collectively to solve problems.
Faasm provides a good base for this sort of project, as it offers lightweight isolation, low overheads and distributed shared state. That said, our current focus is indeed on larger distributed computation, trying to push the limits of the system in terms of parallelism and performance. This focus will only only cover the next few months, and I'd still be interested to hear any more concrete thoughts/ proposals you come up with.
Hope at least some of that has been useful/ triggered some potential ideas!


Stargate upgrade proposal

Initial commit of the stargate upgrade proposal
Rendered: https://github.com/iqlusioninc/governance/tree/zaki-stargate-upgrade-proposal/proposals/2020-11-stargate-upgrade

We have a cover image you can use for this proposal doc if you wish. 🧑🚀  Not sure where it will inevitably be published, but you can use it for the GitHub md in the meantime. 🙂

Security Updates for the Stargate Upgrade
Like all Cosmos Hub upgrade proposals, the Stargate Hub Upgrade Governance Proposal (Prop 35) includes a commit hash which operators have agreed to upgrade to. The commit hash included in Prop 35 may be run for the upcoming upgrade. However, there is also a newer version of Gaia. You may be wondering why, and what to do here.
Here’s the context: Tendermint Core and the Cosmos SDK recently released new versions, 0.34.3 and 0.40.1 respectively, to protect against a DOS vector present in Tendermint Core versions 0.34.0-0.34.2. Gaia was subsequently updated to pick up these security patches, and Gaia v3.0.1 was released.
These changes do not impact the current Cosmos Hub, although the vulnerability is present in the version of Gaia scheduled for the Stargate Hub Upgrade. Specifically, Gaia v3.0.1, the patched version of Gaia, is different from the commit hash present in Governance Proposal 35.
In order to ensure that your validator operations will not be interrupted unexpectedly, node operators are encouraged to choose the latest version of Gaia when doing the Stargate Upgrade instead of Gaia v3.0.0; however, node operators may choose to run Gaia v3.0.0, which is compatible with later versions of Gaia. The conformance of the upgraded network to the governance proposal can be verified by syncing a Gaia v3.0.0 full node to the network, and neither version will break consensus.

Security Updates for the Stargate Upgrade
Like all Cosmos Hub upgrade proposals, the Stargate Hub Upgrade Governance Proposal (Prop 35) includes a commit hash which operators have agreed to upgrade to. The commit hash included in Prop 35 may be run for the upcoming upgrade. However, there is also a newer version of Gaia. You may be wondering why, and what to do here.
Here’s the context: Tendermint Core and the Cosmos SDK recently released new versions, 0.34.3 and 0.40.1 respectively, to protect against a DOS vector present in Tendermint Core versions 0.34.0-0.34.2. Gaia was subsequently updated to pick up these security patches, and Gaia v3.0.1 was released.
These changes do not impact the current Cosmos Hub, although the vulnerability is present in the version of Gaia scheduled for the Stargate Hub Upgrade. Specifically, Gaia v3.0.1, the patched version of Gaia, is different from the commit hash present in Governance Proposal 35.
In order to ensure that your validator operations will not be interrupted unexpectedly, node operators are encouraged to choose the latest version of Gaia when doing the Stargate Upgrade instead of Gaia v3.0.0; however, node operators may choose to run Gaia v3.0.0, which is compatible with later versions of Gaia. The conformance of the upgraded network to the governance proposal can be verified by syncing a Gaia v3.0.0 full node to the network, and neither version will break consensus.

As note, I have synched a full node of gaia v3.0.1 final on stargate-final testnet and the iq validator is now running gaia v3.0.1. This will act as ongoing test of compatibility.
So far positive results.

As note, I have synched a full node of gaia v3.0.1 final on stargate-final testnet and the iq validator is now running gaia v3.0.1. This will act as ongoing test of compatibility.
So far positive results.

Is gaia v3.01 final now the suggested release to run for the proposed upgrade described in Proposal 35?

Is gaia v3.01 final now the suggested release to run for the proposed upgrade described in Proposal 35?

@chris-remus  gaia v3.0.1 -> Cosmos SDK v0.40.1 ->  Tendermint's v0.34.3

@chris-remus  gaia v3.0.1 -> Cosmos SDK v0.40.1 ->  Tendermint's v0.34.3

https://github.com/cosmos/governance/pull/12

https://github.com/cosmos/governance/pull/12

@chris-remus gaia v3.0.1 -> Cosmos SDK v0.40.1 ->  Tendermint's v0.34.3

not easy to remember 😄

@chris-remus gaia v3.0.1 -> Cosmos SDK v0.40.1 ->  Tendermint's v0.34.3

not easy to remember 😄


Add Support to Distributed GPU Training

For a huge model like this, Multi GPU training is the way to go.
Reference:
https://www.tensorflow.org/beta/tutorials/distribute/training_loops

The Project runs perfectly with a Tesla T4 GPU (16 Gigs), with a batch size of 8.


Add support for deflate decompression

EDIT: Oh! I see this was intentionally removed in the past. Feel free to close!
This adds support for responses that were compressed using the deflate algorithm.
As best I can tell, this isn't terribly common for servers to send in the wild—in my own usage of Req for scraping public sites, I see gzip and Brotli way, way more frequently. However, I think it's useful to have in Req for the sake of having decompression support that matches real browsers; Safari, Firefox, and Chrome each send an Accept-Encoding header of gzip, deflate, br.

yeah I'm happy to add it when someone actually needs but tbh I'm still unclear how exactly it should work. thanks for the PR though!


Feedback on Glossaries feature/UI

[x] Remove delete option until hard delete is supported. If someone deleted a glossary, the glossary of the same name can't be created.

[x] Adding a Glossary term as a label doesn't show the corresponding Table in Assets.

[x] When adding a tag or glossary term as a label, tag terms show up faster and glossary terms show up slower. In some cases, typing a few characters does not suggest glossary terms.

[x] Search while adding Related Terms does not work. Type Risk and the glossary term Risk Assessment does not show up. Also the search box says Search for a User.

[x] Need better use of spacing. Adding more space after arrows would be better.

[ ] Make Related Terms clickable

[ ] Long glossary term name/highly nested terms are cut off

[ ] Make parent terms clickable. In the below examples, make Domain and Sales clickable.

[x] Searching for users to set Reviewers is very slow

[x] When hovering over terms in the tree hierarchy, give feedback by highlighting the term hovering over

[x] Remove tags column for assets and just include name, description, and owner. The look and feel could be similar to Explore page results.

[x] When search does not find a glossary term, there is no indication -

[x] Partial searches don't work. For example, using the first-word Annual finds a Glossary Term. But typing revenue does not find matching terms.

[x] Change the tab name to Reviewers for Glossary similar to the Glossary Term page.

[ ] URL does not change as you click through different terms. It just stays as Glossary.

[ ] We should only show leaf glossary term instead of the entire hierarchy in all the places.

[ ] add a hover over on search tags to display full text of a tag or glossary term

@aashitk and @harshach, can we fix this as a blocker.

[ ] GlossaryTerm details page has different ways of showing data is not available. Can we use a consistent way as shown for Assets tab?


Get KeyVault name for sp_check from matrix data

Context
KeyVault name is stored in GitHub secret (which hasn't been set in a number of repos.  Reducing GitHub secrets reduces management overhead and makes this workflow easier to generalise.
Changes proposed in this pull request

Use matrix data to store KeyVault
Pinned major version of rtCamp/action-slack-notify
Fixed setting of SLACK_WEBHOOK env var

Guidance to review
Review test logs here
Checklist

[ ] Attach to Trello card
[x] Rebased main
[x] Cleaned commit history
[ ] Tested by running locally

 Check Service Principal Key Vault Values


Work Pool creation/edit not respecting required fields

When creating a WorkPool, there can be a number of form fields that can required. The UI used to validate these fields before letting you proceed to creating a work pool.
Repro:

Create an AzureContainerInstance WorkPool
Fill in no options for the base job template (note that resource_group_name and subscription_id and required in the response from /views/aggregate-worker-metadata)
Observe the work pool is created successfully

Ah - this was an expected regression from https://github.com/PrefectHQ/prefect-ui-library/pull/1560 but will be un-regressed (re-regressed? fixed.) with the release of this code. Given that the fix is already merged, I'm going to close this.

Thanks @znicholasbrown - if theres not a planned release of this by noon tomorrow could we get one out the door?

Yup 👍🏻

Just kidding, while what I said was true as well, the work pool creation wizard didn't have a validation observer on the infra creation step -this PR adds one and will close this issue properly.
Resolved by: https://github.com/PrefectHQ/prefect-ui-library/pull/1570

#1570 is merged and will go out with the next release (today or tomorrow)


Add normalize option in plot_wigner

Add a normalize option (defaults to True) to dq.plot_wigner and dq.plot_wigner_mosaic.
This is useful for instance when plotting the wigner of a trace 0 density matrix (e.g. a density matrix obtained from Liouvillian diagonalization). It will get renormalized like crazy.
Question: do we remove the warning then?
Closes DYN-151.

To answer your question, to me yes we can remove the warning. If the user chooses to change the default value they should know what they are doing so there is no need to guide them with a warning

The default value is normalize=True, so we should either keep it if we choose to keep the normalisation. However IMO if we allow plotting the Wigner of non-physical states we can just remove altogether the normalize option, the warning and the normalisation code.

I'm in favor of keeping the normalization as the warning saved my life once or twice (too small timesteps, wrong GPU configuration, low truncature)

I'm in favor of keeping the normalization as the warning saved my life once or twice (too small timesteps, wrong GPU configuration, low truncature)

But then you see visually that it's incorrect no?

It's hard to see when  state has a norm between 0.9 and 1.1 at least

Right. So we can bring back the warning then, but maybe not normalize the state ourselves. Maybe the name should be check_normalize or check_state then. Actually a bunch of other methods should check this (e.g. plot_fock) and we could have a common argument name to skip or not the checks.