Rename module to "soundfile"?

I always found the module name pysoundfile a bit clumsy to type and to speak about.
Also, import pysoundfile as sf didn't feel right, because sf isn't really an abbreviation of pysoundfile, it should rather be psf, but that's not nice, either.
I couldn't come up with a better suggestion, until I thought about possible package names for Debian (see #100). I think python-soundfile and python3-soundfile would be reasonable names for packages.
Which led me to the question: Why don't we just call the module soundfile?
This would be simpler, nicer to type and read, and the abbreviation sf would suddenly make sense:
import soundfile as sf

Also, soundfile.SoundFile looks more consistent to me than pysoundfile.SoundFile.
The project name on Github (the "marketing" name) should still be "PySoundFile", but inside Python code, the "py" isn't really necessary.
Of course, the same change could/should be made to PySoundCard, then the canonical header for a Python script would become:
import soundfile as sf
import soundcard as sc

... and code would look something like this:
data, fs = sf.read("myfile.wav")
sc.default.samplerate = fs
sc.play(data)

I had a quick look at PyPI and in the web, and I didn't find any real conflicts.
I know, I could've come up with this idea before 0.6.0, where we changed a lot of the API, but sadly, I just didn't.
Any opinions?

I see your point, and largely agree that soundfile is a better name than pysoundfile. I think we should postpone this to a later point, though. Maybe 1.0?
There have been enough breaking changes for a bit. 0.6 would have been a great time for it.

If you also think it's a better name, we should change it rather sooner than later.
We could make a new release 0.7.0 in a few days, hoping (but not guaranteeing) that we don't have any more major breaking changes from there.
The longer we wait, the more annoying the breakage will be.
I don't think it serves anyone if we wait until 1.0.
Also I hope that the number of users will be orders of magnitude larger when we release 1.0, so it will annoy many more people than it will if we release the breaking change in the next few days.

