[CDC][SPI_DEVICE] W_MASYNC in pinmux FFs

https://github.com/lowRISC/opentitan/blob/master/hw/ip/pinmux/rtl/pinmux.sv#L346-L372
Data (periph_to_dio_i) from spi_dev_clk or spi_dev_passthru_clk are latched by io_div4_clk.
The tool shows periph_to_dio_i are on three clocks : spi_dev_clk, spi_dev_passthru_clk, and io_div4_clk.
But, as I manually checked, io_div4_clk is not found from the path.

i think io_div4_clk might be clocking the retreg, is what what you mean?
On Fri, Jul 22, 2022 at 11:57 AM jeoongp @.***> wrote:

Assigned #13811 https://github.com/lowRISC/opentitan/issues/13811 to
@tjaychen https://github.com/tjaychen.
—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/13811#event-7048508245, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSV74KSDWT2Z5R3JBFLVVLVI7ANCNFSM54MO7VOQ
.
You are receiving this because you were assigned.Message ID:
<lowRISC/opentitan/issue/13811/issue_event/7048508245 <(704)%20850-8245>@
github.com>

Yes
On Fri, Jul 22, 2022, 12:21 PM tjaychen @.***> wrote:

i think io_div4_clk might be clocking the retreg, is what what you mean?
On Fri, Jul 22, 2022 at 11:57 AM jeoongp @.***> wrote:

Assigned #13811 https://github.com/lowRISC/opentitan/issues/13811 to
@tjaychen https://github.com/tjaychen.
—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/13811#event-7048508245, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAH2RSV74KSDWT2Z5R3JBFLVVLVI7ANCNFSM54MO7VOQ
.
You are receiving this because you were assigned.Message ID:
<lowRISC/opentitan/issue/13811/issue_event/7048508245 <(704)%20850-8245>@
github.com>

—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/13811#issuecomment-1192869515,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYS575CPA4YDD7IZCWUVSO3VVLYE5ANCNFSM54MO7VOQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

ah i think we this one we can waive. The io_div4 value capture there is really meant for holding the pad value steady prior to going into low power.
For SPI_CLK stuff it's a little bit funny since that is driven by an external clock. I think for stuff like this we will need some kind of protocol with the external world, ie, we need to shut down spi_device in some way.
@eunchan is there something where we can do that?
@a-will I'm thinking a situation where the device is JUST about to go to sleep, and the external world starts driving clocks etc, and that causes a glitch as we try to latch the value. My assumption though for this kind of stuff, it would be idea if we have some kind of sideband signaling that the system needs to wake.  But it would still be safer if we can quiet everything down internally.

@tjaychen thanks for clarifying. Are these FFS a kind of retention registers before low power?  I agree with a protocol with the external world. Based on scenarios and sequences for transition to low-power mode, corresponding signals are better to be defined.

yeah these FFs are retention registers that can latch the "last seen" value before going to low power.
they are activated when the power manager asserts the sleep_en_i signal, although they are configurable so that not every interface has to use them.
I agree that we may need something like a protocol with the external world.
because even if we add a double flop here to avoid metastability, the retention mux at the output of that flop is still steered with a signal coming from io_div4, so there might still be glitches due to a switch (if the output is actively toggling).
I think we had discussed this in the past and the assumption that things will be stable when going to sleep was the main reason why we did not add double flops in the first place.

one other thing to note: if the pinmux is configured to not enable the retention registers, then the output value will just be tied off to the isolation clamp value when the system goes to sleep.

@msfschaffner Thanks! I waived this one.

@jeoongp sorry i can't remember, was the waiver already merged? If yes, we can probably close this issue.

@tjaychen it's not merged due to a whitespace error in https://github.com/lowRISC/opentitan/pull/13739. I just fixed it.

@tjaychen can you approve #13739? Once it's merged, we can close this ticket.

Hi @jeoongp, can we close this issue now that the PR has been merged? Thanks!

Sorry for late reply. Yes, this issue was resolved by #13739.

Thanks @jeoongp!


Confusing use of "default" for langversion

This document and this one are both extremely confusing, because they use the term "default" to mean 2 different things for the langversion. There's the default that is used if no langversion is specified, and then there's a specific value named default that is a synonym for latestmajor. At least, I think that's how it's supposed to work. I'm making an educated guess here because it's not clearly documented. I understand that removing the value of default may not be an option because it would be a breaking change, but the document needs to be very clear about the difference between this value and the actual defaults that are used when no value is specified.

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

ID: 5694e62b-c756-1f62-4014-2ceafa54c011
Version Independent ID: 3f01c378-22e4-197c-a862-ab4a924506c7
Content: C# language versioning - C# Guide
Content Source: docs/csharp/language-reference/configure-language-version.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Your analysis is correct @elezar42
I've added this to our backlog to address when we next update this article.

Hello everyone!
Maybe I am mistaken, but I understood from the text that if no override is made, the compiler will use the latest language version compatible with your target framework. So, the default that is used if no langversion is specified and the specific value named default that is a synonym for latestmajor are exactly the same thing.
What are you expecting as a solution to this issue?

@luizhlelis From my testing, using the specific value of default (or latestmajor) uses the latest version of the language supported by the compiler, without taking into account the target framework. So building a project that targets .NET 6 from the current version of VS2022 will use C# 10 if no LangVersion is specified, but will use C# 11 if LangVersion is set to default o latestmajor.


terminal freezes after: idfx flash COMx

Hi! I installed WSL2 esp-idf on Windows 11 as per instructions, and first time after install idfx flashes and monitors esp32 fine, but after opening the terminal again, idfx can only build the project, not flash or monitor. python.exe is recognised in WSL2.  Tried to go from Ubuntu to Debian, same thing.
Any clues where to dig? Thanks!

Interesting.
Can you please provide more informations about your environment.
Execute next commands and paste here the outputs.
On Windows execute next commands:
python --version

pip freeze | findstr 'esptool'

In WSL execute next commands:
python --version

which python

idf.py --version

And please paste here any output that you get after executing idfx flash COMx. Are you sure that you are using correct COM port name from the Device Manager? Maybe somehow in meantime Windows has changed the name of port, for example from COM2 to COM3, or something like that? Check one more time please.

Due no activity I will close this issue.
Feel free to open it again if you have more questions.


miui-framework.jar might fix the missing apps on floating app

I once encountered modifying or changing the mi framework.jar then the apps on floating window vanished it was fixed by adding the stock miui-framework.jar from system ext partition might as well as try

please move to discussions tab


vanilla.io thoughts

Should this be a router, seems janky:
https://github.com/cablehead/vanilla/blob/master/vanilla/io.py#L96
Should the layer up be a router? or perhaps deduping a layer up?

Ugh, should have linked to the specific commit version


"Toggle Saka" via shortcut not working

Trying to enable/disable saka by pressing l doesn't work.
I then used Add Binding to create alternate shortcuts to toggle saka. Even the new keys didn't work. (Not a new issue. Even an year back it was like this.)
Everything else is ok.
This was done on a fresh profile with no extensions or any changes made anywhere.
saka key v1.26.3
Firefox 77.0 (stable)
Debian 11 (bullseye, sid)

It seems this issue is a duplicate of #154.

@timacs
Ok, it looks as if the programmers (lusakasa) are using that key 'l' in their other extension 'Saka' which manages tabs. Since this extension is far more popular could they please move this key for use with 'Saka keys' or atleast 'enable' it via some other key so that 'saka keys' can be quickly switched on and off.
For me saka keys conflicts with youtube and other youtube extensions like speed control, so quick enable/disable is important. I have even been thinking of moving to vim keys which I don't know since i am an emacs user, and my brain has got so used to the saka key bindings.
Since this issue has been running from 2017, could the programmers please prioritize this?
Any info from others would also be helpful.

@extralight
I don't know if you noticed that but there are many signs that (active) development of Saka Key stopped about two years ago. I was a happy user of this nice extension and still watch this repo, but the lack of proper support made me switch to another extension – in my case, it's Tridactyl. I am also an Emacs user but used Vim before, so the switch was not too hard for me.

my brain has got so used to the saka key bindings
Most of keyboard-oriented extensions allow you to rebind keys. Of course, it takes some time but isn't that hard.


Managing sound/music from other apps with video components

Bug
I don't want to disable audio from other app on some <Video/> components, and in some cases I need to disable it. As I see in docs, we have a section "Audio Mixing", which tell us to use:

#import <AVFoundation/AVFoundation.h>  // import

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{
  ...
  [[AVAudioSession sharedInstance] setCategory:AVAudioSessionCategoryAmbient error:nil];  // allow
  ...
}

in AppDelegate.m/mm and use ignoreSilentSwitch prop(To be honest, don't understand how it should affect..)
At first it doesn't work personally for me. The second thing is that mixWithOther prop can help in case I need to mix audio, but what am I suppose to do in case, when I actually need to disable sound from other apps?
Platform

Which player are you experiencing the problem on:

iOS

Environment info

React native info output:
System:
    OS: macOS 12.3.1
    CPU: (8) arm64 Apple M1
    Memory: 460.67 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 18.0.0 - /opt/homebrew/bin/node
    Yarn: 1.22.17 - /opt/homebrew/bin/yarn
    npm: 8.6.0 - /opt/homebrew/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
    Android SDK:
      API Levels: 23, 28, 29, 30, 31
      Build Tools: 29.0.2, 30.0.2, 31.0.0
      System Images: android-29 | ARM 64 v8a, android-29 | Intel x86 Atom_64, android-29 | Google APIs ARM 64 v8a, android-29 | Google APIs Intel x86 Atom, android-29 | Google Play ARM 64 v8a, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play ARM 64 v8a, android-31 | Google APIs ARM 64 v8a
      Android NDK: Not Found
  IDEs:
    Android Studio: 2020.3 AI-203.7717.56.2031.7784292
    Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.11 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.68.1 => 0.68.1
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Library version: 5.2.0
Steps To Reproduce

Try to use mixWithOther prop or "Audio Mixing" topic from docs for disabling/enabling sound from other apps

Expected behaviour

I can manage sound from other apps manually

I'm unsure about iOS but we have an working implementation of this on our internal patches. I'll follow up a simple PR when I got some time, just commenting here for my own tracking

Here's what we found out by far:

Android manages the audio playback per playback, as opposed to iOS managing per app. What it means in practice is having two <Video /> playing simultaneously with different mixWithOther or ignoreSilentSwitch props will likely result in unintended behaviour on iOS.
Android doesn't use the ignoreSilentSwitch or mixWithOther prop; it uses onAudioFocusChanged and disableFocus props to manage them. However, pausing/playing a video will result in audio focus change because this behaviour is hard-coded in Android native code.

This is what we had to patch to allow JS code to access the audio focus state.

Almost same here, I'm trying to
Play background music when video muted -> stop background music when video unmuted
But even with the Appdelegate change and ignoreSilentSwitch. if I have
mixWithOther={isVideoMuted ? 'mix' : 'inherit'} the inherit doesn't apply

i believe it should be mixWithOther={isVideoMuted ? 'mix' : 'duck'} - this should lower the background music if the foreground is unmuted


Feature: Product listing

Product listing page
Epic: Product listing page #2
Changes made

[x] Update product listing and homepage styles
[x] Added mock endpoints with mockbee
[x] Added filter and data source context
[x] Added filters,sorting feature in product listing
[x] Abstracted out common components

How does the solution address the problem
This PR will resolve the issue 2
Linked issues
Resolves #2

Pretty clean. Good to go


[Sema] Enable substituting the base of a dependent member type with an opened archetype.

This is supported with the recent generalization of OpenedArchetypeType to behave similar to other archetype types.
Resolves: rdar://87776479 / SR-15751

@swift-ci please smoke test

@swift-ci please test Windows


chore: adding small extra step at installation

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.
Problem/Feature
Description of Changes:

a quick extra step to tweak .zshrc during installation for a more straightforward setup

Checklist

[x] I have self-reviewed the changes being requested
[x] I have updated the documentation (check issue ticket #218)

I would put this reader in the docs, not in a repo's README.

No response to the suggestion, so I am closing the PR.


gii generated code:  Html::resetButton not working after first search is performed

I'm reporting a bug.
Problem: Reset button does not clear input fields after first search is performed.
I'm using the advanced template in Yii2 and working on the frontend.   I used gii to generate model and crud code for 1 table in MySQL called "Maps" with no further modifications made.
MapsController / index action points to the views/maps/index.php view.  In that view file, I uncommented the following code to perform searches on model attributes.
echo $this->render('_search', ['model' => $searchModel]);

The above code enables an ActiveForm in the ( "_search.php") view file with input boxes and a "search" & "reset" buttons.  If I enter data into any of the form fields WITHOUT performing a search function, then hit the "reset" button, the form fields will be cleared of any text I put in.
The problem occurs after I enter text, hit the "search" button, then hit the "reset" button.  It does NOT clear text in the input boxes.
views/maps/index.php file:
<?php

use yii\helpers\Html;
use yii\grid\GridView;

/* @var $this yii\web\View */
/* @var $searchModel app\models\MapsSearch */
/* @var $dataProvider yii\data\ActiveDataProvider */

$this->title = 'Maps';
$this->params['breadcrumbs'][] = $this->title;
?>

<?= Html::encode($this->title) ?>

    <?php echo $this->render('_search', ['model' => $searchModel]); ?>

        <?= Html::a('Create Maps', ['create'], ['class' => 'btn btn-success']) ?>

    <?= GridView::widget([
        'dataProvider' => $dataProvider,
        'filterModel' => $searchModel,
        'columns' => [
            ['class' => 'yii\grid\SerialColumn'],

            'mapID',
            'name',
            'description:ntext',
            'updated',
            'updatedBy',
            // 'make',
            // 'model',
            // 'year',
            // 'eManufacturer',
            // 'eName',
            // 'baffleType',
            // 'eSize',
            // 'isCustomerMap',
            // 'customerID',
            // 'isOriginalMap',
            // 'photoUrl:url',
            // 'created',

            ['class' => 'yii\grid\ActionColumn'],
        ],
    ]); ?>

views/maps/_search.php file:

use yii\helpers\Html;
use yii\widgets\ActiveForm;

/* @var $this yii\web\View */
/* @var $model app\models\MapsSearch */
/* @var $form yii\widgets\ActiveForm */
?>

    <?php $form = ActiveForm::begin([
        'action' => ['index'],
        'method' => 'get',
    ]); ?>

    <?= $form->field($model, 'mapID') ?>

    <?= $form->field($model, 'name') ?>

    <?= $form->field($model, 'description') ?>

    <?= $form->field($model, 'updated') ?>

    <?= $form->field($model, 'updatedBy') ?>

    <?php // echo $form->field($model, 'make') ?>

    <?php // echo $form->field($model, 'model') ?>

    <?php // echo $form->field($model, 'year') ?>

    <?php // echo $form->field($model, 'eManufacturer') ?>

    <?php // echo $form->field($model, 'eName') ?>

    <?php // echo $form->field($model, 'baffleType') ?>

    <?php // echo $form->field($model, 'eSize') ?>

    <?php // echo $form->field($model, 'isCustomerMap') ?>

    <?php // echo $form->field($model, 'customerID') ?>

    <?php // echo $form->field($model, 'isOriginalMap') ?>

    <?php // echo $form->field($model, 'photoUrl') ?>

    <?php // echo $form->field($model, 'created') ?>

        <?= Html::submitButton('Search', ['class' => 'btn btn-primary']) ?>
        <?= Html::resetButton('Reset', ['class' => 'btn btn-default']) ?>

    <?php ActiveForm::end(); ?>

More Detail:

In the views/maps/index.php file, the input box's name attributes resulting from this code:
echo $this->render('_search', ['model' => $searchModel]);

are  the same names for the input boxes when you have the GridView::widget's filterModel enabled.
So there are two inputs with the same names, not sure if that's a problem.  But it's definitely interfering with the reset button.

An html reset button sets the value of the input text to the one that had when the page loaded, it doesn't clear the value, see this fiddle.
http://jsfiddle.net/znw0gy15/2/
When you submit the form it is reloaded with the values you entered as the default ones so the reset button will reset them to that value not to empty.

Thanks!

Reloading the page with an action worked for me as a 'get around'.
Use a link,  Html::a   , instead of Html::resetButton  ,with the action  ['index'], and retain the class,   ['class' => 'btn btn-default']
Replacement code:
 'btn btn-default']) ?>

****


Add icons to redirect to social media on topbar mobile

I still haven't added the behavior of the discord button, because I don't know what the discord of the doiim is


Add full support for MONEY type of PostgreSQL

Goal
Add support for generating sql alchemy type of MONEY.
Description
Currently, the sqlalchemy types classes are set on the domain classes from the sqlalchemy.sql.sqltypes.py, a file on the sqlalchemy library that provides classes more generic for the common sql types.
The MONEY PostgreSQL type has a class on sqlalchemy to set it on the domain files, but it is located on the sqlalchemy.dialects.postgresql.types.py.
To accomplish this task, it would be necessary to change on the generator to, whenever a column of the MONEY type exists, an import of the sqlalchemy.dialects.postgresql.types would have to be added on SqlAlchemyBuilder.py file
Classes to be changed
SQLAlchemyTypesUtils.py
*Add a new class to handle the modifications on the SQLAlchemyBuilder.py file after it has been copied over to the generated API

Commit 35e92f1d7da3728046ed9811814d91ce28413571 fixed this issue


Documentation: nb-action is missing

There is a documentation for nb-actions, but there is no anything for the nb-action.

fixed https://akveo.github.io/nebular/#/docs/components/actions


VDC: vdc controller should access the kubernetes cluster through private IP

The kube config should use the private IP inside the VDC controller
Replace private IP with a public one when downloading the kube config

After adding the feature of redeploying the master node, we can only guarantee that the public IP will not be changed but the private one may be changed. So this issue can't be done.


Makefile?

Just trying to follow from https://youtu.be/z6pYfz_q9tc?t=2006 and I'm a little confused how you:

Setup the cluster
What commands you ran to deploy

Perhaps a Makefile or README would help?
Ideally you could have a Github workflow to boot?

Hey! Thanks for asking! I will update the readme with more info, but here's the high level:

The cluster was deployed with https://eksctl.io/ , It's a super easy single command to configure everything needed for eks. eksctl create cluster --name hello-eks --region us-east-1
To run the dagger pipeline, I just ran go run ci/main.go. The entire pipeline is defined here: https://github.com/kpenfound/hello-eks/blob/main/ci/main.go

Failed with:
Status: Downloaded newer image for ghcr.io/dagger/engine@sha256:41f71f0036167fcc8b926792b079d840c52066eec093f3e177881468fd122c83
                                 Error: buildkit failed to respond
panic: EOF: Error: buildkit failed to respond

I am running:
❯ docker ps
CONTAINER ID   IMAGE                          COMMAND                  CREATED              STATUS              PORTS     NAMES
9c172890339a   ghcr.io/dagger/engine:v0.3.5   "buildkitd --oci-wor…"   About a minute ago   Up
About a minute             dagger-engine-41f71f0036167fcc

Subsequent run was:
❯ go run ci/main.go
#1 resolve image config for docker.io/library/golang:latest
#1 ERROR: failed to do request: Head "https://registry-1.docker.io/v2/library/golang/manifests/latest": net/http: TLS handshake timeout
------
 > resolve image config for docker.io/library/golang:latest:
------
panic: input:1: container.from failed to do request: Head "https://registry-1.docker.io/v2/library/golang/manifests/latest": net/http: TLS handshake timeout

Please visit https://dagger.io/help#go for troubleshooting guidance.

goroutine 1 [running]:
main.main()
        /home/hendry/tmp/kpenfound/hello-eks/ci/main.go:67 +0x1439
exit status 2

Third run is appears stuck at:
#5 extracting sha256:195ea6a58ca87a18477965a6e6a8623112bde82c5b568a29c56ce4581b6e6695 1.0s done
#5 extracting sha256:52908dc1c386fab0271a2b84b6ef4d96205a98a8c8801169554767172e45d8c7
#5 extracting sha256:52908dc1c386fab0271a2b84b6ef4d96205a98a8c8801169554767172e45d8c7 1.0s done
#5 extracting sha256:a2b47720d601b6c6c6e7763b4851e25475118d80a76be466ef3aa388abf2defd

Not sure how I'm supposed to make sense of that! 😬

Fourth run after 9m15s:

Not sure what Cloudflare has to do with https://github.com/kpenfound/hello-eks/blob/main/ci/main.go
Fifth run:
#13 copy /hello /bin/hello
#13 DONE 0.1s