On the other hand, we might just call it 1.0 anyway. I think that what with the documentation being done, and the test suite, and all the improvements we did in the last year, this is truly worthy of going out of beta (actually, it's technically still marked alpha).

I added a commit that renames the module (f80c60e9cd15cf9048df64f71469bf598407852c).
I think the exact time we should go for version 1.0 is largely a matter of taste.
We can do it now, we can also wait a bit; I'm OK with it either way.

is pip install case sensitive?

I'm quite sure it's not.
I just changed it to PySoundFile because that's the "official" spelling and that's what is shown on the PyPI page.
But I can change it back to lowercase, if you prefer.

I rebased the commit (6c6ab4d6d05caa0594d1a2299593eef6d360d0b9).

Let's talk about this in person to decide this issue. I'm still torn about it.

I changed the pip commands to lowercase, now it's ready to merge!


"let" keyword doesn't work sometimes.

Hi, "let" keyword doesn't work when i using <HTMLCanvasElement>
Input:
 "use strict";

var canvas: HTMLCanvasElement = <HTMLCanvasElement>document.getElementById("game"); // Problem
var ctx = canvas.getContext("2d");
/*function kare(ctx) {
    this.x = 0;
    this.y = 0;
    this.w = 0;
    this.h = 0;
    this.draw = function () {

    }
}*/
function image(){
    this.x = 0;
}
let a: number = 1; // Problem
alert(a);

Output:
//Compiled TypeScript

"use strict";
var canvas = document.getElementById("game");
var ctx = canvas.getContext("2d");
function image() {
    this.x = 0;
}
let;
a:
number = 1
alert(a);

Input:
"use strict";

// Deleted line
var ctx = canvas.getContext("2d");
/*function kare(ctx) {
    this.x = 0;
    this.y = 0;
    this.w = 0;
    this.h = 0;
    this.draw = function () {

    }
}*/
function image(){
    this.x = 0;
}
let a: number = 1; // Working good
alert(a);

Output:
//Compiled TypeScript

"use strict";
var ctx = canvas.getContext("2d");
/*function kare(ctx) {
    this.x = 0;
    this.y = 0;
    this.w = 0;
    this.h = 0;
    this.draw = function () {

    }
}*/
function image() {
    this.x = 0;
}
var a = 1; // Working good
alert(a);

From : https://github.com/basarat/typescript-script/blob/163388be673a56128cc1e1b3c76588001a8c1b18/transpiler.js#L60 you can see that the transpile is blindingly simple. Its just whatever ts has. That said I believe the default transpile is in JSX mode. Hence you can't use the <Foo> style assertion syntax and need to use as Foo.
More on Type Assertion and reasons for my (and seemingly the TS team) preference : https://basarat.gitbooks.io/typescript/content/docs/types/type-assertion.html :rose:


In the implementation of LSTM, why don't you just use activation function's backward method?

I noticed in the implementation  of LSTM activation function, ad hoc functions _sigmoid, _grad_sigmoid  and _grad_tanh  are used.
But, why not just use chainer.functions.Sigmoid().forward() and chainer.functions.Sigmoid().backward() to do this part? Is there any special reason makes you to avoid this?
I wonder this, because I am trying to write a chainer version TreeLSTM.

It is performance reason. As you say chainer.functions.Sigmoid works well. We optimized LSTM by making one large kernel. If performance is enough use chainer.functions.Sigmoid. It's easier.
BTW, one of the Tree-LSTM implementation is here.
https://github.com/pfnet/chainer/blob/master/chainer/functions/activation/slstm.py

Yeah, I know that, but what I am trying to implement is the special kind that you have mentioned in #402. Thank you for your answering.

Great! I recommend you to implement functions with combination of chainer.functions quickly. You need to optimize them only when performance is not enough.


Iterate our contribution model

What
Iterate our contribution model, based on:

the team contribution retro
the diversity and inclusion survey ideation session
the contribution and assurance ideation session

Stories

[ ] create a 'to-be' journey map for contribution
[ ] write and prioritise stories

Duplication of #154


Is there a concept missing in this explanation?

housing_code_interview.yml
how_far_back_should_i_go_template
subject: |
What problems should I list?
content: |
If you want to sue your landlord for money, you can list any problems that you have
had over the last 6 years. Even if the problem is fixed now.

Tenants who do not want to sue their landlord usually only list problems that still
need to be fixed.

Is this Tenants who not want to sue their landlord for money or who do not want to sue their landlord at all? Does this mean they just want to write a letter to ask the landlord to fix the repairs, or they want the housing inspector to do something or... who are these tenants and what do they want?

The idea I'm trying to convey is that if someone is only suing to get repairs made, there's no reason to list problems that are already fixed. But they can get money damages for any problem up to 6 years in the past even if it is fixed now.


[ setValue:forUndefinedKey:]: this class is not key value coding-compliant for the key ibCalendarDataSource.

(Required) Version Number:
8.0.0
Description
After upgrading the version from 7.1.7 to 8.0.0 and renaming the required classes (as per the migration guide), a working calendar setup now no longer works.
Steps To Reproduce
Update from v7.1.7 to v8.0.0 and rename necessary classes (including Storyboard references)
Expected Behavior
Calendar works as before.
Additional Context
Prior to the update the implementation was working and was being used in production builds.
v8.0.1 is not available through pod update, and forcing the version causes an error. pod is up to date, ios target is set to 11.0

I got the same error, for some reason, Interface Builder doesn't set module correctly after I renamed JTAppleCalendarView to JTACMonthView. I am using xibs. I manually wrote JTAppleCalendar for module and it worked.

I repeated the migration just to confirm, and found the issue:
My starting point was as with yours, changing the Class to JTACMonthView cleared the Module selection. Without the Module selection also being set, the Exception occurs.


Publish roadmap

Publishes the Roadmap to the website docs.

We should add some contextual heading around the features to show what type features we are focusing on:

Initial user experience
Better error messages
Transactions / Migrations / Relationships?

Call out the Beta release and link to this document.
Show that it is working at scale.


Local installer fails with spin 0.10.0

spin v0.10.0 has removed --follow-all which causes Nomad jobs to fail Remove deprecated --follow-all option
Downgrading to spin v0.9.0 solves this.
Installed on macOS

spin 0.10.0 (e84ac6c 2023-03-02) --> at the time of issue
Nomad v1.5.0
Consul v1.15.0

Example log from nomad job:
WARNING: Bindle support is deprecated and will be removed in a future version.
For remote applications, please use registry commands and features instead.
See https://developer.fermyon.com/spin/spin-oci for more details

2023-03-06T21:24:44.375105Z  INFO spin_loader::bindle::assets: Mounting files from 'webserver/0.1.0+qca333d2' to '/private/tmp/.tmpMhCgAf'
error: Found argument '--follow-all' which wasn't expected, or isn't valid in this context

	Did you mean '--follow'?

	If you tried to supply `--follow-all` as a value rather than a flag, use `-- --follow-all`

USAGE:
    spin trigger http --listen <ADDRESS> --follow <FOLLOW_ID>

For more information try --help
Error: exit status: 2

@ad-ops thank you for the issue.  I see a few options here, in rough order of preference:

Update hippo with usage per spin v0.10+, cut a release, bump hippo (and spin for the remote installers) here
Implement a mechanism to inform users going the local installer route that spin needs to be pinned to v0.9.0
Alternatively, check in w/ the Spin project to see about the feasibility of maintaining a --follow-all alias for --follow in future spin release(s) (would not require hippo release)

Apologies for the delay here.  We do have a new hippo release (https://github.com/deislabs/hippo/releases/tag/v0.20.2) but I'm seeing issues with redeploying apps when using Spin v0.10.1 so it appears there may be other areas in Hippo that need updating for compatibility.
For the near term, we'll have to notify users running the installer locally that Spin needs to be pinned to a certain version -- ideally the same as used by the remote installers -- until Hippo is updated.


Request the feature: "editor.lineHighlightForeground".

Request the feature:  "editor.lineHighlightForeground"
Sometimes it is necessary to change the style colour of the selected text, after "editor.lineHighlightBackground".
An example:
"editor.lineHighlightBackground": "#000", "editor.lineHighlightForeground": "#fff".
Cheers.

So all the text of the selected line would get the same foreground color? You would loose the syntax coloring...

Thanks, this enables to make high-contrast themes, just: https://github.com/llatigid/Pushkin-is-White-Theme/issues/1.

Hello! Any updates?

There are currently no plans on providing this feature, sorry.

There are currently no plans on providing this feature, sorry.

Hello, any chance? :)


Rename add import assist

Based on the https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Fwg-rls-2.2E0/topic/2.20assists.20with.20the.20same.20action.20name/near/187655643 and the related discussion.

bors r+


Alpha search seems to be stuck on

I wanted to poke around a bit and see what the interface was like. When I wanted to quit or turn off my calculator, the [on + sto] shortcut didn't work (that is the way to close Cesium right?).
This follows into my second issue, where on commands don't seem to be working even though they are highlighted in the mode menu. I also tried to do the dark mode short cut, but all it did was move my cursor on the screen to whatever letter was associated with the key. So the [on + sto] command just moved the curser to the bottom of the screen since there's no file that starts with X and [on + ln] moved my cursor to the file that starts with S.
I looked through the closed issues and noticed that you asked for the last part of the serial last time someone has issues with the alpha stuff. Mine is L-0216C. Hope that helps.

The [on] key shortcuts are to be used from outside Cesium, such as on the home screen where you put equations in.


Update label comment

xref https://github.com/galaxyproject/gxformat2/pull/89#issuecomment-1545794899

Thanks for the follow up!


chrome.switches not respected

Using the Serenity BDD framework, we're passing in the following property, which should start Chrome as a maximized window, however in selenoid, the window is not maximized.
chrome.switches=--start-maximized
Doing the same thing pointing to a hub running in Zalenium produces the following capabilities string and results in a maximized window.
Capabilities [{chrome.switches=--start-maximized, record-screenshots=false, browserName=chrome, idle-timeout=180, chromeOptions={args=[test-type, --start-maximized, test-type], extensions=[]}, screenResolution=2880x1800, platformName=ANY, version=, platform=ANY, ACCEPT_SSL_CERTS=true}]

@dforrest88 if you wish to use chrome command line argument, try to use this one:
--window-size=2880,1800

Window manager will be added: https://github.com/aerokube/selenoid-images/issues/54


Cherry-pick fix for #8654 for 2.5.1 release

This change cherry-picks the fix for issue #8654 so that the sample in 2.5.1 is implemented correctly and does not display the multiple event systems warning.

/azp run


Test PR docker images

Test the docker image generation

The docker internals should be much faster now, since the different layers are cached and will only rebuild if changed. Let's see how much we gain here.

@mgrover1 :disappointed: Hmmm, it looks like we are missing push write capabilities in Pull Requests. I've some more ideas in the pocket how to resolve this.
One would be to combine the two builds (container/book) into one and do the image push to ghcr.io only on push-event. In the pull request we would just build the image for testing and use it directly in the book build. I'll have a look into this the next day, if no one beats me to it.

Success! I've added a special workflow for pull requests which builds the image without pushing to the registry. It directly uses this image to create a container to run the book build. Works.


AWS - openshift-install does not honour ~/.aws/credentials

Describe the bug
I have my assumed role AWS Credentials set in ~/.aws/credentials and which are working fine but openshift-install still looks for AWS Access keys
The credentials have full administrator rights
$ cat ~/.aws/credentials
[default]
role_arn              = arn:aws:iam::<account_id>:role/assume-role
source_profile        = parent
region                = ap-southeast-2

[parent]
aws_access_key_id        = <redacted>
aws_secret_access_key    = <redacted>
aws_session_token        = <redacted>
aws_security_token       = <redacted>
x_principal_arn          = arn:aws:sts::<parent_account_id>:assumed-role/administrator/<user>
x_security_token_expires = 2020-09-03T00:53:15+10:00
$

I have also tried just assumed role's AWS Access keys and Session Token
[default]
region                = ap-southeast-2
aws_access_key_id     = <redacted>
aws_secret_access_key = <redacted>
aws_session_token     = <redacted>

Version
$ openshift-install version
openshift-install 4.5.0-0.okd-2020-08-12-020541
built from commit 699277bb61706731d687b9e40700ebf4630b0851
release image quay.io/openshift/okd@sha256:6974c414be62aee4fde24fe47ccfff97c2854ddc37eb196f3f3bcda2fdec17b4
$

How reproducible

Use an assumed role, shown above. It looks openshift-install specifically looks for keywords aws_access_key_id and aws_secret_access_key in ~/.aws/credentials
If I just use the assumed role's AWS Access keys and Session Token, I get below error.
DEBUG   Generating Platform Permissions Check...
INFO Credentials loaded from the "default" profile in file "/home/ec2-user/.aws/credentials"
FATAL failed to fetch Cluster: failed to fetch dependency of "Cluster": failed to generate asset "Platform Permissions Check": validate AWS credentials: checking install permissions: error gathering AWS credentials details: error querying username: ValidationError: Must specify userName when calling with non-User credentials
        status code: 400, request id: 47a70b8b-8ebd-44d1-9d7d-af3a1xxxxxf

See https://docs.okd.io/latest/installing/installing_aws/installing-aws-account.html#installation-aws-iam-user_installing-aws-account - assumed role creds are not supported, The cluster continues to use your current AWS credentials to create AWS resources for the entire life of the cluster, so you must use key-based, long-lived credentials

hey @vrutkovs - Can we take this as a feature request? Long-lived credentials are not good for security.

Can we take this as a feature request?

That would affect OCP, so start with a customer case (if possible) or file a bug at https://bugzilla.redhat.com


super minor change

This is in the second para of the introduction
"However, enhancers are not always active in every cell type or state. Hence, in reality the looks more like below where Enhancer 1 is entirely inactive in Sample 1 while Sample 2 is inactive in Sample 3." Don't you mean, "Enhancer 2 is inactive in Sample 3"?

Thanks a lot for bringing this up! You're right, this is a type and it should be Enhancer 2 is inactive.... I fixed it and will re-deploy.


Show version information on hover over file tab top

Instead of showing version information in the status bar, how about showing it in the hover over on the file name at the top of the tab. VS Code shows the full path filename at the moment. Would it be possible to add the version information below the filename? This would provide the information without occupying valuable space on the status bar.

need to investigate this if possible. I especially like the suggestion because I would like to use the statusBar for something else ( current activity )

Currently vscode does not support to directly change the title of a editor tab. So implementing this isn't possible right now


[BUG] Rancher is showing k8s versions < 1.23 for kev1 AKS/GKE clusters

Setup

Rancher version: 2.7.0-rc6
Browser type & version: Chrome Build 106

Describe the bug

Rancher is showing k8s versions less than 1.23 for kev1 AKS and GKE clusters
To Reproduce

AKS

Install Rancher 2.7.0-rc6
Create kev1 AKS cluster by accessing the cluster from the URL - https:///g/clusters/add/launch/azureaks
View k8s dropdown on UI

GKE

Install Rancher 2.7.0-rc6
Create kev1 GKE cluster by accessing the cluster from the URL - https:///g/clusters/add/launch/googlegke
View k8s dropdown on UI

Result
Rancher shows k8s versions less than 1.23 for both AKS and GKE kev1 clusters. The behavior is not reproducible on kev2 clusters
Expected Result

k8s versions less than 1.23 should not be shown in the UI.
Screenshots

AKS

GKE

I tried to reproduce this in v2.6-head but couldn't. Here's what I tried:

Installed Rancher v2.6-rc7
Went to the /g/clusters/add/launch/azureaksv2 endpoint and installed an AKS cluster. (As discussed, we don't normally support this endpoint, but are using it mainly to simulate a cluster installed in Rancher 2.5.x who should have an upgrade path in the new UI)
Confirmed that v1.22.x is not available as a Kubernetes option

4. Finished provisioning the cluster, went to the dashboard, went to edit the cluster, confirmed the K8s options are the same. (I wouldn't expect any discrepancy between these versions because the same component is reused in both pages)
Then the same for GKE:

Went to the /g/clusters/add/launch/googlegkev2
Confirmed that v1.22.x is not available as a Kubernetes option

After provisioning the cluster, went to edit the cluster and confirmed that the K8s options are the same.

@catherineluse moving back to reopen.
The link you used was still kev2 /g/clusters/add/launch/azureaksv2
The correct link for kev1 would be /g/clusters/add/launch/azureaks
This also applies to the GKE link as well

@nwmac thanks for the clarification. Closing this issue as will not fix. @snasovich


Hands-on with the Bridge2Hyku Toolkit

The University of Houston (UH) Libraries, in partnership and consultation with numerous institutions, was awarded an IMLS National Leadership/Project Grant (LG-70-17-0217-17) to support the creation of the Bridge2Hyku (B2H) Toolkit.  As part of this toolkit, University of Houston is developing several applications to assist in the migration of data from CONTENTdm to Hyku. The workshop will cover two important tools:
CDM Bridge: a stand alone electron-based tool that easily pulls metadata and files out of any CONTENTdm instance for migration and metadata remediation.
HyBridge: a importer gem that would allow Hyku providers an easy insert into their stack for users to migrate with.
This workshop will have attendees work with these applications and the B2H Toolkit to cover the basics in metadata migration. Learn how to use CDM Bridge to get your data out of CONTENTdm, allowing you to assess, reconcile and prepare your data for Hyku. See how using HyBridge can ingest your data into Hyku and learn how to evaluate that data in its new home.
Todd Trocken<EMAIL_ADDRESS>Sean Watkins<EMAIL_ADDRESS>

This is a workshop proposal in the wrong place.


chore: bdd-spring-1570653212 to 0.0.1

chore: Promote bdd-spring-1570653212 to version 0.0.1

/test this


Arch Linux support

I have recently uploaded Arch Linux PKGBUILD (sort of RPM or BSD port spec) for Singularity to AUR [1]. Now I would like to help with Arch Linux bootstrapping in Singularity, if you are interested too.
I gather I'd need to provide an appropriate libexec/mods/linux_build_arch.smod file. What else would be needed? The existing Arch Linux Docker image creation script [2] should be a good starting point. I'm wondering how to go about the "udev doesn't work in containers, rebuild /dev" part. I guess #147 is related. Can you see any other problematic parts in the script?
Does [3] include all the currently supported definition file keywords?
[1] https://aur.archlinux.org/packages/singularity-container/
[2] https://github.com/docker/docker/blob/master/contrib/mkimage-arch.sh
[3] http://singularity.lbl.gov/#bootstrap

Fantastic!
Yes, the primary file to create and work with would be linux_build_arch.smod as it currently stands but for 2.2, I've been brainstorming enhancing the bootstrap'ing capabilities considerably. I'm almost done with the backend work (please check out the 'devel' branch to see current status) and will be moving on to the bootstrap mods right after.
You probably won't have to work with the UDEV issues in Singularity because /dev is basically a bind mount from the host (unless someone uses --contain, and in that case the bootstrap infrastructure already builds the bare minimum devices necessary). The general gist of the script seems fine, but various aspects are redundant with what Singularity already does. I would recommend looking at the redhat and debian modules and just fill in the API with those bits necessary for Arch. If you have any specific questions just let me know (I'll leave this issue open for that purpose).
One other thing to consider is that I would like to allow bootstrap's to happen as users. Right now it only works for the busybox module (because I believe YUM and APT check getuid()). If it could be done with Arch, I think that would be a big win for users using Arch with Singularity!
It is my intention to maintain the current bootstrap support as we move forward, so working on linux_build_arch.smod is not a wasted effort. I'll keep this thread open so we can circle back as necessary to get the required bits into the 2.2 release.
BTW, if you use Slack and want an invite, send me your email.
Thanks!

I'll focus on the smod for 2.x. I'll probably might have something to show next week or so. I'm OK with shell scripting, but the Singularity (and Docker) bootstraping internals are still not 100% clear to me so I might be proceeding slowly.
Is inheriting host's /etc/localtime a good idea or not? How about the locale - should it rather be en_US.UTF-8, the host's setting, else? This will depend on a use case for sure, but I wonder what is the common practice with Singularity images in this regard.
Thanks for the invitation to Slack, but I don't need it at the moment. GitHub tickets are just fine. This way the message exchange is nicely bound to the project itself.

As I mentioned, please feel free to ask me if you have any questions. I'm happy to help.
/etc/localtime is probably best as a bind path actually. That way it is always the same as the host's... But this brings up a compatibility question that I am not familiar enough with. Is the timezone data format always compatible?
We are also on IRC at #singularity on irc.freenode.net if you have any quick questions (depending on who happens to be watching their IRC window at that moment).

Not saying that we have to copy others, but how does Docker handle /etc/localtime?
I do know that various distros have different ways of specifying the timezone info.  I'd be a bit nervous about a mechanism that only works on a few distros or only "half enables" a timezone in a particular container.

The Docker camp will usually hard code the /etc/localtime in the Dockerfile, or they will bind /etc/localtime from the host into the container (but I'm not sure if there is a potential here for issue).
On the other hand (as I'm reading the 'date' manpage, and I can't believe I forgot this...), there maybe a much better way to handle this! The environment variable TZ if set in the user's host environment would pass transparently through into the container where it will also work there. So that is probably the most portable way of handling it.
Thoughts?

Are you thinking we should set TZ appropriately if not already present?  Is there a reasonably nice way to detect the correct value of TZ from /etc/localtime?
I.e., /etc/localtime is a file in RHEL6 and a symlink in RHEL7.

I was thinking that inside the container /etc/localtime should be set to something sane. Perhaps UTC or something specific for a particular use-case.
But it really wouldn't matter if the user had TZ set. So what we should do is document that TZ should always be set in order to handle and/or implement some magic to set it if it is not set in etc/init:
if [ -z "${TZ:-}" ]; then
    TZ=`date +%Z`
    export TZ
fi

ah yes, that would work!  I was thinking about deriving the Olson timezone (America/Chicago), not the compact format (CDT).
Yeah, I like this approach.

The timezone feature has been committed: [devel a72a79c]
@czka were there any other questions or comments you had for this thread? If not, can you close.
Thank you!

In my opininion, ideally:

Locale: def files should accept a keyword for setting an arbitrary number of locales, supporting the format like in these examples: "en_US.UTF-8 UTF-8", "zu_ZA ISO-8859-1", "gl_ES@euro ISO-8859-15". Because GNU/Linux supports multiple locales while POSIX Shell does not support arrays, and strings with spaces are hard to iterate on, this could eg. be a comma-seperated list of locales. Eg. Locale='en_US.UTF-8 UTF-8,zu_ZA ISO-8859-1,gl_ES@euro ISO-8859-15'.

Another keyword for default LANG should be there as well. Eg. Lang=pl_PL.utf8.

Timezone: def files should accept a keyword for setting it to a single string in Olson format. Eg. Timezone='America/Chicago'.

Bootstrap should pass those as variables to a smod file. Each smod would apply them in a distro-specific way. Arch does this as follows: https://wiki.archlinux.org/index.php/Installation_Guide#Locale, https://wiki.archlinux.org/index.php/Installation_Guide#Time_zone. I order to iterate over $Locale, one could use a follwing syntax in his smod:
Locale='en_US.UTF-8 UTF-8,zu_ZA ISO-8859-1,gl_ES@euro ISO-8859-15'

IFS=,
for l in $Locale; do
  echo "doing the distro-specific thing with locale $l"
done
unset IFS

Output:
doing the distro-specific thing with locale en_US.UTF-8 UTF-8
doing the distro-specific thing with locale zu_ZA ISO-8859-1
doing the distro-specific thing with locale gl_ES@euro ISO-8859-15

Until such keywords are supported in def files, I think I'll settle for en_US.UTF-8 UTF-8 and UTC I guess. I shall yet medidate on this tough ;).
@gmkurtzer: Question about the Setup keyword - http://singularity.lbl.gov/#bootstrap reads that it "creates the necessary starting point, files, and components for an OS to be bootstrapped". But I can't find any related code, besides this, in linux_build.smod:
Setup() {
    return 0
}

There's no trace of Setup in bootsrap.exec, functions, bootstrap.sh. Can you tell me how it gets called to do the stuff?

I hope this isn't a stupid/ignorant question, but shouldn't locales be defined within the container by means of the package manager's installation of the relevant packages?
Regarding LANG and Timezone, I feel these should be runtime options rather then hard coded into the container. Granted a particular container may have to have the Locale support built into the container for proper rendering of a particular LANG or timezone, but I feel these should be defined by the user calling the container.
You are right about the Setup keyword. I moved that functionality to PreSetup which is automatically called by the bootstrap driver (bootstrap.sh) so Setup is technically no longer "required'. But I left it there so that a particular smod could define and use it if desired, but this is another case where the code moves faster then the documentation and I surely need help with the docs. lol
Thanks!

shouldn't locales be defined within the container by means of the package manager's installation of the relevant packages?

I don't know about other distros, but on Arch this is done manually. See the https://wiki.archlinux.org/index.php/Installation_Guide#Locale I mentioned before.

Regarding LANG and Timezone, I feel these should be runtime options rather then hard coded into the container.

OK. Still, the Arch installation process requires (or at least recommends) these to be set permanently, system-wide, as described in their guide. But maybe that's just Arch.

I surely need help with the docs

The problem with that is that the person who knows what documentation changes are needed is the code changes author :).

Yep, so the locales should be defined and configured at bootstrap time. It would in theory also be possible to have it checked/updated by the container's /environment file as well if we want runtime support, but I think that is overkill.
Yes, I also agree that both Lang and Timezone configuration must be set globally and permanently, but they should also be able to be overridden by the user. So the global configuration should be something very simple and portable. For example, earlier I suggested UTC for the timezone.
Haha, yes! You have a good point there about the documentation. I am working on it! :-D

Preliminary Arch def and smod uploaded in a681f16. It creates a working image, but some issues crop out:

Installation of filesystem package triggers the following warnings:

(...)
( 4/87) installing filesystem
warning: /usr/var/singularity/mnt/etc/hosts installed as /usr/var/singularity/mnt/etc/hosts.pacnew
warning: /usr/var/singularity/mnt/etc/resolv.conf installed as /usr/var/singularity/mnt/etc/resolv.conf.pacnew
warning: directory permissions differ on /usr/var/singularity/mnt/root/
filesystem: 755  package: 750
warning: directory permissions differ on /usr/var/singularity/mnt/var/tmp/
filesystem: 755  package: 1777
( 5/87) installing glibc
(...)

This is apparently due to PreSetup intitializing those 2 files and 2 dirs in advance.
Is it always a good idea to copy /etc/resolv.conf and /etc/hosts files from the host system? I really don't know. Thoughts?
Not sure about 1777 vs 755 on /var/tmp. But 755 (rwxr-xr-x) on /root doesn't seem OK.

I can ping as root, but can't as a regular user:

Singularity.arch.img> id
uid=0(root) gid=0(root) groups=0(root),1(bin),2(daemon),3(sys),4(adm),6(disk),10(wheel),19(log)
Singularity.arch.img> ping -c1 google.com
PING google.com (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from waw02s05-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=1 ttl=53 time=92.7 ms

--- google.com ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 92.755/92.755/92.755/0.000 ms

Singularity.arch.img> id
uid=1000(user) gid=100(users) groups=100(users),10(wheel),14(uucp),24(rfkill),50(games),996(docker),1002(plugdev)
Singularity.arch.img> ping google.com
ping: socket: Operation not permitted
ping: socket: Address family not supported by protocol

Is that as expected?

We can change how these directories are created (or rather when) to ensure we get the right permissions.
Hrmm, I agree that it may not be a good idea to copy resolv.conf and hosts from the host system. This should be changed.
Actually yes, pinging technically requires root for ICMP and as a result /usr/bin/ping is usually SUID root or it has extended Linux capabilities to allow it privilege to ping. Considering Singularity containers are not "trusted" (e.g. they are provided by the user), no privilege escalation is allowed inside of them... Long story short, yes... Pings are expected to fail.
Is a681f16 a PR or a commit to your local fork? Feel free to send me a PR for that.
Thanks!

it may not be a good idea to copy resolv.conf and hosts from the host system. This should be changed.

On the other hand, if these files are not copied you'll probably have to provide the means to set the DNS and known host names via def file. Maybe just document how to do it using RunCmd's with some echo's (or InstallFile, if the user really wants the files identical as on his host), after Bootstrap? That should be good enough I guess.

We can change how these directories are created (or rather when) to ensure we get the right permissions.

Do you need a separate ticket for that? What are the permissions on /root and /var/tmp on the machines you have at hand? I only have Arch here.

/usr/bin/ping is usually SUID root

Right! I forgot that. Thanks for explaining this.

Is a681f16 a PR or a commit to your local fork?

It's on my fork for now. I'll send you a PR when it's done.

I would suggest that in addition to documentation we create templates using things like public DNS services and a generalized hosts file so that when --contain is passed, there is a sane starting point.
A separate ticket would be helpful to make sure it doesn't slip through the cracks.
Excellent, looking forward to the PR! Thanks!

I'm done, PR against brach 2.x submitted.
The issues we touched on here and which don't yet have their own tickets are inheriting/or not the resolv.conf and hosts file from the host, and the localization. I can open 2 new tickets for these and close this one. OK?

Just merged your PR. Looks great!
Yes, I think that closing this issue and creating two new ones reflecting the specific issues you identified would be great.
Thanks again!

I have come up with a generic solution suggestion in #203. I'm curious what you think about it. Too bad I can't help implementing it. I don't know C. Shell and Python only.


Migrate to VS 2017 CSPROJ

This "worked" but I have no idea if it actually "worked"

the build.cmd doesn't work, but that is expected.


[BUG] BlobClient.UploadAsync do not write stream if this one has not been read once.

Describe the bug
Using BlobClient.UploadAsyncwith a stream wich has not been read once ( lenght = 0, position = 0, canSeek = true ) result in creation of empty blob. ( NOTE The stream does contain data but is not aware of the lenght until first read )
Expected behavior
Using BlobClient.UploadAsync with a stream wich has not been read once ( lenght = 0, position = 0, canSeek = true ) properly read the stream until end and create the blob with the stream content.
Actual behavior (include Exception or Stack Trace)
N/A
To Reproduce
Create Asp.net core APP.
Upload The Request.Body ( which has not been read yet ) of POST Method using BlobUploadAsync.
Environment:
Azure.Storage.Blobs 12.9.1
.Net 5.0.301
For person who look for a fix best way is to OpenWriteAsync a BlockBlobClient and use Stream.CopyToAsync().

Not sure if this is a fix we should do because if we attempt to read a stream that actually is empty (length = 0) then we would run into a null exception for attempting to read at a position we can't make the request.
What's the scenario where UploadAsync has to take in a no-length stream? (this is seems like the Stream itself has a problem) Maybe we can find another solution for this. For now it looks like OpenWrite seems like the best solution for this scenario.


Autoplaylist Functionalities Phase 1: Autostream Implementation

After creating your pull request, tick these boxes if they are applicable to you.

[x] I have tested my changes against the review branch (the latest developmental version), and this pull request is targeting that branch as a base
[x] I have tested my changes on Python 3.5/3.6

Description

I planned to do several pull requests to rewrite and add functionalities to autoplaylist which cover a range of feature requests. This will happen in different phases. Phases are arranged in the order that it won't interfere with each other very much.
phase 1: add autostream and related functionalities
phase 2: make options to allow totally different autoplaylist.txt in different servers (or guild)
phase 3: add an option to show autoplaylist "now playing" notifications

This pull request introduces autostream. It's similar to autoplaylist, but it's for streams.
You can add streams for autoplaying in config/autostream.txt.
If the configuration for randomization and toggling is not set, autostream will play after autoplaylist.
Update 1: autostream and autoplaylist could be set to skip automatically when someone adds stuff to the queue.
Update 2: autostream and autoplaylist can play in 2 modes:

merge: merge autoplaylist and autostream when playback
toggle: use the toggleplaylist command to toggle between them

Update 3:

modes are saved
different modes for different servers now possible

I am currently satisfied with these changes. I don't think I will push more commits except commits for clarity and bugs fixing from b06c8b3 onwards. (This mark an end to more feature implementation to this little PR)
As of now, if #1740 got merged before this. This PR will require some modifications to work properly. If there's plan to merge #1740, merge it first. I will make the required modifications on this PR after #1740 got merged. If #1740 got merged and this message is still here, DO NOT MERGE THIS PR as it will definitely break. I will merge changes required from TheerapakG/MusicBot:autostream_permissions into this.
Related issues (if applicable)
(#1462, #1587)
not a direct implementation of #1587 but introduce "roughly equivalent" features, here is the breakdown:

autostream.txt now exist
you can add or remove a stream using autostream command
toggling can be set using the config file
state of toggling does save so next startup it'll be on whatever mode you've set
you can use the config file to skip the autos when added the new entry (the self-closing stream do exist so I added the config to let you choose to force the autostream to skip or not)

Waiting AutoStream so much! Thx for work man!

Any ETA ?

Any ETA for autostream function ?

You can try it out now by pulling from the branch in my repository. There might be some bugs (I would expect only two or three bugs at most) which I haven't found. If you try it out and found them please let me know! For the date when this will be merged into the main branch, I don't know yet. It depends on the collaborator of this repository.


Update SF-0007 Subprocess to verision 6

Addressed proposal comments from https://forums.swift.org/t/review-2nd-sf-0007-subprocess/76547/26

Added a Configuration based overload for runDetached.
Updated input types to support: Sequence<UInt8>, Sequence<Data> and AsyncSequence<Data> (dropped AsyncSequence<UInt8> in favor of AsyncSequence<Data>).
Added isolation parameter for closure based .run methods.
Dropped sending requirement for closure passed to .run.
Windows: renamed ProcessIdentifier.processID to ProcessIdentifier.value.
Updated TeardownStep to use Duration instead of raw nanoseconds.
Switched all generic parameters to full words instead of a letter.

@swift-ci please test


AttributeError

Hi, when I try to run the file I get the following error:
Traceback (most recent call last):
File "/usr/local/bin/fbcap", line 9, in
load_entry_point('fbchat-archive-parser==0.6.post1', 'console_scripts', 'fbcap')()
File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 357, in load_entry_point
return get_distribution(dist).load_entry_point(group, name)
File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 2394, in load_entry_point
return ep.load()
File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources.py", line 2108, in load
entry = import(self.module_name, globals(),globals(), ['name'])
File "build/bdist.macosx-10.11-intel/egg/fbchat_archive_parser/main.py", line 15, in
AttributeError: 'module' object has no attribute 'App'
Would anyone be able to help? Thanks

Hi there, how are you trying to run the program? Did you install it through pip?

My apologies,
I rebuilt this in a virtualenv and python 3 and this error was fixed. However, I'm now getting a different error:

File "/Users/ericzhu/.virtualenvs/p3/bin/fbcap", line 11, in
sys.exit(main())
File "/Users/ericzhu/.virtualenvs/p3/lib/python3.5/site-packages/fbchat_archive_parser/main.py", line 73, in main
app.run()
File "/Users/ericzhu/.virtualenvs/p3/lib/python3.5/site-packages/clip.py", line 652, in run
self.invoke(self.parse(tokens))
File "/Users/ericzhu/.virtualenvs/p3/lib/python3.5/site-packages/clip.py", line 634, in invoke
self._main.invoke(parsed)
File "/Users/ericzhu/.virtualenvs/p3/lib/python3.5/site-packages/clip.py", line 519, in invoke
self._callback(**{k: v for k, v in iteritems(parsed) if k not in self._subcommands})
File "/Users/ericzhu/.virtualenvs/p3/lib/python3.5/site-packages/fbchat_archive_parser/main.py", line 34, in fbcap
progress_output=sys.stdout.isatty())
File "/Users/ericzhu/.virtualenvs/p3/lib/python3.5/site-packages/fbchat_archive_parser/parser.py", line 98, in init
self.__parse_content()
File "/Users/ericzhu/.virtualenvs/p3/lib/python3.5/site-packages/fbchat_archive_parser/parser.py", line 123, in __parse_content
self.__process_element(pos, element)
File "/Users/ericzhu/.virtualenvs/p3/lib/python3.5/site-packages/fbchat_archive_parser/parser.py", line 297, in __process_element
minutes=-1 * offset[1])
IndexError: list index out of range

Hmm, looks like it may be an issue with timezone formatting. I'm going to guess you're in a place with a timezone that doesn't have a minute offset and is formatted as UTC-X rather than UTC-X:00. I just submitted a new version to PyPI (0.6post2). Please let me know if that fixes it for you.

The new version fixed this issue, thanks!
Yes, the timezone was formatted as follows: Thursday, December 19, 2014 at 10:20pm UTC+11
I'm now getting the same error as in #3
Looking into my messages.htm file in Sublime Text shows that it's including ASCII Control characters as in http://stackoverflow.com/questions/35800772/external-json-api-provides-unsanitized-character-dle-in-sublimetext-failing
Looking into it at the moment.

Great, in that case I will close this issue and we can continue the conversation in #3. There should be a fallback parser that kicks in if the HTML in messages.htm is poorly formatted. The guy who opened #3 never confirmed whether that worked or not.


Server closure failing if <CTRL+C> hit quickly after starting

We observed twice in a dozen tests: when closing (CTRL+C) quickly after starting the application, the closure is not properly completed and we had to kill the process manually.
Successful closure terminal output:
[OPEN-MCT STATIC SERVER] stdout: Received SIGINT ...
Open-MCT Visualizer Server closing: no further incoming requests accepted. Refreshing the visualizer web page will fail.
Open-MCT Visualizer Server closed: all sockets closed.

normalExit: [OPEN-MCT STATIC SERVER] on close: 0, signal: null
iCub Telemetry Server closing: no further "subscribe"/"unsubscribe" requests accepted.
sdafarra@iiticublap260:~/Software/yarp-openmct/iCubTelemVizServer (main)$ iCub Telemetry Server closing: disconnected network ports.
Data transmission ended.
Closing all Telemetry Server and Control Console sockets!
[ 'iCub Telemetry Server closed: all sockets closed.',
  'Control Console Server closed: all sockets closed.' ]

sdafarra@iiticublap260:~/Software/yarp-openmct/iCubTelemVizServer (main)$
sdafarra@iiticublap260:~/Software/yarp-openmct/iCubTelemVizServer (main)$ pkill -9 node
sdafarra@iiticublap260:~/Software/yarp-openmct/iCubTelemVizServer (main)$ Killed

Failed closure terminal output:
[OPEN-MCT STATIC SERVER] stdout: Received SIGINT ...
Open-MCT Visualizer Server closing: no further incoming requests accepted. Refreshing the visualizer web page will fail.
Open-MCT Visualizer Server closed: all sockets closed.

normalExit: [OPEN-MCT STATIC SERVER] on close: 0, signal: null
iCub Telemetry Server closing: no further "subscribe"/"unsubscribe" requests accepted.
sdafarra@iiticublap260:~/Software/yarp-openmct/iCubTelemVizServer (main)$ [INFO] |yarp.os.impl.PortCoreInputUnit| Removing input from /icub/left_leg/stateExt:o to /yarpjs/left_leg/stateExt:o
[INFO] |yarp.os.impl.PortCoreInputUnit| Removing input from /wholeBodyDynamics/left_foot_front/cartesianEndEffectorWrench:o to /yarpjs/left_foot_front/cartesianEndEffectorWrench:i
[INFO] |yarp.os.impl.PortCoreInputUnit| Removing input from /wholeBodyDynamics/right_foot_front/cartesianEndEffectorWrench:o to /yarpjs/right_foot_front/cartesianEndEffectorWrench:i
iCub Telemetry Server closing: disconnected network ports.
[INFO] |yarp.os.impl.PortCoreInputUnit| Removing input from /icub/cam/left to /yarpjs/camLeftEye:i
Data transmission ended.
Closing all Telemetry Server and Control Console sockets!
[ 'iCub Telemetry Server closed: all sockets closed.',
  'Control Console Server closed: all sockets closed.' ]
[INFO] |yarp.os.impl.PortCoreInputUnit| Removing input from /icub/cam/right to /yarpjs/camRightEye:i

Not reproducible. We'll reopen if we it happens again.


Add time counter

Time counter is now a static number. Make it a second counter.

Done, added.

Done, added.


Enable custom response headers in core

resolves #1946

Yep I'll thow in some tests for it & RawScriptResult + figure out a better name :)

I can cover the RawScriptResult tests so you don't have to expand the scope here (was already planning on it).


Fix for Issue #69 - network share case sensitivity on Windows

Implemented flag to allow toggling of forced lowercase file paths on Windows, for example when connected to a Samba network share via map/mount network drive with case sensitivity or through SSHFS such as Dokan.
Just add the following to your .vimrc to enable proper navigation of Samba and SSHFS shares
g:NERDTreeForceLowercase = 0

The problematic code was introduced in this commit 96215c5da6c01d1593eb35ad9ea714cb824ce1c6. I cant remember why - it's likely that someone using nerdtree on windows requested it due to some crazy stuff windows was doing with filename cases.
I have reworked the function so that it should preserve the old behaviour and satisfy your need without adding an extra option.
Unfortunately i dont have access to a windows box, so would you mind trying it out on your end? the replacement method is here: https://gist.github.com/1562739.
The whole point of the relative path code there is so that, if you have %f on your statusline and vims CWD is c:\foo\bar\ and you edit c:\foo\bar\baz\omg.c with nerdtree then it will issue the command :e baz\omg.c  rather than :e c:\foo\bar\baz\omg.c. Issuing the latter would cause the entire path to appear on the statusline rather than just the relative path.
Hope that makes sense.

Just tested, and it seems to work fine, thankyou :)


BitmapMask with Atlas

Version

Phaser Version: 3.17 & 3.18
Operating system: Windows 10
Browser: Chrome

Description
When you have two images as part of the same atlas and you attempt to add a bitmap mask from one image to the other, WEB_GL throws infinite errors and the images are not rendered.
Example Test Code
Paste the following here: https://labs.phaser.io/edit.html
var game = new Phaser.Game({
    type: Phaser.AUTO,
    parent: 'phaser-example',
    scene: {
        preload: preload,
        create: create
    }
});

function preload ()
{
    this.load.atlas('atlas', 'assets/atlas/megaset-1.png', 'assets/atlas/megaset-1.json');
}

function create ()
{
    let titan = this.add.image(400, 400, 'atlas', 'titan-mech');
    let ship = this.add.image(400, 400, 'atlas', 'ship');
    titan.mask = new Phaser.Display.Masks.BitmapMask(this, ship);
}

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


[5.1] stdlib: Add backward deployment versions for the

dynamic-replacement runtime functions.
The recent change of how we do dynamic replacements added 2 new runtime
functions. This patch adds those functions to the Compatibility50 static
archive.
This will allow backward deployment to a swift 5.0 runtime.
Patch by Erik Eckstein with a modification to call the standard
libraries implementation (marked as weak) when it is available.
This ensures we can change the implementation in the future and are not
ABI locked.
rdar://problem/51601233

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 95039481a8b068de6f8dfe5482c538574f905dda

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 95039481a8b068de6f8dfe5482c538574f905dda

Build failed
Swift Test Linux Platform
Git Sha - 95039481a8b068de6f8dfe5482c538574f905dda

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - ab72f855f79186f51db4d2e92cd62c187f4fe1fd

Build failed
Swift Test OS X Platform
Git Sha - ab72f855f79186f51db4d2e92cd62c187f4fe1fd

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - c418d7c25d0d44f8e35e60d20d2585492c7a25b5

Build failed
Swift Test OS X Platform
Git Sha - c418d7c25d0d44f8e35e60d20d2585492c7a25b5


How can I resize a cluster from D2sv3 and D4sv3?

Is there any way to increase the size of the cluster after a cluster has finished building?

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

ID: 0c1c2788-a209-6916-c7bf-fe8c6e0b6b8c
Version Independent ID: 832bd727-e03d-5f2a-93a9-04c9bfcb4f82
Content: Manage Apache Hadoop clusters in HDInsight using Azure portal
Content Source: articles/hdinsight/hdinsight-administer-use-portal-linux.md
Service: hdinsight
GitHub Login: @hrasheed-msft
Microsoft Alias: hrasheed

Hi @Anmolgan81
The VM sizes cannot be changed after the fact.  The Azure portal shows a lock icon on the VM sizes. The CLI and PowerShell scripts have a switch for the number of nodes (--target-instance-count), but not the VM size is not variable.
https://docs.microsoft.com/azure/hdinsight/hdinsight-scaling-best-practices

#please-close
Please let us know in case you have any further questions. We will close this issue now . Should you have any further comments, we will be happy to continue the conversation. Thanks, Jason


Node's Indexer does not resume indexing after stoping the node

Describe the bug
Node's Indexer does not resume indexing after stoping the node (with Ctrl + C or Cmd + C). The expected behaviour is to resume the last indexed block, not starting from deployer block. The issue is fixed if the startBlock is specified within RPCS env var, but it is not user intuitive/friendly to keep updating startBlock.
To Reproduce
Steps to reproduce the behavior:

Run Ocean Node on local with npm run start
Wait the node to sync
Stop the node
Restart node

Expected behavior
The expected behaviour is to resume the last indexed block, not starting from deployer block.
Logs
ocean-node-op-sepolia-indexer.txt

ocean-node container was started using docker compose
database type => typesense

DB_TYPE: 'typesense'
DB_URL: "http://typesense:8108/?apiKey=xyz"
....
typesense:
    image: typesense/typesense:26.0
    container_name: typesense
    ports:
      - "8108:8108"
    networks:
      - backend
    volumes:
      - typesense-data:/data
    command: '--data-dir /data --api-key=xyz'

It looks to me that we could not get the lastIndexedBlock from DB for some reason.. that's the only explanation i can see by looking at the code.. if for some reason we fail to get that info, it will start from either rpcDetails.startBlock or contractDeploymentBlock (whatever is more up to date), but that is expected, as we cannot guess the last indexed block..
I would say the problem here was at DB/connection level

It looks to me that we could not get the lastIndexedBlock from DB for some reason.. that's the only explanation i can see by looking at the code.. and the logs... The expected behaviour is Ok.
But if for some reason we fail to get that info, it will start from either rpcDetails.startBlock or contractDeploymentBlock (whatever is more up to date), but that is expected, as we cannot guess the last indexed block.. The crawlingStartBlock variable is calculated from these
I would say the problem here was at DB/connection level. Maybe we had no data there on the restart? We lost it?
@md00ux can we have the full logs?
We can easily improve the logging, so we know exactly what we are reading:

const lastIndexedBlock = await getLastIndexedBlock()

      const networkHeight = await getNetworkHeight(provider)

      const startBlock =

        lastIndexedBlock && lastIndexedBlock > crawlingStartBlock

          ? lastIndexedBlock

          : crawlingStartBlock

      INDEXER_LOGGER.logMessage(

        `network: ${rpcDetails.network} Start block ${startBlock} network height ${networkHeight}`,

        true

      )

We should add the lastIndexedBlock retrieved to the log message. That will clarify it

Hello @paulo-ocean!
Thank you for jumping on this issue.
Re-trying the same scenario today, and it couldn't be reproduced and maybe it was a database connection issue, as we discussed, when retrieving the last indexed block.
I have a suggestion regarding this issue, does it help to have a retry mechanism if db connection breaks or a known db error occurs to assert it. We can implement a while with number of retries set on 2 for example, because we do not want to loop too much for retry if db connection is down for good for example.
Also I am not sure if retry mechanisms are indicated for working with threads, but that section when retrieving the last indexed block is a safe-thread zone, so it shouldn't represent a problem.
Let me know what you think.

Hi @mariacarmina
Have a look at this PR, to imporve logging and a couple improvements on other functions related with updating/getting the block.
About the retry mechanism, we already have one before starting to index anything.. it only starts with a proper RPC and DB connection.. Apart from this, we also have a ReindexChain command (have another PR to improve that with an optional block number as well).
Now, If the DB connection is lost somewhere in the "middle" of the operations, the "retry" will automatically happen at the next run of the while() loop.. So, we already have that

I tested the same scenario and was not able to reproduce the behavior.

started the ocean-node and waited until the blockchain was fully indexed.
stopped the ocean-node container and started it again.

The logs show that indexing resumed from the last indexed block:
2024-10-30T07:48:19.794Z info: OCEANNODE:	🌊 [ Starting Ocean Node ]
2024-10-30T07:48:19.795Z info: CONFIG:	✅ INDEXER_NETWORKS is not defined, running Indexer with all supported networks defined in RPCS env variable ...
2024-10-30T07:48:19.809Z info: CONFIG:	✅ Starting node with peerID: 16Uiu2HAmMwDUBReP5EXXgB3cTU2SJkxW2LKMShog8N1UC7GRMXfk
2024-10-30T07:48:19.812Z warn: CONFIG:	⚠  Invalid or missing "AUTHORIZED_DECRYPTERS" env variable...
2024-10-30T07:48:19.816Z warn: CONFIG:	⚠  Invalid or missing "ALLOWED_VALIDATORS" env variable...
2024-10-30T07:48:19.818Z warn: CONFIG:	⚠  Invalid or missing "P2P_BOOTSTRAP_NODES" env variable...
2024-10-30T07:48:19.820Z warn: CONFIG:	⚠  Invalid or missing "P2P_ANNOUNCE_ADDRESSES" env variable...
2024-10-30T07:48:19.820Z warn: CONFIG:	Missing "FEE_AMOUNT" env variable. Will use defaults...
2024-10-30T07:48:19.823Z warn: CONFIG:	Missing "FEE_TOKENS" env variable. Will use defaults...
2024-10-30T07:48:19.825Z warn: CONFIG:	⚠  Invalid or missing "OPERATOR_SERVICE_URL" env variable...
2024-10-30T07:48:19.827Z warn: CONFIG:	⚠  Invalid or missing "ALLOWED_ADMINS" env variable...
2024-10-30T07:48:19.828Z warn: CONFIG:	Missing "MAX_REQ_PER_SECOND" env variable. Will use defaults...
2024-10-30T07:48:19.828Z warn: CONFIG:	⚠  Invalid or missing "RATE_DENY_LIST" env variable...
2024-10-30T07:48:19.828Z warn: CONFIG:	⚠  Invalid or missing "UNSAFE_URLS" env variable...
2024-10-30T07:48:21.283Z info: OCEANNODE:	Codebase hash: 2847dd48e570be7dabb33ff65171de11bdf22ed201b37bcb1ea549cf4cfbcd54
2024-10-30T07:48:21.283Z info: DATABASE:	Nonce Database initiated with SQLite provider
2024-10-30T07:48:21.298Z warn: DATABASE:	Property "LOG_DB" is set to "false". This means logs will NOT be saved to database!
2024-10-30T07:48:21.486Z info: P2P:	Binding P2P sockets to IPV4
2024-10-30T07:48:21.487Z info: P2P:	Binding P2P sockets to IPV6
2024-10-30T07:48:21.487Z info: P2P:	Enabling UPnp discovery
2024-10-30T07:48:21.488Z info: P2P:	Enabling AutoNat service
2024-10-30T07:48:21.488Z info: P2P:	Enabling P2P Transports: websockets, tcp, circuitRelay
2024-10-30T07:48:21.636Z info: OCEANNODE:	OCEANNODE:	HTTP port: 8000
2024-10-30T07:48:21.978Z info: INDEXER:	Blockchain connection succeffully established!
2024-10-30T07:48:21.983Z info: INDEXER:	Starting worker for network op-sepolia with {"rpcDetails":{"rpc":"<redacted>","chainId":11155420,"network":"op-sepolia","chunkSize":2000}}
(node:17) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
    at emitExperimentalWarning (node:internal/util:264:11)
    at ModuleLoader.jsonStrategy (node:internal/modules/esm/translators:456:3)
    at callTranslator (node:internal/modules/esm/loader:272:14)
    at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)
2024-10-30T07:48:24.028Z info: CONFIG:	✅ INDEXER_NETWORKS is not defined, running Indexer with all supported networks defined in RPCS env variable ...
secp256k1 unavailable, reverting to browser version
2024-10-30T07:48:25.274Z info: CONFIG:	✅ INDEXER_NETWORKS is not defined, running Indexer with all supported networks defined in RPCS env variable ...
2024-10-30T07:48:25.286Z info: DATABASE:	Nonce Database initiated with SQLite provider
2024-10-30T07:48:25.295Z warn: DATABASE:	Property "LOG_DB" is set to "false".


cancel show keyboard when favorite launched from home screen

Task/Issue URL: https://app.asana.com/0/414709148257752/1199613512009688
Tech Design URL:
CC:
Description:
Cancel showing keyboard when favorite launched from home screen.
Steps to test this PR:

Create a favorite
Enable "new tab" under Keyboard in settings
Use the fire button and wait
The keyboard should launch
Press cancel
Use the fire button but quickly tap the favorite
The page should load but the keyboard should not be launched

Internal references:
Software Engineering Expectations
Technical Design Template

As long as this feels like an improvement, we can tweak it more later.  :). Thanks @samsymons


Owls 95630 - WKO auxiliary image 4.0 runtime functionality