#14 exporting to image
#14 exporting layers
#14 exporting layers 0.7s done
#14 exporting manifest sha256:57d97a485b1da60b550d5cf793c6d141b4284cdf5f3bfa11155f7194dbafbac8 done
#14 exporting config sha256:3a624da7056db2584010c4aa621180add2ac9781d8cbc2b6528040a5402d8694 done
#14 exporting manifest sha256:fe03730a1be27684bfccaede87f5d135caff4528926c45200d9306c6207a7987 done
#14 exporting config sha256:0d47e445990d227f92c57fd785977925128e3e3f4e551ccdc81828eda3898a02 done
#14 exporting manifest list sha256:7d53aa79649f16f11891a87d4ab775bf527f4bd03140df17aa58fa183f545712 done
#14 pushing layers
#14 pushing layers 2.3s done
#14 ERROR: unexpected status: 401 Unauthorized
------
 > exporting to image:
------
panic: input:1: container.publish failed to solve: unexpected status: 401 Unauthorized

Please visit https://dagger.io/help#go for troubleshooting guidance.

goroutine 1 [running]:
main.main()
        /home/hendry/tmp/kpenfound/hello-eks/ci/main.go:67 +0x1439
exit status 2

I suspect it's a region issue.
Hmmm, still same problem after tweaking region:
#14 pushing layers 1.5s done
#14 ERROR: unexpected status: 401 Unauthorized
------
 > exporting to image:
------
panic: input:1: container.publish failed to solve: unexpected status: 401 Unauthorized

Please visit https://dagger.io/help#go for troubleshooting guidance.

goroutine 1 [running]:
main.main()
        /home/hendry/tmp/kpenfound/hello-eks/ci/main.go:67 +0x1439
exit status 2

hello-eks on  main [!] via 🐹 v1.19.3 on ☁  tw (ap-southeast-1) took 1m29s
❯ git diff
───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
modified: ci/main.go
───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
@ ci/main.go:107 @ func rollingDeployment(ctx context.Context, clientset *kubernetes.Clientset, ima
func getKubeClient(ctx context.Context) (*kubernetes.Clientset, error) {
    // Get EKS service
    sess := session.Must(session.NewSession(&aws.Config{
        Region: aws.String("us-east-1"),
        Region: aws.String("ap-southeast-1"),
    }))
    eksSvc := eks.New(sess)

I don't understand where it's trying to push to.
Oh I see, I need to change your repo to mine... and then login aws ecr-public get-login-password --region us-east-1 | docker login --username AWS --password-stdin public.ecr.aws/...
Finally!
#13 copy /hello /bin/hello
#13 CACHED

#14 exporting to image
#14 exporting layers done
#14 exporting manifest sha256:c20032dd8875a68c43b05e354cee858f6bf12882e7d04c44edb4a370ecddd1df done
#14 exporting config sha256:f30cdadc0013bc1a51e001ea826dca80a75290b6760cdfdbca8fff0896f049a0 done
#14 exporting manifest sha256:43fe448ae4c90d72b7453ce9342b6ad9ea6ec758379ab2966ee04520f8f5bada done
#14 exporting config sha256:ec73d78320626dd04dfb573839675df321dc4ab7c65322ff9a6eabc29d5236ba done
#14 exporting manifest list sha256:d6b764ebef6b9198dab8d9f2ff50dca041f1b660275deccc6c3afdba998c95a7 done
#14 pushing layers
#14 pushing layers 8.0s done
#14 pushing manifest for public.ecr.aws/z6b8a8c7/hello:dev@sha256:d6b764ebef6b9198dab8d9f2ff50dca041f1b660275deccc6c3afdba998c95a7
public.ecr.aws/z6b8a8c7/hello:dev@sha256:d6b764ebef6b9198dab8d9f2ff50dca041f1b660275deccc6c3afdba998c95a7
2022/11/28 14:23:35 here
#14 pushing manifest for public.ecr.aws/z6b8a8c7/hello:dev@sha256:d6b764ebef6b9198dab8d9f2ff50dca041f1b660275deccc6c3afdba998c95a7 3.1s done
#14 DONE 11.2s
Error deploying hello-eks: deployments.apps "hello-eks" not foundUpdated hello-eks deployment

I don't understand that last error message: Error deploying hello-eks: deployments.apps "hello-eks" not foundUpdated hello-eks deployment
Maybe a path issue to https://github.com/kpenfound/hello-eks/tree/main/kubernetes

I think I managed to get it working if I manually kubectl apply.
Another comment, why isn't go test run automatically?

Glad you got it working! It looks like some of the issues may have been dockerhub rate limiting 🤔

Another comment, why isn't go test run automatically?

Short answer - this demo repo is meant to be a simple example of deploying an application to EKS, and not necessarily a production-grade system.  For a better idea of how dagger is used in production, check out how we're using it for the dagger repo itself here: https://github.com/dagger/dagger/tree/main/.github/workflows
For more DIY examples, check out https://github.com/dagger/examples or any of the guides at https://docs.dagger.io/

I don't quite understand Dagger. I thought the idea is that it gives you a vendor independent CI/CD workflow, i.e.  not defined by a company like Github or Gitlab's .gitlab-ci.yml. Perhaps I've misunderstood the promise of Dagger?

The headline is: Dagger is a programmable CI/CD engine that runs your pipelines in containers.
At a certain level, it moves your cicd pipeline definition from platform-specific yaml to portable code. Meaning you can run the same pipeline locally or in your CI platform of choice.
More details on the promise of dagger here: https://docs.dagger.io/

"platform-specific yaml to portable code" <-- can you give an example of this, because platform-specific yaml != Github workflows, right? :grimacing:

Yeah that's a fair question 😄 Today, that means you're no longer using platform specific yaml to define your pipeline, however it still defines when and how your dagger pipeline runs.
A practical example of this is the test pipeline here
It runs go test on a matrix of 2 different Go versions and 2 different platforms. Using platform-specific yaml for this task, you'd define a matrix and within the matrix run go test. The drawbacks of that are that this yaml configuration can't be run as-is outside of the CI platform, and the syntax to accomplish this is different between platforms.
For the above dagger pipeline, its executed in both Github Actions and CircleCI. In each there's just some boilerplate to define when the pipeline should run, setup qemu (which this pipeline uses, but not a requirement of dagger in general), and sets up the runtime. The amount of boilerplate is something we'd ideally like to reduce to 0, and there are ways to reduce it more than the above example, but it's doing the "important" parts in code which is the point.
The pipeline is a pretty simple one, but in general yaml gets really complex when it's leaned on for defining matrices, environments/secrets, dependencies/runtimes, branching logic, platform-specific extensions (actions/orbs/plugins). The more complex the yaml, the harder it is to reproduce the CICD pipeline outside of the CI platform, and that's where Dagger comes in.

I see, I guess it needs to be more mature for me to readily use it. It appears to be a problem worth solving, though tbh most my pipelines are pretty simple. Not need for matrixes et al.
Thank you!

My pleasure! If you're not facing the challenges described on the docs site, then definitely no reason to change anything! 😄


Throw an exception in SqlExecutor if no files found

What type of PR is this?

Feature

Description
The sql file execution in:
SqlExecutor(base_module).execute_sql_file(file_pattern)
should throw an exception if no .sql files are found in the base_module folder and the specified file_pattern.
The reason for it is to detect spelling errors in the file or pattern name or choosing the wrong base_module folder by mistake.
Overview
    if not found:
        raise ValueError(f"No matching .sql files found in '{self.base_module}'")

What is the current behavior?
No exception was thrown until now. The method did simply nothing.
What is the new behavior?
A ValueError exception is thrown.
Does this PR introduce a breaking change?
There is a potential breaking change, if people relied on an empty folders being ignored. There is a new parameter that will accomplish the same:
SqlExecutor(base_module, ignore_empty_folder=True).execute_sql_file(file_pattern)

Nice, clean code. Thanks for the contribution.


[IOS] Layout shift

I found an issue with BannerAd that have an layout shift on IOS please see on my attachment

@soulbalz this is likely fixed on the latest Flutter version and is not an issue with the package.

@soulbalz this is likely fixed on the latest Flutter version and is not an issue with the package.

Hi, I have tested on the latest version of native_admob_flutter (the latest is 1.5.0) and it's still not fix.


Optimized image size (roughly 200mb)

condensed RUN statement to optimize layers
added build dependency removal

Not sure if this is from my refactor of the image or because of dependency bugs, but I am getting the following error
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6] Traceback (most recent call last):
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/bin/crossbar", line 9, in <module>
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     load_entry_point('crossbar==0.13.0', 'console_scripts', 'crossbar')()
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/lib/python3.5/site-packages/crossbar/controller/cli.py", line 992, in run
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     options.func(options, reactor=reactor)
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/lib/python3.5/site-packages/crossbar/controller/cli.py", line 560, in run_command_start
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     from crossbar.controller.node import maybe_generate_key
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/lib/python3.5/site-packages/crossbar/controller/node.py", line 52, in <module>
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     from crossbar.worker.router import RouterRealm
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/lib/python3.5/site-packages/crossbar/worker/router.py", line 63, in <module>
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     from crossbar.worker.testee import WebSocketTesteeServerFactory, \
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/lib/python3.5/site-packages/crossbar/worker/testee.py", line 44, in <module>
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     from crossbar.worker.worker import NativeWorkerSession
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/lib/python3.5/site-packages/crossbar/worker/worker.py", line 48, in <module>
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     from crossbar.common.process import NativeProcessSession
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/lib/python3.5/site-packages/crossbar/common/process.py", line 67, in <module>
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     from crossbar.common import checkconfig
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]   File "/usr/local/lib/python3.5/site-packages/crossbar/common/checkconfig.py", line 44, in <module>
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6]     from autobahn.websocket.protocol import parseWsUrl
crossbar_1 | 2016-04-14T23:11:26+0000 [Controller      6] ImportError: cannot import name 'parseWsUrl'

No, not your fault .. we need to do release Crossbar.io now that new versions of AutobahnPython and txaio are out. Yeah, sucks, I know. We soonish publish a proper release schedule, and synchronize releases upcoming ..

@gregkeys I'll try this tomorrow. Interesting .. reducing image size is welcome!


Add heartbeat and zk-connected metrics in defaultMetrics.json

Fixes #16148

Description

I have updated https://github.com/apache/druid/blob/master/extensions-contrib/prometheus-emitter/src/main/resources/defaultMetrics.json this file and added the zk-connected and service/heartbeat metrics.