Changes for WKO 4.0 auxiliary image simplification runtime functionality.

Creates a single implicit Kubernetes EmptyDir volume.
The auxiliary images containing WDT install and/or WDT models are specified at spec.configuration.model.auxiliaryImages.
Domain validations and unit tests changes based on the new specifications.
Changed samples to use new simplified configurations.
Changes to add new integration tests and disable old integration tests which might be used to test the upgrade path.

Integration test runs - https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/8092/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/8097 (ItMiiAuxiliaryImage40)
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-kind-new/8096/ (ItMiiNewCreateAuxImage40)

Will there be documentation for usage or it will be on different PR

The documentation will be in a different PR.


TodoMVC: Short Connector explanation and link

Also see: #397

Thanks! I won't merge this for now, but I'll make sure to document this stuff in examples before the release. For now, I removed <Connector> in favor of connect() because <Connector> is deprecated: https://github.com/gaearon/redux/commit/c1b8acd3ccb84b5536aa54a4d2f11aec4e3c8ecc

We'll later make a pass through examples to add comments.
For now, I'm closing because this is out of date.


Add more guidance for how to refer to different elements

I don't think I see guidance for how to refer to a page that lists all resources.. is it a page? is it a pane?
Would be helpful if you also included these elements on this page

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

ID: 2b0dd435-a0fa-18b0-1dc0-9a7244f0b110
Version Independent ID: ca2b6ba7-74a7-fe4c-e87f-10ddd62434cb
Content: What is the Azure portal? - Azure portal
Content Source: articles/azure-portal/azure-portal-overview.md
Service: azure-portal
GitHub Login: @JnHs
Microsoft Alias: jenhayes