Release note

Improved: Prometheus-emitter extension now emits service/heartbeat and zk-connected metrics.

This PR has:

[ ] been self-reviewed.
[ ] added documentation for new or modified features or behaviors.
[X] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[X] been tested in a test Druid cluster.

Congrats on your first contribution to Druid, @tapojit047 !!


Channel.exchange_declare: Unexpected method frame

Steps to reproduce:
Notice auto_delete changes.
    channel.exchange_declare(exchange,
                          "fanout",
                          false, // pssive
                          false, // durable
                          false, // auto_delete
                          false, // internal
                          false, // nowait
                          Table::new())
        .unwrap();
    channel.exchange_declare(exchange,
                          "fanout",
                          false, // pssive
                          false, // durable
                          true, // auto_delete
                          false, // internal
                          false, // nowait
                          Table::new())
        .unwrap();

Panics with: AMQP Error: Unexpected method frame: channel.close, expected: exchange.declare-ok.
I'd expect exchange_declare to tell me that exchange is already declared without auto_delete feature.

The current implementation is not handling channel errors well. The only expected response from these kind of calls is a corresponding OK method, like exchange.declare-ok, I guess it should also expect channel.close, and return it as an error.
Although, for this particular error it is strange that the server returns an error, because in the protocol specification it says that the server MUST ignore the auto-delete field if the exchange already exists.


rocm3.5 - Could not find a package configuration file provided by "rocblas"

Vanilla rocm3.5:
$ sudo cmake -P install_deps.cmake
...
$ CXX=/opt/rocm/llvm/bin/clang++ cmake -DMIOPEN_USE_COMGR=On -DBUILD_DEV=On -DCMAKE_BUILD_TYPE=debug -DMIOPEN_TEST_FLAGS=--disable-verification-cache ../..
-- The C compiler identification is GNU 7.3.0
-- The CXX compiler identification is Clang 11.0.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /opt/rocm/llvm/bin/clang++
-- Check for working CXX compiler: /opt/rocm/llvm/bin/clang++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.29.1")
-- Checking for module 'sqlite3'
--   Found sqlite3, version 3.17.0
-- Found BZip2: /usr/local/lib/libbz2.a (found version "1.0.6")
-- Looking for BZ2_bzCompressInit
-- Looking for BZ2_bzCompressInit - found
-- Performing Test HAS_HIP
-- Performing Test HAS_HIP - Success
-- Performing Test HIP_CLANG_SUPPORTS_PARALLEL_JOBS
-- Performing Test HIP_CLANG_SUPPORTS_PARALLEL_JOBS - Success
-- Hip compiler flags:  -x hip --hip-device-lib-path=/opt/rocm/lib -mllvm -amdgpu-early-inline-all=true -mllvm -amdgpu-function-calls=false   -D__HIP_ROCclr__=1 -isystem /opt/rocm-3.5.0/hip/../include -isystem /opt/rocm/
llvm/lib/clang/11.0.0/include/..  -D__HIP_PLATFORM_HCC__=1  -D__HIP_ROCclr__=1 -isystem /opt/rocm-3.5.0/hip/include -isystem /opt/rocm/include --hip-device-lib-path=/opt/rocm/lib --hip-link
-- hip compiler: /opt/rocm/bin/clang-ocl
CMake Error at CMakeLists.txt:205 (find_package):
  Could not find a package configuration file provided by "rocblas" with any
  of the following names:

    rocblasConfig.cmake
    rocblas-config.cmake

Fixed after
$ sudo apt-get install rocblas
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
  rocblas
0 upgraded, 1 newly installed, 0 to remove and 817 not upgraded.
Need to get 71.9 MB of archives.
After this operation, 3219 MB of additional disk space will be used.
Get:1 http://repo.radeon.com/rocm/apt/debian xenial/main amd64 rocblas amd64 <IP_ADDRESS>7-b2cceba [71.9 MB]
Fetched 71.9 MB in 19s (3881 kB/s)
Selecting previously unselected package rocblas.
(Reading database ... 225138 files and directories currently installed.)
Preparing to unpack .../rocblas_<IP_ADDRESS>7-b2cceba_amd64.deb ...
Unpacking rocblas (<IP_ADDRESS>7-b2cceba) ...
Setting up rocblas (<IP_ADDRESS>7-b2cceba) ...

Rocbals <IP_ADDRESS>7-b2cceba released for ROCm 3.5.0, https://github.com/ROCmSoftwarePlatform/rocBLAS/releases/tag/rocm-3.5.0

Nobody else complains.


Added Technical University of Munich: tu-muenchen

@Strohgelaender Could you please rename lib/domains/de/tu-muenchen to lib/domains/de/tu-muenchen.txt to reflect this repository rules?

@Strohgelaender I merged the pull request and renamed the file for you. Please note that failing to comply the repositiory rules results in delayed request processing.


Feature Request: clone/get .deta folder only

People who use git to store/backup/track their codebase can have some additional files for development. But they aren't pushed to deta micro using .detaignore. In this case, I clone the project from deta, clone the main codebase from git services and then copy the .deta folder to my git cloned codebase. It would be nicer if I could just get the .deta folder to identify the project to deta.

thanks for the feedback. we will make sure to support this in our next cli release. cc @rohanshiva

sorry @ArnabXD  closing the issue since the repository is now being archived. You can contact us on discord or by email if you have any questions/concerns.


[Bug]Skipping an iteration of the puzzle (no connected peers)  DEBUG snarkos_node_router::handshake: Connecting to <IP_ADDRESS>:4130... 2024-07-03T02:16:36.575195Z  WARN snarkos_node_router: Unable to connect to '<IP_ADDRESS>:4130' - '<IP_ADDRESS>:4130' disconnected before sending "Message::ChallengeResponse"

🐛 Bug Report

how to solve?

+1

Duplicate of https://github.com/AleoNet/snarkOS/issues/3340.


Fix scalar axes in lowering reduction operations

The lowering of reduction operations failed when axes is a scalar tensor, i.e. tensor<i64>.
This patch fixes such a situation and adds a lit test for that.

Jenkins Linux amd64 Build #15483 [push] Fix scalar axes in lower... started at 10:18

Jenkins Linux ppc64le Build #14513 [push] Fix scalar axes in lower... started at 11:29

Jenkins Linux s390x Build #15490 [push] Fix scalar axes in lower... started at 11:18

Jenkins Linux amd64 Build #15483 [push] Fix scalar axes in lower... passed after 1 hr 21 min

Jenkins Linux s390x Build #15490 [push] Fix scalar axes in lower... aborted after 1 hr 46 min

Jenkins Linux ppc64le Build #14513 [push] Fix scalar axes in lower... aborted after 1 hr 46 min


Example only works with non-default dependabot.yml

The example in the README assumes that you configure dependabot.yml like you do by specifying that dependabot should use the dependabot label on PRs.
By default dependabot simply uses dependencies as a label, and that would be a saner default imo.
I was surprised when I used this action and no changelog entry happened when I used this example, and turns out that the default activationLabel makes it silently ignore any dependabot.yml configuration that doesn't explicitly configure dependabot to use the label you specify as default for activationLabel.

Hi @CramBL
Thanks for opening the issue. You are correct that Dependabot doesn't add dependabot as a label by default. I avoided using dependencies as other possible dependency management tools that could also add that label. I see dependencies more as a category and not a good label indicating a workflow to be run. The whole point of the configuration capability is to avoid most of those opinions anyway.
You make a great point that the example makes several assumptions and leads to a new user scratching their heads as to why it didn't "just work". I'll update the example to call out using the dependencies label so that at least copy-and-paste scenarios should work easily if you think that is a reasonable step.
I'm hesitant to update the default as it could cause other users to have broken workflows.

I think updating the example would be very helpful yes.
It's a shame that you don't think dependencies is a sane default because that means a lot of additional boilerplate for dependabot.yml (one per package-ecosystem).
I still think it would be a sane default, and users who want it to only run on the 'dependabot' label could just use the activationLabels to achieve that.
For us the workflow is already guarded by if: github.actor == 'dependabot[bot]' so this is just redundant configuration.

a lot of additional boilerplate for dependabot.yml (one per package-ecosystem).

Instead of updating your Dependabot config, you can easily update the activatoinLabels here to use dependencies. Is that not sufficient?

a lot of additional boilerplate for dependabot.yml (one per package-ecosystem).

Instead of updating your Dependabot config, you can easily update the activatoinLabels here to use dependencies. Is that not sufficient?

Yes it's acceptable thanks.

It may be that in a future version, the best approach is to remove the "activation" concept from this action entirely and allow authors to use if as needed to decide when to run an action. However, it does require reading and parsing labels.
Another approach may be that

if the author is dependabot
and the activationLabels is either empty or the default
run the action

That I think preserves the current default without breaking other workflows while getting a bit closer to your desired end goal of running the action with no config


Paths ~ not being recognized

Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
App
Expected behavior
No issues should have occured as I didnt even touch the codebase. Just created app using the cli.
Actual behavior
It is showing this error, and I posted on the shopify community and seems like a lot of other people are facing this problem recently.

Verbose output
I was able to create a temporary fix by manually setting the paths from ~ to ./

Reproduction steps

No idea

Operating System
Mac Os
Shopify CLI version (check your project's package.json if you're not sure)
3.63
Shell
No response
Node version (run node -v if you're not sure)
No response
What language and version are you using in your application?
No response

This isn't a CLI issue, so will close here, but feel free to drop a question on the support forum or the remix template repo if you think its a bug.
I think what you're seeing though is that shopify.server.js is a JS file: it's ran as-is through node, and isn't processed by Typescript. So, the configuration in tsconfig.json to support ~ style paths doesn't come into things.


[Issue 222:] Version changed from v0.4.2 to v0.5.0 in documents

Signed-off-by: Aparna<EMAIL_ADDRESS>Change log description

Version number should be changed from 0.4.0 to 0.5.0
documentation/streaming.md (Section streamcuts)

Format fix:
documentation/batch.md (section: input streams, stream cuts)

Purpose of the change
Issue #222

@vijikarthi requested for review


Feature: Added advanced options when creating archives

Resolved / Related Issues
This pr improves the creation of compressed archives. Closes #10327.

The dialog has been redone and adds additional possibilities. I tried to stay practical and not technical.
The context menu has been unified with the decompression and offers additional possibilities.
There is no more distinction of the name of the archive according to whether one compresses an element or several.
The progress percentage indication is updated at each percent. Previously, it could never be updated for a large folder.
The compression code has been moved and is no longer static. It should be moved to the backend but the library needs to be updated first.

Validation
How did you test these changes?

[x] Built and ran the app
[ ] Tested the changes for accessibility

Screenshots (optional)

I have a couple of questions,

Is there a reason for having compress when selecting an existing archive?
Can the split options be hidden when they aren't available?
Icons in the dialog aren't necessary, can they be removed?
What does the "Do not compress" option do?
Using x:Load instead of Visibility is a good idea and will automatically fix the extra space at the bottom of the dialog.

I changed the rule. The Compression part no longer appears when the selection is a single file which is an archive.
It's changed.
Icons have been removed.
There are 6 levels of compression (Ultra, High, Normal, Low, Fast, None). Currently, the compression is always in Ultra. It's handy to have None because it's very fast to store many files in one to transfer or store. Media files or documents are often already compressed but recompressing them is slow and unnecessary. I put back the 6 levels because, after testing, the Ultra level is very slow (40 minutes for my extreme test, against 4 minutes in Normal). The default mode, including without going through the dialog, is Normal because it is the best compression/speed compromise. We can optionally not display each level for simplicity (Ultra/Normal/Fast/None).
The extra space was intentional to avoid changing the window size. I changed the behavior to no longer have that empty space. I applied x:Load.

Will do the review tomorrow. I'm running out of bandwidth today :)

A few minor edits.

Opening the expander now automatically gives focus to the password field.
Combobox are too short for SplittingSize texts. To correct this problem, the combobox no longer displays the media when it is closed. The size becomes a minimum size because the "Do not split" text will be longer in many languages and it was truncated.

Other available formats are GZip, BZip2, Tar and XZ. I don't really know these formats but only Tar is managed by Files in decompression.

Other available formats are GZip, BZip2, Tar and XZ. I don't really know these formats but only Tar is managed by Files in decompression.

Can we add support for rar and tar?

You can add tar easily. For rar, you need another library, but also check the licenses. Free compression/decompression softwares doesn't seem to support it.

Let's support whatever the current library supports 👍

I added support for other formats for compression. These formats do not support splitting. I will try to add them also in unzip (including browse content). This will be done in another pr.

I disabled the other formats. bzip2 and xz don't seem to keep filenames. gz and tar do not support passwords. To decide what to do.


WIP: control plane pool

Can somebody help me? I can't figure out how to fix this problem: control_plane.tf:19

@HenkVanMaanen This is the line that does the automatic subnet generation, and in there, we can take into account the control_plane nodepools too.
https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/blob/4e3f34585e89813d2c268b9fb68f03c7ae9e5a5d/locals.tf#L191
The first subnet is reserved for anything; the second was for the control plane nodes, but now, I suggest we keep that only for the first control plane node, and before the agent nodepools subnets, we generate ones for the control planes nodepools.
The hcloud network describe k3s command is very useful for debugging subnets.

Supplanted by https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/pull/149.


[Feature] support view delta join mv rewrite (#18146)

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #17338
Problem Summary(Required) ：

support view delta join mv rewrite
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

run starrocks_fe_unittest


Ranks not Syncing

So, I have 4 servers connected to the same database and they are called: lobby, practice, skyblock, factions. The thing is when I do for example /lp user AleXutzZu parent set owner the rank is applied only on lobby, practice and skyblock but on factions I remain the same. Why?

Check, if BungeeCord and IP-forwarding are enabled on your Factions-Server.
You can find this options in the spigot.yml

Thanks!


Creating dataset

How you collected the dataset for training image classifier ? Or is it classifier model ? can you share your dataset ?

Want dataset as well :)

Actually not at all, I can create my own dataset but ı dont know the point of this dataset.


cleartext traffic?

My scanner just received an upgrade, and today reported on your updated APK:
! repo/net.primal.android_80.apk declares flag(s): usesCleartextTraffic
! repo/net.primal.android_80.apk declares risky permission(s): android.permission.CAMERA

While camera access was easy to figure (QR code scanner): does primal really use unencrypted connections (like HTTP instead of HTTPS)?

Primal is using HTTPS and WSS.
The flag usesCleartextTraffic was enabled in this commit https://github.com/PrimalHQ/primal-android-app/commit/73333c3c because there are many images in notes which are using http links.
Do you have any suggestions how to handle this http links from notes better?

Yes! Primal Android app needs access to camera for QR code scanning. At the moment, we are using camera only in Primal Wallet and in the up-coming weeks/months we plan to add support for scanning nostr profile QR codes as well.
This is our camera permission rational displayed to the users:
<string name="qrcode_scanner_permission_rationale">Primal needs access to your camera to scan QR codes.</string>
We are using zbar native library for QR code detection on purpose to avoid use of Google ML libraries. And speaking of Google, we have two product flavours google and aosp. The main difference is that in google flavour we have integrated Google Play Billing lib because of in-app purchase (for sats in wallet), while the aosp flavour is free of Google.
I see that you are part of F-Droid organisation - do you plan to add Primal Android app on F-Droid?

I see that you are part of F-Droid organisation - do you plan to add Primal Android app on F-Droid?

No, not that – I'm currently not active at F-Droid, and it doesn't look like that will change soon. Your app is listed in my repo, which has some enhancements you won't find on F-Droid (see e.g. here and, for the latest enhancements, here). So if you have any questions on that, feel free to ask!
And should you want another question/suggestion from me: how about establishing Fastlane structures, so you can maintain your app's description, graphics etc. and my updater pulls them along with each release? Be welcome to use my Fastlane Cheat Sheet for guidance with that. And to pick a badge if you want to link to your app's presence in my repo, e.g. from the README :smiley:
Thanks for your help! All "red flags" have been cleared now (both above items have been added to your app's allow-list here, together with their explanation), so this issue is solved. Feel free to ping me from another one e.g. with Fastlane; if you wish, I can provide you what is set up here via PR as a starter package for that.

Thanks for the clarification. Do you use the provided APK from the release page or are you building the app from the source?  My main concerns is that only officially released builds/versions (available on releases page) are published to other stores such as yours or F-Droid. We don't want intermediate alpha builds to end-up publicly available to avoid breaking-changes (which happens often on alpha builds).
We are currently in need for setting up a publishing process aosp builds on F-Droid; we had dozens of requests for that already. If the process is the same for your repo and F-Droid, maybe you could help us to setup for both of them?

Thanks for the clarification.

Gladly!

Do you use the provided APK from the release page

Yes.

only officially released builds/versions (available on releases page) are published to other stores such as yours or F-Droid

That's what happens with my repo. For F-Droid this would require establishing reproducible builds.

 With very few exceptions (where my repo explicitly hosts "test builds" and the app description or name/title make that clear, I don't want such things to enter my repo either :wink:)

If the process is the same for your repo and F-Droid

No, the process is different. I just pick the APKs "provided officially" (and apply a bunch of tests on them to make sure as best as I can they are "good"), F-Droid insists on building from source themselves.
What I however could help you with is establishing Fastlane structures (which F-Droid will want in your app's repo, too) – by opening a PR with what is currently set up with my repo for the very same purpose. You could then build upon that "starter package" and use my Fastlane Cheat Sheet for reference.

Great! Sure, let's do it like that. Thanks!

Done – your turn now :smiley:

Merged, thanks! Can you point me what are the next stops to get the build visible in F-Droid as well?

Merged, thanks!

Gladly! Unfortunately I messed that one up a little (opened another PR to fix that, which then somehow included one of your commits despite me pulling main first afresh, no idea…)

Can you point me what are the next stops to get the build visible in F-Droid as well?

For F-Droid.org?

Quick path to get your app into F-Droid
the process of integrating an app with F-Droid

Sorry to bother you again. All new features on my scanner are now enabled, so more things show up:

android.permission.READ_EXTERNAL_STORAGE*: The * states this permission was granted implicitly as WRITE_EXTERNAL_STORAGE is requested. Can you please let me know what for these two are needed? The app description is still a bit short and gives no clue :wink:
DEPENDENCY_INFO_BLOCK was found in your signing blocks. That's an encrypted BLOB only Google can read.

Remedy for the latter:
android {
    dependenciesInfo {
        // Disables dependency metadata when building APKs.
        includeInApk = false
        // Disables dependency metadata when building Android App Bundles.
        includeInBundle = false
    }
}

For some background: that BLOB is supposed to be just a binary representation of your app's dependency tree. But as it's encrypted with a public key belonging to Google, only Google can read it – and nobody else can even verify what it really contains.

Hey @IzzySoft,
In the latest release we added a feature to Save Image to device. The entire feature was implemented in this commit fb9849b7dc9843c11d0aa417d88502f2592d322a. I had to add permission "WRITE_EXTERNAL_STORAGE" to be able to do this on devices up to SDK 28. You can check MediaDownloader to see what are we actually doing when writing.
Regarding READ_EXTERNAL_STORAGE, we don't use that at all. The only reading from device we do is when attaching a picture to a Note/Post, but this was implemented and released months ago.
I was not aware of 'DEPENDENCY_INFO_BLOCK' before; And not sure why is included now in the builds. Maybe because the project was upgraded to AGP 8.2.2. I'll add your snippet to build.gradle to disable the BLOB for the next releases.

a feature to Save Image to device

Thanks! That explains, and thus justifies. Though as the minimum Android version you support is 8.0, wouldn't SAF (Storage Access Framework) suffice for that? That is, unless you want to save to some media or download directory. But it looks like you already checked that and something didn't work (or the library used for that requires it)?

Regarding READ_EXTERNAL_STORAGE, we don't use that at all.

If you want to get rid of it, you can use tools:node in your AndroidManifest.xml to achieve that – see Removing Unwanted Manifest Permissions With tools:node. Should removing the permission turn out to have side-effects, I'd simply add it to the allow-list here with your proper explanation.

I was not aware of 'DEPENDENCY_INFO_BLOCK' before

Few are.

And not sure why is included now in the builds.

It was probably there all along. It just turned up here now that I added a check for that. Android Studio and IntelliJ IDEA (maybe more, but for those 2 I know) add it automatically when signing. So the above block for your build.gradle tells them not to. Or you use apksigner for signing which generally doesn't include that.

I'll add your snippet to build.gradle to disable the BLOB for the next releases.

Wonderful, thanks! :star_struck:

Oof. Today's report has:
! repo/net.primal.android_98.apk declares sensitive permission(s):
  android.permission.READ_PHONE_STATE android.permission.READ_EXTERNAL_STORAGE*

So now READ_PHONE_STATE was added to the list. Can you please tell me what for?

I dont remember adding that or using it at all. I'll investigate.

I have upgraded compose and some other libs and some of them has added the permission.
I'll remove it manually just like we did for the READ_EXTERNAL_STORAGE.
Thanks for reporting it.

Thanks for your reports. Keep them coming!


Rename ofDefault() to of()

We have static factory methods that return the default implementation of the interface/class.
We named most of the methods of() but a few called ofDefault().
It would be nice if we rename those ofDefault() to of() so that we have a consistent name through our API.

Can I try on this issue?

It is scheduled for 2.0.0, so we are not going to change them in the near future.

Thank you for your comment!


Add ability to build without interacting with available artifact servers

See original issue on GitLab
In GitLab by [Gitlab user @tpollard] on Nov 7, 2018, 14:19
Background
As mentioned in #buildstream, it seems reasonable that bst build should have the option to build everything from source without attempting to reach out to remote caches, especially when consuming junctions in which artifact servers are defined and populated for pulling or have the opportunity to be pushed to. One example of wanting to do this would be to periodically check that a whole project can be built from 'scratch'.
Task description

Add a cli option (could be main or specific to build) and a user config context option with a suitable name for this behaviour override
Use cli/context/app as appropriate to consume this during streams creation of queues, emitting pull and push queues as appropriate
Add tests

Acceptance Criteria
Testable and does not break normal behaviour

In GitLab by [Gitlab user @tpollard] on Nov 7, 2018, 14:21
The use of overriding fetchers and pushers to 0 to handle this situation is also a possibility

In GitLab by [Gitlab user @tpollard] on Nov 7, 2018, 14:22
changed the description

In GitLab by [Gitlab user @toscalix] on Nov 12, 2018, 09:46
assigned to [Gitlab user @tpollard] and [Gitlab user @valentindavid]

In GitLab by [Gitlab user @tristanvb] on Nov 12, 2018, 09:46
A constant goal is to have a minimal API surface at all times, so when deciding on what new things to expose to the CLI and user configuration, we should also consider:

People may want to ignore project configured artifact caches and only observe user configured artifact servers
People may want to specify a single URI to override all artifact server URIs

In GitLab by [Gitlab user @tpollard] on Nov 12, 2018, 11:02
I agree both of those situations are definitely worth considering [Gitlab user @tristanvb], good points

In GitLab by [Gitlab user @tpollard] on Nov 12, 2018, 13:17
I'm going to give this a go whilst I have some overhead time

In GitLab by [Gitlab user @tpollard] on Nov 13, 2018, 17:17
mentioned in merge request !950

In GitLab by [Gitlab user @tpollard] on Nov 13, 2018, 17:19
I've submitted a WIP for this, it's lacking new tests but I hope it captures the original idea and further scope raised

In GitLab by [Gitlab user @tpollard] on Nov 13, 2018, 17:45
I think as a follow up from this it would be beneficial to add a --remote option to build, to match pull & push to specific remotes

In GitLab by [Gitlab user @tpollard] on Nov 14, 2018, 17:52
I've taken !950 out of WIP

In GitLab by [Gitlab user @toscalix] on Dec 17, 2018, 09:29
marked this issue as related to #477

In GitLab by [Gitlab user @tpollard] on Feb 5, 2019, 10:28
At the recent BuildStream gathering we discussed possible use cases involving limiting certain actions to certain remotes. As a base this MR coves splitting all actions into user, project, or no remotes. This should be extended to differentiate project remotes into the top level project and sub projects (i.e junctions). It would also be beneficial to potentially have more granular control over remotes per command type (i.e push/pull, specifying different config for each)


Missing system and content folders for checksum calculation

Missing system and content folders for checksum calculation. Should exist at current server folder. Follow log excerpt:
WARNING: Cannot open file  system/data/surfaces.ini  for checksum, the client check will be disabled for this file
Initializing checksum for track config indy
WARNING: Cannot open file  content/tracks/ks_brands_hatch/indy/data/surfaces.ini  for checksum, the client check will be disabled for this file
Calculating checksum for folder content/cars/ks_abarth500_assetto_corse/data
Error calculating checksum: Folder not found

I merged your pull request and fixed some issues with it. Can you test if this bug still exists and if your new parameters work? To do that, you can pull the latest docker image.

Should work now.
i'm still having trouble trying to run acServer with parameters.
I moved ini creating inside /config/, will request a pull by tomorrow.

Is this working now?

Yes it is. :+1:


Android Device reconnect exception

For each instance of the IDevice on Android side there is an event subscription. This is done using a Lambda. It is preferred to hook this up to an actual delegate for the purpose of event unsubscription. In the constructor of the Device the DeviceConnected event is being subscribed to. In case of disconnect, this event is not cleared. So the next time the same device is discovered and connected to, previous event gets the first notification. The GATT instance in the initial IDevice is null so it throws an exception.

I submitted a pull request to fix that here: https://github.com/xamarin/Monkey.Robotics/pull/41

Thanks Steven. Just one more issue I forgot to mention on the issue I created. Upon triggering the connection made event, the returned device is added to the list of connect devices. However it is never removed. I don't use that list since my app will only talk to one designated BLE device at a time.

I submitted a pull request to address ConnectedDevices property: https://github.com/xamarin/Monkey.Robotics/pull/43


Update google-cloud-bigquery to 2.9.1

Updates com.google.cloud:google-cloud-bigquery from 2.9.0 to 2.9.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-bigquery" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #189.


fix typo in error message: s/piping it a function/piping it to a func…

…tion/

There were some weird notes in GitHub about pressing the merge button, I think because the ongoing work in the compiler has led to a bunch of files getting moved around. I just added the missing word locally to get the fix instead. Thank you for pointing this one out!


Finish 'documents' model

Finish implementing 'documents' model
Looks like the current 'documents' model is only a placeholder, as it only has a filename column

'documents' needs to be finished to actually store a file in the database
Make new documents model polymorphic (see 'notes' table for reference)

Add documents to incurred incidentals

Remove the 'incurred_incidentals_documents' table
Use new polymorphic 'documents' instead

are we actually storing in the database or are we using the filesystem, this line

seems to suggest that we are actually sotring directly in the file system, which unless im mistaken is probably the better way to store images (source, facebook whitepage


[FEATURE]Side Note Section for Every Rem .

Is your feature request related to a problem? Please describe.
-This might be an alternative to limitation of not being able to zoom into rem .
-Will enable to view other documents that might be necessary to view . But, are in a different document .
Describe the solution you'd like
Have a Side note section like for videos.It can be a portal stored just below the rem tagged with Incremental tag

I have been using portals for this

Currently doing that + /columns((c2,c3,etc.)).


Unable to build docs on node 12 (windows) due to incompatible graceful-fs package dependency

Steps to reproduce
Steps to reproduce the behavior:

On windows, have node 12.18 installed.
npm run docs
receive ReferenceError: primordial is not defined. error

Expected behavior
Docs should build
Current behavior
><EMAIL_ADDRESS>docs d:\Data\_CodeOS\LEaflet-jafin
> node ./build/docs.js

Building Leaflet documentation with Leafdoc ...
fs.js:35
} = primordials;
    ^

ReferenceError: primordials is not defined
    at fs.js:35:5
    at req_ (d:\Data\_CodeOS\LEaflet-jafin\node_modules\natives\index.js:143:24)
    at Object.req [as require] (d:\Data\_CodeOS\LEaflet-jafin\node_modules\natives\index.js:55:10)
    at Object.<anonymous> (d:\Data\_CodeOS\LEaflet-jafin\node_modules\sander\node_modules\graceful-fs\fs.js:1:37)
    at Module._compile (internal/modules/cjs/loader.js:1138:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
    at Module.load (internal/modules/cjs/loader.js:986:32)
    at Function.Module._load (internal/modules/cjs/loader.js:879:14)
    at Module.require (internal/modules/cjs/loader.js:1026:19)
    at require (internal/modules/cjs/helpers.js:72:18)

Environment

Leaflet version: 1.7.1
Browser (with version): N/A
OS/Platform (with version): Windows 10,  Node 12.18

Additional context
The version of graceful-fs that is resolved is not compatible with node 12.
See this for reference https://stackoverflow.com/questions/55921442/how-to-fix-referenceerror-primordials-is-not-defined-in-node
One suggestion is to add a resolutions section to package.json and pin the minimum version of graceful-fs.

Ok, confused,  I see the CI server is node14.  Is this a personal config issue?  Does anyone else on Windows node 12 build docs ok?

fixed by #7263

Superseded by #7263


[$500] Distance: "Add a Stop" Functionality Overwrites "Finish" Location

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
1 Click on + > request money > distance
(Eg: I want to go from Dallas to Houston but stop at Austin for 2 hours)
2 On start, choose start location (Eg Dallas)
3 On Finish, choose finish location (Eg Houston)
4 Click on Add a stop button
5 Verify that location on Finish is transferred to stop while my intention was to add another location (Eg Austin) as stop
Expected Result:
Clicking on "Add a Stop" should open a separate input field for specifying a stop location, allowing users to enter a different location without affecting the "Finish" location.
Actual Result:
Actual Result:
The location entered in the "Finish" field is automatically copied to the "Stop" field,
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.66-3
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/38435837/7b519a18-2435-4cf5-baaa-97e7c3e6e49b
https://github.com/Expensify/App/assets/38435837/43069f5f-3081-442b-b601-bd83b0b313eb
Expensify/Expensify Issue URL:
Issue reported by: @avi-shek-jha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693877482555529
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~013b3c43bd61f16cc5
Upwork Job ID:<PHONE_NUMBER>195997184
Last Price Increase: 2023-09-08

Proposal
Please re-state the problem that we are trying to solve in this issue.
[Distance]: The finish address move to the stop waypoint when adding the stop
What is the root cause of that problem?
https://github.com/Expensify/App/blob/f67bf9f88c08fe2cebe16202d1244299a9640ea3/src/libs/actions/Transaction.js#L52-L57
Currently, in the beginning, waypoint1 will be the start point, waypoint2 will be the finish point. When creating a new stop point, start and finish point will remain. The stop point will be waypoint3.
https://github.com/Expensify/App/blob/f67bf9f88c08fe2cebe16202d1244299a9640ea3/src/components/DistanceRequest.js#L187-L196
But while displaying, the finish point is the last point (in this context, last point is waypoint3). It caused this bug
What changes do you think we should make in order to solve the problem?
When adding a new stop point we should change the index of the finish point to the last point. In this context, when adding new stop point, this stop point will be waypoint2 and the finish point will move to waypoint3.
https://github.com/Expensify/App/blob/f67bf9f88c08fe2cebe16202d1244299a9640ea3/src/libs/actions/Transaction.js#L52-L57
[`waypoint${newLastIndex-1}`]: null,
[`waypoint${newLastIndex}`]: lodashGet(existingWaypoints, `waypoint${newLastIndex-1}`, {})

Result
https://github.com/Expensify/App/assets/141406735/fa9f16c1-3996-4f78-b1b9-bd4eb8627f75
What alternative solutions did you explore? (Optional)
In case, we want to remain the finish point always be waypoint 2.
https://github.com/Expensify/App/blob/f67bf9f88c08fe2cebe16202d1244299a9640ea3/src/components/DistanceRequest.js#L187-L196
we can update here to display the finish point when index is 1
With this approach, the stop point will display below the finish point
<img width=“987” alt=“Screenshot 2023-09-05 at 12 04 20" src=“https://github.com/Expensify/App/assets/141406735/34e5cc57-f28b-4bb6-9acf-52c1cd5af3be”>

Proposal for Distance: "Add a Stop" Functionality Overwrites "Finish" Location #27073
Problem Statement
When a user clicks on "Add a Stop," the location entered in the "Finish" field is automatically copied to the "Stop" field. The expected behavior is that clicking on "Add a Stop" should open a separate input field for specifying a stop location, allowing users to enter a different location without affecting the "Finish" location.
Root Cause
The root cause appears to be in the way the waypoints are managed in the code. Specifically, the code uses an index-based system to determine which waypoint is the start, finish, or a stop. When a new stop is added, it becomes the last waypoint, causing the finish point to shift.
Proposed Solution 1: Update Finish Point Index
Code Changes
In Transaction.js, modify the existing code block:
Onyx.merge(`${ONYXKEYS.COLLECTION.TRANSACTION}${transactionID}`, {
    comment: {
        waypoints: {
            [`waypoint${newLastIndex}`]: {},
        },
    },
});

to:
Onyx.merge(`${ONYXKEYS.COLLECTION.TRANSACTION}${transactionID}`, {
    comment: {
        waypoints: {
            [`waypoint${newLastIndex-1}`]: null,
            [`waypoint${newLastIndex}`]: lodashGet(existingWaypoints, `waypoint${newLastIndex-1}`, {})
        },
    },
});

This ensures that the finish point's index is updated to be the last index, and the new stop takes the second last index.
Alternative Solution 2: Update Display Logic
Code Changes
In DistanceRequest.js, modify the existing code block:
if (index === 0) {
    descriptionKey += 'start';
    waypointIcon = Expensicons.DotIndicatorUnfilled;
} else if (index === lastWaypointIndex) {
    descriptionKey += 'finish';
    waypointIcon = Expensicons.Location;
} else {
    descriptionKey += 'stop';
    waypointIcon = Expensicons.DotIndicator;
}

to:
if (index === 0) {
    descriptionKey += 'start';
    waypointIcon = Expensicons.DotIndicatorUnfilled;
} else if (index === 1) {
    descriptionKey += 'finish';
    waypointIcon = Expensicons.Location;
} else {
    descriptionKey += 'stop';
    waypointIcon = Expensicons.DotIndicator;
}

This ensures that the finish point is always displayed when the index is 1, regardless of how many stops are added.
Conclusion
Both solutions aim to resolve the issue of the "Add a Stop" functionality overwriting the "Finish" location.

@m-natarajan Please change the "Reported by" to me as i reported this first.
https://expensify.slack.com/archives/C049HHMV9SM/p1692939840341009

Proposal
Please re-state the problem that we are trying to solve in this issue.

When you add a stop waypoint to a route, the 'Finish Waypoint' becomes the 'Stop' waypoint

What is the root cause of that problem?

The main reason for this problem is that when you add a stop waypoint, a new empty index is generated. As a result, this new waypoint, which is the last one in the list, is mistakenly shown as both the 'Finish Waypoint'.

What changes do you think we should make in order to solve the problem?

To resolve this we can create a function to handle the changes in the index of the waypoints.

function generateStopWaypoint(waypoints) {
    const finishIndex = _.size(waypoints);
    const prevFinishIndex = finishIndex - 1;
    const newStopIndex = finishIndex - 1;

    const finish = lodashGet(waypoints, `waypoint${prevFinishIndex}`, null);

    return {
        [`waypoint${newStopIndex}`]: null,
        [`waypoint${finishIndex}`]: finish,
    };
}

Then calling it in addStop function

function addStop(transactionID) {
    ...rest
    comment: {
        waypoints: generateStopWaypoint(existingWaypoints),
    },

What alternative solutions did you explore? (Optional)

N/A

@burczu bump for review on these proposals thanks

@laurenreidexpensify I'll review it soon - I was ooo due to RNEU conference and now I'm kinda flooded by issues.

Ok, proposals reviewed - all of them looks quite the same but @DylanDylann (link) was first and I think we should choose it because this solution looks correct (main one - the optional one seem to not working).
🎀 👀 🎀 C+ reviewed

@neil-marcellini looks like this is related to distance requests, so assigning to you

@neil-marcellini @burczu This is the latest discussion we still want to display the stop waypoint between the start point and finish point. So this issue should be fixed

@neil-marcellini bump

Sorry for the delay. This is going to be fixed by https://github.com/Expensify/App/issues/26888 so I think we should close it. I don't think we need to pay for the bug report because we were already aware of the behavior and it was the intended behavior before we decided to change the UX.


do we need to resize image from the phone camera first?

looking at your demo code.  it looks like you take the picture directly from the phone camera and send it to caffe to predict.  in my case, the image size from my camera on nexus 6 is 4160 x 3120, but the crop size is only specified as 227.
so how does this work?  does caffe automatically resize the image from 4160 x 3120 to something smaller?
if there is no resize, how does the crop of 227 happens?  i assume that's 227 x 227.  is that in the middle of the image?
my concern is that if it's in the middle and there is no resize, then 227 x 227 is a really small part of the entire image of 4160 x 3120.
any comment is much appreciated.
10-28 15:43:55.966  26041-26041/com.sh1r0.caffe_android_demo D/MainActivity﹕ /storage/emulated/0/Pictures/Caffe-Android-Demo/IMG_20151028_154348.jpg
10-28 15:43:55.966  26041-26041/com.sh1r0.caffe_android_demo D/MainActivity﹕ 4160
10-28 15:43:55.966  26041-26041/com.sh1r0.caffe_android_demo D/MainActivity﹕ 3120

transform_param {
    crop_size: 227
    mean_value: 104
    mean_value: 117
    mean_value: 123
    mirror: false
}

just found this.  does that mean all images are resized to 256 x 256 before prediction?  for example, in my case, the phone original image 4160 x 3120 is resized to 256 x 256.  and then it takes a crop in the middle 227 x 227.  and then run the prediction on the 227 x 227 image?
int CaffeMobile::test(string img_path) {
	CHECK(caffe_net != NULL);

	Datum datum;
	CHECK(ReadImageToDatum(img_path, 0, 256, 256, true, &datum));
	const shared_ptr<MemoryDataLayer<float>> memory_data_layer =
		static_pointer_cast<MemoryDataLayer<float>>(
			caffe_net->layer_by_name("data"));
	memory_data_layer->AddDatumVector(vector<Datum>({datum}));

	vector<Blob<float>* > dummy_bottom_vec;
	float loss;
	clock_t t_start = clock();
	const vector<Blob<float>*>& result = caffe_net->Forward(dummy_bottom_vec, &loss);
	clock_t t_end = clock();
	LOG(DEBUG) << "Prediction time: " << 1000.0 * (t_end - t_start) / CLOCKS_PER_SEC << " ms.";

	const float* argmaxs = result[1]->cpu_data();
	for (int i = 0; i < result[1]->num(); i++) {
		for (int j = 0; j < result[1]->height(); j++) {
			LOG(INFO) << " Image: "<< i << " class:"
			          << argmaxs[i*result[1]->height() + j];
		}
	}

	return argmaxs[0];
}

Yes. These questions are more related to the Caffe core rather than this demo app. If you are curious about the underlying implementation, you can refer to Caffe repository.


Premature "GAME OVER"

Sometimes I experience a "Game Over" before reaching the top of the game-playing area. It seems to happen whenever clearing a row/several rows.

It seems like it might be happening whenever a way back to the bottom level is re-opened after being covered.

I think you have selected game mode B-type and have cleared all the lines. So game over here means you completed the game successfully (not obvious, I know).
Duplicate of #10

Yes, that was the problem, thank you. The difference between the modes is not at all obvious, maybe it could be (more clearly) documented or added directly into the game.


boards/sim: defconfig for hello_rust

Summary
Add defconfig for hello_rust based on sim:sh in:
boards/sim/sim/sim/configs/rust/defconfig

Config is based on sim:nsh configuration.
Impact
nothing as it's new
Testing
Simulation on MacBook Air M1 with Monterey 12.2.1 using rustc rustc 1.58.1 (stable) via Homebrew.

@ptka so, we don't need the special rust runtime?

For this very simple hello_rust, we don't  need any special rust runtime.
In fact there's a "core" crate, which is nearly independent from the OS itself and always available.
Additionally via extern "C", we build the needed rust runtime for this simple application.


Remove Perth course

Did you really mean to remove the bullets below the text?
I think that the result of removing line 18 in the diff result in that. I would revert line 18 and look at the site again.

Ha thanks @tsundberg! I think that has worked.

Looks good now!
On 17 May 2016 at 13:26, Theo<EMAIL_ADDRESS>wrote:

Ha thanks @tsundberg https://github.com/tsundberg! I think that has
worked.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/cucumber/website/pull/148#issuecomment-219690648

--
Thomas Sundberg
M. Sc. in Computer Science
Mobile: +46 70 767 33 15
Twitter: @thomassundberg
http://www.thinkcode.se/
Definition of done: Working software, in production.
Interested in a BDD Kickstart in Stockholm?
https://cucumber.io/events/bdd-kickstart-stockholm-16


Use cross-validation in training script to pick different hyperparameters for allele-specific models

Currently we're only doing this in the experiments directory. Each hyperparameter commandline flag should take a list of options, which are evaluated in all combinations for each allele-specific model.

This is now done, although not quite in the way suggested in this ticket


[confcom] Adding VN2 feature

Adding Virtual Node feature to generate security policies for AKS when using Confidential ACI as a "virtual node"

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
az confcom
General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

confcom


Undocumented .Net core dependencies when used with Windows Desktop

Steps to reproduce

Check out a the latest commit
Add a reference in a .Net Framework project to the only available .csproj in /src/, which is .Net Core

Actual behavior
It complains, within the .Net Framework project, that I need to install System.IO.FileSystem. When I actually try to run it, it throws an exception mentioning System.Security.Cryptography.Primitives. Since I was just quickly testing it, I'm not even sure I've caught everything that would need to be added.
Version of SharpZipLib
The only available version currently, that is, the .Net Core version. The .Net 4.5 project from 9ad7f5845222c99ff009a1daddedc051d25a06b9 (deleted in 661f813fd16e1afaaba5a6fba958a5b57a4fdd37) works fine.
Obtained from (place an x between the brackets for all that apply)

[x] Compiled from source
branch: master
commit: 2ea330b92d4f91295611f0c61047e7cedb8d2b83
[ ] Downloaded DLL from GitHub
[ ] Downloaded DLL from SourceForge
[ ] Downloaded DLL from _______
[ ] DLL included as part of
Package installed using:
[ ] NuGet
[ ] MyGet
[ ] Chocolatey

Yes, the current master on the repo is targetting netstandard (netstandard1.3 to be precise). Now NuGet support has landed for VS2015:
https://blog.nuget.org/20170815/Whats-nu-in-NuGet-with-VS2017-15-3.html#update-to-the-nuget-client-in-visual-studio-2015
This means future packages will be able to be added to almost any project type, given that the target framework supports netstandard1.3:
https://docs.microsoft.com/en-us/dotnet/standard/net-standard
Yes, this will cut off < 4.6 NETFW (for reference https://en.wikipedia.org/wiki/.NET_Framework_version_history#.NET_Framework_4.6)

I think I can shed some light onto the issue now that I changed the samples to require the latest alpha package:
https://github.com/dotnet/corefx/issues/15570
Fix (packages.config snippet):
<package id="System.Security.Cryptography.Algorithms" version="4.2.0" targetFramework="net461" />
<package id="System.Security.Cryptography.Primitives" version="4.0.0" targetFramework="net461" />
You need to revert Algorithms to 4.2 (from 4.3) and Primitives to 4.0 (from 4.3). Don't know of a more elegant solution yet.

https://github.com/dotnet/corefx/issues/23344

@christophwille when you added the reference to the .net standard project from the .net framework project, was the .net framework project using packages.config?  If so then you must ensure you add a dependency to the NETStandard.Library package (this happens automatically when using PackageReference).  Packages.config projects don't automatically flow their nuget dependencies so you always need to copy them.

@ericstj How it happened for me - the samples in https://github.com/icsharpcode/SharpZipLib/tree/master/samples/ICSharpCode.SharpZipLib.Samples were using direct project references (net461). I did the following:
deleted the project reference
installed the preview Nuget of SharpZipLib (alpha1)
This worked fine with the new Nuget 3.6 for VS 2015. Now, on running the sample (eg vb\viewzipfile) I got the loading exception on s.s.c.primitives (it expected 4.0). After some searching, I downgraded both s.s.c.algorithms (4.2; don't know if this even was necessary) and s.s.c.primitives to 4.0. Then it started working. https://github.com/icsharpcode/SharpZipLib/blob/master/samples/ICSharpCode.SharpZipLib.Samples/vb/viewzipfile/packages.config is "plain vanilla" except for the downgrades I manually performed.
What I don't understand is that the netstandard1.3 SharpZipLib binds against 4.0 in the first place.

On creating a repro, I did a blank 461 project. This worked. Maybe those samples/.vb|csproj files are weird because they are like 10+ years old (the only difference seems that it references netstandard 2.0 nuget instead of 1.6.1)

That sounds like you were missing bindingRedirects in the net461 project.
Whenever we have builds of a library (especially different API) we have to give the library its own unique version so that it can be safely installed to the GAC without downgrading someone else's app-local copy.  Even within the single package we have differing versions between different TFMs (like netstandard1.3 and netstandard1.4 or net461) if the API exposed is different for this reason.  The result is that almost always need to enable bindingRedirects for desktop projects consuming nuget packages.  You'd run into the same thing with third-party libraries that version their assemblies (like Json.Net).


add-verbosity-flag - Add a verbosity flag so that kaniko isn't so noisy all the time

Adds a verbosity flag and removes the default debug level (info is default built into kaniko command)
Validates the flag to prevent any injection issues

You also need to bump the image version in the workflow

I didn't spot that in the middle of a line - would it be better broken out as an env
ACTION_VERSION and then ${ACTION_VERSION} in the line?  Or no need?

@pinaki124 @mgoltzsche What's the next step in getting this out?  (sure I can merge it, but I can merge pretty much anything in this place - doesn't mean it's a good idea!)


core 3.5.0rc3 does not deliver the requirements.txt anymore

the viur-cli can no longer check the dependencies this way.
tox.ini is also missing

the viur-cli can no longer check the dependencies this way.
Fixed by https://github.com/viur-framework/viur-core/pull/862

tox.ini is also missing
Is this important? I wonder why they got into the old wheels.

Fixed by https://github.com/viur-framework/viur-core/pull/862


workbook is not getting streamed only worksheets can be streamed

 const data = [
    [{AACol:"AAVal"},{ABCol:"ABVal"}],
    [{BACol:"BAVal"},{BBCol:"BBVal"}],
    [{CACol:"CAVal"},{CBCol:"CBVal"}],
  ];

 const  workSheetNames = ["Test Sheet-1", "Test Sheet-2", "Test `Sheet-3"];

let workBook = XLSX.utils.book_new();
data.forEach((data, index) => {
let workSheet = XLSX.utils.json_to_sheet(data);
XLSX.utils.book_append_sheet(workBook, workSheet, workSheetNames[index])});

var stream = XLSX.stream.to_csv(workBook);
stream.pipe(process.stdout);

In this particular code I am not able to display workbook on my screen in even binary form
and if  I write it in a file the file is empty

Currently only CSV, HTML and JSON are supported, and they only work with a single sheet.  https://github.com/SheetJS/sheetjs/issues/1566#issuecomment-919288407 has more detail on XLS


Delete files when FileSets are deleted

Coverage remained the same at 100.0% when pulling bf60714bac93242b5e1393623259087652c2a472 on cleanup-files into 572a065015e85ee6b91646cb3f427cf901ea59a0 on master.

Makes sense to me!


Release Google.Cloud.Dlp.V2 version 2.0.0-beta03

Adds fields to Finding
Support for hybrid jobs

Had some errors /w the commit step of release, but I just went ahead and created the commit manually. If that step does more than just a properly formatted commit, let me know and I can update accordingly.

Nope, that's all it does. Will look into fixing that tomorrow.

The AppVeyor failure is due to the dependency reordering. Feel free to merge anyway - I'll sort that out.

And now that #4670 is submitted, this passes :)

Yeah, can this be merged then?(I don't have rights, or writes in this case 😆 )

I'll merge and tag, then hand the baton back for the final stage.


ScottPlot5: Improve user control API

After merging #1964 the IPlotControl API and related code could still use some review and fine tuning.

Dead zone: Let's figure out how to better handle the "dead zone" pixel area (where click-drag is not respected) and to ensure the zoom rectangle isn't drawn when returning back to the dead zone.

https://github.com/ScottPlot/ScottPlot/pull/1964#discussion_r923976640
https://github.com/ScottPlot/ScottPlot/pull/1964#discussion_r923977581

Custom interactions: There's a new IPlotInteractions which lets users load a whole set of alternative plot bindings (which mouse buttons do what), but there's probably a more convenient API for this. Let's explore some options.

https://github.com/ScottPlot/ScottPlot/pull/1964#pullrequestreview-1042809240

Overall, you can see I'm going for absolute simplicity when it comes to implementing user controls on new frameworks. Here's the entire WpfPlot control for ScottPlot 5:
https://github.com/ScottPlot/ScottPlot/blob/84ec8c9bd7617d54d7999fc79322d9dd0650b37f/src/ScottPlot5/controls/ScottPlot.WPF/WpfPlot.xaml.cs#L13-L127

Hi Scott, just a quick reaction to your comment,

to ensure the zoom rectangle isn't drawn when returning back to the dead zone.

Some other plotting libraries tend to provide a zoom rectangle that only zooms in horizontal or vertical directions if the mouse is dragged along the horizontal or vertical direction only (within a given tolerance). At the moment, I think this is something that ScottPlot does not allow, unless you lock the limits of the axis, but that's not very flexible.
This could be another nice improvement for v5.

A good example of that is this jsfiddle in plotly: https://jsfiddle.net/Lb476dan/
I find that it switching between types of zooms can be a little fiddly and jarring. They also have a similar thing for only the X axis that I've seen in their financial charts: https://jsfiddle.net/2c8e7qnm/ (might have to scroll down, by default the preview window is pretty small on jsfiddle).

This could be another nice improvement for v5

It would be nice to make this easier! I agree with @bclehmann the zoom example demonstrated in plotly is a bit jarring.
Maybe we can get the best of both worlds by improving the behavior (and rectangle placement) when CTRL or ALT are held down, so it's easier to use the mouse to set axis limits for a single axis.
I'll create a demo of this soon so we can evaluate how it feels

Following-up about the zoom thing, I implemented it in #1978 and it feels pretty good to me! It may be worth porting this functionality over to ScottPlot4.

Regarding the new API, it seems to be evolving nicely and we are moving the daily small talk to Discord (#1966) so I think this issue can be closed to focus attention to #1955


How to get the current index of displayed image?

The current version does not have the privilege to provide the index of the image being currently displayed. This property will be added in the next version and it will be rolled out soon.

The new version of imageview360 provides the callback for image index changed. @luhluh-17  this would be the solution to your problem.


Feature Request script tag with js code.

we  should be able to load a script tag with javascript code in it.!

@code-smith What is the expected use case?


Now empty bodies are allowed by the API

ManageIQ/manageiq-api-common#93 allows empty bodies so these tests no
longer match the api-common expectations.

@gtanzillo this gets sources-api green again