Hi @MashaMSFT - this article isn't meant to be a style guide covering which terms to use, but rather an explanation of the different elements. In general, "pane" is used for your examples (or as mentioned here, "working pane"), and "page" refers to the entire contents of the browser (for example "page header" mentioned here).
I'm going to close this issue as it seems more of a style guidance question, which is outside the scope of this article. Thanks!
#sign-off

#please-close


Highlight clickbox for Herbiboar

Is your feature request related to a problem? Please describe.
Been doing herbiboar for a few days now, and sometimes I forget to harvest the Herbiboar because I am braindead focussing on clickboxes while watching a tv show.
Describe the solution you'd like
A simple clickbox highlight when you are able to harvest the herbiboar would be awesome. Maybe even highlight the Herbiboar run after you harvested the Herbiboar.
Describe alternatives you've considered
There are none.
Additional context

I hope this explains it.

Duplicate

How is this duplicaten it isnt in the game

It's a duplicate because someone has already made an issue about it

It's a duplicate because someone has already made an issue about it.
#1899

Thank you, there is a good solution. Sorry, first time using github and couldn't find it


Missing requirements.txt

Seems like the requirements.txt is missing in the repo.

Thanks for catching it, just added it.


WM_CHAR event does not arrive in custom control

Hi all. I'm trying to create my own editor. In events fn I write
        self.wnd.on().wm_char(move |char| {
                    let sim = char.char_code as u8 as char;
                    println!("char: {}", sim);
                    Ok(())
        });

but this code is not executed! Why?

Actually there is a lot involved in WM_CHAR messages. As a general rule, they're dispatched as the result of WM_KEYDOWN being translated by TranslateMessage function.
It's hard to say without seeing the rest of your code, but try handling WM_KEYDOWN instead.

The test example is simple:
mod ed {
use winsafe::{self as ws, gui, co, prelude::*};
use ws::gui::{Horz, Vert};

#[derive(Clone)]
pub struct Editor {
    wnd: gui::WindowControl,
}

impl GuiWindow for Editor {
    fn as_any(&self) -> &dyn std::any::Any {
        self
    }
    fn hwnd(&self) -> &ws::HWND {
        self.wnd.hwnd()
    }
}

impl GuiChild for Editor {
    fn ctrl_id(&self) -> u16 {
        self.wnd.ctrl_id()
    }
}

impl Editor {
    pub fn new(parent: &impl GuiParent, position: (i32, i32), size: (u32, u32)) -> Self {
        let wnd = gui::WindowControl::new(
            parent,
            gui::WindowControlOpts {
                position,
                size,
                ex_style: gui::WindowControlOpts::default().ex_style | co::WS_EX::CLIENTEDGE,
                resize_behavior: (Horz::Resize, Vert::Resize),
                ..Default::default()
            },
        );

        let new_self = Self {
            wnd,
        };
        new_self.events();
        new_self
    }

//=========================================================
    fn events(&self) {
        self.wnd.on().wm_char(move |ch| {
            let sim = ch.char_code as u8 as char;
            println!("sys char: {}", sim);
            Ok(())
        });

        self.wnd.on().wm_sys_char(move |ch| {
            let sim = ch.char_code as u8 as char;
            println!("sys char: {}", sim);
            Ok(())
        });

        self.wnd.on().wm_paint({
            let self2 = self.clone();
            move || {
                let hdc = self2.wnd.hwnd().BeginPaint()?;
                let i = 3;
                hdc.TextOut(10, 10 + 20 * i as i32, "Test WM_CHAR")?;
                Ok(())
            }
        })
    }
}
}

mod wi {
    use w::co;
    use winsafe::{self as w, gui, prelude::*};

    use crate::ed::Editor;

    #[derive(Clone)]
    pub struct MainWin {
        wnd: gui::WindowMain,
        ed: Editor,
    }

    impl MainWin {
        pub fn new() -> Self {
            let wnd = gui::WindowMain::new(
                // create the container window
                gui::WindowMainOpts {
                    title: "Native controls".to_owned(),
                    size: (580, 240),
                    style: gui::WindowMainOpts::default().style
                        | co::WS::MINIMIZEBOX
                        | co::WS::MAXIMIZEBOX
                        | co::WS::SIZEBOX, // window can be resized
                    ..Default::default()
                },
            );

            let editor = Editor::new(&wnd, (10, 10), (560, 210));

            let new_self = Self {
                wnd,
                ed: editor,
            };

            new_self.events();
            new_self
        }

        pub fn run(&self) -> w::AnyResult<i32> {
            self.wnd.run_main(None)
        }

        fn events(&self) {
            self.wnd.on().wm_char(move |ch|{
                let sim = ch.char_code as u8 as char;
                eprintln!("sys char: {}", sim);
                Ok(())
            })
        }
    }
}

use wi::MainWin;
use winsafe::{self as w, co, prelude::*};

fn main() {
    if let Err(e) = (|| MainWin::new().run())() {
        w::HWND::NULL
            .MessageBox(&e.to_string(), "Uncaught error", co::MB::ICONERROR)
            .unwrap();
    }
}

or even simpler
use winsafe::gui;
use winsafe::prelude::*;

fn main() {
    let window = gui::WindowMain::new(gui::WindowMainOpts {
        title: "test1".into(),
        ..Default::default()
    });
    window.on().wm_create(|_| {
        println!("message");
        Ok(0)
    });
    window.on().wm_char(|ch| {
        println!("char {}", ch.char_code);
        Ok(())
    });
    let _ = window.run_main(None);
}

Does this work for you?
window.on().wm_key_down(|p| {
	println!("vk {}", p.vkey_code);
	Ok(())
});

Not satisfied. The editor is created for multilingual input. WM_CHAR already receives a letter processed depending on the language layout. But in your version comes the keyboard code.

I added a flag to suppress the IsDialogMessage call in the window loop, this should allow WM_CHAR messages to be dispatched to the window procedure:
let window = gui::WindowMain::new(gui::WindowMainOpts {
    process_dlg_msgs: false, // suppress IsDialogMessage call
    ..Default::default()
});

Let me know if this works for you.

Thanks alot. All work perfectly!


Use naive diff for overlay2 when opaque copy up bug present

- What I did
Update overlay2 driver to use naive diff when kernel has bug which copies up opaque directory flag
- How I did it
Added a check on kernels older than 4.8 for the bug.
- How to verify it
Run this Dockerfile on kernel without patched fix and should work
FROM alpine

RUN mkdir /dir1 && touch /dir1/f1
RUN rm -rf /dir1 && mkdir /dir1 && touch /dir1/f2
RUN touch /dir1/f3
RUN [ -f /dir1/f2 ]

Closes #25244
... for all kernels which support overlay2
Ping @tonistiigi

@AkihiroSuda added build tag, updated the error to mention updating to fix, and also added to the status output (if this merges first you will need to rebase your other PR).

@dmcgowan Need to mark that test for linux as well.

LGTM

LGTM

Am I correct to read that this patch means that for kernels without backports yet (RHEL 7), overlay2 will now have same issues with inode exhaustion as overlay in 1.13?
If so, the documentation (https://docs.docker.com/engine/userguide/storagedriver/overlayfs-driver/) should be updated to reflect that the overlay2 benefits do not apply to kernels older than 4.8 without backports, even though it will work.

@tristanz No, this only changes the diff algorithm, making docker commit bit slower. The optimizations from on disk layout still apply.

@tristanz the backport does not impact storage for overlay2. The only potential impact is running into #21555.

@tonistiigi @dmcgowan thanks!   Reading #21555 it looks like this is no longer critical bug after #27209 mitigation.  With this PR and #27209 are any known major issues with overlay2 on RHEL 7.2+ beyond SELinux not working yet?


ha addons info command from example does not work

Describe the bug
Trying to get the CLI to list me information on all addons and it is requiring I provide a slug for a specific addon. When you use ha addons -h it lists an example of ha addons info as a way to do this but when you actually try to use that command you get an error.  See below:

Debug info

SSH Addon type:

[x] Community
[ ] Core
[ ] Custom

Version of the CLI:
I'm running version 7.3.0 of the SSH & Web Terminal community addon, I'm not sure how to determine what CLI version that uses.

Version of Home Assistant Core & the Home Assistant Supervisor
HA Core is 107.6
Supervisor is 209

Output from executing the command with debug info.
e.g. ha hardware info should be run as ha -d hardware info

Expected behavior
I expect the command ha addons info to return me the list of addons and information on them such as slug, installed version, latest version, etc.
Additional context
When you run the command ha addons by itself it seems to output the info I would expect to come from ha addons info. So this may just be a documentation bug showing a bad example. Although I noticed that the command ha addons only seems to work when ssh'ed into HA, if I try to use by passing it in like ssh <HA Username>@<HA IP> 'ha addons' I receive a response like this:
time="2020-03-26T12:06:01-04:00" level=error msg="Unexpected server response"
Unexpected server response

None of the other HA CLI commands seem to run into this issue where they can't be passed into SSH that I tried.

ha addons shows the list you are looking for.
ha addons info should return the information on the current add-on where this command is executed (defaulting to self), this is not working indeed and a bug that needs to be addressed.
I'm not sure where you found -d, that is not a valid global flag, which is not listed either. --log-level debug however, is a valid global flag.

Ah ok thanks. For the -d thing, When you make a new issue in this repo the issue template tells you to do that.  Or at least that's how I read it, it seemed like the instructions were to run your command with -d so I was just trying to follow them.  Here is the new issue template for reference:
**Describe the bug**

A clear and concise description of what the bug is.

**Debug info**

1. SSH Addon type:
- [ ] Community
- [ ] Core
- [ ] Custom

2. Version of the CLI:

3. Version of Home Assistant Core & the Home Assistant Supervisor

4. Output from executing the command with debug info.
e.g. `ha hardware info` should be run as `ha -d hardware info`

**Expected behavior**
A clear and concise description of what you expected to happen.

**Additional context**
Add any other context about the problem here.
e.g. If you are running `ha core check` and the config is known to be broken or good.

Aah, that is a leftover of the previous (Bash based version). I'll update that issue template as well (man, we keep finding history 😓)


Pause Menu: Version Info

This is a pretty simple HUD element that just displays some info about the game and how it's running.
Something like this in the bottom right corner would work:
P2CE - Alpha <Version str>
<Graphics API>
<Physics Engine> Physics

Relies on #44

Could have this show the map name too when the console is open, I do find it annoying that that no longer appears in the top right.


Update dbt_project.yml to correct version

resolves #
This is a:

[x] documentation update
[x] bug fix with no breaking changes
[ ] new functionality
[ ] a breaking change

Description & motivation
The dbt_project.yml fell out of sync with the actual released version, this gets them back together. Fixes #152
Checklist

[x] This code is associated with an issue which has been triaged and accepted for development.
[x] I have verified that these changes work locally

Talked with @joellabes -- because this has no bearing on functionality, it's better to leave this un-updated than arbitrarily try to keep it in sync. We may revisit in the future.


Click offcanvas-outer to hide.

Can't get my head around this:
$($el).on('shown.offcanvas', function() {
    $('.offcanvas-outer').on('click', function() {
      $el.offcanvas('hide');
    });
});

Once hidden the trigger doesn't work correctly.

You can achieve this by setting overlay: true and overlayColor: transparent. The overlay spans over the whole content and closes the offcanvas part on click.  The transparent makes it invisible.

Yeah I tried that, unfortunately overlay: true is giving me performance issues.
I'm using esc in the meantime.
$($el).on("shown.offcanvas", function() {
    $(document).keyup(function(e) {
      if( e.keyCode === 27 ) {
        $el.offcanvas("hide");
      }
    });
});

Thanks anyways.


Rails.application.routes.recognize_path and authenticated route

Hello,
I am trying to use Rails.application.routes.recognize_path on a Paper_trail routes.rb authenticated path (authenticate :user do in config/routes.rb) but I get an exception.
in config/routes.rb :
authenticate :user do
  resources :authenticated_ressources
end

If I call
Rails.application.routes.recognize_path('/authenticated_ressources/')

I get
NoMethodError: undefined method `authenticate!' for nil:NilClass
	from ruby-2.2.1/gems/devise-3.4.1/lib/devise/rails/routes.rb:459:in `block in constraints_for'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/routing/mapper.rb:41:in `call'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/routing/mapper.rb:41:in `block in matches?'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/routing/mapper.rb:39:in `each'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/routing/mapper.rb:39:in `all?'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/routing/mapper.rb:39:in `matches?'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/routing/route_set.rb:828:in `block in recognize_path'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/journey/router.rb:65:in `block in recognize'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/journey/router.rb:59:in `each'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/journey/router.rb:59:in `recognize'
	from ruby-2.2.1/gems/actionpack-4.2.0/lib/action_dispatch/routing/route_set.rb:817:in `recognize_path'
	from (irb):22
	from ruby-2.2.1/gems/railties-4.2.0/lib/rails/commands/console.rb:110:in `start'
	from ruby-2.2.1/gems/railties-4.2.0/lib/rails/commands/console.rb:9:in `start'
	from ruby-2.2.1/gems/railties-4.2.0/lib/rails/commands/commands_tasks.rb:68:in `console'
	from ruby-2.2.1/gems/railties-4.2.0/lib/rails/commands/commands_tasks.rb:39:in `run_command!'
	from ruby-2.2.1/gems/railties-4.2.0/lib/rails/commands.rb:17:in `<top (required)>'
	from script/rails:6:in `require'
	from script/rails:6:in `

Non authenticated routes work well :
Rails.application.routes.recognize_path('/')

returns
{:controller=>"home", :action=>"index"}

My basic problem is I would like to know if two users are currently watching the same page. (same controller, same action and same id). I am currently saving a :last_url adress on each user.
I could compare the strings but the url may have differents GET arguments.
Any idea how to use recognize_path without devise interfering ?
Thanks !

This doesn't appear to be related to PaperTrail at all, so I advise you to direct your question to the Devise repository's issue tracker.

Hello,
I'm really sorry, don't know exactly how I managed that (worked on both Gems at the same time and got confused I guess)
Thanks anyway for your answer and sorry for bothering you !


swift4: Integer fixes for new integer protocols

Bitshift operators were moved from FixedWidthInteger to
BinaryInteger making the _BitShiftable and _IntegerLike
protocols incorrect.

Remove _BitShiftable and _IntegerLike and just use
FixedWidthInteger for constraining the return type
of scan() and scanHex().

Tested on swift 3 and 4

@parkera This tested ok but it needs checking to make sure substituting FixedWidthInteger keeps the same meaning as the two protocols that were removed.

@moiseev can you double check this for us?

LGTM! Thanks @spevans!

This seems ready for merge
@swift-ci please test and merge


add next-dev internal-package

Idea for supporting bindings mocking in next dev using miniflare's new magic proxy API.
Usage
(more details are in the new module's README)
In the next.config.js just add the following:
 /** @type {import('next').NextConfig} */
 const nextConfig = {}

+ const { setupDevBindings } = require('@cloudflare/next-on-pages/next-dev');

+ setupDevBindings({
+     kvNamespaces: [...],
+     durableObjects: {MY_DO: {className: 'MyDo', scriptName: ''}},
+     // ...r2 and d1
+ });

 module.exports = nextConfig

and then running npm run dev (a.k.a. next dev) will populate the process.env with the bindings defined in the config file
Notes

No runtime impact, no extra code at all makes it into the bundled worker
next dev gets the binding mocks but it's still important to note that it is still not very representative of production, so it's still advised to check things with wrangler pages dev before deploying
all the next dev goodness is present, including fast reloading and HMR 🚀

Small very minimal demos/examples that use this new implementation:

https://github.com/dario-piotrowicz/next-on-pages-service-local-demo
https://github.com/dario-piotrowicz/next-on-pages-DO-local-demo
https://github.com/dario-piotrowicz/next-on-pages-R2-local-demo

TODO

[ ] add support for text bindings

resolves #271

Hello @dario-piotrowicz,
Thank you for your work on this, which is going to make my development workflow 100x better. I have been testing this out in one of my projects to mock D1 and I have noticed that the mock database file has a different hash to that of the main wrangler commands.
This causes issues as when I run my migrations via wrangler d1 migrations apply bucket --local they do not apply to the same database as is used via this mock (I have one at e7352547963de7050bd7d94658afc4fe78b61811b7815da12d90be8e863abf4d.sqlite and one at d4f889bae404efba2a3527fb440041c852e355c21bf33365fdd1939d27442be8.sqlite in the same .wrangler state folder). Do you know why this could be and if there is a workaround to have wrangler use the same database file as the one your mock uses?
Cheers!

Hi @millar, first of all thanks so very much for trying this out! I hope it's been working nicely for you! 😄
Regarding the D1 db, my mocks use miniflare under the hood for them which is what also wrangler uses, so the two should have the same exact results.... so I don't really know what you're experiencing that issue, maybe this depends on some options set for miniflare or something like that... I'll have to have a look
Anyways just to be sure, did you check what happens when you run your application with wrangler pages dev? (if you didn't could you please? 🙏) is the d1 db recognized in that case?

@dario-piotrowicz Thanks a lot for this. Local version works fine but having issues with deploying to Cloudflare pages. I'm using bun and it's throwing install errors.

@dario-piotrowicz Thanks a lot for this. Local version works fine but having issues with deploying to Cloudflare pages. I'm using bun and it's throwing install errors.

I'm not sure what the issue is... does bun support installs from non-registry sources such as our prerelease links? (if it works locally I guess it does? maybe only in some OSes/versions or something? 😕:thinking:)
I'll try to look into it but I guess it could be a bug in bun itself? in the meantime, when you have to use prereleases maybe you could switch to npx <cloudflare_prerelease> (just in the Pages integration)?

Plus, queue binding will be a great addition to this.

Thanks for the callout! I'll look into adding queues as well 🙂👍

@dario-piotrowicz Thanks a lot for this. Local version works fine but having issues with deploying to Cloudflare pages. I'm using bun and it's throwing install errors.

Bun has had various issues with our prerelease registry links for many months sadly.
bun x / bunx straight up just doesn't work - https://github.com/oven-sh/bun/issues/3675

As @james-elicx mentioned, it looks like this issue is with bun itself.
What's weird is that it works perfectly fine on local machine but fails while deploying to pages. As per the suggestion by @dario-piotrowicz I've modified the pre-build script for pages deployment and it's now using npm for deploying to Cloudflare pages. Here's what I've done

Set environment variables (For both preview & production)

SKIP_DEPENDENCY_INSTALL=true
UNSTABLE_PRE_BUILD=rm -rf bun.lockb && npm install

Update build script to use npm

npx @cloudflare/next-on-pages@1

While this works for the deployment, it strips away the package installation consistency offered by lock files. So, the deployed app is prone to bugs introduced by patches or minor versions of dependencies. And next.js sometimes breaks things even in the patches.
A better solution is to deploy the pre-release versions to npm registry so that such annoying tooling issues won't occur. Additionally, I'm having trouble finding the pre-release versions of next-on-pages. It being deployed to npm would allow devs to browse through versions and look for changes without installing the pre version.

Prereleases are something we added to help testing changes, they were never added with the intention of being used for any non-minimal period of time. I don't think publish them to npm would make too much sense, please keep in mind that each every commit on an opened PR "generates" a new prerelease, so if we were to publish them in no time we'd have thousand of useless prerelease versions there (each formatting, comment change, etc... on an open PR would generate one).
What you mentioned as pre version is I think more or less what we call beta, as soon as a PR is merged we indeed publish it to npm so it is usable from there (again, if pre would point to the latest prerelease, what would happen when there are multiple PRs opened at the same time? with each new commit pushed each would become the predominant pre? wouldn't that be extremely confusing?)
Also please note that usually we do merge PRs rather quickly so the need to rely on a prerelease is quite rare I believe (and they are used as I mentioned above just to check things), this one has been a bit different as it is a significant addition and I didn't want to rush it in (potentially getting the API wrong). Sorry about that.

PS: additional personal point of view 😅, this also does feel like a bug in bun to me, if that's the case then instead of us changing thing to accommodate bun's shortcomings the fix should happen in bun itself (unless they clearly state that this is not a bug and that they do intend only to support the standard npm registry and nothing else 🤷)

What's weird is that it works perfectly fine on local machine but fails while deploying to pages. As per the suggestion by dario-piotrowicz I've modified the pre-build script for pages deployment and it's now using npm for deploying to Cloudflare pages. Here's what I've done

Set environment variables (For both preview & production)

SKIP_DEPENDENCY_INSTALL=true
UNSTABLE_PRE_BUILD=rm -rf bun.lockb && npm install

@iamvishnusankar you could just leave your package.json as normal with next-on-pages pointing to the latest version. There's no need to use two different ways of installing dependencies for local vs CI... Dario's suggestion was to just use npx https://...... for your build command in your Pages project settings, not change everything to npm install in CI :P
If you want to use the same build command between the two, you could use a package.json script that runs the npx command and use that instead for your build command.
Before we merged our Bun PR, I would have my build command on Pages to use npx https://.... while the Pages CI used Bun for everything.

@james-elicx It seems like you misunderstood this situation here.!
The point of using this pre-release version is to get access to setupDevBindings. Which is needed for local development. And as per the examples provided by @dario-piotrowicz , the local package.json needs to be pointed to pre-release version.
It works fine locally, but when being deployed to Cloudflare pages, the CI will auto detect the bun.lock file and will try to auto install dependencies including the pre-release version. And it results in the error I shared in first comment.
To prevent this auto install, I use SKIP_DEPENDENCY_INSTALL and UNSTABLE_PRE_BUILD steps. And I'm using the latest stable version of  @cloudflare/next-on-pages@1 as the pre-release has no relevance in building the project.

@james-elicx It seems like you misunderstood this situation here.!
The point of using this pre-release version is to get access to setupDevBindings. Which is needed for local development. And as per the examples provided by @dario-piotrowicz , the local package.json needs to be pointed to pre-release version.
It works fine locally, but when being deployed to Cloudflare pages, the CI will auto detect the bun.lock file and will try to auto install dependencies including the pre-release version. And it results in the error I shared in first comment.
To prevent this auto install, I use SKIP_DEPENDENCY_INSTALL and UNSTABLE_PRE_BUILD steps. And I'm using the latest stable version of @cloudflare/next-on-pages@1 as build command as the pre-release has no relevance in building the project.

Oh, derp, you're right, I did misunderstand what you were referring to - totally slipped my mind that you can't use the import if it's not the one in your package.json. My apologies.

What you mentioned as pre version is I think more or less what we call beta, as soon as a PR is merged we indeed publish it to npm so it is usable from there (again, if pre would point to the latest prerelease, what would happen when there are multiple PRs opened at the same time? with each new commit pushed each would become the predominant pre? wouldn't that be extremely confusing?)

@dario-piotrowicz No need to push every PR to npm. May be you could setup an action to pre-release the packages when you create a GitHub pre-release with pre-* tag. This will help to selectively publish the pre-release version to npm and devs like me won't have to wait for the PR to get merged to test things out.

@iamvishnusankar I've checked with the team regarding the queue bindings, we agreed that we can just merge this PR without them and add them as a followup (since it's a simple additive change anyways) 🙂

@dario-piotrowicz Hi, any update on this? or a possible experimental/pre version release?

@iamvishnusankar so sorry for this taking a while, the changes are pretty much ready, I just need a review from the team, I'll ping them right now sorry about that! I will try to get this merged by the end of the week! 🙇 🙏

@iamvishnusankar PR merged 😄, now you can use this with the latest beta release 🙂
But by the way, so you know, I think the API of how you pass the bindings might still change before we do a proper release (so you might need to slightly tweak that later on)


Optimize routes in declarative way

Now the route in the mvp dapp is declared in an imperative way(https://github.com/ckb-js/kuai/blob/91a5ad378b47a420970fde6e86e0b7f2ddf72bda/packages/samples/mvp-dapp/src/app.controller.ts#L89-L92), which is not easy to read and maintain
It could be adjusted in a declarative way, as nest.js did(https://docs.nestjs.com/controllers#routing). This optimization could be implemented in IO layers

how is it going

PR: https://github.com/ckb-js/kuai/pull/190

PR: https://github.com/ckb-js/kuai/pull/191


Catching all assignments

Hi. I want to capture all the assignments in a source code file, which I am doing with
for path, node in tree.filter(javalang.tree.Assignment):
  assign +=1
, but the final result is not the number of assignments I have in the code. A look at the Java specification from the Oracle website doesn't make it clear which Node type should I try to catch for a 'generic' assignment.
Great work with this parser.
Thanks

The code you've given should find assignment expressions, but won't find variable (or field) initializers. For those you can try looking for VariableDeclarator. If this doesn't explain the discrepancy could your provide a code sample which reproduces the issue?

Sorry for the late reply and thanks for your help. I did not immediately realize that variable declarations were a specific tree node and was trying to catch everything through assignments.


Fix build process to not install go inside of sandbox

https://github.com/criblio/appscope/pull/24 mentions reverting some of my changes.
This issue was created to hopefully resolve issues which were seen on that branch (bugs/bugathon-1).

Verified.

Verified.


README: fix typo

What problem does this PR solve?
Fix number of CRD objects.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #620 into master will decrease coverage by 0.17%.
The diff coverage is 21.05%.

@@            Coverage Diff             @@
##           master     #620      +/-   ##
==========================================
- Coverage   55.78%   55.61%   -0.18%
==========================================
  Files          68       68
  Lines        4383     4393      +10
==========================================
- Hits         2445     2443       -2
- Misses       1768     1779      +11
- Partials      170      171       +1

Impacted Files
Coverage Δ

controllers/networkchaos/ipset/ipset.go
37.14% <0.00%> (ø)

pkg/store/experiment/experiment.go
32.20% <0.00%> (-6.58%)
:arrow_down:

pkg/utils/chaosdaemon.go
51.21% <0.00%> (ø)

pkg/utils/grpc.go
0.00% <0.00%> (ø)

controllers/networkchaos/netem/types.go
47.27% <50.00%> (ø)

controllers/podchaos/containerkill/types.go
64.38% <100.00%> (ø)

controllers/timechaos/types.go
61.44% <100.00%> (ø)

pkg/utils/selector.go
51.42% <0.00%> (-0.82%)
:arrow_down:

Continue to review full report at Codecov.

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

@l2dy Thanks for your contribution! Please sign CLA, thanks!


regression check tpch sf1T and sf500 plan shape on 3 BE environment

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Scope GHA runs to relevant changes

Issue summary

Presently a number of checks run on EVERY pull request. These could be scoped to relevant code changes to reduce the consumption of runner minutes.
This example removes a GHA check and does not modify any files:

Some of these could be scoped to changes to any Rust code files, or in the case of documentation the relevant directory.
https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/triggering-a-workflow#using-filters-to-target-specific-paths-for-pull-request-or-push-events
Other information and links

I agree some of those can be better filtered, though I'd like to emphasize that one needs to be careful there.
Rust workflows depend on non-Rust elements - Makefiles, artefacts in the repository, even Go files, shell scripts et cetera. At one point, they even depended on markdown files (I believe someone was checking if the README files can be served by a fake server or something like this). We need to check if this is not the case anymore, otherwise a deletion of such files might trigger failure of consecutive PRs.


Broken link

The link on https://github.com/textlint/markdown-to-ast is broken
It should instead point to https://github.com/textlint/textlint/tree/master/packages/%40textlint/markdown-to-ast

Can you submit a Pull Request to fix this issue?

Unfortunately I can't because the repo has been archived. It would need to be temporarily unarchived, updated, and then rearchived, which means only an admin can do it

Ah, I misunderstood this issue.
I've fixed the link on https://github.com/textlint/markdown-to-ast
Thanks for report!


https://iabgpp.com out of date?

Hi all,
Would you mind clarifying whether https://iabgpp.com is considered as up-to-date?
If yes, should we report bugs on the present issue tracker?
If no, maybe better disable it or at least add a warning banner? Some companies are using it and getting into issues.
Thanks!

Details:

IAB Europe TCF strings generated by that tool mislead publishers, as they use a global scope, which is invalid under the TCF policy version used.
The Allowed Vendors segment and the Disclosed Vendors segment must be removed, and the Core segment's isServiceSpecific flag must not be set to false.
The tool should probably not even include an IAB Europe TCF 2.x string, as those won't be handled through GPP.

@iabmayank : Please kindly take a look at this issue.
Thank you.

Hi @IABTechLab,
Did you get a chance to look into this?
You could help CMP developers greatly by either fixing the tool or clearly deprecating it.
Thanks for your consideration!

Currently the IAB EU TCF is not supported under GPP. You have to write a standalone TCF string to be compliant.

@HeinzBaumann Thank you for your response.
I believe @nicolas-raoul is referring to the TCF 2.2 string itself; but the iabgpp.com or iabtcf.com decoders/encoders are not raising any warning or error when decoding a TCF 2.2 string which contains the deprecated fields like Allowed Vendors, Disclosed Vendors, isServiceSpecific flag must not be set to false.
Can we expect any fix from the IAB end for this decoder tools so that the deprecated/incorrect fields are validated and reported in their UI accordingly.


[Feature Request] - Run script on directly on Insomnia

Hello. Is this possible for a feature that you could run script on insomia? Example.
I have to test something at work where I need to notify my fake user account. so i have this body.
{
  "id": "150129",
  "workstep_final": "Completed"
}

i need to change the id for every request. Since I have a hundred sample ID's I was thinking of putting them in an array and make a simple script that automates my request. sorry if im not being clear.

This is possible with the Chance plugin.
Install the plugin and use the template tag with "Pick One". Edit the tag an enter the values as an array under Options:

Your body in the editor should look like this:

Once you make a request a random item from the array is picked. You will be able to see the value in the timeline:

thank you so much!!! I was thinking of doing a bash script and curl for http. thank you